Rémi Delaporte-Mathurin

Results 247 comments of Rémi Delaporte-Mathurin

@jhdark reopening this

When updating to 0.8.0 I've started noticing [errors in our CI ](https://github.com/festim-dev/FESTIM/actions/runs/8854180067/job/24374186867?pr=764#step:6:45) I also noticed them in the Docker CI though so not sure it's related to this?

`thickness` is an attribute of `BlanketCutterParallels` which is kinda translated into a `distance` attribute for `ExtrudeStraightShape`. A fix would be to have a setter for thickness that also modifies `distance`.

I noticed that a while ago, and it was a tad annoying 😸

One way to solve this would be to have two separate attributes `points` and `connections`. One would only contain coordinates and the latter will only contain strings?

@Shimwell I think this can be closed since it was solved in #839

https://github.com/ukaea/paramak/blob/run_demos/examples/example_parametric_reactors/ball_reactor.ipynb I tried to run some demos locally and push them but Github isn't displaying anything.

Several ideas: - Create a interactive tutorial as in https://jorgensd.github.io/dolfinx-tutorial/index.html - Have static images in the demos notebooks that are automatically updated at every push to dev or main (as...

@Shimwell just to be clear, in the snipset you posted, shouldn't the test for MixedShape points ensure values is a list of list of length 3 (X, Y, connection)?

Additionnaly, this would require to rewrite the @property as well: Example in RotateMixedShape ```python @property def points(self): return super().points @points.setter def points(self, values_in): super(RotateMixedShape, self.__class__).points.fset(self, values_in) ```