Skip to content

fixed corner case [1,1].lcmByFactors#3

Open
chris75vie wants to merge 1 commit intodkmayer:masterfrom
chris75vie:master
Open

fixed corner case [1,1].lcmByFactors#3
chris75vie wants to merge 1 commit intodkmayer:masterfrom
chris75vie:master

Conversation

@chris75vie
Copy link

I'd expect that following lines result the same value

[1,1].reduce(\lcm);
[1,1].lcmByFactors[1];

Actually an empty list is returned in this case.
Probably there's a more elegant way to implement this behaviour.

@dkmayer
Copy link
Owner

dkmayer commented Jun 12, 2021

Hi, thanks for the hint !
I'll add in the next offical version (which might take time).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants