Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Zero work time in Science Tool and Science tab #1318

Closed
mokun opened this issue Jun 10, 2024 · 10 comments · Fixed by #1325
Closed

Zero work time in Science Tool and Science tab #1318

mokun opened this issue Jun 10, 2024 · 10 comments · Fixed by #1325
Labels

Comments

@mokun
Copy link
Member

mokun commented Jun 10, 2024

Describe the bug

  • Zero value in Science Tool's work column when total work is reportedly 75% completed

Expected and unexpected behavior

  • Work in sols should be reported in "Study Detail" panel.

Java classes involved

  • StudyDestailPanel, ScientificStudy

Screenshots
If "Phase Completed" is 75%, then a lot of work should have been done by the following collaborators but no work time is reported
image

Hong Kuang shows zero work in Research and Paperwork column
image

Specifications

  • pre-3.8.0

Details

/**
 * A class representing a scientific study.
 */
public class ScientificStudy implements Entity, Temporal, Comparable<ScientificStudy> {
	// POJO holding collaborators effort
	private static final class CollaboratorStats implements Serializable {
		private static final long serialVersionUID = 1L;
		
		double acheivementEarned = 0D;
		double paperWorkTime = 0D;
		double reseachWorkTime = 0D;
		MarsTime lastContribution = null;
		ScienceType contribution;
		
		CollaboratorStats(ScienceType contribution) {
			this.contribution = contribution;
		}
	}
  • The paperWorkTime and researchWorkTime are always zero.
@mokun mokun added the bug label Jun 10, 2024
@mokun
Copy link
Member Author

mokun commented Jun 10, 2024

@bevans2000

I don't know if it was after CollaboratorStats was added that the work time are no longer being tracked.

Can you see what went wrong ?

@bevans2000
Copy link
Member

I'm pretty sure Collab stats were working. I will check the UnitTests to reproduce it

@bevans2000
Copy link
Member

@mokun This is not a bug. If you look the Study is in the Invite Collaborators phase so no one can contribute yet.
I have expanded the LabTest Unit Test to check collaborators contribution and it is working fine. I will commit this.
Having said that, there r some minor improvements we can make to the Science Study tab.

@mokun
Copy link
Member Author

mokun commented Jun 14, 2024

no one can contribute yet.

But why would the Phase Completed shows 75% ?

See the screenshot above.

@mokun
Copy link
Member Author

mokun commented Jun 14, 2024

When I look through any science studies in Science Tool, I have yet to find anyone having done non-zero work.

What does Work mean ?

Even in Study Proposal phase, it should accumulate the work spent in sols. So far, it's always zero.

@bevans2000
Copy link
Member

Because the Invite phase is that much completed. Each phase goes to 100% before advancing

@mokun
Copy link
Member Author

mokun commented Jun 14, 2024

Q: why is the first person's Contribution column always blank ?

@bevans2000
Copy link
Member

No the logic has always been that work is only counted when research is actually being done via a Task
This would be a big change and definitely break the logic on

@mokun
Copy link
Member Author

mokun commented Jun 14, 2024

image

I assume the first person as the lead researcher would do research on a Subject as well, right ?

@bevans2000
Copy link
Member

Primary researcher has to do the Science of the Study. This is a UI problem I can fix.

bevans2000 added a commit that referenced this issue Jun 16, 2024
Changes:
- Resarchers list in ScientificStudy window show contribution from Primary researcher
- Person Science Study tab reworked to use standard UI compnents, e.g. Border and AttributePanel
- Study model uses EntityModel so user can double click to Science Tool

close #1318
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants