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

Memory improvements (1/3): Introduce new data access layer and schemas #1728

Merged
merged 33 commits into from
Oct 13, 2023

addressed most comments, still some left

b1ceb26
Select commit
Loading
Failed to load commit list.
Merged

Memory improvements (1/3): Introduce new data access layer and schemas #1728

addressed most comments, still some left
b1ceb26
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Oct 5, 2023 in 1s

95.63% of diff hit (target 95.00%)

View this Pull Request on Codecov

95.63% of diff hit (target 95.00%)

Annotations

Check warning on line 115 in covalent/_results_manager/result.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

covalent/_results_manager/result.py#L114-L115 

Added lines #L114 - L115 were not covered by tests

Check warning on line 118 in covalent/_results_manager/result.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

covalent/_results_manager/result.py#L117-L118 

Added lines #L117 - L118 were not covered by tests

Check warning on line 217 in covalent/_results_manager/result.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

covalent/_results_manager/result.py#L217 

Added line #L217 was not covered by tests

Check warning on line 74 in covalent/_serialize/common.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

covalent/_serialize/common.py#L74 

Added line #L74 was not covered by tests

Check warning on line 99 in covalent/_serialize/common.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

covalent/_serialize/common.py#L99 

Added line #L99 was not covered by tests

Check warning on line 161 in covalent/_serialize/common.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

covalent/_serialize/common.py#L161 

Added line #L161 was not covered by tests

Check warning on line 63 in covalent/_serialize/electron.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

covalent/_serialize/electron.py#L63 

Added line #L63 was not covered by tests

Check warning on line 67 in covalent/_serialize/electron.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

covalent/_serialize/electron.py#L67 

Added line #L67 was not covered by tests

Check warning on line 97 in covalent/_serialize/lattice.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

covalent/_serialize/lattice.py#L96-L97 

Added lines #L96 - L97 were not covered by tests

Check warning on line 227 in covalent/_serialize/lattice.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

covalent/_serialize/lattice.py#L227 

Added line #L227 was not covered by tests

Check warning on line 113 in covalent/_serialize/result.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

covalent/_serialize/result.py#L113 

Added line #L113 was not covered by tests

Check warning on line 118 in covalent/_serialize/result.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

covalent/_serialize/result.py#L116-L118 

Added lines #L116 - L118 were not covered by tests

Check warning on line 123 in covalent/_serialize/result.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

covalent/_serialize/result.py#L120-L123 

Added lines #L120 - L123 were not covered by tests

Check warning on line 126 in covalent/_serialize/result.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

covalent/_serialize/result.py#L126 

Added line #L126 was not covered by tests

Check warning on line 132 in covalent/_serialize/result.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

covalent/_serialize/result.py#L128-L132 

Added lines #L128 - L132 were not covered by tests

Check warning on line 134 in covalent/_serialize/result.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

covalent/_serialize/result.py#L134 

Added line #L134 was not covered by tests

Check warning on line 123 in covalent/_shared_files/schemas/electron.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

covalent/_shared_files/schemas/electron.py#L123 

Added line #L123 was not covered by tests

Check warning on line 123 in covalent/_shared_files/schemas/lattice.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

covalent/_shared_files/schemas/lattice.py#L123 

Added line #L123 was not covered by tests

Check warning on line 34 in covalent/_shared_files/schemas/transport_graph.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

covalent/_shared_files/schemas/transport_graph.py#L33-L34 

Added lines #L33 - L34 were not covered by tests

Check warning on line 66 in covalent/_shared_files/util_classes.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

covalent/_shared_files/util_classes.py#L66 

Added line #L66 was not covered by tests

Check warning on line 830 in covalent/_workflow/electron.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

covalent/_workflow/electron.py#L830 

Added line #L830 was not covered by tests

Check warning on line 835 in covalent/_workflow/electron.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

covalent/_workflow/electron.py#L832-L835 

Added lines #L832 - L835 were not covered by tests

Check warning on line 837 in covalent/_workflow/electron.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

covalent/_workflow/electron.py#L837 

Added line #L837 was not covered by tests

Check warning on line 839 in covalent/_workflow/electron.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

covalent/_workflow/electron.py#L839 

Added line #L839 was not covered by tests

Check warning on line 843 in covalent/_workflow/electron.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

covalent/_workflow/electron.py#L842-L843 

Added lines #L842 - L843 were not covered by tests
View more details on Codecov