Fixed your comments! Stepan, please rebuild the release and test! чт, 13 февр. 2020 г. в 17:22, Alexey Zinoviev <[email protected]>:
> Should I wait any ML-related issues in examples? > I could wait a day or two if you are going to continue your testing deals > with ML and examples to push changes together. > > Are you running your examples on the release staging as a ignite > dependency, isn't it? > > > чт, 13 февр. 2020 г. в 14:37, Stepan Pilschikov <[email protected] > >: > >> filled ticket - https://issues.apache.org/jira/browse/IGNITE-12673 >> >> чт, 13 февр. 2020 г. в 13:46, Alexey Zinoviev <[email protected]>: >> >>> I suppose one ticket is enough, collect where all your nites and thoughts >>> >>> чт, 13 февр. 2020 г., 12:12 Stepan Pilschikov <[email protected] >>> >: >>> >>>> And one (three) more thing >>>> >>>> In TutorialStepByStepExample we running 17 examples >>>> First 12 logging is pretty good and looks like "Tutorial step N: name" >>>> -> model -> accuracy -> "Tutorial step N: completed" >>>> But then starting with 13 this pattern is kind of broke, step start and >>>> step completion is missing >>>> >>>> Also want to admin that >>>> Step_8_CV_with_Param_Grid_and_metrics_and_pipeline is haven't step >>>> completion log >>>> And complete log for Step_9_Scaling_With_Stacking looks like 'Tutorial >>>> step 5 (scaling) example completed' >>>> >>>> i don't know, should i made some ticket? One ticket for each problem or >>>> one for all or you can fix it in one commit? because fixes quite small >>>> >>>> ср, 12 февр. 2020 г. в 16:31, Alexey Zinoviev <[email protected]>: >>>> >>>>> Yes, you are right. Missed resources, I'll remove them tomorrow >>>>> >>>>> ср, 12 февр. 2020 г. в 16:23, Stepan Pilschikov < >>>>> [email protected]>: >>>>> >>>>>> With mnist_tf_model in same directory >>>>>> All others looks right >>>>>> >>>>>> ср, 12 февр. 2020 г. в 16:21, Stepan Pilschikov < >>>>>> [email protected]>: >>>>>> >>>>>>> Hmm, interesting, got it >>>>>>> Also found >>>>>>> apache-ignite-2.8.0-bin/examples/src/main/resources/models/mleap >>>>>>> I think its also should be removed then >>>>>>> >>>>>>> ср, 12 февр. 2020 г. в 16:11, Alexey Zinoviev < >>>>>>> [email protected]>: >>>>>>> >>>>>>>> Please, have a look to the next ticket >>>>>>>> https://issues.apache.org/jira/browse/IGNITE-12659 >>>>>>>> Also tensorflow packages are blockers for IGFS deprecation and >>>>>>>> removal. >>>>>>>> >>>>>>>> ср, 12 февр. 2020 г. в 15:58, Alexey Zinoviev < >>>>>>>> [email protected]>: >>>>>>>> >>>>>>>>> They are not lost, they are removed from release branch due to a >>>>>>>>> lot of bugs, cves and so on, don't worry about that. >>>>>>>>> >>>>>>>>> ср, 12 февр. 2020 г., 14:31 Stepan Pilschikov < >>>>>>>>> [email protected]>: >>>>>>>>> >>>>>>>>>> Also lost ignite-ml-tensorflow-model-parser and ignite-tensorflow >>>>>>>>>> in libs/optional >>>>>>>>>> >>>>>>>>>> >>>>>>>>>> ср, 12 февр. 2020 г. в 14:25, Stepan Pilschikov < >>>>>>>>>> [email protected]>: >>>>>>>>>> >>>>>>>>>>> But now have new problems >>>>>>>>>>> I build ignite-2.8 build on TC >>>>>>>>>>> https://ci.ignite.apache.org/buildConfiguration/Releases_ApacheIgniteMain_ReleaseBuild/5043672 >>>>>>>>>>> Previous >>>>>>>>>>> https://ci.ignite.apache.org/buildConfiguration/Releases_ApacheIgniteMain_ReleaseBuild/4988324 >>>>>>>>>>> >>>>>>>>>>> New build much lighter >>>>>>>>>>> Previous: 5.7 Gb >>>>>>>>>>> Current: 4.3 Gb >>>>>>>>>>> And its looks like something missing, on a first look i notice >>>>>>>>>>> that /libs/optional/ignite-ml-mleap-model-parser lost completely >>>>>>>>>>> >>>>>>>>>>> >>>>>>>>>>> ср, 12 февр. 2020 г. в 14:18, Stepan Pilschikov < >>>>>>>>>>> [email protected]>: >>>>>>>>>>> >>>>>>>>>>>> Thanks for fix >>>>>>>>>>>> Duplicated examples deleted and tutorial works on real (1+ >>>>>>>>>>>> node) cluster now >>>>>>>>>>>> I think ticket can be closed, have no power here >>>>>>>>>>>> >>>>>>>>>>>> вт, 11 февр. 2020 г. в 20:44, Alexey Zinoviev < >>>>>>>>>>>> [email protected]>: >>>>>>>>>>>> >>>>>>>>>>>>> Sorry, for that, but I it touches ML-related stuff only and >>>>>>>>>>>>> doesn't influence on any another module. >>>>>>>>>>>>> I merged them to master later (currently, I have a 4 tickets >>>>>>>>>>>>> in a queue). >>>>>>>>>>>>> >>>>>>>>>>>>> Will keep in mind for the next fixes >>>>>>>>>>>>> >>>>>>>>>>>>> вт, 11 февр. 2020 г. в 18:58, Maxim Muzafarov < >>>>>>>>>>>>> [email protected]>: >>>>>>>>>>>>> >>>>>>>>>>>>>> Alexey, >>>>>>>>>>>>>> >>>>>>>>>>>>>> Is the approach when we cherry-pick fixes to the 2.8 only >>>>>>>>>>>>>> after all >>>>>>>>>>>>>> fixes have verified in the master branch better? I just want >>>>>>>>>>>>>> to be >>>>>>>>>>>>>> sure the release branch to be stable enough. >>>>>>>>>>>>>> >>>>>>>>>>>>>> On Tue, 11 Feb 2020 at 18:36, Alexey Zinoviev < >>>>>>>>>>>>>> [email protected]> wrote: >>>>>>>>>>>>>> > >>>>>>>>>>>>>> > Fixed both in release branch 2.8. Please verify and let me >>>>>>>>>>>>>> know >>>>>>>>>>>>>> > >>>>>>>>>>>>>> > вт, 11 февр. 2020 г. в 14:58, Stepan Pilschikov < >>>>>>>>>>>>>> [email protected]>: >>>>>>>>>>>>>> > >>>>>>>>>>>>>> > > And one more >>>>>>>>>>>>>> > > https://issues.apache.org/jira/browse/IGNITE-12658 - >>>>>>>>>>>>>> > > TutorialStepByStepExample example failed if cluster with >>>>>>>>>>>>>> more then 1 node >>>>>>>>>>>>>> > > >>>>>>>>>>>>>> > > вт, 11 февр. 2020 г. в 11:52, Alexey Zinoviev < >>>>>>>>>>>>>> [email protected]>: >>>>>>>>>>>>>> > > >>>>>>>>>>>>>> > >> Great, thank you so much, will fix next week >>>>>>>>>>>>>> > >> >>>>>>>>>>>>>> > >> вт, 11 февр. 2020 г., 11:39 Stepan Pilschikov < >>>>>>>>>>>>>> [email protected] >>>>>>>>>>>>>> > >> >: >>>>>>>>>>>>>> > >> >>>>>>>>>>>>>> > >>> Hi, Alexey, could you please looking on one small >>>>>>>>>>>>>> accident happen in ML >>>>>>>>>>>>>> > >>> examples: >>>>>>>>>>>>>> > >>> https://issues.apache.org/jira/browse/IGNITE-12657 - 2 >>>>>>>>>>>>>> examples exactly >>>>>>>>>>>>>> > >>> the same. >>>>>>>>>>>>>> > >>> >>>>>>>>>>>>>> > >>> Also want to say that all previous examples have one >>>>>>>>>>>>>> common approach for >>>>>>>>>>>>>> > >>> meaningful output, like ">>> Something important". In >>>>>>>>>>>>>> this two i seeing >>>>>>>>>>>>>> > >>> different one, could you please also pay attention to >>>>>>>>>>>>>> it. >>>>>>>>>>>>>> > >>> >>>>>>>>>>>>>> > >>> Best regards, >>>>>>>>>>>>>> > >>> Stepan >>>>>>>>>>>>>> > >>> >>>>>>>>>>>>>> > >> >>>>>>>>>>>>>> >>>>>>>>>>>>>
