Skip to content
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

Merged
merged 57 commits into from
May 13, 2024
Merged

Multilevel Coordinate Search (MCS) #353

merged 57 commits into from
May 13, 2024

Conversation

nforsg
Copy link
Contributor

@nforsg nforsg commented Mar 14, 2024

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.

@nforsg
Copy link
Contributor Author

nforsg commented Mar 14, 2024

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.

@Limmen
Copy link
Owner

Limmen commented Mar 15, 2024

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

@Limmen
Copy link
Owner

Limmen commented Mar 15, 2024

Radera det klonade biblioteket och kör pip install -U scikit-learn

@nforsg
Copy link
Contributor Author

nforsg commented Mar 15, 2024

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
Jag provade bara klona ner det och köra deras tester för att förstå mig på det från början, men då fick jag en hel del errors pga att det är implementerat i python 2.7 och framförallt gammal syntax för numpy och imports. Ska prova nu när jag uinstallerat

@nforsg
Copy link
Contributor Author

nforsg commented Mar 15, 2024

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:/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.

@Limmen
Copy link
Owner

Limmen commented Mar 15, 2024

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:/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.

@nforsg
Copy link
Contributor Author

nforsg commented Mar 15, 2024

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:/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å.

@nforsg
Copy link
Contributor Author

nforsg commented Mar 18, 2024

Har refaktorerat från detta bibliotek nu:
https:/vojha-code/Multilevel-Coordinate-Search

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.

@Limmen
Copy link
Owner

Limmen commented Mar 18, 2024

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å.

@nforsg
Copy link
Contributor Author

nforsg commented Apr 2, 2024

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!

@nforsg
Copy link
Contributor Author

nforsg commented Apr 4, 2024

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.

@Limmen
Copy link
Owner

Limmen commented Apr 4, 2024

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:

git config --global --unset credential.helper
git config credential.helper store
git push origin master

Det borde då fråga efter en nytt password och då ska du ange din Github token för ditt konto

@nforsg
Copy link
Contributor Author

nforsg commented Apr 4, 2024

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:

git config --global --unset credential.helper
git config credential.helper store
git push origin master

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!

@nforsg
Copy link
Contributor Author

nforsg commented Apr 19, 2024

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.

@Limmen
Copy link
Owner

Limmen commented Apr 23, 2024

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.

@nforsg
Copy link
Contributor Author

nforsg commented Apr 26, 2024

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.

@Limmen Limmen changed the title import errors Multilevel Coordinate Search (MCS) Apr 28, 2024
@Limmen Limmen merged commit 45721bd into master May 13, 2024
@Limmen Limmen deleted the dev branch May 15, 2024 13:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants