Settings improvement: Notify plugins of settings change; mark settings as "requiring read-all to take effect" in settings UI

Related to #121: Today, plugins cannot detect when settings are changed. Furthermore, some settings, may require a read-all to take effect.
Changes requested:

Add an "OnSettingsChanged" event which provides plugins with an indication that settings have changed and, ideally, provide a before/after values so the plugin can determine what has changed. This would help inform whether a plugin requires a read-all. Ideally, this fires when the settings view for the plugin is closed, not on every individual value change (ie. on every single value change like in a number scroller or something) in order to suppress notifications for intermediate/non-committed values, thereby reducing false-positives.
Add a setting annotation which plugin authors can use to mark a setting as requiring a read-all if changed. When present, core will render the setting with a visual style which indicates that changing the setting will not take immediate effect. It is a visual indication ONLY. When the plugin receives the new "OnSettingsChanged" event, proposed above, and detects if the setting has in fact changed (because it may have been changed, then changed back), it can set the "RequiresReadAll" property, proposed in #121, in order to request the read-all.

For request 1: This can be accomplished with accessors in your settings class.
e.g.:
class PluginSettings
{
  [SettingDisplayName("Cool Setting")]
  public bool CoolSetting
  {
    get => _cool;
    set
    {
      SomethingINeedToDoWhenThisIsChanged();
      _cool = value;
    }
  }

  [SettingIgnore]
  private bool _cool;
}

A more formal way to be notified of setting value changes would be to implement INotifyPropertyChanged and the interested parties can subscribe to the PropertyChanged event...


Exceljs don't work after build in Angular 7

I have an application built in Angular and it uses exceljs to export some data in .xlsx.
For some reason I do not know, when I generate a project build, exceljs stops working. I found this https://github.com/exceljs/exceljs/issues/825 that refers to a similar problem.
Is there any way to work around the problem until the version that includes the changes is released?
Best regards

In my case, exporting with ExcelJS worked in development but not in production (I think there are some issues around Buffer).
However, you can use this little trick to make it work: https://github.com/exceljs/exceljs/issues/810#issuecomment-496447076

@charllie
It Worked!!! thanks for the tip!

Thanks.. work


Un-retire screenpy-quickstart!

It would be nifty to bring this script back! Seeing this LinkedIn article which mentions that script, i realized folks who are trying to get into this pattern probably really liked having that handy script.
We could make it pretty slick. It could check what extensions are currently installed (screenpy_selenium, screenpy_requests, etc.) and try to access a private file within them that tells screenpy what files to create in the quickstart script. Or each plugin could have its own quickstart script, but even screenpy-selenium-quickstart is getting kinda wordy...
Also make sure to be more explicit about how the file names and structure is arbitrary and can be changed to suit the user's needs whenever they want. This is more for beginners than users making their third or fourth suite.

After coming back to this months later, i think i have an idea using pkg_resources:
>>> import pkg_resources
>>> [pkg.project_name for pkg in pkg_resources.working_set]
[..., 'screenpy', 'screenpy-selenium', 'screenpy-adapter-allure', 'screenpy-pyotp', ...]

For each of the screenpy extensions found, the quickstart script could call extension.quickstart or something with the filepath and compose the end result with each package! I think that could work!

Phew, i found something! importlib.metadata has a packages_distributions convenience method that pretty much negates the need for the stuff i was annoyed with in the previous comment. We can do this instead!
import importlib
import importlib.metadata

extension_names = [
    package
    for package in importlib.metadata.packages_distributions().keys()
    if package.startswith("screenpy_")
]

for extension_name in extension_names:
    extension = importlib.import_module(extension_name)

This raises an interesting question about the style of quickstart.   Does it use pytest or unittest? or is it agnostic?


fix: sync much faster

perf(internal/blocksync): avoid double-calling types.BlockFromProto (#2016)
perf(internal/blocksync): do not ValidateBlock twice (#2026)
add mergify
ci: fix failing golangci-lint action (#1833)
vulncheck
new batch of fixes
add concurrency to removetimedoutpeers

PR checklist

[ ] Tests written/updated
[ ] Changelog entry added in .changelog (we use unclog to manage our changelog)
[ ] Updated relevant documentation (docs/ or spec/) and code comments

Closing due to improvement completed upstream, and backporting to our fork. See https://github.com/osmosis-labs/cometbft/pull/16


Refactored modal dialogs

New tenant added dialog
Exit app dialog
Subdomain alredy exist dialog turned into error text
No tenant found dialog

@LucasBrazi06 I will do another PR for the remaining dialogs


[Metrics] Collect FUSE operation counts

To better understand how FUSE operations drive performance we would like to collect and report the fuse operations as a prometheus metric.  Given that we want to make sure this is anonymized, we do not want to record file names, but just counts.
We could rely on the existing commonmetrics package and include metrics for the operations via the IncOperationCount function.  This is probably the easiest solution, and one that we know will work as Readdir uses commonmetrics to record latency info(see example below).

As an existing example in the code of Readdir
func (n *node) readdir() ([]fuse.DirEntry, syscall.Errno) {
	// Measure how long node_readdir operation takes (in microseconds).
	start := time.Now() // set start time
	defer commonmetrics.MeasureLatencyInMicroseconds(commonmetrics.NodeReaddir, n.fs.layerDigest, start)

	if n.entsCached {
		return n.ents, 0
	}
...

How exactly do we extract the useful information from this? If it's a counter, then wouldn't the counts be dependent on how long the container has been running? Are we expecting that we'd be able to look at a single container's numbers or is this aggregated over containers/fleet?

We will be able to see at a task level the number of operations.  If we compare this anonymized data (perhaps scaling for time) we can understand whether Task X triggers more costly fuse operations then Task Y.  But by no means is this perfect.

Metrics on a running container can be checked over time, counts per hour or day or before and after some notable workload. Also these would be useful for tasks that launch a container, handle a workload, then stop the container, which I believe is similar to the metrics we have previously collected in the "fuse penalty" research.

Updated description to specify this needs to be done at the task aggregation

close as complete by #354. @sbuckfelder please feel free to reopen if it's not fully completed by @354


Fix indentation rules for multiple classes and media queries

Now the rules demand this type of indentation for multiple classes & media queries:
.class-1,
    .class-2,
    .class-3 {
    @media only screen and (min-width: @screen-lg) {
        border: 1px solid #fff;
    }
}

And:
@media only screen and (min-width: @screen-lg) {
    .class-1,
        .class-2,
        .class-3 {
        border: 1px solid #fff;
    }
}

Please fix the rules so linter accepts this indentation:
.class-1,
.class-2,
.class-3 {
    @media only screen and (min-width: @screen-lg) {
        border: 1px solid #fff;
    }
}

And:
@media only screen and (min-width: @screen-lg) {
    .class-1,
    .class-2,
    .class-3 {
        border: 1px solid #fff;
    }
}

This is probably caused by bug in Stylelint: https://github.com/stylelint/stylelint/issues/2791
Also happens with this code:
.class1 input[type="text"],
.class2 input[type="email"] {
    font-size: inherit;
}

Should be fixed in https://github.com/shellscape/postcss-less/issues/96.

Fixed with #17


unbound variable error in in-list tests

Description
in-list unit test fails on macos with unbound variable error probably due to empty array.
 not ok 131 fail without error message for empty list
# (from function `assert_output' in file test/test_helper/bats-assert/src/assert_output.bash, line 194,
#  in test file test/in-list.bats, line 33)
#   `assert_output ""' failed
#
# -- output differs --
# expected :
# actual   : /Users/runner/work/biapy-bashlings/biapy-bashlings/src/in-list.bash: line 51: list[*]: unbound variable
# --

Fixed by following recommendations from Expanding Bash arrays safely with set -u.


How to pass alpha to shader in PIXI.js ?

I have a little question about  Alpha in shader:
I found there two ways in PIXI.js:
first :
pass  uniform float uAlpha , then use gl_FragColor = texture2D(uSampler, vTextureCoord) * vAlpha  in Fragment.
second:
compute color in JS via  hex2rgb
pass  attribute vec4 aColor, then use gl_FragColor = texture2D(uSampler, vTextureCoord) * vColor;  in Fragment.
Which one is correct or better ?
(The first one looks easy & fast )

Depends on what you want to do. If you want it to work across multiple objects in a batch, that have different alpha/color values then you need to pass it in the vertex array (as an attribute). This is why we upload it as an attribute in the vertex array for sprite's shader.
If it is only for one thing, and you only want to pass one number; the first is fine (and is easier to write).

Thanks very much .

gl_FragColor = texture2D(uSampler, vTextureCoord) * vColor;

make sure vColor is pre-multiplied alpha (ra, ga, b*a, a). Textures in pixi are all pre-multiplied

@ivanpopelyshev  , sorry,  I can't understand your words ... tears ...
Could you explain it in more details , please .
Or give me some key words for googling .
Thanks.

@finscn If your alpha is 0.5, all components in vColor should be multiplied by 0.5 too. Either do it in shader, either in js


Add - Size preview iframes to fit contents

Hi, I would like to add this feature that addressed in #205 that supports size preview iframes to fit contents. Let me know about any changes. Thanks a lot!
Change description:
For the host page settings. I use the iframe-resizer-react's IframeResizer component generates iframe and give the prop src by the children html content's url.
For the iframe page settings, because need to include the (iframeResizer.contentWindow.min.js) from iframe-resizer as documented, I use DOMParser to add script tag with the cdnjs version iframeResizer.contentWindow.min.js to achieve the requirements.

[x] upload test screenshots: https://github.com/tarasglek/chatcraft.org/assets/133393905/49bb9503-69dd-4b59-a24a-249ecb4e19b7

@humphd I've made some changes:

[x] use local js file rather than cdn
[x] rename variable to html
[x] remove height capping
[ ] checkOrigin={false}

This is needed to keep even not using cdn js. After checking I guess the reason is our iframe' src will starts with blob:http://localhost, while the <script> src will starts withhttp://localhost
(screenshots: https://github.com/tarasglek/chatcraft.org/assets/133393905/4a8e7dba-37d9-4f7b-a2c4-8959b293d642)

[x] use recommended width setup

Let me know if there are any other changes needed. Thanks a lot!

Really nice work @mingming-ma, great job!  Can't wait to use this.  I've wanted it for a long time.

Live and working on https://chatcraft.org!:

@mingming-ma if you'd like to fix another bug, we should disable updates during loading, because it flickers so much and jumps the page.

@humphd  Thank you so much for your encouragement! I will definitely try to fix that!

@mingming-ma congrats on getting your first contribution in, thank you!

@tarasglek Thank you! More to come!


docs: kmsKeyID doesnt need arn prefix in NodeClass resource

Fixes #N/A
Description
How was this change tested?
Does this change impact docs?

[x] Yes, PR includes docs updates
[ ] Yes, issue opened: #
[x] No

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

@Nadeshan do you have any updates?

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.02%) to 82.547%

Totals

Change from base Build<PHONE_NUMBER>:
0.02%

Covered Lines:
4952

Relevant Lines:
5999

💛  - Coveralls


Pouch won't open

The game is acting as if I have the cursed medal equipped. I'm 100% positive I don't have it, but I have acquired the item from the cursed medal chest, which was bombs, so that might have something to do with it. Medals and items like extra bomb bag work normally, I just can't open the pouch and use bottles or shields. I'm at a fine level of skill so I can beat the game without pouch access, but all of the dungeons are required and I can't get into fire sanctuary or ancient cistern. I won't be able to beat the seed without a bottle, is there a fix to this?

Did you start with pouch? It might be randomized, then you have to find it first before you can use it.
You can also talk to peatrice in the bazaar to make sure you don't have cursed medal

I didn't start with pouch, but I have obtained it, and I can put stuff in it with item check. I don't have the cursed medal in my pouch, I actually haven't gotten it at all yet. The pouch works like normal with medals, like the life medal still gives me an extra heart container, it just doesn't open by holding -  like it would normally. An additional note is that when I go to the gear menu and try to look at my pouch, it doesn't do anything when I push -, and that would normally work even with cursed medal, so the game is sort of acting like I don't have the pouch, but I am sure I have it because I remember getting it and item check works. Thanks for responding and helping me with the issue!

That sounds like the storyflag for the pouch is not set, which is very weird. But that explains why you can't open it and why you can't see it in the inventory menu. Can you post your spoiler log here and tell me which pouch you got? That way I can hopefully reproduce the bug and work on fixing it

Also, did anything weird happen when you got the pouch? I assume you weren't able to open it right after obtaining?


Building on a Mac - Consideration

I've tried to just do straight up dotnet build on a mac... obviously that throws a single error of 461 not available - to be expected.
I've just changed the csproj slightly ( based on this blog )
<TargetFrameworks>netstandard2.0</TargetFrameworks>
<TargetFrameworks Condition="'$(OS)' != 'Unix'">net46;netstandard2.0</TargetFrameworks>

I run this with dotnet build and it works without any errors. This becomes more prevalent when you run dotnet pack (it won't work unless the above)
It might not be a necessary - just thought I would share.

@no1melman There are a few ways to do this. This PR adds support for compiling on Mac/Linux and is generally the preferred approach.

Sweet


Implement float support for String.format

The actual implementation is entirely based on TDecimalFormat apart from padding, which is taken from formatDecimalInt.
This was not tested enough to ensure it works in all possible cases, but it seems correct.

predictDecimalSize is admittedly ugly, but since the width includes other characters and (to my knowledge) DecimalFormat provides no API that accepts a width, it seems needed.
Resetting the precision is also not great, but without that, the precision is either wrong or chars get trimmed by formatGivenString. If you want, I could append manually, though.

predictDecimalSize is admittedly ugly, but since the width includes other characters and (to my knowledge) DecimalFormat provides no API that accepts a width, it seems needed.

Yes, that's why I think either TDecimalFormat could be extended compared to original DecimalFormat, or (which I believe a better solution) TDecimalFormat could be just a thin wrapper around internally implemented formatting library, as well as TFormatter. This also would solve the problem with locale-specific plus sign.

I'm ready to merge this PR, however it produces another error in WASI BE. This is not issue of this particular PR, but another issue in WASI BE, that did not uncover it yet. I'll merge this PR as soon as I fix issue in WASI BE.


Unexpected behavior for Sequence[int] in Json Schema

Initial Checks

[X] I confirm that I'm using Pydantic V2 installed directly from the main branch, or equivalent

Description

[X] Using Sequence[int] for json is inconsistent.
[ ] BaseModel.validate_json should raise if BaseModel.model_json_schema raises. See below

Generating json schema via BaseModel.model_json_schema() raises PydanticInvalidForJsonSchema error which seems as a correct behavior.
However, calling BaseModel.validate_json(...) results in an attempt to validate passed json value while the validation schema is invalid.
Example Code
>>> from pydantic import BaseModel
>>> from typing import Sequence
>>> class Model(BaseModel):
...     int_sequence: Sequence[int]
...     int_list: list[int]
...
>>> Model.model_json_schema()
pydantic.errors.PydanticInvalidForJsonSchema: Cannot generate a JsonSchema for core_schema.IsInstanceSchema (typing.Sequence)

>>> Model.model_validate_json('{"int_sequence": [1, 2, 3], "int_list": [1, 2, 3]}')
pydantic_core._pydantic_core.ValidationError: 1 validation error for Model
int_sequence
  Input should be an instance of Sequence [type=is_instance_of, input_value=[1, 2, 3], input_type=list]

Python, Pydantic & OS Version
pydantic version: 2.0a3
        pydantic-core version: 0.25.0 release build profile
               python version: 3.10.11 (main, Apr  5 2023, 00:00:00) [GCC 13.0.1 20230404 (Red Hat 13.0.1-0)]
                     platform: Linux-6.2.12-300.fc38.x86_64-x86_64-with-glibc2.37
     optional deps. installed: ['devtools', 'email-validator', 'typing-extensions']

Reopening as one test in #5598 is still failing

@lig I'm not sure that test makes sense — see this comment for more detail.
I think this might be coming from a misconception — the generated JSON Schema has nothing to do with actual validation, it is just a generated thing for use by web frameworks, etc.
So it's generally always possible that you have an invalid JSON schema and yet can successfully validate things, or vice versa, that you have a valid JSON schema but cannot actually validate anything. (Validation is done by the pydantic core schema.)
At any rate, I think we made Sequence[int] have a valid json schema (the same as list[int]), which also means that test is maybe no longer checking something meaningful.
Happy to discuss more if I'm misinterpreting something.


Support for new 1D SWG in case of full SIMD unfolding

Adds support for https://github.com/Xilinx/finn-hlslib/pull/61 by automatically selecting the optimized 1D ConvolutionInputGenerator variant.
Already covered by existing test_convert_to_hls_1d_conv_layer, but I had to add support for multiple pixel ("mmv") unpacking in apintstream2npy to make cppsim work.
Depends on: https://github.com/Xilinx/finn-hlslib/pull/61 (commit pointer will require update)

hlslib dependency also merged and commit hash updated in Dockerfile. Looks good, thank you @fpjentzsch !


Soc descriptor harvesting

Fix https://github.com/tenstorrent/tt-umd/issues/102, part of a general effort around soc descriptor https://github.com/tenstorrent/tt-umd/issues/100
Goal of the PR is to move harvesting logic to soc descriptor, with clear interface for coordinate translation
TODO

[ ] Derive logical, virtual and physical coordinates from tt_xy_pair
[ ] Make soc descriptor tests arch independent as much as possible
[ ] Cleanup soc desc code for harvesting on y or x based on the arch
[ ] Revert commented tests
[ ] Move open UMD doc about harvesting to UMD repo

is the idea for tt_xy_pair to have an additional field to denote coordinate type (logical, virtual, physical, etc?)

is the idea for tt_xy_pair to have an additional field to denote coordinate type (logical, virtual, physical, etc?)

I am probably going to derive each type of coordinates from tt_xy_pair, so instead of
using tt_logical_coords = tt::umd::tt_xy_pair;

we are going to have derive tt_logical_coords from tt_xy_pair, so you still have only x and y as fields, but you can use coordinates as types, you will know which coordinates you are using just based on the type of the variable. Also, we want to be sure to not use the API in a way it is not intended, we can provide API for each type of coordinates. Does that make more sense?

is the idea for tt_xy_pair to have an additional field to denote coordinate type (logical, virtual, physical, etc?)

I am probably going to derive each type of coordinates from tt_xy_pair, so instead of
using tt_logical_coords = tt::umd::tt_xy_pair;

we are going to have derive tt_logical_coords from tt_xy_pair, so you still have only x and y as fields, but you can use coordinates as types, you will know which coordinates you are using just based on the type of the variable. Also, we want to be sure to not use the API in a way it is not intended, we can provide API for each type of coordinates. Does that make more sense?

yes that makes sense, thanks!

I know this PR is becoming a bit hard to review with a lot of code, so I will separate TODOs left into first class that I will solve in this PR and then merge it, and into second class tasks that I will open issues for and solve it all as separate PRs. I tried thinking about making this PR smaller as well, but I think this is a good base for harvesting so we can build on top of it. Uplifting this is also going to be seamless since it doesn't change or removes the API, it just adds the functions, so I can make the API function for tt-metal in the following commits.
@broskoTT @joelsmithTT let me know if this is ok for you


Added ManualAuthCode, ManualRefNum and PaymentMode fields in the OrderPaymentPreferenceAndType

TODO: Add the details here

closing this PR, these changes are already in develop.


Merge 3-6 updates to live

OPS Build status updates of commit 09bfa40:
:clock10: Preparing: average preparing time is 3 min(s) 18 sec(s)

OPS Build status updates of commit 09bfa40:
:clock10: Incremental building: average incremental building time is 53 min(s) 52 sec(s)

OPS Build status updates of commit 09bfa40:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Language/Reference/User-Interface-Help/invalid-attribute-in-sub-function-or-property.md
:white_check_mark:Succeeded
View

api/Access.AccessObject.IsLoaded.md
:white_check_mark:Succeeded
View

api/Access.AllForms.md
:white_check_mark:Succeeded
View

api/Access.Application.SysCmd.md
:white_check_mark:Succeeded
View

api/Access.Attachment.Click.md
:white_check_mark:Succeeded
View

api/Access.Attachment.DblClick.md
:white_check_mark:Succeeded
View

api/Access.BoundObjectFrame.Action.md
:white_check_mark:Succeeded
View

api/Access.BoundObjectFrame.Click.md
:white_check_mark:Succeeded
View

api/Access.BoundObjectFrame.DblClick.md
:white_check_mark:Succeeded
View

api/Access.CheckBox.Click.md
:white_check_mark:Succeeded
View

api/Access.CheckBox.DblClick.md
:white_check_mark:Succeeded
View

api/Access.ComboBox.Click.md
:white_check_mark:Succeeded
View

api/Access.ComboBox.DblClick.md
:white_check_mark:Succeeded
View

api/Access.CommandButton.Click.md
:white_check_mark:Succeeded
View

api/Access.CommandButton.DblClick.md
:white_check_mark:Succeeded
View

api/Access.Control.ObjectVerbs.md
:white_check_mark:Succeeded
View

api/Access.CustomControl.OLEClass.md
:white_check_mark:Succeeded
View

api/Access.CustomControl.ObjectVerbs.md
:white_check_mark:Succeeded
View

api/Access.DoCmd.AddMenu.md
:white_check_mark:Succeeded
View

api/Access.DoCmd.ApplyFilter.md
:white_check_mark:Succeeded
View

api/Access.DoCmd.Beep.md
:white_check_mark:Succeeded
View

api/Access.DoCmd.BrowseTo.md
:white_check_mark:Succeeded
View

api/Access.DoCmd.Close.md
:white_check_mark:Succeeded
View

api/Access.DoCmd.CopyObject.md
:white_check_mark:Succeeded
View

api/Access.DoCmd.DeleteObject.md
:white_check_mark:Succeeded
View

api/Access.DoCmd.DoMenuItem.md
:white_check_mark:Succeeded
View

api/Access.DoCmd.Echo.md
:white_check_mark:Succeeded
View

api/Access.DoCmd.FindNext.md
:white_check_mark:Succeeded
View

api/Access.DoCmd.FindRecord.md
:white_check_mark:Succeeded
View

api/Access.DoCmd.GoToControl.md
:white_check_mark:Succeeded
View

api/Access.DoCmd.GoToPage.md
:white_check_mark:Succeeded
View

api/Access.DoCmd.GoToRecord.md
:white_check_mark:Succeeded
View

api/Access.DoCmd.Hourglass.md
:white_check_mark:Succeeded
View

api/Access.DoCmd.LockNavigationPane.md
:white_check_mark:Succeeded
View

api/Access.DoCmd.Maximize.md
:white_check_mark:Succeeded
View

api/Access.DoCmd.Minimize.md
:white_check_mark:Succeeded
View

api/Access.DoCmd.MoveSize.md
:white_check_mark:Succeeded
View

api/Access.DoCmd.NavigateTo.md
:white_check_mark:Succeeded
View

api/Access.DoCmd.OpenDataAccessPage.md
:white_check_mark:Succeeded
View

api/Access.DoCmd.OpenDiagram.md
:white_check_mark:Succeeded
View

api/Access.DoCmd.OpenForm.md
:white_check_mark:Succeeded
View

api/Access.DoCmd.OpenFunction.md
:white_check_mark:Succeeded
View

api/Access.DoCmd.OpenModule.md
:white_check_mark:Succeeded
View

api/Access.DoCmd.OpenQuery.md
:white_check_mark:Succeeded
View

api/Access.DoCmd.OpenReport.md
:white_check_mark:Succeeded
View

api/Access.DoCmd.OpenStoredProcedure.md
:white_check_mark:Succeeded
View

api/Access.DoCmd.OpenTable.md
:white_check_mark:Succeeded
View

api/Access.DoCmd.OpenView.md
:white_check_mark:Succeeded
View

api/Access.DoCmd.OutputTo.md
:white_check_mark:Succeeded
View

api/Access.DoCmd.PrintOut.md
:white_check_mark:Succeeded
View

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


Identifiers validation failed for dataset Inventario herpetofaunístico del valle semiárido de Tehuacán-Cuicatlán (continuación)

Identifier validation failed for the dataset Inventario herpetofaunístico del valle semiárido de Tehuacán-Cuicatlán (continuación):

Crawler attempt: 269
Publishing organization: Comisión nacional para el conocimiento y uso de la biodiversidad
Publishing country: MEXICO
Installation: IPTCONABIO
Number of occurrences indexed for this dataset at the time of the issue: 487
Cause: GBIF IDs hit maximum allowed - 50%, duplicates - 86%, total records count 487, absent records count 417
Pipelines execution steps: http://api.gbif.org/v1/pipelines/history/execution/3073470/step

Publisher email
Hello,
I am contacting you from the GBIF Secretariat about a dataset published by the Inventario herpetofaunístico del valle semiárido de Tehuacán-Cuicatlán (continuación) : https://doi.org/10.15468/mb678i.
We noticed that the occurrenceIDs were changed. We have temporarily paused the ingestions of this dataset.
As you might already know, when an occurrence record has a new occurrenceID for a given dataset, our system considers it to be a new occurrence.
This means that it will be given a new gbifid and a new occurrence URL (like this one: https://www.gbif.org/occurrence/1252968762) and the old gbifid and URL will be deprecated.
In this case, this means that the occurrence URLs would be deprecated when ingesting the newest versions of these datasets.
We would like to check with you if those changes were intentional. Do you know if this is the case?
Please let us know, thanks! We are happy to resume the dataset ingestion.
Note that some users rely on those occurrence URLs and gbifids (like https://bionomia.net for example). In an attempt to improve
the stability of the occurrence URLs and gbifids, we have implemented a warning system to detect these type of changes in datasets (see this news item).
If the data publisher can provide us with a list of old and new occurrenceIDs per record, we can avoid the identifier and URL changes. Could that be an option?
Please let us know if you have any question. Thanks!
All the best,

You can skip/fix identifier validation using the registry UI.

Identifier validation still  failing for the dataset Inventario herpetofaunístico del valle semiárido de Tehuacán-Cuicatlán (continuación):

Crawler attempt: 272
Publishing organization: Comisión nacional para el conocimiento y uso de la biodiversidad
Publishing country: MEXICO
Installation: IPTCONABIO
Number of occurrences indexed for this dataset at the time of the issue: 487
Cause: GBIF IDs hit maximum allowed - 50%, duplicates - 100%, total records count 487, absent records count 487
Pipelines execution steps: http://api.gbif.org/v1/pipelines/history/execution/3094377/step

Publisher email
Hello,
I am contacting you from the GBIF Secretariat about a dataset published by the Inventario herpetofaunístico del valle semiárido de Tehuacán-Cuicatlán (continuación) : https://doi.org/10.15468/mb678i.
We noticed that the occurrenceIDs were changed. We have temporarily paused the ingestions of this dataset.
As you might already know, when an occurrence record has a new occurrenceID for a given dataset, our system considers it to be a new occurrence.
This means that it will be given a new gbifid and a new occurrence URL (like this one: https://www.gbif.org/occurrence/1252968762) and the old gbifid and URL will be deprecated.
In this case, this means that the occurrence URLs would be deprecated when ingesting the newest versions of these datasets.
We would like to check with you if those changes were intentional. Do you know if this is the case?
Please let us know, thanks! We are happy to resume the dataset ingestion.
Note that some users rely on those occurrence URLs and gbifids (like https://bionomia.net for example). In an attempt to improve
the stability of the occurrence URLs and gbifids, we have implemented a warning system to detect these type of changes in datasets (see this news item).
If the data publisher can provide us with a list of old and new occurrenceIDs per record, we can avoid the identifier and URL changes. Could that be an option?
Please let us know if you have any question. Thanks!
All the best,

You can skip/fix identifier validation using the registry UI.


Implemented CursorType for i32/i64

Hi ✋
I implemented CursorType for i32/i64.
Why I want to add it
Many RDB (usually backend of the GraphQL) is using i32/i64 for its id (INTEGER, BIGINT).
I think its common case that binding DB schema to the struct by using ORM.
For example I'm using async-graphql + sqlx in my production. like this.
#[derive(sqlx::FromRow, async_graphql::SimpleObject)]
struct User {
    pub id: i64,
    pub name: String,
};

// This id is BIGINT on Postgres.

I wanted to return this User from async_graphql::connection::query().
But the CursorType only implemented for usize so I had to cast between usize and i64 several part in my code.
And I couldn't impl CursorType for i64 in my crate because of Rust restriction.
I think this PR is convenient for the users who are using RDB for async-graphql's backend.
Thank you 😁

Thanks!😄

thank you!


Add FTM and PWT driver pwm support for NXP frdm_ke17z and frdm_ke17z512

Tested samples/basic/blinky_pwm, samples/basic/fade_led, tests/drivers/pwm/pwm_api and tests/drivers/pwm/pwm_loopback samples.

@ankeXiao you need to fix the merge conflicts

Fixed all merge conflicts

Hi all, please help approve and merge this PR, thanks!

@anangl, could you provide your review so it can move forwards?


Add vcpkg and remove hardcoded dependencies

Summary
This PR adds vcpkg configurations for dependency management and removes
hardcoded dependencies from the codebase. It also adds vcpkg to our GitHub
Actions workflow and uses it to automatically install our dependencies.
Notes

By integrating a package manager, we significantly simplify dependency management, improve our productivity, and promote consistency across environments.
In this configuration, there's no need to add vcpkg as a submodule to the project. Instead, individual users should install vcpkg in their home directory and set up the necessary environment variables (following this guide.)
When we're ready to add minio, all we need to do is update vcpkg.json in the root folder.
I will update the README file with installation instructions and remove the hardcoded json.hpp library from the tests folder in separate PRs.

Test Plan

All existing tests have passed.

Put find_package statements in a separate file to include in the base.

I've responded to the comment. Let me know your thoughts.

Add a note about installing vcpkg in the readme somewhere.

Yes, this was included in my PR summary notes. I initially thought of adding it to a subsequent PR, but it makes sense to update the README file here. I will update it later today.

@aliddell
I added vpkg installation instructions to the README file.

Don't forget to change back to pull_request_target!


Hardened profile may be setting unsuitable kernel parameters?

Describe the bug
https://github.com/NixOS/nixpkgs/blob/fbb76f6ace21d8f1a3956e6f329e404cad4f6136/nixos/modules/profiles/hardened.nix#L34-L39
I don't know enough to say conclusively that this is a problem, but I see it said in https://github.com/anthraxx/linux-hardened/issues/37 that the kernel parameters slub_debug=P and page_poison=1, which are set by the NixOS hardened profile, conflict with another parameter enabled by default by the linux-hardened patchset that we use as of #84522.  I see that the maintainer @anthraxx in general seems to oppose these options and in particular says that "vanilla page poisoning [...] collides with the page verify code of hardened", though I don't know which option specifically is "vanilla page poisoning" (I assume page_poison=1).
I've read the documentation on these parameters at https://www.kernel.org/doc/html/latest/admin-guide/kernel-parameters.html, but I don't see anything particularly enlightening about them, so I leave to more knowledgeable people whether this is a problem in the NixOS hardened profile.
Notify maintainers
@joachifm @emilazy

KSPP seems to imply that init_on_alloc+init_on_free supersede both slub_debug=P and page_poison=1.
https://kernsec.org/wiki/index.php/Kernel_Self_Protection_Project/Recommended_Settings

# Wipe slab and page allocations (since v5.3)
# Instead of "slub_debug=P" and "page_poison=1", a single place can control memory allocation wiping now.
# The init_on_free is only needed if there is concern about minimizing stale data lifetime.
CONFIG_INIT_ON_ALLOC_DEFAULT_ON=y
CONFIG_INIT_ON_FREE_DEFAULT_ON=y

# Wipe slab and page allocations (Since v5.3; supersedes "slub_debug=P" and "page_poison=1" below)
# See CONFIG_INIT_ON_ALLOC_DEFAULT_ON=y and CONFIG_INIT_ON_FREE_DEFAULT_ON=y above.
init_on_alloc=1
init_on_free=1

Additionally, use of slub_debug seems to disable kernel pointer hashing.
https://github.com/torvalds/linux/commit/792702911f581f7793962fbeb99d5c3a1b28f4c3

I agree, there's lots of improvements that could be made here; most of the hardened kernel config is from back in the grsecurity days and out of date or even in some cases harmful (e.g. BPF_JIT_ALWAYS_ON may arguably be better security posture these days than disabling it due to the interpreter's use as ROP gadgets). kconfig-hardened-check is a good source of other ideas for tweaks here.
I meant to get to this years ago but at this point it seems unlikely I will any time soon. I'll try my best to review any PRs to improve the hardened profile's kernel config as I think it's an important part of its value-add.

To elaborate on the above: the parameters (that were set by me, anyway) were based on KSPP recommendations at that time and assumed a vanilla kernel.
If we are now using a hardened patchset it seems prudent to also rely on their config defaults.  I see no particular reason to override them (and if that is needed we should probably try to change the hardened patchset upstream first).

@8573 I'd be happy to accept a PR that updated the kernel parameters.  I agree with your interpretation of "vanilla page poisoning" (I take it that the hardened patchset provides an alternative page sanitisation mechanism).
I don't have bandwidth to keep up with what this patchset is doing, nor what the pedigree of whatever non-vanilla mitigations the patchset provides.  I think it makes sense to follow their config defaults and recommendations for parameters, if we are to use the patchset.

Not only kconfig should be improved, but hardened kernel should also switch to being built with latest clang and its options like CFI which is supported on both x86_64 and aarch64.
Another good tweak in hardened profile would be bind mounting everything as noexec except from /nix/store and directories like /var/lib/flatpak/app if such services are used

With page_poison=1, my hardened VM (x86_64 kvm) shuts down fully during kernel init (I don't have any logs). Removing that parameter allows it to work normally again. Running latest unstable with hardened profile and linux-hardened.


View the job history

Actions do not work as described starting from step 2 to view details.

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

ID: 24523dbf-010f-b844-5ff0-88192eac8849
Version Independent ID: d7ef5962-d94c-96d0-bf08-1efffc6d402b
Content: Run Background tasks with WebJobs - Azure App Service
Content Source: articles/app-service/webjobs-create.md
Service: app-service
GitHub Login: @ggailey777
Microsoft Alias: glenga

@salomon-gilles We went through the 'View Job History' for our triggered webjob and the doc aligned with the experience we encountered. Can you please provide further details as to what does not match your experience? We look forward to your reply.

@salomon-gilles Since we have not heard back from you we will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly reopen the issue and continue the discussion.


Update dependencies for v0.1.0

[x] purescript-const
[ ] purescript-bifunctors
[ ] purescript-lens
[ ] purescript-argonaut

This breaks the CI build for purescript right now (or rather, it doesn't build anything because Bower bails out), just fyi.

Whoops, sorry. I hadn't jumped-to on this due to the potential pending name change, #3. Thoughts on that?

Honestly, I don't think it's worth the hassle. I think Contravariant is a fine name.


@ionic/vue controllers don't pass componentProps properly

Bug Report
Ionic version:

[x] 4.x
Current behavior:

If the componentProps are passed like below
          componentProps: {
            data: {
              content: 'New Content',
            },
            propsData: {
              title: 'New title',
            },
          },

I need to access the props by
this.propsData.title

Expected behavior:

According to this documentation, https://ionicframework.com/docs/api/modal, propsData will be proxied to the vue component's props
I expect to access the props by
this.title

Steps to reproduce:

Related code:

Parent.vue
...
  methods: {
    async openModal() {
      console.log(Modal);
      const m = await this.$ionic.modalController.create({
        component: Modal,
        componentProps: {
          id: this.$route.params.id,
          doc: this.doc
        }
      });
      console.log(m.componentProps);
      return m.present();
    }
  }
...

Modal.vue
...
  mounted() {
    console.log(this.propsData.id);
  },
...

Other information:

Ionic info:

Ionic:

   ionic (Ionic CLI) : 4.12.0

System:

   NodeJS : v11.12.0
   npm    : 6.9.0
   OS     : Linux 4.19

I think that before this line, https://github.com/ionic-team/ionic/blob/master/vue/src/util.ts#L5, the propsData and data should be expanded so that there's no need to access propsData and data in the modal component since later everything will be assigned to the element here, https://github.com/ionic-team/ionic/blob/master/core/src/utils/overlays.ts#L11.

The way to pass props and data should be done in this way:
      return this.$ionic.modalController
        .create({
          component: () => import('@/components/modal/index.vue'),
          componentProps: {
            parent: this,
            data: {
              content: 'New Content',
            },
            propsData: {
              title: 'New title',
              closeMe: () => {
                this.$ionic.modalController.dismiss()
              },
            },
          },
        })
        .then(m => m.present())

as it hooks into Vue here: https://github.com/ionic-team/ionic/blob/master/vue/src/controllers/vue-delegate.ts#L35
Docs on propsData
so the whole componentProps object is being passed to a newly created Vue instance


Custom Titlebar: Can't drag top border to resize window

Issue Type: Bug
While the custom title bar is enabled, hovering over the very top of the titlebar does not give the option to drag and resize the window.
Steps to Reproduce:

Enable the custom
Hover over the very top of the window. There is no option to drag and resize the window

VS Code version: Code 1.25.0 (0f080e5267e829de46638128001aeb7ca2d6d50e, 2018-07-05T13:11:58.697Z)
OS version: Windows_NT x64 10.0.16299
Extensions (4)

Extension
Author (truncated)
Version

classic-asp
ili
0.0.4

vscode-icon-theme
jtl
1.6.2

csharp
ms-
1.15.2

PowerShell
ms-
1.8.0

@jtlowe I believe this issue is resolved in insiders. Would you be able to verify?

@sbatten Yes, it is working correctly for me in Insiders. Thanks for resolving this!

@jtlowe thanks for verifying


[Bug]: Pydantic output parser breaks when using Llama 70B

Bug Description
I am getting this error when i use the pydantic output parser, notice that it actually outputs the response but not as json which i believe causes it to break.

  File "/Users/mac/Documents/GitHub/datagpt/pluralsh/dev/lib/python3.8/site-packages/llama_index/indices/query/base.py", line 23, in query
    return self._query(str_or_query_bundle)
  File "/Users/mac/Documents/GitHub/datagpt/pluralsh/dev/lib/python3.8/site-packages/llama_index/query_engine/retriever_query_engine.py", line 177, in _query
    response = self._response_synthesizer.synthesize(
  File "/Users/mac/Documents/GitHub/datagpt/pluralsh/dev/lib/python3.8/site-packages/llama_index/response_synthesizers/base.py", line 140, in synthesize
    response_str = self.get_response(
  File "/Users/mac/Documents/GitHub/datagpt/pluralsh/dev/lib/python3.8/site-packages/llama_index/response_synthesizers/tree_summarize.py", line 125, in get_response
    response = self._service_context.llm_predictor.predict(
  File "/Users/mac/Documents/GitHub/datagpt/pluralsh/dev/lib/python3.8/site-packages/llama_index/llm_predictor/base.py", line 166, in predict
    output = self._run_program(output_cls, prompt, **prompt_args)
  File "/Users/mac/Documents/GitHub/datagpt/pluralsh/dev/lib/python3.8/site-packages/llama_index/llm_predictor/base.py", line 140, in _run_program
    chat_response = program(**prompt_args)
  File "/Users/mac/Documents/GitHub/datagpt/pluralsh/dev/lib/python3.8/site-packages/llama_index/program/llm_program.py", line 79, in __call__
    return self._output_parser.parse(raw_output)
  File "/Users/mac/Documents/GitHub/datagpt/pluralsh/dev/lib/python3.8/site-packages/llama_index/output_parsers/pydantic.py", line 51, in parse
    json_str = extract_json_str(text)
  File "/Users/mac/Documents/GitHub/datagpt/pluralsh/dev/lib/python3.8/site-packages/llama_index/output_parsers/utils.py", line 68, in extract_json_str
    raise ValueError(f"Could not extract json string from output: {text}")
ValueError: Could not extract json string from output:  Yes, you can set

Version
llama-index
Steps to Reproduce

import os
from pathlib import Path
from llama_index import (
    download_loader,
    VectorStoreIndex,
    StorageContext,
    ServiceContext,
    load_index_from_storage
)
from llama_index.storage.docstore import SimpleDocumentStore
from llama_index.storage.index_store import SimpleIndexStore
from llama_index.vector_stores import SimpleVectorStore
from dotenv import load_dotenv
from llama_index.llms import Anyscale
from llama_index.embeddings import OpenAIEmbedding
from typing import List,Optional
from pydantic import BaseModel

from llama_index.output_parsers import LangchainOutputParser
from llama_index.llm_predictor import StructuredLLMPredictor
from langchain.output_parsers import StructuredOutputParser, ResponseSchema
from llama_index.prompts import PromptTemplate
from llama_index.prompts.default_prompts import (
    DEFAULT_TEXT_QA_PROMPT_TMPL,
    DEFAULT_REFINE_PROMPT_TMPL,
)
import time

# Start the timer
start_time = time.time()
llm_predictor = StructuredLLMPredictor()
# Your code starts here
JSONReader = download_loader("JSONReader")
loader = JSONReader()
documents = loader.load_data(Path("./whogohost.json"))
load_dotenv()
ANYSCALE_API_KEY = os.environ["ANYSCALE_API_KEY"]
model = "command"
temperature = 0
max_tokens = 256

class Reference(BaseModel):
    "Source model of the answer to the question"
    url: List[str]

class SearchResult(BaseModel):
    """Data model for  Search results."""
    question: str  # The original question or query
    answer: str  # The direct answer to the question
    references: List[Reference]  # A list of sources or references supporting the answer
    questions_people_might_ask: List[str]

llm = Anyscale("meta-llama/Llama-2-70b-chat-hf", api_key=ANYSCALE_API_KEY)
embed_model = OpenAIEmbedding(embed_batch_size=42)
service_context = ServiceContext.from_defaults(llm=llm,embed_model=embed_model)

# index = VectorStoreIndex.from_documents(documents,service_context=service_context)
# index.storage_context.persist(persist_dir="storage")
storage_context = StorageContext.from_defaults(
    docstore=SimpleDocumentStore.from_persist_dir(persist_dir="./storage"),
    vector_store=SimpleVectorStore.from_persist_dir(persist_dir="./storage"),
    index_store=SimpleIndexStore.from_persist_dir(persist_dir="./storage"),
)
print("done indexing")

loaded_index = load_index_from_storage(storage_context,service_context=service_context)
query_engine = loaded_index.as_query_engine(output_cls=SearchResult, response_mode="tree_summarize")
response = query_engine.query("can i setup email with gsyys?")
print(response)

# Your code ends here

# End the timer
end_time = time.time()

# Calculate and print the total execution time
total_time = end_time - start_time
print(f"Total execution time: {total_time:.2f} seconds")

Relevant Logs/Tracbacks
No response

@smyja there's no easy way to fix this. The latest version of llama-index has some improved json parsing, but it seems like to me the model didn't even try?
ValueError: Could not extract json string from output:  Yes, you can set
It could also require some prompt engineering, llama is very chatty and always seems to include extra text, which makes parsing difficult.
Overall, open-source LLMs are not very reliable for this task

@smyja there's no easy way to fix this. The latest version of llama-index has some improved json parsing, but it seems like to me the model didn't even try?
ValueError: Could not extract json string from output:  Yes, you can set
It could also require some prompt engineering, llama is very chatty and always seems to include extra text, which makes parsing difficult.
Overall, open-source LLMs are not very reliable for this task

Yeah, it's the prompting that's the problem because the langchain output parsers weren't breaking.


Rename master to main

Fix #737
Type of change

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

Please ignore the netlify failures above, they'll be running properly as soon as https://github.com/finos/rune-dsl/pull/740 is merged

@maoo Since this branch should become the new default branch, I suppose I shouldn't actually merge this PR?
I don't have permissions to modify settings though. I guess someone from the FINOS staff will have to finish this task.

ping<EMAIL_ADDRESS>with the request and we'll switch it in a matter of minutes


Added Calorie Consumed Calculator

Fixes Issue #862 🛠

Closes #862
Description👨💻

The calorie consumed calculator is added. A variety of food categories are added. Users can keep track of the quantity of food consumed.
Type of change📄

[x] New feature (non-breaking change which adds functionality)

How this has been tested✅
I have checked the calculations made by the calculator and cross-verified the calorie values with the internet.

Checklist✅

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have added demonstration in the form of GIF/video file
[x] I am an Open Source Contributor

Screenshots/GIF📷
The new calculator is added

The calculator:

Calculations:

Please check the PR @Rakesh9100
Thank you :)

Pls add a Readme file of your calculator and also place any images by creating assets folder in your calculator folder @debangi29


Only a single Adaptive Card can render before additional user input

In this specific customer scenario, they wish to send 2 different adaptive cards (in sequence not a carousel) as a response to a user.  I have found that when the second card is sent, the first will be removed from chat.
In addition to this, no additional adaptive cards will render until the user sends some other command.  For example, I click on a submit button in the adaptive card which will trigger the bot to do the exact same skill, no adaptive cards will render.  Once the user sends text to the bot, then we get back into the loop of the second adaptive card removing the first.
Here is a quick video of the issue https://youtu.be/pXocEbtEDxg
Edit: This issue does not exist in V3 and we had to roll that customer back to V3 for now.

Hi @jherres, let me get back to you after I reproduce this issue.

I am trying to repro but no luck, I am sure there are something missing. In my bot code:
await context.sendActivity({
  type: 'message',
  text: '# This is some text\n1. this is this first item\n2. the second\n3. last one'
});

await context.sendActivity({
  type: 'message',
  attachmentLayout: 'carousel',
  attachments: [{
    contentType: 'application/vnd.microsoft.card.adaptive',
    content: { ... }
  }]
});

await context.sendActivity({
  type: 'message',
  text: 'This is some more text for you to see'
});

await context.sendActivity({
  type: 'message',
  attachmentLayout: 'carousel',
  attachments: [{
    contentType: 'application/vnd.microsoft.card.adaptive',
    content: { ... }
  }]
});

I am seeing 4 activities coming from the bot, both Adaptive Cards are showing at the same time and they never get lost.
Could you write up a minimal repro to see if anything weird on your side?
This file and this line is where we might remove/overwrite some of the activities. It will be great if you can spot any wrongdoers in this code.

I am working to make V4 compatible with our no-code bot building platform (AtBot).  I am sure we are doing some things outside of the norm. Let me loop in someone who is more familiar with the actual bot code (I can only see what comes over via developer tools or fiddler).

They will be able to have a look later this evening.  But for now, I can provide the fiddler trace of the websocket traffic for those cards.  I have attached the full fiddler archive of the interaction, but the two cards are as follows:
DroppedCards.zip
First Card
{   "activities": [     {       "type": "message",       "id": "FZ0T3tDxqgr7jAax8IuOKv-j|0000004",       "timestamp": "2019-05-03T19:53:29.1256488Z",       "channelId": "directline",       "from": {         "id": "askflowbizzybot",         "name": "Ask SPTechCon"       },       "conversation": {         "id": "FZ0T3tDxqgr7jAax8IuOKv-j"       },       "locale": "en-US",       "text": "",       "attachments": [         {           "contentType": "application/vnd.microsoft.card.adaptive",           "content": {             "type": "AdaptiveCard",             "version": "1.0",             "body": [               {                 "type": "Image",                 "id": "Image",                 "size": "stretch",                 "url": "https://picsum.photos/id/1033/200/200"               }             ]           }         }       ],       "entities": [],       "channelData": {         "clientActivityID": "15569132054630.9267z69pgjf"       },       "replyToId": "FZ0T3tDxqgr7jAax8IuOKv-j|0000002"     }   ],   "watermark": "4" }
Second Card
{   "activities": [     {       "type": "message",       "id": "FZ0T3tDxqgr7jAax8IuOKv-j|0000006",       "timestamp": "2019-05-03T19:53:30.7467866Z",       "channelId": "directline",       "from": {         "id": "askflowbizzybot",         "name": "Ask SPTechCon"       },       "conversation": {         "id": "FZ0T3tDxqgr7jAax8IuOKv-j"       },       "locale": "en-US",       "text": "",       "attachments": [         {           "contentType": "application/vnd.microsoft.card.adaptive",           "content": {             "type": "AdaptiveCard",             "version": "1.0",             "body": [               {                 "type": "Image",                 "id": "HeroImage",                 "size": "large",                 "url": "https://devadmin.atbot.io/images/noimage.png",                 "horizontalAlignment": "center"               },               {                 "type": "TextBlock",                 "id": "Product Name",                 "size": "large",                 "weight": "bolder",                 "text": "This is a card for jooo"               },               {                 "type": "TextBlock",                 "id": "Product Summary",                 "text": "Product summary or description"               },               {                 "type": "FactSet",                 "id": "ProductFacts",                 "facts": [                   {                     "title": "Color",                     "value": "Red"                   },                   {                     "title": "Weight",                     "value": "3"                   },                   {                     "title": "Brand",                     "value": "Puma"                   },                   {                     "title": "Release Date",                     "value": "2019-02-02"                   }                 ],                 "separator": true               }             ],             "actions": [               {                 "type": "Action.Submit",                 "id": "WhoMakesIt",                 "data": {                   "keyword": "wfh"                 },                 "title": "Who Makes This"               },               {                 "type": "Action.Submit",                 "id": "LocateProduct",                 "title": "Locate Product"               }             ]           }         }       ],       "entities": [],       "channelData": {         "clientActivityID": "15569132054630.9267z69pgjf"       },       "replyToId": "FZ0T3tDxqgr7jAax8IuOKv-j|0000002"     }   ],   "watermark": "6" }

Oh, you know Fiddler? That's great.
The other way, you can also send us the stream of activities coming thru. BTW, the network calls could be sensitive, so please scrap personal information if you do.

Got it, will work on it. Ha, we actually have the same idea on how to debug. 🤩

I bet I got it.
In the activity, please remove channelData.clientActivityID. When the bot send an activity, it should not have this one set, unless the bot specifically put it in.
The clientActivityID appears in the message from the user (from Web Chat), it is an ID that Web Chat used to track/acknowledge if the activity successfully reach the bot or not, a.k.a. "our reference ID".
Web Chat send "message A" to Direct Line channel with this clientActivityID. Shortly after the bot response, Direct Line will send "message A" back with the same clientActivityID. When Web Chat receive this second "message A", it will remove the previous one with same clientActivityID, and replace it with this one.
So, if you send two activities with same clientActivityID, they will get replaced. And since this is a mechanism for message originated from user, bot shouldn't send it. So it's safe to remove it.

Ahh, thanks for the second set of eyes.  We confirmed that this is the fix and will make that update on our side.


Unique constraint issue while using MongoDB

Bug description
I am using NestJS and prisma for mongodb, I started facing this issue since I updated my prisma to v5.20.0

How to reproduce

Expected behavior
It will throw an unique constraint error

Prisma information

// Add your schema.prisma
model User {
  uuid                   String                  @id @default(auto()) @map("_id") @db.ObjectId
  email                  String                  @unique
  email_verified         Boolean                 @default(false)
  name                   String
  google_connected       Boolean?                @default(false)
  google_connected_email String?
  @@unique([google_connected_email])
}

// Add your code using Prisma Client (I am using NestJS and it's prisma [recipe ](https://docs.nestjs.com/recipes/prisma))
await this.prisma.user.create({
      data,
      include,
    })

Environment & setup

OS: Windows 11 Pro
Database: MongoDB
Node.js version: 20.10.0

Prisma Version
5.20.0

Hi @aryabhthakur!
Your schema has a unique constraint over google_connected_email field. The error basically tells you are trying to create a document which violates this constraint: meaning there is already a document with the same google_connected_email you are trying to create. If that's not the case for you, please provide a reproduction repository where we could see it happen. Be sure to include the data present in your DB into reproduction.
Thanks!

@SevInf that's the issue! the database is newly formed. It has no entry, I tried deleted indexes and recreating them using db pull but still same!

@aryabhthakur could you provide an isolated reproduction repository where we could see the error happening?

Hi @SevInf  seems it was MongoDB's issue!! I create new database and it worked fine init.


Make README.rst a real file, and docs/index.rst a link instead

Issue #37 reports at least one user having problems the other way
around, on Windows 7 32bit

Current coverage is 95.77%

Merging #38 into master will not affect coverage as of 1de2079

@@            master     #38   diff @@
======================================
  Files            7       7
  Stmts          450     450
  Branches         0       0
  Methods          0       0
======================================
  Hit            431     431
  Partial          0       0
  Missed          19      19

Review entire Coverage Diff as of 1de2079

Powered by Codecov. Updated on successful CI builds.


cigien: Watch (?-i:GFYGtlSY1yY)(?#YouTube)

cigien requests the watch of the watch_keyword (?-i:GFYGtlSY1yY)(?#YouTube). See the MS search here and the Stack Exchange search in text, in URLs, and in code.

(?-i:GFYGtlSY1yY)(?#YouTube) has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by tripleee in Charcoal HQ


Adding python definition

Fixes Issue #111

@heldersepu That looks like some problem with the build system, I don't think it has anything to do with your code.

@jimfoye @dgrunwald  Any chanse this PR can be merged soon?
I also have another PR #107

I'm not a maintainer, actually (:

The Python .xshd looks good; thanks!
I'll have to take a closer look at your change to FoldingManager.cs. The new code certainly seems to make more sense, but we've had problems with this in the past, so I want to treat carefully here.

@dgrunwald
I can't remember why I changed the FoldingManager.cs, I'm looking into it...

@dgrunwald
Take a look at this:
https://github.com/heldersepu/AvalonEdit/commit/9c55316d14f8a5aecd3ed9c35c0df745b4fe4e67
It seem it was when I did a merge

@dgrunwald
You should be able to merge this PR without looking into the FoldingManager.cs
When can we expect an update to the NuGet Package?

i think the change of foldingmanger is also in this pull req: https://github.com/icsharpcode/AvalonEdit/pull/87/commits


Implement an operating system check for FIPS certified distributions

Some operating systems have gone through the certification process, and
we can use that information to let users know if they're using a version
or operating system that hasn't been FIPS certified, yet.

This patch still needs a lot of work, but I wanted to at least sketch out a quick approach based on what I've read in the code so far.

/lgtm
/hold for @kolyshkin

/hold cancel


Fix incorrect function name in BN_bn2bin manpage

Doc-only fix

Do you agree this is trivial? Else we'll need a CLA.

Yes this is totally trivial :)

Merged, closing.


5.8 BelongsTo Relation fix

@jarektkaczyk Please merge <3

@OzanKurt thanks,
could your write a failing test for that too?

I don't know what you mean by that. :/

@OzanKurt
show me how it's broken in a sample code, that will do :)
thanks!

Pretty simple,
// relation -> BelongsTo
Model::joinRelations(['relation'])->get();

the code above generates this Exception:
BadMethodCallException
Call to undefined method Illuminate\Database\Eloquent\Relations\BelongsTo::getQualifiedForeignKey()

I hope this helps.

Please add this update, as the implementation os broken in Laravel 5.8.

@studmon @OzanKurt tagged https://github.com/jarektkaczyk/eloquence-base/releases/tag/v5.6.3

Hi, @jarektkaczyk thanks for the great package. Just to say that this change appears to be breaking for Laravel 5.7. Is not the end of the world as easy to revert but thought perhaps it should go in a major point release?

thanks, will look into it!
Regards,
Jaroslaw Tkaczyk
SOFTonSOFA :: development with pleasure
from Krakow, in Singapore
On Wed, 22 May 2019 at 14:46, coachops<EMAIL_ADDRESS>wrote:

Hi, @jarektkaczyk https://github.com/jarektkaczyk thanks for the great
package. Just to say that this change appears to be breaking for Laravel
5.7. Is not the end of the world as easy to revert but thought perhaps it
should go in a major point release?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/jarektkaczyk/eloquence-base/pull/13?email_source=notifications&email_token=ABU3TMTKZW72KEHELQTFFGDPWTT5FA5CNFSM4HADHOW2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODV6CGKA#issuecomment-494674728,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABU3TMRHD56J4YM5UA36APDPWTT5FANCNFSM4HADHOWQ
.


Table page: Extra html tags, and Browse button leads back to front page

At the top left, there is a set of html closing tags:

The Browse button leads back to the front page, but maybe it should go to the Browse section of Help?

fixed
browse section is good. same behavior as wp site.


replace API from Azure AD Graph API to Microsoft Graph REST API for az ad commands

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

Currently, az ad sp create-for-rbac seems using Azure AD Graph API.
msrest.http_logger : Request URL: 'https://graph.windows.net/xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx/servicePrincipals?$filter=servicePrincipalNames%2Fany%28x%3Ax%20eq%20%27http%3A%2F%2FmyAKSClusterServicePrincipal%27%29&api-version=1.6'

As the article like this recommends migration from Azure AD Graph to Microsoft Graph, az ad commands need to be migrated to new.
Describe the solution you'd like

Describe alternatives you've considered

Additional context

add to S174

Thanks for the feedback. This request is being tracked at https://github.com/Azure/azure-cli/issues/12946.


Multiline label on xAxis cutting off

Thanks for writing this awesome library. Saved us lot of time. We need to display multiline label on xAxis for bar charts. I made the label as word wrap -->    xAxis.wordWrapEnabled = YES; I find only first line displays in my project. Rest of characters are cutting. I tried to replicate in ChartsDemo. I changed the label "Mar test1 test2 test3 test4 test5 test6". On the screen I only see "Mar test1 test2"

Labels are dynamic, and we cant know for sure how long will they be.
So you just need to add extra padding for the legend.


Display dependency description on hover

either read local pom file, or fetch remote pom (query multiple repos if needed), then parse the xml to display project.description (else project.name)

Done with a couple of caveats:

since it's implemented in Typescript, the doc provider naively expects the local maven repo to be ~/.m2/repository, i.e. ignores any Maven settings.xml
variables are not resolved
description is not inherited from parent hierarchy


FIX-#4414: Add missing f prefix on f-strings found at https://codereview.doctor

Fixes #4414

Thanks @code-review-doctor! I've made some minor changes that are required in order for your pull request to be accepted, I hope you don't mind.
@modin-project/modin-core ready to merge.

Changes look good, but don't we need the original author to sign off their commits (instead of Modin contributors signing them)?

@vnlitvinov In situations where the original author can't do this, should we somehow upload pull requests?

@modin-project/modin-core the ball is on your side


Suggestion

Hi, Really like this idea and I haven't found much like this. I have too had some problems. I found that if the computer is not playing any audio apon opening it will not transmit audio.
If you have a problem try opening media and playing and then open the application.
I am not sure how to suggest items as I have only just created an account on github but I have been using it for years. It would be really cool if you could transmit audio and not have to hear it from two systems.  currently I use the program in order to transmit audio from my computer to another room. however I have it playing on my headphones which isn't great and not really needed.  Potentially using a separate audio device. Not sure if that is feasible.

When I have managed to get this connected, I merely muted the speakers on my laptop and the took care of the issue.

PS. acidhax has sent out a couple messages stating that the software will be updated soon, so stay tuned.

Please verify this is still an issue by testing https://github.com/acidhax/chromecast-audio-stream/releases/download/0.3/audio-cast-win32-ia32.zip


"Page Not Found" error message

Additional error message needed in case there is no page at all - right now, this incomplete message is displayed:

Page Not Found
The page you requested, , does not exist.

I'd say the sentence would be fine without the commas.

I think the space between the commas has traditionally printed the URL of the missing page.

The page you requested, /super/zombies/attack/, does not exist

We just discussed this internally: the problem might be that it's the index page that doesn't exist.
(The install doesn't have any pages yet and thus no index page.)

What should appear is the URL that you tried to access. What version of
Symphony is this?

2.3 from integration has been doing it for a while now.

What version of Symphony is this?

Symphony 2.2.5.

What should appear is the URL that you tried to access.

As I mentioned above, we are at the site's root so I guess the page name is empty – Tony's pull request should fix the problem.


[10.x] In MySQL, harvest last insert ID immediately after query is executed

I work on an app whose database is MySQL, and its sql_mode is set to ''; the primary impact of this is that "strict mode" is off, and too long/large data is truncated and inserted, instead of rejected. As you can imagine, this leads to problems later on, when we try to query that truncated data and we're missing pieces, or an integer/decimal is set to a max/min value.
In an attempt to log the warnings that are emitted from the DB in these scenarios, I set up a listener for QueryExecuted, and called SHOW WARNINGS to get the list of warnings, and report them to our error reporting system. This allows us to see the warnings, in the context of the executing query and the PHP stack trace, without crashing our app. We hoped to work these warnings down to an acceptable level, then turn on strict mode.
However, I discovered that executing any queries during the QueryExecuted event causes PDO to drop the last insert ID, and so all tests that insert any data broke, due to getting '0' back from PDO, rather than the actually generated ID. After some Googling, I found this PHP bug report, that claims this is not a bug, but the intended behavior:

From what I understand this is expected behavior, so mysqlnd is backwards compatible with libmysql. The core developers purposefully reset the last insert ID on non INSERT statements to emulate the same behavior as the old libmysql library.

In order to still fetch the last insert ID, but also be able to execute queries during the QueryExecuted event, I've modified the MySQL specific Processor and Connection classes. The test added fails without the changes applies.

It is possible to accomplish this without framework changes, because we can extend the MySQL classes, and register a customer driver via \Illuminate\Database\Connection::resolverFor() and \Illuminate\Database\Connection::getResolver(). In fact, we will probably be doing that until this is merged and we can update to the version its released with. However, I feel it is beneficial to add it to the framework anyway. The other PDO compatible drivers don't seem to have this problem. I did some local testing with PostgreSQL, and it behaved fine without any additional changes: I could execute queries during QueryExecuted and the last insert ID was still harvested correctly.
I acknowledge that my use case is niche, but this should help any future devs who happen to need to execute queries during that event.

I'm targeting 10.x because it's just barely still in "bug support" dates, and this does seem like a bug. I'm happy to rebase to 11.x if it's considered a minor feature, or 12.x, due to the change in signature to \Illuminate\Database\MySqlConnection::insert(). I think that change would affect devs who extend MySQLConnection.

Related:
https://github.com/laravel/docs/pull/9091

Hey @piurafunk,

I think that change would affect devs who extend MySQLConnection.

Just a heads up, this change appears to affect devs who extend MySQLConnection. We have a customer reporting issue with the following composer upgrade path:
• Upgrading laravel/framework (v10.48.15 => v10.48.19)

Here is the related stack trace:
BadMethodCallException: Method October\Rain\Database\Connections\MySqlConnection::getLastInsertId does not exist. in ~/vendor/laravel/framework/src/Illuminate/Macroable/Traits/Macroable.php:112
Stack trace:
#0 ~/vendor/laravel/framework/src/Illuminate/Database/Query/Processors/MySqlProcessor.php(37): Illuminate\Database\Connection->__call('getLastInsertId', Array)
#1 ~/vendor/laravel/framework/src/Illuminate/Database/Query/Builder.php(3507): Illuminate\Database\Query\Processors\MySqlProcessor->processInsertGetId(Object(October\Rain\Database\QueryBuilder), 'insert into `fr...', Array, 'id')
#2 ~/vendor/laravel/framework/src/Illuminate/Database/Eloquent/Builder.php(1982): Illuminate\Database\Query\Builder->insertGetId(Array, 'id')
#3 ~/vendor/october/rain/src/Database/Builder.php(249): Illuminate\Database\Eloquent\Builder->__call('insertGetId', Array)
#4 ~/vendor/laravel/framework/src/Illuminate/Database/Eloquent/Model.php(1334): October\Rain\Database\Builder->__call('insertGetId', Array)
#5 ~/vendor/laravel/framework/src/Illuminate/Database/Eloquent/Model.php(1299): Illuminate\Database\Eloquent\Model->insertAndSetId(Object(October\Rain\Database\Builder), Array)
#6 ~/vendor/laravel/framework/src/Illuminate/Database/Eloquent/Model.php(1138): Illuminate\Database\Eloquent\Model->performInsert(Object(October\Rain\Database\Builder))
#7 ~/vendor/october/rain/src/Database/Model.php(383): Illuminate\Database\Eloquent\Model->save(Array)
....

Related extended classes:

October\Rain\Database\Connections\MySqlConnection
October\Rain\Database\Builder

From what I can see, your Connection class extends the Illuminate Connection class, rather than MySQLConnection. Yours is named MySQLConnection, but doesn't actually extend from the Illuminate MySQLConnection. However, your MySQLConnection does use the default MySQLProcessor, which does have these changes. I'm not sure if it's worth it for you, but maybe extending the Illuminate MySQLConnection would remove the need for your patch.

Ah, yes. You're right, the fragility here could be by our own design. The reason we don't extend the Illuminate MySQLConnection base class, since we also require extending the base Illuminate\Database\Connection class to swap the QueryBuilder instance and other things. Perhaps we could overcome this by using a trait instead. Thanks for your reply.


Enhanced Bank Card Details with Regex and Color Variations

This update adds regex validation for bank card numbers and IBANs, ensuring accurate input checks. It also introduces lighter and darker color variations for each bank's primary color, enhancing the UI with better customization and visual appeal.

Thanks Amir
Good Job!


Url Asset Ingestor saves the same path in migratedFom property

getItemName returns path of the migrated item, if exception occurs - url will be used as Item Name

I would create another pull request


Account char length

What version of Node.js are you using?
Latest
What version of npm are you using?
Latest
What operating system are you using?
Ubuntu Server
What version of SnailyCAD are you currently using?
Up to date with repository.
Describe the Bug
Users are able to create 1-2 char length accounts, but are unable to log in with them.
Expected Behavior
Prevent accounts of length 1-2 from being created.
To Reproduce
No response

Weird. They use the same schema. Could this be if the user first registered with Discord, then set a password?


Enable selection of installation path

our corporate policies do not allow the executables to be run from withing the install-path of the emulator at ...\AppData\Local\Programs\@bfemulatormain

However, there is currently no option to change the installation path in the installer.

This is actually something that has been proposed before, and I believe could be done using one of the electron-builder build flags.
I actually have a branch with this enabled, and it seems to work, we would just need to get some custom images for the installer.
See #1259

Until then I found the following workaround:

install
copy the install folder to C:\ Program files x86
uninstall
run the copy

Closing due to inactivity

What?! Inactivity counts as a fix?

Reopening to track. As this issue still hasn't been fixed.

Thanks @tonyanziano
A few loading graphics to try out. See how they look.

addressed in https://github.com/microsoft/BotFramework-Emulator/pull/2077


GitHub misc

Some fixes for the Github API.
Also introduce Repo::search_issues which allow to list specific issues with an advanced query.

+1

+1


[release-1.1] vmi update admission webhook: fix volume miscount when using filesystems

What this PR does
This is a manual backport of https://github.com/kubevirt/kubevirt/pull/11038

Fixes #
Why we need it and why it was done in this way
The following tradeoffs were made:
The following alternatives were considered:
Links to places where the discussion took place:
Special notes for your reviewer

Checklist
This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR.
Approvers are expected to review this list.

[ ] Design: A design document was considered and is present (link) or not required
[ ] PR: The PR description is expressive enough and will help future contributors
[ ] Code: Write code that humans can understand and Keep it simple
[ ] Refactor: You have left the code cleaner than you found it (Boy Scout Rule)
[ ] Upgrade: Impact of this change on upgrade flows was considered and addressed if required
[ ] Testing: New code requires new unit tests. New features and bug fixes require at least on e2e test
[ ] Documentation: A user-guide update was considered and is present (link) or not required. You want a user-guide update if it's a user facing feature / API change.
[ ] Community: Announcement to kubevirt-dev was considered

Release note

NONE

/cc @xpivarc

/lgtm


File download progress

In interactive scenarios it'd be nice to have a log that shows the download progress, both in terms of the total files to be downloaded and in terms of the percentage of the current file.

Fixed by adding stream option in #10


fix(move css): move css to Core

move css

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots:
Before (If applicable):

After:

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation update
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[x] Build related changes
[ ] CI related changes
[ ] Other... Please describe:

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

Is this ready for review? Seems to be a lot of commented out code/comments.

Is this ready for review? Seems to be a lot of commented out code/comments.

I think the remark is in the building tool and playground css. I've removed it.


Add thread safe search for faiss IVFFlat

Signed-off-by: liliu-z<EMAIL_ADDRESS>issue: #402
Background
Issue from Milvus: https://github.com/milvus-io/milvus/issues/18509
IVFFlat is not thread-safe:

IVF use nprobe to init a array, this nprobe is set by users. And then, in search_preassigned method, it use the field nprobe to iterate the array. This problems also happen to range_search, etc.
this nprobe, and parallel_mode can be set in knowhere by a query, and get changed by another before the first query use it.

Solution
We create a group of new function accept the search param such as nprobe,parallel_mode to make it thread safe.
What does this PR do

We create a IndexFlat-thread_safe.cpp file to maintain all these knowhere created functions. This make us easy to control and delete in the future if needed.
We move search_without_codes, search_preassigned_without_codes, range_search_without_codes and range_search_preassigned_without_codes to this new file. Since all these function is created by knowhere before. (BTW, we need to refactor these code, too many dup code which brings troubles to the future's modification). Also we rename them with thread safe prefix.
We create thread_safe_search and thread_safe_range_search in this new file.
We modify the IVFSearchParameters to have parallel_mode in it. Since we have race condition in this field too.
Modify IndexIVF_NM and IndexIVF in knowhere to support IVFFLAT, IVFPQ and IVFSQ.

Function structure changes
Below shows the function call chains of IVF
Before change
search                         ->     search_preassigned
search_without_codes           ->     search_preassigned_without_codes
range_search                   ->     range_search_preassigned
range_search_without_codes     ->     range_search_preassigned_without_codes

After change
search/{thread_safe_search}                  ->     search_preassigned
{search_without_codes}                       ->     {search_preassigned_without_codes}
range_search/{thread_safe_range_search}      ->     range_search_preassigned
{range_search_without_codes}                 ->     {range_search_preassigned_without_codes}

the function name in the braces is in the new file. We didn't show the name changes here

/lgtm


FancyZones moves PT Settings window into a zone (first run after update)

Microsoft PowerToys version
0.47
Running as admin

[ ] Yes

Area(s) with issue?
FancyZones
Steps to reproduce
Updated to 0.47 today. Had Settings window open, not maximised and not (explicitly) dragged into a zone.
✔ Expected Behavior
No action
❌ Actual Behavior
Settings window suddenly resized to fit one of my zones. (center on display 1, corner on display 2 - don't know the zone number because that disappeared). Happens every 2~5 minuts or so. Happened after the first time PT runs after the update. Not happened when FZ is switched off.
After exiting and running PT again, haven't noticed this again - so far.
ZIP file is created after this moment (forgot to make it the first time)
PowerToysReport_2021-09-30-16-17-20.zip
Other Software
No response

Can you please verify if \AppData\Local\Microsoft\PowerToys\FancyZones\app-zone-history.json contains "app-path": "...\\PowerToys.Settings.exe"?
If it's there, could you please verify what happens if you remove it from that file? Just remove the whole element when PowerToys is off.
{
   "app-path": "...\\PowerToys.Settings.exe",
   "history": [ ... ]
}

Ping me if it would accidentally be snapped to a zone again.

The app-path of the installed version is not present (anymore). Only D:\\Visual_Studio\\Solutions\\PowerToys\\x64\\Debug\\Settings\\PowerToys.Settings.exe is, but that's a different file so should not be relevant. And it hasn't happend ever since - didn't see it on a different device (after updating) either. So, I don't know what caused it.

@SeraphimaZ Just updated my desktop to 49.1 and the same thing happend. This time there is the data you mentioned in the json.

This time I created a new report zip, before closing anything.
PowerToysReport_2021-11-08-14-18-58.zip

Thank you, I'll take a look.

Are you still experiencing this issue? /needinfo


How to get you mAP?

Excuse me.
I want konw how do you get your mAP?
I eval my model with objection_datection/eval.py, but I can't get the value.
Hope you can tell me, thank you.

Information on mAP can be obtained in tensorboard.
During training, you can run an eval process and point tensorboard to the checkpoint directory.
See instructions here.


Bot3

Checked other resources

[X] I added a very descriptive title to this issue.
[X] I searched the LangChain documentation with the integrated search.
[X] I used the GitHub search to find a similar question and didn't find it.
[X] I am sure that this is a bug in LangChain rather than my code.
[X] The bug is not resolved by updating to the latest stable version of LangChain (or the specific integration package).

Example Code

Why__

Error Message and Stack Trace (if applicable)
Hyy
Description
Sir
System Info
Hello

Anybody

Anybody

Hack

Hack


Middleware is used on every request if the parameter "route" is '/'

In the source code https://github.com/senchalabs/connect/blob/master/index.js line 101:
// strip trailing slash   if (path[path.length - 1] === '/') {     path = path.slice(0, -1);   }
Which changes the path from '/' to ''.
If I want to run the middleware just for homepage which path is '/', I have to pass '//' for parameter "router'.

Sorry, I misunderstood the connect usage.
How to set the route if the middleware is just for '/', not for '/foo', '/xxx' and so on?

Hi @jeange1003 you can't; that is not a feature of connect unless you use some routing middleware on top, like https://github.com/pillarjs/router :
var connect = require('connect');
var router = require('router');

var app = connect();
var router = router();

// Mount a router
app.use(router);

// respond to only / requests
router.get('/', function(req, res){
  res.end('Hello from Connect!\n');
});

app.listen(3000);

Got it, thank you.


Dialog for editing your Open Payments Account API settings

The paymentPointer.json file on a user's pod would contain something like:
{
  "id": "https://wallet.example/alice",
  "accountServicer": "https://wallet.example",
  "assetCode": "USD",
  "assetScale": 2,
  "auth" : {
    "authorization_endpoint": "https://auth.wallet.example/authorize",
    "token_endpoint": "https://auth.wallet.example/token",
  }
}

These values should be editable through a GUI.
And maybe also through some redirect-dance with the wallet provider?

Continued in https://solidos.solidcommunity.net/Team/2021/money-pane-project/money pane/state.ttl#Iss1610015215447

Continued in https://solidos.solidcommunity.net/Team/2021/money-pane-project/money pane/state.ttl#Iss1610015215447


Autocorrect for expect(current_path) moves method argument when parens are absent

Code like expect(page.current_path).to eq(article_path article) gets autocorrected to
expect(page).to have_current_path(article_path article, ignore_query: true)
The correct form is
expect(page).to have_current_path article_path(article, ignore_query: true)

Thank you for your feedback! You're absolutely right, so I have opened https://github.com/rubocop/rubocop-capybara/pull/39


Be able to run my .py without SPOTIPY_CLIENT_ID & SPOTIPY_CLIENT_SECRET

Hello,
I think that it could be good that we can run our .py file without SPOTIPY_CLIENT_ID & SPOTIPY_CLIENT_SECRET in the console
Because we should do that in our code  spotipy.util.prompt_for_user_token(username, scope=None, client_id=None, client_secret=None, redirect_uri=None, cache_path=None, oauth_manager=None, show_dialog=False) and I don't want to use redirect_uri

You already can by using SpotifyClientCredentials, see parameters from https://spotipy.readthedocs.io/en/2.13.0/#spotipy.oauth2.SpotifyClientCredentials


Creating the Navigation panel (Sidebar) 🚋

EduPlanner-Frontend: Navigation panel
🌟Features:

[x] - The Sidebar created and styled with basic css.
[x] - The react useState hook ↪ is used for tagging the sidebar states.
[x] - The CSS media query is used to make the sidebar responsive.

Thank you so much 🙏  @gisachris, for reviewing and approving my pull request.
Your nice comments are really appreciated.


feat: add hooks for tracking closest points

Closes: https://github.com/zakodium/react-plot/issues/314

Some brainstorming can be found here: https://hackmd.io/-CYbArNQR_KsMriP0uP4Dg


Reply script with no value

Hi,
Can this be achieved? Let me tell you what is going on here.
I have a script that stays on the screen after executing, i've tried sending it to >dev/null with success but http still waits for response until a timeout is displayed. What i want to do is access the "/command" and the website to reply with no content.
Thanks.

Hi,
What do you mean "stays on the screen"? Please provide minimal example.
Shell script with no content like this: shell2http /test 'echo "text" > /dev/null' - works properly.


Updating Typing for the new method "addChildren"

This is an addition to this MR, https://github.com/bkrem/react-d3-tree/pull/417.
Need to update the type properties for CustomNodeElementProps to include addChildren.

🚢 https://github.com/bkrem/react-d3-tree/releases/tag/v3.6.1


use production rand 0.5 release

0.5 is finally out of prerelease purgatory 🎉

Is there anything blocking getting this merged? I can't build with dalek at the moment because the old rand version causes all kinds of problems.

rand dependencies in many libs get broken without this change.
@isislovecruft could you please merge this and release x25519-dalek 0.3.0?

Any updates on this? This currently forces me to use an outdated version of snow, see https://github.com/mcginty/snow/pull/29

Hi! I'm sorry, I totally missed this in all the noise. I'll merge and put out a point release today.


chore: use SignatureAlgorithm.EthereumPersonalSign for verification's externalSignatureType attribute

Motivation
Last item from https://github.com/farcasterxyz/hub/issues/49
Change Type

[ ] Protocol Feature: changes the design of the hub and the protocol
[ ] Hub Feature: changes the design of the hub, but does not affect the protocol
[ ] Bugfix: solves an issue with the current implementation
[x] Chore: miscellaneous improvement to the system

Change Summary
Replace eip-191-0x45 string with SignatureAlgorithm.EthereumPersonalSign for externalSignatureType field of verifications.
Merge Checklist
Choose all relevant options below by adding an x now or at any time before submitting for review

[x] The title of this PR adheres to the conventional commits standard

Codecov Report

Merging #76 (6a32398) into main (9b1491f) will decrease coverage by 0.10%.
The diff coverage is 75.00%.

@@            Coverage Diff             @@
##             main      #76      +/-   ##
==========================================
- Coverage   89.21%   89.10%   -0.11%
==========================================
  Files          10       10
  Lines         816      817       +1
  Branches      199      199
==========================================
  Hits          728      728
- Misses         87       88       +1
  Partials        1        1

Impacted Files
Coverage Δ

src/client.ts
96.15% <ø> (ø)

src/factories.ts
97.64% <ø> (ø)

src/engine/index.ts
81.16% <50.00%> (-0.37%)
:arrow_down:

src/types/index.ts
100.00% <100.00%> (ø)

src/types/typeguards.ts
98.38% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


#552: Fix zip extraction

Closes #552. Closes also: #550 (edit: PR #615 now handles #550)

Separates extraction for zip and tar
Use ZipInputStream instead of ArchiveInputStream for zip files
Fixes download url containing spaces due to plugin names to be formatted correctly
Fixes two buggy log statements

Pull Request Test Coverage Report for Build<PHONE_NUMBER>6
Details

0 of 0   changed or added relevant lines in 0 files are covered.
69 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.3%) to 65.367%

Files with Coverage Reduction
New Missed Lines
%

com/devonfw/tools/ide/log/AbstractIdeSubLogger.java
9
49.15%

com/devonfw/tools/ide/io/FileAccessImpl.java
60
57.4%

Totals

Change from base Build<PHONE_NUMBER>1:
-0.3%

Covered Lines:
5927

Relevant Lines:
8744

💛  - Coveralls

What is the status here? Can we close this as PR #567 already fixed #552 if I am not mistaken?
There may be some valuable test extensions here but we need to rebase this feature-branch before we can make use of it...

What is the status here? Can we close this as PR #567 already fixed #552 if I am not mistaken? There may be some valuable test extensions here but we need to rebase this feature-branch before we can make use of it...

This PR is now reduced to the test addition and the fix of two buggy log messages. The new implementation of ZIP extraction originally made for this PR had been moved to #552 and already been merged.


Form input values not retained after form submission

Package
filament/forms
Package Version
v2.0.0
Laravel Version
v8.0.0
Livewire Version
v2.10
PHP Version
8.0.0
Problem description
I am using the Rewrite component in my Livewire application and I have noticed that the form input values are not being retained after the form is submitted.
The form resets the input values null.
And if a user fills out the form and clicks on the submit button then he wants to change a value from the select input, all the previously selected options return empty.
I am not sure what could be causing this issue, but it is causing problems for me as I need to retain the old values in the form after it is submitted.
https://user-images.githubusercontent.com/90805670/210137696-163ca991-5b3c-4a15-be88-ad5a61d9d397.mp4
Expected behavior
The form input values should not reinitialize to null.
Steps to reproduce

Go to the page with the Rewrite component.
Fill out the form with some values.
Submit the form.
Observe that the form input values are reset to null.

Reproduction repository
https://github.com/mostafa-amine/form-reset-reproduction
Relevant log output
No response

Downloaded your repo, cannot reproduce this
Uploading Screen Recording 2023-01-02 at 22.04.29.mov…


Feature Request: Native file drag & drop into filesystem

Is your feature request related to a problem? Please describe.
I have a use case which requires dragging files (references to local file paths) from the webview "out" into file system. This is achievable in electron (link), and think it would a beneficial feature of Tauri.
Describe the solution you'd like
In a webview, I would like the ability start a drag which references a file/multiple files and drop it somewhere in the filesystem.
Describe alternatives you've considered
I asked about this in the Tauri discord, there are no alternatives and I was prompted to create this Feature Request.
Additional context
Here is a gif of the behavior in electron

For file explorers based on web technology like Xplorer, this is pretty much a required feature.

Hello, version 1.0 is out right now, is there any chance for this feature to be implemented? 👀

Hi, newer versions of webview2-com have drag and drop methods such as DragOver DragDrop DragEnter and Drop, and afaik these are capable of dragging files into file system

Hi, any updates on that feature?

Does anyone have a way to do this? A plugin maybe?
To advance the cause I put this here: https://webkit.org/blog/8170/clipboard-api-improvements/

I'm also interested in this feature as i'm working on an audio related app and would like to drag the files from the app to audio editors directly.
I have tried with electron and it works using an IPC call, perhaps this can work as a reference implementation for Tauri: https://github.com/electron/electron/pull/6333

Bump! Any hope for this feature?

This feature should not be labeled low priority in my opinion. Many desktop applications require this feature.
Hope to see it added!

I appreciate this being marked higher priority.  windows should support tauri://file-drop events

Do you currently have any solutions

This is crucial feature, which Electron has. We need to support dragging files inter-applications in system.

Thanks to Kino AI, we're pleased to announce that the team at CrabNebula will contribute this long-requested feature upstream to WRY and integrated into Tauri.

blog post wen

blog post wen

Soon. :tm:

Thanks to Kino AI, we're pleased to announce that the team at CrabNebula will contribute this long-requested feature upstream to WRY and integrated into Tauri.

amazing! also nice to discover CrabNebula!

I really hope this feature gets released in v2 or sooner. :)

Some kind of roadmap for this feature would be greatly appreciated, and thanks to the team at CrabNebula for taking this on!

We will be publishing the feature mid next week after the documentation is done.

We will be publishing the feature mid next week after the documentation is done.

Is this the repo? https://github.com/crabnebula-dev/drag-rs
Gave the examples a try on MacOS and dragging is indeed working, great work!

Beautiful. I can't wait to finally start making my file manager program.

Hope to see it added!


Add #37 add verbose v006

Add #37 add verbose v006

Yes it was intended. This request is rather sensitive since it contains the user's/application's credentials. I think it would be best to exclude it all together.
With that being said I am not excluding the Password when executing cybr accounts add, I will add another commit to address this shortly

Yes it was intended. This request is rather sensitive since it contains the user's/application's credentials. I think it would be best to exclude it all together.
With that being said I am not excluding the Password when executing cybr accounts add, I will add another commit to address this shortly

I tested this and it is successfully scrubbing the password with the most recent commit.

I tested this and it is successfully scrubbing the password with the most recent commit.


Convert variadic parameters to arrays [BC break]

This fixes https://github.com/bartfeenstra/fu-php/issues/73 by introducing consistent behavior, but makes calling code slightly harder to read, especially for Iterator methods that can take multiple other iterables, but are called with just one. Such invocations will look like $iterator->zip([$other]), which is not that bad, but slightly harder to read.

This PR does remove some validation, as variadics and required parameters were used to ensure at least one other iterable would be given.

Coverage decreased (-0.006%) to 98.058% when pulling 0c3bffcebfa4ce7adbeb737f68283e7e689058d8 on non-variadic into c4afb56d31da2250efca8f974657544d1d339077 on master.

Coverage decreased (-0.001%) to 99.676% when pulling 86aa3afd0b4010c815887b2ca0911d852551d9a1 on non-variadic into 0b974c9dd10e404d194d3fe6b064cc128f50004c on master.


Different errors raised for similar cases

MidiFile.length raises a ValueError whereas MidiFile.__iter__ a TypeError in case of type 2 MIDI files. I think these should be unified, but I am not sure which one is more appropriate?

Are Type 2 MIDI files not supported?
If so maybe a custom error? "InvalidMidiFileType" or something? I find those are usually more specific than any builtin for something like this.

No, type 2 MIDI files are supported. Only iterating over all messages is not, because the individual tracks of these files are asynchronous and can thus not be combined.
As a side note, maybe adding a __iter__ method to MidiTrack might be a good idea to iterate over the messages of a specific track.


Dev Home: so much white space on summary page

Dev Home version
No response
Windows build number
No response
Other software
No response
Steps to reproduce the bug
Description
Expected result
No response
Actual result
No response
Included System Information
No response
Included Extensions Information
No response

this is larger than just summary, this is just across most things, this doesn't need design as this is stock app design guidelines.

Duplicate of #869


Add AWS CI to telemetry CI checks

Proposed changes:

As a result of findings for issue #8694, I added AWS CI environment variables to the telemetry module to improve tracking of all CI pipelines.

Status (please check what you already did):

[ ] added some tests for the functionality
[ ] updated the documentation
[X] updated the changelog (please check changelog for instructions)
[ ] reformat files using black (please check Readme for instructions)

Is there actually also one for gitlab?

ah, that's covered by CI https://docs.gitlab.com/ee/ci/variables/predefined_variables.html 👍🏻