Fix array.array.frombytes for Python 3.9

Fixes https://github.com/jpype-project/jpype/issues/570.

array.fromstring was removed in favour of frombytes in Python 3.9
Python 2 has been dropped, remove PyPy2 from CI
Sequence is in collections.abc in Python 3.3+. This is commented code, but updated just in case it's uncommented in future

nightly on Travis CI now points to Python 3.9, and it's now green :)

@hugovk we have completed a new release.  Sorry for the delays.

Thank you!


rustfmt: Use show_parse_errors

Following on from the comment in rust-bitcoin #3006,
Quoting tcharding (link removed to prevent spamming rustfmt PR):

There are no behaviour changes with this patch applied.
In rust-lang/rustfmt 5961 the hide_parse_errors was changed to show_parse_errors with default true.
Since we run the formatter with nightly we are getting the following warning
Warning: the hide_parse_errors option is deprecated. Use show_parse_errors instead
Rename the variable and keep the default value true.

Quoting @tcharding (link removed to prevent spamming rustfmt PR):

Lolz, you left the  mention in, I removed it. One day I'll report this bug to github ...

Lolz, you left the mention in, I removed it. One day I'll report this bug to github ...

It's not Github (whose merge messages are basically empty) but rather our merge script, which takes text from the PR title and description and puts them into the merge commit message. Then the Github alerts occur because Github treats mentions in commits as a notification-worthy event whenever something happens to that commit. You can argue that this is a bug or should at least be configurable...but it's "working as designed".

oooo, and that is why no one else knows about it. Thanks for the explanation.

which takes text from the PR title and description and puts them into the merge commit message.

We should just parse out mentions in the script then, right?

The current codebase is not formatted. I'm happy to ack and merge this but I'm curious what the goal of this rustfmt config is, if we're not actively using it.
@tcharding yeah, we could fork the script (which comes from bitcoin-maintainer-tools) to strip them. Or maybe, since it already has logic to warn about this, it could offer to strip them for you, so that we don't have to do this silly "read the warning, edit the PR text, then rerun the script" dance.

The current codebase is not formatted.

That is my mistake, added #14

Oh I assumed we'd patch the bitcoin repo not fork it, seems like the "correct" thing to do for everyone.

We should propose it. I suspect they won't like "silently change the PR description" especially because this would result in tons of false positives since it's hard to distinguish between @-mentions and any other use of the @ character, for example in email addresses. But I think it would be a clear improvement if the script would offer to change the description, since usually "just delete all the @s" is the correct solution.


docs(README): fix key mapping

What problem does this PR solve?
In README.md, it is mentioned using Leader for key mappings.
But the plugin has been implemented by using LocalLeader. So the mapping won't work if we follow the documentation (README.md)
ref. https://github.com/ElmCast/elm-vim/blob/master/ftplugin/elm.vim#L54
How does this PR solve it?
change Leader to LocalLeader in README.md

This is helpful! I was confused at first about why the mappings weren't working, until I realized they weren't actually using my leader key.

Info is misleading, I had to look into source to find out it's not using leader


[GH-1021] Removed the project icon from ticket link tooltip

Summary
Removed the project icon from ticket link tooltip
Screenshot

Ticket Link
Fixes #1021

This is a bit unrelated to this PR, and I know we have this pattern in other similar forms from other plugins, but it seems deceiving when it says "See More", and it actually takes you to the ticket instead of "showing more" of the description.
@matthewbirtch Do you have any thoughts on this? Expanding the description would not work well here since the floating card would need to resize. Maybe we can either remove or reword the "See more" link to make it obvious that you will leave the application to go to Jira if the link is clicked?

To be honest, I'm not sure where the 'see more' link came from. It wasn't in the original design so I would suggest removing it. There are enough other ways to click through to the full ticket (clicking the title or ticket number).

@matthewbirtch I think the "See more" was carried over from the GitHub plugin's tooltip. Do you think we should remove this link from both plugins? As well as GitLab since we are releasing that feature there too

@matthewbirtch I think the "See more" was carried over from the GitHub plugin's tooltip. Do you think we should remove this link from both plugins? As well as GitLab since we are releasing that feature there too

@mickmister my vote would be to remove it in all these cases since it's a potential source of confusion and the result of clicking it is not expected. Alternatively, we change the text to Open in Jira or Open in Github instead, but as I mentioned previously, I think there is enough other ways to do that.


Remove trailing slash from proxy_pass

This implements https://github.com/iv-org/documentation/pull/269 to correct the proxy_pass directive.

Looks good. Thanks! :+1:


question about booster training and inference

Hello, thanks for the shared code and checkpoint.
I am amazed by the great performance on the Booster dataset.
Can you provide more implementation details of training the pre-trained checkpoint on Booster.
I also find in the evaluation code, the booster images are downsampled by a scale of 2. While the paper shows it predicts disparities in the original resolution.
And is there any plan to make the booster checkpoint available?

I appreciate your interest in our work.
We follow the training strategy of crestereo, augmenting the booster dataset with the crestereo dataset to train the model for 13k steps based on the pre-trained model on sceneflow. Furthermore, we individually fine-tune the model on the booster dataset for 3k steps to get the final model.
We are sorry for the inaccurate description of the booster's settings. In fact, the 1/4 output is used to initialize the model at 1/2 resolution.
As for the checkpoint, the code version for benchmark submission is a little different from our final version in github.  You can now download the checkpoint at here and validate the booster dataset using the booster's branch code.  Alternatively, you can retrain the model based on our latest version.

Thanks for your helpful reply.
If I want to use the cascade inference for submission, do I need to upsample the 1/2 resolution results to full resolution?
And are the weights pre-trained on the augmented booster with crestereo available? I need some help to figure out how to train models and make inference on booster dataset.


Run bin tests

Pull request
Description
there seems to be a feature/bug in cargo that cargo test --all does not run tests in a binary, so we now run those tests explicitly.
Related
Fixes the problem in #878 that the test isn't failing
Checklist

[x] The RFC, if required, has been submitted and approved
[x] Any user-facing impact of the changes is reflected in docs.tremor.rs
[x] The code is tested
[x] Use of unsafe code is reasoned about in a comment
[x] Update CHANGELOG.md appropriately, recording any changes, bug fixes, or other observable changes in behavior
[x] The performance impact of the change is measured (see below)

Performance
Just CI changes

Coverage remained the same at 83.537% when pulling 7a0a061cd2ee46bf22e9874d4e11a8a1c0e15f93 on fix-ci-for-tremor-bin into 51832f22d37a2c481b05de95f3ee3cd16bc9ad94 on main.


DockMate: Add option to preferences to enable DarkMode for DockMate

Feature Request
We are using DockMate and it's a nice replacement for hyperdock. :-)
The only thing what we don't like is: in order to make the background of the preview windows dark you have to enable darkmode in system preferences (the problem is that you are switching the whole system to darkmode which most of us don't like).
It would be very nice if you could add an option to enable Darkmode in DockMate (so we don't have to switch the whole system to darkmode) because Dockmate looks very modern and nice with a dark windows background.
Thank you in advance,
Max

Thanks for the feedback, the option will be in the next release.

Thank you very much for your fast response and adding the option to DockMate. :-)


ORM for Get Server

Let me confess the fact that the GetX ecosystem is the most amazing thing that ever happened to Dart. With Fuchsia is on the horizon, it just feels right to have a unified full-stack Dart ecosystem. Get Server is the right step towards the backend of this unified full-stack ecosystem.
However, despite it's too early, but I think the GetX ecosystem as whole needs to have a formal roadmap.
One important part of the modern backend is ORM. Which is currently lacking in Get Server or probably in any other server-side  Dart framework. Yes, I know that @jonataslaw already planning to have one. But the purpose of creating this issue is specific to give some opinion on the implementation of the upcoming ORM which I am expecting to be available sooner than later.
Here's my thought:
I have surfed enough of the backend frameworks from Laravel to Spring to Django to .NET for a while now. What I have found is that the ORMs on Django and .NET (Entity Framework) are two of the best there is. But in my observation, the Django ORM is tightly coupled to the Django framework. For example, you have to extend your model classes from models.Model class to ensure the ORM can detect changes. On the other hand, the .NET Entity Framework has something called Database Context. Which specifies all of the models that should be tracked. Models are simple classes without any extension.
Therefore, in my opinion, I think it would be nice to have an ORM like Entity Framework in Get Server (Or maybe in the Flutter front-end GetX too to be able to communicate with in-app databases like SQLite). We already have the amazing get_cli tool which can be used to generate the database migration classes/files without build_runner and also to sync/update the database (much like manage.py and dotnet-ef tool).
Also, can we have a GitHub organization now? As GetX is growing rapidly?
Thanks for the amazing work.

I thing that some ORM similar to Isar can be nice to have here. I am not .NET expert but isn't Isar similar to EntityFramework in .NET??
And yes some GetX github organization can be useful when GetX is growing up and now have a lot of parts: State management, Dependency Injection, Route management, Server, CLI, etc..
And this all can be separated into different repositories because some people want to use only some parts in their projects.

I am not sure if we want code generation except the migration file. That will defeat the purpose of GetX. Which is no code generation, no build runner. And what I know is that Isar is database itself, not ORM. So no! Isar isn't helpful in this case I think.

https://github.com/CpdnCristiano/mysql_simple_orm
as close to an orm without a code generator as it can get

https://github.com/CpdnCristiano/mysql_simple_orm
An orm without a code generator.
Still in development

Nice work! But if you can find a way to get rid of manually defining the table schema, then it will be useful. The schema code can be generated as soon as we don't need to use them anywhere in our code except the ORM itself. Like I said, like EntityFramework. You just need to register your models as DbSets in the database context and you are done. Then you just use the database context object to do any database operations.

Have you ever tried the sqfentity package ?
I were a .NET developer and searching for an ORM like EF. Finally I found this package to be useful and have an EF-like structure . Try it 👍

Have you ever tried the sqfentity package ?
I were a .NET developer and searching for an ORM like EF. Finally I found this package to be useful and have an EF-like structure . Try it

Definitely not what I wish for.
I am talking about code-first migration and without code-generation except for the actual migration file. Like how the get_cli generates code.


Update /docs/concepts/containers/images/

wrap notes by shortcodes
remove command prompts
I think that using kubectl create -f - without creating a tmp file is better.

/lgtm
/approve


minor cleanup

ci: spelling correction
ci: removed unneeded print

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


JsonSerializer and References

Currenty the JsonSerializer can't (de)serialize References that cannot be resolved.
It would be nice to have an flag in the JsonSerializeOptions to omit serializing the reference?
do i overlook something that makes this impossible?
(am not sure about the derserialization aspects)
our usecase is to test the parser. there we care about the ast structure only and not any errors in resolving

Just to clear things up, because I haven't tried it yet. Does the JsonSerializer throw an error when trying to serialize a non-resolved reference?

no but it includes
{
 "ref": Object {
"$refRef": b,
           "$error": "Could not resolve reference to NamedDotTargetReference named 'b'.",
            }
}

but we dont care about $error


Fix bug where Bean would be tried to register twice and faill apllica…

…tion on startup
:loudspeaker: Type of change

[x] Bugfix
[ ] New feature
[ ] Enhancement
[ ] Refactoring

:scroll: Description

:bulb: Motivation and Context

:green_heart: How did you test it?
:pencil: Checklist

[x] I reviewed submitted code
[ ] I added tests to verify changes
[ ] I updated reference documentation to reflect the change
[x] All tests passing
[ ] No breaking changes

:crystal_ball: Next steps

How/when can this bug happen?

Sorry @maciejwalkowiak  forgot to add a link. #142  this was fixed in 2.3.1


[incubator/airflow] Airflow Helm Chart with KubernetesExecutor

What this PR does / why we need it:
There is no official curated chart for Airflow in helm repositories. Earlier attempts on adding charts has been based on the CeleryExecutor. This version is based on the native KubernetesExecutor and will hopefully have higher probability to get merged.
Which issue this PR fixes
N/A
Special notes for your reviewer:
Please note that Airflow is in Apache incubator and the KubernetesExecutor is just in early stages, but still, as I see; the only good solution when running on top of k8s. That said, at the time of this PR the code has a bug or two. Due to that, the docker image used is built from a PR on Airflow. Thus it is set to latest and not a release of Airflow. But I will update it as soon as it is merged and new release of Airflow is available.
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md

@tekn0ir
Before creating this PR have you watched at https://github.com/helm/charts/pull/3959 ?

/hold

@tekn0ir
So let's add support for KubernetesExecutor to Airflow Chart.
There is no sense to have to charts for the same app.

I also used over the Astronomer Airflow helm chart as inspiration, it is quite similar to #3959: https://github.com/astronomer/helm.astronomer.io/tree/master/charts/airflow but is built into an ecosystem of charts that is impressive. I think they have gathered most experience of what works and what does not.


Antrea Agent Crashed with iptables 1.8.X

Describe the bug
Antrea agent assumes iptables version is < 1.6.2, and iptables-restore doesn't support --wait. So Antrea agent locks on "/var/run/xtables.lock" before running iptables-restore.
In the latest Debian 10 release, the iptables version is 1.8.3. The iptables-restore command supports the --wait option, and it acquires the xtables lock automatically. If Antrea acquires the lock before running iptables-restore, iptables-restore will fail because it thinks other process already occupied the lock.
To Reproduce
In Debian 10, start antrea-agent.
Expected
antrea-agent should run.
Actual behavior
antrea-agent crashed at "pkg/agent/route/route_linux.go", "Initialize" function. Error said it failed to initialize iptables.
Versions:
Please provide the following information:

Antrea version
v0.5.1.
Kubernetes version (use kubectl version).
Client Version: version.Info{Major:"1", Minor:"18", GitVersion:"v1.18.2", GitCommit:"52c56ce7a8272c798dbc29846288d7cd9fbae032", GitTreeState:"clean", BuildDate:"2020-04-16T11:56:40Z", GoVersion:"go1.13.9", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"18", GitVersion:"v1.18.2", GitCommit:"52c56ce7a8272c798dbc29846288d7cd9fbae032", GitTreeState:"clean", BuildDate:"2020-04-16T11:48:36Z", GoVersion:"go1.13.9", Compiler:"gc", Platform:"linux/amd64"}
Container runtime: which runtime are you using
docker
Linux kernel version on the Kubernetes Nodes
4.18.0-25-generic

Additional context
I workaround the problem by modifying "pkg/agent/util/iptables/iptables.go" as follow:
diff --git a/pkg/agent/util/iptables/iptables.go b/pkg/agent/util/iptables/iptables.go
index d275d891..06378165 100644
--- a/pkg/agent/util/iptables/iptables.go
+++ b/pkg/agent/util/iptables/iptables.go
@@ -18,7 +18,6 @@ import (
        "bytes"
        "fmt"
        "os/exec"
-       "time"

        "github.com/coreos/go-iptables/iptables"
        "k8s.io/klog"
@@ -89,22 +88,21 @@ func (c *Client) EnsureRule(table string, chain string, ruleSpec []string) error
 // Otherwise only involved chains will be flushed.
 func (c *Client) Restore(data []byte, flush bool) error {
        var args []string
+       var stdout, stderr bytes.Buffer
        if !flush {
                args = append(args, "--noflush")
        }
+       args = append(args, "--wait=10")
        cmd := exec.Command("iptables-restore", args...)
        cmd.Stdin = bytes.NewBuffer(data)
+       cmd.Stdout = &stdout
+       cmd.Stderr = &stderr
        // We acquire xtables lock explicitly for iptables-restore to prevent it from conflicting
        // with iptables/iptables-restore which might being called by kube-proxy.
        // iptables supports "--wait" option and go-iptables has enabled it.
        // iptables-restore doesn't support the option until 1.6.2, but it's not widely deployed yet.
-       unlockFunc, err := lock(xtablesLockFilePath, 10*time.Second)
-       if err != nil {
-               return err
-       }
-       defer unlockFunc()
        if err := cmd.Run(); err != nil {
-               return fmt.Errorf("error executing iptables-restore: %v", err)
+               return fmt.Errorf("error executing iptables-restore %v: %v\nstderr: %s\nstdout: %s", args, err, stderr.String(), stdout.String())
        }
        return nil
 }

If we switch to the latest Debian image, we need to only support a signle iptables version. I see that kube-proxy is using the latest Debian but with lower iptables version. This is an option, too.


Typo in createSocketPair – connect2 instead of connection2

The return params of createSocketPair should be:
connection_pair["success"] = success;
connection_pair["connection1"] = connection1;
connection_pair["connection2"] = connection2;

but are:
connection_pair["success"] = success;
connection_pair["connection1"] = connection1;
connection_pair["connect2"] = connection2;

👉 https://github.com/Gramps/GodotSteam/blob/47c031aae6caef463638d5ad51be3e9016813c77/godotsteam.cpp#L4611

Hey there!  Hmm, not sure how that got missed!  Haha, thanks for reporting it!

OK, also got this fixed up for the next patch.  Thanks!

Hmm, if this wasn't a main branch fix then it may have been applied to godot4 branch?  I will check it out.  If it is missing, I'll backport it this evening.  Thanks for reporting this!

I will re-open this until I can check it and fix it!

I don't think this was a regression, I think it was part of a patch that got overwritten before it was pushed live; which is my fault.  I patched and pushed the fix for this just now.
What was your problem with createListenSocketP2P?

OK, thanks for the link.  Yeah, it seems all my patches from that time period got wiped instead of pushed.  Hmm, I wonder how many there were.  Sadly no way to really tell; I'll just have to re-patch them as they are found.  Sorry about this!
I'll dig into it and get it fixed up and actually pushed to repo so it doesn't get lost again!

OK, this is now fixed in my local version and is about to undergo a compile test.  Your other issue should also be fixed too.  Once these pass, I will push to repo and then issue new pre-compiles.  Thanks for reporting this!


Reynold/objects shuffle

Description

Wall Conversion + Framing Objects

Type of change

New feature (non-breaking change which adds functionality)

How has this been tested?

Manual Tests (please write what did you do?)

Docs
Have already been updated

Will be updated ASAP- No updates needed

This PR is from the refactor we decided not to go through in the end. Closing this as it should not be merged in!


Make initial connect retry backoff configurable

@makdharma @jcanizales - PTAL at the ObjC changes

@muxi - hoping you can respond to @jcanizales requests

Please merge the other PR referenced above

Changes merged - @jcanizales PTAL

Regenerate projects plz.


GLES2 [visual] shader shows pixelated texture on Android

Godot version:
Godot 3.2.3.rc custom build as of 6 August 2020
SHA-1: 23b553ba0603161346526e1821bff5002520173c
OS/device including version:

GLES2 Android
Motorola G6 Play - Adreno 505
Issue description:

Using Visual Shader to modify uv of Texture to scale the image causes massive pixelation problem on Android.
Windows

Android

Steps to reproduce

Minimal reproduction project:

GLES2textureProb.zip

Duplicate of https://github.com/godotengine/godot/issues/32813
Fixed by
https://github.com/godotengine/godot/pull/33646
In your project settings tick the box next to rendering/gles2/compatibility/enable_high_float.android. that should solve your issue. :)

@clayjohn You are the man! Thank you so much!


Error from submitting timesheet due to permission

Need to remove this line. Seems they won't let 'Billable' be set anymore.
https://github.com/ThorpeJosh/tp-timesheet/blob/d2b605a0078a9cb47f17d049a367ae3bcb0c59d3/tp_timesheet/clockify_timesheet.py#L75C24-L75C24

Need to remove this line. Seems they won't let 'Billable' be set anymore.
https://github.com/ThorpeJosh/tp-timesheet/blob/d2b605a0078a9cb47f17d049a367ae3bcb0c59d3/tp_timesheet/clockify_timesheet.py#L75C24-L75C24

Wow thanks! How did you find out from the error image? :O

No problem. I saw in logs it attempted to post a json with billable: true and it got a 501 error stating that we don't have permission to update the billability.
500 series errors are server side (clockify servers) and usually related to incorrect parameters, features not implemented, server malfunctions, etc. In this case luckily the message was informative.

#111


CancellationToken support and a lot of async improvements

This PR adds the ability to cancel (almost) all asynchronous operations with the built-in CancellationToken. Due to the nature of interfaces, almost all of those changes are breaking the existing API.
It also adds an WatchAsync method to the IKubernetesClient, which is making use of the newer asynchronous enumerables.
I tried to keep it as backwards-compatible as possible, however, this was not possible due to different problems. Most notably, the interfaces no longer have a default implementation to actually break existing code instead of failing silently. If I would've just changed the existing interface (with Task.CompletedTask as the return , current implementations of ReconcileAsync would have not been called anymore. Therefore I removed the default implementations to ensure that all remaining controllers get updated.
Note about the one failing test
During local development with a kind cluster, I could not get the KubeOps.Operator.Test.Events.EventPublisherIntegrationTest.Should_Increase_Count_On_Existing_Event to run successfully. It always broke with some certification error, other tests work fine tho. I could not determine whether it is an issue with the kind cluster or some other error.

Hey @nachtjasmin
Thank you for your contribution!
I don't mind breaking changes at all. While I'm reviewing the stuff, the linter is ranting about some things it seems. dotnet format should be able to help as far as I know.
Another general comment: Why do we need factories? Personally I'm not really a fan of factories. Is there a specific use-case / reason for them?

The linter seems to be angry with you...

The joy of line endings. Running dotnet format was not enough, cause Git would've changed the line endings back to LF on git add. Added a .gitattributes file in order to enforce the correct line endings for the .cs files, now it should work as intended.

all fingers crossed.

Hey @nachtjasmin
Do you have any idea, why the tests won't start?
It seems, that the host does not start in test mode.
The debug output of the github actions:
KubeOps.Operator.Test -> /home/runner/work/dotnet-operator-sdk/dotnet-operator-sdk/test/KubeOps.Operator.Test/bin/Debug/net8.0/KubeOps.Operator.Test.dll
Test run for /home/runner/work/dotnet-operator-sdk/dotnet-operator-sdk/test/KubeOps.Operator.Test/bin/Debug/net8.0/KubeOps.Operator.Test.dll (.NETCoreApp,Version=v8.0)
Microsoft (R) Test Execution Command Line Tool Version 17.9.0 (x64)
Copyright (c) Microsoft Corporation.  All rights reserved.
Starting test execution, please wait...
A total of 1 test files matched the specified pattern.
<7>Microsoft.Extensions.Hosting.Internal.Host[1] Hosting starting
<6>KubeOps.Operator.Watcher.ResourceWatcher[0] Starting resource watcher for V1OperatorIntegrationTestEntity.
<6>KubeOps.Operator.Watcher.ResourceWatcher[0] Started resource watcher for V1OperatorIntegrationTestEntity.
<6>Microsoft.Hosting.Lifetime[0] Application started. Press Ctrl+C to shut down.
<6>Microsoft.Hosting.Lifetime[0] Hosting environment: Production
<6>Microsoft.Hosting.Lifetime[0] Content root path: /home/runner/work/dotnet-operator-sdk/dotnet-operator-sdk/test/KubeOps.Operator.Test/bin/Debug/net8.0
<7>Microsoft.Extensions.Hosting.Internal.Host[2] Hosting started
<6>KubeOps.Operator.Watcher.ResourceWatcher[0] Received watch event "Added" for "OperatorIntegrationTest/test-entity", last observed resource version: 601.
<7>Microsoft.Extensions.Hosting.Internal.Host[3] Hosting stopping
<6>Microsoft.Hosting.Lifetime[0] Application is shutting down...
<6>KubeOps.Operator.Watcher.ResourceWatcher[0] Stopping resource watcher for V1OperatorIntegrationTestEntity.
<6>KubeOps.Operator.Watcher.ResourceWatcher[0] Stopped resource watcher for V1OperatorIntegrationTestEntity.
<7>Microsoft.Extensions.Hosting.Internal.Host[4] Hosting stopped
<7>Microsoft.Extensions.Hosting.Internal.Host[1] Hosting starting
<6>KubeOps.Operator.Watcher.ResourceWatcher[0] Starting resource watcher for V1OperatorIntegrationTestEntity.
<6>KubeOps.Operator.Watcher.ResourceWatcher[0] Started resource watcher for V1OperatorIntegrationTestEntity.
<6>Microsoft.Hosting.Lifetime[0] Application started. Press Ctrl+C to shut down.
<6>Microsoft.Hosting.Lifetime[0] Hosting environment: Production
<6>Microsoft.Hosting.Lifetime[0] Content root path: /home/runner/work/dotnet-operator-sdk/dotnet-operator-sdk/test/KubeOps.Operator.Test/bin/Debug/net8.0
<7>Microsoft.Extensions.Hosting.Internal.Host[2] Hosting started
<7>Microsoft.Extensions.Hosting.Internal.Host[3] Hosting stopping
<6>Microsoft.Hosting.Lifetime[0] Application is shutting down...
<6>KubeOps.Operator.Watcher.ResourceWatcher[0] Stopping resource watcher for V1OperatorIntegrationTestEntity.
<6>KubeOps.Operator.Watcher.ResourceWatcher[0] Stopped resource watcher for V1OperatorIntegrationTestEntity.
<7>Microsoft.Extensions.Hosting.Internal.Host[4] Hosting stopped
<7>Microsoft.Extensions.Hosting.Internal.Host[1] Hosting starting
<7>KubeOps.Operator.Watcher.LeaderAwareResourceWatcher[0] Subscribe for leadership updates.
<6>Microsoft.Hosting.Lifetime[0] Application started. Press Ctrl+C to shut down.
<6>Microsoft.Hosting.Lifetime[0] Hosting environment: Production
<6>Microsoft.Hosting.Lifetime[0] Content root path: /home/runner/work/dotnet-operator-sdk/dotnet-operator-sdk/test/KubeOps.Operator.Test/bin/Debug/net8.0
<7>Microsoft.Extensions.Hosting.Internal.Host[2] Hosting started
<6>KubeOps.Operator.Watcher.LeaderAwareResourceWatcher[0] This instance started leading, starting watcher.
<6>KubeOps.Operator.Watcher.LeaderAwareResourceWatcher[0] Starting resource watcher for V1OperatorIntegrationTestEntity.
<6>KubeOps.Operator.Watcher.LeaderAwareResourceWatcher[0] Started resource watcher for V1OperatorIntegrationTestEntity.
<6>KubeOps.Operator.Watcher.LeaderAwareResourceWatcher[0] Received watch event "Added" for "OperatorIntegrationTest/test-entity", last observed resource version: 600.
<6>KubeOps.Operator.Watcher.LeaderAwareResourceWatcher[0] Received watch event "Added" for "OperatorIntegrationTest/test-entity", last observed resource version: 617.
<6>KubeOps.Operator.Watcher.LeaderAwareResourceWatcher[0] Received watch event "Added" for "OperatorIntegrationTest/test-entity2", last observed resource version: 610.
<6>KubeOps.Operator.Watcher.LeaderAwareResourceWatcher[0] Received watch event "Added" for "OperatorIntegrationTest/test-entity", last observed resource version: 601.
<7>Microsoft.Extensions.Hosting.Internal.Host[3] Hosting stopping
<6>Microsoft.Hosting.Lifetime[0] Application is shutting down...
<7>KubeOps.Operator.Watcher.LeaderAwareResourceWatcher[0] Unsubscribe from leadership updates.
<7>Microsoft.Extensions.Hosting.Internal.Host[4] Hosting stopped
<7>Microsoft.Extensions.Hosting.Internal.Host[1] Hosting starting
<6>KubeOps.Operator.Watcher.ResourceWatcher[0] Starting resource watcher for V1OperatorIntegrationTestEntity.
<6>KubeOps.Operator.Watcher.ResourceWatcher[0] Started resource watcher for V1OperatorIntegrationTestEntity.
<6>Microsoft.Hosting.Lifetime[0] Application started. Press Ctrl+C to shut down.
<6>Microsoft.Hosting.Lifetime[0] Hosting environment: Production
<6>Microsoft.Hosting.Lifetime[0] Content root path: /home/runner/work/dotnet-operator-sdk/dotnet-operator-sdk/test/KubeOps.Operator.Test/bin/Debug/net8.0
<7>Microsoft.Extensions.Hosting.Internal.Host[2] Hosting started
<6>KubeOps.Operator.Watcher.LeaderAwareResourceWatcher[0] Received watch event "Deleted" for "OperatorIntegrationTest/test-entity", last observed resource version: 628.
<6>KubeOps.Operator.Watcher.LeaderAwareResourceWatcher[0] Received watch event "Deleted" for "OperatorIntegrationTest/test-entity", last observed resource version: 631.
<6>KubeOps.Operator.Watcher.LeaderAwareResourceWatcher[0] Received watch event "Deleted" for "OperatorIntegrationTest/test-entity2", last observed resource version: 640.
<6>KubeOps.Operator.Watcher.LeaderAwareResourceWatcher[0] Received watch event "Deleted" for "OperatorIntegrationTest/test-entity", last observed resource version: 649.

At least with the local KinD cluster and with act, the tests are working again. The KubeOps.Operator.Test.Events.EventPublisherIntegrationTest.Should_Increase_Count_On_Existing_Event seems to be flaky for me, I'll try to fix it as well.
After that, I plan to rewrite the commit history to ensure a clean history. Gonna let you know when I'm done with that, just so you don't have to accept my WIP commits all the time. :smile:

Nice work, thank you :-)
Don't worry about the commits, they're gonna be squashed anyway.

And thank you for your effort!

Well, if the commits are squashed anyway, I think I'm done now. :D

The linter is still yapping, but I'll fix the issues right away.


Dark Mode

Hi,
I'm a fellow #100Dev. I've been using this site a lot to keep track of my progress.
I'd love to contribute to this project by creating a "dark mode" version of the site. I've been watching the issues page of this project for someone either requesting or suggesting for this feature but so far I haven't come across one. I asked Ms. Abro regarding this, and she gave me the "go ahead".
I'm a new developer, in fact this would probably be my first open source contribution. I'll be starting on it ASAP. Any suggestion is welcome. Any help is appreciated.
Looking forward to finishing this. I'm so excited.

Thank you for taking this on!
This project uses Tailwind CSS: https://tailwindcss.com/
I'm not sure if you're familiar with Tailwind or not, but it has support for dark mode: https://tailwindcss.com/docs/dark-mode
By default, Tailwind will apply dark mode based on the user's system preference, so there's no need to add a manual toggle. For this PR, please use that method; it's the simplest as it only requires adding Tailwind classes. (Ideally it should be a hybrid of system preference and manual toggle, but I'd like to leave that for a separate issue after the dark mode is implemented).

Thank you for taking this on!
This project uses Tailwind CSS: https://tailwindcss.com/
I'm not sure if you're familiar with Tailwind or not, but it has support for dark mode: https://tailwindcss.com/docs/dark-mode
By default, Tailwind will apply dark mode based on the user's system preference, so there's no need to add a manual toggle. For this PR, please use that method; it's the simplest as it only requires adding Tailwind classes. (Ideally it should be a hybrid of system preference and manual toggle, but I'd like to leave that for a separate issue after the dark mode is implemented).

I'm not familiar with Tailwind but I'm reading on it and it seems quite doable, specially since you want the default functionality which uses the user's system preference.

It's pretty simple once you've worked with it a bit, but if you do run into any issues, you can message me on Discord, happy to talk through it. :)

It's been quite a while since you last posted on this - are you still interested in completing it?

Hi @labrocadabro . I'd like to take a shot at this issue if it has indeed been abandoned.

It probably has, however community taught is going through some changes. The site is going to be integrated into the 100Devs website and one of the things I want to do is make the design of CT consistent with the main site. For that reason, I'm putting a hold on any design changes for now, as they'll probably need to be redone anyway.


Multi App Test Command - Doesn't pick up app name

Bug Report or Feature Request (mark with an x)
- [x] bug report -> please search issues before submitting
- [ ] feature request

Versions.
@angular/cli: 1.3.0
node: 8.2.0
os: win32 x64
@angular/animations: 4.3.6
@angular/common: 4.3.6
@angular/compiler: 4.3.6
@angular/core: 4.3.6
@angular/forms: 4.3.6
@angular/http: 4.3.6
@angular/platform-browser: 4.3.6
@angular/platform-browser-dynamic: 4.3.6
@angular/router: 4.3.6
@angular/cli: 1.3.0
@angular/compiler-cli: 4.3.6
@angular/language-service: 4.3.6

Repro steps.
If you put two apps into the generated angular cli project you can suppply the --app parameter and get each app to serve.
However I can't seem to get this to do the same thing with a test the following steps are contained in this repo:
https://github.com/sethreidnz/angular-cli-multi-app-test-repro
But repeated here:
First clone this repo:
git clone https://github.com/sethreidnz/angular-cli-multi-app-test-repro

Install the npm packages:
npm install
``

If you serve them with the following commands they run fine:

```bash
ng serve --app app1
ng serve --app app2

You will see that the apps have different titles, this is the only difference between their files contained in folders /app1 and /app2.
However if you try to run the tests as in the test help command:
ng test -h

With the parameter --app like so:
ng test --sr --app app1
ng test --sr --app app2

You will get the exact same error:
Expected 'APP 1!' to equal 'app'.

Even though they both have a different title property in their respective app/app.component.ts. You would expect the app2 test to have an error like:
Expected 'app 2!' to equal 'app'.

Desired functionality.
Be able to run each projects test by passing the --app parameter equal to the app definition in the .angular-cli.json file.

Any ideas on why this is happening? Do you want a contrib?

I ran into this problem today, it looks like it always takes the first app when building the webpack config for testing,
https://github.com/angular/angular-cli/blob/1.4.x/packages/%40angular/cli/models/webpack-configs/test.ts#L22
I ended up working around this problem by providing another karma.config.js for the second app that overrides the webpack.entry config with the correct entries.

@rorticus Could you please provide a karma.conf.js snippet for your webpack entries?

I have two apps in separate folders (say app1, app2). The angular-cli.json is in their parent folder.
I have two karma.config.js files in the two folders, this seems to work (app1 and app2 respectively):
var path = require('path');
module.exports = function (config) {
config.set({
...
webpack: {
entry: {
main: [path.resolve('.','app2', 'test.ts')]
}
},
But still needed to import polyfills in the beginning of test.ts for it to work. Why is that?

@csaadaam I think you need to include an extra entry for the polyfills,
webpack: {
    entry: {
        main: path.resolve('./app2/src/test.ts'),
        polyfills: [path.resolve('./app2/src/polyfills.ts')]
    }
}

It works this way as well, thanks!

Fixed via https://github.com/angular/angular-cli/pull/7795


my config cannot init.

Secrets for authentication using PAP
client    server      secret      acceptable local IP addresses
chenhuan    pptpd       chenhuan402    *
amy         pptpd       amy402         *
huanhuan    pptpd       huanhuan402    *
image is my config file
and command is docker run -it --privileged -p 1723:1723 -v /root/chap-secrets:/etc/ppp/chap-secrets mobtitude/vpn-pptp
cannot connect server.
can you help me?

Please try following chap-secrets:
chenhuan	*	chenhuan402	*
amy			*	amy402			*
huanhuan	*	huanhuan402	*


Correct copyright year in transform_volumes_test.go

Ⅰ. Describe what this PR does
This PR corrects copyright year ( 2023 -> 2022 ) in transform_volumes_test.go.
The first commit for it is f8991a19af2940310bd91ea91bfe799c68253099, which was created in 2022-05-14 00:30:47 UTC.
Ⅱ. Does this pull request fix one issue?

NONE
Ⅲ. List the added test cases (unit test/integration test) if any, please explain if no tests are needed.
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

/lgtm

/lgtm
/approve


compose: Don't jump to sent message conversation if setting disabled.

This PR adds a new jump_to_sent_message_conversation setting to allow users to decide whether to automatically go to conversation where they sent a message.
Fixes: #30600

Screenshots and screen captures:
Setting Enabled
screen-capture (1).webm
Setting Disabled
screen-capture (2).webm
Help Docs

Self-review checklist

[ ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Set the new setting to disabled.
Go to an interleaved channel view.
Send a message to a topic in the channel.

Expected: You see the appropriate banner (which we showed before we started jumping to the conversation).
Actual: A banner flashes by and disappears. It's not even the right banner, as it says the message you sent is outside your current view.

Did we used to show the banner in Inbox and Recent conversations, or is that new in this PR?

Works in my testing otherwise (when the message is sent to a place that's outside your current view).

Works for me in manual testing!

I found a bug which leads to "Sent! Your recent message is outside the current search." banner not visible. -- When you send a message while narrowed to a search view.

What is the situation, and what exactly goes wrong? When I try a search, I get the regular "Your message is outside your current view." banner, which seems totally fine to me (whether or not that's what we originally designed).

What is the situation, and what exactly goes wrong? When I try a search, I get the regular "Your message is outside your current view." banner, which seems totally fine to me (whether or not that's what we originally designed).

Search for a word e.g. "are" in search box. (http://localhost:9991/#narrow/search/are)
Now, send a message anywhere
No banner will appear.

Oh, I see. I was testing with a filter plus a keyword, and the banner did appear in that situation.

Oh, I see. I was testing with a filter plus a keyword, and the banner did appear in that situation.

Yeah, the specific search view I mentioned doesn't work. Everything else works fine.

I think the /help/ article is going to want something that sets more context about the trade-offs, but we can probably write that along with the "zulip updates" notice for this change.
Other than my small notes above about managing the API change, this looks good to merge. Hopefully we can do that today.

Merged, thanks @prakhar1144!
I didn't wait to get a resolution on https://github.com/zulip/zulip/pull/30729#discussion_r1669222148 because it's logistically helpful to get finished migrations in. I'm not certain what that logic did, so please follow up on the design thread about what that logic did and why you didn't include it in this PR; we can always add it tomorrow as a follow-up if necessary.


test: Increase unit test coverage in internal/kafka/internal/services

Description
Increase unit test code coverage in internal/kafka/internal/services where possible or ignore files, for which writing unit tests is not feasible in the make test make target.
Jira: Here
Before

After

Verification Steps

Run make test.
Run make test/html/coverage/report.
Should be able to see the improved test coverage.

Checklist (Definition of Done)

[x] All acceptance criteria specified in JIRA have been completed
[x] Unit and integration tests added that prove the fix is effective or the feature works (tested against emulated and non-emulated OCM environment)
[ ] Documentation added for the feature
[ ] CI and all relevant tests are passing
[ ] Code Review completed
[ ] Verified independently by reviewer
[ ] Required metrics/dashboards/alerts have been added (or PR created).
[ ] Required Standard Operating Procedure (SOP) is added.
[ ] JIRA has created for changes required on the client side

The PR needs a rebase @JasonPowr would you mind doing the honor? Thanks
/cc @pawelpaszki @JameelB @ziccardi for another round of reviews

No changes since my last review, my approval still stands :+1:  :slightly_smiling_face:


./cli compose pull之前运行正常！
运行环境 | Environment
- OS: RockyLinux 9.4
- Scheduler: slurm-23.11.1
- Docker: Docker version 24.0.7, build afdd53b
- Docker-compose: Docker Compose version v2.23.3
- SCOW cli: Version 1.5.2
- SCOW: master
- Adapter: Apr 7更新的版本，不记得具体版本了，重新编译更新1.5版本报错

第4步已经解决！
## 1、修改grpc的版本，将1.55修改为1.64

google.golang.org/grpc v1.64.0

## 2、更新版本
go mod tidy

## 3、重新构建
make build

但是使用新构建的scow-slurm-adapter-amd64 问题仍未解决！

重新配置scow，初始化时提示当前正在访问的集群不可用或没有可用集群。请稍后再试或联系管理员。

[root@scow scow]# ./cli compose ps
INFO: Loaded plugins: []
WARN[0000] /scow/docker-compose-1718865568980.yml: `version` is obsolete
NAME                   IMAGE                                         COMMAND                  SERVICE         CREATED              STATUS              PORTS
scow-audit-db-1        mysql:8                                       "docker-entrypoint.s…"   audit-db        About a minute ago   Up About a minute   3306/tcp, 33060/tcp
scow-audit-server-1    mirrors.pku.edu.cn/pkuhpc-icode/scow:master   "./entrypoint.sh"        audit-server    About a minute ago   Up About a minute   80/tcp, 3000/tcp, 5000/tcp
scow-auth-1            mirrors.pku.edu.cn/pkuhpc-icode/scow:master   "./entrypoint.sh"        auth            About a minute ago   Up About a minute   80/tcp, 3000/tcp, 5000/tcp
scow-db-1              mysql:8                                       "docker-entrypoint.s…"   db              About a minute ago   Up About a minute   3306/tcp, 33060/tcp
scow-gateway-1         mirrors.pku.edu.cn/pkuhpc-icode/scow:master   "./entrypoint.sh"        gateway         About a minute ago   Up About a minute   3000/tcp, <IP_ADDRESS>:80->80/tcp, :::80->80/tcp, 5000/tcp
scow-log-1             fluentd:v1.14.0-1.0                           "tini -- /bin/entryp…"   log             About a minute ago   Up About a minute   5140/tcp, <IP_ADDRESS>:24224->24224/tcp, <IP_ADDRESS>:24224->24224/udp, :::24224->24224/tcp, :::24224->24224/udp
scow-mis-server-1      mirrors.pku.edu.cn/pkuhpc-icode/scow:master   "./entrypoint.sh"        mis-server      About a minute ago   Up About a minute   80/tcp, 3000/tcp, 5000/tcp
scow-mis-web-1         mirrors.pku.edu.cn/pkuhpc-icode/scow:master   "./entrypoint.sh"        mis-web         About a minute ago   Up About a minute   80/tcp, 3000/tcp, 5000/tcp
scow-novnc-1           ghcr.io/pkuhpc/novnc-client-docker:master     "/docker-entrypoint.…"   novnc           About a minute ago   Up About a minute   80/tcp
scow-portal-server-1   mirrors.pku.edu.cn/pkuhpc-icode/scow:master   "./entrypoint.sh"        portal-server   About a minute ago   Up About a minute   80/tcp, 3000/tcp, 5000/tcp
scow-portal-web-1      mirrors.pku.edu.cn/pkuhpc-icode/scow:master   "./entrypoint.sh"        portal-web      About a minute ago   Up About a minute   80/tcp, 3000/tcp, 5000/tcp
scow-redis-1           redis:alpine                                  "docker-entrypoint.s…"   redis           About a minute ago   Up About a minute   6379/tcp

mis-server 的日志有报错信息：
[root@scow scow]# ./cli compose logs mis-server
INFO: Loaded plugins: []
WARN[0000] /scow/docker-compose-1718875515775.yml: `version` is obsolete
mis-server-1  |
mis-server-1  | ><EMAIL_ADDRESS>serve
mis-server-1  | > node build/index.js
mis-server-1  |
mis-server-1  | {"level":30,"time":"2024-06-20T09:22:44.986Z","pid":17,"hostname":"bc7a670af1a1","msg":"Hook is not configured."}
mis-server-1  | {"level":30,"time":"2024-06-20T09:22:46.011Z","pid":17,"hostname":"bc7a670af1a1","version":{"commit":"6bde35bd1fc62b5e2187123cbe09cc1227a8ef10"},"msg":"@scow/mis-server: "}
mis-server-1  | {"level":30,"time":"2024-06-20T09:22:46.012Z","pid":17,"hostname":"bc7a670af1a1","config":{"HOST":"<IP_ADDRESS>","PORT":5000,"LOG_LEVEL":"info","LOG_PRETTY":false,"SSH_PRIVATE_KEY_PATH":"/root/.ssh/id_rsa","SSH_PUBLIC_KEY_PATH":"/root/.ssh/id_rsa.pub","AUTH_URL":"","DB_PASSWORD":"must!chang3this"},"msg":"Loaded env config"}
mis-server-1  | {"level":30,"time":"2024-06-20T09:22:46.981Z","pid":17,"hostname":"bc7a670af1a1","msg":"Checking if root can login to hpc-test by login node login"}
mis-server-1  | {"level":30,"time":"2024-06-20T09:22:47.149Z","pid":17,"hostname":"bc7a670af1a1","msg":"Root can login to hpc-test by login node login"}
mis-server-1  | {"level":30,"time":"2024-06-20T09:22:47.155Z","pid":17,"hostname":"bc7a670af1a1","msg":"Update cluster entity started."}
mis-server-1  | {"level":30,"time":"2024-06-20T09:22:47.226Z","pid":17,"hostname":"bc7a670af1a1","plugin":"price","msg":"Default Price Map: {}"}
mis-server-1  | {"level":30,"time":"2024-06-20T09:22:47.226Z","pid":17,"hostname":"bc7a670af1a1","plugin":"price","msg":"Tenant specific prices {}"}
mis-server-1  | {"level":30,"time":"2024-06-20T09:22:48.382Z","pid":17,"hostname":"bc7a670af1a1","plugin":"price","msg":"Executing on hpc01 success"}
mis-server-1  | {"level":40,"time":"2024-06-20T09:22:48.383Z","pid":17,"hostname":"bc7a670af1a1","plugin":"price","msg":"\n      The following price items are missing in platform scope: [\"hpc01.compute.normal\",\"hpc01.compute.low\",\"hpc01.compute.high\"].\n      An error will be thrown when such a job is fetched.\n    "}
mis-server-1  | {"level":30,"time":"2024-06-20T09:22:48.573Z","pid":17,"hostname":"bc7a670af1a1","plugin":"fetch","msg":"Fetch info started."}
mis-server-1  | {"level":30,"time":"2024-06-20T09:22:48.723Z","pid":17,"hostname":"bc7a670af1a1","plugin":"syncBlockStatus","msg":"misConfig.periodicSyncStatus?.cron: 0 4 * * *"}
mis-server-1  | {"level":30,"time":"2024-06-20T09:22:48.725Z","pid":17,"hostname":"bc7a670af1a1","plugin":"syncBlockStatus","msg":"Sync block status started."}
mis-server-1  | {"level":30,"time":"2024-06-20T09:22:48.729Z","pid":17,"hostname":"bc7a670af1a1","plugin":"cache","msg":"Cache clear scheduled task started."}
mis-server-1  | {"level":30,"time":"2024-06-20T09:22:48.785Z","pid":17,"hostname":"bc7a670af1a1","plugin":"syncBlockStatus","msg":"Current clusters list: (Cluster ID: hpc01) : ACTIVATED"}
mis-server-1  | {"level":30,"time":"2024-06-20T09:22:48.802Z","pid":17,"hostname":"bc7a670af1a1","plugin":"syncBlockStatus","msg":"Updated block status in slurm of the following accounts: []"}
mis-server-1  | {"level":30,"time":"2024-06-20T09:22:48.802Z","pid":17,"hostname":"bc7a670af1a1","plugin":"syncBlockStatus","msg":"Updated block status failed in slurm of the following accounts: []"}
mis-server-1  | {"level":30,"time":"2024-06-20T09:22:48.802Z","pid":17,"hostname":"bc7a670af1a1","plugin":"syncBlockStatus","msg":"Updated block status in slurm of the following user account: []"}
mis-server-1  | {"level":30,"time":"2024-06-20T09:22:48.803Z","pid":17,"hostname":"bc7a670af1a1","plugin":"syncBlockStatus","msg":"Updated block status failed in slurm of the following user account: []"}
mis-server-1  | {"level":30,"time":"2024-06-20T09:22:48.819Z","pid":17,"hostname":"bc7a670af1a1","plugin":"syncBlockStatus","msg":"Current clusters list: (Cluster ID: hpc01) : ACTIVATED"}
mis-server-1  | {"level":30,"time":"2024-06-20T09:22:48.820Z","pid":17,"hostname":"bc7a670af1a1","plugin":"syncBlockStatus","msg":"Updated unblock status in slurm of the following accounts: []"}
mis-server-1  | {"level":30,"time":"2024-06-20T09:22:48.821Z","pid":17,"hostname":"bc7a670af1a1","plugin":"syncBlockStatus","msg":"Updated unblock status failed in slurm of the following accounts: []"}
mis-server-1  | {"level":30,"time":"2024-06-20T09:22:48.883Z","pid":17,"hostname":"bc7a670af1a1","msg":"Listening at 5000"}
mis-server-1  | (node:17) DeprecationWarning: Calling start() is no longer necessary. It can be safely omitted.
mis-server-1  | (Use `node --trace-deprecation ...` to show where the warning was created)
mis-server-1  | {"level":30,"time":"2024-06-20T09:23:09.743Z","pid":17,"hostname":"bc7a670af1a1","req":"1","path":"/scow.server.InitService/QuerySystemInitialized","msg":"Starting request"}
mis-server-1  | {"level":30,"time":"2024-06-20T09:23:09.753Z","pid":17,"hostname":"bc7a670af1a1","req":"1","path":"/scow.server.InitService/QuerySystemInitialized","msg":"Request completed."}

麻烦看下是哪里的问题？

使用master分支为以上情况，使用1.5.2分支正常！

升级cli到1.6版本后正常！


QRCode Scanner and Text Copyer on areas which are unable to focus.

Description of the new feature / enhancement
What about add a tool to scan QR code on the current user screen and decode it.
And  there are some area on the window contains text but user cannot copy it. Or some text on the pic.
Scenario when this would be used?
When I need to decode some QR code and inspect it. Now I can only search a website and upload the QR code, which is inconvenient.
When I need to copy some text on the pic or window title bar, I can only type them myself. It's inconvenient.
Supporting information
We can use Tesseract to recognize text on the pic.

Duplicate of #4859

Duplicate of #4371

Issue is a duplicate
Thank you for your issue! We've linked your report against another issue to centralize the thread.
Thanks for helping us to make PowerToys a better piece of software..


Support pypi trove classifiers

Trove classifiers are somewhat important for software classification inside pypi. Moreover, it can help support better some data about the package.
Poet could be great place to abstract this concept in an higher-level way, like :
dev-status = 'beta'  # this add the 'Development Status :: 4 - Beta' classifier

Many information already provided, like compatible python versions, licence, or C/C++ extensions, can directly be understood as trove classifiers (Programming Language :: Python :: *, License :: *, Programming Language :: C*).
This is also a way to control the values given by user : if a licence is not recognized (because not convertible to trove classifiers), user should be warned during package building.

Python versions are already supported. For instance if you put python = ["~2.7", "^3.5"] in your poetry.toml file it will set the following classifiers:
Programming Language :: Python
Programming Language :: Python :: 2,
Programming Language :: Python :: 2.7,
Programming Language :: Python :: 3,
Programming Language :: Python :: 3.5,
Programming Language :: Python :: 3.6

The other classifiers are not currently supported but I think it would be a nice addition.


Add kafka/rabbit binder support

As we're moving away from redis for new app starters, default binders are kafka and rabbit. Work on yarn side is happening in https://github.com/spring-cloud/spring-cloud-dataflow-admin-yarn/issues/86.
Need to make configuration to work so that kafka or rabbit can be used during a service configuration.

Wrong project.


Invalid request. Invalid parameter "/": unexpected parameter "auth".

Describe the bug
After updating plugin and moving zabbix to new server, now getting this message when trying to reauthenticate and connect the plugin.
Expected behavior
Authentication to succeed and healthy response from zabbix api.
Software versions

Grafana
Zabbix
Grafana-Zabbix Plugin

11.3.0
7.2.0rc1
4.5.7

I second this.
Getting the same error in grafana dockerized.

You're using an unsupported version, 7.2 isn't production ready and not supported. Currently support goes till 7.0.x

For me, every version on 7.2 has worked until 7.2 rc1. Wonder what changed to cause this issue? Going to the API page gives http error 412 now. Does the API page work for anyone on RC1?

For me, every version on 7.2 has worked until 7.2 rc1. Wonder what changed to cause this issue? Going to the API page gives http error 412 now. Does the API page work for anyone on RC1?

You should check it out on Zabbix side. As I said, 7.2 isn't production ready, and is not currently supported by the Plug-in.

I'm also getting this same error. Is there no solution? I downloaded the latest versions of zabbix and grafana and they just don't work together? I installed zabbix 7.2 on all my hosts and now I have to go back to 7.0 and start all over again? Did you find a solution? @viper2pt0 @jakub-nowak-pl @isaqueprofeta @sglamm

@lphbraz you shouldn't use zabbix 7.2 yet, it isn't released. Check on https://www.zabbix.com/road map : "Zabbix 7.2 Planned release date: Q4 2024"

It’s a Zabbix issue, not Grafana. Looks like the api for zabbix is broken
or does not function the same in 7.2. Its not production yet so we have to
use it at our own risk.
Sean Lamm
On Fri, Dec 6, 2024 at 11:46 AM lphbraz @.***> wrote:

I'm also getting this same error. Is there no solution? I downloaded the
latest versions of zabbix and grafana and they just don't work together? I
installed zabbix 7.2 on all my hosts and now I have to go back to 7.0 and
start all over again? Did you find a solution? @viper2pt0
https://github.com/viper2pt0 @jakub-nowak-pl
https://github.com/jakub-nowak-pl @isaqueprofeta
https://github.com/isaqueprofeta @sglamm https://github.com/sglamm
—
Reply to this email directly, view it on GitHub
https://github.com/grafana/grafana-zabbix/issues/1914#issuecomment-2523839920,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKHUJBDY6OP4TTI7S2GQVHL2EHPHDAVCNFSM6AAAAABR6CMDBGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMRTHAZTSOJSGA
.
You are receiving this because you were mentioned.Message ID:
@.***>

I have the exact same issue and I'm using an supported version, 7.0.6
[Tue Dec 10 09:37:59.459857 2024] [proxy_fcgi:error] [pid 1251512:tid<PHONE_NUMBER>69152] [client x.x.x.x.:51310] AH01071: Got error 'PHP message: PHP Deprecated:  Parameter "/auth" is deprecated. in /usr/share/zabbix/include/classes/validators/CApiInputValidator.php on line 1516

Starting from version 6.4, it was announced that the auth property would be deprecated and removed in future releases. That moment has simply arrived, and no one was prepared for it.

Odd part about 7.2 is even the API page does not work. Gives 412 error. Not a Grafana issue, but part of the puzzle. I am waiting for some kind of update from Zabbix for a resolution.

@alxcreate thx for the pointer.
Being technical here now, 7.2.0 is out and looking at the API Docs now zabbix does not use the "auth" field for the AuthToken. Now the session/auth token, being an API Key or an authorize sesssion token should be at the header  'Authorization: Bearer {TOKEN}'.
@yesoreyeram @zoltanbedi @ivanahuckova can you take a look? This is a a pretty critical one.

Odd part about 7.2 is even the API page does not work. Gives 412 error. Not a Grafana issue, but part of the puzzle. I am waiting for some kind of update from Zabbix for a resolution.

Trying set token in http header, but get an error:
Invalid params. The "apiinfo.version" method must be called without authorization header.

I've deployed the proxy in docker as well and it seems to be working.  I did notice that there appears to be other API changes though.  For example when interacting with an item in the problem panel:

I've deployed the proxy in docker as well and it seems to be working. I did notice that there appears to be other API changes though. For example when interacting with an item in the problem panel:
Just more of a heads up for the folks working on the official fix.

It seems that Zabbix 7.2 has received other changes besides authentication.
It would be possible to add conditionals to intercept and modify the invalid parameters, but that would be too costly, and I think it’s better to wait for the Grafana team to proceed with their official adjustments.
In the meantime, this adaptation (at least for me) is working well.
Cheers!

if method != 'apiinfo.version':

datasource save&test will report：[Invalid params. The "user.login" method must be called without authorization header.]
change it to:
if method not in ['apiinfo.version', 'user.login']:

I have a PR with the fix ready https://github.com/grafana/grafana-zabbix/pull/1931. If anyone here has time and is up for it, would you verify that it fixes the issue for you. Thank you 🙏 .

I have a PR with the fix ready #1931. If anyone here has time and is up for it, would you verify that it fixes the issue for you. Thank you 🙏 .

I've got all dockerized but If this fix will got into grafana dockerized I can check.

I have a PR with the fix ready #1931. If anyone here has time and is up for it, would you verify that it fixes the issue for you. Thank you 🙏 .

I've got all dockerized but If this fix will got into grafana dockerized I can check.

Same here. I tried to mount the branch into /var/lib/grafana/plugins but that did not work. Do you have a how-to for testing this?

Works fine with Zabbix 7.2.0 - thanks!

i was using grafana OSS not in docker instance is how can i go around that problem cause am still facing it when i upgrade zabbix to 7.2

i was using grafana OSS not in docker instance is how can i go around that problem cause am still facing it when i upgrade zabbix to 7.2

You can manually download plugin and unzip it in plugins folder of Grafana.
But it's not a production choice. In your position, I would continue using the old versions and wait for a published fix.

Same issue here :(
I don't understand why you say Zabbix 7.2 is not a production version, it's the version on their main website, and it seems a prod version.

7.2 is in production and actually it's quite strange to not to work on Zabbix changes sice ver. 7 was out in beta

We have the build available to test through docker.
docker run -p 3000:3000 -e "GF_INSTALL_PLUGINS=https://storage.googleapis.com/integration-artifacts/alexanderzobnin-zabbix-app/4.5.7/main/163fabf651b776bf70adc08fa41bec4f52645374/alexanderzobnin-zabbix-app-4.5.7%2B163fabf6.linux_amd64.zip;alexanderzobnin-zabbix-app" grafana/grafana-enterprise:latest
Once you start the above docker instance, you need to enable the plugin in http://localhost:3000/plugins/alexanderzobnin-zabbix-app and then can test.
Can someone help to test the above build works both in 6.x and 7.x version of zabbix?

Gimme a sec, I'll try it with my zabbix-lab project.

Planning to release by EOD tomorrow.

Tested with the following versions on docker (used the https://github.com/isaqueprofeta/zabbix-lab project)
Zabbix 6.0

Zabbix 6.4

Zabbix 7.0

Zabbix 7.2

When this will be available for typical installation? I mean command grafana-cli plugins install alexanderzobnin-zabbix-app

I updated the environment and I have the same problem, grafana gives the same error as auth, can anyone help me?

When this will be available for typical installation? I mean command grafana-cli plugins install alexanderzobnin-zabbix-app

I updated the environment and I have the same problem, grafana gives the same error as auth, can anyone help me?

The new version with the correction is planned to be release tomorrow at the end of the day.

d to be release tomorrow at the end of the da

tks

I manually updated the plugin and it connected correctly, data is being loaded, but if I create a dashboard that requests large amounts of data from Zabbix, I get a 500 error.

Plugin alexanderzobnin-zabbix-app v4.6.0 already installed.
Zabbix version 7.2.0
Have error in Grafana:
Invalid params. Invalid parameter "/": unexpected parameter "real_hosts".

Works like a charm :)
Thanks for fixing it.

Plugin updating and working.
Thanks.


Run with existing website

Hi Everyone.  Is there a way to run PassCore on an existing web site by adding the application?
For instance, we have https://web.domain.org.  In IIS can I install PassCore as https://web.domain.org/password
It runs fine if we put Pass Core in as its own website, but as an application added to an existing site, we only get Loading PassCore...
Thanks,
James

Duplicate of #71


Define all C++ model constructors explicit

Static analysis on the C++ code at csrc/models/ produces the following warning:
Clang-Tidy: Constructors that are callable with a single argument must be marked explicit to avoid unintentional implicit conversions
By allowing implicit calls the following misleading usage is possible:

This can be stopped by defining all the constructors that receive up to 1 mandatory parameter as explicit:

After the change, the following 2 initializations are valid:
DenseNetImpl x1(100);
DenseNetImpl x2{100};

Note that thought this might be considered theoretically a breaking change on the C++ API, I believe not many users use the old initialization as it's quite misleading.
I tested the changes by manually running the test_cpp_models.py unit-tests. All pass except of test_mnasnet0_5, test_mnasnet0_75 and test_mnasnet1_3 which also fail on master.

With explicit specifier we prevent of using copy initialization: no more of
DenseNetImpl net = 1000; // error: copy-list-initialization

@vfdev-5 we could still do:
DenseNetImpl z{100};
DenseNetImpl x = z;

But as you said this is not allowed:
DenseNetImpl x = 100;

More specifically we don't allow the compiler to do is silently take the integer 100, pass it on the constructor as first argument to create a temporary object and then copy to x (thought optimizations can happen here by compiler). I think it's better to not allow this syntax because it leads to less readable code and potential bugs. Hence it's by default a warning on clang tidy. :)

@datumbox I totally agree ! Definition like that DenseNetImpl x = 100; does not make much sense.


Always getting Unauthorized

const passport = require('passport')
const GitHubStrategy = require('passport-github2').Strategy
const refresh = require('passport-oauth2-refresh')
const database = require('../database/database')
const { encrypt, decrypt } = require('../functions/crypto');

passport.serializeUser(function (user, done) {
    done(null, user);
});

passport.deserializeUser(function (user, done) {
    done(null, user);
});
const strategy = new GitHubStrategy({
    clientID: process.env.GITHUB_CLIENT_ID,
    clientSecret: process.env.GITHUB_CLIENT_SECRET,
    callbackURL: "http://localhost:3001/api/auth/github/callback",
    passReqToCallback: true,
}, async function (req, accessToken, refreshToken, profile, done) {
    if (req.session.passport !== undefined || req.session.passport.user !== undefined) {
        await database.query(
            "UPDATE users SET githubToken=? WHERE id=?",
            [encrypt(accessToken), req.session.passport.user.id]
        )
        return done(null)
    }
});

passport.use(strategy)
refresh.use(strategy)

when going to localhost:3001/api/auth/github and authenticating i get an "Unauthorized" page my router code is as follows
var express = require('express');
var router = module.exports = express.Router();
const passport = require('passport');

router.get('/', passport.authenticate('patreon'))

router.get('/callback',
    passport.authenticate('patreon', { failureRedirect: '/login', failureMessage: true }),
    function (req, res) {
        res.redirect('/');
    }
);

Have you done any debugging to isolate this as an issue with Passport?  Could you provide details?

Your routes indicate Patreon is being used.  But your strategy configuration is using GitHub.
Also, your strategy verify function is not returning a user object:
return done(null)

This would cause Passport to deny the request, because a user object is required.  For example:
return done(null, user)

Have you done any debugging to isolate this as an issue with Passport? Could you provide details?

Thats my bad sent the wrong one. They are both the same anyway

Ok returning the user seems to have fixed this.


[VS Code] The collaboration session has ended due to network going offline. Check your network status.

<!--
For Visual Studio problems/feedback, please use the Report

As the error message says, your computer's network connection was lost.
We have an open feature request about automatically reconnecting in case of temporary loss in connectivity: #67 Please upvote that and/or add comments if you think that would be helpful.


[read-write-set] Trim the inference result for a better performance

Motivation
Trim the analysis result by removing the child information (i.e: struct field access information). This will offer a better performance since we are not resolving those secondary index information anyway and we don't need those finer grained details at runtime.
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
cargo test.
Also this PR will be able to increase the inference speed by 50% on my local benchmarks.

some failures on CI

/land


Open to adding gettid?

Seeing that pthreads is out of nix's scope, could you add linux's gettid system call so that individual threads can be interacted with. At present, system calls can only operate on entire processes as a whole.
Implementing it strait forward since the system call, like getpid, takes no argument. (imo, It's even easier if you're willing to add the syscall crate as a dependency to abstract away architectural differences, but the crate uses no_std).
Side note: Thank you for this crate! I was in the middle of re-implementing it before it caught my eye. Very well made.

Sure!

I would take a PR for this. Going to close the issue for now.


Using C or Rust Libraries in Bend

Hi,
Thanks for Bend, its massively impressive.
Most computational heavy library in python are written in C, like numpy, tf, pandas, etc. This is not the case for Bend, but ability to use already build library would benefit the whole bend ecosystem.
Is it possible to use them?? Maybe via FFI, but this will only applicable to C ABI, libraries like Polars written in Rust would need extra work to port here.

Yes, this is  planned.
Closing as duplicate of https://github.com/HigherOrderCO/Bend/issues/350. See there for more information


Cluster connection stops working on 1.13

Hi there, I added the kubeconfig but it show the red triangle, and cannot connect cluster. I can use the same kubeconfig file with lens
Image
it is Microk8s cluster. It works with the previous Monokle version but not the latest. I had to reinstall the old version of Monokle 1.12

We think it is solved in 1.13.3, so please let us know if it does not work after updating


Feat: Created admin login / logout in frontend successfully issue 344

Pull Request Resolves[#344 ]
Title: Created admin login / logout in frontend successfully.
Description

Created login page for admin.
Added toast for both login and logout.
Modified 404 not found page.

Related Issues
Fixes #344
Changes Made
New features:  Created admin login / logout in frontend.
Screenshots

I certify that I have carried out the relevant code of conduct and provided the requisite screenshot for validation by submitting this pull request.
Thank You for this contribution.

@VaibhavArora314 all three mentors have to review ? I thaught you are the only one. previously you were only for merging and all !

@VaibhavArora314 all three mentors have to review ? I thaught you are the only one. previously you were only for merging and all !

Previously some of them were not active or assigned, but since most of them are present now so it would be great to get reviews from each of them.

@VaibhavArora314 all three mentors have to review ? I thaught you are the only one. previously you were only for merging and all !

Previously some of them were not active or assigned, but since most of them are present now so it would be great to get reviews from each of them.

Ohh I see

@akbatra567 @Ultimateutkarsh11 can you review ASAP because their are lot more issues to cover up and also @manikumarreddyu is waiting for this admin panel so that he can raise new issues.
I hope you consider my request.
Thank You

resolve merge conflicts

@VaibhavArora314 Done 👍

@akbatra567 please review this pr if any changes just let me know 😀


How to use this with a default error handler?

Let's say i have an application with some middlewares (3rd party libraries) which throw errors
How do I set things up so that I get apt http response on frontend.
Status code with an error message.
I usually do this in a error handler:
(req, res, next, error) => {
  if(error is type 1) {
    send response with status code assigned for type1 errors
  }
  if(error is type 2) {
    send response with status code assigned for type2 errors
  }
  if(error is type 3) {
    send response with status code assigned for type3 errors
  }
}

With your package this probably becomes:
(req, res, next, error) => {
  if(error is type 1) {
    send response with status code assigned for type1 errors
  }
  if(error is type 2) {
    send response with status code assigned for type2 errors
  }
  if(error is type 3) {
    send response with status code assigned for type3 errors
  }
  if(error is type HTTP_ERROR_ENHANCED) {
    send response based on error object as it has a statusCode and message
  }
}

Thoughts?

It seems correct to me.
Can you send a repro so I can get a better idea?

No updates so I'm closing this. Feel free to reopen.


Ability to pull icons from a typeface and use on MRTK buttons

Describe the problem
"As a user, I expect my application to run smoothly and performant at runtime."

As a developer of a HoloLens Application, if my application uses multiple "drawables" (aka, icons) and i want to display those drawables on my buttons, then I want to construct those drawables in a way that makes them as performant from a drawcall and memory allocation perspective as possible.

Given the above two statements, then:

if you look at how MRTK utilizes icons on things like buttons, it pulls in raster-based assets, typically 256x256 or 512x512 DXT textures per icon. This means each icon is roughly 200-something kb per asset. If you have 70 icon assets (not uncommon for things like mobile applications), then you've just eaten up a ton of memory the customer could have used for other things.

Describe the solution you'd like

The approach I would like to see implemented in MRTK is to allow the developer the option to set either a raster-based icon on MRTK assets that use icons OR optionally, instead display icons via a text-based atlas (a unique typeface made up of font glyphs). When you use a font-based glyph, you can gang up tons of icons onto a single file and they scale well in 3D space while keeping memory low. For instance, a single 1024x1024 atlas containing over 70 glyph-based icons is roughly .5mb. (i realize these numbers vary somewhat given a variety of conditions, but you get the point).

So, if we look at an MRTK standard button for the icon field, have a toggle to display either image-based raster icons or text-based glyph icons (via a point to a text field where user can point unity to the typeface add the desired character code into the text input to display the appropriate glyph)
Describe alternatives you've considered
Sans the above, we'll probably delta off MRTK to build our own version of the above to achieve the desired result, but I'd love to see it incorporated into the toolkit so we dont have to do that in long run.

Pinging @killerantz since he has a great tool that achieves most of this feature request.

I built a tool a while back to do something similar for our team. A branch was created to publish with MRTK, but never followed through.
Icons should be font based and it's pretty easy these days to take graphics and create a TTF file. With that, we wanted an easy way to manage fonts as well. One issue is that different people on a project would create a text object,  leave the default font or leave the default material. So we wanted an easy way for everyone on the team to create a text object with the correct font and material with one click. The same applies to Icons, we had an Icon font that we could modify as Icon design progressed.

Here's an example of the menu.
We use scriptableObjects to define which fonts show up on the drop down. Then if a user selects a text based object, they can select a font from the dropdown and the font and material are applied. Also works with multiple selected Text objects. You can also right-click on the text component in the inspector and select the font from a context menu.

Pairing the material and font is not an issue using TextMeshPro, but we enabled TextMeshPro as well to declare these are the official fonts to use. We detect which kind of text object is selected and apply the font/material or font asset.

The same system works for Icons. Import the Icon font into the project, add it to an IconSet, assign a name, there's a button to automatically generate a material. You can also create a font asset for TextMeshPro and add that to the IconSet (TMP is not shown in these images, but it exists :))
The last step is importing the glyphs from the icon font, you can view them and even disable individual glyphs in the IconSet UI.
Using the menu shown above, select Icon Viewer

This is a sample set of icons I created and can distribute as a sample. You can pull in your own icon fonts or grab generic icon font sets from places like Fontastic.com. This UI supports multiple font sets using the drop down at the top of the window, the list of icons scroll when list gets too long and the window scales. I've had a list of over 3000 glyphs running in the viewer, we collect and use icons for prototyping.
Select a text based object in the scene and click on a glyph in the Icon Viewer and the font/material or font asset is applied and the correct glyph code is added to the text field.
The idea again is organization, we can go in and turn off glyphs, add new ones, so the team is always working from the latest set of approved icons.
If this is something that is interesting or a variation of it, let me know and I can prepare a new branch with the latest code.

An alternate use case for the Text support is a configurable drop down with not just font, but style associated. This would allow the design team to configure styles, which would scale, adjust size and color, as well as applying font and materials from the drop down or context menu.

This is pretty awesome work. And it looks nice :)

Font icons are supported and recommended in MRTK3 (and MRTK2, now, I believe!)


Make this repo public

There are a few open PRs, can we open this repo once they are merged?
Are there any other outstanding issues preventing us from making it public?
Small tasks to be done once it's open:

[x] Add mock data to this repo - pointless before it's marked public as we won't be able to access those files
[ ] Setup travis - we could use CI, travis will be easy to add
[ ] Build an official docker image and put it on docker hub ?

🎉

Made a PR for travis - #17

will deploy a new build internally and if all goes well we'll tag a release

Repo is public so closing


fix image link for pro-banner on users page

:speech_balloon: :arrow_up:
Relative made it so on pages such as /library broke the banner link.

Ive looked at this error like 70times, but always forgot to fix it
:+1: LGTM


Eureka Server Basic HTTP Auth configuration

Hi,
I've managed to get multiple Eureka servers up, running and synchronized without issue. Looking at the docs,class definitions and other forum posts, there doesn't seem to be a clear way to implement HTTP authentication server-side. Is this possible at all or not? It looks to be possible in the spring-cloud-netflix eureka repo from this post.
Thanks & Regards,
Moiz Khan

@moizkhan2712 our internal set-up do SSL via fronting the eureka tomcat servers with an apache layer.


[pbc] build failure on MacOS

Host Environment

OS: MacOS
Compiler: clang

To Reproduce
Steps to reproduce the behavior:
./vcpkg install pbc
Failure logs

none: its a dependency problem

Additional context
Add any other context about the problem here, such as what you have already tried to resolve the issue.

The PR has been merged, so close this bug.


Perfomance problems

The application is slow, on Linux. I use another Wallets, and all works well less this Wallet.

Solve. I recompile on my system.


Clean up embedded URL on email reporting that badge gained/lost

BadgeApp sends an email when a project gains/loses a badge.  Currently the email includes a URL that is converted to nonsense due to the way sendgrid works.  We should modify the configuration so that the conversion stops happening.

@ejratl
This is just Click Tracking: https://sendgrid.com/docs/User_Guide/Settings/tracking.html

There's more information here:
https://sendgrid.com/docs/API_Reference/SMTP_API/apps.html
It appears that we can change the click tracking setting using the X-SMTPAPI header.


feat: enable multiple AWS rules on WAF

Use a common set of rules from AWS on the WAF
These rules are also used by the COVID Alert server

@dorianjp Yeah exactly. Did you look at the doc https://docs.aws.amazon.com/waf/latest/developerguide/aws-managed-rule-groups-list.html? There is a link in the code. They have a list of managed rules and I picked what seems to be useful for us.


Allow create-wekit-app to run without Contentful login

In order to enable independent Frontend work without the need to establish a content model first, we would want the create-wekit-app script to run through with requiring a npx contentful login first.

Don't think this makes sense ;)
Let's discuss next week


Would be nice if I could use typings to pull my own d.ts definitions without all the versioning and complex setup

For example, if I could do this:
typings install ol2=gist!https://gist.github.com/ca0v/2933117566546d84e4a8ff8b7651685d
Is that totally in-the-face of why typings exists?  Just trying to avoid the pain of learning how to version and setup the json.

Not sure I understand. Can you elaborate ?  There are file: and http(s): already

So sorry.  I finally figured it out.  Numerous problems on my end:
typings install ol2=https://gist.githubusercontent.com/ca0v/2933
117566546d84e4a8ff8b7651685d/raw/e40dc702d2dc6b713032bc19a1e2243ec32e10d7/ol2.d.
ts
Works!

Yes, you can't point it to a HTML file - make sure the file is a .d.ts file and it should be fine.


How to pick the best result model file when I run distributed training?

We run distributed training for ImageNet(inception-bn model) using 8~16 machines.
After finishing the training, several model files are created in each machine.
Then how can I pick the best model file?
Manually with log file? or other method?

models with the same epoch number should be the same.

Thanks for your response.
Then why does it have different validation accuracy at same epoch between machines?
If partitioned val data causes that result, how can I know the validation accuracy for entire val data?
I'm wondering if there is some args for using partitioned val data or entire val data.

Because in BatchNorm network, moving-average/moving-variance is not synced. Different model has different moving parameters. You are supposed to update the moving parameter at the very end of training.

Thanks for your response about BatchNorm.


response_format error

Confirm this is an issue with the Python library and not an underlying OpenAI API

[X] This is an issue with the Python library

Describe the bug
completion = client.chat.completions.create(
    messages=[
        {
            "role": "user",
            "content": "Can you generate an example json object describing a fruit?",
        }
    ],
    model="gpt-3.5-turbo",
    response_format={"type": "json_object"},
)

---------------------------------------------------------------------------
BadRequestError                           Traceback (most recent call last)
Cell In[5], line 1
----> 1 completion = client.chat.completions.create(
      2     messages=[
      3         {
      4             "role": "user",
      5             "content": "Can you generate an example json object describing a fruit?",
      6         }
      7     ],
      8     model="gpt-3.5-turbo",
      9     response_format={"type": "json_object"},
     10 )

File ~/anaconda3/envs/llama/lib/python3.10/site-packages/openai/_utils/_utils.py:299, in required_args.<locals>.inner.<locals>.wrapper(*args, **kwargs)
    297             msg = f"Missing required argument: {quote(missing[0])}"
    298     raise TypeError(msg)
--> 299 return func(*args, **kwargs)

File ~/anaconda3/envs/llama/lib/python3.10/site-packages/openai/resources/chat/completions.py:598, in Completions.create(self, messages, model, frequency_penalty, function_call, functions, logit_bias, max_tokens, n, presence_penalty, response_format, seed, stop, stream, temperature, tool_choice, tools, top_p, user, extra_headers, extra_query, extra_body, timeout)
    551 @required_args(["messages", "model"], ["messages", "model", "stream"])
    552 def create(
    553     self,
   (...)
    596     timeout: float | httpx.Timeout | None | NotGiven = NOT_GIVEN,
    597 ) -> ChatCompletion | Stream[ChatCompletionChunk]:
--> 598     return self._post(
    599         "/chat/completions",
    600         body=maybe_transform(
    601             {
    602                 "messages": messages,
    603                 "model": model,
    604                 "frequency_penalty": frequency_penalty,
    605                 "function_call": function_call,
    606                 "functions": functions,
    607                 "logit_bias": logit_bias,
    608                 "max_tokens": max_tokens,
    609                 "n": n,
    610                 "presence_penalty": presence_penalty,
    611                 "response_format": response_format,
    612                 "seed": seed,
    613                 "stop": stop,
    614                 "stream": stream,
    615                 "temperature": temperature,
    616                 "tool_choice": tool_choice,
    617                 "tools": tools,
    618                 "top_p": top_p,
    619                 "user": user,
    620             },
    621             completion_create_params.CompletionCreateParams,
    622         ),
    623         options=make_request_options(
    624             extra_headers=extra_headers, extra_query=extra_query, extra_body=extra_body, timeout=timeout
    625         ),
    626         cast_to=ChatCompletion,
    627         stream=stream or False,
    628         stream_cls=Stream[ChatCompletionChunk],
    629     )

File ~/anaconda3/envs/llama/lib/python3.10/site-packages/openai/_base_client.py:1063, in SyncAPIClient.post(self, path, cast_to, body, options, files, stream, stream_cls)
   1049 def post(
   1050     self,
   1051     path: str,
   (...)
   1058     stream_cls: type[_StreamT] | None = None,
   1059 ) -> ResponseT | _StreamT:
   1060     opts = FinalRequestOptions.construct(
   1061         method="post", url=path, json_data=body, files=to_httpx_files(files), **options
   1062     )
-> 1063     return cast(ResponseT, self.request(cast_to, opts, stream=stream, stream_cls=stream_cls))

File ~/anaconda3/envs/llama/lib/python3.10/site-packages/openai/_base_client.py:842, in SyncAPIClient.request(self, cast_to, options, remaining_retries, stream, stream_cls)
    833 def request(
    834     self,
    835     cast_to: Type[ResponseT],
   (...)
    840     stream_cls: type[_StreamT] | None = None,
    841 ) -> ResponseT | _StreamT:
--> 842     return self._request(
    843         cast_to=cast_to,
    844         options=options,
    845         stream=stream,
    846         stream_cls=stream_cls,
    847         remaining_retries=remaining_retries,
    848     )

File ~/anaconda3/envs/llama/lib/python3.10/site-packages/openai/_base_client.py:885, in SyncAPIClient._request(self, cast_to, options, remaining_retries, stream, stream_cls)
    882     # If the response is streamed then we need to explicitly read the response
    883     # to completion before attempting to access the response text.
    884     err.response.read()
--> 885     raise self._make_status_error_from_response(err.response) from None
    886 except httpx.TimeoutException as err:
    887     if retries > 0:

BadRequestError: Error code: 400 - {'error': {'message': "Invalid parameter: 'response_format' of type 'json_object' is not supported with this model.", 'type': 'invalid_request_error', 'param': 'response_format', 'code': None}}

To Reproduce

run the code block
error

Code snippets
completion = client.chat.completions.create(
    messages=[
        {
            "role": "user",
            "content": "Can you generate an example json object describing a fruit?",
        }
    ],
    model="gpt-3.5-turbo",
    response_format={"type": "json_object"},
)

OS
22.04.1-Ubuntu
Python version
3.10.13
Library version
1.3.5

The error you are encountering is due to the incorrect use of the response_format parameter in your code. The error message indicates that the response_format parameter set to {"type": "json_object"} is not supported with the model you are using, which is gpt-3.5-turbo.
The response_format parameter in the OpenAI API is used to specify how the response should be formatted. However, not all response formats are supported by all models. In this case, it seems that the gpt-3.5-turbo model does not support the json_object response format.
To resolve this issue, you can either remove the response_format parameter from your request or change the model to one that supports the json_object response format, if such a model exists. If you remove the response_format parameter, the response will default to the standard text format that is generally supported by all models.

Here is a revised version of your code without the response_format parameter:
completion = client.chat.completions.create(
messages=[
{
"role": "user",
"content": "Can you generate an example json object describing a fruit?",
}
],
model="gpt-3.5-turbo",
)

Thanks. It's the exact code from the Readme. Please update the Readme.

Oh sure.

You need to use model of gpt-4-1106-preview or gpt-3.5-turbo-1106.

It's the exact code from the Readme

The README of this project does not showcase response_format. Is there somewhere else you copied this code from?

You need to use model of gpt-4-1106-preview or gpt-3.5-turbo-1106.

It's the exact code from the Readme

The README of this project does not showcase response_format. Is there somewhere else you copied this code from?

Ah, thank you, sorry about that! We'll get that fixed up.

So, let me please ask for confirmation. response_format={"type": "json_object"} is only supported by model gpt-4-1106-preview ?

Only by gpt-4-1106-preview or gpt-3.5-turbo-1106.

response = client.chat.completions.create(
model="gpt-3.5-turbo-1106",
response_format={ "type": "json_object" },
messages=[
{"role": "system", "content": "You are a helpful assistant."},
{"role": "user", "content": "Who won the world series in 2020?"},
{"role": "assistant", "content": "The Los Angeles Dodgers won the World Series in 2020."},
{"role": "user", "content": "Where was it played?"}
]
)
openai.BadRequestError: Error code: 400 - {'error': {'message': "'messages' must contain the word 'json' in some form, to use 'response_format' of type 'json_object'.", 'type': 'invalid_request_error', 'param': 'messages', 'code': None}}

Even I chose to use model "gpt-3.5-turbo-1106", I still got the same error mentioned as Arvin-Feng. Is there anything wrong?

As the error message says, you must use the word "json" in your message.
For example, the last message in @Arvin-Feng 's code could be {"role": "user", "content": "Where was it played? Please respond with json."}

@rattrayalex Thanks. It works once I add the word "json" in the messages


Readability nits

Had a few style and readability nits to pick with a recent PR that I didn't see until after it was merged.
Locally, my changes pass on trunk and cassandra-3.0, and fail as expected (with a bunch of org.apache.cassandra.exceptions.WriteTimeoutException: Operation timed out - received only 0 responses. messages) on 3.0.5, which is consistent with the bug describtion in CASSANDRA-11891.

The while loop logic diff is hard to read. Have a look at the last 2 commits and their messages individually; that should be more understandable.

+1, though I should note we use that while loop logic in several places, and seems like reasonable Python to me?

It's reasonable, but difficult to read. This loop runs until replay is done. The test fails if this loop takes 2 minutes or longer. is more readable than This loop runs for 2 minutes, unless replay finishes. If the loop exits (except via the replay-finishing exit path), the test fails, IMO.

Yes, you are correct.

Thanks for the review. Merging.


Add copy() method to primitives.

Returns a copy of a primitive instance with the same geometry settings.
Copy should not copy the parent, children, transform, material or name attributes.
Copy could take arguments at a later date for quickly setting material, transform etc...

On reflection consider how this should work with the instance behaviour of Mesh. Perhaps it would be better to generalise this somehow?

Generalised instancing to all primitives. Closed by 093145c286815839bc037ae7f65c3c059da53386.


Update go to 1.22.5

What this PR does / why we need it:
Update go to 1.22.5
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note:

Lakom application and lakom extension controller are now build with Go version 1.22.5

Should the go version be changes in go.mod as well?

Should the go version be changed in go.mod as well?

Ah, just found out that's the minimum required go version, so it should be fine


Compiling error: Failed to load PowerToys...

Microsoft PowerToys version
0.79
Installation method
GitHub
Running as admin
No
Area(s) with issue?
General
Steps to reproduce

Copy the repository
Follow dev docs https://github.com/microsoft/PowerToys/blob/main/doc/devdocs/readme.md
Trying to compile it
Get following error in result variable: HRESULT_FROM_WIN32(ERROR_MOD_NOT_FOUND): Specified module not found 	winrt::hresult
Casually ignore it by unchecking Visual Studio checkbox about ignoring the error
Now, as you try to compile you'll get whole bunch of errors in popups (shown in pinned images)

✔ Expected Behavior
To build and work
❌ Actual Behavior
It didn't
Other Software
Visual Studio 17.9

Hi @arany114 , do you try to build the whole solution as specified in the docs?
From the errors, it seems the individual modules haven't been built yet.
Hope this helps.
/needinfo

Hi @arany114 , do you try to build the whole solution as specified in the docs? From the errors, it seems the individual modules haven't been built yet. Hope this helps. /needinfo

Whoops, was i supposed to build before compiling? so project does not support quick debug? What i did i was just running 'runner' right after opening solution. Or do i have to build modules individually if it's even possible?

You need to build all modules. (Right-click solution and build so it builds whole solution, or Solution in the top menu and Build Solution)
Just pressing debug won't work because the runner expects the modules to be built. Hope this helps.
/needinfo