• Login
  • Register
  • Dolphin Forums
  • Home
  • FAQ
  • Download
  • Wiki
  • Code


Dolphin, the GameCube and Wii emulator - Forums › Dolphin Emulator Discussion and Support › Development Discussion v
« Previous 1 … 25 26 27 28 29 … 117 Next »

Telemetry data
View New Posts | View Today's Posts

Thread Rating:
  • 0 Vote(s) - 0 Average
  • 1
  • 2
  • 3
  • 4
  • 5
Threaded Mode
Telemetry data
09-29-2016, 12:10 PM
#1
Wexrio
Unregistered
 
So I just built the latest dolphin git revision 8122d2cc9f4db427c9ecd9fabf8413376bc4314c and had a menu pop up asking about telemetry data. Enough said, why has this not been burned with fire yet?
Reply
09-29-2016, 12:17 PM
#2
Shonumi Offline
Linux User/Tester
**********
Administrators
Posts: 6,526
Threads: 55
Joined: Dec 2011
It's completely optional. You can control if Dolphin sends anything at all. Read more here -> https://dolphin-emu.org/blog/2016/06/19/...reporting/
Website Find
Reply
09-29-2016, 01:05 PM
#3
Wexrio
Unregistered
 
Found the offending commit https://github.com/dolphin-emu/dolphin/c...531cfa1a55

Went browsing through the Core/Core/Analytics* and Core/Common/Analytics* it's fine. But is there a cmake option to disable it? Maybe I could make one...

Also why is a constexpr used and then defined as a u32 on line 138 of Core/Common/Analytics.cpp ? Excuse my noobness in programming but wouldn't that make the buffer size 800 bits instead of dependent on compile time options? Seems like a small buffer for all the data you are grabbing up.
Reply
09-29-2016, 05:14 PM
#4
JosJuice Offline
Developer
**********
Developers (Some Administrators and Super Moderators)
Posts: 9,023
Threads: 7
Joined: Oct 2014
(09-29-2016, 01:05 PM)Wexrio Wrote: But is there a cmake option to disable it? Maybe I could make one...

Yes, there's ENABLE_ANALYTICS, but it's actually completely broken right now. In the rush to get analytics finished before the 5.0 release, no code got added to make it do anything. If you want to fix that, pull requests are welcome.

(09-29-2016, 01:05 PM)Wexrio Wrote: Also why is a constexpr used and then defined as a u32 on line 138 of Core/Common/Analytics.cpp ? Excuse my noobness in programming but wouldn't that make the buffer size 800 bits instead of dependent on compile time options? Seems like a small buffer for all the data you are grabbing up.

I don't know why you're talking about "instead of dependent on compile time options", but anyway... The u32 means that the value 25 is stored in a u32 – it doesn't mean that the size limit is 25 u32s. m_reports_queue contains strings, so the size limit is 25 strings.
Find
Reply
« Next Oldest | Next Newest »


  • View a Printable Version
Forum Jump:


Users browsing this thread: 1 Guest(s)



Powered By MyBB | Theme by Fragma