IntelliSense works properly yet error squiggles appear under include directives

Brief
IntelliSense autocomplete and linter work properly, even if error squiggles and the message "file not found" appear for certain #include directives

OS: ArchLinux 64bit
Version: 1.20.1-1

Steps to Reproduce:

Create a project structure like so:

.
├── common
│   └── foo.hpp
└── module
    └── src
        └── bar.hpp

My tessting has shown that directory names, file names and file extensions do not make a difference

In foo.hpp, create an abstract base class:

class Foo
{
public:
    virtual void method() = 0;
};

In bar.hpp, implement the abstract class:

#include <common/foo.hpp>
class Bar : public Foo
{
public:
    void method() override;
};

My testing has shown that there's no difference between the use of angle brackets and quotes

The result is that IntelliSense warns about common/foo.hpp not being found. However, IntelliSense finds the symbols defined in foo.hpp. Contrast this with commenting out the #include directive: The linter immediately warns about Foo being undefined and method not overriding a method defined in a base class.

Attached are my c_cpp_properties.json and a screenshot for reference
{
    "configurations": [
        {
            "name": "Linux",
            "includePath": [
                "/usr/include/c++/7.3.0",
                "/usr/include/c++/7.3.0/x86_64-pc-linux-gnu",
                "/usr/include/c++/7.3.0/backward",
                "/usr/lib/gcc/x86_64-pc-linux-gnu/7.3.0/include",
                "/usr/lib/gcc/x86_64-pc-linux-gnu/7.3.0/include-fixed",
                "/usr/local/include",
                "/usr/include",
                "${workspaceRoot}"
            ],
            "defines": [],
            "intelliSenseMode": "clang-x64",
            "browse": {
                "path": [
                    "/usr/include/c++/7.3.0",
                    "/usr/include/c++/7.3.0/x86_64-pc-linux-gnu",
                    "/usr/include/c++/7.3.0/backward",
                    "/usr/lib/gcc/x86_64-pc-linux-gnu/7.3.0/include",
                    "/usr/lib/gcc/x86_64-pc-linux-gnu/7.3.0/include-fixed",
                    "/usr/local/include",
                    "/usr/include",
                    "${workspaceRoot}"
                ],
                "limitSymbolsToIncludedHeaders": true
            }
        }
    ],
    "version": 3
}

This is a C++ IntelliSense specific issue, I accidentally posted in the wrong place.


feat: add first-class support for Chinese

Business value

Introduced Xiaolai-Regular for Chinese as a fallback font for all the built-in fonts (not set in stone, could be changed if we find a better candidate)

the font was manually split into multiple woff2 font faces with https://chinese-font.netlify.app/online-split

Improved wrapping to handle CJK codepoints as well as Emojis

in alpha, needs additional testing and possibly tweaks

Added keyboard shortcut for SVG export (ctrl+shift+c on all the platforms) with a toast once completed (similar to PNG).

Completes #8408
Technical parts

https://link.excalidraw.com/readonly/MbbnWPSWXgadXdtmzgeO

Improved overall SVG export to process FontFaces concurrently (with or without workers, fetching and processing max 3 at a time) with some additional perf. optimisations.
Offloaded SVG export in excalidraw-app to WebWorkers due to performance issues on the main thread (i.e. just regular Chinese Xiaolai consists of about 250 font faces, which would make the editor unresponsive for a few seconds).

as part of this introduced initial infra for a pool of short-lived idle workers (could be possibly re-used elsewhere)
added fallback for the main thread in case the WebWorkers are not available (i.e. running in node.js or when everything is one chunk as in the examples/obsidian)
for more fallbacks check the The NOT so happy path in the diagram

Adjusted the SVG export to inline fallback fonts first, so that their glyphs get overridden by the default fonts.
Refactored font and ExcalidrawFontFace-related logic (subsetting, toCSS API, utils build, font-faces in separate modules, introduced FONT_FAMILY_FALLBACKS etc.) and fixed various perf. / caching issues (i.e. bail not working)
Fixed the issues with cross-origin requests for fonts (DO) not being cached by the service-worker.
Adjusted the utils build process to merge all existing fonts with the big Chinese font (with normalized glyphs, merged licenses, etc.)
Adjusted the whole process for the E+ server-side export (processed on the main thread, fetched dataurl, skipped unnecessary expected logging, etc.)

TODO

[ ] Decide on the final Chinese font cc @dwelle
[ ] Test by native speaker cc @ryan-di (we might be missing some bits here, like some problems with Chinese IME, or breaking some semantics by incorrect wrapping)
[ ] Resolve TODO_CHINESE comments @Mrazator

Yeah, thank you for making such a great feature. I am Chinese. I have tested excalidraw-package-example and there is no problem. Especially the export PNG format is no problem.
The only small problem is: when exporting to svg, I put the svg into https://www.svgviewer.dev/ to view it, and the display is incorrect.
It has been done very well, looking forward to online as soon as possible. Great, very good, ...
@Mrazator

The only small problem is: when exporting to svg, I put the svg into svgviewer.dev to view it, and the display is incorrect.

Thanks for the feedback. About the viewer not being able to load the embedded fonts, it's likely they don't support
embedded fonts as dataurl, so it's more of a feature request for them. If you would open the SVG in the browser, you should see the fonts loaded correctly. Similarly, in SVG previews (space in macOS) or in some apps like Github / Notion:
Browser (Arc):

Preview (macOS):

Github:

Notion:

it seems that should be reviewed by @dwelle , expect to lanch sooner , tks

Thanks @Mrazator! Let's roll! 🙌🚀

I'm looking forward to seeing it ~~ Thank you @Mrazator


Stock-control-supervisors/.github

Stock control supervisors

Stock control supervisors

Stock control supervisors

Stock control supervisors


ENH: prepare wheels for pypi

This has several advantages; especially, one does not need C-compilers installed..
https://pythonwheels.com/

Dropped the need for Cython, so now it's just a pure Python package.


accents on characters inside \textit{} are incorrectly positioned

We need to take into account the skew (as well as the height of the character).

I've tracked the bug down to these lines of buildCommon.mathDefault

The bug seems to stem from the choice of font. $A$ renders an A in font family KaTeX_Math style italic, while $\textit{A}$ renders an A in font family KaTeX_Main style italic.  KaTeX correctly detects this, and uses the differing metrics already in fontMetricsData.js:
    "Math-Italic": {
        "47": [0.19444, 0.69444, 0, 0, 0],
        "65": [0, 0.68333, 0, 0.13889, 0.75],
        ...
    }, ...
    "Main-Italic": {
        ...
        "65": [0, 0.68333, 0, 0, 0.74333],
        ...
    },
`

0.13889 is the correct skew.  0 is not.

I assume the choice of fonts is "correct" (though we could of course use the math font for a letter A if we wanted...), so the "real" problem is the font metrics.  @kevinbarabash, any idea why the font metrics for Main-Italic would be different from the font metrics for Math-Italic, when the symbols look the same?

@edemaine not sure.  I'd have to see what we're doing in katex-fonts.  There is post-processing we do to some of the glyphs after extracting the paths from the source fonts.


Add new patreon supporter #372

Fixes #372
Add new supporter to Patreon list.

✅ By submitting this PR, I have verified the following

[x ] Checked to see if a similar PR has already been opened 🤔
[x ] Reviewed the contributing guidelines 🔍
[x ] Added my name to the bottom of the list under the Contributors section in the README.md with a link to my personal website or GitHub profile 👥 (Optional)

Verified changes on https://deploy-preview-379--techqueria.netlify.com/donate/patreon-supporters/
Thanks for the contribution @nathantomsfairsailcom 🙌🏽


Release pipeline improved

This should release both on github and pypi at the same time if a version tag is pushed to git.
Note: these tags can point to any branch, not just master. Of course, that still means only people with push permission to binsync can create them though. Presumably though, you will point your tags to master.

Thanks for your work @zwimer!


Problem with v17.3.1 : The active test run is aborted due to System.TimeSpan deserialization

Description

Tests fail due to this reason: Cannot deserialize the current JSON object (e.g. {"name":"value"}) into type 'System.TimeSpan'
Downgrading to Microsoft.NET.Test.Sdk v17.0.0 resolves this issue.

Steps to reproduce

The integration tests are for RestfulApi Controllers and are deriving from IClassFixture<WebTestFixture<Startup>>
A fake repository is mocked which returns an object containing a  System.TimeSpan property.

Actual behavior

When debugging the tests ,they run successfully until the end without breaking, and there is no indication that the test passed or failed. Output shows System.Exception is thrown from Newtonsoft.Json.Serialization.

Diagnostic logs

The active test run was aborted. Reason: Cannot deserialize the current JSON object (e.g. {"name":"value"}) into type 'System.TimeSpan' because the type requires a JSON primitive value (e.g. string, number, boolean, null) to deserialize correctly.
To fix this error either change the JSON to a JSON primitive value (e.g. string, number, boolean, null) or change the deserialized type so that it is a normal .NET type (e.g. not a primitive type like integer, not a collection type like an array or List) that can be deserialized from a JSON object. JsonObjectAttribute can also be added to the type to force it to deserialize from a JSON object.
Path 'Payload.TestRunCompleteArgs.ElapsedTimeInRunningTests.ticks', line 17, position 16.
Results File: /home/vsts/work/_temp/_fv-az158-963_2022-09-28_03_34_37.trx

Test Run Aborted with error System.Exception: One or more errors occurred.
---> System.Exception: Cannot deserialize the current JSON object (e.g. {"name":"value"}) into type 'System.TimeSpan' because the type requires a JSON primitive value (e.g. string, number, boolean, null) to deserialize correctly.
To fix this error either change the JSON to a JSON primitive value (e.g. string, number, boolean, null) or change the deserialized type so that it is a normal .NET type (e.g. not a primitive type like integer, not a collection type like an array or List) that can be deserialized from a JSON object. JsonObjectAttribute can also be added to the type to force it to deserialize from a JSON object.
Path 'Payload.TestRunCompleteArgs.ElapsedTimeInRunningTests.ticks', line 17, position 16.
at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.CreateObject(JsonReader reader, Type objectType, JsonContract contract, JsonProperty member, JsonContainerContract containerContract, JsonProperty containerMember, Object existingValue)
at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.CreateValueInternal(JsonReader reader, Type objectType, JsonContract contract, JsonProperty member, JsonContainerContract containerContract, JsonProperty containerMember, Object existingValue)
at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.SetPropertyValue(JsonProperty property, JsonConverter propertyConverter, JsonContainerContract containerContract, JsonProperty containerProperty, JsonReader reader, Object target)
at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.PopulateObject(Object newObject, JsonReader reader, JsonObjectContract contract, JsonProperty member, String id)

Environment

Microsoft.NET.Test.Sdk 17.3.1
TargetFramework net6.0
xunit.runner.visualstudio 2.4.5
xunit 2.4.2
coverlet.msbuild 3.1.2

Could you try to bump to 17.4.0 and let us know if you are still having the issue? If so, would it be possible to create a small repro?

I am on version 17.4.1 and still getting the error;
Microsoft (R) Test Execution Command Line Tool Version 17.4.0 (x64)
Copyright (c) Microsoft Corporation.  All rights reserved.

Starting test execution, please wait...
A total of 1 test files matched the specified pattern.
The active test run was aborted. Reason: Error converting value "PT0.002328S" to type 'System.TimeSpan'. Path 'Payload.NewTestResults[0].Duration', line 1, position 865.

Downgrading to Microsoft.NET.Test.Sdk v17.0.0 resolves this issue for me as well.


Broken PF tables

In our tables, we must use the title property for each PatternFly table cell. For example:
cells: [
  { title: <div>...</div> },
],

This was a breaking change with PatternFly 4 beta, but didn't show up in our build until the latest packages were installed.
Fixes https://github.com/project-koku/koku-ui/issues/835

Codecov Report

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

@@          Coverage Diff           @@
##           master    #836   +/-   ##
======================================
  Coverage    74.2%   74.2%
======================================
  Files         210     210
  Lines        3714    3714
  Branches      649     649
======================================
  Hits         2756    2756
  Misses        866     866
  Partials       92      92

Impacted Files
Coverage Δ

src/pages/sourceSettings/rowCell.tsx
63.63% <ø> (ø)
:arrow_up:

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 9fa0bf6...867f725. Read the comment docs.


Add MicroProfile stack deprecation blog

Description
Blog for deprecating the Java MicroProfile stack and switching to use the Java Open Liberty stack

@scottkurz and @SueChaplain  - Changes are made. Can you do a final review? Thanks!


How can i test webview inside my flutter Application using flutter driver.

Hi @atul161
This platform is not meant for assistance on personal code. Please see https://flutter.dev/community for resources and asking questions like this, you may also get some help if you post it on Stack Overflow. Closing, as this isn't an issue with Flutter itself, if you disagree please write in the comments and I will reopen it
Thank you


Add test to cover ETW payload serialization

Add ETW serialization test case for Application Insights Profiler agent.
See comments in #999

@cijothomas @MS-TimothyMothra can I get a review, please?

@cijothomas can you re-test?

I have retriggered the tests  - you can also do that by closing and re-opening the PR.
Btw the unit test failed due to an unrelated issue (AI backend returned 500- transient issue likely) - if it keeps failing, we'll force merge this.


Broken collected filter for available-domains endpoint

Description:
A request to /api/v2/available-domains?collected=eq:false does not return the not collected domains but an empty list instead. However, a request to  /api/v2/available-domains?collected=eq:true returns only collected domains, as expected. It seems the negative filtering is broken. Maybe because the attribute does not exist in this case?
Component(s) Affected:

[ ] UI
[x] API
[ ] Neo4j
[ ] PostgreSQL
[ ] Data Collector (SharpHound, AzureHound)
[ ] Other (tooling, documentation, etc.)

Steps to Reproduce:

Import the sample data from this repository
Perform the request to /api/v2/available-domains?collected=eq:true and see that the three collected domains are returned correctly.
Perform the request to /api/v2/available-domains?collected=eq:false and see that an empty list is returned instead of the two not collected domains.

Expected Behavior:
I would have expected that the negative filtering works as well as the positive filtering does.
Actual Behavior:
Negative filtering does not work.
Environment Information:
BloodHound: 5.9.0
Collector: what ever you used to generate the sample data
Potential Solution (Optional):
I could imagine this is because the collected property does not exist in the neo4j data for not collected domains. This needs to be worked around.
Contributor Checklist:

[x] I have searched the issue tracker to ensure this bug hasn't been reported before or is not already being addressed.
[x] I have provided clear steps to reproduce the issue.
[x] I have included relevant environment information details.
[x] I have attached necessary supporting documents.
[x] I have checked that any JSON files I am attempting to upload to BloodHound are valid.

Hey @exploide - I've validated this issue and I think you have the right idea. The collected property is not being set on all Domain or AZTenant nodes which breaks the query.
For now, I believe you can work around this by leveraging the following cypher query:
MATCH (n) WHERE (n:Domain OR n:AZTenant) AND n.collected IS NULL RETURN n

This was fixed in #684, thank you.


Create LICENSE

Add missing LICENSE file to fix broken links.
Both KevinnZou's original fork, compose-webview-accompanist and this project use broken links to this missing file.
Adding it should fix the issue.

If it's preferred to match the Multiplatform fork nomenclature, LICENSE.txt should created instead. However, that would mean the README.md 's should be edited as well.


Verify metadata.json integrity in CI?

As shown by https://github.com/bazelbuild/bazel-central-registry/pull/306, it's currently possible for the repository metadata to end up in a corrupted format.
I think the previous workflow we had that was removed with https://github.com/bazelbuild/bazel-central-registry/pull/237 would have caught this(though more "on accident" than being dedicated to this), so apparently the BuildKite presubmit checks aren't sufficient to catch this.
How do we want to deal this? Set up a dedicated metadata.json validation workflow?

The verify_stable_archives.py won't be able to catch this because it only reads the source.json file, not the metadata.json file.
We can implement the validation in https://github.com/bazelbuild/bazel-central-registry/blob/main/tools/bcr_validation.py

Copying my comment over from another thread:
I expected our validation script to catch it actually, but maybe json parsing failure isn't a hard error?
https://github.com/bazelbuild/bazel-central-registry/blob/8427967d922497e36dffe06bec46e8b1ead7edb4/tools/registry.py#L236-L239

Tested manually, json.load should throw the error
json.decoder.JSONDecodeError: Expecting ':' delimiter: line 27 column 17 (char 611)

The problem is that in https://github.com/bazelbuild/bazel-central-registry/pull/305 we only changed the metadata.json and not any specific module version, so the presubmit was totally bypassed.
https://buildkite.com/bazel/bcr-presubmit/builds/750#0184e213-962f-487e-ad45-6c1b21b2f68f
No target modules detected in this branch!


[FEAT] Add Role.clone() method

What type of pull request is this?

[x] Non-breaking code change
[ ] Breaking code change
[ ] Documentation change/addition
[ ] Tests change
[ ] CI change
[ ] Other: [Replace with a description]

Description

Two ways to close #731
Changes

add two implementations for Role.clone()

needs another commit to select the wanted implementation (DocStrings will be added then)

add class ClonedRole for one implementation

Test Scenario(s)

Checklist

[x] I've formatted my code with Black
[ ] I've added docstrings to everything I've touched
[ ] I've ensured my code works on Python 3.10.x
[ ] I've ensured my code works on Python 3.11.x
[ ] I've tested my changes
[ ] I've added tests for my code - if applicable
[ ] I've updated the documentation - if applicable

will merge the two methods and add a parameter force which defaults to True

Reference: NAFF #contributors

won't have time for it until Sunday (at least)

finally had time to make the commit
should all work by now


Confusing heading vs the body of the paragraph - m greater than 5

On this page https://btcguide.github.io/quorum-advanced
Shouldn't the heading at the bottom read "M>5: Nonstandard transactions"  rather than "N>5: Nonstandard transactions" ?

I think my info here is very outdated, do you have a source on the current standard sizes?
https://twitter.com/mflaxman/status/1339718993456852993

I don't sorry. I'd like to help out with this guide (I rely on it a lot and think it's fantastic), maybe you could tag some "good first issues" ? Or is there any particular area you'd like the guide to improve in ? Maybe screenshots ? Or adding explanations on Advanced to dos ?


contour_px not respected when outline_alpha == fill_alpha

          @timtreis Probably just an edge case, but if the user increases the outline width when `fill_alpha == outline_alpha` one would expect the labels to get bigger which would not happen since it's treated as no-outline, correct?

Originally posted by @Sonja-Stockhaus in https://github.com/scverse/spatialdata-plot/pull/323#discussion_r1735165359


bump dependencies

Signed-off-by: Zbynek Roubalik<EMAIL_ADDRESS>
This should fix vulnerabilities found in Snyk.io
https://cwe.mitre.org/data/definitions/1333.html
https://cwe.mitre.org/data/definitions/400.html
Checklist

[x] Commits are signed with Developer Certificate of Origin (DCO - learn more)

/run-e2e


Please Add Typscript supoort for better intellisence and better overall developer experience

I am using typescript for my development and the Flutterwave SDK is good however it lacks typescript support which is crucial for catching bugs in build time rather than shipping bugs to customers in production.

I second this. Typescript support is really needed for this package

Hello Everyone 👋  (including the Flutterwave Node Team)
I was planning on integrating Flutterwave into one of my projects as well and was a bit sad that there wasn't typescript support or type sensing.
In order to address this, as well as your concerns, I was able to fork the repository and begin working on adding types to the library.
The library is a big one, so I am starting with making use of JSDoc and generating .d.ts files from them. That way, adding types to the library can be done incrementally, and eventually (with the help of the Flutterwave team), it can be implemented completely in TypeScript
The fork is on my repo: https://github.com/nikeokoronkwo/Node-v3
Let me know what you guys think, and maybe if the Flutterwave team is willing I can finish it up and make a pull request.
Contributions are very welcome, and I would be glad to get any support if possible for this.


Copy action in terminal is inserting empty lines between each line

VSCode Version: 1.28.2 (user setup)
OS Version: Windows 10

Steps to Reproduce:

right-click in terminal, "Copy all"
paste it in notepad for instance
it occurs also when creating a new file in VS Code and pasting in it.

you can se the terminal log without empty lines at the bottom and the pasted content in the editor

Does this issue occur when all extensions are disabled?: Yes/No

this is not a duplicated of the mentioned issue

This bug only seems to happen in the Debug Console, not the integrated terminal.
Was not able to reproduce on Arch Linux, but I could repro on Windows 10.


uv pip install fails on network drive

I'm developing a plugin called pymat built with maturin and deployed as a wheel to a (company) network drive. The folder contains all versions of the plugin and im trying to install the most recent one. This works in pip as expected. When trying to install it with uv (Windows, 0.1.38 (0b23caa18 2024-04-24)), uv panics. Currently two wheels are in the folder, 0.1.0 and 0.2.0.
I invoke with the following command:
uv pip install --no-index --upgrade --find-links "file:\\some.network.net\path\to\folder\pymat" pymat --verbose

INFO Found a virtualenv through VIRTUAL_ENV at: C:\git_work\some_project\.venv
DEBUG Cached interpreter info for Python 3.12.0, skipping probing: .venv\Scripts\python.exe
DEBUG Using Python 3.12.0 environment at .venv\Scripts\python.exe
DEBUG Trying to lock if free: .venv\.lock
DEBUG Using registry request timeout of 30s
DEBUG Found 2 packages in `--find-links` entry: \\some.network.net\path\to\folder\pymat
DEBUG Solving with target Python version 3.12.0
DEBUG Adding direct dependency: pymat*
DEBUG Searching for a compatible version of pymat (*)
DEBUG Selecting: pymat==0.2.0 (pymat-0.2.0-cp312-none-win_amd64.whl)
DEBUG Tried 2 versions: pymat 1, root 1
Resolved 1 package in 67ms
thread 'main' panicked at crates\cache-key\src\canonical_url.rs:35:32:
called `Result::unwrap()` on an `Err` value: ()
stack backtrace:
   0:     0x7ff614e944d2 - git_odb_object_data
   1:     0x7ff614ebdc2d - git_odb_object_data
   2:     0x7ff614e8e091 - git_odb_object_data
   3:     0x7ff614e942fa - git_odb_object_data
   4:     0x7ff614e97009 - git_odb_object_data
   5:     0x7ff614e96cc5 - git_odb_object_data
   6:     0x7ff614e97524 - git_odb_object_data
   7:     0x7ff614e973f9 - git_odb_object_data
   8:     0x7ff614e94dd9 - git_odb_object_data
   9:     0x7ff614e970c6 - git_odb_object_data
  10:     0x7ff61508c747 - git_midx_writer_new
  11:     0x7ff61508cc83 - git_midx_writer_new
  12:     0x7ff614c10439 - git_filter_source_repo
  13:     0x7ff614a12f27 - git_filter_source_repo
  14:     0x7ff6148b3207 - git_filter_source_repo
  15:     0x7ff61489ec32 - git_filter_source_repo
  16:     0x7ff61489e8b3 - git_filter_source_repo
  17:     0x7ff61479fc4b - git_filter_source_repo
  18:     0x7ff6145ce9d5 - git_filter_source_repo
  19:     0x7ff61460cb25 - git_filter_source_repo
  20:     0x7ff6145ac3d0 - git_filter_source_repo
  21:     0x7ff6142b34a5 - git_filter_source_repo
  22:     0x7ff61401897b - <unknown>
  23:     0x7ff6140a795f - git_filter_source_repo
  24:     0x7ff614090316 - <unknown>
  25:     0x7ff614e846d2 - git_odb_object_data
  26:     0x7ff6140a847c - git_filter_source_repo
  27:     0x7ff61502e6ec - git_midx_writer_new
  28:     0x7ffba7a37344 - BaseThreadInitThunk
  29:     0x7ffba99226b1 - RtlUserThreadStart

Further information:

When using the mounted drive G: instead of file:\some.network.net\fs\ the same error occurs.
When copying the folder to C:\temp\pymat I can install with uv pip install --no-index --upgrade --find-links C:\temp\pymat pymat resulting in the following instead of the panic:
Installed 1 package in 41ms

pymat==0.2.0

When the plugin is already installed, auditioning works and doesnt crash uv.

I don't have a network mount to test myself, but could you try https://github.com/astral-sh/uv/pull/3306? This should remove the panic

Sure, will try tomorrow. Do i Need to build from source or can i access the executable from the ci somewhere?

You'd need to build from source, but we release really often so it may just be out soon.

Since it seems my companys "threat detection" keeps deleting uv-trampoline-x86_64-gui.exe I'll check with the next release and report back.

Tried with release 0.1.41 and worked like expected. Thanks alot for the fix and keep up the great work :)


Document options for configuring Dapr command args

I had to go spelunking to understand how to configure individual services Dapr arguments and finally stumbled across it.  In addition, while specifying a custom components path was available in some documentation somewhere, it seems helpful to have it sit alongside the documentation on how to include the Dapr extension.
I'd be happy to include a list of the arguments if that would be useful but didn't want the documentation to get out of sync with the code that reads the Dapr extension configuration.

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

I'd be happy to include a list of the arguments if that would be useful but didn't want the documentation to get out of sync with the code that reads the Dapr extension configuration.

I could go either way; I think it's more of an issue that, without the list, developers may attempt to set some new property added by the Dapr CLI but not yet supported by Tye, which could lead to confusion.  An explicit list would make it more obvious which subset of CLI arguments are supported by Tye.

I could go either way; I think it's more of an issue that, without the list, developers may attempt to set some new property added by the Dapr CLI but not yet supported by Tye, which could lead to confusion. An explicit list would make it more obvious which subset of CLI arguments are supported by Tye.

I've updated with all of the keys that can be specified (I think).  I started with a single table that had an additional column for "Scope" which was Extension, Scope, or Extension, Scope.  I'm divided on whether I prefer a single table or the two I have now.
I also threw enabled in there since, as far as I can tell, it's not documented anywhere else and I figured I'd go for comprehensive since the rest were in there.


Support for Postgresql as configuration store

Description
Added Support for Postgres as configuration store
Issue reference
We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.
Please reference the issue this PR will close: #4551
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[X] Code compiles correctly
[X] Created/updated tests
[X] Extended the documentation / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#2779

@ItalyPaleAle added non-alphanumeric character check and prepared statements to buildQuery

What is the statement to create the table, by the way?

@ItalyPaleAle - configuration API does not support creation on config table/triggers. So these are external pre-requisites until Admin API is supported. The steps to setup postgres as configuration store can be referenced in https://github.com/dapr/docs/pull/2800.
Regarding Prepared statement - We donot pre-define the prepared statements as Users can query on any number of keys and/or metadata attributes. A sample generated query to GET configuration data based on Key and Metadata would be -  "SELECT * FROM cfgtbl WHERE KEY IN ($1) AND $2 = $3".
pgxPool.Pool.Query builds a prepared statement and caches it. Every time the statement that is already cached is used, pgx uses the prepared statement from the cache.

@ItalyPaleAle  - closed comments. FYI Please

Regarding Prepared statement - We donot pre-define the prepared statements as Users can query on any number of keys and/or metadata attributes. A sample generated query to GET configuration data based on Key and Metadata would be - "SELECT * FROM cfgtbl WHERE KEY IN ($1) AND $2 = $3".
pgxPool.Pool.Query builds a prepared statement and caches it. Every time the statement that is already cached is used, pgx uses the prepared statement from the cache.

Using $1 etc is using prepared statements: in this case, the query doesn't contain the exact values, but they are only referenced like you're doing there. Prepared statements are needed to avoid SQL injections

@ItalyPaleAle - We cannot pre define prepared statements as there is no cap on number of keys that can be queried for and/if metadata is provided. So we build the query and depend on the functionality of pgxpool library to convert these to prepared statements. SQL Injection scenarios are addressed via input validation.
Can you please clarify on where you see SQL Injection possibility?

What is the statement to create the table, by the way?

https://github.com/dapr/docs/blob/357ffcf6dd53d5736461c7e63050da5917aa6bbf/daprdocs/content/en/reference/components-reference/supported-configuration-stores/postgres-configuration-store.md

@mukundansundar - can you please check the open clarifications and comment if this PR is good to merge? Please note that these are not code fixes but clarifications only.
@artursouza - FYI Please

@mukundansundar  - closed comments. can you please check

@ItalyPaleAle  - Comments are addressed. If there are no critical issue, please approve the review.

https://github.com/dapr/dapr/issues/5236 - PR to track panic in runtime

LGTM!

@akhilac1 please create documentation if you haven't already

@akhilac1 please create documentation if you haven't already

@berndverst -  https://github.com/dapr/docs/issues/2779


fix GitHub authentication

@ank1traj
The URLs used to configure the API are probably incorrect.
I have replaced the GITHUB_CLIENT_ID and GITHUB_CLIENT SECRET with mine and the API is working fine now.
Please update the URI fields in the API console as follows.

Screenshot of the successful API call.


https://github.com/FreeCodeCamp/FreeCodeCamp.git

FreeCodeCamp Issue template
To Use this Template:

Fill out what you can
Delete what you do not fill out

NOTE: ISSUES ARE NOT FOR CODE HELP - Ask for Help at https://gitter.im/FreeCodeCamp/Help
Issue Description

When Issue Happens
Steps To Reproduce

Browser Information

Browser Name, Version
Operating System

Your Code
If relevant, paste all of your challenge code in here

Screenshot

Reopen this if there is an actual issue and provide details to explain the issue.

Look I don't know what you talking about and I wake up to hear this all day
and haven't been able to do anything without any problems in the
understanding of what people are saying I don't know what you talking about
thank you
Gary M Barker
On Feb 19, 2016 3:13 PM, "Brian"<EMAIL_ADDRESS>wrote:

Reopen this if there is an actual issue and provide details to explain the
issue.
—
Reply to this email directly or view it on GitHub
https://github.com/FreeCodeCamp/FreeCodeCamp/issues/7153#issuecomment-186444969
.


(MODULES-8304) Allow whitespace in mountpoints

WIP and probably broken but works on Linux at least.
This changes spaces to \040 when writing to fstab, then changes back when reading from the file. Also extends the regex mount matcher to get whitespaces in paths.
//RJLI5MAHQOJWPE9/path with spaces on /SERVER/path with spaces type cifs (rw,relatime...)

Passing acceptance run: https://jenkins-master-prod-1.delivery.puppetlabs.net/view/modules/view/Core/view/mount_core/view/adhoc/job/forge-module_puppetlabs-mount_core_intn-sys_nightly-adhoc/2/


[IMPROVEMENT] SPDK client should functions after encountering decoding error

Is your improvement request related to a feature? Please describe (👍 if you like this request)
When issue #6190 is triggered, all following cmd responses will be failed by the invalid json error. This is probably caused by the cache of the reused decoder of the client (but we cannot confirm this).
Describe the solution you'd like

Use a brand new decoder for each cmd call
In case using the separate decoder is not able to resilient this issue, the SPDK engine can re-init the client when encountering the error.

Additional context
See issue #6190 for details. This is just a workaround.

@shuo-wu is this possible to verify by QA?

Since we don't have a way to reproduce the issue, hence testing the fixes are hard.
BTW, there is no need to update the vendor for longhorn-manager

Moving to ready-for-testing, but pending before release to see if this will be reproduced again.
cc @yangchiu Feel free to close this after 1.5.0 GA.

Installed Longhorn v1.5.0-rc3, and tried to trigger spdk volume offline replica rebuilding by deleting replicas or killing the spdk_tgt process in instance-manager. Everything works well without problem. Close this for now.


Build from sources fail with pip3 install -e .

Issue summary
pip3 install -e . and sudo pip3 install -e . fail with the following output:
https://pastebin.com/QrEp9iKD
and the important part of this output is the following:
In file included from mednafen/pce_fast/huc6280.cpp:18:
In file included from mednafen/pce_fast/pce.h:5:
In file included from mednafen/pce_fast/../mednafen.h:12:
./mednafen/git.h:4:10: fatal error: 'string' file not found
#include <string>
         ^~~~~~~~
1 error generated.
make[4]: *** [mednafen/pce_fast/huc6280.o] Error 1
make[3]: *** [retro/cores/mednafen_pce_fast_libretro.dylib] Error 2
make[2]: *** [CMakeFiles/pce.dir/all] Error 2
make[2]: *** Waiting for unfinished jobs....

System information

[Operating system] Mac Os Mojave (10.14)
[Python version] 3.7.0
[Gym Retro version] 0.6.1

I thought the problem was related to the recent update of Mac Os (I've reinstalled xcode, updated brew formulae, etc.) but it has not solved the problem.
Have you an idea from where the problem may come?
Thank you for your answer.

Can you post the output of running make VERBOSE=1 after this fails?

Please see if the mojave-buildfix branch fixes it.

make VERBOSE=1
/usr/local/Cellar/cmake/3.12.2/bin/cmake -H/Users/XXXX/workspace/gym-retro -B/Users/XXXX/workspace/gym-retro --check-build-system CMakeFiles/Makefile.cmake 0
/usr/local/Cellar/cmake/3.12.2/bin/cmake -E cmake_progress_start /Users/XXXX/workspace/gym-retro/CMakeFiles /Users/XXXX/workspace/gym-retro/CMakeFiles/progress.marks
/Library/Developer/CommandLineTools/usr/bin/make -f CMakeFiles/Makefile2 all
/Library/Developer/CommandLineTools/usr/bin/make -f third-party/libzip/lib/CMakeFiles/zip.dir/build.make third-party/libzip/lib/CMakeFiles/zip.dir/depend
cd /Users/XXXX/workspace/gym-retro && /usr/local/Cellar/cmake/3.12.2/bin/cmake -E cmake_depends "Unix Makefiles" /Users/XXXX/workspace/gym-retro /Users/XXXX/workspace/gym-retro/third-party/libzip/lib /Users/XXXX/workspace/gym-retro /Users/XXXX/workspace/gym-retro/third-party/libzip/lib /Users/XXXX/workspace/gym-retro/third-party/libzip/lib/CMakeFiles/zip.dir/DependInfo.cmake --color=
/Library/Developer/CommandLineTools/usr/bin/make -f third-party/libzip/lib/CMakeFiles/zip.dir/build.make third-party/libzip/lib/CMakeFiles/zip.dir/build
make[2]: Nothing to be done for `third-party/libzip/lib/CMakeFiles/zip.dir/build'.
[ 70%] Built target zip
/Library/Developer/CommandLineTools/usr/bin/make -f CMakeFiles/pce-submodule.dir/build.make CMakeFiles/pce-submodule.dir/depend
cd /Users/XXXX/workspace/gym-retro && /usr/local/Cellar/cmake/3.12.2/bin/cmake -E cmake_depends "Unix Makefiles" /Users/XXXX/workspace/gym-retro /Users/XXXX/workspace/gym-retro /Users/XXXX/workspace/gym-retro /Users/XXXX/workspace/gym-retro /Users/XXXX/workspace/gym-retro/CMakeFiles/pce-submodule.dir/DependInfo.cmake --color=
/Library/Developer/CommandLineTools/usr/bin/make -f CMakeFiles/pce-submodule.dir/build.make CMakeFiles/pce-submodule.dir/build
/usr/local/bin/git submodule update --init cores/pce
[ 70%] Built target pce-submodule
/Library/Developer/CommandLineTools/usr/bin/make -f CMakeFiles/pce.dir/build.make CMakeFiles/pce.dir/depend
cd /Users/XXXX/workspace/gym-retro && /usr/local/Cellar/cmake/3.12.2/bin/cmake -E cmake_depends "Unix Makefiles" /Users/XXXX/workspace/gym-retro /Users/XXXX/workspace/gym-retro /Users/XXXX/workspace/gym-retro /Users/XXXX/workspace/gym-retro /Users/XXXX/workspace/gym-retro/CMakeFiles/pce.dir/DependInfo.cmake --color=
/Library/Developer/CommandLineTools/usr/bin/make -f CMakeFiles/pce.dir/build.make CMakeFiles/pce.dir/build
[ 70%] Generating retro/cores/mednafen_pce_fast_libretro.dylib
cd /Users/XXXX/workspace/gym-retro/cores/pce && /usr/local/Cellar/cmake/3.12.2/bin/cmake -E env CFLAGS=-w CXXFLAGS=-w LDFLAGS=-w\  /Library/Developer/CommandLineTools/usr/bin/make -f Makefile CC="/Library/Developer/CommandLineTools/usr/bin/cc" CXX="/Library/Developer/CommandLineTools/usr/bin/c++"
/Library/Developer/CommandLineTools/usr/bin/c++ -c -omednafen/pce_fast/huc6280.o mednafen/pce_fast/huc6280.cpp  -w -DGIT_VERSION=\"" 35e96c4"\" -DHAVE_MKDIR -DWANT_32BPP -DWANT_NEW_API -DFRONTEND_SUPPORTS_RGB565 -DNEED_CD -DHAVE_CHD -D_7ZIP_ST -DPACKAGE_VERSION=\"1.3.2\" -DFLAC_API_EXPORTS -DFLAC__HAS_OGG=0 -DHAVE_LROUND -DHAVE_STDINT_H -DHAVE_STDLIB_H -DHAVE_SYS_PARAM_H -DNEED_TREMOR -O2 -DNDEBUG -funroll-loops -fPIC -mmacosx-version-min=10.1  -I. -I./mednafen -I./mednafen/include -I./mednafen/intl -I./mednafen/hw_sound -I./mednafen/hw_cpu -I./mednafen/hw_misc -I./libretro-common/include -I./deps/zlib -I./deps/crypto -I./deps/flac-1.3.2/include -I./deps/flac-1.3.2/src/libFLAC/include -I./deps/lzma-16.04/C -I./deps/libchdr   -DSIZEOF_DOUBLE=8 -Wall -Wno-sign-compare -Wno-unused-variable -Wno-unused-function -Wno-uninitialized  -Wno-strict-aliasing -DPACKAGE=\"mednafen\" -DMEDNAFEN_VERSION_NUMERIC=931 -DPSS_STYLE=1 -DMPC_FIXED_POINT -DWANT_PCE_FAST_EMU -DWANT_STEREO_SOUND -DSTDC_HEADERS -D__STDC_LIMIT_MACROS -D__LIBRETRO__ -D_LOW_ACCURACY_   -DINLINE="inline" -DFRONTEND_SUPPORTS_RGB565 -DWANT_NEW_API
In file included from mednafen/pce_fast/huc6280.cpp:18:
In file included from mednafen/pce_fast/pce.h:5:
In file included from mednafen/pce_fast/../mednafen.h:12:
./mednafen/git.h:4:10: fatal error: 'string' file not found
#include <string>
         ^~~~~~~~
1 error generated.
make[3]: *** [mednafen/pce_fast/huc6280.o] Error 1
make[2]: *** [retro/cores/mednafen_pce_fast_libretro.dylib] Error 2
make[1]: *** [CMakeFiles/pce.dir/all] Error 2
make: *** [all] Error 2

Fixed and merged into master.


Fix tab switch in menubar

There was a of name collision for switchTab!
It was being set to dispatch(switchTo(...) but because bindActionCreators was declared later, it was overwritten by the switchTab declared in actions/favorite.js.
I renamed switchTab to favoriteSwitchTab and moved the ...bindActionCreators to the first element.

The commits 317b9930e698...ccc71d3dfa48 introduce visual changes on linux. :mag_right: 1 changed

Changed: Chat_Input-Emoji_Open (before) (after)

@keybase/react-hackers

The commits 317b9930e698...721f0ee8fa4c introduce visual changes on linux. :mag_right: 1 changed

Changed: Chat_Input-Emoji_Open (before) (after)

bump


Getting compile error after upgrade --

Hey there!  0.0.8 is giving an error in the latest Dart / Flutter with the error being:
Compiler message:
file:///Users/sjm/Development/tools/flutter/.pub-cache/hosted/pub.dartlang.org/icons_helper-0.0.8/lib/icons_helper.dart:963:31: Error: Getter not
found: 'rendact'.
  'rendact': FontAwesomeIcons.rendact,
                                                      ^^^^^^^

And -- solved -- had to issue this set of commands:
$ flutter clean
$ rm -rf  ~/Development/tools/flutter/.pub-cache/hosted
$ flutter packages get
Then it works again .. strange!

OK -- I was maybe a bit premature -- for some reason doing a "flutter run " works but when I do a flutter build it does not :(
flutter build ios --release lib/main_preview.dart
=== BUILD TARGET Runner OF PROJECT Runner WITH CONFIGURATION Release ===
Building AOT snapshot in release mode (ios-release)...

Compiler message:
file:///Users/sjm/Development/tools/flutter/.pub-cache/hosted/pub.dartlang.org/icons_helper-0.0.8/lib/icons_helper.dart:963:31: Error: Getter
not found: 'rendact'.
  'rendact': FontAwesomeIcons.rendact,
                              ^^^^^^^
Compiler terminated unexpectedly.
Failed to build /Users/sjm/work/mm/mms4/mobile.

Sigh.. any ideas here?

OK -- looking -- icons_helper is looking for rendact as an icon -- looking at font_awesome_flutter 8.2.0 (the latest) that is NOT a valid Icon anymore .. need to remove it from here I think?

PR - https://github.com/VoltaireApp/icons_helper/pull/2  created

@24601 -- Hello!?

Hello nice work, but...
With a new project, I have exact the same problem:
file:///C:/src/flutter/.pub-cache/hosted/pub.dartlang.org/icons_helper-0.0.8/lib/icons_helper.dart:963:31: Error: Getter not found: 'rendact'.
'rendact': FontAwesomeIcons.rendact,

Just use mine :
icons_helper:
git: git://github.com/sjmcdowall/icons_helper

@sjmcdowall   Thanks for your feedback! I will try!!

Excelente aporte!

See #3


Game using if-else.

Making a game using if else only.
I would like to work on this issue.

Wanna work on this issue


HBASE-28769 Create PoC for RangeStoreFileReader to support multi regi…

…on splitting

Created initial range store file reader. Just raised PR for quick glance. Will add test cases and update the PR.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 33s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
hbaseanti
0m  0s

Patch does not have any anti-patterns.

_ master Compile Tests _

+1 :green_heart:
mvninstall
3m 14s

master passed

+1 :green_heart:
compile
3m  0s

master passed

+1 :green_heart:
checkstyle
0m 36s

master passed

+1 :green_heart:
spotbugs
1m 35s

master passed

+1 :green_heart:
spotless
0m 45s

branch has no errors when running spotless:check.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
3m  0s

the patch passed

+1 :green_heart:
compile
3m  0s

the patch passed

+1 :green_heart:
javac
3m  0s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

-0 :warning:
checkstyle
0m 36s
/results-checkstyle-hbase-server.txt
hbase-server: The patch generated 8 new + 0 unchanged - 0 fixed = 8 total (was 0)

+1 :green_heart:
spotbugs
1m 40s

the patch passed

+1 :green_heart:
hadoopcheck
11m 34s

Patch does not cause any errors with Hadoop 3.3.6 3.4.0.

-1 :x:
spotless
0m 37s

patch has 63 errors when running spotless:check, run spotless:apply to fix.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
3m  7s

the patch passed

+1 :green_heart:
compile
0m 57s

the patch passed

+1 :green_heart:
javac
0m 57s

the patch passed

+1 :green_heart:
javadoc
0m 27s

the patch passed

+1 :green_heart:
shadedjars
5m 47s

patch has no errors when building our shaded downstream artifacts.

_ Other Tests _

+1 :green_heart:
unit
210m 40s

hbase-server in the patch passed.

235m 43s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6141/1/artifact/yetus-jdk17-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/6141

Optional Tests
javac javadoc unit compile shadedjars

uname
Linux 7068fbe2dd64 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 20362ad2692cde45da2e12aa0348ac2bd29dd2fd

Default Java
Eclipse Adoptium-17.0.11+9

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6141/1/testReport/

Max. process+thread count
5112 (vs. ulimit of 30000)

modules
C: hbase-server U: hbase-server

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6141/1/console

versions
git=2.34.1 maven=3.9.8

Powered by
Apache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

Added test cases similar to halfstore file reader covering most of the happy path. Need to add few more scenarios of corner case test cases. Currently create RangeReference file but need to define the pattern for range reference and StoreFileInfo should automatically identify the Range Reference.
I have added few more TODOs like java docs etc..Will complete them.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 26s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
buf
0m  0s

buf was not available.

+0 :ok:
buf
0m  0s

buf was not available.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
hbaseanti
0m  0s

Patch does not have any anti-patterns.

_ master Compile Tests _

+0 :ok:
mvndep
0m 10s

Maven dependency ordering for branch

+1 :green_heart:
mvninstall
3m  9s

master passed

+1 :green_heart:
compile
3m 38s

master passed

+1 :green_heart:
checkstyle
0m 42s

master passed

+1 :green_heart:
spotbugs
3m 51s

master passed

+1 :green_heart:
spotless
0m 47s

branch has no errors when running spotless:check.

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 11s

Maven dependency ordering for patch

+1 :green_heart:
mvninstall
3m  4s

the patch passed

+1 :green_heart:
compile
3m 35s

the patch passed

+1 :green_heart:
cc
3m 35s

the patch passed

+1 :green_heart:
javac
3m 35s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

-0 :warning:
checkstyle
0m 36s
/results-checkstyle-hbase-server.txt
hbase-server: The patch generated 20 new + 0 unchanged - 0 fixed = 20 total (was 0)

+1 :green_heart:
spotbugs
4m  7s

the patch passed

+1 :green_heart:
hadoopcheck
11m 35s

Patch does not cause any errors with Hadoop 3.3.6 3.4.0.

+1 :green_heart:
hbaseprotoc
1m 22s

the patch passed

+1 :green_heart:
spotless
0m 43s

patch has no errors when running spotless:check.

_ Other Tests _

+1 :green_heart:
asflicense
0m 17s

The patch does not generate ASF License warnings.

45m 43s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6141/2/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/6141

Optional Tests
dupname asflicense cc buflint bufcompat codespell detsecrets hbaseprotoc spotless javac spotbugs checkstyle compile hadoopcheck hbaseanti

uname
Linux 810b7668a3bb 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 4cc18c7695a2b9ec105e51301b4273fd03fe0009

Default Java
Eclipse Adoptium-17.0.11+9

Max. process+thread count
87 (vs. ulimit of 30000)

modules
C: hbase-protocol-shaded hbase-server U: .

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 12s

Maven dependency ordering for patch

+1 :green_heart:
mvninstall
3m  8s

the patch passed

+1 :green_heart:
compile
1m 30s

the patch passed

+1 :green_heart:
javac
1m 30s

the patch passed

+1 :green_heart:
javadoc
0m 36s

the patch passed

+1 :green_heart:
shadedjars
5m 41s

patch has no errors when building our shaded downstream artifacts.

_ Other Tests _

+1 :green_heart:
unit
0m 33s

hbase-protocol-shaded in the patch passed.

-1 :x:
unit
210m 20s
/patch-unit-hbase-server.txt
hbase-server in the patch failed.

238m 27s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6141/2/artifact/yetus-jdk17-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/6141

Optional Tests
unit javac javadoc compile shadedjars

uname
Linux c4eac847de2a 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 4cc18c7695a2b9ec105e51301b4273fd03fe0009

Default Java
Eclipse Adoptium-17.0.11+9

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6141/2/testReport/

Max. process+thread count
5462 (vs. ulimit of 30000)

modules
C: hbase-protocol-shaded hbase-server U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6141/2/console

versions
git=2.34.1 maven=3.9.8

Powered by
Apache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.


Build Error at Building 'tensorflow/core/distributed_runtime/eager:remote_tensor_handle_data'

System information

OS Platform and Distribution : Linux UBUNTU 18.04
My device (Platform): Odroid XU4
TensorFlow installed from Source
TensorFlow version: 1.15.3 (and this issue appear in 1.15.0, 1.15.1)
Python version: 3
Installed using virtualenv? pip? conda?: No pip package, Only C++ lib
Bazel version : 0.26.1
GCC/Compiler version : 7
CUDA/cuDNN version: No CUDA Suppot, CPU-Only
GPU model and memory: Armhf with 1GB Ram and 4G SWAP Memory

Describe the problem
I'm Trying instatal TensorFlow 1.15 on ODROID XU4, I'm success in installing TensorFlow 1.13 by Source Compiling on my Hardware but for TF 1.15 after few days and many efforts get an Error that i don't know what caused and how fixes.
 FAILED: Build did NOT complete successfully
I think main error log is this LIne:
tensorflow/core/distributed_runtime/eager/remote_tensor_handle_data.cc:30:12: error: 'class tensorflow::EagerContext' has no member named 'GetContextId'; did you mean 'NewContextId'?    if (ctx->GetContextId() != context_id) {             ^~~~~~~~~~~~             NewContextId

@Yasin40
Please paste the error message (using makrdown formatting around it) instead of screenshotting. Screenshots are not searchable so they don't help in looking for the issue and also don't help other people having the same error from finding about the issue.

@Saduf2019 .
Thanks for attention. I pasted error before screenshot. bu if you complete than:
INFO: Found 1 target...

Hi There,
We are checking to see if you still need help on this, as you are using an older version of tensorflow which is officially considered end of life . We recommend that you upgrade to the latest 2.x version and let us know if the issue still persists in newer versions. Please open a new issue for any help you need against 2.x, and we will get you the right help.
This issue will be closed automatically 7 days from now. If you still need help with this issue, please provide us with more information.


I activated Poppovers by clicking the button and got the following error, but my code attribute value print tells me that the system has given the correct Boolean value. But the parcel was not shown in Poppovers

Attempt to present <TtGC7SwiftUI29PresentationHostingControllerVS_7AnyView: 0x124db27e0> on <TtGC7SwiftUI19UIHostingControllerGVS_15ModifiedContentVS_7AnyViewVS_12RootModifier_: 0x107013f20> (from <TtGC7SwiftUI41StyleContextSplitViewNavigationControllerVS_14NoStyleContext: 0x10683d600>) which is already presenting <sniffer_knife.PopoverContainerViewController: 0x116008170>.
The result of the above error：
When clicking on this error, the system can correctly feedback the Boolean value, but the view does not pop up and appear on the screen

I have the same issue here: https://github.com/aheze/Popovers/issues/11. I'm working on a fix.

Wait for the repair

@wisepmlin fixed in 1.2.0! Leave a comment if it still doesn't work.


Screen reader speaks the message "Occurs n times" Repeatedly When Searching For Extensions

VSCode Version:
OS Version:

Steps to Reproduce:

From the application, press CTRL+SHIFT+P.
Slowly type characters and after each one, listen for the phrase. For example, slowly type the word "python."

Does this issue occur when all extensions are disabled?: Yes/No

@jhomme I believe you meant Ctrl + Shift + X to open the extensions viewlet, correct?
@sandy081 I believe the issue is that in the extensions view we use alert too much. Code pointer
To be honest I think most of those alerts are not that useful.
Also I do not understand why an alert here is needed
@sandy081 for now assigning also to you. And also assigning to May as it would be cool if we look next milestone into this.

When I type in a pattern to look for an extension, I hear the number of extensions found and the message occurs N times.
The information regarding the number of extensions found is very useful and should be preserved if possible.

I agree, the only thing we don't need is the counter that a particular alert fired x times. This might be useful for some debugging scenario, but is nothing useful for actual users.

Thanks for feedback.
The issue was the following: we "fire alerts" in a way that we write content to a container with role = alert.
So what could happen in theory is the following

We fire an alert "Hello User"
User does some work
We want to fie an alert "Hello User" again, however since the content of the alert container would not change the new alert would not get read by the screen reader. And due to that we were adding "Occurred n times"  ending.

I have now changed this behavior and we would not add the "Occurred n times" ending and this might lead to some duplicated consectuive alerts not being read. But I think that is fine. So try it out from tomorrow's vscode insiders and let me know.
Still leaving this issue open since I am still not sure why we have an alert here. Once @sandy081 comments we can close this.

One way to make sure the alert always gets spoken when text changes is to set aria-atomic to true. Normally, if Chromium implements the alert role properly, this is implicitly set, in addition to aria-relevant being of value "additions text". So one thing to try is to set aria-atomic="true" on the same element that has role "alert", to see if that fixes the problem if the same text is pushed to the alert. Otherwise, first removing all text, then adding it again, might also trigger proper speaking. Just some ideas.

@MarcoZehe good ideas. However we arleady set aria-atomic on the element. Code pointer.
I just tried setting the textContent to an empty string before resetting it and it did not work (at least with VoiceOver). I also tried some timeouts and also did not help.
So I do understand that my change is a bit risky since two consecutive equal alerts will not be read and we will probably revert that but would still like to try it out.

so not sure how that works in other screenreaders? especially if it's true aria-atomic should make it being read always on change even if the text is the same...

@isidorn the alert here is to inform user that

there are no extensions found or
there is an error while fetching extensions

I cannot repro the alert happening N times this with the steps mentioned using Voice Over in Mac. Can you please help ti repro?

@sandy081 ok makes sense.
As for the N times you can no longer repro out of source due to my commit 9f709d170b06e991502153f281ec3c012add2e42
So either revert that commit or take insiders which does not have it.
Turn on VoiceOver, type in the extensions view slowly and you will notice that sooner or later he will say something like "46 extensions found (occured again)"
I always repro for the following case: type "a", wait a bit delete "a" - two messages with "occured again" happen

Hi,
Maybe this comment is meant to aim at something too specialized. I have never gotten JAWS or NVDA to speak the status line by using the screen reader keys meant for that purpose. If VS Code could re-display alerts by pressing a key stroke, would that be something that should happen or is it the job of the screen reader vendor or, is it the job or whatever of someone who might make an extension meant to enhance accessibility? And I do not know which alerts I would want repeated. I'm too new a user for that right now. I happened to go into Browse mode and realize that there is a lot of good information that it might be convenient to re-read without needing to leave the editor.

@jhomme thanks for providing more details. I think that as a feature makes sense, however for me it feels like it should be implemented in Screen Readers - so users can repeat what got announced.
Maybe you can start a discussion in our accessibility channel?

@isidorn Orca has a command called 'review live region announcement' that I use for this purpose. With this command I can hear the 9 latest ads. Perhaps the jaws and the nvda have something similar.

Hi,
I will check JAWS and NVDA for settings like these.
Jim
==========
Jim Homme
Skype: jim.homme
FreeChess: jhomme
Twitter: jimhomme
Facebook: jimhomme
From: José Vilmar Estácio de Souza<EMAIL_ADDRESS>Sent: Thursday, April 30, 2020 1:43 PM
To: microsoft/vscode<EMAIL_ADDRESS>Cc: Jim Homme<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [microsoft/vscode] Screen reader speaks the message "Occurs n times" Repeatedly When Searching For Extensions (#95693)
@isidorn https://github.com/isidorn  Orca has a command called 'review live region announcement' that I use for this purpose. With this command I can hear the 9 latest ads. Perhaps the jaws and the nvda have something similar.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/microsoft/vscode/issues/95693#issuecomment-622002027 , or unsubscribe https://github.com/notifications/unsubscribe-auth/ABRUZJJXYLA7QMIVSZVPE4TRPG2C5ANCNFSM4MMOJA6A .

Hi,
I used the JAWS key stroke, SHIFT+INSERT+F1 to see what I could find out in the area of the status bar. I did not see aria-live. I then went to this article about aria-live to try to understand how this is supposed to work. The article made me think that perhaps the status line would behave better if it used aria-live. Another thing I thought of as a convenience might be to create a landmark or use a footer tag around the whole area where VS Code displays status information.
Finally, I checked, and the JAWS and NVDA status line reading commands are meant to show the status line of an application, and I do not think this seems to work well in browsers.

@jhomme we actually do set the aria-live attribute on the container which we use to "broadcast" alerts.
We also use aria-live on the status container. This is not being read by NVDA due to this issue which they are looking into fixing https://github.com/nvaccess/nvda/issues/11064

This should be fixed now. Please let me know if you still see this with latest vscode insiders

Verified with NVDA and latest insiders. I suspect this was fixed a while ago because come to think of it its been a while since I've heard that when searching for extensions.

Great, thanks. Adding verified label.


[oneDNN] Add accidentally removed ops back to MklList for auto_mixed_precision

PR-62817 accidentally removed these ops, so adding them back.

Hi @ezhulenev Can you please review this PR ? Thank you!

Hi @ezhulenev Can you please review this PR ? Thank you!


Get file by CID from the IPFS network

Apologies if the question seems naive, but I didn't find any method like 'get'. I need to retrieve a file from the global IPFS network which is not on the local node.

Please see the Helia 101 example and also the @helia/unixfs module.


Header image submission for relayer registry

@burnsben to provide guidance on header image specs

thumbnail dimensions: 216x128px

Gonna re-open this until the PR is in to add the screenshot spec. I also think we should maybe increase the resolution a bit so that we have room to use it for other use cases? (maybe 2x or 3x)
What do you think @burnsben?

@tomhschmidt Good catch. We can always scale down. Let's go with 648x384px


lat and long anomalies

When running the next query:
Select markers.longitude,markers.latitude
From involved_markers_hebrew,markers
where markers.id = involved_markers_hebrew.accident_id and involved_markers_hebrew.road_segment_number= 80
and involved_markers_hebrew.road1=1
limit 100
all lat and long are pretty much the same except two that are off limit,
Write small alg that can find and correct anomalies like that

@benadikt the identifier of an accident is the three: id, provider_code and year
you won't find such anomalies when using the three as identifier

Also in both involved_markers_hebrew or makers_hebrew you will find longitude and latitude and road_segment_name or road_segment_number - no need to join them

Just to clarify - not all accidents in the same segments need to have the same lon lat - but you can check me on this one

not all accidents in the same segments need to have the same lon lat

Lets put them on the map one by one and see the anomalies

Also in both involved_markers_hebrew or makers_hebrew you will find longitude and latitude and road_segment_name or road_segment_number - no need to join them

Also taken but because of platform, and the query as been perform not in real time.
there are no significance to join two tables even if not necessity.
https://www.google.com/maps/place/29°52'50.0"N+35°02'41.6"E/@29.9504694,35.326378,9.5z/data=!4m5!3m4!1s0x0:0x0!8m2!3d29.8805566!4d35.0448942

@benadikt not sure what you mean by that.
If you have a specific example of an anomaly please share, if not I think we can close this issue for now.

@benadikt after a discussion wit Dror - Dror will continue exploring this issue - since it's important to validate the road segments we are counting on.
@benadikt Great work!
@inbalbeka @elashahar01 FYI


Must not chdir in rt-hook-kivy.py

The runtime-hook for PyInstaller changes directory to the directory where the executable lives. This generally is bad habit and leads to programs not behaving for the user as expected. Assume the user wants to pass a file from the current directory as an arguments:
/usr/bin/my-kivy-app ./file-argument.txt

Now as the runtime-hook changes the directory, the app tried to open /usr/bin/file-argument.txt. Adn if the application uses some file-requestor, it will start in /usr/bin, too, instead of the users current directory.
Even worse: Because the chdir happens in the runtime-hook (which is hidden magic for the app developer), the app developer has a really hard time debugging this kind of issues. (And maybe he comes back to us PyInstaller developers reporting a bug we are not responsible for :-(

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/12810639-must-not-chdir-in-rt-hook-kivy-py?utm_campaign=plugin&utm_content=tracker%2F42681&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F42681&utm_medium=issues&utm_source=github).

fixed by https://github.com/pyinstaller/pyinstaller/pull/1678


wireframe

What do you think @harunaltunhr for top- basic version, and for extra features- lower picture if time permits.
This is calculator app from my android phone

project switch


Only one file is uploaded successfully when selecting and uploading multiple files from different locations

Storage Explorer Version: 1.36.0-dev (101)
Build Number: 20240929.1
Branch: rel/1.36.0
Platform/OS: Windows11/Linux Ubuntu 22.04/MacOS Sequoia 15.0(Apple M1 Pro)
Architecture: arm64/x64/arm64
How Found: Exploratory testing
Regression From: Not a regression
Steps to Reproduce

Expand one storage account -> Blob Containers.
Create a blob container -> Click 'Upload -> Upload Files...'.
Select two files that come from different locations.
Check whether succeeds to upload the two files.

Expected Experience
Succeed to upload the two files.
Actual Experience

Succeed to select the two files.
Only one file is uploaded successfully.

Additional Context
Here is the record:

The upload files dialog will now display an error message if you try to upload files from different directories at the same time.

Verified this issue on the main build 20241206.2. Fixed.
The upload files dialog displays an error message when trying to upload files from different directories at the same time


Raising an error when an asset is not found

As per the Rails docs, if one is using sprockets-rails >= 3.2.0 it's possible to set config.assets.unknown_asset_fallback = false and raise an error when an asset is not found. Looking at the current implementation of asset_url we get a warning if propshaft is unable to resolve an asset URL. Would it be possible to have a similar configuration that would allow raising an error instead (especially on assets:precompile)?

It's worth noting that propshaft already does raise a MissingAssetError for assets that aren't resolved when the asset is referenced directly in an actionview helper method like javascript_include_tag, etc., as seen here:
https://github.com/rails/propshaft/blob/62476fa31b5981fdf4e026f18c5031755885e5ec/lib/propshaft/helper.rb#L3-L5
The code linked in @tiagobabo's post is specific to assets that are referenced in a url(...) value within a CSS file, which are potentially transformed by the CssAssetUrls processor.
Note, however, that some expressions inside url() should never be transformed (e.g. an inline data-URI or an fully qualified URL to an external domain). This explains why the current implementation can be useful.
I do see the value in adding such a configuration to throw an exception rather than just logging a warning if an asset inside url() can't be resolved, but it seems like it would have to be restricted to only apply for expressions inside url() that makes sense to be transformed (e.g. relative URLs).

You're right, this only happens for assets required inside url(...), thanks for clarifying that.

My original implementation was for a raise, but when I started migration my own app from sprockets to propshaft, I noticed that it was easier to let all errors to be logged and fix them in batches than doing the crash-fix-reload-crash-fix-reload cycle.
Is there a reason why raising would be better? I'm asking because between our two experiences we might figure out a third option that might be better than logging/raising

I’ll close this issue for now as it’s not a bug but a feature request, and from my experience logging provides a better workflow than raising. If anyone has further input, feel free to comment.


Microsoft Store version shows as "installed" but github version is uninstalled.

Microsoft PowerToys version
.67.1
Installation method
Microsoft Store
Running as admin
None
Area(s) with issue?
Installer
Steps to reproduce
Uninstall a version of powertoys that is not from the MS Store.
Go to MS Store Power Toys, and see if it is available to install, or showing as installed already.
I was just trying to switch from github version to MS Store version. But the store thinks it is still installed.
I can just use the version from github, it works.  I just can't seem to remove it enough in a way that MS Store sees it as gone.
not sure what is hanging around.
I don't remember the previous version from github that I uninstalled prior to trying ms store flavor.
✔ Expected Behavior
If powertoys is uninstalled on the system, I should be able to install it from MS Store.
❌ Actual Behavior
Store version shows as installed already.  So I can't install the store version.

Other Software
No response

The are literally the same installer.  Zero difference between the installers.  I wonder if this could be a store bug but not much from our side we can do.  It could be maybe a delay between the store app detecting the app is no longer installed.
Would suggest filing this on feedback hub as well but this is a pretty edge case I'd imagine.


fix: make burn and issuance timeframe consistent and legible

Think timeframes for burn/issuance should be consistent as generally that's what's expected in standard reporting. Also added a small disclaimer in the tooltip to ensure that folks know that it's based on last 6 months of data annualized as opposed to d7 for issuance and since burn for burn. I would use trailing 1yr data but don't see it in the /api/v2/fees/gauge-rates response.
If H1 is just considering H1 as opposed to last 6 months might be worthwhile to add support for trailing 1yr via the backend.

h1 is not 6 months, but 1 hour 😅. it's h1 instead of 1h because JS doesn't allow leading numbers in keys. if you flip them, m5, h1, d1, d7, d30 start making more sense 5m, 1h, 1d, 7d, 30d 😁.
you can see how gauge rate calculation is implemented here: https://github.com/ultrasoundmoney/eth-analysis-rs/blob/f4d4119346239b612c07644840ab505586293a0a/src/gauges.rs
we choose to determine burn based on all data we have, i.e. since eip-1559 was deployed because it's so unpredictable. for issuance we felt it is much more predictable, and taking the last 7 days of issuance as an indicator for the next 365 would be far closer to the truth than taking the last 365 days of issuance for example. calculating revenue / cost based on the last hour of data for both seems like it'd be much less accurate. burn because it may jump dramatically hour to hour, issuance i forget why exactly we picked it but < 30 day because recent issuance has been a better predictor for future issuance throughout Ethereum's history. Not sure why we didn't go smaller. Maybe epoch boundary issuance swings is the reason. So i wouldn't do 1 hour for both but if you have other suggestions for improvement they're welcome!

going to close this due to inactivity but happy to pick it back up any time. maybe we can discuss in an issue first.


katas aren't being shown in the browser

Can't see any kata, no matters what browser I use keep facing dark-gray rectangle instead of the kata

I Go to 'https://www.codewars.com/dashboard'
and all is shown is a dark-gray rectangle instead of the kata itself

f.lyp3
OS: Ubuntu 16.04 lts
Browser: chrome, opera, firefox

You should check the browser console for errors and mention them here (if there're any), and see whether turning off some browser extensions helps as they can affect the HTML.

You can hover over vertical bar on the left side and click Kata there. It takes you to the page where you can browse all available katas.

@f-lyp3 I deployed a possible fix, can you try again and let me know? If it's not fixed, can you check if there's any errors in console?

@kazk it's working again! seems like you fixed it, thank you


Storage migration feature missing in server 2019 GA

This feature is not present in the GA release.

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

ID: 5c3b0886-820a-0b3c-8382-8d65fc8c3b27
Version Independent ID: 3e310218-abc0-5249-bb16-7c21402e32c0
Content: Storage Migration Service overview
Content Source: WindowsServerDocs/storage/storage-migration-service/overview.md
Product: windows-server-threshold
GitHub Login: @JasonGerend
Microsoft Alias: jgerend

Hi. This happens when you are using the 1809 WAC versus an old insider build of Windows Server. The 1809 SMS WAC extension is version bound to the Windows Server 1809 (or later) build.
If you were to download an older preview SMS WAC extension or wait for Windows Server 2019 to GA, the issue will stop.
I will update our FAQ and requirements to reflect this until the server GAs. Thanks for the comment.

#assign:IngridAtMicrosoft


UI Proposal: Proposal/Discussion names should be clickable links

Improved user experience as users will instinctively click the title itself rather than the "View" link.

Duplicate of https://github.com/paideiadao/paideia-app/issues/71


Missing explanation of how to construct / inject FileSystem (and others) in runable code

Describe the bug
On first usage of this library, we have the README of the repo as guidance and nothing else.
this readme has example of how to use the abstractions in a class, but no example of how to provide the backing type.
To Reproduce

In a new project add only TestableIO.System.IO.Abstractions as the readme would suggest
Create a class like in the example in readme
Try providing the FileSystem type, it doesnt exist, google searchs seem to indicate it should

Expected behavior
Need add some reference in the README (or add wiki) to the TestableIO.System.IO.Abstractions.Wrappers nuget which adds the relevant classes to actually allow running code and not just tests.
Additional context
Hopefully I have not missed some obvious section in some readme or some other place that mentions this,
though I still think it should be added to README file

Oh, that's a good hint. We should actually suggest adding TestableIO.System.IO.Abstractions.Wrappers in the README.

@fgreinacher :
Shouldn't this issue be automatically closed because of #1014?
Or what is the process for closing issues here?

@fgreinacher :
Shouldn't this issue be automatically closed because of #1014?
Or what is the process for closing issues here?

@vbreuss I don't think GitHub supports the syntax you used in the PR: https://docs.github.com/en/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue#linking-a-pull-request-to-an-issue-using-a-keyword

Fixed wichtig #1014


Java instrumentation example

Add an example showing how to inject the Java SDK into a Docker image, run it with the right env vars to point at a local collector, and show the data collected in JSON format in a text file.

I believe this project is a better place for efforts like this: https://github.com/signalfx/tracing-examples.
As an aside, it's not clear to me what purpose these examples have in the collector project that wouldn't be better served as readme or docs improvements for their respective components.


upgrade nacos client version to 1.1.4

Describe what this PR does / why we need it
spring-cloud-alibaba-dependencies
Upgrade nacos client version to 1.1.4
Does this pull request fix one issue?
fix #1017
Describe how you did it
Describe how to verify it
Special notes for reviews

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 2 committers have signed the CLA.:x: yuhuangbin1:x: yuhuangbinYou have signed the CLA already but the status is still pending? Let us recheck it.


Add a JSHint task and some code to test it

A Gulp task to run JSHint on some code would be a good test.

So, it appears that I needed to:

Run the docker agent as root
Install Gulp globally first
Install the dependencies and then run Gulp.

Thankfully, this works.


fixed bug with toggle timeslot

@Arsamat This seems to just be removing some logic (and in doing so the confirm flag will always be true), is this the intended fix? Is there an issue that this is tied to that I could reference?

Yup! This is an issue, there is a logic change with sending emails, its no longer done on toggle, rather there is a separate button. That logic was making it so we couldn't toggle timeslots for applicants on/off

@vitorvicente Ah so is removing the logic the intended fix? If so, shouldn't this also be modified? (starting on line 185)
    let confirm = true;

    if (confirm) {
      // TODO actually send an email for this
      await this.props.firebase.generalSettings().set(settings);
      this.setState({ showToast: true, preSaveSettings: settings });
    } else {
      this.setState({ settings: preSaveSettings });
    }

If the swal never pops and confirm is always true we can remove confirm and simplify the if statement to just be the code in the true case. Then we can also remove preSaveSettings from line 160 as it won't be used in this case.


[PowerRename] Fix UI for Localized Strings

Summary of the Pull Request
Rearrange the checkboxes in PowerRename into two columns to make them properly accommodate localized strings.
Widen the Search for label to use all the space.

PR Checklist

[x] Applies to #7430
[x] CLA signed. If not, go over here and sign the CLA
[ ] Tests added/passed
[ ] Requires documentation to be updated
[ ] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Info on Pull Request
What does this include?
Validation Steps Performed
How does someone test & validate?

@arjunbalgovind
this will not fix all the cases, but at least it should mitigate the current problem.
Can you give it a try with the French translation after it's merged?
Worse case scenario, we use one checkbox per row.
Thanks.

I'm not 100% sure, but I'm afraid that the space that is used for the two labels - "search" and "replace" - will also be too small for languages that use more words or have longer words. Like French, for example. Making those wider would be a safe option. The input fields are scrollable, so that wild never be a problem.

@Jay-o-Way
you are right, we should enlarge it even more.

The screenshot provided in the issue also shows this. Maybe make it twice the width or something?


@auth groups clarification

** Which Category is your question related to? **
Aws Amplify graphql transform / aws appsynce
** What AWS Services are you utilizing? **
Aws appsync
** Provide additional details e.g. code snippets **
Would it be problematic to just put a read and write group for auth on every object that needs access control?

# Individual accessing data
type User @model
@auth(
    rules: [
      {allow: groups, groupsField: "readGroups", queries: [get, list]},
      {allow: groups, groupsField: "writeGroups" mutations: [create, update, delete]}
    ])
{
    id: ID!
    # Name of user
    name: String!
    # Email of user for login and contact
    email: String!
    # Date user was created
    createDate: AWSTimestamp!
    # Last time user logged in
    lastLogin: AWSTimestamp
    readGroups: [String]
    writeGroups: [String]
}

I don't see a problem with it however I think in your case you will want to pass explicit nulls to "mutation" and "query" to make this work as expected. E.G.
@auth(
    rules: [
      {allow: groups, groupsField: "readGroups", queries: [get, list], mutations: null},
      {allow: groups, groupsField: "writeGroups" mutations: [create, update, delete], queries: null}
    ])

Closing this issue. Please, feel free to re-open this issue if the problem still persists.


object reference not set to an instance of an object when trying to download 64bit

object reference not set to an instance of an object when trying to download 64bit Office365ProPlus - current branch
32 bit downloads fine
I seem to be having a lot of trouble deploying 64bit in general. Why 64bit? We currently use 64bit 2010 and would like to keep it the same.
let me know if anyone knows anything about this!

What script were you using when you received the error?

looks like the program has been updated since i last tried. The 64Bit files now download and i can create both msi and exe files. However if I select to embed the installation files I get the same "object reference not set to an instance of an object"'
I am unsure what you mean with which script i am using. I am using the program you can see in the screenshot. Thanks for your assistance.

Any update on this issue as I'm also getting the same "object reference not set to an instance of an object" if I select to embed office installation files.

We have posted a new version of the install toolkit which addressed this issue.  We will leave this issue open for the next 3 days.  If the issue persists please let us know and we will revisit the problem asap.  Thanks!


Add call_parameter_sets to the WhereFilter Protocol

Resolves #109

Description
Please see the linked issue.
Checklist

[x] I have read the contributing guide and understand what's expected of me
[x] I have signed the CLA
[x] This PR includes tests, or tests are not required/relevant for this PR
[x] I have run changie new to create a changelog entry

Rebasing off of main to keep the linear history and get the ruff fix


Unable to copy scripts into empty Scripts folder

System information:

Operating system (distribution) and version Win 11
DBeaver version 21.2.5 UE devel
Additional extensions

Connection specification:

Database name and version
Driver name
Do you use tunnels or proxies (SSH, SOCKS, etc)?

Describe the problem you're observing:
Steps to reproduce, if exist:

As a prerequisite, suppose we have a project dbeaver-dev with some script Script.sql in the Scripts folder.
Switch to the Projects tab in the navigator. Create a new project with the name let's say dbeaver-dev-copy.
Attempt to copy the Script.sql file into the new project. An exception will be thrown and nothing will be copied.

Include any warning/errors/backtraces from the logs
!ENTRY org.jkiss.dbeaver.model 4 0 2021-11-10 13:39:35.317
!MESSAGE Resource '/dbeaver-dev-copy/Scripts' does not exist.
!SUBENTRY 1 org.jkiss.dbeaver.model 4 0 2021-11-10 13:39:35.317
!MESSAGE Resource '/dbeaver-dev-copy/Scripts' does not exist.
!STACK 0
java.lang.Exception: Resource '/dbeaver-dev-copy/Scripts' does not exist.
	at org.eclipse.core.internal.resources.ResourceException.provideStackTrace(ResourceException.java:42)
	at org.eclipse.core.internal.resources.ResourceException.<init>(ResourceException.java:38)
	at org.eclipse.core.internal.resources.Resource.checkExists(Resource.java:330)
	at org.eclipse.core.internal.resources.Resource.checkAccessible(Resource.java:204)
	at org.eclipse.core.internal.resources.File.create(File.java:110)
	at org.eclipse.core.internal.resources.File.create(File.java:179)
	at org.jkiss.dbeaver.ui.navigator.actions.NavigatorHandlerObjectCreateCopy.copyResource(NavigatorHandlerObjectCreateCopy.java:162)
	at org.jkiss.dbeaver.ui.navigator.actions.NavigatorHandlerObjectCreateCopy$1.run(NavigatorHandlerObjectCreateCopy.java:124)
	at org.jkiss.dbeaver.runtime.RunnableContextDelegate.lambda$run$0(RunnableContextDelegate.java:39)
	at org.eclipse.jface.operation.ModalContext$ModalContextThread.run(ModalContext.java:122)

Verified


Post Gems cache remains running indefinitely, does not respect timeout-minutes

Steps to reproduce

Add deploy step to Ruby project that uses actions/cache:

      - name: Gem cache
        id: gem-cache
        timeout-minutes: 15
        uses<EMAIL_ADDRESS>        with:
          path: vendor/bundle
          key: ${{ runner.os }}-gem-${{ hashFiles('**/Gemfile.lock') }}
          restore_keys: |
            ${{ runner.os }}-gem-

Add deploy step that triggers GAE deploy, e.g. using<EMAIL_ADDRESS>
What happens

Post Gem cache step remains running indefinitely even after all other steps finish successfully, e.g. more than an hour:

timeout-minutes of the step is not respected

What should happen

No error should happen
If task hangs, timeout-minutes should be respected

UPDATE: issue occured over a background process following Gem cache step.
When the process got killed, Post Gem cache succeeded.

Which process did you killed?

GoogleCloudPlatform/cloudsql-proxy, but I'm sure any process left in the background with & could help to reproduce this.

@ain do you mind share a link to your workflow execution if the repository is public? We should handle & properly and not hang the workflow.

👋 hashFiles has been updated on the runner to correctly handle symlink loops (see actions/runner#234).
@ain do you still see this issue?


Update nf-iphlpapi-getinterfaceinfo.md

Spelling correction.

@BL1NDX3N0N : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Filter by array column

Hello,
Is it currently possible to filter results based on an array column?
I mean if a tags column for example contains [tag1, tag2, tag3], can I filter all results having the tag3 value into the array?
Thanks

That feature is not yet possible (unless you make a view that does it, and then access the view from the client side.)
What url syntax would you propose to do the array matching?

Thanks for your answer.
What about /collections?tags[]=eq.tag1&tags[]=eq.tag2? Not sure about the eq part.
We should then, receive the rows which have at least tag1 and tag2 into the tags column.
What do you think?

How about adding a new operator?
/collections?tags=contains.tag1&tags=contains.tag2

Or maybe
/collections?tags=eq_any.tag1&tags=eq_any.tag2

Because this is closer to the "tag1" = any(tags) sql
@begriffs Is there a way to make a view for an arbitrarily big query/data? You could split up the array into columns but this puts the limit on the number of entries. Or maybe sort the array and concatenate it into a string and use like to query. But this requires escaping.

Giving this more thought I now prefer the extended column syntax that @RobinBressan proposed.
I implemented a variation of this on my fork. I added syntax for all and for any with support for the existing operators so no other syntax changes are needed.
/entries?tags[any]=eq.Comedy -> where 'Comedy' = any(tags)
/entries?tags[all]=neq.Comedy -> where 'Comedy' != all(tags)
I haven't opened a PR yet since I want to clean up my code a bit and add tests for the new extensions. Please give me some feedback so I know weather to invest time into it.

My two cents are that it would be better to keep the convention column=operator.value.
In this case something like /entries?tags=contains.Comedy or /entries?tags=notcontains.Comedy

@diogob That would mean you cannot do something like /entries?ratings[all]=geq.4 or at least you would need separate operators.
We already have JSON columns which "break" the convention /stuff?json_col->a->>b=eq.2

Has there been anymore updates on this? Is there a way of querying for a value in an array column using postgrest right now?

No updates, we've been busy rewriting most of the code but this feature would probably be easy to implement. When the new version is ready, this will also be included

@edofic your work on the fork is pretty cool. Would you still be interested in leading our efforts to bring this issue to a conclusion?

I think the availability of <@ and @> operators this can be expressed like this
<EMAIL_ADDRESS>which generates something like
select ARRAY['tag1','tag2'] @> '{tag1}'::unknown

@begriffs sorry for the late reply - I somehow missed this. But yes, I can take this over.

@edofic is there any point to extendibg the syntax when the original question can be answered using features already present?

@ruslantalpa I was not aware this is already implemented because I didn't see it in the docs. Tried out your example and I'm quite happy. I opened #626 to update the docs. I think this issue can be closed now.


Replace strings on files by using string or regex patters.

Replace strings on files by using string or regex patterNs.

I do not understand. That is what this plugin actually does.

I think he's referring to the spelling mistake in the project's description.

@michaelhodgins yup, that is what it was meant. thanks for the note!


how to use TextSplitter for GPTSimpleVectorIndex

how can I use TextSplitter for GPTSimpleVectorIndex to split document by such “\n” or/and other?

CharacterTextSplitter has a default parameter separator="\n\n" , you can use another one
text_splitter = CharacterTextSplitter(chunk_size=1000, chunk_overlap=0, separator="\n\n")

thanks a lot!


Bug: new proposal requires refresh

Overview

After creating a new proposal, it doesn’t show up on the Proposals page unless a page refresh happens.

Steps to reproduce
n/a
Screenshots / Video
No response
Version
Dev (app.dev.fractalframework.xyz/)
Browser
Chrome (or Chromium based)
Operating System
None
web3 wallet
None

I'll be tackling this one. Looks like this is only an issue for Multisig DAOs. On a Usul DAO, proposals page seems to get updated automatically with the new proposal


Make fswatcher higher-level for static_handler.go and tlscfg

Which problem is this PR solving?
Redo first step for #3924 as discussed here and here

@yurishkuro I did PR1,2,3 as you said here (if I did only PR1 my PR would be full of errors, since I'm breaking an used API).
I followed your instruction, with a small addition: I added the onRemove functions:
WatchFiles(paths []string, onChange func(), onRemove func()) error
Because I see in some cases (i.e., static_handler.go), Remove is not considered a change to reload and handled differently from Create, Write, Rename.
There're 3 failing checks but I'm not sure they relate to my changes.
Please help review!

Close and redo first step in #4260


Putting any Redstone Arsenal tool into the reforging table causes a crash

When you try and put any Redstone Arsenal flux infused tool inside the reforging table, the game will crash to desktop. Note that the crash only occurs once the reforging options are displayed (or try to be displayed) i.e. when both the tool and the reforging material are placed inside the table.
Crash report: https://pastebin.com/HeKjBfvV

Can you reproduce it after grinding them? Likely a problem about enchantments on them, I guess.

When I was testing the tools had no enchantment on them, they were straight from the creative menu.

I don't believe java 18 is a supported platform.  You have to use java 17.

I just tried downloading and changing to java 17, and the exact same crash occured, so the java version disparity isn't the issue here. Here's the new crash log when using java 17: https://pastebin.com/W656THSY

Alright that's weird.  This only happens with RA items?  This crash shouldn't be unique to any particular item since it's a linkage error, though the source and target classloaders are the same.  However this code path being invoked means an exception was thrown but eventbus is exploding trying to log the error (which is... not really my fault).  I'll have to see if it's possible to debug the error somehow.

So far I've only seen this happen with RA items, though I haven't done a full test of every tool present in the pack I'm playing (as that would take quite a while).

I also have a try, and result is below.
1, only armors, axe, hammer, trident can be put into reforging table, others can't.
2, axe and hammer cause crash.

We encountered this in FTB skies too. During debugging in a minimal setup (RA, apotheosis and a charging mod) it "seems" to be caused by a recursive call of related "getEnchantmentLevel" functions (roughly RA -> minecraft -> forge -> apotheosis's getEnchantmentLevelSpecific -> RA), leading to stack overflow which is not the same as normal game's crash report. Not sure why but for debugging and normal gameplay the behaviors are the same (powered RA sword crash, powered RA chestplate no crash)

Do you have the stacktrace of the stack overflow?  The one here has event bus swallowing the exception

crash-2023-04-21_23.20.42-client.txt
For a quick overview this is one "cycle":
	at shadows.placebo.events.PlaceboEventFactory.getEnchantmentLevelSpecific(PlaceboEventFactory.java:31) ~[Placebo-1.19.2-7.1.7-dev.jar%23187!/:7.1.7] {re:mixin,re:classloading}
	at net.minecraftforge.common.extensions.IForgeItemStack.getEnchantmentLevel(IForgeItemStack.java:178) ~[forge-1.19.2-43.2.0_mapped_parchment_2022.10.09-1.19.2.jar%23179%23186!/:?] {re:mixin,xf:fml:placebo:placebo_get_ench_level_event_specific,xf:fml:placebo:placebo_get_ench_level_event,re:computing_frames,xf:fml:placebo:placebo_get_ench_level_event_specific,xf:fml:placebo:placebo_get_ench_level_event,re:classloading,xf:fml:placebo:placebo_get_ench_level_event_specific,xf:fml:placebo:placebo_get_ench_level_event,pl:mixin:APP:apotheosis.mixins.json:IForgeItemStackMixin,pl:mixin:A}
	at net.minecraft.world.item.enchantment.EnchantmentHelper.getItemEnchantmentLevel(EnchantmentHelper.java:65) ~[forge-1.19.2-43.2.0_mapped_parchment_2022.10.09-1.19.2.jar%23180!/:?] {re:mixin,re:classloading,pl:mixin:APP:apotheosis.mixins.json:EnchantmentHelperMixin,pl:mixin:A}
	at cofh.lib.util.Utils.getItemEnchantmentLevel(Utils.java:396) ~[cofh_core-1.19.2-<IP_ADDRESS>_mapped_parchment_2022.10.09-1.19.2.jar%23196!/:10.2.1] {re:classloading}
	at cofh.lib.api.item.IContainerItem.getMaxStored(IContainerItem.java:19) ~[cofh_core-1.19.2-<IP_ADDRESS>_mapped_parchment_2022.10.09-1.19.2.jar%23196!/:10.2.1] {re:classloading}
	at cofh.redstonearsenal.item.FluxSwordItem.getMaxEnergyStored(FluxSwordItem.java:182) ~[redstone_arsenal-1.19.2-<IP_ADDRESS>_mapped_parchment_2022.10.09-1.19.2.jar%23197!/:<IP_ADDRESS>] {re:classloading}
	at cofh.lib.api.item.IEnergyContainerItem.getEnergyStored(IEnergyContainerItem.java:44) ~[cofh_core-1.19.2-<IP_ADDRESS>_mapped_parchment_2022.10.09-1.19.2.jar%23196!/:10.2.1] {re:classloading}
	at cofh.redstonearsenal.item.IFluxItem.hasEnergy(IFluxItem.java:48) ~[redstone_arsenal-1.19.2-<IP_ADDRESS>_mapped_parchment_2022.10.09-1.19.2.jar%23197!/:<IP_ADDRESS>] {re:classloading}
	at cofh.redstonearsenal.item.IFluxItem.hasEnergy(IFluxItem.java:53) ~[redstone_arsenal-1.19.2-<IP_ADDRESS>_mapped_parchment_2022.10.09-1.19.2.jar%23197!/:<IP_ADDRESS>] {re:classloading}
	at cofh.redstonearsenal.item.FluxSwordItem.canPerformAction(FluxSwordItem.java:82) ~[redstone_arsenal-1.19.2-<IP_ADDRESS>_mapped_parchment_2022.10.09-1.19.2.jar%23197!/:<IP_ADDRESS>] {re:classloading}
	at net.minecraftforge.common.extensions.IForgeItemStack.canPerformAction(IForgeItemStack.java:117) ~[forge-1.19.2-43.2.0_mapped_parchment_2022.10.09-1.19.2.jar%23179%23186!/:?] {re:mixin,xf:fml:placebo:placebo_get_ench_level_event_specific,xf:fml:placebo:placebo_get_ench_level_event,re:computing_frames,xf:fml:placebo:placebo_get_ench_level_event_specific,xf:fml:placebo:placebo_get_ench_level_event,re:classloading,xf:fml:placebo:placebo_get_ench_level_event_specific,xf:fml:placebo:placebo_get_ench_level_event,pl:mixin:APP:apotheosis.mixins.json:IForgeItemStackMixin,pl:mixin:A}
	at shadows.apotheosis.adventure.loot.LootCategory.lambda$static$4(LootCategory.java:49) ~[%23185!/:?] {re:classloading}
	at shadows.apotheosis.adventure.loot.LootCategory.isValid(LootCategory.java:92) ~[%23185!/:?] {re:classloading}
	at shadows.apotheosis.adventure.loot.LootCategory.forItem(LootCategory.java:181) ~[%23185!/:?] {re:classloading}
	at shadows.apotheosis.adventure.affix.AttributeAffix.canApplyTo(AttributeAffix.java:88) ~[%23185!/:?] {re:classloading}
	at shadows.apotheosis.adventure.affix.AffixHelper.getAffixes(AffixHelper.java:77) ~[%23185!/:?] {re:mixin,re:classloading}
	at shadows.apotheosis.adventure.affix.AffixHelper.streamAffixes(AffixHelper.java:86) ~[%23185!/:?] {re:mixin,re:classloading}
	at shadows.apotheosis.adventure.AdventureEvents.enchLevels(AdventureEvents.java:412) ~[%23185!/:?] {re:classloading}
	at shadows.apotheosis.adventure.__AdventureEvents_enchLevels_GetEnchantmentLevelEvent.invoke(.dynamic) ~[%23185!/:?] {re:classloading,pl:eventbus:B}
	at net.minecraftforge.eventbus.ASMEventHandler.invoke(ASMEventHandler.java:73) ~[eventbus-6.0.3.jar%23102!/:?] {}
	at net.minecraftforge.eventbus.EventBus.post(EventBus.java:315) ~[eventbus-6.0.3.jar%23102!/:?] {}
	at net.minecraftforge.eventbus.EventBus.post(EventBus.java:296) ~[eventbus-6.0.3.jar%23102!/:?] {}
	at shadows.placebo.events.PlaceboEventFactory.getEnchantmentLevelSpecific(PlaceboEventFactory.java:31) ~[Placebo-1.19.2-7.1.7-dev.jar%23187!/:7.1.7] {re:mixin,re:classloading}

Should be resolved in 6.2.0


Add columns to the sss_table (food)

The sss database was missing a column for food costs.

Codecov Report

Merging #41 (6b51776) into main (fde257c) will increase coverage by 0.07%.
The diff coverage is 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main      #41      +/-   ##
==========================================
+ Coverage   64.36%   64.43%   +0.07%
==========================================
  Files          10       10
  Lines         477      478       +1
==========================================
+ Hits          307      308       +1
  Misses        170      170

Impacted Files
Coverage Δ

sss/sss_table.py
95.85% <100.00%> (+0.02%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


⚠ PassThePopcorn has degraded performance

In 7fa0059, PassThePopcorn ($PTP) experienced degraded performance:

HTTP code: 502
Response time: 1910 ms

Resolved: PassThePopcorn performance has improved in 240c04e after 6 minutes.