Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature/initial #1

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Feature/initial #1

wants to merge 3 commits into from

Conversation

bwalsh
Copy link
Collaborator

@bwalsh bwalsh commented Apr 8, 2023

This PR creates a bare bones FHIR ResearchStudy with associated DocumentReferences. By design, no other FHIR resources are created.

@bwalsh
Copy link
Collaborator Author

bwalsh commented Apr 8, 2023

@kellrott : Can we run this on exacloud to retrieve MCF10A files?

@kellrott
Copy link

kellrott commented Apr 12, 2023

@kellrott : Can we run this on exacloud to retrieve MCF10A files?

This would be for the nvidia files

@kellrott kellrott closed this Apr 12, 2023
@kellrott kellrott reopened this Apr 12, 2023
@bwalsh
Copy link
Collaborator Author

bwalsh commented Apr 12, 2023

@jordan2lee Can we run this on exacloud to retrieve NVIDIA files?

@bwalsh bwalsh requested review from jordan2lee and removed request for kellrott April 12, 2023 21:22
@jordan2lee
Copy link

jordan2lee commented Apr 12, 2023

looks like requires Python version >= 3.9, <4

@bwalsh is this key to lock this python version? or written with this version but other versions are ok?

@jordan2lee
Copy link

Talked with team and confirmed that those python versions were locked intentionally. Running on python 3.9

On Exacloud, returns error about missing _ctypes when running setup up step pip install -e .

from _ctypes import Union, Structure, Array
      ModuleNotFoundError: No module named '_ctypes'

@jordan2lee
Copy link

Followup, checked if downstream steps ran, and it fails on the test step

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.

4 participants