Skip to content

Conversation

@fvacek
Copy link
Collaborator

@fvacek fvacek commented May 5, 2025

Competitors perspective removed, functionality moved to Runs:

  • competitors statistics report
  • set class in selected rows context menu
  • ranking_pos table column
  • new entries can be done using insert into table Runs

@fvacek fvacek requested review from arnost00 and paukert May 5, 2025 21:12
@fvacek fvacek force-pushed the remove-competitors branch from acbd3b3 to 71649b6 Compare May 8, 2025 20:29
@arnost00

This comment was marked as resolved.

@arnost00

This comment was marked as resolved.

@arnost00

This comment was marked as resolved.

@arnost00
Copy link
Collaborator

arnost00 commented May 9, 2025

Before merging, we need to update translations.

@fvacek
Copy link
Collaborator Author

fvacek commented May 13, 2025

It seems that the "Draw class" and "Remove drawing" buttons do not call the appropriate action. I was not able to find in the original code how the connection is implemented, whether it relies on automatic connection in Qt. It worked before the merge of runs and competitors.

Tested on action build & local build from this branch.

fixed in 8858cc4

arnost00

This comment was marked as resolved.

@fvacek
Copy link
Collaborator Author

fvacek commented May 15, 2025

How to set start time for new competitor in "Edit competitor" dlg? Field "Runs" is empty.

  1. Button Save will fill runs table, this should be automated somehow, now I don't know how.
  2. double click on start_time will open free (vacant) starting times to assign.

@arnost00

This comment was marked as resolved.

@fvacek
Copy link
Collaborator Author

fvacek commented May 18, 2025

In runs UI is missing menu item "Stations"->"Backup memory"

I'm not sure, if backup memory readout is or was working. Does it worth to keep it?

@arnost00
Copy link
Collaborator

In runs UI is missing menu item "Stations"->"Backup memory"

I'm not sure, if backup memory readout is or was working. Does it worth to keep it?

As i wrote earlier : I'm not sure about "Stations"->"Backup memory", maybe its old and unused.
Do we know of anyone who used this functionality?

@lukaskett
Copy link
Collaborator

I'm not sure, if backup memory readout is or was working. Does it worth to keep it?

As i wrote earlier : I'm not sure about "Stations"->"Backup memory", maybe its old and unused. Do we know of anyone who used this functionality?

We were using backup memory function few years ago. It can be used only for event in contact mode (contactless punch is not stored in the unit memory). There is no need to keep the function in my point of view.

@fvacek
Copy link
Collaborator Author

fvacek commented May 19, 2025

In runs table widget is missing column "IOF ID"

fixed in 185c02a

@fvacek
Copy link
Collaborator Author

fvacek commented May 19, 2025

After commit f7dcd70 I cannot add or remove competitor by [+] [-] buttons. obrazek

Button [-] in RunsWidget::editCompetitors ends on

		if(sel_rows.count() <= 1)
			return;

Button [+] in TableView::insertRow call emit editRowInExternalEditor, which is comented in RunsWidget::RunsWidget on line

// connect(ui->wRunsTableWidget->tableView(), &qfw::TableView::editRowInExternalEditor, this, &RunsWidget::editCompetitor, Qt::QueuedConnection);

fixed in 7273ff5

Copy link
Collaborator

@arnost00 arnost00 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work :)
Thanks

@fvacek fvacek merged commit 7638f2f into qe3 May 23, 2025
2 of 4 checks passed
@fvacek fvacek deleted the remove-competitors branch June 8, 2025 14:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants