Skip to content
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

Math module #1088

Open
dullbananas opened this issue Jun 15, 2020 · 4 comments
Open

Math module #1088

dullbananas opened this issue Jun 15, 2020 · 4 comments
Labels
breaking would require a MAJOR release request

Comments

@dullbananas
Copy link

Basics module contains many functions and constants such as pi snd cos which should go in a separate Math module

@github-actions
Copy link

Thanks for reporting this! To set expectations:

  • Issues are reviewed in batches, so it can take some time to get a response.
  • Ask questions a community forum. You will get an answer quicker that way!
  • If you experience something similar, open a new issue. We like duplicates.

Finally, please be patient with the core team. They are trying their best with limited resources.

@rlefevre
Copy link
Member

Hi.

You don't actually present any argument in favor of such a change.
"should" is not an argument.

@dullbananas
Copy link
Author

  • A lot of the math stuff in Basics aren't common enough to need to automatically get imported with (..) compared to stuff like identity and I think it is very bad that the core module that has the most stuff is also the one that always gets automatically imported with (..).

  • Seeing Math.toPolar when reading code is way less confusing than toPolar since that function is not common or well known compared to other things in the core library.

@evancz evancz added request breaking would require a MAJOR release labels Feb 9, 2021
@dullbananas
Copy link
Author

This can also help make Elm more familiar to users of JavaScript, where many math functions are static methods of the Math object

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking would require a MAJOR release request
Projects
None yet
Development

No branches or pull requests

3 participants