Chahak Mehta

Results 33 comments of 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!

![image](https://github.com/geoelements/diffmpm/assets/29800965/1c59cec4-4301-4aef-903d-0d6511b2b9ea) 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...