Add nicer CLI flags and output

Right now,  we just dump a bunch of javascript objects to the command line.
This isn't very helpful.
What would be a nice start, would be to get a lighthouse ci style breakdown like so:

What would be really nice would be to have a flag to output json in a format that works nicely with vega-lite.
This would make it straightforward to talk about analysis using observable, or if you understandably aren't crazy about sharing AWS API keys onto thirty platforms, run the checks locally.
https://beta.observablehq.com/@mbostock/exploring-data-with-vega-lite

It looks like we can use cli table give us a nice table quickly:
https://www.npmjs.com/package/cli-table -
We can also use the same spinner to account for the call taking a while to return
https://www.npmjs.com/package/cli-spinner

The vega output now is in #3 , so I think @vsmart's PR can resolve this one.


Attempts to copy text intermittently triggers Go to definition

VSCode Version: 1.52.1
OS Version: macOS 11.1

I have an issue that has started to become more and more annoying in VS Code. Sometimes when selecting something which has support for "Go to Definition" and attempting to copy the text with Ctrl+C, Visual studio will open a new web browser with the definition link. This is despite there not being any keyboard shortcuts registered for Go to definition.
This is NOT triggered by pressing Ctrl and the mouse button at the same time. I have been able to recreate this even when waiting more than one second after selecting the text. It's also intermittent in that it will not always open the definition, but it seems that it will almost always trigger after switching open files (tabs) or refocusing Code. It seems to happen less if the same file has been open for a while (although repeatedly pressing Ctrl will consistently trigger the issue).
A possible thing that could cause issues is that I have swapped Ctrl and Cmd in the macOS settings, but I have checked and cannot find any matching mappings for either of the keys.. and it doesn't explain why it can be triggered by spamming Ctrl.
Steps to Reproduce:

Switch file in Visual Studio Code
Select any snipped which have any support for "Go to Definition"
Press Ctrl (or cmd), if needed spam the button a bit.
VS Code proceeds to throw you away to a browser page.

Does this issue occur when all extensions are disabled?: No, but that disables "Go to Definition" entierly...

@albgus What programming language are you using? What extensions do you have installed? (Extensions can trigger programmatically "Go to Definition").

I've only noticed this problem with Terraform, although I think that may be the only plugin I'm actively using with Go to definition support.
Is there any way to enable debug logging / find out if this is caused by some plugin?

This behaviour seems to have gone away now and the only thing I have changed is activating "Switch to Alt+Click for Multi-Cursor" from the Selection menu. Not sure why this would have had any impact so it might still be a bug, but at least I have a workaround that doesn't seem to have any downsides now.

The menu item "Switch to Ctrl|Alt+Click for Multi-Cursor" changes the gesture to trigger go to definition. It is either bound to alt+click or to ctrl+click.


Netting system and Hedging system

Hello,
following discussion https://github.com/mhallsmoore/qstrader/issues/127
I think that it will be great if qstrader could support like Metatrader 5, 2 kind of accounting system
https://www.mql5.com/en/articles/2299

Netting system
Hedging system

Kind regards
A similar issue is also opened for zipline (Quantopian backtester)
https://github.com/quantopian/zipline/issues/1375

Isnt't the hedging system just a specific view of the trades (group by something)?

I'd say in fact that I think netting system is a specific view of the trades as they appears in hedging system but grouped by Symbol
This is also a different way to handle trades
With Netting

buy 3 lots of EURUSD (for example) - (you are 3 lots long)
sell 1 lot - (now, you are 2 lots long)
sell 2 lot - (no opened trades - balance=equity)
sell 1 lot - you are 1 lot short

With Hedging

buy 3 lots of EURUSD (for example) - (you are 3 lots long)
You can:
partially close your trade of 1 lot  (you will only see 1 trade)
open a 1 lot sell order (you will see 2 trades)

True. So as long as tags are attached to trades, it's possible to build custom views.

I recently looked at doc of other backtester with such signal, sizer and risk manager mechanisms.

If we only consider a netting system we should have

BOT (Long ie positive quantity)
SLD (Short ie negative quantity)
but also
FLAT (to be out of market ie quantity = 0)

if we consider a hedging system (or we split strategies into entry strategy and exit strategy) should have:

EnterLong
EnterShort
Flatten

but also exit

ExitLong
ExitShort
Flatten

Indeed, I grappled with this issue for quite some time when I first designed the SignalEvent. You can also imagine the situation with a mean reversion trade when you want to trade signals as a pair, with a certain attached "quantity" or "strength" to each, representing a hedging ratio.
This means that signal events are somewhat intertwined for some strategies. It does indeed motivate some addition signal types.


flake8 linting

I want to start working on performance of processing ms records, and also fix some errors in ms position to integer conversion.  I got side tracked by too many flake8 errors in the buffers.  Thus, this PR does the following:

[x] remove all use of from foo import *. from the SHIC version of making feature vectors from ms data.
[x] address all flake8 issues in that file + msTools.py
[ ] address all flake8 issues in fvTools.py

@andrewkern -- flake8 finds several instances of undefined variables in fvTools.py, so this PR has become more than a code formatting exercise.  The linter has found some real bugs.  I have flagged some of them with "AK" in the comments.

@andrewkern the files changed in this PR have had enough flake8 fixes that using a code linter in vim will show the real bugs and not be cluttered with pep8 formatting issues.  I use syntastic for running the linter after every :w.


Custom WMS layers doesn't work

On
http://brouter.de/brouter-web
I tried to add my own wms layers from
https://geoportal.gov.pl/uslugi/usluga-przegladania-wms
eg.:
https://mapy.geoportal.gov.pl/wss/service/PZGIK/ORTO/WMTS/HighResolution
https://mapy.geoportal.gov.pl/wss/service/img/guest/TOPO/MapServer/WMSServer
but none of them has been displayed.
In QGIS I see this layers.
Where could be the problem?
Christopher

WMS is not supported yet for custom layers and WMTS only with a compatible tiling scheme and if you can provide a full XYZ url. See also #337.

Duplicate of #337


docs(devserver): webpack-dev-server open new option type

Documentation for this PR : https://github.com/webpack/webpack-dev-server/pull/2492 in webpack dev server

Thanks!


Update master to main

@tjprescott are the failures in CI expected?

@weshaggard not expected per se, but not caused by your PR. We can merge this and investigate.


Redis service start fail on Debian stable (Wheezy 7.0)

Debian Wheezy installs redis-server 2.4.14 which service never starts.
If you try to start the service you get the following error:
$ sudo service redis-server start
Starting redis-server:
*** FATAL CONFIG FILE ERROR ***
Reading the configuration file, at line 110

'stop-writes-on-bgsave-error yes'
Bad directive or wrong number of arguments
failed

$ dpkg -l |grep redis
ii  redis-server                       2:2.4.14-1                    amd64        Persistent key-value database with network interface

Redis 2.4 is no longer supported by this module.
Please use the packages from backports instead.


[FEAR]: Implement Funding Program for Contribution to our product

Is your feature request related to a problem? Please describe.
Description:
Feature Overview:
In order to accelerate the growth and innovation of T-shirt product, we're initiating a funding program that aims to support external contributors who actively contribute to the development of our product. This program will incentivize contributions and foster collaboration within our community, ultimately leading to an enhanced and more robust product.
Describe the solution you'd like

Implementing it with to raise fund for our product
Designing it as per the ui/ux design

Describe alternatives you've considered
No response
Additional context
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSoC'23 contributor
[X] I want to work on this issue

@amanjaiman1 and @balaji-sivasakthi  assign this issue to me


What is the webpack:// protocol?

Per Webpack's issue template:
<!-- Bug reports and Feature requests must use other templates, or will be closed -->
<!-- Please ask questions on StackOverflow or the webpack Gitter (https://gitter.im/webpack/webpack). -->
<!-- Issues which contain questions or support requests will be closed. -->

I originally tried asking this question on StackOverflow, but with no luck. Since the question will likely be deleted, I will copy the contents below:

I downloaded and searched the documentation repository using my text editor for the string webpack://, and found it located in only two documents:

src/content/configuration/index.md

which goes to: https://webpack.js.org/configuration/

src/content/configuration/output.md

which goes to: https://webpack.js.org/configuration/output/

As far as I can tell, this is not an actual protocol, much less one a browser can understand.
Questions:

What is the webpack: protocol intended to be used for?
Where can I find documentation on it?
(assuming it is used internally by webpack or webpack dev server) Why do I see it output in Source Maps or in some project boiler plates, when the browser has no way of understanding what it should do with the protocol?

Update 2018-05-25
I checked Webpack's source code in addition to their documentation repo, and this is what I found:

https://github.com/webpack/webpack/tree/v4.8.3/examples/source-map/
lib/EvalDevToolModuleTemplatePlugin.js
lib/EvalSourceMapDevToolModuleTemplatePlugin.js
lib/SourceMapDevToolPlugin.js

(Paths instead of links since I lack the reputation to post more than 8)

StackOverflow Comments

I mean that the structure of the source URL in a sourcemap is meaningless to tooling. ie, the protocol doesn't matter. – SLaks May 25 at 15:17
... The protocol shouldn't matter, but for some reason it does, especially when the final sourcemap includes this as a protocol. The browser has no way of understanding a webpack:// URL. It shows up in the links above and in Angular CLI ( github.com/angular/angular-cli/blob/1.7.x/packages/%40angular/… ) when you eject a configuration (I'm currently on 1.7.4, they recently made the jump from 1.x to 6.x, which temporarily removes "eject"). I'd like to know why the webpack and angular teams use it. – nick May 25 at 16:27

Since Angular CLI also uses/used the webpack:// protocol, is there anyone here who might be able to explain it to me?
Thanks and regards.

this is a custom protocol used in webpack source map, you can custom it by checking out this doc:  https://webpack.js.org/configuration/output/#output-devtoolmodulefilenametemplate

how can i override devtoolModuleFilenameTemplate in angular.json? Since i cannot eject webpack config from angular project i need to rewrite this option in angular.json. This option generate many troubles while debugging in chrome or firefox.

@vanmxpx - Your problem doesn't seem to be related to the original topic of this issue, you may be better served opening a new issue to get the correct attention on it. You may also find the following helpful:
https://github.com/angular/angular-cli/blob/v6.0.7/docs/documentation/config.md
Though, I don't know if that applies to webpack configuration under the hood.
Since the docs note that eject is temporarily disabled, I imagine they are aware of it and already have an issue open for resolution in the future.

@npetruzzelli I have create the issue, but still quite. Thank you for response and for ng config, might be helpful.

You're welcome, and good luck!

Hello, we reviewed this issue and determined that it doesn't fall into the bug report or feature request category. This issue tracker is not suitable for support requests, please repost your issue on StackOverflow using tag angular-cli.
If you are wondering why we don't resolve support issues via the issue tracker, please check out this explanation.


Update the ecms_api_recipient installation configuration

Summary
This updates the ecms api recipient module to enable the notification content type by default and updates the configuration form to allow the ecms_api_recipient role to publish nodes using the editorial workflow.
Metadata

Question
Answer

Did you use a meaningful pull request title?
yes

Did you apply meaningful labels to the pull request?
yes

Did you perform a self review first?
yes

Documentation reflects changes?
no

CHANGELOG reflects changes?
yes

Unit/Functional tests reflect changes?
yes

Did you perform browser testing?
yes

Risk level
Low

Relevant links
RIG-76

Ok looks good!
/approved


Split user

Q
A

Is bugfix?
❌

New feature?
❌

Breaks BC?
✔

Fixed issues
#19

Adopt PRs:

Package
PR

yiisoft/app-api
https://github.com/yiisoft/app-api/pull/45

yiisoft/yii-demo
https://github.com/yiisoft/yii-demo/pull/244

yiisoft/demo-api
https://github.com/yiisoft/demo-api/pull/6

Decided on a call that we need to eliminate storage interface and use session only. If session is null, do not write ID.

👍


Improve visual design of the network simulation tool

When displaying a network using the simulation tool as described in #17 should be considered the design specifications based on creating a visual distinction of node balances and channel capacities. It should be 3 options to visually represent the channel capacities:

[x] Low: Noce balances and channel capacities are highlighted with red colour to represent the low balance as a visual warning.
[x] Medium: Node balances and channel capacities are highlighted with orange colour to represent the medium balance as a visual warning.
[x] High: Node balances and channel capacities are highlighted with green colour to represent the high balance as a visual warning.

In the case of the channels, the width of the edges should have a relation with the channel capacity.

This issue has been successfully solved. A screenshot and the outcome of the solution are below:

				if (node.getBalance() > 600) {
					graph.insertVertex(parent, String.valueOf(node.getId()), element, width.get(i), height.get(i), 80, 30,
							"ROUNDED;strokeColor=green;fillColor=green;fontColor=white");
				} else if (node.getBalance() >= 200 && node.getBalance() <= 599){
					graph.insertVertex(parent, String.valueOf(node.getId()), element, width.get(i), height.get(i), 80, 30,
							"ROUNDED;strokeColor=orange;fillColor=orange;fontColor=white");
				} else if (node.getBalance() >= 0 && node.getBalance() <= 199){
					graph.insertVertex(parent, String.valueOf(node.getId()), element, width.get(i), height.get(i), 80, 30,
							"ROUNDED;strokeColor=red;fillColor=red;fontColor=white");
				}


Where is the "ca-bundle.crt" in openssl1.1.1b

I downloaded source code from, and install 1.1.1b like this:
tar -zxv openssl-OpenSSL_1_1_1b.tar.gz
cd openssl-OpenSSL_1_1_1b/
./config --prefix=/usr/local/openssl-1.1.1b
make && make install
./config shared
make clean
make  && make install
cd /usr/local/openssl-1.1.1b/
echo "/usr/local/openssl-1.1.1b/lib/" >> /etc/ld.so.conf
ldconfig -v
then I used ./usr/local/openssl-1.1.1b/bin/openssl version -a and get
OpenSSL 1.1.1b  26 Feb 2019
built on: Wed Jan 27 14:05:38 2021 UTC
platform: linux-x86_64
options:  bn(64,64) rc4(16x,int) des(int) idea(int) blowfish(ptr)
compiler: gcc -fPIC -pthread -m64 -Wa,--noexecstack -Wall -O3 -DOPENSSL_USE_NODELETE -DL_ENDIAN -DOPENSSL_PIC -DOPENSSL_CPUID_OBJ -DOPENSSL_IA32_SSE2 -DOPENSSL_BN_ASM_MONT -DOPENSSL_BN_ASM_MONT5 -DOPENSSL_BN_ASM_GF2m -DSHA1_ASM -DSHA256_ASM -DSHA512_ASM -DKECCAK1600_ASM -DRC4_ASM -DMD5_ASM -DAES_ASM -DVPAES_ASM -DBSAES_ASM -DGHASH_ASM -DECP_NISTZ256_ASM -DX25519_ASM -DPADLOCK_ASM -DPOLY1305_ASM -DNDEBUG
OPENSSLDIR: "/usr/local/openssl-1.1.1b/ssl"
ENGINESDIR: "/usr/local/openssl-1.1.1b/lib/engines-1.1"
I think I have installed openssl1.1.1b successfully, but under /usr/local/openssl-1.1.1b/ssl/certs, there is no this file ca-bundle.crt, if with openssl1.0.1, ca-bundle.crt is under /etc/pki/tls/certs/, whether this file ca-bundle.crt is canceled in 1.1.1b? Or I did not install successfully?

That file has never been distributed as part of OpenSSL. It is typically provided by distributions from another source such as Mozilla's truststore.

That file has never been distributed as part of OpenSSL. It is typically provided by distributions from another source such as Mozilla's truststore.


Merge conflict is shown when edits are done on a different book

When two users working on two different files/books on en_tn, merge their work to master, the user user sees merge conflict and is not able to merge the work done until logging back in.

@elsylambert This seems like a high priority that would happen often. Is that true?

@kintsoogi and @theNerd247 are working on the merge process in gE right now and we should wait until they have a solution to see if it affects this.

Hi @elsylambert This is the issue that I had mentioned in our call yesterday https://github.com/Bridgeconn/uw-lab/blob/50d4ec9b188ff4f7d40be57738a108079366060e/dcs-branch-merger/README.md#checkdtbranchtousrbranch-this-is-not-working-properly
In the DCS Merger RCL the CheckDtBranchToUsrBranch function does not work as the name suggests.
However this function is not used by TC Create so don't think it can be tested in TC Create it can be tested in the RCL if required.
Joel is the one who worked on this RCL POC, he can give the specifics steps  for the issue as required here or in the RCL repo as a new issue.

We will be testing the issue #1609 fixed by Noah above on TC Create and updating it here soon.

Hi @elsylambert This is the issue that I had mentioned in our call yesterday https://github.com/Bridgeconn/uw-lab/blob/50d4ec9b188ff4f7d40be57738a108079366060e/dcs-branch-merger/README.md#checkdtbranchtousrbranch-this-is-not-working-properly In the DCS Merger RCL the CheckDtBranchToUsrBranch function does not work as the name suggests. However this function is not used by TC Create so don't think it can be tested in TC Create it can be tested in the RCL if required. Joel is the one who worked on this RCL POC, he can give the specifics steps for the issue as required here or in the RCL repo as a new issue.

Accidentally closed this issue while adding a comment. I have reponed this issue.

Hi @elsylambert This is the issue that I had mentioned in our call yesterday https://github.com/Bridgeconn/uw-lab/blob/50d4ec9b188ff4f7d40be57738a108079366060e/dcs-branch-merger/README.md#checkdtbranchtousrbranch-this-is-not-working-properly In the DCS Merger RCL the CheckDtBranchToUsrBranch function does not work as the name suggests. However this function is not used by TC Create so don't think it can be tested in TC Create it can be tested in the RCL if required. Joel is the one who worked on this RCL POC, he can give the specifics steps for the issue as required here or in the RCL repo as a new issue.

PR ScreenShot :

dcs-branch-merger:

This function in the DCS branch merger RCL will help check if there are any updates in the master branch (as demonstrated in the RCL demo: "This demo is a test of checking for an update of the user branch from master."). Currently, it checks from the user branch to the master branch, but it should be the other way around—from master to the user branch. Additionally, if you look at the PR screenshot, it shows 'master << test-CV,' which I believe is incorrect. It should create a PR like 'test-CV << master,' which helps determine if the master branch has any changes.

We will be testing the issue #1609 fixed by Noah above on TC Create and updating it here soon.

tested it with the latest release (1.4.1), and it works as expected.
Steps for editing different books:

Login to tC create app in two browsers (use two credentials)
select the same org, resource, and lang for both credentials
select file TIT in one browser and  PHM in another browser.
edit both files and save, then merge the files. (Clicking  "merge my work" from the dashboard)
there are no conflict messages when we try to merge the files

Steps for editing same books:

Login to tC create app in two browsers (use two credentials)
select the same org, resource, and lang for both credentials
select file TIT in both browsers.
edit both files and save, then merge the files. (Clicking  "merge my work" from the dashboard)
there will be conflicts messages when we try to merge the files

Hi @elsylambert , Tested and merged the issue PR to develop and  moved to QA App


When playlist is in shuffle in tvOS, the playlist quits after 25minutes of playing

Not reproducable, right? no support tickets on this either? maybe close?


1.18.2 Allow custom reprocessor recipes

Is your feature request related to a problem? Please describe.
Yes. I am trying to buff reprocessor outputs for all Mystical Agriculture and Mystical Customization crops. It does work for any crop from the base mod (and I can even remove its recipes), but for Mystical Customization, it doesn't seem possible.
Describe the solution you'd like
Either a configuration with the Mystical Customization crops JSON files, using https://github.com/BlakeBr0/MysticalAgriculture/blob/053b62347c694bb590ea2b7c5a17a221c4c692ba/src/main/java/com/blakebr0/mysticalagriculture/api/lib/PluginConfig.java if that can be useful. This would allow disabling the reprocessor recipe, which I can then readd without having a conflict for the Reprocessor machine block.
Or that the recipes' load order be adjusted if that is possible, since it appears Mystical Customization is always loaded last, which overrides anything that was meant to override it at first.
Describe alternatives you've considered
I tried with KubeJS, with a normal datapack, and with a datapack loader such as Paxi. Possibly I'm doing something wrong, but I'm unsure.
Additional context
For example, runewood seeds:
runewood.txt
I tried through JSON at:
data/mysticalagriculture/recipes/seed/reprocessor/runewood.json
data/mysticalagriculture/recipes/runewood_seeds_reprocessor
data/mysticalcustomization/recipes/runewood_seeds_reprocessor
The one that hit the correct location was:
mysticalagriculture/recipes/runewood_seeds_reprocessor
Although Mystical Customization's recipe was always loaded on top, which doesn't result in 2 recipes showing, but the default one like this:

In short, it was never possible to increase the amount of output in the reprocessor (only for Mystical Customization crops). Possibly that information can be described on the wiki as well if you want, if this feature request goes through. I did read all the wiki before and it was very useful, thank you 🙂
Thanks for considering it!

I'm adding the option to disable the auto generated recipes for a crop in the next version. Note that you will need Mystical Agriculture 5.1.1 for the options to function properly.
https://blakesmods.com/docs/mysticalcustomization/adding-crops#recipe-config

That is awesome, thanks so much! 😃


SWC crashes when trying to parse a bundle created by Next.js

Describe the bug
Our use case is that we first produce app bundles through the normal Next.js build, but we leave environment variables. When the app starts upp in an environment we run an AST transform on the bundles to replace the environment variables. We're currently using the TypeScript compiler API to do so, which isn't all that fast. So we wanted to try out SWC for this, but hit a road bump pretty quickly when SWC crashed with a hard error one of our bundles.
Input code
https://github.com/andersekdahl/swc-crash-repro/blob/main/bundle.js

Config
No config, all defaults

Playground link (or link to the minimal reproduction)
https://github.com/andersekdahl/swc-crash-repro
SWC Info output
thread '' panicked at 'index out of bounds: the len is 1141 but the index is 2415', C:\Users\runneradmin.cargo\registry\src\index.crates.io-6f17d22bba15001f\swc_common-0.33.0\src\syntax_pos\hygiene.rs:395:30
note: run with RUST_BACKTRACE=1 environment variable to display a backtrace
node:internal/process/promises:288
triggerUncaughtException(err, true /* fromPromise */);
^
[Error: failed to handle: index out of bounds: the len is 1141 but the index is 2415] {
code: 'GenericFailure'
}
Node.js v18.13.0
Expected behavior
It shouldn't crash
Actual behavior
It crashes
Version
1.3.93
Additional context
No response

I tried it on the playground and it worked. I think the problem is usage of Visitor (JavaScript) which is going to be removed and not tracked.
https://github.com/swc-project/swc/issues?q=is%3Aissue+index+out+of+bounds%3A+the+len+is%3Aclosed


Removed broken link

What does this PR do?
Add resource(s) | Remove resource(s) | Add info | Improve repo
For resources
Description
Why is this valuable (or not)?
How do we know it's really free?
For book lists, is it a book? For course lists, is it a course? etc.
Checklist:

[ ] Read our contributing guidelines.
[ ] Search for duplicates.
[ ] Include author(s) and platform where appropriate.
[ ] Put lists in alphabetical order, correct spacing.
[ ] Add needed indications (PDF, access notes, under construction).
[ ] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

check-urls seems to have some problem: https://github.com/EbookFoundation/free-programming-books/actions/runs/3240117024/jobs/5310347796
@davorpa

check-urls seems to have some problem: https://github.com/EbookFoundation/free-programming-books/actions/runs/3240117024/jobs/5310347796 @davorpa

Force rerun 😉

check-urls seems to have some problem: https://github.com/EbookFoundation/free-programming-books/actions/runs/3240117024/jobs/5310347796 @davorpa

Force rerun 😉

i don't have permissions :(


🐛 Describe the bug

Environment
需要有其它改变吗，非常想使用该项功能

Hi, it only offload optimizer states to disk. It seems your error is not related to optimizer.

Can you provide more info?

We have updated a lot. Please check the latest code. This issue was closed due to inactivity. Thanks.


May I ask you to support continuous questioning in one session？

similar to chatgpt in multiple rounds of questioning in one session?
Instead of opening a new session for each question? The current page does not support asking the context.

#437

#437

你可以自己加个

为啥这么说。。 @linghanwu-code


Support for AllowUnknownFlags

Hi!
I've been using aconfig for a while now and I love it! Thank you for all the hard work. 😊👍
I'm trying to load the same flags using different structs in two places in my program and it always complains about the flag not being defined. I've tried using AllowUnknownFlags without success.
The gist of it is I create a struct with the field A and load it during the initialization of the program, then later on I load a struct with the field B. I then start the program like this: go run main.go --a foo --b bar
When loading the first struct with only A, it says -b is not defined.
Any ideas? I'm on the phone now, but can paste a test I built to debug tomorrow if you'd like.
Thank you again!

Hi @simongottschlag thank you for the kind words and sorry for the troubles. Looks like the problem is realted to the behaviour of the stdlib flag package.
Im pn vacation till Sunday without laptop, will be not able to fix sooner :( will attack this problem on Sunday, ok?

Hi @cristaloleg!
I'm in no hurry, this case is for an example for a pet project. 😊
If I get the time during the week, I'll dig into the problem a bit deeper and see if I can create a PR for you to review next week.
Have a great vacation! 🎉


systemDiskCategory accepted values requires correction

File: docs/reference/pkg/nodejs/pulumi/alicloud/ecs/_index.md
Accepted values for the systemDiskCategory are [cloud ephemeral_ssd cloud_essd cloud_efficiency cloud_ssd local_disk]
Ref: error: alicloud:ecs/instance:Instance resource 'alicloud-web' has a problem: expected system_disk_category to be one of [all cloud ephemeral_ssd cloud_essd cloud_efficiency cloud_ssd local_disk], got cloudSsd

Apologies for a long delay!

got cloudSsd

It looks like perhaps you passed cloudSsd instead of cloud_ssd here?  I'm going to close out, as the document list appears the same as what Ali returns in the error message.  But if you are still having problems here, please do open a new issue in the https://github.com/pulumi/pulumi-alicloud repo.


SwaggerGen empty array examples

I am not sure if this is a user error (question) or a bug / feature request.
But, is there a way for Swashbuckle SwaggerGen to generate empty arrays for examples?
C# Model:
/// <summary>
///     The identifiers of the FilterLists from which this FilterList was forked.
/// </summary>
/// <example>[]</example>
public IEnumerable<long> UpstreamFilterListIds { get; init; } = new HashSet<long>();

Expected:
"upstreamFilterListIds": {
    "type": "array",
    "items": {
        "type": "integer",
        "format": "int64"
    },
    "example": [],
    "description": "The identifiers of the FilterLists from which this FilterList was forked."
},

Generated by Swashbuckle:
"upstreamFilterListIds": {
    "type": "array",
    "items": {
        "type": "integer",
        "format": "int64"
    },
    "description": "The identifiers of the FilterLists from which this FilterList was forked."
},

Since the example node is left out of the spec, Swagger UI then infers a default example of [ 0 ] in its "Try it out" tool which may not actually be a valid example value. Generally an empty array is probably not a useful "example" for consumers, but there may be cases where it is deemed reasonable.

Swashbuckle.AspNetCore.SwaggerGen @ v6.2.3

Have the same issue using latest
 /// <summary>
        /// Shows activity over time per week for users
        /// </summary>
        /// <param name="userIds" example="[]">0, 1 or more user id's</param>
        /// <param name="startDate" example="2022-01-01">startDate</param>
        /// <param name="endDate">endDate</param>
        /// <param name="records">records</param>
        /// <param name="descending">descending</param>
        [HttpPost]
        public object ActivityOverTimePerDayForUsers(int[] userIds, string? startDate = "", string? endDate = "", int records = 100, bool descending = true)
        {
        ...
        }

the .xml is correct although this is not reflected on the Swagger site:
<?xml version="1.0"?>
<doc>
    <assembly>
        <name>MyApp</name>
    </assembly>
    <members>
        <member name="M:MyApp.Controllers.StatsController.ActivityOverTimePerDayForUsers(System.Int32[],System.String,System.String,System.Int32,System.Boolean)">
            <summary>
            Shows activity over time per week for users
            </summary>
            <param name="userIds" example="[]">0, 1 or more user id's</param>
            <param name="startDate" example="2022-01-01">startDate</param>
            <param name="endDate">endDate</param>
            <param name="records">records</param>
            <param name="descending">descending</param>
        </member>
    </members>
</doc>

Works:
    /// <example>"[]"</example>

Works:
    /// <example>"[]"</example>

Ah what you are showing is C# code.
Well for me I just edited the .xml file and changed the line:
<param name="userIds" example="[]">0, 1 or more user id's</param>

into
<param name="userIds" example=""[]"">0, 1 or more user id's</param>

And this worked.
See https://stackoverflow.com/questions/3961505/how-can-i-escape-double-quotes-in-xml-attributes-values


Roll engine 58cdd53f9083..4c79e423dc6f (8 commits)

https://github.com/flutter/engine/compare/58cdd53f9083...4c79e423dc6f
git log 58cdd53f9083..4c79e423dc6f --no-merges --oneline
4c79e423d Roll src/third_party/skia 69083b2b626b..b9998cdceec7 (9 commits) (flutter/engine#6620)
ce36d85d6 Revert "Pass null instead of 'none' locale" (flutter/engine#6622)
9de880cc9 Revert "Roll buildtools to 759274dd801b2f78ae0c4066101d0beca27ffc9a" (flutter/engine#6621)
8cb3965a3 Properly gate android API < 21 and remove extraneous data array. (flutter/engine#6619)
2091c0db6 Update Dart version in Dockerfile (flutter/engine#6618)
739d66eeb Don't attempt to build any platform target on Fuchsia. (flutter/engine#6617)
7d49036d8 Roll Dart to 45f9462398ae7d2e80acd79d3f9c32c7ca6f6f97 (flutter/engine#6616)
6c84c5f6a Pass null instead of 'none' locale (flutter/engine#6613)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.

@liyuqian : New roll has been created by https://autoroll.skia.org/r/flutter-engine-flutter-autoroll

Trybots failed. These were the failed builds: https://cirrus-ci.com/task/5011117594116096


[WIP] feat(charts): Add Bookmarked link metrics in Team page

Add bookmarked link metrics in Team Page
Description
This pull request adds a graphical interface to the team's dashboard page, displaying graphs and charts that count and group all bookmarks per month. This enhancement enables users to visualize their team's contributions over time.
Link to issue
Fix #64
Screenshots

Additional Comments
I utilized Raw SQL since Prisma currently lacks support for grouping by date ranges or formatting dates directly within a groupBy. Additionally, I lowered the "level" in the React Tree where client rendering takes place to enable server-side rendering of the chart. As a result, src/components/pages/Team.tsx is now server-side rendered.

Total bookmarks count seems wrong


Add test on Context

Coverage decreased (-0.03%) to 48.66% when pulling 02c3acb912426f3f098370635530b560d0e668c8 on florian-vuillemot:dev into b85bde0f2917b782df179ac6e13de054f4d526eb on openworm:dev.

These changes have been rebased and merged into dev.
Thanks, @florian-vuillemot !


Render world blocking certain ecs developments

Currently, there are at least two worlds in any given bevy app, the 'normal' world and the render world. The render world is responsible for rendering a scene to the screen and to keep the two worlds in sync, all the entities of the render world are removed at the end of a frame and in the subsequent frame the render world is 'rebuilt'.
There are plans for the ecs (mainly involving Relations) that hinge on more things becoming entities. As entities become more fundamental (components as entities, queries as entities), it becomes a problem when the render world removes all entities.
I'm mainly attempting to lift this problem from Discord (where it has been repeatedly discussed) to a slightly less transient and easier-to-read format.
If I got anything wrong, please tell me. While I think that this problem should be solved, it is sadly also a problem of which I don't know the full details.

Thanks for creating a place to track this.
The main issue is that the render world currently guarantees that extracted entities from the main world will be able to use the same Entity id in the render world. This forces all entity and component data to be dropped each frame to make room for the entities from the main world. This is primarily a problem as mentioned above for using the ECS storage for things like queries, systems and most importantly components which is a requirement for flecs-like fragmenting relations, a long term goal of the project.
Some proposed solutions include:

Stop deleting entities in the render world entirely, move to a resource based render world where extracted data is not store in entities at all.
Instead of ensuring ids match attach a MainWorldEntity component to render world entities to track the mapping and assign arbitrary ids. Then either:
a. Persist entities in the render world, only despawning them when the corresponding entity is despawned.
b. Despawn entities properly each frame (avoid using SparseSet components so entire tables can be dropped).

In my opinion 2b is the only option I'm currently aware of that doesn't require a huge rearchitecting of the renderer, however I am aware there are some who want to pursue 1 or 2a for performance reasons unrelated to the blocked entitification of the ECS.
Regardless I think we are united in wishing to move away from the current model and I'll be happy wherever we end up.

Additional constraint: The render world entity ID must be stable between frames. Lots of temporal rendering techniques require storing and retrieving data across multiple frames per entity.

Additional constraint: The render world entity ID must be stable between frames. Lots of temporal rendering techniques require storing and retrieving data across multiple frames per entity.

Could you use the value from the component in the place of the actual entity ID?
Alternatively this does remind me of another implementation which is to split the address space and use a bit in the identifier to signify a foreign id. This involves a few intricacies with how high entity ids would interact with sparse sets and other ECS internals but we do need to resolve that for relations anyway.

Could you use the value from the component in the place of the actual entity ID?

What component? MainWorldEntity? Yes, so long as MainWorldEntity is stable across frames. I need to be able to do this basically:
// Resource
let mut last_frame_data_per_light = HashMap::new();

// Current frame
for entity in light_query {
     last_frame_data_per_light .insert(entity, foo);
}

// Next frame
for entity in light_query {
   last_frame_data_per_light .get(entity); // Need to know some info about the light stored from last frame
}

My main concern for the render app is performance. In one sense moving away from using the ECS for the main mesh entity flow has been freeing to use whatever we find works best. For me at least it lets me focus on the data and how best to structure and access it to achieve fast rendering, rather than being constrained by how the ECS works.
That said, if there were to be an ECS world shared between the apps so that we could avoid extraction and could access everything that we need, that certainly seems interesting.
And from the original post, 1 sounds ok, 2 sounds a bit clunky but could work.


Proposal: Update the designs of ListView and GridView!

Proposal: Update the designs of ListView and GridView!

Summary

I'm proposing design changes that would affect the way that ListView and GridView show items in their non-resting state (selected, on hover, etc.). The bulk of these design changes would be rounding the corners of List/GridView items, rounding the corners on their selection checkbox in multi-select mode, and changing the color scheme.
Rationale

Many controls in WinUI have rounded corners, ListView and GridView should follow that trend and embrace the modern, Fluent look.
The rounding could easily be turned off, if desired, with the ControlCornerRadius ThemeResource.

Scope

Capability
Priority

ListView and GridView items will have rounded corners when selected, hovered, or in any other non-rest state.
Must

ListView items use a tickmark identifier to show which item is selected.
Must

GridView items should have a white inner border to allow the outer accent-color border to be more visible and distinguishable.
Must

ListView and GridView items will have rounded checkboxes when in multiple selection states.
Should

This proposal won't add new APIs or change the functionality of either of these controls.
Won't

Important Notes

Design Mockups
Here are some  mockups of the visual updates for ListView and GridView, with before and after shots:
GridView in single selection mode:

GridView in multiple selection mode:

ListView in single selection mode:

ListView in multiple selection mode:

A closer look at ListView with items of different sizes, as well as items in different states (hover, selected, focused, etc.):

Open Questions

Are there corner cases that will be affected by this new design?

How would the dark and high contrast variation of the new styles look like?
@mdtauk I think this might be interesting for you!

I will need to have a think on this before I make any concrete suggestions.

How will these work with Reveal and Xbox navigation/reveal focus visuals?
Is the padding/margin between items, something that should be an exposed style value?
What about horizontally aligned and scrolling list views with text and other objects, does the selection indicator still go on the left (or right with RtL flow direction)?
Should there still be a style bundled which either falls back to the polls style, or enables the background colour of the list item to be used for selection indication?
How do these changes affect selected menu items and toggles?  These currently use tick marks and radio bullets.

Personally if I wanted rounded corners, I'd put corner radius inside my item template. I do however see the issue with the selection box then not matching it.
I'd be worried though that this breaks the look of existing apps though, and adds unexpected rounding on things that doesn't look well for the specific design. Is there a guarantee that the corners of the item inside the item template won't "peek out" in the corners but get correctly clipped regardless of the content inside it? (including swap chains)

The rounding could easily be turned off, if desired, with the ControlCornerRadius ThemeResource.

Please don't do just this. It's not discoverable, and takes a considerable amount of time to find. Instead I'd suggest adding an ItemCornerRadius property to the control. See #1626. It might also affect more controls than you intended (you might have a button inside the item template where you do want rounding).
Also I've found the default 2px rounding looks terrible on low-dpi devices. Has this been improved?
Another example where rounding can go wrong: https://github.com/microsoft/microsoft-ui-xaml/issues/1106

I do like this pun a lot

Also I've found the default 2px rounding looks terrible on low-dpi devices. Has this been improved?

The Corner Radius values are changing to 4px for controls, and 8px for Overlays.
The 4px grid that WinUI uses for its layout - may be the reason for this change in value - it was partially discussed in another issue
Originally posted by @Felix-Dev in https://github.com/microsoft/microsoft-ui-xaml/pull/3514#discussion_r513780900

This looks great but instead of the navigation view like selection for list view, why not highlight the border like what GridView does?

This looks great but instead of the navigation view like selection for list view, why not highlight the border like what GridView does?

Multiple list items beside each other, could look very messy and visually heavy with borders around everything.
It could also make it harder to distinguish between a selected item, and a focused item

How about highlighting with an accent colour like the old design?

How about highlighting with an accent colour like the old design?

I would guess they are trying to create less cluttered approaches to handling list selections, and to better distinguish between item states like Hover and pressed, focused states, and selected states.

For single selection, it's fine but multiple selections look weird

The rounded corners on the ListViewItem don't look good to me:

It looks a bit unprofessional for some reason. Like something for a school classroom.
The density is a problem. ListView is used to display data -- with virtualization a lot of data. I don't want the padding there and the first thing I would do is remove it so a decent number of items can be displayed when needed.
I thought we had this decided a while ago... rounded corners on elements in a list don't look right. If you want to round a list view you can only round the top/bottom/left/right corners of the ListView itself -- not the items.

For the GridView the rounded corners and padding actually work better. That control is generally used to display different types of data and your proposal works there.
I think it's great that Microsoft is looking at refreshed designs. But as always before you finish with the last (actually adopting Fluent 1.x in the entire OS) you seem to be moving on to something else. Is more fragmentation a KPI? :)
This is a good time to mention we need AlternatingRowBackground support like what we had in WPF.

For single selection, it's fine but multiple selections look weird

That may well be valid feedback.  The proposed list view Selected indicator falls in line with controls like the NavigationView and Pivot.  Now does that present a behavioural expectation from a list view?  Would it confuse people to have a ListView with this style, alongside the NavigationView's sidebar?
Xbox's Notification display has the new indicator full height with the ListView Item.

I thought we had this decided a while ago... rounded corners on elements in a list don't look right. If you want to round a list view you can only round the top/bottom/left/right corners of the ListView itself -- not the items.

Providing there is enough spacing between items, a list view of "cards" may not look bad.
Other ways to handle it is a border/separator between items.

List and Grid Views are some of the most customised elements used in apps I would say.  So whatever the default is, should not get in the way of whatever app does with theirs.
Something like Xbox or Netflix would use a card or album art style grid or list - whilst
Outlook would be mostly text, icons, or profile images.
Twitter would be blocks of text.
Do these lists exist in a box within a page, or filling columns of a UI layout?  This would greatly alter how these defaults would look.

@mdtauk Apps are free to customize but after a point, it causes inconsistency. Simple customizations are fine but customization like changing how the selection is done should not be done.

@mdtauk Apps are free to customize but after a point, it causes inconsistency. Simple customizations are fine but customization like changing how the selection is done should not be done.

If a developer has not customised the style or template of their List and Grid Views, these would automatically be the new defaults.
The ListViewItems are just containers displaying the Xaml UI that the dev has chosen.  Selection and states are handled by the ViewItem

The new styles seem to bit larger which can be reduced for density purposes but separate cards look better

It would be nice if as part of this work item we could improve the customization story for the selection checkbox of the GridView. See https://github.com/microsoft/microsoft-ui-xaml/issues/2067 for more details. Depending on how this will be implemented, we might not have to introduce new APIs here.
@anawishnoff Looking at the image below, I am a bit confused:

The second image looks to me as if the selection checkbox for a ListView in multi-selection mode will be replaced with a selection indicator (since multiple items appear to be selected), yet in the image directly above it, the new proposed design is shown with a selection checkbox. So in both images I am supposedly looking at a ListView in multi-selection mode, yet a different selection UI is used in each case. Can you clarify whether a selection indicator or a checkbox will be used?

It would be nice if as part of this work item we could improve the customization story for the selection checkbox of the GridView. See #2067 for more details. Depending on how this will be implemented, we might not have to introduce new APIs here.

Maybe add a GridViewItemCheckmark API where you can replace the default checkbox to add some other kind of element.  Maybe it has a placement option too, which could provide greater customisation without the need to re-template.

Wow, lots of feedback already! Thanks everyone for your suggestions and thoughts. Going to do my best to address all of the open-ended feedback here. I've read all of the comments and have started compiling a list of feedback to take back to the team.
@chingucoding I don't have mockups for the high-contrast view yet, but here's a look at dark mode for ListView and GridView:

@mdtauk

Is the padding/margin between items, something that should be an exposed style value?

This is definitely something that we want to stay customizable with the new design changes. It's currently possible by editing the ListViewItemContainerStyle, and I think that should stay consistent when the new design is introduced.

What about horizontally aligned and scrolling list views with text and other objects, does the selection indicator still go on the left (or right with RtL flow direction)?

This is a great question! It would make sense to change the alignment of the indicator based on the orientation of the overall ListView, similar to how NavigationView does. I'm going to add this to my list to look into/prototype.

Should there still be a style bundled which either falls back to the polls style, or enables the background colour of the list item to be used for selection indication?

We'll be implementing an easy global way to un-round the corners of items, but if someone wants to go back to the original styling, I think that's probably best done by editing the control template directly. We do want people to use the new designs after all 😊.

How do these changes affect selected menu items and toggles? These currently use tick marks and radio bullets.

I think the idea would be to eventually phase these changes out so that all list-type controls use the new designs as well. Like for MenuFlyout, the items would have the rounded backplate on hover, and then have the indicator once selected. I don't think it would affect toggles though - only controls with lists inside them.

The example of a selected item on the right, looks odd with the visual indicator not aligned to the text, thanks to the uneven bottom and top margins/padding. So how would the indicator align to the content presenter, and how are the margins handled on the content?

This is a good callout - honestly I think it's a mistake that the text is not vertically aligned in that mockup, but it brings up a good point! The indicator would align to the center of the content presenter, assuming that text is vertically centered (as I believe is the default alignment in a ListViewItem? correct me if I'm wrong). Creating text that is not center-aligned vertically would be a design choice.

Also should the hover background be flush with the edges of the parent container of the list view. At the moment the background seems to butt up against the selection indicator, but that has some spacing between the edge of the container?

There is intentional spacing between all sides of the ListViewItem and the ListView parent container - this makes the rounded, different-colored backplate stand out more easily imo.
Regarding your comments on XBOX - I've taken it as an action item to look deeper into XBOX scenarios, will get back to you with more details on that one.
@dotMorten

I'd be worried though that this breaks the look of existing apps though, and adds unexpected rounding on things that doesn't look well for the specific design. Is there a guarantee that the corners of the item inside the item template won't "peek out" in the corners but get correctly clipped regardless of the content inside it? (including swap chains)

This is definitely an issue that we're aware of and something I'll be discussing with the engineering side of the team before any implementation. With this as a default design there should be no corner-peeking happening.

The rounding could easily be turned off, if desired, with the ControlCornerRadius ThemeResource.

Please don't do just this. It's not discoverable, and takes a considerable amount of time to find. Instead I'd suggest adding an ItemCornerRadius property to the control. See #1626. It might also affect more controls than you intended (you might have a button inside the item template where you do want rounding).

Thanks for this feedback, I'm definitely trying to figure out a way to make the rounded corners easy to turn off... I think the main thing I'm getting at is that if someone wanted to turn rounded corners off all together for LVs/GVs in their app, they'd use this.  I'm also considering the idea of having more specific ThemeResources - say, GridViewItemCornerRadius and ListViewItemCornerRadius. For a per-control basis, I think it'd make more sense to just edit the control template/corner radii directly? Please let me know your thoughts!
@robloo

The density is a problem. ListView is used to display data -- with virtualization a lot of data. I don't want the padding there and the first thing I would do is remove it so a decent number of items can be displayed when needed.

(answer may also be relevant to @Jaiganeshkumaran, @mdtauk):
I'm actually proposing that the entire footprint of a ListViewItem (i.e. if you drew a rectangle around the entire item) stay the same as previously. The rounded backplate of the item is definitely pushed more inside, but there shouldn't actually be any more padding between each item when it comes down to the rectangular footprint. Does that make sense?

 Seems this should align somehow with the new ListView selection indicator.

It's definitely been difficult to figure out how to upgrade GridView for this reason, since it's often displaying images. For non-image scenarios, I think there would be more noticeable updates. I'll update the issue and post a separate comment with these mockups once I clean some things up.
@Felix-Dev

It would be nice if as part of this work item we could improve the customization story for the selection checkbox of the GridView. See #2067 for more details. Depending on how this will be implemented, we might not have to introduce new APIs here.

I like your proposal here - are you talking about specifically customizing the degree of roundness of the checkbox corners? If so, I think new, specific ThemeResources would be the way to go, I'm just wondering how popular this customization use case would be.

Please don't do just this. It's not discoverable, and takes a considerable amount of time to find. Instead I'd suggest adding an ItemCornerRadius property to the control. See #1626. It might also affect more controls than you intended (you might have a button inside the item template where you do want rounding).

@dotMorten @anawishnoff  This has given me a thought.  Is there some way to bubble up the default lightweight styles for the controls through Autocomplete or some kind of designer UI / Fluent Theme Editor / Figma Plugin - to make these more discoverable.  Now they are being formalised with WinUI 3.X

Looking at this image from the Xbox - the edge of the List Item has a selected indicator, rounded only on one side, and the full height of the item container.

Looking at this image from the Xbox - the edge of the List Item has a selected indicator, rounded only on one side, and the full height of the item container.

It definitely should match up with Xbox.

god the listview updates suck. imagine using file explorer with selected states looking like that. y'all need jeebus

Selected Items in ListView could use a drop shadow like box similar to the Xbox selection outlines. That would make much more sense to see that it is "selected".

Looking at this image from the Xbox - the edge of the List Item has a selected indicator, rounded only on one side, and the full height of the item container.

Looking at the what the list looks like on Xbox again, that's a best way of having a list without "bubbling up" everything. It retains the cleanness of rounded style while keeping the cleanness of the square style.

@anawishnoff Please don't remove default reveal effect for list view items. It's so good and I don't want to see it becoming disabled by default.

@anawishnoff ListView and GridView items come with a tilt effect by default - like buttons. The tilt effect for buttons, however, appears ready to be removed as part of a design update for buttons (see https://github.com/microsoft/microsoft-ui-xaml/issues/242#issuecomment-721299484) I suspect the proposed design update here would also include removing the current default tilt effect for these controls. Is that correct?

@shaheedmalik @mdtauk @nikhil-anilkumar

Looking at this image from the Xbox - the edge of the List Item has a selected indicator, rounded only on one side, and the full height of the item container.

Looking at the what the list looks like on Xbox again, that's a best way of having a list without "bubbling up" everything. It retains the cleanness of rounded style while keeping the cleanness of the square style.

There are definitely similarities to the XBOX design - however I think that rounding the full backplate of the item (i.e. all four corners) is necessary to align it with other WinUI controls that have already been rounded. I'd love to have consistency all around, but this is the first step we'll have to take 😊.

@Felix-Dev - yes, I believe the plans are to remove the tilt animations for ListView and GridView items. ListView items will have a new animation that involves the blue selection indicator.

@mdtauk

Please don't do just this. It's not discoverable, and takes a considerable amount of time to find. Instead I'd suggest adding an ItemCornerRadius property to the control. See #1626. It might also affect more controls than you intended (you might have a button inside the item template where you do want rounding).

This has given me a thought. Is there some way to bubble up the default lightweight styles for the controls through Autocomplete or some kind of designer UI / Fluent Theme Editor / Figma Plugin - to make these more discoverable. Now they are being formalised with WinUI 3.X

The Fluent Xaml Theme Editor comes to mind - it lets you play around with corner radii, colors, themes and the like. Though it would be awesome to have some kind of Intellisense for styles. I think that the first step is to document these built-in styles/resources in our guidance docs so they're more discoverable, and to make sure we think carefully before making something a lightweight style resource vs. a property or new API.
In the rounding ListView/GridView corners case, I think ListViewItemCornerRadius and GridViewItemCornerRadius theme resources make sense, as rounding is a purely stylistic change and these resources make it so you don't have to re-template. And, ListView.CornerRadius and ListViewItem.CornerRadius are properties that also shouldn't be over-written (these can be used to un-round if desired).

May I know whether these changes will come to WinUI 2.x? It would be great to see the new designs in a UWP!

@bdbai It's already come to WinUI 2.x, this is also the default style for Windows 11

I agree that new ListView style needs to come over to WinUI2 and Windows 10 -- I'm not sure if it is even default on Windows 11.
https://github.com/microsoft/microsoft-ui-xaml/issues/6469

@dylech30th thanks for pointing that out! However, I could not see the new appearance in a WinUI 2.7.1 UWP project on Windows 10. Steps to reproduce:

Create a new UWP project in Visual Studio 2022.
Install the latest WinUI 2 package.

Apply WinUI resources to App.xaml.

<Application
    x:Class="App3.App"
    xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
    xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
    xmlns:local="using:App3"
    xmlns:controls="using:Microsoft.UI.Xaml.Controls">
    <Application.Resources>
        <controls:XamlControlsResources>
            <controls:XamlControlsResources.MergedDictionaries>
                <!-- Other app resources here -->
            </controls:XamlControlsResources.MergedDictionaries>
        </controls:XamlControlsResources>
    </Application.Resources>
</Application>

Put a ListView in MainPage.

<Page
    x:Class="App3.MainPage"
    xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
    xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
    xmlns:local="using:App3"
    xmlns:d="http://schemas.microsoft.com/expression/blend/2008"
    xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006"
    mc:Ignorable="d"
    Background="{ThemeResource ApplicationPageBackgroundThemeBrush}"
    xmlns:muxc="using:Microsoft.UI.Xaml.Controls">

    <Grid>
        <ListView SelectionMode="Single">
            <x:String>aa</x:String>
            <x:String>bb</x:String>
            <x:String>cc</x:String>
        </ListView>
    </Grid>
</Page>

Hit F5.

The app turns out to use the legacy style of ListView:

Did I miss anything?

Since I'm using WinUI 3 not 2 so I cannot give a definite answer, but from the official documentation it seems that the control with new designs in WinUI 2 are laid in different namespaces, maybe you need to prefix the control with the correct namespace, in your case is muxc

@bdbai You can follow this tutorial, it explains the steps in detail

@dylech30th I tried to prepend muxc: to ListView, but build failed:
Build started...
1>------ Build started: Project: App3, Configuration: Debug x64 ------
1>C:\Users\bdbai\source\repos\App3\MainPage.xaml(13,10): XamlCompiler error WMC0001: Unknown type 'ListView' in XML namespace 'using:Microsoft.UI.Xaml.Controls'
========== Build: 0 succeeded, 1 failed, 0 up-to-date, 0 skipped ==========
========== Deploy: 0 succeeded, 0 failed, 0 skipped ==========

Anyway, thanks for your help😊

@bdbai

Did I miss anything?

You didn't miss anything. There is no new ListView Style in WinUI 2. I've looked through the styles for this extensively in process of fixing #6469 in an app. I ended up writing my own style based on ComboBoxItem which uses the same selection pills.

@robloo

I ended up writing my own style based on ComboBoxItem which uses the same selection pills.

Cool! I noticed a few controls in WinUI 2 that already adopted the new designs for lists, but never thought of exploiting them as a workaround. Thanks for the creative idea!

@bdbai

Did I miss anything?

You didn't miss anything. There is no new ListView Style in WinUI 2. I've looked through the styles for this extensively in process of fixing #6469 in an app. I ended up writing my own style based on ComboBoxItem which uses the same selection pills.

@robloo I also tried and proved that you are correct, but that really makes me curious about how Xaml Controls Gallery can reveal the correct style for ListView using WinUI 2

Hi,
in winui3 the corners are already rounded. Do you know how to revert it?

We also just hit the issue where the WinUI 2 styles on Win 10 for ListView don't match the updated style. This is particularly odd when trying to run the same code with WinUI 3 as it'll properly use the new style.
There's a behavior difference here as the inbox style default's to HorizontalContentAlignment of Left and not Stretch like the new styles, so it means extra XAML is required for this one combination...
Is there a way when referencing the WinUI 2 package to use the new ListViewItem style or is there some OS limitation here? FYI @chigy

There's a behavior difference here as the inbox style default's to HorizontalContentAlignment of Left and not Stretch like the new styles, so it means extra XAML is required for this one combination...

WinUI 3 was supposed to finally default to the expected stretch and that was heavily discussed in another issue and PR. Sounds like it never got in the WinUI 3 repo?

There's a behavior difference here as the inbox style default's to HorizontalContentAlignment of Left and not Stretch like the new styles, so it means extra XAML is required for this one combination...

WinUI 3 was supposed to finally default to the expected stretch and that was heavily discussed in another issue and PR. Sounds like it never got in the WinUI 3 repo?

@robloo sorry for confusion, WinUI 3 is fine as is WinUI 2 on Win 11 - they use the new style which does have HorizontalContentAlignment to stretch and the new left-hand indicator.
But if you're still using WinUI 2 on Windows 10, then the new style with this stretch property and the indicatior are not used and instead the old Reveal style. Which means as an app developer my app has a different experience still using the old inbox styles even though I'm supposed to get the new style by using WinUI 2.
See here, same app, same XAML:

On the left is WinUI 3, and on the right is WinUI 2 (both on Windows 10). But if you run the WinUI 2 app on Windows 11, it'll look like the WinUI 3 app (as I'd expect). At least I think that's what @niels9001 told me, don't have my Win 11 machine with this code up atm.
In either case, since I expect to get the new updates styles by using WinUI 2, I'd expect the apps design to be identical so I don't have to worry about nuances between Windows 10 and 11 and delivering the same experience to the end-user.
For us with the toolkit, it means we have to explicitly call out setting the HorizontalContentAlignment of the container here in the sample for Windows 10 UWP only... 😢 - If WinUI 2 properly provided the correct default ListViewItem style here (which I feel is the root issue here), then they'd be identical instead.

Same app, same XAML on UWP on W11:

Are there any updates on this matter? Thanks!


Put EOF structure behind an Arc

EOF structure is present inside of Bytecode, and is cloned internally in AccountInfo and other places. It would improve performance to put behind an Arc to avoid cloning the underlying memory.

I'm taking a look at why the tests are failing, it seems to be something else and not the changes made here


Nexus 5x captured image is inverted

When taking a picture using a Nexus 5x the picture is inverted (upside down)
I was looking for a fix regarding this issue here since it's Google sample :)

Are you using this sample's source code to take the picture?

Yes I am.
It looks like doing :
CameraManager manager = (CameraManager) getContext().getSystemService(Context.CAMERA_SERVICE);
CameraCharacteristics cameraCharacteristics = manager.getCameraCharacteristics("" + mCameraDevice.getId());
int rotation = cameraCharacteristics.get(CameraCharacteristics.SENSOR_ORIENTATION);
captureBuilder.set(CaptureRequest.JPEG_ORIENTATION, rotation);
fixes the issue (at least on Nexus 5 and Nexus 5x)

Do you want me to submit a PR with this ?

@keyboardsurfer any progress on this ? I can't figure out how to do this right. I'm getting problem on Galaxy S6 which has inverted picture... :(

Hi, @gbero .
Please use the formula below instead of ORIENTATIONS.get(rotation).
private int getOrientation(int rotation) {
    return (ORIENTATIONS.get(rotation) + mSensorOrientation + 270) % 360;
}

I'm applying this change to the sample soon.

Hi @yaraki,
I've tested your
private int getOrientation(int rotation) {     return (ORIENTATIONS.get(rotation) + mSensorOrientation + 270) % 360; }
method and its not working on Nexus 5x on Back camera. Image is still rotated in final output. If you take portrait image it will save it rotated for 90 degrees. On the other side @gbero 's solution works perfectly on Nexus 5X.

@yaraki I'm getting complaints from users with Nexus 6 that FFC picture is inverted and can't figure out how to handle it. Could Google come up with a sample tested on at least devices like N5, N5x, N6 and N6p both front and rear camera ?

Any update here?

Any news on this front? Having the same issue.

I think I fixed this issue but I'm not 100% sure if this is the best fix. I got the sensor orientation from the camera characteristics and then I just added an ExifInterface with the desired fixed orientation. Like this:
  Integer sensorOrientation = characteristics.get(CameraCharacteristics.SENSOR_ORIENTATION);                         if (sensorOrientation != null) {                             switch (sensorOrientation) {                                 case 90:                                     exif.setAttribute(ExifInterface.TAG_ORIENTATION, String.valueOf(ExifInterface.ORIENTATION_ROTATE_90));                                     break;                                 case 180:                                     exif.setAttribute(ExifInterface.TAG_ORIENTATION, String.valueOf(ExifInterface.ORIENTATION_ROTATE_180));                                     break;                                 case 270:                                     exif.setAttribute(ExifInterface.TAG_ORIENTATION, String.valueOf(ExifInterface.ORIENTATION_ROTATE_270));                                     break;                             }                             exif.saveAttributes();

How to rotate for video capture?

Is there a solution for video preview? Cant find anything to solve it.

Still with this fix. I have an issues with it.
On the Phone the Images looks fine. But when I send them with email to my PC. (In preview they still look fine.) Once it's on my pc , it's rotated -90.
From 3 phones that I have LGG4, Samsung J7 and Sony Experia (something).
Only works on the LGG4 ( and it takes 16:9 images , others take 4:3).
I don't think its Windows fault, because when I capture the image and save it. Then load it on a ImageView , its rotated.

The pictures are not being saved ?

Same for me, it is working fine on Pixel, Oppo, Micromax, Gionee but is rotated -90 degree on Samsung, can any one suggest a fix .. please

I have the same problem. The picture in preview mode is rotated -90. I could not find how to fix this except for checking if it's Samsung phone and then rotating the picture my self. I thought that this Camera 2 api's should be much easier to use, but they introduce so many problems that you did not had with Camera 1. And what is worse, there is no real feedback or solution for this problem, in like one year now, with so many people complaining.

Can somebody tell, How to calculate JPEG_ORIENTATION for front camera? I used (ORIENTATIONS.get(rotation) + mSensorOrientation + 270) % 360; for back camera. which is working just fine.

Not Working for Front Camera.Tested on S5(Samsung).

This sample has been migrated to a new location where we can accept Pull Requests (check README for more information).
As recommended by GitHub, we are closing all issues and pull requests now that this older repo will be archived.
If you still see this issue in the updated repo, please reopen the issue/PR there. Thank you!

This sample has been migrated to a new location where we can accept Pull Requests (check README for more information).
As recommended by GitHub, we are closing all issues and pull requests now that this older repo will be archived.
If you still see this issue in the updated repo, please reopen the issue/PR there. Thank you!


Only one class instance.

The class can not be instantiated for any other instance.
It is not possible to have a second notifier instance running with different purpose.
from win10toast import ToastNotifier as T
t1=T()  #instance 1
t2=T()  #instance 2
Raises class already exists error.

Hi @rgkbitw ,
Sorry for the late response,I was caught up with work. Thank you for your interest in this project.
I have updated the package in Pypi. Could you please test and close the issue?


ACM Controller - Import certificate

Is your feature request related to a problem?
No. It's just a feature that isn't implemented yet.
Describe the solution you'd like
As the importing process takes in a private key which should be considered a secret, it wouldn't be wise to have that as a key straight in the definition of the Kind. Although it's possbie to restrict access to that specific kind with RBAC. But still...
I was thinking it could be interesting to load those information from secrets manager, for example.
So the CR could look like this:
apiVersion: acm.services.k8s.aws/v1alpha1
kind: CertificateImport
metadata:
  name: $name
spec:
  certificateBody:
    fromSecretsManager:
      secretName: my-domain-com-body
  certificatePrivateKey:
    fromSecretsManager:
      secretName: my-domain-com-priv-key
  certificateChain:
    fromSecretsManager:
      secretName: my-domain-com-chain

Describe alternatives you've considered
I could use TF to do this, but it wouldn't be nearly as nice.

Hello @migueleliasweb - loading something from KMS would require giving KMS access to the acm-controller... How about loading the certificates for a kubernetes Secret?

Hi @A-Hilaly , yeah there's that.
Maybe there isn't a huge problem on taking that leap as the controller can run with IRSA anyway so we can finely tune the permissions?
Regarding using a Secret from Kubernetes. I wonder if that would be much different than just inlining all of the required fields in the CertificateImport CR and just rely on K8s RBAC to restrict access. Does that make sense?

Maybe there isn't a huge problem on taking that leap as the controller can run with IRSA anyway so we can finely tune the permissions?

@migueleliasweb If we do that we kinda breach one of the principals of ACK controllers, they ideally need to be as minimalist as possible and require the minimum possible to manage resources in a "single" service. Reading from KMS is kinda breaching this contract.
IMO The ACK-way of handling this case would be to use both kms and acm controllers, have the kms-controller export the certificate to a Kubernetes Secret, then import it from an ACM resource. Not an ACM/KMS expert here - correct me if i'm wrong :))
Also, we could definitely discuss this case during the community meeting with more folks

@migueleliasweb If we do that we kinda breach one of the principals of ACK controllers, they ideally need to be as minimalist as possible and require the minimum possible to manage resources in a "single" service. Reading from KMS is kinda breaching this contract.

I see. Yeah that makes sense.

IMO The ACK-way of handling this case would be to use both kms and acm controllers, have the kms-controller export the certificate to a Kubernetes Secret, then import it from an ACM resource. Not an ACM/KMS expert here - correct me if i'm wrong :))

I actually considered this but with with SecretsManager. Turns out the SecretsManager controller isn't out yet. Personally, I reckon it would be more suitable for this kind of work other than KMS.

Also, we could definitely discuss this case during the community meeting with more folks

Yes, please! Let me know what comes out of it.

Maybe now we have a perfect excuse to put out the SecretsManager controller 👀
apiVersion: acm.services.k8s.aws/v1alpha1
kind: CertificateImport
metadata:
  name: $name
spec:
  certificateBody:
    fromSecretsManagerController:
      secretName: my-domain-com-body
  certificatePrivateKey:
    fromSecretsManagerController:
      secretName: my-domain-com-priv-key
  certificateChain:
    fromSecretsManagerController:
      secretName: my-domain-com-chain

https://github.com/aws-controllers-k8s/community/issues/628

@A-Hilaly sorry to bother but it's been a while. Did anything come out of the meeting?
I wonder if there has been any pushback in implenenting this feature and/or what you it take for us to get some traction here 😉  .

Hi @migueleliasweb , we still didn't discuss this in the community meetings unfortunately. However we plans on creating the secretsmanager controller soon.
Also we can do references between acm and secretsmanager using something like https://github.com/aws-controllers-k8s/lambda-controller/blob/main/test/e2e/resources/alias-ref.yaml#L9-L12

Oh, that's some interesting news regarding the secrets manager controller!
The possibility of referencing a lambda is quite cool but would it still make sense to reference a lambda given there will be a secrets manager controller anyway? I'm just thinking from the users perspective as I'd rather not have to create a lambda to bridge that functionality.

@migueleliasweb I just mentionned an example, not sure referencing lambda will make sense. Here is CertificateImport can look like with current code-generator state:
apiVersion: acm.services.k8s.aws/v1alpha1
kind: CertificateImport
metadata:
  name: $name
spec:
  certificateBodyRef:
    from:
      name: my-domain-com-body
  certificatePrivateKeyRef:
    from:
      name: my-domain-com-priv-key
  certificateChainRef:
    ref:
      name: my-domain-com-chain

We have a use case where a CertificateImport kind would be very helpful.
We manage all of our internal secrets including TLS certificates via Hashicorp Vault. Currently we have some Terraform that runs on a schedule in our CI to generate/renew TLS certs then import them into ACM so they can be used in the ALBs that front the Ingresses of the APIs running on our K8s clusters. We'd really like to improve this workflow as it's a little "janky".
We currently use the External Secrets Operator to sync secrets and Cert Manager to generate/renew certs used within our K8s clusters. It would be great if we could use one of these tools to generate/renew the TLS certs and create them as Secret resources in K8s. Then we could define a CertificateImport resource to push these Secret's into ACM. This would really streamline our setup and remove some serious rough edges.


Unable to access indexes of nested arrays.

My data is formatted in the following manner:
schedules: [
[item 1, item 2, item 3],
[item 1, item 2, item3]
]

When trying to update any item in either the first or second nested array,  you get the following error:
'Only numerical values accepted for array index'.
This code works to access the outer array, but it's not possible to access the inner arrays:
scheduleDB.getData('/schedules[1]')
Code I used for trying to update the third item in the first inner array.
scheduleDB.push('/schedules[1][2]', 'Inactive', True)

Work around is to just overwrite the entire inner array with the same information and only replace the information I want to update, but I'd rather just target the individual item in the nested array.

@Belphemur  I found this repo last night when researching small NodeJS NoSQL type databases.  I found that this implementation was super cool, and will be using it for some small projects.  I took a look through the "issues" section to see if there were any major problems, but didn't find much.  Instead I found a possible feature to add to the project dealing with Nested Arrays.
I spent today walking through the code, and figuring out how this project worked.  I have completed the implementation of allowing the use of nested arrays.  I am working on figuring out how to move the branch I created from the "nested-arrays" branch to the repository to create a PR.

@sidblommerswork Any contribution is more than welcome :)
I wanted to work on adding this feature, so all I did was making the tests first. I haven't made any other progress since that (so many other projects ^^').
I'll be happy to review your PR for this feature.
The usual way in GitHub is to fork the project, push your change to your fork in your own "nested-arrays" branch then submit a PR for me to review and merge.

Thanks!  I will do that shortly.   Thank you for providing the
information.  :)
On Sat, Jun 25, 2022, 8:33 PM Antoine Aflalo @.***>
wrote:

@sidblommerswork https://github.com/sidblommerswork Any contribution is
more than welcome :)
I wanted to work on adding this feature, so all I did was making the tests
first. I haven't made any other progress since that (so many other projects
^^').
I'll be happy to review your PR for this feature.
The usual way in GitHub is to fork the project, push your change to your
fork in your own "nested-arrays" branch then submit a PR for me to review
and merge.
—
Reply to this email directly, view it on GitHub
https://github.com/Belphemur/node-json-db/issues/417#issuecomment-1166392898,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWKXHZ6RCS3GVB4GNZHUALTVQ6XNZANCNFSM5YLXECSQ
.
You are receiving this because you were mentioned.Message ID:
@.***>


Admin User Interface Investigation

vc-authn-oidc needs several different configurations to be applied when a new client is registered to authenticate using it.
Currently, the configurations are being set manually in the database: it would be great to have a user interface that allowed users to self-serve and apply their configurations.
There may be different options online, including:

https://github.com/skoruba/IdentityServer4.Admin
https://github.com/IdentityServer/IdentityServer4.Quickstart.UI
https://github.com/brunohbrito/JPProject.IdentityServer4.AdminUI

Investigate capabilities and effort to integrate a UI with vc-authn-oidc.

Added a third option for Admin UI. So far they all seem to be single tenant - as far as I can tell - and be somewhat short of what we need or overly complicated.
Trying a few different options to see if something stands out.

All of the above solutions do not seem to fit our needs:

1 is not an actual admin UI, but a samle UI for login/logout using IdentityServer as IdP
2 and 3 seem to target IdentityServer4 v3 rather than the latest v4 as far as I can understand, and are therefore not compatible/usable

There is a [paid admin UI]https://www.identityserver.com/products/adminui) product, however it doesn't appear to be multi-tenanted from what I can see from the few screenshots and there is no way of knowing more without booking a demo. The price tag is also pretty steep so I am not sure that's where we want to be going anyway...


mon, osd: fix skipped condition

To be properly evaluated the "skipped" conditions must always have the
first place on the list of condition, otherwise the other conditions are
evaluated before and make the task fail.
Signed-off-by: Sébastien Han<EMAIL_ADDRESS>

@leseb You probably want to add Fixes: #1733 to your commit message here.

@fghaas right, thanks!


Text Search For DXF and DGN

Hello,
Is it possible to do a text search on the dxf and dgn formats?  You have an example for the dwg format.  If so, is there an example?  It would seem that the example for dwg doesn't work in this case.

@gmalenko ,
We are checking requirements on our end and will share the feedback with you as ASAP.

@gmalenko ,
Can you please try using the sample code over following Git link on your end for DXF file as well and share if it works for your not. I have tried the same sample code on my end for a DXF and code executed without any issue.
https://github.com/aspose-cad/Aspose.CAD-for-.NET/blob/master/Examples/CSharp/DWG-Drawings/SearchText.cs

@gmalenko ,
Can you please try using the sample code over following Git link on your end for DXF file as well and share if it works for your not. I have tried the same sample code on my end for a DXF and code executed without any issue.
https://github.com/aspose-cad/Aspose.CAD-for-.NET/blob/master/Examples/CSharp/DWG-Drawings/SearchText.cs


feat: use relative URLs internally

This is an alternative approach to that taken in #2212. It solves the same problem, however it may be more flexible. It may have other issues particularly with regard to URLs for directories that don't have a trailing slash, I don't think any such URLs are generated internally though

Hmm, looking into the CI failures

Seems to break on integration tests, specifically with requirejs. Maybe better to go with the more explicit fix in #2212 at least that change does not change anything if the option is not used. This relative url approach does change the behaviour subtly in all cases

Not sure why the AppVeyor build fails, seems to bomb out after npm install and before doing anything. Doesn't happen on the other branch though :s

I agree with you lets go with #2212 that seems to be the safer approach and makes sure that things don't suddenly change for users.

Cool, closing this one :)


Terraform-docs behaving differently on different machines.

Terraform-docs behaving differently on different machines.
We use pre-commit-terraform in our project.
I run the terraform_docs hooker both from a docker container and MacOS.
When running pre-commit, MacOS wants to modify the documentation in the provider section:
-|  [google](#provider\_google) | = 4.19.0 |
+|  [google](#provider\_google) | 4.19.0 |

While in docker it is the opposite
-|  [google](#provider\_google) | 4.19.0 |
+|  [google](#provider\_google) | = 4.19.0 |

This doesn't happen in all the folders where I have a GCP project but only in some of them.
My hook is:
-   repo: https://github.com/antonbabenko/pre-commit-terraform
    rev:  v1.77.0 # Get the latest from: https://github.com/antonbabenko/pre-commit-terraform/releases
    hooks:
    -   id: terraform_docs
        args:
        - --hook-config=--create-file-if-not-exist=true
        - --hook-config=--path-to-file=TFDOC.md

How can we reproduce it?
Run terraform pre-commit hook for terraform_docs in a Docker container and then in MacOS using google as provider.
Environment information
OS: Docker container Win10 with Ubuntu 20.04 on WSL2
OS: MacOS

Tools availability and versions:

GNU bash, version 5.0.17(1)-release (x86_64-pc-linux-gnu)
pre-commit 2.15.0
Terraform v1.1.9
python SKIPPED
Python 3.8.10
checkov 2.0.888
terraform-docs version v0.16.0 1f686b1 linux/amd64
terragrunt version v0.37.1
terrascan terrascan SKIPPED
TFLint version 0.34.1
tfsec tfsec SKIPPED
tfupdate tfupdate SKIPPED
hcledit hcledit SKIPPED

@RiccardoGrigoletto Could you please provide contents of your Terraform required_providers sections from MacOS and from Docker from folders where you observe this behavior? And also versions of terraform-docs from MacOS and from Docker. Thanks.

@yermulnik thank you for the response, updated the description :)

Hi

Just stop specifying version like version = "= 4.19.0"
Use version = "4.19.0" and issue should gone and readability will be improved

https://developer.hashicorp.com/terraform/language/expressions/version-constraints

Rebuild the docker image to match the same version as in MacOS (except Python), now it's slightly outdated


[AMHACK] [UI] Critical usability issue in plotting the API usage stats

Description:
Inconsistency in plotting the API subscriptions count in the 'Overall API usage' graph.
Suggested Labels:
Bug
Critical
APIM Analytics
Affected Product Version:
APIM wum updated - wso2am-<IP_ADDRESS>0928473100.zip
APIM analytics - wso2am-analytics-<IP_ADDRESS>0935606971.zip
OS, DB, other environment details and versions:
Standalone
Ubuntu 15.10
JDK - jdk1.8.0_66
DB - Postgres 9.6.3
Browser - Chrome Version 58.0.3029.96 (64-bit)
Pre -requisites

Several APIs should exist with subscriptions

Steps to reproduce:

Go to Publisher console -> Analytics -> API

Observations

There is huge inconsistency among the points reflected in the graph.
Some of the points are not even clearly visible

IMO this is a critical UX issue.

This is not a UI bug. This is a bubble chart where the size of the bubble represents the usage count. So the reason for the difference sizes of bubble for each api is depending on their usage count.


google.query is not a function

TypeError: google.query is not a function
at module.exports (C:\Users\Torakami\Documents\GitHub\lootmogul\gameplatform-cms\src\extensions\users-permissions\services\providers-list.js:118:10)
at getProfile (C:\Users\Torakami\Documents\GitHub\lootmogul\gameplatform-cms\src\extensions\users-permissions\services\providers.js:38:11)
at processTicksAndRejections (node:internal/process/task_queues:96:5)
This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). The promise rejected with the reason:
Error: connect ECONNREFUSED <IP_ADDRESS>:80
at TCPConnectWrap.afterConnect [as oncomplete] (node:net:1187:16)

Can't really tell without seeing your code.

Feel free to re-open the issue if you have additional info to share.


octokit.pulls.listFiles send wrong url to the server

Bug Report
Current Behavior
After upgraded the probot version to the newest one 11.0.5, I tried to use octokit.pulls.listFiles to get all the files in the PR. But it returns me 404 not found.
  const res = await context.octokit.pulls.listFiles({
    owner,
    repo,
    number,
    per_page
    page,
  });

error
RequestError [HttpError]: Not Found
    at /Users/yihe.guan/Documents/GitHub/pull-request-size/node_modules/@octokit/request/dist-node/index.js:66:23
    at processTicksAndRejections (internal/process/task_queues.js:93:5)
    at async sendRequestWithRetries (/Users/yihe.guan/Documents/GitHub/pull-request-size/node_modules/@octokit/auth-app/dist-node/index.js:440:12)
    at async Job.doExecute (/Users/yihe.guan/Documents/GitHub/pull-request-size/node_modules/bottleneck/light.js:405:18) {
  status: 404,
  headers: {
    'access-control-allow-origin': '*',
    'access-control-expose-headers': 'ETag, Link, Location, Retry-After, X-GitHub-OTP, X-RateLimit-Limit, X-RateLimit-Remaining, X-RateLimit-Used, X-RateLimit-Reset, X-OAuth-Scopes, X-Accepted-OAuth-Scopes, X-Poll-Interval, X-GitHub-Media-Type, Deprecation, Sunset',
    connection: 'close',
    'content-encoding': 'gzip',
    'content-security-policy': "default-src 'none'",
    'content-type': 'application/json; charset=utf-8',
    date: 'Tue, 26 Jan 2021 07:29:42 GMT',
    'referrer-policy': 'origin-when-cross-origin, strict-origin-when-cross-origin',
    server: 'GitHub.com',
    status: '404 Not Found',
    'strict-transport-security': 'max-age=31536000; includeSubdomains; preload',
    'transfer-encoding': 'chunked',
    vary: 'Accept-Encoding, Accept, X-Requested-With',
    'x-content-type-options': 'nosniff',
    'x-frame-options': 'deny',
    'x-github-media-type': 'github.v3; format=json',
    'x-github-request-id': 'E8A9:1D91:35F36A:8E6400:600FC4E5',
    'x-ratelimit-limit': '6700',
    'x-ratelimit-remaining': '6623',
    'x-ratelimit-reset': '1611647607',
    'x-ratelimit-used': '77',
    'x-xss-protection': '1; mode=block'
  },
  request: {
    method: 'GET',
    url: 'https://api.github.com/repos/owner/test_repo/pulls//files?number=33&per_page=100&page=1',
    headers: {
      accept: 'application/vnd.github.v3+json',
      'user-agent': 'probot/11.0.5 octokit-core.js/3.2.5 Node.js/14.15.1 (darwin; x64)',
      authorization: 'token [REDACTED]'
    },
    request: { hook: [Function: bound bound register] }
  },
  documentation_url: 'https://docs.github.com/rest'
}

Environment

Probot version(s): [11.0.5]

The URL in the request seems wrong
url: https://api.github.com/repos/owner/test_repo/pulls//files?number=33&per_page=100&page=1
Why there are two slashes after pull: "pulls//files?". is it expected?

It’s pull_number now, not number. You should have seen a deprecation message for that, there might be others. It’s all mentioned in the v11 release notes

It’s pull_number now, not number. You should have seen a deprecation message for that, there might be others. It’s all mentioned in the v11 release notes

It’s pull_number now, not number. You should have seen a deprecation message for that, there might be others. It’s all mentioned in the v11 release notes

Thank you so much. I was blind. It worked now.

It’s pull_number now, not number. You should have seen a deprecation message for that, there might be others. It’s all mentioned in the v11 release notes

Thank you so much. I was blind. It worked now.


SystemUI: fix black scrim when turning screen on from AOD

TODO: Maybe add same fade anim as pixel 2

Change-Id: I351a212b2989f56df3b6353f41a379281ed1a9f1
Signed-off-by: Simao Gomes Viana<EMAIL_ADDRESS>

Push your commit to our gerrit..
gerrit.resurrectionremix.com


Why does acl-public carry signature and and access key?

https://****.s3.amazonaws.com/vzbit/uploads/link/poster/18/4b4af9cc-3f4d-4ef1-878b-ad80a5866ca2.jpg?AWSAccessKeyId=AKIAJ43P3YSDPBMWFV5A&Expires=1441228284&Signature=k%2FDQ3IQjP8b1d5PkR87wc62MJug%3D

I dont see it being needed at all. How does signing the link affect performance?

The access key, expiration and signature are all added by the aws-sdk gem as part of the public_url method There is a different implementation of this between v1 and v2 of the aws-sdk gems. Here is the location for v1:
vendor/ruby/2.2.0/gems/aws-sdk-v1-1.64.0/lib/aws/s3/s3_object.rb
1277:      def public_url(options = {})

And here is the location for v2:
vendor/ruby/2.2.0/gems/aws-sdk-resources-2.0.48/lib/aws-sdk-resources/services/s3/object.rb
55:      #     s3.bucket('bucket-name').object('obj-key').public_url
59:      def public_url

The v2 implementation doesn't add a signature at all. If upgrading to v1.0.0.pre.rc.1 is an option for you I'd recommend upgrading.

Sorry, I should have asked the app version as well. Be sure you are on v0.7.1, it includes an important fix that helps to respect aws_acl value.

Do I specify the version on my gemfile or should I be pointing it to a branch?

Also, my bundle says it is currently using 0.7.1
Using carrierwave-aws 0.7.1

I am switching to use the rc1 tag

And no dice, I am seeing the same behavior

Will you please share your carrierwave configuration and any uploader specific overrides you are using?

# config/initializers/carrierwave.rb
CarrierWave.configure do |config|
  config.storage    = :aws
  config.aws_bucket = Rails.application.secrets.aws_bucket
  config.aws_acl    = 'public-read'

  # Optionally define an asset host for configurations that are fronted by a
  # content host, such as CloudFront.
  # config.asset_host = 'http://example.com'

  # The maximum period for authenticated_urls is only 7 days.
  # config.aws_authenticated_url_expiration = 60 * 60 * 24 * 7

  # Set custom options such as cache control to leverage browser caching
  config.aws_attributes = {
    # expires: 1.week.from_now.httpdate,
    cache_control: 'max-age=604800'
  }

  config.aws_credentials = {
    access_key_id:     Rails.application.secrets.aws_access,
    secret_access_key: Rails.application.secrets.aws_secret,
    region:            'us-east-1'
  }

  # Optional: Signing of download urls, e.g. for serving private
  # content through CloudFront.
  # config.aws_signer = -> (unsigned_url, options) { Aws::CF::Signer.sign_url unsigned_url, options }
end

# /app/uploaders/page_meta_uploader.rb
# encoding: utf-8

class PageMetaUploader < CarrierWave::Uploader::Base

  # Include RMagick or MiniMagick support:
  # include CarrierWave::RMagick
  # include CarrierWave::MiniMagick

  # Choose what kind of storage to use for this uploader:
  storage :aws

  # Override the directory where uploaded files will be stored.
  # This is a sensible default for uploaders that are meant to be mounted:
  def store_dir
    "vzbit/uploads/#{model.class.to_s.underscore}/#{mounted_as}/#{model.id}"
  end

  # Provide a default URL as a default if there hasn't been a file uploaded:
  # def default_url
  #   # For Rails 3.1+ asset pipeline compatibility:
  #   # ActionController::Base.helpers.asset_path("fallback/" + [version_name, "default.png"].compact.join('_'))
  #
  #   "/images/fallback/" + [version_name, "default.png"].compact.join('_')
  # end

  # Process files as they are uploaded:
  # process :scale => [200, 300]
  #
  # def scale(width, height)
  #   # do something
  # end

  # Create different versions of your uploaded files:
  # version :thumb do
  #   process :resize_to_fit => [50, 50]
  # end

  # Add a white list of extensions which are allowed to be uploaded.
  # For images you might use something like this:
  def extension_white_list
    %w(jpg jpeg gif png)
  end

  # Override the filename of the uploaded files:
  # Avoid using model.id or version_name here, see uploader/store.rb for details.
  def filename
    "#{secure_token}.#{file.extension}" if original_filename.present?
  end

  protected
  def secure_token
    var = :"@#{mounted_as}_secure_token"
    model.instance_variable_get(var) or model.instance_variable_set(var, SecureRandom.uuid)
  end

end

Pretty much doing nothing custom except the file renaming :/

@sorentwo  After some more testing ( and multiple restarts ) , the signed urls are gone! :)

Glad to hear it!

Thank you very much! Lemme know if I can help in anyway :)

I'm getting the same thing, all my uploads are public but every url has AWSAccessKeyId, Expires and Signature parameters. I'm on Heroku.

Since you are using 0.7.1 with aws-sdk 1.x you should try "public_read" instead of "public-read".
If you can, I suggest using the 1.x release of carrierwave-aws.

Ok, switched to 1.0.0 rc1 all good, sorry I really should have just done that first :)


Update Portfolio: Reporting page code to work with changes to augur.js

Clubhouse Story
Earlier today, I submitted https://github.com/AugurProject/augur.js/pull/368, which separates the old augur.reporting.claimReportingFees function into 2 new functions: augur.reporting.claimReportingFeesNonforkedMarkets and augur.reporting.claimReportingFeesForkedMarket.  The former is what will now be used when the user clicks the Claim button on the Portfolio: Reporting page.  (The latter will be used for the forked market card, which will be added later.)
Currently, there seems to be a bug in augur.reporting.getReportingFees that is causing it to return 0 for the unclaimed ETH/REP a user has, even if the user has submitted an initial report or disputed an outcome. As a result, going to the Portfolio: Reporting page currently shows:
REP   -
ETH   -

for the unclaimed ETH/REP totals, and the Claim button is grayed out. This should get resolved once the bug in augur.reporting.getReportingFees is fixed.
Submitter checklist

[ ] Test covering functionality added/fixed in
Check the linked story includes the following:

[ ] Steps to reproduce
[ ] Screenshot (If applicable)

Reviewer Checklist

[ ] Test/lint pass

Does this PR depend on other PRs in other Repos?

Yes, it depends on the augur.reporting.claimReportingFeesNonforkedMarkets function, which was created in https://github.com/AugurProject/augur.js/pull/368. I would wait to test/review this PR until the other change is merged in, since this PR will break things if the other PR isn't merged in first.

Closing this PR, as the code changes for it are also included in the bigger PR at https://github.com/AugurProject/augur/pull/1578.


Add Option to disable Analytics

While one might have the impression that contacting scripts.simpleanalyticscdn.com is just privacy-friendly and does not require user-consent, one might want to disable this completely to ensure the trust of more skeptical users.
Sending any data to some other domain might be considered a GDPR violation (see also the extreme example of Google Fonts).
So I would like to have this disabled as well when using VITE_APP_ENABLE_TRACKING=False (which I would have expected to be opt-in due to the renaming, but the default seems to be True as well - which could also be changed?

Agree on this - VITE_APP_ENABLE_TRACKING is not truly enough to disable all telemetry. Some of us are deploying excalidraw in corporate environments where any external call is unacceptable, privacy-preserving or not.


[ign -> gz] CMake functions

Part of https://github.com/gazebo-tooling/release-tools/issues/719

Fuel tools is expected to fail still, I think