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

[GUI] 'Add project' wizard: distinguish 'web' and 'attach' urls #2579

Closed
RichardHaselgrove opened this issue Jun 27, 2018 · 11 comments
Closed
Assignees

Comments

@RichardHaselgrove
Copy link
Contributor

RichardHaselgrove commented Jun 27, 2018

The latest online update to all_projects_list.xml contains distinct urls:

        <url>http://www.cpdn.org/cpdnboinc/</url>
        <web_url>http://climateprediction.net/</web_url>

(and likewise for Einstein@Home). This is a good move - it resolves problems I encountered with 7.10 - but it needs placing in the repository for @CharlieFenton and others to use. And the attach wizard needs updating to display it: this is from a fresh checkout/build from master today, with the new file in place.

attach-wizard

Attaching a reconciliation of the various versions of all_projects_list as at 27 June 2018 (Excel speadsheet)

Comparison of All_Projects_List versions.zip

@AenBleidd
Copy link
Member

I can fix this.
Just to clarify:
= Project URL
<web_url> = Web site
Is it correct?

@AenBleidd AenBleidd self-assigned this Jun 27, 2018
@RichardHaselgrove
Copy link
Contributor Author

That would be my interpretation too, but let's not be hasty - we can ask at the conference call tomorrow for full planning documentation.

@drshawnkwang has just commented on the scoping of master_url in #2580, and the issues may be related: let's fix them all at once.

For what it's worth, Andy Bowery explained to me that

The reason is that the climateprediction.net is just a Wordpress site and nothing else. The actual BOINC project is on the www.cpdn.org server.

which (by implication, but unconfirmed) suggests that it can't run php scripts and can't respond to get_project_config.

@drshawnkwang
Copy link
Contributor

Just my unenlightened opinion: I don't think #2580 is related, but I could be wrong.

@davidpanderson
Copy link
Contributor

davidpanderson commented Jun 27, 2018 via email

@RichardHaselgrove
Copy link
Contributor Author

RichardHaselgrove commented Jun 27, 2018

But the non-attach BOINC functions (account management, stats, help desk) still need to be on the designated BOINC server.

@RichardHaselgrove
Copy link
Contributor Author

PIcking up a comment from #2438, and extending the (overdue?) review of the 'Attach project' wizard.

  1. intel_gpu not tested for
  2. other GPU tests search for "[cuda", "[ati" (i.e. keyword at start of plan class string). SETI is not detected as ATI/AMD capable, because its plan classes start "[opencl". Test needs generalising.
  3. it would be user-friendly to add tooltips to the icons: it requires one line of code per icon, such as

m_pProjectDetailsSupportedPlatformWindowsCtrl->SetToolTip(_("Windows"));

@adamradocz
Copy link
Member

@RichardHaselgrove I'd suggest to increase the icons size at least to 36 or 48 pixel wide. Is it possible. Currently this size is barely see-able and not so informative.

@RichardHaselgrove
Copy link
Contributor Author

Yes, they are small, but we're tight on space.

attach wizard

With the tool-tip, it works better, but the dual splodges of green at the right are bad. Why is Android not grouped with the other operating systems? Ideally, we should be able to tell "runs ATI, NV, iGPU under Windows, but not under Linux" (SETI is a bad example because it runs everything under everything: Einstein looks even worse because although the devices are supported, they are not detected in the parsing phase from all_projects_list.xml)

We need a designer to write an overview specification.

@RichardHaselgrove
Copy link
Contributor Author

We have another conference call tomorrow: perhaps we can ask again for some documentation. A search for "web_url" in this repository finds only my opening remark for this issue, so the current all_projects_list hasn't yet been committed.

Attaching an xsd schema (renamed as txt to allow uploading), generated from the live file as a basis for documentation.
all_projects_list.txt

@RichardHaselgrove
Copy link
Contributor Author

I think I'm getting there...

new attach icons

RichardHaselgrove added a commit that referenced this issue Aug 10, 2018
1) Use web_url when present
2) Parse all plan class names from
   http://boinc.berkeley.edu/trac/wiki/AppPlan#Planclassnames
   (except WSL)
3) Update AMD, NV GPU icons to modern style
4) Add icons for Linux/ARM and Intel_GPU
   (icons downscaled from hi-res resource used for projects.php)
5) Re-sequence icons to group OS first, then resources
6) Add tooltips to all icons

Items 2-6 now match projects.php display. Implements #2579
@RichardHaselgrove
Copy link
Contributor Author

Implementation completed in #2643. Closing.

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

No branches or pull requests

6 participants