[Settings v2] FancyZones: change spinbutton to slider

For consistency with Shortcut Guide:

Also consider https://github.com/microsoft/PowerToys/issues/3234

I just created an issue #3692 covering all the NumberBoxes.

Let's agree on a common label as well.

Let's agree on a common label as well.

NuberBoxes need a logical min and max too.

You could also look into stepping for the NumberBoxes and Sliders, where relevant.

You could also look into stepping for the NumberBoxes and Sliders, where relevant.

Yes, for the Shortcut Guide duration it has already a 50ms stepping and 100ms min value.

See https://github.com/microsoft/PowerToys/issues/3692#issuecomment-633719149 regarding the limitations of sliders. This is apparent in the FancyZones color picker - try using the color picker and slider to set the default value correctly. Maybe it works in 4K? I noticed this when I noticed that the value I had was 1 off the default.
Now, I get it, the color value being off by one is not in itself a big deal, but this just demonstrates why sliders aren't all that. Certainly in cases such as opacity values, where values are multiplicative with other transparent layers and precision is not 'merely' aesthetic but functional, if you're aiming for consistency, then consistently change everything to a textbox, not a slider.

Fixed in 0.19.0, please visit https://github.com/microsoft/PowerToys/releases/ for the latest release

The slider is failing in the exact manner I mentioned.


feat(ecs): add API support for Describe Clusters

Add support for getting capacity providers using Describe Cluster API.
Backend changes to support: spinnaker/spinnaker#5400

@piradeepk as per your comment I've changed the title of this PR and I will also change the title of the commit in the final commit.

Need to get in unit test, so hold on until then.

Closing Test PR


Formula-Cookbook does not contain all available download strategies

Please note we will close your issue without comment if you delete, do not read or do not fill out the issue checklist below and provide ALL the requested information. If you repeatedly fail to use the issue template, we will block you from ever submitting issues to Homebrew again.

[x] are reporting a bug others will be able to reproduce and not asking a question. If you're not sure or want to ask a question do so on our Discourse: https://discourse.brew.sh
[ ] ran a brew command and reproduced the problem with multiple formulae? If it's a problem with a single, official formula (not cask) please file this issue at Homebrew/homebrew-core: https://github.com/Homebrew/homebrew-core/issues/new/choose. If it's a brew cask problem please file this issue at https://github.com/Homebrew/homebrew-cask/issues/new/choose. If it's a tap (e.g. Homebrew/homebrew-php) problem please file this issue at the tap.
[ ] ran brew update and can still reproduce the problem?
[ ] ran brew doctor, fixed all issues and can still reproduce the problem?
[ ] ran brew config and brew doctor and included their output with your issue?

To help us debug your issue please explain:

What you were trying to do (and why)

Documentation does not correctly represent the available download strategies. Also, it does not explain that download strategies can be used for bottle downloads (from #3966).

What happened (include command output)

From (https://github.com/Homebrew/brew/blob/master/docs/Formula-Cookbook.md):
Download strategies offered by Homebrew are:

:using value
download strategy

:bzr
BazaarDownloadStrategy

:curl
CurlDownloadStrategy

:cvs
CVSDownloadStrategy

:git
GitDownloadStrategy

:hg
MercurialDownloadStrategy

:nounzip
NoUnzipCurlDownloadStrategy

:post
CurlPostDownloadStrategy

:svn
SubversionDownloadStrategy

What you expected to happen
The actual strategies from (https://github.com/Homebrew/brew/blob/master/Library/Homebrew/download_strategy.rb):

    when :hg                     then MercurialDownloadStrategy
    when :nounzip                then NoUnzipCurlDownloadStrategy
    when :git                    then GitDownloadStrategy
    when :github_private_repo    then GitHubPrivateRepositoryDownloadStrategy
    when :github_private_release then GitHubPrivateRepositoryReleaseDownloadStrategy
    when :bzr                    then BazaarDownloadStrategy
    when :s3                     then S3DownloadStrategy
    when :scp                    then ScpDownloadStrategy
    when :svn                    then SubversionDownloadStrategy
    when :curl                   then CurlDownloadStrategy
    when :cvs                    then CVSDownloadStrategy
    when :post                   then CurlPostDownloadStrategy
    when :fossil                 then FossilDownloadStrategy

Step-by-step reproduction instructions (by running brew commands)
N/A

The Formula Cookbook is not and is not intended to be a comprehensive document. If you would like to add some additional values we'll happily accept a pull request, thanks.

Thanks, this makes sense. It was just the only documentation I could find on the download strategies without searching the source. I guess tracking down this would have worked as well. Please consider editing Download strategies offered by Homebrew are: to clarify it's not comprehensive.


Compiled template doesn't find steam_api.so

So I compiled the export template for Godot 3.0 on Linux (both 32 bit and 64 bit) and tried to run a game with them. It gives me the following error, although libsteam_api.so is in the same directory as the excutable:

error while loading shared libraries: libsteam_api.so: cannot open shared object file: No such file or directory

I also tried the pre-compiled binaries and they give me the same error. The windows templates (which I cross-compiled on Linux) work just fine, detect the DLL and run the game. But the Linux binaries fail to find libsteam_api.so even though it's clearly there. Any idea what could be causing this behaviour?

Hey there.  Usually when I compile on Linux I have to add the:
LINKFLAGS='-no-pie'
Or it will give me this error when I try to run it.  It is very possible I  did not do this when exporting for the release file; I'll have to double-check.
Also, if you run (I believe) LD on the executable it may say it is looking for libsteam_api.so in your /usr/bin folder.  There is a weird bit where Steam is supposed to copy the libsteam_api.so into that folder when it installs and it not doing so in some cases.  To fix this you can symlink or move a copy of libsteam_api.so to that folder.  DO DOUBLE-CHECK the folder location before doing so; provided that is where it thinks it is supposed to be.  I'm not 100% why the compiled module would need this, but it does make a difference if this is the case.
Those two would be my guess.  When I move back into my Linux environment I'll take a look at the pre-compiled releases and do an LD on it all.  I believe one of those two things is your culprit.

Thanks, I'll see if I can find anything. One thing that may be important is that I can still run an older game (for Godot 2.1) that uses GodotSteam. For that game I compiled the Linux binaries in the same way earlier this year. These binaries still work and actually find the library, the new ones don't.

Okay, so Steam doesn't install the library for me. I just made a fresh install and it's nowhere to be found in /usr/lib
That means I can't rely on it being there when I ship my game, because other user may run into the same problem. I wonder why the 2.1.4 executable finds the library in its own folder and other Steam games work just fine, while the 3.0.2 binary doesn't find the library. Maybe it has something to do with the build configuration?
Anyway, there's probably a workaround for me now. I think I can write a script that adds a local lib directory to LD_LIBRARY_PATH and then execute that game. I tried it and it works from the command line, so I hope it'll also work when executing from Steam.

Hmm.  What did running LDD on your executable say?
I had this error when playing other games on Linux in Steam. I believe a few actually run a .sh file which uses LD_LIBRARY_PATH then executes the game itself, like you are proposing.
Yeah, I'm not sure what in the build configuration would cause this in Godot 3.  Not sure if it is something with the module building process or if it's in the Godot 3 source itself.  I'll dig more and see if I can find anything out.  If you come across something, let me know!

LDD just told me that it couldn't find the library, but didn't tell me where it was looking. But when I made a link to it in /usr/lib/... it worked, so it's clear that it looked for it in the usual places, but not in the current working directory.
I got it working by setting LD_LIBRARY_PATH, so for now I'm happy that the game is starting and the Steam features are working, so the game can be released with this workaround.

Interesting.  Yeah, something must be wonky somewhere.  Glad your work-around is working though!


Aligned drop-down box and input box

Putting Padding and Margins in 'input' equal to zero.{margin:0, padding:0} in 'master.css' have removed this bug
New Feature Submissions or Bug Fixes:

[x] Does your submission pass build tests locally?
[x] Have you indented your code locally prior to submission?

Changes to Core Features:

[x] Have you added an explanation of what your changes do and why you'd like us to include them?

@rapperdinesh  Great!


SPIKE: Research using move ops we run locally to central admin compute resource in cloud

Describe the issue

Lambda vs ECS for calls to GitHub - Platform Admin Utility Related
The team is exploring developing an Admin Utility. Some operations would require interacting with GitHub repos using common git workflows (eg: git pull, git push).
Need to perform discovery around using Lambda vs ECS with GitHub for git operations

Additional context

We're contemplating using Lambda for these operations but not entirely sure about advantages/disadvantages vs using ECS.

Definition of done

Survey what coding APIs and SDKs are available (depends on if we choose py or js)
Put together recommendations/tradeoffs with using Lambda vs ECS for review by the tech team

not relevant


Tooltip: Add ShowOnFocus parameter

Description
Bear with me as this is my first PR for this repository. Please let me know anything I can do better ;-)
fixes #4319
This is about the MudTooltip component.
I have added a "ShowOnFocus" parameter (name is still up for debate... ;-)), which defaults to false and which controls the behaviour of showing/hiding the tooltip with the onfocusin and onfocusout events. When "ShowOnFocus" is false, the tooltip is not shown with those events, when it is true, the tooltip is shown with these events.
I understand if you wouldn't want to have it defaulted to false, as that is actually a breaking change, and I gladly change the PR if you want that... In that case the name should probably be negative in the way that the default would still be false like "NotOnFocus" for instance... not sure... naming is still hard ;-)
But OTOH, from what I have heard "in the field" (read: on the discord), people are not expecting the tooltip to appear on focus, plus it has the nasty side-effect of showing the tooltip when you come back from another tab or application. So IMO the breaking change would be justified.
How Has This Been Tested?
I have updated the tests to accommodate for this change... please review ;-)
Types of changes

[ x ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ x ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

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

No I didn't see that one... thanks to bring it to my attention... I see it's closed now though?
I still think a tooltip should only show "on hover" by default, just what is expected from a tooltip virtually anywhere. Showing it "on focus" or "on click" are special kinds of tooltips, so I would then also treat those options as "special" and make people opt-in to them. I think it will avoid confusion in the long run.

I don't think so, both MUI on react and vuerify on vue uses all events by default. And i think if there is a tooltip, it means it could open whatever we interact with it. If user want to customize, they can disable some events with parameters.

A big name in UX design is the The Nielsen Norman Group,
perhaps some inspiration can be drawn from their Tooltip Guidelines guidelines.

Thanks but i will close this PR, and this type of behavior will be addressed later but across the whole library.


Please add currencyIsoCode to MerchantAccount

The class MerchantAccount does not provide access to the information of the merchant account's assigned currency (although is included in the source NodeWrapper  in node currency-iso-code).
Would be great to have access to this information in order to implement some validation checks in the code. Thanks!

This is now available in 2.53.0 https://github.com/braintree/braintree_java/commit/35bb511711897618386a4d7256e7eece74108849

Thanks a lot. Really appreciated.


WIP: Added modulefiles instructions

Description
Check the output here.

added instructions for modulefiles (I've rephrased the content provided, so please check if everything's ok)
added tabs for Linux/Windows

Targeted for beta06

the output  is also updated

/intelci: run


notes under the VM specs table need a newline character

notes under the VM specs table need a newline character
here https://learn.microsoft.com/en-us/azure/virtual-machines/dv2-dsv2-series-memory#dsv2-series-11-15

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

ID: 908096d8-f3ad-8402-56c0-381e1be906cf
Version Independent ID: aaef6dd2-65c8-9ce6-f3d0-3f1ffa03bb86
Content: Memory optimized Dv2 and Dsv2-series VMs - Azure Virtual Machines
Content Source: articles/virtual-machines/dv2-dsv2-series-memory.md
Service: virtual-machines
Sub-service: sizes
GitHub Login: @andysports8
Microsoft Alias: shuji

@ciufudean
Thanks for your feedback! We will investigate and update as appropriate.

@ciufudean
Thanks for reporting this! We have created a PR for this issue and the changes should go live soon.


[BUG] "no index mapper found for field" warning appears even if there is no knn-vector mapping

What is the bug?
When multiple indices are configured with settings.index.knn=true, warning messages appears when object (or nested) type document  is created.
Oddly, the warning messages appear for the index where the document is NOT created .
When settings.index.knn=false is set, these warning messages do not appear.
This bug only appears for OpenSearch 1.x (I confirmed with 1.3.3 and 1.3.5), not for OpenSearch 2.x (checked 2.2.1).
How can one reproduce the bug?

run Docker

$ docker run -p 9200:9200 -p 9600:9600 -e "discovery.type=single-node"  opensearchproject/opensearch:1.3.5

execute the command below
(1) create index1 with knn index
(2) create index2 with knn index
(3) put a simple object doc to index1

$ curl --location --request PUT 'https://localhost:9200/index1' \
--header 'Content-Type: application/json' \
--data-raw '{
    "settings": {
        "index": {
            "knn": true
        }
    }
}' \
--user 'admin:admin' \
--insecure

$ curl --location --request PUT 'https://localhost:9200/index2' \
--header 'Content-Type: application/json' \
--data-raw '{
    "settings": {
        "index": {
            "knn": true
        }
    }
}' \
--user 'admin:admin' \
--insecure

$ curl --location --request PUT 'https://localhost:9200/index1/_doc/doc1' \
--header 'Content-Type: application/json' \
--data-raw '{
    "parent": {
        "child": "aaa"
    }
}' \
--user 'admin:admin' \
--insecure

See warning

[2022-09-07T08:08:46,096][INFO ][o.o.c.m.MetadataCreateIndexService] [4acb15cf78d5] [index1] creating index, cause [api], templates [], shards [1]/[1]
[2022-09-07T08:08:46,122][INFO ][o.o.c.m.MetadataCreateIndexService] [4acb15cf78d5] [security-auditlog-2022.09.07] creating index, cause [auto(bulk api)], templates [], shards [1]/[1]
[2022-09-07T08:08:46,226][INFO ][o.o.c.m.MetadataMappingService] [4acb15cf78d5] [security-auditlog-2022.09.07/_M92EiDzTZ-wadZ6wQPWgg] create_mapping [_doc]
[2022-09-07T08:08:50,872][INFO ][o.o.c.m.MetadataCreateIndexService] [4acb15cf78d5] [index2] creating index, cause [api], templates [], shards [1]/[1]
[2022-09-07T08:08:55,360][INFO ][o.o.c.m.MetadataMappingService] [4acb15cf78d5] [index1/VHy9DleQRfCZcHA2-Cx12g] create_mapping [_doc]
[2022-09-07T08:08:55,396][INFO ][o.o.c.m.MetadataMappingService] [4acb15cf78d5] [security-auditlog-2022.09.07/_M92EiDzTZ-wadZ6wQPWgg] update_mapping [_doc]
[2022-09-07T08:08:56,189][WARN ][o.o.i.s.IndexShard       ] [4acb15cf78d5] [index2][0] no index mapper found for field: [_id] returning default postings format
[2022-09-07T08:08:56,189][WARN ][o.o.i.s.IndexShard       ] [4acb15cf78d5] [index2][0] no index mapper found for field: [parent.child] returning default postings format
[2022-09-07T08:08:56,189][WARN ][o.o.i.s.IndexShard       ] [4acb15cf78d5] [index2][0] no index mapper found for field: [parent.child.keyword] returning default postings format

What is the expected behavior?
No warning happens
What is your host/environment?

OS: Max OS X 12.5.1
Version: 1.3.5
Plugins: default plugins (including opensearch-knn)

Hi @fiarabbit
Taking a look into this.

I see. I think this was fixed by #319, which went out in 2.0.
From a high level, there is a way to configure posting formats per field through the PerFieldPostingsFormatCodec. This is where the warning is coming from.
This will get created and registered in the CodecService constructor.
For k-NN, we call this CodecService constructor here: https://github.com/opensearch-project/k-NN/blob/1.3/src/main/java/org/opensearch/knn/plugin/KNNCodecService.java#L19. MapperService should be null so the perfieldpostingformat should never get created.
However, the default constructor for the CodecService is called before our KNNCodecService is called here. Here the MappingService wont be null. As a hack, we get the per field posting format from the default codec and set it for our codec here.
When we look at the mapperservice, however, because no mapping has been specified already, the mapper in the MapperService is null, resulting in this condition to be false in PerFieldPostingsFormatCodec.
I believe if mapping if specified at index time, this error should not be reached in 1.3.
Aside from the warning, is this causing any error in your application?

@jmazanec15
Thanks for your polite reply!

I believe if mapping if specified at index time, this error should not be reached in 1.3.
Actually, even if I specify the mapping at index time, this warning message still appears.

code to reproduce
send sequentially the following requests.
PUT /index1
{
    "settings": {
        "index": {
            "knn": true
        }
    },
    "mappings": {
        "properties": {
            "parent1": {
                "type": "nested",
                "properties": {
                    "child1": {"type": "keyword"}
                }
            }
        }
    }
}

PUT /index2
{
    "settings": {
        "index": {
            "knn": true
        }
    },
    "mappings": {
        "properties": {
            "parent2": {
                "type": "nested",
                "properties": {
                    "child2": {"type": "keyword"}
                }
            }
        }
    }
}

PUT /index1/_doc/doc1
{
    "parent1": {
        "child1": "aaa"
    }
}

Log
[2022-09-16T05:38:31,949][INFO ][o.o.c.m.MetadataCreateIndexService] [fda10f447ef4] [index1] creating index, cause [api], templates [], shards [1]/[1]
[2022-09-16T05:38:36,981][INFO ][o.o.c.m.MetadataCreateIndexService] [fda10f447ef4] [index2] creating index, cause [api], templates [], shards [1]/[1]
[2022-09-16T05:38:43,130][WARN ][o.o.i.s.IndexShard       ] [fda10f447ef4] [index2][0] no index mapper found for field: [parent1.child1] returning default postings format

Aside from the warning, is this causing any error in your application?
No, I haven't found any data inconsistency in our actual application.
I raised this ticket because I cannot judge this is an actual problem, or I can just ignore these messages.

Got it. Yes this should not cause a problem from data consistency standpoint. It should be fixed in 2.x releases.


solves issue no #1821

Why:

Closes #1821
What's being changed: Added the file FindNegativeOfANumber.kt

You are pushing changes from the master branch. Please create a separated branch and push your changes! You can refer steps in CONTRIBUTING.md - make changes.
Thank you!


Develop multi language translator using streamlit and gettext

Article checklist
Pre-submission checks

Please ensure that you have only one open topic suggestion (issue) + in-review article (linked pull request) at a time. This will ensure that we complete the review process in a timely manner from inception to publishing

If you have not already, please go over our Resources Page for more tips and tricks on how to improve your overall technical writing so reviews are swifter, increase the chance of a payout and provide more value to our readers.

For first-time contributors and for more details on our submission guidelines, see our Contributing Guidelines.

Formatting and structure checks

[x] Have you used the correct folder and file structure? - see Contribution Guidelines

[x] Is your article properly formatted in Markdown? - see Github Markdown guide

[x] Have you used our preferred heading structure? - H3 (###) and up

[x] Does your article follow any of the suggested structure formats? - see example formats

[x] Is your article over 750 words? - Tool: Wordcounter.net

[x] Is the article you are submitting an in-depth and unique article? Does it go beyond what is in the official docs and what is covered in other blog sites. See these articles as examples.

[x] Does your article provide enough value and detail about your topic? - Articles should be clear, accurate, and fully explained.

[x] Can your article be understood by beginners? - Assume the audience is smart but has no prior exposure to the common terminology in your article.

[x] Have you included a hero image that is 600x400 pixels and have the copyright to use it? - Tools: Pixlr Image Editor for resizing and Unsplash for Creative Commons images

Grammar & plagiarism checks

[x] Have you spellchecked and grammar checked your article? - Tools: Write&Improve - ProWritingAid - Grammarly
[x] Please place your article through a 3rd party plagiarism checker. We suggest using Quetext, this tool is free and has a daily limit. We typically accept articles with 10% or less.
[x] Have you checked your article for readability? - Tool: Hemmingway
[x] Have you added sources for quotes and images that aren't yours?

Technical checks

[x] Are your code snippets properly formatted for syntax highlighting - see Syntax guide
[x] Have you checked your code runs correctly and you've highlighted all necessary dependencies for installation?
[x] Are the software programs and packages you're highlighting in your article up to date, using current versions and not deprecated?

Topic suggestion this closes

Remove the backticks and add the issue number below to link and close your Topic Suggestion (issue) when your article has been published (PR has been merged). See this video for more details.

`This closes #4278

Finally, delete the article checklist notes in blockquotes and submit your PR. We look forward to reviewing your article.

Tech review

Most of the code are similar to the above blog
New Scan 11_09 AM Copyleaks Report.pdf

@johnniembugua always ensure that your articles files are inside the content/articles folder rather than in the main repo directory.

jumping on this one

Please be sure to go through any edits made to improve future articles and decrease review time


Only create review invitation when the review stage is set.

@celestemartinez You can merge this one whenever you are done reviewing.


Only run pod install if CocoaPods v1.0.0 or greater is installed.

Avoid issues with older versions of CocoaPods breaking the build. Users who genuinely use older versions of CocoaPods will have to run pod install manually when required.

Awesome - thanks Jakob! LGTM
Travis is complaining about one additional call:
  Receiver: Instance of 'IOSWorkflowTestTarget'
  Tried calling: _cocoaPodsInstalledAndMeetsVersionCheck
  dart:core                                               Object.noSuchMethod
  package:flutter_tools/src/ios/ios_workflow.dart 181:11  IOSWorkflow.validate


Deprecate custom status reason phrases

This was a silly thing to have a CVE on.  For precedent, servlets haven't let you set them since nineteen dickety-two, and Finagle also prohibits them.
Also removes the ability to set isEntityAllowed on a custom status.  I'd rather leave that decision to the IANA.

Failures are https://github.com/http4s/http4s/pull/5258


sqlite3_column_type SQLITE_TEXT returns string not []byte

Return as string as opposed to []byte or []uint8 arrays.
This brings sqlite closer in line with other dbs like postgres, allowing downstream consumers to assume the scanned value is string across underlying dbs.
This implementation is a first draft. If it's good as is, great, if not, let's start a dialog on the issue and figure out how best to resolve it.
This change is related to https://github.com/jinzhu/gorm/issues/2276 and https://github.com/mattn/go-sqlite3/issues/606.

What is the status of this? Would be nice to get it pulled in. Agree with @dimroc and @rittneje.

Nothing will be change since the column type in query may be concating or calculating with another type. What query did you run? In SQLite, you have to make the type strictly. If you want to change default behavior, please add struct field in SQLiteDriver to make opt-in. And modify the flag in ConnectHook. A way to hook connection:
https://github.com/mattn/go-sqlite3/blob/3fa1c550ffa69b74dc4bfd5109b4e218f32c87cf/_example/hook/hook.go#L16

@mattn Here is a simple example that demonstrates the problem and the fix.
var x interface{}
if err := db.QueryRow("SELECT 'hello'").Scan(&x); err != nil {
	panic(err)
}
fmt.Printf("%T", x)

With the current version of this library, it outputs []uint8 (i.e., []byte). With the fix, it outputs string. This is because sqlite3_column_type gives back the actual type of the value in all circumstances (SQLITE_TEXT in this case). However, sqlite3_column_decltype will only tell you the declared type of the column, and therefore: (1) is generally meaningless, because SQLite doesn't enforce it at all; and (2) gives back nothing when the column in question was not directly selected from a table column.

If you want to change default behavior, please add struct field in SQLiteDriver to make opt-in.

Again, I consider the present behavior to be a bug. I don't know why someone would run a query like the above and expect it to come out as a []byte. The Go 1 compatibility guidelines do allow for breaking changes for the sake of fixing bugs. As the existing behavior is surprising and unexpected, forcing people to opt into the correct behavior seems counterproductive. If anything, we should force clients to opt into the legacy behavior if it's really necessary.

I don't say I will not change this. Just I didn't understand why current implementation did not work until now. Now, I had  understood why didn't. So I have to add implementation for conversion for sqlite3_column_type except SQLITE_TEXT ?

addressed in https://github.com/mattn/go-sqlite3/pull/692 with test.

Closed by #692


TexasInstruments.TI-Nspire.CXStudent version <IP_ADDRESS>

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

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

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

Microsoft Reviewers: Open in CodeFlow

Only one package per PR.
.validation file needs to stay.

Just reverted these changes. I've been a bit struggling after noticing that I had put "TexasInstrumentsInc" instead of "TexasInstruments". It should be okay now.


fixed typo in for osfamily fact redhat vs RedHat in rsyslog_default.erb

I believe the correct osfamily value is RedHat, not redhat.  This fixes /var/log/messages entries such as:
2015-03-02T22:33:23.712485-05:00 www001 rsyslogd: [origin software="rsyslogd" swVersion="5.8.10" x-pid="378" x-info="http://www.rsyslog.com"] exiting on signal 15.
2015-03-02T22:33:23.827193-05:00 www001 kernel: imklog 5.8.10, log source = /proc/kmsg started.
2015-03-02T22:33:23.827280-05:00 www001 rsyslogd: [origin software="rsyslogd" swVersion="5.8.10" x-pid="879" x-info="http://www.rsyslog.com"] start
2015-03-02T22:33:23.823064-05:00 www001 rsyslogd: WARNING: rsyslogd is running in compatibility mode. Automatically generated config directives may interfer with your rsyslog.conf settings. We suggest upgrading your config and adding -c5 as the first rsyslogd option.
2015-03-02T22:33:23.826979-05:00 www001 rsyslogd: Warning: backward compatibility layer added to following directive to rsyslog.conf: ModLoad immark
2015-03-02T22:33:23.827071-05:00 www001 rsyslogd: Warning: backward compatibility layer added to following directive to rsyslog.conf: MarkMessagePeriod 1200
2015-03-02T22:33:23.827078-05:00 www001 rsyslogd: Warning: backward compatibility layer added to following directive to rsyslog.conf: ModLoad imuxsock

Hi Saz, Do you think this can be merged and would it be possible to cut a new version for the puppetlabs forge?
Thanks@

I've submitted a similar pull request except I perform a case insensitive match against puppet fact @osfamily.

Fixed with the merge of #149


[Bullet] Issue with moving KinematicBody collisions (regression in 3.4 beta4 from 3.3.3)

Godot version
3.4 beta 4
System information
Windows 10, GLES3
Issue description
Two 3D kinematics bodies going towards each other move very fast instead of standing still, see attached video.
It is working correctly in 3.4 beta 3, broken in 3.4 beta 4.
Steps to reproduce
Have a player kinematic body
Have an enemy kinematic body going towards the player
Make the player push the enemy at a slight angle
Minimal reproduction project
PhysicsIssue3.4.4.zip
Video
https://user-images.githubusercontent.com/522725/130333649-fdb2f697-0402-40b6-83ad-659b0c9fbf74.mp4

cc @fabriceci @pouleyKetchoupp

Sorry for the confusion, the regression occurs between 3.4 beta 3 and 3.4 beta 4.

I know what's happens here, I will fix that at the end of the week, need to discuss to see what is the best fix.

@Yogoda Thanks for the clear minimal project and description. Should be now fixed.


Add rename functionality to Copy-SqlLogin and add new function Update-SQlLoginName

Fixes #410
Tested on windows 10, powershell 5 and SQL Server 2008R2
Changes proposed in this pull request:

Adds a hashtable parameter to the copy-sqllogin function to rename multiple users.
Adds Update-SqlLoginName function that is called in copy-sqlloging, and for user as stand alone

How to test this code:

[ ] run copy-sqllogin with rename for windows ( where SID matches ) and an SQL Server login
[ ] run Update-SQlLoginName to change a user on a server

Has been tested on minimum requirements:

[ ]  Powershell 3
[ ]  Windows 7
[ ]  SQL Server 2000

Has been tested on maximum requirements:

[ ]  SQL Server vNext
[ ]  Windows 10
[ ]  Azure Database

Tests for tester:

[ ] Working/useful help content, including link to command on dbatools web site
[ ] All examples work as advertised
[ ] Does not contain template content
[ ] Does not contain excessive/unnecessary amounts of comments
[ ] Works remotely
[ ] Works locally
[ ] Works on lower versions or throws error specifying version not supported
[ ] Works with named instances
[ ] Works with clustered instances
[ ] Handles offline/read only databases
[ ] Supports multiple servers (at the command line or piped from Get-SqlRegisteredServerName)
[ ] No un-handled errors which stop the command working with multiple servers

Done

Updated based on @ctrlbold suggestions.

Renamed new function to Rename-DbaLogin
Support for -WhatIf Switch parameter
Renamed Parameters to be more verbose and accurate.
Added/Updated examples.

It looks likely this should be approved in next release closer to 2/1 since it will take a bit more testing since it's somewhat complex. Thanks so much for all of the effort 👍

Hi @mshamann  I have some questions here.
I have done some tests but I don't have sure if I agree with the result.
Let me try to explain:
Rename is done after copy, which means if already have a login with the original name in the destination instance this will not be copy unless we specify the -Force . This could bring some issues, if I use -Force and use the new parameter to rename from 'Test' to 'TestRenamed' it will run without error but it does what you, as user, are expecting? Not for me.
1 - User 'Test' user was dropped only user 'TestRenamed' remains
2 - If I try to copy user Test so I can keep both will give an error because SID is already in use (SID is in use by TestRenamed)
3 - If I try to use the -Force parameter will do nothing because user with name Test does not exists.
In this scenario how can I have both users?
Should we, in this scenario, create a new user (that will generate new SID)  and give it the new name?
Of course, this could happen if we use actual Copy-SqlLogin and rename it after, then go back and try to copy it again. The SID error will occurs.
We want to address this? Or am I complicating? 😳
Thoughts? @ctrlbold @ConstantineK @PowerDBAKlaas @wsmelton @niphlod

@ClaudioESSilva the current behaviour is indeed not correct.
The solution is not so easy, since there may be different scenario's.
Creating the new user seems mostly the right choice.
When we're starting with one login on origin server, and it becomes two users on destination server, only one can have the SID. Does a user want the login with the same name to have it or the login that is a copy from the origin? I don't know that.

@ClaudioESSilva @PowerDBAKlaas
I guess when I wrote it the idea of ending up with two logins at the end had not occurred to me. Which is why I wrote it this way, pull down the user the same way and then simply rename the login. Because in my head a rename parameter is simply that. Rename the object you specify, not to duplicate and leave the old one.
To end up with two logins might be easy for the SQL Login, but not so for the Windows, given the SID problem.
So we would have to Create a new login in the destination, then sync the permissions from original login to the new login.
I am not sure how we would handle the Windows side of things in this case.

That seems like an edge case that we can support in the next release. Approving for now 👍

Nice work @SirCaptainMitch! <3


Integration and Endpoint tests setup

Project to be added to solution with examples of integration and endpoint tests for further development

Done


Add an option to use more compact ascii-icons

It would be nice to have an option to use small LCD 20x4 (or even 16x4) to show weather icons.
Here's the basic idea:
1........0.........0
   --\
   --/    Unknown
   \ .

   \ /    Prague
  = O =   Sunny 35 C
   / \    0mm

  _/`` /  Paris
(__)  o - Partly Cloudy
     / \  20 C, 0mm

  _/``\   Moscow
(__)__.)  Cloudy 19 C
          0mm

  _/``\   London
(__)__.)  Light Showers
, ‘ , ‘   18 C, 8mm

  _/``\   Dublin
(__)__.)  Heavy Showers
,‘,‘,‘,‘  16 C, 22mm

  _/``\   New York
(__)__.)  Light Snow Showers
* * * *   3 C, 3mm

  _/``\   Ottawa
(__)__.)  Heavy Snow Showers
**_**_**  2 C, 13mm

  _/``\   Bratislava
(__)__.)  Light Rain 11 C
  / / /   5mm

  _/``\   Tegucigalpa
(__)__.)  Heavy Rain 9 C
//_//_//  60mm

  _/``\   ...
(__)__.)  Light Sleet
* / * /

-_-_-_-   ...
- - - -   Fog

@jose1711 Jose, hi
Very cool icons! Thank you. We will integrate them. Do you want to have them only for the 0-days forecast first?

@jose1711 Jose, thank for the icons. Maybe you could give a short technical description
of your project, for them who want to recreate it on their own? And than we could add it
(together with the icons) the contrib/ or doc/ directory and add a link to README.md?
Would it be possible?


Prevent crash when call set_text() on a removed treeItem

Fixes #85650
Also, should I use add more description or just use ERR_FAIL_NULL is sufficent ?

This should be before updating the text IMO, in the relevant methods calling these methods
Unless we can confirm it's valid and doesn't have strange sideeffects to update the text off the tree

This should be before updating the text IMO, in the relevant methods calling these methods

Yes but there are too many places that might call _changed_notify, since  the previous assumption tree items were always in a tree

Yes but there are too many places that might call

I don't think that's a valid argument against adding it there, but if you confirm that it's safe and doesn't cause any side effects to manipulate the text etc. not on the tree we can add it just to the places where tree is accessed, otherwise I'd say we shouldn't allow the manipulation if it's not safe

we'd need to verify this before we make this change

Indeed, you are right. Does anyone familiar with this part of code and can give me some suggestions?

A basic test would be to try your own code and see if there's any noticeable difference, if there isn't studying the code in more detail should help

I'll trust that the manipulation of these is safe off the tree, despite the aspects mentioned above 🙂, but won't approve myself as I can't confirm it

Thanks!


Minor refactor on core crate

Hold on, wait for #25 first due to potential removal of Value.

Closes #25.


Properly convert to microseconds calculation when deserializing token

We started noticing 401s in an application that uses this crate, and when digging into the code I noticed the logic changes for expiry calculation in https://github.com/cboudereau/gcs-rsync/commit/5e3df56c7a25f49f5f6e14dd40f9953c6eb59441

Thank you @ianks, the fix has been published and available here: https://github.com/cboudereau/gcs-rsync/releases/tag/v0.4.2


Move all TrueNAS behind /truenas

Codecov Report

Merging #24 (968f157) into main (e2ec08b) will increase coverage by 5.93%.
The diff coverage is 89.52%.

@@            Coverage Diff             @@
##             main      #24      +/-   ##
==========================================
+ Coverage   66.37%   72.30%   +5.93%
==========================================
  Files           2        3       +1
  Lines         113      130      +17
==========================================
+ Hits           75       94      +19
+ Misses         35       33       -2
  Partials        3        3

Flag
Coverage Δ

unittests
72.30% <89.52%> (+5.93%)
:arrow_up:

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

Impacted Files
Coverage Δ

src/api-proxy/main.go
0.00% <ø> (ø)

src/api-proxy/proxy.go
81.08% <81.08%> (ø)

src/api-proxy/middlewares.go
94.11% <94.11%> (ø)

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


Provide way to configure port that Envoy listens on

Right now envoy is locked to 8080 and contour is 8000. This is fine with pod networking but may not be great with host networking.

Hi @davecheney , I would like to work on this issue if you can guide me through this. I am completely new here.

Duplicate of #48

@dungeonmaster18 thanks for your offer of help. The first step should be to write up either in the issue (#48) a high level design of what needs to change inside contour to implement this feature, and how you plan to test these changes. Once there is agreement that the approach is reasonable, it's time to code.


"Go back" button doesn't work if Ctrl+End or Ctrl+Shift+End pressed

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

VS Code Version: 1.89.1
OS Version: Windows_NT x64 10.0.14393

Steps to Reproduce:

Open VS Code with cursor at the beginning of a long file with a several screen height in the editor
Press Ctrl+End or Ctrl+Shift+End
Cursor moves to the end of file with or without a selection
Try to use "Go back" button or mouse additional button with "back" function
Expected behaviour:
"Go back button" is highlighted, "go back" function returns the cursor to the beginning of the file
Actual result:
"Go back button" remains gray, "go back" function doesn't work.
If there were some actions before, e.g. other files was browsed, it return to the another file. And it is annoying.

Notes:

Ctrl+Home/Ctrl+Shift+Home works fine for "Go back", if the cursor was placed in the end of file.
If the cursor position isn't the first on opening, Ctrl+End works fine for "Go back", Ctrl+Shift+End doesn't work

We require to be 10 lines apart for a new history entry to be recorded:
https://github.com/microsoft/vscode/blob/50f2b2eca8d5c6e19ec95f9d46a59f623e60cac3/src/vs/workbench/browser/parts/editor/textEditor.ts#L352
Here to address this we would need to also consider column distance.

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

We require to be 10 lines apart for a new history entry to be recorded:
https://github.com/microsoft/vscode/blob/50f2b2eca8d5c6e19ec95f9d46a59f623e60cac3/src/vs/workbench/browser/parts/editor/textEditor.ts#L352
Here to address this we would need to also consider column distance.

The problem is valid for 200 and for 1200 lines files.
Ctrl+End moves to the end of file. So the next cursor position is in hundreds and thousands lines apart form the previous position.
Also Ctrl+Home works for the same distance.

Are you aware of the Cursor Undo (Ctrl/Cmd + u) feature?

Are you aware of the Cursor Undo (Ctrl/Cmd + u) feature?
Thank you, but it's not the same. E.g. if I moved to end of file from the beginning and change cursor position several times (for example press arrows keys), I need to press Ctrl+U exact the same times. While I need only once press "back" or mouse side button to return to the file beginning.

I like the "Back" function. Only the thing I need - to move back/forward inside a file without switching to another file. But in another scenarios I need switching files.

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


add bytes/byterray title

Codecov Report

Merging #931 into master will increase coverage by 0.02%.
The diff coverage is 85%.

@@            Coverage Diff             @@
##           master     #931      +/-   ##
==========================================
+ Coverage   64.76%   64.78%   +0.02%
==========================================
  Files          89       89
  Lines       15861    15881      +20
  Branches     3582     3587       +5
==========================================
+ Hits        10272    10289      +17
  Misses       3232     3232
- Partials     2357     2360       +3

Impacted Files
Coverage Δ

vm/src/obj/objbytes.rs
77.48% <100%> (+0.29%)
:arrow_up:

vm/src/obj/objbytearray.rs
79.28% <100%> (+0.25%)
:arrow_up:

vm/src/obj/objbyteinner.rs
75.36% <78.57%> (+0.06%)
:arrow_up:

Continue to review full report at Codecov.

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


add coinhub wallet

Hey, thanks for the PR!
The website says the wallet is open-source, but I cannot find the source code anywhere on the linked GitHub: https://github.com/CoinhubOfficial.

Sorry to make you have doubts, we will open source it in the future, but it is still not open source.

Hey, thanks for the PR!
The website says the wallet is open-source, but I cannot find the source code anywhere on the linked GitHub: https://github.com/CoinhubOfficial.

Sorry to make you have doubts, we will open source it in the future, but it is still not open source.


Nano Feature Highlighting / Selecting Multiple Lines

This feature per my comprehension is missing from micro editor
In a nano text editor, we can copy / cut block of lines using the following steps:
Set the target by moving the cursor to the beginning of the text and pressing CTRL + 6.
Using the arrow keys, highlight the text you want to copy.
To copy, hold ALT + 6.
To cut / delete, hold CTRL + K.

@N1vBruno thnx


Make all generators deterministic, refactor package

Similar to Avro, make all other Gen[T] deterministic.

Is the latter half of this title addressed by #35 ?

Yep

Fixed by #37


system/system: add config SYSTEM_SYSTEM_DUMPINFO

Summary
In some cases, applicationes use system() to excute the nsh command, but it's hard to find out who used the system function to execute the nsh command, so  add this config to print the call backtrace and the command content.
Impact
system command
Testing
sim

[Experimental Bot, please feedback here]
No, the PR summary does not meet the NuttX requirements.
Here's why:

Missing Details in Summary:

Why is the change necessary? The summary mentions difficulty in debugging, but doesn't explain why this is a problem that needs solving. What specific issues does the lack of backtrace information cause?
What functional part of the code is changed? Be specific. Mention the files/modules where you added the config option and backtrace logic.
How does the change work? Briefly describe the mechanism used to print the backtrace (e.g., using a new configuration option).

Incomplete Impact Assessment:

Impact on User: Will users need to enable this config option? Will they see the backtrace output by default?
Impact on Build:  Does this config option change the build process in any way (e.g., new dependencies)?
Other Impacts: The other sections (documentation, security, compatibility) are not addressed at all.

Insufficient Testing Information:

Build Host(s): Provide details about your build environment (OS, CPU architecture, compiler version).
Target(s): Be specific about the architecture and board configuration you used for testing.
Testing Logs: The placeholders for testing logs need to be replaced with actual output. Show the difference in behavior before and after your changes.

To improve the PR:

Expand the Summary: Provide a clear and concise explanation of the problem, the solution, and how it works.
Complete the Impact Assessment:  Address all the impact points, even if the answer is "NO" (in which case, briefly explain why).
Provide Detailed Testing Information:  Give specific details about your testing environment and include relevant logs demonstrating the functionality.

Remember: A well-written PR makes it easier for maintainers to understand and review your changes, increasing the chances of your contribution being accepted.

let's ignore macOS ci temp break.


Implementation of the CIFAR10 autoencoder

Hi, we didn't try our old model but we can get a even better result with the following structure:
input -> 3x3x32 conv -> BN -> Relu -> 3x3x32 conv -> Relu ->BN -> 3x3x3 conv -> output
BTW, since we don't have our old classifier, we used a DenseNet classifier with acc ~92%.

Hope this helps.

Originally posted by @Trevillie in https://github.com/Trevillie/MagNet/issues/2#issuecomment-358668217
In your response to issue #2,  you suggest using an even better structure: input -> 3x3x32 conv -> BN -> Relu -> 3x3x32 conv -> Relu ->BN -> 3x3x3 conv -> output.
But when I implement this structure for CIFAR10 Detector or Reformer, I found it happens to errors because it cannot make the output in the range of [0, 1].
Is that structure really work in your experiments? how do you do to make it work in your experiments?

@kleincup Yes I'm sure that worked. It's been a while but if I remembered correctly, setting your last conv layer to sigmoid will do the trick.

Thank you very much for your reply!
But talking back to your previous network structure in your original paper, it uses sigmoid as the activation function actually.

I have two questions:

why the last layer of this structure is 331? the output channel should be 1?
Since you have suggested:

Try a wider/deeper autoencoder structure.
Originally posted by @Trevillie in https://github.com/Trevillie/MagNet/issues/2#issuecomment-352253183

It the following structure setting make sense to be a detector/reformer like your original structure in your paper
Conv.Sigmoid 3 X 3 X 32 -> Conv.Sigmoid 3 X 3 X 32 -> Conv.Sigmoid 3 X 3 X 3

That should be a typo.
It makes sense and might work, but there's no guarantee for that. Please consider the structure I mentioned in the issue.

That should be a typo.
It makes sense and might work, but there's no guarantee for that. Please consider the structure I mentioned in the issue.

Thank you for the response!
Considering the structure you mentioned in the issue
input -> 3x3x32 conv -> BN -> Relu -> 3x3x32 conv -> Relu ->BN -> 3x3x3 conv -> Sigmoid -> output
What makes me a little confusing is that:

When trying a network structure for detector or reformer, does the activation (sigmoid or Relu)  have a significant effect on the result?
In many well-known structures,  it is common and easy to see a sub-structure conv -> BN -> Relu (like ResNet ). In the first part of your structure, you use that sub-structure, but in the next part, why do you use conv -> Relu -> BN (change the order)?

I am sorry to bother you, but I still hope that you can help me~
Thank you very much!

It sure has influences, but tbh I can't say for sure.
Whether to use BN before or after activation is arguable. See https://www.reddit.com/r/MachineLearning/comments/67gonq/d_batch_normalization_before_or_after_relu/. I guess (as it's been a long time since I did those experiments) I was just experimenting some different structures for fun and that one happens to work. It's not a deliberate design choice.

But in my experiment,  the structure input -> 3x3x32 conv -> BN -> Relu -> 3x3x32 conv -> Relu ->BN -> 3x3x3 conv -> Sigmoid -> output isn't performed well (worst than your results).
The weirdest thing is that when I change the ReLU to Sigmoid, the performance will be better!

Hmm... Not sure why that happens... That's what I used and the other guy in the issue seemed to have no problem reproducing.

No further discussion. Close.


RuntimeError: Tensors must have same number of dimensions: got 2 and 3

@likethesky @Celebio @colesbury @pdollar @minqi
Hi, I have the error at this line when I use multiple points in an image to extract the segments.
My input points and labels used
points = tensor([[ 711,  455],         [1578,  611],         [2019,  640],         [ 412,  739],         [1820,  810]], device='cuda:0')
label = tensor([1, 1, 1, 1, 1], device='cuda:0')
for prediction:
masks, _, _ = predictor.predict_torch(         point_coords=points,         point_labels=label,         multimask_output=False,     )
https://github.com/facebookresearch/segment-anything/blob/6fdee8f2727f4506cfbbe553e23b895e27956588/segment_anything/modeling/prompt_encoder.py#L84
Could anyone help me understand, where am I going wrong here?
Thanks in advance

I think the problem is due to the predict_torch(...) function expecting there to be a batch dimension.
One way to fix the problem is to add in the extra dimension that the function expects. You can do this using the unsqueeze(...) method on the tensors:
x = 0 # or 1, if the 5 points/labels are meant to be separate prompts
points = tensor([[ 711,  455], [1578,  611], [2019,  640], [ 412,  739], [1820,  810]], device='cuda:0').unsqueeze(x)
label = tensor([1, 1, 1, 1, 1], device='cuda:0').unsqueeze(x)

If you use unsqueeze(1) then that is interpreted as meaning that the 5 points/labels are meant to generate 5 separate masks (and you'll get 5 masks as the output). Otherwise, if you use unsqueeze(0), then the 5 points/labels are all interpreted as belonging to a single segmentation prompt, and you'll get 1 mask as an output.
Another solution, if you do mean to only get 1 mask, is to use the predict(...) function (i.e. not the _torch variant), which may be simpler (although it's going to do some extra pre-processing that you may not need if you already did that with your existing point coordinates). It would look something like:
import numpy as np # Need this to make np.array inputs!
points = [[ 711,  455], [1578,  611], [2019,  640], [ 412,  739], [1820,  810]]
label = [1, 1, 1, 1, 1]
masks, _, _ = predictor.predict(
	point_coords = np.array(points),
	point_labels = np.array(label),
	multimask_output=False)

The error you're getting is likely due to not having the right shape for the input points & labels. The shape of the inputs when using the predict_torch function should be:
point_coords = B x N x 2
point_labels = B x N
such as:
import numpy as np # Need this to make np.array inputs!
points = torch.tensor([[[ 711,  455], [1578,  611], [2019,  640], [ 412,  739], [1820,  810]]])
label = torch.tensor([[1, 1, 1, 1, 1]])


Is this no longer published???

libraryDependencies += "com.bryangilbert" %% "slack-scala-client" % "0.1.0"

Is not working

The README mentions you need to do a publishLocal first to make the jar available on your machine. :+1: though, it'd be much better if this was properly published.

HI! I'll try and get this project published in short order.....been bit short on time since I put this out.

I've uploaded the jars to sonatype and it should be available via maven central soon. One thing to note is that I've changed the goupId slightly (readme have been updated to reflect this). You should be able to import the project soon using:
libraryDependencies += "com.github.gilbertw1" %% "slack-scala-client" % "0.1.0"


Widen return types for greater compatibility

This is especially useful for cross-lib-interoperability, e.g. with our SearchBundle which has the SearchableEntityInterface with their getLastModificationTime method but with the return type being \DateTimeInterface.

Please change the PHPDoc types as well.
Sent with GitHawk

Oh my bad. I had them changed in a working copy which was in a different project. :)


Fix faulty merge-conflict resolution -- solves server error

In one of tonight's quick rebases I made a faulty merge resolution, fixed here.

Self-merging this trivial fix.


Updated readme, and moved configs to examples

This commit does the following:

Moves ./configs to ./examples it's easier to understand what they are now.
It adds ./examples/readme.md this should be extended over time, to explain the examples.
Updates the root readme.md
Adds a CODE_OF_CONDUCT.md
Adds a CONTRIBUTING.md document, for use in github. This is a starting point, we will likely add to it over time. (We should update the github project to point here).

The example stacks, have had a header comment added. It would be really good to complete those comments so they can be used. In addition an explanation of what the plugin is doing would be helpful.

Can we dupe README.md and CONTRIBUTING.md to /docs and update all references to configs/ in the documentation please?

I would love to see a tagline more targeting users.  A hook.  Something like:

Why you need Kombustion
Turn bloated cloudformation copy/paste jobs into elegant, concise yaml.

or

Standardise your infrastructure and eliminate novice user errors.

And then a specific section addressing the cynical ones amongst us that feel like we've seen this cfn parser stuff before:

Why not aws cloudformation deploy  ?

because kombustion waits till the job is done and prints out events in the stack creation process so that you never have to enter the console and your CI tools report errors.

Why not Serverless Application Model (SAM)?
SAM is great for a very specific subset of AWS projects however it is not extensible and doesn't even completely support the resources it does implement (namely Dynamodb).  (Link to kombustion implementation of SAM (that I will do) )

Why not serverless
Serverless uses a new template format to recreate cloudformation.  Why learn another template language and read a new set of docs? Kombustion works with Vannilla cloudformation so you can leverage your existing knowledge to get started right away and migrate jobs you need to standardise/protect or simplify incrementally.

Why not Terraform
...

I think we should leave anything Go related for the DEVELOPMENT.md.  IMHO we should target the novice user in the first README.md and aim to motivate and inspire them to try the quickstart.
The Quickstart
I think this is great.  Super concise.  Not intimidating at all.  Novice user can easily see the big picture.  What it is missing is the segue and a message.  Something like:

Now turn this into a plugin! (link to Ians guide on writing your first plugin)

Without that the reader is just thinking. "Umm ... so this is just aws cloudformation deploy"


Open PowerPoint using OSCPoint

Hi Nick,
I can't get OSCPoint to open a file - I can select it, but open file only works if PP is already running.
Then I can scroll through te files and open the file I want.
Don't know if languagesettings could have anything to do with it (Dutch)
I am testing on Companion 3.4 Beta with OSCPoint <IP_ADDRESS>
I did check the box for file-access
Is there an option to start PP directly from a button?
Originally posted by @helex1 in https://github.com/phuvf/oscpoint/issues/41#issuecomment-2165748580

OSCPoint runs as a PowerPoint add-in. This makes it easy to tie in with the PowerPoint application - for example when adding a tab to the PPT ribbon.
However, this means it's not possible to launch PowerPoint from OSCPoint, as OSCPoint will also not be running. This kind of remote control is heading towards RDP or TeamViewer territory - although I can see why it might be useful in some cases.
I don't see this being added to OSCPoint any time soon.


Addressing #445, #455, #456, & #530.

Multiple fixes to issue.

This PR looks odd - I think you need to rebase on master and the extra commits should go away.

I tried rebasing, but I'm not sure it helped.

I see the problem - you have merge commits in your master - one on Oct 22 (5aee6c7) and another on Apr 22 (60ff7a5) - if you get rid of those, this will merge cleanly without any extra work on my part.
git rebase -i is useful in this situation, but if there is just this one commit you want in the PR, this might be easier:
# Starting from a clean slate
git clone https://github.com/srdubya/PSReadLine.git psreadline_tmp
cd psreadline_tmp
git remote add upstream https://github.com/lzybkr/PSReadLine.git
git fetch upstream
git checkout upstream/master
git checkout -b fix_history
git cherry-pick 9c56805
git branch -d master
git checkout -b master

Once you've verified everything is correct, you can run git push -f origin - this will rewrite your history and it will match upstream w/ your one commit.

OK, I followed your "clean slate" process, thank you.  How does it look now?

Much better. In the future, try to avoid merges in master - an easy way to do that is to always create a branch for any fix/feature work.


Link to relevant RFC for HKTs

This PR updates the link regarding HKTs in Rust to the relevant RFC.

thanks!


Add an experimental flag to block samples with timestamp too far in the future

Address: https://github.com/thanos-io/thanos/issues/6167

[ ] I added CHANGELOG entry for this change.
[ ] Change is not relevant to the end user.

Changes

Verification

You could block the samples here probably!


quiver should not run on port 80

Like many server owners, I already run a HTTP server on my box. If I want it to double as a Quiver server, I'll need to hack the install script to run Quiver on some other port. I think Quiver should run on a different port.

@dborkan FYI, I missed this reviewing your change the other day!


Fix whitespace for Make 4.3

Closing this, as I don't see too much value in it. Unless it fixes a specific issue?


Resilence directory is logged regardless if machida is built with it on

Is there a way to make sure that machida was built with it on?
../../../bin//machida --application-module alphabet_partitioned --in <IP_ADDRESS>:7010 \
  --out <IP_ADDRESS>:7002 --metrics <IP_ADDRESS>:5001 --control <IP_ADDRESS>:6000 \
  --data <IP_ADDRESS>:6001 --worker-count 3 --cluster-initializer \
  --external <IP_ADDRESS>:5050 --ponythreads=1 --ponynoblock
****CLUSTERING MODE is active****
****AUTOSCALE MODE is active****
||| Resilience directory: /tmp|||
Set up external channel listener on <IP_ADDRESS>:5050

Perhaps it should be renamed. There is some information that is required in general that isn't "recover state" related but is required to "rejoin cluster after dying" that is always logged.


About [ ReferenceError: MutationObserver is not defined ]

Thanks for creating a nice plugin!
It's very useful and I'm using it.
I just have one thing that bothers me.
I get the following warning during build
⠙ rendering pages...ReferenceError: MutationObserver is not defined
...
[Vue warn]: Component <Anonymous> is missing template or render function.

It doesn't affect my usage, but it bothers me, so I would appreciate it if you could fix it.
Thank you!

Thanks!


Add new param type: json

It would be nice to have an additional param type like json which just takes a go struct and uses it as a default JSON body.
Here is an example:
type LoginPayload struct {
	Email    string `json:"email"`
	UserName string `json:"user_name"`
	// Required: true
	Password string `json:"password"`
}

// swagger:parameters login
type ImplicitWrapperStruct struct {
	// In: body
	Body PersonLoginPayload
}

it already works this way.
There is no param type json but the anntotations you provide should in fact work in that way.

Hi @casualjim. Maybe I didn't make it clear enough. I was hoping for a way without the wrapper struct.

At this moment I'm inclinded to say that this is an optimization with pretty low ROI. But I'll leave the ticket open, and if more people want it they should put their +1 here.

Thanks!

@casualjim I just came across this and for me this wouldn't be about optimising, but more about having to change your Go code so that it works with the swagger generator, opposed to swagger just working with your code, right? I mean, would you find a blog article on writing a standard Go handler func that says "make sure that you use a wrapper struct so that swagger generate can handle it"? Not sure if that makes sense...?
cc @schickling

It was never spelled out in this thread why it is the way it is, so here it goes:
The reason a body struct is wrapped atm is because for swagger a response is headers + body, and so my scanner wants to preserve that behavior.
I wouldn't refuse a PR that addresses this through a swagger:bodyResponse annotation instead of swagger:model or swagger:response

@casualjim  My pr did not address what they were talking about as he Is talking about paramaters not response. I do plan on doing a pr to add swagger:bodyparamaters though


Ignore other warnings when checking warnings

Change Summary
Building on Michał's amazing work in #9040 , extend the pattern to also ignore other warnings that are emitted by pytest.warns() when running under Pytest 8.x. This also continues to support Pytest 7.x.
Co-authored-by: Michał Górny<EMAIL_ADDRESS>Related issue number
Fixes: #9025
Checklist

[x] The pull request title is a good summary of the changes - it will be used in the changelog
[ ] Unit tests for the changes exist
[x] Tests pass on CI
[ ] Documentation reflects the changes where applicable
[x] My PR is ready to review, please add a comment including the phrase "please review" to assign reviewers

Selected Reviewer: @hramezani

please review

@s-t-e-v-e-n-k,
Thanks for your work here! Is the idea that both this and https://github.com/pydantic/pydantic/pull/9040 will need to be merged to fix #9025?
Happy to review more thoroughly, just wanted to better understand the context of this PR :).

@sydney-runkle Thanks for the linting fix, but no, this PR can be merged as-is -- it builds on the work done in #9040 so I wanted to make certain @mgorny was also credited for the work done.

@s-t-e-v-e-n-k and @mgorny,
Thanks for the awesome work here. One lingering question that I still have - for the case where we're catching multiple warnings, what about something like:
import warnings
import pytest

def test_multiple_warnings():
    with pytest.warns((UserWarning, RuntimeWarning)) as record:
        warnings.warn("This is a user warning", UserWarning)
        warnings.warn("This is a runtime warning", RuntimeWarning)

    assert len(record) == 2  # check that two warnings were raised
    assert isinstance(record[0].message, UserWarning)  # check the type of the first warning
    assert str(record[0].message) == "This is a user warning"  # check the first warning message
    assert isinstance(record[1].message, RuntimeWarning)  # check the type of the second warning
    assert str(record[1].message) == "This is a runtime warning"  # check the second warning message

The only thing that I feel is missing from the current implementation is details regarding the second, third, etc warnings being raised, which I think we want to preserve.

I completely agree that would be an excellent change, and also allows us to know which code paths raise what warnings -- however, that change will also drop support for pytest 7.4 -- I'm happy to do so, but it would be a larger change.

@s-t-e-v-e-n-k,
Ok, gotcha. This looks good then. I appreciate the TODO additions to have as a future reference!


Remove sign_password from request parameters

Solves first part of https://github.com/dzonatan/paysera-nodejs/issues/2

Thanks. Kudos to you!
v1.0.2 released.


Save token usage statistics to DB

Summary
In order to introduce spending limits we would want to save the token usage for an agent in the DB
Expected behaviour
Create a new DB table for saving tokens and cost estimations.
Each run should be saved as a run entry.
One to many relationship between usage and agent.

Fixed in #323


Nested JSON with panels results in field re-ordering

For example:
JSON::make('Extras', [
    new Panel('This should be first', [ <Fields> ]),
    Text:make('should be second'),
]),

This results in the the Text field displaying before the panel so the fields aren't maintaining the order given. This can also be confirmed via the ajax response that loads the configuration...It's correctly ordered there, but panels always seem to show last rather than in the order they were defined.
This might not seem like a big deal, but if you need the UI to display in a specific order this totally breaks that.

Disregard it seems to be resolved in a newer version than I had

Sorry for the spam, thought it was addressed, but it seems to still be an issue at least on the index page view specifically. It still is re-ordering fields/panels to always have the panels last.

Fix available on 0.17.0


[Bug] OPEA docker container and habana runtime issue

Priority
P3-Medium
OS type
Ubuntu
Hardware type
Xeon-ICX
Installation method

[ ] Pull docker images from hub.docker.com
[ ] Build docker images from source

Deploy method

[X] Docker compose
[ ] Docker
[ ] Kubernetes
[ ] Helm

Running nodes
Single Node
What's the version?
commit ID: 179fd843629952315e1968789f52764876e7f22d
https://github.com/opea-project/GenAIExamples/tree/main/ChatQnA/docker_compose/intel/hpu/gaudi
Description

The system hits an error when running docker-compose
habana-container-runtime is already installed on the system
Docker daemon.json also points to the habana-container-runtime (see excerpt-1)
Error response below

docker compose up -d

[+] Running 0/4
 ⠋ Container tei-reranking-gaudi-server  Creating                                                                                0.1s
 ⠋ Container tei-embedding-gaudi-server  Creating                                                                                0.1s
 ⠋ Container tgi-gaudi-server            Creating                                                                                0.1s
 ⠋ Container redis-vector-db             Creating                                                                                0.1s
Error response from daemon: unknown or invalid runtime name: habana

Excerpt-1

cat /etc/docker/daemon.json
{
   "default-runtime": "habana",
   "runtimes": {
      "habana": {
         "path": "/usr/bin/habana-container-runtime",
         "runtimeArgs": []
      }
   }
}

Reproduce steps
Steps to reproduce can be found here: Link

export variables

export host_ip="External_Public_IP"
export HUGGINGFACEHUB_API_TOKEN="Your_Huggingface_API_Token"

export host_ip="External_Public_IP"
export HUGGINGFACEHUB_API_TOKEN="Your_Huggingface_API_Token"

export http_proxy="Your_HTTP_Proxy"
export https_proxy="Your_HTTPs_Proxy"
# Example: no_proxy="localhost, <IP_ADDRESS>, <IP_ADDRESS>"
export no_proxy="Your_No_Proxy",chatqna-gaudi-ui-server,chatqna-gaudi-backend-server,dataprep-redis-service,tei-embedding-service,retriever,tei-reranking-service,tgi-service,vllm_service,guardrails

source environment file

source ./set_env.sh

Run docker compose

docker compose up -d

Raw log
N/A

Hi @wangkl2 ,
it looks like I had multiple versions of docker running on the system.
After disabling the bad docker versions and enabling the right ones I see that issue/error message goes away.
However, I am facing an issue, in that the container deployment doesn't download any model or data under the .data folder
# du -sh ./data/
8.0K    ./data/

@anatu-git Please check the container log of tgi backend/embedding/reranking for the error message during downloading the models. If there is connection issues to the huggingface, please check the network connection and check whether there should be some proxy settings. And be sure to set HUGGINGFACEHUB_API_TOKEN.


fix: cleanup volume dir if create volume failed

Volume path should be cleaned up if create volume failed.
Otherwise, the volume will be created failed again next time with a 'file exists' error on os.Mkdir.

How to test this?

How to test this?

This scene is quite rare. There is no space left on device, and a directory is just deleted.
step1
#nerdctl volume create v2
FATA[0000] mkdir /var/lib/nerdctl/1935db59/volumes/default/v2/_data: no space left on device
step2
Cleanup my disk and release some space.
Create volume again.
#nerdctl volume create v2
FATA[0000] mkdir /var/lib/nerdctl/1935db59/volumes/default/v2: file exists
Look into /var/lib/nerdctl/1935db59/volumes/default/v2, it's an empty dir
#nerdctl volume ls
FATA[0000] volume "v2" not found: not found

This case is quite rare, so I expect the user to just run nerdctl volume rm

This case is quite rare, so I expect the user to just run nerdctl volume rm

Volume is not created successfully. The creation process was only halfway complete.
volume rm also got 'not found' error.
#nerdctl volume rm v2
FATA[0000] volume "v2" not found: not found
In fact, get function is called before remove function.
Get function in volume store only check volume dataPath

LGTM on green

@ktock  Could you please rerun the integration test?   The failure is not likely related with nerdctl/volume

@zjumoon01 can you rebase to get the latest change in main? The cosign-related CI errors should be fixed now https://github.com/containerd/nerdctl/pull/2896.

@zjumoon01 can you rebase to get the latest change in main? The cosign-related CI errors should be fixed now #2896.

I rebased, but CI failed again caused by 'run container error' in TestVolumeRemove. Do you have any ideas ?


feat: add tsconfig alias to tsConfig option

Summary
Fixes #1564
Test plan
Added tsconfig scenario to existing relevant test.
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Pull Request Test Coverage Report for Build 4533

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.03%) to 92.872%

Totals

Change from base Build 4520:
0.03%

Covered Lines:
1114

Relevant Lines:
1149

💛  - Coveralls

would you please also update docs ? Here is the doc entry and this is the tsConfig page


Concatenate in go template

Hello guys,
I am trying to create a values.yaml.gotmpl that has the following content :
es:
  uri: http://<username>:<password>@<url>:<port>

Where username and password are retrieved from vault and are declared in the same line.
So I tried a lot of combination, such as :
es:
  uri: {{ "http://" }}ref+vault://secret/path/to/secret?address={{ env "VAULT_ADDR" }}/#username{{ ":" }}ref+vault://secret/path/to/secret?address={{ env "VAULT_ADDR" }}/#password{{ "@domain.net:80" }}

... and a lot of other tests, without any success.
Please can you help ?
Thanks a lot !

@Xat59 Hey! Try using print for concatenating strings and fetchSecretValue for fetching secrets. Perhaps yours should be something like {{ print (fetchSecretValue (print "ref+vault://secret/path/to/secret?address=" (env "VAULT_ADDR") "/#username")) ":" (fetchSecretValue (print "ref+vault://secret/path/to/secret?address=" {{ env "VAULT_ADDR" }} "/#password")) "@domain.net:80" }}

I have finally succeeded with this syntax :
es.uri: {{ print "http://" }}{{ fetchSecretValue ( print "ref+vault://path/to/secret?address=" (env "VAULT_ADDR") "/#username" ) }}{{ print ":" }}{{ fetchSecretValue ( print "ref+vault://path/to/secret?address=" (env "VAULT_ADDR") "/#password" ) }}{{ print "@domain.net:80" }}

Thanks for all the hints.
Regards

{{ $username := fetchSecretValue ( print "ref+vault://path/to/secret?address=" (requiredEnv "VAULT_ADDR") "/#username" ) -}}
{{ $password := fetchSecretValue ( print "ref+vault://path/to/secret?address=" (requiredEnv "VAULT_ADDR") "/#password" ) -}}
{{ print "http://" $username ":" $password "@domain.net:80" }}

It works like a charm ! Thanks a lot @mumoshu


Go to next question when press Enter Key

Are you requesting a feature, reporting a bug or ask a question?
Asking a question or requesting a feature if is not available.
What is the current behavior?
In the survey I press a Enter Key and don't change to next question
What is the expected behavior?
When press the Enter Key go to the next question. If is the last question go to next page. If is the last page submit the survey. Set goNextPageAutomatic  property to true don't work.
Specify your

browser: Chrome
browser version: 58.0.3029.110
surveyjs platform (angular or react or jquery or knockout or vue): jquey
surveyjs version: 0.12.16

Can this be done? Is this a feature request or is it currently possible?

@enricribas It's a feature request. It's included in our TODO list. But I can't give an estimation.

I would LOVE this! I love how TypeForm even allows using the keyboard to answer multiple choice questions, too (https://www.typeform.com/help/whats-the-average-completion-rate-of-a-typeform/).


Prevent Terminal from Auto Upgrading When Terminal is Open and In Use

Description of the new feature/enhancement

Windows Terminal Updates Itself through Microsoft Store or Other Update Mechanisms when the Terminal is Open or In Use, This Could Make the users to lose Operation that were currently running in Terminal
Proposed technical implementation details (optional)

Implement a Feature that Detects, If the Terminal is Open and In Use, If that's true pause the Auto Update Until the Windows Terminal Gets Closed and Not in Use

See issue #3915.

Yep. Also see https://github.com/microsoft/terminal/issues/6726#issuecomment-1147358557 - the Store team is actively working on rolling this out.
/dup #3915
/dup #6726


Revert "Revert "Merge branch 'fix_performance_restore_sdk' of https:/…

…/github.com/fsprojects/Paket""
This reverts commit d96ac114228ea2e2bfb6eafbf0779ba0f7b6b91d.

Second try of https://github.com/fsprojects/Paket/pull/3200 (for reference)
An attempt to fix https://github.com/fsprojects/Paket/issues/3196


How to rspec join record in many-to-many in rails 3.1.0

Here is the create in rfq controller:
  def create
    if has_create_right?
      @rfq = Rfq.new(params[:rfq], :as => :roles_new )
      #save into join table rfqs_standards
      params[:rfq][:standard_ids].each do |sid|
        @rfq.standards << Standard.find(sid.to_i) if !sid.nil? && sid > 0
      end unless params[:rfq][:standard_ids].nil?
    if @rfq.save
        redirect_to URI.escape("/view_handler?index=0&msg=RFQ saved!")
      else
        flash.now[:error] = "RFQ not saved!"
        render 'new'
      end
    end
  end

Here is the rspec code
  describe "'create'" do
    it "should be successful for corp head" do
      session[:corp_head] = true
      session[:user_id] = 1
      s = Factory(:standard)
      rfq = Factory.attributes_for(:rfq)
      rfq.standards << s
      get 'create', :rfq => rfq
      response.should redirect_to URI.escape("/view_handler?index=0&msg=RFQ saved!")
    end
  end

The error is:
  1) RfqsController 'create' should be successful for corp head
     Failure/Error: rfq.standards << s
     NoMethodError:
       undefined method `standards' for #<Hash:0x6741570>
     # ./spec/controllers/rfqs_controller_spec.rb:65:in `block (3 levels) in <top (requi

The problem seems to be that the rfq created by Factory.attributes_for is not an object and rfq.standards causes error. Tried rfq[:standard_ids] = [s.id] and did not work.
What's the right way to test join record? Thanks so much for help.

This isn't an rspec-core issue.  Factory.attributes_for is a factory girl method.  You should ask for help on the factory girl mailing list or issue tracker.


Visual border using particles

@DalBrar contacted me with a visual border implementation, which I think looks nice.
https://github.com/DalBrar/WorldBorder-Particles
It currently only properly supports rectangular border shapes, not round/elliptic, and lacks config options outside of directly editing his Particles.java, but I figured some people will be interested in building it for themselves anyway.
Here's what his implementation looks like:

I like it.
I'd toyed around with using particles to visually mark the border edge in the past (also the possibility of using phantom blocks sent to the client which don't really exist on the server), but my efforts didn't turn out very well and seemed like they might particularly affect client performance, so I'd scrapped the idea. Based on the example he provided above, I'm revisiting the idea, even though I'm generally not actively developing the plugin currently and am rather busy lately.
To properly implement this, I think I would need to implement an additional "inner" border with a slightly smaller radius based on the actual full-sized border, which would be checked initially instead of the actual border to detect players nearing the actual border (so as to only require additional checks on people near the border), then a reasonably clever method to determine and possibly cache the X/Z locations of an elliptic border's boundaries near the player for display. Display of rectangular borders is probably covered well enough by the code above, though determining whether they need to be displayed could be better handled in the new "inner" border checking code for less processing overhead.
Hmm. Not sure when/if I'll get around to it, but his code has made me somewhat interested in trying to more fully implement this for a visual border.

I love the concept and use of the barrier block for a world border.  It makes perfect sense to use it when the world border is established.   I would love it if this came with configuration options to vary server-by-server:

Allow for showing or hiding the barrier/border.  We know barrier blocks can be seen in creative mode, but no survival mode. So having something like - barrierBorderVisible: true/false would be great.
Allow for selection of using barrier block or bedrock.  This would allow server-owners to make the other side of the world visible or not.  - bedrockBorderBreakable: true/false and - barrierBorderBreakable: true/false
Allow for either block used to allow custom textures to override its visual appearance

I've seen some configuration files that allow for breaking bedrock or not, so having an option to turn it off for the border would make sense for both barrier and bedrock blocks.  At least this would be a nice enhancement to it later. :-)

hello, does anyone have the source (or even compiled jar) for this? looks like the repo was made private or deleted. @DalBrar do you still have that somewhere?

@trollogyadherent  Sorry for the late reply, yes I still have the code that was suggested as the original idea. Hasnt been touched in an extremely long time, do with it what you'd like.
MC-WorldBorder-Particles-master.zip
.


Review roadmap to include UX design work for reporting work

Describe the task
We need to review the roadmap to put down what the first cycle of UI/UX work will look like for reporting work. We have some ideas on stretching out lo-fidelity work first, as well as some iteration for the design process in general.
Acceptance Criteria

[x] Review existing roadmap
[x] Meet with Patricia to understand the broader vision for reporting + what we need from UX for user story mapping
[x] Create a rough plan for how we want to tackle design process for reporting
[x] Amend existing roadmap to have latest design process

Additional context

Add any other context about the task here.
Or here

Hey team! Please add your planning poker estimate with Zenhub @jaimiebutton @suhafa

@patriciarussellCAS hey Patricia! This ticket is completed. I've amended the roadmap to include UX work for reporting that aligns with this process

Thanks Suha!


feat: tekton task urls in the environment sub-command output

Changes

:gift: Print tekton tasks urls in output of func environment.

PTAL @jrangelramos @gauron99

/retest-required


Remove Runtime Uses of EnableKeyword/DisableKeyword for shader_feature Keywords

Describe the bug
A handful of issues have been reported around shader features working in editor but not working within a built player. This is due to a few MRTK scripts using the EnableKeyword and DisableKeyword methods at runtime for keywords which utilize the shader_feature pragma. For more information as to why this is an issue please see "Special requirements for Scripting with the Standard Shader."
To reproduce
For example, if you look at ConstructGltf.CreateMRTKShaderMaterial() you can see code which does the following:
material.EnableKeyword("_ALPHATEST_ON");
material.DisableKeyword("_ALPHABLEND_ON");
material.DisableKeyword("_ALPHAPREMULTIPLY_ON");

This is perfectly valid within editor scripts, but not at runtime.
Expected behavior
Runtime calls to EnableKeyword/DisableKeyword should be removed and replaced with explicit material references. Please see this thread for more information.
Target platform

Any built Unity player.

Like @FreakTheMighty, I've come across this issue when trying to load glTF models dynamically as per #7543.
I'm on a slightly old toolkit and so in order to temporarily get around the problem with directional light, I've switched the _DIRECTIONAL_LIGHT in the MRTK standard shader to be marked with the pragma multi_compile rather than shader_feature - I'm assuming that I'll take a hit for this and it doesn't solve the potential issues with the other uses of Enable/DisableKeyword in the glTF code but for my needs it seemed to make things a lot better (i.e. I got some light on my models on device).

@mtaulty that should be a totally fine workaround for your scenario. You will notice slightly longer build times as more shader variants will get included in the build. But, I wouldn't expect you experience any significant runtime costs.

We appreciate your feedback and thank you for reporting this issue.
Microsoft Mixed Reality Toolkit version 2 (MRTK2) is currently in limited support. This means that Microsoft is only fixing high priority security issues. Unfortunately, this issue does not meet the necessary priority and will be closed. If you strongly feel that this issue deserves more attention, please open a new issue and explain why it is important.
Microsoft recommends that all new HoloLens 2 Unity applications use MRTK3 instead of MRTK2.
Please note that MRTK3 was released in August 2023. It features an all new architecture for developing rich mixed reality experiences and has a minimum requirement of Unity 2021.3 LTS. For more information about MRTK3, please visithttps://www.mixedrealitytoolkit.org.
Thank you for your continued support of the Mixed Reality Toolkit!


Missing dependencies after update to 0.3.4 or 0.3.2

Module not found: Can't resolve 'ethers' in '/Users/kacper/Projects/earnbase/node_modules/@usedapp/core/dist/src/helpers'
Remove node_modules does not help.

Hmm, should be working with 0.3.3 onwards.
Try adding     "ethers": "^5.1.0", to your package.json and let us know if it helps.

It helps thanks

btw we fixed dependencies in the new version and now it should work "out-of-the-box"


Agent AI #22SIDHU

Run:done:pin:

Looks like this was created by accident, closing. If needed, feel free to reopen with more details, thank you!


Fix #554

Documentation rename/update for #554

I think the Travis CI build has failed because I'm out of Travis CI credits (after a recent change in their policy). I'll try to switch us over to GitHub actions before we merge this PR.

That makes a lot of sense. The build fail confused me a bit.
I've noticed another mention of "Perl 6," so I've also updated this. To tidy the whole thing up, I've merged the 4 commits into 1.

LGTM :+1:


More efficient datastructure for storing hunks

Hunks are currently stored in a list and accessed using an index. This operation have to iterate the list from the beginning to reach the desired hunk. Which is slow when looking at the end of a very large diff.
More efficient datastructures for this are: Array or Zipper.

@yokurang is assigned this issue.
Let me just add a link to a good resource on Zippers: https://pavpanchekha.com/blog/zippers/huet.html


[ROCM] enable some multigpu tests

This PR enables some of the multi-gpu tests that were disabled in a previous pr https://github.com/tensorflow/tensorflow/pull/50331

@micmelesse thank you for the PR, please let us know when it is ready for review , also add some description for PR.

@rthadur This pr is ready for review. Sorry about that.

@gbaned @cheshire Is there something that I should to address the "feedback/copybara" failure  or is it a ci issue? I cannot access the details of the test

@gbaned @cheshire  All the tests pass is it possible to merge this now

@micmelesse Can you please resolve conflicts? Thanks!

@gbaned sorry for the delayed response. Conflicts are resolved.

@cheshire Can you please review this PR ? Thanks!

@cheshire Can you please review this PR ? Thanks!

@micmelesse Can you please resolve conflicts? Thanks!

@micmelesse Any update on this PR? Please. Thanks!

@gbaned Apologies I have resolved the conflicts.

@gbaned @cheshire I am not sure the failures have anything to do with the changes in this pr. It just enables tests on rocm. The ROCM ci passes.


droppy 6.3.17 running into problem.

       .:.
:::  .:::::.   droppy 6.3.17 running on node 6.5.0

..:::..  :::     config at /home/tk/.droppy/config
':::'   :::     files at /home/tk/.droppy/files
'
[ERROR] =============== Uncaught exception! ===============
[ERROR] TypeError: Expected options to be either an object or a string, but got function instead
at throwOptionsError (fs.js:81:9)
at Object.fs.writeFileSync (fs.js:1326:5)
at write (/home/tk/Desktop/blog-feeds-incr-sync/tkarchive/node_modules/droppy/server/db.js:123:6)
at /home/tk/Desktop/blog-feeds-incr-sync/tkarchive/node_modules/droppy/server/db.js:21:11
at /home/tk/Desktop/blog-feeds-incr-sync/tkarchive/node_modules/mkdirp/index.js:48:26
at FSReqWrap.oncomplete (fs.js:123:15)

Fixed that mistake. 6.3.18 is building and should be available in a few minutes.

droppy does not have any basic test covered? Hey, this is terrifying if droppy does not check if it is usable before publishing.

No automatic tests yet. I do test on Mac and Linux on latest Node.js, where apparently this issue wasn't showing.

So the problem is old version node complains about writeFileSync misusage?


Avoid using printStackTrace in SDK

Using printStackTrace in the release application might introduce a potential security risk. It should be removed from SDK.
https://salemove.atlassian.net/browse/MOB-875
https://salemove.atlassian.net/browse/SSD-19423

!squash

!merge


lib/core: Fix substring issue in ASCIIFlatString

A bug in the validation part of ASCIIFlatString::substring caused some substrings to be created with a negative length, which could be the cause of various negative side-effects.
In the example added in the substring test, this caused stdout to close due to a call to fwrite with a negative count, which was in turn converted to unsigned (I suspect this is undefined behaviour), provoking the closure of the stream in the nit part.
Reported by @Morriar
Close #2089

+1, maybe state somewhere that it fixes #2089

Close #2089
Was preparing the fix when someone opened the issue, this should autoclose it then

+1

toaster: test this please

test this please

There was a failing test, but I could not reproduce the failure on my machine.
I'm retesting on Jenkins to make sure this was not a transient bug.

toaster: test this please
build was aborted by a kernel upgrade

that was a piece of bread damn hard to toast


Re-enable nightly checks and add new dependency duplicate

Objective

Nightly checks where disabled because of a bug in Rust
Dependency checks are failing because of a new duplicate

Solution

Now that https://github.com/rust-lang/rust/pull/92175 has been merged, re-enable nightly checks
Add the new duplicate dependency to the known list
Removed Inflector dependency as it's not used anymore

bors r+


Fix multicall issues

Previously, when a value from a batch_call could not be decoded a error was raised. When multicall was implemented it was refactored to return the raw bytes. Some parts of the tx service depends on this logic, so if raise_exeception=True a exception will be raised if there's a decoding error or an empty response during multicall

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

5 of 5 (100.0%)  changed or added relevant lines in 2 files are covered.
11 unchanged lines in 4 files lost coverage.
Overall coverage decreased (-0.2%) to 90.026%

Files with Coverage Reduction
New Missed Lines
%

gnosis/eth/typing.py
2
83.33%

gnosis/safe/safe_tx.py
2
88.72%

gnosis/eth/ethereum_client.py
3
85.43%

gnosis/eth/oracles/oracles.py
4
88.69%

Totals

Change from base Build<PHONE_NUMBER>:
-0.2%

Covered Lines:
2419

Relevant Lines:
2687

💛  - Coveralls


Adding asyncio.sleep() to Asyncified function

I seem to hit a conceptualization issue trying to do this. When I reach the FunctionDef, I look for await statements and if there's none, I want to add asyncio.sleep(0) to the last line, I search in the body and try an InsertAfter the last node of the body, but that errors out
Well, the crude method of adding the Expr at the end of body kinda works, but an empty line is inserted as well and it felt clunky

Just a bit confused somehow, I simply need to use LazyInsertAfter on the last node then Replace the method. I was creating the new ast.AsyncFunctionDef then had context issues with the LazyInsertAfter


Lambdas in json files

Looking at the php source, it appears to be able to call user code in the hashes, per mustache spec. However, a function placed in the .json file for a molecule appears not to be parsed at build time.

Hi @dmolsen, there is nothing about mustache helpers inside the patternlab docs. How to use them?


Cache prev_hashes input prover data

When proving consecutive blocks, we should reuse already retrieved data.

Isn't it what https://github.com/0xPolygonZero/zk_evm/issues/282 was supposed to do already? Or do you mean if you kick of a job proving blocks 1 to 10. And then a next job proving blocks 11 to 30, the latter should retrieve the previously fetched hashes?

@Nashtare #282 just halved the number of calls to retrieve the previous 256 hashes. We still call fetch_other_block_data for every block. We also call provider.get_block on multiple places for the same block. Everything should be cached.

Ah ok. I got confused because the linked issue in #282 wasn't about going from 256 to 128 RPC calls but really about caching previously fetched block hashes (effectively what this new issue wants to address). But thanks for clarifying.

@Nashtare aha, OK. I opened the #282 to port https://github.com/0xPolygonZero/zero-bin/pull/62 to alloy and it didn't implemented cache.


saml plugin conflict with spring security plugin

can anyone get me out of below problem.
2017-03-26 16:52:12,961 [localhost-startStop-1] INFO  commons.TagLibArtefactHandler  - There are conflicting tags: es.salenda.grails.plugin.springsecurity.saml.SamlTagLib.sec:loggedInUserInfo vs. grails.plugin.springsecurity.SecurityTagLib.sec:loggedInUserInfo. The former will take precedence. 2017-03-26 16:52:12,962 [localhost-startStop-1] INFO  commons.TagLibArtefactHandler  - There are conflicting tags: es.salenda.grails.plugin.springsecurity.saml.SamlTagLib.sec:ifNotGranted vs. grails.plugin.springsecurity.SecurityTagLib.sec:ifNotGranted. The former will take precedence. 2017-03-26 16:52:12,962 [localhost-startStop-1] INFO  commons.TagLibArtefactHandler  - There are conflicting tags: es.salenda.grails.plugin.springsecurity.saml.SamlTagLib.sec:switchedUserOriginalUsername vs. grails.plugin.springsecurity.SecurityTagLib.sec:switchedUserOriginalUsername. The former will take precedence. 2017-03-26 16:52:12,962 [localhost-startStop-1] INFO  commons.TagLibArtefactHandler  - There are conflicting tags: es.salenda.grails.plugin.springsecurity.saml.SamlTagLib.sec:ifLoggedIn vs. grails.plugin.springsecurity.SecurityTagLib.sec:ifLoggedIn. The former will take precedence.

Hi @prasadtechnology .  Thank-you for reporting this issue!
Would you be so kind as to provide more details.  For example:

What version of Grails are you using?
What version of the grails-spring-security-ui plugin are you using?
What version of the grails-spring-security-core plugin are you using?
What version of Java are you using?
What are the steps needed to reproduce this issue?
What are your expected vs actual results?
Are you able to attach a an app with a test to this PR (or point to a repo) that reproduces the problem?

Closing this issue because I cannot reproduce and no further details have been provided.
@prasadtechnology I will be happy to reopen this issue if it is still a problem and if you are able to provide the details needed to reproduce it.


Not able to see debug console log after update VSCode

Issue Type: Bug
It was working fine till the previous version of VSCode. Since I updated to the latest version it stops displaying debug console details in section.
Can you please help me with a workaround, It is difficult to run and debug together and find code issues.

VS Code version: Code 1.47.1 (485c41f9460bdb830c4da12c102daff275415b53, 2020-07-13T22:42:53.386Z)
OS version: Windows_NT x64 10.0.18362

System Info

Item
Value

CPUs
Intel(R) Core(TM) i3-6098P CPU @ 3.60GHz (4 x 3600)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: unavailable_offprotected_video_decode: unavailable_offrasterization: unavailable_offskia_renderer: disabled_off_okvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
15.96GB (5.98GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (2)

Extension
Author (truncated)
Version

vscode-javascript-booster
sbu
14.0.1

code-spell-checker
str
1.9.0

Facing same issue, debug console log working with node v12.11.1, but not with other version

I had this issue with node versions v11.9.0 and v12.18.2 although I'm not sure it's specific to the node version.
I was able to resolve by adding "outputCapture": "std" to my launch config.

Thanks for the issue! Can either @tgs-vaibhav, @roshangupta07, or @jameslh collect a trace log using these instructions?

If you're able to, add "trace": true to your launch.json and reproduce the issue. The location of the log file on your disk will be written to the Debug Console. Share that with us.
⚠ This log file will not contain source code, but will contain file paths. You can drop it into https://microsoft.github.io/vscode-pwa-analyzer/index.html to see what it contains. If you'd rather not share the log publicly, you can email it to<EMAIL_ADDRESS>