DrugExposureDiagnostics failed

When running the cdmOnboarding function with the following settings, an error occurs during the execution of DrugExposureDiagnostics, resulting in an empty section in the report for this diagnostic:
results <- CdmOnboarding::cdmOnboarding(
connectionDetails = connectionDetails,
cdmDatabaseSchema = 'omop',
resultsDatabaseSchema = 'results',
databaseId = '...',
authors = '...',
runVocabularyChecks = TRUE,
runDataTablesChecks = TRUE,
runPerformanceChecks = TRUE,
runWebAPIChecks = TRUE,
runDedChecks = TRUE,
baseUrl = 'http://localhost:8080/WebAPI',
outputFolder = outputFolder,
dqdJsonPath = '...',
smallCellCount = 5
)

Here is the error that appears in the logs:
...
2024-07-24 08:17:54	[Main thread]	INFO	CdmOnboarding	.execute	Running DED checks
2024-07-24 08:17:54	[Main thread]	INFO	CdmOnboarding	.runDedChecks	Starting execution of DrugExposureDiagnostics for 11 ingredients
2024-07-24 08:19:07	[Main thread]	ERROR	CdmOnboarding	3	Execution of DrugExposureDiagnostics failed: [1m[33mError[39m in `dplyr::collect()`:[22m [38;5;232m[33m![38;5;232m Failed to collect lazy table.[39m [1mCaused by error in `value[[3L]]()`:[22m [33m![39m org.postgresql.util.PSQLException: ERROR: syntax error at or near ","   Position: 11
2024-07-24 08:19:07	[Main thread]	INFO	CdmOnboarding	.execute	Done.
2024-07-24 08:19:07	[Main thread]	INFO	CdmOnboarding	.execute	Complete CdmOnboarding took 13.26 minutes
...

Environment

Ubuntu 22.04.4 LTS
R version 4.2.3
PostgreSQL 14.12
CdmOnboarding version 3.2.0

@GrigorisPapapostolou Could you execute DrugExposureDiagnostics in isolation and share the full trackeback?
library(DBI)
library(RPostgres)
library(DrugExposureDiagnostics)

server_parts <- strsplit(connectionDetails$server(), "/")[[1]]

connection <- DBI::dbConnect(
  RPostgres::Postgres(),
  dbname = server_parts[2],
  host = server_parts[1],
  user = connectionDetails$user(),
  password = connectionDetails$password()
)

cdm <- CDMConnector::cdm_from_con(
  connection,
  cdm_schema = cdmDatabaseSchema,
  write_schema = scratchDatabaseSchema
)

options(error = traceback)
DrugExposureDiagnostics::executeChecks(
  cdm = cdm,
  ingredients = c(
    528323,
    954688,
    968426,
    1119119,
    1125315,
    1139042,
    1140643,
    1154343,
    1550557,
    1703687,
    40225722
  ),
  checks = c("missing", "exposureDuration", "type", "route", "dose", "quantity", "diagnosticsSummary"),
  minCellCount = 5,
  sample = 1e+06,
  earliestStartDate = "2010-01-01"
)

I successfully executed the code mentioned earlier and have attached the output. Should I run the cdmOnboarding package again?
output.txt

Thanks @GrigorisPapapostolou, that looks like a successful run.
What version of DrugExposureDiagnostics did you use for this run? I suspect that this was using an older version of DED compared to what was executed as part of CdmOnboarding.

The versions of the packages DrugExposureDiagnostics and CdmOnboarding that I am using are '1.0.6' and '3.2.0', respectively.

I had the same problem. In my case it was caused by observation_period where observation_period_start_date > observation_period_end_date

This issue was solved with release 3.2.1, by disabling the validation done by CdmConnector


How to compile frontend files?

How do you compile the frontend files?
For example there are these REPLACE_BY_BUILD placeholders and some js files are minified and bundled in the release builds.
This does not seem to be part of the go building pipeline.

For the release builds we use Rollup.js to package and minify the JS code. We do something similar with CSS files but with an internal tool.
After this, the placeholders REPLACE_BY_BUILD in files and file names are replaced with the current build number.
 They are optional however, as r3 runs fine without them. We do this for the release builds to reduce code size and to make sure that browsers cannot use their caches for newer versions. Otherwise, "unoptimized" builds are functionally identical.

Thank you for the explanation.


Integrate all modules into pipeline

The pipeline is here!

Integrates all modules into the pipeline.
Adds a beautiful CLI for Schaapi.
Adds a cute little smoke test.

@FWDekker Any idea why the smoke test is failing on both CI services? (works on my machine :P)

@gandreadis Wait, it works on your machine? That's interesting, because I forgot to add the JAR that is used by the user project as a dependency, causing the compilation to fail (hence why Travis shows a MavenInvocationException). Interestingly it also works on my second machine. Pushing the missing JAR in 3... 2... 1

@gandreadis It was actually even less impressive than that: I forgot to install Maven before running it. Of course, on our machines it was already installed, so no problemo, but on CI? Nope.
Well, I hope that was the problem. We'll see.

@FWDekker Hmm, now getting a ZipException in the CI builds...

Codecov Report

Merging #58 into master will increase coverage by 2.26%.
The diff coverage is 88.54%.

@@             Coverage Diff              @@
##             master      #58      +/-   ##
============================================
+ Coverage     83.37%   85.63%   +2.26%
- Complexity      220      251      +31
============================================
  Files            21       21
  Lines           427      564     +137
  Branches        103      124      +21
============================================
+ Hits            356      483     +127
- Misses           29       36       +7
- Partials         42       45       +3

Impacted Files
Coverage Δ
Complexity Δ

...cafejojo/schaapi/projectcompiler/MavenInstaller.kt
100% <100%> (ø)
3 <1> (ø)
:arrow_down:

...fejojo/schaapi/testgenerator/SootClassGenerator.kt
89.36% <100%> (+4.25%)
29 <0> (+2)
:arrow_up:

src/main/kotlin/org/cafejojo/schaapi/Schaapi.kt
88.17% <88.17%> (+21.5%)
0 <0> (-2)
:arrow_down:

...afejojo/schaapi/patterndetector/PatternDetector.kt
97.14% <0%> (+2.85%)
23% <0%> (+2%)
:arrow_up:

...raphgenerator/compare/GeneralizedSootComparator.kt
92.1% <0%> (+3.42%)
44% <0%> (+8%)
:arrow_up:

...g/cafejojo/schaapi/usagegraphgenerator/SootNode.kt
84.61% <0%> (+34.61%)
21% <0%> (+21%)
:arrow_up:

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 d9701cf...4671df2. Read the comment docs.


TypeError with board["id"] prevents importing

Import fails with the following traceback message:
Traceback (most recent call last):
  File "/home/peter/tmp/env_trello_migration/bin/trello-to-deck", line 52, in <module>
    createdBoard = api.createBoard(truncate(board.title, MAX_TITLE_LENGTH), board.color)
  File "/home/peter/tmp/env_trello_migration/lib/python3.8/site-packages/trello_to_deck/deck.py", line 110, in createBoard
    boardId = board["id"]
TypeError: list indices must be integers or slices, not str

To Reproduce
Steps to reproduce the behavior:

create new venv:

python3 -m venv ~/tmp/env_trello_migration; source ~/tmp/env_trello_migration/bin/activate; pip install -U pip setuptools

clone & install repo:

cd ~/tmp; git clone https://github.com/maxammann/trello-to-deck.git
cd ~/tmp/trello-to-deck
pip install .

up till now, no obvious issues (a deprecation warning during the install mentions a change in pip 21.3 functionality, but it works as intended, as far as I can tell)..
and then the import:
trello-to-deck ~/Downloads/fmri_procedural.json [...]

before the trace-back, I get some "stats" on the json:
Found:
6 labels
4 stacks
35 cards (archived and non-archived)

Operating system:
Ubuntu 20.04.2 LTS
with python 3.8.5
Nextcloud version:
It's my uni's cloud instance, so cannot say much about it. But also doesn't seem relevant.
If useful for debugging, could post the json.

This is very strange as it seems like the API responded with an array instead of an object. I suppose that there is an issue with the Nextcloud instance. The version of deck should not be the problem as I added a compatibility check some time ago.
The JSON in your Downloads folder is not the problem.
Can you edit the file /home/peter/tmp/env_trello_migration/lib/python3.8/site-packages/trello_to_deck/deck.py and put a print(board) before line 110?

This seems to return the list of boards I already have on that cloud instance:
[{'title': 'Feedback Leo', 'owner': {'primaryKey': '4dcc0f53-e83f-4e8c-ae93-4bb21a69c537', 'uid': '4dcc0f53-e83f-4e8c-ae93-4bb21a69c537', 'displayname': 'Peter Vavra', 'type': 0}, 'color': '87665b', 'archived': False, 'labels': [], 'acl': [], 'permissions': {'PERMISSION_READ': True, 'PERMISSION_EDIT': True, 'PERMISSION_MANAGE': True, 'PERMISSION_SHARE': True}, 'users': [], 'shared': 0, 'stacks': [], 'deletedAt': 0, 'lastModified':<PHONE_NUMBER>, 'settings': {'notify-due': 'assigned', 'calendar': True}, 'id': 152, 'ETag': '8544528bdc49c1636e82728c7d932111'}, {'title': 'feedback prisma', 'owner': {'primaryKey': '4dcc0f53-e83f-4e8c-ae93-4bb21a69c537', 'uid': '4dcc0f53-e83f-4e8c-ae93-4bb21a69c537', 'displayname': 'Peter Vavra', 'type': 0}, 'color': '6ea68f', 'archived': False, 'labels': [], 'acl': [], 'permissions': {'PERMISSION_READ': True, 'PERMISSION_EDIT': True, 'PERMISSION_MANAGE': True, 'PERMISSION_SHARE': True}, 'users': [], 'shared': 0, 'stacks': [], 'deletedAt': 0, 'lastModified':<PHONE_NUMBER>, 'settings': {'notify-due': 'assigned', 'calendar': True}, 'id': 185, 'ETag': '40b54e2bc1aa484a5cd9966c13dda10a'}, {'title': 'gardening', 'owner': {'primaryKey': '4dcc0f53-e83f-4e8c-ae93-4bb21a69c537', 'uid': '4dcc0f53-e83f-4e8c-ae93-4bb21a69c537', 'displayname': 'Peter Vavra', 'type': 0}, 'color': 'ffffff', 'archived': False, 'labels': [], 'acl': [], 'permissions': {'PERMISSION_READ': True, 'PERMISSION_EDIT': True, 'PERMISSION_MANAGE': True, 'PERMISSION_SHARE': True}, 'users': [], 'shared': 0, 'stacks': [], 'deletedAt': 0, 'lastModified':<PHONE_NUMBER>, 'settings': {'notify-due': 'assigned', 'calendar': True}, 'id': 188, 'ETag': '965f80fc2252459808d8bc3495643f43'}, {'title': 'home improvement', 'owner': {'primaryKey': '4dcc0f53-e83f-4e8c-ae93-4bb21a69c537', 'uid': '4dcc0f53-e83f-4e8c-ae93-4bb21a69c537', 'displayname': 'Peter Vavra', 'type': 0}, 'color': 'ffffff', 'archived': False, 'labels': [], 'acl': [], 'permissions': {'PERMISSION_READ': True, 'PERMISSION_EDIT': True, 'PERMISSION_MANAGE': True, 'PERMISSION_SHARE': True}, 'users': [], 'shared': 0, 'stacks': [], 'deletedAt': 0, 'lastModified':<PHONE_NUMBER>, 'settings': {'notify-due': 'assigned', 'calendar': True}, 'id': 191, 'ETag': '27a4485b5de0f9e66b70db0cc8a57d69'}, {'title': 'desc vs exp', 'owner': {'primaryKey': '4dcc0f53-e83f-4e8c-ae93-4bb21a69c537', 'uid': '4dcc0f53-e83f-4e8c-ae93-4bb21a69c537', 'displayname': 'Peter Vavra', 'type': 0}, 'color': 'ffffff', 'archived': False, 'labels': [], 'acl': [], 'permissions': {'PERMISSION_READ': True, 'PERMISSION_EDIT': True, 'PERMISSION_MANAGE': True, 'PERMISSION_SHARE': True}, 'users': [], 'shared': 0, 'stacks': [], 'deletedAt': 0, 'lastModified':<PHONE_NUMBER>, 'settings': {'notify-due': 'assigned', 'calendar': True}, 'id': 194, 'ETag': '7c6583cf51e78047e1efa092ce239460'}, {'title': 'fmri ug', 'owner': {'primaryKey': '4dcc0f53-e83f-4e8c-ae93-4bb21a69c537', 'uid': '4dcc0f53-e83f-4e8c-ae93-4bb21a69c537', 'displayname': 'Peter Vavra', 'type': 0}, 'color': 'ffffff', 'archived': False, 'labels': [], 'acl': [], 'permissions': {'PERMISSION_READ': True, 'PERMISSION_EDIT': True, 'PERMISSION_MANAGE': True, 'PERMISSION_SHARE': True}, 'users': [], 'shared': 0, 'stacks': [], 'deletedAt': 0, 'lastModified':<PHONE_NUMBER>, 'settings': {'notify-due': 'assigned', 'calendar': True}, 'id': 197, 'ETag': 'f09d904d0e355314c8b79e84badbd026'}, {'title': 'General', 'owner': {'primaryKey': '4dcc0f53-e83f-4e8c-ae93-4bb21a69c537', 'uid': '4dcc0f53-e83f-4e8c-ae93-4bb21a69c537', 'displayname': 'Peter Vavra', 'type': 0}, 'color': 'f0c960', 'archived': False, 'labels': [], 'acl': [], 'permissions': {'PERMISSION_READ': True, 'PERMISSION_EDIT': True, 'PERMISSION_MANAGE': True, 'PERMISSION_SHARE': True}, 'users': [], 'shared': 0, 'stacks': [], 'deletedAt': 0, 'lastModified':<PHONE_NUMBER>, 'settings': {'notify-due': 'assigned', 'calendar': True}, 'id': 305, 'ETag': 'd2efb54b52476a873219c46e861e521e'}, {'title': 'Teaching', 'owner': {'primaryKey': '4dcc0f53-e83f-4e8c-ae93-4bb21a69c537', 'uid': '4dcc0f53-e83f-4e8c-ae93-4bb21a69c537', 'displayname': 'Peter Vavra', 'type': 0}, 'color': 'ce6bfc', 'archived': False, 'labels': [], 'acl': [], 'permissions': {'PERMISSION_READ': True, 'PERMISSION_EDIT': True, 'PERMISSION_MANAGE': True, 'PERMISSION_SHARE': True}, 'users': [], 'shared': 0, 'stacks': [], 'deletedAt': 0, 'lastModified':<PHONE_NUMBER>, 'settings': {'notify-due': 'assigned', 'calendar': True}, 'id': 308, 'ETag': '1b133322006c6296b8193374cd40c729'}]

Ok that indicates that trello-to-deck somehow sends a GET instead of a POST. I'll see whether I can find a reason

Can you execute this curl command?
curl -X POST $base_url/index.php/apps/deck/api/v1.0/boards --header "Content-Type: application/json" --data '{"title":"xyz","color":""}' --header "OCS-APIRequest: true" -u $user:$token_password

I suppose that your nextcloud server is returning a redirect http code here.

returns the following:
{"title":"xyz","owner":{"primaryKey":"4dcc0f53-e83f-4e8c-ae93-4bb21a69c537","uid":"4dcc0f53-e83f-4e8c-ae93-4bb21a69c537","displayname":"Peter Vavra","type":0},"color":"","archived":false,"labels":[{"title":"Finished","color":"31CC7C","boardId":329,"cardId":null,"lastModified":1620057704,"id":1328,"ETag":"02fe60eb89eb0783a257e4ac29a189cc"},{"title":"To review","color":"317CCC","boardId":329,"cardId":null,"lastModified":1620057704,"id":1331,"ETag":"02fe60eb89eb0783a257e4ac29a189cc"},{"title":"Action needed","color":"FF7A66","boardId":329,"cardId":null,"lastModified":1620057704,"id":1334,"ETag":"02fe60eb89eb0783a257e4ac29a189cc"},{"title":"Later","color":"F1DB50","boardId":329,"cardId":null,"lastModified":1620057704,"id":1337,"ETag":"02fe60eb89eb0783a257e4ac29a189cc"}],"acl":[],"permissions":{"PERMISSION_READ":true,"PERMISSION_EDIT":true,"PERMISSION_MANAGE":true,"PERMISSION_SHARE":true},"users":[],"stacks":[],"deletedAt":0,"lastModified":0,"settings":[],"id":329,"ETag":"cfcd208495d565ef66e7dff9f98764da"}

and it did create the board xyz

And you executed the curl command as-is, right without -L, --location flats, right?
If yes then I really do not have an idea right now what is going on. The python code clearly calls requests.post(..) which does a post request. A post request can never return a list of already existing boards.

yes, just prepended it with the variable assignments, i.e. base_url=[...]; user=[...]; token_password=[..]; [copy-paste-here], all in one call.
I've also asked our IT to give me some details on versions of both nextcloud and decks installation, but have to wait till I hear back.

Which python-requests version are you using? python3 -c "import requests; print(requests.__version__)"

2.25.1
just a quick follow-up on the -L flag on curl: I'm using zsh with oh-my-zsh. I doubt it, but maybe that sets some defaults on curl.. I'll try to see whether I find something on it..

Oke, next steps would be to create a minimal example in python. I'll do this in the next days.

Probably the issue is not the python code but the server: https://stackoverflow.com/questions/26149181/python-requests-post-doing-a-get
Can you send me the url of the server via mail<EMAIL_ADDRESS>Did you maybe use http:// instead of https://?

I was indeed using http:// :(
Replacing the calls with https allowed me to import my jsons.
Thanks for hunting this down and sorry it turned out to be such a simple mistake on my part..
Issue can be closed, unless you want me to still send you stuff/test things.
And thanks for the amazing tool and your support!

Good to hear! I almost went crazy while looking at the 2 lines of code which do weird stuff! :)
I will leave this open, as it still is bad behavior. I probably will prohibit redirects and simply abort the import with a proper error message. Therefore, I'll leave this open :)


Escape hash before use

https://github.com/filamentgroup/dialog/pull/33/files#diff-cefbae0b0d60a69dfd910cd59e69537aR35
Does that need to be updated to use a decoded id?

Sorry, a working link:
https://github.com/filamentgroup/dialog/blob/f26c717f7566d1ed0a31c9cf484fcef21b8a2f09/src/dialog-linker.js#L35

@johnbender no that line should stay encoded. but we can check for both when looking for a link


Update and refactor the kubeadm documentation + add HypriotOS instructions as well

@errordeveloper @lukemarsden @mikedanese
PTAL

This change is

Hey @luxas! I think the ARM/rpi stuff should be moved into a separate document and linked to from here. It's really cool that kubeadm supports ARM, but having it all inline here "if ARM, do this, otherwise, do that" makes it much harder to follow for the 80% case which is users installing Kubernetes on x86-64.

PTAL @errordeveloper @lukemarsden

Hey @luxas! I think the ARM/rpi stuff should be moved into a separate document and linked to from here. It's really cool that kubeadm supports ARM, but having it all inline here "if ARM, do this, otherwise, do that" makes it much harder to follow for the 80% case which is users installing Kubernetes on x86-64.
Please have a go at splitting it out into a separate doc and then resubmit for review. Thanks!

I don't think you addressed this, @luxas?

This needs a rebase.

Rebased and updated.
PTAL @mikedanese @lukemarsden @errordeveloper

I'll take a look tomorrow...

Thanks a lot!


⚠ AsianCinema has degraded performance

In 742d569, AsianCinema ($ACM) experienced degraded performance:

HTTP code: 200
Response time: 1667 ms

Resolved: AsianCinema performance has improved in 86aa4b8 after 8 minutes.


com.docker.network.bridge.host_binding_ipv6 without effect

I'm trying to setup ipv6nat on my machine.
I need a dedicated IPv6 address for the NAT as I want my main machine to be reachable on port 22 at its own IP.
If I'm reading the documentation correctly I can only define this on a user-defined network. Using the default bridge would probably easier but I don't see how i could set com.docker.network.bridge.host_binding_ipv6 there.
Now I created  a user-defined network as documented in Option B. But when I do a ip6tables -L -t nat -v I see the following result:
Chain DOCKER (2 references)
 pkts bytes target     prot opt in     out     source               destination
    0     0 RETURN     all      br-d152d505a932 any     anywhere             anywhere
    0     0 DNAT       tcp      !br-d152d505a932 any     anywhere             anywhere             tcp dpt:ssh to:[fd00:dead:babe::2]:22

If I compare it to the ipv4 entry I see in the destination that there should be the defined host binding IP instead of anywhere.
If I inspect the created network I see that "com.docker.network.bridge.host_binding_ipv6" is set to the correct IP.
Any idea what could be wrong?

I think I figured it out.
In the Readme the option is called  com.docker.network.bridge.host_binding_ipv6 which matches the docker setting com.docker.network.bridge.host_binding_ipv4
But if I look in the code here https://github.com/robbertkl/docker-ipv6nat/blob/master/state.go#L172 I see that the option is called  com.docker.network.bridge.binding_ipv6.
The second error I had was adding a subnet to the IP (like /80). I was mislead by the mentioning of ::/0 in the docs.
So it seems to work now, but the documentation or the code needs fixing.

Hi, you are exactly right, on both accounts. The name should match the IPv4 version, I'll make the necessary fixes to code and/or documentation right away, including your comment about the subnet.
I'm actually not using the option myself, for exactly the reason you mentioned before: the default bridge network is created on docker startup and it's options come from: --ip=x.x.x.x translates into com.docker.network.bridge.host_binding_ipv4=x.x.x.x on the bridge network). Unfortunately, there is no way to do this for my "user-defined" IPv6 bind option.
My server actually has just 1 IPv6 address configured, so the default bind to :: is fine for me.

All fixed in v0.1.2; new image is on Docker Hub as well.
Thanks for reporting @heinemml !


Feature selection not working

I tried these 2 methods:
'regsubsets' from library("leaps") returned in-correct results.
Tried 'rfeControl' from library(caret). Did not converge due to long training time. Code available in the open pull request to test
Here is good resource for feature selection libraries :
https://www.machinelearningplus.com/machine-learning/feature-selection/

I've tried other methods and they crashed too.

rfeControl is not crashing for me but it did not converge for around 1 hr and I had to kill the process.
Is rfeControl  also crashing for you @laihanel ?


Initialize variables in surface/call.c

Thanks to mbennett@

LGTM
On Thu, Jun 30, 2016, 10:26 AM David G. Quintas<EMAIL_ADDRESS>wrote:

Assigned #7175 https://github.com/grpc/grpc/pull/7175 to @ctiller
https://github.com/ctiller.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/grpc/grpc/pull/7175#event-709481253, or mute the
thread
https://github.com/notifications/unsubscribe/AJpudc7qoXDjX7pH_6Nj0IDkXk8h-QKiks5qQ_w6gaJpZM4JCZHC
.

LGTM. Maybe we should change the style guide to mandate that we always initialize pointers.

+1 @ctiller , what do you think. I can make the change in a jiffy.

@dgquintas: you can make another PR to the style guide change :-)

Yes, will do.
On 6 Jul 2016 11:16 a.m., "Nicolas Noble"<EMAIL_ADDRESS>wrote:

Merged #7175 https://github.com/grpc/grpc/pull/7175.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/grpc/grpc/pull/7175#event-714723078, or mute the
thread
https://github.com/notifications/unsubscribe/AAHVmeSSW-mgait6appf-JnRNJcHIO_Yks5qS_EGgaJpZM4JCZHC
.

Done: https://github.com/grpc/grpc/pull/7243


Lint errors in cloned webpack repo

Do you want to request a feature or report a bug?
Lint errors ~bug
What is the current behavior?
When running yarn test on a local machine (Windows), ESLint is reporting several errors, where at least 1 error seems to be legitimate. However, this was also tested on another machine (Mac), where no errors were shown during linting. Could there be a reason for this behavior?
If the current behavior is a bug, please provide the steps to reproduce.
Execute yarn test in the cloned webpack repository.
Environment:
OS: Windows 10
Node: v6.10.2
yarn: v0.23.2
ESLint: v3.19.0
Example file where the lint fails:
test\configCases\parsing\extended-api\webpack.config.js
other: './index' => Rule: Strings must use doublequote  quotes
This report seems to be correct?
What is the expected behavior?
No lint errors should be shown.
If this is a feature request, what is motivation or use case for changing the behavior?
I was about to create a PR for changing something else and was then more or less blocked by finding out what is causing these errors and how they can be fixed. Since this only occurred on one of my machines, it could be a configuration problem, on the other side, the aforementioned example doesn't seem to be wrong.
I have already changed a couple of files (6 in total) and could create a PR for this (if desired).

Was now able to test this on another Windows 10 machine in a different environment with Node v7.9.0. The same errors occurred on this one, so it seems to be Windows related.

Yes I noticed this too. windows seem to be correct here. Could be related to ** in the npm scripts. It would be great if you can locate the issue, fix the linting issues and send a PR.


feat(BlockchainTree): ShareableBlockchainTree and pending state

Create shareable blockchain tree that holds Tree behind RwLock.
Introduce traits

BlockchainTreeEngine for write operations used by engine
BlockchainTreeViewer for read operations and getting info data
BlockchainTreePendingStateProvider for creating pending state data needed to execute on

Removes dependency from the engine on the provider and executor.
Refactor the Tree so the pending state data can be extracted and used with the state provider factroy.

Codecov Report

Merging #2007 (45e3a37) into main (fdf2792) will decrease coverage by 55.72%.
The diff coverage is 2.79%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff             @@
##             main    #2007       +/-   ##
===========================================
- Coverage   73.03%   17.32%   -55.72%
===========================================
  Files         426      430        +4
  Lines       52210    52359      +149
===========================================
- Hits        38131     9070    -29061
- Misses      14079    43289    +29210

Flag
Coverage Δ

integration-tests
17.32% <2.79%> (-2.15%)
:arrow_down:

unit-tests
?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

bin/reth/src/node/mod.rs
0.00% <0.00%> (-6.07%)
:arrow_down:

crates/consensus/beacon/src/engine/mod.rs
0.00% <0.00%> (-96.62%)
:arrow_down:

...ates/executor/src/blockchain_tree/block_indices.rs
0.00% <0.00%> (-88.06%)
:arrow_down:

crates/executor/src/blockchain_tree/chain.rs
0.00% <0.00%> (-85.09%)
:arrow_down:

crates/executor/src/blockchain_tree/externals.rs
0.00% <0.00%> (-91.67%)
:arrow_down:

crates/executor/src/blockchain_tree/mod.rs
0.00% <0.00%> (-94.94%)
:arrow_down:

...es/executor/src/blockchain_tree/post_state_data.rs
0.00% <0.00%> (ø)

crates/executor/src/blockchain_tree/shareable.rs
0.00% <0.00%> (ø)

crates/executor/src/lib.rs
0.00% <ø> (-100.00%)
:arrow_down:

crates/interfaces/src/blockchain_tree.rs
0.00% <0.00%> (ø)

... and 15 more

... and 290 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


lint in CI

ref https://github.com/nebula-contrib/ngbatis/pull/19

before that @CorvusYe could you PR to format all code first?

It maybe hard for me. There has  more then 3.5k checkstyle violations...T^T

Sorry, I thought there are auto-styling wheels for this, I will do it, then, never mind :D

I tried the reformat code tool in ide, and it can only handle half of it.
Do you have any tools to recommend.

I tried the reformat code tool in ide, and it can only handle half of it.
Do you have any tools to recommend.

I am not a java dev, just spent 7 hours😭(openrewreite+googot-format + manual work) to finish #25 phase 0, now it's 800 left and most of them are javadoc/import*/refactor related.

7h..great! You are really patient

7h..great! You are really patient

I planned to do it in like one hour. And time flied and I was doing this with dirty words in mind for for most of the time😅

I don't think any programmer would like this task.
It doesn't matter. I'll find a tool to deal with it.
There are some running errors in this version. We should consider rolling back first.
My mistake.

I don't think any programmer would like this task.
It doesn't matter. I'll find a tool to deal with it.
There are some running errors in this version. We should consider rolling back first.
My mistake.

Oh sorry for that

Oh sorry for that
It's not your fault.
I haven't formed the habit of doing code reviews on branches. T^T


Whats the plan for this repo to move to angular 8?

Whats the plan for moving the repo from angular 7 to angular 8?
We're moving our project to angular 8 and w need MSAL to be on Angular 8 as well to be able to use it.. Is there a plan for that?

@tbudukh We are currently working on upgrading @azure/msal-angular to support the newest versions on Angular, including 6/7/8/9, and are committed to having this work completed by the end of the year. We will make an announcement when that work has been completed and released.


Include Input filters/validators for non-body requests

The content validation and filters only apply to requests with bodies (PUT, POST, etc) but could also easily apply to non-body requests such as GET. It's out-of-scope under the assumption fields in Apigility only apply to the target data, though, so this type of feature could end up being fulfilled as 'per-method filters/validators'.
Currently, that logic would just go into the resource.

This is a planned feature, but realistically, unless somebody contributes it, we won't be doing it before 2.0.
Additionally, such a feature should also have a corresponding UI - which is the main blocker for why we haven't pursued it yet (it's non-trivial to make a useful UI for this, we found).

I've submitted a pull request #64 regarding this issue. I created the patch to address one our requirements and didn't see this issue. But I'll take a look at providing corresponding UI patch for the added functionality too.

Closed with #64


feat(cheatcodes): access broadcast artifacts

Motivation
Builds upon #9098
Closes #4732 + Closes #9083

Solution

// Returns the most recent broadcast for the given contract on `chainId` matching `txType`.
function getBroadcast(string memory contractName, uint64 chainId, BroadcastTxType txType) external returns (BroadcastTxSummary memory);

//  Returns all broadcasts for the given contract on `chainId` with the specified `txType`.
//  Sorted such that the most recent broadcast is the first element, and the oldest is the last. i.e descending order of BroadcastTxSummary.blockNumber.
function getBroadcasts(string memory contractName, uint64 chainId, BroadcastTxType txType) external returns (BroadcastTxSummary[] memory);

// Returns all broadcasts for the given contract on `chainId`.
// Sorted such that the most recent broadcast is the first element, and the oldest is the last. i.e descending order of BroadcastTxSummary.blockNumber.
function getBroadcasts(string memory contractName, uint64 chainId) external returns (BroadcastTxSummary[] memory);

[x] Tests

also we need a test for it, let's add one here https://github.com/foundry-rs/foundry/blob/master/crates/forge/tests/cli/test_cmd.rs which would execute some script and read produced artifacts

@klkvr ptal, I've added the cases in testdata/ with some dummy artifacts.

I'd prefer us to simplify artifacts reading with walkdir.
What we could do is:

iterate over all entires in broadcast dir

filter .json files which can be deserialized into ScriptSequence or MultiScriptSequence

After that we can just deal with decoded objects for filtering

Current logic ended up pretty massive and makes many assumptions on directory structure and filenames which is not great

Yep! Just realised that while writing the test
walkdir approach is way better.

I don't think this will close #9083 because it mentions the need for access of transaction hashes during script execution, which is not really feasible with current script design. This PR only adds access to artifacts after script was broadcasted
And #4732 mentions access of deployed contract addresses vs broadcasted transactions data, so I think we should either follow its design where you are getting address by string memory contractName ot at least add helpers matching the proposed getDeployment cheats to make API simpler

@klkvr added the getDeployment helpers.


Use non Integer values on image sizes

First of all, thank you for this awesome library! It is helping me a lot in a printing module (browser usage).
That module prints bills/licenses and I'm facing some issues when trying to put images of specific sizes inside one of that documents.
As docx-templates use an HTMLImageElement, just non negative integers can be used as values for width and height attributes. This matter limits a lot the posibilites of place an image within the document.
Does anyone know any way to deal with it? Does someone comes up with how decimal values could be used setting the sizes of an image?
Thanks in advance

I am using 1.5 for my height and width, they come through as I would expect.  Perhaps this has been added since this request was made?

Thanks @Gary-Delancy . I don't see anything in the code that would limit users to integers at this time. We're also not using HTMLImageElement (anymore).


amdvlk fails to create device chain

I'm trying to get vkdt (darktable raw image developing extremly fast) working. It fails with libvulkan_radeon from Mesa, though I thought to give amdvlk a try.
The vulkan loader fails in vkCreateDevice()
5605     res = loader_create_device_chain(internal_device, pCreateInfo, pAllocator, inst, dev, layerGIPA, nextGDPA);
5606     if (res != VK_SUCCESS) {
5607         loader_log(inst, VK_DEBUG_REPORT_ERROR_BIT_EXT, 0, "vkCreateDevice:  Failed to create device chain.");
5608         goto out;
5609     }

This is where the vulkan loader leaves with VK_ERROR_FEATURE_NOT_PRESENT

@hanatos FYI

Wait VK_KHR_swapchain                    : extension revision 70 is reported by vulkaninfo, but vkCreateDevice() returns VK_ERROR_FEATURE_NOT_PRESENT.

I thought it is the missing VK_KHR_swapchain in the beginning but it turns out the feature is there.
However it fails when requesting VK_KHR_SWAPCHAIN_EXTENSION_NAME with the device features of:
  VkPhysicalDeviceDescriptorIndexingFeaturesEXT idx_features = {
    .sType = VK_STRUCTURE_TYPE_PHYSICAL_DEVICE_DESCRIPTOR_INDEXING_FEATURES_EXT,
    .runtimeDescriptorArray = 1,
    .shaderSampledImageArrayNonUniformIndexing = 1,
  };
  VkPhysicalDeviceFeatures2 device_features = {
    .sType = VK_STRUCTURE_TYPE_PHYSICAL_DEVICE_FEATURES_2_KHR,
    .pNext = &idx_features,

    .features = {
      .robustBufferAccess = 1,
      .fullDrawIndexUint32 = 1,
      .imageCubeArray = 1,
      .independentBlend = 1,
      .geometryShader = 1,
      .tessellationShader = 1,
      .sampleRateShading = 0,
      .dualSrcBlend = 1,
      .logicOp = 1,
      .multiDrawIndirect = 1,
      .drawIndirectFirstInstance = 1,
      .depthClamp = 1,
      .depthBiasClamp = 1,
      .fillModeNonSolid = 0,
      .depthBounds = 1,
      .wideLines = 0,
      .largePoints = 0,
      .alphaToOne = 1,
      .multiViewport = 0,
      .samplerAnisotropy = 1,
      .textureCompressionETC2 = 0,
      .textureCompressionASTC_LDR = 0,
      .textureCompressionBC = 0,
      .occlusionQueryPrecise = 0,
      .pipelineStatisticsQuery = 1,
      .vertexPipelineStoresAndAtomics = 1,
      .fragmentStoresAndAtomics = 1,
      .shaderTessellationAndGeometryPointSize = 1,
      .shaderImageGatherExtended = 1,
      .shaderStorageImageExtendedFormats = 1,
      .shaderStorageImageMultisample = 1,
      .shaderStorageImageReadWithoutFormat = 1,
      .shaderStorageImageWriteWithoutFormat = 1,
      .shaderUniformBufferArrayDynamicIndexing = 1,
      .shaderSampledImageArrayDynamicIndexing = 1,
      .shaderStorageBufferArrayDynamicIndexing = 1,
      .shaderStorageImageArrayDynamicIndexing = 1,
      .shaderClipDistance = 1,
      .shaderCullDistance = 1,
      .shaderFloat64 = 1,
      .shaderInt64 = 1,
      .shaderInt16 = 1,
      .shaderResourceResidency = 1,
      .shaderResourceMinLod = 1,
      .sparseBinding = 1,
      .sparseResidencyBuffer = 1,
      .sparseResidencyImage2D = 1,
      .sparseResidencyImage3D = 1,
      .sparseResidency2Samples = 1,
      .sparseResidency4Samples = 1,
      .sparseResidency8Samples = 1,
      .sparseResidency16Samples = 1,
      .sparseResidencyAliased = 1,
      .variableMultisampleRate = 0,
      .inheritedQueries = 1,
    }
  };

[ERR] device does not support requested feature alphaToOne
[ERR] device does not support requested feature shaderInt16

Radeon RX 470

The libvulkan_radeon.so driver from Mesa seems to support shaderInt16.

Please re-open the new if the problem is still there


8470p No audio Mojave

So after a previous issue of my machine not booting after a fresh install everything works fine except audio. I have had exactly the same issue no matter how I try and load Macos on this machine (previously used Rehabmans Elitebook clover guide) I resorted to using this program in the hopes it would fix this issue.
Awesome work by the way Chris! This has massively simplified the whole process of booting Macos on my machine!
Back to the no audio Issue I am completely stumped, This was never an issue back in the El capitan Days which I previously had working on this machine.

Deleted APPLE Folder just so I could upload here but OC and BOOT is present
EFI.zip

DSDT.aml is not create follow the guide
Mandatory Boot from USB Create  install Media HP installer, if not No DSDT was create

I have absolutely no idea, If I am being honest I used a previously existing install media that was clover based I think. Machine boots very fast and everything works great other than sound! I will start from scratch.

You need start from scatch  to make DSDT.aml create with the patcher and Audio will working after booting from the Drive

So I started from scratch, my first problem is following the guide the usb installer will not boot at all. I followed your video guide step by step. I have gone back to the install I had working because DSDT.aml is in the OC/ORIGIN/ folder. I have found that audio will only work if I select AirportBrcmFixup.kext in the Post installer this completely breaks my Wifi though which is already a compatible card. For what reason would AirportBcrm fix my audio?

I have many, many user with the 8470p without any issue for installing so you are probably missing some thing

I can attach my EFI folder again. I will completely delete it and run post installer. I will upload 2 one with AirportBcrmFixup.kext which will have working audio but kills my wifi and one without Airport fixup but audio will not work but wifi will. I will delete my EFI folder twice so I can do this. I am completely stumped why this isn't working correctly as this is not the first time i have installed MacOs on this laptop.

Ok by some unholy miracle I have fixed this. I do not understand how or why but basically I had to run the installer TWICE with the same options selected after a reboot. The first time running everything worked Other than audio. After a reboot and running the program again with the exact same selected options and another reboot audio started to work alongside my wifi.

After first install clearly no Audio but everything working. Rebooted and ran the installer again with the same options and now everything works. I have been deleting my EFI folder between installs previously just in case there are issues or conflicts. This is the first time I have reinstalled without deleting the EFI.

Thats not work like this, you need booting and installing macOS from USB HP Media
the guide is verry clear anyway good luck


[WIP] New docs

This is still a work in progress, but keeping it as an open PR as we had people wonder why the docs don't reflect the current state of the project. This will also generate a preview for whoever wants to access them (at your own risk :D).
Definitely not ready for review, so don't waste your time on it yet :)
Closes #42.
Closes #56.
Closes #57.
Closes #3.
Closes #24.
Closes #71.
Closes #83.
TODO

[x] Update favicon (white version that we tried to use in 3.0?)
[x] Move search index to its own file instead of embedding in html
[x] "IRC" has two underlines in docs (one from link, one from abbr)
[ ] Guide names shouldn't all start with "How to...", redundant and constraining. Thoughts?
[ ] Write page "How to set up LDAP authentication for The Lounge"
[ ] Write page "How to authenticate to IRC servers using SASL"
[ ] Write page "How to identify users using identd or oident"
[ ] Write page "How to mask hosts and IPs using Hexip"
[ ] Write page "How to use ZNC with The Lounge"

You might not need it

[ ] Write page "How to setup The Lounge as a standalone application"

Depends on OS, browser, etc.
Find better URL / title?

[ ] Write page "How to setup The Lounge as a standalone application"

See https://kiwiirc.com/docs/webirc
WEBIRC netuser netpass userip :userhostname
Need to make clear that: 1) This needs collaborating with the IRC network; 2) config is per hostname
You might not need it

[x] Pre-release install button takes full-width on homepage
Mobile / Responsiveness issues:

[ ] Community page: Backers snippet goes out of the screen
[ ] Community page: Youtube embed goes out of the page
[ ] Configuration page: Long ldapFilter string is not breaking to the next line
[x] Header and menu look poor
[x] "Edit this page" button weirdly aligns on most pages

Update the upgrade guide:

[x] Mention logs timestamps are now UTC (see this). Check if they weren't UTC before v3 anyway.
[x] Docker breaking changes
[x] Node.js v9 is not supported

[ ] Menu does not mark Guides/API as active when visiting one of the guides or the APIs

Won't fix

Color contrast on links are not AA compliant: Will do on re-style post-docs

An additional thing we need to add to the todo list: theme/package instructions. You've got the API but not instructions on installing/using.

@McInkay, thanks for checking out! :)
I did add something actually, in the Usage section: https://deploy-preview-84--thelounge-chat.netlify.com/docs/usage#installing-additional-themes
At the moment, it's only about themes because there are no packages out there at all, but as soon as https://www.npmjs.com/search?q=keywords%3Athelounge-package has a single package, I'll update the section to be about both themes and packages :)
Chicken and egg situation, because it could also be confusing to tell users how to install packages when there is not a single one.
I am confident you will be publishing the very first package, so I'm happy to work on the doc if you do so!

I have a number of packages ready to be published once my PRs are merged :-P so yeah, that's fine leaving it as just themes just now, I get that.
However, I'm wondering whether that's obvious enough or not. Maybe it is.

It should be noted in install docs that since Yarn installs into user folder by default, the binary isn't going to be available in $PATH.
On linux the fix is to add Yarn global path to your $PATH like export PATH="$PATH:$(yarn global bin)"

@xPaw, I'd rather not do this because it's very platform-dependent. For example, on macOS (though I know server install is rarely on macOS, but still, we do advertise it as cross-OS :D), it's available by default if you install with Homebrew.
You probably remember, but long time ago (ahem) we were documenting how to install Node on Ubuntu, Mac, and Windows, then we had to add CentOS/Fedora/RHEL, at which point we decided to redirect to Node's comprehensive package manager install document (as it is on the current doc). The Yarn install page is actually really good, lets you pick your OS and gives instructions accordingly, so I think I'd rather send reader to this page early in the install section, similarly to what we do with Node. You'd be okay with that?

Yep, that's what I meant, adding a link to this page and making it all a bit better. Will do!

Closing this PR now. Everything has been moved to other PRs/issues and in most cases merged. Only leftovers of this PR are things that do not need to be merged.


Updating Ubuntu versions in README.md

cf https://github.com/whoenig/thinkpad-p14s-gen2-ubuntu/issues/6

Thanks!


⚠ Bofrost.de has degraded performance

In 07f8f20, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1420 ms

Resolved: Bofrost.de performance has improved in a238511 after 6 minutes.


Fix small typo

confuration -> configuration

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: 11Takanori
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: nmittler
If they are not already assigned, you can assign the PR to them by writing /assign @nmittler in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Hi @11Takanori. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: 11Takanori, howardjohn
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: nmittler
If they are not already assigned, you can assign the PR to them by writing /assign @nmittler in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@11Takanori: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests.sh
8b91fd4b4703d51333266dd5959a40df15b3586f
link
/test e2e-simpleTests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@11Takanori: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests.sh
8b91fd4b4703d51333266dd5959a40df15b3586f
link
/test e2e-simpleTests

prow/istio-unit-tests.sh
8b91fd4b4703d51333266dd5959a40df15b3586f
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test istio-unit-tests

ping @rshriram @andraxylia

:thinking: :bug: You appear to be fixing a bug in Go code, yet your PR doesn't include updates to any test files. Did you forget to add a test?
Courtesy of your friendly test nag.


Home page tailwindcss development

Purpose

The purpose of this PR is to fix #50
Goals

The UI was developed referring the Home Page mockup
TailwindCSS was utilized to style the fonts and buttons in the home page
Flex and Grid attributes in TailwindCSS are used to align the elements in the home page

Approach

The UI of the home page of ScholarX version 2 was migrated to TailwindCSS by removing the Ant Design components. Refer TailwindCSS documentation for more information.
Screenshots

Navbar and Hero Section

Statistics and Testimonial Section

Success Stories and Archive Section

FAQ for Mentors and Mentees Section

Footer Section

Checklist

[x] This PR doesn't commit any keys, passwords, tokens, usernames, or other secrets.
[x] I have read and understood the development best practices guidelines ( http://bit.ly/sef-best-practices )
[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation

Remove ant design from the package json and run npm install.

Done !


IGNITE-5225: Fixed bug in TcpCommunicationSpi.

Fixed:

Thread pool isn't shutdown in case of Exception in some cases.
Unresolved ClusterNode addresses cause NPE while node address being checked for reachability.


Dubbo 2.7.5, 2.7.9, 2.7.11, 2.7.12, 3.0.0, 3.0.1 are not compliant with nacos-spring-context 1.1.0

Environment

Dubbo version: 2.7.5 or 2.7.9 or 2.7.11 or 2.7.12,  3.0.0 or 3.0.1
Operating System version: centos7
Java version: 1.8
nacos-spring-context 1.1.0

Steps to reproduce this issue
create a maven spring-boot test application which pom.xml has dubbo first and nacos-spring-context 1.1.0 later,
with dubbo which depend on spring-context-support 1.0.10:
a ClassCastException - com.sun.proxy.$Proxy...NNN... cannot be cast to java.util.Map - will be throwed at AnnotationNacosInjectedBeanPostProcessor.java line 145.
with dubbo which depend on spring-context-support 1.0.8 or less:
An attempt was made to call a method that does not exist (com.alibaba.spring.util.BeanUtils.getBeanNames)
The reason is nacos-spring-context 1.1.0 depends  on spring-context-support 1.0.11 which isn't compliant with that dubbo depends now.

if exclude spring-context-support of dubbo in pom.xml, ClassCastException - java.lang.String cannot be cast to java.lang.Class - will be throwed at DubboAnnotationUtils.java line 95.

I think we can remove the dependency of spring-context-support in the future.

the root cause is alibaba spring-context-support 1.0.11 set default value of two new added elements classValuesAsString and nestedAnnotationsAsMap to true, which destroys dubbo ReferenceAnnotationBeanPostProcessor findFieldAnnotationMetadata and findAnnotatedMethodMetadata functions.

we make dubbo 2.7.12 and nacos 1.4.2 working well with each other by #8217 :

change alibaba spring-context-support version of dubbo pom.xml to 1.0.11
set classValuesAsString and nestedAnnotationsAsMap attributes of ReferenceAnnotationBeanPostProcessor to false
@horizonzy   @AlbumenJ

@kylixs PTAL

related issues:
#7115
#7385

@zrlw  Since dubbo 3.0.0, the AbstractAnnotationBeanPostProcessor of alibaba spring-context-support is no longer used, and this problem should not occur, please check it again.
Furthermore, dubbo 3 will gradually remove alibaba spring-context-support.

dubbo 2.7.x has merge pull request: https://github.com/apache/dubbo/pull/8217

@zrlw Please add a sample for regression testing:
https://github.com/apache/dubbo-samples/tree/master/dubbo-samples-nacos

@zrlw Since dubbo 3.0.0, the AbstractAnnotationBeanPostProcessor of alibaba spring-context-support is no longer used, and this problem should not occur, please check it again.
Furthermore, dubbo 3 will gradually remove alibaba spring-context-support.

you're right，i didn't notice that dubbo 3.x had used its own AbstractAnnotationBeanPostProcessor.

@zrlw Please add a sample for regression testing:
https://github.com/apache/dubbo-samples/tree/master/dubbo-samples-nacos

I  tried to run Junit Test of dubbo-samples-nacos-configcenter, only got IllegalStateException:
org.springframework.beans.factory.UnsatisfiedDependencyException: Error creating bean with name 'org.apache.dubbo.samples.configcenter.DemoServiceIT': Unsatisfied dependency expressed through field 'demoService'; nested exception is org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'demoService': FactoryBean threw exception on object creation; nested exception is java.lang.IllegalStateException: No registry config found or it's not a valid config! The registry config is: <dubbo:registry />
what else should i prepared?

i want to change these codes in NacosUtils.java from
String serverAddr = System.getProperty("nacos.address", "localhost");
Properties properties = new Properties();
properties.put(PropertyKeyConst.SERVER_ADDR, serverAddr);
ConfigService configService = NacosFactory.createConfigService(properties);

to
@NacosInjected(properties = @NacosProperties(serverAddr = "${nacos.address:localhost}"))
private ConfigService configService;

I tried to run Junit Test of dubbo-samples-nacos-configcenter, only got IllegalStateException:
org.springframework.beans.factory.UnsatisfiedDependencyException: Error creating bean with name 'org.apache.dubbo.samples.configcenter.DemoServiceIT': Unsatisfied dependency expressed through field 'demoService'; nested exception is org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'demoService': FactoryBean threw exception on object creation; nested exception is java.lang.IllegalStateException: No registry config found or it's not a valid config! The registry config is: <dubbo:registry />

what else should i prepare?

Please start nacos server first

modified dubbo-samples-nacos-configcenter:  apache/dubbo-samples#352


Fix the bug

User story: As an end user I want that all information in the downloaded pdf file is readable (text is not overlapping).
https://issues.apache.org/jira/projects/OFBIZ/issues/OFBIZ-12110?filter=allopenissues

Hours used: 1

Priority: High


messages.cs-cz.js created

Czech translation added

In order to merge this pull request I will need also an cs-cz translation for all gijgo widgets. Here is the list:
dialog/js/messages/
editor/js/messages/
gird/js/messages/


Null object prototype.

Not sure if this is something we should do, right now we're defaulting to the object prototype which isn't great.

It is actually making it pretty slow. Better use classical prototype pollution solutions.


add link to medium post

it's a nice writeup, adding a link in your readme will allow more people to learn from you
https://medium.com/myheritage-engineering/how-controllable-react-components-maximize-reusability-86e3d233fa8e

I put the wrong link in "what is controllable?"
Thank you very much! 😀


As a user, I would like my capture to automatically launch a classification request

Description
Currently, when a user wants to execute a classification on an image, they need to capture the image an manually classify every picture they took. Being able to automatically ask for a classification on an image can improve the workflow of our users by enabling them to continue working on the video feed while the classification occurs.
Expected Behavior
When the automatic classification is active, the capture button should capture the image and ask a classification request without needing the users to change feed to manually classify the seeds. The users should be notified when the classification is done.
Current Behavior
The capture button only capture the image. If the user wants to classify it needs to manually do it.
Suggested Implementation
Add an automatic classification mode that let the user choose the model it wants for the classification.

When the images are being submitted and waiting for the result, the user can see the state next to the image.

When done a ✔ could be displayed next to the image.

Project was not set

love that we're focusing on creating an optimised flow for our users. would having an hotkey (spacebar) to capture and schedule classify also be helpful?

I like the idea! A hotkey sounds more effective than clicking on the capture button.


⚠ Bofrost.es has degraded performance

In 5d0e97d, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 2670 ms

Resolved: Bofrost.es performance has improved in 1c59914 after 8 minutes.


worker: Expose additional QEMU runtime args via docker-compose

Change-type: patch
Signed-off-by: Kyle Harding<EMAIL_ADDRESS>

@balena-ci I self-certify!


Never ending black screen + some error

Hey, I think twitch found a work around for ublock origin because it stopped working for me last night. I found this and tried it out but the screen stays black with a loading symbol in the middle and never ends up showing the stream (tried waiting ~5 minutes)
I'm not sure if this is related but when I view my extensions, I noticed the extension had errors and it were these:

I did the same thing as you; i suspect that this plugin currently doesnt work...

same issue. only just downloaded the plugin yesterday and it worked fine for about 2 hours then infinite loading

Yeah, this extension blocks most Twitch streams from loading. I found a handful of streams I tried out that do work, but most do not. Disabling the extension allows streams to load again, but Twitch has found a way to bypass both Ublock Origin and AdBlock Plus, because once this extension is disabled, I get ads.
Not sure why some streams still work, though. Maybe the streamer has to opt into the new ad nonsense, and the ones that work have not yet.

I've tested locally and this issue is fixed by pull request #29. Would be great if we can get it merged.

Confirmed request #29, i manually made the changes and worked.

@dotnetdan I know you're not the project maintainer but do you know if any one is doing a build?

Hi. Here's a installable xpi (remove .zip from file name) if of any help :
twitch_hls_ad_block-0.6.3-fx.xpi.zip

this is now fixed in ublock official update
https://github.com/gorhill/uBlock/releases/tag/1.30.4

Any chance somebody can provide a crx build as well,or get that stuff merged already?
The forced pre-roll ads have gotten quite relentless lately.
Really wish somebody could find a way to influence change to make the ads not interrupt the stream playback but instead play alongside the stream so as to not interrupt it so you don't end up missing anything.

Twitch is super determined to make people watch ads. We're now back to this addon not working (streams never load past the black screen with this enabled) and ads are bypassing Ublock/ABP again. I've never seen a turn around this quick.

@JasperKazai As mentioned in this issue, #29 fixes the issue with loading streams.
A big issue for me is that the current implementation will strip ALL segments when there are ANY ad segments. This means a couple of things:

You have a longer delay to view the stream as live segments / ad segments initially start off together
When it strips all segments the player will instantly re-request the main m3u8. This results in thousands of m3u8 requests whilst it looks for non-ad segments (literally a DDoS if used widely).

Additionally there's no indicator on screen that the extension is in the process of stripping ads. Some text at the top of the stream would be useful and fairly simple to add.
I've made a uBlockOrigin script based on this extension which addresses the above issues, but it misses out on the sequence tracking that this extension uses which I think would break at mid-roll ads https://gist.github.com/pixeltris/ee77c8338d7ebe1b9973dfcc973409d8
@instance01 do you have any intention of continuing to work on this? Or would it be best to fork?

I wonder if an attempt has been made to replace any and all mid-roll/pre-roll ads via a script with a video that lasts 1 second or even shorter and at super low quality so it loads even faster?
The script would have the tiny video copy the name of the ad vid and trick it into being said ad vid.
Another thing would be to detect known lengths of ad vids to target those specifically by having a cache of them and their lengths,also updating occasionally every time the list of them changes.
One more idea could be to alter the loaded ad vid "copy" to trick it into being much shorter in length,making it end prematurely.
Another thing could be to trick it into being a past broadcast then load the stream through that,if possible.

Detecting and removing ad segments is easy at the moment as they are conveniently tagged. There isn't any need to process the content of the actual video stream.
Even if they were to implement it perfectly, it would still be detectable as they will still need to flag the start/end time so that the player knows to make the stream/ad clickable so that they can get their money for the click-through. It's extremely unlikely they would ever serve zero engagement ads.

@pixeltris best to fork. I'm currently loaded with a few other projects. Also, I've stopped watching Twitch for quite some time now.
If anyone likes to maintain a fork, I will link to it in the Readme.


setting initial position moved to separate function

This has been already merged in #126 but somehow didn't make it from es6 branch to master.

nice!


gc: Use nm connection id as key

At some scenarios is convenient to save nm keyfiles with the same name
as the one expected from nmstate, this change convert Vec as
HashMap<String, String> for other backends it can be still used.
Signed-off-by: Enrique Llorente<EMAIL_ADDRESS>

Closing as this is fixed by https://github.com/nmstate/nmstate/pull/1916

This is breaking API for no good reason.
You should never assume the connection.id is unique. And never assume network backend generate a unique key.

This is breaking API for no good reason.
You should never assume the connection.id is unique. And never assume network backend generate a unique key.

The idea was to return to users the name of the .nmconnection that nmstatectl apply would have generated, so they generate exacly nm connections with same name.


MDI door icons missing

Trying to use mdi:door, mdi:door-closed, etc do not render.
Customization config:
sensor.door_basement_state:
  icon: mdi:door-closed

MDI icons are not instantly available inside HA....they are updated periodically.

@arsaboo - ahh, didn't realize that!  How often is periodically?

Usually the gap is not larger than 2 weeks.

The icons are updated. @danodemano can you close the issue please?


[fix] fix dangerously error on landing

Fix #50 based on this https://github.com/ReactTooltip/react-tooltip/issues/587#issuecomment-619675399
After this error, there was another server-rendering issue with react-select “Warning: Prop id did not match” that I fixed with a fixed instanceId. A better fix would be to use the useId() hook but it's only available on React 18.
import React, { useId } from 'react';
There is one remaining issue for mac devices only I think Extra attributes from the server: aria-activedescendant.
Everything seems linked to the fact that react-select does not render isomorphically.
There is an attempt to fix it specifically here, I'd suggest to wait and see if the PR get merged

lgtm !
I'm on macos and I confirm that I have the aria-activedescendant issue.
Thanks for your work @matthieuauger !

Warm welcome to the Walnut community and thank you for your contribution @matthieuauger ! Feel free to join our contributor telegram chat here: https://t.me/+DYI4FMia43I1NDI8
I will create a new issue to bump React to 18 and switch to useId(). We should also keep an eye on https://github.com/JedWatson/react-select/issues/5859 and fix when ready so thanks for linking


RAR archive mistaken as Tar archive

SharpCompress will not open and read the following RAR file correctly:
RAR file download link:http://wikisend.com/download/295490/RAR_identified_as_Tar.rar
Password: adamhathcock
(The download link expires after 90 days!)
The RAR archive is NOT in RAR5 format, and it contains only one file "PROTONIX.D64".
ArchiveFactory.Open(...) will return a SharpCompress.Archive.Tar.TarArchive instance for this RAR file.
And ReaderFactory.Open(...) will return a SharpCompress.Reader.Tar.TarReader.
SharpCompress version used: 0.11.1

I guess, the file format check should be changed so that Tar is not tested before other archive formats such as RAR.
In ArchiveFactory: https://github.com/adamhathcock/sharpcompress/blob/master/SharpCompress/Archive/ArchiveFactory.cs#L129
In ReaderFactory: https://github.com/adamhathcock/sharpcompress/blob/master/SharpCompress/Reader/ReaderFactory.cs#L65

Did your suggestion: https://github.com/adamhathcock/sharpcompress/commit/21087323af2caa5cfb394eee30ecd919b68421c6


[Feature Request] Add option to disable scrollIntoView when printing

Is your feature request related to a problem? Please describe.
What I am currently trying to do is to render strings with line breaks in a for loop, also using start() and stop() methods, and for each iteration the page keeps scrolling automatically until I stop the command and break the loop.
Describe the solution you'd like
https://github.com/mkrl/ttty/blob/7570c9a0442200e0603a23f147bae1c64345a7f4/src/api/print.ts#L12C3-L12C25
Here I would like to pass a boolean to specify that I don't want to scroll and avoid applying input.scrollIntoView(), it could be an optional parameter and scroll by default like it does until now, or any other more elegant solution that solves this issue is fine.
Thanks for your work, I really like this project, it's really fancy :)

Thank you! I'm surprised this package actually has some consumers :)
Just released in 1.7.0, you can check out the linked PR for details.

Such a quick response lol, thank you so much!


change dir name: src/blackwidow -> src/storage

change dir name: src/blackwidow -> src/storage

ork/pika/pika/include/pika_partition.h:9:10: fatal error: blackwidow/blackwidow.h: No such file or directory
9 | #include "blackwidow/blackwidow.h"
|          ^~~~~~~~~~~~~~~~~~~~~~~~~
compilation terminated.
please rebase and fix compile error


Consider merging or retitling questions 2.1 and 2.8

By title, they appear to be exactly the same. By content, 2.8 is more about exposing data across origins, whereas 2.1 is about exposing data at all.
It may be reasonable to merge the two questions, or maybe we should simply retitle them so the difference is clearer. Personally, I'm leaning toward merging them.
@pes10k thoughts?

@hober i think thats a great idea.  I vote for collapsing 2.8 into 2.1, and just mentioning in the body of the new 2.1 to specify:

what info is newly available to 1ps
what info is newly available to 3ps
what currently available info is mirrored (1p and / or 3p)

If that sounds good, and if you dont already have a PR for me to build from, i'd be happy to put a PR together!

@pes10k wrote:

If that sounds good,

It does.

and if you dont already have a PR for me to build from,

I don't.

I'd be happy to put a PR together!

That'd be great!

re convo with @samuelweiler, will instead see if we can remove both 2.1 and 2.8 and instead move them into intro text

closed by https://github.com/w3ctag/security-questionnaire/pull/100


Titanfall 2 Game Mode ideas

Titanfall trivia

has a lot of commands for the host to control the game
has a "ai" if no host selected
'prize"
leaderboard? with a plugin?

titan control

1 titan in the middle
closest player gets titan petted
when the titan dies he gets into the bt pose from crashsite
batteries spawn around the map
they are used to power the titan
the team that powers the titan gets it
this repeats until a team gets enough time in the titan

find the viper
Players go on maps in search for a way to kill viper again.
they solve puzzles on the way.

Drivable dropship
puzzles
convar for map completion
c2c logic
viper

more interactions

interactable stuff on maps like bridges, elevators, special events, launch pads, etc
have a callback system for maps.
the call back system should be a separate mod ( it will be used by find the viper too )

Server Side Tips????????????????????????????????????????????
just make a pr like for dilaogs

race track with MARVIN-radiant and CAR mod


nimble/host/include/host/ble_gap: expose ble_gap_adv_active_instance

This PR exposes ble_gap_adv_active_instance, it's sometimes useful to check if there is an active advertisement on an instance before stopping it or changing some parameter.

Is this one ok now @sjanc?

Thank you for the review! @sjanc


Updated Code

@aviral-mishra-hunter , please add jira ticket numbers description

Merging the PR to unblock qa testing of functionalities, will add review changes as separate jira tickets


Minor Improvement:

Add final
Remove redundant initializer
Replace String in loop by StringBuilder
Use Empty  array

Also please rebase on master to pick up the GitHub build addition.


Co-operate with Refile (RoR)

Refile is a file upload gem gaining popularity in the rails community.
It has support for background uploads (see documentation on direct, true). When this feature is enabled, although ezdz correctly manipulates the form, refile no longer submits the input.
Hopefully that's an adequate explanation of the problem.

My mistake, whatever I did it's working now: ignore me.


Add partial support for scoped packages

npm supports scoped packages, whose name matches the following scheme:
@somescope/somepackagename

These are used all over the modern NPM ecosystem, most notably by the Babel transpiler.
Unfortunately, Servant is not expressive enough so the code is a bit messy. Also, the GET /:scope/:name route gets shadowed by GET /:name/:version. It is still an improvement, since running npm install in a project with a lock file only seems to care about the tarball route, which works both scoped and unscoped.
Partial fix for https://github.com/nmattia/napalm/issues/27

See that all routes work except for the shadowed one:
$ curl -I<EMAIL_ADDRESS>HTTP/1.1 200 OK
Date: Sun, 01 Nov 2020 06:20:07 GMT
Server: Warp/3.3.13
Content-Type: application/octet-stream

$ curl -I http://localhost:3030/yaeti/-/yaeti-0.0.6.tgz
HTTP/1.1 200 OK
Date: Sun, 01 Nov 2020 06:20:27 GMT
Server: Warp/3.3.13
Content-Type: application/octet-stream

$ curl -I<EMAIL_ADDRESS>HTTP/1.1 200 OK
Date: Sun, 01 Nov 2020 06:20:48 GMT
Server: Warp/3.3.13
Content-Type: application/json;charset=utf-8

$ curl -I http://localhost:3030/yaeti/0.0.6
HTTP/1.1 200 OK
Date: Sun, 01 Nov 2020 06:20:59 GMT
Server: Warp/3.3.13
Content-Type: application/json;charset=utf-8

$ curl -I http://localhost:3030/yaeti
HTTP/1.1 200 OK
Date: Sun, 01 Nov 2020 06:21:03 GMT
Server: Warp/3.3.13
Content-Type: application/json;charset=utf-8

$ curl -I http://localhost:3030/@babel/helper-create-regexp-features-plugin
HTTP/1.1 500 Internal Server Error
Date: Sun, 01 Nov 2020 06:21:11 GMT
Server: Warp/3.3.13
Content-Type: text/plain; charset=utf-8

But in my experience, npm only uses the tarball route.

Hey! Thanks for the PR. I've never run into problems with scoped packages, can you share an example where this breaks?

I was getting ton of 404s from npm install. Currently I have this mess trying to fix node-gyp from #11, make parcel run and it does not even succeed:
{ pkgs ? import <nixpkgs> {} }: # nixos unstable for npm 7

let
  src = pkgs.fetchFromGitHub {
    owner = "jtojnar";
    repo = "pengu";
    rev = "464198bf06a96e698a37d2f21047337530d22a0f";
    sha256 = "lOdV1iN9cADHg51+GjVGK1h2CxD+tv7K8HZ53ii4gU4=";
  };
  nodejs = pkgs.nodejs_latest;
  napalm = import /home/jtojnar/Projects/napalm { pkgs = pkgs // { inherit nodejs; }; }; # this branch
in
  # Inspired by https://github.com/nmattia/napalm/pull/11
  napalm.buildPackage src {
    npmCommands = [
      # Just download and unpack all the npm packages,
      # we need napalm to patch shebangs before we can run install scripts.
      "npm install --loglevel verbose --ignore-scripts"
    ];

    nativeBuildInputs = with pkgs; [
      autoPatchelfHook

      nodejs.passthru.python

      # These tend to be needed when
      autoconf
      automake
      gettext
      gnum4
      gnused
      nasm
      pkg-config
    ];

    # Add hooks to configure phase.
    # https://github.com/nmattia/napalm/pull/26
    configurePhase = ''
      runHook preConfigure

      export HOME=$(mktemp -d)

      runHook postConfigure
    '';

    postConfigure = ''
      # Add node headers for node-gyp.
      mkdir -p $HOME/.node-gyp/${nodejs.version}
      echo 9 > $HOME/.node-gyp/${nodejs.version}/installVersion
      ln -sv ${nodejs}/include $HOME/.node-gyp/${nodejs.version}/include

      # Do not try to find npm in napalm-registry –
      # it is not there and checking will slow down the build.
      npm config set update-notifier false

      # Same for security auditing, it does not make sense in the sandbox.
      npm config set audit false
    '';

    postBuild = ''
      # Fix all the binaries with the .so from buildInputs. Magic!
      # autoPatchelf node_modules

      # Let’s install again, this time running scripts.
      npm install --loglevel verbose
    '';

    installPhase = ''
      runHook preInstall

      mkdir $out
      cp -r node_modules $out/

      runHook postInstall
    '';
  }

I use npm workspaces but I would not expect it to have any effect on napalm – as I understand it, npm will just install the workspace dependencies transparently using the registry, just like any other dependency.

Here is more minimal and completely hermetic example:
let
  nixpkgs = builtins.fetchTarball {
    url = "https://github.com/NixOS/nixpkgs/archive/1dc37370c489b610f8b91d7fdd40633163ffbafd.tar.gz"; # nixos-unstable
    sha256 = "1qvfxf83rya7shffvmy364p79isxmzcq4dxa0lkm5b3ybicnd8f4";
  };
  pkgs = import nixpkgs { };
  src = pkgs.fetchFromGitHub {
    owner = "jtojnar";
    repo = "pengu";
    rev = "464198bf06a96e698a37d2f21047337530d22a0f"; # master
    sha256 = "lOdV1iN9cADHg51+GjVGK1h2CxD+tv7K8HZ53ii4gU4=";
  };
  nodejs = pkgs.nodejs_latest;
  napalm = import (pkgs.fetchFromGitHub {
    owner = "nmattia";
    repo = "napalm";
    rev = "3c456911f9a0ae9804334344473173bd8ac314c5"; # master
    sha256 = "Ev31Fi24Zw93oJZUeFwBvR6ZUlbYodk+Am6ca6aMKGM=";
  }) { pkgs = pkgs // { inherit nodejs; }; };
in
  napalm.buildPackage src {
    postConfigure = ''
      # Do not try to find npm in napalm-registry –
      # it is not there and checking will slow down the build.
      npm config set update-notifier false

      # Same for security auditing, it does not make sense in the sandbox.
      npm config set audit false
    '';
  }

It produces the following error without this patch:
npm verb stack Error: 404 Not Found - GET<EMAIL_ADDRESS>npm verb stack     at /nix/store/ngga5g4siw4q9i0bqjxckg8aaq35rzgs-nodejs-15.0.1/lib/node_modules/npm/node_modules/npm-registry-fetch/check-response.js:123:15
npm verb stack     at runMicrotasks (<anonymous>)
npm verb stack     at processTicksAndRejections (node:internal/process/task_queues:93:5)
npm verb statusCode 404
npm verb pkgid @types/q<EMAIL_ADDRESS>npm verb cwd /build/source
npm verb Linux 5.4.0-52-generic
npm verb argv "/nix/store/ngga5g4siw4q9i0bqjxckg8aaq35rzgs-nodejs-15.0.1/bin/node" "/nix/store/ngga5g4siw4q9i0bqjxckg8aaq35rzgs-nodejs-15.0.1/bin/npm" "install" "--loglevel" "verbose"
npm verb node v15.0.1
npm verb npm  v7.0.3
npm ERR! code E404
npm ERR! 404 Not Found - GET<EMAIL_ADDRESS>npm ERR! 404
npm ERR! 404  '@types/q<EMAIL_ADDRESS>is not in the npm registry.
npm ERR! 404 You should bug the author to publish it (or use the name yourself!)
npm ERR! 404
npm ERR! 404 Note that you can also install from a
npm ERR! 404 tarball, folder, http url, or git url.

With this patch, I get a bit further, having issues with node-gyp.

I see, thanks a lot for this!

It is still curious why would it work for you. Do you have some idea?

Really unclear, maybe I dreamt it. @peterpeterparker we do use scoped packages in the DDG starter correct?

@nmattia indeed all our components are published as org scoped packages
npm i @deckdeckgo/core
npm i @deckdeckgo/deck-utils
npm i @deckdeckgo/slide-title
etc.


MagicWand example wrong dimensions in self trained model

Dear tflite-micro team,
Me and my friend were working on getting the MagicWand example to work with our own data.
While the example would run, with the included model, it wouldn't work, because we have a different accelerometer. That's why we wanted to train our own model on our own data. The process of making the model worked perfectly. The MagicWand example also built and compiled without problems, but when we tried running it on our Microcontroller we got an error at line 87 in main_functions.cc.

if ((model_input->dims->size != 4) || (model_input->dims->data[0] != 1) ||
(model_input->dims->data[1] != 128) ||
(model_input->dims->data[2] != kChannelNumber) ||
(model_input->type != kTfLiteFloat32)) {
TF_LITE_REPORT_ERROR(error_reporter,
"Bad input tensor parameters in model");
return;
}

When we tried training the model with your data from

download.tensorflow.org/models/tflite/magic_wand/data.tar.gz

we got the same error, even when we tried building and training it in the Google collab, were they say:

It will produce the same model used in the magic_wand example application.

We cant make out a difference between the models other than the size, which is:

20288 Bytes in our self trained models,
9760 Bytes in our selftrained models, when quantized
and 19616 Bytes in the given model from the MagicWand example.

When we look up the values in model_input->dims, we can see, that the values that are checked in line 87 and following aren't correct, but we don't know at which part of the process the mistake happens.
Also: We tried making the .tflm file into a .cc file with the xxd command in Linux, as well as with our own Python Script. The output was the same.
Additional information:

We are working in IAR RISC-V
on a RISC-V based microcontroller

We would love to get some help on this issue.
Best regards
Max & Benjamin

Sorry you're hitting problems! Can you share a link to your model (privately with<EMAIL_ADDRESS>if necessary) and the error message you see?


Fixing the link to Z3 project

Its broken now,
the text is 'https://github.com/Z3Prover/z3' and this should be also the link.

Thank you, it has been fixed.


mssql-server-is is not contained in the Ubuntu 20.04 repos, installation instructions cannot be used to install SSIS.

mssql-server-is is not in the Ubuntu 20.04 repos, so these instructions cannot be used to install SSIS on Ubuntu 20.04.

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 41654ae8-cbd1-c5dd-f15a-69466a765fda
Version Independent ID: 94311dce-80f0-b8ce-7296-b595ff50d640
Content: Install SQL Server Integration Services on Linux - SQL Server
Content Source: docs/linux/sql-server-linux-setup-ssis.md
Service: sql
Sub-service: linux
GitHub Login: @lrtoyou1223
Microsoft Alias: lle

@schwartzpub -- Jacob, thank you for your feedback.  Please consider these resources:

SQL Server on Microsoft Q&A
DBA Stack Exchange
Stack Overflow

@lrtoyou1223 -- Leo, please look into this issue.

The problem is the package does not exist in the repository defined in the documentation -- is it possible the documentation has been updated in advance of the package being made available?

@schwartzpub You are correct. SSIS is not available for Ubuntu 20.04. Since this is not a documentation issue, please direct your query to Microsoft Support.
I am closing this issue for now, but please feel free to respond.

#please-close

Note: I will be updating the existing document to reflect this.

#reassign:rwestMSFT

@schwartzpub Docs have been refreshed to remove support for SSIS on SQL Server 2022. When 2022 support is added, the article will be updated to reflect that. Thanks again for logging this issue.


NHMA - Taxonspid is not carried over from the App and into the specimen table

Describe the bug
NHMA: When using the alternative taxonomic identifier to apply the taxon name the taxonspid is always 0 (numerical zero) in the specimen table. This goes for all ranks.
To Reproduce
Steps to reproduce the behavior:

Launch Digi App and sign in as NHMA
Use the taxonomic ID field to apply the taxonomic name
Barcode + Save record
Observe the specimen table and see the new record having taxonspid = 0

Expected behavior
All names already in the taxonomy irrespective of entry should have the taxonspid property set to a number.
Please complete the following information:

version is 1.1.18
Column taxonspid in specimen is set with the correct taxonspid number. Novel names will have 0 of course.

moved to for recview

Fixed.


Add curated resource for book "Working Effectively with Legacy Code" by M. Feathers.

What I would do for this is to copy and edit what I wrote for Trilinos at:

http://trac.trilinos.org/wiki/TribitsLegacySoftwareChangeAlgorithm

For me, the Legacy Software Change Algorithm is the most important contribution from that book.
The content on that page is longer than most of the existing contributions for the BSS website but I think that summary is pretty good and is not all that long.
@maherou, what do you think about this appraoch?

Another option is to create a How-To and What-Is for the Legacy Software Change Algorithm.  The How-To document could be called "What is Legacy Code and the Legacy Software Change Algorithm?".  The What-Is document could be called "How to apply the Legacy Software Change Algorithm".  Then the BSS site could just provide a short overview and link to these.
What about that approach?

Hi Ross -- Mike and I discussed, and we agree that it would be great to develop a variant of the second option: WhatIs and HowTo docs, following the 2-page (or less) format used for prior WhatIs and HowTo docs.  Let's follow up offline to discuss approach, including incorporating input from Mark Miller.  Thanks.

Sorry, I did not see this when I created #331.  I will close this one as duplicate since I put #331 in the commits in PR #332.


Add F22 OSIE job posting

This is the job posting for the Open Source Infrastructure Engineer.
We had several rounds of reviews at https://docs.google.com/document/d/1kmD3mxfl0HdZi_ELPsbkao5TEq-mjnf5F-KbIvniYA0/edit#heading=h.xttlgd1v7308, so I expect to merge it and make it official pretty soon.

I am going to self-merge this one given the fact it was actually reviewed several times, so I am confident the content is good enough as it is. Let's make it on the public website now!


Makes better use of json-schema to test the changes to record resource

Also mark the YAML test for the record resource as xfail for now. This is
because this test currently fails for registers that use "integers" for keys.
This happens because we do not surround our keys with quotes in our API and the python yaml
library attempts to convert this into various datatypes. We then get a string comparison failure.
We need to work out if this is a problem with our API or the YAML library before removing the xfail.

Looks good, thanks for going through it with me.


Random string function

Is your feature request related to a problem? Please describe.
Would be nice to have a built-in random string generation function in bicep. Not for password (as discussed here) but just to do something like this instead of manually:
resource storageAccount 'Microsoft.Storage/storageAccounts@2019-06-01' = {
  name: take('${replace(guid(resourceGroup().name, location, 'stg'), '-', '')}', 22) // build unique storage account name from 'stg', location and guid() which is not longer than 22 chars and has no dashes

Describe the solution you'd like
A clear and concise description of what you want to happen.
resource storageAccount 'Microsoft.Storage/storageAccounts@2019-06-01' = {
  name: randomString(length: 22, seeds: ( resourceGroup().name, 'stg') )

optionally you can specify things like

can contain digits: yes/no
can start with digit: yes/no
etc

Have you tried uniqueString()? You can't control the number of characters returned (it is always 13), but otherwise looks like what you need.

ehm...yeah thats exactly what I was looking for :D disregard my issue, its Friday...


ralpne overwrites fimp(02) in IN.DAT

In GitLab by @hlux on Oct 14, 2016, 13:52
The thermal helium fraction is set via ralpne, but all ion species are also stored in fimp(). Therefore ralpne should always be equal to fimp(02). However, both can be set separately in IN.DAT and ralpne always overwrites fimp(02).
Ideally conflicting values should be flagged up in any PROCESS run, but definitely in the GUI changing one should also change the other!

As we no longer have a GUI for PROCESS, there is no longer confusion in this regard with setting conflicting values.
The behaviour is still the same, ralpne overwrites the fimp(02). A simple solution is to simply remove fimp(02) from any template input files in favour of ralpne. I think a review of the impurity radiation data structure is due, but this is a larger piece of work.

Should this really be closed? GUI or not, the issue is still there. I don't think this is about reviewing the impurity radiation data structure, I think it's about preventing one input from overwriting the other. Should either , ralpne or fimp(02) be removed?

This value is controlled by ralpne , in fact changing fimp(2) does not change the value of the parameter. It also doesn't make sense that a user would set these two parameters to different values in the first place.
However, as fimp(2) is part of the fimp array, it's not straight forward to remove it as an input variable. All members of fimp(1:13) are inputs, and all of them are output (and still used by other parts of the code).  A generic fortran function loop for looping through arrays is used to set them as inputs. It doesn't seem elegant to add a specific exception to this function.
I have added a warning to the user when they change fimp(2) from the default value, and they are told to use ralpne instead.


Custom Blade Directives Do Not Get Rendered

With blade minification turned on, any custom directives just get turned into plain text, example:
Blade Minification On

Blade Minification Off

The force option doesn't seem to affect the behavior....

You need to register your custom blade stuff in your BOOT method, not in register.

For anybody coming here with the same issue:
My custom blade directives were defined in the boot() method of a ServiceProvider but were still not being rendered in the minified output.
Make sure your ServiceProvider that adds those custom directives appears AFTER 'GrahamCampbell\HTMLMin\HTMLMinServiceProvider' in your app.providers config.


docker-compose throwing error "No command specified"

version: '3'

services:
  myservice:
    privileged: true
    image: myimage:latest
    container_name: mysensor
    volumes:
      - "/dev/bus/usb:/dev/bus/usb"
      - "/home/pi/healthcheck:/root/healthcheck"
    networks:
      myvlan:
        ipv4_address: <IP_ADDRESS>
    tty: true
    labels:
       - "autoheal=true"
    healthcheck:
       test: ["CMD-SHELL", "/root/healthcheck/check_ERROR.sh" ]
       interval: 5s
       retries: 1
       start_period: 5s
       timeout: 19s
  autoheal:
    image: autoheal:latest
    tty: true
    container_name: autoheal
    network_mode: none
    environment:
      - AUTOHEAL_CONTAINER_LABEL=all
      # check every 5 seconds
      - AUTOHEAL_INTERVAL=5
      # wait 0 seconds before first health check
      - AUTOHEAL_START_PERIOD=0
      # Docker waits max 10 seconds (the Docker default) for a container to stop before killing during restarts
      - AUTOHEAL_DEFAULT_STOP_TIMEOUT=10
      # Unix socket for curl requests to Docker API
      - DOCKER_SOCK=/var/run/docker.sock
      # --max-time seconds for curl requests to Docker API
      - CURL_TIMEOUT=30
    volumes:
      - /var/run/docker.sock:/var/run/docker.sock
    restart: always

Gives me this error when i try to start the containers:

pi@raspberrypi:~$ docker-compose -f docker-compose-autoheal-test.yml up -d
myservice is up-to-date
Creating autoheal ... error

ERROR: for autoheal  Cannot create container for service autoheal: No command specified

ERROR: for autoheal  Cannot create container for service autoheal: No command specified
ERROR: Encountered errors while bringing up the project.

What command is it expecting autoheal service to start with?

docker run -d \
    --name autoheal \
    --restart=always \
    -e AUTOHEAL_CONTAINER_LABEL=all \
    -v /var/run/docker.sock:/var/run/docker.sock \
    autoheal:latest

Gives me this error too:
pi@raspberrypi:~$ docker run -d \
>     --name autoheal \
>     --restart=always \
>     -e AUTOHEAL_CONTAINER_LABEL=all \
>     -v /var/run/docker.sock:/var/run/docker.sock \
>     autoheal:latest
docker: Error response from daemon: No command specified.
See 'docker run --help'.

close issue, image was not loaded properly into registry on the pi


Move YdbRepository start behavior from YdbRepository constructor to separate method

YdbRepository in the constructor establishes a connection to the database, because it is impossible to separate the stages of object initialization and start.
It seems that what is needed here is a factory object which will create a SessionManager and return a Repository

:zany_face: This is an unexpected change in YDB SDK v2, constructing a GrpcTransport immediately establishes a GRPC connection:
    public GrpcTransport build() {
        YdbTransportImpl impl = new YdbTransportImpl(this);
        try {
            impl.init();

We might just add a Guava Suppliers.memoize() for GrpcTransport and SessionManager to make initialization lazy, as it was in repository-ydb-v1.

:arrow_right: See PR with lazy initialization here:
https://github.com/ydb-platform/yoj-project/pull/61

Fixed, the fix will appear in 2.2.11.


Added a test and benchmarking for shifted qr

Added a test and benchmarking for shifted qr. Optimized the code and added struct UpperTriangularMatrix.

Seems like ListOfRotations is not yet defined where it's used. Maybe change the order of the includes?


ref(docker): use cache mounts for build cache

Motivation
We've been using external tools and custom approaches to improve the caching layers in Rust, and avoid invalidating the caches if no Rust-code is changed.
This update eliminates the need for external tools like cargo-chef to leverage caching layers, resulting in an average build time reduction of 4m30s (~36% improvement).
While this solution doesn't fully resolve the issues mentioned in https://github.com/ZcashFoundation/zebra/issues/6169#issuecomment-1712776391, it represents the best possible approach without resorting to custom solutions, which we'd prefer to avoid.
Depends-On: #8817
Specifications & References

https://docs.docker.com/language/rust/develop/

Solution

Use cache mounts instead of cargo-chef
Remove custom rsync steps

Others

Add a RUST_VERSION arg so we don't invalidate our cache so frequently with Rust upgrades
Use --link in our runtime stage, to reduce the resulting artifact
Remove the installation of Google OS Config agent, as this is no longer required

Tests

Tests are passing for CI and CD deployments
Rebuilding is taking less time (~8 minutes instead of ~13 minutes)

Follow-up Work

Do not run our runtime stage with the root user
Add SBOM and provenance attestations https://docs.docker.com/build/ci/github-actions/attestations/

PR Author's Checklist

[x] The PR name will make sense to users.
[ ] The PR provides a CHANGELOG summary.
[x] The solution is tested.
[x] The documentation is up to date.
[x] The PR has a priority label.

PR Reviewer's Checklist

[ ] The PR Author's checklist is complete.
[ ] The PR resolves the issue.

@arya2 this one is ready now

Do we want to close #6169 for now or are we planning on making any other improvements in the short term?

I've just linked #6169, for automatic closure.
This is not solving the underlying issue, but this is as far as we can get right now, until the mtime issue is solved in Rust.


searching status

Give some sort of status while the search is happening instead of expecting the user to wait around wondering what's going on.

added in 1a45706f4fa6a7b08d7707a29c580f3e682c18f9


Add vite-plugin-react-swc

Manual trigger just passed on my fork: https://github.com/ArnaudBarre/vite-ecosystem-ci/actions/runs/3608780832


RequestBody draining still allocates per-request buffers

This change: https://github.com/aspnet/KestrelHttpServer/commit/4bba074d778933b312011df0ade8ff875250cded
Fixed a serious allocation issue, but also added a 4kb buffer to each frame, which is more per-request allocation. Simply just making this static as well may not do what we want, as this might lead to a lot of contention for this cache line.
I think we need to revisit the original proposal which was to implement CopyToStreamAsync in a smarter way.
Allocation data for 3000 requests:

In my sample app, this is responsible for 12/82mb allocated

/cc @halter73 @benaadams

This is only on fresh connections, not keep alive already open connections?

Raised PR #335 "Use pooled array"

I guess I didn't look at the PR that added _nullBuffer closely enough. I thought _nullBuffer was static. Is there any reason it can't be static? I think that would even be more efficient.

All of these solutions involve copy bytes from one location to another in memory. Wouldn't it be better to override CopyToAsync and special case Stream.Null?

@rynowak That's definitely an option. We were considering a change to make CopyToAsync bufferless in all cases, but that seemed to be a little to complex for a last minute rc1 fix.
It might make more sense just to add a new method to FrameRequestStream that efficiently consumes the rest of the request rather than overriding CopyToAsync and checking if the argument is Stream.Null only to do the same thing.

:arrow_up: either of those would work.

I thought _nullBuffer was static. Is there any reason it can't be? I think that would even be more efficient.

Because you'll have multiple threads writing to it on different cores and cause cpu cache thrashing.

add a new method to FrameRequestStream that efficiently consumes the rest of the request

:+1:
Could even just call it directly in Frame

Something that fasts forwards to the end rather than doing the copy to a buffer would be better; as should be less work?


Please OS X user manual to README and how to set breakpoint in lldb?

If submitting a bug please make sure

[x] use lldb

[x] lldb --version is apple version
[x] it works on the command line with lldb-mi

Xcode command tools won't provide lldb-mi, but this command is shipped with Xcode.app
just one command can make OS X user use this extension.
ln -s/Applications/Xcode.app/Contents/Developer/usr/bin/lldb-mi /usr/local/bin/lldb-mi
And how to configure launch.json to make it support breakpoint?

These are the languages which have breakpoint support enalbed: https://github.com/WebFreak001/code-debug/blob/master/package.json#L189-L210
Basically if you use some language from that list you can simply hover over the line counter at the left in your files and click the red circle. The red circle indicates a breakpoint, you can also right click the circle and edit it. In that input box you can enter a break condition (for example i > 40 if you have a variable named i and it will only break if i is greater than 40)
If you are using a language with no breakpoint support but LLDB/GDB support just make an issue or fork and PR yourself. Its a really simple task which just requires you to add a string to the LLDB/GDB array.
I'm gonna add that ln command to the README

Thanks.
I just forget to add --debug : (

For anyone that may come back to this later, the URL with the languages that include breakpoint support referenced was https://github.com/WebFreak001/code-debug/blob/f0f10251e05bcd6fad5d137c21e0f72826cb245e/package.json#L189-L210. It seems more split currently (and I do wonder why "cobol" is listed under lldb-mi, but not under gdb, and guess its common [debuggable] id should be additional gnucobol )