wreissenberger (Wolfgang Reissenberger)
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
Jul 16 2018, 6:19 AM (44 w, 3 d)
Availability
Available

Recent Activity

Yesterday

wreissenberger updated the diff for D21291: Observatory module with basic dome and weather handling created.

Action check box for stopping the scheduler added (not implemented)

Wed, May 22, 9:00 PM · KDE Edu
wreissenberger added a comment to D21291: Observatory module with basic dome and weather handling created.

Et voilà.

Wed, May 22, 6:53 PM · KDE Edu
wreissenberger updated the diff for D21291: Observatory module with basic dome and weather handling created.

Observatory windows title set to Observatory

Wed, May 22, 6:52 PM · KDE Edu
wreissenberger updated the diff for D21291: Observatory module with basic dome and weather handling created.

Execute actions for weather warnings and alerts

Wed, May 22, 5:28 PM · KDE Edu
wreissenberger updated the summary of D21291: Observatory module with basic dome and weather handling created.
Wed, May 22, 8:13 AM · KDE Edu
wreissenberger updated the diff for D21291: Observatory module with basic dome and weather handling created.

Taking actions due to weather warnings or alerts implemented

Wed, May 22, 8:12 AM · KDE Edu

Mon, May 20

wreissenberger updated the diff for D21291: Observatory module with basic dome and weather handling created.
  • Handling disconnects for weather and dome added
  • UI elements for observatory actions and status added - not implemented yet
Mon, May 20, 5:21 PM · KDE Edu
wreissenberger added a comment to D21291: Observatory module with basic dome and weather handling created.

Currently, the interference with with other modules isn't that heavy, so technically it is not necessary. Nevertheless, I would prefer a feature branch so that we can launch the module with a mature set of functionality.

Mon, May 20, 7:12 AM · KDE Edu

Sun, May 19

wreissenberger updated the diff for D21291: Observatory module with basic dome and weather handling created.

Tool tips added to the Observatory module

Sun, May 19, 8:30 PM · KDE Edu
wreissenberger updated the diff for D21291: Observatory module with basic dome and weather handling created.

D-Bus interface added.

Sun, May 19, 8:05 PM · KDE Edu
wreissenberger requested review of D21291: Observatory module with basic dome and weather handling created.
Sun, May 19, 1:25 PM · KDE Edu

Wed, May 15

wreissenberger updated the summary of D21222: Meridian flip control removed from Capture module.
Wed, May 15, 6:34 AM
wreissenberger requested review of D21222: Meridian flip control removed from Capture module.
Wed, May 15, 6:31 AM

Tue, May 7

wreissenberger requested review of D21073: Impact of scheduler settings on total imaging time corrected.
Tue, May 7, 5:50 PM · KDE Edu

Sun, May 5

wreissenberger requested review of D21034: Always check meridian flip before starting next capture.
Sun, May 5, 2:25 PM · KDE Edu

Thu, May 2

wreissenberger requested review of D20975: Meridian flip state logging extended.
Thu, May 2, 9:33 PM · KDE Edu
wreissenberger abandoned D20001: Simplifying Scheduler.

We follow another path with D20068 - closed.

Thu, May 2, 6:51 PM · KDE Edu

Wed, May 1

wreissenberger closed D19429: Selection of rows in Capture instead of single cells.

Solved in D19456.

Wed, May 1, 6:07 PM · KDE Edu
wreissenberger abandoned D14280: Handling multiple occurrence of the same sequence job corrected.

Solved in another diff.

Wed, May 1, 6:05 PM
wreissenberger added a comment to D19456: Selection in imaging sequence keeps position after row deletion.

Seems like this is not merged into master.

Wed, May 1, 5:48 PM · KDE Edu
wreissenberger added a comment to D20150: Meridian flip checks after capture completion removed.

I think this diff has not been merged yet.

Wed, May 1, 5:47 PM · KDE Edu
wreissenberger updated the diff for D19648: Keep GUI parameters for scheduler and capture in sync with row selection.

Update diff after rebase (second try)

Wed, May 1, 5:23 PM · KDE Edu
wreissenberger added a comment to D19648: Keep GUI parameters for scheduler and capture in sync with row selection.

Damn, that was wrong! Please wait for another update...

Wed, May 1, 6:03 AM · KDE Edu
wreissenberger updated the diff for D19648: Keep GUI parameters for scheduler and capture in sync with row selection.

Rebased upon latest master version.

Wed, May 1, 6:02 AM · KDE Edu
wreissenberger added a comment to D19648: Keep GUI parameters for scheduler and capture in sync with row selection.

I'm waiting for Eric's review. I can post a rebased version later if required.

Wed, May 1, 5:27 AM · KDE Edu

Apr 21 2019

wreissenberger requested review of D20716: Saving after reordering jobs enabled.
Apr 21 2019, 6:34 PM · KDE Edu

Apr 19 2019

wreissenberger requested review of D20682: Bug fix for calculating required light frames.
Apr 19 2019, 9:11 AM · KDE Edu

Apr 13 2019

wreissenberger added a comment to D20029: Capture counting corrected.

I would opt for 3.2. The behavior is better than without the fix.

Apr 13 2019, 2:18 PM · KDE Edu

Apr 11 2019

wreissenberger added a comment to D20029: Capture counting corrected.

When a scheduler job aborts, it does not change the completed frame count. So probably an in-sequence focus failing might do the job?

Aborting, right, but restarting resets the counters to zero.

Apr 11 2019, 11:46 AM · KDE Edu
wreissenberger added a comment to D20029: Capture counting corrected.

Back to the original question, how to proceed. I tried to construct a situation where the calculation of captured frames is call after the partial completion of a scheduler job where exactly that problem occurs we discussed above - I failed, I could not bring kstars into such a state. So maybe we have a theoretical discussion here.

Apr 11 2019, 4:23 AM · KDE Edu

Apr 10 2019

wreissenberger added a comment to D20029: Capture counting corrected.

Related? https://indilib.org/forum/ekos/4995-ekos-scheduler-eats-frames-from-the-sequence.html#37550

With the latest information posted - YES. It's exactly the situation with a capture job of 18*L+3*R+3*G+3*B.

Turned out meanwhile that it is not related. It is simply missing awareness of the option "Remember job progress". Maybe we should consider moving this option directly to the scheduler tab - or show at least there, that the option is active.

Apr 10 2019, 3:57 PM · KDE Edu
wreissenberger added a comment to D20029: Capture counting corrected.

Related? https://indilib.org/forum/ekos/4995-ekos-scheduler-eats-frames-from-the-sequence.html#37550

With the latest information posted - YES. It's exactly the situation with a capture job of 18*L+3*R+3*G+3*B.

Apr 10 2019, 12:25 PM · KDE Edu
wreissenberger added a comment to D20029: Capture counting corrected.

It's unclear, but maybe, yes.

Agreed, the issue we are working here is a good hint. I asked for more details on the forum.

Could I offer my own implementation on the two fixes that are in this differential? I'd like to first fix the FindNextJob issue, then in another diff the frame counting via messages from the capture module.

Absolutely fine, I do not have the ambition that I fix it. I simply want it to be fixed asap. :-)

My previous message is about the calculation : the order of operators produces 1 when less than a full sequence is executed. It also considers sequences are distributed equally between jobs, which I disagree with as the code, in remember mode, is trying to gather frames to complete sequences in order, then schedule remaining ones.

You are right, the calculation of captured frames of a certain sequence job is only correct as long as the entire capture job has completed. If it does not run completely, the frames taken in the last cycle are not counted correctly.
Let's take an example with a LLLRGB sequence that completes twice and terminates after two L frames. In the calculation, we have schedJob->getCompletedCount() = 14, capturesPerRepeat=6 and seqJob->getCount() = 3. As a result we get 14/6*3 = 6 - which is wrong, it should be 8.

Apr 10 2019, 12:00 PM · KDE Edu

Apr 7 2019

wreissenberger requested review of D20365: Clean module registration.
Apr 7 2019, 7:08 PM · KDE Edu
wreissenberger added a comment to D20029: Capture counting corrected.

That's a good idea, but weeell I have two disagreements : first this is integer calculation and you probably need to reorder your operators, and second, if I understand correctly, you are considering the amount of captures to get equiprobable scattering over all sequence jobs.
I'll nonetheless test this asap.

Could you please be more specific? To be honest, I do not understand what you mean.

Apr 7 2019, 6:51 PM · KDE Edu

Apr 6 2019

wreissenberger updated the diff for D20029: Capture counting corrected.

Update: There was an another bug in calculating whether light frames are required by a schedule job. In the case that "Repeat for x runs" is selected and one sequence has more captures than required and another one has less, the scheduler job assumes than no light frames are required - which is wrong.

Apr 6 2019, 2:17 PM · KDE Edu
wreissenberger added a comment to D20029: Capture counting corrected.

Can you please check this?

https://indilib.org/forum/general/4908-meridian-flip-issue-with-the-scheduler.html

Resolved with D20150

Apr 6 2019, 1:50 PM · KDE Edu

Mar 31 2019

wreissenberger requested review of D20151: Unchecking the meridian flip checkbox reset the meridian flip state.
Mar 31 2019, 9:12 PM · KDE Edu
wreissenberger requested review of D20150: Meridian flip checks after capture completion removed.
Mar 31 2019, 8:27 PM · KDE Edu

Mar 29 2019

wreissenberger added a comment to D20029: Capture counting corrected.

Hm, maybe, but I think it's a different issue. But thanks for the hint, I will try to reproduce it.

Mar 29 2019, 5:45 AM · KDE Edu

Mar 28 2019

wreissenberger added a comment to D19648: Keep GUI parameters for scheduler and capture in sync with row selection.

OK, but I checked for the reordering problem:

  • It's possible to sort jobs while the Scheduler is running using the "Reset state and sort observations" button. That is not a regression, that is an existing bug.
  • It's possible to reorder and reset jobs while the Scheduler is running using the "Reset state and force reevaluation" button. That is a regression in this differential.

Eric, I cannot reproduce it. When I start the scheduler, all buttons on the top left side of the queue are deactivated.

Mar 28 2019, 7:54 AM · KDE Edu

Mar 27 2019

wreissenberger updated the diff for D20029: Capture counting corrected.

I think I found a way to correct the counting of frames for the case that remembering is ticked off.

Mar 27 2019, 11:19 PM · KDE Edu
wreissenberger added a comment to D19648: Keep GUI parameters for scheduler and capture in sync with row selection.

https://bugs.kde.org/show_bug.cgi?id=405901 related?

As mentioned above, right, this is an existing problem, but this diff neither addresses it nor makes it worse. Should be fixed separately.

Mar 27 2019, 8:47 PM · KDE Edu
wreissenberger added a comment to D20029: Capture counting corrected.

Well, filling the map in any cases will not help, because the intention of not ticking "Remember progress" is exactly, that we start counting from zero - no matter how many are in the file system.

Mar 27 2019, 4:07 PM · KDE Edu
wreissenberger added a comment to D20029: Capture counting corrected.

Your comment about capture_completed is right, I'm trying to fix it. The problem is, that captures_completed = schedJob->getCompletedCount() is called for each sequence job entry and added to totalCompletedCount, i.e. the total amount of the entire capture run is added multiply.

Mar 27 2019, 1:10 PM · KDE Edu
wreissenberger added a comment to D20001: Simplifying Scheduler.

@TallFurryMan: Absolutely not, that's the nature of discussion. And I appreciate the ambition of making a sports car out of it :-)

Mar 27 2019, 8:03 AM · KDE Edu

Mar 26 2019

wreissenberger added a comment to D20001: Simplifying Scheduler.

OK, @TallFurryMan , I give up :-) Here is the alternative with the score calculations shifted to SchedulerJob: D20068.

Mar 26 2019, 7:31 PM · KDE Edu
wreissenberger requested review of D20068: Shifted scheduler job specific scorings from Scheduler to SchedulerJob.
Mar 26 2019, 7:28 PM · KDE Edu
wreissenberger accepted D20058: Fix altitude restriction log and allow restriction down to -15 degrees.

OK, agreed.

Mar 26 2019, 6:47 PM · KDE Edu
wreissenberger updated the diff for D19648: Keep GUI parameters for scheduler and capture in sync with row selection.

Removal of table dis-/enabling completed.

Mar 26 2019, 6:37 PM · KDE Edu
wreissenberger updated the diff for D19648: Keep GUI parameters for scheduler and capture in sync with row selection.

Active job selected, enabling/disabling queue tables removed.

Mar 26 2019, 6:29 PM · KDE Edu
wreissenberger added a comment to D20029: Capture counting corrected.

By the way: I found out that the null pointer exception occurs also when there is only one scheduler entry. So I think it's quite urgent that we at least fix this issue...

Mar 26 2019, 8:55 AM · KDE Edu
wreissenberger added a comment to D20058: Fix altitude restriction log and allow restriction down to -15 degrees.

Better than the rather arbitrary limit of 15. But why -15 and not -90? And what about changing the maximum to 90 instead of 89.9?

Mar 26 2019, 8:50 AM · KDE Edu
wreissenberger added a comment to D19648: Keep GUI parameters for scheduler and capture in sync with row selection.

Currently I am reluctant interfering directly since I am not really familiar with style sheets. I understand that it follows the CSS syntax, but I do not want to override a single place with fixed values.

Mar 26 2019, 8:47 AM · KDE Edu

Mar 25 2019

wreissenberger added a comment to D19648: Keep GUI parameters for scheduler and capture in sync with row selection.

Now rows might only be selected when capture or scheduler are idle.

Mar 25 2019, 8:38 PM · KDE Edu
wreissenberger updated the diff for D19648: Keep GUI parameters for scheduler and capture in sync with row selection.

Job table disabled when capture or scheduler are running

Mar 25 2019, 8:36 PM · KDE Edu
wreissenberger requested review of D20029: Capture counting corrected.

Please check my comments, from my point of view everything should be fine.

Mar 25 2019, 3:38 PM · KDE Edu

Mar 24 2019

wreissenberger updated the summary of D20029: Capture counting corrected.
Mar 24 2019, 10:03 PM · KDE Edu
wreissenberger requested review of D20029: Capture counting corrected.
Mar 24 2019, 9:59 PM · KDE Edu
wreissenberger added a comment to D20001: Simplifying Scheduler.

Right, that's what I first thought. But does it really make sense that a scheduler job rates its own score? Shouldn't be the rating of a job something that we would like to keep outside of a single job? That's why I came up with the idea to shift it to a separate class being associated with the Scheduler keeping the scoring logic.

Mar 24 2019, 3:52 PM · KDE Edu
wreissenberger added a comment to D19998: Remove bypass of preliminary steps when no light frames are needed..

Hm, seems like both changing it and leaving it have good arguments. What about if we freeze the scheduler, leave it currently as it is and concentrate on refactoring as begun in D20001? Doing both in parallel might be too much...

Mar 24 2019, 12:49 PM · KDE Edu
wreissenberger planned changes to D19648: Keep GUI parameters for scheduler and capture in sync with row selection.

Ouch, that's embarrassing to admit, but I simply forgot to test this case :(
I'm afraid, scheduler and capture take the values at least partially directly from the UI and not from the model.

Mar 24 2019, 12:29 PM · KDE Edu
wreissenberger added a comment to D20001: Simplifying Scheduler.

I have done both. But relocating the functionality to SchedulerJob has the problem, that calculations for weather and darkness are not specific to a scheduler job. In fact, weather and darkness are properties of the entire schedule, not of the single scheduler job.
And yes, the idea points exactly in that direction that we may have several ones. In fact we have two: one where the sequence is set manually and one where the sequence is sorted by altitude. But both use the same scoring strategy. See ScheduleStrategy as base class.

Mar 24 2019, 10:45 AM · KDE Edu

Mar 23 2019

wreissenberger added a comment to D19998: Remove bypass of preliminary steps when no light frames are needed..

This does not work with multi-night setup. Suppose I have a scheduler list with Light + Calibration frames at the end. When I run it daily, I expect it to finish whatever is left. If only calibration frames are left what happens? It tracks and focus and guide on what? What if I only had dark frames left?

What type of setup do you have in mind? In case that there is a separate scheduler job for the calibration frames, it simply depends, whether guiding etc. is selected in the scheduler job. If it is not selected, no guiding will happen, right?

Mar 23 2019, 10:40 PM · KDE Edu
wreissenberger updated the diff for D19648: Keep GUI parameters for scheduler and capture in sync with row selection.

Rebased on the current head. From my perspective, it is mature enough to be merged into master.

Mar 23 2019, 3:52 PM · KDE Edu
wreissenberger accepted D19998: Remove bypass of preliminary steps when no light frames are needed..

Makes sense, it's more obvious this way. If somebody enforces guiding while capturing flats etc - he should know what he is doing.

Mar 23 2019, 3:44 PM · KDE Edu
wreissenberger requested review of D20001: Simplifying Scheduler.
Mar 23 2019, 3:13 PM · KDE Edu

Mar 17 2019

wreissenberger added a comment to D19528: Making Scheduler robust against guiding problems.

This diff contains a severe bug. Please apply D19840 to resolve it.

Mar 17 2019, 6:46 PM · KDE Edu
wreissenberger requested review of D19840: Bugfix: proper usage of abort() for finishing a capture sequence queue.
Mar 17 2019, 6:41 PM · KDE Edu
wreissenberger added a comment to D19794: Bugfix #405325: Keeping meridian flip in capture and mount in sync..

At any rate, is this good to go now and reliable?

From my perspective, yes.

Mar 17 2019, 3:43 PM · KDE Edu
wreissenberger updated the diff for D19794: Bugfix #405325: Keeping meridian flip in capture and mount in sync..

Value changes to meridian flip setup exposed to D-Bus

Mar 17 2019, 3:17 PM · KDE Edu
wreissenberger added a comment to D19648: Keep GUI parameters for scheduler and capture in sync with row selection.

Now the activation of buttons should prevent moving list entries when the scheduler is running. But in both the scheduler and in the capture module, enabling/disabling buttons is distributed. Should be cleaned up somewhen...

Mar 17 2019, 2:32 PM · KDE Edu
wreissenberger updated the diff for D19648: Keep GUI parameters for scheduler and capture in sync with row selection.
  • Moving sequences enabled only if state is idle
  • Typo in job selection corrected
Mar 17 2019, 2:29 PM · KDE Edu
wreissenberger planned changes to D19794: Bugfix #405325: Keeping meridian flip in capture and mount in sync..

OK, understood. I will change the signal handling such that they are using the D-Bus.

Mar 17 2019, 1:49 PM · KDE Edu
wreissenberger planned changes to D19648: Keep GUI parameters for scheduler and capture in sync with row selection.

Good point, I will check enabling/disabling for capture and scheduler. At least I did not change anything intentionally, so I would expect it has been before. But nevertheless, let's clean it up here.

Mar 17 2019, 12:56 PM · KDE Edu
wreissenberger added a comment to D19528: Making Scheduler robust against guiding problems.

Hello, could we get this D19528 and then D19393 merged? We'll continue from that baseline, which is factually better than the trunk state. Thanks!

Whom do you mean with we? Generally spoken, yes, makes sense.

Mar 17 2019, 12:34 PM · KDE Edu
wreissenberger updated the diff for D19648: Keep GUI parameters for scheduler and capture in sync with row selection.

Bugfix for emptying queues in scheduler and capture

Mar 17 2019, 10:08 AM · KDE Edu
wreissenberger planned changes to D19648: Keep GUI parameters for scheduler and capture in sync with row selection.

Created an index violation in clearSequenceQueue() - needs to be corrected.

Mar 17 2019, 7:31 AM · KDE Edu

Mar 16 2019

wreissenberger added a comment to D19794: Bugfix #405325: Keeping meridian flip in capture and mount in sync..

setMeridianFlipValues is not a DBus function? doesn't need to be added to the Mount.xml file?

No, I changed based the implementation on slots and signals. Changing a value in either capture or mount sends out a signal and the other side receives it and sets the values accordingly. This seems to me a more elegant way than using calls via DBus.

Mar 16 2019, 11:13 AM · KDE Edu

Mar 15 2019

wreissenberger updated the summary of D19794: Bugfix #405325: Keeping meridian flip in capture and mount in sync..
Mar 15 2019, 8:49 PM · KDE Edu
wreissenberger requested review of D19794: Bugfix #405325: Keeping meridian flip in capture and mount in sync..
Mar 15 2019, 8:45 PM · KDE Edu

Mar 10 2019

wreissenberger added a comment to D19393: Prevent rescheduling aborted jobs until all jobs are processed.

Shall I give it a try?

Sure, and I have suggestions :) let's do this really step by step. At the extreme, because the source is so complex, I'd say let's go function by function, ensuring that they match features.

Fully agreed! Wouldn't it better if we lead this discussion in a separate thread or on the kstars forum?

Mar 10 2019, 11:31 AM · KDE Edu
wreissenberger updated the diff for D19648: Keep GUI parameters for scheduler and capture in sync with row selection.

Weather check GUI update shifted to syncGUIToJob

Mar 10 2019, 11:15 AM · KDE Edu
wreissenberger updated the test plan for D19648: Keep GUI parameters for scheduler and capture in sync with row selection.
Mar 10 2019, 9:23 AM · KDE Edu
wreissenberger requested review of D19648: Keep GUI parameters for scheduler and capture in sync with row selection.
Mar 10 2019, 9:22 AM · KDE Edu

Mar 9 2019

wreissenberger added a comment to D19393: Prevent rescheduling aborted jobs until all jobs are processed.

Well, horrible is such a strong word :-P Fully agree! Remember, the idea was making the scheduler job more intelligent. Shall I give it a try?

Mar 9 2019, 8:21 PM · KDE Edu

Mar 8 2019

wreissenberger accepted D19393: Prevent rescheduling aborted jobs until all jobs are processed.

Looks good, both restarting aborted jobs and the cutoff limit work fine. If I am the only one who doubts the necessity of a cutoff for the altitude limit, let's go. Maybe a hint in the hover help about the existence of the parameter might be helpful.

Mar 8 2019, 9:04 PM · KDE Edu

Mar 7 2019

wreissenberger added a comment to D19393: Prevent rescheduling aborted jobs until all jobs are processed.

Hm, I am not so sure whether such a cut-off makes it better. At the end we have two parameters that describe the same limit. For me it was confusing, because I was not aware of the additional cut-off parameter.
Regarding re-sorting completed/error/invalid jobs, I would vote against. We have two modes for the scheduler: either manual sequence or automatically sorted. In the manual sort, I would prefer no change of the order at all. With the automatic sort, that's another story, there it makes sense.

Mar 7 2019, 2:50 PM · KDE Edu

Mar 6 2019

wreissenberger added a comment to D19393: Prevent rescheduling aborted jobs until all jobs are processed.

I would have expected abort of the first job and continuing the second one. What type of restriction do you use?

Mar 6 2019, 9:37 PM · KDE Edu
wreissenberger added a comment to D19393: Prevent rescheduling aborted jobs until all jobs are processed.

I tested it with the first job with an Alt restriction of 15deg, but it seems not to work. The scheduler changes to sleeping mode when Alt reaches approx. 18deg. Bug or feature?

Mar 6 2019, 9:36 PM · KDE Edu
wreissenberger updated the diff for D19528: Making Scheduler robust against guiding problems.

Restricted to handling aborted guiding.

Mar 6 2019, 8:52 PM · KDE Edu
wreissenberger added a comment to D19528: Making Scheduler robust against guiding problems.

I think I could easily separate restarting of aborted jobs from the rest quite easily. Just give me 1-2 h to check it...

Mar 6 2019, 7:13 PM · KDE Edu
wreissenberger added a comment to D19528: Making Scheduler robust against guiding problems.

I propose you keep your differential focused on restoring functionality after a guiding failure, even if the aborted job isn't managed that well afterwards (that means not bypassing of aborted jobs at the beginning of evaluation).
I will rebase D19393 on yours, and add a fix to the block removing jobs that are not to be evaluated, so that aborted jobs are kept in place and not touched until they are the only ones remaining.
In this context, I will tackle both the re-evaluation without order change and possible state instabilities like the altitude restriction causing the job to repeatedly abort and reschedule.

What do you think?

Agreed. James should keep in mind, that your fix should be landed shortly after mine is merged. Without fixing handling of aborted jobs, capture first tries to restart guiding five times, then aborts the job and as a next step restarts it again. That's not nice, but we can live with if for a short timeframe.

Mar 6 2019, 3:21 PM · KDE Edu

Mar 5 2019

wreissenberger added a comment to D19528: Making Scheduler robust against guiding problems.

Oh wow, so you investigated that too? I have the same observation about aborted jobs! Did you have a look at D19393? Coincidence :)

Nope. Maybe you should add me as a reviewer? :-)

I need some time to examine the part about guiding : we need the suspension feature to be usable both with and without scheduler, and reading this I don't readily understand if that's OK.

Suspending works in both modes. If used with the scheduler, the scheduler thinks simply that capturing is running although capturing is suspended. Restarting a suspended guiding is handled by the capture module.

About aborted jobs set for restart, the approach here is slightly different from D19393. D19393 is not trying to restart aborted jobs, only making sure they don't interfere with other jobs, specifically when the scheduler is running. I needed to include the scheduler running/not running part, but I don't recall why now (I'm in business trip).

Ah, interesting, I have to take a closer look at it.

So that's cool, but needs careful tests. I'll try to do that beginning of this week with the simulators.

Fully agreed! One thing with aborted jobs is not so nice currently: They are put at the end of the list, i.e. sorting of targets is changed, when aborted jobs get restarted.

Mar 5 2019, 9:33 PM · KDE Edu
wreissenberger added a comment to D19528: Making Scheduler robust against guiding problems.

That's great! A few users reported issues regarding this problem.

Good to know that I'm not alone :-)

What about when internal guider loses a star and reacquires it? I don't believe that's registered as an aborted guiding, right?

I haven't found a way hot to test lost guiding star directly. As far as I know at least PHD2 tries to re-aquire a guiding star. If this fails within a certain amount of time, it aborts.

Mar 5 2019, 9:24 PM · KDE Edu

Mar 4 2019

wreissenberger updated the diff for D19528: Making Scheduler robust against guiding problems.

Updated to latest master version.

Mar 4 2019, 10:18 PM · KDE Edu
wreissenberger requested review of D19528: Making Scheduler robust against guiding problems.
Mar 4 2019, 8:53 PM · KDE Edu

Mar 1 2019

wreissenberger added a comment to D19429: Selection of rows in Capture instead of single cells.

Posted the remaining in D19456.

Mar 1 2019, 9:47 PM · KDE Edu
wreissenberger requested review of D19456: Selection in imaging sequence keeps position after row deletion.
Mar 1 2019, 9:41 PM · KDE Edu
wreissenberger added a comment to D19429: Selection of rows in Capture instead of single cells.

I've already included the changes in the committed version.

Mar 1 2019, 5:30 PM · KDE Edu