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

008/18-34 fixes, round 1 #142

Merged
merged 81 commits into from
Feb 13, 2024
Merged

008/18-34 fixes, round 1 #142

merged 81 commits into from
Feb 13, 2024

Conversation

nvolk
Copy link
Contributor

@nvolk nvolk commented Jan 18, 2024

Add various fixes for 008/18-34 for all types of material. This is the easy half. More complex cases such as BK 008/18-21 are not fixed in this PR.

I'll eventually write a validator that cleans up erronous data from 008/18-34, justifies and sorts data (eg. in BK 008/18-21). After that we can work out merge rules for the rest.

@nvolk nvolk requested a review from ammsalme January 18, 2024 20:06
@nvolk nvolk requested a review from a team as a code owner January 18, 2024 20:06
@nvolk nvolk changed the base branch from main to next January 24, 2024 07:23
Copy link
Contributor

@ammsalme ammsalme left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's check both source and base typeOfMaterial to avoid weirdness.

@nvolk
Copy link
Contributor Author

nvolk commented Jan 25, 2024

Löysin lisää kehitettävää. Elkäätte tehkö mitään toistaiseksi.

nvolk added 24 commits February 12, 2024 12:43
@nvolk
Copy link
Contributor Author

nvolk commented Feb 13, 2024

Nyt tää olis järkevässä kohdassa...

Tuolla on siis kaikki 00X-alueen yhdistämiset kasassa. Joissain marginaalissa tapauksissa, sekä yksittäiset (yksi- tai monimerkkiset arvot) ja useasta merkistä koostuvat kohdat. (Esim. BK kuvitus 008/18-21).

Monimerkkiset

  • säilyttää alkuperäisen relevantit arvot
  • preprendaa lisää arvoja sourcesta, jos mahtuu
  • sorttaa arvot tarvittaessa
    Yli puolelle monimerkkisistä sain geneeriset säännöt kasaan. Loppuja joutui vähän räätälöimään. Erityisesti nature of contents (BK 008/24-27 vs CR 008/24 ja 008/25-27) oli hassu, kun CR:n puolella maksimipituus on 3 merkkiä ja BK:ssa 4... Mutta 008:t ja niitä vastaavat 006:t on mielestä toteutettu täysin.

007:ssa vähän jätin sourcen arvoja ottamatta, kun en jaksanut/osannut kirjoittaa marginaalisten aineistotyyppien monimerkkisille tapauksille aina täydellisiä sääntöjä, ja käytettyjen arvojen määrä Melindassa lähesty nollaa.. (Esim. taktiilit 007/00=f mp. 03-04 ja 06-08 on semanttisesti hyvinkin sekavat, ja kannassa ei oo esimerkkejä. Lisäksi skippasin007/00=m, joku 007/17-22 film inspection date) laistin, kun noita on niin vähän kannassa, että joku raja. Noissa siis sourcesta otetaan kaikki tai ei mitään, mutta erilaisia arvoja ei yritetä yhdistää.

Lisäksi PR:ssä on mukana poikkeussääntö, joka sallii keeppaamattoman 1XX-kentän ja keepatun 7XX-kentän yhdistämisen toisiinsa (1XX-kenttä ei pitäs keepata koskaan.)

Dependenssit on päivitetty (isbn3 et al)

@ammsalme ammsalme merged commit 309ebd3 into next Feb 13, 2024
10 checks passed
ammsalme added a commit that referenced this pull request Feb 15, 2024
* Merge 00X-controlfields (006, 007, 008)

* Allow 1XX vs 7XX to merge even if 7XX has a $9 <KEEP> subfield. NB! $9 <KEEP> is not merged into 1XX.

* Update deps

* 2.1.0-alpha.1

Co-authored-by: nvolk <[email protected]>
@ammsalme ammsalme deleted the formOfItem branch February 15, 2024 11:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants