Slackbot
12/12/2023, 9:25 AMElijah Ben Izzy
12/12/2023, 3:29 PMad_hoc_utils.create_temporary_module
). So, while its fine for classes to co-exist along with functions that get turned into nodes, the hamilton parser/converter will not be aware of them, and thus not include them in visualization.
We should probably be a bit more explicit, but this is close(ish):
• https://hamilton.dagworks.io/en/latest/concepts/hamilton-function-structure/#modules-and-helper-functions
What are you trying to do?Roel Bertens
12/14/2023, 10:59 AMRoel Bertens
12/14/2023, 11:00 AMElijah Ben Izzy
12/14/2023, 7:32 PMRoel Bertens
12/15/2023, 11:20 AMRoel Bertens
12/15/2023, 12:19 PMElijah Ben Izzy
12/15/2023, 8:42 PM@with_columns
it works, but that’s slightly overkill for df -> df functions. That said, I think that we can probably find a way to add that in. One option would be to wrap the @tag
decorator to signify the column names. Then, we could customize the viz slightly (its not that complicated) to display the names of the columns as part of the node for a spark df — I think it would be a valuable thing. It could be its own node (and you could do this with some basic graph manipulations), or it could just show up on the viz…
2. There’s a parameter to turn this off!
3. So yeah, we don’t have documentation generation/cataloging (except through the DAGWorks product, which is free to try/use in a limited quantity, happy to walk you through), but from an OS perspective, you have the pieces. You should use @tag
to indicate which nodes you want then Driver.list_available_variables(…)
. @miek recently looked into this, might have some insights!
If you want, I’d be happy to sit down and hack on this for an hour/walk you through some pieces next week if you’re interested. One thing I’m thinking about is a custom visualizer for the spark plugin that has all the bells and whistles — if you were interested in contributing I think we could make quick work of thisRoel Bertens
12/18/2023, 12:28 PMmiek
12/18/2023, 1:03 PMElijah Ben Izzy
12/18/2023, 5:57 PMRoel Bertens
12/18/2023, 7:19 PM