Registry file visualizer application

Description of the new feature / enhancement
Wouldn't it be great to see complex reg files in a visual manner?
Here is an early POC by @randyrants

Scenario when this would be used?
Any time you need to apply a reg file before you actually do it?
Supporting information
No response

https://github.com/microsoft/PowerToys/tree/feature/regPreview will coding area

https://github.com/microsoft/PowerToys/pull/23709 has initial code drop

I suggest to implement the syntax highlighting definition from our Monaco preview pane.
cc: @Aaron-Junker

Lets get the base code compiling, style cop goodness, integrated into PowerToys, THEN do some extra polish. :)
Right now trying to run the code even before doing the analyzer work, i'm seeing a "System.DllNotFoundException: 'Unable to load DLL 'Microsoft.ui.xaml.dll' or one of its dependencies: The specified module could not be found. (0x8007007E)'"

I suggest to implement the syntax highlighting definition from our Monaco preview pane.
cc: @Aaron-Junker

I agree with @crutkas that this doesn't have a big priority, but it's definitively something that can be implemented later. I like the idea!

Can we implement support for registry.pol file?

registry.pol
Viable feature request, but it will take some doing.  It looks like registry.pol files are not clear text so it's not a direct "just open the POL files" thing.  That said, there have been other tools that can read the format directly or even convert the contents to a REG file, so it should be possible with enough knowledge of the file format to enable previewing and possibly even edits.

Right now trying to run the code even before doing the analyzer work, i'm seeing a "System.DllNotFoundException: 'Unable to load DLL 'Microsoft.ui.xaml.dll' or one of its dependencies: The specified module could not be found. (0x8007007E)'"

Two different issues that are causing this error While I was working on the app, this project lived alongside a Package project - handy for APPX/APPXBUNDLE creation - which is how the application was run while in the IDE.  I think my thinking at the start of my coding was that this would be the easiest way to send it to the Store, if I wanted to, but I opted on a different avenue.
That choice introduced one issue and one dependency I didn't code for:

The csproj file had to be updated to be self-contained.  Since this is usually done at project creation, I spelunked into one of the apps here and copied some Properties, which enables the application to be compiled and run from the IDE, without having needing anything else.
The code would load a few different bits of data assuming that the app would be run in an ApplicationContainer.  Since this is no longer the case, a few of those assumptions had to be corrected.

One side effect of this - and I'm open to suggestions - is that I hard coded the GUID the app was using while in a package.  If the app stays on this path, this can be changed to a different location in the AppData\Local path or at least into a name that is more readiable.
I've pushed the update to this branch, so it should be visible in the most recent commit: https://github.com/microsoft/PowerToys/pull/23709/commits/80bf8891dd1c8b1af2f3abdac9dc0ef75952eaae

I agree that this isn't a high priority, but it can definitely be incorporated in the future. I like the concept!

@crutkas
I think this can be closed as released.


Permission Denied when running scripts/bootstrap.sh

root@in_dev_docker:/apollo# sudo bash scripts/bootstrap.sh
scripts/bootstrap.sh: line 30: ./scripts/monitor.sh: Permission denied
scripts/bootstrap.sh: line 31: ./scripts/dreamview.sh: Permission denied
root@in_dev_docker:/apollo# bash scripts/bootstrap.sh
scripts/bootstrap.sh: line 30: ./scripts/monitor.sh: Permission denied
scripts/bootstrap.sh: line 31: ./scripts/dreamview.sh: Permission denied

find . -name ".sh" | xargs chmod a+x
find . -name ".bash" | xargs chmod a+x

find . -name ".sh" | xargs chmod a+x
find . -name ".bash" | xargs chmod a+x

@Eclipsehelio Hi,
Thank you. solve this problem. Now dreamview webpage shows nothing. Module monitor and dreamview are running. I have followed previous issues already, but it still fails to show.
My host server is Ubuntu 18.04
root@in_dev_docker:/apollo# sudo bash scripts/bootstrap.sh Module monitor is already running - skipping. Module dreamview is already running - skipping.

@michaelowenliu Sorry for the delayed response.
Are you visiting dreamview on another computer?
What's the web browser you are using(recommended to use firefox)?
I think you should use wget command to verify that dreamview is actually running in the docker container firstly, just like this:
wget localhost:8888
As the first step is ok, then you could do it outside the container.
Or all the above steps do failed, could you upload the dreamview's log(/apollo/data/log/)?

@michaelowenliu Sorry for the delayed response.
Are you visiting dreamview on another computer?
What's the web browser you are using(recommended to use firefox)?
I think you should use wget command to verify that dreamview is actually running in the docker container firstly, just like this:
wget localhost:8888
As the first step is ok, then you could do it outside the container.
Or all the above steps do failed, could you upload the dreamview's log(/apollo/data/log/)?

@Eclipsehelio Hi, after updating the repo, it works. Thank you.

monitor.log.info.20190819-174547.13339
Log file created at: 2019/08/19 17:45:47
Running on machine: in_dev_docker
Log line format: [IWEF]mmdd hh:mm:ss.uuuuuu threadid file:line] msg
I0819 17:45:47.556493 13339 class_loader_utility.h:78]  registerclass:Monitor,apollo::cyber::ComponentBase,/apollo/bazel-bin/modules/monitor/libmonitor.so

log.tar.gz
这是相关的日志


[Master] Fix 'dapr list' to include cli PID and info

Description
Fix 'dapr list' to include cli PID and info
Issue reference
We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.
Please reference the issue this PR will close: #296
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

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

I am testing on Windows to make sure. Please, do not merge until I confirm it.

Do not merge my master into branch until the hotfix is merged first.

Works on windows too.


Retry mechanism for failed push [$15]

If the push failed because some changes from remote were not fetched, we should try to fetch, merge them and in the end, retry the push.

---
There is a **[$15 open bounty](https://www.bountysource.com/issues/26277156-retry-mechanism-for-failed-push?utm_campaign=plugin&utm_content=tracker%2F7081451&utm_medium=issues&utm_source=github)** on this issue. Add to the bounty at [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F7081451&utm_medium=issues&utm_source=github).

Sample log Jan 27 15:38:04 sftp-5 GitFS:  on /home/dragosroua/connecthubro [12571]: SyncWorker: Push failed Traceback (most recent call last):   File "/root/.pex/install/gitfs-0.3.3-py2-none-any.whl.9e0c81d540e5018c3766cced5c105864e35d8bd4/gitfs-0.3.3-py2-none-any.whl/gitfs/worker/sync.py", line 122, in sync     self.repository.push(self.upstream, self.branch)   File "/root/.pex/install/gitfs-0.3.3-py2-none-any.whl.9e0c81d540e5018c3766cced5c105864e35d8bd4/gitfs-0.3.3-py2-none-any.whl/gitfs/repository.py", line 119, in push     remote.push(["refs/heads/%s" % (branch)])   File "/root/.pex/install/pygit2-0.22.0-cp27-none-linux_x86_64.whl.6071446922e1dec09888986ed337431aa6163780/pygit2-0.22.0-cp27-none-linux_x86_64.whl/pygit2/remote.py", line 322, in push     check_error(err)   File "/root/.pex/install/pygit2-0.22.0-cp27-none-linux_x86_64.whl.6071446922e1dec09888986ed337431aa6163780/pygit2-0.22.0-cp27-none-linux_x86_64.whl/pygit2/errors.py", line 64, in check_error     raise GitError(message) GitError: Cannot push because a reference that you are trying to update on the remote contains commits that are not present locally.


Add NC Multitool and TAug Impetus Linker to Morphotool

also alias QMD to Nuclearcraft (tells auto-morph to select Multitool on QMD blocks)

Thanks!


http server address and port should be configurable

In https://github.com/chrislusf/glow/blob/master/driver/rsync/http_server.go#L91 is a random port selected which makes it very hard when access is restricted via firewalls or in a container environment where ports are exposed.
The remote address which is used with the heartbeat can be a local one. Then it's impossible for any client outside to connect. I had these problems when I tried to connect to a local docker cluster from the host system.

this was actually on my to do list. Do you want to pick it up? Just make the port number default to 0. Just let me know if it involves too much changes.

@chrislusf This isn't a blocker for me at the moment. I just wanted to make a note to not forget about it.

fixed now.


Enable operating on microvalues in addong resizer

@gmarek

cc @Q-Lee

Looks good, please add a test for exponential estimator (should return 0.308) and some explanation to the PR why do need it (something like this comment)

@piosz - comment applied; PTAL

LGTM


#5772 nsfw changes in gallery

Thank you for your contribution to the KodaDot - One Stop Shop for Polkadot NFTs.
👇 __ Let's make a quick check before the contribution.
PR Type

[ ] Bugfix
[x] Feature
[ ] Refactoring

Context

[x] Partially Related #5772
[ ] Requires deployment <snek/rubick/worker>

Before submitting pull request, please make sure:

[x] My contribution builds clean without any errors or warnings
[x] I've merged recent default branch -- main and I've no conflicts
[x] I've tried to respect high code quality standards
[x] I've didn't break any original functionality

Optional

[ ] I've tested it at </ksm/collection>
[ ] I've tested PR on mobile
[ ] I've written unit tests 🧪
[ ] I've found edge cases

Had issue bounty label?

[x] Fill up your KSM address:
Payout

Community participation

[x] Are you at KodaDot Discord?

Screenshot 📸

[x] My fix has changed something on UI; a screenshot is best to understand changes for others.

Copilot Summary

🤖 Generated by Copilot at 4aab639
This pull request adds support for NSFW media items in the gallery, by blurring them and allowing the user to toggle the blur. It modifies the GalleryItem, MediaItem, and GalleryItemDescription components to use the NSFW trait as a prop and an icon.

🤖 Generated by Copilot at 11e5d62

Blinded by the lewdness, you can't see the truth
isLewd is the judge, the traitor and the proof
Reveal the hidden content, with a click of fate
But beware of the console.log, it might be too late

Leftovers:

[ ] Adding "hide all content tagged with nsfw" in settings, I am not sure how would we filter this in explore
[ ] NSFW Cards - again not sure where to get nsfw values for cards since it isnt done yet for nftentities

@kodadot/qa-guild

so this doesn't blur the home page preview but only if you click on an NFT?
it works fine when clicking on a NSFW NFT but does nothing

so this doesn't blur the home page preview but only if you click on an NFT?
it works fine when clicking on a NSFW NFT but does nothing

yeah for explore we need https://github.com/kodadot/rubick/issues/173 and https://github.com/kodadot/loligo/issues/22 to clear first

can we merge this?

pay 45 usd

😍 Perfect, I’ve sent the payout
💵 $45 @ 26.01 USD/KSM ~ 1.73 $KSM
🧗 EzGc4s9PgCPx1YnF3fqzhLzVHpHMTL4LWPScwpDrR8JKgSU
🔗 0x93d2e570a32ec205a3f73ed52ea55bc68c3af4c89ccd6f3f1234b9d770105d39
🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues


Cluster Migration : failed to check s3 bucket: invalid header field value

Hi,
I'm having trouble trying to migrate my cluster from RKE to EKS.
I followed the instructions of this documentation https://rancher.com/docs/rancher/v2.x/en/backups/v2.5/migrating-rancher/.
After applying the restore ressource on the target cluster, I've got this status message on the ressource :
    Message:              failed to check s3 bucket:s3-rancher2-backups, err:Head https://<bucket_name>.s3.dualstack.eu-west-3.amazonaws.com/: net/http: invalid header field value "AWS4-HMAC-SHA256 Credential=\x00\xa2\x00X\x1d\x96@.\xc0,>\x8b@\xa1C/20201104/eu-west-3/s3/aws4_request, SignedHeaders=host;x-amz-content-sha256;x-amz-date, Signature=<redacted>" for key Authorization

Here is the logs of the rancher-backup pod (this is looping endlessly)
INFO[2020/11/04 10:00:34] Processing Restore CR restore-migration
INFO[2020/11/04 10:00:34] Restoring from backup xxx.tar.gz
INFO[2020/11/04 10:00:34] invoking set s3 service client                s3-accessKey="\x00\xa2\x00X\x1d\x96@.\xc0,>\x8b@\xa1C" s3-bucketName=<bucket_name> s3-endpoint=s3.eu-west-3.amazonaws.com s3-endpoint-ca= s3-folder= s3-region=eu-west-3

I guess there is something buggy with ths accesskey ?
I added the aws credentials following the instructions, with accesskey and secretkey in plain text.
Here is the ressource I applied :
---
apiVersion: resources.cattle.io/v1
kind: Restore
metadata:
  name: restore-migration
spec:
  backupFilename: xxx.tar.gz
  prune: false
  storageLocation:
    s3:
      credentialSecretName: s3-creds
      credentialSecretNamespace: default
      bucketName: <bucket_name>
      region: eu-west-3
      endpoint: s3.eu-west-3.amazonaws.com

And the secret
---
apiVersion: v1
kind: Secret
metadata:
  name: s3-creds
type: Opaque
data:
  accessKey: ABCDABCDABCD
  secretKey: ABcdAbcd

I tried to remove these and re-apply without success, I also tried to drop the whole cluster and re-create it, without success.
I may have miss something but I can't find what :(
Thanks for your help !

If anyone passes by and sees this, the problem was that the secret were created from a yaml file, I just re-created the secret like so :
kubectl create secret generic s3-creds --from-literal=accessKey=ABCDABCD --from-literal=secretKey=AbcdAbcd

and everything worked flowlessly.

Sorry for bothering, thanks for your work !

The values for secrets are always base64-encoded.  --from-literal does that for you, putting them directly into yaml doesn't.

i confirm this works. for some reason when i make the secret from the yaml and base64 encode the keys myself it doenst work. seems you have to use the literal through kubectl


4.2dev6 GraphNode no close button and close_request signal

Godot version
Godot 4.2 dev6
System information
MacOS 10.15.7 (19H2) Intel Iris Graphics 6100 1536 MB
Issue description
There is a show_close option on GraphNode in 3.X ，but cannot find a counterpart in 4.2dev6. And there is no close_request signal on GraphNode while the latest 4.2 Document said there is a close_request() signal in GraphElement.
How to handle this issue?
Steps to reproduce
no content
Minimal reproduction project
no content

Seems these were removed intentionally in #79311, see: https://github.com/godotengine/godot/pull/79311#issuecomment-1701027224. cc @Geometror

The button was removed intentionally. You can add it if you need it using the new HBoxContainer used for the titlebar.
And the signal has been renamed to delete_request, the online documentation is just slightly out of date with all the recent changes.

The button was removed intentionally. You can add it if you need it using the new HBoxContainer used for the titlebar.
And the signal has been renamed to delete_request, the online documentation is just slightly out of date with all the recent changes (we usually synchronize it over the weekend). You can find the most accurate documentation in the class reference directly in the editor.

Thank you so much~


docs: create narrative documentation

What I did
create some docs surrounding writing plugins
How I did it
added some rst files in docs folder.
How to verify it
$ cd ape/docs && python3 build_docs.py --rsync=/tmp/ape && python3 -m http.server --directory /tmp/
then open http://localhost:8000/ape/latest/
Checklist

[x] Passes all linting checks (pre-commit and CI jobs)
[ ] New test cases have been added and are passing
[x] Documentation has been updated
[x] PR title follows Conventional Commit standard (will be automatically included in the changelog)

Some general feedback: This article does great at telling me how the internals of the plugin system work, which I may or may not be interested in.
Here are something I think are missing that important, mainly from the viewpoint of an external developer coming here with questions when they are trying to implemented a plugin:

The plugin project template from GitHub and the benefits of using it.
Simple python code (code blocks) showing some of the methods being implemented, including the import statements.
Links to actual plugins implementing these APIs.
Any "gotchas". For example, the accounts _cli.py module is responsible for all account management and users should not count on ape accounts commands applying to their account plugin, even if the logic is similar. Another one is that not merged yet would be that provider plugins that use web3 will be responsible for raising some form of VirtualMachineError when sending transactions (or making calls) fails.

Is this an appropriate list of the methods that I need to add docstrings to?
https://github.com/ApeWorX/ape/blob/main/src/ape/api/__init__.py
    "AccountAPI",
    "AccountContainerAPI",
    "Address",
    "AddressAPI",
    "ContractInstance",
    "ContractLog",
    "ConverterAPI",
    "EcosystemAPI",
    "ExplorerAPI",
    "ProviderAPI",
    "ProviderContextManager",
    "NetworkAPI",
    "ReceiptAPI",
    "TransactionAPI",
    "TransactionStatusEnum",
    "create_network_type",
]

let me know how am I doing on this and if I am missing some stuff? @Ninjagod1251
before i send it for more re-review higher up

It looks like all these comments are now outdated unless I'm missing something. Please let me know if there's anything missing / wrong and I can add or fix

@unparalleled-js since you last reviewed I forced pushed to fix https://github.com/ApeWorX/ape/pull/193/commits/2e881c64af3608ea33b975201209aad485875693 to conventional commit name to pass checks.


Kubernetes http probe failing with sporadic 503 errors

Describe the bug
I get sporadic error with Kubernetes http probes on istio sidecar proxy. I have two distincts environnements in two differents namespaces but only one with istio enabled. However, http probes are receiving sometimes 503 error from istio proxy. It is about 1 to 3% of the traffic.
The environment is stable with no deployments or pods restarting during my investigation.
Here is a part of the istio sidecar proxy with debug mode:
[2018-10-27 20:20:55.249][22][debug][filter] src/envoy/http/mixer/filter.cc:122] Called Mixer::Filter : decodeHeaders
[2018-10-27 20:20:55.249][22][debug][filter] src/envoy/http/mixer/filter.cc:211] Called Mixer::Filter : check complete OK
[2018-10-27 20:20:55.249][22][debug][router] external/envoy/source/common/router/router.cc:252] [C3328][S7692589569429134489] cluster 'inbound|8011||XXX-api.integration.svc.cluster.local' match for URL '/api/XXXXX'
[2018-10-27 20:20:55.249][22][debug][router] external/envoy/source/common/router/router.cc:303] [C3328][S7692589569429134489] router decoding headers:
':authority', 'XXXX-api:8011'
':path', '/api/XXXXXX'
':method', 'GET'
':scheme', 'http'
'x-b3-parentspanid', '4b3b89478a5f654b'
'accept', 'application/json, text/plain, */*'
'user-agent', 'axios/0.18.0'
'x-forwarded-proto', 'http'
'x-request-id', '8903fbbe-3cdc-9e73-a932-30731f121d22'
'x-b3-traceid', '92de5d35c719997f'
'x-b3-spanid', 'f0a2a5f4555f70d1'
'x-b3-sampled', '1'
'content-length', '0'

[2018-10-27 20:20:55.249][22][debug][pool] external/envoy/source/common/http/http1/conn_pool.cc:89] [C3626] using existing connection
[2018-10-27 20:20:55.249][22][debug][router] external/envoy/source/common/router/router.cc:971] [C3328][S7692589569429134489] pool ready
[2018-10-27 20:20:55.344][21][debug][filter] src/envoy/tcp/mixer/filter.cc:78] [C419] Called tcp filter onRead bytes: 220
[2018-10-27 20:20:55.344][22][debug][connection] external/envoy/source/common/network/connection_impl.cc:451] [C3626] remote close
[2018-10-27 20:20:55.344][22][debug][connection] external/envoy/source/common/network/connection_impl.cc:133] [C3626] closing socket: 0
[2018-10-27 20:20:55.344][22][debug][client] external/envoy/source/common/http/codec_client.cc:81] [C3626] disconnect. resetting 1 pending requests
[2018-10-27 20:20:55.344][22][debug][client] external/envoy/source/common/http/codec_client.cc:104] [C3626] request reset
[2018-10-27 20:20:55.344][22][debug][router] external/envoy/source/common/router/router.cc:457] [C3328][S7692589569429134489] upstream reset
[2018-10-27 20:20:55.344][22][debug][filter] src/envoy/http/mixer/filter.cc:191] Called Mixer::Filter : encodeHeaders 2
[2018-10-27 20:20:55.344][22][debug][http] external/envoy/source/common/http/conn_manager_impl.cc:1083] [C3328][S7692589569429134489] encoding headers via codec (end_stream=false):
':status', '503'
'content-length', '57'
'content-type', 'text/plain'
'date', 'Sat, 27 Oct 2018 20:20:55 GMT'
'server', 'envoy'

[2018-10-27 20:20:55.344][22][debug][filter] src/envoy/http/mixer/filter.cc:257] Called Mixer::Filter : onDestroy state: 2
[2018-10-27 20:20:55.344][22][debug][pool] external/envoy/source/common/http/http1/conn_pool.cc:122] [C3626] client disconnected
[2018-10-27 20:20:55.345][22][debug][filter] src/envoy/http/mixer/filter.cc:273] Called Mixer::Filter : log
[2018-10-27 20:20:55.345][22][debug][connection] external/envoy/source/common/network/connection_impl.cc:451] [C3328] remote close
[2018-10-27 20:20:55.345][22][debug][connection] external/envoy/source/common/network/connection_impl.cc:133] [C3328] closing socket: 0
[2018-10-27 20:20:55.345][22][debug][main] external/envoy/source/server/connection_handler_impl.cc:51] [C3328] adding to cleanup list
[2018-10-27 20:20:55.346][21][debug][filter] src/envoy/tcp/mixer/filter.cc:88] [C419] Called tcp filter onWrite bytes: 6552
[2018-10-27 20:20:55.348][22][debug][router] external/envoy/source/common/router/router.cc:583] [C3330][S11586225769463235658] upstream headers complete: end_stream=false
[2018-10-27 20:20:55.348][22][debug][filter] src/envoy/http/mixer/filter.cc:191] Called Mixer::Filter : encodeHeaders 2
[2018-10-27 20:20:55.348][22][debug][http] external/envoy/source/common/http/conn_manager_impl.cc:1083] [C3330][S11586225769463235658] encoding headers via codec (end_stream=false):
':status', '200'

The client here is 'Axios' because it came from a liveness probe from another service. However, I have the same type of 503 errors from pods that just receives basic /health checks. Exact same occurence (1-3%).
I've increased the timeout to 10 seconds instead of 1 without any luck.
Here's an example of my readiness/liveness probe config:
readinessProbe:
      failureThreshold: 3
      httpGet:
        path: /ping
        port: 8011
        scheme: HTTP
      initialDelaySeconds: 3
      periodSeconds: 5
      successThreshold: 1
      timeoutSeconds: 10

I've also checked the kubelet logs without noticing anything out of the ordinary.
Expected behavior
No 503 errors but 200. I have the exact same configuration in another namespace without Istio and I have no 503 errors from any service coming from my liveness probes.
Version
Kubernetes version: 1.10.7
Kops: 1.10
Istio: 1.0.2
Installation
Official Helm Chart
Environment
Kops, AWS, Weave CNI

Did you ever get a resolution on this issue?  I've experienced the same problem (and also 503 errors with the ingressgateway, and also with ambassador's api gateway).  I outline my findings on this issue:
https://github.com/istio/istio/issues/9906
I was able to correct the issue by setting the field spec->http->retries->attempts in my VirtualService specification.  I believe there's a race condition with the tcp keepalive setting in Istio's connection pooling mechanisms versus Envoy's stream idle timeout.

I have no idea what was causing the issue. However, since the update to Istio 1.0.4 (we are still using this version) everything has been smooth since.
Google Istio offering option only offers Istio 1.0.3 for now (in beta). So we did not tried to test it in our QA environments yet.


neo-gui is crashing while resyncing from scratch around block, 1,170,000

Once it fails, it's repeatable.  neo-gui will sync for a few blocks and then fails within a few seconds of starting.  I'm running neo-gui in the Visual Studio debugger.
Traceback
Exception thrown: 'System.NotSupportedException' in Neo.VM.dll
Exception thrown: 'System.NotSupportedException' in Neo.VM.dll
Exception thrown: 'System.NotSupportedException' in Neo.VM.dll
Exception thrown: 'System.NotSupportedException' in Neo.VM.dll
Exception thrown: 'System.NotSupportedException' in Neo.VM.dll
Exception thrown: 'System.NotSupportedException' in Neo.VM.dll
Exception thrown: 'System.NotSupportedException' in Neo.VM.dll
Exception thrown: 'System.NotSupportedException' in Neo.VM.dll
Exception thrown: 'System.NotSupportedException' in Neo.VM.dll
Exception thrown: 'System.NotSupportedException' in Neo.VM.dll
Exception thrown: 'System.NotSupportedException' in Neo.VM.dll
Exception thrown: 'System.NotSupportedException' in Neo.VM.dll
System.Transactions Critical: 0 : <TraceRecord xmlns="http://schemas.microsoft.com/2004/10/E2ETraceEvent/TraceRecord" Severity="Critical"><TraceIdentifier>http://msdn.microsoft.com/TraceCodes/System/ActivityTracing/2004/07/Reliability/Exception/Unhandled</TraceIdentifier><Description>Unhandled exception</Description><AppDomain>neo-gui.exe</AppDomain><Exception><ExceptionType>System.InvalidOperationException, mscorlib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089</ExceptionType><Message>Operation is not valid due to the current state of the object.</Message><StackTrace>   at Neo.VM.RandomAccessStack`1.Peek(Int32 index)
   at Neo.SmartContract.ApplicationEngine.CheckArraySize()
   at Neo.SmartContract.ApplicationEngine.Execute()
   at Neo.Implementations.Blockchains.LevelDB.LevelDBBlockchain.Persist(Block block)
   at Neo.Implementations.Blockchains.LevelDB.LevelDBBlockchain.PersistBlocks()
   at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state, Boolean preserveSyncCtx)
   at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state, Boolean preserveSyncCtx)
   at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state)
   at System.Threading.ThreadHelper.ThreadStart()</StackTrace><ExceptionString>System.InvalidOperationException: Operation is not valid due to the current state of the object.
   at Neo.VM.RandomAccessStack`1.Peek(Int32 index)
   at Neo.SmartContract.ApplicationEngine.CheckArraySize()
   at Neo.SmartContract.ApplicationEngine.Execute()
   at Neo.Implementations.Blockchains.LevelDB.LevelDBBlockchain.Persist(Block block)
   at Neo.Implementations.Blockchains.LevelDB.LevelDBBlockchain.PersistBlocks()
   at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state, Boolean preserveSyncCtx)
   at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state, Boolean preserveSyncCtx)
   at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state)
   at System.Threading.ThreadHelper.ThreadStart()</ExceptionString></Exception></TraceRecord>
An unhandled exception of type 'System.InvalidOperationException' occurred in Neo.VM.dll
Operation is not valid due to the current state of the object.

mwherman2000 - Today at 6:39 AM
@birmas C:\NEO\neo-gui-developer\packages\Neo.VM.2.0.3\lib\net461\Neo.VM.dll

mwherman2000 - Today at 6:41 AM
No ...I went back to an older copy of the project.
@birmas C:\NEO\neo-gui-developer\packages\Neo.2.3.3\lib\net461\Neo.dll

Closing this after discussions in discord: end result is that we believe the Neo.VM being used didn't contain the latest opcode support that had been added to neo-vm


fix(Editor): disable unnecessary features in Monaco editor

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] All the files I changed are in the same world language, for example: only English changes, or only Chinese changes, etc.

Closes #38541

/cc @lasjorg for visibility.

@raisedadead Disabled more features. Please check

Can you share some screenshots of the before and after of the updates you made? I am having trouble to confirm this locally.

@raisedadead Thanks for the heads up.
Two things I found handy:

Enabling the formatOnPaste option. It

Controls if the editor should allow to move selections via drag and drop.

Disabling the dragAndDrop option. It

Enable the suggestion box to pop-up on trigger characters.

Which I found a little annoying as well

@AshNaz87 We may also want to disable the lightbulb.
lightbulb: {
  enabled: false
}

https://microsoft.github.io/monaco-editor/api/interfaces/monaco.editor.ieditorlightbulboptions.html
@raisedadead Not sure removing items from the context menu is exposed by the API (I know you can add actions).
https://github.com/Microsoft/monaco-editor/issues/1280
Here is some not so pretty CSS that removes the context menu items and separators, except the cut/copy and except items added by some context, like selecting text will add a new format selection item to the menu.
.monaco-menu .monaco-action-bar.vertical .action-item:nth-child(-n + 8),
.monaco-menu .monaco-action-bar.vertical .action-item:last-child,
.monaco-menu .monaco-action-bar.vertical .action-label.separator {
  display: none !important;
}

I'm also not sure you can disable the Ctrl + Space suggest menu.
https://github.com/microsoft/monaco-editor/issues/1681
You can set it to just 1 item suggest: { maxVisibleSuggestions: 0 } (yes 0 leaves 1 item, go figure) but it still shows up.
https://microsoft.github.io/monaco-editor/api/interfaces/monaco.editor.isuggestoptions.html#maxvisiblesuggestions
We can again hide it with CSS.
.monaco-editor .editor-widget {
  display: none !important;
}

Hiding stuff with CSS is less that optimal, but it might be the only choice.
Some relavant files:
monaco-editor CSS files:
node_modules\monaco-editor\esm\vs\base\browser\ui\actionbar\actionbar.css
node_modules\monaco-editor\dev\vs\editor\editor.main.css
Guessing the CSS changes would go in this file:
freeCodeCamp\client\src\templates\Challenges\classic\classic.css

I think all of the above suggestions (using CSS) are okay for us. @AshNaz87 can you please update the PR taking the advice above.

Made the changes
@raisedadead, @lasjorg Please check

@AshNaz87 Sorry, I just realized that depending on the challenge the context menu will contain different amounts of items. Which breaks the CSS that hides everything but the cut/copy.
Old context menu with fewer items:

New context menu with fewer items:

I'm also starting to doubt just how useful that menu really is. Maybe we should just disable it completely instead like @raisedadead suggested from the start? So keep the CSS that hides the .editor-widget and just disable the context menu in the options contextmenu: false.
The only issue disabling the context menu causes is it makes the native browser context menu show up instead but you can't use it for anything (like copy/paste for example), which might be a bit confusing UX.

@AshNaz87 Or we can count backwards in the CSS and that might be consistent across all context menus. If you can help check that I'm correct about this that would be great.
.monaco-menu .monaco-action-bar.vertical .action-item:nth-last-child(n + 5),
.monaco-menu .monaco-action-bar.vertical .action-item:last-child,
.monaco-menu .monaco-action-bar.vertical .action-label.separator {
  display: none !important;
}

The separators can be made to look a little nicer like this:
.monaco-menu a.action-label.codicon.separator.disabled {
  border: 0 !important;
  border-bottom: 1px solid var(--secondary-color) !important;
}

so in night mode it looks like this:

and day mode:

I think it's worth keeping the menu, since there's a bunch of useful things in there and, as @lasjorg pointed out, removing it would make right-clicks behave oddly.

@ojeytonwilliams Good shout

@ojeytonwilliams I don't mind having the context menu. The thing is, we are already removing "useful" features, like intellisense/code completion.
The issue I made that got this PR rolling was just about making sure that the hover didn't cover the code.
Personally, I would keep code completion as I think it can be helpful and increases discoverability. Dotting into an object/array/string and seeing all the methods on the object is a useful feature to have even for beginners, in my opinion.
It just sounded like it had been decided at some point that the editor should be totally stripped of all such functionality. Not sure what the consensus is here?

We should remove the code completion. I think we are fine being opinionated about it, its more distraction unless we perfect the config to be without the additional bugs and complaints it brings.
As for the right-click menu, if its too hacky with css, we can let it be.

@AshNaz87
Here is what this PR should do:

Remove the code completion and anything additional other than the right-click menu.
The right-click menu if it can be removed using any config options, should be removed, if not let's leave it as is.

I'm dumb, sorry everyone. I thought we just wanted the spacers cleaning up, not removing actions from it.
Anyway, the right-click menu can be disabled by adding contextmenu: false to the options. @raisedadead if there's an ideal menu you want, I can take a look at customising it, but you just want it gone, right?

if there's an ideal menu you want, I can take a look at customising it, but you just want it gone, right?

Ideally we want:

All context based auto-suggestions, hover based definitions, etc. to be gone.
The right-click menu can stay or go depending on complexity. If it can stay with only copy / paste menu items, even better.

If it can stay with only copy / paste menu items, even better.

In principle it could be pared down to just copy + cut (paste isn't an option - not sure why).  I'll see if I can figure out how.

Sorry, @lasjorg I didn't read your comments carefully enough - apologies for just repeating what you were saying!
I'm leaning towards just removing the context menu, since the css solution seems brittle.  It could easily break if we update the editor.  Hopefully they will add an api for that at some point, then we can easily configure it.
Also, all the commands will be accessible via keyboard shortcuts, so people who're used to vscode should still be fine.

@ojeytonwilliams No worries.
I think the counting backward CSS I posted for the context menu is less likely to break as I don't think new items are added to the end of the menu. But I agree it's far from foolproof, any CSS solution is bound to be more susceptible to breakage. For the widget, I don't think there is another option.
BTW. The missing Paste functionality is a browser limitation.
https://github.com/microsoft/monaco-editor/issues/391#issuecomment-312237052

Thanks, I was wondering which browsers could paste, since that could also break the css solution. If no modern ones can, at least that isn't a big problem.
My main concern is that we can't test this at all. I'll take another look at #37053 since that could potentially help.

No need to wait for that PR. If I get it working, I'll add tests for the context menu then.

@AshNaz87
With the current state of the PR, when nothing is selected right-click leads to:

@raisedadead Should work now
Would be great if a final consensus can be gathered

Do you mind sharing a screenshot of a before and after before I pull this down.

Would be great if a final consensus can be gathered

If you mean about the approach for this PR, I think my comment still stands

Ideally we want:

All context based auto-suggestions, hover based definitions, etc. to be gone.
The right-click menu can stay or go depending on complexity. If it can stay with only copy / paste menu items, even better.

The best you are able to achieve should be good enough iteration. A screenshot of the various things that this PR fixes would help in the QA.
Most importantly we should not be introducing additional annoyance in a change. :)

Current PR code has:

Code completion.

Ctrl + Space will open the editor widget.

The CSS that hides the editor-widget takes care of both 1 and 2 as you can't see the widget even if it is triggered (without having to disable anything related in the options).
.monaco-editor .editor-widget {
  display: none !important;
}

Just to be clear. I was not saying to put code completion back, I was expressing my personal preference and asking for what the consensus was. @raisedadead has the final say, not me.

The right-click context menu is back.

A. Use the latest CSS I posted:
.monaco-menu .monaco-action-bar.vertical .action-item:nth-last-child(n + 5),
.monaco-menu .monaco-action-bar.vertical .action-item:last-child,
.monaco-menu .monaco-action-bar.vertical .action-label.separator {
  display: none !important;
}

Using the nth-last-child selector should be much more reliable. The last two context menu groups (command palette and cut/copy/paste) looks reserved to me. I would imagine that new items are not very likely to be added to these groups. I have tested the CSS in challenges with HTML/CSS/JS/React/D3 and the menu stays the same (has cut/copy, with or without making a selection). The only exception here would be in IE where the paste menu item will show up. But as far as I know, the curriculum already has incompatibilities with IE.
B. Remove the context menu completely using the option, which is easy and unlikely to break.
contextmenu: {
  enabled: false
}

@lasjorg Thanks for your suggestions
A few pointers:

I've implemented your CSS suggestions. Am also in favour of keeping the cut/copy options on right click
Edited options:

hover set to false
dragAndDrop set to true. This enables users to literally drag and drop text. Think it's useful
lightbulb object enabled to false, which to my understanding is another suggestion box

@AshNaz87 Looks pretty good to me. Let's wait and see what others have to say.
There is one UI thing I want to ask about. Unused variables get colored differently, they are sort of faded. We can turn it off using the showUnused: false option.
@raisedadead Is turning that off something we should consider? Personally I think it's nice to have an indication but I'm wondering if it can be confusing to beginners that don't know why it looks like that.
For now, I'm just posing it as a question.
Unused variables on:

Unused variables off:

showUnused: false
https://microsoft.github.io/monaco-editor/api/interfaces/monaco.editor.ieditoroptions.html#showunused

... Unused variables get colored differently ...

We should consider updating the theme(s) instead. Maybe in a separate PR.


vTaskDelay in pollTask configurable

I need fully control of the poll timing in my application. So it would be nice to have the vTaskDelay configurable. Btw, I'am not sure if the poll function should use a delay other than 0. So maybe the poll function should not have the delay parameter and should call pollTask with 0.

Can you provide some more details about what you are trying to do.  The polling loop takes a variable amount of time depending on a wide variety of variables.  Anything that assumes a specific time may run into trouble later if I add new features in the polling functions or if HomeKit extends functionality, etc.
If you have code that is sensitive to timing, I recommend using autoPoll() to place HomeSpan polling in a separate task, and then you can add your time-sensitive code to the main sketch loop() where it will not be blocked by polling, and polling will not be blocked by any delays you may need to add within the loop().

I use the lib in the KNX SmartHomeBridge project. It is combined with a KNX-stack and the alexa. Unfortunately, the homespan did not work with the autoPoll function, regardless of the used core. So I decide to control the loop calls by myself and found that the call to loop takes to long, so I found the vTaskDelay which makes now sense if the loop called manually.

After re-reviewing this section of the code I realized that I only added the call to vTaskDelay() because it is needed when spawning pollTask() into a separate task using homeSpan.autoPoll() (the task needs to periodically yield to allow other tasks to run).  However, it is NOT needed at all if you are calling homeSpan.poll() directly from within the main loop().  So rather than provide a mechanism to change the delay duration, I simply made the use of vTaskDelay() conditional on the use of autoPoll(). In other words, if you call poll() directly, there is no longer any call to vTaskDelay().  If users do want to create a delay in the main loop() they can always add delay() if needed, though of course this will delay poll() and should be use judiciously.
The change described above is available in the latest development branch: release-1.8.1-dev.  Please give it a try and let me know if it works.

Tested now and works fine for me. But I'am not sure if your implementation is the best for the readability. Maybe it would be better to add the delay function to the autoPoll function instead of checking the for the taskhandle in the poll function:
void autoPoll(uint32_t stackSize=8192, uint32_t priority=1, uint32_t cpu=0){     // start pollTask()
    xTaskCreateUniversal([](void *parms){
      for(;;)
      {
        homeSpan.pollTask();
        vTaskDelay(5);
      }}, "pollTask", stackSize, NULL, priority, &pollTaskHandle, cpu);
    LOG0("\n*** AutoPolling Task started with priority=%d\n\n",uxTaskPriorityGet(pollTaskHandle));
  }

That's a good idea and it is more consistent with the way the Arduino-ESP32 library handles the main loop.  Will make the change.

Done.  Code is updated in the release-1.8.1-dev branch.  Thanks for the suggestion.

Anything more to test on this, or can I close out the PR?

No, thank you for your help


Can I remove @timestamp or @version?

According to README I can add new fields, but I need to remove default e.g. @timestamp or @version, can I do this?

You have full access to the event in the customize_event block, so you can do any mutations you need to it, including removing fields. I'll clarify this in the readme.


Dev

code reformatted
pipenv substituted by poetry
mypy checker added
python 3.10, 3.11 support added
dev dependencies updated

Codecov Report

Merging #10 (677d771) into master (fa791ce) will increase coverage by 0.00%.
The diff coverage is 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##           master      #10   +/-   ##
=======================================
  Coverage   99.21%   99.22%
=======================================
  Files           3        3
  Lines         254      257    +3
=======================================
+ Hits          252      255    +3
  Misses          2        2

Flag
Coverage Δ

unittests
99.22% <100.00%> (+<0.01%)
:arrow_up:

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

Impacted Files
Coverage Δ

crontools/__about__.py
100.00% <100.00%> (ø)

crontools/__init__.py
100.00% <100.00%> (ø)

crontools/crontab.py
99.18% <100.00%> (+0.01%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


IsBlank, In and StartsWith delegable support updated

I've run some tests and the CDS and Dynamics 365 CE connectors do support delegation of IsBlank, StartsWith and In from inside a Filter query.
E.g.
Filter(Accounts, StartsWith(name,TextSearchBox1.Text), IsBlank(address2_city))
will result in the fetchxml
<filter type="and">
		<condition attribute="name" operator="like" value="contoso%" />
		<filter type="or">
			<condition attribute="address2_city" operator="null" />
			<condition attribute="address2_city" operator="eq" value="" />
		</filter>
</filter>

and also:
Filter(Accounts, AccountSearchText.Text in name)
with result in the fetchxml
<filter type="and">
	<condition attribute="name" operator="like" value="%contoso%" />
</filter>

Thanks, @scottdurow, for the feedback!
@lancedMicrosoft Does this match your understanding? Do CDS for Apps and Dynamics support delegation of these functions?

The detailed CDS delegation table should go out today.

@scottdurow Closing this PR, now that we've removed the topic that it updates and moved the information that's still relevant to the delegation overview.


Pass request data to Verifier::create_from_request via struct

Signed-off-by: Miroslav Kovar<EMAIL_ADDRESS>

Codecov Report

Merging #402 (2a3f793) into feature/send-presentation-ack (f891415) will decrease coverage by 19.96%.
The diff coverage is 41.66%.

:exclamation: Current head 2a3f793 differs from pull request most recent head 0e47a08. Consider uploading reports for the commit 0e47a08 to get more accurate results

@@                        Coverage Diff                         @@
##           feature/send-presentation-ack     #402       +/-   ##
==================================================================
- Coverage                          66.31%   46.34%   -19.97%
==================================================================
  Files                                153      152        -1
  Lines                              14462    13110     -1352
  Branches                            2866     2670      -196
==================================================================
- Hits                                9590     6076     -3514
- Misses                              2540     5518     +2978
+ Partials                            2332     1516      -816

Flag
Coverage Δ

integration-aries-vcx
?

unittests-agency-client
22.51% <ø> (ø)

unittests-aries-vcx
48.54% <41.66%> (+0.01%)
:arrow_up:

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

Impacted Files
Coverage Δ

aries_vcx/tests/test_integration.rs
6.00% <0.00%> (-76.60%)
:arrow_down:

aries_vcx/tests/utils/devsetup_agent.rs
0.00% <0.00%> (-77.82%)
:arrow_down:

...c/handlers/proof_presentation/verifier/verifier.rs
36.03% <87.50%> (-27.76%)
:arrow_down:

...dlers/proof_presentation/verifier/state_machine.rs
67.46% <100.00%> (-5.46%)
:arrow_down:

...ndlers/proof_presentation/prover/states/initial.rs
0.00% <0.00%> (-100.00%)
:arrow_down:

aries_vcx/src/libindy/utils/anoncreds.rs
9.20% <0.00%> (-67.84%)
:arrow_down:

...x/src/handlers/proof_presentation/prover/prover.rs
15.66% <0.00%> (-62.01%)
:arrow_down:

..._vcx/src/handlers/proof_presentation/prover/mod.rs
21.42% <0.00%> (-57.15%)
:arrow_down:

... and 79 more

Continue to review full report at Codecov.

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


Typo in Knowledge: Attributes and properties page

Expected behavior
In Knowledge: Attributes and properties, the example of "Attributes can also be set from javascript, and will update the attribute in the DOM" should be
myDiv.setAttribute('foo', 'not-bar');

Actual Behavior
The current example use getAttribute instead.
myDiv.getAttribute('foo', 'not-bar');

fixed by https://github.com/open-wc/open-wc/pull/2293


deselect_all.jstree event not properly passing current selection

According to the docs, the deselect_all.jstree event should be passing an array of the current (which I take to mean the new) selections. Except what I'm actually receiving is what looks like such an array when you dump it to the console, but which has had the length truncated, so you can't actually access the data in it.
I presume this is because of line 3202:
this._data.core.selected = [];
Should that be moved down below line 3214, so that it happens after the trigger is fired?

also note: calling get_selected() from inside my deselect_all event handler returns the same thing, for what looks like the same reason.

Sorry, one more note: I thought I had the latest, but I was actually referencing version 3.2.1; the issue still exists in 3.3.1, but the line in question is now 3244.

selected will always be an empty array inside a deselect_all.jstree event handler, as the whole selection was just cleared (which is what deselect_all does, as the name suggests).
If you need the old selection - it is available as well.
Best regards,
Ivan


fix: downgrade monaco version

Types

[x] 🐛 Bug Fixes

Background or solution
Changelog
回退 monaco editor 到 0.29.1-patch.1 版本

Codecov Report

:exclamation: No coverage uploaded for pull request base (v2.15@247250d). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff            @@
##             v2.15     #508   +/-   ##
========================================
  Coverage         ?   59.01%
========================================
  Files            ?     1200
  Lines            ?    73918
  Branches         ?    15299
========================================
  Hits             ?    43622
  Misses           ?    27615
  Partials         ?     2681

Continue to review full report at Codecov.

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


⚠ Bofrost.lu has degraded performance

In 8286f81, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 2709 ms

Resolved: Bofrost.lu performance has improved in 4dc50e0 after 13 minutes.


Support builds from behind a proxy

Trying to run ./build.sh at the office, I quickly found out that building from behind a proxy didn't work.
This fixes that as long as the https_proxy, http_proxy and no_proxy environment variables are set appropriately for the environment where the build is performed.  Any of these variables can be left unset if not needed.

Hi @paddy-hack !
Thanks for taking time into this, looking good here!


🚀 Feature: Add a Deployment Guide for DigitalOcean

🔖 Feature description
Write a brief article on how users can easily deploy a self-hosted instance of DocsGPT on a DigitalOcean.
🎤 Why is this feature needed?
This enables the users to be able to deploy on another cloud provider other than Amazon LightSail.
✌ How do you aim to achieve this?
Try deploying the instance on DigitalOcean Droplet and document it step by step.
🔄 Additional Information
This article can be written and published on any popular blogging platform and a link to it can added in the Deployment section on the Docs.
👀 Have you spent some time checking if this feature request has been raised before?

[X] I checked and didn't find a similar issue

Are you willing to submit PR?
Yes, I am willing to submit a PR!

@dartpain Please assign this to me. Thanks.

@Rutam21 Assigned. Thank you!


Why the p values and mean values of clusterA-clusterB are different with the clusterB-clusterA?

Hello! I think it is a great tool!
But I am curious why the p values and mean values of clusterA-clusterB are different with the clusterB-clusterA?
Thank you!

Hi,
When reading the outputs, is IMPORTANT to note that the interactions are not symmetric. Partner A expression is considered for the first cluster/cell type, and partner B expression is considered on the second cluster/cell type. In other words:
clusterA_clusterB = clusterA expressing partner A and clusterB expressing partner B.
clusterA_clusterB and clusterB_clusterA values will be different.
You will find this information in the documentation.
Luz


Refactoring + fakejuggler binary

Add .gitignore
Move job definition into lib
Refactor usage of strings as statuses, now they are consts
Add fakejuggler binary to help with testing of UI/Clients

@yarikk I've renamed lib to juggler (best meaningful name I could come up with)

Addressed comments


Adding support for named components and configuration to test framework.

Each component can be created with a name and optionally a configuration. This allows multiple echo instances, policy backends, envoy proxies etcetera to be managed independently.

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

Needs approval from an approver in each of these files:

pkg/OWNERS
tests/integration2/OWNERS

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

Moved this PR over to master and dropped the other, in theory this is in good state but my git foo is weak so I possibly horribly screwed something up.

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

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
c114ba0361e62ac763d8f966df55af2dfe716891
link
/test istio-pilot-multicluster-e2e

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

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

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
c114ba0361e62ac763d8f966df55af2dfe716891
link
/test istio-pilot-multicluster-e2e

prow/istio-integ-k8s-tests.sh
c114ba0361e62ac763d8f966df55af2dfe716891
link
/test istio-integ-k8s-tests

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

/test istio-pilot-multicluster-e2e
/test istio-integ-k8s-tests

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

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
c114ba0361e62ac763d8f966df55af2dfe716891
link
/test istio-integ-k8s-tests

prow/istio-pilot-multicluster-e2e.sh
c114ba0361e62ac763d8f966df55af2dfe716891
link
/test istio-pilot-multicluster-e2e

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

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: nmittler, smawson
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pkg/OWNERS [nmittler]
tests/integration2/OWNERS [nmittler]

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

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

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
c114ba0361e62ac763d8f966df55af2dfe716891
link
/test istio-pilot-multicluster-e2e

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


added user promt to ask if a data set should be downloaded again  if it already exists locally

Found this adaptation helpful, since for large data sets, the script downloads all repositories which are configured in darus_config.json. While I like that one is able to store all data sets relevant to a project in the darus_config.json, the run time of the script becomes quickly very long for large data sets (~1GB). Being promted if data sets shoud be redownloaded if it already exists locally makes it easier to update the project with a new data set, while making it not necessary to redownload all data that might already exist.
However, this might be a specific adaptiation useful for my workflow and not applicable to other workflows.
So maybe we should consider other workflows before merging.

Looks good to me otherwise

Awesome. Thanks for your contribution.


future of Flynn

Flynn is awesome and as a small startup, we built everything around it. it gave us the freedom to focus on other things.
Flynn IRC support channel became so quiet. I no longer see as many activity in flynn github either. I rarely see any new features. I'm worried about the future of Flynn. is everything ok?
if Flynn closes shop, would you open source everything else so we can keep our clusters running?
I hope this is not the case

This is a duplicate of #4242.
We still have a full-time team working on Flynn, and are financially stable. As I mentioned in the issue referenced above, our focus is on our paying customers since they are what allow us to continue building Flynn. The reason you may not see us as often in IRC is because most of our support time is spent in private Slack channels with our customers.
Everything you need to run Flynn forever without paying us anything is already free and open source in this repo. We have no closed source dependencies in the open source project, and that won't change.


How to download publicly available objects from blob storage?

I have some files in blob storage that I want publicly available, but I"m unsure of how someone would fetch them.
It looks like the azure.storage.common.storageclient module is the right way to go, but It's unclear how to use this.

Hi @apengwin, please start by reading the tutorial.
storageclient is not the way to go, please take a look at the sample.

@zezha-msft  I'm confused by how container names are uniquely identified. I currently have two separate accounts each with a blob container called azure-webjob-hosts, and am unsure how
block_blob_service.get_blob_to_path(azure-webjob-hosts, 'myblockblob', 'out-sunset.png') would get resolved.

Hi @apengwin, when you create a BlockBlobService, you have to specify your storage account. Account name + container name is a unique identifier for containers. These concepts are all explained in the tutorial that I linked to previously.

@zezha-msft I understand that the storage account name and container name are a unique identifier for each container, but the docs actually aren't clear about that, or what that means.
Explicitly, I have an account with name foo and container container1, which is public.
I'm trying to access it with account bar, so I'm guessing something like
block_blob_service = BlockBlobService(account_name='bar', account_key='bar_key')
block_blob_service.get_blob_to_path( what_do_i_put_for_container_name_here, blob_i_want, local_path)

How do specify the container name/ is it foo/container1? or foocontainer1?

The sample code linked above is very unclear.
The proper way to do this is given that you've published a container azure-webjob-hosts with account namefoo, you would call -
bar_bbs = BlockBlobService(account_name='bar', account_key=None)
bar_bbs.get_blob_to_path(azure-webjob-hosts, blob_i_want, local_path)

I hope that helps, @apengwin.
It would be nice to have better sample code for future reference.

Thanks @richardliaw!

Hi @richardliaw, thanks for the help, but that is not right at all. Like the sample shows, When you create a BlockBlobService, you specify the account name that owns the container. In this case, you want to access the container azure-webjob-hosts on account foo, so you should do this:
anonymous_service = BlockBlobService('foo')
anonymous_service.get_blob_to_path('azure-webjob-hosts', 'blob_name', local_path)

@apengwin Accessing the public blob has nothing to do with the account 'bar', because the container is not owned by 'bar'. As the tutorial explains, each blob can be thought of as a resource, and can be access with an url like this https://storagesample.blob.core.windows.net/mycontainer/blob1.txt. storagesample is the account name, mycontainer is the container name, and blob1.txt is the blob name.
Please let me know if you have any other question.

ah sorry, edited above. thanks for the catch.


Nested views resize lag

When resizing the parent container, the width resizing logic of the child view lags behind.
This can be reproduced directly on the example in the docs (I changed the background colors to make it more obvious):

https://codesandbox.io/s/restless-firefly-rnpfyh?file=/src/app.tsx
Related: https://github.com/mathuo/dockview/issues/156#issuecomment-1252905492

I have a feeling the scrollbar you see is actually the overflow scrollbar for the tabs rather than a resize lag?

Hi,
It’s not about the scrollbar but the black area that appears on the right when you resize fast to the left.
In the docs this is almost not noticeable because it happens only on the tabs section:

But for some reason when we use different background colors it happens on the whole view.

Ah, so dockview uses a ResizerObserver to determine when the components parent has resized and change it's dimensions accordingly, however this may lag a couple of frames behind the parent due to the fact it then executes the resize call within a requestAnimationFrame callback for performance reasons.
This frame out-of-sync'ness is seen in your example of nested dockviews.
To get around this dockview does allow the user to explicitly control the containers size. For inner dockviews you should listen to the parents resize events and then explictly resize the inner views whilst disabling the auto-resizing on the inner view with the disableAutoResizing props.
Working example here.
Sorry for the delayed response but I hope that clears up the question.
const InnerDockview = (props: IDockviewPanelProps) => {
  const api = React.useRef<DockviewApi>();

  const onReady = (event: DockviewReadyEvent) => {
    api.current = event.api;
  };

  React.useEffect(() => {
    const disposable = props.api.onDidDimensionsChange((event) => {
      api.current?.layout(event.width, event.height);
    });

    return () => {
      disposable.dispose();
    };
  }, [props.api]);

  return (
    <DockviewReact
      onReady={onReady}
      components={components}
      disableAutoResizing={true}
    />
  );
};
**

Thanks a lot for your detailed answer! Manual resizing indeed solves this issue.


Introduce orientation:change event

As event name we suggest orientationchange. The event parameter should be the new orientation in one of these forms:

landscape-primary
landscape-secondary
portrait-primary
portrait-secondary

where "secondary" stands for upside-down. The names follow an emerging W3C standard - credits go to them ;-). See also MDN.
Together with the event, we should also introduce a property orientation. Values should be the same. I'm afraid that this may be in conflict with the existing property.

@ralfstx Since the payload of a notify is a map you properly imply to use orientation as the key?

Yes, sounds right.

When should the event be fired?
a) When the UI is about to begin to rotate
b) When the UI finished rotation

Since the new orientation should correspond with the updated device.screenWidth/screenHeight properties, I'd say it should be fired when the rotation is finished.


Better Docker Support PR

Feature request
I would like to edit the configuration logic as well as the Dockerfile to make it possible to configure headscale using the environment variables only. I'd need this for my portainer setup.
I'd suggest adding a configuration "SKIP_CONFIG_READ" which would skip checking the config file and instead rely on the environment variables only.
I also think it would be nice to use a double underscore as environment separator, like some other projects like gitea do, as I think that helps differentiate the variables better, but this would break backwards compatibility, so it's probably a bad idea.
If you are ok with my suggestion, I'd get to work on a PR.
Edit: I should probably clarify my intent a bit better. My goal would be to run headscale without any additional configuration as bind mounts so it's easier to deploy. If you'd want this feature for consistency, I could  also autocreate the sqlite db if it hasn't been initialized yet.

This is relevant to me too. Can it be reopened?


solved #4911 #4912 #4913

To Suspense thriller books:
Added quiz
Added solutions to quiz
Added approach to read
Fixes: #4911 #4912 #4913

Description
[Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.]

Type of PR

[ ] Bug fix
[ ] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Screenshots / videos (if applicable)
[Attach any relevant screenshots or videos demonstrating the changes. Make sure to attach before & after screenshots in your PR.]
Checklist:

[ ] I have made this change from my own.
[ ] I have taken help from some online resources.
[ ] My code follows the style guidelines of this project.
[ ] I have performed a self-review of my own code.
[ ] I have commented my code, particularly in hard-to-understand areas.
[ ] I have made corresponding changes to the documentation.
[ ] My changes generate no new warnings.
[ ] I have tested the changes thoroughly before submitting this pull request.
[ ] I have provided relevant issue numbers and screenshots after making the changes.

@atmajaa please merge this pr


Tensorflow failed to find 'TRTEngineOp' when building pip-package and libtensorflow_cc.so

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Linux Ubuntu 18.04
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary): source
TensorFlow version: 1.14
Python version: 3.6
Installed using virtualenv? pip? conda?: Build from source
Bazel version (if compiling from source): 0.24.1
GCC/Compiler version (if compiling from source): gcc 7.4.0
CUDA/cuDNN version:10.0
GPU model and memory: 2080ti 11GB

Describe the problem
In brief, I built tensorflow for both python (pip_package) and c++ (libtensorflow_cc.so), I want to train a model, convert it to tf-trt in python, then run it on c++. And it failed when running it on c++.
Here are the details:
I'm trying to build tensorflow python version by bazel build --config=opt --config=cuda //tensorflow/tools/pip_package:build_pip_package and build c++ version by bazel build --config=opt --config=monolithic --config=cuda //tensorflow:libtensorflow_cc.so. And I want them both with tensorrt support. After building it sequentially (configure first, build_pip_package and then libtensorflow_cc.so), I find that the python version could load tensorrt successfully, but when I'm trying to run session run on c++, it gives me the error saying:
Check failed: status.ok() Loading error: Op type not registered 'TRTEngineOp' in binary running on my_dev_docker. Make sure the Op and Kernel are registered in the binary running in this process. Note that if you
are loading a saved graph which used ops from tf.contrib, accessing (e.g.) `tf.contrib.resampler` should be done before importing the graph, as contrib ops are lazily registered when the module is first accessed.

Since I'm using tf 1.14 where the tensorrt has been promoted to first-class citizen from tf.contrib. So I believe that I don't need to dynatically load it in my c++ code using c api "TF_LoadLibrary". So I started to suspect that maybe the tensorrt support is successfully installed but not on the c++ one and I didn't find any good reference on the internet on how to enable tf-trt for tensorflow_cc.so.
The success of python tensorflow with tensorrt support is tested by running "from tensorflow.python.compiler.tensorrt import trt_convert as trt". Although I have to run "export LD_LIBRARY_PATH=$LD_LIBRARY_PATH:/usr/local/TensorRT-<IP_ADDRESS>/lib" first otherwise it will say "ImportError: libnvinfer.so.5: cannot open shared object file: No such file or directory".
The following is the exact command:
PYTHON_BIN_PATH=/usr/bin/python3.6 \
PYTHON_LIB_PATH=/usr/local/lib/python3.6/dist-packages \
TF_ENABLE_XLA=1 \
TF_NEED_OPENCL_SYCL=0 \
TF_NEED_ROCM=0 \
TF_NEED_CUDA=1 \
TF_CUDA_VERSION=10 \
TF_CUDA_PATHS=/usr/local/cuda,/usr/lib/x86_64-linux-gnu,/usr/include \
CUDA_TOOLKIT_PATH=/usr/local/cuda \
CUDNN_INSTALL_PATH=/usr/lib/x86_64-linux-gnu \
TF_CUDNN_VERSION=7 \
TF_NEED_TENSORRT=1 \
TF_TENSORRT_VERSION=5 \
TENSORRT_INSTALL_PATH=/usr/local/TensorRT-<IP_ADDRESS> \
TF_NCCL_VERSION=2 \
TF_CUDA_COMPUTE_CAPABILITIES="6.1,7.5" \
TF_CUDA_CLANG=0 \
TF_DOWNLOAD_CLANG=0 \
GCC_HOST_COMPILER_PATH=/usr/bin/gcc \
CLANG_CUDA_COMPILER_PATH=/usr/local/clang_8.0.0/bin/clang \
TF_NEED_MPI=0 \
CC_OPT_FLAGS="-mavx -Wno-sign-compare" \
TF_SET_ANDROID_WORKSPACE=0 \
./configure
bazel build --config=opt --config=cuda //tensorflow/tools/pip_package:build_pip_package
./bazel-bin/tensorflow/tools/pip_package/build_pip_package /tmp/tensorflow_pkg
pip3 install /tmp/tensorflow_pkg/tensorflow-1.14.0-cp36-cp36m-linux_x86_64.whl
bazel build --config=opt --config=monolithic --config=cuda //tensorflow:libtensorflow_cc.so
Any other info / logs
I run the conversion in python by:
from tensorflow.python.compiler.tensorrt import trt_convert as trt
converter = trt.TrtGraphConverter(input_saved_model_dir=input_saved_model_dir)
converter.convert()
converter.save(output_saved_model_dir)
I run the converted model in c++ by:
const auto status =
LoadSavedModel(session_options, run_options, net_param_.model_path(),
{tensorflow::kSavedModelTagServe}, &bundle_, gpu_id);
And I get error message from status.error_message();
Could you kindly provide some hints on what I might be doing wrong?

Did you try importing tftrt module in addition to tensorflow module?
 import the tftrt module in addition to import tensorflow

Hi @qcraftai, I can see a dependency chain based on:
bazel query 'somepath(tensorflow:libtensorflow_cc.so, tensorflow/compiler/tf2tensorrt:trt_conversion)'

. When you run bazel build on tensorflow:libtensorflow_cc.so, can you try adding --config=tensorrt? Or make sure TensorRT is configured during the build by checking that environment variables like TF_NEED_TENSORRT and TENSORRT_INSTALL_PATH are set?

Hi There,
We are checking to see if you still need help on this, as you are using an older version of tensorflow which is officially considered end of life . We recommend that you upgrade to the latest 2.x version and let us know if the issue still persists in newer versions. Please open a new issue for any help you need against 2.x, and we will get you the right help.
This issue will be closed automatically 7 days from now. If you still need help with this issue, please provide us with more information.


s3 open seek operation read rest of file into buffer, which makes seek O(N)

Here makes API call to fetch rest of file into buffer when calling seek, which makes seek very slow. The API call may be put in read method instead.

Are you sure that actually reads the file?
self._body = _get(self._object, self._version_id, Range=range_string)['Body']

I think in this case, the _body in this case is a file object. Until you actual .read() from it, nothing gets transferred over the wire.
If you have sample code that demonstrates the opposite, I'd be interested in having a look.

Here reads the API call to get_object is made, and the bytes are transferred.

sample code under work.

import boto3
from moto import mock_s3
from smart_open import open

if __name__ == '__main__':
    with mock_s3():
        c = boto3.client('s3')
        c.create_bucket(Bucket='bucket')
        c.put_object(Bucket='bucket', Key='key', Body=b'value')
        f = open('s3://bucket/key', 'r')
        f.seek(0)  # if bytes get transferred, `value` is in buffer, otherwise, `value` should not be in memory yet.
    # we exited mock_s3, which means s3 is not available now
    print(f.read())  # value

I think you're misunderstanding how this works.
First, just because you've exited the context manager doesn't mean "S3 is unavailable now".  Try this:
import boto3
from moto import mock_s3

with mock_s3():
    c = boto3.client('s3')
    c.create_bucket(Bucket='bucket')
    c.put_object(Bucket='bucket', Key='key', Body=b'value')

    session = boto3.Session()
    s3 = session.resource('s3')
    obj = s3.Object('bucket', 'key')
    body = obj.get()['Body']  # At this stage, we've opened a stream, but have not read any bytes

print(body.read())

Second, you can verify that smart_open isn't loading anything into its buffers by slightly modifying your original example:
import boto3
from moto import mock_s3
from smart_open import open

if __name__ == '__main__':
    with mock_s3():
        c = boto3.client('s3')
        c.create_bucket(Bucket='bucket')
        c.put_object(Bucket='bucket', Key='key', Body=b'value')
        f = open('s3://bucket/key', 'r')
        f.seek(0)
        print("f.stream: %r buffer length: stream._buffer: %r" % (f.stream, len(f.stream._buffer)))

Running the above code:
(smart_open) misha@cabron:~/git/smart_open$ python seek.py
f.stream: <smart_open.s3.SeekableBufferedInputBase object at 0x7fd2d1d72e48> buffer length: stream._buffer: 0

So, after seeking, you can see that the buffer is empty.

Yes, you opened the streams but not read from it yet, but the network traffic is made, s3 server has sent the bytes to client already. s3 server do not have an API that send the bytes individually. Other than exiting mock_s3, you could suspend the network as well.
import boto3
from moto import mock_s3
from smart_open import open

import socket

def guard(*args, **kwargs):
    raise Exception("I told you not to use the Internet!")

if __name__ == '__main__':
    with mock_s3():
        c = boto3.client('s3')
        c.create_bucket(Bucket='bucket')
        c.put_object(Bucket='bucket', Key='key', Body=b'value')
        f = open('s3://bucket/key', 'r')
        f.seek(0)  # if bytes get transferred, `value` is in buffer, otherwise, `value` should not be in memory yet.

    # we suspend the network and fake s3
    socket.socket = guard
    print(f.read())  # value

    f.seek(0)  # network error

OK, so what's the actual problem here? Something out there may be doing some buffering, but are you sure it reads the full file? Have you tried opening a large file on actual S3 and seeking? On my end, seeking e.g. to the end of a large file is much faster than reading it.

I will try that, thanks for help so far.

I just upload a large file (s3://oss-playground/big.bin) (~1 GB) to s3 and try to capture the network traffic.
So, the get_object call won't read the whole file content into memory as a whole directly, but keeps an HTTP streaming connection to s3 server, which has some kind of buffer (~4K). And that's why socket trick does not prevent the read because the connection has already created and the size is too small.
So the issue becomes, when you call get_object in seek, it will create a HTTP connection to s3, try to read it through end. But if the read happened too late, the s3 server will close the connection on there side, and you will receive 'Connection reset by peer' error.
So the propose is still the same: move the get_object call to read func other than seek func. Otherwise if the time between seek - read or read - read is way too long to aws s3, which is kind of common if you try to do something chunk by chunk, s3 server will reset the connection and you can not read from it anymore.

OK, thank you for investigating.
I think your proposal is OK. We can delay this block until read.
Are you able to make a PR? We can probably make a pre-read callback or something to call that code just before reading happens.


Adjust changelog_uri

Adjust the URI so that link to the changelog is correct on RubyGems:

.

LGTM, thank you!


Add concurrency option to all operator.

Hi.
In my project promise use case, I use all operator with over hundred download promise tasks.
If all of the tasks execute at same time, tasks exhaust huge resources(cpu, network, memory).
So I added concurrency option to all operator, it will suppress resource usage with all operator behavior and parallel tasks execution benefit.
Regard.

Awesome addition! Thank you!


ui: Remove the unnecessary spinners from table pagination component

Please see the commit for details.

Please add Resolves #835. to the commit message to auto-resolve that issue on the PR merge.

Please add Resolves #835. to the commit message to auto-resolve that issue on the PR merge.

Already did, although I must say, it rarely actually auto-resolves the issue - even when I create a branch out of the issue.

it rarely actually auto-resolves the issue - even when I create a branch out of the issue.

@mnonnenmacher experienced this as well... probably some issue related to the Eclipse setup of the GitHub org. In "vanilla GitHub" the feature works just fine.


[Tracker] ODIS 2.0.0 code cleanup

Catch-all ticket for thorough (preliminary) code review of #9693 and any miscellaneous work that comes out of it. Will include cleaning up TODO statements, making new tickets when necessary, and identifying gaps in testing.
Note that this ticket is separate from #9729, which will happen later

[ ] @alecps break up the list of todos below

During spring planning:

Catch all ticket for small todos that are difficult to estimate.
We'll try to address being better at scoping in the remaining time of this sprint planning. We'll break this into separate tickets.
There are multiples scattered todos in the code that can't be addressed immediately to avoid going down a rabbit hole.

https://github.com/celo-org/celo-monorepo/pull/9782

See TODO list in #8645


Support for latest API level

Hello,
We are currently using your library in one of our application. Actually we need a favour from your side. According to the Google Play Policy, all apps have to provide support for at least API level 26 which is satisfied in your case. We are targeting API level 28. So can you please provide us support for API level 28 in your current repository?

Please check out the Android official docs. We just want to make sure that all the libraries support at least API level 26 and we are targeting API level 28. Please update as soon as possible.

bump

Hi you can try to check this library: https://github.com/Chivorns/SmartMaterialSpinner.
Hope it can help what you need


[FEATURE] Drive Motor Velocity Controller

Problem
Currently, drivetrain motors are controlled directly by the motion algorithms simply by controlling their voltage. This reduces the effectiveness of the motion algorithms, and makes velocity units confusing
Proposed Solution
The chassis class should have velocity controllers for drivetrain motors. This way, motion algorithms can simply request the drivetrain to move at a specified speed (e.g 6 in/s)

Dibs when I get some time available

I'm also going to include a turnAtSpeed function because its in a similar vein

Closing, as this is now up to motions to implement


Runtime as a dependency

I would like to make a first step towards https://github.com/google/traceur-compiler/issues/437, by simply having an option to enable adding a dependency to a custom-named traceur-runtime package, and then having that provide the reference to $traceurRuntime instead of a global.
It would be a very simple first PR, that would then lead to enabling splitting into a granular runtime over time with requires like import { $class } from 'traceur-runtime/class' when this option is enabled.
It would be useful for the jspm workflow as runtime becomes another module to handle at the bundling level instead of an exception to be magically handled somehow.
Let me know your thoughts, and any implementation ideas then I'll see if I can get together a PR.

We have too many issues covering this exact thing. Closing this.
See #1628 for a recent discussion and some early work in a branch of mine.


Add Emanuel School domain

Official Website: https://www.emanuelschool.nsw.edu.au/

@ezradevs Pull request merged. Thank you.


Updating the rad download url from old CDN to github releases.

Description
Updated the rad download url from old CDN to github releases for cli nightly test.
Type of change

This pull request fixes a bug in Radius and has an approved issue (issue link required).
This pull request adds or changes features of Radius and has an approved issue (issue link required).
This pull request is a minor refactor, code cleanup, test improvement, or other maintenance task and doesn't change the functionality of Radius (issue link optional).

Fixes: #issue_number
Auto-generated summary

🤖 Generated by Copilot at 325fb84
Summary
🔄🌐✅

Updated the test.yaml workflow to use the GitHub release URL and verify its status for downloading the rad binary.

Download file step
Changed to use GitHub URL
No more waste in spring

Walkthrough

Modify the Download file step to use the GitHub release URL and check the HTTP status code (link)

/ok-to-test


Using region from package config

Replacing hardcoded region (eu-west-1) with a value from the package config

Thanks for this!


Get iPhone Model Number

Description

Get iPhone Model Number. For example in iPhone X: A1865, A1901, A1902 (Japan)
Use Case
To identify if the device mobile data (4G especially) can work or not. Because some device have CDMA network.

According to this, the string we return is sufficient for you to tell CDMA or GSM or 4G etc
https://stackoverflow.com/questions/11197509/how-to-get-device-make-and-model-on-ios


Metrics Insights

Hi I am currently training a faster rcnn model for an obj detection task and am looking at the metrics provided in test.py.

Are there any ways for my model to output the number of TPs, FPs, TNs and FNs instead of just the mAP@IoU values?
What is the difference between COCO bbox eval and PASCAL_VOC mAP eval for an object detection task
For an object detection task, does the metric consider any predicted bounding box predicted by the model even with a low confidence score? it seems like the evaluate function of the datasets do not take into account of any model confidence score

@junhuplim Hi

You need to rewrite the dataset.evaluate function
The difference between them can be found through the official website introduction of COCO and VOC
In fact, bbox confidence score is considered when calculating TP and FP

My answer is relatively concise, if you have any questions, you can communicate.

@hhaAndroid Hello, thanks for the response!

Got it, am dabbling around there now
Are you able to direct me with the link? Have been looking around the docs but can't find the required information
I see, again are you able to point me to the relevant code blocks? I am currently looking at mean_ap.eval_map and mean_ap.average_precision functions now, (I am evaluating on a Custom dataset) but am unable to see how bbox confidence score is considered there. Am I looking at the wrong place?

Thanks again, really appreciate the help!

Okay, for (3), I think i have to look around the mean_ap.tpfp_default function instead - since you said it was how TP and FP is calculated. I realise that the det_bboxes has shape of (m, 5) while gt_bboxes has shape of (n, 4). Am i write to say that det_bboxes[:, -1] is then the bbox confidence score? If I am right, I still don't see how the confidence score is being considered

det_bboxes[:, -1] is the scores.
The confidence score is often used to sort the TP bboxes to calculate P-R curve.
When evaluating mAP, scores may be used to find top-k bboxes to calculate mAP. The most common is to take top-100, but there may also be a need to take top-10 or top-50.

Got it, thanks for your help!

Hi @hhaAndroid. How is the bbox confidence score on coco datasets actually calculated? Can you please refer me to any code/papers?
Thanks in advance.


Use two different database files for config and history

⚠ Please verify that this feature request has NOT been suggested before.

[X] I checked and didn't find similar feature request

🏷 Feature Request Type
Other
🔖 Feature description
I would like to be able to store the configuration of Uptime Kuma and the history data in two different database files. Ideally, the history database can be thrown away at any time (while Uptime Kuma is not running) without causing any problems.
✔ Solution
A database file in /app/data/config/kuma.config.db (with -shm, -wat etc.) containing all of Uptime Kuma’s configuration, and a database file in /app/data/history/kuma.history.db  (with -shm, -wat etc.) containing all of Uptime Kuma’s saved history information, the latter of which can be safely discarded of whenever Uptime Kuma is not running, and is silently recreated if missing at the next startup.
❓ Alternatives
For backups of the Uptime Kuma configuration, I considered

Making use of the deprecated config JSON export (which is 1. deprecated and may not export all information relevant for recreating my Uptime Kuma instance and 2. must be done manually after every configuration change)
Backing up the current database, which leaves me with unneccesarily large incremental backups.

📝 Additional Context
Currently, all information, including history, appears to be stored in the /app/data/kuma.db database. I would like to include the configuration information in my server’s incremental backups, so that after restoring the server from a backup, Uptime Kuma continues to work without any additional effort. The history information in the same database however

is quite large compared to the configuration (several 100 MBs), and
changes constantly while the configuration changes rarely,
so I end up with either large incremental backups every day or no backup of my configuration at all.

The proposed solution can be used to bind the docker container’s /app/data/config to some folder in my home directory, which is backed up regularly, while the docker container’s /app/data/history would be bound to some unimportant directory like /var/cache that is excluded from backups.

I strongly support this. Moving to a time series database like InfluxDB could be the next big step.

I'd go even further and use a sqlite database per monitor.

No, for that you would use a git-ops based notification system instead.
See gatus for example.

No, for that you would use a git-ops based notification system instead. See gatus for example.

If this is in response to my comment, I might have been misunderstood. I meant by splitting data/history from the config, you could backup the settings of UptimeKuma without backing up any data it's pings/requests/tests aggregate by running. I.e. you could backup the settings of the app without any of the data it produces. Unless I misunderstand, gatus is for an entirely different purpose than version control so I am assuming I was unclear before.


more examples needed for druid sql

druid sql is very highly under documented to the point of being unsuable.
at the very least, there should be example queries showing:

how to select a timerange, eg can you do something as simple as Interval(begin, end) or is a full begin>= x and end <=y needed every time
how to specify granularity
how to select time series data.
edges cases like what output is expected if data is missing in between
explaination of  generated json, like 'virtual columns' that are completely undocumented.

Hi @pdeva, the answers to your questions are,

A time filter looks like __time >= TIMESTAMP '2000-01-01 00:00:00' AND __time < TIMESTAMP '2000-01-02 00:00:00'.
If you group by a TIME_FLOOR function, that's like specifying granularity.
Not sure what you mean by this?
Not sure what you mean by this?
They are undocumented, but they won't be for long. http://druid.io/docs/latest/misc/math-expr.html might help you understand what they're doing.

I agree an examples section would be nice; @jon-wei was working on some new tutorials for querying, maybe he can chime in with whether he had planned on adding SQL examples.

maybe he can chime in with whether he had planned on adding SQL examples.

I have one example query for "wikipedia top pages" in the new tutorials (the idea was more to show the tools/workflow vs being a tutorial on expressing queries in SQL, do you have any suggestions for common queries that would be good examples?

@gianm  clarifying:
3. show atleast one query that selects data that is not a singular value but output as a time series. in all existing slides, examples i could find all queries output a singular value, vs something that would be shown on a graph as time series data.

say a query for time series data between 1am and 10 am, grouped hourly. and assume there are no segments for time period 10am-11am. will the output contain a result for 10-11 with a ‘value’ of 0 or will 10-11 period be simply missing from output. this is an important edge case to know about when showing data in graph.

(3) sounds like it's really the same question as (2), and TIME_FLOOR is the answer. (4) adheres to standard SQL: result rows are only emitted where there is actual data. So there would be no row for 10am–11am.

I have one example query for "wikipedia top pages" in the new tutorials (the idea was more to show the tools/workflow vs being a tutorial on expressing queries in SQL, do you have any suggestions for common queries that would be good examples?

Maybe four queries: one that plans to a scan, one that plans to a timeseries, one that plans to a topN, and one that plans to a groupBy?

@gianm regarding 4. is the behavior of native queries (topn, timeseries, groupby) the same as sql for this case? in that will they omit 10-11 from the results too?

@gianm regarding 4. is the behavior of native queries (topn, timeseries, groupby) the same as sql for this case? in that will they omit 10-11 from the results too?

topN and groupBy will; for timeseries it depends on the skipEmptyBuckets setting.

The new query tutorial has more SQL examples:
http://druid.io/docs/latest/tutorials/tutorial-query.html (0.12.2/0.12.3)
https://staging-druid.imply.io/docs/latest/tutorials/tutorial-query.html (master)

explaination of generated json, like 'virtual columns' that are completely undocumented.

Virtual columns are now documented: http://druid.io/docs/latest/querying/virtual-columns.html

The docs have been tremendously updated for this 0.16.0 release.
Furthermore the web console will not WRITE QUERIES FOR YOU 🚀

I am considering this 'solved'. If you have some more specific ideas for how to improve the docs please do not hesitate to file a new issue ❤

druid __time how to transfer datetime


Lots of errors while compiling.

I get a lot of errors relating to 'starts_with' and 'char8_t'. Any help is well, helpful.

Maybe i should actually update VS 2019.

close


Skipping E2E test for Packages

Skipping E2E Test for packages until a updated solution is created for packages.

/hold

/unhold

/cherrypick release-0.11

/approve


Make design system for Breadcrumbs

Make and awesome design system for Breadcrumbs Component.
See the Breadcrumbs and make one.
Refer to : Button Design System for any reference or ask me in any confusion.
Tip :
You can use Tailwind Playground

@Rajdip019 please assign me I would love to work on the issue.

@Rajdip019  How does this look ?

Made Few Changes. How is this one ? @Rajdip019
On Hover it changes white shadow to a little dark shade.

You need to make a design system for the breadcrumbs. But the first one looks good you can raise an issue @RavindraP04 that create breadcrumb 2. And then you can make a PR for this.

@Kumar-Ankit56 you can go ahead.

@Kumar-Ankit56 you can go ahead.

Thanks

You need to make a design system for the breadcrumbs. But the first one looks good you can raise an issue @RavindraP04 that create breadcrumb 2. And then you can make a PR for this.

@Rajdip019 Can you please elaborate, what do you mean by Design System ? I am unable to understand.

You need to make a design system for the breadcrumbs. But the first one looks good you can raise an issue @RavindraP04 that create breadcrumb 2. And then you can make a PR for this.

@Rajdip019
Ok, Now I understood that by Design System you meant to create mdx file like you mentioned for the button,
Btw, In the new issue I need to make a new file breadcrumb-2.html and then raise the PR for that, Right ?

You need to make a design system for the breadcrumbs. But the first one looks good you can raise an issue @RavindraP04 that create breadcrumb 2. And then you can make a PR for this.

@Rajdip019 I have raised the issue an also PR #677
I have created new file Breadcrumb/src/Breadcrumb-2.html and have added the code in that.
Do, Checkout.


Integration test for multichain provider flow

Fixes # .
Changes proposed in this PR:

added new integration test
added new utils file for intergration tests

closing this final version of the PR without unneeded code will be in #1698 PR


Auto permissions enabling and synthetic data for improving reasoning capability

Addressed the inconvenience of manually enabling various permissions during the workshop by enhancing the original Bicep script to automate the process of enabling the required permissions.
Implemented a method that integrates evolutionary learning and Monte Carlo Tree Search (MCTS) to synthesize training data, enabling the SLM to enhance its reasoning capabilities in a manner similar to O1.

@butterluo please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.
@microsoft-github-policy-service agree [company="{your company}"]

Options:

(default - no company specified) I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer.

@microsoft-github-policy-service agree

(when company given) I am making Submissions in the course of work for my employer (or my employer has intellectual property rights in my Submissions by contract or applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer. By signing below, the defined term “You” includes me and my employer.

@microsoft-github-policy-service agree company="Microsoft"

Contributor License Agreement

@microsoft-github-policy-service agree