Fixes to updatepetjson and ecatheaderupdate
#369
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Running
updatepetjsongives:And running
ecatheaderupdategives:These errors can be fixed by the changes to
pyproject.tomlBut then, trying to run
updatepetjsongives the following error:This is due to a circular dependency:
update_json_pet_file.pyimports fromis_pet.pyis_pet.pyimportsecat.pyecat.pyimportsdcm2niix4pet.pydcm2niix4pet.pyimports fromupdate_json_pet_file.pyDeferring the import of
is_petto the function where it's used,get_metadata_from_spreadsheet()resolves this.Running
ecatheaderupdategives:Fixing the import statement to import the modules themselves instead of objetcs from modules resolves this.
📚 Documentation preview 📚: https://pet2bids--369.org.readthedocs.build/en/369/