Skip to content

added 2 notes to installation.rst and changed from print to warning in all_wavelengths.py #106

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.

Sign up for GitHub

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

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

added 2 notes to installation.rst and changed from print to warning in all_wavelengths.py #106

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
6090b73
added some notes to facilitate the windows installation: L136 and L229
Nov 30, 2023
2afd080
changed print statement to a warning in all_wavelengths.py, this way …
Nov 30, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion Py6S/SixSHelpers/all_wavelengths.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@

import copy
import sys
import warnings

import numpy as np

Expand Down Expand Up @@ -86,7 +87,7 @@ def f(wv):
print(wavelengths)
print(type(wavelengths))

print("Running for many wavelengths - this may take a long time")
warnings.warn("Running for many wavelengths - this may take a long time")
results = pool.map(f, wavelengths)

pool.close()
Expand Down
8 changes: 8 additions & 0 deletions doc/source/installation.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -133,6 +133,10 @@ sections below.
string to the end of its contents::
C:\Program Files\GNUWin32\bin;C:\G77\bin

Note: make sure you check the correct path for `GNUWin32\bin`,
as in some cases it is installed in `Program Files (x86)`
instead of `Program Files`.

:OS X: Install ``gfortran`` with Homebrew_.

::
Expand Down Expand Up @@ -222,6 +226,10 @@ sections below.
Note: on Windows, make sure you run this in the standard Command Prompt
(cmd.exe), not Powershell (PowerCmd.exe).

Note2: on Windows sometimes the executable might be saved only as `sixsV1.1`
(without the `.exe` extension). If that's the case, then run the same command
as the :Linux/OS X: one above.

#. If this is working correctly you should see a number of screen's
worth of output, finishing with something that looks like::
*******************************************************************************
Expand Down