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

Minor optimization new beacon process - "Action needed" when adding a project #1762

Closed
Quezacoatl1 opened this issue Jun 27, 2020 · 5 comments · Fixed by #2414
Closed

Minor optimization new beacon process - "Action needed" when adding a project #1762

Quezacoatl1 opened this issue Jun 27, 2020 · 5 comments · Fixed by #2414
Assignees
Milestone

Comments

@Quezacoatl1
Copy link
Contributor

Hi @cyrossignol,

I have just created a new CPID and ran it for a few hours. I sent a beacon. Then I added a few new projects that had another CPID. The Wallet showed "Action needed" and wanted me to send another beacon until I updated those projects in the BOINC Manager and refreshed the CPID list in the wizard.

Could we include a check for this situation an state "You already have sent a beacon but not all CPIDs (Cross Project IDs) do match. Please try to hit the "Update" button in the BOINC Manager when having selected all projects. Wait a few minutes and hit the "Refresh" button in the "Projects" menu"?

In few situations it is also possible that the CPID completely changes to another one (i.e. you start with a new CPID, send a beacon and add an older CPID then). In this situation the wizard should check whether ALL CPIDs are the same but diffrent from the one the beacon used. Then we have to push the user to send a new beacon...

I guess this is like the lowest priority item but may prevent some support requests ;)

Kind Regards
Quez

@cyrossignol
Copy link
Member

cyrossignol commented Jun 28, 2020

Yes, this is a good suggestion. The wallet doesn't have a way to track the previous CPIDs and beacons that it sent since it relies on the blockchain and the local BOINC installation as the sources of truth. This is a planned enhancement 🙂

@jamescowens
Copy link
Member

We will schedule this for Gladys. I don't think we can get it into Fern. We really are in a feature freeze right now, and this is an enhancement, not really a bug.

@jamescowens jamescowens added this to the Gladys milestone Jun 28, 2020
@jamescowens jamescowens modified the milestones: Gladys, Hilda Oct 27, 2020
@jamescowens jamescowens modified the milestones: Hilda, Ingrid Feb 25, 2021
@jamescowens
Copy link
Member

@cyrossignol What do we want to do with this?

@jamescowens
Copy link
Member

We addressed a bit of this with #2404.

@jamescowens
Copy link
Member

I think I have largely solved this. We don't need full historization. @Pythonix in #2404 changed the CPID detection to essentially favor the CPID that is associated with an already activated beacon. #2414 implements a split cpid boolean in the research class and gui researcher model, and then has that drive a warning message for the research summary page.

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

Successfully merging a pull request may close this issue.

3 participants