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

feat: New simple query for handling Plexos objects #18

Merged
merged 5 commits into from
Oct 21, 2024
Merged

feat: New simple query for handling Plexos objects #18

merged 5 commits into from
Oct 21, 2024

Conversation

pesap
Copy link
Collaborator

@pesap pesap commented Oct 21, 2024

List of changes

  • Added new file simple_object_query.sql that contains a simplified version to get the information of the objects
  • Added standard NOSPACE to object schema,
  • Added functionality to disable custom collates using create_collations: bool = False for the PlexosSQLite() class,
  • Added new setting for the SQLite table: conn.execute("PRAGMA journal_mode = OFF") for testing purposes,

@codecov-commenter
Copy link

codecov-commenter commented Oct 21, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.45%. Comparing base (7144af0) to head (2c75301).

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #18   +/-   ##
=======================================
  Coverage   97.44%   97.45%           
=======================================
  Files           5        5           
  Lines         587      589    +2     
=======================================
+ Hits          572      574    +2     
  Misses         15       15           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@pesap pesap merged commit db16280 into main Oct 21, 2024
7 checks passed
@pesap pesap deleted the ps/query branch October 21, 2024 17:24
@ktehranchi ktehranchi mentioned this pull request Oct 29, 2024
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