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.

Sign up for GitHub

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

Jump to bottom

add influx parser (#10) #13

Merged
merged 34 commits into from
Jul 11, 2024
Merged

add influx parser (#10) #13

merged 34 commits into from
Jul 11, 2024

Conversation

Treesarj
Copy link
Collaborator

Add influx db parser and tests

Description:

Type of change:

  • Bug fix.
  • New feature.
  • Documentation update.

Checklist for the reviewer:

This checklist should be used as a help for the reviewer.

  • Is the change limited to one issue?
  • Does this PR close the issue?
  • Is the code easy to read and understand, including clearly named variables?
  • Do all new feature have an accompanying new test?
  • Has the documentation been updated as necessary?

@Treesarj
add influx parser (#10)
ee9fc41 
Add influx db parser and tests
@Treesarj Treesarj linked an issue Jul 10, 2024 that may be closed by this pull request
Merge in influx db parser and tests #12
Closed
@Treesarj Treesarj self-assigned this Jul 10, 2024
@Treesarj Treesarj requested a review from CasperWA July 10, 2024 08:47
CasperWA
CasperWA requested changes Jul 10, 2024
View reviewed changes
oteapi_dlite/strategies/oceanlab_influx_parser.py Outdated Show resolved Hide resolved
oteapi_dlite/strategies/oceanlab_influx_parser.py Outdated Show resolved Hide resolved
oteapi_dlite/strategies/oceanlab_influx_parser.py Outdated Show resolved Hide resolved
oteapi_dlite/strategies/oceanlab_influx_parser.py Outdated Show resolved Hide resolved
oteapi_dlite/strategies/oceanlab_influx_parser.py Outdated Show resolved Hide resolved
requirements.txt Outdated Show resolved Hide resolved
oteapi_dlite/strategies/oceanlab_influx_parser.py Outdated Show resolved Hide resolved
tests/strategies/test_influx_db_parser.py Outdated Show resolved Hide resolved
tests/strategies/test_influx_db_parser.py Outdated Show resolved Hide resolved
tests/strategies/test_influx_db_parser.py Outdated Show resolved Hide resolved
Treesarj and others added 16 commits July 10, 2024 12:17
@Treesarj @CasperWA
Apply suggestions from code review
ab1b3d9 
Co-authored-by: Casper Welzel Andersen <[email protected]>
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
a2a90cc 
for more information, see https://pre-commit.ci
@Treesarj
code review changes
ece677f
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
e5a5b78 
for more information, see https://pre-commit.ci
@Treesarj
fix imports
6de8ce4
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
05e658b 
for more information, see https://pre-commit.ci
@Treesarj
code review suggestions
f138f9c
@Treesarj
Merge branch '12-merge-in-influx-db-parser-and-tests' of github.com:S…
048141d 
…INTEF/oteapi-dlite-Mod into 12-merge-in-influx-db-parser-and-tests
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
7eef732 
for more information, see https://pre-commit.ci
@Treesarj
fix models
b93bf78
@Treesarj
Update oceanlab_influx_parser.py
ff2c003
@Treesarj
Update oceanlab_influx_parser.py
9f50a8d
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
e26c51a 
for more information, see https://pre-commit.ci
@Treesarj
fix models
2040ede
@Treesarj
Merge branch '12-merge-in-influx-db-parser-and-tests' of github.com:S…
d9ea89a 
…INTEF/oteapi-dlite-Mod into 12-merge-in-influx-db-parser-and-tests
@Treesarj
fix models
b63bc64
@Treesarj Treesarj requested a review from CasperWA July 10, 2024 12:49
CasperWA
CasperWA reviewed Jul 10, 2024
View reviewed changes
oteapi_dlite/strategies/oceanlab_influx_parser.py Outdated Show resolved Hide resolved
oteapi_dlite/strategies/oceanlab_influx_parser.py Outdated Show resolved Hide resolved
oteapi_dlite/strategies/oceanlab_influx_parser.py Outdated Show resolved Hide resolved
oteapi_dlite/strategies/oceanlab_influx_parser.py Outdated Show resolved Hide resolved
oteapi_dlite/strategies/oceanlab_influx_parser.py Outdated Show resolved Hide resolved
oteapi_dlite/strategies/oceanlab_influx_parser.py Outdated Show resolved Hide resolved
@Treesarj Treesarj requested a review from CasperWA July 11, 2024 08:34
@Treesarj Treesarj merged commit fbe1094 into master Jul 11, 2024
8 of 9 checks passed
@Treesarj Treesarj deleted the 12-merge-in-influx-db-parser-and-tests branch July 11, 2024 09:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@CasperWA CasperWA CasperWA approved these changes

Assignees

@Treesarj Treesarj

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Merge in influx db parser and tests
2 participants
@Treesarj @CasperWA