Allen Lee

icon indicating a website link https://search.asu.edu/profile/956135

icon company Arizona State University icon location Arizona icon location Tech lead for open computational social science and complex adaptive systems research @comses @openmodelingfoundation @virtualcommons @openskope @carpentries

Results 147 comments of Allen Lee

Update 14-looping-data-sets.md

Agreed, thanks for the PR!

Replace the basic summation example with a simple simulation of coin-…

Thanks for the PR! I like this change personally but would like to gather feedback from the rest of the community as well before moving forward. I believe the original...

make sure plot elements are not truncated

I think it's a good addition as well. Only thing a bit leery of is the caveat in the docs `This is an experimental feature and may not work for...

Possible incorrect link to Anaconda install instructions

oof, thanks for letting us know! I think this is a broader question for the carpentries python maintainers and curriculum advisors @swcarpentry/curriculum-advisors as I understand it atm these are shared...

Use pop() instead of del in the Lists lesson?

I think this is a quite reasonable suggestion, especially as the [novice-inflammation lesson](https://github.com/swcarpentry/python-novice-inflammation/pull/686) articulates well, staying consistent with list [operations and methods](http://swcarpentry.github.io/python-novice-gapminder/11-lists/index.html#appending-items-to-a-list-lengthens-it) as opposed to introducing a new language construct....

Use pop() instead of del in the Lists lesson?

I think this StackOverflow post articulates well the differences between the two and I like the answer as well: https://stackoverflow.com/questions/24547223/is-del-or-pop-preferred-when-removing-elements-from-dicts `del` makes it clear you are only interested in removing...

Examples / exercises encourage anti-patterns in 12-for-loops.md

Thanks for the feedback! Minor pedantic note that the simplest way to sum to 10 doesn't need the first parameter, `sum(range(11))` suffices. The string reverse slice operator requires a clear...

generate openapi schema for comses.net

I think we can table this for the time being in favor of setting up an explicit metadata page like https://ascl.net/home/getwp/3297

update digest archives

sounds good! would be nice to revamp how those secondary tab menus are set up; it's kind of a pain at the moment and involves a fair bit of duplication....

provide automated DOI registration

> The core doi creation logic seems alright to me, the only real improvement I can think of other than minor cleanup is doing metadata updates on post_save signals rather...