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

Database records can be changed by reference (C++, Python) #70

Merged
merged 279 commits into from
Jul 23, 2024

Conversation

gdmiron
Copy link
Member

@gdmiron gdmiron commented Jul 23, 2024

  1. Fixed refresh all_elements data after setElement, addElement

  2. Added reference functions to C API

  3. Added trace output functionality for the database API

  4. Added new functionality to the python API

  5. Implemented test script of new functions into the Python API

    /examples/database_api.py

gdmiron and others added 30 commits November 25, 2019 09:53
Development

* function to calculate Psat for HGK returns ThermoScalar

* improved database parsing; refurbished calculation of reaction properties from reactants

* removed files left from ThermoHubClient

* cleaned thermofun main include

* first look at automatic tests using pytest

* added pytest to dependencies

* fixes to pytest calls

* fixes

* functions to append data to database

* get python path

* fix

* fixed path to python

* fixed deltas

* fixes in install scripts for travis and pipelines ci

* in ci pipelines tests are executed in install script

* fix test script execution

* tests only executed in linux ci
to avoid problems with precision in MacOS vs Linux
Development

Approved-by: George Dan Miron <[email protected]>
fix for json parse crash in release mode, yet unknown reason
temporarily disabled tests in travis ci
changed binder link to jupyter lab, fixes to readme
changes to accomodate new database structure containing lists of 'substances':[], 'reactions':[], 'elements':[]

Approved-by: George Dan Miron <[email protected]>
gdmiron and others added 29 commits December 6, 2022 14:34
Fixed&debug cmake and CI scripts (using spdlog in header-only mode or lib)

Approved-by: George Dan Miron
Change get functions in so that the values can be changed by references
@gdmiron gdmiron merged commit c5e692c into thermohub:master Jul 23, 2024
7 checks passed
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.

3 participants