Ticket #2147 (closed unplanned_task: obsolete)

Opened 10 years ago

Last modified 8 years ago

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 10 years ago by deyan

  • Owner set to deyan
  • Status changed from new to s1a_analysis_started

comment:2 Changed 10 years ago by mira

  • Status changed from s1a_analysis_started to s1b_analysis_finished

comment:3 Changed 10 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:4 Changed 10 years ago by deyan

  • Importance set to 99

Batch update from file 0911261.csv

comment:5 Changed 10 years ago by mira

  • Owner changed from deyan to mira
  • Status changed from s1c_analysis_ok to s2a_design_started

comment:6 Changed 10 years ago by mira

  • Status changed from s2a_design_started to s2b_design_finished

comment:7 Changed 10 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 10 years ago by mira

  • Status changed from s2c_design_ok to s3a_implementation_started

comment:9 Changed 10 years ago by mira

  • Status changed from s3a_implementation_started to s3b_implementation_finished

comment:10 Changed 10 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 10 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 10 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 10 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

Merged to the trunk at [8354]...

Too slow again, possible bugs with the text layout...

3.5p

comment:14 Changed 8 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.