lanl / dsi

LANL Data Science Infrastructure Project
https://lanl.github.io/dsi
5 stars 4 forks source link

Danieljohnson/issue17 #24

Closed DanielRJohnson closed 1 year ago

DanielRJohnson commented 1 year ago

Merges EnvProvPlugin + tests into the most recent qwofford/issue14_refineplugin work.

qwofford commented 1 year ago

That Plugin class hierarchy looks really nice! Good work figuring that out. I'll see if I can incorporate this into our documentation.

qwofford commented 1 year ago

I tried to simplify the plugin filename structure which seems to have broken the graph generator.

DanielRJohnson commented 1 year ago

Tests were failing because of an import typo ("StructuredMetadataPlugin" vs "StructuredMetaPlugin", fixed now), and changing one import fixed the graph generator.