Breaks if path is behind as symlink and the symlink is changed

I am trying to use this in conjunction with https://github.com/kubernetes/git-sync
Namely, I've got a k8s pod that has 2 containers, git-sync and your image.
I've got a shared emptyDir volume shared between the two.
The way git-sync works is:
It checks out the revision under rev-<hash>, and creates a symlink name-of-repo -> rev-<hash>.
And then periodically checks for new changes, checks out new rev-<hash> and repoints the symlink.
I am pointing your image to serve name-of-repo/config, and initially it works, but I observe that once git-sync checks out a new revision, your image starts returning 404s.
I am no rust expert, but I believe as you wire your server up, you are probably resolving the symlink to a real path along the way, hence prevent any serving of files once the symlink is changed.

Can you please provide the version that you are using and the os in order to try to reproduce it?

I am pointing your image to serve name-of-repo/config, and initially it works, but I observe that once git-sync checks out a new revision, your image starts returning 404s

I assume that you are changing the root dir which is a symlink right ?

Yes.
Given:
root@you:/tmp/tmp.pjklPfxOlR# mkdir one
root@you:/tmp/tmp.pjklPfxOlR# ln -s one root
root@you:/tmp/tmp.pjklPfxOlR# date >one/a

Which looks like:
root@you:/tmp/tmp.pjklPfxOlR# ls -lah
total 20K
drwx------  3 root root 4.0K Jul  5 21:35 .
drwxrwxrwt 28 root root  12K Jul  5 21:35 ..
drwxr-xr-x  2 root root 4.0K Jul  5 21:35 one
lrwxrwxrwx  1 root root    3 Jul  5 21:35 root -> one

if I run the image as:
docker run -it --rm -v /tmp/tmp.pjklPfxOlR:/mnt -p 3000:3000 joseluisq/static-web-server:1.17.1-alpine --port 3000 --root /mnt/root/ --log-level trace

In the logs, when doing curl localhost:3000/a, I see:
2021-07-05T20:37:08 [TRACE] - Opening path /mnt/one/a

Even in --root is pointing at /mnt/root/.
If I remove one and repoint /mnt/root/ to something else, that could have a, I just 404's.

Ok, the behavior of "pointing always to the root dir " is expected for this server. Even if your symlinked root is changed later on, you will always get the path what was assigned first.
The reason is that the root path can only be changed during server startup but not after dynamically, meaning that the only way to reflect the root path change is restarting the server.
However a workaround could be to try to symlink your content only (inside the root) but not the root itself.
Allow to changing the root path dynamically should be cause a weird behavior or even represent security concerns mapping files on system.
Anyway If you can propose some alternative of solution for your use case please let me know I open to discussion.
I'm going to close this issue since it's not a bug.
However feel free to let your comments or purpose a solution if you want so.

The reason is that the root path can only be changed during server startup but not after dynamically, meaning that the only way to reflect the root path change is restarting the server.

But the path I gave it is /mnt/root, not /mnt/one, the server went on and decided to do it's own thing and changed it itself, by resolving the symlink.
Why is it doing that in the first place?

Allow to changing the root path on the fly could cause unexpected behavior or even represent security concerns mapping files on system.

Sure, I buy a good security argument, but I don't see this one as being a good one.
Why security is only important for the root, but not for any other path down the line?
If anything, the default behaviour on nginx is the completely the opposite.
It permits the root to be a symlink, but does not follow symlinks within the root, unless there is a special directive in the server block permitting that.
That is because attackers can manipulate files within the root, but not outside of the root, hence by definition cannot change what the root symlink points to.

I'm going to close this issue since it's not a bug.

If this is intentional, can you point me where this intention happens?
Don't want to sound offensive, but it feels like you are not sure why/where this happens, and it's easier to just shrug it off as "not-a-bug".
For the record, I've already moved on to using nginx which solves my issue, but I am still going to fight my ground here, as I believe the arguments are quite weak and conclusion is wrong, based on what other web servers do.

I understand your point more clear now regarding the symlink for the root, but as I told you, it's not because I want to avoid your use case is because  the server always resolves root and assets dirs that includes symlinks. That's why I labeled this issue as "is not a bug".
If you want the server behaves differently is another history and we can discuss it, but we can not thread this issue as a bug at first place.
Regarding the label, closing the issue doesn't mean nothing since I can easily reopen it if necessary.

But the path I gave it is /mnt/root, not /mnt/one, the server went on and decided to do it's own thing and changed it itself, by resolving the symlink. Why is it doing that in the first place?

So to clarify again, the server always resolves the root and assets dir paths to its absolutely paths and that includes symlinks that's why your surprise.
So if you don't want that behavior please file another issue explain your use case and why you want it to behaves differently. Then let's discuss it.

I understand what the server does, it's clear from the logs. I think I even know which part of the code triggers this behaviour.
I don't understand why it does that and what it tries to achieve be doing that.

I don't understand why it does that and what it tries to achieve be doing that.

The reason is "simple", convenience across platforms.
We chosen that approach because as you know a symlink can not point only to a dir but also to a file so it was a convenient solution that I preferred ad hoc in order to resolve every root or asset dir path independently of it was a symlink, relative or absolute path.

I still don't follow how resolving at startup is beneficial/better than resolving at request time?
It delivers exactly the same end result, so I must be missing something here.
Please, do not say performance, because it's exactly one extra syscall querying a data structure that is already in memory.
I don't see how preventing usage of features of the filesystem is a convenience, and why "across platforms" is relevant here, given usage (not creation) of symlinks is identical across all of them.
Can you give a scenario where this (pre-resolving symlinks at startup, but for the root and nothing else) makes it convenient, or a scenario where doing that would be in inconvenient?

I still don't follow how resolving at startup is beneficial/better than resolving at request time?

First, for the most common cases that I have seen using a web server to serving static files I have never seen a server with dynamic root path at request time. I see that use case personally rare, even so, your use case for example can be addressed conditioning your content under the root.
Second, I never has said that this server approach is better, but you are claiming that.
What I'm saying is that my opted solution is convenient for this server in order to resolve directory paths: symlinks or relative ones to their absolutes ones (paths by their own with no dependencies). That's all.
What is the problem with it? Is a bug? No. It's intentional. That's why I have labeled your claiming as Not A Bug.

It delivers exactly the same end result, so I must be missing something here.

Please provide an example or even better, code if you can.

Please, do not say performance, because it's exactly one extra syscall querying a data structure that is already in memory.
That is if you choose to manually resolve it, it's zero extra syscalls if you just choose to just open the file.

Honestly I don't care about it, since the "overhead" resolving the root or assets paths is insignificant and all this happens at startup time.

I don't see how preventing usage of features of the filesystem is a convenience, and why "across platforms" is relevant here, given usage (not creation) of symlinks is identical across all of them.

This is clearly a fallacy ad hominem since I have repeated myself comments above that if you provide me enough details in a separate issue we can discuss it and why not to consider to conditioning the current approach.
The point is not the inclusion of a feature like this, that it could happen, but the refactoring and the conditioning of the existing code.

Can you give a scenario where this (pre-resolving symlinks at startup, but for the root and nothing else) makes it convenient, or a scenario where doing that would be in inconvenient?

First, I take it again from the comment above:
I preferred ad hoc in order to resolve every root or asset dir path independently of it was a symlink, relative or absolute path.
The escenario is to have a path (either relative or symlink) resolved (with all intermediate components normalized and symbolic links resolved). It means a path with no dependencies. That is a path that exist by its own: absolute path.
I call the approach as a convenient for this server, you can call it differently if you want.
So here there is no "prevention of system features" or something like that.
For "symlinks being an inconvenience", you are claiming about it.
I don't see why not to adopt a symlink approach. You have misunderstood, there is nothing against it. But make it happen if you are interested in or file a feature request if you can not do it.

I have never seen a server with dynamic root path at request time.

I am not sure what you mean by dynamic, but nginx, apache, lighttpd all support being pointed to a symlink as the content root, and trust the operating system to re-resolve the symlink on every request.
The only reason I dislike using them is that configuring them requires an actual config file, oppose to just command line parameters.
I've raised this one specifically for the issue/use case being discussed here, hence I don't trust that raising another one, repeating the same content explaining what usec ase doesn't work, will suddenly resolve this.
Given we are getting nowhere, and in order to save both of our time, I think we can consider this "resolved".
Thanks for your time, and good luck.

Thanks also for the discussion.
I guess I did try to explain what was involved in the decision of chosing path resolving at first place.
Anyway as I said, this project is open for improvements. And this issue is welcome as an improvement, that means as a feature rather than an issue or bug to fix.

Just FYI or future people interesting in this.
In the new major v2.0.0 the root directory path is not resolved to an absolute at start up time anymore, meaning that your use case with root symlinks updates is covered.


Example fails with error - Cannot process argument transformation on parameter 'Action'

To resolve:
$PS = PowerShell.exe
should be
$PS = New-ScheduledTaskAction -Execute "C:\windows\system32\WindowsPowerShell\v1.0\powershell.exe"

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

ID: d8e0ae68-fc55-853d-8b6c-77d326d78e13
Version Independent ID: 4e093fe1-bdec-0e8f-2d35-ef44181f617d
Content: Register-ScheduledTask (scheduledtasks)
Content Source: docset/winserver2012r2-ps/scheduledtasks/Register-ScheduledTask.md
Product: powershell
Technology: powershell
GitHub Login: @brianlic
Microsoft Alias: brianlic

@officedocsbot assign @e0i

@discombobulater
The issue has been noted and the document will be revised in the light of your feedback.
Thank you.


nord.css module not found

I am using nord-highlightjs within my Nextjs app after trying to link nord.css stylesheet inside head of html document using
<link href="node_modules/nord-highlightjs/dist/nord.css" rel="stylesheet" type="text/css" />

it says module is not found also I have checked package in node_modules and there was no dist folder and there is no code but only these files

.

is the package depricated because even the unpkg link<EMAIL_ADDRESS>gives 404


Rewrite/cleanup/simplify ListComboBox code

Split the editable and non-editable list combo boxes
Fix up item state management as it was utter chaos
Use the appropriate names for style items
Add iconTextGap to list item metrics
Only do preview selection when mouse-moving over an item, not when hovering, to reduce glitches when scrolling
Fix margins and colours
Create PopupManager to track and change a popup's visibility and use it in (List)(Editable)ComboBox
More!

Fixes #707

The CI is red here but it should be fine when #724 is merged


Bug/158

Fixes #158.
Update to the error messaging as follows:

Server error message nodes, and inserted errors within labels are still used, but with role="alert" removed

A summary div with role="alert" is added at the top of the form.  Requires the CSS in the site to hide it in an accessible manner, otherwise it could be styled as an overall error summary for the form.

Error spans are added to this summary at the point of validation.  These are connected to the original input by means of a data attribute matching the group name so that they can be selectively removed later.

Tests updated and added to support this.

Latest update includes an option to enable the new functionality in the settings for backwards compatibility:
useSummary : true
Also includes an update to the ReadMe and tests to match the new setting.   I have copied these changes back into a local project.  Initial tests of the original PR were working well in NVDA/Chrome, but I intend to upload a version which I can access on an iOS device to test VO.

Have run into this issue:
https://github.com/nvaccess/nvda/issues/8557#issuecomment-911863803
NVDA is not currently announcing additional  elements on a  with role="alert" in Chrome.  It works in Edge with some re-arrangement of the role="alert" positioning.   Using span elements works in both, but is less semantic.

Ouch. I can understand why adding role=alert to a ul overwriting the default role=list would confuse NVDA but not the if the alert is a containing div.
We could revert to the spans if it was testing well? The intended benefit of using semantic mark up was improved AT UX, so if that is borne out in testing I have no qualms in reverting.
Alternatively, if we really want to try to make the semantics happen we could test something like

Well crafted error message
Another well crafted error message

Closing PR as per discussions around the error summary requirement


"Cant create temporal dir for (user/repo)"

CLOC now fails and gives me the alert "Cant create temporal dir for (user/repo)" every time I try to use the site.

It seems fixed with 0.7.18

It works now, thanks!


Added support for removing images/tags

Soft delete support has been supported in Docker Distribution since at least
2.1. This was not enough to implement the removal of images/tags in Portus
because there was no support to GC these soft deleted blobs. Since 2.4, it's
possible to not just delete the manifest, but also to GC blobs no longer
referenced by any image manifest. This means that after being able to track
digests, we can now safely provide image/tag removal support. For safety
concerns, tags with an empty digest will not be allowed to be removed (this
is more likely to be the case of Portus versions that have been running for
some time). In previous PRs this has already been addressed, so admins can
update their DB filling in the empty gaps (e.g. see PRs #825 or #830).
The main downside of this change is that there is no way for a client to detect
whether a remote registry supports GC. Because of this, a configuration option
has been provided, which is disabled by default. The rationale is that
administrators that are really sure about the availability of GC on their
private registry will have to enable it explicitly.
Fixes #197
Signed-off-by: Miquel Sabaté Solà<EMAIL_ADDRESS>

Some screenshots:
Removing an image with tags:

After the previous screenshot, we push the same image back:

The UI for removing images and tags:

LGTM

Fixed typos pointed out by @monstermunchkin. I'll squash my commits once reviewing is over :wink:

Beside my NITPICK LGTM

It looks good to me.
A couple of considerations:

users will probably want to delete a namespace and all its contents. This will have an impact on the activity events we are currently creating.
this PR does the soft delete inside of the registry, the user has still to run the GC manually and we still have to add a maintenance mode for that. Am I right?

@flavio pushed a commit fixing your comments. I'll squash the commits once the review is over.

users will probably want to delete a namespace and all its contents. This will have an impact on the activity events we are currently creating

Yes. My plan is to do the same as I'm doing for removed images/tags: update the recipient and add stuff into the parameters attribute.

this PR does the soft delete inside of the registry, the user has still to run the GC manually and we still have to add a maintenance mode for that. Am I right?

Yes and no :smile:.  About maintenance mode... well, it's recommended but not enforced :sweat_smile:. That is, you can run a GC operation without being in maintenance mode and the registry won't complain. However, if you do that, then there are race conditions. So, from the point of view of the registry, it's recommended but not enforced. This should also be in the documentation :wink:

@flavio pushed a commit fixing your comments. I'll squash the commits once the review is over.

LGTM

users will probably want to delete a namespace and all its contents. This will have an impact on the activity events we are currently creating

Yes. My plan is to do the same as I'm doing for removed images/tags: update the recipient and add stuff into the parameters attribute.

Sounds good to me.

this PR does the soft delete inside of the registry, the user has still to run the GC manually and we still have to add a maintenance mode for that. Am I right?

Yes and no :smile:.  About maintenance mode... well, it's recommended but not enforced :sweat_smile:. That is, you can run a GC operation without being in maintenance mode and the registry won't complain. However, if you do that, then there are race conditions. So, from the point of view of the registry, it's recommended but not enforced. This should also be in the documentation :wink:

I'm fine with that as we discussed the ongoing plan about "remove all the things!"


Add a means of disaggregating agencies within IATI publishers

e.g. unitedstates includes a number of different US agencies.

NB This is definitely now possible with the BDD Tester (since pwyf/bdd-tester@de46c87)


Rename yamls

Now that we'll have just update_dashboard and not update_dashboard_group as well (same for index, index_group), I'm removing deprecated yamls and renaming the former *_group yamls to match our other renaming.
With update_dashboard_group becoming update_dashboard (same for index_group to index, etc), I'm removing deprecated yamls and renaming the ones we'll be using to match.

I think you'll also want to make changes in:

tests/lf_workflow_dash/test_module_action.py
.github/workflows/main.yml

And I'm not sure why the unit test workflow didn't run? It would have failed with the missing yaml files...


False positives of UnusedPrivateMember in loops

Sometimes the loop index doesn't have to be used inside the loop, for example, when you write a retry logic:
        for (i in 0 until MAX_ATTEMPTS) {
            try {
                return allocateBitmap(size)
            } catch (ignored: OutOfMemoryError) {
                Log.w(TAG, "OOM with size: x=" + size.x + ", y=" + size.y)
            }
            size.x /= 2
            size.y /= 2
        }

Detekt reports i as being unused private member but it's actually used internally by the loop itself.
However, in many cases such check would be beneficial as usually you want to use the loop index in the loop code block.
I wish there was _ for the index variable in loops in Kotlin.
Expected Behavior
No UnusedPrivateMember rule violation.
Observed Behavior
UnusedPrivateMember rule violation.
Steps to Reproduce
See a code snippet above
Context
I believe there is no way to workaround it - either the check should be disabled for every loop or this false-positive will pop-up from time to time.
Your Environment

Version of detekt used: 1.16.0
Version of Gradle used (if applicable): 6.8

Apart from catching an OutOfMemoryError most likely not being such a great idea, you could just do
repeat (MAX_ATTEMPTS) {
    ...
}

@marschwar great, thanks. I didn't know about repeat. I don't think you need to do anything on your side.


Group elements by name value in svg-exporter fork

https://github.com/jschleuss/svg-exporter

duplicates fork of repo https://github.com/jschleuss/svg-exporter/issues/1


utils/data/datasets/coco.py  has_valid_annotation  always return false?

https://github.com/soeaver/RP-R-CNN/blob/dfcc227723d4bf6566356cda4513a84ed9a8ba30/utils/data/datasets/coco.py#L21-L35

https://github.com/soeaver/RP-R-CNN/blob/dfcc227723d4bf6566356cda4513a84ed9a8ba30/utils/data/datasets/coco.py#L21-L35

I also met this problem. Wonder how you solve that. Thanks so much!!!

https://github.com/soeaver/RP-R-CNN/blob/dfcc227723d4bf6566356cda4513a84ed9a8ba30/utils/data/datasets/coco.py#L21-L35

I also met this problem. Wonder how you solve that. Thanks so much!!!

https://github.com/soeaver/RP-R-CNN/blob/dfcc227723d4bf6566356cda4513a84ed9a8ba30/utils/data/datasets/coco.py#L21-L35

I also met this problem. Wonder how you solve that. Thanks so much!!!

just change Line 35， return True .....

https://github.com/soeaver/RP-R-CNN/blob/dfcc227723d4bf6566356cda4513a84ed9a8ba30/utils/data/datasets/coco.py#L21-L35

I also met this problem. Wonder how you solve that. Thanks so much!!!

just change Line 35， return True .....


refactor(shopware): Optimize getPlugins to use json output

Refactoring the getPlugins function to not parse regular commandline output anymore, instead use the --json flag plugin:list. Refactoring 'sw:plugin:update:all' to not upgrade every installed plugin, but every installed plugin that in fact is upgradable.

[ ] Bug fix #…?

[ ] New feature?

[ ] BC breaks?

[ ] Tests added?

[ ] Docs added?
Please, regenerate docs by running next command:
$ php bin/docgen

Thank you so much for taking care of that. It is on my list for months.

And please regenerate the doc:
Please, regenerate docs by running next command:
$ php bin/docgen
Thanks! :)

@Schrank so, I've removed the explode (obviously^^), and used these changes within a real projects deployment, looking good to me. And I've also updated the docs 😊
And thank you very much for your effort on this project! ❤

Awesome, thank you!


(enhancement): Apply modin repartitioning where required only

Feature or Bugfix

Enhancement

Detail
Currently modin repartitioning is applied as soon as a write method (to_csv, to_parquet) is called. This is not only a wasteful operation performance-wise but also does not prevent the need to repartition once more if the dataframe is altered within the method (e.g. if a column is cast to a new type).
Instead, repartitioning should be applied when it's currently required, that is before a modin group by only.
Relates

https://github.com/modin-project/modin/issues/3435
https://github.com/modin-project/modin/issues/4165#issuecomment-1039617868

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

AWS CodeBuild CI Report

CodeBuild project: GitHubLoadTests5656BB24-s6u9F3qN9oFy
Commit ID: 01934e24e9346dabe811548f99b1ec5391f73555
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: GitHubDistributedCodeBuild6-jWcl5DLmvupS
Commit ID: 01934e24e9346dabe811548f99b1ec5391f73555
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: GitHubStandardCodeBuild8C06-llutOAimTATs
Commit ID: 01934e24e9346dabe811548f99b1ec5391f73555
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: GitHubDistributedCodeBuild6-jWcl5DLmvupS
Commit ID: 72c39034232ef3160a9843c6c5b87aaabcaa3921
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: GitHubStandardCodeBuild8C06-llutOAimTATs
Commit ID: 72c39034232ef3160a9843c6c5b87aaabcaa3921
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: GitHubLoadTests5656BB24-s6u9F3qN9oFy
Commit ID: 72c39034232ef3160a9843c6c5b87aaabcaa3921
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@jaidisido This would be a good addition to Modin as well, would you be willing to contribute this back to Modin?

@csnatarajan do you mean @modin_repartition decorator?

@kukushking is there anything related to that decorator that would be good to include in Modin? Is the purpose of the decorator to work around bugs like #3435?

@kukushking also, would it help if modin allowed users to configure the column and row partitioning separately as here? That way you wouldn't have to keep undoing the column partitioning in modin_repartition (from a cursory look I think that's what that decorator does).


Unresolve variable or method

All types defined through ///   must be manually specified in tsconfig, otherwise WebStorm throws an warning Unresolve variable or method
tsconfig
"files": [
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>etc...
],

That is not an issue, it's literally in the "Usage" section.

@Katiushin any update on this one, doesn't appear to be a problem in VSCode for example - so potentially a WebStorm specific problem and out of scope of this project - unless there's a simple generic fix that doesn't require adding every file to the tsconfig


neon-desktop: switch to Azure cluster from another cluster blinks red

Here's what happened:

I create an Azure cluster for a demo, and everything worked fine
Then I switched to an existing AWS cluster and did a demo there, and then went fine too
Then I removed the AWS cluster
neon-desktop switched back automatically to the Azure cluster after deleting the AWS cluster
...but the app icon was blinking red
I tried restarting neon-desktop, but that didn't help

I haven't seen this before.

The cluster is reporting being partially configured

CLOSING: this code has been refactored since this was reported


autotools-language-server v0.0.19 is broken and should be reverted to v0.0.18

This commit has broken the installation of the autotools-language-server.
v0.0.19 uses a pypi package that has been yanked.

Fixed by https://github.com/mason-org/mason-registry/pull/6050.


Strange Behavior

Hi,
I notice this strange behavior during the test, could you please review?
Thanks
Issue.mp4.zip

@Jercik could you please help me on this?
Thanks

@bombez is this still an issue?

Closed due to inactivity.


Add build flags to improve RTOS view

I'm the engineer that added this feature and we've seen some customer confusion about best practices for configuring their projects to get the most out of the RTOS view. This PR documents which build flags are required to get information from each RTOS.

Learn Build status updates of commit 8b01075:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/embedded/rtos-view.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

@gcampbell-msft,
Can you review the proposed changes?
IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off

@corob-msft, can you approve/merge this or advise who can? Thanks.


Update dependencies

I really like the API, would it be ok to make a PR which updates dependencies to newest ones? I have some conflicts when installing with bower (most notably purescript-eff).
How this relates to #11 ?? Do we plan to still maintain this library?

Thank you. I will continue to merge PRs, as long as there are some!


[[CHORE]] Add my new name to .mailmap

In this way my past contributions are mapped correctly.

CLA Hub complains I haven't signed the CLA but when I click on "Details" it says I have. ¯\_(ツ)_/¯

Coverage remained the same at 98.744% when pulling ed78e256c3a9842ca28dc050d667402c5c38b7ea on mgol:mailmap into c7c5c42885aa618e384de3f6589793fd419de7c1 on jshint:master.

Looks like the CLA problem cleared itself up.


Add unique page descriptions

Added to /team, /donate, /apply
Suggestions/PRs for better page descriptions are welcome!

Taking over this issue because @jkwok91 currently has her hands full with a workshop.

Ok @zachlatta, it looks like it passed

Max, you are fantastic.


Template V2 target format change

Since the start, we have imagined targets data as stating a base year (starting point of a data series), a start year (publication date of the target), an end date (the date by which the target reduction should be achieved), and a base year quantity, which is the starting data amount that will be reduced by the target reduction amount.
The OECM Production-Centric benchmark complicates this, because it folds sector-specific S3 data into S1 metrics (which directly or indirectly also affects S1+S2 metrics).  The stipulation that Company X has, in year Y, S1+S2 base emissions of 1.2 Mt CO2e and S3 base emissions of 3.6 Mt CO2e means we must show that for year Y, it actually has emissions of 4.8 Mt CO2e, because that is the production-centric amount.  Imagine that it has an initial intensity expressed in S1+S2 terms, also in year Y, of 1 Mt/GWh, with a goal of reducing that 50% to 0.5 Mt CO2e/GWh in 2030.
Question 1: Do we likely believe that Company X has already done the production-centric math to create a production-centric number?  Almost certainly not.
If we want to create a production-centric intensity number, we need to add the production intensity of S3 into S1+S2, which would be 3.6/1.2 = 3 Mt/GWh, for a total production-centric S1+S2 intensity of 4 Mt/GWh, and a production-centric intensity goal of 2 Mt/GWh.
Question 2: Should we drop the concept of storing base year data in the target sheet and instead ensure that we have base year data (which can be production-centric-corrected or not, depending on user or ITR tool choices)?  The V2 template makes this easy (and the V1 template makes it annoyingly difficult).
For quality and robustness purposes, we ideally want to have only a single source of truth for any number.  By eliminating base year data from the target template and holding it only in the esg data page, we have a single source of truth (whose truth value can be consistently converted to production-centric when required).  Otherwise, we need to perform production-centric conversions on all the target data separately, and ensure the completeness and consistency of those operations with conversions applied to the general disclosure data.
Question 3: Should we convert target specifications that mention S3 objectives into production-centric terms, doing all our projection math in production-centric terms, or should we attempt to maintain everything within their separate scopes and only do the production-centric conversion at the very end, when we report data?
Presently we do a mix of both approaches, which leads to inconsistencies that are causing unit tests (rightly) to fail.
Another way to frame Question 3: should we require that sectors using production-centric benchmarks present only S1+S2 data (in esg and targets), flagging as an error any S3 data in that context?  Or should we accept S3-related data and targets, project accordingly, and simply fold S3 projections into S1 projections when computing final temperature scores and constructing the final amended portfolio?  In the latter case, the tool will not report S1+S2+S3 statistics as such, but only S1+S2 as a production-centric number.
Please comment with your thoughts @LeylaJavadova @ImkeHorten @BertKramer

Hi Michael,
I agree with moving base year data from the target sheet to the data sheet.
Extending the limited S1+2 target to a production-centric version in the way you suggest in the lines between Q1 and Q2 intuitively makes sense, but it is actually not that straightforward. The span of control companies have over the different scopes is different, and therefore also how easy they can reduce their emissions per scope. So even though a company might have set a S1+S2 reduction target, non-published underlying reduction targets might be quite different between S1 and S2, and extending to include S3 is nontrivial.  Let’s elaborate further on this early next year.
Bert
From: Michael Tiemann @.>
Sent: zaterdag 17 december 2022 21:16
To: os-climate/ITR @.>
Cc: Bert Kramer @.>; Mention @.>
Subject: [os-climate/ITR] Template V2 target format change (Issue #173)
Since the start, we have imagined targets data as stating a base year (starting point of a data series), a start year (publication date of the target), an end date (the date by which the target reduction should be achieved), and a base year quantity, which is the starting data amount that will be reduced by the target reduction amount.
The OECM Production-Centric benchmark complicates this, because it folds sector-specific S3 data into S1 metrics (which directly or indirectly also affects S1+S2 metrics). The stipulation that Company X has, in year Y, S1+S2 base emissions of 1.2 Mt CO2e and S3 base emissions of 3.6 Mt CO2e means we must show that for year Y, it actually has emissions of 4.8 Mt CO2e, because that is the production-centric amount. Imagine that it has an initial intensity expressed in S1+S2 terms, also in year Y, of 1 Mt/GWh, with a goal of reducing that 50% to 0.5 Mt CO2e/GWh in 2030.
Question 1: Do we likely believe that Company X has already done the production-centric math to create a production-centric number? Almost certainly not.
If we want to create a production-centric intensity number, we need to add the production intensity of S3 into S1+S2, which would be 3.6/1.2 = 3 Mt/GWh, for a total production-centric S1+S2 intensity of 4 Mt/GWh, and a production-centric intensity goal of 2 Mt/GWh.
Question 2: Should we drop the concept of storing base year data in the target sheet and instead ensure that we have base year data (which can be production-centric-corrected or not, depending on user or ITR tool choices)? The V2 template makes this easy (and the V1 template makes it annoyingly difficult).
For quality and robustness purposes, we ideally want to have only a single source of truth for any number. By eliminating base year data from the target template and holding it only in the esg data page, we have a single source of truth (whose truth value can be consistently converted to production-centric when required). Otherwise, we need to perform production-centric conversions on all the target data separately, and ensure the completeness and consistency of those operations with conversions applied to the general disclosure data.
As it stands, the conversion of esg disclosure data and non-conversion of target data creates a gaping inconsistency.
Please comment with your thoughts @LeylaJavadovahttps://github.com/LeylaJavadova @ImkeHortenhttps://github.com/ImkeHorten @BertKramerhttps://github.com/BertKramer
—
Reply to this email directly, view it on GitHubhttps://github.com/os-climate/ITR/issues/173, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AUWPCCOGLCS566C4J5E3EE3WNYNQNANCNFSM6AAAAAATCDWDWQ.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>


Pass backup policy parameters as volume labels

Each container should be able to expose backup policy parameters are labels, in the form io.conplicity.<volume>.param.
Examples:

io.conplicity.pgdata.driver: postgresql
io.conplicity.pgdata.retention: 7d

Two labels were added in 0.5.0:

io.conplicity.ignore=true ignores the volume
io.conplicity.full_if_older_than=<value> sets the time period after which a full backup is performed. Defaults to the FULL_IF_OLDER_THAN environment variable value

@raphink I think we can close this one

Conplicity support volume labels, but unfortunately Docker Engine does not support volume labels update yet: https://github.com/moby/moby/issues/25291


Populating route certificates from secrets in another namespace

Hi, I was looking trough the documentation to setup cert manager with let's encrypt on a openshift cluster and I came across this operator.  I have a situation where I need to create a wildcard Let's encrypt certificate and use it in routes defined in 4-5 namespaces.
I saw that the annotation cert-utils-operator.redhat-cop.io/certs-from-secret: "<secret-name>" can load from a secret and that functionality works very well but cannot load the secret from another namespace as it assumes it to be in the same namespace as the route,
I saw that this is possibile for the annotation cert-utils-operator.redhat-cop.io/injectca-from-secret: <secret namespace>/<secret name>.
Is this a limitation or a necessary choice due to the way  kubernetes work or the operator could be modified to allow it to specify a namespace for the secrets?

The design to not allow you to load certificates from another namespace is for security reason. CA bundles are public info, so it's ok to do cross namespace look-ups.

So the recommended solution in this case is to create separate secrets in every namespace with the wildcard certificate and sync them in some way (cronjob, etc..) ?

my recommended solution is described here:
https://www.openshift.com/blog/self-serviced-end-to-end-encryption-for-kubernetes-applications-part-2-a-practical-example
This is a rather old article and the automation may not work anymore, but
the concepts are still valid.
On Mon, Feb 8, 2021 at 9:10 AM Paolo Nicodemo<EMAIL_ADDRESS>wrote:

So the recommended solution in this case is to create separate secrets in
every namespace with the wildcard certificate and sync them in some way
(cronjob, etc..) ?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/redhat-cop/cert-utils-operator/issues/85#issuecomment-775176016,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABPERXFV54CQTKY4UKKCGSTS57WEBANCNFSM4XI6LDLQ
.

--
ciao/bye
Raffaele


z.enum to allow non-string options

I recently opened the issue #2312 where I elaborated that the typings of z.enum were too restrictive as they didn't allow arbitrary list of strings.
This is the extension of the above problem which goes not only about typings but also about implementation. (I am opening a separate issue as I realise this is all breaking changes and maintainers tend to disregard them... so I give these separate issues more chance to survive :)
Problem
Imagine I have a schema that validates a value to be one of the predefined integers. Like when a user chooses the length of the secret key to be either 512, 1024 or 2048 bits (just random example). Naturally, I would imagine there was a Zod schema for that... like z.enum([512, 1024, 2048])? Unfortunately, that doesn't work, as z.enum only accepts string values.
Current workaround
One can use z.union([z.literal(512), z.literal(1024), z.literal(2048)] but that's cumbersome (and also arguably slow as it validates each sub-schema independently).
Suggested approach
What if z.enum simply allowed any list of anything as valid options?
Like, what if z.enum([1, 5, "testing", true]) simply worked and validated the value to be one of the options (with options.includes(value) as the implementation)? The resulting type could be derived from the options with ValueOf<Options> recipe.

Hey 👋

z.enum is a Zod-native way to declare a schema with a fixed set of allowable string values.
To get autocompletion with a Zod enum, use the .enum property of your schema:
~ README.md

Zod enums are for string values known at compile-time, so they could provide autocompletion.
What happens if you pass z.enum([5, '5']), what should the type and runtime value of parsed.enum['5'] be?

@igalklebanov I agree, that is valid point. I didn't consider schema.enum and it will indeed break. (I must have disregarded it as I saw no point in it for my scenarios, but I realise that in some circumstances it makes sense to have it).
However, I don't think z.union should be changed. It is by its nature an aggregating schema builder and should do that. Instead, I believe it's better to introduce z.options which will be basically z.enum without the enum auto-completion magic but accepting anything as the list of options (not even necessarily primitives).


C2, R2 how to replace fan cowl with new, printer version tutorial

will get to this once jason updates and sends me the new fan cowl

https://help.robo3d.com/hc/en-us/articles/115002540172-Mod-your-C2-with-a-3D-printed-fan-duct


Feat: Download Chats as Images

https://github.com/iyashjayesh/monigo/issues/19
Download Charts as Images

@Viv-ek-Pandey Sorry for the delay; I haven’t been well for the past few days.
Would it be possible to share an example of a downloaded image?

@iyashjayesh
Hi , Please have a look .  :) .


I should see others availability in the grid when joining an event

App currently shows grid with other users' availability only after submitting your own and joining the event. It would be useful to know this information as you add in your own availability.

Right now, the Grid opens in "edit mode" right after you join the event. We thought it'd be good UX but I realise that it might actually be a little confusing.
We can just not open the grid in "edit mode": the user in this case will see the grid open as
the heatmap with everybody's availability right after they join.
The other solution (which I'm thinking of implementing anyway) is to have a "Cancel" button which would take the grid out of "edit mode" without saving the user's availability.

My opinion, we can display a faded and light-grayed heat-map... behind the grid...  so the user can see the actual heat-map and pick the superposed times....

#248


Ensure codecData is emitted for input streams

I noticed that codecData is never triggered if you use an input stream because of this code in ffmpeg.c:
     if (stdin_interaction) {
         av_log(NULL, AV_LOG_INFO, "Press [q] to stop, [?] for help\n");
     }

This patch adds an alternative check for ^Output # to see if codecData should be emitted if there is at least one input stream.

Thanks for that (and sorry for the delay).
I just fixed the test suite, could you please rebase your PR on current master and add a test for the issue it fixes?

Nice catch. Can we get this into the next release?

Of course. I'm waiting for every open bug to be closed before théenext version bump.

Merged in 643fc64b653826993ec5cb396357407ced144615

:+1:


Fix up create family method retain counts in cases other than just andReturn

Case where a value is being returned as part of an andDo or other invocation actions.
Given that invocation actions can be chained, we need to make sure to only affect
the final action in the chain.
RealObjectForwarders are treated specially because their implementations will handle
the case correctly, so we need to balance out the retainCounts.

I addressed the problem described in the comment above in 596d866394185b4b6a5b076df0f9c7031b0d798d.

Nice catch Erik. I've mostly convinced myself that my previous solution
"worked" but only in the sense that it was relying on undefined ARC
behavior, and it may break when built with optimizations on.
I tested your solution with our codebase and didn't run into any issues.
On Sat, Nov 28, 2020 at 4:44 PM Erik Doernenburg<EMAIL_ADDRESS>wrote:

I addressed the problem described in the comment above in 596d866
https://github.com/erikdoe/ocmock/commit/596d866394185b4b6a5b076df0f9c7031b0d798d
.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/erikdoe/ocmock/pull/457#issuecomment-735311175, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AACOFSPDFBBFC3YFR5Q6JYTSSGKNBANCNFSM4P6SS53A
.


Restructuring Layout

🛠 Fixes Issue
This issue fixes #36

👨💻 Changes proposed

Adds a Layout
Adds Google Font for Sans
Added flexboxes where needed
Worked on re-alignment

🔧 Check List (Check the type of issue it fixes)

[x] Bug Fix
[ ] New Extension
[x] Enhancement
[ ] Documentation

✔ Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] My changes generate no new warnings
[x] Any dependent changes have been merged and published in downstream modules
[x] I have done git pull before adding my new changes
[x] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers
📷 Screenshots
Pre

Post

Also using _app.js can you make the header added to all the pages

Please check out #33 which was merged in the Dev branch, it would be better if you worked on that branch instead

@AbhijithGanesh kindly take a look at these small issues.

I was under the assumption that we were working on main branch, apologies 😅


file tags input crashed when supplied with list

file tags input says that it requires a "An optional list of tags...", which made me think that I need to supply it a list like this:
{"a", "b", "c"}
That unfortunately is not true. It crashed Dynamo when I tried giving it a list. I am going to have a look at it, we might want to add a list handling for this input or make the description clearer as to what input is expected.

Thanks @ksobon . I think It should handle both cases similar to how Grasshopper component does. Either a list of values or values separated by ',' or ';'.

Thanks @ksobon, I need to go back and setup similar safety parameters for all the inputs to avoid crashing.  @mostaphaRoudsari I am just finishing up finals at school so things have been crazy but I plan on addressing a lot of cleanup/refinement over my winter break.

Thanks @alfarok! No worries. Best of luck with your finals. :moon:

Yeah dude, don't worry about it. Good luck!

Should be able to handle any type of string with delimiter or list now.


9/20 Publish

@Taojunshen,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-autogen/com.microsoft.azure.sdk.iot.service.devicetwin._device_twin_device.yml
:white_check_mark:Succeeded

docs-ref-autogen/com.microsoft.azure.sdk.iot.service.jobs._job_type.yml
:white_check_mark:Succeeded

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


Add encode/decode receipts capabilities

Users should be able to index receipts, for this they will need to decode the data of the receipts. This could be done by the current tooling we already have as the rust SDK or typescript SDK. To solve this we can support it on a infrastructure level or just by providing examples on how to do it.

close in favor of: #21


Re-vamp links from projectcalico.org to calico-containers

The links from the projectcalico.org site (http://www.projectcalico.org/getting-started/) could do with a substantial re-work to better reflect the variety of options available in calico-containers.

@eepyaich Are you the right assignee for this?

This issue was moved to projectcalico/calico#62


[SPO, CMO, SITE] - OrdersDashboard - Last Modified date and time should be the status date

Now the Last Modified column is the same as Request Date

Please get rid of lastModified property from the order model

The issue remains. Please see the print screen. The true time of cancelation was 15:48.

The issue is fixed.


Fixed Side Panel Pop up on developer mode

Description

What is the purpose of this pull request?

[ ] New feature
[ ] Documentation update
[x ] Bug fix
[ ] Refactor
[ ] Release
[ ] Other

I added a comment on the original issue
https://github.com/onlook-dev/studio/issues/156#issuecomment-2292651543

Sorry this wasn't very clear. I believe @drfarrell meant the Chrome Devtool which should just be to remove this line @SkyC0der https://github.com/onlook-dev/studio/blob/main/app/electron/main/index.ts#L48

Done! I thought he meant the styles panel earlier. Thank you for the clarification.


nixos: adding jormungandr service

Motivation for this change
Adding service for jormungandr blockchain node, along with a test.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Notify maintainers
cc @

@GrahamcOfBorg test jormungandr

@GrahamcOfBorg test jormungandr


Do you plan to release the training process?

Thank you for the excellent work. I don't see any training codes by now. Do you have any plan to release them?

The configuration files are compatible with the GitHub repository for our other publication, RTFS-Net.


Handle the case when arraystruct->meminfo is null to close gh-965

Closes gh-965 by raising Python exception when arraystruct->meminfo field is null, and unhandled exception would have caused dereferencing of the null pointer and subsequent crash.

[x] Have you provided a meaningful PR description?
[ ] Have you added a test, reproducer or referred to an issue with a reproducer?
[ ] Have you tested your changes locally for CPU and GPU devices?
[ ] Have you made sure that new changes do not introduce compiler warnings?
[ ] If this PR is a work in progress, are you filing the PR as a draft?

@oleksandr-pavlyk @chudur-budur the solution is fine for now especially as it has already been merged.
I wanted to implement it in a different way. Look at what Numba does for NumPy arrays: https://github.com/numba/numba/blob/ce69f3010c77942123815d86900cb841c0e5d2fa/numba/core/runtime/_nrt_python.c#L366
Numba uses the arraystruct.meminfo if the meminfo object is populated. Otherwise it lets the ndarray parent to be NULL when calling PyArray_NewFromDescr.
    array = (PyArrayObject *) PyArray_NewFromDescr(retty, descr, ndim,
                                                   shape, strides, arystruct->data,
                                                   flags, (PyObject *) miobj);

We should have equivalent C-API for tensor and dpnp and handle the case where we do not have meminfo.

@diptorupd I think it should be reimplemented as needed. The fix went in like this to comply with SDL requirements.


[BUG] All panics become "A hanging Promise was canceled" error

Is there an existing issue for this?

[X] I have searched the existing issues

What version of workers-rs are you using?
0.0.18
What version of wrangler are you using?
3.30.1
Describe the bug
Here is a minimal failing worker, which can be easily reproduced on a new copy of the template:
use worker::*;

#[event(fetch)]
async fn main(req: Request, env: Env, ctx: Context) -> Result<Response> {
    panic!("test");
    Response::ok("Hello, World!")
}

Run npm run dev and hit the endpoint locally. It returns an unhelpful error:
[wrangler:inf] Ready on http://localhost:8787
✘ [ERROR] A hanging Promise was canceled. This happens when the worker runtime is waiting for a Promise from JavaScript to resolve, but has detected that the Promise cannot possibly ever resolve because all code and events related to the Promise's I/O context have already finished.

✘ [ERROR] Uncaught (in response) Error: The script will never generate a response.

As far as I can tell, this happens to all panics, which makes debugging very difficult. If this is expected, I'm surprised there isn't more information about how to handle panics. I would expect a more useful error message.
Steps To Reproduce

Clone latest template (optionally upgrade to latest worker and wrangler packages)
Add panic!() to the starter code
Run npm run dev
Hit endpoint with curl or whatever
See unhelpful error

You should probably use https://github.com/rustwasm/console_error_panic_hook
I'm not sure if the panic behavior has changed over time (from an exception to the hanging Promise), but either way this crate should log the true error.

Thanks, that does seem to work. Perhaps there is a future in which the worker runtime sets the panic hook automatically. But this is good for now.


Whitelabel

There are many places that make direct reference to "Firefly". To make this software more useful for other organizations with similar art grant processes:

Allow easy configuration of the organization name
Call out specific things that might need to change when used by other organizations (e.g. email copy)

It would also be cool if we could put our contract templates in github, which I'd been avoiding due to possible legal implications.  I'll try to get a definitive answer on that particular issue


NT-1539: Unprompted Edit Rewards Alert

📲 What
Added a additional filter and check to prevent the alert from presenting when navigated away from the app.
🤔 Why
When the user updates their pledge, and decides to select another reward a modal pops up confirming that their selected add-ons might not be available if they switch. This alert also pops up when the user navigates to another app and comes back to the kickstarter app.
🛠 How
The project activity notifies the other sub fragments when the user resumes their session and this toggles an update to the reward clicked stream. If the user already prompted the modal before, this modal will come up again. The fix was to apply a filter on the reward clicked so that the alert only shows when it is prompted from a selection of the reward
📋 QA
See steps to reproduce in ticket
Story 📖
NT-1539

Will close this PR to allow for a simpler solution that Isabel found. We need to spend more time refactoring the streams as a whole


Concurrency Control

Implements all transaction semantics related to execution (begin, commit, abort, maintenance of versions, etc.)
Parts related to garbage collection (global table for running transactions) is not implemented.

Coverage remained the same at 93.519% when pulling<PHONE_NUMBER>0c9429b949f676e4383c7e6d6926de on tianyu-txn into bb8591182744531137f8519f41ed6a9278d0551c on master.

More tests will come in in a separate PR. We want to get this in since this modifies API, and hopefully before any other PRs come in relying on stale API.


Possible to use image creation ?

Is it possible to use image creation api with this bot ?

No, it does not


Update /about page with new contributors

@NinaKos, @nejcjelovcan, @gandalfar should also be listed. This could be a nice intro task for one of the new kittens.
Also, didn't @ercchy prepare a more stylish version of this page some time ago? :cat2:

Not the case anymore. This was probably the case when all ambassadors were on one page.


move integration tests from ZooKeeper 3.4.x to 3.5.x

as we deprecate ZooKeeper 3.4.x, the first step is to move our integration tests from 3.4.x to 3.5.x
see #10780 for context
@clintropolis @gianm what's the process to build and update the base image?

I think it'd be good for now to test both ZK 3.4 and 3.5 since we're meant to be in a migratory period where both servers are supported. Do you think you could add a smoke test that uses ZK 3.4? (I don't think we need to run every single test, but it'd be good to run some subset of the ITs.)

@clintropolis @gianm what's the process to build and update the base image?

I think someone on the Imply team would need to push a new base image. That's probably the easiest way.
We could also move the base image to the "apache" Docker account? We started using a base image from Imply in #5060 for performance reasons, and IIRC that pre-dates when Druid joined ASF, so using the ASF account wouldn't have been an option at the time. Maybe it is now.

I think it'd be good for now to test both ZK 3.4 and 3.5 since we're meant to be in a migratory period where both servers are supported. Do you think you could add a smoke test that uses ZK 3.4? (I don't think we need to run every single test, but it'd be good to run some subset of the ITs.)

@clintropolis @gianm what's the process to build and update the base image?

I think someone on the Imply team would need to push a new base image. That's probably the easiest way.
We could also move the base image to the "apache" Docker account? We started using a base image from Imply in #5060 for performance reasons, and IIRC that pre-dates when Druid joined ASF, so using the ASF account wouldn't have been an option at the time. Maybe it is now.

Hmm, since the integration tests are using docker-compose now, I wonder if we should just pull stock zookeeper (and kafka) images and use those in the compose files instead...

Hmm, since the integration tests are using docker-compose now, I wonder if we should just pull stock zookeeper (and kafka) images and use those in the compose files instead...

@gianm agree we should ideally test both. I'll see if it's easy to add separate 3.4.x integration test. All our "unit" tests that rely on curator's TestingCluster and TestingServer classes would continue to test against embedded 3.4.x nodes until we move to 3.5.x

@gianm agree we should ideally test both. I'll see if it's easy to add separate 3.4.x integration test. All our "unit" tests that rely on curator's TestingCluster and TestingServer classes would continue to test against embedded 3.4.x nodes until we move to 3.5.x

Hi @xvrl, I'm planning to finish the 0.21.0 release early next week. Do you think we can get this PR merged before that? Let me know if there is anything I can help.

Hi @xvrl, I'm planning to finish the 0.21.0 release early next week. Do you think we can get this PR merged before that? Let me know if there is anything I can help.

@jihoonson I've updated the integration tests to remove the need for an external image build and added some ZK 3.4 integration tests.

@jihoonson I've updated the integration tests to remove the need for an external image build and added some ZK 3.4 integration tests.

@xvrl great, thanks! I will take a look this weekend.

@xvrl great, thanks! I will take a look this weekend.


Code of Conduct and Contributing section

Addresses #169
@dialelo @rafaparadela Please review. Thanks!

LGTM


Improvements for logs functionality

Date file - when we do any actions, PULL, there is no visibility of which repository is used for this command.

Without internet: No errors in application log, the message command executed is shown(do we need message here: "You do not have internet"???). Try to include?
There is no log when close the app
Git branch should be after Application started - change order? Possibly yes

Log file for each repository(see attachment)

Improve error message from stacktrace(Error file) - find ways to trace it to our code instead of generic message (TBD - experiment)

If source path is not correct(or empty) - Application is closed but there is no error in log file.

Roman Koshovskyy (4/27/2022):
Transfer logs into .json, then work on smaller details

Roman Koshovskyy (4/27/2022):
QA: Work with .exe where possible? - Look for exes - distributed for release (Production like) - https://ci.appveyor.com/project/Yuriy-Pelekh/GitTank/builds/43359299/artifacts use Appveyor to obtain artifacts

Ivan Nadych (5/02/2022):
@ONadiak Did you implement this one "Git branch should be after Application started - change order? Possibly yes"

Solomiia Nikolaichuk (5/03/2022):
Are the logs sorted by DateTime? Need to think about a solution for multi-threading:

Ostap Nadiak (5/03/2022):
@nadychivan it looks like this command is needed to start an application, so I didn't implement this

Roman Koshovskyy (5/03/2022):
Multithreading logging logic - how to improve?

Add thread ID into log

For later:

Combine commands instead of DateTimeStamp
Improved expectations for log functionality added to (add wiki page)


Add title field to Zod - close #1831

Title is a useful field when presenting or converting to JSON schema. Title is used for a human-readable name of a variable. Very practical when combined with description.
Add optional title field.
Add titled setter method.
Add test suit.
Update docs next to describe for ease of use.

One note about this: I was unsure if I should use the name of titled or title. I think title have a better DX, but looking at all other methods, seems like description is following another pattern with describe so I tried to follow the same pattern.

Hey @colinhacks, any chance you can check this out? We are adding Zod to Novu, and I'd love to have this title available. I am happy to add more information if this can be helpful.
Thanks!

Hey, wanna give a +1 to this PR. title indeed is useful and its part of the annotation keyword in JSON schema.
https://json-schema.org/understanding-json-schema/reference/annotations

To help advocate for this PR further, in OpenAI latest recommendation on data structure for json schema response at https://platform.openai.com/docs/guides/structured-outputs/tips-for-your-data-structure

The mention about defining title as one of the tip.
In pydantic (zod equivalent in python), they do allow defining title in field:
https://docs.pydantic.dev/latest/api/fields/#pydantic.fields.Field

bump to check if we can get this in?


PR: Sync subrepo with spyder-kernels#271

Description of Changes
This is necessary to fix the problem reported on issue #14472
Issue(s) Resolved

Fixes #14472.
Affirmation
By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.

I certify the above statement is true and correct: @ccordoba12

Hello @ccordoba12! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file external-deps/spyder-kernels/spyder_kernels/console/tests/test_console_kernel.py:

Line 610:1: E722 do not use bare 'except'
Line 615:80: E501 line too long (105 > 79 characters)


[Feature] concat_ws supports array type parameters

Why I'm doing:
When I use hive, the second parameter of concat_ws can be of array type. After migrating to starrocks, it needs to be changed to array_join, and the order of parameters also needs to be changed, which makes migration difficult.
What I'm doing:
A new concat_ws function that supports array input parameters is added in be, and the code logic of array_join is used
Fixes #issue
What type of PR is this:

[ ] BugFix
[x] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[x] Yes, this PR will result in a change in behavior.
[ ] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[x] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[x] I have added test cases for my bug fix or my new feature
[x] This pr needs user documentation (for new or modified features or behaviors)

[x] I have added documentation for my new feature or new function

[ ] This is a backport pr

Bugfix cherry-pick branch check:

[ ] I have checked the version labels which the pr will be auto-backported to the target branch

[ ] 3.2
[ ] 3.1
[ ] 3.0
[ ] 2.5

@packy92 @mofeiatwork @Seaven Could you please review it again?

@Seaven @mofeiatwork @packy92 Could you please review it again?


feat: graduate ScheduleDaemonSetPods to GA

ref: https://github.com/kubernetes/enhancements/issues/548
/assign @k82cn
/cc @irvifa

/assign @daminisatya

@draveness Thank you for the update.
Do we need to update this file as well? - https://github.com/kubernetes/website/blob/master/content/en/docs/reference/command-line-tools-reference/feature-gates.md
It's a question Im not sure though. Let me know about it.

/lgtm
/approve

@draveness Thank you for the update.
Do we need to update this file as well? - https://github.com/kubernetes/website/blob/master/content/en/docs/reference/command-line-tools-reference/feature-gates.md
It's a question Im not sure though. Let me know about it.

Thanks, I've updated the file.

Thanks @draveness
/lgtm

/cc @daminisatya

Thanks @draveness
/lgtm

Thanks for the lgtm, I just removed the section which has already been removed from the codebase in 1.17

/approve

/lgtm
/approve

/milestone 1.17


Assertion lowering mhlo.dot from Brax program

I'm trying to compile a program coming from Brax with this Colab notebook.
Reproducers here: https://gist.github.com/ScottTodd/4f4434a25f3036f1a37cda88aa872add
Compilation from Python (with asserts disabled?) fails with
error: failed to legalize operation 'mhlo.dot' that was explicitly marked illegal

Repro command using native tools with asserts enabled: iree-translate -iree-input-type=xla -iree-mlir-to-vm-bytecode-module -iree-hal-target-backends=dylib-llvm-aot inference_fn_tf-mhlo.mlir
Callstack with the assert:
Assertion failed: Index < Length && "Invalid index!", file D:\dev\projects\iree\third_party\llvm-project\llvm\include\llvm/ADT/ArrayRef.h, line 255

iree-translate.exe!HandleAbort(int Sig) Line 408 (d:\dev\projects\iree\third_party\llvm-project\llvm\lib\Support\Windows\Signals.inc:408)
ucrtbase.dll!00007fff4ba31881() (Unknown Source:0)
ucrtbase.dll!00007fff4ba32851() (Unknown Source:0)
ucrtbase.dll!00007fff4ba341b5() (Unknown Source:0)
ucrtbase.dll!00007fff4ba344f1() (Unknown Source:0)
[Inline Frame] iree-translate.exe!llvm::ArrayRef<__int64>::operator[](unsigned __int64) Line 255 (d:\dev\projects\iree\third_party\llvm-project\llvm\include\llvm\ADT\ArrayRef.h:255)
iree-translate.exe!mlir::`anonymous namespace'::GetDotOperationType(mlir::mhlo::DotOp dot_op) Line 1371 (d:\dev\projects\iree\third_party\mlir-hlo\lib\Dialect\mhlo\transforms\legalize_to_linalg.cc:1371)
iree-translate.exe!mlir::`anonymous namespace'::DotOpConversion<1,mlir::linalg::MatvecOp>::matchAndRewrite(mlir::mhlo::DotOp op, mlir::mhlo::DotOpAdaptor adaptor, mlir::ConversionPatternRewriter & rewriter) Line 1414 (d:\dev\projects\iree\third_party\mlir-hlo\lib\Dialect\mhlo\transforms\legalize_to_linalg.cc:1414)
iree-translate.exe!mlir::OpConversionPattern<mlir::mhlo::DotOp>::matchAndRewrite(mlir::Operation * op, llvm::ArrayRef<mlir::Value> operands, mlir::ConversionPatternRewriter & rewriter) Line 427 (d:\dev\projects\iree\third_party\llvm-project\mlir\include\mlir\Transforms\DialectConversion.h:427)
iree-translate.exe!mlir::ConversionPattern::matchAndRewrite(mlir::Operation * op, mlir::PatternRewriter & rewriter) Line 1702 (d:\dev\projects\iree\third_party\llvm-project\mlir\lib\Transforms\Utils\DialectConversion.cpp:1702)
iree-translate.exe!mlir::PatternApplicator::matchAndRewrite(mlir::Operation * op, mlir::PatternRewriter & rewriter, llvm::function_ref<bool __cdecl(mlir::Pattern const &)> canApply, llvm::function_ref<void __cdecl(mlir::Pattern const &)> onFailure, llvm::function_ref<mlir::LogicalResult __cdecl(mlir::Pattern const &)> onSuccess) Line 201 (d:\dev\projects\iree\third_party\llvm-project\mlir\lib\Rewrite\PatternApplicator.cpp:201)
iree-translate.exe!`anonymous namespace'::OperationLegalizer::legalizeWithPattern(mlir::Operation * op, mlir::ConversionPatternRewriter & rewriter) Line 1985 (d:\dev\projects\iree\third_party\llvm-project\mlir\lib\Transforms\Utils\DialectConversion.cpp:1985)
iree-translate.exe!`anonymous namespace'::OperationLegalizer::legalize(mlir::Operation * op, mlir::ConversionPatternRewriter & rewriter) Line 1891 (d:\dev\projects\iree\third_party\llvm-project\mlir\lib\Transforms\Utils\DialectConversion.cpp:1891)
iree-translate.exe!`anonymous namespace'::OperationConverter::convert(mlir::ConversionPatternRewriter & rewriter, mlir::Operation * op) Line 2404 (d:\dev\projects\iree\third_party\llvm-project\mlir\lib\Transforms\Utils\DialectConversion.cpp:2404)
iree-translate.exe!`anonymous namespace'::OperationConverter::convertOperations(llvm::ArrayRef<mlir::Operation *> ops, llvm::function_ref<void __cdecl(mlir::Diagnostic &)> notifyCallback) Line 2453 (d:\dev\projects\iree\third_party\llvm-project\mlir\lib\Transforms\Utils\DialectConversion.cpp:2453)
iree-translate.exe!mlir::applyPartialConversion(llvm::ArrayRef<mlir::Operation *> ops, mlir::ConversionTarget & target, const mlir::FrozenRewritePatternSet & patterns, llvm::DenseSet<mlir::Operation *,llvm::DenseMapInfo<mlir::Operation *,void> > * unconvertedOps) Line 3267 (d:\dev\projects\iree\third_party\llvm-project\mlir\lib\Transforms\Utils\DialectConversion.cpp:3267)
iree-translate.exe!mlir::applyPartialConversion(mlir::Operation * op, mlir::ConversionTarget & target, const mlir::FrozenRewritePatternSet & patterns, llvm::DenseSet<mlir::Operation *,llvm::DenseMapInfo<mlir::Operation *,void> > * unconvertedOps) Line 3275 (d:\dev\projects\iree\third_party\llvm-project\mlir\lib\Transforms\Utils\DialectConversion.cpp:3275)
iree-translate.exe!mlir::iree_compiler::MHLO::`anonymous namespace'::ConvertMHLOToLinalgOnTensorsPass::runOnOperation() Line 367 (d:\dev\projects\iree\iree\compiler\InputConversion\MHLO\MHLOToLinalgOnTensors.cpp:367)
iree-translate.exe!mlir::detail::OpToOpPassAdaptor::run(mlir::Pass * pass, mlir::Operation * op, mlir::AnalysisManager am, bool verifyPasses, unsigned int parentInitGeneration) Line 393 (d:\dev\projects\iree\third_party\llvm-project\mlir\lib\Pass\Pass.cpp:393)
iree-translate.exe!mlir::detail::OpToOpPassAdaptor::runPipeline(llvm::iterator_range<llvm::pointee_iterator<std::unique_ptr<mlir::Pass,std::default_delete<mlir::Pass> > *,mlir::Pass> > passes, mlir::Operation * op, mlir::AnalysisManager am, bool verifyPasses, unsigned int parentInitGeneration, mlir::PassInstrumentor * instrumentor, const mlir::PassInstrumentation::PipelineParentInfo * parentInfo) Line 452 (d:\dev\projects\iree\third_party\llvm-project\mlir\lib\Pass\Pass.cpp:452)

this is the line from the top of the stack: https://github.com/tensorflow/mlir-hlo/blob/96fe02f272c4cf8c874a20497a902b59527ecab8/lib/Dialect/mhlo/transforms/legalize_to_linalg.cc#L1371

It is seem a typo, I think it  shoule be:
  if (rhs_shape.size() == 2 && lhs_shape.size() == 2 &&
      shape_matches(lhs_shape[1], rhs_shape[0]))

I will double check and fix it then.

Ah! Good eye :)
  if (lhs_shape.size() == 1 && rhs_shape.size() == 1 &&
      shape_matches(lhs_shape[0], rhs_shape[0])) {
    return DotOperationType::kVectorDot;
  }
  if (lhs_shape.size() == 2 && rhs_shape.size() == 1 &&
      shape_matches(lhs_shape[1], rhs_shape[0])) {
    return DotOperationType::kMatrixVector;
  }
  if (rhs_shape.size() == 2 && rhs_shape.size() == 2 &&
      shape_matches(lhs_shape[1], rhs_shape[0])) {
    return DotOperationType::kMatrixMatrix;
  }

lhs, rhs
lhs, rhs
rhs, rhs (the same check)

Ah! Good eye :)
  if (lhs_shape.size() == 1 && rhs_shape.size() == 1 &&
      shape_matches(lhs_shape[0], rhs_shape[0])) {
    return DotOperationType::kVectorDot;
  }
  if (lhs_shape.size() == 2 && rhs_shape.size() == 1 &&
      shape_matches(lhs_shape[1], rhs_shape[0])) {
    return DotOperationType::kMatrixVector;
  }
  if (rhs_shape.size() == 2 && rhs_shape.size() == 2 &&
      shape_matches(lhs_shape[1], rhs_shape[0])) {
    return DotOperationType::kMatrixMatrix;
  }

lhs, rhs lhs, rhs rhs, rhs (the same check)
I check the ir you posted. and find the dot is ilegalize. xla doc  states that the matrix-vector vector must be on the right. I think also need manual verify the dot op.

%364 = "mhlo.dot"(%363, %arg4) {precision_config = ["DEFAULT", "DEFAULT"]} : (tensor<87xf32>, tensor<87x32xf32>) -> tensor<32xf32> loc(#loc33)

Waiting to roll this to IREE head so we can validate.

Need I summit a patch integrated latest tf?

That fix is integrated now, thanks! I think we can close this issue, but there are new errors (for a new issue?)
https://colab.research.google.com/gist/ScottTodd/cdddd801a328111c1565ff6dfdccb3a5/brax-training-iree.ipynb
<unknown>:0: error: The following illegal operations still remain:
	mhlo.concatenate (count: 1)
	mhlo.broadcast_in_dim (count: 14)
	mhlo.divide (count: 6)
       ... (lots of these)

<unknown>:0: note: Pipeline failed while executing [`VerifyInputLegality` on 'builtin.func' operation: @main]: reproducer generated at `/tmp/iree/core-reproducer.mlir`

maybe some API change in iree-jax? -iree-input-type does seem to be set already... should be possible to repro outside of Python using the files that generated

Synced to head and ran iree-translate manually. Problem appears to be mhlo.tuple being used at the end of the program. Because we do not support tuples natively in IREE there is no good mapping to the type.

mhlo has flatten tuple pass.

There is a flag for enabling some extra legacy cleanups, including tuple elimination: --iree-input-type=xla. This is basically the same as "MHLO" but with extra cleanups that we hope to drop some day. Let us know if there is any limitation with this.

Sorry, just scrolled to the top and saw you are using the xla input type. This is likely a limitation of the flatten tuple pass, which would need to be extended to handle whatever case is in this program.

@ScottTodd
In the link you posted above, I always failed to run and couldn't download this file/tmp/iree/tf-mhlo.mlir. Can you paste some mhlo module information?This will help me fix this problem. :)

@ScottTodd In the link you posted above, I always failed to run and couldn't download this file/tmp/iree/tf-mhlo.mlir. Can you paste some mhlo module information?This will help me fix this problem. :)

Ah, sorry about that, @lipracer ! Changing the runtime type may help (Runtime -> Change runtime type -> set "Hardware accelerator" to None). This gist has the latest tf-mhlo.mlir and xla-iree-input.mlir: https://gist.github.com/ScottTodd/b99155a8cd4a7ee6afbd2aa344bfa2f4

Thanks and happy new year.:)

I run above IR module with iree-mhlo-flatten-tuples-in-cfg pass and it work well.I notic the input_type of pipe_line option is none.It seem the mhlo clean up not enable.

Looks like https://github.com/google/iree/pull/8502 -> https://github.com/google/iree/releases/tag/candidate-20220309.71 fixed this. The Brax program in that notebook compiles successfully now :D (with default CPU options, I'm poking at it further to see about using it with other backends)


Discord as Skype

I don't know if this should replace the original per se.. I just following the naming convention and that happened unintentionally. Not sure if we could do a discord-skype-full or something?

:tada: This is awesome! What we've done with other collisions is append a number (eg vim-emacs-01.svg and vim-emacs-02.svg)
Could we name the original discord-skype-01.svg and this one discord-skype-02.svg?

🎉 This is awesome! What we've done with other collisions is append a number (eg vim-emacs-01.svg and vim-emacs-02.svg)
Could we name the original discord-skype-01.svg and this one discord-skype-02.svg?

Done :)

Related parent PR for cross-referencing: https://github.com/mkrl/misbrands/pull/90


Need Latest AIConfiguration

Hi Team,
Can we have latest AIConfiguration which will connect to Dialogflow V2 API version.
Thanks and Regards,
Shishir Bhujbal

Hello Shishir,
May I suggest you just to use Dialogflow V2 client instead?


Not updating apps on LineageOS

Anyone with LineageOS can reproduce?
(submitted via the app)

Does not update apps installed by other app stores on LineageOS

another message received

Which OS version? Which device?

LineageOS 21 / Android 14 updates apps no problem.

@xjmzx curious did you consider, and compare grapheneOS to LineageOS

I would think most people using LineageOS is because they don't own a Pixel

@xjmzx curious did you consider, and compare grapheneOS to LineageOS

I would like to try Graphene, no pixel.
Lineage seems to work well so far.

Working for other LineageOS users too, closing for now


Rewrite the codebase to TypeScript

Rewrite the codebase (src/) to TypeScript.
Make sure that both distribution options (JavaScript library and NPM package) will work after a codebase rewrite.
Testing details:

you can test the JavaScript library by running gulp watch and opening demo in browser at localhost:3000,
you can test the NPM package by testing in other public repository e.g. Typopo for VS Code,
as a bonus, you can write new test for these checks.

@greegus asked to look at this issue.

Let's lower the priority on this one and close it for now.


Bucket Sort

Overview

Write code for Bucket Sort in C++.
Add that code into a directory named "Sorting", if the directory doesn't exist, create one and add the code file.

Please take care of these things while creating the pull request.

The code file was added to the correct directory.
Code quality looks good.
The programming language used is the same as mentioned in the issue.

Hey , I can work on this issue

please assign me this issue


fix: Redirect to notification targetHref in case notification type is not supported

Description
Redirect to notification targetHref in case notification type is not supported.
We do the same in Force: https://github.com/artsy/force/blob/85e9dc5b077b5b8e166895a59b208b2d242e74d0/src/Components/Notifications/Notification.tsx#L49
PR Checklist

[ ] I have tested my changes on iOS and Android.
[ ] I hid my changes behind a feature flag, or they don't need one.
[ ] I have included screenshots or videos, or I have not changed the UI.
[ ] I have added tests, or my changes don't require any.
[ ] I added an app state migration, or my changes do not require one.
[ ] I have documented any follow-up work that this PR will require, or it does not require any.
[ ] I have added a changelog entry below, or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

iOS user-facing changes

Android user-facing changes

Dev changes

Need help with something? Have a look at our docs, or get in touch with us.

Warnings

:warning:

❌ An error occurred while validating your changelog, please make sure you provided a valid changelog.

  Generated by :no_entry_sign: dangerJS against 64d5ddd88ffffa7bbcb3c8bc059bae9b926922ef


Why transaction commit closes db connection?

I wonder what is reason for transaction to close database connection on commit? I'm working on rather intricate scenario where I have to synchronise two requests access to the database and this behaviour breaks my logic.
I planned to use 'sp_getapplock/sp_releaseapplock' MS SQL procedures to implement database mutex which guards access to shared resource. I implemented 'class DbMutex : IDisposable' which acquire lock in constructor and release it in Dispose.  Use case for this class looks like this:
using (var t = dbContext.Database.StartTransaction()) {
  //acquire lock
  using (var dbMutex = new DbMutex(dbContext.GetConnection(), t, 'shared_lock_id')) {
    //this code is executed only by one request at a time
    if (MySharedResourceUpdated()) {
      //other request have updated my shared resource
      return;
    }
    UpdateSharedResource();
    //commit transaction so other request can see results
    t.Commit();
  }
  //release lock in Dispose
}

This code breaks since t.Commit() closes transaction and then my DbLock.Dispose fails to execute 'sp_releaseapplock' on closed connection.
Further technical details
EF Core version: 1.1.0
Database Provider: Microsoft.EntityFrameworkCore.SqlServer
Operating system: Windows 10
IDE: Visual Studio 2015

When EF opens a connection, it will close it when it thinks it is done. In your case, you have additional code, that EF isn't aware of, that requires the connection to be left open. There are open/close APIs on the connection to allow you to control this manually. EF won't close a connection if it didn't open it.


Option to disable font variants

Could you provide simple option that could be set in init.vim so it would disable italics?

Hey @andynameistaken,
Melange compiles Lua to vimscript to make the colorscheme usable in vim8. Adding this to the compile step would be kind of tedious.
I could however, add an option for the Lua file, but you'd have to use Lush yourself to load the colorscheme:
lua require("lush")(require("melange"))

Maybe you could set your package manager to run make and compile a colorscheme without color variants.
I'm currently making rewriting some thing in the colorscheme, I'll add this after that, and check if compiling would work.

I added this a couple of weeks ago but forgot to write a follow up.
You can now disable font variants by setting the global variable melange_enable_font_variants to 0, and sourcing the colorscheme with Lush. To make the change persistent, you should call :lua require("melange.build").build().
In Lua:
function melange_no_variants()
    vim.g.melange_enable_font_variants = 0    -- disable font variants
    require("lush")                           -- make sure Lush is installed
    require("melange.build").build()          -- Compile to vimscript
end

You'd only need to call this function once. Afterwards, :colorscheme melange should set the colorscheme without font variants.

Thanks for the option and for the theme in general - it is really nice thing between gruvbox and earthsong and fixes  visual issues of those two: it doesn't use red for 'normal' information (problem with gruvbox) and doesn't use white too much (earthsong)

I will say after trying out the recommended IBM Plex Mono font instead of the Consolas font, italics look much better. Here's Consolas:

vs. IBM Plex Mono:


Added optional SIMD implementation for /blocks/clamp

Will automatically compile if supported by Pothos installation and XSIMD found
Added testing function to expand test data to make sure SIMD is tested

See: https://github.com/pothosware/PothosCore/pull/215

Worth noting that xsimd is BSD-3 (I probably should relicense those files...). Does this affect bringing it in?

has conflicts :-D

Fixed, just diverging CMakeLists


fix: first IP of CIDR range

fix first IP from CIDR notation input.
CIDR <IP_ADDRESS>/21 :
Correct CIDR range: <IP_ADDRESS> ~ <IP_ADDRESS>

The firstIP variable specifies the lower IP address within the range for the package.
What are your thoughts on this?

CIDR / subnet range should be constant wichever condition is, which is /21 accounts for 2048 hosts.
therefore first IP should be first IP from designated within specified CIDR range (/21), not based on their IP inputs (<IP_ADDRESS>).
If the inputs on the middle of so called 'subnets' (<IP_ADDRESS>/21), it should be an single IP address, not IP address range, therefore it put /21 (subnets) higher priority than single IP address, if specified, and treadted them as subnet, therefore we need to convert specified IP into their respective network ID, <IP_ADDRESS>/21.

Okay, thank you for your input. I agree with you.
I believe the firstIP parameter is no longer needed since we can obtain it from firstIp.Mask(cidr.Mask). Could you please remove the firstIp parameter from both the GenerateIPListFromCIDR function and the GenerateIPListFromCIDRString function?

on WIP

Okay, thank you for your input. I agree with you.
I believe the firstIP parameter is no longer needed since we can obtain it from firstIp.Mask(cidr.Mask). Could you please remove the firstIp parameter from both the GenerateIPListFromCIDR function and the GenerateIPListFromCIDRString function?

firstIp net.IP needed to assign first IP address, from firstIp.Mask(cidr.Mask), while its same as cidr.IP, but results different, not sure why,
but if using cidr.IP, its looped through 192.x.y.z as class A addresses, not CIDR /21 looped through 192.168.24.x/21

I'm facing a similar issue.
The cidr.IP in the cidr variable is incremented when cidr.IP is utilized.

However, if using ip.Mask(cidr.Mask) instead, it is normal

The problem lies in passing the pointer of cidr to the incr() function, which causes the IP to be incremented and results in an infinite loop.
I think it is possible to eliminate the firstIP parameter in these functions. Instead, we can use ip.Mask(cidr.Mask) as a firstIp, where the ip variable gets from the net.ParseCIDR() function.

lgtm


Hard to understand when when example doesn't have null safety

With the main channel (i.e. after Dart 3), on a package where the package itself has null safety but the example doesn't:
No clue here:
mit-macbookpro7:another_flushbar-1.12.29 mit$ flutter pub get
Running "flutter pub get" in another_flushbar-1.12.29...
Resolving dependencies...
  flutter_lints 1.0.4 (2.0.1 available)
  lints 1.0.1 (2.0.1 available)
Got dependencies!
Resolving dependencies in ./example...
Resolving dependencies in ./example failed. For details run `flutter pub get --directory ./example`

Trying manually in the example
mit-macbookpro7:another_flushbar-1.12.29 mit$ cd example/
mit-macbookpro7:example mit$ flutter pub get
Running "flutter pub get" in example...
Resolving dependencies...
The current Dart SDK version is 3.0.0-55.0.dev.

Because another_flushbar_example requires SDK version >=2.7.0 <3.0.0, version solving failed.

This is expecially bad as flutter pub get --directory ./example doesnt' work right now: https://github.com/dart-lang/pub/issues/3717

Resolving: after discussion we think this behavoir is fine as long as we get https://github.com/dart-lang/pub/issues/3717 fixed.


HTTP/3 patch bugs with h2load HTTP/2 benchmark tests ?

I might have found a bug doing h2load HTTP/2 tests

Nginx 1.16.1 with Cloudflare HTTP/3 patch with dual RSA 2048bit + ECDSA 256bit SSL certificates configured seems to fail on most RSA cipher tests and only pass when h2load testing against ECDSA ?? Raw h2load results here.
Using normal Nginx 1.17.4 without HTTP/3 patch, both RSA and ECDSA cipher tested h2load HTTP/2 tests pass fine. Raw h2load results here
Nginx HTTP brolti encoded requests via h2load nearly all fail on Nginx 1.16.1 with CF HTTP/3 patch too using Google's ngx_brotli nginx module.
Or might be h2load bug as when I specify specific --ciphers fails

h2load -t1 --ciphers=ECDHE-RSA-AES128-GCM-SHA256 -H 'Accept-Encoding: gzip' -c100 -n1000 https://http2.domain.com/
starting benchmark...
spawning thread #0: 100 total client(s). 1000 total requests

finished in 6.67ms, 0.00 req/s, 0B/s
requests: 1000 total, 0 started, 0 done, 0 succeeded, 1000 failed, 1000 errored, 0 timeout
status codes: 0 2xx, 0 3xx, 0 4xx, 0 5xx
traffic: 0B (0) total, 0B (0) headers (space savings 0.00%), 0B (0) data
                     min         max         mean         sd        +/- sd
time for request:        0us         0us         0us         0us     0.00%
time for connect:        0us         0us         0us         0us     0.00%
time to 1st byte:        0us         0us         0us         0us     0.00%
req/s           :       0.00        0.00        0.00        0.00   100.00%

not specifying it passes as it's preferencing ECDSA SSL cert in nginx dual RSA + ECDSA SSL cert config.
h2load -t1 -H 'Accept-Encoding: gzip' -c100 -n1000 https://http2.domain.com/
starting benchmark...
spawning thread #0: 100 total client(s). 1000 total requests
TLS Protocol: TLSv1.2
Cipher: ECDHE-ECDSA-AES128-GCM-SHA256
Server Temp Key: ECDH P-256 256 bits
Application protocol: h2
progress: 10% done
progress: 20% done
progress: 30% done
progress: 40% done
progress: 50% done
progress: 60% done
progress: 70% done
progress: 80% done
progress: 90% done
progress: 100% done

finished in 91.50ms, 10929.08 req/s, 25.08MB/s
requests: 1000 total, 1000 started, 1000 done, 1000 succeeded, 0 failed, 0 errored, 0 timeout
status codes: 1000 2xx, 0 3xx, 0 4xx, 0 5xx
traffic: 2.29MB (2405900) total, 274.41KB (281000) headers (space savings 24.05%), 2.00MB (2102000) data
                     min         max         mean         sd        +/- sd
time for request:     1.02ms     19.33ms      3.83ms      3.57ms    94.90%
time for connect:    34.17ms     56.90ms     45.53ms      9.83ms    51.00%
time to 1st byte:    53.50ms     60.16ms     56.97ms      2.20ms    58.00%
req/s           :     111.42      133.51      119.43        6.19    67.00%

but don't have problems with unpatched version or maybe Nginx 1.17.4 vs 1.16.1 related?

Have you tried unpatched 1.16.1? Also, what's the NGINX configuration?

Each Nginx configuration is linked in raw h2load results links above.
But I think I recall now why it is failing. Nginx dual RSA + ECDSA SSL certificated support which was added in Nginx 1.11 only works with OpenSSL. It doesn't work with BoringSSL built Nginx. I totally forgot about this LOL. Silly me 😊
So it isn't a bug just Nginx doesn't support dual SSL certificates when built against BoringSSL !

Its sounds like this is not an issue with quiche or the H3 patch, so I'm going to close this.

Yeah sorry for the flash alert 😊

@centminmod have you thought about applying the pending patches to OpenSSL and trying that?


After running the tool, still have qualsys issue

So I was receiving this error message, but it still persists after using this tool.
"This server's certificate chain is incomplete. Grade capped to B."
sudo ./cert-chain-resolver/src/cert-chain-resolver.sh -o domain.bundle.pem domain.crt

I'm wondering if the problem is that the CA certs contain an anchor as after running this tool I've gone from having a single extra-download with one path, to two paths, one with one extra-download, the other with two extra downloads... I've never encountered this issue before, normally I can just cat the CA files and I get an A+...
CentOS7 Apache2+mod_ssl
TIA

Hi, can you attach the original certificate, so I can debug it?

Sorry, but that would be problematic. I solved the problem by switching to my regular SSL certificate provider, and got a refund on the cert in question

That's no problem, even if the certificate is revoked and not used on any site, I can use it for debugging. Just send me the domain.crt file, without private key. If you don't want to publish it here, you can find the email on my personal website.


popup vim-clap floating window when quickfix needs openning

Is there any way to use vim-clap as an instead of quickfix window.
I know Clap quickfix, but it doesn't trigger automatically.
Many plugins like nvim-lsp send results to quickfix and open automatically,  I need to close quickfix and then open Clap quickfix again.

I actually have no idea about what your request is, can you describe some more concrete use cases? BTW vim-clap is not intended to be an alternative to quickfix window.

I actually have no idea about what your request is, can you describe some more concrete use cases? BTW vim-clap is not intended to be an alternative to quickfix window.

Em... I indeed to find some way to make vim-clap to an alternative to quickfix window.
Some plugin or built-in command sends result to quickfix windows and pop up it, I want to it can be send to vim-clap.

Em... I indeed to find some way to make vim-clap to an alternative to quickfix window.
Some plugin or built-in command sends result to quickfix windows and pop up it, I want to it can be send to vim-clap.

You can certainly do that by preparing your own data source and then feed them into vim-clap. In another word, just write your own clap provider.

You can certainly do that by preparing your own data source and then feed them into vim-clap. In another word, just write your own clap provider.

Closing as basically there is no general solution, if you want to show something using vim-clap, write a clap provider.

Closing as basically there is no general solution, if you want to show something using vim-clap, write a clap provider.


v1.9.3 Release

Overview

Test Notes/Sample Code

Notes:

Skipping this release and going straight to v2.0.0. @lauramaxwell let's discuss reassigning issues from this release that are still relevant to the v2.0.0 release.


Cardio duration

I did not initially realize that Flexify could track cardio. I took photos of all of my stop screens and I will enter them in.
Distance and incline seem good, but duration does not make sense to me. Is duration minutes? It looks like I can enter a decimal value, but I would expect to see HH:MM:SS format or some variation of that. To accurately log time, I would need to divide seconds by 60 to get a decimal if I understand this correctly that duration is minutes.

Yeah I didn't put much thought into the cardio feature since i've only recently started doing cardio 😢
It's stored as a float so i'll probably treat it as milliseconds under the hood, and then make visual pickers for minutes/seconds in the relevant places like how we are doing it in Settings.

That probably makes sense to keep it as a float. Walking/jogging/climbing would make sense to track in minutes while plank might be seconds. Or the user could be presented with a HH:MM:SS sort of input that is converted to float.


When I try to run PowerPaint_outpaint, the workflow, this computer at work is working fine. But the one at my home, the command line message pops up after this paragraph and then it just keeps not moving And then it just keeps stopping moving. I don't know what it is.

[rgthree] Using rgthree's optimized recursive execution.
clip missing: ['text_projection.weight']
PowerPaint base CLIP file:  D:\ComfyUI\models\clip\model.fp16.safetensors
Successfully add external embeddings: P_ctxt, P_shape, P_obj. current
Successfully add trainable external embeddings: P_ctxt, P_shape, P_obj current
It's going to get stuck at this node that I've circled in this red box and not move.

help！！

And the company's computer will be able to successfully execute below this red box

@nullquant

It is stuck at loading PowerPaint CLIP stage. No errors? Strange, may be something wrong with pytorch_model.bin file?

Diagnostics-1715256083.log

It worked, re-downloaded and replaced the file and it's up and running, it's awesome!


Sharing meta

Added tags that render nice HTML meta tags for sharing:
<meta property="og:title" content="Home" />
<meta property="og:description" content="COVID-19 Data Portal for Swedish Research" />
<meta property="og:type" content="website" />
<meta property="og:url" content="http://localhost:1313/" />
<meta property="og:image" content="http://localhost:1313/img/site_logo/CV19DP_logo_oneliner_SWE.png"/>
<meta property="og:site_name" content="COVID-19 Data Portal Sweden" />

<meta name="twitter:card" content="summary_large_image"/>
<meta name="twitter:image" content="http://localhost:1313/img/site_logo/CV19DP_logo_oneliner_SWE.png"/>

<meta name="twitter:title" content="Home"/>
<meta name="twitter:description" content="COVID-19 Data Portal for Swedish Research"/>

Note that we should set description in each page front-matter for this to work nicely (else I think it pretty much just dumps most of the page contents).
Can also set page-specific images in the front-matter for sharing if we want to.

Are the localhost:1313 entries in the above text due to running the hugo server? I didn't see any mention in the actual code about it.

Yup, that was just a copy + paste example of the output from running locally. Links should be valid in deployment.