✨ write an adapter for banks-style PACTA data outputs

[jdhoffa]

Not sure how feasible this is, but if we could adapt this data generation repo to optionally take banks data as input, banks could also benefit from the dashboard:
https://rmi-pacta.github.io/pacta-dashboard-svelte/

Obviously no action required here yet
Just musing.

FYI @jacobvjk

[AlexAxthelm]

Seems like it might be better to have P4B output the expected format, than to have multiple formats of input on the workflow.pacta.* reporting tools

[jacobvjk]

@jdhoffa @AlexAxthelm I am not familiar with this repo at all yet. When you say "data generation repo" you are referring to this one or another one?

[jdhoffa]

Seems like it might be better to have P4B output the expected format, than to have multiple formats of input on the workflow.pacta.* reporting tools

That's a trickier ask, since r2dii is currently in production, and has several active users. Any breaking changes in output format would cause trouble for those users.

One option would be to create an adapter package

[jdhoffa]

@jdhoffa @AlexAxthelm I am not familiar with this repo at all yet. When you say "data generation repo" you are referring to this one or another one?

This repo. It's basically just a data formatting repo

[jacobvjk]

Seems like it might be better to have P4B output the expected format, than to have multiple formats of input on the workflow.pacta.* reporting tools

That's a trickier ask, since r2dii is currently in production, and has several active users. Any breaking changes in output format would cause trouble for those users.

One option would be to create an adapter package

yea, fundamentally changing p4b output format seems like a non-starter. Agree that either another repo or adding an adapter function somewhere may be more realistic

[AlexAxthelm]

Vote to move this issue to a different repo (with the idea that building a general adapter makes sense, since it could be used for Interactive report too)

[jdhoffa]

makes sense, I just don't think another (better) repo exists yet...
Maybe r2dii.analysis?