search

ip-tools / patzilla

PatZilla is a modular patent information research platform and data integration toolkit with a modern user interface and access to multiple data sources.
https://docs.ip-tools.org/patzilla/
GNU Affero General Public License v3.0
97 stars 21 forks source link

Migrate to Python3 #70

Open papoteur-mga opened 1 year ago

papoteur-mga commented 1 year ago

For the migration, I started from the last state of the main branch, then applied the patches from the python3 branch. Thus the last commits from the branch peds are not used,

amotl commented 1 year ago

Can you also update the CI/GHA configuration? From what I can see, adjusting those spots may be enough. Let's start with the Python version you are using on your workstation, and expand the test matrix later. You are currently using Python 3.8, right?

https://github.com/ip-tools/patzilla/blob/72d80e72d0e9871c37dd8948a7ea47bd078ee5b7/.github/workflows/tests.yaml#L29 https://github.com/ip-tools/patzilla/blob/72d80e72d0e9871c37dd8948a7ea47bd078ee5b7/.github/workflows/tests.yaml#L106

codecov-commenter commented 1 year ago

Codecov Report

Patch coverage: 33.25% and project coverage change: -7.47 :warning:

Comparison is base (0933634) 51.75% compared to head (c7af549) 44.28%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #70 +/- ## ========================================== - Coverage 51.75% 44.28% -7.47% ========================================== Files 100 100 Lines 7770 7878 +108 ========================================== - Hits 4021 3489 -532 - Misses 3749 4389 +640 ``` | Flag | Coverage Δ | | |---|---|---| | unittests | `44.28% <33.25%> (-7.47%)` | :arrow_down: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=ip-tools#carryforward-flags-in-the-pull-request-comment) to find out more. | [Impacted Files](https://codecov.io/gh/ip-tools/patzilla/pull/70?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=ip-tools) | Coverage Δ | | |---|---|---| | [patzilla/access/epo/espacenet/pyramid.py](https://codecov.io/gh/ip-tools/patzilla/pull/70?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=ip-tools#diff-cGF0emlsbGEvYWNjZXNzL2Vwby9lc3BhY2VuZXQvcHlyYW1pZC5weQ==) | `32.25% <ø> (ø)` | | | [patzilla/access/epo/ops/api.py](https://codecov.io/gh/ip-tools/patzilla/pull/70?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=ip-tools#diff-cGF0emlsbGEvYWNjZXNzL2Vwby9vcHMvYXBpLnB5) | `11.44% <0.00%> (-58.32%)` | :arrow_down: | | [patzilla/access/epo/publicationserver/client.py](https://codecov.io/gh/ip-tools/patzilla/pull/70?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=ip-tools#diff-cGF0emlsbGEvYWNjZXNzL2Vwby9wdWJsaWNhdGlvbnNlcnZlci9jbGllbnQucHk=) | `37.03% <0.00%> (+2.42%)` | :arrow_up: | | [patzilla/access/generic/exceptions.py](https://codecov.io/gh/ip-tools/patzilla/pull/70?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=ip-tools#diff-cGF0emlsbGEvYWNjZXNzL2dlbmVyaWMvZXhjZXB0aW9ucy5weQ==) | `42.85% <0.00%> (-13.25%)` | :arrow_down: | | [patzilla/access/ificlaims/api.py](https://codecov.io/gh/ip-tools/patzilla/pull/70?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=ip-tools#diff-cGF0emlsbGEvYWNjZXNzL2lmaWNsYWltcy9hcGkucHk=) | `16.35% <0.00%> (+0.52%)` | :arrow_up: | | [patzilla/access/office.py](https://codecov.io/gh/ip-tools/patzilla/pull/70?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=ip-tools#diff-cGF0emlsbGEvYWNjZXNzL29mZmljZS5weQ==) | `15.21% <0.00%> (+1.88%)` | :arrow_up: | | [patzilla/access/uspto/pdf.py](https://codecov.io/gh/ip-tools/patzilla/pull/70?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=ip-tools#diff-cGF0emlsbGEvYWNjZXNzL3VzcHRvL3BkZi5weQ==) | `30.76% <0.00%> (+0.76%)` | :arrow_up: | | [patzilla/navigator/services/\_\_init\_\_.py](https://codecov.io/gh/ip-tools/patzilla/pull/70?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=ip-tools#diff-cGF0emlsbGEvbmF2aWdhdG9yL3NlcnZpY2VzL19faW5pdF9fLnB5) | `46.66% <0.00%> (ø)` | | | [patzilla/navigator/services/admin.py](https://codecov.io/gh/ip-tools/patzilla/pull/70?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=ip-tools#diff-cGF0emlsbGEvbmF2aWdhdG9yL3NlcnZpY2VzL2FkbWluLnB5) | `24.19% <0.00%> (+2.52%)` | :arrow_up: | | [patzilla/navigator/services/dpma.py](https://codecov.io/gh/ip-tools/patzilla/pull/70?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=ip-tools#diff-cGF0emlsbGEvbmF2aWdhdG9yL3NlcnZpY2VzL2RwbWEucHk=) | `27.27% <0.00%> (+4.10%)` | :arrow_up: | | ... and [57 more](https://codecov.io/gh/ip-tools/patzilla/pull/70?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=ip-tools) | | ... and [11 files with indirect coverage changes](https://codecov.io/gh/ip-tools/patzilla/pull/70/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=ip-tools) Help us with your feedback. Take ten seconds to tell us [how you rate us](https://about.codecov.io/nps?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=ip-tools). Have a feature suggestion? [Share it here.](https://app.codecov.io/gh/feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=ip-tools)

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