Change for Streamlit 1.27.0

change st.exprimental_rerun to st.rerun

thanks for the PR, the alert quite spammy 😅 i've merged it into v0.1.6 and added a conditional import to also support previous streamlit versions


CLI: Give helpful error message if --http/--tls is given without colon

Fixes #134

Could you re-write the error strings to be more in-line with the other strings?
Something like The --http switch only accepts interface:port or :port for its argument.

Done :)


RPRBLND-1798: Fix list of MaterialX node trees in Material

PURPOSE
MaterialX node trees selection menu in Material panel shows all node trees including USD node trees and grouped node trees.
EFFECT OF CHANGE
Fixed list of MaterialX node trees in selection menu in Material panel.
TECHNICAL STEPS

Created operators and menu: HDUSD_MATERIAL_OP_link_mx_node_tree, HDUSD_MATERIAL_OP_unlink_mx_node_tree, HDUSD_MATERIAL_MT_mx_node_tree.
Added menu and operators to HDUSD_MATERIAL_PT_material panle istead of template_ID.

Tested. Works ok.


Add question feature

What this PR does

add question service
add question validation
add question model
add question controller
add question route

good job


Implement or scrap "Gfx.resizeimage"

Currently broken.
(or maybe: scrap it? It's a slightly confusing concept!)

 This was also kinda nice in Zeedonk, where you almost always want to scale images.
Before the starling upgrade, this was a substantial optimization for some programs - scaling was expensive (it's not anymore).

On the con side:

It's confusing. One early tester of haxegon used Gfx.resizeimage() instead of Gfx.scale() because they didn't really understand the difference, which is fair enough.

On balance, I think it's better to just scrap it.


Example does not work out of the box: wrong API calls

There are plenty of errors in the published code.
For example:
everywhere:
$envelope_api = new \DocuSign\eSign\Api\EnvelopesApi($api_client);
should be:
$envelope_api = new \DocuSign\eSign\Client\Api\EnvelopesApi($api_client);

@mariobig  can you tell me which version of the SDK and the API you are using?

Hi ,
I followed step by step installation instruction published  at https://github.com/docusign/eg-03-php-auth-code-grant.
Mario

I understand, but can you still try to find the versions and let us know?

In my opinion it has nothing to do with versions of SDK or API. Official DocuSign SDK is correct and up to date.
Simply, the code itself published here in examples contains a lot of bugs:
Examples:
src/EG001EmbeddedSigning.php
line 109: $api_client = new \DocuSign\eSign\ApiClient($config);
should be: $api_client = new \DocuSign\eSign\Client\ApiClient($config);
src/EG002SigningViaEmail.php
line 109: $api_client = new \DocuSign\eSign\ApiClient($config);
should be: $api_client = new \DocuSign\eSign\Client\ApiClient($config);
src/EG008CreateTemplate.php
line 94: $api_client = new \DocuSign\eSign\ApiClient($config);
should be: $api_client = new \DocuSign\eSign\Client\ApiClient($config);
src/EG003ListEnvelopes.php:	does not work
src/EG006ListEnvelopes.php:	does not work
src/EG007ListEnvelopes.php:	does not work (same bug like in EG001EmbeddedSigning.php and src/EG002SigningViaEmail.php)
src/EG009ListEnvelopes.php:	does not work beacuse example 8 does not return template ID
src/EG016ListEnvelopes.php:	does not work (line 112 same bug like in EG001EmbeddedSigning.php and src/EG002SigningViaEmail.php)
src/EG018ListEnvelopes.php:	does not work (same bug like in EG001EmbeddedSigning.php and src/EG002SigningViaEmail.php and others bugs as well)
src/EG019ListEnvelopes.php:	does not work (error in eg-03-php-auth/vendor/twig/twig/src/Loader/FilesystemLoader.php on line 250)

Thank you @mariobig for the bug report. You're correct, the example should work properly out of the box.
I have filed bug DEVDOCS-1710

Confirmed fixed, closed.
https://github.com/docusign/eg-03-php-auth-code-grant/blob/master/src/Services/ClientService.php#L11


devbox.findPackagesByName should be constructed from Config packages

Summary
This PR refactors devbox.findPackagesByName to be constructed from devconfig.Package.
Motivation
devbox.Update calls findPackageByName and has special handling for pkg.IsLegacy.
The implementation of pkg.IsLegacy (next PR #1358) is dependent on isInstallable,
which is provided when devpkg.Package is constructed from devconfig.Package.
Previously, devpkg.Package was constructed via PackagesFromString where
isInstallable would default to true.
How was it tested?
compiles.
In #1358, I do devbox update with a package that is excluded from my system,
and previously the "update" implementatation would go into the IsLegacy code path.

Current dependencies on/for this PR:

main

PR #1357

PR #1360   👈

PR #1358

This comment was auto-generated by Graphite.

All tests pass. Going to land the parent PR, rebase and force-land.


Fix NPE issue when no ApplicationInsights.xml found

Codecov Report

Merging #702 into develop will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #702   +/-   ##
========================================
  Coverage    33.37%   33.37%
========================================
  Files           83       83
  Lines         3200     3200
  Branches       483      483
========================================
  Hits          1068     1068
  Misses        2052     2052
  Partials        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 3cd20f1...d8228b6. Read the comment docs.


Activate IPv6 in CentOS 7 AMI

High Level Description
IPv6 currently deactivated in the CentOS 7 AMI. This PR makes the AMI run dual stack.
Related Issues

DCOS-15500 Verify that all DC/OS AMI have dual stack enabled.

@mesosphere-mergebot label Work In Progress

@mesosphere-mergebot bump-ee

@mesosphere-mergebot label Ready For Review

I tested the new AMI in an IPv6 enabled cluster. I was able to start containers and do all the usual operations. The master and agent nodes had routed v6 addresses and were able to e.g. ping6 google.com. For the private agents I had manually added an Egress only gateway to the VPC and adjusted routing tables and security policies accordingly.

@lloesche @lingmann @vespian FYI these images for cloud integration tests are now pinned here: https://github.com/mesosphere/dcos-test-utils/blob/master/dcos_test_utils/aws.py


Support Other Than Amazon Hosts ?

Just tried to use digital ocean's spaces but it failed:
s4cmd ls
[Runtime Exception] An error occurred (InvalidAccessKeyId) when calling the ListBuckets operation: The AWS Access Key Id you provided does not exist in our records.
Traceback (most recent call last):
  File "/usr/local/bin/s4cmd.py", line 1928, in main
    CommandHandler(opt).run(args)
  File "/usr/local/bin/s4cmd.py", line 1557, in run
    CommandHandler.__dict__[cmd + '_handler'](self, args)
  File "/usr/local/bin/s4cmd.py", line 129, in wrapper
    ret = func(*args, **kargs)
  File "/usr/local/bin/s4cmd.py", line 1628, in ls_handler
    self.pretty_print(self.s3handler().list_buckets())
  File "/usr/local/bin/s4cmd.py", line 129, in wrapper
    ret = func(*args, **kargs)
  File "/usr/local/bin/s4cmd.py", line 694, in list_buckets
    for bucket in self.s3.list_buckets().get('Buckets') or []:
  File "/usr/local/bin/s4cmd.py", line 402, in wrapped_method
    ret = getattr(self.client, method)(*args, **merged_kargs)
  File "/usr/local/lib/python3.7/dist-packages/botocore/client.py", line 357, in _api_call
    return self._make_api_call(operation_name, kwargs)
  File "/usr/local/lib/python3.7/dist-packages/botocore/client.py", line 661, in _make_api_call
    raise error_class(parsed_response, operation_name)
botocore.exceptions.ClientError: An error occurred (InvalidAccessKeyId) when calling the ListBuckets operation: The AWS Access Key Id you provided does not exist in our records.

Are other hosts supported ?

Take a look at the --endpoint-url option. It lets me use s4cmd with an on premises S3 device.

@diginfo
If anyone comes here from google, here is how you can use digitalocean spaces. My ~/.s3cfg was configured fine for s3cmd but s4cmd didn't pick up the correct host.
replace sfo2.digitaloceanspaces.com with your digital ocean spaces host.
s4cmd --endpoint-url sfo2.digitaloceanspaces.com  sync s3://bucket1/ s3://bucket2/

Duplicate of #158.

Unfortunately specifying --endpoint-url like suggested by @skunkworker does not work for me confused
[Runtime Exception] Unable to connect to s3: Invalid endpoint: ams3.digitaloceanspaces.com
Traceback (most recent call last):
  File "/usr/bin/s4cmd", line 687, in connect
    self.s3 = BotoClient(self.opt)
  File "/usr/bin/s4cmd", line 387, in __init__
    self.client = self.boto3.client('s3', endpoint_url=opt.endpoint_url)
  File "/usr/lib/python3/dist-packages/boto3/__init__.py", line 91, in client
    return _get_default_session().client(*args, **kwargs)
  File "/usr/lib/python3/dist-packages/boto3/session.py", line 258, in client
    return self._session.create_client(
  File "/usr/lib/python3/dist-packages/botocore/session.py", line 831, in create_client
    client = client_creator.create_client(
  File "/usr/lib/python3/dist-packages/botocore/client.py", line 83, in create_client
    client_args = self._get_client_args(
  File "/usr/lib/python3/dist-packages/botocore/client.py", line 285, in _get_client_args
    return args_creator.get_client_args(
  File "/usr/lib/python3/dist-packages/botocore/args.py", line 99, in get_client_args
    endpoint = endpoint_creator.create_endpoint(
  File "/usr/lib/python3/dist-packages/botocore/endpoint.py", line 286, in create_endpoint
    raise ValueError("Invalid endpoint: %s" % endpoint_url)
ValueError: Invalid endpoint: ams3.digitaloceanspaces.com

I think you just need https://ams3.digitaloceanspaces.com, I guess you missed the https:// part.


Adding a object detection feature using YOLO v8

Is your feature request related to a problem? Please describe.
We can add a object detection feature using YOLO which is pre-trained and also fine tune it using a custom dataset we have created.
Describe the solution you'd like
This feature will be able to detect objects by surrounding them with boundary boxes and labelling them as a person or wto whichever category it belongs to.
Describe alternatives you've considered
We can also use RCNN based approach where we can use Tensorflow Object Detection and fine tune it with a custom dataset.
Additional context
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

@khusheekapoor please assign this issue to me. I would like to contribute as a GSSOC contributor.

Hi, we have many similar issues. Hence, we cannot assign you this one.


Feature: maximize window

Good day to you!
Suggestion:
ability for keymapping to maximize window (to keep outer gaps)
Thank you!

what? fullscreen exists

fullscreen:

takes whole output
some apps (e.g. chromium) enters in fullscreen-mode, where it's hide tabs

maximize:

takes only workarea, so outer gaps stays visible
apps don't enter in fullscreen mode

ok

added, use
bind=MOD,KEY,fullscreen,1

for maximizing, 0 for the old style.

@vaxerski
almost good, but still it sends application to fullscreen mode, e.g. Brave

after entering fullscreen it hides tab-bar:

ah shoot oops

fixed

it works good.
thank you!


squeak: <IP_ADDRESS>14 -> 5.3-19459

Credit to @BenBals for starting work on this patch in https://github.com/NixOS/nixpkgs/pull/86741. This package probably still needs more testing, but basic inisqueak execution works out of the box.
This patch doesn't use a newer release of the OpenSmalltalk VM because current Squeak doesn't use one. I don't know enough about Squeak to feel comfortable bumping the VM version.
Motivation for this change
inisqueak works in a fresh directory. I like that.
This patch tries to replicate the recommended sorta-bootstrapping arrangement for Squeak, as best as I can tell. It's not super well documented. https://github.com/squeak-smalltalk/squeak-app was a helpful reference.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
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 packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 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.

Everything is broken, I'm not sure if that's just the state of Smalltalk or a problem with the VM.

@ehmry What sorts of broken? This all worked fine during my tests before opening the PR.

Excerpts from bb010g's message of March 29, 2022 2:04 pm:

@ehmry What sorts of broken? This all worked fine during my tests before opening the PR.

I don't understand Smalltalk yet so I'm getting version mismatches from the VM being too new or too old.
For now I'm running a patched binary from the alpha release: https://git.syndicate-lang.org/ehmry/syndicate-flake/src/commit/cecebbc118547c13faaab197b062adf83eb85c1f/squeak/default.nix


react-scripts: command not found after node-sass-chokidar installed

Is this a bug report?
Yes.

Can you reproduce the problem with npm 4.x?
Yes. same

Have you read the Troubleshooting section and searched the User Guide?

Yes
Environment

node -v: 6.10.2
npm -v: 5.0.4 / 4.6.1
yarn --version (if you use Yarn): null
npm ls react-scripts (if you haven’t ejected): No

Then, specify:

Operating system: macOS 10.12.5
Browser and version (if relevant): chrome version 60

Steps to Reproduce

install create-react-app globally
create my app & npm install
install node-sass-chokidar -D
edit 'scripts' part fllowing documentation
npm start

I tried remove node_modules , clear cache and reinstalled twice, it still happened.
And it works before I install node-sass-chokidar
here's npm script:
"scripts": {
    "build-css": "node-sass-chokidar src/ -o src/",
    "watch-css": "npm run build-css && node-sass-chokidar src/ -o src/ --watch --recursive",
    "start-js": "react-scripts start",
    "start": "npm-run-all -p watch-css start-js",
    "build": "npm run build-css && react-scripts build",
    "test": "node scripts/test.js --env=jsdom"
  },

Expected Behavior
App is running.

Actual Behavior
Threw this error

Reproducible Demo
none.

This does sound like an npm 5 bug. I would double check that you can reproduce it with npm 4 or npm 3.
There is a very minor chance that something is wrong in the documentation. However we can't determine that without a reproducing demo.
I'm going to tag this as low priority because the symptoms match npm bugs and there is no demo. If somebody else confirms the issue and creates a demo, we can investigate further, otherwise we'll eventually close the issue.
I would recommend you to either try downgrading npm again (try both 4 and 3), or to try using Yarn instead of npm.
It would also help if you could create a bug report for npm. The symptom is that when you install an unrelated package, node_modules/.bin/react-scripts disappears.
Hope this helps!

When I uninstalled node-sass-chokidar and use original script, it works well.
I dont't know what's the problem is.

At the very least, your reproduction instructions are incomplete. It is definitely not enough to install node-sass-chokidar. The guide also mentions npm install --save-dev npm-run-all since you used the version that includes npm-run-all. This is why we're asking for exact steps to reproduce and a demo: I just spent time on this issue but I still don't know if I ended up with the same setup as yours or not.
After I installed both these packages, I couldn't reproduce the problem. So I'm closing because the documentation is correct, and therefore it is either an npm bug or a mistake in following the documentation. I would have loved to help more, but lack of reproducing demo makes it impossible.
I hope that trying again with a non-buggy version of npm, or with Yarn, will work for you.

@gaearon I am experiencing the same issue. What I found was that after creating a fresh create-react-app project, installing node-sass-chokidar messes with the node_modules/.bin/ folder in that react-scripts is gone. Very weird.

@gaearon I have the same issue here, as well: sh: react-scripts: command not found.
Followed the steps exactly. npm 5.0.3 node v8.1.2

npm 5.x has more than 50 known issues.
Please downgrade to npm 4.x until they're solved.
https://github.com/npm/npm/issues/16991

I had the same issue but got it working if I switched from npm to yarn

If anyone is using TypeScript and experiencing this issue, substitute react-scripts-ts for react-scripts in your package.json.

I had the same issue, downgrading to npm 4.x worked

You can fix this just if run again npm install or yarn install
worked for me

I’ll lock this because it is completely out of our control.
npm can help you with this.
Please file issues with them: github.com/npm/npm/issues
Thanks!


linux/arm64 runtime.Breakpoint times out

Please answer these questions before submitting your issue. Thanks!

What version of Go are you using (go version)?
go 1.6 and current tip (go version devel +fc6bcde Fri Mar 18 23:26:36 2016 +0000 linux/arm64)

What operating system and processor architecture are you using (go env)?
GOARCH="arm64"
GOBIN=""
GOEXE=""
GOHOSTARCH="arm64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH=""
GORACE=""
GOROOT="/home/odroid/go"
GOTOOLDIR="/home/odroid/go/pkg/tool/linux_arm64"
CC="gcc"
GOGCCFLAGS="-fPIC -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build271441235=/tmp/go-build -gno-record-gcc-switches"
CXX="g++"
CGO_ENABLED="1"

What did you do?
If possible, provide a recipe for reproducing the error.
A complete runnable program is good.
A link on play.golang.org is best.

Using a bootstrap environment built on linux/amd64, built Go from sources on linux/arm64.

What did you expect to see?
ALL TESTS PASSED

What did you see instead?

runtime.Breakpoint test times out. all other tests succeed. as minux suggested, here's the gdb backtrace of runtime/testdata/testprog Breakpoint when it hangs:
$ pwd
/home/odroid/go/src/runtime/testdata/testprog
$ go version
go version devel +fc6bcde Fri Mar 18 23:26:36 2016 +0000 linux/arm64
$ gdb ./testprog
GNU gdb (Ubuntu 7.11-0ubuntu1) 7.11
Copyright (C) 2016 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later http://gnu.org/licenses/gpl.html
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.  Type "show copying"
and "show warranty" for details.
This GDB was configured as "aarch64-linux-gnu".
Type "show configuration" for configuration details.
For bug reporting instructions, please see:
http://www.gnu.org/software/gdb/bugs/.
Find the GDB manual and other documentation resources online at:
http://www.gnu.org/software/gdb/documentation/.
For help, type "help".
Type "apropos word" to search for commands related to "word"...
Reading symbols from ./testprog...done.
warning: File "/home/odroid/go/src/runtime/runtime-gdb.py" auto-loading has been declined by your `auto-load safe-path' set to "$debugdir:$datadir/auto-load".
To enable execution of this file add
add-auto-load-safe-path /home/odroid/go/src/runtime/runtime-gdb.py
line to your configuration file "/home/odroid/.gdbinit".
To completely disable this security protection add
set auto-load safe-path /
line to your configuration file "/home/odroid/.gdbinit".
For more information about this security protection see the
"Auto-loading safe path" section in the GDB manual.  E.g., run from the shell:
info "(gdb)Auto-loading safe path"
(gdb) r Breakpoint
Starting program: /home/odroid/go/src/runtime/testdata/testprog/testprog Breakpoint
[New LWP 23014]
[New LWP 23015]
[New LWP 23017]
[New LWP 23016]
[LWP 23011 exited]
^C
Thread 2 "testprog" received signal SIGINT, Interrupt.
[Switching to LWP 23014]
runtime.usleep () at /home/odroid/go/src/runtime/sys_linux_arm64.s:141
141		SVC
(gdb) bt
#0  runtime.usleep () at /home/odroid/go/src/runtime/sys_linux_arm64.s:141
#1  0x00000000000432c8 in runtime.mstart1 ()
at /home/odroid/go/src/runtime/proc.go:1082
#2  0x0000000000000001 in ?? ()
Backtrace stopped: previous frame identical to this frame (corrupt stack?)

What is the output of "thread apply all bt" in gdb?

$ gdb ./testprog
GNU gdb (Ubuntu 7.11-0ubuntu1) 7.11
Copyright (C) 2016 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later http://gnu.org/licenses/gpl.html
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.  Type "show copying"
and "show warranty" for details.
This GDB was configured as "aarch64-linux-gnu".
Type "show configuration" for configuration details.
For bug reporting instructions, please see:
http://www.gnu.org/software/gdb/bugs/.
Find the GDB manual and other documentation resources online at:
http://www.gnu.org/software/gdb/documentation/.
For help, type "help".
Type "apropos word" to search for commands related to "word"...
Reading symbols from ./testprog...done.
warning: File "/home/odroid/go/src/runtime/runtime-gdb.py" auto-loading has been declined by your `auto-load safe-path' set to "$debugdir:$datadir/auto-load".
To enable execution of this file add
add-auto-load-safe-path /home/odroid/go/src/runtime/runtime-gdb.py
line to your configuration file "/home/odroid/.gdbinit".
To completely disable this security protection add
set auto-load safe-path /
line to your configuration file "/home/odroid/.gdbinit".
For more information about this security protection see the
"Auto-loading safe path" section in the GDB manual.  E.g., run from the shell:
info "(gdb)Auto-loading safe path"
(gdb) r Breakpoint
Starting program: /home/odroid/go/src/runtime/testdata/testprog/testprog Breakpoint
[New LWP 13288]
[New LWP 13289]
[New LWP 13290]
[New LWP 13291]
[New LWP 13292]
[New LWP 13293]
[LWP 13285 exited]
^C
Thread 2 "testprog" received signal SIGINT, Interrupt.
[Switching to LWP 13288]
runtime.usleep () at /home/odroid/go/src/runtime/sys_linux_arm64.s:141
141		SVC
(gdb) thread apply all bt
Thread 7 (LWP 13293):
#0  runtime.futex () at /home/odroid/go/src/runtime/sys_linux_arm64.s:308
#1  0x000000000002021c in runtime.notesleep (n=0x1616b0 <runtime.sched+16>)
at /home/odroid/go/src/runtime/lock_futex.go:145
#2  0x0000000000044438 in runtime.stopm ()
at /home/odroid/go/src/runtime/proc.go:1519
#3  0x000000000004448c in runtime.stopm ()
at /home/odroid/go/src/runtime/proc.go:1522
#4  0x0000000000000001 in ?? ()
Backtrace stopped: previous frame identical to this frame (corrupt stack?)
Thread 6 (LWP 13292):
#0  runtime.futex () at /home/odroid/go/src/runtime/sys_linux_arm64.s:308
#1  0x000000000002021c in runtime.notesleep (n=0x1616b0 <runtime.sched+16>)
at /home/odroid/go/src/runtime/lock_futex.go:145
#2  0x0000000000044438 in runtime.stopm ()
at /home/odroid/go/src/runtime/proc.go:1519
#3  0x000000000004448c in runtime.stopm ()
at /home/odroid/go/src/runtime/proc.go:1522
#4  0x0000000000000001 in ?? ()
Backtrace stopped: previous frame identical to this frame (corrupt stack?)
---Type  to continue, or q  to quit---
Thread 5 (LWP 13291):
#0  runtime.futex () at /home/odroid/go/src/runtime/sys_linux_arm64.s:308
#1  0x000000000002021c in runtime.notesleep (n=0x1616b0 <runtime.sched+16>)
at /home/odroid/go/src/runtime/lock_futex.go:145
#2  0x0000000000044438 in runtime.stopm ()
at /home/odroid/go/src/runtime/proc.go:1519
#3  0x000000000004448c in runtime.stopm ()
at /home/odroid/go/src/runtime/proc.go:1522
#4  0x0000000000000001 in ?? ()
Backtrace stopped: previous frame identical to this frame (corrupt stack?)
Thread 4 (LWP 13290):
#0  runtime.futex () at /home/odroid/go/src/runtime/sys_linux_arm64.s:308
#1  0x000000000002021c in runtime.notesleep (n=0x1616b0 <runtime.sched+16>)
at /home/odroid/go/src/runtime/lock_futex.go:145
#2  0x0000000000044438 in runtime.stopm ()
at /home/odroid/go/src/runtime/proc.go:1519
#3  0x000000000004448c in runtime.stopm ()
at /home/odroid/go/src/runtime/proc.go:1522
#4  0x0000000000000001 in ?? ()
Backtrace stopped: previous frame identical to this frame (corrupt stack?)
Thread 3 (LWP 13289):
---Type  to continue, or q  to quit---
#0  runtime.futex () at /home/odroid/go/src/runtime/sys_linux_arm64.s:308
#1  0x000000000002021c in runtime.notesleep (n=0x1616b0 <runtime.sched+16>)
at /home/odroid/go/src/runtime/lock_futex.go:145
#2  0x0000000000044438 in runtime.stopm ()
at /home/odroid/go/src/runtime/proc.go:1519
#3  0x000000000004448c in runtime.stopm ()
at /home/odroid/go/src/runtime/proc.go:1522
#4  0x0000000000000001 in ?? ()
Backtrace stopped: previous frame identical to this frame (corrupt stack?)
Thread 2 (LWP 13288):
#0  runtime.usleep () at /home/odroid/go/src/runtime/sys_linux_arm64.s:141
#1  0x00000000000432c8 in runtime.mstart1 ()
at /home/odroid/go/src/runtime/proc.go:1082
#2  0x0000000000000001 in ?? ()
Backtrace stopped: previous frame identical to this frame (corrupt stack?)
(gdb)

@minux Any clues from that backtrace?

I didn't see anything special in the backtrace. Somehow the BKPT
instruction doesn't trigger a SIGTRAP signal and make program stuck.

We have a builder for arm64 GNU/Linux, and it is running the tests, and is not showing any problem.
That said, according to https://gcc.gnu.org/ml/gcc-patches/2013-11/msg02228.html we should be using BRK $0x1000.  Which unfortunately does not seem to work with our assembler.
Could you try running this C program with and without an argument and tell us the output and exit status for each case?  Thanks.
#include <stdio.h>

int main(int argc, char **argv) {
    if (argc > 1) {
	    asm("brk #0x1000");
	    printf("continuing after brk #0x1000\n");
    } else {
	    asm("brk #0");
	    printf("continuing after brk #0\n");
    }
}

FWIW, I can't reproduce this on a pine64 board running linux/arm64.
Using tip @ c83e6f50d983d81166d21736ff9ab0ad2182f0fa, linux jessie, GNU gdb (Debian 7.7.1+dfsg-5) 7.7.1. I did:
$ cd src/runtime/testdata/testprog
$ go build
$ gdb ./testprog
// ...
(gdb) r Breakpoint
Program received signal SIGTRAP, Trace/breakpoint trap.
runtime.breakpoint () at /home/debian/go/src/runtime/asm_arm64.s:97
97		BRK

In addition to trying the C program from @ianlancetaylor, you might also try manually sending a SIGTRAP (kill -s SIGTRAP <pid>) when the process is hung inside gdb, just to confirm that that piece of it works.

fyi - not able to reproduce it on my Odoid-C2
Using tip @ a65a96, UbuntuMate, GNU gdb (Ubuntu 7.11-0ubuntu1) 7.11
$ gdb ./testprog
This GDB was configured as "aarch64-linux-gnu".
(gdb) r Breakpoint
Starting program: /root/go/src/runtime/testdata/testprog/testprog Breakpoint
[New LWP 3269]
[New LWP 3270]
[New LWP 3272]
[New LWP 3271]
Thread 1 "testprog" received signal SIGTRAP, Trace/breakpoint trap.
runtime.breakpoint () at /root/go/src/runtime/asm_arm64.s:97
97		BRK

Thanks for trying.
@9nut I'm going to close this issue until we get more information about your system.  See questions above.


Mac OS import issue

WIth snorkel-metal-0.2.3, we are experiencing this issue again. Any idea how to fix it?
    from matplotlib.backends import _macosx
E   ImportError: Python is not installed as a framework. The Mac OS X backend will not be able to function correctly if Python is not installed as a framework. See the Python documentation for more information on installing Python as a framework on Mac OS X. Please either reinstall Python as a framework, or try one of the other backends. If you are using (Ana)Conda please install python.app and replace the use of 'python' with 'pythonw'. See 'Working with Matplotlib on OSX' in the Matplotlib FAQ for more information.
``

Dealing with this in #28


Many errors when using oneOf for switch/case validation

What version of Ajv are you using? Does the issue happen if you use the latest version?
6.10
Ajv options object

JSON Schema

{
  $schema: 'http://json-schema.org/draft-07/schema#',
  $id: 'carsSchema',
  type: 'object',
  oneOf: [
    { $ref: '#/$defs/trainRequiredFields' },
    { $ref: '#/$defs/busRequiredFields' },
    { $ref: '#/$defs/carRequiredFields' }
  ],
  additionalItems: false,
  required: ['vehicleType'],
  $defs: {
    trainRequiredFields: {
      type: 'object',
      properties: {
        vehicleType: { const: 'train' },
        name: {
          type: 'string'
        },
        trainType: {
          type: 'string'
        }
      },
      required: ['name', 'trainType']
    },
    busRequiredFields: {
      type: 'object',
      properties: {
        vehicleType: { const: 'bus' },
        name: {
          type: 'string',
        },
        isSchool: {
          type: 'boolean'
        }
      },
      required: ['name', 'isSchool']
    },
    carRequiredFields: {
      type: 'object',
      properties: {
        vehicleType: { const: 'car' },
        name: {
          type: 'string'
        },
        engineType: {
          type: 'string'
        },
        hp: {
          type: 'number'
        }
      },
      required: ['name', 'engineType']
    }
  }
}

Sample data

Your code

Validation result, data AFTER validation, error messages
Message:
Value "car" does not match const.
Schema path:
carsSchema#/$defs/busRequiredFields/properties/vehicleType/const
Message:
Value "car" does not match const.
Schema path:
carsSchema#/$defs/trainRequiredFields/properties/vehicleType/const
Message:
Invalid type. Expected Number but got String.
Schema path:
carsSchema#/$defs/carRequiredFields/properties/hp/type
Message:
Required properties are missing from object: isSchool.
Schema path:
carsSchema#/$defs/busRequiredFields/required
Message:
Required properties are missing from object: trainType.
Schema path:
carsSchema#/$defs/trainRequiredFields/required

What results did you expect?
Does it correct behaviour to add additional fields per "vehicleType" eg. only the "car" type has the "hp" property. I'm getting many errors from other types too. How to create create a schema that will validate only one particular type? eg. only train if the vehicleType is "train"
Are you going to resolve the issue?

most likely you are using allErrors: true option, in which case this is the expected behaviour.

I'm testing it using the https://www.jsonschemavalidator.net service. Setting the allErrors to false doesn't help on the application side.

How to create create a schema that will validate only one particular type?

you need to use discriminator for that (in ajv v8) - keyword and option.

Thank you!


[SPARK-11301][SQL] fix case sensitivity for filter on partitioned columns

cc @liancheng

Merged build triggered.

Merged build started.

Test build #44320 has started for PR 9271 at commit d0d866b.

Test build #44320 has finished for PR 9271 at commit d0d866b.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44320/
Test PASSed.

Merged build finished. Test PASSed.

test this please

Merged build triggered.

Merged build started.

@cloud-fan Seems we also need to fix it in branch 1.5?

Test build #44636 has started for PR 9271 at commit d0d866b.

Test build #44636 has finished for PR 9271 at commit d0d866b.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44636/
Test PASSed.

LGTM. Merging to master. Looks it will not apply cleanly with 1.5 branch. Can you have a separate pr for 1.5 branch?

This part of code has been changed a lot by https://github.com/apache/spark/pull/8916/files , should we backport that too?

@cloud-fan Seems not very hard to back port #8916 as well. How about we have a pr against 1.5 to include both #8916 and this one?


Add option to pass custom DB session on method call

We use it to pass the active transaction to authority when updating our users.
I've added a test that check this functionality.

@harranali Can you please take a look at this change and at #8?
Thanks

@harranali What do you think about this change?
Also we need to merge #8 for users to be able to install and use this package correctly

Hi @harranali, Can we please push this change?

Hi @harranali, I would really like us to merge this PR alongside https://github.com/harranali/authority/pull/8
We use this fork for a while now in production and we want to use the upstream.
Thank you

Hi @asafalima ,
I have just added transactions with a simpler approach by adding 3 functions BeginTX, Rollback(), and Commit(),
for more info please refer to the develop branch code and the readme file.
Thanks.


[AutoDiff] Disable debug type verification for AutoDiff-generated decls.

Workaround for TF-1232.
Robust support for type reconstruction is non-trivial; it involves name lookup
for declarations generated in SILOptimizer, after name lookup caches are
typically populated.

TF-1232 explains an incomplete attempt to implement type reconstruction for
AutoDiff-generated decls.

@swift-ci Please test

Obsolete after https://github.com/apple/swift/pull/30863: type reconstruction fix for AutoDiff-generated declarations.


[Snyk] Upgrade @opentelemetry/api from 1.4.1 to 1.6.0

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade @opentelemetry/api from 1.4.1 to 1.6.0.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 2 versions ahead of your current version.
The recommended version was released a month ago, on 2023-09-12.

Release notes

Package name: @opentelemetry/api

1.6.0 - 2023-09-12API 1.6.0
🐛 (Bug Fix)

Revert "feat(api): add attributes argument to recordException API #4071"

This feature was an unintentional breaking change introduced with 1.5.0

1.5.0 - 2023-09-11API 1.5.0
Warning
This release included an unintenitional breaking change in. This change was rolled back in https://github.com/open-telemetry/opentelemetry-js/releases/tag/api%2Fv1.6.0
🚀 (Enhancement)

feat(api): add attributes argument to recordException API #4071

1.4.1 - 2023-03-13

    from @opentelemetry/api GitHub release notes

Commit messages

Package name: @opentelemetry/api

faf939c chore: release API 1.6.0/SDK 1.17.0/Experimental 0.43.0 (#4138)
27897d6 Revert "feat(api): add attributes argument to recordException API (#4071)" (#4137)
5fcd8cf chore: prepare release API 1.5.0, SDK 1.16.0/0.42.0 (#4122)
5d94185 chore(deps): update actions/checkout action to v4 (#4134)
ba38804 chore(deps): update all patch versions (#4114)
415b198 fix(ci): pin nx to 15.9.6 for Node.js v14 (#4124)
4d662cf fix(sdk-metrics): metric names should be case-insensitive (#4059)
9452607 docs(guidelines) tiny fix, add comma on docs (#4119)
6d174e2 chore(deps): update dependency markdownlint-cli to v0.36.0 (#4113)
cacb825 fix: limit npm to <10.0.0 on node v14 and v16 (#4111)
5b7f619 chore(deps): update dependency @ babel/preset-env to v7.22.14 (#4103)
1da2b1b chore(deps): update dependency prettier to v3.0.3 (#4092)
cd4998a feat(api): add attributes argument to recordException API (#4071)
c84637f fix(exporter-logs-otlp-http): add @ opentelemetry/api-logs as dependency (#3968)
432d439 chore(sdk-node): add notice to manually install '@ opentelemetry/exporter-jaeger' (#4068)
d8a14ea chore(deps): update all patch versions (#4087)
6e7df29 fix(deps): update dependency axios to v1.5.0 (#4089)
a793321 feat: update PeriodicExportingMetricReader and PrometheusExporter to accept optional metric producers (#4077)
0cc9a0b fix(renovate): remove faulty '@ types/node' rule and explicitly apply it to the whole repo (#4091)
8ac369f chore(deps): update all patch versions (#4050)
c230076 chore(deps): update dependency gh-pages to v6 (#4058)
dfe9bdd chore(deps): update prettier to v3 and prettier-plugin-eslint to v5 (#4075)
863c4d4 chore(sdk-node): deprecate methods in favor of constructor options (#3996) (#4076)
92050b5 docs: fix broken link to renovate.json (#4085)

Compare

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

New PR created at https://github.com/DTS-STN/secure-client-hub/pull/427


How to debug this code in app_process ?

HI, I revise some code in ape, but I don't know how to debug this. Could you tell me how you debug this code? Thank you very much !

Sorry, I have no idea how to debug the code. I have no experience of Android app development. Perhaps we could use the recorded GUI tree to implement a debugging agent that simulates the execution of Android apps.


fix(modal): make modal a focus trap

ref carbon-components #1637, #1075
Adds focus-trap package and implements a focus trap on the modal component. By default the focus trap is active, but users can disable if they need to.
Changed

implement <FocusTrap> component
set initial focus to first interactable element (close button)

Removed
functionality provided by focus-trap-react

remove componentDidMount method
remove focusButton method
remove onBlur handler
remove onTransitionEnd method
remove escape key handler

To test
Observe that by default the modal is a focus trap. Tab moves from the close button to the first button and then to the second and loops back around. shift-tab cycles in reverse.
In your dev environment set the focusTrap prop in Modal-story.js to false to see previous modal focus behavior of being able to move focus out of the modal.
attn: @carmacleod, @asudoh

@dakahn Awesome! Looking forward to being able to use it in our product.  :)

@dakahn can you update the snapshots to pass CI?

Yeah, sorry, I'm still working on this one -- for some reason it's not letting me assign the WIP label? 😕

Looks a cool addition @dakahn!

@asudoh Super sorry I totally acknowledged your review comment and just didn't implement the changes 😿
The fix is in now


Improve softuart implementation.

Hello :)
I just play a bit with softuart library from extras and notice that is very error prone.
So I put softuart_put() in between  taskENTER_CRITICAL();  and taskEXIT_CRITICAL();
and it seams that help a lot. Error still happens but not so often without it.
If you have any other idea how we could improve softuart implementation feel free and share your ideas.

Hi @hetii !
That's great! Would you please create the pull request?

Sure as soon as I do more tests and add more common operating modes.

had the same problems... and basically the same solution. If you neen some inspiration: softwareUart


Enum fields are not properly scanned (and documented)

Problem statement
Although not documented here https://goswagger.io/use/spec/model.html the codescan attempts to parse the // Enum: comments but the result is not as expected. Also, the generated documentation comment is impossible to parse when enum items are strings and contain spaces or special chars.
Steps to reproduce
here is a spec that illustrates both problems:
swagger spec:
swagger: "2.0"
info:
  version: "1.0.0"
  title: "Test"

paths: {}
definitions:
  test_enum:
    type: object
    properties:
      enum_string_field:
        type: string
        enum: ["1", "with space", 'with "quotes"']
      enum_int_field:
        type: integer
        enum: [-1, 2, 6000]

When code is generated using this spec (go run cmd/swagger/swagger.go generate model -t test/ -f test/swagger.yml) we get the following result:
// TestEnum test enum
//
// swagger:model test_enum
type TestEnum struct {

	// enum int field
	// Enum: [-1 2 6000]
	EnumIntField int64 `json:"enum_int_field,omitempty"`

	// enum string field
	// Enum: [1 with space with "quotes"]
	EnumStringField string `json:"enum_string_field,omitempty"`
}

It is already clear that the EnumStringField items can not be parsed because it is impossible to reconstruct where the spaces were initially. However, here is the generate spec result (go run cmd/swagger/swagger.go generate spec  -w test/ --scan-models --include=test  -o test/swagger-gen.yml):
definitions:
    test_enum:
        description: TestEnum test enum
        properties:
            enum_int_field:
                description: enum int field
                enum:
                    - '[-1 2 6000]'
                format: int64
                type: integer
                x-go-name: EnumIntField
            enum_string_field:
                description: enum string field
                enum:
                    - '[1 with space with "quotes"]'
                type: string
                x-go-name: EnumStringField
        type: object
        x-go-name: TestEnum
        x-go-package: github.com/go-swagger/go-swagger/test/models
paths: {}
swagger: "2.0"

And we see how enum are parsed as a single item.
Fix proposal.
First we need a better documentation format for the items, and we can use json:
type TestEnum struct {

	// enum int field
	// Enum: [-1,2,6000]
	EnumIntField int64 `json:"enum_int_field,omitempty"`

	// enum string field
	// Enum: ["1","with space","with \"quotes\""]
	EnumStringField string `json:"enum_string_field,omitempty"`
}

Then we can fix the parsing issue.
Environment
swagger version: v0.30.5-25-g10db6e00
go version: go1.18.3 linux/amd64
OS:   Ubuntu 20.04

I would like to fix both issues if there are no objections to the documentation format changes.

fixed with #3004  merged


Feature/2267 standalone functions

Submisison Checklist

[x] Run unit tests: ./runTests.py src/test/unit
[x] Run cpplint: make cpplint
[x] Declare copyright holder and open-source license: see below

Summary
Add support for standalone function compilation. See #2267.
There are a lot of outstanding questions about whether this is the right way to implement it. I am copying those from #2267, as they seem to got lost there:

The decision to choose mode of compilation (functions only / model) by file extension and the choice of the .stanfuncs extension are both important and should be discussed by the core team (there could be a need for more compilation modes in the future, especially with regards to #2224).

My reasoning behind this decision was:

The mode of compilation should be deducible from the file to be compiled alone and should not require the user to check some external configuration/build/... file. This is good for both reading code and editor support.
Deciding what to do with a file based on its extension is a common practice, in particular it is what gcc does.

I did some changes/refactoring to some very core files and to the test infrastructure to avoid duplicating code, but I felt very uncomfortable making the changes without supervision from someone who understands the project better.
Test functions files now reside in test/test-models/good-standalone-functions - is this a reasonable place in the tree? (an obvious alternative would be test/test-standalone-functions/good)

Note that I cannot put .stanfuncs files simply in  test/test-models/good because it is then impossible to distinguish the resulting .hpp files from each other (and I don't think .stanfuncs should produce something like .hpp-funcs)

I had trouble with the 80 char limit, I don't like some of the indents / line breaks it forced me to do, but I couldn't find a better way - could you please check that the code reads nice and is aligned with what others write? (e.g. the two-line for cycle in stanc_helper.hpp line 293).
When compiling standalone functions, it is definitely necessary to check whether other blocks are contained in the input file. I see two possible ways of doing this - what would you prefer?

keep the default parser and check whether the members of stan::lang::program are all empty except function_decl_defs_ (requires maintanence when new types of program blocks are introduced or when the program struct changes)
create a specific parser variant that only recognizes the functions block (would require maintanance when the parser structure changes and a specific parser for other compilation modes would have to be created as well)

Should compile_functions() give a warning or error when other blocks are present?

Intended Effect
Add .stanfuncs as a possible source file extension that then results in compiling a file with instantiations of all templated functions.
How to Verify
.stanfuncs files are compiled as part of test build, also tests in src/test/unit/lang/parser/functions_standalone_instantiate_test.cpp and src/test/unit/lang/parser/functions_standalone_test.cpp
Side Effects
Touched a lot of core files, please verify it is OK.
Documentation
Is rudimentary and pending approval of the core design.
Reviewer Suggestions
Copyright and Licensing
Please list the copyright holder for the work you are submitting (this will be you or your assignee, such as a university or company):
By submitting this pull request, the copyright holder is agreeing to license the submitted work under the following licenses:

Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

Header tests failed.  Please let me know when you think this is ready to review and I can look it over.  I can do that before the header tests get fixed if it's otherwise done.  And thanks for submitting a pull request.

My bad, forgot I could run header tests locally. Fixed (hopefully).

Thanks.  I'll get to reviewing this week.

I checked out this branch from Martin's repo.  As a test I took the file src/test/test-models/good-standalone-functions/basic.stanfuncs and copied it to src/test/test-models/good/basic.stan.  Also reformatted it for consistent indentation and placement of open curly brackets, so contents of both files is:
functions {
  real my_log1p_exp(real x) {
    return log1p_exp(x);
  }

  vector my_vector_mul_by_5(vector x) {
    vector[num_elements(x)] result = x * 5.0;
    return result;
  }
}

I compiled both files using Stan's makefile, which invokes stanc on the files basic.stanfuncs and basic.stan:
make src/test/test-models/good-standalone-functions/basic.hpp-test
make src/test/test-models/good/basic.hpp-test

To compare these files, I edited out the model definition and the model typedef statement which is the last line of the transpiled stan program, then ran diff:
diff -U 3 src/test/test-models/good/basic.hpp src/test/test-models/good-standalone-functions/basic.hpp
--- src/test/test-models/good/basic.hpp	2017-05-28 11:50:02.000000000 -0400
+++ src/test/test-models/good-standalone-functions/basic.hpp	2017-05-28 11:51:13.000000000 -0400
@@ -1,24 +1,19 @@
 // Code generated by Stan version 2.15.0

-#include <stan/model/model_header.hpp>
-
-namespace basic_model_namespace {
+#include <stan/model/standalone_functions_header.hpp>

+namespace basic_functions {
 using std::istream;
 using std::string;
 using std::stringstream;
 using std::vector;
-using stan::io::dump;
 using stan::math::lgamma;
-using stan::model::prob_grad;
 using namespace stan::math;

 typedef Eigen::Matrix<double,Eigen::Dynamic,1> vector_d;
 typedef Eigen::Matrix<double,1,Eigen::Dynamic> row_vector_d;
 typedef Eigen::Matrix<double,Eigen::Dynamic,Eigen::Dynamic> matrix_d;

-static int current_statement_begin__;
-
 template <typename T0__>
 typename boost::math::tools::promote_args<T0__>::type
 my_log1p_exp(const T0__& x, std::ostream* pstream__) {
@@ -91,6 +86,14 @@
     }
 };

+double
+my_log1p_exp(const double& x, std::ostream* pstream__){
+	return my_log1p_exp<double>(x, pstream__);
+}

+Eigen::Matrix<double, Eigen::Dynamic,1>
+my_vector_mul_by_5(const Eigen::Matrix<double, Eigen::Dynamic,1>& x, std::ostream* pstream__){
+	return my_vector_mul_by_5<double>(x, pstream__);
 }

+ }

Two things:

why use a different header?
compiled stan program header file include:

#include <stan/model/model_header.hpp>

compiled stan functions-only block header file include:
#include <stan/model/standalone_functions_header.hpp>

The set of includes in  standalone_functions_header.hpp is a subset of the files in model_header.hpp.  It would be better to just use the model_header.hpp file - first off, it's not clear to me that the standalone functions header has all necessary includes, secondly, maintaining two sets of includes is more work.
I reviewed the changes to the code in the src/stan/lang/ and it seems OK.
However, we need clarification from @bgoodri as to what the comments need to be generated so that the transpiled hpp files can be included directly from R.

Please add test models for rng and lp functions.
Suggested rng function test:
functions {
  real foo_rng(real mu, real sigma) {
    return normal_rng(mu, sigma);
  }
}

Did we also want a test that both ode_integrators work?

At the top, it needs // [[Rcpp::depends(rstan)]]. Then, it needs // [[Rcpp::export]] without a semicolon before each function (but not the corresponding functor), as in
// [[Rcpp::export]]
inline double
my_function(...)

I would also suggest to add a default argument of the null pointer to each std::ostream* pstream__ argument because that cannot be passed from R. For _lpdf and _lpmf functions, the trailing lp_accum__ should be defaulted to zero. The functions that end in _rng are difficult. I currently replace the base_rng__ argument with an int seed and then at the first line of the function body, I have
static boost::ecuyer1988 base_rng__(seed);

You can pass both the RNG and stream of you have helper functions for
producing them.
On Sun, May 28, 2017, 9:00 PM bgoodri<EMAIL_ADDRESS>wrote:

At the top, it needs // [[Rcpp::depends(rstan)]]. Then, it needs //
[[Rcpp::export]] without a semicolon before each function (but not the
corresponding functor), as in
// [[Rcpp::export]]
inline double
my_function(...)
I would also suggest to add a default argument of the null pointer to each std::ostream*
pstream__ argument because that cannot be passed from R. For lpdf and
lpmf functions, the trailing lp_accum should be defaulted to zero. The
functions that end in rng are difficult. I currently replace the
base_rng_ argument with an int seed and then at the first line of the
function body, I have
static boost::ecuyer1988 base_rng__(seed);
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/stan-dev/stan/pull/2310#issuecomment-304552033, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAfA6QmN31TLE45r530NzTSCRaN6bWQFks5r-hiPgaJpZM4Nfc1M
.

Thanks for the feedback, it is appreciated. However, all of the comments so far deal with (in my view) minor technical stuff. Does that mean that you are OK with the high-level structure and the main design decisions behind the way I implemented this feature? Especially the points 1-3 in the pull request description. Any opinion on 5&6 would also be welcome.
@mitzimorris The reduced set of headers was explicitly requested (https://github.com/stan-dev/stan/issues/2267#issuecomment-292600155) and should lead to shorter compilation times - the model includes are IMHO a huge beast. I currently don't have a strong opinion whether this is worth the extra maintanence cost, just wanted to point out the reasoning behind this choice.

@mitzimorris the test for _rng, _lp and _lpdf functions was already there, in src/test/test-models/good-standalone-functions/special_functions.stanfuncs and it is tested that it both compiles and executes (in src/test/unit/lang/parser/functions_standalone_test.cpp and src/test/unit/lang/parser/functions_standalone_instantiate_test.cpp. Is it sufficient or do you suggest extending it?
@bgoodri I added the RCpp comments and the null default to pstream (the default was added to all functions - even in normal Stan - is that a problem?), however, I cannot reasonably give default the lp_accum__ argument as it is a double&. I have also added a default __create_rng(int seed) function that is also exported to RCpp (currently for standalone functions only).

OK, so adding default for the pstream__ argument turned out to be harder than expected: if a forward declaration is present, the default needs to be put there and only there. But AFAIK, I cannot check in the generator whether the function I am currently generating has a forward declaration somewhere above.
So I added the default only to instantiations, but that means that it will not be present for functions which do not need instantiations (e.g. int-only arguments). Maybe it will be better to create a helper function to get a stream as well?

BTW: this is more or less what the helper function in R would look like, similar for the stream (you could even pass Rcpp::rcout from the helper so the output would appear correctly.
#include <Rcpp.h>

// [[Rcpp::export]]
Rcpp::XPtr<boost::ecuyer1988> get_rng(const int& seed=0) {
  boost::ecuyer1988 base_rng__(seed);
  Rcpp::XPtr<boost::ecuyer1988> ptr(base_rng__);
  return ptr;
}

No, we can't do what @sakrejda is suggesting in https://github.com/stan-dev/stan/pull/2310#issuecomment-304935311 because we can't introduce dependencies on R or Rcpp (or any other GPL-ed code) in Stan itself.

Bob, I'm not suggesting putting Rcpp stuff into Stan code. The point is we
can leave the RNG argument in these generated functions because it's
trivial for rstan to provide a function that fills it in!
On Tue, Jun 13, 2017, 4:31 PM Mitzi Morris<EMAIL_ADDRESS>wrote:

@mitzimorris requested changes on this pull request.
In src/stan/lang/generator/generate_function_instantiation_body.hpp
https://github.com/stan-dev/stan/pull/2310#discussion_r121788601:

* @param[in] fun function declaration

* @param[in] is_rng true if function is an RNG

* @param[in] is_lp true if function accesses log density

* accumulator

* @param[in] is_log true if function is log probability function

* @param[in] rng_class class of the RNG being used (required by xxx_rng

* functions)

* @param[in,out] o stream for generating

*/

void generate_function_instantiation_body(const function_decl_def& fun,

                               bool is_rng, bool is_lp, bool is_log,

                               const std::string& rng_class,

                               std::ostream& o) {

 o << "{" << EOL;

 o << "\t";

 if (!fun.return_type_.is_void()) {

replace all \t with 2 spaces - generated code should contain spaces not
tabs - change this everywhere.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/stan-dev/stan/pull/2310#pullrequestreview-43846067,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAfA6bcwKtt8KIZRAMIeUlA2FDl4vHkeks5sDvG3gaJpZM4Nfc1M
.

@sakrejda  Great --- as long as the helper R function stays in RStan, that's fine.

I'd go and make these changes myself, but I don't know how to work on remote branches.  @seantalts --- you said you'd show me how to do this, but I'm going to be out of town for a week.  Would it be easy for me to somehow change all the \t to two spaces?  I think that's the last thing waiting to do before this gets merged.

I've replaced the tabs with spaces. Should be ready to go.


Specific BBT citation key format is no longer working for my use case after update

Zotero version: 5.0.97-beta.52
BBT version: 5.6.3
Support log ID:
YQ4IHZ2I-refs-euc
Expected behavior:
For the particular reference I've chosen as an example, I am expecting a citation key of 'AzLe19a'. More generally, I want a citation key format with the following structure:
a) If the reference has a single author, [first three letters of first author surname] [last two digits of year] ['a' if no conflicts, otherwise next alphabetical char available]
b) If the reference has two authors, [first two letters of first author surname][first two letters of second author's surname][same as above]
c) If the reference has three authors, [first letter of first author surname][first letter of second author surname][first letter of third author surname][same as above]
d) If the reference has four or more authors, first letter of first author surname][first letter of second author surname][first letter of third author surname][plus sign][same as above]
This was working with the following format until today (Zotero updated today and I also think I allowed for the update of some BBT translators or something, I'm really not sure):
[auth1_1][auth1_2][auth1_3][auth1_4:>0]+[shortyear][postfix+1=%(a)s]|
[auth1_1][auth1_2][auth1_3:>0][auth1_3][shortyear]|
[auth2_1][auth1_2:>0][auth2_2][shortyear]|
[auth4_1][shortyear]

Actual behavior:
Starting today, the citation keys are being generated with the following format:
[first letter of every author up to four][plus-sign][last two digits of year]['a' if no conflicts, otherwise next alphabetical char available]

I have a change coming up that would break your use-case, but there's an equivalent pattern that will work now and after the change:
[postfix+1=%(a)s]|
[auth1_1][auth1_2][auth1_3][auth1_4:>0]+[shortyear]|
[auth1_1][auth1_2][auth1_3:>0][auth1_3][shortyear]|
[auth2_1][auth1_2:>0][auth2_2][shortyear]|
[auth4_1][shortyear]

the reason is that before, the postfix would be set when the pattern was read; after the change, the postfix will be set as the pattern is executed, and [auth1_4:>0]+ can jump to the next pattern before the postfix is executed. This change I show here will always execute the postfix.


Added a test for elastic go-freelru cache

Introduce a benchmark for go-freelru cache by elastic
( also a small compilation issue fix for Ristretto )

@maypok86 @mekegi please merge if you have write access

Thanks for the contribution! :)


Enhancement: Being able to select may choices

This library currently doesny allow to be able to select multiple choices. It would be great to do so.
(an use case, I have 7 radio buttons, one for each week day, and I need to select 3 of those days)

@eddieatkinson this module implements radio buttons. what you are looking for is checkboxes. i think your request is a bit out of scope for what this package is.


[3.5.5] factory roboport is somehow deleting bots if you don't give them bots initially (on loading save)

I had previously open #67 and this is slightly about that issue but something different too
so, you place a factory down and it steals 10 of my construction bots
I go in, take back my bots cause I haven't even made a logistic network or anything and I really didn't intend to have to maintain this when I did the research, but here I am
so I started factorio up a half hour ago, saw updates, updated.
now running factorissimo-2-notnotmelon_3.5.5
as soon as I load the game, I see my bots head to the nearby factory
then I get 6 alerts saying all my factories are missing 1 construction bot
I go back into the factory and get my bots but there aren't 10 but 6
I had a stack of 50 now 39
so I visit each factory and grab the bots it steals from me but in the end after visiting all 7 of my factories, I'm still left with 39 bots
so they went ... somewhere?
So again, is there some way to disable this behavior?  If I need bots in a factory, I'm more than capable of just putting them in there myself.
Doesn't need a gui element, just needs a config option.  Is that still "non trivial" to fix?
I have 50 bots here

so I reloaded my save to get those screenshots and this time it took 10 and I got 10 back
so tbh I don't know how my initial load and subsequent running around lost me 11 bots in total
but also, I have these alerts stuck now till I visit each factory
I may have run into the factory while the bots were still approaching.  not even sure at this point

so I just went around to my other factories cause I couldn't avoid it really
went into each one.  when I was all done, I am now missing 5 bots that I have no idea where they went.  I've now checked all the factories a second time to verify.

Thanks for your report. Sorry for prematurely closing #67
I currently have no way to see notifications for new messages on closed bug reports.

I'm more than capable of just putting them in there myself.

The whole point is to have it request bots when you are in remote view on another planet.

I had a stack of 50 now 39 so I visit each factory and grab the bots it steals from me but in the end after visiting all 7 of my factories, I'm still left with 39 bots

Sorry about this. However it seems you encountered this bug over multiple hours. In this situation it's impossible for me to reproduce the edge case. If you can give me short, reliable reproduction steps then I am happy to fix the issue.

I have rewritten all code related to this. Please open a new ticket if this is still an issue.

I'm more than capable of just putting them in there myself.

The whole point is to have it request bots when you are in remote view on another planet.

oh that makes a lot of sense.  so you can construct things remotely, etc.  I haven't gotten to space yet so I didn't see the point.
Thanks very much for the clarification


[np-47582] Add initial CloudFormation templates

Ticket: https://sikt.atlassian.net/browse/NP-47971
Changes:

Adds a CFN template for the root pipeline that deploys other resources. This sets up one shared pipeline that deploys a vault for each environment.
Adds a CFN template for a secure backup vault shared with specified accounts

The best review of this is to deploy it.

It's deployed to the tooling account temporarily, so the result can be inspected there.


feat: Adding a script for updating package.json dev dependencies

This PR adds a new script to the SDK: npm run upgrade. When run within the context of a custom library created by the SDK, it will automatically upgrade its package.json to point at the latest versions of Web and the Web SDK. It will also explicitly add/update any of Web's dependencies as direct dependencies of the project. Without this, VS Code autocomplete and intellisense don't work for things like arcgis-extensions or react.

Will check asap Eric!

@eric-parsons I've finally had time to test it, sorry for this delay. But, it works great on my end :-)!

@eric-parsons I've finally had time to test it, sorry for this delay. But, it works great on my end :-)!

Great, glad it worked for you. Thanks for testing it. I've merged it in now. You will need to manually update the version of the SDK to the latest and manually add an upgrade script to your project's package.json, but after that you should be able to just run npm run upgrade in the future.


Use right primary dark colors

Right now, the app uses the same color for toolbar and status bar, and that makes it look a bit iOS-ish and that annoys me a bit.
Can you please use the right primary dark colors for both themes?
Light: #e0e0e0
Btw, for dark theme, can you please fix both primary and primary dark colors?
Primary Dark: #000000
Primary: #212121
PS: If you're using a transparent status bar, please add a black 20% opacity overlay, or #33000000. Height: Android 6+: 24dp Android 5 and below: 25dp.
Thanks in advance :)

I am added a ViewOverlay to the toolbar for the light theme.
I dont like the look of the semitransparent toolbar with a darker statusBar in the dark theme, thats why I am  leaving the toolbar in the dark as it is.
Btw you shouldn't use fixed values for the statusBar inset, because you are not guaranteed to have a statusBar (e.g. split screen), an OnApplyWindowInsetsListener is a better option to get the top inset for the statusBar. (only work on Lollipop and up).

I am sorry, but I changed my mind on this one.
I think without the darker statusBar it looks nicer and cleaner.

@kollerlukas
Oh, yeah, sorry, didn't thought about how it would look in split-screen.
Btw, can you please keep the darker status bar, even as an option in settings? I really love how it looks. Please.
Thanks in advance.


Add Formatter::formatResponseForRequest()

Q
A

Bug fix?
no

New feature?
yes

BC breaks?
no

Deprecations?
yes

Related tickets
fixes #145

Documentation
-

License
MIT

During 1.x lifecycle:
deprecation.INFO: User Deprecated: Class "Http\HttplugBundle\Collector\Formatter" should implement method "Http\Message\Formatter::formatResponseForRequest(ResponseInterface $response, RequestInterface $request): string": Formats a response in context of its request.

Next major release should add the method for real.

Im happy with this.
But this will only have a "real" effect in v2. Currently v2 is not planned.
Introducing a new interface would make more sense and would make this change "real" now. I see that it was discussed in #145

i will update at least logger plugin as such, using method_exsists trick till v2.
im still not convinced we need moar interfaces :')

But this will only have a "real" effect in v2. Currently v2 is not planned.

aparently this is the symfony way of "adding" methods to interfaces without breaking BC. the consumer would have to use method_exists as i understood

i agree with @ro0NL's reasoning. ok for you @Nyholm or do you have a strong feeling that that would be wrong?

about deprecating formatResponse(): https://github.com/php-http/message/issues/145#issuecomment-1030182461
i think it makes sense :+1: and ideally there's 1 responsible for method for response formatting. Let me know if should be part of this PR.

if we do the new method on the same interface, i would indeed deprecate the old formatResponse method. and best to this in the same MR.

thanks! i will see if we can get the build to green again, that would be good to see before tagging the release.

and released https://github.com/php-http/message/releases/tag/1.13.0


Adding to multi with the same id will block a call to perform

For example, the following will block on the multi.perform call:
multi = EventMachine::Synchrony::Multi.new

multi.add(:shared_id, EM::HttpRequest.new("http://google.com")))
multi.add(:shared_id, EM::HttpRequest.new("http://yahoo.com"))

deferred = multi.perform

Have not tried calling the iterators but I presume the effect will be the same.

Agreed. That's what I would expect as well (raise exception if same name is used again).


Run the heavy processing with lower cpu priority.

Best ensure does not impact gameplay.

Release v1.6b addresses this issue.


Ablation Studies for ZSD

I notice that the Semantic Mask Head can improve the performance of ZSD. In my mind，this component is used for instance segmentation. How can the SMH influence the ZSD. Could you explain it for me? And where is the code. Thanks a lot.

The code of SHM is in here
The principle that SHM affects ZSD is consistent with the principle that mask head in Mask R-CNN can improve detection performance. The gradient generated by detection and instance segmentation as parallel training tasks can simultaneously supervise the learning of the previous model parameters.

I got it.Thank you.%%%


Add health check for container status.

docker-compose now supports health checks (since version 1.10.0), and delaying start-up of containers until their dependencies are up and healthy. See https://docs.docker.com/compose/compose-file/#healthcheck for docs on this.
There seems to be different standards for running the health check.  Some use a health_check.sh file or some poll the rest server itself.
I'm not sure where to start looking to add another endpoint to the rest server.  I see wait-for-it is called but not sure.
Thanks

need the same for deployment in the cloud

Hi @jeffm89 & @jzillmann, apology for late to response. I've been busy lately. I will try to find time to work on an implementation if i could find a reasonable time.
Anyway if you can submit a PR, it would be great too. ;)


ProvidedMethod with static parameters in generative TP "does not have an invoker "

Description
The issue is the following:
I am working on a generative TP, and I am trying to generate non-static providedMethods with static parameters.
However I have 2 cases where this is not working :

either I have add the generated types to a temporary assembly like :

let assemblyPath = Path.ChangeExtension(System.IO.Path.GetTempFileName(), ".dll")
let assembly = ProvidedAssembly assemblyPath
assembly.AddTypes [ty]

However, I get a compilation error :

ProvidedMethod: no invoker for ExampleMethWithStaticParam on type GenerativeTypeProviderExample.Provided.Example

2nd solution, I can remove those 3 lines from the code, therefore I don't have any compilation error any longer.
However I get a runtime error of the form :

internal error: null: convTypeRefAux

Repro steps
To reproduce the error the following step should make it happen :

Go to the following https://github.com/Lleutch/StaticMethodGTP and clone it
Compile it on VS, it doesn't work currently with the fake script (I didn't took the time to make the fake script work for the repro repo. If that's necessary just tell me about it and I will do it 🙂 )
go to Examples/Examples.fsx and this is a small example using the TP where you'll see the issue.
(Optional) Comment/Uncomment the 3 lines mentionned in comments in Library1.fs to have either of the two issues.

Expected behavior
I should be able to generate types that are not erased at run-time and that can contain these non-static provided methods with static parameters.
The reason for having made the choice of a generative TP, comes from the fact that this TP is used to include session-types in F#, and within session-types the branching choice mechanism is something that relies on run-time receive branching which makes it "necessary" to have the types generated.
Known workarounds
I haven't found any workaround.
Related information

Operating system : Windows 10
.NET Runtime, CoreCLR or Mono Version : .Net framework 4.5

If there is any information that might be lacking there, please do not hesitate to ask me i'll do my best to help.
Thank you in Advance for your time.

I'm not sure if the static parameter support was intended to work on generative type providers. All of the examples I've seen (and samples from original repository) were on erased...

After a discussion on twitter with @dsyme : https://twitter.com/dsyme/status/895588269265571841 , we now know that for generative TPs it is not possible to have static parametrized method.
It is possible for erased TPs though.
I will then close the issue as we know that it is not feasible for generative TPs.
:slightly_smiling_face:


feat(bench): circuit table size dep

Issue Link / Motivation
Close #249
Changes Overview
In this branch I am performing a specific test. Depending on the results, this benchmark may or may not be included in the main. At the moment, this benchmark is running on our server

I'll also add a test that does all assignments to a single region, to see if it affects performance, vs. multiple regions

I ran a benchmark overnight with the following parameters:

k = 20
Number of nested poseidon-hash from 0 to 800 with step=50


CAMEL-13786: add delivey delay to jms camel componenet

@mahdiKarami98 you are welcome to keep wokring on this with the suggestions from Jan, and then open it again if you have it updated.

@mahdiKarami98 you are welcome to keep wokring on this with the suggestions from Jan, and then open it again if you have it updated.

Thanks claus. I worked on comments of Jon but I understood another one work on  this issue so I thought  close this PR. I will search another issue.

It was me, I wasn't saying you had to close the PR. I was just saying, for the next time, to take a look at comments. I think you can re-open this and go ahead.

OK. so I will continue again :)

How it is going @mahdiKarami98? Do you have any issues we can help you with?

How it is going @mahdiKarami98? Do you have any issues we can help you with?

It's been a long time since I'm done. But building camel is so dramaticly get my time. In building camel-jira ,the attlasian reop ip is banned in my contry (Iran). And I couldn't get libraries. At this time I searched vpn to connect to the repo. So I think tommorw I will finished building camel.

You do not have to build the entire Camel source code, just what you need to build for camel-jms.

You do not have to build the entire Camel source code, just what you need to build for camel-jms.

but changing camel-jms and building camel changed camel-activemq. it's true?
if yes, then maybe another component is relates to camel-jms?

Yeah camel-activemq extends camel-jms so that is why its changed too.

I have a half build camel and build camel-jms.
can you check it now?

Do not worry with full build, if you have issues with it. We will regenerate the remaining after merge.

LGTM. This will need some regen after merge. @oscerd can I merge this, or is it better to wait until 3.0.0.RC1 is cutted?

I think it is safe to merge. Maybe just take a look at the CI build. We can eventually revert and rework after the RC1. @davsclaus do you agree too?

I think the option should also be set when creating the InOut JMS template. And does setting the -1 value always (as default) is that okay with Spring JMS as they use the same value as default?

InOut template reuses the InOnly template. It shouldnt be needed to set again (createInOutTemplate() calls createInOnlyTemplate() as first statement).
initialization to -1 should be OK - JmsTemplate initializes this to -1 in its constructor and there is no other logic in the setter.

I can confirm with integration test that this is working well with and without delay for both InOnly and InOut @davsclaus . I will rebase this locally and will commit regen and integration test.

Thanks for contribution @mahdiKarami98 .  Regen in commit dd29fb0e16e133685b29760e37fc49f41601b361. Integration test in commit 4c2f7fb0131d636b8a2182aeda6f3359a8a7a2f8

LGTM


WIP: Fix crash on signal/resume to dangling target

Adaptation of 54bdc1e1f6a7fb85a5b193c9b8ecf0dcf06949e6 (from #23094) to 3.0.

Marking as WIP for now since #23094 caused a regression (#23661).

Oh, I missed this one. Thanks, @akien-mga.


Databinding and Dependency Properties?

I found your article and am the one who started the issue (https://github.com/aspnet/Blazor/issues/374) where the Blazor team responded about MVVM support. This is near and dear to my heart and I was a little disappointed by that response.
After exploring this issue in more detail myself I've concluded that while using INotifyPropertyChanged gets you a good part of the way, true data binding (a la WPF / UWP / Xamarin Forms), and something equivalent to dependency properties (which can be assigned Binding instances as well as property values), are nonetheless the way to go. Without that we'll never have true separation of concerns between the UI markup and the viewmodel code.
I've been able to make a great deal of progress toward an internal, proprietary framework for doing this, but would be interested in working on an open source version if there's community interest.

I was very excited about how smooth things are in Blazor -- until I need to update a model field that is used by 2 components. Doing a bit of research and... there's just no clear supported solution. Just hacks here and there.
From what I'm seeing, MVVM just doesn't seem to be drawing a lot of interest. Is there an alternative solution that is being encouraged by Microsoft or by the community?

@mysteryx93 the thing is, Blazor doesn't require property change notification as it has other mechanisms, so I wouldn't recommend it for green field projects. However,  there is a major opportunity to migrate WPF and other MVVM apps to Blazor. I'm working on a reference app to demonstrate what I think is a good practice for view models and commands in Blazor apps. You can see the work in progress here:
https://github.com/JeremyLikness/ExpressionPowerTools/tree/miscellaneous-nice-things/samples/Blazor/BlazorMvvmExample
The key to updating multiple components is here:
https://github.com/JeremyLikness/ExpressionPowerTools/tree/miscellaneous-nice-things/samples/Blazor/BlazorMvvmExample/BlazorMvvmExample/Client/Shared
(See the MVVMWrapper).
As for an alternate solution, defining a service class and using Cascading Values seems to be a common approach.


[google_maps_flutter] A crash on the Android emulator

After updating Flutter, a crash began to occur on the Android emulator due to Google maps.
This is not on the ios simulator.
Code:
pubspec.yaml:
google_maps_flutter: ^1.0.3
google_maps_flutter_web: ^0.1.0+5

main.dart:
import 'package:flutter/material.dart';
import 'package:google_maps_flutter/google_maps_flutter.dart';

void main() async {
  runApp(App());
}

class App extends StatelessWidget {

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      home: AppCore(),
    );
  }
}

class AppCore extends StatefulWidget {
  @override
  _AppCoreState createState() => _AppCoreState();
}
class _AppCoreState extends State<AppCore> {

  @override
  Widget build(BuildContext context) {
    return GoogleMap(
      initialCameraPosition: const CameraPosition(target: LatLng(0, 0), zoom: 4),
    );
  }
}

Logs
Running "flutter pub get" in flutt...
Launching lib/main.dart on Android SDK built for x86 64 in debug mode...
 lib/main.dart
✓ Built build/app/outputs/flutter-apk/app-debug.apk.
Connecting to VM Service at ws://<IP_ADDRESS>:58638/ADe2loFh0mg=/ws
D/        ( 7032): HostConnection::get() New Host Connection established 0x7db9ab276740, tid 7047
D/        ( 7032): HostConnection::get() New Host Connection established 0x7db9ae63c260, tid 7068
D/        ( 7032): HostConnection::get() New Host Connection established 0x7db9acc675e0, tid 7056
W/org.test( 7032): Verification of com.google.android.gms.maps.model.BitmapDescriptor io.flutter.plugins.googlemaps.Convert.toBitmapDescriptor(java.lang.Object) took 112.524ms
I/zzbz    ( 7032): Making Creator dynamically
W/org.test( 7032): Unsupported class loader
I/DynamiteModule( 7032): Considering local module com.google.android.gms.maps_dynamite:0 and remote module com.google.android.gms.maps_dynamite:221
I/DynamiteModule( 7032): Selected remote version of com.google.android.gms.maps_dynamite, version >= 221
V/DynamiteModule( 7032): Dynamite loader version >= 2, using loadModule2NoCrashUtils
W/org.test( 7032): Unsupported class loader
W/org.test( 7032): Skipping duplicate class check due to unsupported classloader
I/Google Maps Android API( 7032): Google Play services client version: 12451000
I/Google Maps Android API( 7032): Google Play services package version: 16089023
W/org.test( 7032): Accessing hidden field Ljava/nio/Buffer;->address:J (light greylist, reflection)
D/        ( 7032): HostConnection::get() New Host Connection established 0x7db9ae63cf20, tid 7103
D/EGL_emulation( 7032): eglCreateContext: 0x7db9951fdc60: maj 1 min 0 rcv 1
D/EGL_emulation( 7032): eglMakeCurrent: 0x7db9ae650e60: ver 3 0 (tinfo 0x7db99ba49220)
D/EGL_emulation( 7032): eglMakeCurrent: 0x7db9951fdc60: ver 1 0 (tinfo 0x7db99ba10500)
E/GLConsumer( 7032): [SurfaceTexture-0-7032-0] attachToContext: invalid current EGLDisplay
E/flutter ( 7032): [ERROR:flutter/shell/platform/android/platform_view_android_jni_impl.cc(44)] java.lang.RuntimeException: Error during attachToGLContext (see logcat for details)
E/flutter ( 7032): 	at android.graphics.SurfaceTexture.attachToGLContext(SurfaceTexture.java:289)
E/flutter ( 7032): 	at io.flutter.embedding.engine.renderer.SurfaceTextureWrapper.attachToGLContext(SurfaceTextureWrapper.java:55)
E/flutter ( 7032):
F/flutter ( 7032): [FATAL:flutter/shell/platform/android/platform_view_android_jni_impl.cc(1065)] Check failed: CheckException(env).
F/libc    ( 7032): Fatal signal 6 (SIGABRT), code -6 (SI_TKILL) in tid 7056 (1.raster), pid 7032 (org.test)
*** *** *** *** *** *** *** *** *** *** *** *** *** *** *** ***
Build fingerprint: 'google/sdk_gphone_x86_64/generic_x86_64:9/PSR1.180720.093/5456446:userdebug/dev-keys'
Revision: '0'
ABI: 'x86_64'
pid: 7032, tid: 7056, name: 1.raster  >>> org.test <<<
signal 6 (SIGABRT), code -6 (SI_TKILL), fault addr --------
Abort message: '[FATAL:flutter/shell/platform/android/platform_view_android_jni_impl.cc(1065)] Check failed: CheckException(env).
'
    rax<PHONE_NUMBER>000000  rbx<PHONE_NUMBER>001b90  rcx 00007dba35429b98  rdx<PHONE_NUMBER>000006
    r8  00007db99ddef17e  r9  00007db99ddef17e  r10 00007db99ddef17e  r11<PHONE_NUMBER>000246
    r12<PHONE_NUMBER>000000  r13 00007db9ae624808  r14<PHONE_NUMBER>001b78  r15 00007db99db54648
    rdi<PHONE_NUMBER>001b78  rsi<PHONE_NUMBER>001b90
    rbp<PHONE_NUMBER>000007  rsp 00007db99db54638  rip 00007dba35429b98
backtrace:
    #00 pc<PHONE_NUMBER>026b98  /system/lib64/libc.so (syscall+24)
    #01 pc<PHONE_NUMBER>029775  /system/lib64/libc.so (abort+101)
    #02 pc<PHONE_NUMBER>33f8b5  /data/app/org.test-zHr-upERaYj4WxzVix_2Ww==/lib/x86_64/libflutter.so (offset 0x1334000)
    #03 pc<PHONE_NUMBER>335268  /data/app/org.test-zHr-upERaYj4WxzVix_2Ww==/lib/x86_64/libflutter.so (offset 0x1334000)
Lost connection to device.
Exited (sigterm)

Flutter doctor
% flutter doctor -v
[✓] Flutter (Channel master, 1.24.0-4.0.pre.50, on Mac OS X 10.15.7 19H2 darwin-x64, locale en-CY)
    • Flutter version 1.24.0-4.0.pre.50 at /Users/Sammius/Flutter/flutter
    • Framework revision 6823fbf918 (3 hours ago), 2020-10-24 00:37:03 -0400
    • Engine revision 38d75c9e77
    • Dart version 2.11.0 (build 2.11.0-242.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    • Android SDK at /Users/Sammius/android
    • Platform android-29, build-tools 28.0.3
    • ANDROID_HOME = /Users/Sammius/android
    • ANDROID_SDK_ROOT = /Users/Sammius/android
    • Java binary at: /Library/Java/JavaVirtualMachines/jdk-10.0.2.jdk/Contents/Home/bin/java
    • Java version Java(TM) SE Runtime Environment 18.3 (build 10.0.2+13)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 12.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.1, Build version 12A7403
    • CocoaPods version 1.10.0.rc.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[!] Android Studio (not installed)
    • Android Studio not found; download from https://developer.android.com/studio/index.html
      (or visit https://flutter.dev/docs/get-started/install/macos#android-setup for detailed instructions).

[✓] IntelliJ IDEA Community Edition (version 2019.2.3)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 39.0.5
    • Dart plugin version 192.7402

[✓] VS Code (version 1.50.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.15.1

[✓] Connected device (3 available)
    • Android SDK built for x86 64 (mobile) • emulator-5554 • android-x64    • Android 9 (API 28) (emulator)
    • Web Server (web)                      • web-server    • web-javascript • Flutter Tools
    • Chrome (web)                          • chrome        • web-javascript • Google Chrome 86.0.4240.111

! Doctor found issues in 1 category.

Hello @Sammius ,
Could you please post the outputs of flutter run -v and a complete minimal reproducible code sample.
Thank you!

@Sammius
Tried your sample code on Android emulator 10 using latest master version (1.24.0-4.0.pre.57) and (1.24.0-4.0.pre.55), but didn't see the crash. The map was rendered properly (ignore the blank screen, since api key wasn't correct).

flutter doctor -v
[✓] Flutter (Channel master, 1.24.0-4.0.pre.57, on Mac OS X 10.15.4 19E2269
    darwin-x64, locale en-IN)
    • Flutter version 1.24.0-4.0.pre.57 at
      /Users/dhs/documents/Fluttersdk/flutter
    • Framework revision fdac8a9d1f (9 hours ago), 2020-10-25 19:47:02 -0400
    • Engine revision d4a3c9c471
    • Dart version 2.11.0 (build 2.11.0-256.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.0)
    • Android SDK at /Users/dhs/Library/Android/sdk
    • Platform android-30, build-tools 30.0.0
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 12.0.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.0.1, Build version 12A7300
    • CocoaPods version 1.9.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 46.0.2
    • Dart plugin version 193.7361
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)

[✓] VS Code (version 1.50.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.11.0

[✓] Connected device (6 available)
    • SM A260G (mobile)                   • 5200763ebcfa861f
      • android-arm    • Android 8.1.0 (API 27)
    • sdk gphone x86 arm (mobile)         • emulator-5554
      • android-x86    • Android 11 (API 30) (emulator)
    • iPhone SE (2nd generation) (mobile) • 6C85835D-FBFD-4AB3-8DE8-B4FAD35E5367
      • ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-0 (simulator)
    • macOS (desktop)                     • macos
      • darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Web Server (web)                    • web-server
      • web-javascript • Flutter Tools
    • Chrome (web)                        • chrome
      • web-javascript • Google Chrome 86.0.4240.111

• No issues found!

What version you were earlier on when you updated ?

Hi @darshankawar.
I upgraded but it didn't help. I will look for a mistake in myself.


Upgrading Bitcoin CoreRaspibolt3

What
Possible failure mode as written; LND may not re-connect to bitcoind after restart
For the future: upgrade Bitcoin Core
...
...
...
...

Download, verify, extract and install the Bitcoin Core binaries as described in the Bitcoin section of this guide.

Restart the Bitcoin Core systemd unit
$ sudo systemctl restart bitcoind

ADD:
LND may not reconnect to bitcoind.  Check this with
$ sudo journalctl -f -u lnd

If it shows any errors, LND may need to be restarted with
$ sudo systemctl restart lnd

Thanks for your contribution!
1º For this PR shouldn't be necessary to change 49 files - you must create a new branch and modify only bitcoin-client.md file: https://github.com/raspibolt/raspibolt/blob/master/guide/bitcoin/bitcoin-client.md
BTW: You should fork the original master branch, not another forked project.
https://github.com/raspibolt/raspibolt
https://github.com/VajraOfIndra/RaspiBolt
2º We are all supposed to know that when the BTC Core is restarted the rest of the services that depend on it must be configured in the code to wait for the BTC Core to be available again. Otherwise, the rest of the services must be restarted manually. Maybe we should put this notice in another more general section, not in every service
I think that this PR is not correctly opened

As requested by OP, closing this PR for the time being. OP will create a new one from a forked original RaspiBolt branch later on.


Revert #1774

Pr #1774 was delivered during release freeze and after a lot of debugging on the release, we found that it breaks the image publish and the push doesn't contain the tags, so it tried to push latest:
https://github.com/confidential-containers/cloud-api-adaptor/actions/runs/8664530199/job/23765073592
Using default tag: latest
The push refers to repository [quay.io/confidential-containers/cloud-api-adaptor]
tag does not exist: quay.io/confidential-containers/cloud-api-adaptor:latest

Revert it and then cut a new release.

@stevenhorsman ah, sorry for I forget that we are in release freeze window, won't merge any PRs after we finish the release process.


PR: Ensure QFont().setPointSize() receives an int argument

Description of Changes
Thanks for your work on this version!  I'm close to getting it working for Debian....
The test spyder/plugins/shortcuts/widgets/tests/test_summary.py::test_shortcutssummary fails with Python 3.10 because QFont().setPointSize() requires an integer argument, and Python no longer performs implicit float -> integer conversions.  This patch fixes the cause.
Affirmation
By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.

I certify the above statement is true and correct: juliangilbey

Note: This depends on https://github.com/spyder-ide/spyder/pull/17641 for the tests to pass

Ooh, I have done a rebase on a pull request before, but I don't remember exactly how; a quick reminder would be helpful, so that I don't mess this up!

Sure! So you will need to have/set the original Spyder repo as a new remote like for example upstream and fetch it. So something like:
git remote add upstream https://github.com/spyder-ide/spyder.git
git fetch upstream

Then be sure to have your branch checkout and rebase using the 5.x from the new upstream remote. So something like this:
git checkout make_point_size_integer
git rebase upstream/5.x

Finally force push the changes with something like:
git push -f

OK, thanks!  I thought it was as simple as that, but didn't want to force-push without being sure :smile:   Have done it now - all being well, the tests will pass!


Incorrect parameter name IncludeTotalResultCount

Hi there - there parameter named IncludeTotalResultCount that is mentioned in this article is incorrect. The correct name is includeTotalCount. I believe this needs to be updated elsewhere in the docs as well.

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

ID: 71857399-795d-8987-e076-419107564600
Version Independent ID: bbdda1cc-8bcc-f451-adc3-336cdc222621
Content: Azure Cognitive Search client library for JavaScript
Content Source: docs-ref-services/search-documents-readme.md
Product: azure
Technology: azure
GitHub Login: @maggiepint
Microsoft Alias: magpint

Thanks for reporting @dereklegenzoff
We will track this in https://github.com/Azure/azure-sdk-for-js/issues/13082

Thanks for reporting @dereklegenzoff
We will track this in https://github.com/Azure/azure-sdk-for-js/issues/13082


Added profile picture selection and capture and roster view

Hello,
This being my second PR i expect to comeback and make changes because i probably missed something or didn't do something right which i would be happy to do. I look forwards to getting feedback.
I added the translations of the buttons in the 3 languages offered, but i used google translate for that and i'm not sure that they are completely correct.

Hello, I will have a look and let you know quickly.

Thanks @thdebay ,
I closed the PR because this error appeared Git Error: You need to resolve your current index first. I'm having some trouble with the branches and syncing everything. When i fix everything i'll resubmit the PR.

The screenshots look good, I can't wait to test it!
Maybe you need to rebase your branch on master in order to solve the git error?
However, if all the changes you wanted to include are here, it's okay to keep the PR as it is, I think I'll be able to merge it. I'll solve the conflicts if there are some

I nearly had a heart attack while tinkering with gitbash because it deleted all my changes and synced it with the upstream academico, but i downloaded the branch Manage-student-profile-picture branch with all the changes luckily.
I'll try to rebase the branch, hopefully it works. This branch is 19 commits behind the main one, wouldn't that cause any trouble ?
Also i noticed one more thing, the changes i made to the backpack RegisterController  showRegistrationForm method where i added the Permissions-Policy header disappeared because i believe git doesn't include the vendor file. For the getUserMedia to be able to ask the user for their attached media the page needs to have that header per the documentation, is there a way to configure backpack to generate that part?


Base PR for 2020 reviews

Progress:

[x] 001_query_chembl
[x] 002_compound_adme
[x] 003_compound_unwanted_substructures
[x] 004_compound_similarity
[x] 005_compound_clustering
[x] 006_compound_maximum_common_substructures
[x] 007_compound_activity_machine_learning
[x] 008_query_pdb
[x] 009_compound_ensemble_pharmacophores
[x] 010_binding_site_comparison

Global changes that are pending:

[x] Agree on a CSV export style. Some use sep=";", and we can just defautl to ,. This affects T002 (saving), and 005-007 (loading).
[X] Add assertions for the # TODO CI comments

[ ] Enable NBVAL_CHECK_OUTPUT as metadata instead of comments
[x] Make sure tests are actually passing...

[ ] Clean outputs (?)
[ ] Render outputs from Treebeard in the Sphinx site (async query to get the raw JSON?)
[x] Streamline all talktorials' header references to the initial TeachOpenCADD publication.
[x] seed_everything as available in T010 should go to library and be used in other notebooks that need a fixed random seed.
[ ] Some markdown content is not rendered correctly in some markup (headers and links do not allow backticks or bold, for example)
[X] Run Black-nb -l 99 and add checks

Woohoo, all green! This means our notebooks are now:

Running correctly
Consistently outputting the same result

There might be changes with different RDKit versions and things like that, but at least given a certain environment, they are well behaved now.

As discusses internally, I am merging this, taking into account that:

We won't remove the # NBVAL_CHECK_OUTPUT comments for now because we prefer being explicit.
We won't clear the outputs because they are needed for the nbval cell tests. Instead, we try to remove any references to individual machines (mainly absolute paths that leak the local username or the machine hostname).
Changes in the docs site will be left for another PR (markup limitations). No need to render docs from the TreeBeard runs since we have the outputs. Alternatively, we can run them from the nbval outputs using job.needs and artifact outputs in GHA (example here).


How to add UIRefreshControl

How do I add a UIRefreshControl to the treeview when the tableView (or scrollview) is not accessible?

Currently it is not possible using public API

I already created my own treeview implementation. It was easy to add a pull to refresh control to the tableView.


Added checks for protected branches in pull requests

Fixes #2875. This pull request adds checks whether the user is allowed to push to the base branch of a pull request.
If a user may not push to the base branch, the green merge button is hidden on the pull request view. If the user tries merge via the web api, an error (http code 500) with
{"message":"branch is protected [name: %s]","url":"https://godoc.org/github.com/go-gitea/go-sdk/gitea"}
is returned.
There shouldn't be a way to circumvent the branch protection via a pull request.

Codecov Report

Merging #3544 into master will decrease coverage by 0.01%.
The diff coverage is 17.39%.

@@            Coverage Diff             @@
##           master    #3544      +/-   ##
==========================================
- Coverage   35.75%   35.73%   -0.02%
==========================================
  Files         284      284
  Lines       40773    40794      +21
==========================================
  Hits        14577    14577
- Misses      24037    24053      +16
- Partials     2159     2164       +5

Impacted Files
Coverage Δ

models/pull.go
51.68% <0%> (-0.48%)
:arrow_down:

models/error.go
32.65% <0%> (-0.48%)
:arrow_down:

routers/repo/issue.go
33.24% <36.36%> (-0.03%)
:arrow_down:

models/repo_list.go
65.62% <0%> (-1.57%)
:arrow_down:

models/repo_indexer.go
48.3% <0%> (ø)
: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 7b29780...fed2b26. Read the comment docs.

With moving that check to function would allow to reuse that function to add additional conditions for merge limitations

Sorry, had a problem with the first commit after your change request :( I hope the last commit meets your requirements.

The CI build failed because of a git error (https://drone.gitea.io/go-gitea/gitea/3754/2):
+ git init
Initialized empty Git repository in /srv/app/src/code.gitea.io/gitea/.git/
+ git remote add origin https://github.com/go-gitea/gitea.git
+ git fetch --tags --depth=50 origin +refs/pull/3544/merge:
fatal: The remote end hung up unexpectedly
exit status 128

I don't think that this has something to do with my latest commit, is there some way to tell drone to retry?

LGTM

need @lafriks 's review.


onTap for Dismissible, or seperate actions for left and right swipe

I am working on a To do list app and i displayed the tasks added by the user in a ListView.builder. i essentially want to use Dismissble to delete and mark a task as done by adding a LineStrike. I am using a database to use store, display and delete funtionalities.
I had two approaches in mind :
Approach 1: use onDismissed to delete and use an onTap to mark as complete.
Approach 2: use the left to right swipe to mark the task as complete and right to left to delete the task. The below gif is an idea of how approach 2 would look like but instead of displaying an icon, it would simply delete or mark the task as complete by swiping left or right
Either of the suggested features would be of great help .
(credits to https://github.com/letsar/flutter_slidable for the gif).

@AnuragVadd This is a question better suited for StackOverflow, since there doesn’t seem to be anything wrong with Flutter itself. Closing, as this is not an issue report. See https://flutter.dev/community for support options.