Skip to content

Conversation

@JaGeo
Copy link
Member

@JaGeo JaGeo commented May 6, 2024

How to treat old (non-core) codes of MP

Context and Problem Statement

Some codes in the context of the Materials Project have only been developed based on short-term funding, and long-term support is hard/impossible to guarantee.
This code might still be used in the community. One of the recent example codes is Matminer that triggered the discussion.

Decision Drivers

  • These codes are still useful to the community
  • There will likely not be any long-term funding for such codes

Considered Options

Use specific versions of the dependencies (via toml or setup.py-based installations) and then release a final version (or release candidate)

  • Good as the code can still be used
  • Bad as it might not be easy to use together with other software

Transfer code ownership to a fork outside of MP (if someone wants to take over code)

  • Good as the code will still be maintained

Give maintaining rights to people outside of MP

  • Good as more people can merge and contribute
  • Not clear how such a decision can be made

Ask on the repo if someone might want to take over

  • Good as someone else might maintain the code in the future
  • Connected with transferring code ownership to someone else

Decision Outcome

Implementation Plan

More Information

@JaGeo JaGeo changed the title Create 0017-how-to-treat-old-codes 0017-how-to-treat-old-codes May 6, 2024
@JaGeo JaGeo changed the title 0017-how-to-treat-old-codes 0027-how-to-treat-old-codes May 11, 2024
@JaGeo
Copy link
Member Author

JaGeo commented Jul 1, 2024

@Andrew-S-Rosen Could you help updating this PR with your suggestion? I am not sure I got all details on the release candidates.

@JaGeo JaGeo changed the title 0027-how-to-treat-old-codes 0027-how-to-retire-old-codes Jul 1, 2024
@JaGeo
Copy link
Member Author

JaGeo commented Jul 1, 2024

I personally think we can vote on this decision the next time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants