Skip to content

fix: avoid using namespace std; in header files#2437

Merged
wanghan-iapcm merged 1 commit intodeepmodeling:develfrom
e-kwsm:std
Apr 3, 2023
Merged

fix: avoid using namespace std; in header files#2437
wanghan-iapcm merged 1 commit intodeepmodeling:develfrom
e-kwsm:std

Conversation

@e-kwsm
Copy link
Contributor

@e-kwsm e-kwsm commented Apr 2, 2023

No description provided.

Signed-off-by: Eisuke Kawashima <e-kwsm@users.noreply.github.com>
@njzjz njzjz changed the base branch from master to devel April 2, 2023 20:23
@codecov
Copy link

codecov bot commented Apr 2, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (3ac8c4c) 75.20% compared to head (dbfa586) 75.20%.

Additional details and impacted files
@@           Coverage Diff           @@
##            devel    #2437   +/-   ##
=======================================
  Coverage   75.20%   75.20%           
=======================================
  Files         223      223           
  Lines       22502    22502           
  Branches     1658     1658           
=======================================
  Hits        16923    16923           
+ Misses       4504     4496    -8     
- Partials     1075     1083    +8     
Impacted Files Coverage Δ
source/api_cc/include/AtomMap.h 100.00% <ø> (ø)
source/lib/include/SimulationRegion_Impl.h 58.11% <ø> (ø)

... and 4 files with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@wanghan-iapcm wanghan-iapcm merged commit 117fc91 into deepmodeling:devel Apr 3, 2023
@e-kwsm e-kwsm deleted the std branch April 3, 2023 08:31
nahso pushed a commit to HPC-AI-Team/deepmd-kit-se_atten-models that referenced this pull request Apr 12, 2023
Signed-off-by: Eisuke Kawashima <e-kwsm@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants