Skip to content
This repository has been archived by the owner on Jan 17, 2019. It is now read-only.

Is it necessary to have the concept of manifest combination? #22

Closed
iherman opened this issue Sep 5, 2016 · 6 comments
Closed

Is it necessary to have the concept of manifest combination? #22

iherman opened this issue Sep 5, 2016 · 6 comments

Comments

@iherman
Copy link
Member

iherman commented Sep 5, 2016

The current document uses the concept of manifest combination. However, the algorithm is complex, and requires the definition of how to combine each manifest item. This may become very complex. Also, it goes beyond what Web manifests do and may jeopardize reusing the work elsewhere.

@rdeltour
Copy link
Member

rdeltour commented Sep 5, 2016

+1 on the questioning.

As far as I understood, the intent was to describe the situation in the most complete way, so that we're not limited in the solution. That said, I would absolutely vote for a simpler implementation approach with no combination.

@iherman
Copy link
Member Author

iherman commented Sep 5, 2016

On 5 Sep 2016, at 16:41, Romain Deltour notifications@github.com wrote:

+1 on the questioning.

As far as I understood, the intent was to describe the situation in the most complete way, so that we're not limited in the solution. That said, I would absolutely vote for a simpler implementation approach with no combination.

The way I propose rewriting the PWP document is:

  1. the core text would use a simple approach (first come first served, so to say)
  2. move the 'complex' approach (ie, with combination) into an appendix for now, referring back to the corresponding use case), and leave it there for now (essentially, I do not want to 'loose' that part yet…)

WDYT?

@rdeltour
Copy link
Member

rdeltour commented Sep 5, 2016

  1. the core text would use a simple approach (first come first served, so to say) 2. move the 'complex' approach (ie, with combination) into an appendix for now, referring back to the corresponding use case), and leave it there for now (essentially, I do not want to 'loose' that part yet…)

SGTM, that would definitely make the core text easier to read.

@iherman iherman added this to the Sync with the UCR documents milestone Sep 12, 2016
@iherman iherman added To Do and removed To Do labels Sep 12, 2016
@iherman
Copy link
Member Author

iherman commented Sep 12, 2016

Done in e8a57c9

@HadrienGardeur
Copy link

I don't think that's necessary and achievable.

@iherman
Copy link
Member Author

iherman commented Nov 15, 2016

This is a leftover, actually... we had a separate discussion around the UCR document and we already decided to shy away from this.

(It is actually achievable, just fairly complicated...)

Closing the issue.

@iherman iherman closed this as completed Nov 15, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants