Conversation
This is needed in various CI and RTD builds by ScopeSim et al., where it is currently downloaded every time. My understanding of ScopeSim_Data is to provide (some of) those things. This PR would bring that close to reality. This will only work though if SpeXtra gets told where to look for these files, or becomes smart enough to do so. But including the files here is required to test that there, so yeah. This adds around 5 MB, less than a 10 % increase in this repo.
|
I don't know whether it makes sense to add the spextra library data to ScopeSim_Data. We already have some of the data in speXtra itself, so maybe add this data there too? Or otherwise, move all that data here; that would make sense. If we do want to store the data here, then we should do it the other way around: make this directory in ScopeSim_Data the download directory of speXtra (when ScopeSim_Data is installed). Then a P.R. with the necessary data will be created automatically when the ScopeSim_Data CI is ran. |
Only some SVO filter curves are bundled in the package. I think it's sensible to keep the package itself as slim as possible, i.e. not add more data there. The spextra repo also has a
Yeah that makes sense (I think). So instead add something like this (I'm not sure I can do this on Windows?) first and then modify the cache location in spextra? Sure, let's try that! I'm not sure how Then we also have a confirmation that the CI here actually works! |
|
Yeah not sure how to do this properly with pooch. Because we should have a multi-tier setup in some way, with these considerations:
|
|
One simple (hacky? nah...) solution might be to first "manually" check ScopeSim_Data (via |
This is needed in various CI and RTD builds by ScopeSim et al., where it is currently downloaded every time. My understanding of ScopeSim_Data is to provide (some of) those things. This PR would bring that close to reality.
This will only work though if SpeXtra gets told where to look for these files, or becomes smart enough to do so. But including the files here is required to test that there, so yeah.
This adds around 5 MB, less than a 10 % increase in this repo.