-
Notifications
You must be signed in to change notification settings - Fork 20
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.
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
Multilevel Coordinate Search (MCS) #353
Conversation
jag förstår inte riktigt varför jag får modulenotfounderrors när jag kör msc.py i scikits, det ska egentligen inte hända. Jag initierar setup-filen i yttre foldret med "pip install -e .", det löser en del errors, men criterion ska ju finnas som modul och funka i foldern "optimization". Jag jobbar på det, i värsta fall får jag implementera denna själv. |
Behöver du göra ändringar i scikit-biblioteket? Annars finns det ingen anledning att klona det då kan du göra pip install istället |
Radera det klonade biblioteket och kör |
|
Alltså det kan vara så att jag missförstått hur man använder det här, men biblioteket sciki-learn https://scikit-learn.org/stable/index.html verkar inte ha något att göra med det biblioteket du skickade via mailen: https://github.com/mbrucher/scikit-optimization eller har jag fel? det är nämligen 12 år gammalt, när man kör pip install -U scikit-learn installerar man ett nyare bibliotek (det som är länkat) det har inte mcs-alghoritmen. |
Du har rätt, sorry. Förvirrande att de har samma namn jag trodde den fanns i officiella scikit. Då kan vi strunta i att förlita oss på 3:e parts biblioteket och implementera MCS från scratch, men vi kan fortfarande ta inspiration från källkoden i det där gamla biblioteket. |
Okej, då fattar jag, ingen fara! Yes då gör jag så. |
Har refaktorerat från detta bibliotek nu: Upplever att algoritmen är ganska komplicerad, men nu tror jag att jag ska kunna få till körningar och därefter göra en lite bättre implementation än den befintliga. |
Låter bra, börja med att få den att köra sedan kan vi börja förfina koden. I PR just nu är det 62 nya filer, det borde gå att få ner det till 5 eller så. |
Kanske nämnde det när jag var förbi senast, men jobbar torsdag och fredag denna vecka då tentan är klar imorgon kl 13! |
Har jobbat nu under förmiddagen och det är som jag beskrev senast, eval_theta är det som slukar, mcs_agent fungerar felfritt om man istället kör en enkel 2:a-gradare. Så jag jobbar på att få till en sabb implementation av den. Har däremot svårt att pusha koden till github, då jag får "Authentication failed", jag kom ihåg från i somras att det var någon liten grej som gjorde att det inte fungerade, men att det var svårt att hitta exakt vad det var, har du stött på samma problem? Jag förnyade min acces token för en vecka sen, men det brukar inte orsaka några problem. Jag jobbar på under tiden. |
Det kan vara att din authentication token har gått ut. Du kan testa:
Det borde då fråga efter en nytt password och då ska du ange din Github token för ditt konto |
Juste, det vara bara det att jag glömde att token var lösenordet, tack! |
Nu är den hyfsat redo för review. Det är eventuellt några buggar som är lite svåra att detektera, då dom sällan utgör utfall av algoritmen. Jag har i stort sett rensat linter-meddelandena, men det är några väldigt märkliga implementationer som strider mot linter men som ger error vid körning om man gör "rätt", framförallt rad 24 och 29 i gls_utils. Utöver det är en en del variabel-/parameternamn som kanske behöver förfinas. Men loggningen tror jag är rätt bra, och den gör nog det den ska nu. |
Bra. Du kan fortsätta förfina koden så mycket du kan (refaktorera, kommentera, type-hinta, etc) vid de kvarstående jobbtillfällena så mergar jag efter det. |
Har gjort klart en del linter och mypy idag samt initierat en test-suite, fortsätter i helgen pga jag behöver göra annat idag. |
Har provat klona ner scikit-optimization och köra lokalt, men har haft problem med ett import-error på en fil som heter default än sålänge, jobbarmed att fixa det.