Skip to content
This repository was archived by the owner on Aug 15, 2023. It is now read-only.

fix: mongo collection renaming #596

Merged
merged 8 commits into from
Jul 15, 2022
Merged

fix: mongo collection renaming #596

merged 8 commits into from
Jul 15, 2022

Conversation

ascariandrea
Copy link
Collaborator

No description provided.

@vecna
Copy link
Member

vecna commented Jul 12, 2022

would review soon, thanks

@vecna
Copy link
Member

vecna commented Jul 15, 2022

v@NN:~/D/trex/platforms/tktrex/backend$ node bin/count-o-clock.js 
node:internal/modules/cjs/loader:936
  throw err;
  ^

Error: Cannot find module '@shared/utils/decode.utils'
Require stack:
- /home/v/D/trex/platforms/tktrex/backend/lib/utils.js
- /home/v/D/trex/platforms/tktrex/backend/lib/aggregated.js
- /home/v/D/trex/platforms/tktrex/backend/bin/count-o-clock.js

the same command works in yttrex, and this is quite important because is the tool that generates the stats.

I tested it here because one of the commit also imply to update the collection/timevar considered to count how many experiments are registered, but probably the bug is also present in master

@vecna
Copy link
Member

vecna commented Jul 15, 2022

ready to merge for me, but the issue with count-o-clock is blocking

@ascariandrea
Copy link
Collaborator Author

You need to use ts-node to resolve the paths defined by the tsconfig.json. Adding this to the platforms/yttrex/backend/package.json should work:

"count-o-clock": "ts-node bin/count-o-clock.js",

@vecna vecna merged commit e081a6a into daily Jul 15, 2022
@vecna vecna deleted the fix/mongo-collection-renaming branch July 15, 2022 20:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

directives and experiments collection in mongodb deserve review and renaming
2 participants