8000 Image pipeline by v1docq · Pull Request #200 · aimclub/FEDOT · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Image pipeline #200

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

Jump to bottom
Merged
merged 32 commits into from
Apr 22, 2021
Merged

Image pipeline #200

merged 32 commits into from
Apr 22, 2021

Conversation

v1docq
Copy link
Collaborator
@v1docq v1docq commented Dec 8, 2020

[InputData] Support images as InputData

@v1docq v1docq self-assigned this Dec 8, 2020
@v1docq v1docq linked an issue Dec 8, 2020 that may be closed by this pull request
[InputData] Support images as InputData #197
Closed
@nicl-nno nicl-nno mentioned this pull request Dec 17, 2020
Merged
4 tasks
v1docq added 3 commits December 30, 2020 18:12
Merge remote-tracking branch 'origin/master' into image_pipeline
36f6bf3 
# Conflicts:
#	fedot/core/data/data.py
#	fedot/core/repository/data/model_repository.json
#	requirements.txt
minor fixes
16040a2
update requirements
61c9646
@codecov
Copy link
codecov bot commented Dec 30, 2020 •
edited
Loading

Codecov Report

Merging #200 (ef9fccb) into master (2a054fe) will decrease coverage by 0.09%.
The diff coverage is 88.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #200      +/-   ##
==========================================
- Coverage   81.71%   81.61%   -0.10%     
==========================================
  Files          79       80       +1     
  Lines        6033     6131      +98     
==========================================
+ Hits         4930     5004      +74     
- Misses       1103     1127      +24
Impacted Files Coverage Δ
fedot/core/utils.py 68.62% <ø> (ø)
...aluation/operation_implementations/models/keras.py 86.20% <86.20%> (ø)
fedot/core/data/data.py 78.08% <100.00%> (+1.43%) ⬆️
fedot/core/operations/evaluation/classification.py 89.04% <100.00%> (+0.15%) ⬆️
fedot/core/repository/dataset_types.py 100.00% <100.00%> (ø)
...composer/optimisers/gp_comp/operators/crossover.py 77.08% <0.00%> (-14.59%) ⬇️
.../composer/optimisers/gp_comp/operators/mutation.py 70.65% <0.00%> (-8.70%) ⬇️
...on_implementations/models/discriminant_analysis.py 88.37% <0.00%> (-2.33%) ⬇️
...oser/optimisers/gp_comp/param_free_gp_optimiser.py 76.77% <0.00%> (-0.65%) ⬇️
... and 2 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 2a054fe...ef9fccb. Read the comment docs.

nicl-nno
nicl-nno requested changes Jan 3, 2021
View reviewed changes
v1docq added 2 commits March 30, 2021 13:56
Merge remote-tracking branch 'origin/master' into image_pipeline
df3b431 
# Conflicts:
#	fedot/core/data/data.py
#	fedot/core/utils.py
#	requirements.txt
add composite model and fixes
a5f4a45
@pep8speaks
Copy link
pep8speaks commented Mar 30, 2021 •
edited
Loading

Hello @v1docq! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 11:121: E501 line too long (121 > 120 characters)
Line 198:27: E712 comparison to True should be 'if cond is True:' or 'if cond:'

Comment last updated at 2021-04-22 12:03:15 UTC

nicl-nno
nicl-nno reviewed Mar 31, 2021
View reviewed changes
nicl-nno
nicl-nno reviewed Mar 31, 2021
View reviewed changes
nicl-nno
nicl-nno requested changes Mar 31, 2021
View reviewed changes
nicl-nno
nicl-nno requested changes Apr 1, 2021
View reviewed changes
nicl-nno
nicl-nno requested changes Apr 1, 2021
View reviewed changes
Copy link
Collaborator
@nicl-nno nicl-nno left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ещё пара комментов

v1docq added 3 commits April 5, 2021 20:08
test reworked v 1.05
c0e9def
Merge remote-tracking branch 'origin/master' into image_pipeline
3f092ad 
# Conflicts:
#	fedot/core/data/preprocessing.py
#	fedot/core/models/evaluation/keras_eval.py
#	fedot/core/repository/data/model_repository.json
#	test/unit/data_operations/test_preprocessing.py
update after preprocessing
0757157
nicl-nno
nicl-nno reviewed Apr 16, 2021
View reviewed changes
@nicl-nno nicl-nno requested a review from J3FALL April 16, 2021 11:33
J3FALL
J3FALL suggested changes Apr 20, 2021
View reviewed changes
J3FALL
J3FALL suggested changes Apr 22, 2021
View reviewed changes
@v1docq v1docq merged commit 2ed17b1 into master Apr 22, 2021
J3FALL pushed a commit that referenced this pull request Apr 22, 2021
@v1docq @J3FALL
Image pipeline (#200)
24aeae0 
* Image pipeline implementation
J3FALL added a commit that referenced this pull request Apr 22, 2021
@J3FALL
Revert "Image pipeline (#200)"
9177ba8 
This reverts commit 24aeae0.
@nicl-nno nicl-nno deleted the image_pipeline branch May 17, 2021 16:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nicl-nno nicl-nno nicl-nno approved these changes

@J3FALL J3FALL J3FALL approved these changes

Assignees

@v1docq v1docq

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[InputData] Support images as InputData
4 participants
@v1docq @pep8speaks @nicl-nno @J3FALL
0