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

Feature/grib jump changes #42

Merged
merged 5 commits into from
Mar 19, 2024
Merged

Feature/grib jump changes #42

merged 5 commits into from
Mar 19, 2024

Conversation

ChrisspyB
Copy link
Member

Changes made to metkit by myself during the development of grib jump, and by Tiago when preparing the polytope demo.

  • GribHandle constructor starting from a specified offset, rather than always from the beginning of the file.
  • Method for splitting MarsRequests along keys.
  • Silent option for GribAccessors.
  • Replace eckit::Log::debug with LOG_DEBUG_LIB macro.

@FussyDuck
Copy link

FussyDuck commented Feb 21, 2024

CLA assistant check
All committers have signed the CLA.

@@ -370,7 +370,7 @@ Length DHSProtocol::retrieve(const MarsRequest& request)
{
Endpoint callbackEndpoint = callback_->endpoint();

Log::debug() << "DHSProtocol: call back on " << callbackEndpoint << std::endl;
LOG_DEBUG_LIB(LibMetkit) << "DHSProtocol: call back on " << callbackEndpoint << std::endl;
Copy link
Member

Choose a reason for hiding this comment

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

this slightly changes the log behaviour: it now depends on the env variable METKIT_DEBUG instead of DEBUG.
It might be even better...

Copy link
Member

Choose a reason for hiding this comment

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

it is more efficient

Copy link
Member

Choose a reason for hiding this comment

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

and correct

Copy link
Member

@tlmquintino tlmquintino left a comment

Choose a reason for hiding this comment

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

All good. I had helped write the code so Im a bit biased

@@ -370,7 +370,7 @@ Length DHSProtocol::retrieve(const MarsRequest& request)
{
Endpoint callbackEndpoint = callback_->endpoint();

Log::debug() << "DHSProtocol: call back on " << callbackEndpoint << std::endl;
LOG_DEBUG_LIB(LibMetkit) << "DHSProtocol: call back on " << callbackEndpoint << std::endl;
Copy link
Member

Choose a reason for hiding this comment

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

it is more efficient

@@ -370,7 +370,7 @@ Length DHSProtocol::retrieve(const MarsRequest& request)
{
Endpoint callbackEndpoint = callback_->endpoint();

Log::debug() << "DHSProtocol: call back on " << callbackEndpoint << std::endl;
LOG_DEBUG_LIB(LibMetkit) << "DHSProtocol: call back on " << callbackEndpoint << std::endl;
Copy link
Member

Choose a reason for hiding this comment

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

and correct

@tlmquintino tlmquintino merged commit f7112c4 into develop Mar 19, 2024
66 checks passed
@tlmquintino tlmquintino deleted the feature/grib-jump-changes branch March 19, 2024 11:47
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.

4 participants