Triggering the in-game interaction menu leads to mouse glitching

Hyprland Version
 System/Version info
Hyprland, built from branch HEAD at commit<PHONE_NUMBER>014997776e091ff8903acfb311dd2f  (props: bump ver to 0.38.0).
Date: Mon Apr 1 18:30:37 2024
Tag: v0.38.0

flags: (if any)

System Information:
System name: Linux
Node name: nebula
Release: 6.8.2-arch2-1
Version: #1 SMP PREEMPT_DYNAMIC Thu, 28 Mar 2024 17:06:35 +0000

GPU information:
01:00.0 VGA compatible controller [0300]: NVIDIA Corporation AD102 [GeForce RTX 4090] [10de:2684] (rev a1) (prog-if 00 [VGA controller])

os-release: NAME="Arch Linux"
PRETTY_NAME="Arch Linux"
ID=arch
BUILD_ID=rolling
ANSI_COLOR="38;2;23;147;209"
HOME_URL="https://archlinux.org/"
DOCUMENTATION_URL="https://wiki.archlinux.org/"
SUPPORT_URL="https://bbs.archlinux.org/"
BUG_REPORT_URL="https://gitlab.archlinux.org/groups/archlinux/-/issues"
PRIVACY_POLICY_URL="https://terms.archlinux.org/docs/privacy-policy/"
LOGO=archlinux-logo

plugins:

Bug or Regression?
Bug
Description

Holding and pressing the f-key + pressing the left mouse button leads to the mouse glitching downwards & left (character looks down and turn to the left), see video below.
Holding and pressing the f-key + pressing the right mouse button leads to the mouse glitching downwards & right, see video below.
However, when slightly moving the mouse while doing the same 'press + hold-down f + click left mouse button', does not lead to the 'mouse glitching'. This is not shown in the video below.

Tested versions of hyprland:

The following versions worked fine: 0.33, 0.34, and 0.35.
The following versions have the glitchy behaviour: 0.37 and 0.38 (I believe 0.37 introduced hyprcursor to my system).

Star Citizen runs in wine-ge-8-25-x86_64.
In Star Citizen, pressing the f-key changes the mouse interaction mode so that the player can interact with menus and objects in the game (normal behaviour is as if in first person shooter). The interaction mode is an essential system in the game.
Some speculation from my side: It appears as if an incorrect mouse movement direction is used when under these conditions.
However, I'm not certain how to troubleshoot / debug this to obtain better information.
How to reproduce

Ensure hyprland 0.37 or 0.38 is installed
Start Star Citizen, once enter into the persistent universe
Failure issue 1: press + hold f + click left mouse button -> character will look down and turns left
Failure issue 2: press + hold f + click right mouse button will open the interaction menu, the mouse will instantly go to the lower right corner of the screen.

Crash reports, logs, images, videos
No response

My video didn't get added to the original post.
https://github.com/hyprwm/Hyprland/assets/13607845/858d22ed-9479-4ed2-b43d-4bdd4c99218e

Today, I had some time for git-bisect.
The commit which introduced the glitching mouse behaviour is: 7a76ab01d16cdeb3bbb11449476d2b8ad84a3fda

As a test, I commented out the line which leads to the glitching behaviour on hyprland v0.38.1, i.e., https://github.com/hyprwm/Hyprland/blob/89f775aec20fad9d0fec27bd61139d5e3b0f5c6a/src/managers/input/InputManager.cpp#L161
This stops the glitchy mouse behaviour.
Obviously, that is not the good way to resolve the issue, however, it is not fully clear to me what that function is supposed to do or why it is needed. The documentation for that function isn't helpful to me as I'm not familiar with wlroots nor wayland programming.
https://wayland.emersion.fr/wlroots/wlr/types/wlr_seat.h.html#func-wlr_seat_pointer_send_motion

void wlr_seat_pointer_send_motion(struct wlr_seat *wlr_seat, uint32_t time_msec, double sx, double sy);
Send a motion event to the surface with pointer focus. Coordinates for the motion event are surface-local. This function does not respect pointer grabs: you probably want wlr_seat_pointer_notify_motion() instead.

From some debugging and adding some debug logging of the following block of code:
https://github.com/hyprwm/Hyprland/blob/89f775aec20fad9d0fec27bd61139d5e3b0f5c6a/src/managers/input/InputManager.cpp#L155-L161
For my conditions, i.e., in-game, I notice that the BOX coordinates are always (0, 0) which leads to CLOSEST & CLOSESTLOCAL being identical.
https://github.com/hyprwm/Hyprland/blob/89f775aec20fad9d0fec27bd61139d5e3b0f5c6a/src/managers/input/InputManager.cpp#L161
That line seems to add the value of the pointer location to the current pointer location, effectively doubling the pointer coordinates.
That also seems to be in line with what I notice at the end of my video above, i.e., the cursor teleports to the bottom right corner.

That line seems to add the value of the pointer location to the current pointer location, effectively doubling the pointer coordinates.

wlr_seat_pointer_send_motion doesn't add, it sets, wdym double?

wlr_seat_pointer_send_motion doesn't add, it sets, wdym double?

I mean that the coordinates of my pointer seem to be somehow doubled when trying to use the in-game interaction menu.
The steps I do during debugging which trigger the issue:

My viewpoint in first person is at the center of my screen
I press and hold 'F' without moving the mouse -> I'm thus still looking at the center of my screen.
While still holding 'F' + clicking a mouse button (L or R, both show the issue) makes my cursor instantly move downwards. In particular, when clicking the right mouse button, the cursor shows up at the right bottom, for the left mouse button, It's not clear what happens as it looks like I rotate left while looking down.

I added some logging to both Hyprland around https://github.com/hyprwm/Hyprland/blob/89f775aec20fad9d0fec27bd61139d5e3b0f5c6a/src/managers/input/InputManager.cpp#L161 and to wlroots around https://github.com/swaywm/wlroots/blob/0855cdacb2eeeff35849e2e9c4db0aa996d78d10/types/seat/wlr_seat_pointer.c#L233-L248
Screenshot of the 'debug' lines I added, Hyprland on the left and wlroots on the right:

From the logs, it appears that my pointer coordinates somehow indeed end up at the lower right corner of my screen.
I believe this is the moment in the logs when I'm pressing 'F' and it seems to create a new mouse constraint and setup a surface?
When entering the lines of code where I put my debugging -> the cursor location is at (5119, 1439) which is the right bottom of my screen, yet, I did not mouse my mouse there at all.

[wlr] [types/wlr_pointer_constraints_v1.c:278] new confined_pointer 0x5c0c2ec61b70 (res 0x5c0c2eae4350)
[LOG] New mouse constraint at 5c0c2ec61b70
[LOG] Registered signal for owner 5c0c2ebc7810: 5c0c2ec61c30 -> 5c0c2ebc78d8 (owner: CConstraint)
[LOG] Registered signal for owner 5c0c2ebc7810: 5c0c2ec61c40 -> 5c0c2ebc7870 (owner: CConstraint)
[wlr] [types/wlr_pointer_constraints_v1.c:378] constrained 0x5c0c2ec61b70
[LOG] cursorImage request: surface 5c0c2eb84ea0
[LOG] Registered signal for owner 5c0c2eaad550: 5c0c2eb851b8 -> 5c0c2eaad598 (owner: CWLSurface)
[LOG] Registered signal for owner 5c0c2eaad550: 5c0c2eb85178 -> 5c0c2eaad600 (owner: CWLSurface)
[LOG] CWLSurface 5c0c2eaad550 called init()
[wlr] [types/output/cursor.c:371] Falling back to software cursor on output 'WL-1'
[wlr] [types/output/cursor.c:371] Falling back to software cursor on output 'WL-1'
[wlr] [/home/neburski/code/Hyprland/src/managers/input/InputManager.cpp:160] HL 1 -> CLOSEST: (5119.000000, 1439.000000); CLOSESTLOCAL: (5119.000000, 1439.000000); BOX: (0.000000, 0.000000); SCALED: 1.000000
[wlr] [/home/neburski/code/Hyprland/src/managers/input/InputManager.cpp:174] HL 2 -> CLOSEST: (5119.000000, 1439.000000); CLOSESTLOCAL: (5119.000000, 1439.000000); BOX: (0.000000, 0.000000); SCALED: 1.000000
[wlr] [types/seat/wlr_seat_pointer.c:248] WLR -> (sx, sy) = (5119.000000, 1439.000000); (sx_fixed, sy_fixed) = (1310464, 368384);
[wlr] [types/seat/wlr_seat_pointer.c:255] WLR NEQ -> (ptr_state.sx, ptr_state.sy) = (688896, 135680); (sx_fixed, sy_fixed) = (1310464, 368384);
[wlr] [/home/neburski/code/Hyprland/src/managers/input/InputManager.cpp:160] HL 1 -> CLOSEST: (5119.000000, 1439.000000); CLOSESTLOCAL: (5119.000000, 1439.000000); BOX: (0.000000, 0.000000); SCALED: 1.000000
[wlr] [/home/neburski/code/Hyprland/src/managers/input/InputManager.cpp:174] HL 2 -> CLOSEST: (5119.000000, 1439.000000); CLOSESTLOCAL: (5119.000000, 1439.000000); BOX: (0.000000, 0.000000); SCALED: 1.000000
[wlr] [types/seat/wlr_seat_pointer.c:248] WLR -> (sx, sy) = (5119.000000, 1439.000000); (sx_fixed, sy_fixed) = (1310464, 368384);

Apologies, I'm not familiar with Hyprland, wlroots, not wayland terminology and code, so I'm mostly trying  to figure things out in the hopes to find the root cause.

Having the same experience here on my side 🤔

can you print the region extents

I'm a super-noob when it comes to these things 😅 and I don't know how to go about printing the region extents, sorry.
However if you or someone can give me information, a link, anything; on how to do that, I'll gladly try to get the region extents, I just need to know/figure out how.

CRegion has an extents method which returns a CBox of which you can print x,y,w,h

I'm not familiar with C++. At the moment that does not mean much to me, I don't know how to even begin to add that code - sorry. But I'm willing to help out if I have more knowledge about this.
I use NixOS, so I was able to confirm that if I comment out this line of code, the issue in my game disappears, and I can play (Star Citizen) normally without the cursor jumping to the bottom-right corner when I open the "focus/thought" using "F" and click.

I created a patch using git diff > wlr_seat_pointer_send_motion_comment.patch.
I applied the patch:
    overlays = [
       (final: prev: {
        hyprland = prev.hyprland.overrideAttrs (o: {
          patches = (o.patches or [ ]) ++ [
            ./wlr_seat_pointer_send_motion_comment.patch
          ];
        });
      })
    ];

And now I can play without issues. I also don't notice any negative effects of removing this line of code - everything appears to continue to work as expected without it.
My issues is resolved because of my patch, but I would like to be able to give more information to be able to assist in resolving these issues.

My c++ knowledge is very rusty (rather lost to time), so I got the extents just before printing as follows:
auto ss = CWLSurface::surfaceFromWlr(g_pCompositor->m_pLastFocus);
auto cc = ss->constraint();
auto ll = cc->logicConstraintRegion();
auto bb = ll.getExtents();

I created some logs where I also print the bb.x, bb.y, bb.w, bb.h as requested.
The logs have been made using `tail -f /tmp/hypr/..../hyprlandd.log' to be able to see the log populate in real time. Besides the full log, each one of the other logs was created by restarting the 'tail -f ...' command just before initiation an action to create logs for each action separately.
The following logs are attached:

2024-04-13_hyprlandd.log is the full log
2024-04-13_press_f.log only pressing  the F key, no other action was done beside the pressing F.
2024-04-13_press_f_and_LMB.log pressing  the F key followed by pressing the left mouse button.
2024-04-13_press_f_and_RMB.log pressing  the F key followed by pressing the right mouse button.

2024-04-13_press_f.log
2024-04-13_press_f_and_LMB.log
2024-04-13_press_f_and_RMB.log
2024-04-13_hyprlandd.log

This issue has disappeared since Hyprland 0.39.1.
Now there is another issue which seems to be similar as mentioned here: #5774, #5881, and #5895. It seems that in the latest Hyprland (0.40+) these issues have been fixed.
Closing this issue.

Hi there, it appears this issue is actually still persisting. Any known resolution to fix?

I'm also experiencing this issue on 718afe2, so it has not disappeared.

Same issue here. It probably reappeared for some or was never gone

same problem for me. Has anybody found a solution ?

@Neburski hey im sorry for the ping. I dont know if your fix still works but could you maybe explain how you did it ? im a noob and i dont know where i need to comment out that stuff. thanks

So I don't know how the original author fixed his problem. But I just switched to Gnome. It works perfectly over there.
It isn't even close to being similar thou, so if you want something similar maybe search for a different desktop.

Yes the issue still lives on. I have now switched to Arch recently. The offending line now lives here:
https://github.com/hyprwm/Hyprland/blob/c86db7bbb0cf14d4955ee3a4d13c0ed9f8a0e0ae/src/managers/input/InputManager.cpp#L220
Commenting this out locally and rebuilding/compiling again fixes it for me in Start Citizen.


No Unicorn Icon?

So, everything seems to work fine, but I can't get a unicorn icon to fly to my location for pickup...

I got the same issue when I had tried this a couple of months back. There was no unicorn icon flying to the pickup point.
Is there anyone who has faced this issue and was able to identify what went wrong?

Just went through the complete workshop. Everything seems fine except for the unicorn showing up. Cleared cache, separate browsers, the normal stuff.. No unicorns!

Went through the API gateway. Seems I forgot to check the proxy integration on the Post method.
(i. Check the box for Use Lambda Proxy integration.)
Deleted the API and recreated (setting the proxy checkbox did not fix it). It now works.
If you ran into the same issue then you have this in the CloudWatch log: TypeError: Cannot read property 'authorizer' of undefined at exports.handler (/var/task/index.js:26:30).

You do not need to delete the API and recreate it. You have access to those options in the Integration Request box:
Go to resources, cliikc on the POST method, it will take you to the Method Execution screen. Then click on the Integration Request box and check the Use Lambda Proxy Integration.

I had to make the following changes for assets to load properly:
<!-- ride.html -->
- <link rel="stylesheet" href="/css/ride.css">
- <link rel="stylesheet" href="/css/message.css">
+ <link rel="stylesheet" href="css/ride.css">
+ <link rel="stylesheet" href="css/message.css">

and
// esri-map.js
- url: '/images/unicorn-icon.png',
+ url: 'images/unicorn-icon.png',

It looks like this can be caused either by a misconfiguration of your API as well as if you are using the path-style URLs for S3 (i.e. http://s3.amazonaws.com/bucket) instead of the website endpoint. The latter was an issue because of a problem with absolute URL references assuming ride.html was being served from the root of the domain. I've updated the website content to avoid absolute path references so this should not be an issue anymore.


#feature request => adding different layouts on the home page  for different sections such as ( continue watching , recommended courses,  Upcoming courses, Field wise course suggestion)

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Additional context
Add any other context about the feature request here.

We can enter the videos for each section in the horizontal view layout.

I would like to add this functionality on the home page


Add more info from the latest Home Connect update.

The last app update introduced new information regarding devices such as number of cycles, average power and water usage.
U do not actually know if this info is available from the API but this can be a nice addition.
Attached an image for a better clarification.

The integration is automatically creating entities that are exposed by the API, so if it was available in the API it would have been automatically added by the integration. This feature request should first be directed to Home Connect, so I'm going to close it for now.


Invalid file type

While using the addon on 2.78, I tried importing a model made by a specific person and it said the file type was invalid. I have tried importing models made by the same person earlier and they imported just fine. I have no idea what the problem is.

Sorry but this project is currently not being maintained here. Fortunately there is a very good fork in active development. Check this #53 for more info

@powroupi is the current developer of mmd_tools.
You can find lots of great bug fixes and additional features in the version of mmd_tools by @powroupi:
https://github.com/powroupi/blender_mmd_tools/archive/dev_test.zip
https://github.com/powroupi/blender_mmd_tools
https://github.com/powroupi/blender_mmd_tools/issues


Compatibility issue with n8n 0.210.0 and above

Hi, it looks like the last compatible n8n version is 0.209.4 - later versions have dependency compatibility issues. The installation of the packages results in:
npm ERR! code ERESOLVE
npm ERR! ERESOLVE could not resolve
npm ERR!
npm ERR! While resolving<EMAIL_ADDRESS>
npm ERR! Found<EMAIL_ADDRESS>
npm ERR! node_modules/ioredis
npm ERR<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR!   node_modules/bull
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peerOptional<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR! node_modules/typeorm
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Conflicting peer dependency<EMAIL_ADDRESS>npm ERR! node_modules/ioredis
npm ERR!   peerOptional<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR!   node_modules/typeorm
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force, or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.

This is the Dockerfile we're using to build a puppeteer enabled image for n8n. Build it with docker build -t n8n-local .:
FROM n8nio/n8n:0.210.2

# Installs latest Chromium package
RUN apk add --no-cache \
  chromium \
  nss \
  freetype \
  harfbuzz \
  ttf-freefont \
  yarn

# Tell Puppeteer to skip installing Chrome. We'll be using the installed package.
ENV PUPPETEER_SKIP_CHROMIUM_DOWNLOAD=true \
  PUPPETEER_EXECUTABLE_PATH=/usr/bin/chromium-browser

# Install n8n-nodes-puppeteer
RUN cd /usr/local/lib/node_modules/n8n \
 && npm install n8n-nodes-puppeteer

# Install fonts
RUN apk --no-cache add --virtual fonts msttcorefonts-installer fontconfig \
 && update-ms-fonts \
 && fc-cache -f \
 && apk del fonts \
 && find  /usr/share/fonts/truetype/msttcorefonts/ -type l -exec unlink {} \; \
 && rm -rf /root /tmp/* /var/cache/apk/* && mkdir /root

Same issue here.

I made it works on a debian python based dockerfile (https://github.com/naskio/n8n-nodes-python).
I was able to use the node but I got this error :
Failed to launch the browser process! [No usable sandbox! Update your kernel or see https://chromium.googlesource.com/chromium/src/+/main/docs/linux/suid_sandbox_development.md for more information on developing with the SUID sandbox. If you want to live dangerously and need an immediate workaround, you can try using --no-sandbox.
There's 3 ways of fixing this :

sysctl -w kernel.unprivileged_userns_clone=1    ==> Result to openning vulnerabilities on your host
--no-sandbox  ==> same like it's said in the error message
Follow this guide : https://chromium.googlesource.com/chromium/src/+/main/docs/linux/suid_sandbox_development.md

Are you still having issue to build the docker image ? I can share mine if needed

I think its not working on 1.1+

@Loocos Mind sharing your docker file?

I'm using the plugin with the latest versions of n8n without issue.


Changes to the Iron documentation

I changed the command for checking the files after the building process in doc of [Creating a workspace].
For 5 Build the workspace with colcon
Original:
Used the ls command to list files.
Modified:
Now using ls for Linux and dir for Windows to list files.

Hi, @clalancette, I submit a pull request.

Hi, @fujitatomoya, thank you for your feedback. I just used iron to test this change, so this targets to iron. Since I am new to ROS, I not sure whether this will also apply for rolling or humble.
Let me know if there is anything else I can help.

That's fine, we can merge it into iron first and then port it over to rolling and humble using the bot.

@Mergifyio backport rolling humble


Changed publish-nuget action and bumped to 0.0.2

Closes #24

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
453

Relevant Lines:
626

💛  - Coveralls


Note: building //web/src:styles on macOS needs sandboxed execution

It seems that the SassCompiler is not hermetic.
If you build //web/src:styles without sandboxed execution, it will fail with:
ERROR: yngdieng/web/src/BUILD.bazel:36:1: Compiling Sass failed (Exit 65)
Error: Can't find stylesheet to import.
  ╷
1 │ @import '@angular/material/theming';
  │         ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  ╵
  web/src/custom-theme.scss 1:9  root stylesheet

Error: Can't find stylesheet to import.
  ╷
1 │ @import '@angular/material/theming';
  │         ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  ╵
  web/src/custom-theme.scss 1:9  @import
  web/src/styles.scss 3:9        root stylesheet
Target //web/src:styles failed to build

However, neither custom-theme.scss nor styles.scss is in the input for //web/src:styles.
For now I disabled --spawn_strategy=local in .bazelrc. This might lead to reduced build performance on macOS.  (However not very significant as far as I can tell)

ref https://github.com/bazelbuild/bazel/issues/8230


Removes deadcode found in Doppler

[#125997765]
Since this was done in flex-hour, submitting for review.

Related Chore here

Thanks!


How to pass additional query string in Webchat channel

Hi Team,
We are using Bot framework sdk v4 enabled webchat channel. We are able to get username & userid in the Bot.
'https://webchat.botframework.com/embed/TestBPAFinnBot?s=YOUR_SECRET_HERE&userid=USER_EMAIL_ID&username=USER_NAME'
Now we want to pass additional query string in the endpoint (for example inputdata=Contract) and use it in Bot for our logic.
But we are not able to get/ view this value in turnContext (IConversationUpdateActivity).
'https://webchat.botframework.com/embed/TestBPAFinnBot?s=YOUR_SECRET_HERE&userid=USER_EMAIL_ID&username=USER_NAME&inputdata=TEXT'
Kindly help.

@Vijayalakshmi-R1 - Unfortunately, it isn't possible to send other properties via the query string as that list is fairly limited and specific? There is a way to send it via the conversationUpdate activity but it would require changing how Web Chat is implemented as it's not able to be done from the iframe embed. Is this a possibility?

Closing due to lack of response.


ARROW-16749: [Go] Fix pqarrow writer for null array

Hello world,
When converting from Arrow to Parquet it looks like there is a bug with arrays of type arrow.NULL. Here is a snippet of code to reproduce the bug:
package main

import (
	"fmt"
	"log"
	"os"

	"github.com/apache/arrow/go/v9/arrow"
	"github.com/apache/arrow/go/v9/arrow/array"
	"github.com/apache/arrow/go/v9/arrow/memory"
	"github.com/apache/arrow/go/v9/parquet/pqarrow"
)

const n = 10

func run() error {
	schema := arrow.NewSchema(
		[]arrow.Field{
			{Name: "f1", Type: arrow.Null, Nullable: true},
		},
		nil,
	)

	rb := array.NewRecordBuilder(memory.DefaultAllocator, schema)
	defer rb.Release()

	for i := 0; i < n; i++ {
		rb.Field(0).(*array.NullBuilder).AppendNull()
	}

	rec := rb.NewRecord()
	defer rec.Release()

	for i, col := range rec.Columns() {
		fmt.Printf("column[%d] %q: %v\n", i, rec.ColumnName(i), col)
	}

	f, err := os.Create("output.parquet")
	if err != nil {
		return err
	}
	defer f.Close()

	w, err := pqarrow.NewFileWriter(rec.Schema(), f, nil, pqarrow.DefaultWriterProps())
	if err != nil {
		return err
	}
	defer w.Close()

	err = w.Write(rec)
	if err != nil {
		return err
	}

	return nil
}

func main() {
	if err := run(); err != nil {
		log.Fatal(err)
	}
}

I'm not 100% sure if my fix is the best, so feel free to correct me.
Thanks
A

Yes, it doesn't look like there is an equivalent Parquet type. Nonetheless, it seems that decision to convert Arrow's Null to Parquet's Int32 has already been made in the code base. See here.
The use case is quite simple: in real world databases there are columns which are full of NULL values and I guess that's why Arrow has Null. Some might say that's a bad practice but it exists...
I don't see any reason why we should not allow converting record batches with Null arrays to Parquet.
What do you think?
In the mean time I will look into others implementations to see what's being done here.

@alexandreyc Huh, I didn't even remember that I did that. Looks like parquet does have a NULL logical type so it makes sense.
That said, instead of allocating an entire int32 slice for this. I believe the better solution would be to manually call wr.WriteBatchSpaced(nil, defLevels, repLevels, learArr.NullBitmapBytes(), 0) and then break. No need to allocate an entire int32 slice for this case.

@zeroshade thanks for your answer.

I added a test to exercise the bug
I had to fix an issue with column writers (feel free to double check since I'm not very familiar with the parquet's code base)
I regenerate parquet's code (it looks like there has been modifications to metadata statistics that hasn't been taken into account)

@alexandreyc Looks good to me now! Thanks much for this!


[master] apiaddresses in etcd are being overwritten by pod ips scheduled to server nodes

Cluster Configuration:
any server
Describe the bug:

Wrong endpoints are being saved as apiaddresses if they are deployed in the default namespace
Steps To Reproduce:

Installed single k3s server node and run the service
Deploy yaml:

apiVersion: v1
kind: Service
metadata:
  name: busyb
spec:
  selector:
    app: busy
  clusterIP: None
  ports:
  - name: foo # Actually, no port is needed.
    port: 1234
    targetPort: 1234
---
apiVersion: apps/v1
kind: Deployment
metadata:
  name: busydep
spec:
  replicas: 3
  selector:
    matchLabels:
      app: busy
  template:
    metadata:
      labels:
        app: busy
    spec:
      containers:
        - name: busybox
          image: busybox:1.28
          command:
            - sleep
            - "3600"
          securityContext:
            runAsUser: 1000
            runAsNonRoot: true

check k3s/apiaddresses key in etcd

Expected behavior:

You should see api addresses
Actual behavior:

you see endpoints for the for the service you deployed

Validated on master branch commitid 8b857eef9c15ed97180079c0dcf5cf0f8cb888de
Followed the same steps mentioned in https://github.com/rancher/rke2/issues/1706 but replaced rke2 with k3s


Capital error in iOS SDK

I found a problem that could not be compiled due to spelling errors.

@lforme  Please rebase to master first, too many commits.


draggable false not working

It is supposed to disable draggable function when the parameter is false?

If yes, it is not working.
If it is not, can be made as an enhancement with default 'true'?
true:

 or

false:

Hi @tobiasrighi ,
I have implemented you suggestion in it. Thanks you so much to for this.
Please have a  look and let me know what you think about.

Hi @CoderAjay ,
I am trying to use this feature as mentioned by @tobiasrighi and i am not able to make it work.
in html file i have the following

where the variable "isDraggable" will be toggled between true and false in the ts file.
please let me know if this has been fixed.
Thank you

Hi @cacev000,
The issue has been fixed and you can use this as

Thanks.


img file loader path err

Laravel Mix Version: lastest
Node Version (node -v): 8.9.3
NPM Version (npm -v): 5.5.1
OS: Mac OS

Description:
npm run hot
assets/image/2.png
assets/image/test/2.png

两张图片名字一样,路径不同,但是生成路径时没有加入 path info, 导致图片会被覆盖
Steps To Reproduce:

I have handled this for the time being
I tried to listen configReady, delete default config,

What was the error message?
Would you like to share your webpack.mix.js

@ankurk91  no err, Two graphs should be generated. One is images/2.png and the other is images/test/2.png. However, the path information is not added, resulting in only one images/2.png
my webpack.mix.js is  a layer of my package this


Release v1.105.0 - Update validation changelog

Description 📝
Update Validation package's changelog to include changes for UpdateCertificateSchema.

This probably should have an entry in /manager for Upcoming Features ("Add Update Certificate Drawer for AGLB (#9723)") but that isn't as big of a deal as missing the changelog entry for validation entirely. Thanks again!


AVRO-3356 simplify naming in DataFileReader

Make sure you have checked all steps below.
Jira

[ ] My PR addresses the following Avro Jira issues and references them in the PR title. For example, "AVRO-1234: My Avro PR"

https://issues.apache.org/jira/browse/AVRO-3356

Tests

[ ] My PR does not need testing for this extremely good reason: non functional changes

Commits

[ ] My commits all reference Jira issues in their subject lines. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters (not including Jira issue reference)
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Documentation

[ ] In case of new functionality, my PR adds documentation that describes how to use it.

All the public functions and the classes in the PR contain Javadoc that explain what it does

Thank you, @KyleSchoonover !


Lykhoyda/add snap

Please see https://github.com/Lykhoyda/WebZjs/pull/1/files

Please see https://github.com/Lykhoyda/WebZjs/pull/1/files

I think it's also a good strategy so far to make it more minimalistic as we don't require any customization since we split the wasm packages. I merged your PR and added some adjustments. Please take a look 👍


🍁Happy Hactoberfest! Read on to contribute....🍁

🍁🍂🎃Welcome to Hacktoberfest 2022! We welcome contributions to our Tech Dictionary. Here's how to get started:
🐱💻WHO can contribute?

People at any level of experience (yep, no experience necessary!)
People who agree to behave according to our Code of Conduct

🐱⚽HOW do you contribute?

Read through our Contributing HowTo.
Find a dictionary entry on our website or submit an entry to be added! You do not need to be assigned an issue - you may make a PR for any dictionary entry you choose. Kindly note that we will not assign any dictionary entries. Read through the dictionary and submit a PR for any entry that interests you.
If you don't want to make any commits, you can always submit an issue for it. You don't get Hacktoberfest credit, but we still appreciate the effort!

There are no time limits to contributions. You may contribute during Hacktoberfest, but you are also welcome to contribute any time of the year.
🙀💥WHAT to do if you run into trouble?
If you run into any trouble (for example, getting set up or picking a dictionary entry), feel free to submit a general issue, and we'll try to help as best we can.

Hello Sir,
I have gone through the repository and the given instructions. I am willing to solve this issue by adding definition of technical terms .I humbly wait for you to assign the task to me.

Hello Sir,
I have gone through the repository and the given instructions. I am willing to solve this issue by adding definition of technical terms .I humbly wait for you to assign the task to me on 1st Oct.

Hi, welcome! You're welcome to edit any dictionary entry on our website that you would like to add a definition.

Will I Commit On 1st Oct

I would like to contribute to this repository. Request you to assign the task of adding definition and perspective to me.

Hi, I've gone through the contribution guide and I'll like to contribute to the project by adding definitions and perspective.

Hey, I'm incredibly interested in contributing to this project by adding definitions and perspectives to a number of dictionary entries.

Will I Commit On 1st Oct

You're welcome to commit whenever you like.

I would like to contribute to this repository. Request you to assign the task of adding definition and perspective to me.

@Dhanshriborse You're welcome to find any dictionary entry you like and submit a PR for consideration - no issues will be assigned.

I would like to contribute to this repository. can you assign me on this? or should I make PR without mentioning issue?

@RohitRajvaidya5 You're welcome to make a PR without mentioning issue - choose any dictionary entry you like, or add one of your own.

Hi, I've gone through the contribution guide and I'll like to contribute to the project by adding definitions and perspective.

@ofonime41 You're welcome to find any dictionary entry, or contribute one of your own - no need to wait for an issue.

Done !!!

Hey should we assign by some one or directly make PR in this repo?

Hey should we assign by some one or directly make PR in this repo?

Hi @nik132-eng You should take whatever dictionary entry you like and directly make a PR.

I have Done PR can you review it.

@RohitRajvaidya5 Yep we'll be on it! We've had a lot of interest, so we may need a week or two, but we will definitely get to it! :)


Review hierarchy: Autosomal dominant tubulointerstitial kidney disease

MONDO_0008264 Autosomal dominant tubulointerstitial kidney disease (not equiv to an OMIM)
Children:
-MONDO_0019511 'autosomal dominant medullary cystic kidney disease with hyperuricemia' Orphanet:88950
-MONDO_0012356 'glomerulocystic kidney disease with hyperuricemia and isosthenuria'
-MONDO_0020726 'medullary cystic kidney disease 1' OMIM:603860
-MONDO_0011379 'medullary cystic kidney disease 2' OMIM:603860

[ ] I think MONDO_0019511 'autosomal dominant medullary cystic kidney disease with hyperuricemia'  (equiv to Orphanet:88950) and MONDO_0011379 'medullary cystic kidney disease 2'  (equiv to OMIM:603860) should be merged. Both have UMOD mutation.

MONDO_0000608 'familial juvenile hyperuricemic nephropathy' (OMIMPS:162000)
Children:
-MONDO_0008073 'familial juvenile hyperuricemic nephropathy type 1', OMIM:162000, **Orphanet:209886

[ ]  Orphanet:209886  was obsoleted and replaced by Orphanet_88950 UMOD-related autosomal dominant tubulointerstitial kidney disease (which is equiv to MONDO_0019511 'autosomal dominant medullary cystic kidney disease with hyperuricemia').

If I were to do the merger above, and then if I were to merge MONDO_0008073 'familial juvenile hyperuricemic nephropathy type 1' with MONDO_0011379 'medullary cystic kidney disease 2', then OMIM:603860 and OMIM:162000 would be equivalent. Not sure if @ahamosh would agree with that? :)**
This comment above is related to https://github.com/monarch-initiative/mondo/issues/1880
-MONDO_0013128 'familial juvenile hyperuricemic nephropathy type 2' OMIM:613092

[ ] is this equiv to Autosomal dominant tubulointerstitial kidney disease due to REN mutations (GARD:0013461)? GARD classifies this under MONDO_0008264 'autosomal dominant medullary cystic kidney disease with or without hyperuricemia'

-MONDO_0013643 'hyperuricemic nephropathy, familial juvenile type 3'
-MONDO_0014891 'hyperuricemic nephropathy, familial juvenile type 4'
related to #1878

@ahamosh @cmungall @tsneddon @galyea-gard @ana-rath-orphanet @annieolry @Orphanet
I hope this is not too confusing the follow. Seems like these terms are represented in different hierarchies in your respective terminologies.

@ahamosh @tsneddon @galyea-gard @ana-rath-orphanet @annieolry @Orphanet
pinging you all again :)

If I were to do the merger above, and then if I were to merge MONDO_0008073 'familial juvenile hyperuricemic nephropathy type 1' with MONDO_0011379 'medullary cystic kidney disease 2', then OMIM:603860 and OMIM:162000 would be equivalent. Not sure if @ahamosh would agree with that? :)**
This comment above is related to #1880

these have been since merged in OMIM


Error converting simple pptx file to pdf

Hi there,
When I attempt to convert a simple pptx file to pdf using the POST forms/libreoffice/convert endpoint in gotenberg I get a 500 error.
The pptx file that I am attempting to convert to pdf using the libre office engine is very simple. It has one slide with two images and a text label.
I think the issue is related to the images used in the pptx but I'm not sure why they would cause the pdf conversion to fail.  When I remove one of the images from the pptx the pdf conversion runs successfully.
See the below screen grab of the erroneous slide in the pptx. I have also attached the pptx that is causing the conversion failure

The following is a snippet of debug logs central to the issue. Full logs are here
{"level":"debug","ts":1654172407.363404,"logger":"api.formslibreofficeconvert.usrliblibreofficeprogramsoffice.bin","msg":"unix process already killed","trace":"656a0796-af74-4b8c-9acd-456409a7fff5"}
{"level":"error","ts":1654172407.3656774,"logger":"api","msg":"convert to PDF: unoconv PDF: unix process error: wait for unix process: exit status 6","trace":"6922e459-19b5-41e2-aff1-ab95297d3c4d","remote_ip":"<IP_ADDRESS>","host":"gotenberg-service:3000","uri":"/forms/libreoffice/convert","method":"POST","path":"/forms/libreoffice/convert","referer":"","user_agent":"Apache-HttpClient/4.5.13 (Java/11.0.10)","status":500,"latency":14209214897,"latency_human":"14.209214897s","bytes_in":775610,"bytes_out":21}
{"level":"debug","ts":1654172407.3685727,"logger":"api.formslibreofficeconvert.usrliblibreofficeprogramsoffice.bin","msg":"start unix process: /usr/lib/libreoffice/program/soffice.bin --headless --invisible --nocrashreport --nodefault --nologo --nofirststartwizard --norestore -env:UserInstallation=file:///tmp/bd7353dc-0119-4d19-aa82-308b96069ef8 --accept=socket,host=<IP_ADDRESS>,port=41679,tcpNoDelay=1;urp;StarOffice.ComponentContext","trace":"6922e459-19b5-41e2-aff1-ab95297d3c4d"}
{"level":"debug","ts":1654172409.1584835,"logger":"api.formslibreofficeconvert.usrliblibreofficeprogramsoffice.bin","msg":"unix process already killed","trace":"6922e459-19b5-41e2-aff1-ab95297d3c4d"}
{"level":"debug","ts":1654172409.1585321,"logger":"api.formslibreofficeconvert","msg":"got exit code 81, e.g., LibreOffice listener first start","trace":"6922e459-19b5-41e2-aff1-ab95297d3c4d"}

Docker file used to build Gotenberg image
FROM gotenberg/gotenberg:7.5.2
USER root
COPY /resources/fonts/* /usr/local/share/fonts/
USER gotenberg
CMD ["gotenberg", "--pdfengines-engines=uno-pdfengine", "--api-disable-health-check-logging=true", "--log-level=debug", "--api-timeout=60s"]

Any help with this would be much appreciated!
Thanks,
Keith

Hello @keithcaff,
LibreOffice may struggle with some files, usually for unknown reasons alas.
That being said, Gotenberg 7.5.2 is known to host an unstable version of LibreOffice. You may have more success with the version 7.4.3 😄

Hi, @gulien
Thanks for your reply. Unfortunately with version 7.4.3 I'm still getting a 500 error.
The following are the debug logs I am now seeing
-------------------------------------------------------
[SYSTEM] modules: api chromium gc libreoffice logging pdfcpu pdfengines pdftk prometheus qpdf unoconv unoconv-pdfengine webhook
[SYSTEM] prometheus: collecting metrics
[SYSTEM] api: server listening on port 3000
[SYSTEM] gc: application started
{"level":"debug","ts":1654248843.2161143,"logger":"unoconv.usrbinunoconv","msg":"start unix process: /usr/bin/unoconv --listener --user-profile ///tmp/766bd52b-ae57-4277-a73c-7e5b8f12b6e3 --port 34909 -vvv"}
[SYSTEM] pdfengines: unoconv-pdfengine
[SYSTEM] unoconv: listener started on port 34909
{"level":"debug","ts":1654248843.4895248,"logger":"unoconv.usrbinunoconv.stderr","msg":"Verbosity set to level 3"}
{"level":"debug","ts":1654248920.6886685,"logger":"api.formslibreofficeconvert","msg":"form data values: map[]","trace":"7584b0fa-fbbe-4df6-8b1c-7341d5a01efa"}
{"level":"debug","ts":1654248920.6894274,"logger":"api.formslibreofficeconvert","msg":"form data files: map[8a4c45b2-afa7-41e2-a053-351ade53d5cd.pptx:/tmp/d93bb79d-71db-4c56-99bd-051ce5805283/8a4c45b2-afa7-41e2-a053-351ade53d5cd.pptx]","trace":"7584b0fa-fbbe-4df6-8b1c-7341d5a01efa"}
{"level":"debug","ts":1654248920.690464,"logger":"api.formslibreofficeconvert","msg":"unoconv lock acquired","trace":"7584b0fa-fbbe-4df6-8b1c-7341d5a01efa"}
{"level":"debug","ts":1654248920.691568,"logger":"api.formslibreofficeconvert","msg":"print to PDF with: {Landscape:false PageRanges: PDFArchive:false}","trace":"7584b0fa-fbbe-4df6-8b1c-7341d5a01efa"}
{"level":"debug","ts":1654248920.6917465,"logger":"api.formslibreofficeconvert.usrbinunoconv","msg":"start unix process: /usr/bin/unoconv --format pdf --port 34909 -vvv --output /tmp/d93bb79d-71db-4c56-99bd-051ce5805283/5bef1829-bf89-405a-9bcf-df5b6e079445.pdf /tmp/d93bb79d-71db-4c56-99bd-051ce5805283/8a4c45b2-afa7-41e2-a053-351ade53d5cd.pptx","trace":"7584b0fa-fbbe-4df6-8b1c-7341d5a01efa"}
{"level":"debug","ts":1654248920.8108542,"logger":"api.formslibreofficeconvert.usrbinunoconv.stderr","msg":"Verbosity set to level 3","trace":"7584b0fa-fbbe-4df6-8b1c-7341d5a01efa"}
{"level":"debug","ts":1654248920.8127246,"logger":"api.formslibreofficeconvert.usrbinunoconv.stderr","msg":"DEBUG: Connection type: socket,host=<IP_ADDRESS>,port=34909,tcpNoDelay=1;urp;StarOffice.ComponentContext","trace":"7584b0fa-fbbe-4df6-8b1c-7341d5a01efa"}
{"level":"debug","ts":1654248920.9760728,"logger":"api.formslibreofficeconvert.usrbinunoconv.stderr","msg":"Input file: /tmp/d93bb79d-71db-4c56-99bd-051ce5805283/8a4c45b2-afa7-41e2-a053-351ade53d5cd.pptx","trace":"7584b0fa-fbbe-4df6-8b1c-7341d5a01efa"}
{"level":"debug","ts":1654248935.7764158,"logger":"unoconv.usrbinunoconv.stdout","msg":"Using office base path: /usr/lib/libreoffice"}
{"level":"debug","ts":1654248935.8413193,"logger":"unoconv.usrbinunoconv.stdout","msg":"Using office binary path: /usr/lib/libreoffice/program"}
{"level":"debug","ts":1654248935.8413668,"logger":"unoconv.usrbinunoconv.stdout","msg":"Start listener on <IP_ADDRESS>:34909"}
{"level":"debug","ts":1654248935.841392,"logger":"unoconv.usrbinunoconv.stdout","msg":"Restarting LibreOffice (first start - 81 exit code)"}
{"level":"debug","ts":1654248936.4876368,"logger":"api.formslibreofficeconvert.usrbinunoconv.stderr","msg":"unoconv: RuntimeException during import phase:","trace":"7584b0fa-fbbe-4df6-8b1c-7341d5a01efa"}
{"level":"debug","ts":1654248936.487844,"logger":"api.formslibreofficeconvert.usrbinunoconv.stderr","msg":"Office probably died. Binary URP bridge disposed during call ./binaryurp/source/bridge.cxx:613","trace":"7584b0fa-fbbe-4df6-8b1c-7341d5a01efa"}
{"level":"debug","ts":1654248936.488449,"logger":"api.formslibreofficeconvert.usrbinunoconv.stdout","msg":"Using office base path: /usr/lib/libreoffice","trace":"7584b0fa-fbbe-4df6-8b1c-7341d5a01efa"}
{"level":"debug","ts":1654248936.4888082,"logger":"api.formslibreofficeconvert.usrbinunoconv.stdout","msg":"Using office binary path: /usr/lib/libreoffice/program","trace":"7584b0fa-fbbe-4df6-8b1c-7341d5a01efa"}
{"level":"debug","ts":1654248936.6989663,"logger":"api.formslibreofficeconvert.usrbinunoconv","msg":"unix process already killed","trace":"7584b0fa-fbbe-4df6-8b1c-7341d5a01efa"}
{"level":"debug","ts":1654248936.7014818,"logger":"api.formslibreofficeconvert","msg":"unoconv lock released","trace":"7584b0fa-fbbe-4df6-8b1c-7341d5a01efa"}
{"level":"debug","ts":1654248936.7059922,"logger":"api.formslibreofficeconvert","msg":"'/tmp/d93bb79d-71db-4c56-99bd-051ce5805283' removed","trace":"7584b0fa-fbbe-4df6-8b1c-7341d5a01efa"}
{"level":"error","ts":1654248936.7155395,"logger":"api","msg":"convert to PDF: unoconv PDF: unix process error: exit status 6","trace":"7584b0fa-fbbe-4df6-8b1c-7341d5a01efa","remote_ip":"<IP_ADDRESS>","host":"gotenberg-service:3000","uri":"/forms/libreoffice/convert","method":"POST","path":"/forms/libreoffice/convert","referer":"","user_agent":"Apache-HttpClient/4.5.13 (Java/11.0.10)","status":500,"latency":16033351171,"latency_human":"16.033351171s","bytes_in":775608,"bytes_out":21}
{"level":"debug","ts":1654249205.218242,"logger":"gc","msg":"'/tmp/OSL_PIPE_1001_SingleOfficeIPC_5d5fd1854aa05ac155dfddba328619' removed"}
{"level":"debug","ts":1654249205.2200477,"logger":"gc","msg":"'/tmp/lu20y91uu6.tmp' removed"}

Hello guys.
This is not a contribution, but a comment on the experience I had while converting pptx into pdf using LibreOffice.
In the past, we faced serious problems with a lot of different pptx files, and after some research, we found out that the OnlyOffice plataform does a really good job on converting MS Office documents into PDF.. way better than LibreOffice.
So, I don't really know if this would be an alternative, but I think it would be good to have OnlyOffice, at least as an optional alternative for these kinds of conversions.

@douglascrp indeed, but last time I checked, their license is not great for Gotenberg alas 🥲

Yes, that is true.
But as I said, I think it would be nice to have it as an option, if the final user decides to use it.

Hi @gulien, @douglascrp
I was able to workaround the errors I was seeing by giving our gotenberg container increased memory and cpu.

@keithcaff great! I have to update the documentation with a troubleshooting section now 😅

Documentation updated!

I just encountered this problem today. Minor suggestion, I think its best to document the recommended minimum specification for this service, so people could configure their hardware/container/pod in advance.


DataStreamer: Scroll view causes the plot to zoom out only (not zoom in)

Issue: When running a Datastream the Graph cannot be zoomed into further than the largest plotted data. Even though it has long dissapeared from the visible Data. (could there be a way to discharge the first data set in the DataStreamer Points arg?
ScottPlot Version: 5.0.23
Code Sample:

using ScottPlot;

namespace FinalFinal
{
    public partial class Form1 : Form
    {
        private BackgroundWorker UIWorker;

        public new List<double> dataX;
        public new List<double> dataY;
        public bool firstCon = false;

        public ScottPlot.Plottables.DataStreamer StreamX;
        public ScottPlot.Plottables.DataStreamer StreamY;

        public Int32 AverageX1;
        public Int32 AverageY1;

        public int DisplayX1 = 0;
        public int DisplayY1 = 0;

        public Form1()
        {
            InitializeComponent();

            if (firstCon == false)
            {
                initvoid();
            }

            UIWorker.DoWork += UIWorker_DoWork;
        }

        private void initvoid()
        {

            UIWorker = new BackgroundWorker();
            StreamX = formsPlot1.Plot.Add.DataStreamer(100000);
            StreamY = formsPlot1.Plot.Add.DataStreamer(100000);

            dataX = new List<double>();
            dataY = new List<double>();
            firstCon = true;

            StreamY.ViewScrollLeft();
            StreamX.ViewScrollLeft();
        }

        public void NewRealtimeData(Int32 X1, Int32 Y1)
        {
		// DISPLAYX1 AND DISPLAYY1 ARE JUST TWO VALUES WHICH I MEASURE WITH A SENSOR

            // Add X1 value to dataX list
            dataX.Add(DisplayX1);
            dataY.Add(DisplayY1);

            //If dataX list is full, add it to the Streamer
            if (dataX.Count >= 1000 && !UIWorker.IsBusy) // Adjust the threshold as needed
            {
                    UIWorker.RunWorkerAsync();
            }
        }

        private void UIWorker_DoWork(object sender, DoWorkEventArgs e)
        {
            if (IsHandleCreated)
            {
                Invoke(new Action(() =>
                {
                    if (chkX.Checked)
                    {
                        StreamX.AddRange(dataX.ToArray()); // Add dataX list to Streamer
                    }

                    if (chkY.Checked)
                    {
                        StreamY.AddRange(dataY.ToArray());
                    }

                    formsPlot1.Refresh();
                }));
            }

            dataX.Clear(); // Clear dataX list after adding to Streamer
            dataY.Clear();
        }
    }
}

This is a gif of what I mean (please ignore the two lines "Alarm")

Also I have a question regarding this / similar :
I want to display the X/Y Points in a scatter Plot. How can I only show the 100 newest points and discard the older ones?

have the sampe problem too.mark this.

DataStreamer tracks the min and max when new data comes in, which means the min can only get smaller and the max can only get bigger. It seems like we need to set axis limits manually for now to scale the plot down when using DataStreamer .
Source for Add function in DataStreamerSource:
https://github.com/ScottPlot/ScottPlot/blob/7d54aac659fd3c61abec85c6cd79e9d3bb988a92/src/ScottPlot5/ScottPlot5/DataSources/DataStreamerSource.cs#L56C3-L68C6

I added a ContinuouslyAutoscale flag to support this behavior. It's a little disorienting for some datasets so it's an opt-in feature, but it should do well for the applications described here. This feature will be in the next release! 🚀

Confirmed it works well when multiple streamers are in use


Add i18n-zh pages for contribute module

closes: #ISSUE_Number

Change logs
Add i18n-zh pages for contribute module.
Contributor's checklist
Here are some reminders before you submit your pull request:

Make sure that your Pull Request has a clear title and commit message. You can take the Git commit template as a reference.
Sign the Contributor License Agreement as prompted for your first-time contribution (One-time setup).
Learn the code contribution and doc contribution guides for better collaboration.
Make sure that your changes deployment preview is successful.
List your communications in the GitHub Issues or Discussions (if has or needed).
Feel free to ask for the @cloudberrydb/doc team or other people to help review and approve.

@TomShawn thanks for your review work. 👍


Further work on the spec

@olafurpg Thank you! This will free me up for other stuff in the meanwhile.


Yii2 controller and dash

this bug has occurred when i use a controller with dash.
class SiteController extends yii\web\Controller{
    function actionIndex(){
        return $this->render('index');
    }
}

goto http://example.com/web/site-
or
goto http://example.com/web/site------
error
The view file does not exist: ....\views\site-\index.php
or
The view file does not exist: ....\views\site------\index.php

Q
A

Yii version
2.0.12

Verified. Controller is executed while it should not.


Maintainer approval check

Description
Implements proposal from https://github.com/opensearch-project/OpenSearch/issues/10613#issue-1942146755

Replace Codeowners to control accepting pull requests with a GitHub action check that reads from the list of maintainers any verifies they have approved the PR. Codeowners can still be used for mapping subject matter experts that want to be notified, but isn't a requirement for every maintainer to be listed on the top level.
By modifying the branch protection rules 'maintainer has approved' check, could be required (only bypassable by an Admin), which would offer effectively the same security as Codeowners.

This is a new approval check that effectively doubles as the existing CODEOWNERS checks on this repository - which will allow that code owners check to be removed in favor of this one.
This check pulls the list of maintainers from GitHub repository settings.  This also codifies the number of approvals needed in the repository in the event we increase or alter these requirements.
UX when PRs are blocked:

Example workflow:
https://github.com/peternied/OpenSearch-1/actions/runs/7026869463/job/19120342204?pr=138
Rollout plan:
After this PR has been merged, and been broadly accepted, a repository admin will change the required checks to include this workflow - this prevents changes from being merged even if this check isn't passing.  Then  the code owners file can be updated so maintainers can direct the kinds of changes they would like to be notified about.
Related Issues

Related https://github.com/opensearch-project/OpenSearch/issues/10613

Check List

[ ] ~New functionality includes testing.~

[ ] ~All tests pass~

[ ] ~New functionality has been documented.~

[ ] ~New functionality has javadoc added~

[x] Failing checks are inspected and point to the corresponding known issue(s) (See: Troubleshooting Failing Builds)
[X] Commits are signed per the DCO using --signoff
[x] Commit changes are listed out in CHANGELOG.md file (See: Changelog)
[ ] ~Public documentation issue/PR created~

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@peternied +1 to give it a try, thank you. Just to clarify for myself, the list of collaborators automatically include ones listed in CODEOWNERS as well?

@peternied +1 to give it a try, thank you. Just to clarify for myself, the list of collaborators automatically include ones listed in CODEOWNERS as well?

@reta  CODEOWNERS doesn't include permissions in the github sense - it defines a mapping between selected reviewers on PR and the file hierarchy - and it can be enforced by a branch protection setting which is currently enabled.  The permissions for users are from the repo settings which is where the list of collaborators comes.
After this change is merged, a maintainer could remove themselves from the CODEOWNERS entirely and still approve & merge PRs.  Personally, I plan to change my setting to watch our governance files on the root, GHAs, identity, http, and a few targeted areas of interest.  When future me is notified I know it was either because I self selected or was @mentioned. :shaking-with-excitement:

I'm really excited about this change.
Before people start removing themselves from CODEOWNERS, though, I think we'll need to update (or remove?) some of the tooling that @dblock and @prudhvigodithi have to make sure that contributors and CODEOWNERS are kept in sync.

I'm really excited about this change.
Before people start removing themselves from CODEOWNERS, though, I think we'll need to update (or remove?) some of the tooling that @dblock and @prudhvigodithi have to make sure that contributors and CODEOWNERS are kept in sync.

Correct, please open an issue on opensearch-build at least? cc: @bbarani

I like the approach but this might be a problem in one case, what if a user raised a PR (who is not in slack) and dont know whom to tag/mention? With the existing approach of notifying every codeowner, a codeowner can tag right person (SME for the PR topic) who can review the PR. @peternied WDYT?

@prudhvigodithi The MAINTAINERS.md is always an option for finding someone, same is available with commit history. I'm not sure how often this is a problem - could you collect up data and make an issue? I'd like to address that out of band of this change.

@prudhvigodithi The MAINTAINERS.md is an option for finding someone, same with browsing PR history. I'm not sure how often this is a problem - could you collect up data and make an issue? I'd like to address that out of band of this change.

Sure Peter, we dont have to stop this change for the problem I have raised, as we never have a PR with empty reviewers with existing CODEOWNERS file setup, this would be something new and we can iterate as we learn, but is the expectation to make this the de facto standard for all repos? If so then when we run a repo audit we need to exclude CODEOWNERS. CC: @bbarani

is the expectation to make this the de facto standard for all repos

@prudhvigodithi Not generally - smaller repos are more manageable and this would just add overhead whereas the default tools from GitHub work well.  OpenSearch and OpenSearch Dashboards are good candidates for this change to help maintainers track incoming PRs.
However, we can always build on this change to make more improvements.

FYI - I was doing some additional validation on the workflow and I've run into an issue with the peternied/approved-by-maintainers erroring on logins with casing :/ I'll update when this has been resolved

Thanks @peternied we should consider standardizing else it would deviate the standard baseline process for few repos.
@dblock @msfroh @bbarani WDYT?

FYI - I was doing some additional validation on the workflow and I've run into an issue with the peternied/approved-by-maintainers erroring on logins with casing :/ I'll update when this has been resolved

This has been resolved - successful verification workflow: https://github.com/peternied/OpenSearch-1/actions/runs/7119425260/job/19421619223?pr=138
@reta @msfroh @dblock What do you think about merging this?

All tests are passing - only github workflow file changes - https://github.com/opensearch-project/OpenSearch/issues/7396 will require a rebase, I'm merging to expedite this process.

@prudhvigodithi (CC @opensearch-project/admin)
Since the check is now available an running in 1.3, 2.x and main.  Could you make the following change to the branch protection on OpenSearch?

Uncheck Require review from Code Owners

Check Require status checks to pass before merging

Do not check or uncheck Require branches to be up to date before merging
Add Minimum approval count to the list of Status checks that are required.

@peternied I have just updated the rule with the changes you mentioned.


MessageType fullName error

file.proto
package example.simple;
message Message1 {
optional string bestHuman = 1;
}
this proto file will generate example.simple.file.Message1
by google proto compiler
but square/protoparser will
getFullyQualifiedName as example.simple.Message1

What do you mean by "generate example.simple.file.Message1 by google proto compiler"? Do you mean that its internal model will use this format or do you mean that the code that gets generated (like Java) will use it.

the java code generated by protoc is  example.simple.file.Message1
but square/protoparser 's
getFullyQualifiedName return example.simple.Message1.

ProtoParser is a representation of the schema. You could assemble example.simple.file.Message1 from the component parts of the package name, file name, and message name. This project does not create any files though.
If you are referring to Wire that's a different project with different views. One specifically being that we ignore the file name and stick to traditional package + message naming concatenation.


Additional Dutch translations

Added DateRange stuff
Added ShareStreamer stuff
Removed duplicate 'SuggestedStreamers'

Thank you for the updates Rick!


tensorflow and mace produce different output result

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
Ubuntu 16.0
NDK version(e.g., 15c):
17b
GCC version(if compiling for host, e.g., 5.4.0):
MACE version (Use the command: git describe --long --tags):
master
Python version(2.7):
2.7
Bazel version (e.g., 0.13.0):
15

Model deploy file (*.yml)
......

Describe the problem
Hi , I have TEST tensorflow model with Conv2D , DepthwiseConv2D, and BiassAdd layers. After converting to mace format it produces BITEXACT output on Google Pixel 2 !!!!!
But when we converted our Real model with additional to test model Relu, FuseBatchNorm, Reshape, Shape, Pack, StridedSlice, Pack, Const layers - results are completely wrong !!!!
Please advise how to resolve it.
Thanks.
P.S.  Latest Mace versoin show very good performance result compares to tensorflow lite on Google Pixel 2 with our model, but this shortstopper preventing form us to move forward.

Can you describe this issue with more details?
Which model did you run on Pixel 2? On CPU or GPU? How do you decide if your results are correct or wrong?

Hi @liyinhgqw ,

Model similar to Resnet50.
CPU.
3.Input data same for tensorflow and Mace.

Can you attach your model and yaml config file for us to debug?

Hi @liyinhgqw ,
download atached files .
Do to size limitation I split zip file in 10 parts .

remove zip extension from files Data1.z01.zip - Data1.z10.zip -> Data1.z01 - Data1.z10
2.use winzip to combine all parts  together .
3.Unzip zip file.
Data1.z01.zip
Data1.z02.zip
Data1.z03.zip
Data1.z04.zip
Data1.z05.zip
Data1.z06.zip
Data1.z07.zip
Data1.z08.zip
Data1.z09.zip
Data1.z10.zip
Data1.zip.zip

Hi @liyinhgqw , any progress ?

Could you resend your models and yaml by email:
<EMAIL_ADDRESS>?

@dimitryn Thanks for report. It's suggested to use Google drive for large files sharing or using free zip tools like 7zip to make splits. Winzip is usually not available for us. (I tried to unzip it using a trail version winzip but failed)

@llhe I send model and instructions to  @liyinhgqw

Maybe you can replace following ops with const shape(1,32768)
Resface/Logits/Flatten/flatten/Shape
Resface/Logits/Flatten/flatten/strided_slice
Resface/Logits/Flatten/flatten/Reshape/shape

The data format of MACE/CPU/float is NCHW, so we need to transform
FormerOp(4D)->Reshape(2D)->MatMul(2D) to FC to convert tensorflow model to MACE model.
Otherwise, input to MatMul would be in wrong format.

You can try tf.layers.dense after Conv4

@lee-bin
1.tf.layers.dense  is not an option for us.
2.Replace flatten with const shape as you suggested did not change output, there is still a difference between mace and tensorflow.

We try to move all graph into NCHW but there been an error on conversion :

Generic conv implementation only supports NHWC tensor format for now:
[[Node: Resface/Conv1/Conv1_pre/Conv2D = Conv2D[T=DT_FLOAT, data_format="NCHW", dilations=[1, 1, 1, 1], padding="SAME", strides=[1, 1, 2, 2], use_cudnn_on_gpu=true, _device="/job:localhost/replica:0/task:0/device:CPU:0"](transpose, Resface/Conv1/Conv1_pre/weights)]]

We believe there is a bug in flatten / Matmul  and it does not work.

Did you replace flatten with reshape(batch, height * width * depth)?
MACE/CPU/float converter can transform FormerOp(4D)->Reshape(2D)->MatMul(2D) to FormerOp(4D)->FC(4D) and perform right, but not FormerOp(4D)->Flatten(2D)->MatMul(2D).
MatMul(2D)->Reshape(2D)->FusedBatchNorm(4D) is not gonna work either, due to NCHW format.
So if you transform to dense and batchnorm, that would work fine on MACE.
If that't still not working, you can send us the python code so that we can see if we can help you.

@lee-bin I attached  flatten replace with reshape

@lee-bin does shape in the attached image correct?

This seems fine.

@lee-bin Update.
1.After another check MatMul working bitexact with tensorflow , flatten replace with reshape working, Thanks.
2.problem still exist after MatMul  and before final output_features layers.
Any suggestion?
T

haks.

As I mentioned, Reshape maybe a problem due to different data formats. So if you can replace Shape
after MatMul to const shape and just get FusedBatchNorm's output as result, that would work fine.
MACE currently does not remove Reshape after MatMul when transform to FC, so maybe you can send me a new model after replacing Shape with Const shape, so that I can test.
My email is<EMAIL_ADDRESS>

thanks for help


[BACKLOG-24628] Update search behavior for the field selection dialog

Build Completed
:x: This pull request has errors. They will need to be addressed before it can be accepted. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
mvn -B -f 'pom.xml' -pl 'ui,plugins/json/core,plugins/get-fields/core' -P '!assemblies' -Dsurefire.runOrder=alphabetical -Daudit -amd clean install

Cleanup Commands
mvn -B -f 'pom.xml' -pl 'ui,plugins/json/core,plugins/get-fields/core' -P '!assemblies' -amd build-helper:remove-project-artifact

Changed files
  plugins/get-fields/core/src/main/java/org/pentaho/getfields/dialog/GetFieldsDialog.java
  plugins/get-fields/core/src/main/java/org/pentaho/getfields/extension/GetFieldsExtensionPoint.java
  plugins/get-fields/core/src/main/javascript/app/app.component.js
  plugins/get-fields/core/src/main/javascript/app/app.css
  plugins/get-fields/core/src/main/javascript/app/app.html
  plugins/get-fields/core/src/main/javascript/app/app.module.js
  plugins/get-fields/core/src/main/javascript/app/components/tree/tree.component.js
  plugins/get-fields/core/src/main/javascript/app/components/tree/tree.css
  plugins/get-fields/core/src/main/javascript/app/components/tree/tree.html
  plugins/get-fields/core/src/main/javascript/app/directives/resize.directive.js
  plugins/json/core/src/main/java/org/pentaho/di/ui/trans/steps/jsoninput/JsonInputDialog.java
  ui/src/main/java/org/pentaho/di/ui/core/GetFieldsDialogOperation.java

Unit Test Coverage
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.

:warning: Coverage Changes (click to expand)
org.pentaho.di.trans.steps.googleanalytics.GaInputStepMeta

Branch Change: -1.9% :small_red_triangle_down:
Complexity Change: -1.1% :small_red_triangle_down:
Instruction Change: -0.1% :small_red_triangle_down:
Line Change: -0.3% :small_red_triangle_down:

Checkstyle violations
The code submitted is not complying with one or more of our guidelines. To perform these checks locally, please use this tool.
/home/buildguy/project/plugins/json/core/src/main/java/org/pentaho/di/ui/trans/steps/jsoninput/JsonInputDialog.java

:heavy_exclamation_mark: ')' is not preceded with whitespace. at Line: 1,487 Column: 66

Build Completed
:white_check_mark: This pull request has passed all validations.
Build Commands
mvn -B -f 'pom.xml' -pl 'ui,plugins/json/core,plugins/get-fields/core' -P '!assemblies' -Dsurefire.runOrder=alphabetical -Daudit -amd clean install

Cleanup Commands
mvn -B -f 'pom.xml' -pl 'ui,plugins/json/core,plugins/get-fields/core' -P '!assemblies' -amd build-helper:remove-project-artifact

Changed files
  plugins/get-fields/core/src/main/java/org/pentaho/getfields/dialog/GetFieldsDialog.java
  plugins/get-fields/core/src/main/java/org/pentaho/getfields/extension/GetFieldsExtensionPoint.java
  plugins/get-fields/core/src/main/javascript/app/app.component.js
  plugins/get-fields/core/src/main/javascript/app/app.css
  plugins/get-fields/core/src/main/javascript/app/app.html
  plugins/get-fields/core/src/main/javascript/app/app.module.js
  plugins/get-fields/core/src/main/javascript/app/components/tree/tree.component.js
  plugins/get-fields/core/src/main/javascript/app/components/tree/tree.css
  plugins/get-fields/core/src/main/javascript/app/components/tree/tree.html
  plugins/get-fields/core/src/main/javascript/app/directives/resize.directive.js
  plugins/json/core/src/main/java/org/pentaho/di/ui/trans/steps/jsoninput/JsonInputDialog.java
  ui/src/main/java/org/pentaho/di/ui/core/GetFieldsDialogOperation.java

@ddiroma


Docs for Stream in Brixton docs appear to have broken images

Broken image links here: http://cloud.spring.io/spring-cloud-static/Brixton.SR7/#_main_concepts

Hi, @rseroter. Thanks for reporting! The recent Dalston.RELEASE (includes Stream's Chelsea.SR1) render the images properly. We will address the Brixton docs.

@rseroter Unfortunately, there is an issue with how earlier Spring Cloud versions aggregate documents. As a side note, Spring Cloud Stream publishes its own documentation (as it is a release train in and by itself) - see http://cloud.spring.io/spring-cloud-stream/ and the links for reference documentation for Brooklyn (included in Camden), Chelsea (included in Dalston) and 1.0 (included in Brixton).

Closing this as it is fixed in Dalston http://cloud.spring.io/spring-cloud-static/Dalston.RELEASE/#_rabbitmq_binder as indicated by @marcingrzejszczak.


Interlock -- Consume Latest Core-Contracts for Unmarshal Bug Fix

See original issue here
https://github.com/edgexfoundry/go-mod-core-contracts/issues/80
There's a bug in the unmarshaling of event readings whereby the value is being dropped. Once the relevant PR is merged for the above bug fix, it should be consumed ASAP in edgex-go.

closed via #1293


[BUG]can't change x-tunnel-server-svc service type from nodeport to loadbalancer

What happened:
when x-tunnel-server-svc was updated with service type from nodeport to loadbalancer, it will revert this change just in minutes.
also will recreate when been deleted.
What you expected to happen:
I want to use loadbalancer instead of nodeport to connect to yurt-tunnel-server
How to reproduce it (as minimally and precisely as possible):
just delete it or change the yaml
Anything else we need to know?:
none
Environment:

OpenYurt version: 0.6.0
Kubernetes version (use kubectl version): 1.20.11
OS (e.g: cat /etc/os-release): aliyun centos7.9
Kernel (e.g. uname -a): aliyun centos7.9
Install tools: yurtcluster-operator
Others: using aliyun ack with 3 masters 1 cloud node

others
/kind bug

@shao77622 The core configurations are guarded by the operator.
Please add operator.openyurt.io/skip-reconcile: "true" annotation to the object which you want to change.


Error when installing through ESPHome

Hello arpiecodes.
Trying to install this on my Senorbox 2 and get this error when installing through EspHome. Me being a non-coder, I can not make any sense of the Logs, Could you possible help me understand what the problem is. Thank you. See attached full Logg

/config/esphome/sensorbox-2.yaml: In function 'void setup()':
/config/esphome/sensorbox-2.yaml:178:27: error: 'esphome::sensor::Sensor* esphome::smartevse_sensorbox::SmartEVSESensorbox::ct1_current_' is private within this context
return { id(sensorbox)->ct1_current_, id(sensorbox)->ct2_current_, id(sensorbox)->ct3_current_};
^~~~~~~~~~~~
In file included from src/esphome.h:39,
from src/main.cpp:3:
src/esphome/components/smartevse_sensorbox/smartevse_sensorbox.h:77:79: note: declared private here
esphome::sensor::Sensor* ct1_current_ = new esphome::sensor::Sensor();

<
logs_sensorbox-2_compile.txt

Hello @Xnygra, I've merged the latest code from develop branch (working properly here for months) into main. This should also contain a fix for the private error on compile. Could you try it out?

Hi arpiecodes, Thank you for very swift reply :)
Unfortunately I got another set of errors:
INFO ESPHome 2023.11.6
INFO Reading configuration /config/esphome/sensorbox-2.yaml...
Failed config
smartevse_sensorbox: [source /config/esphome/sensorbox-2.yaml:83]
id: sensorbox
uart_id: uart_pic
use_ct_readings: False
ct_rotation: CW
ct_wires: 4WIRE
'power_consumed_phase_2' is a required option for [p1_sensors].
p1_sensors:  [source /config/esphome/sensorbox-2.yaml:90]
dsmr_version: dsmr_version
[current_phase_1] is an invalid option for [p1_sensors]. Did you mean [voltage_phase_1]?
current_phase_1: current_phase_1

[current_phase_2] is an invalid option for [p1_sensors]. Did you mean [voltage_phase_2]?
current_phase_2: current_phase_2

[current_phase_3] is an invalid option for [p1_sensors]. Did you mean [voltage_phase_3]?
current_phase_3: current_phase_3
voltage_phase_1: voltage_phase_1
voltage_phase_2: voltage_phase_2
voltage_phase_3: voltage_phase_2

required key not provided
required key not provided

Ah, I forgot to update the example yaml in the README. It's now updated, as there were some changes in develop branch. You will probably need to update your YAML as well according to the new example in the main branch.

I found another potential issue after the merge from develop to main which I just fixed with a commit. Should all be good to go now! Let me know how it works out for you.

Works like a charm.
Now I just have to figure out how to use it :)
Thank you very much 🥇
INFO ESPHome 2023.11.6
INFO Reading configuration /config/esphome/sensorbox-2.yaml...
INFO Updating https://github.com/arpiecodes/esphome-modbus-server.git@master
INFO Updating https://github.com/arpiecodes/esphome-smartevse-sensorbox.git@main
INFO Detected timezone 'Europe/Stockholm'
INFO Starting log output from sensorbox.local using esphome API
INFO Successfully connected to sensorbox in 0.342s
INFO Successful handshake with sensorbox in 0.114s
[18:49:31][I][app:102]: ESPHome version 2023.11.6 compiled on Dec 16 2023, 18:44:33
[18:49:31][C][status_led:019]: Status LED:
[18:49:31][C][status_led:020]:   Pin: GPIO32


Add kubernetes metadata labels: namespace, pod, container

Refer to this link for build results (access rights to CI server needed):
https://fabric8-ci.fusesource.com/job/fabric8-quickstarts-master-pullreq/667/
Test PASSed.


add kfuncs docs

Description

Fixes # (issue)
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?

[ ] Test A
[ ] Test B

Test Configuration:

Firmware version:
Hardware:
Toolchain:
SDK:

Checklist

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

为啥删了libbpf的submodule？

为啥删了libbpf的submodule？

看到了，改成软链了


generate-example script for terra-icon failing

Bug Report
Description

generate-example script for terra-icon fails with
> node scripts/lib/generate-example/index.js

events.js:291
      throw er; // Unhandled 'error' event
      ^

Error: ENOENT: no such file or directory, open '../../packages/terra-icon/src/terra-dev-site/doc/example/IconThemeable.jsx'
Emitted 'error' event on WriteStream instance at:
    at internal/fs/streams.js:361:12
    at FSReqCallback.oncomplete (fs.js:156:23) {
  errno: -2,
  code: 'ENOENT',
  syscall: 'open',
  path: '../../packages/terra-icon/src/terra-dev-site/doc/example/IconThemeable.jsx'
}
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>generate-example: `node scripts/lib/generate-example/index.js`
npm ERR! Exit status 1

Generated files were migrated to packages/terra-core-docs/src/terra-dev-site/doc/icon/example/, but the path in the scripts is still the old one:
generateStaticIcons.js
generateThemeableIcons.js
Steps to Reproduce

Run npm run migrate-cerner-one-icons in terra-icon

Expected Behavior

Generated scripts are written to their home in terra-core-docs
Possible Solution

Update path in lines linked above to packages/terra-core-docs/src/terra-dev-site/doc/icon/example/...
Environment

Component Name and Version: terra-icon v3.46.1
Browser Name and Version: n/a
Node/npm Version: node v12.22.9/npm v6.14.15
Webpack Version:
Operating System and version (desktop or mobile): MacOS 12.1 Monterey

@ Mentions

Note: this overlaps with work we'll be doing on the icon docs page in the next few weeks. It can be resolved alongside that if nobody thinks it's more urgent.


Sorted SN Domains and New Tool

Delete commented hosts in the part "SN Domains"
Sort "SN Domains" part by address to facilitate insertion of new addresses
Add new address of "npo"
Add a small tool to list a specific group of sn domains(r1 to r20) to save the time spent on
update_hosts.py. Suppose you know a new group domain
r3---sn-i3beln7k.googlevideo.com and want to know domains of this
whole group, use command ./list-sn-spec.py i3beln7k > i3beln7l.txt. This can replace Step 1 to 3 mentioned in Wiki if you are in a hurry.

Actually I only sort the domains like r20---sn-o097znee.googlevideo.com, domains like r6---sn-uxaxufv-8pxl.googlevideo.com are not sorted.
Using the new tool can help find domains like the latter.

One small suggestion, make pull request smaller will make the reviewer focus on the changes more clear, for example you can make "Add new tool" into one single PR, and make other changes to another.


GrabButton requests always fail because of the length

GrabButton requests fail because of the event mask field: the EventMask struct contains a u32, but the request wants a u16. One possible solution would be to make a new struct for PointerEvents, like in the X Protocol Docs (see SETofPOINTEREVENT).

It seems that this was a 2.0.0 problem. We will begin shifting to newer versions soon, but because of how massively the structure of everything has changed, it will be difficult.

Sounds good, I wrote 3.0 to fix many of the issues that 2.0 had. Let me know if you have any questions.


Concurrent session writes may result in SQL exception

Describe the bug
The uniqueness of the session key ensures that there is only one record in the session table. However, when
there are many parallel requests the SQLs may run at the same time and the current duplication prevention is
faulty, it does not synchronise properly.
Additional context
Session cache should be moved to SessionMaintenanceTask (maybe rename to reflect function) and all session operatoins should be sent through the channel.

doesn't seem to be a problem atm


Added examples on how to use pretend

The examples should now cover all
features.

Codecov Report

Merging #9 (0da5f8b) into main (1d2ec78) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main       #9   +/-   ##
=======================================
  Coverage   94.11%   94.11%
=======================================
  Files           5        5
  Lines         153      153
=======================================
  Hits          144      144
  Misses          9        9

Impacted Files
Coverage Δ

pretend/src/lib.rs
85.71% <ø> (ø)

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 1d2ec78...0da5f8b. Read the comment docs.


[Bug Report]: A Large Config Array Uses All System Resources

System Info
  System:
    OS: macOS 12.6.3
    CPU: (10) arm64 Apple M1 Max
    Memory: 15.62 GB / 64.00 GB
    Shell: 5.8.1 - /bin/zsh
  Binaries:
    Node: 16.19.1 - ~/.local/share/rtx/installs/nodejs/16.19.1/bin/node
    Yarn: 1.22.19 - ~/.local/share/rtx/installs/nodejs/16.19.1/bin/yarn
    npm: 8.19.3 - ~/.local/share/rtx/installs/nodejs/16.19.1/bin/npm
  Browsers:
    Chrome: 110.0.5481.177
    Firefox: 110.0.1
    Safari: 16.3
  npmPackages:
    @rspack/cli: latest => 0.1.2

Details
At some point when using an array of configurations (7 when I first encountered it, 10 in the repro repo) the build slows to a crawl, making the build take 10x as long.
Here is how the repro repo build looks on my M1 Mac 64GB:
Building 1 configs...
build: 288.429ms
Building 2 configs...
build: 297.402ms
Building 3 configs...
build: 395.663ms
Building 4 configs...
build: 317.321ms
Building 5 configs...
build: 339.679ms
Building 6 configs...
build: 414.946ms
Building 7 configs...
build: 586.755ms
Building 8 configs...
build: 564.843ms
Building 9 configs...
build: 831.296ms
Building 10 configs...
build: 9.221s          <-- Right here!
Building 11 configs...
build: 10.404s
Building 12 configs...
build: 7.152s

Reproduce link
https://github.com/BeeeQueue/repro-rspack-config-array
Reproduce Steps

Install deps - pnpm install
Run build - pnpm build
See in logs how at some point the build slows to a crawl

It would make sense, in the project I initially encountered it the entire computer freezes intermittently while at 100% CPU usage

Not saying this is 100% the case, but in webpack i accidentally ran a parallel build task over module-federation-examples which issues 179 webpack builds at once. My computer froze up and was not able to even exit terminal.
Blind guess is that there may need to be a "management" thread for Rust to communicate with JS bindings over. If the CPU is choked it might put some resource limits on Rust-JS transfer as JS, even with multiple processes seems to have a few global limits.
Defiantly worth looking into a little deeper, because i can see a NX monorepo trying to build in parallel and we don't want to impact those common use cases.

@BeeeQueue I found a workaround and I need your assistance to verify if this works.
Inside<EMAIL_ADDRESS>add options.parallelism = 1; below line 29:
class MultiCompiler {
    constructor(compilers, options) {
        options.parallelism = 1;

And comment out line 444
                if (result) {
                    node.result = undefined;
                    // stats.push(result);
                }

There are two problems at hand:

Running the multi compiler in parallel will cause Rspack to go haywire on the Rust side
Sending a huge stats array through napi will cause it to crash

This should be fixed by the referenced issues and PRs, feel free to reopen if this is still happening.

From my quick testing it has not fixed it.

From my quick testing it has not fixed it.

Do you mind updating your repro so I can take a quick look?

It had already been updated but I updated it to the latest version again and there's still no difference.
Running it locally:

Details
repro-rspack-config-array main |  v18.16.0  took 23s
❯ nr build

><EMAIL_ADDRESS>build /Users/ahaglund/projects/repro-rspack-config-array
> for i in {1..12}; do AMOUNT="$i" rspack build; done

Building 1 configs...
Time: 135ms
Building 2 configs...
Time: 157ms

Time: 151ms
Building 3 configs...
Time: 143ms

Time: 134ms

Time: 138ms
Building 4 configs...
Time: 172ms

Time: 170ms

Time: 169ms

Time: 160ms
Building 5 configs...
Time: 171ms

Time: 161ms

Time: 157ms

Time: 164ms

Time: 163ms
Building 6 configs...
Time: 204ms

Time: 209ms

Time: 219ms

Time: 190ms

Time: 201ms

Time: 213ms
Building 7 configs...
Time: 273ms

Time: 279ms

Time: 231ms

Time: 224ms

Time: 331ms

Time: 224ms

Time: 280ms
Building 8 configs...
Time: 374ms

Time: 305ms

Time: 397ms

Time: 353ms

Time: 335ms

Time: 385ms

Time: 364ms

Time: 347ms
Building 9 configs...
Time: 822ms

Time: 768ms

Time: 842ms

Time: 797ms

Time: 760ms

Time: 799ms

Time: 783ms

Time: 788ms

Time: 794ms
Building 10 configs...
Time: 6569ms

Time: 6812ms

Time: 6812ms

Time: 6837ms

Time: 6788ms

Time: 6763ms

Time: 6763ms

Time: 6834ms

Time: 6754ms

Time: 6843ms
Building 11 configs...
Time: 4480ms

Time: 5780ms

Time: 5686ms

Time: 5812ms

Time: 5784ms

Time: 5730ms

Time: 5555ms

Time: 5787ms

Time: 5647ms

Time: 5728ms

Time: 5731ms
Building 12 configs...
Time: 12812ms

Time: 14080ms

Time: 14280ms

Time: 14315ms

Time: 14165ms

Time: 14290ms

Time: 14225ms

Time: 14350ms

Time: 14283ms

Time: 14200ms

Time: 14341ms

Time: 14345ms

The workaround/fix from before still works as well
https://github.com/BeeeQueue/repro-rspack-config-array/pull/1

Bump

This is related to the parallelism to the related tasks of the module graph creations. webpack's limit is 100 by default. We should support this as well.

Hi there! Is a there a known workaround for this, so some kind of limit?
Our build peaks at a usage of more then 30gb (8gb in webpack).
We are using an array of 11 configs 🗡

In my repro repo it seems that the issue has been resolved.
The build times increase in a logical manner the more builds you have going on at once.

seems we can close this issue now


Automated install of CLI extension on azure VM via custom script extension

I'm working on a project to help people use the azure CLI for doing azure administrations.  I've created a VM using the custom-script extension which to installs a bunch of tools (via chocolatey), including VSCODE and would like to auto-install this extension on provision.
The issue I'm running into is the extension seems to install to the user path rather than to a common location.
I was using this:
#Install Visual Studio Code Extensions
& 'C:\Program Files\Microsoft VS Code\bin\code.cmd' --install-extension ms-vscode.azurecli
Do you have suggestions?
See this repo for the vm: https://github.com/Build5Nines/az-kung-fu-vm
I'm going to be releasing a large set of samples using the extension here: https://github.com/Build5Nines/az-kung-fu

Extensions are installed in default extensions directory in user home. You can change it using --extensions-dir argument.


Paste broken for entering secret key of new credential

Yubico Authenticator version: 4.3.2

Operating system and version: MacOS 10.13.3

YubiKey model and version: Any

Bug description summary:
I recently updated my Yubico Authenticator on MacOS. The older version would permit me to enter the secret key for new credentials via cut and paste, but the current (4.3.2) version does not. Some investigation showed that the current version will only accept a paste that does not contain spaces. Yet when Google provides secret key it is displayed with spaces preventing a simple cut and paste operation. The older versions of Yubico Authenticator simply ignored invalid characters rather than preventing their entry. I am quite sure the user experience of the older version has more utility than the current version as I imagine few users of the application prefer to type rather than cut and paste. Even if the user is typing, the spaces as used by Google make it much easier to type correctly.

Steps to reproduce
Attempt to cut and paste TOTP secret key from Google to Yubico Authenticator
Expected Result
Paste fails silently.  (fix by editing spaces out of cut text)

Thanks for the report! We'll look into it.


Need client endpoints for user/role/organization/capability management

The new user/role/organization/capability endpoints need to be added to the python client.

Fixed in https://github.com/terminusdb/terminusdb-client-python/pull/334


Fix install instructions

Bit of a copy & paste error here ;)

Haha yeah my bad when I was copying the readme file from ElementaryLua :P. Merging it.


Type soundness

The previous commits of fixing proofs due to change in memory

I noticed a lot of comments going like "N should be u32".
It's true in the official spec that many fields are declared as u32s rather than nats or vectors of max size 2^32 rather than lists, but all of the relevant range restrictions are already enforced statically at parse-time, so I would choose the type that has the best Coq standard library support/integration (which may end up being a compcert i32 type, I don't know). The official spec can pun between int/nat/u32 whenever it suits them, but moving to a badly-supported u32 type in the mechanisation might have a real readability/engineering trade-off.


cni: properly cleanup completed pods

Followup to https://github.com/istio/istio/pull/51429.
When a pod is moved to terminal stage, we call RemovePodFromMesh. This
has a variety of cleanup steps, and aborts early if any fail.
Because the pod is terminated, we will fail to cleanup the in-pod rules.
This is OK! The pod is terminated anyways. But with the current logic,
this also means we do not send ztunnel the removed message, and it will
leak proxy instances.
Additionally, this does a variety of logging cleanups so that we
consistently format messages and always show which pod we are acting on.

Followup to #51429.
When a pod is moved to terminal stage, we call RemovePodFromMesh. This has a variety of cleanup steps, and aborts early if any fail.
Because the pod is terminated, we will fail to cleanup the in-pod rules. This is OK! The pod is terminated anyways. But with the current logic, this also means we do not send ztunnel the removed message, and it will leak proxy instances.
Additionally, this does a variety of logging cleanups so that we consistently format messages and always show which pod we are acting on.

Both Remove and Del will remove the pod from ztunnel - do you mean "if either of those fail before we tell ztunnel"?

Oh I see. If the netns is gone when we go to remove the rules, the rule removal will fail.
I tested this locally in #51429 (kind) because I wasn't sure if the netns WOULD stick around after the job pod was terminated, and I did see the rules removal succeed, but I didn't do exhaustive testing and it very likely could be timing dependent.

@howardjohn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
883ea846c99cf1ccc54bbbf966562ab56dbe051e
link
true
/test lint

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@howardjohn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests_istio
bf7c554501547b2aa030167cb60b77c4b7cc7244
link
true
/test unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests_istio
bf7c554501547b2aa030167cb60b77c4b7cc7244
link
true
/test unit-tests

gencheck_istio
27aa5469854ef6c2291867f8917cb9016545f679
link
true
/test gencheck

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests_istio
bf7c554501547b2aa030167cb60b77c4b7cc7244
link
true
/test unit-tests

unit-tests-arm64_istio
27c58f65d79c15cbe2a89b8ea7e8873c992407b2
link
true
/test unit-tests-arm64

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
27c58f65d79c15cbe2a89b8ea7e8873c992407b2
link
true
/test unit-tests-arm64

unit-tests_istio
27c58f65d79c15cbe2a89b8ea7e8873c992407b2
link
true
/test unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

In response to a cherrypick label: #51713 failed to apply on top of branch "release-1.22":
Applying: cni: properly cleanup completed pods
Using index info to reconstruct a base tree...
M	cni/pkg/iptables/iptables_linux.go
M	cni/pkg/nodeagent/informers.go
M	cni/pkg/nodeagent/net.go
M	cni/pkg/nodeagent/net_test.go
M	cni/pkg/nodeagent/server.go
M	cni/pkg/nodeagent/ztunnelserver.go
Falling back to patching base and 3-way merge...
Auto-merging cni/pkg/nodeagent/ztunnelserver.go
Auto-merging cni/pkg/nodeagent/server.go
Auto-merging cni/pkg/nodeagent/net_test.go
Auto-merging cni/pkg/nodeagent/net.go
Auto-merging cni/pkg/nodeagent/informers.go
CONFLICT (content): Merge conflict in cni/pkg/nodeagent/informers.go
Auto-merging cni/pkg/iptables/iptables_linux.go
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 cni: properly cleanup completed pods
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

In response to a cherrypick label: new issue created for failed cherrypick: #51727


Implement retry handling for POST

Hey! I am hosting a new master branch for this repo since the old creator has been MIA for 2 years. Hoping to get this repo back on it's feet and in good standing. I've been working on updating tests and adding all new functionality and have merged some of the PR's that I thought were sufficient for merging. If your PR didn't make it, resubmit it please.
Thanks!
https://github.com/Harrison97/spotipy

Hi @hmhrex, thanks for that. 2 years later, the package is maintained again. I know that's a late answer but is there a chance you can give this a new look and fix conflicts?

Sure! I’ll take a look at this this week. Glad it’s back!

Thanks a lot @hmhrex for your earlier efforts on this but I'm going to close it in favor of https://github.com/plamere/spotipy/pull/457 which uses urllib3.Retry


Terraform 0.12 upgrade

Note, this depends on https://github.com/cloudposse/terraform-aws-ec2-autoscale-group/pull/14 getting merged and then making a change here to reference that new tag.
This does the upgrade and also copies the new arguments relevant to the underlying autoscale group. It also removes some arguments we no longer need because of the addition of "null" that allows us to remove the accompanying boolean values.
The example is not ported as some of the dependency modules aren't 0.12 compatible.

@osterman @goruha @aknysh - Is there anything I can do to help move this forward?


Get next available stream ID more efficiently

The itertools.count() not only satisfies stream ID requirement, but also runs faster, because it remembers current stream ID and is implemented by C.

I don't think this is correct, you are changing the state of the connection on every call, which is not the current behaviour?

Closing in favour of #403


🌱 Bump kind and kustomize and update bmh CRD

Apart from the bumps, this PR also removes the old way of installing kustomize which is through ensure-kustomize script. We have the kustomize dependency in hack/tools/go.mod and we can leverage that. This PR also updates the BMH crd in the example folder.

/test-centos-e2e-integration-main

/test-ubuntu-integration-main

There are some strange things in the make help output.
tools:
  controller                                          Build controller-gen from tools folder.
  golangci-                                           golangci-lint
  moc                                                 Build mockgen from tools folder.
  envs                                                Build envsubst from tools folder.
  kustomize                                           Build a local copy of kustomize.
  kusto                                               Build kustomize from tools folder.
  envsubst                                            Build envsubst from tools folder.

I would say that some of these should not be in the help text at all because they are not meant for humans, just as necessary dependencies. For example, there is no point having a help text for both kustomize and kusto. The first issue is kusto is not a real target so it is not possible to do make kusto, and the second is that it is unclear which to use. Therefore I would say we should hide kusto. Same goes for controller-gen, mockgen and envs (short for envsubst) and golangci-. It could be useful to have a proper target for golangci-lint though, but having one that does not work in the help text is not useful.

true, checking

Fixed.  Keeping envsubst binary to be built locally for the user.
tools:
  controller                                          Build controller-gen from tools folder.
  golangci-lint                                       Build a local copy of golangci-lint.
  kustomize                                           Build a local copy of kustomize.
  envsubst                                            Build envsubst from tools folder.

building clusterctl binary is broken since we do not have any dependancy for clusterctl in go.mod. Will fix it in a followup. https://github.com/metal3-io/cluster-api-provider-metal3/issues/1180

/test-ubuntu-integration-main
/test-centos-e2e-integration-main

/test-ubuntu-integration-main
/test-centos-e2e-integration-main

/lgtm


Inverse of Matrix

Enter calculator name -
Inverse of Matrix
Describe calculator functionality -
Take size of matrix as input from user then take matrix of the specified size as input and display the inverse of the matrix.
Note - Max. size of matrix = 6x6
Under which category you wish to place it -
[] Computer Science
[x] Mathematics
[] Physics
[] Chemistry
[] Biology
[] Finance
[] Scientific
[] Health
[] Sports
[] Let project owner decide
[] Suggest a new category - (Answer here)

@SarthakKeshari kindly assign me this issue

Hey @Sam-bhav-20,
Assigned.
You may contribute your code under the folder path -
calc_for_everything\calculators\src\mathematical\matrix_inverse
Issue completion deadline - 4th Oct 8:00:00 am IST
NOTE - In case the issue completion deadline is not met, the issue will be assigned to a new participant

Hey @Sam-bhav-20, Glad to see your eagerness 😃. Will definitely review shortly.