-
Notifications
You must be signed in to change notification settings - Fork 0
Pbl data merge #209
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
base: main
Are you sure you want to change the base?
Pbl data merge #209
Conversation
|
Alright, ik heb er even naar gekeken en er vallen mij een aantal dingen op, die eerst nog wel even opgelost moeten worden voor dat we het mergen met de main vind ik:
-> Laten we na de vakantie hier even rustig met zijn tweeen/drieen doorheen lopen om te kijken wat de beste aanpak is. |
Also a 'null' check has been added to certain parameters.
|
Misschien belangrijk om ook duidelijk te hebben welke van de pbl parameters verplicht zijn om de pbl methode te gebruiken, sommige? Allemaal? Aan de hand daarvan zouden we de null check ook kunnen maken. Voor nu is het even snell een boolean geworden die je op true moet zetten in je project specifieke loader. default is het false. |
1ste versie van loss factor o.b.v. pbl data.
Als het goed is werkt de code zonder pbl data nog gewoon op de oude manier maar check dat vooral even!
Als jullie vinden dat dit in override functies in de project specifieke loader moet zou ik dat nog om moeten bouwen, maar wat bij betreft is dit de beste data om heterogene huizen mee te parametriseren dus zou het goed zijn dat in een aantal stappen steeds verder in de default loader te brengen