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

Refactor out ontology-specific configuration into ontology-specific configuration files #437

Closed
9 of 10 tasks
Yagnap opened this issue Oct 9, 2024 · 10 comments
Closed
9 of 10 tasks
Assignees
Milestone

Comments

@Yagnap
Copy link
Collaborator

Yagnap commented Oct 9, 2024

What new feature do you want?

Most of the times, we hardcoded something concerning subontologies.
Examples, not necessarily complete:

  • <title>SNIK Ontology Graph</title> and icon
  • Filters for subontologies, rdfs:subclassof and sniko:meta
  • Option for separate subontologies
  • Layout > BB/OB Chapter search, subontology connectivity
  • Links (SPARQL Endpoint, RDF Browser, but possibly also Help)
  • If possible, remove hardcoded subontology references while still retaining same functionality and not more complexity than there already is
  • Configuration having "defaultSubOntologies" etc
  • legend at the bottom
  • References to SNIK in I18N
  • ...
@KonradHoeffner
Copy link
Collaborator

KonradHoeffner commented Dec 18, 2024

@Yagnap:
Wie ist der Stand, schaffen wir das noch dieses Jahr zu releasen?
Ich habe es gerade mal ausprobiert mit hitoConf und es sieht super aus, vielen Dank!

@Yagnap
Copy link
Collaborator Author

Yagnap commented Dec 18, 2024

Da würde ich morgen weiter machen, aber an sich gehen alle Funktionen.
Gibt halt nur ein paar Felder, wo dann noch "SNIK" steht oder die Subontologies referenziert werden, obwohl HITO gerade drin ist.
Aber an sich geht es.

@KonradHoeffner
Copy link
Collaborator

Ich hab auch noch was refactored gerade, die SNIK spezifischen Optionen sind jetzt in einem config.ontology.snik Unterobjekt, dadurch sind auch einige typescript-ignore lines weggefallen.

@Yagnap Yagnap modified the milestones: 25.01, 25.06 Jan 13, 2025
@KonradHoeffner
Copy link
Collaborator

Related to #451 (keep config separate after build).

Wie ist der Stand?

@Yagnap
Copy link
Collaborator Author

Yagnap commented Feb 3, 2025

Sollte erstmal soweit fertig sein; @KonradHoeffner kannst du es dir nochmal angucken, ob du noch was findest? Sonst würde ich es erstmal schließen hier.

@KonradHoeffner
Copy link
Collaborator

KonradHoeffner commented Feb 4, 2025

Also jetzt nur mal ganz oberflächlich nach "snik" gesucht findet man noch viel:

js$ ack "snik" -ic | grep -v "config" | grep -v "lang"
layout.ts:2
loadGraphFromSparql.ts:6
sparql.ts:7
rdf.ts:14
property.ts:1
help.ts:9
browser/menu.ts:19
browser/init.ts:15
browser/graph.ts:1
browser/style.ts:3
browser/util.ts:2
browser/contextmenuEdges.ts:2
browser/load.ts:3
browser/teststyle.ts:1
browser/classuse.ts:2
browser/chaptersearch.ts:2
browser/search.ts:1
browser/browsercheck.ts:1
browser/save.ts:6

Aber das scheinen größtenteils die snik-spezifischen Teile und deren Abfragen zu sein, die so ja nach der derzeitigen Methode noch drin sind.
Der beste Weg, das zu prüfen ist wahrscheinlich, HITO zu aktivieren und es manuell zu testen, das mache ich mal.
Der Teil aus der ontology.dist.ts könnte vielleicht auch mit zum Ontology-Part, zumindest der ontology-Part.

        repo: {
            ontology: "https://github.com/snikproject/ontology",
            application: "https://github.com/snikproject/graph",
        },

Der output von npm run build sieht auch OK aus:

dist$ rg snik -M 500 
html/troubleshooting.html
30:		<a href="https://github.com/snikproject/graph/issues">view the list of known issues</a>
32:		<a href="https://github.com/snikproject/graph/issues/new">submit a bug report</a>

html/contribute.html
12:		<a href="https://github.com/snikproject/graph">hosted on GitHub</a>

assets/main-BsBOSsnc.js
13:[Omitted long line with 4 matches]
46:[Omitted long line with 10 matches]
50:[Omitted long line with 4 matches]
66:[Omitted long line with 3 matches]
301:  FROM <http://www.snik.eu/ontology/${t}>
308:  FROM <http://www.snik.eu/ontology/${e}>
312:[Omitted long line with 16 matches]
327:[Omitted long line with 1 matches]
329:[Omitted long line with 2 matches]
355:[Omitted long line with 1 matches]

assets/main-BsBOSsnc.js.map
1:[Omitted long line with 78 matches]

@KonradHoeffner
Copy link
Collaborator

Sollte man den Namen "SNIK Graph" selbst auch ändern können?
Ich meine RickView und LodView heißen immer gleich, aber bei SNIK Graph ist das Problem, dass eine bestimmte Ontologie im Namen steckt.

@KonradHoeffner
Copy link
Collaborator

KonradHoeffner commented Feb 4, 2025

Beim Layouten bekomme ich einen Fehler:

Uncaught TypeError: can't access property "checked", this.separateSubsBox is undefined
    separateSubs menu.ts:97
    action menu.ts:239
    Bd hotkeys.esm.js:373
    Od hotkeys.esm.js:485
    g hotkeys.esm.js:553

Update: Als einziges Element in snikOptions in js/browser/menu.ts wird es gar nicht erzeugt bei HITO, daher schlägt dann das layouten fehl.

KonradHoeffner added a commit that referenced this issue Feb 4, 2025
@KonradHoeffner
Copy link
Collaborator

Die ontology issue templates sind auch ein Problem aber für diese Issue würde ich das mal außer Acht lassen.

@KonradHoeffner
Copy link
Collaborator

OK ich denke wir können es schließen, vielen Dank @Yagnap!
Für einzelne Sachen mache ich dann neue Issues auf gegebenenfalls.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants