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

Flash install #162

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Flash install #162

wants to merge 4 commits into from

Conversation

jlhitt1993
Copy link
Collaborator

I updated the instructions for using the flash drive and removed the batch file for the old method. The new method will contain the batch file inside the env so no need for it here.

@codecov-io
Copy link

Codecov Report

Merging #162 (98224ff) into master (8cc4364) will increase coverage by 0.70%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #162      +/-   ##
==========================================
+ Coverage   72.34%   73.04%   +0.70%     
==========================================
  Files           8        8              
  Lines         282      282              
==========================================
+ Hits          204      206       +2     
+ Misses         78       76       -2     
Impacted Files Coverage Δ
tests/test_config_reader.py 78.94% <0.00%> (+10.52%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8cc4364...98224ff. Read the comment docs.

Copy link
Member

@sbillinge sbillinge left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

couple of suggestions (see inline)

=============================================================================

**You must have a USB drive with pytentiostat and pytent_env to use this method.**
**If you do not have such a drive, you can download the source code at** https://github.com/juami/pytentiostat
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These instructions are confusing I think. Much better is to give people step by step instructions, and then alternatives if things fail.

For example, I am already stumped on the first one. I have a USB drive (though I don't really need to be told I need this if I am installing from a USB), but I have no idea if I have pytent_env, and no idea how to check or what to do if I don't have it. => frustration and dissatisfaction.

Can it be written more like (without the bold warnings at the top of what can go wrong).

  1. start and log in to your computer
  2. place the USB drive into a USB slot in your computer
  3. <describe what they will see, then what to do, "double click on the folder icon" or sthg like that>
  4. Drag and drop the folder named pytentiostat onto your desktop (or into the folder where you want it located)
  5. Look inside the pytentiostat folder for pytent_env and inside that folder for a file called
    pytent_GUI.bat.
  6. To run the graphical user interface, double click on pytent_GUI.bat to run the software.
  7. If you prefer to run the command-line interface to the software (less common) , double click on pytent_cmd.bat

@pauchery
Copy link

Hi @jlhitt1993, I don't see the pytent_env folder with a pytent_GUI.bat and a pytent_cmd.bat file in this pull request. Do you have these or are these not yet created? Just trying to figure out where you left off so we don't reinvent the wheel. Thanks!

@jlhitt1993
Copy link
Collaborator Author

Hey @pauchery I think there was initally one install_from_local batch file for the flash drive that installed both the GUI and command line version located in the scripts folder, however it requires local copies of libraries in .whl format and it had a bug with the PyQt library that didn't work sometimes. It looks like that file is still in the scripts folder but you'll have to download the whl files for each library on their individual websites.

@pauchery
Copy link

@jlhitt1993 ok thank you. We've been working with those .whl and batch files and got PySide2 to install with the batch file.

@codecov-commenter
Copy link

codecov-commenter commented Jul 23, 2024

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 73.04%. Comparing base (8cc4364) to head (98224ff).
Report is 21 commits behind head on master.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #162      +/-   ##
==========================================
+ Coverage   72.34%   73.04%   +0.70%     
==========================================
  Files           8        8              
  Lines         282      282              
==========================================
+ Hits          204      206       +2     
+ Misses         78       76       -2     

see 1 file with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8cc4364...98224ff. Read the comment docs.

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.

5 participants