Skip to content

Emon api input bulk v0.2.3 #20

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
Merged
mano8 merged 29 commits into from
Jan 16, 2025
Merged

Emon api input bulk v0.2.3 #20

mano8 merged 29 commits into from
Jan 16, 2025

Conversation

@mano8
Copy link
Member

@mano8 mano8 commented Jan 16, 2025

Changes:

  • improve code style
  • Improve structure
  • Fix bugs
  • Add ability to post input bulk
  • Improve test coverage

mano8 added 20 commits January 15, 2025 10:01
@mano8
apply sync modules changes to async modules
7149d45
@mano8
move compute_response from api_utils.Utils to emon_api_core.EmonReque…
fbcfbfd 
…stCore
@mano8
add get_utc_timestamp and add ability to select timezone for methods …
6407669 
…get_start_day and get_string_datetime_from_timestamp
@mano8
fix compute_response method content-type
09e76c0
@mano8
fix get_input_filters_from_structure typing
4167b8b
@mano8
add encode_url_path and encode_params methods to EmonRequestCore
2fac4b1
@mano8
add prep_input_bulk method to EmonInputs
71492f6
@mano8
factorise encoding path and params
275609b
@mano8
improve code
e3978cf
@mano8
add input bulk ability
e206f53
@mano8
on create feeds return nb created feeds
4bb4d6c
@mano8
fix execute_request_timeout error message
bd3496a
@mano8
improve create_structure return value
c35c43e
@mano8
add get_extended_structure method
fb0056d
@mano8
improve code style
4417f08
@mano8
rename objects from emon_api_core module to fix unique object names
0ccfdf8
@mano8
improve validate_node and is_valid_node method
af32ce1
@mano8
fix empty string in encode_url_path method
a7668a2
@mano8
apply code changes in unit tests
57637b7
@mano8
Improve example with input bulk example
f1fda36
mano8
mano8 commented Jan 16, 2025
View reviewed changes
Copy link
Member Author

@mano8 mano8 left a comment

Choose a reason for hiding this comment

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

ok

@codecov
Copy link

codecov bot commented Jan 16, 2025 •
edited
Loading

Codecov Report

Attention: Patch coverage is 95.90164% with 10 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
emon_tools/async_emonpy.py 94.91% 0 Missing and 3 partials ⚠️
emon_tools/emon_api_core.py 95.71% 0 Missing and 3 partials ⚠️
emon_tools/utils.py 85.71% 1 Missing and 1 partial ⚠️
emon_tools/emon_api.py 96.96% 0 Missing and 1 partial ⚠️
emon_tools/emonpy.py 96.55% 0 Missing and 1 partial ⚠️
Files with missing lines Coverage Δ
emon_tools/__init__.py 100.00% <100.00%> (ø)
emon_tools/api_utils.py 94.20% <100.00%> (+0.57%) ⬆️
emon_tools/async_emon_api.py 90.47% <100.00%> (+11.66%) ⬆️
emon_tools/emon_api.py 96.06% <96.96%> (+2.77%) ⬆️
emon_tools/emonpy.py 92.02% <96.55%> (+0.36%) ⬆️
emon_tools/utils.py 98.34% <85.71%> (-1.66%) ⬇️
emon_tools/async_emonpy.py 89.13% <94.91%> (+35.68%) ⬆️
emon_tools/emon_api_core.py 97.34% <95.71%> (+0.74%) ⬆️

mano8
mano8 commented Jan 16, 2025
View reviewed changes
Copy link
Member Author

@mano8 mano8 left a comment

Choose a reason for hiding this comment

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

ok

mano8
mano8 commented Jan 16, 2025
View reviewed changes
Copy link
Member Author

@mano8 mano8 left a comment

Choose a reason for hiding this comment

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

ok

mano8
mano8 commented Jan 16, 2025
View reviewed changes
Copy link
Member Author

@mano8 mano8 left a comment

Choose a reason for hiding this comment

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

ok

mano8
mano8 commented Jan 16, 2025
View reviewed changes
Copy link
Member Author

@mano8 mano8 left a comment

Choose a reason for hiding this comment

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

ok

@mano8 mano8 changed the title Emon api input bulk Emon api input bulk v0.2.3 Jan 16, 2025
@mano8 mano8 merged commit f60a480 into main Jan 16, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@mano8