Ticket #2147 (closed unplanned_task: obsolete)
TEXT_CHAIN_AUTO_R0
Reported by: | deyan | Owned by: | mira |
---|---|---|---|
Priority: | major | Milestone: | X3 |
Component: | uncategorized | Version: | 2.0 |
Keywords: | Cc: | ||
Category: | unknown | Effort: | |
Importance: | 99 | Ticket_group: | |
Estimated Number of Hours: | 0 | Add Hours to Ticket: | 0 |
Billable?: | yes | Total Hours: | 0 |
Analysis_owners: | deyan | Design_owners: | mira, diana |
Imp._owners: | mira, diana, mira | Test_owners: | |
Analysis_reviewers: | meddle | Changelog: | |
Design_reviewers: | pap | Imp._reviewers: | meddle, todor, deyan, pap |
Test_reviewers: | Analysis_score: | 3 | |
Design_score: | 3 | Imp._score: | 3.5 |
Test_score: | 0 |
Description
Change History
comment:1 Changed 15 years ago by deyan
- Owner set to deyan
- Status changed from new to s1a_analysis_started
comment:2 Changed 15 years ago by mira
- Status changed from s1a_analysis_started to s1b_analysis_finished
comment:3 Changed 15 years ago by meddle
- Design_owners set to mira, diana
- Status changed from s1b_analysis_finished to s1c_analysis_ok
- Imp._owners set to mira, diana
- Analysis_reviewers set to meddle
- Analysis_score changed from 0 to 3
I'm not good at analysis reviews, but:
- There are some unclear things that you and mira talked about, but you didn't add them in the analysis, I think if for the designer something is unclear you should add it to the design after conversations...
- The implementation idea is 'много повърхностна'.
- Write the things you talked with mira
3p
comment:5 Changed 15 years ago by mira
- Owner changed from deyan to mira
- Status changed from s1c_analysis_ok to s2a_design_started
comment:6 Changed 15 years ago by mira
- Status changed from s2a_design_started to s2b_design_finished
comment:7 Changed 15 years ago by pap
- Status changed from s2b_design_finished to s2c_design_ok
- Design_score changed from 0 to 3
- Design_reviewers set to pap
- It looks predictable. The ideas are nice but the explanations and quality of the design is ... speechless I am.
- Not a single mentioning of design-related code.
- The bugs you fixed are of some methods that are used a lot and you didn't write any tests about that(or you said so) during the development process.
- As I said I don't like how the ChainingMode enumeration looks like and its persistence-like strings.
- Please reconsider the idea about a centralized place keeping all the chains in the book. It may be a bit faster that what we have now.
- I quite dislike big prolonged tasks as this.
comment:8 Changed 15 years ago by mira
- Status changed from s2c_design_ok to s3a_implementation_started
comment:9 Changed 15 years ago by mira
- Status changed from s3a_implementation_started to s3b_implementation_finished
comment:10 Changed 15 years ago by meddle
- Status changed from s3b_implementation_finished to s2c_design_ok
- Imp._score changed from 0 to 2
- Imp._reviewers set to mira, meddle
We will have to fail the implementation at this stage:
- When you create second chain from the last page of the first chain (for example) in most of the cases either sophie stops responding forever or an exception is thrown Class cast exception from ResourceH to HeadTextFrameH (rarely).
- The first chain is fast, but if you do a second one, that begins from one of the pages of the first either Sophie stops responding forever or it stops responding for more that 10 minutes, which is the same for us... (we talk about 10 pages here...)
- The System.outs and the commented code in the FakeAuthorMain are bad things, but big deal for now, we must think of a way to do the chaining with more than one chain usable....
2p
comment:11 Changed 15 years ago by mira
- Status changed from s2c_design_ok to s3a_implementation_started
I fixed the exceptions.
I removed the System.out`s
The performance is still a problem.
comment:12 Changed 15 years ago by mira
- Status changed from s3a_implementation_started to s3b_implementation_finished
- Imp._owners changed from mira, diana to mira, diana, mira
comment:13 Changed 15 years ago by meddle
- Status changed from s3b_implementation_finished to s3c_implementation_ok
- Imp._score changed from 2 to 3.5
- Imp._reviewers changed from mira, meddle to meddle, todor, deyan, pap
comment:14 Changed 13 years ago by meddle
- Status changed from s3c_implementation_ok to closed
- Resolution set to obsolete
Closing all the tickets before M Y1
Note: See
TracTickets for help on using
tickets.