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

Files with spaces in names from Biomodels repos get copied into workspace with + instead of space #943

Open
pgleeson opened this issue Feb 3, 2025 · 3 comments
Assignees
Milestone

Comments

@pgleeson
Copy link
Member

pgleeson commented Feb 3, 2025

e.g. try https://v2.opensourcebrain.org/repositories/3325

Files look fine in repo view:

Image

but when copied into workspace:

Image
@pgleeson pgleeson added this to the Urgent milestone Feb 3, 2025
@pgleeson
Copy link
Member Author

pgleeson commented Feb 3, 2025

This doesn't seem to be an issue with GitHub based repos, i.e. spaces in filenames are preserved fine.

@sanjayankur31 do you know where in the code the file copy happens?

@sanjayankur31
Copy link
Collaborator

It's done here:

I don't see us doing any replacements in the script there ourselves, so it's possible that the biomodels API perhaps returns the modified file names. I'll check tomorrow.

@sanjayankur31
Copy link
Collaborator

This is something server side. For example, if one tries this URL to download a file:

https://www.ebi.ac.uk/biomodels/model/download/BIOMD0000001031?filename=Al-Tuwairqi2020 - Phase I virotherapy.xml

you'll get the file with the spaces replaced by "+". Since this is how biomodels wants to do it, we should probably leave it as is.

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

No branches or pull requests

2 participants