Chahak Mehta
Chahak Mehta
> I don't think you should snake the scaling through to the low-level object. get_sizes should just return what it's always returned, with an explanation in the docs. My only...
I understand that, but wouldn't having the default behaviour such that it doesn't change anything that already exists resolve that issue? And if they are instead using `markersize` then they...
Meeting 9/3: Think instead about making `markersize` an alias for `s` and have a scaling factor as the kwarg to control all this. Also look into how to make this...
@jklymak sorry, I should've put a comment in earlier but I've changed `markersize` to be an alias for `s` and added a `markerscale` variable as we discussed last time in...
Sorry for the label, was looking for something else!
 Ran optimization with 2D Uniaxial stress example. Given stress, estimated Young's Modulus. This is with the stateless implementation. This is possible because https://github.com/geoelements/diffmpm/pull/27/commits/1034d4f03c2d9e72277137befc3a58333047875a changes all for loops to scans...
Thanks for pointing this out. I also realised this and #27 should resolve this too.
I haven't implemented state variables yet but I think it's just a dictionary. See https://github.com/cb-geo/mpm/blob/53fcb7609375fd44bf3481d24ee18bfdf0393adb/include/materials/bingham.tcc#L32-L37
Thanks for pointing that out. I added it to the init method. The reason it worked fine was that every time a new value is assigned to it so it...
@SachinJalan Let me see if I can find something. Can you be a little clearer about where you think the error is stemming from? Trying to pin in the issue...