regression on command 0.7.0: stdout can not be loaded as json using json.loads()

What happened?

command plugin updated from 0.6.0 to 0.7.0
pulumi up aborts with error message that it can not parse the supposed json string at char 0 line 0
downgrading command to 0.6.0 and modifying pulumi stack with the resourceid and version of the old command plugin fixes the issue

Steps to reproduce
the corresponding code snippet is as following:
vault_ca = command.local.Command(
    "{}_vault_ca".format(name),
    create="./__vault_pipe.sh --yes",
    stdin=json.dumps(vault_config),
    opts=pulumi.ResourceOptions(
        parent=self,
        additional_secret_outputs=["stdout"],
        protect=True,
        # XXX make pulumi find legacy name for ca_factory_fault_ca
        aliases=[Alias(name="vault_ca")] if name == "ca_factory" else [],
    ),
)
ca_secrets = vault_ca.stdout.apply(lambda x: json.loads(x))

pulumi fails on ca_secret = ... while trying lambda x: json.loads(x)
it is probably reproducable by calling any external command, which should return a json formatted output to be parse by pulumi
Expected Behavior
same functionality as 0.6.0
Actual Behavior
pulumi bails out while trying to read stdout of the command as json
Output of pulumi about
CLI
Version      3.46.1
Go Version   go1.19.3
Go Compiler  gc
Host
OS       arch
Version  22.0.0
Arch     x86_64
Backend
Name           power
URL            file:///home/wuxxin/code/athome/state/pulumi
User           wuxxin
Organizations
Additional context
No response
Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

@dixler Could you please take a look?

I have the same issue

Hi. Thanks @wuxxin for filing the issue with the repro code (it helped me write a test case) and @AMKamel for the context it really helped narrow down what was going on. When moving from 0.6.0 to 0.7.0, fields being computed were dropped and that explains why it was misbehaving in preview.
This is an issue in the pulumi-go-provider library. I have opened https://github.com/pulumi/pulumi-go-provider/issues/75 to track this.

I'm seeing spurious quote marks around command.local.Command.stdout which is causing a similar issue with JSON parsing (in my case, from an AWS CLI command).

We can simply the repro a bit. Given the following program:
import { local } from "@pulumi/command";

const cmd = new local.Command("json", {
  create: `echo '{"json": true}'`,
});

export const parsed = cmd.stdout.apply(JSON.parse);

pulumi preview fails with:
  pulumi:pulumi:Stack (ts-dev):
    error: Running program '$DIR' failed with an unhandled exception:
    SyntaxError: Unexpected end of JSON input

pulumi up --skip-preview succeeds.

Are we expecting a fix here by this merge?
https://github.com/pulumi/pulumi-go-provider/pull/105#pullrequestreview-1589460460


chore(docs): Make sure we copy docs recursively

:tada: This PR is included in version 3.3.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[HTTP HEAD] problem

Give me this error:
In file included from C:\Users\Gianni\Documents\Arduino\libraries\WiFiManager-ESP32\examples\AutoConnect\AutoConnect.ino:14:0:
C:\Users\Gianni\Documents\Arduino\libraries\WiFiManager-ESP32/WiFiManager.h:36:22: error: 'const char HTTP_HEAD []' redeclared as different kind of symbol
const char HTTP_HEAD[] PROGMEM            = "<html lang="en"><meta name="viewport" content="width=device-width, initial-scale=1, user-scalable=no"/>{v}";
                  ^

In file included from C:\Users\Gianni\AppData\Local\Arduino15\packages\esp32\hardware\esp32\1.0.0\libraries\WebServer\src/WebServer.h:30:0,
             from C:\Users\Gianni\Documents\Arduino\libraries\WiFiManager-ESP32\examples\AutoConnect\AutoConnect.ino:12:

C:\Users\Gianni\AppData\Local\Arduino15\packages\esp32\hardware\esp32\1.0.0\libraries\WebServer\src/HTTP_Method.h:10:3: note: previous declaration 'HTTPMethod HTTP_HEAD'
HTTP_HEAD    = 0b00100000,
^

Hi!
Same problem here
Anyone taking care of this?
Thanks

There is an async version of this manager without the issues here (I'm already using it)
https://github.com/alanswx/ESPAsyncWiFiManager

i will check with your suggestion, thanks a lot , and I will send you how
is this working on, again thank you very much.
Em sáb, 15 de set de 2018 às 12:06, Garito<EMAIL_ADDRESS>escreveu:

There is an async version of this manager without the issues here (I'm
already using it)
https://github.com/alanswx/ESPAsyncWiFiManager
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/zhouhan0126/WIFIMANAGER-ESP32/issues/17#issuecomment-421583904,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AoU37xS8Bq_gh21YFdTeI9JiKwshPaoUks5ubReNgaJpZM4Wf-As
.

I'm having the same issue here.
@Garito, I just tried your suggestion but the author of WiFiAsyncManager says to add "<WiFiManager.h>", which is this library. I don't get it.

The guy copied the project and make the modifications but not updated the readme and so
Check the examples
If implemented even the websocket server using them

I was having this issue with the examples as well.
To fix it I replaced
`
#if defined(ESP8266)
#include <ESP8266WiFi.h>          //https://github.com/esp8266/Arduino
#else
#include <WiFi.h>          //https://github.com/esp8266/Arduino
#endif
//needed for library
#include <DNSServer.h>
#if defined(ESP8266)
#include <ESP8266WebServer.h>
#else
#include <WebServer.h>
#endif
#include <WiFiManager.h>         //https://github.com/tzapu/WiFiManager
`
With
#include <WiFi.h> #include <DNSServer.h> #include <ESP8266WebServer.h> #include <WiFiManager.h>

@bobermen1, are you using ESP32 or ESP8266?
I attempted replacing all HTTP_HEAD with HTTP_HEADER and it compiles and uploads it but I couldn't get the library to work. I tried this with the AutoConnect example that comes with the library.

I tried all the things above, but didn't work any.

Yeh @kotran88, same here. If you have selected the board ESP32, the compiler won't fetch the packages for ESP8266 and vice-versa. Different libraries for different cores. There seems to be a conflict with the variable name HTTP_HEAD and WebServer library.

To be a little more specific (for the people from the future with similar problem to find), this was "Brownout detector was triggered" flag thrown in the serial monitor. So if you have this problem, try a better cable. It seems to be a common problem when using the WiFi module of the ESP32. If you don't use the WiFi module, things seem peachy.
Long story long, initially I thought it was a grounding problem with my USB port on my computer, then I thought maybe it was the cable so I went out and bought a new one... which threw me off because the problem wasn't solved and it was a new cable... Also, when I powered the board with a battery bank, everything worked fine except the WiFi. At the end, after reading a bunch of forums and brainstorming I went out and got a better cable, problem solved.

I was having the same HTTP_HEAD problem and the solution was simpler than it looked. I just edited the WiFiManager.h files and WifiManager.cpp replacing (HTTP_HEAD) with (HTTP_HEADER) and there was no more compilation problem.

I had the same problem,
so I located the file "HTTP_Method.h"
in the "C: \ Arduino \ hardware \ espressif \ esp32 \ libraries \ WebServer \ src" folder, modified
line 10   HTTP_HEAD = 0b00100000,  --> HTTP_HEAD1 = 0b00100000,
and everything worked correctly.
MineirinRV

There is a pull request pending for this problem. I've created a fork and solved this issue, you can find it here Brunez3BD/WifiManager

Brunez3BD,
Thank you very much.
I found my WiFiManager files, opened it with Wordpad, replaced with your program and everything is fine.
Thank you again.

i am facing problem with esp32 , it was running very good uploading debugging, all of sudden warning came no more support esp board, i have tried all out, re installing ide, reinstalling ,packages,now every thing looks ok board is read on screen , up load ok data verified no error message, but application does not start, can any one give cue what is going one i have 4 esp board all of them same result. thanks in advance.

@shootdaj thanks for the workaround, it worked for me. However, could there be a typo in your message? I think it's not:
HTML_HEAD -> HTTP_HEAD_HTML
but actually:
HTTP_HEAD -> HTTP_HEAD_HTML.

I was having this issue with the examples as well.
To fix it I replaced
#if defined(ESP8266)
#include <ESP8266WiFi.h>          //https://github.com/esp8266/Arduino
#else
#include <WiFi.h>          //https://github.com/esp8266/Arduino
#endif

//needed for library
#include <DNSServer.h>
#if defined(ESP8266)
#include <ESP8266WebServer.h>
#else
#include <WebServer.h>
#endif
#include <WiFiManager.h>         //https://github.com/tzapu/WiFiManager

With
#include <WiFi.h>
#include <DNSServer.h>
#include <ESP8266WebServer.h>
#include <WiFiManager.h>

This fixed for me using LilyGo-T-Call-SIM800L.

There is an async version of this manager without the issues here (I'm already using it)
https://github.com/alanswx/ESPAsyncWiFiManager

Hi Garito, thank you so much for the solution, works 100% to me.
Regards,
Rogerio

A better solution imo is to rename the constant in WifiManager.h:
HTML_HEAD -> HTTP_HEAD_HTML
and then rename all the references in WifiManager.cpp:
HTML_HEAD -> HTTP_HEAD_HTML
I verified that this works. This way you are not breaking WebServer if it references that constant anywhere (or any other libs that depend on it)

Thanks mate thats work as well 👍👍❤


[Style] Make Centripetal Force Calculator responsive

Description
Centripetal-Force-Calculator is not responsive
Screenshots

Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[X] I am a GSSOC'24 contributor.
[ ] I am a IWOC'24 contributor.
[ ] I am a JWOC'24 contributor.
[ ] I am a SWOC'24 contributor.

Code of Conduct

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

@agastya3636 please tell how are you planning to fix

Go ahead @agastya3636

Go ahead @agastya3636
It will be made responsive for every screen size @jessicachawla

Provide your PR @agastya3636

If no one is working on this, please assign me this issue.

Go ahead @kauxp


WEBSOCKET SERVER DISCONNECTED: transport error

Hey,
whenever i open vim or bash inside the WebSSH2 terminal i get a transport error.
What am i doing wrong?

Good question. I'm not able to re-create this issue...
Can you provide:

Version of node WebSSH2 is running on
OS name/version, release of SSH, and version of vi that your ssh server is running

And I can investigate further.

thanks for your response,
I have cloned the repo and i am currently on following commit:
commit f890c46de33e8c5ccbc82adc0a53a17fa4a5d242
Author: billchurch<EMAIL_ADDRESS>Date:   Tue Jan 30 18:35:13 2018 -0500

    fix binaries

That's my vim version: VIM - Vi IMproved 7.4 (2013 Aug 10, compiled Nov 24 2016 16:44:48)
> uname -a  says:
Linux mail 4.4.0-101-generic #124-Ubuntu SMP Fri Nov 10 18:29:59 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux
bash --version shows:
GNU bash, version 4.3.48(1)-release (x86_64-pc-linux-gnu)
WebSSH2 Login: user=blub from=<IP_ADDRESS> host=<IP_ADDRESS> port=2135 sessionID=lIMw3s8WOlbtDjg0PP8p7a2xriIUwNAV/xWiKYKqF62kHVu_YAAAX allowreplay=false term=xterm-color
WebSSH2 Logout: user=blub from=<IP_ADDRESS> host=<IP_ADDRESS> port=2135 sessionID=lIMw3s8WOlbtDjg0PP8p7a2xriIUwNAV/xWiKYKqF62kHVu_YAAAX allowreplay=false term=xterm-color
WebSSH2 error: transport close

Perfect. I’ll work to try and reproduce. Something else you might try in the meantime, it will be information overload but you can run “export DEBUG=*” and then “npm run start” should output every key character and every ssh and websocket packet. May point to something.

I tested it again mit DEBUG=* npm start 2> webssh.log
You can find the output in the file below. Maybe this will help.
webssh.log

Nothing seems immediately obvious from the log. Can you run cat /etc/issue, I'm on Ubuntu 14.04.2 LTS and my VIM is:

VIM - Vi IMproved 7.4 (2013 Aug 10, compiled Jan  2 2014 19:39:02)

Just had a thought, I recently updated socket.io to 2.x, do me a favor and run:
npm --save install socket.io@1
npm run build
and try again and tell me if that changes anything.

After installing npm --save install socket.io@1 it finally works!
But vim and tmux sessions open in small windows with 80 characters in width.
Do you know why this happens?

hmmm... what does "tput cols" and "tput lines" say before you run vim and tmux?

@mjrudnik check out my latest commits, they're not in a release yet but did a lot with resizing and terminal geometry, i think you'll like it... Let me know if this solves your problem.

tput cols says 80 and tput lines says 24 even after pulling your latest changes.

Okay, i found out that when resizing my chrome window it changes width and height to the correct values.

Yes, that was a tricky one to figure out, I just released v0.2.0 which should handle a lot of resize use cases across a lot of browsers. I'm going to close this out now, if you're still having issues let me know and thanks for reporting this!

fixed by #43

run:
npm --save install socket.io@1
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules/fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"linux","arch":"x64"})

<EMAIL_ADDRESS>updated 1 package in 18.383s

Hello Bill, I have installed WebSSH2 on windows10 system.  I'm trying to open vim (installed using gvim81.exe from www.gvim.org)  I'm getting the below message.  Can you please help?
C:\Program Files\OpenSSH\home\riverwba>vim test.json
Vim: Error: This version of Vim does not run in a Cygwin terminal

@entriver not sure this is related to anything with WebSSH2 I can assist with. Sounds like your problem might be with Cygwin or Vim on Cygwin.

Hello Bill, Sorry forgot to update this post.  The issue is not related to WebSSH2. There is a openssh service available which is cygwin based. "Vim: Error: This version of Vim does not run in a Cygwin terminal" appears, if you are using wrong OpenSSH service.  We should install openssh as described in your manual.


Seurat convertibility check is suboptimal

This operation in the seurat convertibility check is performing an iterative count of nonzero matrix values when it could instead reference a dict that is already being populated (for non-csr matrix types, at least).

The function _count_matrix_nonzero already checks for the cached nonzero number count if its available, so we re already performing this optimization

@nayib-jose-gloria - per your comment, can this be closed?

@brianraymor I believe this can be closed, but I'd like @danieljhegeman to confirm that this is the behavior he was describing before closing. just in case there is a misunderstanding and there is a further optimization here

I agree with @nayib-jose-gloria -- my original reading missed the reference to the cache at the beginning of the function.


Add NGINX server support

I guess there are a lot of developers that prefer Nginx instead of Apache
The main problem is that both Nginx and Apache binds to 80-th and 443-d port, and can not work together. And I can't install synthesize and quickly test the Graphite on existing "nginx-machines"

@d3QUone I empathize, but Synthesize is an opinionated installer using a very specific set of technologies. As I don't personally use Graphite with NGINX, I don't have the expertise to support that configuration effectively. IMHO, the official Graphite documentation should include a dedicated document for NGINX installations. I'd recommend opening an issue over there for this.


frontend - correctly abbreviate long token amounts

Currently, in the header of the transfer history, we display a shorten version of the token amount that was sent. The value get shorten to two decimals after the dot. While this seems fine for DAI, this falls of for WETH. So we need a smarter solution here.
Please note that within the expanded body of the transfer, the exact amount is always visible.

The same issue can also be found inside the user's token balance message.


Fix typo: "s/sately/safely/"

Fix typo: s/sately/safely/.
(thank you for your amazing work)

Thanks!


Tensors of type TensorImpl do not have strides

Describe the bug
when I train model by the following command: ns-train nerfacto --data data/nerfstudio/poster
tinycudann raise error
RuntimeError: Tensors of type TensorImpl do not have strides
Where's the problem?
Screenshots

Looks like it is relate to this - https://github.com/NVlabs/tiny-cuda-nn/issues/117

thanks for your reply. it seems to be a compatibility problem with tiny-cudann1.7. I already fixed it by built a new conda environment.


Fix for alpine3.10

Updated the fork from project upstream and cherry picked a fix for Alpine Linux 3.10 and greater out of https://github.com/vishnubob/wait-for-it/pull/81
Context:

https://github.com/vishnubob/wait-for-it/issues/71
https://github.com/vishnubob/wait-for-it/pull/81

Please hold on merging until I can double of check a couple of things. I wanted to get the diff showing so I can see what we are working with here


Typo in Documentation

pack_install(lfbgs). should be pack_install(lbfgs). at the beginning of the file docs/index.rst.

fixed


refactor: add schedule type

Which issue does this PR close?
Closes #
Rationale for this change
The current scheduler implementation will cause CeresDB to fail to restart when using local storage, and cannot open the shard again. We need to add schedulerType to support this startup mode.
What changes are included in this PR?

Add LocalStorageShardScheduler, It allocates shards in a fixed manner and does not allow shards to move.
Add scheduleType config, It will determine CeresMeta's scheduling strategy for CeresDB clusters.

Are there any user-facing changes?
None.
How does this change test
Pass all unit tests and the verification is correct in the local cluster.

Codecov Report

Merging #168 (5a2a048) into main (494e15e) will increase coverage by 0.73%.
The diff coverage is 35.27%.

@@            Coverage Diff             @@
##             main     #168      +/-   ##
==========================================
+ Coverage   39.07%   39.81%   +0.73%
==========================================
  Files          33       34       +1
  Lines        4217     4197      -20
==========================================
+ Hits         1648     1671      +23
+ Misses       2317     2301      -16
+ Partials      252      225      -27

Impacted Files
Coverage Δ

server/cluster/metadata/cluster_metadata.go
0.00% <0.00%> (ø)

server/cluster/metadata/table_manager.go
0.00% <0.00%> (ø)

server/cluster/metadata/topology_manager.go
0.00% <0.00%> (ø)

server/cluster/metadata/types.go
0.00% <0.00%> (ø)

server/coordinator/procedure/manager_impl.go
0.00% <0.00%> (ø)

server/coordinator/procedure/util.go
0.00% <0.00%> (-38.89%)
:arrow_down:

server/etcdutil/util.go
45.45% <0.00%> (-6.79%)
:arrow_down:

server/coordinator/factory.go
4.45% <4.45%> (ø)

server/storage/types.go
63.54% <38.23%> (-7.01%)
:arrow_down:

server/storage/storage_impl.go
62.30% <40.00%> (-1.21%)
:arrow_down:

... and 14 more

... and 2 files with indirect coverage changes


[release-2.6] Update react-form-wizard to 1.8.6

Includes a fix for radio labels not always corresponding to the correct
radio input.
Refs:

https://github.com/stolostron/backlog/issues/25009

Signed-off-by: Justin Kulikauskas<EMAIL_ADDRESS>

/cc @KevinFCormier

/cc @jamestalton

/lgtm


what about running locally?

There should be a section on running app insights locally (which packages to include, where to put the key, etc)

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

ID: 13672b67-22ea-3dda-8b6b-3332c24e6030
Version Independent ID: fdc60aeb-9971-1d9f-a6ba-b3ad818cf0d5
Content: Monitor Azure Functions
Content Source: articles/azure-functions/functions-monitoring.md
Service: azure-functions
GitHub Login: @ggailey777
Microsoft Alias: glenga

@crgarcia12 , Thank you for raising the issue. We will investigate and update you.

@crgarcia12 , We do not recommend running locally with the same instrumentation key, since that could mess up the data being collected from the production app. However, we can try to create a local-only App Insights instance (a separate key) and use that for local collection.

You can specify a dummy instrumentation key in local.settings.json and AppInsights will start working in VisualStudio.

@crgarcia12 , Just wanted to understand if you need any other assistance on this issue. Please let me know if I can go ahead and close this issue.

@crgarcia12 , We have not heard back form you. I will go ahead and close the issue. Please feel free to reopen if you need further assistance.


Blackmagic drivers cannot be installed on windows

There is no manufacturer signature on the blackmagic drivers. So windows throws The third-party INF does not contain digital signaturers. Windows machines cannot communicate over the blackmagic COM ports until this is resolved.

Closing this issue as it appears to be with the bmp itself and the st-link has the same functionality as bmp since #1


renaming interpreter.md

fixes https://github.com/openxla/stablehlo/issues/514
cc: @bhack

The op-level eval as mentioned in (2)

What is (2)?

What is (2)?
Thanks for the catch. Fixed it in the doc.

I will separate more 1 and 2 in two section but if the text is too short it is ok.

Let us keep the bullets as is.
Fixed the indentation.


Conversion data callback reports all installs as organic

In-organic installs (Facebook Ads) showing up on Appsflyer dashboard correctly,
but conversion data callback reports all installs as organic.
(Tested only in Android)
final appsflyerSdk = AppsflyerSdk(AppsFlyerOptions(
  afDevKey: "MYDEVKEY",
  appId: Platform.isIOS ? "IOSKEY" : "ANDROIDKEY",
  showDebug: true,
));

  appsflyerSdk
      .initSdk(
    registerConversionDataCallback: true,
  )
      .then((_) {
    appsflyerSdk.conversionDataStream.listen((data) {
      logger.d(data);
      if (data != null) {
        amplitudeAnalytics.setUserProperties({"conversion_data": data});
      }
    });
  });

Hi @DohanKim. Thank you for opening this issue.
For attribution issues please contact our support team<EMAIL_ADDRESS>

Please reach out to us or<EMAIL_ADDRESS>if you still need help on this matter!


[1548] Alert if retrieve trn or qts/award jobs time out

Context
https://trello.com/c/cetXU8sR/1548-alert-if-retrievetrn-qts-jobs-time-out
Jobs were timing out silently
Changes proposed in this pull request

Adds module with send_message_to_slack method that uses SlackNotifierService. The method is then used in both jobs, module is included. When polling takes longer than 4 days, the job ceases to continue polling and instead uses that method to send a failure message to #twd_publish_register_tech channel.
Adds extra method stub and assertions for job specs

Guidance to review
Will rebase onto master when #797 is merged in

 [M] Alert if RetrieveTRN/QTS jobs time out


Nostr

:tada: This PR is included in version 1.6.0 :tada:
Your semantic-release bot :package::rocket:


pg_ctl: directory "data/postgres" does not exist

When I run a clean install I get no output after executing ./governor.py postgres1.yml however, if I ctrl+c the application I receive the following error:
postgres@sql1:~/governor$ ./governor.py postgres1.yml
^CTraceback (most recent call last):
  File "./governor.py", line 48, in <module>
    time.sleep(5)
KeyboardInterrupt
pg_ctl: directory "data/postgres" does not exist

Why is the data directory not begin created during first time run?

Creating the data directory manually gives a new error:
$ mkdir -p data/postgres
postgres@sql1:~/governor$ ./governor.py postgresql.yml
^CTraceback (most recent call last):
  File "./governor.py", line 48, in <module>
    time.sleep(5)
KeyboardInterrupt
pg_ctl: directory "data/postgres" is not a database cluster directory

So after adding some more debugging I am actually stuck at
        while not synced_from_leader:
            logging.info("I am not in sync")
            leader = etcd.current_leader()
            print(leader)
            if not leader:
                logging.info("I am not the leader, waiting 5 seconds")
                time.sleep(5)
                continue

because my leader is None. Not sure how to recover from that..

curl http://<IP_ADDRESS>:2379/v2/stats/leader
{"message":"not current leader"}

Ok, I had to clear/rm the initialize key as there was probably a previous value defined?
postgres@sql2:~/governor$ etcdctl -o extended get /service/batman/initialize
Key: /service/batman/initialize
Created-Index: 8
Modified-Index: 8
TTL: 0
Etcd-Index: 54
Raft-Index: 520353
Raft-Term: 9

sql1

Typically, if you are starting a new test with governor, it is best to stop etcd and any governor processes.  Then, run rm -rf data/* from the governor directory.  Then, restart etcd + the governor processes.


Question

Is possible just use player name expect from using uuid?

Hey yichifauzi, no, with the current version it isn't possible to use the name instead of tje uuid. It is possible to use the uuid command to retrieve the uuid from the username and then pass it to the player command directly with some slight modifications. I'll be working on this issue as soon as possible and notify you when it will be implemented

@yichifauzi your request has been implemented in version 2.1.1, you might want to check it out.
I'm closing the issue as it has been resolved. If you have any questions, feel free to create a new issue or reopen this one.
I hope this was helpful.


test: add tests for LSP1 Type IDs

What does this PR introduce?
🧪 Tests
This PR introduces tests for LSP1 Type IDs hashes, so they are consistent in both javascript and solidity

PR Checklist

[ ] Wrote Tests
[ ] Wrote & Generated Documentation (readme/natspec/dodoc)
[ ] Ran npm run lint && npm run lint:solidity (solhint)
[ ] Ran npm run format (prettier)
[ ] Ran npm run build
[ ] Ran npm run test

Task linked: DEV-8861 Add tests for verifying the LSP1 TypeIDs and ERC725YDataKeys in constants.ts


Provide a tooltip for the path displayed in the list of recipes

Bug description
The recipe list is now showing the path where a recipe has been cloned.
Unfortunately, the path is too long and there is no way to see it all.

There should be a tooltip that is at least showing "Cloned at path: "
Finally, what's the value of displaying this path here? Is the user supposed to be able to copy it?
Shouldn't it be in the recipe's details, rather than on the catalog?
Operating system
Ventura
Installation Method
None
Version
next (development version)
Steps to reproduce
No response
Relevant log output
No response
Additional context
No response

Duplicates of https://github.com/containers/podman-desktop-extension-ai-lab/issues/1214


Feature/using-pvc-as-lower-bazel-out-layer

https://do.pingcap.net/jenkins/blue/organizations/jenkins/pingcap%2Ftidb%2Fghpr_build/detail/ghpr_build/2720/pipeline
https://do.pingcap.net/jenkins/blue/organizations/jenkins/pingcap%2Ftidb%2Fghpr_check/detail/ghpr_check/339/pipeline
https://do.pingcap.net/jenkins/blue/organizations/jenkins/pingcap%2Ftidb%2Fghpr_check2/detail/ghpr_check2/284/pipeline
https://do.pingcap.net/jenkins/blue/organizations/jenkins/pingcap%2Ftidb%2Fghpr_unit_test/detail/ghpr_unit_test/2018/pipeline
https://do.pingcap.net/jenkins/blue/organizations/jenkins/pingcap%2Ftidb%2Fghpr_unit_test/detail/ghpr_unit_test/2017/pipeline


📝 Add Front Door Process

Add front door process information (links / steps / meetings etc) to the team site.

Tom to get instructions/text for the update


fix conflict with class mysql::backup::xtrabackup

Currently if you want to use mysql::backup::xtrabackup to backup your Galera cluster you'll hit a duplicate declaration error:

Error: Could not retrieve catalog from remote server: Error 400 on SERVER: Evaluation Error: Error while evaluating a Resource Statement, Duplicate declaration: Package[percona-xtrabackup] is already declared; cannot redeclare at /etc/puppet/environments/development/modules/mysql/manifests/backup/xtrabackup.pp:25

Both classes galera and mysql::backup::xtrabackup require the package to be installed. My patch makes it possible to control the installation of these additional packages by setting $manage_additional_packages to false to resolve this conflict.

FWIW, the failed tests are not related to my PR:

Gem::InstallError: beaker requires Ruby version >= 2.2.5.
An error occurred while installing beaker (3.2.0), and Bundler cannot continue.


Updated documentation for the examples

Codecov Report
Base: 95.98% // Head: 95.98% // No change to project coverage :thumbsup:

Coverage data is based on head (1e6d000) compared to base (7a8953f).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #24   +/-   ##
=======================================
  Coverage   95.98%   95.98%
=======================================
  Files           7        7
  Lines         274      274
  Branches       34       34
=======================================
  Hits          263      263
  Misses          6        6
  Partials        5        5

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Move override scripts to respective repos

Is your feature request related to a problem? Please describe.
Custom bundle build scripts in https://github.com/opensearch-project/opensearch-build/tree/main/bundle-workflow/scripts/components need to be moved into individual repositories.
Describe the solution you'd like
Add these scripts to each repo under scripts/build.sh, and delete them from https://github.com/opensearch-project/opensearch-build/tree/main/scripts/bundle-build/components.
Additional context
Leftover from #150, part of https://github.com/opensearch-project/opensearch-build/issues/133

[Triage] @dblock is this isssue still valid? Looks like Meta issue is closed

Yes, we still want this, i.e. deleting everything under https://github.com/opensearch-project/opensearch-build/tree/main/scripts/components.

Hi @prudhvigodithi ,
This isse is different than the one you specified. This is about moving all the component scripts, build.sh and integtest.sh scripts to respective repositories than maintaining in build repo which might different in version. See db's comment above.

In past I have migrated build.sh to respective repos, https://github.com/opensearch-project/opensearch-build/pull/2471#issuecomment-1218224133. We should list of what is remaining start to create a campaign.

I still below component scripts in build repo: https://github.com/opensearch-project/opensearch-build/tree/main/scripts/components

OpenSearch
functionalTestDashboards
notifications-core
OpenSearch-Dashboards
alerting
notifications
performance-analyzer

@peterzhuamazon Wondering if we can move below components scripts to their own repo:

OpenSearch
OpenSearch-Dashboards
alerting
performance-analyzer


Upgrade go to 1.20.5

Pull Request Test Coverage Report for Build b5739ffaa-PR-931

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 46.894%

Totals

Change from base Build 62bfffe1d:
0.0%

Covered Lines:
6553

Relevant Lines:
13974

💛  - Coveralls


chore: refactor of AtomM

There's no functional change here. I just found it weird that AtomM.Context was carrying around evalAtom, but none of the AtomM functions used it, and instead internal users had to retrieve it and use it themselves.

LGTM from what's here, but aren't there more users of the interface not mentioned here? The behavior of addAtom is changed in this PR so you should be sure to change all the client code too.

Maybe a better alternative is to remove evalAtom from AtomM entirely, and have code that uses AtomM manage it (as they essentially do already: they just mysteriously store it in the AtomM.Context...)

This was never intended to be a black box abstraction, it's just common that you need to store a list of expressions and find things in them. The evalAtom is in there because it was the cleanest way to express the recursive simping that ring_nf does, and it turns out that most of the other uses are also doing some pre-simplification before interning.

I guess if you're happy with this moving up to Std eventually as-is, then I don't really mind about this detail.

How about leaving addAtom as is and adding addAtomWithResult like in the PR?


sbt compilation failure

Hi, I would like to contribute a PR but am getting this error when compiling:
jwt-scala(master)$ sbt compile
[info] Loading settings from plugins.sbt ...
[info] Loading project definition from /git/jwt-scala/project
[info] Loading settings from build.sbt ...
[info] Resolving key references (15488 settings) ...
usage: git ls-remote [--heads] [--tags]  [-u <exec> | --upload-pack <exec>] <repository> <refs>...
[error] java.util.NoSuchElementException: head of empty stream
[error]         at scala.collection.immutable.Stream$Empty$.head(Stream.scala:1104)
[error]         at scala.collection.immutable.Stream$Empty$.head(Stream.scala:1102)
[error]         at com.typesafe.sbt.SbtGit$.$anonfun$buildSettings$21(SbtGit.scala:138)
[error]         at sbt.internal.util.Init$Value.$anonfun$apply$3(Settings.scala:810)
[error]         at sbt.internal.util.EvaluateSettings.$anonfun$constant$1(INode.scala:202)
[error]         at sbt.internal.util.EvaluateSettings$MixedNode.evaluate0(INode.scala:219)
[error]         at sbt.internal.util.EvaluateSettings$INode.evaluate(INode.scala:164)
[error]         at sbt.internal.util.EvaluateSettings.$anonfun$submitEvaluate$1(INode.scala:87)
[error]         at sbt.internal.util.EvaluateSettings.sbt$internal$util$EvaluateSettings$$run0(INode.scala:98)
[error]         at sbt.internal.util.EvaluateSettings$$anon$3.run(INode.scala:94)
[error]         at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
[error]         at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
[error]         at java.lang.Thread.run(Thread.java:745)
[error] java.util.NoSuchElementException: head of empty stream
[error] Use 'last' for the full log.
Project loading failed: (r)etry, (q)uit, (l)ast, or (i)gnore?

Any idea what it could be?
sbt -v
[process_args] java_version = '1.8'
# Executing command line:
java
-Xms1024m
-Xmx1024m
-XX:ReservedCodeCacheSize=128m
-XX:MaxMetaspaceSize=256m
-jar
/libs/sbt-1.1.1/bin/sbt-launch.jar

Which Git version are you using? Try updating it maybe...

Exactly the problem. Using git 2.7.2 and issue is gone.

I am having the exact same problem on Red Hat Linux 2.6 , git version 2.9.5, sbt 1.2.1. What can I do to fix this?

@radumanolescu 	I have no idea myself... It looks like an sbt problem to me. Have you tried opening an issue on their repo?

@pauldijou I made sure I was running bash (not /bin/sh) and the version of git was >= 2.7.2


Fix and enforce linter

Description

This gets the repo to a baseline of linting based on the .golangci.yml file generated from running lint-install.
Why is this needed

Fixes: #
How Has This Been Tested?

How are existing users impacted? What migration steps/scripts do we need?

Checklist:
I have:

[ ] updated the documentation and/or roadmap (if required)
[ ] added unit or e2e tests
[ ] provided instructions on how to upgrade

fantastic!. A couple of minor comments and one nit. The nit is that you're mixing 2 different ways to define functions whose arguments are all ignored. Some changes you're doing here are like:
func(_ type1, _ type2, _ type3)

and
func(type1, type2, type3)

I prefer the latter and I think its the most common way. Can you make it so only one way is used please?

yeah, seems reasonable. updated.

:( I always forget to check of the "viewed" button while reviewing and then need to go through the whole thing again later ...


#933: fix: Fixes output of LPUSH command

Fixes #933 by returning the length instead of OK.
Minor side change - Integration test for RPUSH used LPUSH command like below, which was fixed
https://github.com/DiceDB/dice/blob/887d693113e266c8525c75b103346ced5fbdf997/integration_tests/commands/async/deque_test.go#L124
Would also like to point out the existence of a duplicate issue #961

Closing, merged as part of https://github.com/DiceDB/dice/pull/969


Implement submenus for analytics

We need the following sub menus

Downloads (App downloads + Magazine downloads)
Visits
Users

No longer needed.


onnxruntime: 1.0.0 -> 1.1.0

Motivation for this change
bumping packages I maintain
had to disable LTO due to https://github.com/microsoft/onnxruntime/issues/2828
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 pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build onnxruntime

@GrahamcOfBorg eval


Should it work on mobile web ?

When test it on mobile it run ok on android, but when try to run it on ios safari, i get:
TypeError: this.node.animate is not a function. (In 'this.node.animate(this.keyframes, this.timing)', 'this.node.animate' is undefined)
startAnimation — animation.js:63
componentDidMount — animation.js:99
measureLifeCyclePerf — ReactCompositeComponent.js:75
(anonymous function) — ReactCompositeComponent.js:264
notifyAll — CallbackQueue.js:76
close — ReactReconcileTransaction.js:80
closeAll — Transaction.js:206
perform — Transaction.js:153
perform — Transaction.js:140
flushBatchedUpdates — ReactUpdates.js:172
flushBatchedUpdates
closeAll — Transaction.js:206
perform — Transaction.js:153
dispatchEvent — ReactEventListener.js:147
dispatchEvent
https://s3.amazonaws.com/infash-assets-staging/brands_icons/tommy_hilfiger_unpressed.png

@bennyhan9 it should.. What version of safari are you running on?

@bennyhan9, I tested it recently on mobile safari and it work with the polyfill loaded


Would it be possible to publish new version to crates.io? (0.26.5)

Thanks a lot! Sorry for bothering you! 🙏

@callanoc sure.

@callanoc published:
https://crates.io/crates/similari-trackers-rs

Thank you so much!!


Inferred ints can take floats in a release build

Godot version:

3.2.3 rc6
Not sure about master
Issue description:

Check this code:
extends Node2D

var test1 := 120
var test2: int = 120

func _ready() -> void:
	test1 = 120.15
	test2 = 120.15
	$Label.text = str(test1)
	$Label2.text = str(test2)

When you run it in editor or debug build, both labels show 120. In a release build, the first label shows 120.15
Minimal reproduction project:

ReproductionProject.zip

I can reproduce this on 3.2.2 (official Linux release export template).

Impossible to test in master due to #42127

Can't reproduce in https://downloads.tuxfamily.org/godotengine/testing/4.0/4.0-dev.20210727/
I mean it shows 120 in release build.

Here is a project with exports to check. I hope it helps

In 3.3.2:

var test1 := 120 fail
var test2: int = 120 works

Confirmed fixed in master, so this is a 3.x only issue now.

Fixed by #57851.


peer_certs() not available in middleware layer

Issue

Version
tonic v0.9.0
tonic-build v0.9.0
Platform
22.4.0 Darwin Kernel Version 22.4.0: Mon Mar  6 20:59:28 PST 2023;
root:xnu-8796.101.5~3/RELEASE_ARM64_T6000 arm64
Description
I need to be able to authenticate the certificate serial number in the case of mTLS.
For this I need a connection to a database.
So I modified the middleware example to this:
#[derive(Debug, Clone, Default)]
struct MyMiddlewareLayer {
    conn: DatabaseConnection,
}

impl<S> Layer<S> for MyMiddlewareLayer {
    type Service = MyMiddleware<S>;

    fn layer(&self, service: S) -> Self::Service {
        MyMiddleware {
            inner: service,
            conn: self.conn.clone(),
        }
    }
}

#[derive(Debug, Clone)]
struct MyMiddleware<S> {
    inner: S,
    conn: DatabaseConnection,
}

impl<S> Service<hyper::Request<Body>> for MyMiddleware<S>
where
    S: Service<hyper::Request<Body>, Response = hyper::Response<BoxBody>> + Clone + Send + 'static,
    S::Future: Send + 'static,
{
    type Response = S::Response;
    type Error = S::Error;
    type Future = futures::future::BoxFuture<'static, Result<Self::Response, Self::Error>>;

    fn poll_ready(&mut self, cx: &mut Context<'_>) -> Poll<Result<(), Self::Error>> {
        self.inner.poll_ready(cx)
    }

    fn call(&mut self, mut req: hyper::Request<Body>) -> Self::Future {
        // This is necessary because tonic internally uses `tower::buffer::Buffer`.
        // See https://github.com/tower-rs/tower/issues/547#issuecomment-767629149
        // for details on why this is necessary
        let clone = self.inner.clone();
        let mut inner = std::mem::replace(&mut self.inner, clone);

        req.extensions_mut().insert(AppState {
            connection: self.conn.clone(),
        });

        Box::pin(async move {
            // Do extra async work here...
            let response = inner.call(req).await?;

            Ok(response)
        })
    }
}

And how I pass the connection:
 Server::builder()
        .layer(MyMiddlewareLayer { conn: conn.clone() })
        .tls_config(tls)?
        .add_service(
            UsersServer::new(UserOperations {}),
        )
        .serve(addr)
        .await?;

So now I have a connection pool available in the middleware, however, the request does not have .peer_certs() so I can extract the client cert used and validate the serial number against a database.
I tried to access it in the call fn:
fn call(&mut self, mut req: Request<Body>) -> Self::Future {
...

What would be my solution here?

This is a bit hacky but you could try to https://docs.rs/tonic/latest/tonic/struct.Request.html#method.from_http convert it into a tonic request which will then allow you to call that method. Then you convert back into a http::Request via into_parts. You then will have to convert each part into their http variant via https://docs.rs/tonic/latest/tonic/metadata/struct.MetadataMap.html#method.into_headers and https://docs.rs/tonic/latest/tonic/struct.Extensions.html#method.into_http and it should work. Its a bit hacky but should work for now.

I ended up doing this:
req.extensions()
      .get::<TlsConnectInfo<TcpConnectInfo>>()
      .and_then(|i| i.peer_certs())
      .and_then(|certs| certs.first().cloned())


Automated cherry pick of #5714: fix: decouple getUserInfo at apigateway from capabilities call

Cherry pick of #5714 on release/3.1.
#5714: fix: decouple getUserInfo at apigateway from capabilities call

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/3.1@ddda51d). Click here to learn what that means.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             release/3.1   #5717   +/-   ##
=============================================
  Coverage               ?   7.44%
=============================================
  Files                  ?     583
  Lines                  ?   98366
  Branches               ?       0
=============================================
  Hits                   ?    7322
  Misses                 ?   90473
  Partials               ?     571

Flag
Coverage Δ

#aFlag
7.44% <0.00%> (?)

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 ddda51d...b29d752. Read the comment docs.


Gallery item icon not perfect square

after #10066

gallery item in share

perfect square please
40x40

Originally posted by @exezbcz in #10066 (comment)

remove class here
GalleryItemMoreActionBtn.vue

and update here

after #10066
gallery item in share

perfect square please
40x40

Originally posted by @exezbcz in #10066 (comment)

remove class here
GalleryItemMoreActionBtn.vue

and update here

I think we should remove the icon action class instead since min-w-10 and h-10 is the way to go now according to figma? minimum is w-10

Has this issue been resolved?

@asterism0115 nope, up for grabs

👋

👋


Question regarding input of Pose Refinement?

Hi, I am trying to use colmap for pose correction and I end up here. I was wondering what's the length of tvec and qvec in the input? Should it be the same as the length of 2D-3D points (N)? Also, I have different intrinsics for each view. How I should feed it to pose refinement? it looks it only accept the same intrinsics for all views. Also, I don't understand exactly what's inlier_mask?
Please let me know if there is documentation for pose refinement. Thanks!

The pose refinement only refines one pose / camera at a time. If you have multiple cameras you will need to call the function multiple times, once for each set of 2D-3D matches, initial pose, and intrinsics.
The qvec is a 4-dimensional vector and tvec is a 3-dimensional one representing the initial pose in COLMAP convention (https://colmap.github.io/format.html). Inlier mask is a boolean vector which marks as True the 2D-3D matches which are inliers (i.e., the ones to be considering during optimization). If all your matches are to be considered, you can pass a vector full of True with the size N.

Ok. Great. Thanks @mihaidusmanu.


chore: fix prettier changelog

:tada: This PR is included in version 1.1.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Updated System.Runtime.CompilerServices.Unsafe package

Updated System.Runtime.CompilerServices.Unsafe so the calls to Unsafe members in ExternalProcessMemory are resolvable.

Whoa, how did I even manage to break it with such a small change? Works fine for me, even after cloning freshly.

LGTM


chore: fix broken dev reload

[Future-Friday]
Description
This PR fixes the broken dev reload inside Eigen.
Context: When we first updated our navigation infra, we had a few attempts to fix this and it worked if the app didn't crash but it never worked if the app crashed. This made the DX unfortunately painful when you are building feature and make mistakes that require an app refresh.

Note: The same hook needs to be used in nested independent navigation containers + it can also be used in Energy in case we have issues with that there.

graph TB
    A[Start] ==> B{DEV?};
    B -->|Yes| C{Same Session};
    C --> |Yes | D{Cached Nav State};
    D --> E[Restore State];
    %% D -.-> B;
    B ---->|No| F[Go to Home];
    C ---->|No| F[Go to Home];
    D ---->|No| F[Go to Home];

Before
https://github.com/artsy/eigen/assets/11945712/6bc51eeb-7f53-4370-ae8f-c80143685d97
After
https://github.com/artsy/eigen/assets/11945712/e18a0473-1b3d-4a12-a17a-a31ad0cd722a

PR Checklist

[x] I have tested my changes on iOS and Android.
[x] I hid my changes behind a feature flag, or they don't need one.
[x] I have included screenshots or videos, or I have not changed the UI.
[x] I have added tests, or my changes don't require any.
[x] I added an app state migration, or my changes do not require one.
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added a changelog entry below, or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

iOS user-facing changes

Android user-facing changes

Dev changes

fix broken dev reload - mounir

Need help with something? Have a look at our docs, or get in touch with us.

This PR contains the following changes:

Dev changes (fix broken dev reload - mounir)

  Generated by :no_entry_sign: dangerJS against 8fdfc7fb11558873b90b40bf6e88bbe55ca696b2

Merging this now to get time to test it before the next release - glad to address any bugs raised in a separate PR

Thank you!


Multiple breaking changes

Codecov Report
Base: 100.00% // Head: 99.61% // Decreases project coverage by -0.38% :warning:

Coverage data is based on head (9bd0c8b) compared to base (a7b4336).
Patch coverage: 93.17% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff             @@
##            master     #383      +/-   ##
===========================================
- Coverage   100.00%   99.61%   -0.39%
===========================================
  Files          110      114       +4
  Lines        10389    10482      +93
===========================================
+ Hits         10389    10442      +53
- Misses           0       40      +40

Impacted Files
Coverage Δ

fugue/bag/array_bag.py
0.00% <0.00%> (ø)

fugue/execution/factory.py
100.00% <ø> (ø)

fugue/extensions/transformer/transformer.py
100.00% <ø> (ø)

fugue/bag/bag.py
77.77% <77.77%> (ø)

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

fugue/bag/__init__.py
100.00% <100.00%> (ø)

fugue/collections/__init__.py
100.00% <100.00%> (ø)

fugue/collections/partition.py
100.00% <100.00%> (ø)

fugue/dataframe/array_dataframe.py
100.00% <100.00%> (ø)

fugue/dataframe/arrow_dataframe.py
100.00% <100.00%> (ø)

... and 30 more

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


git pull to fetch latest changes.

this has a hard git commit in dockerfile that I think is causing problems with updated code.
https://github.com/cghawthorne/magenta/blob/246e8ea3483fc90dcd3c5c4347ca0bcb64156e1a/magenta/tools/docker/Dockerfile


Nuke: Load full model hierarchy by default

Brief description
Fix Nuke model loader to select all geo nodes in the hierarchy for loading by default
Description
The current implementation only selects top level geo nodes form the hierarchy, this fixes it by always selecting all geo nodes
Additional info
Nuke pops up the Hierarchy window during Alembic loading to show which geo nodes are selected for loading. To avoid this popup, you have to set the following in preferences:
preferencesNode = nuke.toNode('preferences')
preferencesNode.knob('DfltAbcAlwaysCreateAllInOne').setValue(True)

We put this into Nuke's init.py in our pipeline, we didn't want to hardcode it in here.  Without it loading/version switching especially on multiple geo nodes is really annoying.  Maybe there's a more streamlined solution to avoid this window?

Can you use the scene API?
# Ensure all items are imported and selected.
        scene_view = node['scene_view']
        scene_view.setImportedItems(scene_view.getAllItems())
        scene_view.setSelectedItems(scene_view.getAllItems())

It works flawlessly, thanks!

Thank you @gabormarinov


Decrease minimum allowed value of maxScreenLineLength from 500.

Summary
Propose reducing the minimum allowed value of maxScreenLineLength. As of this writing, the default value is 500, and the minimum value is also 500. Suggest that a value of 80 might be a better minimum.
Motivation
Some people don't like to scroll back and forth on long lines. The existing enforced minimum value does not allow such people to configure Atom in a way that is friendly to their needs / desires.
Describe alternatives you've considered
AFAIK there isn't an alternative other than changing the minimum as defined in src/config-schema.js in this repo. E.g., if you try to set the value to less than 500 right now (e.g. by manually editing your config.cson, Atom will remove the setting the next time it writes out that file.
Additional context
Section of the schema file in question that would need to be updated: https://github.com/atom/atom/blob/master/src/config-schema.js#L459
      maxScreenLineLength: {
        type: 'integer',
        default: 500,
        minimum: 500,
        description: 'Defines the maximum width of the editor window before soft wrapping is enforced, in number of characters.'
      },

Thanks for the suggestion.
This value is intended to be used to enforce a minimum performance standard on excessively long lines, not to create a soft wrap boundary in a pane item that is wider than a certain size. For that, see the settings "Soft Wrap At Preferred Line Length" and the "Preferred Line Length" settings.
I hope that helps!

Near as I can tell, the settings you mention don't do what I'm trying to achieve. In particular, it looks like if I set my "preferred length" to (say) 250 and turn on both "soft wrap" and "soft wrap at preferred length", Atom will still wrap to my window width if that's less than my preferred length.
What I'm looking for is to have soft wrapping occur at a particular width, regardless of my window geometry. Near as I can tell, setting maxScreenLineLength does exactly what I want, except that it has a strangely-large enforced minimum.

@danfuzz The maxScreenLineLength is not really a user setting; it's for performance, but some users complained it was too low.
The part about soft wrap not using the windows width is interesting though. It should be a straightforward change to make this optional.

Thanks for the explanation. I'd certainly be happy with the outcome you describe, though (rhetorically speaking) I remain confused why the existing setting can't be "retconned" into a bona fide user setting, especially since it's already exposed to users in the main Editor settings panel.

So it goes. I appreciate your effort nonetheless. Thanks!


"MdInputDirective" has no "focus" method

Bug, feature request, or proposal:
Might be a bug.
What is the expected behavior?
"MdInputDirective" could call focus
What is the current behavior?
Could not call "focus" after getting "MdInputDirective"
What are the steps to reproduce?
Template is
<md-input-container > 	  <input  mdInput  placeholder="Owner" #owner> </md-input-container>
TS is
ViewChild('owner') ownerInput : MdInputDirective;
But I could not call ownerInput.focus() which is documented.
Instead I need to call as this.ownerInput['nativeElement'].focus();
And I also try ContentChild('owner') ownerInput : MdInputDirective like md container does, but owner will be empty.
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Angular is 4.1.2, material is 2.0.0-beta.6, Typescript is "~2.2.0"

Provider a plunker reproduction.

http://plnkr.co/edit/uAWJWQwbpVdMEp2Ge6Gl?open=app%2Fapp.component.ts&p=preview
And I try the contentChild, not work either. But it the doc of input control, there is a method called focus exposed under MdInputDirective.
I do not know if it is by design or wrong usage?

Try this
@ViewChild(MdInputDirective) ownerInput : MdInputDirective;

or
@ViewChild('owner', { read: MdInputDirective }) ownerInput: MdInputDirective;

thanks a lot.
But first one might be used only when there is only one MdInputDirective in the parent, right?
@fxck  Could you explain me a little bit about the mechanism behind, and why my code does not work?

Have you read the documentation of ViewChild? https://angular.io/docs/ts/latest/api/core/index/ViewChild-decorator.html

yes, but not quite understand. I will search the web and figure out what they means.
Thanks all your guys.


Let's rename masternode privkey to pairingkey

hi all !
One of Dash forks (Dynamic, DYN), uses the term dynode pairingkey, which reflects it's purpose much better than privkey.
privkey term is ambiguous, because it can refer to masternode privkey (pairing key) or to callateral private key (1000 DASH).
dynamic.conf:
dynode=1
dynodepairingkey=5...
Not functional change.

Thanks for the idea! This old term is going away in 12.4 with the split into Operator and Voting keys. See DIP 3 for more info.

The new key names were actually implemented in this PR from a few days ago: https://github.com/dashpay/dash/pull/2248
I'll go ahead and close this, and feel free to add more info if your concern wasn't addressed.


LCM of two numbers using recurion

💥 Proposal
I would like to add the program for finding the LCM of two numbers using recursion in C++. LCM is the smallest common multiple of two integers a and b.
Have you read the Contributing Guidelines on Pull Requests?
Yes

/assign


327 rework transition modelling

Merge Request - GuideLine Checklist
Guideline to check code before resolve WIP and approval, respectively.
As many checkboxes as possible should be ticked.
Checks by code author:

[x] There is at least one issue associated with the pull request.
[x] The branch follows the naming conventions as defined in the git workflow.
[x] New code adheres with the coding guidelines
[ ] Tests for new functionality has been added
[ ] A local test was succesful
[ ] There is appropriate documentation of your work. (use doxygen style comments)
[ ] If new third party software is used, did you pay attention to its license? Please remember to add it to the wiki after successful merging.
[ ] If new mathematical methods or epidemiological terms are used, has the glossary been updated ? Did you provide further documentation ?
is present or referenced. Please provide your references.
[ ] The following questions are addressed in the documentation*:  Developers (what did you do?, how can it be maintained?), For users (how to use your work?), For admins (how to install and configure your work?)
For documentation: Please write or update the Readme in the current working directory!

Checks by code reviewer(s):

[ ] Is the code clean of development artifacts e.g., unnecessary comments, prints, ...
[ ] The ticket goals for each associated issue are reached or problems are clearly addressed (i.e., a new issue was introduced).
[ ] There are appropriate unit tests and they pass.
[ ] The git history is clean and linearized for the merge request.
[ ] Coverage report for new code is acceptable.

closes #327

Please ignore the tests and rather check for logical and stylistic changes.

@dabele Would be nice if you could have a look, in particular about the new structure. Please contact me if you see parts that can be improved. Thanks!


[BUG] MSI authentication fails due to expiry date format mismatch

Driver version
7.4.1
Client details
Running on an Azure App Server Java 11 / Linux instance in the UK west region
OS: Alpine 3.8 Docker container
JVM: OpenJDK Runtime Environment 19.1-(Zulu-11.29+3-linux-musl-x64)-Microsoft-Azure-restricted (build 11.0.2+7-LTS)
Problem description

Configure an App Services resource (Java 11 in "UK West" region) and Azure SQL DB for authentication using MSI, following instructions here
Deploy code that attempts MSI authentication, e.g.:

//...
public static void main(String[] args) throws Exception {
    SQLServerDataSource ds = new SQLServerDataSource();
    ds.setServerName("my-db-instance.database.windows.net");
    ds.setDatabaseName("my-db-name");
    ds.setAuthentication("ActiveDirectoryMSI");
    Connection connection = ds.getConnection();
}
//...

The connection fails, with an opaque "MSI Token failure: Failed to acquire token from MSI Endpoint" error.

Unfortunately there's no stack trace or log output that shows the underlying problem (even at java.util.logging.Level.ALL), but by retrying with a patched version of the driver with extra logging I was able to narrow the issue down to SQLServerConnection.java:4376 which throws a java.text.ParseException when attempting to parse the token expiry timestamp.  The driver implementation is looking for a 12hr time format - "M/d/yyyy h:mm:ss a X", but the MSI API server (for my App Service instance in UK West at least) is returning a 24hr format, e.g.
"08/30/2019 01:08:02 +00:00".
It's not clear to me whether the issue is with the MSI server (possibly just in some regions?) or the driver.  Even if the issue is with the MSI server (in which case I'd appreciate advice on how to raise such an issue), I'd suggest it's probably a bug that the relevant exception details are swallowed by the driver without any logging.  I've attached the full log traces from the unmodified 7.4.1 driver from the start of the MSI request so you can see what I mean.
output-trimmed.log
All best,
Joe

Hi @yjo, thanks for letting us know. I haven't had a chance to look into this today, but I'll look into this and get back to you shortly.

Hi @yjo, sorry for the wait. I've talked to an Azure engineer and it's an error that the expiry date information is coming in a String format, and it should've been an integer value instead. I will make changes to the driver to reflect that when they deploy the changes to Azure, and that will fix this issue as well. The ballpark time for when the changes will be deployed on Azure is within the next month or two.
In the meantime, I do have a fix for the driver that fixes this issue against UK West region. Would you like to use it, or are you okay with waiting until the fix is deployed on Azure?

Hi @peterbae, I think we're happy working around the issue with another authentication method until the MSI fix is on the mainline release.  Many thanks for looking into this issue!

It's too bad I only find this issue after a long day of troubleshooting my first Docker/Tomcat app service deployment with JDBC connection (West-Europe). Initially I thought my configuration was wrong, until I found out that the service actually returned a valid token. The hardest part was figuring out what went wrong due to the absence of a stacktrace. I eventually compiled my own driver version which did log a stacktrace to find the exact same problem.
I'm happy to read that a solution is on its way. Meanwhile, I can use my own patched driver.

@peterbae Any progress on when this fix will be deployed to Azure?

We're not the team that deploys the fix to Azure. The last I heard (a week ago), the fix has been checked in but the release is TBD.

Update, the fix has been deployed on Windows but not the Linux deployed Azure web apps. The fix for Linux should be deployed on Azure in February or March.

Also attaching a patched version of the 8.2.0 driver with a fix for this problem. The fix will not be merged as the problem will likely solve itself when the Azure team deploys their fix.
github-1135-temp.zip

Issue has been fixed on Azure's end, and PR #1308 contains the driver jar that accommodates this fix.

#1308 merged.

Still doesn't seem to work in 8.2.2.jre8 release. Same setup: AppService, Linux app plan + North Europe region.
{"access_token":"","expires_on":"06/27/2020 12:14:35 +00:00","resource":"https://database.windows.net/","token_type":"Bearer","client_id":""}
java.text.ParseException: Unparseable date: "06/27/2020 12:14:35 +00:00"

@walk0r this fix is available starting from v.8.3.1 release.

@peterbae true, my bad. Are there any due dates for stable release?

The next stable release is targeted to be released by end of this July.


Allow options.wdioConfig to support absolute and relative paths

Current Behavior
options.wdioConfig must be located in the project root, else generated wdio config improperly references the file.
Expected Behavior
options.wdioConfig should support config files located at arbitrary file system locations.
The generated wdio config file should properly reference the config file in its import statement.

@Roozenboom What would you need added/removed/changed in this PR to merge it in to main?

LGTM 🎉


Pass query parameters to runOnDemand commands

Which version are you using?
r557.93ba061-1
Enhancement request
It would be nice if it would be possible to pass the full URL given to the server to runOnDemand commands as i would like to parse the query parameters of a given URL in order to react on them. Ideally it would be possible to get the full URL given to the server but any method to pass on query parameters would be fine. With sufficiently helping hints i might be able to implement this by myself but currently i feel a little lost. it looks like the environment gets initialized here but i am not able to determine how i would be able to add the query parameters there. How is this initialized when using regexp matches?

Hello, this could be implemented but it's difficult - paths are initialized and stored exclusively by name, the query part of the request is discarded.


update etl and spinning sales contract

Coverage decreased (-0.08%) to 63.796% when pulling ccba0c49304da44f20f8d648fa0ab27dc2efe267 on leslieaula:dev into 85d398595a3d7284147c926fb0d96efa95af82c2 on danliris:dev.


add warn_once

resolves #

Description

Checklist

[ ] I have read the contributing guide and understand what's expected of me
[ ] I have signed the CLA
[ ] I have run this code in development and it appears to resolve the stated issue
[ ] I have opened an issue to add/update docs, or docs changes are not required/relevant for this PR
[ ] I have run changie new to create a changelog entry

Codecov Report
Attention: Patch coverage is 66.66667% with 2 lines in your changes missing coverage. Please review.

Project coverage is 67.35%. Comparing base (98ccfce) to head (678d5dd).

Files with missing lines
Patch %
Lines

dbt_common/behavior_flags.py
66.66%
2 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #196      +/-   ##
==========================================
- Coverage   67.35%   67.35%   -0.01%
==========================================
  Files          52       52
  Lines        3370     3376       +6
==========================================
+ Hits         2270     2274       +4
- Misses       1100     1102       +2

Flag
Coverage Δ

unit
67.35% <66.66%> (-0.01%)
:arrow_down:

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

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


2.0.0-preview023 - message is not defined on large requests with relations

I have this schema:

model A {
  id         String      @aliedp
  b     B?
}

model B {
  id String @id
}

I have ~2000 A's in the DB. It's a postgresSQL db running  on google cloud.
if I do a graphq-request to get all a's, i get a response:
as {
 id
}

# returns all entries

but if i want to join in B and fetch all 2000 entries, i get an error
as {
 id
 b {
   id
}
}

it will return:
{
  "errors": [
    {
      "message": "message is not defined",
      "locations": [
        {
          "line": 4,
          "column": 5
        }
      ],
      "path": [
        "a",
        1,
        "b"
      ],
      "extensions": {
        "code": "INTERNAL_SERVER_ERROR"
      }
    },
    {
      "message": "message is not defined",
      "locations": [
        {
          "line": 4,
          "column": 5
        }
      ],
      "path": [
        "a",
        2,
        "b"
      ],
      "extensions": {
        "code": "INTERNAL_SERVER_ERROR"
      }
    },
// and so on

if i add a limit (say first 10 entries), it will return (not fast, but without error).
it seems that this overloads the db or so. It seem to have worked in previous version (021) that we used.
I know that this request is too large anyway, but it should not crash the db.
Also the "message is not defined"-error seems to be this: https://github.com/prisma/prisma2/issues/1758, this other issue prevents debugging this, because it does not show the real error behind this.

Can you please share the GraphQL server implementation that you are using? I wasn't able to reproduce this with my initial tries.


add menu action for draw view and man view

I want close menu in draw view and show draw view in nav bar.
:)

hmm, can you explain more about your use case? Most people expose actions through the root component, and either provide it via props or context. I normally use https://github.com/gnoff/react-tunnel to make reduce the boilerplate related to using context.
I am concerned about injecting props into the children, both because it is non explicit and because there may be some performance cost if the drawer if being re-rendered often.

Like this,I don't injecting drawer props to Route, so.. Route not open or close Drawer.
 <Drawer
            panCloseMask={1} //can close with right to left swipe anywhere on screen
            //panOpenMask={1} //can close with right to left swipe anywhere on screen
            openDrawerThreshold={.1}
            type="static"
            openDrawerOffset={0.6}
            styles={{main: {shadowColor: "#000000", shadowOpacity: 0.4, shadowRadius: 3}}}
            tweenHandler={Drawer.tweenPresets.parallax}
            content={<Menu />}
            >
          <Route />
        </Drawer>

But I really don't know  https://github.com/gnoff/react-tunnel  project.. I will be try that.
Thanks.

I am not comfortable injecting props like that. There are other workable albeit less terse solutions that I think we want to push people to use.


FlexibleSpaceBar background disappears when app resumes on Android

node: this issue does not happen on iOS so it looks like it is Android specific.
Steps to Reproduce

Create a SliverAppBar with a flexibleSpaceBar that has a background image set.
Run app on Android so that background image is visible
Go to home screen
Return to app
Note that the background image is not visible. If you adjust the SliverAppBar the image returns.

Logs
D/EGL_emulation(21302): eglMakeCurrent: 0xebf05a20: ver 3 0 (tinfo 0xd55a9dc0)
D/EGL_emulation(21302): eglCreateContext: 0xf0545100: maj 3 min 0 rcv 3
D/EGL_emulation(21302): eglMakeCurrent: 0xf0545100: ver 3 0 (tinfo 0xebc133d0)
D/EGL_emulation(21302): eglMakeCurrent: 0xf0545100: ver 3 0 (tinfo 0xd55a9410)
D/EGL_emulation(21302): eglMakeCurrent: 0xebf05a20: ver 3 0 (tinfo 0xd55a9dc0)

###Analyze
No issues found! (ran in 2.7s)

##Flutter doctor
[✓] Flutter (Channel master, v0.11.10-pre.1, on Mac OS X 10.13.6 17G3025, locale en-US)
[✓] Android toolchain - develop for Android devices (Android SDK 28.0.3)
[!] iOS toolchain - develop for iOS devices (Xcode 10.0)
    ✗ Verify that all connected devices have been paired with this computer in Xcode.
      If all devices have been paired, libimobiledevice and ideviceinstaller may require updating.
      To update with Brew, run:
        brew update
        brew uninstall --ignore-dependencies libimobiledevice
        brew uninstall --ignore-dependencies usbmuxd
        brew install --HEAD usbmuxd
        brew unlink usbmuxd
        brew link usbmuxd
        brew install --HEAD libimobiledevice
        brew install ideviceinstaller
    ✗ ios-deploy out of date (1.9.4 is required). To upgrade with Brew:
        brew upgrade ios-deploy
[✓] Android Studio (version 3.2)
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
[✓] IntelliJ IDEA Ultimate Edition (version 2018.2.5)
[!] VS Code (version 1.28.1)
[✓] Connected device (2 available)

Could you add some code to test ? did you try on iOS?

Well of course! when I try to produce a simple example of this and respond with a snarky emoji it works as expected on both emulator and device.
Likely user error on this one, sorry for the distraction.


Allegheny County Jail PA Phone calls

Someone has requested help on a data project:
Submission notes
How much is the average phone call in Allegheny County Jail?
Location
Allegheny County Jail - PA
Requirements
Cost policy
How to help
Leave a comment if you'd like to help. Do you have ideas, questions, or need any more information to get started?

It appears records are not kept of this. Recommended a survey of incarcerated people, as has been helpful in the past.


Custom autoscaling based on Buildkite/ScheduledJobsCount

This is the beginning of autoscaling based on the metrics provided by https://github.com/buildkite/buildkite-cloudwatch-metrics-publisher.
You can specify an AutoscaleStrategy of scheduledjobs to get this new behaviour.
A future PR will likely bring in the Lambda functions directly into this stack so we have a one-stop-shop.

:clap:


Using Axon Framework together with Spring Boot version >3.1.8 doesn't do JpaAutoConfiguration correctly

Basic information

Axon Framework version: 4.9.1
JDK version:  21

Steps to reproduce
Create new project with spring-boot-starter-parent 3.1.8 or higher, include spring-boot-starter-data-jpa and axon and see the following:
Description:
Parameter 2 of method eventStorageEngine in nl.goopher.mychatapp.config.AxonConfig required a bean of type 'org.axonframework.common.jpa.EntityManagerProvider' that could not be found.
Action:
Consider defining a bean of type 'org.axonframework.common.jpa.EntityManagerProvider' in your configuration.

Expected behaviour
The JpaAutoConfiguration class sets up the EntityManagerProvider correctly.
Actual behaviour
Application won't start because of a missing bean.
I think the issue might be this commit from Spring Boot:
https://github.com/spring-projects/spring-boot/issues/38904

Apologies, see now that it's fixed with later 4.9 versions.


Truffle migrate showing error *** stack smashing detected ***: terminated

Was going through documentation:
https://besu.hyperledger.org/stable/private-networks/tutorials/quickstart#generate-the-tutorial-blockchain-configuration-files
Was facing error "*** stack smashing detected ***: terminated" when running the sample dapps (pet-shop) provided in dev quickstart.
Running ./run_dapp.sh results in error when it reached truffle migrate command

Any idea or workaround?
Thanks

Hi @leeronkit
Am not sure what the issue is here. I've tried this out and am unable to reproduce the issue

To address issues that do not require attention, run:
  npm audit fix

To address all issues possible (including breaking changes), run:
  npm audit fix --force

Some issues need review, and may require choosing
a different dependency.

Run `npm audit` for details.

Compiling your contracts...
===========================
> Compiling ./contracts/Adoption.sol
> Compiling ./contracts/Migrations.sol
> Artifacts written to /home/jfernandes/workspace/protocols/quorum-dev-quickstart/quorum-test-network/dapps/pet-shop/pet-shop-box/build/contracts
> Compiled successfully using:
   - solc: 0.5.16+commit.9c3226ce.Emscripten.clang

Compiling your contracts...
===========================
> Everything is up to date, there is nothing to compile.

Starting migrations...
======================
> Network name:    'quickstartWallet'
> Network id:      1337
> Block gas limit: 16234336 (0xf7b760)

1_initial_migration.js
======================

   Deploying 'Migrations'
   ----------------------
   > transaction hash:    0xdb87825b7af1e0ba1697bff4b09cf557185e8d15c87b99ce374543c04c9a745e
   > Blocks: 1            Seconds: 4
   > contract address:    0x8CdaF0CD259887258Bc13a92C0a6dA92698644C0
   > block number:        100
   > block timestamp:<PHONE_NUMBER>
   > account:             0x627306090abaB3A6e1400e9345bC60c78a8BEf57
   > balance:             89999.9995168925
   > gas used:            193243 (0x2f2db)
   > gas price:           2.5 gwei
   > value sent:          0 ETH
   > total cost:          0.0004831075 ETH

   > Saving migration to chain.
   > Saving artifacts
   -------------------------------------
   > Total cost:        0.0004831075 ETH

2_deploy_contracts.js
=====================

   Deploying 'Adoption'
   --------------------
   > transaction hash:    0x0dbb6aa5ce84fac4879cca32f25c7b4eeb55ea88b026dd8092d1d3d808361306
   > Blocks: 0            Seconds: 4
   > contract address:    0x345cA3e014Aaf5dcA488057592ee47305D9B3e10
   > block number:        102
   > block timestamp:<PHONE_NUMBER>
   > account:             0x627306090abaB3A6e1400e9345bC60c78a8BEf57
   > balance:             89999.99889301
   > gas used:            203815 (0x31c27)
   > gas price:           2.5 gwei
   > value sent:          0 ETH
   > total cost:          0.0005095375 ETH

   > Saving migration to chain.
   > Saving artifacts
   -------------------------------------
   > Total cost:        0.0005095375 ETH

Summary
=======
> Total deployments:   2
> Final cost:          0.000992645 ETH

Using network 'quickstartWallet'.

Compiling your contracts...
===========================
> Everything is up to date, there is nothing to compile.

  0 passing (0ms)

[+] Building 24.5s (8/8) FINISHED                                                                                docker:default
 => [internal] load build definition from Dockerfile                                                                                0.0s
 => => transferring dockerfile: 126B                                                                                0.0s
 => [internal] load .dockerignore                                                                                0.0s
 => => transferring context: 2B                                                                                0.0s
 => [internal] load metadata for docker.io/library/node:12.14.1-stretch-slim                                                                                3.5s
 => [1/3] FROM docker.io/library/node:12.14.1-stretch-slim@sha256:59ac2f2c3a0c490d8424306032f9b638f5ea83327ffaf23c66490e0026d1a000                                       12.3s
 => => resolve docker.io/library/node:12.14.1-stretch-slim@sha256:59ac2f2c3a0c490d8424306032f9b638f5ea83327ffaf23c66490e0026d1a000                                        0.0s
 => => sha256:59ac2f2c3a0c490d8424306032f9b638f5ea83327ffaf23c66490e0026d1a000 1.21kB / 1.21kB                                                                            0.0s
 => => sha256:28197e7ddd657ac51c3f775d5df9f4aac10d155f0da5af438e41b8a2a54e5795 1.37kB / 1.37kB                                                                            0.0s
 => => sha256:2f7e25ad14eaded84e39417dbd6868720ddd7de6f7239cc2ee42b2b043128f08 6.42kB / 6.42kB                                                                            0.0s
 => => sha256:619014d83c026bca3d21f5eb42a629ac2dbe15bdb37b77f25fc1ac5fbada68e5 22.52MB / 22.52MB                                                                          7.7s
 => => sha256:8c5d9aed65fb4440d886ef0385d0d6db5e1a494a70d0bd482f7307c338fe6581 4.14kB / 4.14kB                                                                            0.9s
 => => sha256:aaabe8e9daf2f6e14d996a99f341fe99f7c189007d2ba1484ddce01a980272e2 23.94MB / 23.94MB                                                                          6.1s
 => => sha256:f7567fa7b9f39122f4c3d27fe24f4fd830753bd5ead8a031157380fac94d2d5e 1.85MB / 1.85MB                                                                            2.1s
 => => sha256:a989ed5f800bc9def10547dc1a72acf5b078ae783e2be24c038df30d48f6e730 295B / 295B                                                                                2.6s
 => => extracting sha256:619014d83c026bca3d21f5eb42a629ac2dbe15bdb37b77f25fc1ac5fbada68e5                                                                                1.6s
 => => extracting sha256:8c5d9aed65fb4440d886ef0385d0d6db5e1a494a70d0bd482f7307c338fe6581                                                                                0.0s
 => => extracting sha256:aaabe8e9daf2f6e14d996a99f341fe99f7c189007d2ba1484ddce01a980272e2                                                                                2.5s
 => => extracting sha256:f7567fa7b9f39122f4c3d27fe24f4fd830753bd5ead8a031157380fac94d2d5e                                                                                0.1s
 => => extracting sha256:a989ed5f800bc9def10547dc1a72acf5b078ae783e2be24c038df30d48f6e730                                                                                0.0s
 => [internal] load build context                                                                                4.9s
 => => transferring context: 418.41MB                                                                                4.7s
 => [2/3] WORKDIR /app                                                                                0.1s
 => [3/3] COPY . /app                                                                                4.0s
 => exporting to image                                                                                4.3s
 => => exporting layers                                                                                4.3s
 => => writing image sha256:1a3cd1bb95059ae2a24f1dcef6861d07fd6535aed0934880be173400992abc66                                                                              0.0s
 => => naming to docker.io/library/quorum-dev-quickstart_pet_shop                                                                                0.0s
83972b5cb60e6decdd8813d49c6fa1260952ed1149c59fdd1e3946c1c9833f70

Suggest maybe creating an issue with Truffle and give them specs of your environment. I've used Win11 WSL2 (ubuntu 22.04) with the latest version of the quickstart and am unable to reproduce the above
Keep us posted
Cheers


The value of a feed cannot be a tf.Tensor object. Acceptable feed values include Python scalars, str

When i feed the value, unfortunately , it is a tensor.
i have already known that i should convert it to a array or other types to feed. But if i use sess.run() or .eval(). The speed makes me crazy, for the shape is [3600,14,25500].
if other ways are available?

This question is better asked on  StackOverflow since it is not a  bug or feature request. There is also a larger community that reads questions there. Thanks!


feature add max num of same name meta information in catalog recycle bin

Proposed changes
Issue Number: close #14292
Problem summary
new fe conf : max_same_name_catalog_trash_num, it can tell fe just save max mun of catalog trash with same name, if some meta information with same name beyond this num, fe can delete immediately these meta information that be dropped earliest. default value: 3
Checklist(Required)

Does it affect the original behavior:

[x] Yes
[ ] No
[ ] I don't know

Has unit tests been added:

[x] Yes
[ ] No
[ ] No Need

Has document been added or modified:

[x] Yes
[ ] No
[ ] No Need

Does it need to update dependencies:

[ ] Yes
[x] No

Are there any changes that cannot be rolled back:

[ ] Yes (If Yes, please explain WHY)
[x] No

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 35.53 seconds
load time: 432 seconds
storage size:<PHONE_NUMBER>4 Bytes
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20221122114438_clickbench_pr_50746.html


Autocomplete google form added

Maybe not install all the npm packages for now until we understand the Autocomplete better.


Strict pattern matching

What is the problem this feature would solve?
Walls of unneeded lambdas when pattern matching harm readability. For example:
import { Data } from "effect"

type Example = Data.TaggedEnum<{
  A: {},
  B: {},
  C: {},
  // etc
}>

const { $match } = Data.taggedEnum<Example>()

$match({
  A: () => "it is A",
  B: () => "this time it is B",
  C: () => "this is wild, it's C!",
  // etc
})

What is the feature you are proposing to solve the problem?
matchX simply makes each case strict. This has different performance characteristics, however in many cases it's viable and improves readability:
$matchX({
  A: "it is A",
  B: "this time it is B",
  C: "this is wild, it's C!",
  // etc
})

This is particularly relevant when there are potentially dozens of sum members.
What alternatives have you considered?
constant, tolerating the lambdas.

Given that JS has eager evaluation you generally don't want to compute the alternatives unless they match, this isn't something we would like to add for the time being

Given that JS has eager evaluation you generally don't want to compute the alternatives unless they match, this isn't something we would like to add for the time being

I'm really thinking primarily of the example where rather than expensive computations it's just strings or other primitives. We find in our codebase that this comes up quite a lot in pattern matching, and I perhaps naively wonder if this might be more performant than a redundant anonymous function call.

Given that JS has eager evaluation you generally don't want to compute the alternatives unless they match, this isn't something we would like to add for the time being

I'm really thinking primarily of the example where rather than expensive computations it's just strings or other primitives. We find in our codebase that this comes up quite a lot in pattern matching, and I perhaps naively wonder if this might be more performant than a redundant anonymous function call.

If that kind of perf is important to you probably you shouldn't even use a matcher in the first place, preventive optimization is usually bad

If that kind of perf is important to you probably you shouldn't even use a matcher in the first place, preventive optimization is usually bad

I agree, but wasn't performance your primary concern?
Maybe there's unease about further complicating the pattern matching API, which'd make sense. Just thought I'd raise this though as, in considering a migration path from the fp-ts ecosystem, we've found ourselves quite liking matchX in sum-types.

If that kind of perf is important to you probably you shouldn't even use a matcher in the first place, preventive optimization is usually bad

I agree, but wasn't performance your primary concern?
Maybe there's unease about further complicating the pattern matching API, which'd make sense. Just thought I'd raise this though as, in considering a migration path from the fp-ts ecosystem, we've found ourselves quite liking matchX in sum-types.

 Thanks for engaging. 🙂

Wondering if we could use the same api with an optional parameter though, as long as we don't duplicate functions and we can support it across the types I am not against

You could also create an identity function that takes an argument, and return a function that return that argument.
const always = <Value>(value: Value) => () => value;

match({
  A: always("it is A"),
  B: always("this time it is B"),
  C: always("this is wild, it's C!"),
});

Similar to what Lodash or Ramda do.

You could also create an identity function that takes an argument, and return a function that return that argument.
[...]

FYI there is const already.


let test success because of a tiflash bug

What problem does this PR solve?
let test success because of a tiflash bug
https://github.com/pingcap/tics/issues/1572
insert 1969-12-31 return 1970-01-01 when disable chunk rpc and set timezone
What is changed and how it works?
Check List
Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)
No code

Code changes

Has exported function/method change
Has exported variable/fields change
Has interface methods change
Has persistent data change

Side effects

Possible performance regression
Increased code complexity
Breaking backward compatibility

Related changes

Need to cherry-pick to the release branch
Need to update the documentation
Need to update the tidb-ansible repository
Need to be included in the release note

/run-all-tests test-flash=true


Slack Channel Connection giving "No valid credentials found for PagerDuty provider."

What happened?
I created the Service(using pagerduty.Service) and datadog Integration of this service using pagerduty.ServiceIntegration. Both these works fine. However when I try use the pagerduty.SlackConnection to connect slack channel to this service, getting the error.
pagerduty:index:SlackConnection (test-slack):
    error: 1 error occurred:
    	*

    No valid credentials found for PagerDuty provider.
    Please see https://www.terraform.io/docs/providers/pagerduty/index.html
    for more information on providing credentials for this provider.

Example
Here is the sample code.
const service = new pagerduty.Service(`${serviceName}-service`, {
            name: product.name,
            autoResolveTimeout: 'null',
            acknowledgementTimeout: 'null',
            escalationPolicy: policy.id,
            alertCreation: 'create_alerts_and_incidents',
            description: `${product.escalationPolicy} Service`
        });

        const datadog = pagerduty.getVendor({
          name: 'Datadog',
        });

        const serviceDatadogIntegration = new pagerduty.ServiceIntegration(`${serviceName}-service-dd-integration`, {
          name: datadog.then(datadog => datadog.name),
          service: service.id,
          vendor: datadog.then(datadog => datadog.id)
        })

            const slackChannelConnection = new pagerduty.SlackConnection(`${serviceName}-test-slack`, {
            sourceId: service.id,
            sourceType: 'service_reference',
            channelId: 'XXXXX', //slackChannel,
            workspaceId: 'YYYYY', //slackWorkspace,
            notificationType: 'responder',
            configs: [{
              events: [
                'incident.triggered',

              ],
            }]
          });

Output of pulumi about
`CLI
Version      3.116.1
Go Version   go1.22.2
Go Compiler  gc
Plugins
KIND      NAME       VERSION
language  nodejs     unknown
resource  pagerduty  4.12.2
Host
OS       darwin
Version  12.5.1
Arch     arm64
This project is written in nodejs: executable='/Users/xxxx/.asdf/shims/node' version='v20.13.1'
Backend
Name           pulumi.com
URL            https://app.pulumi.com/xxxx_admin
User           xxxx_admin
Organizations  xxxx_admin, xxxxx
Token type     personal
Dependencies:
NAME               VERSION
@types/node        20.12.12
param-case         3.0.4
@pulumi/pagerduty  4.12.2
@pulumi/pulumi     3.116.1`
Additional context
It is creating the service and service integration with the datadog but it is only failing to creation slackconnection for this service with a slack channel.
Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

Hey @knayak84. Thanks for filing an issue. Have you successfully linked your slack workspace to pagerduty in the pagerduty web portal? (as described here)

NOTES for using this resource:
To first use this resource you will need to map your PagerDuty account to a valid Slack Workspace. This can only be done through the PagerDuty UI.
This resource requires a PagerDuty user-level API key. This can be set as the user_token on the provider tag or as the PAGERDUTY_USER_TOKEN environment variable.
This resource is for configuring Slack V2 Next Generation connections. If you configured your Slack integration (V1 or V2) prior to August 10, 2021, you may migrate to the Slack V2 Next Generation update using this migration instructions, but if you configured your Slack integration after that date, you will have access to the update out of the box.

Thank you @iwahbe. I was having the same issue and that was what I was missing, setting PAGERDUTY_USER_TOKEN and not only PAGERDUTY_TOKEN

We're glad this resolved the issue for you!


Implement seen and timezone plugins

Test and add these plugins to the default config

https://github.com/Appleman1234/arisu/tree/master/Seen
https://github.com/Appleman1234/arisu/tree/master/Time
https://github.com/redwallhp/errbot-timezones

Something a bit fun the play with, at least in test/offtopic rooms , and combined help people understand why someone isnt able to do a review of their PR, cause they are probably asleep.

Some alternatives
https://github.com/errbotio/err-time
https://github.com/Gnoffel/errbot-seen
https://github.com/jasedit/errbot-lastseen
https://github.com/errbotters/err-lastseen
https://github.com/pythiannunez/err-pythianbot
https://github.com/jerrykan/recordwho
https://github.com/ghoti/Rooster/blob/master/plugins/timeBot.py
https://github.com/errbotio/err-stalkerbot
https://github.com/aherok/errbot_plugins

I have a question regarding the seen feature. Is it viable to include this plugin given that some of the chat backends already have this "seen by' feature?


Markdown tables aren't rendered

The markdown content at https://github.com/ricardoboss/trenz-docs-test/blob/fd75ad8b2471bec70d1b7300072195e65ce4206f/docs/GHFM.md is rendered like this:

Opened issue over at xoofx/markdig#651

Easiest solution would be to just replace the image url using a regex instead of parsing the markdown source and replacing the url based on the node type. This way, we can preserve as much original markdown as possible and marked+prism can render it in the browser properly.

This just occurred to me: if we disable the pipe tables extension, the table symbols in the original markdown are left as is as they are just text for the markdown pipeline 🙃

2de721919030186751f472c5a253601a2f31de20 temporarily fixes this


Improve admin's projects page display and latency to better handle many projects per user

some users may select to create a large amount of projects, improve display and latency when admins create 30+ projects

some improvements to project loading queries done as part of #525 - once those changes are merged, this ticket should evaluate whether or not project level settings should be loaded for the /projects page.

PR: #638
some changes also previously merged under PR: #634


Methylnick patch 1 gzipped

fastq input is now gzipped

implemented gzipped fastq files as input into this tool.


regression - view quick open broken

trying to verify https://github.com/Microsoft/vscode/issues/58555

hit F1
select all and type view
🐛 instead of viewlet, panels et a I see file resuts

@chrmarti https://github.com/Microsoft/vscode/commit/3f7e8b4d3d2d6b44685f7c4ac918ad8dd5ab3c53 ?


No GPS fix whatever I do

Hi,
I have HackRF one with ANT500, trying to spoof my android 10 phone GPS, I follow exactly instruction and couple videos, removed my sim card turned high accuracy off in Android, turn on airplane mode. but it will never fix
my android is rooted, is there a way we can check logs maybe to see why it not fix or do I do something wrong?
Thanks

Try to generate fake signal with -b 8 option like this:
gps-sdr-sim -b 8 -e brdc1090.20n -l 55.2768359,6.2364598,100
then push it to the air using:
hackrf_transfer -t gpssim.bin -f<PHONE_NUMBER> -s 2600000 -a 1 -x 0
As far I know HackRF One requires an 8 bit signed IQ raw file with no header information.
Works for me.

Try to generate fake signal with -b 8 option like this:
gps-sdr-sim -b 8 -e brdc1090.20n -l 55.2768359,6.2364598,100
then push it to the air using:
hackrf_transfer -t gpssim.bin -f<PHONE_NUMBER> -s 2600000 -a 1 -x 0
As far I know HackRF One requires an 8 bit signed IQ raw file with no header information.
Works for me.

Thanks you sir, I already did -b 8 with no success, however I started to get one device locked after I add -C -10

@majedalanni ..Is your lock stable for the duration of simulation (default 300 secs) ?

Same, I tried with two different
I'm inside without any GPS signal.
When launching signal transmission.
In view I see tons of GPS (30+).
Same setup that the post : mi9 android phone, airplane mode and every A-gps feature disabled.
I'm also using an IoT device with M8N module + Taoglass antenna ; no fix either.
Tried -C -10 without result.

Phone and IoT device are on the antenna.

what does the -x represents please?


Refactor: move Timeline, TimelineEvent, TimelineSummary and related code from flutter_driver to flutter_test

These codes are post-processing for Dart timeline, mostly for performance purposes. They are not necessarily only used in flutter_driver. To build host independent tests and report performance metrics, as well as follow up to #58789 we hope to use these but not introducing flutter_driver dependence.

We decided no to.


Inline occurence issue

Previous Code
  const temp3 = [];         temp3[0] = temp1;         temp3[1] = temp2;

After refactoring
[][0] = temp1;         [][1] = temp2;

Thanks for the bug report! I'll update when I have a fix.

Hey @chandan192 - I agree that inlining here does not make sense, and you would typically not invoke it.
However, I'm not sure I understand what the issue with the refactoring is? The resulting code compiles and executes for me, and it indeed the expected result of an inline operation on temp3.
What would you expect to happen instead?

Hi @lgrammel,
I was just experimenting with the refactoring using p42 and tslint. I got the p42 refactoring suggestion on the context menu.
Inline 2 occurrences [p42]
I also checked the p42 suggestion on the activity bar but there was no suggestion. It was only on the manual context menu refactoring.

I see - in this case it was not a 'suggestion' (an in, P42 recommended), but it showed up in the quickfix menu. I'll add a warning similar to what is shown for new Objects.

I've made a couple of improvements in v1.103.1 to address this issue:

inline variable warns you about inlining literals that are used in multiple places
inline variable is only availalbe as a quickfix when it is a suggestion, otherwise you need to use the refactor or inline context menus / shortcuts

Thanks for the bug report!