just load url in site tree without scan and save the session and exit.

Hi I am looking out for some help with the zap-cli where I need to just add a url into site tree and donot perform any scan and exit by saving the session. I want this because I want to automate scanning for different urls via Jenkins and for that I need zap session for every url. Please I need way via zapcli to just add url to site tree and not scan it and exit by saving.
Thank You...

Hi @prem9753! You can do this pretty easily, the sequence of commands would just be:
# Start ZAP
zap-cli start
# Open a URL through the proxy to add it to the site tree
zap-cli -v open-url http://<IP_ADDRESS>/index.php?foo=bar
# Save the session to a file
zap-cli -v session save /path/to/file/some-session-name.session
# Shutdown ZAP
zap-cli -v shutdown

This will save the session to the file located at /path/to/file/some-session-name.session. Then if you want to use that session later, you can simply load it into ZAP (if ZAP is running) using:
zap-cli -v session load /path/to/file/some-session-name.session

I hope that helps!

Thank you very much you are a savior it worked for me i put them across the batch file and executing them and its working fine for me.
But it working fine for me when i do spider (zap-cli -v spider https://<IP_ADDRESS>/index.php?foo=bar)
but when i give open-url with https it throws the error  as below(zap-cli -v open-url https://<IP_ADDRESS>/index.php?foo=bar).
Is it something related to certificates i need to disable...? does open-url cannot work with https requests? Please could you provide some help in this regards. Thank you...
=============================ERROR======================
C:\Python27>python.exe Scripts\zap-cli-script.py open-url https://xx.xxx.x.xx/
←[1m←[36m[INFO]            ←[0mAccessing URL https://xx.xxx.x.xx/
C:\Python27\lib\site-packages\requests\packages\urllib3\util\ssl_.py:334: SNIMis
singWarning: An HTTPS request has been made, but the SNI (Subject Name Indicatio
n) extension to TLS is not available on this platform. This may cause the server
to present an incorrect TLS certificate, which can cause validation failures. Y
ou can upgrade to a newer version of Python to solve this. For more information,
see https://urllib3.readthedocs.io/en/latest/advanced-usage.html#ssl-warnings
SNIMissingWarning
C:\Python27\lib\site-packages\requests\packages\urllib3\util\ssl_.py:132: Insecu
rePlatformWarning: A true SSLContext object is not available. This prevents urll
ib3 from configuring SSL appropriately and may cause certain SSL connections to
fail. You can upgrade to a newer version of Python to solve this. For more infor
mation, see https://urllib3.readthedocs.io/en/latest/advanced-usage.html#ssl-war
nings
InsecurePlatformWarning
Traceback (most recent call last):
File "Scripts\zap-cli-script.py", line 11, in
load_entry_point('zapcli==0.9.0', 'console_scripts', 'zap-cli')()
File "C:\Python27\lib\site-packages\click\core.py", line 664, in call
return self.main(*args, **kwargs)
File "C:\Python27\lib\site-packages\click\core.py", line 644, in main
rv = self.invoke(ctx)
File "C:\Python27\lib\site-packages\click\core.py", line 991, in invoke
return _process_result(sub_ctx.command.invoke(sub_ctx))
File "C:\Python27\lib\site-packages\click\core.py", line 837, in invoke
return ctx.invoke(self.callback, **ctx.params)
File "C:\Python27\lib\site-packages\click\core.py", line 464, in invoke
return callback(*args, **kwargs)
File "C:\Python27\lib\site-packages\click\decorators.py", line 26, in new_func
return ctx.invoke(f, ctx.obj, *args[1:], **kwargs)

File "C:\Python27\lib\site-packages\click\core.py", line 464, in invoke
return callback(*args, **kwargs)
File "C:\Python27\lib\site-packages\zapcli\cli.py", line 102, in open_url
zap_helper.open_url(url)
File "C:\Python27\lib\site-packages\zapcli\zap_helper.py", line 132, in open_u
rl
self.zap.urlopen(url)
File "C:\Python27\lib\site-packages\zapv2_init_.py", line 124, in urlopen
return requests.get(url, proxies=self.__proxies, verify=False, *args, **kwar
gs).text
File "C:\Python27\lib\site-packages\requests\api.py", line 70, in get
return request('get', url, params=params, **kwargs)
File "C:\Python27\lib\site-packages\requests\api.py", line 56, in request
return session.request(method=method, url=url, **kwargs)
File "C:\Python27\lib\site-packages\requests\sessions.py", line 488, in reques
t
resp = self.send(prep, **send_kwargs)
File "C:\Python27\lib\site-packages\requests\sessions.py", line 609, in send
r = adapter.send(request, **kwargs)
File "C:\Python27\lib\site-packages\requests\adapters.py", line 497, in send
raise SSLError(e, request=request)
requests.exceptions.SSLError: [Errno 8] _ssl.c:499: EOF occurred in violation of
protocol

That looks like an SSL issue on either your machine or your site. I'd suggest upgrading Python 2.7 to the latest version on your machine if it's not already, and seeing if that fixes it.


Maybe a more powerful extension manager is needed.

Like that abilities to anylize which extension makes vscode loaded too slow.(Now only can do that in "Performance")
and report broken or malfunctioned extensions and make alert of extensions with malicious actions.

Did you try Developer: Show running extensions command?


 somehow conflicts with mini.clue

I am using dial.nvim alongside mini.clue and have recently noticed that <C-a> in visual mode doesn't work. In normal mode <C-a> works, and <C-x> (and also g* variants) work in both modes.
I have also opened similar issue with mini.clue: https://github.com/echasnovski/mini.nvim/issues/1059
MWE:
local root = vim.fn.fnamemodify("./.repro", ":p")
for _, name in ipairs({ "config", "data", "state", "cache" }) do
  vim.env[("XDG_%s_HOME"):format(name:upper())] = root .. "/" .. name
end
local lazypath = root .. "/plugins/lazy.nvim"
if not vim.loop.fs_stat(lazypath) then
  vim.fn.system({ "git", "clone", "--filter=blob:none", "https://github.com/folke/lazy.nvim.git", lazypath, })
end
vim.opt.runtimepath:prepend(lazypath)

local plugins = {
    "monaqa/dial.nvim",
    {
        "echasnovski/mini.clue",
        opts = function()
            return {
                triggers = {
                    { mode = "n", keys = "g" },
                    { mode = "v", keys = "g" }
                }
            }
        end
    }
}

require("lazy").setup(plugins, { root = root .. "/plugins", })

vim.keymap.set("n", "<C-a>", function()
    print("n <C-a>")
    require("dial.map").manipulate("increment", "normal")
end)
vim.keymap.set("v", "<C-a>", function()
    print("v <C-a>")
    require("dial.map").manipulate("increment", "visual")
end)
vim.keymap.set("n", "<C-x>", function()
    print("n <C-x>")
    require("dial.map").manipulate("decrement", "normal")
end)
vim.keymap.set("v", "<C-x>", function()
    print("v <C-x>")
    require("dial.map").manipulate("decrement", "visual")
end)

Reproduced the problem, and it helped me to determine the cause. Thanks for the report.
dial.nvim uses the command g@ internally, and the definition of g@ was changed inside mini.clue, causing the conflict.
We don't know why g@ doesn't work as expected (you'll have to ask mini.clue to figure it out).
However, at least using the default g@ on dial.nvim should solve the problem, so I'll try to fix it that way.

@simonmandlik #95 should fix the problem. Could you try the following configuration, update the plugin (with :Lazy + U), and check whether v_CTRL-A works? (it worked in my environment)
 local plugins = {
-    "monaqa/dial.nvim",
+    { "monaqa/dial.nvim", branch = "fix-cmd_normal_bang" },
     {
         "echasnovski/mini.clue",
         opts = function()
             return {
                 triggers = {
                     { mode = "n", keys = "g" },
                     { mode = "v", keys = "g" },
                 },
             }
         end,
     },
 }

Thanks for the quick fix!
Preferring :normal! ... instead of :normal ... for executing some built-in functionality is indeed usually a safer way. Not the OP, but for me it indeed seems to fix it.

Yes, I can confirm the fix works, thanks!


Readme changed with the new way of getting access token method

Description
The existing method to get the access token from Launch browser console is no longer available. So, updated the readme with the new method 'copy(adobeIMS.getAccessToken())'

Related Issue
https://github.com/adobe/reactor-sdk-javascript/issues/53

Motivation and Context
Correction in technical documentation. So, everyone can continue using the library without blocking.

How Has This Been Tested?
N/A

Screenshots (if appropriate):
Types of changes

[x ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x ] I have signed the Adobe Open Source CLA.
[x ] My code follows the code style of this project.
[ x] My change requires a change to the documentation.
[ x] I have updated the documentation accordingly.
[ x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Gonna close this one since it's a duplicate of #55 and I saw and reviewed that one first.


disable HTTP connection reuse in AWS e2e tests

Fixes: #1110

Summary

Release Note

Documentation

/hold

This has passed on 4 runs after previously failing pretty consistently.

To confirm, how many times will this retry?


Add reboot parameters.

MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[ ] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.

adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[ ] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

/azp run automation - sdk

Can one of the admins verify this patch?

@Priya91 Can you please update the examples to reflect the parameter changes

/azp run automation - sdk

@mechandr Added examples.

/azp run automation - sdk

@ChenTanyi Are you the reviewer for ARM? Can we get your feedback on whether the 'isMaster' change is OK?

New property in preview api-version. Signing off from ARM side.

@TimLovellSmith Does your comment need @Priya91 to reply or change? If not, I will merge the PR. I think ARM approved with the isMaster property due to it is a preview version.


Software Engineer Access/Setup for Angela DeGolier

Need

[ ] regular Azure access
[ ] SU
[ ] VPN
[ ] Keybase (adegolier)

I think? Not sure what all you need for this, but presumably you'll need my CDC user which is UOQ2. If there's anything else, please let me know!

@jeremy-page checking to see if we can get some traction on this ticket. Please let me know if we can provide anything to help. Thanks!

Please make sure we know the ticket is in the queue.

That was probably my fault. I know I posted the original one (that went to the black hole) to Slack, but I may not have re-posted this one. Is that the best way to let y'all know that the ticket is in the queue or is there something else we should do in the future?

@GilmoreA6 Kant and Jamie will need this same access, correct? If so, I will start a ticket for them

@LaFreda1 Do they have CDC access? Because if not, they'll need that first.


Modified test files to use project uri's that point to project root r…

Minor change to test files to use project uri's that point to project root rather then submodules.

I am afraid that I am out of my depth here. I don't know what the tests are supposed to test. I was working on a minor PR #711, where Sam noticed the paths pointing to submodule rather then root, and asked to change to root. I will do what you suggest, just be aware that I am doing this a bit blind.

Hi @carolynvs. I went over the tests and the reason they all work is because they test only parsing/hashing of the manifest and lock files. No checks against actual repositories are performed. The checks now work properly as the hases/refs etc are all in alignment. Changing them to point to actual repos will bring nothing to the tests, as I said actual repos are not being queried. But I am afraid of breaking the multitude of tests, as they are testing all sorts of functionality. I have no problem closing this PR if you feel that the whole change should still be done. But I will do this when I feel more confident about the extent of the code I will be impacting.

Thanks for checking! Since this is all just "data that should parse correctly", I'm fine with leaving off tweaking the other fields. If becomes an issue later, then we can fix that later. 😀

Thanks for grabbing this, @carolynvs - sorry I dropped it, @mikijov, i've been really swamped these past couple weeks.

Thanks to both of you. I'm learning go and the codeset at the same time, so I am grateful you are also being patient with me.

@mikijov Thank you for the fixes! Every bit helps.


Appstudio update test-component-pac-yblf

Pipelines as Code configuration proposal

Pipelines as Code CI/test-component-pac-yblf-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
6 seconds
init

✅ Succeeded
16 seconds
clone-repository

✅ Succeeded
36 seconds
build-container

✅ Succeeded
11 seconds
clair-scan

✅ Succeeded
31 seconds
clamav-scan

✅ Succeeded
13 seconds
deprecated-base-image-check

✅ Succeeded
14 seconds
sanity-inspect-image

✅ Succeeded
8 seconds
sbom-json-check

✅ Succeeded
6 seconds
sanity-label-check

✅ Succeeded
7 seconds
sanity-optional-label-check

✅ Succeeded
5 seconds
show-summary

Pipelines as Code CI/test-component-pac-yblf-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
6 seconds
init

✅ Succeeded
16 seconds
clone-repository

✅ Succeeded
36 seconds
build-container

✅ Succeeded
14 seconds
sanity-inspect-image

✅ Succeeded
13 seconds
deprecated-base-image-check

✅ Succeeded
11 seconds
clair-scan

✅ Succeeded
31 seconds
clamav-scan

✅ Succeeded
8 seconds
sbom-json-check

✅ Succeeded
7 seconds
sanity-optional-label-check

✅ Succeeded
6 seconds
sanity-label-check

✅ Succeeded
5 seconds
show-summary


Upgrade reactor-core to 3.2.x

Reactor 3.2.0 (Californium) was released nearly 9 months ago. The 3.2.x series is more stable with a variety of improvements and bug fixes.
The outdated dependency conflicts with another library my project uses, so upgrading would be very appreciated. However, I'm not a Reactor power user, so my knowledge is pretty limited and I'm not sure how painful this upgrade would be.
Is there a reason for the delay? Let me know if there's anything I can do to help you upgrade to 3.2 :-)

@JamesGallicchio Thanks for your comment.
There are 2 main reasons that the upgrade gets delayed:

I’m in the middle of major refactoring that have breaking backward compatibility. I’m trying to make it backward compatible.
My project has completed using the earlier version and doesn’t have major changes required. Hence my activity on this is a little bit lower than usual.

However if you need to upgrade to 3.2, I can look into it in the next couple of days. Alternatively, feel free to raise PR and I can take a look and help on releasing it.

@JamesGallicchio It turns out that upgrade to 3.2.x is already on the way in the new version. It's a major version upgrade with some of old classes being deprecated.
You can start trying version 0.4.0

I'd be happy to help with upgrading.


Failing test case where a used method is not created properly by Specimin

In particular, Specimin does not create the org.example.Foo#isGood() method (I'm not sure why).
The goal of this test was to make sure that Specimin correctly makes methods/fields that need to return booleans (because they are used in if guards) actually have boolean returns (instead of a synthetic return class). That part seems to be working (good!), but I'm not sure why the isGood() method isn't created. @LoiNguyenCS can you take a look at this and either 1) fix it, or 2) let me know why my test case isn't correct? Thanks.

Professor, thank you for the test. The codes to update types for fields were incorrect. I have fixed that code and also created a separate method for ease of future maintenance.

For some reasons I can not request reviews from you for this PR. So I will tag your name here: @kelloggm.


Update ActivitiesController.php

similar to https://github.com/pipedrive/client-php/pull/10

similar to https://github.com/pipedrive/client-php/pull/10

@peppelauro  thank you for your PR, but due_date is used in camelCase while using the method. Closing this PR as it was fixed in https://github.com/pipedrive/client-php/pull/22


Update ISEmoji View to support recent emojis, styling.

https://github.com/isaced/ISEmojiView/pull/19

Thank you, don't have a lot of time right now though. If you're willing to open a PR that will be very welcomed ;)


Add Support for Neovim (LSP or Coc)

Add Support for Neovim (LSP or Coc)

Hi, not sure if this is the one you're looking for but here is the instructions for nvim-lspconfig: https://github.com/neovim/nvim-lspconfig/blob/master/doc/server_configurations.md#vls


Helm Charts

As a potential user of NKG
I want to use a Helm chart to install the NGINX Kubernetes Gateway
So that I can use one command to install the NGINX Kubernetes Gateway
And so that I can easily update.
Acceptance

A helm chart to install NKG is accessible to users
Documentation is added to our installation documentation to show both Helm and YAML methods of installation.
Users are able to pull from GitHub charts to install.

Ingress Controller: https://github.com/nginxinc/kubernetes-ingress/pkgs/container/charts%2Fnginx-ingress

Users are able to pull both edge and release versions

Do we want contain all of our CRDs in a single yaml so they can be replaced by Helm? This should allow an easy upgrade to our CRDs.

To add context, this is something that NGINX Ingress Controller is specifically looking at doing to be easier to automate as well as maintain.


Trouble getting started with the example project

FIrstly - Unity won't allow you to open scene files that are installed via UPM so I copied the examples to my assets directory. At this point I got errors because of duplicate names. I closed Unity, removed the examples entry from packages.json and restarted.
However - the example scenes seemed to have lost references and nothing worked.
Personally speaking, I'd recommend publishing the example as a full Unity project. It's a much more reliable way to distribute working code and the presence of a ProjectSettings directory will clarify which Unity version it's expected to work with.
Another issue is the use of Odin. It appears to only be used for inspector buttons and could probably be replaced with https://github.com/dbrizov/NaughtyAttributes which has the same functionality and is open source.
I'm still interested in trying Graphene so any pointers on fixing the issues above would be gratefully received. I'm happy to contribute PRs wherever that would be helpful but I need to get the examples working before I can be much use in that department.

Hi @andybak ! I'll heed your advice and split up the online demo from the sample project, and provide the latter in the form of a Unity Project.
I also experience the lost references issue - unfortunately this seems to be an issue with Unity git packages that are re-imported (e.g. by upgrading to a different version) whilst the editor is open. Restarting the editor has always fixed this issue for me, but I agree it is something that needs to be figured out structurally.
The Odin issue has since been resolved, thanks for bringing it to my attention. Very happy to hear you're willing to contribute. I'll keep you posted with the changes!

Hi @andybak  - sorry for the long wait. I've started working on improving the usability of the project this week, including its documentation. You can find the new sample project under the following repo:
https://github.com/LudiKha/Graphene-Sample-Project
It should be plug & play. I'll be updating the documentation further this week.


Avoid duplication of a method by expanding out the struct into their constituting fields.

This is an implementation detail level change, not needed for the current milestone, but can be added if desired since it is fairly low risk.
Follow-up from https://github.com/Azure/azure-sdk-for-cpp/pull/2590#discussion_r667240012

/check-enforcer override

The CI leg is actually green, but GitHub UI isn't showing it:
https://dev.azure.com/azure-sdk/public/_build/results?buildId=1012203&view=results


Node js real time

Hello,
thank you very much for this library!
I'm trying to run a small face detection program in node js. It takes as input a frame read from the webcam and output the detection object.
I see that the browser version does realtime detection while with the node version it takes around 350ms per frame which is very low compared.
I took some measures of just the prediction step
console.time("detect")
const detections = await faceapi.detectAllFaces(frame)
console.timeEnd("detect")

detect: 442.732ms
detect: 363.256ms
detect: 365.847ms
detect: 338.513ms
detect: 334.374ms
detect: 340.412ms
detect: 324.549ms
detect: 333.849ms
detect: 327.226ms
detect: 325.315ms
detect: 322.517ms
detect: 322.398ms
I am already importing the tfjs-node library and I am running the program on a 2,6 GHz Intel Core i7 quad-core 16gb ram MacBook.
I just want to know if that frame rate is common or I am facing slow performances.
Thank you

@tonxxd - I'm at pretty much the same point as you and coming up against this issue running tfjs in Node on a Rasp Pi 4. The best performance I can get at the moment is around 550-600ms....
Detecting Faces: 595.007ms
Detecting Faces: 588.997ms
Detecting Faces: 549.516ms
Detecting Faces: 616.711ms
Detecting Faces: 580.152ms
Detecting Faces: 562.429ms
Detecting Faces: 540.243ms
Detecting Faces: 551.98ms
Detecting Faces: 532.101ms
Detecting Faces: 578.383ms
From reading a number of other issues, here are some things to check.

Do you have the same version of @tensorflow/tfjs-node as is in the current version of face-api.js's package.json? Currently this is...

    "@tensorflow/tfjs-core": "1.7.0",
    "@tensorflow/tfjs-node": "1.7.0",
    "@tensorflow/tfjs-node-gpu": "1.7.0",
    "face-api.js": "0.22.2",

Have you tried using tfjs-node-gpu over tfjs-node, this made little difference for me on Raspberry Pi but may help on a Macbook.
What nets are you using, apparently some have different performance impacts?

Hi @ChrisDalley thanks for the quick response!
I tried the tiny face detector and it gives much higher performances (~ 60ms per frame). Changing to node-gpu didn't increase performances (strange(?)).
Anyway, what I don't understand is why the node version is slower than the browser one when tensorflow has access to the Native C modules in the node env.
I have the exact same app in python (with tensorflow for python and opencv-python) and I get real-time detections without issues.

I'm honestly not too sure on the difference between Python and the node implementation, it's my first time touching this library today. I'm going to do some more digging over the next few days and see what I can figure out - even getting down to 60ms for me would be better than where it's at right now!

@ChrisDalley yes but that was on my MacBook, since I plan to run the app on fewer resources I am looking for a better strategy as you

@ChrisDalley could you able to get any improvement? I am also facing same issue in my MacBook. It is taking ~7.5 sec for each image which is quite a lot in these days. I used GPU as well, but not much difference.
Please help if you got any breakthrough.
Thanks in advance.


feat: Add orgnaization tabs and settings tab skeleton

Adds tabs for organization that is hidden if the available tabs <= 1
Adds skeleton for settings page that can be reused in project settings page.

Yes @lovincyrus is correct, the intent was for only one tab bar to show at a time. The org tab bar when you're on an org page, the project tab bar when you're on a project page.

@lovincyrus The outline is not really visible to users. The start of rill logo aligns with the tab background when hovered. You can see that in the 1st image you posted.
Side note the rill logo doesnt have a hover state in mocks but it does in UI. We should figure it out separately.


Fixed an issue where void was ending up in the schema definition

Some APIs do not have a response type, Like a 404 response.
Currently this causes the void "object" ending up in the schema definition.
To reproduce just add [(int) HttpStatusCode.NotFound)] to one of the function endpoints.
This change will filter out void from the schema definition

Great, thank you!


[WIP] PyThaiNLP 4.0 change log

This post will give you the change log for PyThaiNLP 4.0. PyThaiNLP published the first version is 0.0.4 to PyPI at 6 years ago, so PyThaiNLP 4.0 will have special codename. The codename for PyThaiNLP 4.0 is PyThaiNLP 4.0 (Real).
Schedule

First development release: [WIP]
Beta release: [WIP]
Production release: [WIP]

See 3.2 Milestone.
What is new?
Deprecation and other API changes

Delete all LST20 model #728
https://github.com/PyThaiNLP/pythainlp/commit/947c7be9ce4199af58ecf042629dda4d752dbcd6 Change pythainlp.tools.misspell to pythainlp.tools.misspell.misspell

Improve

Reduce import time #719
Fix/broken numeric data format (#652) #723

Tokenizer

Add blackboard cls #732
Add  rule to TCC and Change TCC rule for newmm #741

Tag

Add blackboard pos_tag #733

Util

Add pythainlp.util.count_thai_chars #748
Add thai_strptime and convert_years #767

Transliterate

Add Thai2Rom ONNX model #743

Parse

Add ud_goeswith #757

Corpus

Add new science word #763

[WIP]

Schedule

Beta release: 1 April 2023
Production release: 14 April 2023


Fix for the loop end conditions in the ReferenceSegmenterParser.

Coverage remained the same at 34.74% when pulling 8a8688447990d1c06ce2665e1b64378740ddd237 on aoboturov:partial-merge-pr-216 into 7f0eea314c6eeb257e4d402a81b59180162b8828 on kermitt2:partial-merge-pr-216.


Function unit callstack is not exposed

Function unit callstack is not exposed -
When multiple functions are executed, it is not tracked which function took a long time to process or whether an error occurred.

@mathisharp
Node agents do not yet support code level visibility.


Large dispose time of multiple MudPopover

Bug type
Component
Component name
MudPopover - MudTooltip
What happened?
We are using a table with several tooltips (MudPopOver). The load time of this is fine, but when we try to dispose the components the render time becomes about 7 seconds.
After linking our codebase to the repo and doing some testing we found out that the problem was the Dispose of the MudPopover, more specific the js call: mudPopover.disconnect.
This one is called for each item and when disabling this line the performance significantly improves.

Our temporary work around is to implement our own version of the IMudPopoverService with the current implement without the call to js. We haven't experienced any downside's yet.

Expected behavior
Fast dispose time.
Reproduction link
https://try.mudblazor.com/snippet/QOwnkdmOfEptKENI
Reproduction steps
Load multiple mudpoperover/mudtooltips and dispose them by removing them from the view.
Version (bug)
6.1.9
Version (working)
No response
What browsers are you seeing the problem on?
Chrome
On what operating system are you experiencing the issue?
Windows
Pull Request

[X] I would like to do a Pull Request

Code of Conduct

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

Our current override service looks like:
public class MudPopoverNoUnRegisterService : IMudPopoverService
{
    private readonly Dictionary<Guid, MudPopoverHandler> _handlers = new();
    private readonly IJSRuntime _jsRuntime;
    private readonly PopoverOptions _options;
    private readonly SemaphoreSlim _semaphoreSlim = new(1, 1);
    private bool _isInitialized;

    public MudPopoverNoUnRegisterService(IJSRuntime jsInterop, IOptions<PopoverOptions> options = null)
    {
        _options = options?.Value ?? new PopoverOptions();
        _jsRuntime = jsInterop ?? throw new ArgumentNullException(nameof(jsInterop));
    }

    public event EventHandler FragmentsChanged;

    public bool ThrowOnDuplicateProvider => _options.ThrowOnDuplicateProvider;
    public IEnumerable<MudPopoverHandler> Handlers => _handlers.Values.AsEnumerable();

    public async Task InitializeIfNeeded()
    {
        if (_isInitialized)
        {
            return;
        }

        try
        {
            await _semaphoreSlim.WaitAsync();
            if (_isInitialized)
            {
                return;
            }

            await _jsRuntime.InvokeVoidAsyncWithErrorHandling("mudPopover.initialize", _options.ContainerClass,
                _options.FlipMargin);
            _isInitialized = true;
        }
        finally
        {
            _semaphoreSlim.Release();
        }
    }

    public MudPopoverHandler Register(RenderFragment fragment)
    {
        var handler = new MudPopoverHandler(fragment, _jsRuntime, () =>
        {
            /*not doing anything on purpose for now*/
        });
        _handlers.Add(handler.Id, handler);

        FragmentsChanged?.Invoke(this, EventArgs.Empty);

        return handler;
    }

    public async Task<bool> Unregister(MudPopoverHandler handler)
    {
        if (handler == null)
        {
            return false;
        }

        if (_handlers.Remove(handler.Id) == false)
        {
            return false;
        }

        // await handler.Detach();
        await Task.CompletedTask;
        FragmentsChanged?.Invoke(this, EventArgs.Empty);

        return true;
    }

    public async ValueTask<int> CountProviders()
    {
        if (!_isInitialized)
        {
            return -1;
        }

        var (success, value) = await _jsRuntime.InvokeAsyncWithErrorHandling<int>("mudpopoverHelper.countProviders");
        if (success)
        {
            return value;
        }

        return 0;
    }

    //TO DO add js test
    [ExcludeFromCodeCoverage]
    public async ValueTask DisposeAsync()
    {
        if (_isInitialized == false)
        {
            return;
        }

        await _jsRuntime.InvokeVoidAsyncWithErrorHandling("mudPopover.dispose");
    }
}

And service registration:
        services.AddScoped<IMudPopoverService, MudPopoverNoUnRegisterService>();
        services.AddMudServices();

I do not think the above fix is a proper fix so please discuss a fix to resolve this issue.

In MudPopover.js we can see that dispose call disconnect for every item in the map. I would be inclined to believe that your method should just work ?
 disconnect(id) {
    if (this.map[id]) {

        const item = this.map[id]
        item.mutationObserver.disconnect();
        item.resizeObserver.disconnect();
        item.contentNodeObserver.disconnect();

        delete this.map[id];
    }
}

dispose() {
    for (var i in this.map) {
        disconnect(i);
    }

    this.contentObserver.disconnect();
    this.contentObserver = null;
}

MudPopover.razor.cs method DisposeAsync calls the unregister from the Service

Which then calls Detach

That will call disconnect with the semaphore making sure only one task is executed at a time

the service DisposeAsync, which calls dispose in JS seems to be running only with tests

Maybe there should be a difference between unregister of a single component and disposing of a group ?

@Mr-Technician, can this issue be closed now with this being merged https://github.com/MudBlazor/MudBlazor/pull/5367, or is it still an ongoing issue?

@ScarletKuro There was someone on Discord who still had this issue but they may have been using an old version of Mud. If so, this issue can be closed.

I am using 6.2.2, which is higher than 6.2.1 which another issue (possibly similar to this) we resolved in. And I still get a major screen hang when re rendering components that have toootips used in them.

What i found is that the problem lies with with SamahoreSlim.WaitAsync from the MudPopoverHandler in the Detach.
We have fixed this by overwritten the IMudPopoverService that calls the js directly _jsRuntime.InvokeVoidAsyncWithErrorHandling("mudPopover.disconnect", handler.Id);
This doesn't disconnect all handlers but also doesn't create a massive screen hang.
Example of our Unregister
   public async Task<bool> Unregister(MudPopoverHandler handler)
    {
        if (handler == null)
        {
            return false;
        }

        if (_handlers.Remove(handler.Id) == false)
        {
            return false;
        }

        await _jsRuntime.InvokeVoidAsyncWithErrorHandling("mudPopover.disconnect", handler.Id);
        FragmentsChanged?.Invoke(this, EventArgs.Empty);

        return true;
    }

Hi @Gareth064 Yes but i did change the Unregister to remove it (before it left it alive) but without the freeze. Please let me know if you if resolves it for you?

What i found is that the problem lies with with SamahoreSlim.WaitAsync from the MudPopoverHandler in the Detach.

It appears that the semaphore was added in order to prevent potential concurrency issues between the dispose and initialize processes, as mentioned in the pull request #3963.
However, I'm open to suggestions if anyone has a better alternative or idea to address this situation.

I have made the change to add your override service. Annoyingly, I don't have much data to render right now (The app track open orders in a shop) so will have to wait till it gets a bit busy again to see if I feel the impact of the change.

SamahoreSlim.WaitAsync

@Mr-Technician, I was thinking about potential ways to address the performance. One idea that came to mind is fine-tuning the SemaphoreSlim by adjusting the initialCount and maxCount parameters.
Another idea is implementing a batch detachment mechanism.
Does anyone have a reproduction repository with

a major screen hang when re rendering components that have toootips used in them

Mine is a private app so I wouldn't have be able to share a repo etc.
The Try Mud link from this issue shows it well. Would be easy to take the code from it and create a repo?
https://try.mudblazor.com/snippet/QOwnkdmOfEptKENI

A small update: I've created a functional prototype that eliminates browser hang-ups and significantly reduces refresh time in this snippet https://try.mudblazor.com/snippet/QOwnkdmOfEptKENI
However, I can't provide a definite timeline for when this will become a pull request since it requires extensive testing. Additionally, I'm not entirely satisfied with our current API design. In my opinion, the responsibilities of the MudPopoverHandler extend beyond what they should be, and functions like UpdateFragmentAsync, Initialize, and Detach should have been incorporated into the Service instead. This would prevent duplication of Semaphore and JsRuntime in both places. Ideally, it would be better to make a v2 and deprecate the current handler and service. Regrettably, I'm currently lacking the time to undertake all of these tasks.

Performance of old PopoverService:

Performance of new PopoverService:

I hope this speed will satisfy everyone (second gif). It's still using the lock mechanisms just in case.

I hope this speed will satisfy everyone (second gif). It's still using the lock mechanisms just in case. Soon I will make a PR.
New Popover Service API

Is it correct that this is only the interface and not the implementation of the interface?

@thopdev you are correct. The old interface will be obsoleted but remains usable in v6.x.x. The plan is to remove it in v7.x.x
However, if I understood @ScarletKuro correctly the old popoverservice will still benefit from the performance gains of the new implementation by forwarding.

I like the new API, indeed very easy to use.

The new service looks a lot faster. Noob question though.... there still appears to be a screen freeze, is that correct? Does this mean we still can't show some sort of loading UI when pressing that refresh button because the UI is freezing?

The new service looks a lot faster. Noob question though.... there still appears to be a screen freeze, is that correct? Does this mean we still can't show some sort of loading UI when pressing that refresh button because the UI is freezing?

As demonstrated in the GIF, we still have the ability to add loading functionality. Although there is a slight delay when creating a large number of elements, it is not a complete freeze like the previous version that would hang the browser. The UI remains semi-responsive, allowing scrolling and clicking, although some animations may experience slight delays.
The delay is not solely related to the popover service. Even if we remove all the locking mechanisms, there will still be an initial delay when loading such a significant number of popovers. This delay occurs because each popover is updated, triggering a StateHasChanged call for each popover element. Additionally, when the popovers are added to the list, it results in 400 notifications for the MudPopoverProvider to render them, resulting in a total of 800 re-render calls.
In the future, we could consider introducing a feature like <MudPopover Queue="true"/>, which would lazily queue the popovers for rendering. This way, we could update them in batches to minimize the number of re-renders and mitigate the current issue. The new IPopoverService allows for such extensions, but for now, the current implementation is considered sufficient.

By the way, I wanted to share some additional information with everyone. I mentioned earlier the possibility of reducing the frequency of StateHasChanged calls to a minimum. I conducted some testing on top of this PR #6953, and the results showed that it only reduced the execution time by around 200ms. Although the implementation of this optimization is relatively straightforward, as the new PopoverService provides great flexibility and we could include it as experimental feature in the MudGlobal, I don't believe it is worth pursuing. The main reason is that implementing this feature is easy, but testing it is a different story. The feature introduces a slight delay when the popover is shown or hidden, which significantly complicates unit testing efforts.
Considering these factors, I have concluded that it is not advisable to proceed with the optimization at this time. The potential benefits are relatively small compared to the added complexity and challenges it introduces for testing.
But maybe in future we could try another idea - when we drop .NET 6 support. One potential option is to leverage the [JsImport] attribute for WAS) scenarios. This approach avoids marshaling objects to JSON, which theoretically could improve performance, especially when there are numerous JavaScript calls involved, such as when dealing with multiple popovers.


solution needed for below problems in C++

Add solution for these problems and add their solution links under solution section in readme file
14  Lucky Numbers
15  Numbers Histogram
26  Binary Search

!assign

Hey there! Can you please assign this to me.


add trivy dojo report operator to docs

Description
Add docs for the trivy dojo report operator.
Checklist
This checklist is for your information.

[ ] Make sure to rebase your PR against the very latest dev.
[ ] Features/Changes should be submitted against the dev.
[ ] Bugfixes should be submitted against the bugfix branch.
[ ] Give a meaningful name to your PR, as it may end up being used in the release notes.
[ ] Your code is flake8 compliant.
[ ] Your code is python 3.11 compliant.
[ ] If this is a new feature and not a bug fix, you've included the proper documentation in the docs at https://github.com/DefectDojo/django-DefectDojo/tree/dev/docs as part of this PR.
[ ] Model changes must include the necessary migrations in the dojo/db_migrations folder.
[ ] Add applicable tests to the unit tests.
[ ] Add the proper label to categorize your PR.

Extra information
Please clear everything below when submitting your pull request, it's here purely for your information.
Moderators: Labels currently accepted for PRs:

Import Scans (for new scanners/importers)
enhancement
performance
feature
bugfix
maintenance (a.k.a chores)
dependencies
New Migration (when the PR introduces a DB migration)
settings_changes (when the PR introduces changes or new settings in settings.dist.py)

Contributors: Git Tips
Rebase on dev branch
If the dev branch has changed since you started working on it, please rebase your work after the current dev.
On your working branch mybranch:
git rebase dev mybranch

In case of conflict:
 git mergetool
 git rebase --continue

When everything's fine on your local branch, force push to your myOrigin remote:
git push myOrigin --force-with-lease

To cancel everything:
git rebase --abort

Squashing commits
git rebase -i origin/dev

Replace pick by fixup on the commits you want squashed out
Replace pick by reword on the first commit if you want to change the commit message
Save the file and quit your editor

Force push to your myOrigin remote:
git push myOrigin --force-with-lease

@rndmh3ro PRs need to be made against the dev branch, not master.  For details see https://github.com/DefectDojo/django-DefectDojo/blob/master/readme-docs/CONTRIBUTING.md
Closing this PR. Please re-submit against the dev branch.


Remove files having same content

Describe the task
There are two conflicting files that have the same content. These files are for displaying the navbar on the main page.
Expected behavior
Remove the file which does not reflect any changes on the navbar.

Closing


Cloudflared metrics

resolve https://github.com/STRRL/cloudflare-tunnel-ingress-controller/issues/9

Codecov Report

Merging #13 (5f85cb0) into master (440ba79) will decrease coverage by 0.61%.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##           master      #13      +/-   ##
==========================================
- Coverage   24.70%   24.09%   -0.61%
==========================================
  Files           9        4       -5
  Lines         587      249     -338
==========================================
- Hits          145       60      -85
+ Misses        434      189     -245
+ Partials        8        0       -8

see 5 files with indirect coverage changes


Starting possible fix for #1200

I wanted to see if we were happy with the direction of this fix for #1200
If the proxy address has a query on the end ?some=value then discard any additional path information .
eg
localhost:8001
proxy / localhost:4151/pub?topic=caddy

then the url localhost:8001/big/long/url/index.php?other=value
would proxy to localhost:4151/pub?topic=caddy&other=value
the information contained in big/long/url/index.php is discarded.
Is this the desired behaviour?
ps. I know I will need to add test before submitting.

Actually I propose to disable query in the destination endpoint to proxy to. So `proxy / localhost:4151/pub?topic=caddy' will be invalid.

@elvarb Do you have an opinion on this?

@tobya this solution would probably work in my case. This is a tougher problem to solve than I thought.
In this case would the url with other=value1 replace the proxy value of other=value2 ?
URL: localhost:8001/big/long/url/index.php?other=value1
Configured proxy: localhost:4151/pub?topic=caddy&other=value2
Results: localhost:4151/pub?topic=caddy&other=value1
Or
Results: localhost:4151/pub?topic=caddy&other=value2

I would need to test it but I would guest it will be
Results: localhost:4151/pub?topic=caddy&other=value2&other=value1

What happens then is up to the server.
I am starting to agree with @tw4452852 that this may just be illegal to have a query in a proxy url.
I think what you want is really a redir?

Hmm, what does it really mean to proxy to a URL with a query string? Are you adding those parameters, overwriting them, or replacing all of them?

Adding the parameters seems reasonable
Overwriting them (setting them) seems a bit cavalier
Replacing the entire query string with the one in the proxy config seems destructive

Duplicate QS parameters are allowed, I think that would be okay, no?
Either way, I think that the current (?) behavior of detecting whether the upstream is a directory or a file by looking for a file extension is not very robust. What are the consequences of not adding the / at all?

@mholt  The problem really is what to do in all instances.
We can not add the / when there is no extra path added to the incoming path to be proxied, but what do we do when there is an incoming path to be proxied?
eg. given caddyfile we are dealing with
localhost:8001
proxy / localhost:4151/pub?topic=caddy

then the url localhost:8001?other=value
is fine, it would proxy to
localhost:4151/pub?topic=caddy&other=value

but what happens when there is an incoming path
localhost:8001/big/long/url/index.php?other=value

Now you need to make a  choice.  do we treat pub as part of the path or a file?
You can proxy to

localhost:4151/pub/big/long/url/index.php?topic=caddy&other=value

or

localhost:4151/pub?topic=caddy&other=value

We need to have a very clear indication as to what happens.  I am begining to agree with @tw4452852 that this should not be allowed.  I think what @elvarb is trying to do is better achieved with a redir

Thanks for the explanation @tobya. Do we know how other servers do this? e.g. Nginx.
But as I read that, with the instance given here, I'm feeling like the first destination is probably more correct: localhost:4151/pub/big/long/url/index.php?topic=caddy&other=value


swift got a error:  "Error validating transaction: Transaction 01dde1a3d0521f7b00394674de58a9fb2da1db8fe8e4e084aa2eaa377975eb54 orphaned, missing reference 828725f838eea6136f54113ad975017d272151ebd174f8427b0f45e4941ca9b2."

ios swift
My btc address:  bc1qvut9je4y3hvnnlqw4ezyd5mn778jj9flqkky7c

create a transaction :
01000000000101b2a91c94e4450f7b42f874d1eb5121277d0175d93a11546f13a6ee38f82587820000000000ffffffff02a086010000000000160014874cc80fdcee23f59e7b9b09a7ebd90c45b9fcbfe26e01000000000016001467165966a48dd939fc0eae4446d373f78f29153f02483045022100e40a09d72f1b28b17e5a29fab69989bc25871464ce640780a483c0d90122a251022049c905370f1dcde682e58b0fa08b1740e51405aba9a010b5fcc85926f3df37a3012102a901f355b553d3617e64969f5f67133c15540dd5ddc3138e36cd8a75403dfe2e00000000
send transaction got a error:
error: "Error validating transaction: Transaction 01dde1a3d0521f7b00394674de58a9fb2da1db8fe8e4e084aa2eaa377975eb54 orphaned, missing reference 828725f838eea6136f54113ad975017d272151ebd174f8427b0f45e4941ca9b2."
And I descoderawtransaction got the result:

Could you please check?

BitcoinUnspentTransaction.outPoint.hash order, usually you need to reverse it if you get from API
BitcoinSigningInput.scripts, you should set scripts[<script_hash>] = <BitcoinScript.buildPayToPublicKeyHash> for p2wpkh

I try again and agin, got the same err:
"Error validating transaction: Transaction 9a5433797dbc9804e4cecfbe8beb341ff1f5fb27fc637dd3f407bd2631259052 orphaned, missing reference b2a91c94e4450f7b42f874d1eb5121277d0175d93a11546f13a6ee38f8258782."

I update the code as follow:

And I try again and agin, got the same err:
"Error validating transaction: Transaction 9a5433797dbc9804e4cecfbe8beb341ff1f5fb27fc637dd3f407bd2631259052 orphaned, missing reference b2a91c94e4450f7b42f874d1eb5121277d0175d93a11546f13a6ee38f8258782."
I don't know how to do....please help me,thx!

try this:
let script = BitcoinScript.buildForAddress() // utxo address
let scriptHash = script.matchPayToWitnessPublicKeyHash() // to redeem p2wpkh
input.scripts[scriptHash.hexString] = BitcoinScript.buildPayToPublicKeyHash(hash: scriptHash).data

The same error again!!
from:  bc1qvut9je4y3hvnnlqw4ezyd5mn778jj9flqkky7c

this is testnet: https://api.blockcypher.com/v1/bcy/test/txs/push

your utxo is in mainnet

it worked: https://btc1.trezor.io/tx/9a5433797dbc9804e4cecfbe8beb341ff1f5fb27fc637dd3f407bd2631259052

Yes, I see. Thank you very much.  Thank you again for your trouble.

@togiter we'd like to include this tx as a real world test case, would you mind to share the private key of bc1qvut9je4y3hvnnlqw4ezyd5mn778jj9flqkky7c? Thanks

@togiter sorry brother, how are you solving the problem?

Yes, I see. Thank you very much.  Thank you again for your trouble.

hi,How did you solve the problem of testnet,please?

Yes, I see. Thank you very much.  Thank you again for your trouble.

hi,How did you solve the problem of testnet,please?

Please check this error?
Do I need to update SDK ?

@akhilccrb try updating to latest SPM build


fix: ensuring consistent interface for scrollToId

fixes #134 and fixes scrollToId to correctly allow both string and number, as NodeId allows: https://github.com/diogofcunha/react-virtualized-tree/blob/master/index.d.ts#L9
Would be great to get this merged in, so that everyone using the npm package could benefit from the updated TS typings.
Thanks!

Coverage remained the same at 94.498% when pulling 8bf901b5412077aca7cd098936a6d899330763a3 on mateodelnorte:fix/consistent-signature-for-scrollToId into dfaaf7402d56a3eaf1034fbebfc86770d41f3700 on diogofcunha:master.

@diogofcunha Can this PR be merged? There should be zero side effects to the fix.

#153 also addresses this


Configure Error States

We should be able to configure the error states based on both status code (specifying when to rebound on different status codes) as well as being able to interrogate the response body to force a rebound or an error. Especially in a SOAP web service, it's likely that the context about an error will live in the response body, not the status code.

Closing due to related issue


Freeze backbone weights

Hi, thank you for your great work.
I wonder if there is any way that I can freeze the weights of the backbone(swin) and only finetune the mask rcnn model.

You could use config option frozen_stages as this says https://mmdetection.readthedocs.io/en/latest/tutorials/config.html?highlight=frozen_stages#an-example-of-mask-r-cnn

As roselidev points out, you can use config option frozen_stages. You may refer to the implementation here https://github.com/SwinTransformer/Swin-Transformer-Object-Detection/blob/6a979e2164e3fb0de0ca2546545013a4d71b2f7d/mmdet/models/backbones/swin_transformer.py#L557


Add streaming protocol support

DirectLine constructor
streamingWebSocket is introduced. This parameter can be used to tell DirectLine to use the streaming protocol.
postActivity and postMessageWithAttachments
Both methods now use the streaming protocol to send messages if there is a streaming connection.
**Tests
Happy paths for posting activities, activities with attachments  and receiving welcome messages have  automated tests
Unhappy paths of unable to reconnect/connection break were manually tested. The reconnect function has not been tested. I will update its implementation during the review.

Please do some testing in IE11.
There is a blocking bug on botframework-streaming because they are publishing non-ES5 code to NPM. https://github.com/microsoft/botbuilder-js/issues/1670


[WIP] Set tolerance on isometry conversion

Summary
Added a tolerance to the Isometry class so that users have the option to define for themselves instead of stuck with hard-coded value of 1e-10.
Fixes https://github.com/Qiskit/qiskit-terra/issues/3789
Details and comments
The Isometry class has a new eps attribute that can be passed in during initialisation.
I could do with some guidance on:

if the structure of this change "fits" the repos code-style.
whether or not this needs additional tests (the current test_isometry.py file doesn't use the Isometry class explicitly anywhere?)
Where should this change be documented?

@JamesSeaward hi, any update on this?

Since this PR is fixing a "good first issue" and seems stalled, Im closing it to allow other newcomers to have a shot. If @JamesSeaward is back, it can be reopen.


Improve cut list view

It is still cumbersome to use, especially if you want to edit automatic detection results

Probably the best way to improve it is to remove editing capabilities


feat: add Continuum Power CCA

This PR introduces Continuous Power CCA (CPCCA) model to the API (close #178 ). Along with this, significant refactoring of the _CrossBaseModel class was required, leading to the addition of several new features.
Key Changes

New Features:

Added the ContinuousPowerCCA class and its Hilbert version ComplexCPCCA.
Enhanced MCA classes with new functionalities related to correlation coefficients, explained variance  and a predict method for forecasting Y from X (see here.
Introduced homogeneous_patterns() and heterogeneous_patterns() methods for the ComplexMCA model.

Testing:

Added basics tests for ContinuousPowerCCA
Updated some of the existing tests for MCA classes to align with changes in default parameters

Miscellaneous:

Refactored MCA class (now a special case of ContinuousPowerCCA)  along with related clases: ComplexMCA, MCARotator, ComplexMCARotator.
Split the original base models into three classes, separating functionalities regarding serialization and lazy model evaluation into _BaseModel, with specific model implementations now in _BaseModelSingleSet and _BaseModelCossSet.

Open To Dos

[x] update and streamline docstrings in new models and methods
[x] allow full lazy evaluation mode; currently there are a few hiccups in the Whitener class

Hey @slevang, would you like to review this PR? It’s a bit extensive - sorry about that! No worries if you’re busy; I plan to merge it into the develop branch along with some other breaking changes before it goes live.

I'll try to take a look at the serialization issue in the next few days, but probably won't have time to dig into the rest of the implementation right now. Nice work though, this looks cool!


Remove obsolete Yarn

Yarn lock file is being 4 month old and NPM lock seems to be the SSOT when it comes to dependencies. Decided to remove the former one, gitignored, and add a note about that into a relevant doc file. I guess it brings some clarity which package manger should be used.

Thanks @jacekk, I think this PR makes sense,
I know there's a bug where for windows users only yarn helped, and npm failed.
I'm going to try reproduce that tomorrow, and see what's the state of things, and if it's safe to drop yarn lock file.
Should be safe to merge after it

I'm still working on getting a windows laptop, should have it in the next few days :)

@NothingEverHappens @kirjs
For the last couple days I've been playing with Win10 (x64) Stable 1809 on a VirtualBox taken from this page: https://developer.microsoft.com/en-us/microsoft-edge/tools/vms/ . Installing GIT, NodeJS and Yarn went somewhat smooth, but unfortunately, I was not able to install dependencies using NPM. Even running npm install -ddd --no-bin-links failed couple times. Usual npm install failed as well. Most of those failures were system restart, but some were similar to npm ERR! { Error: EPERM: operation not permitted. Most SO advices sounded like use admin mode or run npm cache clean, but I think those should be unnecessary in normal workflow.
However, yarn install went successful in the first try. What is important, yarn.lock was not present on the tested branch (the one from this very PR). It is important, because I am aware we loose specific versions of dependencies by removing lockfile for Yarn, but since it was not up to date in coparison to package-lock.json versions, I find it even more confusing what I should do right now with this PR.
What's the result of your tests? In case those failed as well, I am totally fine with closing this ticket.

P.S.
Installing deps on my linux machine, the node_modules directory takes up to 1.5GB space. Way too much, IMHO.
P.S. 2
I feel really sorry for anyone who is developing bigger nodejs/npm apps on Windows. It's their choice, though :/

Awesome, thanks for the quick merge!

Hey @jacekk, I'm writing a twitter thread about our ivy upgrade, and would like to recognize this PR, as it helped us through the process.
Would it be ok to mention you on twitter, if yes, what's your twitter handle?

@kirjs No Twitter. No Facebook. No LinkedIn. No [you-name-it]. Beside having a GitHub account, I am socially dead ;)


core-api: add initial set of composability plumbing

Hey, I just made a Pull Request!
Fixes #3415, fixes #3416, fixes #3417, fixes #3418, fixes #3461
Skipping changeset for now as this doesn't actually introduce any exported things yet.
:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[x] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)

Workflows plz


adding in6_ifreq to apple.

close #3611

CI fails: https://github.com/rust-lang/libc/actions/runs/8884641673/job/24394199472?pr=3617

@JohnTitor Why does the ci fail on sparc64-unknown-linux-gnu target, it seems to be not related to this PR, I think it can be merged.

@cavivie

@JohnTitor Why does the ci fail on sparc64-unknown-linux-gnu target, it seems to be not related to this PR, I think it can be merged.

I think sparc64 problems are unrelated to this PR, too. Sparc problems was apparently fixed in https://github.com/rust-lang/libc/pull/3707 , so this PR just need to be rebased

I think sparc64 problems are unrelated to this PR, too. Sparc problems was apparently fixed in #3707 , so this PR just need to be rebased

@devnexen Could you please work on this, we may be able to merge soon.


Updated Rabbit Rush: Added new features and UI enhancement

PR Description 📜
As mentioned , I have done subtle changes that were necessary for the game such as UI  improvement and also added new features and buttons. I have tried my best to make the game better.
Fixes # <3369>

Mark the task you have completed ✅

[x] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[x] I have performed a self-review of my own code or work.
[ ] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[ ] I have added screenshot for website preview in assets/images
[ ] I have added entries for my game in main README.md
[x] I have added README.md in my folder
[x] I have added working video of the game in README.md (optional)
[x] I have specified the respective issue number for which I have requested the new game.

Add your screenshots(Optional) 📸

Thank you soo much for contributing to our repository 💗

Thank you @Changingthegame ,for creating the PR and contributing to our GameZone 💗
Review team will review the PR and will reach out to you soon! 😇
Make sure that you have marked all the tasks that you are done with ✅.
Thank you for your patience! 😀

Hey @Changingthegame,
Please make sure to link the relevant issue using the appropriate syntax, such as "#issueNumber" 👀.
Follow the proper guideline and make a new PR again 😀.
Happy Hacking 💗

Thank you  @Changingthegame , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀


add context and getRefId

Coverage decreased (-0.08%) to 88.305% when pulling aa8536a02e01e29722b4ad851bb99b3a21b9c64a on context into 8854271adcdd0dd2ad0c14ca17f23ab49b392616 on master.


Update Readme

About:

[ ] reconciliation
[ ] new transformers (map_remove_special_characters for example, and others)

[ ] string transformer
[ ] translate transformer


Release Google.Cloud.Functions.V2 version 1.0.0-beta01

Changes in this release:
Initial release.

Created release for Google.Cloud.Functions.V2-1.0.0-beta01


When lib.accounts.yaml is missing the browser frontend goes into a page refresh cycle

In 2021.6.16.


bugfix(settings): allow scrolling on settings panel

Before submitting a pull request, please make sure the following is done...

[ ] Ensure the pull request title and commit message follow the Commit Specific in English.
[ ] Fork the repo and create your branch from main.
[ ] If you've added code that should be tested, add tests!
[ ] If you've changed APIs, update the documentation.
[ ] Ensure the test suite passes (npm test).
[ ] Make sure your code lints (npm run lint) - we've done our best to make sure these rules match our internal linting guidelines.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

SettingPanel不需要支持滚动的，我是希望内部元素自己处理滚动，这样才方便比如NodePath这样的组件固定在顶部


PostAgent: allow sending arbitrary string data

When content_type contains a MIME type and payload is not a Hash the string in payload will be send as the HTTP body and content_type is send as the Content-Type header.
#1361

Nice!

Thanks for the review.


Adctl/Admob Upgrade: Modernizing implementation

Admob has many features like Video Ads and so on. The QtAdmob is buggy and needs rework.
Ideas/Propositions
-We need better interfaces for executing ads, we shouldn't be calling 4 methods to place and show an ad.

Better async loading, adctl uses a bizzard way for getting the state of the ads. Ads should report themselves from java/ios to cpp.
Adctl should implement a bridge for more third party to be integraded. These simple may consist of many callbacks. For example: JNI_OnLoad is probably the main bridge for android apps.
Adctl asynchronous: It is really complicates to have good threading between Java-OBJc/cpp/qml but sometimes things hangs when loading and requesting. I have no idea how to achieve this at the moment.

In general i think the Admob and Adctl should merge into one project, already the project is way to complex for the size of code managed in it.
For new integration

A base interface is needed for Java imp and Objc. So we could just Plug things in from Java and C.
Find more people to integrate things, in gplus there are two big comminities where people are always posting in. If we create a simple interface for integration, those guys might just feel like contributing.

We could use those sources and integrade them.
https://github.com/googleads/googleads-mobile-unity

@kafeg I just did something somewhat crazy. I wrapped adctl admob around http://www.sdkbox.com/plugins/admob , which gives us alot more functionality.

What about the new Firebase C++ SDK?
https://github.com/firebase/quickstart-cpp

Hi! Two month ago i will start develop new version of AdCtl. It will be
smaller and implements only G-Services API.
But, after develop proof of-concept, i had to switch to other tasks (work
=) ).
New library use CocoaPods under iOS and Gradle under Android.
Archive with code is in attachment, if you want to see it.

qtgpgs_adctl_replacement.zip
https://drive.google.com/file/d/0B1uxKzS9K_04SFgwMjZCYjZqaTA/view?usp=drive_web

2016-12-05 22:36 GMT+03:00 Larpon<EMAIL_ADDRESS>
What about the new Firebase C++ SDK?
https://github.com/firebase/quickstart-cpp
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kafeg/adctl/issues/14#issuecomment-264953617, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ACOcV9SsMbMVmhBZGx0JtjfWQhfSgthdks5rFGfPgaJpZM4IFF1o
.

--
С уважением, Виталий Петров
FullStack разработчик на Qt/C++, Yii2 и MeteorJS
Best regards, Vitaly Petrov
FullStack developer: Qt/C++, Yii2 and MeteorJS
http://github.com/kafeg | http://bitbucket.org/kafeg | http://skid.kz |
http://kellot.ru

Awesome. Is it done - or does it still need work?

It is done only for GAnalytics and Google Sign In (iOS/Android). All other
modules need to work.
2016-12-06 12:10 GMT+03:00 Larpon<EMAIL_ADDRESS>
Awesome. Is it done - or does it still need work?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kafeg/adctl/issues/14#issuecomment-265098232, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ACOcV-Xw9gJpVvwIIndWyNt5a4T_x719ks5rFSaQgaJpZM4IFF1o
.

--
С уважением, Виталий Петров
FullStack разработчик на Qt/C++, Yii2 и MeteorJS
Best regards, Vitaly Petrov
FullStack developer: Qt/C++, Yii2 and MeteorJS
http://github.com/kafeg | http://bitbucket.org/kafeg | http://skid.kz |
http://kellot.ru

Nice work - I can't get it to compile though:
/home/user/Environments/Qt/OpenSource/5.7/android_armv7/src/android/java/src/org/qtproject/qt5/android/bindings/QtActivity.java:41: error: cannot find symbol
import android.app.Fragment;
                  ^
  symbol:   class Fragment
  location: package android.app

First you need to install all dependencies for Gradle. As i say, at now it's only concept, without readme and build instructions)

@kafeg FYI I've started a Qt QML/C++ Firebase implementation here:
https://github.com/Larpon/QtFirebase
Your idea of using CocoaPods has helped me a lot - thanks!
I need more time to document it though - but I have fully working Google Analytics and AdMob QML items working. In contrary to your code I'm planing to keep it Firebase only (so I won't have to touch any Objective-C or Java JNI). The Firebase C++ SDK has proven to be very picky about it's asynchronous code paths so I've tried to safe-guard crashes in my Qt wrapping - it works pretty well on my test devices. Contributions are welcome of course

@Larpon that's great news. I just wanted to let you know we have a Slack chat for mobile development with Qt, you might be interested in joining it, there might be someone interested in joining your effort: http://slackin.qtmob.org

@marco-piccolino thanks for letting me know!
I'm completely new to slack - where do I start? Do I need a team invite or something?


Run in docker: ERROR/locust.runners: RPCError found when receiving from client: ZMQ network broken

Describe the bug

it cann not work when running in docker.
[+] Running 3/0
 ⠿ Container myexample-master-3  Created                                                                                0.0s
 ⠿ Container myexample-worker-2  Created                                                                                0.0s
 ⠿ Container myexample-worker-3  Created                                                                                0.0s
Attaching to myexample-master-3, myexample-worker-2, myexample-worker-3
myexample-master-3  | [2022-03-20 04:43:43,888] 374ef93331c9/INFO/locust.main: Starting web interface at http://<IP_ADDRESS>:8089 (accepting connections from all network interfaces)
myexample-worker-3  | 0
myexample-worker-3  | I'm on a worker or standalone node
myexample-worker-2  | 0
myexample-worker-2  | I'm on a worker or standalone node
myexample-worker-3  | [2022-03-20 04:43:43,890] 55e0acab805b/INFO/locust.main: Starting Locust 2.8.4
myexample-worker-2  | [2022-03-20 04:43:43,890] 37541a5124ca/INFO/locust.main: Starting Locust 2.8.4
myexample-master-3  | 0
myexample-master-3  | I'm on master node
myexample-master-3  | [2022-03-20 04:43:43,901] 374ef93331c9/INFO/locust.main: Starting Locust 2.8.4
myexample-master-3  | [2022-03-20 04:43:51,069] 374ef93331c9/WARNING/locust.runners: You can't start a distributed test before at least one worker processes has connected
myexample-worker-2  | [2022-03-20 04:56:13,710] 37541a5124ca/INFO/locust.util.exception_handler: Exception found on retry 1: -- retry after 1s
myexample-worker-3  | [2022-03-20 04:56:13,710] 55e0acab805b/INFO/locust.util.exception_handler: Exception found on retry 1: -- retry after 1s
myexample-worker-3  | [2022-03-20 04:56:13,710] 55e0acab805b/ERROR/locust.util.exception_handler: ZMQ sent failure
myexample-worker-3  | Traceback (most recent call last):
myexample-worker-3  |   File "/opt/venv/lib/python3.9/site-packages/locust/rpc/zmqrpc.py", line 20, in send
myexample-worker-3  |     self.socket.send(msg.serialize(), zmq.NOBLOCK)
myexample-worker-3  |   File "/opt/venv/lib/python3.9/site-packages/zmq/green/core.py", line 216, in send
myexample-worker-3  |     msg = super(_Socket, self).send(data, flags, copy, track, **kwargs)
myexample-worker-3  |   File "/opt/venv/lib/python3.9/site-packages/zmq/sugar/socket.py", line 547, in send
myexample-worker-3  |     return super(Socket, self).send(data, flags=flags, copy=copy, track=track)
myexample-worker-3  |   File "zmq/backend/cython/socket.pyx", line 718, in zmq.backend.cython.socket.Socket.send
myexample-worker-3  |   File "zmq/backend/cython/socket.pyx", line 765, in zmq.backend.cython.socket.Socket.send
myexample-worker-3  |   File "zmq/backend/cython/socket.pyx", line 247, in zmq.backend.cython.socket._send_copy
myexample-worker-3  |   File "zmq/backend/cython/socket.pyx", line 242, in zmq.backend.cython.socket._send_copy
myexample-worker-3  |   File "zmq/backend/cython/checkrc.pxd", line 22, in zmq.backend.cython.checkrc._check_rc
myexample-worker-3  | zmq.error.Again: Resource temporarily unavailable
myexample-worker-3  |

Environment

OS:
Python version:python3.8
Locust version: 2.8.4
Locust command line that you ran: docker-compose -f run_by_locust.yaml up --scale worker=2
Locust file contents (anonymized if necessary):
docker-compose.yml

version: '3'

services:
  master:
    image: locustio/locust:latest
    ports:
      - 8089:8089
    volumes:
      - ./:/myexample
    command: -f /myexample/locustfile.py WebsiteUser --master

  worker:
    image: locustio/locust:latest
    links:
      - master
    volumes:
      - ./:/myexample
    command: -f /myexample/locustfile.py WebsiteUser --worker --master-host=<IP_ADDRESS>

No idea whats going on but is it possible you had some old containers that are interfering? It is suspicious that the master would be named master-3

@cyberw it cannot work. do you give me advise?

Sorry, I have no way to know what is wrong, you need to do more digging (currently running docker containers investigate if a process is bound to that port etc)


ConductorServiceTimeoutException

we have three servers creating publication to one subscription server
On the publisher we seeing below logs
2017-07-12 21:38:58,993  INFO aeron-client-conductor  Image Unavailable:io.aeron.Image@26d4fb20 session -1939625920
2017-07-12 21:38:58,993  INFO aeron-client-conductor  Image Unavailable :io.aeron.Image@39b5cc72 session<PHONE_NUMBER>
2017-07-12 21:38:58,993  INFO aeron-client-conductor new  Image Unavailable :io.aeron.Image@659e0d43 session -1535749004
io.aeron.exceptions.ConductorServiceTimeoutException: Timeout between service calls over<PHONE_NUMBER>ns
at io.aeron.ClientConductor.onCheckTimeouts(ClientConductor.java:508)
at io.aeron.ClientConductor.doWork(ClientConductor.java:431)
at io.aeron.ClientConductor.doWork(ClientConductor.java:143)
at org.agrona.concurrent.AgentRunner.run(AgentRunner.java:140)
at java.lang.Thread.run(Thread.java:745)
On the subscriber, we see
subscriber 1
2017-07-12 21:39:13,483 ERROR aeron-client-conductor Image Unavailable :1901581559
subscriber 2
2017-07-12 21:39:14,274 ERROR aeron-client-conductor Image Unavailable :1901581558
subscriber 3
2017-07-12 21:39:13,779 ERROR aeron-client-conductor Image Unavailable :1901581561
Once we get the image unavailable, we clear and close the publication. then we try to create new publication, but we still getting same publication from the api "Aeron.addPublication" which is closed.
Could you help us how to get new publication.

An interservice timeout indicates very long GC pauses or resource starvation on the machine which is not a situation to be in for low-latency applications.
Can you provide more information on what Aeron version, JDK, and OS you are using?
When this happens it is useful to run AeronStat to get a view of the counters on each machine?

After a ConductorServiceTimeoutException you will need to close the Aeron client and reconnect.

Thanks Martin, actually we doing functional testing and got into this issue .. seems our functional testing servers are slow.
We using aeron 1.3, JDK 8, and CentOS release 6.6
Thanks for the input, we implementing reconnect mechanism too.


TLS client cert conflicts with ACME

When configuring client cert with PR https://github.com/containous/traefik/pull/461, ACME does not work anymore. Redacted error message:
server.go:2317: http: TLS handshake error from <IP_ADDRESS>:49733: tls: client didn't provide a certificate
level=error msg="map[...:acme: Error 400 - urn:acme:error:tls - Failed to connect to ...:443 for TLS-SNI-01 challenge...]"

I haven't looked into the ACME implementation. Maybe it's possible to add a special case that disables client cert for ACME challenges.

Closed in profit of #942  - Feel free to subscribe there for updates.


Items of Black Hole Unis disappearing in inventory

Industrial Foregoing Version: 1.6.6
TeslaCoreLib Version: 1.12-1.0.12
Crashlog (If Any): No Crashlog
There is a bug with the Black Hole Units, when you Shift-click to fast in Black Hole Units to put items instant in your inventory. You can add 4-5 stacks of items without problems, but when you add more the items are very buggy. You can see them in your inventory, but when you close the GUI and open another inventory, most of the stacks are disappearing and again in the Unit. They don't get lost, because when you look into the Unit the disappeared stacks are in it again, but it's very annoying if you can only take a few stacks with you.
I hope you can understand the problem :)

That's a lag issue, I can't do much, but I will try to make something about it


Assertion error on line 5166

I'm getting this assertion error: i < m_dim[0] && i >= 0 && j < m_dim[1] && j >= 0 && k < m_dim[2] && k >= 0
Any information on the cause of this would be appreciated.

That would be a bug. Can you please give me a repro case? Meaning a Wavefront OBJ file and the exact parameters you used?

My mistake, I was using mesh data from a GLTF file, which I falsely assumed would have compatible values. Upon comparing the data from the two file types I realized the problem (here is sample data from a cube):

OBJ Verts
GLTF Verts
OBJ Index
GLTF Index

-100
-1
1
3

-100
0
2
6

100
-0
0
0

-100
-100
3
10

100
-100
6
19

100
100
2
8

-100
0.375
7
23

-100
1
4
14

-100
0
6
20

-100
-1
5
15

100
-0
0
2

-100
-100
4
13

100
-100
6
18

-100
100
0
1

100
0.125
2
7

100
0.25
3
11

100
0
5
16

100
0
7
22

100
1
1
3

-100
-100
3
9

-100
-100
2
6

100
100
3
10

100
0.375
7
21

-100
0
6
19

7
23

5
17

4
14

5
15

1
4

0
2

6
18

4
12

0
1

3
11

1
5

5
16


Fix Fontawesome

What this PR does / why we need it:
Enable full range of Fontawesome icons.
Which issue this PR fixes:
fixes #159
Special notes for your reviewer:
Builds on #174
Release note:

Fontawesome icons now need to declare the full icon class: `fab fa-facebook` instead of `fa-facebook`

@mpourismaiel merge conflict prevention. We would have run into one with that change. So I based it on #174.


Could you please share fmri_net_sim for fmri netsim dataloading

Dear @ElleZWQ
Thank you sharing code online, could you please share the "fmri_net_sim.py" to load the fmri data?
I also have another question, can this code be able to run without groundtruth?

Thank you for your reminder. I have uploaded the file.
Of course, you can run this code without ground truth by deleting or
Comment the related ground truth part.
Our appendix also talks about this.


⚠ Bofrost.at has degraded performance

In 1076612, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2105 ms

Resolved: Bofrost.at performance has improved in 1b172b3.


Features/add bootstrap

👷 Deploy Preview for benevolent-sprinkles-4567d6 processing.

Name
Link

🔨 Latest commit
aeffc714bfe3316c31c45d7faa82111b20b2c9df

🔍 Latest deploy log
https://app.netlify.com/sites/benevolent-sprinkles-4567d6/deploys/65ba9341d3422d0008af5b07


Lightly refactor main pubsub handler

Part of #482, after #512.
I'm pretty sure what I want to do is loop over multiple orgs in this handler (getsentry, codecov). This sets me up for that.
~This also switches to requiring that repos be in payload. Can we do that? 🤔~ Decided against this for now, too controversial.

This also switches to requiring that repos be in payload. Can we do that? 🤔 Decided against this for now, too controversial.

What do you mean here? Isn't repos already in the payload?

Do you mean hard requiring repos to be in the payload? And raising if it isn't?

What do you mean here? Isn't repos already in the payload?

It's optional, though ...
https://github.com/getsentry/eng-pipes/blob/78b702140e21b3ed551a301179eafd765924fc40/src/webhooks/pubsub/index.ts#L17
... and we fall back to a default.
https://github.com/getsentry/eng-pipes/blob/78b702140e21b3ed551a301179eafd765924fc40/src/webhooks/pubsub/index.ts#L65
Where is the payload defined? In GCP somewhere?

that's defined in the ops repo already so we have full control over that. https://github.com/getsentry/ops/blob/master/terraform/super-big-data/super-big-consumers/schedule.tf

The missing link! Thank you! 😁 🙏


Documentation improvement: improve main README VDK images - gif

This PR is part of the issue https://github.com/vmware/versatile-data-kit/issues/1533
The goal is to create a gif to preview vdk CLI usage.

Consider a bullet list the commands...
A preview of the VDK CLI commands:

vdk create
vdk run
vdk deploy

I still want to make the black bar nicer, maybe gray too, or idk yet
But I'd like to hear from you @tozka if it's the right direction and if you like it there. Because the commands are actually pretty self-explanatory and with the list of what I'm doing it might not be necessary.

I think this GIF would be great to include in the tutorial on the VDK Wiki, especially since it demonstrates the VDK commands clearly. For example, we could replace "my-job" with "Hello World" as shown in the tutorial https://github.com/vmware/versatile-data-kit/wiki/getting-started.
However, for a first quick overview of the CLI capabilities, it may not be sufficient. To make it more effective, it should be tailored towards a specific problem or use case. And if data or analytics engineers have similar use case, this would give users the impression that the VDK CLI can help them solve their challenges.
For instance, Let's say we have a scenario where a data engineering team needs to process and transform large amounts of customer data for a retail company. The team needs to extract data from multiple sources, transform it into a unified format, and load it into a data warehouse for further analysis. That's a common scenario in the data engineering world
The first VDK CLI command that appears is vdk create --name retail-data --team data-engineering.
Next, the GIF shows an animation of the VDK CLI command vdk run retail-data, which starts the data job. The GIF shows the status of each step in the job as it progresses, such as extracting data from each source, performing transformations, and loading the data into the warehouse. So the steps should be named appropriately.
After the job completes successfully, the GIF shows an animation of the VDK CLI command vdk deploy --name retail-data-pipeline --team data-engineering, which deploys the pipeline to the cloud. The GIF shows the pipeline being versioned and uploaded to the cloud, the job being deployed and ready for use.


RSpec helpers

I've been thinking about how the RSpec helpers would look. I think it would be wise setting an explicit expectation of what the #refine method returns.
RSpec.describe ArticlesController do
  it "refines article parameters" do
    get :create, { title: "Hello", body: "World", published: true, tags: ["blog"] }
    expect(subject).to refine(:article).as({ title: "Hello", body: "World", tags: ["blog"] })
  end
end

class ArticleRefinery < Struct.new(:object,:user)
  def create
    [:title, :content, tags: []]
  end
end

class ArticlesController < ActionController::Base
  include Arcane
  def create
    @article = Article.create params.for(Article).refine
  end
end

Thoughts @ngw?

Seems fine to me.


Add a bailout path to protocol requirement source formation

The GSB will try to form and note invalid constraints, but there are
a few paths that aren't prepared for error types to pop up.  Add
a defensive check to formProtocolRelativeType to make sure we don't wind
up force-casting an error type.
Fixes rdar://56116278

@swift-ci please smoke test and merge


Gemspec ignore test files for reduce gem size from 7.5MB to 7.5KB

https://rubygems.org/gems/marcel
For reduce ActiveStorage size
-- -rw-r--r--   1 jason  staff   7.5M  9 15 10:34 marcel-0.3.2.gem
++ -rw-r--r--   1 jason  staff   7.5K  9 15 10:45 marcel-0.3.2.gem

Wow, thank you!

ActiveStorage has dependency this gem by using marcel ~> 0.3.1, so maybe if you can release a new version 0.3.3, the issue can fix now.

I just published version 0.3.3. :heart:


Mutually referenced generic type constraints freeze compiler

TypeScript Version: 2.1.4
Code
function prop<K extends string>(name: K): <T extends { [P in K]: U }, U extends T[K]>(obj: T) => U {
    return (obj) => obj[name];
}

let x = prop("name");
x({ name: 10 });

Expected behavior:
Running tsc test.ts should result in successful transpilation or compiler errors.
Actual behavior:
tsc never exits.

Should be fixed in typescript@next


Update TVM section

The content of section Microsoft Defender for Endpoint also includes Threat & Vulnerability Management (TVM). How is this Azure Defender vulnerability assessment extension different? might be obsolete. TVM now supports Linux too. References:
https://www.microsoft.com/security/blog/2021/05/11/threat-and-vulnerability-management-now-supports-all-major-platforms/
https://docs.microsoft.com/en-us/microsoft-365/security/defender-endpoint/tvm-supported-os?view=o365-worldwide

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

ID: 4fdbbf80-92cd-d6b2-85fd-a830589a8ed9
Version Independent ID: d89d9eef-daed-636d-7414-0b15f3195734
Content: Security Center's integrated vulnerability assessment solution for Azure and hybrid machines
Content Source: articles/security-center/deploy-vulnerability-assessment-vm.md
Service: security-center
GitHub Login: @memildin
Microsoft Alias: memildin

@smedin4 Thanks for the feedback. We will engage the content development team for further investigation, they will verify and perform necessary changes in the document as needed.

Hi @smedin4, thanks for the feedback and for taking the time to help us improve the docs.
As the FAQ entry makes clear, Linux support is only one of the differentiators. The Qualys-powered vulnerability assessment extension remains a valuable tool and a widely adopted feature from Azure Defender.
I'm going to close this issue, but feel free to comment again here or open a new one if you have additional requests.
#please-close


Rework Attach to work on arbitrary Analog hierarchies

Now you can just attach Analogs without any concept of "source". It's just representing an electrical connection.

Unfortunately it appears that alias doesn't work in design compiler. Looking into alternatives....

Now with improvements!

Verified that generated code is synthesizable (test case lives in Chisel, will push to that branch soon)
Found that Verilator really doesn't support the general case

emit `ifdef verilator `error in those cases since the generated Verilog is still synthesizable and works with alias
Will bump Verilator Issue so that this restriction can be lifted

Added code to infer zero or single source attaches so that simpler Verilog can be emitted

These cases are supported by Verilator (so no  `error emitted)
This should work for the majority of use cases (notable excluding a module that tries to attach two inout ports)
Useful improvement would be to dead code eliminate / constant propagate extra wires on an attach so that they don't prevent single source inference

@azidar I believe I've addressed all of your concerns. I reworked how ExpandWhens combines the attaches because it wasn't preserving order like I had intended. There are now tests to illustrate that the order is preserved in an intuitive way.


[FEATURE] Cron sidecar

I am having trouble adding cron capabilities to the DockerFile
I have added the lines:
RUN apt-get install -y cron
COPY cron.d /etc/cron.d
RUN cat /etc/cron.d/* | crontab -
RUN touch /var/log/cron.log

When I rebuild the container, start it with the docker-compose and exec into the container I can see the following:

Cron is installed (present /etc/init.d and /usr/sbin)
Executing crontab -l shows the expected crontab
Executing service --status-all shows that the cron service is stopped.

I cannot find any logging that indicates why cron would have failed or not be running.
I know this isn't exactly a problem with the current Docker Compose package but I have been trying to get this work without success for a while now, and I'm hoping you can spot something obvious I'm doing wrong.
Any assistance is greatly appreciated.

Hi,
you need to start an additional Docker container, since in each container should only run 1 PID. More information on this can be found here. Inside the cron container, you need the exact same php version as used in the webserver. Afterwards you should mount the document root of the webserver. Furthermore it would be a good thing wenn you output the cron log to a seperate logfile with simple output redirection or giving the cron daemon this option in startup.
If someone else is interested in adding a cron sidecar let me know. If there is enough interest I could add this to the stack.

That would be great

That would be great


Small typo of NodeJS version in README.md

Can one of the admins verify this patch?

Can one of the admins verify this patch?


Update UG

fixes #477
Added explanation for what an assignment is.

Codecov Report

Merging #478 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #478   +/-   ##
=========================================
  Coverage     19.10%   19.10%
  Complexity      360      360
=========================================
  Files           220      220
  Lines          4193     4193
  Branches        505      505
=========================================
  Hits            801      801
  Misses         3317     3317
  Partials         75       75

Continue to review full report at Codecov.

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


About pydw

I follow your note, but there are some trouble with me.
1 I can't install pydtw, it It prompts such errors

2 the link of "dtw" file is removed, can I find it somewhere else?

@Sample-design-alt Hey. Could you provide more details? For example, what operation system you are using? And what the error you are facing? pydtw has C code, so I guess the problem could be here.
What link is removed?

I use Ubantu system, the error as follows:
ERROR: Command errored out with exit status 1:
command: /usr/local/bin/python -c 'import io, os, sys, setuptools, tokenize; sys.argv[0] = '"'"'/data/chenrj/SimTSC-main/pydtw/setup.py'"'"'; file='"'"'/data/chenrj/SimTSC-main/pydtw/setup.py'"'"';f = getattr(tokenize, '"'"'open'"'"', open)(file) if os.path.exists(file) else io.StringIO('"'"'from setuptools import setup; setup()'"'"');code = f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, file, '"'"'exec'"'"'))' develop --no-deps
cwd: /data/chenrj/SimTSC-main/pydtw/
Complete output (565 lines):
running develop
running egg_info
creating dtw.egg-info
writing dtw.egg-info/PKG-INFO
writing dependency_links to dtw.egg-info/dependency_links.txt
writing requirements to dtw.egg-info/requires.txt
writing top-level names to dtw.egg-info/top_level.txt
writing manifest file 'dtw.egg-info/SOURCES.txt'
reading manifest file 'dtw.egg-info/SOURCES.txt'
writing manifest file 'dtw.egg-info/SOURCES.txt'
running build_ext
building 'dtw' extension
creating build
creating build/temp.linux-x86_64-3.9
creating build/temp.linux-x86_64-3.9/src
gcc -pthread -B /usr/local/compiler_compat -Wl,--sysroot=/ -Wno-unused-result -Wsign-compare -DNDEBUG -O2 -Wall -fPIC -O2 -isystem /usr/local/include -fPIC -O2 -isystem /usr/local/include -fPIC -I/usr/local/lib/python3.9/site-packages/numpy/core/include -I/usr/local/include/python3.9 -c src/deque.c -o build/temp.linux-x86_64-3.9/src/deque.o -O2 -Wall -fPIC -pedantic -Wextra
gcc -pthread -B /usr/local/compiler_compat -Wl,--sysroot=/ -Wno-unused-result -Wsign-compare -DNDEBUG -O2 -Wall -fPIC -O2 -isystem /usr/local/include -fPIC -O2 -isystem /usr/local/include -fPIC -I/usr/local/lib/python3.9/site-packages/numpy/core/include -I/usr/local/include/python3.9 -c src/dtw.c -o build/temp.linux-x86_64-3.9/src/dtw.o -O2 -Wall -fPIC -pedantic -Wextra
Are some packages missing from me?
2 The link of multivariate dtw is available now. There was some problem with my net before.

According to your notes, I encountered some troubles: After downloading pydtw, the process of executing pip install -e . displays the following error:

The system used is Windows 10; IDE is pycharm. Looking forward for your reply, thank you!

@mingshanz I didn't test on Windows. Could you try on a Linux machine?

@mingshanz我没有在Windows上进行测试。你能在Linux机器上试穿吗？

ok, im trying. thx

When I run pip install -e under Ubuntu's linux, this problem occurs. I have searched many places and have not been able to solve it. Does the author have a solution?

@mingshanz May I know which operation system you are using?

@mingshanz May I know which operation system you are using?

ubuntu20.04 ,At first, I suspected that GCC was not installed, but through inspection, I installed GCC. THX

@mingshanz Will removing -Wextra work?

its the same error, im trying 2 weeks, not found the solution,

@mingshanz Sorry I am not sure what the problem could be since I can not reproduce your errors. Have you tried directly building the UCR Suite https://www.cs.ucr.edu/~eamonn/UCRsuite.html

@mingshanz Maybe the GCC version could cause this problem? I used the following GCC and it seems to work well.
gcc version 7.5.0 (Ubuntu 7.5.0-3ubuntu1~18.04)

@mingshanz Maybe the GCC version could cause this problem? I used the following GCC and it seems to work well.
gcc version 7.5.0 (Ubuntu 7.5.0-3ubuntu1~18.04)

okay, im trying this.  thx

@mingshanz Sorry I am not sure what the problem could be since I can not reproduce your errors. Have you tried directly building the UCR Suite https://www.cs.ucr.edu/~eamonn/UCRsuite.html

im not sure it works. But, i will try this way. thx

@mingshanz Sorry I am not sure what the problem could be since I can not reproduce your errors. Have you tried directly building the UCR Suite https://www.cs.ucr.edu/~eamonn/UCRsuite.html

Unfortunately, I only came into contact with this direction, so I won't build UCR suite directly.

Excuse me, after pip3 install -e . dtw, the result shown in the figure below appears, and then python3 creat_dtw.py has such a situation. Does it mean that pip3 install -e . was unsuccessful?

@daochenzha It is unsuccessful. There could be something wrong when installing the extension.

Excuse me, windows just can't install pydw, only linux can be used

@cookerXIN Did you make it with Linux?

@cookerXIN Did you make it with Linux?

I use Windows to do it. May I ask you one more question? Is this part of the index directly read from the dataset all together?

@cookerXIN Did you make it with Linux?

我想请问下，pip  install -e . 这个语句主要是安装什么包呀，有没有这个包的github链接，我想直接下载到本地安装。因为pip对我一点用都没有，谢谢！


Watch flag should indicate when a resource is deleted

Feature suggestion/improvement:
kubectl get pods --watch should indicate when a pod is actually deleted.
Currently it just appends another line or two indicating that it has a terminating status, around when it gets deleted.
This might apply to other resource types too.
Kubernetes version (use kubectl version):
Client: 1.8.4
Server: 1.8.0 (via minikube v0.24.1)
Environment:
Minikube v0.24.1
Host Env:
Ubuntu 17.10
Linux Laser 4.13.0-16-generic #19-Ubuntu SMP Wed Oct 11 18:35:14 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux
What happened:
$ kubectl get pods --watch
NAME                               READY     STATUS        RESTARTS   AGE
my-pod-5c7dd74bcb-2xj54   0/1       Terminating   6          11m
my-pod-7bcddb9489-j4qzz   1/1       Running       0          51s
my-pod-5c7dd74bcb-2xj54   0/1       Terminating   6         11m
my-pod-5c7dd74bcb-2xj54   0/1       Terminating   6         11m

What you expected to happen:
There should be some sort of 'Terminated' or 'Deleted' status.
How to reproduce it (as minimally and precisely as possible):

run kubectl get pods --watch while deleting pods.

Anything else we need to know:

ran via bash shell
I created/deleted pods via helm, but that shouldn't matter

This might also just be an issue of the lines not getting cleared properly in the bash shell, when a row is supposed to go away.

I'd like to see this feature. It'd help with rapid deletion/creation of the same pod.
/remove-lifecycle rotten

/remove-lifecycle stale

/sig cli
/area kubectl

/priority P2

/remove-lifecycle stale

/remove-lifecycle stale

/remove-lifecycle stale


H2ogpt openai endpoints

Hi, can someone explain how I can test these endpoints? I keep getting the same error as shown in the screenshot. If I need to get a key, how can I obtain it?

OpenAI endpoints are on port 5000 by default.

Thank you for responding. Do I need an API key to test the endpoints, or can I just set the api_key to empty

EMPTY is default.   You can set the key, like I showed in the FAQ for how to setup gpt.h2o.ai
https://github.com/h2oai/h2ogpt/blob/0e1597644dfee5a2b368d753d84086d966647bd4/docs/FAQ.md#L408

Thank you, I can now test H2O GPT on Open Web UI. My question is: will it use the documents I have already uploaded to H2O GPT, or will it follow the RAG process and parameters of Open Web UI? Does it interact with the files in my vector database?

Thank you, I can now test H2O GPT on Open Web UI. My question is: will it use the documents I have already uploaded to H2O GPT, or will it follow the RAG process and parameters of Open Web UI? Does it interact with the files in my vector database?

The database is separate.  On the backend operations would use h2oGPT except for the database.


tf: cleanup docker deadcode

Please provide a description for what this PR is for.
And to help us figure out who should review this PR, please
put an X in all the areas that this PR affects.
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[ ] Does not have any changes that may affect Istio users.

@howardjohn: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-pilot-k8s-tests_istio
88daf4ad80fc1527f77b4ba0868a1687b90c4b2d
link
/test integ-pilot-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest


Path should include dollar sign (as in $HOME)

If I use environment variables in paths, such as in \addbibresource{$HOME/ref.bib}, then it is parsed as an error, as if the dollar sign were an opening delimiter for math expressions, even though it compiles fine with any LaTeX engine.

I encountered a similar issue with strings starting with $. The following code has wrong highlighting in the rm -rf line,
To clear \lstinline{biber} cache, run
\begin{lstlisting}[language=bash]
rm -rf "$(biber --cache)"
\end{lstlisting}

The rm -rf line is not LaTeX and therefore not relevant to the highlighter (this should be handled on the tree-sitter level by injecting a bash parser).

Thanks for the report. https://github.com/latex-lsp/tree-sitter-latex/commit/fbbf4c4e90604dee1f9e349686b24f7248832121 should fix the issue with dollar and other characters inside paths. The highlighting queries over at nvim-treesitter will need a small update though.

Nor sure, actually; should test first :) ( @theHamsta )

We have a CI jobs that updates the parsers regularly. When no nodes are renamed, removed, added no manual action is required from nvim-treesitter side.
I would recommend to add the example above to corpus.txt to test it. So just test it locally with tree-sitter test and if the parsed AST is correct create a PR to add the test case to this repo.

Yeah, I don't think we use the (word) capture for the argument of \input{}, \bibliography{}, etc., so there shouldn't be a breakage.

Sorry for the long wait. There are now test cases for both examples. The source code listing is handled using an external scanner, which outputs a (source_code) node.