-
Notifications
You must be signed in to change notification settings - Fork 24
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
Add installation from PyPI #4
Comments
I've had a quick look at |
@ebranlard any thoughts? I'm about to start developing some more open-source openfast-based tools so it would be really useful to get some versions of this package both tagged on GitHub and published to PyPI |
@andrew-platt @deslaughter you might have an opinion, as well |
@ptrbortolotti might have more of an opinion on this than I do. I'm definitely open to making this easier to install through PyPI or docker. However, I'm not really a regular user of this package, so I don't have a strong opinion. |
@rthedin may also have an opinion here. |
@cortadocodes This repository has largely been a side project for a few NREL researchers, and no one is directly responsible for maintaining and supporting it. My primary concern with publishing it more broadly is:
A possible solution is to state it clearly in the repository that this project is community maintained, and encourage users to fix anything that they see missing including the above points. Being the person to bring it up, this may fall on you 😉 If that sounds good to you, I'm happy to work with you to review a pull request to add the upload to PyPI to this repo's CI and manage the project on PyPI. |
Community maintenance sounds good to me! I think it will be beneficial to the project and its users and increase the speed we can get features that we need 🙂 We're happy at Octue to be one of the contributors.
I think fixes and changes could be done on an as-needed and when-developers-are-available basis
I'm happy to set up some CI workflows that automate the releases including publishing to PyPI. We'd just need to adopt a gitflow where all updates to |
I think this issue can be combined with OpenFAST/openfast#1851 - let's carry on the conversation there? |
I'm using
openfast_toolbox
as part of a dockerised productionised app but running into issues with the current installation method. The usual way I'd install a package would beor
Having to install by cloning and then editably installing makes it difficult to properly manage the dependencies for
openfast_toolbox
alongside the other packages in my app and means I have to create a specificDockerfile
just to run this installation method instead of using one of the generic ones we create for our apps. Editably installing is also not best practice for a productionised app but I'm unsure whether it's ok to install it non-editably.Is there a reason why
openfast_toolbox
isn't available on the PyPI package registry? If not, would you be able to publish it on PyPI so we can more easily install and use it?The text was updated successfully, but these errors were encountered: