-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Hitting NoTransportOrigin
from weighted measures
#93
Comments
The thing is, transport needs both
|
With If you replace |
I made a little proof of concept here: The advantage is that the approach is very similar to what we do with The other side of this is, it's entirely possible that we might be better off dropping the current basemeasure approach altogether and instead changing that to match the transport implementation in master. |
The "transport origin" of a
WeightedMeasure
needs to be weighted. I had thought this might be easy:but this leads us to
I understand that this is just how dispatch is set up for transports, but it's bitten me a few times, and I think it will be at least as confusing for new users as it has been for me. I think we can keep the current high-level functionality but change the dispatch patterns under the hood so it works more like
basemeasure
. That would also help new devs, because getting used to one (basemeasures or transports) will make the other easier.@oschulz I could try to prototype this, but I know you're also working on improving type stability for transports, and I don't want us to collide more than we have to :)
The text was updated successfully, but these errors were encountered: