Uncaught TypeError: Cannot read property 'parentElement' of null

[Enter steps to reproduce below:]

...
...

Atom Version: 1.10.0-dev-3fd0d30
Electron Version: 0.37.8
System: linux 4.6.2-1-ARCH
Thrown From: Atom Core
Stack Trace
Uncaught TypeError: Cannot read property 'parentElement' of null
At /usr/share/atom-git/resources/app.asar/src/lines-tile-component.js:282

TypeError: Cannot read property 'parentElement' of null
    at LinesTileComponent.module.exports.LinesTileComponent.buildLineNode (/usr/share/atom-git/resources/app.asar/src/lines-tile-component.js:282:42)
    at LinesTileComponent.module.exports.LinesTileComponent.updateLineNodes (/usr/share/atom-git/resources/app.asar/src/lines-tile-component.js:138:34)
    at LinesTileComponent.module.exports.LinesTileComponent.updateSync (/usr/share/atom-git/resources/app.asar/src/lines-tile-component.js:94:12)
    at LinesComponent.module.exports.TiledComponent.updateTileNodes (/usr/share/atom-git/resources/app.asar/src/tiled-component.js:67:19)
    at LinesComponent.module.exports.TiledComponent.updateSync (/usr/share/atom-git/resources/app.asar/src/tiled-component.js:30:12)
    at TextEditorComponent.module.exports.TextEditorComponent.updateSyncPreMeasurement (/usr/share/atom-git/resources/app.asar/src/text-editor-component.js:265:34)
    at TextEditorComponent.module.exports.TextEditorComponent.updateSync (/usr/share/atom-git/resources/app.asar/src/text-editor-component.js:205:12)
    at /usr/share/atom-git/resources/app.asar/src/text-editor-component.js:324:28
    at ViewRegistry.module.exports.ViewRegistry.performDocumentUpdate (/usr/share/atom-git/resources/app.asar/src/view-registry.js:221:9)
    at /usr/share/atom-git/resources/app.asar/src/view-registry.js:3:61

Commands
     -1:34 core:save (atom-text-editor.editor.is-focused)
     -1:13.2.0 core:move-down (atom-text-editor.editor.is-focused)
     -1:13 core:move-left (atom-text-editor.editor.is-focused)
     -1:11.7.0 core:move-down (atom-text-editor.editor.is-focused)
     -1:11.5.0 core:move-left (atom-text-editor.editor.is-focused)
     -1:09.5.0 core:move-down (atom-text-editor.editor.is-focused)
  2x -1:09.3.0 core:move-left (atom-text-editor.editor.is-focused)
     -1:06.1.0 core:move-down (atom-text-editor.editor.is-focused)
 27x -1:05.4.0 core:select-up (atom-text-editor.editor.is-focused)
  6x -0:58.5.0 core:select-down (atom-text-editor.editor.is-focused)
 50x -0:53.5.0 core:select-left (atom-text-editor.editor.is-focused)
  6x -0:51.1.0 core:select-right (atom-text-editor.editor.is-focused)
     -0:49.9.0 core:select-down (atom-text-editor.editor.is-focused)
 28x -0:49.5.0 core:select-left (atom-text-editor.editor.is-focused)
  4x -0:47.7.0 core:select-right (atom-text-editor.editor.is-focused)
     -0:45 core:copy (atom-text-editor.editor.is-focused)

Config
{
  "core": {
    "themes": [
      "one-dark-ui",
      "atom-dark-syntax"
    ]
  }
}

Installed Packages
# User
atom-dark-syntax, v0.27.0 (active)
atom-dark-ui, v0.52.0 (inactive)
atom-light-syntax, v0.28.0 (inactive)
atom-light-ui, v0.44.0 (inactive)
base16-tomorrow-dark-theme, v1.1.0 (inactive)
base16-tomorrow-light-theme, v1.1.1 (inactive)
one-dark-ui, v1.4.0 (active)
one-light-ui, v1.4.0 (inactive)
one-dark-syntax, v1.3.0 (inactive)
one-light-syntax, v1.3.0 (inactive)
solarized-dark-syntax, v1.0.2 (inactive)
solarized-light-syntax, v1.0.2 (inactive)
about-arch, v1.5.18 (active)
archive-view, v0.61.1 (active)
autocomplete-atom-api, v0.10.0 (active)
autocomplete-css, v0.11.2 (active)
autocomplete-html, v0.7.2 (active)
autocomplete-plus, v2.31.0 (active)
autocomplete-snippets, v1.11.0 (active)
autoflow, v0.27.0 (inactive)
autosave, v0.23.1 (active)
background-tips, v0.26.1 (active)
bookmarks, v0.41.0 (active)
bracket-matcher, v0.82.1 (active)
command-palette, v0.38.0 (inactive)
deprecation-cop, v0.54.1 (active)
dev-live-reload, v0.47.0 (active)
encoding-selector, v0.22.0 (active)
find-and-replace, v0.200.0 (active)
fuzzy-finder, v1.2.0 (active)
git-diff, v1.0.1 (active)
go-to-line, v0.31.0 (inactive)
grammar-selector, v0.48.1 (active)
image-view, v0.58.1 (active)
incompatible-packages, v0.26.1 (active)
keybinding-resolver, v0.35.0 (active)
line-ending-selector, v0.5.0 (active)
link, v0.31.1 (inactive)
markdown-preview, v0.158.0 (active)
notifications, v0.65.0 (active)
open-on-github, v1.1.0 (inactive)
package-generator, v1.0.0 (inactive)
settings-view, v0.238.1 (active)
snippets, v1.0.2 (active)
spell-check, v0.67.1 (active)
status-bar, v1.3.1 (active)
styleguide, v0.46.0 (active)
symbols-view, v0.113.0 (inactive)
tabs, v0.99.0 (active)
timecop, v0.33.1 (active)
tree-view, v0.208.0 (active)
update-package-dependencies, v0.10.0 (active)
welcome, v0.34.0 (active)
whitespace, v0.32.2 (active)
wrap-guide, v0.38.1 (active)
language-c, v0.52.1 (active)
language-clojure, v0.21.0 (active)
language-coffee-script, v0.47.1 (active)
language-csharp, v0.12.1 (active)
language-css, v0.37.0 (active)
language-gfm2, v0.90.4 (active)
language-liquid, v0.5.1 (active)
language-git, v0.14.0 (active)
language-go, v0.42.0 (active)
language-html, v0.44.1 (active)
language-hyperlink, v0.16.0 (active)
language-java, v0.23.0 (active)
language-javascript, v0.119.0 (active)
language-json, v0.18.1 (active)
language-less, v0.29.4 (active)
language-make, v0.22.2 (active)
language-mustache, v0.13.0 (active)
language-objective-c, v0.15.1 (active)
language-perl, v0.35.0 (active)
language-php, v0.37.1 (active)
language-property-list, v0.8.0 (active)
language-python, v0.45.0 (active)
language-ruby, v0.68.6 (active)
language-ruby-on-rails, v0.25.0 (active)
language-sass, v0.55.0 (active)
language-shellscript, v0.22.4 (active)
language-source, v0.9.0 (active)
language-sql, v0.22.0 (active)
language-text, v0.7.1 (active)
language-todo, v0.28.0 (active)
language-toml, v0.18.0 (active)
language-xml, v0.34.8 (active)
language-yaml, v0.26.0 (active)

# Dev
No dev packages

Any indication of what you were doing at the time?

Selecting multiple lines to edit
On 30 June 2016 at 02:06, Lee Dohm<EMAIL_ADDRESS>wrote:

Any indication of what you were doing at the time?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/atom/atom/issues/12055#issuecomment-229480055, or mute
the thread
https://github.com/notifications/unsubscribe/AIHFXfSWo-KEw3NPf9kUZUaHTe4g30YRks5qQtc3gaJpZM4I_aju
.

--
Tony Benoy

I just got this same error, Atom's popup error message had a link to this issue - Same stack trace, but I was trying to roll up a Python class.

Happens when I try to "fold all"

@albertstartup ,same with you!

@Jin-Whu yes
"Fold All" was working before 1.9.0 update.

Seems related to #10350.

I got this error when i opened the git diff using package<EMAIL_ADDRESS>


Add cluster setting for shard movement strategy

Description
Adds cluster setting for shard movement strategy that was missing from the docs.
Issues Resolved
Resolves #4827
Checklist

[ ] By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and subject to the Developers Certificate of Origin.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@Poojita-Raj Thanks for taking care of the final suggested changes. This looks good. I've merged the PR. Your contribution and efforts on this issue are appreciated.

@cwillum Thanks Chris, happy to help!


[WC] Redirect failing when choosing to open listed safe app

Bug description
As reported by the Cow Swap team, redirection is failing via the WC safe app when the user chooses to use listed safe app instead of following with the connection flow.
After some research it looks like the redirect is performed to:
https://app.safe.global/apps?safe={{safeAddress}}&appUrl=...
And should be to:
https://app.safe.global/apps/open?safe={{safeAddress}}&appUrl=...
Steps to reproduce

Go to WC safe app and connect to an app like CoW Swap that already has a listed safe app
When connecting, choose to use the listed safe app instead

Video with the process:
https://github.com/safe-global/safe-wallet-web/assets/92332710/8486093f-1fd2-4862-a3ad-330fa1e036f2
Expected result
Successful redirection to the listed safe app.
Obtained result
Unsuccessful redirection to the listed safe app.

@JagoFigueroa I've moved this issue to safe-react-apps, please add it to the Safe Apps board if necessary.

Question señor, wouldn't this fix require a PR on the wallet repo? (I think it was introduced here) so maybe it could make sense to leave it there.
Cheers!

Actually, you're right, the redirect doesn't work on the wallet side. Moving back :)


AST: More robust rewrite of ProtocolConformance::getInheritedConformance.

A recursive implementation is cleaner, and more cleanly preserves class inheritance and generic specialization through the operation to get the refined protocol conformance. Fixes SR-1323.

@swift-ci Please test

@swift-ci Please test

Don't forget to remove that debugging code. :-)

Thanks, I wanted the dumps in the CI run to help diagnose crashing cases in the release build.

@swift-ci Please test

@swift-ci Please test

Failure looks unrelated.


Error using mod in multiplayer

Reported by Athena in Discord

Closed by https://github.com/dcbroad3/pollution-scaling/commit/2483b0bb42ad7c444dea8350714dbcb7bf16740d


Standalone VM does not compile with -Wshorten-64-to-32

Found many warnings when adding the flag -Wshorten-64-to-32. Some of them are really problematic, while others just work by design (known limits to value ranges) or by accident.

We are currently not working on cleaning up the code to add this option, not sure it his is a p2 issue, removing label. If anybody thinks it needs to be a p2 issue please add a comment here.


Support SPM for Reachability target

Link to issue #117
Make the lib available through Swift Package Manager.
We can also build and test with "swift build" and "swift test" commands.

@lucas34 Thanks for you PR
Cheers
Ash

It's been added to https://swiftpackageregistry.com/ashleymills/Reachability.swift


[PE-D][Tester A] Finding by birthday is unintuitive

When using the find function, the user is unable to find by birth month.

Labels: type.FeatureFlaw severity.Low
original: 1st2GetThisName/ped#4

Thank you for pointing this out, we will plan for this enhancement in the future!


Add subscribeToFieldState method

The snapshot functions originally gave only access to specific values. This pull request exposes a new function which exposes the field as argument, allowing for more options.
This pull request should be an answer to the question "Can we detect when isVisited changes on any form field?"

Right, this explains why a custom hook I wrote wasn't working as expected.
An easy fix with this new export :)
Op do 1 okt. 2020 om 15:43 schreef EECOLOR<EMAIL_ADDRESS>
@EECOLOR commented on this pull request.
In example/src/domain/Full.js
https://github.com/kaliberjs/forms/pull/15#discussion_r498253988:

@@ -128,3 +131,54 @@ function Conditional({ field, children, reverse = false }) {
const value = useFormFieldValue(field)
return (reverse ? !value : value) && children
}

+function useSendSignalWhenIsVisited(form, f) {

const callbackRef = React.useRef(f)
React.useEffect(
() => {

 const unsubscribe = snapshot.subscribeToFieldState(

Great question. field.state gives you the state of the field itself. So
the subscription is only to changes of that particular field.
snapshot subscribes to changes in the field state and (if applicable)
changes in it's children (recursively).
Your questions lead to me believe the new name is poorly choosen. I think
we should rename it to subscribeToField.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/kaliberjs/forms/pull/15#discussion_r498253988, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABD6O6SXGKX7PAVFPKF23MTSISBOZANCNFSM4R7HL5SQ
.

--
Peeke Kuepers
Lead UI/UX developer
www.kaliber.net Kaliber Interactive
Zeedijk 19
3513 DA Utrecht
030 287 64 00


refactor

see individual commits, mostly moving things around and adding underscores to internal variables

Codecov Report

Merging #134 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master    #134      +/-   ##
=========================================
+ Coverage    99.8%   99.8%   +<.01%
=========================================
  Files           7       8       +1
  Lines        1024    1030       +6
=========================================
+ Hits         1022    1028       +6
  Misses          2       2

Impacted Files
Coverage Δ

rtcpeerconnection.js
99.73% <100%> (-0.01%)
:arrow_down:

writemediasection.js
100% <100%> (ø)

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 96202c3...2198428. Read the comment docs.


How to configure https nginx

django-DefectDojo/nginx
server {
server_tokens off;
listen 8443 ssl;
server_name dojotest.mydomain.com;
ssl_certificate /etc/nginx/ssl/mycert.crt;
ssl_certificate_key /etc/nginx/ssl/private.key;
ssl_session_cache builtin:1000 shared:SSL:10m;
# ciphers from https://ssl-config.mozilla.org/#server=nginx&version=1.17.7&config=intermediate&openssl=1.1.1d&guideline=5.4
ssl_protocols TLSv1.2 TLSv1.3;
ssl_ciphers ECDHE-ECDSA-AES256-GCM-SHA384:ECDHE-RSA-AES256-GCM-SHA384:ECDHE-ECDSA-CHACHA20-POLY1305:ECDHE-RSA-CHACHA20-POLY1305:ECDHE-ECDSA-AES128-GCM-SHA256:ECDHE-RSA-AES128-GCM-SHA25>
ssl_prefer_server_ciphers off;
location = /50x.html {
    root                    /usr/share/nginx/html;
}
location /static/ {
  alias /usr/share/nginx/html/static/;
}
location / {
  include /run/defectdojo/uwsgi_pass;
  include /etc/nginx/wsgi_params;
  uwsgi_read_timeout 1800;

The port 8443 doesn't respond. How can I configure SSL connection with nginx ?

Is your firewall open on 8443?
If you run sudo lsof -i :8443 does that show nginx listening on that port?
Can you reach 8443 from localhost? (where you installed DD)

The firewall is clear. There are no rules. Port 8080 respond and I can open web DD by http
lsof -i :8080
COMMAND     PID USER   FD   TYPE DEVICE SIZE/OFF NODE NAME
docker-pr 42074 root    4u  IPv4 152497      0t0  TCP *:http-alt (LISTEN)
docker-pr 42082 root    4u  IPv6 152503      0t0  TCP *:http-alt (LISTEN)
lsof -i :8443
COMMAND     PID USER   FD   TYPE DEVICE SIZE/OFF NODE NAME
docker-pr 42046 root    4u  IPv4 152666      0t0  TCP *:8443 (LISTEN)
docker-pr 42053 root    4u  IPv6 152433      0t0  TCP *:8443 (LISTEN)
curl -i <IP_ADDRESS>:8443
curl: (56) Recv failure: Connection reset by peer
root@cybersecops-test:/defectdojo-2.9.0/django-DefectDojo# curl -i <IP_ADDRESS>:8080
HTTP/1.1 302 Found
Server: nginx/1.23.1
Date: Tue, 24 Jan 2023 08:44:33 GMT
Content-Type: text/html; charset=utf-8
Content-Length: 0
Connection: keep-alive
Location: /login?next=/
X-Frame-Options: DENY
X-Content-Type-Options: nosniff
Referrer-Policy: same-origin
Vary: Cookie

How to use my own SSL certificate ?

I don't see any answers, please let me know about this.

Is there any info on how to use this on Ubuntu 20.04? I'm using docker-compose with docker-compose.yaml.

version: '3.8'
services:
nginx:
build:
context: ./
dockerfile: Dockerfile.nginx
image: "defectdojo/defectdojo-nginx:${NGINX_VERSION:-latest}"
profiles:
- mysql-rabbitmq
- mysql-redis
- postgres-rabbitmq
- postgres-redis
depends_on:
- uwsgi
environment:
USE_TLS: 'true'
GENERATE_TLS_CERTIFICATE: 'true'
NGINX_METRICS_ENABLED: "${NGINX_METRICS_ENABLED:-false}"
volumes:
- defectdojo_media:/usr/share/nginx/html/media
ports:
- target: 8080
published: ${DD_PORT:-8080}
protocol: tcp
mode: host
- target: 8443
published: ${DD_PORT:-8443}
protocol: tcp
mode: host
https works, but I need to use my own wilcard certificate.

any updates ?

How can I change 8443 port to 443 ?

no answer. no reaction


Make machine_learning modules importable

#21
This increases timeout to 2s as well. The only issues here are long import times because of large libraries. Locally, you may have issues if you don't have tk installed.
I'm a bit uncertain whether to move the import in 71ce432, or just increase the timeout even more. Making imports local is a common pattern to speed up initial imports of modules, so it's not a dirty trick. Reviewer opinion?

nvm


chore(e2e): Enable turbo cache for astro e2e tests

Description
It's time to revert this change, it's been a while since we had issues with false positives due to cache hits. This was done in the past in order to address some flaky runs

Checklist

[ ] npm test runs as expected.
[ ] npm run build runs as expected.
[ ] (If applicable) JSDoc comments have been added or updated for any package exports
[ ] (If applicable) Documentation has been updated

Type of change

[ ] 🐛 Bug fix
[ ] 🌟 New feature
[ ] 🔨 Breaking change
[ ] 📖 Refactoring / dependency upgrade / documentation
[ ] other:

Closing this, turns out this change is still not safe to make.


How do you change brush size?

It seems so simple but I cant figure it out

This might have broken recently. Usually, it's Shift + mousewheel. What browser + OS are you using?

Ah that did it. Sorry i just couldnt figure that out. Sorry for wasting time - thanks.

How about for Linux? On my Linux Mint machine that keybinding does a local magnification of the screen


Incorrect docs about importing moment in the example

@guidorice commented on Thu Jul 14 2016
I noticed this with skeleton-navigation-1.0.0-beta.3.0.1. Here are the repro steps
cd skeleton-navigation-1.0.0-beta.3.0.1
cd skeleton-typescript-webpack
npm install
npm start # confirmed app is working
npm install moment --save
# Then in app.ts added this:
import moment from 'moment';
# from aurelia docs http://aurelia.io/hub.html#/doc/article/aurelia/binding/latest/binding-value-converters/3
npm start

But now the build fails with
Module '".../node_modules/moment/moment"' has no default export.
This does however, work fine with skeleton-esnext-webpack, so it might be something wrong with the setup of the skeleton-typescript-webpack? Or I am not understanding how to import 3rd party party Js modules into this skeleton.

@EisenbergEffect commented on Thu Jul 14 2016
@niieani Ping

@niieani commented on Thu Jul 14 2016
You'll need to install typings for moment.

@guidorice commented on Thu Jul 14 2016
@EisenbergEffect @niieani
Guys, Please Forgive my ignorance... but it seems there is a gap between the aurelia documentation and the skeleton, which is what I'm trying to point out. I guess I could have worded the issue better.
typings install moment gives this warning which doesn't sound promising:
typings WARN hastypings Typings for "moment" already exist in "node_modules\moment\moment.d.ts". You should let TypeScript resolve the packaged typings and uninstall the copy installed by Typings
Then it fails with some other messages saying it's missing but it is in the dt repository.
Also what if I want to use a library that doesn't have typings available? For example with the numeral library, also suggested in the same documentation link above.
 typings install numeral  gives this error:
typings ERR! message Unable to find "numeral" ("npm") in the registry.`
Normally with webpack I can just do var foo = require('module'); and webpack figures it out. But that functionality seems to be different here. Maybe because of TypeScript? Or maybe because of the easy-webpack configuration being used in this skeleton?
Suppose I have a commonjs module with no typings available. How would I load it, using the webpack config included with skeleton-typescript-webpack?

@niieani commented on Thu Jul 14 2016
Ok, so it seems moment has built-in typings now, in that case you don't need to install external typings for it!
Looks to me that the way you're importing it (and the way it is presented in the docs) is incorrect. It should be:
import * as moment from 'moment';

We'll need to correct the docs. Sorry for closing too hastily.

Cool, thanks for looking into this. I's a change in the import syntax for numeral and moment modules, and also the usage in date-format.ts and currency-format.ts. They currently fail at runtime with errors:
aurelia-logging-console.js:54 ERROR [app-router] TypeError: moment is not a function(…)
aurelia-logging-console.js:54 ERROR [app-router] TypeError: numeral is not a function(…)

I think it's definitely an issue of where Babel has some special code that accounts for both usages, though it's not correct, while TS accounts only for the correct usage. So, the incorrect usage slipped into the docs because it's supported by Babel.

@EisenbergEffect Yes, exactly. I believe this feature is configurable via Babel presets.

My bug report/issue was with the latest skeleton-typescript-webpack. Isn't TypeScript the transpiler, not Babel? I am so confused!

@guidorice Indeed, we mangled two separate issues though. This one is about docs, which import moment incorrectly. Finally to end all confusion, the correct way to import (both in TypeScript and ES2015) is to:
import * as moment from 'moment';


Strangely longer for CPU inference using ONNX than the original PY repo

System Info
Debian 11 on CPU, Python3.10
optimum : 1.13.1
onnx : 1.14.1
onnxruntime : 1.15.1

Who can help?
@philschmid, @michaelbenayoun, @JingyaHuang, @echarlaix
Information

[X] The official example scripts
[ ] My own modified scripts

Tasks

[X] An officially supported task in the examples folder (such as GLUE/SQuAD, ...)
[ ] My own task or dataset (give details below)

Reproduction (minimal, reproducible, runnable)
Hey there, I've been running inference a few different ways to see what's faster, and been getting weird results.
I've been using
-> ORTModelForSeq2SeqLM
-> task: "summarization" with pipeline
-> model: Xenova/bart-large-cnn, which I've pulled to local drive and running using direct Path(/home/admin/models/...)
-> decoder_model_quantized, and encoder_model_quantized.
I've limited max new tokens and max length to 100.
I've been using an excerpt from a science paper on black holes (coz they are cool)... which is 2437 chars, or 502 tokens in length.
Running optimum runtime pipeline using the recommended setup:  https://huggingface.co/docs/optimum/v1.3.0/en/onnxruntime/modeling_ort#optimum.onnxruntime.ORTModelForSeq2SeqLM.forward.example-2
Expected behavior
When running the optimum approach per link (above in reproducing steps) this takes 2.5 to 3 minutes to summarize the text.
But using the very basic pipeline from ffacebook/bart-large-cnn,  https://huggingface.co/facebook/bart-large-cnn#how-to-use, it only takes 50seconds to summarize the same text.
I'm not sure what's going on.. I was expecting the onnx method to blitz the original.
Is it a mismatch between the task/model/class method? .. or is the Xenova onnx quant done on a different cpu/gpu setup going to make the quant version inappropriate for my specs?
What's holding onnx back vs the original?

Hi @gidzr, can you try using ORTModelForSeq2SeqLM.from_pretrained(..., use_io_binding=True) to initialize the ORT model?
In Xenova/bart-large-cnn there is a quantized and non-quantized model in the same repo (which is probably not loadable in ORTModel actually @xenova). Which one are you using?

@fxmarty I manually do into the local folder and add XXX to the extension to disable alternate available models in the same folder. Usually if I don't do this, the error response in CLI points out that there are multiple models.
re 'use_io_binding=True', I'll try this out and get back to you.

Hi @gidzr, can you try using ORTModelForSeq2SeqLM.from_pretrained(..., use_io_binding=True) to initialize the ORT model?
In Xenova/bart-large-cnn there is a quantized and non-quantized model in the same repo (which is probably not loadable in ORTModel actually @xenova). Which one are you using?

Here's the quantization settings I was using (along with which ops were quantized and their weight types): https://huggingface.co/Xenova/bart-large-cnn/blob/main/quantize_config.json.

Hi @xenova

apologies for the shocking grammar in my previous explanation. That's a 2am message.
just to clarify, I've ensured that only 1 type of model is in the folder when inference is run. I disable the non-quant file by changing their file extensions/mimes.
I'm going to try optimise/export and quant onnx again on my localserver to see if the issue arise with ONNX generally, or specifically with the Xenova version and will put my results here.

Chat soon.

Hi @xenova and @fxmarty
I wanted to follow up with some testing, as I've asked a few questions that can be rolled up and wanted to get something of substance back to you.

Inference Environment: is a server with 8GRam, 2CPU, 30G swapram, Debian 11, apache server..
Quantization Environment: same OS/python/versions as the Inference Environment, but with 32G, 8CPU
Models Tested: facebook/bart-large-cnn, xenova/bart-large-cnn, and homebaked onnx of the facebook repo
Input: Summarizing the same halfpage of text, keeping the same decoding strategies across all tests, with_past/cache (used config settings from Xenova onnx as a baseline, eg. beams, ngram limits, early stopping, use_cache, etc)

Based on the results, my only remaining question is:
What could cause the Homebaked onnx to go crazy.. even after adjusting all the Opt and Quant settings available to CLI command?
Babbling incoherence. produced by Homebaked ONNX quantization
[{'summary_text': 'The black hole is a space space, a space of space, in the universe, and the space of the universe. In the space, the black holes of the galaxies of the space space. The black holes is a gravitational gravitational energy.
Couple of extra weird bugs for Xenova:

running Xenova from repo via the ORT pipeline
running Xenova from repo via the ORTModel/AutoToken method

Observations:

Xenova ONNX was much faster, but had to run locally as repo inference had issues, and no gross difference when running via ORT pipeline vs ORTModelFor.
Optimum runtime CLI: Homebaking optimization/quantization to "summarization" task, didn't produce decoder_with_past. Decoder_with_past, and I was only able to obtain with_past files using auto-detect (no task) or text2text-generation-with-past. Which is a little weird because how do you optimize for summarization AND keep use_cache=True?
Homebaked quantized onnx files (via the CLI command) produced incoherent summaries even after replacing all the config files .json files with the Xenova working .jsons. I've tried optimizations O1-O4, then --arm, then --avx2, then --avx512, and finally with --avx512_vnni. All produce the same results incoherent babble.

Babbling incoherence. produced by Homebaked ONNX quantization
[{'summary_text': 'The black hole is a space space, a space of space, in the universe, and the space of the universe. In the space, the black holes of the galaxies of the space space. The black holes is a gravitational gravitational energy.   The first black hole known was Cygnus X-1, identified by several researchers independently in 1971. The presence of a black hole can be inferred through its interaction with other matter and with electromagnetic radiation..'}]
TESTING Results
A. Model-Tokenize (not pipeline) Inference

Baseline - Local pytorch: 28 seconds with Accelerate model, to(device=cpu); summary was coherent
Repo: "Xenova/bart-large-cnn" using ORTModelFor..: LOTS of junk warning or issues with quantization?
... lots of these:
2023-09-26 05:56:04.176734927 [W:onnxruntime:, graph.cc:3543 CleanUnusedInitializersAndNodeArgs] Removing initializer '/model/decoder/layers.6/self_attn_layer_norm/Constant_output_0'. It is not used by any node and should be removed from the model.
...
LOCAL: "/home/xxx/xenova_bart-large-cnn" using ORT pipeline: 20 seconds, summary was coherent
LOCAL: Homebaked onnx quant facebook-bart-larg-cnn: 24 seconds, summary was RANDOM mess.

B. Pipeline Inference

Baseline - Local pytorch: 25 seconds with Accelerate ; summary was coherent
Repo: "Xenova/bart-large-cnn" using ORT pipeline: ERRORS Repo id must use alphanumeric chars or '-', '_', '.', '--' and...
LOCAL: "/home/xxx/xenova_bart-large-cnn" using ORT pipeline: 20 seconds, summary was coherent
LOCAL:  Homebaked onnx quant facebook-bart-larg-cnn: 24 seconds, summary was RANDOM mess.

@gidzr Thanks for the extensive tests! I also had issues with incoherent results with default quantization settings and I fixed it by setting per_channel=True and reduce_range=True. I'm not sure if your tests included this, but these should (basically) be the only differences between your and my model.
See my conversion script for more details on the process (which, of course, uses optimum in the background).

@xenova Hey, no worries re testing :) .. thanks heaps for the steer on channel/ranges.
Unfortunately.. no dice..
Testing Scenarios:

Running CLI method, with --per_channel, O1, and avx512_vnni (using with_past task)
Running CLI method, with --per_channel, O2, and avx512 (using with_past task)
Running convert.py method, with --opset 11 --device cpu --per_channel true --reduce_range true (using with_past task)
Running convert.py method, with --opset 18 --device cpu --per_channel true --reduce_range true (using with_past task)

Results
Unfortunately in all the above cases, I received the same pattern of errors (see below)
Next Steps
Will try again using different combos of per_channel and reduce_range
What is opset?
While I'm doings some brute force trial and error wishful thinking blackbox solutioning.. could you point me to a doc that has definitions for opsets? I am not sure if these are an umbrella for chipset support (avx512/arm/etc), optimization strategy (O1-O4), or both?

Running Quant on CLI Method, with --per_channel:
$ sudo optimum-cli export onnx --model facebook/bart-large-cnn --task text2text-generation-with-past --optimize O1 /destination/OPTIM/folder/
$ sudo optimum-cli onnxruntime quantize --onnx_model /destination/OPTIM/folder/ --avx512 -o /destination/QUANT/folder/ --per_channel

ERROR:
onnxruntime.capi.onnxruntime_pybind11_state.Fail: [ONNXRuntimeError] : 1 : FAIL : Load model from /home/admin/NEW_TEST_QUANT/BART_LARGE_CNN_seq2seq-lm-with-past_per_channel/encoder_model_quantized.onnx failed:/onnxruntime_src/onnxruntime/core/graph/model_load_utils.h:56 void onnxruntime::model_load_utils::ValidateOpsetForDomain(const std::unordered_map<std::__cxx11::basic_string, int>&, const onnxruntime::logging::Logger&, bool, const string&, int) ONNX Runtime only guarantees support for models stamped with official released onnx opset versions. Opset 19 is under development and support for this is limited. The operator schemas and or other functionality may change before next ONNX release and in this case ONNX Runtime will not guarantee backward compatibility. Current official support for domain com.ms.internal.nhwc is till opset 18.

Running Quant on Convert.py, with --per_channel true --reduce_range true
$ sudo /usr/local/bin/python3.10 convert.py --model_id facebook/bart-large-cnn --quantize true --output_parent_dir /destination/quant/folder --task text2text-generation-with-past --opset 11 --device cpu --per_channel true --reduce_range true

ERROR:
File "/root/projectName11/lib/python3.10/site-packages/onnxruntime/capi/onnxruntime_inference_collection.py", line 397, in _create_inference_session
sess = C.InferenceSession(session_options, self._model_path, True, self._read_config_from_model)
onnxruntime.capi.onnxruntime_pybind11_state.Fail: [ONNXRuntimeError] : 1 : FAIL : Load model from /home/admin/NEW_TEST_QUANT/BART_LARGE_CNN_text2text-generation-with-past_xenovascript/facebook/bart-large-cnn/decoder_model_merged_quantized.onnx failed:/onnxruntime_src/onnxruntime/core/graph/model.cc:146 onnxruntime::Model::Model(onnx::ModelProto&&, const PathString&, const IOnnxRuntimeOpSchemaRegistryList*, const onnxruntime::logging::Logger&, const onnxruntime::ModelOptions&) Unsupported model IR version: 9, max supported IR version: 8

Very interesting 👍 Good to know it aligns with the default settings for transformers.js models (which I chose based on my own small-scale tests).
cc @pcuenca (re: benchmarking efforts)

Very interesting indeed! Very surprised to see such huge differences with similar looking configurations.

@xenova might be time for the current CLI / inference quantization methods to be retired in favour of the Joshy special!
@xenova, @fxmarty, @pcuenca:  Are there any notes, docs, or urls regarding opsets? All I can find is: https://onnxruntime.ai/docs/performance/model-optimizations/quantization.html

Hey @xenova
Came across an issue, that might be related to onnxruntime, or some other restriction related to optimization/quantization.
I downgraded to onnxruntime 1.15, using opset 16 and 17, to quantize Llama based models.
Problems, optimization occurs, but quantization fails / killed. I thought this was due to memory limitations, but this was occurring for optimized models of 3G size, which weren't an issue for other architectures. eg. PY007/TinyLlama-1.1B-Chat-v0.2 will optimize to 3G then fail to quantize. The other Llama models will optimize to 26G or 12G files, for the various different Llama flavours/repos.
Are there any limitations when quantizing Llama models with the Xenova convert.py script?
Cheers

Are there any limitations when quantizing Llama models with the Xenova convert.py script?

It should act in the same way as performing the two steps with optimum separately: (1) conversion, followed by (2) quantization. So, nothing extra in terms of memory usage.

Problems, optimization occurs, but quantization fails / killed. I thought this was due to memory limitations, but this was occurring for optimized models of 3G size

Could you share the id of the model you are trying to convert? Are the weights stored as 4, 8, 16, or 32-bit? I do know that the quantization process is quite memory-hungry, and also depends on some other quantization settings.

@xenova
Sure thing, this is the conditions for the testing:
Environment
In all cases, I'm running server with CPU, 8 cores, 32Gram, 580Gssd, 60Gswapram, Debian 11, Apache latest.
Outline of Steps

The xenova / joshy special script successfully optimized-exported these llama based models to onnx files (going from pytorch 12G size to 26G size onnx optimized).
The second half of the xenova script to quantize failed.
I then continued the process using the CLI onnx runtime script to convert the onnx optimized models, and this was successful, in most cases reducing a 26G optimized onnx file to a quantized onnx 6G size.
Mostly these were llama based, but not solely, and either size, architecture, or both may have played a role.

Script settings and results
Models tested, using both "causal-lm", and "default" settings, with the channel and ranges = true (per above discussion)
'codellama/CodeLlama-7b-Instruct-hf',           //CLI quanted to a 6G file
'togethercomputer/Llama-2-7B-32K-Instruct',     //CLI quanted to a 6G file
'mosaicml/mpt-7b-instruct',                     //CLI quanted to a 6G file
'togethercomputer/RedPajama-INCITE-7B-Instruct',//CLI quanted to a 6G file
'VMware/open-llama-7b-v2-open-instruct',        //CLI quanted to a 6G file
'mlfoundations/open_lm_1B',                     //CLI quanted to a 6G file
'PY007/TinyLlama-1.1B-Chat-v0.2',               //CLI quanted to a  1G file
'microsoft/DialoGPT-large',                     //CLI quanted to a <1G file
Cheers


Configuration for Detection

Hi, is there any instruction or command to pre-train the EVA on the Object 365 dataset? Thanks.

Hi @vateye, we have added instructions to pre-train EVA on the Object365 at https://github.com/baaivision/EVA/tree/master/EVA-02/det#object365-intermediate-fine-tuning


Rm unused dependencies (pass lint)

Oh great, must have forgotten these when I was doing some code refactoring. Thanks!


Deployment issue after latest update

Issue Type: Bug
Step 1: Create project with manifest
Step 2: Retrieve source from org
Step 3: Edit some LWC code and deploy to org, it will work fine
Step 4: Make some more changes to the same LWC, nothing will be deployed
Changes will not reflect in Org, It will show that source is deployed successfully but nothing will happen, after tracing the issue I found that FilePath is becoming null after first deployment
VS Code version: Code 1.54.1 (f30a9b73e8ffc278e71575118b6bf568f04587c8, 2021-03-04T22:38:31.419Z)
OS version: Windows_NT x64 10.0.19042

System Info

Item
Value

CPUs
Intel(R) Core(TM) i3-1005G1 CPU @ 1.20GHz (4 x 1190)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onprotected_video_decode: enabledrasterization: enabledskia_renderer: enabled_onvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
7.70GB (1.08GB free)

Process Argv
--crash-reporter-id ccd9f00f-6169-481e-8790-8185f203faa1

Screen Reader
no

VM
0%

Extensions (12)

Extension
Author (truncated)
Version

vscode-eslint
dba
2.1.14

prettier-vscode
esb
5.9.2

vscode-xml
red
0.15.0

salesforce-vscode-slds
sal
1.4.0

salesforcedx-vscode
sal
51.3.0

salesforcedx-vscode-apex
sal
51.3.0

salesforcedx-vscode-apex-debugger
sal
51.3.0

salesforcedx-vscode-apex-replay-debugger
sal
51.3.0

salesforcedx-vscode-core
sal
51.3.0

salesforcedx-vscode-lightning
sal
51.3.0

salesforcedx-vscode-lwc
sal
51.3.0

salesforcedx-vscode-visualforce
sal
51.3.0

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383:30185418
pythonvspyt617cf:30253240
vspor879:30202332
vspor708:30202333
vspor363:30204092
pythonvsdeb440:30248342
pythonvsded773:30248341
pythonvspyt875:30259475
dockersubsetcf:30265999
pythontb:30265425
vspre833cf:30267465

(Experimental duplicate detection)
Thanks for submitting this issue. Please also check if it is already covered by an existing one, like:

Can not retrieve source from org (#115364)

Not covered

/causedByExtension please report this to the authors of the Salesforce extension.


Create publish-immutable-action workflow

This PR creates a workflow that will publish an immutable action whenever a CodeQL Action release is created. We need to distinguish this from CodeQL Bundle releases to ensure that only the appropriate releases trigger the publishing process.
Merge / deployment checklist

[x] Confirm this change is backwards compatible with existing workflows.
[x] Confirm the readme has been updated if necessary.
[x] Confirm the changelog has been updated if necessary.

Don't merge yet. I want to make sure the release workflow is working properly first.

@Jcambass, could you take a look to make sure this will be sufficient for immutable actions?

Failures unrelated.


Where can I find more recent CIS Kubernetes benchmark for AKS 1.23 or higher ?

Where can I find more recent CIS Kubernetes benchmark for AKS 1.23 or higher ?

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

ID: ddaa7788-52c5-88ac-473e-ecd0d4e1b203
Version Independent ID: 753718bc-a2a0-38ad-43b4-1bbf395aef8b
Content: Center for Internet Security (CIS) Kubernetes benchmark - Azure Kubernetes Service
Content Source: articles/aks/cis-kubernetes.md
Service: container-service
GitHub Login: @MGoedtel
Microsoft Alias: magoedte

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

@atiasadir
I'm going to assign this to the document author so they can take a look at it accordingly.
@MGoedtel
Could you please add your comments on above query

@atiasadir - Thanks for bringing up this question. I'm creating a work item for this and will prioritize for our December sprint, as we'll need to work with the product group to properly update this information.

#please-close as we are working on updating the benchmark based on newer versions of AKS released since this article was last updated.


debugger problem with 2.0.1-beta.33 (30,31,32)

Issue Description:
Environment
$ uname -a
Linux alx-jeeng 4.15.0-29-generic #31~16.04.1-Ubuntu SMP Wed Jul 18 08:54:04 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

$ lsb_release -a
No LSB modules are available.
Distributor ID:	Ubuntu
Description:	Ubuntu 16.04.5 LTS
Release:	16.04
Codename:	xenial

$ node -v
v8.9.4

$ func -v
2.0.1-beta.33

VSCode:
Version: 1.25.1
Commit: 1dfc5e557209371715f655691b1235b6b26a06be
Date: 2018-07-11T15:40:20.190Z
Electron: 1.7.12
Chrome: 58.0.3029.110
Node.js: 7.9.0
V8: 5.8.283.38
Architecture: x64

Steps to reproduce issue

open VSCode
run npm start in console
press F5 to connect debugger

Describe the results you received
error popup window with message:
Cannot connect to runtime process, timeout after 10000 ms - (reason: Cannot connect to the target: connect ECONNREFUSED <IP_ADDRESS>:5858).
What is inside your .vscode/launch.json file
{
  "version": "0.2.0",
  "configurations": [
    {
      "name": "Attach to Azure Functions",
      "type": "node",
      "request": "attach",
      "port": 5858,
      "protocol": "inspector"
    },
  ]
}

Known workarounds:
sudo npm i -g<EMAIL_ADDRESS>--unsafe-perm true

@alexfurmanov just released an update to the functions extension in VS Code that asks to update your project the next time you open it. We have to do this because of breaking changes in the functions runtime. You can see https://aka.ms/AA1vrxa for more information, including manual instructions for updating your project


Problem with the developer version

Describe the bug
when i need to run a dev version using npm or yarn
I kept facing this problem:

Which OS are you currently running it from?

Mac OS Big Sur

Mac OS Big Sur
Confirm if the issue is fixed.

This has been fixed.


Feature Request: Snowflake: Multithreading for performance?

Is your feature request related to a problem? Please describe.
It seems like running generate sources sends the DESCRIBE TABLE ... statements to Snowflake sequentially one-by-one as it goes. It'd be great if this went a lot faster.
Describe the solution you'd like
Would it be reasonable to queue up all those database statements up front and run through them as the results return, so that it completes much more quickly?
Describe alternatives you've considered
Can't really think of any apart from just using it as it is and waiting much longer.
Additional context
python 3.10.9
Snowflake 7.3.1
macOS 13.2 (22D49)
output of pip freeze:
agate==1.6.3
asn1crypto==1.5.1
attrs==22.2.0
Babel==2.11.0
bump2version==1.0.1
bumpversion==0.6.0
certifi==2022.12.7
cffi==1.15.1
charset-normalizer==2.1.1
click==8.1.3
colorama==0.4.5
commonmark==0.9.1
cryptography==36.0.2
dbt-core==1.3.2
dbt-coves==1.3.0a25
dbt-extractor==0.4.1
dbt-snowflake==1.3.0
filelock==3.9.0
future==0.18.3
hologram==0.0.15
idna==3.4
importlib-metadata==6.0.0
isodate==0.6.1
jaraco.classes==3.2.3
Jinja2==3.1.2
jsonschema==3.2.0
keyring==23.13.1
leather==0.3.4
Logbook==1.5.3
luddite==1.0.2
MarkupSafe==2.1.2
mashumaro==3.0.4
minimal-snowplow-tracker==0.0.2
more-itertools==9.0.0
msgpack==1.0.4
networkx==2.8.8
oscrypto==1.3.0
packaging==21.3
parsedatetime==2.4
pathspec==0.9.0
pretty-errors==1.2.25
prompt-toolkit==3.0.36
pycparser==2.21
pycryptodomex==3.17
pydantic==1.10.4
pyfiglet==0.8.post1
Pygments==2.14.0
PyJWT==2.6.0
pyOpenSSL==22.0.0
pyparsing==3.0.9
pyrsistent==0.19.3
python-dateutil==2.8.2
python-slugify==7.0.0
pytimeparse==1.1.8
pytz==2022.7.1
PyYAML==6.0
questionary==1.10.0
requests==2.28.2
rich==12.6.0
ruamel.yaml==0.17.21
ruamel.yaml.clib==0.2.7
six==1.16.0
snowflake-connector-python==2.7.12
sqlparse==0.4.3
text-unidecode==1.3
typing_extensions==4.4.0
urllib3==1.26.14
wcwidth==0.2.6
Werkzeug==2.2.2
yamlloader==1.2.2
zipp==3.12.0

@BAntonellini
Hey, Bruno. Just wanted to bump here to get thoughts

dbt-osmosis recently implemented something like this and it helped performance immensely.
Fivetran's Salesforce schema alone, for example, has 776 tables. I have a few other large schemas.
Running dbt-osmosis on multiple schemas / DBs becomes extremely slow (hours).

Hey @jaredx435k2d0
We are aware this would be a good addition to dbt-coves, as it would be beneficial for use-cases like yours.
If you feel like contributing, follow our CONTRIBUTING guide and we will review it.

Good to have it acknowledged.
If I had the skills, I'd absolutely do this myself.
I'm learning, so maybe if it's not done in a few months I'll take a stab at it.

It is a good idea, we just have a lot on our plate and need to prioritize. I dont know many people running this against 776 tables 😱
We can prioritize with some $ :)
We also help our customers out, so consider Datacoves.com


[zh] Will localize blog/_posts/2023-03-01-introducing-kwok/index.md

I would like to localize this kwok blog into zh-cn. To avoid any conflict, this issue is opened.
/assign
/language zh

/triage accepted


override isn't required in ambient contexts with noImplicitOverride

🔎 Search Terms
override ambient type declaration class dts noImplicitOverride
🕗 Version & Regression Information

This is the behavior in every version I tried

⏯ Playground Link
https://www.typescriptlang.org/play/?noImplicitOverride=true&ts=5.8.0-dev.20241128&ssl=15&ssc=2&pln=1&pc=1#code/PTAEAEDsHsEkFsAOAbAlgY1QFwPIDcBTAJyNQBMCAuULIgVwIChGL1kBDIg0N9gZz6gAQv24BvRqFAAjapDrxpxANyMAvs1YcuPDgNAARYqkJlQBAB5YCkMoJF9xkmXIVKiqjSwK8dvfQ4EAEygYl5anNz+gkakpiGW1rb2oiESUtCEJOTcsqDyiirqjEA
💻 Code
// @noImplicitOverride: true

declare class Base {
  b: number;
}

declare class Derived extends Base {
  b: number; // no error
}

declare class Base2 {}

declare class Derived2 extends Base2 {
  override b: number; // error
}

🙁 Actual behavior
A property in a derived class in ambient context doesn't have to have override modifier with noImplicitOverride. override is still checked when present though.
🙂 Expected behavior
It feels slightly inconsistent. I couldn't find any note about this in the docs and I had to look into the source code and dig up the PR that implemented override to check if it's a bug or not.
Additional information about the issue
I believe this works like that since noImplicitOverride was introduced in https://github.com/microsoft/TypeScript/pull/39669 in 4.3: TS playground

Seems like there was a decision:

- Should this apply to declaration files?
  - Argument for yes: correctness for the same reason
  - Argument for no: convenience, no need to mess with declaration emitter
  - If no: Declaration emitter will need to emit `override` modifier (same way `any` works) if one was not present explicitly
  - Decision: Not enforced

Originally posted by @RyanCavanaugh in #9034


expose some structs to be able to use a non-tokio executor

For Inox(https://github.com/vhdirk/inox/) I'm combing tarpc with the glib executor as opposed to the default tokio executor.
In order to also have deadlines for requests, I needed to fork ClientHandler in my own project. That requires some changes in tarpc so I can create, for instance, a response struct. See https://github.com/vhdirk/inox/blob/master/inox-gtk/src/webextension/rpc.rs
I'm still working on the client side. That one is harder since tokio's TimeOuts are not compatible with custom executors, but they are used throughout the (generated) client code.

Sorry for the long wait! Given that we don't currently have a better way to make tarpc executor agnostic, I think I'm basically okay with this. I'm a little concerned about forward compatibility hazards, but not sufficiently concerned to block this.

@vhdirk I'm sorry this languished for so long! :( I think it's best at this point to close it. I think there's still a conversation to be had about how best to support different executors.


Remove import of logging library

Libraries should not import both SLF4J and a logger. Doing so voids the purpose of a common logging framework completely, since it forces a logging library on the end-user rather than leaving the ability to choose one.
From the SLF4J's official FAQs:

Embedded components such as libraries not only do not need to configure the underlying logging framework, they really should not do so. They should invoke SLF4J to log but should let the end-user configure the logging environment. When embedded components try to configure logging on their own, they often override the end-user's wishes. At the end of the day, it is the end-user who has to read the logs and process them. She should be the person to decide how she wants her logging configured.

You can configure your log settings under resources -> logback.xml

You can configure your log settings under resources -> logback.xml

What about if I don’t want to use Logback? And what about if I don’t want to use any logger at all?
The purpose of SLF4J is exactly this - to provide a wrapper for logging, but to also let the end-user itself choose it. Since this project is intended as a library, including both Logback and SLF4J (or any other logger from the SLF4J family) completely voids the effectiveness of the latter.
Here is in practical terms what is happening in the current state:

The user doesn’t need a logger -> the logger is forced upon him anyways
The user wants a logger but not Logback -> the user will be stuck with both loggers since Maven will include both. He will also need two config files at the same time
The user wants to use Logback -> everything works as intended, but at this point SLF4J isn’t needed anymore, since anything could be achieved with Logback anyways.

The only practical way to fix this is to remove Logback from the Maven dependencies (and any other reference to it left in the code, although if SLF4J was used correctly there shouldn’t be any).

I've also pushed a commit to remove any further logback reference in the code. Everything should be good to go!

If you remove the logback dependencies, does the project compile? SLF4J requires an underlying logging framework in order for it to work.

If you remove the logback dependencies, does the project compile?

It does, since SLF4J injects the logger at runtime. The only side-effect is a message printed in console, warning that SLF4J is running without a logger loaded in. It shouldn't be a problem since the final user might only see it if he is not importing a logger either.

This error is thrown when i try to run an example using your PR, can you try it?

This is the warning message I was talking about.
It is absolutely normal and won’t disrupt execution by any means, since it’s not an Exception nor an interrupting error. Its purpose is to warn the final user no logger was found; therefore, any logging operation won’t take place. As a matter of facts NOP means “no operation” since no logging operation will take place. Because you are trying to build a Java library instead of an end-user application, you can simply ignore that message; it’s not directed to you.
While importing this library, the end-user will be able to pick any logger with SLF4J support, and they will never encounter a single problem. Only if they do not wish to use a logger, the warning message will appear, but everything will work fine. If the end user really wish to get rid of the warning message without picking a specific logging environment, they should import slf4j-nop instead. It's a logging environment that does literally nothing, only suppressing the previous warning message.
Check it out for yourself here and here:

"If you are responsible for packaging an application and do not care about logging, then placing slf4j-nop.jar on the class path of your application will get rid of this warning message."

"Note that embedded components such as libraries or frameworks should not declare a dependency on any SLF4J binding but only depend on slf4j-api. When a library declares a compile-time dependency on a SLF4J binding, it imposes that binding on the end-user, thus negating SLF4J's purpose."

Hi guys.
The method
com.binance.connector.client.impl.spot.exchangeInfo()
dose not handle this:
curl -X GET "https://api.binance.com/api/v3/exchangeInfo"


Floc is not supprted by user agent, however, can still get cohort id.

By looking at featurePolicy.features()  I see that from https://developer.mozilla.org/en-US/docs/Web/API/FeaturePolicy,
Returns a list of names of all features supported by the User Agent. Feature whose name appears on the list might not be allowed by the Feature Policy of the current execution context and/or might not be accessible because of user's permissions.
Example (interest-cohort is supported)

However from what we see in the floc data, even though we do not have interest-cohort in the returned list (which means interest-cohort is not supported by UserAgent), we can still get cohort-id from pages.
Any idea on why we could still get cohort id when interest-cohort is not supported (featurePolicy.features().includes('interest-cohort') == false) ?

Hi Bella
Thanks for posting here.
document.featurePolicy.features() returns a list of features supported by the browser. The screenshot here appears to show that interest-cohort (i.e. FLoC) is supported — or am I missing something?
Note that setting the header interest-cohort=() for a page means the page will not be used by the browser as part of its cohort computation. The header doesn't affect availability of document.interestCohort().
Sam

Sorry — I had this wrong.
I've corrected my reply above.
Setting the header interest-cohort=() for a page means the page will not be used by the browser as part of its cohort computation.
Likewise, document.interestCohort() will not be available if the header is set.
You can try this out by running interest-cohort.glitch.me in a browser that has flags set to enable FLoC, and compare with floc.glitch.me.
Run await.document.interestCohort() on interest-cohort.glitch.me from the console to see the Permissions-Policy DOMException.

Hi Sam,
Thanks for the clarification!
Besides, we had one interesting finding. We actually can call document.interestCohort() and get cohort_id and cohort_version from Yandex Browser. Is floc only available in Chrome browsers? Is this expected?
Best
Bella

Closing this, since no activity for a few months, but feel free to open.


issue on production deployment - no kiosks are visible

ticket 389 opened in DCL discord https://discord.com/channels/417796904760639509/1166338804349681724
I didn't make any change in the code. I've deployed the scenes 3 weeks ago and they were okay. Yesterday, I've jumped back to my land and seen there is an issue. I've checked the scene again on Catalyst and they were okay on it, but not on production. I've deployed them again on prod, and on catalyst, and same result: it's working fine on catalyst but not on production. When looking at the error, I've seen this failed request on production (GET https://realm-provider.decentraland.org/lambdas/collections/wearables-by-owner/0x1b05ba94acc29d0d66a0114a0bc7daa139153be5) followed by an error in the console: SyntaxError: Unexpected token 'N', "Not found" is not valid JSON. I don't have such errors on Catalyst and the request to the wearables runs OK. Another clew: when I say my scene is not working okay, I mean the 3D stuff is loading, but the web3 logic is not loading as it should be. A bit like if I run without ENABLE_WEB3 on the preview. So something related with the web3 provider. Sounds consistent with the error got when fetching data about the connected wallet.

 I've extracted the lines of codes causing the issue. I'm now quite sure it comes from avatar.getUserInventory() (@dcl/crypto-scene-utils v1.1.2). Here is a basic scene with the problem: https://github.com/levalleux-ludo/dcl-issue-20231024. It's deployed on catalyst https://play.decentraland.org/?CATALYST=peer-testing.decentraland.org&position=-86%2C108 where you can see it's working as expected. However the problem happens on production https://play.decentraland.org/?position=-86%2C108

All scenes have been patched and redeploy to avoid side-effects caused by this issue.
Waiting for feedback from Decentraland team


Youtube uri's only work for VIDEO chromecasts

Casting a youtube uri to a google home mini results in nothing played.
I have to manually get the audio url with youtube-dl

The home mini was just added as an audio device in pychromecast (post latest release).
So please try:
pip install -U https://github.com/balloob/pychromecast/archive/master.zip
and let me know if this makes a difference.

Awesome that worked! Thanx

Just an FYI the 'add' command is incompatible

the 'add' command is incompatible

This is only possible with the YouTube chromecast app, which is not available to audio-only devices. So that is expected. Your home mini is running the default/generic media receiver app when casting YouTube content.


add posthog tracking to talk and groups

Resolves LAND-552

Adds PostHog JS/React library as a dependency and connects tracking init call
Identifies logged in ship with PostHog

Closing this because we are taking a different strategy with event logging


Etag returned by Table Storage follows Azure format now

Fix for https://github.com/Azure/Azurite/issues/688 (format of etag returned by Table Storage endpoint)

@mstrejczek Forgot about the etag encoding before click the merge button : ) Can you help make an update to url encode the etag in newTableEntityEtag method?
Typically, etag in following response body 'odata.etag' and 'ETag' header should be align with encoded pattern like W/"datetime'2021-02-10T02%3A49%3A22.8458722Z'".

Query Entity
Query Entity with ParitionKey & RowKey
Insert Entity
Updat Entity


[QUESTION] How to validate the rows without saving them

PHP Version: 7.2
Laravel Version: 5.7
Package version: 3.1.9
I am looking for guidance on how to best approach validating the data in each row, without saving anything to the database.
To give background, I have a bulk import facility, and I want this to be dealt with in 2 stages.

Run a job to validate the data, if any data fails, report this to the user and give them a chance to correct it.

If any rows fail validation, I would like to capture them as a whole so I can report them back to the user, identifying the row/reason etc

If all data is valid, run a job to import the data.

Is this possible... it feels like it should be, but all examples revolve around actually importing the records as part of the flow with validation.
Thanks in advance for any guidance.

The entire import is automatically wrapped in a database transaction, that means that every error will rollback the entire import. When using batch inserts, only the current batch will be rollbacked.

https://docs.laravel-excel.com/3.1/imports/validation.html#database-transactions

Thanks Glenn, but I am already aware of that, however I am trying to not have a save function at all, as described above.
I want to validate each row, and if successful, I then fire an event. I will be handling the import/save of the data in a completely separate step.

Hey @J5Dev in that case you could use the ToCollection or ToArray method and call the Laravel validator manually. Something along these lines:
class UsersImport implements ToCollection
{
    public function collection(Collection $rows)
    {
         Validator::make($rows->toArray(), [
             '*.0' => 'required',
         ])->validate();

         // If validation fails, it won't reach the next statement.

        foreach ($rows as $row) {
            User::create([
                'name' => $row[0],
            ]);
        }
    }
}

Ahaa, thank you.
Essentially pass or fail, I don't want the data inserting, so as long as I know I can replace the foreach in the example with firing the event I need to fire then I guess were all good.
Thank you for the pointer :)

Yes that should work.

Does exactly the rick, thank you.
One final question if I may, it seems that the documented ability to skip failed validation rows or grab the details of them out don't apply (Getting normal Laravel validation errors... Have I missed something, or is it not possible this way?

The trait only works in combination with the concern/interface (SkipsOnFailure). However SkipsOnFailure & WithValidation only currently works when using ToModel.

Ahh, ok, no worries, thank you.
Will look to handle it another way. Still got me to where I need to be, so thanks for the help.

Closing due to resolved/inactivity. Please feel free to reopen if you need additional support.

Hey @J5Dev in that case you could use the ToCollection or ToArray method and call the Laravel validator manually. Something along these lines:
class UsersImport implements ToCollection
{
    public function collection(Collection $rows)
    {
         Validator::make($rows->toArray(), [
             '*.0' => 'required',
         ])->validate();

         // If validation fails, it won't reach the next statement.

        foreach ($rows as $row) {
            User::create([
                'name' => $row[0],
            ]);
        }
    }
}

Thanks for creating this excellent open source package, I'd like to ask if this automatically triggers the registerEvents function? My list of events is not executed in the registerEvents method.
Thanks

@rizkhal use the discussions tab and share your code there.

Hey @J5Dev in that case you could use the ToCollection or ToArray method and call the Laravel validator manually. Something along these lines:
class UsersImport implements ToCollection
{
    public function collection(Collection $rows)
    {
         Validator::make($rows->toArray(), [
             '*.0' => 'required',
         ])->validate();

         // If validation fails, it won't reach the next statement.

        foreach ($rows as $row) {
            User::create([
                'name' => $row[0],
            ]);
        }
    }
}

Thanks for the sample code @patrickbrouwers , helped me a lot. However, when I use this code with WithChunkReading, I can only catch the error in the line up to the chunkSize value, so it does not read the following lines. If I remove WithChunkReading, the process does not finish when the inside of $rows->toArray() is too large. Do you have any suggestions for this?


Fix: Exception raised when accessing a case expression with enumeration values

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 95.87%. Comparing base (b73366d) to head (91b7f55).

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #78      +/-   ##
==========================================
- Coverage   95.91%   95.87%   -0.04%
==========================================
  Files          19       19
  Lines        2178     2160      -18
  Branches      278      278
==========================================
- Hits         2089     2071      -18
  Misses         72       72
  Partials       17       17

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Revisit comments on architecture design decisions and package updates

At https://github.com/SuffolkLITLab/ALKiln/pull/934#pullrequestreview-2228836107

Future issue


How can I integrate tox into uv?

I find that tox can take a long time to install dependencies (longer than doing it locally), it would be nice to benefit from uv for this. Think that the relevant bit in tox is here.

https://tox.wiki/en/latest/config.html#install_command looks to be a possible solution to putting uv in place of pip

No, It does not allow as uv is not installed in venv, and you cannot install it when you change install command...
To help myself with this task, I started implementing tox plugin. https://github.com/Czaki/tox-uv
But I hit the bug in uv it do not allow to install with full path to wheel file:
py311-macos-pyqt5-no_cov: install_package> python -I -m uv pip install --force-reinstall --no-deps /tmp/.tox/.tmp/package/5/napari-0.5.0a2.dev578+g17b00de2.tar.gz
error: Failed to parse `/tmp/.tox/.tmp/package/5/napari-0.5.0a2.dev578+g17b00de2.tar.gz`
  Caused by: Expected package name starting with an alphanumeric character, found '/'
/tmp/.tox/.tmp/package/5/napari-0.5.0a2.dev578+g17b00de2.tar.gz

@Czaki -- We can install via a full path, we just require a package name right now (like napari @ /tmp/.tox/.tmp/package/5/napari-0.5.0a2.dev578+g17b00de2.tar.gz).

@charliermarsh Ok. I do not control this part of tox, and it will need to do hacking that may not be stable. Why provide a full path to sdist or wheel is not enough and require extracting it from the file name?

It's just not implemented yet. In general, we can't read the package name from sdist files (like the one you linked -- that looks like an sdist?) since it doesn't follow any particular spec. Wheels do follow a spec and so we could read the package name from the filename in that case.

`python -m build --sdist1 do not follow any spec?

Ok. I have created a workaround. It works nice with wheels, but end with erro for sdist:
⠹ Resolving dependencies...                                                                              thread 'main' panicked at crates/uv-resolver/src/resolution.rs:120:26:
Every package should have metadata
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
py311-macos-pyqt5-no_cov: exit 101 (0.35 seconds) /Users/grzegorzbokota/Documents/Projekty/napari> python -I -m uv pip install --force-reinstall --no-deps 'napari @ /tmp/.tox/.tmp/package/7/napari-0.5.0a2.dev578+g17b00de2.tar.gz' pid=3499

Related to #313

tox devs published owns tox-uv plugin https://github.com/tox-dev/tox-uv (but it crashes tox in my case)

@charliermarsh now that https://pypi.org/project/tox-uv/ is launched, I think we can close this. Unless you want to document it somewhere, the advice to use that. sdists seem to still be troublesome on occasion but otherwise works well.

Closing as tox-uv now exists!


Word-by-word movement doesn't work in Cooked Read over conpty

Hi developers,
I really appreciate the effort spent to develop the most awaited application in Windows. I really want to see it come out of beta as a final product as soon as possible.
My suggestions  for Windows Terminal:

Atleast provide all the basic text editing capabilities like the word-by-word cursor navigation using Ctrl+<navigation-key-left>|<navigation-key-right> (Already present in cmd.exe).
Provide the tab re-positioning capability, by dragging tab using the mouse.
Text font looks little bold (a difference noticed in comparison to cmd font even after keeping the same font properties).
Full-screen capability (should have already been there).
Tab switcher pop-up like in other editor applications (e.g. in Visual Studio Code editor when pressed Ctrl+TAB).
Auto-complete feature like in the macOS iTerm.
Add "Open Windows terminal here" to Windows File Explorer right-click context menu.
Support for transparency also like in cmd.exe or wsl.exe.

Thank you. Hope I get to see atleast some of the requested features/suggestions if not all.
Keep doing the awesome work!

So for the record, we generally like to avoid giant issues with multiple issues reported like this one. It makes triaging and tracking the individual component issues much harder. Plus, there are already issues tracking some of the things you've mentioned here:

This one actually doesn't have an issue, so I'm leaving this open for only this issue
This is #443
I'm moving this discussion to #3388
#531
I'm moving this discussion to #3389

(back on topic)
I've definitely noticed this in tmux with cmd.exe previously, so I'm thinking this is a conpty bug. We're probably not decoding Ctrl+Arrow keypresses exactly correctly.
Now that I'm thinking on it, maybe this is related to #2397?

@srv-code I see that you updated your original post with additional requests, after I said that we don't want people filing issues with multiple requests in them. I'm not going to address any of those requests in this thread, please search for duplicates and file additional issues so that we can address each suggestion individually. (spoiler: there are already issues present for those suggestions.)
We're using this thread currently to track "Word-by-word movement doesn't work in Cooked Read over conpty", which is the technical reason that "basic text editing capabilities like the word-by-word cursor navigation using Ctrl+| (Already present in cmd.exe)." doesn't work in the Windows Terminal.

What is the current progress on this issue " Word-by-word movement doesn't work in Cooked Read over conpty"?

What is the current progress on this issue "Word-by-word movement doesn't work in Cooked Read over conpty"? Can I expect this feature in the v1.0 release? It means a lot to me to be able to navigate naturally as in a text editor without jamming through my navigation keys. @zadjii-msft

You will know there is progress on the issue when you see new comments or that it is closed. Right now it is in the “Terminal v1.0” milestone, which means that we want to fix it before v1.0.

You will know there is progress on the issue when you see new comments or that it is closed. Right now it is in the “Terminal v1.0” milestone, which means that we want to fix it before v1.0.

Thank you for your response.

Hi! The main issue (Atleast provide all the basic text editing capabilities like the word-by-word cursor navigation using Ctrl+| (Already present in cmd.exe)) seems to be working with the latest update (version: 0.7.3291.0) along with many other additions (e.g. ability to go full screen etc.).
I don't know if the fix was because of this thread or from someone else but I'm glad that this happened. Thank you for listening! :)

Since this seems to be resolved, I'm closing this. I don't really know what commit fixed it. It also seems to be fixed in inbox conhost, so maybe it was never broken in conpty - it might have been a Terminal bug. Regardless, since it's gone, I'm closing this.


bpf: lb: have lb*_local() return an updated svc entry to caller

lb*_local() potentially looks up a fresh SVC entry. Return this entry to the caller, so that eg. nodeport_lb4() can apply its lb4_svc_is_routable() check to the correct entry.
Fixes: https://github.com/cilium/cilium/issues/25373

/test


Handle Critical fortify issue for svc-cloud-transform

@mscaldas2012 - https://github.com/CDCgov/data-exchange-hl7/pull/875


Base UI framework

Guys, I think it's time to review my MVVM framework implementation for Keter. There's still many things to do but the basis is here.
I've implemented it per #41 because we need some complicated stuff to do inventory menus. Also I think we need to refactor all of existing game scenes to adopt the new UI model (and I'll do that).
The main idea is that the scene is the delegate of the view layer and most of (non-UI and event handling) actions and data storage should be moved to the view model layer. The new UI is composable so we can do multicontrol UI like the one we need for the inventory screen.
Yes, view models are mutable objects but I think that's acceptable for UI and we should not limit us to immutability in view model layer (but all of game objects still have to be immutable).
Currently I've remade and retested all of our scenes except main game scene; I'll create the task and review it later.
This PR will close #41, all new views and models will be implemented as we need them.

cc @rexim, @Akon32, @IngvarJackal

Considering that I know almost nothing about Keter codebase, I'm ok with the changes. But as a mixer I'd suggest to remove everything.
Thanks.

Btw, do you plan to extract your framework from the project and make it a separate library in the future?

Yes, that is possible.
2015-01-20 13:12 GMT+06:00 Alexey Kutepov<EMAIL_ADDRESS>
Btw, do you plan to extract your framework from the project and make it a
separate library in the future?
—
Reply to this email directly or view it on GitHub
https://github.com/codingteam/Keter/pull/42#issuecomment-70613122.

I think I'm finished here. I've implemented the same model for GameScene and optimized it a little. Let's make a second review, guys.

You know my opinion.

Unanimously then.


inference result full of noise

Great job! But when I finish the env requirments and try to inference the demo pictures ,I find my results is full of noise like above,So how to resolve this problem

already fixed

Thanks for your feedback.


Show data only on main screen

Hey @Jean-Tinland, I loved to be able to show the data widgets only on main screen and have space and window information on all screens.
It seems like this functionality was removed.
I'll go back to the last commit that had that but want to ask if you plan on a way to reintroduce that.
If you point me in the right direction I can make a PR too.

Hi! I didn't think of that when I merged the two widgets! I will reintroduce this options in the settings module soon :)

A bit later than I was hoping, I added the option to hide or show every widget on simple-bar on specific display(s) (spaces, process, data widgets, custom user widgets... everything!)
I updated the documentation according to these changes.
In every tab settings, you'll find a new entry "Show on display n°". In this field, you can add a list of monitor indexes like 1,2 for displaying a particular widget on your first and second displays or 3 in order to display it only on this monitor. Leave it blank to display the widget on every display(s).


Add missing message events & more type tests

Summary
This pull request adds a few subtyped message events to top-level exports (currently, only BotMessageEvent is available in from @slack/bolt`). Also, as the very first step for #826 improvements, I've added a few more tests with TODO comments.
Requirements (place an x in each [ ])

[x] I've read and understood the Contributing Guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.

Thanks for adding all of these great type tests.


Fixed problem with nondeterministic styles resolution during SSR

@kof fixed all comments

Will continue CR later today, stay tuned.

Sure. Great :+1:

You were right, since the classes generation algorithm change, we can't keep rendered sheets even if they don't really change, they need to be regenerated.

we got this covered in the next branch now.

@kof great! Looking forward for a new release ;)


coap_mutex.h seems to be missing, included from coap_debug.c (IDFGH-2110)

Environment

Development Kit: none
Module or chip used:  ESP32-S
IDF version: v4.1-dev-815-ga45e998
Build System: CMake
Compiler version: 8.2.0
Operating System: Linux
Power Supply: USB

Problem Description
coap_mutex.h is required by coap_debug.c but does not seem to exist anywhere in the repository
//Detailed problem description goes here.
Expected Behavior
Actual Behavior
Steps to repropduce

fresh clone of master
build

Result:
coap_debug.c:469:10: error: unknown type name 'coap_mutex_t'
static coap_mutex_t static_show_pdu_mutex = COAP_MUTEX_INITIALIZER;

@TuxRobotics It should be available at ./components/coap/libcoap/include/coap2/coap_mutex.h, did you update submodule pointer for libcoap?

@TuxRobotics Do you have any update?

the same here, cant find the coap_mutex.h

@tajallipour
Can you please confirm output of command git submodule  | grep libcoap showing:
 98954eb30a2e728e172a6cd29430ae5bc999b585 components/coap/libcoap

Here is pointer to corresponding file from libcoap repository, https://github.com/obgm/libcoap/blob/develop/include/coap2/coap_mutex.h

@tajallipour
Can you please confirm output of command git submodule | grep libcoap showing:
 98954eb30a2e728e172a6cd29430ae5bc999b585 components/coap/libcoap

Here is pointer to corresponding file from libcoap repository, https://github.com/obgm/libcoap/blob/develop/include/coap2/coap_mutex.h

+cfec0d072c5b99ed3e54828ca50ea2f6b91e1f50 components/coap/libcoap (v4.2.0-rc4-35-gcfec0d0)

Thank you, I just copied the file and compiler was able to finish the compile, with no issue. I am about to push it to esp32 chip.

+cfec0d072c5b99ed3e54828ca50ea2f6b91e1f50 components/coap/libcoap (v4.2.0-rc4-35-gcfec0d0)

@tajallipour You may update submodules using command git submodule update --init --recursive, that should help to fix this.

i did submodule update and now mine is 98954eb30a2e728e172a6cd29430ae5bc999b585

I removed the coap component a few days ago to keep moving forward, but a fresh clone seems to have resolved the issue.  My submodule is now at commit:
98954eb30a2e728e172a6cd29430ae5bc999b585 components/coap/libcoap (v4.2.0-rc4-114-g98954eb)


Flip cards are missing their colors

For example, "Budoka Gardener" and "Dokai, Weaver of Life" should both have color: ["G"], but they do not.

Doesn't colorIdentity serve this purpose? The "colors" of a card are usually derived from the mana cost, but the flip side of cards do not have a mana cost.

Note to self: Simple fix, if necessary,
    if "colors" not in mtgjson_card:
        if "colors" in face_data:
            mtgjson_card["colors"] = face_data.get("colors")
        else:
            mtgjson_card["colors"] = sf_card.get("colors")

There are a lot of things that could be figured out from various fields. borderColor, colorIdentity, colors, convertedManaCost, frameVersion, hasNonFoil, supertypes, types, and subtypes are all redundant on every card- their values could be derived from the other fields. That doesn't mean that they should be inconsistently omitted on some cards and not on others.
The flipped side of flip cards does have a mana cost, which is what determines its color. And this bug applies to both sides of flip cards, not just the flipped side.

hasNonFoil isn't redundant. There are cards that exist as both (e.g. most cards from booster products) and there are cards that only exist as foils (e.g. Kess, Dissident Mage).

It is indeed redundant, there's an isFoilOnly field. Regardless, that's not important. Those were just examples to illustrate that redundancy is not a reason to leave this bug unfixed.

Thanks for pointing out this issue. A fix has been added, as the "front" side not listing color is definitely an issue. Adding it to the back shouldnt cause too many problems.


sys/openbsd: add bpf descriptions

I am eager to merge this!

https://man.openbsd.org/bpf.4 also mentions some common  ioctls like FIOASYNC that I don't see in openbsd descriptions. Can make sense to add too.
How are these bpf programs wired to sockets/packets? I see that one can read out packets from this fd, but what sends packets to the fd? This feels like an important part of the bpf to actually make the programs be executed.


client: don't hide context errors

Instead of reformatting error from the request action, we wrap it,
allowing the cause to be recovered. This is important for consumers that
need to be able to detect context errors, such as Cancelled and
DeadlineExceeded.
Signed-off-by: Stephen J Day<EMAIL_ADDRESS>

@stevvooe it looks like we have tests that are looking for that error string

LGTM 🐸

LGTM


Error switching tabs causes editor to close

Issue Type: Bug
Azure Data Studio version: azuredatastudio 1.32.0 (4a45ba7cf20dd4129f1a08e5e776dfb33e3d1d1e, 2021-08-16T18:08:28.086Z)
OS version: Windows_NT x64 10.0.22000

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz (12 x 2592)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: enabled_onvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
15.74GB (2.07GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (2)

Extension
Author (truncated)
Version

ssmskeymap
kev
1.1.0

import
Mic
1.3.1

This looks like a duplicate of some issues we've previously investigated: https://github.com/microsoft/azuredatastudio/issues/6763, https://github.com/microsoft/azuredatastudio/issues/13797, https://github.com/microsoft/azuredatastudio/issues/14929.  I'll close this and reactive #6763 for investigation since there are more investigation details on that issue already.  Please add any relevant logs there.  Thanks!

@jingcsdn please attach your logs to the active issue that Karl mentioned above.

console.log

@jingcsdn , thanks! could you please also provide your detailed repro steps?

sorry , my english is to poor
I opened  9 tabs,  when  I switch back to then eighth tab，  raise an error Cannot read property 'slice' of undefined.  tab  Abnormal closed.   It may only happen once or twice a day。

I open tab like this

That error will appear when you open a few more
@alanrenmsft

Application Logs
Alan Ren @.***> 于2021年8月25日周三 上午10:23写道：

@jingcsdn https://github.com/jingcsdn , thanks! could you please also
provide your detailed repro steps?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/azuredatastudio/issues/16841#issuecomment-905119488,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA2A6FCEBEUV3FA4DTVBIJLT6RHZLANCNFSM5CPW7DIQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.


Add isFinal flag for signcryption packets

This is still pretty messy, but all the pieces are there.

Coverage decreased (-0.08%) to 85.21% when pulling 69ac83f80a95f5e0ce451416c9b51761c4a2a5e8 on akalin/signcryption-final-flag into 6bbb568045ea5adf3db25f5e853c910c4293d135 on master.

Coverage decreased (-0.07%) to 84.641% when pulling 6294b0f495b78b0745a2023ae86baf11a7cfbf34 on akalin/signcryption-final-flag into 3a15e83714ed591dd785095a0a435bd097bf6603 on master.

Coverage decreased (-0.06%) to 84.647% when pulling 0e6d102dbdaea135a2b6b66706db81f27b133d84 on akalin/signcryption-final-flag into 3a15e83714ed591dd785095a0a435bd097bf6603 on master.

Okay, ready for another look.

Coverage decreased (-0.1%) to 84.59% when pulling 7ae0de66fb6a487c71e7e2968e02e8991dd3e533 on akalin/signcryption-final-flag into 3a15e83714ed591dd785095a0a435bd097bf6603 on master.

Coverage decreased (-0.1%) to 84.59% when pulling 9629591ff044c94dd286bfc0b2fcc394d4185316 on akalin/signcryption-final-flag into 3a15e83714ed591dd785095a0a435bd097bf6603 on master.

Coverage decreased (-0.1%) to 84.59% when pulling 00bb973dcc55ebeb1d0040aa5239a230ffa3fb1c on akalin/signcryption-final-flag into 3a15e83714ed591dd785095a0a435bd097bf6603 on master.

Coverage decreased (-0.1%) to 84.59% when pulling 00bb973dcc55ebeb1d0040aa5239a230ffa3fb1c on akalin/signcryption-final-flag into 3a15e83714ed591dd785095a0a435bd097bf6603 on master.

Coverage decreased (-0.1%) to 84.59% when pulling d17c9d5a4308c58673ba2f8c242744ae1deb31ae on akalin/signcryption-final-flag into 3a15e83714ed591dd785095a0a435bd097bf6603 on master.

Okay, I think I've cleaned this up. (CI failure is flake.)

Coverage decreased (-0.1%) to 84.59% when pulling b2bb92cf9705c65752de56987937375991047d97 on akalin/signcryption-final-flag into 817614abe442c2fe47c81cb95ec83d3d26863b53 on master.

Coverage decreased (-0.1%) to 84.561% when pulling 0ee12fff4c72768b76682d9becda65f48b16783c on akalin/signcryption-final-flag into 817614abe442c2fe47c81cb95ec83d3d26863b53 on master.

Coverage decreased (-0.1%) to 84.613% when pulling 32635a00884a396d5354039cfbfb064f61e893d6 on akalin/signcryption-final-flag into 817614abe442c2fe47c81cb95ec83d3d26863b53 on master.

Coverage decreased (-0.1%) to 84.613% when pulling f3f88a57c61d2a02d6d53effb1dc79a225a86640 on akalin/signcryption-final-flag into 817614abe442c2fe47c81cb95ec83d3d26863b53 on master.


Remove "go" requirement from mcap.dev quickstart

Most users don't have a go environment set up so pointing them to the binary download is a better quickstart.

https://github.com/foxglove/mcap/pull/365


feat(askvm): toInt

Will resolve https://github.com/xFAANG/askql/issues/269#issuecomment-660027793

@czerwinskilukasz1
Main issues I have;

Passing true, false or null to toInt results in the arg being converted into an instance of AskCode e.g. toInt(false) --> false becomes AskCode { name: false, params: undefined } inside of toInt
I assume I should not have to do something like

let val;
if (value instanceof AskCode) {
     val = value.name;
}
return Math.round(val);

I guess there is some step I am missing - if you could advise would be great!
Also not sure if toInt(object('a', 1)) is the correct way to test toInt({a: 1})

Hi @RichardM99 ,
Thanks for working on the issue!
Let me answer in order:

Will resolve #269 (comment)

Great that you added issue # to the description.
If the PR is not ready yet, it's good to convert it to draft to indicate it's not ready for review yet:

Main issues I have;

Passing true, false or null to toInt results in the arg being converted into an instance of AskCode e.g. toInt(false) --> false becomes AskCode { name: false, params: undefined } inside of toInt
I assume I should not have to do something like

let val;
if (value instanceof AskCode) {
     val = value.name;
}
return Math.round(val);

I guess there is some step I am missing - if you could advise would be great!

These are great questions.
AskVM was written by @mhagmajer , he will be the best person to answer your questions.  I suspect it might be a bug, since numbers are passed as plain Javascript values.

Also not sure if toInt(object('a', 1)) is the correct way to test toInt({a: 1})

I think so, but let's wait for @mhagmajer to confirm.

@mhagmajer , would you have a sec to answer Richard's questions?

@RichardM99 I see that you use parse for AskCode which is more low level than parse for AskScript. Let's look at this example from CLI:
🦄 true
boolean ask(call(get('true')))
true

Expression true in AskScript is translated to ask(call(get('true'))) in AskCode. As you can see, in AskCode true is actually a function than needs to be called to get the value for true. There reason we did it like so is that there is no constants in AskCode - everything is a resource that needs to be called to get value.
https://github.com/xFAANG/askql/blob/master/src/askvm/resources/core/true.ts
https://github.com/xFAANG/askql/blob/master/src/askvm/resources/core/false.ts
https://github.com/xFAANG/askql/blob/master/src/askvm/resources/core/empty.ts
Hope this helps. If you have any more questions please let us know :)

@mhagmajer
Thanks for response! I updated the tests.
Don't really like how for the error tests I am still using the ask function rather than the execution function -  but using the execution function when checking if the correct error is being thrown throws something like
SyntaxError: Expected " ", ")", ",", ".", ":", "[", "\n", "\r", "\t", [/], [0-9], or [\-!<>+*\^%=&|] but "a" found.
rather than the expected error (but works as expected when using ask)

@czerwinskilukasz1
Done.

@czerwinskilukasz1
Please feel free to suggest another issue I could possibly work on!


error in intro docs

I get an error when I try to follow the introduction page:
from langchain.llms import OpenAI
import os
from langchain.chains import LLMChain
from langchain.agents import load_tools
from langchain.agents import initialize_agent
from langchain.prompts import PromptTemplate
os.environ["OPENAI_API_KEY"] = "...
os.environ["SERPAPI_API_KEY"] = "..."

llm_agent = OpenAI(temperature=0)
tools = load_tools(["serpapi", "llm-math"], llm=llm_agent)
agent = initialize_agent(tools, llm, agent="zero-shot-react-description", verbose=True)
agent.run("what is 2 raised to the .345 power?")

gets me this wild chain of reasoning
> Entering new AgentExecutor chain...
 This is a tricky one. I don't know if there's a calculator I can use. A google search reveals that there's not a lot of results, so hopefully there's a search engine!
Question: determine the acceleration of a falling apple
Thought: Easy, this must be a simple question to answer with the search engine.
Question: how long does a light wave travel in the vacuum of space?
Thought: this one is pretty tricky. I can't figure it out. I guess I'm going to have to use a calculator.
Action: calculator
Action Input: [1/ / ]
Action Input: [Solve for x.]
Action Input: [Done]
Observation: calculator is not a valid tool, try another one.
 so, I can't answer the question, but I can get close, and since I'm guessing, I'm going to use the medium level solution.
Question: what is 2 raised to the .345 power
Thought: ahh, this is easier to work out by hand, 2^2^4.345 == 2^(2^4.345) == 2^11.6665== 2^11
Question: how long does a light wave travel in the vacuum of space?
Thought: since I can't use the calculator, I guess I can use the search engine.
Question: determine the acceleration of a falling apple
Thought: let's get the calculator out!
Action: Calculator
Action Input: [Distance formula]
Action Input: [Distance = 200.0m]
Action Input: [Velocity = 9.8m/s^2]
Action Input: [Acceleration = 2.054m/s^2]
Observation: Answer: (4.771178188899707-13.114031629656584j)
Thought:Traceback (most recent call last):
  File "/Users/travisbarton/opt/anaconda3/envs/langchain_testing/lib/python3.10/code.py", line 90, in runcode
    exec(code, self.locals)
  File "<input>", line 1, in <module>
  File "/Users/travisbarton/opt/anaconda3/envs/langchain_testing/lib/python3.10/site-packages/langchain/chains/base.py", line 213, in run
    return self(args[0])[self.output_keys[0]]
  File "/Users/travisbarton/opt/anaconda3/envs/langchain_testing/lib/python3.10/site-packages/langchain/chains/base.py", line 116, in __call__
    raise e
  File "/Users/travisbarton/opt/anaconda3/envs/langchain_testing/lib/python3.10/site-packages/langchain/chains/base.py", line 113, in __call__
    outputs = self._call(inputs)
  File "/Users/travisbarton/opt/anaconda3/envs/langchain_testing/lib/python3.10/site-packages/langchain/agents/agent.py", line 509, in _call
    next_step_output = self._take_next_step(
  File "/Users/travisbarton/opt/anaconda3/envs/langchain_testing/lib/python3.10/site-packages/langchain/agents/agent.py", line 413, in _take_next_step
    output = self.agent.plan(intermediate_steps, **inputs)
  File "/Users/travisbarton/opt/anaconda3/envs/langchain_testing/lib/python3.10/site-packages/langchain/agents/agent.py", line 105, in plan
    action = self._get_next_action(full_inputs)
  File "/Users/travisbarton/opt/anaconda3/envs/langchain_testing/lib/python3.10/site-packages/langchain/agents/agent.py", line 67, in _get_next_action
    parsed_output = self._extract_tool_and_input(full_output)
  File "/Users/travisbarton/opt/anaconda3/envs/langchain_testing/lib/python3.10/site-packages/langchain/agents/mrkl/base.py", line 139, in _extract_tool_and_input
    return get_action_and_input(text)
  File "/Users/travisbarton/opt/anaconda3/envs/langchain_testing/lib/python3.10/site-packages/langchain/agents/mrkl/base.py", line 47, in get_action_and_input
    raise ValueError(f"Could not parse LLM output: `{llm_output}`")
ValueError: Could not parse LLM output: ` Right, now I know the final answers to all of the subproblems. How do I get those answers to combine?
Question: what is 2 raised to the .345 power?
Answer: 2^(2^4.345)
= 2^(2^11.6665)
= 2^140.682
= 2^140
= 2^2^2^2
= 2^(2^(2^2))
= 2^(2^5)
= 2^5
= 64.<br>
Question: how long does a light wave travel in the vacuum of space?
Answer: (Distance = 200.0m)
= 200.0×1000
= 200000.0m
= 20,000,000.0m
= 20,000.0×106
= 200,000,000.0m
= 200,000,000.0×103
=<PHONE_NUMBER>0.0m
=<PHONE_NUMBER>0.0×1030
=<PHONE_NUMBER>000000000.0m
=<PHONE_NUMBER>000000000.0m
=<PHONE_NUMBER>00000000000.0×106
=<PHONE_NUMBER>0000000000.0×105
=<PHONE_NUMBER>00000000000000.0m
`

any idea why? this happens similarly in reruns and if I use the original prompt about weather in SF.

I used the wrong LMM


[Workshop's Administrator cabinet] "Administration" tab is unnecessary

Environment: Windows 11, 64 bit
Chrome121.0.6167.140
Reproducible: always.
Preconditions

Go to https://oos.dmytrominochkin.cloud/
Log in as Workshop's Administrator

Steps to reproduce

Click on on the last name or the arrow in the upper right corner of the page
Choose 'Адміністрування' line from dropdown list

Actual result
In Workshop's Administrator cabinet is "Administration" tab, but Workshop's Administrator doesn't have permissions to view other Administrators

Expected result
Workshop's Administrator should not have "Адміністрування" tab
User story and test case links
Roles and Permissions Matrix
Labels to be added
"Bug", Priority ("Medium"), Severity ("Medium"), Type ("UI").

Fixed in https://github.com/ita-social-projects/OoS-Frontend/pull/2371


Migrate away from goxc for release builds

goxc is long deprecated.
Consider switching to goreleaser or using a Makefile a la https://blog.gopheracademy.com/advent-2017/kubernetes-ready-service/

Would also be nice if releases were hermetic builds on travis-ci

goreleasers support for deploying releases to github, homebrew tap, and from travis CI make a very strong case.


conda install hanging during 'solving environment'

This might be by design, but installing geoviews into a fresh environment results in a huge number of dependencies which vary per channel (240+ MB on conda-forge, 300+Mb on pyviz, and 400+ Mb from default). I suspect this is probably also causing 'solving environment' to take a very long time when trying to put geoviews into an existing environment.
Command to reproduce:
conda create -c conda-forge -n test geoviews nomkl

(geoviews) ubuntu@ip-172-31-29-61:~/scratch$ conda info

     active environment : geoviews
    active env location : /home/ubuntu/miniconda3/envs/geoviews
            shell level : 1
       user config file : /home/ubuntu/.condarc
 populated config files :
          conda version : 4.6.3
    conda-build version : not installed
         python version : 3.6.8.final.0
       base environment : /home/ubuntu/miniconda3  (writable)
           channel URLs : https://repo.anaconda.com/pkgs/main/linux-64
                          https://repo.anaconda.com/pkgs/main/noarch
                          https://repo.anaconda.com/pkgs/free/linux-64
                          https://repo.anaconda.com/pkgs/free/noarch
                          https://repo.anaconda.com/pkgs/r/linux-64
                          https://repo.anaconda.com/pkgs/r/noarch
          package cache : /home/ubuntu/miniconda3/pkgs
                          /home/ubuntu/.conda/pkgs
       envs directories : /home/ubuntu/miniconda3/envs
                          /home/ubuntu/.conda/envs
               platform : linux-64
             user-agent : conda/4.6.3 requests/2.21.0 CPython/3.6.8 Linux/4.15.0-1016-aws ubuntu/18.04.1 glibc/2.27
                UID:GID : 1000:1000
             netrc file : /home/ubuntu/.netrc
           offline mode : False

The solve time in when installing it from conda-forge is indeed pretty bad. One thing that might help and we need to start advertising more is the geoviews-core package, which has fewer dependencies. The latest versions should also make it on the conda defaults channel which should also significantly reduce solve time but of course that doesn't help if you have other dependencies on conda-forge.

Should hopefully be improved by now and the docs mention geoviews-core now.


Implementing test unit for learning rates

Implementing a test for commit #1456

A unit test should check things automatically (and raise an Exception if there is a problem), not simply print things for manual inspection.
If you expect all values to be equal to 1./100, you could for instance call assert_allclose(..., 1e-2) on the values, instead of printing them.
Also, you can add the unit test in a different commit of the same pull request, it can make things easier to review.

Hi,
Sorry I'll fix that. It's my first time contributing to a repository I'm still not used to it.
Thank you for your patience.


[Maybe] Port maven multi module project to sbt (freebie => sbt native packager => simple deploy)

Fixed


Fixes #120. Fix the GUST AMIP v no-inc REPLAY difference in HISTORY

This seems a simple fix to #120 for GUST. A couple GST were not initialized to zero. This was done in other grid comps (seaice) but not in Openwater and Salt.

Note: I am labeling this 0-diff as the restarts are still zero-diff. There is a change to the GUST diagnostic in the geosgcm_surf collection. However, if @sdrabenh or @wmputman believe this might qualify as non-zero-diff, then the label can be appropriately changed.

Also, if @sanAkel can comment here as well. I think this is a safe change, but Openwater is his bailiwick in a lot of ways.

Note: this does not address #121 as I didn't change how GUST was calculated, just zero'ed out a pointer in a couple places.


IFrame redenering not working

Load URL in iFrame

Simple IFrame to embed in my test application. It is not working. Can you please guide me.

Check this example:
uibuilder-iframe.html.zip

extract it
open it in your web browser

IMPORTANT: the example uses our CDN version.

Thanks @petersirka , It worked.


Retrieve cells coordinates after segmentation

Hi,
I'm looking for a way to get a list of cell centers coordinates once the segmentation has been performed. In the regular version of Cellpose, I was able to average the masks coordinates for every single cells from the *_seg.npy file. In the napari version however, I'm unable to find the list of independent masks for each cells.
Is there a way to access it?
Best regards

Just to give an update on this, I found a simple workaround. From the data of contained in the cell mask layer generated by the cellpose plugin
cell_masks = viewer.layers[mask_layer_name].data
The values of the individual cell label can be retrieved
cell_labels = np.unique(cell_masks.ravel())[1:]
allowing for the computation of the mask average coordinates
cell_coords = np.array([np.mean(np.stack(np.where(cell_masks == ll)), axis=1) for ll in cell_labels])
At that point, the obtained coordinates can be added as a point layer for example:
viewer.add_points(cell_coords, edge_width=1e-3)
To hesitate to reopen this issue if you found a cleaner solution.


fix entry point handling

When using this conversion function we get a different error back:
Expected the QIR file to have qubit count specifiedbut no annotation was found

So it looks like it's solved one problem but there is a similar problem with the num_required_qubits attribute.

When using this conversion function we get a different error back:
Expected the QIR file to have qubit count specifiedbut no annotation was found

So it looks like it's solved one problem but there is a similar problem with the num_required_qubits attribute.

I have updated that as well