Typo makes resource type targeting ambiguous

The example under "Use VM in pipelines" has a typo in the YAML - this makes the result of targeting by resource type unclear.
The text reads:
  environment:
   name: VMenv
   resourcesNAme: VMenv
   resourceType: virtualMachine

But should read:
  environment:
   name: VMenv
   resourceType: virtualMachine

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

ID: 77d95db6-9983-7346-d0eb-4b7443e4e252
Version Independent ID: 0a22cccc-318d-592f-d1ab-09ec01d88087
Content: Create target environment - Azure Pipelines
Content Source: docs/pipelines/process/environments.md
Service: azure-devops-pipelines
Sub-service: azure-devops-pipelines-process
GitHub Login: @juliakm
Microsoft Alias: jukullam

Nope. Not sure why the link to the content that automatically added when I raised the issue is incorrect. The live document is here: https://learn.microsoft.com/en-us/azure/devops/pipelines/process/environments-virtual-machines?view=azure-devops
And in the "Use VM in pipelines" section it is still incorrect.

@Brubning -- thank you for your followup; I'll update the article.  It is odd that the wrong page showed up.

@Brubning Thanks, a fix will go out shortly.


'React/RCTEventEmitter.h' file not found - RN 0.43.2

Description

Initialized new react native project following the react native getting started.
Encounter failed build with the error within terminal:
'React/RCTEventEmitter.h' file not found

Reproduction Steps and Sample Code
react-native init AwesomeProject
cd AwesomeProject
react-native run-ios
Result:
fatal error: 'React/RCTEventEmitter.h' file not found
--Running in XCode--

Additional Information

React Native version: 0.43.2 (latest)
Platform: iOS
Development Operating System: macOS Sierra v10.12.4 (latest)
Dev tools: XCode v8.3.1 (latest)
Build Options on Xcode (default)

Same issue.

I confirm I'm gettting it too unfortunately. RunninUsing React Native 0.43.2 too

I have seen and done the procedure before. It has not worked for me still :/
I tried adding the scheme in the order you described but it has also not worked.
It seems my RCTLinking is still broken because I still get:
'React/RCTEventEmitter.h' file not found in xcode

I solved this problem yesterday with:

Replace all #import "RCT*.h" with #import <React/RCT*.h>.
Since I have used CocoaPods, my RN native modules in Pods still had errors. I haven't found the solution, so I moved my RN native modules from Pods to project, it works.

Ensure your have unchecked Parallelize Build and removed all react-native related header search paths from all targets.

@guangmingzizai The documentation for Firebase says to use react-native as one of the header search paths. Can you explain why you're removing them?

Hi there! This issue is being closed because it has been inactive for a while. Maybe the issue has been fixed in a recent release, or perhaps it is not affecting a lot of people. Either way, we're automatically closing issues after a period of inactivity. Please do not take it personally!
If you think this issue should definitely remain open, please let us know. The following information is helpful when it comes to determining if the issue should be re-opened:

Does the issue still reproduce on the latest release candidate? Post a comment with the version you tested.
If so, is there any information missing from the bug report? Post a comment with all the information required by the issue template.
Is there a pull request that addresses this issue? Post a comment with the PR number so we can follow up.

If you would like to work on a patch to fix the issue, contributions are very welcome! Read through the contribution guide, and feel free to hop into #react-native if you need help planning your contribution.

@hramos I just endup on that thread because I'm facing this issue. It might not be dead

@debrice that's entirely possible. See my last comment for next steps.

Sometimes you just need to make sure you are building React first, before building anything else.

Same issue
react: 16.2.0
react-native:0.53.2
MacOS: 10.13.3
XCode: 9.2

same issue

same issue
any solution yet?

hi there,
fyi: react-native upgrade did the job for me 👍

I'm having this issue with React-Native 0.55 and Calabash from the -cal target (while the non -cal target compiles just fine).

I had this crop up when I added react-native-sentry


Reduce width of window border chrome elements

To create more blend with the OS, the borders (mostly the top) should be reduced in width. Also, the search icon should be moved to the left. expand the images for better scale.
For example Spotify:

Everything is far more condensed. Perhaps even pushing up into close button & fullscreen territory might be an option.
Finder:

Current Radiant GUI:

Now tell me that that doesn't look clunky and chunky in comparison. Totally not bashing on your work, it's great; but this is the beauty of open source, quality collabroation.

Closing this - the UI here is from an outdated version of the app.


Issue #713

Fix for issue #713.

Change the wizard pick project dialog window to use control templates rather than visibility.
Change the wizard pick project dialog to use a DataGrid rather than an ListBox per @chrsmith.

Codecov Report

Merging #716 into master will decrease coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master    #716      +/-   ##
=========================================
- Coverage    6.66%   6.66%   -0.01%
=========================================
  Files         470     470
  Lines       17066   17075       +9
=========================================
  Hits         1138    1138
- Misses      15928   15937       +9

Impacted Files
Coverage Δ

...gs/ProjectIdDialog/PickProjectIdWindowContent.xaml
0% <ø> (ø)
:arrow_up:

...tension/GoogleCloudExtension/Resources.Designer.cs
2.05% <0%> (-0.01%)
:arrow_down:

Continue to review full report at Codecov.

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


chore(type): annotate that peft does contains type hints

This helps type checker to know that peft indeed does provide type hints.
Signed-off-by: Aaron<EMAIL_ADDRESS>

See https://peps.python.org/pep-0561/

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

A question for my understanding @aarnphm : The majority of the code base is untyped at the moment. Would this be problematic or is it just ignored when adding py.typed?

Adding py.typed would address the following:
/home/runner/work/OpenLLM/OpenLLM/src/openllm/_configuration.py
  /home/runner/work/OpenLLM/OpenLLM/src/openllm/_configuration.py:101:12 - error: Import "peft" could not be resolved (reportMissingImports)

However, I'm not sure what's the plan for typing with peft, but I believe that adding this is one of the first step towards fully typed the library. It would be problematic if you are using strict type checking, but I think for most python library, it is impossible to make strict type anw.

Thank you @aarnphm for the explanation!

qq: Is there a release date for 0.4.0?


windowsdesktop-runtime: Update to version 5.0.3

Finally got it fixed...

This replaces #5637, which matches the latest desktop runtime version.
LTS version (3.x) is added in https://github.com/ScoopInstaller/Versions/pull/260

/verify checkver has to fail unless there is cdn for latest endpoint

/verify checkver has to fail unless there is cdn for latest endpoint

Even the uncached feed is still 5.0.2.
https://dotnetcli.blob.core.windows.net/dotnet/WindowsDesktop/Current/latest.version

Can't do anything but wait

Changed Checkver URL to non-cached one from https://github.com/dotnet/install-scripts/blob/master/src/dotnet-install.ps1
As suggested from last comment, unless MS update their feed, check will fail, can't do anything about unless remove checkver.


Make a review and clean up of the code before releasing Fulma

And this reminder might be unnecessary, but don't forget to update the gh-pages branch when Fulma is released, so that all the new documentation you've written will show up on the website. :-)

For info, I published the new docs site last week.


New features and bugfixes

Added API server for use in Kodi and Mediaportal:

Mediaportal (stable): https://forum.team-mediaportal.com/threads/hyperionscreencapture-ambilight-plugin-madvr-capture-alternative-to-atmolight.137035/
Kodi (WIP): https://github.com/RickDB/script.hyperionscreencapture

Fixed multi-monitor readouts, now uses display name for comparison as index didn't work most of the time which isn't ideal but other way is to use handle ID which can change as well (fixes issue #9 and #15)
Fixed DPI bug from issue #10 , thanks @SodaleJetzedle
Fixed changing monitor index from system tray.
Made ToggleCapture a function for consistent usage
Time exclusions for API so that user can keep them off during certain hours.
Added standby handling in rare cases where capture or connections are not recovering properly.
Function renames to keep it consistent with the rest of the code.

It does require a couple of new settings in default config when shipping:
`
<add key="apiEnabled" value="False" />

<add key="apiExcludedTimesEnabled" value="False" />

<add key="apiExcludeTimeStart" value="23-7-2017 08:00:00" />

<add key="apiExcludeTimeEnd" value="23-7-2017 20:00:00" />

<add key="captureOnStartup" value="False" />

Also the Grapevine dependency for API / Webserver which is a new .dll file.
`

Would really like to see this merged! 👍

Please merge this as it contains important bug fixes


WIP 0.0.3 (Configurable memoize / variadic dependencies / pass props to selector)

Work in progress experiments to address issues #7 , #20 , #27. The API section of the README has some examples.
There is one breaking change. createSelectorCreator now takes a memoize function instead of a valueEquals function.

If it has a breaking change, it better be 0.1.0, not 0.0.3.
Otherwise you'll break any consumers that have ^0.0.2 in package.json.

Anyway, I suggest you jump to 1.0 soon :-). People are using it in production.

Yes, good point. I'll rename this and make it the 1.0.0 branch.


Manually allocating a stack and growing is more expensive than

recursing as that is allocation free will have a smaller L1 footprint.
@havardpe or @arnej27959 PR

we know from bitter experience that this risks using a lot of stack space (potentially more than 256 KB or whatever our current limit is).  If we want to do this generally we should just use 8MB stacks, removing all manual tuning of stack sizes for threads.

Yes, in general recursion must be used with care. However in this case it is only the maximum depth of the tree
that matters and we already

Use recursion in other parts of query setup
Collapses the depth of the query tree before leaving the container.

So I think this should be perfectly safe to use here. As far as I can tell each recursion will use 24 or 32 bytes per iteration so depth should be on the safe side.
However as a orthogonal thing we might consider using larger stacks.
It is more than 10 years since those were set.


Design: kubernetes mixin

We should build an in-tree mixin (like exec) for doing things with kuberntes. This should probably behave like kubectl apply -f <...> and kubectl delete -f <...>

I'll make a Gist with a design outline for this and link here for review.


Update whatsapp topic

Please confirm this pull request meets the following requirements:

[x] I followed the contributing guidelines: https://github.com/github/explore/blob/main/CONTRIBUTING.md.
[x] I have no affiliation with the project I am suggesting (as a maintainer, creator, contractor, or employee).

Which change are you proposing?

[x] Suggesting edits to an existing topic or collection
[ ] Curating a new topic or collection
[ ] Something that does not neatly fit into the binary options above

Editing an existing topic or collection
I'm suggesting these edits to an existing topic or collection:

[ ] Image (and my file is *.png, square, dimensions 288x288, size <= 75 kB)
[x] Content (and my changes are in index.md)

Facebook Inc. is now called Meta Platforms Inc.

Curating a new topic or collection

[ ] I've formatted my changes as a new folder directory, named for the topic or collection as it appears in the URL on GitHub (e.g. https://github.com/topics/[NAME] or https://github.com/collections/[NAME])
[ ] My folder contains a *.png image (if applicable) and index.md
[ ] All required fields in my index.md conform to the Style Guide and API docs: https://github.com/github/explore/tree/main/docs

Please replace this line with an explanation of why you think this topic or collection should be curated.

Something that does not neatly fit into the binary options above

[ ] My suggested edits are not about an existing topic or collection, or at least not a single one
[ ] My suggested edits are not about curating a new topic or collection, or at least not a single one
[ ] My suggested edits conform to the Style Guide and API docs: https://github.com/github/explore/tree/main/docs

Please replace this line with an explanation of your proposed changes.

Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.

I wish I knew what this all meant. I feel like someone is acting as me.
On Mon, Sep 19, 2022, 10:57 AM Ash Wilson @.***> wrote:

@.**** approved this pull request.
👍 Thanks for adding the alias there, too. 🙇🏻
—
Reply to this email directly, view it on GitHub
https://github.com/github/explore/pull/3291#pullrequestreview-1112409460,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXKUZ3L26A4RB2IURI2OE4TV7B5NTANCNFSM6AAAAAAQPOP26Q
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Show us how to use GitHub to overpower money by application data with
online real time API to manipulate rich greed money accounts using two
books Tsing style or double entries accounting.
On Sun, Sep 18, 2022 at 8:48 AM Ayush Shah @.***> wrote:

Please confirm this pull request meets the following requirements:

I followed the contributing guidelines:
https://github.com/github/explore/blob/main/CONTRIBUTING.md.
I have no affiliation with the project I am suggesting (as a
maintainer, creator, contractor, or employee).

Which change are you proposing?

Suggesting edits to an existing topic or collection
Curating a new topic or collection
Something that does not neatly fit into the binary options above

Editing an existing topic or collection
I'm suggesting these edits to an existing topic or collection:

Image (and my file is *.png, square, dimensions 288x288, size <= 75
kB)
Content (and my changes are in index.md)

Facebook Inc. is now called Meta Platforms Inc.
Curating a new topic or collection

I've formatted my changes as a new folder directory, named for the
topic or collection as it appears in the URL on GitHub (e.g.
https://github.com/topics/[NAME] or
https://github.com/collections/[NAME])
My folder contains a *.png image (if applicable) and index.md
All required fields in my index.md conform to the Style Guide and
API docs: https://github.com/github/explore/tree/main/docs

Please replace this line with an explanation of why you think this topic
or collection should be curated.
Something that does not neatly fit into the binary options above

My suggested edits are not about an existing topic or collection, or
at least not a single one
My suggested edits are not about curating a new topic or collection,
or at least not a single one
My suggested edits conform to the Style Guide and API docs:
https://github.com/github/explore/tree/main/docs

Please replace this line with an explanation of your proposed changes.

Please note: we will close your PR without comment if you do not check
the boxes above and provide ALL requested information.
You can view, comment on, or merge this pull request online at:
https://github.com/github/explore/pull/3291
Commit Summary

1ee4113
https://github.com/github/explore/pull/3291/commits/1ee4113b0001b57a10f02e58d088b34fe785a72a
Update whatsapp topic

File Changes
(1 file https://github.com/github/explore/pull/3291/files)

M topics/whatsapp/index.md
https://github.com/github/explore/pull/3291/files#diff-2ef36505a6899d662a6405389eb423c896556b584d8260de1dda822c7fe331f7
(4)

Patch Links:

https://github.com/github/explore/pull/3291.patch
https://github.com/github/explore/pull/3291.diff

—
Reply to this email directly, view it on GitHub
https://github.com/github/explore/pull/3291, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZSADCTV7FYRIHRQVYM6LWLV64MSXANCNFSM6AAAAAAQPOP26Q
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

👍 Thanks for adding the alias there, too. 🙇🏻

👍 Thanks for adding the alias there, too. 🙇🏻

What alias

Rich greed political and bank rulers thought they can get away with money
worth monopoly to their own selfish-piggy benefits to the expense of
wonderful creations of humankind and mother natures.
Anyone wants to play?
On Mon, Sep 19, 2022 at 10:58 PM Betterdayahead @.***>
wrote:

👍 Thanks for adding the alias there, too. 🙇🏻
👍 Thanks for adding the alias there, too. 🙇🏻
What alias
—
Reply to this email directly, view it on GitHub
https://github.com/github/explore/pull/3291#issuecomment-1251805411, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AZSADCWOXF26MWYYLBQLA3TV7EY6DANCNFSM6AAAAAAQPOP26Q
.
You are receiving this because you commented.Message ID:
@.***>


NFR: Add AuditIncludeExplicitly attribute

In my project I have a problem when something is deleted then only PK key property is saved even if IgnorePropertyUnchanged = false;, but I need all information, at least some referenced keys (FKs)
it will be very good to add AuditIncludeExplicitly (to property), so it will add this property no-matter-what (all cases)

Currently using hack on EFCore context level and works well
public void SetPropertyModified(object entity, string propertyName, bool isModified)
{
     var entry = _context.Entry(entity);
     var property = entry?.Property(propertyName);
     if (property != null)
     {
          property.IsModified = isModified;
     }
}

Hello @ViltusVilks ,
Thank you for your good suggestion.
So you want to include always properties even if unchanged exactly like we do with the primary key.
We will check about this request.
Best Regards,
Jonathan

Performance Libraries
context.BulkInsert(list, options => options.BatchSize = 1000);
Entity Framework Extensions • Entity Framework Classic • Bulk Operations •  Dapper Plus
Runtime Evaluation
Eval.Execute("x + y", new {x = 1, y = 2}); // return 3
C# Eval Function • SQL Eval Function

Yes, I want to include "navigation" FKs identifiers, so I can grab later history.
Without setting isModified=true hack when You do Remove() on virtual collection (children)  audit log is created OK for PK only, but because object was deleted permanently then You can't see what exactly was deleted, and no chance to reconstruct entire tree.
Now I am doing IsModified=true plus delete entity from root DbSet, because we have a bit bigger graph modifications.
current workaround
var rootEntity = context.FirstOrDefault(...);
var forDelete = rootEntity.Childrens.Where(...).ToArray();
foreach (var childEntity in forDelete)
{
  // it is needed if you want to walk later
  rootEntity.Childrens.Remove(childEntity );
  deleteList.Add(childEntity);
}

 deleteList.forEach( childEntity => {
  // will record all props into log
  context.MyTable.Remove(childEntity);
});

Is there any way to do this for EF6? It seems the checking for IsModified is only for EFCore when I glanced at the source.
I have the same case as @ViltusVilks wherein I need to preserve navigation FKs as I will need them when I aggregate related audit tables for viewing for the client.


search ux: selection persists if query didnt make new matches

closes #341

this solves half of it! but a new issue in this branch is applying the same selection multiple times. i quickly got into a scenario where my search gave results and they were highlighted, then didnt unselect as i typed wrong queries, then went back to a good query and it doubled up the selection handles.
maybe the solution is just to only clear if there's matches?
i got better results with this code:
    // if (!fn) SelectorEngine.unselect_all()
    if (matches.length) {
      SelectorEngine.unselect_all()
      matches.forEach(el =>
        fn
          ? fn(el)
          : SelectorEngine.select(el))
    }

there's some questionable ux around search. i need to think deeper here and come up with some improvements. but the above snippet i believe pacifies this ticket. thoughts?

maybe the solution is just to only clear if there's matches?

Yes I believe so.
  // if (!fn) SelectorEngine.unselect_all()
    if (matches.length) {
      SelectorEngine.unselect_all()
      matches.forEach(el =>
        fn
          ? fn(el)
          : SelectorEngine.select(el))
    }

That looks fine to me.


RPM-based Distro support, parameter parser, numerous updates

Don,
as previously discussed, here is another pull request for puppet-cephdeploy. This comes with a number of additional changes compared to the first pull request; in order to make puppet-cephdeploy usable in the Puppet dashboard, I have added a parameter parser class. I did my best to maintain downward compatibility, so in fact, existing setups should just continue to work.
I have added a switch to make repository deployment configurable as per your request.
Plus, I have worked in the majority of Bastian Ballmann's changes, except for the one I fail to understand; once Bastian has clarified, I will add the according functionality as well. I have commented on individual commits if the purpose of them was not obvious (as in fixing typos or something), but should you have any additional questions, please don't hesitate to ask. Thank you very much again!

Hey Don! I made the branches mergeable again ... You should now be able to just commit the changes, if you are still interested, of course. ;)


[Fix #3631] Add cop to check for spaces in '-> (' format lambdas

I've created the cop with two configuration settings, require_space and require_no_space.
The default is require_no_space.

Before submitting the PR make sure the following are checked:

[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Used the same coding conventions as the rest of the project.
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[x] All tests are passing.
[x] The new code doesn't generate RuboCop offenses.
[x] The PR relates to only one subject with a clear title
and description in grammatically correct, complete sentences.
[x] Updated cop documentation with rake generate_cops_documentation (required only when you've added a new cop or changed the configuration/documentation of an existing cop).

I agree with "lambda literal." -> is special syntax that Ruby interprets like a call to lambda, just like quotes and brackets are special syntax for string, array, and hash literals that Ruby interprets as calls to String.new, etc.

@bbatsov WDYT of this?

I merged this manually.
A small note - not sure whether this cop should warn for multiple spaces between -> and (. Some people might be using them for alignment, but it seems like a bad idea to me.

@bbatsov Thanks.
I am still relatively new to this workflow. In what cases would you manually merge a PR? Is it just when you would like to test the changes locally on your machine?

When there are conflicts in the changelog and I don't want to bother the authors to rebase.

Thanks for this! Any plans for a release yet? :)

I don't want to bother the authors to rebase.

Btw, you can now edit the PR author branch (add commits, rebase existing commits) assuming the PR is set with edit rights to maintainers (it is by default).
It is roughly the same amount of steps, but with the appreciable perk that you don't have to close the PR.


403 Rate Limiting while creating a cluster

Hello - I am being rate limited by Hetzner when trying to create a cluster.
 It fails to create all master nodes
#<HTTP::Response/1.1 403 Forbidden {"Date"=>"Thu, 17 Feb 2022 15:17:56 GMT", "Content-Type"=>"application/json", "Content-Length"=>"202", "Connection"=>"close", "ratelimit-limit"=>"3600", "ratelimit-remaining"=>"3576", "ratelimit-reset"=>"1645111095", "x-correlation-id"=>"xxxx", "Strict-Transport-Security"=>"max-age=15724800; includeSubDomains", "Access-Control-Allow-Origin"=>"*", "Access-Control-Allow-Credentials"=>"true"}> Error creating server k3s-cpx11-master1. Response details below:
Any change to implement a recovery mechanism or a pause?

Hi, it doesn't look like you hit a rate limit because the requests remaining were almost the hourly allotted number and the ratelimit-reset field says that the limit would be reset in a few seconds from that request. 403 means that something went wrong while authorizing the request, so I would check that the token is correct and has read and write permissions. Did any resources get created?

Good point, I had forgotten about that. Although if that's the case 403-unauthorized is a poor choice for the status code for those requests. They should return some more helpful error message explaining the limit.


Added lazy type.

A use case came up for this while fixing a Babylon Native bug where an XR-related global needs to be lazily instantiated (or some alternative) so that it's never created on computers that don't have an OpenXR runtime. We can do without this -- the precipitating issue is small enough to be gracefully handled with a bespoke implementation -- but lazy evaluation is a common enough pattern that I was actually a little surprised there doesn't appear to be an STL type for this (there's an experimental one in Rust, but I didn't see one in C++).
Note: If we decide we actually want this, a little more thought will need to be given to things like implicit constructors and copyability. This draft is just to propose the type and discuss if it's something we think is worth having.

Can you achieve the same thing with std::async(std::launch::deferred, ...)?

@ryantrem Looks like it! I've seen that before, but it looks awesome. I'll test it to double-check it does what I think it does, then abandon this PR if the lazy type is proved redundant.


Add remaining toMatch matchers

Fixes #97
Adds toMatchUrl and toMatchValue to finish the simplification of our matchers.

Codecov Report

Merging #105 (997ea99) into master (964e7c9) will increase coverage by 0.08%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #105      +/-   ##
==========================================
+ Coverage   98.97%   99.06%   +0.08%
==========================================
  Files          15       17       +2
  Lines         195      213      +18
  Branches       26       26
==========================================
+ Hits          193      211      +18
  Misses          2        2

Impacted Files
Coverage Δ

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

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

src/matchers/toMatchTitle/index.ts
83.33% <100.00%> (ø)

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

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

src/matchers/utils.ts
100.00% <100.00%> (ø)

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 964e7c9...997ea99. Read the comment docs.


[BUG]RF Duplication

I just started looking at this mod after seeing the DW20 spotlight. I immediately saw a use for this, controlling a sawmill based power plant. I have EIO stirling engines connected, 2 per connector and 1 plus a sag mill on another. This should give a max output of 7*80 (560 RF/t). However it appears to be giving 1040 RF/t
https://puu.sh/uQAVd/d3ef468797.jpg

I will check this out

I can reproduce this and will look for a solution now

Fixed next release (later today)


Setup

This is an attempt to automate the setup procedure for github repositories as described in https://github.com/giantswarm/github/blob/master/README.md#setting-up-a-repository
It does:

Set repository settings (issues, squash merge, auto-merge, delete branch on merge, etc ..)
Set repository collaborators (Employees as admin and bots as push/write)
Set branch protection

TODO:

Open a PR in giantswarm/github
Handle creation

Allow creation from template

Handle visibility (private/public)

Here is a example run :
$ devctl repo setup giantswarm/heartbeatctl
{"caller":"github.com/giantswarm/devctl/pkg/githubclient/client_repository.go:58","level":"debug","message":"get repository giantswarm/heartbeatctl","time":"2022-06-02T15:48:26.305979+00:00"}
{"caller":"github.com/giantswarm/devctl/pkg/githubclient/client_repository.go:70","level":"debug","message":"got repository giantswarm/heartbeatctl","time":"2022-06-02T15:48:26.769302+00:00"}
{"caller":"github.com/giantswarm/devctl/pkg/githubclient/client_repository.go:79","level":"debug","message":"setting repository giantswarm/heartbeatctl settings","time":"2022-06-02T15:48:26.769357+00:00"}
{"caller":"github.com/giantswarm/devctl/pkg/githubclient/client_repository.go:109","level":"debug","message":"set repository giantswarm/heartbeatctl settings","time":"2022-06-02T15:48:27.102804+00:00"}
{"caller":"github.com/giantswarm/devctl/pkg/githubclient/client_repository.go:128","level":"debug","message":"grant admin permission to Employees on giantswarm/heartbeatctl repository","time":"2022-06-02T15:48:27.10287+00:00"}
{"caller":"github.com/giantswarm/devctl/pkg/githubclient/client_repository.go:137","level":"debug","message":"granted admin permission to Employees on giantswarm/heartbeatctl repository","time":"2022-06-02T15:48:27.392222+00:00"}
{"caller":"github.com/giantswarm/devctl/pkg/githubclient/client_repository.go:128","level":"debug","message":"grant push permission to bots on giantswarm/heartbeatctl repository","time":"2022-06-02T15:48:27.392321+00:00"}
{"caller":"github.com/giantswarm/devctl/pkg/githubclient/client_repository.go:137","level":"debug","message":"granted push permission to bots on giantswarm/heartbeatctl repository","time":"2022-06-02T15:48:27.690808+00:00"}
{"caller":"github.com/giantswarm/devctl/pkg/githubclient/client_repository.go:189","level":"debug","message":"get repository commit statuses for main branch on giantswarm/heartbeatctl repository","time":"2022-06-02T15:48:27.690871+00:00"}
{"caller":"github.com/giantswarm/devctl/pkg/githubclient/client_repository.go:223","level":"debug","message":"got 0 repository commit statuses for main branch on giantswarm/heartbeatctl repository","time":"2022-06-02T15:48:27.926035+00:00"}
{"caller":"github.com/giantswarm/devctl/pkg/githubclient/client_repository.go:172","level":"debug","message":"configure protection for main branch on giantswarm/heartbeatctl repository","time":"2022-06-02T15:48:27.926109+00:00"}
{"caller":"github.com/giantswarm/devctl/pkg/githubclient/client_repository.go:180","level":"debug","message":"configured protection for main branch on giantswarm/heartbeatctl repository","time":"2022-06-02T15:48:28.207724+00:00"}
completed repository setup for heartbeatctl

Potential improvements :

Open a PR in giantswarm/github
Handle creation

Allow creation from template

Handle visibility (private/public)

Changed the logging system to produce nicer output messages
$ devctl repo setup giantswarm/heartbeatctl
INFO[0000] get repository details for "giantswarm/heartbeatctl"
INFO[0000] configure repository settings
INFO[0000] grant permission on repository
INFO[0000] configure protection for "main" branch
INFO[0001] completed repository setup

output with debug log
devctl (gs-ginger:ginger-prometheus)$ go run . repo setup giantswarm/heartbeatctl --log-level=debug
INFO[0000] get repository details for "giantswarm/heartbeatctl"
DEBU[0000] repository details
{
  "id": 316560254,
  "node_id": "*** hidden ***",
  "owner": {
    "login": "giantswarm",
    "id": 7556340,
    "node_id": "*** hidden ***=",
    "avatar_url": "https://avatars.githubusercontent.com/u/7556340?v=4",
    "html_url": "https://github.com/giantswarm",
    "gravatar_id": "",
    "type": "Organization",
    "site_admin": false,
    "url": "https://api.github.com/users/giantswarm",
    "events_url": "https://api.github.com/users/giantswarm/events{/privacy}",
    "following_url": "https://api.github.com/users/giantswarm/following{/other_user}",
    "followers_url": "https://api.github.com/users/giantswarm/followers",
    "gists_url": "https://api.github.com/users/giantswarm/gists{/gist_id}",
    "organizations_url": "https://api.github.com/users/giantswarm/orgs",
    "received_events_url": "https://api.github.com/users/giantswarm/received_events",
    "repos_url": "https://api.github.com/users/giantswarm/repos",
    "starred_url": "https://api.github.com/users/giantswarm/starred{/owner}{/repo}",
    "subscriptions_url": "https://api.github.com/users/giantswarm/subscriptions"
  },
  "name": "heartbeatctl",
  "full_name": "giantswarm/heartbeatctl",
  "description": "heartbeatctl is a CLI tool to manage OpsGenie heartbeats",
  "default_branch": "main",
  "created_at": "2020-11-27T17:16:33Z",
  "pushed_at": "2022-05-17T08:46:44Z",
  "updated_at": "2022-05-18T13:01:16Z",
  "html_url": "https://github.com/giantswarm/heartbeatctl",
  "clone_url": "https://github.com/giantswarm/heartbeatctl.git",
  "git_url": "git://github.com/giantswarm/heartbeatctl.git",
  "ssh_url": "git@github.com:giantswarm/heartbeatctl.git",
  "svn_url": "https://github.com/giantswarm/heartbeatctl",
  "language": "Go",
  "fork": false,
  "forks_count": 1,
  "network_count": 1,
  "open_issues_count": 0,
  "open_issues": 0,
  "stargazers_count": 0,
  "subscribers_count": 8,
  "watchers_count": 0,
  "watchers": 0,
  "size": 39,
  "organization": {
    "login": "giantswarm",
    "id": 7556340,
    "node_id": "*** hidden ***=",
    "avatar_url": "https://avatars.githubusercontent.com/u/7556340?v=4",
    "html_url": "https://github.com/giantswarm",
    "type": "Organization",
    "url": "https://api.github.com/users/giantswarm",
    "events_url": "https://api.github.com/users/giantswarm/events{/privacy}",
    "repos_url": "https://api.github.com/users/giantswarm/repos"
  },
  "permissions": {
    "admin": true,
    "maintain": true,
    "pull": true,
    "push": true,
    "triage": true
  },
  "allow_rebase_merge": false,
  "allow_update_branch": true,
  "allow_squash_merge": true,
  "allow_merge_commit": false,
  "allow_auto_merge": true,
  "allow_forking": true,
  "delete_branch_on_merge": true,
  "archived": false,
  "disabled": false,
  "private": false,
  "has_issues": true,
  "has_wiki": false,
  "has_pages": false,
  "has_projects": false,
  "has_downloads": true,
  "is_template": false,
  "url": "https://api.github.com/repos/giantswarm/heartbeatctl",
  "archive_url": "https://api.github.com/repos/giantswarm/heartbeatctl/{archive_format}{/ref}",
  "assignees_url": "https://api.github.com/repos/giantswarm/heartbeatctl/assignees{/user}",
  "blobs_url": "https://api.github.com/repos/giantswarm/heartbeatctl/git/blobs{/sha}",
  "branches_url": "https://api.github.com/repos/giantswarm/heartbeatctl/branches{/branch}",
  "collaborators_url": "https://api.github.com/repos/giantswarm/heartbeatctl/collaborators{/collaborator}",
  "comments_url": "https://api.github.com/repos/giantswarm/heartbeatctl/comments{/number}",
  "commits_url": "https://api.github.com/repos/giantswarm/heartbeatctl/commits{/sha}",
  "compare_url": "https://api.github.com/repos/giantswarm/heartbeatctl/compare/{base}...{head}",
  "contents_url": "https://api.github.com/repos/giantswarm/heartbeatctl/contents/{+path}",
  "contributors_url": "https://api.github.com/repos/giantswarm/heartbeatctl/contributors",
  "deployments_url": "https://api.github.com/repos/giantswarm/heartbeatctl/deployments",
  "downloads_url": "https://api.github.com/repos/giantswarm/heartbeatctl/downloads",
  "events_url": "https://api.github.com/repos/giantswarm/heartbeatctl/events",
  "forks_url": "https://api.github.com/repos/giantswarm/heartbeatctl/forks",
  "git_commits_url": "https://api.github.com/repos/giantswarm/heartbeatctl/git/commits{/sha}",
  "git_refs_url": "https://api.github.com/repos/giantswarm/heartbeatctl/git/refs{/sha}",
  "git_tags_url": "https://api.github.com/repos/giantswarm/heartbeatctl/git/tags{/sha}",
  "hooks_url": "https://api.github.com/repos/giantswarm/heartbeatctl/hooks",
  "issue_comment_url": "https://api.github.com/repos/giantswarm/heartbeatctl/issues/comments{/number}",
  "issue_events_url": "https://api.github.com/repos/giantswarm/heartbeatctl/issues/events{/number}",
  "issues_url": "https://api.github.com/repos/giantswarm/heartbeatctl/issues{/number}",
  "keys_url": "https://api.github.com/repos/giantswarm/heartbeatctl/keys{/key_id}",
  "labels_url": "https://api.github.com/repos/giantswarm/heartbeatctl/labels{/name}",
  "languages_url": "https://api.github.com/repos/giantswarm/heartbeatctl/languages",
  "merges_url": "https://api.github.com/repos/giantswarm/heartbeatctl/merges",
  "milestones_url": "https://api.github.com/repos/giantswarm/heartbeatctl/milestones{/number}",
  "notifications_url": "https://api.github.com/repos/giantswarm/heartbeatctl/notifications{?since,all,participating}",
  "pulls_url": "https://api.github.com/repos/giantswarm/heartbeatctl/pulls{/number}",
  "releases_url": "https://api.github.com/repos/giantswarm/heartbeatctl/releases{/id}",
  "stargazers_url": "https://api.github.com/repos/giantswarm/heartbeatctl/stargazers",
  "statuses_url": "https://api.github.com/repos/giantswarm/heartbeatctl/statuses/{sha}",
  "subscribers_url": "https://api.github.com/repos/giantswarm/heartbeatctl/subscribers",
  "subscription_url": "https://api.github.com/repos/giantswarm/heartbeatctl/subscription",
  "tags_url": "https://api.github.com/repos/giantswarm/heartbeatctl/tags",
  "trees_url": "https://api.github.com/repos/giantswarm/heartbeatctl/git/trees{/sha}",
  "teams_url": "https://api.github.com/repos/giantswarm/heartbeatctl/teams",
  "visibility": "public"
}
INFO[0000] configure repository settings
DEBU[0000] repository settings
{
  "allow_rebase_merge": false,
  "allow_update_branch": true,
  "allow_squash_merge": true,
  "allow_merge_commit": false,
  "allow_auto_merge": true,
  "delete_branch_on_merge": true,
  "archived": false,
  "has_issues": true,
  "has_wiki": false,
  "has_projects": false
}
DEBU[0000] configured repository settings
INFO[0000] grant permission on repository
DEBU[0000] permissions
map[Employees:admin bots:push]
DEBU[0000] grant "admin" permission to "Employees"
DEBU[0000] granted "admin" permission to "Employees"
DEBU[0000] grant "push" permission to "bots"
DEBU[0001] granted "push" permission to "bots"
DEBU[0001] granted permission on repository
INFO[0001] configure protection for "main" branch
DEBU[0001] get commit statuses for "main" branch
DEBU[0001] found 0 commit statuses for "main" branch:
[]
DEBU[0001] branch protection settings
{
  "required_status_checks": null,
  "required_pull_request_reviews": {
    "dismiss_stale_reviews": false,
    "require_code_owner_reviews": false,
    "required_approving_review_count": 1
  },
  "enforce_admins": true,
  "restrictions": null,
  "allow_force_pushes": false,
  "allow_deletions": false
}
DEBU[0001] configured protection for "main" branch
INFO[0001] completed repository setup


The master port is optional now but sets the default as 6066

Hi,
Thanks for making the master port as optional. However when the port is not set its defaulted to 6066. Which is not desirable when the master is sitting behind a proxy.
This is in reference to the bug
Master port is mandatory
One option is to explicitly allow the user to set the port as null. In which case we might have to remove this check in the restclient.build() function

Kindly ignore this issue. It was a mistake on my part


feat: update declaration of Braavos account in genesis

Feature Request
Describe the Feature Request
Currently in genesis we have these Braavos account class hashes
pub const BRAAVOS_ACCOUNT_CLASS_HASH_CAIRO_0: &str =
    "0x0244ca3d9fe8b47dd565a6f4270d979ba31a7d6ff2c3bf8776198161505e8b52";
pub const BRAAVOS_PROXY_CLASS_HASH_CAIRO_0: &str = "0x06a89ae7bd72c96202c040341c1ee422474b562e1d73c6848f08cae429c33262";

When we try to declare a new account using Braavos, we get errors:
{"jsonrpc":"2.0","error":{"code":40,"message":"Contract error"},"id":0}
{"jsonrpc":"2.0","error":{"code":20,"message":"Contract not found"},"id":0}

because the latest wallet expects a different class hash declared.
In Madara logs, the following class hash is not found 0x03ae692aaf1ded26a0b58cf42490f757563850acea887ed57b4894fee8279063
which is a Braavos account as seen on Starkscan
Compiled program link
Add this class hash and code to the genesis contracts deployed on Madara and verify the flow of setting up a new account on Braavos. This issue is also linked to transaction version which blocks both starknet foundry flow and Braavos, but the temporary removal of the is_query check allows to decouple the Braavos flow from starknet foundry.

Hello, may I take this?

Please do!

Hey @ftupas what's the status on this ?

 I started to continue working on this, apologies for the delay

This was recently fixed but broke again with a new Braavos update. Assigning it to myself for now.

@apoorvsadana what is the status here?

Asked on the group, will update

@apoorvsadana can you document what to do?

Ya, so basically, we want to declare Braavos contracts on Madara and check if using the wallet extension are we able to use Braavos with Madara or not. If it works fine, we can add the new contracts to the genesis. I will ask @ocdbytes to pick it up.

I will take this

@ocdbytes thanks a lot

repository archived in favor of https://github.com/madara-alliance/madara


OVN Test: Do not Merge

Signed-off-by: Vishal Thapar<EMAIL_ADDRESS>

Is this still needed?


GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Also, @roger-mike, I appreciate that you've split out the pylint changes into multiple PRs, that's really helpful. Could you also try to distribute review requests for these among other committers that are familiar with the Python SDK? That should get you feedback quicker since they won't all be bottlenecked on me. Here are a few suggestions (in no particular order):

@tvalentyn
@udim
@chamikaramj
@pabloem
@ibzib

oops, I meant to approve last commit, I thought this was a new PR that as @TheNeuralBit suggested.


Updated opennebula im and vmm to 4.14 version

Signed-off-by: Kristián Feldsam<EMAIL_ADDRESS>

Hi, I compare your modifications with original 4.8 files and made those ones in new 4.14 files.

Hi @feldsam
the problem is that ONE has changed some scripts in the new version. We are thinking about how to manage further changes in ONE and the integration with CloudVAMP. One option is to distribute per-version CloudVAMP-ed ONE files, but we think that it is not efficient. We are working on it.
Thank you very much for exposing this problem.

Hi, I think that we can use separate branches for every ONE version, hmm?


improve ball type decision maker

Now if you ran out of pokeballs it will use Great balls instead. If ran out both - use Ultra balls.

I want to this commit be merged :)

shouldn't we wait for Gary's version?

the merge of Gary should be very simple, there're two commits on this line mean users need it heavily.
since https://github.com/PokemonGoF/PokemonGo-Bot/pull/319
is easily and with a keep 20 in stock, we should merge #319

Thanks for the PR, let's close for now.


fix(components): fix bug with italic font in tooltips, add tooltips to speakers catalog buttons

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes # (issue)
Type of change
Please delete options that are not relevant.

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

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] Any dependent changes have been merged and published in downstream modules

Task linked: CU-86c0ty8n5 Tooltips


Create OnFailure method that will be invoked if specific rule fails.

Right now OnAnyFailure can acceps single parameter - Binding Model, same that is used for creating Validator class.
I'd like to log particular validation errors to database, so I this method is one I should use.
For example:
I have three rules:
  RuleFor(u => u.ID)
    .Cascade(CascadeMode.StopOnFirstFailure)
    .NotEmpty().WithMessage("User Id is required")
    .Must(IdentityCardNumberValidator.IsValid).WithMessage("User Id is invalid")
    .OnAnyFailure(LogOnFailure,"UserId");

RuleFor(u => u.Gender)
    .Cascade(CascadeMode.StopOnFirstFailure)
    .NotEmpty().WithMessage("Gender is required")
    .Must(g => g.ToUpper() == "M" || g.ToUpper() == "F").WithMessage("Gender has illegal value");

RuleFor(u => u.PhoneNumber)
    .Cascade(CascadeMode.StopOnFirstFailure)
    .NotEmpty().WithMessage("Phone number is required")
    .Matches("^[0-9]{9}$").WithMessage("Phone number has invalid format")
    .OnAnyFailure(LogOnFailure,"Phone number");

Inside OnAnyFailure I'd like to get those information's:
-field name (second parameter)
-error massage
-value entered by user
-information about other errors.
So ideally I'd like to have this method:
public static IRuleBuilderOptions<T, TProperty> OnAnyFailure<T, TProperty>(this IRuleBuilderOptions<T, TProperty> ruleBuilder, Action<T, TProperty, FluentValidation.Validators.PropertyValidatorContext> onFailure);

This way I would have access to all properties and error messages.
If there is better way please let me know. As I mentioned I'd like to log only particular validation errors.

I'm not sure OnAnyFailure is really what you're looking for in this case, as it's only ever invoked once (ie if there's at least one failure). Would it be more useful to have something that's invoked for each failure?

@JeremySkinner thanks for reply. In my case I'm using Cascade(CascadeMode.StopOnFirstFailure) so that function would be called once for each Rule.
Extension called OnFailure would be even better. This way it would be possible to call specific function when failure occurs, but I really would like to be able to get PropertyValidatorContext there, because I want to log error message to database and current context values and even change Error Message.
I've tried creating extension method that would be called last for each rule, like this:
RuleFor(u => u.PhoneNumber)
    .Cascade(CascadeMode.StopOnFirstFailure)
    .NotEmpty().WithMessage("Phone number is required").OnFailure(LogError)
    .Matches("^[0-9]{9}$").WithMessage("Phone number has invalid format").OnFailure(LogError2,"User {ID} entered {PhoneNumber}"
    .OnAnyFailure(LogOnFailure,"Phone number");

public void LogError(BindingModel model, PropertyValidatorContext context){
   //log to database
}

public void LogError2(BindingModel model, PropertyValidatorContext context, string info){
    //if You would enter 6 as ID and 1234 as PhoneNumber info would be "User 6 entered 1234"
    //log to database
}

Can above be done and do you think it would be useful?
I can try creating something and if I success do PR, but first I need to know where to start.

So you can actually do this already using a decorator that wraps each validator. This is done by making a custom implementation of IPropertyValidator that calls your logging method and delegates to the original underlying validator to do the work.
This can be done with all the existing extensibility points so shouldn't need to do a PR. Here's an example that illustrates how to do it (untested, but hopefully you get the idea):

public class OnFailureValidator<T, TProperty> : FluentValidation.Validators.NoopPropertyValidator {
	private IPropertyValidator _innerValidator;
	private Action<T, TProperty, FluentValidation.Validators.PropertyValidatorContext> onFailure;

	public OnFailureValidator(IPropertyValidator innerValidator, Action<T, TProperty, PropertyValidatorContext> onFailure) {
		_innerValidator = innerValidator;
		this.onFailure = onFailure;
	}

	public override IEnumerable<ValidationFailure> Validate(PropertyValidatorContext context) {
		var results = _innerValidator.Validate(context);

		if (results.Any()) {
			onFailure((T) context.Instance, (TProperty) context.PropertyValue, context);
		}

		return results;
	}
}

You can then wrap the preceding validator like this:
public static class MyExtensions {
	public static IRuleBuilderOptions<T, TProperty> OnFailure<T, TProperty>(this IRuleBuilderOptions<T, TProperty> rule, Action<T, TProperty, PropertyValidatorContext> action) {
		return rule.Configure(cfg => {
			cfg.ReplaceValidator(cfg.CurrentValidator, new OnFailureValidator<T,TProperty>(cfg.CurrentValidator, action));
		});
	}
}

...Usable like this:
RuleFor(x => x.Foo).NotNull().OnFailure((item, propertyval, context) => { /* your code goes here */ });

@JeremySkinner if I understand this correctly what this code do is replacing current validator with custom one that is calling original validator and if anything goes wrong (any validation errors occurs) my custom OnFailure function is called.
I'll try that tomorrow in the morning.
Last thing: Would it be possible to pass third optional argument similar to string inside WithMessage method. I'd like to be able to format message that will be passed as argument to my OnFailure function. Like shown in my previous comment (LogError2 function)

Yes, that should be pretty easy to include...just modify the signature for
the method and then pass it into the custom validator's constructor. You
can then use it inside the Validate method.
On Tuesday, 19 July 2016, Tomek<EMAIL_ADDRESS>wrote:

@JeremySkinner https://github.com/JeremySkinner if I understand this
correctly what this code do is replacing current validator with custom one
that is calling original validator and if anything goes wrong (any
validation errors occurs) my custom OnFailure function is called.
I'll try that tomorrow in the morning.
Last thing: Would it be possible to pass third optional argument similar
to string inside WithMessage method. I'd like to be able to format
message that will be passed as argument to my OnFailure function. Like
shown in my previous comment (LogError2 function)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/JeremySkinner/FluentValidation/issues/299#issuecomment-233740764,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAFgEsZA2HEQggHC0kEohDbDG-WG9L7pks5qXSYtgaJpZM4JOnHl
.

I'm unsure how to fill that message placeholders with data.
I've created this rule:
    RuleFor(u => u.BirthDate)
        .Cascade(CascadeMode.StopOnFirstFailure)
        .NotNull().WithMessage("This field is required")
        .Must(c => c > new DateTime(1920, 01, 01)).WithMessage("Can't be that old").OnFailure(OnDateError,"User {0} entered {1}",u=>u.ID,u=>u.BirthDate);

and my OnDataError function:
private void OnDateError(CreateAccountBindingModel model, DateTime birthDate, PropertyValidatorContext ctx, string message)
{
    Debug.WriteLine(message);
}

Unfortunately message isn't filled.
Here is my validator:
public class OnFailureValidator<T, TProperty> : NoopPropertyValidator
{
    private readonly IPropertyValidator _innerValidator;
    private readonly Action<T, TProperty, PropertyValidatorContext> _onFailureSimple;
    private readonly Action<T, TProperty, PropertyValidatorContext,string> _onFailureComplex;
    private readonly string _message;

    public OnFailureValidator(IPropertyValidator innerValidator, Action<T, TProperty, PropertyValidatorContext,string> onFailure, string message)
    {
        _innerValidator = innerValidator;
        _onFailureComplex = onFailure;
        _message = message;
    }

    public OnFailureValidator(IPropertyValidator innerValidator, Action<T, TProperty, PropertyValidatorContext> onFailure)
    {
        _innerValidator = innerValidator;
        _onFailureSimple = onFailure;
    }

    public override IEnumerable<ValidationFailure> Validate(PropertyValidatorContext context)
    {
        var results = _innerValidator.Validate(context);

        if (results.Any())
        {
            if(string.IsNullOrWhiteSpace(_message))
            {
                _onFailureSimple((T)context.Instance, (TProperty)context.PropertyValue, context);
            }
            else
            {
                _onFailureComplex((T) context.Instance, (TProperty) context.PropertyValue, context, _message);
            }
        }

        return results;
    }
}

and my extension method
public static class FluentValidationExtensions
{
    public static IRuleBuilderOptions<T, TProperty> OnFailure<T, TProperty>(this IRuleBuilderOptions<T, TProperty> rule, Action<T, TProperty, PropertyValidatorContext> action)
    {
        return rule.Configure(cfg =>
        {
            cfg.ReplaceValidator(cfg.CurrentValidator, new OnFailureValidator<T, TProperty>(cfg.CurrentValidator, action));
        });
    }

    public static IRuleBuilderOptions<T, TProperty> OnFailure<T, TProperty>(this IRuleBuilderOptions<T, TProperty> rule, Action<T, TProperty, PropertyValidatorContext, string> action, string message)
    {
        return rule.Configure(cfg =>
        {
            cfg.ReplaceValidator(cfg.CurrentValidator, new OnFailureValidator<T, TProperty>(cfg.CurrentValidator, action, message));
        });
    }

    public static IRuleBuilderOptions<T, TProperty> OnFailure<T, TProperty>(this IRuleBuilderOptions<T, TProperty> rule, Action<T, TProperty, PropertyValidatorContext, string> action, string message, params object[] messageArgs)
    {
        return rule.Configure(cfg =>
        {
            cfg.ReplaceValidator(cfg.CurrentValidator, new OnFailureValidator<T, TProperty>(cfg.CurrentValidator, action, message));
        });
    }

    public static IRuleBuilderOptions<T, TProperty> OnFailure<T, TProperty>(this IRuleBuilderOptions<T, TProperty> rule, Action<T, TProperty, PropertyValidatorContext, string> action, string message, params Func<T, object>[] funcs)
    {
        return rule.Configure(cfg =>
        {
            cfg.ReplaceValidator(cfg.CurrentValidator, new OnFailureValidator<T, TProperty>(cfg.CurrentValidator, action, message));
        });
    }

    public static IRuleBuilderOptions<T, TProperty> OnFailure<T, TProperty>(this IRuleBuilderOptions<T, TProperty> rule, Action<T, TProperty, PropertyValidatorContext, string> action, string message, params Func<T, TProperty, object>[] funcs)
    {
        return rule.Configure(cfg =>
        {
            cfg.ReplaceValidator(cfg.CurrentValidator, new OnFailureValidator<T, TProperty>(cfg.CurrentValidator, action, message));
        });
    }
}

I know I'm not formatting that message, but unfortunately I don't know how to do that, could You guide me on how to do that?
Is OnFailureValidator correct? I'v created two actions (_onFailureSimple and _onFailureComplex) but maybe this can be done with only one?

I've managed to create this extension class:
public static class FluentValidationExtensions
{
    public static IRuleBuilderOptions<T, TProperty> OnFailure<T, TProperty>(this IRuleBuilderOptions<T, TProperty> rule, Action<T, PropertyValidatorContext> action)
    {
        return rule.Configure(cfg =>
        {
            cfg.ReplaceValidator(cfg.CurrentValidator, new OnFailureValidator<T, TProperty>(cfg.CurrentValidator, action));
        });
    }

    public static IRuleBuilderOptions<T, TProperty> OnFailure<T, TProperty>(this IRuleBuilderOptions<T, TProperty> rule, Action<T, PropertyValidatorContext,string> action, string message)
    {
        return rule.OnFailure(action, message, null as object[]);
    }

    public static IRuleBuilderOptions<T, TProperty> OnFailure<T, TProperty>(this IRuleBuilderOptions<T, TProperty> rule, Action<T, PropertyValidatorContext,string> action, string message, params object[] messageArgs)
    {
        var funcs = ConvertArrayOfObjectsToArrayOfDelegates<T>(messageArgs);
        return rule.OnFailure(action, message, funcs);
    }

    public static IRuleBuilderOptions<T, TProperty> OnFailure<T, TProperty>(this IRuleBuilderOptions<T, TProperty> rule, Action<T, PropertyValidatorContext, string> action, string message, params Func<T, object>[] funcs)
    {
        Func<T, TProperty, object>[] tmp = funcs.Select(func => new Func<T, TProperty, object>((instance, value) => func(instance))).ToArray();
        return rule.OnFailure(action, message, tmp);
    }

    public static IRuleBuilderOptions<T, TProperty> OnFailure<T, TProperty>(this IRuleBuilderOptions<T, TProperty> rule, Action<T, PropertyValidatorContext, string> action, string message, params Func<T, TProperty, object>[] funcs)
    {
        return rule.Configure(cfg =>
        {
            cfg.ReplaceValidator(cfg.CurrentValidator, new OnFailureValidator<T, TProperty>(cfg.CurrentValidator, action, message,funcs));
        });
    }

    static Func<T, object>[] ConvertArrayOfObjectsToArrayOfDelegates<T>(object[] objects)
    {
        if (objects == null || objects.Length == 0)
        {
            return new Func<T, object>[0];
        }
        return objects.Select(obj => new Func<T, object>(x => obj)).ToArray();
    }
}

and this validator:
public class OnFailureValidator<T, TProperty> : NoopPropertyValidator
{
    private readonly IPropertyValidator _innerValidator;
    private readonly Action<T, PropertyValidatorContext> _onFailureSimple;
    private readonly Action<T, PropertyValidatorContext,string> _onFailureComplex;
    private readonly string _message;
    private readonly Func<T, TProperty, object>[] _funcs;

    public OnFailureValidator(IPropertyValidator innerValidator, Action<T, PropertyValidatorContext, string> onFailure, string message, Func<T, TProperty, object>[] funcs)
    {
        _innerValidator = innerValidator;
        _onFailureComplex = onFailure;
        _message = message;
        _funcs = funcs;
    }

    public OnFailureValidator(IPropertyValidator innerValidator, Action<T, PropertyValidatorContext> onFailure)
    {
        _innerValidator = innerValidator;
        _onFailureSimple = onFailure;
    }

    public override IEnumerable<ValidationFailure> Validate(PropertyValidatorContext context)
    {
        var results = _innerValidator.Validate(context);

        if (results.Any())
        {
            if(string.IsNullOrWhiteSpace(_message))
            {
                _onFailureSimple((T)context.Instance, context);
            }
            else
            {
                var messageFormatter = new MessageFormatter();
                messageFormatter.AppendPropertyName(context.PropertyName);
                messageFormatter.AppendArgument("PropertyValue", context.PropertyValue);
                try
                {
                    messageFormatter.AppendAdditionalArguments(
                        _funcs.Select(func => func((T) context.Instance, (TProperty) context.PropertyValue)).ToArray());
                }
                catch (Exception ex)
                {
                    Debug.WriteLine(ex);
                }

                var msg = messageFormatter.BuildMessage(_message);
                _onFailureComplex((T) context.Instance, context, msg);
            }
        }

        return results;
    }
}

This way I can create this rule:
RuleFor(u => u.BirthDate)
    .Cascade(CascadeMode.StopOnFirstFailure)
    .NotNull().WithMessage("This field is required")
    .Must(c => c > new DateTime(1920, 01, 01)).WithMessage("Can't be that old").OnFailure(OnError,"User {0} entered {1}",u=>u.ID,u=>u.BirthDate);

and have single OnError function that will log what I want. Below is that function:
private void OnError(CreateAccountBindingModel model, PropertyValidatorContext ctx, string message)
{
    Debug.WriteLine("{0} - {1}", ctx.PropertyName, ctx.PropertyValue);
    Debug.WriteLine(message);
}

I've remove TProperty from my code, because with it I couldn't create single OnError function for all properties, because they had different types.
I've tested it a bit in development environment, but if You could leview it I will be grateful.
I would like to remove:
private readonly Action<T, PropertyValidatorContext> _onFailureSimple;
private readonly Action<T, PropertyValidatorContext,string> _onFailureComplex;

but I don't know if this is even possible. Also I'm not sure if MessageFormatter is properly used.

@JeremySkinner Could You please take a look at this? Maybe I can create PR if You'd have that functionality in core.

Take a look at the code for PropertyValidator.cs- it shows how the MessageFormatter is used to build the message within the Validate message:
https://github.com/JeremySkinner/FluentValidation/blob/master/src/FluentValidation/Validators/PropertyValidator.cs

@JeremySkinner I know this thread is almost 2 years old, but still OnFailure is missing.
As I mentioned I have build extension methods that allow me to specify callback when a specific rule fails.
I'm using it for almost two years so if You are interested I can create PR with it.

@Misiu Yes if you'd like to create a PR I'd be happy to review it. Please note that 8.0 development is under way so please base your changes on the 8.0-preview01 branch. Note that the internals have changed significantly in some areas, so let me know if there's anything you'd like me to clarify.
Please create a new issue to continue this discussion, rather than using this old issue.


Fix Size Based Rotation

Restoring new version of CustomSizeAndTimeBasedArchiveRemover to fix new problems with Logback's size based rotation. Originally introduced in #24 and then removed in #44, this new version fixes issues with the latest 1.1.7 release of Logback.

You can see it in action here: https://github.com/ArpNetworking/metrics-client-java/pull/30


Increasing world size only adds ocean beyond vanilla limits.

Running into a similar thing. except my world is mostly covered in Ashlands sea.


STREAMZ-289: Upgraded version of python in Travis CI from 3.6 -> 3.7.…

… This closes: 289

To be fair do we actually use the python version that travis supplies? (since we are now using the envrionment.yaml)

Honestly I’m not enough of a Travis expert to know if it’s using that in some other capacity or not? Thought maybe just be on the safe side
Get Outlook for iOShttps://aka.ms/o0ukef

From: Christopher J. Wright<EMAIL_ADDRESS>Sent: Sunday, December 8, 2019 9:44 PM
To: python-streamz/streamz
Cc: Jeremy Dyer; Author
Subject: Re: [python-streamz/streamz] STREAMZ-289: Upgraded version of python in Travis CI from 3.6 -> 3.7.… (#290)
To be fair do we actually use the python version that travis supplies? (since we are now using the envrionment.yaml)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/python-streamz/streamz/pull/290?email_source=notifications&email_token=AAQHLA47PMWEJK4RV7QRQO3QXWWIRA5CNFSM4JYBVWGKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEGHUVHY#issuecomment-563038879, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAQHLA2ROEBKTSRX5DRGJ6DQXWWIRANCNFSM4JYBVWGA.


add additional factor dims in t4 and variance

According to discussions on #10 #9 .
https://github.com/bobhancock/goxmeans/blob/master/doc/BIC_notes.pdf
I add a dimension factor in t4 and variance.

Looks good to me! Thanks for the contribution.


[BUG] example.txt included in cold package

Expected Behavior
example.txt should not be included in the cold package
Observed Behavior
example.txt is in the cold package
Steps to Reproduce
compile the program, use your favorite decompiler to view example.txt in the LemLib.a binary
Environment

LemLib Version: 0.5.0-rc1 and newer
PROS Kernel Version: 3.8.0
PROS CLI Version: 3.4.3
Additional Templates: None

Additional Information
New users are confused when their program does not compile if they have example.txt in the static folder.
Potential Fix
Modify asset.mk so it will add assets from the cold_static folder to the cold package, and add assets from the static folder into the hot package

@SizzinSeal This can be closed


agent.service.deregister failed when service id contains '#'

test on macOS 10.12.2 and centos 6.5(final)

consul 0.7.5
python: 2.7.13
python-consul: 0.7.0

for example:
result = consul_client.agent.service.deregister("app#<IP_ADDRESS>#3000")

result is true, bug deregister failed.
consul log is :
2017/07/08 09:56:48 [WARN] agent: Failed to deregister service "app": Service does not exist

c.health.service("app#<IP_ADDRESS>#3000") is also an issue

Because fragments were not sent in HTTP request messages, consul received app instead of app#<IP_ADDRESS>#3000. So it's better to quote using Percent-encoding before send directly.
Sample code below:
https://github.com/cablehead/python-consul/blob/master/consul/base.py#L58
from six.move import urllib_parse

url = urllib_parse.quote(url, safe='/:')
ret = {'http': url, 'interval': interval}

I have a fix based on @trdcaz 's comment. However, note that this does introduce the possibility that other RFC "safe" characters may need to be added to the safe list later on. However, I cannot think of a case off the top of my head.
In addition, this issue also raises another concern in that the service.deregister is returning True. However, this is sort of expected as this is the behaviour of the server itself. Considering we intend to provide a lightweight library, error checking can be left as an exercise for application developers - but I do think we need to document this behaviour if not done so already.
$ curl -v -X PUT -d '{"name": "foobar"}' 'http://localhost:8500/v1/agent/service/register'
*   Trying ::1...
* TCP_NODELAY set
* Connected to localhost (::1) port 8500 (#0)
> PUT /v1/agent/service/register HTTP/1.1
> Host: localhost:8500
> User-Agent: curl/7.53.1
> Accept: */*
> Content-Length: 18
> Content-Type: application/x-www-form-urlencoded
>
* upload completely sent off: 18 out of 18 bytes
< HTTP/1.1 200 OK
< Date: Mon, 31 Jul 2017 07:10:28 GMT
< Content-Length: 0
< Content-Type: text/plain; charset=utf-8
<
* Connection #0 to host localhost left intact

$ curl -v -X PUT 'http://<IP_ADDRESS>:8500/v1/health/service/deregister/foobar'
*   Trying <IP_ADDRESS>...
* TCP_NODELAY set
* Connected to <IP_ADDRESS> (<IP_ADDRESS>) port 8500 (#0)
> PUT /v1/health/service/deregister/foobar HTTP/1.1
> Host: <IP_ADDRESS>:8500
> User-Agent: curl/7.53.1
> Accept: */*
>
< HTTP/1.1 200 OK
< Content-Type: application/json
< X-Consul-Index: 9
< X-Consul-Knownleader: true
< X-Consul-Lastcontact: 0
< Date: Mon, 31 Jul 2017 07:11:04 GMT
< Content-Length: 2
<
* Connection #0 to host <IP_ADDRESS> left intact
[]

$ curl -v -X PUT 'http://<IP_ADDRESS>:8500/v1/health/service/deregister/foobar'
*   Trying <IP_ADDRESS>...
* TCP_NODELAY set
* Connected to <IP_ADDRESS> (<IP_ADDRESS>) port 8500 (#0)
> PUT /v1/health/service/deregister/foobar HTTP/1.1
> Host: <IP_ADDRESS>:8500
> User-Agent: curl/7.53.1
> Accept: */*
>
< HTTP/1.1 200 OK
< Content-Type: application/json
< X-Consul-Index: 9
< X-Consul-Knownleader: true
< X-Consul-Lastcontact: 0
< Date: Mon, 31 Jul 2017 07:11:10 GMT
< Content-Length: 2
<
* Connection #0 to host <IP_ADDRESS> left intact


fix: xor with boolean (#409)

Fixes #409

@asashour thank you!
You changing code so fast, this is great! And I think I need to merge type-inference-wip branch or it will be hard to resolve conflicts. Be prepared for changes ))

I would say, once you have a green build, then merge it.  There are not so many resources available, to support two evolving branches.


New package: Ritlabs.TheBat version 11.2.2

Checklist for Pull Requests

[x] Have you signed the Contributor License Agreement?
[x] Resolve #177788

Manifests

[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

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

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

Microsoft Reviewers: Open in CodeFlow

Verify:

The splash screen is fast and I had to pause the VM to catch it.


[Feature] Introduce Help Screen.

Introduce Help Screen.
Content in Help Screen :
Create ExpansionTile view for below 3 QNA.

If I rate someone, will they know how many stars I gave them?
No, they will only receive a notification that you rated them.

How can I disable my location from showing on the map?
You can disable your location in the privacy settings.

How can I hide my profile picture?
You can hide it in the privacy settings.

Reference for UI :https://pub.dev/packages/expansion_tile_group/versions/0.0.1

Plz Assign to me


feat(readme): updated README

What:
Updated README

Small stuff but LGTM

LGTM. I realized that this plugin assumes that request.auth.credentials.api_key is set, but this should be generalized so that users can setup a redis key based on their own application specifications. https://github.com/lob/hapi-rate-limit/blob/master/lib/index.js#L24
In a future PR, we should generalize this. Let's discuss IRL the best way to do this @siddhantdange @robinjoseph08.


More options in config file.

Thanks for the PR. Makes sense to support more parameters from the config. However, by adding them in the extension as optional parameters with default values, do you not introduce a breaking change? Would adding a new overloaded method be a better option?

I've used the same defaults as in ElasticsearchSinkOptions, but I can change it to an overload.

I can't make an overload because of the optional parameters (ambiguous error).

Ok, then we use the optional parameters. Thanks for the PR, I will merge it to dev.


Rate limits for Websocket trading

Hi! Looks like there is no built-in support for rate limiting in the case of WebSocket spot trading. Here is an axample of the exchange response if I sent Place/Amend requests more often than 10/1s.
{"header":{"response_time":"1728899982230","status":"429","channel":"spot.order_amend","event":"api","client_id":"client_id","conn_id":"5c87dfa92caf9951","trace_id":"83169a6f64116902cb7a6c260fa868be"},"data":{"errs":{"label":"TOO_MANY_REQUESTS","message":"Request Rate limit Exceeded"}},"request_id":"161"}

client_id was changed by me, other field as I got in response.

Hi, I can't find any info on this rate limit for the client websocket. Do you know where the 10/1s rule is defined?
If this is a specific rate limit for order placement/editing I also think it should not be part of the client rate limiting as you don't want to delay these calls, and it's better to let the server decide whether it's within or outside the limit.

I can't find any documentation about WebSocket rate limits. The gateio AI-support-bot says there are no limits, the only limit is a limit of 300 connections at a time. But it looks like the placing/amending  order rules for websocket is the same as for REST https://www.gate.io/docs/developers/apiv4/en/#trade-ratio-rate-limiting.


Add docs about customizing http response for OData action and OData from microflow

WIP docs for the feature.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Dima Fedorenko seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Hi @dfedorenko , are these updates for 10.16?

@dfedorenko @joostverhoog Is this PR still in progress or is it ready for review?

@quinntracy it is coming shortly


⚠ ETF has degraded performance

In 7d05027, ETF (https://markets.hankyung.com/equity-product/etf) experienced degraded performance:

HTTP code: 200
Response time: 9974 ms

Resolved: 비트코인 performance has improved in db1d1d8.


Fix issue with wrong using index instead of camera_index in opencv

Fix issue with wrong usage of OpenCVCamera
Examples:

Title
Label

The camera check break with AttributeError: 'OpenCVCamera' object has no attribute 'index'
(🐛 Bug)

How it was tested
Follow the instructions in https://github.com/huggingface/lerobot/blob/main/examples/7_get_started_with_real_robot.md
The main branch will break at running the following script -
python lerobot/common/robot_devices/cameras/opencv.py \
    --images-dir outputs/images_from_opencv_cameras

I see similar comment here - https://github.com/huggingface/lerobot/pull/410/files#diff-b5b8202443d3e86c655080c767525695923a3837e3e1c807440a7140961556d1R112

@helper2424 Thank you! I will improve our unit test to capture this.


How to set width for Columns

Hi,
Please help me. How to set width for Columns for excel file export.
Thank you

How to set columns autofit

Hi,
I want to convert json to .xlsx format buffer (I guess I should use this). I want to set width of excel  columns as well

I also want to know the method

automatic cell width you can try to see it
https://github.com/dave-wind/excel

Tried both wch or width, neither works
var wscols = [ {wch:20}, {wch:30}, {wch:40}, {wch:30},{wch:10}, {wch:10}];
ws['!cols'] = wscols;
var wscols = [ {width:20}, {width:30}, {width:40}, {width:30},{wch:10}, {width:10}];
ws['!cols'] = wscols;

Tried both wch or width, neither works
var wscols = [ {wch:20}, {wch:30}, {wch:40}, {wch:30},{wch:10}, {wch:10}];
ws['!cols'] = wscols;
var wscols = [ {width:20}, {width:30}, {width:40}, {width:30},{wch:10}, {width:10}];
ws['!cols'] = wscols;

Hi have the same problem. Tried both wch and width, neither works.
Any suggestions?
Thanks.

It worked for me doing this:
var wscols = [ {wch:20}, {wch:30}, {wch:40}, {wch:30},{wch:10}, {wch:10}];
ws["Sheets"]["Sheet1"]['!cols'] = wscols;

@MWAN001 @Luperco please test against the latest version.  e59de73f99e63624a6447434db9ddcdec3a6a408 added support for ODS

Perhaps this helps: https://github.com/SheetJS/sheetjs/issues/1473#issuecomment-1641573655


Changes to get select to work

Still need to fix tests but looks like that will need to be a separate branch to get tests to run in order

My pull request LGTM


Direction of player changing incorrectly when interacting with platforms roofs while jumping

https://github.com/shoc71/dont-stop-now/assets/129419169/2b43a540-0751-45ec-90a0-418eb94702ec
https://github.com/shoc71/dont-stop-now/assets/129419169/1ec01f27-fc54-4ff1-b236-495638b012ba

there are also moments where this happens in "letters" with the letter e middle tooth - more testing necessary

I suspect the new changes being made to change the setting of the player vaule of 1 or -1 that could affect the velocity of this thing as before it was "self.right" or "self.left". Test from your side

v.0.1.8 release for collision bug

The fix has been released in v.0.1.8 fix 3. The problem included not recording x positions and hitbox being too large, which allowed the change in left and right to occur at unexpected areas.


request for update

You can update data for Age of empires II - definitive edition?

Agreed. using this for custom database, an API is missing half of the civs in AEO2de.

Hey guys, big fan of this API. Would love to see this updated with Age 2 DE content :)

Please see this PR for updates on Age 2 civs. #14

Hello! thanks @AndyTheNerd for taking care of this! When I first created this it was a pet project to get myself around some Python and haven't had the time to go back at it much. It's great to see people using it :) if you feel that something is missing, feel free to add them in the .csv


Use a more uniform structure for cython classes

Currently we have two patterns for Cython classes and it would be nice to be more consistent
Subclass pattern
Have a Cython class as a superclass of the Python class
cdef class cyClass:
    ...

class Class:
    ...

examples: cyVariables and Variables, cyConstrainedQuadraticModel and ConstrainedQuadraticModel
The major drawback of this pattern is it does not allow multiple data types, because Cython does not allow templated classes (nor is such a concept really meaningful).
Attribute pattern
Have the Python class hold the Cython class as an attribute
cdef class cyClass
    ...

class Class:
    def __init__(self):
        self.data = cyClass()

examples: cyBQM and BQM
This pattern gives us the ability to support multiple underlying data types, but IMO it's harder to understand and requires a lot of indirect methods like
def f(self):
    return self.data.f()

which adds clutter and maintenance and creates a performance problem (partially mititgated by forwarding_method).

One potential third option would be something like
import abc

import numpy as np

class BQM_float64:
    pass

class BQM_float32:
    pass

class BQM(abc.ABC):
    def __new__(cls, *args, dtype=np.float64, **kwargs):
        if dtype == np.float32:
            return BQM_float32(*args, **kwargs)
        elif dtype == np.float64:
            return BQM_float64(*args, **kwargs)
        else:
            raise RuntimeError

BQM.register(BQM_float64)
BQM.register(BQM_float32)

where we create a stub BQM class that just constructs one of the two other datatype classes. Which themselves can inherit from the Cython classes.
That said, this has some issues. Namely it's a bit surprising for a user, and the docs etc will break.


Setter for coap code for message

This PR contains the code for issue-47.

hey again @jkralik ,
do you know why TestServingIPv4MCastBlockWiseSzx256 seems to fail in the CI. I couldn't get the connection between my change and the failure.
thanks,

Thx fox contributing. The problem is test of muticast - some of coap server respond with default handler (response has code not found) - problem with clearing.


AWS Role added to all authentications

Terraform user issue closes #77

Out of date as our deploy approach has changed.


nixos/prometheus: systemd hardening for alertmanager/pushgateway

Description of changes
This PR hardens the systemd services for AlertManager/PushGateway/webhook-logger.
Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[X] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[X] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

looks good but out of curiosity where did you get these hardening settings from... trial and error, or some external suggestion, etc...?

Looks like there is systemd-analyze security now which I didn't knew before this PR added them to the tests.

@aanderse Yep, simply sticking that systemd-analyze in the tests and iterating until there's a balance between security settings and working E2E tests.

thanks everyone!


Bugfix/pow withcomponents

The code was not working with a TB with more than one component, now also, the code is cleaner.
I added a test for the specific case and ported the test to pytest

:books: Documentation preview :books:: https://equistore--210.org.readthedocs.build/en/210/

Just merged #208, so this branch will require a rebase and addition of a small inplace test @DavideTisi . Once done I'm happy for you to merge this branch provided CI is happy :)


[SNOW-1844465] Avoid creating a CTE out of simple select start on top of a select entity

Which Jira issue is this PR addressing? Make sure that there is an accompanying issue to your PR.
<!---
In this section, please add a Snowflake Jira issue number.
<p>Note that if a corresponding GitHub issue exists, you should still include
the Snowflake Jira issue number. For example, for GitHub issue
https://github.com/snowflakedb/snowpark-python/issues/1400, you should
add "SNOW-1335071" here.
--->
Fixes SNOW-1844465

Fill out the following pre-review checklist:

[x] I am adding a new automated test(s) to verify correctness of my new code

[ ] If this test skips Local Testing mode, I'm requesting review from @snowflakedb/local-testing

[ ] I am adding new logging messages
[ ] I am adding a new telemetry message
[ ] I am adding new credentials
[ ] I am adding a new dependency
[ ] If this is a new feature/behavior, I'm adding the Local Testing parity changes.
[x] I acknowledge that I have ensured my changes to be thread-safe. Follow the link for more information: Thread-safe Developer Guidelines

Please describe how your code solves the related issue.
Please write a short description of how your code change solves the related issue.

pandas daily job started https://ci-dev-142.int.snowflakecomputing.com/job/SnowparkPythonSnowPandasDailyRegressRunner/1087/
snowpark daily job started https://github.com/snowflakedb/snowpark-python/actions/runs/12170949440


Implement SMS Templates for Auth

Currently the SMS that goes out says "Your code is 000000"
Ideally we are able to customize a template and insert the code as a variable:
Nhost - Here is your code: {{verification_code}}. Don't share it with anyone
To push things even further, you could even make Auth variables available for the templates:
{{first_name}}, here is your code: {{verification_code}}. Don't share it with anyone

A AUTH_SMS_TEMPLATE may be the quickest way forward, but we would miss translations. Eventually we will likely need to implement the same system as for email templates

Can I pick this up next days? I can implement the template approach.

Yes, definitely!


Test GuiObject with multiple images of different sizes

Test whether a GuiObject can have images with different sizes and how thats handled. For example: what happens if a button has an image of one size for its default state, and an image with a different size for its pressed state? How is this reflected in the UI and in calls to getWidth/getHeight?

Closing all modern skin rendering related issues since I don't plan to work on it in the near term.


auto-mt threadsafe garbage collector, noise words, fixes, etc.

Don't pull this yet, but when you have a minute, start taking a look/review.  And try a build on the Mac if you want.  Will give a warning on first builds of rogo/roguec, but subsequent ones should be fine.
Here's a test program.
Sidenote: I have no idea what the ordering of commits is in "Conversation" view, but it's awful.  Switch to the "Commits" tab if you want it to make any sense at all.

Replacing with new PR.


sphinxcontrib-bibtex: patch prematurely uploaded 2.6.0 and 2.6.1 packages

Ohh this yaml patching is actually quite nice 😄. Been away for two years or so and suddenly everything is so fancy and shiny 😬
Read 56 total patch yaml docs
Downloading: https://conda.anaconda.org/conda-forge/noarch/repodata_from_packages.json.bz2
Downloading: https://conda.anaconda.org/conda-forge/noarch/repodata.json.bz2
noarch::sphinxcontrib-bibtex-2.6.0-pyhd8ed1ab_0.conda
-    "dataclasses",
-    "docutils >=0.8",
+    "docutils >=0.8,!=0.18.*,!=0.19.*",
-    "python >=3.6",
-    "sphinx >=2.1"
+    "python >=3.7",
+    "sphinx >=3.5"
noarch::sphinxcontrib-bibtex-2.6.1-pyhd8ed1ab_0.conda
-    "dataclasses",
-    "docutils >=0.8",
+    "docutils >=0.8,!=0.18.*,!=0.19.*",
-    "python >=3.6",
-    "sphinx >=2.1"
+    "python >=3.7",
+    "sphinx >=3.5"
Downloading: https://conda.anaconda.org/conda-forge/linux-64/repodata_from_packages.json.bz2
Downloading: https://conda.anaconda.org/conda-forge/linux-64/repodata.json.bz2
Downloading: https://conda.anaconda.org/conda-forge/linux-armv7l/repodata_from_packages.json.bz2
Downloading: https://conda.anaconda.org/conda-forge/linux-armv7l/repodata.json.bz2
Downloading: https://conda.anaconda.org/conda-forge/linux-aarch64/repodata_from_packages.json.bz2
Downloading: https://conda.anaconda.org/conda-forge/linux-aarch64/repodata.json.bz2
Downloading: https://conda.anaconda.org/conda-forge/linux-ppc64le/repodata_from_packages.json.bz2
Downloading: https://conda.anaconda.org/conda-forge/linux-ppc64le/repodata.json.bz2
Downloading: https://conda.anaconda.org/conda-forge/osx-64/repodata_from_packages.json.bz2
Downloading: https://conda.anaconda.org/conda-forge/osx-64/repodata.json.bz2
Downloading: https://conda.anaconda.org/conda-forge/osx-arm64/repodata_from_packages.json.bz2
Downloading: https://conda.anaconda.org/conda-forge/osx-arm64/repodata.json.bz2
Downloading: https://conda.anaconda.org/conda-forge/win-32/repodata_from_packages.json.bz2
Downloading: https://conda.anaconda.org/conda-forge/win-32/repodata.json.bz2
Downloading: https://conda.anaconda.org/conda-forge/win-64/repodata_from_packages.json.bz2
Downloading: https://conda.anaconda.org/conda-forge/win-64/repodata.json.bz2

Diff seems good to me, @jakirkham, @AA-Turner, @chohner can you please double-check?
Checklist

[x] Used a static YAML file for the patch if possible (instructions).
[x] Only wrote code directly into generate_patch_json.py if absolutely necessary.
[x] Ran python show_diff.py and posted the output as part of the PR.
[x] Modifications won't affect packages built in the future.

<!-- Put any other comments or information here --!>

Thanks Christian! 🙏
Matt is helping make the repodata patching process smoother 🙂


No copy path for (non-workspace) file

opened a log file
missing reveal in finder
missing copy path

Sandy can you check, I think this came in from https://github.com/microsoft/vscode/issues/40575. I wonder why we don't just open the file on disk and why this needs a custom log viewer. We can mark the editor as readonly.

also related to https://github.com/microsoft/vscode/issues/192271

Yeah, this is opened as log viewer to make it readonly. If we can mark readonly for normal files, then we do not need this spl log viewer.

@sandy081 I think there are 2 options here:

make the setting files.readonlyInclude dynamic by adding the logs folder by default
ad-hoc configure the logs folder or logs file to open as readonly via the new IFilesConfigurationService.updateReadonly method

I think I would go with the second choice because that is quite straightforward.

Adopted to ad-hoc readonly state change, but it is allowing user to write into the file

Is it possible to prevent that? I know user can open the file directly from the folder and can edit, but I would like to prevent this action when opened using the action in output

currently not, but I don't think its a big deal


There is no “label for” property for visual instructions

URL: http://clarity-ks.surge.sh/radios
Tools: Web Accessibility Toolbar (WAT2.0) / ARIA Markup
Reproduced Steps:

Login Clarity Kitchen Sink with web browser.
Go to Radios tab.
Check the markup in Radios tab.

Actual Results:
There are no labels associating "Radio Buttons" text, “Default/Stacked radio button group” text, “Inline radio button group” text and “These are disabled radio buttons” text.
For more details, please refer to "Radios-NoMarkup.zip".
Expected Results:
Set “label for” property for "Radio Buttons" text, “Default/Stacked radio button group” text, “Inline radio button group” text and “These are disabled radio buttons” text.
Note:
This issue occurs on other tabs, such as Progress tab, Radios tab, Wizards tab.
Violation of Section 508 Standards:
Section 1194.22 (n) When electronic forms are designed to be completed on-line, the form shall allow people using assistive technology to access the information, field elements, and functionality required for completion and submission of the form, including all directions and cues.
User Impact: Blind.
Screen shot:

Radios-NoMarkup.png

LabelForAndID'sMarkup.png

NOTE: this may also be a problem on the website

As we're not using fieldsets, which is an option to consider, I've added  role="radiogroup" aria-labelledby="label_id" to the form-group divs.
It works well with screen readers, the group label is read when changing between radio groups.


AMBARI-25545: fix downloaded zip-file name shows empty in the fileview page

What changes were proposed in this pull request?
(Please fill in changes proposed in this fix)
How was this patch tested?
(Please explain how this patch was tested. Ex: unit tests, manual tests)
(If this patch involves UI changes, please attach a screen-shot; otherwise, remove this)
Please review Ambari Contributing Guide before opening a pull request.

Should reopen pr, this pr maybe incorrect.


Remove: Feixiang.Feixiang version 3.0.1

403 Forbidden
Checklist for Pull Requests

[x] Have you signed the Contributor License Agreement?
[ ] Is there a linked Issue?

Manifests

[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.6 schema?

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

Microsoft Reviewers: Open in CodeFlow

URL: https://dl1.flyele.net/application/windows32/3.0.1/flyele_windows32_prod_3.0.1-7336c04952_guangwang.exe
Response status code does not indicate success: 403 (Forbidden).
(Automated message - build 876)


Correct way to use ngFor loop on selector directive? It doesn't seem to work.

      <mz-card *ngFor="let Package of packages; last as isLast; odd as isOdd" class="col-12" [class.last-col]="isOdd">

Package Name:
               {{ Package.Name }}

               Description:
               {{ Package.Description }}

               Price:
               ${{ Package.OriginalPackagePrice }}

               Package Discount:
               {{ Package.Discount.Amount * 100 }}%

               Package Price:
               ${{ Package.DiscountedPackagePrice }}

      </mz-card>

This does not work. It produces blank cards. But if I change

<mz-card *ngFor="let Package of packages; last as isLast; odd as isOdd" class="col-12" [class.last-col]="isOdd">

to

It works as expected.

Expected Behavior

Interpolation should work inside of selector directives.
Current Behavior

It doesn't work.

If this is a big change, I can just continue using regular Materialize, since it doesn't provide much gain to use the selector directive. But I wanted to bring it to your attention since it might be happening with other selector directives as well.

You need to include your content inside <mz-card-content> tag.

  <mz-card *ngFor="let Package of packages; last as isLast; odd as isOdd" class="col-12" [class.last-col]="isOdd">
    <mz-card-content>

Package Name:
          {{ Package.Name }}

          Description:
          {{ Package.Description }}

          Price:
          ${{ Package.OriginalPackagePrice }}

          Package Discount:
          {{ Package.Discount.Amount * 100 }}%

          Package Price:
          ${{ Package.DiscountedPackagePrice }}

    </mz-card-content>
  </mz-card>

Great, that fixed it. Thanks

Only weird thing is that is wraps the mz-cart-content inside of p
<mz-card class="col-12" _nghost-c13="">

<mz-card-content>
Package Name:
            Pro Pack

               Description:
               The pack for the experienced agent

               Price:
               $925

               Package Discount:
               20%

               Package Price:
               $605.82

         </mz-card-content>

</mz-card>

I did some test and my conclusion is that we can remove the p tag without breaking the card component.
So it would generate the HTML like this after we remove the p :
<mz-card class="col-12" _nghost-c13="">

<mz-card-content>
Package Name:
            Pro Pack

               Description:
               The pack for the experienced agent

               Price:
               $925

               Package Discount:
               20%

               Package Price:
               $605.82

         </mz-card-content>

</mz-card>

@scote Great, yes that looks perfect. Thanks

You can do a PR if you want.

This will be fix in our next release.

Thanks for doing that, it was on my todo list but didn't get around to it yet.


Roll Flutter Engine from f3d48be76630 to af987df261c2 (10 revisions)

https://github.com/flutter/engine/compare/f3d48be76630...af987df261c2
2024-01-28<EMAIL_ADDRESS>Roll Dart SDK from 639cf51f4472 to 1d0b890287a4 (1 revision) (flutter/engine#50129)
2024-01-28<EMAIL_ADDRESS>Roll Skia from 40205cafdb4e to d81204e28dcb (1 revision) (flutter/engine#50127)
2024-01-28<EMAIL_ADDRESS>Roll Fuchsia Linux SDK from GBTh3gOOgmndwT70X... to 5aijurFz23iIuRaMQ... (flutter/engine#50126)
2024-01-28<EMAIL_ADDRESS>[Impeller] Fix alpha management issues for advanced blends. (flutter/engine#50070)
2024-01-28<EMAIL_ADDRESS>Roll Skia from 0c1e352e87aa to 40205cafdb4e (1 revision) (flutter/engine#50124)
2024-01-27<EMAIL_ADDRESS>Roll Dart SDK from 422d048880a8 to 639cf51f4472 (1 revision) (flutter/engine#50123)
2024-01-27<EMAIL_ADDRESS>Revert "[Windows] Introduce egl::Surface and egl::WindowSurface" (flutter/engine#50104)
2024-01-27<EMAIL_ADDRESS>Roll Dart SDK from d0b48a008559 to 422d048880a8 (1 revision) (flutter/engine#50121)
2024-01-27<EMAIL_ADDRESS>Roll Skia from b9b80230c87b to 0c1e352e87aa (1 revision) (flutter/engine#50120)
2024-01-27<EMAIL_ADDRESS>Roll Dart SDK from 141ab6c14cd1 to d0b48a008559 (1 revision) (flutter/engine#50119)
Also rolling transitive DEPS:
fuchsia/sdk/core/linux-amd64 from GBTh3gOOgmnd to 5aijurFz23iI
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC jacksongardner@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Now I need to specify a deployment in the getImages() function and I cannot create a Dall-E deployment.

The latest version of the @azure/openai 1.0.0-beta.8 adds a deployment parameter to the client.getImages() function.  The sample does not show this parameter being used like in this sample https://github.com/Azure/azure-sdk-for-js/blob/main/sdk/openai/openai/samples/v1-beta/javascript/getImages.js. Specifying dall-e or dall-e-3 throws a 404 error.  I do not have an option in the Azure OpenAI studio to create a dall-e deployment.

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

ID: a47134e6-8e58-e26f-a452-b08e57c5cdf0
Version Independent ID: 2fe238ea-1d13-f8d9-c549-09bdb6858c03
Content: Quickstart: Generate images with Azure OpenAI Service - Azure OpenAI
Content Source: articles/ai-services/openai/dall-e-quickstart.md
Service: azure-ai-openai
GitHub Login: @PatrickFarley
Microsoft Alias: pafarley

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

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


Option to redirect standard input to /dev/null

:zap: Summary

Similar to a subset of the options in https://www.freedesktop.org/software/systemd/man/latest/systemd.exec.html#Logging and Standard Input/Output
Value
There are situations in complex CI environments where Lefthook will stall, when not using use_stdin: true. How can a user make sure that commands can never use stdin without causing crashes (by closing the file descriptor)? By redirecting stdin to /dev/null or a similar concept in Windows. Currently, the user has to configure use_stdin: true with every command that could possibly stall Lefthook, even if they don't want those commands to use standard input at all.

Behavior and configuration changes
Nothing, although that options remains open in my view. But this suggestion is to extend behavior by offering a default false redirection option.

Hey @sanmai-NL, I am currently solving an issue related to STDIN right now :)
In this PR I explicitly pass null reader (a reader that returns EOF on every read) to commands that don't explicitly specify use_stdin: true or interactive: true.
I think this is the behavior that prevents commands from being stuck on waiting for input. Do you agree?

I think that should work, but see my reference. Managing which process controls the tty could also solve this.

@mrexox Was this also resolved with https://github.com/evilmartians/lefthook/pull/732?

This now works by default (not via an option). All commands get just EOF when they try to read from STDIN if they don't specify use_stdin: true or interactive: true.
I have concerns of whether a global option is useful. Do you think we should close the feature request or specify it?

Can be closed IMO.


Static Filters causing empty POIs

https://github.com/graphprotocol/graph-node/pull/3941

This was fixed with https://github.com/graphprotocol/graph-node/pull/3951, correct?

Yes, we ended up spliting the fix from the optimisation, I will close #3941 for now