[Bug]: npm run generate extension causes external error

Please confirm that you have:

[X] Searched existing issues to see if your issue is a duplicate. (If you’ve found a duplicate issue, feel free to add additional information in a comment on it.)
[X] Reproduced the issue in the latest CLI version.

In which of these areas are you experiencing a problem?
Extension
Expected behavior
I expected it to generate the extension with all it's files.
Actual behavior
I got an external error:
Error coming from `npm install<EMAIL_ADDRESS>--save-prod`

Command failed with exit code 1: npm install<EMAIL_ADDRESS>--save-prod
npm ERR! code ERESOLVE
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR!
npm ERR! While resolving<EMAIL_ADDRESS>npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/react
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS><18.0.0" from<EMAIL_ADDRESS>npm ERR! node_modules/@shopify/checkout-ui-extensions-react
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force, or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.

Verbose output
0 verbose cli C:\Program Files\nodejs\node.exe C:\Program Files\nodejs\node_modules\npm\bin\npm-cli.js
1 info using<EMAIL_ADDRESS>2 info using<EMAIL_ADDRESS>3 timing npm:load:whichnode Completed in 0ms
4 timing config:load:defaults Completed in 1ms
5 timing config:load:file:C:\Program Files\nodejs\node_modules\npm\npmrc Completed in 2ms
6 timing config:load:builtin Completed in 4ms
7 timing config:load:cli Completed in 2ms
8 timing config:load:env Completed in 0ms
9 timing config:load:file:S:\projects\testest\checkout-app-test\.npmrc Completed in 1ms
10 timing config:load:project Completed in 5ms
11 timing config:load:file:C:\Users\connor\.npmrc Completed in 0ms
12 timing config:load:user Completed in 1ms
13 timing config:load:file:C:\Users\connor\AppData\Roaming\npm\etc\npmrc Completed in 0ms
14 timing config:load:global Completed in 0ms
15 timing config:load:validate Completed in 0ms
16 timing config:load:credentials Completed in 1ms
17 timing config:load:setEnvs Completed in 1ms
18 timing config:load Completed in 17ms
19 timing npm:load:configload Completed in 18ms
20 timing npm:load:mkdirpcache Completed in 0ms
21 timing npm:load:mkdirplogs Completed in 0ms
22 verbose title npm install<EMAIL_ADDRESS>23 verbose argv "install" "<EMAIL_ADDRESS>"--save-prod"
24 timing npm:load:setTitle Completed in 1ms
25 timing config:load:flatten Completed in 3ms
26 timing npm:load:display Completed in 4ms
27 verbose logfile logs-max:10 dir:C:\Users\connor\AppData\Local\npm-cache\_logs
28 verbose logfile C:\Users\connor\AppData\Local\npm-cache\_logs\2023-04-07T19_55_56_398Z-debug-0.log
29 timing npm:load:logFile Completed in 5ms
30 timing npm:load:timers Completed in 0ms
31 timing npm:load:configScope Completed in 0ms
32 timing npm:load Completed in 29ms
33 timing arborist:ctor Completed in 0ms
34 silly logfile start cleaning logs, removing 5 files
35 timing idealTree:init Completed in 313ms
36 timing idealTree:userRequests Completed in 3ms
37 silly idealTree buildDeps
38 silly fetch manifest<EMAIL_ADDRESS>39 silly logfile error removing log file C:/Users/connor/AppData/Local/npm-cache/_logs/2023-04-07T19_46_53_015Z-debug-0.log [Error: EPERM: operation not permitted, lstat 'C:\Users\connor\AppData\Local\npm-cache\_logs\2023-04-07T19_46_53_015Z-debug-0.log'] {
39 silly logfile   errno: -4048,
39 silly logfile   code: 'EPERM',
39 silly logfile   syscall: 'lstat',
39 silly logfile   path: 'C:\\Users\\connor\\AppData\\Local\\npm-cache\\_logs\\2023-04-07T19_46_53_015Z-debug-0.log'
39 silly logfile }
40 timing arborist:ctor Completed in 1ms
41 silly logfile error removing log file C:/Users/connor/AppData/Local/npm-cache/_logs/2023-04-07T19_47_21_746Z-debug-0.log [Error: EPERM: operation not permitted, unlink 'C:\Users\connor\AppData\Local\npm-cache\_logs\2023-04-07T19_47_21_746Z-debug-0.log'] {
41 silly logfile   errno: -4048,
41 silly logfile   code: 'EPERM',
41 silly logfile   syscall: 'unlink',
41 silly logfile   path: 'C:\\Users\\connor\\AppData\\Local\\npm-cache\\_logs\\2023-04-07T19_47_21_746Z-debug-0.log'
41 silly logfile }
42 silly logfile error removing log file C:/Users/connor/AppData/Local/npm-cache/_logs/2023-04-07T19_47_21_786Z-debug-0.log [Error: EPERM: operation not permitted, unlink 'C:\Users\connor\AppData\Local\npm-cache\_logs\2023-04-07T19_47_21_786Z-debug-0.log'] {
42 silly logfile   errno: -4048,
42 silly logfile   code: 'EPERM',
42 silly logfile   syscall: 'unlink',
42 silly logfile   path: 'C:\\Users\\connor\\AppData\\Local\\npm-cache\\_logs\\2023-04-07T19_47_21_786Z-debug-0.log'
42 silly logfile }
43 silly logfile done cleaning log files
44 http fetch GET 200 https://registry.npmjs.org/react 370ms (cache revalidated)
45 silly fetch manifest<EMAIL_ADDRESS>46 http fetch GET 200 https://registry.npmjs.org/@shopify%2fcheckout-ui-extensions-react 1398ms (cache revalidated)
47 silly fetch manifest<EMAIL_ADDRESS><18.0.0
48 timing idealTree Completed in 2117ms
49 timing command:install Completed in 2125ms
50 verbose stack Error: unable to resolve dependency tree
50 verbose stack     at [failPeerConflict] (C:\Program Files\nodejs\node_modules\npm\node_modules\@npmcli\arborist\lib\arborist\build-ideal-tree.js:1381:25)
50 verbose stack     at [loadPeerSet] (C:\Program Files\nodejs\node_modules\npm\node_modules\@npmcli\arborist\lib\arborist\build-ideal-tree.js:1347:34)
50 verbose stack     at async [buildDepStep] (C:\Program Files\nodejs\node_modules\npm\node_modules\@npmcli\arborist\lib\arborist\build-ideal-tree.js:930:11)
50 verbose stack     at async Arborist.buildIdealTree (C:\Program Files\nodejs\node_modules\npm\node_modules\@npmcli\arborist\lib\arborist\build-ideal-tree.js:211:7)
50 verbose stack     at async Promise.all (index 1)
50 verbose stack     at async Arborist.reify (C:\Program Files\nodejs\node_modules\npm\node_modules\@npmcli\arborist\lib\arborist\reify.js:153:5)
50 verbose stack     at async Install.exec (C:\Program Files\nodejs\node_modules\npm\lib\commands\install.js:145:5)
50 verbose stack     at async module.exports (C:\Program Files\nodejs\node_modules\npm\lib\cli.js:78:5)
51 verbose cwd S:\projects\testest\checkout-app-test
52 verbose Windows_NT 10.0.22621
53 verbose node v18.12.1
54 verbose npm  v8.19.2
55 error code ERESOLVE
56 error ERESOLVE unable to resolve dependency tree
57 error
58 error While resolving<EMAIL_ADDRESS>58 error Found<EMAIL_ADDRESS>58 error node_modules/react
58 error<EMAIL_ADDRESS>from the root project
58 error
58 error Could not resolve dependency:
58 error peer<EMAIL_ADDRESS><18.0.0" from<EMAIL_ADDRESS>58 error node_modules/@shopify/checkout-ui-extensions-react
58 error<EMAIL_ADDRESS>from the root project
58 error
58 error Fix the upstream dependency conflict, or retry
58 error this command with --force, or --legacy-peer-deps
58 error to accept an incorrect (and potentially broken) dependency resolution.
58 error
58 error See C:\Users\connor\AppData\Local\npm-cache\eresolve-report.txt for a full report.
59 verbose exit 1
60 timing npm Completed in 2279ms
61 verbose unfinished npm timer reify<PHONE_NUMBER>546
62 verbose unfinished npm timer reify:loadTrees<PHONE_NUMBER>551
63 verbose unfinished npm timer idealTree:buildDeps<PHONE_NUMBER>869
64 verbose unfinished npm timer idealTree:#root<PHONE_NUMBER>870
65 verbose code 1
66 error A complete log of this run can be found in:
66 error     C:\Users\connor\AppData\Local\npm-cache\_logs\2023-04-07T19_55_56_398Z-debug-0.log

Reproduction steps
npm init @shopify/app@latest
cd your-app
npm run shopify app generate extension
Operating System
Windows 11
Shopify CLI version (check your project's package.json if you're not sure)
3.45.0
Shell
bash
Node version (run node -v if you're not sure)
v18.12.1
What language and version are you using in your application?
Node

Hi @ConnorIngold . Thank for rising up this issue!. It seems that there is a bug as the cli-kit module has been upgraded to use Reacy 18. We’ll have to revisit that part of the graph to make React not a peer dependency
There is a workaround fixing the version using resolutions https://classic.yarnpkg.com/lang/en/docs/selective-version-resolutions/

Delete  node_modules , package-lock.json and pnpm-lock.yaml generated in previous dependencies installation (the ones that applies to your package manage setup)
Add this content to the root package.json

{
  "resolutions": {
    "react": "18.2.0"
  }
}

Run yarn install and once it finishes yarn shopify app generate extension again

Please let me know if this works for you. Anyway, we are working on a fix for this problem. Sorry for the inconveniences.

Hi @alvaro-shopify, No luck I'm afraid, but I am using NPM over Yarn, so that might have something to do with. Is 100% neccary to use yarn here? I would prefer not too but the requests from merchants for this work are coming in and I need to get to grips with it,  😅
Regardless thank you and I've pushed it up to a public repo:
https://github.com/ConnorIngold/shopify-checkout-extention-test

Hi @ConnorIngold. Sorry, I didn't notice that you were using npm. Could you run the command generate using the --fore parameter like this?
npm run shopify app generate extension --force

it should do the same effect as using resolutions with yarn

I got this error today too - is running this command with --force safe?

It should be, it's forcing to use the last React version which is mandatory for the last version of the CLI UI Kit.

TypeError: Cannot read properties of undefined (reading 'isBatchingLegacy')
at ensureRootIsScheduled (/Users/tanmay/Desktop/Apps/Shopify-App-Edit-Order/node_modules/ink/node_modules/react-reconciler/cjs/react-reconciler.development.js:17731:35)
at scheduleUpdateOnFiber (/Users/tanmay/Desktop/Apps/Shopify-App-Edit-Order/node_modules/ink/node_modules/react-reconciler/cjs/react-reconciler.development.js:17636:5)
at Object.updateContainer (/Users/tanmay/Desktop/Apps/Shopify-App-Edit-Order/node_modules/ink/node_modules/react-reconciler/cjs/react-reconciler.development.js:21022:5)
at Ink.render (file:///Users/tanmay/Desktop/Apps/Shopify-App-Edit-Order/node_modules/ink/build/ink.js:226:20)
at render (file:///Users/tanmay/Desktop/Apps/Shopify-App-Edit-Order/node_modules/ink/build/render.js:19:14)
at renderString (file:///Users/tanmay/Desktop/Apps/Shopify-App-Edit-Order/node_modules/@shopify/cli-kit/dist/private/node/ui.js:39:22)
at renderOnce (file:///Users/tanmay/Desktop/Apps/Shopify-App-Edit-Order/node_modules/@shopify/cli-kit/dist/private/node/ui.js:7:33)
at renderFatalError (file:///Users/tanmay/Desktop/Apps/Shopify-App-Edit-Order/node_modules/@shopify/cli-kit/dist/public/node/ui.js:207:12)
at handler (file:///Users/tanmay/Desktop/Apps/Shopify-App-Edit-Order/node_modules/@shopify/cli-kit/dist/public/node/error.js:113:5)
at file:///Users/tanmay/Desktop/Apps/Shopify-App-Edit-Order/node_modules/@shopify/cli-kit/dist/public/node/error-handler.js:27:20
/Users/tanmay/Desktop/Apps/Shopify-App-Edit-Order/node_modules/ink/node_modules/react-reconciler/cjs/react-reconciler.development.js:17731
if ( ReactCurrentActQueue$1.isBatchingLegacy !== null) {
^
TypeError: Cannot read properties of undefined (reading 'isBatchingLegacy')
at ensureRootIsScheduled (/Users/tanmay/Desktop/Apps/Shopify-App-Edit-Order/node_modules/ink/node_modules/react-reconciler/cjs/react-reconciler.development.js:17731:35)
at scheduleUpdateOnFiber (/Users/tanmay/Desktop/Apps/Shopify-App-Edit-Order/node_modules/ink/node_modules/react-reconciler/cjs/react-reconciler.development.js:17636:5)
at Object.updateContainer (/Users/tanmay/Desktop/Apps/Shopify-App-Edit-Order/node_modules/ink/node_modules/react-reconciler/cjs/react-reconciler.development.js:21022:5)
at Ink.unmount (file:///Users/tanmay/Desktop/Apps/Shopify-App-Edit-Order/node_modules/ink/build/ink.js:284:20)
at EventEmitter.emit (node:events:524:35)
at EventEmitter.emit (node:domain:489:12)
at emit (/Users/tanmay/Desktop/Apps/Shopify-App-Edit-Order/node_modules/signal-exit/index.js:105:13)
at process.processEmit [as emit] (/Users/tanmay/Desktop/Apps/Shopify-App-Edit-Order/node_modules/signal-exit/index.js:193:7)
at process.exit (node:internal/process/per_thread:209:15)
at process. (file:///Users/tanmay/Desktop/Apps/Shopify-App-Edit-Order/node_modules/@shopify/cli/dist/index.js:12:13)
Node.js v19.5.0
I get this error when I update my cli :(


Still not able to install Access database redistributable

I have 32 bit office on my machine, I have installed x86 POwer BI. I am tring to install 32 bit Access exe but getting an error " you have 64 bit office products on your machine".
Tried with 64bit Access bit exe , gave me an error " you have 32 bit office products on your machine".
tried passive way as well, even that didn't work out.
Please help

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

ID: 680485c7-5823-2c7f-df5d-b8c8a8205759
Version Independent ID: 5a217642-7723-6980-28ae-e26559c24201
Content: Resolve Access and .XLS import issues in Power BI Desktop - Power BI
Content Source: powerbi-docs/desktop-access-database-errors.md
Service: powerbi
Sub-service: powerbi-desktop
GitHub Login: @davidiseminger
Microsoft Alias: davidi

Hi, @ajityadav83 -- this sounds like a support issue. You can try the community to see if they have any suggestions: https://community.powerbi.com/.
If not, please contact https://powerbi.microsoft.com/en-us/support/
Hope that helps!


add INFO macro for easy exploration of complex data structures

I use specter to first "explore" complex data structures, before using specter to select relevant pieces.
I found it very useful in this context to create my own INFO macro,
which returns, depending on the current structure different information:

names of keys, if map
length of sequence, if sequential
a "shortened string", if string

Having this INFO macros at the end of the specter path guaranties as well that the returned data from select will not be too long to crash emacs /cider, if printed by accident.
It allows a very iterative way of exploring a data structure by adding :+1: keyword or "FIRST" to the path just before the INFO in order to easily dig into complex structures.
The code for this INFO macro itself is very short.
Maybe something like this could be added to specter.
(defn shorten-string [s]
  (if (> (count s) 1000)
    (str (.substring s 0 1000) " ...")
    s)
  )

(defn info [x]
  (cond (string? x)   (shorten-string x)
        (map? x) (str "keys: "(keys x))
        (sequential? x) (str "count: " (count x))
        true x))

(defnav INFO []
    (select* [this structure next-fn]
             (next-fn (info structure)))
  (transform* [this structure next-fn]
              nil))

Exploring/visualization of data structures is an interesting problem space that I think is best done as its own project


Is This Version or Modded version have backdoor , ip grabbing or malware ?

is this beatrun version doesn't have malware , ip grabbing or backdoors ?
( when i heard datae do that stuff. its scares me cuz i dont want let my lil'bro download his version)
(my lil'bro want it so much and im afraid this beatrun have backdoor installed)

this version is fully clear of any malware.

No, my fork or @jeffthekillerz's version doesn't contain anything like that because we are cool B) (not like datae)

Whats with (Delete or disable VManip (Base) that downloaded from Workshop!) ? Is it going track the modded beatrun ?

no, it's just an older version. vmaniprework in this repo is up-to-date as it was acquired from the official beatrun discord server and it was modified by us to work with beatrun well.

wanted to ask but didn't want to make a whole "issue" thing (because it really isn't). is there a reason that windows defender flags the downloaded ZIP as being/having a Trojan?

It's a false positive, ignore it. I think it doesn't likes the included .exe (Anims installer). Anims installer's source is included with the download, I just complied it so you don't have to


ci(phpstan): check @throws

Closes #1331

Please mention me when you are done.

Yup

@spawnia le mention

I would like to see the impact a change like https://github.com/webonyx/graphql-php/pull/1343 has on the code base with this static analysis active. Once we merge it into master, can you please update this branch with it so I can judge the impact. Please do not use force push for that, I would like to see the diff.
Side note, I am generally not too fond of force pushes in merge requests. It makes reviews harder for me.


Update Publisher in nzbget.nzbget to exactly match ARP value

Related to https://github.com/microsoft/winget-pkgs/pull/192344#discussion_r1848126017

cc @phnzb

Microsoft Reviewers: Open in CodeFlow

Also added post-install notes from https://github.com/nzbgetcom/nzbget/blob/develop/windows/pkgs-info.md#post-install into the manifest. These will be displayed to the user post install when WinGet finishes installation of the package

@mdanish-kh
Thank you for the quick fix. In the next version, I'll keep these changes


suggest chain (needed for testnets)

@pyramation was this feature implemented? for our chain (AssetMantle) the suggest chain is not getting automatically triggered in keplr extension. we are picking the details of the chain from chain-registry itself.


how to set big batch for transformer base model to accelerate training process

Hi!
I have 8 V100 with 32G RAM , and I want to make the best use of the RAM to reproduce Transformer base model in WMT16 EN-DE dataset.So I think I should try to increase mini-batch. Could someone please give some examples or command lines  for big batch? It seems the paper Scaling Neural Machine Translation didn't give model configuration for transformer base model. And any tricks to make the training process as fast as possible  ? Thx!

Facing the same question

@myleott 😊can you give some suggustions?

it's in the

def add_dataset_args(parser, train=False, gen=False)
section in the options.py file

You can set the --update-freq to delay gradient updates until you have processed more samples, thus increasing the batch size, or you can try to set --max-tokens to a larger number to pack the GPU. The batching is already pretty efficient to reduce the amount of padding.

@huihuifan But learning rate are closely correlated with batch. I am wondering what lr should I set if I set a larger batch

In some past issue posts the author gives some suggestions about LR and batchsize


Fix addition/deletion of namespaces and exception on startup

Fix exception in startup_fn() because of no namespace name in clustersecrets metadata.
Fix exception in on_field_data() when encountering a deleted namespace.
Fix cache update in on_field_data() so a newly added namespaces could get an up-to-date secret data.
Fix the addition of new namespaces to the list for synchronization.
Fix storing body in cache so the new namespaces could get the data.
Fix storing clustersecret's namespace name in cache in create_fn().
Update kopf to fix mutable default <class 'kopf._cogs.structs.diffs.Diff'> for field diff is not allowed: use default_factory with Python 3.11+.
Fixes https://github.com/zakkg3/ClusterSecret/issues/117
Fixes https://github.com/zakkg3/ClusterSecret/issues/116

I have to fix the e2e test and if that passes ill merge this. I guess in the next days I hope I find the time for this.
sorry for the delay.


Pod crashlooping can cause excessive full pushes from Pilot

When a service drops down to 0 endpoints, we remove all references to it. This was added in https://github.com/istio/istio/pull/11183 to prevent memory leaks. When a new service is created, a full push is triggered to every envoy.
These two together allow a pod that is crashlooping (going from 0 to 1 endpoints repeatedly) to trigger constant full pushes. In one case I saw roughly 20x pilot CPU usages from full pushes being triggered every 5s or so.
This is especially bad because a single pod crashing can cause a mesh-wide failure.
We should also be aware of the memory leak fixed in 11183 and make sure we do not repeat this

@nishanth-pinnapareddy /cc

@hzxuzhonghu fyi

I think we can fix it by  delete the associated key of  EndpointShardsByService only when svc deleted.

Or another option:
Seems it is viable to remove the below logic to trigger full push. I am not sure why it is needed.
	// Update the data structures for the service.
	// 1. Find the 'per service' data
	ep, f := s.EndpointShardsByService[serviceName]
	if !f {
		// This endpoint is for a service that was not previously loaded.
		// Return an error to force a full sync, which will also cause the
		// EndpointsShardsByService to be initialized with all services.
		ep = &EndpointShards{
			Shards:          map[string][]*model.IstioEndpoint{},
			ServiceAccounts: map[string]bool{},
		}
		s.EndpointShardsByService[serviceName] = ep
		if !internal {
			adsLog.Infof("Full push, new service %s", serviceName)
			requireFull = true
		}
	}

@costinm You first added it, any comments?

Closing as duplicate of https://github.com/istio/istio/issues/13822


How to set ROI?

I would like to use ROI to get a smaller focus portion of the image, in hope to has less data to pack and unpack, hence increase performance (lower latency). I see that is an available option on the camera message but how do I set this? And if i use this, would it affect the latency in the negative way since introduce extra process to pylon camera node?
Hope for some input. Thanks!

Same issue here
I'm trying with ex. camera.Width.SetValue(xxx) anda a get the message:
genicam.AccessException: Node is not writable. : AccessException thrown in node 'Width' while calling 'Width.SetValue()' (file 'IntegerT.h', line 77)
Works when I use AutoFunctionROIWidth node, but for me isn't seen correct.

You can set the ROI with the ~set_roi service:
rosservice call /pylon_camera_node/set_roi "target_roi: {x_offset: 1220, y_offset: 1000, height: 200, width: 400, do_rectify: false}


php  exec  apktool  without apk package out

Information

Apktool Version (2.1) -
**Operating System  windows
**my apk

i install a wamp server on window ,and use php script to call apktool to recompile my apk ,but the execution only generate build/class.dex ,without apk file ,and no errors , any idea please?
thanks very much

Not really enough details to go on here, sorry. You can try executing with -v to log more information.


Documentation - Fix typo in command

Summary

Documentation - Fix typo
Change webhhok to webhook

Fixes typo in https://github.com/mattermost/mattermost-plugin-jira/pull/851


Stack order is not maintained upon "Show open windows"

Pressing "Show the desktop" and then restoring windows with "Show open windows" can sometimes have them in the wrong order they were minimized in.

Fixed now https://github.com/DustinBrett/daedalOS/commit/46a73984bac0465b7cad33245e3615faf07d3e3f


API Change to disable floating labels.

Feature Request:
API Change to disable floating labels. This will aid developers who want to use the control but continue to use their standard labels.
<xfx:XfxEntry IsLabelFloating="False" />

Should default to true.
Android should be straight forward, just call HintEnabled=Element.IsLabelFloating
iOS will just hook into the animation trigger of the FloatedLabel.

This is done in a separate branch, forgot to setup tracking... whoops.


sync: internal/poll.TestSplicePipePool fails on 32-bit platforms after sync.Pool CL 303949

After CL 303949 , there are a few failures for internal/poll.TestSplicePipePool fails on 32-bit platforms, e.g.
https://build.golang.org/log/6063d1764ac5d5c3644528362f002fb0a8a685d6   386
https://build.golang.org/log/de4fd7cd548eb95d0c76efc7d72513bdecf389d4   ARM
https://build.golang.org/log/19f144a0780b4f29d1533079c9141022de5d5672   MIPS
And internal/poll.Splice does use sync.Pool. So this CL is probably related.
@dr2chase @kels-ng

See previously #45059 (CC @panjf2000 @ianlancetaylor @bradfitz).

This also seems to be affecting some TryBot platforms (such as linux-arm-aws):
https://storage.googleapis.com/go-build-log/ea802e88/linux-arm-aws_4aeefd09.log

Fixed by revert in CL 313630.


[prometheus-cloudwatch-exporter] Added securityContext configuration

What this PR does / why we need it:
Allow user to specify securityContext configuration onto the pods
Checklist

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md

@gianrubio, I just rebased, should be good to go 👍

/assign @nestorsalceda @torstenwalter

/lgtm


How to insert clear ad video in encrypted video

We have a business model for inserting advertisements.
We will insert an unencrypted advertisement slice into the encrypted positive at any time.
I thought I can do it like this:
#EXT-X-DISCONTINUITY
#EXT-X-MAP:URI="init.mp4"
#EXT-X-KEY:METHOD=NONE
#EXTINF:9.968,
video_00001.mp4
but I failed, and shaka player stopped after downloading audio.m3u8.
My playurl：
‘https://175883.oss-cn-beijing.aliyuncs.com/05007D00005EF9B92A8BB78000000066C57289-6C3A-4310-B5F7-E11B7ED0059B/05007D00005EF9B92A8BB78000000066C57289-6C3A-4310-B5F7-E11B7ED0059B_1.m3u8’
Can you please help me for what is wrong with it?
Or should I implement it in other ways?

We don't currently support FairPlay content.  See #150.

我们目前不支持FairPlay内容。参见＃150。
I use two '#EXT-X-KEY:' to support Widevine&FairPlay.
Do you mean that I need to delete FairPlay‘s ’'#EXT-X-KEY:'‘？

#EXT-X-KEY:METHOD=SAMPLE-AES,URI="data:text/plain;base64,AAAASnBzc2gAAAAA7e+LqXnWSs6jyCfc1R0h7QAAACoSEPMxe/pvakPomHaYYIjpZgQSEPMxe/pvakPomHaYYIjpZgRI88aJmwY=",KEYID=0xF3317BFA6F6A43E89876986088E96604,IV=0x3B8B98A2DD9A5CA663815DF9886E67F1,KEYFORMATVERSIONS="1",KEYFORMAT="urn:uuid:edef8ba9-79d6-4ace-a3c8-27dcd51d21ed"
#EXT-X-KEY:METHOD=SAMPLE-AES,URI="skd://f3317bfa6f6a43e89876986088e96604",KEYFORMATVERSIONS="1",KEYFORMAT="com.apple.streamingkeydelivery"

If you want to play Widevine HLS content, you need to disable src= mode so we play in MSE mode.  This is done with setting the streaming.useNativeHlsOnSafari configuration to false.  I don't think you need to remove the extra #EXT-X-KEY.
Unfortunately, this project uses v2.5.11 of Shaka Player, which doesn't support #EXT-X-DISCONTINUITY.  Once we update to v3.0 we should be able to play it.  You should try playing this content in the Shaka Player demo to make sure you can play it in v3.0.

If you want to play Widevine HLS content, you need to disable src= mode so we play in MSE mode. This is done with setting the streaming.useNativeHlsOnSafari configuration to false. I don't think you need to remove the extra #EXT-X-KEY.
Unfortunately, this project uses v2.5.11 of Shaka Player, which doesn't support #EXT-X-DISCONTINUITY. Once we update to v3.0 we should be able to play it. You should try playing this content in the Shaka Player demo to make sure you can play it in v3.0.

Thank you for your help!
I update to v3.0,then it success play.


I see public deployments from other projects in the Deployment view

It feels a bit odd that we see deployments from other projects in view even though they are public. What should the behavior be here?

Please specify better what your bug is. Are you refering to your projects/deployments view then its a bug. If you se other peoples models that are publicly available under root /deployments then it is as intended.

Right. I was meaning to attach a screenshot. Yes, the issue is that I see them in projects/deployments.

Well than its a bug!


clicking on facebook notification to renew token is broken

...due to facebook's X-Frame-Options. from the browser console:
GET https://apps.facebook.com/256884317673197//facebook/start [HTTP/2.0 200 OK 80ms]
POST https://brid.gy/facebook/start [HTTP/2.0 302 Found 528ms]
...
GET https://www.facebook.com/v2.6/dialog/oauth [HTTP/2.0 302 Found 79ms]
GET https://brid.gy/facebook/oauth_handler [HTTP/2.0 200 OK 1728ms]
Load denied by X-Frame-Options: https://www.facebook.com/ does not permit framing by https://apps.facebook.com/256884317673197//facebook/start?fb_source=notification.  (unknown)

docs for sending facebook API notifications here.
from what i can tell, click on the notification opens https://brid.gy/facebook/start in an iframe on a page with origin apps.facebook.com. bridgy starts the facebook oauth dance and redirects to https://www.facebook.com/v2.6/dialog/oauth, which is blocked by apps.facebook.com's X-Frame-Options.
bridgy does set X-Frame-Options (and Content-Security-Policy frame-ancestors itself. it includes www.facebook.com, but not apps.facebook.com. i don't think adding that would fix this. but i could try.
Content-Security-Policy: script-src https: localhost:8080 'unsafe-inline'; frame-ancestors 'self' https://www.facebook.com/; report-uri /csp-report;
X-Frame-Options: ALLOW-FROM https://www.facebook.com/

might be this open facebook bug? https://developers.facebook.com/bugs/531093333589159
this SO question suggests using target="_top" for the link, but that's not possible in a notification, so their setup must be different: http://stackoverflow.com/questions/14915152/loading-iframe-facebook-load-denied-by-x-frame-options
this seems related but not quite the same: http://stackoverflow.com/questions/38278750/top-level-redirect-at-client-side-facebook-canvas-x-frame-options-deny
thanks to @gRegorLove for reporting!

full headers for the www.facebook.com request that failed X-Frame-Options:
https://www.facebook.com/v2.6/dialog/oauth?scope=&client_id=256884317673197&redirect_uri=https://brid.gy/facebook/oauth_handler?state=%7B%22operation%22%3A%22add%22%7D&response_type=code

Request
===
Host: www.facebook.com
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.11; rv:50.0) Gecko/20100101 Firefox/50.0
Accept: text/html,application/xhtml+xml,application/xml;q=0.9,*/*;q=0.8
Accept-Language: en-US,en;q=0.5
Accept-Encoding: gzip, deflate, br
Referer: https://apps.facebook.com/
Cookie: ...
DNT: 1
Connection: keep-alive
Upgrade-Insecure-Requests: 1
Pragma: no-cache
Cache-Control: no-cache

Response
===
Cache-Control: private, no-cache, no-store, must-revalidate
Content-Length: 0
Content-Type: text/html
Date: Tue, 10 Jan 2017 15:50:33 GMT
Expires: Sat, 01 Jan 2000 00:00:00 GMT
Location: https://brid.gy/facebook/oauth_handler?state=%7B%22operation%22%3A%22add%22%7D&code=...
Pragma: no-cache
Set-Cookie: wd=deleted; expires=Thu, 01-Jan-1970 00:00:01 GMT; Max-Age=-1484063432; path=/; domain=.facebook.com
Strict-Transport-Security: max-age=15552000; preload
X-Content-Type-Options: nosniff
X-Firefox-Spdy: h2
X-Frame-Options: DENY
X-XSS-Protection: 0
content-security-policy: default-src * data: blob:;script-src *.facebook.com *.fbcdn.net *.facebook.net *.google-analytics.com *.virtualearth.net *.google.com <IP_ADDRESS>:* *.spotilocal.com:* 'unsafe-inline' 'unsafe-eval' fbstatic-a.akamaihd.net fbcdn-static-b-a.akamaihd.net *.atlassolutions.com blob: data: 'self';style-src data: 'unsafe-inline' *;connect-src *.facebook.com *.fbcdn.net *.facebook.net *.spotilocal.com:* *.akamaihd.net wss://*.facebook.com:* https://fb.scanandcleanlocal.com:* *.atlassolutions.com attachment.fbsbx.com ws://localhost:* blob: *.cdninstagram.com 'self';
facebook-api-version: v2.6
p3p: CP="Facebook does not have a P3P policy. Learn why here: http://fb.me/p3p"
public-key-pins-report-only: max-age=500; pin-sha256="..."
...; report-uri="http://reports.fb.com/hpkp/"
x-fb-debug: ...

facebook is indeed iframing bridgy when the user clicks on the notification. i tried dropping X-Frame-Options and Content-Security-Policy from bridgy's response headers, and that fixed it. so the browser error https://www.facebook.com/ does not permit framing by https://apps.facebook.com/... is misleading; it's actually bridgy that was blocking the framing. whee.
ideally though, i'd add frame busting to bridgy instead of dropping those headers.


Documentation on input formats for ReFrame

I was really excited to read your preprint using the ReFrame software you implemented. I was able to install the libraries and conda environments to run the example datasets described in the readme.
I want to run the software on my own data, but I am really confused about the formatting of the input data. Can you describe what each of the columns represent? Or provide more guidance how users can analyze their own data with ReFrame?

I'm sorry for the very late reply.
I have now added a readme-file to ReFrame to better describe the input format:
https://github.com/atb-data/neoantigen-landscape-msi/tree/master/ReFrame
Long story short - you need two types of files to apply ReFrame to your own data:

a reference peak-height file for normalisation reflist.xlsx.
your actual input xlsx files in the in directory.

Now, the actual data you need is the heights of the peaks from your fragment-size analysis for the main peak size (MP, usually the highest peak in a non-mutated reference-sample) and the secondary peaks at sizes MP - 4 (L4) to MP + 3 (R3). In each of your input files, you'll want to have a sheet named "Heights" (case sensitive) with the following format:

Tumor ID
Run ID
L4
L3
L2
L1
MP
R1
R2
R3

ignored
ignored
L4-size
L3-size
L2-size
L1-size
MP-size
R1-size
R2-size
R3-size

tumor-identifier
run-identifier
peak-height L4
peak-height L3
peak-height L2
peak-height L1
main-peak-height
peak-height R1
peak-height R2
peak-height R3

tumor-identifier
run-identifier
...
...
...
...
...
...
...
...

With one line of "tumor-identifier run-identifier ..." for each tumor sample you have. Each file in the in directory should contain the data for one marker of interest.
 That is, take the median peak heights for each marker of interest and compute the relative peak height as (peak-height) / (sum of peak-heights).
I hope this answers some of your questions. If anything else is unclear, I'll make sure to answer as soon as possible and add it to the readme.

Dear Michael and Alexej
Thank you for your response. The readme is very helpful and I am getting a
better understanding of the input format
I have a data set of 6 samples that were run on capillary electrophoresis.
3 of them are reference (wildtype) and 3 others have a cMS frameshift
(based on cell line data) I want to use ReFrame to confirm the presence of
the frameshifts.
I'm still confused how to get the relative peak heights. To get the
relative peak heights to populate reflist.xslx, does the sum of peak
heights  equal to  sum of peak heights L1-4,MP, R1-4 for each reference
sample?
relative peak height peak-height / sum of peak-heights
I am still a little confused on how to get the relative peak height for the
reflist.xslx.  To compute the sum of peak heights do I  add the peak
heights of  L1-R4,MP, R1-R4 for each reference sample? Then for 3 samples
that have a cMS, I just get the peak heights for the L1-4, MP, and R1-4
I was not able to get GeneMapper to work on my machine, but the equivalent
ThermoFisher cloud application MSA is able to export peak heights. If you
can confirm my understanding of how to compute relative peak heights, then
I should be able to prepare the input files for my own data.
Again, thank you so much for your responsive help and updated Readme on
your git repo for ReFrame.
Sincerely,
Amit
On Thu, Jul 16, 2020 at 3:39 AM Michael Jendrusch<EMAIL_ADDRESS>wrote:

I'm sorry for the very late reply.
I have now added a readme-file to ReFrame to better describe the input
format:
https://github.com/atb-data/neoantigen-landscape-msi/tree/master/ReFrame
Long story short - you need two types of files to apply ReFrame to your
own data:

a reference peak-height file for normalisation reflist.xlsx.
your actual input xlsx files in the in directory.

Now, the actual data you need is the heights of the peaks from your
fragment-size analysis for the main peak size (MP, usually the highest
peak in a non-mutated reference-sample) and the secondary peaks at sizes MP

4 (L4) to MP + 3 (R3). In each of your input files, you'll want to have a
sheet named "Heights" (case sensitive) with the following format:
Tumor ID Run ID L4 L3 L2 L1 MP R1 R2 R3
ignored ignored L4-size L3-size L2-size L1-size MP-size R1-size
R2-size R3-size
tumor-identifier run-identifier peak-height L4 peak-height L3 peak-height
L2 peak-height L1 main-peak-height peak-height R1 peak-height R2 peak-height
R3
tumor-identifier run-identifier ... ... ... ... ... ... ... ...

With one line of "tumor-identifier run-identifier ..." for each tumor
sample you have. Each file in the in directory should contain the data
for one marker of interest.
 That is, take the median peak heights for each marker
of interest and compute the relative peak height as (peak-height) / (sum of
peak-heights).
I hope this answers some of your questions. If anything else is unclear,
I'll make sure to answer as soon as possible and add it to the readme.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/atb-data/neoantigen-landscape-msi/issues/1#issuecomment-659328309,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADPQLQ6YP3YI4YS4UOKRM3R33KHTANCNFSM4NKPMMVQ
.

Right, so for the reflist, you take your reference samples (non-mutated) and for each marker, you then take the median of peak heights for all reference samples for that marker. Then, you should have median peak heights L1-4 MP R1-4 for each marker.
To get the relative peak heights, you sum over these peak heights for each marker and divide the peak heights by the sum. So now, for each marker, you should have relative peak heights L1-4 MP R1-4 which sum to 1.
I'll update the README with a better description of how to set up the reflist.xlsx.


trace/http: Add semantic convention for total request and response length

The current http.request_content_length  and http.response_content_length  are specified to be the length of the body excluding headers. I'd like to add an attribute for the total message size.
For names, I think http.request_length and http.response_length make the most sense.
Rationale:

the seemingly related metrics http.server.request.size and http.server.response.size are in fact measure of total bytes sent
some systems only provide the total HTTP message length, eg Google Cloud Load Balancer logs and AWS ALB logs; these are logs, but we process those logs into spans

Just caught up on the effect of open-telemetry/opentelemetry-specification#3355 which. The ECS names for the fields I'd like are:

http.request.bytes
http.response.bytes

By analogy with the renamings introduced in that PR, the new attributes should be

http.request.size
http.response.size

just adding the same heads up here from https://github.com/open-telemetry/semantic-conventions/pull/84#issuecomment-1579282520 (for anyone else viewing the issue):

the HTTP semantic convention WG isn't considering any additions until the HTTP semantic conventions have at least been marked Frozen, so it may be a few weeks until we get to reviewing this proposal


Impement EWA Lanczos

I would be nice to have EWA Lanczos for upscaling.

Probably not before a while, because it would need a design different to other “separable” scalers.


./scripts/build-xserver issue

It's me again...
I'm now having issue with the build XServer script.
Can you tell me if it's working on your side ?
[gallit@archi jetson]$ ./scripts/build-xserver
perl: warning: Setting locale failed.
perl: warning: Please check that your locale settings:
LANGUAGE = (unset),
LC_ALL = (unset),
LC_PAPER = "en_US.UTF-8",
LC_MONETARY = "en_US.UTF-8",
LC_NUMERIC = "en_US.UTF-8",
LC_MEASUREMENT = "en_US.UTF-8",
LC_TIME = "en_US.UTF-8",
LANG = "en_US"
are supported and installed on your system.
perl: warning: Falling back to the standard locale ("C").
autoreconf: Entering directory `.'
autoreconf: configure.ac: not using Gettext
autoreconf: running: aclocal --force -I m4
perl: warning: Setting locale failed.
perl: warning: Please check that your locale settings:
LANGUAGE = (unset),
LC_ALL = (unset),
LC_PAPER = "en_US.UTF-8",
LC_MONETARY = "en_US.UTF-8",
LC_NUMERIC = "en_US.UTF-8",
LC_MEASUREMENT = "en_US.UTF-8",
LC_TIME = "en_US.UTF-8",
LANG = "en_US"
are supported and installed on your system.
perl: warning: Falling back to the standard locale ("C").
perl: warning: Setting locale failed.
perl: warning: Please check that your locale settings:
LANGUAGE = (unset),
LC_ALL = (unset),
LC_PAPER = "en_US.UTF-8",
LC_MONETARY = "en_US.UTF-8",
LC_NUMERIC = "en_US.UTF-8",
LC_MEASUREMENT = "en_US.UTF-8",
LC_TIME = "en_US.UTF-8",
LANG = "en_US"
are supported and installed on your system.
perl: warning: Falling back to the standard locale ("C").
perl: warning: Setting locale failed.
perl: warning: Please check that your locale settings:
LANGUAGE = (unset),
LC_ALL = (unset),
LC_PAPER = "en_US.UTF-8",
LC_MONETARY = "en_US.UTF-8",
LC_NUMERIC = "en_US.UTF-8",
LC_MEASUREMENT = "en_US.UTF-8",
LC_TIME = "en_US.UTF-8",
LANG = "en_US"
are supported and installed on your system.
perl: warning: Falling back to the standard locale ("C").
autoreconf: configure.ac: tracing
autoreconf: running: libtoolize --copy --force
libtoolize: putting auxiliary files in '.'.
libtoolize: copying file './ltmain.sh'
libtoolize: putting macros in AC_CONFIG_MACRO_DIRS, 'm4'.
libtoolize: copying file 'm4/libtool.m4'
libtoolize: copying file 'm4/ltoptions.m4'
libtoolize: copying file 'm4/ltsugar.m4'
libtoolize: copying file 'm4/ltversion.m4'
libtoolize: copying file 'm4/lt~obsolete.m4'
perl: warning: Setting locale failed.
perl: warning: Please check that your locale settings:
LANGUAGE = (unset),
LC_ALL = (unset),
LC_PAPER = "en_US.UTF-8",
LC_MONETARY = "en_US.UTF-8",
LC_NUMERIC = "en_US.UTF-8",
LC_MEASUREMENT = "en_US.UTF-8",
LC_TIME = "en_US.UTF-8",
LANG = "en_US"
are supported and installed on your system.
perl: warning: Falling back to the standard locale ("C").
perl: warning: Setting locale failed.
perl: warning: Please check that your locale settings:
LANGUAGE = (unset),
LC_ALL = (unset),
LC_PAPER = "en_US.UTF-8",
LC_MONETARY = "en_US.UTF-8",
LC_NUMERIC = "en_US.UTF-8",
LC_MEASUREMENT = "en_US.UTF-8",
LC_TIME = "en_US.UTF-8",
LANG = "en_US"
are supported and installed on your system.
perl: warning: Falling back to the standard locale ("C").
perl: warning: Setting locale failed.
perl: warning: Please check that your locale settings:
LANGUAGE = (unset),
LC_ALL = (unset),
LC_PAPER = "en_US.UTF-8",
LC_MONETARY = "en_US.UTF-8",
LC_NUMERIC = "en_US.UTF-8",
LC_MEASUREMENT = "en_US.UTF-8",
LC_TIME = "en_US.UTF-8",
LANG = "en_US"
are supported and installed on your system.
perl: warning: Falling back to the standard locale ("C").
autoreconf: running: /usr/bin/autoconf --force
configure.ac:660: error: possibly undefined macro: XTRANS_CONNECTION_FLAGS
If this token and others are legitimate, please use m4_pattern_allow.
See the Autoconf documentation.
autoreconf: /usr/bin/autoconf failed with exit status: 1
[gallit@archi jetson]$

It is working for me, yes. What is your host distribution?

Linux archi 4.2.5-1-ARCH #1 SMP PREEMPT Tue Oct 27 08:13:28 CET 2015 x86_64 GNU/Linux
Should be a 32 bits host ?

x86_64 -> it's 64 bits
And what is your distribution?

ArchLinux

Uh. Same as me, that's interesting. Let me look in more detail.

Sorry for my english (maybe we could talk in french...).
I know it's a 64 bits, i wondered if 32 bits was a required.
OK, thanks for the help.

Can you try installing the "xtrans" package on your host and try again after removing out/build/arm/ArchLinuxArm/xserver ? Normally this package is installed on the guest FS and it should be enough, but maybe the script is not looking at the right place...

Ok, I managed to reproduce your error by removing the xtrans package from my host, so installing it on your end should work as a workaround. I don't think it should be needed though, so keeping this issue open until I get to the bottom of it.

Found it - ACLOCAL_PATH needed to be defined. I have updated the scripts, please repo sync and re-try to build xserver, it should pass without any problem now. Thanks for reporting this!

Ok, thanks for the support.


Organisation Class enum values need updated

Current Behavior
enum values for the Organisation Class property are:

DEMO
TRIAL
STARTER
STANDARD
PREMIUM
PREMIUM_20
PREMIUM_50
PREMIUM_100
LEDGER
GST_CASHBOOK
NON_GST_CASHBOOK
ULTIMATE
LITE

Expected Behavior
The new classes need added fairly urgently since they drop in three weeks

IGNITE
GROW
COMPREHENSIVE

Possible Solution
Add them!
Context
The PHP SDK will start throwing exceptions when it encounters any organisation with the new types. The spec needs updated so an update can happen to the SDK.
I'm happy to do a PR, but I'm not sure of the process here - presumably this is fed from somewhere else.
The API reference at https://developer.xero.com/documentation/api/accounting/types/#organisation also hasn't been updated to include these new values so I'd be hesitant to do so without verifying first.

This has now been resolved - see https://github.com/XeroAPI/Xero-OpenAPI/pull/594


Missing folders and files in the project

Hi,
I have tried to run 3d viewer in my local. I ensured my browser supports WebGL to run this package. It seems assets/ folder and some important files like data.json are missing here.

So it's not running the way I expected. Is there any workaround or better way to run this package? Because I need these sample files to analyze this 3d-viewer and implement in my project. Thanks in advance.

Hello,
You are correct that the data model is not provided in this repository. You can see the software in action with panorama pictures taken and provided by our project partner bup-nbg at: https://50-jahre-informatik.cs.fau.de/
If you would like to create your own data model from panorama pictures and global coordinates, check out the wiki page to see what the data model needs to look like.
Alternatively you can go back to some earlier releases, where we included a minimal example in a local assets folder.


TLS handshake cert error

I'm implementing a protocol using this and get this error in the handshake to an external server.
javax.net.ssl.SSLHandshakeException: QUICHE_ERR_TLS_FAIL: error:1000007e:SSL routines:OPENSSL_internal:CERT_CB_ERROR
The certificates are self signed, with an extension, and I have a custom trustmanager which is correctly verifying the certs and extension.
The same cert code works fine with TLS over TCP and netty. I've found this doc on the openssl error: https://www.openssl.org/docs/man1.0.2/man3/SSL_CTX_set_cert_cb.html which suggest some kind of error in the cert callback.
Any advice would be much appreciated, thank you for the awesome project.

Upon digging deeper, that link above says the callback should return 1 on success and 0 on error when verifying certs. But stepping through in the debugger I see BoringSSLCertificateVerifyCallback.java is returning BoringSSL.X509_V_OK, which is 0.
Am I barking up the wrong tree?

can you share your impl ? Also are you sure its not throwing an exception etc ?

Sure, this is the relevant bit of code (Kotlin):
https://github.com/Peergos/jvm-libp2p/blob/feat/quic/src/main/kotlin/io/libp2p/transport/quic/QuicTransport.kt#L158
The TrustManager implementation is here (I checked it's returning fine without error):
https://github.com/Peergos/jvm-libp2p/blob/feat/quic/src/main/kotlin/io/libp2p/security/tls/TLSSecureChannel.kt#L193

There are examples of the kind of certs it's receiving here: It is always a single self signed cert with an extension:
https://github.com/Peergos/jvm-libp2p/blob/feat/quic/src/test/kotlin/io/libp2p/security/tls/CertificatesTest.kt

I've pushed a minimalish test that reproduces it here:
https://github.com/Peergos/jvm-libp2p/blob/feat/quic/src/test/java/io/libp2p/core/QuicPingTestJava.java#L17
This test assumes the install-run-ipfs.sh in root has been run first to install and run the (Golang) server.

I've implemented the server side as well now, and got a pure java test that shows the same error, so no need for the external Golang server.  Thank you for any suggestions.
This is the java client -> java server test:
https://github.com/Peergos/jvm-libp2p/blob/feat/quic/src/test/java/io/libp2p/core/QuicServerTestJava.java#L15

I've tracked this down to us using Ed25519 certificates, so it was Boringssl that was throwing the exception because of an unsupported key type (BoringSSLCertificateCallback line 161). It seems like they don't support Ed25519 certificates:
https://github.com/cloudflare/boring/issues/113
https://github.com/cloudflare/quiche/issues/1482

So I guess can close this one ?

The 2nd linked issue suggests it's possible to add ed25519 to the boringssl config in the signature algorithms list. It would be great to enable it by default if possible.

@ianopolous I wonder if its a good idea to enable it by default. I guess we could allow to enable it tho .

@ianopolous I am mostly "caution" here as BoringSSL doesn't do it by default. Usually these people have a good reason for doing so :)

Sure, it's a good question. Just having the option at all would be great.

/cc @davidben ... Any thoughts ?

Ed25519 isn't enabled by default for TLS in BoringSSL. At this layer of the stack, introducing key types has a very long runway since it goes through your CAs, etc. Ed25519 is nicer[*] than ECDSA P-256, but it's not enough of an improvement to justify the cost of having too many options here.
Also, it's 2023. Putting a lot of effort into a new classical algorithm is kinda silly, when we'll need to replace it all with postquantum algorithms anyway.
If Netty really wants to enable it, the feature does exist in the library. But doing so will diverge from our defaults and what web browsers do. (In hindsight, I probably shouldn't have wasted time wiring it up to the TLS library, but, ah well, it's there now. 😄)
[*] Broadly. The cofactors are a nuisance, and some corners of verification were hideously underspecified, which left quite a mess.

Thanks for your thoughts @davidben Our use case is the ipfs and libp2p ecosystem, which thankfully doesn't depend on CAs or web browsers.
@normanmaurer Having the option at all to enable it would be great.

@ianopolous let me look into it

Hello @normanmaurer. Any updates on allowing ed25591 certificates?
I also have a use case that requires it.

@jaymansfield @ianopolous I think you should be able to enable these as we just merged these two PRs:
https://github.com/netty/netty-incubator-codec-quic/pull/754
https://github.com/netty/netty-incubator-codec-quic/pull/755
A release with these included will be cut this week

This was released as of today


Some supplements and fixes in CodeWriter

Add for del expression.
del a
del a[0]
del a, b, c

Add for RaiseStatNode node.
raise RuntimeError
raise RuntimeError('detail messages')
raise RuntimeError('detail messages') from exc

Fix visit_TryExceptStatNode visit_ExceptClauseNode visit_TryFinallyStatNode
try:
    pass
except:
    pass
except IOError as e:
    pass
except RuntimeError as e:
    pass
except (IOError, RuntimeError) as e:
    pass
else:
    pass
finally:
    pass

Add for AssertStatNode node.
assert a == 10
assert a == 10, "detail messages"

Add for GlobalNode and 'NonlocalNode' node.
global a, b
nonlocal a, b

Add for visit_LambdaNode
lambda x : x + 2

Add for visit_CascadedCmpNode
>>> 5 < 6 <= 7 > 4 >= 3 > 2 != 1
>>> True

Fix LinesResult , used byremove method, which use an undefined method named endswith.

@scoder Any comments?

Hi @scoder, @da-woods,
This pr is a subsequent fixes for the cython's CodeWriter (see also #3514 and #3795) and I want to know your comment on this PR.
BTW, would you mind to cut a new release (maybe in alpha channel, e.g., 3.0a7) after this PR get merged? Otherwise we need to include a set of monkey patches to fixes those issues when deliver our package to pypi. Seems that cython doesn't have any newer release since Aug.

Thanks @da-woods, but distributing Cythonized .c files on pypi doesn't in our case, we distribute a SDK, which, could translate some user-defined functions in python syntax to a decorated cython ast, then build it using cython, thus we need to cython's CodeWriter works as expected.

@scoder @da-woods Any other comments ?

@lidongze0629 Could you please rebase to master, and take a look at the current failures ?

@lidongze0629 Could you please rebase to master, and take a look at the current failures ?

Thanks, I will take look soon.

Rebase to the master and passed CI.

@scoder CI turns green after rebasing, may we consider to review and land this pr now?

This comment of mine is still open:

I think the ExpressionWriter should simply use a LinesResult instead of a plain string. It doesn't need it, but the subclasses do.

Is it clear what should be done? Or is there anything I could clarify or help with?

Is it clear what should be done? Or is there anything I could clarify or help with?

Thanks, scoder, the previous commit about using LinesResult instead of a plain string introduced some test failures
I need to take a closer look at how to use it.


feat: allow the hash param to be a string or a boolean.

Since the version 1.5.0, vue-mapbox allow the hash param to be a boolean or a string.
Same code as https://github.com/soal/vue-mapbox/pull/155

Hey @francois2metz,
Thanks for this PR, but you need to rebase and reword your commit as this repo follows the commitlint convention.
$ git rebase -i HEAD~1

Then in the vim editor, reword
- Allow the hash param to be a string or a boolean.
+ feat: allow the hash param to be string or boolean

Then push it to your fork
$ git push --force

LGTM


Error while installing

Maybe it is just me but I don't exactly understand step 3. Is it a case that this is a script that needs to be run? This is what I get when trying to run it. If that is not the correct way to do it please let me know.
./zsh: line 2: setopt: command not found
./zsh: line 3: syntax error near unexpected token `('
./zsh: line 3: `for rcfile in "${ZDOTDIR:-$HOME}"/.zprezto/runcoms/^README.md(.N); do'

Are you executing the 3rd step in zsh?

I dinally figured out to do that but now when running zsh after making a .zshrc with the info from step 3 I get this ln: /Users/username/.zshrc: File exists

I have fixed the issue. I had to mauanly add a symlink to .zprezto/init.zsh. I found the answer here.


Aurora process priority

What are you experiencing an issue with?

[x] Aurora
[ ] Aurora supported game, (Fill in the game)
[ ] Aurora Wrapper
[ ] Aurora Profile
[ ] Aurora supported device, (Fill in problematic devices)
[ ] Aurora Scripts
[ ] Aurora Updater
[x] Aurora Lighting issue
[ ] Other, (Fill in another issue here)

What kind of an issue is it?

[ ] Crash
[x] Performance problems
[x] Visual bug
[ ] Localization issue
[x] Enhancement suggestion
[ ] Device issue
[ ] Other, (Fill in another type of issue here)

I've noticed that recently (don't know from which version exactly) Aurora started to run with lowered process priority and this is causes visual effects on keyboard to lag under high load.
I considering the keyboard illumination as an GUI and when it lags I feel uncomfortable. It's like when game freezes, everyone hates it but now I have an freezing keyboard also.
As a developer I am a heavy PC user. Running Visual Studio, Aurora, Chrome with tons of tabs opened and a bunch of messengers make my weak CPU to be on high load very often.
My scripts provides a lot of animations, especially on high CPU load so this lags are very noticeable and annoying me. In worst cases I can see illumination hangs for several seconds.
I understand that this probably was made to not steal cycles from games, but if user has a game that is grabbing all his CPU he may be considering ~buying a new one~ setting high priority for the game process.
Of course I've easily workaround this for myself by setting permanent process priority for Aurora, but what I am talking here is the best default behavior. Personally I think Aurora should run with normal priority (or even higher like a media process). As another approach this can be leaved to user choice by providing a corresponding setting.
What you think about it?

I feel as though we would want to try and avoid having Aurora set at high priority at all times in case there is a scenario where it could be reducing the performance of the game being played. It depends on a person to person basis so it would probably make sense to ensure that it has at least a normal priority at all times and maybe have an option to make it run at a higher priority.

Yeah that is probably the best approach. Setting high priority is good in theory, cause I believe GUI should be responsive and fluid, but in reality almost no one GUI uses high priority, sadly even games. So we also shouldn't break this bad tradition to not affect other apps.

I've added an option to run Aurora at high priority which is disabled by default


[FORMATTING] Handle Polymorphic Table Functions formatting in Trino

Input data
Which SQL and options did you provide as input?
SELECT
  *
FROM
  TABLE (
    postgresql.system.query (query => 'SELECT * FROM tiny.orders LIMIT 100')
  );

Expected Output
SELECT
  *
FROM
  TABLE (
    postgresql.system.query (query => 'SELECT * FROM tiny.orders LIMIT 100')
  );

Actual Output
SELECT
  *
FROM
  TABLE (
    postgresql.system.query (query = > 'SELECT * FROM tiny.orders LIMIT 100')
  );

Usage

How are you calling / using the library?
format(sql, { language: "trino", linesBetweenQueries: 1, tabWidth: 2 });
What SQL language(s) does this apply to?
Trino
Which SQL Formatter version are you using?
11.0.2

More about PTF - https://trino.io/docs/current/functions/table.html

Thanks for reporting.
Should be a simple fix. Simply need to add support of the => operator to Trino.

This is now fixed in 12.0.0-beta.3 release.

Thank you


Merge Join team Sample

Merging sample.

@a1mery Thank you so much for the sample. If you are interested in showing this on one of the community calls as a demo please let @PopWarner or I know and we can get you scheduled.

Thanks @dcashpeterson for reviewing my PR 👍


Instructions for Tiny Core Linux

I have tried the commands below but it doesn't seem to be able to progress beyond stage2 before running out of space. There is no boot partition on Tiny Core Linux, also.
tce-load -wi coreutils grub2-multi bash git squashfs-tools
git clone https://github.com/jeaye/nixos-in-place
sudo su
./install -g /dev/shm -r /dev/sda1 -w /mnt/sda1/tmp

Looks like it runs out of space because it writes to /nix, however in Tiny Core Linux the root filesystem is in RAM so it's limited. Perhaps it should be configurable where to put /nix?


feat!: pass base asset ID for withdrawals

Closes #2394
Breaking Changes:

assembleTransactionSummary, getOperations and getWithdrawFromFuelOperations now all require passing an assetId

Won't be merging this until @LuizAsFight has had a change to review


BitDefender marks Xpipe as a virus when attempting to launch a WSL shell

BitDefender intercepted my attempt to launch a shell through Xpipe and marked the attempt as an infestation.

I know that Windows SmartScreen marks the installation as dangerous because of the lack of an EV certificate, but the BitDefender detection is worrying. Is this a known issue? What is the reason for the detection?

I'm actually not sure what the reason is for the detection. I guess launching shell processes and executing commands in them is always treated as suspicious activity by some antivirus programs. Maybe in the future, when I manage to sign it with an EV certificate, some antivirus programs will look at that more leniently.
As part of the CI pipeline, all artifacts of xpipe are uploaded to virustotal.com and there haven't been any issues. If you want to make sure, you can upload the release you downloaded yourself there, it should instantly show results as all files have been already analyzed beforehand.
If I install bitdefender myself, is there any way to get some details about the reason of the detection?

Thanks for the quick response, @crschnick. It's not unheard of for these antivirus programs, especially the ones that use heuristics to flag legitimate programs as suspicious, so that might be what's happening.
You should be able to see the application being flagged if you install BitDefender, please enable the "Advanced Threat Defense" option if you do. I got the detection when I was trying to scan my local WSL connections.
I've also filed a false positive with BitDefender, I'll update this issue if they respond.

For reference, this is what the notification log looks like:

Almost seems like it's tagging cmd.exe as the malicious application!

Haha, let's hope that it doesn't isolate or delete cmd.exe here. I guess technically cmd.exe is the one making all the suspicious calls. XPipe doesn't execute anything except the shell process itself, all execution is done by feeding the commands into cmd.exe

Haha, yeah, I would have bigger problems if it blocked cmd.exe!
This is what Xpiped shows when it is blocked:

So I looked at this in more detail there's nothing I can do from my end except reporting false positives to vendors in case this becomes a more widespread problem.
I updated the security page to include a note on this issue: https://github.com/xpipe-io/xpipe/commit/50e3dffd9b94659cb5b618e578915ff07821cd55

Fair enough, hopefully BitDefender updates their detection library after investigating my false positive report. I think I should be able to add Xpipe to the exceptions list until then.
Thanks for looking into this!

Oh my god it actually happened for me, bitdefender quarantined cmd.exe while testing.

I can't run cmd anymore, I have to see I can restore that. I got reports from other people as well that bit defender did not like xpipe, so I thought why not think about a better solution to this.

From what I have seen so far, I would never trust Bitdefender to do anything right with detection rules that bad.

One thing I could implement is checking whether bitdefender is installed on your system prior to running any shell process. That way I have a chance to display a warning about rough times ahead in the application itself, not in the documentation.

I have the same issue today with MCAfee

@piperoc Do you have exactly the same issue, i.e. it flags it as malware because it opens a shell program or immediately on startup?

@crschnick  the error looks different, but also I was doing something different:
Launched XPipe --> MCAfee complained about (but I couldn't catch the offending exe, and they don't have log files by default).
I disabled MCAfee but still see the error.
Chances are that some files got quarantined. I will try to uninstall/reinstall when I find some time and try to replicate the behavior because now MCafee does not complain anymore but XPipe does not work. Thanks

I might consider looking into properly signing the xpipe executables with an EV certificate assuming that this would fix the issue. (I don't think it will also block executables from recognized publishers)
However this would be a big commitment as you are required to do this as a formal company, so I would have to create one first

How would something like this look to you?

I think that's a reasonable warning 👍 if you'd like, you could also link to the VirusTotal results page.

I might consider looking into properly signing the xpipe executables with an EV certificate assuming that this would fix the issue. (I don't think it will also block executables from recognized publishers)
However this would be a big commitment as you are required to do this as a formal company, so I would have to create one first

That's unfortunate :( seems like MS and co are pushing for validation as a means of coercement into their paid programs.

I think that's a reasonable warning 👍 if you'd like, you could also link to the VirusTotal results page.

The CI workflow action actually gives me the Virustotal links for all analyzed files, however I'm not sure where to put and link them properly.
Also, I think that this would not really be convincing as you would have to manually check whether the linked analysis actually corresponds to the file you downloaded anyway. And the best way of doing that is uploading the files manually.

Ah ok. Might be fine just mentioning it, although people might ask for the link. I agree that it's not particularly convincing for the reasons you mentioned 🤷♂
The warning and popup LGTM! 👍

You can test out this feature in the new staging test version: https://github.com/xpipe-io/xpipe_staging/releases/tag/1.5.0
Essentially, it now checks whether a problematic antivirus is installed and shows this message on first startup. So if everything works correctly, you should see this warning message when bitdefender is installed.

Alright this warning is now live in the latest release. If you're interested in how it looks like for you, you can go to the settings menu under troubleshoot -> Delete caches and restart. This is necessary as it only will show on first startup.

Closing this as I think the solution to this is satisfactory


[OperationalInsights] Fix DateTimeOffset format string

Resolve #33536, #33571
The generated code should be property.Value.GetDateTimeOffset("O") instead of property.Value.GetDateTimeOffset().
The former method will call an extension method of JsonElement written by us to treat the JsonElement as Int64 (Unix Timestamp) or string and the string case can support multiple formats despite the "O" format (ISO8601) being passed in. See code.
The latter method calls the original GetDateTimeOffset() of JsonElement and seems to only support ISO8601 format.
Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

API change check
API changes are not detected in this pull request.


Clarification on parameters

--fragment-size (Single end reads only) Size of the sequenced fragment. The center of the the fragment will be taken as half the fragment size. Default 150

Would it be easier to pass the actual fragment size as IP'ed and seen on a gel/TapeStation?
--effective-genome-fraction

When --genome is passed, this does not seem set by default. However, in e.g. MACS2, passing a genome uses automatically the effective genome fraction.
--drop-duplicates

The flag name seems to suggest that duplicates are dropped when the flag is set, however the parameter seems to do the opposite of what the name suggests
Keep reads mapping to the same position on the same strand within a library. Default is to remove all but the first duplicate.

Thanks.
For 1 I do not know what you mean, embarrassingly. I am not a wet-lab biologist though.

When using --genome the egf should be automatically selected. This is a bug then.

This is a bug in the docs, not the tool/flag.

I'll look into all of these tomorrow :) Thanks for reporting.

With the latest merge, the two last issues should have been fixed. I think I undrestand what you mean with the first.
                        (Single end reads only) Size of the
                        fragment. The center of the the fragment will be taken
                        as half the fragment size. Default 150.

Is correct no? Just remove sequenced, which is wrong.

Thanks, yes that sounds more correct. Residual question: what does this mean?
The center of the the fragment will be taken as half the fragment size

Hmm. Should probably mean "each read is extended half the fragment size from the 5' end" :)
Thanks for bothering to ask such questions :)

I would recommend you dl the newest version from PyPI (pip install epic2 == 0.0.3) and redo your analyses. I have implemented several bugfixes :) Sorry for the inconvenience.


[rospack] Error: package 'cartographer_rosbag_validate not found

I'm getting this error when I try to validate my .bag file. My bagfile is outside of catkin_ws. How do I fix this problem? Do I need to move some folders around?

jeahn@jeahn-VirtualBox:~$ rosrun cartographer_rosbag_validate -bag_filename 2024-06-20-11-57-44.bag [rospack] Error: package 'cartographer_rosbag_validate' not found jeahn@jeahn-VirtualBox:~$ rosrun cartographer_ros cartographer_rosbag_validate -bag_filename 2024-06-20-11-57-44.bag [rospack] Error: package 'cartographer_ros' not found jeahn@jeahn-VirtualBox:~$

Go to the installed_isolated/bin directory to find it. Then run it like its a normal executable. No need to use rosrun.

Thanks! It worked


Namespaces On Same Line As PHP Opening Tag Are Prefixed With A New Line

Since #3083 (v2.7.2) was merged namespaces that are defined on the same line as the opening PHP tag are prefixed with a newline when using the no_blank_lines_before_namespace rule.
For example the following:
<?php namespace Some\Name\Space;

Is transformed into:
<?php
namespace Some\Name\Space;

Thanks @SpacePossum !

you're welcome :)  thanks!


I have updated the heading of "EMERGENCY CASE"

I have proposed the change from "EMERGENCY CASE" TO "EMERGENCY SERVICES"

User Proper Character count. You may change this and try again if you'd like.


Adding a support for supplemental XML

renamed the supplement to supplemental in XML definition
added a support for unwrapping the data
increased version by one

This looks fine. The only reservations I have are that

I don't have a clear idea of what the supplemental field is for. It would be useful to note this somewhere
I can only merge pull requests by inspection, since there is no testing framework for these tools

Good point. Where should I describe it?
The idea is to add one field that can be used for anything that is going to be needed. For example, this year, we have "source" and "clean source", so the supplemental with type "clean_source" is used to store this data.
And I wanted to make it general to allow any different type of data to be added without always updating the XML template.

Maybe you could add this description to the README? You could add a heading like "change log" or "updates".
It's the only "documentation" that there is so far.


clarify for host scenario only

This tutorial hasn't been updated for standalone yet, so add content to clarify that it uses a host program.

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology.

Article
Acrolinxscore
Lowestcategoryscore
Spellingissues
Scorecard
Processed

articles/quickstart.md
86
62
3
link
:white_check_mark:

More information about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues
Troubleshooting issues with Acrolinx

Docs Build status updates of commit d142a69:
:warning: Validation status: warnings

File
Status
Preview URL
Details

ValidatePublishRequest
:warning:Warning

Details

articles/quickstart.md
:white_check_mark:Succeeded
View

ValidatePublishRequest

[Warning-RepositoryOwnerAuthorizedCrendentialInvalid] The owner of the repository https://github.com/MicrosoftDocs/quantum-docs-pr, 'adkinn' no longer has Admin permission to the repository. This has no impact to published pages or Docs site, but may cause potential issues for repository management. To resolve this issue, please ask anyone of Admin permission of the repo to set a new owner on Docs portal (https://ops.microsoft.com).

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology.

Article
Acrolinxscore
Lowestcategoryscore
Spellingissues
Scorecard
Processed

articles/quickstart.md
86
62
3
link
:white_check_mark:

More information about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues
Troubleshooting issues with Acrolinx

Docs Build status updates of commit cb49e59:
:warning: Validation status: warnings

File
Status
Preview URL
Details

ValidatePublishRequest
:warning:Warning

Details

articles/quickstart.md
:white_check_mark:Succeeded
View

ValidatePublishRequest

[Warning-RepositoryOwnerAuthorizedCrendentialInvalid] The owner of the repository https://github.com/MicrosoftDocs/quantum-docs-pr, 'adkinn' no longer has Admin permission to the repository. This has no impact to published pages or Docs site, but may cause potential issues for repository management. To resolve this issue, please ask anyone of Admin permission of the repo to set a new owner on Docs portal (https://ops.microsoft.com).

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Get Total number of scenarios and specs during run time

I am using dotnet plugin and c# for my automated tests. Is there any way to get total number of scenarios and spec files during the execution?
I know I can obtain current spec and scenario name but I need total number of spec and scenarios in current execution.

This information is not available yet, a good place to add this is under ExecutionInfo

Gauge will not extending ExecutionInfo to add this info at the moment.
This means that total number of spec and scenarios can only be obtained after a run. However a PR implementing this feature in a way that does not affect backward compatibility will be merged.


Added Ghost aggression level information

I found in the AudioClips of the game that a few possible hints from the Radio at the start of the game do indicate possible ghost aggression levels. Using the names of the files (aggressive, none-friendly, friendly) I've put together a piece of optional functionality that allows a user to get a brief overview of what type of ghost they may be dealing with.
As the could be used as unfair knowledge however, this may want to be put under a "spoiler" mode in the future.
I debated with myself whether to also add the option to preview the voice line AudioClips, but decided to keep them out.

I sware I don't normally do just single commits lol

Just noticed I missed off the reset functionality, working on that now.

Cool stuff, I've added it. I'll need to format it a bit more but it looks great for now. Thanks!

Cheers, happy I could make a worthy contribution.
As mentioned previously, I'll keep digging and seeing if there is any more useful information I can pool together.


Use agate.Table.print_table() in csvlook, closes #515

@onyxfish Please review the changes to the tests (having to do with type inference) to see if this is the desired new behavior. If so, we'll need to document the changes in the changelog.

Hmm, agate seems to completely ignore whatever column names were already there...

Aha, it has to do with my use of agate in csvlook.py. Fixing.

Issues:

--no-header-row prints the first row as the header, instead of creating dummy column names
--linenumbers is broken, as agate's print_table doesn't understand this (only its csv.writer takes a line_numbers argument)

Removing the failing test for now so that I'm not blocked.


Unknown device after installing GPU driver

After installing the GPU driver as directed on the page, there is still a problem.  Device Manager correctly shows the NVIDIA card under Display Adapters but it also shows an unknown device:

Also, the GPU does not show up in the Performance tab of Task Manager.
I believe there is something else that must be done to ensure the graphics card is installed correctly in the VM's operating system.

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

ID: 3c5bc749-3f68-e230-37ff-f74ce72efb59
Version Independent ID: 2a382803-e085-82e2-4139-be5c8b9e1a0d
Content: Azure N-series NVIDIA GPU driver setup for Windows - Azure Virtual Machines
Content Source: articles/virtual-machines/windows/n-series-driver-setup.md
Service: virtual-machines
Sub-service: sizes
GitHub Login: @vikancha-MSFT
Microsoft Alias: vikancha

@yusidd
Thanks for your feedback! We will investigate and update as appropriate.

Hi @yusidd
If you have installed the NVIDIA GPU driver as directed on the page and are still experiencing issues with the graphics card, there may be additional steps you need to take to ensure that the card is installed correctly in the VM's operating system.
One possible issue is that the unknown device in Device Manager may be related to the NVIDIA High Definition Audio driver. This driver is often installed along with the NVIDIA GPU driver and may be causing the unknown device to appear. To resolve this issue, you can try installing the NVIDIA High Definition Audio driver from the NVIDIA website.
Another possible issue is that the GPU is not being recognized by the operating system. To check if this is the case, you can try running the nvidia-smi command in a command prompt or PowerShell window. If the GPU is not recognized, you may need to check the BIOS settings of the VM to ensure that the GPU is enabled and properly configured.
If you are still experiencing issues after trying these steps, to gain a better understanding of your issue, I'd recommend working closer with our support team via an Azure support request.

Hi.  I installed the official and latest NVIDIA driver from their driver download page:

There is only one driver to download for this chip.  If I search for High Definition Audio driver, I do not find one for that GPU/OS.
The nvidia-smi tool shows the GPU as operating.  However, I don't see the GPU in Windows Task Manager.  That is one reason I am not confident that it installed correctly.

@yusidd
It's possible that the NVIDIA driver installed correctly, but the GPU is not being recognized by Windows Task Manager. Here are a few things you can try to troubleshoot the issue: Check NVIDIA Control Panel, Windows settings, BIOS settings,
If none of these steps resolve the issue, You may need to contact NVIDIA support for further assistance. They may be able to provide additional guidance on how to troubleshoot the issue or provide a solution to the problem.  or I'd recommend working closer with our support team via an Azure support request.

@yusidd
We are going to close this thread, if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.
#Please-close


feat(angular-table): angular adapter implementation with signals

This is still a wip implementation of the angular adapter with a mixed approach:
The adapter will always return a signal table. This is useful if passed as a component input. That signal change on every state/option change. If a signal has been used within the options, the table object will be updated automatically.
@Component({
  selector: 'app-root',
  standalone: true,
  imports: [FilterComponent, FlexRenderDirective],
  template : `

        @for (headerGroup of table().getHeaderGroups(); track headerGroup.id) {

            @for (header of headerGroup.headers; track header.id) {
              ...
            }

        }

        @for (row of table().getRowModel().rows; track row.id) {
          ...
        }

  `,
  styleUrl: './app.component.scss',
})
export class AppComponent {
  readonly data = signal([]);

  // This will re-run on every "data" change
  readonly table = createAngularTable(() => ({
    data: this.data,
    columns: this.columns,
  })
}

I've also added in this pr the proxy implementation, where the signal it's extended with a proxified object that allows state slicing. This may not be useful for the adapter, I'm not sure about this and and it would be nice to have some feedback, but maybe "it's too much work" and it's a pain to mantain.
Also, we need to check if there are some caveats with that approach. Internally I'm using computed which memoize the value by default, so in case the view will marked as dirty only if the value instance change (Object.is). This could be a premature optimization considering that almost every property would be transformed.
The first one (signal-only) is safer, and even with it you can do "state slicing", you just need to manually use computed.
class HelloComponent {
  readonly data = signal([]);
  readonly pagination = signal<PaginationState>({pageIndex: 0, pageSize: 20});

  // This will re-run on every "data" and "pagination" change
  readonly table = createAngularTable(() => ({
    data: this.data,
    columns: this.columns,
    state: {
      pagination: this.pagination()
    },
    onPaginationChange: (updater) => {
      this.pagination.set(...)
    }
    // [..]
  }));

  constructor() {
    effect(() => {
      // run on every state/option change
      this.table().getPageOptions();
    });

    // more granular state, run on every "getPageOptions" change (e.g. only when pagination state change)
    effect(() => {
      this.table.getPageOptions()
    });

    // granular state is still possible using computed, but it must be done manually
    const pageOptions = computed(() => this.table().getPageOptions());
    effect(() => {
      pageOptions()
    });

    // these two are lines does the same thing, they access to the same table instance property.
    this.table().setPageSize(...)
    this.table.setPageSize() // this one is evaluated lazily with proxy
  }
}

I've updated demo code but I may opt to make them simpler as was done for the others. Also, the unit/integration test part is missing. Note also that this implementation doesn't cover required inputs signals (need to wrap with lazy-signal-initializer / lazy-init as described here)

@riccardoperra Wondering how much of your proxy can be replicated with TanStack Angular Store: https://tanstack.com/store/latest/docs/framework/angular/quick-start. I originally thought we'd have to wait until TanStack Table v9 to introduce state subscriptions, but this looks interesting.

Not sure about that. Hope I understood, but In this case I think the table signal could be replaced into a store
- const table = signal(createTable(resolvedOptionsSignal()), {
-  equal: () => false,
- })
+ const table = injectState(createTable(resolvedOptionsSignal()))

then in the effect we should update the store instead of the signal
- table.update(v => v)
+ table.setState(...)

Then for each property that should be treated as a "slice", we should replace the computed with the injectStore
https://github.com/TanStack/table/pull/5442/files#diff-a3500075d578d363191c8bc27e94c7f42a2426e4b6e5c582551cfafe9d2576b4R46-R50
value = injectStore(table, (state) => state[property](), { injector })

However, I think this implicate that for each property a new writeable signal and an effect will be created, wouldn't it be less performant / consuming more memory than a computed fn?

I'd like to get one of these approaches in the main PR ASAP so that we can march forward on the rest of it with signals. Thoughts @crutchcorn when you have time?

Not sure having an approach where the table is componentized is right (does the other frameworks do this aswell?)
@Component({
  selector: 'my-user-table',
  template: `
    @if (table(); as table) {
       ...
    }
  `,
  standalone: true,
})
class MyUserTableComponent {
  table = input.required<TableResult>();
}

@Component({
  template: `
    <my-user-table [table]="table()/>
 `
})
class AppComponent {
  table = createAngularTable(() => ({
    data: this.data
  });

  constructor() {
    this.http.get('...').subscribe(() => this.data.set(data))
  }
}

I've updated the signal code approach due to some failing test: even with equals: () => false, the view is not updated if the table result is passed as an input (basically, you'll empty data even if it's present)

Not sure having an approach where the table is componentized is recommended, (does the other frameworks do this as well?.

@riccardoperra I'm not really sure what that means. But if it means passing the table instance around to components as a param/prop, that's common. https://github.com/KevinVandy/material-react-table/blob/v2/packages/material-react-table/src/components/table/MRT_Table.tsx

Once a direction has been chosen, let's mark this as ready to review and merge to the main PR

this is weird. I don't remember merging this, but it's merged into my PR. @riccardoperra can we move forward with one of the signal adapters you proposed this week and keep working on it?


fix(renderer): Only draw text bg if the color differs from the bar bg color

Fixes #759 by only drawing text background when its color is different from the background color of the bar itself. Original check was made invalid by this change.
There is one small issue though: if the text background color is explicitly set equal to background-0 it will be ignored.

I don't think there is any way to solve this without a lot of control variables or where explicitly setting the background color to the bar background doesn't break the expected behaviour with gradients.
So I think this is the best solution for this problem.
Can you please add a comment explaining the reason for this and its pitfalls in the code.
I will do some more testing on this.

@patrick96 yeah, I figured as much as well. Regarding the comment, is it fine if I simply push to this branch or am I supposed to do a rebase first since it is behind master?

Do a push without rebasing and if this page shows a conflict, do a rebase

Thank you very much!
I have also added a note on the Wiki about this for future reference


2020-08-24 tc1702_M2SD Parameters not generated for profile with only 3 data points

For tc1702_M2SD, there's a single profile (SDEID==1791634817) with 3 datapoints. No selections were made for FLGEXKEL, however, none of the non-KEL related parameters were generated for this profile. There is nothing remarkable about the data for this profile. DOF and DOSE are defined.
Note for this testcase, CEST_C0 is reported in the estimated concentration dataset incorrectly for timepoints greater than zero.
All datasets and inputs are loaded to onedrive.

Further analysis indicates that the Actual Times PKATPD were missing for this profile. This issue can be closed.


Update review-sherlock to latest

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] The commit message includes the cask’s name and version.

Thank you for the contribution. It was merged directly as f8eb884954d22ecbfa62cfba7ce6b54fdecf6c5c. Your contribution is still credited to you.


dt2w: add native support for armani

Redmi 1s DT2W support

Signed-off-by: hemantbeast<EMAIL_ADDRESS>

@yoinx
Please add this dt2w native support for Redmi 1s (armani).

Sorry for the late reply.... I've been really busy in real life...
Just adding this value to the constants doesn't really do anything. You have to actually call it from the appropriate spots.
What are the appropriate values for this? What do they do?

This is to support the DT2W for ft5x06 display driver. Its just get the Enable/Disable control of DT2W. It is working fine with the Redmi 1s device.
And sorry for my bad english. I hope you understand what I'm saying.

I get that the path might be right...  But just adding it to the constants file like that doesn't do anything.

Comment on pull request

Comment on pull request

This will not function as submitted.


failed to build pouch, container exited

Issue Description
I cloned pouch,  then build it with ./hack/make.sh, and failed.
Describe what happened:
while Download and install containerd ,docker container exited
[root@host pouch]# ./hack/make.sh
sha256:c558bc629d9b87c22f372f8dc5ce99f0297c0155531a7a31d4b04a3ad80ba005
Build pouch binaries and execute unit tests.
fmt
lint
vet
validate-swagger
2017/12/12 11:59:37
The swagger spec at "apis/swagger.yml" is valid against swagger specification 2.0
GOOS=linux go build -o pouchd
go build -o pouch github.com/alibaba/pouch/cli
unit-test
?   	github.com/alibaba/pouch	[no test files]
?   	github.com/alibaba/pouch/apis/metrics	[no test files]
?   	github.com/alibaba/pouch/apis/server	[no test files]
?   	github.com/alibaba/pouch/apis/types	[no test files]
?   	github.com/alibaba/pouch/cli	[no test files]
ok  	github.com/alibaba/pouch/client	0.082s
?   	github.com/alibaba/pouch/ctrd	[no test files]
?   	github.com/alibaba/pouch/daemon	[no test files]
?   	github.com/alibaba/pouch/daemon/config	[no test files]
?   	github.com/alibaba/pouch/daemon/containerio	[no test files]
ok  	github.com/alibaba/pouch/daemon/meta	0.023s
ok  	github.com/alibaba/pouch/daemon/mgr	0.090s
?   	github.com/alibaba/pouch/daemon/spec	[no test files]
?   	github.com/alibaba/pouch/internal	[no test files]
?   	github.com/alibaba/pouch/pkg/client	[no test files]
?   	github.com/alibaba/pouch/pkg/collect	[no test files]
?   	github.com/alibaba/pouch/pkg/debug	[no test files]
?   	github.com/alibaba/pouch/pkg/exec	[no test files]
?   	github.com/alibaba/pouch/pkg/httputils	[no test files]
?   	github.com/alibaba/pouch/pkg/jsonstream	[no test files]
ok  	github.com/alibaba/pouch/pkg/kmutex	5.002s
?   	github.com/alibaba/pouch/pkg/randomid	[no test files]
ok  	github.com/alibaba/pouch/pkg/reference	0.004s
ok  	github.com/alibaba/pouch/pkg/ringbuff	10.002s
?   	github.com/alibaba/pouch/pkg/serializer	[no test files]
ok  	github.com/alibaba/pouch/pkg/utils	0.003s
?   	github.com/alibaba/pouch/registry	[no test files]
?   	github.com/alibaba/pouch/version	[no test files]
?   	github.com/alibaba/pouch/volume	[no test files]
?   	github.com/alibaba/pouch/volume/driver	[no test files]
?   	github.com/alibaba/pouch/volume/error	[no test files]
?   	github.com/alibaba/pouch/volume/examples/demo	[no test files]
?   	github.com/alibaba/pouch/volume/modules/ceph	[no test files]
?   	github.com/alibaba/pouch/volume/modules/local	[no test files]
?   	github.com/alibaba/pouch/volume/modules/tmpfs	[no test files]
?   	github.com/alibaba/pouch/volume/store	[no test files]
?   	github.com/alibaba/pouch/volume/store/boltdb	[no test files]
?   	github.com/alibaba/pouch/volume/types	[no test files]
?   	github.com/alibaba/pouch/volume/types/meta	[no test files]
Download and install containerd.

Describe what you expected to happen:
How to reproduce it (as minimally and precisely as possible):

clone pouch
cd pouch/
./hack/make.sh

Anything else we need to know?:
Environment:

pouch version (use pouch version):  latest version
OS (e.g. from /etc/os-release):

[root@host pouch]# cat /etc/os-release
NAME="CentOS Linux"
VERSION="7 (Core)"
ID="centos"
ID_LIKE="rhel fedora"
VERSION_ID="7"
PRETTY_NAME="CentOS Linux 7 (Core)"
ANSI_COLOR="0;31"
CPE_NAME="cpe:/o:centos:centos:7"
HOME_URL="https://www.centos.org/"
BUG_REPORT_URL="https://bugs.centos.org/"
CENTOS_MANTISBT_PROJECT="CentOS-7"
CENTOS_MANTISBT_PROJECT_VERSION="7"
REDHAT_SUPPORT_PRODUCT="centos"
REDHAT_SUPPORT_PRODUCT_VERSION="7"

Kernel (e.g. uname -a):
Linux host.localdomain 4.10.4-1.el7.elrepo.x86_64 #1 SMP Sat Mar 18 12:50:10 EDT 2017 x86_64 x86_64 x86_64 GNU/Linux

Install tools:

Others:

Maybe you should try to remove --quiet in wget --quiet https://github.com/containerd/containerd/releases/download/v1.0.0/containerd-1.0.0.linux-amd64.tar.gz -P $TMP
Then you can get what happened in the wget procedure. @zzchun

It's really because of network problems
thanks @allencloud


helmfile-stderr logs a generic message

Excerpt of my workflow
      - name: Run Helmfile Diff 📝
        id: helmfile_diff
        if: github.event_name == 'pull_request'
        uses<EMAIL_ADDRESS>        with:
          helmfile-version: "v0.160.0"
          helm-version: "v3.14.2"
          helmfile-workdirectory: "./charts"
          helm-plugins: >
            https://github.com/databus23/helm-diff
          helmfile-args: diff --environment development

      - name: PR Comment 💬
        if: github.event_name == 'pull_request' && (contains(steps.helmfile_diff.outputs.helmfile-stdout, 'has been removed') || contains(steps.helmfile_diff.outputs.helmfile-stdout, 'has been added') || contains(steps.helmfile_diff.outputs.helmfile-stdout, 'has changed') || steps.helmfile_diff.outputs.helmfile-stderr != '')
        uses: actions/github-script@v7
        env:
          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
        with:
          script: |
            const output = `
            #### Helmfile Diff Results for Development ♠
            Helm Diff Output

            \`\`\`
            Output
            ${{ steps.helmfile_diff.outputs.helmfile-stdout }}
            Errors (if any)
            ${{ steps.helmfile_diff.outputs.helmfile-stderr }}
            \`\`\`

            `;

            github.rest.issues.createComment({
              issue_number: context.issue.number,
              owner: context.repo.owner,
              repo: context.repo.repo,
              body: output
            })

As you see, I do not expect the message if specified strings are not found and stderr is empty.
However, I get comment as below (result from Actions logs)
Run actions/github-script@v7
  with:
    script: const output = `
  #### Helmfile Diff Results for Development ♠
  Helm Diff Output

  \`\`\`
  Output
  Comparing release=secret-creator, chart=secret-creator

  Errors (if any)
  Building dependency release=secret-creator, chart=secret-creator

  \`\`\`

  `;

  github.rest.issues.createComment({
    issue_number: context.issue.number,
    owner: context.repo.owner,
    repo: context.repo.repo,
    body: output
  })

Expected:
No generic messages in stderr.
Thank you for considering the issue.

@abseht you wish diff return empty when no diff?

@yxxhero i think it is fair to return some message as "...nothing to apply..." and people will use it as they please.

@abseht yeah. you can post an feature request in helmd-diff project.

@yxxhero this issue is about not logging 'stderr' if there is no errors.

ok


Ch3 Notebook: Random Forest Model

In the chapter 3 notebook, I noticed that when the train test split function is as follows: X_train, X_test, y_train, y_test = train_test_split(X, y, test_size=0.5) This function shuffles the data by default. I believe this is a problem because it introduces look ahead bias into the time series (Maybe we could consider it a light type of label leakage). This is also the reason why purge k fold and embargo CV is introduced in later chapters.
To correct this the code needs to read X_train, X_test, y_train, y_test = train_test_split(X, y, test_size=0.5, shuffle=False)

Nice catch. Updated the notebooks. Fixed dd47196ae766c385b4a6ffe1519c39c89b740851