Elijah ben Izzy
Elijah ben Izzy
**Is your feature request related to a problem? Please describe.** Its a cool feature but its basically unusable. Current limitations: 1. `@does` is limiting -- has to have a function...
Short description explaining the high-level reason for the new issue. # Current behavior It breaks, dependencies marked as optional are not carried over. ## Stack Traces (If applicable) ## Screenshots...
IMO this is actually just sloppiness on the implementation of lru_cache, not layerable. Need to verify 100% that this is the cause, but we should fix. # Current behavior ##...
Short description explaining the high-level reason for the new issue. # Current behavior This breaks: ```python def foo() -> frozenset[str]: ... def bar(foo: Set[str]) -> ...: ... ``` But it...
[Short description explaining the high-level reason for the pull request] ## Changes ## How I tested this ## Notes ## Checklist - [ ] PR has an informative and human-readable...
[Short description explaining the high-level reason for the pull request] A few minor changes -- just playing around with this. Intent is not to merge. ## Changes ## How I...
inspect.getmodule(fn) depends on the state of sys.modules. This can break if you're doing repeated parsing -- E.G. running it then wiping sys.modules. Instead, we can just ask the function for...
https://github.com/stitchfix/hamilton/discussions/276 This is a little rough -- we need more testing. But it should work. Followed the spec at it was fairly clean -- using typed dicts, and validating on...
--- PR TEMPLATE INSTRUCTIONS (1) --- Looking to submit a Hamilton Dataflow to the sf-hamilton-contrib module? If so go the the `Preview` tab and select the appropriate sub-template: * [sf-hamilton-contrib...
**Is your feature request related to a problem? Please describe.** [StaticValidator](https://github.com/DAGWorks-Inc/hamilton/blob/788040a6217165f36c3bd3e3316f6884f924a4ad/hamilton/lifecycle/api.py#L545) does, but the rest don't. We should fix this. We just need to keep it backwards compatible, and we...