Page MenuHomeSoftware Heritage

Clean up code after frontier redefinition from previous commit
ClosedPublic

Authored by aeviso on Dec 15 2021, 4:45 PM.

Details

Summary

The known flag from IsochroneNode is no longer being used, and
maxdate calculation can thus be simplified. Isochrone graph related
tests are updated as well to reflect the changes.

Diff Detail

Event Timeline

Build is green

Patch application report for D6846 (id=24808)

Rebasing onto eb48f7960d...

Current branch diff-target is up to date.
Changes applied before test
commit 960770e7694db2640c2e74270f27e740647b051a
Author: Andres Ezequiel Viso <aeviso@softwareheritage.org>
Date:   Wed Dec 15 16:43:02 2021 +0100

    Clean up code after frontier redefinition from previous commit
    
    The `known` flag from `IsochroneNode` is no longer being used and
    `maxdate` calculation can thus be simplified, Isochrne graph related
    tests are updated as well to reflect the changes.

See https://jenkins.softwareheritage.org/job/DPROV/job/tests-on-diff/545/ for more details.

aeviso edited the summary of this revision. (Show Details)

Build has FAILED

Patch application report for D6846 (id=24833)

Rebasing onto eb48f7960d...

Current branch diff-target is up to date.
Changes applied before test
commit 294fcff0812f0453dc1e99895c982fb6db5a4179
Author: Andres Ezequiel Viso <aeviso@softwareheritage.org>
Date:   Wed Dec 15 16:43:02 2021 +0100

    Clean up code after frontier redefinition from previous commit
    
    The `known` flag from `IsochroneNode` is no longer being used, and
    `maxdate` calculation can thus be simplified. Isochrone graph related
    tests are updated as well to reflect the changes.

Link to build: https://jenkins.softwareheritage.org/job/DPROV/job/tests-on-diff/546/
See console output for more information: https://jenkins.softwareheritage.org/job/DPROV/job/tests-on-diff/546/console

Build has FAILED

Patch application report for D6846 (id=24833)

Rebasing onto eb48f7960d...

Current branch diff-target is up to date.
Changes applied before test
commit 294fcff0812f0453dc1e99895c982fb6db5a4179
Author: Andres Ezequiel Viso <aeviso@softwareheritage.org>
Date:   Wed Dec 15 16:43:02 2021 +0100

    Clean up code after frontier redefinition from previous commit
    
    The `known` flag from `IsochroneNode` is no longer being used, and
    `maxdate` calculation can thus be simplified. Isochrone graph related
    tests are updated as well to reflect the changes.

Link to build: https://jenkins.softwareheritage.org/job/DPROV/job/tests-on-diff/547/
See console output for more information: https://jenkins.softwareheritage.org/job/DPROV/job/tests-on-diff/547/console

Build is green

Patch application report for D6846 (id=24841)

Could not rebase; Attempt merge onto eb48f7960d...

Updating eb48f79..6524bf0
Fast-forward
 swh/__init__.py                                    |  3 +--
 swh/provenance/graph.py                            | 27 +++----------------
 swh/provenance/revision.py                         |  4 +--
 .../tests/data/graphs_cmdbts2_lower_1.yaml         | 31 ----------------------
 .../tests/data/graphs_cmdbts2_lower_2.yaml         | 31 ----------------------
 .../tests/data/graphs_cmdbts2_upper_1.yaml         | 27 -------------------
 .../tests/data/graphs_cmdbts2_upper_2.yaml         | 26 ------------------
 .../tests/data/graphs_out-of-order_lower_1.yaml    | 19 +++----------
 swh/provenance/tests/test_isochrone_graph.py       |  1 -
 9 files changed, 10 insertions(+), 159 deletions(-)
Changes applied before test
commit 6524bf0140453eebbbe2c5f3cf2f89e0e4c642f5
Author: Andres Ezequiel Viso <aeviso@softwareheritage.org>
Date:   Wed Dec 15 16:43:02 2021 +0100

    Clean up code after frontier redefinition from previous commit
    
    The `known` flag from `IsochroneNode` is no longer being used, and
    `maxdate` calculation can thus be simplified. Isochrone graph related
    tests are updated as well to reflect the changes.

commit 83d45b7c2be2f81e306693001be8f0dc31fca88a
Author: Andres Ezequiel Viso <aeviso@softwareheritage.org>
Date:   Thu Dec 16 14:31:18 2021 +0100

    Drop unnecessary type annotation to fix mypy 0.920 issue

See https://jenkins.softwareheritage.org/job/DPROV/job/tests-on-diff/549/ for more details.

Build is green

Patch application report for D6846 (id=24843)

Could not rebase; Attempt merge onto eb48f7960d...

Updating eb48f79..3da57b2
Fast-forward
 swh/__init__.py                                    |  3 +--
 swh/provenance/graph.py                            | 27 +++----------------
 swh/provenance/revision.py                         |  4 +--
 .../tests/data/graphs_cmdbts2_lower_1.yaml         | 31 ----------------------
 .../tests/data/graphs_cmdbts2_lower_2.yaml         | 31 ----------------------
 .../tests/data/graphs_cmdbts2_upper_1.yaml         | 27 -------------------
 .../tests/data/graphs_cmdbts2_upper_2.yaml         | 26 ------------------
 .../tests/data/graphs_out-of-order_lower_1.yaml    | 19 +++----------
 swh/provenance/tests/test_isochrone_graph.py       |  1 -
 tox.ini                                            |  2 +-
 10 files changed, 11 insertions(+), 160 deletions(-)
Changes applied before test
commit 3da57b271d046d8fa9c0dc589ae9d5ca3c164618
Author: Andres Ezequiel Viso <aeviso@softwareheritage.org>
Date:   Wed Dec 15 16:43:02 2021 +0100

    Clean up code after frontier redefinition from previous commit
    
    The `known` flag from `IsochroneNode` is no longer being used, and
    `maxdate` calculation can thus be simplified. Isochrone graph related
    tests are updated as well to reflect the changes.

commit bef0ab385dfa94a41f8dd256817a633d60b11605
Author: Andres Ezequiel Viso <aeviso@softwareheritage.org>
Date:   Thu Dec 16 14:31:18 2021 +0100

    Drop unnecessary type annotation to fix mypy 0.920 issue

See https://jenkins.softwareheritage.org/job/DPROV/job/tests-on-diff/551/ for more details.

Build is green

Patch application report for D6846 (id=24873)

Rebasing onto bef0ab385d...

Current branch diff-target is up to date.
Changes applied before test
commit 1926b9d575fb7c406f02fea52f03cb9ab0ef5b70
Author: Andres Ezequiel Viso <aeviso@softwareheritage.org>
Date:   Wed Dec 15 16:43:02 2021 +0100

    Clean up code after frontier redefinition from previous commit
    
    The `known` flag from `IsochroneNode` is no longer being used, and
    `maxdate` calculation can thus be simplified. Isochrone graph related
    tests are updated as well to reflect the changes.

See https://jenkins.softwareheritage.org/job/DPROV/job/tests-on-diff/553/ for more details.

olasd added a subscriber: olasd.

ACK, makes sense.

This revision is now accepted and ready to land.Dec 21 2021, 3:50 PM