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

Remove old Bu2019lm pickle file and updates to installation documentation #105

Draft
wants to merge 319 commits into
base: main
Choose a base branch
from

Conversation

tylerbarna
Copy link
Collaborator

@tylerbarna tylerbarna commented Jun 14, 2023

Resolves #91

Removes the old Bu2019lm.pkl file in svdmodels and updates the documentation to include links to the oversized pickle files required to use the Bu2019lm model. As discussed in the referenced issue, the current pickle file causes errors when attempting to use the Bu2019lm (kilonova) model.

Also added a note about creating a symbolic link between svdmodels folders, reorganized the installation guide and potential issues for clarity; changed some of bare urls to be hyperlinks.

Peter T. H. Pang and others added 30 commits March 23, 2022 21:58
…rma049/file

Add notes about cython to installation file.
…rma049/docfix

This PR uses Bu2019lm as the example.
bfhealy and others added 19 commits May 26, 2023 10:00
…orkflow-updates

This PR attempts to run the workflow in python-package.yml on pull requests. It also standardizes the syntax of deploy-github-pages.yml with the other workflow files.
Removes the old Bu2019lm.pkl file in svdmodels and updates the documentation to include links to the oversized pickle files required to use the Bu2019lm model. Also added a note about creating a symbolic link between svdmodels folders and reorganized some of the referenced installation issues for clarity and changed some of bare urls to be hyperlinks.
@tylerbarna tylerbarna added bug Something isn't working documentation Improvements or additions to documentation labels Jun 14, 2023
@tylerbarna tylerbarna requested a review from mcoughlin June 14, 2023 17:59
@tylerbarna tylerbarna self-assigned this Jun 14, 2023
@mcoughlin
Copy link
Member

@Theodlz Is trying to fix downloading the files from Zenodo, so will tag him as a reviewer to update this once that happens.

@mcoughlin mcoughlin requested a review from Theodlz June 14, 2023 18:02
@mcoughlin
Copy link
Member

@tylerbarna Please update this PR to reflect the new Zenodo download scheme.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bu2019lm svdmodel