Marnik Bercx

Results 301 comments of Marnik Bercx

Will leave this open for now in case we still want to make this material available in a branch.

Thanks for raising the issue, @ltalirz! I agree it makes sense to parse these and _at least_ add a warning to the report. If the charge density is _very_ negative,...

Thanks for raising the issue, @ltalirz! I agree it makes sense to parse these and _at least_ add a warning to the report. If the charge density is _very_ negative,...

@eimrek pretty much already implemented this for some EMPA tools, by the way: https://github.com/nanotech-empa/aiidalab-empa-nanoribbons/blob/7bc965f8a318165da8aac4bcd1ca8c78e143f284/nanoribbon/viewers/pdos_computed.py#L196 So it should just be a matter of integrating his work into the plugin, if he...

> Subsections are fine because only (Module) sections have subsections, but it is a bit confusing to have (tutorial) sections and (module) sections. I agree, I've already struggled with this...

I'm starting to get used to calling them "Units" right now: - Unit - Module - Section - Subsection This is also the convention I've used in the changes for...

Thanks for highlighting this @giovannipizzi, I'll have a closer look.

If we do decide to rely on the attribute syntax, we should really explain the concept of an `AttributeDict` somewhere. What do you think @ramirezfranciscof, @ltalirz?

I only quickly parsed the issue, but maybe this is something that should be opened on https://github.com/marvel-nccr/quantum-mobile? Pinging @chrisjsewell to see if he knows of any problems with downloading the...