Skip to content

Conversation

@fvacek
Copy link
Collaborator

@fvacek fvacek commented Jan 23, 2026

No description provided.

@fvacek fvacek requested a review from arnost00 January 23, 2026 21:58
@arnost00
Copy link
Collaborator

2026-01-24 16_27_30-pbm_12_jml_2025_6_tst_courses E1 2026-01-24 16_29_33-pbm_12_jml_2025_6_tst_courses E1 2026-01-24 16_28_04-pbm_12_jml_2025_6_tst_courses E1

Sirka comboboxu dle sirky sloupce je pri automaticke sirce sloupce nepouzitelne :(
Kdyz rozsirim sloupec tak je to citelne. Viz obrazky ...

@arnost00
Copy link
Collaborator

arnost00 commented Jan 24, 2026

A s ohledem na pocty kategorii (a tim i trati) pri poradani zavodu, navrhuji vetsi pocet radku v comboboxu. Napr. 20-30

Seznam kategorii v comboboxu ma 100, viz : m_cbxClasses->setMaxVisibleItems(100);

A s ohledem na pocet kategorii bych zvazil zda combobox nevybavit fitrem (kdy zacnu psat nazev trate a zuzuje se mi seznam trati)

@arnost00

This comment was marked as outdated.

@fvacek fvacek requested review from arnost00 and lukaskett January 24, 2026 21:03
@fvacek
Copy link
Collaborator Author

fvacek commented Jan 24, 2026

A s ohledem na pocty kategorii (a tim i trati) pri poradani zavodu, navrhuji vetsi pocet radku v comboboxu. Napr. 20-30

Seznam kategorii v comboboxu ma 100, viz : m_cbxClasses->setMaxVisibleItems(100);

A s ohledem na pocet kategorii bych zvazil zda combobox nevybavit fitrem (kdy zacnu psat nazev trate a zuzuje se mi seznam trati)

e2304fd

@fvacek fvacek force-pushed the multicourse-classes branch from e2304fd to e1f6718 Compare January 24, 2026 21:06
@arnost00
Copy link
Collaborator

Editing - It looks good for individual races
Editing - I'll look into relays soon.

Card readout - doesn't use custom course at all.

@arnost00
Copy link
Collaborator

I don't know why, but I couldn't manually add the course to the race.
When I add control codes it only write this to log :

2026-01-27T23:32:13.154	Error		SELECT courses.id AS courses__id, courses.name AS courses__name, courses.length AS courses__length, courses.climb AS courses__climb, courses.note AS courses__note, courses.mapCount AS courses__mapCount, COUNT(runs.id) AS run_count, (SELECT GROUP_CONCAT(code) FROM (SELECT 'S' || (code - 10) AS code, position FROM coursecodes INNER JOIN codes ON codes.id = coursecodes.codeId WHERE (coursecodes.courseId = courses.id) AND code >= 10 AND code < 30 UNION SELECT CAST(code AS TEXT) AS code, position FROM coursecodes INNER JOIN codes ON codes.id = coursecodes.codeId WHERE (coursecodes.courseId = courses.id) AND code >= 30 AND code <= 899 UNION SELECT 'F' || (code - 900) AS code, position  FROM coursecodes INNER JOIN codes ON codes.id = coursecodes.codeId WHERE (coursecodes.courseId = courses.id) AND code >= 900 ORDER BY position) )AS code_list, (SELECT COUNT(*) FROM coursecodes LEFT JOIN codes ON codes.id=coursecodes.codeId WHERE (coursecodes.courseId=courses.id AND codes.code >= 30 AND codes.code <= 899)) AS code_count FROM courses INNER JOIN classdefs ON classdefs.courseId=courses.id LEFT JOIN classes ON classes.id=classdefs.classId LEFT JOIN competitors ON competitors.classId=classes.id LEFT JOIN runs ON runs.competitorId=competitors.id AND (runs.isRunning) WHERE (classdefs.stageId=1) AND (id=33) GROUP BY courses.id ORDER BY courses.name 
 ambiguous column name: id Unable to execute statement	query.cpp	50		
2026-01-27T23:32:13.154	Error		"ok == true" - SQL Error: ambiguous column name: id Unable to execute statement
SELECT courses.id AS courses__id, courses.name AS courses__name, courses.length AS courses__length, courses.climb AS courses__climb, courses.note AS courses__note, courses.mapCount AS courses__mapCount, COUNT(runs.id) AS run_count, (SELECT GROUP_CONCAT(code) FROM (SELECT 'S' || (code - 10) AS code, position FROM coursecodes INNER JOIN codes ON codes.id = coursecodes.codeId WHERE (coursecodes.courseId = courses.id) AND code >= 10 AND code < 30 UNION SELECT CAST(code AS TEXT) AS code, position FROM coursecodes INNER JOIN codes ON codes.id = coursecodes.codeId WHERE (coursecodes.courseId = courses.id) AND code >= 30 AND code <= 899 UNION SELECT 'F' || (code - 900) AS code, position  FROM coursecodes INNER JOIN codes ON codes.id = coursecodes.codeId WHERE (coursecodes.courseId = courses.id) AND code >= 900 ORDER BY position) )AS code_list, (SELECT COUNT(*) FROM coursecodes LEFT JOIN codes ON codes.id=coursecodes.codeId WHERE (coursecodes.courseId=courses.id AND codes.code >= 30 AND codes.code <= 899)) AS code_count FROM courses INNER JOIN classdefs ON classdefs.courseId=courses.id LEFT JOIN classes ON classes.id=classdefs.classId LEFT JOIN competitors ON competitors.classId=classes.id LEFT JOIN runs ON runs.competitorId=competitors.id AND (runs.isRunning) WHERE (classdefs.stageId=1) AND (id=33) GROUP BY courses.id ORDER BY courses.name virtual int qf::gui::model::SqlTableModel::reloadRow(int)	sqltablemodel.cpp	525		

@fvacek
Copy link
Collaborator Author

fvacek commented Jan 30, 2026

@arnost00 issues should be fixed, review pls

@arnost00
Copy link
Collaborator

arnost00 commented Feb 2, 2026

@arnost00 issues should be fixed, review pls

I tried it briefly today, and so far I think there are still some problems. Unfortunately I need more time to be able to replicate it repeatedly and describe it in this issue.

@fvacek
Copy link
Collaborator Author

fvacek commented Feb 8, 2026

@arnost00 issues should be fixed, review pls

I tried it briefly today, and so far I think there are still some problems. Unfortunately I need more time to be able to replicate it repeatedly and describe it in this issue.

@arnost00 Do you have some more issues pls?

@arnost00
Copy link
Collaborator

arnost00 commented Feb 8, 2026

If I open a race and then want to open another race (new or existing), it will crash :
2026-02-08 23_34_00-atomic_base h - Quickbox F - Qt Creator
2026-02-08 23_34_45-courseitemdelegate cpp (quickevent_app_quickevent_plugins_Classes_src @ quickbox

@arnost00

This comment was marked as outdated.

@arnost00
Copy link
Collaborator

arnost00 commented Feb 8, 2026

There is probably some problem with displaying courses that do not have a direct category. They are not displayed in the course editor, but they can be assigned to category and competitor.

2026-02-08 23_58_42-courses_test_3 E1 2026-02-08 23_58_49-Formulář 2026-02-08 23_58_57-courses_test_3 E1

But we need to display and edit these tracks too, because of butterflies, odd/even, etc.

@arnost00
Copy link
Collaborator

arnost00 commented Feb 8, 2026

SI readout for the individual race with custom courses in one category looks good.
merged_splits_small

@arnost00
Copy link
Collaborator

arnost00 commented Feb 9, 2026

Relays:

  • You cannot change the course for a single runner by double-clicking on it as in individual race.
  • The course can be changed by right-clicking and selecting "Set course on selected rows". The course will be set in the runs table in database, but it will not be used when reading SI cards and will not be displayed.

SQL table runs :
2026-02-09 01_05_15-QT SQL Monitor
QE table 'Runs' :
2026-02-09 01_04_46-courses_test_4_staf E1

I was hoping it would work in relays as well. If we can set up a special course for each runner, why can't we use it in relays as well (is it a realistic requirement that a competitor needs to use a different course than the one defined by the number)?

@fvacek
Copy link
Collaborator Author

fvacek commented Feb 9, 2026

If I open a race and then want to open another race (new or existing), it will crash : 2026-02-08 23_34_00-atomic_base h - Quickbox F - Qt Creator 2026-02-08 23_34_45-courseitemdelegate cpp (quickevent_app_quickevent_plugins_Classes_src @ quickbox

fixed in 27b321b

@fvacek
Copy link
Collaborator Author

fvacek commented Feb 9, 2026

Crash fixed, two feature issues created, can we continue with merge?

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.

3 participants