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

Testing multi-OS URL enhancement. #82

Open
DavidBrainard opened this issue Jan 28, 2018 · 2 comments
Open

Testing multi-OS URL enhancement. #82

DavidBrainard opened this issue Jan 28, 2018 · 2 comments

Comments

@DavidBrainard
Copy link
Collaborator

I added a feature that you can specify a URL as an array of strings, with an even number of of entries. In this case, the odd entries specify strings that might be returned by Matlab's computer command, while even number strings specify the corresponding URL. Thus

"url": [
"MACI64"
"https://stanford.box.com/shared/static/j89elq3upt0ojxaju4cxwyft6heuiwnk.zip"
"GLNXA64"
"https://stanford.box.com/shared/static/wnrmyhud3og6zcnevmpf2147e5trza7x.zip"
]
will fetch from different URLs in an OS-dependent manner.

This only works in 2016b or later.

I tested this feature for the one "webget" case I needed it for. But the processing of this format occurs early in the TbTb pipeline and should work for any TbTb strategy.

This feature is not well tested, so I am posting this issue just to note that it might be broken, or break working code.

The most likely boo-boo is that I did not defend sufficiently against the fact that versions of Matlab before 2016b do not contain the isstring function. But I tried to, and the validations all pass under 2015b.

Post here if it seems like this enhancement broke something, or doesn't work as expected.

@gkaguirre
Copy link

Excellent! Going to give this a try now...

@DavidBrainard
Copy link
Collaborator Author

DavidBrainard commented Jan 28, 2018 via email

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