Skip to content
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

New package: MinCostFlows v0.1.2 #120355

Merged

Conversation

JuliaRegistrator
Copy link
Contributor

@JuliaRegistrator JuliaRegistrator commented Nov 28, 2024

Copy link
Contributor

github-actions bot commented Nov 28, 2024

Hello, I am an automated registration bot. I help manage the registration process by checking your registration against a set of AutoMerge guidelines. If all these guidelines are met, this pull request will be merged automatically, completing your registration. It is strongly recommended to follow the guidelines, since otherwise the pull request needs to be manually reviewed and merged by a human.

1. New package registration

Please make sure that you have read the package naming guidelines.

2. AutoMerge Guidelines are all met! ✅

Your new package registration met all of the guidelines for auto-merging and is scheduled to be merged when the mandatory waiting period (3 days) has elapsed.

3. To pause or stop registration

If you want to prevent this pull request from being auto-merged, simply leave a comment. If you want to post a comment without blocking auto-merging, you must include the text [noblock] in your comment.

Tip: You can edit blocking comments to add [noblock] in order to unblock auto-merging.

@JuliaRegistrator JuliaRegistrator force-pushed the registrator-mincostflows-62286e6e-v0.1.2-9a8e4d9d64 branch from 8d4ad5e to 4cc28d3 Compare November 28, 2024 21:52
JuliaRegistrator referenced this pull request in NREL/MinCostFlows.jl Nov 28, 2024
@GordStephen
Copy link

Regarding the license: there is indeed a license, but it's "modified" MIT Expat . My understanding is that, at least in 2019 when this software was licensed, it was corporate policy to require that "THE UNITED STATES, THE UNITED STATES DEPARTMENT OF ENERGY, OR ANY OF THEIR EMPLOYEES" be explicitly added to the liability waiver language.

Hopefully this isn't a blocker. If it is, I can talk to our tech transfer people about relicensing the software, but if there's any latitude to accept it as-is that would make things a lot easier.

[noblock]

@goerz
Copy link
Member

goerz commented Nov 29, 2024

[noblock]

My understanding is that there was a decision that an OSI-approved license is a hard requirement. There is no leeway for any kind of modifications beyond what the automated bot still manages to recognize. I wasn't involved in that discussion, but I think the argument was that the Julia community is not equipped to understand what modifications to a license might legally imply. Even if it's a trivial modification such as this one, I'm not sure if there's any registry maintainer who would want to manually override the check somehow. This is especially true because AFAIK the license check is not just performed on the initial registration, but also on all subsequent updates. So, every update of the package would have to be merged manually.

Your best bet is probably to use a LICENSE file that makes the bot happy. If it's not possible to sneak in the extra text somehow (the bot does swallow some minor modifications, possibly at the beginning and end, not sure), I would include the extra text in some other place. For example, put the current modified license text at the bottom of your README. I'm not a lawyer, but I don't think the unmodified MIT and your modified MIT are in any way incompatible, or that the note would be any more or less binding if it appeared in the README. Quite honestly, my common-sense legal opinion is that the extra note about the U.S. government is completely redundant. In that vein, you could try to clarify whether that extra language is still necessary. Or, probably better, don't make a fuss about it and find a pragmatic workaround. I suspect you're not going to get into any kind of trouble (even if you were to just completely drop the extra language from the LICENSE), but make your own judgement on that.

@GordStephen
Copy link

GordStephen commented Nov 29, 2024

Ok, thanks - that's understandable. I suspect this won't be a big deal to change on our end, but with the holiday here it'll just take a few days to get sorted.

[noblock]

UUID: 62286e6e-1779-56f1-888a-1c0056788ce0
Repo: https://github.com/NREL/MinCostFlows.jl.git
Tree: 8f52000f0db84f9bda8114c874410e69ce36f067

Registrator tree SHA: 17aec322677d9b81cdd6b9b9236b09a3f1374c6a
@JuliaRegistrator JuliaRegistrator force-pushed the registrator-mincostflows-62286e6e-v0.1.2-9a8e4d9d64 branch from 4cc28d3 to 767ef3c Compare December 3, 2024 02:59
JuliaRegistrator referenced this pull request in NREL/MinCostFlows.jl Dec 3, 2024
@JuliaTagBot JuliaTagBot merged commit 76ed869 into master Dec 3, 2024
9 checks passed
@JuliaTagBot JuliaTagBot deleted the registrator-mincostflows-62286e6e-v0.1.2-9a8e4d9d64 branch December 3, 2024 03:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants