GestureDetector fails to run on_enter and on_exit events because of KeyError exception in HoverEvent

Duplicate Check

[X] I have searched the opened issues and there are no duplicates

Describe the bug
When I create a GestureDetector and attach to the on_enter and on_exit functions it throws a KeyError exception when trying to create the HoverEvent to pass to the functions.
Code sample
Code
import flet as ft

def main(page: ft.Page):
    page.window.width = 100
    page.window.height = 100
    page.padding = ft.padding.all(0)
    page.controls.append(
        ft.Container(
            ft.GestureDetector(
                on_enter=lambda _: print("Entered"),
                on_exit=lambda _: print("Exited"),
                width=100,
                height=100,
            ),
            width=100,
            height=100,
            bgcolor=ft.colors.BLUE,
        )
    )
    page.update()

ft.app(main)

To reproduce
Run the example code and move the mouse cursor in and out of the window to see exceptions being thrown.
Expected behavior
It should just be running the lambdas that print the simple "Entered" and "Exited" messages.
Screenshots / Videos

Captures

Operating System
macOS
Operating system details
macOS 15.1
Flet version
0.23.2
Regression
I'm not sure / I don't know
Suggestions
On lines 818 and 819 of flet_core/gesture_detector.py it is currently an unsafe dictionary get for optional fields
        self.delta_x: Optional[float] = d["dx"]
        self.delta_y: Optional[float] = d["dy"]

This could be updated to be a safe get like so:
        self.delta_x: Optional[float] = d.get("dx", None)
        self.delta_y: Optional[float] = d.get("dy", None)

If I update my local library file with that safe get then the events work again. But maybe there are reasons unknown to me that it wouldn't work to do that?
Logs
Logs
Task exception was never retrieved
future: <Task finished name='Task-13' coro=<__run_socket_server.<locals>.on_event() done, defined at /Users/zhynem/Documents/python/amoria/venv/lib/python3.12/site-packages/flet_runtime/app.py:217> exception=KeyError('dx')>
Traceback (most recent call last):
  File "/Users/zhynem/Documents/python/amoria/venv/lib/python3.12/site-packages/flet_runtime/app.py", line 219, in on_event
    await conn.sessions[e.sessionID].on_event_async(
  File "/Users/zhynem/Documents/python/amoria/venv/lib/python3.12/site-packages/flet_core/page.py", line 905, in on_event_async
    await handler(ce)
  File "/Users/zhynem/Documents/python/amoria/venv/lib/python3.12/site-packages/flet_core/event_handler.py", line 18, in fn
    ce = self.__result_converter(e)
         ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/zhynem/Documents/python/amoria/venv/lib/python3.12/site-packages/flet_core/gesture_detector.py", line 275, in <lambda>
    self.__on_enter = EventHandler(lambda e: HoverEvent(e))
                                             ^^^^^^^^^^^^^
  File "/Users/zhynem/Documents/python/amoria/venv/lib/python3.12/site-packages/flet_core/gesture_detector.py", line 818, in __init__
    self.delta_x: Optional[float] = d["dx"]
                                    ~^^^^^^
KeyError: 'dx'
Task exception was never retrieved
future: <Task finished name='Task-14' coro=<__run_socket_server.<locals>.on_event() done, defined at /Users/zhynem/Documents/python/amoria/venv/lib/python3.12/site-packages/flet_runtime/app.py:217> exception=KeyError('dx')>
Traceback (most recent call last):
  File "/Users/zhynem/Documents/python/amoria/venv/lib/python3.12/site-packages/flet_runtime/app.py", line 219, in on_event
    await conn.sessions[e.sessionID].on_event_async(
  File "/Users/zhynem/Documents/python/amoria/venv/lib/python3.12/site-packages/flet_core/page.py", line 905, in on_event_async
    await handler(ce)
  File "/Users/zhynem/Documents/python/amoria/venv/lib/python3.12/site-packages/flet_core/event_handler.py", line 18, in fn
    ce = self.__result_converter(e)
         ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/zhynem/Documents/python/amoria/venv/lib/python3.12/site-packages/flet_core/gesture_detector.py", line 277, in <lambda>
    self.__on_exit = EventHandler(lambda e: HoverEvent(e))
                                            ^^^^^^^^^^^^^
  File "/Users/zhynem/Documents/python/amoria/venv/lib/python3.12/site-packages/flet_core/gesture_detector.py", line 818, in __init__
    self.delta_x: Optional[float] = d["dx"]
                                    ~^^^^^^
KeyError: 'dx'

Additional details
No response

This issue has been fixed.
You need the latest prerelease: pip install flet -U --pre

This issue has been fixed.
You need the latest prerelease: pip install flet -U --pre

Is it stable??

Give it a try, I will say.

Installing the pre-release version does fix this particular issue but introduces other new problems to the application I was working on, so your milage may vary?

Feel free to report new issues, so they can be addressed accordingly :)


Add my orgs page and versions list in image page

Purpose

Add my orgs page and versions list in image page

Goals

Add my orgs page and versions list in image page

Approach

Add my orgs page and versions list in image page

User stories

N/A

Release note

N/A

Documentation

N/A

Training

N/A

Certification

N/A

Marketing

N/A

Automation tests

Unit tests

N/A

Integration tests

N/A

Security checks

Followed secure coding standards in http://wso2.com/technical-reports/wso2-secure-engineering-guidelines? yes
Ran FindSecurityBugs plugin and verified report? yes
Confirmed that this PR doesn't commit any keys, passwords, tokens, usernames, or other secrets? yes

Samples

N/A

Related PRs

N/A

Migrations (if applicable)

N/A

Test environment

N/A

Learning

N/A

Codecov Report

Merging #62 into master will decrease coverage by 1.47%.
The diff coverage is 1.78%.

@@            Coverage Diff             @@
##           master      #62      +/-   ##
==========================================
- Coverage   21.86%   20.39%   -1.48%
==========================================
  Files          45       46       +1
  Lines        1230     1324      +94
  Branches      205      229      +24
==========================================
+ Hits          269      270       +1
- Misses        870      952      +82
- Partials       91      102      +11

Impacted Files
Coverage Δ

...onents/portal/src/components/image/ImageVersion.js
3.07% <ø> (ø)
:arrow_up:

components/portal/src/utils/constants.js
100% <ø> (ø)
:arrow_up:

components/portal/src/components/Home.js
18.18% <ø> (+10.48%)
:arrow_up:

components/portal/src/components/explore/Orgs.js
4.16% <0%> (-0.28%)
:arrow_down:

...ponents/portal/src/components/image/VersionList.js
2.73% <0%> (-6.94%)
:arrow_down:

...ts/portal/src/components/sdk/FederatedIdpSelect.js
17.64% <0%> (-3.79%)
:arrow_down:

components/portal/src/components/user/MyOrgs.js
3.07% <0%> (-13.59%)
:arrow_down:

...mponents/portal/src/components/appLayout/NavBar.js
4.16% <4.16%> (ø)

...mponents/portal/src/components/appLayout/Header.js
75% <50%> (+69.73%)
: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 7e3c169...4478ab1. Read the comment docs.


What would happen if I generate more than 12 keys at once?

Hi! I'm working on a bot that generates activation keys for my accounts. I need to generate 35+ keys every 30 minutes for all my accounts. Will that work, or is there a specific daily limit per IP? Also, why is the maximum set to exactly 12?

The number 12 is actually chosen randomly, it is not the maximum possible. The API used to generate keys actually has limitations on the number of keys generated, but this value, according to my observations, is not static and depends on the nature of key generation. In general, I can say this: generating more than 20 keys at a time can lead to an error, after which you will have to wait about 5-10 minutes. If you will generate several keys with small time intervals, then most likely there will be no problems. In this case, you will have to experiment and look for a way to achieve maximum performance and not fall within the limits.

I actually tested this and the maximum amount per ip is 24 ( two sets of 12 ) and its pretty reasonable.

perhaps the developers changed these limits a little, because when I checked it, the second generation of 12 was no longer working (There is even an issue somewhere below about this). therefore, I recommend that you check for yourself how many codes you can generate without problems in how long.

When generating keys for which game does this error occur? It is usually associated with a too small `eventsDelay' value. I remember that in some games there was this error, but it did not lead to a generation error. Although in this case, increasing the delay for this game may even speed up the generation.


Connection refused when fetching status for root key of local replica

I'm encountering this error when calling fetchRootKey() of HttpAgent in _initAgent():
E/flutter (23155): [ERROR:flutter/lib/ui/ui_dart_state.cc(199)] Unhandled Exception: http request failed because SocketException: OS Error: Connection refused, errno = 111, address = localhost, port = 46950
E/flutter (23155): #0      HttpAgent._defaultFetch (package:agent_dart/agent/agent/http/index.dart:433:7)
E/flutter (23155):
E/flutter (23155): #1      HttpAgent.status (package:agent_dart/agent/agent/http/index.dart:362:20)
E/flutter (23155):
E/flutter (23155): #2      HttpAgent.fetchRootKey (package:agent_dart/agent/agent/http/index.dart:246:20)
E/flutter (23155):
E/flutter (23155): #3      AgentFactory._initAgent (package:agent_dart_example/init.dart:63:7)
E/flutter (23155):
E/flutter (23155):
This means that the port somehow gets changed since I specified port 40161 in the url of the AgentFactory (this is where my local replica runs).
I also tried to directly set the port with the host parameter of HttpAgentOptions but still get the same error.
Am I missing something?

https://github.com/AstroxNetwork/agent_dart/blob/ca5f341b14b0527a0cfa01a5366ff6a6a245f4c3/example/lib/main.dart#L40
Just do it here, like  http://localhost:40161
or can you post your code here to see what's going on there

Yes, that's how I did it
I also tried to set the host directly when initializing HttpAgentOptions at
https://github.com/AstroxNetwork/agent_dart/blob/ca5f341b14b0527a0cfa01a5366ff6a6a245f4c3/example/lib/init.dart#L60 :
like options: HttpAgentOptions()..identity=_identity..host='localhost:40161',
but still got the same error..
As far as my setup is concerned, your scripts to build the binaries and link them worked without errors and I'm using an Android Pixel 2 API 29 emulator. (also had to change to $ANDROID_NDK_HOME in variables.sh)
Btw, I found a typo here:
https://github.com/AstroxNetwork/agent_dart/blob/ca5f341b14b0527a0cfa01a5366ff6a6a245f4c3/lib/agent/agent/http/index.dart#L101
This should probably be defaultPort

Yes, that's how I did it
I also tried to set the host directly when initializing HttpAgentOptions at
https://github.com/AstroxNetwork/agent_dart/blob/ca5f341b14b0527a0cfa01a5366ff6a6a245f4c3/example/lib/init.dart#L60
:
like options: HttpAgentOptions()..identity=_identity..host='localhost:40161',
but still got the same error..
As far as my setup is concerned, your scripts to build the binaries and link them worked without errors and I'm using an Android Pixel 2 API 29 emulator. (also had to change to $ANDROID_NDK_HOME in variables.sh)
Btw, I found a typo here:
https://github.com/AstroxNetwork/agent_dart/blob/ca5f341b14b0527a0cfa01a5366ff6a6a245f4c3/lib/agent/agent/http/index.dart#L101
This should probably be defaultPort

First, thanks for pointing out the typo. will be fixed in next release.
Second, if you use Android emulator , the emulator connects localhost will use "<IP_ADDRESS>" instead of "localhost" or "<IP_ADDRESS>".
In your case, you should try "http://<IP_ADDRESS>:40161" just in the url field

Thanks that was it!
Closing this issue.


All Cams not arming home/away

Hello
I don’t know why but arming home/away from my automations has stopped working for all my cameras and home base.
I have homebase2, homebase3, and a lot of indoor and outdoor cameras, and they have all stopped arming/disarming.
A few weeks ago I opened a similar issue for homebase3, but it was solved. It worked for a few weeks perfectly but now the problem is back and not only on homebase3 but on all my cameras!!
What is going on guys?
Thank u

Follow bug template

What??Inviato da iPhoneIl giorno 14 gen 2023, alle ore 14:56, Fuat Akgün @.***> ha scritto:
Follow bug template
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>


Volatile DB: Extend q-s-m tests to test reopening

When db closes, the Internal State remains completely empty, so reOpen needs to parse all blocks from scratch and create in-memory indexes. pr https://github.com/input-output-hk/ouroboros-network/pull/163 introduced this functionality already.  Also q-s-m tests already use reOpening.

So we can close this?

I think yes

Ok, added a comment to #273 that we shouldn't forget about the close-corrupt-open case.


dmc humanoid_CMU

#103

Can you create another align/deterministic python test for non suite.BENCHMARK envs? That would be more efficient for testing.

Can you create another align/deterministic python test for non suite.BENCHMARK envs? That would be more efficient for testing.

It turns out atari pretrain test taking the longest time to pass: 586s.


ROX-15024: Workload CVEs Overview - Default filters modal

The button at the top right corner of the page should be a button
The button badge should reflect the number of filters set globally
Clicking on the button should open the Default filters modal
The modal should look like the above and populate as empty at first
Clicking the checkboxes and cancelling should not update local storage
Clicking the checkboxes and hitting the 'Apply filters' button should save the filters to local storage
Opening the modal on refresh should load the filters saved from local storage

Implementation notes:

I saved the empty state to be preferences: {       defaultFilters: {           Severity: [],           Fixable: [],       },   }, because those fields will always exist (vs possibly being empty) and thus it didn't make much sense to omit them in the empty state and end up adding conditionals in the code

Also, it looks like the e2e test failures are false positives but the style check is due to not having the component props typed.


History virtualize scroll

Bug fixes:
Performance:

Compact mode is now much faster to load and more responsive when resizing the split view or browser window
Grid mode is now much faster to load and more responsive when resizing the split view, browser window, or switching to Summary / Full details

Grid header does not cover items in summary view or full details view
Compact mode now scrolls to the item selected in the graph
Fixed header from increasing height on smaller browser windows
Switching between Compact and Grid will now correctly show the selected event in the table and graph (if selected)
Improvements:
Grid columns are now a fixed size to allow more space for the summary column
Improved grid readability at smaller sizes
Allow split view to completely collapse one panel to maximize view of the other panel. Also useful for viewing the JSON tab.
When refreshing the page (or opening a new window) and an event is selected, it will scroll to the event in either compact or grid mode once the list has loaded.
Notes:
Page mode is now disabled and uses split view by default because vue-virtual-scroller needs a height for the content container otherwise it cannot function correctly.
Removed table and replaced with class style equivalents because vue-virtual-scroller does not support HTML tables.
There is now a warning log displayed when starting the page because there is a bug in vue-virtual-scroller when trying to call scrollToItem when the view has just been rendered. Unless I raise a PR to fix their code, I am not able to work around it as it is internal state to vue-virtual-scroller.
Added CSS loader to Webpack as it is a requirement for using vue-virtual-scroller library.
Removed recalcThWidths as I don’t believe this is required.
General code refactoring to match Vue best standard practices.

 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.


Manually set types for runtimeConfig are overwritten by the inferred types in nuxt.config

Environment

Operating System: Darwin
Node Version:     v18.12.1
Nuxt Version:     3.2.3
Nitro Version:    2.3.3
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:      ssr, app, modules, typescript, runtimeConfig, imports, vite, i18n, tailwindcss
Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Build Modules:    -

Reproduction
Set some types manually in index.d.ts file:
declare module 'nuxt/schema' {
    interface PublicRuntimeConfig {
        APP_ENV: 'development' | 'staging' | 'production'
    }
}

export {}

Then in nuxt.config set this same variable to another type:
runtimeConfig: {
    public: {
        APP_ENV: 123, // will infer `number` type and overwrite the manual types
    },
}

Using useRuntimeConfig().public.APP_ENV will now be typed as a number and there is no type error reported in nuxt.config.
Describe the bug
I'd like to define union type for an env variable in a env.d.ts file. If I then also reference this variable in nuxt.config.ts under runtimeConfig, that union type is lost and overwritten by the auto-inferred type.
Additional context
No response
Logs
No response

Issue still exists

Same problem here

Had the same error today. Could circumvent it by now by doing something like this in nuxt.config.ts
hooks: {
    "prepare:types": ({ tsConfig }) => {
      tsConfig.include?.unshift("../types/index.d.ts");
    },
  },

This prepends ../types/index.d.ts in the tsconfig and forces the custom config to be loaded first.

@PatrickBauer Thanks, your solution worked.
Just to add to this for anyone also struggling—I had to declare module "@nuxt/schema" (note the leading @).
Without that nuxt-vuefire kept overriding the declaration as there were two conflicting versions (nuxt/schema and @nuxt/schema) and only the former actually read by TS.


Fixed storage queue race condition

Is there a chance to add a UnitTest for that to make sure it will not break again in the future?

@chkr1011 That's a good idea.  I'll look into it next week.

@chkr1011 Unit testing this issue is an interesting problem.  It's not something that can reliably be reproduced, so no test can conclusively prove there isn't a race condition in the code.  The best we can do is a kind of heuristic approach that publishes for a few minutes and monitors the storage queue to ensure it goes down to zero (or something along these lines).  Is this good enough for you?

For me it would be fine to check the overall flow (roughly points).

Setup everyting.
Connect with server.
STOP server when client is connected.
Publish message.
Check for existence in storage after 500 ms.
Start server.
Wait for connect.
Check if the message is properly removed from the storage.

This can be done using the TestEnvironment.

Thank you for your contribution. Good work 👍


Static analysis tools in CI chain

isn't this covered by SonarCube already?

@JensFastrup No, static analysis tools are something that checks if things af compileable for example (see task 1 here https://github.com/itu-devops/lecture_notes/blob/master/sessions/session_07/README_TASKS.md)

Static analysis tools, that check

Code quality
Maintainability
Security
Technical debt
etc.
has been integrated with tools in the CI with Sonarcloud, Code Climate, Snyk


Implement Add method for header

Signed-off-by: RRobot-lm<EMAIL_ADDRESS>

@RRobot-lm Nice job! Thanks for your contribution!

@RRobot-lm Could you please create a new PR to the official envoyproxy repo, since the Go filter extension has been merged to envoyproxy repo, which doesn't include this PR. Thanks again.


Make Trail Sense a share target for images / PDFs

This would open the map import screen.

I could do that, is that ok?

Yes, but be aware that this needs to respect the experimental feature setting for maps (i.e. no share target while the experimental setting is off) - if that's possible, then it should be alright to implement this.
And note, I don't want TS to have the write/read external files permission - so if the images can't be imported this way without it then this issue can be closed


Code for #46: Clean up the large-display layout

Changing the layout of the flowers, turning the TimerControlComponent into more of a bottom-bar, and making sure that it works decently well in different screen resolutions.
Closes #46.

Oops, that's supposed to be a draft!

I don't think I'm going to add the three-dot-menu yet. We still don't know what sort of controls we would put in there, so I'm going to leave it off until we have a concrete need for it.


polar plot missing hemisphere

There have been some complaints about polar plots (plot set 7) not working for one pole or another.  They work for me!  This may be a problem only for certain versions of UV-CDAT, or maybe a problem with it being too easy to mis-type one of the keywords needed for this.  Or maybe there's no issue here at all any more.

Jerry Potter reported that the problem is that an initial space is required, e.g. in " Southern Hemisphere".  This space should be permitted but not required.


perform correct type assertion for response from GetCandidate

Fixes #327

Codecov Report

Merging #328 into master will decrease coverage by 0.00%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##           master     #328      +/-   ##
==========================================
- Coverage   54.21%   54.20%   -0.01%
==========================================
  Files         127      127
  Lines        7161     7164       +3
==========================================
+ Hits         3882     3883       +1
- Misses       2773     2774       +1
- Partials      506      507       +1

Impacted Files
Coverage Δ

pkg/p2p/peer/responding/candidatebroker.go
71.42% <50.00%> (-10.39%)
:arrow_down:

Continue to review full report at Codecov.

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


Didn't add any delay but there is delay in job execution

I am using robfig V2 on golang v1.10. I have a job that is executed every 5 mins. I have around 2k jobs to be started at once. Now when it starts, I did not add any custom delay but second execution for some job (that should be done after 5-10 mins approx) occurred after 30 mins which is a big difference. Can you help me over this ?

Sorry, I'm having trouble exactly understanding the scenario. Can you provide code for me to replicate what you're doing?

i hava the same question for this issue.When i deploy my cron service in docker, after i reboot the server, there is a time interval before the first cron job. I had tried to fix it by change the time zone, but didn't  make it.
if anything could help me to slove the problem, please tell me, thanks a lot :)
Env:

Raspberry Pi 4B 4 CPU core 4GB RAM
Linux raspbian 5.4.47-OPENFANS+20200622-v8 #1 SMP PREEMPT Mon Jun 22 21:17:56 CST 2020 aarch64 GNU/Linux

CornVersion:

github.com/robfig/cron/v3 v3.0.0

BuilderImg:

golang:1.15.1-alpine3.12

DeploymentImg:

alpine:3.12.0

Code:
import (
	......
	_ "time/tzdata"
)

func JobTimer() {
	c := cron.New()

	// when the cron service start, log the container time and timezone info
	Logger.Info("[定时器]", zap.Time("运行时间", time.Now()), zap.String("时区", c.Location().String()))

	// add a cron job, the CommitTime is a const var -> "00 7,12 * * *"
	eid, err := c.AddFunc(CommitTime, func() {
		// my custom job, i make sure nothing will block the goroutine
		go Job()
		...

		// and after the job finished, print when the next job will do
		for _, entry := range c.Entries() {
			Logger.Info("[GHC定时器]", zap.Time("下一次执行时间", entry.Schedule.Next(time.Now())))
		}
	})
	if err != nil {
		Logger.Error("[定时任务错误]", zap.Error(err))
		return
	}

	// this func will print the next job will do also
	Logger.Info("[GHC定时器]", zap.Time("下一次执行时间", c.Entry(eid).Schedule.Next(time.Now())))

	c.Run()
}

Log:
// my server will reboot at a random time before 7:00
// the docekr will start the container after reboot
// log when the next job will be do
{"L":"INFO","T":"2020-10-07T05:17:22.376+0800","C":"ghc/main.go:262","M":"[GHC定时器]","下一次执行时间":"2020-10-07T07:00:00.000+0800"}

// in fact, the next job will be do after about 45 min, and i dont know why
{"L":"INFO","T":"2020-10-07T07:45:23.196+0800","C":"ghc/main.go:332",....}

// the second job will do at 12:00
{"L":"INFO","T":"2020-10-07T07:45:23.197+0800","C":"ghc/main.go:255","M":"[GHC定时器]","下一次执行时间":"2020-10-07T12:00:00.000+0800"}

// but there is no delay before the second job
{"L":"INFO","T":"2020-10-07T12:00:00.000+0800","C":"ghc/main.go:332",....}

maybe the problem caused by reboot?


Fix documentation table misalignment

Thanks for the fix.
If it isn't to much work, can you add the url to the documentation to a pr? That made it more easy to see the situation. https://phpmd.org/rules/design.html


Pass rotation along position and guid when delegating custom Spawn and Unspawn

Hello,
Following the explanation in this link:
https://mirror-networking.com/docs/Guides/GameObjects/SpawnObjectCustom.html
I'd like to have the ability to pass extra parameters (Like rotation, or a netID that belongs to the owner) in spawner and unspawner when registering the delegation.
(I hope my English is understandable)

Fixed by #1215


Eslintrc

[x] I already read and understand the contribution guide

:tada: This PR is included in version @latipun7/commitlintrc-v1.1.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version @latipun7/releaserc-v2.1.3 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Asynchronous nesting

Hey there,
How would I achieve passing data through to a nested tag when the data is collected asynchronously?
An example might better explain my question:
<account>
  <subscription  data={ data } />

</account>

<subscription>

{ opts.data.name }

</subscription>

The above example renders the subscription tag first and never updates the subscription's opts.data property. So far the only way I can see to achieve this is:
<acount>

</account>

<subscription>

{ opts.data.name }

</subscription>

:confused: I'm hoping there's a better way.

My bad... it does work. As you were.


[FLINK-7689] [Streaming Connectors] Added metrics to JDBCOutputFormat in order to be able to m…

Added metrics to JDBCOutputFormat in order to be able to measure JDBC batch flush rate, latency and size.
What is the purpose of the change
This pull request adds some useful metrics to the flink-jdbc output format.
Brief change log

Added metrics to JDBCOutputFormat in order to be able to measure jdbc batch flush rate, latency and size.
Fixed failing existing tests.
Added tests for metrics setup.

Verifying this change
This change added tests and can be verified as follows:

Run the JDBCOutputFormatTest suite.
Write a flink app that depends on flink-jdbc from this pr and flink-table and use it to write some records in a JDBC compliant database. Open the Flink UI and see the stats in the Task Metrics.

Does this pull request potentially affect one of the following parts:

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

Documentation

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

Thanks for reviewing Fabian. I have added the suggested changes. Please have a look. The build is failing for a non-related issue.

@fhueske @asicoe is this PR still current / ongoing? I'm willing to driving it to the end if there's anything left to do.. : ) - Or perhaps it's almost ready to merge?

@zentol @pabloem @fhueske I've addressed all the latest comments, simplified the metrics (got rid of histograms) and updated tests. Please have a look!

Hi @asicoe,
Sorry for the lack of activity in the PR. Let me know if you are still interested in contributing those metrics. I can help review this PR.
If not I will close this PR after some time.

I'm closing this as "Abandoned", since there is no more activity and the code base has moved on quite a bit. Please re-open this if you feel otherwise and work should continue.


chore: Update config merge_commit_template

Updated the config.yaml to add the merge_commit_template.
This is a test to check if the template works.

/lgtm
/hold
feel free to unhold when you think is ready to test the commit merge

/unhold

/lgtm


Password validation problem using Redeem Invitation page

Hello everyone,
I have set up portal locally and connect it to test crm system. In this CRM some rules for password validation are set:

I have tried to register new user on  portal using redeem invitation code.  After inserting code to form for registration, system redirects me to Login Registration form:

So far so good. This form should validate my password on submit. Portal returns  an error when I don't insert password or username, but does not return an error if password is not OK (complexity etc).. If I insert password 123 and confirm it, portal just redirects me to home page and does not show wrong password error. Username is not inserted to contact in  CRM. Am I doing something wrong or is it a bug?
Thank you for your help. Best regards Matej

I am also having this issue, although upon submitting the page just refreshes. Did you solve this issue?

I have the same issue, haven't been able to resolve it.

I did manage to overcome my issue. The way i got around this was commenting out an if statement in the RegistrationManager.cs within the MasterPortal/Areas/Account/Models path. The if statement is on line 207 and looks like this
if (!result.Succeeded) { var urlHelper = this.GetUrlHelper(); this.loginManager.AddErrors(result); this.loginManager.HttpContext.Response.Redirect(urlHelper.Action("RedeemInvitation", "Login", new { InvitationCode = invitationCode })); }
When i was stepping through the code, it looked like this would cause the page to refresh which meant the error message would not show.

Fixed by PR #93.


cmd/go: go get cannot create zip

What version of Go are you using (go version)?

$ go version
1.19

Does this issue reproduce with the latest release?
Yes.
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="arm64"
GOBIN=""
GOCACHE="/root/.cache/go-build"
GOENV="/root/.config/go/env"
GOEXE=""
GOEXPERIMENT=""
GOFLAGS=""
GOHOSTARCH="arm64"
GOHOSTOS="linux"
GOINSECURE=""
GOMODCACHE="/root/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/root/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/nix/store/wc4py67prnfb5izkvwjpn4v53vzi1amk-go-1.19/share/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/nix/store/wc4py67prnfb5izkvwjpn4v53vzi1amk-go-1.19/share/go/pkg/tool/linux_arm64"
GOVCS=""
GOVERSION="go1.19"
GCCGO="gccgo"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/dev/null"
GOWORK=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -pthread -fno-caret-diagnostics -Qunused-arguments -Wl,--no-gc-sections -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build3183675079=/tmp/go-build -gno-record-gcc-switches"

What did you do?
go get -d github.com/sourcegraph/sourcegraph@f659e61

What did you expect to see?
A successful download.
What did you see instead?
root@b64f345f7401:/# go get -d github.com/sourcegraph/sourcegraph@f659e61
go: downloading github.com/sourcegraph/sourcegraph v0.0.0-20220805202456-f659e618d762
go: github.com/sourcegraph/sourcegraph@f659e61: create zip: internal/search/query/testdata/TestParseStandard/patterns_are_literal_and_slash-delimited_patterns_/.../_are_regexp.golden: malformed file path "internal/search/query/testdata/TestParseStandard/patterns_are_literal_and_slash-delimited_patterns_/.../_are_regexp.golden": invalid path element "..."

The problem file: https://github.com/sourcegraph/sourcegraph/tree/6cf24568ac2c070f96ab9fed5470a2d26f088315/internal/search/query/testdata/TestParseStandard/patterns_are_literal_and_slash-delimited_patterns_/...
Although weird, I don't see anything "wrong" with this. A directory named "..." is fine with Linux and fine with Go's archive/tar package.

Sort of related? #45549

cc @bcmills @matloob

Duplicate of #45549


Seg fault

Occasionally the node crashes with a segmentation fault:
SIGSEGV: segmentation violation
PC=0x1fe20 m=13 sigcode=1
signal arrived during cgo execution

goroutine 1170 [syscall]:
runtime.cgocall(0x1061500, 0xc0066550d8)
	/usr/local/go/src/runtime/cgocall.go:157 +0x4b fp=0xc0066550b0 sp=0xc006655078 pc=0x409b2b
github.com/project-illium/ilxd/crypto._Cfunc_secret_key_from_seed(0xc00996e1c0, 0xc00996e1e0)

This is from the rust code related to key generation:
https://github.com/project-illium/ilxd/blob/master/crypto/rust/src/lib.rs#L38
And it's called by the go code here: https://github.com/project-illium/ilxd/blob/master/crypto/nova.go#L245
Help with this would be appreciated.

For some background. I'm running tests on regtest on local host. And I'm intentionally trying to trigger duplicate and conflicting blocks. So I'm starting a number of nodes, changing a line of code, recompiling, and then starting another node with the modified code. And for some reason when I start the tx generation code one of the original nodes always crashes with a segfault.
I don't know if it has anything to do with recompiling or not. Possibly dynamic linking?
I have seen the segfault in other contexts before but I thought I fixed those. This is now the only context in which I've seen it.

I've see the segfault again. Again it was with one node running, modifying code, recompiling, and using the cli. Seems like there's some kind of issue with the linked library.
As long as this never happens without recompiling it's not an issue. But let's hope it stays that way.

Closing this for now. Pretty sure it was the dynamic linking. We've since switched to static linking.


Add tests for similar, get function signatures to work

The old signatures would cause errors if you tried similar(delta(...)).

Tests passed - ok to go?


Increase clarity for host regarding which files they're sharing

Description
In https://github.com/atom/teletype/issues/268#issuecomment-359944857, @Haacked noted:

I found the behavior to add all open files to the portal a bit surprising. It may be because my particular workflow is more focused on note taking than on pair programming.
I tend to have a lot of buffers open in Atom. Sometimes I want to pair with someone on a single file and don't want them to see all the others I have open. It's not immediately clear to me when I share a workspace what exactly is being shared. ...
And after sharing the file, I might need to work on something unrelated to the pairing session in private. I don't want to accidentally share that file by simply opening it. I think it needs to be crystal clear what's being shared, what's not being shared.

Thanks for bringing this up! :zap:
Potential solutions
Available today: Open new window before sharing a portal
When a host shares a portal, that portal only exists in the current Atom window. If the user has other Atom windows open, the portal has no knowledge of those other Atom windows, and therefore the portal cannot share any of the buffers that are open in those other Atom windows.
With that in mind, users may want to open a new Atom window before sharing a portal. Doing so will allow those users to continue using their other Atom windows without having to worry about potentially sharing the buffers that are open in those other windows.
Note: This admittedly doesn't address the need to make it crystal clear which items are being shared. We should address that need. This solution is merely offered as an option for concerned users in the meantime.
Ideas going forward
In https://github.com/atom/teletype/issues/268#issuecomment-360448423, @simurai explores a handful of potential solutions for us to consider once we're ready to tackle this issue.

When beginning to share a workspace, I can potentially see a UI that asks which of the currently open files the host wants to share, and if new files should be automatically shared as well.  I'm not sure how to do the other option though (e.g. opening a file and not sharing it when the option to share new files is selected).

I think something we should strive for: "Keep the act of sharing as quick and simple as possible".
Like whenever you wanna share, you shouldn't have to think too much. Seeing a list of files that you could exclude/include, and then having to decide if new files also should be shared or not, might feel like too much extra work? :stuck_out_tongue_closed_eyes: The mental model of "always the whole window is shared" keeps things simple and you don't have to be paranoid of juggling between individual files.
Another idea: Share in the same window.
There are these "project switching" packages like project-neue. They just replace the current project with a different one. See the gif below where I keep switching between the last used projects (teletype + about):

It's super fast and feels almost instant. So if you only want to share a single file, instead of having to do this:

"Open In New Window"
"Close Other Tabs"
"Remove Project Folder" (in the tree-view)
Start sharing portal

It could be reduced to:

"Open In Same Window"
Start sharing portal

I know "Open In Same Window" sounds super strange.. :laughing: There might be a better way of saying "Open this file in an empty project in the same window". Anyways it would be super fast to "hide" everything else and when you're done sharing, you can just as quickly switch back to the previous project and continue where you left off. Could also be an option on entire folders in the tree-view and not just on tabs.
It's more a "core" feature, but Teletype users would benefit too.

I think something we should strive for: "Keep the act of sharing as quick and simple as possible".  Like whenever you wanna share, you shouldn't have to think too much.

Yes! Thanks for bringing this up, @simurai. :bow::heart:
I contend that we can encourage more collaboration by reducing barriers to entry. Ultimately, I'd love to see Teletype provide the world's fastest transition from "I want to collaborate" to "I am collaborating." 🚀🤞We have quite a ways to go still, but we have a better chance of getting there if we avoid adding any complexity to the existing collaboration initiation flow.

A portion of the solution, for showing which files are currently shared could be to have the tab's colors changed when they are shared, like this post mentions (I have not tested this). This could possibly be added to editor-binding, so that all of the editors would get updated.

In the near-term, we're focused on other improvements to Teletype.
To make progress on those improvements, we'll need to limit the amount of time we invest into this issue in the near-term. With that in mind, @simurai's design in https://github.com/atom/teletype/issues/268#issuecomment-360448423 feels like it would nicely address the core need (i.e., "it needs to be crystal clear what's being shared") with seemingly minimal effort:

More prominent "is sharing" mode
Currently only the colored status-bar icon is a reminder that you're currently sharing your workspace. There could be stronger indication, like a border (like when screen sharing in Zoom):

If anyone wants to open a pull request to explore that approach, I'd be happy to review it and try to get it merged.

You can try the blue border with the following in your styles.less file:
// Add border when Teletype is active
@import "ui-variables";
.teletype-Host::after,
.teletype-Guest::after {
  content: "";
  z-index: 10;
  position: absolute;
  top: 0;
  left: 0;
  right: 0;
  bottom: 0;
  box-shadow: inset 0 0 10px 3px @background-color-info;
  pointer-events: none;
}

Hmmm.. 🤔  it doesn't look that great. Personally, I would probably remove it again. :stuck_out_tongue_winking_eye:

Here a more solid border:
// Add border when Teletype is active
@import "ui-variables";
.teletype-Host::after,
.teletype-Guest::after {
  content: "";
  z-index: 10;
  position: absolute;
  top: 0;
  left: 0;
  right: 0;
  bottom: 0;
  border-radius: 5px;
  border: 3px solid @background-color-info;
  pointer-events: none;
}

Ohh.. maybe this.. with the status-bar icon included:
// Add border when Teletype is active
@import "ui-variables";
.teletype-Host,
.teletype-Guest {
  &::after {
    content: "";
    z-index: 10;
    position: absolute;
    top: 0;
    left: 0;
    right: 0;
    bottom: 0;
    border-radius: 5px;
    border: 3px solid @background-color-info;
    pointer-events: none;
  }
  .PortalStatusBarIndicator {
    color: white;
    background-color: @background-color-info;
  }
}

Then it's more clear that the blue border belongs to Teletype.


setup jenkins

on master and PR, then put its status into readme.

https://github.com/twosigma/beakerx/commit/73061061e946abcdc333e56bec415c11cbd03a70


cleanup code: move c++ wrapper and go code into top level dir

cleanup code: move c++ wrapper and go code into top level dir

close due to it is now possible for now.

all non-Go source code for the package should be stored in the package directory, not in subdirectories.
https://pkg.go.dev/cmd/cgo


Create a Security.md file

A method of reporting significant bugs or vulnerabilities. Think LOG4J for example, and that it might result in a high priority fix to be released quickly.

Still just a draft, so I will close this.
Security issues can be reported via my email<EMAIL_ADDRESS>


label studio slowes down when importing OCR preannotations

Hello,
Following this post , I converted my OCR output and were able to successfully import them and see the document and bounding boxes in label studio. I am importing bounding box for 10 images and and each image has between 700-800 bounding boxes. Once I open an image to annotate and look at the bounding boxes my browser slows down significantly and does not let me annotate data properly.
I have tried latest versions of Firefox and Chrome. Although Chrome was slightly faster, still it is very slow that I cannot annotate data properly.
Do you have any recommendations or solution for this scenario?
Thank you

I can confirm. Also multiple boxes cannot be selected with the Shift key as the documentation claims (https://labelstud.io/guide/labeling.html#Select-multiple-regions). Sadly, this makes interface hardly usable for OCR word boxes labeling.


not working for new HttpClient request from angular 4.3.0

An interceptor should be added.

@ronzeidman I have the same issue, do you have any example code?

I'm also interested in this.

Try npm i<EMAIL_ADDRESS>Works with HttpClient (angular >5.0.0).

Look at the changelog for details

version 8.0.0 works with angular 5,


Option to run only on HTTPS

Currently the apache and nginx configurations create virtual host for both port 80 and 443. I would like to have a feature to run the app only on port 443 and forward all the requests from port 80 to 443.

If you set prefer_https to true on the Hostname models of choice, this logic will do a redirect from within the app. I hadn't considered redirection from the server level, which makes a bit more sense eventually. A good idea for 3.x


docs: Add migrating to classix instructions

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@6eb20d8). Click here to learn what that means.
The diff coverage is n/a.

:exclamation: Current head a9a327d differs from pull request most recent head 8d764cf. Consider uploading reports for the commit 8d764cf to get more accurate results

@@           Coverage Diff            @@
##             main       #32   +/-   ##
========================================
  Coverage        ?   100.00%
========================================
  Files           ?         1
  Lines           ?         9
  Branches        ?         2
========================================
  Hits            ?         9
  Misses          ?         0
  Partials        ?         0

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Test commit, do not merge

This is a test of ReadTheDocs PR integration, do not merge

Coverage remained the same at 60.745% when pulling a991380b32c5f3d01b941e1919d49c8c24ad4bb8 on jtschladen:test-do-not-merge into 66e7acf77325286ed26327d61f1593efbec5cc5c on Netflix:master.


Full hybrid MT with termino management

What was proposed in #447 is very interesting, but brings several bad behaviours, because of the way the neural model is handling the live-tuning.
Here is a new version, reverting roles between statistical and neural MT:

to be efficient, you need a statistical MMT2 model trained on a short amount of basis data, like 2M sentences Europarl. With large data, it would be too slow.
then, for each sentence translation.. Suppose we would like to translate the following FR sentence:
SRC="Le Tribunal de Commerce de Nanterre a statué sur le sort de l'accusé."
create a unique temporary memory TEMPMEM000X where "X" is a sentence counter (waiting for a feature asked elsewhere to be able to sent sentences while translating)
translate the input sentence with all available MT. Neural MMT2 (RNN) is very good to preserve local phrasings, while neural MMT3 (Transformer) is very good to handle long and complex formulations. All kind of MT can be added. Feed all translated pairs in the TEMPMEM000X memory of the above statistical MMT2.
collect all known terminology and its translation. For example, in the above sentence spaCy is finding the entity "Tribunal de Commerce de Nanterre". Suppose we would like to preserve this entity in the translation. Prefix all words of the source part with a coding char like "ǂ". Feed the TEMPMEM000X with the coded terminology pairs, like:
SRC=""ǂTribunal ǂde ǂCommerce ǂde ǂNanterre"
TRG=""Tribunal de Commerce de Nanterre"
translate the sentence with the same coding on known termino, specifying a context vector using only the TEMPMEM000X memory with a high weight:
"Le ǂTribunal ǂde ǂCommerce ǂde ǂNanterre a statué sur le sort de l'accusé."
delete the TEMPMEM000X memory to avoid interferences with other translations

The MMT2 statistical model behaves as a multi-translations/termino mixer, using the good neural translations as sentence references, replacing the coded termino with the known provided values, creating this result (experimented with the FR-EN NeuroSpell interface):

Thanks @EtienneAb3d, hybrid MT is an interesting topic. Regarding the phrases which do not need translations, do you have better solutions to detect them. Spacy is somehow not accurate enough.

Yes. Have a look here:
https://github.com/EtienneAb3d/OpenNeuroSpell
;-)


Rename BUILD_DIR to PUBLISH_DIR

The BUILD_DIR constant has been renamed to PUBLISH_DIR.

Thanks! Would it be possible to make a new npm release?

released as<EMAIL_ADDRESS>


replays by game_map

I notice in the API docs that the game_map query param is currently disabled. can we get a scope on this issue? I'd be willing to work on it given some requirements.

Problem is a poor query performance. You can import SQL dump on your development server and try to optimize query execution times.

Ok cool I can work on that.  I don't see a CONTRIBUTING.md file in the repo.  Do you have any PR requirements before I start working?

Nope, no requirements

Alright cool.  I'll probably write some integration / end-to-end tests and it is likely I'll use the query builder instead of eloquent to query replays.

Just make sure that it will be easy extend your code in case we want to add more params in the future or alter resulting JSON in some way (I think you'll still need to convert result to eloquent object to use Resources for output)

Ok I'll be sure to cover with tests.  I probably won't change your getQuery method, rather add a new method to use in index. If you are satisfied with the change it could be applied to the paged endpoint as well.

And i'll be sensitive to extension efforts.

where can I get my hands on a sqldump?

http://s3.hotsapi.net/hotsapi.sql.gz

Import may take quite a while on lower end servers. I plan to make .csv export for larger tables to speed up the process, I think I'll get to it in a couple of days.

Lol, this is crushing my VM

So the import slowdown is also impacted by constraints, like indexes (especially unique ones). If we were to strip the constraints before the import, blast all the data in, and then reattach the constraints, it would probably import way faster.

That's what I did when I was migrating DB server from AWS to Google Cloud, coupled with loading large tables from .csv. But it doesn't help much - data import becomes indeed quite fast but rebuilding indexes and constraints takes forever (almost 10x longer than import) so the resulting import time is still almost the same - it took around 6 hours to import 3M replays.

Understood. My VM is probably pretty underpowered for this import. It is still going.. 15 hours later.

Still choking on the player_talent table which is of course locked so I can't get a count on it.

That's the biggest table with 300M rows. You can get row count approximation from information_schema.tables

Ah thanks. 176M so far.

By the way, hotsapi production server is also pretty underpowered - 1.5Gb RAM and 0.5 CPU. Though on import I had a bit larger instance with 4 (or 2) CPUs.

What version of mysql is it running?

Or is that on a google server now?

MySQL 5.7, yes currently on Google Cloud SQL

@mlynam Did you ever get anywhere on this?

I think this query will be deprecated once BigQuery dataset will become available, closing for now


Failing serialization with concurrency

I found another problem with the serialization code. If I don't code TestSerialization as a concurrent test, then there's no issue, but as soon as you add the concurrency it fails in one of two ways.
goon_test.go:2686: error deserializing - goon: Failed to decode field Played - extra data in buffer
goon_test.go:2731: error deserializing - goon: Failed to decode field Played - gob: decoding into local type [4]goon.Card, received remote type [4]int
I've looked over the (get/free)serialization(de/en)coder() functions and they look correct, but If I comment out the List usage and just recreate the gob (en/de)coder, it works with no bugs. From reading the gob package docs, I'm unsure of exactly what the issue is, but it doesn't seem the gob package was intended to be re-used the way goon is doing it.
go test -test.run Serialization
Sorry for the large test code, it's been ~4 hours and I haven't been able to figure out exactly what the problem is in the test code but I have figured out how to reproduce it at least.
https://travis-ci.org/mzimmerman/goon

Thanks again for reporting this. I know it took me 5 years to get to fixing things, but hey better late than never.
The issue isn't encoding/gob related after all. It's much simpler. The serializer reused bytes.Buffer on which it had used the Bytes() method. The documentation states that the slice is valid for use only until the next buffer modification. Because the returned []byte was only used internally by goon in a sequential fashion the problem only showed up when using goon itself concurrently. Basically any subsequent calls to Get would rewrite the serialized []byte contents. If the previous Get call's memcache API call wasn't already executed then this could lead to cache corruption.
I fixed this issue and included it in my PR for a new serialization engine, which also happens to completely drop encoding/gob and is about 3x faster.


[device-report] Redragon IRELIA K658 PRO

Device Info

Sinowealth Device: SH68F90A
IC Label: BYK916
Product Page: _https://redragonshop.com/products/irelia-k658-pro-90-full-transparent-keyboard_

Part Info
firmware_size: 61440
vendor_id: 0x258a
product_id: 0x0049

Operations Tested

[x] Read
[x] Write (Linux Mint only)

Platforms Tested

[x] linux
[ ] macos
[x] windows

Checksums

Bootloader MD5: 3e0ebd0c440af5236d7ff8872343f85d
Stock Firmware MD5: da07d2bde42817f2c8ac51fdbe49c3d2

HID Dump
A dump from usbhid-dump, win-hid-dump or mac-hid-dump

HID Tool Output
# Redragon K658 Irelia Pro using win-hid-dump
...
258A:0049: BY Tech - Gaming Keyboard
PATH:\\?\hid#vid_258a&pid_0049&mi_00#8&65fed84&0&0000#{4d1e55b2-f16f-11cf-88cb-001111000030}\kbd
DESCRIPTOR:
  05  01  09  06  A1  01  05  07  19  E0  29  E7  15  00  25  01
  35  00  45  01  65  00  55  00  75  01  95  08  81  02  95  38
  81  03  05  08  19  01  29  05  95  05  91  02  95  03  91  03
  C1  00
  (50 bytes)
...

Thanks for the report @Sherman729!
Could you please share the HID Dump for the keyboard when it is in its stock firmware mode (not bootloader mode)?

Hi! Could this be the right one?

Deleted the old one, added the new. Need the bootloader one too?

@Sherman729 sorry, I might have mistaken your previous dump for when the device was in bootloader mode.
Still, since windows divides these HID devices into multiple TLCs I would like to see all records, but only one of them is shared.
Can you please add all records that have the path prefix PATH:\\?\hid#vid_258a&pid_0049 into the HID Tool Output?

The HID dumper crashes a lot, but let me get it running in a clean machine
and I'll go back to it.
On Sat, Apr 13, 2024, 15:19 Karolis Stasaitis @.***>
wrote:

@Sherman729 https://github.com/Sherman729 sorry, I might have mistaken
your previous dump for when the device was in bootloader mode.
Still, since windows divides these HID devices into multiple TLCs I would
like to see all records, but only one of them is shared.
Can you please add all records that have the path prefix
PATH:\?\hid#vid_258a&pid_0049 into the HID Tool Output?
—
Reply to this email directly, view it on GitHub
https://github.com/carlossless/sinowealth-kb-tool/issues/61#issuecomment-2053736716,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BBR4DCXK7TFHMXFYIDAUPBLY5GANHAVCNFSM6AAAAABFYABJQ6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANJTG4ZTMNZRGY
.
You are receiving this because you were mentioned.Message ID:
@.***>

I got it to work in a clean linux install, kb is the only device attached to the machine. Dumped with usbhid-dump, as follows:
001:010:001:DESCRIPTOR<PHONE_NUMBER>.200599
06 01 00 09 80 A1 01 85 01 19 81 29 83 15 00 25
01 95 03 75 01 81 02 95 01 75 05 81 01 C0 05 0C
09 01 A1 01 85 02 19 00 2A FF 02 15 00 26 FF 7F
95 01 75 10 81 00 C0 06 00 FF 09 01 A1 01 85 03
15 00 26 FF 00 09 2F 75 08 95 03 81 02 C0 05 01
09 06 A1 01 85 04 05 07 19 04 29 70 15 00 25 01
75 01 95 78 81 02 C0 06 00 FF 09 01 A1 01 85 05
15 00 26 FF 00 19 01 29 02 75 08 95 05 B1 02 C0
06 00 FF 09 01 A1 01 85 06 15 00 26 FF 00 19 01
29 02 75 08 96 07 04 B1 02 C0 05 01 09 02 A1 01
85 07 09 01 A1 00 05 09 15 00 25 01 19 01 29 05
75 01 95 05 81 02 95 03 81 01 05 01 16 00 80 26
FF 7F 09 30 09 31 75 10 95 02 81 06 15 81 25 7F
09 38 75 08 95 01 81 06 05 0C 0A 38 02 95 01 81
06 C0 C0 06 00 FF 09 01 A1 01 85 08 15 00 26 FF
00 09 00 75 08 96 7D 01 B1 02 C0
001:010:000:DESCRIPTOR<PHONE_NUMBER>.202649
05 01 09 06 A1 01 05 07 19 E0 29 E7 15 00 25 01
95 08 75 01 81 02 95 01 75 08 81 03 95 06 75 08
15 00 26 FF 00 05 07 19 00 2A FF 00 81 00 25 01
95 05 75 01 05 08 19 01 29 05 91 02 95 01 75 03
91 03 C0

Yes, thank you!

Glad to hear it!
On Sat, Apr 13, 2024 at 5:08 PM Karolis Stasaitis @.***>
wrote:

Yes, thank you!
—
Reply to this email directly, view it on GitHub
https://github.com/carlossless/sinowealth-kb-tool/issues/61#issuecomment-2053761909,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BBR4DCTKTVMXWBSSHDRXF73Y5GNFPAVCNFSM6AAAAABFYABJQ6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANJTG43DCOJQHE
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Saludos,
Norman De Barath


The code doesn't run- got: missing PocketSphinx modul

I can't get the result of the code, I got this message:
Sphinx error; missing PocketSphinx module: ensure that PocketSphinx is set up correctly.
This is the code I have:
import speech_recognition as sr
from os import path
AUDIO_FILE = path.join(path.dirname(path.realpath("init")), "test.wav")
r = sr.Recognizer()
with sr.AudioFile(AUDIO_FILE) as source:
audio = r.record(source)
try:
print("Sphinx thinks you said " + r.recognize_sphinx(audio))
except sr.UnknownValueError:
print("Sphinx could not understand audio")
except sr.RequestError as e:
print("Sphinx error; {0}".format(e))
I expected to have to spoken words as a text.
Please note that:
My system is <Windows 7 x64>.
My Python version is <2.7.12>.
My SpeechRecognition library version is <3.5.0>.
I didn't install PyAudio since I want to use the speech recognition offline and from existing file.
I installed PocketSphinx from (pip install pocketsphinx) .. I am not sure though.

Hi @RafahF,
It seems like you don't have PocketSphinx installed properly. To test that it is, run each of the following commands and post the output of all of them here:
python -V
pip -V
python -c "from pocketsphinx import pocketsphinx; print(pocketsphinx)"

For example, when I run it on my machine, I get the following:
$ python -V
Python 3.5.2
$ pip -V
pip 9.0.0 from /usr/local/lib/python3.5/dist-packages (python 3.5)
$ python -c "from pocketsphinx import pocketsphinx; print(pocketsphinx)"
<module 'pocketsphinx.pocketsphinx' from '/home/anthony/.local/lib/python3.5/site-packages/pocketsphinx/pocketsphinx.pyc'>

@Uberi
$ python -V Python 2.7.12 $ pip -V pip 9.0.1 from c:\python27\lib\site-packages (python27) $ python -c "from pocketsphinx import pocketsphinx; print(pocketsphinx)" Traceback (most recent call last):   File "<string>", line 1, in <module>   File "C:\Python27\lib\site-packages\pocketsphinx\__init__.py", line 35, in <mo dule>     from sphinxbase import *   File "C:\Python27\lib\site-packages\sphinxbase\__init__.py", line 32, in <modu le>     from .ad import *   File "C:\Python27\lib\site-packages\sphinxbase\ad.py", line 35, in <module>     _ad = swig_import_helper()   File "C:\Python27\lib\site-packages\sphinxbase\ad.py", line 34, in swig_import _helper     return importlib.import_module('_ad')   File "C:\Python27\lib\importlib\__init__.py", line 37, in import_module     __import__(name) ImportError: No module named _ad
Can you help me fixing it please?

Hi @RafahF,
Try installing pocketsphinx 0.0.9 from the provided wheels in the third-party directory of this project, using something like pip install third-party/pocketsphinx-0.0.9-cp35-none-win_amd64.whl. The version you have installed appears to have some SWIG-related bugs.


Check installation of shell completions and man-pages via packages

[x] Fedora / Redhat (.rpm)
[x] Debian (.deb)
[x] Alpine
[x] HomeBrew
[x] Arch Linux

Done


UTF-8 aware rich text

Bluesky's rich text works with UTF-8 indices for indicating which part of the text is part of a feature, the code currently in use in Skyview currently ignores this fact and uses the indices to slice text in UTF-16 (JavaScript) instead.
my recommendation would be to follow the example I had provided here
this makes sure that non-Latin posts are being rendered correctly

Closing since it's now relying on the RichText API from @atproto/api

Sorry didn't check the tracker before inplementing it :)
On Tue, Oct 17, 2023, 22:37 Prana Adiwira @.***> wrote:

Closed #7 https://github.com/badlogic/skyview/issues/7 as completed.
—
Reply to this email directly, view it on GitHub
https://github.com/badlogic/skyview/issues/7#event-10683510322, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAD5QBGEHJD5GLQIXYYEGLTX73T63AVCNFSM6AAAAAA6D7QIQSVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJQGY4DGNJRGAZTEMQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

It's all good, so long as the posts are being rendered correctly :)


[fud2] First pass for custom testbench support

This PR is a first pass at bringing the custom testbench (from our work on the FIRRTL backend) to fud2.
Changes

fud2 supports both types of primitives support for the FIRRTL backend

The existing Verilog primitive implementations via --through firrtl (we called this FIR_SV in our OSDA paper)
Metaprogramming FIRRTL primitive implementations via --through firrtl-with-primitives (we called this FIR_FIR in our OSDA paper)

fud2 now has two additional stages (verilog-refmem (for Verilator with custom testbench) and verilog-refmem-noverify (for Icarus with custom testbench)).
We use the YXI backend to produce memory information for the testbench generator to read.
There is a small fix for the YXI backend denoting comb_mem_d* as Sequential.

Issues

Fixed! Thanks @sampsyo :) (Punchline: Don't use the rsrc config, instead use e.rsrc().)

Simulation using the custom testbench take more cycles than using the standalone testbench (#1956)

The YXI backend does not emit memory information for ref cells (#1993)

TODO: I also ran into an error when using the external-to-ref pass, but I should look into that one. (It would be ideal to remove the sed commands)

I've introduced lots of states/transitions to the fud2 graph, so we may want the custom testbench to replace the standalone testbench.

Once the above is done, we should have a larger switch to always using the custom testbench.

The YXI backend should be built by default.
We should deprecate @external.
The Verilog backend should always be in "synthesis" mode (don't produce readmemh/writememh)

Usage
Note that you must build Calyx with the YXI backend (via cargo build --features yxi) for the below to work.
To run a FIRRTL-translated Calyx program using Verilog primitives, run fud2 with --through firrtl.
ex)
fud2 examples/tutorial/language-tutorial-iterate.futil --to dat -s sim.data=examples/tutorial/data.json --through firrtl

To run a FIRRTL-translated Calyx program using FIRRTL primitives, run fud2 with --through firrtl-with-primitives.
ex)
fud2 examples/tutorial/language-tutorial-iterate.futil --to dat -s sim.data=examples/tutorial/data.json --through firrtl-with-primitives

Both examples should currently yield
{
  "cycles": 76,
  "memories": {
    "mem": [
      42
    ]
  }
}

Excellent! Just to do a little more high-level planning here, IMO let's try to get this ready to merge in close to its current state. Totally fine if there is plenty of outstanding work to do, as long as it doesn't disrupt the current workflow (the old-style testbench). In particular, I say we should not attempt to solve these problems:

New-testbench executions are slower than old testbench executions.
YXI is not built by default.
ref vs. @external troubles; the sed replacement trickery; etc.

IMO each of those could be its own separate follow-up PR (or even multiple PRs!) as we get closer to "flipping the switch", i.e., changing the default and removing the old-style test bench. So once this lands, we make a tracking issue to address those things (as well as the switch-flipping business).
So for now, let's focus our work on whatever's required to make this new functionality minimally work and not "take over" as the default when you do --from calyx --to verilog or whatever. Then let's merge!

I looked over all the lib.rs changes (and the snapshot tests) once again and everything looks pretty good!

Great! Thanks for taking another look :) Will go ahead and merge now, and make a tracker issue later!

Tracker issue is #2086!


If revalidateOnFocus and suspense are used together, when an error occurs onfoucs listeners are never cleaned up

Bug report
Description / Observed Behavior
If revalidateOnFocus and suspense are used together, when an error occurs onfoucs listeners are never cleaned up
Expected Behavior
When errors are caught in error boundary, onfocus listeners are cleaned up, a page refocus never triggers a revalidation.
Additional Context
SWR version: 0.5.5

Hi, could you please provide a codesandbox repro ?
FYI: https://codesandbox.io/s/suspense-focus-5c0zf?file=/src/App.js
This seems works

I think this is tricky – when a component throws and gets caught by the error boundary, the hooks will be unmounted so you can't re-render anymore. The same applies to a Suspense boundary as well.
This needs more discussion, also related to #168.


Fix for image reuse form

Involves fixing CSS for image "stretched links", and makes destroy/put/patch/post button_to helpers behave more consistently.
The change is this: when you use button_to, in Rails < 7, you have to pass a block to be sure it prints a form with a button[type='submit']. If you pass a text arg, it prints a form with an input[type='submit'] element. It's a known inconsistency in Rails and is fixed in Rails 7, so forcing this here will make the app future friendly as much as possible.
Also fixes an intermittent bug arising from the too-close times for created_at of the two Peltigera obs fixtures. Changes the name of peltigera_rolf_obs to peltigera_mary_obs to reflect fixture.

Coverage: 93.665% (+0.003%) from 93.661% when pulling f315f44fd84159cff04b7aca9d0e55cbe402f404 on nimmo-fix-button_to-links into 4644b4bcbe9b9d2717dde893db5a98c7b912c71d on main.


Header UI Enhance with drop down menu

Website seems to be with a incorporate header. so i ll be adding a header wtih new fonts and styling

@Sudhanshutiwari-cs assigned

Opened Pull Request for this task @Mujtabaa07 Accept it  and assign me SWOC label for this issue
Thank you

Done


Update energy history

Have HA component build energy history dictionary

Polling the energy usage in update() will cause HA to make a lot of API calls as additional devices are added. I think we should probably implement something that would lead to a one time poll for energy details at startup and then not poll energy details for long periods of time as the historical data won't be changing very often.
There might be event helpers we can tap into - https://dev-docs.home-assistant.io/en/master/api/helpers.html#module-homeassistant.helpers.event

I agree, I was thinking we can have the user choose weekly, monthly and/or yearly histories to retrieve.
I think it would be either homeassistant.helpers.event.async_track_point_in_time(hass, action, point_in_time) or homeassistant.helpers.event.async_track_point_in_utc_time(hass, action, point_in_time) to update the data either once every 24 hours or at a specified time daily.
Another idea might be to set an entity on the creation of the platform that we test to see if a certain amount of time has passed.  If greater than the amount of time update energy and time entity, so every time the vesync switches are updated and the specified time hasn't passed it won't update.

Yup, the last part of your comment is probably the best match and something I've started work on. Any ideas on the update interval for energy details? Every 6 hours? 12? 24?


No such property: maven.mainClass / Java/Maven build failures

Recently I experience build failures while running the codeql-java-autobuild:
  [2024-12-18 05:24:27] [autobuild] > /home/runner/work/mailclena/mailclena/./mvnw clean package -f pom.xml -B -V -e -Dfindbugs.skip -Dcheckstyle.skip -Dpmd.skip=true -Dspotbugs.skip -Denforcer.skip -Dmaven.javadoc.skip -DskipTests -Dmaven.test.skip.exec -Dlicense.skip=true -Drat.skip=true -Dspotless.check.skip=true -s /home/runner/.m2/settings.xml -t /home/runner/.m2/toolchains.xml
  [2024-12-18 05:24:27] [autobuild] Picked up JAVA_TOOL_OPTIONS:  -Dhttp.keepAlive=false -Dmaven.wagon.http.pool=false
  [2024-12-18 05:24:28] [autobuild] org.codehaus.plexus.classworlds.launcher.ConfigurationException: No such property: maven.mainClass
  [2024-12-18 05:24:28] [autobuild] 	at org.codehaus.plexus.classworlds.launcher.ConfigurationParser.filter(ConfigurationParser.java:355)
  [2024-12-18 05:24:28] [autobuild] 	at org.codehaus.plexus.classworlds.launcher.ConfigurationParser.parse(ConfigurationParser.java:114)
  [2024-12-18 05:24:28] [autobuild] 	at

on project https://github.com/ottlinger/mailclena
As I do not configure any of the variables in
https://github.com/ottlinger/mailclena/blob/master/.github/workflows/codeql.yml
I wonder how to proceed.
Thanks

It looks like your codeql scans started failing on this commit: https://github.com/ottlinger/mailclena/actions/runs/11750645752 where you upgraded to mvn 4-beta-5.
In your codeql workflow you are using autobuild, which uses heuristics to try to build a project using standard build commands. I'm guessing that autobuild is trying to use an incorrect maven command.
The easiest thing to do is to use build-mode: none. It is a small change to your workflow.

I assume the problem is somewhere in the autobuild script as it adds an unknown property:
 [2024-12-18 05:24:28] [autobuild] org.codehaus.plexus.classworlds.launcher.ConfigurationException: No such property: maven.mainClass
The build is back in the green via https://github.com/ottlinger/mailclena/commit/1d8dc079937fd360c8374afe982742e6f6f9b44f but I assume that is not what you intended.
Thanks

That's another option. You're using an explicit build command. If that works for your repository, then there are no problems. I'll close this issue.

I thought that you would include something on your side to not issue this parameter by default in Maven builds as it is incompatible with newer Maven versions .... but I'm fine if my build is back in the green :) Thanks again.

I'll discuss this with the team maintaining the Java extractor.


Any plan to integrate Turbo Model?

plus account can access Turbo Model, any plan to integrate Turbo Model?

I don't know the name of the Turbo model, what's it? You are free to update the model name manually in your settings.json with any model you'd like

I also want to use it would be nice if you could implement that feature :)

@cowith @shroominic can one of you find out the model used in the Turbo Plus following the instructions here: https://github.com/gencay/vscode-chatgpt/issues/93#issuecomment-1418260609. If that's a different model, you can overwrite the model in your settings.json using that new value although it will show a warning message you can ignore it. Let me know if you get response properly by overwriting the settings.json like this:
"chatgpt.gpt3.model": "<turbo_model_name>"

{
    "models": [
        {
            "slug": "text-davinci-002-render-paid",
            "max_tokens": 4097,
            "title": "Default",
            "description": "The standard (Plus) ChatGPT model",
            "tags": [
                "concise",
                "plus"
            ]
        },
        {
            "slug": "text-davinci-002-render-sha",
            "max_tokens": 4097,
            "title": "Turbo",
            "description": "Optimized for speed (alpha)",
            "tags": [
                "plus"
            ]
        }
    ]
}

Just rechecked it inside the payload and its the same model: "text-davinci-002-render-sha"

@shroominic thank you. I added Turbo Plus model to settings in V3.5.11. It'll be available on the marketplace in a few minutes. Once upgraded, select the Turbo Plus model to try it out! Since I don't have a Turbo Plus subscription at the moment, I am not able to verify it, but once you get the new version can you check if it works?

Getting error 400 when selecting the Turbo Plus model - maybe there is still something different with the api

@shroominic could you take a screenshot (Similar to what I shared on the other issue) of the payload - feel free to strip the ids - I am interested in what's being sent to /conversations API with Turbo plus model

it worked for me. Thanks @gencay

@cowith @shroominic could you try the Turbo Plus model again on the V3.6.1? I think it may work now on the new version, could you please confirm?

After reinstall the plugin, it worked! Thanks @gencay


Weird S3 Bucket and BucketV2 behavior

What happened?
I was initially trying to create an S3 bucket with the BucketV2 resource, but it ended up failing with a cryptic message when I ran either pulumi up or pulumi preview. However, after doing some digging, I found the same thing happened with the Bucket resource. Both failed with the same error.
I continued testing stuff and I found that the error only occurs if I import Bucket or BucketV2 directly (i.e. import { Bucket, BucketV2 } from "@pulumi/aws/s3"). If I reference the resources from their parent package, then it works as expected (i.e import { s3 } from "@pulumi/aws" or import * as aws from "@pulumi/aws").
Steps to reproduce
The following should fail:
import { Bucket } from "@pulumi/aws/s3";
new Bucket("testing", {});

The following should succeed:
import { s3 } from "@pulumi/aws";
new s3.Bucket("testing", {});

import * as aws from "@pulumi/aws";
new aws.s3.Bucket("testing", {});

To test, run either pulumi preview or pulumi up. In all 3 examples, Bucket can be replaced with BucketV2 and the same behaviour should occur.
Expected Behavior
Pulumi previews/creates the S3 bucket properly in all 3 cases.
Actual Behavior
An error gets thrown before the preview can be completed when importing the resources directly.
Previewing update (dev)

View Live: https://app.pulumi.com/<preview url>

     Type                 Name         Plan       Info
 +   pulumi:pulumi:Stack  testing-dev  create     1 error

Diagnostics:
  pulumi:pulumi:Stack (testing-dev):
    error: Running program '/home/alex/Downloads/testing' failed with an unhandled exception:
    TypeError: Class extends value undefined is not a constructor or null
        at Object.<anonymous> (/home/alex/Downloads/testing/node_modules/@pulumi/cloudwatch/eventRuleMixins.ts:70:56)
        at Module._compile (node:internal/modules/cjs/loader:1103:14)
        at Object.Module._extensions..js (node:internal/modules/cjs/loader:1157:10)
        at Module.load (node:internal/modules/cjs/loader:981:32)
        at Function.Module._load (node:internal/modules/cjs/loader:822:12)
        at Module.require (node:internal/modules/cjs/loader:1005:19)
        at require (node:internal/modules/cjs/helpers:102:18)
        at Object.<anonymous> (/home/alex/Downloads/testing/node_modules/@pulumi/cloudwatch/cloudwatchMixins.ts:16:1)
        at Module._compile (node:internal/modules/cjs/loader:1103:14)
        at Object.Module._extensions..js (node:internal/modules/cjs/loader:1157:10)

Versions used
CLI
Version      3.30.0
Go Version   go1.17.9
Go Compiler  gc
Plugins
NAME    VERSION
aws     5.2.0
nodejs  unknown
Host
OS       fedora
Version  35
Arch     x86_64
This project is written in nodejs (/home/alex/.nvm/versions/node/v16.14.2/bin/node v16.14.2)
Current Stack: dev
Found no resources associated with dev
Found no pending operations associated with dev
Backend
Name           pulumi.com
URL            https://app.pulumi.com/akrantz01
User           akrantz01
Organizations  akrantz01, wafflehacks
NAME            VERSION
@pulumi/aws     5.2.0
@pulumi/pulumi  3.30.0
@types/node     14.18.15
Pulumi locates its logs in /tmp by default
Additional context
No response
Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

Hi @akrantz01 - thank you for filing this issue. I believe this is a duplicate of https://github.com/pulumi/pulumi-aws/issues/772 and we're not sure if a fix is on the roadmap.
We apologize for any inconvenience here.


Hold (rather than disable) notifications

With #123, we have the ability to disable notifications. However, we do not have the ability to hold notifications until a state is set, then fire them after.

I think this issue came about from looking at traitlets - their widget framework can lead to many traits being modified rapidly at once. In that case, holding notifications is useful. There are no race conditions during interaction, just the final notifications when all the widget dust settles. However, properties is designed with more of a focus on direct user interactions, with single, slow, non-conflicting updates.
There is still a possibility multiple properties are set at once through kwargs - this creates ambiguity for the order of notifications (if you set ({'a': ..., 'b': ...}) does 'a' notification or 'b' notification fire first?). But holding notifications does not provide a solution to that, it only postpones the problem.
Bottom line is I do not think we need to hold notifications.


The innovation is similar to Autoformer, especially in frequency attention module

I wouldn't even call it attention really. It's just a straightforward fft ifft, selecting for frequencies with highest amplitude


Can I build DCNV2 on Windows?

I tried it, but failed. Is it possible if I run DCNV2 on windows?

I think it is impossible because the project will use .so file.

Yes, it is possible I was just able to build. The process was a pain... I hope I capture it all.
I am running Windows 10 and used Visual Studio 2017 to build.

You want pytorch==1.4 and torchvision==0.5
From within Visual Studio Installer click modify and install the following.

Individual Component > Windows Universal CRT SDK
Desktop Development with C++
As part of the Desktop Development with C++ you'll need the following optional components

Windows 10 SDK
VC++ 2017 toolset

I had to update this line of code https://github.com/CharlesShang/DCNv2/issues/14. You may not have to depending on the version you are on.
Add the following to your environment vars (your paths may be slightly different so pay special attention here).

INCLUDE="C:\Program Files (x86)\Windows Kits\10\Include\10.0.16299.0\ucrt;C:\Program Files (x86)\Windows Kits\10\Include\10.0.16299.0\shared"
LIB="C:\Program Files (x86)\Windows Kits\10\Lib\10.0.16299.0\um\x64;C:\Program Files (x86)\Windows Kits\10\Lib\10.0.16299.0\ucrt\x64"
Add this to your PATH "C:\Program Files (x86)\Windows Kits\10\bin\10.0.16299.0\x64"

After all of that I was able to get it to build by opening up x64 Native Tools Command Prompt for VS 2017 navigating to the repo, activating my conda environment and running python setup.py build develop

@seth-cohen Hi, how i add INCLUDE、LIB and PATH on my window PC.

I still have the same question. How can you solve it?

For all latest pytorchs with cuda on Windows:
https://github.com/rathaROG/DCNv2_Windows


Consolidate xinetd service restart

To address #117, where xinetd service restart rate limiting is hit.
Add restart_xinetd arg to deploy_problem, with actual restart held off
from per-problem to after all problems are deployed.

I did a quick test by bringing the changes in and redeploying the shell machine - on the run of the mass deployment I now get this response, looks like once one gets blocked out in causes a cascade until its cleaned later.

journalctl2.txt

After discussion, this would only help if shell_manager deploy were fed a list of problem names, as it supports. However the most common use currently is to go through list of names and run deploy on each individually (as done even in current ansible playbook).  Will revise and include a command line --no-restart flag, so that a user can restart xinetd manually when done with all deployments.


backup tests need longer wait for sync

Issue
backup tests fail due to the timeout for resync
Context
resync is successful - but it takes longer than the time we provide in the tests
Solution
Add a longer timeout period after resync

this was a testing PR to enter CI - closing as no longer in use.
Note that the tests eventually pass - but the CI fails due to upterm


Arithmetic is not parsed

Looks like arithmetic becomes an ArithSubst String, and it isn't parsed any further. What's the plan there? Should I instead try to parse the arithmetic strings as C?

I admit that I didn't implement it because my original use case (parsing config files) didn't use arithmetic expressions.
According to the docs, the innards of a $((...)) undergo variable expansion etc before the arithmetic is parsed. So really, we should parse the innards as a Word instead of a String. Since this expansion can only be performed at runtime, we can't parse the actual arithmetic expression when we parse the script.
However, it would be nice if we have a side module that can parse arithmetic from a string (similar to the Cond and Expand modules). For now, using a C parser could work (like the language-c package).
The fact that there could be command substitutions etc in the arithmetic expression is a little annoying. If you're writing a static analysis tool or a transpiler, maybe you could only accept Words that are plain strings in the arithmetic expression to rule out any funny business.

Took me a while to find it so including it here: the bash arithmetic parser: http://git.savannah.gnu.org/cgit/bash.git/tree/expr.c


added a remember function

this is a remember function. eg: we tell our assistant to remember "my keys are in my desk"...... and later ask the assistant "do you remember" ..... then the assistant will reply "you said me to remember that my keys are in my desk" . @rohitjoshi6

Good work @ParamTatiya02
Here's a suggestion : Its a good practice to mention the issue number whenever you are making a PR
Reference Link : https://docs.github.com/en/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue
Thank you for contributing


fix the compilation error against prometheus/client-golang master

This fixes the issue: https://github.com/docker/go-metrics/issues/12
Signed-off-by: tifayuki<EMAIL_ADDRESS>

This fix doesn't look quite right. What was the change in prometheus that caused this compilation error?
The issue here is that prometheus.HistogramVec is no longer guaranteed. I think you need to change the field type to the observer interface.

@stevvooe
Have changed to type assertion to interface observer.
Once this PR and #15 get merged, we can use it to add prometheus metrics to the docker/distribution

@stevvooe
Do you have time to review this PR?
Thank you

Any update on this? Would love to see https://github.com/docker/distribution/pull/2466 happen


Neomake not working in neovim for windows, when using bash as shell

Expected behavior
Neomake in Neovim for windows using bash as shell
should behave just like
Neomake in Neovim for windows using cmd as shell
Steps to reproduce

set shell=bash for neovim in Windows10.
:Neomake a python file.

Output from :NeomakeInfo
Neomake: [15.21:1:1] Starting async job [string]: bash /s /c python "C:\Users\foo\.config\nvim\plugged\neomake\autoload\neomake\makers\ft\python\compile.py" best.py.
Neomake: [15.21:1:1] cwd: C:\Users\foo\sandbox.
Neomake: [15.21:1:1] output on stderr: ['/usr/bin/bash: /s: No such file or directory', ''].
Neomake: [15.21:1:1] exit: python: 127.
Neomake: [15.21:1:1] python: completed with exit code 127.
Neomake: [15.21:1:1] python: unexpected output on stderr: /usr/bin/bash: /s: No such file or directory.
Neomake: python: unexpected output on stderr: /usr/bin/bash: /s: No such file or directory
Neomake: [15.21:1:1] python: unexpected output. See :messages for more information.
Neomake: [15.21:1:1] Cleaning jobinfo.

By the way things look, seems like Neomake is passing the same string to bash that is passed when using cmd as shell. (The /s /c part)
Thanks in advance.

Does it work with set shell=bash?
I've started a while ago to get tests running on AppVeyor for Windows, but it is a mess: https://github.com/neomake/neomake/pull/1575

Also related: https://github.com/neomake/neomake/pull/1896

And https://github.com/neomake/neomake/issues/1893.

Do you mean if it works with
set shell=cmd ?
If not, no. This bug was produced with set shell=bash.
If so, Yes. I am not having any issues with set shell=cmd

If so, Yes. I am not having any issues with set shell=cmd

Ah, at least.
I do not have much motivation myself to fix Windows issues, but will gladly accept a PR - please review the referenced issues/PRs though.. a fix might be in there already.

Looks like this is a Neovim Issue;
The value 'shellcmdflag' should be set according to the value of 'shell', but apparently it isn't.
Closing the Issue.

https://github.com/neomake/neomake/blob/53284f8750e095ee3c1c4506379a6536f796bbde/autoload/neomake/utils.vim#L66-L68
return &shell=='cmd' && (has('win32') || has('win64'))

Assuming that the user is not using windows when they are not using the 'cmd' shell seems to work(at least for me)
Should I make a PR?

No, this gets used also in other places, e.g. to use "NUL" instead of "/dev/null".
After all, you are still running Windows.


Add variable units to API response

Data in the source shapefiles does not contain units. Units are critical for interpreting and analyzing the data. We can either:

add units as constants in the code, to be appended as fields to the API response
add units as additional fields to each dynamo db entry

Briefly discussed in App Sci meeting and thought adding in the code instead of the db would be preferred path

Implement by adding units as items to each database record to keep code clean. Can read units from shp.xml sidecar file.

Going to start work on adding units to the API response.


[Perf] Failed to reproduce the performance result for Single-node AllReduce mentioned in README.md

Hi,
I am trying to compare mscclpp and nccl on the allreduce preformance. I used the following script to get the performance metrics for mscclpp and nccl respectively.
For mscclpp:
log_dir="logs"
mkdir -p ${log_dir}

for((k=0;k<8;k++));
do
    maxBytes=4g
    if ((k==2))
    then
        maxBytes=2g
    fi
    mpirun --bind-to numa -np 8 ./build/test/mscclpp-test/allreduce_test_perf \
        -b 3m \
        -e ${maxBytes} \
        -G 10 \
        -n 30 \
        -w 10 \
        -f 2 \
        -k ${k} \
        -o "${log_dir}/ar.txt"
done

For nccl using nccl-tests from Nvidia:
mpirun -np 8 ./build/all_reduce_perf \
    -b 3M \
    -e 4G \
    -G 10 \
    -f 2 \
    -d int32 \
    -R 0

What mscclpp got is:

As showed in the above figure, mscclpp's maximum algBw is underneath 100GB/s while it's supposed to be around 140GB/s as showed in README.md.
What nccl got is:

The achieved maximum algBw matches README.md roughly.
My nccl version is libnccl.so.2.20.5 and all tests are carried out on a machine with 8xH800.
As we can see, nccl's performance is much better than mscclpp especially when size is big. I suspect mscclpp may not work in its best setting. It would be helpful to reproduce the performance declared in README.md if you could share the test script and the environment configuration which your tests were carried out against.

Could you try with the python benchmark? https://microsoft.github.io/mscclpp/getting-started/quickstart.html#performance-benchmark.  The mscclpp-test maybe not suitable for your case.
Also seems you enabled nvlink-sharp for nccl-test but run mscclpp-test without nvlink-sharp. Test with python benchmark will enable nvlink sharp for both scenarios.
Besides, for A100, the nvlink speed is 600GB/s, for H800 is 400GB/s. It makes sense for A100 machine is faster than H800

Thank you for your swift reply. I'll give python benchmark a try and share the result with you later on.

@Binyang2014 My kernel version is 4.18.0, older than 5.6, so I encountered some problems when compiling mscclpp python benchmark with nvls support. Updating OS would take some time. Instead I turned nvls off for nccl to see what would happen. The following is what I got. The result now starts to make sense and aligns with your hypothesis.

Please make sure nvidia_peermem driver is running on your machine. https://github.com/microsoft/mscclpp/blob/main/docs/quickstart.md#prerequisites

Oh, I missed that you are using H800 GPUs. Then your numbers already look making sense.


Feature/668 php8 compatibility

Q
A

Branch
7.1.x

Bug fix?
no

New feature?
no

BC breaks?
potentially (see below)

Deprecations?
no

Tests pass?
yes

Fixed issues
chameleon-system/chameleon-system#668

License
MIT

Changes for supporting PHP8 in chameleon. Most of the code changes are PHPUnit updates.
Potential backwards compatibility changes
There is one change in actual code, that is important here:
Before:
    public static function StripTextWordSave($length = null, $sText, $sAddToText = '')

After:
    public static function StripTextWordSave($length, $sText, $sAddToText = '')

declaring optional arguments before non-optional ones is not valid anymore in PHP8. However, I don't quite know if this changes behaviour in PHP7.x.

I can not see how the change in TTools causes problems in 7.4. The only way to trigger the default would be to call the method with no parameters - which will trigger a warning anyway since the second parameter is not optional.


Declare constructors as static members of base Hooker constructor

grunt output
Running "default" task

Running "lint:files" (lint) task
Lint free.

Running "test:files" (test) task
hooker_test.js: ......................

OK: 75 assertions (15ms)

Running "concat" task

Running "concat:dist/ba-hooker.js" (concat) task
File "dist/ba-hooker.js" created.

Running "min" task

Running "min:dist/ba-hooker.min.js" (min) task
File "dist/ba-hooker.min.js" created.
Uncompressed size: 6505 bytes.
Compressed size: 707 bytes gzipped (1397 bytes minified).

Done, without errors.

If these methods weren't private, it would be a great idea to namespace them. And while I don't particularly care about a few bytes here or there, this change would add a whole bunch of them. While this is a really great example of grunt output in a pull request, I'm going to close this.

Nice! Sorry I forgot to close this after our discussion.


The ActionCable connection is not open! this.isActionCableConnectionOpen() must return true before calling this.stimulate()

Bug Report
Describe the bug
In production, clicking on a reflex gives the error:
The ActionCable connection is not open! `this.isActionCableConnectionOpen()` must return true before calling `this.stimulate()`
To Reproduce
I was thinking the error was in ActionCable, but now I'm not sure where the error is occurring.
Using the Hello, Reflex example at https://docs.stimulusreflex.com/quickstart, this is working in development, but not in production.
Expected behavior
In development, clicking on "increment" is working to increment the counter. In production, I get the error.
Trying to log the connection ...
module ApplicationCable
  class Connection < ActionCable::Connection::Base
    def connect
      Rails.logger.level = Logger::DEBUG
      Rails.logger.info "Connected to ActionCable!"
    end
  end
end

I'm getting "Connected to ActionCable!" in my production log, so I assume the cable is connected.
Screenshots or reproduction
Live example is here:
http://supervise.casehandler.com
Versions
StimulusReflex

Gem: 3.1.4
Node package: 3.1.4

External tools

Ruby: 2.7.0

Rails: 6.2.2

Node: 14.1.0

Passenger: 6.0.4

Nginx

Browser

Browser Firefox Developer Edition
Version 77.0b1

This is what I'm seeing when I visit the website.

So it seems like the actioncable in some way is not entirely correctly set up.

Thanks, this doesn't surprise me. First time using ActionCable ...

Hi @futura! Let's get you sorted. Fortunately or unfortunately - depending on your perspective - the fact that it's working does point to a configuration issue with how Passenger is (or isn't) upgrading connections to websockets.
When I look at the websockets connections tab in the Network inspector, I see that it opens a new cable connection every 5-10 seconds after clicking the link. Each attempt shows the connection sending a message to the server but not receiving any reply. If you try this in development, you'll see the server respond here.
Can you try a really basic ActionCable example and see if it works? Like you said, we need to narrow down where this is happening. I suspect that Passenger is causing us some grief.
Are you seeing action cable errors in your production log? Try temporarily setting your production.rb: config.log_level = :warn

I changed the log level and Nginx error log still is empty.
I'm wondering if it's related to this:
https://stackoverflow.com/questions/47807569/actioncable-in-production-repeated-101-responses
In development, the cable only opens once and persists. In production, I get 101 after 101 and occasionally maybe 502. I'm using a subdomain, so maybe that's an issue?
I have this in routes.rb:
  # Serve websocket cable requests in-process
  mount ActionCable.server => '/cable'

Virtual host config:
server {
  listen 80;
  server_name supervise.casehandler.com;

  # Tell Nginx and Passenger where your app’s ‘public’ directory is
  root /home/dwayne/apps/supervise/public;

  access_log /home/dwayne/apps/supervise/log/nginx.access.log;
  error_log /home/dwayne/apps/supervise/log/nginx.error.log warn;

  # Turn on Passenger
  passenger_enabled on;
  passenger_ruby /usr/local/rvm/gems/ruby-2.7.0@supervise/wrappers/ruby;

  # ActionCable
  location /cable {
    passenger_app_group_name supervise_action_cable;
    passenger_force_max_concurrent_requests_per_process 0;
  }

}

In environments/production.rb:
  config.action_cable.url = ['wss://supervise.casehandler.com/cable', 'ws://supervise.casehandler.com/cable']
  config.action_cable.allowed_request_origins = [
    'http://supervise.casehandler.com'
  ]

Sorry, it's looking like this isn't a StimulusReflex issue, but thanks for your help. If anything comes to mind, I'd love to hear it.
I'm loving StimulusReflex so far.

Solved. The one thing I didn't share here was config/cable.yml. My redis url was wrong.

That’s great news! Nice catch.

Try
development:
  adapter: redis
  url: <%= ENV.fetch("REDIS_URL") { "redis://localhost:6379/0" } %>

test:
  adapter: test

production:
  adapter: redis
  url: <%= ENV.fetch("REDIS_URL") { "redis://localhost:6379/0" } %>

Thanks for the fast response, I'm very new to all this, would I need a redis server running? If so can you share a guide for ubuntu 20.04, I'm using mina for deployment and could probably add a command in my deploy.rb to start the redis server.
  deploy do
    # Put things that will set up an empty directory into a fully set-up
    # instance of your project.
    invoke :'git:clone'
    invoke :'deploy:link_shared_paths'
    invoke :'bundle:install'
    invoke :'rails:db_migrate'
    invoke :'rails:assets_precompile'
    on :launch do
      command "sudo systemctl restart #{fetch(:user)}"
    end
  end


mysql >= 5.7.6 breaks (at least on CentOS)

mysql-community-server 5.7.6 is missing mysqld_safe (at least in the CentOS package) causing a 'No such file or directory' error.
Reproduce (tested on CentOS 7)
mysql_service 'default' do
  port '3306'
  version '5.7'
  initial_root_password 'very secret password'
  action [:create, :start]
end

mysql_config 'default' do
  instance 'default'
  source 'default.conf.erb'
  version '5.7'
  action :create
	notifies :restart, 'mysql_service[default]'
end

Workaround

Use version 5.6
Modify the mysql cookbook to stick to version 5.7.5 (this cannot be done by simply passing it in the LWRP)

Do you know if this is a bug in the packaging?
Or if they've removed the binary in purpose and now want people to initialize databases differently?

Looks like they've pulled the binaries...
https://bugs.mysql.com/bug.php?id=75848

PS: Does the package_version parameter on the mysql_service resource not work?

Missed that one, that probably works.

Fixed in 3ce278d0b5be851b060c58468f52e3ee9d00a9d9
Released as 6.0.17


ScreenSizeInFullscreen() panics on browsers

Ebitengine Version
v2.7.2
Operating System

[X] Windows
[ ] macOS
[ ] Linux
[ ] FreeBSD
[ ] OpenBSD
[ ] Android
[ ] iOS
[ ] Nintendo Switch
[ ] PlayStation 5
[ ] Xbox
[X] Web Browsers

Go Version (go version)
go1.22.2
What steps will reproduce the problem?
Compiling and executing the following program on the browser (e.g. $env:GOOS="js"; $env:GOARCH="wasm"; go build -o exec.wasm main.go):
package main

import "github.com/hajimehoshi/ebiten/v2"

type Game struct {}
func (*Game) Layout(w, h int) (int, int) { return w, h }
func (*Game) Update() error { return nil }
func (*Game) Draw(*ebiten.Image) {}

func main() {
	w, h := ebiten.ScreenSizeInFullscreen()
	_, _ = w, h
	err := ebiten.RunGame(&Game{})
	if err != nil { panic(err) }
}

I also tried with runtime.LockOSThread(), go1.18.10, both Firefox and Chrome, etc.; everything panics. Using Monitor().Size() works, though.

No panic.
What happens instead?
Panic.
Anything else you feel useful to add?
No response

What about Chrome?