Allow MultipartStreamBuilder subclass to manipulate data

Change property $data visibility from "private" to "protected".
Leaving more flexibilities for subclassing.

Q
A

Bug fix?
no

New feature?
yes

BC breaks?
no

Deprecations?
no

Related tickets

Documentation
This is one simple signature change for subclassing.

License
MIT

What's in this PR?
Allow subclass of MultipartStreamBuilder to manipulate $data property.
Why?
I am recently working on Range Request support
for my application. For a range request with multiple specified ranges, I need to return a "multipart/byteranges" response.
The signature of MultipartStreamBuilder::addResource forces me to add a field name ($name), which is totally non-sense for
my use case.
I tried to extend the MultipartStreamBuilder with my own class and have an extra method that can properly add
the streams without the unnecessary extra handling, but since the visibility of MultipartStreamBuilder::data is "private",
I cannot do that.
Example Usage

class MyBuilder extends MultipartStreamBuilder
{
    public function addStream($resource, array $options = [])
    {
        $options += ['headers' => []];
        $this->data[] = [
            'contents' => $this->createStream($resource),
            'headers' => $options['headers'],
        ];
    }
}

// If you added new features, show examples of how to use them here
// (remove this section if not a new feature)
$response = $resposneFactory->createResponse(416);

// Assuming I have to deal with a range request "Range: bytes=0-8192, 65536-131072"
$steam = (new MyBuilder($streamFactory))
  ->addPart(custom_splice_resource_func(fopen('/path/to/file1), 0, 8192), ['Content-Range', 'bytes 0-8192/8192'])
  ->addPart(custom_splice_resource_func(fopen('/path/to/file1), 65536, 131072), ['Content-Range', 'bytes 65536-131072/65536']);

// Emit the response to client
$emitter->emit($response->withBody($stream));

Checklist

[ ] Updated CHANGELOG.md to describe BC breaks / deprecations | new feature | bugfix
[ ] Documentation pull request created (if not simply a bugfix)

To Do

[ ] If the PR is not complete but you want to discuss the approach, list what remains to be done here

this sounds really cool and makes me wonder if it would make sense to add that functionality into this repository, if you are motivated to contribute it.
looking at how we put things into data in
https://github.com/php-http/multipart-stream-builder/pull/49/files#diff-28351bcab63ba2060821f4c2f4e0c12877a30426b7f6819c2faf72e6db1250c1R107 i wonder if instead it would make sense to have a protected method addData(stream $contents, array $headers, string $filename) and keep $data private? otherwise it is rather easy to mess things up.
hm, and it seems that the 'filename' in the $data is not used, only the one in the 'headers' seems relevant... so it might be just addData(stream $contents, array $headers)?

I am currently writing a multipart stream implementation from ground up to fulfill my own needs. My approach is quite different than yours so I don't think it is suitable as a PR.
Instead of building the entire stream content string, I opt to dynamically seek and read the underlying resources on StreamInterface::read. This approach, although heavier in code (thus probably slower), would be more memory efficient for large file uploads / downloads.
I am still in the middle of my process. I can share more with you later.

this sounds really cool and makes me wonder if it would make sense to add that functionality into this repository, if you are motivated to contribute it.
looking at how we put things into data in
https://github.com/php-http/multipart-stream-builder/pull/49/files#diff-28351bcab63ba2060821f4c2f4e0c12877a30426b7f6819c2faf72e6db1250c1R107 i wonder if instead it would make sense to have a protected method addData(stream $contents, array $headers, string $filename) and keep $data private? otherwise it is rather easy to mess things up.
hm, and it seems that the 'filename' in the $data is not used, only the one in the 'headers' seems relevant... so it might be just addData(stream $contents, array $headers)?

This approach makes sense to me. If you're OK, I can write a PR of it.

would be great if you can refactor it to have a clean, protected addData method. that would allow you to do the thing you want, right?
and i see, this different approach to a stream builder sounds interesting but can not replace the one here. in that case i think it is best to keep it separate. if you open source it, we'd be happy to link to it from the readme here, with the explanation that the php-http implementation does not support range requests, and people who need that should look at your implementation.

Rewritten into a public addData function. Please check.

can we have addResource call the addData method? though there is some handling on the thing we created with getStream to determine the "filename" from the uri. would it make sense to have that filename header determined that way in addData as well, if it is not set?

If the 'filename' key is not used at all elsewhere, can I omit it in MultipartStreamBuilder::data? Seem to me it's only used in MultipartStreamBuilder::prepareHeaders.

If the 'filename' key is not used at all elsewhere, can I omit it in MultipartStreamBuilder::data? Seem to me it's only used in MultipartStreamBuilder::prepareHeaders.

i agree, i can't see it being used anywhere once it goes into data. i guess its a leftover after a previous refactoring. and with data being private, we can be sure there is no BC break if we drop it from the $data.

Please see if 7323e85 (rewrite addResource with addData) is OK.

Updated. Please see if 0db0d5c works.

Updated. Please check.

i added a changelog entry in https://github.com/php-http/multipart-stream-builder/commit/9addfcf2409a8514c96ae5d53e6b6508a4270e0b
thanks a lot! can you rebase your other MR on master to get the latest changes in?


Update/module pattern

This PR updates/creates separate files for handling the following areas in THP:
-email sign up
-rep cards
-table handler
-url params
-video submit
This is a WIP towards modular components for our site. #213

the upload video doesn't seem to be working


Way to access all Formsy enabled children?

I'm trying to build a re-usable form component that looks something like this:
// MyForm:
<Formsy.Form>
  { this.props.children }
  <button>Save</button>
</Formsy.Form>

Then I can essentially re-use this form container over and over:
// SomeRandomForm:
<MyForm>
  <MyInput type='text' name='name' />
  <div className='foobar'>
      <MyInput type='text' name='pet' />
  </div>
</MyForm>

// it would be nice if we could access all the children form inputs from the parent container

I want to be able to re-use control elements like Save/Cancel buttons which show up for every single form.
Is there a way to access Formsy enabled children from a container form?

I think refs will help you out.

Hi there,
The next version should allow you to do this as it uses context instead of props. I will soon give some information on how to try out the new version which requires React 0.14. Stay tuned :-)

Awesome! Being able to iterate across all the form inputs would be really useful if we want to build a re-usable form container. There's no way right now for us to iterate across all the child inputs without knowing exactly which refs to use.

Check out this issue: https://github.com/christianalfoni/formsy-react/issues/158 :-)


block reddit plz

can you block reddit comments please sir
i am using a surface pro and using reddit on edge. I dunno how all the coding shtuff wrks.

I've applied a fix. Try updating the definitions on the options page and see if it looks okay now.


Keyboard Issue

23H2 22631.3810
offset 0x3690 changed to 0x36a8
Tested 0x36a8 works

23H2 22631.3810 offset 0x3690 changed to 0x36a8 Tested 0x36a8 works

how did you find that tho

23H2 22631.3810 offset 0x3690 changed to 0x36a8 Tested 0x36a8 works

Thanks, I'll take a look at it when i have time and then update the library if anything did change.

Looks like you're right, is 23H2 22631.3810 the start of this change?

Looks like you're right, is 23H2 22631.3810 the start of this change?

which file is that in ida

@Metick

Looks like you're right, is 23H2 22631.3810 the start of this change?

which file is that in ida

for the 2nd it's win32kbase.sys, 0x3110 is in win32ksgd.sys

Looks like you're right, is 23H2 22631.3810 the start of this change?

which file is that in ida

for the 2nd it's win32kbase.sys, 0x3110 is in win32ksgd.sys

0x3110 has never changed


Managing pointers in cpp target

At the moment my code looks something like this
class BGame extends BGroup {
	#if cpp
	public var world:Pointer<BtDiscreteDynamicsWorld>;
	#else
	public var world:BtDiscreteDynamicsWorld;
	#end
}

Luckily .value() works equivalently in js and cpp targets, but I still feel like these conditionals are annoying. I also suppose ammo.js has automatic garbage collection, but in cpp we need to call pointer.destroy(), right?
I think those are the only two things keeping the js and cpp versions not fully unified.

At this point I'm thinking either hxcpp manages pointers automatically, or if it doesn't maybe this will help: http://api.haxe.org/cpp/vm/Gc.html#setFinalizer

There's maybe other ways to do this, but this is the most obvious one to me:
#if js
typedef BTDiscreteDynamicsWorld = BtDiscreteDynamicsWorld;
#elseif cpp
typedef BTDiscreteDynamicsWorld = Pointer<BtDiscreteDynamicsWorld>;
#end

JS's create() and CPP's create() would both return a BTDiscreteDynamicsWorld, and in the user's classes fields would be defined as BTDiscreteDynamicsWorld too. Unless I'm overlooking something this would allow us to treat instances exactly the same in both targets, granted we're calling .value() first.
BTDiscreteDynamicsWorld instead of BtDiscreteDynamicsWorld is just an example, we would need a much more different typedef naming scheme to avoid confusion.

Even better:
typedef BulletDiscreteDynamicsWorld =
#if js
BtDiscreteDynamicsWorld;
#elseif cpp
Pointer<BtDiscreteDynamicsWorld>;
#end

Betterer?
typedef BulletDiscreteDynamicsWorld =
#if cpp
Pointer<
#end
BtDiscreteDynamicsWorld
#if cpp
>
#end
;

Hmm only way that stays fully compatible with FlashDevelop's autocomplete is
#if cpp
typedef BulletDiscreteDynamicsWorld = Pointer<BtDiscreteDynamicsWorld>;
#end
#if js
typedef BulletDiscreteDynamicsWorld = BtDiscreteDynamicsWorld;
#end

for (i in 0...9000) {ctrlc();ctrlv();};
Done! https://gist.github.com/Ohmnivore/081488802e5696746391

Hey,
Is it not the same like the thing already included?
https://github.com/luboslenco/haxebullet/blob/master/Sources/haxebullet/Bullet.hx#L1351
I went with 'Pointer' indication but it is easy to use your own typedefs as you do.
Ammo.js used to require calling destroy() too, but not sure if that is still the case. Ideally we get rid of that if possible:)

Oops. Didn't realize you had typedefs already.
I honestly have no idea what the .destroy() usage should be on cpp/js, i guess i'll figure it out when i find memory leaks.

Cleaning up, if someone needs to take care of .destroy() usage urgently feel free to open another issue.


False positive warning for foreach variable: "Multiple definitions for variable"

I think internal foreach variables can be "defined" multiple times as a block-scoped variable. Same applies for keys.

@Jigsik Multiple definitions for foreach are fixed in 1.0.2. But now I see it like used, but it is unused. I moved it to a new issue: https://github.com/nette-intellij/intellij-latte/issues/54


Adds support for dashes in tag names

Thanks for the work on this library! I was wondering if you would be open to expanding what slime will accept:
my-component text -> <my-component>text</my-component>
In my case, this would help me work with some angular directives that bind to specific DOM elements by tag. FWIW, ruby's slim supports this case.

Hi @tokafish!  I have no objections to this change.  @henrik, @lpil, or @Rakoth have any thoughts?

Looks good to me. 👍

:+1:

:+1:

Great, it sounds like we're all in favor of this change.  Thank you for the contribution @tokafish!

I'm going to release this as 0.12.1 unless there are any objections.

:shipit:

Done!  Thanks again @tokafish


epmdless

Description
I would rather not have to deal with epmd and  whether it is started automatically or not, or have people have to click another dialog on macOS that say "allow incoming connections".
The alternative is to use the epmdless (Hex) library to be able to cluster nodes locally without epmd.
Notes

I believe we can avoid the dialog when the "beam" process boots up if I notarize and sign the macOS exes, but epmd is usually installed via the system package manager and will not be signed. This is another reason to find an alternative to epmd.
Another alternative would be using a TCP server to communicate between the runtime nodes and the LSP node, but I'd rather not add another layer of complexity (explicit complexity, one can argue clustering is implicit complexity).

I'm not sure this us needed anymore


main: use slices.Concat

Description

main: use slices.Concat.

Test Plan

Unit tests.

Welcome to Codecov :tada:
Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.
Thanks for integrating Codecov - We've got you covered :open_umbrella:


LLAMA-8021 : [LLDEV-29794] - IP - Llama Technical Test – Putting

In/Out of Standby the AVR, audio Auto setting doesn’t really persist Additional Fix added
Reason for change: Invoke the SAD on ARC InitiationEvent Test Procedure: SEE JIRA
Risks: None.
Signed-off-by<EMAIL_ADDRESS><EMAIL_ADDRESS>

Closing older PRs. Pls rebase and create new PR on latest sprint/main branch if still required.


Create capstone project board

Delivers:
Project board
Link:
https://github.com/users/King-Benx/projects/3
This is the project kanban for the meals capstone project we will be doing
Members:

Dilsher
Ben

Link to project:
https://github.com/King-Benx/meal-capstone-project

Hi @King-Benx and @DilsherB ,
While you put in a lot of hard work in this project, unfortunately, I won't be able to proceed with the review of your code.

Invalid Code Review Request 🚫

You have submitted a Kanban board that doesn't give information about the number of people in their group.

Your Code Review Request will be marked as invalid in your Dashboard, but don't worry you can always try again! 💪
Cheers and Happy coding!👏👏👏
If you have any questions or need clarification on anything, feel free to leave a comment in the pull request thread and don't forget to tag me so I'll receive the notification.
Please note

To avoid confusion and delays, please do not open a new pull request for re-reviews. Use the same pull request submitted for the first review, unless otherwise instructed.

Invalid Code Review Request does not count towards the code reviews limit.

Hi Team,
🎉 Good job so far! You're making great progress on this project, but there are still a few areas that need attention before we can move on to the next step.

Highlights

Kanban setup ✔
Issues assigned ✔
Project created ✔

Required Changes 🛠

[ ] Well done on the Kanban board 💯 but since this is a team with 2 people, and all cards with Student C are to be removed, the number of cards on board should be equal to 15. Please refer to the template

[ ] Good job assigning the cards 👍🏽 but cards should be assigned to team members in a fair way, @King-Benx's assigned tasks equal to 2 points while @DilsherB's assigned tasks equal to 3 points (there's one card assigned to him that doesn't have any points). 🤔

Please review the comments under the review for specific changes that still need to be made.

Optional Suggestions 💡

While these comments with the [OPTIONAL] prefix won't prevent the approval of this pull request, I strongly recommend taking them into account as they can improve your code.

🚀 Keep pushing forward, you're almost there! If you have any questions or need clarification on anything, feel free to leave a comment in the pull request thread and don't forget to tag me so I'll receive the notification.
Please note

To avoid confusion and delays, please do not open a new pull request for re-reviews. Use the same pull request submitted for the first review, unless otherwise instructed.

As per the Code reviews limits policy, you have a limited number of reviews per project. If you feel that the code review was unfair, you can request a second opinion using this form.
Cheers and happy coding! 👏👏👏

**Hi Team
You did great. Your project is complete!  There is nothing else to say other than... it's time to start coding:shipit:
Congratulations! 🎉

Highlights
✔ Clean kanban board
Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
m05e5

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.**


chore(deps): update vite and rollup to latest

I update vite, rollup and other deps to latest.
test has spa error. because test vite config file use adapter: process.env.START_ADAPTER, i remove this option. test is ok.

#660

Can you elaborate a bit on what you did to make the spa test work? Where in the code

For me this actually passes even with the spa-test left as is.

I've made a reduced PR here #767 , that only handles Vite 4. I was a bit worried that bumping this many dependencies at a time could introduce issues, like node-resolve 15: https://github.com/solidjs/solid-start/commit/20de367addbba2fc3876572bcb746ec8efee77e2

Fix #713

This PR is a bit too big and has also fallen behind the main branch. @rtritto , if you make a new smaller PR that only has this small change you seek to fix 713, it would be a lot easier to review and merge it.

Yeah I appreciate this but I went through and did a bit myself. It seems like node-resolve may be fine now. We are going to try it. I've updated everything else.


Add CI/CD

This includes two GitHub Actions workflows:

CI: This will run Detekt and build the Android app on pull requests or pushes on main
CD: This will create a new GitHub release (containing the app bundle) when pushing a git tag vX.Y.Z.

This requires the following CI variables or else the pipeline will fail:

[ ] KEYSTORE_BASE64
[ ] SIGNING_PASSWORD

Looking good!


Commit changes within 2 projects; multiple lint-staged & git hooks

Commit changes within 2 projects; multiple lint-staged & git hooks
I've Backend & Frontend project configured on the same repository, let's say I've Backend with ESLint & Frontend with TSLint
Can I have somehow run both lint-staged configurations when I commit changes on both projects?

I can't see why not. I assume they are on different sub-paths and 🚫💩 lint-staged works with globs.
It triggers Frontend precommit but doesn't trigger Backend precommit. There's only 1 git hook ofc

Actually since 1st (Frontend) lint-staged precommit is successful, the 2nd one (Backend) doesn't trigger because 1st eslint precommit passed.
Any idea how can I fix this?

No. I don't know anything about your setup. Sounds like it's config issue

Okay after looking at the config, I think the problem is that you have 2 package.json files with separate lint-staged configs but you only have 1 pre-commit hook.
I'd suggest:

Move the lint-staged configuration to a separate .lintstagedrc file in the root of your project
Add both configs to it like this

{
  "{Frontend,Backend}/**/*.js": "eslint"
}

Point your pre-commit hook (sorry I could not see it how you add it in your configs) to run lint-staged only once for both configs

This might still make lint-staged confused about file paths so you have to experiment a bit. The setup you have is rather exotic :)

Can I close this one?

Testing it; sorry for taking it long. Give me some hours please

@okonet

Point your pre-commit hook (sorry I could not see it how you add it in your configs) to run lint-staged only once for both configs

I used husky for managing hooks; how can I make sure it only executes once?

@okonet
I've errors cuz one project uses eslint-plugin-redux-saga and the other doesn't how to read modules from it's right project folder? Or what am I missing?
Output
[14:54:22] Running tasks for Frontend/**/*.js [failed]
[14:54:22] → 🚫 eslint found some errors. Please fix them and try committing again.

Oops! Something went wrong! :(

ESLint couldn't find the plugin "eslint-plugin-redux-saga". This can happen for a couple different reasons:

1. If ESLint is installed globally, then make sure eslint-plugin-redux-saga is also installed globally. A globally-installed ESLint cannot find a locally-i
nstalled plugin.

2. If ESLint is installed locally, then it's likely that the plugin isn't installed correctly. Try reinstalling by running the following:

    npm i eslint-plugin-redux-saga@latest --save-dev

If you still can't figure out the problem, please stop by https://gitter.im/eslint/eslint to chat with the team.

25h25h
husky > pre-commit hook failed (add --no-verify to bypass)

What I've seen is that hook runs both package precommit scripts. Any idea how to avoid this?
Also lint-staged run linter for all files and not only staged files.

Did you try my suggestions?

Also lint-staged run linter for all files and not only staged files.

If there is a regression, please file a separate issue with a repo to reproduce.

@SuEric I think this may be helpful for you - okonet/lint-staged#how-to-use-lint-staged-in-a-multi-package-monorepo

Documentation has been added which addresses the question raised here, closing the issue.


[question] Need clarification on the purpose and performance benefits of GarbageCollection class

For the impl
I have several questions about the motivation and use cases for this class:
Could you provide examples of scenarios where this class can improves performance? compare against default Python GC?
To my understanding, during backward, activation cuda memory should be released timely when we run backward in computational graph, will the GarbageCollection affect how we release cuda memory?
What are the tradeoffs of disabling automatic GC (gc.disable())?

This was added in https://github.com/pytorch/torchtitan/pull/148. You can check the discussions there for more context.
cc: @lessw2020 @yifuwang -- The GC change we made seems worth better documentation.

In distributed training, Python's garbage collection (GC) can kick in at different times for different ranks, leading to slow communication and stragglers. To avoid this issue, we need to perform manual GC to ensure all ranks collect garbage simultaneously. Consider a scenario where we're using FSDP and Python triggers garbage collection for rank0 during the first allgather, rank1 during the second allgather, and so on. In this case, communication will be severely slowed down due to the straggling effect caused by asynchronous garbage collection.
While disabling Python GC might seem like an easy solution, it's not feasible due to potential circular dependencies in the code. We still rely on GC to recycle some Python objects. Disabling GC altogether could lead to CPU or GPU memory exhaustion.
So, performing manual GC is the best approach to resolve this issue.

hi @fegin  thank for your response. If we disable GC and run collection manually, will it fail to release CUDA memory in a timely manner? For example, during the backward pass, activations from later layers should ideally be released after backpropagation uses them for gradient computation in the previous layer as GC not collect zero reference count variables?

@yifuwang @fegin  thank you for explanation! looks good to me


New page about cockroach sqlfmt

Fixes #3473.

This change is

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/0c77573e8b37081eeeefc62d469179b9931d2f32/

Closing this because #4092 was merged


Important fix : throughputVsLatency is not handled on Tx

throughputVsLatency influence number of FPGA sample batch in .
On litlle Soc like Raspberry Pi, this help a lot to decrease the CPU.
After investigation there is a mistake here :
https://github.com/myriadrf/LimeSuite/blob/abf8214056f059764669ea9d7755edac8752a047/src/protocols/Streamer.cpp#L646
And should be replaced by
const uint8_t packetsToBatch = dataPort->CheckStreamSize(txBatchSize);
After this fix, higher SampleRate could be transmit on litlle SOC (around 8MS on LimeNetMicro)

Thanks for noticing this. Fixed 8efa0c5a94e5b1148ba9f88074bc90f30a0ecadd


ArgumentsTest fails under Windows

The given test uses *ux specifics and does not work on Windows

Fixed by #490


(bot) Update Helm release rook-ceph to v1.6.5

This PR contains the following updates:

Package
Update
Change

rook-ceph
patch
v1.6.2 -> v1.6.5

rook/ceph
patch
v1.6.2 -> v1.6.5

Release Notes

rook/rook
v1.6.5
Compare Source
Improvements
Rook v1.6.5 is a patch release limited in scope and focusing on small feature additions and bug fixes.
We are happy to announce the availability of a Helm chart to configure the CephCluster CR.
Please try it out and share feedback! We would like to declare it stable in v1.7.
Ceph

Experimental Helm chart for CephClusters (#7778, @henryzhangsta)
Disable insecure global id if no insecure clients are detected. If insecure clients are still required, see these instructions. (#7746, @travisn)
Enable host networking by default in the CSI driver due to issues with client IO hangs when the driver restarts (#8102, @Rakshith-R)
Add a disaster recovery guide for an accidentally deleted CephCluster CR (#8040, @travisn)
Do not fail prepareOSD job if devices are not passed (#8098, @degorenko)
Ensure MDS and RGW are upgraded anytime the ceph image changes (#8060, @sp98)
External cluster config enables v1 address type when enabling v2 (#8083, @aruniiird)
Create object pools in parallel for faster object store reconcile (#8082, @leseb)
Fix detection of delete event reconciliation (#8086, @BlaineEXE)
Use RGW admin API for s3 user management (#7998, @leseb)

v1.6.4
Compare Source
Improvements
Rook v1.6.4 is a patch release limited in scope and focusing on small feature additions and bug fixes.
Ceph

Support for separate tolerations and affinities for rbd and cephfs CSI drivers (#8006, @Rakshith-R)
Update ceph version to 15.2.13 (#8004, @leseb)
External cluster upgrades fix for CRD schema (#8042, @leseb)
Build with golang 1.16 instead of 1.15 (#7945, @subhamkrai)
Retry starting CSI drivers on initial failure (#8020, @Rakshith-R)
During uninstall stop monitoring rbd mirroring before cleanup (#8031, @leseb)
Update the backend path for RGW transit engine (#8008, @thotz)
If reducing mon count only remove one extra mon per health check (#8011, @travisn)
Parse radosgw-admin json properly for internal commands (#8000, @satoru-takeuchi)
Expand OSD PVCs only if the underlying storage class allow expansion (#8001, @iamniting)
Allow the operator log level to be changed dynamically (#7976, @subhamkrai)
Pin experimental volume replication to release-v0.1 branch (#7985, @Madhu-1)
Remove '--site-name' arg when creating bootstrap peer token (#7986, @leseb)
Do not configure external metric endpoint if not present (#7974, @leseb)
Helm chart to allow multiple filesystems (#7930, @SerialVelocity)
Rehydrate the bootstrap peer token secret on monitor changes (#7935, @leseb)

v1.6.3
Compare Source
Improvements
Rook v1.6.3 is a patch release limited in scope and focusing on small feature additions and bug fixes.
Ceph

Ensure correct devices are started for OSDs after node restart (#7951, @BlaineEXE)
Write reconcile results to events on the CephCluster CR (#7222, @iamniting)
Updated dashboard ingress example for networking v1 (#7933, @travisn)
Remove obsolete gateway type setting in object store CRD (#7919, @satoru-takeuchi)
Support specifying only public network or only cluster network or both (#7546, @rohan47)
Generate same operator deployment for OKD as OCP (#7898, @RyuunoAelia)
Ensure correct hostpath lock for OSD integrity (#7886, @satoru-takeuchi)
Improve resilience of mon failover if operator is restarted during failover (#7884, @travisn)
Disallow overriding the liveness probe handler function (#7889, @leseb)
Actively update the service endpoint for external mgr (#7875, @leseb)
Remove obsolete CSI statefulset template path vars from K8s 1.13 (#7877, @Rakshith-R)
Create crash collector pods after mon secret created (#7867, @subhamkrai)
OSD controller only updates PDBs during node drains instead of any OSD down event (#7726, @sp98)
Allow heap dump generation when logCollector sidecar is not running (#7847, @leseb)
Add nullable to object gateway settings (#7857, @travisn)

Configuration
📅 Schedule: At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about these updates again.

[ ] If you want to rebase/retry this PR, check this box.

This PR has been generated by Renovate Bot.

Renovate Ignore Notification
As this PR has been closed unmerged, Renovate will now ignore this update (v1.6.5). You will still receive a PR once a newer version is released, so if you wish to permanently ignore this dependency, please add it to the ignoreDeps array of your renovate config.
If this PR was closed by mistake or you changed your mind, you can simply rename this PR and you will soon get a fresh replacement PR opened.


Silver and Gold (GFG)(JOB-A-Thon)

Details
I would like to write code on Silver and Gold Problem which was asked in GeeksforGeeks Job-A-Thon: For Internship Contest
Problem statement:
There are N coins kept in a line. Each coin has two sides - one is colored gold and the other silver. You can flip two adjacent coins any number of times. You need to make the gold-colored side of every coin facing up. You are given the initial status of coins in a binary string s where 1 represents the gold side facing up and 0 represents the silver side facing up. If it is possible to make the gold-colored side of every coin facing up, return  Yes, otherwise return  No
Input:
N = 8, s = "11001100"
Output:
Yes
Explanation:
Flipping 3rd and 4th coin together and 7th
and 8th coin together will do the task.
Input:
N = 8, s = "10010100"
Output:
No
Explanation
It is not possible to make the gold
colored side of every coin facing up.
Programming language

[ ] C
[ ] C++
[x] Java
[ ] Python

I would like to solve this problem in Java language.
Please assign this problem to me under LGMSoC'21.

Please assign this issue to me under LGM-SoC'21
@ravikr126 @sameeksharl @Amit366 @Ayush7614


UserDto setId() mission

In Data module manager/impl AuthMangagerImpl you have saveUser(FirebaseUser user, Subscsriber subscriber)
There you create a new UserDto and call setId.
The only set Id i found was from Entity, am I doing something wrong, the code won't compile for me?
Should there be getters and setters for the Dto?

Hello @Mfrenchy77 !
Thanks for your question. I use Lombok library, and its annotation @Data. This annotation defines getters/setters, toString and other useful methods (See details on their site).
Android Studio IDE does not compile your code, because you need to install Lombok Plugin.
For this, go to Preferences -> Plugins -> Browse Repositories and search for "Lombok plugin".
After you successfully plugin install and Android Studio restart , everything should work fine, compilation errors related to "getters/setters" should not occur.


ember-cli-addon-docs

I like an idea to leave documentation concerns to another cool community tool.
https://ember-learn.github.io/ember-cli-addon-docs/
From the main page:

Show current and versioned guides, ideally whose content is verified by automated tests
Show current and versioned API documentation derived from structured comments in source code
Make it easy for contributors to correct documentation errors in addition to submitting code fixes

Sorry. Not this time.
According to https://github.com/ember-learn/ember-cli-addon-docs/issues/2 versioned docs is not implemented yet

YES! let's do it

To use Addon Docs, your addon must have a devDependency of Ember v2.8 or higher.
Note that your addon can still support older versions of Ember – it's just that you won't be able to run your Addon Docs documentation site against those older versions. (This means you also cannot write acceptance/application tests against your docs sites on older versions of Ember.)

Since this addon currently has an ember-lts-2.4 ember-try scenario, it's worth discussing before anyone attempts to implement. Either drop 2.4 or run no acceptance tests against the docs (which doesn't seem like a good idea).

Good finding.
I believe we are more coupled to the test-helpers rather than Ember.
We can keep running acceptance tests for all the Ember versions, as we do now, while testing of the guide examples against the supported(by addon-docs) versions of Ember.

I believe we are more coupled to the test-helpers rather than Ember.

The addon docs leverage the test's dummy app directly, which uses the addon's devDependency ember version.  Ember cli addon docs uses contextual components which locks you into 2.3+. If you look at this closed issue you can see the discussion.  But they set the explicit support at 2.8+.
During Ember try,  if we continue to have a 2.4 scenario, our acceptance test for 2.4 does not fit ember-cli-addon-docs's support matrix, which IMO is a risk worth discussing before implementing this feature.
As far as I see it, the reasonable options are:

Drop support for ember-lts-2.4
Some explicit filter scheme that excludes the docs + their tests from the ember 2.4 scenario, which I imagine would not be trivial to implement.
Hope that despite explicit support, the 2.4 tests will work just fine (which they did in a test I just ran)

Ember cli addon docs uses contextual components which locks you into 2.3+.

That's true only in case if you visit the page which uses contextual components.

Drop support for ember-lts-2.4

I hope, we wouldn't allow docs tool to dictate us support matrix limits 😆

Some explicit filter scheme that excludes the docs + their tests from the ember 2.4 scenario

I think we can simply mark tests related to addon-docs by some tag, e.g.:
module('[docs] Page 1', ....)

Then we just need to customize the ember-try scenario for<EMAIL_ADDRESS>to use QUnit filter, which excludes the docs tests:
  command: 'ember test --filter ![docs]',

This way we wouldn't run the tests which depends on addon-docs for<EMAIL_ADDRESS>scenario only.

These were just the options, not ranked in order of most reasonable ;)
I agree that 2 is the right way and seems easier than I presumed it might be. I've been wanting to play with the addon docs lib so whenever this next release with updated documentation happens, I'd be happy take a stab at it

That sounds exciting!

The other intriguing question to me is what are our options to host the new docs. Should we preserve the old guides with some clever redirect mechanism(for SEO) or is it possible to migrate the old guides content the new guides and preserve the URL structure.
I believe a working stab can unlock us to experiment with what addon-docs does allow us to achieve in this area..

Update on this http://www.ember-cli-mirage.com/ switched from old versioned jekyll docs to Ember-CLI-Addon-Docs and @samselikoff discussed some of the process on the recent EmberMap Podcast.
I know there were definitely hurdles to get over, but it is doable and I think would really help make the docs more unified with the Ember community of other addons and allow more people to contribute

I'll try to find sometime in the next week to start the process. Where's the best place to hear that podcast @rtablada?

@mistahenry https://embermap.com/podcast/mirage-meet-addon-docs

started here: https://github.com/mistahenry/ember-cli-page-object/tree/enhancement/addon-docs

The addon docs leverage the test's dummy app directly, which uses the addon's devDependency ember version. Ember cli addon docs uses contextual components which locks you into 2.3+. If you look at this closed issue you can see the discussion. But they set the explicit support at 2.8+.

I think it should be solved by https://github.com/ember-learn/ember-cli-addon-docs/pull/409


libxklavier: 5.3 -> 5.4

Motivation for this change
dependency of Xfce 4.13

Success on x86_64-linux (full log)
Attempted: libxklavier
Partial log (click to expand)
strip is /nix/store/4qvrxzxa535y8304mk195x50b6p9607d-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/4yjb65iv4i2a25m9icmf6v4r9v56f4fa-libxklavier-5.4/lib
patching script interpreter paths in /nix/store/4yjb65iv4i2a25m9icmf6v4r9v56f4fa-libxklavier-5.4
checking for references to /build in /nix/store/4yjb65iv4i2a25m9icmf6v4r9v56f4fa-libxklavier-5.4...
shrinking RPATHs of ELF executables and libraries in /nix/store/ky2gn13wji2sl4m0yq3xhfw1fabcqb41-libxklavier-5.4-dev
strip is /nix/store/4qvrxzxa535y8304mk195x50b6p9607d-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/ky2gn13wji2sl4m0yq3xhfw1fabcqb41-libxklavier-5.4-dev/lib
patching script interpreter paths in /nix/store/ky2gn13wji2sl4m0yq3xhfw1fabcqb41-libxklavier-5.4-dev
checking for references to /build in /nix/store/ky2gn13wji2sl4m0yq3xhfw1fabcqb41-libxklavier-5.4-dev...
/nix/store/4yjb65iv4i2a25m9icmf6v4r9v56f4fa-libxklavier-5.4

Success on aarch64-linux (full log)
Attempted: libxklavier
Partial log (click to expand)
strip is /nix/store/0pjsgkxz0rp5baycq5sp2s72lrr5q9sg-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/jks00cdyks1y6a75rviviv41kacyly1m-libxklavier-5.4/lib
patching script interpreter paths in /nix/store/jks00cdyks1y6a75rviviv41kacyly1m-libxklavier-5.4
checking for references to /build in /nix/store/jks00cdyks1y6a75rviviv41kacyly1m-libxklavier-5.4...
shrinking RPATHs of ELF executables and libraries in /nix/store/ahb1p6fy8rzl3pzymxfcgdif6gb12das-libxklavier-5.4-dev
strip is /nix/store/0pjsgkxz0rp5baycq5sp2s72lrr5q9sg-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/ahb1p6fy8rzl3pzymxfcgdif6gb12das-libxklavier-5.4-dev/lib
patching script interpreter paths in /nix/store/ahb1p6fy8rzl3pzymxfcgdif6gb12das-libxklavier-5.4-dev
checking for references to /build in /nix/store/ahb1p6fy8rzl3pzymxfcgdif6gb12das-libxklavier-5.4-dev...
/nix/store/jks00cdyks1y6a75rviviv41kacyly1m-libxklavier-5.4

included to https://github.com/NixOS/nixpkgs/pull/43427

Success on x86_64-linux (full log)
Attempted: libxklavier
Partial log (click to expand)
strip is /nix/store/4qvrxzxa535y8304mk195x50b6p9607d-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/4yjb65iv4i2a25m9icmf6v4r9v56f4fa-libxklavier-5.4/lib
patching script interpreter paths in /nix/store/4yjb65iv4i2a25m9icmf6v4r9v56f4fa-libxklavier-5.4
checking for references to /tmp/nix-build-libxklavier-5.4.drv-0 in /nix/store/4yjb65iv4i2a25m9icmf6v4r9v56f4fa-libxklavier-5.4...
shrinking RPATHs of ELF executables and libraries in /nix/store/ky2gn13wji2sl4m0yq3xhfw1fabcqb41-libxklavier-5.4-dev
strip is /nix/store/4qvrxzxa535y8304mk195x50b6p9607d-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/ky2gn13wji2sl4m0yq3xhfw1fabcqb41-libxklavier-5.4-dev/lib
patching script interpreter paths in /nix/store/ky2gn13wji2sl4m0yq3xhfw1fabcqb41-libxklavier-5.4-dev
checking for references to /tmp/nix-build-libxklavier-5.4.drv-0 in /nix/store/ky2gn13wji2sl4m0yq3xhfw1fabcqb41-libxklavier-5.4-dev...
/nix/store/4yjb65iv4i2a25m9icmf6v4r9v56f4fa-libxklavier-5.4

Success on aarch64-linux (full log)
Attempted: libxklavier
Partial log (click to expand)
strip is /nix/store/0pjsgkxz0rp5baycq5sp2s72lrr5q9sg-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/jks00cdyks1y6a75rviviv41kacyly1m-libxklavier-5.4/lib
patching script interpreter paths in /nix/store/jks00cdyks1y6a75rviviv41kacyly1m-libxklavier-5.4
checking for references to /build in /nix/store/jks00cdyks1y6a75rviviv41kacyly1m-libxklavier-5.4...
shrinking RPATHs of ELF executables and libraries in /nix/store/ahb1p6fy8rzl3pzymxfcgdif6gb12das-libxklavier-5.4-dev
strip is /nix/store/0pjsgkxz0rp5baycq5sp2s72lrr5q9sg-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/ahb1p6fy8rzl3pzymxfcgdif6gb12das-libxklavier-5.4-dev/lib
patching script interpreter paths in /nix/store/ahb1p6fy8rzl3pzymxfcgdif6gb12das-libxklavier-5.4-dev
checking for references to /build in /nix/store/ahb1p6fy8rzl3pzymxfcgdif6gb12das-libxklavier-5.4-dev...
/nix/store/jks00cdyks1y6a75rviviv41kacyly1m-libxklavier-5.4


[CHANGES] Add prism note to 2.61 release blog and CHANGES.md

Add the following note to the release blog and CHANGES.md

Use Prism as a fallback to the Python Portable runner when running a pipeline with the Python Direct runner #32876

(a similar change will be backup dated to the 2.61 release branch).

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI or the workflows README to see a list of phrases to trigger workflows.

R: @liferoad


Text null in  Select component options in Microsoft Edge & IE11

It's only for Select component with template. Here a screenshot:

Issue #278

Closing in favor of #278


Commits view on empty repository is broken

The commits view on an empty repository throws an error.
Cheers

Fixed in https://github.com/feenkcom/gt4git/commit/30828cbb5cdd21fcea10ecd9c7bc1a95a883769d


RFC Process Improvements

The RFC documents in our project have proven themselves to be a successful way to communicate designs for larger projects. When feedback is given, we're able to adjust and iterate.
However, there has been some friction.
Friction Points:

Marking RFCs as "reviewed" in one way or another from relevant stakeholders takes too long
Criteria for approving RFCs is outdated -- Originally it was set up in such a way that newer folks to the team would have someone who has more context to verify that all the context was considered. This needs to be revisited.
Criteria for landing RFCs has always been nebulous -- Should it land when it's approved? When it's implemented?
CI runs on RFC and documentation changes
TODO

Action Items:

[ ] Propose a new process - Aneesha
(use "request changes" / "approve" buttons)
[ ] Explore technical solutions to prevent friction (bots? ci fixes?)

Exit Criteria:

Change required reviewers to a relevant set of folks (tech leads?)
CI shouldn't run unrelated jobs (only markdown linting or something)
Institute culture of new process via shared in eng group (async or meeting)

@bkase we do skip some CI steps for RFCs via the script skip_if_only_frontend_or_rfcs.sh

Draft - https://www.notion.so/codaprotocol/Revisiting-the-RFC-Process-c556bea42cbe472a9985f1f97d628285


feat(core): default options configuration for primitive-textfield

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows Conventional Commits
[ ] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Refactoring
[ ] Code style update
[ ] Build or CI related changes
[ ] Documentation content changes

What is the current behavior?
Issue #133 is not fully implemented.
What is the new behavior?
Partial implementation of #133 for primitive-textfield component.
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Codecov Report

Merging #1249 (3f27416) into main (dfbc8a4) will increase coverage by 3.26%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #1249      +/-   ##
==========================================
+ Coverage   64.64%   67.91%   +3.26%
==========================================
  Files         738      584     -154
  Lines        8582     6963    -1619
  Branches     1727     1452     -275
==========================================
- Hits         5548     4729     -819
+ Misses       2616     1864     -752
+ Partials      418      370      -48

Flag
Coverage Δ

addon-charts
?

addon-doc
?

addon-editor
?

addon-mobile
?

addon-table
?

addon-tablebars
?

cdk
82.33% <ø> (ø)

core
68.64% <100.00%> (+0.09%)
:arrow_up:

kit
63.70% <ø> (ø)

summary
67.91% <100.00%> (+3.26%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...primitive-textfield/primitive-textfield-options.ts
100.00% <100.00%> (ø)

...imitive-textfield/primitive-textfield.component.ts
73.33% <100.00%> (+0.73%)
:arrow_up:

projects/core/constants/version.ts
100.00% <100.00%> (ø)

...jects/addon-charts/components/bar/bar.component.ts

...nts/table-pagination/table-pagination.component.ts

projects/addon-editor/utils/to-gradient.ts

...on-charts/components/bar-chart/bar-chart.module.ts

...charts/components/bar-chart/bar-chart.component.ts

...onents/table-pagination/table-pagination.module.ts

...ts/addon-doc/src/components/main/main.component.ts

... and 148 more

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 6c9963e...3f27416. Read the comment docs.


style: unstable change version

@machinly hello, @baerwang just changed a few configurations in this PR. However, the ci failed.

The reason for the failure is that it is need pika 3.5 image, I will push the image later, and will rerun of the ci workflow after completion .

@machinly hello, @baerwang just changed a few configurations in this PR. However, the ci failed.

The reason for the failure is that it need pika 3.5 image, I will push the image later, and will rerun of the ci workflow after completion .

done


IE Rendering Horizontal & Vertical Scrollbars on Cells

By default IE is rendering horizontal & vertical scrollbars on each cell even though there is nothing to scroll. I removed this by adding    -ms-overflow-style: -ms-autohiding-scrollbar;  style to the outer div.
This removed the scrollbars by default but now they still show when I hover over any cell.
How can I get rid of this?
I'm not entirely sure if it's an issue with this library but I figured I'd ask the community who've worked with this and hope someone has encountered this before.
Thanks!

@globehealer-developer, sorry for lately reply. I'm not sure the problem in you case and bootstrap should be work well on IE and other browser. So maybe there's some styling to cause this problem.


Deemphasized page toc color

Sets text color for page toc to gray.
Before

After

This reduced the visual weight of the page toc and reduces it's importance, which makes sense because this is secondary content.
Closes #20

cc @PaarthAgarwal You nerd sniped me with this one 😅

Sets text color for page toc to gray.
Before
After
This reduced the visual weight of the page toc and reduces it's importance, which makes sense because this is secondary content.
Closes #20

Excuse me I made a PR 2 min earlier 😌

cc @PaarthAgarwal You nerd sniped me with this one 😅

What did I do🤓


prefix not working properly?

I have a repository tagged with x/v1.0.0
Now when configuring semver-action like this:
  - name: Get Next Version
    id: semver
    uses<EMAIL_ADDRESS>    with:
      token: ${{ github.token }}
      branch: main
      noVersionBumpBehavior: current
      prefix: x/

I get this:
Run<EMAIL_ADDRESS>with:
token: ***
branch: main
noVersionBumpBehavior: current
prefix: x/
minorList: feat, feature
patchList: fix, bugfix, perf, refactor, test, tests
patchAll: false
skipInvalidTags: false
Comparing against latest tag: x/v1.0.0
/home/runner/work/_actions/ietf-tools/semver-action/v1.5.1/dist/index.js:4998
const error = new requestError.RequestError(toErrorMessage(data), status, {
^
RequestError [HttpError]: Not Found
at /home/runner/work/_actions/ietf-tools/semver-action/v1.5.1/dist/index.js:4998:21
at processTicksAndRejections (node:internal/process/task_queues:96:5)
at async main (/home/runner/work/_actions/ietf-tools/semver-action/v1.5.1/dist/index.js:31171:24) {
status: 404,
What am I doing wrong? If I omit the prefix i get "Latest tag is invalid (does not conform to semver)!"
So clearly the prefix does something...

I think what's happening is that when the URL for the Github API is generated, it already removed the prefix from the tag?
So if you have a tag myapp/1.0.0 and you specify a prefix of myapp/ then the API call to Github to get the changes looks for 1.0.0...main instead of myapp/1.0.0...main.

I ran into the same issue and @marcammann is right about the necessary change. I created a PR for this: https://github.com/ietf-tools/semver-action/pull/35.


Wwise plugin downloads not working

PD compiles with no warnings, no errors.
Other downloads work, just not any Wwise related ones.
I get:
"The requested URL was not found on the server. If you entered the URL manually please check your spelling and try again."
Plugin here:
https://enzienaudio.com/h/seanfwilson/DelayPrototype/

@seanfwilson sorry about this, currently our Wwise plugins don't support the [adc~] object as we're generating source plugins at the moment. FX plugins on the TODO!
Thanks for finding that bug, in the next release it should trigger an error when this happens.

Ah I see. Thanks for the quick reply!
Perhaps a little note near the head of the Wwise documentation could be useful for the moment too?
I noticed it's reference to source plugins, but was unsure if this was just for example purposes.

Yeah you're right apologies! It's coming soon anyway!


Readline missing from setup.py?

Trying to install this fresh on a Windows box today.
Traceback (most recent call last):
  File "C:\Users\Spencer\Apps\python\lib\runpy.py", line 197, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "C:\Users\Spencer\Apps\python\lib\runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "C:\Users\Spencer\Apps\python\Scripts\polyfile.exe\__main__.py", line 4, in <module>
  File "C:\Users\Spencer\Apps\python\lib\site-packages\polyfile\__init__.py", line 2, in <module>
    from .__main__ import main
  File "C:\Users\Spencer\Apps\python\lib\site-packages\polyfile\__main__.py", line 18, in <module>
    from .debugger import Debugger
  File "C:\Users\Spencer\Apps\python\lib\site-packages\polyfile\debugger.py", line 12, in <module>
    from .repl import ANSIColor, ANSIWriter, arg_completer, command, ExitREPL, log, REPL, SetCompleter
  File "C:\Users\Spencer\Apps\python\lib\site-packages\polyfile\repl.py", line 7, in <module>
    import readline
ModuleNotFoundError: No module named 'readline'

It looks like readline is a new requirement and isn't listed in the requirements: https://github.com/trailofbits/polyfile/blob/9c2d20b0013c670e36e8051cfbb3199f9f36d762/setup.py#L123-L134
Pip version: pip 21.3.1 from C:\Users\Spencer\Apps\python\lib\site-packages\pip (python 3.9)

Curious, readline has been bundled with Python since at least version 3.6, and the documentation doesn't mention any restrictions with OS 🤔. We haven't had any issues on macOS or Linux, so I suspect this is a Windows issue. I'm going to try adding support for Windows via https://github.com/pyreadline3/pyreadline3

PolyFile v0.4.1 should now support Windows. @ross-spencer are you able to confirm that this resolved your issues?


ytsc no longer visible in Firefox add-ons site

link no longer works: https://addons.mozilla.org/et/firefox/addon/youtube-comment-search/

Confirming this as well, thanks for posting. Went to search if there was a keyboard shortcut to close the search pane, then found the dead link as posted above. Also, I severely searched the Mozilla extension list for anything "youtube comment search" related and came up blank.

Fixed.


Versioning of *.csproj

Hi
I would like to adjust versionize to work for the workflow I use, before I send a Pull Request I want to discuss my suggestion. there is a "Directory.Build.props" file in the root directory. It contains the attributes VersionPrefix and VersionSuffix. I expect versionize to pick to increment the VersionPrefix.
Example of a the file:
<?xml version="1.0" encoding="utf-8"?>
<Project>
  <PropertyGroup>
    <VersionPrefix>1.0.0</VersionPrefix>
    <VersionSuffix>rc1</VersionSuffix>
    <Authors />
    <Product>Xy</Product>
    <RepositoryUrl />
    <Nullable>enable</Nullable>
  </PropertyGroup>
</Project>

My suggestion is simply to extend the Discover-method of Projects.cs to scan also for *.props. Furthermore, Project.cs has to be adjusted to pick and write to VersionPrefix, if it is present. Else use Version like it does now.
These changes make also other scenarios possible where one uses Version in the .props file or also using VersionPrefix int .csproj files.
Let me know what you think.
Marc

Is the use case that you want to release prerelease version such as rc1? The props file solution looks quite like a special use case to me.

Yes, this is the use case.
Well, I would not consider it as a special use case. The props file is the way for defining shared values in the new project-system. IMHO quite common among the folk who uses .NET Core.

I've a pre release functionality on my roadmap that should work somehow like this:

If the --pre-release switch was set the version increment would be calculated and a for example -rc1 is appended
In case the last version is a pre release the pre release version is incremented

Would that work for you?
The props files are standard but I want to avoid pulling to much msbuild functionality in the project.

Thanks for your answer. Unfortunatly, this isn't a possibility for us. Hence, I forked the project and will adjust it for our needs. Anyway, thank you for your work, I appreciate it!

Still interested in a PR in case the solution fits for more use cases. Ping me if you'd like to merge it back here


feat: empty rawValues in defaults

APL-137

Tested on dev cluster.
Observations:
After upgrade I see that all core app files got the _rawValues: {} entry.


Broken path for deploying coastlines

Defining the Path for the coastlines
NGL_PATH=${HOME}/.conda/envs/wrf_env/lib/python*/site-packages/ngl/ncarg/rangs
is still brocken.
The path variable is empty so the coastlines are downloaded in the shell script folder.
This needs to be fixed. The creation of the model output will will because the scripts cannot find its coastlines.

Fixed with #25.


Validation is executed on disabled fields (Angular)

Six Webcomponents version
4.0.3
Steps to reproduce
Simple example:
form.component.ts
testForm = this.fb.group({
    testInput: ['', [Validators.required, Validators.minLength(2)]],
  });

submitTestForm(event: SubmitEvent) {
    console.log('valid:', this.testForm.valid);
  }

form.component.html
<form [formGroup]="testForm" sixForm (sixSubmit)="submitTestForm($event)">
  <six-input label="test input" type="text" formControlName="testInput" required></six-input>
  <six-button submit>Submit Test</six-button>
</form>
<six-button (click)="testForm.controls.testInput.disabled ? testForm.controls.testInput.enable() : testForm.controls.testInput.disable()">Toggle</six-button>

returns in the console:
valid: false

What is expected?
Whenever a form control is disabled, the validation should not be executed.
What is actually happening?
Validation is executed regardless of state
System Info
No response
Any additional comments?
No response

This behavior is expected due to how Angular forms work.
If all form controls within a form are disabled, Angular considers the entire form as disabled, and no validation checks are performed and the valid field is set to false. It's important to note that a form can have one of several statuses: VALID, INVALID, PENDING, or DISABLED.
In the example provided above, the form is in the status DISABLED, if the input field is disabled.
Therefore, in your scenario, you should check the status of the form using this.testForm.status rather than relying on the valid property.

To clarify: The problem is not regarding the status of the FormGroup/FormControl, it is how the SIX web components handle the disabling of a FormControl/FormGroup and the following validation of those FormControls/FormGroups.
The ControlValueAccessors implemented by the SIX web components do not take into account if a control is disabled and show the error message regardless.
Angular handles this differently, if a control is disabled in Angular, it is extempt from validation and therefore doesn't show an error message: https://angular.io/api/forms/AbstractControl#disable
IMO we should handle this the same way to be in line with Angular validation. I will reopen this issue and provide a fix for this.


Update to RESTORE syntax in 21.1 / 21.2

Fixes DOC-2116
Syntax for RESTORE across the bulk docs needed updating to the RESTORE FROM 'x' in 'y' rather than the older RESTORE FROM 'y/x' syntax.
Plus a couple of other opportunistic edits (BACKUP syntax in places was not up-to-date, removed stacked notes at top of RESTORE doc into considerations, typo fix)

This change is

Sorry for the delay on this review! LGTM, with one nit spanning the versions.

Reviewable status: :shipit: complete! 0 of 0 LGTMs obtained (waiting on @kathancox, @livlobo, @msbutler, and @taroface)

v21.1/backup.md, line 218 at r3 (raw file):

To store the backup in a specific subdirectory in the storage location:

~~~ sql

Do you want to have a {% include_cached copy-clipboard.html %} here?
v21.1/backup.md, line 331 at r3 (raw file):

To store the backup in a specific subdirectory in the storage location:

~~~ sql

Ditto
v21.1/backup.md, line 446 at r3 (raw file):

To store the backup in a specific subdirectory in the storage location:

~~~ sql

Ditto
v21.2/backup.md, line 218 at r3 (raw file):

To store the backup in a specific subdirectory in the storage location:

~~~ sql

Ditto
Code quote:
~~~ sql
BACKUP DATABASE bank INTO 'subdirectory' IN 's3://{BUCKET NAME}/{PATH}?AWS_ACCESS_KEY_ID={KEY ID}&AWS_SECRET_ACCESS_KEY={SECRET ACCESS KEY}' \

v21.2/backup.md, line 350 at r3 (raw file):

To store the backup in a specific subdirectory in the storage location:

~~~ sql

Ditto
v21.2/backup.md, line 484 at r3 (raw file):

To store the backup in a specific subdirectory in the storage location:

~~~ sql

Ditto

Yes! I always forget this! TFTR Ryan!


Fix missing ln for B in step 5.2

Looks like there is a mistake plus one slight deviation from the paper, that can sometimes cause the algorithm to loop forever.

Thanks for the patch! I will cut a new release with this fix ASAP.


Should skip unique keys generation for local functions having same names with keyed composables

Environment
Any
Reproduction
https://stackblitz.com/edit/github-x5babp?file=app.vue
Describe the bug
The composable-keys plugin only detects imports, we should detect local variables too.
For example,
function useState() {}
useState()

This may not that useful if the function is defined in user's code. But when some libraries has such functions and got bundled, there's chances those functions become local and break something. (e.g. DatePicker component of @arco-design/web-vue, )
Additional context
I am willing to send a pr
Logs
No response

If you're happy to look at it, that would be great, thanks! ❤
We could add an additional property here specifying where the composable needs to be imported from, for example:
https://github.com/nuxt/nuxt/blob/b3a09a62c96a0c10559c7790d08e351b14cae9cb/packages/schema/src/config/build.ts#L147-L155

ok


rake task doesn't load .env.test

Steps to reproduce

I started a fresh new Rails application using:

rails new . --api --database=postgresql

I inserted dotenv-rails into the Gemfile:

group :development, :test do
  gem 'dotenv-rails'
end

I setup two environment file .env.development and .env.test with:

DATABASE_URL=postgres://val@localhost/myapp_dev

and
DATABASE_URL=postgres://val@localhost/myapp_test

respectively.

Run rake db:create.

Expected behavior
I am expecting rake db:create to load .env.test environment variable as pointed by the README.
Example:
Created database 'myapp_dev'
Created database 'myapp_test'

Actual behavior
But I get:
Created database 'myapp_dev'
ERROR:  database "myapp_dev" already exists
STATEMENT:  CREATE DATABASE "myapp_dev" ENCODING = 'unicode'
Database 'myapp_dev' already exists

As you can see, instead of creating the development and test database, it attempts to create the development database twice. Clearly,  dotenv is ignoring the .env.test.
System configuration
dotenv version:
2.2.1
Rails version:
5.1.0
Ruby version:
2.4.1

I think the problem is that rails won't switch environments between creating the databases.. this whole command is running in the development environment.
So to solve your issue you could use a .env file:
MYAPP_DEV_DATABASE=myapp_dev
MYAPP_TEST_DATABASE=myapp_test

Yes, that is exactly how I did it. That's a shame, really :( We could have cleaner approach, by having the same environment key everywhere and just stubbing inside the environment which would drastically minimize the if Rails.env.test?...

Is there really no solution to this?
Here's what's happening basically: https://github.com/rails/rails/blob/f507085fa2a7625324e6f4e3ecef9b27dabefb4f/activerecord/lib/active_record/tasks/database_tasks.rb#L339
One approach I could see is monkey patching DatabaseConfig.configs_for to reload envs. That's obviously bad but a hack in your Rakefile might still be better than having polluted production code. Any other ideas?
I'd argue to keep this issue open / mention it somewhere because it's causing headaches very often and this reference is hard to find.

I created a gem that patches ActiveRecord to fix this issue: dotenv_rails_db_tasks_fix
Also reported it to Rails: https://github.com/rails/rails/issues/32926

..and wrote a blogpost detailing some of the possible solutions<EMAIL_ADDRESS>


[4.8.0] Postgres coroutine client

Merge https://github.com/openswoole/ext-postgresql into https://github.com/openswoole/swoole-src
You can enable the native postgres coroutines client support by adding flag --with-postgres[=DIR] when compiling.

Related PR: https://github.com/openswoole/swoole-src/pull/57

Available in v4.8.0


Pass an empty vm_name for VMDK update operation

The launch_from_s3 method is now expecting a vm_name to be always
provided. For VMDK updates, this can be set to None.

LGTM


Multi-runners

Map/filter support running in parallel

/lgtm
/approve

Codecov Report

Merging #882 (792ed14) into main (0b5b4c0) will increase coverage by 0.42%.
The diff coverage is 77.77%.

@@            Coverage Diff             @@
##             main     #882      +/-   ##
==========================================
+ Coverage   69.46%   69.89%   +0.42%
==========================================
  Files         192      201       +9
  Lines        8473     8952     +479
  Branches     1390     1484      +94
==========================================
+ Hits         5886     6257     +371
- Misses       2180     2270      +90
- Partials      407      425      +18

Impacted Files
Coverage Δ

towhee/dag/operator_repr.py
93.33% <71.42%> (-4.11%)
:arrow_down:

towhee/engine/operator_context.py
84.78% <100.00%> (ø)

towhee/functional/mixins/computer_vision.py
37.50% <0.00%> (-9.56%)
:arrow_down:

towhee/functional/mixins/dispatcher.py
78.57% <0.00%> (-6.43%)
:arrow_down:

towhee/operator/base.py
73.68% <0.00%> (-0.97%)
:arrow_down:

towhee/hub/repo_manager.py
80.18% <0.00%> (-0.91%)
:arrow_down:

towhee/types/__init__.py
100.00% <0.00%> (ø)

towhee/functional/mixins/dataset.py
0.00% <0.00%> (ø)

towhee/functional/mixins/__init__.py
100.00% <0.00%> (ø)

towhee/engine/factory.py
87.30% <0.00%> (ø)

... and 14 more

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 0b5b4c0...792ed14. Read the comment docs.


add darker Antergos icon

Hi , here a darker Antergos icon which I think fit better to the theme .

Thanks! Could you rename the old one os_antergos_thin.png, just to keep it for anyone who may want it :)

done  :)


Section headers don't stick properly when section data is changed

https://user-images.githubusercontent.com/98796522/174580517-0f017b94-1050-4712-bac1-c782dadb89ec.MP4
Description
When we change the sections data provided to a sectionList component, the expected behaviour is for the sectionList component to recalculate and update the sticking positions of the section headers.
But, when section data is changed, the section headers don't work properly.
Version
0.64.3
Output of npx react-native info
info Fetching system and libraries information...
System:
OS: macOS 12.2.1
CPU: (10) arm64 Apple M1 Pro
Memory: 77.31 MB / 16.00 GB
Shell: 5.8 - /bin/zsh
Binaries:
Node: 14.19.1 - ~/.nodenv/versions/14.19.1/bin/node
Yarn: 1.22.18 - ~/.yarn/bin/yarn
npm: 6.14.16 - ~/.nodenv/versions/14.19.1/bin/npm
Watchman: 2<IP_ADDRESS> - /opt/homebrew/bin/watchman
Managers:
CocoaPods: 1.11.3 - /Users/adithya_v/.rbenv/shims/pod
SDKs:
iOS SDK:
Platforms: DriverKit 21.2, iOS 15.2, macOS 12.1, tvOS 15.2, watchOS 8.3
Android SDK: Not Found
IDEs:
Android Studio: 2021.1 AI-211.7628.21.2111.8092744
Xcode: 13.2.1/13C100 - /usr/bin/xcodebuild
Languages:
Java: 11.0.11 - /usr/bin/javac
npmPackages:
@react-native-community/cli: Not Found
react: 17.0.1 => 17.0.1
react-native: 0.64.3 => 0.64.3
react-native-macos: Not Found
npmGlobalPackages:
react-native: Not Found
Steps to reproduce

Create a sectionList with sticky section headers
The sections provided to the sectionList could have variable number of items
From the sections provided to the sectionList, remove 1 or 2 sections. ["section1", "section2","section3"] => ["section1", "section3"]
Section headers will disappear, and will reappear if we scroll up or down

Reproduced on  Pixel6 Pro,
iPhone8
Snack, code example, screenshot, or link to a repository
https://snack.expo.dev/@adithyamenu/bad-ramen

Hello @adithyamenu, if I understand you correctly, you expect the List to automatically scroll to some section position after its content is changed. That's not how it actually behaves. The List tries to preserve its content offset in this case.
You can try to achieve desired behaviour using this method https://reactnative.dev/docs/sectionlist#scrolltolocation.

@dy93
Thanks for your reply.
Actually the issue is something else.
The section list's headers are not sticky anymore after the list data is updated. The headers aren't always shown at the top of the list.
Give it a try, I have attached a link to the expo snack demonstrating this bug.

@adithyamenu thank you for the clarification!

@adithyamenu here is what I have in Expo: https://user-images.githubusercontent.com/14206200/174579835-8ffe0ef5-294a-4075-b635-950bea3efa2f.mov
The expected behavior is that Section 3 header is still visible on top of the list after I press Toggle at 00:18. Is that correct?

@adithyamenu great, thank you!

Hi @dmytrorykun @adithyamenu , I'd like to contribute on this issue if it still happens.
I'm not able to reproduce the bug at least on the expo example and rn tester app.

I have the same issue, when i have a lot of items, and scrolled down for a long time and list updates, the header wasn't sticky for a while

Faced the same issue, when working with a list that contains a lot of items. If the list-data was updated, the header looses it's sticky ability.

Sticky incorrect headers when processing lists containing a large number of items


Release metadata checker patches

Description
This was branched off v.next and the PR is to resolve the metadata checker errors to merge v.next to main for the ArcGIS Runtime 100.15.0 release

@TADraeseke Few metadata checker bugs that will need to be fixed. It's up to you if this should be resolved prior to the release or not:

The letter case in the word "Arcade" is not considered acceptable for the sample title, but we will be using the letter case for the title.
Removal of the following samples is causing an error with the metadata checker:

java/feature-layer-feature-service
java/feature-layer-geodatabase
java/feature-layer-geopackage
java/feature-layer-shapefile
java/generate-geodatabase
kotlin/feature-layer-feature-service
kotlin/feature-layer-geodatabase
kotlin/generate-geodatabase


Deprecate waitForPromise and waitForCondition plugin APIs.

Adding Blocking Proxy opens the door to greatly simplifying Protractor by no longer needing to wrap the WebDriver API. However, as long as plugins support the waitForPromise and waitForCondition hooks, we'll need to continue to wrap WebDriver.
We could replace these with a hook that adds a client-side script for Blocking Proxy to wait on. This is a breaking change, since currently plugins run waitForPromise and waitForCondition in the Protractor node process, not in the browser under test.
The only publicly available plugin I can find using these hooks is protractor-testability-plugin, which could easily switch to a client-side script hook. The same also applies to other, less-public plugins. When people implement waitForPromise, it seems that they're usually just doing return browser.executeAsyncScript(, more or less.

We don't really support hybrid mobile apps yet (i.e. native apps with webviews), but we might someday, and if we do I could easily imagine someone wanting to add a wait that can't be replicated with a client side script.  So I'd prefer to leave the hook as is.
I'm still fine with deprecating these hooks and asking people to change over, but let's not actually delete them just yet, since we may need them again in the future.


Needs actual documentation

This nuget package doesn't work out of the box, and only seems to install dll's.
The elmah.mvc package does a number of things. Either make the behavior of the two match, or actually write some docs, because it doesn't work the way you describe.

Happy to accept a pull request.


[Package Request]: Speakeasy CLI

How can we help?
I would like help so I can submit the manifest.
Please read and ensure the following

[x] The installer meets the above requirements

Please provide the following information
Download Page Url: https://github.com/speakeasy-api/speakeasy/releases/tag/v1.451.5
Publisher: Speakeasy Inc
Package Name: speakeasy.cli
Description: This CLI is a tool for interacting with the Speakeasy platform - the CLI brings the functionality of Speakeasy into your development workflow. It can be run locally or in your CI/CD pipeline to validate your API specs, generate SDKs and more.
  Current functions of the CLI include:

  Generating idiomatic client SDKs from OpenAPI3.X specs:
  Live: Go, Python3, Typescript(Node), Java, PHP, Ruby, Terraform
  Validating the correctness of OpenAPI3 specs. The CLI has a built in command to validate your spec and post helpful error messages.
  Authenticating with the platform and managing API keys.

Package Version: 1.451.5
Installer Url: https://github.com/speakeasy-api/speakeasy/releases/download/v1.451.5/speakeasy_windows_amd64.zip

Updated PR https://github.com/microsoft/winget-pkgs/pull/196383

Updated PR #196383

Hello @TristanSpeakEasy 👋🏼, glad to see you resolve this issue on your own ❤.

[!TIP]
Modify your very first comment as follows. This can link the pull request to this issue. When PR is merged, this issue can be closed as completed automatically. It can also prevent other contributors from confusing the issue state.

  Checklist for Pull Requests

  * [x]  Have you signed the [Contributor License Agreement](https://cla.opensource.microsoft.com/microsoft/winget-pkgs)?
- * [x]  Is there a linked Issue?
+ * [x]  Resolve #196381

Great thank you done


list commits included in a request

There should be an option to list all of a request's commits, including their SHAs and commit messages. Maybe change 'git review show x --full' to work like that and move the diff to a separate command 'git review show x --diff'.

Already in there for quite some time now.


fix: docs - docosaurus on gh-pages

Fixed gh actions to deploy to gh pages. Maybe need to adjust some settings in this repo, but it works on my fork:

https://github.com/maxhr/near-api-js/actions/runs/2945068582
https://maxhr.github.io/near-api-js/

Made it work with docosaurus to match docs.near.org look & feel and for advanced UI
Upgraded TypeDoc version to support the TypeScript upgrade
Fixed all internal typedoc links.

@morgsmccauley thanks for the review! I disagree because:

It's not a releasable npm package
The developer never has run any scripts from it, only the CI does
Soon it'll also contain documentation about the overlall monorepo, not only the individual lerna packages

So i'll rename the dir to docs, it makes more sense.

@morgsmccauley now when I think about it, we should extract the docs into a separate repo. Let's release it as is now, and I'll attend to it next.

@morgsmccauley can you re-review this please?

@maxhr just curious - why should this live in a separate repo? Wouldn't we want builds to trigger off this repo, so it would make sense to keep it here?


⚠ Bofrost.at has degraded performance

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

HTTP code: 200
Response time: 1536 ms

Resolved: Bofrost.at performance has improved in 0535995 after 18 minutes.


[monitoring] segment payload size larger than supported

The content server is exhibiting a warning related to tracking of events.
Your message must be < 32kb. This is currently surfaced as a warning to allow clients to update. Versions released after August 1, 2018 will throw an error instead. Please update your code before then. {
 userId: '0xdc13378dafca7fe2306368a16bcfac38c80bfcad',
  event: 'Catalyst Content Upload',
  properties:
   { server: '2b235332-2e10-4d07-9966-b95efb6146ec',
     type: 'scene',
     cid: 'QmUNpSvuqLVXfRgSsXXSKeRsixu8SnK7tmEimJo9ncQJTg',
     pointers:

Hi @abarmat, I've just closed this issue as https://github.com/decentraland/catalyst/pull/229 was merged.
The payload from the report information sent to Segment was bigger than 32kb, so that was triggering this error.
The solution for this issue was reducing the payload sent to Segment by removing the pointers and file information from it, as those items were arrays they were responsible for increasing the size of the request when an entity hasd many pointers and/or many fields.


Segfault when running test on macOS

Possibly related to issue #5657 (napari repo).
Tests fail with seg fault when using macOS both in tox and locally.

seems resolved


The variables "aks_podSubnet", "aks_podsubnet" differ only in casing

\infrastructure\AKS-Construction\bicep\network.bicep(173,5) : Error BCP353: The variables "aks_podSubnet", "aks_podsubnet" differ only in casing. The ARM deployments engine is not case sensitive and will not be able to distinguish between them.
: Error BCP353: The variables "aks_podSubnet", "aks_podsubnet" differ only in casing. The ARM deployments engine is not case sensitive and will not be able to distinguish between them.
\AKS-Construction\bicep\network.bicep(409,9) : Warning BCP334: The provided value can have a length as small as 0 and may be too short to assign to a target with a configured minimum length of 3.
\main.bicep(94,12) : Error BCP104: The referenced module has errors.
\AKS-Construction\bicep\main.bicep(130,16) : Error BCP104: The referenced module has errors.
Exception reported when upgraded AZ BICEP.
Version: Bicep CLI version 0.21.1 (d4acbd2a9f)

@luticm , fixed on PR #30


Uncaught Exception in index.js, line 314

In case the redis instance is down while a reconnect is attempted, the line
self.stream.connect(self.port, self.host);
will throw an exception (ECONNREFUSED). It would be cool to either swallow this exception or use the existing error handling facility.

@benfleis this is swallowed in newer version by using the retry_strategy option.


Fatal error: Call to undefined method

I have just downloaded this module and I installed it.
But I can work with it because of this fatal error in the screenshot.
I actually use Zend 2.4.9.
What have I missed please?
Here is my code
use SphinxSearch\Search;
use SphinxSearch\Db\Sql\Predicate\Match;
$search = new Search($adapter);
$rowset = $search->search('foo', new Match('?', 'ipsum dolor'));
echo 'Founds row:' . PHP_EOL;
foreach ($rowset as $row) {
echo $row['id'] . PHP_EOL;
}

Which PHP version are you using?

I use PHP 5.6

It seems an issue related to the specific zend-db version, we can't reproduce currently.
We need further investigation.
Could you try your example with another version?
Or could you provide us a full reproducible example of the issue (i.e. source code, composer.json etc..), please?
Thanks in advance.

It seems this processWithinGroupOrder() method has been removed from SelectDecorator class in Zend Framework 2.4.9
I can't find it in SelectDecorator nor Select and AbstractPreparableSql and AbstractSql classeses.
However I found this processWithinGroupOrder()  method in SphinxSearch\Db\Sql\Select.
Here is my sample code
`<?php
namespace AppSearch\Controller;
use App\Controller\AppAbstractActionController;
use SphinxSearch\Search;
use SphinxSearch\Db\Sql\Predicate\Match;
class IndexController extends AppAbstractActionController {
public function indexAction() {
    $isIndexPage = false;

    $search = new Search($this->getDbAdapter());
    $rowset = $search->search('doopins', new Match('?', 'ipsum dolor'));

    echo 'Founds row:' . PHP_EOL;
    foreach ($rowset as $row) {
        echo $row['id'] . PHP_EOL;
    }

    \Zend\Debug\Debug::dump($rowset); exit;

    return [
        'isIndexPage' => $isIndexPage
    ];
}

}`

Could you attach your composer.json too?
I suppose you're using the zend-db 2.4.8 (because 2.4.9 is the zend framework version, but the zend-db 2.4.9 does not exist).
Actually, process* methods are called by zend-db itself (composing the method name in [Zend\Db\Sql\AbstractSql](https://github.com/zendframework/zend-db/blob/b78b12c68bdafffaecb87f684426ad446b95204f/src/Sql/AbstractSql.php#L63(), so I'm not sure if the issue is within the zend-db itself OR is an incompatibility with our library.

Sure!
The attached file is the composer.json file renamed to composer.txt
composer.txt

However if you need more infos, you can have them here on Github in Zend framework repository.
Due to a bug in Version 2.2.3 in the past I have downloaded manually the 2.4.9 release zip file and replace my old library folder.
So you can go on ZF2 repository the release 2.4.9 to better explore Classes

This is also an issue for me..

I closed the issue because I had thought it was resolved, sorry.
Could you provide us more detail?
I need some some code to reproduce  the issue (provide composer.json and composer.lock too, please), then I'll work on it ASAP.

No problems mate,
This is the received error:
http://puu.sh/w7xPB/dacc6e7e92.png
Attached is the files requested
composer.txt
composer_lock.txt
The following code is used:
$this->getSphinx()->search('upload', new Match('?', '123'));
public function getSphinx()     {         if (!$this->sphinx) {             $sm = $this->getServiceLocator()->get('Config');             $adapter = new \Zend\Db\Adapter\Adapter($sm["sphinxql"]);             $this->sphinx = new Search($adapter);         }         return $this->sphinx;     }
Apologises for the weird formatting that github places on the code

@DSCmattb I see you used \Zend\Db\Adapter\Adapter directly, but this library cannot work in that way.
This library requires a custom configuration for Zend\Db\Adapter\Adapater that's provided by the included adapter service factory (config documentation here).
Could you try your code with our service factory instead of $adapter = new \Zend\Db\Adapter\Adapter($sm["sphinxql"]) and then let me a feedback?
Thanks in advance!

Can agree, defining it as the documentation works perfectly, sorry for that 👍

@DSCmattb no problem, may be we need to improve docs in order to make the service factory requirement more clear 👍
I'll do that prior to close this issue.
Finally, I'd like to ask to @Doopin if the issue can be considered solved, isn't it?

@leogr we can close this issue.
Due to a deadline for my application, I had to switch to another search engine solution and it doesn't require this module anymore.
I'm sorry but I can't test any of above solutions since sphinx has been removed from my workspace.
Thank you.

Ok, no problem!
Thank you for your feedback.


Web Service modules documentation is missing

hello, we are missing information on some of Designer modules, namely all under Web Service block:
WebService input an WebService output. What they are and what they do.
please also add the link in portal experience.

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

ID: 47f410a2-2269-9f11-1db1-1e6a9b002c29
Version Independent ID: 8062d741-06b6-6afd-ca62-0b10f82df261
Content: Algorithm & module reference - Azure Machine Learning
Content Source: articles/machine-learning/algorithm-module-reference/module-reference.md
Service: machine-learning
Sub-service: core
GitHub Login: @PeterCLu
Microsoft Alias: peterlu

@galinagpolyakova Thanks for the feedback. We are investigating the issue and will update you shortly.

@galinagpolyakova , thank you for the report. Good catch, this is missing documentation and has been added to our documentation backlog. It should be addressed in the next few weeks. I'll proceed to #please-close this issue.
@likebupt for visibility


Possible lint error

I have an possible lint error (does not effect build):  block-size: 280, ⇔ height: 280, 💪Logical Properties(physical-property-detected). It appears on
export default function () {
  showUI({
    height: 280,
    width: 240
  });

Basically wherever I use the words width and height. Any ideas how to fix this? Thanks.

I think you might need to turn off your lint rules causing the error. Unfortunately it’s not reasonable to change the API to remove all instances of width/height in order to prevent lint errors


After changing window 10 username my linux distribution setup is lost

I have installed Wsl2 and Linux distribution successfully
I was having issue with window username, I just changed my window username, after doing so when I started window terminal, my Linux distribution was not shown there

when I separately run ubuntu from start menu, instead of loading old setup it initialized new setup and asked for username and password to get ready
I lost my previous setup of all installed distributions (ubuntu, kali)
is there any configuration where I can tell wsl2 my updated user 'name' so that instead of initializing setup from start it load stuff from there
tried google and official documentation for wsl2 and Linux distributors, nothing helped.
Thanks in Advance

Your Windows username is not ever expected to change, and a great many things can break when it does.

It probably would help if everything was tied to the SID instead of username unless that also changes (though I believe it shouldn't based on experience with AD).

@hanzlahabib
What does wsl.exe -l -v return from the commandline after you've changed your username.

Your Windows username is not ever expected to change, and a great many things can break when it does.

Why is it possible to modify it if:

it's not supposed to happen
bad things may happen if you do so

Sorry, I conflated the user's name with the user's profile path. The first can change, but the second ends up hardcoded in a great number of places across the system (mostly in the registry.)


Changed source of package to patch project

A fixed build and changed source of package to tag v2.6.2-p0

Merged. Please set destination branch develop next time, see https://github.com/ruslo/hunter/wiki/dev.contribution. Thanks!

https://github.com/ruslo/hunter/releases/tag/v0.12.34


Addind text caption to every image

Hello,
Thank you for this great app.
I want to use this to label some images that need a caption to help the annotator.
My folder structure is like this :
|---- folder 1
|       ||-- img1.png
|       ||-- img2.png
|       ||-- caption.txt
|----folder 2
|       ||-- img3.png
|       ||-- img4.png
|       ||-- caption2.txt
I would like to if it is possible to open img1 and img2 with the caption from caption.txt, then to open img3 and img4 with the caption from caption2.txt.
Thank you very much in advance,
PS : I tried to install with build-for-mac-os-x.sh, it doesn't work but it worked with the basic installation.

OS: Mac OS X
PyQt version: PyQt 5

Sorry for the late response, but no, this is not currently possible.


set api url during runtime

Coverage decreased (-0.1%) to 78.68% when pulling 713c10b265087089e1682b49bc868794d72e7a0a on Tr0nYx:master into 43b24b9364334d4bddb07d60f69f7cad7b4eb813 on Alpha-Trader:master.


Import / Export JSON of local data

As a user
I want to export/import my not indexed data
So that I have all my transactions and bonded atoms visible if I switch my device.

@faboweb is this still a feature we need to consider including — considering developments in the conversation about indexing transactions etc...?

I would prefer a system where this is not needed.

me too. i'll close this issue.
@nylira @mappum @keppel


Chore: Use catalogs to manage versions of common dependencies

Linked Issue
Closes #2807
Description
Uses a monorepo root catalog to define versions for every package so it's easier to upgrade, manage, etc.
See: https://pnpm.io/catalogs

Note for contributors: this will bump PNPM to require a minimum version of 9.5.0 or higher!

Changsets
Instructions: Changesets automate our changelog. If you modify files in /packages, run pnpm changeset in the root of the monorepo, follow the prompts, then commit the markdown file. Changes that add features should be minor while chores and bugfixes should be patch. Please prefix the changeset message with feat:, bugfix: or chore:.
Checklist
Please read and apply all contribution requirements.

[x] This PR targets the dev branch (NEVER master)
[x] Documentation reflects all relevant changes
[x] Branch is prefixed with: docs/, feat/, chore/, bugfix/
[x] Ensure Svelte and Typescript linting is current - run pnpm ci:check
[x] Ensure Prettier linting is current - run pnpm format
[x] All test cases are passing - run pnpm test
[x] Includes a changeset (if relevant; see above)

this looks good now, but we'll first have to resolve why vercel's deployment is failing before we merge


Unpin libgit2 version

The pinned libgit2 version does not support modern macOS, since it relies on an outdated libgit2.
I've verified that permitting the latest libgit2 (via changing requirements.txt) appears to work fine.

Thanks! Would you like to make a pull request? A core problem here is that this project doesn't have any automated tests yet.

Unfortunately my employer prevents me from making FOSS contributions.
On Thu, Oct 22, 2020 at 2:38 PM Dan Davison<EMAIL_ADDRESS>wrote:

Thanks! Would you like to make a pull request? A core problem here is that
this project doesn't have any automated tests yet.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dandavison/chronologer/issues/5#issuecomment-714717192,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAA3IGHJZZJSJ6IYEEVHV7DSMCC2NANCNFSM4S3T5P4A
.

--
Zach Riggle


Enhancement: Add new response asserts

This PR adds two new test asserts to improve the test suite

[x] Assert No Flash Set, which checks that there is no flash set
[x] Assert Target Url Contains, which checks if the target url of the redirect matches the given (sub) string

The assert target url contains method makes sure it is a redirect response first, since the getTargetUrl method isn't part of the normal response object.
Somewhat related to #599, since this should help improve tests

@BackEndTea
Needs a quick run of
$ make cs

to get the build to pass, then I think this is good to go.

@localheinz I thought i could be quick and push it before heading to lunch, guess i overlooked that.

@BackEndTea
Ha, no worries, hope you had a great lunch!


https://github.com/ngi-nix/liberaforms-flake

Duplicate https://github.com/ngi-nix/ngipkgs/issues/40


Website is down?

Not sure where to post this. July 13.
https://doc.deno.land/https/github.com/denoland/deno/releases/latest/download/lib.deno.d.ts

Interesting, the site is working for me:

Could you visit https://doc.deno.land/api/docs?entrypoint=https%3A%2F%2Fgithub.com%2Fdenoland%2Fdeno%2Freleases%2Flatest%2Fdownload%2Flib.deno.d.ts&force_reload=true and then visit the page again? This will clear out the server side cache.

Yeah it works now, you can close 🤷
Sorry if it was just me for the noise.


Deskpi Super6c USB ports not working

dmesg | grep usb:
[    1.527450] usb usb2: We don't know the algorithms for LPM for this host, disabling LPM.
[    1.527641] usb usb2: New USB device found, idVendor=1d6b, idProduct=0003, bcdDevice= 5.15
[    1.527660] usb usb2: New USB device strings: Mfr=3, Product=2, SerialNumber=1
[    1.527673] usb usb2: Product: xHCI Host Controller
[    1.527685] usb usb2: Manufacturer: Linux 5.15.105-0-rpi4 xhci-hcd
[    1.527697] usb usb2: SerialNumber: fe9c0000.xhci

@ouziel-slama you said that the problem was resolved by flashing the CM4 bootloader or something like that?

yes given that it works well with certain CM4s I am fairly sure that it comes from the bootloader. The deskpi's micro-USBs are too fragile, I ordered a CM4 board to test updating the bootloader of a CM4...

unfortunately this does not solve the problem :-/
I just updated the CM4's eeprom and it still doesn't work for this CM4! I compiled the master branch from https://github.com/raspberrypi/usbboot so I'm sure to have the latest version! (btw I didn't manage to do it on the Lenovo but only on the Mac, probably a USB port problem...). I'll keep looking but no real ideas.

Have you not heard back from customer support? We bought these devices from a company—they should help us get it to work!

Also have you tried other OSs?

Have you not heard back from customer support? We bought these devices from a company—they should help us get it to work!

nope! two emails and no answers!

Also have you tried other OSs?

yep! and several version!
just miss to test the cm4 with another board ! waiting a mini nvme ssd to test with the board I used to update the eeprom.

(this one https://www.amazon.com/BIGTREETECH-Pi4B-Raspberry-Compatible-Supports/dp/B0B82MH21K)

Moving to 0.6.0!

some more tests:

the USB does not work on another board.
of the 6 cm4 that I have there is only the one with wifi whose USB does not work.
I don't have any other CM4s with wifi to know if this particular CM4 has a problem or all the CM4s with wifi.

Just ordered a CM4102000! let's hope it works well!

Works well with the  CM4 I just received from Welectron! So it's just a broken a CM4 :/


how to use the type resolution to make ForbiddenMethodCall to check bigdecimal.equals using bigdecimal == bigdecimal?

Hi guys i want to make the ForbiddenMethodCall to prevent the use of bigDecimal.equals,
  ForbiddenMethodCall:
    active: true
    methods: ['kotlin.io.println', 'kotlin.io.print', 'kotlin.io.print', 'java.math.BigDecimal.equals']

running gradle detekMain works for bigDecimal.equals(bigdecimal) but in kotlin you use it as bigdecimal == bigdecimal so detekt don't work in this case using the == instead of .equals
I read that maybe I need to enable the type resolution?
checking in the doc website
I need to add this:
    withType<Detekt> {         // Target version of the generated JVM bytecode. It is used for type resolution.         this.jvmTarget = "1.8"     }
but in build.gradle.kts my IntelliJ Detekt is in red and also if i try to import the package detekt is not found
inside tasks {}

I already have this line in
plugins {
id("io.gitlab.arturbosch.detekt") version "1.12.0"
}
Any ideas? Thanks.

Hi,
detektMain is already configured with type resolution.
The plain detekt task is not.
This is actually a false negative. The rule does not look at operator calls.
Thanks for reporting!


Profiles Not Tied To Application

We have been using the config server for a few months now and really like it. However, we would like to be able to declare global properties that are not tied to a given application and then reference them using profiles. The reason is we have a lot of properties that are copy and pasted between micro-services and would like to move them up to a global level based on a named profile.
The documentation shows that everything is tied to an application.
The HTTP service has resources in the form:
/{application}/{profile}[/{label}]
/{application}-{profile}.yml
/{label}/{application}-{profile}.yml
/{application}-{profile}.properties
/{label}/{application}-{profile}.properties

We would like to add:
The HTTP service has resources in the form:
...
/{profile}.yml

As an example we have multiple micro-services that make soap calls to an SAP backend and they will only ever have 3 configurations (dev,test,prod). We would like to be able to name these sap-dev.yml, sap-tst.yml, and sap-prd.yml, which of course would be the name of a profile. These would not be tied to any given application name.
We are coming from a monolith application and trying to split things out, but it still feels like there would be some overlap between resources that you need to run applications with even if we were just starting out fresh.
We are not using a git to pull the properties.

Why not application-sapdev.yml?

Kind of a duplicate of #286. I think I prefer to keep the endpoints as they are though and deal with the "global" properties in the internals. Having said that, there are already features that support global properties: any "application*.*" resources that are not part of the config server configuration are treated as global (in the default repo for git repositories or in the root of the base directory for a native one). (What @spencergibb said in other words.)

We are not using a git to pull the properties.

Why? I'm sure you have reasons, but they are probably worth re-evaluating, given the extra level of support you get from git being the primary backend.

But that is the crux of the problem. We have 2 spring boot applications that have names web and shop that both need access to SAP. By being tied at the application scope we have to copy and paste the configuration because the most global we can get is web-sapdev.yml and shop-sapdev.yml. Both files would still be the same.
We dug into the code and think we know where everything is tied to the application (name). Just wondering if there would be any chance to make this more optional.

Why? I'm sure you have reasons, but they are probably worth re-evaluating, given the extra level of support you get from git being the primary backend.

Still wading into the waters of cloud. We will be looking at this more in the future. I only mentioned it so you would know my setup.

Maybe there is some misunderstanding, application-sapdev.yml will be applied to all applications with the sapdev profile.

Perfect! Exactly what we were looking for. I did not gather that from the docs. Thank you!


Buggy graphs

Note the gaps in the cpu graph caused by (I'm guessing) interpolation of the word null instead of a given number.
This may be caused by the EACCESS at the top right... I have no idea why that would happen.

There's some weird effects happening when selecting a second container. I tried to address it in #12 but have not found the root cause yet.
With #12 It should not happen for the first container selected.


Names for indices

Description
I suggest the indices.py file uses long, descriptive names for indices (heating_degree_days) instead of hdd. Then, in the ICCLIM module, we can map the abbreviations to the long names. I think it will make the library more "readable".

Done.


Pass-Through Transcription and Message Results

What this PR does / why we need it

This depends on this PR getting merged in the Go SDK: https://github.com/dvonthenen/symbl-go-sdk/pull/60
This allows a simplified version of the Recognition Message (usually used for transcription) and the Message Response (usually used for chat windows) be allowed to pass through to the client application.
Which issue(s) this PR fixes

NA
Describe testing done for PR

Made sure:

existing functionality is not broken
the recognition messages usually used for transcription, pass through to the client
messages (or fully formed sentences) usually used for chat, pass through to the client

Special notes for your reviewer

NA

Notified community of change here:
https://symbldotai.slack.com/archives/CB0QF76PK/p1675901153406459
Merging due to lazy consensus.


Automated cherry pick of #7827

Cherry pick of #7827 on cloud.
/cc  @mickmister

/release-note-none


test: add e2e testing for plugin

Test cases to cover:

sourcing from a private bucket
sourcing from multiple buckets
sourcing from a bucket with over 1000 objects (using the ContinuationToken, waiting for #20)

Ideally the tests are ran in CI on a real example Gatsby site pulling from real S3 buckets (no fixtures). This would ensure that the core functionality of the plugin is covered.

:tada: This issue has been resolved in version 2.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket: