desihub / fastspecfit

Fast spectral synthesis and emission-line fitting of DESI spectra.
https://fastspecfit.readthedocs.org
BSD 3-Clause "New" or "Revised" License
13 stars 2 forks source link

additional fujilupe v2.0 updates #96

Closed moustakas closed 1 year ago

moustakas commented 1 year ago

This PR is a follow-up to #95 and includes:

In addition to more testing, I still need to:

moustakas commented 1 year ago

@weaverba137, in my top-level unit test I'm grabbing the templates (which are too large to include in the repo itself) from https://data.desi.lbl.gov/desi/public/external/templates/fastspecfit/1.0.0/ftemplates-chabrier-1.0.0.fits using wget on this line but I'm getting a 401 Unauthorized error-- https://github.com/desihub/fastspecfit/actions/runs/3990504910/jobs/6844347749#step:6:105

What's strange is that I can successfully use the wget command within a terminal, so maybe it's a setting in my Github actions configuration file? Do you have any suggestions / thoughts?

weaverba137 commented 1 year ago

Do you have a ~/.netrc file when you are using wget within a terminal?

weaverba137 commented 1 year ago

I take that back. The problem is the URL.

You have to use the base URL https://data.desi.lbl.gov/public/....

moustakas commented 1 year ago

I take that back. The problem is the URL. You have to use the base URL https://data.desi.lbl.gov/public/....

In a Chrome incognito tab, https://data.desi.lbl.gov/desi/public prompts me for a login/password. What's the correct url?

weaverba137 commented 1 year ago

Sorry, I thought I already told you:

You have to use the base URL https://data.desi.lbl.gov/public/....