-
Notifications
You must be signed in to change notification settings - Fork 39
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
gpr labs #53
Conversation
gpgLabs/GPR/Attenuation.py
Outdated
@@ -7,13 +7,25 @@ | |||
except Exception, e: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@dccowan : This is outdated.
Only do:
from ipywidgets import
Notebooks/GPRWidget.ipynb
Outdated
} | ||
], | ||
"source": [ | ||
"ifield = interact(PrimaryFieldWidget, tinterp = (0, 80, 2), epsr = (1, 40, 1))" | ||
"radargramImage = \"../figures/GPR/ubc_GPRcmp.png\"\n", | ||
"PrimaryFieldWidget(radargramImage)" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@dccowan : Fix the window size (ax.set_xlim(), ax.set_ylim())
The window size change with the widget, which it should not as we are looking at 'fix' data
Notebooks/GPRWidget.ipynb
Outdated
} | ||
], | ||
"source": [ | ||
"i = interact(PipeWidget, epsr = (0, 100, 1), h=(0.1, 2.0, 0.1), xc=(0., 40., 0.2), r=(0.1, 3, 0.1) )" | ||
"radargramImage = \"../figures/GPR/ubc_GPRdata.png\"\n", | ||
"PipeWidget(radargramImage)" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@dccowan : The figure is fairly small. We can make it bigger
try fig = plt.figure(figsize = (10,5))
Notebooks/GPRWidget.ipynb
Outdated
} | ||
], | ||
"source": [ | ||
"i = interact(WallWidget, epsr = (0, 100, 1), h=(0.1, 2.0, 0.1), x1=(1, 35, 1), x2=(20, 40, 1) )" | ||
"radargramImage = \"../figures/GPR/ubc_GPRdata.png\"\n", | ||
"WallWidget(radargramImage)" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same, the figure is fairly small.
@dccowan : For parameters descriptions, it could be nice to highlight them as it is done in most em_examples.
|
No description provided.