When the computer on which Jest tests are executed performs other parallel tasks, some time-sensitive Jest tests may fail.

https://legacy.reactjs.org/docs/introducing-jsx.html#jsx-prevents-injection-attacks

...

• https://www.wikidata.org/wiki/Wikidata:Main_Page
• https://www.wikidata.org/wiki/Wikidata:Wikibase
• https://www.wikidata.org/wiki/Wikidata:OpenStreetMap
• https://www.wikidata.org/wiki/Wikidata:SPARQL_query_service

...

• https://www.wikidata.org/wiki/Wikidata:Glossary
• https://www.wikidata.org/wiki/Help:Items
• https://www.wikidata.org/wiki/Help:Statements
• https://www.wikidata.org/wiki/Help:Sources
• https://www.wikidata.org/wiki/Help:Ranking
• https://www.wikidata.org/wiki/Help:Qualifiers
• https://www.wikidata.org/wiki/Help:Properties
• https://www.wikidata.org/wiki/Help:Data_type

...

• https://en.wikibooks.org/wiki/SPARQL
• https://en.wikibooks.org/wiki/SPARQL/WIKIDATA_Qualifiers,_References_and_Ranks
• https://en.wikibooks.org/wiki/SPARQL/WIKIDATA_Precision,_Units_and_Coordinates

...

• https://www.mediawiki.org/wiki/Wikibase/DataModel/Primer
• https://www.mediawiki.org/wiki/Wikibase/Indexing/RDF_Dump_Format#Data_model
• https://www.mediawiki.org/wiki/Wikidata_Query_Service/User_Manual/MWAPI

https://stackoverflow.com/questions/34719785/how-to-add-submodule-files-to-a-github-release

The goal is to hide both matrices inside factory.

Refactor entry point, add try-catch-finally, improve readability. Previous versions of ASP.NET have Startup.cs and host configuration separately, so that all the bloat is moved to specific files.

The line below makes the while-loop run through O(C) vertices in each iteration.

https://github.com/zhukovdm/smartwalk/blob/1f09d789e668311639e1cd780e98244c161ff991/app/backend/SmartWalk.Core/Heuristics/OgHeuristic.cs#L135C1-L136C1

For almost linear orderings, e.g. Kahn's algorithm would run in O(C) for all iterations worst.

.NET 6.0 end of life is expected on 12 Nov, 2024. Thus, upgrade is expected during the summer 2024.

• https://learn.microsoft.com/en-us/dotnet/core/compatibility/8.0
• https://learn.microsoft.com/en-us/lifecycle/products/microsoft-net-and-net-core

The code might benefit from clearer implementation (e.g. matching capabilities and more).

• https://github.com/MewsSystems/FuncSharp

Since the group of developers was small, no convention was properly followed.

The problem can be encountered during enrichment from the DbPedia knowledge graph. No further investigation has been done as the correctness of the output is not critical.

Actual output

> Fetched N entities for given 100 identifiers.            11:45:45 PM
> Fetched N entities for given 100 identifiers.            11:45:45 PM

Expected output

> Fetched M entities for given 100 identifiers.            11:45:42 PM
> Enriched N from this batch, enriched total X entities.   11:45:42 PM

https://github.com/zhukovdm/smartwalk/blob/0707ea68f395275c576599dba37e5f7524aedbbb/app/backend/SmartWalk.Service/SearchService.cs#L65C8-L65C8

Dependency Injection can be more general, than today's implementation. In particular, we may get rid of TContext and avoid creating explicit handlers as the one below.

EntityService is the correct one.

I was not able to get this test right. All test steps are executed, but actual swap does not happen. Nonetheless, dragging should be possible¹ and works as expected.

Authentification should rely on new HTTP-based endpoints. Dropdown menu from Log in button should offer two options: Standard (new) and Solid (old).

The following two tests shall be merged into one. For some reasons, useNavigate called from a callback does not update the history of a MemoryRouter, but works well with BrowserRouter.

• https://stackoverflow.com/q/73405665

compare with model validation and service-level validation

This query does not work well for some categories because of the recursive pattern. Furthermore, there is really no reason for not getting all possible points in a bounding box. For example, Prague has ~35000 items with location. So, the amount of transferred data is comparatively small.

Test the code at https://cses.fi/problemset/task/1678.

All Singleton classes shall implement lazy instantiation.

• https://csharpindepth.com/Articles/Singleton#lazy
• https://learn.microsoft.com/en-us/dotnet/api/system.lazy-1?view=net-6.0#thread-safety
• https://stackoverflow.com/a/610837

The construction below injects environment variables into a container. However, these variables are not visible in the file, and ${VAR} ends up being undefined. The true value of this particular variable is defined in the ./infra/.env file, while variables for other services are defined in the compose file.

services:
  routing:
    environment:
      - REGION_FILE=...
    entrypoint: ...${REGION_FILE}...

HttpClient has a connection pool and is IDisposable (quite a surprise).

• https://www.stevejgordon.co.uk/introduction-to-httpclientfactory-aspnetcore
• https://learn.microsoft.com/en-us/dotnet/fundamentals/networking/http/httpclient
• https://okyrylchuk.dev/blog/how-to-use-httpclient-properly-in-dotnet/
• https://www.aspnetmonsters.com/2016/08/2016-08-27-httpclientwrong/

Requests towards Wikidata or DBPedia often fail. Make fetch resilient to request failures (try block, etc.) with retry.