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

Add elixir implementation #42

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Munksgaard
Copy link
Contributor

It is very slow... Here's the result of running make on my laptop:

Name             ips        average  deviation         median         99th %
irreg          0.110         9.08 s     ±2.55%         9.08 s         9.24 s
rgbbox        0.0591        16.91 s     ±0.00%        16.91 s        16.91 s

Copy link
Owner

@athas athas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Elixir isn't really a parallel functional language, but I suppose it's close enough.

I trust this code is reasonable enough that the Elixir people won't come running and be mad at me, right?

I'll merge it if I manage to run it on my own machine.

elixir/README.md Outdated Show resolved Hide resolved
@Munksgaard
Copy link
Contributor Author

Munksgaard commented Jul 21, 2023

I trust this code is reasonable enough that the Elixir people won't come running and be mad at me, right?

I probably wouldn't publish the numbers loud and proud yet. It doesn't seem like it should be this slow. I'll take another look at some point next week and see if I can speed it up, but I just wanted to commit my work somewhere for now.

I'll merge it if I manage to run it on my own machine.

I think nix-shell -p elixir erlang --run make should be enough.

It is very slow... Here's the result of running `make` on my laptop:

```
Name             ips        average  deviation         median         99th %
irreg          0.110         9.08 s     ±2.55%         9.08 s         9.24 s
rgbbox        0.0591        16.91 s     ±0.00%        16.91 s        16.91 s
```
@athas
Copy link
Owner

athas commented Jul 21, 2023

I get this:

Name             ips        average  deviation         median         99th %
irreg          0.148         6.76 s     ±0.85%         6.76 s         6.80 s
rgbbox         0.112         8.89 s     ±0.07%         8.89 s         8.89 s

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