search

neurodata / ProgLearn

NeuroData's package for exploring and using progressive learning algorithms
https://proglearn.neurodata.io
Other
35 stars 42 forks source link

DOC fix typos & FIX modify python version for website builder #547

Open PSSF23 opened 2 years ago

PSSF23 commented 2 years ago

Reference issue

Close #546 Close #543

Type of change

Documentation, Website

What does this implement/fix?

jdey4 commented 2 years ago

@PSSF23 what did you do about the MLKNN unit test?

PSSF23 commented 2 years ago

@PSSF23 what did you do about the MLKNN unit test?

Nothing. Unrelated.

jdey4 commented 2 years ago

The unit tests seem to fail.

PSSF23 commented 2 years ago

The unit tests seem to fail.

Yes. But it's not related to this PR. I made another issue #548 for that.

jdey4 commented 2 years ago

The unit tests seem to fail.

Yes. But it's not related to this PR. I made another issue #548 for that.

It's bad practice to merge PRs without all checks being passed! It will be better to possibly take out that unit test or use @pargma no cover to ignore the unit test.

codecov[bot] commented 2 years ago

Codecov Report

Base: 92.36% // Head: 88.54% // Decreases project coverage by -3.81% :warning:

Coverage data is based on head (0d1a7cf) compared to base (af84f50). Patch has no changes to coverable lines.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## staging #547 +/- ## =========================================== - Coverage 92.36% 88.54% -3.82% =========================================== Files 7 7 Lines 419 419 =========================================== - Hits 387 371 -16 - Misses 32 48 +16 ``` | [Impacted Files](https://codecov.io/gh/neurodata/ProgLearn/pull/547?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=neurodata) | Coverage Δ | | |---|---|---| | [proglearn/forest.py](https://codecov.io/gh/neurodata/ProgLearn/pull/547/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=neurodata#diff-cHJvZ2xlYXJuL2ZvcmVzdC5weQ==) | `100.00% <ø> (ø)` | | | [proglearn/network.py](https://codecov.io/gh/neurodata/ProgLearn/pull/547/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=neurodata#diff-cHJvZ2xlYXJuL25ldHdvcmsucHk=) | `100.00% <ø> (ø)` | | | [proglearn/progressive\_learner.py](https://codecov.io/gh/neurodata/ProgLearn/pull/547/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=neurodata#diff-cHJvZ2xlYXJuL3Byb2dyZXNzaXZlX2xlYXJuZXIucHk=) | `87.00% <ø> (ø)` | | | [proglearn/voters.py](https://codecov.io/gh/neurodata/ProgLearn/pull/547/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=neurodata#diff-cHJvZ2xlYXJuL3ZvdGVycy5weQ==) | `74.71% <0.00%> (-18.40%)` | :arrow_down: | 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=neurodata). 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=neurodata)

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