Friday Facts #231 - Belt compression & Crash log uploading

Regular reports on Factorio development.

DanGio
Fast Inserter
Fast Inserter
Posts: 122
Joined: Sat May 10, 2014 6:22 pm

Re: Friday Facts #231 - Belt compression & Crash log uploading

Post by DanGio » Fri Feb 23, 2018 9:09 pm

Great !

User avatar
Gergely
Filter Inserter
Filter Inserter
Posts: 416
Joined: Sun Apr 10, 2016 8:31 pm

Re: Friday Facts #231 - Belt compression & Crash log uploading

Post by Gergely » Fri Feb 23, 2018 9:11 pm

Thank you for finally addressing privacy. I have been dealing with log files myself and I can't help but wonder why people still feel unsafe about it. OR/AND worry about those who do not even care.

I still recommend not even making this an option though.

ske
Filter Inserter
Filter Inserter
Posts: 319
Joined: Sat Oct 17, 2015 8:00 am

Re: Friday Facts #231 - Belt compression & Crash log uploading

Post by ske » Fri Feb 23, 2018 9:14 pm

Do you have an affected Ryzen processor yourself for running tests on?

Twinsen
Factorio Staff
Factorio Staff
Posts: 674
Joined: Tue Sep 23, 2014 7:10 am

Re: Friday Facts #231 - Belt compression & Crash log uploading

Post by Twinsen » Fri Feb 23, 2018 9:18 pm

ske wrote:Do you have an affected Ryzen processor yourself for running tests on?
Nope, we were thinking of possibly getting one. But the issue is still very rare and we will most probably not be able to trigger the crash on our computer.
Last time we got and AMD computer with weird hardware we did a few tests and now it's mostly collecting dust.

Lorash
Manual Inserter
Manual Inserter
Posts: 4
Joined: Sat Jul 15, 2017 6:34 pm

Re: Friday Facts #231 - Belt compression & Crash log uploading

Post by Lorash » Fri Feb 23, 2018 9:24 pm

Here's a crazy idea: try making a release with optimization settings changed for that one function and see if the frequency of crash reports change. There's a chance you found an erratum, maybe another instance of this one? https://www.phoronix.com/scan.php?page= ... ixed&num=1

Jap2.0
Smart Inserter
Smart Inserter
Posts: 1631
Joined: Tue Jun 20, 2017 12:02 am

Re: Friday Facts #231 - Belt compression & Crash log uploading

Post by Jap2.0 » Fri Feb 23, 2018 9:27 pm

I never had many problems when you added this and this reassured me a bit, however I'm still going to leave it disabled, for the following reasons:
  • 1. I'm going to upload it here anyway
    2. I can provide much more information on the forum than a single log
    3. There are many bugs that don't involve crashes, and in those cases a log won't get sent
    4. In the hypothetical case I try to reproduce a crash and in the process the game crashes 10 times, you have no way of knowing that 10 different people didn't suddenly have the same crash
Again, nothing against them doing this, those are just my reasons for opting out.
There are 10 types of people: those who get this joke and those who don't.

Omarflyjoemacky
Long Handed Inserter
Long Handed Inserter
Posts: 64
Joined: Tue Nov 15, 2016 10:56 pm

Re: Friday Facts #231 - Belt compression & Crash log uploading

Post by Omarflyjoemacky » Fri Feb 23, 2018 9:27 pm

As always, awesome FFF. Awfully grateful for the belt compression.

I'm surprised in this day and age, people would still be upset about you devs knowing how much RAM they have. I have no problem with the crash log reporting - it makes for a better game in the long run. Keep up the great work.
"And then Bender ran."

User avatar
<NO_NAME>
Fast Inserter
Fast Inserter
Posts: 173
Joined: Tue Aug 02, 2016 9:52 am

Re: Friday Facts #231 - Belt compression & Crash log uploading

Post by <NO_NAME> » Fri Feb 23, 2018 9:31 pm

If you can't find the bug, it is 90% chance that this is thread synchronization.
I am a translator. And what you did for Factorio?
Check out my mod "Realistic Ores" and my other mods!

dasiro
Long Handed Inserter
Long Handed Inserter
Posts: 87
Joined: Fri Jun 03, 2016 5:55 pm

Re: Friday Facts #231 - Belt compression & Crash log uploading

Post by dasiro » Fri Feb 23, 2018 9:33 pm

Jap2.0 wrote:4. In the hypothetical case I try to reproduce a crash and in the process the game crashes 10 times, you have no way of knowing that 10 different people didn't suddenly have the same crash
chances are high there's a lot of info in the log or dump that is so unique it must be either the same savegame or the same mod being used so they'll pretty fast figure out it's the same system/person even without having any other traceable information that leads back to you

User avatar
steinio
Smart Inserter
Smart Inserter
Posts: 1932
Joined: Sat Mar 12, 2016 4:19 pm

Re: Friday Facts #231 - Belt compression & Crash log uploading

Post by steinio » Fri Feb 23, 2018 9:36 pm

Why not connect the uploaded crash log with the logged in user credentials and in the user area there is a overview of my own crash logs with a case number which i can quote in the forum?
ImageImage
Transport Belt Repair Man
My little mods: Link | My favourite mods: Bob's Mods | Angel's Mods | Yuoki Railway Core | EvoGUI | Logistic Train Network
Factorio Cheat Sheet by Denis Zholob

View unread Posts

ske
Filter Inserter
Filter Inserter
Posts: 319
Joined: Sat Oct 17, 2015 8:00 am

Re: Friday Facts #231 - Belt compression & Crash log uploading

Post by ske » Fri Feb 23, 2018 9:37 pm

Twinsen wrote:
ske wrote:Do you have an affected Ryzen processor yourself for running tests on?
Nope, we were thinking of possibly getting one. But the issue is still very rare and we will most probably not be able to trigger the crash on our computer.
Last time we got and AMD computer with weird hardware we did a few tests and now it's mostly collecting dust.
There was talk about compiler crashes on earlier versions of the processors. AMD offers replacement for those earlier processors that are affected by the compiler crashes. Threadripper/EPYCs were already fixed for that particular crash type at release. It might be some weird caching problem that occurs in conjunction with tight loops. Do you check and include the CPU stepping in the crash logs?

QGamer
Fast Inserter
Fast Inserter
Posts: 120
Joined: Fri Apr 14, 2017 9:27 pm

Re: Friday Facts #231 - Belt compression & Crash log uploading

Post by QGamer » Fri Feb 23, 2018 9:45 pm

I must confess
Congratulations on the bugs you've already fixed, and best of luck to you as you try to find the mysterious bug!
As always, it was a great FFF!

Jap2.0
Smart Inserter
Smart Inserter
Posts: 1631
Joined: Tue Jun 20, 2017 12:02 am

Re: Friday Facts #231 - Belt compression & Crash log uploading

Post by Jap2.0 » Fri Feb 23, 2018 10:06 pm

dasiro wrote:
Jap2.0 wrote:4. In the hypothetical case I try to reproduce a crash and in the process the game crashes 10 times, you have no way of knowing that 10 different people didn't suddenly have the same crash
chances are high there's a lot of info in the log or dump that is so unique it must be either the same savegame or the same mod being used so they'll pretty fast figure out it's the same system/person even without having any other traceable information that leads back to you
True. They might think it's very common for me, though, even if it was actually rare. Good point nonetheless.
There are 10 types of people: those who get this joke and those who don't.

ixnorp
Inserter
Inserter
Posts: 39
Joined: Mon Jun 27, 2016 11:06 pm

Re: Friday Facts #231 - Belt compression & Crash log uploading

Post by ixnorp » Fri Feb 23, 2018 10:12 pm

You've made the right choices regarding crash logs. You'll never make everyone 100% happy no matter what you do, but this will make the largest number of people happy in the long run.

torham
Fast Inserter
Fast Inserter
Posts: 163
Joined: Sun May 25, 2014 1:40 pm

Re: Friday Facts #231 - Belt compression & Crash log uploading

Post by torham » Fri Feb 23, 2018 10:21 pm

Its so true that people would never send crash dumps to you. I click "don't send" automatically out of habit. I have been doing it ever since internet became a thing( 17+ years). I do not know of an instance where anyone would send a crash log to microsoft, and it would actually solve something. I think that message is there just to troll.

PacifyerGrey
Filter Inserter
Filter Inserter
Posts: 959
Joined: Wed Jun 29, 2016 10:02 am

Re: Friday Facts #231 - Belt compression & Crash log uploading

Post by PacifyerGrey » Fri Feb 23, 2018 10:30 pm

While it has potential I am not happy with compression just yet
https://forums.factorio.com/viewtopic.php?f=182&t=57998
https://forums.factorio.com/viewtopic.php?f=7&t=58046
And some splitter compression bugs related to its small internal buffer

Waiting patiently for those to be fixed!

User avatar
bobingabout
Smart Inserter
Smart Inserter
Posts: 5841
Joined: Fri May 09, 2014 1:01 pm

Re: Friday Facts #231 - Belt compression & Crash log uploading

Post by bobingabout » Fri Feb 23, 2018 10:34 pm

Image
Woo, lines 13, 26 and 27. Bobingabout contributions~
Creator of Bob's mods. Expanding your gameplay since version 0.9.8.

Ringkeeper
Long Handed Inserter
Long Handed Inserter
Posts: 97
Joined: Wed Feb 03, 2016 7:16 pm

Re: Friday Facts #231 - Belt compression & Crash log uploading

Post by Ringkeeper » Fri Feb 23, 2018 10:54 pm

can you tell us which Ryzen CPU is affected or when it is happening (what are players doing)? So it might be tested by players?
Have Ryzen 7 1700X myself on 2 PC ^^

Wubinator
Burner Inserter
Burner Inserter
Posts: 8
Joined: Fri Feb 19, 2016 4:52 pm

Re: Friday Facts #231 - Belt compression & Crash log uploading

Post by Wubinator » Fri Feb 23, 2018 10:58 pm

Why not ask for the opt-in / opt-out once in the main menu for the people who haven't made a choice yet.

Show it in a popup over the main menu, that way it isn't much different than the release notes popup. Because people aren't in a active game they have the time to read the information in the popup. And you can tell more details if you want to because the size wouldn't matter anymore.
Useful information I can think of: tell what it's for and where in the options people can disable it.
And I don't know if this still an issue in certain area's but maybe tell about the average file size for people with bandwidth caps

Highlight the opt-in button as the default for the people that just hit enter to hide the box?

You can show the same information again with a tool tip on the option in the menu



Maybe it's also an option (for those that opted out) to see some kind of alert icon in the main menu when there are crash logs so people could still cherry-pick crash logs to commit.

Post Reply

Return to “News”