TallFurryMan (Eric Dejouhanet)
User

Projects

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Saturday

  • Clear sailing ahead.

User Details

User Since
Apr 13 2018, 9:37 PM (57 w, 5 d)
Availability
Available

Recent Activity

Mon, May 20

TallFurryMan added a comment to D21291: Observatory module with basic dome and weather handling created.

Do we create a feature branch for this, or do we use the trunk?

Mon, May 20, 4:48 AM · KDE Edu

Fri, May 3

TallFurryMan accepted D20975: Meridian flip state logging extended.
Fri, May 3, 6:32 AM · KDE Edu

Thu, May 2

TallFurryMan accepted D19648: Keep GUI parameters for scheduler and capture in sync with row selection.

No more problem with reordering of jobs when the scheduler is running. The behavior is now coherent with the Capture module.
However, the end-user now has the same feedback when clicking (job characteristics are displayed) and double-clicking (job characteristics are edited).
But that's a better behavior I think (and it matches with the draft UI I'm preparing).

Thu, May 2, 7:38 AM · KDE Edu

Wed, May 1

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

Oh sorry this requires my review. I'll have a look this morning.

Wed, May 1, 9:01 AM · KDE Edu

Tue, Apr 30

TallFurryMan added a comment to D20068: Shifted scheduler job specific scorings from Scheduler to SchedulerJob.

I think we should merge this. I'll then separate scheduler.cpp into several file to try to clarify what functions are for. I'll also separate the interface from the implementation, so we don't have too many private stuff cluttering.

Tue, Apr 30, 8:21 PM · KDE Edu
TallFurryMan updated the diff for D20068: Shifted scheduler job specific scorings from Scheduler to SchedulerJob.

Rebased to 20190430 state.

Tue, Apr 30, 8:18 PM · KDE Edu

Apr 11 2019

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

I'll double check this as soon as I can. Sorry for the delay.

Apr 11 2019, 8:48 AM · KDE Edu
TallFurryMan added a comment to D20029: Capture counting corrected.

I'm sorry my availability is really low these days. Hopefully "better" in the coming w16 as I'll be in holidays.
When a scheduler job aborts, it does not change the completed frame count. So probably an in-sequence focus failing might do the job?

Apr 11 2019, 8:47 AM · KDE Edu

Apr 10 2019

TallFurryMan added a comment to D20029: Capture counting corrected.

It's unclear, but maybe, yes. 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.

Apr 10 2019, 10:59 AM · KDE Edu

Apr 7 2019

TallFurryMan 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.

Apr 7 2019, 5:47 PM · KDE Edu

Mar 27 2019

TallFurryMan 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, 5:28 PM · KDE Edu
TallFurryMan added a comment to D20029: Capture counting corrected.

Situation is clear, thanks, and sorry for missing the point at first.

Mar 27 2019, 3:44 PM · KDE Edu
TallFurryMan 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 27 2019, 8:08 AM · KDE Edu
TallFurryMan added a comment to D20068: Shifted scheduler job specific scorings from Scheduler to SchedulerJob.

I propose we then divide SchedulerJob into SchedulerJobModel and SchedulerJob, deriving from SchedulerJobModel.
We can then move all the accessor stuff away into SchedulerJobModel, and focus on making SchedulerJob more a controller.
We should be able to tweak SchedulerJobModel to comply with the QT model interface perhaps.

Mar 27 2019, 8:03 AM · KDE Edu
TallFurryMan accepted D20068: Shifted scheduler job specific scorings from Scheduler to SchedulerJob.

I cannot disagree with this, this is what you proposed last year, and this is the approach I support!
Obviously we need to be very careful to not moving the complexity from one class to the other.

Mar 27 2019, 7:57 AM · KDE Edu
TallFurryMan added a comment to D20001: Simplifying Scheduler.

Oh no I'm sorry if I sounded too annoying to you, that wasn't my intent. Let me propose a preliminary differential too based on your work last year and with the same idea as this one. And then we both hit it until it has the shape of a sports car (well hopefully).

Mar 27 2019, 7:53 AM · KDE Edu
TallFurryMan added a comment to D19648: Keep GUI parameters for scheduler and capture in sync with row selection.

Active job selected, enabling/disabling queue tables removed.

@TallFurryMan: This is a misunderstanding, I did not touch enabling/disabling of the table buttons. This already works correctly and remains as it is.

Mar 27 2019, 7:49 AM · KDE Edu

Mar 26 2019

TallFurryMan added inline comments to D20029: Capture counting corrected.
Mar 26 2019, 8:52 PM · KDE Edu
TallFurryMan added a comment to D19648: Keep GUI parameters for scheduler and capture in sync with row selection.

I disagree with disabling the queue table. I think checking whether the scheduler is running when enabling or disabling the button that allows sorting, and duplicating the check in the function that does the actual sorting, is a better option.

Mar 26 2019, 4:51 PM · KDE Edu
TallFurryMan added a comment to D20058: Fix altitude restriction log and allow restriction down to -15 degrees.

I had the same reflection. Why -15 and not -90: I don't even know why it was 15 in the first place, so my change is the minus sign. It would take a high and isolated place to require a restriction under -15 degrees.
For 90 instead of 89.9 I think 90 is taboo in altaz/radec conversions because of the indefinite value of the azimuth. On the other hand, -90 is used for coherence with "no altitude restriction" in XML files. But agreed, there's no particular other justification.

Mar 26 2019, 4:45 PM · KDE Edu
TallFurryMan requested review of D20058: Fix altitude restriction log and allow restriction down to -15 degrees.
Mar 26 2019, 8:40 AM · KDE Edu

Mar 25 2019

TallFurryMan added a comment to D20001: Simplifying Scheduler.

I agree with your points, but I still tend to think SchedulerJob should be able to produce a score by itself, answering "job, what's your score?".
SchedulerJob knows when it starts, knows the restrictions, and there's no added value in Scheduler for this purpose that I can see. Scheduler is just a tunnel providing global variables in most cases.
In the case of darkness, Scheduler needs the startup time of SchedulerJob. For dawn and dusk, when that will be fixed, Scheduler will still need the startup and completion time of SchedulerJob. Same again for Moon separation.
Now, I agree, altitude calculations need SchedulerJob times and non-Scheduler global variables, but also require a startup time that pertains to the previous job in the schedule. So, my opinion is obviously criticizable.
As a side note, scoring is only a can/cannot run flag in the end, and would better be completely removed at some point. That would spare us some hair.

Mar 25 2019, 7:34 AM · KDE Edu
TallFurryMan requested changes to D20029: Capture counting corrected.
Mar 25 2019, 7:06 AM · KDE Edu

Mar 24 2019

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

Agreed, this differential was here just to provide a code baseline. We've got more urgent stuff to do. Let's see if the forum thread gets traction.

Mar 24 2019, 11:52 PM · KDE Edu
TallFurryMan added a comment to D20001: Simplifying Scheduler.

I fully agree, but that means darkness and weather should go in ScheduleStrategy, and altitude score and moon separation score calculations should go to SchedulerJob.
Also, are you sure you need a separate class for now? Could you keep the code in Scheduler, but move it to a different source file?
That would prepare the codebase with zero risk of regression, and discuss what goes where. scheduler_strategy.cpp and scheduler_schedulerjob.cpp?

Mar 24 2019, 2:17 PM · KDE Edu
TallFurryMan added a comment to D20001: Simplifying Scheduler.

OK mostly, I'd rather have calculation pertaining to SchedulerJob relocated in SchedulerJob. "Strategy" would mean we have multiple ways of scheduling, which is not really the case today. We have "Restrictions" that schedules must comply with, and we have calculations leading to a score. I'm not a fan of that "score" value, but it pertains to the SchedulerJob, it should be the one owning the calculation. I think your previous work in moving stuff there was more interesting (but I can't find it anywhere...).
Let me add an additional comment later on, I'm a bit busy now.

Mar 24 2019, 8:58 AM · KDE Edu
TallFurryMan added a comment to D19998: Remove bypass of preliminary steps when no light frames are needed..

@mutlaqja If you want to mix lights with calibration, I wonder if that is really clearer that you need to remember the progress of jobs in that case. In that situation the scheduler will be able to determine what's left to do, and not redo lights if only calibration remain.
If not remembering, the minimal granularity of the work to do is the full job, which contains in your case lights and calibration.
This may push users to run lights and calibration frames as separate jobs just to be sure on what happens.
But in both cases starting the execution will still schedule the job for capture as if there were lights. There are two sides to this: scheduling and running. This differential is lacking work on scheduling, agreed.
You get a good point. Also a scheduler job with only calibration frames should issue a certain amount of warnings: twilight, altitude, etc. The current UI is not prepared for this...

Mar 24 2019, 8:08 AM · KDE Edu

Mar 23 2019

TallFurryMan requested review of D19998: Remove bypass of preliminary steps when no light frames are needed..
Mar 23 2019, 2:07 PM · KDE Edu

Mar 17 2019

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

That was a "we" for both authors, but the incentive is of course for Jasem to merge if he accepts :)

Mar 17 2019, 2:52 PM · KDE Edu
TallFurryMan requested changes to D19648: Keep GUI parameters for scheduler and capture in sync with row selection.

Capture module.
Functionality tested OK.
This is a change of UI behavior: now clicking once on a job does sync the left fields with the properties of the selected job, as mentioned in the changeset.
However, it makes the end-user think the job can be modified in-place, while there is actually no effect. Counter-however, the change makes adding a new instance of an existing job easier.
I will say the change makes the UI slightly better in terms of usability, but the ergonomy is still flawed.
I observe that arrow buttons are always enabled even if the selected row is first or last while scheduler has this proper (but not always), but this is completely minor. The code doing that should probably be refactored somewhere.

Mar 17 2019, 12:47 PM · KDE Edu
TallFurryMan 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!

Mar 17 2019, 10:33 AM · KDE Edu

Mar 16 2019

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

I think what Jasem means is that tab modules may work with slots and signals internally, but should communicate between each other with dbus. Just as if they were independent processes, possibly distributed over different machines.

Mar 16 2019, 11:07 PM · KDE Edu

Mar 10 2019

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

Shall I give it a try?

Mar 10 2019, 8:27 AM · KDE Edu

Mar 9 2019

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

Ah yes I forgot to comment on this. I think the feature is important because for now we cannot estimate properly how long preliminary steps will take. So that's one more of them for the altitude. It was there before, I don't remove it and I make it customizable, as I will for others. This said, I fully agree it is quite hidden for now. I want to rework the UI of the Scheduler to better manage all these parameters and I will take this into account. But before that, I will follow up on your attempts to divide the code of the Scheduler because frankly that file is horrible :)

Mar 9 2019, 11:16 AM · KDE Edu

Mar 8 2019

TallFurryMan accepted D19528: Making Scheduler robust against guiding problems.
Mar 8 2019, 6:28 PM · KDE Edu
TallFurryMan updated the test plan for D19393: Prevent rescheduling aborted jobs until all jobs are processed.
Mar 8 2019, 6:25 PM · KDE Edu
TallFurryMan added reviewers for D19393: Prevent rescheduling aborted jobs until all jobs are processed: mutlaqja, wreissenberger.
Mar 8 2019, 6:25 PM · KDE Edu
TallFurryMan added a comment to D19393: Prevent rescheduling aborted jobs until all jobs are processed.

D19528 is needed, and is independent of D19393. D19528 does fix a guiding recovery issue, but relies on D19393 to fix a problem with aborted jobs being rescheduled and pushing following jobs away in that situation.
The current discussion on D19393 is about whether all jobs should be left in the original schedule order or completed/errored/invalid jobs should be moved at the end. The current patch implements the second item, but should still be tested. I can do the first item as a later improvement over the original 3.0 design.
I'd say D19528 and D19393 should be merged at the same time in that order. In terms of merge, however, they are independent and should not cause conflicts.

Mar 8 2019, 6:24 PM · KDE Edu

Mar 7 2019

TallFurryMan updated the diff for D19393: Prevent rescheduling aborted jobs until all jobs are processed.

Updated the patch to properly handle aborted jobs.
Now, we can discuss whether it makes sense to reorder completed/invalid/error jobs.
The altitude cutoff, while confusing, remains important to avoid focusing/aligning and finding the job under the restriction when starting capture.

Mar 7 2019, 7:15 PM · KDE Edu
TallFurryMan added a comment to D19393: Prevent rescheduling aborted jobs until all jobs are processed.

When Scheduler assigns a startup date to the job, it considers the altitude restriction and adds the altitude cutoff, then checks if the job has time to run.
On the opposite, when Scheduler is executing the job, it only aborts it when the altitude restriction is crossed. The altitude cutoff is not considered.
Because that's confusing to the end-user, I first provided the customization of that cutoff as an option. That's not extremely clearer that way, but at least a simpler test can be run by setting the cutoff to zero.

Mar 7 2019, 8:02 AM · KDE Edu

Mar 6 2019

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

Thanks, I'll start from here.

Mar 6 2019, 11:14 PM · KDE Edu
TallFurryMan added a comment to D19528: Making Scheduler robust against guiding problems.

Agreed.
It seems the situation is still better than before with your patch. I should be able to push a diff tonight, based on yours. I can rework the part where you bypass the aborted job, and eventually rebase if you change your diff. No problem.

Mar 6 2019, 6:13 PM · KDE Edu
TallFurryMan 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? :-)

That patch was unfinished, but my obs setup accepts a list of differentials to apply when updating itself so I pushed it without reviewers. I like the fact that we both found the same issue at that location :)

Mar 6 2019, 7:48 AM · KDE Edu

Mar 5 2019

TallFurryMan 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 :)

Mar 5 2019, 6:39 AM · KDE Edu

Mar 1 2019

TallFurryMan accepted D19429: Selection of rows in Capture instead of single cells.
Mar 1 2019, 3:36 PM · KDE Edu
TallFurryMan requested changes to D19429: Selection of rows in Capture instead of single cells.

Looks good except for the unrelated changes.

Mar 1 2019, 5:54 AM · KDE Edu

Feb 28 2019

TallFurryMan accepted D19426: Deleting arbitrary imaging sequence in the Capture module corrected..

Good point! Would you care to make the whole line selected too in an additional diff? Right now the selection is per cell, which is confusing.

Feb 28 2019, 5:33 PM · KDE Edu

Feb 27 2019

TallFurryMan planned changes to D19393: Prevent rescheduling aborted jobs until all jobs are processed.

I upload this commit as first step to fixing the issue. Work is still needed to reschedule the aborted jobs when all runnable jobs are complete.

Feb 27 2019, 5:18 PM · KDE Edu
TallFurryMan requested review of D19393: Prevent rescheduling aborted jobs until all jobs are processed.
Feb 27 2019, 5:17 PM · KDE Edu

Feb 2 2019

TallFurryMan added a comment to D18510: Ring-field focusing feature..

Great, I just tested it now and appears to work fairly well. The only issue I noticed now is that the exclusion pattern remains as overlay even when I uncheck "Full Field". It should be disabled by then.

Feb 2 2019, 10:55 AM · KDE Edu
TallFurryMan added a comment to D18235: Meridian flips handled by Mount.

Congratulations to all of you, this really is an improvement to the platform.

Feb 2 2019, 8:41 AM · KDE Edu

Jan 31 2019

TallFurryMan added a reviewer for D18235: Meridian flips handled by Mount: alainz.
Jan 31 2019, 6:13 PM · KDE Edu
TallFurryMan added a comment to D18235: Meridian flips handled by Mount.

D18627 requires attention. Is it compatible/complementary/unnecessary with this diff?

Jan 31 2019, 6:12 PM · KDE Edu

Jan 25 2019

TallFurryMan requested review of D18510: Ring-field focusing feature..
Jan 25 2019, 8:46 AM · KDE Edu

Jan 24 2019

TallFurryMan added a comment to D18235: Meridian flips handled by Mount.

Cristal clear, thanks.

Jan 24 2019, 5:44 AM · KDE Edu

Jan 23 2019

TallFurryMan added a comment to D18235: Meridian flips handled by Mount.

There's something I'd like clarified.
First, if the mount is set east of the meridian at an original position, and is not tracking, and we wait long enough that if it was tracking, it would flip. What happens if we then start tracking from that eastern position? Do we get an unneeded and ineffective flip request?
Second, if a capture overlaps the meridian flip, I don't readily understand whether the capture is aborted, the flip executed, and the capture restarted, or if the capture continues and delays the flip, potentially for very long. This situation could be pre-planned by Scheduler.
I'm sorry, I couldn't test all this. But this seems really good.

Jan 23 2019, 8:26 PM · KDE Edu

Jan 16 2019

TallFurryMan added inline comments to D18235: Meridian flips handled by Mount.
Jan 16 2019, 8:37 AM · KDE Edu

Jan 14 2019

TallFurryMan added a comment to D18235: Meridian flips handled by Mount.

That's great, agreed. I added a few (as usual) pedantic comments. My setup is out of order while I install the fixed obs, so I don't know when I will be able to test.

Jan 14 2019, 12:15 PM · KDE Edu

Jan 7 2019

TallFurryMan added a comment to D18041: Fixed START_AT jobs disregarded while evaluating, and fixed the update of their score..

I found some minor things while playing around, but nothing serious:

  • How does priority and fix startup time correlate? Currently, a job with a lower priority needs to have a earlier startup time. Otherwise, the other one will be marked invalid. To be discussed, I would expect that such a schedule is valid.
  • A culmination determined job, that precedes but overlaps a fix startup time job, is set to aborted. I would expect that the fix startup time job is invalid.
  • A job with fix startup time and duration set to a "repeat until" time that is earlier than the startup time leads to status "completed". Hm, formally correct, but I would consider this as an invalid configuration.
Jan 7 2019, 8:14 PM · KDE Edu
TallFurryMan requested review of D18041: Fixed START_AT jobs disregarded while evaluating, and fixed the update of their score..
Jan 7 2019, 2:43 PM · KDE Edu

Dec 28 2018

TallFurryMan added inline comments to D17159: bug fix #400265: meridian flip check before capturing.
Dec 28 2018, 2:30 PM · KDE Edu

Dec 19 2018

TallFurryMan requested review of D17690: Proper removal of existing jobs by the Mosaic Tool.
Dec 19 2018, 3:46 PM · KDE Edu

Dec 7 2018

TallFurryMan updated the diff for D16429: Rewrite of Scheduler planning.
  • Minor grammar typo.
Dec 7 2018, 7:30 AM · KDE Edu
TallFurryMan updated the diff for D16429: Rewrite of Scheduler planning.
  • Fix regression on current job status.
Dec 7 2018, 7:25 AM · KDE Edu

Dec 3 2018

TallFurryMan updated the test plan for D16429: Rewrite of Scheduler planning.
Dec 3 2018, 7:24 AM · KDE Edu

Dec 2 2018

TallFurryMan added a reviewer for D16429: Rewrite of Scheduler planning: lancaster.
Dec 2 2018, 2:35 AM · KDE Edu
TallFurryMan updated the diff for D16429: Rewrite of Scheduler planning.

Complete list of changes. Job selection highlight is now under control.

Dec 2 2018, 2:33 AM · KDE Edu

Nov 28 2018

TallFurryMan added a comment to D16429: Rewrite of Scheduler planning.

OK the index concept of currentRow and the index concept of selectRow are different. I will move to a custom QModelIndex as soon as possible.

Nov 28 2018, 12:51 PM · KDE Edu
TallFurryMan added a comment to D16429: Rewrite of Scheduler planning.

@wreissenberger Thanks your patch! I observed that when sorting jobs per altitude manually and sorting is not expected to change the order, the selection moves up by itself one row at a time. I'll check this.

Nov 28 2018, 11:07 AM · KDE Edu

Nov 26 2018

TallFurryMan accepted D17159: bug fix #400265: meridian flip check before capturing.
Nov 26 2018, 8:34 PM · KDE Edu
TallFurryMan added a comment to D17148: Make Star Profile Viewer localizable.

In the case of floating point, either QString("%L1").arg() or ki18n().subs() should be used, and HFR is a float (no i18n with args here).
But I didn't investigate whether, while the QString method systematically used the system locale, the ki18n could handle a locale switch in the application without system locale changes.

Nov 26 2018, 8:38 AM · KDE Edu
TallFurryMan requested changes to D17159: bug fix #400265: meridian flip check before capturing.
Nov 26 2018, 6:52 AM · KDE Edu
TallFurryMan added a comment to D17148: Make Star Profile Viewer localizable.

You should use QString("%L1").arg(variable, width, 'f', précision) to properly format floating point values using the decimal point of the locale

Nov 26 2018, 5:43 AM · KDE Edu

Nov 22 2018

TallFurryMan added a comment to F6436162: reorderJobs.diff.

OK that's clear :) thanks for taking the time to check this!

Nov 22 2018, 6:20 AM
TallFurryMan added a comment to D16429: Rewrite of Scheduler planning.

Go ahead and post, I can integrate it to this differential.

Nov 22 2018, 12:10 AM · KDE Edu

Nov 21 2018

TallFurryMan added a comment to D16429: Rewrite of Scheduler planning.

With the option "Sort Jobs per Altitude and Priority" enabled, re-sorting in the edit mode leads to a wrong selection in the queue table.

Test case:

  • Create a schedule with two jobs, where the two jobs have different priorities. Edit the job with the higher priority and change it to a value that its new priority is lower than that of the other job.
  • As a result, the schedule is automatically reordered.
  • Nevertheless, the selection in the queue table remains at the same position.
Nov 21 2018, 5:01 PM · KDE Edu

Nov 20 2018

TallFurryMan added a comment to D16429: Rewrite of Scheduler planning.

Issue found: when changing the start-up constraint from asap to culmination time, and there are two jobs in the list, the second job is overwritten with the first job contents.
Investigation in progress to determine the root cause.

Nov 20 2018, 6:39 AM · KDE Edu

Nov 18 2018

TallFurryMan updated the test plan for D16429: Rewrite of Scheduler planning.
Nov 18 2018, 9:10 AM · KDE Edu

Nov 17 2018

TallFurryMan planned changes to D15837: Fix for the Dome interface.

This went out of my list, I'll check this. I think the main point - the dome - was merged.

Nov 17 2018, 9:50 PM · KDE Edu
TallFurryMan updated the test plan for D16429: Rewrite of Scheduler planning.
Nov 17 2018, 2:33 PM · KDE Edu
TallFurryMan updated the summary of D16429: Rewrite of Scheduler planning.
Nov 17 2018, 2:09 PM · KDE Edu
TallFurryMan updated the test plan for D16429: Rewrite of Scheduler planning.
Nov 17 2018, 2:09 PM · KDE Edu
TallFurryMan updated the test plan for D16429: Rewrite of Scheduler planning.
Nov 17 2018, 2:07 PM · KDE Edu

Nov 15 2018

TallFurryMan added a comment to D16429: Rewrite of Scheduler planning.

Progressing on use cases at https://github.com/TallFurryMan/kstars/wiki.

Nov 15 2018, 7:29 AM · KDE Edu

Nov 12 2018

TallFurryMan updated the summary of D16429: Rewrite of Scheduler planning.
Nov 12 2018, 7:29 AM · KDE Edu

Nov 10 2018

TallFurryMan updated the summary of D16429: Rewrite of Scheduler planning.
Nov 10 2018, 4:11 PM · KDE Edu
TallFurryMan updated the diff for D16429: Rewrite of Scheduler planning.

Differential changes are now complete.
Please start testing now while I write the changeset documentation and describe integration tests!

Nov 10 2018, 4:09 PM · KDE Edu

Nov 9 2018

TallFurryMan updated the diff for D16429: Rewrite of Scheduler planning.

Still WIP, not entirely rebased, but shows were the rewrite is at.
UI is probably prettier than it was before, but that won't help the first Ekos tab.
Altitude sorting is clarified with the UI rework, but clicking the sort button is not stable.
Row selection is still to be fixed.
All schedule consolidation issues are now fixed!
I'm nearly finished, still quite a bit of testing needed.

Nov 9 2018, 8:43 AM · KDE Edu

Nov 3 2018

TallFurryMan planned changes to D16429: Rewrite of Scheduler planning.
Nov 3 2018, 8:39 PM · KDE Edu
TallFurryMan reclaimed D16429: Rewrite of Scheduler planning.
Nov 3 2018, 8:39 PM · KDE Edu
TallFurryMan abandoned D16429: Rewrite of Scheduler planning.
Nov 3 2018, 8:38 PM · KDE Edu
TallFurryMan updated the diff for D16429: Rewrite of Scheduler planning.

Still WIP, need to rebase, fix a few logs and add altitude as a column.

  • Rewrote algorithm documentation.
  • Fixes from Phabricator comments.
Nov 3 2018, 8:38 PM · KDE Edu
TallFurryMan added inline comments to D16429: Rewrite of Scheduler planning.
Nov 3 2018, 8:37 PM · KDE Edu
TallFurryMan planned changes to D16429: Rewrite of Scheduler planning.
Nov 3 2018, 2:52 PM · KDE Edu
TallFurryMan added inline comments to D16429: Rewrite of Scheduler planning.
Nov 3 2018, 2:52 PM · KDE Edu

Nov 2 2018

TallFurryMan updated the diff for D16429: Rewrite of Scheduler planning.

This is still a WIP. Posting advancements to share progress.

Nov 2 2018, 9:08 PM · KDE Edu

Oct 28 2018

TallFurryMan added a comment to D16429: Rewrite of Scheduler planning.

I also plan to add altitude for startup time, with an icon showing whether the target is rising or setting.

Oct 28 2018, 5:48 PM · KDE Edu
TallFurryMan added a comment to D16429: Rewrite of Scheduler planning.

This weekend I found out that most calculations in the scheduler were using the current date instead of the job date (which could be the next day, thus offset with ~15min), and that sometimes local to universal conversions were using the time zone of the system running KStars, instead of the geographic location of the observatory. But trying for instance to calculate the culmination of a target at a future date is somewhat tricky with the current code.

Oct 28 2018, 5:46 PM · KDE Edu
TallFurryMan added a comment to D16429: Rewrite of Scheduler planning.

Thanks for this report!

  1. Yes, agreed, the way the QWidgetTable works for selection is implemented and documented in such a contrived way that it appears to be bugged. SelectRow() should be used, but never results in the right effect. There are apparently several follow-up signals that make a mess with the rendering...
  2. I agree also there, the behavior of the sort option is opaque. Well, for the altitude at least. It works well only when the targets are all asap, and all rising. I plan to change the order predicate so that prioritary targets are the one setting, not the highest ones. Moreover, there's no point in sorting both in priority and altitude probably. Except for infinitely looping jobs, priority is a weird feature now...
Oct 28 2018, 3:48 PM · KDE Edu

Oct 25 2018

TallFurryMan added reviewers for D16429: Rewrite of Scheduler planning: mutlaqja, wreissenberger.
Oct 25 2018, 7:48 PM · KDE Edu