Skip to content

Conversation

@dheshanm
Copy link
Member

No description provided.

dheshanm added 2 commits July 23, 2024 09:32
Signed-off-by: Dheshan Mohandass <[email protected]>
@dheshanm dheshanm requested a review from kcho July 23, 2024 15:32
@dheshanm dheshanm self-assigned this Jul 23, 2024
@dheshanm
Copy link
Member Author

Proposed Schema:

IMG_50F1E610899B-1

Comments from Kevin:

  1. Store history of file_path for local PHOENIX files Table
  2. Add modality variable to file_mapping? eg) phone, interview, mri, eeg and others to the table?
  3. Consider adding ID of RA who added each remote_file. How about we add more info to Remote_file table eg) owner, modifier, last_modified ?

Copy link
Member

@kcho kcho left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the commits! They look great. I'll see what I can do in sqlalchemy. I don't see the crawlers to add records to the db. Are we going to review the db models first before we work on crawlers?

dheshanm and others added 15 commits July 30, 2024 01:16
Signed-off-by: Dheshan Mohandass <[email protected]>
Signed-off-by: Dheshan Mohandass <[email protected]>
Signed-off-by: Dheshan Mohandass <[email protected]>
Signed-off-by: Dheshan Mohandass <[email protected]>
Signed-off-by: Dheshan Mohandass <[email protected]>
Signed-off-by: Dheshan Mohandass <[email protected]>
Signed-off-by: Dheshan Mohandass <[email protected]>
Signed-off-by: Dheshan Mohandass <[email protected]>
Signed-off-by: Dheshan Mohandass <[email protected]>
Signed-off-by: Dheshan Mohandass <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants