Skip to content

Safer HSP Heuristics#83

Merged
guilyx merged 7 commits intomainfrom
safe-hsp-heuristics
Jan 4, 2021
Merged

Safer HSP Heuristics#83
guilyx merged 7 commits intomainfrom
safe-hsp-heuristics

Conversation

@guilyx
Copy link
Copy Markdown
Member

@guilyx guilyx commented Jan 4, 2021

Delete Relaxation heuristics now can be tried on any domain. Failure to compute H will not create a breach in the code.

Closes #82

@guilyx guilyx added bug Something isn't working enhancement New feature or request labels Jan 4, 2021
@guilyx guilyx added this to the Polished Product milestone Jan 4, 2021
@codecov
Copy link
Copy Markdown

codecov Bot commented Jan 4, 2021

Codecov Report

Merging #83 (8e2d2eb) into main (42f18d0) will increase coverage by 4.52%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #83      +/-   ##
==========================================
+ Coverage   88.95%   93.47%   +4.52%     
==========================================
  Files          16       17       +1     
  Lines         878      966      +88     
==========================================
+ Hits          781      903     +122     
+ Misses         97       63      -34     
Impacted Files Coverage Δ
jupyddl/automated_planner.py 94.44% <100.00%> (+15.59%) ⬆️
jupyddl/heuristics.py 90.69% <100.00%> (+0.10%) ⬆️
tests/test_automated_planner.py 100.00% <100.00%> (ø)
tests/test_basic_astar.py 100.00% <100.00%> (ø)
tests/test_basic_search.py 100.00% <100.00%> (ø)
tests/test_heuristics.py 100.00% <100.00%> (ø)
tests/test_hsp_astar.py 100.00% <100.00%> (ø)
tests/test_node.py 100.00% <100.00%> (ø)
... and 6 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5aee9f4...8e2d2eb. Read the comment docs.

@guilyx guilyx merged commit 1f40c6a into main Jan 4, 2021
@mergify mergify Bot deleted the safe-hsp-heuristics branch January 4, 2021 14:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Try catch when using DRH

1 participant