Maxime Liquet

Results 368 comments of Maxime Liquet

*Recording some issues.* This snippet used to run fine and `allow_None`, despite being set to `False` in the Parameter constructor, is actually dynamically re-set to `True` internally (`P().param.s.allow_None` is `True`)....

Replying to my own comment above https://github.com/holoviz/param/pull/605#issuecomment-1438862047. This is the same behavior as *main*, which is correct, with `allow_None` being automatically set to `True` if the provided `default` is `None`....

I think I am going to stop there for now :) Early on I felt that the test suite was not good enough to capture all the changes that would...

Thanks for the review @jbednar, I applied your suggestions and renamed another function so that they follow a consistent naming scheme `_compute_...`. I also needed to make a small change...

Just noting that this will need some tests and documentation before being merged.

Indeed, because there's a lot of code in `__init__.py` the auto-completion suggests many irrelevant objects like `param.sys` or `param.OrderedDict`. Moving that code to another module would help. Or importing them...

My plan was to do this refactoring once there are a sufficient number of PRs merged, as that will cause a lot of conflicts.

Does it fix https://github.com/holoviz/param/issues/398 too?

I no longer see much value in the changes I made in this PR. As stated above`@param.depends()` is valid. `@param.depends` could either raise an error or be accepted and made...