murveit (Hy Murveit)
User

Projects

User does not belong to any projects.

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Saturday

  • Clear sailing ahead.

User Details

User Since
Aug 18 2019, 6:32 AM (16 w, 4 d)
Availability
Available

Recent Activity

Sat, Dec 7

murveit added a comment to D25645: New UI to allow the user to modify the stretch being used..

Thanks for the review and testing Jasem.
I addressed your tooltip-concern in my latest update.
I don't quite understand how to respond to your other concern (re hiding the contols).

Sat, Dec 7, 8:05 PM · KDE Edu
murveit updated the diff for D25645: New UI to allow the user to modify the stretch being used..

Adjusted stretch button tooltips depending on their state.

Sat, Dec 7, 7:52 PM · KDE Edu

Wed, Dec 4

murveit added a comment to D25645: New UI to allow the user to modify the stretch being used..

The difference is subtle and IMHO it would not be obvious to many users.
I think this is much clearer.
You can try it by disabling the clear--comment out line 184 in fitstab.cpp

Wed, Dec 4, 6:55 PM · KDE Edu
murveit added a comment to D25645: New UI to allow the user to modify the stretch being used..

Rob, That's working as intended. I clear the button and disabled it when selecting it didn't make sense. It only appears when there's something to do.
I tried it the way you suggest and it was confusing--the user could hit the button and nothing really would happen (e.g. I made an info pop-up, but in the end, I though the better thing to do was this).

Wed, Dec 4, 6:49 PM · KDE Edu

Tue, Dec 3

murveit updated the diff for D25645: New UI to allow the user to modify the stretch being used..

Moved from QCheckBox to QPushButton for the stretch and auto buttons.
Auto button is only enabled when it makes sense.
Sorry, the blank auto-button is a little wider when disabled,
couldn't figure out how to avoid that.

Tue, Dec 3, 9:31 AM · KDE Edu

Mon, Dec 2

murveit added a comment to D25645: New UI to allow the user to modify the stretch being used..

Rob: Thanks for describing the bug related to the slider value display. I was able to reproduce and fix. You should not see that behavior anymore.

Mon, Dec 2, 7:11 AM · KDE Edu
murveit updated the diff for D25645: New UI to allow the user to modify the stretch being used..

Fix bug related to updated slider text value when auto is checked.

Mon, Dec 2, 6:59 AM · KDE Edu
murveit added a comment to D25645: New UI to allow the user to modify the stretch being used..

I implemented Rob's suggestion to disable the sliders and their labels and the auto checkbox when stretching is disabled.
I think this was a good idea.

Mon, Dec 2, 4:47 AM · KDE Edu
murveit updated the diff for D25645: New UI to allow the user to modify the stretch being used..

Disable most of the stretch UI when stretching is disabled.

Mon, Dec 2, 4:45 AM · KDE Edu
murveit added a comment to D25645: New UI to allow the user to modify the stretch being used..

Rob, thanks for all the suggestions and questions. Let me respond before I make any code changes.

Mon, Dec 2, 3:40 AM · KDE Edu
murveit added a comment to D25645: New UI to allow the user to modify the stretch being used..

Thanks again for testing and checking out the interface.

Mon, Dec 2, 12:18 AM · KDE Edu
murveit updated the diff for D25645: New UI to allow the user to modify the stretch being used..

Fixed issue when stretching float or double fits file.

Mon, Dec 2, 12:13 AM · KDE Edu

Sun, Dec 1

murveit added a comment to D25645: New UI to allow the user to modify the stretch being used..

BTW, I see the problem with the floating point file.
The issue is that I need to normalize the number--the values needed to stretch are 1/64K as big as the 64K-scale-shorts, and so they underflow the 10000 scale I was using.
I will address this in the next couple days.

Sun, Dec 1, 6:32 PM · KDE Edu
murveit added a comment to D25645: New UI to allow the user to modify the stretch being used..

Jasem & Rob, thanks for the initial comments.

Sun, Dec 1, 6:25 PM · KDE Edu
murveit updated the diff for D25645: New UI to allow the user to modify the stretch being used..

Fix bug that hid the labels.

Sun, Dec 1, 6:11 PM · KDE Edu
murveit requested review of D25645: New UI to allow the user to modify the stretch being used..
Sun, Dec 1, 8:12 AM · KDE Edu

Tue, Nov 26

murveit added a comment to D24256: Enable the auto-stretch Fits setting to be undone..

This is out-of-date, I guess abandoned, and the new scheme is an improvement, IMHO.
Is there something I need to do to delete/abandon it?

Tue, Nov 26, 9:05 PM · KDE Edu

Sun, Nov 24

murveit added a comment to D25497: Fix bug re filter-change-autofocus.

Perhaps I'm missing the obvious, but I don't see it. I don't want to overwrite secondsLabel. How would I use a fall through?

Sun, Nov 24, 10:43 PM · KDE Edu
murveit updated the diff for D25497: Fix bug re filter-change-autofocus.

Added secondsLabel update to current change.
Capture should show that it's focusing while waiting for the filterManager.

Sun, Nov 24, 10:06 PM · KDE Edu
murveit added a comment to D25497: Fix bug re filter-change-autofocus.

Wolfgang: Makes sense.
I assume what you mean is that I should do something like this:

secondsLabel->setText( i18n("Focusing..."));

just before the call to QTimer::singleShot(), where the text is dependent on filterManagerState.
E.g. if filterManagerState == FILTER_FOCUS then "Focusing...", if FILTER_CHANGE then "Changing Filters...", if FILTER_OFFSET then "Changing Focus Offset...".
I'll modify my change and do that.

Sun, Nov 24, 9:07 PM · KDE Edu
murveit added a comment to D25497: Fix bug re filter-change-autofocus.

That's true, though, as far as I know, that shouldn't happen.

Sun, Nov 24, 8:34 PM · KDE Edu
murveit added a comment to D25497: Fix bug re filter-change-autofocus.

Sorry for not filling that out. Here's what to check.

Sun, Nov 24, 7:11 AM · KDE Edu

Sat, Nov 23

murveit requested review of D25497: Fix bug re filter-change-autofocus.
Sat, Nov 23, 10:40 PM · KDE Edu
murveit requested review of D25483: Fix off-by-one bug in stretch..
Sat, Nov 23, 2:19 AM · KDE Edu

Tue, Nov 19

murveit updated the diff for D25318: Update the auto-stretch scheme and UI.

Fixed file permissions, which were accidentally set wrong.

Tue, Nov 19, 9:40 AM · KDE Edu
murveit updated the diff for D25318: Update the auto-stretch scheme and UI.

Removed some of the templating and refactored to simplify code.
Added highlighting to the stretch buttons when the image is stretched.
Needed to run initDisplayImage always in rescale, and not sure why.
Will continue to investigate.

Tue, Nov 19, 9:32 AM · KDE Edu

Sat, Nov 16

murveit updated the diff for D25318: Update the auto-stretch scheme and UI.

Fixing compile error.
Remove some compiler warnings.

Sat, Nov 16, 5:51 AM · KDE Edu

Fri, Nov 15

murveit requested review of D25318: Update the auto-stretch scheme and UI.
Fri, Nov 15, 8:46 AM · KDE Edu

Nov 2 2019

murveit updated the diff for D25114: Directly specify arcsecond error in Align UI..

The error is printed in yellow if it is between 1 and 1.5X the accuracy tolerance.

Nov 2 2019, 9:06 AM · KDE Edu
murveit added a comment to D25114: Directly specify arcsecond error in Align UI..

I found in the code that you have a yellow region between 1.0 and 1.5 x the accuracy threshold.
I'll modify it to do that.

Nov 2 2019, 8:42 AM · KDE Edu
murveit added a comment to D25114: Directly specify arcsecond error in Align UI..

Note sure I agree, but would be happy to add a color if you felt strongly about it. It seems like anything at or below the Accuracy spec should be green. I could add a yellow, but in what range would you recommend? Twice the accuracy? Within 1 arc-minute? ...

Nov 2 2019, 8:37 AM · KDE Edu
murveit requested review of D25114: Directly specify arcsecond error in Align UI..
Nov 2 2019, 8:11 AM · KDE Edu

Oct 31 2019

murveit added a comment to D25060: Fixing ASTAP Path on OS X and Setting Autostar as default for PHD2 Guiding.

Wolfgang: Re testing, if you look at the log, WITHOUT this change, even if the stars didn't "flip" after the meridian flip, you should see Ekos sending a "set_lock_position" message to PHD2 at some point after the meridian flip (I'm not sure what happens with the internal guider). If the simulator's stars are in the same position post-flip, then set_lock_position won't hurt it, but the message would still be there. This message is what screwed things up in "real life".

Oct 31 2019, 7:29 PM · KDE Edu
murveit added a comment to D25060: Fixing ASTAP Path on OS X and Setting Autostar as default for PHD2 Guiding.

That sounds OK to me.

Oct 31 2019, 2:21 AM · KDE Edu

Oct 30 2019

murveit added a comment to D25060: Fixing ASTAP Path on OS X and Setting Autostar as default for PHD2 Guiding.

Actually, I wouldn't be surprised if the thread you're pointing to is really talking about the problem that's being fixed in this PR.
I bet that if that person selected "auto star" his issues would be fixed, and/or if we removed the "if(guiderType == GUIDE_PHD2)" in this change.

Oct 30 2019, 11:27 PM · KDE Edu
murveit added a comment to D25060: Fixing ASTAP Path on OS X and Setting Autostar as default for PHD2 Guiding.

Here are the logs:

Oct 30 2019, 5:23 PM · KDE Edu
murveit added a comment to D25060: Fixing ASTAP Path on OS X and Setting Autostar as default for PHD2 Guiding.

I replied to to Wolfgang's queston with a Reply-All email, since I was attaching a log and forwarding an email, but I see it went to everyone but Wolfgang.
Sorry about that. I'm now re-sending inside of phabricator and I'll try and make google drive links for the logs:

Oct 30 2019, 5:17 PM · KDE Edu
murveit added a comment to D25060: Fixing ASTAP Path on OS X and Setting Autostar as default for PHD2 Guiding.

@wreissenberger: I reported the issue to Rob. The issue comes up only if the "auto star" button is unchecked in the Guide tab. For me, 4 or 5 nights in a row last week I failed to guide after a meridian flip until I looked into the code and logs and discovered the root cause.

Oct 30 2019, 4:37 PM · KDE Edu
murveit added a comment to D25060: Fixing ASTAP Path on OS X and Setting Autostar as default for PHD2 Guiding.

You said: --Also right now with the current behavior with Ekos and PHD2, if the user has auto-star unchecked but has not clicked on a star to force PHD2 to use that star for guiding, then it lets PHD2 select a star automatically and doesn't force it to use a different lock position. It is only if the user has auto-star unchecked and then the user selects a star to use for guiding in the guide image that it actually makes PHD2 use that star when guiding. So really you don't have to have auto star checked as long as you don't select a guide star it will still guide automatically.

Oct 30 2019, 4:48 AM · KDE Edu

Oct 29 2019

murveit added a comment to D25060: Fixing ASTAP Path on OS X and Setting Autostar as default for PHD2 Guiding.

Robert, Thanks. These changes do see to address my issue, and I believe would be an important improvement. However I have a couple questions/suggestions:

Oct 29 2019, 9:40 PM · KDE Edu

Oct 27 2019

murveit added a comment to D24976: Write FITS file on separate thread.

I tested this with my ASI1600, where the .fits files are 32.8Mb. Works fine.
In fact, it's likely that the larger the file, the more the speedup--as I guess the disk-write would take longer.
The only place where there would be extra memory used is where I do the memcpy in indiccd.cpp:1470.
I wasn't sure I needed to do that, but I didn't think I could rely on the blob memory staying put, so I conservatively
ran the memcpy, and created that buffer that I keep around. That memcpy takes less than 100ms on my RPi4 with Release build.

Oct 27 2019, 7:14 AM · KDE Edu
murveit requested review of D24976: Write FITS file on separate thread.
Oct 27 2019, 4:11 AM · KDE Edu

Oct 13 2019

murveit requested review of D24605: Adds an option to mouse-pan the HiPS graphics..
Oct 13 2019, 3:36 AM · KDE Edu

Oct 8 2019

murveit updated the diff for D24485: Complete the implementation of Capture tab gain control..

small changes: tr --> i18n and slotChanged --> valueChanged

Oct 8 2019, 6:22 AM · KDE Edu
murveit updated the diff for D24485: Complete the implementation of Capture tab gain control..

Added a don't-set-gain value.

Oct 8 2019, 5:39 AM · KDE Edu
murveit requested review of D24485: Complete the implementation of Capture tab gain control..
Oct 8 2019, 3:44 AM · KDE Edu

Oct 5 2019

murveit requested review of D24417: Set gain each capture.
Oct 5 2019, 3:35 AM · KDE Edu

Oct 2 2019

murveit requested review of D24356: Remove dead code from fitsviewer.
Oct 2 2019, 7:55 AM · KDE Edu

Oct 1 2019

murveit requested review of D24352: Fix focus HFR-averaging bug.
Oct 1 2019, 9:12 PM · KDE Edu

Sep 27 2019

murveit added a comment to D24256: Enable the auto-stretch Fits setting to be undone..

OK, I'll take a look at it and come back to you with a proposal or code-review.

Sep 27 2019, 6:00 PM · KDE Edu
murveit added a comment to D24256: Enable the auto-stretch Fits setting to be undone..

I agree it's not a perfect UI. I do think it is nice have the ability to see the unstretched image.

Sep 27 2019, 5:47 AM · KDE Edu
murveit requested review of D24256: Enable the auto-stretch Fits setting to be undone..
Sep 27 2019, 2:23 AM · KDE Edu

Sep 18 2019

murveit added a comment to D24041: # Enter a commit message. # # Changes: # # kstars/fitsviewer/fitshistogram.cpp.

When I look at the image you sent me, either in Pixinsight or Fitsviewer, it seems like it's a 1-channel image (I assume it's not debayered--it has seems to look like a bayer pattern).
That is, it doesn't seem to display as a color image before or after my change.
Does it work differently for you?
Please send instructions on how to test this image, or please send me an image that would display with color (or show 3 medians for R, G & B) on the original software.

Sep 18 2019, 8:57 AM · KDE Edu
murveit updated the diff for D24041: # Enter a commit message. # # Changes: # # kstars/fitsviewer/fitshistogram.cpp.
  1. Updating D24041: # Enter a commit message. # # Changes: # # kstars/fitsviewer/fitshistogram.cpp #
  2. Enter a brief description of the changes included in this update.
  3. The first line is used as subject, next lines as comment. #
  4. If you intended to create a new revision, use:
  5. $ arc diff --create
Sep 18 2019, 8:00 AM · KDE Edu
murveit requested review of D24041: # Enter a commit message. # # Changes: # # kstars/fitsviewer/fitshistogram.cpp.
Sep 18 2019, 7:31 AM · KDE Edu

Aug 19 2019

murveit added a reviewer for D23256: Sample the fits viewer's histogram calculation: mutlaqja.
Aug 19 2019, 6:45 AM · KDE Edu
murveit requested review of D23256: Sample the fits viewer's histogram calculation.
Aug 19 2019, 6:42 AM · KDE Edu