-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Final submission #669
Final submission #669
Conversation
it seems there's another trick to try: 1brc/src/main/java/dev/morling/onebrc/CalculateAverage_serkan_ozal.java Lines 171 to 176 in 7f0e517
along with 1brc/calculate_average_serkan-ozal.sh Lines 31 to 35 in 7f0e517
|
Nice!
|
Hey @merykitty, couldn't find another way to reach out to you, so commenting here: would you perhaps be interested in teaming up with Thomas, Roy, Elliot, and me for doing a live stream on #1BRC tomorrow with Nicolai Parlog: https://twitter.com/gunnarmorling/status/1752288991909478611. As one of the fastest JVM-based implementations, it would be great to have your perspective on the challenge. Let me know :) |
@gunnarmorling I would be more than happy to join. Can you give me more information regarding the subjects or what can I contribute to the conversation? My email is [email protected]. |
Perfect, I've added you to the DM chat with the other participants. Thanks
a lot, looking forward to it!
… Message ID: ***@***.***>
|
Some small improvements:
min/max
Some that I have tried:
Unsafe
: Spawning a new segment each time introduces regression, most likely due to failures to inline even with that threshold of inlining, and using aLong.MAX_VALUE
-size segment introduces additional bound checks.Check List:
./mvnw verify
and the project builds successfully./test.sh <username>
shows no differences between expected and actual outputs)calculate_average_<username>.sh
(make sure to match casing of your GH user name) and is executablecalculate_average_baseline.sh