activate query params by default only if required

The API Playground client seems to set all the query parameters as "on" by default.  (See checkboxes in screenshot).
In my opinion I think that, unless the parameter is defined as required, it would be better to default to the parameters being unchecked, unless a value is explicitly entered into the value field.

Originally posted by JP in https://github.com/scalar/scalar/discussions/1710

this was driving me nuts https://github.com/scalar/scalar/pull/2569


tokei: 7.0.3 -> 8.0.0

Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

cc @gebner

Success on x86_64-linux (full log)
Attempted: tokei
Partial log (click to expand)

installing
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/mn07rwg30zg96h67kjlplrryqsgls5wm-tokei-8.0.0
shrinking /nix/store/mn07rwg30zg96h67kjlplrryqsgls5wm-tokei-8.0.0/bin/tokei
strip is /nix/store/h0lbngpv6ln56hjj59i6l77vxq25flbz-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/mn07rwg30zg96h67kjlplrryqsgls5wm-tokei-8.0.0/bin
patching script interpreter paths in /nix/store/mn07rwg30zg96h67kjlplrryqsgls5wm-tokei-8.0.0
checking for references to /build in /nix/store/mn07rwg30zg96h67kjlplrryqsgls5wm-tokei-8.0.0...
/nix/store/mn07rwg30zg96h67kjlplrryqsgls5wm-tokei-8.0.0

Success on aarch64-linux (full log)
Attempted: tokei
Partial log (click to expand)

installing
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/805iync8b9z0hxzgxycxmf6j98xn11rp-tokei-8.0.0
shrinking /nix/store/805iync8b9z0hxzgxycxmf6j98xn11rp-tokei-8.0.0/bin/tokei
strip is /nix/store/y4ymnvgxygpq05h03kyzbj572zmh6zla-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/805iync8b9z0hxzgxycxmf6j98xn11rp-tokei-8.0.0/bin
patching script interpreter paths in /nix/store/805iync8b9z0hxzgxycxmf6j98xn11rp-tokei-8.0.0
checking for references to /build in /nix/store/805iync8b9z0hxzgxycxmf6j98xn11rp-tokei-8.0.0...
/nix/store/805iync8b9z0hxzgxycxmf6j98xn11rp-tokei-8.0.0


Bump syn to version 2

syn recently released version 2.
I'm just auditing all the deps in my Cargo.lock for syn 1, opening issues on projects that haven't switched over yet.
Any chance you could switch over? The quicker everyone moves to version 2, the sooner projects won't have to build both syn 1 and 2 at the same time :)

Ah yes I saw the announcement. I'll update as soon as possible.

Done

Thanks!


ng build prod - 'require' function calls are not supported

OS?
Windows 10
Versions.
@angular/cli: 1.0.0-beta.30
node: 6.9.5
os: win32 x64
@angular/common: 2.4.6
@angular/compiler: 2.4.6
@angular/core: 2.4.6
@angular/forms: 2.4.6
@angular/http: 2.4.6
@angular/platform-browser: 2.4.6
@angular/platform-browser-dynamic: 2.4.6
@angular/router: 3.4.6
@angular/compiler-cli: 2.4.6
@angular/language-service: 2.4.6
@angular/cli: 1.0.0-beta.30
Repro steps.
ng build --prod
The log given by the failure.
ERROR in Error encountered resolving symbol values statically. Calling function 'require', function calls are not supported. Consider replacing the function or lambda with a reference to an exported function, resolving symbol AccountComponent ...
Mention any other details that might be useful.
ng serve runs fine but not the ng build --prod

@doczoidberg  can you share the offending code?

...

@Component({

  selector: 'footer',
  template: require('./footer.html'),
  styles: [require('./footer.css')],
})
export class Footer {
...

I have the same issue using '@ngtools/webpack', when I try to use require to load an HTML file.

@doczoidberg can you try without the require?  the cli uses a loader under the hood to inline the resources.
@Component({
  selector: 'footer',
  template: './footer.html',
  styleUrls: ['./footer.scss']
})

this does not work. ./footer.html is rendered as a string not the file.

Apologies. I meant templateUrl

the error is gone


iisues with  #include <esp_private/periph_ctrl.h>

#include <esp_private/periph_ctrl.h> it seems that this part of the library is not included.

There are changes in ESP32 board manager version 3.
ESP32 version 2.0.11 —> use ACAN_ESP32 version 1.1.2
ESP32 version 3.0.0alpha3 —> use ACAN_ESP32 version 2.0.0

Le 22 févr. 2024 à 20:35, Alexander @.***> a écrit :
#include <esp_private/periph_ctrl.h> it seems that this part of the library is not included.
—
Reply to this email directly, view it on GitHub https://github.com/pierremolinaro/acan-esp32/issues/11, or unsubscribe https://github.com/notifications/unsubscribe-auth/AEWKZVB5RPUBE6QLDWX5JFLYU6MYDAVCNFSM6AAAAABDVRJUOSVHI2DSMVQWIX3LMV43ASLTON2WKOZSGE2DSOBQGIYDCNA.
You are receiving this because you are subscribed to this thread.

To compile in Platformio with:
[env:esp32dev]
platform = espressif32 @ 6.9.0
board = esp32dev
and
lib_deps =

inaro/ACAN_ESP32 @ ^2.0.1

you need to use <driver/periph_ctrl.h> for "normal" ESP32 boards!!!
//#include <esp_private/periph_ctrl.h> // For ESP32 board manager
#include <driver/periph_ctrl.h> // For M5Stack board manager


Cannot set property 'body' of undefined

Sorry it appears that I didn't fixed this properly in my last pull request :(
/node_modules/crawler-ninja/lib/http/http-request.js:126
result.body='';
^
TypeError: Cannot set property 'body' of undefined
at onContent (/node_modules/crawler-ninja/lib/http/http-request.js:126:22)
at /node_modules/crawler-ninja/lib/http/http-request.js:97:30
at Gunzip.onError (zlib.js:212:5)
at emitOne (events.js:77:13)
at Gunzip.emit (events.js:169:7)
at Zlib._handle.onerror (zlib.js:366:10)

Can you give me  the url that you crawl in order to check this error ?
Thanks


mDNS repeater: cosmetic fixes

Extracted from https://github.com/opnsense/plugins/pull/2762 for easier review.

Merged, thanks!


LDAP Check

Every now and then we need to check the status of accounts in Activity Insight, we download the list of active users that are staff and faculty. We have the ability to cross check most faculty from a report pulled from Workday, but we are finding that we have to cross check faculty and staff at http://www.work.psu.edu/ldap/ because the faculty may not be appearing in our list. Sometimes the number of faculty/staff we need to double check can be 200 or more and as you can imagine that would take us a lot of time.
What I would like to do is to upload a csv file that contains First Name, Last Name, and PSU Access ID to the FAMS Tool, and it check "ldap" and then update the file we uploaded with the following fields:
EduPerson Primary Affiliation
Title
Business Area
Department
Campus
I will then use these results to disable or enable faculty accounts.
Eventually, I would like to build off of this process so it automatically disables the accounts where the LDAP information comes back with EduPerson Primary Affiliation with "Member" or comes back with the user cannot be found.


⚠ Bofrost.nl has degraded performance

In 3494eef, Bofrost.nl (https://www.bofrost.nl) experienced degraded performance:

HTTP code: 200
Response time: 892 ms

Resolved: Bofrost.nl performance has improved in ff87135 after 6 minutes.


DataFrame <-> NumpyEvent

[x] DataFrame To Numpy Event
[x] Numpy Event To DataFrame
[x] Tests
[x] Ensure it works with index with only timestamps

LGTM once what we discussed (e.g. timestamps is improvented).


setMock does not work for installed dependencies

Do you want to request a feature or report a bug?
Bug
What is the current behavior?
Initially I was unsure if this is a bug but could not find anything in related issues that could help.
I've added
What is the expected behavior?
Please provide your exact Jest configuration
Run npx envinfo --preset jest in your project directory and paste the
results here

I'm guessing this was user error?


Failure to honor // +build and xxxx_windows.go for optional code building

As you can see, using go build works (my build machine is macOS/amd64), but using fyne-cross to cross-compile a windows binary does not.
$ make go-build
go build -o test-bin ./gui/...

$ make fyne-cross
fyne-cross windows -app-id net.synfin.FyneCrossTest \
		-developer "Aaron Turner" -app-version 0.0.1 \
		-icon /Users/aturner/go/src/github.com/synfinatic/fyne-cross-test/gui/Icon.png \
		-name windows.exe /Users/aturner/go/src/github.com/synfinatic/fyne-cross-test/gui
[i] Target: windows/amd64
[i] Cleaning target directories...
[✓] "dist" dir cleaned: /Users/aturner/go/src/github.com/synfinatic/fyne-cross-test/fyne-cross/dist/windows-amd64
[✓] "temp" dir cleaned: /Users/aturner/go/src/github.com/synfinatic/fyne-cross-test/fyne-cross/tmp/windows-amd64
[✓] "bin" dir cleaned: /Users/aturner/go/src/github.com/synfinatic/fyne-cross-test/fyne-cross/bin/windows-amd64
[i] Checking for go.mod: /Users/aturner/go/src/github.com/synfinatic/fyne-cross-test/go.mod
[✓] go.mod found
Wrote manifest: /app/fyne-cross/tmp/windows-amd64/windows.exe.manifest
[i] Building binary...
# github.com/synfinatic/fyne-cross-test/gui
gui/test_windows.go:3:6: Test redeclared in this block
	previous declaration at gui/test_unix.go:4:6
gui/test_windows.go:7:16: (*Test).Text redeclared in this block
	previous declaration at gui/test_unix.go:8:6
[✗] exit status 2
make: *** [fyne-cross] Error 1

As you can see the offending two files should prevent these errors with fyne-cross:
$ head -3 gui/test*
==> gui/test_unix.go <==
// +build linux darwin
package main

==> gui/test_windows.go <==
package main

type Test struct {

you can look at and play with the code here: https://github.com/synfinatic/fyne-cross-test

At first glance seems related to different Go versions between your local ENV and the fyne-cross one.
To use build constraint the // +build linux tag must be followed by a blank line otherwise will be ignored.

Yup, that is it.
On a side note, I did also clone the fyne-cross repo and modify the base docker image to use the same version of go as I was using on my laptop and that did not fix it.

Glad it fixed

just a fyi, this doesn't appear to be a constraint per the docs, only that preceding lines need to be empty: https://pkg.go.dev/cmd/go#hdr-Build_constraints
And the fact that normal go build works, would indicate that this is indeed a fyne-cross bug.

The docs reports:

To distinguish build constraints from package documentation, a series of build constraints must be followed by a blank line.

oh, i missed that.  thanks!

Also - fyne-cross uses the same go compiler, we’re not delivering our own ;)

makes sense... just very confused why go build was working and fyne-cross did not. ¯_(ツ)_/¯


RUMM-3021 internal logger 1/2

What does this PR do?
Update the usage of the Internal Logger to adjust level, target(s) and/or message.
Next PR will add a lazy logging API

Codecov Report

Merging #1482 (eb62360) into feature/sdkv2 (7d94736) will decrease coverage by 0.13%.
The diff coverage is 79.38%.

@@                Coverage Diff                @@
##           feature/sdkv2    #1482      +/-   ##
=================================================
- Coverage          82.89%   82.76%   -0.13%
=================================================
  Files                421      423       +2
  Lines              14542    14574      +32
  Branches            2339     2355      +16
=================================================
+ Hits               12054    12062       +8
- Misses              1800     1803       +3
- Partials             688      709      +21

Impacted Files
Coverage Δ

...sistence/file/advanced/ConsentAwareFileMigrator.kt
81.40% <0.00%> (ø)

...atadog/android/core/internal/utils/RuntimeUtils.kt
100.00% <ø> (ø)

...otlin/com/datadog/android/v2/api/InternalLogger.kt
84.62% <ø> (ø)

...g/android/log/internal/logger/DatadogLogHandler.kt
76.58% <0.00%> (ø)

...g/android/rum/internal/debug/UiRumDebugListener.kt
92.86% <ø> (ø)

.../rum/internal/domain/event/RumEventDeserializer.kt
78.38% <ø> (ø)

...datadog/android/rum/utils/ComponentPredicateExt.kt
53.85% <0.00%> (ø)

...adog/android/sqlite/DatadogDatabaseErrorHandler.kt
60.87% <0.00%> (ø)

...ndroid/telemetry/internal/TelemetryEventHandler.kt
78.67% <0.00%> (-0.67%)
:arrow_down:

...ssionreplay/internal/processor/MobileSegmentExt.kt
50.00% <0.00%> (-50.00%)
:arrow_down:

... and 59 more

... and 5 files with indirect coverage changes


Email to users about Twitter login removal plans

Create an email to alert users that use only Twitter to log in that they should add another login method.

Updated spreadsheet  after twitter enrollment cutoff sorted by Shib Login Methods and last activity.
I think Shib Login Methods is best for identifying twitter only WP users. I've added our user meta about opting out of communications, but I think this supersedes that setting.
I've added SAML Login Methods which is more accurate after the migration to simplesamlphp and satosa, but I think you can safely filter by Shib Login Methods with one exception. MLA Commons users that have Twitter and Legacy MLA but have only used Twitter since the simplesamlphp cutover might effectively be Twitter only users if their MLA membership has expired.
Blank SAML Login Methods are people that have not been on the system in a long time.

Updated user list: https://docs.google.com/spreadsheets/d/12g6f3VV-wcUrU4sH1leVpcBTXktLGa95fc3jhcDaWTY/edit#gid=0


SdkEvent::PaymentSucceeded event is emitted for each payment after restoring

SDK is firing SdkEvent::PaymentSucceeded event for each payment when restoring a wallet.
This is becoming a bit of a burden on binding event handlers when restoring wallets with big payment histories.
Would it be possible to differentiate reconnect & restore flows and silence them when restoring?

Looks like a duplicate of #300

Closing in favor of #300


xemu: 0.7.122 -> 0.7.123

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/xemu-project/xemu/releases.
meta.description for xemu is: Original Xbox emulator
meta.homepage for xemu is: https://xemu.app/
meta.changelog for xemu is: https://github.com/xemu-project/xemu/releases/tag/v0.7.123
Updates performed

Version update

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.7.123 with grep in /nix/store/zki6zrhy8skqfm2gzp49ssbb2gylwnsq-xemu-0.7.123
found 0.7.123 in filename of file in /nix/store/zki6zrhy8skqfm2gzp49ssbb2gylwnsq-xemu-0.7.123

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

xemu

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/zki6zrhy8skqfm2gzp49ssbb2gylwnsq-xemu-0.7.123 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A xemu https://github.com/r-ryantm/nixpkgs/archive/bc77f1f793ed68ab0c0a9db765474e349d6868d6.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/bc77f1f793ed68ab0c0a9db765474e349d6868d6#xemu

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/zki6zrhy8skqfm2gzp49ssbb2gylwnsq-xemu-0.7.123
ls -la /nix/store/zki6zrhy8skqfm2gzp49ssbb2gylwnsq-xemu-0.7.123/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

2 packages built:

xemu
xemu.debug

Maintainer pings
cc @AndersonTorres for testing.

@NixOS/nixpkgs-merge-bot merge


simple fcgi.Serve(nil, handler) call crashes

% go build foo.go
% ./foo
2016/02/16 04:04:09 file file+net /dev/stdin: getsockopt: socket operation on non-socket
Attached source code
foo.go.txt
Happens on OSX & Linux x86_64

I think this is intended behavior. To use nil, you need to run it in an environment where stdin is the proper type of descriptor. It won't work with bash as its parent process.


Add support for JSON Schema validation keywords

Closes #28
This PR implements most of the "validators" vocabulary for the 2020-12 draft of JSON Schema. Specifically it allows the following properties to be used on the following types:
Numeric (Integer and Float shale types)

multiple_of => multipleOf
maximum => maximum
exclusive_maximum => exclusiveMaximum
minimum => exclusiveMinimum
exclusive_minimum => exclusiveMinimum

String

max_length => maxLength
min_length => minLength
pattern => pattern

The format keyword from the Format Annotation vocabulary is also allowed.
Array (Collection shale type)

max_items => maxItems
min_items => minItems
unique => uniqueItems
max_contains => maxContains
min_contains => minContains

Object

required => required
'max_properties' => maxProperties
min_properties => minProperties
dependent_required => dependentRequired

This PR doesn't add support for the remaining keywords for "any" type:

enum
const

Lastly, right now these don't receive any sort of validation other than making sure that the wrong keywords don't get added to the wrong types.

@kgiszczak I'd be happy to add an example or two to the README once you're satisfied with the changes.

@bkjohnson it looks really great, and yes please add some examples to README.
Outstanding items to do:

Add examples to readme
Make sure 100% test coverage
Run it through rubocop and make sure no warnings

Thank you, outstanding job.

Some more changes I forgot about:
4. Update changelog with your change (add it to unreleased section) e.g
## [1.1.0] - [unreleased]

### Added
- [bkjohnson or your full name if you prefer] Lorem ipsum

Squash everything to a single commit

Test coverage shows 100% and there are no rubocop warnings for the lib/ or spec/ directories. I limited it this way because there are a number of rubocop warnings in the benchmarks/ directory and gemspec.

Also, did you know about GitHub's "Squash and Merge" feature? It's a repo setting, but it can let you squash a PR into a single commit for merging.
You'll see it in the merge button dropdown:

Here are some more details including how to change the default commit message for the squash commit.

Awesome job, thank you.


What about schemas ?

Hi,
I tested your web interface and I can't navigate through schemas. :-)
Is it normal ?
Is this supported ?
Regards
Guillaume S

Can you be more specific? Also, are you using latest master or latest binary release?
On Wed, Jan 13, 2016 at 8:34 AM, gstoffer<EMAIL_ADDRESS>wrote:

Hi,
I tested your web interface and I can't navigate through schemas :-)
Is it normal ?
Is this supported ?
Regards
Guillaume S
Reply to this email directly or view it on GitHub:
https://github.com/sosedoff/pgweb/issues/113

Closing due to no response.


Expression is always true for Array destructuring

Bug Report

TSLint version: 5.20.0
TypeScript version: 3.6.3
Running TSLint via: CLI

Reproduction using TSLint Playground

TypeScript code being linted
// tslint:disable-next-line:no-magic-numbers
const [beg, end] = "me".split(":", 2);

if (end !== undefined) {
    // tslint:disable-next-line:no-console
    console.log("Got some colons");
}

with tslint.json configuration:
{
    "extends": ["tslint:all"],
    "rules": {}
}

Actual behavior
When running tslint from CLI (strange it doesn't happen in WebStorm or on the TSLint PlayGround), it says that Expression is always true. at the line with the if condition.
Expected behavior
As you can see, expression is definitely not always true.

💀 It's time! 💀
TSLint is deprecated and no longer accepting pull requests other than security fixes. See #4534. ☠
We recommend you instead use typescript-eslint to lint your TypeScript code with ESLint. ✅
👋 It was a pleasure open sourcing with you!

🤖 Beep boop! 👉  TSLint is deprecated 👈  and you should switch to typescript-eslint! 🤖
🔒 This issue is being locked to prevent further unnecessary discussions. Thank you! 👋


Fix whatever is breaking the hosted site from working

Yeah, this is kinda a big one

Fixed! Now firebase hosted site works


Tauri / Cargo Security Issues

For use in our corporate environment, we are scanning open source projects for known vulnerabilities.  It seems that there are many vulnerabilities with the crates pulled in via the bruno-tauri package.  Can we update the dependencies that cargo is pulling in?
I am new to frontend development, and unsure how the bruno-tauri package is even used with the build?  I don't really see references to it throughout the code?
It almost seems that Tauri was abandoned in favor of Electron?  I could be way off, but if that is the case, I can work on a PR to remove the bruno-tauri package.

Probably it could be removed based on this https://github.com/usebruno/bruno/issues/20#issuecomment-1488185033


Request reviews on old PRs

If no reviewer was available when the state changed to needs_merge (or needs_review once that is implemented), we should go back after some time and try to find a reviewer again.

Done in #30.


[ai-form-recognizer] Remove DocumentImage from public API surface.

This feature was removed from the next preview deployment of the service for standard resources, and we'll revisit adding it in the future when it's enabled on more accounts.

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-ai-form-recognizer


PCB Creation AW9523 GPIO Expander is not working

Expierience with version v01
8.04.2022 - xweiser: first hardware test carried out
The board is not working
1st reason: Switch is wrongly mounted (need to be turned by 90 degr.) However footprint does not match
2nd reason:
I2C Interface is not responding usind i2C Scanner in Arduino examples
Tested:
Arduino UNO and Arduino Leonardo
VCC = 5 Volt and VCC=3.3 Volt - not responding
Jumper 1 = 1 shortend: not responding
Soldered form JLPCB and soldered myselfe: no difference = not responding

Thanks to K.H. the I2C error could be resolved.
The RST pin is missing a 10K Ohm pullup resistor.
Add a 10K Ohm resistor asside the board to VCC then the interface i2c is responding to the i2c scanner
The wrong connection to the swithces will be fixed in version v02 of the board (not implemented)


Feature request: run commands through a standard shell, or add option to specify

which shell to use. I can manually specify this in the build commands, it just gets very long.

I prefer Fish shell, which uses slightly different syntax to wrap commands inside others:
#!/usr/bin/env bash
echo $(uname)

#!/usr/bin/env fish
echo (uname)

I'd like to be able to run my dotfiles install script to install fish and switch it to the default shell, then run the script again from within fish when I want to update to the latest version of my dotfiles config. So, the current behavior is:

I write my intended install script for bash
I run it, and it works
I run it again (with my shell now chshed to fish) and it fails

My expected behavior was that it would default to bash or sh or perhaps zsh (something commonly installed on most-to-all systems) and always run the install script there.

Yeah, I agree that bash -c everywhere is a little annoying. I want to keep the current behavior of using the user's shell by default, but maybe we can make it smoother to use a specific shell. How about a shell: <name> option for shell commands? And then this could be specified as a default, so a config could look like:
- defaults:
    shell:
      shell: bash
- shell:
    - echo hi
    - mkdir ~/foo

The shell: shell: bash reads a bit weird to me, but I can't think of a better word for the option right now.

I would also like to see this. After brew install fish I would like to do stuff inside fish shell directly. What would be the best way to do this?

@zoomido as a workaround for now, fish -c "echo hello" run in /bin/bash will run echo hello in fish and print hello

@sdaitzman That worked! I forgot to add the -c flag. But I'm still having problems with this line:
command: /bin/bash -c "$(curl -fsSL https://raw.githubusercontent.com/Homebrew/install/master/install.sh)"
install homebrew [/bin/bash -c "$(curl -fsSL https://raw.githubusercontent.com/Homebrew/install/master/install.sh)"]
/usr/bin/ldd: line 41: printf: write error: Broken pipe
/usr/bin/ldd: line 43: printf: write error: Broken pipe

The command seems to run fine but the error always appears

It might be that the program is expecting stdin/stdout/stderr to be wired up to your terminal input/output rather than /dev/null. You could try specifying the options stdout: True, stderr: True, stdin: True.
(Or you could figure out how to make the brew install work in an unattended mode, though this may be hard because I think it requires your root password? E.g. see https://stackoverflow.com/questions/24426424/unattended-no-prompt-homebrew-installation-using-expect and https://discourse.brew.sh/t/silent-automated-homebrew-install/3180).

I was thinking of adding the shell: option, so you could e.g. specify shell: bash, but I think another approach might be cleaner. Since Dotbot already looks at your $SHELL variable to decide which shell to use, you can modify your install launcher script to have a export SHELL=bash, and I think that should do the trick.

Closing due to inactivity, feel free to re-open if anyone has anything to say.


UI - /vapor/proposals/{id} - Proposal details

This page, apart from showing the general details of a proposal, shows the list of contributors and their contribution. Use this function for that:
const { result: contributions } = await contract.functions.get_contributions_by_proposal({
  proposal_id: 17,
  start: "", // address to start searching, empty to start from the beginning
  limit: 100, // number of contributors
  direction: 0,
});

since the contract is not ready use this dummy data as example:
const contributions = [
  {
    contributor: "13e7ugtkg8CQD8JiPgNa6HaZr89xmTiwXn",
    shares: "500000000000",
  },
  {
    contributor: "1FEANCfbP6scSHTS6FZZCGXFtNsqA6RRvJ",
    shares: "1500000000000",
  },
];

Updated method name: get_proposal_contributions
const { result: contributions } = await contract.functions.get_proposal_contributions({
  proposal_id: 17,
  start: "", // address to start searching, empty to start from the beginning
  limit: 100, // number of contributors
  direction: 0,
});


[Tables] Use entity etag in match conditions

The etag already present in the entity should be used if a match condition is specified, and no additional etag supplied.

@annatisch does this same logic apply for batch operations?

I would say yes - that seems logical.


Not clear how to propose a change/update for a standard[BUG]

Describe the bug
I want to propose a change to a standard. Going to the github Issues page, it isn't clear which option to choose.
To Reproduce
Steps to reproduce the behavior:
1.Go to github issues page
2. try to work out where to raise a change to a standard
Expected behavior
There should be a clear, unambiguous choice to suggest a change to a standard, principle or pattern
Screenshots
Desktop (please complete the following information):
POISE laptop
Smartphone (please complete the following information):
N/A
Additional context
N/A

Original intent was for the 'Propose review of content' Github Issue template to cover this use case (https://github.com/UKHomeOffice/engineering-guidance-and-standards/issues/new?assignees=&labels=&projects=&template=review-content.md&title=)
Description and name for that issue template could be amended to address this bug. e.g.
`Propose a review/change of standards, patterns, principles or other site content
Worth also considering adding a clear link to a standards page on the site (e.g. 'suggest a change to this standard/pattern/principle') that links to the same issue template


Creating simultaneous users' cursors movement example for SDK.

What this PR does / why we need it?
Creating simultaneous users' cursors movement example for SDK.
Tasks:

[x] Implement basic functionality: 1. cursor movement display in local & multiple 2.  cursor shapes display in local & multiple
[x] Implement animation/drawing functionality(both local & multiple):

[x] Pen cursor drawing animation
[x] Heart/Thumbs cursors animation(visually aesthetic)

Any background context you want to provide?
Show another example of how Yorkie could be used.
What are the relevant tickets?

Fixes #
Checklist

[x] Added relevant tests or not required
[x] Didn't break anything

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@YoonKiJin Some commits were pushed after changing the status from draft to ready in this PR. Is this PR ready to review?

I reconverted to a draft. I'll make some additional changes before resubmitting for a review.

I changed the base of this PR from main to v0.4.5-example, because server v0.4.5 is not deployed yet to https://api.yorkie.dev.
@YoonKiJin If you are ready to review this PR. please let us know.


Godot 3.1 joining bones of dae skeleton that shouldn't be joined

Godot version:

3.1 Stable Mono Official
OS/device including version:

Windows 10 Pro 1803
Issue description:

Here's a link to an imgur album of the issue I'm experiencing
Here's a link to the reddit post I made on r/godot about the issue before i knew more
I downloaded some animations off of mixamo to learn about animations in godot (I'm a newbie) and I'm having an issue with godot joining bones that shouldn't be joined and aren't joined when I open the files in c4d or maya. It's a bit hard to explain and the imgur album probably does a better job at explaining the problem.
Steps to reproduce:

Download the files I was using
Import them into godot
Select T-Pose.dae and set root node to kinematic body
Import T-Pose.dae to new inherited scene and add collisionshape
Import Idle.dae as new inherited scene
Copy animation and paste into T-Pose animation player
Select Idle as current animation.

Minimal reproduction project:

GodotAnimationBug.zip

I have also problems, with the new skeleton feature of stable godot release 3.1:
I used instructions from godot documentation https://docs.godotengine.org/en/3.1/tutorials/animation/2d_skeletons.html#doc-2d-skeletons
After that, if I move one bone, it is also connected to another polygon, which I haven't connected before. Also deleting and reassign the connections of polygon property skeleton, it is still connected with two polygons.
As I can see, it seems to be, that the Polygon2D-UV-Editor have a refresh problem with the right view of bones with synchronized polygons. Seeming, it select another bones from the list, as I have selected and confirmed with "ok" before.

@Rossosaurus @pjlg Can you (or anyone else) still reproduce this bug in Godot 3.2.2 or any later release?
If yes, please ensure that an up-to-date Minimal Reproduction Project (MRP) is included in this report (a MRP is a zipped Godot project with the minimal elements necessary to reliably trigger the bug). You can upload ZIP files in an issue comment with a drag and drop.

Closed due to no activity.

Closed due to no activity.


Fixing build of px4fmu-v3_default with arm-none-eabi-g++ 4.9.3 20150529

That pull-request fixing the following build problem from tags/v1.7.3 with the latest available arm-none-eabi-g++ on Ubuntu - 4.9.3 20150529:
$ **make px4fmu-v3_default**
[2/8] Building CXX object src/drivers/bmp280/CMakeFiles/drivers__bmp280.dir/bmp280_spi.cpp.obj
FAILED: /usr/bin/arm-none-eabi-g++   -DBUILD_URI=localhost -DCONFIG_ARCH_BOARD_PX4FMU_V2 -DMODULE_NAME=\"bmp280\" -DPX4_MAIN=bmp280_app_main -D__DF_NUTTX -D__PX4_NUTTX -D__STDC_FORMAT_MACROS -I. -Isrc -Isrc/modules -I../../src -I../../src/drivers/boards/px4fmu-v2 -I../../src/include -I../../src/lib -I../../src/lib/DriverFramework/framework/include -I../../src/lib/matrix -I../../src/modules -I../../src/platforms -INuttX/nuttx/arch/arm/src/armv7-m -INuttX/nuttx/arch/arm/src/chip -INuttX/nuttx/arch/arm/src/common -INuttX/nuttx/include -INuttX/nuttx/include/cxx -I../../platforms/nuttx/NuttX/apps/include -Iexternal/Install/include -fno-common -ffunction-sections -fdata-sections -mcpu=cortex-m4 -mthumb -march=armv7e-m -mfpu=fpv4-sp-d16 -mfloat-abi=hard -fno-common -ffunction-sections -fdata-sections -g -fno-exceptions -fno-rtti -std=gnu++11 -fno-threadsafe-statics -DCONFIG_WCHAR_BUILTIN -D__CUSTOM_FILE_IO__ -fcheck-new -Wall -Warray-bounds -Wdisabled-optimization -Werror -Wextra -Wfatal-errors -Wfloat-equal -Wformat-security -Winit-self -Wlogical-op -Wmissing-declarations -Wmissing-field-initializers -Wpointer-arith -Wshadow -Wuninitialized -Wunknown-pragmas -Wunused-variable -Wno-unused-parameter -Wunused-but-set-variable -Wformat=1 -Wdouble-promotion -Wno-missing-field-initializers -Wno-overloaded-virtual -Wreorder -fvisibility=hidden -include visibility.h -fno-strict-aliasing -fomit-frame-pointer -funsafe-math-optimizations -ffunction-sections -fdata-sections -fno-strength-reduce -fno-builtin-printf -Os -DNDEBUG   -fno-strict-aliasing -fomit-frame-pointer -funsafe-math-optimizations -ffunction-sections -fdata-sections -fno-strength-reduce -fno-builtin-printf -Wframe-larger-than=1024 -MMD -MT src/drivers/bmp280/CMakeFiles/drivers__bmp280.dir/bmp280_spi.cpp.obj -MF src/drivers/bmp280/CMakeFiles/drivers__bmp280.dir/bmp280_spi.cpp.obj.d -o src/drivers/bmp280/CMakeFiles/drivers__bmp280.dir/bmp280_spi.cpp.obj -c ../../src/drivers/bmp280/bmp280_spi.cpp
../../src/drivers/bmp280/bmp280_spi.cpp: In constructor 'BMP280_SPI::BMP280_SPI(uint8_t, uint32_t, bool)':
../../src/drivers/bmp280/bmp280_spi.cpp:90:67: error: declaration of 'external' shadows a member of 'this' [-Werror=shadow]
 BMP280_SPI::BMP280_SPI(uint8_t bus, uint32_t device, bool external) :
                                                                   ^
compilation terminated due to -Wfatal-errors.
cc1plus: all warnings being treated as errors
[2/8] Generating git hash header
ninja: build stopped: subcommand failed.
Makefile:153: recipe for target 'px4fmu-v3_default' failed
make: *** [px4fmu-v3_default] Error 1

$ **/usr/bin/arm-none-eabi-g++ --version**
arm-none-eabi-g++ (15:4.9.3+svn231177-1) 4.9.3 20150529 (prerelease)
Copyright (C) 2014 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

@LorenzMeier, does last change is ok?
Also, seems like something is wrong with environment of cl/circlecl, the error is not related to that PR.

@theg4sh could you try rebasing on master?

@dagar that change required for tagged branch v1.7.3 (stable) only. In master that issue is fixed by renaming method argument external to externalL (odd naming, imho) by Shivam Shuang at 2018-03-18 in commit 062e4406.
So, I have no reason to rebase into master due to the issue is fixed there.
BTW, that change was made for v1.7.3 and the issue is still opened in stable tag.
Is there possible to make new stable tagged commit with included 062e4406?

Looks good to merge once CI passes.


Add the delta store to the daemon's store map

The LayerDownloadManager uses this map to look up existing layers.
Connects-to: #24
Connects-to: #98
Change-type: patch
Signed-off-by: Robert Günzler<EMAIL_ADDRESS>Replaces #120

@robertgzr I believe this will cause balena to re-use the layer from the delta store and not copy it to the current store. Is this true? What we want is to copy the layer from the delta store in case we pull a delta image that shares a layer with an image from the delta store

@petrosagg maybe I'm misunderstanding what we want the delta store to be?
You are right. It reuses the layer from the delta store because I thought that's why we have this option, to be able to have a separate store for deltas...
Although thinking about it that way, currently the image delta command is not aware of the delta store at all. So if I'm not wrong shouldn't they be put there if we run with delta-data-root?

@petrosagg @pdcastro
I finished writing up a full test for this. We can probably extract a more narrow scoped one for just the delta feature itself from what is there.
This fails even with the one-line change introduced in this PR, because I assert based on the comment https://github.com/balena-os/balena-engine/pull/121#issuecomment-433572474

@petrosagg adding to my comment from last week:
Can you review the test to check if https://github.com/balena-os/balena-engine/pull/121/commits/074e022656932abbf9b6b1d83fbc7e5e3bf853e7#diff-f71215024c19a7220bde5322289e81fbR41
is the final state we want?

After some further investigation I think the solution might not be as simple as in the first commit here :stuck_out_tongue:
My change enables the balenaEngine to find images in the delta store and not fail there but like @petrosagg said it only reuses them... If we want to copy them over to the root store I would add a new field to distribution.Config that allows us to access only the delta store itself (and lookup layers) and put them into the root store (using the current Config.ImageStore)

With the latest changes this now works if I try to reproduce as described in #98
As for tests... I was not able to make it work for the third test (where we push/pull from a registry). The image id of the delta changes in between creation and my final check and makes the test fail, since that is what I use to see if the operation was successful.


Read cloud credential from Secret

A Secret representing the service account credential will be created by
Rancher using the CloudCredentials controller. This change adds the
ability to retrieve the credential from a Secret, identified by
<namespace>:<name>, where the JSON-encoded credential will be stored
under key gkecredentialConfig-credential.

Rebased on master to fix ci

Condensing this into #4


DBZ-1829 (Don't merge)

https://issues.redhat.com/browse/DBZ-1829
Hey @bhardesty, this removes the deprecated options discussed in DBZ-1829. I'd prefer to not actually apply this upstream, as we should list options there, also if deprecated. But you should apply this change downstream. Let me know if it's possible and I can close this PR. Thanks!

@gunnarmorling I will create a separate PR to handle this for the downstream. I just need to apply some ifdef conditions so that the deprecated options remain in the upstream, but do not show up in the downstream.

Superseded by #1331.


Azure Custom Translate API :

There is a[ dedicated API (currently in preview) that allows users to manage creating or updating training on a regular basis through their own app or webservice.
In most of the requests it is necessary to provide an "authorization" parameter, that is your Access token. How is it possible to get this Access token?
Another question concerns uploading training documents using API :

Method = POST
Path = /api/texttranslator/v1.0/documents/import

What should be fed into Files  parameter? Url or binary array?
Thank you in advance

@ajakupov please provide the link to the article that you are following so that we can better assist you.

There is no article explaining how to use Custom Translator API, that is why we are facing some challenges using this service from the code.
Here is the link to the API.
We are going to automate the training with a web service that will automatically generate training documents (approved by service users) and we wonder if there is a way to upload documents and train models using API, without having to connect to the Custom Translator Portal.

As @ajakupov mentioned, it looks like this is the closest doc:
https://docs.microsoft.com/en-us/azure/cognitive-services/translator/custom-translator/overview
I'm going to loop in cognitive for visibility.

@ajakupov Thanks for bringing this up. Since the API is currently under preview the option to provide feedback is directly through the API page by contacting our developers at custommt[at]microsoft[dot]com, We are also internally following up with our team to provide more examples of the API's.

@ajakupov Adding sample code to the swagger UI to describe the usage of the API is in the pipeline and you should see updates early next month(Jan)
If you need additional information or have feedback on a specific API usage, please drop an email to our developer team at custommt[at]microsoft[dot]com so they can address accordingly.

@ajakupov We will now proceed to close this thread. If there are further questions regarding this matter, please tag @RohitMungi-MSFT in your reply.


Extends url_monitor to execute POST requests.

For internal use we needed to be able to monitor API requests using POST. url_monitor was nearly perfect so we just extended it.
Hope it can get merged upstream so we no longer have to import this as a custom plugin.

Looks good - thanks @terwey ! We'll update the public plugin directory today.


Can't close scanner within a service

I am using barcode scanner to read QR while a service is running on background to receive instant payments.
I am receiving the payment and I thought I could fire an event subscribed in the page where the barcode is scanning. The event is also correctly fired but the camera is not closed with this.navCtrl.pop(). and if I alert something is displayed behind the CaptureActivity
Is there any way to close the Scanner without "scanning" ?
Thanks in advance.

+1
We also need to close barcode scanner automatically when user session expires.


update keycloak prod config

Motivation
Update the Keycloak Prod config to use the new OIDC config as per raincatch-1160.
"keycloakConfig": {
    "realm": "",
    "bearer-only": true,
    "auth-server-url": "",
    "ssl-required": "",
    "resource": "",
    "use-resource-role-mappings": true
 }

Changes Unknown when pulling 29b53dafbb7fc25096babe68e6dc151db3c5e8f7 on update-keycloak-prod-config into ** on master**.

Thanks for contributing to Raincatcher  :1st_place_medal:
Your change was reviewed and accepted by our team


Support CocoaPods

Issue: Link
Short description
Support CocoaPods
Solution
I flatted the project hierarchy and included a .podspec with the specification of the library.
Implementation

[x] Flat the library folders hierarchy.
[x] Include a .podspec file in the repository
[ ] Update the RELEASE process.
[ ] Release a new version of xcodeproj.

GIF

This PR is blocked by https://github.com/carambalabs/xcodeproj/pull/32 because we need to get rid of the CommonCrypto dependency.

This is now unblocked

Yeah, I'll finish this one tomorrow. 😛

 CocoaPods support


please put a way to download all chapters, regardless if its already read or havent been read before.

Describe your suggested feature
when downloading chapters, besides the options of next chapter, next 5, 10, 25, or unread chapters, there should be an "ALL" chapters download.
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I have updated the app to version 0.16.5.
[X] I will fill out all of the requested information in this form.

there once was but it has been removed and it won't be back. mark the chapters as unread if you are gonna read them anyway

there once was but it has been removed and it won't be back. mark the chapters as unread if you are gonna read them anyway

why it was removed?
marking chapter will take too damn long though......

there once was but it has been removed and it won't be back. mark the chapters as unread if you are gonna read them anyway

why it was removed? marking chapter will take too damn long though......

To avoid slamming sources with loads of requests.
I encourage you to explore the alternative settings the app offers, from selections to Download Ahead and Delete After Read.

yeah, thats just take more steps.
the previous app only has 3 steps.

tap title.
tap download
select "all" button

thats it. period. thats why i want it back.

yeah, thats just take more steps.
the previous app only has 3 steps.

tap title.
tap download
select "all" button

thats it. period. thats why i want it back.

"The previous app" was the one who removed it. It's not coming back. Either figure out an alternate way or better yet, don't bulk download literally everything. Look into Download Ahead in the settings.
Closing this now as its been clearly denied and the discussion isn't going anywhere either.


FlashAttention for TransformerDecoder

I went through torchtune/modules/transformer.py but could not find anything about FlashAttention. Is it supported? Otherwise, it would be great to see it in the future. I'm using Llama 3.1 with very long contexts and I believe FlashAttention will save me significant memory.

Hey @NeuralFlux, our modules use PyTorch's nn.scaled_dot_product_attention which automatically selects the best backend for the attention calculation. In most cases, flash attention will be used if your hardware supports it and you're using bf16. If you pass in a non-causal mask then it will use memory-efficient attention (as is the case for sample packing).
If you're using our configs/recipes and are not modifying the modules or using sample packing, then you should be using flash attention.

That answers my question, thank you. Btw, I'm relatively new to this and want to know what non-causal mask exactly means. Where can I find more info?

Happy to explain.
Causal mask simply is a way to ensure that the next predicted token only depends on the previous token. In practice, this materializes as a lower triangular matrix:
                 good morning how are  you
                |====|======|===|===|====|
good            │ ■  │      |   |   |    |
morning         │ ■  │  ■   |   |   |    |
how             │ ■  │  ■   | ■ |   |    |
are             │ ■  │  ■   | ■ | ■ |    |
you             │ ■  │  ■   | ■ | ■ | ■  |

The filled square indicates that the token on the left "attends" to the token on the top. So you can see that "morning" only attends to "good morning" and "you" can attend to all of "good morning how are you". Without the causal mask, the transformer decoder can "cheat" and look-ahead at the rest of the sentence to predict the next token.
This is the default in most cases, but there are situations where you'd need to use a different mask. For example, in sample packing where you concatenate multiple unrelated sequences into a single sample, you only want each sequence to attend to itself. So you get a block triangular mask instead (see here for a great comparison). Or in the case of multimodal models, the mask used for cross-attention between images and masks does not follow the causal pattern either (see here for a depiction).
You can try this article for a deeper dive into the math behind this.
Hope this is helpful. Let me know if you have more questions.

Got it, thanks for the references too!


Document return values for void methods in OpenRPC

I.e., methods where response.result is null or undefined.

@BelfordZ, how should I do this according to the spec?

@BelfordZ, how should I do this according to the spec?

@rekmarks JSON Schema has a null type. You can make a content descriptor whose json schema type is null. As for undefined, you can put schema: true, since any result is ultimately valid there. Not super ideal way to handle undefined... ill try to think of a better one.

@rekmarks JSON Schema has a null type. You can make a content descriptor whose json schema type is null. As for undefined, you can put schema: true, since any result is ultimately valid there. Not super ideal way to handle undefined... ill try to think of a better one.

this looks great BTW. Thank you for your efforts!!

this looks great BTW. Thank you for your efforts!!


feat: avoid to duplicate value of analyzed/nodeScopeIndex fields in :fulltext

The initial intent of this PR was to completely get rid of :fulltext field.
This is not just used for analyzed/nodeScopeIndex fields though but also for other features (renditions, spellcheck).
The field is still part of the mapping but it's not used to store analyzed/nodeScopeIndex properties. The query side uses a multi-match cross-fields query. This improves both indexing and query performance.

Thank you for submitting an entry to our competition. Your entry has been safely received and we will be in touch in due course if you are selected.
Best regards,
The Nature Communications Editorial Team


feat: restore the module docstring code snippet

We had this in mathlib3, allowing one to type mo (then press tab, at least for me) in an empty file and have a snippet template of a module docstring inserted with the title named to the filename, which can save a little time.
The implementation is now a bit overkill as it splits on _ and capitalizes still to get from the filename to the module title, which likely has no affect for most lean 4 filenames.

could we also have the copyright snippet?

could we also have the copyright snippet?

I think that's already there, did you try it?

indeed, fuzzy memory:)


Ogc flashing on and off and making game stutter

Hi, i used open games camera the other day and it was working fine, perfectly intact and now today I’ve just tried to take some shots and it’s playing up.
Here’s a link to a video showing my problem ... it’s basically when I click home key or freeze frame, like I said I know for a fact it was working perfectly 2 days ago :(
https://we.tl/t-cks9NBIIKn

Fixed! It was a keyboard problem! Sorry for wasting any time and thanks for making a great program!


Fix incorrect module listing command [Fixes #850]

OPS Build status updates of commit ef9589c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-conceptual/azurermps-6.13.0/install-azurerm-ps.md
:white_check_mark:Succeeded
View (azurermps-6.13.0)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Can I preload image so I can set an expire date?

Title pretty much says it.

You're talking about two different things here.

preloading images (this essentially preloads the images by default)
setting expiry dates (in the image contents? on the loading of the image?)

Please mark this issue ticket as closed if you feel the topic has been addressed or if you have found a solution.
(Bonus: please provide your solution if you have solved it outside of the conversation here)


Any ideas on how the database schema should be ?

Thank you guys for this awesome "Library" !
and ideas how to store entities postion, rotation, scale etc... in a MySql database ?

@khalilyamoun when I used it in a production app, I used Realm to store entities. It's easier with JSON-like databases. For SQL you'll need to write wrappers


Season1/episode7

Merge request for EP007

Trying to merge locally past midnight to see if the update will work


fix: Remove private widgets from autocompletion

Description
This PR removes private widgets from autocomplete hints
Fixes #10534
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?
Manual
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

Test coverage results :test_tube:
:green_circle: Total coverage has increased
// Code coverage diff between base branch:release and head branch: fix/remove-private-widgets-from-autocomplete

Status
File
% Stmts
% Branch
% Funcs
% Lines

:green_circle:
total
55.12 (0.01)
36.88 (0.01)
35.01 (0.02)
55.6 (0.01)

:red_circle:
app/client/src/sagas/PostEvaluationSagas.ts
16.44 (-0.11)
0 (0)
0 (0)
20 (-0.17)

:green_circle:
app/client/src/workers/evaluationUtils.ts
59.69 (1.54)
55.56 (1.07)
64.91 (1.95)
57.54 (1.63)

/ok-to-test sha=cb0a26b


Is this project dead?

The last closed issue was 2 years back.
The last commit was 3 years back.
52 open issues with no comments/response.
linkedin's fork of dust seems to be much more active.

So is it still being actively developed? Or should we be looking at linkedin's fork?

@mrchief can you provide a link to the fork?

https://github.com/linkedin/dustjs


No titles in Configuration screen

Hi, I got it all working and noticed that the titles do not show up in the configuration screen.  See screenprint enclosed

I had that happen 2-months ago running a dev version of HA in a docker Container on a windows computer. As I remember, someone else had that running in a virtual environment. I upgraded my RPi version of HA and everything was displayed correctly. Somehow, it was related to HA  and the frontend running in that type of environment.  I inspected the code the front end built and everything was missing. I never found the actual cause.
Btw, this is a dead repository that was put up for the initial docs development. Issues should be posted on the iCloud3_v3 repository here. Click the’bug’ above the ‘Actions’ On the Event Log in the upper right and it will take you there.


DinoCatalog v3

Resubmitting code refactoring from yesterday along with two, hopefully fixed, odd Hound warnings.

Nice refactor.  Take a look at the comments and think about how you would refactor.  No need to resubmit.


feature(policies): check if policy name already exists, change input error message position and change model type classes

LGTM


Anybody continuous his own branch? I can join to develop.

Project development is stopped, as I understood.  May be anybody continuous his own branch? I can join to develop.

@icfantv, can I be added as collaborator to continue with this repo ?

@navarroaxel I would strongly encourage you to have a frank discussion with some OSS project maintainers, preferably more than one and with very popular projects, to get a sense of what, exactly you're getting yourself into.  Managing a popular OSS repo is a full-time job in and of itself and it's primarily thankless.  It also usually takes multiple people.
You need to have high standards and you cannot let the desire to just get stuff done (or just please the community) interfere with maintaining those high standards.  Example, there are a TON of PRs which lack tests.
If you want to fork and maintain this repo, it's OSS, I cannot stop you.  If nothing else, it will be a huge, eye-opening experience for you.  But I'm not comfortable giving you commit privileges to this repo at this time.
I'll go ahead and leave this issue open for a little while to see if people see it and you can get folks on board.  I will say, however, that I don't feel continuing to maintain this is the right way to go.

@icfantv I'm sure I'm not a hero and maybe I can't handle the size of this OSS. But the way this project was closed was sadly incorrect.
The latest release was in January and no more news about Merges and releases until your latest update saying this was closed due to maintainers moved to other cooler or more rewarded projects. This project is used by a lot of people and there is still bugs in the demo page. I think critical and high priority issues should being fixed.
There area a lot of people who sent PRs since February fixing things and nothing was doing until your update saying this project is feature-complete with bugs in the own demo page.
I really don't know what to do, but if anyone forks and creates his own package we are dividing our efforts and disintegrating this awesome project. This is an awesome project because this replaced jquery :clap:. The close of this project is so sad to me, even when I'm moving to another non-deprecated technology.
Maybe we can build another group of maintainers with good reputation and this project can receive fixes in an enough good time of response. Any time is better than 10 months of nothing.

@navarroaxel I was really debating how, or if, to respond to this.  Intentional or not, you touch on some of the enormous difficulties of running a popular OSS repo:

There will always be people who disagree with you.  ALWAYS.  Responses will vary from taking it in stride to launching personal attacks because some people treat OSS like it's there to serve only their needs.  One needs to have a VERY thick skin on OSS projects.
People [may] search/read, but they do not see.  People post issues asking for help even though the ISSUE_TEMPLATE indicates how to get help.  The PR template says exactly what we expect from PRs and yet we still have PRs missing tests or breaking existing functionality.  Your comment about there being no news is incorrect because we posted this issue in early January about contributors/maintainers.  At that point, only @wesleycho was able to donate [a very limited amount of] time.

For us to add anyone as a contributor, we need to see a solid history of his/her contribution work to learn what type of project ambassador he/she would be.  Your comment about having a PR open since March gives the optics, fair or not,  of you just wanting your PR merged so you can move on.  And as I mentioned before, many PRs added simply do not include tests, breaks existing functionality of the library, or both, so them languishing is not an indication of actual contribution.  One needs to understand the full ramifications of a code change before just accepting it and that comes with time.  Time that, unfortunately, no maintainer now has.

@icfantv, I completely understand (and appreciate...) that you and your team have spent countless hours of your free time nurturing this project, but I also understand that it's time for you guys to move on, that's life.
My problem now is that our team maintains and develops several AngularJS applications with tens of thousands of lines of code and an upgrade (*cough* rewrite *cough*) to Angular 4 or even moving to an another framework simply won't happen, because reasons.
So we're stuck using Angular 1.x and by the looks of it we're not alone; according to builtwith.com there are about half a million live web sites using AngularJS and judging by the amount of stars this project has here on github there are probably tens (hundreds?) of thousands of projects relying on your package. Heck, you even have about half a million of downloads per month on npmjs.com!
This being an open source project means you have no obligations no to no one; you can do whatever you want to, period. All I can do is urge you to don't just abandon the project like this, don't just call it quits and walk away, leaving thousands of developers up the creek without a paddle...
You have already decided that you need to move on and I'm not arguing with that. What rubs me the wrong way is that you're leaving without handing the project over to a new team of maintainers. Sure, anyone could fork the project and keep going but it would be a lot easier if you let someone else take over as a maintainer.
Here's to hoping that this project will find a new maintainer... 🍺
Thanks for all your hard work, we're sad to see you go ☺

@icfantv I humbly offer some constructive criticism, which I hope does not come across as harsh.
TL;DR: Acknowledge that the current situation isn't ideal, and revise the project status notice in the README to include a call for contributors and maintainers.
The optics of the current situation, fair or not, paint the picture that you (the current maintainers) are not terribly interested in handing off this project or seeing bugs get fixed.  Giving you the benefit of the doubt, I assume that's not actually what you want, so let me explain why it looks that way.
On the one hand, you are saying that you want to be very discriminating about who you add as a contributor/maintainer.  This makes sense, and I'm sure is rooted in the desire for this project to continue to succeed.  You don't want quality and community support to fall off because of poor leadership.  I get that, and I get why you want to be careful.
On the other hand, the result is that the project has had a long period without active leadership or progress.  Plus, the community has had no way to gauge how long it will take before the someone else is allowed to take leadership and start making progress again.  That's bad for the project too.
The fact that your responses here don't acknowledge the downside of having no active leadership or progress implies (whether true or not) that you don't understand that downside.  It would help a lot with appearances if you simply said something like "We realize the current situation is not ideal and would very much like to see this project flourish under someone else's leadership.", especially when discussing your standards for adding someone as a contributor.  For instance, it would have provided an entirely different context if you had said that and then "However," before continuing with "for us to add anyone as a contributor...".
I see the call for contributors and maintainers from back in January, but please realize that right now that issue is at the very end of page 5 out of 11 pages of open issues.  Unless someone is actually looking for that or just happens to get lucky with a search term, no one will see it.  Based on the dates on the subsequent open issues, that issue got bumped off page 1 over 8 months ago, so it's been out of easy sight for quite a while.  Even worse, the project status notice at the top of the README says "this project is considered feature-complete and is no longer being maintained".  The way that reads, it sounds like you (the current maintainers) don't think there is even a need for bug or documentation fixes, and strongly implies you are NOT in fact seeking any additional contributors/maintainers.  This creates the appearance, fair or not, that you're either unaware that there is room for bugfixes and other improvements, or that you just don't care whether or the project gets them.  And that appearance is what leads to calls for project forks, like this one.
I strongly suggest someone revise the project status notice to state clearly that there IS a need for maintenance and bugfixes, and that you are seeking experienced OSS contributors/maintainers.  Maybe even put a link to that January issue so that anyone interested can easily speak up.  It would go a long way toward improving appearances, not to mention improving your chances of having someone step up to take over the project.
I am 100% willing to flesh out and PR the README changes I just suggested, if you think it will actually get reviewed and merged.

@Foxandxss, @pkozlowski-opensource, @wesleycho, you guys seem to be the owners, the main contributors and the publishers of this repo, please help us continue maintaining this repo!

I want to echo the same sentiments as @zacronos - if you're interested in this project not just dying on the vine, then a call for maintainers and contributors needed to happen months ago to get it passed off. This project - ui-bootstrap is not feature-complete, and in fact has many outstanding bugs, even taking out of consideration Bootstrap 4 changes, which are legion (and I wouldn't expect to be tackled if you're passing off the maintenance as it is).
If you're not interested in handing off this repo, I suggest both the steps that @zacronos calls for, AND that you call for, and adopt an official fork that you forward people to in the README, instead of just telling current users of the repo, and newcomers, quite incorrectly, that the project is bug-free with Angular 1.x and Bootstrap 3.x.

This should be part of a case-study in how not to bail on a popular repo. There have certainly been others, so this isn't to single out this one project. It should be part of a larger general discussion. This is like the dark-side of open-source (probably another dark-side is being a maintainer). Is there a package.json value that 'npm install' or 'npm outdated' could pick up on to give a heads up statuses like 'call-for-maintainers', 'deprecated', 'abandoned', etc.

Any plans to support AngularJS 1.7?

@stevenvachon Angular decided to revert the breaking change in version 1.7.1 with v1.7.2. Just make sure you update to the latest version and you should be all good.
https://github.com/angular/angular.js/issues/16594#issuecomment-396208825

So, any "canonical" fork to contribute to?

Here's a continuation of this project that works pretty well.  It has a couple bugs that could probably be handled pretty well if there were some more people supporting it.  It's been updated to use Bootstrap 4:
https://github.com/Morgul/ui-bootstrap4

@icfantv would you agree to mark this repository as archived ?
There has not been any commits for ~1.5 year, and it would make it even clearer that this project is no longer maintained.
I can do that, if you agree.

@PowerKiKi TBH, I'm not really sure it's necessary as the number of new PRs and Issues filed at this point is virtually non-existent (and the closing of all open issues/tickets feels heavy-handed to me).
Also, I don't want to push out a new release that only contains an updated README just to mention the project is being archived since the last time we released, all we did was update the README to say we weren't working on the project anymore.

Publishing a new release isn't required to archive a repo. Actually since the README already contains this kind of info, I wouldn't even touch it at all. So it really just a click on a button to convey without any doubt the status of the project. But that's up to you. I just wanted to make sure that you were aware of the possibility.

Oh, ok.  I'll ping the other two devs to see if they care and if they don't (which they probably won't), we can archive.  Thanks.


Animated Logo

I tried to make a animated logo, kind of a gif if that's ok..

Woow this looks incredible! But better name the file as reddymahendra52.gif so that we can distinguish between the submissions of everyone.

I made the required changes


Fix render layers

This is a rebased version of #49 which looks like it only needed merge conflicts resolved

Hey Chris! Thanks for the assist. I'll take a look shortly.

This almost works. I'll probably commit to this branch if that's ok with you, Chris. This is a good base.

@simbleau totally cool with me!

This is stickier than I thought. See #58 for additional discussion.
Nonetheless, I'm making progress.

@ChristopherBiscardi
Render layers have been obeyed, there's a new render_layers example that uses Gizmos and render layers as well. Feel free to bend it to your will and let me know if behavior is as you expect.

awesome! thank you! I'll let you know how it goes.


Still up to date?

Dear developer,
is this repo still up-to-date or not?

@theobouwman it sure is, everything still works as advertised. I'm using this in a few (unfortunately still closed-source) projects.


Update site-recovery-deploy-source.md

added note for general linux extension requirement

#sign-off


feat(playwright)/new api

Motivation/Description of the PR
The reason for the PR is the Integration of the new Playwright API into CodeceptJS and the bumping Playwright version to 1.32.3.
A list of the new Playwright API, which is important to the CodeceptJS framework, but has not yet been included:

Added in v1.28 of Playwright

.blur()
.clear()

Added in v1.14 of Playwright

.focus()

As part of this PR added implementations of these methods in Codeceptjs.

Bringing the name of options to a general form: opts vs options

Added tests, HTML forms for tests and documentation:

I.focus()
I.blur()
I.clear()  | old I.clearField() In fact, didn't clean the field, but replaced its value with an empty string. It doesn't always work. So it's better to use the new Playwright API (.clear) for this. I.clearField() marked as deprecated.

Applicable helpers:

[X] Playwright

Type of change

[X] :rocket: New functionality
[X] :nail_care: Polish code

Checklist:

[X] Tests have been added
[X] Documentation has been added (Run npm run docs)
[X] Lint checking (Run npm run lint)
[X] Local tests are passed (Run npm test)

Thank you so much!


add unifacs

@rafaleite This request is a duplicate to already approved #8938. It is identical, thus I merge it. Thank you.


Allow the user to set highlight without %nick%

My nick is dan on some networks - including german networks. dann is a common word in german so I get push notifications A LOT because %nick% highlights on *dan*.
I made %nick% the default highlight value and now the user can change it without having %nick% part of the value. This means I can do _%nick% _%nick%: or something like that. IMO this is a nicer way of handling this anyway - what else are default values for?

also +1

Would setting _dan list other keywords here -dan to highlight help?

My understanding is that %nick% is automatically appended to the list, so I think ending your list with -%nick% will nullify it.

Can you update the readme to document the network_blacklist config?

@jreese I already did that :scream:

Yeah, just realized this pull request includes parts that were already merged elsewhere.  Which commits still need to be reviewed/merged?  Just the first %nick% change?

@jreese IIRC this removes the part that auto-adds %nick% to the list and makes it a default parameter instead (so you can override that setting)

@omnidan Can you clean up this PR branch so only the relevant commit(s) will be merged?
I'd be happy to do it myself, but 1) this PR is so old it predates the GitHub feature allowing maintainers to edit the source branch, and 2) even if that option was enabled, I don't really want to mess with someone else's master without permission. :smile_cat:


parser: improve index merge hint

What problem does this PR solve?
Schema
create table t3(a int primary, b int, c int);
create index t3b on t3(b);
create index t3c on t3(c);
SQL
(1) select /*+ use_index_merge(t3 primary, t3b, t3c) */ * from t3 where a < 1 or b < 2 or c < 3;
(2) select /*+ use_index_merge(t3, primary, t3b, t3c) */ * from t3 where a < 1 or b < 2 or c < 3;
(3) select /*+ use_index_merge(t3, t3b, primary, t3c) */ * from t3 where a < 1 or b < 2 or c < 3;
(4) select /*+ use_index_merge(t3, t3b, t3c, primary) */ * from t3 where a < 1 or b < 2 or c < 3;
The first one is right. The (2)-(4) will throw warnings in grammar.
What is changed and how it works?
add a match branch for indexNameList
Check List
Tests

Unit test

Code changes
gram.y
Side effects
Related changes

(Trying to refresh CLA status)

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Hi @hailanwhu, would you mind signing the CLA in https://github.com/pingcap/parser/pull/590#issuecomment-547255713? Thanks!

I sign it.


Make outline consistent for all focusable elements in the APP across platforms.

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Open any convo on E.cash on the web.
Try to use the tab for navigation.
See multiple elements get focus with an outline that is different across the platforms.

Expected Result:

same Color of outline for each platform.

Actual Result:
Describe what actually happened

Web(chrome) shows black solid.
Web Firebox shows dotted line.
Desktop has a yellow border.
ETC...

Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Just a UI thing.
Platform:
Where is this issue occurring?
Web :heavy_check_mark:
iOS
Android
Desktop App :heavy_check_mark:
Mobile Web
Version Number: Latest
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation

Expensify/Expensify Issue URL:
View all open jobs on Upwork
view this job on Upwork
cc: @shawnborton

Triggered auto assignment to @stephanieelliott (AutoAssignerTriage), see https://stackoverflow.com/c/expensify/questions/4749 for more details.

Was able to demo this by following the reproduction steps 👍

Triggered auto assignment to @ctkochan22 (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

Triggered auto assignment to @jboniface (External), see https://stackoverflow.com/c/expensify/questions/8582 for more details.

To fix this easiest way is to override the global Outline style so that it reflects on each platform.
Add Global CSS style to index.html
*::focus{
	outline: 1px solid auto;
}

or we can use box-shadow as bootstrap uses for the blurred outline.
If don't want to use the global style which I am aware of, the best option to do is to disable the default outline and  only apply to the specific elements with
addOutlineWidth()

I would say its better to handle the web-specific global styles as a global stylesheet. We can call this browser-reset. They are used to reset the browser-applied styles which we don't want.

Triggered auto assignment to @tgolen (Exported), see https://stackoverflow.com/c/expensify/questions/7972 for more details.

@tgolen please check Rajat's proposal when you get a chance!

I'm OK with the CSS route. Is that going to make sure that android and ios are consistent too?

Yeah. I think Android or IOS does not have an outline for focused elements. And for those elements which we want to show outline, are already configured to show correct one using addOutlineWidth. But I will double-check this.

Hm, OK. That sounds like things would still not be consistent then across platforms, since ALL web elements would get the outline, but android/ios would have it only for specific elements with addOutlineWidth.

So maybe using addOutlineWidth across all platforms would be the most consistent approach then.

No so Global styles would be overwritten by addOutlineWidth. There are many elements on the Web that are not configured to show outline but the browser shows a default outline. So it's better to tackle them via Global style and native devices do not have default outlines so no elements show outline unless added.

OK, but with your proposal above, it would ADD a default outline to everything in web. Shouldn't the global style be removing the outline from everything by default, and then addOutlineWidth would add it to the elements necessary? I guess I'm expecting the global style to be:
*::focus{
	outline: 0px;
}

I am not sure if disabling the outline is a good idea. It's better to pull the design team for that. We just closed one of the issues https://github.com/Expensify/App/issues/3041 like that.
Yeah, this style will be used to hide them but if we want to hide.

Yeah, this style will be used to hide them but if we want to hide.

Sorry, I'm not quite sure what you mean here. Can you please restate that?

I mean,
*::focus{
	outline: 0px;
}

can be used to hide, but only if we want to hide outlines.

Ah right, gotcha. So, I think if the problem is "focus styles aren't consistent across platforms", then on web, hiding all outlines by default would fix that problem. Right?

Late to the convo here, but hiding the outline styles might be a bad idea for accessibility right?

Yes, hiding outlines is bad.

OK, yes, that probably is. But what I'm trying to say is that while the problem that this issue describes is solved by the proposal, it also opens up a brand new issue where focus styles are still inconsistent between platforms. Because in web, any focused element will have an outline now, whereas android/ios will only get outlines for ones that are configured to have outlines.

We already have outlined on WEB but not on Native devices. And you want to fix this problem thus remove outlines on the web. Keeping selected elements aside.
I am saying we set the color correctly as the outline is important. There are few things the web handles differently.

We already have outlined on WEB

OK, I think this is the part I didn't realize. If web already has a default focus for everything, then at least we aren't creating a new problem. In that case, making the colors consistent is at least a step in the right direction.
🟢

@shawnborton What would be the outline color?

I suppose we could make the outline color the same color that we use for :focus on inputs, which is our brand blue?
That being said, a lot of products have different outline colors based on the element. Take GitHub for example:

Anyways, I think the simplest thing to do is use our blue color for now and then we can revisit individual components if we want to override the default blue behavior. How does that sound?

Yes. Those individual elements already reflect what is assigned to them.

reminder to self to pay bonus for this (if i fail to do this let me know, raj)

@jboniface PR is merged, Reopening for payment

@parasharrajat it looks like this PR introduced a regression.
On commit f99dea6bf I see this:

@Julesssss  This is not a regression from this PR. I fixed the outline styles for the whole app which was wrong. If any other input in the app is showing this effect and this is a regression from this PR. but all other inputs are working fine.
I just checked that that Component is using overflow:hidden which will hide every outline which I verified by reverting my changes.

Ah, I see. Thanks for explaining! Yep that would be great, but I'll discuss further on the raised issue.

@tgolen, @parasharrajat, @jboniface it looks like no one is assigned to work on this job.
Please double the price or add a comment stating why the job isn't being doubled.

Ding Ding. It's done.

paid

@tgolen @jboniface  Be sure to fill out the Contact List!


[TYPO] LocalesApi.accountLocalesRaw()

There's a typo in the line 137 of the LocalesApi file
Where "account_id" should be "id" since it's the one defined in the parameters.
const response = await this.request({
      path: `/accounts/{account_id}/locales`.replace(`{${"id"}}`, encodeURIComponent(String(requestParameters.id))), // HERE
      method: 'GET',
      headers: headerParameters,
      query: queryParameters,
  });

Since you are using the OpenAPI Generator, the error must come from the definitions of the backend for the interface of the arguments, most likely.

Hi @Paul7Peterson, thanks for reporting this. This looks indeed incorrect and I'll try to adapt this

I've just released version 1.5.2, which should fix this


I can't play DRM protected HLS contents on both Big Sur 11.3 & iOS 14.4.2

Have you read the FAQ and checked for duplicate open issues?
YES
What version of Shaka Player are you using?
3.0.10
Can you reproduce the issue with our latest release version?
Yes
Can you reproduce the issue with the latest code from master?
Yes
Are you using the demo app or your own custom app?
our own custom app
If custom app, can you reproduce the issue using our demo app?
yess
What browser and OS are you using?
Safari Version 14.1 (166<IP_ADDRESS>.3)
mac OS Big Sur 11.3
iOS 14.4.2
For embedded devices (smart TVs, etc.), what model and firmware version are you using?
What are the manifest and license server URIs?

What configuration are you using?  What is the output of player.getConfiguration()?

DRM Configuration
What did you do?

load manifest
What did you expect to happen?
I was expecting it to play the video
What actually happened?

Video playback fails with { code: 3016 } error
It is working without any problem on Chrome (widevine). Our partners can stream the content on their side but we can't, on Macbook or iPhone...

Received more details via email.  @michellezhuogg, please look for that during your triage rounds this week.

@clotairemarmignon Hi.
I have the same problem as yours.
Is that another way to resolve it?


Pause current playing

Feature Reuqest:
When hitting "Play to Kodi" the current playing Youtube Video in Chrome should be paused.
Keep up the good work :)

Sound reasonable. Will look into it.

Sounds great. Looking forward to it :)
Might have a look here: https://developers.google.com/youtube/js_api_reference?csw=1
player.pauseVideo():Void
Let me know when you updated it, I'm happy to test.

When hitting Play now on Youtube the new option doesn't work "Videos on YouTube, Vessel and AnimeLab will auto pause when sent to Kodi."

Sorry, this only works when using the context menu for YouTube at the
moment. Which is why I haven't mark this as compete. My bad I forgot to
point this out in the changelog.
On Mon, 15 Jun 2015 1:15 am Andulien<EMAIL_ADDRESS>wrote:

When hitting Play now on Youtube the new option doesn't work "Videos on
YouTube, Vessel and AnimeLab will auto pause when sent to Kodi."
—
Reply to this email directly or view it on GitHub
https://github.com/khloke/play-to-xbmc-chrome/issues/56#issuecomment-111840680
.

Commit: 89a9ae3
Working with the newest Version <IP_ADDRESS>. Just tested it :+1:
When hitting PlayNow on the Chrome Icon, it will pause the current playing tab.
Question: Should this also be added for Queued and PlayNext?

I intend to make this work with Queue only when there is nothing playing at
the moment and it starts playing but not when adding to a queue. As for
play next, I won't make it pause for the same reason as queuing.  I think
it makes more sense that way.
On Mon, 15 Jun 2015 11:41 pm Andulien<EMAIL_ADDRESS>wrote:

Commit: 89a9ae3
https://github.com/khloke/play-to-xbmc-chrome/commit/89a9ae378e989ca7222feca944c7d6a2e771ecd8
Working with the newest Version <IP_ADDRESS>. Just tested it [image: :+1:]
When hitting PlayNow on the Chrome Icon, it will pause the current playing
tab.
Question: Should this also be added for Queued and PlayNext?
—
Reply to this email directly or view it on GitHub
https://github.com/khloke/play-to-xbmc-chrome/issues/56#issuecomment-112074390
.


Make SVE-enabled ARM64 targets in DYNAMIC_ARCH builds depend on a relatively recent compiler

fixes #3874 (fixes #3805)

Sorry for the mess, visiting family and tried to sneak in some discreet coding via the gh web editor. Will clean up ASAP

Sorry for the mess, visiting family and tried to sneak in some discreet coding via the gh web editor. Will clean up ASAP

No worries, just here to help 😸

This has now uncovered a latent bug in the MSA support detection/handling for MIPS. To be continued, hopefully later today

mips msa fixes to go into a separate PR, merging to get this out if the way


How to disable privacy pop-up boxes in non-EU areas

MAX Plugin Version
12.1.0
Flutter Version
3.6.0
Device/Platform Info
Android，iOS
Current Behavior
An ugly privacy agreement pop-up box is displayed in the non-EU area, what can I do to get rid of it?

Expected Behavior
An ugly privacy agreement pop-up box is displayed in the non-EU area, what can I do to get rid of it?
How to Reproduce
When using the SDK in a non-EU area, an ugly privacy agreement pop-up box will be displayed.
Additional Info
No response

My Code:
    if (privacyPolicyUrl != "") {         AppLovinMAX.setTermsAndPrivacyPolicyFlowEnabled(true);         AppLovinMAX.setPrivacyPolicyUrl(privacyPolicyUrl);       }       if (termsOfServiceUrl != "") {         AppLovinMAX.setTermsOfServiceUrl(termsOfServiceUrl);       }       final maxResult = await AppLovinMAX.initialize(adsServiceConfig.maxKey);

@Yibei-ayuo, those privacy and term settings are totally optional.  You don't have to have it.


feat(bb.js): Use an environment variable to set the transcript URL

Related to #1749
Checklist:
Remove the checklist to signal you've completed it. Enable auto-merge if the PR is ready to merge.

[ ] If the pull request requires a cryptography review (e.g. cryptographic algorithm implementations) I have added the 'crypto' tag.
[ ] I have reviewed my diff in github, line by line and removed unexpected formatting changes, testing logs, or commented-out code.
[ ] Every change is related to the PR description.
[ ] I have linked this pull request to relevant issues (if any exist).

This does not fully resolve the issue since we have a NodeJS CLI


pythonPackages.impacket: 0.9.22 -> 0.9.23

Motivation for this change
Fixes CVE-2021-31800.
https://github.com/SecureAuthCorp/impacket/releases/tag/impacket_0_9_23
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[x] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review run on x86_64-linux 1

6 packages built:

enum4linux-ng
python38Packages.impacket
python38Packages.lsassy
python38Packages.patator
python39Packages.impacket
python39Packages.lsassy

Result of nixpkgs-review pr 127449 at 31c39c11 run on aarch64-linux 1

5 packages built successfully:

enum4linux-ng
python38Packages.impacket
python38Packages.lsassy
python39Packages.impacket
python39Packages.lsassy


Victor VosMottor: Watch dustandsplinters(?:@gmail.com)?

Victor VosMottor requests the watch of the watch_keyword dustandsplinters(?:@gmail\.com)?. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

dustandsplinters(?:@gmail\.com)? has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Spevacus in Charcoal HQ


Dataloader shuffle is not reproducible.

🐛 Bug
Dataloader shuffle is not reproducible. As far as I know dataloader in pytorch is reproducible if you set the seed. However, in pytorch geometric in each start the results are different using the same seed.
To Reproduce
In dataloader put the argument shuffle to true

Expected behavior

Environment

OS: Ubuntu 18
Python version: 3.6
PyTorch version: 1.1
CUDA/cuDNN version: 10
GCC version: 7.3
Any other relevant information:

Actually, this is not reproducible for me, and I do not see a reason what should prevent this from happening (as we simply use a different collate function and call the internal PyTorch DataLoader), e.g.:
torch.manual_seed(12345)

loader = DataLoader(dataset, batch_size=1, shuffle=True)
for data in loader:
    print(data)

I tried your example and it is working.. using modelnet dataset. I will double check my dataset, maybe I am doing something wrong...
Thanks and sorry for the inconvenience.

I checked again, and the dataloader and dataset are not the problem... I found that they are generating the same results in each execution.. However, in each execution I am obtaining different results... What are the random libraries used in pytorch_geometric? At this moment I am seeding numpy, torch, torch.cuda and random libraries. Am I missing something?

See here :(

Oh I see! I always forget about this behaviour on gpu. However I do not understand the reason of that. In my case, I am using the NNConv, the operations done for this conv are mult and sums in essence. Why is it affecting the execution order to the results?  In addition, if I am using the the same gpu ... the floating point precision should be same, right? Then I do not understand why is it happening this  :( I suppose I am not taking into account something.
Thanks for your time!!

The problem is the aggregation, because the order of neighbors is not determined on GPU. Hence features can be summed up in different oder, even for the same GPU.

Oh, you mean that the updated feature of one node is used in some aggregations and others the old state is used, right?

No, I mean that for each node, we aggregate neighboring node features in a random order on the GPU., which may result in small numerical instabilities.


pass RST_STREAM and MAX_STREAM_DATA directly to the stream

Depends on #1011 (and probably more, but I lost track of that).
@lucas-clemente: Just the last 3 commits need to be reviewed, the rest belongs to other PRs.

Codecov Report

Merging #1015 into master will decrease coverage by <.01%.
The diff coverage is 94.74%.

@@            Coverage Diff            @@
##           master   #1015      +/-   ##
=========================================
- Coverage   90.51%   90.5%   -<.01%
=========================================
  Files         102     103       +1
  Lines        7067    7085      +18
=========================================
+ Hits         6396    6412      +16
- Misses        433     435       +2
  Partials      238     238

Impacted Files
Coverage Δ

server.go
89.33% <ø> (ø)
:arrow_up:

stream_framer.go
97.4% <100%> (+1.95%)
:arrow_up:

packet_packer.go
85.45% <100%> (+0.14%)
:arrow_up:

crypto_stream.go
100% <100%> (ø)

stream.go
94.07% <88%> (-1.56%)
:arrow_down:

session.go
84.67% <95.45%> (+0.43%)
:arrow_up:

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 489ea7f...a753285. Read the comment docs.

We have coupling between stream and wire already (in AddStreamFrame), and another PR will add PopStreamFrame. I found myself looking up session when RegisterRemoteError is actually called, so I thought it's easier to have the stream handle all frames that effect its state machine.

I'm not convinced this improves things, but feel free to merge anyway.

I'm not 100% convinced as well, but it will make the further refactoring easier for now, so I'll merge it. It's internal API, so we can easily change it back later.


Set tags=[manual] for *Local targets, thus exclude them from the wildcard target patterns

This should fix #4.


install open-ce in action

Checklist before submitting

[x] Did you read the contributor guide?
[ ] Did you update any affected documentation?
[ ] Did you write any tests to validate this change?

Description
pip install open-ce to fix import failures.
Review process to land

All tests and other checks must succeed.
At least one maintainer must review and approve.
If any  maintainer requests changes, they must be addressed.

@cdeepali I'm merging this in so the PRs for feedstocks aren't broken anymore.


Update "app.name" property for openshift extension  defined in arquillian.xml for any booster while creating project

Why?
Boosters have a special profile which defines system property called app.name set for integration tests execution (failsafe). This is customizable, as the user can name his project anyhow he wants while using quickstart wizard.
The same property is then used as a service name when injecting @RouteUrl in the integration test itself. This however, does not work in Che as the property is not recognized.
By assuming boosters have app.name defined in arquillian.xml something like
 <extension qualifier="openshift">
    <property name="app.name">project_artifact</property> // to make it workable by default in all
     boosters
   </extension>

If launcher-backend is able to update project artifact in arquillian.xml while creating project, then cube can resolve @RouteUrl in the integration test from che as well once https://github.com/arquillian/arquillian-cube/issues/996 has been fixed.
Changes proposed in PR

Update of <property name="app.name">project_artifact</property> to <property name="app.name">user_input</property> for openshift extension
Adds unit tests

Can one of the admins verify this patch?

@dipak-pawar can you squash your commits into a single one and rebase with the master branch?

Ok to test


Add Star Rating for dishes

Feature
We already have it in the model, so it's time to display some nice stars
URL References

https://github.com/BioPhoton/angular-star-rating

done


Level editor now shows new maps when you press +

Currently the level editor doesn't update to show when you add new world/level maps. This fixes it.

ARG. I can't believe I left a debugging print statement in there. I am so bad at this. I'm going to delete this and resubmit it.


Recursively list all type parameters when getting TSType in macro

Currently, type for which the TSType must be generated cannot be used without defining implicits for them:
case class A(field: String)

// Error: Missing implicit for default TSType[Seq[ListObj]]. Bring it in scope with `import DefaultTSTypes._` or `extends DefaultTSTypes`
implicitly[TSType[Seq[A]]]

This is because even though an implicit conversion for TStype[A] => TSType[Seq[A]] is available by default, and TSType[A] can be generated with the macro, the implicit conversion does not apply because TSType[A] is not yet in scope
To fix this, instead of using getOrGenerate[Seq[A]] in the macro, if we encounter a type with generic parameters we first make implicits for the generic arguments:
implicit val tparam0 = getOrGenerate[A]
// Now that TSType[A] is in scope, the implicit conversion for TSType[A] => TSType[Seq[A]] applies
getOrGenerate[Seq[A]]

However, this needs to be done recursively, otherwise we run into the same problem for a type requiring multiple conversions like Seq[Seq[A]]:
implicit val tparam0 = getOrGenerate[A]
implicit val tparam1 = getOrGenerate[Seq[A]] // A => Seq[A] will apply
getOrGenerate[Seq[Seq[A]] // Seq[A] => Seq[Seq[A]] will apply

Fixes #176

Note: this has the potential to ruin performance, might need to start caching TSTypes


Doesn't work if there is no main branch

docker run -it --rm -e GIT_URL=https://github.com/vercel/next.js ghcr.io/coder/envbuilder:0.0.2

envbuilder - Build development environments from repositories in a container
#1: 📦 Cloning https://github.com/vercel/next.js to /workspaces/next.js...
Error: clone "https://github.com/vercel/next.js": couldn't find remote ref "refs/heads/main"
error: clone "https://github.com/vercel/next.js": couldn't find remote ref "refs/heads/main"

This is fixed in the latest version!


[Video] remove three dots / disable the current behavior which is buggy

The videos have a button with three dots that shows up at the bottom right.
Currently, when you click that button, the sidebar resets to the top (and the video stops playing).
We should either remove the button (seems possible: https://stackoverflow.com/questions/54507905/html5-video-player-controls-in-chrome-three-dots-on-the-right-open-blank-scree), or make it so that clicking the button does not do anything.

    disablepictureinpicture
    controlslist="nodownload noplaybackrate"


Add auto-install

Auto installs dependencies as you code.
Source:
https://github.com/siddharthkp/auto-install

Incorrect section. You want the CLI apps section.

@sindresorhus Sorry, honest mistake.


[AutoPR web/resource-manager] Add Microsoft.Web ResourceHealthMetadata swagger

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/2588

(message created by the CI based on PR content)

Codecov Report

Merging #2097 into restapi_auto_web/resource-manager will decrease coverage by 0.04%.
The diff coverage is 23.45%.

@@                          Coverage Diff                          @@
##           restapi_auto_web/resource-manager    #2097      +/-   ##
=====================================================================
- Coverage                              53.05%   53.01%   -0.05%
=====================================================================
  Files                                   4775     4778       +3
  Lines                                 119534   119696     +162
=====================================================================
+ Hits                                   63419    63457      +38
- Misses                                 56115    56239     +124

Impacted Files
Coverage Δ

azure-mgmt-web/azure/mgmt/web/models/__init__.py
100% <100%> (ø)
:arrow_up:

...ure-mgmt-web/azure/mgmt/web/operations/__init__.py
100% <100%> (ø)
:arrow_up:

...t-web/azure/mgmt/web/web_site_management_client.py
23.45% <100%> (+0.41%)
:arrow_up:

.../operations/resource_health_metadata_operations.py
16.66% <16.66%> (ø)

.../azure/mgmt/web/models/resource_health_metadata.py
62.5% <62.5%> (ø)

.../mgmt/web/models/resource_health_metadata_paged.py
80% <80%> (ø)

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 1d59a7a...d2bf4b7. Read the comment docs.


More refinement to core requested keys

Description
As the channel, locale and customer_group are the main keys of the core portions. So added some method which will handle the fallback also, so that developers want needs to worry about the fallback from the request.
Code additions

Added some methods which will fetch requested local, channel, and customer group.
Optional fallback parameter is given, if the developer doesn't want to fallback they can pass false as the argument.

Marking this as a draft, till merging of this PR,
https://github.com/bagisto/bagisto/pull/4915


Fix incorrect link

Link is pointing to incorrect source

@drewbatgit : PR passed acceptance checklist. Please evaluate the proposed changes further and merge if appropriate. Thanks!

This is a duplicate of change 1048 and is causing merge conflicts.

@drewbatgit
There where two links that was swapped. This addressed one of them. The other was addressed in another PR that was merged, and therefore this one causes a merge conflict. The link corrected in this PR is still wrong