GradedTraversable.traverse stack-traces in unchecked_cast

If I understand the stack trace correctly, this unchecked_cast fails: https://github.com/stephentetley/graded-lib/blob/master/src/GradedTraversable.flix#L46 when using GradedTraversable.traverse.
Stack trace:
 FAIL  test4
    java.lang.ClassCastException: class Parser$Parser$Obj$Obj$Obj cannot be cast to class IParser$Char$Obj$Obj (Parser$Parser$Obj$Obj$Obj and IParser$Char$Obj$Obj are in unnamed module of loader ca.uwaterloo.flix.language.phase.jvm.FlixClassLoader @1ad8df52)
        at Clo$traverse$157471.invoke(Unknown Source)
        at Cont$Obj.unwind(Cont$Obj)
        at Def$test4.invoke(Unknown Source)
        at Cont$Bool.unwind(Cont$Bool)
        at Ns.m_test4(Unknown Source)
        at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
        at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at java.base/java.lang.reflect.Method.invoke(Method.java:566)
        at ca.uwaterloo.flix.language.phase.jvm.JvmBackend$.$anonfun$link$1(JvmBackend.scala:298)
        at ca.uwaterloo.flix.language.phase.jvm.JvmBackend$.$anonfun$getCompiledDefs$2(JvmBackend.scala:271)
        at ca.uwaterloo.flix.tools.Tester$TestRunner.runTest(Tester.scala:182)
        at ca.uwaterloo.flix.tools.Tester$TestRunner.$anonfun$run$7(Tester.scala:153)
        at ca.uwaterloo.flix.tools.Tester$TestRunner.$anonfun$run$7$adapted(Tester.scala:152)
        at scala.collection.immutable.Vector.foreach(Vector.scala:1856)
        at ca.uwaterloo.flix.tools.Tester$TestRunner.run(Tester.scala:152)

Sources of test4: https://github.com/Release-Candidate/flix-test/blob/main/src/FlixParse.flix#L143
You can use this small test project to reproduce the error, if needed.

Hi - thanks for the bug report, I'll look into it today.

I think the problem is that FlixParse's point (and also empty) are Pure rather than being effect polymorphic.
The cast in the GradedApplicative instance was then causing the runtime cast failure.
It's a bit unintuitive that point should be polymorphic in its effect rather than pure. I've made that mistake myself - it is still present in the functions empty and singleton in my revision of Iterator in Flix's stdlib. I didn't realized Iterator was a graded monad when I wrote it and some effects "float" to the right hand side of the type signatures when they should be captured in the Iterator type constructor. I've hesitated from updating Iterator in the stdlib because at the moment I don't know if Iterator should be a graded monad or an effect handler and you can't currently define effect handlers in Flix code (they have to defined in the compiler).
These modications should clear the runtime cast failure:

mod Parser {
    pub def empty(): Parser[i, ef, a] =
        Parser.Parser(_ -> checked_ecast(Err(Error.Empty)))

    pub def point(x: a): Parser[i, ef, a] =
        Parser.Parser(input -> checked_ecast(Ok(x, input)))

...

instance GradedApplicative[Parser[i]] {

    pub def point(x: a): Parser[i, ef, a] =
        Parser.point(x)

...

Thanks for your help, that was the problem.
Yes, I wouldn't have thought that moving the cast from the GradedApplicative.point to Parser.point would solve the issue.
I also thought that at least one of the point function would show up on the stack trace.

Thanks - Flix's stack trace is a bit funny for locating cast errors. It will show you something is wrong, but because more often that not it will be in the code of a higher order function it doesn't always point you to the error itself.
By the way, I've just made a new release of graded-lib but it might need a compiler built from the Flix repo's "head" rather than the last release (0.35.0).

graded-lib 0.3.1 works for me with Flix 0.35.
As you are working on the PR of the Graded* type classes, do you know if there is going to be for notation support for GradedApplicator and -Monad?

Hopefully there will be syntax support for graded monads at some point, though it really depends on what form effects take when they are fully integrated and users can define their own effects / handlers.
My feeling is that Flix will still need graded monads as they provide a way to sequence operations using effects and I think at the implementation level sequencing will need to be in CPS to avoid stack overflows as Flix is strict. A graded Codensity / CPS monad can provide stack safe sequencing and I don't think you could have CPS as an effect. It might be that the Flix standard libraries end up having one graded CPS monad and many effects.
At the moment without special syntax I use the "explicit bind" style that was sometimes used in Haskell:
https://github.com/stephentetley/flix-basicdb/blob/fc51065ff75e73eb3a01eb2e0348ccec5782e2e9/src/BasicDb/Reader.flix#L27


Eureka: Can I get the  service's name which is DOWN from Eureka Server?

I want to know which service is down, but I don't know how. Is there any API provide such data?

Try:
http://host:8761/eureka/apps
It will return all registered apps with details.
Or use EurekaClient which provides method getApplications.

👍 to what @Aloren said


:bug: Ensure data source is ready during reset

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #1178

Codecov Report

Merging #1310 (bf2d220) into master (7bb9af9) will decrease coverage by 1.69%.
The diff coverage is 44.89%.

:exclamation: Current head bf2d220 differs from pull request most recent head 076d672. Consider uploading reports for the commit 076d672 to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##           master    #1310      +/-   ##
==========================================
- Coverage   22.79%   21.10%   -1.69%
==========================================
  Files          22       16       -6
  Lines        1610     1284     -326
==========================================
- Hits          367      271      -96
+ Misses       1179      962     -217
+ Partials       64       51      -13

Impacted Files
Coverage Δ

internal/agent/config.go
0.00% <ø> (ø)

internal/agent/interactive_install.go
0.00% <ø> (ø)

internal/agent/notify.go
0.00% <0.00%> (ø)

internal/agent/recovery.go
0.00% <ø> (ø)

internal/agent/reset.go
0.00% <0.00%> (ø)

internal/agent/upgrade.go
0.00% <0.00%> (ø)

pkg/config/config.go
0.00% <0.00%> (-51.36%)
:arrow_down:

internal/agent/install.go
5.10% <21.21%> (+5.10%)
:arrow_up:

pkg/config/collector/options.go
64.51% <37.50%> (ø)

pkg/config/collector/collector.go
61.13% <61.13%> (ø)

... and 2 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

does this make sense? I mean, 5 minutes waiting for source+60 seconds for reset is quite a lot...we need to fix the datasource wait!

yep agreed, data source pulling code is rather suboptimal :/

Ok all green, merging!


SQL Error [22021]: ERROR: invalid byte sequence for encoding "UTF8"

The error occurs when I open a table with data that has special characters.
The database have a default encoding of SQL_ASCII;
Collate: C
Ctype: C
I think when Dbeaver retrieves the data it uses the UTF-8 as the default charset, as the database have SQL_ASCII character encoding the JDBC can't transform the special character into UTF-8
Here are the logs generated by Dbeaver itself:
Caused by: org.postgresql.util.PSQLException: ERROR: invalid byte sequence for encoding "UTF8": 0xd1 0x4f
	at org.postgresql.core.v3.QueryExecutorImpl.receiveErrorResponse(QueryExecutorImpl.java:2440)
	at org.postgresql.core.v3.QueryExecutorImpl.processResults(QueryExecutorImpl.java:2183)
	at org.postgresql.core.v3.QueryExecutorImpl.execute(QueryExecutorImpl.java:308)
	at org.postgresql.jdbc.PgStatement.executeInternal(PgStatement.java:441)
	at org.postgresql.jdbc.PgStatement.execute(PgStatement.java:365)
	at org.postgresql.jdbc.PgStatement.executeWithFlags(PgStatement.java:307)
	at org.postgresql.jdbc.PgStatement.executeCachedSql(PgStatement.java:293)
	at org.postgresql.jdbc.PgStatement.executeWithFlags(PgStatement.java:270)
	at org.postgresql.jdbc.PgStatement.execute(PgStatement.java:266)
	at org.jkiss.dbeaver.model.impl.jdbc.exec.JDBCStatementImpl.execute(JDBCStatementImpl.java:327)
	at org.jkiss.dbeaver.model.impl.jdbc.exec.JDBCStatementImpl.executeStatement(JDBCStatementImpl.java:130)

Here are the character encoding both server and client
 server_encoding
-----------------
 UTF8
(1 row)

 client_encoding
-----------------
 UTF8
(1 row)

Please tell DBeaver, version, the database and steps to reproduce. Also full logs can help us https://github.com/dbeaver/dbeaver/wiki/Log-files

Dbeaver version is 7.3.0
Steps to reproduce the error.

Create a database with default encoding of SQL_ASCII, a Collate Type of C and Ctype of C.
Create a dummy table with a dummy varchar data type column.
Insert a data with special characters such as Ñ/ñ using a different database manager such as pgAdmin.
Go to dbeaver and open the data of the created dummy table.

Here is the log file.
dbeaver-debug.log

Same issue, PgAdmin read ok. But dbeaver show me that wrong.


KBFS client not found on Windows 7 after multiple repair, reboot, install attempts.

Hi,
I couldn't get the K: drive to mount. I tried several things that I saw in different support requests about the "KBFS client not found" message, including repairing the installation and then reboot, update and then reboot, etc. I don't know what to do anymore, and this happens on both of my windows clients (one at work and one at home, totally different network but same OS, Windows 7 professional). Most of the time it's a Driver Install error for Dokan or a Start Error.
What is the clean way to do this ? Are there any known compatibility issues with antivirus or security software such as Kaspersky Antivirus ?
my log id: 4d33df133248d3e978dc8d1c

Cc: @zanderz, please investigate?

Here's a more recent logId for the "kbfsdokan -debug -mount-from-service" command bf468633b6b82b397575cf1c

Keybase does not think the Dokan drivers (our virtual filesystem implementation) are installed properly, though I do see Dokan1 listed as a filesystem driver in your logs. If you open a command prompt as administrator and enter sc query dokan1, what does it say?
 Occasionally a required Windows service pack fails to install. I can't quite tell if that happened because your logs are not in English.

Hi, sorry for the late reply
SERVICE_NAME: dokan1
TYPE               : 2  FILE_SYSTEM_DRIVER
STATE              : 3  STOP_PENDING
(STOPPABLE, NOT_PAUSABLE, IGNORES_SHUTDOWN)
WIN32_EXIT_CODE    : 0  (0x0)
SERVICE_EXIT_CODE  : 0  (0x0)
CHECKPOINT         : 0x0
WAIT_HINT          : 0x0
Which logs are you talking about ? My system is in French (guilty, adopted Canadian :D ) but the keybase logs seemed to be in English, what am I missing ?

Soooo 45ae766bee632d5339746e1c for the latest log ids

Oh and sc query dokan1 produces this
SERVICE_NAME: dokan1
TYPE               : 2  FILE_SYSTEM_DRIVER
STATE              : 4  RUNNING
(STOPPABLE, NOT_PAUSABLE, IGNORES_SHUTDOWN)
WIN32_EXIT_CODE    : 0  (0x0)
SERVICE_EXIT_CODE  : 0  (0x0)
CHECKPOINT         : 0x0
WAIT_HINT          : 0x0
But still no K: drive (and no, no other drive already uses this letter)

What are the results of entering the following in Keybase Shell?
keybase kbfsmount get
keybase kbfsmount getall
If get shows no drive letter selected, and getall shows K: available, it may just be a matter of issuing keybase kbfsmount set K:. Your logs look like some configuration error prevented a letter from being selected.

C:\Users\GoJiTa972\AppData\Local\Keybase>keybase kbfsmount get K: C:\Users\GoJiTa972\AppData\Local\Keybase>keybase kbfsmount getall [I: J: K: N: O: Q: T: U: W: Y: Z:] C:\Users\GoJiTa972\AppData\Local\Keybase>

Do you have any history of people having issues with Kaspersky Antivirus and Keybase ? I also have Ext2FS volume manager installed, but no ext 3/4 drives currently connected. And all the internal drives besides C: are located on software Raid 1 arrays (Intel Rapid Storage)

I think I have a lead. The ShadowProtect service (great backup/restore utility IMHO). I uninstalled it from one of my machines and the K: drive appeared at the next reboot. Now it is still installed on my main machine (and won't be uninstalled), would it be possible for you guys to test a keybase install on Windows 7 with this third party software installed ?

Confirming that this was caused by Dokan drivers not working properly.
Various versions of different AV products cause issues from time to time with the Dokan drivers (and we try to fix them).
Is https://www.storagecraft.com/products/shadowprotect-spx the version of ShadowProtect you are using?

Oh no, an older one, it's
ShadowProtect Desktop 5 - Home User Bundle
https://buy.ca.storagecraft.com/ShadowProtect-Backup-for-Desktop-Operating-Systems-C18.aspx

Yup, didn't change anything.

Any news on this subject ?

We've been evaluating an updated set of drivers, available in this experimental installer build: http://s3.amazonaws.com/prerelease.keybase.io/windows/Keybase_1.0.39-20171221112423%2B31648cae1.386.exe
It would be very interesting to see if this helps your issue.


mobile app initial revision

Description
Mobile app that allows the user to label an item in AR and save the result to the cloud.
The significant files are the Swift source code. Even if you are new to Swift, it should be readable. The most important ones are:

ARViewContainer.swift, which has the augmented reality UI code. It has some comments describing specifically what it does with AR. Mostly it calls iOS AR APIs.
Firebase.swift, which saves user-annotation data to the cloud.

There are a number of .png files, which are just app icons of different sizes.
There is the Resnet50 image classification model, which is 100MB and not included.
There is a Google plist file for Firebase access, which is also not included, for security reasons.
Type of change
Please check the options that are relevant.

[ ] Bug fix (non-breaking change that fixes an issue)
[ ] Proposes a change (non-breaking change that isn't necessarily a bug)
[ ] Refactor
[X] New feature (non-breaking change that adds a new functionality)
[ ] Breaking change (fix or feature that would break some existing functionality downstream)
[ ] This is a unit test
[ ] Documentation only change
[ ] Datasets Release
[ ] Models Release

Type of requested review

[ ] I want a thorough review of the implementation.
[X] I want a high level review.
[ ] I want a deep design review.

Checklist:

[X] I have performed manual end-to-end testing of the feature in my environment.
[ ] I have added Docstrings and comments to the code.
[ ] I have made changes to existing documentation where needed.
[ ] I have added tests that show that the PR is functional.
[ ] New and existing unit tests pass locally with my changes.
[ ] I have added relevant collaborators to review the PR before merge.
[ ] [Polymetis only] I ran on hardware (1) all scripts in tests/scripts, (2) asv benchmarks.

https://user-images.githubusercontent.com/87049088/152593440-770cf8bb-0dd0-43f6-b637-fd639fbee3fc.mp4

It's worth checking in the binary assets into the Amazon s3 bucket.
We're not going to do well by checking in all our binary assets into the git repo.

as commented above

Thanks for the review. I removed the ML model, as well as the Google Firebase access plist. The app icons I kept in the commit, as this is standard practice for a mobile app.

Codecov Report

Merging #978 (4400331) into main (12a165d) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #978      +/-   ##
==========================================
- Coverage   32.73%   32.72%   -0.01%
==========================================
  Files         349      349
  Lines       34493    34493
==========================================
- Hits        11290    11289       -1
- Misses      23203    23204       +1

Impacted Files
Coverage Δ

interpreter/robot/tests/world.py
74.45% <0.00%> (-0.73%)
:arrow_down:

agents/_agent.py

agents/droidlet_agent.py
51.57% <0.00%> (ø)


Update service name of core packages to "Core"

I checked and found .NET docs uses Core node so aligning with that.

@danieljurek thanks for the context.  Closing for now.


[BUG]: Self-Hosted agent executables unable to start / hang indefinitely

What happened?
Let's start with an important bit - the affected agents are hosted on servers under few corporate group policies, that may be the cause of our problems. Still, we believe it's fixable with assisstance of people who know better how agents work inside, so please bear with us :)
Last week we upgraded our on-prem ADO server to version 2022.1 (patch 2). The upgrade also forced our build agents to update themselves to v3.225.2 and that's when our problems started.
The first observation we had was that even though agents appeared online and would receive jobs, no actual work would start and we soon would get "We stopped hearing from agent (...)" error.
We suspected it may be ADO 2022.1 configuration issue, but that was ruled out, as deployment agents (which by the way are still v2.218 and are working perfectly) kept on going - just needed a restart.
Second guess was some kind of networking issue - maybe there were some new endpoints that agents couldn't reach - but that also got ruled out, as fiddler uncovered absolutely perfect traffic (we could even capture moment when agent receives job that actually never gets started).
That's when we started to check the binaries of agents and where we found (in our opinion) the actual issue. What we noticed is that both Agent.Listener and Agent.Worker have trouble with actually starting themselves. It was first observed when running ./config.cmd - 7/10 times it would get stuck immediately and not output anything.
Then, even if we managed to setup listener and run it as service, it would be unable to "spawn" Agent.Worker for incoming job - that's why we observed jobs getting queued (listener was pooling), but no work was done (Worker didn't start in time).
In the logs included, you can see that even though Worker process technically starts, it immediately stops responding and after 30s is getting terminated - while also ignoring both cancel and terminate signals.
We modified sources of Agent.Listener and Agent.Worker to include some extra log information as the apps start - we observed no output, suggesting that we're not even reaching Main methods in them.
Also, just in case it's important - we run our Listener services as NT AUTHORITY\NETWORK SERVICE.
To sum up:

ADO 2022.1 server itself works fine
there are no networking issues
Agent.Listener has issues with starting most of the times (requires machine restart to start working)
Agent.Worker doesn't start at all

We did some extra checks and found some other interesting bits:

v2 agents on same servers work perfectly, even though ADO 2022.1 wants agents v3
v3 agents we have issues with work perfectly on linux machines
on our test server (also Windows Server 2019) that is not under corporate policies, agents v3 also seem to work

Our conclusion is that corporate policies interfere with how agents v3 work. At the same time v2 agents work fine, so our question would be - what exactly changed between those? Are we perhaps missing some file permissions for v3, which makes them unable to start properly? Or maybe there is a different underlying issue?
Versions
Agent versions we tried: v3.225.2 - v3.235.2
Server OS: Windows Server 2019
Environment type (Please select at least one enviroment where you face this issue)

[X] Self-Hosted
[ ] Microsoft Hosted
[ ] VMSS Pool
[ ] Container

Azure DevOps Server type
Azure DevOps Server (Please specify exact version in the textbox below)
Azure DevOps Server Version (if applicable)
2022.1
Operation system
Windows Server 2019
Version controll system
git
Relevant log output
[2024-02-20 22:08:55Z INFO ProcessInvokerWrapper] Process started with process id 4028, waiting for process exit.
[2024-02-20 22:08:55Z INFO JobDispatcher] Send job request message to worker for job xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx (24 KB).

--- 30 seconds passess ---

[2024-02-20 22:09:25Z INFO JobDispatcher] Job request message sending for job xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx been cancelled after waiting for 30 seconds, kill running worker.
[2024-02-20 22:09:25Z INFO ProcessInvokerWrapper] Kill entire process tree since both cancel and terminate signal has been ignored by the target process.
[2024-02-20 22:09:25Z INFO ProcessInvokerWrapper] Scan all processes to find relationship between all processes.
[2024-02-20 22:09:26Z INFO ProcessInvokerWrapper] Find all child processes of process '4028'.
[2024-02-20 22:09:26Z INFO ProcessInvokerWrapper] Need kill all child processes trees before kill process '4028'.
[2024-02-20 22:09:26Z INFO ProcessInvokerWrapper] Child process '6668' needs be killed first.
[2024-02-20 22:09:26Z INFO ProcessInvokerWrapper] Find all child processes of process '6668'.
[2024-02-20 22:09:26Z INFO ProcessInvokerWrapper] Kill process '6668'.
[2024-02-20 22:09:26Z INFO ProcessInvokerWrapper] Kill process '4028'.
[2024-02-20 22:09:26Z INFO ProcessInvokerWrapper] STDOUT/STDERR stream read finished.
[2024-02-20 22:09:26Z INFO ProcessInvokerWrapper] STDOUT/STDERR stream read finished.
[2024-02-20 22:09:26Z INFO ProcessInvokerWrapper] Exited process 4028 with exit code -1
[2024-02-20 22:09:26Z INFO ProcessInvokerWrapper] Finished process 4028 with exit code -1, and elapsed time 00:00:30.5899395.
[2024-02-20 22:09:26Z INFO JobDispatcher] worker process has been killed.
[2024-02-20 22:09:26Z INFO JobDispatcher] Stop renew job request for job xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx.
[2024-02-20 22:09:26Z INFO JobDispatcher] job renew has been canceled, stop renew job request xxxxxx.

Hi @EnjoyTheNoise. Thank you for your question!
Initially, there was no big difference between v2 and v3, except we started using dotnet 6.0 instead of dotnet core 3.1.
But v2 is not being actively developed anymore - only v3, so some changes (usually minor) appeared.
We have v2.220.0 and v3.220.0 which are mostly the same besides the dotnet version. Could you please try to reproduce the issue using those versions?

Hi @EnjoyTheNoise,
Unfortunately we didn't get any response from you. I will close issue now, but please, feel free to re-open it, if you still experiencing this problem.


[ButtonGroup][joy] Add ButtonGroup component

Preview: https://deploy-preview-37407--material-ui.netlify.app/joy-ui/react-button-group/
Improvements from Material UI

Support IconButton as children
Able to stretch the buttons
Work with arbitrary elements that wrap the buttons
Customizable separator color using a CSS variable
Able to detach the buttons

[x] I have followed (at least) the PR section of the contributing guide.

Hey @siriwatknp and @danilo-leal, I started iterating this component with some styles from the default theme, and it'd be cool to have your inputs on the Figma file. What do you think?

While reviewing I noticed that the disabled solid buttons don't have enough contrast

@mnajdova I made a few changes to this component, but is still on Figma, please tell me what you think 😊

Not related specifically for this PR, but should we make the focus indicator follow the color of the element it wraps? For e.g. using green color in this use-case

In WCAG 2.2 there are many instructions on how to improve the focus accessibility, so I believe we definitely have room for improvement. However, I don't think changing the color would help. I'm afraid this would be confusing during the keyboard navigation in a page with many components of different colors, making it slightly harder to follow.

In WCAG 2.2 there are many instructions on how to improve the focus accessibility, so I believe we definitely have room for improvement. However, I don't think changing the color would help. I'm afraid this would be confusing during the keyboard navigation in a page with many components of different colors, making it slightly harder to follow.

Makes sense 👍 maybe we can consider using a grey color that would work nicely with all colors. It's just a thought, not tight to this PR in any way :)

@mnajdova I made a few changes to this component, but is still on Figma, please tell me what you think 😊

Looks great 👍

While reviewing I noticed that the disabled solid buttons don't have enough contrast:

I think this should be fixed in another PR. It is related directly to the global variants, not specific to components.

@zanivan just pushed the updated version according to the figma file. I notice that the divider of the plain variant is too bright, do you think it should have the same value as the soft variant?

@zanivan just pushed the updated version according to the figma file. I notice that the divider of the plain variant is too bright, do you think it should have the same value as the soft variant?

Yep, definitely. I'll update it on the Figma file


ISPN-8619 Rewrite Hot Rod client using Netty

Preview at this moment since I need to rebase this on top of cluster counters.

I certainly welcome this. I'll look at details tomorrow or after the break.
Is this compatible with Sebastian's plans around the Hot Rod client? He had planned to rewrite the client to do Hot Rod over HTTP 2.0. Deficiencies of current client had been noted here.
This should also be reviewed by @karesti and a member of the Vert.x team so that the client follow their required guidelines. Just a week ago I was having a discussion about this with @cescoffier.

@galderz the plan is to implement a single-port HTTP/1.1 server which can be "upgraded" to HTTP/2 or Hot Rod simply via an HTTP Upgrade request: https://en.wikipedia.org/wiki/HTTP/1.1_Upgrade_header
This would just mean that the Hot Rod client needs to recognize whether it is talking to an HTTP endpoint or a native one and send the Upgrade header in the first case.
Eventually, when we design Hot Rod 3, we can consider an alternative transport for it

@galderz While I have not consulted HTTP 2 with @slaskawi I don't see any issues here. Netty here is just a vehicle and I am using the 'basic API'; no fancy use of decoders etc. Netty has some support for HTTP 2 and I assume that changes during negotiation will affect the 'activation' part - see o.i.c.h.impl.transport.netty.ChannelInitializer. My plans to use Netty for async communication are mentioned in the 'deficiencies' document you've linked.

I've rewritten multimap and counter operations (counter listeners required non-trivial changes, though), so removing the 'Preview' label.


Allow dropdown_menu to accept any EdgeInsetsGeometry

Allow dropdown_menu to accept any EdgeInsetsGeometry instead of just strictly EdgeInsets.
This follows normal Flutter widget practices when accepting a padding parameter, like Container: https://github.com/flutter/flutter/blob/main/packages/flutter/lib/src/widgets/container.dart#L307C9-L307C27
Issue not created, trivial change (EdgeInsets -> EdgeInsetsGeometry).
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] I followed the breaking change policy and added Data Driven Fixes where supported.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@TytaniumDev can you update the pr description and is this related to #151686?
I second that we will need a test before merging.

Updated description, and it's unrelated to #151684. Also added a test.

I'm assuming target branch should be master.

New CI failure is unrelated to the PR. Once it is resolved will add the autosubmit.

@TytaniumDev it looks like this change needs to be rebased with master so we can land it. Can you take a look? Thanks!

Messed up something with the rebase, going to just start fresh

Fresh PR up here: https://github.com/flutter/flutter/pull/153053


Look into / report graphql enum issue

 """type field predicates"""
  type: TechnologyType
  typeNEQ: TechnologyType
  typeIn: [TechnologyType!]
  typeNotIn: [TechnologyType!]

Types are replaced randomly by other types that have *technology.Type in their go name. Might be an auto bind issue, might be something else

Issue has been reported, can be found here: https://github.com/ent/ent/issues/1811


Breaking changes

The goal of the previous PR is to update https://github.com/saltstack-formulas/sysctl-formula/blob/master/sysctl/map.jinja and remove the ugly backward compatiblility patch.
@myii  how do you want to do?

@n-rodriguez For these non-ng breaking changes, let's have two stages:

PR to add deprecation warning -- gives a little time for users to get prepared.
PR to merge v1.0.0.


Fixed system design question number 2 displaying wrong explanation

@edgargonzalez2020 thanks for the PR, but we would need you to fill in the PR template that was presented when open this PR.
Its still available here:
https://raw.githubusercontent.com/freeCodeCamp/freeCodeCamp/staging/.github/PULL_REQUEST_TEMPLATE.md
Can you please update the opening post and make the necessary changes.
Also, we would recommend a short and concise commit message and PR title.
Checkout our Guidelines for Contributing.
This is just to encourage you to learn the Github flows and contributing patterns and processes.
Happy contributing!

Thank you for the feedback. Will make sure to do that next time!


C150960: Changing three dashes by three underscores to avoid issues on localized pages

Localization team has reported source content issue that causes localized version to have broken/different format compared to en-us version.
Description: Three dashes in a row act similar to backticks (```) and leave content un-localizable

Hi @scottaddie, can you please approve and merge this PR?
Thanks in advance.

@v-maudel  thanks!

Thanks @Rick-Anderson


Redirect from OP tests

This adds tests for openidcs reactions on the authorization code response.
Next up will be tests for the token request and with that id_token validation.

completely part of #99


Rolling shutter speed of realsense D415?

Required Info

Camera Model
D415

Firmware Version
5.15.1

Operating System & Version
Ubuntu 22.04.4 LTS

Kernel Version (Linux Only)

Platform
PC

SDK Version
<IP_ADDRESS>

Language
python

Segment
Robot

Question
We need to know the exact timer the rolling shutter takes to read the 1080 lines of the color image at 30FPS and 1920x1080 resolution.
In Issue https://github.com/IntelRealSense/librealsense/issues/2724 the realsense customer support has provided the line time T_line for the depth imagers. Is there equivalent information available for the color imager (at 30FPS)?
Background to understand the relevance of the question
We are moving the camera over mostly flat ground at a distance of 500mm and 3km/h. At this speed, the color image gets visibly distorted due to the rolling shutter of the color imager, making it impossible to track and map objects correctly.
If we knew the rolling shutter speed (i.e. T_line or the time it takes to read the full 1080 lines), we could calculate a rectified image and largely remove this distortion.

Hi @janderwolf  The shutter speed is fixed on RealSense cameras.  As the shutter speed is fixed, its speed is instead configured indirectly with exposure time.
In regard to T_Line for the RGB sensor though, at https://github.com/IntelRealSense/librealsense/issues/5457#issuecomment-570744452 it is stated to be 30us.
A workaround for lag / blur of the RGB sensor due to the rolling shutter when auto-exposure is enabled is to set depth to 30 FPS and RGB to 60 FPS and have depth set at 30 FPS.  This method requires using 848x480 resolution for RGB though instead of 1280x720.
When using manual exposure, lag / blur can be reduced if RGB is configured for an exposure value of 78 and an FPS of 6 (not 60).  The higher RGB resolutions of 1280x720 and 1920x1080 can be used with 6 FPS.

Thanks for your quick reply. T_line of 30us corresponds with our measurements: We filmed a conveyor belt with marks on it and tried to measure the distortion created + the speed of the belt and thus calculated the T_line to be between 26,6us and 36.3us, so still quite an uncertanty. Do you know if these 30us are an exact value?
We need to use full 1920x1080 resolution, so 60 FPS is not an option unfortunately
I just tested the setting of 6FPS + exposure 78, but this produces an even greater rolling shutter distortion (+ some blur due to the long shutter time):

without movement, the marks on the belt should look like this:

with our current setting, the belt looks like this:

speed in the images is about 6km/h. That's 2x as fast as we need it in our application but still at 3kmh the effect is clearly visible.
Anyway, with T_line=30us, we can try to rectify our images

The D415 model excels at capturing static objects but has difficulty with fast motion due to the slow rolling shutter on both sensors.  If you are capturing fast motion with both depth and RGB then one of the D455 type models would be better as they have a fast global shutter on both the depth and RGB sensors.  The D435 type models have global on depth and rolling on RGB.
30us is the only figure for the RGB sensor that my research could find.

o

one more question: do you or somebody else know the T_line @ 60FPS?

I do not have a value for T_line at 60 FPS.https://github.com/IntelRealSense/librealsense/issues/2724#issuecomment-442546730 suggests that it can be calculated though if you know the Horizontal Total Size (HTS) for a particular resolution.

Hi @janderwolf   Do you require further assistance with this case, please?  Thanks!

Thanks, we can work with the line time of 30us @ 30fps, so we're ok:
We tested 30fps and 60fps now. As the rolling shutter effect is still visible in the 60fps images and we prefer the higher resolution possble with 30fps, we go for this framerate. And for thisframerate, we can calculate the rolling shutter error knowing the T_line you provided.

Thanks very much for the detailed update!


fix(runtime-core): remove all blocks related to the current instance to avoid memory leaks in ssr

close: #3100

I created a new project based on the project provided by @aprilcai above, which provides four heap memory snapshots:

leak.start.heapsnapshot
leak.end.heapsnapshot
healthy.start.heapsnapshot
healthy.end.heapsnapshot

And, you can directly load these snapshots into chrome and analyze memory leaks, u can also execute the following two commands to generate a snapshot locally:
yarn heapdump-leak
yarn heapdump-healthy

My analysis result is:

I created a new project based on the project provided by @aprilcai above, which provides four heap memory snapshots:

leak.start.heapsnapshot
leak.end.heapsnapshot
healthy.start.heapsnapshot
healthy.end.heapsnapshot

And, you can directly load these snapshots into chrome and analyze memory leaks, u can also execute the following two commands to generate a snapshot locally:
yarn heapdump-leak
yarn heapdump-healthy

My analysis result is:

@HcySunYang which priority label is this? More of a common situation or an edge case?

I think it's an edge case because it only occurs when rendering errors

Actually, since there can never be nested renderComponentRoot calls, we can just always reset blockStack when there's a render error. See da944cb3


rb_opts_dict does not appear in the code of rb/circuits.py

Informations
rb_opts_dict appear in the documentation and notebooks of RB,
but does not appear in the code of the function randomized_benchmarking_seq in rb/circuits.py

Qiskit Ignis version:
Python version:
Operating system:

What is the current behavior?
Steps to reproduce the problem
What is the expected behavior?
Suggested solutions
Add rb_opts_dict to the function randomized_benchmarking_seq in rb/circuits.py and return it

This was by design. Originally I coded as passing in a dictionary, but we decided to change it to explicit arguments to be more in line with the rest of qiskit and for readability. Python allows you to pass a dictionary into a function by unpacking into an argument list so that's what's done in the notebooks.


Project license

Hello Bruno!
As your code is open in Github, I think you should choose a license for the project, so people will know if it can be used for private use or not, if you accept collaboration and how you want it.
Thanks! :)

Hey Tasso, thank you for your interest!
I have added a license to the project, as suggested.


Feature: Support multiple project slugs

Hey there,
First of all: Thanks for this repo!
This PR:

allows that by supporting to set a whitespace separated list of project slugs (set :sentry_project, 'my-foo my-bar'),
fixes existing rubocop offenses because I assumed that the existing .rubocop.yml is not deprecated and
sets a required ruby version via spec.required_ruby_version = '>= 2.5.0' because I noticed that the code would fail running on 2.4.9 because of String.delete_suffix usage.

Reasoning: I found myself in a position where I have separate frontend and backend projects in Sentry (to allow users to fine tune their notifications more easily), but one git repository for both project's code ("historical reasons"). Therefore I need to notify Sentry about one release being deployed to both projects. This PR intends to support exactly that.
Please let me know if i should split up these changes into several PRs or if any other changes are required.
Thanks!

Opening a PR on master was not that smart of me.


Fix/timeout bug

Description
Autofix Diffs flow (web view flow) was implemented slightly differently than the code action flow. The diff flow treated an empty list of suggestions as an error and invalid response. This is not the correct behavior because the AI Fix pipeline might run successfully and there can be no valid fixes found.
The code was polling until the suggestion's length was not 0. This causes the returned successful response with empty suggestions to be ignored and the code aggressively keeps polling until the timeout hits. The timeout takes 2 minutes whereas the result is available after several seconds. The timeout also hides the real issue from the user, making the user think something unexpected happened whereas we could not compute any valid fix.
This PR corrects the flow for autofix diffs by

adding status into the return value similar to how it is done for code actions
corrects the logic in the polling function

Checklist

[ ] Tests added and all succeed
[ ] Linted
[x] README.md updated, if user-facing
[x] License file updated, if new 3rd-party dependency is introduced

I think the failing test is not related to my changes but because of an RCE vulnerability detected by Snyk Code. Is anyone already looking at it?


Pod that is destroyed while cilium-agent is down gets stuck attempting to restore

Cilium v1.6.3
If an endpoint is deleted while cilium-agent is not running, then the state on the filesystem remains but the network links (and containers/pods) may disappear. The next time that Cilium starts up and restores the endpoints from the filesystem, Cilium constantly complaining in cilium status with a controller that is unhealthy:
$ cilium status
...
Controller Status:   52/53 healthy
...
  endpoint-2594-regeneration-recovery         never          4m25s ago    188     regeneration recovery failed

This message continually repeats in the endpoint log for the relevant endpoint:
2019-11-12T15:58:42Z   Failure   ready                   Error regenerating endpoint: Failed to replace Qdisc for lxc714f0a329989: Link not found
2019-11-12T15:58:42Z   OK        ready                   Completed endpoint regeneration with no pending regeneration requests
2019-11-12T15:58:42Z   OK        regenerating            Regenerating endpoint: one or more identities created or deleted
2019-11-12T15:58:42Z   OK        waiting-to-regenerate   Triggering endpoint regeneration due to one or more identities created or deleted

We should detect this case where the underlying devices/containers have been removed, log a warning that we are failing to restore an endpoint because it disappeared before Cilium started up, then either do not start the endpoint-xxx-regeneration-recovery controller, or stop that controller and remove the endpoint.

More information:
I tried:

Deploying a demo application
Deploying Cilium without tolerations
Tainting a node
Deleting Cilium pods (so it's not redeployed)
Deleting the demo app
Removing the tolerations so that Cilium will redeploy/restore

In this case, upon restore, Cilium attempts to probe k8s to check that the pod is still alive, and because the resource was deleted, it reports:
level=warning msg="Unable to restore endpoint, ignoring" endpointID=2059 error="kubernetes pod not found" k8sPodName=default/testclient-j625r subsys=datapath

...which suggests that the problem is a bit more nuanced than simply removing the pod. It's possible that the pod is still available in some capacity, but not fully removed. I have not attempted recreating the pod again with the same name to see if this allows the problem to be reproduced.

PR https://github.com/cilium/cilium/pull/9586 should probably help with this...

@aanm @joestringer Did #9586 fix this?

@tgraf As far as I'm aware, the original reporter of the bug has not yet tried a release of Cilium with #9586 in it to confirm.

Fixed and backported by https://github.com/cilium/cilium/pull/9551


Fix the wrong IndexKey

I tried to run the getProductInfo function with inclTokens on, but occurred an error that appid is not defined.
So based on the context, I think you wanna put packageid here, but for some reasons you put appid instead of packageid.

:+1: Thanks!


feat: migrate to Django and drf

Currently this project uses FastAPI, but now I'm thinking to rewrite this in django and django-rest-framework.
since I use this kinda scraping logics for mangacore ( a project for streaming mangas ), it would be better to try this in django and drf.
I'd like hear your opinions.

This is not necessary. Instead of writing this here, why not open an issue in mainline coreproject ?

why not open an issue in mainline coreproject ?

Yes, but mainline project is already using django x drf. Also I thought this would be a nice warmup for me :)

Yes, but mainline project is already using django x drf.


Test issue from WalletCore

Replicated connection and logic from WalletDesktop ...

Exalate commented: GitHub source link is https://github.com/Lumerin-protocol/WalletCore/issues/32

Alan Sorber commented: Updated Label to include WalletCore

This is a comment from GitHub

Alan Sorber commented: Moved to done then added final comment


Satellites doesn't verify a card was melded before trying to auto-execute

NOTE: Before you begin working on this, please assign this issue to yourself. If you do not do this, then there's the possibility that 2 people will end up doing the same task, which is not a good use of time.
This is a reported bug on BGA.
Basically, the logic makes the assumption that a card was melded:
            case "90N3A":
                $card = self::getCardInfo(self::getGameStateValue('id_last_selected')); // The card the player melded from his hand
                self::executeNonDemandEffects($card); // "Execute each of its non-demand dogma effects"
                break;

It doesn't verify that one was melded with the last dogma step.  Socialism can take the 8s in hand before the originator has a chance to go.  In that case, satellites should not do anything in that last step.

a simple if ($n > 0) would do the trick to fix this behavior.

This was deployed a few weeks ago.


Load historical soundtracks.

Override EffectListener. Add display soundtrack. Those records which were already listened shouldn't be loaded again during one session.

Fixed by commit 3a3ee30592d1784bd60ffaaed988a4a3f4ebb5bb


Flow losing types when importing function which returns classes.

Hello!
Flow types works correctly with the code below:
/* @flow */
const movable = (Base): Class<*> => {
  return class extends Base {
    move() {
      console.log('move');
    }
  }
}

class Person {
  firstName: string;

  constructor(firstName: string) {
    this.firstName = firstName;
  }
}

const MovablePerson = movable(Person);
const p = new MovablePerson('John');

p.firstName = 1;
// Flow =>
// 22: p.firstName = 1;
//                  ^ number. This type is incompatible with
// 12:   firstName: string;
//                  ^ string

But the same code splitted into multiple .js files don't:
// Movable.js
export const movable = (Base): Class<*> => {
  return class extends Base {
    move() {
      console.log('move');
    }
  }
}

// Main.js
import {movable} from './Movable.js';

class Person {
  firstName: string;

  constructor(firstName: string) {
    this.firstName = firstName;
  }
}

const MovablePerson = movable(Person);
const p = new MovablePerson('John');

p.firstName = 1;
// Flow doesn't complain anymore but should

Hope that will help

These import related issues should be resolved a long time ago since types first


POP3 support

Hi,
Could you add pop3/pop3s support to mail server?
Regards,
Zoli

@ptzool : You can now enable POP3(S) protocol with ENABLE_POP3 environment variable.

Wow! Thank you!


fix(deps): update dependency date-fns to v2.14.0

This PR contains the following updates:

Package
Type
Update
Change

date-fns
dependencies
minor
2.13.0 -> 2.14.0

Release Notes

date-fns/date-fns
v2.14.0
Compare Source
Kudos to @julamb, @JacobSoderblom, @justingrant, @dragunoff, @jmate0321, @gbhasha, @rasck, @AlbertoPdRF, @sebastianhaberey and @giogonzo for working on the release!
Fixed

Fixed DST issues with add, addDays and addMonths.
Fixed "quarter" translation in the Bulgarian locale.
Fixed formatDistance strings in the Hungarian locale.
Fixed Danish month abbreviations.
Fixed parsing of mei in the Dutch locale.
Fixed missing preposition in formatLong in the Spanish locale.
Fixed formatRelative in the Italian locale.

Added

Added eachQuarterOfInterval.
Added Basque (eu) locale.
Added Indian English (en-IN) locale.
Added eachHourOfInterval.

Renovate configuration
:date: Schedule: At any time (no schedule defined).
:vertical_traffic_light: Automerge: Disabled by config. Please merge this manually once you are satisfied.
:recycle: Rebasing: Whenever PR becomes conflicted, or if you modify the PR title to begin with "rebase!".
:no_bell: Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

Available now for Enterprise: Renovate Pro with real-time webhook handling and priority job queue.

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

v5.0.46
GitHub release

Your semantic-release bot :package::rocket:


Change Image - User App in Teams

Do I have to reupload the manifest zip to update the image for the User app in Teams?
We would like to use a custom image as the thumbnail

Hi @JoeSMGraham ,
You will have to reupload the manifest zip to use the custom image.

@JoeSMGraham , Could you please confirm if we are good to close this request?


there is no arg for --model_name in prepare_whisper_configs.py.

In the readme file, under the header "Create an ONNX Model", where it says to "Run the following commands":
python prepare_whisper_configs.py --model_name openai/whisper-tiny.en --no_audio_decoder python -m olive.workflows.run --config whisper_cpu_int8.json --setup python -m olive.workflows.run --config whisper_cpu_int8.json
there is no arg for --model_name in prepare_whisper_configs.py.

you're referring to https://github.com/microsoft/onnxruntime-inference-examples/tree/main/js/ort-whisper#create-an-onnx-model , correct?
+@guschmue who commited the README file.

at the time when I write this, I'm sure that worked.
Hope olive did not change args. Let me check.

for my sanity: 3abdb393ce3d420ba5a6fd9077eb42bfc8f3d614 had this option, still need to see when it changed.
fyi @devang-ml

We need to update this README. It was removed as part of the example code reorg. See https://github.com/microsoft/Olive/tree/main/examples/whisper

There is a PR to reintroduce the --model_name options. Waiting for it to be merged.
https://github.com/microsoft/Olive/pull/404


Feature/contact contribution

hi, here is the first change i have done :)
packages had to be updated to resolve dependencies on macOS

I have deployed this live @indiraberaud :)


prints

There's lots of prints in the code. They should be deleted, or converted to use status bar instead

Solved in latest push

Solved in latest push


[DOC] Remove reference to reset_index_inplace in example notebook

#262
https://github.com/ericmjl/pyjanitor/blob/dev/examples/notebooks/pyjanitor_intro.ipynb

.reset_index_inplace() is a pyjanitor stand-in for df.reset_index(inplace=True), because the latter does not return the DataFrame when the in place mode is used and is therefore not chainable.

I would like to work on this issue


Add Visualforce pages to "Mass Action Admin" permission set

Based on this conversation.
In the 2.2 release, the Mass Action Admin permission set does not include the Visualforce pages referenced by the app, which leads to LC_API: Timeout trying to establish connection to iframe errors.

Release 2.3 is now available per #99, please see the release notes for full details. Thank you!


rename koordinator priority class name

Signed-off-by: Tao Li<EMAIL_ADDRESS>Ⅰ. Describe what this PR does
Ⅱ. Does this pull request fix one issue?

Ⅲ. Describe how to verify it
Ⅳ. Special notes for reviews

/retest

/lgtm
/approve


Does not work with dynamic Swift methods (Swift 2.0)

I have a Swift subclass of UITableViewCell and an @objc method that takes one parameter (NSError).
Hooking this selector gives no errors, but the aspect hook is NOT invoked.
(Xcode 7 beta 6)

Haven't tried this with Swift yet - feel free to dig on to see what's up here.
cheers,
Peter

On 02 Sep 2015, at 13:25, Sergey Novitsky<EMAIL_ADDRESS>wrote:
I have a Swift subclass of UITableViewCell and an @objc method that takes one parameter (NSError).
Hooking this selector gives no errors, but the aspect hook is NOT invoked.
(Xcode 7 beta 6)
―
Reply to this email directly or view it on GitHub.

Was my fault - I forgot the "dynamic" keyword, and only marked my method as @objc (which was NOT enough!). Duh... :-)

Thanks for the awesome library, btw! I am using it in almost every project (for diagnostics logging).

Great to hear! Thanks!

On 02 Sep 2015, at 13:59, Sergey Novitsky<EMAIL_ADDRESS>wrote:
Thanks for the awesome library, btw! I am using it in almost every project (for diagnostics logging).
―
Reply to this email directly or view it on GitHub.

@serieuxchat Can you please share some snippets of how are you using with Swift and what keywords need to be added ? any limitations ?


ctr use client's underlying service

Signed-off-by: Tao Qingyun<EMAIL_ADDRESS>

Codecov Report

Merging #1673 into master will increase coverage by 0.19%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1673      +/-   ##
==========================================
+ Coverage   48.72%   48.92%   +0.19%
==========================================
  Files          27       27
  Lines        4076     4076
==========================================
+ Hits         1986     1994       +8
+ Misses       1671     1667       -4
+ Partials      419      415       -4

Impacted Files
Coverage Δ

metadata/snapshot.go
50.28% <0%> (+0.93%)
:arrow_up:

metadata/db.go
67% <0%> (+1.5%)
:arrow_up:

Continue to review full report at Codecov.

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

LGTM
Thanks for the long overdue cleanup!

LGTM


Documentation doesn't work in Microsoft Edge

Microsoft Edge fullscreen compared to Google Chrome on top.

seems to be fine on my edge.


Create boo

Collaborator cannot find remote user mapping for pull request creator, review created by system administrator

Collaborator: Review for this pull request was created. Link to review


How to create an instance of a generic dictionary?

I tried to follow the example using "dictionaryClass.GetGeneric({BNM::GetType(), BNM::GetType()})" but it leads to the errors
07-13 21:53:32.474 18414 18414 E MD      : An attempt to use dead class!
07-13 21:53:32.474 18414 18414 E MD      : Null object in the iterator
07-13 21:53:32.474 18414 18414 E MD      : An attempt to use dead class!
07-13 21:53:32.474 18414 18414 E MD      : Null object in the iterator

Try update BNM to v2.0.2


[BI-1144] correct parsing of scale value and scale label

for https://breedinginsight.atlassian.net/browse/BI-1144

@davedrp Can you change the target branch for this PR to be releases/0.4 please?


add packages to setup.py

I didn't set up a database to run all of the tests, so I don't think it's worth adding those "tricks". Ultimately I set up an environment on golem so that I could easily access the NSRDB data.


Add jest unit test

What does this PR do?
This PR is raised to implement the following features: 👍 :

[x] Setup jest to add unit tests.
[x] Add unit test for the add task function.
[x] Add unit test for the remove task function.
[x] Install buble to use es6 import and export. 😀

Description of the tasks completed 👇

[x] Add the HTML for the structure.
[x] Add the CSS file for the styling.
[x] Add the JavaScript file for adding interactions.
[x] Add unit tests for add and remove functions.
[x] Use the linter tool that analyzes my source code to flag programming errors, bugs, stylistic errors, and suspicious constructs. 💯
[x] Use Flexbox a layout tool in CSS that helps you easily position HTML elements on the page.
[x] Check the accessibility of this project which means that websites, tools, and technologies are designed and developed so that people with disabilities ♿ can use them. For more information about Web accessibility Click here☺

How should it be tested
Clone the repository
git clone https://github.com/UbdaNam/To-Do-list.git

Navigate to the folder
cd To-Do-list

Checkout to the branch
git checkout Add-jest-unit-test

Run tests
npm test

Thanks for reviewing my PR 😊

@aamirkhan2478  Thanks for your review there was an unnecessary test report folder added. I've removed that and am ready to request another review thanks 👍


Add a parameter name_len to kni_set_name() in virtio_user.c

To suppress the compilation warning.
The default size VDEV_NAME_SIZE is 256, but the size of ifr.ifr_name is 16.

hi zcjie1,
Thank you for contributing code to dperf. I have seen your repository and it looks great. I look forward to your contribution of more features to dperf.


Deployment example values.yaml for Google Kubernetes Engine

Create example values.yaml to deploy Terrakube in GKE

I have run into specific issue on GKE, it would be nice to have this finished.

I have run into specific issue on GKE, it would be nice to have this finished.

You could check this issue for gcp
https://github.com/AzBuilder/terrakube-helm-chart/issues/61


Error log

Hi,
The log shows these errors, how to fix?

Traccar expect JSON data in the attributes field in the database. Looks like you have some other format there.

I use the original version ... in that format are then?

What version are you using? Have you updated from some older version?

This: https://d.traccar.litvak.su

That is not original version. It's not official Traccar project.

Here are the main bugs of the version mobile:
The zoom touch doesn't work with the android default browser.
The zoom touch works property on Firefox and chrome browser but the my localization function doesn't work.
Please check it
Thanks

we will contact you via email for the development of an app ...


unexpected keyword argument 'serial_number'

Installed v.0.2.5 on Home Assistant v.2023.9.3 via HACS v.1.34.0. After starting up HA, got the following error for this component:
Log details (ERROR)
This error originated from a custom integration.
Logger: homeassistant.config_entriesSource: custom_components/bradford_white_connect/__init__.py:47Integration: Bradford White Connect (documentation, issues)First occurred: 6:44:44 PM (1 occurrences)Last logged: 6:44:44 PM
Error setting up entry<EMAIL_ADDRESS>for bradford_white_connect
Traceback (most recent call last):
  File "/srv/homeassistant/lib/python3.11/site-packages/homeassistant/config_entries.py", line 387, in async_setup
    result = await component.async_setup_entry(hass, self)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/homeassistant/.homeassistant/custom_components/bradford_white_connect/__init__.py", line 47, in async_setup_entry
    device_registry.async_get_or_create(
TypeError: DeviceRegistry.async_get_or_create() got an unexpected keyword argument 'serial_number'

Went to that __init__.py file & commented out the line (a couple after line 47) that assigns the "serial_number". Then restarted HA and the component worked without errors.
Not sure how important that line is, but I don't mind missing my water heater's serial number. If there's a better fix, feel free to do so.

Thanks, it seems like we should update the code to allow for these type of fields to be missing.

This should be fixed in the latest release.

Updated to v.0.2.7 and the issue persists. Had to comment out the lines that assign the "serial_number".


Add functionality to stop command execution of script_runner

We don't have any functionality to stop execution of command.txt lines, like say if we have these lines:
/home/orangepi/Desktop/projects/orangepi/stream:python3 esp32cam.py
/home/orangepi/Desktop/projects/orangepi/stream:python3 lenovo_cam.py

What if we want to stop execution of second line for now, I have to remove it, but we need to add a functionality so a user can skip its execution without removing it like this:
/home/orangepi/Desktop/projects/orangepi/stream:python3 esp32cam.py
[skip]/home/orangepi/Desktop/projects/orangepi/stream:python3 lenovo_cam.py


PID doesnt update until current temp change

A fundamental flaw in this PID i have seen so far is the lack of updating while the current temp stays the same. There are scenarios you can think of where the temperature is stable but the setpoint is changed 2 degrees higher. For as far as i have seen this PID only changes its output when the current temp is changing. so if for example the temperature is 10 degrees to cold it wil stay at 0 as long as the current temp is not changing. whil it should tune up the value as long as there is a difference. not waiting for the temp to change.

What sort of thermostat doesn't change its output when the setpoint changes? I need it to preferably change whenever the input, p|i|d parameter or setpoint changes. I tried forcing it to update using the homeassistant update entity service but that does not work (neither the pid sensor itself or the temperature sensor)


Support calculating number of chars for StringArray

Which issue does this PR close?

Closes #1493 .
Rationale for this change
We need a method to calculate the number of chars for StringArray
What changes are included in this PR?
Provide a pub method num_chars.

Codecov Report

Merging #1503 (bc9ef92) into master (f1eba3c) will increase coverage by 0.03%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #1503      +/-   ##
==========================================
+ Coverage   82.70%   82.73%   +0.03%
==========================================
  Files         188      188
  Lines       54403    54359      -44
==========================================
- Hits        44993    44974      -19
+ Misses       9410     9385      -25

Impacted Files
Coverage Δ

arrow/src/array/array_string.rs
97.70% <100.00%> (+0.04%)
:arrow_up:

arrow/src/datatypes/datatype.rs
66.40% <0.00%> (-0.40%)
:arrow_down:

arrow/src/array/transform/mod.rs
86.46% <0.00%> (-0.12%)
:arrow_down:

arrow/src/buffer/mutable.rs
90.53% <0.00%> (+0.02%)
:arrow_up:

arrow/src/compute/kernels/take.rs
95.41% <0.00%> (+0.13%)
:arrow_up:

parquet/src/arrow/array_reader.rs
86.70% <0.00%> (+2.34%)
:arrow_up:

Continue to review full report at Codecov.

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

Thanks @HaoYang670  and @liukun4515  for the review


blockprovider test flakes

Type of change

Improvement (improvement to code, performance, etc)

Description
Increase Eventually timeout to reduce flakes.

Thanks, LGTM.
This one closes #4832, right?

I hope so...


Improve blob reader performance

Description of changes:
Improve performance of collecting blob body by usingarrayBuffer()method.
If one or more of the packages in the /packages directory has been modified, be sure yarn changeset add has been run and its output has
been committed and included in this pull request. See CONTRIBUTING.md.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

https://caniuse.com/mdn-api_blob_arraybuffer


Removes DBT_REPOSITORY_PATH

Removes an unnecessary variable, and removes duplicated code from our init scripts.
Done while working on https://github.com/openedx/openedx-aspects/issues/73.

Thanks for the pull request, @pomegranited! Please note that it may take us up to several weeks or months to complete a review and merge your PR.
Feel free to add as much of the following information to the ticket as you can:

supporting documentation
Open edX discussion forum threads
timeline information ("this must be merged by XX date", and why that is)
partner information ("this is a course on edx.org")
any other information that can help Product understand the context for the PR

All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here.
Please let us know once your PR is ready for our review and all tests are green.

@pomegranited 🎉 Your pull request was merged!  Please take a moment  to answer a two question survey so we can improve your experience in the future.


TCI fails to compress a low rank tensor

Hi again :)
More than an issue this is something that together with a colleague we have observed when trying to compress certain objects with TCI and compare the result with what we would obtain with SVD.
We are trying to learn a 14-qubit state with wave-function coefficients being all positive and non-zero. As an example: when applying tci2 on the state we see that tci2 converges to maxbonddim 80 although the maxbonddim parameter is set to 128. Moreover, we observe that the pivoterrors are all 0 but the relative error of the entries is 40%! We also see that when doing the usual svd compression on the tensor we obtain an MPS with maxbonddim~5.
Is there a way to check where the issues come from here? Why would tci2 fail? Is there anything we can try to make it work?
Would greatly appreciate any suggestion/insight you can offer
Thanks!

Hi, could you give us more details?

How do you define the relative error? The error devided by the true value elementwise?
What are the tolerance for TCI2 and SVD?

Sorry for the late reply. I was trying to prepare a minimal working example where the issue emerges. It seems the pivot optimization does not manage to find a good set of initial pivots so perhaps this could be the problem...
I attach the code and an h5 file with an example of a problematic tensor
Archive.zip
Thanks once again!

Thank you for the minimal example. It helped a lot.
I think there is seems to be an issue in the test script rather than TCI.jl.
The following code assumes row-major but Julia uses column-major like Fortran:
f(x::Vector{Int}) = vector[parse(Int, join(x .- 1); base=2)+1]

We can avoid this issue simply as
tensor = reshape(vector, localdims...)
f(x::Vector{Int}) = tensor[x...]

Enclosed includes my fix (Please find the comment "HERE IS THE FIX").
test_fixed.jl.zip

Oh, I apologize for this mistake :( No matter what I tried nothing worked and look where the error was ...
Thanks again and sorry for all the trouble

@RaimelMedina
You do not need to apologize!
If you want to handle quantics indices, you could use QuanticsGrids.jl, which provides conversions between different representations such as quantics and linear grid index.


Update requirements.txt

Existing requirements.txt wasn't sufficient to run the code.  Also requires the added dependencies.  There appears to be some flexibility with respect to sklearn and pillow version but newer versions of h5py introduce incompatibilities with torch 1.0.0.

thanks Matthew, I have given you the access to this repo and you can merge this PR.


test: add code coverage

Added extension to display coverage reports
Updated docs with steps on how to generate and visualize code coverage

Awesome addition!
For the future, as we talked about, we can consider checking to automatically run the coverage as part of running tests and we could also look into generating a report with the pipeline so we can view the coverage as part of PR review.

Good idea to integrate coverage into VSCode test explorer, it's probably worth considering pytest-cov python 3.x - Running pytest-cov along with pytest in VS Code - Stack Overflow, but it causes some known issues with debugger - Debugger doesn't stop at breakpoints with pytest if pytest-cov is used · Issue #693 · microsoft/vscode-python (github.com) forwhich there might be a workaround - Testing Python in Visual Studio Code. Needs further testing. As agreed, I will close this PR as it is right now and we can have this as a possible improvement.


Improvement suggestion for deletion interface

You can use almost the same syntax as updating:
del table["column1", "WHERE column2='something'"]

BTW I'm EntityBlank from FXP, I think we were in a Skype group together for a while :)

Hey! Thank you for the suggestion.
I really want to do it but I'm not sure how it works.
And yeah, we were on Omer's group. Aren't you in the current Whatsapp group? 🤔

I really want to do it ut I'm not sure how it works.

Check out the __delitem__ magic method

Aren't you in the current Whatsapp group?

Nope, I'll send you my phone number to your email :)

Added and pushed into GitHub.
PyPi packaged is also updated.
Thank you very much! ;)


Memory leak in client

Last night I noticed that my ZDEW client was consuming a vast amount of memory and was a bit unresponsive.  So, I crashed out of it, and then restarted.  Tonight I checked back in, and it was back being high again.  I have not used the ZDEW for anything.  The laptop has been put to sleep a few times, but that is it.  Just by watching the Task Manager over 5-10 minutes I can see the memory usage increasing.

App version is <IP_ADDRESS>
How can I help to produce any data for this.

Yikes. Are there any network configuration changes that have happened lately? This just happened out of nowhere? Best thing to do will be to get a set of logs to share, assuming you're comfortable with that and assuming it's reproducible. If it's NOT reproducible, just collect a feedback zip and send to us to inspect.
If you can reproduce the issue:

Stop zdew
Clean up/remove existing log files (you'll have to do this manually)
start zdew
wait until you see the memory spike up over 500 mb
Get a feedback zip file, send to help at OpenZiti.org

So, I have been doing some investigation.  You mentioned if any network changes, so have been doing some experimentation.  It seems to go balistic when I have virtual box open.
This is what is repeating in the logs (UI log) when I have Virtual Box machine open
[2023-06-15T09:42:43.205Z] DEBUG	ZitiDesktopEdge.MainWindow	==== ControllerEvent    : action:connected identifier:C:\WINDOWS\system32\config\systemprofile\AppData\Roaming/NetFoundry/KelvinWorkLaptop.json
[2023-06-15T09:42:43.277Z] TRACE	ZitiDesktopEdge.ServiceClient.DataClient	{"Op":"identity","Action":"added","Fingerprint":"KelvinWorkLaptop","Id":{"Name":"HDC Kelvin Work Laptop","Identifier":"C:\\WINDOWS\\system32\\config\\systemprofile\\AppData\\Roaming/NetFoundry/KelvinWorkLaptop.json","FingerPrint":"KelvinWorkLaptop","Active":true,"Loaded":true,"Config":{"ztAPI":"https://ziticontroller.horowhenua.govt.nz:443"},"ControllerVersion":"v0.27.9","IdFileStatus":true,"MfaEnabled":false,"MfaNeeded":false,"Metrics":{"Up":0,"Down":0},"MfaMinTimeout":0,"MfaMaxTimeout":0,"MfaMinTimeoutRem":0,"MfaMaxTimeoutRem":0,"MinTimeoutRemInSvcEvent":0,"MaxTimeoutRemInSvcEvent":0,"Deleted":false,"Notified":false}}
[2023-06-15T09:42:43.277Z]  INFO	ZitiDesktopEdge.Models.ZitiIdentity	Identity: HDC Kelvin Work Laptop updated To {"Name":"HDC Kelvin Work Laptop","FingerPrint":"KelvinWorkLaptop","Identifier":"C:\\WINDOWS\\system32\\config\\systemprofile\\AppData\\Roaming/NetFoundry/KelvinWorkLaptop.json","Active":true,"Config":{"ztAPI":"https://ziticontroller.horowhenua.govt.nz:443","id":null,"configTypes":null},"Status":null,"Services":null,"Metrics":{"TotalBytes":0,"Up":0,"Down":0},"ControllerVersion":"v0.27.9","MfaEnabled":false,"MfaNeeded":false,"MinTimeout":0,"MaxTimeout":0,"MfaLastUpdatedTime":"0001-01-01T00:00:00"}
[2023-06-15T09:42:43.277Z] DEBUG	ZitiDesktopEdge.MainWindow	==== IdentityEvent    : action:added identifer:C:\WINDOWS\system32\config\systemprofile\AppData\Roaming/NetFoundry/KelvinWorkLaptop.json name:HDC Kelvin Work Laptop
[2023-06-15T09:42:43.319Z] DEBUG	ZitiDesktopEdge.MainWindow	IDENTITY EVENT. Action: added identifier: C:\WINDOWS\system32\config\systemprofile\AppData\Roaming/NetFoundry/KelvinWorkLaptop.json
[2023-06-15T09:42:43.319Z] TRACE	ZitiDesktopEdge.ServiceClient.DataClient	{"Op":"controller","Action":"disconnected","Identifier":"C:\\WINDOWS\\system32\\config\\systemprofile\\AppData\\Roaming/NetFoundry/KelvinWorkLaptop.json","Fingerprint":"KelvinWorkLaptop"}
[2023-06-15T09:42:43.319Z] DEBUG	ZitiDesktopEdge.ServiceClient.DataClient	Controller event received
[2023-06-15T09:42:43.319Z] DEBUG	ZitiDesktopEdge.MainWindow	==== ControllerEvent    : action:disconnected identifier:C:\WINDOWS\system32\config\systemprofile\AppData\Roaming/NetFoundry/KelvinWorkLaptop.json
[2023-06-15T09:42:43.524Z] TRACE	ZitiDesktopEdge.ServiceClient.DataClient	{"Op":"identity","Action":"added","Fingerprint":"KelvinWorkLaptop","Id":{"Name":"HDC Kelvin Work Laptop","Identifier":"C:\\WINDOWS\\system32\\config\\systemprofile\\AppData\\Roaming/NetFoundry/KelvinWorkLaptop.json","FingerPrint":"KelvinWorkLaptop","Active":true,"Loaded":true,"Config":{"ztAPI":"https://ziticontroller.horowhenua.govt.nz:443"},"ControllerVersion":"v0.27.9","IdFileStatus":true,"MfaEnabled":false,"MfaNeeded":false,"Metrics":{"Up":0,"Down":0},"MfaMinTimeout":0,"MfaMaxTimeout":0,"MfaMinTimeoutRem":0,"MfaMaxTimeoutRem":0,"MinTimeoutRemInSvcEvent":0,"MaxTimeoutRemInSvcEvent":0,"Deleted":false,"Notified":false}}
[2023-06-15T09:42:43.524Z]  INFO	ZitiDesktopEdge.Models.ZitiIdentity	Identity: HDC Kelvin Work Laptop updated To {"Name":"HDC Kelvin Work Laptop","FingerPrint":"KelvinWorkLaptop","Identifier":"C:\\WINDOWS\\system32\\config\\systemprofile\\AppData\\Roaming/NetFoundry/KelvinWorkLaptop.json","Active":true,"Config":{"ztAPI":"https://ziticontroller.horowhenua.govt.nz:443","id":null,"configTypes":null},"Status":null,"Services":null,"Metrics":{"TotalBytes":0,"Up":0,"Down":0},"ControllerVersion":"v0.27.9","MfaEnabled":false,"MfaNeeded":false,"MinTimeout":0,"MaxTimeout":0,"MfaLastUpdatedTime":"0001-01-01T00:00:00"}
[2023-06-15T09:42:43.524Z] DEBUG	ZitiDesktopEdge.MainWindow	==== IdentityEvent    : action:added identifer:C:\WINDOWS\system32\config\systemprofile\AppData\Roaming/NetFoundry/KelvinWorkLaptop.json name:HDC Kelvin Work Laptop
[2023-06-15T09:42:43.552Z] DEBUG	ZitiDesktopEdge.MainWindow	IDENTITY EVENT. Action: added identifier: C:\WINDOWS\system32\config\systemprofile\AppData\Roaming/NetFoundry/KelvinWorkLaptop.json
[2023-06-15T09:42:43.552Z] TRACE	ZitiDesktopEdge.ServiceClient.DataClient	{"Op":"controller","Action":"connected","Identifier":"C:\\WINDOWS\\system32\\config\\systemprofile\\AppData\\Roaming/NetFoundry/KelvinWorkLaptop.json","Fingerprint":"KelvinWorkLaptop"}
[2023-06-15T09:42:43.552Z] DEBUG	ZitiDesktopEdge.ServiceClient.DataClient	Controller event received
[2023-06-15T09:42:43.552Z] DEBUG	ZitiDesktopEdge.MainWindow	==== ControllerEvent    : action:connected identifier:C:\WINDOWS\system32\config\systemprofile\AppData\Roaming/NetFoundry/KelvinWorkLaptop.json
[2023-06-15T09:42:43.564Z] TRACE	ZitiDesktopEdge.ServiceClient.DataClient	{"Op":"identity","Action":"added","Fingerprint":"KelvinWorkLaptop","Id":{"Name":"HDC Kelvin Work Laptop","Identifier":"C:\\WINDOWS\\system32\\config\\systemprofile\\AppData\\Roaming/NetFoundry/KelvinWorkLaptop.json","FingerPrint":"KelvinWorkLaptop","Active":true,"Loaded":true,"Config":{"ztAPI":"https://ziticontroller.horowhenua.govt.nz:443"},"ControllerVersion":"v0.27.9","IdFileStatus":true,"MfaEnabled":false,"MfaNeeded":false,"Metrics":{"Up":0,"Down":0},"MfaMinTimeout":0,"MfaMaxTimeout":0,"MfaMinTimeoutRem":0,"MfaMaxTimeoutRem":0,"MinTimeoutRemInSvcEvent":0,"MaxTimeoutRemInSvcEvent":0,"Deleted":false,"Notified":false}}
[2023-06-15T09:42:43.564Z]  INFO	ZitiDesktopEdge.Models.ZitiIdentity	Identity: HDC Kelvin Work Laptop updated To {"Name":"HDC Kelvin Work Laptop","FingerPrint":"KelvinWorkLaptop","Identifier":"C:\\WINDOWS\\system32\\config\\systemprofile\\AppData\\Roaming/NetFoundry/KelvinWorkLaptop.json","Active":true,"Config":{"ztAPI":"https://ziticontroller.horowhenua.govt.nz:443","id":null,"configTypes":null},"Status":null,"Services":null,"Metrics":{"TotalBytes":0,"Up":0,"Down":0},"ControllerVersion":"v0.27.9","MfaEnabled":false,"MfaNeeded":false,"MinTimeout":0,"MaxTimeout":0,"MfaLastUpdatedTime":"0001-01-01T00:00:00"}
[2023-06-15T09:42:43.564Z] DEBUG	ZitiDesktopEdge.MainWindow	==== IdentityEvent    : action:added identifer:C:\WINDOWS\system32\config\systemprofile\AppData\Roaming/NetFoundry/KelvinWorkLaptop.json name:HDC Kelvin Work Laptop
[2023-06-15T09:42:43.615Z] DEBUG	ZitiDesktopEdge.MainWindow	IDENTITY EVENT. Action: added identifier: C:\WINDOWS\system32\config\systemprofile\AppData\Roaming/NetFoundry/KelvinWorkLaptop.json
[2023-06-15T09:42:43.615Z] TRACE	ZitiDesktopEdge.ServiceClient.DataClient	{"Op":"controller","Action":"disconnected","Identifier":"C:\\WINDOWS\\system32\\config\\systemprofile\\AppData\\Roaming/NetFoundry/KelvinWorkLaptop.json","Fingerprint":"KelvinWorkLaptop"}
[2023-06-15T09:42:43.615Z] DEBUG	ZitiDesktopEdge.ServiceClient.DataClient	Controller event received
[2023-06-15T09:42:43.615Z] DEBUG	ZitiDesktopEdge.MainWindow	==== ControllerEvent    : action:disconnected identifier:C:\WINDOWS\system32\config\systemprofile\AppData\Roaming/NetFoundry/KelvinWorkLaptop.json
[2023-06-15T09:42:43.819Z] TRACE	ZitiDesktopEdge.ServiceClient.DataClient	{"Op":"identity","Action":"added","Fingerprint":"KelvinWorkLaptop","Id":{"Name":"HDC Kelvin Work Laptop","Identifier":"C:\\WINDOWS\\system32\\config\\systemprofile\\AppData\\Roaming/NetFoundry/KelvinWorkLaptop.json","FingerPrint":"KelvinWorkLaptop","Active":true,"Loaded":true,"Config":{"ztAPI":"https://ziticontroller.horowhenua.govt.nz:443"},"ControllerVersion":"v0.27.9","IdFileStatus":true,"MfaEnabled":false,"MfaNeeded":false,"Metrics":{"Up":0,"Down":0},"MfaMinTimeout":0,"MfaMaxTimeout":0,"MfaMinTimeoutRem":0,"MfaMaxTimeoutRem":0,"MinTimeoutRemInSvcEvent":0,"MaxTimeoutRemInSvcEvent":0,"Deleted":false,"Notified":false}}
[2023-06-15T09:42:43.819Z]  INFO	ZitiDesktopEdge.Models.ZitiIdentity	Identity: HDC Kelvin Work Laptop updated To {"Name":"HDC Kelvin Work Laptop","FingerPrint":"KelvinWorkLaptop","Identifier":"C:\\WINDOWS\\system32\\config\\systemprofile\\AppData\\Roaming/NetFoundry/KelvinWorkLaptop.json","Active":true,"Config":{"ztAPI":"https://ziticontroller.horowhenua.govt.nz:443","id":null,"configTypes":null},"Status":null,"Services":null,"Metrics":{"TotalBytes":0,"Up":0,"Down":0},"ControllerVersion":"v0.27.9","MfaEnabled":false,"MfaNeeded":false,"MinTimeout":0,"MaxTimeout":0,"MfaLastUpdatedTime":"0001-01-01T00:00:00"}
[2023-06-15T09:42:43.819Z] DEBUG	ZitiDesktopEdge.MainWindow	==== IdentityEvent    : action:added identifer:C:\WINDOWS\system32\config\systemprofile\AppData\Roaming/NetFoundry/KelvinWorkLaptop.json name:HDC Kelvin Work Laptop
[2023-06-15T09:42:43.858Z] DEBUG	ZitiDesktopEdge.MainWindow	IDENTITY EVENT. Action: added identifier: C:\WINDOWS\system32\config\systemprofile\AppData\Roaming/NetFoundry/KelvinWorkLaptop.json
[2023-06-15T09:42:43.858Z] TRACE	ZitiDesktopEdge.ServiceClient.DataClient	{"Op":"controller","Action":"connected","Identifier":"C:\\WINDOWS\\system32\\config\\systemprofile\\AppData\\Roaming/NetFoundry/KelvinWorkLaptop.json","Fingerprint":"KelvinWorkLaptop"}
[2023-06-15T09:42:43.858Z] DEBUG	ZitiDesktopEdge.ServiceClient.DataClient	Controller event received
[2023-06-15T09:42:43.858Z] DEBUG	ZitiDesktopEdge.MainWindow	==== ControllerEvent    : action:connected identifier:C:\WINDOWS\system32\config\systemprofile\AppData\Roaming/NetFoundry/KelvinWorkLaptop.json
[2023-06-15T09:42:43.858Z] TRACE	ZitiDesktopEdge.ServiceClient.DataClient	{"Op":"identity","Action":"added","Fingerprint":"KelvinWorkLaptop","Id":{"Name":"HDC Kelvin Work Laptop","Identifier":"C:\\WINDOWS\\system32\\config\\systemprofile\\AppData\\Roaming/NetFoundry/KelvinWorkLaptop.json","FingerPrint":"KelvinWorkLaptop","Active":true,"Loaded":true,"Config":{"ztAPI":"https://ziticontroller.horowhenua.govt.nz:443"},"ControllerVersion":"v0.27.9","IdFileStatus":true,"MfaEnabled":false,"MfaNeeded":false,"Metrics":{"Up":0,"Down":0},"MfaMinTimeout":0,"MfaMaxTimeout":0,"MfaMinTimeoutRem":0,"MfaMaxTimeoutRem":0,"MinTimeoutRemInSvcEvent":0,"MaxTimeoutRemInSvcEvent":0,"Deleted":false,"Notified":false}}
[2023-06-15T09:42:43.858Z]  INFO	ZitiDesktopEdge.Models.ZitiIdentity	Identity: HDC Kelvin Work Laptop updated To {"Name":"HDC Kelvin Work Laptop","FingerPrint":"KelvinWorkLaptop","Identifier":"C:\\WINDOWS\\system32\\config\\systemprofile\\AppData\\Roaming/NetFoundry/KelvinWorkLaptop.json","Active":true,"Config":{"ztAPI":"https://ziticontroller.horowhenua.govt.nz:443","id":null,"configTypes":null},"Status":null,"Services":null,"Metrics":{"TotalBytes":0,"Up":0,"Down":0},"ControllerVersion":"v0.27.9","MfaEnabled":false,"MfaNeeded":false,"MinTimeout":0,"MaxTimeout":0,"MfaLastUpdatedTime":"0001-01-01T00:00:00"}
[2023-06-15T09:42:43.858Z] DEBUG	ZitiDesktopEdge.MainWindow	==== IdentityEvent    : action:added identifer:C:\WINDOWS\system32\config\systemprofile\AppData\Roaming/NetFoundry/KelvinWorkLaptop.json name:HDC Kelvin Work Laptop
[2023-06-15T09:42:43.929Z] DEBUG	ZitiDesktopEdge.MainWindow	IDENTITY EVENT. Action: added identifier: C:\WINDOWS\system32\config\systemprofile\AppData\Roaming/NetFoundry/KelvinWorkLaptop.json
[2023-06-15T09:42:43.929Z] TRACE	ZitiDesktopEdge.ServiceClient.DataClient	{"Op":"controller","Action":"disconnected","Identifier":"C:\\WINDOWS\\system32\\config\\systemprofile\\AppData\\Roaming/NetFoundry/KelvinWorkLaptop.json","Fingerprint":"KelvinWorkLaptop"}
[2023-06-15T09:42:43.929Z] DEBUG	ZitiDesktopEdge.ServiceClient.DataClient	Controller event received
[2023-06-15T09:42:43.929Z] DEBUG	ZitiDesktopEdge.MainWindow	==== ControllerEvent    : action:disconnected identifier:C:\WINDOWS\system32\config\systemprofile\AppData\Roaming/NetFoundry/KelvinWorkLaptop.json
[2023-06-15T09:42:44.445Z] TRACE	ZitiDesktopEdge.ServiceClient.DataClient	{"Op":"identity","Action":"added","Fingerprint":"KelvinWorkLaptop","Id":{"Name":"HDC Kelvin Work Laptop","Identifier":"C:\\WINDOWS\\system32\\config\\systemprofile\\AppData\\Roaming/NetFoundry/KelvinWorkLaptop.json","FingerPrint":"KelvinWorkLaptop","Active":true,"Loaded":true,"Config":{"ztAPI":"https://ziticontroller.horowhenua.govt.nz:443"},"ControllerVersion":"v0.27.9","IdFileStatus":true,"MfaEnabled":false,"MfaNeeded":false,"Metrics":{"Up":0,"Down":0},"MfaMinTimeout":0,"MfaMaxTimeout":0,"MfaMinTimeoutRem":0,"MfaMaxTimeoutRem":0,"MinTimeoutRemInSvcEvent":0,"MaxTimeoutRemInSvcEvent":0,"Deleted":false,"Notified":false}}
[2023-06-15T09:42:44.445Z]  INFO	ZitiDesktopEdge.Models.ZitiIdentity	Identity: HDC Kelvin Work Laptop updated To {"Name":"HDC Kelvin Work Laptop","FingerPrint":"KelvinWorkLaptop","Identifier":"C:\\WINDOWS\\system32\\config\\systemprofile\\AppData\\Roaming/NetFoundry/KelvinWorkLaptop.json","Active":true,"Config":{"ztAPI":"https://ziticontroller.horowhenua.govt.nz:443","id":null,"configTypes":null},"Status":null,"Services":null,"Metrics":{"TotalBytes":0,"Up":0,"Down":0},"ControllerVersion":"v0.27.9","MfaEnabled":false,"MfaNeeded":false,"MinTimeout":0,"MaxTimeout":0,"MfaLastUpdatedTime":"0001-01-01T00:00:00"}
[2023-06-15T09:42:44.445Z] DEBUG	ZitiDesktopEdge.MainWindow	==== IdentityEvent    : action:added identifer:C:\WINDOWS\system32\config\systemprofile\AppData\Roaming/NetFoundry/KelvinWorkLaptop.json name:HDC Kelvin Work Laptop
[2023-06-15T09:42:44.474Z] DEBUG	ZitiDesktopEdge.MainWindow	IDENTITY EVENT. Action: added identifier: C:\WINDOWS\system32\config\systemprofile\AppData\Roaming/NetFoundry/KelvinWorkLaptop.json
[2023-06-15T09:42:44.474Z] TRACE	ZitiDesktopEdge.ServiceClient.DataClient	{"Op":"controller","Action":"connected","Identifier":"C:\\WINDOWS\\system32\\config\\systemprofile\\AppData\\Roaming/NetFoundry/KelvinWorkLaptop.json","Fingerprint":"KelvinWorkLaptop"}
[2023-06-15T09:42:44.474Z] DEBUG	ZitiDesktopEdge.ServiceClient.DataClient	Controller event received
[2023-06-15T09:42:44.474Z] DEBUG	ZitiDesktopEdge.MainWindow	==== ControllerEvent    : action:connected identifier:C:\WINDOWS\system32\config\systemprofile\AppData\Roaming/NetFoundry/KelvinWorkLaptop.json
[2023-06-15T09:42:44.488Z] TRACE	ZitiDesktopEdge.ServiceClient.DataClient	{"Op":"identity","Action":"added","Fingerprint":"KelvinWorkLaptop","Id":{"Name":"HDC Kelvin Work Laptop","Identifier":"C:\\WINDOWS\\system32\\config\\systemprofile\\AppData\\Roaming/NetFoundry/KelvinWorkLaptop.json","FingerPrint":"KelvinWorkLaptop","Active":true,"Loaded":true,"Config":{"ztAPI":"https://ziticontroller.horowhenua.govt.nz:443"},"ControllerVersion":"v0.27.9","IdFileStatus":true,"MfaEnabled":false,"MfaNeeded":false,"Metrics":{"Up":0,"Down":0},"MfaMinTimeout":0,"MfaMaxTimeout":0,"MfaMinTimeoutRem":0,"MfaMaxTimeoutRem":0,"MinTimeoutRemInSvcEvent":0,"MaxTimeoutRemInSvcEvent":0,"Deleted":false,"Notified":false}}
[2023-06-15T09:42:44.488Z]  INFO	ZitiDesktopEdge.Models.ZitiIdentity	Identity: HDC Kelvin Work Laptop updated To {"Name":"HDC Kelvin Work Laptop","FingerPrint":"KelvinWorkLaptop","Identifier":"C:\\WINDOWS\\system32\\config\\systemprofile\\AppData\\Roaming/NetFoundry/KelvinWorkLaptop.json","Active":true,"Config":{"ztAPI":"https://ziticontroller.horowhenua.govt.nz:443","id":null,"configTypes":null},"Status":null,"Services":null,"Metrics":{"TotalBytes":0,"Up":0,"Down":0},"ControllerVersion":"v0.27.9","MfaEnabled":false,"MfaNeeded":false,"MinTimeout":0,"MaxTimeout":0,"MfaLastUpdatedTime":"0001-01-01T00:00:00"}
[2023-06-15T09:42:44.488Z] DEBUG	ZitiDesktopEdge.MainWindow	==== IdentityEvent    : action:added identifer:C:\WINDOWS\system32\config\systemprofile\AppData\Roaming/NetFoundry/KelvinWorkLaptop.json name:HDC Kelvin Work Laptop
[2023-06-15T09:42:44.529Z] DEBUG	ZitiDesktopEdge.MainWindow	IDENTITY EVENT. Action: added identifier: C:\WINDOWS\system32\config\systemprofile\AppData\Roaming/NetFoundry/KelvinWorkLaptop.json
[2023-06-15T09:42:44.529Z] TRACE	ZitiDesktopEdge.ServiceClient.DataClient	{"Op":"controller","Action":"disconnected","Identifier":"C:\\WINDOWS\\system32\\config\\systemprofile\\AppData\\Roaming/NetFoundry/KelvinWorkLaptop.json","Fingerprint":"KelvinWorkLaptop"}
[2023-06-15T09:42:44.529Z] DEBUG	ZitiDesktopEdge.ServiceClient.DataClient	Controller event received

That sure looks like a tight loop bug... Strange. Is the virtual box also running windows perhaps? Can you also look at the service log when this happens? Here, the UI is responding to IPC events from the tunneler running on this machine. I'm wondering if the vm is also running ZDEW or not, and I'm wondering what the service/data portion of ZDEW is doing. It's the one sending these IPC events to the UI. Still, the UI should not be growing it's memory uncontrollably. When this happens I expect you'll see one full cpu used by theziti-edge-tunnel as well.
Thanks for digging in. If you're feeling up to it, I would wonder how the preview/new UI would behave in this situation. https://github.com/openziti/desktop-edge-ui/releases/tag/3.2.3

Thanks.  Yes it was pegging the CPU on a core.  It was the 'KelvinWorkLaptop.json' that was causing the problem.  I noticed that I had an identity with a long name - like GUID which was my original identity when starting to play around with Ziti.  That went in, but had no services, then I found that my certs had expired on my main overlay network.  Anyway, so the issue here was a bad identity.  I have not deep dived further into the bad identity as my own overlay down is more important!


can't connect to  Github Enterprise

I'm recording the issue here, hope it can help anyone who may meet the same issue...
Error Keyword:
"Unable to connect to GitHub API: org.kohsuke.github.HttpException"
Problem Plugin version:

GitHub plugin  v1.27
GitHub Pull Request Builder  v1.37

Error Details
in Jenkins Admin --> Configuration --> GitHub Pull Request Builder

when "Test Connection" it will always fail even the "curl -u user:pwd  https://ent2git.cec.lab.***.com/api/v3/user  " works well in the Jenkins master.
the "Test Connection error message" looks like below( sorry, I lost the failure snapshot and I can't afford to downgrade the plugin again to reproduce the issue (it's our production Jenkins), so I can't recalled every character correctly for below error messages . )
Unable to connect to GitHub API: org.kohsuke.github.HttpException
{"message":"null","xxxxx_url":"https://entgit.cec.lab.***.com/api/v3/user"}

Fix
the issue was gone when I upgrade the both plugs in

GitHub plugin  v1.28
GitHub Pull Request Builder  v1.39
so I don't know which upgrade fixed the problem.

But another Jenkins instance runs below lower plugin version , and issue does not appear there

GitHub plugin   1.26
GitHub Pull Request Builder   1.35

I suspect this problem in git Github Api Plugin. But if it works now I will go ahead and close the issue.


When will release code？

Your work is very good, authors. But will you release your code?

I appreciate your interest in our paper!
We are preparing our code to release in this repository.
There are some delays in releasing our code, but we will release our official code ASAP.


Added LZ4 support for TFRecordWriter

Related to https://github.com/tensorflow/tensorflow/pull/53385
The current API of TFRecordOptions only supports:

no compression
GZIP compression
ZLIB compression

With this PR, I have added support for a 4th compression algorithm, LZ4: https://github.com/lz4/lz4
The API for TFRecordOptions is therefore updated to accept the following parameters:
      compression_type: `"GZIP"`, `"ZLIB"`, `"LZ4"` or `""` (no compression).
      TODO

Maintaining backward compatibility with the current API.

@IAL32 This PR is in draft, any update on this? Please. Thank you!

@IAL32 This PR is in draft, any update on this? Please. Thank you!

Hi @gbaned , still no update on this from my side. As in https://github.com/tensorflow/tensorflow/pull/53385 it might take some time for me to get back to this.

@IAL32 This PR is in draft, any update on this? Please. Thank you!

Closing as it stalled. Please open a new one when coming back to this


Use custom C# DataAnnotation attributes on NSwag DTO Objects based on Data Type format

Is it possible to generate custom DataAnnotations on DTO object based on Data Type format?
"Country": {
"type": "string",
"format": "country",
"description": "Country"
}
It seems JsonSchemaProperty.ActualSchema.Format could be used in Class.Property.Annotations.liquid, but JsonSchemaProperty is a private field of PropertyModel.

Added the property to the model so you can use it in the custom template.

Added the property to the model so you can use it in the custom template.


pull request for e2e test at 12/24/2016 14:45:07

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


fix: Addition of default pad token in tokenizer when EOS and PAD token are equal

Description of the change
For any model when tokenizer has PAD token equal to EOS token, then the DataCollatorForCompletionOnlyLM masks the EOS token from labels of dataset batch which leads to repetitive predicted_target during inference time thus decreasing f1-micro for models like granite-34b-gptq
Related issue number
#1309
How to verify the PR
Running HF inference after QLoRA tuning of granite-34b-gptq model.
Was the PR tested

[ ] I have added >=1 unit test(s) for every new method I have added.
[x] I have ensured all unit tests pass

Thank you @Abhishek-TAMU . Lets wait for another review by @kmehant since he was closely involved with the tokens update.

Pushed the suggested changes. Thank you!
@kmehant @Ssukriti


Provide a way to conditionally include a field of an input object variable

Right now there's no way to specify a composite GraphQL variable that includes a field that might either be included or omitted conditionally...but there really should be!
The GraphQL.Request.Builder.Variable module currently exposes the nullable function that can be used to specify that a variable or some field within a variable might be sent as an explicit null. However, sending explicit null values is not a very common requirement – it is more common for nullable input object fields to simply be omitted from the JSON variables that accompany the GraphQL document in the request.
(Sometimes explicit null values are treated the same as an omission, sometimes they are treated differently, and unfortunately GraphQL schemas do not define this aspect of the server's behaviour.)
It wouldn't quite work to create another function like nullable for this purpose, because nullable operates on the level of the description of a lone value as opposed to a whole field. Instead we can introduce a new optionalField function that works much like the existing field function, with the difference that it would require the extractor function passed to it to return Maybe values:
optionalField
    :  String
    -> (objVariableSource -> Maybe fieldVariableSource)
    -> VariableSpec nullability fieldVariableSource
    -> Field objVariableSource

Here's an example usage:
type alias UpdateUserInputData =
    { id : String
    , name : Maybe String
    , email : Maybe String
    }

userDataVar : Variable { vars | userData : UpdateUserInputData }
userDataVar =
    required "userData"
        .userData
        (object "UpdateUserInput"
            [ field "id" .id id
            , optionalField "name" .name string
            , optionalField "email" .email string
            ]
        )

When either the name or email field is Nothing in the supplied UpdateUserInputData value, the corresponding field of the JSON variables would be omitted from the request.

How would this work with nullable fields? It should be possible to pass null or omit the field completely.

@akheron this approach doesn’t inhibit explicit nulls in any way, if you want to be able to pass them then you can use the nullable function as usual:
optionalNullableEmail : Field { a | email : Maybe (Maybe String) }
optionalNullableEmail =
    optionalField "email" .email (nullable string)

The email field is now double-Maybe-wrapped, where Nothing means omitting the field and (Just Nothing) sends an explicit null.

Implemented optionalField in e3d0b8c2a1bda0a22a97db0490f89e4db155d2be, published in 1.7.0.


Purge expired jobs

Just adding this in as a reminder:
'Disabled scheduled jobs for now. Will revisit purging expired jobs when we implement the scheduler for automated periodic exports.' - Brandon Liu

Make sure that HDX downloads shouldn't be deleted. (Denormalize jobs table to include flag re: whether it's an HDX job?)
Reconsider renaming download / staging directories re: whether they should be based on the job or the run.

eab52279d33418eec01256688fc6d7985c526828 disabled them.

Revisiting this, the previous implementation was purging private ("unpublished") exports after 48 hours. There's definitely convenience in keeping them around since individuals may be sharing them with broader groups even if the export itself isn't public.
I did some cleanup in the process (ensuring that the staging directory gets cleared up when an export finishes), but I'm going to leave private jobs available indefinitely.


[RFC] setPanelBehavior

How do you envision this feature/change to look/work like?
Currently, running chrome.sidePanel.setPanelBehavior({ openPanelOnActionClick: true }) in background.ts is the only way to toggle a side panel when the user clicks the browser extension in Chrome.  This isn't very browser-independent.  Can we create a generic option that enables/disables this behavior across all browsers (e.g. safari, firefox, chrome, etc ...)?
What is the purpose of this change/feature? Why?
See above.
(OPTIONAL) Example implementations
No response
(OPTIONAL) Contribution

[ ] I would like to contribute to this RFC via a PR

Verify canary release

[X] I verified that the issue exists in plasmo canary release

Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I checked the current issues for duplicate problems.

this would be awesome

Yes, this would be so good!

Anyone have find a nice workaround for when chrome 123 > removed the side panel button?

@tianhuil did you ever get the side panel working for a non-chrome browser?

Hey guys; question: when I do chrome.sidePanel.setPanelBehavior({ openPanelOnActionClick: true }), I get cannot read property ... of undefined (reading setPanelBehavior).
Is there any solution/workaround for that?

I'm running into the same issue right now. Looks like sidePanel doesn't exist on my chrome object for some reason, even though I've set permissions: [ "sidePanel" ] in my manifest.json. Did you ever figure this out?

Did you ever figure this out?

It's been a while, but I quickly gave up on Plasmo since, IMO, it's trying to do too much abstraction, which ... concrete example here ... doesn't seem to work!


Issue/1423/only warn both unset

Summary:
related issue (https://github.com/MobilityData/gtfs-validator/issues/1423)
Add a NoticeFilter interface that can be implemented to filter validation notices.
Add a filterNotice method in NoticeContainer to filter notices that are produced after validation.
Expected behavior:
Only feed_contact_email and feed_contact_url are all unset will warn missing_recommended_field for feed_info.txt.
Please make sure these boxes are checked before submitting your pull request - thanks!

[√ ] Run the unit tests with gradle test to make sure you didn't break anything
[√ ] Format the title like "feat: [new feature short description]". Title must follow the Conventional Commit Specification(https://www.conventionalcommits.org/en/v1.0.0/).
[ √] Linked all relevant issues
[ ] Include screenshot(s) showing how this pull request works and fixes the issue(s)

 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.

Hi @l646505418, thanks for your contribution. Reading #1423 and the best practices, re-using the MissingRecommendedFieldNotice is not clear enough in this case.  The MissingRecommendedFieldNotice only covers one field within a file. I suggest creating a new notice to cover multiple fields(or just two). To accomplish this, we can create a validator's class inheriting from SingleEntityValidator interface to trigger the notice when both fields are missing. This can give the consumer more insight into the issue and how to fix it. An example of a validator implementation that looks for two fields within an entity can be found here,  RouteColorContrastValidator

Hello, thanks for your contribution. After discussing with David, we decided to implement a FeedContactValidator class which has a static class member MissingFeedContactEmailAndUrlNotice. Please see details in this PR:
https://github.com/MobilityData/gtfs-validator/pull/1514


Corrected license classifer

LICENSE file states BSD-licensed code, but setup.py contained a value for classifier that would have people find the package in PyPi under AGPLv3+. (The two license types are incompatible. It's presumed that the author intended BSD license, but sending in a PR to verify.)

Indeed. Thank you!


chore(log): make log api separate from player

 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.

 All committers have signed the CLA.


Add additional test coverage to messaging extesnions

Looks like I need to fix up the ActiveMQ itest. Odd because I'm sure it was passing locally 🤔


Feedback from Tomasz

Hi @ThomasKronhol
This is a very good submission! Thank you! It's very informative! WOW!
I would just ask. you to

[ ]  include ADF test results for diff to determine the integration order of all variables as in the example in ba4d69ffd3774ca52d4526ab5ffc5bf6151cb741

Good luck!

Hi @ThomasKronhol
Hey, it's OK to reject the pull request. Would you please comment on what you have done with the proposed changes in a note in this PR? Thanks.

Hi @donotdespair
Thanks for your feedback!
I have implemented all suggested changes, but I was worried that it would change my newest file, and chose therefore not to merge!
Thanks
Thomas


elmRepl and elmReactor fix

Reactor can be stopped without exiting sbt
fixes #21

care to create a release?


2022 Week 1

Open Template Hub - Feature Request
Description
Code Cleanup and fix TODOs.
Supplementary Info
Links, images, screenshots etc.
Open Template Hub © 2021

Closed.