Calc to agg from calc to agg in separate resource errors about ash_postgres

[zachdaniel]

This was harder than I thought it was going to be 😆 Can you try main please? It should be fixed.

[lifeofdan]

@zachdaniel After pulling main I now have a different error.

[zachdaniel]

🤔 I've added the resource we're using to hydrate the expression, perhaps that will add some insight?

[lifeofdan]

That gives a little insight? It is the post resource, although that was probably already apparent.

[zachdaniel]

This one is tough :) Still working on it.

[zachdaniel]

Going to take some time to address, will let you know when I've got it figured out.

[zachdaniel]

You know what...I just realized I've been chasing down something that we actually just don't currently support 😢 Specifically, aggregates can't use other aggregates as their field. In this case, it's an aggregate using a calculation that refers to an aggregate. I'd like to make this work, and it was always on the docket to do so, but it is quite complex and really amounts to an additional feature in some ways.

[lifeofdan]

So, if this doesn't work (which is fine, I understand this is a bit nested). Why does test "complex calculation while using actor on related resource passes reference" work? Isn't this also a calc referring to an agg referring to a calc that refers to an agg? What is the big difference between these two?

[zachdaniel]

That one just happens to work I think 😆 It has to do with the fact that they are "first optimizable aggregates" AFAICT.

[lifeofdan]

Interesting. I'm glad our other use case works as it is much more important to us! 😄

If you want to close this as wontfix that is okay. We will use another method to do what we need since this is not actually supported. My bad! Ty for your time on this. :)

[zachdaniel]

I'm going to see what I can do to make this work, but it may involve some changes in core and a bit more work :)

[zachdaniel]

AFAIK this is fixed :)