[FEATURE REQUEST] Link to github or official page from within extension

Is your feature request related to a problem? Please describe.
I'd like to be able to navigate back to the github (or if there is an official) page from the Chrome extension. Currently we're using the icon as a global on/off switch, but nothing else links out.
Describe the solution you'd like
Adding an about link in:

popup
settings page

Describe alternatives you've considered
Maybe making the icon the global toggle is weird?
It's neat, but I think normal behavior is clicking the icon brings you to the HOME page (whatever that might be)
Additional context
Add any other context or screenshots about the feature request here.

The "bug" icon brings users to the issues page and the "Support" page in the settings has many links that could get users here. It's not a goal of the Toolkit to get users to our Github page unless they have a bug so I don't see the need to add more links to get here?


ETG reset mod the gungeon

installed mod the gungeon and when i star it after a few seconds i get loaded in the next chamber but its like hell

As I recall, this happens whenever you install it to a pirated copy of the game. No?


Require-dev for PHPUnit

If not set, it breaks the root composer.json generation in travis-support, since it falls back to PHPUnit 3.7.
See https://github.com/silverstripe-labs/silverstripe-travis-support/blob/master/src/ComposerGenerator.php
See https://github.com/silverstripe/silverstripe-cms/commit/d9e292ba281342cc7943ae25538daa1d173fa0c6
for same fix on cms module.

At least reports doesn't have behat to fail. :)


Watch changes to the alertmanager secret in public namespace

In order for the following entry in our documentation to be fully true "After a few moments, Managed Service for Prometheus picks up the new config Secret and enables the managed Alertmanager with your settings.", we would need watch that config. Currently we only propagate changed from this file when OperatorConfig is reconcile. Improve this.
AC

Updates to desired secret in the public namespace that is referenced by the OperatorConfig.managedAlertmanager.configSecret are propagated to the Alertmanager.

Workarounds
Users can change OperatorConfig in any way (e.g. add or remove noop field, maybe comment?)

Actually, this should be fine, given this code and this. Got confused by a customer observation.


regression: blank diff output

Using the development trunk I'm having rendering issues with commit diffs.  git bisect indicates the bug was introduced in aa35498.
This is what diff output looks like at 14becd0 (one commit before):

And this is what the output looks like at aa35498 (as well as current HEAD 1341ef9):

I'm getting this issue on both Linux and Windows, as well as Internet Explorer 9 and Firefox v31-36

Fixed in #672


cant connect to device with the example app

Hi guys,
I've received my first Movesense device not long ago and wanted to test it in Flutter.
I'm using the example app provided here but so far I could not establish a connection.
I'll attach some log files but I think this is the most important bit:
D/BleManager(31592): connecting to 0C:8C:DC:38:93:5A
D/BleManager(31592): isConnected: RxBleConnectionState{DISCONNECTED}
D/BleManager(31592): Connection try: 0
D/BleManager(31592): Disconnect happened
D/BleManager(31592): removeBleMac: bleMac: 0C:8C:DC:38:93:5A
D/BleManager(31592): removeBleMac: wbAddress: null
D/BleManager(31592): getOrCreateWbAddress: bleMac: 0C:8C:DC:38:93:5A
D/BleManager(31592): getOrCreateWbAddress: wbAddress: null
V/BleManager(31592): getOrCreateWbAddress: handle: 268435457
D/BleManager(31592): getOrCreateWbAddress: wbAddress2: 10000001
D/BleManager(31592): getConnectedBleDevicesCb()
D/BleManager(31592): Found connected devices: []

And then it keeps repeating:
D/BleManager(31592): Trying to reconnect to device: 0C:8C:DC:38:93:5A
D/BleManager(31592): connecting to 0C:8C:DC:38:93:5A
D/BleManager(31592): isConnected: RxBleConnectionState{DISCONNECTED}
D/BleManager(31592): Connection try: 0
D/BleManager(31592): Disconnect happened
D/BleManager(31592): removeBleMac: bleMac: 0C:8C:DC:38:93:5A
D/BleManager(31592): removeBleMac: wbAddress: 10000001
D/BleManager(31592): getOrCreateWbAddress: bleMac: 0C:8C:DC:38:93:5A
D/BleManager(31592): getOrCreateWbAddress: wbAddress: null
V/BleManager(31592): getOrCreateWbAddress: handle: 268435458
D/BleManager(31592): getOrCreateWbAddress: wbAddress2: 10000002
D/BleManager(31592): getConnectedBleDevicesCb()
D/BleManager(31592): Found connected devices: []

Basically, it seems like whenever MDS is trying to connect to the device, it receives a Disconnect event.
The interesting thing is that I tried the Showcase app from the Android mobile library and that connects (to the same device, on the same android phone) without any issues:
D/BleManager: connecting to 0C:8C:DC:38:93:5A
    isConnected: RxBleConnectionState{DISCONNECTED}
D/BleManager: Connection try: 0
I/BluetoothAdapter: stopLeScan()
    STATE_ON
I/BluetoothAdapter: STATE_ON
D/BluetoothLeScanner: Stop Scan with callback
I/Dialog: mIsSamsungBasicInteraction = false, isMetaDataInActivity = false
I/DecorView: [INFO] isPopOver=false, config=false
    updateCaptionType >> DecorView@17a6b41[], isFloating=true, isApplication=true, hasWindowDecorCaption=false, hasWindowControllerCallback=false
D/DecorView: setCaptionType = 0, this = DecorView@17a6b41[]
I/BluetoothAdapter: STATE_ON
D/BluetoothGatt: connect() - device: 0C:8C:DC:38:93:5A, auto: false
I/BluetoothAdapter: isSecureModeEnabled
D/BluetoothGatt: registerApp()
D/BluetoothGatt: registerApp() - UUID=89ba9c89-c109-46b0-ab27-df054290d445
D/BluetoothGatt: onClientRegistered() - status=0 clientIf=12
D/ScrollView: initGoToTop
D/BluetoothGatt: onClientConnectionState() - status=0 clientIf=12 device=0C:8C:DC:38:93:5A
D/BleManager: BLE connection to 0C:8C:DC:38:93:5A done!

Did anyone else face this before or can maybe reproduce?
Any tips on how to resolve the issue would be also much appreciated of course.
log_android.txt
log_flutter.txt

Have you found out what the issue is?
I'm very interested in this technology and would like to try it out in flutter.

Not yet unfortunately.
Debugged the flutter part a bit but didn't find anything odd there. (But could only get disconnect messages from the device as mentioned no matter what.)
Only tested on android so far so I'll try iOS next + debug the native code a bit...
Will update the thread with any new infos.

Warning: Mapping new ns http://schemas.android.com/repository/android/common/02 to old ns http://schemas.android.com/repository/android/common/01
Warning: Mapping new ns http://schemas.android.com/repository/android/generic/02 to old ns http://schemas.android.com/repository/android/generic/01
Warning: Mapping new ns http://schemas.android.com/sdk/android/repo/addon2/02 to old ns http://schemas.android.com/sdk/android/repo/addon2/01
Warning: Mapping new ns http://schemas.android.com/sdk/android/repo/repository2/02 to old ns http://schemas.android.com/sdk/android/repo/repository2/01
Warning: Mapping new ns http://schemas.android.com/sdk/android/repo/sys-img2/02 to old ns http://schemas.android.com/sdk/android/repo/sys-img2/01
FAILURE: Build failed with an exception.

What went wrong:
Could not determine the dependencies of task ':mdsflutter:compileDebugAidl'.

Could not resolve all task dependencies for configuration ':mdsflutter:debugCompileClasspath'.
Could not find any matches for :mdslib:+ as no versions of :mdslib are available.
Required by:
project :mdsflutter

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

Get more help at https://help.gradle.org

BUILD FAILED in 3s
Exception: Gradle task assembleDebug failed with exit code 1


Add raid bank swapper

Added another option in menu entry swapper to swap the right click of raids storage units:

Switches shared to private for ironman as suggested by #6015
Default is off

Merge conflicts. The positions  from the menu entry swapper config were removed with #6071.


fix setup for sidekiq 6.1.0

https://github.com/mperham/sidekiq/compare/v6.0.7...v6.1.0 shows internal API changes. The setup needs to initialize the fetcher instead of passing the fetch class. 🤷♂
https://github.com/mperham/sidekiq/issues/4614
I tried to fix that the easy way.

fixed the rubocop issue but the CI has to run again 🕵♂

Waiting for this a lot

is there any way to retrigger the CI? a empty commit?

@hmaack awesome!
Is there anything we can do to push this forward?
@jobwat it seems that it only runs once when the pull request is created. It would require a new PR.
https://github.com/sensortower/sidekiq-throttled/blob/master/.github/workflows/ci.yml#L7

The throttler gem supports versions below sidekiq 6.1, this PR is still not compatible for versions < 6.1. So in this case I would recommend either to check the version of sidekiq in the #setup! method and decide how to pass / setup the Fetcher. I would like to have an opinion from a contributer / maintainer about this. What do you think @ixti ? ✌

Hey guys, I'm waiting for this merge, there is any thing I can help you to merge as soon as possible?
currently using as
git 'https://github.com/9feet/sidekiq-throttled', branch: 'fixup/fetcher-init-sidekiq-6.1.0' do
  gem "sidekiq-throttled"
end

Hi @ixti , I switched all conditional checks to Gem::Version.new(Sidekiq::VERSION) < Gem::Version.new("6.1.0") because I got confused after reviewing the different styles, I hope thats OK for you. Check bd2b2e0 and 0307564 they were also wrong at some points. So I tried to use uniform approach.

Squash-merged in 700507c

Released as v0.13.0


[BUG] Dashboard's settings page is empty

Describe the bug
Unable to access Options in Setitngs page, it is empty.

https://user-images.githubusercontent.com/5169694/164731548-03b6391b-ac69-4654-8df4-7e8239835a71.mp4
To Reproduce
Steps to reproduce the behavior:

Install Harvester
Navigate to Advanced Settings

Expected behavior
Options should displayed
Environment:

Harvester ISO version: master-c6640cf1-head
Underlying Infrastructure (e.g. Baremetal with Dell PowerEdge R630): qemu/KVM single node

Verified this bug has been fixed.
Test Information

Environment: qemu/KVM single node
Harvester Version: master-c6640cf1-head
ui-source Option: Auto

Verify Steps:

Follow To Reproduce in https://github.com/harvester/harvester/issues/2177#issue-1212417089


Stateful Binary Patricia Contracts

Description
Partially addresses YAS-380, adding a stateless State Manager, by providing a full tree.  Handles insertion, retrieval, inclusion, and non-inclusion proofs.  Next step is to change the storage format to work statelessly by storing sibling hashes instead of full sibling pairs.
Questions

Still need to measure performance of a big inclusion proof, will do that before merging
Maybe this should go in ovm package?
Hashing is likely different than what we'll end up with, but it's abstracted out so probably best to wait for govm progress.

Metadata
Fixes

Fixes # 380

Contributing Agreement

[x] I have read and understood the Optimism Contributing Guide and Code of Conduct and am following those guidelines in this pull request.

Closing, as we determined we need a more memory/storage efficient solution and are rewriting!


install.sh 0.14.1 - CSV "packageserver" failed to reach phase succeeded

Bug Report
What did you do?
I deployed a K8s 1.17.3 cluster (with kubeadm)
Then tried to install OLM with
wget https://github.com/operator-framework/operator-lifecycle-manager/releases/download/0.14.1/install.sh
bash install.sh 0.14.1

I hit this error
Package server phase: Installing
CSV "packageserver" failed to reach phase succeeded

I checked it and it is stuck in Installing.
kubectl get csv -n olm packageserver
NAME            DISPLAY          VERSION   REPLACES   PHASE
packageserver   Package Server   0.14.1               Installing

logs:
kubectl logs -n olm csv/packageserver
error: no kind "ClusterServiceVersion" is registered for version "operators.coreos.com/v1alpha1" in scheme "k8s.io/kubectl/pkg/scheme/scheme.go:28"

Environment

operator-lifecycle-manager version: 0.14.1

Kubernetes version information: 1.17.3

Kubernetes cluster kind: kubeadm 1.17.3 & kubectl 1.17.3

hi @dkirrane, we sometimes see this flakiness when setting up OLM via the script for multiple reasons. I think in this case it may be the CSV CRD didn't get installed fast enough on the api-server before being used elsewhere in the install script.
Could you try again by chance? I'm reasonably sure it would work. We do need to address this flakiness though, it occurs both locally on minikube and in live clusters.

I just hit this same problem this on IBM Cloud IKS 1.16
The script has a 50 retries with sleep 1 (1 second), after a few minutes debugging the the packageserver  was "succeed".
I think increasing the sleep maybe to 10, would go a long way. What do you think @exdx ?


Is there a baseline,when use mobilenet as the backbone for faster-rcnn or mask-rcnn?

Describe the feature
Motivation
A clear and concise description of the motivation of the feature.
Ex1. It is inconvenient when [....].
Ex2. There is a recent paper [....], which is very helpful for [....].
Related resources
If there is an official code release or third-party implementations, please also provide the information here, which would be very helpful.
Additional context
Add any other context or screenshots about the feature request here.
If you would like to implement the feature and create a PR, please leave a comment here and that would be much appreciated.

For now, we only provide yolov3 with mobilenet backbone. For Faster R-CNN and Mask R-CNN, you can change their backbones to mobilenet by adapting their configs.


Document example with Nuxt

From the [nuxt vue plugins documentation](https://nuxtjs.org/docs/directory-structure/plugins/#vue-plugins it would seem all I need to do to get auto-animate to work is the following:
❯ npx nuxi init site
❯ yarn add @formkit/auto-animate
❯ yarn install
❯ yarn nuxi add plugin autoanimate

plugins/autoanimate.ts:
import Vue from 'vue'
import { autoAnimatePlugin } from '@formkit/auto-animate/vue'
Vue.use(autoAnimatePlugin)

nuxt.config.ts:
...
  plugins: ['~/plugins/autoanimate.ts']
...

But this doesn't seem to work, could you add an example to the documentation of how auto-animate works with nuxt?

The directive in Auto Animate is made for Vue 3 — Should work great with Nuxt 3, but not with Nuxt 2. However, you can pretty easily use AutoAnimate on Vue 2 by using the autoAnimate function with a ref — or you can define the custom directive (v-auto-animate) out of the box — for example:
https://codesandbox.io/s/priceless-thunder-m89xo6?file=/src/main.js
Good luck!

how do I register the auto-animate library in nuxt 3
this is what I have in my formkit.config.ts, but it is not working
import { en } from '@formkit/i18n'
import { DefaultConfigOptions } from '@formkit/vue'
import { createAutoAnimatePlugin } from '@formkit/addons'
import '@formkit/themes/windicss'

const config: DefaultConfigOptions = {
  locales: { en },
  locale: 'en',
  plugins: [
    createAutoAnimatePlugin()
  ]
}

export default config


how to use Android channel id

I'm looking to add the android channel id to the Firebase message.
Can i use the withAdditionalData method? Or do I need to use the from Raw method?
I would be grateful if someone could give me an example on how to add this.

Please use new package: https://github.com/gg-innovative/larafirebase!


firefox: set meta.mainProgram to launcherName in wrapper

Description of changes
After #235912 changed the binary name for some Firefox packages, the meta.mainProgram value was no longer accurate and commands such as nix run nixpkgs#firefox-devedition failed because /bin/firefox was executed instead of /bin/firefox-devedition.
This PR sets meta.mainProgram to launcherName, which contains the nameSuffix set in #235912, instead of inheriting the unwrapped package's mainProgram, which is always firefox with no suffix.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

The commit message is missing the "why" part. I had to read the PR message on github to understand the rationale. Please fix?

The commit message is missing the "why" part. I had to read the PR message on github to understand the rationale. Please fix?

Fixed!


Citing this code

I'm preparing an academic paper which includes data generated using this LSTM. How would you like this project to be cited? A footnote URL to the GitHub repo, or is there a publication you would rather me cite?

Submission deadline has been and gone 😄  (in the end, I went with the footnote)


Support for key selection

Currently Biscuit::parse expects a single private key, but it should be possible to pass a function that selects a private key based on the optional root_key_id value.
This might be tricky to achieve with the current rust API, since it expects a closure for key selection. It is possible to pass a closure from JS to rust, but then it's trick to turn the closure result JsValue into a PublicKey. A possible solution would be to provide specific helpers in the rust lib that deserialize the outer proto layer and provides the key id, before continuing.

See biscuit-auth/biscuit-go#150 for related discussion.


To test published metadata and compare it against configured metadata, I want to fetch an entity's published metadata

Includes commits from #27.

[x] Merge #27
[x] Tests

@arothuis @DRvanR


IHttpMaxRequestBodySizeFeature and IFormFeature should return 413

Is your feature request related to a problem? Please describe.
IHttpMaxRequestBodySizeFeature and IFormFeature.FormOptions.MultipartBodyLengthLimit throw different Exceptions when the limit is exceeded.
When these operations fail it results in  500 status code.
Describe the solution you'd like
The correct Kestrel server response should be as IIS does, to return a 413 Payload Exceeded error, which would help in identifying the error on the front end, javascript, spa etc. and providing a relevant error message to the user.
Additional context
I can implement a custom app.UseExceptionHandler to manage this, but it involves catching two different exceptions, and then parsing the exception message to see if it is actually a size error, rather than another communication error.
Of note IIS Express returns a 404.13 error, rather than 413

Agreed, though where the error happens doesn't really have a way to control the status code. This would need to be caught elsewhere and turned into a the relevant status code.

Not sure if this is completed for Kestrel/IIS because I see the proper 413 for both. However it is still present in the HttpSys server. On the client side we show a generic error for 500 like "sorry for your inconvenience, the error has been logged, please try again later" etc, but in this case trying again later would just waste both bandwidth and user/support time.
Ideally the server should return 413 out of the box, or at least indicate with a 4xx that the request in fact failed due to a client error.
Possible workarounds are tedious and error prone:

catch the exception in the exception handler middleware, detect that it signals a request body too large rejection and set the status code to 413 instead of 500. HttpSys throws a generic IOException: https://github.com/dotnet/aspnetcore/blob/0518cb5107372d6057708837cec6650fb962d2b1/src/Servers/HttpSys/src/RequestProcessing/RequestStream.cs#L435 https://github.com/dotnet/aspnetcore/blob/0518cb5107372d6057708837cec6650fb962d2b1/src/Servers/HttpSys/src/RequestProcessing/RequestStream.cs#L453 The only way to detect the rejection is by parsing the exception message and/or stack trace.
mirror the exact same calculations as done by the server: check the content length header, wrap the request stream and throw if the read limit is reached.

If returning 413 is not permissible due to observable behavioral change then at least throw a more specific exception (like BadHttpRequestException). This would make workaround no. 1 easier to implement.


Totals Row in Tables

Hi,
I must be thinking about this wrong, but I couldn't find a way to show the totals row when exporting to a table.

Only a table can be created. Don't recall enabling adding totals. I believe it would be possible if you use -PassThru and then using PowerShell to set the the table properties to enable this.
@jhoneill ?

Add-ExcelTable
$xlfile = "$env:TEMP\test.xlsx"
rm $xlfile -ErrorAction SilentlyContinue

$data = ConvertFrom-Csv @"
Region,Item,TotalSold
East,saw,29
South,nail,87
North,peach,87
West,avocado,51
West,banana,58
West,peach,14
South,lemon,15
North,melon,51
North,banana,69
East,screws,58
"@

$xl = Export-Excel -InputObject $data -Path $xlfile -AutoSize -PassThru

$worksheet = $xl.Sheet1

$table = Add-ExcelTable -PassThru -Range $worksheet.cells[1, 1, 11, 3] -TableName "AllResults" `
    -TotalSettings @{"TotalSold"="Sum"; "Region" = "Count"}

Close-ExcelPackage $xl -Show


Add missing 'inline' modifiers to etcpal::Logger functions

These were failing compilation because they were not marked 'inline'.

Although it looks like the build error was not due to this PR so it should be good to merge.

We are changing CI systems on this project soon. I am going to wait to address the Mac tests until after that.


Fix clap doc-comment

clap always removes the last single period from a doc-comment.

before:
~$ lsd --help
  -a, --all                          Do not ignore entries starting with

after:
~$ lsd --help
  -a, --all                          Do not ignore entries starting with .

TODO

[x] Use cargo fmt
[ ] Add necessary tests
[ ] Update default config/theme in README (if applicable)
[ ] Update man page at lsd/doc/lsd.md (if applicable)

To fix thix, Clap also has a flag called verbatim_doc_comment to avoid removing trailing dots and so on, but solution of this PR seems more specific (and not breaking other benefits of Clap's pre-processing).

thanks so much for this good and funny catch @outloudvi


(BKR-879) beaker 3.0 gem audit

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/2954/

@puppetlabs-jenkins retest this please

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/2967/

@puppetlabs-jenkins retest this please

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/2983/

failure was based on Ruby 1.9, which we expect to fail with these changes. Removed 1.9 from the axis for now to test this further, re-kicking:
@puppetlabs-jenkins retest this please

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/2988/

was still using Ruby 1.9 😿 . Once more, with feeling!
@puppetlabs-jenkins retest this please

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/2998/

LGTM! 👍


Develop

Hello @EdinsonRequena! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file introduccion-al-computo-con-python/modulo_II/complementos/tarea.py:

Line 6:21: E231 missing whitespace after ','
Line 7:6: E225 missing whitespace around operator
Line 7:9: W292 no newline at end of file


Support for ARM64 Architecture

Having just received my PinePhone (Arm64 CPU) I want to start using the native Mattermost client.
With Apple going ARM as well now, also Electron starts supporting ARM. Will someone start porting the Mattermost client to ARM?

It's seems like this issue should be reopened as no Linux-compatible ARM-builds has been published on releases page.

I agree that For PinePhone (Pro) / Pinebook (Pro) users, having an aarch64 Linux build available would definitely be very useful. Currently only amd64 and i386 builds are listed, even though aarch64 is much more commonly used than i386 is, as nearly every distribution has dropped i386 support in the last years.

I'm trying to run the arm64 build artifact on a Pinebook Pro, and it doesn't seem to work:
A JavaScript error occurred in the main process
Uncaught Exception:
Error: /home/kevin/Downloads/mattermost-desktop-5.3.0-develop.1-linux-arm64/resources/app.asar.unpacked/node_modules/macos-notification-state/build/Release/notificationstate.node: cannot open shared object file: No such file or directory
    at process.func [as dlopen] (node:electron/js2c/asar_bundle:5:1810)
    at Module._extensions..node (node:internal/modules/cjs/loader:1205:18)
    at Object.func [as .node] (node:electron/js2c/asar_bundle:5:2037)
    at Module.load (node:internal/modules/cjs/loader:988:32)
    at Module._load (node:internal/modules/cjs/loader:829:12)
    at c._load (node:electron/js2c/asar_bundle:5:13339)
    at Module.require (node:internal/modules/cjs/loader:1012:19)
    at require (node:internal/modules/cjs/helpers:102:18)
    at bindings (/home/kevin/Downloads/mattermost-desktop-5.3.0-develop.1-linux-arm64/resources/app.asar/node_modules/bindings/bindings.js:112:48)
    at Object.<anonymous> (/home/kevin/Downloads/mattermost-desktop-5.3.0-develop.1-linux-arm64/resources/app.asar/node_modules/macos-notification-state/lib/index.js:1:34)

Closed by https://github.com/mattermost/desktop/pull/2428


Only define status or response if intercepting

This fixes an error in using this library with Tauri

Hey @miniBill -- thank you for submitting the PR. It makes sense. I'll accept it now and make a patch release on the weekend.

Released in 1.2.1


Extract -map causes outputs to display as inputs

Running the extract -map example 1 from the February 2016 tutorial slides (slide 96) causes outputs to appear as inputs (below). The example is supposed to map a multiply-accumulate operation, but the outputs are flipped when run on the current version of yosys. I am not sure if this is a bug inside of the show command, or due to an underlying issue in extract.

Inputs are as listed:
/*macc_simple_test.v*/
module test(a, b, c, d, y);
input [15:0] a, b;
input [31:0] c, d;
output [31:0] y;
assign
y = a*b + c + d;
endmodule

/*macc_simple_xmap.v*/
module macc_16_16_32(a, b, c, y);
input [15:0] a, b;
input [31:0] c;
output [31:0] y;
assign y = a*b + c;
endmodule

# Yosys script
read_verilog macc_simple_test.v
hierarchy -check -top test
extract -map macc_simple_xmap.v;;
show

I am not sure if this is a bug inside of the show command, or due to an underlying issue in extract.

It is neither. There is no explicit information on a cell what ports are inputs and what ports are outputs. This is inferred from the cell declaration. However, in your example, there is no cell declaration loaded for the macc_16_16_32 cell type. Simply add something like
read_verilog -lib macc_simple_xmap.v

somewhere to the script before calling show, and show will know which ports are in- and outputs. (Alternatively you can call show -lib macc_simple_xmap.v to let the show command use the port declarations from macc_simple_xmap.v, but not add them to the design in memory.)


@apply styles do not work until a full page reload

Using the example project:
https://github.com/Poimen/stencil-tailwind-plugin-example
Reproduction:

Run the dev server (npm start)
Add a style via @apply to e.g. hero-section.css

After the above, the component is refreshed but the @apply directive is directly put onto the component, without being transformed into it's proper tailwind class. For example if I add
@apply text-clip

My component ends up with this style:

If I manually reload the page, the style appears properly (e.g. text-overflow: clip; in the above example). But it would be great if this worked properly. Happy to help with a fix but not really sure where to start.

This is a stencil dev server reload issue.
As a work around you can use:
  devServer: {
    reloadStrategy: 'pageReload'
  }

as part of the settings - see readme
The HMR is a little off - the HMR runs before the transform, but the transformed code is not used in the HMR ... it uses the return from the HMR, but that source hasn't been transformed yet either.
I was erring on the side of not processing the same files multiple times - so once in the HMR and again in the main loop. The main loop happens during build and dev server, but HMR only in the dev server configuration. Hence, can't/don't want to fiddle with the main loop.
I've looked more into the HMR recently, and I think there could be a fix to apply the TW transform (which should be pretty quick).
I'm just fiddling with some test cases, but should be able to push a new version tomorrow :+1:

@Poimen thanks for the detailed explanation, much appreciated.
I did wonder if that workaround mentioned in the readme was for this issue I hit but I wasn't 100% sure.
Amazing that you think you might have a fix - do let me know if I can help in any way, I'm not an expert in Stencil internals but happy to give anything a go!

@Poimen sorry I see you already made the changes.
To let you know I've just tried your latest version out locally and it's fixed the problem for me, nice work!  🥳

Awesome sauce 👍
I've published 1.7.0 with those changes. There is also a new config function setPluginConfigurationDefaults that saves some of the boiler plate configuration being repeated.
I'm going to close this now, but if 1.7.0 doesn't work, then feel free to re-open 😃

oh, I've also updated the dependencies for the example project 👍

@Poimen nice one, and yea I tried out the setPluginConfigurationDefaults and that seemed to work nicely for me 👍


Validator does not report missing IDs in the consecutive rows.

There are missing IDs in TIT 1:1 line 8, 9 and 10.

Validator reports Missing ID for line 8, but not for line 9 and line 10 which is an empty row.

Yes, there's definitely some funny stuff going on here. There is an error for Line 10, but it's the r2gl row! I suspect that the data being sent to the CV functions is NOT what's being displayed in the top screenshot.


Roles / Security Core Adm

In gitlab by @CalumTowers on Aug 1, 2016, 16:39
Users with "Admin / Membership / Account / Roles /" or "Admin / Membership / Account / Security /" are unable to access security information or update permissions of members despite being given the permission in Roles.

In gitlab by @NFarrington on Aug 6, 2016, 19:06
Fixed by 18ca014d47dd9604b554a9b1313fb0b5204b81c8


Fix/crash in sampleapp

DWU library will replace the implementation of cellForIndexPath for UITableView and UICollectionView.
Using generic of CollectionViewController will cause the unstable of DWU method swizzling since CollectionViewController will have different address because of different generic types
TableViewController is working because menuViewController will have method swizzling for the super class at the beginning.
To fix crash, I can think of three ways

Removed method swizzling (That will be hard to inject label to LayoutKit and implementation has to be modified.)
Removed generics (That's the best solution I can come with. The cons is to make the class not clean as generic one)
Fixing the logic in DWU (It may resolve for this time, but it might be broken again in the future if Apple changes anything on generics)

Codecov Report

Merging #114 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #114   +/-   ##
=======================================
  Coverage   92.63%   92.63%
=======================================
  Files          53       53
  Lines        3053     3053
=======================================
  Hits         2828     2828
  Misses        225      225

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 d8c7781...0b12e54. Read the comment docs.

Moved to a simple solution https://github.com/linkedin/LayoutKit/pull/121


Track latest bevy main branch

Hi, team
Could you create a branch to track latest bevy main branch? I tried myself https://github.com/lewiszlw/bevy_rapier/tree/bevy_track, but there are blocks for me and I don't have enough knowledge to solve them.
For example, in src/plugin/plugin.rs file, bevy_rapier depends on bevy's internal system transform_propagate_system , but this system has been changed to private in https://github.com/bevyengine/bevy/pull/4775.
   pub fn get_systems(stage: PhysicsStages) -> SystemSet {
        match stage {
            PhysicsStages::SyncBackend => {
                let systems = SystemSet::new()
                    .with_system(systems::update_character_controls) // Run the character controller befor ethe manual transform propagation.
                    .with_system(
                        bevy::transform::transform_propagate_system
                            .after(systems::update_character_controls),
                    ) // Run Bevy transform propagation additionally to sync [`GlobalTransform`]
                    .with_system(
                        systems::init_async_colliders
                            .after(bevy::transform::transform_propagate_system),
                    )
                  ...
                  ...

another example

Thanks for helping update bevy_rapier!
I understand the need to update dependencies in a project, PRs are welcome for that, due to the quick changing nature of bevy, as you discovered it's a lot of work to maintain such a branch, with commits added every day, you'll likely end up with your own fork anyway, to match up other dependencies too.
it's usually done near the time of a new release, possibly with a slight delay.
Bevy is currently experimenting a "release candidate" period, which might help with perceived reactivity.


Parallel Routes

This PR adds basic support for Parallel Routes, a feature I've only seen in Next.js.
The following is my reasoning for why Solid Router should support them, but I think Next and this Remix Proposal do a much better job of explaining them.
Proposal
Parallel routes allow layouts to have multiple children, aka slots. Each of these slots define their own routes, generate their own set of matches, and can be placed anywhere in the layout just like regular children (children in fact are just a fancy slot).
This is useful for situations like the following from Mattrax (simplified for this example).

At a high level, there's a root layout that renders a header including breadcrumbs, and then the actual page is rendered below that.
Currently, the header would need to use something like useCurrentMatches, deciding which breadcrumbs should be rendered manually.
With parallel routes, the breadcrumbs become their own slot. The router calculates a separate branch of matches specifically for the breadcrumbs' routes, and provides the rendered components inside props.slots.breadcrumbs instead of props.children. It really is just allowing multiple sets of children.
The config for this would look something like this:
const routes = {
  component: (props) => {
    return (
      <>
        {props.slots.breadcrumbs}
        {props.children}
      </>
    )
  },
  children: [
    {
      path: "/o/:orgSlug",
      children: [
        { path: "/", component: () => "Org Information" },
        { path: "/t/:tenantSlug", component: () => "Tenant Information" }
      ]
    }
  ],
  slots: {
    breadcrumbs: {
      component: (props) => {props.children},
      children: [{
        path: "/o/:orgSlug",
        component: (props) => (
          <>
            Org Breadcrumb
            {props.children}
          </>
        ),
        children: [
          {
            path: "/t/:tenantSlug",
            component: () => Tenant Breadcrumb,
            // necessary to render on subpaths
            children: [{ path: "/*rest" }]
          },
          // necessary to render on subpaths
          { path: "/*rest" }
        ]
      }]
    }
  }
}

Another thing that's great about this is that since slots get their own matches, they also get their own loaders, so our breadcrumbs can define the exact data they need to be preloaded at a route level, rather than having to cram it all into one loader for the root layout.
Unresolved Questions
Default
In Next.js, when a slot mounts but doesn't have any matches it will render a default.js view if one is available.
I'm not sure if solid router needs an equivalent default: ... field for slots. Catchall routes in the slot + its nested children do the same job (as they would in nextjs), but having a single default option to fall back on would be kinda nice.
It would also need to be used in the case that only slots have matches and no children do, in which case the layout route itself would need default rather than its slots.
To see this in action, go to this playground and notice that while the audience slot has a match, the other slots do not and are fallback back to their default page. It's basically a catchall that applies to nested routes too.
Also notice that loading the home route and the navigating to a slot-specific route doesn't display the default page, because of the below behaviour.
Match persistence
In Next.js, when slot A and B both have a match, but then a navigation happens that causes only slot A to have a match, slot B will continue display the match it had previously. I haven't implemented this but it seems pretty cool to have.
To see this in action, navigate between routes in the audience slot of this playground and notice that the views slot remains on 'Home'. If you navigate between the routes in the views slot, the last selected route in the audience slot remains selected, unless 'Home' is picked in either slot, at which point both slots match and 'Home' ends up being selected in both slots.
TODO before merge

Ensure that disposal is handled 100% correctly for slot branches

[x] Initial navigations between breadcrumb layers in Mattrax currently cause the navItems slot to remount Was being caused by not accounting for props.children being undefined while lazy loading nav items
[x] Need to make sure there's 0 memory leaks - Looks like everything's being disposed of properly from working on Mattrax

Ok looking at this now. First couple notes on the PR. I've done some updates so this will need to be rebased. Should be minor but putting that out there. Also you might notice I tend to avoid optional chaining in code. Mostly it is because some CDNs have issues with it. I got into that habit with Solid core and have continued with the other libraries. That's why there are so many chained &&s
As for the feature itself. Yeah I think it is pretty compelling and doesn't look like it adds too much extra code . I do think we need to have opinions on the unresolved things. Can you show me the syntax for the default for a slot. Is it another field at the same level as component/children directly inside the slot.. like:
breadcrumbs: {
      component: (props) => {props.children},
      default: {
        component: //...
      }
      children: //....

And you are saying that is the equivalent of adding a children with path * inside the slot? If so I'm tempted not to bother with default here. Maybe that can be a file system routing thing.
More generally slots can have slots right?.. Basically anywhere there is a children field there could be a slot field beside it. Slots get injected at the same level as children. I think default also being at the same level as children starts getting ambiguous. Like routes not using slots could also set default as well and the only difference is not having access to the named wildcard in the path I think? I guess every slot resolves out on the matches on its own as well so they would have their own path params. Like you could name :id and name it :userId in a slot and as long as the pattern matched for both they'd have that respective name in each location.
Match Persistence seems like it is one of the reasons people would go for such an approach in the first place. It is interesting that on refresh you need default slots because not all navigation information is encoded in the URL. But I have to admit I'm having a hard time picturing any UI that looked like that Next example that I've ever used that had any connection to routing. Honestly I'm having a hard time looking past breadcrumbs or maybe like more info pangels for the use of this feature. That's enough mind you. But I feel things like Match Persistence are the things this can do that would be hard otherwise.
In any case I've seen a number of thumbs up on the issue and there is obviously interest on my part. I'd love to understand the use cases or how people would use the feature a bit more because while it looks like the code doesn't get much trickier this is a big  concept to teach and explain.

Also you might notice I tend to avoid optional chaining in code. Mostly it is because some CDNs have issues with it

Ah ok, maybe we can get typescript to expand the optional chains so we can keep the nicer DX.

Can you show me the syntax for the default for a slot. Is it another field at the same level as component/children directly inside the slot.. like:

It'd be similar to what you did except the component wouldn't be necessary, since default itself just expects a component.
breadcrumbs: {
      component: (props) => {props.children},
      default: (props) => "Default",
      children: //....

And you are saying that is the equivalent of adding a children with path * inside the slot?

I think so, but i'm not 100% certain on the behaviour of * catchalls. I'll try and get some insight from the Next team for why they didn't just use catchalls.

More generally slots can have slots right?

Yes, as an example Mattrax uses a few nested slots. We have the @topbar slot of the main layout, and then that has both @breadcrumbs and @navItems slots so that the shared topbar layout can have top and bottom sections that route independently.

Basically anywhere there is a children field there could be a slot field beside it. Slots get injected at the same level as children

Basically yeah, since children itself is just a slot with a dedicated label.

I think default also being at the same level as children starts getting ambiguous. Like routes not using slots could also set default as well and the only difference is not having access to the named wildcard in the path I think?

This isn't the case, default could only be applied to route configs directly inside a slots object, not just adjacent to any children field. RouteDefinition would look something like this:
export type RouteDefinition = {
  path?: string;
  children?: RouteDefinition | RouteDefinition[];
  slots?: Record<string, Omit<RouteDefinition, 'path'> & { default?: Component }>
}

I guess every slot resolves out on the matches on its own as well so they would have their own path params

The slots themselves wouldn't have path params, but the slot's children could. Note that path is being omitted from RouteDefinition in the type above.

But I feel things like Match Persistence are the things this can do that would be hard otherwise.

Yeah I'm not sure when I'd actually use match persistence, but having it seems nice since it's a feature that has to be implemented at router-level.

After playing with the Next example more, I've realised this implementation doesn't act the same way as Next.
This implementation requires that there be a match for children for a slot to render as well, but with Next you can have just a slot be matched like this and the main children will show default.js. Next treats children literally as an implicit slot and does everything in parallel, I'll change this to do the same.

tried renaming the branch to parallel-routes but github didn't mention that'll also delete the main branch 🤦

turns out 'rename branch' just means 'delete and recreate with a new name' 🤦

Is there any help currently needed with this PR to get consideration continued? I'm increasingly finding that our designs have needs that could be much easier accomplished with this parallel routes support instead of needing to wrap every route file in the generic layout and passing props individually. The ability to have more generic layouts higher up and avoid re-rendering large parts of them, while still changing others would save a lot of hassle.
Happy to beta test or help out if we think this can keep moving forward!

I haven't had time to make this implementation more like Next's, and at some point I'd like to chat with @tannerlinsley about his ideas for parallel routes. Some help with the former would be appreciated, though the router's internals are quite something to navigate 😅


v7.0.0rc1

Update processing logic
Replace old concept queryRule with new one called transformation
Re-organize source code structure

Coverage decreased (-0.8%) to 98.34% when pulling 36f74c77c840faba949920214c050944e056524e on 7.0.0 into e9e5492631c786a6456a33a928e3ed3451b7533f on main.


In the Development Guide, minimum compatible version in the cluster folder should be 1.14.

Minimum compatible version in the cluster folder is 1.14 not 1.11.0

Thanks for the PR. I think it is 1.15 as we seek.

Ok, Thanks Roman, I will close this one and I will open another one with the sign-off


[WFLY-19599]:Upgrade OpenSAML to 4.3.0

https://issues.redhat.com/browse/WFLY-19599

Thanks @jimma and @emmartins


Amberscript configuration fix

The configuration is set to the workflow filename but the id is needed by the code.
Change configured workflow filename to id in the configuration.

I actually do not know if theres is a pr needed for this?

Yes, but you could have merged it yourself if you argue that it's documentation… which I think it is.
Anyway… looks good merging.


Investigate GraphQL interface based on Gaffer Schema

It may be possible to generate a GraphQL schema using graphql-java based on the Gaffer Schema.
This would give us a much simpler remote interface option for developers, although it is unlikely to have the same functionality as the full Gaffer REST API. I think it should be possible to at least traverse from vertexes to Entities/Edges and read properties.

This has been added to the gaffer-tools repository.


[BUG] - Issue when input chinese characters

Description
Firstly thanks so much for your great work. I just started learning rust and tui is my interested area. When I run the demo code, I can input english without problem, but when input chinese, it can display all right, but the cursor is not moving correctly. I believe that is because one character in chinese is 2 letter wide.
Steps to reproduce
cargo run demo.rs
Expected behaviour
cursor moving to the end of the input string, not in the middle.
Environment

OS: linux
Architecture:  x86_64
Rust version: 1.52.1
tui-realm version: 0.3.0

Additional information
Noting at this moment.

You're welcome, thank you for reporting the issue!
I should have fixed this. Try with the new 0.3.1 version 😉

Yes, I confirm that 0.3.1 fixed this issue. Thanks!


Release a version for GHC 9.0 to hackage

It seems that some work has already done migrating this to GHC 9.0, would be great to see a release on hackage...
Atm, I am getting a compilation error with --allow-newer:
[78 of 79] Compiling GitHub.Request   ( src/GitHub/Request.hs, dist/build/GitHub/Request.o, dist/build/GitHub/Request.dyn_o )

src/GitHub/Request.hs:510:18: error:
    • Expecting one more argument to ‘Link’
      Expected a type, but ‘Link’ has kind ‘* -> *’
    • In the type signature: isRelNext :: Link -> Bool
      In an equation for ‘getNextUrl’:
          getNextUrl req
            = do linkHeader <- lookup "Link" (responseHeaders req)
                 links <- parseLinkHeaderBS linkHeader
                 nextURI <- find isRelNext links
                 ....
            where
                isRelNext :: Link -> Bool
                isRelNext = any (== relNextLinkParam) . linkParams
                relNextLinkParam :: (LinkParam, Text)
                relNextLinkParam = (Rel, "next")
    |
510 |     isRelNext :: Link -> Bool
    |                  ^^^^

This seems to be fixed by https://github.com/phadej/github/commit/514b175851dd7c4a9722ff203dd6f652a15d33e8 already, though.

This also breaks for LTS-18.2 /cc @phadej

ping Hackage maintainers: @phadej @jwiegley @mxswd @mike-burns @nataren


Add support for Mongodb Enterprise with Debian

A non documented global variable 'use_enterprise_repo' existed to allow
the use of the Entreprise packages for RedHat.
This patch adds the option to Debian. It also tries to choose the right
APT key depending on the MongoDB version and adds the global variable to
the documentation.

Merged as part of #374


Code and syntax highlighting

How about adding code and syntax highlighting?  following is an example
https://zen-austin-0c80be.netlify.app/posts/basic-markdown-syntax/#code-and-syntax-highlighting

Sorry I accidentally posted in the wrong repo


Disable the generators that are not yet supported in V3

Some generators don't support yet V3

@asyncapi/dotnet-nats-template
@asyncapi/ts-nats-template
@asyncapi/python-paho-template
@asyncapi/nodejs-ws-template
@asyncapi/java-spring-cloud-stream-template
@asyncapi/go-watermill-template
@asyncapi/java-spring-template
@asyncapi/nodejs-template
@asyncapi/java-template
@asyncapi/php-template

Current behavior
In V3 document (example) not supported generators are displayed even though they are not supported.

Expected behavior
For a better Developer Experience(DX):

Disable not supported generators
Inform the user

Tracking link: https://github.com/asyncapi/studio/issues/980
Note: Currently there is a bug in the Doc generation for V3, the fix is WIP https://github.com/asyncapi/cli/pull/1200

Sure @devilkiller-ag please take it

Hi @Amzani, I noticed there's no indication of which generators are supported. Do you think it would be a good idea to include an array listing the supported generators?

@kalpesh-d it's in the description

Is this issue is fixed now?

Is this issue is fixed now?

Yes is current being resolved and is near finishing.

Sorry @Amzani, I got busy with my exams this week, I was planning to resolve this today. @kalpesh-d This issue was assigned to me. You should have asked here before starting your work on this. I am happy that you resolved this issue, but kindly keep in mind to ask the assignee before working on it out of the blue.

@devilkiller-ag, I apologize for not checking with you first.

Hi, trying to generate docs for v3 using markdown and the studio.asyncapi.com is reporting that is not supported. Is that something that I am missing?


map "roomId" back to node in the SUNCG json files

is it possible to map "roomId" or the roomIndex in the *.arch.json back to the node in the *.json file (SUNCG) ?
Looks "roomId" is not assigned based on the ID (e.g. '1_1') specified in the SUNCG annotation.
Thanks

Yes, the roomId value is a linearized contiguous index of the room node ids sorted in ascending order of the first id part x then the second id part y in x_y.  The zeroth value corresponds to null, and room nodes are numbered with index 1 and larger.  The logic that constructs this index is in https://github.com/smartscenes/sstk/blob/master/client/js/lib/scene/SceneState.js#L1398 .  If you want to map from the index value back to the room node you can construct the same room id array and index into it.

Yes, that's correct.  Note however that the linearized index is stable for a given house since the contiguous sorted order of room node ids does not change.


feat(@angular/cli): add ng cache command

With this change we create a new command ng cache that can be used control and check the disk cache settings.
This command has 4 subcommands

ng cache enable which can be used to enable the cache.
ng cache disable which can be used to disable the cache.
ng cache delete which can be used to delete the cache from disk.
ng cache info which will print statistics and information about the cache.

I haven't created sub-commands to change the cache environment and path. As changing this is more of a one of thing which can already be done through ng config. We can always reconsider this if users demand friendlier commands to change these 2 settings.


OMP Scan of Spark Code

@jmertic Looking for a scan results from this repo we'd like to put in the GenevaERS/Spark repo.  https://github.com/IBM/IBMSAFR-Spark

Adding @swinslow who handles these for the project.

Scan complete


New project name?

It is my sincere hope not to start a big bike shedding thread going here, so could we keep this to "I :heart: the name system_registry" or "how about calling it xyz" comments? I'm just wondering if there's a better name for this project prior to setting it in stone for 1.0.
I keep getting Windows 95 vibes from the current name, but I don't have any good suggestions for a better name... I promise to get over my issues if nothing pops up. Feel free to flame me on slack...

system_registry seems fine to me. You could try system_directory, system_index, or system_storage if you feel like you need something new.

I originally had the Windows vibe, but I got over it. At this point I have started to like it. Not sure I have any alternative name suggestions. I think we should stick it out, especially since it's on hex. It would have to be a really good name for me to be persuaded at this stage.

Same. Felt like Windows, then it didn't. Now I just load up regedit, change my KVs and life is good. In all seriousness, I'm 👍  with system_registry at this point too.

I was never a big fan of the name, but I think at this point, lots of people are already familiar with it. We've even talked about it from the keynote at ElixirConf, so unless it's something clearly more descriptive of what SystemRegistry is, I think we're committed.
The word "Registry" in SystemRegistry doesn't bother me any more than the Registry module in Elixir itself. 🤷♂

I have two concerns. One is the initial reaction - dealing with Windows Registry jokes is getting old. I know that people aren't serious, but the Windows scars run deep with some people and I hate that coloring feelings towards a library. The second is accuracy and consistency in naming. I'll describe more.
I think that the system part is mostly accurate. There is only one instance of the system_registry by design. Everything is global - kv pairs are stored in a global container (at least the abstraction makes it look that way). Reports contain global state. I'm not sure that we'll ever allow users to segment their config/status kv pairs between different instances of the system_registry. We'd want them to segment their kv pairs in the tree - probably at the root level.
Regarding registry, I've felt that the word "registry" in Elixir and Erlang is always used for kv structures where the value is always a pid. For Nerves stuff, registry is inconsistent with the name that we gave to the Nerves.Runtime key-value store (Nerves.Runtime.KV). Both of these map keys to non-pid terms (the latter may not work for non-String types, but I could see us fixing that).
Also, there's a pub-sub aspect to this library that's not conveyed in the name.

From @GregMefford on slack

I think Cortex is my favorite so far, since it just implies both storing information and distributing it to interested parties. And it’s more of a marketing term instead of a “thing”

I feel like the ship has sailed on renaming the lib. If something really really really good pops up, feel free to reopen this issue.


Clarify instructions in readme

Updates readme based on feedback by @steowens in #879.

Thanks for that clarification.  I managed to get the app I was working on and now I have a sample on line that shows creating two windows.  But of course it works OK until you shut one down and then the application freezes.  It's interesting though because multi windows would be pretty cool if it could work.

Thanks for the feedback!


Missing docs on cookie encryption/signing

Please add to the documentation:

What encryption and signature algorithm do the encrypted cookie store use?
Are cookie names encrypted, or only the content?
What is the structure of a signed cookie? If I want to read it using JavaScript, how to do it?

Thanks for flagging this!
Additional documentation around cookies was introduced in martenframework/marten@db18167, which covers these points. We can close this issue now.


Update to System.IO.Pipelines

Channels -> Pipelines and they've been merged to corefx. We should update to use those instead.

Done https://github.com/aspnet/SignalR/commit/4aa65cf0bffde56fc1dd02fa44756dc7b0a87cef


test(csharp/test/Drivers/Interop/Snowflake): Numeric operator tests for Snowflake

Adds numeric operator tests for Snowflake c-sharp ADBC driver.

[x] Unary +
[x] Unary -
[x] Addition
[x] Subtraction
[x] Multiplication
[x] Division
[x] Modulo

@lidavidm / @CurtHagenlocher - Hi both. Just checking to see if either of you have some time to check this test PR.


remove crazyegg from master

Description
Somehow crazyegg still was in master
this branch ade from master brings the fix back to develop.
By pushing back into master it should (because of ancestry fix it)
Type of Change

Bug fix ('fix', 'repair', 'bug')

Our latest release resolved this by itself somehow so this shouldn't be necessary.


Hi, How can I configure the tex_template to write Arabic In Manim?

I want to display the text "عشق" in Manim.
This issue is pretty similar to #840
I have tried to do what was suggested for this issue (#840 that is) but I was unable to make it work.

Firstly, I changed TEX_USE_CTEX in manimlib/constants.py to True.

Then I changed the file ctex_template.tex as follows:

The original ctex_template.tex file was:
    \documentclass[preview]{standalone}

    \usepackage[english]{babel}
    \usepackage{amsmath}
    \usepackage{amssymb}
    \usepackage{dsfont}
    \usepackage{setspace}
    \usepackage{tipa}
    \usepackage{relsize}
    \usepackage{textcomp}
    \usepackage{mathrsfs}
    \usepackage{calligra}
    \usepackage{wasysym}
    \usepackage{ragged2e}
    \usepackage{physics}
    \usepackage{xcolor}
    \usepackage{microtype}
    %\DisableLigatures{encoding = *, family = * }
    \usepackage[UTF8]{ctex}
    \linespread{1}

    \begin{document}

    YourTextHere

    \end{document}

I have change it to:
    \documentclass[preview]{standalone}

    %\usepackage[english]{babel}
    \usepackage{amsmath}
    \usepackage{amssymb}
    \usepackage{dsfont}
    \usepackage{setspace}
    \usepackage{tipa}
    \usepackage{relsize}
    \usepackage{textcomp}
    \usepackage{mathrsfs}
    \usepackage{calligra}
    \usepackage{wasysym}
    \usepackage{ragged2e}
    \usepackage{physics}
    \usepackage{xcolor}
    \usepackage{microtype}
    %\DisableLigatures{encoding = *, family = * }
    \usepackage[UTF8]{ctex}
    %\linespread{1}

    \usepackage{polyglossia}
    \setmainlanguage{arabic}
    \setotherlanguage{english}
    \newfontfamily\englishfont[Scale=MatchLowercase]{Linux Biolinum O}
    \newfontfamily\arabicfont{Droid Arabic Naskh}

    \begin{document}

    YourTextHere

    \end{document}

After this I tried rendering the following scene:

class Test(Scene):
    def construct(self):
        text = TextMobject("عشق", font="Droid Arabic Naskh")
        self.add(text)

I got the following error:
Traceback (most recent call last):
File "/Users/sameedmac/PycharmProjects/ManimPyCharmTrial/manim/manimlib/extract_scene.py", line 155, in main
  scene = SceneClass(**scene_kwargs)
File "/Users/sameedmac/PycharmProjects/ManimPyCharmTrial/manim/manimlib/scene/scene.py", line 75, in __init__
  self.construct()
File "/Users/sameedmac/Desktop/ManimTemplatePyCharm/main.py", line 15, in construct
  text = TextMobject("عشق", font="Droid Arabic Naskh")
File "/Users/sameedmac/PycharmProjects/ManimPyCharmTrial/manim/manimlib/mobject/svg/tex_mobject.py", line 147, in     __init__
SingleStringTexMobject.__init__(
File "/Users/sameedmac/PycharmProjects/ManimPyCharmTrial/manim/manimlib/mobject/svg/tex_mobject.py", line 42, in __init__
file_name = tex_to_svg_file(
File "/Users/sameedmac/PycharmProjects/ManimPyCharmTrial/manim/manimlib/utils/tex_file_writing.py", line 21, in tex_to_svg_file
dvi_file = tex_to_dvi(tex_file)
File "/Users/sameedmac/PycharmProjects/ManimPyCharmTrial/manim/manimlib/utils/tex_file_writing.py", line 69, in tex_to_dvi
raise Exception(
Exception: Latex error converting to dvi. See log output above or the log file: media/Tex/1fdc1280131ec656.log

Process finished with exit code 0

I am not sure what I am supposed to do now. Any help with this issue would be great.
Thank you:)

Try to use Text, like #1015

Try to use Text, like #1015

Hi, thanks for the response.
I have tried this now. Here is what I have done:
I tried to render the following scene:
class Test(Scene):
    text = Text("عشق", font="Droid Arabic Naskh")
    self.add((text))

I get the output as follows:

This should not happen. It should instead look like:

So, what it is doing is that it is:

Not joining the letters properly
Printing the letters from left to right instead of right to left.

I am guessing I will have to use the original approach of making changes in ctex_template.tex so that I can use proper XeLatex to print arabic text properly.
Is there a way to fix the original issue that I have laid out?
Are there any other ways to do this?
And this: https://github.com/3b1b/manim/issues/1015#issuecomment-621834526_ be used here somehow??
Any help with this issue would be great. Thanks a lot in advance:)

try to use "texmobjext("\text{عشق}")"

try to use "texmobjext(r"\text{عشق}")"

Hi! Thanks for the response.
I tried doing that and for some reason it is giving the same error...
Is it working for you?

try to use "texmobjext(r"\text{عشق}")"

Hi! Thanks for the response.
I tried doing that and for some reason it is giving me an error...
Is it working for you?
i tried it.But,it give me a error"list out of range",i add same number in it ,it work but just have numbers.

try to use "texmobjext(r"\text{عشق}")"

Hi! Thanks for the response.
I tried doing that and for some reason it is giving me an error...
Is it working for you?
i tried it.But,it give me a error"list out of range",i add same number in it ,it work but just have numbers.

I think you might be better off trying https://github.com/ManimCommunity/manim , I've heard that they've added support for Arabic.

Alsalam Alikoum
After trying some Ideas ,I found this to work , I hope this solves the issue . Salam Alikoum in Arabic with some text in English
change your manim>tex_template.tex to
\documentclass[preview]{standalone}
\usepackage[utf8]{inputenc}
\usepackage[main = arabic,english]{babel}
\usepackage{arabtex}
\usepackage[LAE,LFE]{fontenc}
\usepackage[T1]{fontenc}
\usepackage{amsmath}
\usepackage{amssymb}
\usepackage{dsfont}
\usepackage{setspace}
\usepackage{tipa}
\usepackage{relsize}
\usepackage{textcomp}
\usepackage{mathrsfs}
\usepackage{calligra}
\usepackage{wasysym}
\usepackage{ragged2e}
\usepackage{physics}
\usepackage{xcolor}
\usepackage{microtype}
%\DisableLigatures{encoding = *, family = * }
\usepackage[UTF8]{ctex}
\linespread{1}
\begin{document}
YourTextHere
\end{document}
note that this will change the default language to Arabic so to render English text you need to use text()
For arabic ,You use TextMobject
Example :
https://user-images.githubusercontent.com/69925471/103449184-a6010700-4c72-11eb-856f-8aa382a0bf75.mp4
class Arabic(Scene):
def add_text(self):
text_1 = TextMobject("السلام عليكم " )
self.play(Write(text_1))
self.wait(2)
self.play(Uncreate(text_1))
self.wait(1)
example_text = Text(
"This is some text",
tex_to_color_map={"text": YELLOW}
)
example_tex = TexMobject(
"\sum_{k=1}^\infty {1 \over k^2} = {\pi^2 \over 6}",
)
group = VGroup(example_text, example_tex)
group.arrange(DOWN)
group.set_width(FRAME_WIDTH - 2 * LARGE_BUFF)
self.play(Write(example_text))
self.play(Write(example_tex))
self.wait()
def construct(self):
self.add_text()
best

Alsalam Alikoum
After trying some Ideas ,I found this to work , I hope this solves the issue . Salam Alikoum in Arabic with some text in English
change your manim>tex_template.tex to
\documentclass[preview]{standalone}
\usepackage[utf8]{inputenc}
\usepackage[main = arabic,english]{babel}
\usepackage{arabtex}
\usepackage[LAE,LFE]{fontenc}
\usepackage[T1]{fontenc}
\usepackage{amsmath}
\usepackage{amssymb}
\usepackage{dsfont}
\usepackage{setspace}
\usepackage{tipa}
\usepackage{relsize}
\usepackage{textcomp}
\usepackage{mathrsfs}
\usepackage{calligra}
\usepackage{wasysym}
\usepackage{ragged2e}
\usepackage{physics}
\usepackage{xcolor}
\usepackage{microtype}
%\DisableLigatures{encoding = *, family = * }
\usepackage[UTF8]{ctex}
\linespread{1}
\begin{document}
YourTextHere
\end{document}
note that this will change the default language to Arabic so to render English text you need to use text()
For arabic ,You use TextMobject
Example :
https://user-images.githubusercontent.com/69925471/103449184-a6010700-4c72-11eb-856f-8aa382a0bf75.mp4
class Arabic(Scene):
def add_text(self):
text_1 = TextMobject("السلام عليكم " )
self.play(Write(text_1))
self.wait(2)
self.play(Uncreate(text_1))
self.wait(1)
example_text = Text(
"This is some text",
tex_to_color_map={"text": YELLOW}
)
example_tex = TexMobject(
"\sum_{k=1}^\infty {1 \over k^2} = {\pi^2 \over 6}",
)
group = VGroup(example_text, example_tex)
group.arrange(DOWN)
group.set_width(FRAME_WIDTH - 2 * LARGE_BUFF)
self.play(Write(example_text))
self.play(Write(example_tex))
self.wait()
def construct(self):
self.add_text()
best

https://github.com/3b1b/manim/issues/1173#issuecomment-796085675


Allow configurable destination directory

One problem I have is that a lot of tools will traverse into the internal directory if I do for example go vet ./...
It causes a lot of false positives and things like that when I have a lot of dependencies. If I put them into an underscore directory like _third_party, then it does not have this problem. Could we add an option to make the destination directory configurable?

Not a bad request. However the direction I see vendoring going right now is to use the "vendor" directory with what is now the "GO15VENDOREXPERIMENT" flag. I know it isn't even the default, but the plan is to turn it on by default in go1.6.

To follow up, is there a reason not to transition to the "vendor" folder experiment? The actual code change to add a parameter is tiny.

What are your immediate needs?
What is your long range plan?

I do plan to move to vendor, but not until it is the default bahaviour. I suppose at that time I will need to confront the problem of vendored code being included with vet and such anyway, until I find a good way to exclude those packages. I'm sure I can come up with a fancy bash incantation to invoke vet on only my code or something like that.
If this feature was implemented, I would certainly use it, but if you chose not to, I would also understand.

I could see allow setting an environment variable to set the directory
location. The change would be so small, it would probably be worth it.
Would that work for you?
On Thu, Oct 8, 2015 at 8:09 AM Craig Peterson<EMAIL_ADDRESS>wrote:

I do plan to move to vendor, but not until it is the default bahaviour. I
suppose at that time I will need to confront the problem of vendored code
being included with vet and such anyway, until I find a good way to exclude
those packages. I'm sure I can come up with a fancy bash incantation to
invoke vet on only my code or something like that.
If this feature was implemented, I would certainly use it, but if you
chose not to, I would also understand.
—
Reply to this email directly or view it on GitHub
https://github.com/kardianos/govendor/issues/53#issuecomment-146574259.

I'm open to other ideas as well.

That sounds good to me.


Perform hot exit backup/discard operations in a queue

There were issues with this branch, closing this.

See https://github.com/Microsoft/vscode/pull/19537


Not clear how we detect if subscription is active?

Thanks for this great library, saved me a ton of time!
But; would be nice if we had a short explanation on how we actually know if the user has subscribed or not?
if (renewalInfo.autoRenewStatus == AutoRenewStatus.Off) {

The following code always returns "Off" even though they are still subscribed (although Sandbox)?

Hi Oliver,
The auto-renew status tells you whether the user's subscription is going to renew at the end of the subscription period. To check whether the user is subscribed right now you should look at the expiry date and the revocation date (which is present if the subscription purchase was refunded by Apple). I'd suggest you refer to Apple's documentation:
https://developer.apple.com/app-store/subscriptions/

@agisboye indeed, perhaps the readme could use a short sentence.


Fixed wrong package name

This package requires "serializer-pack" with a version from the "serializer" package. Since "serializer" package already requires "serializer-pack", my guess is this is a mistake.

symfony/serializer does not require the pack (while the ˋserializer` alias is an alias to the pack).
In order to have all the required normalisers to serialise objects, we need the pack, so that dependency is correct :)

That's great but
`$ composer require amqp
Using version ^1.0 for symfony/amqp-pack
./composer.json has been updated
Loading composer repositories with package information
Updating dependencies (including require-dev)
Your requirements could not be resolved to an installable set of packages.
Problem 1
- Installation request for symfony/amqp-pack ^1.0 -> satisfiable by symfony/amqp-pack[v1.0.0].
- symfony/amqp-pack v1.0.0 requires symfony/serializer-pack ^3.3|^4.0 -> no matching package found.
`

Maybe the version is borrowed from serializer instead of serializer-pack which only has 1.0.0 and 1.0.1

it should be "*" actually

Oups, yep, should be "*" as the version.
On Tue, May 22, 2018 at 5:14 PM Nicolas Grekas<EMAIL_ADDRESS>wrote:

it should be "*" actually
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/symfony/amqp-pack/pull/1#issuecomment-391050488, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAxHESCDtw4F80DdwRDz6hVmjzT4A451ks5t1DlXgaJpZM4UI2kO
.

New version 1.0.1 released.


Add option to specify yuv output format

Hi,
Would it be possible to add an option for Vapoursynth & Avisynthplus in order to specify the yuv format from the input file (like yuv420p or yuv420p10le, etc...) ?
Thanks in advance.

core.bs.VideoSource(...).resize.Bicubic(format=vs.YUV420P10) should do the job.

No, it's pointless and would be identical to doing format conversion separately.


Preparation of release v1.5

This PR will be used to finalise the v1.5 release.

Note: See PR #136 for failing Appveyor CI
Appveyor CI will pass test cases inconsistently. All tests have been checked, passing locally and on Travis.


Exclude collision for selected children and its objects

Aims to fix #5. Basically excludes collision objects and CSG objects from the raycast.

Everything seem to work, so merging this. Will come back if new issues are found.


Refactor echo.go -> customskill package

This is a refactor of the code inside of echo.go to a new subpackage called customskill.
This refactor introduces a new concept, called an envelope, to both the request and response. This envelope contains both the metadata for the request or response and then the request or response itself. Ideally the envelope would never be exposed to the developer and would be used, behind the scenes, to properly unmarshal and marshal the request and response respectively. It also allows the use of familiar names for the request inside of the envelope (request versus the made-up body used currently) Currently this is mostly achieved except with response where the envelope is exposed. I have a proposal to address this below (see Proposal)
Request
Request unmarshaling is performed in customskill/request/request.go and is achieved using the BootstrapFromJSON function. This function uses a temporary "dummy" envelope to determine the request type. Once the request type is determined the correct type of request envelope is used to properly unmarshal the request. Instead of returning the request envelope at this stage the envelope's metadata and request is returned. This is done to mirror the way other Alexa SDKs work and to not expose the concept of the envelope to the user.
Tests for customskill/request/request.go are in customskill/request/request_test.go and currently stand at 100% coverage.
Response
Request logic is present in customskill/response/response.go and differs from the code in echo.go slightly. Firstly, most functions are now prefixed with Set to properly communicate what they are used for. Secondly, the String() function has been updated to properly implement the Stringer interface (returning just a string).
Tests for customskill/response/response.go are in customskill/response/response_test.go and currently stand at 100% coverage.
Custom Skill
Custom skill handling logic is present in customskill/skill.go. The new Skill struct defines a customskill's valid application IDs and request handlers. The Handler function is passed an io.Writer and the JSON envelope and returns just an error. It bootstraps request, validates the application ID, and calls the appropriate handler for a response. Finally it marshals the response and writes it to the io.Writer.
Tests for customskill/skill.go are in customskill/skill_test.go and currently stand at 100% coverage.
Proposal/Question
It's arguable if the concept of "envelopes" should exist at all in the request and response packages. One could argue they are actually a concept of the custom skill logic and should exist in the root customskill package. The package could be refactored to move enevelope-related logic to the root customskill package. If done the request and response packages would be "more pure" but it would also turn the request package into a package of nothing by types. Thoughts? If we did do this response.go would no longer expose the concept of the envelope.

Friendly ping @mikeflynn to give this a look over.
FYI: I've developed a test skill using this and it works well.

I like the idea of add VideoPlayer support but I don't want to make this PR any larger than it already is.

Let me know when the discussed changes are on the branch and I'll do a final review.

Happy to make the changes but I first wanted to make sure everyone is ok with the envelope concept being exposed to the user. I don't think it has to be. Instead of exposing the envelope we could have customskill take the Response and Session attributes, construct the envelope itself, and return. Thoughts?

I generally like to keep things a simple as possible for the user and let the library shoulder the complexity, so I'd vote for not exposing it to the user/developer. Since it's just a wrapper for the request/response I don't think there's any benefit to accessing it.

Hey gang, finally had time to address this PR (new laptop + work peak = no time :( ). All of @rking788's review comments have been addressed. I also implemented the Envelope refactor that @mikeflynn supportted in his last comment. All files in customskill have 100% line coverage. Hopefully this closes out this PR. I don't think we are quite ready to merge the refactor branch into mainline but we should be close. I'd like to discuss a few other ideas but I think that can happen in another PR.

This seems like it will do what it's intended to do (rewrite of echo.go to a subproject) but I'm starting to worry that we're losing track of the goal, which is to make writing an Alexa skill as easy as possible. 100% we need to break up "skillserver" so that people can integrate it in to their existing APIs and use the features that they want, but we can't break it up so much that making an Alexa app is back to being a pain in the ass.
Again, this specific PR looks fine. I haven't made an app with it but the diff looks great...however, before we talk merging to the master, we need to go back to the examples and make sure the core goals are met.
We need the following examples:

A complete app that works like skillserver does today, in which the dev has no existing app and just wants a server from nothing to respond to Alexa requests.
An Alexa Skill app that is easily integrated in to an existing go web application.
An existing web application that wants to cherry pick a subset of the features (maybe the security checking) but then handle the raw request itself.

Any other use cases I'm missing? I think those are core set. If we have a good story to tell on those three then we've got something that is a true "2.0"...but not really a 2.0 because golang isn't down with versions.
Thoughts? I admittedly hijacked this PR with this but I've had some time at the end of a crazy busy year to think about this project more and I wanted to get this out there.
(Unless I'm missing something, the example in this branch doesn't actually do anything regarding handling an Alexa Skill request so we need do something there regardless.)

Thanks for taking the time to provide this feedback. I agree with you 100% that this PR is not quite "master-worthy" and that more examples are needed. My ultimate plan was to provide examples for different types of apps (normal custom skill, skill with audio player, skill with Echo Show display features, etc) but the examples you mention are definitely higher-priority and ones I had not considered. I will work on getting those written next.
As for core use cases I'd like to propose one more: the ability to use this package to write a custom skill, in Go, for AWS Lambda. While there isn't yet official support for Go with Lambda (though it has been announced that it is coming) there are various shims to support this use case. I'm happy to provide an example of this.
So, moving forward how about we merge this PR into the refactor branch now and my next PR will be examples to support the examples you mentioned (thoughts on where to put them?) and my Lambda example. I'll do them as 4 separate short PRs unless you prefer one big PR.

Yes, I like the Lambda use case. There are a few project out there that already make deploying go to lambda pretty easy so we can target those for now.
Agreed on merging this in, I just wanted to put hijack this PR momentarily to refocus.


Password authentication

Hi! Could you please add possibility to authenticate with login-password? Because in my home network i don't use keys, and i have many devices)

Just done, npm is on the way, release 1.3.0

available through npm


Replace event-stream with something secure and supported

audit-ci depends on dominictarr/event-stream which was added in #97. The author stopped supporting the module in 2016 and there are suggestions of other packages here: https://github.com/dominictarr/event-stream/issues/102#issuecomment-248754581. In 2018 the author archived the repo, months before it was added to this package as a dependency.
This means that audit-ci (that we use for security scanning) includes a dependency on a library that isn't currently maintained and is likely insecure. In fact, event-stream has had an issue with a supply chain attack (https://github.com/dominictarr/event-stream/issues/115) that while resolved raised concerns (and poor etiquette) from the community.

With all that being said, here are some of the alternatives that I found:
https://github.com/uhop/stream-json
https://github.com/creationix/jsonparse
https://github.com/dscape/clarinet
https://github.com/jgranstrom/zipson
https://github.com/Faleij/json-stream-stringify
In my opinion, stream-json seemed like the most reasonable alternative based on the criteria I set out. I have yet to identify the performance difference.

Thanks for helping me understand this better. I thought I responded last week but don't see that here in the thread, so apologies for the late reply.
Agreed that the current state of the security is more important than historical. I highlighted the issue with event-stream because it appears to be the last known action on that repo.
I guess I don't have anything better to offer and you've given me something to consider.


Movedir

We have users who depend on the MPS classes, so we can't just remove them. Everything inside examples isn't included in the pip release.


odbc support in docker container

This adds ODBC to the main Dockerfile where you can use --build-arg "ODBC_ENABLED=true" to build an image with odbc support.
This also adds support for dynamic scripts in /etc/docker-entrypoint.d where any sh file is executed before the main sqlpad server allowing others to inject scripts without altering sqlpad entrypoint.
Warning: This changes the base image from alpine to debian buster (node:lts-buster-slim).
This also upgrades to node to lts (node 14) (as recommended by node to use latest LTS for production or in this case the docker images)
The reason to change the base image to lts-buster-slim is the common incompatibility with libc in alpine. Many 3th party odbc providers only provide pre-build binaries see as an example oracle-odbc-image and those are generally not compatible with musl-libc.
NOTE: Do not attempt to provide pre-build docker images for oracle or other 3th party drivers on docker hub you may end in a license hell. And the provided example for oracle is only intended as demo for the changes made to the main Dockerfile

@rickbergfalk  I'm not that familar with odbc connections. Please could you provide an example for a connection string I can use in the sqlpad dialog:

@hrbu I'm not using DSN, i'm using DBQ. DSN is used when you populate /etc/odbc.ini and create data sources that are system static. Using DSN's with sqlpad i.m.o. is a bad choice since it locks you to that one config. Using DBQ instead, allows you to use multiple connections to the same service with different credentials without editing the container each time using DBQ only requires you to alter the setup if your connected systems change.
So connection string Driver={Oracle 21 ODBC driver};DBQ=name_of_service_in_tnsnames
And schema:
SELECT OWNER as table_schema, TABLE_NAME as table_name, COLUMN_NAME as column_name, DATA_TYPE as data_type FROM SYS.ALL_TAB_COLUMNS WHERE OWNER != 'SYS' and OWNER != 'SYSTEM' and OWNER != 'AUDSYS'

For full documentation on the options please check oracle's documentation, https://docs.oracle.com/en/database/oracle/oracle-database/12.2/adfns/odbc-driver.html#GUID-618B141E-DD46-4907-99C2-486E801CA878

@JstDust So now the connection string and credentials are accepted. But when testing the connection I finally got the following error [Oracle][ODBC][Ora]ORA-00923: FROM keyword not found where expected with the following log output
{"level":50,"time":"2021-09-01T07:33:51.172Z","pid":1,"hostname":"8c836a660d69","name":"sqlpad-app","odbcErrors":[{"state":"HY000","code":923,"message":"[Oracle][ODBC][Ora]ORA-00923: FROM keyword not found where expected\n"}],"stack":"Error: [odbc] Error executing the sql statement","type":"Error","msg":"[odbc] Error executing the sql statement"}
My Container Config

My SqlPad Test & Result

@hrbu Sorry for the late response, I just ignore the test, the reason that the test does not work and the reason it will fail with a lot of drivers is because the default test query used against the odbc driver is not compatible with the DB see source. With the schema query you should be able to just save and list the schema as verification. There is sadly no good test query that would work on all odbc back-ends so no point of loosing sleep over that test. I considered using the schema query as the test but, the size of that query can be extreme, and may also have security related issues (some db's require permissions for schema stuff)


Version 1.4 does not work with webpack

In version 1.4 you have removed the use of isomorphic-fetch. This however causes problem in webpack, as it now tries to build the app with both whatwg-fetch and node-fetch, which fails.
A possible workaround for this issue would be to configure webpack so it uses something like a null-loader for node-fetch, but I consider this more like a hack than a real solution.

Seeing the same issue when using with React Native's packager

I'm playing catch-up here, but what was the point of removing isomorphic-fetch? I'm getting this same issue with a standard isomorphic webpack build

Any update on this?

Same here. Is the project dying out? @arunoda

The solution is comment these lines on node_modules/lokka-transport-http/dist/index.js
if (typeof fetch === 'function') {
// has a native fetch implementation
fetchUrl = fetch;
} /else if (isNode()) {
// for Node.js
fetchUrl = require('node-fetch');
fetchUrl.Promise = _promise2.default;
} else {
// for the browser
require('whatwg-fetch');
fetchUrl = fetch;
}/


The order of Settings-Sync settings in Settings.json

I want to stop sync of some specific settings but it doesn't work depending on the order of settings in Settings.json.
Let's say there are statements below in Settings.json,
"foo.bar1": "value1",
"settingsSync.ignoredSettings": [
	"foo.bar1",
	"foo.bar2"
],
"foo.bar2": "value2",

"foo.bar1" will be ignored in sync but "foo.bar2" will be not.
Therefore, please enable to ignore the sync without considering the order of setting statements.
Or please let it to always put "settingsSync.ignoredExtensions"and "settingsSync.ignoredSettings" under all setting statements.
VS Code: 1.55.2
Windows10: 20H2
Mac: 11.3

Order is not considered, seems like the value of ignoredSettings is not proper. You are missing comma after first value. Please fix it and try again.

Certainly, I forgot to insert a comma but it's not the essence of this issue.

Ok let me check

I just checked and it seems ignored settings are not synced irrespective of the order. Is it possible to provide a repro steps?


Backend tests

Changes Unknown when pulling 192ade5cdc93e9510db761819fd58d2587669584 on backend-tests into ** on development**.

Changes Unknown when pulling 07a9e6961439bdbe5704753e914b6cc25ea811dc on backend-tests into ** on development**.


How to make the crop pan to always fit the height and width on the image.

I have images with different height and width.
How to make the cropper pan always fit the image height and width.
I tried different aspect ratios but it does not worked.

Either you have to set the cropper aspect ratio to the same value as the image one or you should use the cover object fit.

Either you have to set the cropper aspect ratio to the same value as the image one or you should use the cover object fit.

<div className="relative w-full aspect-square">
            <Cropper
              image={value}
              crop={crop}
              zoom={zoom}
              onCropChange={setCrop}
              onCropComplete={onCropComplete}
              onZoomChange={setZoom}
              objectFit="cover"
              showGrid={false}
            />
          </div>

This is my code but the crop pan , do not fit with the height and width of the image show inside div. Would you please help me to resolve this one.

Please provide a CodeSandbox showing your issue. It's hard to understand with just words 😅

If I understand correctly what you want, you should set the cropper aspect ratio to the same value as the image one.
So if the image is 1920x1000px, set the aspect prop to 1920/1000.

Closing as there was no answer for a while. Comment to re-open 😉

I have similar issues, I cant set the (max) cropSize to the image's size.
The image is 1500x1100 px, but when I set the cropSize to that, the light box is much larger than the image itself.
Zoom is fixed to 1.

I really recommend using the aspect prop instead of cropSize.

I tried that as well, not helped. Also, I need to cut the image pixel perfect.

Do you need a sample app for reproduction?

Yes definitely, this would help

I think the problem is, there is no possibility to keep the image size the same 1-1px ratio with zoom=1. The image will always squeezed. I have tried without cover param, the default is "cover".
I checked the div size of the cropper component: 1336*980

I got it now, and find logically strange to use the cropSize prop relative to bounding div`s size.
It would be better if we can set it eighter in percentage of the image or pixels of the image(as the initialCroppedArea).
In a responsive UI, it is hard to maintain the same result/state(by sliders), if the div is changing.

That's why I don't like the cropSize prop and that it's discouraged to be used from the Readme. aspect is what you are looking for and is meant to be used on responsive UIs.
The pixels size shouldn't matter in the crop UI: it only matters when actually cropping the image to produce the new image.

I came to handle cropSize in this way:

if you want a controlled aspect ration of the cropper, provide the image dimensions,
if you want that the cropper by default tak the full width and height same as the image src provided,
then you can achieve that using  onMediaLoaded function of the cropper.

// controlled dimension of the cropper.
const dimensions={
height:1024,
width:1024
}

const [cropperSize, setcropperSize] = useState<Size | undefined>(undefined)

// calculate the aspect ratio
const aspectRatio = useMemo(() => {
    if (dimensions) {
      return heightWidthToAspectRatio({
        width: dimensions?.width ?? 0,
        height: dimensions?.height ?? 0
      })
    } else if (cropperSize) {
      return cropperSize.width / cropperSize.height
    } else {
      4 / 3
    }
  }, [cropperSize, dimensions])

  <Cropper
              ...
              aspect={aspectRatio}
              onMediaLoaded={({ naturalHeight, naturalWidth }) => {
                setcropperSize({ height: naturalHeight, width: naturalWidth })
              }}
              ...
 />


Fix response headers usage with upcoming node changes

Since I never heard anything back from my question / thoughts above, I took a closer look and (1) added a build in Travis CI against Node.js 8 nightly and then (2) simplified the implementation that kept the library compatible with http2 and the like.


Unit test cases

I take that #14 supersedes this PR... is that correct? If so, can you close this?


AAD v1 vs v2

Are AAD v2 (aka converged) apps inherently multi-tenant or is there a step needed to make those so?

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

ID: 326f7173-a654-fda7-2438-6905da3fdbef
Version Independent ID: 7dbc5a10-9d61-26e1-f532-973b4bdfae9a
Content: How to build an app that can sign in any Azure AD user
Content Source: articles/active-directory/develop/active-directory-devhowto-multi-tenant-overview.md
Service: active-directory
GitHub Login: @BryanLa
Microsoft Alias: bryanla

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

Hello @ydogandjiev. l found some information in https://docs.microsoft.com/en-us/azure/active-directory/develop/active-directory-devhowto-multi-tenant-overview that specifies "By default, web app/API registrations in Azure AD are single tenant. You can make your registration multi-tenant by finding the “Multi-Tenanted” switch on the properties page of your application registration". It doesn't specify v1 or v2 so I assume this is just the default AAD behavior. Hope this helps.

Hello @MohitGargMSFT , please reassign this from @BryanLa to myself. I've taken over the Azure AD dev docs (from the APEX team) and I'll be working with @elisolMS to get the topic updated based on her comments above. Got a separate tracking work item for this in VSTS. Thank you.

Hello @ydogandjiev - Here's the PR that includes updates to address your input: https://github.com/MicrosoftDocs/azure-docs-pr/pull/39271 (CC: @elisolMS)

Hello @MohitGargMSFT - The fix for this issue is now live (topic updated) so we can close this issue. Thank you. #please-close


window_sdl2.screenshot uses pygame

https://github.com/kivy/kivy/blob/master/kivy/core/window/window_sdl2.py#L147

Want to back this issue? Place a bounty on it! We accept bounties via Bountysource.

I can't reproduce the issue using sdl2 from mercurial on a mac or a window machine.

same for me, not reproducable on my osx / linux / windows.

Closing, unless we have a way to reproduce.


TimeZone

Hi
Can i somehow change the timezone on the clock?
This time
started at
2017-09-10 02:18:40 PM
Is completely wrong.

Timezone is currently defaulted to US/Eastern, its a simple change to add this as an argument and use that as the default. If you wanna do a PR or I can add this next time I make a change to the bot.

Was this change ever implemented, lost trying to solve this myself.

This was fixed in the master with the --time-zone flag. For valid arguments see:
https://gist.github.com/heyalexej/8bf688fd67d7199be4a1682b3eec7568


highlight is under the text and not behind that

Highlight is under the text and not behind as espected.
Is the correct behaviour?
Code:
Text: {
                x: 0,
                y: 450,
                text: {
                    text: 'Text a little too long so this part won\'t be visible',
                    highlight:true,
                    highlightColor:0xFF557722
                }
            },
            Text2: {
                x: 150,
                y: 100,
                text: {
                    fontSize: 80,
                    text: 'STYLE IT UP! ',
                    fontStyle: 'italic bold',
                    textColor: 0xff00ffff,
                    shadow: true,
                    shadowColor: 0xffff00ff,
                    shadowOffsetX: 2,
                    shadowOffsetY: 2,
                    shadowBlur: 2,
                    highlight:true,
                    highlightColor:0xFF557722
                }
            },
            Text3: {
                mountX: 1,
                x: 530,
                y: 230,
                text: {
                    fontSize: 48,
                    textAlign: 'left',
                    text: 'you can also add\nline breaks\nusing\n\\n!',
                    lineHeight: 48,
                    highlight:true,
                    highlightColor:0xFF557722
                }
            },

This is not correct behaviour, we have fixed in the following branch:
https://github.com/WebPlatformForEmbedded/Lightning/tree/fix/text-highlight
Will notify you when it's in master

Fixed in 1.3.1
example


Is there a plan to support the collection of Trace and Log ？

Is there a plan to support the collection of Trace and Log, similar to Signoz, to become an all in one observable platform

@tripitakav Coroot already has the Distributed Tracing capability, and full-featured log monitoring is on the way.

@def Thanks! Really looking forward to the log monitoring.

@tripitakav we've released log monitoring recently: https://github.com/coroot/coroot/releases/tag/v0.20.0


Exposure detected at 8ft apart for more than 15 mins (Android version 9) app v53

(test case 31)
device 1: iphone 8 +diagnosed.
device 2: Samsung Galaxy S8 OS v9 app v53 (UUID: Mm_s8-17)
COVID 19 Exposure notifications = On
Bluetooth = On
Notifications = on
July 15: device 2 was exposed to device 1 at more than 8ft for more than 15 mins
July 16: no exposure detected/COVID checks were being done. Upgraded app to version 53 @ 249pm
July 17: 625 am, no push notifications / no exposure detected (app running in background) no COVID19 checks for July 17
closed app, reopened.
home screen displays You've been exposed
COVID Checks for 626 am shows 1 key provided and 1 matched key
Expected behaviour: no exposure detected

@jeberhardt ☝

this phone is currently at 63% not sure what it was at exposure.

Exposure detected at 8ft apart for more than 15 mins

This sounds expected to me. May I know why do we expect no exposure detected in this case?

@emilykuret
as requested, repeated similar scenario July 28
iPhone SE iOS 13.6 app version 66
exposed to +diagnosed XR for 30 mins at 16ft.
July 29 rec'd push notification at 925am that I was exposed.

@emilykuret @mmiron-HC can re-test and or close this out?

Exposure configurations are being rested - closing this.