[JOSS Review] Typos in CONTRIBUTING.md

Re joss-reviews/issues/5968
Hi @gbouras13,
There are a couple of places in the CONTRIBUTING.md file which reference one of your other projects (phrokka) rather than dnaapler. Could you please take a look and make sure that the contribution guidelines are appropriate for this repo? It might also be nice to link them in the main README.md.

Thanks @mkerin ,
This has been addressed in #54 and #55. I will also included a link to the README.md in a later PR.

Thank you for taking a look at this - I agree that the issue is resolved. Closing now.


Let themes change radialMenu text colors

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)

Proposed changes

As described in https://github.com/node-red-contrib-themes/theme-collection/issues/239, the text in the radial menu becomes unreadable when using themes. This PR fixes that.

Now
	Then

Default theme

Dark theme

Checklist

[x] I have read the contribution guidelines
[ ] For non-bugfix PRs, I have discussed this change on the forum/slack team.
[ ] I have run grunt to verify the unit tests pass
[ ] I have added suitable unit tests to cover the new/changed functionality

Coverage: 68.261%. Remained the same when pulling f7371626973bbaf4cb84713179ff0b4248020fba on bonanitech:radialMenu into b3ce0c007907b93aeb52e13631597ff258719254 on node-red:master.


Wasm packages

Linux package builds failing due to:
https://github.com/InsightSoftwareConsortium/ITKPythonPackage/issues/263


Please create a new release

Your latest release is from 2017 and has bugs which have since been fixed (e.g. #100). Can you provide a new release please?

would love for this to happen, impossible to install via homebrew.

Yes please create a new binary release. You added the admin sub-command in da7df58a9cec1a57067d4cb0d8c9b87665a1bd53 but this is not available in the latest released binary.

In the meantime i've forked this and new v12.1.1 binaries can be founed here

https://github.com/birdayz/kaf is a good alternative


[code gardening] minor memoization fix in spec; trim trailing whitespace...

...s in code

Coverage remained the same at 98.13% when pulling 4a13fc3b4f4a0f9bb16f0f399ddb3e1236ad37aa on ipoval:code-gardening into 3483a9640ce5fd7dbc2aa2f1506d97df2a0ba01a on andrew:master.

Thanks @ipoval


Live reload

How should you set up live code reloading?

Hi @Immortalin !
What do you mean? You can run npm run dev which use nodemon to reload the server whenever you made a modification.
If you though about frontend reload you should use something like Browersync but this isn't in the scope of Adonis.

@romainsync it would be great if client side reload can be integrated into Adonis. Frameworks such as phoenix already do this

What do you mean by client side reload? Forcing the client (browser) to reload when you modify your code?

Closing as not directly related to AdonisJs. Frontend code reloading is outside the scope.


DRILL-5845: Columns returned by select with "ORDER BY" and "LIMIT" cl…

…auses are not in correct order.

+1, LGTM.


a11y - sharing page - Save/cancel buttons - cms ui

Describe the bug

When navigating using the keyboard, the 'Save' button appears before the 'Cancel' button. This is not an expected behavior.

To Reproduce

Run through the page using the keyboard

Expected behavior
The cancel button need to appear first and after that the save one.
Screenshots

Software (please complete the following information):
Volto Version 17.0.0-alpha.19
Plone Version 6.0.4

Can i start working on this bug?

@Ranjan-bej you need to sign the Contributor Agreement to contribute to Plone:
https://6.docs.plone.org/contributing/index.html

@Ranjan-bej see also https://6.docs.plone.org/contributing/first-time.html.

Is the issue still open I would like to work on it

@kanugurajesh please follow https://6.docs.plone.org/contributing/index.html and https://6.docs.plone.org/contributing/first-time.html

Hey can i work on this issue?

@abhinaisai2002 please read and follow Contributing to Plone and First-time contributors.

I have signed the Contributors document

Opened a PR #5191 resolving this issue

I'm excited to take on this issue as my first open-source contribution. Please consider assigning it to me. I'm eager to contribute and eager to learn!

@16achint note that @ZubairImtiaz3 has already started work on this issue, and we strongly encourage new contributors not to duplicate efforts. Additionally please read and follow Contributing to Plone and First-time contributors.

Thank you for the guidance! I appreciate the response and have already learned a valuable lesson about open-source collaboration. Thanks, @stevepiercy!


FactorFile and MapFile question

Hi guys, I'm implementing my own layer to read data not from file but from a database, could you please help me understand what FactorFile and MapFile are about? I see they're used everywhere in SubscriptionDataReader regardless of data source / data type i.e even if it's crypto exchange these are used. Tho they're stored in equity folder. So I struggle to understand will these values be even used if I use crypto type of data?

Right now I have FactorFileRow implemented as a candle for binance API ... I'm not sure if that's correct... Could you please just explain what this model is about?

I also struggle to understand MapFiles.. What is permtick? What are these mapped symbols about?

Right now I have MapFiles implemented as market symbols (e.g BTCUSDT, ETH_BTC etc) and the current date

Hey there, great to see you digging into LEAN! Please post user-group style questions to the LEAN forum on QuantConnect
https://www.quantconnect.com/forum/discussions/1/lean


Fix CertificateStateful not reporting failure

Changelog category :

New Feature
Bug Fix
Improvement
Performance Improvement
Backward Incompatible Change
Build/Testing/Packaging Improvement
Documentation (changelog entry is not required)
Other
Not for changelog (changelog entry is not required)

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
...
Documentation entry for user-facing changes

[ ] Documentation is written (mandatory for new features)

@hustnn PR is ready for review. Summary of changes:

Fix results.sh to report test case failure for each suit (if any)
Disable CertificateStateful failing test cases in multi-worker setup
Remove concurrency for CertificateStateful test suite
Add retry for Timeout cases and fix AttributeError when CREATE DATABASE for test case fails

Disable CertificateStateful failing test cases in multi-worker setup

@vivek-debug Please create an issue to track the disabled cases. Thanks.


Fix: avoid duplicate loop key in err handler search

Very minimal change. The code is self-explanation. code in for c in [code, None] may be None. Made a deduplication before do the loop.

Checklist:

[ ] Add tests that demonstrate the correct behavior of the change. Tests should fail without the change.
[ ] Add or update relevant docs, in the docs folder and in code.
[ ] Add an entry in CHANGES.rst summarizing the change and linking to the issue.
[ ] Add .. versionchanged:: entries in any relevant code docs.
[x] Run pre-commit hooks and fix any issues.
[x] Run pytest and tox, no tests failed.

Adjusted the code a little to avoid typing error, and rebased onto the HEAD of branch '2.0.x'.


French translation

Hi,
I've just translated this package's errors.php and validation.php files to French for a project. Would you be interested in a PR for this? I can see there was one for Dutch a few months ago, but just wanna make sure you'd actually be interested first.
Cheers,
Yannick

Yes happy to take a PR to add other languages!

@lindyhopchris cool, will do when I get a minute


Update approved-label.yml

This is my first contribution to the project. I’ve added the "pull_request: write" permission to the GitHub Actions workflow to ensure that the 'approved' label is correctly added to pull requests after review approval. It was a great learning experience, and I look forward to contributing more!

I'm sorry but a pull request already exists for this issue
#2732


"cascade_rcnn_r50_fpn_1x_coco_20200317-0b6a2fbf.pth is not a checkpoint file"

Thanks for your error report and we appreciate it a lot.
Checklist

I have searched related issues but cannot get the expected help.
The bug has not been fixed in the latest version.

Describe the bug
R-50-FPN | pytorch | 1x - checkpoint is still 1.0, when i use mmdet 2.0, i get the error
"cascade_rcnn_r50_fpn_1x_coco_20200317-0b6a2fbf.pth is not a checkpoint file"
the other model file works, just this didn't work.
and i view the log file https://open-mmlab.s3.ap-northeast-2.amazonaws.com/mmdetection/v2.0/cascade_rcnn/cascade_rcnn_r50_fpn_1x_coco/cascade_rcnn_r50_fpn_1x_coco_20200316_214748.log.json
it shows this model file is generated by mmdet 1.0
please upload the latest 2.0 model file, thanks alot

I can download cascade rcnn r50 fpn 1x coco directly by clicking the URL in README.md.
Could you have a try on this link https://open-mmlab.s3.ap-northeast-2.amazonaws.com/mmdetection/v2.0/cascade_rcnn/cascade_rcnn_r50_fpn_1x_coco/cascade_rcnn_r50_fpn_1x_coco_20200316-3dc56deb.pth again?
Some models are trained before mmdet v2.0 release. So the version may not be updated.

Same problem encountered when I am trying to load model from res2net101_v1d_26w_4s_mmdetv2-f0a600f9.pth provided by https://open-mmlab.s3.ap-northeast-2.amazonaws.com/pretrain/third_party/res2net101_v1d_26w_4s_mmdetv2-f0a600f9.pth. Actually, mmcv(version==0.5.5) raised KeyError: 'res2net101_v1d_26w_4s' if I directly set pretrained='open-mmlab://res2net101_v1d_26w_4s',


Allow specifying a webmixer class to use when creating ricecooker topic tree

Test run:
PYTHONPATH=. examples/gitta_chef.py -v --token=yourtoken --reset

See that zip files uploaded are much smaller (~200KB vs ~5MB).
View on Studio:
https://api.studio.learningequality.org/channels/3a9b468b3c5d544c90105ce47bcc5a09/edit/00fc279
Note that this requires this pending PR to be merged into Webmixer, and also this small change to get webmixer to work for local file:// files.

Okay, I got this running locally with the steps you listed, and everything looks good! 👍 Now I just need to finish up the webmixer side of things!

Awesome, thanks for testing this out locally!!


go get is not working

I think in the readme file the right command to run should be:
  go get github.com/twitchtv/twirp-example/...

It still downloads the package and puts it in GOPATH, so it still works despite the error message and non-zero error code. Still, your proposed change might be nicer to avoid confusing users.


[Event Request] table 99000854 Inventory Profile Offsetting - Reschedule "OnRescheduleGetActionMessage"

Can we please get the following event?
  local procedure Reschedule(var SupplyInvtProfile: Record "Inventory Profile"; TargetDate: Date; TargetTime: Time)
    begin
        SupplyInvtProfile.TestField("Planning Flexibility", SupplyInvtProfile."Planning Flexibility"::Unlimited);

        if (TargetDate <> SupplyInvtProfile."Due Date") and
           (SupplyInvtProfile."Action Message" <> SupplyInvtProfile."Action Message"::New)
        then begin
            if SupplyInvtProfile."Original Due Date" = 0D then
                SupplyInvtProfile."Original Due Date" := SupplyInvtProfile."Due Date";

            IsHandledL := false;
            OnRescheduleGetActionMessage(SupplyInvtProfile, IsHandledL); <-- NEW EVENT
            if not IsHandledL then begin
		if SupplyInvtProfile."Original Quantity" = 0 then
			SupplyInvtProfile."Action Message" := SupplyInvtProfile."Action Message"::Reschedule
		else
			SupplyInvtProfile."Action Message" := SupplyInvtProfile."Action Message"::"Resched. & Chg. Qty.";
	    end;
        end;
        SupplyInvtProfile."Due Date" := TargetDate;
        if (SupplyInvtProfile."Due Time" = 0T) or
           (SupplyInvtProfile."Due Time" > TargetTime)
        then
            SupplyInvtProfile."Due Time" := TargetTime;
        SupplyInvtProfile.Modify();
    end;

	[IntegrationEvent(false, false)]
    local procedure OnRescheduleGetActionMessage(SupplyInvtProfileP: Record "Inventory Profile"; var IsHandledP: Boolean)
    begin
    end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


fix(patches): apply Nginx patch for detecting HTTP/2 stream reset attacks early (CVE-2023-44487)

fix(patches): apply Nginx patch for detecting HTTP/2 stream reset attacks early (CVE-2023-44487)
From nginx/nginx@6ceef19

Summary

Patch was acquired from here:
https://mailman.nginx.org/pipermail/nginx-devel/2023-October/S36Q5HBXR7CAIMPLLPRSSSYR4PCMWILK.html
It was slightly modified to apply cleanly on our nginx-1.21.4 (just small offset differences),
See: https://www.cve.org/CVERecord?id=CVE-2023-44487
SIR-435
Checklist

[na] The Pull Request has tests
[x] A changelog file has been created under changelog/unreleased/kong or skip-changelog label added on PR if changelog is unnecessary. README.md
[na] There is a user-facing docs PR against https://github.com/Kong/docs.konghq.com - PUT DOCS PR HERE

@windmgc Will this be backported to 3.3.x also?

Will this be backported to 2.8.x?  Thanks


cnn_head_pose_estimation problems

Dear mpatacchiola & anyone reading this,
I am using your program in order to determine the head pose estimation. and im facing this error  while im  running the ex_cnn_head_pose_estimation:
my_head_pose_estimator.load_roll_variables(os.path.realpath("../../etc/tensorflow/head_pose/roll/cnn_cccdd_30k.tf"))
my_head_pose_estimator.load_pitch_variables(os.path.realpath("../../etc/tensorflow/head_pose/pitch/cnn_cccdd_30k.tf"))
my_head_pose_estimator.load_yaw_variables(os.path.realpath("../../etc/tensorflow/head_pose/yaw/cnn_cccdd_30k.tf"))
it give me this error
ValueError: [DEEPGAZE] CnnHeadPoseEstimator(load_roll_variables): the roll file path is incorrect.
can you help me to handle this error ?
NOTE: im using python Python 3.6.9  and tensorflow 1.14

This seems an error related to the path where you have stored the file encoding the weights of the network. Check if the files are located in the path you have passed to the function. If the file are present then it could be an issue of the way you are pointing to them in your call to realpath().


Use DirectIO on linux

We don't need to stress the pagecache with the recordlog. Just adding the custom flag on the OpenOptions should do the trick.

@fulmicoton  from my understanding to bypass the pagecache we need this .custom_flags(libc::O_DIRECT | libc::O_SYNC).

O_DIRECT have some alignment requirement that needs to be satisfied (I have a solution)
I still need to confirm O_DIRECT works in tokio::fs am not sure this is supported in tokio
Right now I want to confirm with you if it's these flags you were thinking of or any other flag. As I am still exploring the possibilities on this issue.

We call fsync explicitly (and not always). No need for O_SYNC.

I still need to confirm O_DIRECT works in tokio::fs am not sure yet this works in tokio.

Stock tokio does not do anything special with regard to io (unfortunately).
This is just regular file descriptor running regular blocking io.
tokio simply runs them on its blocking thread pool.

What do you mean by failing?
Can you share your branch?
Your replit is on an unoptimized build and I don't know what you are running.

What do you mean by failing?
DIRECT_IO should be memory aligned so flush operations fail with Invalid input meaning the problem of memory alignment.

Can you share your branch?
https://github.com/quickwit-oss/mrecordlog/pull/19

Yes, the replit is an unoptimized build, but I  was thinking both are running in the same context. I am still experimenting with measuring.
The replit is running on:
VERSION="20.04.2 LTS (Focal Fossa)"
ID=ubuntu
ID_LIKE=debian
PRETTY_NAME="Ubuntu 20.04.2 LTS"
VERSION_ID="20.04"
VERSION_CODENAME=focal
UBUNTU_CODENAME=focal

I have updated the replit to run the release build release
The margin is still large on both local box and replit with release

ok let's not use direct io then. Can you see if we can fadvise that we don't need this to be page cached for too long?

Ok will check that


tsdb: observe tombstone files as well

I worry that I have tangled myself into a mess with the WithObserver pattern in order to avoid a mandatory, optional, parameter everywhere. It seems fragile to future changes in that one must consider if upon any opening of a TSMFile, if it may generate a tombstone. Open to suggestions.

During a run of megacheck the following issues were discovered:

During a run of megacheck the following issues were discovered:

During a run of megacheck the following issues were discovered:


restore hash as filename suffix for shader replacement

["Use handle id in filename for shader extract and replace." had transitioned from a hash of the shader contents to using the shader module handle.](https://github.com/LunarG/gfxreconstruct/commit/84fb52ddded424ebf68d7c7c84d708da9827e371
#668 reports a regression in shader replacement.  It looks like 9e1a0316bc54926cafc33bd872e2e6aabcd441ab in #342 changed the filename suffix to be the ShaderModule handle (it had previously been a hash of the shader blob).  https://github.com/LunarG/gfxreconstruct/commit/84fb52ddded424ebf68d7c7c84d708da9827e371 changed the code in VulkanReplayConsumerBase::OverrideCreateShaderModule back to using a hash.
Since 84fb52d doesn't change gfxrecon-extract back to using a hash, I think it's reasonable to assume this was a mistake.
This PR restores the handle behavior.

CI gfxreconstruct build # 1544 passed.


how to change zigbee-switches to another room?

Hi,
I'm new to homebridge. I just succesfully connected my Hue Bridge to my Homebridge by using the first example in the configuration wiki.
My zigbee-switches are now visible in homekit, great! But i can't change them to the correct room. They appear to be fixed to the room where the Homebridge is "placed".
Is there an easy way to fix this issue?

You can only assign or change rooms from a HomeKit app.  The accessory itself (nor Homebridge nor any Homebridge plugin) has no visibility on rooms, groups, scenes, nor automations.

Thanks, i did understand that. But when i try to move it to another room in Homekit, it changes back to its original room (where the homebridge is located in homekit) when i close te settings-page. I do not have this problem with other plugins (for example: Ring, iRobot)

It’s technically impossible to be related to the plugin.
Config changes spontaneously reverting in my experience is due to failing iCloud synchronisation across Apple devices.  Make sure all your Apple devices are on the latest OS version.  Otherwise, there’s very little to do about it, other than waiting for the dust to settle.  You might try and power down some of the devices, I am particularly suspicious of the HomePod mini.

You were right! One of my iPads running on an older version of OS15 was the cause.  After updating the OS version, everything works fine.
Thanks for your support!


NSG typo

Hi @snobu, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;

@naziml can you verify this customer's tweak to the heading?

@tdykstra can you please verify for @naziml?

@ggailey777 Looks good, corrects a typo

@snobu thank you for finding and fixing this typo. We appreciate your help improving the quality of Azure docs.


serverinfo timestamp is wrong

it's off by 2h for some reason

This also happens in the logs when the timezones are involved. When you set the mod logs' timezone as EST (it refuses EDT for some reason), it is an hour behind than what it really should be.
I'm not sure of whether this is a separate issue or not but I told AEnterprise that I was going to comment on it.


[zkevm]: add polygon zkevm

Description
add polygon zkevm, test to be added in kmp

can you at least add TWCoinTypeTests?


⚠ Bofrost.es has degraded performance

In e152537, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1799 ms

Resolved: Bofrost.es performance has improved in 2a8d2c9 after 23 minutes.


Update for mc1.20.3-4

Placeholder PR as no branch currently exists for mc1.20.4.

hi, are there any available builds for this?

hi, are there any available builds for this?

Refer to the CurseForge project


Expanding on how to contribute  and chat sections of README.md file

This file has changed a lot. Closing for age and conflicts with master branch at this point.


Merged main dev 1.30

This PR syncs the dev-1.30 branch with the main branch to keep the dev-1.30 branch up to date.
cc: @drewhagen @Princesso @chanieljdan @fsmunoz

The merge conflict which was resolved, ptal:
diff --cc content/en/docs/concepts/workloads/pods/sidecar-containers.md
index 76eca5cf59,0ad25781ff..0000000000
--- a/content/en/docs/concepts/workloads/pods/sidecar-containers.md
+++ b/content/en/docs/concepts/workloads/pods/sidecar-containers.md
@@@ -9,43 -9,21 +9,53 @@@ weight: 5

  Sidecar containers are the secondary containers that run along with the main
  application container within the same {{< glossary_tooltip text="Pod" term_id="pod" >}}.
 -These containers are used to enhance or to extend the functionality of the main application
 -container by providing additional services, or functionality such as logging, monitoring,
 +These containers are used to enhance or to extend the functionality of the primary _app
 +container_ by providing additional services, or functionality such as logging, monitoring,
  security, or data synchronization, without directly altering the primary application code.

 +Typically, you only have one app container in a Pod. For example, if you have a web
 +application that requires a local webserver, the local webserver is a sidecar and the
 +web application itself is the app container.
 +
  <!-- body -->

 -## Enabling sidecar containers
 +## Sidecar containers in Kubernetes {#pod-sidecar-containers}
 +
++<<<<<<< HEAD
 +Kubernetes implements sidecar containers as a special case of
 +[init containers](/docs/concepts/workloads/pods/init-containers/); sidecar containers remain
 +running after Pod startup. This document uses the term _regular init containers_ to clearly
 +refer to containers that only run during Pod startup.
 +
 +Provided that your cluster has the `SidecarContainers`
 +[feature gate](/docs/reference/command-line-tools-reference/feature-gates/) enabled
 +(the feature is active by default since Kubernetes v1.29), you can specify a `restartPolicy`
 +for containers listed in a Pod's `initContainers` field.
 +These restartable _sidecar_ containers are independent from other init containers and from
 +the main application container(s) within the same pod.
 +These can be started, stopped, or restarted without effecting the main application container
 +and other init containers.
 +
 +You can also run a Pod with multiple containers that are not marked as init or sidecar
 +containers. This is appropriate if the containers within the Pod are required for the
 +Pod to work overall, but you don't need to control which containers start or stop first.
 +You could also do this if you need to support older versions of Kubernetes that don't
 +support a container-level `restartPolicy` field.
 +

I accepted the changes in the current branch (HEAD) rather than the incoming changes (upstream/main).

/lgtm
@sftim I was told you were involved with these changes so I wanted to tag and ask if this resolution looks good to you.

@sftim I was told you were involved with these changes so I wanted to tag and ask if this resolution looks good to you.

No need for me to check as https://github.com/kubernetes/website/pull/45195 is not yet approved / merged.


bug: LazyVim.telescope("files") fails on mapped network drive

Did you check docs and existing issues?

[X] I have read all the LazyVim docs
[X] I have searched the existing issues of LazyVim
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
0.9.5 stable release
Operating system/version
Windows 10 Pro / 10.0.19045 Build 19045
Describe the bug
If I launch Neovim with LazyVim config in a folder that is placed on a mapped network drive, then, even though the telescope menu properly appears, it does not list any directory item, no matter what string I type.
For context, Telescope find_files works fine on the same location, so probably this bug is not upstream.
Steps To Reproduce

Launch Neovim with LazyVim in a folder that is located on a mapped network drive.
Type :=LazyVim.telescope("files")() .

Expected Behavior
The telescope fuzzy finder menu shows up as usual, except no item is found, no matter the typed in string. Also the following error message appears:
[telescope] [WARN  11:01:00] C:/Users/botond.kalocsai/.local/share/nvim-data/lazy/telescope.nvim/lua/telescope/pickers.lua:671: Finder failed with msg:  .../share/nvim-data/lazy/telescope.nvim/lua/telescope/_.lua:77: Failed to spawn process: {
[telescope]   command = "rg",
[telescope]   pid = "ENOENT: no such file or directory",
[telescope]   stderr = {
[telescope]     close = <function 1>,
[telescope]     eof_rx = <function 2>,
[telescope]     eof_tx = <function 3>,
[telescope]     read_start = <function 4>,
[telescope]     start = <function 5>,
[telescope]     <metatable> = <1>{
[telescope]       __call = <function 6>,
[telescope]       __index = <table 1>,
[telescope]       __tostring = <function 7>,
[telescope]       new = <function 8>,
[telescope]       super = <2>{
[telescope]         __call = <function 6>,
[telescope]         __index = <table 2>,
[telescope]         __tostring = <function 7>,
[telescope]         close = <function 9>,
[telescope]         new = <function 10>,
[telescope]         super = <3>{
[telescope]           __call = <function 6>,
[telescope]           __index = <table 3>,
[telescope]           __tostring = <function 7>,
[telescope]           extend = <function 11>,
[telescope]           implement = <function 12>,
[telescope]           is = <function 13>,
[telescope]           new = <function 14>
[telescope]         },
[telescope]         <metatable> = <table 3>
[telescope]       },
[telescope]       <metatable> = <table 2>
[telescope]     }
[telescope]   },
[telescope]   stdin = {
[telescope]     close = <function 15>,
[telescope]     eof_rx = <function 16>,
[telescope]     eof_tx = <function 17>,
[telescope]     read_start = <function 18>,
[telescope]     start = <function 19>,
[telescope]     <metatable> = <table 1>
[telescope]   },
[telescope]   stdout = {
[telescope]     eof_rx = <function 20>,
[telescope]     eof_tx = <function 21>,
[telescope]     handle = <userdata 1>,
[telescope]     <metatable> = <4>{
[telescope]       __call = <function 6>,
[telescope]       __index = <table 4>,
[telescope]       __tostring = <function 7>,
[telescope]       iter = <function 22>,
[telescope]       new = <function 23>,
[telescope]       read = <function 24>,
[telescope]       super = <table 2>,
[telescope]       <metatable> = <table 2>
[telescope]     }
[telescope]   },
[telescope]   uv_opts = {
[telescope]     args = { "--files", "--color", "never" },
[telescope]     cwd = "\\3DH-FILE01.3dhistech.com\\Dev_CT_Main\\Users\\Botond_Kalocsai_work\\Python packages",
[telescope]     stdio = {
[telescope]       [2] = <userdata 1>
[telescope]     }
[telescope]   },
[telescope]   <metatable> = <5>{
[telescope]     __index = <table 5>,
[telescope]     _for_each_pipe = <function 25>,
[telescope]     close = <function 26>,
[telescope]     new = <function 27>
[telescope]   }
[telescope] }
[telescope] stack traceback:
[telescope] 	.../share/nvim-data/lazy/telescope.nvim/lua/telescope/_.lua:77: in function 'spawn'
[telescope] 	...cope.nvim/lua/telescope/finders/async_oneshot_finder.lua:48: in function 'finder'
[telescope] 	.../nvim-data/lazy/telescope.nvim/lua/telescope/pickers.lua:667: in function <.../nvim-data/lazy/telescope.nvim/lua/telescope/pickers.lua:666>
[telescope] 	[C]: in function 'pcall'
[telescope] 	.../nvim-data/lazy/telescope.nvim/lua/telescope/pickers.lua:666: in function <.../nvim-data/lazy/telescope.nvim/lua/telescope/pickers.lua:611>

Repro
-- DO NOT change the paths and don't remove the colorscheme
local root = vim.fn.fnamemodify("./.repro", ":p")

-- set stdpaths to use .repro
for _, name in ipairs({ "config", "data", "state", "cache" }) do
  vim.env[("XDG_%s_HOME"):format(name:upper())] = root .. "/" .. name
end

-- bootstrap lazy
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)

-- install plugins
local plugins = {
  "folke/tokyonight.nvim",
  "folke/LazyVim",
  -- add any other plugins here
}
require("lazy").setup(plugins, {
  root = root .. "/plugins",
})

vim.cmd.colorscheme("tokyonight")
-- add anything else here

Then just use find_files instead? files will use git_files or find_files where needed


Implement Reflect for InputMap

Currently it's not possible to directly use an InputMap<A> as an asset (e.g. Handle<InputMap<A>>), as trait Asset now requires TypePath.

Can you swap this to a Reflect impl instead? It will have the same effect, but is much more powerful.

Sure thing. Didn't know Reflect also implements TypePath, but I'll try to derive it for other types too

So since PetitSet is a foreign type, I had to create a newtype for it just to have a reflectable PetitSet, and then manually implement Reflect (and other reflect related traits). It doesn't seem like a bad solution, but one major issue is that UserInput::Chord is not a Box anymore (since Box<dyn Reflect> is not reflectable, yet), therefore it drastically increased in size and I'm not really sure what to do about it.

Right, okay :) Let's merge this as is then.

Um. so this PR is pretty much the same as #331 (although it's slightly outdated). And I've seen that there is an ongoing work to remove a PetitSet as a dependency, so I'm not sure what's the best course of action here..

We should merge this and unblock this feature request, then remove PetitSet in another PR.

I'm going to tackle this in #400, which should be done shortly :) This was super helpful to push me towards "remove petitset" as the right architectural decision, thank you.


[Feature] Like the post Feature

Deliverables:

[ ] Backend API for keeping the record of likes of the post.

[ ] Frontend UI to like the post on the Blog card.

any Resources {i search it but....}??  i don't know anything About it but want to know...

I am interested in working on this issue.

https://stackoverflow.com/questions/15407985/django-like-button
https://www.youtube.com/watch?v=VoWw1Y5qqt8
https://www.youtube.com/watch?v=PXqRPqDjDgc
I would like to work on this if you're not interested @ritiksoni00. Pls comment if you're not.

@AkM-2018 I'm interested I want to work as I'm a beginner so if it is possible that we can work together?? @Kajol-Kumari  Please suggest us What should we do???

https://stackoverflow.com/questions/15407985/django-like-button
https://www.youtube.com/watch?v=VoWw1Y5qqt8
https://www.youtube.com/watch?v=PXqRPqDjDgc
I would like to work on this if you're not interested @ritiksoni00. Pls, comment if you're not.

Thank you for this @AkM-2018
will I able to solve this issue via these resources ???

You need to know how to work with django, some basic knowledge. Also some basic html.
You can then follow the vid. It will be straight forward then.

Hey @ritiksoni00 as you already have two issues assigned to you, I am assigning it to @AkM-2018.

hahaha ! It's not like I want to assigned me for all the issues i just want to try new challenges

You need to know how to work with django, some basic knowledge. Also some basic html.
You can then follow the vid. It will be straight forward then.
I don't know how we'll work together. Take the issue. If you can't solve it within some days, I'll ask @Kajol-Kumari reassign it to me.
Is it okay, @Kajol-Kumari ?

Thankyou @AkM-2018

Just to notify...
Since the individual blog posts pull request is in process, I've put the like button directly on the home page(for now).

The user can like a post only when logged in. Else he is redirected to the login page.
One problem I'm facing is, whenever someone likes the post he is sent back to the top of the home page due to HttpRedirectResponse(reverse('blog-home')).

@Kajol-Kumari, that was not the problem I was facing. Anyways, I've put a pull request. Lemme know the changes if any

Hey @Kajol-Kumari can I work on this issue??

Hey @AkM-2018 can you please provide me your work status for this issue as you were supposed to open up a new PR for this issue. @kritikaparmar-programmer Let's wait for 3 days, if @AkM-2018 won't response i'll assign it to  you.

@Kajol-Kumari  Okay 👍

There's some bug in my code which I'm not able to figure out. I've spent a lot of time on this issue. Pls, reassign it to someone else.

@kritikaparmar-programmer would you like to go ahead with this issue?

Yess @Kajol-Kumari

@kritikaparmar-programmer are you still working on this issue.

Yes I'm working on this issue though facing some issues I'll let you know @harsh-9in

okk @kritikaparmar-programmer

hey @harsh-9in and @Kajol-Kumari please assign this issue to someone else. I don't know why I'm facing issues. Better not to waste time.

Can I try this ? @Kajol-Kumari

@Kajol-Kumari I think we can use Ajax to implement like button. Furthermore, using jQuery will immensely decrease the difficulty. I would like to work on this.

Can I be assigned this issue? @harsh-9in .

Go ahead :+1:


API Key exposed

API keys exposed in covid-tracker-master/lib/network-requests/api_service.dart. Please check and rectify.

Thanks for pointing it out. I made this project as a beginner and never took care of this thing. I'll convert it into an environment variable right away.

Also, just saying. Might wanna check this out here

Since the project was open sourced with the intent of allowing people to take inspiration, it's fine if someone clones it. The licensing was quite open.

Sure, just letting you know. But, the license was deleted, which was in contravention to the license rules itself. That's all.


pusher.connection is not mocked meaning mocking a bind on connection 'state_change' is unavailable

Currently connection isn't mocked within PusherMock so when using the following code in my component, my tests break:
pusher.connection.bind(
	'state_change',
	(states) => {
		console.log({states});
	}
);

As a workaround for this, the functionality that I needed to test was the bind and emit on the connection to make sure functions on connection state changes It's not the best solution but I mocked it as a PusherChannelMock within the code, you can see alterations on my fork (https://github.com/DeadEnglish/pusher-js-mock).
Obviously this should be mocked correctly when possible but for now it'll do until I get time to properly comeback and possibly mock the entire ConnectionManager correction.

Got it, @DeadEnglish, thanks for opening the issue and explaining how to work around this. If you have time, please submit a PR 🙏

@nikolalsvk Have created a PR for you, cheers for the switch reply 😄 #31

We have an application test that requires this too! Thank you @DeadEnglish for pushing this through!

Released the new changes by @DeadEnglish in 0.3.3 version, enjoy!

This seems to be working for TS but not for JS counterpart.
https://github.com/nikolalsvk/pusher-js-mock/issues/28
I had to redefine the mock to get this to work:
pusherMock.connection = new PusherChannelMock('connection');
pusherMock.connection.emit(...)

This seems to be working for TS but not for JS counterpart.
https://github.com/nikolalsvk/pusher-js-mock/issues/28
I had to redefine the mock to get this to work:
pusherMock.connection = new PusherChannelMock('connection');
pusherMock.connection.emit(...)


add Vite PWA plugin

The service worker will check for updates on page load, downloading them in the background ready to present on the next page load.

[x] Test update
[x] Double check that you do not have caching features enabled, especially immutable, on locations like:

/
/sw.js
/index.html
/manifest.webmanifest

$ curl -s -I -X GET localhost:3000/index.html
HTTP/1.1 200 OK
Server: nginx/1.24.0
Date: Wed, 05 Jul 2023 15:09:06 GMT
Content-Type: text/html
Content-Length: 4612
Last-Modified: Wed, 05 Jul 2023 15:07:18 GMT
Connection: keep-alive
Vary: Accept-Encoding
Cache-Control: public, max-age=0, s-maxage=0, must-revalidate
Accept-Ranges: bytes

https://vite-pwa-org.netlify.app/deployment/nginx.html
Running production nginx server using docker compose, test command: curl -s -I -X GET localhost:3000/index.html

Service worker downloads update in the background... (since I reverted the last commit, those files were already in the disk cache!). New code is used on next page reload.


Exec tachikoma update<PHONE_NUMBER>3426

:hamster::hamster::hamster:   Strategy none
Powered by Interval Pull Request as a Service - Tachikoma.io Announcement: 2015-06-28 Updated permissions - Tachikoma.io

Coverage remained the same at 68.0% when pulling b765fe259b04f3c1baeb7b409d340550aa77efa1 on pazjacket:tachikoma/update-20160410163426 into 0c477cea399180e0dc0455b31d685efdbee0f3af on sanemat:master.


[Concurrency] Store child record when async let child task spawned

Child tasks create a child record and store it when they start.
Without this the new test added here would hang forever, as during cancellation of the parent task we traverse the records in the parent, and cancel all children we find in there. If we didn't record these records it would mean that we would not find the child by traversing the parent on cancellation in swift_task_cancelImpl and they'd never get cancelled.
Group children already worked before, so this is only about async lets.
We also had the idea of all child tasks being task group children, but that's not implemented.
It would mean that every function implicitly becomes a task group, and then children do the exact same as groups do -- this could be quite useful, but that's not yet done in the exact "desugar to exactly an implicit group" way.
Resolves rdar://74464296
Replaces #36645

@swift-ci please smoke test

@swift-ci please smoke test

Unrelated failure?
00:32:48.138   Swift(macosx-x86_64) :: ModuleInterface/BadStdlib.swiftinterface

@swift-ci please smoke test

Also solves rdar://76446550

Just to sanity check we're still good here.

@swift-ci please smoke test and merge

Unrelated failure... 00:54:23.248   lldb-api :: lang/swift/async/stepping/step-in/TestSwiftStepInAsync.py

@swift-ci please smoke test macOS

@swift-ci please smoke test macOS Platform

Disabled the failing test https://github.com/apple/llvm-project/pull/2854

@swift-ci please smoke test macOS Platform

@swift-ci please smoke test macOS

01:03:39.169
/Users/buildnode/jenkins/workspace/swift-PR-macos-smoke-test/branch-main/swift-tools-support-core/Tests/TSCBasicTests/ProcessTests.swift:187: error: -[TSCBasicTests.ProcessTests testSignals] : failed - timeout waiting for signals to be processed

😢

@swift-ci please smoke test macOS Platform

note to self, the closure is now non escaping thanks to https://github.com/apple/swift/pull/36975


[Helix] Razor Test.Common.TestCompilation.GetMetadataReferences will return null

Tracking for tests failing on helix due to:
        D:\a\1\s\src\Razor\test\testassets\Razor.Test.Common\TestCompilation.cs(25,0): at Microsoft.CodeAnalysis.TestCompilation.GetMetadataReferences(Assembly assembly)
        D:\a\1\s\src\Razor\test\testassets\Razor.Test.Common\TestCompilation.cs(46,0): at Microsoft.CodeAnalysis.TestCompilation.Create(Assembly assembly, SyntaxTree syntaxTree)

Microsoft.AspNetCore.Mvc.Razor.Extensions.ViewComponentTagHelperDescriptorProviderTest.DescriptorProvider_FindsVCTH()

https://github.com/aspnet/AspNetCore/blob/haok/helix/src/Razor/test/testassets/Razor.Test.Common/TestCompilation.cs#L25

Seems this is an issue with the helix specifically, where the deps file isn't being properly published. @HaoK assigning this to you. Feel free to close if this is not relevant any more.


_FanOutBuild should depend on BeforeBuild and AfterBuild

It's fairly surprising that making a target run before/after these extension points makes it not trigger at all for the top-level build. #68 should be addressed together with this so that such an extension target knows whether it's running in the top-level build.

The current behavior actually matches what MSBuild and the .NET SDK does when you set multiple TFMs; only the inner builds trigger BeforeBuild/AfterBuild. I'm not sure why I thought otherwise.
That said, it still seems useful to provide a hook for this use case.


[Feature]  remove AddHeadersPolicy and AddHeadersFromContextPolicy and add appropriate policies

AddHeadersPolicy and AddHeadersFromContextPolicy both policies should not be used to create client. AddHeaderPolicy should not hard to find replacement policies but we need to redesign AddHeadersFromContextPolicy. The reason is pipeline should only take one header at one policy. Both policies take multiple headers.

No longer a requirement.


Section controller work for template versioning

For #1373

Moved sections controller, views and routes to org_admin namespace
removed admin_ prefix from action names
updates views to work with new routing
updated tests
added 'edit', 'new', 'index' and 'show' actions that we can hopefully use when updating the UI

Update sections controller to use versionable methods (also added in StandardError handling for phases controller).
@jollopre The tests are failing due to an issue with the older version of Versionable.find_in_space(). The recursion is not happening properly.

synced with latest sprint2 branch


Impl Deref for newtypes

Wanted to get opinions on this:
On FmodStudio it's a clear improvement in my eyes.
But on AudioSource it gets a bit hairy (internally, user-facing looks ok) because AudioSinkPlayback contains some function members with the same name as the ones in EventInstance. So you still need the .0 syntax in some places (internally).

Can you elaborate what your initial goal was with this PR?

Mainly to reduce the amount of code the user needs to write. This is easiest to see in the diff of the parameters example


Update Validator.php

Add aliases for built-in validators: 'bool', 'float', 'int' - to set validator by PHP type, 'range' and 'regex' - similar to class name
Update examples in phpDoc's

Q
A

Is bugfix?
❌

New feature?
❌

Breaks BC?
❌

Tests pass?
✔

Fixed issues

Thanks for contributing but I don't think that's a good idea to have multiple aliases since it hurts overall readability. In some places it may be one alias, in another - another alias.

@samdark
really?
'number' => 'yii\validators\NumberValidator',
'double' => 'yii\validators\NumberValidator',

Yes :) We did it but it doesn't change the fact that it was not a good idea.

@samdark L-logic...


error running the code

Main branch, standard settings used. When running the following error appear.
Tried to solve with cleaning cache, re-download models, etc. but didn't work.
(Dev is working)

error solved


Fix: Typo in the introduction blurb of readme.md

s/this is a still a young projects/this is still a young project/

@sgebbie for your perusal


On ubuntu can't update package list without adding gpg pub key

Reading package lists... Done
W: GPG error: http://ppa.launchpad.net/o2sh/onefetch/ubuntu focal InRelease: The following signatures couldn't be verified because the public key is not available: NO_PUBKEY 956960ACA210E348
E: The repository 'http://ppa.launchpad.net/o2sh/onefetch/ubuntu focal InRelease' is not signed.
N: Updating from such a repository can't be done securely, and is therefore disabled by default.
N: See apt-secure(8) manpage for repository creation and user configuration details.

Solution: in installation guide add step where this GPG key is added.

I updated the wiki. If it's correct, please close this ticket.
https://github.com/o2sh/onefetch/wiki/Installation#ubuntu-ppa


[Feature] Introduces no resampling strategy to enable fitting on the whole dataset

This PR includes the following:

Added NoResamplingStrategyTypes to resampling_strategy.py with an option for shuffling and not shuffling the data
Added FitEvaluator which handles fit and predict a pipeline in case the resampling strategy is in NoResamplingStrategyTypes
Fixes bug in BaseDataLoader where test loader was not being created
Adds capability to base trainer choice to use val_data_loader or test_data_loader depending on availability.

P.S. if you have any suggestions for a better name than FitEvaluator please let me know.

Addresses #191


0.0.16 Release

Hey @stevenheidel, you can also use sbt release on top of master to make the release without setting the versions manually. There's no problem in doing this manually though.


🎃🎃 Hacktoberfest Tasks for Contributors 🎃🎃

If you are willing to contribute to verdaccio and have no idea how to start, here we have a list of task that might be a good starting point for you.

Before putting your hands into the code I'd recommend the following links:

Don't miss our contributing guidelines
Wiki
Build Project
Check all our repos
How to Debug Verdaccio
Any ticket is tagged with PR-Welcome
Do you need some guidance? Chat with us
Do you want to help translating the documentation?
Next Major Release Roadmap

🚨All these tasks should be started branching from 4.x and try to keep them as small as possible to simplify the code reviews. 🚨
⛑ If you start a new task, comment here with the reference of your PR and the name of the task to mark it as done when PR is being merged. ⛑
Refactoring Tasks for Contributors

[x] remove all import assert from 'assert'; in favor of Jest matchers in testing (https://jestjs.io/docs/en/expect) #1042 @awshanks (still many test files (test/**/*.js) to update)
[ ] Improve flow types, eg: remove any and create custom types
[ ] Remove async dependency in favor of ES6 features
[x] Enforce use HTTP_STATUS namespaces instead .expect(404) in multiple files
[ ] All API errors used on test or any core file should be on API_ERROR
[ ] Increase coverage in utilities files (https://codecov.io/gh/verdaccio/verdaccio/tree/master/src/lib) one PR peer file
[ ]  Replace with better code the recursive functions https://github.com/verdaccio/verdaccio/blob/master/src/lib/auth.js#L126
[x] Camel case for all functions eg: https://github.com/verdaccio/verdaccio/blob/master/src/lib/utils.js#L550 (one PR peer file) @ashishsurana
[x] Get rid off require('../lib/logger'); in favor of ES modules import xxx from xxx. @varijkapil13
[x] Replace import * as Utils from '../../lib/utils'; by destructuring format import {xxx, yyy} from '../../lib/utils'; @varijkapil13
[ ] Refactor logger.js with better and clean code
[x] Remove status-cats.js middleware, I'm looking for suggestions here
[x] // $FlowFixMe was added to fix temporary Flow lack of types, check places where is not need it anymore @Utwo
[ ] Refactor configuration https://github.com/verdaccio/verdaccio/issues/991
[ ] Add Flow and apply Flow types, create new ones in some cases to the files are not tagged with //@flow
[ ] Refactor normalizeUserList(oldFormat: any, newFormat: any) Avoid use arguments within the function (unit test included)

🤙🤙🤙🤙🤙 Feel free to suggest new things
🚨Don't forget sign up here🚨 https://hacktoberfest.digitalocean.com/
https://twitter.com/github/status/1044315660342505473

@juanpicado I am in!!! 💃

Created #1040 for

Get rid off require('../lib/logger'); in favor of ES modules import xxx from xxx. WIP @varijkapil13

Created #1041 for

Replace import * as Utils from '../../lib/utils'; by destructuring format import {xxx, yyy} from '../../lib/utils'; WIP @varijkapil13

Created #1049 for

remove all import assert from 'assert'; in favor of Jest matchers in testing

Created #1050 for

Enforce use HTTP_STATUS namespaces instead .expect(404) in multiple files

Created #1051 for

// $FlowFixMe was added to fix temporary Flow lack of types, check places where is not need it anymore

Thanks guys 🎃🎃🎃🎃


"An error occurred"  Generic error when running this script.

Thank you for your work on this tool. I was pointed this direction from E-ISAC. I had no issues running the crowdstrike reporting tool, but sparrow is giving me trouble.
🐛 Summary
What's wrong? Please be specific.
Running sparrow.ps1 - I get prompted for azure cloud instance and exchange instance, I get prompted for username/password and MFA, it accepts and moves on listing the modules and asking if I have an E5 license, I respond yes. Would I like to investigate a certain application, no.  It starts listing its verbose steps, getting to: "VERBOSE: Searching for PowerShell logins using known PS application ID's in the UAL. Then 'Warning, result set may have been truncated; narrow start/end date.'.
It prompts me for my credentials again, my MFA, I enter them and get this error:
"
An error occurred
An error occurred. Contact your administrator for more information.
Error details
•Activity ID: 987bf970-392b-4293-a91c-f2c891eb61a1
•Error time: Fri, 08 Jan 2021 18:11:08 GMT
•Cookie: enabled
•User agent string: Mozilla/4.0 (compatible; MSIE 7.0; Windows NT 10.0; Win64; x64; Trident/7.0; .NET4.0C; .NET4.0E)
"
To reproduce
Steps to reproduce the behavior:

Listed in detail above

Expected behavior
I am expecting it to pass authentication like it does after the first request.
Any helpful log output or screenshots
Paste the results here:
An error occurred

An error occurred. Contact your administrator for more information.

Error details
•Activity ID: 987bf970-392b-4293-a91c-f2c891eb61a1
•Error time: Fri, 08 Jan 2021 18:11:08 GMT
•Cookie: enabled
•User agent string: Mozilla/4.0 (compatible; MSIE 7.0; Windows NT 10.0; Win64; x64; Trident/7.0; .NET4.0C; .NET4.0E)

Add any screenshots of the problem here.

You might need to reduce the following line (7):
[datetime] $StartDate = [DateTime]::UtcNow.AddDays(-364)
To a lower number such as:
[datetime] $StartDate = [DateTime]::UtcNow.AddDays(-90)
Can you please try this and let us know how it works?

You might need to reduce the following line (7):
[datetime] $StartDate = [DateTime]::UtcNow.AddDays(-364)
To a lower number such as:
[datetime] $StartDate = [DateTime]::UtcNow.AddDays(-90)
Can you please try this and let us know how it works?

Hello, we have not heard back from you regarding this issue. If no response is given within three days, I will go ahead and close out this issue.

Hello, we have not heard back from you regarding this issue. If no response is given within three days, I will go ahead and close out this issue.

Hello - apologies. I was able to resolve the issue but not with the suggestion above. We are in GCC low and I thought maybe we'd need to pick the gov cloud or one of the others (in the beginning) but I found if I just chose the o365 and azure defaults then the script runs successfully.
Thank you - hopefully that helps provide some context.

Hello - apologies. I was able to resolve the issue but not with the suggestion above. We are in GCC low and I thought maybe we'd need to pick the gov cloud or one of the others (in the beginning) but I found if I just chose the o365 and azure defaults then the script runs successfully.
Thank you - hopefully that helps provide some context.


Tcp support

Per offline discussion can we remove the secure support for now since I don't think we need it and it will be less to review/test. Obviously can keep it in a branch. Will look again when updated.


How to update firmware?

There is an update instruction for LilyGo-T-Call-SIM800. Is it suitable for this board?
https://github.com/Xinyuan-LilyGO/LilyGo-T-Call-SIM800/blob/master/doc/How to update firmware.md

Also I can not fully follow the instructions given in this documenation here
https://github.com/Xinyuan-LilyGO/T-A7670X/blob/main/docs/update_fw.md


fix: handle undefined mediaDevices

Description
Fix: useMediaDevices(): TypeError: Cannot read property 'addEventListener' of undefined
Type of change

[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 not work as before)

Checklist

[x] Read the Contributing Guide
[x] Perform a code self-review
[-] Comment the code, particularly in hard-to-understand areas
[-] Add documentation
[-] Add hook's story at Storybook
[-] Cover changes with tests
[x] Ensure the test suite passes (yarn test)
[-] Provide 100% tests coverage
[x] Make sure code lints (yarn lint). Fix it with yarn lint:fix in case of failure.
[-] Make sure types are fine (yarn lint:types).

:tada: This PR is included in version 13.22.5 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Fix FIFO pipe handling

Originally it was possible to upload data from a FIFO pipe, but at some point that feature was lost. This commit brings it back.

Has this project repository been abandoned?


Fixed OptionsTest for different cultures

The test was failing for en-GB, as on the parsing side the invariant culture is used

We took a similar PR from someone else yesterday, But thanks for submitting this!  https://github.com/aspnet/Extensions/pull/1204


Support Mixed-Mode debugging

We should be able to step into other .NET code from our PS scripts!

+1!

That's great!
In one of my PowerShell talks, I fire up visual studio, compile a C# class and in the debug tab launch PowerShell ISE. I load up a script that does an Add-Type on the DLL just built (and is running in VS). I set a break point in the PowerShell script and then step into the C#.


Tasks as structs

Have you considered representing tasks by structs of their context instead of functions? This way we could avoid explicit context deserialization and related bugs. E.g. this would be impossible:
struct Foo;
struct Bar;

#[job]
pub async fn foo(job: CurrentJob) -> Result<..> {
    let context: Foo = job.json()?;
}
foo.builder().set_json(&Bar).spawn(&db).await?;

It would be something like:
#[derive(Job)]
struct MyJob;

impl CurrentJob for MyJob {
   fn run(self) -> Result<..> {
   }
}

MyJob { ... }.spawn(&db).await?;

It could make sense. There are some questions to answer around how this would integrate with the existing system - would it replace job functions? What if you want to use some other serialization than JSON? etc. It's all solvable but needs some work to prototype and get a good interface.

It seems straightforward with the trait to give an ability to redefine the serialization method:
enum Payload {
   Jsonb(String),
   Bytea(Vec[u8]),
}

trait CurrentJob {
   fn serialize(&self) -> Result<Payload> {
       serde_json::to_string(self).map(Payload::Jsonb)
   }
}

would it replace job functions?

At the first glance it doesn't seem to be less agile comparing to functions so why have both?

Hmm, it might make more sense to have a builder() trait method, and then change JobBuilder to use Cow so that JobBuilder can be used as the return type. This would help bridge the gap between the two approaches.

Do you mean moving decision on serialization method to the job spawning step? I can't imagine a job which would make sense to serialize differently. So I'd rather prefer the serialization to be defined once on the level of the trait implementation (where it could be derived). Otherwise it could lead to different serializations by mistake.

Actually the idea was that other fields on the struct could be utilized when spawning the job. eg. you could have a field in the struct which was used as a delay, or as a channel argument. Changing the serialization is also possible though, and useful in those circumstances, since the job payload might be only one part of the struct.

Otherwise it could lead to different serializations by mistake.

So could any approach where the user can control the serialization. For example, in your suggested approach you need a matching deserialize method, and that method needs to use the same type for deserializing.
If you really want to reduce the risk of that, you could do something like this:
trait Job {
    type Payload;
    fn serialize_payload(builder: &mut JobBuilder, payload: Self::Payload) { <default impl> }
    fn deserialize_payload(current_job: &mut CurrentJob) -> Result<Self::Payload> { <default impl> }
    fn builder(self) -> JobBuilder;
}

Where the derive would by default produce an implementation where Self::Payload == Self.

Actually the idea was that other fields on the struct could be utilized when spawning the job

Do you mean there would be some required fields for the struct?

For example, in your suggested approach you need a matching deserialize method, and that method needs to use the same type for deserializing

And we also need to know to what db type map the Payload, maybe an associated const PAYLOAD_DB_TYPE = PayloadDbType::Jsonb;?

This change to structs removes the possibility of custom context arguments though, right?
You would provide the context to the struct if I understood that correctly, but that means every spawner needs access to all context required. I think this is a big downside, as message based systems usually benefit from a single runner owning resources and noone having access, right?

Do you mean JobRegistry::set_context? Why can't we pass context as an argument to the job:
trait CurrentJob {
   fn run(self, ctx: Context) -> Result;
}

Yeah that would work, then I misunderstood part of the intent. Of course you would need to extract context manually from the map.

you would need to extract context manually from the map

Not necessarily, can't we make conext a generic parameter? Then set_context would just replace it:
struct JobRegistry<C> {
    ...
    context: C
}

impl<C> JobRegistry<C> {
    fn set_context(&mut self, context: C) {
        self.context = context;
    }
}

This way in a general case we're statically checked without a need for that extraction from the map. But still if someone wants to sacrifice the type safety for an ability to add fields to the context dynamically they could just make their context a map themselves.

Also with structs we could address another pain point of the current implementation. Currently we can JobBuilder::spawn a job, but if we forget to add it to the JobRegistry::new(&[forgotten_job]) we get a runtime error.
If instead the registry would be bound to enum of jobs and the scheduling is done on the enum rather than on a job itself, there won't be a way to schedule a job until you add it to the enum (I removed job context for clarity, here's the full version):
use serde::{de::DeserializeOwned, Deserialize, Serialize};
use std::fmt::Debug;
use std::marker::PhantomData;

struct JobRegistry<J> {
    jobs: PhantomData<J>,
}

trait CurrentJob: Debug + Sized + DeserializeOwned {
    fn run(self) {
        println!("{:?}", self);
    }
}

trait JobList: CurrentJob + Serialize {
    fn schedule(&self) -> String {
        serde_json::to_string(self).unwrap()
    }
}

impl<J> JobRegistry<J>
where
    J: CurrentJob,
{
    fn spawn_from_db(&self, db_data: &str) {
        let job = serde_json::from_str(db_data).unwrap();
        self.spawn_internal(job)
    }

    fn spawn_internal(&self, job: J) {
        job.run()
    }
}

#[derive(Debug, Deserialize, Serialize)]
struct JobFoo;

impl CurrentJob for JobFoo {}

#[derive(Debug, Deserialize, Serialize)]
enum Jobs {
    Foo(JobFoo),
}

// These impls could be probably automated by a macro
impl JobList for Jobs {}
impl CurrentJob for Jobs {
    fn run(self) {
        match self {
            Self::Foo(j) => j.run(),
        }
    }
}

fn main() {
    let r = JobRegistry {
        jobs: PhantomData::<Jobs>,
    };

    // Scheduling is done using job list rather than a particular job,
    // so you can't schedule a job until you add it to the list
    let db_data = Jobs::Foo(JobFoo).schedule();

    r.spawn_from_db(&db_data)
}


pull request for e2e test at 02/06/2017 12:15:05

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Finally switch to ARM GCC 9, new Cygwin, new Ant, new openJDK

ARM GCC 9 is supported by the NuttX we use since https://github.com/PX4/Firmware/pull/15139 and the Windows build got fixed with https://github.com/PX4/Firmware/pull/16040 so I'm switching to ARM GCC 9 to use the toolchain on PX4 master
Apache Ant always deletes the old version (URL no longer available) when a new one comes out so I'm updating the version
For the Windows toolchain I hosted an old portable JDK version for a long time now. This is not really a security issue since the version does not get installed or is accessible through the PATH. It only resides in its folder and gets used to build jmavsim. Nevertheless it makes sense to update and since I discovered openJDK is providing portable zip archives which Oracle is not I'm switching.
Also correct the SITL build target which changed a long time ago.

@MaEtUgR No .msi file? (or on Amazon)? When that exists also need to update https://dev.px4.io/master/en/setup/dev_env_windows_cygwin.html

@hamishwillee Right, I tagged to make CI build a new installer but it failed. Then I debugged locally since the error message is unexpected and it's reproducable but still hard to find. Maybe it has something to do with the size or number of files. Not solved yet. I'll run CI again once I figured it out.

Thanks @MaEtUgR . Been wondering when this would arrive :-)

I figured the main problem is file size. The files to compress into the installer archive exceed 2GB and that's not supported in WiX. So I need to either find a way to strip it down or split it up into multiple archives (which is tedious to download). I checked and the reason is Cygwin increasing its size of the base installation.


feat(filter): add support to use unused partitions in os-disk

Why is this PR required? What issue does it fix?:
Sometimes the nodes may have only one large disk with many partitions. NDM currently ignores the complete disk if ti contains a system partition. This PR adds support to create blockdevice resources from the unused partitions on the OS disk.
What this PR does?:

adds a new feature gate to enable / disable using OS disk for blockdevices
modify the os disk filter to exclude only the partition that contains the system mountpoints and not the entire disk

Does this PR require any upgrade changes?:
No
If the changes in this PR are manually verified, list down the scenarios covered::
Any additional information for your reviewer? :
Mention if this PR is part of any design or a continuation of previous PRs
Checklist:

[x] Fixes openebs/openebs#3133
[x] PR Title follows the convention of  <type>(<scope>): <subject>
[x] Has the change log section been updated?
[x] Commit has unit tests
[ ] Commit has integration tests
[ ] (Optional) Are upgrade changes included in this PR? If not, mention the issue/PR to track:
[ ] (Optional) If documentation changes are required, which issue on https://github.com/openebs/openebs-docs is used to track them:

Codecov Report

Merging #524 (cd9680f) into master (85c8faa) will decrease coverage by 0.26%.
The diff coverage is 40.00%.

@@            Coverage Diff             @@
##           master     #524      +/-   ##
==========================================
- Coverage   47.87%   47.61%   -0.27%
==========================================
  Files          73       73
  Lines        3348     3352       +4
==========================================
- Hits         1603     1596       -7
- Misses       1584     1591       +7
- Partials      161      165       +4

Impacted Files
Coverage Δ

cmd/ndm_daemonset/filter/osdiskexcludefilter.go
74.35% <ø> (-1.26%)
:arrow_down:

pkg/features/features.go
100.00% <ø> (ø)

pkg/mount/mountutil.go
72.00% <40.00%> (-1.95%)
:arrow_down:

cmd/ndm_daemonset/probe/eventhandler.go
42.59% <0.00%> (-5.56%)
:arrow_down:

cmd/ndm_daemonset/probe/udevprobe.go
52.51% <0.00%> (-2.24%)
:arrow_down:

Continue to review full report at Codecov.

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


Add AIP for GHO Cross-Chain Launch

https://governance.aave.com/t/arfc-gho-cross-chain-launch/17616

After reviewing the changes, the fixes seem ok to me

for the GHO capo adapter I would recommend to also create a PR in aave-capo

@ianflexa just pushed https://github.com/bgd-labs/aave-capo/pull/34

Moved GHO listing out of this AIP so it can be executed after Chainlink lane activation.
AIP for GHO onboard to Arbitrum Aave V3 Pool here: https://github.com/bgd-labs/aave-proposals-v3/pull/356


[Addition]: 18 Self Witnessing Pisces hotspot Clusters that keep moving around the country but the clusters stay in hexes from new wallet

Hotspot b58 Addresses
112GKokbwQmJorQMwgFrKQJ5fy2XSPvfoF56Kn7i5UMW1YRfRAf7
112hJMRvwnTPxk3jmmTezn2Nxi9mmSbKjn35U7T4yDmfZ2nPUtCE
11CCwSMfqHUwzisEwu1H8Ez7VNiNNerUTfeLp7YyBkaizLq7HmG
11gJiB7inBJXohTHjhFoqtr57NWeaYf4RvYAtWW1jqbQhNcp8Vo
112BWg8nhhEQWr13d6qoPQkBBkedSQYWUuDrB1bUT5Q8soRiPAoq
1121whWWHiUvqTCngfdwkwpZSyfHGibSVZABa4VeqMFDxMKZiBLa
112MdzmpWfnPSD5J7nd5JgNuetSASiDcKWLPRE5wsWqUGpPyz3dH
112QDZmYmTNBXokefcJm9cHgMSPnjeGef5jn63EXGYsp426JSckv
11NREvZTBwCkiQkHuQcN7uhYRkRKuqWiNU1aLz6eiH7d7qSypqS
11R4SnDgdTyimn5xbkZuRiNgEgDduvdTwvHVTrRGPko49DQNoRy
112j6f3Gi7XKP89YtVKXHLrCkBnPUZYFenWLg3mvFt4E85N7WgYa
11a2TY9GFog64TjdveyPwdxdJpc5xGfWhqu43v2TGYuumzsjVj5
11tREcGyYe3inVzj4VDLJWpeWzRVRopP892ucn672koP3LnNgYb
1123Y91TWLtxgxT6qe8aynHvERKQTQTsusmgKrcDEWMSdmW6kekU
11fjdXHvLpgbEF4QNT9wpVaFeUvzUCBdfxH9AVZ94KmjaePwDhZ
11236jkuS2jRe2dT7Z91p8cAWJhJVH4wiXuM7g2DKrZunK9LWPbP
11dfKhQFxnFBHDxdXoRyYE1p66D3gyayJjPSbhLy5WcttN6vTKR
112H2qCh8eChj8XvW4oeivmkxf3TzgnhMYHgFvUcdzYHaXwDGZam
Discord Handle
No response
Reason(s)
Self witnessing Pisces Clusters of at least 18 hotspots on 3 wallets(SO FAR).
The first the cluster moved all the across the country and a new cluster with a new wallet popped up in its place, after I noticed that the cluster moved and new Pisces from a new wallet I had to investigate.
Upon checking the beacons I found another cluster same setup just new wallet. I will keep digging if I find any more connected to these.
https://explorer.helium.com/accounts/14jQpCH7N9kQDDvGU6XoKivepJx5jYQjoaQawNuBiHStm6GCL7f/hotspots
https://explorer.helium.com/accounts/13vL8Y1qsFv2Z5HaZoB9VCquV8Cu4QRSFzN6JTwjkDBvL4zT6Em/hotspots
https://explorer.helium.com/accounts/138KcRyqb8h1JWkWUAn1zGc3EVzdSn6KikDUWpZKCGPJVpzPoKq/hotspots

I think thos one got skipped? Has anyone looked into this one yet?

All of the coins from these spoofers have reached an end point wallet.  Here is where they all land.
https://explorer.helium.com/accounts/14Qr1XTKZtMhotRaJEYt3wK9es4yDXXk57X2qCJcHNLHdnKDL4Y

Upon tracking down the above end point wallet i found a massive amount of miners spoofing in the same manner. Self witnessing on different wallets. You can check this transaction and see all involved in addition.
https://explorer.helium.com/txns/dmgJ4pdRHzz4S8OITjcaTEH1eXyWgrFuryeKnaKe-mM

Tracking the endpoint wallet reveals alot of questionable activities. All seem to be connected.

Even more hotspots connected all pisces miner self witnessing across multiple wallets.  This puts it at over 30+ hotspots all connected in self witnessing clusters
https://explorer.helium.com/accounts/141Gou6n9inLHbWF5KhpAJA2rLpXrVLdR1t86rzbuyPXxBigzux/hotspots
https://explorer.helium.com/accounts/13JmehLBSpHNEFCGz7JrbPnufKjG8W31RiJ9hthQEJPnnhcGVyi/hotspots


Security Issue

I have found some security issues I would like to responsibly disclose. Please let me know your preferred process to resolve this.

Please write<EMAIL_ADDRESS>


One import function does not exist.

from dqrf.ops.functions.ms_deform_attn import SamplingAttention_RA, SamplingEncAttention, SamplingAttention_dec
the function 'SamplingAttention_dec' seems missed in dqrf/ops/functions/ms_deform_attn.py, would you please kindly add this function to the script?

Hi,
You can comment out SamplingAttention_dec, it is the vanilla deformable attention from https://github.com/fundamentalvision/Deformable-DETR

And also, we try to repeat this paper, unfortunately, we can't achieve the performance published in the paper with this open-sourced codes.

Hi,
As I said, you can remove SamplingAttention_dec in the transformer architecture. In configs you can use set RECTIFIED_ATTENTION to true.
Here are some results on COCO dataset for reference:

Model
AP

Baseline
46.2

RA
46.4

Baseline with 1000 queries
46.6

Baseline with DQ
46.3


跑不起来

os: Macos 11 Big Sur
flutter: 1.22.3
editor: vscode
../../../sdk/flutter/.pub-cache/hosted/mirrors.tuna.tsinghua.edu.cn%2547dart-pub/flutter_svg-0.19.2/lib/src/picture_provider.dart:58:53: Error: Method not found: 'Directionality.maybeOf'.
        textDirection: context != null ? Directionality.maybeOf(context) : null,
                                                        ^^^^^^^
    Command PhaseScriptExecution failed with a nonzero exit code
    note: Using new build system
    note: Building targets in parallel
    note: Planning build
    note: Constructing build description

Because gsy_github_app_flutter depends on flutter_svg >=0.19.2 which requires Flutter SDK version >=1.24.0-6.0.pre, version solving failed.
pub get failed (1; Because gsy_github_app_flutter depends on flutter_svg >=0.19.2 which requires Flutter SDK version >=1.24.0-6.0.pre, version solving failed.)

不应该啊，因为  0.19.2 的 flutter_svg  需要 Flutter SDK version >=1.24.0-6.0.pre ，我刚刚提交把 flutter_svg: 0.19.0 的 ^ 去掉，强制不升级，因为 flutter_svg-0.19.2 不支持现在的 release sdk


Rephrased the caution text

Let me know what you think.

I believe README.md needs a similar change.

@krancour https://github.com/akuity/kargo/pull/827

Thank you @wojtekidd !


[SPARK-42483][TESTS] Regenerate benchmark results

What changes were proposed in this pull request?
This aims to regenerate benchmark results on master branch as a baseline for Spark 3.5.0 and a way to comparing Apache Spark 3.4.0 branch.
Why are the changes needed?
These are reference values with minor changes.
- OpenJDK 64-Bit Server VM 1.8.0_352-b08 on Linux 5.15.0-1023-azure
+ OpenJDK 64-Bit Server VM 1.8.0_362-b09 on Linux 5.15.0-1031-azure

- OpenJDK 64-Bit Server VM 11.0.17+8 on Linux 5.15.0-1023-azure
+ OpenJDK 64-Bit Server VM 11.0.18+10 on Linux 5.15.0-1031-azure

- OpenJDK 64-Bit Server VM 17.0.5+8 on Linux 5.15.0-1023-azure
+ OpenJDK 64-Bit Server VM 17.0.6+10 on Linux 5.15.0-1031-azure

Does this PR introduce any user-facing change?
No.
How was this patch tested?
Manual review.

When you have some time, could you review this, @viirya ? I want to merge this to proceed the further investigations.

Thank you so much always for your help, @viirya !
Merged to master for Apache Spark 3.5.


Add Contributing.md

Our second top-level COMPUTER-YELLING.md document.
Whereas "GUIDE.md" is intended to inform readers about the material we write and our relation to that material, this document is intended to tell readers about the community around this project and their relation to it. As such this addresses various goals the reader might have, such as:

What to expect when interacting on the tracker,
How to change things,
How to join the team,
How to act as an editor,
How to merge changes.

Additionally we make it known that by participating in the tracker they are asserting that they are authorized to make contributions, and that they agree to the code of conduct. If they wish to join the team they are informed that they must be exemplars of the values of the community.
Thoughts on this? My specific questions are:

Are the notes around joining the team clear? The idea is to pair newcomers with existing editors as the project goes on. We're going to be lax about this in the beginning to get off the ground.
My intent with "one reviewer at a time" is to not overload folks with comments when they submit documentation and make the editing process more conversational. Is this overkill?

/cc @nodejs/diversity

I think it has been talked about that Rust's CoC is a little "weak". I'd suggest adopting whatever the Diversity WG adopts (should be soonish I think). Perhaps ok for the interim.

@Fishrock123 Yep, I heard that at Nodeconf as well — the plan is to replace to reflect the larger project's CoC as soon as that's in place. For the time being we're just inheriting what's there so we don't duplicate work.

LGTM

Merging this now. Thanks all! If there are questions / comments / things you'd like to change, open an issue / PR, or email me (chris at neversaw dot us) if you'd rather discuss them privately.


[InfluxDB] Rework the functions page

Specify which functions work with GROUP BY time() clauses
Using TOP() or BOTTOM() with a GROUP BY time() clause returns several points with the same timestamp:
> SELECT top(usage_idle,2) FROM cpu WHERE cpu='cpu-total' AND time >= '2016-10-04T19:30:00Z' and time <= '2016-10-04T19:38:00Z' GROUP BY time(3m)
name: cpu
---------
time                    top
2016-10-04T19:30:00Z    99.79959919839698
2016-10-04T19:30:00Z    99.69939879759518
2016-10-04T19:33:00Z    99.79959919839641
2016-10-04T19:33:00Z    99.70000000000027
2016-10-04T19:36:00Z    99.79959919839698
2016-10-04T19:36:00Z    99.59919839679338

Adding an INTO clause to that query works, but the first point per interval is overwritten by the second point:
> SELECT top(usage_idle,2) INTO "newmeas" FROM cpu WHERE cpu='cpu-total' AND time >= '2016-10-04T19:30:00Z' and time <= '2016-10-04T19:38:00Z'  GROUP BY time(3m)
name: result
------------
time                   written
1970-01-01T00:00:00Z   6

> select * from newmeas
name: newmeas
-------------
time                    top
2016-10-04T19:30:00Z    99.69939879759518
2016-10-04T19:33:00Z    99.70000000000027
2016-10-04T19:36:00Z    99.59919839679338

Props to @desa for catching this.
Regular expressions
Clarify that you can use regular expressions in the function to specify field keys.

Also check other functions.

Include LAST() behavior with multiple lasts in the SELECT clause: https://groups.google.com/forum/#!topic/influxdb/gVOL6Ws1N6s

Selector functions also allow you to specify a field key after the parentheses:
SELECT MAX(oh_dear),<field_key>

Remove "The timestamp reflects the first time the field value appears in the data" from the DISTINCT() description. That's wrong :(

Highlight behavior of wildcards with selectors: https://github.com/influxdata/docs.influxdata.com/issues/1095


There is no bitcoin.conf file to switch on RPC

Hello,
i've been trying forever to find this so called bitcoin.conf file. During the first startup of bitcoin core, I changed the default directory to an external hdd dedicated to bitcoin as the blockchain is so huge. This is where everything, the blockchain and my wallet is stored. There is no bitcoin.conf, or cookie file anywhere on it, tried searching whole drive, even my entire home directory on linux, just in case. The only file stored within my home directory, is in ~/.config/Bitcoin/Bitcoin-Qt.conf, which isn't the correct file for this case. I've tried looking through any commands within bitcoin-cli for use start RPC and can't get it working. When attempting to run electrum server, it just fails that it couldn't find said cookie file, I couldn't enter a user/pass as I didn't ever set one as I can't.
Know what the issue may be?
Im running Kubuntu 20.10, x64. Using offical release of Bitcoin Core from bitcoin.org and have verified its digital signatures. Latest release of Bitcoin Core as of writing.

I figured out that this file had to be manually created, with server=1 written in. Documention made it same to belive this file was there by default, didnt want to create it manually in case it could pose security risk if not done right.
Closing.


Fix #34 SecurityException in the example

When reading images in the example application from the External Storage.

Oh, right! I tested it only on the emulator. Thanks! :+1:


add ../storybook/**/*.exs to tailwind.config.js content path

Otherwise the classes you use in your storybooks are not picked up by tailwind.
I'm not sure this is great though, because it can increase the size of your overall css. But I'm also not sure if there is a good way to solve this?

There is nothing much to do in order to fix this issue: the content path that should be amended is not in the storybook's tailwind.config but in the application's config.
You can look at the phoenix_storybook_sample repository and its button story.
If you run it locally and update the class attribute, tailwind's JIT kicks in as needed.


How to modify field property TableRelation

Property TableRelation of field "No." in table Item needs to be extended with an extra condition.
So instead of ... if (Type = CONST (Item)) Item WHERE (Blocked = CONST (false),
"Sales Blocked" = CONST (false))
it needs to be … if (Type = CONST (Item)) Item WHERE (Blocked = CONST (false),
"Sales Blocked" = CONST (false),
"Standard Item" = CONST (true))
I've added boolean field "Standard Item" as a table extension on table Item.
How do I solve this issue?

In the spring release we will support adding additional conditional relations like
If(Type = Const(Resource)) Resource WHERE(Blah)
we don't currently have a way to modify an already existing line.

OK, Thanks for your answer.
We'll wait till spring.

The current design for spring won't solve your problem. You can add new lines, but since the lines are evaluated top down you will get the original line first without the additional Where(StandardItem = Const(true))
We will evaluate if we can change the design so it is possible to also support your scenario.

After the spring release, we can modify the table relation but the system does not recognize it.
Is it planned to break the top-down evaluation for table relations?

It seems like this should be reopened, if it didn't solve the problem and you still need to evaluate a larger fix.

Please file a BCIdea for requests for additional enhancements.


Remove "Restart" option from menu when app isn't running

I believe that the restart option is superfluous when the app isn't running. Start suffices, right?

Yup, you're right @ericronne! Thanks for noticing that 😄


Fix for addSource()

The css selector used to get the options by value were not quoted and therefore would break depending on the source.id.

@styfle
Awesome! Thanks for your contribution.


Cool Down on WayPoints?

Is it possible to add in a timer for how often a waypoint can be used by a user?
Awesome work btw!

Thanks!
I've no plan to add a cooldown for waypoint right now, there's plan to split off chatcommands features that doesn't directly correlate to RPGMods.
So that might need to happens first...


Feature Request: Allow to copy  logs in Epinio UI directly from screen

Is your feature request related to a problem?
Currently in Epinio UI, after deploying an app, if an user wants the logs he/she has to download them but can't copy directly from screen.

Solution you'd like
As an internal user noticed the screen where logs appear should allow to copy the text.
Alternatives you've considered
No response
Anything else?
No response

This will be fixed when the next merge from dashboard master goes in to epinio-dev

Linking issue that will resolve it https://github.com/epinio/ui/issues/203

Checked on latest epinio-dev and working:
https://github.com/epinio/ui/assets/37271841/d4b15745-e4a4-46ec-966c-7db186cd47b4
Check also copying from in-built shell is possible:
https://github.com/epinio/ui/assets/37271841/76acc027-31e6-438e-855c-c754d0b877f7

Rechecked on v1.9.0-rc3


Getting Started Issue with stocks invocation

Working within a docker container, I've used composer to pull polygon-io onto a PHP7.4 running apache with Debian 11 environment:
`php -v
PHP 7.4.29 (cli) (built: May 11 2022 14:19:20) ( NTS )
cat /etc/os-release
PRETTY_NAME="Debian GNU/Linux 11 (bullseye)"
NAME="Debian GNU/Linux"
VERSION_ID="11"
`
Composer JSON:
{     "require": {         "monolog/monolog": "2.5.*",         "polygon-io/api" : "*"     } }
When I try the default example for Rest:
require __DIR__ . '/vendor/autoload.php'; use PolygonIO\Rest\Rest; $rest = new Rest('<MyWorkingAPIKey>'); print_r($rest->forex->realTimeCurrencyConversion->get('USD', 'EUR', 10));
I get:
Warning: array_merge(): Expected parameter 2 to be an array, null given in /var/www/html/vendor/polygon-io/api/src/Rest/RestResource.php on line 63
Fatal error: Uncaught GuzzleHttp\Exception\ClientException: Client error: GET https://api.polygon.io/v1/conversion/USD/EUR resulted in a 401 Unauthorized response: {"status":"ERROR","request_id":"8ff237f073510360d8f36cab29e25062","error":"API Key was not provided"} in /var/www/html/vendor/guzzlehttp/guzzle/src/Exception/RequestException.php:113 Stack trace: #0 /var/www/html/vendor/guzzlehttp/guzzle/src/Middleware.php(69): GuzzleHttp\Exception\RequestException::create(Object(GuzzleHttp\Psr7\Request), Object(GuzzleHttp\Psr7\Response), NULL, Array, NULL) #1 /var/www/html/vendor/guzzlehttp/promises/src/Promise.php(204): GuzzleHttp\Middleware::GuzzleHttp{closure}(Object(GuzzleHttp\Psr7\Response)) #2 /var/www/html/vendor/guzzlehttp/promises/src/Promise.php(153): GuzzleHttp\Promise\Promise::callHandler(1, Object(GuzzleHttp\Psr7\Response), NULL) #3 /var/www/html/vendor/guzzlehttp/promises/src/TaskQueue.php(48): GuzzleHttp\Promise\Promise::GuzzleHttp\Promise{closure}() #4 /var/www/html/vendor/guzzlehttp/promises/src/Promise in /var/www/html/vendor/guzzlehttp/guzzle/src/Exception/RequestException.php on line 113
If I send an array as my third parameter:
print_r($rest->forex->realTimeCurrencyConversion->get('USD', 'EUR', [10]));
I get to this point:
Fatal error: Uncaught GuzzleHttp\Exception\ClientException: Client error: GET https://api.polygon.io/v1/conversion/USD/EUR?apiKey=qr5awPnGKxOCH1B6FZgFbj43RTzodzIF&amount=100&precision=2&0=10 resulted in a 403 Forbidden response: {"status":"NOT_AUTHORIZED","request_id":"60695e7c911dd93ffba6bda276415743","message":"You are not entitled to this data. (truncated...)
Which is probably fine, my focus is on stock info, so I tried to switch from forex to just a staple (easy) Rest API. I looked at the structure of the $rest variable and saw you have ->stocks->snapshotAllTickers, so I thought this would invoke that for me:
var_dump(json_encode($rest->stocks->snapshotAllTickers->get()));
And while it seems to be trying to process something, all I get back is:
Fatal error: Uncaught TypeError: Argument 1 passed to PolygonIO\Rest\Common\Mappers::snapshotQuote() must be of the type array, null given, called in /var/www/html/vendor/polygon-io/api/src/Rest/Common/Mappers.php on line 88 and defined in /var/www/html/vendor/polygon-io/api/src/Rest/Common/Mappers.php:150 Stack trace: #0 /var/www/html/vendor/polygon-io/api/src/Rest/Common/Mappers.php(88): PolygonIO\Rest\Common\Mappers::snapshotQuote(NULL) #1 /var/www/html/vendor/polygon-io/api/src/Rest/Stocks/SnapshotAllTickers.php(34): PolygonIO\Rest\Common\Mappers::snapshotTicker(Array) #2 [internal function]: PolygonIO\Rest\Stocks\SnapshotAllTickers->PolygonIO\Rest\Stocks{closure}(Array) #3 /var/www/html/vendor/polygon-io/api/src/Rest/Stocks/SnapshotAllTickers.php(36): array_map(Object(Closure), Array) #4 /var/www/html/vendor/polygon-io/api/src/Rest/RestResource.php(79): PolygonIO\Rest\Stocks\SnapshotAllTickers->mapper(Array) #5 /var/www/html/vendor/polygon-io/api/src/Rest/Stocks/SnapshotAllTickers.php(22): PolygonIO\Rest\RestResourc in /var/www/html/vendor/polygon-io/api/src/Rest/Common/Mappers.php on line 150
Now I haven't paid for quotes, so I'm wondering if this is a glitch where the lack of a quote response is messing up the PHP?
Anyway, this is my first rodeo with particular library, so if you have some clean usage examples on the stocks front, I'd appreciate it. I tried the websocket example too and it failed, but I'll wait to action anything on that until I can at least 'hello world'.
Cheers!

Making some progress, seems I was correct on the Quote issue for the Rest request, the quote isn't returned unless the API key is authorized (paid enough) for it, so you need to avoid NULL response in your mapping:
Line 88 of Mappers.php:
$snap['lastQuote'] = Mappers::snapshotQuote((is_array($snap['lastquote']) ? $snap['lastquote'] : array()));
Also, not sure why, but parameters are not being passed into your RestResource.php as an array so on line 64 I needed:
(is_array($params) ? $params: array($params) )
Which for your example REST request gets me past the PHP error and into the "Forbidden" error, which I think could be handled without a FATAL error, but maybe that's a limitation of guzzlehttp.
I'm looking at the websocket layer now.

Well, I got connected only to be immediately disconnected:
`use PolygonIO\Websockets\Websockets;
$client = new Websockets('stocks','');
$connection= $client->stocks->connect('AM.AAPL',
function($data) {
echo "Received data:";
var_dump(json_encode($data));
}
);`
At least this gets me this:
Received data:string(73) "[{"ev":"status","status":"connected","message":"Connected Successfully"}]"
Received data:string(74) "[{"ev":"status","status":"auth_failed","message":"authentication failed"}]"
But not sure why I'm getting rejected at step 2. My Stock Developer plan says it includes websockets, so I think that's about as far as I can take this without some assistance. Anyone monitoring this?
Cheers!

Finally got connected! When I directly instantiate the Websockets class I don't pass $topic into the constructor, and then it will connect. Also, because I'm not on "realtime" I had to change line 9 of WebsocketResource.php from:
public const SOCKET_URI = 'wss://socket.polygon.io:443';
To:
public const SOCKET_URI = 'wss://delayed.polygon.io:443';
So this is the working code:
use PolygonIO\Websockets\Websockets;
$client = new Websockets('MyWorkingAPIKey');
$connection= $client->stocks->connect('AM.AAPL',
function($data) {
echo "Received data:";
var_dump(json_encode($data));
//And then do cooler stuff
}
);
I've seen other posts on here about multi-threading and MQ, but we'll cross that road when we come to it:
Received data: string(73) "[{"ev":"status","status":"connected","message":"Connected Successfully"}]"
Received data: string(67) "[{"ev":"status","status":"auth_success","message":"authenticated"}]"
Received data: string(71) "[{"ev":"status","status":"success","message":"subscribed to: AM.AAPL"}]"
I'm hoping to see some actual data come across once it gets to USA trading hours, but I guess we'll just wait and see!


Split out build info into another module

7c40e2a3 ("build: Share core bpftrace code with test binary")
broke the libbpf build info b/c libbpftrace target wasn't given
the libbpf cmake vars.
This commit splits out the buildinfo such that the buildinfo is only
needed inside libbpftrace target. This is nicer b/c then we don't have
to pass cmake vars to multiple targets. It's also nicer b/c main.cpp
is less concerned with internal details like build flags.
Also note that this commit also defines LLVM_VERSION_* preprocessor
variables. I'm not sure where those variables were coming from (maybe
one of the llvm header files). It's probably better we define it
ourselves so things are more explicit.

Checklist

[ ] Language changes are updated in docs/reference_guide.md
[x] User-visible and non-trivial changes updated in CHANGELOG.md
[ ] The new behaviour is covered by tests

Gonna merge this tomorrow if there's no objections. The broken --info output is confusing

Looks ok, go for it:)


Plex not recognizing antennas

Hi,
I am using Plex Version <IP_ADDRESS>66 (Ubuntu, 64bit) and tvheadend 4.3-991~g9718e8e.
I started antennas using:
sudo docker run -p 5004:5004 -e ANTENNAS_URL=http://<IP_ADDRESS>:5004 -e TVHEADEND_URL=http://admin:password@<IP_ADDRESS>:9981 thejf/antennas
Status Page of antennas says "all system go". When adding the device to plex over web interface the device is not recognized, even when typing in <IP_ADDRESS>:5004.
Plex Logfile:
Jan 20, 2018 10:28:49.310 [0x7fd9c0bff700] DEBUG - DVR:Device: Discovering and refreshing devices.
Jan 20, 2018 10:28:49.311 [0x7fd9c0bff700] DEBUG - DVR:Grabber: HDHomerun discovered 0 compatible devices.
Jan 20, 2018 10:28:49.311 [0x7fd9c0bff700] DEBUG - DVR:Device: Testing grabber HDHomerun device device://tv.plex.grabbers.hdhomerun/12345670 at http://<IP_ADDRESS>:5004
Jan 20, 2018 10:28:49.311 [0x7fd9c0bff700] DEBUG - HTTP requesting GET http://<IP_ADDRESS>:5004/discover.json
Jan 20, 2018 10:28:49.312 [0x7fd9c0bff700] DEBUG - HTTP 200 response from GET http://<IP_ADDRESS>:5004/discover.json
Jan 20, 2018 10:28:49.313 [0x7fd9c0bff700] DEBUG - HTTP requesting GET http://<IP_ADDRESS>:5004/lineup_status.json
Jan 20, 2018 10:28:49.314 [0x7fd9c0bff700] DEBUG - HTTP 200 response from GET http://<IP_ADDRESS>:5004/lineup_status.json
Jan 20, 2018 10:28:49.319 [0x7fd9c0bff700] DEBUG - DVR:Device: Device was alive, refreshed device://tv.plex.grabbers.hdhomerun/12345670.
Jan 20, 2018 10:28:49.319 [0x7fd9c0bff700] DEBUG - DVR:Grabber: Mystery discovered 0 compatible devices.
Jan 20, 2018 10:28:56.102 [0x7fd9ae3fe700] DEBUG - DVR:Device: Testing grabber HDHomerun device device://tv.plex.grabbers.hdhomerun/12345670 at http://<IP_ADDRESS>:5004
Jan 20, 2018 10:28:56.107 [0x7fd9ae3fe700] DEBUG - DVR:Device: Device device://tv.plex.grabbers.hdhomerun/12345670 died
Jan 20, 2018 10:28:56.108 [0x7fd9ae3fe700] DEBUG - DVR:Grabber: Mystery discovered 0 compatible devices.

When I take a look on the network in chrome, I see the discover process of Plex. It returns a XML:
<?xml version="1.0" encoding="UTF-8"?>
<MediaContainer size="2">
<Device key="1" uuid="device://tv.plex.grabbers.tunerservice/dvb%23v4l%23%2fsys%2fdevices%2fpci0000%3a00%2f0000%3a00%3a01.0%2f0000%3a01%3a00.0%3a14f1%3a8880" uri="" protocol="livetv" status="alive" state="enabled" lastSeenAt="1516441352" make="Hauppauge" model="WinTV-quadHD" modelNumber="1607" source="1" sources="0,1,2" thumb="/:/resources/dvr/device-pcie-560.png" tuners="1">
<Setting id="transcodeDuringRecord" label="Convert video while recording" default="0" summary="[Experimental] This can save disk space and improve compatibility. Converting requires a fast CPU." type="int" value="0" hidden="0" advanced="0" group="" enumValues="0:Off|2:Transcode" />
</Device>
<Device key="2" uuid="device://tv.plex.grabbers.hdhomerun/12345670" uri="http://<IP_ADDRESS>:5004" protocol="livetv" status="dead" state="enabled" lastSeenAt="1516440529" canTranscode="1" deviceAuth="test1234" make="Silicondust" model="Antennas" modelNumber="HDTC-2US" source="1" sources="1" thumb="/:/resources/dvr/device-generic-560.png" tuners="6">
<Setting id="transcodeProfile" label="Transcoder Quality" default="heavy" summary="The quality of the transcoding used when recording. Selecting original format may result in very large files." type="text" value="heavy" hidden="0" advanced="0" group="" enumValues="none:Original format|heavy:Highest quality|mobile:High quality (30fps limit)|internet540:Medium quality (540p)|internet480:Medium quality (480p)|internet360:Low quality (360p)|internet240:Low quality (240p)" />
</Device>
</MediaContainer>

I don't know what to do. Everything seems fine for me :(

Plex will show the device as dead, but it will still stream. I'd love to know what Plex is looking for to show dead or alive in the device list, but not having a HDHomeRun is making it hard to reverse-engineer that.
Or is it a case of it not streaming at all (especially when you also have a HDHomeRun?)

Problem identified, and currently working on a fix.

Care to tell us what's causing it? :)

Actually, I haven't quite figured out the whole cause of the problem. I thought it was simply due to Plex using SSDP to discover, and Antennas not advertising with SSDP. So, I implemented SSDP on the server and it now advertises itself, but still it doesn't show up in Plex.
I now have a HDHomeRun and I'm looking at how it talks to Plex, trying to figure out what Plex wants for it to find Antennas and not think that it's dead, and I'm getting closer, but unfortunately, what I thought was the solution didn't seem to be the whole thing.

Little side note from the telly project: is your SSDP implementation responding to M-HEAD requests from Plex?
Mine isn’t and I’m suspecting that’s the reason the device is marked as dead.

@HazCod Hey! Sorry, just noticed this message. I believe it now shows up as alive with SSDP, and it seems you got yours working (and to find it automatically too :D ) so yay!

Works in latest release, assuming Antennas is hosted correctly. Will attempt to make it easier for Antennas to be corrected in the future, but closing this for now.


Troubleshooting doc link is broken

Please list the package(s) involved in the issue, and include the version you are using
N/A
Describe the bug

Steps to reproduce the behavior:

Visit this page: https://shopify.dev/docs/api/checkout-ui-extensions/2024-01#troubleshooting
Click the "Learn more" CTA
Observe that the link you're sent to is a 404.

Expected behavior

Not a 404.
Screenshots

Additional context

Issue is fixed.
Thanks for filing it @edhgoose.


How can I create a torrent from a file on my system

Using this library how can I create the torrent (and seed, as the creator will be first seeder) from a file in my system, and download that file in another system. Upon googling I found out that I need to make a SHA1 hash out of info dictionary of the torrent, and when I looked into the code of dtorrent_task, I found out it expects parameters like 'name', 'path', 'file' to properly make the dictionary (under the function parseTorrentFileContent).
Am I on the right track? Is it possible to create a torrent using this library? If not, then how could I do it in dart?

Technically you  can use parseTorrentFileContent to create a torrent model from a map that contains all the required info and then use _torrentModel2Bytebuffer (which unfortunately is private currently)to convert the model to a bytebuffer to be saved
note this is not tested, the parseTorrentFileContent is not intended for this use, we need a type safe wrapper around it for creating torrents, which is easy to implement if someone has the time

Ok got it, but a major issue i'm facing is that I'm unable to SHA1 hash the file I want to share through torrent


version of latest release is incorrect

Describe the bug
The version of the latest release is incorrect
To Reproduce
Steps to reproduce the behavior:

Download latest release rosetta-cli-0.7.2-linux-amd64.tar.gz
Extract it
Run ./rosetta-cli-0.7.2-linux-amd64 version
and you will getv0.6.7

Expected behavior
You should get of 0.7.2
Additional context

Hi @ramilexe
Thanks for raising this issue. #254 resolves this. So this should be fixed in next release.

This should be fixed with the new release


engine

i saw your new leorik engine, you said it is the successor of your minimalchess engine.
does it mean that your minimalchess engine is abandoned ?

It's not abandoned! But the goal with MinimalChess was to make something minimal and simple to understand for a normal C# programmer. It already has a few features here and there where I felt conflicted because it made the engine better but not simpler. So Leorik is where I'm currently focusing on because I personally want to try new features and make a strong, competitive engine. But it can't be minimal and simple at the same time.
I still plan to release a version 1.0 for MinimalChess eventually. It's currently more or less feature complete but not perfectly tuned and polished. I'm still learning. ;)


youtube-dl download all .ts files from m3u8 but doesn't merge them into a single file

I use the latest version of youtube-dl.
ffmpeg is installed and accessible in path.
I have a local .m3u8 file. And I am trying to download the video from it with this command:
youtube-dl --no-part -a video.m3u8
It downloads all .ts files from the m3u8 but doesn't merge them into a single file when it is done.
Am I missing something?

-a must provide path to a batch file not m3u8 playlist. It's not supposed to merge anything.


Use etcd v3.5.3 for Kubernetes 1.22+

What type of PR is this?
/kind bug
What this PR does / why we need it:
Deploy etcd v3.5.3 for clusters running Kubernetes 1.22 or newer. etcd v3.5.3 includes a fix for the data inconsistency issues announced by the etcd maintainers.
To upgrade etcd for an existing cluster, you need to force upgrade the cluster as described here.
More information about this etcd release can be found here:

https://github.com/etcd-io/etcd/issues/13894
https://github.com/etcd-io/etcd/releases/tag/v3.5.3

Does this PR introduce a user-facing change?:
Deploy etcd v3.5.3 for clusters running Kubernetes 1.22 or newer. etcd v3.5.3 includes a fix for the data inconsistency issues announced by the etcd maintainers: https://groups.google.com/a/kubernetes.io/g/dev/c/B7gJs88XtQc/m/rSgNOzV2BwAJ
To upgrade etcd for an existing cluster, you need to force upgrade the cluster as described here: https://docs.kubermatic.com/kubeone/v1.4/guides/etcd_corruption/#enabling-etcd-corruption-checks

/assign @kron4eg

/cherrypick release/v1.4

/cherrypick release/v1.3