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

Switch from GPLv3 to more liberal MIT for atlite #263

Merged
merged 11 commits into from
Jan 13, 2023
Merged

Conversation

euronion
Copy link
Collaborator

@euronion euronion commented Nov 21, 2022

Similar to PyPSA and PyPSA-EUR we would like to change the license for atlite from the copyleft GNU General Public License Version 3 (GPLv3) to the more liberal MIT License.
This change is suggested because the benefits of a more liberal license seem to outweigh the benefits of a copyleft approach.

For a discussion of the benefits of either license see PyPSA/PyPSA#274 but also please feel free to also discuss below.

This change requires the consent of all people who have substantially contributed code to atlite.
If you consent, please reply with a ✔️ below. If you do not consent, please reply with a ❌ below. Please respond by 11 December 2022.

Thanks for your help and contributions!

Consent Essential (>= 20 lines of code)

Consent Appreciated (< 20 lines of code)

(I'll update the PR if everyone has agreed to the license switch later to include all the necessary changes to files for the license switch to be complete)

@codecov
Copy link

codecov bot commented Nov 21, 2022

Codecov Report

Base: 72.40% // Head: 72.40% // No change to project coverage 👍

Coverage data is based on head (a109aa4) compared to base (a37e8ae).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #263   +/-   ##
=======================================
  Coverage   72.40%   72.40%           
=======================================
  Files          19       19           
  Lines        1587     1587           
  Branches      220      220           
=======================================
  Hits         1149     1149           
  Misses        370      370           
  Partials       68       68           
Impacted Files Coverage Δ
atlite/aggregate.py 72.72% <ø> (ø)
atlite/convert.py 67.02% <ø> (ø)
atlite/csp.py 42.85% <ø> (ø)
atlite/cutout.py 72.82% <ø> (ø)
atlite/data.py 86.74% <ø> (ø)
atlite/datasets/__init__.py 100.00% <ø> (ø)
atlite/datasets/era5.py 89.93% <ø> (ø)
atlite/datasets/gebco.py 40.74% <ø> (ø)
atlite/datasets/sarah.py 25.53% <ø> (ø)
atlite/gis.py 89.84% <ø> (ø)
... and 9 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@fneum
Copy link
Member

fneum commented Nov 21, 2022

✔️

1 similar comment
@nworbmot
Copy link
Member

✔️

@davide-f
Copy link
Contributor

Agree

@FabianHofmann
Copy link
Contributor

agree

@pz-max
Copy link
Contributor

pz-max commented Nov 21, 2022

✅ agree

@p-glaum
Copy link
Contributor

p-glaum commented Nov 21, 2022

✔️ agree

@leonsn
Copy link
Contributor

leonsn commented Nov 21, 2022

✔️

@zoltanmaric
Copy link
Contributor

✔️ I consent

@coroa
Copy link
Member

coroa commented Nov 21, 2022

✔️

1 similar comment
@LukasFrankenQ
Copy link
Collaborator

✔️

@thesethtruth
Copy link
Contributor

Great initiative! I consent ✔️

@euronion
Copy link
Collaborator Author

Ping @schlachtberger

@euronion
Copy link
Collaborator Author

I received consent by @schlachtberger off list, so we can go ahead with relicensing.

@euronion
Copy link
Collaborator Author

RTR @FabianHofmann or @fneum

@euronion euronion marked this pull request as ready for review January 13, 2023 08:24
@euronion euronion linked an issue Jan 13, 2023 that may be closed by this pull request
Copy link
Contributor

@FabianHofmann FabianHofmann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great!

@euronion euronion merged commit 610edb7 into master Jan 13, 2023
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.

Relicense from GPL to MIT?