increase job name varchar size in Mysql table to allow long job name to survive

Users reported that log missed when composed job name was too long. The reason is that Mysql doesn't allow to insert a row when name's length is bigger than 128. This PR increases the length to allow long job name to survive.
Since we don't know the Mysql prefix settings for other Azkaban users, we try to meet the requirement for defaults. In Default Mysql settings, the index key prefix length limit is 767 bytes for innoDB. We have to assume most users using innoDB.
Besides, users should gain difference regarding character seeting in Mysql. In latin1, each character is exactly one byte long. In utf8, a character can consist of more than one byte, usually 3 or 4. At our settings, we use latin1, so we can hold 512 character long name for a job.
Because varchar size enhancement is not a necessary upgrade, schema update file is not recommended here. If people run into the issues above, they should follow the below instructions:

maximize the Mysql varchar size limit introduced in this PR.
change the Mysql character setting to latin1, if your Azkaban only stores English or other latin characters.

For h2 database, there is no limit for the following entities, except the memory and storage capacity: maximum identifier length (table name, column name, and so on)
http://www.h2database.com/html/advanced.html#limits_limitations

A bit less than year ago we bumped execution_jobs.flow_id up to VARCHAR(767) (just used the max size possible) for similar reason in our Azkaban DBs. It was indeed the flow_id column, so different than this PR. The problem occurred for us when there were many embedded sub-flows, as the flow_id was composed from all ancestor flow names.
I wonder, has the DB structure been changed since then, or why is it that this PR is not changing execution_jobs.flow_id but execution_jobs.job_id? Or is it maybe some other kind of case, not one with multiple nested flows with rather long names?

Hi @juhoautio , one of the major bottlenecks should be name in table execution_jobs, which is the combination and flow name and job name. I believe we should want to maximize name firstly.
Furthermore, the reason we don't change flow_id is AZ webserver being able to identity too-long-flow name when user uploads the project. Too-long flow_id exception is thrown immediately after we upload the project. I experimented embedded flow case, and same there. However, too long 'job_id' cannot be identified at the beginning. it is only thrown when job starts to run. Therefore, I believe we should pay more attention to en-long job_id firstly, to guarantee uploaded flows running smoothly.
I understand you use case here, because you are trying to maximize the embeded_flow length. Unless we enforce AZ users modify mysql server defaults and increase the innodb prefix length, Could you think of more possible solution? @juhoautio

This change has been in production for a while. We should merge this change.

@kunkun-tang

I experimented embedded flow case, and same there.
I don't think is true here. I have seen user reporting the issue for over long flow_id.

Yes, @chengren311 the error does not occur when the flow is uploaded. It's because for embedded flows the concatenated id (parent:flows:embedded_flow:job_id) is only used as a key to store jobs logs. That doesn't happen on project upload. This model is quite limiting though. The index key prefix length limit of 767 is easily too low when you have any deeper nesting of embedded flows.

@juhoautio totally agree.
So max allowed varchar length is 767? It seems surprisingly low to me. If this is true, how about TEXT?

@chengren311 it's not the limit of varchar type itself, it's because that column is used as an index key. The term is "index key prefix" and innodb (mysql) has a default limit of 767 for that.
I found this though:

The index key prefix length limit is 767 bytes if you use the version of MySQL either 5.5 or 5.6. Or you use 5.7 the default is 3076 bytes.

^ https://blog.codingecho.com/2017/08/09/index-key-prefix-length-limitation-on-innodb/
We're using AWS RDS as our MySql DB. Looks like 5.7 is available so we might consider upgrading. Not a perfect fix, but at least it would offer 4x longer values than currently.

@chengren311 We already simplified the flow-id composition format in flow 2.0 Project. The length of new flow-id has been optimized, if users us flow 2.0.

What about https://github.com/azkaban/azkaban/issues/1809, did it occur in your end in a flow that uses the traditional flow format? Would using flow 2.0 somehow prevent that from happening?

@juhoautio it happened when using traditional flow format.

Rewind my comment a bit, we recently got an error in writing to execution_logs, but that's no surprise as we still had name        VARCHAR(128) there. We had only bumped up execution_jobs.flow_id to VARCHAR(767).


Question: Access pubsub outside of resolvers

Hey, is there a way to access the pubsub from outside of any resolver function to publish events?

I couldn't find it but there is no reason why it's not exposed as a decorator in the fastify instance.
Would you like to send a Pull Request to address this issue? Remember to add unit tests.


Vaadin 24.3.0 breaks existing tests with exception in ErrorStateRenderer

Karibu tests are failing after upgrading to Vaadin 24.3.0 with the following exception:
...
c.v.f.r.internal.ErrorStateRenderer      : The same exception com.vaadin.flow.router.AccessDeniedException has been thrown several times during navigation. Can't use any HasErrorParameter view for this error.

com.vaadin.flow.router.internal.ErrorStateRenderer$ExceptionsTrace: Exceptions handled by HasErrorParameter views are :com.vaadin.flow.router.NotFoundException, com.vaadin.flow.router.AccessDeniedException
	at com.vaadin.flow.router.internal.ErrorStateRenderer.handle(ErrorStateRenderer.java:93) ~[flow-server-24.3.0.jar:24.3.0]
	at com.vaadin.flow.router.internal.AbstractNavigationStateRenderer.reroute(AbstractNavigationStateRenderer.java:722) ~[flow-server-24.3.0.jar:24.3.0]
	at com.vaadin.flow.router.internal.AbstractNavigationStateRenderer.handleTriggeredBeforeEvent(AbstractNavigationStateRenderer.java:680) ~[flow-server-24.3.0.jar:24.3.0]
	at com.vaadin.flow.component.internal.JavaScriptNavigationStateRenderer.handleTriggeredBeforeEvent(JavaScriptNavigationStateRenderer.java:99) ~[flow-server-24.3.0.jar:24.3.0]
	at com.vaadin.flow.router.internal.AbstractNavigationStateRenderer.sendBeforeEnterEvent(AbstractNavigationStateRenderer.java:603) ~[flow-server-24.3.0.jar:24.3.0]
	at com.vaadin.flow.router.internal.AbstractNavigationStateRenderer.sendBeforeEnterEvent(AbstractNavigationStateRenderer.java:578) ~[flow-server-24.3.0.jar:24.3.0]
	at com.vaadin.flow.router.internal.AbstractNavigationStateRenderer.createChainIfEmptyAndExecuteBeforeEnterNavigation(AbstractNavigationStateRenderer.java:454) ~[flow-server-24.3.0.jar:24.3.0]
	at com.vaadin.flow.router.internal.AbstractNavigationStateRenderer.handle(AbstractNavigationStateRenderer.java:211) ~[flow-server-24.3.0.jar:24.3.0]
	at com.vaadin.flow.component.internal.JavaScriptNavigationStateRenderer.handle(JavaScriptNavigationStateRenderer.java:78) ~[flow-server-24.3.0.jar:24.3.0]
	at com.vaadin.flow.component.UI.handleNavigation(UI.java:1853) ~[flow-server-24.3.0.jar:24.3.0]

Our application is using Spring Security and view based permissions and the Karibu tests try to verify proper role checks per view.
As it is quite complicated to explain and reproduce, I prepared a small sample app, which reproduces the problem.
There are 2 tests in the sample app:
 @Test
  fun `User with valid role is allowed to access page`() {
    integrationTestPreparer.withUserForRole( "valid", "USER")

    UI.getCurrent().navigate("/")
    // simulate a button click as if clicked by the user
    LocatorJ._click(LocatorJ._get(Button::class.java) { spec: SearchSpecJ<Button?> -> spec.withCaption("Say hello") })
  }

  @Test
  fun `User with invalid role is not allowed to access page`() {
    integrationTestPreparer.withUserForRole("invalid", "INVALID_ROLE")

    val ex = assertThrows<NotFoundException> {
      UI.getCurrent().navigate(HelloWorldView::class.java)
    }
    Assertions.assertThat(ex.message).contains("No route found for 'hello'")
  }

These tests run successful with Vaadin 24.2.6. With 24.3.0 the second test dealing with an AccessDenied error fails. You can reproduce by changing the version in the pom.xml
Here is the sample app:
vaadin24-karibu-sample-app.zip

Hi, I also noticed the following exception in karibu-tests themselves:
The same exception com.vaadin.flow.router.AccessDeniedException has been thrown several times during navigation. Can't use any HasErrorParameter view for this error.

I think this is the reason why the test fails, but unfortunately I haven't got the chance to analyze this change of Vaadin's behavior in depth.

Looks like the problem is as follows:

Navigation to a security-protected route fails. NavigationAccessControl:259 reroutes to exception AccessDeniedException
RouteAccessDeniedError picks up the call and redirects to NotFoundException at line 39
Vaadin tries to navigate to MockRouteNotFoundError, but it lacks any security annotations
NavigationAccessControl is triggered and prevents access to MockRouteNotFoundError by re-routing to AccessDeniedException
Vaadin decides it has seen enough and breaks the navigation cycle.

Let me write a test for this, and hopefully fix this issue by adding appropriate security annotations to MockRouteNotFoundError

Alrighty, I've fixed the issue by introducing MockRouteAccessDeniedError which simply throws the AccessDeniedException immediately instead of redirecting to RouteNotFoundError. I've also added @AnonymousAllowed to MockRouteAccessDeniedError and MockRouteNotFoundError, so hopefully this bug should be fixed.
Fixed in Karibu-Testing 2.1.7, will be released shortly.

Please use Karibu-Testing 2.1.8, there was a bug in 2.1.7.


Special package sometimes raises "Command not found"

Example: https://mssqltools.visualstudio.com/cli/_build/results?buildId=59875&view=ms.vss-test-web.build-test-results-tab&runId=1173214&resultId=100204&paneView=debug
Seems to fail on Linux frequently.

Should be fixed with recent PR #441


Support for apache poi 4

Hi, are there plans to support apache poi 4?
I'm not sure what changes that would entail

I would be also very interested in version with poi 4 supported.

I think there's been some API changes in 4.x, so best to change the pom and see what breaks...

I've added support for 4.1.0 and released v1.2 (https://github.com/tobyweston/simple-excel/releases). Have a go and report back.
I haven't tested it against projects I'm using so there well might be problems, things I will change. I'll do a 1.2.x release if required.
PS. are there specific features of 4.x that you want to use?


grpc_cli error reporting is insufficient

The command line tool, almost never prints useful info when an action fails. grpc_cli should be useful for debugging and it's very hard to debug a (possibly broken) service and not seeing useful error messages when things fail.

Some work has been done here https://github.com/grpc/grpc/pull/10432 but I believe there's more work.


Template validation may fail

Domain name for Azure cannot start with digits.
Current implementation might generate domain name started with a digit for ERT Public IP and cause the validation to fail.

@HDYA thanks! I was wondering how to solve that problem, haha.


[Question] Changing Library Path (Docker)

Hey!
I'm running Mango on Docker. So far, everything's been working fine, but the library_path set is /root/mango/library.
I'd like to change it to a folder in my Documents folder (C:/Users/XXX/Documents/Mango Library). However, I'm not sure how to do this. I've tried the following, all to no avail:

'/mnt/c/Users/XXX/Documents/Mango Library'
'/mnt/host/c/Users/XXX/Documents/Mango Library'
'/root/Documents/Mango Library'

Any idea on how to get this to work? Thanks!

Hi, thanks for reaching out!

I've tried the following, all to no avail

Could you be more specific? It's always easier for me to diagnose the issue if you could list the steps you did. E.g., Did you update the config file? Did you correctly map the volume to Docker?

Hi, thanks for reaching out!

I've tried the following, all to no avail

Could you be more specific? It's always easier for me to diagnose the issue if you could list the steps you did. E.g., Did you update the config file? Did you correctly map the volume to Docker?

Oops sorry, I meant to say that I've tried all of the following in the library_path variable of "config.yml":

'/mnt/c/Users/XXX/Documents/Mango Library'
'/mnt/host/c/Users/XXX/Documents/Mango Library'
'/root/Documents/Mango Library'

Mango reads it properly, but it says that the directories don't exist. It then tries to create the directory, and after I restart Mango the message goes away (presumably because it's made that directory somewhere... though I'm not sure where). Still, it doesn't point to the intended one because it says "Scanned 0 titles."
I'm not sure if I've correctly mapped the volume to Docker. I didn't do any Docker configurations besides the initial installation/default configurations. My wsl.conf file is this:
[automount] root =/mnt/host crossDistro = true options = "metadata"
If I open wsl I can successfully run cd "/mnt/host/c/Users/XXX/Documents/Mango Library"

Oops sorry, I meant to say that I've tried all of the following in the library_path variable of "config.yml":

'/mnt/c/Users/XXX/Documents/Mango Library'
'/mnt/host/c/Users/XXX/Documents/Mango Library'
'/root/Documents/Mango Library'

Mango reads it properly, but it says that the directories don't exist. It then tries to create the directory, and after I restart Mango the message goes away (presumably because it's made that directory somewhere... though I'm not sure where). Still, it doesn't point to the intended one because it says "Scanned 0 titles."
I'm not sure if I've correctly mapped the volume to Docker. I didn't do any Docker configurations besides the initial installation/default configurations. My wsl.conf file is this:
[automount] root =/mnt/host crossDistro = true options = "metadata"
If I open wsl I can successfully run cd "/mnt/host/c/Users/XXX/Documents/Mango Library"

It looks like you didn't map the volumes to Docker. The Docker container itself is an isolated environment, and you need to map some folders on your host machine to the container so Mango can read from them. Have you followed the Docker installation guide?

It looks like you didn't map the volumes to Docker. The Docker container itself is an isolated environment, and you need to map some folders on your host machine to the container so Mango can read from them. Have you followed the Docker installation guide?

If you are still struggling to configure Docker, you can try to use the pre-built binary instead. You can simply download it from the latest release and run it from your WSL.

If you are still struggling to configure Docker, you can try to use the pre-built binary instead. You can simply download it from the latest release and run it from your WSL.

Got it to work! I just edited the volumes part of the docker-compose.yml file.
Quick question: I've been looking everywhere but I can't seem to figure out how (and if it's recommended) to move the .docker, mango, and .config folders (basically all the Docker-related folders that are currently in C:/Users/XXX/) to my Documents folder instead of the default location (C:/Users/XXX/). Do you by any chance know how to do this (currently using Docker Desktop, if that makes a difference).

Got it to work! I just edited the volumes part of the docker-compose.yml file.
Quick question: I've been looking everywhere but I can't seem to figure out how (and if it's recommended) to move the .docker, mango, and .config folders (basically all the Docker-related folders that are currently in C:/Users/XXX/) to my Documents folder instead of the default location (C:/Users/XXX/). Do you by any chance know how to do this (currently using Docker Desktop, if that makes a difference).

Glad to know that it's working :D
I don't use Windows at all so I can't really help with that. Perhaps you can ask the folks at r/docker?

Glad to know that it's working :D
I don't use Windows at all so I can't really help with that. Perhaps you can ask the folks at r/docker?

Ah I see. Thanks! :)

Ah I see. Thanks! :)


Characters and other graphics don't load/.

During cutscenes, characters don't load properly. Additionally, chests aren't visible in the world and can only be identified if they are locked inside of a red ring.

i have the same problem
Tested everysingle available wine version and the whisky wine, none of them work even with character resolution turned off

Disable Dynamic Character Resolution.


Investigate long running z3 commands.

While working on https://github.com/kframework/evm-semantics/pull/1102, I noticed that the proof takes roughly an hour to complete. About 40 minutes are spent waiting for z3.
So far, I've recorded an smt-transcript, which shows some suspicious behavior. There are multiple very long sequences of declare-fun-commands followed by one assert-command with one giant conjunction over all the declared symbols:
(declare-fun <0> () Bool )
; success

(declare-fun <1> () Bool )
; success

; ...

(declare-fun <9183> () Bool )
; success

(assert (and <2> (and <3> (and <4> .... (and <9182> <9183>) ... ))))

I am not sure if this is where z3 spends most of its 40 minutes, but it seems suspicious.
z3.1.log

Where we translate patterns for the SMT: https://github.com/kframework/kore/blob/master/kore/src/Kore/Rewrite/SMT/Translate.hs
Where we send predicates to the SMT: https://github.com/kframework/kore/blob/29888c3567cb479fc5ee619f21a1545d88ff66e8/kore/src/Kore/Rewrite/SMT/Evaluator.hs#L179

We should still investigate what's happening here, but if a workaround in KEVM was found then we can postpone this for a bit.

I have implemented a workaround for the issue where I first observed this behavior. However, the simplification rules that I added there are rather specific to the test case under question. I don't expect a general speed-up from these rules.
Meanwhile, @calin1304 took a look at tests/specs/mcd/vat-move-diff-rough-spec.k and also found it very slow. A quick look at the smt-transcript revealed the same pattern I described above, so this is likely related. This test case might be a better start for investigation because the behavior can be reproduced in the first proof-step.
We also found that we are both working on a Windows+WSL2. It would be interesting to know if this issue can be reproduced on a native Linux or Mac, just to rule out the possibility that this is some weird WSL behavior.

I was able to reproduce the issue on native Linux. I just installed a fresh copy of Ubuntu 21.04 and tried to run tests/specs/mcd/vat-move-diff-rough-spec.k and had the same issue as when running it on WSL2. After about 3-5 minutes memory usage increases past 20+ GB, taking up all my memory.

This performance issue is solved as a side-effect of https://github.com/kframework/kore/pull/2800.
The problem was that we were sending big concrete sets to the solver.
https://github.com/kframework/kore/pull/2814 adds the original evm-test, which revealed this behavior to our regression-test-suite.


loadMultiple for new rows

I'm having a problem with
Model::loadMultiple($items, Yii::$app->request->post())
assume I am posting 5 new items here (5 rows in view). I create 5 new models ($items).
If my POST is not indexed from zero (0,1,2,3,4), loadMultiple does not load the items properly to $items.

this is expected, you have to create models so that their keys match what is in POST. This is to ensure correct changes on update where you could index the input fields using the models primary key.

thanks.


Syntax highlighting for sudo apt as in image not working

I am on ubuntu 16.04 and I am using gnome-terminal. The default shell for gnome-terminal is bash and i have read somewhere that does not support syntax highlight like in the image. Am I wrong or are you using another shell? I am able to get the background colors and text colors to work but the syntax highlighting does not work.
Any help is appreciated.
Thank you.

As I said I was able to get the background color and text color to work.

But I cant get the colors like in the image for the sudo apt-get. My sudo apt-get is all a single color.

ZSH
That is not part of Gogh
I got this script for that, is for personal use, i don't have support and you have to used by your own responsibility :)
https://github.com/Mayccoll/Linux-Utils#install-zsh
The script install:

Zsh
Oh-My-ZSH
Insert .zshrc
Zsh Syntax highlighting
ZSH Git Prompt
Awesome Tterminal Fonts

Thank you so much! Makes a lot of sense right now.


VISA Shell

I have ported the lantz shell to PyVISA. Now you are able to talk to instruments via a shell. Currently supported commands are:

list (list resources)
open (open a resource and make it active)
read/write/query (equivalent operations)
attr (get an attribute, set not implemented yet)
close (close the active resource)

This is quite complete right now. We will open again when a new feature is requested.


[5.8] Pass along the duration in the remaining WorkspaceDelegate calls as well (#6020)

This is the 5.8 nomination of https://github.com/apple/swift-package-manager/pull/6020.

@swift-ci please smoke test


Cool project with a big thorn

I noticed we've both been following Pak on twitter (http://twitter.com/muratpak) therefore I believe you already know he's one of the lead motion designers worldwide. Your project is very cool but the similarity and use of the circle as the main icon is extremely bothering in this sense. Specially when it's considered that your project is also a motion design library for the web.
You should consider this issue.
http://i.imgur.com/TrOouDr.png
http://i.imgur.com/hBJN0le.png
http://i.imgur.com/eTcYu99.jpg

@mzgnr DM'ed to Murat in twitter


#237 Reduce docker reliance in PIXL-EHR

remove unused rabbitmq instance
add docker compose to testconf
remove run_tests scripts

Still needs docker because it tests whether something has happened in the DB. Not familiar enough with things to tell if that's necessary.
But if it is, perhaps we could make a dev container with all the test infrastructure running locally? That way we don't have to keep turning docker on and off for each test run

Nice! I think regarding the PIXL postgres database, we might also be able to mock it instead of having to spin up the Docker service? But not sure how feasible that is.
I definitely like the idea of a devcontainer.

Nice! I think regarding the PIXL postgres database, we might also be able to mock it instead of having to spin up the Docker service? But not sure how feasible that is. I definitely like the idea of a devcontainer.

I would generally agree about mocking when it comes to unit tests. However, these are not unit tests - they are functional black box tests, in the sense that they don't test specific pieces of logic but rather the function of the module (and really its communication with other modules, dbs, etc).


Extract text from a PDF

Script Title -  Extract text from a PDF
Brief -
Python can also be used to easily extract text from PDFs using the PyPDF2 package. Getting text from a PDF file proves useful for data mining, invoice reconciliation, or report generation, and the extraction process can be automated in just a few lines of code.
Instructions

Create a new folder for your script and file/folder name should be appropriate.
Create a README.md (using this template) in your folder for program Instructions
add requirements.txt if needed

Programming Language

[x] Python

Happy Coding

I would like to work in this issue. Couldnyou assign It to me.
Thanks

Can you please assign this issue to me? Thank you

Closed by #900


FIX to correct DNS server for ipv4 and ipv6 clusters

jetlag generates the local inventory file for the installation and, based on the address family, it selects the correct DNS server. However, I was seeing without this change a wrong DNS in my inventory. This change corrects that.

@sferlin is this still being actively worked on or can we close it?

Not active. Let's close it.


Add info about dataset items in Actor QA

We should add to https://docs.apify.com/platform/actors/publishing/test that the Actor needs to finish with non-empty dataset.
https://apifier.slack.com/archives/C03HYGWF42J/p1701184464432029

https://github.com/apify/apify-docs/pull/990


[FEATURE] Provide a way to customize the URL and title of home page's avatar

Describe the feature you want 描述你的功能需求
It seems that currently DoIt automatically navigates to the first menu when clicked on the avatar. It would be better if the URL of it can be specified through the config.
Also, the title should be customizable too.
Useful reference 有价值的参考
No response

I always thought it was a feature.


Search 3D Touch should pop root and focus search bar

Bug Report Dump (Auto-generated)

Version 1.11.0 (2013)
Device: iPhone 7 Plus (iOS 11.0.3)
TestFlight: false

Should also be clearer: what search? Add a subtitle “search repositories” or something similar
Sent with GitHawk

Does focus search bar now, but does not pop any modals (eg. new issue). Not sure if that's expected? If so, this can be closed.


Ts ERROR

Windows7

Plugin version:
Platform: iOS or Android
OS version:
Device manufacturer and model:
Cordova version (cordova -v):6.5.0
Cordova platform version (cordova platform ls):
Plugin configuration options:
Link to your project:

Context
when i'm testing that exemple https://www.joshmorony.com/adding-background-geolocation-to-an-ionic-2-application/ ,  i get this error in my  provider.ts : [ts] Cannot find name 'Geoposition'.
Expected Behavior

Actual Behavior

Possible Fix

Steps to Reproduce

Context

Debug logs

Hi @rambey ,
As stated in the frontpage: "As version 2.0 platform support for Windows Phone 8 was removed."
There're no windows sources on latest version of this plugin.
The ionic documentation is outdated: http://ionicframework.com/docs/v2/native/background-geolocation/

Notice: this issue has been closed because it has been reported for branch 2.x or later. It could be already fixed in v3 or is not relevant anymore. You may reopen this issue if it has been closed in error.


Concurrent restorer.

for #1370. Pass by some tests for actual data snapshots recovery.
add a command line option -c/--worker to set concurrent workers number.

Codecov Report

Merging #1371 into master will decrease coverage by 6.14%.
The diff coverage is 2.07%.

@@            Coverage Diff             @@
##           master    #1371      +/-   ##
==========================================
- Coverage   51.07%   44.92%   -6.15%
==========================================
  Files         136      137       +1
  Lines       13524    13761     +237
==========================================
- Hits         6907     6182     -725
- Misses       5767     6784    +1017
+ Partials      850      795      -55

Impacted Files
Coverage Δ

internal/restic/restorer2.go
0% <0%> (ø)

cmd/restic/cmd_restore.go
60.39% <55.55%> (-2.11%)
:arrow_down:

internal/backend/b2/b2.go
0% <0%> (-79.93%)
:arrow_down:

internal/backend/azure/azure.go
0% <0%> (-75.35%)
:arrow_down:

internal/backend/swift/swift.go
0% <0%> (-73.9%)
:arrow_down:

internal/backend/gs/gs.go
0% <0%> (-65.6%)
:arrow_down:

internal/backend/swift/config.go
34.37% <0%> (-56.25%)
:arrow_down:

cmd/restic/lock.go
52.5% <0%> (-7.5%)
:arrow_down:

internal/backend/test/tests.go
59.38% <0%> (+0.57%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7507a65...25ecb06. Read the comment docs.

I'm closing this for now, it is superseded by #1719. Thanks!


Specify username and avatar in webhook

Describe the feature you'd like
I love webhook feature, glad to see it implemented here!
I use webhooks to notify a channel called #notifications on my RC instance. That channel was created at first to notify our Taiga system to all our IT department.
I have several services pointing to that channel: taiga, monit,  rundeck, custom webhooks, and now Bookstack! But I saw a little issue, the user who appear to had sent the notification is the default user for that channel, instead of an alias for Bookstack, as I have in other services.
Ideally, shoud appear as Taiga does, showing its own avatar and name, so a human can easily visually track the message among all the notifications

Monit send the default avatar, but at least can specify the alias

But Bookstack send no avatar nor alias for username, so it appears as the default user

Describe the benefits this would bring to existing BookStack users
Showing its own avatar and username alias, can help humans to visually detect which notifications are important for them the most without reading all of them, specially when there are many of them
Can the goal of this request already be achieved via other means?
We could create a separate chat room just for Bookstack notifications, and perhaps there are people that prefer to have one room per service, but there are others who will prefer to have all notifications in one room, for instance all system-related notifications from all services being received in one room, all new entries from kanban, bookstack, gitlab issues, etc on another room, etc.
Have you searched for an existing open/closed issue?

[X] I have searched for existing issues and none cover my fundemental request

How long have you been using BookStack?
1 to 5 years
Additional context
No response

Hi @adocampo,
With webhooks I'm not really looking to officially support functionality or integrations of any specific chat applications, outside of providing the text property as a convenience helper since it's used by many messaging services, since it massively widens our scope of support.
I'd think this is generally something that can be achieved from the side of the chat application.
Are you not able to add an additional "Incoming Webhook" integration, specifically for BookStack, with a custom name and avatar configured, then point that into the same channel?

Thank you for your answer @ssddanbrown
Yes, it can do the trick, indeed. I didn't get to set the avatar yet, but at least it shows the alias "Bookstack", so it will do just fine for me.
Thank you very much!


Use correct fonts

The generated og images do not use the fonts from vocabulary: Source Sans Pro and Roboto Condensed. They should be added and there should be a dropdown to choose which one is in use.
One difficulty here—the Puppeteer instance that generates the images doesn't have network access, so the woff2 files for the fonts will need to be added locally, encoded as base64, and inlined in the css.
You can see how to do this here: https://github.com/vercel/og-image/blob/8026662bccaa750af4783a00e87614d1676b3b2e/api/_lib/template.ts#L10

Hey @zackkrida, I would like to work on this one.

@rajat2502 go for it!


[Statie] Pages array

I am looking for a pages array as a variable in the statie code.
In other applications i use jekyll. There I use to use/abuse the array of pages to set up my menu dynamically. But cannot find an array of pages in my generated content. Can i find out where my pages are or is the status right now just to know the urls for the pages and manually set up my menu.
Is there a way or plan to generate this array of pages, and assign them to the templates.
https://jekyllrb.com/docs/pages/

Hi,
what exactly belongs to pages?
Basically all files are generated here:
https://github.com/Symplify/Symplify/blob/cafb506e75f0feff4e7cac4e0812e371279151b1/packages/Statie/src/Application/StatieApplication.php#L104-L123

Pages is all the pages found outside _posts, or any of the generated lists from a type. So just the main files.
$files = $this->renderableFilesProcessor->processFileInfos($fileInfos);

If i debug, the $files variable. Then i can use those to loop through and create a menu. As example;
{% for my_page in site.pages %}
    {% if my_page.title %} {# or in_menu or ... #}
        ... menu link
    {% endif %}
{% endfor %}

I see. That should be pretty easy.
To add $files variables to all $files, just implement FileDecoratorInterface, with low priority to run it last.
Use addConfiguration method.
From top of my head something like:
final class AddFilesVariableFileDecorator implements FileDecoratorInterface
{
    public function decorateFiles(array $files)
    {
        foreach ($files as $file) {
            $file->addConfiguration(['files' => $files]);
        }

        // ...
    }
}

Any update?

No, tried the decorator and i think i got it setup and running. But it did not update my local variables in my files sadly.
And as for i have no test case online and no example i will close this one for now. If i have more info when i have a case to use your site generator i will retry. So for now i'll just close this one. Thanks for the help for now.


Scroll-bar doesn't work with touchpad gestures while selecting custom themes

Description
The scrollbar doesn't seem to work when scrolling down using the touchpad gesture,
The indication of the scrollbar and the themes tab are not in sync, which is making selecting themes below the visible region impossible.
While there is a fix for this by clicking and dragging the scroll bar once, the touchpad gesture doesn't work without doing this.
https://user-images.githubusercontent.com/74333675/142752580-d9fe92b4-f49a-4c85-ac3b-0ddc29a7e39d.mp4
Steps To Reproduce
1.Go to settings.
2.Click on the drop-down icon on custom themes (assuming some custom themes have been added).
3.Use the touchpad gesture to scroll downwards/upwards to select your desired theme. (Doesn't work)
4.Click and drag the scrollbar thumb.  (Works)
5.Try the touchpad gesture again. (works now)
Expected behavior
The custom theme-tab is expected to move in sync with the scrollbar-thumb while using the touchpad gesture.
Files Version
Version: <IP_ADDRESS> OS Version: 10.0.22000.318
Windows Version
Windows 11 Pro Version	21H2 22000.318
Relevant Assets
https://user-images.githubusercontent.com/74333675/142752580-d9fe92b4-f49a-4c85-ac3b-0ddc29a7e39d.mp4

This issue should be resolved in main.


Magic Bytes are part of checksum

Because checksum is calculated last, magic_bytes contribute to the value; some rewriting required.

After some research, it seems its normal for the magic bytes to affect the checksum value.


Hello, is there a download link for the pretraining model of ResNet101_ibn_a？？

Please refer to https://github.com/XingangPan/IBN-Net.


fix: remove memory leak in library_sort.cpp

Description of Change

There is a memory leak in library_sort.cpp. This PR fixes it.
Checklist

[x] Added description of change
[x] PR title follows semantic commit guidelines
[x] Search previous suggestions before making a new one, as yours may be a duplicate.
[x] I acknowledge that all my contributions will be made under the project's license.

Notes:
Remove memory leak in library_sort.cpp.

@Panquesito7, @realstealthninja: please dont-close.


error for label when to coco

I have downloaded the latest conversion program and also downloaded the labels of detection2020 from the website. An error in the picture occurred during the conversion process. I'd be most grateful if you could tell me what to do.

Thanks for other's questions. I already know how to solve them
There are two common errors: if it ends with "for label" : just pip install pydantic==1.10.12.
If it ends with "for frame" : you need to download a new annotation from the official website.

Thanks for other's questions. I already know how to solve them There are two common errors: if it ends with "for label" : just pip install pydantic==1.10.12. If it ends with "for frame" : you need to download a new annotation from the official website.

In fact, https://github.com/scalabel/scalabel/issues/515#issuecomment-1583160523 shows that all you need to do is downgrade pydantic first :)

你的信件我已收到啦！拜托你等一会儿好不好？就一个月之内！我尽力赶紧回复！ok？


Deleting Invited Users

I have a list of all invited users where an admin can delete any invited user regardless of having accepted the invitation or not. When I try to delete one it halts at the :resource_from_invitation_token method. I've tried solutions by scambra for similar issues with no luck ( #476  #371 setting cofig) among almost all other related issues I could find. Additionally I found when the token expiration date is uncommented and set to 1+ weeks and I try to delete it, the token will be the same but when it's commented out and set to never expire by default the tokens will be different.
Started GET "/users/invitation/remove?invitation_token=**1f3800357d4ddf6e62b15ed6649f20ea854aedbe1262cb16504d2eab573fadc4**" for <IP_ADDRESS> at 2017-09-20 05:22:53 +0000
web_1          | Cannot render console from <IP_ADDRESS>! Allowed networks: <IP_ADDRESS>, ::1, <IP_ADDRESS>/<IP_ADDRESS>
web_1          | Processing by Users::InvitationsController#destroy as HTML
web_1          |   Parameters: {"invitation_token"=>"**1f3800357d4ddf6e62b15ed6649f20ea854aedbe1262cb16504d2eab573fadc4**"}
web_1          |   User Load (0.8ms)  SELECT  "users".* FROM "users" WHERE "users"."invitation_token" = $1 ORDER BY "users"."id" ASC LIMIT $2  [["invitation_token", "a1e658cf3fb2713438985e54b8ab395b7d2fd71ff8c619a9189cd35c854ec782"], ["LIMIT", 1]]
web_1          | Redirected to http://localhost:3000/
web_1          | Filter chain halted as :resource_from_invitation_token rendered or redirected
web_1          | Completed 302 Found in 180ms (ActiveRecord: 0.8ms)

invitations_controller.rb
class Users::InvitationsController < Devise::InvitationsController
  before_action :update_sanitized_params, only: :update
  before_action :fetch_invited_users, only: [:new]

  def new
    super
  end

  def update
    respond_to do |format|
      format.js do
        invitation_token = Devise.token_generator.digest(resource_class, :invitation_token, update_resource_params[:invitation_token])
        self.resource = resource_class.where(invitation_token: invitation_token).first
        resource.skip_password = true
        resource.update_attributes update_resource_params.except(:invitation_token)
      end
      format.html do
        super
      end
    end
  end

  protected

  def update_sanitized_params
    devise_parameter_sanitizer.permit(:accept_invitation, keys: [:password, :password_confirmation, :invitation_token, :name, :phone, :country, :state, :state, :address])
  end

  private

  def fetch_invited_users
    @invited_users = User.where.not(invitation_sent_at: nil)
  end
end

invitations/new.html.erb
<%= link_to "Delete", remove_user_invitation_path(invitation_token: invited.invitation_token) %>
Any and all help is appreciated, I'm kind of lost

invitation token are not saved to db, a hash of the token is saved, so raw token is only in email, so for admin deleting invited users, it would be better to use different controller and remove with id.
That destroy action was added in pull request #294, is not useful unless remove link is added to email or accept invitation view.

Ah thanks!


Post API: Excerpts

I think it would be very useful to be able to request a plaintext excerpt via the public api. The main use case for this would be adding the first little bit of a blog post to a site for people to click through to the full post.
ghost.url.api('posts', {
   limit: 3,
   excerpt: {
      type: "words",
      limit: 50
   }
});

@karai17 if you're using the API it's assumed that the API consumer is completely responsible for any display logic, we won't be adding any display-specific parameters to API endpoints


kube: switch job api namespace to batch/v1 (k8s 1.6)

This is for compatibility with kubernetes 1.6; otherwise things don't deploy.  For some reason we don't need to touch deployment for now.

Outdated PR, we can redo the work when we actually take this.


Vue

Will it work with .vue files ?
Are there any examples for such a setup ?

Hi,
It's a good question. With the latest Vue package on NPM there is a problem, the « advertised » package for module is  "module": "dist/vue.runtime.esm.js".
It's great (if you don't need templating), but in the dist folder there is an another ESM build (i think the one we want) vue.esm.browser.js.
With the current version of @pikapkg/Web we can't choose which ESM build to pick, so for me, with VueJS we are stuck with the current packaging configuration.
PS: I know its not directly related to pikapkg…

@c4software you actually can point to the correct file, if you'd like. See here, where I got around a package that only exported a "main" entrypoint: https://github.com/lukejacksonn/perflink/issues/15/#issuecomment-480684060
As for .vue dependencies, they probably won't install correctly. @pika/web doesn't do any transpilation, and my understanding is that .vue files are not valid JS.
If you mean that you are writing .vue source files, then @pika/web dependencies should work like any other import! If someone can get an example together, I'd love to add it to the EXAMPLES list!

@FredKSchott Perfect ! I had to upgrade Pikapkg/Web but its working flawlessly 👍

And for the .vue file's are indeed not valid JS, those file are transpiled when bundling, but (i didn't try it) there is a http-vue-loader package on github : https://github.com/FranckFreiburger/http-vue-loader

Thanks for great responses. So, in theory, .vue source files should work if transpiled when bundling it or we could use the http-vue-loader ?
It would be great if someone could provide a full example.

@c4software Huh, marked & prism are interesting. I actually never realized that AMD packages would be supported by default just due to the fact that they don't have any imports to worry about.  🧐
+1 for a full example. No one really publishes .vue files to npm (correct me if I'm wrong on this!) so then the only concern is just getting an example of a .vue source file being compiled correctly using @pika/web dependencies.

@FredKSchott Its working really well Sample
I will do a quick sample using http-vue-loader this week-end.

@FredKSchott Its working really well Sample
I will do a quick sample using http-vue-loader this week-end.

@c4software Thanks for your example, but i got a mistake when follow your webDependencise config. It's always show
Error: Cannot find module '/app/node_modules/vue/dist/vue.esm.browser.js/package.json'
Have you seen this problem before?
Here is the sample code https://glitch.com/~vue-esm-test

@orozot try updating @pika/web, you may be using an older version

@orozot Yep update @pika/web and the problem will be fixed

Hi,
should vue imports be working at this stage? I've set up snowpack using
const commonjs = require('rollup-plugin-commonjs');
const vue = require('rollup-plugin-vue');

module.exports = {
  rollup: {
    plugins: [
      commonjs(),
      vue(),
    ],
  },
};

Serving it with npx servor --reload I get
Loading module from “http://localhost:8080/src/App.vue” was blocked because of a disallowed MIME type (“application/octet-stream”). when loading the app.

Right now Snowpack is a purely dependency-transforming tool, so you'll need to to set up Babel or another source tool for source code transformation.
Snowpack v2 will have a built in dev server for you to use as well. You can track progress in #270.
You should also check out Vite: the unbundled dev server from the Vue team that works with Snowpack: https://github.com/vuejs/vite


Suspend event deliveries until middlewares are ready

This patch fixes an important issue where a middleware will miss out
on augmenting any events while it's being prepped up. Those events will
reach all the destinations as-is without going through the said
middleware.
With this patch, addSourceMiddleware is now classified as a critical
task, and thus will seize all event deliveries until all middlewares are
in an operational state. The suspension happens internally, and the
public API is unaffected.

Changeset needed (either patch or minor)

On Jul 20, 2022, at 6:38 PM, changeset-bot[bot] @.***> wrote:

⚠ No Changeset found
Latest commit: cac3e8b
Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.
This PR includes no changesets
Click here to learn what changesets are, and how to add one.
Click here if you're a maintainer who wants to add a changeset to this PR
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because your review was requested.


Changes to PTR-RBL

made the following changes to PTR-RBL:
%% Pg. 27, PTR-RBL
b= dec2bin(2^4-1:-1:0)-'0';
b1=b(:,1);b2=b(:,2);b3=b(:,3);ba=b(:,4);
LHS = min(reshape(-1*b1.*b2.*b3,2,[]));
RHS = min(reshape((ba -b1 + b2 + b3) - b1.*b2 - b1.*b3 + b1,2,[]));
isequal(LHS,RHS)

Thanks for this contribution @RohitPrashant! Are you also able to verify the quadratization given in the summary section? The z  variables are described on page 1 of the book, and due to the ternary variable, the RHS will have a size of 2x2x2x3=24 before the reshape whereas the LHS will have a size of 2x2x2=8. I'm CCing @Jdewbury, the other student that selected a quadratization involving z variables.

@ndattani Hi, I just got done with my last final, I will try to verify the quadratization and finish it by tonight. Sorry about the asv file I'm not really sure how that happened but I will make sure it does not happen again.

@ndattani Hi, I have added the verification for the z-variable part, but I was not entirely sure how to work with the qutrit variable.

Your LHS1 does not equal RHS1, in fact they don't even have the same size. You'll need to use the kron function a lot more.

@ndattani Is it possible to give me more of a hint on how to fix this?

@RohitPrashant I'm a bit confused. I've never seen that message before! Basically you permanently deleted your fork of this repository, and that's why the pull request got closed?

Hi Nike,
I was trying to add projects to my git and I accidentally deleted all the
repos in my account. I’m sorry for the inconvenience.
I also wanted to ask if you are hiring for winter 2023 coop?
On Mon, Nov 21, 2022 at 3:26 AM Nike Dattani @.***>
wrote:

@RohitPrashant https://github.com/RohitPrashant I'm a bit confused.
I've never seen that message before! Basically you permanently deleted your
fork of this repository, and that's why the pull request got closed?
—
Reply to this email directly, view it on GitHub
https://github.com/HPQC-LABS/Book_About_Quadratization/pull/57#issuecomment-1321682286,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYCXTQQCRQUC6SXJFYROMODWJMW5BANCNFSM5UER35TA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hi Rohit,
I see two public repositories on your profile: Truecaller-api (with the last commit being in September) and Parkinsons-detector (with the last commit being yesterday). It looks like not all of your repositories were permanently deleted.
I'm curious how it got deleted? Deleting a repository on GitHub requires a fairly rigorous confirmation check, in which the user is given a stern warning and is asked to type in something precisely with no errors.
As for co-op positions, a more appropriate place to ask would be via email or the HPQC Discord group (in the "General" channel). Please avoid adding any  attachments. If you need me to see a file, you can upload it somewhere and send me the link.

Hi Nike,
I was attempting to add my new project to my GitHub and did not realize
that deleting the repo from my git would cause all these issues. My
truecaller API project still does not show on my GitHub for whatever
reason. I am incredibly sorry for the inconvenience.
Thank you,
Rohit
On Tue, 22 Nov 2022 at 05:34, Nike Dattani @.***> wrote:

Hi Rohit,
I see two public repositories on your profile: Truecaller-api (with the
last commit being in September) and Parkinsons-detector (with the last
commit being yesterday). It looks like not all of your repositories were
permanently deleted.
I'm curious how it got deleted? Deleting a repository on GitHub requires a
fairly rigorous confirmation check, in which the user is given a stern
warning and is asked to type in something precisely with no errors.
As for co-op positions, a more appropriate place to ask would be via email
or the HPQC Discord group (in the "General" channel). Please avoid adding
any attachments. If you need me to see a file, you can upload it somewhere
and send me the link.
—
Reply to this email directly, view it on GitHub
https://github.com/HPQC-LABS/Book_About_Quadratization/pull/57#issuecomment-1323453167,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYCXTQXMXZCJRU2WASDXHBTWJSOSZANCNFSM5UER35TA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hi Rohit,
Adding a new project, and deleting an old one, are two different things. One might argue that these two things are exact opposites of each other! How is it that adding a new project, deleted an old project? I understand that you didn't realize what deleting the repo would do, and I appreciate your apology for the inconvenience, but I just don't understand how adding a new repository can be used as the explanation for why a repository was deleted.
As for the truecaller API project, I seem to be able to see it perfectly as a publicly available repository here: https://github.com/RohitPrashant/Truecaller-api. I see that the last commit was made by you in September. I seem to be able to see everything about that repository.


NTLM without random Session Key

Hi there,
I'm implementing "NTLM over HTTP" and use sspi-rs to process the NTLM Messages.
https://learn.microsoft.com/en-us/openspecs/windows_protocols/ms-ntht/f09cf6e1-529e-403b-a8a5-7368ee096a6a
I'm using actix-web as web-server and chrome as NTLM client.
So far the process works until a valid looking NTLMSSP_AUTH Message (Step 3), but it does not have a "Random Session Key" inside.
I've traced the messages, and it seems, that Chrome sends the NEGOTIATE message without "Negotiate Key Exchange" Flag, so it is valid, that the Random Session Key is not inside the AUTHENTICATE message.
However, the "read_header" function of authenticate.rs stumbles over this line:
https://github.com/Devolutions/sspi-rs/blob/master/src/ntlm/messages/server/authenticate.rs#L96
if encrypted_random_session_key.buffer.len() != ENCRYPTED_RANDOM_SESSION_KEY_SIZE {
    return Err(crate::Error::new(
        crate::ErrorKind::InvalidToken,
        String::from("Invalid encrypted random session key"),
    ));
}

hence
pub const ENCRYPTED_RANDOM_SESSION_KEY_SIZE: usize = 16;

I'm wondering:

should the session-key size or the check at all here depend on the NEGOTIATE_KEY_EXCHANGE flag?
https://learn.microsoft.com/en-us/openspecs/windows_protocols/ms-nlmp/f9e6fbc4-a953-4f24-b229-ccdcc213b9ec

If NTLMSSP_NEGOTIATE_KEY_EXCH is set, the server MUST check if client supplied a valid EncryptedRandomSessionKey in the AUTHENTICATE_MESSAGE (section <IP_ADDRESS>); otherwise, the server MUST return SEC_E_INVALID_TOKEN.

do I have to modify the AUTHENTICATE message before passing it to ntlm, like calculating the encrypted session key manually on serverside?

Any comments are highly appreciated :)

Hi!

should the session-key size or the check at all here depend on the NEGOTIATE_KEY_EXCHANGE flag?

Yes, Ideally It should depend on in. The client side of the NTLM protocol checks if the flag is set or not(https://github.com/Devolutions/sspi-rs/blob/master/src/ntlm/messages/client/authenticate.rs#L186). But the server side doesn't, always require it and tries to read from incoming messages.

do I have to modify the AUTHENTICATE message before passing it to ntlm, like calculating the encrypted session key manually on serverside?

There are two options:

Set the session key in the AUTHENTICATE  message(workaround).
You are welcome to contribute. PRs are highly appreciated.
I think I will require to add an extra if in the read_header and add some extract checks in the encrypt message and decrypt message functions in case there is no session key, and modify https://github.com/Devolutions/sspi-rs/blob/master/src/ntlm/messages/server/authenticate.rs#L144 function as encrypted_random_session_key won't be present every time.

Okay, thanks! I will try to provide a PR.
I also tried your workaround, but its to difficult to manipulate the message but beside its really a pain to perform the bit operations correctly, it breaks on mic-check:
Error { error_type: MessageAltered, description: "Message Integrity Check (MIC) verification failed!" }
What to you think about typing the encrypted_random_session_key as an
Option<MessageFields>

and see where it takes us?

What to you think about typing the encrypted_random_session_key as an
Option<MessageFields>
and see where it takes us?

Sounds good


 The OAS 3.0.3 was released in Feb, see here.
The changes were made to improve the specification in terms of readability and accuracy. None of these modifications change the behavior of the spec and I take it that NSwag isn't affected and fully supports 3.0.3.
It will be nice to get confirmation from others, preferably the author himself.

Yes, i would agree that from the view of NSwag this should not have any impact, i.e. the spec generator might not use all new features but the generated spec is still valid - but some new features might not be supported by the client generator... (but it seems 3.0.3 does not add new spec features, just updated docs).

I don't think it generates a 3.0.3 document yet though (still 3.0.0 I believe due to this) so any PRs that use new semantics causing it to generate a document not compatible with 3.0.0 would require that field to be updated right? I don't think there are any such changes in the spec until OAS 3.1 comes out (it is currently rc0)

Thank you Rico and barry for the quick response.

I don't think it generates a 3.0.3 document yet though

That's correct - in theory we could just change that to 3.0.3, but i dont think this makes a difference, no?

For 3.0.0 I think it is probably wrong to update that field to 3.0.3 unless it was required due to some commit that made the document no longer compatible with 3.0.0 (assuming any defects that cause an invalid document per spec are not applicable).
For 3.1 reading the RC spec I think it would be appropriate to use whichever is the current one being adhered to because they state they are no longer following semver. I think that is hard to say though without knowing what is in 3.1.1 or later because 3.1 might just be not following semver due to the breaking changes from 3.0 and not due to plans to have breaking changes in point releases.


LUCENE-10287: Fix startup script of module enabled Luke to pass jdk.unsupported as module

See https://issues.apache.org/jira/browse/LUCENE-10287

If there's anything missing (like another startup script mention in some other readme file), please tell me. I lost overview!

I grepped for lucene.luke and module-path and these are the only relevant hits.


chore: update coffescript

update coffeescript to 2.2.1 from 1.10.0

@jluria have you had a chance to look at this again?

I don't think we should update until we drop support for node v4 and v5. Thanks for the contribution, we can revisit this in the future!


union results in 2 datasets

previous = from(bucket: "Bucket")
  |> range(stop: v.timeRangeStart, start: 0)
  |> filter(fn: (r) => r["_measurement"] == "Tags.PLC_252.TANK1.LEVEL")
  |> filter(fn: (r) => r["_field"] == "value")
  |> last()

current = from(bucket: "Bucket")
  |> range(start: v.timeRangeStart, stop: v.timeRangeStop)
  |> filter(fn: (r) => r["_measurement"] == "Tags.PLC_252.TANK1.LEVEL")
  |> filter(fn: (r) => r["_field"] == "value")
  |> aggregateWindow(every: v.windowPeriod, fn: last)

union(tables: [previous, current])
  |> set(key:"_field", value:"test")
  |> yield()

This results in 2 series in grafana, both named "test". One containing a single value, the other one containing the range.
For grafana, this is equivalent:
from(bucket: "Actemium")
  |> range(stop: v.timeRangeStart, start: 0)
  |> filter(fn: (r) => r["_measurement"] == "Tags.PLC_252.TANK1.LEVEL")
  |> filter(fn: (r) => r["_field"] == "value")
  |> last()
  |> yield(name:"A")

from(bucket: "Actemium")
  |> range(start: v.timeRangeStart, stop: v.timeRangeStop)
  |> filter(fn: (r) => r["_measurement"] == "Tags.PLC_252.TANK1.LEVEL")
  |> filter(fn: (r) => r["_field"] == "value")
  |> aggregateWindow(every: 1s, fn: last)
  |> yield(name:"B")

To get the desired result, i need to use grafana's merge transform. But if there are multiple queries, they will all get merged instead. This means that it seems impossible to have multiple queries in the same panel?

To remedy, it would seem to be as simple of piping the union to window(every: inf)


Check output of the Input component is correct is CSS

The input component should support icons in CSS for both classless and classy versions of the output.
The selector for the classless version seems to prevent the classless version from supporting pre/post icons as per designs.

Think this is done now via other updates


🩹 workaround for qiskit-aer and qiskit-terra raising a warning

Description
This resolves the CI that has been stuck for weeks on a small incompatibility between the latest qiskit-terra and qiskit-aer versions. The underlying bug has already been fixed upstream in qiskit-aer and will be part of qiskit-aer==0.12.1.
At that point, a dependabot update should trigger here and the upper caps on the qiskit-terra and qiskit-aer versions can be removed again.
As an additional change, this PR changes the dependencies to explicitly pull qiskit-terra and qiskit-aer instead of all of qiskit, which should make the overall package a little more lightweight.
Checklist:

[X] The pull request only contains commits that are related to it.
[X] I have added appropriate tests and documentation.
[X] I have made sure that all CI jobs on GitHub pass.
[X] The pull request introduces no new warnings and follows the project's style guidelines.

Coverage changes are unrelated and a flake due to the nature of the decoder tests.


Update to Jammy Jellyfish

Hirsute Hippo is EOL and not secure anymore.
When reverse proxying this docker it should be as secure as possible.

That sounds like a good idea! 👍


RFM -plantronicshub

Current Maintainer: amoisis

[ ] I am the maintainer of the package and wish to pass it to someone else;

I DON'T Want To Become The Maintainer

[X ] I have followed the Package Triage Process and I do NOT want to become maintainer of the package;
[X ] There is no existing open maintainer request for this package;

Checklist

[ X] Issue title starts with 'RFM - '

Existing Package Details
Package URL: https://chocolatey.org/packages/plantronicshub
Package source URL: n/a
Date the maintainer was contacted: 2019-04-27 & previous unknown date earlier in the year
How the maintainer was contacted: https://chocolatey.org/packages/plantronicshub/ContactOwners

From<EMAIL_ADDRESS><EMAIL_ADDRESS>On Behalf Of bcurran3
Sent: Saturday, April 27, 2019 8:11 PM
To: bcurran3
Subject: [Chocolatey.org] Message for maintainers of the package 'plantronicshub'
User bcurran3  sends the following message to the maintainers of Package 'plantronicshub'.
Hi. Just an FYI that this is a couple versions out of date.
Package Url: https://chocolatey.org/packages/plantronicshub
Pro-tip: It is recommended to (also) post your answer to the plantronicshub comment section so that the information is publicly available.
This can prevent you from receiving the same question from multiple users.
Package comments: http://chocolatey.org/packages/plantronicshub#disqus

If no one else is interested, I will probably pick it up.

Status: In progress

Waiting to be added as maintainer (request already sent to Chocolatey contact-site admins).
Updated package ready to be published: https://github.com/chtof/chocolatey-packages/tree/master/automatic/plantronicshub

Status: Review
https://chocolatey.org/packages/plantronicshub

Status: Published
https://chocolatey.org/packages/plantronicshub
Can be closed.


How Can I Stop Auto-Syncing?

I would like to prevent auto-sync for the synchronized folders I have. I would like to synchronize on request from either computer. Is there any way of doing this?

As this is a Syncthing issue, you'll get better support on the Syncthing forum.
You can pause the folder from Syncthing's UI.
Another option might be to disable the folder watching in SyncTrayzor (Settings -> Folders -> Uncheck 'Watched'), and set the Rescan Interval to 0 (or a really high number if 0 doesn't work) in Syncthing's folder settings. That will prevent your folder from detecting changes and sending them to other devices, but will not prevent other devices from sending changes to your computer.


Don't worry！

This project has great potential. Magic Blessings!
I am currently using the genepi project, and I will switch back to it once your project is complete.
The entire world is transitioning from the Period 8 of Gen Earth to the Period 9 of Li Fire. All difficulties are temporary.


bug: Focus isn't allowed in api.patch "/users/@me"

What happened?
So apparently the focus presence isn't defined in the api.patch function, returning a typeerror since it doesn't match "Online" | "Idle" | "Busy" | "Invisible" | undefined | null.

But when testing, It does indeed change the status to Focus.

Turns out i forgot to update my npm packages. Closing the issue...


Entity restored and unavailable never recovers from unavailable

The problem

I have a desktop that runs Plex and Sabnzbd and if the desktop is asleep when I restart HA then the entities become unavailable with restored state but when the desktop is awake the states of Plex and Sabnzbd never recover from their unavailable state.
Environment

Home Assistant release with the issue: 0.105.5
Last working Home Assistant release (if known): Never
Operating environment (Hass.io/Docker/Windows/etc.): Docker
Integration causing this issue: Plex
Link to integration documentation on our website: https://www.home-assistant.io/integrations/plex/

Problem-relevant configuration.yaml

plex:
  host: <IP_ADDRESS>

Traceback/Error logs

I believe this log below is when Plex entity was not reached during a restart of HA when the desktop running Plex was suspended.  I did not see any other log entries for Plex after this, such as no attempts to reconnect.
ERROR (MainThread) [homeassistant.components.plex] Plex server could not be reached: http://<IP_ADDRESS>:32400
ERROR (MainThread) [homeassistant.components.plex] Plex server (http://<IP_ADDRESS>:32400) could not be reached: [HTTPConnectionPoo
l(host='<IP_ADDRESS>', port=32400): Max retries exceeded with url: / (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0xb11f9510>: Failed to establish a new
 connection: [Errno 113] Host is unreachable'))]

Additional information
I've worked around this issue by restarting HA when my desktop becomes available:
This is my automation:
- alias: 'Turn on compute'
  trigger:
    - platform: state
      entity_id: binary_sensor.desktop_ping
      to: 'on'
  action:
    - service: script.restart_ha

I removed this automation when I upgraded to latest HA as I saw in release notes about entities getting put into "restored: true" state but seems the entities never recover.

Sounds like the fix I put in 2 days ago will fix this: https://github.com/home-assistant/home-assistant/issues/32071. You can try in the beta or wait until 0.106 is released.


Query String To View Other Users Profiles

defineUser() function will check if there is a query string for a profile and define the user string as that, otherwise default to the logged in user.


getPreventDefault() is deprecated.  Use defaultPrevented

clueless newbie here.
Ubuntu 14.04 x86 / Firefox 38.0 / npm installed Bacon 0.7.65 just now,
I always get the warning "Use of getPreventDefault() is deprecated.  Use defaultPrevented instead."
`

`

Bacon.js does not use getPreventDefault. I don't think this is Bacon.js related. Can you narrow it down to somethint more specific.

ok, sorry for the noise. probably due to me trying to use some version of jquery?


Remove Kotlin dependency from Cronvoy

It turns out that removing Kotlin from Envoy Mobile is not really needed at this point. In reality, the Kotlin classes are a layer on top of the Java classes, and Cronvoy is a layer too: one level of indirection for the callbacks is enough. So instead Cronvoy uses the java classes directly. The main drawback is the CronvoyConfiguration - there is some duplication.
Description: Have Cronvoy to only use the Envoy Mobile Java classes for its implementation
Risk Level: None
Testing: CI
Docs Changes: N/A
Release Notes: N/A
Fixes #1658
Signed-off-by: Charles Le Borgne<EMAIL_ADDRESS>

Rerunning some tests (we've had some spurious failures on RBE).

Overall, I'm fine with this approach and I think it's great we found a lower-friction solution for the time being.

Allan, I was surprised too by how few changes were required. And the good news it is now much easier for me to used the explicitFlowControl


Improve speed

I want to make brunch the fastest build system.
@damassi what exact speed issues do you have? What compilation time says? (info: compiled in 70ms etc)

@paulmillr Not sure if that's the project size (186 jsx component files), but my initial build (babel + react) is 75s to 100s and changing one file in the watcher is 1 to 1.5s.
Is that expected?

100s for first build is fine if the project is big.
1.5 seconds when changing one file seems bad. Could you open a new issue clarifying your plugins, project, brunch / node version, machine config etc?


Fixes invisible cursor in long editor lines

The problem lies somewhere in browser rendering optimization,
in case when the translate3d is turned off everything
works correctly.
fixes #8828

@alexandrudima FYI the issue was present also before the cursor animation refactor.

The root cause of #8828 we think was the interaction of mix-blend-mode: luminosity; from the tabs with translate3d from the cursors. As the cursors are now using CSS animations, I think it is fine to remove their own gpu layer.

Thank you!


[Documentation] broken Fireblade url

Hi 👋 just to tell you that the Fireblade game engine url is broken.
Best regards.

@RichardFevrier thanks for reporting this. Fixed it.


[Feature] Support S3DIS data pre-processing and dataset class

Provide instructions to S3DIS data downloading
Provide scripts and meta_data for S3DIS data pre-processing
Add a S3DISSegDataset class
Add a config file for S3DIS data loading

Different from other datasets like ScanNet, S3DIS doesn't have a train-val split! S3DIS has 6 areas of data (can think of them as 6 subsets, collected in different buildings/regions). The common setting is training on 5 of them, and testing on the remaining one. People often report an Area_5 test result, and an average result. The average result means the average mIoU of 6 experiments (leaving Area_1/2/.../6 respectively).
Therefore, in create_data.py, I generate one info file for one area, so that we can contact 5 to form a training dataset and use the remaining one for test dataset.

Work in progress, planning to support S3DISSegDataset class and unittest and config files in later commits.

Please check all the docstring, there are still many nonstandard formats.
Next time we can split similar PR into several small ones. It is so heavy to review and merge with so many things in the single one 0.0

I just copy-paste the pre-processing code of S3DIS from other repo so it's dirty, I am sorry. I check it and find many unnecessary functions. I will remove and make the code cleaner.
Yes, next time I will split into small PRs. I apologize for that.

I just copy-paste the pre-processing code of S3DIS from other repo so it's dirty, I am sorry. I check it and find many unnecessary functions. I will remove and make the code cleaner.
Yes, next time I will split into small PRs. I apologize for that.

OK, it doesn't matter. You have done a good job :)

Need to add pipeline support in S3DISSegDataset.show/evaluate() after PR#430 is merged.

Need to add pipeline support in S3DISSegDataset.show/evaluate() after PR#430 is merged.

Done. Now the PR is ready for review @ZwwWayne @Tai-Wang. Thanks.

Resolve conflicts.


parse and write ">" in attribute

Hello, I load a xml file and write to output directly. the attribute value was changed.
input file:
<note>
<to>Tove</to>
<from>Jani</from>
<heading attr="Don't <forget me> this weekend!">Reminder</heading>
Don't forget me this weekend!
</note>

output:
<?xml version="1.0"?>
<note>
	<to>Tove</to>
	<from>Jani</from>
	<heading attr="Don't <forget me> this weekend!">Reminder</heading>
	Don't forget me this weekend!
</note>

Unescaped > is allowed in attribute values (unlike unescaped <) by the XML standard. pugixml tries to produce the minimal encoded output, so it doesn't escape characters that don't need to be escaped.


Teleporting near Hosidius grape vines causes client crash

Describe the bug
After using a teletab to Falador from the Hosidius grape vine bank chest, the client crashed.
To Reproduce
Steps to reproduce the behavior:
Haven't reproduced
Expected behavior
No crash
Screenshots
If applicable, add screenshots to help explain your problem.
Environment (please complete the following information):

OS: Windows 10
RuneLite version: <IP_ADDRESS>
Launcher version: 2.7.1

Additional context
Logs:

2024-06-06 00:16:01 PDT [Client] ERROR n.r.c.p.t.farming.FarmingTracker - Offset error! Observed new offset of 4, previous observed offset was 31 (1) Player Loc:WorldPoint(x=1270, y=3728, plane=0)
2024-06-06 00:41:05 PDT [Client] ERROR n.r.c.p.t.farming.FarmingTracker - Offset error! Observed new offset of 4, previous observed offset was 31 (1) Player Loc:WorldPoint(x=1809, y=3566, plane=0)
2024-06-06 00:41:05 PDT [Client] ERROR n.r.c.p.t.farming.FarmingTracker - Offset error! Observed new offset of 4, previous observed offset was 31 (1) Player Loc:WorldPoint(x=1809, y=3566, plane=0)
2024-06-06 00:41:05 PDT [Client] ERROR n.r.c.p.t.farming.FarmingTracker - Offset error! Observed new offset of 4, previous observed offset was 31 (1) Player Loc:WorldPoint(x=1809, y=3566, plane=0)
2024-06-06 00:41:05 PDT [Client] ERROR n.r.c.p.t.farming.FarmingTracker - Offset error! Observed new offset of 4, previous observed offset was 31 (1) Player Loc:WorldPoint(x=1809, y=3566, plane=0)
2024-06-06 00:41:05 PDT [Client] ERROR n.r.c.p.t.farming.FarmingTracker - Offset error! Observed new offset of 4, previous observed offset was 31 (1) Player Loc:WorldPoint(x=1809, y=3566, plane=0)
2024-06-06 00:41:05 PDT [Client] ERROR n.r.c.p.t.farming.FarmingTracker - Offset error! Observed new offset of 4, previous observed offset was 31 (1) Player Loc:WorldPoint(x=1809, y=3566, plane=0)
2024-06-06 00:41:05 PDT [Client] ERROR n.r.c.p.t.farming.FarmingTracker - Offset error! Observed new offset of 4, previous observed offset was 31 (1) Player Loc:WorldPoint(x=1809, y=3566, plane=0)
2024-06-06 00:41:05 PDT [Client] ERROR n.r.c.p.t.farming.FarmingTracker - Offset error! Observed new offset of 4, previous observed offset was 31 (1) Player Loc:WorldPoint(x=1809, y=3566, plane=0)
2024-06-06 00:41:05 PDT [Client] ERROR n.r.c.p.t.farming.FarmingTracker - Offset error! Observed new offset of 4, previous observed offset was 31 (1) Player Loc:WorldPoint(x=1809, y=3566, plane=0)
2024-06-06 00:41:05 PDT [Client] ERROR n.r.c.p.t.farming.FarmingTracker - Offset error! Observed new offset of 4, previous observed offset was 31 (1) Player Loc:WorldPoint(x=1809, y=3566, plane=0)
2024-06-06 00:41:05 PDT [Client] ERROR n.r.c.p.t.farming.FarmingTracker - Offset error! Observed new offset of 4, previous observed offset was 31 (1) Player Loc:WorldPoint(x=1809, y=3566, plane=0)
2024-06-06 00:41:05 PDT [Client] ERROR n.r.c.p.t.farming.FarmingTracker - Offset error! Observed new offset of 4, previous observed offset was 31 (1) Player Loc:WorldPoint(x=1809, y=3566, plane=0)
2024-06-06 00:50:47 PDT [Client] WARN  n.runelite.client.eventbus.EventBus - Uncaught exception in event subscriber
java.lang.IndexOutOfBoundsException: Varbit 10733 does not exist
at client.getVarbitValue(client.java:54049)
at client.getVarbitValue(client.java:11957)
at rs117.hd.scene.LightManager.trackImpostorChanges(LightManager.java:783)
at rs117.hd.scene.LightManager.handleObjectSpawn(LightManager.java:755)
at rs117.hd.scene.LightManager.handleObjectSpawn(LightManager.java:727)
at rs117.hd.scene.LightManager.onGameObjectSpawned(LightManager.java:1010)
at net.runelite.client.eventbus.EventBus$Subscriber.invoke(EventBus.java:70)
at net.runelite.client.eventbus.EventBus.post(EventBus.java:223)
at net.runelite.client.callback.Hooks.post(Hooks.java:202)
at jy.gb(jy.java:40796)
at jy.xp(jy.java:16465)
at client.fd(client.java:55043)
at client.kr(client.java:57730)
at client.ii(client.java:3098)
at client.bp(client.java:1156)
at be.ag(be.java:371)
at be.fk(be.java)
at be.run(be.java:31308)
at java.base/java.lang.Thread.run(Unknown Source)
2024-06-06 00:50:47 PDT [Client] ERROR client-patch - Client error
java.lang.IndexOutOfBoundsException: Varbit 10733 does not exist
at client.getVarbitValue(client.java:54049)
at client.getVarbitValue(client.java:11957)
at rs117.hd.scene.LightManager.trackImpostorChanges(LightManager.java:783)
at rs117.hd.scene.LightManager.handleObjectSpawn(LightManager.java:755)
at rs117.hd.scene.LightManager.loadSceneLights(LightManager.java:650)
at rs117.hd.HdPlugin.swapScene(HdPlugin.java:2380)
at client.fd(client.java:19105)
at client.kr(client.java:57730)
at client.ii(client.java:3098)
at client.bp(client.java:1156)
at be.ag(be.java:371)
at be.fk(be.java)
at be.run(be.java:31308)
at java.base/java.lang.Thread.run(Unknown Source)

Varbit 10733 seems to correspond to the NPC Meredith at the GE.
https://static.runelite.net/api/runelite-api/constant-values.html#net.runelite.api.Varbits

Thanks for the detailed error report. This will be fixed in the next release.


CLI tools don't preserve v:progname, v:progpath

Describe the bug
Vim exposes the name of the binary that was executed in the v:progname variable. Unfortunately, invoking the various vim CLI tools installed by MacVim always reports v:progname as being Vim. This is because they're all symlinks of the embedded mvim script, which ultimately invokes
exec "$binary" $opts …

In particular, it's not providing the -a flag, so the binary is always invoked with the executable name of Vim.
The same issue also affects v:progpath (for obvious reasons), wherein it always points to the absolute path to the MacVim.app/Contents/MacOS/Vim binary.
This is an issue because v:progname and v:progpath are potentially useful to use in my vimrc to customize based on what binary I used to invoke it.
Replacing this with exec -a "$0" "$binary" … should solve the problem.
On a related note, launching MacVim from the Finder also sets v:progname to Vim. I can't decide if this matters.
To Reproduce
Detailed steps to reproduce the behavior:

Run ex
Type echo v:progname or echo v:progpath

Expected behavior
It should print "ex" as the progname and either "ex" or the absolute path to ex as the progpath.
Observed behavior
It always prints Vim as the progname and the absolute path to MacVim.app/Contents/MacOS/Vim as the progpath.
Environment (please complete the following information):

Vim version 8.2.1719
OS: macOS 11.2.3 (20D91)
Terminal: Apple Terminal.app or GUI

Experimenting right now with the proposed exec -a solution, this actually seems to fail in a few cases:

If I pass exec -a gvim, Vim exits instantly (unless I pass -f as well). exec -a gvim ./Vim -g also exits instantly. exec ./Vim -g works (as this is what the mvim script is doing), so something is rather broken with its handling of the gvim name.
The -g flag is broken with other arbitrary names too, e.g. exec -a foo ./Vim -g does nothing, though it works (and preserves the progname) when used without the -g flag, unless of course the name starts with "g". For example, exec -a gfoo ./Vim -g fails (as does the version without -g).

It looks like the gvim issue could be worked around with an explicit fork+disown, something like ( exec -a "$0" "$binary" --nofork ) & disown. This should only be done for jobs with $gui set of course. And the logic that clears $gui when SSH_COMMAND is set will want to reset the binary name too (it could try to trim any leading "g"s from the front, after skipping a single leading "r", this way e.g. gview will still be interpreted as view)
Unfortunately we still need to peek at the binary name to set flags, as all of this fails if the Vim binary itself is replaced with a shell script (which is what happens in the Nix package when using the vim configuration feature, as we need to pass a custom vimrc). This is because bash seems to ignore argv[0] when passed a filename.

Another wrinkle: Spawning several vim processes with the --nofork + disown combo all produces windows in the same MacVim GUI process, but spawning a process without this (e.g. invoking mvim) produces a separate GUI process. This is strange.

Huh, seems like this requires some thought since as you pointed out it's not that easy to pass it in. It does feel like it should be fixed. Let me think about it a little bit (feel free to keep experimenting).
That said, I'm just curious by how come you need to access v:progname? Are you trying to tell if you are launching from e.g. view or vim?

I was actually experimenting to see if I could figure out how to detect ex -s, as I'm replicating the normal initialization routine in a custom vimrc provided with -u. v:progname is not actually helpful here, as it doesn't tell me about the -s flag or tell me if I used vim -e, but it's theoretically useful if I want to detect view. I'm also pondering the idea of making a new alias that I detect in my vimrc to disable a bunch of stuff, e.g. for a faster-launching more minimal setup.

@ichizok c143fa0778fa0d8744867318bb7f7a2e63cf37d7 appears to be unrelated, I think perhaps you typed the wrong issue number?

Oh... the issue number in upstream vim matches this.

 However I personally usually never use these aliases even with normal vim in other OSes.
I usually did this with VimPlug by adding { 'on': 'TableModeEnable' } or { 'for': 'python' } to a plugin definition to not enable a plugin before I really need it. Thus most plugins are off when I edit a new empty buffer


[Feature]: Enable logical grouping of widgets and actions

Is there an existing issue for this?

[X] I have searched the existing issues

Summary

Currently, there is no way to logically group a set of widgets and/or actions. I am imagining the end effect to be something like what CSS does for HTML elements. For e.g. consider the following use cases that may be addressed by this feature:

a user has 20 different input widgets on a page and they want to update the alignment property for each of them. The current way is to go through each widget one by one and set the property individually. However, this is very painful and time consuming. A better approach could be to assign all the widgets to a logical group and update the alignment property for this logical group which should automatically apply the same to all the member widgets.
a user has 20 action and he wants to set each of them to run on page load. One way is to go through each action one by one and set their property individually, however, it is very painful and time consuming. Hence a better approach would be to group them together and assign the on page load property for the group which should automatically apply the same to the actions as well.

Why should this be worked on?

Speeds up development.
Improves user experience.

Closing this as we're picking up Group styling separately and setting 20 queries on page load is not a very common use case


Crashing when playing FLAC files.

Shows the following error.
CODE 4 : DEMUXER_ERROR_COULD_NOT_OPEN: FFmpegDemuxer: open context failed

To confirm the issue about chromium, if you're on windows you can try to open the flac file with microsoft edge from the context menu and you'll see that edge will also not be able to play that song whereas something like firefox will be able to play it perfectly.

Yeah, it's a known bug that occurs while playing some flac files because of chromium: #190 #184
What I usually do to 'fix' this issue is change some tags using mp3tag. And then select all files on mp3tag and use this to change the file names:  Just select any format for file renaming and then apply the changes.
You can also try to do a flac to flac convert using foobar2000 and see if they song plays then (navidrome/navidrome#2437)

I found out an easy fix. Select all files in Mp3tag editor and change the album covers to 500x500. Every FLAC files play now.


Inexplicable Connectivity

I have a server setup with three nodes, each running redis and sentinel. I can connect to the redis and sentinel instances using redis-cli, switching between them by changing the port number.  I can run commands to see keys, the client list, etc.
When using ioredis from our nodeJS project I can connect to redis nodes and run queries. However, when I use the sentinel port number it never responds.  No errors, it takes whatever get/set commands I send but never returns anything. If I set "enableOfflineQueue" to false I get back "Error: Stream isn't writeable and enableOfflineQueue options is false".
So it appears to me I have a connectivity problem from code (but redis-cli works) but this is the only error message I've been able to coax out of it.  I also subscribed to the "connect", "ready" and "error" events but they never return when using the Sentinel address.
I've been poring through what info I can find for quite a while, would really appreciate any tips you might have on connectivity issues or just better error messages. I am new to redis, having learned what I know digging through code I inherited.
thanks!

Could you enable the debug mode DEBUG=ioredis:* node yourapp.js and post the logs here?

I'm closing this. Feel free to reopen it with details.


mips64-n32 failed to start

Attempting to use mips64-n32, i get the following:
arm_now start mips64-n32
(use --clean next time) An aarch64 image exists, delete ? (y/n) y

Downloading /root/.config/arm_now/templates/mips64-n32.tar.xz from https://github.com/nongiach/arm_now_templates/raw/master/mips64-n32.tar.xz                                                                                [*] 1.4 MB / 1.4 MB @ 3.0 MB/s [##################] [100%, 0s left]
[+] Installed
Tempdir /tmp/tmpuk8185gs
File not found by ext2_lookup
WARNING: e2rm file already suppressed
Starting qemu-system-mips64
stty intr ^]
       export QEMU_AUDIO_DRV="none"
       qemu-system-mips64 -kernel arm_now/kernel -hda arm_now/rootfs.ext2 -append 'root=/dev/hda console=ttyS0 rw physmap.enabled=0 noapic'                -m 256M                -nographic                -serial stdio -monitor /dev/null                                  -no-reboot
       stty intr ^c

WARNING: Image format was not specified for 'arm_now/rootfs.ext2' and probing guessed raw.
         Automatically detecting the format is dangerous for raw images, write operations on block 0 will be restricted.
         Specify the 'raw' format explicitly to remove the restrictions.
qemu-system-mips64: qemu: could not load kernel 'arm_now/kernel': Failed to load ELF
+++++++++++++++++++++++++++ Checking the filesystem ++++++++++++++++++++++++++++
e2fsck 1.44.1 (24-Mar-2018)
Pass 1: Checking inodes, blocks, and sizes
Pass 2: Checking directory structure
Pass 3: Checking directory connectivity
Pass 4: Checking reference counts
Pass 5: Checking group summary information

         397 inodes used (1.59%, out of 24960)
           0 non-contiguous files (0.0%)
           1 non-contiguous directory (0.3%)
             # of inodes with ind/dind/tind blocks: 19/4/0
        9061 blocks used (8.85%, out of 102400)
           0 bad blocks
           0 large files

          60 regular files
          41 directories
           1 character device file
           0 block device files
           0 fifos
           0 links
         286 symbolic links (286 fast symbolic links)
           0 sockets
------------
         388 files

I met the same issue，have you solved this？

The kernel file downloaded by the script from https://github.com/nongiach/arm_now_templates/blob/master/mips64-n32.tar.xz is an empty file:
(default) ➜  _M4000_SIP_F7.20A.258.119.cmp.extracted wget https://github.com/nongiach/arm_now_templates/blob/master/mips64-n32.tar.xz\?raw\=true
--2020-09-25 11:07:43--  https://github.com/nongiach/arm_now_templates/blob/master/mips64-n32.tar.xz?raw=true
Resolving github.com (github.com)... <IP_ADDRESS>
Connecting to github.com (github.com)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 302 Found
Location: https://github.com/nongiach/arm_now_templates/raw/master/mips64-n32.tar.xz [following]
--2020-09-25 11:07:43--  https://github.com/nongiach/arm_now_templates/raw/master/mips64-n32.tar.xz
Reusing existing connection to github.com:443.
HTTP request sent, awaiting response... 302 Found
Location: https://raw.githubusercontent.com/nongiach/arm_now_templates/master/mips64-n32.tar.xz [following]
--2020-09-25 11:07:43--  https://raw.githubusercontent.com/nongiach/arm_now_templates/master/mips64-n32.tar.xz
Resolving raw.githubusercontent.com (raw.githubusercontent.com)... <IP_ADDRESS>
Connecting to raw.githubusercontent.com (raw.githubusercontent.com)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 200 OK
Length: 1467500 (1.4M) [application/octet-stream]
Saving to: 'mips64-n32.tar.xz?raw=true'

mips64-n32.tar.xz?raw=true                 100%[========================================================================================>]   1.40M  5.48MB/s    in 0.3s

2020-09-25 11:07:43 (5.48 MB/s) - 'mips64-n32.tar.xz?raw=true' saved [1467500/1467500]

(default) ➜  _M4000_SIP_F7.20A.258.119.cmp.extracted mv mips64-n32.tar.xz\?raw=true mips64-n32.tar.xz
(default) ➜  _M4000_SIP_F7.20A.258.119.cmp.extracted tar -xJvf mips64-n32.tar.xz
arm_now/
arm_now/rootfs.ext2
arm_now/kernel
arm_now/arch
(default) ➜  _M4000_SIP_F7.20A.258.119.cmp.extracted ls -lh arm_now
total 101M
-rw-r--r-- 1 enki enki   10 Jun  3  2018 arch
-rw-r--r-- 1 enki enki    0 Jun  3  2018 kernel
-rw-r--r-- 1 enki enki 100M Jun  3  2018 rootfs.ext2
(default) ➜  _M4000_SIP_F7.20A.258.119.cmp.extracted

When you specify --real-source it downloads the proper kernel from bootlin:
(default) ➜  _M4000_SIP_F7.20A.258.119.cmp.extracted arm_now install --real-source mips64-n32
Scrawling kernel for arch 'mips64-n32'
Kernel=https://toolchains.bootlin.com/downloads/releases/toolchains/mips64-n32/test-system/mips64-n32--glibc--stable-2020.02-2-vmlinux, DTB=None, rootfs=https://toolchains.bootlin.com/downloads/releases/toolchains/mips64-n32/test-system/mips64-n32--glibc--stable-2020.02-2-rootfs.ext2

Downloading arm_now/kernel from https://toolchains.bootlin.com/downloads/releases/toolchains/mips64-n32/test-system/mips64-n32--glibc--stable-2020.02-2-vmlinux
 [*] 8.2 MB / 8.2 MB @ 2.6 MB/s [##################] [100%, 0s left]

Downloading arm_now/rootfs.ext2 from https://toolchains.bootlin.com/downloads/releases/toolchains/mips64-n32/test-system/mips64-n32--glibc--stable-2020.02-2-rootfs.ext2
 [*] 60.0 MB / 60.0 MB @ 1.8 MB/s [##################] [100%, 0s left]
[+] Installed
(default) ➜  _M4000_SIP_F7.20A.258.119.cmp.extracted ls -lh arm_now
total 69M
-rw-rw-r-- 1 enki enki   10 Sep 25 11:09 arch
-rw-rw-r-- 1 enki enki 8.2M Sep 25 11:08 kernel
-rw-rw-r-- 1 enki enki  60M Sep 25 11:09 rootfs.ext2

and boots correctly:
(default) ➜  _M4000_SIP_F7.20A.258.119.cmp.extracted arm_now start mips64-n32
WARNING: arm_now/ already exists, use --clean to restart with a fresh filesystem
Tempdir /tmp/tmpm636kpn5
File not found by ext2_lookup
WARNING: e2rm file already suppressed
Starting qemu-system-mips64
stty intr ^]
       export QEMU_AUDIO_DRV="none"
       qemu-system-mips64 -kernel arm_now/kernel -hda arm_now/rootfs.ext2 -append 'root=/dev/hda console=ttyS0 rw physmap.enabled=0 noapic'                -m 256M                -nographic                -serial stdio -monitor null  -nic user                                 -no-reboot
       stty intr ^c

WARNING: Image format was not specified for 'arm_now/rootfs.ext2' and probing guessed raw.
         Automatically detecting the format is dangerous for raw images, write operations on block 0 will be restricted.
         Specify the 'raw' format explicitly to remove the restrictions.
Linux version 4.19.91 (root@runner-37d48167-project-3290221-concurrent-2) (gcc version 8.4.0 (Buildroot 2020.02-00011-g7ea8a52)) #1 SMP Sat Apr 4 23:30:09 UTC 2020
earlycon: uart8250 at I/O port 0x3f8 (options '38400n8')

[...]

Saving random seed: random: dd: uninitialized urandom read (512 bytes read)
OK
pcnet32 0000:00:0b.0 eth0: link up
press ctrl+] to kill qemu

Welcome to arm_now
buildroot login: root
# uname -m
mips64

The https://github.com/nongiach/arm_now_templates/blob/master/mips64-n32.tar.xz should be rebuilt to properly fix this.

Congrats, you found the issue 👍
What about you introduce a fix via a pull request ? :)


CORE-410 - User Impersonation Bugs

Closes CORE-410

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


Timesheets README file update (with gif)

Fixed README formatting, setup instructions, and last updated.

Hey the new gif looks great. This PR is showing diffs on your README file and that you have a merge conflict. I'm not sure how great merge a gif is going to go. If you can get the README to show no changes on this branch I'll be happy to approve or feel free to open a clean new branch.

It looks like you pushed a change but the merge conflict is still showing up and you have changed lines on the README.


MOD plugin

Research how to let the chorus work as MOD plugin.
It is derivative of classic lv2, whose UI obeys to this specification http://moddevices.com/ns/modgui/
Design should be separate made with HTML and CSS.
This has some hint on running MOD on Linux computer
https://forum.moddevices.com/t/simulating-a-mod-on-my-linux-pc/660/11

Made a simple modgui. Not tested


SoundChecker - Test THD and SNR - App crashed when selecting output device as "USB audio -..."

Connect Pixel 7Pro with Vantec Audio Adapter (connect loop back cables on the device)

Launch SoundChecker
Select "Test THD and SNR"
Select Input device as USB Audio
Select Output device as "USB Audio - ICUSBAUDIO7D USB device card=1;device=0"

EXPECTED
Test can be run
ACTUAL
Sound-Checker app crashed

This is being tracked internally with:
b/316181327 | P2 | SoundChecker - Test THD and SNR - App crashed when selecting output device as "USB-Audio .."


Here’s the issue we encountered with redisson-3.11.5.jar

Our service connects to the SLB (Server Load Balancer) using a domain name, which maps through a port to the Redis server nodes.
For example:

Service IP --> Domain A:2700 --> EIP (<IP_ADDRESS>):27000 --> Redis server: <IP_ADDRESS>:27000

The Redis sentinel nodes are:

<IP_ADDRESS>:27000
<IP_ADDRESS>:27000
<IP_ADDRESS>:27000

Redisson performs two types of checks: scheduleSentinelDNSCheck and scheduleChangeCheck. These checks result in the following situations:

redis://<IP_ADDRESS>:27000 added
redis://<IP_ADDRESS>:27000 was down
redis://<IP_ADDRESS>:27000 added
redis://<IP_ADDRESS>:27000 was down

This causes a polling effect, even though they ultimately point to the correct sentinel nodes.
How to Resolve This Issue

scheduleSentinelDNSCheck will add <IP_ADDRESS> to the sentinels.
scheduleChangeCheck (when it uses <IP_ADDRESS>:27000 to connect and retrieve other sentinels: redis://<IP_ADDRESS>:27000, redis://<IP_ADDRESS>:27000 and add <IP_ADDRESS>:27000 to a newUris list to form a new set of three sentinels) will remove <IP_ADDRESS> from the sentinels.

because <IP_ADDRESS> doesn't have <IP_ADDRESS> in sentinel list. You can verify this by setting trace logging level for org.redisson package.

Yes, it seems that Redisson doesn't fully support this SLB-based connection method, right? Although it doesn't directly impact business functionality, it does lead to continuous adjustments in the sentinels variable, otherwise we'll keep seeing "down added" occurrences. Now, the issue we're facing is that the service logs continue to record these changes persistently. How should I address this? Should we consider abandoning the use of SLB?

Did you try natMapper setting ? to resolve EIP to a real Redis node IP

这是否意味着使用 SLB 时同一个 Redis 节点有两个 IP？一个是在sentinel配置中定义的，另一个是用于SLB的？

"Yes, the reason we use SLB (Server Load Balancer) is because we have multiple Redis instances deployed on the same machine, each with 2 IP addresses.
All services are connected via domain names. Most of the Redis instances are in master-slave mode to ensure failover. Therefore, we adopt the SLB approach. We dynamically modify the server addresses under the SLB port after detecting master-slave switches."

Does it mean that with SLB the same Redis node has two IPs ? one is defined in sentinel configs and another is used for SLB?

"Yes, the reason we use SLB (Server Load Balancer) is because we have multiple Redis instances deployed on the same machine, each with 2 IP addresses.
All services are connected via domain names. Most of the Redis instances are in master-slave mode to ensure failover. Therefore, we adopt the SLB approach. We dynamically modify the server addresses under the SLB port after detecting master-slave switches."

你尝试natMapper设置了吗？将EIP解析为真实的Redis节点IP

你尝试natMapper设置了吗？将EIP解析为真实的Redis节点IP

Did you try natMapper setting ? to resolve EIP to a real Redis node IP

Doesn't seem to make any sense

Unfortunately, it's a specific case. You can define natMapper like below.
                .setNatMapper(new NatMapper() {
                    @Override
                    public RedisURI map(RedisURI uri) {
                        if (uri.getHost().equals("<IP_ADDRESS>")) {
                            return ew RedisURI(uri.getScheme(), "<IP_ADDRESS>", uri.getPort());
                        }
                        ...
                    }
                })

不幸的是，这是一个具体案例。您可以像下面一样定义 natMapper。
                .setNatMapper(new NatMapper() {
                    @Override
                    public RedisURI map(RedisURI uri) {
                        if (uri.getHost().equals("<IP_ADDRESS>")) {
                            return ew RedisURI(uri.getScheme(), "<IP_ADDRESS>", uri.getPort());
                        }
                        ...
                    }
                })

thank you

Another solution is to use org.redisson.api.HostNatMapper implementation.
Map<String, String> map = new HashMap<>();
map.put("<IP_ADDRESS>", "<IP_ADDRESS>");

config.setNatMapper(new HostNatMapper(map))


New Luxttronik Firmware V3.89.2-10024 did not connect

do i have to change something
I get no connection after new install of firmware

do i have to change something
I get no connection after new install of firmware

After a firmware upgrade: Please check the network config from your heatpump and try to connect via mobile app. e.g. Luxprobe.
After the firmware upgrade the Luxtronik often lost the DHCP and IP Config.
Or it is reachable in the network, but the socket doesn't answer.
Can you restart your heat pump? E.g. login as service in the web interface and click restart or power off the heat pump when it's idled.

No problem here with the latest update. (I liked how it showed the update in HA, by the way!)

Yes after an complete reboot of HA everything ist ok !

Okay, you reboot HA not the heatpump.
Sounds like the reconnect doesn't work anymore. I ty to check it.
BTW: You can restart the integration in the devices view. So you don't need to restart HA. I fixed the integration unload last month.

Okay, you reboot HA not the heatpump.
Sounds like the reconnect doesn't work anymore. I ty to check it.
BTW: You can restart the integration in the devices view. So you don't need to restart HA. I fixed the integration unload last month.

@ChrisMisker  I liked how it showed the update in HA, by the way!

I have extend the information in the dialog:


{Compute} Update help of --version of vm/vmss extension

Description

https://github.com/Azure/azure-cli/issues/11961
--version
The version of the extension. If you want to use the exact version number specified, please specify --no-auto-upgrade.
Auto upgrade is on by default. So remind users.
Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

add to S168


Email addresses as wallet names don't work

Make a wallet like<EMAIL_ADDRESS>try to open the wallet after restarting NCC. It won't work.

ractive is viewing the '.' as a field access expression :/

Then don't allow "." as wallet names, when creating a new wallet?

fixed in f736a9daa85a3d74e0bdf85c6fdfd8fa8ae60da2


add test coverage to set.go

What type of PR is this?

/kind cleanup

What this PR does / why we need it:
Missing coverage for a few of the set methods. This PR introduces additional coverage.
Release note:
NONE

/lgtm


Hardware Details: Builds/Boots/Tests tabs

[x] #450
[x] #462
[x] #494
[x] #476
[x] #495
[x] #478
[x] #479
[x] #480
[x] #514

All subtasks are now complete


improvement: better errors when mutated value or on_change

We can educate when people try to mutate

Nice. Since we're here, does copying a UI element break things? It looked like it when I was playing around

@dmadisetti i'm not sure. @akshayka would know better

I think copying might break things, but I need to double check. There is special logic for cloning.