Skip to content

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

Draft
tylerbarna wants to merge 319 commits into
nuclear-multimessenger-astronomy:mainfrom
tylerbarna:remove-Bu2019lm-pickle
Draft

Remove old Bu2019lm pickle file and updates to installation documentation#105
tylerbarna wants to merge 319 commits into
nuclear-multimessenger-astronomy:mainfrom
tylerbarna:remove-Bu2019lm-pickle

Conversation

@tylerbarna
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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