Skip to content
This repository was archived by the owner on Nov 26, 2025. It is now read-only.

Commit 2861415

Browse files
mcraveiromcraveiro
committed
agile: backlog grooming
1 parent 4af51dc commit 2861415

1 file changed

Lines changed: 101 additions & 29 deletions

File tree

‎doc/agile/v1/sprint_backlog_28.org‎

Lines changed: 101 additions & 29 deletions
Original file line numberDiff line numberDiff line change
@@ -13,29 +13,29 @@
1313
** Active
1414

1515
#+begin: clocktable :maxlevel 3 :scope subtree :indent nil :emphasize nil :scope file :narrow 75 :formula %
16-
#+CAPTION: Clock summary at [2020-11-01 Sun 12:45]
16+
#+CAPTION: Clock summary at [2020-11-01 Sun 16:36]
1717
| <75> | | | | |
1818
| Headline | Time | | | % |
1919
|-------------------------------------------------------------+---------+-------+-------+-------|
20-
| *Total time* | *77:04* | | | 100.0 |
20+
| *Total time* | *80:01* | | | 100.0 |
2121
|-------------------------------------------------------------+---------+-------+-------+-------|
22-
| Stories | 77:04 | | | 100.0 |
23-
| Active | | 77:04 | | 100.0 |
24-
| edit release notes for previous sprint | | | 3:44 | 4.8 |
22+
| Stories | 80:01 | | | 100.0 |
23+
| Active | | 80:01 | | 100.0 |
24+
| edit release notes for previous sprint | | | 3:44 | 4.7 |
2525
| Create a demo and presentation for previous sprint | | | 0:39 | 0.8 |
26-
| Sprint and product backlog grooming | | | 4:23 | 5.7 |
27-
| Directory names and postfixes are PMM properties | | | 5:27 | 7.1 |
26+
| Sprint and product backlog grooming | | | 4:25 | 5.5 |
27+
| Directory names and postfixes are PMM properties | | | 5:27 | 6.8 |
2828
| Move =enabled= and =overwrite= into =enablement_properties= | | | 0:19 | 0.4 |
29-
| Analysis on a formatables refactor | | | 0:31 | 0.7 |
30-
| Tracing of orchestration chains is incorrect | | | 1:00 | 1.3 |
31-
| Presentation for APA | | | 5:11 | 6.7 |
32-
| Move C++ locator into physical model | | | 40:15 | 52.2 |
29+
| Analysis on a formatables refactor | | | 0:31 | 0.6 |
30+
| Tracing of orchestration chains is incorrect | | | 1:00 | 1.2 |
31+
| Presentation for APA | | | 5:11 | 6.5 |
32+
| Move C++ locator into physical model | | | 40:15 | 50.3 |
3333
| Create a video series on the formatables refactor | | | 0:37 | 0.8 |
34-
| Add full and relative path processing to PM | | | 2:27 | 3.2 |
35-
| Create a factory transform for parts and archetype kinds | | | 5:30 | 7.1 |
36-
| Issues with emacs | | | 1:09 | 1.5 |
34+
| Add full and relative path processing to PM | | | 2:27 | 3.1 |
35+
| Create a factory transform for parts and archetype kinds | | | 5:30 | 6.9 |
36+
| Move stand-alone formatables to physical/logical models | | | 8:27 | 10.6 |
37+
| Issues with emacs | | | 1:09 | 1.4 |
3738
| Rename =name= to =codec= name | | | 0:20 | 0.4 |
38-
| Move stand-alone formatables to physical/logical models | | | 5:32 | 7.2 |
3939
#+tblfm: $5='(org-clock-time%-mod @3$2 $2..$4);%.1f
4040
#+end:
4141

@@ -207,6 +207,7 @@ Time spent creating the demo and presentation.
207207
***** The logical-physical space
208208
*** STARTED Sprint and product backlog grooming :story:
209209
:LOGBOOK:
210+
CLOCK: [2020-11-01 Sun 16:34]--[2020-11-01 Sun 16:36] => 0:02
210211
CLOCK: [2020-10-23 Fri 09:41]--[2020-10-23 Fri 10:31] => 0:50
211212
CLOCK: [2020-10-09 Fri 09:51]--[2020-10-09 Fri 10:15] => 0:24
212213
CLOCK: [2020-10-01 Thu 10:17]--[2020-10-01 Thu 10:23] => 0:06
@@ -572,6 +573,30 @@ Notes:
572573
now need to make sure we can still convert archetypes into facets. A
573574
lot of the code is going to break once we add path.
574575

576+
*** COMPLETED Move stand-alone formatables to physical/logical models :story:
577+
CLOSED: [2020-11-01 Sun 16:34]
578+
:LOGBOOK:
579+
CLOCK: [2020-11-01 Sun 16:33]--[2020-11-01 Sun 16:34] => 0:01
580+
CLOCK: [2020-11-01 Sun 16:24]--[2020-11-01 Sun 16:32] => 0:08
581+
CLOCK: [2020-11-01 Sun 16:08]--[2020-11-01 Sun 16:23] => 0:15
582+
CLOCK: [2020-11-01 Sun 13:36]--[2020-11-01 Sun 16:07] => 2:31
583+
CLOCK: [2020-11-01 Sun 11:30]--[2020-11-01 Sun 12:45] => 1:15
584+
CLOCK: [2020-10-31 Sat 13:30]--[2020-10-31 Sat 16:31] => 3:01
585+
CLOCK: [2020-10-31 Sat 11:01]--[2020-10-31 Sat 12:17] => 1:16
586+
:END:
587+
588+
Move the assorted types which are not related to helpers to their
589+
final location:
590+
591+
- =facet_properties=
592+
- =project_items=
593+
- ODB related types.
594+
- streaming properties
595+
- =cpp_standards=
596+
- =test_data_properties=
597+
- =aspect_properties=: c++ and c#. Features are still located in TS
598+
specific namespaces.
599+
575600
*** STARTED Issues with emacs :story:
576601
:LOGBOOK:
577602
CLOCK: [2020-10-02 Fri 14:51]--[2020-10-02 Fri 16:00] => 1:09
@@ -593,22 +618,69 @@ Notes:
593618

594619
- variability is also using the name class.
595620

596-
*** STARTED Move stand-alone formatables to physical/logical models :story:
597-
:LOGBOOK:
598-
CLOCK: [2020-11-01 Sun 11:30]--[2020-11-01 Sun 12:45] => 1:15
599-
CLOCK: [2020-10-31 Sat 13:30]--[2020-10-31 Sat 16:31] => 3:01
600-
CLOCK: [2020-10-31 Sat 11:01]--[2020-10-31 Sat 12:17] => 1:16
601-
:END:
621+
*** Move c++ helper related classes to logical model :story:
602622

603-
Move the assorted types which are not related to helpers to their
604-
final location:
623+
Classes to move:
605624

606-
- =facet_properties=
607-
- =project_items=
608-
- ODB related types.
609-
- streaming properties
610-
- =cpp_standards=
611-
- =test_data_properties=
625+
- =helper_descriptor=
626+
627+
*** Move default constructor work from resolver :story:
628+
629+
At present we are populating the default constructor for the bundle in
630+
the resolver:
631+
632+
#+begin_src c++
633+
void resolver::resolve_feature_template_bundles(const indices& idx,
634+
entities::model& m) {
635+
for (auto& pair : m.variability_elements().feature_template_bundles()) {
636+
auto& fb(*pair.second);
637+
for (auto& ft : fb.feature_templates()) {
638+
resolve_name_tree(m, idx, fb.name(), ft.parsed_type());
639+
if (ft.parsed_type().is_current_simple_type())
640+
fb.requires_manual_default_constructor(true);
641+
}
642+
}
643+
}
644+
#+end_src
645+
646+
This is very confusing because one would assume the resolver just
647+
resolves. We need to move this logic to =technical_space_properties=.
648+
649+
*** Create a technical space specific property for default functions :story:
650+
651+
In assistant we have:
652+
653+
#+begin_src c++
654+
bool assistant::supports_defaulted_functions() const {
655+
return !is_cpp_standard_98();
656+
}
657+
658+
bool assistant::supports_move_operator() const {
659+
return !is_cpp_standard_98();
660+
}
661+
#+end_src
662+
663+
This should really be in =technical_space_properties=. Check to see if
664+
we missed any.
665+
666+
*** Default constructor incorrectly generated in C++ 98 :story:
667+
668+
We have this logic in =technical_space_properties_transform=:
669+
670+
#+begin_src c++
671+
/*
672+
* In C++ 98 we must always create a default constructor because
673+
* we cannot make use of the defaulted functions.
674+
*/
675+
if (is_cpp_standard_98 || src_tsp.requires_manual_default_constructor())
676+
dest_tsp.requires_manual_default_constructor(true);
677+
#+end_src
678+
679+
This is actually incorrect: we can use default constructors in C++ 98,
680+
as long as there are no other constructors. The problem is we are
681+
relying on the default constructor in test data generator so if we fix
682+
this with an =&&= instead of an =||= we break that code. We need to
683+
figure out what the correct implement is.
612684

613685
*** Detect absence of configuration in bundles :story:
614686

0 commit comments

Comments
 (0)