ESOUIMods / LoreBooks

An Elder Scrolls Online Addon which displays map pins for Shalidor's Library books and Eidetic Memory Scrolls
Other
1 stars 3 forks source link

PorygonA Updates #7

Open Sharlikran opened 1 year ago

Sharlikran commented 1 year ago

@PorygonA Okay lets just discuss in an issue rather then the comments section now that you are set up.

So the px and pyare the LibGps coordinates for the x, y location of the book. The tagpm` is the primary map ID for the pin. This means it is where you are standing when you get the information and the location the book is to be displayed primarily.

[7114] = { ["e"] = { [1] = { ["px"] = 0.0815220019, ["py"] = 0.5830836017, ["pm"] = 2114, }, }, }, { ["pnx"] = 0.7893669605, ["pny"] = 0.4129191339 }, -- Green Serpent Bounty, systres/u34_systreszone_base_0

The reason for the [7114] = { ["e"] = { and the rest is if it's a new book I have not entered yet. So what you really want is basically this part.

[1] = { ["px"] = 0.0815220019, ["py"] = 0.5830836017, ["pm"] = 2114, },

So essentially that is all you are adding. Then comes the hard part. We will use the book you want to update.

  [7114] = {
    ["cn"] = "Systres Tomes and Scrolls",
    ["n"] = "Green Serpent Bounty",
    ["e"] = {
    },
    ["c"] = false,
  },

The above is what you start with. Then you paste in the line I mentioned.

Step one

  [7114] = {
    ["cn"] = "Systres Tomes and Scrolls",
    ["n"] = "Green Serpent Bounty",
    ["e"] = {
      [1] = { ["px"] = 0.0815220019, ["py"] = 0.5830836017, ["pm"] = 2114, },
    },
    ["c"] = false,
  },

Add the line for the book.

Step two

  [7114] = {
    ["cn"] = "Systres Tomes and Scrolls",
    ["n"] = "Green Serpent Bounty",
    ["e"] = {
      [1] = { ["px"] = 0.0815220019, ["py"] = 0.5830836017, ["pm"] = 2114, },
    },
    ["c"] = true,
  },

Change ["c"] to true this means it has data in it.

Then you have to be a human. Is this on an overland map? If it is only an overland book you are done. If not now you have to ask are you in a city?

For the city remember you are in it at the time you get the book. So your map ID might be 2114 for example for that city. The fake pin slash command could return 10 as an example. I don't know what High Isle is but we will use 9999 to avoid confusion for what it really is. So with that pretend ID number this is how a pin looks when you are entering a pin for a city.

Step three, add the parent zone map ID

  [7114] = {
    ["cn"] = "Systres Tomes and Scrolls",
    ["n"] = "Green Serpent Bounty",
    ["e"] = {
      [1] = { ["px"] = 0.0815220019, ["py"] = 0.5830836017, ["pm"] = 2114, ["zm"] = 9999, },
    },
    ["c"] = true,
  },

Now with that it is complete. You have the pin location while you are standing in the city px, py, and pm and you have told Lorbooks that the parent zone is 9999 by adding that with ["zm"] = 9999,. When you are looking at the entire zone Lorebooks will know how to draw the book in both places.

Dungeon example

  [2011] = {
    ["c"] = true,
    ["cn"] = "Lore and Culture",
    ["n"] = "Mastery of Discipline",
    ["q"] = 4959,
    ["e"] = {
      [1] = { ["px"] = 0.2941044142, ["py"] = 0.3928760078, ["pm"] = 420, ["zx"] = 0.2927644048, ["zy"] = 0.3921875965, ["zm"] = 20, ["d"] = true, },
    },
  },

So this example is a dungeon pin. px, py and pm are the same. It is where you are when you are underground in the dungeon. So what is new is the ["d"] because that means you are in the dungeon since it is set to true. That prevents the pin from being drawn on the overland map off in some weird location you can not walk to.

What makes it draw on the overland map correctly is the zx, zy, and zm. They are the x and y locations of the pin where you are when you get the fake pin information. Which you normally get at the entrance to the delve. Dungeon usually means anything not a city or the overland map like a delve or undercroft.

So that is why the dungeon pin has two sets of coordinates. One set for where you are in the dungeon and one set for the overland map so people see a pin at about the entrance to the Delve.

Sharlikran commented 1 year ago

Now commit: https://github.com/ESOUIMods/LoreBooks/commit/46eb008815d622396448f40daef9abbdfd731a6a is the first change as long as it is just an overland book. Now add the ["zm"] if it is in a city, or the other data if it is in a Delve. Then update the commit on your end to your branch rather then master

PorygonA commented 1 year ago

So the one I'm working on

  [7033] = {
    ["cn"] = "Systres Tomes and Scrolls",
    ["n"] = "Report for the Head Jailer",
    ["e"] = {
      [1] = {
        ["px"] = 0.0763844033, ["py"] = 0.5852776124, ["pm"] = 2114, ["zm"] = , },
    },
    ["c"] = true,
  },

2114 is the High Isle zone number. How exactly do I find the zm for Amenos Station, the name of the city this is found in?

Sharlikran commented 1 year ago

Okay so this is one that may or may not need an extra note. I'm going to say it doesn't but I'm still going to show you anyway. The notes however are not complete sentences so they can fit on the tooltip. For example you don't say, "Go around the corner up the stairs to the Green Plant by the sleeping cat." That would be too long.

First when you get the coordinates there are three slash commands

For this one it is /lbpos and /lbfake

image

That is the Lorebook to be recorded

image

I will refer to this later but notice how I am not really in the building. Your player Arrow is outside the building, sorta. You are underground when you are in the Jailers' Keep. This is a weird concept but something you need to be familiar with as you need to comprehend that as you make fake pins for places where it is more complicated.

This is where I am currently. The primary location for the book. It is always where you are standing. It could be that you are in a Zone, Dungeon, Delve, Undercroft, etc. but try to remember it is where you are physically standing.

When you use /lbpos there is a special command that forces the game to update its internal map information to where you are standing to make sure the location information is accurate. If I didn't do that then because the developers really don't need that there could be information from another location you were at recently.

Knowing that technical stuff isn't needed but I hope it helps you understand why /lbpos behaves the way it does. It can change the map you have selected and if you don't know why then it could be confusing.

Then we type the command to get the information with the book open. If not I could not get the information for the book.

image

image

Then you see the text in chat which would end up being this.

[7033] = { ["e"] = { [1] = { ["px"] = 0.0763696033, ["py"] = 0.5852916124, ["pm"] = 2214, }, }, }, { ["pnx"] = 0.5842999220, ["pny"] = 0.3729238212 }, -- Report for the Head Jailer, systres/u34_amenosstation_city_base_0

There is several things there but what we want for now is the [1] part and we know the book ID is 7033.

  [7033] = {
    ["cn"] = "Systres Tomes and Scrolls",
    ["n"] = "Report for the Head Jailer",
    ["e"] = {
      [1] = { ["px"] = 0.0763696033, ["py"] = 0.5852916124, ["pm"] = 2214, },
    },
    ["c"] = true,
  },

So where you are now is map ID 2214.

image

You don't need that extra window to do this. It's all done for you with the slash command.

Now you need the fake pin information. If it were a Dungeon you enter it differently but for a city where you are standing works for both the city map and the overland map. That's why for a city you only need the other map ID from the parent map. Call it what you want but it's a Map ID. There are other things like a ZoneId or mapIndex but those are different.

To get that you have to zoom the map out first.

image

When you use /lbfake it won't zoom the map into where you are standing so it will get the information for where you are standing in relation to the map you the player have selected. This would be required for a City like Vvardenfel where you have the Map ID for the Zone itself, the map ID for the city, and the map ID for where you are inside the buildings. For example the Fighters Guild. Those need three or sometimes 4 map pins.

That is why /lbfake gives you the information where you are for the map you selected because once entered it will put the book's Map Pin where you feel it should go to direct the user to it.

[fake] = { ["e"] = { [1] = { ["px"] = 0.0763696035, ["py"] = 0.5852916015, ["pm"] = 2114, ["fp"] = true, }, }, }, { ["pnx"] = 0.6805444360, ["pny"] = 0.4595537484 }, -- fake Eidetic Memory location, systres/u34_systreszone_base_0

Now notice the few differences. First it says [fake] and the map texture is at the end. Most of that is there so I know where the player is from what they copy paste into the comments. The part you want, which will initially be confusing is the ["pm"] because that is where you are standing for the map you selected. So the mapID for the overland map is 2114. But I have to have a unique letter for the data files so it's ["zm"] for the zone's map.

  [7033] = {
    ["cn"] = "Systres Tomes and Scrolls",
    ["n"] = "Report for the Head Jailer",
    ["e"] = {
      [1] = { ["px"] = 0.0763696033, ["py"] = 0.5852916124, ["pm"] = 2214, ["zm"] = 2114, },
    },
    ["c"] = true,
  },

The numbers are similar so that can add to the confusion however, that is the correct way to get the pin entered for the City.

Extra Locations Details

Just know this is an option but something I don't feel you need to do on a regular basis and we don't for this book. It's an option for things that are super complicated to find.

This information is under the Language folder and then the respective language like EN. You have to enter the English value for all Languages regardless of whether or not it is translated or there will be an error.

--[[These are added as Location Details or ["ld"]
]]--
LoreBooks.locationDetails = {
  [1] = "Guest Room",
  [2] = "Attic Room",
  [3] = "Hidden Basement",
}

So what would have to happen is another new note would be added and you might make a comment. I haven't because I would normally be the only person.

LoreBooks.locationDetails = {
  [1] = "Guest Room",
  [2] = "Attic Room",
  [3] = "Hidden Basement",
  [4] = "Jailers' Keep",
}

Then the complete pin information for the extra details

  [7033] = {
    ["cn"] = "Systres Tomes and Scrolls",
    ["n"] = "Report for the Head Jailer",
    ["e"] = {
      [1] = { ["px"] = 0.0763696033, ["py"] = 0.5852916124, ["pm"] = 2214, ["zm"] = 2114, ["ld"] = 4, },
    },
    ["c"] = true,
  },

image

Then you have the additional note so you aren't trying to look in the bushes or on a large rock for the book. The reason being there is no small map for the Jailers' Keep yet you will sorta zone under the world in the city into the Jailers' Keep so that's why your player Arrow is off outside the building where you could never really walk to.

I still wouldn't add the extra details but that would be how you get the City pin, how you enter it, how to get the fake pin and use that for the ["zm"] portion, and if it is really needed an extra note to help the user find the book. I prefer we talk about the notes first though.

Sharlikran commented 1 year ago
    ["e"] = {
      [1] = { ["px"] = 0.0763696033, ["py"] = 0.5852916124, ["pm"] = 2214, ["zm"] = 2114, },
    },

I know it's nit-picky but enter all the information on one line. I ask that because when you have say 4 pin locations for a complicated book then I want it that way so that a year form now it's easier to remember which lines belong together for a book. Because you can enter comments at the end like -- Book Name and then everyone would know the 4 lines are for the same book.

Sharlikran commented 1 year ago

@PorygonA After the holidays keep in touch and I can answer any questions. In the forums I can be a bit abrupt but for learning something when I know it is new to you I'm happy to help. Especially if you have trouble with Git For Windows and making updates.

PorygonA commented 1 year ago

Will do.

I've been starting with the books that I've entered the data for, and auditing to make sure they show up on the map. What is the ["zt"]?

An example:

[5450] = { ["c"] = true, ["e"] = { [1] = { ["d"] = true, ["py"] = 0.6902691936, ["zt"] = 1121, ["px"] = 0.5083984238, ["pm"] = 1655, }, }, ["n"] = "Letter to Bakul", ["cn"] = "Anequina Archives", },

Sharlikran commented 1 year ago

That is more or less unused. It is for special cases where I could not convert the old data with the information provided.

If you have a book, if the old data has that but you have all the new data, only enter the new data please.

Sharlikran commented 1 year ago

Not accusing anyone but I do want to mention this to each of you because there are a few books this way. I just don't remember if there was support for the Reach in v23 or not. Plus I have been the only person entering data so far. So I don't see how this got this way.

The data from what is copy pasted into chat is all you need except you may need to rename ["pm"] to ["zm"] for fake pins.

This is incorrect

  [6447] = {
    ["c"] = true,
    ["e"] = {
      [1] = {
        ["py"] = 0.2792400000,
        ["px"] = 0.3814500000,
        ["pm"] = 1814,
        ["mn"] = 42,
        ["d"] = false,
        ["r"] = false,
        ["i"] = 12,
      },
    },
    ["n"] = "Namira's Dance",
    ["cn"] = "The Reach Reader",
  },

This is all that is needed

  [6447] = {
    ["c"] = true,
    ["e"] = {
      [1] = { ["px"] = 0.3814180048, ["py"] = 0.2793443956, ["pm"] = 1814, },
    },
    ["n"] = "Namira's Dance",
    ["cn"] = "The Reach Reader",
  },
Sharlikran commented 1 year ago

@PorygonA and use your branch dev-porygona. Switch to it in Git For Windows. I may rebase and if I do just clone the repo again. You might wan to work on your book entries in the .\live\Addons folder so you see the changes. Then use any free program or a program like BeyondCompare known as a diff program. That way you copy your changes into whatever is current.

I'm not sure how to keep your branch up to date with you being new to Git so not sure how to handle things the best.

Another thing we could do is let me know when you have something to submit and I can rebase the branch first and then you can copy your changes using a diff program so you don't revert other changes. Then I can look at the changes on your branch.

It might take some trial and error to get a good workflow.

Again enjoy the Holidays this can wait.

PorygonA commented 1 year ago

I've only touched High Isle, Rockgrove, and Sunspire. No clue about the Reach thing.

I've been working on my book entries in the \Live\Addons folder, no worries there. I gotta make sure everything actually shows up properly.

I'll have the High Isle + Ascending Tide up soon.

Sharlikran commented 1 year ago

Okay I'll get your branch ready. I appreciate you understanding about what I said. There was no tonality in it or accusation. I'm typing in a browser and no way to position myself to be sure you know it's just for clarification. Thanks.

Sharlikran commented 1 year ago

Because I'm updating the routines to draw the pins and I found this.

  [6395] = {
    ["c"] = true,
    ["e"] = {
      [1] = {
        ["px"] = 0.3426200000,
        ["py"] = 0.2874700000,
        ["pm"] = 1858,
        ["mn"] = 42,
        ["d"] = false,
        ["r"] = false,
        ["i"] = 12,
        ["zm"] = 1814,
      },
    },
    ["n"] = "An Imperial in Markarth",
    ["cn"] = "The Reach Reader",
  },

And I'm like, how? If I did that in the beginning and that was a long time ago when I didn't understand how things worked.

Sharlikran commented 1 year ago
  [5899] = {
    ["c"] = true,
    ["q"] = 6480,
    ["e"] = {
      [1] = { ["px"] = 0.7014312166, ["py"] = 1.5382663980, ["pm"] = 1809, ["zx"] = 0.7014312222, ["zy"] = 1.5382664378, ["zm"] = 1747, ["d"] = true, ["qp"] = true, },
    },
    ["n"] = "Miner's Journal",
    ["cn"] = "Solitude Soliloquies",
  },

["q"] = QuestID to be used for the name and whether or not you are actively doing the quest or whether or not you completed the quest. ["qp"] = true, Quest In Progress. Means you have to have the quest is your Journal for the book to be shown. ["qc"] = true, = Quest Completed. Means you have to have completed the quest for the book to be shown.

["i"] and ["l"] have been depreciated.

Sharlikran commented 1 year ago

@PorygonA The current syntax should be updated for the Readme.

PorygonA commented 1 year ago

Gotcha, looks great!

Question: What exactly should I be doing right now? Yachoor seems to have uploaded all of the data that I posted in the thread, so what exactly needs to be done still?

Sharlikran commented 1 year ago

I had no idea anyone was helping and I would bet he did not think anyone would try to help. I don't think anyone would have. I thought I would have to do everything after the new year like I was planning.

However, I doubt everything has been found or is updated enough.

This is why I'm so picky about how things are entered. Because you could just go to a zone on an alt char that has not done any books and look for books for that character. As you go if you see the new information with one line like I have been suggesting then it's fine, just look at something else. If not and the old format is there then update that.

It's volunteer work anyway. Approach the situation that way. If you feel everything is updated and fine then that's fine you don't have to do anything. I'm pretty laid back about all this. Next year at some point there will be a new zone.

I would also think there are still books that have not been discovered since The Reach because that was the record above I was confused about. I bet that was just part of the conversion process and I had no idea how things worked. Although It's all changed now. I just don't think everything is completed but I really don't have a way to generate a report. So looking at the one line entries vs the multiple line entries is really it. Yachoor did make some that were two lines or just added the dual pin information as one line. Having the ["zm"] is an indicator it's updated at least a little bit.

See where things take you. If you are just out playing and you see something record it and go over it at a later time. Just let me know so I can update your branch before you copy anything to it so it doesn't override things already updated. I can pick though it and go form there.

Sharlikran commented 1 year ago

Another thing as well. There are going to be books you can only see while on the quest. There are also books devs place outside the instance because after you do the quest you can not go there anymore. There has never been a way to handle that. ZOS added two new functions recently that allow me to check if the player is on a quest by the quest ID and if they have completed it without the dependency of LibQuestData.

I know there are some books in Greymoor keep that I have pin locations for but I do not have accurate definitions for them. Meaning if you can't read them or they don't even show up on the table, rock, floor or whatever unless you are on the quest then that needs updated. Because it's confusing if you see a pin and you are running around trying to find it only to realize you can't unless you are doing the story quests.

Those kind of fine details are still needed especially since I just refined the syntax to support Quests and things like that. Then you have the Locations Details where if a pin is pretty hard to find we can add something to help with that.

With quests because I have LibQuestData I can look up the Quest ID by the name. So if you are just playing, leveling a char to 50 and you notice a book you could not find and now it's on the map because you are on the quest, well nothing tells you the quest ID. So just stop a moment, write down the quest name and then get the books location and it's fake pin

Awesome Side Quest Pin FakePin

If you have all three of those things I can update that. If you want to then just make sure you save the information. If I need to refine it I can and then you can see the difference if I make any changes so you can do the same in the future.

If you don't have Quest Map already get that also and then you have both Lorebooks, Quest Map, and Skyshards so you can just level a character and talk to guildies and have fun.

Sharlikran commented 1 year ago
  [338] = {
    ["c"] = true,
    ["q"] = 4060,
    ["e"] = {
      [1] = { ["px"] = 0.6301764233, ["py"] = 0.2815471957, ["pm"] = 61, ["qp"] = true, },
    },
    ["n"] = "Orders from Fildgor",
    ["cn"] = "Plots and Schemes",
  },

I just updated that one. I had to be on an alt and doing the quest but now that is properly updated. I won't see that on other characters now and try to find it, looking under a tent on tables or whatever because it's not there unless you are doing the quest.

I found a book that I could update just deconstructing things.

  [548] = {
    ["c"] = true,
    ["e"] = {
      [1] = { ["px"] = 0.6405204272, ["py"] = 0.2620299999, ["pm"] = 160, ["zm"] = 61, },
      [1] = { <<-- Going to remove this one
        ["mn"] = 13,
        ["py"] = 0.2620300000,
        ["px"] = 0.6405100000,
        ["pm"] = 61,
      },
    },
    ["n"] = "Note from Thulvald's Logging Camp",
    ["cn"] = "Mysteries and Clues",
  },

So the second one was the old info. It has the ["mn"] thing which won't work anymore and the ["pm"] was for the overland map so I could not see it in Windhelm.

You won't know something like that until you are playing and happen along a situation like that.

PorygonA commented 1 year ago

Oh yeah that reminds me, I updated Archdruid Michiel's Orders [7271] on my dev branch. It only shows up on the quest, did I update that right?

Sharlikran commented 1 year ago

With Git For Windows you have to copy the changes from the Addons folder to the Git folder. Then commit the changes. Once that is done then I might be able to see it but I'm not sure where to look.

A Fork is https://github.com/PorygonA/LoreBooks or a copy of the entire project.

A branch is like a copy of the .lua files used for Lorebooks or like how you are in one instance and the group leader is in another instance. You have to travel to the leader to be in the same instance.

The master branch is the main project (group leader) and dev-porygona is like your specific instance.

Until you submit a change to master (which I don't prefer but it's okay) or dev-porygona (which is my preferance) then I can't see the changes.

What did you do after you copied 7271 to your branch, what did you click or do in GitForWindows?

PorygonA commented 1 year ago

Yeah I'm on the dev-porygona branch, and did a commit to dev-porygona with summary of "README Typo, Prologue Book"

image

Here

Sharlikran commented 1 year ago

image

On your fork you have the one branch which is just master.

image

If I go to my repository I see what I did 5 days ago but not anything you are trying to submit.

So no worries we can get it straightened out. I am happy to help. Keep a backup of your changes somewhere so when I update Lorebooks Minion doesn't overwrite it.

For now copy and paste the change into a comment here while we work on getting everything to work for you.

Sharlikran commented 1 year ago

While you are doing that, compare it to my previous mention of the syntax changes.

[338] = {
  ["c"] = true,
  ["q"] = 4060,
  ["e"] = {
    [1] = { ["px"] = 0.6301764233, ["py"] = 0.2815471957, ["pm"] = 61, ["qp"] = true, },
  },
  ["n"] = "Orders from Fildgor",
  ["cn"] = "Plots and Schemes",
},

Now that is for a regular pin on the overland map with no need for a secondary map ID or secondary x,y coordinates. However, that's the correct way to add a pin that will only show up when the quest is in your quest journal and you have not completed the quest yet.

I have to refine and troubleshoot things today but my intention was to have the book show up if you completed the quest regardless of whether or not the game will display the Lorebook. Because that's what the completed pins are anyway when you toggle them on.

PorygonA commented 1 year ago
 [7115] = {
    ["c"] = true,
    ["cn"] = "Systres Tomes and Scrolls",
    ["n"] = "Aurelia's Letter",
    ["q"] = 6791,
    ["e"] = {
    },
    ["c"] = false,
      [1] = { ["px"] = 0.0694004026, ["py"] = 0.6027752052, ["pm"] = 2163, ["zm"] = 2114, },
    }
  },

["cn"] = "Systres Tomes and Scrolls",
    ["n"] = "History of House Dufort",
    ["e"] = {
      [1] = { ["px"] = 0.0715032013, ["py"] = 0.5897176011, ["pm"] = 2114, ["r"] = true, },
    },
    ["c"] = false,
    ["c"] = true,
  },
Sharlikran commented 1 year ago

Close

  [7130] = {
    ["c"] = true,
    ["cn"] = "Systres Tomes and Scrolls",
    ["n"] = "History of House Dufort",
    ["e"] = {
      [1] = { ["px"] = 0.0715032013, ["py"] = 0.5897176011, ["pm"] = 2114, },
    },
  },

For 7130, use what I have above. You can not have ["c"] twice because it sets up a routine to tell Lorebooks that there is information to look at under ["e"]. Also ["r"] does mean random but it is for Bookshelves because random books appear in them.

["r"] has been used incorrectly in the past and I have made changes as I mentioned in the readme.

  [653] = {
    ["c"] = true,
    ["cn"] = "Handbills, Posters, and Decrees",
    ["n"] = "Shad Astula Curriculum",
    ["e"] = {
    },
    ["r"] = true,
    ["m"] = {
      [300] = 25,
      [10] = 34,
      [26] = 8,
      [27] = 5,
    },
  },

That is the correct use for ["r"] because it is accompanied by ["m"] with map IDs and a count. The count is irrelevant. It could as well be just [300] = true,

In the above example ["e"] is empty because that book appears in bookshelves in one of the 4 zones indicated under ["m"].

You can have both ["e"] and ["m"] now because of recent changes.

[7115] = {
    ["c"] = true,
    ["cn"] = "Systres Tomes and Scrolls",
    ["n"] = "Aurelia's Letter",
    ["q"] = 6791,
    ["e"] = {
      [1] = { ["px"] = 0.0694004026, ["py"] = 0.6027752052, ["pm"] = 2163, ["zm"] = 2114, },
    },
  },

The line with px, py, and pm etc goes under ["e"] for the Eidetic Books rather then under ["c"].

Use what I have above instead for what you are working on.

PorygonA commented 1 year ago

Yeah, History of House Dufort shows up in a bookshelf.

Sharlikran commented 1 year ago

I rewrote my comment a few times, so I'd refresh the page.

Sharlikran commented 1 year ago

Also do this to resolve your issue with GitForWindows.

Make sure you have a backup of your work outside of the Repository folder.

image

Open the Repository in the windows explorer and then

image

Go to the parent folder indicated by the red box. The folder before Lorebooks.

image

Delete your clone or delete the folder in the purple box.

Close and repoen GitForWindows and it will tell you it can not find Lorebooks. Then choose Clone Again. After that switch to your branch and see if you can submit a change.

PorygonA commented 1 year ago

After I commit, am I meant to push upstrream?

Sharlikran commented 1 year ago

I believe it should say that. I have not done it that way. If it denies you the ability to do that let me know and I have a special command I can have you try.

Sharlikran commented 1 year ago

or I can add you hang on

Sharlikran commented 1 year ago

In your browser see if you can find where to go to accept an invite to this repository. I believe it will highlight something.

Then try the push to upstream thing you saw.

PorygonA commented 1 year ago

Alrighty, accepted and I believe I've pushed upstream.

Sharlikran commented 1 year ago

Okay let me fetch your changes

Sharlikran commented 1 year ago

image

Yeah I can see it now

Sharlikran commented 1 year ago

Okay so lets try, so we can avoid the learning curve, to submit one change at a time for one book at a time. Then I can use a Git thing called Cherry-Pick and get the books you changed one at a time once I review them.

Sharlikran commented 1 year ago

Question though. You have [7115] showing to be related to the quest 6791, does that book appear only while you are on the quest, after you complete it, or does it appear in that location regardless?

Lorebooks has the ability now to get that nit-picky so unfortunately for quest related books they need to be that nit-picky and accurate.

Sharlikran commented 1 year ago

With [7130] you have indicated that the book can be found in that location OR in a bookshelf. If I go to that location to check myself will find a book on a table, or a Bookshelf? I have a different system for bookshelves.

PorygonA commented 1 year ago

[7115], Aurelia's Letter, shows up on as a certain quest stage during "The Princess Detective", quest 6791. So yes, while on it, but it disappears afterwards.

[7130] shows up on a crate at that location, and at random bookshelves in the zone.

image

Sharlikran commented 1 year ago

Then [7115] needs the ["q"] tag in order to define the quest number, and in the book entry you need ["qp"] = true because that indicates while the quest is "In Progress".

  [7115] = {
    ["c"] = true,
    ["cn"] = "Systres Tomes and Scrolls",
    ["n"] = "Aurelia's Letter",
    ["q"] = 6791,
    ["e"] = {
      [1] = { ["px"] = 0.0694004026, ["py"] = 0.6027752052, ["pm"] = 2163, ["zm"] = 2114, ["qp"] = true, },
    },
  },

Then [7130] is fine if it appears on a crate and in bookshelves.

Sharlikran commented 1 year ago

image

Everything in the green box goes in the book entry line under ["e"]

  [1] = { ["px"] = 0.0694004026, ["py"] = 0.6027752052, ["pm"] = 2163, ["zm"] = 2114, ["qp"] = true, },
Sharlikran commented 1 year ago

Well whatever applies. You don't add things that don't need to be there. For example don't put ["qp"] = false, if it does not relate to a quest that would be redundant.

Sharlikran commented 1 year ago

I'm gonna be unavailable except on Discord for a while.

Sharlikran commented 1 year ago

Your version of 7130 was close it needs to be slightly different.

  [7130] = {
    ["c"] = true,
    ["cn"] = "Systres Tomes and Scrolls",
    ["n"] = "History of House Dufort",
    ["e"] = {
      [1] = { ["px"] = 0.0715032013, ["py"] = 0.5897176011, ["pm"] = 2114, },
    },
    ["r"] = true,
    ["m"] = {
      [2114] = true,
    },
  },

So ["e"] and ["m"] are like the chapter in a book and what is under them are the paragraphs. So there has to be a begin and an end.

    ["e"] = {
      <<insert info here>>
    },
PorygonA commented 1 year ago

Alrighty, updated a ton of Galen stuff.

PorygonA commented 1 year ago

Been a while, how do I get the new empty data in my branch?

Sharlikran commented 1 year ago

Not sure what you mean by new empty data. Plus I can't really add the default book anymore to the database without it messing up the order of things. ESO doesn't save these arrays in order like 1, 2, 3, 4, 5 it's maybe 4968, xxxx, xxxx, xxxx, 5, xxxx, 1 and whatever. It's all mixed up. Just update or add whatever you need to.

I probably won't add the pull request right away but thanks for that, I appreciate it.

I'll help reset your branch after I get everything finished and then you would just reclone the repository.

PorygonA commented 1 year ago

So when Firesong dropped, a bunch of new empty entries for Firesong books got added, like [7467], [7605], [7416] (as an example). I'm wondering how to get the empty ones for Scribes of Fate.

Sharlikran commented 1 year ago

I can't add the default book anymore to the database without it messing up the order of things. ESO doesn't save these arrays in order like 1, 2, 3, 4, 5. They are out of order for example 4968, xxxx, xxxx, xxxx, 5, xxxx, 1 and whatever. It's all mixed up. Just update or add whatever you need to.

I had to quote myself. I can't add that. Add the book when you find it.

PorygonA commented 1 year ago

Submitted the Scribes of Fate books, could you also check and upload the things I submitted on Feb 9, 15, and Mar 10?