RKE2 Cluster does not show an Error state when cluster upgrade has failed

On 2.6-head commit id: 024e6fa

Deployed an RKE2 cluster - 3 etcd, 2 cp and 3 worker nodes
Upgrade strategy is set on the cluster
Edit cluster --> Upgrade k8s version from 1.22 to 1.23
Error seen on the cluster -
Draining etcd node(s) sowmya-do3-pool1-6c6f5f7f4f-x66td: error draining machine sowmya-do3-pool1-6c6f5f7f4f-x66td: error when evicting pods/"calico-typha-6546b78f-qpg5l" -n "calico-system": global timeout reached: 30s
But the state of the cluster is Provisionining
And in another scenari, on the control plane error Draining control plane node(s) sowmya-test-pool2-56d9c84585-xx5hz: error draining machine sowmya-test-pool2-56d9c84585-xx5hz: cannot delete Pods with local storage (use --delete-emptydir-data to override): cattle-system/system-upgrade-controller-5b6457d644-n8qhx

Expected:
Cluster state must show Error

The error message show occurs when a cluster is waiting to finish draining.  The upgrade has not technically failed because it is just waiting for the drain to complete.  @thedadams did some work to make it clearer when a cluster is waiting to drain.  Hopefully this will make it clearer to the user why their upgrade has seemed to stall.

the option --delete-emptydir-data is disabled by default when enabling an upgrade strategy. And it seems the user will hit this error without enabling it. Would it be a backend or dashboard request to enable this by default? or is there a case against enabling this by default?

Personally I don't have any issues with moving to a model where --delete-emptydir-data is enabled by default; however, I think we should very clearly state this with as many disclaimers as possible b/c it can cause data loss for those that are not expecting it. Even though emptyDir is expected to be "ephemeral", there are users out there that rely on emptyDir persisting over the life of a pod and if a pod's lifecycle is prematurely interrupted due to an "upgrade" this may cause issues.
With a disclaimer, we can fall back on the "it said it would do that" argument around it.
Additionally, I'm not sure exactly what RKE1 does, but we should probably just mimic that (I'm pretty sure RKE1 drains forcefully by default)

as a result of the above comment and offline discussion, the above UI ticket was opened and we will close this one.


feat(ECS): enable selection of listener rules

What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes https://github.com/pipe-cd/pipecd/issues/4530
Does this PR introduce a user-facing change?:

How are users affected by this change:

apiVersion: pipecd.dev/v1beta1
kind: ECSApp
input:
  taskDefinitionFile: taskdef.yaml
  serviceDefinitionFile: servicedef.yaml
  targetGroups:
    primary:
    canary:

Is this breaking change: no
How to migrate (if breaking change):

@khanhtc1202
I have recreated PR #4540 to incorporate the latest updates and resolve existing issues. Your review and feedback on the new PR would be greatly appreciated.

@moko-poi Please sign the commit with git commit -s to pass the CI

@khanhtc1202
I've just signed the commit using git commit -s as you suggested. This should ensure that the commit passes the CI checks now.
Thank you for pointing this out

@khanhtc1202
Fixed routing to target group when action type is forward.


Can't use chartjs-plugin-datalabels

I am trying to add the chartjs-plugin-datalabels plugin to my charts and I think I've figured out how to add it.
I've created a minimal case here.
The problem is that I get this error:
chartjs-plugin-datalabels.js?a9be:155 Uncaught TypeError: Cannot read property 'x' of null
    at orient (chartjs-plugin-datalabels.js?a9be:155)
    at Object.fallback (chartjs-plugin-datalabels.js?a9be:379)
    at coordinates (chartjs-plugin-datalabels.js?a9be:847)
    at Object.draw (chartjs-plugin-datalabels.js?a9be:1011)
    at Object.afterDatasetsDraw (chartjs-plugin-datalabels.js?a9be:1330)
    at Object.notify (coreui-vue-chartjs.common.js?f485:10882)
    at Chart.drawDatasets (coreui-vue-chartjs.common.js?f485:12767)
    at Chart.draw (coreui-vue-chartjs.common.js?f485:12695)
    at Chart.render (coreui-vue-chartjs.common.js?f485:12649)
    at Object.callback (coreui-vue-chartjs.common.js?f485:5071)

I did some research and found this but I can't figure out how to implement it.  There are also suggestions of downgrading to a 2.7.x version of chartjs but I think I'd have to rebuild coreui/vue-chartjs to even attempt that.
Any suggestions?

I fixed my issue.  (just register datalabels plugin LOCALLY per chart)
 TypeError: Cannot read property 'x' of null
ChartDataLabels plugin was registered globally in another chart that I had. instead I register locally and that fixed my issue. Docs
basically I just had to remove this line in all my chart definitions
// Register the plugin to all charts:
Chart.register(ChartDataLabels);

instead I just need it the ChartDataLabels in my plugins object:
  plugins: [ChartDataLabels], ...```

Here are the steps that worked for me:
npm install chartjs-plugin-datalabels --save

Here is the modified working example from the repo.
<template>
  <CChartDoughnut :data="defaultData" :plugins="defaultPlugins" />
</template>

<script>
import { CChartDoughnut } from '@coreui/vue-chartjs'
import ChartDataLabels from 'chartjs-plugin-datalabels'

export default {
  name: 'CChartDoughnutExample',
  components: { CChartDoughnut },
  computed: {
    defaultData() {
      return {
        labels: ['VueJs', 'EmberJs', 'VueJs', 'AngularJs'],
        datasets: [
          {
            backgroundColor: ['#41B883', '#E46651', '#00D8FF', '#DD1B16'],
            data: [40, 20, 80, 10],
            datalabels: {
              color: ['#000000', '#ffffff'],
              font: {
                size: 16,
              },
            },
          },
        ],
      }
    },
    defaultPlugins() {
      return [ChartDataLabels]
    },
  },
}
</script>

Diff for just the changes.
diff --git i/src/views/charts/CChartDoughnutExample.vue w/src/views/charts/CChartDoughnutExample.vue
index 653c8c8..e0e0df1 100644
--- i/src/views/charts/CChartDoughnutExample.vue
+++ w/src/views/charts/CChartDoughnutExample.vue
@@ -1,9 +1,11 @@
 <template>
-  <CChartDoughnut :data="defaultData" />
+  <CChartDoughnut :data="defaultData" :plugins="defaultPlugins" />
 </template>

 <script>
 import { CChartDoughnut } from '@coreui/vue-chartjs'
+import ChartDataLabels from 'chartjs-plugin-datalabels'
+
 export default {
   name: 'CChartDoughnutExample',
   components: { CChartDoughnut },
@@ -15,10 +17,19 @@ export default {
           {
             backgroundColor: ['#41B883', '#E46651', '#00D8FF', '#DD1B16'],
             data: [40, 20, 80, 10],
+            datalabels: {
+              color: ['#000000', '#ffffff'],
+              font: {
+                size: 16,
+              },
+            },
           },
         ],
       }
     },
+    defaultPlugins() {
+      return [ChartDataLabels]
+    },
   },
 }
 </script>


ESP8266 difficulties

I'm having some trouble getting the library to read a DS18B20+PAR with a Wemos D1 Mini. (ESP8266EX)
The code (a close copy of the 'simple' example) works fine if run instead on an Arduino Pro Micro.
On the Wemos D1 Mini I just get:
Starting temperature read...done. Temperature is -127.00
Starting temperature read...done. Temperature is 85.00
Starting temperature read...done. Temperature is -127.00
Starting temperature read...done. Temperature is 85.00
...

If I detach the 4.7K pullup resistor, no change.
If I attach the pullup resistor to 5V instead of 3.3V I instead get:
Starting temperature read...done. Temperature is 127.94
Starting temperature read...done. Temperature is 127.94
...

Code is below.
(The +PAR variant is ONLY capable of parasitic mode, and pin 3 is NC)
Can you please advise? I do not know what the temperatures indicate.
#include <OneWire.h>
#include <DallasTemperature.h>

// Data wire is plugged into port D6 on the Wemos D1
#define ONE_WIRE_BUS D6
// Setup a oneWire instance to communicate with any OneWire devices
OneWire oneWire(ONE_WIRE_BUS);
// Pass our oneWire reference to Dallas Temperature.
DallasTemperature sensors(&oneWire);
/*
 * The setup function. We only start the sensors here
 */
void setup(void)
{
  // start serial port
  Serial.begin(115200);
  Serial.println("Dallas Temperature IC Control Library Demo");

  // Start up the library
  sensors.begin();
  sensors.setWaitForConversion(false);
}

/*
 * Main function, get and show the temperature
 */
void loop(void)
{
  if (1!=sensors.getDeviceCount()) {
    Serial.print("Did not find 1 device, found ");
    Serial.println(sensors.getDeviceCount());
    return;
  }
  if (!sensors.isParasitePowerMode()) {
    Serial.println("Expected parasite mode");
    return;
  }

  Serial.print("Starting temperature read...");
  sensors.requestTemperatures();
  delay(1000);
  Serial.print ("done. Temperature is ");
  float T = sensors.getTempCByIndex(0);
  Serial.println(T);
}

Is this Wemos D1 Mini a 3.3V device? If so the 3.3V might be too low (e.g 3.0 V effective) to "feed" the sensor reliably. Can you measure the actual voltage?
According to the datasheet 3.3V should be sufficient.
https://datasheets.maximintegrated.com/en/ds/DS18B20-PAR.pdf
Still a bidi levelconvertor like  https://www.sparkfun.com/products/12009  might help to

It's 3.3V with a 5V=>3.3V regulator, so both voltages are available on board.
(Pins are 3.3V logic with ~12mA source capacity, and are 5V tolerant)
I measured the steady voltage of the regulator to be 3.3V.
Capturing the signal to check for sag during that communication will be tricky, but I'll give it a try.
Also the test is just on breadboard, wire length is <10cm.

I amended my code to just show raw scratchpad data;
void loop(void)
{
  Serial.print("Starting temperature read...");
  sensors.requestTemperaturesByAddress(addr);
  delay(1000);
  Serial.println("done.");
  //--------------------------------------------------------
  Serial.print("Raw scratchpad is: ");
  if (!sensors.isConnected(addr,pad)) { Serial.println("ERROR bad connection"); return; }
  char buf[27];
  sprintf(buf,"%02X:%02X:%02X:%02X:%02X:%02X:%02X:%02X:%02X",pad[0],pad[1],pad[2],pad[3],pad[4],pad[5],pad[6],pad[7],pad[8]);
  Serial.println(buf);
  //--------------------------------------------------------
}

5V Behaviour
With a 5V pullup I get a constant value;
Raw scratchpad is: FF:07:4B:46:7F:FF:01:10:2F
Starting temperature read...done.
Raw scratchpad is: FF:07:4B:46:7F:FF:01:10:2F
...

Not sure what that value is supposed to be, but it doesn't change when the thermometer is held. (as I saw on the pro micro platform)
I've tried looking at the voltage of the data line with a DSO:  5V Pullup
The pullup voltage during the conversion time stays at 5V, I think there's not a power supply problem.
3.3V Behaviour
If I use a 3.3V pullup I get stranger behaviour;
3 times in a row through loop() I get "ERROR: Bad Connection", then the 4th time returns that same scratchpad value as above, and the cycle repeats.
The DSO shows the data line's voltage is at a steady ~1.6V during those 3 bad reads, then it jumps up to 3.3V for the one that reads successfully. 3.3V Pullup
I'm getting very confused!

More testing, found something that works; reducing the pullup resistor value.
Not sure exactly what's going on, but it's working properly now. Thanks for your library. :-)

By lowering the value of the resistor the DS18B20 gets more power/energy to load itself. Apparently with 1K5 it gets enough to do the actual measurement, the math and IO internally to give a decent response.


Not working in 9.249

When I click in "Bulk Tasks" I get this error.
This is only occurring after updating Foundry

Weird, I can't seem to replicate it, and nothing in the new update should have affected it.

I just tried again, with only the Bulk Tasks module and it works, therefore it must be a conflict with another module.
I'll have to dig and find the culprit.

Apologies for re-opening but, I was able to confirm the issue happens with just "Bulk Tasks" module enabled.

foundry.js:316 TypeError: An error occurred while rendering BulkMenu 95: Cannot read properties of undefined (reading 'permission')
    at bulkMenu.js:268:16

https://user-images.githubusercontent.com/45291719/153245159-b0f33b4b-abdd-4fb5-ab0f-f25d5a4b7f49.mp4

It seems like there's a document that is kind of borked. Can you see if bulk tasks works in a new world with just a few items?

Clean world with a few monsters, no issues...
Any idea how I can find which document has the issue?

I was able to do some debugging and pinpoint the issue to a Playlist Folder. I have renamed this as it seems to not be related to PF2E system.
Here's how to replicate.

Install and enable a Music Module such as "Ivan Duch's Music Packs"
Navigate to Compendium > Playlist
Right-click the Music Playlist and select "Import All Content" with default options
After Import, a folder is created in the "Audio Playlists" tab
Verify that "Bulk Tasks" no longer works

Hmm, I see. Thank you for the diagnosis. I'll take a look at the module and see what I can do about it. If I'm correct the way they create those playlists is incorrect. I'll provide a mitigation in the module to just not render anything that has an incorrect format.

Should be fixed in the next update.


freetype: makeWrapper doesn't cross compile to windows

Issue #120726
Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

Curious if this actually works because of this comment...
https://github.com/NixOS/nixpkgs/issues/120726#issuecomment-1518984250

It looks like in this PR they solved the same problem by conditionally removing makeWrapper if the hostPlatfrom was windows.
https://github.com/NixOS/nixpkgs/pull/229465

Hey. I wish someone replied to my comments before this got merged. @Mindavi. Has this been tested on windows? I would rather have a package that fails to build for windows vs a package that builds for windows but doesn't function correctly.

They were addressed, but I didn't test this otherwise. Are you willing to test it?

Oh okay I see. My bad. I can test on windows, but it might be a little til I get back on this :(

Hey. I wish someone replied to my comments before this got merged. @Mindavi. Has this been tested on windows? I would rather have a package that fails to build for windows vs a package that builds for windows but doesn't function correctly.

The only wrapped binary is the freetype-config in the dev output. -config binaries should be avoided because they break cross-compilation, instead pkg-config should be used.

So I tried to build this for mingwW64 and I get an error for pkg-config.
gslice.c: In function 'smc_notify_free':
gslice.c:1506:24: error: format '%u' expects argument of type 'unsigned int', but argument 4 has type 'size_t' {aka 'long long unsigned int'} [-Werror=format=]
 1506 |       fprintf (stderr, "GSlice: MemChecker: attempt to release non-allocated block: %p size=%" G_GSIZE_FORMAT "\n", pointer, size);
      |                        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~                               ~~~~
      |                                                                                |
      |                                                                                size_t {aka long long unsigned int}
In file included from gslice.c:22:
glibconfig.h:62:28: note: format string is defined here
   62 | #define G_GSIZE_FORMAT "I64u"
gslice.c:1511:24: error: format '%u' expects argument of type 'unsigned int', but argument 4 has type 'SmcVType' {aka 'long long unsigned int'} [-Werror=format=]
 1511 |       fprintf (stderr, "GSlice: MemChecker: attempt to release block with invalid size: %p size=%" G_GSIZE_FORMAT " invalid-size=%" G_GSIZE_FORMAT "\n", pointer, real_size, size);
      |                        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~                                                                ~~~~~~~~~
      |                                                                                |
      |                                                                                SmcVType {aka long lo>
glibconfig.h:62:28: note: format string is defined here
   62 | #define G_GSIZE_FORMAT "I64u"
gslice.c:1511:24: error: format '%u' expects argument of type 'unsigned int', but argument 5 has type 'size_t' {aka 'long long unsigned int'} [-Werror=format=]
 1511 |       fprintf (stderr, "GSlice: MemChecker: attempt to release block with invalid size: %p size=%" G_GSIZE_FORMAT " invalid-size=%" G_GSIZE_FORMAT "\n", pointer, real_size, size);
      |                        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~                                                                           ~~~~
      |                                                                                |
      |                                                                                size_t {ak>
glibconfig.h:62:28: note: format string is defined here
   62 | #define G_GSIZE_FORMAT "I64u"
gslice.c:1516:24: error: format '%u' expects argument of type 'unsigned int', but argument 4 has type 'size_t' {aka 'long long unsigned int'} [-Werror=format=]
 1516 |       fprintf (stderr, "GSlice: MemChecker: attempt to release non-allocated block: %p size=%" G_GSIZE_FORMAT "\n", pointer, size);
      |                        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~                               ~~~~
      |                                                                                |
      |                                                                                size_t {aka long long unsigned int}

Okay interesting. If I try to build for pkgs.pkgsCross.mingw32. It works. If I try to build for pkgs.pkgsCross.mingwW64 it does not work.

https://github.com/NixOS/nixpkgs/pull/245775


chore: add .gitignore to test project

Issue
The example specs in factory/test-project include two temporary files in the cypress/fixtures which are created as the result of running tests:

factory/test-project/cypress/fixtures/profile.json
factory/test-project/cypress/fixtures/users.json

These temporary files are ignored in the upstream repo cypress-io/cypress-example-kitchensink/.gitignore and they should not be committed to this repo either.
Change

Remove the above files from factory/test-project/cypress/fixtures.
Add a .gitignore for the above temporary artifacts to factory/test-project

There is currently an ongoing Actions Incident https://www.githubstatus.com/


Custom Abilities get duplicated by /reload

I have a custom ability called "Haunting" that I noticed was being applied to mobs multiple times (eg. Haunting Speedster Rust Haunting Zombie). The ability would also activate twice, so it's not just a visual bug. This behavior stopped when I restarted the game, but would start again whenever I used /reload

Create a custom ability via datapack
While in-game use /reload any amount of times
Use /summon_eldritch etc to spawn a mob with abilities, repeat until you get a mob with multiple instances of the custom ability
Restart your game to fix

My haunting.json ability, just in case it's relevant:
{
	"name": "Haunting",
	"type": "ACTIVE",
	"subtype": "OFFENSIVE",
	"activationType": "hasTarget",
	"cooldown": 500,
	"command": "execute at {target} run summon bewitchment:ghost ~ ~ ~"
}

Fixed in latest release.


Update examples for 0.1.3-beta

Updated all 3 examples to the latest released beta.
Resolves
Fixes #145
Description
Updated the 3 examples to work with the latest released beta.

Ready for review @aslotte

@aslotte Sure, I'll take a look today, may have some more questions :)

@aslotte feedback please :) Also caught a spelling mistake and a comment section with no comments.

Thanks for the contribution!


Create build.yml

What's this ?

It is for automatically build scss files on push on master branch.
(also, pls change branch name to main and wait, i need to fix a thing with this)

I renamed It

ok, you can now merge!


Compile Options not working

Even using the example options listed in the README, the compile options don't seem to work.  Let me know what else you need to get this working.

Update: did some more testing and it looks like sourcemap: true is the issue.  Once I removed that the other options work.

I also met this issue.
sourcemap: true doesn't work.
And... the owner doesn't seem to have used even GitHub since Oct 2018.


How to highlight a range/selection in the editor

I want to highlight a specific range/selection within the editor with a special background color. Is this currently supported?
For examples does the diff editor highlight lines. I want to highlight only a range/selection and not a complete line.

Solution: https://microsoft.github.io/monaco-editor/playground.html#interacting-with-the-editor-line-and-inline-decorations

@ebertmi @ebertmi Hi, I want to highlight conflict blocks like VSCode does, do you know how to do it?
https://stackoverflow.com/questions/57246356/how-to-highlight-merge-conflict-blocks-in-monaco-editor-like-vscode


Interpolation in an elements slot creates invalid output if element is created with  and has contentediable attribute

Vue version
3.2.37
Link to minimal reproduction
https://sfc.vuejs.org/#eNp9UU1r3DAQ/StTUbADawmaQ8HshpRS6KE99dCLLt7VZKNWloQ09iYY//eOvNnt5oMcDJ6ZN++N3pvElxjlOKBoxTrvko0EGWmIN9rbPoZEMEHCuxUE/zMMntDADHcp9FDxVqW99rvgM0Gf97Ap0Lr6js4F+B2SMx+qqxPgI7ongB+c47b2Z866voLNDUzaQ8HJsXMDys6YbyN6+mEzocdUV1s3pGoFF2iAwh4cShf29f9dwgf6GpjbE0sVHB94nPEVb+IKbGZw+QBOZ1NX3lUZOy6PXaujS+wPF4R9dB0hVwDrXWDHPDNBa/NGC17VotxX2NFY6rYOuc8rlhsP1ATvHhnCpnCbj+L/NtPjgpoO95bwV+x22EIVEzaH1MVq1mJRA5imxfR5XsTVWZ3Ha3W+TKzEMcim76L8k4PnqBfr9NMga9GezNSCUy21FvdEMbdKDT7+3UtmV7c8U4kDsz02JvS31/KTvP6sDOdz2ZeY+2abwiFjYkUtVhfkipsjpiahN5gwvSv2AvtM8MXslegSJ6cp5n8XQv7g
Steps to reproduce

double click to edit content
use enter to make a line
input something, like  cc
blur

What is expected?
the content is Hello World \n cc
What is actually happening?
the content is Hello World \n cc \n cc
System Info
No response
Any additional comments?
No response

temporally use v-text to solve this problem

It will add a new text Node after press enter. Vue only patch the focused text Node when blur. That's why the textContent is incorrent.


Use YAML for project files instead of JSON?

Why not use a more human readable and editable format like YAML instead of using the more verbose project.json format? I know that JSON is a more popular format, but YAML is pretty popular as well and is so dead simple and clean that it seems like it would make a nice clean new project system even cleaner and simpler.
Here is a sample in both JSON and YAML format. Judge for yourself.
https://gist.github.com/ejsmith/f8ff96a10675845d4293

Besides Swagger, I've not used YAML extensively BUT I find it terrible. JSON looks and works better for me.

I fooled around with ways of using different formats over here  https://github.com/aL3891/CustomPackageReferences


clean code and add docstring

hello @matrixgram
what are you trying to do with this PR?

Thank you. We plan to use black as autoformatter instead of autopep8. I'll take care of it at some point soon.


Add ability to specify username and password for elasticsearch

Adds the functionality as described in #98

breaking change.
you could have add the new parameters as a default params.
also you could have add a new ctor , or Ctor that gets a config object for future changes.
can you consider that?

The person you are trying to reach is no longer at Just Energy, please redirect your email.


bug in ndarray: incompatible type error

I think there is a bug in ndarray type hint.
I think that in case of np.array that has one row with only integers (type int or int[64]) and one row with at least one float (e.g. type float), then it produces an error since rows have different types so conflict like "np.ndarray[float] != np.ndarray[int]" occurs.
Reproducing code example:

# bug.py
import numpy as np
arr = np.array([[4.2, 2, 3.5], [12, 3, 6]])

and run mypy:
$ mypy bug.py

Error message:
bug.py:3: error: Argument 1 to "array" has incompatible type "List[object]"; expected "Union[List[bool], List[List[bool]], List[List[List[bool]]], List[List[List[List[bool]]]]]"

NumPy/Python/data-science-types versions information:
1.19.2 / 3.8.5 / 0.2.20

Yeah, that's a bug. I'm not sure we can fix it. Maybe we could use List[Union[int, float]].


Global Python version preceding Python venv

Describe the bug
On a  fresh install of RTX using homebrew, when you have a global version of python it takes precedence over virtualenv once you activate it.
What happens:
PATH=/Users/randomuser/.local/share/rtx/installs/go/1.20.2/go/bin:/Users/randomuser/.local/share/rtx/installs/go/1.20.2/packages/bin:/Users/randomuser/.local/share/rtx/installs/python/3.11.2/bin:/Users/randomuser/.local/share/rtx/installs/nodejs/18.15.0/bin:/Users/randomuser/dev/test/venv/bin:/Users/randomuser/Library/.......
What was expected:
PATH=/Users/randomuser/dev/test/venv/bin:/Users/randomuser/.local/share/rtx/installs/go/1.20.2/go/bin:/Users/randomuser/.local/share/rtx/installs/go/1.20.2/packages/bin:/Users/randomuser/.local/share/rtx/installs/python/3.11.2/bin:/Users/randomuser/.local/share/rtx/installs/nodejs/18.15.0/bin:/Users/randomuser/Library/.......
To Reproduce
Assuming you installed RTX using Homebrew and installed a version of Python:

Create a folder: mkdir tmp-rtx-venv
Create a python virtual environment: python -m venv .venv
Set a global Python version: rtx global python 3.11.2
Make a note of your current path. I'm using fish so I run env to dump all the variables.
Make a note of which python: which python
Activate the python virtual environment: source .venv/bin/activate.fish (for the fish shell, for others please adjust)
Recheck your PATH and python: env && which python

Expected behavior
Virtualenv path would be prepended to the PATH (eg. PATH = VENV_PATH + PATH) variable. Right now it's being added but the GLOBAL RTX config takes priority (eg. PATH = GLOBAL_RTX + VENV_PATH + PATH)
rtx doctor output
rtx version:
  1.26.2 macos-arm64 (6fbbcd3 2023-03-23)

build:
  Target: aarch64-apple-darwin
  Features: BREW
  Built: Thu, 23 Mar 2023 13:13:46 +0000
  Rust Version: rustc 1.67.1 (d5a82bbd2 2023-02-07)
  Profile: release

shell:
  /opt/homebrew/bin/fish
  fish, version 3.6.0

rtx environment variables:
  RTX_SHELL=fish

settings:
  {"experimental": "false", "missing_runtime_behavior": "prompt", "always_keep_download": "false", "legacy_version_file": "true", "plugin_autoupdate_last_check_duration": "10080", "verbose": "false", "asdf_compat": "false", "jobs": "4", "disable_default_shorthands": "false", "log_level": "INFO", "raw": "false"}

config files:
  /Users/$$$$/.tool-versions

plugins:
  go       https://github.com/kennyp/asdf-golang.git#f006a12
  nodejs   https://github.com/jdxcode/rtx-nodejs.git#a9af74f
  python   https://github.com/jdxcode/rtx-python.git#e21a6da

toolset:
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>
No problems found

Additional context
rtx ls output
⏵  go 1.20.2                 (set by ~/.tool-versions)
⏵  nodejs 18.15.0            (set by ~/.tool-versions)
⏵  python 3.11.2             (set by ~/.tool-versions)

which python output
/Users/randomuser/local/share/rtx/installs/python/3.11.2/bin/python

you can either set the version to "system" with rtx local python@system or you can use the experimental virtualenv support built into rtx

Just tested and it works fine with the experimental virtualenv support. I'm assuming this is the way forward for venvs & rtx?

yeah that's what I would do


Move to OpenCV 3.x

Version 1.1.1 of openalpr-android uses old OpenCV library 2.4.9 built on 2014-04-25. There are a lot of NEON optimizations in recent OpenCV 3x which should bring better performance to openalpr-android. Please update it to the latest stable OpenCV. OpenALPR should work with OpenCV 3 without any problem.

OpenALPR "SHOULD" or "WORKS" with OpenCV 3 without a problem? Have you done any tests on this? Can you provide a compiled OpenALPR?

Should
https://github.com/openalpr/openalpr/issues/131

@alexeyvasilyev check this: https://github.com/openalpr/openalpr/issues/273

Hi.. Are you planning to release a new version with OpenCV 3 ??

Hi @davidpini,
check this:  https://github.com/openalpr/openalpr/issues/273
I will try to compile it by myself, but it is not easy.

@SandroMachado thanks for trying.
I have already tried myself, but I found many errors.. I hope you will be luckier.


Improve landing README

Changes
Provide more useful links
Submitter Checklist

[ ] Includes tests if functionality changed/was added
[ ] Includes docs if changes are user-facing
[x] Set a kind label on this PR
[x] Release notes block has been filled in, or marked NONE

See the contributor guide
for details on coding conventions, github and prow interactions, and the code review process.
Release Notes
NONE

Merging per the lack of a review.


Chinese input method don't work when renaming a Node ?

Godot version
4.0 alpha 10
System information
Windows 11
Issue description
Some other input field is able to directly use Chinese pinyin input method.
However, when I was trying to rename a Node in a scene tree after pressing f2, pinyin input won't work.
-- the input is locked to English and I couldn't switch, just like no input field is detected.
Probably because when renaming a Node, OS hasn't found that there was a input filed. (other input field worked correctly)
(now I am having to use clipboard to rename Nodes into Chinese )
Steps to reproduce
just rename a Node from SceneTree, and try inputting Chinese character using pinyin.
Minimal reproduction project
No response

Windows version of #60372.

Duplicate of #60372


Support hybrid search in Aggregate

What's being changed:
The existing hybrid search functionality was hardcoded to work only with Get{} queries. This PR generalizes the hybrid search code, and uses it to support hybrid search within Aggregate{} queries, for use with filters and grouping.
Review checklist

[ ] Documentation has been updated, if necessary. Link to changed documentation:
[x] Chaos pipeline run or not necessary.

Link to pipeline: https://github.com/semi-technologies/weaviate-chaos-engineering/actions/runs/3888090623

[ ] All new code is covered by tests where it is reasonable.
[ ] Performance tests have been run or not necessary.

Closes #2482

@etiennedi yes, I think we should target stable/v1.17 to get it out earlier

@parkerduckworth I made a quick test with the latest master version according to https://weaviate.io/developers/weaviate/current/installation/#run-an-unreleased-version
I got the following error:

@eostis currently, a search vector is required for aggregation, since the aggregator has no access to the modules. I will look into providing the aggregator access to the module system. But in the meantime, is providing a search vector a reasonable workaround for your use case?

@parkerduckworth For e-commerce, aggregation should be able to work the same for Where, NearText, or hybrid. I'll wait for the fix. Do you want me to create a new issue ?

@eostis I just made a fix that will allow the query from your example above. When the CI pipeline is complete, I will send you an image tag so you can try it out

@eostis try this:
semitechnologies/weaviate:preview-support-implicit-search-vectors-for-aggregation-queries-518a5fd

@parkerduckworth Looks good now !


The dashboard should be connector fault tolerant

If the Abiquo connector is enabled, without valid credentials, the dashboard shows an error and doesn't display anything

Doesn't apply anymore.
And the Abiquo connector is not maintained anymore.


502 bad gateway

After changing a combo that had generated many variants to be a non-generator, the system crashed.

Solved with c4d7653b7650944f8b189b0767e6d831f54d8555


Command-line arguments (ARGV) not properly separated

For example, it took me a while to realize that the sentences given as arguments in the Pangram Grep hole are not one and the same argument, and I thought I had to recognize punctuation too. I'd say that a new textbox should be made for each argument or something.

Or just seperate them by newlines.

Yep, these are all good ideas, I'll try and knock out a prototype this weekend, there's a few improvements I want to try on the hole page.

@JRaspass Personally, I would avoid separation with \n and instead make many text boxes, otherwise what if the argument contains \n in it too?

@GolfingSuccess IMO that would take up too much space. I'd separate them by newlines unless they contain \n.

@somebody1234 The problem is that, then, I can't separate between only one argument which happens to contain at least one \n and many arguments which don't contain any \ns. But maybe the list can be scrollable.


Overload EmbeddedHeadersMessageConverter method

Overload extractHeaders method to accept bytes payload
This will help avoiding unnessary message creation during ad-hoc extractHeader invocation
add test

Resolves #780

Merged as https://github.com/spring-cloud/spring-cloud-stream/commit/e1171a38f737e4b801a50831de8cdf3d930ef10d and backported as https://github.com/spring-cloud/spring-cloud-stream/commit/cded4e0c87c83f3d9d7b292cfecc158c771d0af4


Fix #543: build with correct image names

Problem

Method extendImage created image with the default name ignoring --image-name option.
The image could not be tagged after building, because CLI exited with ERROR: denied: requested access to the resource is denied after pushing attempt.

Solution

Added image names from --image-names option at building stage.

@microsoft-github-policy-service agree


feat(lamp): add MusicBar & AppMusicCard component

Owner

[x] Nickname(Tuya): 木叶

Description
tuya-panel-lamp-sdk add MusicBar & AppMusicCard component.
Package

[x] tuya-panel-lamp-sdk

Type of change
Please select the relevant option.

[x] New feature (non-breaking change which adds functionality)

Review CheckList
Don't edit this section.

[ ] Intra-group review
[ ] ESLint lint
[ ] Example Run
[ ] Jest Run
[ ] No Grammer Error
[ ] No Spell Error

Codecov Report

Merging #102 (327fe72) into main (0a6a78f) will decrease coverage by 2.11%.
The diff coverage is 27.11%.

@@            Coverage Diff             @@
##             main     #102      +/-   ##
==========================================
- Coverage   61.57%   59.45%   -2.12%
==========================================
  Files         169      194      +25
  Lines        6417     7286     +869
  Branches     1493     1697     +204
==========================================
+ Hits         3951     4332     +381
- Misses       2452     2938     +486
- Partials       14       16       +2

Impacted Files
Coverage Δ

...ic/music-card/app-music-card/utils/MusicManager.ts
18.27% <18.27%> (ø)

...s/music/music-card/app-music-card/AppMusicCard.tsx
25.42% <25.42%> (ø)

...el-lamp-sdk/src/components/music/music-bar/Bar.tsx
40.00% <40.00%> (ø)

...mp-sdk/src/components/music/music-bar/MusicBar.tsx
64.70% <64.70%> (ø)

...music/music-card/app-music-card/config/appMusic.ts
100.00% <100.00%> (ø)

...ges/tuya-panel-lamp-sdk/src/hooks/useEffectOnce.ts
100.00% <100.00%> (ø)

...a-panel-ipc-sdk/src/ty-ipc-loading/style/index.tsx
100.00% <0.00%> (ø)

...remote-sdk/src/components/pusher-animate/index.tsx
92.53% <0.00%> (ø)

...remote-sdk/src/components/roller-animate/index.tsx
67.20% <0.00%> (ø)

... and 21 more

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 0a6a78f...327fe72. Read the comment docs.

LGTM~


TO:Internal server error while using traffic_portal in production build

In my case production db was running in a separate container and traffic_ops&portal are running in same container. While access the api/1.3/user/current its getting internal server error response code from traffic_ops server. DB container connection also connected successfully.

@itsmesridhar can you provide the logs from Traffic Ops for the internal server error you're getting?

@rawlinp  Request was not forwarded to traffic_ops that's the problem here. Traffic_ops & portal are running in container with the ip: <IP_ADDRESS>. While hit https request from host browser its asking for certificate exception permission and moving forward to load the static assets and its successful but it cannot able to access traffic_ops api. Traffic_portals logs are printed below.
configured portal base URL: http://localhost:8080/!#/
[root@c6974f816ccb /]# tail -f /var/log/traffic_portal/traffic_portal.log
at exports._exceptionWithHostPort (util.js:1043:20)
at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1099:14)
Error: connect ECONNREFUSED <IP_ADDRESS>:8080
at Object.exports._errnoException (util.js:1020:11)
at exports._exceptionWithHostPort (util.js:1043:20)
at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1099:14)
Error: connect ECONNREFUSED <IP_ADDRESS>:8080
at Object.exports._errnoException (util.js:1020:11)
at exports._exceptionWithHostPort (util.js:1043:20)
at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1099:14)
`

you updated /opt/traffic_portal/conf/config.js appropriately?

Yes, I have updated  /opt/traffic_portal/conf/config.js with the traffic_ops baseURL. The problem which I have faced was resolved. Problem happens because traffic_ops go app not started properly in Centos7 container. Thanks for your help :-)

hello, i have a same problem with you, internal server error when traffic portal open, so i cant add the delivery services. by the way im using rocky linux to build all the components. do you have any solution about my problem? thank you before


Right Axis Line using same path effect as x axis

Summary
Right now I'm trying to draw two different things in my chart:
1.- A grid on the x Axis that looks like this:

using:
   xAxis.enableGridDashedLine(10, height, 0);
2.- A Line all the way to the right of the chart:

using
    rightAxis.setEnabled(true);
    rightAxis.setAxisMaximum(this.graphMaximum);
    rightAxis.setAxisMinimum(this.graphMinimum);
    rightAxis.setDrawAxisLine(true);
    rightAxis.setAxisLineWidth(2);
    rightAxis.setAxisLineColor(this.colors.label);

The problem is that if I set a path effect for the x Axis (the little lines at the bottom of the chart), the right axis line also gets that path effect, ending up looking like this:

Is this intended or a bug in the code? if it is intended, is there a workaround for it? Thank you.

@DiegoMacBD i cant see to see anything looking at the code. Could you create a repro sample? (it is abug)

@DiegoMacBD should be fixed in 1.2.13

Thank you so much, for all the help, indeed it is fixed :)


Add type annotations to all examples

empirically having type annotations on all functions helps people learn the language faster.

Any evidence to support this?
I have had it both ways over the past few years and someone always objects to whatever choice is made. Beginners tend to say that the types are confusing. Experts say that they are obviously helpful and necessary. I think it has a good balance as is.


Quran Script is missing stop signs

The original Quran Script should contain necessary stop signs for better readability. Consider using tanzil.net's data.

Done ✅✅


Add 'lightweight & fast' feature in docs.phcode.dev homepage

Introduced a New Feature: 'Lightweight & Fast' Highlight on Homepage
Before the change

After the change

Thanks @devvaannsh for contributing.
Please sign the CLA by following the steps @ https://phcode-dev.github.io/contributor-license-agreement/

@devvaannsh for the image, Please chek the license too. We can only add images with MIT/CC or permissive licenses. We also need to credit the image in a new credits.md file. We dont have one now as the images we have currently are owned by us and given by our UX desigers.
The image needs to be inline with the overall aesthetic of the page, since other two images are monochrome, alin with that design too.


Implement cost_bar.py into the process package

https://github.com/ukaea/PROCESS/pull/2871

Fixed in #3266


REST API proposal

Auth

POST /login/auth/{provider} - Associate user with local account (if needed) and return JWT token, that encapsulates the user id and other user information, that must be used for subsequent API calls.

{
  "userid": "34678",
  "email"<EMAIL_ADDRESS>}

The following API calls are all authenticated and require the JWT token from the login endpoint to call. This means that each of these APIs scoped to a user.
Users

GET /user - Get a user's information

Conversations

POST /conversations/ - Create a conversation (passing a list of contact ids)

{
  "contacts": [ 12, 23, 45 ]
}

GET /conversations - Get all conversations for a user
DELETE /conversations/{conversationId} - Delete a conversation (maybe a soft delete)

Contacts

GET /contacts/ - Get a user's contact list
POST /contacts/{contactEmail}  - Add a contact for a this user by email
DELETE /contacts/{contactId} - Delete a contact

Messages

POST /messages/{conversationId} - Add a new message to a conversation
GET /messages/{conversationId} - Get chat history for a conversation (this needs to be paged)

Started working on this in PR #23  and PR #26.

This all done!? Nice!

This all done!? Nice!

Yup. We just have to build on the login stuff but the basics are there and functional.


[Road Map] Dark Theme

For the dark theme,

Would you rather save the settings on local storage,
A field in the wink_users table,
Add a new table (wink_user_settings) that stores this setting and/or any new future user setting added to the application?

Create new stylesheet for dark theme and override tailwind classes,
Check dark/light settings per component and add/remove tailwind classes accordingly?

I believe that these settings should be per author, probably will be stored in a JSON field in the users table. I like JSON fields because it makes us avoid joining tables when it's not really necessary.
I think for the dark theme I think we'll utilize css variables and see if that gives us decent results. It'll require some experimenting. If that didn't work then I guess we'll use a separate tailwind config file and stylesheet.


#enhance/improve-user-messaging-on-node-delete-by-specifying-the-vault-of-the-deleted-node

#enhance/improve-user-messaging-on-node-delete-by-specifying-the-vault-of-the-deleted-node
Pull Request Checklist
You can go to dendron pull requests to see full details for items in this checklist.
General
Quality Assurance

[ ] make sure all the existing tests pass
[x] do a spot check by running your feature with our test workspace
[ ] after you submit your pull request, check the output of our integration test and make sure all tests pass

NOTE: if you running mac/linux, check the windows output and vice versa if you are developing on windows

Added test.


Lottie glitch with Rubberhose - subframe interpolation

Description
When exporting to Lottie with rubberhose animation, I got some glitch.
This is the same bug as this two tickets (Lottie web) :
issue 1
or
issue 2
This problem seems solved by using
anim.setSubframe(false)

There is no method available in the react native Lottie library as described.
There is a quote of the answer found for the Lottie web library :

usually those glitches are due to subframe interpolation, if the developer calls anim.setSubframe(false) , they will probably get fixed

I spent time to work around but I didn't solved the problem.
I re-do the animation a lot of time but nothing change.
Steps to Reproduce
Expected behavior: [What you expected to happen]
No glitch inside my animation
Actual behavior: [What actually happened]
Glitch are there in the web browser preview
and inside the react native app.
Bodymovin animation preview is ok.
Versions
lottie-ios: 3.2.3
lottie-react-native: ^5.0.1
react-native": 0.65.1

After effect Rubberhose v2.09 and last version of after effect CC

I'm having the same issue and even have the shape layer/hose vanish when previewing on my desktop within the lottie preview app. Any resolution here?

Not all After Effects features are supported by Lottie. If you notice there are some layers or animations missing check this list to ensure they are supported.


Issues/29 port gatk germline pipeline

Added preprocessing script for the GATK germline pipeline.

@fnothaft -- Do you want to work on this PR or should I merge it now?

Thanks @jpfeil! Have you tested this on your side? If you've tested, then I am good for merge @jvivian.

No problem! I tested it on single machine and it works. @jvivian @fnothaft

Thanks @jpfeil! @jvivian this is good to go from my side.


Improve handling of saturated data

Issue is that PVSS/frequency analysis of saturated data is hot garbage.
System should

Alert user to the existence of saturated channels
Not use saturated data for all or some processing

Duplicates #170


Added links to drivers APIs

Added links to drivers APIs and made edits.

@mhopkins-msft : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Missing info on corp account balances

The notes on the corp_accountbalance page notes the following

accountKey: This is always 1000 for a character’s account balance as opposed to a corporation, where accountKey ranges from 1000 to 1006 for the seven corporate wallet divisions.

I am seeing another accountKey on my corporation's info. '10000', is this possibly market escrow?

I believe accountKey 10000 is the DUST wallet.

Yea, 10,000 should be DUST.

Ah, I see. I'd suggest adding a note on the page but with DUST closing shortly I imagine this will be removed anyway.


KeyError: 'pip._vendor.urllib3.contrib'

$ zappa deploy
Calling deploy for stage dev..
Oh no! An error occurred! :(
==============
Traceback (most recent call last):
File "/Users/ox/.local/share/virtualenvs/ox-zje1HLlw/lib/python3.6/site-packages/zappa/cli.py", line 2610, in handle
sys.exit(cli.handle())
File "/Users/ox/.local/share/virtualenvs/ox-zje1HLlw/lib/python3.6/site-packages/zappa/cli.py", line 505, in handle
self.dispatch_command(self.command, stage)
File "/Users/ox/.local/share/virtualenvs/ox-zje1HLlw/lib/python3.6/site-packages/zappa/cli.py", line 539, in dispatch_command
self.deploy(self.vargs['zip'])
File "/Users/ox/.local/share/virtualenvs/ox-zje1HLlw/lib/python3.6/site-packages/zappa/cli.py", line 709, in deploy
self.create_package()
File "/Users/ox/.local/share/virtualenvs/ox-zje1HLlw/lib/python3.6/site-packages/zappa/cli.py", line 2171, in create_package
disable_progress=self.disable_progress
File "/Users/ox/.local/share/virtualenvs/ox-zje1HLlw/lib/python3.6/site-packages/zappa/core.py", line 447, in create_lambda_zip
import pip
File "/Users/ox/.local/share/virtualenvs/ox-zje1HLlw/lib/python3.6/site-packages/pip/init.py", line 45, in
from pip.vcs import git, mercurial, subversion, bazaar  # noqa
File "/Users/ox/.local/share/virtualenvs/ox-zje1HLlw/lib/python3.6/site-packages/pip/vcs/mercurial.py", line 9, in
from pip.download import path_to_url
File "/Users/ox/.local/share/virtualenvs/ox-zje1HLlw/lib/python3.6/site-packages/pip/download.py", line 40, in
from pip._vendor import requests, six
File "/Users/ox/.local/share/virtualenvs/ox-zje1HLlw/lib/python3.6/site-packages/pip/_vendor/requests/init.py", line 98, in
from . import packages
File "/Users/ox/.local/share/virtualenvs/ox-zje1HLlw/lib/python3.6/site-packages/pip/_vendor/requests/packages.py", line 12, in
sys.modules['pip._vendor.requests.packages.' + mod] = sys.modules["pip._vendor." + mod]
KeyError: 'pip._vendor.urllib3.contrib'

settings:
{
"dev": {
"app_function": "hello.app",
"aws_region": null,
"profile_name": null,
"project_name": "zappa-hey-yo",
"runtime": "python3.6",
"s3_bucket": "zappa-hwydhsa7d"
}

I was about to post this same issue in order to help others. After debugging for a while, I found that this is happening because of pip version 9.0.2, released 3 days ago (March 16, 2018).
If any one else is facing the same issue, downgrade to  pip version 9.0.1 pip install pip==9.0.1 and Zappa will work as usual.

Duplicate of #1446

Nice work @suriya, thank you!

thank you, was just about to start debugging!

Closing duplicate. Track upstream here: https://github.com/pypa/pip/issues/5081

For people on Windows in a virtualenv, downgrading won't work (you'll uninstall pip and then won't be able to reinstall the older version).
The solution is to uninstall pip (pip uninstall pip), then get the get-pip.py installation script from the pip docs and then do python get-pip.py pip==9.0.1

Thank you sooo much, but why?


Workspace - Invalid chat is created if inviting account via secondary login

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

Action Performed:
Prerequisites: accountB has a secondary login email

Under accountA create workspace
Under accountA Invite accountB using accountB secondary login

Expected Result:
Only one WS direct chat with accountB should be created
Actual Result:
2 chats are created using main and secondary emails. The one with secondary login is invalid
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[x] Android / Chrome
[ ] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/93399543/7d0038d8-e71f-4089-8cf8-d17be598d8b4
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub

Ah, sorry Ted for the back to back! I'm reassigning some of my more urgent GHs that I couldn't get to before OOO/weekend.

Ugh - Singapore wifi does not like NewDot for some reason

2 chats are created using main and secondary emails. The one with secondary login is invalid

I cannot get this to work. Every time I invite<EMAIL_ADDRESS>which is the secondary login for<EMAIL_ADDRESS>I can't get the two workspace chats?
https://github.com/Expensify/App/assets/9133401/dea5c204-baa2-4380-8036-0a04ae78644f
I do just get a report no longer exists if I try to navigate to that chat DM, but I can't find a workspace DM:

I'm not entirely sure what should happen if you invite a secondary login. On oldDot, we just automatically update to show the primary login instead:

So I think that should be what we should do here - but surface that message as well when adding them. cc @Expensify/product

If we're grabbing a message from web-E, I think this might need to be internal though.

Yep this will be a backend change.
@mananjadhav removing you since there won't be anything for you to review, all changes will be internal. Thanks!

I have read and reviewed this Project Manager Application!

All done, the fix was deployed to our backend last week.


Update bundle version from 4.4.8 to 4.5.0-rc.5

/retest

I assume nothing is confused by the -rc.5?

@cfergeau you mean not using the rc bits here or anything else?

/hold
Will also try to fix the operator test case with same PR.

I assume nothing is confused by the -rc.5?

@cfergeau you mean not using the rc bits here or anything else?

Not suggesting changing anything, just mentioning that I can't think of crc code which would be impacted/confused by the -rc.5


crash with too many threads

after upgrading my linux mint to the newest version with ubuntu 24.04 as base for unknown reasons tundra thinks I have a machine with 128 cores and crashes. I see that the max threads is limited to 64 and right afterwards it crashes. when taking debugging-logs I can see that 128 threads are reported (dunno why, I'm using a virtual machine with 2 or 4 cores configured), 128 threads are created and then it crashes.
manually setting number of threads to a low number fixes the issue, but using more than 64threads will crash tundra after it clamped the number of threads to 64... so it's easy reproducable by setting the threadnumber to 65 and let tundra run.
the complete log is here:
[D] digest cache initialized -- 0 entries
[D] .tundra2.dag: successfully mapped at 0x74ac3765e000 (756736 bytes)
[D] checking file signatures for DAG data
[D] DAG signatures match - using existing data w/o Lua invocation
[D] .tundra2.state: successfully mapped at 0x74ac37acd000 (33920 bytes)
[D] .tundra2.scancache: successfully mapped at 0x74ac3764b000 (74384 bytes)
[D] Scan cache initialized from frozen data - 475 entries
[D] Node selection finished with 5 nodes to build
[D] Node remap: 657 src nodes, 217 active nodes, using 13888 bytes of node state buffer space
[D] Max # expensive jobs: 128
[W] too many build threads (128) - clamping to 64
[D] build queue initialized; ring buffer capacity = 256
[D] starting build thread 1
[D] starting build thread 2
[D] starting build thread 3
[D] starting build thread 4
[D] starting build thread 5
[D] starting build thread 6
[D] starting build thread 7
[D] starting build thread 8
[D] starting build thread 9
[D] starting build thread 10
[D] starting build thread 11
[D] starting build thread 12
[D] starting build thread 13
[D] starting build thread 14
[D] starting build thread 15
[D] starting build thread 16
[D] starting build thread 17
[D] starting build thread 18
[D] starting build thread 19
[D] starting build thread 20
[D] starting build thread 21
[D] starting build thread 22
[D] starting build thread 23
[D] starting build thread 24
[D] starting build thread 25
[D] starting build thread 26
[D] starting build thread 27
[D] starting build thread 28
[D] starting build thread 29
[D] starting build thread 30
[D] starting build thread 31
[D] starting build thread 32
[D] starting build thread 33
[D] starting build thread 34
[D] starting build thread 35
[D] starting build thread 36
[D] starting build thread 37
[D] starting build thread 38
[D] starting build thread 39
[D] starting build thread 40
[D] starting build thread 41
[D] starting build thread 42
[D] starting build thread 43
[D] starting build thread 44
[D] starting build thread 45
[D] starting build thread 46
[D] starting build thread 47
[D] starting build thread 48
[D] starting build thread 49
[D] starting build thread 50
[D] starting build thread 51
[D] starting build thread 52
[D] starting build thread 53
[D] starting build thread 54
[D] starting build thread 55
[D] starting build thread 56
[D] starting build thread 57
[D] starting build thread 58
[D] starting build thread 59
[D] starting build thread 60
[D] starting build thread 61
[D] starting build thread 62
[D] starting build thread 63
[D] starting build thread 64
[D] starting build thread 65
[D] starting build thread 66
[D] starting build thread 67
[D] starting build thread 68
[D] starting build thread 69
[D] starting build thread 70
[D] starting build thread 71
[D] starting build thread 72
[D] starting build thread 73
[D] starting build thread 74
[D] starting build thread 75
[D] starting build thread 76
[D] starting build thread 77
[D] starting build thread 78
[D] starting build thread 79
[D] starting build thread 80
[D] starting build thread 81
[D] starting build thread 82
[D] starting build thread 83
[D] starting build thread 84
[D] starting build thread 85
[D] starting build thread 86
[D] starting build thread 87
[D] starting build thread 88
[D] starting build thread 89
[D] starting build thread 90
[D] starting build thread 91
[D] starting build thread 92
[D] starting build thread 93
[D] starting build thread 94
[D] starting build thread 95
[D] starting build thread 96
[D] starting build thread 97
[D] starting build thread 98
[D] starting build thread 99
[D] starting build thread 100
[D] starting build thread 101
[D] starting build thread 102
[D] starting build thread 103
[D] starting build thread 104
[D] starting build thread 105
[D] starting build thread 106
[D] starting build thread 107
[D] starting build thread 108
[D] starting build thread 109
[D] starting build thread 110
[D] starting build thread 111
[D] starting build thread 112
[D] starting build thread 113
[D] starting build thread 114
[D] starting build thread 115
[D] starting build thread 116
[D] starting build thread 117
[D] starting build thread 118
[D] starting build thread 119
[D] starting build thread 120
[D] starting build thread 121
[D] starting build thread 122
[D] starting build thread 123
[D] starting build thread 124
[D] starting build thread 125
[D] starting build thread 126
[D] starting build thread 127
[I] begin pass Compile generator (nodes: 0 -<PHONE_NUMBER><PHONE_NUMBER>))
tundra2 -D -w terminated by signal SIGSEGV (Address boundary error)

and the call-stack:
t2::BuildQueueBuildNodeRange(t2::BuildQueue*, int, int, int)
t2::DriverBuild(t2::Driver*)
??
??

update: after updating my codebase the crash is gone, I think it's detecting the number of threads correct now. but still, if you start tundra with "-j 65" it crashes, so I will keep this ticket open, even when it's not happening for me by default anymore.

hm, no idea what is going on, here https://github.com/deplinenoise/tundra/blob/e48e03bbd8a193889337d25b42cfbd64c64c1c33/src/BuildQueue.cpp#L742  the number of threads is clamped, but still (as seen in the logs) more threads are created than the clamped number (and it will crash because there is not enough space for all the threads and memory will be overwritten). I've compiled tundra with clang, is it an error in the alias analysis of the compiler and with gcc it's working?

No idea, but I'll try to repro this

At first glance the code looks correct to me

Yeah I can sort of repro on this random Ubuntu machine I have, compiling with gcc. I don't create more threads than 64 even if I specify more on the command line, but it does crash with a stack smash protection when it happens:
[D] joining with build thread 57
[D] joining with build thread 58
[D] joining with build thread 59
[D] joining with build thread 60
[D] joining with build thread 61
[D] joining with build thread 62
[D] joining with build thread 63
*** stack smashing detected ***: terminated
Aborted (core dumped)

This doesn't happen with -j 64 so something is up.


Add raise error_handler

This change is

Coverage decreased (-0.05%) to 50.341% when pulling 0e89af30b4ddb31c85efebac6645761b7f287109 on annawzz:a258001251842882_refactoring into 2d83f5b7b304bb7249e83119b5d1f7f48dccc0e0 on openprocurement:a258001251842882_refactoring.

А чому Coverage лише 50%


[BUG] Netspace showing vastly lower number than correct

Describe the bug
On installing 1.16 fresh, network is synced to latest block and up to date but showing netspace as 1.536mib only when current netspace is in pib
To Reproduce
Steps to reproduce the behavior:
Fresh install
Expected behavior
Expect netspace to show correct current network storage space
Screenshots

Desktop (please complete the following information):

OS: Windows 10 x64
CPU: Intel Core i7 10700KF

@ab212121 the issue doesn't occur anymore so I'm closing the issue.


Make autotune tests optional

[ ] Closes #xxxx (Replace xxxx with the GitHub issue number)
[ ] Tests added and passed if fixing a bug or adding a new feature
[ ] All code checks passed.
[ ] Added type annotations to new arguments/methods/functions.
[ ] Added an entry in the latest docs/release_notes/index.md file if fixing a bug or adding a new feature.
[ ] If the changes are patches for a version, I have added the on-merge: backport to x.x.x label.

Closing in favor of #2316


Adds 'blit', 'copy', 'fill' and 'sub' functions to Array.

A minor contribution.
Happy Easter :)

Nice work, thank you! Happy Easter to you too!


Having an error with less

Hi there,
First great tool! Thank you for your effort :)
I installed the tool on my Linux VM (Ubuntu 12.04.5 LTS) and everything went well. I installed it with the dependencies mentioned in the detailed instructions.
I can login to my PostgreSQL instance but every time I run a command in pgcli I received the following message: There is no no-initXF option ("less --help" for help)
and nothing is returned except the "Command Time" and "format Time" output.
It seems a problem with Less pager. Google the problem but cannot find anything related to it.
Would you have any idea? The output of my $less --version is: less 444

That's interesting. I'm overriding the LESS env var to not clear the screen while inside the pgcli environment, I didn't think of the --no-init option. I've got a fix for that in the master. Would you mind trying it out?
You can install it directly from the git repo as follows:
pip uninstall pgcli         # This will uninstall the existing pgcli.
pip install -U -e git+https://github.com/amjith/pgcli.git#egg=pgcli

Done it and it fixed the issue :)
Thanks!

0.15.4 released to PyPI.


How to get the label of image3(right image)

Dear Li:
Thanks a lot for sharing your code!
But I'm quite confused how do you get the GT label of the right image during RPN stage.
Looking forward to your reply.
Best wishes.

Because the extrinsic parameter is given, so we can project the 3D bounding box to the right image to get the 2D box.
Please refer to https://github.com/HKUST-Aerial-Robotics/Stereo-RCNN/blob/master/lib/model/utils/kitti_utils.py#L223

Great appreciation for your reply!
May I ask a personal question?
If I want to project the Lidar point from the velodyne coordinate system to camera 3 coordinate system, what's projection equation like?

https://github.com/HKUST-Aerial-Robotics/Stereo-RCNN/blob/master/lib/model/utils/kitti_utils.py#L303

https://github.com/HKUST-Aerial-Robotics/Stereo-RCNN/blob/master/lib/model/utils/kitti_utils.py#L303

Thanks for your reply, but it's the transform to camera2, how to get the coordinate under camera3?

You can transform it to any frame following this function.

You can transform it to any frame following this function.

Thanks for your patience.
But I found that while projecting the lidar cloud to camera2, your code is a little different from that in AVOD.
https://github.com/kujason/wavedata/blob/c4b5aabd9eb3b74fad777349f75161032d3460fa/wavedata/tools/core/calib_utils.py#L371
And the result is a little different.Can you explain what the frame_calibration_info.p2_2/frame_calibration_info.p2_3/t_cam2_cam0 mean?
Best wishes.

Seems the kitti 3D object label is on the camera_2 frame, so we calculate the extrinsic between the camera_3 and camera2, then transform it to the camera_3 frame.
You can easily validate it by visualizing the 3D box on the right image.


fix: validate info when importing ldap user/group/domain

这个 PR 实现什么功能/修复什么问题:
修正：ldap导入用户、组、域时检查信息的有效性
是否需要 backport 到之前的 release 分支:

release/2.10.0
release/2.11.0

/area keystone

/approve
/lgtm

Codecov Report

Merging #1885 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master   #1885      +/-   ##
=========================================
- Coverage     7.6%    7.6%   -0.01%
=========================================
  Files         412     412
  Lines       63672   63672
=========================================
- Hits         4844    4841       -3
- Misses      58419   58421       +2
- Partials      409     410       +1

Flag
Coverage Δ

#aFlag
7.6% <ø> (-0.01%)
:arrow_down:

Impacted Files
Coverage Δ

pkg/util/rbacutils/rbac.go
73.83% <0%> (-0.94%)
:arrow_down:

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 e28da4e...2165560. Read the comment docs.


Feature/arrow 16.1.0

Does this recipe build for you? You define -DARROW_PARQUET=ON which in turn requires thrift.
My build with MACHINE=intel-corei7-64 from the meta-intel BSP was terminated because it wanted to download thrift-0.16.0.tar.gz (among others, including xsimd for which a recipe is pending) and do_compile does not allow downloading.
Adding thrift to DEPENDS works for apache-arrow-native (a build dependency for python3-pyarrow-native) but not for the target build for some reason:
| CMake Error at /home/zozo/test-yocto-5.0/tmp-sicom-glibc/work/corei7-64-oe-linux/apache-arrow/16.1.0/recipe-sysroot/usr/lib/cmake/thrift/ThriftConfig.cmake:42 (message):
|   File or directory /usr/include/thrift referenced by variable
|   THRIFT_INCLUDE_DIR does not exist !
| Call Stack (most recent call first):
|   /home/zozo/test-yocto-5.0/tmp-sicom-glibc/work/corei7-64-oe-linux/apache-arrow/16.1.0/recipe-sysroot/usr/lib/cmake/thrift/ThriftConfig.cmake:58 (set_and_check)
|   cmake_modules/FindThriftAlt.cmake:56 (find_package)
|   cmake_modules/ThirdpartyToolchain.cmake:291 (find_package)
|   cmake_modules/ThirdpartyToolchain.cmake:1772 (resolve_dependency)
|   CMakeLists.txt:543 (include)
|
|
| -- Configuring incomplete, errors occurred!

Manually defining -DTHRIFT_INCLUDE_DIR=... in EXTRA_OECMAKE does not help either.

That is odd. The recipe does build for me. My machine is a custom aarch64 based machine, but I don't understand how that should be affecting this particular aspect of the build process.
What apache-arrow is doing for me in this case is build thrift on it's own if I don't provide it. See an extract from my apache-arrow log.do_configure:
CMake Warning at cmake_modules/FindThriftAlt.cmake:56 (find_package):
  By not providing "FindThrift.cmake" in CMAKE_MODULE_PATH this project has
  asked CMake to find a package configuration file provided by "Thrift", but
  CMake did not find one.

  Could not find a package configuration file provided by "Thrift" (requested
  version 0.11.0) with any of the following names:

    ThriftConfig.cmake
    thrift-config.cmake

  Add the installation prefix of "Thrift" to CMAKE_PREFIX_PATH or set
  "Thrift_DIR" to a directory containing one of the above files.  If "Thrift"
  provides a separate development package or SDK, be sure it has been
  installed.
Call Stack (most recent call first):
  cmake_modules/ThirdpartyToolchain.cmake:291 (find_package)
  cmake_modules/ThirdpartyToolchain.cmake:1772 (resolve_dependency)
  CMakeLists.txt:543 (include)

-- Could NOT find ThriftAlt: (Required is at least version "0.11.0") (found ThriftAlt_LIB-NOTFOUND)
-- Building Apache Thrift from source

Let me post an updated verison of the apache-arrow recipe that is more similar to the previous one. Maybe I introduced a few unnecessary changes here.

Thank you very much for taking your time to review my contribution and for sharing your knowledge and findnings!

You're welcome. Thanks for your contribution.
FYI, I have use the "Rebase and merge" option to merge your commit, so the hash is different in my repo than in yours.
Also added a new commit so at least SIMD usage is somewhat more sensible than before.


Fix scroll for texture_region and polygon_2d editor

The Travis fail seems to be on their side, not on the PR.
You may want to trigger it again by doing git commit --amend and then git push --force.

You may want to trigger it again by doing git commit --amend and then git push --force.

No need, I restarted the failed build.

Thanks!


Create an intro

Add an intro about yourself: who are you, what is your strength, why are you here.

Wow good idea.


add support for sphinxcontrib-video

Added support to process video nodes and publish assets to Confluence.
sphinxcontrib-video - https://github.com/sphinx-contrib/video

Had time and was able to adjust the review suggestions. Thanks for the contribution to support this new third-party extension.


[App]: Filter option "NOT IN" doesn't work correctly

Current Behavior
Within a particular group, selecting the filter option "NOT IN" with a set of options doesn't show any run. But using each of the options individually with "is not" works.
Expected Behavior
The runs shown for  "NOT IN a,b,c" should be same as "is not a"+"is not b"+"is not c"
Steps To Reproduce
No response
Screenshots
No response
Environment
OS: Mac OS Catalina
Browsers: Chrome
Version: 102.0.5005.61
Additional Context
No response

Hey @rahulsiripurapu, thanks for writing in. We are aware of this issue. I'll bump up the priority of the ticket and notify you once there are updates on this.


Fork description

It is mentioned that newsboat is a fork of newsbeuter. But there's no mention of why it is forked and what is changed in readme.md or newsboat.org. No need to have full list of changes, but at least create an entry in readme for fundamental differences.
As newsbeuter user, I am interested in this product, but I have no idea what to expect from it and I believe that some people who discovered this project have same feeling about it.

That's a good point; thanks! I've updated the README to make the difference more apparent.
I don't want to explicitly set any expectations because I don't have an explicit agenda myself. Looking at those two years when I maintained Newsbeuter, I guess one could say that my priorities are bugfixing and gradual modernization. Those are emergent, though; I didn't plan to concentrate on those things, they just happened. You can probably expect more of them in the near future (couple of years), then you'll have to re-calibrate depending on how Newsboat does.


Applied self review suggestions

Description
Applied some immediate review suggestions I had:

Fix import formatting
Change e2e import path
Remove usage of time.Now()

closes: #XXXX

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[ ] Targeted PR against correct branch (see CONTRIBUTING.md)
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Code follows the module structure standards.
[ ] Wrote unit and integration tests
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/)
[ ] Added relevant godoc comments.
[ ] Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
[ ] Re-reviewed Files changed in the Github PR explorer
[ ] Review Codecov Report in the comment section below once CI passes

Colin, thank you so much for both the review and the PR.
I think the time.now -> blocktime thing is very important.
<3


Adds config file/error handling

Since you seem to be accepting pull requests now I figured I'd submit these changes again, not that I have any use for this project anymore but it might be useful for other people :)

I'll take a closer look at this.

No worries, I added in a check to see if at least 1.ogg exists and is readable to help prevent issues such as #14 and #13 popping up, perhaps this should be stricter in actually checking the filetype but meh.

Any updates on this @CodeBrauer

I will check this asap, currently have to much projects at work for this :)

Guess this is dead in the water, also noticed a minor mistake I made. will commit fix in a sec.

@CodeBrauer Any updates?


PR for Release

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

npm package (@latest dist-tag)
GitHub release
v1.1.0
v1.1.0

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release
v1.1.0
v1.1.0

Your semantic-release bot :package::rocket:


Adds the sumconnector component

Description:
Adds the sum connector to the distro for turning logs/spans/etc into metrics
Link to Splunk idea: Discussed internally
Testing: No additional testing other than what is in the connector.
Documentation: None added

Replaced by https://github.com/signalfx/splunk-otel-collector/pull/5733


Include Brazil Flag

@esganzerla Thank you for contributing. :smile: I have left few fixes required. Apart from that, please update the README.md as per Contributions guidelines and also add snapshot of the flag.

@ManrajGrover I've fixed those issues. In fact I worked a little more to make to organize better the spacings. Thanks

@esganzerla This differs a little when compared to original flag. Can you share some sources where one developed by you is official? Brilliant work btw. :smile:

@ManrajGrover You mean like this?

@esganzerla Yeps

Hey @ManrajGrover, I actually managed to insert the stars to the flag. Should do anything else to close this pull request?

@esganzerla Brilliant work! :smile: This looks perfect to me. One thing left is to resolve the merge conflicts.

Seems to me that your are the one who can merge, isn't it?
I have experience with pull requests, can you help me if I have to do it mysel?
Thanks

@esganzerla Thank you for fixing the conflicts! 😄


Added jpeg, pdftools, and geospatial dependencies

Dependencies added from survey of R users local packages. I've also added the common dependencies used by spatial R packages.
One exception is the sf package that requires version >2 of gdal, which isn't in the ubuntu default rep. Can be added here sudo add-apt-repository -y ppa:ubuntugis/ubuntugis-unstable but not part of this pull request.

👍


verifyBulk errors when creating new Model.UsVerification after successful API call

Summary
I'm getting an error from within the @lob/lob-typescript-sdk when I attempt to verify bulk address.
Expected Behavior
verifyBulk is able to construct new UsVerification objects after a successful response to the verifyBulk API
Current Behavior
verifyBulk throws an error after successful response from Live API
TypeError: Models.UsVerification is not a constructor
   at UsVerifications.set (/path-to-project/node_modules/@lob/models/us-verifications.ts:51:30)
   at new UsVerifications (/path-to-project/node_modules/@lob/models/us-verifications.ts:27:21)
   at /path-to-project/node_modules/@lob/api/us-verifications-api.ts:288:16
   at processTicksAndRejections (node:internal/process/task_queues:96:5)
node_modules/@cspotcode/source-map-support/source-map-support.js:594
Process exited with code 1

Possible Solution
There seems to be some issue with how that code is being compiled. See UsVerifications inside the compiled index.cjs.js file which is most easily found inside the node_modules folder of any given project.
Steps to Reproduce

Set up a project similar to: https://github.com/jbaum012/lob-verifyBulk-error
execute/debug the verifyBulk method

##Context

Hey @jbaum012,
Thanks for bringing this to our attention, I will be looking into it in the next week or so and will keep you updated.

@jbaum012  I have put out a new version of the sdk (v1.1.0) that can be found on npm or directly here. I will leave this issue open for a few days to give you a chance to verify everything works. Thanks!

Thank you for the quick response @BennyKitchell! Unfortunately I still see the error when debugging and also in the compiled js script.  I've updated my package.json in the example repo if you'd like to test/reproduce.

@BennyKitchell I did a little more digging, and I do see that the UsVerifications reference in index.mjs looks correct; however, the common js version, index.cjs.js is still using new Models.UsVerification.  Not sure if that helps, but it might be a good lead.

@jbaum012  I will continue digging and see what I can find. I wasn't able to replicate it using the data you had in the example repo. Are you using your lob live key or your test key when performing these calls?
And just to confirm, you are making these calls using Javascript and not Typescript? I just want to make sure I have as close to a clone as possible for testing.

I'm running into the same issue as @jbaum012 on version 1.2.0 of the Typescript SDK. Adding to the hypothesis that the issue is in index.cjs.js, I manually changed this block of the sdk's package.json
  "main": "./dist/index.cjs.js",
  "module": "./dist/index.mjs",
  "types": "./dist/index.d.ts",
  "exports": {
    "node": {
      "import": "./dist/index.mjs",
      "require": "./dist/index.cjs.js"
    },

to this to force referencing the ecma module instead of the common js one
  "main": "./dist/index.mjs",
  "module": "./dist/index.mjs",
  "types": "./dist/index.d.ts",
  "exports": {
    "node": {
      "import": "./dist/index.mjs",
      "require": "./dist/index.mjs"
    },

The issue went away.

@jbaum012 @deloreyj  I have instituted the change suggested above. I will be releasing 1.2.1 by EoD today. Please test it and let me know if this change fixes the issue. Thanks so much for the patience!

@deloreyj @jbaum012  New version can be found here Thanks!

Hey @BennyKitchell I'm not sure if that did the trick or not.  I'm getting a new error (in multiple projects):
Error: Cannot find module `[path to my<EMAIL_ADDRESS>
Downgrading seems to also have the same issue though...so I'm not sure if this is related in any way to new code you've deployed 🤔.  When I install the package (both 1.1.0 and 1.2.1) and I look in node_modules there is indeed nothing in there for dist/index.cjs.js

@jbaum012  I see the issue, missed a commit. This is what I get for trying to rush a release directly after holidays. Will push out a new version soon and update here. Thanks for your patience!

@jbaum012 @deloreyj  v1.2.2 has been released here, please let me know if there's any problems with this release. Thanks!

@BennyKitchell Progress, but I'm getting a new error in 1.2.2.

Error [ERR_REQUIRE_ESM]: require() of ES Module [path to<EMAIL_ADDRESS>not supported.
Instead change the require of [path to<EMAIL_ADDRESS>to a dynamic import() which is available in all CommonJS modules.
at Object. ([path to project]/build/index.js:40:28) {
code: 'ERR_REQUIRE_ESM'
}

@jbaum012  With your permission, I am going to pull down your bulkverify repo and work on it locally on my own time until I can find where the problem is hiding.

Success! Looks like everything is working in 1.2.3


MultipleSelectList setSelected return Function anonymous

<MultipleSelectList setSelected={(val) => console.log(val)} />
With this code val has Function anonymous as value. Why is this?

same issue, has not been resolved yet

issue still exists....

Non-engineers should not write such software. We are wasting time because of these plugins written by unprofessional people...

This issue still persists

I managed to get the value on setSelected doing like this:
<MultipleSelectList
  setSelected={(getSelected: (val: unknown[]) => unknown[]) => {
    const val = getSelected(currentValuesArr as unknown[]);
    console.log('val', val); // here val is the array of selected values updated with the latest entry
    setCurrentValuesArr(val) // update the state array
  }}
  ...rest
/>

You can see how the returning array is managed here around these lines in the component file: MultipleSelectList.tsx


Cannot use the RMatrixComponent

It seems I can't do the following operation:
VAR_IN_OUT
    DK: HMatrix; // Direct kinematic
END_VAR
VAR
    DK_R : RMatrix;
END_VAR

---

DK_R.CopyFrom(DK.RMatrixComponent);
// VAR_IN_OUT parameter 'M' of 'CopyFrom' needs variable with write access as input

TcMatrix: 1.4.2
TcTransform: 0.3.34

In the documentation for VAR_IN_OUT CONSTANT it is said that the "transfer of properties is not permitted". I wonder if function results are permitted? If so, I can probably just add an HMatrix.GetRMatrixComponent() helper function?

That would be awesome! The same should be done to the vector component.

I'm working on adding these simple getters, but I realized that in your specific use case you could be using simple assignment (" := ") to get the behavior you want:
DK_R := HMatrix.RMatrixComponent


fix: Improve OpenSearch import errors

Detail
If a customer has opensearchpy installed but not jsonpath_ng, the import of awswrangler will fail. This goes against our current protocol, which is that we throw an appropriate import error when the customer tries to invoke a method which requires an optional dependency.
The jsonpath_ng dependency is also only required for certain cases when loading JSON data, so I'm only appending the annotation to the _get_documents_w_json_path function.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

AWS CodeBuild CI Report

CodeBuild project: GitHubCodeBuild8756EF16-4rfo0GHQ0u9a
Commit ID: f386193de40baa565e5de463a7d34544c816f546
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: GitHubDistributedCodeBuild6-jWcl5DLmvupS
Commit ID: f386193de40baa565e5de463a7d34544c816f546
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


Mentions dont show up when I try to use quill.getText()

adding the mentions do get added to the quill editor
but when I try to display the text using quill.getText(), it doesn't show the mentions

Any chance this behavior can be added, soon? I'd help / do it myself, but I'm a complete quill-rookie. So no idea where to start. I guess it would be cool, if a transform-function could be passed (how to translate these mentions to pure text), by default null. So the current behavior won't change / it is not a breaking change.

I just checked the editors getText() method and it seams like that's something, mention can't control:
  getText(index: number, length: number): string {
    return this.getContents(index, length)
      .filter(op => typeof op.insert === 'string')
      .map(op => op.insert)
      .join('');
  }

But since it's quite easy, I've created my own getText-method, respecting mentions:
quill.getContent().ops
    .filter(op => typeof op.insert === 'string' || op.insert?.mention instanceof DOMStringMap)
    .map(op => typeof op.insert === 'string' ? op.insert : `$$${op.insert.mention.id}`)
    .join('');

Hope this solution is useful for somebody out there.


Center panel? What is it? For what?

I don't get it from readme.

What actually you are looking for?


[BugFix]lambda arguments cannot be seen as normal column ref

Signed-off-by: fzhedu<EMAIL_ADDRESS>What type of PR is this：

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes # https://github.com/StarRocks/starrocks/issues/13524
Problem Summary(Required) ：

lambda arguments cannot be seen as normal column ref, which is just used within lambda functions.
Checklist:

[x] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

[FE PR Coverage Check]
:heart_eyes: pass : 9 / 9 (100.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/sql/optimizer/operator/OperatorType.java
1
1
100.00%
[]

:large_blue_circle:
com/starrocks/sql/optimizer/transformer/SqlToScalarOperatorTranslator.java
1
1
100.00%
[]

:large_blue_circle:
com/starrocks/sql/optimizer/base/ColumnRefFactory.java
5
5
100.00%
[]

:large_blue_circle:
com/starrocks/sql/optimizer/operator/scalar/ColumnRefOperator.java
2
2
100.00%
[]

[FE PR Coverage Check]
:heart_eyes: pass : 9 / 9 (100.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/sql/optimizer/operator/OperatorType.java
1
1
100.00%
[]

:large_blue_circle:
com/starrocks/sql/optimizer/transformer/SqlToScalarOperatorTranslator.java
1
1
100.00%
[]

:large_blue_circle:
com/starrocks/sql/optimizer/base/ColumnRefFactory.java
5
5
100.00%
[]

:large_blue_circle:
com/starrocks/sql/optimizer/operator/scalar/ColumnRefOperator.java
2
2
100.00%
[]

run starrocks_admit_test

@mergify backport branch-2.5


Subtype AbstractArray?

Jotting down some quick, initial thoughts.
Since setindex! is now optional (https://github.com/JuliaLang/julia/pull/46833), it may now be sensible to have CPD <: AbstractArray.
Alternatively, SVD <: Factorization so another option is to define TensorFactorization abstract type and subtype that.
Ref: https://github.com/JuliaLang/julia/blob/35d12890aba9201c6cf0f43998856e3eea2dba3e/stdlib/LinearAlgebra/src/svd.jl#L59

Probably best to leave this as is for now.


Maximum amount of solar PV plants

I think the maximum amount of solar PV plants that can be built is too low. For instance: in the NL 2013 dataset (in the ad file), land_available_for_solar = 18475.714
The land use of a solar PV plant is 0.113 km2, so you should be able to build 163,496 plants. However, when you set the slider all the way tot the right, it only goes to 5,817. Still quite a lot, but shouldn't you be able to utilise the full potential?

@BerendSmits please enlighten me.

Where is this parameter currently set and how was it determined?
What do you suggest we do to fix this?

Your argument sounds convincing enough, but I wonder if there was some reason for using the other maximum number.

Well spotted, @BerendSmits . I can answer @AlexanderWirtz' first question for you. In https://github.com/quintel/etsource/commit/e992463e3c51489594407bf8f853d479385d8d99#diff-fadc5ee5840b4efbacbb15285feac51f, the maximum value of this slider was made dependent on a query that takes, amongst others, the total electricity production into account. Simply changing that query to something that takes the land_available_for_solar into account should fix this. Would you like to give it a try, @BerendSmits ? Let me know if you need help.

Thanks @jorisberkhout! I had no idea where max slider values were set..
I've changed it to using the full land available in the branch: https://github.com/quintel/etsource/tree/solar_max_value
The land available for solar is equal to the areable land for the NL dataset (and for a few other datasets as well I think), so the max amount changed quite a bit, from 5,800 to 138,000 solar plants. Setting the slider with your mouse is a bit rough now, the least I could build at a time was one thousand.
What do you think @jorisberkhout @AlexanderWirtz? Keep it the old way (max is set at matching the total base year electricity production) or change it to using the total available land?

new way, i.e. total available land!

I like the new way better too!
The branch solar_max_value contains a commit (https://github.com/quintel/etsource/commit/2e1ff945ab4a6e703cb9b3e58e82d457e9981f6c) that I guess does not belong there. If you could fix this, @BerendSmits , I can review and merge the resulting pull request.

Closed by https://github.com/quintel/etsource/pull/1269/commits/5d340cbdbc6cdfc89005bf14ad9950538e7fdd71.


YARN-11312: [UI2] Refresh buttons don't work after EmberJS upgrade

Change-Id: Iffd45222b073cc1bc35bbb3153ffe6583aefaa74

Description of PR
How was this patch tested?
For code changes:

[x] Does the title or this PR starts with the corresponding JIRA issue id (e.g. 'HADOOP-17799. Your PR title ...')?
[ ] Object storage: have the integration tests been executed and the endpoint declared according to the connector-specific documentation?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE, LICENSE-binary, NOTICE-binary files?

Thanks @susheel-gupta, merged to trunk. Can you please check if branch-3.3 and/or 3.2 has the same EmberJS version?

@brumi1024  Yes, the emberJS version in branch-3.2 (https://github.com/apache/hadoop/blob/branch-3.2/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-ui/src/main/webapp/bower.json#L4) and branch-3.3 (https://github.com/apache/hadoop/blob/branch-3.3/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-ui/src/main/webapp/bower.json#L4) as trunk.
Do we need to fix this issue in branch-3.2 and branch-3.3 ?.
If yes, Question: I'm unable to fetch this branch-3.2 and branch-3.3 locally, how to ensure these fix is compatible with branch-3.2 and branch-3.3 ?.

@susheel-gupta
How do you mean you can't fetch those branches?
If you do git fetch and git branch -r you should see branch-3.2 and branch-3.3 in the list.
You can also see these branches on the GitHub UI.


rbspy: 0.11.1 -> 0.12.1

Description of changes
Update rbspy.
Changes:
0.12.1

    Add --force-version flag for testing with pre-release or custom Ruby versions (https://github.com/rbspy/rbspy/pull/357)
    Fix CLI bug when using the summary-by-line format (https://github.com/rbspy/rbspy/issues/361)
    Improve library examples (https://github.com/rbspy/rbspy/pull/358) and help text

0.12.0

    Add support for Ruby 2.6.10, 2.7.6, 3.0.4, and 3.1.2
    Use directories crate to find OS-appropriate cache directory

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Test logs:
running 35 tests
test core::initialize::tests::test_initialize_with_nonexistent_process ... ok
test core::initialize::tests::test_initialize_with_disallowed_process ... FAILED
test core::initialize::tests::test_get_trace_when_process_has_exited ... FAILED
test core::initialize::tests::test_current_thread_address ... FAILED
test core::initialize::tests::test_get_trace ... FAILED
test core::initialize::tests::test_get_exec_trace ... FAILED
test core::ruby_version::tests::test_get_ruby_stack_trace_1_9_3 ... ok
test core::ruby_version::tests::test_get_ruby_stack_trace_2_1_6 ... ok
test core::ruby_version::tests::test_get_ruby_stack_trace_2_5_0 ... ok
test core::ruby_version::tests::test_get_ruby_stack_trace_2_4_0 ... ok
test core::ruby_version::tests::test_get_ruby_stack_trace_2_7_3 ... ok
test core::ruby_version::tests::test_get_ruby_stack_trace_3_1_0 ... ok
test core::ruby_version::tests::test_get_ruby_stack_trace_3_0_2 ... ok
test sampler::tests::test_sample_single_process ... FAILED
test sampler::tests::test_sample_single_process_with_time_limit ... FAILED
test sampler::tests::test_sample_subprocesses ... FAILED
test core::ruby_version::tests::test_get_ruby_stack_trace_3_0_4 ... ok
test ui::callgrind::tests::stats_write ... ok
test ui::callgrind::tests::stats_aggregate ... ok
test ui::flamegraph::tests::test_collapsed ... ok
test ui::flamegraph::tests::test_flamegraph_from_collapsed ... ok
test ui::flamegraph::tests::test_stats ... ok
test ui::summary::tests::stats_by_function ... ok
test ui::summary::tests::stats_by_line_number ... ok
test ui::pprof::test::can_collect_traces_and_write_to_pprof_format ... ok
test core::ruby_version::tests::test_get_ruby_stack_trace_3_0_3 ... ok
test core::ruby_version::tests::test_get_ruby_stack_trace_3_0_0 ... ok
test core::ruby_version::tests::test_get_ruby_stack_trace_2_7_6 ... ok
test core::ruby_version::tests::test_get_ruby_stack_trace_2_7_2 ... ok
test core::ruby_version::tests::test_get_ruby_stack_trace_2_7_4 ... ok
test core::ruby_version::tests::test_get_ruby_stack_trace_2_7_5 ... ok
test core::ruby_version::tests::test_get_ruby_stack_trace_3_0_1 ... ok
test core::ruby_version::tests::test_get_ruby_stack_trace_3_1_2 ... ok
test core::ruby_version::tests::test_get_ruby_stack_trace_3_1_1 ... ok
test core::ruby_version::tests::test_get_ruby_stack_trace_2_1_6_2 ... ok

failures:

---- core::initialize::tests::test_initialize_with_disallowed_process stdout ----
thread 'core::initialize::tests::test_initialize_with_disallowed_process' panicked at 'called `Option::unwrap()` on a `None` value', src/core/initialize.rs:529:14

---- core::initialize::tests::test_get_trace_when_process_has_exited stdout ----
thread 'core::initialize::tests::test_get_trace_when_process_has_exited' panicked at 'failed to execute process: Os { code: 2, kind: NotFound, message: "No such file or directory" }', src/core/process.rs:54:18

---- core::initialize::tests::test_current_thread_address stdout ----
thread 'core::initialize::tests::test_current_thread_address' panicked at 'failed to execute process: Os { code: 2, kind: NotFound, message: "No such file or directory" }', src/core/process.rs:54:18

---- core::initialize::tests::test_get_trace stdout ----
thread 'core::initialize::tests::test_get_trace' panicked at 'failed to execute process: Os { code: 2, kind: NotFound, message: "No such file or directory" }', src/core/process.rs:54:18
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

---- core::initialize::tests::test_get_exec_trace stdout ----
thread 'core::initialize::tests::test_get_exec_trace' panicked at 'called `Result::unwrap()` on an `Err` value: Os { code: 2, kind: NotFound, message: "No such file or directory" }', src/core/initialize.rs:591:14

---- sampler::tests::test_sample_single_process stdout ----
thread 'sampler::tests::test_sample_single_process' panicked at 'failed to execute process: Os { code: 2, kind: NotFound, message: "No such file or directory" }', src/core/process.rs:54:18

---- sampler::tests::test_sample_single_process_with_time_limit stdout ----
thread 'sampler::tests::test_sample_single_process_with_time_limit' panicked at 'failed to execute process: Os { code: 2, kind: NotFound, message: "No such file or directory" }', src/core/process.rs:54:18

---- sampler::tests::test_sample_subprocesses stdout ----
thread 'sampler::tests::test_sample_subprocesses' panicked at 'failed to execute process: Os { code: 2, kind: NotFound, message: "No such file or directory" }', src/sampler/mod.rs:383:14

failures:
    core::initialize::tests::test_current_thread_address
    core::initialize::tests::test_get_exec_trace
    core::initialize::tests::test_get_trace
    core::initialize::tests::test_get_trace_when_process_has_exited
    core::initialize::tests::test_initialize_with_disallowed_process
    sampler::tests::test_sample_single_process
    sampler::tests::test_sample_single_process_with_time_limit
    sampler::tests::test_sample_subprocesses

test result: FAILED. 27 passed; 8 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.81s

error: test failed, to rerun pass '--lib'

Some tests were fixed, some disabled (wrong assumptions about what's possible in isolated environment)

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/920

Result of nixpkgs-review pr 172166 run on x86_64-linux 1

1 package built:

rbspy