Taker loses connection too often to be ignored

I installed the latest version (ghcr.io/itchysats/itchysats/taker:0.4.10)  on my RPi last night and over night it lost connections to the maker way too often. Imho this is no good
docker logs itchysats_web_1 2>&1| grep "Disconnecting due to lack of heartbeat"

2022-03-31 22:41:42  WARN daemon::connection: Disconnecting due to lack of heartbeat seconds_since_heartbeat=20 heartbeat_timestamp=2022-03-31 22:41:21.663311405 +00:00:00
2022-03-31 22:49:22  WARN daemon::connection: Disconnecting due to lack of heartbeat seconds_since_heartbeat=23 heartbeat_timestamp=2022-03-31 22:48:59.328525515 +00:00:00
2022-03-31 23:05:27  WARN daemon::connection: Disconnecting due to lack of heartbeat seconds_since_heartbeat=23 heartbeat_timestamp=2022-03-31 23:05:04.628064967 +00:00:00
2022-03-31 23:05:51  WARN daemon::connection: Disconnecting due to lack of heartbeat seconds_since_heartbeat=20 heartbeat_timestamp=2022-03-31 23:05:31.903511622 +00:00:00
2022-03-31 23:14:56  WARN daemon::connection: Disconnecting due to lack of heartbeat seconds_since_heartbeat=21 heartbeat_timestamp=2022-03-31 23:14:35.322721124 +00:00:00
2022-03-31 23:45:20  WARN daemon::connection: Disconnecting due to lack of heartbeat seconds_since_heartbeat=20 heartbeat_timestamp=2022-03-31 23:45:00.363055529 +00:00:00
2022-04-01 00:08:01  WARN daemon::connection: Disconnecting due to lack of heartbeat seconds_since_heartbeat=21 heartbeat_timestamp=2022-04-01 0:07:40.037393274 +00:00:00
2022-04-01 01:00:52  WARN daemon::connection: Disconnecting due to lack of heartbeat seconds_since_heartbeat=21 heartbeat_timestamp=2022-04-01 1:00:31.091371689 +00:00:00
2022-04-01 01:01:20  WARN daemon::connection: Disconnecting due to lack of heartbeat seconds_since_heartbeat=24 heartbeat_timestamp=2022-04-01 1:00:55.663922496 +00:00:00
2022-04-01 01:01:43  WARN daemon::connection: Disconnecting due to lack of heartbeat seconds_since_heartbeat=20 heartbeat_timestamp=2022-04-01 1:01:22.998385084 +00:00:00
2022-04-01 01:05:58  WARN daemon::connection: Disconnecting due to lack of heartbeat seconds_since_heartbeat=23 heartbeat_timestamp=2022-04-01 1:05:34.944995439 +00:00:00
2022-04-01 01:06:21  WARN daemon::connection: Disconnecting due to lack of heartbeat seconds_since_heartbeat=20 heartbeat_timestamp=2022-04-01 1:06:01.884885181 +00:00:00
2022-04-01 01:11:02  WARN daemon::connection: Disconnecting due to lack of heartbeat seconds_since_heartbeat=24 heartbeat_timestamp=2022-04-01 1:10:37.720660066 +00:00:00
2022-04-01 01:12:32  WARN daemon::connection: Disconnecting due to lack of heartbeat seconds_since_heartbeat=22 heartbeat_timestamp=2022-04-01 1:12:09.546219226 +00:00:00
2022-04-01 01:12:58  WARN daemon::connection: Disconnecting due to lack of heartbeat seconds_since_heartbeat=20 heartbeat_timestamp=2022-04-01 1:12:38.848515161 +00:00:00
2022-04-01 01:16:06  WARN daemon::connection: Disconnecting due to lack of heartbeat seconds_since_heartbeat=23 heartbeat_timestamp=2022-04-01 1:15:43.035288327 +00:00:00
2022-04-01 01:16:28  WARN daemon::connection: Disconnecting due to lack of heartbeat seconds_since_heartbeat=20 heartbeat_timestamp=2022-04-01 1:16:08.452119638 +00:00:00
2022-04-01 01:17:57  WARN daemon::connection: Disconnecting due to lack of heartbeat seconds_since_heartbeat=24 heartbeat_timestamp=2022-04-01 1:17:32.554049214 +00:00:00
2022-04-01 01:18:20  WARN daemon::connection: Disconnecting due to lack of heartbeat seconds_since_heartbeat=20 heartbeat_timestamp=2022-04-01 1:18:00.545456533 +00:00:00
2022-04-01 01:22:05  WARN daemon::connection: Disconnecting due to lack of heartbeat seconds_since_heartbeat=21 heartbeat_timestamp=2022-04-01 1:21:43.499158584 +00:00:00
2022-04-01 01:23:49  WARN daemon::connection: Disconnecting due to lack of heartbeat seconds_since_heartbeat=23 heartbeat_timestamp=2022-04-01 1:23:26.159476725 +00:00:00
2022-04-01 01:26:21  WARN daemon::connection: Disconnecting due to lack of heartbeat seconds_since_heartbeat=24 heartbeat_timestamp=2022-04-01 1:25:57.208446423 +00:00:00
2022-04-01 01:26:44  WARN daemon::connection: Disconnecting due to lack of heartbeat seconds_since_heartbeat=20 heartbeat_timestamp=2022-04-01 1:26:24.916827547 +00:00:00
2022-04-01 01:27:20  WARN daemon::connection: Disconnecting due to lack of heartbeat seconds_since_heartbeat=20 heartbeat_timestamp=2022-04-01 1:27:00.014081001 +00:00:00
2022-04-01 01:28:10  WARN daemon::connection: Disconnecting due to lack of heartbeat seconds_since_heartbeat=24 heartbeat_timestamp=2022-04-01 1:27:45.310570367 +00:00:00
2022-04-01 01:33:26  WARN daemon::connection: Disconnecting due to lack of heartbeat seconds_since_heartbeat=22 heartbeat_timestamp=2022-04-01 1:33:03.967123479 +00:00:00
2022-04-01 01:33:54  WARN daemon::connection: Disconnecting due to lack of heartbeat seconds_since_heartbeat=20 heartbeat_timestamp=2022-04-01 1:33:34.953854616 +00:00:00
2022-04-01 01:42:57  WARN daemon::connection: Disconnecting due to lack of heartbeat seconds_since_heartbeat=23 heartbeat_timestamp=2022-04-01 1:42:34.119915086 +00:00:00
2022-04-01 02:05:57  WARN daemon::connection: Disconnecting due to lack of heartbeat seconds_since_heartbeat=21 heartbeat_timestamp=2022-04-01 2:05:36.237897351 +00:00:00
2022-04-01 02:29:58  WARN daemon::connection: Disconnecting due to lack of heartbeat seconds_since_heartbeat=22 heartbeat_timestamp=2022-04-01 2:29:35.820037103 +00:00:00

My first guess would be that this is related to performance problems on the maker. Let's see what happens after we release https://github.com/itchysats/itchysats/pull/1722.

This was for an old version. The same problem exists on libp2p aswell #takerconnect :/


pairing issue

everytime i try to pair a ps move controller, it gets to verifying connection then it goes back to attempting connection
trying to use a different bluetooth adapter just makes it not connect at all
this was working before idk why its no longer working

logs:
https://pastebin.com/b73MaKen psmoveconfigtool logs
https://pastebin.com/tVGVN1xy psmoveservice logs
hardware:
onboard bluetooth adapter
psmove with the model number of CECH-ZCM1E and the serial number of<PHONE_NUMBER>10 (im not sure about this though because there is a barcode above it so it might just be the barcode number)
forgot to say this but that bluetooth adapter i mentioned does work, it just results in the same thing
it says microsoft on it and windows identifies it as "Microsoft Nano Transreciever v1.0"

also found out that my other psmove works

Did you use Legacy PSMS before PSMS-EX? Remove all "Motion Controller" from your Bluetooth settings and factory reset PSMS-EX configs. Then pair again and upload the new log please thanks!

yes i did use legacy psmoveservice before psmoveservice ex, and where do i factory reset the configs

Either using going in %AppData%/PSMoveService or just download Virtual Device Manager and do it from the UI.

ok i can pair it now, its either because of the new update or factory resetting psmoveservice
logs:
https://pastebin.com/9wgREdNr psmoveservice
https://pastebin.com/Av5Uxp8q psmoveserviceconfigtool


Add 'Note' Feature : Sort Note

Codecov Report

Merging #117 (9836e21) into master (b15c0a2) will decrease coverage by 0.26%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master     #117      +/-   ##
============================================
- Coverage     54.16%   53.90%   -0.27%
  Complexity      616      616
============================================
  Files           135      136       +1
  Lines          2640     2653      +13
  Branches        255      255
============================================
  Hits           1430     1430
- Misses         1119     1132      +13
  Partials         91       91

Impacted Files
Coverage Δ
Complexity Δ

...main/java/seedu/dictionote/logic/LogicManager.java
44.11% <ø> (ø)
3.00 <0.00> (ø)

...edu/dictionote/logic/commands/SortNoteCommand.java
0.00% <0.00%> (ø)
0.00 <0.00> (?)

...eedu/dictionote/logic/parser/DictionoteParser.java
67.64% <0.00%> (-2.05%)
19.00 <0.00> (ø)

src/main/java/seedu/dictionote/model/Model.java
100.00% <ø> (ø)
2.00 <0.00> (ø)

...main/java/seedu/dictionote/model/ModelManager.java
53.71% <0.00%> (-0.91%)
30.00 <0.00> (ø)

src/main/java/seedu/dictionote/model/NoteBook.java
69.23% <0.00%> (-5.77%)
10.00 <0.00> (ø)

...rc/main/java/seedu/dictionote/model/note/Note.java
60.52% <0.00%> (-0.81%)
14.00 <0.00> (ø)

...va/seedu/dictionote/model/note/UniqueNoteList.java
51.35% <0.00%> (-2.94%)
10.00 <0.00> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b15c0a2...9836e21. Read the comment docs.


RemarkLint: switch plugin + fix broken link

RemarkLint: switch plugin
The remark-lint-are-links-valid-alive plugin has been abandoned and doesn't allow to skip false positives.
The remark-lint-no-dead-urls plugin is still maintained and does allow adding some configuration to skip false positives.
This commit switches the one plugin out for the other.
README: fix link to PHPUnit 10 changelog

Oops, this was merged to the wrong branch. Undone and redoing via PR #118.


CSS Compatibility with Vuematerial

The styling will work with both bootstrap and vuematerial

Hello @abhimanusharma ,
I will check this, thanks for your contribution!!

@sethsandaru Please merge this PR

Hi @abhimanusharma
I checked your commit. LGTM.
FYI: this will go to the branch v2.0.1. For me, it's like a patch release to improve the Vue Form Builder / Bug fixings.


Fix empty Postgres array syntax

Closes #527
Empty arrays for Postgres and Redshift are switched from ARRAY[] to '{}' so that PSQL can cast it to the right column type. This would previously give the following error:
ERROR:  cannot determine type of empty array at character XXX
HINT:  Explicitly cast to the desired type, for example ARRAY[]::integer[].

Coverage increased (+0.0005%) to 98.274% when pulling eab15a8e78121e60dd3da78957cf0965ab82d6a7 on Apexal:fix_postgres_empty_arrays into d94277496e2a51b272ae21e9a8b7792bfc355290 on kayak:master.

Coverage increased (+0.0005%) to 98.274% when pulling eab15a8e78121e60dd3da78957cf0965ab82d6a7 on Apexal:fix_postgres_empty_arrays into d94277496e2a51b272ae21e9a8b7792bfc355290 on kayak:master.

Please add a test for the specific case you are trying to solve (i.e. the case that throws an error without the fix).

Please add a test for the specific case you are trying to solve (i.e. the case that throws an error without the fix).


unexpected ')' on Ruby 2.7

SyntaxError:
  /home/travis/.rvm/gems/ruby-2.7.1/gems/console-1.16.2/lib/console/progress.rb:77: syntax error, unexpected ')'
  			@output.info(@subject, ...)
  			                          ^
  /home/travis/.rvm/gems/ruby-2.7.1/gems/console-1.16.2/lib/console/progress.rb:106: syntax error, unexpected end-of-input, expecting `end'
# /home/travis/.rvm/gems/ruby-2.7.1/gems/console-1.16.2/lib/console/logger.rb:10:in `require_relative'
# /home/travis/.rvm/gems/ruby-2.7.1/gems/console-1.16.2/lib/console/logger.rb:10:in `<top (required)>'
# /home/travis/.rvm/gems/ruby-2.7.1/gems/console-1.16.2/lib/console.rb:10:in `require_relative'
# /home/travis/.rvm/gems/ruby-2.7.1/gems/console-1.16.2/lib/console.rb:10:in `<top (required)>'
# /home/travis/.rvm/gems/ruby-2.7.1/gems/async-1.30.3/lib/async/logger.rb:23:in `require'
# /home/travis/.rvm/gems/ruby-2.7.1/gems/async-1.30.3/lib/async/logger.rb:23:in `<top (required)>'
# /home/travis/.rvm/gems/ruby-2.7.1/gems/async-1.30.3/lib/async.rb:24:in `require_relative'
# /home/travis/.rvm/gems/ruby-2.7.1/gems/async-1.30.3/lib/async.rb:24:in `<top (required)>'

This is causing a fair amount of problems, so let me check how easy it is to restore compatibility.

I would personally suggest updating to 2.7.6 too, which will fix this issue.

Okay, after trying it out, the best solution by far is for you to update to Ruby 2.7.3+.


Enable GKE control plane external access on composer

TL;DR
The feature will enable user to connect to GKE control plane through VM in the same VPC but in different subnet.
This feature can be enabled through console, need a way to enable using terraform
Terraform Resources
No response
Detailed design
No response
Additional information
No response

@Rishabh027 can you provide a link to the command you used? My understanding is that the GKE control plane internal IP is only restricted by location [1], not by ranges. Could this be why it's not reachable?

https://cloud.google.com/kubernetes-engine/docs/how-to/private-clusters#cp-global-access


De-quarantine simple clone functest

What this PR does / why we need it:
This PR https://github.com/kubevirt/kubevirt/pull/8120 solved flakieness of "simple clone" functest.
Following this process, since 3 days passed without failures, we can remove the quarantine.
Useful links:
Test grid: https://testgrid.k8s.io/kubevirt-periodics#periodic-kubevirt-e2e-k8s-1.24-sig-storage&width=20&show-stale-tests=
CI search: https://search.ci.kubevirt.io/?search=.with+a+simple+clone.&maxAge=48h&context=1&type=bug%2Bjunit&name=&excludeName=&maxMatches=5&maxBytes=20971520&groupBy=job
Release note:

NONE

/cc @ShellyKa13 @mhenriks

according to the process need to see it doesnt fail for 2 weeks. do you plan to merge only then?

/approve
@brianmcarey and I discussed this and afaik this was actually a code problem. Therefore I think it is not really a flaky test. We can always react fast if we see this flaky.

I also support removing it from quarantine so it is approved I approve :)

/hold cancel


Cannot set DevChat Access Key

The buttons lead me to nowhere...

This screenshot may show the problem more clearly.

@yangbobo2021  After inputing the DevChat access key in the Command Palette, the check still fails and the key doesn't work.

Resolved in devchat-vscode v0.1.30

It seems to have happened again. Now, when I click the button, the input key dialog doesn't appear. The version is the latest one, v0.1.32.


Initialize device / Firmware update for KFDMicro not possible

Unable to update firmware on x86 or x64 gui for KFDMicro.

This feature is not implemented yet.


how to train the model on multiple GPUs?

thanks.

Hi I am not the project maintainer but this question is more oriented towards a question about how PyTorch works. PyTorch has great documentation on multi-gpu training which you can find here https://pytorch.org/tutorials/beginner/ddp_series_intro.html
You will need to modify some of the variables in the train() function to do match the above documentation. It believe it is fairly straightforward as long as you are familiar with pytorch

thanks~


Xcode9's new build system produces : No such module 'minizip'

When using Zip with Xcode9's new build system, we are getting the "No such module 'minizip'" error. This is not an issue when using Xcode9 with the old build system or Xcode 8.3.3.
We just wanted to flag this as an issue.
Thanks.

We have worked around this by including the Zip project in our workspace. This seems to solve the issue, but is not ideal.

This should be fixed with #119.


[bugfix] danger on PR from forks

@VahidGarousi Can you please include detailed information about the bugs for future contributors?


Run: pass missing env variables

Description

In the past, I opened this PR: https://github.com/dapr/cli/pull/770. Changes look like a mess. I close the PR and made this one.
CLI only passes 03 env variables to applications. See run.go#L185-L187
There are more.  See Docs
Python SDK has a different view on such variables. See global_settings.py. This leads to not-working tutorial

Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[x] Add tests for CLI (run command) to validate env variables
[x] Modify code to pass the missing variables
[x] Refactor code (run.go can be cleaner)
[x] Update documentation: https://github.com/dapr/docs/pull/1835

@yaron2 PTAL

LGTM overall.
@mukundansundar can you please take one last look before we merge?

@mukundansundar I wish this PR is merged on Jan 1st 2022. Would you pls make my dream come true?

LGTM overall ...

Overall looks good ... But my question still remains as to whether we need to expose these variables as ENV vars ... that too without DAPR prefix ...

@mukundansundar Happy new year :)
For your question, I already made a PR in our docs. Pls see

The PR: https://github.com/dapr/docs/pull/1835
The doc: https://docs.dapr.io/reference/environment/

@yaron2 @mukundansundar  yay. You make my day. Thank you!

@mukundansundar - Please prioritize this to get into 1.6 CLI release.


A11y_SQLAzureDataStudio_AddAzureAccount_AddAzureAccount_ScreenReader:The tree view is announced as table

“Check out Accessibility Insights! - Identify accessibility bugs before check-in and make bug fixing faster and easier.”
GitHubTags:#A11y_SQLAzureDataStudioBenchmark;#A11yMAS;#A11ySev2;#A11yTCS;#SQL Azure Data Studio;#Benchmark;#DesktopApp;#MacOs;#MAS1.3.1;#VoiceOver;#SQLADS-MAC-Feb2020;#AINotInScope;
Environment Details:
Application Name: Azure Data Studio
Version: 1.43.0-insider (Universal)
Commit: d3d594e8267f536c481bb3e4cdfefde56ddfdb59
Date: 2023-03-09T18:15:11.027Z
VS Code: 1.67.0
Electron: 19.1.8
Chromium: 102.0.5005.167
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
OS: Darwin arm64 22.4.0
Additional Details:
Mac OS: Ventura 13.3
Screen reader: Voice Over.
MAS References: MAS1.3.1
Repro Steps:
Launch Azure Data Studio application.
Connect to server with windows credentials.
Navigate to Connections from left menu.
Select Azure, navigate to + button & hit enter.
Navigate to Add account button in right side window & hit enter.
Launch screen reader as Voice over
Actual:
On navigating to the tree view items "Steffi " it is announced as table.
Each item is announced as list box.
There is not level announced for the items.
Expected:
On navigating to the tree view items "Steffi " it is not a table so the word table should not be announced.
Each item should be announced as a button
There should be proper hierarchy levels for the items.
User Impact:
Screen reader users will receive the wrong information of the structure as well as the control and hierarchy will be conveyed in a wrong manner.
Attachment link for Reference:

Reference bug: https://github.com/microsoft/azuredatastudio/issues/9234

@BhanuPrakashGit do you know if this bug exists in the intel Mac version of ADS? I only ask because I do not have access to an ARM64 device for debugging.

Hi @BhanuPrakashGit, https://github.com/cssuh/treeViewExample/blob/main/test1.html I have made a simple example tree view where the voiceover on mac will read it as a table, even though its role is a treeview.  It can also be seen here: https://microsoftedge.github.io/DevToolsSamples/bugs/list-as-table.html and in the W3 example for treeview where voiceover on mac will announce the treeview as a table.
This only happens in voiceover on mac, as these treeviews do not get identified as a table in Windows.

Logged External IRT bug.

Closing as this is an external bug


fix(examination-timetable.njk): remove links to project overview page

Remove links to overview page from exam timetable page.
Also make the closed date footer logic only appear if the exam timetable is shown.

:tada: This PR is included in version 2.7.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 2.6.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


deepcase cannot resolve its dependencies in local installation

How to reproduce
git clone https://github.com/deep-foundation/dev.git
cd dev
npm ci
rm -rf packages/deepcase
rm -rf packages/deepcase-app
rm -rf packages/deeplinks
npm run packages

npm run local

Error
2:59:50 PM - Starting compilation in watch mode...
[4]
[0]
[0] ><EMAIL_ADDRESS>dev
[0] > next dev "-p" "3007"
[0]
[2]
[2] ><EMAIL_ADDRESS>docker
[2] > cross-env JWT_SECRET="{\"type\":\"HS256\",\"key\":\"3EK6FD+o0+c7tzBNVfjpMkNDi2yARAAKzQlk8O2IKoxQu4nF7EdAh8s3TwpHwrdWT6R\"}" HASURA_GRAPHQL_DATABASE_URL=postgres://postgres:postgrespassword@postgres:5432/postgres HASURA_GRAPHQL_ENABLE_CONSOLE=true HASURA_GRAPHQL_DEV_MODE=true HASURA_GRAPHQL_LOG_LEVEL=debug HASURA_GRAPHQL_ENABLED_LOG_TYPES="startup, http-log, webhook-log, websocket-log, query-log" HASURA_GRAPHQL_ADMIN_SECRET=myadminsecretkey HASURA_GRAPHQL_ENABLE_REMOTE_SCHEMA_PERMISSIONS=true HASURA_GRAPHQL_UNAUTHORIZED_ROLE=undefined POSTGRES_USER=postgres POSTGRES_PASSWORD=postgrespassword PGGSSENCMODE=disable PGSSLMODE=disable PGREQUIRESSL=0 MINIO_ROOT_USER=minioaccesskey MINIO_ROOT_PASSWORD=miniosecretkey HASURA_STORAGE_DEBUG=true HASURA_METADATA=1 HASURA_ENDPOINT=http://host.docker.internal:8080/v1 S3_ENDPOINT=http://host.docker.internal:9000 S3_ACCESS_KEY=minioaccesskey S3_SECRET_KEY=miniosecretkey S3_BUCKET=default S3_ROOT_FOLDER=default POSTGRES_MIGRATIONS=0 POSTGRES_MIGRATIONS_SOURCE=postgres://postgres:postgrespassword@host.docker.internal:5432/postgres?sslmode=disable npm run docker-local
[2]
[1]
[1] ><EMAIL_ADDRESS>dev
[1] > (cd local && docker-compose down || true) && cross-env PORT=3006 npm run launch
[1]
[3]
[3] ><EMAIL_ADDRESS>dev
[3] > next dev
[3]
[1] time="2024-01-09T14:59:50+06:00" level=warning msg="The \"MOESIF_TOKEN\" variable is not set. Defaulting to a blank string."
[1] time="2024-01-09T14:59:50+06:00" level=warning msg="The \"GITPOD_GIT_USER_EMAIL\" variable is not set. Defaulting to a blank string."
[1] time="2024-01-09T14:59:50+06:00" level=warning msg="The \"RESERVED_LIFETIME_MS\" variable is not set. Defaulting to a blank string."
[1] time="2024-01-09T14:59:50+06:00" level=warning msg="The \"MIGRATIONS_DATE_TYPE_SQL\" variable is not set. Defaulting to a blank string."
[1] time="2024-01-09T14:59:50+06:00" level=warning msg="The \"DEFAULT_RL_CRON_SCHEDULE\" variable is not set. Defaulting to a blank string."
[1] time="2024-01-09T14:59:50+06:00" level=warning msg="The \"MIGRATIONS_RL_TABLE\" variable is not set. Defaulting to a blank string."
[1] time="2024-01-09T14:59:50+06:00" level=warning msg="The \"DEEPLINKS_HASURA_STORAGE_URL\" variable is not set. Defaulting to a blank string."
[1] time="2024-01-09T14:59:50+06:00" level=warning msg="The \"MIGRATIONS_SCHEMA\" variable is not set. Defaulting to a blank string."
[1] time="2024-01-09T14:59:50+06:00" level=warning msg="The \"DEEPLINKS_PORT\" variable is not set. Defaulting to a blank string."
[1] time="2024-01-09T14:59:50+06:00" level=warning msg="The \"GITPOD_TASKS\" variable is not set. Defaulting to a blank string."
[0] ready - started server on <IP_ADDRESS>:3007, url: http://localhost:3007
[2]
[2] ><EMAIL_ADDRESS>docker-local
[2] > cd ./local && docker-compose -p deep up -d && cd ../
[2]
[3] - ready started server on <IP_ADDRESS>:4000, url: http://localhost:4000
[2]  Container deep-postgres  Running
[2]  Container deep-minio  Running
[2]  Container deep-hasura  Running
[2]  Container deep-hasura-storage  Starting
[1]
[1] ><EMAIL_ADDRESS>launch
[1] > npm run package:build && node ./index.js;
[1]
[2]  Container deep-hasura-storage  Started
[3] - info Loaded env from /home/freephoenix888/Programming/dev/packages/sdk/.env.development
[2] npm run local-engine exited with code 0
[0] warn  - Invalid next.config.js options detected:
[0] warn  -     The root value has an unexpected property, webpack5, which is not in the list of allowed properties (amp, analyticsId, assetPrefix, basePath, cleanDistDir, compiler, compress, configOrigin, crossOrigin, devIndicators, distDir, env, eslint, excludeDefaultMomentLocales, experimental, exportPathMap, generateBuildId, generateEtags, headers, httpAgentOptions, i18n, images, modularizeImports, onDemandEntries, optimizeFonts, output, outputFileTracing, pageExtensions, poweredByHeader, productionBrowserSourceMaps, publicRuntimeConfig, reactStrictMode, redirects, rewrites, sassOptions, serverRuntimeConfig, skipMiddlewareUrlNormalize, skipTrailingSlashRedirect, staticPageGenerationTimeout, swcMinify, target, trailingSlash, transpilePackages, typescript, useFileSystemPublicRoutes, webpack).
[0] warn  -     The root value has an unexpected property, future, which is not in the list of allowed properties (amp, analyticsId, assetPrefix, basePath, cleanDistDir, compiler, compress, configOrigin, crossOrigin, devIndicators, distDir, env, eslint, excludeDefaultMomentLocales, experimental, exportPathMap, generateBuildId, generateEtags, headers, httpAgentOptions, i18n, images, modularizeImports, onDemandEntries, optimizeFonts, output, outputFileTracing, pageExtensions, poweredByHeader, productionBrowserSourceMaps, publicRuntimeConfig, reactStrictMode, redirects, rewrites, sassOptions, serverRuntimeConfig, skipMiddlewareUrlNormalize, skipTrailingSlashRedirect, staticPageGenerationTimeout, swcMinify, target, trailingSlash, transpilePackages, typescript, useFileSystemPublicRoutes, webpack).
[0] warn  -     The root value has an unexpected property, strictMode, which is not in the list of allowed properties (amp, analyticsId, assetPrefix, basePath, cleanDistDir, compiler, compress, configOrigin, crossOrigin, devIndicators, distDir, env, eslint, excludeDefaultMomentLocales, experimental, exportPathMap, generateBuildId, generateEtags, headers, httpAgentOptions, i18n, images, modularizeImports, onDemandEntries, optimizeFonts, output, outputFileTracing, pageExtensions, poweredByHeader, productionBrowserSourceMaps, publicRuntimeConfig, reactStrictMode, redirects, rewrites, sassOptions, serverRuntimeConfig, skipMiddlewareUrlNormalize, skipTrailingSlashRedirect, staticPageGenerationTimeout, swcMinify, target, trailingSlash, transpilePackages, typescript, useFileSystemPublicRoutes, webpack).
[0] warn  - See more info here: https://nextjs.org/docs/messages/invalid-next-config
[3] - warn Invalid next.config.js options detected:
[3] - warn     The root value has an unexpected property, strictMode, which is not in the list of allowed properties (amp, analyticsId, assetPrefix, basePath, cleanDistDir, compiler, compress, configOrigin, crossOrigin, devIndicators, distDir, env, eslint, excludeDefaultMomentLocales, experimental, exportPathMap, generateBuildId, generateEtags, headers, httpAgentOptions, i18n, images, modularizeImports, onDemandEntries, optimizeFonts, output, outputFileTracing, pageExtensions, poweredByHeader, productionBrowserSourceMaps, publicRuntimeConfig, reactStrictMode, redirects, rewrites, sassOptions, serverRuntimeConfig, skipMiddlewareUrlNormalize, skipTrailingSlashRedirect, staticPageGenerationTimeout, swcMinify, target, trailingSlash, transpilePackages, typescript, useFileSystemPublicRoutes, webpack).
[3] - warn See more info here: https://nextjs.org/docs/messages/invalid-next-config
[1]
[1] ><EMAIL_ADDRESS>package:build
[1] > tsc --project tsconfig.json
[1]
[0] warn  - The `app` directory is experimental. To enable, add `appDir: true` to your `next.config.js` configuration under `experimental`. See https://nextjs.org/docs/messages/experimental-app-dir-config
[0] warn  - The `app` directory is experimental. To enable, add `appDir: true` to your `next.config.js` configuration under `experimental`. See https://nextjs.org/docs/messages/experimental-app-dir-config
[3] - event compiled client and server successfully in 213 ms (20 modules)
[3] - wait compiling...
[0] event - compiled client and server successfully in 754 ms (186 modules)
[3] - event compiled client and server successfully in 173 ms (20 modules)
[0] warn  - The `app` directory is experimental. To enable, add `appDir: true` to your `next.config.js` configuration under `experimental`. See https://nextjs.org/docs/messages/experimental-app-dir-config
[1] automation/remove-build-files.ts(1,1): error TS1208: 'remove-build-files.ts' cannot be compiled under '--isolatedModules' because it is considered a global script file. Add an import, export, or an empty 'export {}' statement to make it a module.
[1] npm run deeplinks exited with code 2
[0] wait  - compiling...
[0] event - compiled successfully in 107 ms (39 modules)
[4] imports/client-handler.tsx(8,29): error TS2307: Cannot find module 'axios-hooks' or its corresponding type declarations.
[4] imports/package-publisher.tsx(6,30): error TS2307: Cannot find module 'axios-hooks' or its corresponding type declarations.
[4] imports/packager-interface/packager-interface-tabs-content.tsx(13,30): error TS2307: Cannot find module 'axios-hooks' or its corresponding type declarations.
[4] imports/packager-interface/packager-interface.tsx(13,30): error TS2307: Cannot find module 'axios-hooks' or its corresponding type declarations.
[4] imports/test.ts(1,1): error TS1208: 'test.ts' cannot be compiled under '--isolatedModules' because it is considered a global script file. Add an import, export, or an empty 'export {}' statement to make it a module.
[4]
[4] 3:00:08 PM - Found 5 errors. Watching for file changes.
[0] wait  - compiling / (client and server)...
[0] warn  - The `app` directory is experimental. To enable, add `appDir: true` to your `next.config.js` configuration under `experimental`. See https://nextjs.org/docs/messages/experimental-app-dir-config
[0] error - ../deepcase/imports/client-handler.js:29:0
[0] Module not found: Can't resolve 'axios-hooks'
[0]   27 | import { useMinilinksFilter } from "@deep-foundation/deeplinks/imports/minilinks";
[0]   28 | import axios from 'axios';
[0] > 29 | import * as axiosHooks from 'axios-hooks';
[0]   30 | import * as classnames from 'classnames';
[0]   31 | import React, { useEffect, useRef, useState } from 'react';
[0]   32 | import * as motion from 'framer-motion';
[0]
[0] https://nextjs.org/docs/messages/module-not-found
[0]
[0] Import trace for requested module:
[0] ../deepcase/imports/cyto/editor.js
[0] ./pages/index.tsx
[0] warn  - The `app` directory is experimental. To enable, add `appDir: true` to your `next.config.js` configuration under `experimental`. See https://nextjs.org/docs/messages/experimental-app-dir-config
[0] warn  - The `app` directory is experimental. To enable, add `appDir: true` to your `next.config.js` configuration under `experimental`. See https://nextjs.org/docs/messages/experimental-app-dir-config
[0] wait  - compiling /_error (client and server)...
[0] warn  - The `app` directory is experimental. To enable, add `appDir: true` to your `next.config.js` configuration under `experimental`. See https://nextjs.org/docs/messages/experimental-app-dir-config
[0] error - ../deepcase/imports/client-handler.js:29:0
[0] Module not found: Can't resolve 'axios-hooks'
[0]   27 | import { useMinilinksFilter } from "@deep-foundation/deeplinks/imports/minilinks";
[0]   28 | import axios from 'axios';
[0] > 29 | import * as axiosHooks from 'axios-hooks';
[0]   30 | import * as classnames from 'classnames';
[0]   31 | import React, { useEffect, useRef, useState } from 'react';
[0]   32 | import * as motion from 'framer-motion';
[0]
[0] https://nextjs.org/docs/messages/module-not-found
[0]
[0] Import trace for requested module:
[0] ../deepcase/imports/cyto/editor.js
[0] ./pages/index.tsx

I have tried to do the same again and the same error happens. Reproducible:
3:05:59 PM - Starting compilation in watch mode...
[4]
[1]
[1] ><EMAIL_ADDRESS>dev
[1] > (cd local && docker-compose down || true) && cross-env PORT=3006 npm run launch
[1]
[3]
[3] ><EMAIL_ADDRESS>dev
[3] > next dev
[3]
[0]
[0] ><EMAIL_ADDRESS>dev
[0] > next dev "-p" "3007"
[0]
[2]
[2] ><EMAIL_ADDRESS>docker
[2] > cross-env JWT_SECRET="{\"type\":\"HS256\",\"key\":\"3EK6FD+o0+c7tzBNVfjpMkNDi2yARAAKzQlk8O2IKoxQu4nF7EdAh8s3TwpHwrdWT6R\"}" HASURA_GRAPHQL_DATABASE_URL=postgres://postgres:postgrespassword@postgres:5432/postgres HASURA_GRAPHQL_ENABLE_CONSOLE=true HASURA_GRAPHQL_DEV_MODE=true HASURA_GRAPHQL_LOG_LEVEL=debug HASURA_GRAPHQL_ENABLED_LOG_TYPES="startup, http-log, webhook-log, websocket-log, query-log" HASURA_GRAPHQL_ADMIN_SECRET=myadminsecretkey HASURA_GRAPHQL_ENABLE_REMOTE_SCHEMA_PERMISSIONS=true HASURA_GRAPHQL_UNAUTHORIZED_ROLE=undefined POSTGRES_USER=postgres POSTGRES_PASSWORD=postgrespassword PGGSSENCMODE=disable PGSSLMODE=disable PGREQUIRESSL=0 MINIO_ROOT_USER=minioaccesskey MINIO_ROOT_PASSWORD=miniosecretkey HASURA_STORAGE_DEBUG=true HASURA_METADATA=1 HASURA_ENDPOINT=http://host.docker.internal:8080/v1 S3_ENDPOINT=http://host.docker.internal:9000 S3_ACCESS_KEY=minioaccesskey S3_SECRET_KEY=miniosecretkey S3_BUCKET=default S3_ROOT_FOLDER=default POSTGRES_MIGRATIONS=0 POSTGRES_MIGRATIONS_SOURCE=postgres://postgres:postgrespassword@host.docker.internal:5432/postgres?sslmode=disable npm run docker-local
[2]
[2]
[2] ><EMAIL_ADDRESS>docker-local
[2] > cd ./local && docker-compose -p deep up -d && cd ../
[2]
[0] ready - started server on <IP_ADDRESS>:3007, url: http://localhost:3007
[1] time="2024-01-09T15:06:00+06:00" level=warning msg="The \"DEEPLINKS_HASURA_STORAGE_URL\" variable is not set. Defaulting to a blank string."
[1] time="2024-01-09T15:06:00+06:00" level=warning msg="The \"RESERVED_LIFETIME_MS\" variable is not set. Defaulting to a blank string."
[1] time="2024-01-09T15:06:00+06:00" level=warning msg="The \"MIGRATIONS_DATE_TYPE_SQL\" variable is not set. Defaulting to a blank string."
[1] time="2024-01-09T15:06:00+06:00" level=warning msg="The \"DEFAULT_RL_CRON_SCHEDULE\" variable is not set. Defaulting to a blank string."
[1] time="2024-01-09T15:06:00+06:00" level=warning msg="The \"GITPOD_TASKS\" variable is not set. Defaulting to a blank string."
[1] time="2024-01-09T15:06:00+06:00" level=warning msg="The \"MIGRATIONS_SCHEMA\" variable is not set. Defaulting to a blank string."
[1] time="2024-01-09T15:06:00+06:00" level=warning msg="The \"MIGRATIONS_RL_TABLE\" variable is not set. Defaulting to a blank string."
[1] time="2024-01-09T15:06:00+06:00" level=warning msg="The \"DEEPLINKS_PORT\" variable is not set. Defaulting to a blank string."
[1] time="2024-01-09T15:06:00+06:00" level=warning msg="The \"GITPOD_GIT_USER_EMAIL\" variable is not set. Defaulting to a blank string."
[1] time="2024-01-09T15:06:00+06:00" level=warning msg="The \"MOESIF_TOKEN\" variable is not set. Defaulting to a blank string."
[3] - ready started server on <IP_ADDRESS>:4000, url: http://localhost:4000
[2]  Container deep-minio  Running
[2]  Container deep-postgres  Running
[2]  Container deep-hasura  Running
[2]  Container deep-hasura-storage  Running
[3] - info Loaded env from /home/freephoenix888/Programming/dev/packages/sdk/.env.development
[2] npm run local-engine exited with code 0
[0] warn  - Invalid next.config.js options detected:
[0] warn  -     The root value has an unexpected property, webpack5, which is not in the list of allowed properties (amp, analyticsId, assetPrefix, basePath, cleanDistDir, compiler, compress, configOrigin, crossOrigin, devIndicators, distDir, env, eslint, excludeDefaultMomentLocales, experimental, exportPathMap, generateBuildId, generateEtags, headers, httpAgentOptions, i18n, images, modularizeImports, onDemandEntries, optimizeFonts, output, outputFileTracing, pageExtensions, poweredByHeader, productionBrowserSourceMaps, publicRuntimeConfig, reactStrictMode, redirects, rewrites, sassOptions, serverRuntimeConfig, skipMiddlewareUrlNormalize, skipTrailingSlashRedirect, staticPageGenerationTimeout, swcMinify, target, trailingSlash, transpilePackages, typescript, useFileSystemPublicRoutes, webpack).
[0] warn  -     The root value has an unexpected property, future, which is not in the list of allowed properties (amp, analyticsId, assetPrefix, basePath, cleanDistDir, compiler, compress, configOrigin, crossOrigin, devIndicators, distDir, env, eslint, excludeDefaultMomentLocales, experimental, exportPathMap, generateBuildId, generateEtags, headers, httpAgentOptions, i18n, images, modularizeImports, onDemandEntries, optimizeFonts, output, outputFileTracing, pageExtensions, poweredByHeader, productionBrowserSourceMaps, publicRuntimeConfig, reactStrictMode, redirects, rewrites, sassOptions, serverRuntimeConfig, skipMiddlewareUrlNormalize, skipTrailingSlashRedirect, staticPageGenerationTimeout, swcMinify, target, trailingSlash, transpilePackages, typescript, useFileSystemPublicRoutes, webpack).
[0] warn  -     The root value has an unexpected property, strictMode, which is not in the list of allowed properties (amp, analyticsId, assetPrefix, basePath, cleanDistDir, compiler, compress, configOrigin, crossOrigin, devIndicators, distDir, env, eslint, excludeDefaultMomentLocales, experimental, exportPathMap, generateBuildId, generateEtags, headers, httpAgentOptions, i18n, images, modularizeImports, onDemandEntries, optimizeFonts, output, outputFileTracing, pageExtensions, poweredByHeader, productionBrowserSourceMaps, publicRuntimeConfig, reactStrictMode, redirects, rewrites, sassOptions, serverRuntimeConfig, skipMiddlewareUrlNormalize, skipTrailingSlashRedirect, staticPageGenerationTimeout, swcMinify, target, trailingSlash, transpilePackages, typescript, useFileSystemPublicRoutes, webpack).
[0] warn  - See more info here: https://nextjs.org/docs/messages/invalid-next-config
[3] - warn Invalid next.config.js options detected:
[3] - warn     The root value has an unexpected property, strictMode, which is not in the list of allowed properties (amp, analyticsId, assetPrefix, basePath, cleanDistDir, compiler, compress, configOrigin, crossOrigin, devIndicators, distDir, env, eslint, excludeDefaultMomentLocales, experimental, exportPathMap, generateBuildId, generateEtags, headers, httpAgentOptions, i18n, images, modularizeImports, onDemandEntries, optimizeFonts, output, outputFileTracing, pageExtensions, poweredByHeader, productionBrowserSourceMaps, publicRuntimeConfig, reactStrictMode, redirects, rewrites, sassOptions, serverRuntimeConfig, skipMiddlewareUrlNormalize, skipTrailingSlashRedirect, staticPageGenerationTimeout, swcMinify, target, trailingSlash, transpilePackages, typescript, useFileSystemPublicRoutes, webpack).
[3] - warn See more info here: https://nextjs.org/docs/messages/invalid-next-config
[1]
[1] ><EMAIL_ADDRESS>launch
[1] > npm run package:build && node ./index.js;
[1]
[1]
[1] ><EMAIL_ADDRESS>package:build
[1] > tsc --project tsconfig.json
[1]
[3] - event compiled client and server successfully in 223 ms (20 modules)
[3] - wait compiling...
[3] - event compiled client and server successfully in 163 ms (20 modules)
[0] warn  - The `app` directory is experimental. To enable, add `appDir: true` to your `next.config.js` configuration under `experimental`. See https://nextjs.org/docs/messages/experimental-app-dir-config
[0] error - ./pages/_document.jsx:3:0
[0] Module not found: Can't resolve '@deep-foundation/deepcase/imports/theme/theme'
[0]   1 | import Document, { Html, Head, Main, NextScript } from 'next/document';
[0]   2 | import { ColorModeScript } from '@chakra-ui/react';
[0] > 3 | import theme from '@deep-foundation/deepcase/imports/theme/theme';
[0]   4 |
[0]   5 | class MyDocument extends Document {
[0]   6 |   static async getInitialProps(ctx) {
[0]
[0] https://nextjs.org/docs/messages/module-not-found
[0] warn  - The `app` directory is experimental. To enable, add `appDir: true` to your `next.config.js` configuration under `experimental`. See https://nextjs.org/docs/messages/experimental-app-dir-config
[1] automation/remove-build-files.ts(1,1): error TS1208: 'remove-build-files.ts' cannot be compiled under '--isolatedModules' because it is considered a global script file. Add an import, export, or an empty 'export {}' statement to make it a module.
[1] npm run deeplinks exited with code 2
[0] wait  - compiling...
[0] event - compiled successfully in 200 ms (39 modules)
[0] wait  - compiling / (client and server)...
[4] imports/client-handler.tsx(8,29): error TS2307: Cannot find module 'axios-hooks' or its corresponding type declarations.
[4] imports/package-publisher.tsx(6,30): error TS2307: Cannot find module 'axios-hooks' or its corresponding type declarations.
[4] imports/packager-interface/packager-interface-tabs-content.tsx(13,30): error TS2307: Cannot find module 'axios-hooks' or its corresponding type declarations.
[4] imports/packager-interface/packager-interface.tsx(13,30): error TS2307: Cannot find module 'axios-hooks' or its corresponding type declarations.
[4] imports/test.ts(1,1): error TS1208: 'test.ts' cannot be compiled under '--isolatedModules' because it is considered a global script file. Add an import, export, or an empty 'export {}' statement to make it a module.
[4]
[4] 3:06:18 PM - Found 5 errors. Watching for file changes.
[0] wait  - compiling /_error (client and server)...
[0] warn  - The `app` directory is experimental. To enable, add `appDir: true` to your `next.config.js` configuration under `experimental`. See https://nextjs.org/docs/messages/experimental-app-dir-config
[0] error - ../deepcase/imports/client-handler.js:29:0
[0] Module not found: Can't resolve 'axios-hooks'
[0]   27 | import { useMinilinksFilter } from "@deep-foundation/deeplinks/imports/minilinks";
[0]   28 | import axios from 'axios';
[0] > 29 | import * as axiosHooks from 'axios-hooks';
[0]   30 | import * as classnames from 'classnames';
[0]   31 | import React, { useEffect, useRef, useState } from 'react';
[0]   32 | import * as motion from 'framer-motion';
[0]
[0] https://nextjs.org/docs/messages/module-not-found
[0]
[0] Import trace for requested module:
[0] ../deepcase/imports/cyto/editor.js
[0] ./pages/index.tsx
[0] warn  - Fast Refresh had to perform a full reload due to a runtime error.
[0] warn  - The `app` directory is experimental. To enable, add `appDir: true` to your `next.config.js` configuration under `experimental`. See https://nextjs.org/docs/messages/experimental-app-dir-config
[0] warn  - The `app` directory is experimental. To enable, add `appDir: true` to your `next.config.js` configuration under `experimental`. See https://nextjs.org/docs/messages/experimental-app-dir-config

Looks like npm install or npm clean-install was not executed in deepcase:
freephoenix888@FreePhoenix888:~/Programming/dev/packages/deepcase$ npm i axios-hooks

up to date, audited 3245 packages in 8s

226 packages are looking for funding
  run `npm fund` for details

54 vulnerabilities (23 moderate, 31 high)

To address issues that do not require attention, run:
  npm audit fix

To address all issues possible (including breaking changes), run:
  npm audit fix --force

Some issues need review, and may require choosing
a different dependency.

Run `npm audit` for details.
freephoenix888@FreePhoenix888:~/Programming/dev/packages/deepcase$ npm ls axios-hooks
<EMAIL_ADDRESS>/home/freephoenix888/Programming/dev
└─┬<EMAIL_ADDRESS>-> ./packages/deepcase
  └──<EMAIL_ADDRESS>
freephoenix888@FreePhoenix888:~/Programming/dev/packages/deepcase$ git status
On branch main
Your branch is up to date with 'origin/main'.

nothing to commit, working tree clean

freephoenix888@FreePhoenix888:~/Programming/dev/packages/deepcase$ grep "axios-hooks" ./package.json
    "axios-hooks": "^4.0.0",

@Konard , why do we run npm run packages but not npm run packages:ci? We have such gulp script:
https://github.com/deep-foundation/dev/blob/bafba15b0744dc046ab72a96bd581c83daa1eb65/gulpfile.ts#L119

@Konard , why do we run npm run packages but not npm run packages:ci?

We do run it:
https://github.com/deep-foundation/dev/blob/83b8f56d61e55f1bfd620ed05c41477962e860be/package.json#L17

@Konard , why do we run npm run packages but not npm run packages:ci?

We do run it:
https://github.com/deep-foundation/dev/blob/83b8f56d61e55f1bfd620ed05c41477962e860be/package.json#L17

🤔 Why were not deepcase dependencies installed then? What do you think?

🤔 Why were not deepcase dependencies installed then? What do you think?

Because it is npm workspace now. And all its dependencies are installed using npm ci in dev repository.
https://github.com/deep-foundation/dev/blob/83b8f56d61e55f1bfd620ed05c41477962e860be/package.json#L115-L119
We use workspaces to allow simplification of development workflow. Now we can just make changes in deeplinks, deepcase and deepcase-app repositories and all of them immediately synced with Deep.Case App opened on 3007 port.

🤔 Why were not deepcase dependencies installed then? What do you think?

Because it is npm workspace now. And all its dependencies are installed using npm ci in dev repository.
https://github.com/deep-foundation/dev/blob/83b8f56d61e55f1bfd620ed05c41477962e860be/package.json#L115-L119
We use workspaces to allow simplification of development workflow. Now we can just make changes in deeplinks, deepcase and deepcase-app repositories and all of them immediately synced with Deep.Case App opened on 3007 port.

Why does this error happen then?

Why does this error happen then?

Because than you make npm ci or npm i in deepcase repository, you break symbolic link with dev/node_modules.
We now use workspaces, so for these repositories:
https://github.com/deep-foundation/dev/blob/83b8f56d61e55f1bfd620ed05c41477962e860be/package.json#L115-L119
We may no longer npm ci or npm i do as before.

Why does this error happen then?

Because than you make npm ci or npm i in deepcase repository, you break symbolic link with dev/node_modules.
We now use workspaces, so for these repositories:
https://github.com/deep-foundation/dev/blob/83b8f56d61e55f1bfd620ed05c41477962e860be/package.json#L115-L119
We may no longer npm ci or npm i do as before.

This error was caused when I have used this code:
git clone https://github.com/deep-foundation/dev.git
cd dev
npm ci
rm -rf packages/deepcase
rm -rf packages/deepcase-app
rm -rf packages/deeplinks
npm run packages

npm run local

I have not used npm i/npm ci until I tried to fix that error


More Actors Stuff

Description
Removed need for ActorService.
Implemented AbstractActor.
Implemented ActorRuntime.
Created concept of ActorRuntimeContext.
Issue reference
We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.
Please reference the issue this PR will close: #44
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[X] Code compiles correctly
[X] Created/updated tests
[ ] Extended the documentation

There are many changes piling up in this PR since the reviewers are on vacation. Since we are using a temporary branch for this work, I will merge this change to unblock others that are also working on the Java SDK. I am happy to do a major review before we merge to master.


Image selector implementations and integration with GCE backend

This is currently working in staging: https://staging.travis-ci.org/meatballhat/yolo-octo-adventure/builds/435428


PathHelper.GetSecretsPathFromSecretsId() will resolve to C:\WINDOWS\system32\config\systemprofile\AppData\Roaming\Microsoft\UserSecrets when debugging under IIS

Following the article here:
https://docs.microsoft.com/en-us/aspnet/core/security/app-secrets?tabs=visual-studio
The secrets.json file DOES exist in %APPDATA%... for my Windows user account. However, when I debug in Visual Studion under IIS (using Admin account) the %APPDATA% path resolves to C:\WINDOWS\system32\config\systemprofile\AppData\Roaming which does not have secrets.json
I've verified this with
Environment.GetEnvironmentVariable("APPDATA")
and
PathHelper.GetSecretsPathFromSecretsId("...")

P.S. Using ASP.NET Core 2.0.1 with .NET 4.7.1 on Win10 x64 (RuntimeIdentified is set to win10-x64)
Using Visual Studio 2017 Pro, version 15.5.5
Using this profile to debug:
"profiles": {     "IIS": {       "commandName": "IIS",       "launchBrowser": true,       "launchUrl": "http://localhost/DTI.Gentium.Services.CreditApp/",       "environmentVariables": {         "ASPNETCORE_ENVIRONMENT": "Development"       }     }   }

@natemcmaster Any thoughts on this one?

Duplicate of https://github.com/aspnet/DotNetTools/issues/172. At the time we backlogged it, running aspnetcore under IIS was less common as VS only supported IIS Express. Since then, VS has added support for launching aspnetcore under full IIS. Secret storage is specific to the Windows user account. When full IIS (not IIS Express) launches aspnetcore, it runs under a different user account, so user secrets will not be accessible. To fix this, we would need to provide some way for machine-global secrets, or secrets shared between users, which complicates the user-secrets implementation. So far, we have not been able to justify this work, especially when so many other configuration providers are available and are easier to work with.

Thanks Nate.


Fix Spark 4.0 sql tests

Describe the bug
Regarding https://github.com/apache/datafusion-comet/pull/537, there are 105 Spark 4.0 sql tests failing.

sql1 93 tests failing
sql2 12 tests failing

Fix comet shims for the Spark 4.0 profile and remove IgnoreComet for those tests. Some of the tests may share same root causes
sql-1

[ ] SPARK-34212 Parquet should read decimals correctly
[ ] SPARK-43402: FileSourceScanExec supports push down data filter with scalar subquery
[ ] [SPARK-43226] extra constant metadata fields with extractors
[ ] SPARK-34212 Parquet should read decimals correctly
[ ] parquet widening conversion ShortType -> IntegerType
[ ] parquet widening conversion IntegerType -> ShortType
[ ] parquet widening conversion IntegerType -> LongType
[ ] parquet widening conversion ShortType -> DoubleType
[ ] parquet widening conversion IntegerType -> DoubleType
[ ] parquet widening conversion DateType -> TimestampNTZType
[ ] parquet widening conversion ByteType -> DecimalType(10,0)
[ ] parquet widening conversion ByteType -> DecimalType(20,0)
[ ] parquet widening conversion ShortType -> DecimalType(10,0)
[ ] parquet widening conversion ShortType -> DecimalType(20,0)
[ ] parquet widening conversion ShortType -> DecimalType(38,0)
[ ] parquet widening conversion IntegerType -> DecimalType(10,0)
[ ] parquet widening conversion IntegerType -> DecimalType(20,0)
[ ] parquet widening conversion IntegerType -> DecimalType(38,0)
[ ] parquet widening conversion LongType -> DecimalType(20,0)
[ ] parquet widening conversion LongType -> DecimalType(38,0)
[ ] parquet widening conversion ByteType -> DecimalType(11,1)
[ ] parquet widening conversion ShortType -> DecimalType(11,1)
[ ] parquet widening conversion IntegerType -> DecimalType(11,1)
[ ] parquet widening conversion LongType -> DecimalType(21,1)
[ ] unsupported parquet conversion ByteType -> DecimalType(1,0)
[ ] unsupported parquet conversion ByteType -> DecimalType(3,0)
[ ] unsupported parquet conversion ShortType -> DecimalType(3,0)
[ ] unsupported parquet conversion ShortType -> DecimalType(5,0)
[ ] unsupported parquet conversion IntegerType -> DecimalType(5,0)
[ ] unsupported parquet conversion ByteType -> DecimalType(4,1)
[ ] unsupported parquet conversion ShortType -> DecimalType(6,1)
[ ] unsupported parquet conversion LongType -> DecimalType(10,0)
[ ] unsupported parquet conversion ByteType -> DecimalType(2,0)
[ ] unsupported parquet conversion ShortType -> DecimalType(4,0)
[ ] unsupported parquet conversion IntegerType -> DecimalType(9,0)
[ ] unsupported parquet conversion LongType -> DecimalType(19,0)
[ ] unsupported parquet conversion ByteType -> DecimalType(3,1)
[ ] unsupported parquet conversion ShortType -> DecimalType(5,1)
[ ] unsupported parquet conversion IntegerType -> DecimalType(10,1)
[ ] unsupported parquet conversion LongType -> DecimalType(20,1)
[ ] unsupported parquet timestamp conversion TimestampType (TIMESTAMP_MICROS) -> DateType
[ ] unsupported parquet timestamp conversion TimestampType (TIMESTAMP_MILLIS) -> DateType
[ ] unsupported parquet timestamp conversion TimestampNTZType (INT96) -> DateType
[ ] unsupported parquet timestamp conversion TimestampNTZType (TIMESTAMP_MICROS) -> DateType
[ ] unsupported parquet timestamp conversion TimestampNTZType (TIMESTAMP_MILLIS) -> DateType
[ ] parquet decimal precision change Decimal(5, 2) -> Decimal(7, 2)
[ ] parquet decimal precision change Decimal(5, 2) -> Decimal(10, 2)
[ ] parquet decimal precision change Decimal(5, 2) -> Decimal(20, 2)
[ ] parquet decimal precision change Decimal(10, 2) -> Decimal(12, 2)
[ ] parquet decimal precision change Decimal(10, 2) -> Decimal(20, 2)
[ ] parquet decimal precision change Decimal(20, 2) -> Decimal(22, 2)
[ ] parquet decimal precision change Decimal(7, 2) -> Decimal(5, 2)
[ ] parquet decimal precision change Decimal(10, 2) -> Decimal(5, 2)
[ ] parquet decimal precision change Decimal(20, 2) -> Decimal(5, 2)
[ ] parquet decimal precision change Decimal(12, 2) -> Decimal(10, 2)
[ ] parquet decimal precision change Decimal(20, 2) -> Decimal(10, 2)
[ ] parquet decimal precision change Decimal(22, 2) -> Decimal(20, 2)
[ ] parquet decimal precision and scale change Decimal(5, 2) -> Decimal(7, 4)
[ ] parquet decimal precision and scale change Decimal(5, 2) -> Decimal(10, 7)
[ ] parquet decimal precision and scale change Decimal(5, 2) -> Decimal(20, 17)
[ ] parquet decimal precision and scale change Decimal(10, 2) -> Decimal(12, 4)
[ ] parquet decimal precision and scale change Decimal(10, 2) -> Decimal(20, 12)
[ ] parquet decimal precision and scale change Decimal(20, 2) -> Decimal(22, 4)
[ ] parquet decimal precision and scale change Decimal(7, 4) -> Decimal(5, 2)
[ ] parquet decimal precision and scale change Decimal(10, 7) -> Decimal(5, 2)
[ ] parquet decimal precision and scale change Decimal(20, 17) -> Decimal(5, 2)
[ ] parquet decimal precision and scale change Decimal(12, 4) -> Decimal(10, 2)
[ ] parquet decimal precision and scale change Decimal(20, 17) -> Decimal(10, 2)
[ ] parquet decimal precision and scale change Decimal(22, 4) -> Decimal(20, 2)
[ ] parquet decimal precision and scale change Decimal(10, 6) -> Decimal(12, 4)
[ ] parquet decimal precision and scale change Decimal(20, 7) -> Decimal(22, 5)
[ ] parquet decimal precision and scale change Decimal(12, 4) -> Decimal(10, 6)
[ ] parquet decimal precision and scale change Decimal(22, 5) -> Decimal(20, 7)
[ ] parquet decimal precision and scale change Decimal(5, 2) -> Decimal(6, 4)
[ ] parquet decimal precision and scale change Decimal(10, 4) -> Decimal(12, 7)
[ ] parquet decimal precision and scale change Decimal(20, 5) -> Decimal(22, 8)
[ ] parquet decimal type change Decimal(5, 2) -> Decimal(3, 2) overflows with parquet-mr
[ ] partition pruning in broadcast hash joins with aliases
[ ] partition pruning in broadcast hash joins
[ ] SPARK-32817: DPP throws error when the broadcast side is empty
[ ] SPARK-36444: Remove OptimizeSubqueries from batch of PartitionPruning
[ ] SPARK-38674: Remove useless deduplicate in SubqueryBroadcastExec
[ ] SPARK-39338: Remove dynamic pruning subquery if pruningKey's references is empty
[ ] SPARK-39217: Makes DPP support the pruning side has Union
[ ] partition pruning in broadcast hash joins with aliases
[ ] partition pruning in broadcast hash joins
[ ] different broadcast subqueries with identical children
[ ] SPARK-32817: DPP throws error when the broadcast side is empty
[ ] SPARK-36444: Remove OptimizeSubqueries from batch of PartitionPruning
[ ] SPARK-38674: Remove useless deduplicate in SubqueryBroadcastExec
[ ] SPARK-39338: Remove dynamic pruning subquery if pruningKey's references is empty
[ ] SPARK-39217: Makes DPP support the pruning side has Union
[ ] join with ordering requirement

sql-2

[ ] collations.sql
[ ] SPARK-39166: Query context of binary arithmetic should be serialized to executors when WSCG is off
[ ] SPARK-39175: Query context of Cast should be serialized to executors when WSCG is off
[ ] SPARK-39190,SPARK-39208,SPARK-39210: Query context of decimal overflow error should be serialized to executors when WSCG is off
[ ] SPARK-40389: Don't eliminate a cast which can cause overflow
[ ] postgreSQL/float8.sql
[ ] postgreSQL/groupingsets.sql
[ ] postgreSQL/int4.sql
[ ] SPARK-47120: subquery literal filter pushdown
[ ] SPARK-47120: subquery literal filter pushdown
[ ] view-schema-binding-config.sql
[ ] view-schema-compensation.sql

Steps to reproduce
No response
Expected behavior
No response
Additional context
No response

Search https://github.com/apache/datafusion-comet/issues/551 in dev/diffs/4.0.0-preview1.diff to find ignored tests

Hmm, this failed tests are additional to Spark 3.4?

Hmm, this failed tests are additional to Spark 3.4? I.e., they are passed in Spark 3.4 + Comet but fail in Spark 4.0?

Could be both. Could be due to ANSI. This ticket is for getting help from the community after https://github.com/apache/datafusion-comet/pull/537 is merged

@parthchandra is looking into

SPARK-43402: FileSourceScanExec supports push down data filter with scalar subquery
[SPARK-43226] extra constant metadata fields with extractors


work out kinks in list_quantum_computers

This PR cleans up the different get_devices-type methods, ultimately encouraging the user to pass through list_quantum_computers.
An example of good news:
In : get_qc(list_quantum_computers()[0], as_qvm=True)
Out: <pyquil.api._quantum_computer.QuantumComputer at 0x10bec0eb8>

And an example of bad news:
In : from pyquil.api import list_quantum_computers, get_qc
In : list_quantum_computers()
ValueError:
        list_lattices encountered an error when querying the Forest 2.0 endpoint.

        Very likely this means that you do not have valid user authentication information, and very
        likely this is because you haven't yet been invited to try QCS.  We plan on making our
        device information publicly accessible soon, but in the meanwhile, you'll have to use
        default QVM configurations and to use `list_quantum_computers` with `qpus = False`.

In list_quantum_computers above the changes (so I can't do line comments) there's some old code marked with a #TODO that can probably be removed

That #TODO also pointed to some other things I'd forgotten to link up.

fixes #586

can you merge in master to fix the ci after #594


Remove Lead Source Picklist Values

Critical Changes
Remove Lead Source from Picklist values
Changes
Issues Closed

@mkolodner made the change real quick.  Running a test via metaci to see if things work now: https://sfdo-community-ci.herokuapp.com/builds/2948

I was also working on a test with what I expect is the same change.

Outside of deleting the lead source lines, I also added Mx as a Saluation option as that wasn't included before.

It looks like my test worked. But since mine was not from an entirely clean org (I re-ran dev_org on the demo org that had the error), it's probably best to ensure that yours works from scratch.

Mine was a success too so I think we're good to go here.  I'll assign you
as the reviewer since you did the test via the demo org.
Nickers<EMAIL_ADDRESS>On Fri, Jul 17, 2020 at 3:04 PM Michael Kolodner<EMAIL_ADDRESS>wrote:

It looks like my test worked. But since mine was not from an entirely
clean org (I re-ran dev_org on the demo org that had the error), it's
probably best to ensure that yours works from scratch.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/SFDO-Community/OmbudsmanCloudCare/pull/83#issuecomment-660313253,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACUVOYR5DNXYTJOSFSZ6TY3R4CVETANCNFSM4O6ZIL6Q
.

Looks good to me. But I don't seem to have the ability take any action. I can click "Close and comment," which seems like it would close the issue. But I've got a message that merging is blocked because a test hasn't been completed yet. However, I don't see how to do the "Feature Test" check.

@jlantz we are working through our first branch and noticed the feature test didn't get a result back.  Is this something you'd be able to look into this week?

Looks like I can trigger a feature test via MetaCI so I will run that here.

@mkolodner I was able to manually get this branch to pass the feature test too via metaci.  Mind confirming the contact record types work as expected on this branch before we merge.


Specify maximum width in pixels for tabs

Hi,
I have added a new property to customize the maximum possible width of a tab.
maxTabWidth for Tab component which specifies maximum width of a tab in pixels.
I needed this feature so forked and added.  Might be useful for others too.

Same to #2 :-)


build error

Is this a BUG REPORT or FEATURE REQUEST? (choose one):
BUG REPORT

I've got errors as below during building nauta on kubernetes pod system as build system.
---------------------------------------------------------------------------------------------
fatal: [local]: FAILED! => {"ansible_job_id": "452303190776.86355", "attempts": 5, "changed": false, "finished": 1, "msg": "Error building nauta/rpm/python - code: 127, message: The command '/bin/sh -c pip install -U pip==19.0.3 virtualenv==16.0.0 setuptools==39.2.0 wheel==0.31.1' returned a non-zero code: 127, logs: ['Step 1/12 : ARG BASE_IMAGE=shared/centos/rpm-packer', '\n', 'Step 2/12 : ARG PYTHON2_PIP_RPM_IMAGE=shared/build/rpm/python2-pip', '\n', 'Step 3/12 : FROM ${PYTHON2_PIP_RPM_IMAGE} as python2_pip_rpm_image', '\n', ' ---> 03175a7225d0\n', 'Step 4/12 : FROM ${BASE_IMAGE}', '\n', ' ---> 0cf01b1e3621\n', 'Step 5/12 : ENV RPM_VERSION=2.7', '\n', ' ---> Using cache\n', ' ---> 1a58dc8d7507\n', 'Step 6/12 : ENV RPM_RELEASE=0', '\n', ' ---> Using cache\n', ' ---> 54fe0479563b\n', 'Step 7/12 : RUN yum update -y && yum install -y python-devel python libffi-devel openssl-devel gcc gcc-c++', '\n', ' ---> Using cache\n', ' ---> d08159f4c7fb\n', 'Step 8/12 : RUN curl "https://bootstrap.pypa.io/get-pip.py\" | python', '\n', ' ---> Using cache\n', ' ---> 753d99098bff\n', 'Step 9/12 : RUN pip install -U pip==19.0.3 virtualenv==16.0.0 setuptools==39.2.0 wheel==0.31.1', '\n', ' ---> Running in 7ca17da74ecf\n', '\x1b[91m/bin/sh: pip: command not found\n\x1b[0m', 'Removing intermediate container 7ca17da74ecf\n']"}
---------------------------------------------------------------------------------------------
This message say that pip command is not found, but pip command is exist.
Cluster configuration details:
I tried on my own system.

Cloud provider or hardware configuration:
Operating system: (printf "$(uname -srm)\n$(cat /etc/os-release)\n"):
Linux 4.4.214-1.el7.elrepo.x86_64 x86_64
NAME="Ubuntu"
VERSION="16.04.6 LTS (Xenial Xerus)"
ID=ubuntu
ID_LIKE=debian
PRETTY_NAME="Ubuntu 16.04.6 LTS"
VERSION_ID="16.04"
HOME_URL="http://www.ubuntu.com/"
SUPPORT_URL="http://help.ubuntu.com/"
BUG_REPORT_URL="http://bugs.launchpad.net/ubuntu/"
VERSION_CODENAME=xenial

This system info about Docker container, actually k8s on nauta.

Nauta version and commit: (nctl version)(git rev-parse --short HEAD):
git cloned today(2020/03/10)

Nauta component related with bug: (build system/installer/nctl(cli)/dashboard/documentation/k8s/any of nauta container)
make k8s-installer-build

What is the current behavior?
What is the expected behavior?
Steps to reproduce:
*
*
Anything else do we need to know:

Is any problem building on container(actually nauta)

Hi, it looks like cached docker layers are not working correctly for some reason (I can build this image without problems when docker cache is not used) - https://github.com/IntelAI/nauta/pull/43 PR should mitigate this issue.

Thank you for your reply.
Would you tell me what I should do to debug this issue?

Best way would be to simply update nauta's code to latest version (PR that I've mentioned above is already merged) and run the build again, nauta/rpm/python should work now

Hi, it doesn't work.
I updated source and run build, but I've got message as like,
Wednesday 11 March 2020  05:38:05 +0000 (0:00:00.113)       0:02:28.448 *******
fatal: [local]: FAILED! => {"ansible_job_id": "216598653249.52805", "attempts": 193, "changed": false, "finished": 1, "msg": "Error building nauta/rpm/python - code: 127, message: The command '/bin/sh -c curl "https://bootstrap.pypa.io/get-pip.py\" | python && pip install -U pip==19.0.3 virtualenv==16.0.0 setuptools==39.2.0 wheel==0.31.1' returned a non-zero code: 127, logs: ['Step 1/11 : ARG BASE_IMAGE=shared/centos/rpm-packer', '\n', 'Step 2/11 : ARG PYTHON2_PIP_RPM_IMAGE=shared/build/rpm/python2-pip', '\n', 'Step 3/11 : FROM ${PYTHON2_PIP_RPM_IMAGE} as python2_pip_rpm_image', '\n', ' ---> 3661c933cfd2\n', 'Step 4/11 : FROM ${BASE_IMAGE}', '\n', ' ---> 417d11842739\n', 'Step 5/11 : ENV RPM_VERSION=2.7', '\n', ' ---> Using cache\n', ' ---> 6aa5a75ced5f\n', 'Step 6/11 : ENV RPM_RELEASE=0', '\n', ' ---> Using cache\n', ' ---> c0a3c1d3389a\n', 'Step 7/11 : RUN yum update -y && yum install -y python-devel python libffi-devel openssl-devel gcc gcc-c++', '\n', ' ---> Using cache\n', ' ---> e5e646062f42\n', 'Step 8/11 : RUN curl "https://bootstrap.pypa.io/get-pip.py\" | python && pip install -U pip==19.0.3 virtualenv==16.0.0 setuptools==39.2.0 wheel==0.31.1', '\n', ' ---> Running in 691e706bd5e1\n', '\x1b[91m  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current\n       \x1b[0m', '\x1b[91m                          Dload  Uploa\x1b[0m', '\x1b[91md   Total\x1b[0m', '\x1b[91m   Spent    Left  Speed\n\r  0     0   \x1b[0m', '\x1b[91m 0     0    0\x1b[0m', '\x1b[91m     0      0      0 -\x1b[0m', '\x1b[91m-:--:-- --:--:-\x1b[0m', '\x1b[91m- --:--:\x1b[0m', '\x1b[91m--     0\x1b[0m', '\x1b[91m\r
.........
.........
.........
0\x1b[0m', '\x1b[91m\r  0     0    0     0    0     0      0      0 --:\x1b[0m', '\x1b[91m--:--  0:05:00 -\x1b[0m', '\x1b[91m-:--:--  \x1b[0m', '\x1b[91m   0\n\x1b[0m', '\x1b[91mcurl: (28) Operation t\x1b[0m', '\x1b[91mimed out after 30021\x1b[0m', '\x1b[91m3 milliseconds wi\x1b[0m', '\x1b[91mth \x1b[0m', '\x1b[91m0 out of\x1b[0m', '\x1b[91m 0 \x1b[0m', '\x1b[91mbyt\x1b[0m', '\x1b[91mes recei\x1b[0m', '\x1b[91mve\x1b[0m', '\x1b[91md\n\x1b[0m', '\x1b[91m/bin/sh: pip: command not found\n\x1b[0m', 'Removing intermediate container 691e706bd5e1\n']"}

I built nauta on bare-metal successfully, but I failed on Kubernetes pod,
error message is below, it reads that it could not download nginx from mirror site.
But I downloaded it easy by using command "wget http://nginx.org/packages/mainline/centos/7/x86_64/RPMS/nginx-1.13.9-1.el7_4.ngx.x86_64.rpm"
What can I do? or Is it impossible to build on kubernetes pod?
error message partial ---
Trying other mirror.\n
", "http://nginx.org/packages/mainline/centos/7/x86_64/RPMS/nginx-1.13.9-1.el7_4.ngx.x86_64.rpm: [Errno 12] Timeout on http://nginx.org/packages/mainline/centos/7/x86_64/RPMS/nginx-1.13.9-1.el7_4.ngx.x86_64.rpm: (28, 'Operation too slow. Less than 1000 bytes/sec transferred the last 30 seconds')\n
", 'Trying other mirror.\n
', '1:nginx-1.13.9-1.el7_4.ngx.x86_64: [Errno 256] No more mirrors to try.\n
', 'Removing intermediate container 12b10818fba0\n']"
Thank you,

Could you tell me what particular image build is failing? If you mean that you want to build Nauta on some Kubernetes pod (like running make k8s-installer-build inside of Kubernetes container), be aware that it may not work, we are not supporting such case. Although, error message suggests that it is a network related error, so I'd recommend checking if proxy settings are configured correctly (see: https://intelai.github.io/nauta/installation-and-configuration/How_to_Build_Nauta/HBN.html#proxy-setting-requirements).

Thank you for your reply,
I tried to build on kubernetes pod made on Nauta cluster.
I intended to test most of my jobs on Nauta nodes.
If the Nauta build is not supposed to be run on container, I will not try any more.
That's not problem. I just tested.
I will try to test others including deep learning training.
I really appreciate your reply and help. It was very helpful.
I am testing more, and asking more.
Thank you very much.


Azure.Monitor.Ingestion: Empty Check prevents streaming enumerables like BlockingCollection GetConsumingEnumerator()

https://github.com/Azure/azure-sdk-for-net/blob/767dcb4fa7f5a2b8410352c1279c19097f1d5d1f/sdk/monitor/Azure.Monitor.Ingestion/src/LogsIngestionClient.cs#L335
This check prevents the use of GetConsumingEnumerator in BlockingCollection to feed this client in a producer/consumer pattern, since the MoveNext in it will consume an item, and that item will not get logged. Using a normal enumerable is not ideal from a memory usage perspective.
Instead you should just do a noop in the case of an empty enumerable, maybe log that aspect. An alternative would be to check the type of the collection and if it is a ConsumingEnumerator then don't do this assertion check.
Workaround: Add a "dummy" object to be consumed, but that relies on implementation detail, and if this is ever fixed, then the first log will start silently not being logged until the program is corrected, so this workaround is not ideal.
CC @nisha-bhatia

Thank you for your feedback.  Tagging and routing to the team member best able to assist.


Building API Docs on RTD Fails For Unit Tests

I am not sure that I can build these docs on RTD when I am autodoc-ing the tests unless I properly install them. I don't think I want to A) make the test suite it's own package with it's own setup.py or B) make the tests an installed part of Selector. If there is some way to install the tests selectively, or some other way to get RTD to build them, that would be cool.
I am going to back off of this, for now, and keep the issue open. I want to treat the tests as first class code and document them in this way, but, until I figure out how, I am going to be pragmatic and at least build the selector module's public API docs in the mean time.
...OTOH, maybe should the API docs just be limited to the Public API that which I want to consider Selector's public API, WRT SemVer. I may potentially just choose to close this and leave it that way...

I've decided to close this. Basic docs are building. I don't think tests were appropriate here anyway. Neither was fabfile and I removed that too. I think this should just be the *Public API of Selector itself.


Title Change for Pulumi Query for Kubernetes

Proposed changes

Unreleased product version (optional)

Related issues (optional)

Note that this feedback still isn't incorporated: https://github.com/pulumi/docs/pull/2024#discussion_r348747784.


[Subscription Request] - Table 36 "Sales Header" - OnBeforeSetWorkDescription

Describe the request
Dear Support,
Could You please add a new subscription for the beginning of the SetWorkDescription procedure?
<<<<< Original >>>>>
    procedure SetWorkDescription(NewWorkDescription: Text)
    var
        OutStream: OutStream;
    begin
        Clear("Work Description");
        "Work Description".CreateOutStream(OutStream, TEXTENCODING::UTF8);
        OutStream.WriteText(NewWorkDescription);
        Modify();
    end;

<<<<< Modified >>>>>
    procedure SetWorkDescription(NewWorkDescription: Text)
    var
        OutStream: OutStream;
    begin
        OnBeforeSetWorkDescription(Rec,NewWorkDescription);
        Clear("Work Description");
        "Work Description".CreateOutStream(OutStream, TEXTENCODING::UTF8);
        OutStream.WriteText(NewWorkDescription);
        Modify();
    end;

    [IntegrationEvent(false, false)]
    local procedure OnBeforeSetWorkDescription(var SalesHeader: Record "Sales Header"; var NewWorkDescription: Text)
    begin
    end;

Thanks for in advance!
BR,
Laszlo
Additional context
With these modifications we are able to modify and read the content of the work description avoiding the multiple database operations.
Internal work item: AB#558508

OK

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 27498.


What is the best way to grant CORS for all subdomains dynamically?

I need to provide CORS permissions for many subdomains (i do not know all), that are located on known list of domains. E.g. *.example.com, *.corp-example.com, etc.
To allow CORS for any origin, i can do the following:
import aiohttp_cors
from aiohttp.web_app import Application

app = Application()
aiohttp_cors.setup(app, defaults={
    "*": aiohttp_cors.ResourceOptions(
        allow_credentials=True,
        expose_headers="*",
        allow_headers="*",
    )
})

for handler in handlers:
    # Register handler
    route = app.router.add_route('*', handler.URL_PATH, handler)

    # Handler should be handled with CORS
    app['aiohttp_cors'].add(route)

It works, but is not secure. What is simplest way to check if client origin meets some requirements (e.g. that origin matches ^[a-z0-9_-]+.example.com$) and if does not - to deny request?
I supposed that it is possible to extend some basic method to add such checks, or to provide lambda to aiohttp_cors.setup defaults, that would accept origin as input parameter and return appropriate config.

As i see, for preflight CORS requests, it is enough to extend views from both aiohttp.web.View and aiohttp_cors.CorsViewMixin, and re-define _get_config(self, request, origin, request_method)method.
if origin is not passing custom checks - raise KeyError, otherwise call super()._get_config():
class BaseHandler(aiohttp.web.View, aiohttp_cors.CorsViewMixin):
    async def _get_config(self, request, origin, request_method):
        # Handle pre-flight requests
        if not origin_is_allowed(origin):
            raise KeyError

        return super()._get_config(request, origin, request_method)

I think yes, overriding _get_config() is legitimate here.

@asvetlov and what is the correct way for preflight requests? Is the only way to implement checks - to extend _CorsConfigImpl._on_response_prepare and CorsConfig.init (to replace hard-coded _CorsConfigImpl)?

Hmm, hacking _CorsConfigImpl smells ugly.
Ideally _CorsConfigImpl._on_response_prepare() should be not changed but cors configuration.
Could you share your code to demonstrate why do you do it and what _on_response_prepare() lines are changed?

Thank you for fast response.
My first idea was to do something like that:
class _CustomCorsConfigImpl(_CorsConfigImpl):
    @staticmethod
    def check_origin_allowed(origin: str) -> bool:
        # Do origin checks
        return True

    async def _on_response_prepare(self,
                                   request: web.Request,
                                   response: web.StreamResponse):
        # Terminate CORS if origin is not allowed
        if not self.check_origin_allowed(request.headers.get(hdrs.ORIGIN)):
            return
        return super()._on_response_prepare(request, response)

But later i found that it is much easier to extend ResourcesUrlDispatcherRouterAdapter class.
It can affect both preflight and non-preflight responses behavior:
class CustomResourcesUrlDispatcherRouterAdapter(ResourcesUrlDispatcherRouterAdapter):
    def __init__(self, *args, allowed_origin_patterns: Collection[re.Pattern], **kwargs):
        self.allowed_origin_patterns = allowed_origin_patterns
        super().__init__(*args, **kwargs)

    def check_origin_allowed(self, origin: str) -> bool:
        for allowed_origin_pattern in self.allowed_origin_patterns:
            if allowed_origin_pattern.match(origin):
                return True
        return False

    async def get_preflight_request_config(
            self,
            preflight_request: web.Request,
            origin: str,
            requested_method: str):
        # Do origin checks
        if not self.check_origin_allowed(origin):
            # raising web.HTTPForbidden seems not a very good idea here
            raise KeyError

        return super().get_preflight_request_config(preflight_request, origin,
                                                    requested_method)

    def get_non_preflight_request_config(self, request: web.Request):
        # Do origin checks and return empty config {} if checks are not passed
        if not self.check_origin_allowed(request.headers.get(hdrs.ORIGIN)):
            return {}

        return super().get_non_preflight_request_config(request)

also, it is possible to configure aiohttp_cors with custom ResourcesUrlDispatcherRouterAdapter class easily:
cors_defaults = {
    "*": aiohttp_cors.ResourceOptions(
        allow_credentials=True,
        expose_headers="*",
        allow_headers="*",
    ),
}
app['aiohttp_cors'] = CorsConfig(
    app,
    defaults=cors_defaults,
    router_adapter=CustomResourcesUrlDispatcherRouterAdapter(
        app.router, cors_defaults,
        allowed_origin_patterns=[re.compile('[a-z0-9_-]+.example.com$')],
    )
)

That seem be not so ugly and does not use internal api.

Extending adaptor looks better.
Sorry, the library is underpowered.
I know there are several bug reports that are not fixed yet.
The library initial author is retired, I have a limited spare time that I spend mostly on asyncio and aiohttp.
If somebody wants to maintain the project -- you are welcome!


Added required Microsoft header

Yes, we should do this for all our code. @vtbassmatt and I looked it up. https://docs.opensource.microsoft.com/content/releasing/copyright-headers.html


Error when restarting a configured database on Oracle Database 18.4.0-xe

I built the docker image for the 18.4.0-xe version of Oracle Database, I started it up and it configured a working database. When I tried to stop and restart the container, I got this error message:

[FATAL] [DBT-06607] The data file (/opt/oracle/oradata/XE/system01.dbf) already exists on the file system.
CAUSE: This configuration    is going to create the data file location (/opt/oracle/oradata/XE/system01.dbf). But it is detected that there is an existing database using this data file location or the data files from a previous configuration may be left behind.
ACTION: Clean up the existing data files, or provide a different db_unique_name.

After a lot of trial and error/re-reading the documentation, I found out the mistake. I was to blame for this anomaly, but I feel like there could be some improvement made to the database configuration script as well.
The problem was that I was reading the documentation on how to run a container based on the Standard Edition, which means that I set the ORACLE_PWD and ORACLE_SID environment variables as well, which confused the configuration script, creating folders/files with names based on those variables, when it should just ignore them. Since these folders and files are named with the above mentioned variables in mind, but the start script is not, the script will find that there is no database configured, but the configuration script will find that there actually is a database configured, meaning it will fail with error.
Maybe more info can be found here

You installed 18c XE with success in which OS?

Have experienced this issue today as well - Docker image of Oracle XE 18c build with buildContainerImage.sh. Rebuilding the Docker container and using an existing volume containing an already configured Oracle XE database gives me the same error.

@ckristo Can you try Oracle Database 21.3.0 XE image, and see if the problem persists ?
While running this image, please specify -e ORACLE_SID=XE in the docker run command.

@rolaca11 with XE version 21.3.0 onwards, we have common scripts to configure the enterprise/express edition. The only differentiating point is the ORACLE_SID which should be equal to XE. If one choose not to specify ORACLE_SID for the express edition, it will automatically take ORACLE_SID as XE but if wrong value is given to ORACLE_SID the database setup is bound to fail.

Unable to reproduce the same with Oracle 21.3.0 XE container image. Closing this issue for now. Please reopen if the issue exist in 21.3.0 XE container image.


Verify wheels are pure Python

There must be only one wheel and it must be pure Python.
The test will have to be removed before merging as it causes CI to error.
Closes #2

I'll need to delete the second commit, but you can see the error message here: https://github.com/OpenAstronomy/build-python-dist/actions/runs/2110548117


SLF4J: Defaulting to no-operation (NOP) logger implementation
SLF4J: See http://www.slf4j.org/codes.html#StaticLoggerBinder for further details.
OpenJDK 64-Bit Server VM warning: You have loaded library /tmp/BridJExtractedLibraries8853146268352694915/libbridj.so which might have disabled stack guard. The VM will try to fix the stack guard now.
It's highly recommended that you fix the library with 'execstack -c ', or link it with '-z noexecstack'.
Not enough args for null OpenIMAJGrabber.startSession(int, int, double)

These are warnings from SLF4J and BridJ. First one is caused by the fact that you did not add any logger implementation and second one is related to BridJ dyncall implementation. Both are harmless.

How do I fix this? I'm new to libraries and I wanna make my program seamless.

There is nothing to fix since these are not errors, but just an ordinary message.

To get rid of SLF4J: see https://github.com/sarxos/webcam-capture/issues/278 or google solution - there are tons of pages about how to clean this.
To get rid of Not enough args for null OpenIMAJGrabber.startSession you need to use to BridJ 0.6.3 or 0.7-SNAPSHOT
The OpenJDK 64-Bit Server VM warning is also BridJ related. Maybe the above will clean this message as well.


OpenAPIParser allows path params to be non-required

As the spec states, path params are always required:
https://swagger.io/docs/specification/2-0/describing-parameters/
We encountered, that this validation is not performed in OpenAPIParser.
A small example defining a path with two path params - one is required=true and one is required=false. This appears valid when reading with new OpenAPIParser().readLocation(path,auth,options);
openapi: 3.0.0
info:
  description: issue2137
  version: 1.0.1
  title: issue2137
paths:
  '/users/{user_id}/description':
    get:
      operationId: description
      parameters:
        - $ref: '#/components/parameters/UserId'
      responses:
        '200':
          description: OK
          content:
            application/json:
              schema:
                $ref: '#/components/schemas/goraUrl'
  '/gora/{gora_id}/test':
    get:
      operationId: test
      parameters:
        - name: gora_id
          description: issue-2137
          in: path
          required: false
          schema:
            type: string
      responses:
        '200':
          description: OK
          content:
            application/json:
              schema:
                $ref: '#/components/schemas/goraUrl'
components:
  parameters:
    UserId:
      name: user_id
      description: Unique identifier of a user
      in: path
      required: false
      schema:
        type: string
  schemas:
    gora:
      type: object
      description: Information about de product
      properties:
        text:
          type: string
    goraUrl:
      allOf:
        - type: object
          required:
            - url
          properties:
            url:
              type: string
              description: Url with information or picture of the product
        - $ref: '#/components/schemas/gora'

Committed and merged the changes to master. Hence closing this issue.


Added Devs folder and some content

Hey, first of all, sorry but I forgot to open an issue before. Florian send us a mail yesterday where it said:

Die Developer-Dokumentation sollte beschreiben, wie euer Code in etwa aufgebaut ist (mit einem Diagramm o.ä.), also welche Module/Klassen/wichtige Funktionen gibt es etc. Noch dazu sollten hier eure Algorithmen beschrieben werden.

Looking at our readme I feel like we have very extensive user documentation and a short devdoc. So we made it the other way around haha 😆 I thought it was a good idea to make a separate devs folder rather than adding more and more to rhe README.
At the same time, I am not sure how and what we should describe for the architecture and algorithms. I am also not sure if we should keep it as a markdown. Important algorithms from my point of view would be how the ant moves (with and without pheromone influence) and maybe the pheromones but what else? Should the explanation be broad like right now or go into detail?
I left it broad for now to see what you guys say!
If you don't agree with some of the contents or the general idea feel free to change or discuss. The diagram should also be added here.

Good idea to put the developer documentation in its own folder!
Unfortunately, this is not true, but it would still be a good implementation: Obstacle Avoidance: Ants detect and navigate around obstacles encountered during their search.

nice work! looks more structured this way with the seperate devs folder

I deleted the wrong sentence about obstacles now.


Lazarus Involved Atomic Wallet Hack

/qualitycheck

⚠ Checking failed due to 5 errors

@danisztls can you assist me about failure cause?

@JediFaust sure. Looked at log and it is a billing issue.
@marina-chibizova may be able to help.

Hey! @danisztls @JediFaust @marina-chibizova
I know what the problem is because I caused the problem by using our API. We reached our API hard limit due to my usage, and it will be reset on Aug 1, though @evgenydmitriev also contacted OpenAI to raise the limits asap, but no answer yet as I know.

/payout
@jhirschkorn, please adjust the payout amount manually if needed.

Thanks, @JediFaust! 4183 characters were added/changed in this PR. At a rate of 5¢/char your contribution is worth $209.15. @albina-at-inca will process your payment.

@albina-at-inca payout on this one should be 2x

Thanks, @JediFaust! 4183 characters were added/changed in this PR. At a rate of 5¢/char your contribution is worth $209.15. @albina-at-inca will process your payment.

@albina-at-inca payout on this one should be 2x

Noted.


add relation bind

add Wrappter and Builder oprate outside for builder apis

Codecov Report

Merging #11 into master will decrease coverage by 0.42%.
The diff coverage is 58.97%.

@@            Coverage Diff             @@
##           master      #11      +/-   ##
==========================================
- Coverage   82.57%   82.14%   -0.43%
==========================================
  Files          11       11
  Lines         660      689      +29
==========================================
+ Hits          545      566      +21
- Misses         70       77       +7
- Partials       45       46       +1

Impacted Files
Coverage Δ

builder.go
81.94% <100%> (+2.36%)
:arrow_up:

relation.go
76.04% <52.17%> (-2.44%)
:arrow_down:

wrapper.go
81.56% <58.33%> (-1.87%)
:arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 147a894...46c4416. Read the comment docs.


Topic/394 accessible colors and typography

closes #394

@xonic Please update your commit messages to the "subject" doesn't start with "*"

👍

👍

is the #394 still open?


ComponentArrays.jl crashes Julia in VSCode but not in REPL

I'm crossposting from Discourse
The following code crashes Julia in VSCode, but the REPL can run it with no problem.
using ComponentArrays

szE = 2
szG = 2
szA = 9
szK = 5
szT = 44
szB = 2
szZ = 2

costdata = Array{Float64}(undef, szT)
wwdata = Array{Float64}(undef, szG, szE, szA, szZ, szK, szT)
Xdata = Array{Int}(undef, szG, szE, szA, szT)
N19data = Array{Int}(undef, szG, szB, szE, szT-6)
Adata = Array{Float64}(undef, szT)
ω = Array{Float64}(undef, szG, szA, szT)
Mdata = Array{Int}(undef, szA, szA, szE, szE, szT)
Smdata = Array{Int}(undef, szA, szE, szT)
Sfdata = Array{Int}(undef, szA, szE, szT)
LLdata = Array{Float64}(undef, szG, szE, szA, szZ, szK, szT)
P_1data = Array{Float64}(undef, szK, szT-1)
edata = Array{Float64}(undef, szK, szT)

## block

data = ComponentArray(
    ww = wwdata,
    e = edata,
    P_1 = P_1data,
    X = Xdata,
    LL = LLdata,
    N19 = N19data,
    A = Adata,
    ω = ω,
    cost = costdata,
    M = Mdata,
    Sm = Smdata,
    Sf = Sfdata
)

println("success")

I suspect it has to do with memory use because if I comment out the line with ww in the definition of the ComponentArray the code runs without crashing. What could be happening?
I'm using Julia 1.5.3, VSCode 1.53.0 and ComponentArrays.jl 0.8.11.

Oh, that's a weird one. I'm not sure what would cause that and I am not able to reproduce the error on my side (same Julia and ComponentArrays version, but VSCode 1.53.0 instead).
One thing you can try is changing the ComponentArray( part to ComponentArray{Union{Float64, Int}}(. ComponentArrays, like regular arrays, will try to convert every element to the same type. So in this case it will try to make things all Float64s. Giving an explicit eltype of Union{Float64, Int} will allow elements to keep their type. If this works, than there must have been some issue when converting from Int to Float64. But I can't imagine what that would be (or why it only happens in VSCode).

That seemed to do it. Thank you. One thing though is that the same problem happens with Atom/Juno and VSCodium.

Hmmm.. that is weird. I'll have to try it on my other computer that has Juno on it tomorrow. Also, what OS are you using?

MacOS 11.2.

Hm. I'm on 11.1. I'll see if this happens when I upgrade to 11.2. But also I'm running it through Rosetta so that might change things.

Okay, I was able to reproduce it on a different computer. I had a line that did vcat(array...) on construction that I used to promote everything to a common type. It was being dispatched in this case to a vcat(::Float64, ::Float64, ::Float64, ::Vararg{Number,N} where N) method and for some reason failing there. I just changed it to reduce(vcat, array) and it seems to work. I know there's gotta be a better way to do this by directly promoting the array eltype myself, but everything I try seems to be much slower or doesn't work like I want it to.

Hmm, but why didn't that crash the REPL?
One thing that is not clear to me is that the ComponentArrays promotes everything to a common type, so is it futile to attempt to keep arrays as Ints or Floats? am I better off simply converting everything to Float64 before feeding that to a CA?

Yeah, I'm still not sure why it didn't crash in the REPL. I'm not totally sure why it crashed in the first place, I just noticed that that was the line that seemed to do it for me.
You'll want to use the Union trick if you want to have mixed eltypes, just like you would with normal Arrays. Promotion with ComponentArrays works the same as it does for normal Arrays because CA's store everything as Arrays under the hood. If you'd like the elements to keep their type, you have to explicitly annotate a wider element type. Here's an example:
julia> ComponentArray(a=1, b=2.0) # default conversion to Float64
ComponentVector{Float64}(a = 1.0, b = 2.0)

julia> ComponentArray{Int}(a=1, b=2.0) # explicit conversion to Int
ComponentVector{Int64}(a = 1, b = 2)

julia> ComponentArray{Real}(a=1, b=2.0) # widen the eltypes to Real
ComponentVector{Real}(a = 1, b = 2.0)

julia> ComponentArray{Union{Float64,Int}}(a=1, b=2.0) # widen the eltypes to Union{Float64,Int}
ComponentVector{Union{Float64, Int64}}(a = 1, b = 2.0)

This is analogous to what you would do with normal Arrays:
julia> [1, 2.0] # default conversion to Float64
2-element Array{Float64,1}:
 1.0
 2.0

julia> Int[1, 2.0] # explicit conversion to Int
2-element Array{Int64,1}:
 1
 2

julia> Real[1, 2.0] # widen the eltypes to Real
2-element Array{Real,1}:
 1
 2.0

julia> Union{Float64,Int}[1, 2.0] # widen the eltypes to Union{Float64,Int}
2-element Array{Union{Float64, Int64},1}:
 1
 2.0

For your situation, it depends. If you don't need the integer elements to actually be integers (i.e. you aren't using them to index or anything), converting everything to Float64 will be fastest because homogeneous arrays are always faster than heterogeneous.. If you do need the integers to remain integers, Union{Float64,Int} is the best way to do it (it is faster than Real because the compiler can specialize for unions of less than 4 types).

I see, thank you for the detailed explanation.


Packaging for release 9.0.1

We had rolled out a warning header to all admin and storefront APIs (REST and Graphql) if an unsupported or soon to be unsupported version pf the API was used in Shopify Core.
Shopify/shopify#232580
Now we have added a warning message within the Shopify API gem as well.
https://github.com/Shopify/shopify_api/pull/685
This version bump is done to release the changes in shopify api gem

/shipit


Create item on the fly

When the text entered as a filter does not match any of the predefined items, pressing Enter should create a new item on the fly (with an undefined id) and notify the parent component.
Possible options:

Ability to create new items on the fly (boolean)
Case-sensitivity during the "filter text / item" matching process
ID of the newly created items (maybe "undefined" by default, but a configurable value would be helpful)

Duplicate #105
This feature will not be added in release 1.0.0.


README doesn't use syntax highlighting in Go example code

README doesn't use syntax highlighting in Go example code. Github Markdown supports that.

👏


Add random map generation

ValueNoise is added to generate the TileMap
Completely untested right now. Will test as soon as rendering is added

Closed until rendering is added

Duplicate with #13


Use npm packages for selenium/chromedriver so they don't go out-of-date

Having the chromedriver and selenium configs in custom scripts is different from our other repos, where we use the npm packages that handle downloading, starting, stopping and correctly setting the path for the respective parts of this system.
If we don't do this, we will have another https://github.com/LLK/scratch-blocks/issues/1567 in a short while

I'll pull any change for this into Blockly as well.


Fix memory leak

hi,
Improve memory consumption by forcing GC between project and by use a smaller object.
Fixes #109

Thanks for your contribution! I've published https://www.nuget.org/packages/SourceBrowser/1.0.22 with your fix. It should get indexed soon.

Your welcome. Thank for your work and to accept PR so quickly.


Create ponyc options

Refactor the option processing code in main to a libponyc. This will allow
easier testing of standard ponyc options.

As it turns out test/libponyc doesn't yet have a way to test an end to end compilation, where an executable is created using genexe. Instead it uses gen_jit_and_run and a executable binary is never created. So my initial thoughts on using this to "easily" test --bin-name isn't so easy, at least for me.
I think this change still might be desired as eventually I believe it could be useful, but if not let me know and I'll close it.

I'm not sure what I've done wrong, but all builds are failing. It works on my Ubuntu 14.04 system so for now I'm going to close this and maybe come back to it later.


How can I start a member as a read-only member?

Hi @greensky00 ,
Thank you for your great job!
I found that all members start as a leader. I have 4 members (s0, s1, s2, s3), and I want only s0, s1 and s2 to participate in voting. How can I avoid s3 becoming a leader?

Hi @ZTJiu
In srv_config, there is a boolean flag indicating whether it is a learner:
https://github.com/eBay/NuRaft/blob/498cc30fb1e15fa4d1901821e9231a0727bfedfb/include/libnuraft/srv_config.hxx#L108-L112
You can make the flag true, and use it for initializing s2
https://github.com/eBay/NuRaft/blob/498cc30fb1e15fa4d1901821e9231a0727bfedfb/examples/in_memory_state_mgr.hxx#L34
then s2 will not be a leader at the beginning.
After that, you can use the same srv_config for adding s2 to the leader.
https://github.com/eBay/NuRaft/blob/498cc30fb1e15fa4d1901821e9231a0727bfedfb/include/libnuraft/raft_server.hxx#L114-L125
Once adding leader is successfully done, you can see a log like this:
2020-06-02T10:46:15.276_630-07:00 [8d67] [INFO] new configuration: log idx 4, prev log idx 3
peer 1, DC ID 1, tcp://<IP_ADDRESS>:11111, voting member, 50
peer 2, DC ID 1, tcp://<IP_ADDRESS>:11112, voting member, 50
peer 3, DC ID 1, tcp://<IP_ADDRESS>:11113, voting member, 50
peer 4, DC ID 1, tcp://<IP_ADDRESS>:11114, learner, 50
my id: 1, leader: 1, term: 1    [handle_commit.cxx:693, reconfigure()]

Hi @greensky00


Updating naming for Conjur editions

Updating naming for Conjur editions

converts Conjur OSS to Conjur Open Source
converts DAP/Dynamic Access Provider to Conjur Enterprise
maintains Conjur OSS Suite and Conjur Open Source Suite

This is part of an automated batch of PRs across many repos listed in cyberark/community#92

@rpothier added a (Formerly DAP) note at first mention of Conjur Enterprise


ir: ScheduleTree level

The ultimate goal is adding BC support to the YASK backend. This preliminary PR improves the IR to make life easier when it comes to do transformations in the YASK backend.
The main addition of this PR is the ScheduleTree (stree), an intermediate level of representation between that of Clusters and IET. When lowering from a sequence of Clusters to an IET, two main things are required:
1- the logic to determine what iteration spaces can be shared by different clusters;
2- the construction of the IET itself
previously, these two passes were intertwined in the same routine (iet_make), at  the same level of abstraction. This PR decouples them; an stree now takes care of point 1 above, by "unwinding"  the sequence of clusters into a minimal, mutable tree data structure. iet_make now only traverses the stree to emit a new, "fatter", immutable tree, namely IET.
This pass allowed me to drastically simplify the profiling engine, which is now vastly more clear (to understand) and robust than it used to be. I compared its output to that of master on a set of notable examples, but I'd like some of you (@navjotk, @tjb900, @ggorman, @mloubout  ) to do the same. Let me emphasise that all these changes are crucial for the development of the YASK backend.

Codecov Report

Merging #560 into master will increase coverage by 0.31%.
The diff coverage is 91.08%.

@@            Coverage Diff             @@
##           master     #560      +/-   ##
==========================================
+ Coverage   75.82%   76.14%   +0.31%
==========================================
  Files          74       77       +3
  Lines        7021     7143     +122
  Branches     1427     1436       +9
==========================================
+ Hits         5324     5439     +115
- Misses       1524     1533       +9
+ Partials      173      171       -2

Impacted Files
Coverage Δ

devito/ir/iet/visitors.py
81.29% <ø> (+0.25%)
:arrow_up:

devito/yask/operator.py
0% <0%> (ø)
:arrow_up:

devito/ir/equations/equation.py
79.16% <100%> (ø)
:arrow_up:

devito/profiling.py
95.89% <100%> (+3.1%)
:arrow_up:

devito/symbolics/inspection.py
94.64% <100%> (+6.18%)
:arrow_up:

devito/ir/stree/__init__.py
100% <100%> (ø)

devito/dse/backends/advanced.py
98.79% <100%> (+0.01%)
:arrow_up:

devito/ir/iet/utils.py
78.12% <100%> (ø)
:arrow_up:

devito/ir/stree/algorithms.py
100% <100%> (ø)

devito/ir/clusters/cluster.py
95.04% <100%> (+1.16%)
:arrow_up:

... and 12 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d93c002...b5f27ec. Read the comment docs.

Merged.