EmotiBit / EmotiBit_Docs

Documentation and tutorials for the EmotiBit
MIT License
71 stars 11 forks source link

Docs not giving correct info for MacOSX config file location #126

Open mikkokotila opened 3 months ago

mikkokotila commented 3 months ago

Description

At least in MacOSX Sonoma, the current doc information fails to identify any means by which the config file can be found. The doc should make it very clear how and where to find it.

Type of correction

Link

https://github.com/EmotiBit/EmotiBit_Docs/blob/master/Working_with_emotibit_data.md#emotibit-oscilloscope-settings

Screenshots

nitin710 commented 3 months ago

@mikkokotila Can you highlight which config file you are talking about? The link to config.txt is common for all operating systems.

And the location of the Oscilloscope Settings-files are linked in the location you have already highlighted. If you have specific comments on the problems you faced, please let us know and we can address those.

If you are talking about another configuration file, please let us know and we can update the documentation to increase the clarity.

mikkokotila commented 2 months ago

Have a look at the link https://github.com/EmotiBit/EmotiBit_Docs/blob/master/Working_with_emotibit_data.md#emotibit-oscilloscope-settings and in there you will find under that heading references to various config files. None of this can be found using that info at least on two different Mac OSX versions, so I assume the docs are based on some old version of Mac OSX.

nitin710 commented 3 weeks ago

@mikkokotila the application package is os agnostic.

For example, the docs say that the settings files can be found in EmotiBitSoftware-macOS/EmotiBitOscilloscope.app/Contents/Resources/ and that is where it is, even in the latest macos-Sequoia 15 image

Maybe the .app is causing the confusion? Macos by default hides the extension on application bundles, therefore EmotiBitOscilloscope.app shows up as EmotiBitOscillosocpe. If that is indeed the confusion you are highlighting I understand and we can make that patch to update to clarity.