-
Notifications
You must be signed in to change notification settings - Fork 5
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
Deriving nuclear data from source data #1
Comments
Also, I'd like to know if you are stuck on using the GPL license rather than a MIT license for the open source software. Many folks who use our reactor modeling framework develop proprietary code and we are of course concerned with stipulations in licensing space for requiring the release / open sourcing of code that uses a library like this. |
I can easily implement the majority of ARMI's proposed specification that doesn't already exist in this library. The only difficulty would be establishing a structure that reads from files. There are several approaches to this.
|
Hi @JASory - I appreciate your response! I am learning Rust, so I wanted to use this as an opportunity to start to contribute to a project that we could also get some good use from. I'll digest your comments and try to propose some next steps where I could help/work with you on. Thanks for checking out the proposed specification on the ARMI repository. We will tidy that up and make sure our requirements and proposed test cases are clearly defined too. |
Hi @JASory, I have been doing some research to find a package that could meet our needs on terrapower/armi#460 as a general nuclide library that can be imported across different languages, mainly Python at this time. I am learning Rust and came across this package and thought that it might be nice to collaborate!
After looking through your codebase, I am curious if we could work on the following functionalities:
I'd love to set up some discussions if you are interested and see if there is a way that we could develop requirements, documentation, and further testing! Please let me know 🙂
The text was updated successfully, but these errors were encountered: