Provide remote db name for ThaliReplicationPeerAction during runtime #1873

This change is

PR is added to the queue for testing as 1. task. (e186250)

Test 122449695 (e186250) build started.

Test (Success) 122449695 build is completed (e186250)
See https://github.com/ThaliTester/TestResults/tree/122449695e186250_Provide_remote_db_name_for_ThaliReplicationPeerAction_during_runtime__1873_mlesnic/ for the logs

Test 122449695e186250(e186250) has failed
See https://github.com/ThaliTester/TestResults/tree/122449695e186250_Provide_remote_db_name_for_ThaliReplicationPeerAction_during_runtime__1873_mlesnic/ for the fail logs

We really need new tests added for thaliReplicationPeerAction.start where we prove that we can replicate both with a default name and with an explicit name for a database that isn't the localDbName.

Reviewed 4 of 4 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Reviewed 4 of 4 files at r1.
Review status: all files reviewed at latest revision, 1 unresolved discussion, all commit checks successful.

thali/NextGeneration/replication/thaliReplicationPeerAction.js, line 193 at r1 (raw file):

 * when creating HTTP requests related to this action. Note that this is where
 * the PSK related settings are specified.
 * @param {string} remoteDbName The name of remote DB that we will try to replicate

You should state here, that this is optional parameter e.g: @param {string} [remoteDbName] An optional string specifying the name of the remote DB ...

Comments from Reviewable

PR is added to the queue for testing as 1. task. (31d3bc8)

Test 122449695 (31d3bc8) build started.

Review status: 2 of 6 files reviewed at latest revision, 1 unresolved discussion.

thali/NextGeneration/replication/thaliReplicationPeerAction.js, line 193 at r1 (raw file):
Previously, jareksl (Jarosław Słota) wrote…
You should state here, that this is optional parameter e.g: @param {string} [remoteDbName] An optional string specifying the name of the remote DB ...

Done.

Comments from Reviewable

Test (Success) 122449695 build is completed (31d3bc8)
See https://github.com/ThaliTester/TestResults/tree/12244969531d3bc8_Provide_remote_db_name_for_ThaliReplicationPeerAction_during_runtime__1873_mlesnic/ for the logs

Test<PHONE_NUMBER>1d3bc8(31d3bc8) has failed
See https://github.com/ThaliTester/TestResults/tree/12244969531d3bc8_Provide_remote_db_name_for_ThaliReplicationPeerAction_during_runtime__1873_mlesnic/ for the fail logs

Please answer the question about why the remoteDbName is on start and not on the constructor. Because if it's supposed to be on the constructor (and I would think it should be) then all the tests have to change.

Reviewed 2 of 4 files at r2.
Review status: 4 of 6 files reviewed at latest revision, 2 unresolved discussions.

thali/NextGeneration/replication/thaliReplicationPeerAction.js, line 193 at r2 (raw file):

 * when creating HTTP requests related to this action. Note that this is where
 * the PSK related settings are specified.
 * @param {string} [remoteDbName] Optional parameter that specifies name of remote

I'm confused. Why is remoteDbName here and not on the constructor?

Comments from Reviewable

Review status: 4 of 6 files reviewed at latest revision, 2 unresolved discussions.

thali/NextGeneration/replication/thaliReplicationPeerAction.js, line 193 at r2 (raw file):
Previously, yaronyg (Yaron Y Goland) wrote…
I'm confused. Why is remoteDbName here and not on the constructor?

The idea was that the customer of thali, with his own peer pool manager implementation, will be able to determine the remote db name, inside his peer pool manager. This allows customer to name his dbs in any way he wants to, for example, with some custom prefix or anything else. In peer pool manager, we can only manage already created PeerAction, we do not have control over creating it.
In case we wanted to pass remoteDbName in ThaliReplicationPeerAction constructor, we would have to strictly lock the way remote db name should be resolved, for example based on peer's public key.
In my opinion, we should be more flexible and allow thali customer to name his dbs in any way he likes to.

Comments from Reviewable

Review status: 4 of 6 files reviewed at latest revision, 2 unresolved discussions.

thali/NextGeneration/replication/thaliReplicationPeerAction.js, line 193 at r2 (raw file):
Previously, mlesnic (Marcin Lesniczek) wrote…
The idea was that the customer of thali, with his own peer pool manager implementation, will be able to determine the remote db name, inside his peer pool manager. This allows customer to name his dbs in any way he wants to, for example, with some custom prefix or anything else. In peer pool manager, we can only manage already created PeerAction, we do not have control over creating it.
In case we wanted to pass remoteDbName in ThaliReplicationPeerAction constructor, we would have to strictly lock the way remote db name should be resolved, for example based on peer's public key.
In my opinion, we should be more flexible and allow thali customer to name his dbs in any way he likes to.

Also, in that case, we can resolve remote db name during runtime. In case every user have db named in different way, if we wanted to pass this in constructor, we would have to literally restart thali every time new user occurs, with different remote db name parameter.

Comments from Reviewable

Reviewed 2 of 4 files at r2.
Review status: all files reviewed at latest revision, 3 unresolved discussions.

test/www/jxcore/bv_tests/testThaliReplicationPeerAction.js, line 274 at r2 (raw file):

  });

test('Make sure docs replicate with remote db with different name than local db',

We need just one more test. What happens when we have the wrong remote name? We need to make sure we fail gracefully.

Comments from Reviewable

PR is added to the queue for testing as 1. task. (60d457d)

Test 122449695 (60d457d) build started.

PR is added to the queue for testing as 2. task. (f36eb0f)

Test (Success) 122449695 build is completed (60d457d)
See https://github.com/ThaliTester/TestResults/tree/12244969560d457d_Provide_remote_db_name_for_ThaliReplicationPeerAction_during_runtime__1873_mlesnic/ for the logs

Test 122449695 (f36eb0f) build started.

Test<PHONE_NUMBER>0d457d(60d457d) has failed
See https://github.com/ThaliTester/TestResults/tree/12244969560d457d_Provide_remote_db_name_for_ThaliReplicationPeerAction_during_runtime__1873_mlesnic/ for the fail logs

Test (Success) 122449695 build is completed (f36eb0f)
See https://github.com/ThaliTester/TestResults/tree/122449695f36eb0f_Provide_remote_db_name_for_ThaliReplicationPeerAction_during_runtime__1873_mlesnic/ for the logs

Test 122449695f36eb0f(f36eb0f) has failed
See https://github.com/ThaliTester/TestResults/tree/122449695f36eb0f_Provide_remote_db_name_for_ThaliReplicationPeerAction_during_runtime__1873_mlesnic/ for the fail logs

Reviewed 1 of 1 files at r4.
Review status: all files reviewed at latest revision, 2 unresolved discussions.

thali/NextGeneration/replication/thaliReplicationPeerAction.js, line 193 at r2 (raw file):
Previously, mlesnic (Marcin Lesniczek) wrote…
Also, in that case, we can resolve remote db name during runtime. In case every user have db named in different way, if we wanted to pass this in constructor, we would have to literally restart thali every time new user occurs, with different remote db name parameter.

Yup. You're right and I'm wrong. The replication action is created by the Thali infrastructure in thaliPullReplicationFromNotification so if the peer policy is going to set the DB name then it has to happen in start.

Comments from Reviewable

Comments from Reviewable

Reviewed 3 of 4 files at r2, 1 of 1 files at r4.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

Comments from Reviewable


how to set up phantun on debian12 correctly?

vps:

homepc:

Could you describe what was the issue?


[KeyboardManager] Fix shift on numpad

Summary of the Pull Request
Fix using a numpad key as shift.

PR Checklist

[X] Closes: #22346
[X] Communication: I've discussed this with core contributors already. If work hasn't been agreed, this work might be rejected
[ ] Tests: Added/updated and all pass
[ ] Localization: All end user facing strings can be localized
[ ] Dev docs: Added/updated
[ ] New binaries: Added on the required places

[ ] JSON for signing for new binaries
[ ] WXS for installer for new binaries and localization folder
[ ] YML for CI pipeline for new test projects
[ ] YML for signed pipeline

[ ] Documentation updated: If checked, please file a pull request on our docs repo and link it here: #xxx

Detailed Description of the Pull Request / Additional comments
When receiving a numpad specific key, check if there is the numpad lock key with the same scan code that is overwriting shift and if it does, we overwrite the original key with the numpad key. I do this because when using for example numpad1 as shift we get the following scenario:

Press numpad1 -> we get in the Windows hook event key pressed for numpad1, then send shift
We get shift pressed, we ignore this event
We get shift release, we ignore this event
We get numpad end button instead of numpad1 release since shift was also pressed.

Validation Steps Performed
Replace numpad 1 with shift. Check that when releasing numpad1 shift gets toggled off.

@microsoft-github-policy-service agree company="Microsoft"

/azp run

/azp run


Show version number in template source

Describe the desired behavior
It would be nice to know which version of the template was used when creating a project with the CLI. Perhaps we can add a comment at the top of one of the main copied files that says the version so it's easy to identify during testing, e.g.:
/*
* This project was scaffolded with version x.x.x of the XYZ template
* */

Describe the current behavior
We can't tell what version of the template was used to build a project. We can get a sense if versions are different from each other by looking at the dependencies in package.json, but it's kind of a pain and not very exact.
Is this request related to a current issue?
Related to the testing going on for supporting alpha and beta versions of templates in the CLI.
Additional Context

It would be cool if we could programmatically pull this out of the package.json file so we don't have to remember to update it every time.

Instead of putting this at the top of a source file, we can inject it at the top of the README in the new project root folder:
This project was generated using the PX Blue CLI, template XXX...

If we prepend this information to the README, we can do the work in the CLI itself (one place) instead of duplicating the information across all of the templates.


TypeError: Object doesn't support property or method 'next' in IE browser

When we try to play video content in IE we are getting the below error.
Can someone help me to close this issue?

Can you share the code at line 438 in ima3.js, where it's presumably calling Array.from?
Even better, could you use the debugger or console.log and figure out exactly what arguments are being passed to it?

Hi @ljharb,
Thank you. I debugged the code as suggested and found the below line is causing the issue in ima3.js
bw = function () {
var a = (window.location.origin || "null").trim(), b = "null" === a, c = window == window.top, d = !1;
try {
d = null != window.top.location.hostname;
}
catch (f) {
}
var e = null != Array.from(document.body.getElementsByTagName("script")).find(function (f) {
return f.src && f.src.includes("connatix");
});
Ps.B().report(121, { location: Pv(), top_origin: Qv(), origin: a, is_null: b, is_top: c, is_friendly_to_top: d, is_vpaid_adapter: W.zb(), is_connatix_player: e });
};

Could you please help us in resolving this?
We have a production bug regarding this and our application has failed to launch.

Specifically it looks like Array.from(document.body.getElementsByTagName("script")); can you confirm that running that code in the console reproduces the same error?
Which version of IE is this?

can you confirm that running that code in the console reproduces the same error? - Yes, we are getting the same error when we use it in the console log.
Which version of IE is this? - IE11
Are you sure you're not loading any other polyfills, like babel-runtime/core-js? Yes, we are using the below dev dependency. So that it will automatically add babel-runtime/core-js.
Please help us with what has to do?
"@uirouter/angularjs": "^0.2.18",
"angular": "~1.5.0",
"angular-animate": "~1.5.0",
"angular-cookies": "~1.5.0",
"angular-messages": "~1.5.0",
"angular-resource": "~1.5.0",
"angular-route": "~1.5.0",
"angular-sanitize": "~1.5.0",
"angular-socialshare": "^2.3.11",
"angular-timer": "^1.3.4",
"angular-touch": "~1.5.0",
"es6-shim": "^0.35.1",
"humanize-duration": "^3.20.1",
"jquery": "^3.4.1",
"moment-timezone": "^0.5.11",
"ng-device-detector": "^3.0.1",
"re-tree": "^0.1.7",
"slick-carousel": "~1.4.1",
"underscore": "^1.9.1"

Can we add polyfill for this issue?
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/from#Polyfill

core-js and es6-shim are never intended to be used together; they do the same things. If you can’t avoid using babel-runtime, you’ll need to use a bundler to omit core-js if you want to use es6-shim.

The polyfills on MDN are not production quality and should never be used.

Can you please share any  helpful document or link to create bundler for omit core-js in angular

It depends on what bundler you're using; webpack and browserify each have their own docs and tutorials all over the web.
Typically only the top-level app would be pulling in core-js or es6-shim tho; are you sure your own code isn't doing it?


factorio-experimental: 1.1.74 -> 1.1.76

Description of changes
Changelog 1.1.76
Changelog 1.1.75
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Played a savegame created with version 1.1.74 for some minutes successfully.

May you drop the merge commit with an rebase?

I just squash merged :)

May you drop the merge commit with an rebase?

Yeah sorry I tried to figure this one out, got distracted by real-life.
@SuperSandro2000
Thanks for merging!


[Bug] [user management] User password is incorrectly modified

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
When configuring a user, nothing has been modified on the page, and the password has changed

配置用户时，页面上没有修改任何内容，密码就发生了改变
What you expected to happen
Password does not change
How to reproduce

Create a new user. According to the document, the default password of ordinary users is 123456, and the password field of the user table is stored as e10adc3949ba59abbe56e057f20f883e
Switch to the ordinary user and log in successfully with the default password
Switch back to the admin user, configure the user without modifying anything, and click Finish. At this time, the password of the user table changes, and the value is 14e1b600b1fd579f47433b88e8d85291
Log out of the admin user and log in with the ordinary user again, the password is 123456, the login fails; use e10adc3949ba59abbe56e057f20f883e as the password, the login is successful

新建一个用户，根据文档，普通用户默认密码为123456，user表password字段存储为e10adc3949ba59abbe56e057f20f883e
切换到该普通用户，使用默认密码登录成功
切换回admin用户，对该用户进行配置，没有修改任何内容，点击完成，此时user表password发生变化，值为14e1b600b1fd579f47433b88e8d85291
退出admin用户，再次使用该普通用户登录，密码为123456，登录失败；使用e10adc3949ba59abbe56e057f20f883e作为密码，登录成功

Anything else
No response
Version
0.6.0
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

This is a bug. The password is also updated when the user information is modified.

There is another problem in this place. No nickname is configured when creating a new ordinary user. After logging in with this ordinary user, the user is always loading in the upper right corner of the page, and there is no button to log out. I have to clear the cookie to log out.

这个地方还有一个问题，新建普通用户时不配置昵称，当使用这个普通用户登录后，页面右上角用户一直在加载的状态，同时没有退出登录的按钮，我不得不清空Cookie退出

There is another problem in this place. No nickname is configured when creating a new ordinary user. After logging in with this ordinary user, the user is always loading in the upper right corner of the page, and there is no button to log out. I have to clear the cookie to log out.
这个地方还有一个问题，新建普通用户时不配置昵称，当使用这个普通用户登录后，页面右上角用户一直在加载的状态，同时没有退出登录的按钮，我不得不清空Cookie退出

Ordinary users who configure nicknames do not have this problem

配置昵称的普通用户没有这个问题

I will fix it soon. Thank u.

thx.


running winget shows nothing and waits indefinitely

Brief description of your issue
running winget or winget --info in powershell does nothing, just an empty line with flickering cursor
Please help me what to do.. I've tried multiple commands to reinstall winget to no avail.
Steps to reproduce
I don't know,
Expected behavior
winget command works
Actual behavior
an empty line with flickering cursor is shown
Environment
Windows: Win 11 Pro, build 22631.2861

What version of winget are you using? (winget -v to find out)
This looks related to https://github.com/microsoft/winget-cli/issues/3652 where some older versions had their CDN retired.
Can you upgrade to the latest version of winget from https://aka.ms/getwinget  or upgrade the "App Installer" package from Microsoft Store and try again?

@mdanish-kh thanks for your reply and sorry for my late response, I haven't noticed earlier. As I wrote originally, running winget <anything> just stalls my console and does not render/do anything (for 10 minutes at least). Several things stopped working on my windows, unfortunately, so I will have to reinstall it probably (I cannot run any windows app, Store app is missing, etc.)


feat: Allow custom Laravel bootstrap file path

Changes
There where previous attempts (https://github.com/larastan/larastan/pull/851, https://github.com/larastan/larastan/pull/852, https://github.com/larastan/larastan/pull/857) to make the detection of the Laravel bootstrap file better. However, after reviewing the code and discussions it appears to me that the follow up PR never landed.
That said, there is no ultimate way to guess where people have their bootstrap file located. For that very reason this PR introduces a similar approach as Laravel uses here  for the storage path.
Added value:
This PR allows to either define the path statically in your environment
# ~/.bashrc or ~/.zshrc
export LARAVEL_BOOTSTRAP_FILE_PATH=Framework/bootstrap/app.php

or to prepend it to the command.
export LARAVEL_BOOTSTRAP_FILE_PATH=Framework/bootstrap/app.php; ./vendor/bin/phpstan analyse

Breaking changes
I chosed a currently unused var name. No B/C implications, imho.
Notes

Not sure where to document this. Keen to add a paragraph where ever you'd like to have it.

Thank you!
That command line is really
LARAVEL_BOOTSTRAP_FILE_PATH="Framework/bootstrap/app.php" ./vendor/bin/phpstan analyse

How about matching the variable name in the framework??
https://github.com/laravel/framework/blob/1a1a61068bc3c5594376a559e424ae09ec3fe64a/src/Illuminate/Foundation/Application.php#L253-L259
🤔

How about matching the variable name in the framework?? https://github.com/laravel/framework/blob/1a1a61068bc3c5594376a559e424ae09ec3fe64a/src/Illuminate/Foundation/Application.php#L253-L259 🤔

Pre-bootstrapping Laravel uses the LARAVEL_ prefix. I followed that convention.
https://github.com/laravel/framework/blob/11.x/src/Illuminate/Foundation/Application.php#L598
I believe this makes sense, because at this point the app does not yet exist – it's about to be bootstrapped.

You can already add your bootstrap file to the PHPStan's config. Does that not work?

You can already add your bootstrap file

There is a misunderstanding.
This is not PHPStan's "bootstrap" but a file for the Laravel framework.
https://github.com/laravel/laravel/blob/11.x/bootstrap/app.php

Yeah, that's what I meant. If Larastan can't find bootstrap your app for some reason, you can write your own bootstrap file that boots up your app. And that file to the config.

If you include bootstrap/app.php through PHPStan's "bootstrap" our bootstrap.php (modified in this PR) will throw an exception.

If you include a custom bootstrap/app.php through PHPStan's "bootstrap" our bootstrap.php (modified in this PR) will throw an exception.

This is correct. I didn't find a way around, even it was mentioned as an option in one of the discussions linked above.

Yeah, that's what I meant. If Larastan can't bootstrap your app for some reason, you can write your own bootstrap file that boots up your app. And that file to the config.

Following up here just in case it's not really clear what the problem is. Larastan – via it's vendor/larastan/larastan/extension.neon loads vendor/larastan/larastan/bootstrap.php. Even if I add
parameters:
	bootstrapFiles:
		- Framework/bootstrap/app.php

to my main phpstan.neon, Larastans extension.neon loads the vendor/larastan/larastan/bootstrap.php which will throw new Exception('Could not find Laravel bootstrap file nor Testbench is installed. Install orchestra/testbench if analyzing a package.'); because non of the defined bootstrap paths are valid in our application.
There is two solutions I can think of:

The one I proposed with this PR.
Modify vendor/larastan/larastan/bootstrap.php to something like the following, to no longer throw an exception if the bootstrap file wasn't found and allow the applications phpstan.neon to do it's own bootstrapping after.

if (file_exists($applicationPath = getcwd().'/bootstrap/app.php')) { // Applications and Local Dev
    $app = require $applicationPath;
} elseif (file_exists($applicationPath = dirname(__DIR__, 3).'/bootstrap/app.php')) { // Relative path from default vendor dir
    $app = require $applicationPath;
} elseif (trait_exists(CreatesApplication::class)) { // Packages
    $app = ApplicationResolver::resolve();
}

if(isset($app)) {
    if ($app instanceof Application) {
        $app->make(Kernel::class)->bootstrap();
    } elseif ($app instanceof LumenApplication) {
        $app->boot();
    }

    if (! defined('LARAVEL_VERSION')) {
        define('LARAVEL_VERSION', $app->version());
    }
}

If there is anything I can do to bring this forward, please let me know. 🙏

There is two solutions I can think of:
1. The one I proposed with this PR.

2. Modify `vendor/larastan/larastan/bootstrap.php` to something like the following, to no longer throw an exception if the bootstrap file wasn't found and allow the applications `phpstan.neon` to do it's own bootstrapping after.

Something to consider from a user's perspective:
Netbeans expects all options to be in the config file.
I don't know whether other IDEs will allow for prepending arguments to the CLI or not.
This PR allows defining the path in the environment, which is a step forward, but following PHPStan's options IMO is prefereable.
(In the meantime I'm investigating whether I need to install orchestra/testbench or whether manually patching the second option above will work for me.)

Something to consider from a user's perspective:
Netbeans expects all options to be in the config file.
I don't know whether other IDEs will allow for prepending arguments to the CLI or not.
This PR allows defining the path in the environment, which is a step forward, but following PHPStan's options IMO is prefereable.

Unfortunately, I don't know how Netbeans handles things and I cannot add any valuable answer here.
But I am happy to modify my PR to the second approach I proposed, if the maintainers would like me to.

(In the meantime I'm investigating whether I need to install orchestral/testbench or whether manually patching the second option above will work for me.)

My temporary solution looks something like the below. On each auto-load dump delete the bootstrap file in the vendor folder; then symlink the actual bootstrap file.
Not beautiful, and hopefully really only a temporary solution, but it works.
composer.json
    "scripts": {
        "pre-autoload-dump": [
            "[ $COMPOSER_DEV_MODE -eq 0 ] || rm -f vendor/larastan/larastan/bootstrap.php && ln -s ./path/to/actual/bootstrap.php"
        ],

Maybe this helps.

Fixed in https://github.com/larastan/larastan/pull/2077


Add code satchel to apache dependencies

If we're going to be using .wsgi files in the repo, those should be deployed before site configs get reloaded (this caused some problems today while debugging staging downtime)

The "code" satchel is an external dependency not in burlap, so it wouldn't be appropriate to add it here. The dependency should be added the other way, in the code satchel. I've never had to do that before, but it would be something like:
apache = get_satchel('apache')
apache.configure.deploy_before.append('code')


[Bug]: Eufy Indoor Cam 2k alarm state issue

What happened?
Hi, I have both Homebase 2 and Eufy Indoor Cam 2k. When I change alarm state for example from Away to Home in HomeKit, I see that alarm state of Indoor Cam doesn't change, it changes only after I manually open Eufy App and check state there. After this state of Indoor Cam in HomeKit changes immediately. With Homebase 2 I don't have this problem, it's alarm state changes at once. Any ideas? Thx
Device Type
Indoor Camera (Supported)
Plugin Version
beta (Best Effort)
HomeBridge Version
v1.4.x (Supported)
NodeJS Version
v16 (Supported)
Operating System
Raspbian 10+ (Supported)
Relevant log output
No response

As discussed in #39

Yes it occurs every time, but on earlier betas it was ok. When I arm it doesn't show it's armed at all, after I open Eufy app and refresh, it becomes armed. And when I disarm it hangs in arming state. Honestly I'm not sure if it hangs indefinitely, maybe a couple of times it was disarmed by itself, maybe after few minutes or so.

@njuart Can you specify on which version of the plugin it was working?
Also can you supply log output with debug enabled? Your config without the personal information would also be helpful. Thanks!

As discussed in #39

Yes it occurs every time, but on earlier betas it was ok. When I arm it doesn't show it's armed at all, after I open Eufy app and refresh, it becomes armed. And when I disarm it hangs in arming state. Honestly I'm not sure if it hangs indefinitely, maybe a couple of times it was disarmed by itself, maybe after few minutes or so.

@njuart Can you specify on which version of the plugin it was working? Also can you supply log output with debug enabled? Your config without the personal information would also be helpful. Thanks!

Sorry, but not sure I can recall on which beta exactly it was working. Here's what log shows:

I've armed Camera to Away from Home app
Saw infinite arming state
(Red arrow pointer) It's when I've opened Eufy app and refreshed Security tab
After I got back to Home app state was set to Away

Hello, I've the same issue on my setup. When arming we send a request to arm to the camera then we wait to receive the arming notification which can happend minutes after the ask. I don't see anything we can implement in order to force it. This issue is only related to camera operating their own alarm system. I don't have this issue with the homebase.

Hello, I've the same issue on my setup. When arming we send a request to arm to the camera then we wait to receive the arming notification which can happend minutes after the ask. I don't see anything we can implement in order to force it. This issue is only related to camera operating their own alarm system. I don't have this issue with the homebase.

Yes, with Homebase it's fine. But I'm pretty sure it was working fine some time ago.

Yes, with Homebase it's fine. But I'm pretty sure it was working fine some time ago.

Well since I don't own a solo camera I have difficulties to help here.
From looking through the code there is not much going on. We're doing exactly one thing and that is to give the command to the eufy-security-client package and then wait. I don't believe that this was changed much in the earlier versions but didn't have a closer look.
Honestly there are a myriad of things that could've changed that could cause your issue (software updates from eufy, maybe an update in the eufy-security-client dependency, ...) but as mentioned I don't believe that the cause is in this plugin.
You can revert to earlier version of the plugin and test if it has another behavior. Maybe then I can find a difference in the code bases.

I've added a log message that should be triggered if the mode change isn't done.
It's part of 2.0.1-beta.2. Until it's published you could download it directly from the repo.
Then there will be exactly one retry. But as stated before in #39 and from the information given I don't believe that the retry will do any good.
Please report if this helps.

I've added a log message that should be triggered if the mode change isn't done.
It's part of 2.0.1-beta.2. Until it's published you could download it directly from the repo.
Then there will be exactly one retry. But as stated before in #39 and from the information given I don't believe that the retry will do any good.
Please report if this helps.

Didn't help unfortunately. There is a timeout =( Even after retry.

Didn't help unfortunately. There is a timeout =( Even after retry.

Yeah, I expected that really.
I suppose there is a power management setting in the eufy app. Could you test whether different settings have different results in the homebridge behavior?

Hmm, didn't find any in Eufy app. Maybe because my Indoor 2k is wired.

Ok, another try.
I've hacked a eufy-test-client application which shall help me debug some issues in the future. For now there are only two options to change guard mode.
You can find it here: https://github.com/thieren/eufy-test-client
It's just quick and dirty, so don't expect too much.
Can you try something with this please?

If you use the standard 'change guard mode' option, you should get the same result as with the plugin.

The alternative option is a clutch at a straw to get this working for you.

Please report back, if this does anything for you.

Wow, thx. Not sure if I'll be near computer today, so probably tmrw. Maybe @lenoxys can assist you also, as he reported the same issue for him a bit a above in this thread.

@thieren Hey, again. Luckily managed to do it today and got the log. So:

I've changed Camera guard mode to 0 (AWAY) via your client
After few minutes opened Home app, nothing changed there (as expected unfortunately)
Opened Eufy app, Camera guard mode there was already AWAY, which I suppose means that it was successfully changed via your client
After it opened Home app again and status there was also AWAY this time

Here's the log
eufylog_1654003655751.txt

Did you try the alternative change guard mode option also? According to the log you just used one.

Did you try the alternative change guard mode option also? According to the log you just used one.

I wanted to, but got disconnected, and for some reason I can't connect with your client again.

I wanted to, but got disconnected, and for some reason I can't connect with your client again.

As I stated: just a quick and dirty solution. Can you write to me on Discord? Maybe we can work on this. But I think this is out of scope for this issue.

Hey @njuart
did update my application to handle 2fa and captchas. You should now be able to login again I think.
Let me know if it helps.

Hey @njuart
did update my application to handle 2fa and captchas. You should now be able to login again I think.
Let me know if it helps.

I didn't update yet, but yesterday I've disabled 2FA to be able to login, first time it worked, but had no luck after. Will try a new version and come back. Thx!

@thieren Just tried new client, but I don't see where I should put 2FA code, even though I get it on my mail while initialization.

Confirmed with @njuart that this is not a plugin issue, but rather with eufy-security-client
Maybe @bropat can do something about this.
Closed.

Should be fixed with latest beta.


Large MatMuls use slow reference kernel on ARM

@Wovchena This is related to #406 but goes deeper, hence I decided to make this a new issue.
TL;DR

For greedy_causal_lm inference on arm, large matmuls (e.g. 1x2x4096:4096x4096 in query/key/value projection) fall back to using a slow reference matmul implementation (ref_any_bf16)
Smaller matmuls (e.g. 1x32x2x2:1x32x2x128 in dot-product attention) use a faster gemm_acl_f16 kernel, which indicates that ACL kernels through oneDNN are available but not being used for point (1).
On x86 most matmuls use brgemm_avx512_bf16 kernel resulting in much faster inference.

Question: Is there a heuristic in OpenVINO that causes large matmuls on arm to fall back to the reference implementation?
Details
Following #327, I added code in greedy_causal_lm.cpp to save profiling information after inference. The table below shows the kernels used on x86 vs arm and their runtime (in microseconds) from the first decoder block of meta-llama/Llama-2-7b-hf (HuggingFace, compressed to INT8_ASYM).

Operation
Shape
x86 kernel
x86 time (mcs)
arm kernel
arm time (mcs)

q_proj
1x2x4096:4096x4096
brgemm_avx512_bf16
584
ref_any_f16
44714

k_proj
1x2x4096:4096x4096
brgemm_avx512_bf16
699
ref_any_f16
39713

v_proj
1x2x4096:4096x4096
brgemm_avx512_bf16
5775
ref_any_f16
37816

o_proj
1x2x4096:4096x4096
brgemm_avx512_bf16
833
ref_any_f16
36469

mlp.gate_proj
1x2x4096:4096x11008
brgemm_avx512_bf16
1023
ref_any_f16
97537

mlp.up_proj
1x2x4096:4096x11008
brgemm_avx512_bf16
965
ref_any_f16
101677

mlp.down_proj
1x2x11008:11008x4096
brgemm_avx512_bf16
953
ref_any_f16
98749

classifier (logits)
1x2x4096:4096x32000
brgemm_avx512_bf16
1813
ref_any_f16
284708

For small matmuls however, I noticed gemm_acl_f16 being called on arm.

Operation
Shape
arm kernel
arm time (mcs)

A = q_proj * k_proj ^ T
1x32x2x128:1x32x2x128
gemm_acl_f16
226

A * v_proj
1x32x2x2:1x32x2x128
gemm_acl_f16
69

This indicates that ACL kernels are available during inference but not being used only for large matmuls. To check that this diversion is not coming from oneDNN, I compiled oneDNN with ACL with following version combinations and benchmarked the above matmul sizes with benchdnn. Results are shown below, which indicate that gemm:acl is used on arm for all of the above sizes when the two compile correctly.

oneDNN source
oneDNN version
ACL version
Kernel used on arm
Comments

oneapi-src/oneDNN
3.3.3
24.02.1
Build fails; error from ACL
References to ACL v24.02.1 were found in openvino docs.

oneapi-src/oneDNN
3.3.3
23.11
Build fails; error from ACL
N/A

oneapi-src/oneDNN
3.3.3
23.08
gemm:acl
Expected kernel is used.

openvinotoolkit/oneDNN
3.3.3
24.02.1
gemm:jit_f32
ACL compilation flag ignored by cmake

openvinotoolkit/oneDNN
3.3.3
23.11
gemm:jit_f32
ACL compilation flag ignored by cmake

openvinotoolkit/oneDNN
3.3.3
23.08
gemm:jit_f32
ACL compilation flag ignored by cmake

I think ACL kernels should ideally be used for any matmul on arm since they are available and being used for some operations. Since oneDNN is not the one causing a reference kernel to be used, I am led to believe that a heuristic within OpenVINO is causing this fallback. I'm hoping to know whether that is indeed the case, and (if yes) where this heuristic has been implemented.

Hi @NishantPrabhuFujitsu
Thank you for submitting the issue, I'm reviewing it.
What ARM did you use to reproduce the issue? Is it Apple M1/M2/M3 or something else?

@alvoron Thanks for taking it up. The ARM CPU I'm using are on Graviton3 instances on AWS.

Hi @alvoron. Have you gotten any insights on this yet?

@NishantPrabhuFujitsu yes, I have some ideas what goes wrong.
Will you be able to check a custom OpenVINO build if I provide you OpenVINO wheel package?

@NishantPrabhuFujitsu
Since you're using C++ API you need C++ distribution of OpenVINO.
I've prepared a branch with needed fix - https://github.com/alvoron/openvino/tree/alvoron_acl_master
Could you please build OpenVINO on this branch?
OpenVINO build procedure (you don't need additional build options, just the first 4 steps): https://github.com/openvinotoolkit/openvino/blob/master/docs/dev/build_linux.md
And then 1 installation command: https://github.com/openvinotoolkit/openvino/blob/master/docs/dev/installing.md

Hi @alvoron.
The fix worked! Thank you so much for your support. The execution times for MatMul (and FullyConnected) layers are much better now. I found out from the conversation here that this was an ACL bug. If I may ask, what was the bug, and what was done to fix it?
Thanks again for helping out.

Marking as closed since the issue has been fixed.

@NishantPrabhuFujitsu
The issue was related to ACL multi-isa feature: ACL could be built using one ISA (e.g. without FP16 support), however this build could be run on a machine with another ISA (with FP16 support) and FP16 kernels should be available. The issue is some FP16 kernels are not compiled on ISA without FP16.
It's already fixed in ACL developer repository, however, there is no ACL release with this fix. My branch uses ACL Release Candidate that contains needed changes.
Please feel free to contact me if you face any issues or have any requests/suggestions on ARM functionality/performance.

@alvoron Thanks for letting me know. I'll be sure to get in touch if I need more help with running this on ARM.
Just one more request: would it be possible for you to send a .whl file for python 3.11.0 with your changes? I need to perform some benchmarking through the Python API as well. It's also okay if you tell me how I can build it locally.

@NishantPrabhuFujitsu
You can build wheel locally. You should follow the same build procedure you used before to build C++ package.
The only difference is to add -DENABLE_PYTHON=ON -DENABLE_WHEEL=ON to cmake configure command.
You find whl file in <INSTALL_DIR>/tools directory.
Please let me know if you face any issues, I'll help.

@NishantPrabhuFujitsu

Definitely we don't expect to see significant performance results between Python and C++. I'd suggest you to create a dedicated issue for that, we need to analyse this case.
OpenVINO on ARM supports fp16 and fp32 natively. bf16 is not supported, I need to review why bf16 is mentioned in the response. Is there any reason you'd prefer bf16 rather than fp16? i8 is not supported as well, however we're already working on i8 support.
We expect all kernels are taken from ACL or oneDNN. I'll double check with colleagues what component brings OpenBLAS dependency. This method consumes only 0.02% - it seems to me it's not kernel execution.

@alvoron Hope you had a good weekend.

I discovered something regarding the speed difference between Python and C++ inference (code linked below point to scripts in my fork of your repo).

Language
Implementation overview
Link to code/script
Throughput (tokens/sec)

C++
Ported and built greedy_causal_lm.cpp with necessary changes from openvino.genai into samples/
link
6.47

Python
Using OVModelForCausalLM from HuggingFace optimum.intel
link
0.07

Python
Using infer_request API with asynchronous inference from openvino package directly
link
7.70

It appears using infer_request.start_async() for inference delivers the best performance on ARM so far (comparable to that on x86). Perhaps optimum uses synchronous/non-threaded inference making it slow? Let me know if you have any ideas. Secondly, python inference is now (somewhat) faster than C++; any ideas on that? I ran the experiment many times and the third implementation mentioned above was the fastest every time. In any case, I don't think I should open a new issue for this since the problem doesn't seem to have to do with openvino now.

I've experienced better model output quality (accuracy?) with bf16 precision compared to fp16 in the past, hence it has higher preference vs fp16. Also, great to know that int8 support is on the way.
Please find the entire perf report here: t5_perf.txt

@eaidova could you please comment optimum behaviour, that @NishantPrabhuFujitsu highlighted above?

@NishantPrabhuFujitsu Let me try to help with your questions:
1.1 I would not expect any perf issues with optimum.intel which should have more or less the same OV calls underneath as in your third example. Have you checked your python env? Maybe it is still old OV version is used (w/o the fix from @alvoron)?
1.2 I would not expect C++ API to be slower than Python. I guess we can look on that.
As a side note I would recommend to take a look on our LLM bench which helps to evaluate performance in different scenarious (greedy/beam_search, different prompt or output size, separate timing for prompt processing and generate stages etc): https://github.com/openvinotoolkit/openvino.genai/tree/master/llm_bench/python
Additionally we are finalizing C++ Generate API for genai, might be interesting for you: https://github.com/openvinotoolkit/openvino.genai/pull/334

Based on our experience some models shows better accuracy with fp16 and some with bf16, so it depends. The tricky thing is performace. AFAIK ACL has limited BF16 support (as well as HW itself), so bf16 inference might not be as performant as fp16. Anyway this is smt we can consider as an option for the future development.

Based on the log sgemm is called as part of torch inference, not OpenVINO. Pytorch infenrece is used for tokenizer model and as part of beam search algo inside generate call.

@dmitry-gorokhov @alvoron Thanks for your response.

Regarding performance related points:

The poor throughput from optimum is most likely due to the right openvino package not being installed, like you pointed out. I ran several experiments today but I can't derive any conclusions right now due to large variations in performance I'm observing for the other two experiments (compared to yesterday). This variation is due to issues with the instances I'm using for my experiments, not with OpenVINO. I'll try and resolve them at my end and come back to you with some reliable data soon.
Between C++ and Python API, I noticed that the python script (third experiment in my previous message) was slightly faster than C++ even on x86. For example, in one experiment the C++ script yielded 13.40 tokens/sec while the Python script yielded 14.02 tokens/sec. Regardless, I will take these readings again once I resolve the issue I mentioned above.
I tried out llm_bench on ARM with Llama2, and got roughly the same performance numbers as I did from my script. The output of llm_bench is much more informative however, so I'll probably continue using that going forward 😄 .
#334 -> very useful; eagerly awaited.

I discussed with some of my colleagues, and they found it interesting that FP16 and BF16 had comparable accuracy as per your findings. We've almost always had BF16 give better accuracy, across many architectures (not limited to LLMs) and tasks. However, I can understand that there could be situations where this is not the case. It's fine to take this up at a later point; not very urgent.

Thanks for clarifying.

@alvoron @dmitry-gorokhov I hope you're doing well.

I spent some time correcting my benchmarking procedure, and have the following statistics summarized from ~10 runs. Below, "Precision" refers to inference precision specified using the inference precision hint.

Based on the above, I can say the following.

The python implementation is about as fast as the C++ one, but the C++ is slightly faster on an average as expected. So my point about Python being faster than C++ mentioned earlier stands corrected.
About optimum: I am fairly confident there's an issue with inference using optimum. In several experiments where I ensured that the right openvino whl file is installed (on ARM), I get much poorer performance using optimum than with C++ or python using openvino directly (see benchmark below). Perhaps this could be investigated further?

arm (Graviton3E)
x86 (Sapphire Rapids)

Throughput (tokens/sec)
1.8423
8.3086

When you mentioned earlier that int8 inference is being worked on, was that only for arm? I noticed that on x86 (Sapphire Rapids) not specifying any inference precision hint or using inference precision FP32 with DNNL_DEFAULT_FPMATH_MODE=BF16 generates oneDNN logs that show weight tags with "u8" precision:

onednn_verbose,primitive,exec,cpu,inner_product,brgemm:avx512_core,forward_inference,src_f32::blocked:ab::f0 wei_u8:a:blocked:AB16b64a::f0 bia_undef::undef::: dst_f32::blocked:ab::f0,attr-scratchpad:user attr-fpmath:bf16 attr-scales:wei:1 attr-zero-points:wei:1 ,,mb1ic4096oc4096,0.154053
onednn_verbose,primitive,exec,cpu,inner_product,brgemm:avx512_core,forward_inference,src_f32::blocked:ab::f0 wei_u8:a:blocked:AB16b64a::f0 bia_undef::undef::: dst_f32::blocked:ab::f0,attr-scratchpad:user attr-fpmath:bf16 attr-scales:wei:1 attr-zero-points:wei:1 ,,mb1ic4096oc11008,0.419189
onednn_verbose,primitive,exec,cpu,inner_product,brgemm:avx512_core,forward_inference,src_f32::blocked:ab::f0 wei_u8:a:blocked:AB16b64a::f0 bia_undef::undef::: dst_f32::blocked:ab::f0,attr-scratchpad:user attr-fpmath:bf16 attr-scales:wei:1 attr-zero-points:wei:1 attr-post-ops:eltwise_swish:1 ,,mb1ic4096oc11008,0.422852
onednn_verbose,primitive,exec,cpu,inner_product,brgemm:avx512_core,forward_inference,src_f32::blocked:ab::f0 wei_u8:a:blocked:AB16b64a::f0 bia_undef::undef::: dst_f32::blocked:ab::f0,attr-scratchpad:user attr-fpmath:bf16 attr-scales:wei:1 attr-zero-points:wei:1 ,,mb1ic11008oc4096,0.39209
onednn_verbose,primitive,exec,cpu,inner_product,brgemm:avx512_core,forward_inference,src_f32::blocked:ab::f0 wei_u8:a:blocked:AB16b64a::f0 bia_undef::undef::: dst_f32::blocked:ab::f0,attr-scratchpad:user attr-fpmath:bf16 attr-scales:wei:1 attr-zero-points:wei:1 ,,mb1ic4096oc4096,0.155029
onednn_verbose,primitive,exec,cpu,inner_product,brgemm:avx512_core,forward_inference,src_f32::blocked:ab::f0 wei_u8:a:blocked:AB16b64a::f0 bia_undef::undef::: dst_f32::blocked:ab::f0,attr-scratchpad:user attr-fpmath:bf16 attr-scales:wei:1 attr-zero-points:wei:1 ,,mb1ic4096oc4096,0.147949
onednn_verbose,primitive,exec,cpu,inner_product,brgemm:avx512_core,forward_inference,src_f32::blocked:ab::f0 wei_u8:a:blocked:AB16b64a::f0 bia_undef::undef::: dst_f32::blocked:ab::f0,attr-scratchpad:user attr-fpmath:bf16 attr-scales:wei:1 attr-zero-points:wei:1 ,,mb1ic4096oc4096,0.152832

Just wanted to check whether the above means that operations are being performed at int8 precision.

Do you have any idea how one can set the number of threads to use for inference for optimum? I looked through the documentation for optimum.intel and the code for quite some time but couldn't get this to work. I also have the same question about the correct way of setting inference precision.

How inference precision and number of threads were set
I'm including this to show how I set inference precision hints and thread count (in appropriate sections of the code). If this is not the right way, please point me to the correct method.
C++
ov::CompiledModel compiled_model = core.compile_model(
    std::string{argv[1]} + "/openvino_model.xml",
    "CPU",
    ov::inference_num_threads(nthr), // read and converted to uint from argv
    ov::hint::inference_precision(ov::element::f32) // or f16
);

Python
import openvino as ov
import openvino.properties as props
import openvino.properties.hint as hints

core = ov.Core()
config = {
    props.inference_num_threads: args.num_threads, # read from CLI args
    hints.inference_precision: "f32", # or f16
}

model = core.read_model(f"{MODEL_DIR}/openvino_model.xml")
compiled_model = core.compile_model(model=model, device_name="CPU", config=config)
lm = compiled_model.create_infer_request()


Unable to override BackgroundJobServerOptions when initialising Hangfire

Contact Details
No response
Version
8.x
On which operating system(s) are you experiencing the issue?
No response
Using which broker(s) did you encounter the issue?
No response
What are the steps required to reproduce the issue?
No Steps Required, code flexibility issue

What is the expected behavior?
Should allow changing BackgroundJobServerOptions for Hangfire server in host
What actually happened?
Find from code AddHangfireConsumer is calling Add HangfireBusObserver through service.AddBusObserver and using HangfireEndpointOption.
However, inside HangfireBusObserver, it doesn’t allow to override the default BackgroundJobServerOptions, I will need to change the SchedulePollInterval (of hangfire) to 1 second. Please refer issue #4171 that faced the exact same issue using NEW syntax (old syntax should be deprecated?)
Can you extend HangfireBusObserver to allow HangfireEndpointOption have a flexibility to self define BackgroundJobServerOptions for overriding the default one? Thanks
Related log output, including any exceptions
[16:44:52 INF] Using the following options for Hangfire Server:
Worker count: 20
Listening queues: 'mt-message-queue'
Shutdown timeout: 00:00:15
Schedule polling interval: 00:00:15

Link to repository that demonstrates/reproduces the issue
No response

Should align with old syntax for providing flexibility to override? (4171 have mentioned that)

The old syntax should be considered deprecated, I'm not sure what it would take to expose access to the options with it.

Do you think it is fine for adding a field for changing schedulepollinginterval when initialising the Hangfire consumer (if you don’t prefer to expose the whole object), or if ok to expose the whole thing, should I pull and work on that?

Sure expose the whole object.

Okay, so from what I'm seeing, you can already customize every one of the options with UseHangfireScheduler.
For instance:
configurator.UseHangfireScheduler(_queueName, cfg =>
{
    cfg.ServerCheckInterval = TimeSpan.FromSeconds(1);
    cfg.SchedulePollingInterval = TimeSpan.FromSeconds(1);
});


error running live stats

have tried -all and -all -recalc_all but continue getting these errors -
Udpating NFL Statistics (2017-09-22 00:43:03)
Reading NFL stats for Week 1
Got all games
--- 0.318994045258 seconds ---
KC (42) at NE (27) - Q: Final, 00:39
NYJ (12) at BUF (21) - Q: Final, 00:39
ATL (23) at CHI (17) - Q: Final, 00:03
BAL (20) at CIN (0) - Q: Final, 00:25
PIT (21) at CLE (18) - Q: Final, 00:36
ARI (23) at DET (35) - Q: Final, 00:18
JAX (29) at HOU (7) - Q: Final, 00:32
OAK (26) at TEN (16) - Q: Final, 00:07
PHI (30) at WAS (17) - Q: Final, 00:25
IND (9) at LA (46) - Q: Final, 00:00
SEA (9) at GB (17) - Q: Final, 00:39
CAR (23) at SF (3) - Q: Final, 00:25
NYG (3) at DAL (19) - Q: Final, 00:04
NO (19) at MIN (29) - Q: Final, 00:28
LAC (21) at DEN (24) - Q: Final, 00:01
Purged 0 nfl_statistic rows.
--- 5.89258694649 seconds ---
Calculating and storing fantasy values

******   6596 fantasy_statistics to calculate

Traceback (most recent call last):
File "livestats.py", line 654, in
main()
File "livestats.py", line 46, in main
update_games(year, week, weektype, args.all)
File "livestats.py", line 76, in update_games
update_fantasy_statistics(year, week, weektype)
File "livestats.py", line 432, in update_fantasy_statistics
scoring_def = get_scoring_def_dict(leagueid, year)
File "livestats.py", line 511, in get_scoring_def_dict
cur.execute(query)
File "/usr/lib64/python2.7/site-packages/MySQLdb/cursors.py", line 205, in execute
self.errorhandler(self, exc, value)
File "/usr/lib64/python2.7/site-packages/MySQLdb/connections.py", line 36, in defaulterrorhandler
raise errorclass, errorvalue
_mysql_exceptions.OperationalError: (1054, "Unknown column 'None' in 'where clause'")
when I try for week 2 it won't return game scores -
python livestats.py -week 2 -all -recalc_all
Udpating NFL Statistics (2017-09-22 00:46:35)
Reading NFL stats for Week 2
Traceback (most recent call last):
File "livestats.py", line 654, in
main()
File "livestats.py", line 46, in main
update_games(year, week, weektype, args.all)
File "livestats.py", line 67, in update_games
(live_changes_made, livegamecount) = update_nfl_statistics(year, week, weektype, update_all)
File "livestats.py", line 109, in update_nfl_statistics
games = nflgame.games(int(year), week=int(week), kind=weektype)
File "/usr/lib/python2.7/site-packages/nflgame/init.py", line 229, in games
return list(games_gen(year, week, home, away, kind, started))
TypeError: 'NoneType' object is not iterable

What do you see when you run livestats.py -hello ?

Year:2017, week: 3, weekrype: REG
On Thu, Sep 21, 2017 at 11:20 PM Nick Frerichs<EMAIL_ADDRESS>wrote:

What do you see when you run livestats.py -hello ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/nickfrerichs/fflproject/issues/28#issuecomment-331339844,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ARwi2xzWAlmR5arz7L6p5PhyOi3FInWmks5skyd4gaJpZM4PgF3-
.

Edit livestats.py, on line 510 (line before cur.execute(query)), add print query.  Then run it and let me know what the output looks like.

here is output for week 1
python livestats.py -week 1 -all -recalc_all
Udpating NFL Statistics (2017-09-22 12:23:48)
Reading NFL stats for Week 1
Got all games
--- 0.299476861954 seconds ---
KC (42) at NE (27) - Q: Final, 00:39
NYJ (12) at BUF (21) - Q: Final, 00:39
ATL (23) at CHI (17) - Q: Final, 00:03
BAL (20) at CIN (0) - Q: Final, 00:25
PIT (21) at CLE (18) - Q: Final, 00:36
ARI (23) at DET (35) - Q: Final, 00:18
JAX (29) at HOU (7) - Q: Final, 00:32
OAK (26) at TEN (16) - Q: Final, 00:07
PHI (30) at WAS (17) - Q: Final, 00:25
IND (9) at LA (46) - Q: Final, 00:00
SEA (9) at GB (17) - Q: Final, 00:39
CAR (23) at SF (3) - Q: Final, 00:25
NYG (3) at DAL (19) - Q: Final, 00:04
NO (19) at MIN (29) - Q: Final, 00:28
LAC (21) at DEN (24) - Q: Final, 00:01
Purged 0 nfl_statistic rows.
--- 5.74099588394 seconds ---
Calculating and storing fantasy values

******   6596 fantasy_statistics to calculate

select scoring_def.nfl_scoring_cat_id, per, points, round, is_range, range_start, range_end, nfl_position_id from scoring_def join nfl_scoring_cat on nfl_scoring_cat.id = scoring_def.nfl_scoring_cat_id where league_id = 1 and year = 2017
select scoring_def.nfl_scoring_cat_id, per, points, round, is_range, range_start, range_end, nfl_position_id from scoring_def join nfl_scoring_cat on nfl_scoring_cat.id = scoring_def.nfl_scoring_cat_id where league_id = 2 and year = None
Traceback (most recent call last):
File "livestats.py", line 654, in
main()
File "livestats.py", line 46, in main
update_games(year, week, weektype, args.all)
File "livestats.py", line 76, in update_games
update_fantasy_statistics(year, week, weektype)
File "livestats.py", line 432, in update_fantasy_statistics
scoring_def = get_scoring_def_dict(leagueid, year)
File "livestats.py", line 511, in get_scoring_def_dict
cur.execute(query)
File "/usr/lib64/python2.7/site-packages/MySQLdb/cursors.py", line 205, in execute
self.errorhandler(self, exc, value)
File "/usr/lib64/python2.7/site-packages/MySQLdb/connections.py", line 36, in defaulterrorhandler
raise errorclass, errorvalue
_mysql_exceptions.OperationalError: (1054, "Unknown column 'None' in 'where clause'")
This is output when trying week 2:
python livestats.py -week 2 -all -recalc_all
Udpating NFL Statistics (2017-09-22 12:24:43)
Reading NFL stats for Week 2
Traceback (most recent call last):
File "livestats.py", line 654, in
main()
File "livestats.py", line 46, in main
update_games(year, week, weektype, args.all)
File "livestats.py", line 67, in update_games
(live_changes_made, livegamecount) = update_nfl_statistics(year, week, weektype, update_all)
File "livestats.py", line 109, in update_nfl_statistics
games = nflgame.games(int(year), week=int(week), kind=weektype)
File "/usr/lib/python2.7/site-packages/nflgame/init.py", line 229, in games
return list(games_gen(year, week, home, away, kind, started))
TypeError: 'NoneType' object is not iterable

It looks like you have two leagues setup and one of them doesn't have any scoring definitions defined.  The easiest way for you to get up and running would be to add at least one scoring definition for the league that doesn't have any.
Longer term, I'll fix the bug by updating livestats.py to skip any leagues without scoring definitions.

thanks - appreciate it -
Brandon
On Fri, Sep 22, 2017 at 2:17 PM, Nick Frerichs<EMAIL_ADDRESS>wrote:

It looks like you have two leagues setup and one of them doesn't have any
scoring definitions defined. The easiest way for you to get up and running
would be to add at least one scoring definition for the league that doesn't
have any.
Longer term, I'll fix the bug by updating livestats.py to skip any leagues
without scoring definitions.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/nickfrerichs/fflproject/issues/28#issuecomment-331522814,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ARwi28VEqm3K8KdOufNPMJU2dWAIViYIks5sk_m8gaJpZM4PgF3-
.

This has been fixed and committed to the develop branch.


Fix hanging commits on dot etcd-backup-dot-cronjob

The following command hangs, and this is the reason why the cron jobs are also hanging on the etcd-backups.
dm switch etcd-backup-dot-cronjob
dm commit -m "blah"

dm logs seem to be spinning on trying to procure the filesystem for the dot
"[timestamp=2018-04-12T11:13:14Z] [instance_id=8764352245442910897] [pod_id=dotmesh-lzz7j] Error procuring filesystem: timed out trying to procure 47fef5a4-7788-45c7-7953-171f35e8ab2e, please try again, pausing and trying again...\n"
"[timestamp=2018-04-12T11:13:14Z] [instance_id=8764352245442910897] [pod_id=dotmesh-lzz7j] 2018/04/12 11:13:14 Attempting to move 47fef5a4-7788-45c7-7953-171f35e8ab2e from 806ed3b7173f9693 to me (2a1f51b88403779)\n"
"[timestamp=2018-04-12T11:13:14Z] [instance_id=8764352245442910897] [pod_id=dotmesh-lzz7j] 2018/04/12 11:13:14 globalFsRequest: setting '/dotmesh.io/filesystems/requests/47fef5a4-7788-45c7-7953-171f35e8ab2e/0c9b8eda-469f-4761-7044-fe72ceb35bc6' to '{\"Name\":\"move\",\"Args\":{\"target\":\"2a1f51b88403779\"}}'\n"
"[timestamp=2018-04-12T11:13:14Z] [instance_id=8764352245442910897] [pod_id=dotmesh-lzz7j] 2018/04/12 11:13:14 globalFsRequest 47fef5a4-7788-45c7-7953-171f35e8ab2e <Event move: target: 2a1f51b88403779>\n"
"[timestamp=2018-04-12T11:13:13Z] [instance_id=4557252902214069511] [pod_id=dotmesh-vkg6x] Error procuring filesystem: timed out trying to procure 47fef5a4-7788-45c7-7953-171f35e8ab2e, please try again, pausing and trying again...\n"
"[timestamp=2018-04-12T11:13:13Z] [instance_id=4557252902214069511] [pod_id=dotmesh-vkg6x] 2018/04/12 11:13:13 Attempting to move 47fef5a4-7788-45c7-7953-171f35e8ab2e from 806ed3b7173f9693 to me (198cdf7c12622838)\n"
"[timestamp=2018-04-12T11:13:13Z] [instance_id=4557252902214069511] [pod_id=dotmesh-vkg6x] 2018/04/12 11:13:13 globalFsRequest: setting '/dotmesh.io/filesystems/requests/47fef5a4-7788-45c7-7953-171f35e8ab2e/b8f682a5-bd45-41e9-4e6c-0b2c8e043cb1' to '{\"Name\":\"move\",\"Args\":{\"target\":\"198cdf7c12622838\"}}'\n"
"[timestamp=2018-04-12T11:13:13Z] [instance_id=4557252902214069511] [pod_id=dotmesh-vkg6x] 2018/04/12 11:13:13 globalFsRequest 47fef5a4-7788-45c7-7953-171f35e8ab2e <Event move: target: 198cdf7c12622838>\n"

possibly caused by https://github.com/dotmesh-io/ops/issues/55, if it's not happening right now then we've lost it, closing for now.


How do we involve/include Hack Duke with NC Open Pass?

Hack Duke is happening on October 28th and 29th it s a 24 hour hackathon, wondering if there is any opportunity to partner with them. Although the dates don't really work out, but may be a good opp for teams to attend and "sprint" on their projects.

Who has a contact with organizers?

This is a great idea.

Adam Martin was our contact last year.
On Wed, Sep 27, 2017 at 12:36 PM, aikimark<EMAIL_ADDRESS>wrote:

This is a great idea.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/codeforraleigh/NCOpenPass/issues/450#issuecomment-332580986,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQ6Ixs0cHRcUNNELFnWcxULjYvtQ418Xks5smnlwgaJpZM4OcJKE
.

--
Zach Ambrose
<EMAIL_ADDRESS>919-438-2752 (office)
919-616-9969 (cell)
www.AmbroseStrategy.com
https://www.linkedin.com/in/zambrose


Stream RTSP and/or MJPEG possible?

Is it possible to stream RTSP and/or MJPEG sources with XamarinMediaManager?
Thanks in advance
//Eddy

Depeding on the device you run it on, this is possible.


Is this module still needed or should we use individual sub-modules?

Details
Sorry if this question might seem a bit unnecessary, but I'm not understanding if this collection module is still needed/recommended, or if it is better to use its submodules like og-image, sitemap, robots, and so on.
On the new documentation website, each module seems to be explicitly referenced as being installed separately, and there are no instructions to setup the whole SEO package.
Is this a case of still work-in-progress docs, or the installation strategy is indeed different now?
Thank you for your patience:  :)

Hey @stefanobartoletti, thanks for the question. Sorry for the confusing roadmap with this module
There are plans for an official v2, but it requires all modules to be bumped a major. The big difference between v1 and v2 is that v1 only works when prerendering and the i18n support is limited, whereas this will be solved in v2.
While most the modules have this working, not all of them, so I still have 2 modules left to bump and a new module to release. However, I'll be pushing up some more beta versions in the next few days

Thank for your reply, I think I will stick to V1 and upgrade when everything will be ready.
And thank you for the awesome work behind the modules  too✌

Hey @harlan-zw I was curious the status of i18n?
I'm starting an extensively large project that uses i18n, different currencies, etc. I've never used nuxt before and there are just so many SEO modules here it is a bit confusing.
Some questions:

should we still use individual modules?
the experimental module seems to include og tags, but theres a separate og module for that, which is right?

Currently the ones I'm looking to use are:
    "nuxt-og-image": "^2.0.13",
    "nuxt-schema-org": "^2.2.0",
    "nuxt-simple-sitemap": "^3.1.5",

    'nuxt-og-image',
    'nuxt-schema-org',
    'nuxt-simple-sitemap',

along with nuxt i18n module.

You can use individual modules or check https://github.com/harlan-zw/nuxt-seo-kit/releases to see the status on nuxt-seo-kit v2.

There is an initial migration guide for v2, it's currently in beta but feel free to try it out.
https://nuxtseo.com/nuxt-seo/getting-started/migration-guide

@harlan-zw thanks!


Dollar sign prefix ($) for query parameters

Filtering on the /users endpoint didn't work for me so in investigated - I found this piece in the documentation:

On the beta endpoint, the $ prefix is optional. For example, instead of $filter, you can use filter. On the v1 endpoint, the $ prefix is optional for only a subset of APIs. For simplicity, always include $ if using the v1 endpoint.

(source https://docs.microsoft.com/en-us/graph/query-parameters)
Reproduce Issue
let client = Graph::new(&std::env::var("TOKEN").unwrap());
let response = client
    .v1()
    .users()
    .list_user()
    .filter(&["userType eq 'guest'"])
    .send()
    .unwrap();

println!("{:#?}", response)

Error (shortened)
thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: ....
{
  url: "https://graph.microsoft.com/v1.0/users?filter=userType+eq+%27guest%27",
  status: 400,
  error: ErrorStatus {
    code: Some("Request_BadRequest"),
    message: "Unrecognized query argument specified: 'filter'."
  ...
  }
}

Test with $filter
Manually setting $filter works and returns the results as expected
let client = Graph::new(&std::env::var("TOKEN").unwrap());
    let response = client
        .v1()
        .users()
        .list_user()
        .query("$filter", "userType eq 'guest'")
        .send()
        .unwrap();

    println!("{:#?}", response)

I'm not sure if this should be introduced as a breaking change just in case...

This is a great find. Not a big deal to change that for v1 endpoints only. Thanks for taking a look at this.


µTP Socket API

Would be nice to expose native µTP implementation, but initially it would likely be a lot easier to adapt pure JS implementation over UDP.

Feedback from @mafintosh

an added complexity here is hole punching which usually means being able to pass the underlying udp socket to the utp impl after it's been hole punched. I can share some more info here if needed.

https://gist.github.com/mafintosh/a67474e0e8b44901239c594b958e19d4 (the utp + hole punching stuff)

Taking this off the list as it can be implemented using UDPSocket API.


Add a field to indicate the languages in which a course is available.

Feature Request
Is your feature request related to a problem or unsupported use case? Please describe.
We need to be able to indicate, on the course syllabus, the languages in which the course content is available.
Describe the solution you'd like
I think this field should be on the course run object because each course run could be available in different languages. In particular, edX does not support multilingual course content, so the way to do multilingual courses could be to open several course runs, each one for a specific language.
Technically, the languages field could be a Django ArrayField of CharField with a list of ISO 639 languages as choices.
Discovery, Documentation, Adoption, Migration Strategy

[x] The languages field should be required to create a course run,
[x] We should add a field to select multiple languages on the CMS wizard form to create new course runs,
[x] The list of languages should be displayed and editable on the course pages and on the course run pages,
[x] ~Language should be a filter on the search page. It should be linked to the dates of the related course run so that when we filter by a language and a status, we only see the courses that have a course in this status for the selected language.~ The language filter on the search page should be made a facet.

Do you want to work on it through a Pull Request?
:+1:

This seems like a fine solution to me 👍.
We'll just have to be careful to return the "most correct" course run per language & display it in the "most correct" language available when we generate search results for courses with a language filter.

Hi, good point. Some observations.
1- I take for granted an equivalent search page like the edX one, see https://www.edx.org/course/?, and where we can search for course langage and other fields.
2- to be more precise, we should use internally only the ISO-639-1, the 2 letters code, and not mention the country specific locale. And as a parameter for Ritchie admin, we should be able to specify the preferred suggested langage (french, english, ...) in front of the complete langage list;

Hi @robertgerinlajoie!
Some elements of answer:

yes, that's what we have in mind. Contrary to the search engine you link, we will be able to filter on one but also on several languages,
Whether you use ISO 639-1 two-letter codes or country+language locales is up-to-you in Django. It depends on what you declare in the LANGUAGES setting. The preferred language in Django is also configurable via the LANGUAGE_CODE setting and could be either a two-letter language or a full locale if I'm not mistaken.

Please note that your CMS pages will be translated in some languages: maybe english and french for example. But you should still be able to declare a course run in other languages in which your CMS pages are not translated. So the language field on the course run should propose all the languages defined in ISO 639 and not just the few languages activated in your CMS. Are we in line on this as well?

We moved back to indexing courses instead of course runs. PR https://github.com/openfun/richie/pull/496 finalizes this issue as languages are now a facet and filters through a nested field via a painless script.


KMS controller - can't create an alias

Describe the bug
I want to create an alias for the key managed by ACK but it is impossible
Steps to reproduce
Create a KMS key with manifest like:
apiVersion: kms.services.k8s.aws/v1alpha1
kind: Key
metadata:
  name: rds-dbserver-8
  namespace: infra-production
spec:
  description: "Key for rds-dbserver-8"
  enableKeyRotation: true
  tags:
    - tagKey: Name
      tagValue: rds-dbserver-8
    - tagKey: env
      tagValue: production

Then apply manifest for the alias:
apiVersion: kms.services.k8s.aws/v1alpha1
kind: Alias
metadata:
  name: alias-rds-dbserver-8
  namespace: infra-production
spec:
  name: alias-rds-dbserver-8
  targetKeyRef:
    from:
      name: rds-dbserver-8

The alias is not created:
status:
  ackResourceMetadata:
    ownerAccountID: '966321756598'
    region: eu-west-2
  conditions:
    - lastTransitionTime: '2023-08-22T13:41:53Z'
      status: 'True'
      type: ACK.ReferencesResolved
    - message: "ValidationException: Alias must start with the prefix \"alias/\". Please see https://docs.aws.amazon.com/kms/latest/developerguide/kms-alias.html\n\tstatus code: 400, request id: 87750736-3cec-4256-99c7-3c41d7140492"
      status: 'True'
      type: ACK.Terminal
    - lastTransitionTime: '2023-08-22T13:41:53Z'
      message: Resource not synced
      reason: resource is in terminal condition
      status: 'False'
      type: ACK.ResourceSynced

Expected outcome
Alias is created within the appropriate key

Finally it worked with the next manifest
apiVersion: kms.services.k8s.aws/v1alpha1
kind: Alias
metadata:
  name: rds-dbserver-8
  namespace: infra-production
spec:
  name: alias/rds-dbserver-8
  targetKeyRef:
    from:
      name: rds-dbserver-8

But it was a little bit ambiguous, as from the Amazon Web UI there is no necessity to add alias/ prefix...
Probably the "alias/" prefix could be added programmatically or Alias object can be protected by admission controller to disallow creation of improper objects (just refuse to create an Alias without alias/ on k8s level)

/remove-lifecycle stale

/remove-lifecycle stale


Update two OMIABIS with OBIB IDs

OBIB assigned OBIB prefix IDs to OMIABIS terms. So, we need to remove two following terms from externalByhand.owl
OMIABIS_0001011 count of current total human study participant
OMIABIS_0001018 cross sectional study design
Import the terms using OntoFox from OBIB:
OBIB_0000687 count of current total human study participant
OBIB_0000694 cross sectional study design
Need work on it when new OBIB release is out.

OBIB release is out.


Refactor schemaToSampleObj so it returns an array of examples

As @mrin9 mentioned here, PR #350 broke some examples. This PR fixes those, by refactoring schemaToSampleObj so it returns an array of examples, and handling cases where a nested call returns multiple examples:

for a oneOf: merge all examples for every oneOf-item
for a anyOf: combine all examples for the regular properties with every example for every anyOf-item
for an object: if a property has multiple examples, create an example for the object with every example of the property
for an array: create an example with every option for the items schema

Thanks for the PR !!!

I have rebased the branch, since it had conflict on the minified js etc

I have rebased the branch because it had conflicts on the minified js etc.


Grid and initialIndex not working on iOS but working on android

Experiencing an issue with iOS PhotoBrowser where the initialIndex does not load the correct photo and the grid does not go to the correct photo when tapped.
I get this most of the time when I click on a photo in the grid or try to load an initial photo through the initialIndex prop.

I'm calling the PhotoBrowser component as shown in the examples:
      <PhotoBrowser
        onBack={() => goBack()}
        mediaList={media}
        initialIndex={initIndex}
        displaySelectionButtons={false}
        displayActionButton={showActionButtons}
        displayTopBar={true}
        startOnGrid={false}
        enableGrid={true}
        onActionButton={this.onActionButton}
        alwaysDisplayStatusBar={true}
        customTitle={(index, rowCount) => `${index} sur ${rowCount}`}
      />

Again, Android works like a charm.  Not sure what's throwing off the iOS version?
Anyone experience this issue?

And I've got the same problem


HowTo Package Mac Version

Try to package for an Mac. But with to given pom it will not package. Whats wrong.  Release Source 4.3.0.

Can you provide more detail in what you doing? How is Maven building it? Any errors? OS?
If you use the archi.product file you can actually export a packaged build from the "Eclipse Product Export Wizard"

hi i build it in a console without running eclipse.  is that wrong . we would like to use archi but the securtiy policies did not allow to use MacApp without a valid zertificate
for an Eclipse Product Export Wizard" i need a running eclipse.
kind regards

Are you calling the top-level root pom.xml file?
But be aware that the product files that are produced by Maven cannot be used "as is". The files and directories have to be re-arranged to produce a working .app file.
Can you add a certificate to the existing .app file?

Are you calling the top-level root pom.xml file?

Yes

But be aware that the product files that are produced by Maven cannot be used "as is". The files and >> directories have to be re-arranged to produce a working .app file.

What have to be re-arranged. Curently i have after a build no .app file

I've just taken a closer look at this (it's a long time since I did the pom.xml files).

The top-level pom.xml file is configured for build and unit tests. This is used by Travis CI for automated testing.
To produce a product build the pom.xml file needs to be edited. I have just now committed a new pom.xml file containing comments on how to do this. You can uncomment some lines and remove one line to achieve this.
When Maven builds the product binary zip files they are placed in the \com.archimatetool.editor.product\target\products directory
In my experience the produced zip files are not very useful as they require additional post processing to create a proper final product. Especially so for Mac.

I don't use the Maven pom.xml files for final builds. I use Eclipse's Product Export wizard and then a series of post-processing scripts to copy and move files into their correct locations.


ToGlib should take Self

Fixes #410
Requires https://github.com/gtk-rs/gir/pull/1093

This breaks the ToGlib impl for SignalHandlerId and similar cases

This breaks the ToGlib impl for SignalHandlerId and similar cases

/me blames Sublime Text's search being so slow that I thought there were no search results left. Should be handled now

I mean SignalHandlerId and similar are not Copy :)

glib: mark SourceId/SignalHandlerId copy

I mean they are intentionally not Clone or Copy :)

I mean they are intentionally not Clone or Copy :)

How should this be handled? Have them not implement ToGlib?

I have no idea what's causing
error[E0277]: the trait bound `&Animal: ToGlib` is not satisfied
  --> glib-macros/tests/test.rs:27:49
   |
27 |     #[derive(Debug, Eq, PartialEq, Clone, Copy, GEnum)]
   |                                                 ^^^^^ the trait `ToGlib` is not implemented for `&Animal`
   |
   = help: the following implementations were found:
             <Animal as ToGlib>
   = note: required by `to_glib`
   = note: this error originates in a derive macro (in Nightly builds, run with -Z macro-backtrace for more info)

error[E0277]: the trait bound `&MyFlags: ToGlib` is not satisfied
   --> glib-macros/tests/test.rs:115:5
    |
115 |     #[gflags("MyFlags")]
    |     ^^^^^^^^^^^^^^^^^^^^ the trait `ToGlib` is not implemented for `&MyFlags`
    |
    = help: the following implementations were found:
              <MyFlags as ToGlib>
    = note: required by `to_glib`
    = note: this error originates in an attribute macro (in Nightly builds, run with -Z macro-backtrace for more info)

I mean they are intentionally not Clone or Copy :)

How should this be handled? Have them not implement ToGlib?

Ah indeed. I guess we can just not implement ToGlib for them, there are only two of them :)

Ah indeed. I guess we can just not implement ToGlib for them, there are only two of them :)

Are there any other types where you had to add Clone / Copy?

I have no idea what's causing
[...]

The enum/flags macros should implement the traits on T instead of &T, use them as such and the types themselves need to derive Clone and Copy.

Are there any other types where you had to add Clone / Copy?

There were the only types, I ended up adding a pub unsafe fn as_raw(&self) -> T

I have no idea what's causing
[...]

The enum/flags macros should implement the traits on T instead of &T, use them as such and the types themselves need to derive Clone and Copy.

The issue comes from the SetValue trait I think where we get a &Self that doesn't implement to_glib(). Trying to clone &Self to get Self back gives me a
error[E0015]: calls in statics are limited to constant functions, tuple structs and tuple variants
  --> glib-macros/tests/test.rs:27:49
   |
27 |     #[derive(Debug, Eq, PartialEq, Clone, Copy, GEnum)]
   |                                                 ^^^^^
   |
   = note: this error originates in a derive macro (in Nightly builds, run with -Z macro-backtrace for more info)

error[E0015]: calls in statics are limited to constant functions, tuple structs and tuple variants
   --> glib-macros/tests/test.rs:115:5
    |
115 |     #[gflags("MyFlags")]
    |     ^^^^^^^^^^^^^^^^^^^^
    |
    = note: this error originates in an attribute macro (in Nightly builds, run with -Z macro-backtrace for more info)

error: aborting due to 2 previous errors

which really doesn't help ^^

Can you give the output with -Z macro-backtrace? That probably makes more sense

Here's the output with -Z macro-backtrace
error[E0015]: calls in statics are limited to constant functions, tuple structs and tuple variants
   --> glib-macros/tests/test.rs:27:49
    |
27  |     #[derive(Debug, Eq, PartialEq, Clone, Copy, GEnum)]
    |                                                 ^^^^^ in this macro invocation
    |
   ::: /home/bilelmoussaoui/Projects/gtk-rs/glib-macros/src/lib.rs:247:1
    |
247 | pub fn genum_derive(input: TokenStream) -> TokenStream {
    | ------------------------------------------------------ in this expansion of `#[derive(GEnum)]`

error[E0015]: calls in statics are limited to constant functions, tuple structs and tuple variants
   --> glib-macros/tests/test.rs:115:5
    |
115 |     #[gflags("MyFlags")]
    |     ^^^^^^^^^^^^^^^^^^^^ in this macro invocation
    |
   ::: /home/bilelmoussaoui/Projects/gtk-rs/glib-macros/src/lib.rs:335:1
    |
335 | pub fn gflags(attr: TokenStream, item: TokenStream) -> TokenStream {
    | ------------------------------------------------------------------ in this expansion of `#[gflags]`

error: aborting due to 2 previous errors

It doesn't give any useful information sadly :/

Time to give a try to cargo expand then.

Time to give a try to cargo expand then.

I ended up doing that & fixed the issue, next time I should stop being lazy.

The expanded code from the macro must be calling to_glib() or similar in a constant scope somewhere.

The expanded code from the macro must be calling to_glib() or similar in a constant scope somewhere.

It was exactly that :-)

Clippy doesn't seem to like to_glib as a method name, maybe we can rename it to into_glib?

Clippy doesn't seem to like to_glib as a method name, maybe we can rename it to into_glib?

That would be the correct naming in this case, yes.

Should the trait itself be renamed to IntoGlib as well?

That would make sense

We now have into_raw for Value and for Error. Should these be changed to into_glib?

This should be rebased and ready. I will open a corresponding PR against gtk4-rs/gstreamer-rs as well a bit later today, don't merge before that :)

We now have into_raw for Value and for Error. Should these be changed to into_glib?

I don't think so, they are not Copy types.

@sdroege this one is ready to go :)


fatal: oracle_soa_oracle_home' is undefined"

I followed the README and overall all went well but I had to do to manual steps.
First fix the JAVA_HOME and PATH vairable for the user oraclefmw.
I did this by modifying the global bashrc file located at /etc/bashrc added the following two lines
export JAVA_HOME=/usr/java/jdk1.8.0_74
export PATH=${JAVA_HOME}/bin:${PATH}
Then I reran ansible and I got:

< TASK [sysco-middleware.oracle-weblogic : lineinfile] >
    \   ^__^
     \  (oo)\_______
        (__)\       )\/\
            ||----w |
            ||     ||

fatal: [default]: FAILED! => {"failed": true, "msg": "'oracle_soa_oracle_home' is undefined"}
to retry, use: --limit @oracle-retailapps.retry

so I set the variable oracle_soa_oracle_home in my yml ansible script
so the role looked something like the following
INSTALL MIDDLEWARE
- role: sysco-middleware.oracle-weblogic
  oracle_weblogic_version: 12c
  oracle_weblogic_release: 12.2.1
  oracle_weblogic_quick_installation: yes
  oracle_weblogic_install_type: Complete with Examples # WebLogic Server, Coherence, Complete with Examples
  oracle_wls_installers_12_2_1_1of1: /srv/files/fmw_<IP_ADDRESS>.0_wls_quick.zip
  oracle_weblogic_jar: /srv/files/fmw_<IP_ADDRESS>.0_wls_quick.jar
  oracle_soa_oracle_home: /opt/oraclefmw/product/oracle_home

So question is should you set the value so it has this by default or just remove the fix-wlst-securerandom-12c.yml type script ?
I am not sure what the fix-wlst-securerandom-12c.yml does so not sure if its needed since seems to be related to SOA.
Thanks

Hi @drgeb, Thanks to point this issues.
Should be solved with this commit: ec2783a784791028617def49dd3142b1480b42c8
This script adds a param to avoid slowlyness when you run WLST in Docker or new VMs: http://davidkerwick.blogspot.pe/2014/03/really-slow-wlst-and-adminserver-start.html
Let me know if its working now to close this issue


⚠ Bofrost.fr has degraded performance

In 4b7417c, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1254 ms

Resolved: Bofrost.fr performance has improved in 0d0715e after 23 minutes.


onClick in overlay area, event has interrupted

please fix it

Same problem here. Can not be click in any button while tooltip are open.

check the ClosePolicy: https://github.com/sephiroth74/android-target-tooltip/blob/master/library/src/main/java/it/sephiroth/android/library/tooltip/Tooltip.java#L112
See the usage in the sample app too

I want to click outside, close the tooltip and trigger components in the background.
Its possible?

yes, it's already possible

If [onTouchEvent](https://github.com/sephiroth74/android-target-tooltip/blob/master/library/src/main/java/it/sephiroth/android/library/tooltip/Tooltip.java#L1278) returns true, the event stops.

Got it.
It is NO CONSUME option, right?

yes, set consume outside to false


the .from() method shows "in x time" instead of "x time ago"

inserting a Date type into the to() method ex: dayjs().to(dayjs(Date))  method shows "x time ago". that is different from what was being shown in the docs.
the .from() method shows "in X time" when used similarly.
the variables of Date type had data in this format: x: Date = "2016-01-01 00:00:00"

doc updated, thx


Test PR: DO NOT MERGE!

Description

How Has This Been Tested?

Merge criteria:

[ ] The commits and have meaningful messages; the author will squash them after approval or will ask to merge with squash.
[ ] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[ ] The developer has manually tested the changes and verified that the changes work

Closing test PR without merging, please ignore!


idea: Non-blocking signal of Nighthawk breakages as part of CI

Background:
Nighthawk is (by design) closely coupled to Envoy internals. A company that uses both has to import both at the same time and resolve any build breakages. Nighthawk has its own set of owners that maintain it and submit updates that catch up with Envoy's changes (e.g. https://github.com/envoyproxy/nighthawk/pull/699).
Problem:
Envoy developers making code changes are in the dark when making decisions on whether a piece of code is being used / required. While the Envoy developers aren't required to maintain Nighthawk, having some visibility into how a PR affects the Nighthawk's use case may save developer cycles.
Example:
See https://github.com/envoyproxy/envoy/pull/16865. If the Envoy dev who worked on the original PR knew about the Nighthawk use case, they could have decided to add the accessor methods. The end result is the same, but two more developers were involved in resolving the issue.
Proposal for discussion:
We could add a non-blocking CI workflow that would just build (i.e. not test to ensure lack of flakes) the Nighthawk repo at each Envoy PR. This would achieve two things:

The Envoy devs would be making informed decisions.
The NH devs would also get an early warning about breakages.
To be noted (2) could be achieved by a CI check in Nighthawk's repository, however (1) is considered more valuable here.

Assumptions:
We retain the existing split of responsibilities, Envoy devs aren't responsible for and aren't going to fix Nighthawk breakages. This would serve as an optional signal they may take into account when making design decisions.
Tradeoffs:

Additional CI cost.
A build breakage indicated at a PR even if non-blocking could cause confusion if not documented properly.

The confusion could be achieved by a well formed error message with the right documentation.
Please let me know what you think.
/cc @alyssawilk @htuch

I see value here in providing this signal providing NH maintainers are on the hook and responsive to issues here (@mum4k tells me this is the case). As a general principle, I don't think we want an unbounded number of CI jobs tracking projects that depend on Envoy, but Nighthawk is in envoyproxy organization and a key tool that we want to bring into Envoy CI anyway, so this makes sense from that angle.
@envoyproxy/maintainers @phlax thoughts?

not sure exactly about this particular case - but my main reservation would be that we want to avoid adding tests for downstreams if changes in that project are likely to break envoy's CI - otherwise it kinda becomes a dependency for us - but from my limited understanding it may well be a good idea here

Thank you for starting the discussion. The reservation about downstreams breaking envoy's CI is a good one and should be addressed by the fact that this CI build will be non-blocking (serving as an optional signal only).
That of course leaves us open to seeing the signal report a breakage on a couple of PRs until the Nighthawk maintainers perform the necessary change after a breaking commit. However nothing should be blocked or slowed down due to its optionality.

I think though we do want Nighthawk in Envoy CI in the not too distant future.

@htuch you did raise a good point. We are planning to finalize Salvo and bring it into Envoy's CI as per the OSS benchmarking of Envoy Requirements in not so distant future.
Once that is done, Nighthawk will become more important to Envoy and its performance. The idea proposed here could very well be used to pave our way towards it.

I started working on a prototype and since this will be a pipeline that checks out multiple repositories (Envoy and Nighthawk), it looks like we will need a service connection to Github.
Assuming we are all good to move forward here, can someone indicate who has the necessary admin rights to set this up?

A fair few of us do @mum4k. @lizan might be best as he knows the AZP plumbing, but I can help out if it's just clicking some boxes.

@lizan your help / advice is appreciated. As noted above, I think that I need a service connection to Github defined in AZP in order to be able to check out the Nighthawk repository.
Could you confirm if this is correct and if so, help me to set one up?

Pending time allocation, but will be addressed.

No we don't really need service connection since both repository are public. Check how filter example repo are set up in CI here: https://github.com/envoyproxy/envoy/blob/main/ci/filter_example_setup.sh#L19


Add the Generic Scheme?

If it is supported by Alchemy Earth. Is it?
Add the Scheme here: https://github.com/dOrgTech/DAOcreator/blob/faedbcc49c0c3fcc6afb6b47f66636e6c8573763/src/lib/integrations/daoStack/arc/schemes.ts#L4
Add the Scheme to the UI Wizard here: https://github.com/dOrgTech/DAOcreator/blob/faedbcc49c0c3fcc6afb6b47f66636e6c8573763/src/components/pages/DAOcreator/index.tsx#L36

@sogasg it should be supported by alchemy earth !!

So we've decided not to support this in DAOCreator for the reasons of that a user should be able to vote to add this scheme with a particular contract address rather than specifying one upon initialization.


Permission denied error while replicating

unable to replicate the paper in colab, error at 7 . How do I fix it??
 5 assert BUCKET, 'Must specify an existing GCS bucket name'
      6 OUTPUT_DIR = 'gs://{}/lasertagger/{}'.format(BUCKET, TASK)
----> 7 tf.gfile.MakeDirs(OUTPUT_DIR)
      8 print('***** Model output directory: {} *****'.format(OUTPUT_DIR))

PermissionDeniedError: Error executing an HTTP request: HTTP response code 403 with body '{
  "error": {
    "code": 403,
    "message"<EMAIL_ADDRESS>does not have storage.objects.get access to contextualexp/lasertagger/wikisplit.",
    "errors": [
      {
        "message"<EMAIL_ADDRESS>does not have storage.objects.get access to contextualexp/lasertagger/wikisplit.",
        "domain": "global",
        "reason": "forbidden"
      }
    ]
  }
}
'
	 when reading metadata of gs://contextualexp/lasertagger/wikisplit

Please note that we haven't provided any Colabs to replicate the paper experiments but the following script:
https://github.com/google-research/lasertagger/blob/master/run_wikisplit_experiment.sh
Perhaps the colab you refer to is from this issue. The error that you shared doesn't seem to be related to any model code but to Cloud settings.

Oh ok thank you


*: Upgrade rust toolchain to nightly-2023-12-10

What is changed and how it works?

Issue Number: Close #15581

What's Changed:

Related changes

[ ] PR to update pingcap/docs/pingcap/docs-cn:
[ ] Need to cherry-pick to the release branch

Check List
Tests

[ ] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Release note

The performance before and after this change is almost the same.
Benchmark results:

Commit
TPCC
sysbench

4702b9bf6ed8d72d20bedf3de184f63111055b8e

e3702be579323bb145729d742618f49375656129

/hold  better resolves the coments

/merge

/unhold


Fix parallel execution of shrinkVectorArtwork

This is the effective solution to the parallelism problem that 417ee421f6a41e1c46293c9be8c780800e9465be aimed to solve. The latter may be reverted at some point.

Depends on #79


Support for webdrivercss

It would be nice feature, if protractor supports also webdrivercss(https://github.com/webdriverio/webdrivercss) which is a plugin for webdriverjs. Or, is it already possible? Thanks :)

webdrivercss is based off webdriverio, whereas protractor is based off webdriverjs. There is no plans (at least nothing near-term) to support webdriverio.


Quantification over two or more variables not supported

---- MODULE Issue ----
EXTENDS Naturals

VARIABLE x, y

Init ==
    /\ x = 0
    /\ y = 0

Next ==
    \E i,j \in 1..3:
    	/\ x' = i
        /\ y' = j
====

Workaround:
---- MODULE Issue ----
EXTENDS Naturals

VARIABLE x, y

Init ==
    /\ x = 0
    /\ y = 0

Next ==
    \E i \in 1..3, j \in 1..3:
    	/\ x' = i
        /\ y' = j
====

Should be addressed as of 052e1422.
See spec.


Feature inquiry, save txt

Hello,
It may be useful to have an option to save the transcribed speech to a text file.
Like all of the text that is printed to the console is appended to a text file (or other format).
Are there any plans to implement a feature similar to this?
Thank you

Good idea to add.  Could add a listen_to_file method

@arschoon yes I take PRs

This could be done somewhat easily with current code.  Will take a PR or may do myself in the future.


Add back pressure mechanism for multi-cursor queue

What changed?
Implement back pressure mechanism for multi-cursor queue. Basic idea is to slow down (backoff 5 seconds) task loading for non-default queue if it can't submit tasks to task processor (meaning task chan is full).

Why?
Part of multi-cursor implementation

How did you test it?
Unit test, integration test, local canary

Potential risks
N/A, feature flag disabled

Coverage decreased (-0.04%) to 67.741% when pulling cbc375fcd9b9545e039e298ef9851cf5f9b40404 on yycptt:multi-cursor-back-pressure into efb2e7c6cd83f188d0d5f1b24ee71c8ed37093a0 on uber:master.


Add initial architecture documentation

This commit adds roughly the first 40% of the gopass
architecture documentation. Once completed this is supposed
to make it easier for new contributors to ramp up.
RELEASE_NOTES=n/a
Signed-off-by: Dominik Schulz<EMAIL_ADDRESS>

Thanks a lot. aspell caught on ton, but not everything.


Fix release process with omitting Chart.lock unrelated updates

Comparison between Chart.lock before and after helm dep build execution is required to prevent circular dependency between Chart.lock update, which is included in release [Auto commit], and next unnecessary release pipeline execution. Allowed change in Chart.lock is only when dependencies are different. Dependencies might change only when Redpanda chart dependencies got new release.
Fixes #1185

Do we also need to do similarly to the other charts?

We don't have such golden tests in other charts, so the answer is no.

I wonder if we should instead just set the generated field of Chart.lock to a static time with yq and lint that the time is set correctly in CI.

I would not recommend that path.

Anyone that runs helm deps build could generate some weird diffs.

Yes, but I would leave that to any dev that need to maintain this repository.

I wonder if we should instead just set the generated field of Chart.lock to a static time with yq and lint that the time is set correctly in CI.

From my perspective any invocation of helm dep build inside our test suite would bump generated timestamp in charts/redpanda/Chart.lock. We should always run helm dep build as downstream dependencies could change (new release) that could change the golden files output. If none of the Redpanda chart dependnecies changed, then our release process should accommodate that to not create endless loop. I chosen the closes place (that is not bash script) where helm dep build is invoked to handle a change in Chart.lock.
The proposal of handling static time with yq is not that close to the developer workflow of calling go test ./.... The invocation would be codify in the Github Action Workflow which in my opinion creates friction in developer experience, because any one running go test would need to remember to exclude Chart.lock unrelated change from the commit. To add more yq might be additional dependency where go code uses the stdlib (less dependency).


Access right side menu from keyboard.

Is there a way to access the right side menu in GamePadUI from a keyboard?  Want to adjust the overlay at times but can only access it when a notification is showing.

ctrl +2

Many thanks.


Is this support Spring cloud Dalston.SR5

Not support, Dalston has not be supported  till Dec 2018

官方已经不维护D版了