1，no worker condition , master will while ture wait for worker startup 2，worker response task status sync wait for result

Tips

Thanks very much for contributing to Apache DolphinScheduler.
Please review https://dolphinscheduler.apache.org/en-us/community/index.html before opening a pull request.

What is the purpose of the pull request
(For example: This pull request adds checkstyle plugin.)
Brief change log
(for example:)

Add maven-checkstyle-plugin to root pom.xml

Verify this pull request
(Please pick either of the following options)
This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added dolphinscheduler-dao tests for end-to-end.
Added CronUtilsTest to verify the change.
Manually verified the change by testing locally.

Codecov Report

Merging #2420 into dev will decrease coverage by 0.07%.
The diff coverage is 5.26%.

@@             Coverage Diff              @@
##                dev    #2420      +/-   ##
============================================
- Coverage     32.89%   32.82%   -0.08%
+ Complexity     2275     2272       -3
============================================
  Files           423      423
  Lines         19839    19868      +29
  Branches       2366     2373       +7
============================================
- Hits           6527     6521       -6
- Misses        12726    12759      +33
- Partials        586      588       +2

Impacted Files
Coverage Δ
Complexity Δ

...ver/master/consumer/TaskPriorityQueueConsumer.java
0.00% <0.00%> (ø)
0.00 <0.00> (ø)

...r/server/worker/processor/TaskCallbackService.java
51.11% <0.00%> (-9.87%)
8.00 <0.00> (-1.00)

...lphinscheduler/service/process/ProcessService.java
0.00% <0.00%> (ø)
0.00 <0.00> (ø)

...uler/server/master/processor/TaskAckProcessor.java
22.22% <9.09%> (-7.19%)
1.00 <0.00> (ø)

...server/master/processor/TaskResponseProcessor.java
23.07% <10.00%> (-8.18%)
1.00 <0.00> (ø)

...scheduler/alert/template/AlertTemplateFactory.java
42.85% <0.00%> (-28.58%)
2.00% <0.00%> (-1.00%)

...he/dolphinscheduler/common/thread/ThreadUtils.java
71.66% <0.00%> (-6.67%)
13.00% <0.00%> (-1.00%)

Continue to review full report at Codecov.

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


Update quick-create-portal.md

Hi Team,
This PR fixes a typo. The dot has been wrongly translated in Japanese version.
Best regards,

@momoto : Thanks for your contribution! The author, @cynthn, has been notified to review your proposed change.

#sign-off


Move the Emergence top menu to a sidebar

The idea is to make the UI more mobil friendly and adopt a simpler UI.
Inspiration: https://almsaeedstudio.com

fixed with #13


Handle when output file would be in subFolder specified by outDir/declarationDir

Fixes #35328

@typescript-bot cherry-pick this to release-3.7


Intro screen displays incorrect current version for codebase

The login screen displays ..1.2, or so, I believe?

That's located in data/motd this is a great issue to test the pull request waters for someone new

Heard

Closed with #346


feat(tools) eslint-plugin

Closes #2383, #2382
What I did

added @patternfly/eslint-plugin-elements
add no-lit-decorators-index-import to above
lint sources with same

Testing Instructions

check demos of affected elements

A-Mazing!


Concurrency Field - Not Reacting to Parameters Changing

Description
Moved from https://github.com/dotnet/blazor-samples/issues/378 ...
https://github.com/dotnet/blazor-samples/blob/d000234099558a1db2258a77d242e04ef2a9cfe7/8.0/BlazorWebAppEFCore/Components/ConcurrencyField.razor#L36-L47
This should be using the OnParametersSet, such that when the DbModel changes the property is checked again. Also none of the properties of the Contact have ConcurrencyCheck attributes, so the newest EF core will not automatically fail. I suggest adding the ConcurrencyCheck attribute to at least one property or every property because the UI implies the whole object is checked for concurrency.
cc: @crazyfox55
Page URL
https://learn.microsoft.com/en-us/aspnet/core/blazor/blazor-ef-core?view=aspnetcore-8.0
Content source URL
https://github.com/dotnet/AspNetCore.Docs/blob/main/aspnetcore/blazor/blazor-ef-core.md
Document ID
2ebc5e4d-bab4-6d30-9018-9a712c482236
Article author
@guardrex

@crazyfox55 ... Before I ping the product unit for assistance, what database provider are you using?

The database I'm using is sqlite, just like the sample uses.

No 🎲🎲 with that provider ... it no-ops that way for sure. I added a paragraph on it to the article in the Sample app section a while back ...

The sample demonstrates use of EF Core to handle optimistic concurrency. However, native database-generated concurrency tokens aren't supported for SQLite databases, which is the database provider for the sample app. To demonstrate concurrency with the sample app, adopt a different database provider that supports database-generated concurrency tokens (for example, the SQL Server provider).

To confirm if your report is correct on that point about what the product unit sent over in the sample app, you would need to use a different provider that can take advantage of the automatic tokens ... OR ... do what you did (I think) and set up the concurrency manually and not try to rely on the automatic tokens.
Perhaps, what I should do here is take that paragraph and make a section out of it, which should help readers discover it.
... and then I'm not sure if ...

using the OnParametersSet, such that when the DbModel changes the property is checked again

... is a separate problem or not.
Pinging @mikekistler to take a look at this issue; and although he's no longer working Blazor issues, this is a sample that was originally provided by @JeremyLikness, and he might have a sec to remark on this.

Pinging @mikekistler again to take a look at ...

using the OnParametersSet, such that when the DbModel changes the property is checked again

Note that on the other point about the sample not working OOB for concurrency that it uses SQLite OOB, which doesn't support database-generated concurrency tokens. Presumably, changing to SQL Server, for example, will result in the concurrency feature lighting up in the app. The article already covers it, but I'm going to make a section out of the remarks to make it easier for readers to discover this point.

If he knows, sure.
I think I probably should just use SQL Server for the app to light-up the concurrency feature ... and then it should be easy to confirm if the ConcurrencyField component should be using OnParameterSet instead of OnInitializedAsync.
Not sure when I'll reach this. I'm still working on the WASM+Identity 2FA/TOTP stuff, and I still have a handful of .NET 9 items that I'd like to address before we bug out for the 🦃 day holiday.

UPDATE (12/5): I'm working on getting back to this issue, but we have internal projects going on and a backlog of higher-priority items + holiday vacations + exhausted folks 😩. I'll get back here ASAP 🏃♂.


[OMGUbuntuBridge] a bridge for omgubuntu.co.uk

Added a bridge for omgubuntu.co.uk .

When I enter omgubuntu.com, it redirects to Ubuntu.com. Is that ok?

When I enter omgubuntu.com, it redirects to Ubuntu.com. Is that ok?

oh woops, its actually https://www.omgubuntu.co.uk/ :|
ill fix that real quick

Is https://www.omgubuntu.co.uk/feed not enough?

Is https://www.omgubuntu.co.uk/feed not enough?

No, because it does not give the full articles.

There is an article called "New PinePhone with 3GB RAM and USB Dock Goes on Sale". It has iframe to youtube video.
https://feed.eugenemolotov.ru/pr1675/?action=display&bridge=OMGUbuntu&format=Html
Iframe is escaped in feed. Could you replace iframe with something like this?
See the PinePhone’s new dock in action

About youtube iframes.
You can make more simpler way like you did in RaceDepartmentBridge. Somewhen later there will be helper to add youtube video links with ease.

Wordpress bridge works, so I'm just using that.


Make int.parse throw on overflow

From @lrhn
"More importantly, we should fix int.parse to throw on overflow instead of (probably?) truncating to 64 bits."

The VM is correctly parsing integers in the -2^63..2^63-1 range, throwing on overlarge values, and accepting "0x..." literals in the -2^63 .. 2^64-1 range.
Dart2js just uses the underlying JS double parser  which does not throw on values that cannot be correctly represented. I would prefer if we could treat int.parse the same way as integer literals and make it an error if the exact numeric value of the numeral cannot be represented by the int class.

I think detecting 64-bit overflow on JavaScript is the wrong choice. It breaks the invariant ...
int.parse(n.toString()) == n

... by introducing an unnecessary exception on large but exact numbers.
I have more sympathy for detecting rounding during parsing, but that also has problems.
int.toString() may be rounded to reduce the number of non-zero digits, e.g.
36028797018963968 prints as<PHONE_NUMBER>8963970. Demanding xxx68 from parsing would also break the invariant.
We can't change the formatting of int without changing the formatting of double - in a dynamic context there is no way to tell if the value is int or double.
We have feedback that int.parse is too slow, so I don't see being able to add much in the way of additional checking.

I agree that dart2js shouldn't necessarily throw for values above 2^63.
I do think it should catch inexact values, but I can see the problem arising from the toString not being exact for integer values, even when not using exponential notation. Another example like:
print(9223372036854774784); //<PHONE_NUMBER>854775000

It's useless if int.parse(9223372036854774784.toString()) throws.
I guess there is no good solution (short of a very inefficient int.toString implementation).
Doubles! Can't live with them ... 'nuff said.


Retry strategies

Description
Still need to do further testing but putting this up before that to get feedback on the direction
This was motivated by the "model overloaded" errors that OpenAI has a habit of throwing every now and then.
Adding a customizable retry strategy abstraction which can be attached to a LLMPredictor wrapper class.
Default behavior is to not do any retrying (i.e. use the NoRetryStrategy strategy). I was tempted to make ExponentialBackoffRetryStrategy be the default strategy but I figured that may incur unexpected additional API billing charges.
The ExponentialBackoffRetryStrategy uses tenacity for the implementation of the exponential backoff algorithm.
Type of Change
Please delete options that are not relevant.

[x] New feature (non-breaking change which adds functionality)
[x] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[x] Added new unit/integration tests
[ ] Added new notebook (that tests end-to-end)
[x] I stared at the code and made sure it makes sense

Suggested Checklist:

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

@sourabhdesai I'm guessing we decided to go through with this? Feel free to close the PR if not! 👍🏻

@sourabhdesai going to close this for now -- we added tenacity back to the OpenAI LLM class recently


Huge increase in NativeIoExceptions after upgrade to 4.1.5.Final from 4.1.0.CR2

After upgrade from netty version 4.1.0.CR2 to 4.1.5.Final we have started seeing a lot of
NativeIoException syscall:read(...)() failed: Connection reset by peer
I am not sure how to reproduce this as this seems to occur only on high load.
netty version : 4.1.5-Final
java version: 1.8.0_u20
openssl version: 1.0.1k
linux version: 3.16.0-4-amd64 Debian

This exception is expected if your peer sends a TCP RST. Can you verify that using a packet sniffer? Also is there an issue caused as a side effect of this behavior?

I will try that out. But these issues did not come up in the older version and with the upgrade not much but a few classes that netty had deprecated have been changed with respect to the code.

tcpdump -ilo -n -v 'tcp[tcpflags] & (tcp-rst) != 0'
a few RST packets are going across but that does not equal the rate of Exceptions. Both the server and client are netty based. Why would the RST packet be sent when a close is initiated ?

Setting SO_LINGER to 0 is one way to cause a RST (see [1] for some extra details).
Can you provide a reproducer?
 Sorry about that :(

@jknair thanks for closing the loop! :)


New plugin: download files before starting the child process [JENKINS-39237]

In Jenkins project we use this library for managing Windows Agents/Slaves. It's being done by launching a slave.jar via Java. Since the remoting library (effectively slave.jar) is being updated frequently, we would like to have an option to download it from the Jenkins instance directly.
For such purpose I propose to implement a WinSW plugin
Jenkins project issue: JENKINS-16490

Hi Oleg,
I’m currently investigating a Jenkins setup for our company. I have one Windows slave configured and when I upgraded recently to the new Jenkins release the Windows slave turned offline because the slave was to old. I found this project here on GitHub and it perfectly meets our requirements to update the slave.jar automatically before the service on the Windows slave is started.
In the configuration file for the service I used the option
<download from=“https://jenkins.corp/jnlpJars/slave.jar” to=“%BASE%\slave.jar”/>
but I received an authentication error.
The Jenkins server is using the reverse proxy plugin for authentication in combination with Kerberos authentication via mod_aut_kerb. The connection to the master is working but the download fails.
My question is: how the download is working, I assume that it is Java based?! If yes, is there a possibility to pass the required Java options for GSSAPI authentication to the download?
Thanks!

@nightman68
Hi Torsten,

how the download is working, I assume that it is Java based?!

No, it is .NET-based. The code is here: https://github.com/kohsuke/winsw/blob/929b87c383ab1ac89afff1d2ed02162be49aaddc/src/Core/WinSWCore/Download.cs . So it just downloads via the GET request using the specified from field, no other options. In this ticket I consider extending the flexibility of the referenced class

The Jenkins server is using the reverse proxy plugin for authentication in combination with Kerberos authentication via mod_aut_kerb. The connection to the master is working but the download fails.

"jnlpJars/slave.jar" is a publicly accessible endpoint, so you should not need authentication. Likely the issue is with the self-signed certificate. Likely you will need to somehow register it in the system

Hi Oleg,
thanks for the quick response!
I configured the Apache to require authentication for all content. I know that it is recommended to leave some content open but we have constraints and this behavior keeps the Apache configuration simple - my POV.
I will try to check what the code gives for opportunities to add an optional authentication and will come back later on.

Likely it will be done as an enhancement of the existing download configuration entry. #195 and #197 are the pending changes

I assume it is closed in 2.1.0
Other enhancements can be submitted as separate issues

Hi Oleg, what Jenkins release will include this enhancement?

@nightman68 there is no explicit request in Jenkins for that. If you needs it for Windows master or Agents, could you please create an issue in JENKINS JIRA and assign it to me? https://issues.jenkins-ci.org/secure/Dashboard.jspa, my account ID is oleg_nenashev.
If you have no account in JIRA, I can create it on your behalf. I just need to understand reason why the update is a blocker for you. Is it because of the automatic WinSW "downgrade" for agents (JENKINS-43603)?

@oleg-nenashev it's not a blocker but I applied 2 or 3 times an upgrade to my master and every time the Windows slave was set to offline because the release was to old. May be is missed something somewhere in the documentation but I had to upgrade the slave.jar manually?! This is not the prefect solution for an deployment in my company. I want a behavior like for the Linux slaves which are upgraded automatically. Makes this sense?
I have an account for Jira and can open an Improvement but I'll wait for your feedback.

@nightman68 There was no automatic update of slave.jar till 2.50: https://jenkins.io/changelog/#v2.50 . In that version I enabled auto-update by default for new agent installations. Here is upgrade guide for old agents: https://github.com/jenkinsci/windows-slave-installer-module#upgrading-old-agents
In jenkins auto-update is powered by download, but there is no need in authentication there if it is not behind proxy. Anyway, maybe it makes sense to update for the latter case

@oleg-nenashev Sorry, may be I was not precise enough. Everything is fine so far but how I mentioned in my second post here:

I configured the Apache to require authentication for all content. I know that it is recommended to leave some content open but we have constraints and this behavior keeps the Apache configuration simple - my POV.

I configured my Apache (reverse proxy plug-in + Kerberos) in that way that every access needs authentication - so the download option in the XML file needs the auth= parameter. This was the purpose for the enhancement requested by me. And now it works as expected by me.
I will open an enhancement, when it will be integrated...
BTW: is an update/enhancement of this Wiki  for the slave configuration welcome? I made this here because our Subversion servers also forcing Kerberos auth.


[Bug] Randomly got an error message on a running cluster

I have a 5-node Raspberry Pi cluster (ARM64) with Raspbian OS (64-bit). It has been running fine for a few days but suddenly csi-plugin pod on one of the nodes started crashing in loop. I got the following stack trace from the node-driver-registrar pod -
fatal error: index out of range
runtime: panic before malloc heap initialized

runtime stack:
runtime.throw(0x653dcd, 0x12)
	/usr/local/go/src/runtime/panic.go:1117 +0x54 fp=0x7fd72b23f0 sp=0x7fd72b23c0 pc=0x435b4
runtime.panicCheck1(0x608e4, 0x653dcd, 0x12)
	/usr/local/go/src/runtime/panic.go:34 +0xc0 fp=0x7fd72b2420 sp=0x7fd72b23f0 pc=0x40b30
runtime.goPanicIndexU(0xee678000ee665, 0x13e490)
	/usr/local/go/src/runtime/panic.go:91 +0x3c fp=0x7fd72b2470 sp=0x7fd72b2420 pc=0x40c8c
runtime.moduledataverify1(0xa7f100)
	/usr/local/go/src/runtime/symtab.go:546 +0x4e4 fp=0x7fd72b2560 sp=0x7fd72b2470 pc=0x608e4
runtime.moduledataverify(...)
	/usr/local/go/src/runtime/symtab.go:527
runtime.schedinit()
	/usr/local/go/src/runtime/proc.go:633 +0x4c fp=0x7fd72b25c0 sp=0x7fd72b2560 pc=0x46c6c
runtime.rt0_go(0x7fd72b3575, 0x7fd72b3590, 0x7fd72b3596, 0x7fd72b35b2, 0x0, 0x7fd72b3603, 0x7fd72b3645, 0x7fd72b3668, 0x7fd72b3699, 0x7fd72b36af, ...)
	/usr/local/go/src/runtime/asm_arm64.s:82 +0xa8 fp=0x7fd72b25f0 sp=0x7fd72b25c0 pc=0x74938
root@rpi-rnd-master:/var/lib/kubelet/plugins# kubectl -n longhorn-system logs longhorn-csi-plugin-2cvxt node-driver-registrar
fatal error: index out of range
runtime: panic before malloc heap initialized

runtime stack:
runtime.throw(0x653dcd, 0x12)
	/usr/local/go/src/runtime/panic.go:1117 +0x54 fp=0x7fd72b23f0 sp=0x7fd72b23c0 pc=0x435b4
runtime.panicCheck1(0x608e4, 0x653dcd, 0x12)
	/usr/local/go/src/runtime/panic.go:34 +0xc0 fp=0x7fd72b2420 sp=0x7fd72b23f0 pc=0x40b30
runtime.goPanicIndexU(0xee678000ee665, 0x13e490)
	/usr/local/go/src/runtime/panic.go:91 +0x3c fp=0x7fd72b2470 sp=0x7fd72b2420 pc=0x40c8c
runtime.moduledataverify1(0xa7f100)
	/usr/local/go/src/runtime/symtab.go:546 +0x4e4 fp=0x7fd72b2560 sp=0x7fd72b2470 pc=0x608e4
runtime.moduledataverify(...)
	/usr/local/go/src/runtime/symtab.go:527
runtime.schedinit()
	/usr/local/go/src/runtime/proc.go:633 +0x4c fp=0x7fd72b25c0 sp=0x7fd72b2560 pc=0x46c6c
runtime.rt0_go(0x7fd72b3575, 0x7fd72b3590, 0x7fd72b3596, 0x7fd72b35b2, 0x0, 0x7fd72b3603, 0x7fd72b3645, 0x7fd72b3668, 0x7fd72b3699, 0x7fd72b36af, ...)
	/usr/local/go/src/runtime/asm_arm64.s:82 +0xa8 fp=0x7fd72b25f0 sp=0x7fd72b25c0 pc=0x74938

Everything went back to normal after restarting the node and it hasn't happened again since then.

@Prakhar225 What's your kubernetes and longhorn version?

Hi Derek, sorry for the late response. Here are the version details -

Kubernetes Server Version - {Major:"1", Minor:"23", GitVersion:"v1.23.9+k3s1", GitCommit:"f45cf3267307b153ed8b418ae5b8ea6c6b9ebaca", GitTreeState:"clean", BuildDate:"2022-07-19T00:40:44Z", GoVersion:"go1.17.5", Compiler:"gc", Platform:"linux/arm64"}
Longhorn/Longhorn-CRD version -  {Chart: "longhorn-100.1.2+up1.2.4", App: "v1.2.4"}


Update storing-values.md

seems more logical this way (at least for me).

I don't see how removing the sentence makes the docs clearer. Closing it for that reason.


[Complex Component] Footer #1

credit: https://www.pinterest.com/pin/494059021629174635/

https://dribbble.com/shots/3925978-Online-Shop-Footer-Design


Slimfast™

Cuts a lot of fat out of GUI
Also, can we please stop having i2c panic?  Please see the code for my proposed solution

I hate computers.

@nicholastmosher you'd think that, but it's zero indexed and starts with the first argument passed, which is usually either cargo or ./<binary>


Upload the same file content to the same hatrac namespace generate multiple files on server

I updated a file to "Protocol.Figure" and then delete then re-uploaded. I would expect to see only one file on hatrac, but it seemed like multiple files were created on hatrac most of them have 0 byte but with the same filename.
https://staging.gudmap.org/chaise/record/#2/Protocol:Figure/RID=W-R0BC

@karlcz confirmed that it is the frontend that started an upload job that got aborted or never got cleaned up.. I could see this consistently  on rbk-staging. However, staging doesn't have the latest code.

The changes to fix this issue have been merged into master. Reassigning to @hongsudt  to verify it no longer happens and close this issue.


vr video example file show "Error video load error"

Hi there
I just download the code from here, and unzip it in my disk, I do nothing else , the panoramic picture sample page  shows correct while video sample page shows "Error video load error".
I tried open sample file directly from windows explorer with Chrome , and I tried set it up in my IIS web server, the result are the same. I tried both in PC Chrome browser and Mobile (Android) Chrome browser, also result are the same.
Were there any mistake or miss step ?
BTW,  I can see the video preview in Adobe Dreamweaver Editor (real mode).
Waiting Online,

Sounds like you're tryihng to do this from file://, you need to be running on http:// URL.


[BLOCKER]Consents are displayed as unchecked if logout from any form page

Environment Details
Spartacus: epic/anonymous-consents branch
Browser: chrome
Device: mac
Happens on: http://dev-com-14.accdemo.b2c.ydev.hybris.com:4200/electronics-spa/en/USD/
Steps to Reproduce

On homepage, scroll down and click on "Consent Preferences"
Click on Accept All.
Click on Sign/Register link and Register a new user.
Login with that user.
Go to Consent Management Page -> All 3 consents are checked as expected.
Logout

Actual Result

Minor Issue: You will see the consents unchecked for 1 second, right before redirecting to the homepage.
Big issue is: If you login again after that, and go to Consent Management page, you will see the contents as unchecked (when they are checked in the backend).
If you try to check one of them, you will get an error msg that "it already exists" since it is already given in the backend.

This is in fact a blocker issue. Consent management functionality cannot be used given consents cannot be granted nor withdrawn. Which is not in compliance with GDPR.

Please test again on the server, I deployed a fix.

Tested on dev14. Works now!


Make nullmove use R=3.

Because we need to reduce depth by one for the recursion, we're currently really only using R=2. It's not the 1990's any more, so use R=3.
5+0.5 test:
Elo difference: 18.9 +/- 10.3, LOS: 100.0 %, DrawRatio: 41.4 %
SPRT: llr 2.95 (100.2%), lbound -2.94, ubound 2.94 - H1 was accepted

Thanks.


enhancement:  Be able to show customized attribute from User

Hello.
On user-selector.blade.php you use the 'name' attribute to show available users.
In my system it comes blank, because I don't save names in User model.
I would like to suggest a change, we could be able to specify what user attribute we want to display in selector, for example on my case I would like to show 'username' other people could prefer 'email' etc.
It would be nice to have this option in configuration file.
Thank you.
Robson

Yeah, we talked about simply publishing the view so you could change it however you see fit.

Something personal, I do not like to have several views published in my resources folder.. Mainly because the package is not loaded in production so there is no reason to have views published. I would still believe that a simple variable in config to set the attribute we wish to show would be the best solution, but again only something personal.
Thank you.


Changed Office Team to Office Staff

@abhishekshree Actually my terminal is not working so I made the changes online only and due to that I got a little confused and merged it with master branch only.

Closed in #18


Encoding Error, Portuguese Characters [ç ã] are not being displayed properly on the question.

This problem is showing up when I execute the function, these characters get corrupted, I'm not well informed about encoding, but I suppose the default encoding 'utf8' should work fine with theses characters. This is the Code I was Trying to run.

const leia = require('readline-sync');

leia.setDefaultOptions({ encoding: 'utf8' });
function opcoes() {
  let resp = parseInt(leia.question("Qual operação você deseja?\n1:Inserção\n2:Remoção\n3:Busca\n4:Percorrer\nDigite: ", { encoding: 'UTF-8' }));
  return resp;
}

I even tried using the setDefaultOptions from the Documentation, Although I did not have success.
Is this a problem of Windows  CMD ? because I use it without any plugin.  I hope it's just a bad configuration option or if I just done goofed.
EDIT #1

That's the result when executed.

I was doing somemore testing, when I print the same "query" line on console.log it shows up correctly. I dunno what's going on

Hi @Werner1201, thank you for your comment.
That seems a problem of the CMD.
These may help you:
https://github.com/anseki/readline-sync/issues?q=is%3Aissue+encoding

So, I saw a solution of using a console.log inside the question. how would I do to make the input prompt to not be one line down ?

Well, since the problem was Kinda Fixed by that, I think I'm done. Thanks for the Help, A I'm just waiting for your anwser on the previous commnent

I'm glad if I could help you. :smile:
So, could you close this issue if it was solved?

Yes, I could


⚠ Syncthing-KA-BI has degraded performance

In 1f562d2, Syncthing-KA-BI ($URL_SYNCTHING_MERZ_NIMBUS_BI) experienced degraded performance:

HTTP code: 200
Response time: 16294 ms

Resolved: Syncthing-KA-BI performance has improved in 08cbd56 after 7 minutes.


[CARBONDATA-2459][DataMap] Add cache for bloom filter datamap

Loading bloom filter from bloomindex file is slow. Adding cache for this procedure will surely improve the query performance. In this commit, the cache is implemented using guava cache instead of cache implemented by carbon. The reason is that there will be a cyclic dependency between module carbon-core and carbon-bloom if we use carbon cache in cabon-bloom.
Be sure to do all of the following checklist to help us incorporate
your contribution quickly and easily:

[x] Any interfaces changed?
NO
[x] Any backward compatibility impacted?
NO
[x] Document update required?
NO
[x] Testing done
Please provide details on
- Whether new unit test cases have been added or why no new tests are required?
No
- How it is tested? Please attach test report.
Tested in 3-node cluster.
- Is it a performance related change? Please attach the performance test report.
Query performance is enhanced
- Any additional information to help reviewers in testing this change.
NO
[x] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.
NA

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/4689/

SDV Build Success , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/4892/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/5846/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/4690/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/5849/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/4693/

retest this please

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/4695/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/5851/

LGTM


cinder can be disabled

Allow for cinder disablement.
It will be enabled by default which matches the current behavior.

This looks good to me. Unsure why tests are failing though.


Warning: TT: undefined function: 22 &32

For some specific document i am getting the console warnings:

Warning: TT: undefined function: 22
Warning: TT: undefined function: 32

Can you attach the PDF here?


🐛 [BUG]: Canvas captures scroll even when all scroll-related features are disabled

Is there an existing issue for this?

[X] I have searched the existing issues and this is a new bug.

Current Behavior
When all scroll behaviors are disabled (pinchToZoom, scrollToZoom, scrollToPan), and a canvas is inserted in a scrolling document, scrolling the page such that the pointer enters the canvas boundaries will immediately halt the scroll
Expected Behavior
Scrolling should not be halted if the scroll event is not used by vue-flow to perform some action
Steps To Reproduce

Disable all scroll-related functionality
Scroll the mouse over the canvas

Relevant log output
No response
Anything else?
My thought is that this code should probably only run and "trap" the event if the handler detects that some action should be performed based on the event and current settings. As it stands this issue can be worked around somewhat by adding nowheel to the entire canvas, but this only works if it's desired that all events are disabled. if the settings are adjusted so that scroll-to-zoom only happens on cmd/ctrl press, then it's not possible to prevent this issue with nowheel.

 I wasn't aware of that prop, is it recent? Let me try it out.

Not really, has existed for more than a year now but isn't really that well documented 😅

@bcakmakoglu I don't believe that prop does what I'm looking for. in fact, it doesn't seem to have any effect whatsoever in my app. As a more concrete example, go to https://vueflow.dev/examples/ and add :zoom-on-scroll="false" then place the mouse outside the canvas (above or below it) and try to smoothly scroll the page. if you end a scroll with the mouse over the canvas, all further scrolls get eaten by the canvas even though they aren't used.
In my app, I have a rather thin + wide canvas which is inline on a longer page. so scrolling this page becomes pretty annoying because to a user it seems like their scroll wheel just stops working, since the canvas is stopping all scroll events regardless of whether or not it ends up using them to produce an effect.

What you describe is exactly the use-case of preventScrolling. You need to set it to false in order for it to not stop scrolling the page when you hit the VueFlow container with a wheel.
It is what's preventing any zoom/pan on scroll when you open vueflow.dev (the top flow with the little buttons to go to different sections of the docs).  I was setting it to true. But this still doesn't do exactly what I want - it does allow the page to scroll without interference, but it can't be combined with :zoom-on-scroll="false" and zoom-activation-key-code="Ctrl" in order to have no scroll action by default, but still allow scroll-to-zoom with a modifier. That is what I really want. Is there a way to do that?

You could track if the Ctrl key is pressed and toggle preventScrolling.

I guess, but my point is why should I have to do that? why is the library calling preventDefault on events it doesn't use? surely that would be a better solution. I would have to duplicate quite a bit of the code inside the library relating to OS detection and key defaulting which is quite fragile.

I see your point I'm just giving workarounds until it's fixed 😄

ah ok 😄 I am ok with waiting for a fix, appreciate it!

Ah, I wondered what the hold up was. Is there some counter argument they have? Is the discussion public?

Not a public discussion no, the hold up is that the devs haven't gotten back to me yet 😅
Pinged recently though I'll ask next week again ^^

Alright, so the decision about this is that the event should not be caught ^^
Will be fixed in the next patch or minor version (soon)

I'll be releasing a patch for this issue over the weekend 👍

Fixed with 1.33.7


Shadow copying and Failing to load e_sqlcipher.dll

We have a dll that is a task and gets executed in another application that shadow copies the dlls. It is a .net framework application. Is there another way to load the dll or is the only way to disable shadow copying on the app domain?

That's the only solution I know of, but I suspect there is a workaround.  But my depth in that particular area (.net framework and shadow copying) is weak.


[Bug]: Cannot convert text to speech from tts tab

Project Version
Latest version of the project
Platform and OS Version
Window 10
Affected Devices
Window 10
Existing Issues
No response
What happened?
I can't do conversion from the model I trained. They are not showing error on screen. However, in CMD they are showing error without "voice". I think this is because they are not being taken from the model I trained when I left blank "Select the TTS voice to use for the conversion." Maybe I did something wrong, I tried to make modification to take from model but I don't have enough knowledge to do that. Please give me feedback. Best regards!
Steps to reproduce
Attached with video. Model is taken directly from the link you provided.
Expected behavior
An error occurred connecting to Discord: Could not find Discord installed and running on this machine.
Theme Applio successfully loaded.
Running on local URL:  http://<IP_ADDRESS>:6969
To create a public link, set share=True in launch().
Traceback (most recent call last):
File "E:\mywork\Applio\rvc\lib\tools\tts.py", line 20, in
asyncio.run(main())
File "E:\mywork\Applio\env\lib\asyncio\runners.py", line 44, in run
return loop.run_until_complete(main)
File "E:\mywork\Applio\env\lib\asyncio\base_events.py", line 647, in run_until_complete
return future.result()
File "E:\mywork\Applio\rvc\lib\tools\tts.py", line 15, in main
await edge_tts.Communicate(text, voice, rate=rates).save(output_file)
File "E:\mywork\Applio\env\lib\site-packages\edge_tts\communicate.py", line 279, in init
raise ValueError(f"Invalid voice '{voice}'.")
ValueError: Invalid voice 'None'.
Converting audio 'E:\mywork\Applio\assets\audios\tts_output.wav'...
An error occurred during audio conversion: An error occurred loading the audio: File not found: E:\mywork\Applio\assets\audios\tts_output.wav
Traceback (most recent call last):
File "E:\mywork\Applio\rvc\lib\utils.py", line 61, in load_audio_infer
raise FileNotFoundError(f"File not found: {file}")
FileNotFoundError: File not found: E:\mywork\Applio\assets\audios\tts_output.wav
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "E:\mywork\Applio\rvc\infer\infer.py", line 261, in convert_audio
audio = load_audio_infer(
File "E:\mywork\Applio\rvc\lib\utils.py", line 81, in load_audio_infer
raise RuntimeError(f"An error occurred loading the audio: {error}")
RuntimeError: An error occurred loading the audio: File not found: E:\mywork\Applio\assets\audios\tts_output.wav
Temporary MP4 file saved as: C:\Users\admin\AppData\Local\Temp\tmpu5cogwpl.mp4
Exception in callback _ProactorBasePipeTransport._call_connection_lost(None)
handle: <Handle _ProactorBasePipeTransport._call_connection_lost(None)>
Traceback (most recent call last):
File "E:\mywork\Applio\env\lib\asyncio\events.py", line 80, in _run
self._context.run(self._callback, *self._args)
File "E:\mywork\Applio\env\lib\asyncio\proactor_events.py", line 162, in _call_connection_lost
self._sock.shutdown(socket.SHUT_RDWR)
ConnectionResetError: [WinError 10054] An existing connection was forcibly closed by the remote host
Attachments
No response
Screenshots or Videos
recording.webm

Additional Information
No response

how can you have your pudding if you don't eat your meat?
It tells you what the problem is. You can not do TTS without selecting a TTS voice. You can not use a voice in a language that does not match the language of the text you provide.


Add ability to save text to localStorage

Enable saving the current text to localStorage.
We probably want to add an option to create a new note that clears the current text too.

i would like to work on this


relax pandas constraints

Closes #221

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Can not register multiple components

I have two components (Component1 and Component2). Sometimes I need both of them on the same page, sometimes just one.
I don't want load scripts that I do not need on the page. So, I separated load files like this:
javascript/packs/component1_bundle.jsx
import ReactOnRails from 'react-on-rails';
import Component1 from '../bundles/components/Component1';

ReactOnRails.register({ Component1 });

javascript/packs/component2_bundle.jsx
import ReactOnRails from 'react-on-rails';
import Component2 from '../bundles/components/Component2';

ReactOnRails.register({ Component2 });

Then, when I need both components on the page, I do this:
= javascript_pack_tag 'component1_bundle'
= javascript_pack_tag 'component2_bundle'
= react_component("Component1")
= react_component("Component2")
The second register overrides the first one, and I have this error:
Uncaught Error: ReactOnRails encountered an error while rendering component: Component1.
Original message: Could not find component registered with name Component1.
Registered component names include [ Component2 ]. Maybe you forgot to register the component?

There is no error and everything works fine when I register components in the same file:
javascript/packs/component1_and_component2_bundle.jsx
import ReactOnRails from 'react-on-rails';
import Component1 from '../bundles/components/Component1';
import Component1 from '../bundles/components/Component2';

ReactOnRails.register({ Component1 });
ReactOnRails.register({ Component2 });

How to register components separately, only ones that I really need?

Same issue. Surprised this didn't work out of the box.

@chumakoff Did you try to extract vendor bundle into separate chunk?
You can check Webpack configs from tutorial rails app to understand how to do it.

@railsme No, I didn't. And I can't understand how to do that.

@chumakoff Prevent code duplications in webpack

@chumakoff You can create separate bundles. I don't think the rails/webpacker way supports this, but I could be mistaken.
@railsme is correct on this issue.

@chumakoff
https://github.com/rails/webpacker/blob/master/docs/webpack.md#add-common-chunks

There seem to be two separate issues here:

Using ReactOnRails.register globally in a way that multiple invocations across bundles work together so that the last call doesn't clobber previous ones.
Creating webpack chunks to avoid code duplication across multiple bundles.

Some comments here are addressing (2), which is indeed related, but (1) seems to be the proximate issue from the original poster.

This issue appears to be that, by doing import ReactOnRails twice, you are creating two separate internal component stores. I was able to get around this by doing the following, which is not ideal, but keep in mind ReactOnRails exports itself to window. anyways:
window.ReactOnRails = window.ReactOnRails || require('react-on-rails').default;

import Component1 from '../bundles/components/Component1';

ReactOnRails.register({ Component1 });

window.ReactOnRails = window.ReactOnRails || require('react-on-rails').default;

import Component1 from '../bundles/components/Component2';

ReactOnRails.register({ Component2 });

And then this works:
= javascript_pack_tag 'component1_bundle'
= javascript_pack_tag 'component2_bundle'

= react_component("Component1")
= react_component("Component2")

Hope this helps others who are running into the same problem, and just need to get this working without restructuring their webpacker configuration or bundle setup.

@rubiety are you using Webpack v4? I'm thinking that this maybe should be handled at lower level. Nevertheless, thank you for posting this! If you like, consider adding a file to the docs directory in a PR.

Is this still the way to make this work? I'm running into the same issue.


Storefront: Investigate Sanity CMS

I want to test Sanity to make an estimation on how much work it would take to implement it on the Storefront.

Data structure and schema
Next.js

How does it integrate
Test live preview

How easy is it to use the interface for editors
Media library and file storage
How does using reusable blocks / components work in the page builder
Pricing model
Deployment and datasets
How does menus work and linking pages to it
What field types does it support

I tested using Sanity with Next.js in a small POC and had an overall really great experience.

It was easy to work with the schema system to create the data structure, and it worked fine for what we need with the Storefront.
The new live preview system was a bit buggy. It worked most of the time but sometimes didn't sync the data poperly from the editor UI to the website preview. This will get better over time.
I liked the interface / UI that is presented to the editors to create content. It is very easy to use and it has everything we need.
The media library and file storage seemed to work just fine.
The pricing seemed very reasonable with 15 dollars per seat per month.
There are 2 datasets (databases) available that I think will work fine for what we want to do.
Deployments are super easy for both the source code and for updating the database from the UI.
Creating static menus with Sanity worked just fine. I had a bit more trouble when the menus had to be dynamic based on the document structure. More research has to be done on this, it seems possible.
Sanity has all the field types that one would need. Texboxes, images, checkboxes and so on.


Enhancements

Thank you for this library. I would prefer this one over medici because I think the use of a relational database is more sane for accounting purposes than using a NoSQL DB.
But there is one drawback with this library which makes it not useable for my purposes: I can't store and query metadata with the journal entries like with medici.
Looking at your example makes no sense to me:
myBook.balance({ account:'Assets:Accounts Receivable' }).then((balance) => {     console.log("Joe Blow owes me", balance.total); });
When I look this up in medici, it makes sense again:

In the journal entry, I can store metadata like the client
.credit('Income', 1000, {client: 'Joe Blow'})
And then I can use the metadata for the queries
myBook.balance({  account:'Assets:Accounts Receivable',  client:'Joe Blow' }).then((balance) => {  console.log("Joe Blow owes me", balance); });

Another thing is the REST service. I really like the idea of having a kind of microservice for this purpose. But the drawback here: In the REST service I can't do everything which I can do with the API, e.g. Voiding Transactions.
Thanks again for this project.

Any thoughts on this?

Hi.. So unfortunately I don't have time to do any work on this project at the moment and probably won't have time for several months.
It looks like you have a good idea of how to add the feature you want. Why not implement it yourself and submit it as a PR?


Show exact timestamp for alerts too

Hi @prymitive,
thank you for your work on Karma!
I like the human readable date/time indication for alerts (e.g. 1 day ago); some times when investigating alerts I also find it useful to know the exact timestamp. To avoid clutter I'm thinking it could be a tooltip shown on hover of the timestamp, what do you think ?

thank you!

Not stale, waiting for feedback

I think it would be really useful to have a tooltip with ISO 8601 like timestamp with seconds resolution ("2023-01-10 02:04:25 UTC"), and even better show both UTC and local (browser) timezone as two line.
When communicating with others, or building reports, it is often very desirable to pinpoint the times of events quickly. I know all this can be extracted from ALERTS_* metrics in Prometheus, but it is just more steps, and not actually the time the alertmanager received it.


Problem on transformExtent with Y coordinates

I'm using ol 5.3.3 and trying to transform an extent from EPSG:4326 to EPSG:3857 but it results in a wrong result for Y coordinates while X coordinates are OK :
Ex : ol.proj.transformExtent(["2.224225", "48.815607", "2.469769", "48.902008"], "EPSG:4326", "EPSG:3857")
Result in [247599.59440966893, -5039577.618739517,<PHONE_NUMBER>5701247, -5026806.090671426]
The correct result should be [247599.5944, 6243631.4800,<PHONE_NUMBER>, 6258250.5290]

Extents (and coordinates) are defined as arrays of numbers.  Strings are likely to cause problems elsewhere, not just in transforms.  If you have obtained an array from, for example, splitting a string, a quick fix is
ol.proj.transformExtent(["2.224225", "48.815607", "2.469769", "48.902008"].map(Number), "EPSG:4326", "EPSG:3857")

Thanks for your help, the transform is now correct with an array of numbers.


Change login command to always display login URL

In case the browser fails to open properly users can always refer to the URL printed on the commandline.

@Overv i made the message a bit more explicit, if you're OK with that you can merge it yourself


Documentation related issues, questions and possible work items

This issue is opened to record issues, questions and possible work items when we update onnx-mlir documents.

Are we following docs/Documentation.md now? What's the role of docs/_data/navigation.yml?
What's our plan for the doc check tool?
Merge of CONTRIBUTING.md and docs/Workflow.md
In the workflow, is it required to clone with ssh instead of https in order to create PR? Should we change the document accordingly?
In gen_onnx_mlir.py, there are several ways to generate extraClassDeclaration. Can be consolidated.
In gen_onnx_mlir.py, the customized builder may be avoided by using returnType in rewrite rules.
Restructure gen_onnx_mlir.py following PR#998.

Answers for items for which I have an opinion

Are we following docs/Documentation.md now? What's the role of docs/_data/navigation.yml?

navigation is used for the onnx.ai/onnx-mlir website. This is what we have as the primary reference for our work.

What's our plan for the doc check tool?

check is to ensure that the script that we list in the md page is honest with the scripts. I am in favor of keeping it.

Merge of CONTRIBUTING.md and docs/Workflow.md.

I suggest to merge info that is in Workflow into CONTRIBUTING. CONTRIBUTING is a very standard file to have in GitHub's projects.

In the workflow, is it required to clone with ssh instead of https in order to create PR? Should we change the document accordingly?

Personally, I think it should be required to work on a clone. I don't like it when folks push PRs directly on the main repo. SSH vs something else, I would refer to folks more expert in the field of security. I have no personal preference.

In gen_onnx_mlir.py, the customized builder may be avoided by using returnType in rewrite rules.

I have used custom builders and find this helpful to do very simple preprocessing. For example, when an op has a lot of parameters that are optional with default value, I like to have one with fewer parameters. They are trivial to write. Not familiar with the returnType's approach.

In the workflow, is it required to clone with ssh instead of https in order to create PR? Should we change the document accordingly?

What exactly does this question mean? Why should we worry about how one clones the repo?

Personally, I think it should be required to work on a clone. I don't like it when folks push PRs directly on the main repo. SSH vs something else, I would refer to folks more expert in the field of security. I have no personal preference.

@AlexandreEichenberger What do you mean by "push PRs directly to the main repo"?

7. In the workflow, is it required to clone with ssh instead of https in order to create PR? Should we change the document accordingly?

What exactly does this question mean? Why should we worry about how one clones the repo?

Indeed, https and ssh are just different ways to establish a secure connection to the github repo. They have no bearing on git operations such as pull, push, merge, etc.

Personally, I think it should be required to work on a clone. I don't like it when folks push PRs directly on the main repo. SSH vs something else, I would refer to folks more expert in the field of security. I have no personal preference.

@AlexandreEichenberger What do you mean by "push PRs directly to the main repo"?

I think he meant "work on a clone of a forked repo" instead of "work on a clone of the onnx-mlir repo", which I would agree.
Instead of cloning directly the original onnx-mlir repo and creating and pushing PRs on the onnx-mlir repo, one should create a fork of the original onnx-mlir repo and clone and work on the fork instead. We have privileged users working directly on the original onnx-mlir repo, which is not recommended.

Instead of cloning directly the original onnx-mlir repo and creating and pushing PRs on the onnx-mlir repo, one should create a fork of the original onnx-mlir repo and clone and work on the fork instead. We have privileged users working directly on the original onnx-mlir repo, which is not recommended.

yes, exactly


python311Packages.ax: 0.3.4 -> 0.3.6

Description of changes
failing on hydra
updating to new release https://github.com/facebook/Ax/releases/tag/0.3.6
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

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

4 packages built:

python310Packages.ax
python310Packages.ax.dist
python311Packages.ax
python311Packages.ax.dist

Result of nixpkgs-review pr 279342 run on aarch64-linux 1

4 packages built:

python310Packages.ax
python310Packages.ax.dist
python311Packages.ax
python311Packages.ax.dist


chore: update mdbook, catppuccin, admonish

updated themes using mdbook-admonish/catppuccin install

I overwrote https://github.com/ratatui-org/ratatui-book/pull/108/commits/3c64e1b8dc11a313cdaf3b418d53588fa9057b6b without seeing what it contained. Can you push that to a new PR perhaps?

Did you not want to use cargo make for this?

Did you not want to use cargo make for this?

I didn't see it before force pushing an amended commit, so I can't comment on what was included.
Generally I'd prefer to use xtask over cargo make if given a choice though.

I'm fine with it leaving it the way it is, but if we want to use cargo-make like in the main repo, here's the patch:
diff --git a/Makefile.toml b/Makefile.toml
new file mode 100644
index 0000000..b19a710
--- /dev/null
+++ b/Makefile.toml
@@ -0,0 +1,49 @@
+[config]
+skip_core_tasks = true
+
+[tasks.install-mdbook]
+command = "cargo"
+args = ["install", "mdbook", "--version", "0.4.35"]
+
+[tasks.install-mdbook-admonish]
+command = "cargo"
+args = ["install", "mdbook-admonish", "--version", "1.13.0"]
+
+[tasks.install-mdbook-svgbob2]
+command = "cargo"
+args = ["install", "mdbook-svgbob2", "--version", "0.3.0"]
+
+[tasks.install-mdbook-linkcheck]
+command = "cargo"
+args = ["install", "mdbook-linkcheck", "--version", "0.7.7"]
+
+[tasks.install-mdbook-mermaid]
+command = "cargo"
+args = ["install", "mdbook-mermaid", "--version", "0.12.6"]
+
+[tasks.install-mdbook-emojicodes]
+command = "cargo"
+args = ["install", "mdbook-emojicodes", "--version", "0.2.2"]
+
+[tasks.install-mdbook-catppuccin]
+command = "cargo"
+args = ["install", "mdbook-catppuccin", "--version", "2.0.1"]
+
+[tasks.install]
+dependencies = [
+    "install-mdbook",
+    "install-mdbook-admonish",
+    "install-mdbook-svgbob2",
+    "install-mdbook-linkcheck",
+    "install-mdbook-mermaid",
+    "install-mdbook-emojicodes",
+    "install-mdbook-catppuccin"
+]
+
+[tasks.build]
+command = "mdbook"
+args = ["build"]
+
+[tasks.serve]
+command = "mdbook"
+args = ["serve", "--open"]
diff --git a/src/developer-guide/book.md b/src/developer-guide/book.md
index fa7ff99..dd47c4f 100644
--- a/src/developer-guide/book.md
+++ b/src/developer-guide/book.md
@@ -12,12 +12,14 @@ Feel free to make contributions if you'd like to improve the documentation.
 If you want to set up your local environment, you can run the following:

 ```bash
-cargo install mdbook --version 0.4.30
-cargo install mdbook-admonish --version 1.9.0
-cargo install mdbook-svgbob2 --version 0.3.0
-cargo install mdbook-linkcheck --version 0.7.7
-cargo install mdbook-mermaid --version 0.12.6
-cargo install mdbook-emojicodes --version 0.2.2
+cargo install cargo-make
+cargo make install
+```
+
+You can run the following to view a local copy of the documentation:
+
+```bash
+cargo make serve

These plugins allow additional features.

makes sense - yeah a single command to perform the install would be nice.
We can also achieve that by using a workspace cargo.toml file though with those dependencies called out.


Fix search and replace for imagePullPolicy

Proposed changes
Only replace the values that have a : in the yaml files
Closes #2780
Checklist
Before creating a PR, run through this checklist and mark each as complete.

[ ] I have read the CONTRIBUTING doc
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have checked that all unit tests pass after adding my changes
[ ] I have updated necessary documentation
[ ] I have rebased my branch onto main
[ ] I will ensure my PR is targeting the main branch and pulling from my branch from my own fork

Release notes
If this PR introduces a change that affects users and needs to be mentioned in the release notes,
please add a brief note that summarizes the change.

NONE

https://github.com/nginxinc/nginx-gateway-fabric/pull/2902 should fix the pipeline issue seen here


Stats changes

Still need to support the Long measure/measurement for backwards compatibility with OC
Move Measure creation to StatsRecorder to allow Measure to take a reference to the implementation for optimizations like no recording if Measure is not register with a view.

Fixed in https://github.com/bogdandrutu/openconsensus/pull/200.


[AutoPR compute/resource-manager] Update ResourceIdentity to EncryptionSetIdentity

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/7617

This PR has been merged into https://github.com/Azure/azure-sdk-for-go/pull/6142


chore: go-demo-6 to 1.0.405

chore: Promote go-demo-6 to version 1.0.405

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Feature request: source specific filtering

Is your feature request related to a problem? Please describe
Sometimes you have tags on one source which you don't have on others or have a different tag for the same. Currently you can't filter it for one source and still have the same results for the other sources.
Describe the solution you'd like
I would like to have a solution that you can filter a tag for one source or for example have all sources except one filter that tag. (for example anime-pictures use erotic instead of safe, questionable and explict etc)

Indeed, I've had this in the back of my head for a while.
Not only should tag formats get automatically converted between each other (capital letters, underscores/spaces, etc.), I think this is also necessary. There's already a basic implementation that automatically remove all modifiers (for example rating:safe) if they do not exist on the other source, but there isn't yet any mapping logic.
Good suggestion, it'll definitely happen when the tag re-write system is finished 👍

@Bionus Could you maybe think about this again since I think that now the tag system has been reworked.


Tab Indexing is not consistent in modals

Current Behavior
The tab indexing lacks consistency; pressing the tab key should transition between fields uniformly. Currently, it changes for some fields but not for all.

Expected Behavior

Screenshots/Logs

Environment

Host OS: Mac Linux Windows
Platform: Docker or Kubernetes
Meshery Server Version: stable-v
Meshery Client Version: stable-v

Contributor Guides and Handbook

🛠 Meshery Build & Release Strategy
📚 Instructions for contributing to documentation

Meshery documentation site and source

🎨 Wireframes and designs for Meshery UI in Figma (open invite)
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

@Yashsharma1911 where are the schemas being stored now?
@MUzairS15 the state of our schemas is confusing. Can you rectify this? or can you @theBeginner86?

...

@leecalcote We are currently sourcing schema from Meshkit, and here is schema link of the environment https://github.com/meshery/meshkit/blob/master/schemas/configuration/environment.json


Guys, its done ):

What is your question?
discussions wont come back

):
):

Close this

close this

https://github.com/mayo78/PSYCHDISCUSSIONS/discussions/1


feat: add support to line numbers in markdown

highlight-plugin.js now read global and local { lineNumber: true}.
Added tests and snapshots for the feature.
Receive and validate lineNumber from markdown settings.
Add some CSS for the line numbers.
Document both the markdown config and the inline option.

Summary
What kind of change does this PR introduce? (check at least one)

[ ] Bugfix
[x] Feature
[ ] Code style update
[ ] Refactor
[x] Docs
[ ] Build-related changes
[ ] Other, please describe:

If changing the UI/CSS related code, please provide the before/after screenshot:
Before

After

Does this PR introduce a breaking change? (check one)

[ ] Yes
[x] No

If yes, please describe the impact and migration path for existing applications:
The PR fulfills these requirements:

[x] When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)

You have tested in the following browsers: (Providing a detailed version will be better.)

[x] Chrome
[x] Firefox
[ ] Safari
[x] Edge
[ ] IE

If adding a new feature, the PR's description includes:

[x] A convincing reason for adding this feature
[x] Related documents have been updated
[x] Related tests have been updated

To avoid wasting your time, it's best to open a feature request issue first and wait for approval before working on it.
Other information:
Tried to keep CSS at minimum to maintain the feel of the current styles.
Had to change renderPreWrapper (packages/saber/lib/markdown/highlight-plugin.js) signature since xo limits arguments to 4.
Took minor edits over highlightLines (packages/saber/lib/markdown/highlight-plugin.js) since the current implementation assumed it was the only option.
Hopefully my edits on the docs are to your liking.

Did a small change to make line numbers fixed and support line highlighting.

Looks great to me :)


Update annonars dependency, adjust to ClinVar protobuf changes

Is your feature request related to a problem? Please describe.
We need to update the annonars dependency in mehari. This will trigger some issues in the changed ClinVar protobuf.
Describe the solution you'd like

adjust to these changes
write out VCV instead of RCV

Describe alternatives you've considered
N/A
Additional context
N/A

Is this still relevant?


Performing an adversarial attack on face_recognition software, is it possible?

I am part of a CTF and I have the task of finding out how to bypass a face recognition software by performing an adversarial attack. The base software is written with face_recognition library and it works as it should work. Now, I am curious if it is possible to perform an adversarial attack on this piece of software, how is it possible?
P.S: Code samples are super appreciated.

I am also interested in this issue. Is it possible to perform an attack with, for example, Adversarial Robustness Toolbox (ART)?


cocoapods and swift package manager support

Experimental new analyzers to support SWIFT that we have used for a while internally:

CocoaPodsAnalyzer.java
SwiftPackageManagerAnalyzer.java
handle when both co-exit in DependencyBundlingAnalyzer.java

@biancajiang Thank you for the PR! truly appreciate the enhancements and additions.
--Jeremy


OnRowSelect rerender all components in table

Describe the bug
Hi,
I have problem with row selecting. Every time when I click on new row all components in the table rerender. Probably it looks like problem with using useFormContext from react-hook-form. Is this possible to optimize this behaviour and not refresh all component if it is not needed?
Your minimal, reproducible example
https://codesandbox.io/s/virtualized-form-table-using-react-table-react-window-react-hook-form-gyyym3
Steps to reproduce

Open this example in separate window and set as enable "Highlight updates when components render." in you profiler extention.

Click on any row. Selected row should have red background but with delay. All components rerender.

Expected behavior
Click on row shouldn't refresh all components in the table and row should be highlighted immediately
How often does this bug happen?
Every time
Screenshots or Videos

Platform
Chrome
react-table version
17.0.1
TypeScript version
No response
Additional context
No response
Terms & Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I understand that if my bug cannot be reliable reproduced in a debuggable environment, it will probably not be fixed and this issue may even be closed.

This is just React doing what it's supposed to do. The state for the table lives in the useReactTable hook, so when you update a row selection, the component that houses your table hook instance will rerender, including rerendering any and all child components. Here's my suggestions:

Generally, you want this. This is what keeps your UI in sync with your state.
Consider rendering fewer rows. 100 rows * 10 columns is at least 1000 component instances. Paginate or virtualize your rows if necessary.
Instead of avoiding rerenders, make them faster. If your table components or cell/header templates are slow, your table will also be slow. Memoize where necessary
As a last resort, you can use React.Memo or useMemo to memoize your actual components or their output, but this means that they will only rerender when their dependencies change. Unless you know what you're doing here, you'll likely run into more problems with out-of-sync UI doing this.


Locale for Tigrinya (ti) is not supported

Describe the bug
The locales for Tigrinya are missing.
To Reproduce
Steps to reproduce the behavior:

Requesting the ti locale fails

Expected behavior
The ti locale should be supported.
Desktop (please complete the following information):

OS: any
Browser any
Version any

Smartphone (please complete the following information):

Device: any
OS: any
Browser any
Version: any

Moment-specific environment

The time zone setting of the machine the code is running on: Europe/Berlin
The time and date at which the code was run: 07. Aril 2020
Other libraries in use (TypeScript, Immutable.js, etc): none

require('moment').locale('ti') returns 'en'
Additional context
none

@maxammann  you should create a locale setting with your customizations.this is a very cool feature provided by moment.
https://momentjs.com/docs/#/customization/

Problem is that I do not speak Tigrinya but our app should support it.
Maybe we can find somebody knowledgeable :)

The authors and members of this library may be busy with other work. This is a very awkward situation. Maybe we can use a translator :).

I'll discuss this within our project. Maybe we can do something

I think we can make a proposal and let it check by a professional translator: https://lh.2xlibre.net/locale/ti_ER/

Is there a procedure from the moment.js for adding locales? My current Idea would be to create the locale and then generate sample sentences which can be checked by non-tech professional translators.

You can express your thoughts on gitter : (https://gitter.im/moment/moment?utm_source=badge&utm_medium=badge&utm_campaign=pr-badge&utm_content=badge)

Amharic Tigrinya Speaker here. How can I help

@yosiasz See https://github.com/moment/moment/blob/develop/src/locale/en-ie.js and https://momentjs.com/docs/#/i18n/ . Add Tigrinya internationalization support.

Sure, we'd look at a PR for this.
I closed it in favor of https://github.com/moment/moment/issues/4949


added blog.html

@shivamvishwakarm thanks for the pr but better way is to first comment on issue and i suggest u to give this pr after 1st oct to be counted in hacktoberfest

if u want i will merge it now , let me now


No motion binary_sensor

Description
I don't have a binary_sensor for motion anymore
Reproduction Steps
no binary_sensor entity under device entities
Expected behavior
I don't have a binary_sensor for motion anymore
If applicable, add error logs.
No errors in logs
Device Firmware
1.0 (1.3.6)
Using stream component
Yes
Does camera work via official integrations?
Yes
Camera has all attributes filled out in developer tools
Yes
HASS Environment
Proxmox VM
Search for similar issues
Yes
Additional information
Binary_sensor was removed round about core version 2023.4

Firmware 1.3.6 has broken pull endpoint, thanks to new version webhooks can be used which still work.
Check faq for information.

If you have webhooks enabled, and your Home Assistant internal URL is reachable on HTTP, make sure camera can reach it.
Make sure you have correct IP set for Home Assistant. Turn on Advanced Mode under /profile. Go to /config/network and under Network Adapter verify correct IP is shown for the device. If it is not correct, under Home Assistant URL uncheck Automatic next to Local Network and set it to http://<some IP address>:8123. DO NOT USE HTTPS.
Certain camera firmwares have pullpoint broken, with only webhooks working. If you are not able to run webhooks because of above (https, or vlan setup), binary sensor will never show up.
Try walking in front of the camera


XSS

[naughty link](javascript:alert('xss'))
I believe #109 may be related.

Note that the sanitize config argument will fix this: https://github.com/chjj/marked#sanitize

This is resolved with the sanitize option after:  https://github.com/chjj/marked/pull/592

Note that stripping html is not enough to secure markdown, as explain in Showdown's article :


Migrate duffle driver implementations to cnab-go

The driver interface is already defined in the driver package. Many of the driver implementations from duffle are generally useful outside of duffle and should be made available without the need for declaring a dependency on duffle.

While the APIs in cnab-go are fairly new, this would also help minimize the downstream impacts of refactors like https://github.com/deislabs/cnab-go/pull/28.

I agree with moving the drivers here, as it's a main concept in CNAB implementations, and sharing common code between all CNAB runtime clients will reduce interoperability issues 👍


Forking django-mongodb for DocumentDB Compatibility

Hi there! First, I just want to say that you're doing an amazing job with this library—thanks for all the hard work!
I'm reaching out because I'm using AWS DocumentDB, which only supports around 34% of the MongoDB API. Unfortunately, DocumentDB has limited support for $expr, which makes it challenging to use the library in its current state since recent updates rely on $expr queries.
I understand that $expr migration was an important step for your roadmap, and I definitely don’t want to slow down your progress by asking for special accommodations for DocumentDB. Instead, I'll be forking the repository to adjust it for DocumentDB compatibility.
Would you be comfortable with this approach, or do you have any concerns or advice for me? I’ll keep an eye on your updates and align my fork with future improvements whenever possible. Thanks again for all your efforts!

Thanks for the quick response!
To make it compatible with DocumentDB, we would need to roll back the changes from this merge request (#54), which moved toward $expr-based queries. I agree that rolling back this MR isn't the best idea given the library's MongoDB-focused goals.
With that in mind, would you be okay with me maintaining a fork specifically for DocumentDB compatibility? This would allow me to adjust for DocumentDB's API limitations without impacting the primary project’s development.
Thanks again for your support and guidance!

Would you be comfortable with me maintaining this fork specifically for DocumentDB compatibility?

As long as you comply with the terms of the license that sounds fine. Good luck!

Thanks!


Group presentations 2

Oral presentation of systems specification and demo of release 1

Bravo ! Everyone :grinning:


fabio not honoring strip= when using websockets..

Hello!
I've just had a fun debugging session, and discovered:
When I have a service configured with strip=/foo, and it upgrades the connection, fabio is hitting the backend at /foo instead of /.
I was going to paste some tcpdump showing this, but it's just a verbose way of saying what I've said above :)
Running fabio 1.5.2.
Here are the headers fabio sent with the outbound request to the service:
GET /vertica-websocket-streamer/ HTTP/1.1
Host: <IP_ADDRESS>
Connection: Upgrade
Forwarded: for=<IP_ADDRESS>; proto=ws; by=<IP_ADDRESS>; httpproto=http/1.1
Origin: http://fabio-dev:9998
Sec-Websocket-Extensions: permessage-deflate; client_max_window_bits
Sec-Websocket-Key: hMmX3g/wxm2LMUnC3kuSrQ==
Sec-Websocket-Version: 13
Upgrade: websocket
X-Forwarded-For: <IP_ADDRESS>
X-Forwarded-Port: 80
X-Forwarded-Prefix: /vertica-websocket-streamer
X-Forwarded-Proto: http
X-Real-Ip: <IP_ADDRESS>
Oh, also, fabio isn't logging any of these requests. If I hit it without an Upgrade it will proxy and log the 400 response from the backend (it returns "expecting upgrade request"), but absolutely nothing is logged for upgraded requests - which in this case are returning 404, because the backend is expecting /.

That's no fun. I'm looking.

@manos Good catch and sorry for the trouble. The ws proxy was only using the Host part of the URL which made not obvious that I had to modify the request URL as well. It was literally a one line fix :)
I'll address the logging separately.

Amazing turnaround time, thanks @magiconair  :)


Verification of the transition number of the basic SIR model with the DSL

The transitions of the basic SIR model with the DSL are given by:
transitions: #(
			S -- lambda --> I.
			I -- gamma --> R.)

We check that the number of transitions of the basic SIR model is 2

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

15 of 15 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+1.07%) to 40.091%

Totals

Change from base Build<PHONE_NUMBER>:
1.07%

Covered Lines:
5798

Relevant Lines:
14462

💛  - Coveralls


Organization name

Wox has already been taken.
Any ideas?
https://github.com/qianlifeng/Wox/issues/300

Wox-Launcher?

I have created a Wox-launcher Organization and invited you.

A better name would've been github.com/getwox/wox, but this will suffice too.

@qianlifeng what do you think? The organization name can be changed.

I prefer Wox-launcher :smile:


Pass the request object as the first argument to authenticate()

Django 1.11 started passing the request object to auth backends.
Django 2.1+ requires auth backends to accept the request object as their
first position argument. django-admin-sso should pass the request object
to authenticate() to be a good citizen.
This change allows subclasses of DjangoSSOAuthBackend to add error messages
via Django's messages framework, which requires the request object to work.
Fixes #5

Thanks! 💜

Thank you! Let me know when you push a release to PyPI.

@andymccurdy I have uploaded 3.1 to PyPI a few seconds ago. Thanks again :-)


Inverse_transform raise a ValueError below

Hi Christos,
I have studied this case for my assignment and encountered the ValueError. Could you please let me know how you solved this issue? Thanks!

https://github.com/Housiadas/forecasting-energy-consumption-LSTM/blob/e2b0db686aea8c351beb480b8d32e89af9b31413/lstm/performance.py#L26

ValueError                                Traceback (most recent call last)
 in ()
26 vals = test_df.values.copy()
27 #print(vals)
---> 28 vals = scaler.inverse_transform(vals)
29 print(f'vals: {vals.shape}')
30 print(test_x.shape)
/usr/local/lib/python3.7/dist-packages/sklearn/preprocessing/data.py in inverse_transform(self, X, copy)
1033         else:
1034             if self.with_std:
-> 1035                 X *= self.scale
1036             if self.with_mean:
1037                 X += self.mean_
ValueError: operands could not be broadcast together with shapes (8439,5) (4,) (8439,5)

Hello,
Sorry it's been two years since I last run this code. I don't remember at all.
Good luck!


Add a method  to CLLoader to modify tranfsormers

Done https://github.com/Continvvm/continuum/pull/29


[fix] Enable CI for pull_requests

Description of Change

Sets up two separate GitHub Actions -

on pull-request

checks for cpplint
checks compilation on Linux, MacOs and Windows

on the author's fork

performs clang-format and commit if needed
update DIRECTORY.md and commit if needed
perform same checks as 1 on the fork

Checklist

[x] Added description of change
[ ] Added file name matches File name guidelines
[ ] Added tests and example, test must pass
[ ] Added documentation so that the program is self-explanatory and educational - Doxygen guidelines
[ ] Relevant documentation/comments is changed or added
[x] PR title follows semantic commit guidelines
[ ] 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:

@ayaankhan98 this one is important to add please review.
Thanks

@cclauss let know if you have any suggestions

done. please merge.


Could not open socket with RM Mini

Hello, I am using Python 2.7, also set the device_type to 'rm' and using the MAC and hostname lookup.
I have checked the port it's not used
#./mqtt.py
[2017-04-27 18:34:35,601] DEBUG Connected to RM2 Broadlink device at ('<IP_ADDRESS>', 80)
Traceback (most recent call last):
  File "./mqtt.py", line 237, in <module>
    mqttc.connect(cf.get('mqtt_broker', 'localhost'), int(cf.get('mqtt_port', '1883')), 60)
  File "/usr/lib/python2.7/site-packages/paho/mqtt/client.py", line 700, in connect
    return self.reconnect()
  File "/usr/lib/python2.7/site-packages/paho/mqtt/client.py", line 822, in reconnect
    sock = socket.create_connection((self._host, self._port), source_address=(self._bind_address, 0))
  File "/usr/lib64/python2.7/socket.py", line 571, in create_connection
    raise err
socket.error: [Errno 111] Connection refused

Any help please ?

There are two TCP connections and you should separate them:

first one is connection to Broadlink device. In your case (device_type = 'rm') you should specify parameters in device_host/device_mac parameters
second one is connection to MQTT broker. mqtt_broker/mqtt_port parameters are used here

I see that connection to Broadlink device was successful but to MQTT broker was not
Do you have MQTT broker run in your local network and mqtt_broker points to the host where it is run?


User properties for Maven plugin?

Can any of the configuration be set via user properties, e.g. from the command line with -D?
If so, please document what they are.

RTFM https://jeremylong.github.io/DependencyCheck/dependency-check-maven/plugin-info.html

Aha. I was using this page https://jeremylong.github.io/DependencyCheck/dependency-check-maven/configuration.html
Being used to the layout at http://maven.apache.org/plugins/index.html, I though that was the only Goals documentation you had.


ServiceProvider namespace changed after release 1.0.4

I had some problems after composer update. Reason was the release of Version 1.0.4 where the provider has been moved.
Zalazdi\LaravelImap\LaravelImapServiceProvider::class,
To:
Zalazdi\LaravelImap\Providers\LaravelServiceProvider::class,

Thanks man!


[4.2][Refactoring] Fix crasher in switch with non-nominal subject type

Cherry-pick of #18177 reviewed by @nkcsgexi for swift-4.2-branch.

Explaination: Getting list of applicable refactoring kind on switch or default checks if the subject type is "expandable" or not. Currently, this is limited to single enum type. Previously there was wrong assumption where subject type is always nominal type. But that is not true if the subject is, for example, tuple type.
Scope: Affect getting list of applicable refactoring kind.
Issue: rdar://problem/42098130
Risk: Very-low. Just added nullptr check where it should be.
Testing: Added regression test cases.
Reviewed By: Xi Ge (https://github.com/apple/swift/pull/18177)

@swift-ci Please test

@swift-ci Please nominate


[Feat] Add getMapboxRef prop

Adds a getMapboxRef prop which you can set to your custom function. That function is passed down into KeplerGl's MapContainer and is used as a callback to expose the MapContainer's react-map-gl InteractiveMap instance. In other words, getMapboxRef gives KeplerGl's parent component a reference to the inner Mapbox map.
Is this a good pattern or an anti-pattern? See my update to examples/demo-app/src/app.js on how it would be used.
See #268. Exposing access to the full Mapbox GL API should also open a solution for #293 and #118.
Kepler.gl has powerful client-side visualizations for GeoJSON. By having access to the Mapbox GL API, developers can easily add raster and vector layers from a variety of sources, and further customize map interactions.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

I believe the build failure is unrelated to my diff.

@heshan0131, thank you for the review.
I believe 5975a95f6 addresses your concerns... from the extended commit message:

This simplifies the handling of getMapboxRef so that KeplerGL forwards your callback function to MapContainer unchanged. It also adds support for multiple maps. Use 'index' (0 or 1) to identify which map triggered the callback. If 'mapbox' is null, then that map has been removed by toggleSplitMap.

README updated

Thanks, this looks good to me. I am working with the FOSSA team to fix the build, but we can land this now if needed

No hurry, I'll use my fork until version v0.2.3 lands.

The force-push was to rebase robert-claypool/expose-mapbox-map against kepler.gl/master.


[FLINK-11788][table-planner-blink] Support Code Generation for RexNode

What is the purpose of the change
This is the basic implementation to support code generation for RexNodes.
Brief change log
Flink/Blink's code generation is not clean and the generate.... methods is existed in various util class (e.g. CodeGenUtils, BinaryStringCallGen, CallGenerator, ScalarOperators ...). This pull request does some code cleanup to put the generations into a new structure.
All the generate.... methods are split and placed into only 2 util classes.

GenerateUtils: generate code for general purpose, e.g. generateCallIfArgsNotNull, generateFieldAccess, generateLiteral. It can be used in various CodeGenerator.
SqlOperatorGens: generate code for SQL scalar operators, e.g. arithmetic operator. It is mostly used in ExprCodeGenerator.

Some other classes are:

CallGenerator interface: generate a call, usually used when the generation depends on some additional parameters.
CodeGenUtils: a utility for codegen, e.g. newName, primitiveTypeTermForType, but not include generate... methods.
ExprCodeGenerator:  responsible for generating codes for a given calcite RexNode.
CodeGeneratorContext: context for code generator, maintaining various reusable statements

Verifying this change
Add a ExpressionTestBase base class to check the code generation for expressions(RexNode).
We introduce 5 expression validation test class and 9 expression test class.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable)

+1 to merge

Rebased and squashed.
Waiting travis to turn green...


Overhaul SDK page

📝 Changed routes:

/
/sdk

(view full size)
(view full size)

Commit a255809e34865ecf1fea60cb32ec8ca86e181135 (https://thirdweb-ji7hrgu9g.thirdweb-preview.com).


A way to split the dashboard config file

Hello,
First of all, thank you for your work !
So, I don't know if there is a way to do it, but, I'd like to split the dashboard config file in smaller pieces.
Like " !include: " in the configuration.yaml file.
Because, when you have to custom a bit your dashboard the file is unreadable.
I think, splitting files is more comfortable for maintenance or evolution of our dashboard :)
Thank you

I believe the !include directives aren't supported for yaml file of the dashboard configuration.
If I'm wrong, I'd like to hear about it.
For now, I'll leave this issue opened.
Maybe we can make up another method for splitting up the configuration, but it will always involve placing additional yaml files on your Home assistant device manually.
This might be out of scope of this plugin.
@AalianKhan What do you think?

Currently, the Raw configuration editor from the UI doesn't support !include. However, you can add some lines to your configuration.yaml to add YAML dashboard which does support !include. See (docs)[https://www.home-assistant.io/dashboards/dashboards/#adding-more-dashboards-with-yaml]

Thank you for the answer :)
I'll check the doc you linked me


⚠ RPC server - Avalanche has degraded performance

In a3a84a0, RPC server - Avalanche ($AVALANCHE_RPC_URL) experienced degraded performance:

HTTP code: 502
Response time: 146 ms

Resolved: RPC server - Avalanche performance has improved in 4d605bc after 9 minutes.


There's good no way to run after VITE_PRELOAD gets handled. Add bundleComplete hook?

Describe the bug
I have a plugin that post-processes the built files for i18n using the generateBundle hook, but in recent Vite versions it runs before Vite replaces the VITE_PRELOAD strings.
The importanalysis plugin always runs after user defined plugins:
https://github.com/vitejs/vite/blob/6f8a3206653127a1ca9e20880af117d3a7c4fadc/packages/vite/src/node/plugins/index.ts#L99-L108
The reason I picked generateBundle is so that I don't have to worry about where the files are stored. writeBundle doesn't get called when using the API and closeBundle doesn't seem to have the information to find the built files.
Should there be one more plugin hook, e.g. bundleComplete? Or should I be doing the post-processing differrently?
Reproduction
https://github.com/wmertens/compiled-i18n/blob/e33db19080efbe5d02bdad691cf8e71a250158a9/src/vite.ts#L251-L276
Steps to reproduce
Use compiled-i18n in a project, bundle it, and notice that VITE_PRELOAD is still present in the i18n-ed versions.
System Info
System:
    OS: Linux 6.1 NixOS 24.05 (Uakari) 24.05 (Uakari)
    CPU: (8) x64 Intel(R) Core(TM) i7-1065G7 CPU @ 1.30GHz
    Memory: 3.73 GB / 15.20 GB
    Container: Yes
    Shell: 5.2.26 - /run/current-system/sw/bin/bash
  Binaries:
    Node: 20.9.0 - /nix/store/a1hckfqzyys4rfgbdy5kmb5w0zdr55i5-nodejs-20.9.0/bin/node
    npm: 10.1.0 - /nix/store/a1hckfqzyys4rfgbdy5kmb5w0zdr55i5-nodejs-20.9.0/bin/npm
    pnpm: 8.15.0 - ~/bin/pnpm
  npmPackages:
    vite: ^5.1.5 => 5.1.5

Used Package Manager
pnpm
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

You can use generateBundle.order: 'post'. https://rollupjs.org/plugin-development/#build-hooks

@bluwy unfortunately, I am already using post but the build analysis changes still run after that.
Perhaps you can change it to run at normal priority?

We can't do that. We have similar requests in the past to get before or after Vite plugins. Maybe we need to expand the order modifiers to initial, pre, normal, post, final 🤔

@bluwy unfortunately, I am already using post but the build analysis changes still run after that. (See two lines above where I linked)

I don't quite see that. It's using enforce: 'post', but I mean order: 'post'

Oh, this is interesting. So order: 'post' in Rollup goes after our enforce: 'post'? During dev, we are making order and enforce have exactly the same meaning: https://github.com/vitejs/vite/blob/d44342859a45a295b9497775f8716de83ca1c03d/packages/vite/src/node/plugins/index.ts#L139
Do we have an alignment issue, also with pre between dev and build, then?

it don't think we have an alignment issue. enforce is for Vite to sort plugins, order is for Vite/Rollup to sort hooks. The function name is a bit misleading because we shouldn't be re-sorting plugins based on hooks, but in practice we don't use the sorted plugins list, but only their targeted hooks, so the behaviour should still be fine in Vite. But I get why we re-sort it though, mainly to retrieve the plugin names by each hook it seems.

Maybe we need to expand the order modifiers to initial, pre, normal, post, final 🤔

For this issue I'm hoping that the post gets aligned, but perhaps it should be allowed to mutate the plugins array in configResolved for special cases?

Ok so enforce doesn't really do much then? Shouldn't enforce set the order on the hooks?

it don't think we have an alignment issue. enforce is for Vite to sort plugins, order is for Vite/Rollup to sort hooks. The function name is a bit misleading because we shouldn't be re-sorting plugins based on hooks, but in practice we don't use the sorted plugins list, but only their targeted hooks, so the behaviour should still be fine in Vite. But I get why we re-sort it though, mainly to retrieve the plugin names by each hook it seems.

Ok, now I recall why we couldn't deprecate enforce in favour of order. And also that I was mixing the deprecation we did of transformIndexHtml.enforce in favour of transformIndexHtml.order, where we could.
See we are going to need to stick with both, and then I agree, you can use order: 'post' in the hook to get ahead of our plugins.

I must say that I assumed that enforce: 'post' would enforce that all the hooks would be ordered post...


Corresponding Network Service Tag

Could you please add some info?
We can use some service tags for "Network firewall requirements".
*.ods.opinsights.azure.com, *.oms.opinsights.azure.com -> AzureMonitor service tag
*.blob.core.windows.net -> Storage service tag
Also see https://docs.microsoft.com/en-us/azure/virtual-network/security-overview#service-tags

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

ID: 6b71519d-1b94-c489-ce51-7210eb05b109
Version Independent ID: fb6adce7-aa81-28e9-ad54-8c90dd86afa2
Content: Collect log data with Azure Log Analytics agent
Content Source: articles/azure-monitor/platform/log-analytics-agent.md
Service: azure-monitor
Sub-service: logs
GitHub Login: @MGoedtel
Microsoft Alias: magoedte

@nakamacchi Thanks for your question. We are checking on this and will respond to you soon.

Assigning content owner @MGoedtel for visibility, review and document enhancement considerations if required.

@nakamacchi + @KrishnaG-MSFT - We will take this feedback for review, validation, and add to the documentation.  I'll create a work item for this now and see if we can address in our next sprint.  Thanks!  #please-close


Broken links to the "wiki" in README.

README.md has two broken links to a "wiki" page.
L75: See more [examples in the wiki](https://github.com/erezsh/lark/wiki/Examples)
L98: See the full list of [features in the wiki](https://github.com/erezsh/lark/wiki/Features)

Both link redirects to the repo main page. Well, technically, they are not broken in the sense of 404, but surely that is not the expected behaviour.
Are the examples and list of features lost with the wiki? Or are they hiding somewhere in the repo but only not being correctly linked to?

Yes, the documentation isn't lost, it's just elsewhere.
Thanks for bringing this to my attention.

Thanks again


Forked a subset of JAX configuration APIs

These APIs are internal to JAX and should not be used by other projects for managing their configuration.

We're currently processing your upload.  This comment will be updated when the results are available.

Thanks for working on it!

We only have bool flags now, but with this setup, does adding non-bool flags just requires forking a few more lines?

Since we no longer rely on an existing framework, can we have a simple unit test for this? To make sure the runtime update works, and that the flag value will not reset after re-importing the configuration.py. Thank you!

Thanks for having a look, Ivy!

Yes, adding support for new flag types should be easy. E.g. here is how int_flag could look
def int_flag(name: str, default: bool, help: str) -> FlagHolder[int]:
  name = name.lower()
  if default_env := os.getenv(name.upper()):
    try:
      default = int(default_env)
    except ValueError:
      raise ValueError(f'Invalid value "{default_env}" for flag {name}')
  config._add_option(name, default)
  fh = FlagHolder[int](name, help)
  setattr(Config, name, property(lambda _: fh.value, doc=help))
  return fh

Will add a test, sure!


update this fork with microsofts default

Reference Issues/PRs

What does this implement/fix? Explain your changes.
Any other comments?

seems too complicated for what we want to do which is just get a repo for the JOSS paper


Running via crontab and updating apps

Rather than transcode on download, I'd like to run it overnight.  Plex is generally tolerant of files getting renamed (as mkv to mp4) but sonarr and radarr will think the file is missing.
Is it possible to update a movie (radarr) or TV series (sonarr) when running manual.py?  I'm comfortable writing a script that runs it, but I don't know of a good way to build a queue and it would be far easier to run manual.py -a in the library folder, if that makes sense.
Thanks!

On the first bit, if you told sonarr to ignore deleted files it would be happy and pick them back up in a later library scan.

That's a good observation.  Unfortunately I use that feature to clean up my movie libraries (to make deleted movies unmonitored).  For sonarr only, I suppose it's probably the same as setting all shows to monitored.

On the first bit, if you told sonarr to ignore deleted files it would be happy and pick them back up in a later library scan.

You could write a wrapper for postSonarr or postRadarr potentially or just write a postProcess script that notifies radarr/sonarr to refresh (you could reference postSonarr/postRadarr for who this refresh is triggered)


FR: Configure the normalised min/max values when using "min_max" with DataProcessor

TL;DR The DataProcessor hard-codes min-max normalisation to [-1, 1], but we now have a use case for mapping to [0, new_max]. So we need a new feature that makes this configurable to the user. Motivation below:
DeepSensor now supports a ConvCNP with a Bernoulli-Gamma likelihood (thanks @wesselb for implementing this upstream in neuralprocesses!), see https://github.com/alan-turing-institute/deepsensor/issues/95. This is useful for modelling lower-bounded variables which can take values exactly equal to the lower bound, e.g. precipitation which has many values of exactly zero (no rain).
The current Bernoulli-Gamma implementation in neuralprocesses hard-codes the delta/'spike' component of the Bernoulli-Gamma mixture to take a value of zero (which is reasonable). However, when normalising data with DeepSensor's DataProcessor, using data_processor(data, method="min_max") hard-codes the new min and max to be -1 and +1, respectively. This is incompatible with ConvNP(..., likelihood="bernoulli-gamma"), because any target values below 0 results in a NaN loss (and NaN weights). So, we now need to configure the new min/max of the normalised data.
I would envisage this being configurable through the __call__ so that it can be different for each variable, like
normalised_data = data_processor(raw_data, method="min_max", new_min=0, new_max=1), which would be passed from __call__ to map to map_array, with new logic for "min_max" here.
A few extra bits of housekeeping needed:

Set the default values to -1 and +1 for backwards compatibility. We should also consider how this will break user's existing DataProcessor configs (which will not have these parameter values).

Store in the DataProcessor config, eg like:

 'elevation': {'method': 'min_max', 'params': {'max': 4504.4375, 'min': -185.125, 'new_min': 0, 'new_max': 1}},

Unit test this in deepsensor/tests/test_data_processor.py, in particular adding this new normalisation feature to the unit test that asserts normalised data is the same after saving and loading a DataProcessor.

cc @wesselb, an alternative would be to make the location of the spike in Bernoulli-Gamma configurable in neuralprocesses, and expose this through construct_convgnp. We could then keep the [-1, +1] hard-coding in DeepSensor and set the spike location to -1. I'm not sure which approach would be simpler!

The Bernoulli-gamma distribution handles unbounded data. Instead of normalising to a bounded interval, perhaps it would be best to normalise to the unbounded the interval [0, inf)? I'm thinking something like x -> (x - min(x)) / scale(x) where scale(x) = std(x) or scale(x) = median(x).

Good shout @wesselb! We could then add this as a new DataProcessor normalisation method (for example, 'min_median', or perhaps more explicitly 'positive_semidefinite').
This would avoid having to worry about backwards compatibility of the 'min_max' method (which we can keep as normalising to [-1, +1]).

Specific list of steps to implement this feature:

Update list of valid methods: https://github.com/alan-turing-institute/deepsensor/blob/fbbbd9e64f1564af5e95bf243b661abd36263cf6/deepsensor/data/processor.py#L100
Compute the relevant parameters of the data (min/median or min/stddev): https://github.com/alan-turing-institute/deepsensor/blob/main/deepsensor/data/processor.py#L292-L295
Add the normalisation / unnormalisation: https://github.com/alan-turing-institute/deepsensor/blob/main/deepsensor/data/processor.py#L501-L527
Add the new methods to the call docstring: https://github.com/alan-turing-institute/deepsensor/blob/main/deepsensor/data/processor.py#L612

The test_data_processor already loops over the valid_methods of the DataProcessor, so we shouldn't need to explicitly add a unit test: https://github.com/alan-turing-institute/deepsensor/blob/main/tests/test_data_processor.py#L66

Closed by: https://github.com/alan-turing-institute/deepsensor/commit/88a98182d07edcaf0ac490ea75378f15b1c45dfb
Choices:

'positive_semidefinite' for the DataProcessor normalisation method name
divide by the standard deviation for linear normalisation mapping (rather than the median), to avoid very large tail values

Thanks @wesselb for the idea!