Benoit Bovy

Results 452 comments of Benoit Bovy

> I'm not sure if I completely understand what you want to accomplish ### Current problem Let me show a simple example: ```python @xs.process class ProcessA: foo = xs.variable(intent='inout') @xs.process...

> Aah, now that explains all the on_demand variables in fastscape?! :) Not exactly, `on_demand` variables are more for some model diagnostics (variables that are not useful in other processes...

> if an `inout` variable is declared, it does not have automatic dependencies. Therefore, the user can add them. Yes, exactly! The user *must* add them, except in the two...

> However, normally during normal dependency building, A->C would be there, so we have to skip that dependency using skip_deps. I don't think it's necessary to skip it. The topological...

Great! I commented on #177. > done that, maybe a set is better? it needs one conversion less here, but maybe users are not as familiar with sets? Yep internally...

Hi @Joeperdefloep, sorry for the wait, I've been silent in this repository for too long. `any_object` actually implicitly defines `VarIntent.OUT`, so I think in theory it shouldn't be possible to...

`xsimlab.Model` could provide a `default_setup` argument that accepts a dictionary with the exact same structure than the `xsimlab.create_setup` arguments (i.e., `clocks`, `input_vars`, `output_vars`). If `default_setup` is provided, then `xsimlab.create_setup` is...

Mmm I'm not sure that we really need something like `xsimlab.variable_info` to get variable metadata inside process classes: - Using an `attr.Attribute` to store an `attr.Attribute` looks a bit weird...

Thanks! > I'll come back here if/when I come up with more concrete examples. Yes please! This is very much appreciated!

Thinking again about this, a possible API would be: ```python # Get the value of `self.var` as a DataArray. If `self.var` is not a DataArray, # construct a new DataArray...