WebCuratorTool / webcurator

The root of the webcurator tool project, containing all modules needed to run a fully functional webcurator tool.
Apache License 2.0
2 stars 1 forks source link

3.1 Harvest Visualization and Patching Documentation #63

Closed obrienben closed 2 years ago

obrienben commented 2 years ago

Changes

To build the docs, instructions are in the webcurator-docs readme

obrienben commented 2 years ago

@hannakoppelaar have fixed the TODOs

hannakoppelaar commented 2 years ago

@leefrank9527 if you're happy with it, I will merge this PR

leefrank9527 commented 2 years ago

@hannakoppelaar Hi Hanna, I'm reading and testing it. Will finish that soon.

leefrank9527 commented 2 years ago

@obrienben Got some comments for the doc:

  1. The description below "Figure 27. Linked domain nodes": Star nodes are a single domain of URLs that also contain the primary seed of the crawl. In fact, both primary seed contained nodes and secondary seed contained nodes will be drew as star bodes.
  2. Can add some description about the right click menu of the domain analysis table below "Figure 30. Domain Analysis", similar to "Figure 29. Node right-click menu".
  3. Above "Figure 36. Filtering and search options", the filtering is available with the Path View as well. But it seems there is a performance problem for a big Path View for a broad filter.
  4. "To reset the view and clear any fitlered or search results, click the reset button image108."" The icon image is not the latest one.
  5. "Figure 52. A running patch in harvest results", "Figure 53. Patch Summary screen": Are they located in the opposite places?
obrienben commented 2 years ago

Thanks @leefrank9527. I have made the relevant updates:

  1. No change - there is already further description below Figure 27 detailing the star node and secondary seeds.
  2. Updated to mention right-click menu for domain analysis table
  3. Updated to mention that filtering is available in Crawler Path view also
  4. Updated screenshots with latest icon
  5. No change - to me the current descriptions match the screenshots

@hannakoppelaar ready for you to merge