FreeUKGen / FreeUKRegProductIssues

Repo for user-reported FreeUKReg product issues
2 stars 0 forks source link

County of Yorkshire (ericb) #79

Closed FreeUKGenIssues closed 9 years ago

FreeUKGenIssues commented 9 years ago

Issue reported by ericb at 2014-11-23 09:31:11 UTC Time: 2014-11-23T09:22:52+00:00 Session ID: 13c4b61f6d0fcf4603f34f9e47fe08c1 Problem Page URL: /places Previous Page URL: http://test2.freereg.org.uk/manage_counties/select_action?county=YKS Reported Issue: There is an issue with this County. There are no places or therefore batches linked to it. The County Code either needs taking out of the County Names list, OR a bit of code writing that gets all of the Places & Records for the 3 ridings - ERY, NRY + WRY - since that's where the place names and the data is held. I prefer the latter of these.

If you want a separate not in the Extra Features file for option 2, I will do one. Else it is a snag as it stands. The effect apples in SEARCH as well as in Manage Counties. Does this also apply to the REgion names ENGLAND, SCOTLAND & WALES where they appear in the search county pick list ?

Screenshot

Captainkirkdawson commented 9 years ago

Countries do appear in county pick lists and there are no entries,

Sherlock21 commented 9 years ago

Indeed. But i am suggesting that's illogical. Either populate then as suggested above, Or, If we are not going to populate them, then I am suggesting we take them out of the list instead of confusing all the Researchers.

benwbrum commented 9 years ago

I think I'm with Eric here. The region/country names England, Scotland, and Wales do not actually do anything at present, and I feel like we should make a decision there, as well as of in Yorkshire.

Yorkshire could be accommodated by adding special code for expanding YKS => [ERY,WRY,NRY] within the code, and the benefits (to my mind) might be worth the effort.

But what about "England"? I see three possibilities:

Captainkirkdawson commented 9 years ago

Me too! on accepting there is an issue that needs addressing.

Not sure there is a need for country level searching that would justify the performance impacts so would be inclined to remove until there is a clear use,

As for Yorkshire I was under the impression that Yorkshire actually has a small number of places that are considered unique and that Yorksire is not simply the 3 parts.

Do we really want to get into regional conglomerations there are many we could identify; East Anglia, Midlands, East Midlands etc etc?

K

Kirk Dawson Fairmont Hot Springs, B.C. V0B 1L1

On Sun, Nov 23, 2014 at 6:59 PM, Ben W. Brumfield notifications@github.com wrote:

I think I'm with Eric here. The region/country names England, Scotland, and Wales do not actually do anything at present, and I feel like we should make a decision there, as well as of in Yorkshire.

Yorkshire could be accommodated by adding special code for expanding YKS => [ERY,WRY,NRY] within the code, and the benefits (to my mind) might be worth the effort.

But what about "England"? I see three possibilities:

  • Eliminate "England" and the other countries as menu options.
  • Convert "England" and other countries into display-only elements using HTML OptionGroups to aggregate the constituent counties into a hierarchy within the display list. Those options would not, however, be selectable on their own.
  • Make "England", "Scotland", and "Wales" act like we've proposed for Yorkshire -- expand queries on those options into all of their constituent counties. The down-side is that that would require the search form on the researcher's browser to fetch and render every single place for which we have data in "England" or "Wales". This has proven to be a major performance problem in the past.

— Reply to this email directly or view it on GitHub https://github.com/FreeUKGen/FreeUKGenProductIssues/issues/79#issuecomment-64145338 .

Sherlock21 commented 9 years ago

Now Ben has written it out in this manner, i agree they are similar but not equal issues. I would like Yorkshire to be used as BenB suggests. But lets drop England, Wales & Scotland for the county names list because: A) the quantities get too big and b) if the researcher wants the whole of England, then let them take ALL counties option so long as they have the Names fields in the search - so as not to try and 'get' the whole of the database! answer to Kirks' point is in next email.

ericb I support

Sherlock21 commented 9 years ago

I did reply to Kirk's queries about some places in Yorkshire not being in any of the Ridings - includingYork itself. But that reply seems not to have got into this thread. So here it is again.

The Ainsty of York comprises 21 villages around York and was associated with the City of York - which itself was NOT in any of the Ridings of Yorkshire. In 1835, under the Municipal Reform Act, the Ainsty was disassociated from York, and individual Villages were put into the West Riding and the City of York put into the North Riding. For In FreeReg1, some of the Ainsty villages were places in the County of Yorkshire - but not all were. There was/is no County code for The Ainsty unless one uses Yorkshire but this is not consistent, since all the contents of the 3 Ridings are also in theYorkshire County.

In FreeReg2, having been requested not to recreate The Ainsty as a separate entity, I have put the Ainsty Villages into the relevant Ridings along with York City. Hence there are no places just in Yorkshire in FR2. However, as said before, the logical was forward would be to leave the County Code of YKS and populate it as Ben suggests with a bit of summary coding.

The treatment of Regional Codes (in #90) are different to this case, and a decision on one should not necessarily be applied to both issues.

benwbrum commented 9 years ago

Thanks, Eric -- that's a great deal clearer than what I was thinking.

benwbrum commented 9 years ago

The Yorkshire issue has come up again -- I think we need to research what would be involved in supporting it, and consider it for a future version.

benwbrum commented 9 years ago

Reviewing the source code, I think that we will have some work making chapman_code an array so that records in NRY would contain [NRY, YKS] and match queries for NRY or YKS. That's the approach which would support arbitrary hierarchies (Countries or Regions).

However, we could do a Yorkshire-only approach and modify search_query to convert any search on YKS to a search on each of the constituent ridings. That might even be achievable for version 1.1

Sherlock21 commented 9 years ago

Ben I have written a few email comments in the other View exchanged area today, but now you have put this above response in here, I record here that I would strongly go for the approach in your second para. That is exactly what I had in mind. AND all the Transcriptions will fit that method.

Captainkirkdawson commented 9 years ago

Really is a duplicate of 24

edickens commented 9 years ago

This came in today... I use FreeREG and have contributed Somerset transcriptions.

As a user, I find it very irritating that Yorkshire is split. 99 times out of 100 I know that a subject had a Yorkshire connection but have no idea which Riding I should enter.

I'm sure I'm not alone in this.

Why split Yorkshire?

If you must, why not keep them as they are but also combined under "Yorkshire"?

Brian Jones

Sherlock21 commented 9 years ago

Ben's note ( fourth item down) refers to a previous discussion Ben & Myself had on my suggested solution to the issue of Ridings & County of Yorkshire. - which is what's needed to satisfy Eric Dickens' correspondent's input circulated this morning: (comment above this one). Its clearly an issue with some transcribers so why can't you write my suggested bit of code and satisfy them once and for all?

EricB

FreeUKGenIssues commented 9 years ago

I also favor the YKS -> [NRY, WRY, ERY] expansion in the search code, but think we should address that a bit later, after we work out kinks in the deployment. We're just too close to our release date to be modifying the search engine code right now.

On Sun, Mar 29, 2015 at 7:57 AM, Sherlock21 notifications@github.com wrote:

Ben's note ( fourth item down) refers to a previous discussion Ben & Myself had on my suggested solution to the issue of Ridings & County of Yorkshire. - which is what's needed to satisfy Eric Dickens' correspondent's input circulated this morning: (comment above this one). Its clearly an issue with some transcribers so why can't you write my suggested bit of code and satisfy them once and for all?

EricB

— Reply to this email directly or view it on GitHub https://github.com/FreeUKGen/FreeUKGenProductIssues/issues/79#issuecomment-87408570 .