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

trace_limit 제한 증가 (fix #27) #36

Merged
merged 1 commit into from
Apr 4, 2024
Merged

Conversation

puzzlet
Copy link
Member

@puzzlet puzzlet commented Apr 4, 2024

#27 은 PyPy의 버그가 아니라 trace_limit의 최대치 기본값이 30000 미만이어서 생긴 문제였습니다.

@youknowone
Copy link
Member

변경전 + trace_limit 기본값

❯ time ./rpaheui-orig --trace-limit=-1 snippets/logo/logo.aheui > /dev/null
./rpaheui-orig --trace-limit=-1 snippets/logo/logo.aheui > /dev/null 0.49s user 0.31s system 97% cpu 0.817 total
❯ time ./rpaheui-orig --trace-limit=-1 snippets/logo/logo.aheui > /dev/null
./rpaheui-orig --trace-limit=-1 snippets/logo/logo.aheui > /dev/null 0.49s user 0.30s system 98% cpu 0.799 total
❯ time ./rpaheui-orig --trace-limit=-1 snippets/logo/logo.aheui > /dev/null
./rpaheui-orig --trace-limit=-1 snippets/logo/logo.aheui > /dev/null 0.48s user 0.30s system 98% cpu 0.795 total
❯ time ./rpaheui-orig --trace-limit=-1 snippets/logo/logo.aheui > /dev/null
./rpaheui-orig --trace-limit=-1 snippets/logo/logo.aheui > /dev/null 0.49s user 0.30s system 98% cpu 0.807 total

변경전 + trace_limit 15000

❯ time ./rpaheui-orig --trace-limit=15000 snippets/logo/logo.aheui > /dev/null
./rpaheui-orig --trace-limit=15000 snippets/logo/logo.aheui > /dev/null 0.46s user 0.30s system 88% cpu 0.857 total
❯ time ./rpaheui-orig --trace-limit=15000 snippets/logo/logo.aheui > /dev/null
./rpaheui-orig --trace-limit=15000 snippets/logo/logo.aheui > /dev/null 0.46s user 0.29s system 97% cpu 0.770 total
❯ time ./rpaheui-orig --trace-limit=15000 snippets/logo/logo.aheui > /dev/null
./rpaheui-orig --trace-limit=15000 snippets/logo/logo.aheui > /dev/null 0.46s user 0.30s system 98% cpu 0.773 total
❯ time ./rpaheui-orig --trace-limit=15000 snippets/logo/logo.aheui > /dev/null
./rpaheui-orig --trace-limit=15000 snippets/logo/logo.aheui > /dev/null 0.46s user 0.29s system 98% cpu 0.763 total

변경후 + trace_limit 기본값

❯ time ./rpaheui-trace-big --trace-limit=-1 snippets/logo/logo.aheui > /dev/null
./rpaheui-trace-big --trace-limit=-1 snippets/logo/logo.aheui > /dev/null 0.49s user 0.31s system 96% cpu 0.821 total
❯ time ./rpaheui-trace-big --trace-limit=-1 snippets/logo/logo.aheui > /dev/null
./rpaheui-trace-big --trace-limit=-1 snippets/logo/logo.aheui > /dev/null 0.49s user 0.30s system 97% cpu 0.809 total
❯ time ./rpaheui-trace-big --trace-limit=-1 snippets/logo/logo.aheui > /dev/null
./rpaheui-trace-big --trace-limit=-1 snippets/logo/logo.aheui > /dev/null 0.49s user 0.30s system 98% cpu 0.804 total
❯ time ./rpaheui-trace-big --trace-limit=-1 snippets/logo/logo.aheui > /dev/null
./rpaheui-trace-big --trace-limit=-1 snippets/logo/logo.aheui > /dev/null 0.48s user 0.30s system 97% cpu 0.801 total

변경후 + trace_limit 15000

❯ time ./rpaheui-trace-big --trace-limit=15000 snippets/logo/logo.aheui > /dev/null
./rpaheui-trace-big --trace-limit=15000 snippets/logo/logo.aheui > /dev/null 0.46s user 0.30s system 97% cpu 0.784 total
❯ time ./rpaheui-trace-big --trace-limit=15000 snippets/logo/logo.aheui > /dev/null
./rpaheui-trace-big --trace-limit=15000 snippets/logo/logo.aheui > /dev/null 0.46s user 0.29s system 97% cpu 0.773 total
❯ time ./rpaheui-trace-big --trace-limit=15000 snippets/logo/logo.aheui > /dev/null
./rpaheui-trace-big --trace-limit=15000 snippets/logo/logo.aheui > /dev/null 0.46s user 0.30s system 98% cpu 0.771 total
❯ time ./rpaheui-trace-big --trace-limit=15000 snippets/logo/logo.aheui > /dev/null
./rpaheui-trace-big --trace-limit=15000 snippets/logo/logo.aheui > /dev/null 0.46s user 0.29s system 97% cpu 0.773 total

변경후 + trace_limit 30000

❯ time ./rpaheui-trace-big --trace-limit=30000 snippets/logo/logo.aheui > /dev/null
./rpaheui-trace-big --trace-limit=30000 snippets/logo/logo.aheui > /dev/null 0.45s user 0.29s system 97% cpu 0.762 total
❯ time ./rpaheui-trace-big --trace-limit=30000 snippets/logo/logo.aheui > /dev/null
./rpaheui-trace-big --trace-limit=30000 snippets/logo/logo.aheui > /dev/null 0.45s user 0.29s system 97% cpu 0.757 total
❯ time ./rpaheui-trace-big --trace-limit=30000 snippets/logo/logo.aheui > /dev/null
./rpaheui-trace-big --trace-limit=30000 snippets/logo/logo.aheui > /dev/null 0.45s user 0.29s system 97% cpu 0.764 total
❯ time ./rpaheui-trace-big --trace-limit=30000 snippets/logo/logo.aheui > /dev/null
./rpaheui-trace-big --trace-limit=30000 snippets/logo/logo.aheui > /dev/null 0.45s user 0.29s system 98% cpu 0.753 total

@youknowone youknowone merged commit 07088ed into aheui:main Apr 4, 2024
2 checks passed
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