-
Notifications
You must be signed in to change notification settings - Fork 70
Deprecate Repo #644
base: master
Are you sure you want to change the base?
Deprecate Repo #644
Conversation
Noting that tests have failed due to bad TeX dependencies. Since this PR is for deprecation, I'm okay with merging even with failing tests. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM but probably best to wait for merge until pacta.portfolio.analysis is public, otherwise it's a dead link
PACTA now lives in a new repo (https://github.com/RMI-PACTA/pactaverse) that covers all relevant steps of data preparation and analysis. The previously linked repository will be deprecated, see RMI-PACTA/PACTA_analysis#644
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it might make more sense to change the link to this:
https://github.com/RMI-PACTA/pactaverse
Since this repo has been split up into many packages
# PACTA_analysis | ||
# PACTA_analysis (DEPRECATED) | ||
|
||
This repo is deprecated, with continuing work available at: https://github.com/RMI-PACTA/pacta.portfolio.analysis |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This repo is deprecated, with continuing work available at: https://github.com/RMI-PACTA/pacta.portfolio.analysis | |
This repo is deprecated, with continuing work available at: https://github.com/RMI-PACTA/pactaverse |
@AlexAxthelm can we change the link and merge this? |
No description provided.