resolve an ambiguity

[putianyi889]

Apparently Base._sum must accept a dim argument, otherwise _sum(f, A::something) and _sum(A::something, dim) technically introduce an ambiguity.

[dlfivefifty]

Can you add a test for the ambiguity?

[codecov]

Codecov Report

Attention: 3 lines in your changes are missing coverage. Please review.

Comparison is base (1f44868) 64.03% compared to head (78ed98d) 64.08%.

Files	Patch %	Lines
src/quasifill.jl	0.00%	3 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #102      +/-   ##
==========================================
+ Coverage   64.03%   64.08%   +0.05%     
==========================================
  Files          23       23              
  Lines        1824     1824              
==========================================
+ Hits         1168     1169       +1     
+ Misses        656      655       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[putianyi889]

Can you add a test for the ambiguity?

The ambiguity doesn't have any actual meaning I suppose. It just messes with the ambiguity check.

As for the coverage, I don't add any new lines so it doesn't change. Apparently the lines are never covered before.

[dlfivefifty]

But can you add tests that cover those lines? We need to make sure it hasn't broken anything.

[putianyi889]

I don't even know the use case of QuasiFill. Is there a downstream that depends on it?

[dlfivefifty]

representing constant functions

[putianyi889]

added tests