stratigraphic / phaser-app

GitHub pages repository to host the PHASER application
2 stars 0 forks source link

Is the add stratigraphic relationship editing currently disabled? #20

Closed Keith-May closed 2 years ago

Keith-May commented 2 years ago

I wanted to add a single relationship for context8503 Above context1437 (which is somehow missing in the XSM10 dataset No data is showing in the stratigraphic relationships on Context tab? Is this part of improving loading efficiency?

cbinding commented 2 years ago

Aagh, yes sorry there is something commented out when performance testing that I've failed to re-enable before putting this version live. The strat controls load all possible context IDs to choose from and I was looking for a better way. I'll fix it.

Keith-May commented 2 years ago

Hi Ceri,

Thanks for responding. No huge deal right now, but while I was testing some data I found a missing relationship for those 2 contexts and noticed I could no longer add it in. It is probably something I only need to do rarely once the context data has been loaded (entered).

I’m still enjoying the whole wealth of possibilities the Temporal Relations work has provided, not least the proof that they are all there implicitly in the matrix data! Thanks so much for getting this running and I’m looking forward to showing James later today.

Cheers K

AHRC Leadership Fellow – The Matrixhttps://www.researchgate.net/project/The-Matrix-connecting-and-re-using-digital-records-and-archives-of-archaeological-investigations?_sg=alaJc57bvwRBeZpJg3BQN5vf-_twsBDNSxp8O7hvb3Aex3dlZhjgKxoiCLQ3Q4-_fSgTz7QoHF65t3_ygfdIiA4mGSpBbCOV70Xr Analytics Department Policy & Evidence Historic England Mobile: 07585 034315 May, K. 2020 The Matrix: Connecting Time and Space in Archaeological Stratigraphic Records and Archives, Internet Archaeology 55. https://doi.org/10.11141/ia.55.8 The Matrix project web page: https://stratigraphic.github.io/matrix/

From: Ceri Binding @.> Sent: 28 January 2022 10:49 To: stratigraphic/phaser-app @.> Cc: May, Keith @.>; Author @.> Subject: Re: [stratigraphic/phaser-app] Is the add stratigraphic relationship editing currently disabled? (Issue #20)

THIS IS AN EXTERNAL EMAIL: do not click any links or open any attachments unless you trust the sender and were expecting the content to be sent to you

Aagh, yes sorry there is something commented out when performance testing that I've failed to re-enable before putting this version live. The strat controls load all possible context IDs to choose from and I was looking for a better way. I'll fix it.

— Reply to this email directly, view it on GitHubhttps://github.com/stratigraphic/phaser-app/issues/20#issuecomment-1024097111, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AH7XKB5UCJRJRRUPFJ75BKTUYJYBHANCNFSM5NAL5YTA. You are receiving this because you authored the thread.Message ID: @.**@.>>

[Historic England Logo]http://www.historicengland.org.uk/

Work with us to champion heritage and improve lives. Read our Future Strategy and get involved at historicengland.org.uk/strategyhttp://historicengland.org.uk/strategy. Follow us: Facebookhttps://www.facebook.com/HistoricEngland | Twitterhttps://twitter.com/HistoricEngland | Instagramhttps://www.instagram.com/historicengland/ Sign up to our newsletterhttp://webmail.historicenglandservices.org.uk/k/Historic-England/historic_england_preference_centre

This e-mail (and any attachments) is confidential and may contain personal views which are not the views of Historic England unless specifically stated. If you have received it in error, please delete it from your system and notify the sender immediately. Do not use, copy or disclose the information in any way nor act in reliance on it. Any information sent to Historic England may become publicly available. We respect your privacy and the use of your information. Please read our full privacy policyhttps://www.historicengland.org.uk/terms/privacy-cookies/ for more information.

cbinding commented 2 years ago

Fixed in deployed version 1.11