Skip to content

replace local paths with OSF#147

Open
tamsinrogers wants to merge 6 commits into
enh/remote-datafrom
osf-path-setup
Open

replace local paths with OSF#147
tamsinrogers wants to merge 6 commits into
enh/remote-datafrom
osf-path-setup

Conversation

@tamsinrogers
Copy link
Copy Markdown
Collaborator

@kaitj I'm not sure this is exactly correct, trying to follow along with #142. Let me know if the paths should be structured differently!

@tamsinrogers tamsinrogers changed the title replace local paths wit OSF replace local paths with OSF May 21, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 21, 2026

Coverage

Coverage Report
FileStmtsMissCoverMissing
neuromaps_prime
   __init__.py00100% 
   fetcher.py190100% 
   niwrap.py560100% 
neuromaps_prime/graph
   __init__.py80100% 
   builder.py112298%329, 397
   cache.py1130100% 
   core.py920100% 
   models.py840100% 
   utils.py530100% 
neuromaps_prime/graph/transforms
   __init__.py00100% 
   surface.py970100% 
   volume.py400100% 
neuromaps_prime/remote
   __init__.py20100% 
   osf.py310100% 
neuromaps_prime/resources
   __init__.py120100% 
neuromaps_prime/transforms
   __init__.py00100% 
   surface.py350100% 
   utils.py290100% 
   volume.py190100% 
TOTAL802299% 

Tests Skipped Failures Errors Time
238 2 💤 0 ❌ 0 🔥 12.876s ⏱️

@tamsinrogers tamsinrogers marked this pull request as ready for review May 21, 2026 20:38
@tamsinrogers tamsinrogers requested a review from kaitj May 21, 2026 20:38
@kaitj
Copy link
Copy Markdown
Contributor

kaitj commented May 21, 2026

@kaitj I'm not sure this is exactly correct, trying to follow along with #142. Let me know if the paths should be structured differently!

It's not quite right. You'll have to use the view only link (passing the token) and grab it via the osf api. I had an example of this somewhere at one point, I'll see if I can find it and send it to you!

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.

2 participants