[Feature] Support skip reconcile application

Is your feature request related to a problem? Please describe.

In the case that we do not want KubeVela controller to reconcile application, we can support an annotation "controller.core.oam.dev/skip-reconcile". When application carries this annotation and the value is "true", the application controller will ignore it.
Describe the solution you'd like

Describe alternatives you've considered

Additional context

/assign

Label and functions are added here https://github.com/kubevela/pkg/pull/68.

Label and functions are added here kubevela/pkg#68.
ok

Label and functions are added here kubevela/pkg#68.
ok

Sorry, we have just raised the priority of this issue so it is just implemented in #5732. You can try another good first issue. Apologize for the late response.


Secondary cache

Add secondary cache interface

Entries evicted from DRAM will be inserted into secondary cache
Get from secondary cache will move entry to DRAM first and return

Add a secondary cache implentmention based on CacheLib Hybrid Cache

Implement SOC and LOC
Use Direct IO because data already buffered
No cache persistence in first version

Codecov Report
Patch coverage: 77.32% and project coverage change: -6.69 :warning:

Comparison is base (c516ee0) 92.22% compared to head (bc6a128) 85.54%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #21      +/-   ##
==========================================
- Coverage   92.22%   85.54%   -6.69%
==========================================
  Files          13       24      +11
  Lines        1454     2470    +1016
==========================================
+ Hits         1341     2113     +772
- Misses         76      256     +180
- Partials       37      101      +64

Impacted Files
Coverage Δ

internal/entry.go
95.06% <ø> (ø)

serializers.go
0.00% <0.00%> (ø)

internal/nvm/preallocate/preallocate.go
20.00% <20.00%> (ø)

internal/nvm/preallocate/preallocate_linux.go
38.09% <38.09%> (ø)

internal/persistence.go
85.29% <50.00%> (-5.34%)
:arrow_down:

internal/nvm/nvm.go
61.37% <61.37%> (ø)

internal/nvm/serializers/memory.go
65.51% <65.51%> (ø)

nvm.go
69.76% <69.76%> (ø)

internal/nvm/blockcache.go
75.00% <75.00%> (ø)

internal/nvm/bighash.go
78.92% <78.92%> (ø)

... and 10 more

... and 1 file with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


pre-commit, dependabot and CI workflows

Closes #3
Adds:

pre-commit
As a versioned dependency in a requirements.txt file.
As a CI job.
Hooks for:

Check JSON
Check YAML
Fix end-of-file
Remove trailing whitespace
Run markdownlint, fixing files in-place, where possible.
Run ci-cd's "Update Landing Page (index.md) for Documentation" hook.

Dependabot
Update dependencies (currently only pre-commit) in requirements.txt.
Update GitHub Actions used in CI workflows.
CI workflows

CI - Test
Runs for all PRs and pushes to main.
Runs the pre-commit job, which runs pre-commit for the whole repository and fails if changes are detected.
CI - Activate auto-merging for automated PRs
Runs when a PR is opened by Dependabot that targets the main branch.
NOTE: Using a "new" pathway to activate auto-merging, which might fail - keep tabs on this when a Dependabot PR is opened.

Also:

Adds a markdownlint configuration file.
This sets the line length to 88 characters for code blocks and headers.
Otherwise it sets the line length to an arbitrarily large value to ensure we can manually implement the "1-line-per-sentence" rule.
Fix JSON files and markdown files according to outcome of running pre-commit.

Potentially missing from this PR:

[ ] Write installation and usage instructions in README or a new documentation file.

it looks fine. Let's find out first how it should be used and then write the documentation.
I wonder if the text you wrote in your pull request should be put in a documentation file. It could be helpful to have written down (for humans) what the pipelines are doing, or what is configure in plain text. I am thinking about educating new developer.


BUG: 'React' refers to a UMD global, but the current file is a module. Consider adding an import instead. [2686]

Description
This arises when import React from 'react is not used in a file.

This is a false negative due to incorrect config. React v17 and above don't require react to be imported in every file.
Related Tickets & Documents
somewhat relates to [#646]. The fix should be easy, just need to add "jsx": "react-jsx" to the tsconfig.json
stack overflow discussion

Some errors with rollup after changing to react-jsx. As usual, never as easy as it seems. hmm.


feat(view): add option for dynamic height

Description
Adds an option which disables the behavior of the window trying to be below the cursorline.
Need to add some docs in the config and perhaps change the name.
Edit: can be configured per mode as well now. Now there also needs to be a discussion about the default value :)
Related Issue(s)
Fixes https://github.com/folke/which-key.nvim/issues/649
Screenshots
Before:

After:

Added opts.win.no_overlap


Consider compiler-erroring instead of panicking in DST APIs

This issue goes partway towards addressing #325.
For example, Ref::from_prefix_with_trailing_elements documents a panic if Self is a ZST. However, the ZST-iness (pronounced "zestiness") of a type relevant here is always a static property and can be reported via postmonomorphization error.
The advantages of this are several-fold:

it allows us to maintain the consistent policy that our APIs do not panic (with the exception of panics in user-supplied TryFromBytes validators)
it is forwards compatible with us relaxing the compile error to a panic in future non-breaking releases
it may have optimizer advantages

Postmonomorphization errors currently have a bit of a 'weirdness' factor (and an unwieldy name), but I suspect that will change. The recent stabilization notes for inline const exprs cite ZST-checking as a motivating case of what the feature will make possible.

Note that this partially addresses #325. I think we should leave #325 open and continue the main discussion there, but it's good to have this as a separate issue so we can block #671 on it.


Fix missing description and http --> https

Docs Build status updates of commit fc5d865:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/windows/dhcpserver/Get-DhcpServerv6Statistics.md
:white_check_mark:Succeeded

docset/winserver2012-ps/dhcpserver/Get-DhcpServerv6Statistics.md
:white_check_mark:Succeeded

docset/winserver2012r2-ps/dhcpserver/Get-DhcpServerv6Statistics.md
: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.

Docs Build status updates of commit 6898d70:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/windows/dhcpserver/Get-DhcpServerv6Statistics.md
:white_check_mark:Succeeded

docset/winserver2012-ps/dhcpserver/Get-DhcpServerv6Statistics.md
:white_check_mark:Succeeded

docset/winserver2012r2-ps/dhcpserver/Get-DhcpServerv6Statistics.md
: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.

@nenonix @JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@get-itips @tseward
Peer review needed for this PR. Thanks.

Docs Build status updates of commit fcf0db3:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/windows/dhcpserver/Get-DhcpServerv6Statistics.md
:white_check_mark:Succeeded

docset/winserver2012-ps/dhcpserver/Get-DhcpServerv6Statistics.md
:white_check_mark:Succeeded

docset/winserver2012r2-ps/dhcpserver/Get-DhcpServerv6Statistics.md
: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.


Force it to every route?

Hi,
is it possible to force the last scrolling position to every route? At the moment I need to add #maintainScroll hash to every url to make it work.

Hey! I'm confused, could you please clarify? What is the behaviour you are trying to achieve?
If #maintainScroll was added on every route, then that would defeat the purpose of this package

@kdanninger The examples aren't working on my devices (iPhone and Mac, Safari and Chrome). It only works if I explicit add the "#maintainScroll" hash to every link. When I install tomwasd:flow-router-autoscroll everything works fine, even without the hash.

Weird, the example apps are working for me.
What exactly is not working?

If I click on a link and then go back, I'm back at the top of the page and not at the last position where I've leaved the route.

Hello @dnish, at OK GROW we no longer use this package in our projects. We will no longer be dedicating the resources to maintaining this package. If you would like to support this package, feel free to fork it.


Scan Report Fields

Example: /scanreports/226/tables/840/fields
Linked to #681, should be able to mostly reuse the frontend components and code from there.
Frontend Tasks:

[ ] Recreate the existing UI
[ ] Define types in Typescript for what is needed
[ ] Add pagination
[ ] Add filtering
[ ] Add sorting
[ ] Add the "Add Concept" POST
[ ] Review

Backend Tasks:

[ ] New API endpoint
[ ] With pagination, filtering, and sorting
[ ] Testing
[ ] Add the URLS to proxy it
[ ] Review

Should also fix #677

Hello @AndyRae @AndrewThien , I can see that the new UI is back, however the sorting function on 'value' is not there anymore. Is this intentional?

Hi @erummas - yes it's disabled for now, as it's potentially one of the areas that causes high demand (and therefore downtime), as the legacy UI didn't have this as a feature.
I am hopeful to be able to bring it back quite soon though, we're just providing a graceful incremental approach right now.


DSD-545: POC using the Figma API.

Fixes JIRA ticket DSD-545
This PR does the following:

Uses the Figma API to fetch design tokens from the NYPL Figma file.
Follow the instructions in figmaKeys.example.js to add your Figma access token.
Run node figmaApi.js to execute the script.
Follow the link in the JIRA ticket for the walkthrough and questions document.

How has this been tested?
Locally.
Checklist:

[ ] I have updated the Storybook documentation accordingly.
[ ] All new and existing tests passed.

Front End Review:

[ ] View the example in Storybook

Tugboat has finished building the preview for this pull request!
Link:

https://pr752-tcdnk2cjzk0vhsxiui45nqkh8e9tbxle.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/6184510cde4970a9cdd89a6e

Tugboat has finished building the preview for this pull request!
Link:

https://pr752-tcdnk2cjzk0vhsxiui45nqkh8e9tbxle.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/6184510cde4970a9cdd89a6e

Tugboat has finished building the preview for this pull request!
Link:

https://pr752-tcdnk2cjzk0vhsxiui45nqkh8e9tbxle.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/6184510cde4970a9cdd89a6e

Tugboat has finished building the preview for this pull request!
Link:

https://pr752-tcdnk2cjzk0vhsxiui45nqkh8e9tbxle.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/6184510cde4970a9cdd89a6e

Tugboat has finished building the preview for this pull request!
Link:

https://pr752-tcdnk2cjzk0vhsxiui45nqkh8e9tbxle.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/6184510cde4970a9cdd89a6e

Tugboat has finished building the preview for this pull request!
Link:

https://pr752-tcdnk2cjzk0vhsxiui45nqkh8e9tbxle.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/6184510cde4970a9cdd89a6e

Tugboat has finished building the preview for this pull request!
Link:

https://pr752-tcdnk2cjzk0vhsxiui45nqkh8e9tbxle.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/6184510cde4970a9cdd89a6e

Tugboat has finished building the preview for this pull request!
Link:

https://pr752-tcdnk2cjzk0vhsxiui45nqkh8e9tbxle.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/6184510cde4970a9cdd89a6e

Tugboat has finished building the preview for this pull request!
Link:

https://pr752-tcdnk2cjzk0vhsxiui45nqkh8e9tbxle.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/6184510cde4970a9cdd89a6e

Tugboat has finished building the preview for this pull request!
Link:

https://pr752-tcdnk2cjzk0vhsxiui45nqkh8e9tbxle.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/6184510cde4970a9cdd89a6e

Tugboat has finished building the preview for this pull request!
Link:

https://pr752-tcdnk2cjzk0vhsxiui45nqkh8e9tbxle.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/6184510cde4970a9cdd89a6e

Tugboat has finished building the preview for this pull request!
Link:

https://pr752-tcdnk2cjzk0vhsxiui45nqkh8e9tbxle.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/6184510cde4970a9cdd89a6e

Tugboat has finished building the preview for this pull request!
Link:

https://pr752-tcdnk2cjzk0vhsxiui45nqkh8e9tbxle.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/6184510cde4970a9cdd89a6e

Tugboat has finished building the preview for this pull request!
Link:

https://pr752-tcdnk2cjzk0vhsxiui45nqkh8e9tbxle.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/6184510cde4970a9cdd89a6e

Tugboat has finished building the preview for this pull request!
Link:

https://pr752-tcdnk2cjzk0vhsxiui45nqkh8e9tbxle.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/6184510cde4970a9cdd89a6e

Tugboat has finished building the preview for this pull request!
Link:

https://pr752-tcdnk2cjzk0vhsxiui45nqkh8e9tbxle.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/6184510cde4970a9cdd89a6e

Tugboat has finished building the preview for this pull request!
Link:

https://pr752-tcdnk2cjzk0vhsxiui45nqkh8e9tbxle.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/6184510cde4970a9cdd89a6e

Tugboat has finished building the preview for this pull request!
Link:

https://pr752-tcdnk2cjzk0vhsxiui45nqkh8e9tbxle.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/6184510cde4970a9cdd89a6e

Tugboat has finished building the preview for this pull request!
Link:

https://pr752-tcdnk2cjzk0vhsxiui45nqkh8e9tbxle.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/6184510cde4970a9cdd89a6e

Tugboat has finished building the preview for this pull request!
Link:

https://pr752-tcdnk2cjzk0vhsxiui45nqkh8e9tbxle.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/6184510cde4970a9cdd89a6e

Tugboat has finished building the preview for this pull request!
Link:

https://pr752-tcdnk2cjzk0vhsxiui45nqkh8e9tbxle.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/6184510cde4970a9cdd89a6e

Tugboat has finished building the preview for this pull request!
Link:

https://pr752-tcdnk2cjzk0vhsxiui45nqkh8e9tbxle.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/6184510cde4970a9cdd89a6e

Tugboat has finished building the preview for this pull request!
Link:

https://pr752-tcdnk2cjzk0vhsxiui45nqkh8e9tbxle.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/6184510cde4970a9cdd89a6e

Tugboat has finished building the preview for this pull request!
Link:

https://pr752-tcdnk2cjzk0vhsxiui45nqkh8e9tbxle.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/6184510cde4970a9cdd89a6e

Tugboat has finished building the preview for this pull request!
Link:

https://pr752-tcdnk2cjzk0vhsxiui45nqkh8e9tbxle.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/6184510cde4970a9cdd89a6e

Tugboat has finished building the preview for this pull request!
Link:

https://pr752-tcdnk2cjzk0vhsxiui45nqkh8e9tbxle.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/6184510cde4970a9cdd89a6e

Tugboat has finished building the preview for this pull request!
Link:

https://pr752-tcdnk2cjzk0vhsxiui45nqkh8e9tbxle.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/6184510cde4970a9cdd89a6e

Tugboat has finished building the preview for this pull request!
Link:

https://pr752-tcdnk2cjzk0vhsxiui45nqkh8e9tbxle.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/6184510cde4970a9cdd89a6e

Tugboat has finished building the preview for this pull request!
Link:

https://pr752-tcdnk2cjzk0vhsxiui45nqkh8e9tbxle.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/6184510cde4970a9cdd89a6e

Tugboat has finished building the preview for this pull request!
Link:

https://pr752-tcdnk2cjzk0vhsxiui45nqkh8e9tbxle.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/6184510cde4970a9cdd89a6e

Tugboat has finished building the preview for this pull request!
Link:

https://pr752-tcdnk2cjzk0vhsxiui45nqkh8e9tbxle.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/6184510cde4970a9cdd89a6e

Tugboat has finished building the preview for this pull request!
Link:

https://pr752-kg3zsaf6bikflmuwhqfqfwlsfdhymu89.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/626303e88a4c5796e7bcf719

Tugboat has finished building the preview for this pull request!
Link:

https://pr752-kg3zsaf6bikflmuwhqfqfwlsfdhymu89.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/626303e88a4c5796e7bcf719

Tugboat has finished building the preview for this pull request!
Link:

https://pr752-kg3zsaf6bikflmuwhqfqfwlsfdhymu89.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/626303e88a4c5796e7bcf719

Tugboat has finished building the preview for this pull request!
Link:

https://pr752-kg3zsaf6bikflmuwhqfqfwlsfdhymu89.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/626303e88a4c5796e7bcf719

Tugboat has finished building the preview for this pull request!
Link:

https://pr752-kg3zsaf6bikflmuwhqfqfwlsfdhymu89.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/626303e88a4c5796e7bcf719

Tugboat has finished building the preview for this pull request!
Link:

https://pr752-kg3zsaf6bikflmuwhqfqfwlsfdhymu89.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/626303e88a4c5796e7bcf719

too old and don't want to fix conflicts 👿


Feature request: option to save requests

When updating a request, it is auto-saved, so the next time we go back to it, it is at the latest version used.
This behavior does not work when experimenting with a saved request and when we do not want to save the changes made to it. We have to undo changes made to experiment to make sure that the request is persisted in a usable state.
Is there a way to not auto-save the requests? (and have save/cancel button appear when it is changed?)

I have the same mistake, make same changes in any request, close then when open again doesn't save my last changes 🙄


Add support function execute #14

Coverage decreased (-0.5%) to 95.97% when pulling c3c06e43d7a272e9d75a541f2975b1d06c58be2e on vitalvas:master into edfc348898cc8c6778292512bcdd0dd306d75ac7 on roistat:master.

Coverage increased (+0.03%) to 96.474% when pulling 7e2f57497a4e40c6a6bddd244a96424ad2742b0c on vitalvas:master into edfc348898cc8c6778292512bcdd0dd306d75ac7 on roistat:master.

Coverage increased (+0.03%) to 96.474% when pulling 876dd03c2af410c1731752220619f14c101583bb on vitalvas:master into edfc348898cc8c6778292512bcdd0dd306d75ac7 on roistat:master.


Create tea.yaml

please merge my code

This is spam. Please close and report this user. Reference https://github.com/TryGhost/Ghost/pull/19743


Connect-wugserver using -Username and -Password still prompts for credentials

Prerequisites
Put an X between the brackets on each line to confirm you have completed them:

[X ] The issue is still present in the latest version of the module.
[X ] The issue has not been previously reported.

Describe the bug
When supplying the username and password using the proper parameters, the credential input still pops up.
To Reproduce
Steps to reproduce the behavior:

Open Powershell
Type command "connect-wugserver -protocol https -serverUri fqdnofserver -Username "username" -Password "password"
Prompt appears for username and password even though already supplied

Expected behavior
Should connect to WhatsUp Gold server without prompting for credentials.
System Details

Operating System: Windows
Operating System Version (Version and Build, eg Windows 10 Build 1709):  Windows 11 Build 22621.4037
PowerShell Version: 5.1.22621.3958
Architecture for PowerShell Session: 64bit
Is this system an Azure Automation Worker?: No
Other loaded modules: [Include the output of Get-Module]
ModuleType Version    Name                                ExportedCommands

Manifest   <IP_ADDRESS>    Microsoft.PowerShell.Management     {Add-Computer, Add-Content, Checkpoint-Computer, Clear-Content...}
Manifest   <IP_ADDRESS>    Microsoft.PowerShell.Security       {ConvertFrom-SecureString, ConvertTo-SecureString, Get-Acl, Get-AuthenticodeSignature...}
Manifest   <IP_ADDRESS>    Microsoft.PowerShell.Utility        {Add-Member, Add-Type, Clear-Variable, Compare-Object...}
Script     2.0.0      PSReadLine                          {Get-PSReadLineKeyHandler, Get-PSReadLineOption, Remove-PSReadLineKeyHandler, Set-PSReadLineKeyHandler...}
Script     0.1.11     WhatsUpGoldPS                       {Add-WUGDevice, Add-WUGDevices, Connect-WUGServer, Convert-HTMLTemplate...}

Fixed with latest version!  Thanks!


Renderer rewriting --> Renderer milestone ?

This is the first post of the road map for rewriting the renderer.
@louen perhaps you must create a milestone Renderer-Storm (or something like that) to identify easily all feature requests and other comments on the refactoring/rewriting of the renderer.
Fell free to express yourself.
The work (experimental from now to mid april) will start on branch expe-mathias.
In a first time, we will keep the Renderer interface and rewrite all the openGL stuff. (before summer)
In a second time, we will propose a new renderer interface (in the summer).
On objective of this rewriting is to make rendering development more efficient and suppress the maintenance by the team of the low level interface. But we want to keep the hability to work at low level. Thats why we do not plug Unreal Engine (or others) to the Radium.
I'll manage the renderer development from now until it is mature enough to be merged with master.
cheers.

Done


Updated Python.Python.3 version 3.9.2150.0

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.0 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

/azp run


Keep MIME resources for sending email in a native app

Fixes #68

Do you have a sample app which reproduces the problem?

I have a ARM lambda locally that showed the issue, and passes with this change.
How do we go about getting it into the pipeline?  It only fails when compiled natively

There is a demo SES CLI app over here (if you have an AWS account with SES permissions) that shows the issue
https://github.com/bloidonia/micronaut-native-mail-97
Publishing this branch to maven local, and then commenting https://github.com/bloidonia/micronaut-native-mail-97/blob/main/build.gradle.kts#L22 out should show it failing when you do nativeRun
email addresses probably need changing for it to work, that's the one I registered with SES on my account
https://github.com/bloidonia/micronaut-native-mail-97/blob/2fc134bf9064e8ab10e4efae320e22a42c029b8a/src/main/java/micronaut/NativemailCommand.java#L41-L42


support to vue 3?

Does it have supporto for vue 3?

Sorry, it does not support Vue 3 yet, I am rewriting this library with TypeScript and redesigning the lib for cleaner, performant, interoperability and ease of use but I don't have a date for it yet.

Sorry, it does not support Vue 3 yet, I am rewriting this library with TypeScript and redesigning the lib for cleaner, performant, interoperability and ease of use but I don't have a date for it yet.

Does it have supporto for vue 3?

I created a Vue 3 component based on this repository.
You can find it at https://github.com/DerYeger/vue-masonry-wall.
It has been rewritten in TypeScript, does not rely on external dependencies, takes padding in regard when calculating the column count, and supports item removal.
A demo is available at https://vue-masonry-wall.yeger.eu/.

Does it have supporto for vue 3?

I created a Vue 3 component based on this repository.
You can find it at https://github.com/DerYeger/vue-masonry-wall.
It has been rewritten in TypeScript, does not rely on external dependencies, takes padding in regard when calculating the column count, and supports item removal.
A demo is available at https://vue-masonry-wall.yeger.eu/.

Would it be much work to support Vue 2, for example with vue-demi package?
Many of us seem to be stuck with Vue 2 until some major libraries release Vue 3 support. In our case, Bootstrap Vue library has went silent although its a hugely used library and they started developing Vue 3 support already late last year... :/


Install .NET 6.0.x SDK. Remove DotNetCoreVersion param. Undo DOTNET_ROLL_FORWARD - take 2

This is take two on this PR:

https://github.com/Azure/azure-sdk-tools/pull/5395

after it had to be reverted by:

https://github.com/Azure/azure-sdk-tools/pull/5398

This time it works because I am installing .NET 6.0 SDK as opposed to .NET 6.0 runtime.
Testing done
A build that proves it works (previously this pipeline was failing, see above):
https://dev.azure.com/azure-sdk/public/_build/results?buildId=2179897&view=results

/check-enforcer override


Add --ring-buffer-size option to --run

Also fix -vv and remove documentation for -u.

I really like the patch. I'm uneasy about the lack of testing; I expect rarely-used options will break again. Could you add a shell script that runs this front-end with all the options that can be reasonably run without hardware and dies if they fail, and have that get run by 'make test', perhaps via our selftest.sh or a similar mechanism?

LGTM.

This front-end only runs with hardware, so there is no obvious test.

We need a script that runs a bunch of tests that require hardware, on hardware. Even if it's not run by normal CI, it could be part of the release process (although not a seamless one, given distro considerations).

@kbara ACK; i'll file an issue, in the future with tasks like that please file an issue directly :)


Change elected summarizer when not summarizing for too long

Fixes #5222
Fixes #1791
The goal of the change is to have all clients unanimously and deterministically change their elected summarizer client whenever too many ops pass without a summary ack. In order to ensure eventual consistency here, the decision is purely based on inbound op processing: we use the sequence numbers of summary acks and other ops to figure it out.
The current behavior is that the oldest interactive client is responsible for summarizing. If 3000 inbound ops are seen since the last summary ack, that client will be marked as invalid, and the next oldest client will be elected. If no summary ack is seen since that client is elected, we use the first inbound op since they were elected as the baseline for the sequence number comparison.
Implementation Details
OrderedClientCollection - doubly linked list which tracks all the clients in the Quorum, sorted by their join sequence number.
OrderedClientElection - this is an adapter over the OrderedClientCollection which allows the caller to specify an isEligibleFn function to filter out non-interactive clients (or other criteria) and track a client as an "elected" client. It provides APIs to increment and reset the elected client. The rule is that the oldest eligible client is the elected client, and when increment is called the next oldest eligible client is elected instead (which means it should only be called deterministically for all clients to achieve eventual consistency). At any point, the elected client can be reset back to the oldest eligible client again, which should also be deterministically called.
The above 2 classes achieve something similar to the old leader election or task picking, but without requiring any additional ops or signals to be used. This is because the information is based on other ops already being passed around: join op sequence numbers, summaryAck sequence numbers, and other op sequence numbers.
ElectedSummarizerManager - this is a helper class used by SummaryManager to handle electing which client is responsible for generating summaries. It leverages the above OrderedClientElection to achieve this, and decides to reelect after some number of ops have passed since the last successful summary. If all clients are elected, it directly resets and goes back to electing the oldest client again.
Added tests for all 3 above classes.

    // computed summarizer client id

this comment is no longer accurate given the above

Refers to: packages/runtime/container-runtime/src/summaryManager.ts:275 in a46ce7a. [](commit_id = a46ce7a5ab7ff8aa070415c9fe3cc32fab488e1d, deletion_comment = False)

Still reviewing, but it think we should wait and check this in after the 0.41 release, so it has the full amount of time to bake in our stress runs. also sorry it took sooooo long to review here, been having trouble finding a solid chuck of time to really dig in here

Some high level thoughts:

3K ops is too small of a number. Please take a look at some of the recent numbers of TAP40 simulation - we continuously hit 3K, and even 10K in this run. There are a lot of questions on workload being representative, proper feedback to client, and throttling being not balanced (i.e. ideally it happens earlier and not allow that many ops to be submitted, if we can't summarize it). BUT even with all these ifs and pending investigations, we should assume that hitting 3K limit in production will be routine, and switching summarizer at that boundary will create more pressure on SPO (more calls to load container, more posted summaries), and it will also increase chances of two summarizers working in parallel. So I feel like this kind of switch should be really action of last resort.
Is this mostly to address client misbehaving, or service misbehaving? Have we considered it driven from service side, in a form of forceful disconnect of lingering summarizer?
I worry about summarization process becoming more and more complex.


run packagemanifests: registry server should use world-writable directory to write files

Description of the change:
Write the bundle.db and termination.log file to /tmp when running the registry server for the run packagemanifests command.
Motivation for the change:
Resolves the issue raised here.
Fixes #3863
Checklist
If the pull request includes user-facing changes, extra documentation is required:

[x] Add a new changelog fragment in changelog/fragments (see changelog/fragments/00-template.yaml)
[ ] ~Add or update relevant sections of the docs website in website/content/en/docs~

@Doout can you try this branch out to see if it resolves this issue you brought up here?

@joelanford I comment back on the issues, but still hit the permission denied with this PR.

@Doout @jmccormick2001 I updated this PR to set the registry pod working directory to /tmp, spun up a 4.5 cluster, and confirmed that the registry pod started successfully with this latest change.

one thing I noticed is that when the pod, memcached-operator-registry-server-675559c6d5-6fwm5, is started, it is using imagePullPolicy of Always...I wonder if this is intended?

@jmccormick2001 imagePullPolicy defaults to Always when the image tag is latest; otherwise it defaults to IfNotPresent.
There's probably a separate discussion to be had about whether we should keep using latest, but this is what we've been using for the run packagemanifests command for awhile so that behavior hasn't changed.


Docs: Inputs and Outputs

Add more details on how to work with outputs, examples of using global methods

Fix the link to outputs doc in providers doc after this change.


Fix: restore mobile pairing + fix the selected chain in it

What it solves
Restores #3753 and #3800. Mobile pairing however needs to be fixed (see https://github.com/gnosis/safe-ios/issues/2304).
It seems the wc instance inside the provider isn't initialized properly. If you add provider.enable() in the provider init function, it fixes the issue.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

21 of 55 (38.18%)  changed or added relevant lines in 8 files are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.08%) to 35.414%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/components/AppLayout/Header/components/ProviderDetails/PairingDetails.tsx
0
1
0.0%

src/logic/wallets/pairing/utils.ts
1
2
50.0%

src/logic/wallets/pairing/module.ts
15
24
62.5%

src/logic/wallets/pairing/hooks/usePairing.ts
0
23
0.0%

Files with Coverage Reduction
New Missed Lines
%

src/logic/wallets/pairing/module.ts
3
60.78%

Totals

Change from base Build<PHONE_NUMBER>:
0.08%

Covered Lines:
3597

Relevant Lines:
9210

💛  - Coveralls


Tested EQB3 with newest firmware 1.46 -> Works -> suggesting readme change.

Hey there,
I just updated all my units to 1.46 -> works fine.
Pairing is needed and the units are then secure with changing PINs.
Added a changed Readme you can direclt start using.
Change is:
"Attention: Firmware from version 120 is recommended, successfully tested also with firmware 1.46 on 2023-12-07 15:14:48 (no newer firmware is currently known)"
README.md

Thanks! I updated the readme with the instructions.


Estimate gas isn't estimating enough gas for Send Only ETC?

Example TX: https://gastracker.io/tx/0x3387cebe2f0686ed83168f92fed638d376219704c16b5aa003a72c721dcceced
I can get it to go through with 60000.
WTF.

i added the new estimate gas to both gh-pages and mercury, it should fix the issue.

Verified only etc and only eth are going thru successfully with estimate gas to brand new accounts and existing accounts. If anyone is still having TX's fail for only etc or only eth, please comment here.


Image not appearing on post

I got cloned the flasky tutorial and git  checkout 11h. When I type this link -> img src="/static/logo.png with other text. I saw the image  and the text at the pagedown  view. But when I click submit. I do not see the image on the post but only the text. What do I do make the image appear together with the text.

The <img> tag is not allowed as a valid tag. See https://github.com/miguelgrinberg/flasky/blob/master/app/models.py#L300-L302. You can add it to the list so that it isn't filtered.

After adding the :
allowed_tags = ['a', 'abbr', 'acronym', 'b', 'blockquote', 'code',
'em', 'i', 'li', 'ol', 'pre', 'strong', 'ul',
'h1', 'h2', 'h3', 'p','img']
Result remains the same
This is the write up and the pagedown

this is the post result

Can you check what was stored in your database?

this is the content of the database

Okay, the issue is that bleach also needs a white list for attributes. You can see an example specifically to allow images in the documentation: https://bleach.readthedocs.io/en/latest/clean.html#as-a-dict.
You would need to also add a attributes={'img': ['src']} argument to the bleach.clean() call.

Here is the result, Up and running
def on_changed_body(target, value, oldvalue, initiator):
allowed_tags = ['a', 'abbr', 'acronym', 'b', 'blockquote', 'code',
'em', 'i', 'li', 'ol', 'pre', 'strong', 'ul',
'h1', 'h2', 'h3', 'p','img']
target.body_html = bleach.linkify(bleach.clean(
markdown(value, output_format='html'),
tags=allowed_tags, attributes={'img':['src']},strip=True))

THANKS A LOT

Great. And thank you for updating the post with your final solution for others to benefit as well.


Add additional expression tests

Wrote these tests to quickly verify the evaluator behavior outlined in https://github.com/istio/istio/issues/3734.

@ozevren: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
c932f2167f35bc8e35f115ae3da5e236ef1d33d5
link
/test istio-unit-tests

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

/test istio-unit-tests

@ozevren: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
c932f2167f35bc8e35f115ae3da5e236ef1d33d5
link
/test istio-unit-tests

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

@ozevren: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests.sh
95e1e0e7dd234129fad1582be8c0408094d35a87
link
/test e2e-simple

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

@ozevren: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests.sh
95e1e0e7dd234129fad1582be8c0408094d35a87
link
/test e2e-simple

prow/e2e-bookInfoTests.sh
95e1e0e7dd234129fad1582be8c0408094d35a87
link
/test e2e-bookInfo

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

/test e2e-bookInfo

/test e2e-simple


Prevent dependabot from running CD

Dependabot doesn't have access to repository secrets, and as such it cannot run the CD. The current setup had a bug which resulted in workflows failing rather than being skipped.

Sure, will do. It's something I want to experiment with though, because I know GH added a dependabot sercrets section. However, even if it were possible, I would rather not have dependabot run the CD, which seems unecessary for typical dependency updates.


[Task]: Document research done around performance analysis of datasources

SubTasks
This task is to create notion doc for all research and analysis of datasources performance issues

Noted all points here: https://www.notion.so/appsmith/Performance-delays-in-datasources-and-formConfig-Api-72d3fff9880c468e9d0e966a9ce31f69#8491fabe5bf44b8488bc0bd816d37922


Implement UTransport for MQTT v5

This pr implements UTransport for MQTT v5.
Items needed before this PR gets out of draft:

[x] Description of how to use clients in README
[x] Comprehensive testing - In progress (following zenoh's test_case model)
[x] Address TODO's - mainly error handling
[x] Fix Lint and Test workflow

Hi,
I think there is a small bug here
https://github.com/eclipse-uprotocol/up-transport-mqtt5-rust/blob/0bf157e2c5033e67dd1372bc77f9f89b817d5fcf/src/lib.rs#L467-L472
basically if the mqtt subscribe fails the free_subscription_ids is not released.


website: remove podling report

We should replace this by PMC reports

We should replace this by PMC reports

Will update it today.


Fix a DivisionByZeroError in case there is no line covered in a contract

"brownie test --coverage" crashed because of that when I included a contract that was left untouched for now.

You're on a roll!

This is passing all tests except for the broken links in the docs, which will be fixed by
f9a47d3.  Merging now.


[CSM] imds event type

What does this PR do?

This PR introduces the imds event. This event is generated by capturing the raw HTTP traffic to and from the IP of the cloud metadata services (i.e. <IP_ADDRESS>). For now, only AWS credentials are supported in response, and we're storing only the AccessKeyID (secrets and other sensitive credentials aren't even parsed).
New SecL rules can be written to be notified when specific IMDS traffic patterns are detected (such as the use of IMDSv1 instead of IMDSv2 on AWS). For example:
imds.cloud_provider == "aws" && imds.aws.is_imds_v2 == false

imds events are also added to Activity Trees, meaning they can be used to monitor continuously which processes are making imds requests, along with retrieving which processes resolved which AccessKeyID. This AccessKeyID can then be leveraged to correlate signals in the backend. In more technical terms, imds events are sent as part of Activity Dumps and Drift events.
Motivation

There are 2 main motivation for this change:

help monitor the usage of IMDSv1 in order to make a transition to IMDSv2 easier (and make sure it won't break services)
keep track of which processes in an infrastructure had access to which AccessKeyID so that process, service and app context can be provided to Cloud based signals. It will also allow for accurate correlation between signals from different products.

Describe how to test/QA your changes

In order to test this PR you'll have to setup a local IMDS server and generate traffic to it.

Set up a local dummy interface which will be used to route traffic to the instance metadata service IP address <IP_ADDRESS>. To do so, run the following commands:

ip link add dummy0 type dummy
ip addr add <IP_ADDRESS>/32 dev dummy0
ip link set dummy0 up

Create a dummy IMDS server that binds on the IMDS IP address. A go example is provided below:

package main

package main

import (
	"encoding/json"
	"fmt"
	"net/http"
)

func main() {
	http.HandleFunc("/latest/meta-data/iam/security-credentials/hello", func(w http.ResponseWriter, r *http.Request) {
		// Define your custom JSON data
		data := map[string]interface{}{
			"AccessKeyId":     "ASIAIOSFODNN7EXAMPLE",
			"SecretAccessKey": "FQoDYXdzEL3/FQoDYXdzEL3/FQoDYXdzEL3",
			"Token":           "FQoDYXdzEL3FQoDYXdzEL3FQoDYXdzEL3FQoDYXdzEL3FQoDYXdzEL3FQoDYXdzEL3FQoDYXdzEL3FQoDYXdzEL3FQoDYXdzEL3FQoDYXdzEL3FQoDYXdzEL3FQoDYXdzEL3FQoDYXdzEL3FQoDYXdzEL3FQoDYXdzEL3FQoDYXdzEL3FQoDYXdzEL3FQoDYXdzEL3FQoDYXdzEL3FQoDYXdzEL3FQoDYXdzEL3FQoDYXdzEL3FQoDYXdzEL3FQoDYXdzEL3FQoDYXdzEL3FQoDYXdzEL3FQoDYXdzEL3FQoDYXdzEL3FQoDYXdzEL3FQoDYXdzEL3FQoDYXdzEL3FQoDYXdzEL3FQoDYXdzEL3FQoDYXdzEL3FQoDYXdzEL3FQoDYXdzEL3FQoDYXdzEL3FQoDYXdzEL3FQoDYXdzEL3FQoDYXdzEL3FQoDYXdzEL3FQoDYXdzEL3FQoDYXdzEL3",
			"Expiration":      "2324-05-01T12:00:00Z",
			"Code":            "Success",
			"LastUpdated":     "2012-04-26T16:39:16Z",
			"Type":            "AWS-HMAC",
		}

		// Convert data to JSON
		response, err := json.Marshal(data)
		if err != nil {
			http.Error(w, err.Error(), http.StatusInternalServerError)
			return
		}

		// Set Content-Type header to application/json
		w.Header().Set("Content-Type", "application/json")

		// Write JSON response
		w.Write(response)
	})
	fmt.Println("Server listening on port 8080...")
	http.ListenAndServe("<IP_ADDRESS>:8080", nil)
}

Add the following policy to your policy folder:

version: '1'
rules:
- id: imds_v1_rule
  version: 1
  description: Detect IMDSv1 traffic
  expression: imds.cloud_provider == "aws" && imds.aws.is_imds_v2 == false
  agent_version: ''
  filters: []
- id: touch_tmp_test
  version: 1
  description: touch /tmp/test
  expression: open.file.path == "/tmp/test"

Start the agent.

There are 3 things to test:

Run the following command and check if an imds_v1_rule event was generated:

curl http://<IP_ADDRESS>:8080/latest/meta-data/iam/security-credentials/hello

Run the following command and check if a populated aws_security_credentials field with the access key id ASIAIOSFODNN7EXAMPLE was sent as part of the process context of the touch_tmp_test event:

python3 -c 'import requests; requests.get("http://<IP_ADDRESS>:8080/latest/meta-data/iam/security-credentials/hello"); open("/tmp/test", "w")'

Do the same thing within a container and make sure:

that IMDS nodes are properly created and serialized (including in the .dot graph)
that drift events are generated for IMDS events

/merge


⚠ ClamAV has degraded performance

In e1d5c7d, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 57 ms

Resolved: ClamAV performance has improved in 9be2483.


[bazel] further enhance manufacturer test hooks repo setup

This further enhances the manufacturer test hooks bazel repository to:

support writing multiple sets of test hooks in the same MANUFACTURER_HOOKS_DIR,
support toggling between which test hooks are enabled via a bazel command line switch, e.g., --define test_hooks=<hooks library>,
support writing multiple closed source opentitan_functest tests in the same MANUFACTURER_HOOKS_DIR as the test hooks, and
updating the README.md and BUILD.bazel comments with detailed instructions on how extend these features.

Signed-off-by: Timothy Trippel<EMAIL_ADDRESS>Note: this depends on #13004.

CC: @MaayanKashani

PR dependency removed. Ready for review.


Add new section for AnnData in Changelog

Closes #189
Here, I created a new entry in the Changelog that includes everything relevant to AnnData additions. I added a placeholder for the heading that will need to be replaced with the date of release when the time comes.

Are there any items that you think are not relevant to be in the Changelog entry?
Are there any items that should be there that are missing? (I didn't include mention of the clustering assignments, but I could)

Thanks for looking at this @jaclyn-taroni. I incorporated your wording suggestions and then also added a link to the metadata section of the downloads for the bullet about added metadata included in the download. This should be ready for another look.


Client support for offline file storage

Motivation
Allow to store files in temporary storage if internet is not available.
Usage
cd raincatcher-demo-mobile
npm install --save<EMAIL_ADDRESS>
ping @col1985
See https://issues.jboss.org/browse/RAINCATCH-636 for verification steps.

@wtrocki this is working well in current solution. One issue is that we need to return the metaData object with reference to the file in gridFS in the done callback.
I have also updated createFile to accept fileName. Can you please review and publish as a pre-release?
Thanks,
Colúm

@col1985 Reviewed your PR and integrated changes into this PR.

Resolved all issues. Reviewed by @JameelB
Verified by @witmicko


Question: using PATH through GAMS

I have been hoping that something like this is developed. Great work!
But how does one use the PATH solver here. For example to solve the following simple problem:
x >=    0
2x - 1 ⟂ x

GAMS.jl doesn't support complementarity constraints yet. You will have to use PATH.jl directly.

Thanks for the feature request. I haven't addressed complementarity constraints yet, because I haven't seen them in the list of supported MathOptInterface constraints. I was not aware of MathOptInterface.Complements. I will look into this.

@odow. PATH.jl is great. Tried out some test problems. Quite pleased.
@renkekuhlmann. Nice that I for one can make use of the GAMS solvers I have access to. PATH would be great additional to have eventually.

@Jovansam There is a Complementarity.jl package as well which I've used to do the same job as I once did in GAMS (solving large Nash-Cournot models).

@sarkispa thanks very much for the heads up. I tested my model out and it works.

Support of complementarity constraints has been added to GAMS.jl (v0.2.0). If complementarity constraints are detected in the model, GAMS.jl will set the model type to MPEC. If you want to use the GAMS type MCP instead (having no objective function), please call:
set_optimizer_attribute(model, GAMS.ModelType(), "MCP")

Sorry, for the very long delay to support this feature. This was a very good training task for a new student, so this required some extra learning time. Please let us know, if you think there is something that can be improved for this feature. Thanks again very much for posting the feature request!

Support of complementarity constraints has been added to GAMS.jl (v0.2.0). If complementarity constraints are detected in the model, GAMS.jl will set the model type to MPEC. If you want to use the GAMS type MCP instead (having no objective function), please call:
set_optimizer_attribute(model, GAMS.ModelType(), "MCP")

Sorry, for the very long delay to support this feature. This was a very good training task for a new student, so this required some extra learning time. Please let us know, if you think there is something that can be improved for this feature. Thanks again very much for posting the feature request!


kie-issues#599: enable sonar analysis in PR check

apache/incubator-kie-issues#599
Enabling sonar analysis in PR checks.

PR job #2 was: UNSTABLE
Possible explanation: This should be test failures

Reproducer
build-chain build full_downstream  -f 'https://raw.githubusercontent.com/${AUTHOR:apache}/incubator-kie-kogito-pipelines/${BRANCH:main}/.ci/buildchain-config-pr-cdb.yaml' -o 'bc' -p apache/incubator-kie-kogito-runtimes -u https://github.com/apache/incubator-kie-kogito-runtimes/pull/3275 --skipParallelCheckout
NOTE: To install the build-chain tool, please refer to https://github.com/kiegroup/github-action-build-chain#local-execution

Please look here: https://ci-builds.apache.org/job/KIE/job/kogito/job/main/job/pullrequest_jobs/job/kogito-runtimes-pr/job/PR-3275/2/display/redirect
Test results:

PASSED: 3388
FAILED: 1

Those are the test failures:

org.kie.kogito.quarkus.workflows.PojoServiceIT.testPojo
Expecting value to be true but was false

PR job #7 was: UNSTABLE
Possible explanation: This should be test failures

Reproducer
build-chain build full_downstream  -f 'https://raw.githubusercontent.com/${AUTHOR:apache}/incubator-kie-kogito-pipelines/${BRANCH:main}/.ci/buildchain-config-pr-cdb.yaml' -o 'bc' -p apache/incubator-kie-kogito-runtimes -u https://github.com/apache/incubator-kie-kogito-runtimes/pull/3275 --skipParallelCheckout
NOTE: To install the build-chain tool, please refer to https://github.com/kiegroup/github-action-build-chain#local-execution

Please look here: https://ci-builds.apache.org/job/KIE/job/kogito/job/main/job/pullrequest_jobs/job/kogito-runtimes-pr/job/PR-3275/7/display/redirect
Test results:

PASSED: 3388
FAILED: 1

Those are the test failures:

org.kie.kogito.quarkus.workflows.PojoServiceIT.testPojo
Expecting value to be true but was false


KLS: Avoid repeated documentation requests. [phab:D2183]

Created by fromberger at 2018-02-12 19:15:26:
Prior to this commit, the language server calls for Documentation each time
time hover documentation is activated, even if the hoverdoc for that reference
has already been expanded.
Cache hoverdoc in the ref to avoid repeating these lookups.

schroederc accepted this revision at 2018-02-12 19:25:51

jrtom wrote at 2018-02-12 19:31:08:
Approvinated, now that I understand that doc.xrefs() was already referencing an existing cache that you're just writing additional stuff into.  :)

jrtom accepted this revision at 2018-02-12 19:31:17

danielmoy-google accepted this revision at 2018-02-12 19:47:14

fromberger closed via commit 743083471b02630dea86e4415c065e21f35a5d7f at 2018-02-12 19:58:06


Deleted ssr server readme file

Deleted ssr server readme file

Is all of this information either (a) outdated or (2) documented elsewhere?

Yeah. Most of it is outdated and deleting the file anyway was the plan.


Module 4 exercises [Amanda Selasie Doagbodzie]

WS2_M4E1.docx

Well done, Amanda. You clearly grasped the material outlined in Module 4 and successfully applied it to these exercises. Thank you for your participation! Please be sure to fill out the post-Module survey before you begin the next Module.


Add instructions to README for setting up a local database

For users who want to try the workflow but don't have experience with databases.

Solved by adding instructions to the installation section of elements.datajoint.org


infoq extractor broken

$ youtube-dl http://www.infoq.com/presentations/A-Few-of-My-Favorite-Python-Things --verbose
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'http://www.infoq.com/presentations/A-Few-of-My-Favorite-Python-Things', u'--verbose']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2015.06.25
[debug] Python version 2.7.10 - Darwin-13.4.0-x86_64-i386-64bit
[debug] exe versions: ffmpeg 2.6.3, ffprobe 2.6.3
[debug] Proxy map: {}
[InfoQ] A-Few-of-My-Favorite-Python-Things: Downloading webpage
ERROR: Unable to extract HTTP base URL; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 654, in extract_info
    ie_result = ie.extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 273, in extract
    return self._real_extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/infoq.py", line 46, in _real_extract
    'HTTP base URL')
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 555, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)
RegexNotFoundError: Unable to extract HTTP base URL; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

$ youtube-dl --version
2015.06.25

Thanks for the report. InfoQ extractor will be fixed in the next version.


HDDS-1321. TestOzoneManagerHttpServer depends on hard-coded port numb…

…ers. Contributed by Arpit Agarwal.
Change-Id: If17c851b4aea7070064069a3596a144ad80d284c

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
25
Docker mode activated.

_ Prechecks _

+1
@author
0
The patch does not contain any @author tags.

+1
test4tests
0
The patch appears to include 1 new or modified test files.

_ trunk Compile Tests _

+1
mvninstall
997
trunk passed

+1
compile
48
trunk passed

+1
checkstyle
16
trunk passed

+1
mvnsite
29
trunk passed

+1
shadedclient
732
branch has no errors when building and testing our client artifacts.

+1
findbugs
38
trunk passed

+1
javadoc
19
trunk passed

_ Patch Compile Tests _

+1
mvninstall
33
the patch passed

+1
compile
23
the patch passed

+1
javac
23
the patch passed

+1
checkstyle
15
the patch passed

+1
mvnsite
25
the patch passed

+1
whitespace
0
The patch has no whitespace issues.

-1
shadedclient
1087
patch has errors when building and testing our client artifacts.

+1
findbugs
48
the patch passed

+1
javadoc
18
the patch passed

_ Other Tests _

+1
unit
44
ozone-manager in the patch passed.

+1
asflicense
26
The patch does not generate ASF License warnings.

3315

Subsystem
Report/Notes

Docker
Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/hadoop-multibranch/job/PR-633/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/633

Optional Tests
dupname  asflicense  compile  javac  javadoc  mvninstall  mvnsite  unit  shadedclient  findbugs  checkstyle

uname
Linux 2890069db7a0 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 17:16:02 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / 548997d

maven
version: Apache Maven 3.3.9

Default Java
1.8.0_191

findbugs
v3.1.0-RC1

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-633/1/testReport/

Max. process+thread count
411 (vs. ulimit of 5500)

modules
C: hadoop-ozone/ozone-manager U: hadoop-ozone/ozone-manager

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-633/1/console

Powered by
Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
49
Docker mode activated.

_ Prechecks _

+1
@author
0
The patch does not contain any @author tags.

+1
test4tests
0
The patch appears to include 1 new or modified test files.

_ trunk Compile Tests _

+1
mvninstall
1229
trunk passed

+1
compile
55
trunk passed

+1
checkstyle
20
trunk passed

+1
mvnsite
34
trunk passed

+1
shadedclient
765
branch has no errors when building and testing our client artifacts.

+1
findbugs
47
trunk passed

+1
javadoc
21
trunk passed

_ Patch Compile Tests _

+1
mvninstall
35
the patch passed

+1
compile
24
the patch passed

+1
javac
24
the patch passed

+1
checkstyle
13
the patch passed

+1
mvnsite
28
the patch passed

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedclient
801
patch has no errors when building and testing our client artifacts.

+1
findbugs
53
the patch passed

+1
javadoc
20
the patch passed

_ Other Tests _

+1
unit
66
ozone-manager in the patch passed.

+1
asflicense
29
The patch does not generate ASF License warnings.

3381

Subsystem
Report/Notes

Docker
Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/hadoop-multibranch/job/PR-633/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/633

Optional Tests
dupname  asflicense  compile  javac  javadoc  mvninstall  mvnsite  unit  shadedclient  findbugs  checkstyle

uname
Linux 28910357a9dc 4.4.0-138-generic #164~14.04.1-Ubuntu SMP Fri Oct 5 08:56:16 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / 548997d

maven
version: Apache Maven 3.3.9

Default Java
1.8.0_191

findbugs
v3.1.0-RC1

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-633/2/testReport/

Max. process+thread count
350 (vs. ulimit of 5500)

modules
C: hadoop-ozone/ozone-manager U: hadoop-ozone/ozone-manager

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-633/2/console

Powered by
Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
45
Docker mode activated.

_ Prechecks _

+1
@author
0
The patch does not contain any @author tags.

+1
test4tests
0
The patch appears to include 1 new or modified test files.

_ trunk Compile Tests _

+1
mvninstall
1211
trunk passed

+1
compile
55
trunk passed

+1
checkstyle
20
trunk passed

+1
mvnsite
34
trunk passed

+1
shadedclient
780
branch has no errors when building and testing our client artifacts.

+1
findbugs
45
trunk passed

+1
javadoc
19
trunk passed

_ Patch Compile Tests _

+1
mvninstall
33
the patch passed

+1
compile
25
the patch passed

+1
javac
25
the patch passed

+1
checkstyle
14
the patch passed

+1
mvnsite
26
the patch passed

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedclient
802
patch has no errors when building and testing our client artifacts.

+1
findbugs
53
the patch passed

+1
javadoc
17
the patch passed

_ Other Tests _

+1
unit
63
ozone-manager in the patch passed.

+1
asflicense
25
The patch does not generate ASF License warnings.

3354

Subsystem
Report/Notes

Docker
Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/hadoop-multibranch/job/PR-633/3/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/633

Optional Tests
dupname  asflicense  compile  javac  javadoc  mvninstall  mvnsite  unit  shadedclient  findbugs  checkstyle

uname
Linux c664e0ade47b 4.4.0-138-generic #164~14.04.1-Ubuntu SMP Fri Oct 5 08:56:16 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / 548997d

maven
version: Apache Maven 3.3.9

Default Java
1.8.0_191

findbugs
v3.1.0-RC1

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-633/3/testReport/

Max. process+thread count
352 (vs. ulimit of 5500)

modules
C: hadoop-ozone/ozone-manager U: hadoop-ozone/ozone-manager

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-633/3/console

Powered by
Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

+1


Fix/6584 content type for lakectl local

Closes #6584
Change Description
Background
Followed code from lakectl fs upload.
Bug Fix
If this PR is a bug fix, please let us know about:

Problem - lakectl local didnt support --content-type flag.
Solution - added content-type support

Testing Details
How were the changes tested?
They were tested by running lakectl local to upload files with content-type flag
Contact Details
<EMAIL_ADDRESS>

Thanks @itaiad200,
Yes I will add --include-content-type flag with the automation.

THANKS
for picking up this issue and working to fix it.  This is really appreciated, and a great way to improve lakeFS as a product.

Hi @0x-chaitu, could you please provide an update on the PR status?

Hi @0x-chaitu, could you please provide an update on the PR status?

Hi @nopcoder, @itaiad200 has self-assigned the issue.

Hey @0x-chaitu , I self-assigned the PR so I could track this better. We do that normally for all contributors. Do you still intend to work on this PR or should we take it?

Hey, @itaiad200 Please take it.


The number of threads in envoy of ingress gateway is inconsistent with that configured by kubernees CPU static policy

I want to know if there is a way to configure ingress gateway to make the number of threads consistent with the k8s CPU policy
This is what my current machine looks like：
1、View the process ID used by ingress gateway
kubectl describe pod/istio-ingressgateway-669f7ff885-68jsm -n istio-system | grep "Container ID"
Container ID:  containerd://31fe853ee4f7c2553d8810421ffdf13ed48aabb3b979027a5161de64e306a9c6
crictl inspect 31fe853ee4f7c2553d8810421ffdf13ed48aabb3b979027a5161de64e306a9c6 | grep pid
"pid": 862374,
2、View the CPUID of the process binding
taskset -c -p 862374
pid 862374's current affinity list: 2,82
3、Number of CPUs on the node
grep -c 'processor' /proc/cpuinfo
160
4、View the number of envoy threads in the pod
kubectl exec pod/istio-ingressgateway-669f7ff885-68jsm -n istio-system -- curl <IP_ADDRESS>:15000/server_info | grep concurrency
160
I don't know whether it will affect the CPU cache affinity of envoy in ingress gateway. Who can tell me, thank you, and how to set it to 2 to be the same as the k8s CPU policy value

This is not a k8s/istio support forum.

ok


ERROR: mkdir /tmp/docker-squash169074396/usr/lib/perl5/5.8.8/pod: file exists

Hi there,
Thanks for the great tool.
I am using current release on Mac2016. I issued cmd below
sudo /Users/xfgavin/Downloads/docker-squash -i aaaa_20171003.tar -o aaaa_20171003_squashed.tar
but I got the error everytime
ERROR: mkdir /tmp/docker-squash169074396/usr/lib/perl5/5.8.8/pod: file exists

+1
:(

Same issue here, is there any way to make tar ignore these errors?


Commit author + Serverless

Hi @Philmod ,
The objective of this commit is two fold:

Have the slack message mention the github commit author if GITHUB_TOKEN is defined
Add the option of using the serverless framework to manage the cloud function

Along with these 2 major features, I've linted, updated packages and renamed the package to google-cloud-build-slack
Many thanks to @iliasbhal and @juanborre for their help on this.
There is a vital piece missing atm, determining how to retrieve the github OWNER and REPO values, at this time it is assumed that Cloud Source Repository mirrors of github repos are in the form of github_<OWNER>_<REPO> which is not necessarily always the case.
TODO:

[x] ~fool proof way to retrieve github OWNER and REPO~ mention of naming requirement for cloud source repo in order for github commit author info to be retrieved in README
[x] add serverless framework and github commit author instructions in README
[x] Add unit tests

Finally, it is worth mentioning that the introduction of async/await in this commit has forced the use of nodejs8 as the runtime

Hey! Thanks for the PR.

Could you send me a link to what you call "serveless"?
Can you write some unit tests?

My pleasure!

https://serverless.com/framework/docs/providers/google/
Yes! I'll work on that.

Hi @Philmod, I've added some unit tests, however I need to be able to unset GITHUB_TOKEN in a test to cover 2 more lines of code. I am researching on how to do that atm, if you have any suggestion please let me know.

Hi @Philmod, unit tests have been added!

Thank @n8than for help on last commit (extracting dependency and passing down octokit as option param)

bump

bump

I'm very sorry for responding so slowly, I've been very busy at work lately

No worries at all!!!!!!!

Thanks!

Hi @Philmod , thanks for merging, I noticed this failed: https://github.com/Philmod/google-cloud-build-slack/runs/98545329
I cannot access the cloudbuild logs, if they cannot be made visible could you paste the error here please?

https://github.com/Philmod/google-cloud-build-slack/pull/21 Created. Should fix the error


Model weather wind speed

Wind speed as a stochastic variable
Wind speed should not change instantaneously

Fixed in 40fcb75


Change class methods to private

Change send* class methods to private in BaseClient class

I'll take this.

Sure :)

Paid


Change detection doesn't work inside parallel queries

Bevy version
0.10.0
What you did
I have a query using a Ref parameter and is_changed() to detect if one of two conditions (position changed or override flag) is true in order to avoid calculating FoV every frame. Since it's an expensive operation, I also want to use par_iter_mut in order to make the query parallel.
My problem is that, once I add par_iter_mut to my query, change detection seems to break and position.is_changed() is never true.
Example code
query
    .par_iter_mut() // If I remove this line, then change detection starts working again.
    .for_each_mut(|mut to_change, maybe_changed| {
        if maybe_changed.is_changed() || to_change.some_flag {
            to_change.expensive_change();
        }
   });

What went wrong
I would expect change detection to work in either case, but it seems to only work with sequential queries.
Additional information
In case it changes something, the specific query I'm having trouble with is also inside a ParamSet.
Also, this is a regression from bevy 0.9, where it worked fine.

I believe this is the same issue already fixed in https://github.com/bevyengine/bevy/pull/8029

Oops, my mistake. Looked for an issue before posting, but didn't think to search for a PR unassociated to an issue.
Thanks.


Handle right click on a bell

My first PR ever, don't really know how to structure it.
The feature is quite simple so i dont really need to explain it, i guess.

Seems good, let's now wait for @DeltaEvo opinion on https://github.com/aresrpg/aresrpg/pull/572#pullrequestreview-1205052568

My last commit was helpfull for test but may cause probleme because the mobs.json file from floor1 when starting the server. I dont know how without requesting a pull on floor1 but modifications on mobs.json should be discarded or the last commit should be ignored.


How to disable all your command

Hello,
is possible with a setting disable all commands already present?

As far as I know, no. You could edit the files in the src/Commands folder and set the $enabled variable to false or simply delete the command files you don't need. This might raise issues when updating through composer though, not sure what would happen.
It's a great idea though, will see if we can implement it in a future version, thanks!
@MBoretto @akalongman
Please correct me if I'm wrong.

The restructuring of the commands is being discuss in #87

Hi, so even "New command structure" is closed, there is still no way to disable all commands, right?
I am using composer and when I add my custom commands, example commands(whoami, weather, etc) still show up and available to users. The only ways for me to disable them:

Delete them from original library's folder. Will need to do it with each composer update.
For each command that is currently present in src/Command, create same command in my custom commands folder and set $enabled to false. This will solve the problem with Composer updates. Unless somebody will add another example command. In this case, it will automatically show as available to all my users.

Is my understanding right? Is there another way to use the library without modifying its contents?
Thank you for your work! It's amazingly useful and needed!

We haven't implemented this feature yet, which is why the issue is still open. But as you mention, creating a custom command with the same name and setting $enabled to false is a simple workaround for now.
We're aiming to have this implemented in version 0.29.
At the moment we're working on releasing 0.28 soon!
Thanks for the patience :+1:

Command could be disable in group chat or vice-versa. Keep this in mind developing this feature

A good approach is this one that I included in this example:
https://github.com/MBoretto/Nostickers/blob/master/app/Commands/UserCommands/EchoCommand.php
Hope that help

As of 0.44.0, default commands have been moved to a separate repository.


AWS OTel Collector templates

Description:

Link to tracking Issue:
Testing:
Documentation:

Codecov Report

Merging #788 (04e9efa) into main (e265860) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #788      +/-   ##
==========================================
- Coverage   88.97%   88.95%   -0.02%
==========================================
  Files         411      411
  Lines       20840    20840
==========================================
- Hits        18543    18539       -4
- Misses       1780     1783       +3
- Partials      517      518       +1

Flag
Coverage Δ

integration
69.28% <ø> (+0.06%)
:arrow_up:

unit
87.82% <ø> (-0.02%)
:arrow_down:

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

Impacted Files
Coverage Δ

receiver/k8sclusterreceiver/watcher.go
95.29% <0.00%> (-2.36%)
:arrow_down:

receiver/carbonreceiver/transport/tcp_server.go
66.00% <0.00%> (-1.00%)
:arrow_down:

processor/groupbytraceprocessor/event.go
95.96% <0.00%> (-0.81%)
:arrow_down:

Continue to review full report at Codecov.

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

@mat-rumian this needs to be rebased


Search in Rotated Sorted Array

Problem : There is an integer array nums sorted in ascending order (with distinct values).

Prior to being passed to your function, nums is rotated at an unknown pivot index k (0 <= k < nums.length) such that the
resulting array is [nums[k], nums[k+1], ..., nums[n-1], nums[0], nums[1], ..., nums[k-1]] (0-indexed). For example,
[0,1,2,4,5,6,7] might be rotated at pivot index 3 and become [4,5,6,7,0,1,2].
Given the array nums after the rotation and an integer target, return the index of target if it is in nums, or -1 if it is not in
nums.
You must write an algorithm with O(log n) runtime complexity.

Problem Link : https://leetcode.com/problems/search-in-rotated-sorted-array/
Platform : LeetCode
Language : C++

Please assign this to me

Can I do this in c++?

I would like to contribute in java language

@Amit366  can I try this in python ?


Adding languages that are supported for the lang bar on orchestration

Currently this enables go, shell, javascript and ruby.

@kenperkins Did you mean for this PR to go directly to master? Also, PHP code samples are also present, but in the dev branch (via PR #1095).

I did, because orchestration is available in the left rail. I was un-aware of PHP in dev branch.

LGTM. Merging.


Support Proxy-Authenticate: Negotiate

VSCode Version: 1.6.0-insider
OS Version: Windows 7

Corporate proxy server: squid for windows
Authentication method: Negotiate
http://wiki.squid-cache.org/Features/NegotiateAuthentication
http://wiki.squid-cache.org/Features/Authentication
I use ntlmaps for debug the connection process.
ntlmaps log:

New client header:
CONNECT marketplace.visualstudio.com:443 HTTP/1.1
Proxy-Authorization: Basic PASSWORD
Host: marketplace.visualstudio.com:443
Connection: close
User-Agent: Mozilla/4.0 (compatible; MSIE 6.0; Windows NT 5.1; SV1)
Accept: image/gif, image/x-xbitmap, image/jpeg, image/pjpeg, application/vnd.ms-excel, application/msword, application/vnd.ms-powerpoint, /
Connecting to remote server...(proxyserver:8080)...Done.
Sending client request header to remote server...Done.
Got remote server response header.

Remote server header:
HTTP/1.1 407 Proxy Authentication Required
Server: squid
Mime-Version: 1.0
Date: Thu, 06 Oct 2016 13:52:13 GMT
Content-Type: text/html;charset=utf-8
Content-Length: 84718
X-Squid-Error: ERR_CACHE_ACCESS_DENIED 0
Proxy-Authenticate: Negotiate
X-Cache: MISS from <IP_ADDRESS>
Via: 1.1 <IP_ADDRESS> (squid)
Connection: close
Server 'Content-Length' found to be 84718.
Authentication routine started.
Got Error 407 - "Proxy authentication required".
Authentication methods allowed: Negotiate
There are no supported authentication methods in remote server response.
Passing 407 to client.
Authentication routine finished.
Sending remote server response header to client...Done.

Did you use the http.proxy setting?

Yes, it doesn´t work.
It´s returning 407 in every attempt.
Note the authentication method is ONLY "Negotiate", it´s not allowed, in our squid configuration, the "Basic" method.
I try to use cntlm to mediate the transaction, but the cntlm 0.92.3 do not support the Negotiate method.

Do you mind going through #6480? Let me know if you find something.

Yes, i try the suggested :
Proxy-Authorization: Basic dXNlcm5hbWU6cGFzc3dvcmQ=
(The base64 string is just username:password encoded)
But my proxy don´t accept Basic authentication, like i mentioned.

So, what kind of authentication does your proxy accept?

ONLY: Proxy-Authorization: Negotiate
http://wiki.squid-cache.org/Features/Authentication

#22369


FTS4 engine support?

Today I found out sqlite3_flutter_libs: 0.5.10  is not supported fts4 engine. Any plan for the full-text engine?
    CREATE VIRTUAL TABLE textContent USING fts4(
        text, tokenize=icu mn_MN
    )

Ref: https://www.sqlite.org/fts3.html

We support fts5 in sqlite3_flutter_libs on all platforms. I don't see a good reason to support previous fts iterations as well. The compiled binaries should be reasonably small, so I don't want to enable too many extensions.


Passing --ignore-checksums to builds

HI Quick question,
Is it possible to pass a --ignore-checksums to a build?  Adobe Reader for example, works fine if you do a choco install adobereader from the sources, but when I attempt to internalize it, I get this:
WARNING: Checksum of D:\Scratch\adobereader\2023.3.20244\tools\AcroRdrDCUpd2300320244_MUI.msp invalid, file removed.
Wanted d2c75ba11c9d0681b5a23a6ffff08a0cd8b2a76370f63e669637203889d7ca77 got
38DA8AC0A689BD28CEB37F3B7F74FB59653128ED547F3577EF512D74EF1AAF81CE6B290B236FAA58CEE9B342905A34053342F6265E09537A7E1894F
8C7A8E790
WARNING: adobereader 2023.3.20244 failed downloading or editing
WARNING: Error details:
Invalid checksum after download for
https://ardownload2.adobe.com/pub/adobe/reader/win/AcrobatDC/2300320244/AcroRdrDCUpd2300320244_MUI.msp
Throw "Invalid checksum after download for $url"
at Get-File, D:\choco-remixer\public\Get-File.ps1: line 56
at Get-FileWithCache, D:\choco-remixer\public\Get-FileWithCache.ps1: line 29
at Convert-adobereader, D:\choco-remixer\pkgs\PkgFunctions-webfile.ps1: line 37
at Invoke-InternalizeChocoPkg, D:\choco-remixer\public\Invoke-InternalizeChocoPkg.ps1: line 195
at , : line 1
Thanks for your help.

Generally, a broken checksum is a warning that something is going wrong. In this case, the package changed, and now includes the 64 bit installer, which broke the checksum type in choco-remixer. I've fixed it, if you pull the latest commit, internalization should be working again.
In cases that the package itself is broken, instead of choco-remixer being broken, that is something that should be investigated manually, and likely fixed by the maintainer of the package. Or in cases like googlechrome, with the always latest version url, the .nuspec version should be updated to reflect the current version when the checksum breaks, which requires manual internalization.
I'm open to other suggestions, or further clarification if that does not answer your question fully.

Thank you.  Makes perfect sense to me.  I'll get the update configs and apply them.
Thanks again.
From: TheCakeIsNaOH @.>
Sent: Tuesday, August 8, 2023 11:34 PM
To: TheCakeIsNaOH/choco-remixer @.>
Cc: Robert Christopher Musick @.>; Author @.>
Subject: Re: [TheCakeIsNaOH/choco-remixer] Passing --ignore-checksums to builds (Issue #15)
Generally, a broken checksum is a warning that something is going wrong. In this case, the package changed, and now includes the 64 bit installer, which broke the checksum type in choco-remixer. I've fixed it, if you pull the latest commit, internalization should be working again.
In cases that the package itself is broken, instead of choco-remixer being broken, that is something that should be investigated manually, and likely fixed by the maintainer of the package. Or in cases like googlechrome, with the always latest version url, the .nuspec version should be updated to reflect the current version when the checksum breaks, which requires manual internalization.
I'm open to other suggestions, or further clarification if that does not answer your question fully.

Reply to this email directly, view it on GitHubhttps://github.com/TheCakeIsNaOH/choco-remixer/issues/15#issuecomment-1670601891, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ACSZL2VWP6J2F24YIT67BYTXUMAK3ANCNFSM6AAAAAA3IRAZPY.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>CAUTION: This email was sent from someone outside of the university. Do not click links or open attachments unless you recognize the sender and know the content is safe.


Issue with /material.angular.io/cdk/listbox/overview stack blitz code example not compiling(Unforked/base code)

Describe the problem that you experienced
fails to compile Listbox with value binding example code Unforked/base code
see exception/error box below
Happens on 2 listbox stackblitz code examples I tried
Enter the URL of the topic with the problem
No response
Describe what you were looking for in the documentation
No response
Describe the actions that led you to experience the problem

go to https://material.angular.io/cdk/listbox/overview
go to Listbox with value binding example code
click on the stackblitz link

Describe what you want to experience that would fix the problem
No response
Add a screenshot if that helps illustrate the problem
No response
If this problem caused an exception or error, please paste it here
~/projects/1mpfwq--run
❯ npm install && turbo start
warn preInstall No description field
warn preInstall No repository field
warn preInstall No license field
┌ [1/4] 🔍  Resolving dependencies
└ Completed in 56.094s
┌ [2/4] 🚚  Fetching dependencies
│ info pruneDeps Excluding 14 dependencies. For more information use `--verbose`.
└ Completed in 34.967s
┌ [3/4] 🔗  Linking dependencies
└ Completed in 31.553s
info security We found `install` scripts which turbo skips for security reasons. For more
information see https://turbo.sh/install-scripts.
└─<EMAIL_ADDRESS>info Some packages were automatically replaced with polyfills. For more information see
https://turbo.sh/polyfills.
└─<EMAIL_ADDRESS>=><EMAIL_ADDRESS>
success Saved lockfile "package-lock.json"
success Updated "package.json"

success Install finished in 122.986s
$ ng serve
⠋ Generating browser application bundles (phase: setup)...    TypeScript compiler options "target" and "useDefineForClassFields" are set to "ES2022" and "false" respectively by the Angular CLI. To control ECMA version and features use the Browerslist configuration. For more information, see https://angular.io/guide/build#configuring-browser-compatibility
✔ Browser application bundle generation complete.

Initial Chunk Files   | Names         |  Raw Size
vendor.js             | vendor        |   6.28 MB |
styles.css, styles.js | styles        | 338.55 kB |
polyfills.js          | polyfills     | 313.71 kB |
main.js               | main          |  27.93 kB |
runtime.js            | runtime       |   6.52 kB |

                      | Initial Total |   6.96 MB

Build at: 2022-12-21T17:58:39.659Z - Hash: 559226122a123ac7 - Time: 37687ms

Error: src/app/cdk-listbox-value-binding-example.html:7:7 - error NG8002: Can't bind to 'cdkListboxValue' since it isn't a known property of 'ul'.

7       [cdkListboxValue]="starter"
        ~~~~~~~~~~~~~~~~~~~~~~~~~~~

  src/app/cdk-listbox-value-binding-example.ts:7:16
    7   templateUrl: 'cdk-listbox-value-binding-example.html',
                     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    Error occurs in the template of component CdkListboxValueBindingExample.

Error: src/app/cdk-listbox-value-binding-example.html:8:49 - error TS2339: Property 'value' does not exist on type 'Event'.

8       (cdkListboxValueChange)="starter = $event.value"
                                                  ~~~~~

  src/app/cdk-listbox-value-binding-example.ts:7:16
    7   templateUrl: 'cdk-listbox-value-binding-example.html',
                     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    Error occurs in the template of component CdkListboxValueBindingExample.

Error: src/app/cdk-listbox-value-binding-example.html:12:9 - error NG8002: Can't bind to 'cdkOption' since it isn't a known property of 'li'.

12         [cdkOption]="pokemon"
           ~~~~~~~~~~~~~~~~~~~~~

  src/app/cdk-listbox-value-binding-example.ts:7:16
    7   templateUrl: 'cdk-listbox-value-binding-example.html',
                     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    Error occurs in the template of component CdkListboxValueBindingExample.

** Angular Live Development Server is listening on localhost:4200, open your browser on http://localhost:4200/ **

✖ Failed to compile.
✔ Browser application bundle generation complete.

5 unchanged chunks

Build at: 2022-12-21T17:58:44.943Z - Hash: 559226122a123ac7 - Time: 5025ms

Error: src/app/cdk-listbox-value-binding-example.html:7:7 - error NG8002: Can't bind to 'cdkListboxValue' since it isn't a known property of 'ul'.

7       [cdkListboxValue]="starter"
        ~~~~~~~~~~~~~~~~~~~~~~~~~~~

  src/app/cdk-listbox-value-binding-example.ts:7:16
    7   templateUrl: 'cdk-listbox-value-binding-example.html',
                     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    Error occurs in the template of component CdkListboxValueBindingExample.

Error: src/app/cdk-listbox-value-binding-example.html:8:49 - error TS2339: Property 'value' does not exist on type 'Event'.

8       (cdkListboxValueChange)="starter = $event.value"
                                                  ~~~~~

  src/app/cdk-listbox-value-binding-example.ts:7:16
    7   templateUrl: 'cdk-listbox-value-binding-example.html',
                     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    Error occurs in the template of component CdkListboxValueBindingExample.

Error: src/app/cdk-listbox-value-binding-example.html:12:9 - error NG8002: Can't bind to 'cdkOption' since it isn't a known property of 'li'.

12         [cdkOption]="pokemon"
           ~~~~~~~~~~~~~~~~~~~~~

  src/app/cdk-listbox-value-binding-example.ts:7:16
    7   templateUrl: 'cdk-listbox-value-binding-example.html',
                     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    Error occurs in the template of component CdkListboxValueBindingExample.

✖ Failed to compile.

If the problem is browser-specific, please specify the device, OS, browser, and version
No response
Provide any additional information here in as much as detail as you can
Happens on 2 listbox example I tried

Yeah CdkListboxModule is missing from material.module.ts.
I could provide a PR, if someone could point me where to look/do it !


Tabs: Fix disabling prevButton after upsizing (#6235)

Description
fixes #6235
Prev button remains visible after upsizing.
How Has This Been Tested?
unit | visually
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)

Before

After

Checklist:

[x] The PR is submitted to the correct branch (dev).
[x] My code follows the code style of this project.
[x] I've added relevant tests.

Idk why codecov shows decreased coverage on file that wasn't touched

Thanks @gaplin!


look into gradle texture packing

the runtime texture packing is less cool that i expected, probably easier at least for now with how things are, to just do that right now.

looks good so far, but seems there's some strange issue with my jenkins server running 'clean dist' with gradle wrapper...i cannot reproduce it from  locally built ones, only jenkins built ones. but it looks like the ui atlas gets not done/overridden by the builtin one coming from gdx. or something.

done, everything works..regular working built jars after each commit, uploaded to github releases


New option for postcss introducing import bug

Hello,
'postcss-import': {}, was added to .postcssrc.js in last release.
Currently this option is failing build using ~import in sass files for me.
I have one stylesheet with an import like : @import "~flatpickr/dist/flatpickr.css".
With 'postcss-import': {}, set, an error is thrown :
Module build failed: Error: Failed to find '~flatpickr/dist/flatpickr.css'
Without it, everything is fine.

Hm, that's very weird. Inside .vue file or inside a .scss file?
I don't really understand why because SCSS is supposed to take care of those imports ...

Sure quite easy : here you go

Thanks. Will take a look in a few hours, will be on the road until then.

So, I could verify the behaviour, and could also verify that removing the tilde makes it work:
``sass
@import 'flatpickr/dist/flatpickr.css'

But that is not a real solution because people used to sass-loader expect the code to work *with* the tilde.

Unfortunately, I'm a bit baffled why postcss-loader even processes this import. sass-loader is supposed to do that, itsn't it?

Note: postcss-import doesn't respect the ~ like other loaders, like css-loader or sass-loader do. That explains reason for the error, at least.
as to why postcss-loader is called in the first place: I think sass-loaer hands off loading the .css-file to the loader-config for that file type - which uses postcss-loader. I still don't find it logical that postcss-import is executed in that step (rather than on @imports inside of a .css file only), but that sems to be what happens.
So the easy way out would be to simply say: 'don't use tildes when importing .css, but use it for .sass and other preprocessor files', but that won't really be an acceptable solution, we'll have to figure out something else.

Thanks for the quick "hacky" fix. I do agree that remembering a special case is not ideal though.
We may have to reach out to the postcss-import maintainers no ?

No, I don't think that would be promising. I Dr a a couple of issues around this question, and it seems this is the "postcss" way of handling imports.
I will probably attempt use use css-loader's importLoaders option to let css-loader handle the imports, but then still need a solution for vue-loader which applies postcss without postcss-loader, so this option is not on the table.
Maybe we even need a fix in vur-loader itself.

I ran into this same problem yesterday and the issue is because

vue-loader uses PostCSS directly instead of relying on postcss-loader
sass-loader skips any imports for paths that end with .css

I was able to workaround the issue by omitting the file extension in the import statement

@jarrodldavis thanks for this.
While I don't think the first point is entirely valid/important (I also reproduced it with an in port I main.js, the second point was the missing piece for me.
The behaviour makes much more sense now, thanks.
I will still try and improve this behaviour, but now I understand what'd going on and that it's "expected".

I have the same issue with Bootstrap SASS files. They use their imports without extension (See Sass Basics) and I get the same error as @Tirke:
_functions.scss
// Normal code

bootstrap.scss
@import "functions";
// More imports

Error message
Module build failed: Error: Failed to find 'functions'
  in [
    project-path\node_modules\bootstrap\scss
  ]
    at resolveModule.catch.catch (project-path\node_modules\postcss-import\lib\resolve-id.js:35:13)
    at <anonymous>

 @ ./node_modules/vue-style-loader!./node_modules/css-loader?{"sourceMap":true}!./node_modules/vue-loader/lib/style-compiler?{"vue":true,"id":"data-v-04c2046b","scoped":false,"hasInlineConfig":false}!./node_modules/vue-loader/lib/selector.js?type=styles&index=0!./src/App.vue 4:14-296 13:3-17:5 14:22-304
 @ ./src/App.vue
 @ ./src/main.js
 @ multi (webpack)-dev-server/client?http://localhost:8080 webpack/hot/dev-server ./src/main.js

A temporal solution for me was importing main SCSS files in main.js, instead of SASS @import, like this
import Vue from 'vue'
import App from './App'

require('bootstrap/scss/bootstrap.scss');
require('../src/style/mycustomstyles.scss');

And it works like a charm 😉

I have the same issue when use vue-nprogress plugin (Failed to find '~nprogress/nprogress.css'), is there any solution for this issue yet, I dont want to custom this plugin by remove the tilde.

I'm having a more complicated and sticky problem with this.
vue-multiselect has styles that we would have to clobber with !important rules, unless we import them in scope. But our project uses submodules, and the main project is where package.json is .. so effectively I have to use webpack aliases like:
@import<EMAIL_ADDRESS>

that works when loaded from main.js in an scss file:
main.js
require('sass/app.scss')
//...

app.scss
@import<EMAIL_ADDRESS>...

but does not work from the component in the submodule:
<style lang="scss" type="text/scss" scoped>
@import<EMAIL_ADDRESS>
no other formulation works from the submodule either:
@import<EMAIL_ADDRESS>@import "~/vue-multiselect/dist/vue-multiselect.min.css";

@robbiemu
It may use css-loader to load .css file.

The error message says post-css

@robbiemu
For me, this does work:
<style type="text/css" lang="scss">
@import 'github-markdown-css/github-markdown.css'
</style>

But this fails:
<style type="text/css" lang="scss">
@import '~github-markdown-css/github-markdown.css'
</style>


Added export button to download conversation as csv

Related Issue
Closes #246

Description
Added export button to download conversation as csv
How Has This Been Tested?
local
Screenshots (if applicable)
https://github.com/user-attachments/assets/7e67f2b5-757c-47f1-835b-da8c522e8118

Type of Change

[ ] Bug fix
[x] New feature
[ ] Code style update
[ ] Breaking change
[ ] Documentation update

Use export-from-json npm package or similar for the task
Why your output CSV is different?
I thought it would be like
Headers
System, <System Prompt>
User, <User Query>
AI, <AI Response>
...

@kom-senapati : I am only fetching prompt from user and response from bot (2 columns in csv)

okay, let me try export-from-json

@kom-senapati : updated code to use export-from-json package.
Also, for csv columns, see the data

-- I think we only need user query and response fields

Oh that's also fine


added scale argument

This is a bit of a hack since I made this a required argument, but I found it nice to be able to scale my output images.  Thought others might as well.

Not to be nit picky, but there already is a "scale" PR that I am merging right now. Will close this once I do. ;)

added

Whoops my bad, jumped the gun.  And I see the other pr did that nasty nested "if" I was to lazy to bother with.  Looks good!


fix(browser): Error on browser platform

If you try to use this while running cordova run browser, this line will always fail because URI starts with 'http://localhost:8000/'. Changing it to 3 fixes this issue, but I'm unclear if there are unintended consequences. I'm not familiar enough with the code base to fully test etc, but hopefully someone with more knowledge can easily tell.

Rerun tests

CI: Once more please.


Automatically create GitHub issue when post-publish tests fail

Currently, our release-publish-suite-commit.yml action doesn't have a way to notify us of sanity test failures. This story is to make sure we open issues if those tests fail.
AC:

[ ] release-publish-suite-commit opens an issue in the repo if the prerequisite tests fail

This task is no longer valid since we now open PRs instead (which run tests as part of the automated steps).


Query Info status is not updated in Hive

The operation should be closed after the fetch is complete (has_more =false). To ensure that handleSet in Hive is recycled and query status is updated in a timely manner

One issue is then that you would not be able to download the results in certain cases.
There is some progress made with https://github.com/cloudera/hue/issues/1526 which will allow early closes of queries.