DurhamARC / classroom-abm

Agent-based modelling for a classroom
MIT License
2 stars 2 forks source link

Automated runs #112

Closed alisonrclarke closed 3 years ago

alisonrclarke commented 3 years ago
alisonrclarke commented 3 years ago

Great! I can't fault this.

My one question is why click is no longer used for the cli - I felt that made it clear in the code what the args do.

Which cli do you mean?

MarkLTurner commented 3 years ago

Great! I can't fault this. My one question is why click is no longer used for the cli - I felt that made it clear in the code what the args do.

Which cli do you mean?

I was thinking of the code in hamilton/parameter_analysis - for example, merge_best_results.py

alisonrclarke commented 3 years ago

Great! I can't fault this. My one question is why click is no longer used for the cli - I felt that made it clear in the code what the args do.

Which cli do you mean?

I was thinking of the code in hamilton/parameter_analysis - for example, merge_best_results.py

Ah, I see - I thought you meant I'd removed click from places it was already used!

Yes, I'll look into adding click to some of these scripts - they're all a bit of a mess and at least click would make them self-documenting!