Raft 56

Fix for #56

// rebased it (thanks for write access to the repo @colin-scott, will make rebasing during merging much easier)


Add support for logging in cpp backend

See #56

Logging is working now in the cpp backend. Some notable differences:

Odom is not yet supported (see also #130). However, this is properly documented now.
The generic log topics use LogDataCustom, rather than individual std_msgs (see discussion in #56). I think we would need to unify this behavior in the cflib backend as well. @knmcguire, let me know if I should do that here, or if that will be a separate PR.

We can do that unification as a separate PR, that is probably better


Text color in IPAD same as background color

Hi, the picker is not working on IPAD, it seems that the textcolor is the same as background. I have tried to change it with Control.SetTitleColor (UIKit.UIColor.Black, UIControlState.Normal ); but no luck.

I cannot reproduce this issue in sample project https://github.com/XAM-Consulting/FreshEssentialsSample
In that sample, the ItemsSource should be bindable, click "Change Source of Picker" button will show different source.

So late Jesse. Thanks for all.


Missing --once param in start.ps1

You describe how to Create and build the Dockerfile for the Windows images. In this section you offer the start script start.ps1 to include as an entrypoint for the docker image. Later in the section Start the image you state:

If you want a fresh agent container for every pipeline run, pass the --once flag to the run command.

But this will actually not work with your provided start script because there is neither a param to passed to the start.ps1 nor to the run.cmd (start.ps1 line 59).
Please provide an accurate start script for agents running on Windows containers, like you do for Linux containers. Thanks!

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

ID: b31d4f6c-d16d-2ba1-1294-6aa94e328046
Version Independent ID: 88aec599-4410-697d-1002-1c41b1c17bb6
Content: Run a self-hosted agent in Docker - Azure Pipelines
Content Source: docs/pipelines/agents/docker.md
Service: azure-devops-pipelines
Sub-service: azure-devops-pipelines-agents
GitHub Login: @steved0x
Microsoft Alias: sdanie

@clma91 -- Claudio, thank you for your thorough feedback.  Please consider these resources:

Azure DevOps Services
Azure DevOps on Stack Overflow

@steved0x -- Steve, please look into this issue.

@WilliamAntonRohm thanks for the quick response. I just noticed something in addition that might be linked to this issue.
In contrast to the Linux agents, Windows agents won't de-register from Azure DevOps if the container is stopped/removed by the container runtime (e.g. Docker).
So, if I start a Windows agent in a container, it will register at Azure DevOps. If I stop the container after a while, it will not de-register at Azure DevOps but will obviously become offline as it looses connection. But it stays forever in Azure DevOps until it will be started again under the exact same agent name.
On the other hand, a Linux agent started in a container de-registers itself right away after it is stopped.
As a workaround I adapted for the start.ps1 with the .\run.cmd --once call. I noticed that after a pipeline ran with the --once flag set, it worked perfectly fine and de-registered the agent correctly:
2023-03-09 13:23:37Z: Listening for Jobs
2023-03-09 13:23:39Z: Running job: build_application
2023-03-09 13:25:21Z: Job build_application completed with result: Succeeded
Cleanup. Removing Azure Pipelines agent...
Removing agent from the server
Connecting to server ...
Succeeded: Removing agent from the server
Removing .credentials
Succeeded: Removing .credentials
Removing .agent
Succeeded: Removing .agent

@clma91 -- Claudio, thank you for your followup, research, and solution.
cc @steved0x for awareness


Remove  unnecessary database lock on discount during checkout complete mutation

I want to merge this change because removing unnecessary database lock on discount in checkout complete.

Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

Task linked: SALEOR-4482 Unnecessary database lock on discount during checkout complete mutation


Add ability to hide or ignore automatic collections

ChimeraOS automatically adds the "ChimeraOS Playable" and "ChimeraOS Verified" collections. Despite deleting them, they show back up on the next Steam restart.
It would be helpful to have the ability to remove these completely. I appreciate and understand why they're there, but I personally have no need for them nor want them in my list of collections.

Closing since #282 got moved here and I didn't notice.

Closing since #282 got moved here and I didn't notice.

Actually closing.

🤦♂


change the line interpolation

Change line from cardinal interpolation which looks like this.

To step-before interpolation which looks like this

Also here are some unlikely animal friends

:+1:


Hi,
It seems like you might need to adjust the time.sleep values (see Adjusting time.sleep https://github.com/HCA97/jw_bot/issues/2#issuecomment-1698020617) or you might have selected the Bluestack client incorrectly (see How to select Blue Stack https://github.com/HCA97/jw_bot/issues/2#issuecomment-1698020617). Have you checked the comment or the issue in general? In this discussion, you can find helpful tips and tricks on how to use jw_bot. Another person had similar problems with running jw_bot, but they managed to resolve it now. Our conversation might also provide some assistance for your issue.

Note Please keep in mind that depending on your hardware and internet speed, the game might run slower for you. Therefore, you may need to adjust the time.sleep parameters. https://github.com/HCA97/jw_bot/issues/2#issuecomment-1703087665,

Note Additionally, it's important to accurately select the Bluestack client since the button locations are hardcoded. If you select it incorrectly, the bot might get stuck at certain steps.

P.S. Regarding the Bluestack settings, I used the default shortcuts and set the CPU and RAM to the highest level.

Hi,
It seems like you might need to adjust the time.sleep values (see Adjusting time.sleep #2 (comment)) or you might have selected the Bluestack client incorrectly (see How to select Blue Stack #2 (comment)). Have you checked the comment or the issue in general? In this discussion, you can find helpful tips and tricks on how to use jw_bot. Another person had similar problems with running jw_bot, but they managed to resolve it now. Our conversation might also provide some assistance for your issue.

Note Please keep in mind that depending on your hardware and internet speed, the game might run slower for you. Therefore, you may need to adjust the time.sleep parameters. #2 (comment),

Note Additionally, it's important to accurately select the Bluestack client since the button locations are hardcoded. If you select it incorrectly, the bot might get stuck at certain steps.

P.S. Regarding the Bluestack settings, I used the default shortcuts and set the CPU and RAM to the highest level.

I think you right I have to adjust time.sleep but that many time.sleep how I know I mean what the line of time.sleep that use for  waiting the bot detect when click on dino or supply drop or when them ramdomly click and wait for detect ?

Alright, it seems like you're encountering a similar issue to what someone else faced before https://github.com/HCA97/jw_bot/issues/2#issuecomment-1703087665. The problem lies in the transition from launching the drone to shooting the dino. There's a check in place to see if there's a loading screen during this transition. This is done by comparing two frames, as you can see in the code here. If there's significant lag, this check might get messed up and you'll end up stuck in the dino shooting state.
I suggest increasing the values for these two time.sleep parameters by adding a second or more:
https://github.com/HCA97/jw_bot/blob/552902657dd8d3536420480daa1a2cbf35d3473a/jw_bot.py#L809
https://github.com/HCA97/jw_bot/blob/552902657dd8d3536420480daa1a2cbf35d3473a/jw_bot.py#L799
Additionally, try increasing these two time.sleep parameters by adding half a  second or more:
https://github.com/HCA97/jw_bot/blob/552902657dd8d3536420480daa1a2cbf35d3473a/jw_bot.py#L782
https://github.com/HCA97/jw_bot/blob/552902657dd8d3536420480daa1a2cbf35d3473a/jw_bot.py#L791
If this doesn't fix the issue, try increasing all the time.sleep values in the collect_dino function.
If you still have the problem you can have a look on shoot_dino function and increase the time.sleep values.
Please note that setting up the bot initially can be a bit tricky because Jurassic World is a very laggy game and the amount of lag everyone faces is different.

Alright, it seems like you're encountering a similar issue to what someone else faced before #2 (comment). The problem lies in the transition from launching the drone to shooting the dino. There's a check in place to see if there's a loading screen during this transition. This is done by comparing two frames, as you can see in the code here. If there's significant lag, this check might get messed up and you'll end up stuck in the dino shooting state.
I suggest increasing the values for these two time.sleep parameters by adding a second or more:
https://github.com/HCA97/jw_bot/blob/552902657dd8d3536420480daa1a2cbf35d3473a/jw_bot.py#L809
https://github.com/HCA97/jw_bot/blob/552902657dd8d3536420480daa1a2cbf35d3473a/jw_bot.py#L799
Additionally, try increasing these two time.sleep parameters by adding half a second or more:
https://github.com/HCA97/jw_bot/blob/552902657dd8d3536420480daa1a2cbf35d3473a/jw_bot.py#L782
https://github.com/HCA97/jw_bot/blob/552902657dd8d3536420480daa1a2cbf35d3473a/jw_bot.py#L791
If this doesn't fix the issue, try increasing all the time.sleep values in the collect_dino function.
If you still have the problem you can have a look on shoot_dino function and increase the time.sleep values.
Please note that setting up the bot initially can be a bit tricky because Jurassic World is a very laggy game and the amount of lag everyone faces is different.

Almost are better now, but sometimes click dino and show nothing there
or click dino then show launch but go out not launch again and go other
then comeback and real launch this thing happen 1-2 times.
How to make the detect launch(go shoot dino) better ?

Almost are better now, but sometimes click dino and show nothing there

Sometimes the dino might be mistakenly detected as a supply drop. This can happen from time to time, so it's not unusual.

click dino then show launch but go out not launch again and go other

Hmm, I'm not entirely clear on this. It could still be related to the time.sleep settings. I recommend trying to increase these two parameters:

Additionally, try increasing these two time.sleep parameters by adding half a second or more:
https://github.com/HCA97/jw_bot/blob/552902657dd8d3536420480daa1a2cbf35d3473a/jw_bot.py#L782
https://github.com/HCA97/jw_bot/blob/552902657dd8d3536420480daa1a2cbf35d3473a/jw_bot.py#L791

If the issue persists, it might be related to how you're selecting the Bluestack client. Please review the How to select Blue Stack and Running BlueStacks in Non-Full Screen Mode sections in this comment. This might provide some further insight.


Gamepad mappings are broken in the latest beta

Godot version:
Godot 3.2.2 beta 2
OS/device including version:
Windows 10
Xbox One gamepad
Issue description:
It seems like face buttons and D-pad are swapped in the latest beta 2 for version 3.2.2. I haven't tried beta 1, but the issue was not present in version 3.2.1 stable
Steps to reproduce:
Set up a simple project with some input mappings that use D-pad and some that use face buttons.
Minimal reproduction project:

Hi. I have exactly the same problem. My gamepad is a generic one from logitech. the buttons and the joystick do not work, or they do it, with a function that does not correspond to them. I use hp laptop with windows 10. As you said, in version 3.2.1 it did not happen. Greetings.

Duplicate of #38554, fixed by #38557.


fix: use prisma generate --no-engine and reinstate prompts script

Description

In Vercel builds where we are using Prisma Accelerate we should be using the --no-engine flag for prisma generate
I think that potentially in the move to the new repo / Vercel project, or somewhere prior to that we seem to have lost this configuration flag
This adds a new turbo command db-generate:no-engine which is in the dependsOn array for turbo build
We also seem to have lost the prompt generation script so we are possibly reliant on the application to create new prompt variants at runtime instead of build time. This also reinstates that script in the build pipeline
In the prompt variant generation script, this also now upserts the "app" rows if they are missing in local dev

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


fix spec test broken by 92f15de9b052cd531666b922389feedee622a6b4 in #49

fix spec test broken by 92f15de9b052cd531666b922389feedee622a6b4 in #49

I have a manheim branch running in my fork that incorporates:

PR #40
PR #41
PR #53
PR #54
some additional rspec test fixes.

I currently have that tagged as pr40,41,53,54_and_spec_fixes. Feel free to use if you wish.


⚠ Samsung has degraded performance

In cadaf57, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 3160 ms

Resolved: Samsung performance has improved in 9b07413.


Fails to exclude source files from HQRM tests

Details of the scenario you tried and the problem that is occurring
When running the HQRM test for a repo using Pester 4 it seems it fails to exclude source files that are excluded in the build configuration.
Se more information in PR https://github.com/dsccommunity/SharePointDsc/pull/1440.
Steps to reproduce the problem
In a source file suppress the rule PSAvoidUsingWriteHost, for example in the source file in the relative path SharePointDsc/Modules/SharePointDsc.Reverse/SharePointDsc.Reverse.psm1:
[CmdletBinding()]
[Diagnostics.CodeAnalysis.SuppressMessageAttribute('PSAvoidUsingWriteHost', '', Scope = 'Function', Target = '*')]
param ()

Then try to exclude the relative path in the build configuration:
DscTest:
  ExcludeSourceFile:
    - output
    - 'SharePointDsc/Modules/SharePointDsc.Reverse/SharePointDsc.Reverse.psm1'

Expected behavior
Should exclude the file and the tests should not fail.
Current behavior
Does not exclude the file and the test fails due to suppressing a required rule.
Suggested solution to the issue
It should exclude the source files according to the code but it doesn't happen. Haven't debugged so not sure where the issue is.
It should filter out the excluded files here:
https://github.com/dsccommunity/DscResource.Test/blob/14acda0bc0a6a9698bfb8fd41da42adfe08b09f0/source/Tests/QA/PSSAResource.common.v4.Tests.ps1#L82
Using the filter: https://github.com/dsccommunity/DscResource.Test/blob/14acda0bc0a6a9698bfb8fd41da42adfe08b09f0/source/Private/WhereSourceFileNotExcluded.ps1#L1
The operating system the target node is running
Windows and also using Pester 4
Version and build of PowerShell the target node is running
PowerShell 5.1
Version of the module that was used
v0.16.3

$ExcludeSourceFile is not passed down to/available in the filter. I think the logic on the if statement is broken too, but without the exclusions being seen in the filter it never does anything.

Apparently it works, and maybe the reason was the single quotes 🤔  Closing this and it can be reopen if there is more information in the future.


fix #377, use python3 in /py/ folder

let's try to use python3 FTW

This change is

I have one question: how to ensure that the code is python3-ready now?

Coverage remained the same at 31.746% when pulling 30f2583635102f884f80a0347579723d1e5a5d9c on Jimexist:use-py3 into 0de450f866b4af88e580eb59c01da7c65aacfddb on tensorflow:master.

I have one question: how to ensure that the code is python3-ready now?

i am not sure, but its python, let's
try:
   use python3
catch:
   sorry

My suspicion is that this isn't really changing whether we are using python3.
The scripts in py/ are used as part of our test infrastructure. Basically we run an Argo workflow in which each step is a python script. This is described here.
The workflow is defined here. Each step just calls python so whether we are using python2.7 or python3 depends on that container and what it points to.
I'm guessing python points to 2.7. You could easily check by downloading the image and creating a shell.
So I think what we need to do is

Ensure Python3 is installed in the container

The container is defined here
If we need to install python3 lets not change the version used by python as that will affect kubeflow/kubeflow and we don't want to do that until we know those tests will pass.

Modify the [workflow]here so that it explicitly uses Python3.

At this point the presubmits will run using Python3. So if the presubmits pass then we know the code is python3 compatible and we're good to go.

@jlewi let me try changing that as well

waiting for https://github.com/kubeflow/testing/pull/15 to be merged and then this PR can be updated.

@jlewi @gaocegege now that https://github.com/kubeflow/testing/pull/15 is merged let me try to use python3 in this PR.

INFO|2018-02-10T03:16:35|/src/kubeflow/testing/py/kubeflow/testing/run_e2e_workflow.py|129| URL for workflow: http://testing-argo.kubeflow.io/timeline/kubeflow-test-infra/tf-k8s-presubmit-378-fb781c9-513-30c8;tab=workflow
Traceback (most recent call last):
  File "/usr/lib/python2.7/runpy.py", line 174, in _run_module_as_main
    "__main__", fname, loader, pkg_name)
  File "/usr/lib/python2.7/runpy.py", line 72, in _run_code
    exec code in run_globals
  File "/src/kubeflow/testing/py/kubeflow/testing/run_e2e_workflow.py", line 232, in <module>
    success = main()
  File "/src/kubeflow/testing/py/kubeflow/testing/run_e2e_workflow.py", line 222, in main
    return run(args, file_handler)
  File "/src/kubeflow/testing/py/kubeflow/testing/run_e2e_workflow.py", line 130, in run
    status = github_status.GithubStatus()
  File "/src/kubeflow/testing/py/kubeflow/testing/github_status.py", line 51, in __init__
    Github.__init__(self)
  File "/src/kubeflow/testing/py/kubeflow/testing/github_status.py", line 24, in __init__
    raise Exception('Missing environment variable GIT_TOKEN')
Exception: Missing environment variable GIT_TOKEN

should I be worried about this?

@Jimexist its a different piece of the test infrastructure. Its run.sh. You need to change that to Python3 but if you do it will probably break because the underlying container image hasn't been rebuilt yet.
The tests were broken because kubeflow/testing broke.

/test all

/test all

The release step is failing
Traceback (most recent call last):
 File "/usr/lib/python3.5/runpy.py", line 184, in _run_module_as_main
   "__main__", mod_spec)
 File "/usr/lib/python3.5/runpy.py", line 85, in _run_code
   exec(code, run_globals)
 File "/mnt/test-data-volume/tf-k8s-presubmit-378-fb781c9-521-e668/src/tensorflow/k8s/py/release.py", line 19, in <module>
   import yaml
ImportError: No module named 'yaml'

It might be easier to put this PR on hold until

kubeflow/testing#22 is fixed
tensorflow/k8s is moved to kubeflow/tf-operator

At that point I can give you permission to rebuild  gcr.io/kubeflow-ci/test-worker:latest which is the Docker image used in the Argo workflow steps. You could then rebuild the image as needed and verify that the tests for this PR pass.

to be fixed with https://github.com/kubeflow/testing/pull/34

Per #404 it seems like we are now using Python3 in the test worker. As soon as the test is fixed (I'm working on it). We should be able to sync and continue work on this PR.

Pull Request Test Coverage Report for Build 904

0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
25 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.1%) to 31.868%

Files with Coverage Reduction
New Missed Lines
%

pkg/apis/tensorflow/v1alpha1/defaults.go
6
100.0%

pkg/util/util.go
19
70.83%

Totals

Change from base Build 900:
-0.1%

Covered Lines:
580

Relevant Lines:
1820

💛  - Coveralls

/retest

/retest

@jlewi any idea how to resolve this kind of errors?
{"kind":"Status","apiVersion":"v1","metadata":{},"status":"Failure","message":"pods \"tf-k8s-presubmit-378-fb781c9-521-e668-1610936534\" not found","reason":"NotFound","details":{"name":"tf-
k8s-presubmit-378-fb781c9-521-e668-1610936534","kind":"pods"},"code":404}

kubeflow-tf-operator-presubmit has a similar error to #417
http://testing-argo.kubeflow.io/timeline/kubeflow-test-infra/kubeflow-tf-operator-presubmit-tfjob-e2e-378-ab6432e-107-c428;tab=workflow;node=kubeflow-tf-operator-presubmit-tfjob-e2e-378-ab6432e-107-c428-519405431:logs
I think it is caused by your code.

/retest

close, see https://github.com/kubeflow/tf-operator/pull/439


Added GameMaker icon

Closes https://github.com/PKief/vscode-material-icon-theme/pull/1555

@PKief please reopen 😨

@PKief help

@okineadev the button is disabled, becaue this PR has no commits and no changes

Could you please create a new PR for it?

or just push something to the mentioned branch in your repo...

or just push something to the mentioned branch in your repo...

I will try to


Maintain FPGA Compatibility

Currently, some bare-metal tests have issues running on the Xilinx Virtex Utralscale+ board. We have to debug these issues using Vivado's post-synth simulation and add compatibility tests to our regression suite. (example: https://github.com/black-parrot/black-parrot/pull/706)

@Shashank-Vijay is working on post-synth vivado sims, not with the Xilinx IPs though. The bug in #706 would have been caught without the IPs, but I can imagine bugs which would not

@Shashank-Vijay is working on post-synth vivado sims, not with the Xilinx IPs though. The bug in #706 would have been caught without the IPs, but I can imagine bugs which would not

And to be clear, we are okay with Vivado synthesis and VCS simulation.
Vivado is 10X slower than VCS.
On Fri, Jan 8, 2021 at 2:10 PM Dan Petrisko<EMAIL_ADDRESS>wrote:

@Shashank-Vijay https://github.com/Shashank-Vijay is working on
post-synth vivado sims, not with the Xilinx IPs though. The bug in #706
https://github.com/black-parrot/black-parrot/pull/706 would have been
caught without the IPs, but I can imagine bugs which would not
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/black-parrot/black-parrot/issues/728#issuecomment-757024103,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEFG5AE2MWBMBII2I4I3VDDSY57G7ANCNFSM4V23ZD4Q
.

And to be clear, we are okay with Vivado synthesis and VCS simulation.
Vivado is 10X slower than VCS.
On Fri, Jan 8, 2021 at 2:10 PM Dan Petrisko<EMAIL_ADDRESS>wrote:

@Shashank-Vijay https://github.com/Shashank-Vijay is working on
post-synth vivado sims, not with the Xilinx IPs though. The bug in #706
https://github.com/black-parrot/black-parrot/pull/706 would have been
caught without the IPs, but I can imagine bugs which would not
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/black-parrot/black-parrot/issues/728#issuecomment-757024103,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEFG5AE2MWBMBII2I4I3VDDSY57G7ANCNFSM4V23ZD4Q
.

And to be clear, we are okay with Vivado synthesis and VCS simulation.
Vivado is 10X slower than VCS.

That's interesting! Probably want to support both, but run most regressions with VCS for speed

And to be clear, we are okay with Vivado synthesis and VCS simulation.
Vivado is 10X slower than VCS.

That's interesting! Probably want to support both, but run most regressions with VCS for speed

Just in terms of spending time on things, bigger fish to fry if it takes a
lot of work to get Vivado sim going -- people can use Verilator
On Fri, Jan 8, 2021 at 2:50 PM Dan Petrisko<EMAIL_ADDRESS>wrote:

And to be clear, we are okay with Vivado synthesis and VCS simulation.
Vivado is 10X slower than VCS.
That's interesting! Probably want to support both, but run most
regressions with VCS for speed
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/black-parrot/black-parrot/issues/728#issuecomment-757036900,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEFG5AG3UV3MVENFF5HL7QLSY6D3TANCNFSM4V23ZD4Q
.

Just in terms of spending time on things, bigger fish to fry if it takes a
lot of work to get Vivado sim going -- people can use Verilator
On Fri, Jan 8, 2021 at 2:50 PM Dan Petrisko<EMAIL_ADDRESS>wrote:

And to be clear, we are okay with Vivado synthesis and VCS simulation.
Vivado is 10X slower than VCS.
That's interesting! Probably want to support both, but run most
regressions with VCS for speed
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/black-parrot/black-parrot/issues/728#issuecomment-757036900,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEFG5AG3UV3MVENFF5HL7QLSY6D3TANCNFSM4V23ZD4Q
.

Just in terms of spending time on things, bigger fish to fry if it takes a
lot of work to get Vivado sim going -- people can use Verilator

True, although one major use case is students working with just Vivado for FPGA-based classes. In any case, the idea is for the makefile flow to support orthogonal synth and sim by switching out simulation flists, so it shouldn't be too bad.  But point taken, won't spend an inordinate amount of time there

Just in terms of spending time on things, bigger fish to fry if it takes a
lot of work to get Vivado sim going -- people can use Verilator

True, although one major use case is students working with just Vivado for FPGA-based classes. In any case, the idea is for the makefile flow to support orthogonal synth and sim by switching out simulation flists, so it shouldn't be too bad.  But point taken, won't spend an inordinate amount of time there

Just in terms of spending time on things, bigger fish to fry if it takes a
lot of work to get Vivado sim going -- people can use Verilator

From talking to Dustin and Jon, not worth supporting vivado sim.  Too many bugs!

Just in terms of spending time on things, bigger fish to fry if it takes a
lot of work to get Vivado sim going -- people can use Verilator

From talking to Dustin and Jon, not worth supporting vivado sim.  Too many bugs!

Yep. There is one use case though that is not covered by Verilator, which
is if the Xilinx IP blocks are encrypted. But cross that bridge later.
M
On Sat, Jan 9, 2021 at 12:44 PM Dan Petrisko<EMAIL_ADDRESS>wrote:

Just in terms of spending time on things, bigger fish to fry if it takes a
lot of work to get Vivado sim going -- people can use Verilator
From talking to Dustin and Jon, not worth supporting vivado sim. Too many
bugs!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/black-parrot/black-parrot/issues/728#issuecomment-757364550,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEFG5ADLKLAB7S3KMCYFANLSZC5ZJANCNFSM4V23ZD4Q
.

Yep. There is one use case though that is not covered by Verilator, which
is if the Xilinx IP blocks are encrypted. But cross that bridge later.
M
On Sat, Jan 9, 2021 at 12:44 PM Dan Petrisko<EMAIL_ADDRESS>wrote:

Just in terms of spending time on things, bigger fish to fry if it takes a
lot of work to get Vivado sim going -- people can use Verilator
From talking to Dustin and Jon, not worth supporting vivado sim. Too many
bugs!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/black-parrot/black-parrot/issues/728#issuecomment-757364550,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEFG5ADLKLAB7S3KMCYFANLSZC5ZJANCNFSM4V23ZD4Q
.


Clean up and documentation

@jakevdp @mmp2

[ ] Should we move covar_plotter.py somewhere inside the package? Should consider writing an example using it as well.
[ ] example.py & dev_example.py should be removed. Should turn example.py into an iPython tutorial that we can host on the website. Perhaps we can do this as we teach the API to Grace?
[ ] examples/ and astrodemo/ should be removed and replaced with the example from the previous step
[ ] The example usage & non-auto doc documentation for: Geometry, Adjacency, Laplacian, Affinity & all the embeddings need to be fixed with respect to new API changes.
[ ] The sphinx build needs to be re-run when API changes are made.
[ ] Add some example plots to the website

Should we move covar_plotter.py somewhere inside the package?
Dear all, I thought of this several times, and decided not to. Because
this function is not really integrated with the package, it's just an
expedient we use for convenience. If we ever write a visualization module,
then it should belong there. But for now,given that there is no other
postprocessing or visualization functionality there, I think we can leave
it out.
Marina
On 3/4/16, 9:27 AM, "James McQueen"<EMAIL_ADDRESS>wrote:

@jakevdp https://github.com/jakevdp
@mmp2 https://github.com/mmp2
-[ ] Should we move covar_plotter.py somewhere inside the package? Should
consider writing an example using it as well.
-[ ] example.py & dev_example.py should be removed. Should turn
example.py into an iPython tutorial that we can host on the website --
perhaps we can do this as we teach the API to Grace?
-[ ] examples/ and astrodemo/ should be removed and replaced with the
example from the previous step
-[ ] The example usage & non-auto doc documentation for: Geometry,
Adjacency, Laplacian, Affinity & all the embeddings need to be fixed with
respect to new API changes.
-[ ] The sphinx build needs to be re-run when API changes are made.
-[ ] Add some example plots to the website
�
Reply to this email directly or
view it on GitHub https://github.com/mmp2/megaman/issues/25.


Implement @behaviour for Routes.Repo

Summary of changes
Asana Ticket: Define a RoutesApi behaviour for V3Api.Routes

This draft PR is an attempt to achieve what was discussed in https://github.com/mbta/dotcom/pull/665


SignalR GA - Add more properties to the models, add usage API and complete tests

Description
The change is based on the spec PR: https://github.com/Azure/azure-rest-api-specs/pull/3686 . We plan to release this version as the first GA version <IP_ADDRESS>.
Changes:

Add version to the SignalRInstance model
Add connection strings to the SignalRKeys model
Add usage operations for quota limit related query
Add tests

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] Please add REST spec PR link to the SDK PR
[x] I have read the contribution guidelines.
[x] The pull request does not introduce breaking changes.

General Guidelines

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

SDK Generation Guidelines

[x] If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
[x] The generate.cmd file for the SDK has been updated with the version of AutoRest, as well as the commitid of your swagger spec or link to the swagger spec, used to generate the code.
[x] The *.csproj and AssemblyInfo.cs files have been updated with the new version of the SDK.

Hi @dsgouda , we plan to stay in preview before Ignite and work on stable api-version and SDK release after that.
Could you please help to reopen the PR? I have prepared the changes according to your review comments locally and will update after reopening.
Thanks,

Thanks for confirming. Reopening PR. Will review once changes are pushed

Updated the code according to the review comments and rebased to the latest psSdkJson6 branch.


Unable to get mobile responsiveness working

Hello,
I am building a web app with BeerCSS. I love how easy it is to get off the ground quickly! It looks great on desktop, but unfortunately does not seem to be responsively scaled on mobile. I am using firefox, but I also see the same issue in chromium browsers.
App: https://codemu.ch/cookbook
Source Code: https://github.com/cmvanb/cookbook
Base HTML template: https://github.com/cmvanb/cookbook/blob/master/cookbook/templates/base.html
Custom CSS: https://github.com/cmvanb/cookbook/blob/master/cookbook/static/main.css
When I inspect the HTML/CSS I don't see any major differences in structure. I must be missing something to enable mobile responsiveness, any idea what I'm doing wrong?
Here is a visual comparison:

App
BeerCSS

best regards,
Casper

Hi @cmvanb,
it seems like you forgotten to set the viewport on the website. Just add this line to your <head> and it should work:
<meta name="viewport" content="width=device-width, initial-scale=1, maximum-scale=1">

Without that, beercss and many other CSS-Frameworks won't work, because without viewport, the browser doesn't know what width/height to calculate with. More information here: https://developer.mozilla.org/en-US/docs/Web/CSS/Viewport_concepts
Also I would recommend always to start with the template in the README, than you should be on the safe side: https://github.com/beercss/beercss#html

That works perfectly. Thank you for your quick and informative response!


fix nightly package name

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


Faker::Internet.safe_email makes bad emails for some locales

Faker::Config.locale = "ja"

Faker::Internet.safe_email
# =><EMAIL_ADDRESS>
Faker::Config.locale = "ko"

Faker::Internet.safe_email
# =><EMAIL_ADDRESS>

:+1: I just experienced this myself. Is there a plan to get a fix in and released?

I'm having the same problem here.
This is what I get in the rails console
Faker::Config.locale
# => :"zh-TW"

Faker::Internet.email
# =><EMAIL_ADDRESS>
Faker::Config.locale='en'

Faker::Internet.email
# =><EMAIL_ADDRESS>

Fixed here:
https://github.com/stympy/faker/pull/525

@jphastings it appears that this issue was resolved in #525 - are you satisfied with that or is there a reason to keep this issue open?

It looks good now! Thanks for the prod :)


Content warning not intuitive

When using networks that support content warnings/folds (e.g. Mastodon), there's presently no way to keep sensitive images or warned content from being displayed automatically.  Likewise, creating a new Mastodon post with a CW is not intuitive.

Yes, really. There is no special support of Mastodon's "CW" feature. Just more generic "Note Name/Title" notion is adapted to be used with Mastodon.
Mastodon doesn't support "Name/Title", and I figured out that having mapping of more generic "Name" (which is used in Pump.io and ActivityPub) to Mastodon's "CW" is better than nothing...

I agree, it's better than nothing on submitting (perhaps labeling the fields as such when posting to Mastodon and any other networks that support CWs would help).  But on the reading side, not having CWs collapsed by default (or at least a way to enable collapsing them) is problematic due to NSFW posts.

Do you think that "Collapse notes with titles" option (initially without identifying, if the Title is a CW or a simply Name/Title) and "show" button in collapsed notes ("toots"...) in AndStatus Settings could help?

Mostly, yes.  If it's something that can be set as a default, even better.
Obviously show the title even if it's CW stuff, but collapse the rest.
Something similar should happen with media on posts with sensitive media
that isn't CW.  By comparison, Tusky get sensitive media and content
warning right but can't hang on phones with lots of data dropouts.

This is anther case, as I understand:

Something similar should happen with media on posts with sensitive media that isn't CW...

I didn't understand fully, starting with this phrase.
How do I know if the media is sensitive if the note doesn't have any title/name/CW text?

I believe it's a subset of the same case, I don't fully understand the
internal workings of Mastodon, but it is possible to send a toot that has
the images hidden and the text not.
On Thu, May 2, 2019, 09:40 Yuri Volkov<EMAIL_ADDRESS>wrote:

This is anther case, as I understand:
Something similar should happen with media on posts with sensitive media
that isn't CW...
I didn't understand fully, starting with this phrase.
How do I know if the media is sensitive if the note doesn't have any
title/name/CW text?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/andstatus/andstatus/issues/507#issuecomment-488700212,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAWDFBS6HNQEJXZWCAD7CPTPTL4OHANCNFSM4HJRTPJA
.

The second case with media only is not the the same, if we don't extract Mastodon - specific "sensitive" flag from the Note...

OK, so I see that instead of a hack we really need to add separate "Sensitive" property that could be used both in Mastodon and at least in ActivityPub.
See this thread regarding on ActivityPub: https://github.com/w3c/activitypub/issues/231 In ends with the proposed extension: "sensitive" property" (see https://www.w3.org/wiki/Activity_Streams_extensions#as:sensitive_property )

Released in v.50.01


[Add]

Details
The Folded Paper Button is a distinctive and visually appealing UI element that adds depth and realism to our project's user interface.
@Spyware007 Please assign it to me.
Screenshots
No response
Type of Contribution

[ ] Update to an existing Animated Button
[X] Adding a new Button
[ ] Resolving a bug
[ ] Proposal to the Repository
[ ] Changes related to documentation or README.md
[ ] Other Changes

Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I am willing to work on this issue
[X] I am a GSSoC'23 contributor

@Spyware007 Please assign it to me


Inputs not working outside of main viewport in docking branch.

In the docking branch, if the main veldrid window is out of focus, scrolling right click etc does not work in ImGui windows outside of the the viewport.

I'll look into it, but it's not really a high priority since docking_branch is just a Demo/Proof-of-Concept.

@pyboers I'm not familiar with Veldrid, however I am working with an event based windowing system that needs a similar controller setup as the Veldrid example.  You have to wire up input event routing inside each of the created windows independently of the main window and send them through ImGui's IO event system.  Keep in mind that when using viewports ImGui expects the input events to be in raw screen space coordinate, so you might have to do some coordinate conversions.
@zaafar as an aside, why hasn't the docking branch been pulled up yet?  I'm pretty sure viewports and docking have been merged into imgui master by now.  I don't think there's any reason it shouldn't be in ImGui.NET either in terms of "it's supported, you have to do the legwork to make it work for your use case".

@zaafar as an aside, why hasn't the docking branch been pulled up yet? I'm pretty sure viewports and docking have been merged into imgui master by now. I don't think there's any reason it shouldn't be in ImGui.NET either in terms of "it's supported, you have to do the legwork to make it work for your use case".

That's essentially what the status is. The docking branch in this repo just has extra code in the sample application to enable multi-viewports and drive the extra windows, but it's unstable/experimental enough that it has always stayed in its own branch.  You should be able to do roughly the same thing from the master branch, or from the NuGet package, although IIRC there's a few extra things that need to be wrapped that exist in the native library that aren't in the master branch.

You should be able to do roughly the same thing from the master branch, or from the NuGet package, although IIRC there's a few extra things that need to be wrapped that exist in the native library that aren't in the master branch.

That makes sense.  You do need the docking branch of ImGui, then obviously the appropriate branch for cimgui in order to support it.  So this isn't an ImGui.NET issue, it's an upstream issue really.  Gotcha.

You do need the docking branch of ImGui, then obviously the appropriate branch for cimgui in order to support it.

yes imgui.net master branch is doing exactly that. Actually to be 100% clear the demo code in imgui.net repo can do docking feature with 0 code change ( you just have to enable the docking flag ).
It's the (add,remove,get,update) viewport part which you folks are really talking about.

Keep in mind that when using viewports ImGui expects the input events to be in raw screen space coordinate, so you might have to do some coordinate conversions.

could you show example?


Fix double memory allocation during operator setup

This is a fix for issue being seen in https://github.com/apache/incubator-pinot/issues/5610
The call stack for the OOM is
`Caused by: java.lang.OutOfMemoryError: Direct buffer memory
	at java.nio.Bits.reserveMemory(Bits.java:694) ~[?:1.8.0_252]
	at java.nio.DirectByteBuffer.<init>(DirectByteBuffer.java:123) ~[?:1.8.0_252]
	at java.nio.ByteBuffer.allocateDirect(ByteBuffer.java:311) ~[?:1.8.0_252]
	at org.apache.pinot.core.io.reader.impl.ChunkReaderContext.<init>(ChunkReaderContext.java:38)org.apache.pinot.core.io.reader.impl.v1.VarByteChunkSingleValueReader.createContext(VarByteChunkSingleValueReader.java:93)
org.apache.pinot.core.io.reader.impl.v1.VarByteChunkSingleValueReader.createContext(VarByteChunkSingleValueReader.java:
	at org.apache.pinot.core.operator.docvalsets.SingleValueSet.<init>(SingleValueSet.java:35) ~[pinot-all-0.5.0-SNAPSHOT-jar-with-dependencies.jar:0.5.0-SNAPSHOT-01cdc55f514b10bee2d8108d9736d4b57c48b517]
	at org.apache.pinot.core.operator.blocks.SingleValueBlock.<init>(SingleValueBlock.java:41) ~[pinot-all-0.5.0-SNAPSHOT-jar-with-dependencies.jar:0.5.0-SNAPSHOT-01cdc55f514b10bee2d8108d9736d4b57c48b517]
	at org.apache.pinot.core.segment.index.datasource.BaseDataSource.getNextBlock(BaseDataSource.java:105) ~[pinot-all-0.5.0-SNAPSHOT-jar-with-dependencies.jar:0.5.0-SNAPSHOT-01cdc55f514b10bee2d8108d9736d4b57c48b517]
	at org.apache.pinot.core.operator.BaseOperator.nextBlock(BaseOperator.java:49) ~[pinot-all-0.5.0-SNAPSHOT-jar-with-dependencies.jar:0.5.0-SNAPSHOT-01cdc55f514b10bee2d8108d9736d4b57c48b517]
	at org.apache.pinot.core.common.DataFetcher.<init>(DataFetcher.java:65) ~[pinot-all-0.5.0-SNAPSHOT-jar-with-dependencies.jar:0.5.0-SNAPSHOT-01cdc55f514b10bee2d8108d9736d4b57c48b517]
	at org.apache.pinot.core.operator.ProjectionOperator.<init>(ProjectionOperator.java:46) ~[pinot-all-0.5.0-SNAPSHOT-jar-with-dependencies.jar:0.5.0-SNAPSHOT-01cdc55f514b10bee2d8108d9736d4b57c48b517]
	at org.apache.pinot.core.plan.ProjectionPlanNode.run(ProjectionPlanNode.java:51) ~[pinot-all-0.5.0-SNAPSHOT-jar-with-dependencies.jar:0.5.0-SNAPSHOT-01cdc55f514b10bee2d8108d9736d4b57c48b517]
	at org.apache.pinot.core.plan.TransformPlanNode.run(TransformPlanNode.java:103) ~[pinot-all-0.5.0-SNAPSHOT-jar-with-dependencies.jar:0.5.0-SNAPSHOT-01cdc55f514b10bee2d8108d9736d4b57c48b517]
	at org.apache.pinot.core.plan.SelectionPlanNode.run(SelectionPlanNode.java:55) ~[pinot-all-0.5.0-SNAPSHOT-jar-with-dependencies.jar:0.5.0-SNAPSHOT-01cdc55f514b10bee2d8108d9736d4b57c48b517]
	at org.apache.pinot.core.plan.CombinePlanNode$1.callJob(CombinePlanNode.java:122) ~[pinot-all-0.5.0-SNAPSHOT-jar-with-dependencies.jar:0.5.0-SNAPSHOT-01cdc55f514b10bee2d8108d9736d4b57c48b517]`

This implies that error happens essentially during execution setup phase when the operators are being setup and haven't begun the execution by passing data between them.
As per @fx19880617 , the call stack seems to be consistently reproducible as soon as the client upgrades to 0.4.0 in production and runs a SELECT * query on segments created in 0.3.0. At the top of the stack, there is ChunkReaderContext which allocates a chunk and fails with direct memory OOM. @fx19880617 verified that chunk size and numDocsPerChunk etc are same so it is the not case that in 0.4.0, we are suddenly allocating more memory in the ChunkReaderContext and thus failing.
Either there is a leak or something else.
As part of PR https://github.com/apache/incubator-pinot/pull/5510, one change made to SingleValueSet (also part of the call stack) was that reader context is now being created in the call to constructor as opposed to during every read call. There are two implications of this:

Earlier since chunk reader context objects were created on a per call basis, they were essentially short lived and probably never made their way to perm gen. Thus were garbage collected thereby also leading to garbage collecting the direct byte buffer reference inside them and freeing up direct memory -- this is essentially how direct memory is freed up in JVM unless cleaner is called. Now since they are being allocated in the constructor, they have essentially become long-lived objects and are unlikely to be GCed as quickly as they were in the previous code. Thus, there will be memory pressure.

The second implication as explained below is worse and probably the actual root cause of the OOM reported by the customer.

See this code for creation of ProjectionOperator (note that it is part of call stack)
`public ProjectionOperator(Map<String, DataSource> dataSourceMap, BaseOperator<DocIdSetBlock> docIdSetOperator) {
    _dataSourceMap = dataSourceMap;
    _dataBlockMap = new HashMap<>(dataSourceMap.size());
    for (Map.Entry<String, DataSource> entry : dataSourceMap.entrySet()) {
      _dataBlockMap.put(entry.getKey(), entry.getValue().nextBlock());
    }
    _docIdSetOperator = docIdSetOperator;
    _dataBlockCache = new DataBlockCache(new DataFetcher(dataSourceMap));
  }`

_dataBlockMap.put(entry.getKey(), entry.getValue().nextBlock()); creates a block by going down the path of nextBlock() -> SingleValueBlock -> SingleValueSet -> constructor -> create reader context (direct memory allocated with the new PR)
We then create DataFetcher with dataSourceMap and this code again creates a block
dataSourceMap.get(column).nextBlock().getBlockValueSet() thus essentially going down the same path again eventually creating SingleValueBlock -> SingleValueSet -> reader context. -> allocating direct memory
So the memory is being allocated twice and somewhere in the middle of doing this for the DataFetcher for a given column, we fail with OOM

Amazing work!
We definitely need a backward compatibility test for segments.

@kishoreg , there is no backward incompatibility here. The PR #5510 actually results in double memory allocation (as explained above in the description)
W.r.t tests for backward compatibility of raw index reader writer, we already have them. There are files checked in from prior versions to ensure current readers can read old versions. We have them for both compressed and uncompressed. I recently added more tests too. See VarByteChunkSingleValueWriterTest.java

Got it. I thought there was something wrong with the metadata in the segment header.

Good finding!
In order to fix this, I would recommend replacing _dataBlockMap with Map<String, DataSourceMetadata> or Map<String, FieldSpec>`. Setting blocks in operator constructor is not correct.
Besides, the real root cause of this issue is allocating direct memory within reader context without freeing it explicitly. There is no guarantee when the direct memory can be released. We need to figure out a way to free the memory explicitly.


added puncuation

Describe your changes
I added punctuation(support,) to the sentence "An empty alt attribute has the most wide-spread support so that should be your go-to choice when using icons." for better readability.
Screenshots - If Any (Optional)
Link to issue

Closes #367
Checklist before requesting a review

[x] I have performed a self-review of my code.
[x] Followed the repository's Contributing Guidelines.
[x] I ran the app and tested it locally to verify that it works as expected.
[x] I have checked my code with an automatic accessibility tool such as Axe Dev Tools or Wave
and it shows no errors.

Additional Information (Optional)
Any additional information that you want to give us.

@EmmaDawsonDev please review this PR.

Alright.

@EmmaDawsonDev will I need to create another PR?

No, you should just be able to make the changes and push them and they will be reflected here.

I have made some code changes, removed the prettier formatter, and also added content for the link. Please check and let me know if the issue persists.

Alright. I have seen all your comments. I make the corrections and I will make a push again.

I'm going to merge this into a temporary branch and fix the issues when I get time. Thanks so much for your help so far :)

I apologize for finishing with this. I have been down for some time. Please let me know if there are anything I need to add.

No worries. I fully understand that life gets in the way sometimes.


Issue with Skip Prologue and experimentalLazyRoomInitialization

Hello,
I found an issue with the latest version of BattletechPerformanceFix 1.2.3 (didn't check other version).
When you create a new custom campaign and check the Skip Prologue option if you have experimentalLazyRoomInitialization to true it will skip the first mission ("Three Years Later") to go directly to the dropship.
Everything seems to work fine until you try to advance time and nothing happens.

I updated to BTML v0.5.0alpha1 and ModTek v0.3.1 and the issue remains.
Note that once you passed that mission activating experimentalLazyRoomInitialization seems to be working fine.

You recommend not using the feature at all?
Because the difference in game is amazing and I'd rather take 10s to activate it afterward than not using it.

There may be other areas where it causes issues, I haven't had a chance to test it much. You're welcome to use it, just be aware of this.

Fixed with fe98d4b


[stable/redis-ha]: allow DNS requests to CIDR <IP_ADDRESS>/16 in haproxy NetworkPolicy

What this PR does / why we need it:
This PR adds a rule to the redis-ha haproxy NetworkPolicy to allow haproxy pods to send DNS requests to CIDR <IP_ADDRESS>/16.
This rule is already present in the redis-ha NetworkPolicy.
Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

@lgatellier I think the idea behind this originally was to intentionally not included it when these options were added originally.
I'd prefer people have full control as there will be bare metal / other cloud providers where this private DNS could be used for something else and end up causing some poor admin a security headache without realizing.


UPDATE is re-enabled after DevPortal page is refreshed when Provisioned Oauth Client credentials are provided

Description
After provisioning the OAuth Keys the UPDATE button is disabled, upon refreshing the DevPortal page or clicking on the Production/Sandbox Keys, the UPDATE button is re-enabled.
Steps to Reproduce
Configure IS-KM and APIM
Create a Service Provider in IS-KM as external OAuth2 client
Add the OAuth Client Key to the Service Provider Description
Create a new application in DevPortal with the same Service Provider name
Subscribe to an API
Under Production/Sandbox Keys in Devportal provide the OAuth Client Key and Client Secret of the Service Provider as the Consumer Key and Secret under “Provide Existing OAuth Keys”
After clicking Provide, the Update button appears as disabled, but is re-enabled when the page is refreshed.
Affected Component
APIM
Version
3.1.0
Environment Details (with versions)
No response
Relevant Log Output
No response
Related Issues
No response
Suggested Labels
No response

Closing as the public fix is already available.


Update Posts “goodbye-2023-welcome-2024”

Automatically generated by Netlify CMS

👷 Deploy Preview for shiny-babka-998c7e processing.

Name
Link

🔨 Latest commit
7ca73850080477c37d838b34ced0281c37aade63

🔍 Latest deploy log
https://app.netlify.com/sites/shiny-babka-998c7e/deploys/659beafcdbba0000085f467e


load images

Documentation says to render the image I need a texture id.
and provides example
texture_id = imgui.get_io().fonts.texture_id
fonts?...
I can't figure out how can I get the image from the file? do I need to convert image object to texture somehow? what the kind of texture it is?

Hi! dear ImGui (and thus, pyimgui) doesn't handle loading images ("textures") by itself – that's up to your rendering backend. Please see the How can I display an image? section in imgui's FAQ at the top of this file for some pointers¹.
¹ pun not intended...

@romanesko I just found this example for how to load an image using the PyGame/OpenGL backends and it worked for me: https://github.com/swistakm/pyimgui/issues/47#issuecomment-334451418
@swistakm I think it would be really helpful to have example code for each of the graphics backends documenting how to load images into pyimgui. If you need some help getting docs in place please let us know!

@Glench I definitely need some help with the docs because I didn’t have enough time for that. Your help would be appreciated:)

@Glench your code might work, but it might also leak some GPU memory since it offloads a new texture to the GPU at each frame and never releases it. Some GPU cards will be nice and forget about the texture since you never use it again; some others will hang after a moment.
So, remember to call gl.glDeleteTextures([textures_id_array]) from time to time.
Another example :

Loading : here, _image_rgb_to_texture_impl(img_rgb) uploads a  RGB buffer to the GPU and returns a texture id
Clearing: here gl.glDeleteTextures([textures_id_array]) deletes the previously loaded textures that are not needed anymore

@pthom Here's the actual code I ended up using, which doesn't reload the texture every frame:
from __future__ import absolute_import

import sys

import pygame
import OpenGL.GL as gl

from imgui.integrations.pygame import PygameRenderer
import imgui

def main():
    pygame.init()

    size = 800, 600

    pygame.display.set_mode(size, pygame.DOUBLEBUF | pygame.OPENGL)

    renderer = PygameRenderer()

    io = imgui.get_io()
    io.fonts.add_font_default()
    io.display_size = size

    def load_image(image_name='test.png'):
        image = pygame.image.load(image_name)
        textureSurface = pygame.transform.flip(image, False, True)

        textureData = pygame.image.tostring(textureSurface, "RGBA", 1)

        width = textureSurface.get_width()
        height = textureSurface.get_height()

        texture = gl.glGenTextures(1)
        gl.glBindTexture(gl.GL_TEXTURE_2D, texture)
        gl.glTexParameteri(gl.GL_TEXTURE_2D, gl.GL_TEXTURE_MAG_FILTER, gl.GL_LINEAR)
        gl.glTexParameteri(gl.GL_TEXTURE_2D, gl.GL_TEXTURE_MIN_FILTER, gl.GL_LINEAR)
        gl.glTexImage2D(gl.GL_TEXTURE_2D, 0, gl.GL_RGBA, width, height, 0, gl.GL_RGBA,
                     gl.GL_UNSIGNED_BYTE, textureData)

        return texture, width, height

    image_texture, image_width, image_height = load_image('test.png')

    while 1:
        for event in pygame.event.get():
            if event.type == pygame.QUIT:
                sys.exit()

            renderer.process_event(event)

        imgui.new_frame()
        imgui.text('An image:')
        imgui.image(image_texture, image_width, image_height)

        # note: cannot use screen.fill((1, 1, 1)) because pygame's screen
        #       does not support fill() on OpenGL surfaces
        gl.glClearColor(1, 1, 1, 1)
        gl.glClear(gl.GL_COLOR_BUFFER_BIT)
        imgui.render()

        pygame.display.flip()

if __name__ == "__main__":
    main()

For me, it was most important to have one simple command for loading and displaying the image from a filename.

Hi all after a day of playing around with imgui python I came across this issue and I found a fix :). It seems like the issue stems from impl.process_event(event)  in the example code impl.process_event(event)  is in  the event loop. Every time that loop is run for some reason the png is not rendered to screen. So what I did was remove impl.process_event(event)  outside the pygame event loop and created a  variable to hold the pygame events and pass that variable to impl.process_event(var) outside of the event loop. Here is my code `
from future import absolute_import
import sys
import pygame
import OpenGL.GL as gl
from imgui.integrations.pygame import PygameRenderer
import imgui
display = (800,600)
ui_widget.init(display)
def loadImage(image):
textureSurface = pygame.transform.flip(image, False, True)
textureData = pygame.image.tostring(textureSurface, "RGBA", 1)

width = textureSurface.get_width()
height = textureSurface.get_height()

texture = gl.glGenTextures(1)
gl.glBindTexture(gl.GL_TEXTURE_2D, texture)
gl.glTexParameteri(gl.GL_TEXTURE_2D, gl.GL_TEXTURE_MAG_FILTER, gl.GL_LINEAR)
gl.glTexParameteri(gl.GL_TEXTURE_2D, gl.GL_TEXTURE_MIN_FILTER, gl.GL_LINEAR)
gl.glTexImage2D(gl.GL_TEXTURE_2D, 0, gl.GL_RGBA, width, height, 0, gl.GL_RGBA,
             gl.GL_UNSIGNED_BYTE, textureData)

return texture, width, height

def main():
pygame.init()
size = 800, 600
pygame.display.set_mode(size, pygame.DOUBLEBUF | pygame.OPENGL | pygame.RESIZABLE)

imgui.create_context()
impl = PygameRenderer()

io = imgui.get_io()
io.display_size = size
texture, width, height = loadImage(pygame.image.load("icon.png"))

event_holder = None;

while 1:
    for event in pygame.event.get():
        if event.type == pygame.QUIT:
            sys.exit()
        event_holder = event;

    impl.process_event(event_holder)

    imgui.new_frame()

    imgui.begin("Custom window", True)
    imgui.push_id("1")
    if imgui.image_button(texture, width, height):
        print("Yay")
    imgui.pop_id()
    imgui.end()

    if imgui.begin_main_menu_bar():
        if imgui.begin_menu("File", True):

            clicked_quit, selected_quit = imgui.menu_item(
                "Quit", 'Cmd+Q', False, True
            )

            if clicked_quit:
                exit(1)

            imgui.end_menu()
        imgui.end_main_menu_bar()

    imgui.show_test_window()

    imgui.begin("Custom window", True)
    imgui.text("Bar")
    imgui.text_colored("Eggs", 0.2, 1., 0.)
    imgui.end()

    # note: cannot use screen.fill((1, 1, 1)) because pygame's screen
    #       does not support fill() on OpenGL sufraces
    gl.glClearColor(1, 1, 1, 1)
    gl.glClear(gl.GL_COLOR_BUFFER_BIT)
    imgui.render()
    impl.render(imgui.get_draw_data())

    pygame.display.flip()

if name == "main":
main()
`
I hope this help someone

thank you dude
for anyone that has imgui already running and just want to implement use this
        image = pygame.image.load("yourimage.png")

        textureSurface = pygame.transform.flip(image, False, True)

        textureData = pygame.image.tostring(textureSurface, "RGBA", 1)

        width = textureSurface.get_width()
        height = textureSurface.get_height()

        texture = gl.glGenTextures(1)
        gl.glBindTexture(gl.GL_TEXTURE_2D, texture)
        gl.glTexParameteri(gl.GL_TEXTURE_2D, gl.GL_TEXTURE_MAG_FILTER, gl.GL_LINEAR)
        gl.glTexParameteri(gl.GL_TEXTURE_2D, gl.GL_TEXTURE_MIN_FILTER, gl.GL_LINEAR)
        gl.glTexImage2D(gl.GL_TEXTURE_2D, 0, gl.GL_RGBA, width, height, 0, gl.GL_RGBA,
                     gl.GL_UNSIGNED_BYTE, textureData)

I made an all in one helper class that doesn't rely on pygame and also supports animated (GIF) images and also glob patterns. More details here https://github.com/pyimgui/pyimgui/issues/282#issuecomment-1127963181

code to display image using opengl and imgui withou pygame
def draw_image_with_pyimgui(opencv_img):
    height ,width, depth = opencv_img.shape
    if depth==3:
        opencv_img = cv2.cvtColor(opencv_img, cv2.COLOR_BGR2RGBA )

    texture = gl.glGenTextures(1)
    gl.glBindTexture(gl.GL_TEXTURE_2D, texture)
    gl.glTexParameteri(gl.GL_TEXTURE_2D, gl.GL_TEXTURE_MAG_FILTER, gl.GL_LINEAR)
    gl.glTexParameteri(gl.GL_TEXTURE_2D, gl.GL_TEXTURE_MIN_FILTER, gl.GL_LINEAR)
    gl.glPixelStorei(gl.GL_UNPACK_ROW_LENGTH,0)
    gl.glTexImage2D(gl.GL_TEXTURE_2D, 0, gl.GL_RGBA, width, height, 0, gl.GL_RGBA, gl.GL_UNSIGNED_BYTE, opencv_img)

    imgui.image(texture,width,height)

Or you could just look at the image helper I made and shared right above your comment


running tests on native elements in flutter appium driver.

Appium version: 1.15.0-beta.0
Device: iOS Simulator
App: Flutter app included plugin (enableFlutterDriverExtension())
language: java
the app consist of some screen which are "flutter" say (login) and some screens are native. say "camera"
I am able to run the tests using appium flutter driver on the flutter screen.
once I am on the native screen I am switching the context
driver.context("NATIVE_APP");
then using normal appium script,  I am trying to do actions on native elements.
public MobileElement <element_name> = (MobileElement) driver.findElement(By.xpath("<xpath_of_element>"));
wait.until(ExpectedConditions.visibilityOf(element_name));
element_name.click();
while running the script
it is throwing
java.lang.NullPointerException on populating the mobile element on this line..
public MobileElement <element_name> = (MobileElement) driver.findElement(By.xpath("<xpath_of_element>"));
any help regading this.
@truongsinh @shibupanda

Frankly, without a piece of code of app-under-test (AUT), it's quite difficult for us to re-produce the crash. Can you try some of the suggestion below:

upgrade to appium 1.16.0
provide more detail for the exception (e.g. is the null pointer exception on the By object or driver object, or possible loner stack stace)?

@shibupanda . is this not implemented. I found an open issue https://github.com/flutter/flutter/issues/36259

Yeah I opened that issue awhile ago, but I don't think it's the cause in this case, because driver.context("NATIVE_APP"); is supposed to help you with the native screen

@pradarttana-sharechat and @truongsinh
"NullPointerException" is something if you are passing something null as an argument or you are getting null value as a response.
@pradarttana-sharechat
Could you please share the complete logs as well using gist.

yes, I got it. I made a mistake on my part.


Separate C Conformance page

Even though #407 has been closed, there is still no explicit mention of C conformance, much less a table for tracking the various different aspects. Searching for "C Conformance" leads to the C++ Conformance page, which has a tiny snippet:

C11 Compiler support for C11 requires Visual Studio version 16.8, or higher. C11 library support requires Windows SDK version 20211, or higher. The Universal CRT implements the parts of the C11 Standard Library that are required by C++17, with the exception of C99 strftime() E/O alternative conversion specifiers, and C11 aligned_alloc(). The latter is unlikely to be implemented because the Windows operating system doesn't support aligned allocations.

There is an install page for C11/C17, but that doesn't help with conformance.
Having an explicit conformance page would be highly appreciated, since Visual Studio 16.8 has so far not implemented any of the C11 optionals (though it mentions that atomics & threading are on the roadmap) - those differences are crucial for authors having to work with the lowest common denominator across several compilers.
This will also become ever more relevant with the growing list of features for C2x/C23.

There's now a section regarding "C Standard library features". However, the language side seems to be missing.
In particular, while C++ conformance tracks the as-yet unreleased C++23, there's no sign of conformance for the upcoming C23 - this is going to be a very important update to the C standard (with lots of substantial improvements), so I sincerely hope that MSFT will not things slide again like for C99.  As a reference, here's the conformance page for clang, which contains a relatively up-to-date list of papers that have been accepted into C23 so far.
Note that C23 was supposed to be effectively feature-complete already, so while some papers are still in flight, there shouldn't be huge changes anymore - certainly stable enough to reflect on a conformance page.

@TylerMSFT @corob-msft
Any update here? 17.5 now has C11 atomics support (even if experimantal), so this information should be present somewhere. Also, C23 is now feature-complete and just going through the ISO-motions for publications. A lot of people (especially on the C++ side) are waiting for features like #embed, which happened to land in the C standard before they did in C++. This really should be visible somewhere.
CC @TartanLlama @StephanTLavavej @CaseyCarter

Agreed. However, as I'm the only one maintaining over 10,000 C++ topics now, as well as writing all the new ones, it will be a while until I can get to this.

According to the 17.8 preview 2 blogpost, C11 threads are now supported! 🥳
It looks like the C conformance section gained a bit more detail too, thanks for that! Still, that table could probably use an update for the two topics above (much less for C23). 🙃


WIP: Allow all extensions in links

It's nice to be able to link to more types of files than the default. That could be code (py, js, sh), data (csv), configuration files (yaml), or anything else.
This fixes #592  and #593.
I've tried this out, and it works well. It's WIP because:

If this approach is desired, the extension code probably needs some work. There's no need for otherExts. The code could treat extensions as one of: markdown, images, and everything else.

I assume tests need to be update.

Hi @tshort, currently all extensions are not allowed for performance reasons (workspace cache is rebuilt on restart), for example, there could be users where knowledge base is located next to deeply nested folders like node_modules with lots of files. Another suggestion could be to allow custom extensions to be configurable via settings or if all are needed allow a wildcard like *.

I'll play around with configurable options.


Capitalize Word

The first word should always be capitalize but in the actors they are not. I would like to fix this one

Hey @Pratham271 This is a repeat issue as of #832 please close this issue!


Support for other namespace and other KAFKA services!

Deployed a Kubeless + Kubeless UI on cloud using my_namespace as my namespace.
When using Kubeless UI, functions are been created in the default namespace and can't use pubsub because can't set ENVs vars for runtime.
How to propagate the correct env vars for via ui for runtime function containers¿?
How to set Kubeless o functions aren't create in the default namespace¿?
Cannot propagate KUBELESS_KAFKA_SVC and KUBELESS_NAMESPACE to runtime envs via UI.
PD: thanks to @andresmgot for support on bitnami slack channel.

for the record, supporting a different Kafka SVC will require changes in Kubeless (https://github.com/kubeless/kubeless/issues/196)

let's ping @arapulido so we can see how to develop this.
Of course PR welcome as well :)

Thanks @sebgoa for the nudge. What do we need to do here?

What do we need to do here?

We need to adapt the UI to be able to use a different namespace.
Apart from that we need to address the issue I linked before in Kubeless and then adapt the UI again.


PhanTypeMismatchArgument when merging arrays with +

When you "merge" arrays with the + operator, Phan complains that the argument is float|int but should be array.
Demo:
$x = ['foo' => 1];
$y = ['bar' => 2];
function add(array $in) {
}
add($x + $y);

PhanTypeMismatchArgument Argument 1 (in) is float|int but \add() takes array defined at phan.test...
Is this something that can be fixed in Phan? Array works with + operator although it may not be the best style.

Thanks for the report @black-silence. This should be fixed now.

Great, thanks.


Reporting False Positive Finding

False positive on library org.springframework.batch:spring-batch-core:3.0.10.RELEASE and org.springframework.batch:spring-batch-infrastructure:3.0.10.RELEASE - reported as

cpe:/a:pivotal_software:spring_framework:3.0.10
cpe:/a:pivotal_software:spring_batch:3.0.10
cpe:/a:pivotal:spring_framework:3.0.10

<dependency>
   <groupId>org.springframework.batch</groupId>
   <artifactId>spring-batch-core</artifactId>
   <version>3.0.10.RELEASE</version>
</dependency>
<dependency>
   <groupId>org.springframework.batch</groupId>
   <artifactId>spring-batch-infrastructure</artifactId>
   <version>3.0.10.RELEASE</version>
</dependency>

CVEs:

CVE-2016-9878
CVE-2018-1270
CVE-2018-1271
CVE-2018-1272

@jeremylong With 6.0.3-SNAPSHOT these dependencies only trigger a verified vulnerability on the transient dependency spring-core v 4.3.21.RELEASE.
This FP issue can be reported solved by the 6.0.3 release when you get to it. No longer triggering the mentioned false positives.
You may want to consider preventing future false-positives on spring-batch-infrastructure, as that maps in addition to the proper spring batch CPE also to an unrelated vmware CPE: cpe:2.3:a:vmware:infrastructure:3.0.10:release:*:*:*:*:*:*


Issues before year 2018

I would like to propose to look at and close, if possible all issues before year 2018. Because some issues are not relevant anymore.

Hi @Prefix1802, I completely agree, we are planning on cleaning up the issues.

Closed as outdated


[Draft] Release v2.5.7 to Alire

preparing for future v2.5.7 release to Alire (all those warnings being removed would be great to have ;) )

These changes are already in master from previous PR and commit 0fa2c284d508f9f669a0daf70844b7bbcee197d2.


how to make this work for an astro + solidjs website?

Use the vite configuration as described in the README
import { defineConfig } from 'astro/config';
import solid from '@astrojs/solid-js';

// https://astro.build/config
export default defineConfig({
    // Enable Solid to support Solid JSX components.
    integrations: [solid()],
    vite: {
        plugins: [
            Icons({
                compiler: 'solid'
            })
        ]
    }
});


Touch up french

Changed the README and markdown cells to say 4 use cases and fixed the typos. Reran the notebooks to make sure it worked


log rotation

We should probably add a log rotation for crypt.
Questions to probably be answered:

Should we move log location or make that a preference?
What is the threshold (in size) for log rotation?

My opinions

Should we move log location or make that a preference?
No.
What is the threshold (in size) for log rotation?
5 MB

That file size gives plenty of data. For companies that want to store these logs for longer amounts of time, they should ship off box into a Splunk/ES/Logstash/etc.

@erikng @clburlison we use a newsyslog config for this.
/etc/newsyslog.d/crypt.conf
# logfilename             [owner:group]       mode   count size(KiB) when   flags [/pid_file] # [sig_num]
/var/log/crypt.log        root:wheel          644    3     2560      *      JN

I don’t object to putting this in the default package.

cool, I'll add it.

two years later 👀
didn't even know this issue still existed but sounds good to me as well


docs: Adding a MIT Licenses badge and some more text to Licenses section in readme

what's wrong with the documentation?
The readme file of the repo should have a MIT Licenses badge and some more text regarding licenses in the  Licenses section to make it more professional.
Add screenshots

Record

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSoC'23 contributor
[X] I want to work on this issue

@1Shubham7 thanks for creating the issue. I think it's unnecessary


Question regarding model?

I don't really have an "issue" per-se I just had a general question if you wouldn't mind answering, the girl in the pictures that you've uploaded and her body looks quite realistic, would you mind sharing what stable diffusion model, sampling method & sampling steps and other settings that you used to create it? I would really really appreciate it.
Thanks!

model is standard v1-5-pruned-emaonly and vae is vae-ft-ema-560000-ema-pruned
girl is a textual inversion i've trained
i'm sorry, but seems like during last "cleanup" i've deleted that batch, so no longer have exact prompt and params used, but as far as i recall, it was quite a simple prompt (something like "beautiful young woman  sitting on a bed, posing, high detailed, etc...") and using DPM2 Karras sampler with 25 steps


⚠ Bofrost.de has degraded performance

In fc44daf, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1734 ms

Resolved: Bofrost.de performance has improved in 995ae46 after 6 minutes.


Connect wallet UI

Per frontend discussion, this is a step 1 ticket to resolving the RPC usage issue.

Nice to meet you, Mr amytsang.
I am strongly interested in this issue. May I?


Breast cancer

I have breast cancer images, benign and malignant, is it possible to use this code?

Generally speaking - yes


Test merge editor: "Toggle Current Conflict from Left" should only appear when valid

Testing #158819
When my cursor isn't inside of a conflict, this command does nothing. Should it only be visible in the quickpick when it does something? Or should there be an error message when you try to run it but it doesn't do anything? Same with the "Toggle Current Conflict from Right".

I fear if we hide it when there is nothing to toggle, users won't discover that there is a (keybindable) command for this functionality.
How did you find this command? Was your cursor in a conflicting region?

It wasn't in a conflicting region at the time

Let's show an error message if this command does not do anything.

If I run the toggle command in an invalid region, I get a notification, which is good.
But, if I then toggle a valid region after, the notification still stays on the screen for a while. Can the notification be hidden when the user toggles a valid region?

Can notifications be removed? I thought that is not possible. Also, I think this use-case is quite rare. The notification is just to teach that you should be in a conflict.


ShellOperation completes and then errors: RuntimeError: PID 51828 failed with return code 1.

Runs fine when checking links from a small website such as  "https://example.com", perhaps because not opening multiple streams?
Error Reproduction

from prefect import flow
from prefect_shell import ShellOperation

@flow(log_prints=True)
def check_links():

    res = ShellOperation(commands=["linkchecker https://docs.prefect.io"]).run()
    print(res[8:])

if __name__ == "__main__":
    check_links()

Version:             2.10.11
API version:         0.8.4
Python version:      3.10.8
Git commit:          8c651ffc
Built:               Thu, May 25, 2023 2:59 PM
OS/Arch:             darwin/arm64
Profile:             default
Server type:         cloud

prefect-shell             0.1.5                    pypi_0    pypi

Traceback / Example
...
That's it. 1058 links in 1243 URLs checked. 0 warnings found. 2 errors found.
Stopped checking at 2023-06-02 10:19:52-004 (3 minutes, 9 seconds)

10:19:52.136 | ERROR   | Flow run 'axiomatic-otter' - Encountered exception during execution:
Traceback (most recent call last):
  File "/Users/jeffhale/miniforge3/lib/python3.10/site-packages/prefect/engine.py", line 742, in orchestrate_flow_run
    result = await flow_call.aresult()
  File "/Users/jeffhale/miniforge3/lib/python3.10/site-packages/prefect/_internal/concurrency/calls.py", line 181, in aresult
    return await asyncio.wrap_future(self.future)
  File "/Users/jeffhale/miniforge3/lib/python3.10/site-packages/prefect/_internal/concurrency/calls.py", line 194, in _run_sync
    result = self.fn(*self.args, **self.kwargs)
  File "/Users/jeffhale/Desktop/prefect/linkcheck/linkcheck.py", line 9, in check_links
    res = ShellOperation(commands=["linkchecker https://docs.prefect.io"]).run()
  File "/Users/jeffhale/miniforge3/lib/python3.10/site-packages/prefect/utilities/asyncutils.py", line 260, in coroutine_wrapper
    return call()
  File "/Users/jeffhale/miniforge3/lib/python3.10/site-packages/prefect/_internal/concurrency/calls.py", line 245, in __call__
    return self.result()
  File "/Users/jeffhale/miniforge3/lib/python3.10/site-packages/prefect/_internal/concurrency/calls.py", line 173, in result
    return self.future.result(timeout=timeout)
  File "/Users/jeffhale/miniforge3/lib/python3.10/concurrent/futures/_base.py", line 451, in result
    return self.__get_result()
  File "/Users/jeffhale/miniforge3/lib/python3.10/concurrent/futures/_base.py", line 403, in __get_result
    raise self._exception
  File "/Users/jeffhale/miniforge3/lib/python3.10/site-packages/prefect/_internal/concurrency/calls.py", line 218, in _run_async
    result = await coro
  File "/Users/jeffhale/miniforge3/lib/python3.10/site-packages/prefect_shell/commands.py", line 396, in run
    await shell_process.wait_for_completion()
  File "/Users/jeffhale/miniforge3/lib/python3.10/site-packages/prefect_shell/commands.py", line 177, in wait_for_completion
    raise RuntimeError(
RuntimeError: PID 51828 failed with return code 1.
10:19:52.299 | ERROR   | Flow run 'axiomatic-otter' - Finished in state Failed('Flow run encountered an exception. RuntimeError: PID 51828 failed with return code 1.\n')
Traceback (most recent call last):
  File "/Users/jeffhale/Desktop/prefect/linkcheck/linkcheck.py", line 28, in <module>
    check_links()
  File "/Users/jeffhale/miniforge3/lib/python3.10/site-packages/prefect/flows.py", line 518, in __call__
    return enter_flow_run_engine_from_flow_call(
  File "/Users/jeffhale/miniforge3/lib/python3.10/site-packages/prefect/engine.py", line 198, in enter_flow_run_engine_from_flow_call
    retval = from_sync.wait_for_call_in_loop_thread(
  File "/Users/jeffhale/miniforge3/lib/python3.10/site-packages/prefect/_internal/concurrency/api.py", line 232, in wait_for_call_in_loop_thread
    return call.result()
  File "/Users/jeffhale/miniforge3/lib/python3.10/site-packages/prefect/_internal/concurrency/calls.py", line 173, in result
    return self.future.result(timeout=timeout)
  File "/Users/jeffhale/miniforge3/lib/python3.10/concurrent/futures/_base.py", line 451, in result
    return self.__get_result()
  File "/Users/jeffhale/miniforge3/lib/python3.10/concurrent/futures/_base.py", line 403, in __get_result
    raise self._exception
  File "/Users/jeffhale/miniforge3/lib/python3.10/site-packages/prefect/_internal/concurrency/calls.py", line 218, in _run_async
    result = await coro
  File "/Users/jeffhale/miniforge3/lib/python3.10/site-packages/prefect/client/utilities.py", line 40, in with_injected_client
    return await fn(*args, **kwargs)
  File "/Users/jeffhale/miniforge3/lib/python3.10/site-packages/prefect/engine.py", line 292, in create_then_begin_flow_run
    return await state.result(fetch=True)
  File "/Users/jeffhale/miniforge3/lib/python3.10/site-packages/prefect/states.py", line 92, in _get_state_result
    raise await get_state_exception(state)
  File "/Users/jeffhale/miniforge3/lib/python3.10/site-packages/prefect/engine.py", line 742, in orchestrate_flow_run
    result = await flow_call.aresult()
  File "/Users/jeffhale/miniforge3/lib/python3.10/site-packages/prefect/_internal/concurrency/calls.py", line 181, in aresult
    return await asyncio.wrap_future(self.future)
  File "/Users/jeffhale/miniforge3/lib/python3.10/site-packages/prefect/_internal/concurrency/calls.py", line 194, in _run_sync
    result = self.fn(*self.args, **self.kwargs)
  File "/Users/jeffhale/Desktop/prefect/linkcheck/linkcheck.py", line 9, in check_links
    res = ShellOperation(commands=["linkchecker https://docs.prefect.io"]).run()
  File "/Users/jeffhale/miniforge3/lib/python3.10/site-packages/prefect/utilities/asyncutils.py", line 260, in coroutine_wrapper
    return call()
  File "/Users/jeffhale/miniforge3/lib/python3.10/site-packages/prefect/_internal/concurrency/calls.py", line 245, in __call__
    return self.result()
  File "/Users/jeffhale/miniforge3/lib/python3.10/site-packages/prefect/_internal/concurrency/calls.py", line 173, in result
    return self.future.result(timeout=timeout)
  File "/Users/jeffhale/miniforge3/lib/python3.10/concurrent/futures/_base.py", line 451, in result
    return self.__get_result()
  File "/Users/jeffhale/miniforge3/lib/python3.10/concurrent/futures/_base.py", line 403, in __get_result
    raise self._exception
  File "/Users/jeffhale/miniforge3/lib/python3.10/site-packages/prefect/_internal/concurrency/calls.py", line 218, in _run_async
    result = await coro
  File "/Users/jeffhale/miniforge3/lib/python3.10/site-packages/prefect_shell/commands.py", line 396, in run
    await shell_process.wait_for_completion()
  File "/Users/jeffhale/miniforge3/lib/python3.10/site-packages/prefect_shell/commands.py", line 177, in wait_for_completion
    raise RuntimeError(
RuntimeError: PID 51828 failed with return code 1.

Hmm -- linkchecker finds errors in the site, it returns with a nonzero exit code, ShellOperation throws an exception since the command returned failure, and thus the flow failed-- this all seems as expected to me?

Not exactly the same but is it safe to assume a return code other than zero is an error? I'm using Robocopy and I get the same error using it in prefect-shell since the return codes under 8 are not errors. This is from the Microsoft site (https://learn.microsoft.com/en-us/troubleshoot/windows-server/backup-and-storage/return-codes-used-robocopy-utility)


Double Cart Pole with a Spring env

A new RL environment described in J-F. Hren thesis "Planification optimiste pour systèmes déterministes" page 76.

I took the liberty to add a video for the doc. Tell me if you are ok with it.

I took the liberty to add a video for the doc. Tell me if you are ok with it.

I made a longer training so that the video will be more interesting. I have also changed the default parameters so that the default environment is simpler to solve.


[Enhancement]  Method calls equals on an enum instance [WebhookPushRequest]

Search before asking

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

Enhancement Request

located at:
eventmesh-runtime/src/main/java/org/apache/eventmesh/runtime/core/protocol/grpc/push/WebhookPushRequest.java line 239 241 250 252
analysis and explanation:
This method calls the equals(Object) method on an enum instance. Since enums values are singletons, you can use == to safely compare two enum values. In fact, the implementation for Enum.equals does just that.
Describe the solution you'd like
Change to use "==".
Are you willing to submit PR?

[x] Yes I am willing to submit a PR!

I want to fix it.


Provide functionality for turning on and off websocket

Ideas for a possible solution include threading or asncio.run

https://stackoverflow.com/questions/49005651/how-does-asyncio-actually-work


added ansi colors for container logs

I've added ansi2html and Strict Contextual Escaping ($sce) to the ContainerLogsController which does the trick:

refs #524

All i see is a black background with all text in white. i dont see any colours. Is there something i need to do to enable (or see) colours

You'll need an application which sets the colors. Can you try ls -la or htop? Which OS is your container running?

Any updates on this?
Would love to see this supported.
Thanks.

Yup, all good. Anthony, you can accept.
Rgds,
Neil Cresswell
On 3/02/2017, at 10:22 PM, Niklas <notifications<EMAIL_ADDRESS>wrote:
Any updates on this?
Would love to see this supported.
Thanks.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_portainer_portainer_pull_533-23issuecomment-2D277201294&d=DQMCaQ&c=euGZstcaTDllvimEN8b7jXrwqOf-v5A_CdpgnVfiiMM&r=0fx0h4vB56iTLpw2McH1ZD6TqG_QGpbggVOB-PfMJpM&m=BjEFDfbx_Hrhnvrdi5VHBJ8BbamA58YWErfnX2PkC5M&s=P-L4lDVwVTQvU68ZB-_hi-YDVIsGTRg6JrjD2-frmh8&e=, or mute the threadhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AWGrlbrlkeCPG34jlDhmcbjlRjFMrewDks5rYvHWgaJpZM4Lq4KN&d=DQMCaQ&c=euGZstcaTDllvimEN8b7jXrwqOf-v5A_CdpgnVfiiMM&r=0fx0h4vB56iTLpw2McH1ZD6TqG_QGpbggVOB-PfMJpM&m=BjEFDfbx_Hrhnvrdi5VHBJ8BbamA58YWErfnX2PkC5M&s=N_R_ESkTdKHdxquxw8ZoNxwCQhGCQO2AVLH8PupTiX4&e=.

All right, I'll do the tech review soon.

Did nothing special, just catch me if you have questions.

ping ;)

Hi @morph027
I do not have time to review this at the moment, will try to have a look later.

Sorry i have just seen your pull request.
I have done the same integration today : #1169
The 2 differences are i have used a bower dependency and i have changed the code of ansi2html to remove the default style because the default style is the current style it's not necessary to override the current style.

This is OK, I'll review the new one ASAP. I prefer to have the ansi2html as a bower dependency than shipped directly in the codebase.
Will close this one in favor of #1169

I'm ok with this, as long as i get colored terminal :grin:


Moving the reactions popover arrow point to point towards chevron

Fixes: #3174

Automated message from Dropbox CLA bot
@anirudhjain75, it looks like you've already signed the Dropbox CLA. Thanks!

@anirudhjain75 this looks great, though your commit message's first line is too long.  I've fixed that for you this time, but please write your commit messages correctly in the future.
Check out the Zulip commit message guidelines for more details: https://zulip.readthedocs.io/en/latest/version-control.html#commit-messages
Merged as dcc13c504b1fed3f0054a8efa8be6e0543ef6211, thanks for fixing this!


docs: Fix grammar mistakes in README.md

Fix lack of proper spacing in some of the links in the README.md file.

As described, there are some links that are missing a space between words. This PR simply adds proper spacing between some of the words so that it reads better.

(closing due to the fact that someone else already opened a PR regarding this but I didn't see it before opening)


PasswordPolicy HTTP 400 Bad Request

Current Behavior
Hi team,
When i try to create a simple realm with a password policy containing passwordHistory(5) or specialChars(1)
the import failed with HTTP 400 Bad Request all other password policy are created correctly.
The same work fine with kcadm.sh
2023-07-27 14:57:16.735  INFO 494 --- [           main] d.a.k.config.KeycloakConfigApplication   : Starting KeycloakConfigApplication v5.8.0 using Java 11.0.19 on 8c935b46ebcd with PID 494 (/opt/keycloak-config-cli/keycloak-config-cli.jar started by AzDevOps_azpcontainer in /__w/1/s)
2023-07-27 14:57:16.741  INFO 494 --- [           main] d.a.k.config.KeycloakConfigApplication   : No active profile set, falling back to 1 default profile: "default"
2023-07-27 14:57:18.217  INFO 494 --- [           main] d.a.k.config.KeycloakConfigApplication   : Started KeycloakConfigApplication in 3.168 seconds (JVM running for 4.087)
2023-07-27 14:57:20.811  INFO 494 --- [           main] d.a.k.config.KeycloakConfigRunner        : Importing file 'file:./sle-dev.json'
2023-07-27 14:57:28.735 ERROR 494 --- [           main] d.a.k.config.KeycloakConfigRunner        : HTTP 400 Bad Request
2023-07-27 14:57:28.739  INFO 494 --- [           main] d.a.k.config.KeycloakConfigRunner        : keycloak-config-cli running in 00:08.686.
Expected Behavior
password policy are created correctly
Steps To Reproduce
create a realm with property :
"passwordPolicy": "specialChars(1) and forceExpiredPasswordChange(365) and length(12) and lowerCase(1) and upperCase(1) and digits(1) and maxLength(128) and notUsername(undefined) and notEmail(undefined) and passwordHistory(5) and specialChars(1)"

Environment

Keycloak Version: 21.1.2
keycloak-config-cli Version: 5.8.0
Java Version: openjdk-11-jdk:amd64 (11.0.19+7~us1-0ubuntu1~22.04.1)

Anything else?
No response

Had the same issue and ended up with splitting my realm json into two files. The first one 00_master-realm-config.json sets password policy to null:
...
"passwordPolicy": "",
...

And then within my second json  00_master-realm-global.json I'm setting the global configs, e.g. password policy;
...
  "sslRequired": "external",
  "registrationAllowed": false,
  "registrationEmailAsUsername": false,
  "rememberMe": false,
  "verifyEmail": false,
  "loginWithEmailAllowed": true,
  "duplicateEmailsAllowed": false,
  "resetPasswordAllowed": false,
  "editUsernameAllowed": false,
  "bruteForceProtected": true,
  "permanentLockout": false,
  "maxFailureWaitSeconds": 900,
  "minimumQuickLoginWaitSeconds": 60,
  "waitIncrementSeconds": 60,
  "quickLoginCheckMilliSeconds": 1000,
  "maxDeltaTimeSeconds": 43200,
  "failureFactor": 10,
  "passwordPolicy": "length(12) and notEmail(undefined) and passwordHistory(3) and digits(1) and specialChars(1) and upperCase(1) and lowerCase(1) and notUsername(1)",
...

Because of the naming the global file gets execution last:
 2024-06-15 06:53:09.639  INFO 1 --- [           main] d.a.k.config.KeycloakConfigApplication   : Started KeycloakConfigApplication in 17.785 seconds (JVM running for 24.971)
2024-06-15 06:53:26.548  INFO 1 --- [           main] d.a.k.config.KeycloakConfigRunner        : Importing file 'file:/config/00_master-realm-config.json'
2024-06-15 06:53:26.648  INFO 1 --- [           main] d.a.k.config.provider.KeycloakProvider   : Wait 30 seconds until http://keycloak-service:8080 is available ...
2024-06-15 06:53:52.084  INFO 1 --- [           main] d.a.k.config.KeycloakConfigRunner        : Importing file 'file:/config/00_master-realm-global.json'

HTH

Hello @csemio.
Thank you for reporting your issue.
It looks like you did the import wrongly, or there is an little error you made. i have used several versions of KC-CLI and keycloak.
Here is the output with succesfull import of password policies using  bitnami/keycloak-config-cli:5.8.0 and   keycloak/keycloak:22.0.0
{
  "realm": "secure-realme",
  "enabled": true,
  "clients": [
    {
      "clientId": "example-client",
      "enabled": true,
      "clientAuthenticatorType": "client-secret",
      "secret": "your-client-secret",
      "redirectUris": ["https://your-app.example.com/*"],
      "webOrigins": ["https://your-app.example.com"]
    }
  ],
  "roles": {
    "realm": [
      {
        "name": "user",
        "description": "Default user role"
      }
    ]
  }
}

Now, if i include the passwordPolicy, i will get this

{
  "realm": "secure-realme",
  "enabled": true,
  "passwordPolicy": "specialChars(1) and forceExpiredPasswordChange(365) and length(12) and lowerCase(1) and upperCase(1) and digits(1) and maxLength(128) and notUsername(undefined) and notEmail(undefined) and passwordHistory(5) and specialChars(1)",
  "clients": [
    {
      "clientId": "example-client",
      "enabled": true,
      "clientAuthenticatorType": "client-secret",
      "secret": "your-client-secret",
      "redirectUris": ["https://your-app.example.com/*"],
      "webOrigins": ["https://your-app.example.com"]
    }
  ],
  "roles": {
    "realm": [
      {
        "name": "user",
        "description": "Default user role"
      }
    ]
  }
}

Please @csemio  try to use the latest version too.  Please we await for your feedback to know if it works for you.


move regexp definition in lookupFiles.

Description of the Change
I am using mocha programmatically, along with its utility lookupFiles.
Upon calling lookupFiles A regular expression must be evaluated to lookup files although it is not needed in the majority of cases. Shifting the re variable definition down to where it is needed will avoid this issue.
The main reason for this change is to eliminate the requirement of always sending an array to the extensions parameter, moving the re variable definition to just right above where it is needed will achieve both goals.

Alternate Designs

An if statement with a default value for the extensions input param. The proposed version is the neatest and most verbose way to do it.
Why should this be in core?

I am using mocha programmatically along with its utility, this is just a tiny enhancement. also it dictates less knowledge about the function internals to use it since it marks the extension parameter as optional, rather than a required array.
Benefits

Extensions input parameter to the function is now optional, no need to always send an array.
No need to evaluate a regex that might not be used in the majority of use cases.

Possible Drawbacks

Its a super minor change so i dont see any drawbacks.
Applicable issues
Patch release, a tiny enhancement.

Coverage increased (+0.05%) to 89.85% when pulling ffc98f3f19bdda616c10e306cd0f2f783d2e520c on Bamieh:patch-1 into da901da1af3d911670da7c6df6f9818d89c39ae1 on mochajs:master.

@Bamieh Thanks.  This looks good.  Was wondering if you could add a test to assert that extensions is no longer required?

Coverage increased (+0.05%) to 89.995% when pulling 1c19c8362e9d1bfd1aff009946f1f0b05be38a4b on Bamieh:patch-1 into 3200d414c5853de15d7ae2f6d7e841c356176746 on mochajs:master.

@boneskull i added a couple of unit tests

Coverage increased (+0.05%) to 89.995% when pulling 1c19c8362e9d1bfd1aff009946f1f0b05be38a4b on Bamieh:patch-1 into 3200d414c5853de15d7ae2f6d7e841c356176746 on mochajs:master.


feat: add react-i18n package to Hilla React dependencies

This feature was originally intended to be introduced in 24.5 alpha but it's not going into 24.4 since that version is late. I wonder if we should make sure we remember to revert this change from the 24.4 branch once it's created?