8000 Expunged code using the GNU Scientific Library from CAMP. by jeff-cohere · Pull Request #25 · open-atmos/camp · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Expunged code using the GNU Scientific Library from CAMP. #25

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

Merged
merged 2 commits into from
Sep 3, 2024

Conversation

jeff-cohere
Copy link
Collaborator

This code violates the terms of the GNU Public License, since CAMP uses a non-GPL license, which is disallowed in ANY distributed code that calls GPL-licensed code. I've gone ahead and removed it.

This code violates the terms of the GNU Public License, since CAMP uses a non-GPL
license, which is disallowed in ANY distributed code that calls GPL-licensed code.
I've gone ahead and removed it.
Copy link
Collaborator
@mattldawson mattldawson left a comment

Choose a reason for hiding this comment

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

nice!

@mattldawson mattldawson merged commit ba192d9 into open-atmos:main Sep 3, 2024
1 check passed
@jeff-cohere jeff-cohere deleted the remove-gsl branch September 3, 2024 17:10
jtgasparik added a commit to jtgasparik/camp_ that referenced this pull request Feb 18, 2025
* Expunged code using the GNU Scientific Library from CAMP. (open-atmos#25)

* Expunged code using the GNU Scientific Library from CAMP.

This code violates the terms of the GNU Public License, since CAMP uses a non-GPL
license, which is disallowed in ANY distributed code that calls GPL-licensed code.
I've gone ahead and removed it.

* Adding make as an explicit dependency in Dockerfiles.

* Uses the standard C bool type, which is defined in stdbool.h. (open-atmos#26)

* Updates to work with new intel compiler (open-atmos#29)

update to work with new intel compiler

* Simpol multiple layers (open-atmos#30)

* addition of is_at_surface flag to num_phase_instances function

* testing partmc coupling

* testing

* fixed num_phase_instances function, now passes

* fixed function and added test

* one more test

* testing

* reverting change

* changes to phase_id function

* deleted duplicate num_instances variable

* testing SIMPOL reaction with multiple layers added to the json - currently failing

* update unique names function to have is_at_surface flag

* remote diagnostic output

* added test for unique_name function

* ive broken the SIMPOL rxn again by adding layers

* adding mass

* minor changes to SIMPOl test

* fix to write statement - still not passing tests

* added tests for phase_ids function to single particle

* test added for phase id for modal/binned rep

* added second particle to test suite for single particle c test - in progress

* adding unit tests to single particle c functions with multiple particles and different layers -- in progress

* adding second particle to single particle c tests --  in progress and failing

* adding another test particle and phase to single particle test aero rep test -- in progress

* testing the same phase in a different particle works. testing a different phase in a different particle is in-progress

* testing differnt particles and different layers

* second test phase added -- failing for test phase 1 and 3

* print statements to repo

* modified print statements

* update calls to aero phase functions (#9)

* MULTIPLE LAYERS WORK - fix to tests

* SIMPOL with two layers and some mass in second layer

* added test to SIMPOL

* added HL test with multiple layers, plus minor SIMPOL test changes

* one edit

* testing HL with mass in layer 2

* test from new laptop and fix to small typo

* another test and fix to typo

* Update src/aero_rep_data.F90

Co-authored-by: Matt Dawson <mattldawson@gmail.com>

* Update src/aero_reps/aero_rep_single_particle.F90

Co-authored-by: Matt Dawson <mattldawson@gmail.com>

* resolving comment

* fixes to PR

* Update src/aero_reps/aero_rep_single_particle.F90

Co-authored-by: Matt Dawson <mattldawson@gmail.com>

---------

Co-authored-by: Matt Dawson <mattdawson@ucar.edu>
Co-authored-by: Matt Dawson <mattldawson@gmail.com>

---------

Co-authored-by: Jeffrey N. Johnson <jeff@cohere-llc.com>
Co-authored-by: Matt Dawson <mattdawson@ucar.edu>
Co-authored-by: Matt Dawson <mattldawson@gmail.com>
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
0