[Actions] Create sonar4fe.yml

What type of PR is this：

[y] feature

Which issues of this PR fixes ：
trigger maven check for fe use sonarcloud
Problem Summary(Required) ：
trigger maven check for fe use sonarcloud

 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.You have signed the CLA already but the status is still pending? Let us recheck it.


Create a github resume theme, base github api

I already forgot what I actually want, but I made this: https://saika.dev/#/theme/portfolio


Automated Package Upgrade

:tada: This PR is included in version 26.0.23 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Enable Pull Request Review API for GitHub Enterprise

Now that Pull Request Review API became official1, it's time to use in
GitHub Enterprise too.
If GitHub Enterprise version is too low to use new API, you should use old
review dog version. (<0.9.7)
Fixes #96

Thanks @haya14busa!


Added check to find accept_ra file exists

Reason for Change:

Incase if user disables ipv6 kernel setting, then accept_ra file would not exist and sysctl cmd would fail. This PR addresses that scenario
Issue Fixed:

Requirements:

[ ] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests

Notes:

this is a great change,
one of our customer are pending on this issue, they tried to disable the ipv6.
and then the azure cni can not work.
@tamilmani1989 could you please merge this and build a new image? thanks in advance.


Renaming module?

I renamed my main module (root folder called app) manually and also updated some files that reference the module. But whenever I create a directive or services on the main module, it still uses the old name. Where can I find the config so I can also change the module name from there, so YO will generate the correct module name. Thanks.

Sorry the late response.
Currently, ng-poly looks at the name in package.json to determine the root level module name. Sorry for the inconvenience.
I'm going to mark this as an enhancement as what you expected to happen seems like a reasonable assumption.

No worries. Im not at work for a couple of days now anyway. Thanks for the reply.

Released in v0.11.0.


[ScopeRequest] Codeunit 8510 "Over-Receipt Mgt."

Please remove OnPrem scope from all functions in Codeunit 8510 "Over-Receipt Mgt.".

Already done


Remove references from analyzers to Microsoft.CodeAnalysis.Workspaces

Analyzers are not allowed to directly or indirectly reference the Microsoft.CodeAnalysis.Workspaces assembly. This means analyzers cannot reference members of code fix providers.

Refactor several classes to make it easier to spot violations of this rule.
Fixes #1659, which was caused by the use of properties from TokenSpacingCodeFixProvider inside of various analyzers.

Current coverage is 79.15%

Merging #1674 into master will not affect coverage as of 87e61af

@@            master   #1674   diff @@
======================================
  Files          539     541     +2
  Stmts        31799   31799
  Branches      8928    8928
  Methods
======================================
+ Hit          25169   25171     +2
+ Partial       5184    5182     -2
  Missed        1446    1446

Review entire Coverage Diff as of 87e61af

Powered by Codecov. Updated on successful CI builds.

:+1: Changes look good to me.
What a nasty kind of dependency problems. Do you think its worth to have some unit tests that will verify this kind of dependency?

@vweijsters I took a more extreme approach. With two assemblies, it is now impossible for our analyzers to reference types that would cause this problem.

:+1:


Optimize Metrics

Rationale
The metrics on the home page are re-queried every time the page updates rather than being cached. They are also all run at once as background callbacks so they can fail without retry.
Suggested Solution
Schedule these in the queue with the major queries for figures. Add retry in case of failure. Cache results to prevent unnecessary round-trips.

Metric caching is still relevant.

Closing this issue because we don't use the Metrics page anymore


Re-license mid-stream?

Is it possible to re-license mid stream - without user facing any interruption?
For example - say a license expires in 15 minutes, and the client retrieves a new license within 10 minutes, is it possible to continue the playback as though nothing changed?

Yes, this is called a license renewal.  Licenses can be renewed without interruption.
Renewal is driven by the CDM and the license itself.  Some licenses specify renewal, and some do not.  If the license specifies renewal on some interval, the CDM periodically sends a 'message' event to the application (in this case, to Shaka Player).  Shaka routes the message to the configured license server, and routes the response back to the CDM.  The expiration time is then extended by the renewal.
Does this answer your question?

Does this answer your question?
And then some! :)

Feel free to close the ticket... though I do have a few followup questions like how I can spin up a licensing server to test this with, and if I can control the frequency of the CDM messaging. Is there a discussion group around this somewhere?
Thanks!

Spinning up a license server depends on your DRM provider.
If you want to use Widevine, you can contact Widevine here: http://www.widevine.com/contact.html  Widevine supports renewals and allows you to control both license duration (expiration) and frequency of renewal.
I don't have any information on hand about Microsoft Playready, but their website is here: https://www.microsoft.com/playready/
You can use the clear key CDM for testing and experimentation without depending on any DRM provider, but it appears that clear key does not support renewals or expiration.
Hope that helps!


terrascan: 1.3.3 -> 1.4.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/accurics/terrascan/releases.
meta.description for terrascan is: "Detect compliance and security violations across Infrastructure"
meta.homepage for terrascan is: "https://github.com/accurics/terrascan"
meta.changelog for terrascan is: ""
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 1.4.0 with grep in /nix/store/bwdj49wnfkix3s274592yawvkmfycp45-terrascan-1.4.0

directory tree listing: https://gist.github.com/9ce2adc74269dea472fd10255d645c82

du listing: https://gist.github.com/d8f28a6b50e57a522879af3e4e751cf4

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
terrascan

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/bwdj49wnfkix3s274592yawvkmfycp45-terrascan-1.4.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A terrascan https://github.com/r-ryantm/nixpkgs/archive/eb1a98b142d20e13d502f23e5d5b71b920f4c750.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/bwdj49wnfkix3s274592yawvkmfycp45-terrascan-1.4.0
ls -la /nix/store/bwdj49wnfkix3s274592yawvkmfycp45-terrascan-1.4.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

terrascan

Maintainer pings
cc @fabaff @kalbasit for testing.

Please give maintainers some days to check PR's before merging.


Multiple Blueprints mounted at same url_prefix (keyword argument to app#register_blueprint)

This issue tracker is a tool to address bugs in Flask itself.
Please use the #pocoo IRC channel on freenode or Stack Overflow for general
questions about using Flask or issues not related to Flask.
If you'd like to report a bug in Flask, fill out the template below. Provide
any any extra information that may be useful / related to your problem.
Ideally, create an MCVE, which helps us
understand the problem and helps check that it is not caused by something in
your code.

Expected Behavior
I have mounted multiple blueprints for serving static files at the same base url, as I use this pattern when developing with other web frameworks (though I am unfamiliar with the limitations of python web programming). Perhaps this is the incorrect suggested mechanism for accomplishing this with Flask?
app.register_blueprint(Blueprint('mybp', __name__, static_folder='static', static_url_path=''), url_prefix='/a')
app.register_blueprint(Blueprint('mybp1', __name__, static_folder='static_', static_url_path=''), url_prefix='/a')

Actual Behavior
The files in static where accessible and the ones in static_ were not. The behavior was silent, there was no traceback.
Environment

Python version: 2.7.12
Flask version: 0.12.2
Werkzeug version: 0.14.1

thanks so much, y'all! I'm just trying to figure out how to make an abstraction for my many folders to be accessible from a single route that serves static files.

That's not how blueprint static folders work. They're not all combined together. Since they both share the same url (/a/...), the first blueprint takes precedence. See #348


Release 3.5.15

Release scheduled for tomorrow, 20.08. Todo:

[ ] Bump dependencies
[ ] Update sponsors
[ ] Release notes

https://github.com/dzikoysk/reposilite/releases/tag/3.5.15


⚠ Bofrost.fr has degraded performance

In ecfaed0, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 2038 ms

Resolved: Bofrost.fr performance has improved in 6c49663.


Cannot update / sync library

Steps to reproduce

Open Neko
Refresh library / Sync Mangadex follows

Expected behavior
Step 2 should trigger the library  update, displaying a progress bar in the notification area.
Actual behavior
No progress bar displaying and (off course) the library does not update.
Crash logs
No log available. The only thing we can see is a notification remaining fixed without showing more information.

Neko version
2.12.2
Android version
Android 12
Device
Redmi Note 11 Pro
Other details
It probably related to issue #1274 and the few operations that the Mangadex team performs around its authentication system.

Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open issue.
[X] I have written a short but informative title.
[X] I have tried the troubleshooting guide.
[X] I have updated the app to the newest version Latest.
[X] I have filled out all of the requested information in this form.

Was probably a side effect of what they did.  I had the same issue.  When previously no issue on this release.

Was probably a side effect of what they did. I had the same issue. When previously no issue on this release.

Seems like it yeah
But it looks like the problem was solved with the 2.12.3 release. So thank you for the hard work. :)


Handle Hash to Protobuf.Map conversion for Ruby

Details can be found here: https://github.com/googleapis/gax-ruby/issues/45

@jmuk Can this be a quick fix from you?

I could do but that would be a good task to learn about the code generation. I like someone else to do that.
(by the way, I think you don't have to file this as a separate issue, just keeping googleapis/gax-ruby#45 is fine).

I think this issue can be addressed in the GAPIC generated code, but I can also see this being fixed in google/protobuf, since it appears that is where the bad behavior originates.


Ensure translation of Hot Module social share copy

cc @mikefantini

So the Hot Module social share copy is translated, but we need to make sure to add a proper translation for the reportback_noun_verb. Also, when specifying the deadline it indicates it via english language using "... by October 31st" or whatever the deadline is. We should probably translate by.

@weerd we need to translate October also yeah?

FYI, I did not translate "playbooks" properly; it's definitely not "playbook-os"

Oh yeah... translations for the months would be good too!

Will be attempting to update the code here to translate the "by", etc:
https://github.com/DoSomething/phoenix/blob/dcb3d3306408f63e3895b81e6a30de896ff7332d/lib/modules/dosomething/dosomething_campaign/dosomething_campaign.theme.inc#L323


ADD a test to verify ZR optic firmware version

Streaming telemetry test for 400ZR firmware version

Pull Request Functional Test Report for #2784 / f6ac2d05147369d826c934ed1fb8b246bd06844f
Virtual Devices

Device
Test
Test Documentation
Job
Raw Log

Arista cEOS

TRANSCEIVER-3: elemetry: 400ZR Optics firmware version streaming.
910cf0c9
Log

Cisco 8000E

TRANSCEIVER-3: elemetry: 400ZR Optics firmware version streaming.
f373dbb8
Log

Cisco XRd

TRANSCEIVER-3: elemetry: 400ZR Optics firmware version streaming.
d78a75ad
Log

Juniper ncPTX

TRANSCEIVER-3: elemetry: 400ZR Optics firmware version streaming.
fc8e3d54
Log

Nokia SR Linux

TRANSCEIVER-3: elemetry: 400ZR Optics firmware version streaming.
15704be7
Log

Openconfig Lemming

TRANSCEIVER-3: elemetry: 400ZR Optics firmware version streaming.
53fdc8ca
Log

Hardware Devices

Device
Test
Test Documentation
Raw Log

Arista 7808

TRANSCEIVER-3: elemetry: 400ZR Optics firmware version streaming.

Cisco 8808

TRANSCEIVER-3: elemetry: 400ZR Optics firmware version streaming.

Juniper PTX10008

TRANSCEIVER-3: elemetry: 400ZR Optics firmware version streaming.

Nokia 7250 IXR-10e

TRANSCEIVER-3: elemetry: 400ZR Optics firmware version streaming.

Help

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 54.484%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1847

Relevant Lines:
3390

💛  - Coveralls


License clarification

The LICENSE of the repository seems to be the Apache License 2.0. However, the website seems to have a different license at https://www.learnpython.org/tos
What does this mean?

I'll solve the licensing issue in the next few weeks.

I'll solve the licensing issue in the next few weeks.

This has been fixed, so the issue can be closed @ronreiter!


[2pts] Inventory Shopping List

Should show the list of food that is missing by comparing the food in the recipe with the food in the inventory.
Should count the total food items and total price of the missing food.


[SPARK-23957][SQL] Remove redundant sort operators from subqueries

What changes were proposed in this pull request?
Subqueries (at least in SQL) have 'bag of tuples' semantics. Ordering
them is therefore redundant (unless combined with a limit). This patch
adds a new optimizer rule that removes sort operators that are directly
below subqueries (or some combination of projection and filtering below
a subquery).
How was this patch tested?
New unit tests. All sql unit tests pass.

Test build #89234 has started for PR 21049 at commit bb992c2.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2238/
Test PASSed.

Test build #89234 has finished for PR 21049 at commit bb992c2.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89234/
Test FAILed.

I am not sure we can actually do this. If we have for instance:
select a + b from (select a, b from tb1 order by a) t1

IIUC the plan would be transformed so that the sort is removed...but actually this will produce a change in the output, since keeping the sort would make the output of the query ordered by a instead of with a random order.
What do you think?

@mgaido91 This is a common rule. Last year, @dilipbiswal @ioana-delaney @nsyca and I had a discussion about this. We forgot to continue the efforts after finishing the first stage of the correlated subquery. https://issues.apache.org/jira/browse/SPARK-18455.
@henryr Thank you for contributing to this!
@dilipbiswal Do you have the bandwidth to review this PR?

@henryr @gatorsmile I agree with Sean. To the best of my knowledge, spark does not treat "select ... from ( query)" as a subquery. It treats it as an aliased query. Please see the the grammar  under "relationPrimary" rule. The subqueries supported in spark may mostly originate from the projection (non scalar) or predicate of the main query. So basically, we see this as expressions either under the Project or Filter operators of the main query block. We can look at SubquerySuite to find usage examples.

@dilipbiswal Thanks! Although Spark doesn't necessarily parse the query in the from clause as a subquery, is it fair to say it plans it as one? (Since the planner puts the alias under a SubqueryAlias node). The optimization of removing the sorts seems valid to me in any case, since neither an alias nor a subquery should be sorting except in the presence of a limit. Or do you think that this optimization should only be applied to subqueries that aren't in the from clause?

@henryr Since SubqueryAlias is used as a correlation name and used mostly for resolving attributes, in my understanding  its not safe to apply this optimization. I will borrow @gatorsmile 's example here.  Please note that the alias is specified after the sort. Below is plan after this optimization that removes sorts under SubqueryAlias->child.
scala> Seq((1, 2, "1"), (3, 4, "3")).toDF("int", "int2", "str_sort").orderBy('int.asc).as('df1)
res0: org.apache.spark.sql.Dataset[org.apache.spark.sql.Row] = [int: int, int2: int ... 1 more field]

scala> res0.explain(true)
== Parsed Logical Plan ==
SubqueryAlias df1
+- AnalysisBarrier
      +- Sort [int#7 ASC NULLS FIRST], true
         +- Project [_1#3 AS int#7, _2#4 AS int2#8, _3#5 AS str_sort#9]
            +- LocalRelation [_1#3, _2#4, _3#5]

== Analyzed Logical Plan ==
int: int, int2: int, str_sort: string
SubqueryAlias df1
+- Sort [int#7 ASC NULLS FIRST], true
   +- Project [_1#3 AS int#7, _2#4 AS int2#8, _3#5 AS str_sort#9]
      +- LocalRelation [_1#3, _2#4, _3#5]

== Optimized Logical Plan ==
LocalRelation [int#7, int2#8, str_sort#9]

== Physical Plan ==
LocalTableScan [int#7, int2#8, str_sort#9]

In this case we should not be removing the top level sort from user's query right ?
cc @gatorsmile for his opinion.

@henryr
Is there any reason to actually use an alias at the root of a plan like this (outside of composing with other plans, where this optimization would apply)?
I can't think of a reason :-). Just that the API allows users do that.
How about this query ?
 scala> spark.sql("with abcd as (select * from t1 order by t1.c1) select * from abcd").explain(true)
18/04/29 23:28:45 WARN ObjectStore: Failed to get database global_temp, returning NoSuchObjectException
== Parsed Logical Plan ==
CTE [abcd]
:  +- 'SubqueryAlias abcd
:     +- 'Sort ['t1.c1 ASC NULLS FIRST], true
:        +- 'Project [*]
:           +- 'UnresolvedRelation `t1`
+- 'Project [*]
   +- 'UnresolvedRelation `abcd`

== Analyzed Logical Plan ==
c1: int, c2: int, c3: int
Project [c1#7, c2#8, c3#9]
+- SubqueryAlias abcd
   +- Sort [c1#7 ASC NULLS FIRST], true
      +- Project [c1#7, c2#8, c3#9]
         +- SubqueryAlias t1
            +- HiveTableRelation `default`.`t1`, org.apache.hadoop.hive.serde2.lazy.LazySimpleSerDe, [c1#7, c2#8, c3#9]

== Optimized Logical Plan ==
HiveTableRelation `default`.`t1`, org.apache.hadoop.hive.serde2.lazy.LazySimpleSerDe, [c1#7, c2#8, c3#9]

== Physical Plan ==
HiveTableScan [c1#7, c2#8, c3#9], HiveTableRelation `default`.`t1`, org.apache.hadoop.hive.serde2.lazy.LazySimpleSerDe, [c1#7, c2#8, c3#9]

IMHO, its probably better to correctly detect the real subqueries and apply this optimization in order to be fully sure about it.
cc @gatorsmile

I might be a bit of a hardliner on this, but I think it's correct to eliminate the {{ORDER BY}} from common table expressions (e.g. MSSQL agrees with me, see this link).
However, given the principle of least surprise, I agree it might be a good idea to at least start with scalar and nested subqueries, and leave inline views for another day. That might be a bit harder to do (I think the rule will need a whitelist of operators it's ok to eliminate sorts below), and in general I think there'll be some missed opportunities, but it's a start :)
Alternatively we could extend the analyzed logical plan to explicitly mark the different subquery types (i.e. have a InlineView node, a NestedSubquery node and so on). That would make these optimizations easier to express, but I have some reservations about the semantics of introducing those nodes. What do you think @dilipbiswal / @gatorsmile ?

@henryr
I might be a bit of a hardliner on this, but I think it's correct to eliminate the {{ORDER BY}} from common table expressions (e.g. MSSQL agrees with me, see this link).
DB>> Yeah.. I had seen this. I had checked DB2 behaviour on CTE and it does not seem to remove the sort.
db2 => with x as (select * from t1 order by 1) select * from x

C1          C2
----------- -----------
          0           0
          1           1
          1           1
          2           2
          2           2

  5 record(s) selected.

So perhaps the SQL standard does not explicitly clarify this ?

@dilipbiswal I tried hard to find something in the SQL standard that clarified the situation, but couldn't (of course that could be because the standard is pretty hard to parse... :)).
So let's go with the idea of not dropping ORDER BY in inline views, but we can safely drop them in scalar subqueries and nested subqueries. What do you think?

@henryr Could you update the PR based on the review? We can safely drop them in scalar subqueries and nested subqueries

@dilipbiswal Can you take this over?

@gatorsmile Sure.


Suggestion: Remove message forward button

Another month another discord useless feature to clutter the ui and mess with your muscle memory. A way to disable it would be appreciated maybe withing the extra adblock list. I can't seem to find the element name to attempt this myself, so I was hoping you could help.

i think it's out of scope for the project, but i would actually appreciate this feature. i've almost never used the 'forward' feature, and it's always available in the dropdown menu if you need it.
currently, the forward button is on every message, and the reply button is only on other people's messages. i guess they don't think replying to your own message is common? i do it all the time...
i'd remove the forward button from all messages and ensure the reply button is visible on all messages (even your own).

replying to your own message is common? i do it all the time...
Same, as for being in scope or not, there is already the extended list of blocking more "debatable" elements which some people may want to keep, so i figured this was another candidate for that.

getting rid of Forward is easy even if it doesn't becoem part of addblock-extras.css. try adding this to your quick css:
/* Remove `Forward` hover bar button */
div[class^=hoverBarButton][aria-label=Forward] {
    display: none;
}

i don't think it's possible to force Reply into the hover bar for the user's own messages because it looks like the element's not there at all. i guess someone needs to make a vencord plugin to do that?

 Adding it for my personal use.

 Adding it for my personal use.

it was actually surprisingly annoying to figure it out. the damn hoverbar disappears when using the keyboard shortcut for the element picker in the dev tools. i've never had a problem like that before!
i solved it by doing setTimeout(()=>{debugger;},5000); and then making the hover bar appear before the 5s were up and the page froze 😉

it was actually surprisingly annoying to figure it out. the damn hoverbar disappears when using the keyboard shortcut for the element picker in the dev tools. i've never had a problem like that before!

Trust me, i know. I tried doing it myself and it was a nightmare. Very clever solution there, and kinda beyond my level of experience. If you would't mind another challenge of sorts, maybe you could take a look at another issue i've reported that I eventually managed to address myself, but I feel it's not done in the cleanest way - it seems to leave an empty element behind.

@VL4DST3R

If you would't mind another challenge of sorts, maybe you could take a look at another issue i've reported that I eventually managed to address myself, but I feel it's not done in the cleanest way - it seems to leave an empty element behind.

sure lol
https://github.com/CroissantDuNord/discord-adblock/pull/43/files#diff-f379e967a22a3a74f335f0a718715ae3232b7c3020a286e0dd6b626bfd80531eR68


feat: 0x1B - SHL opcode

SINCE
GROUP

Constantinople
Comparison & Bitwise Logic Operations

Index 1 is top of the stack. See PUSH.NotesShift the bits towards the most significant one. The bits moved after the 256th one are discarded, the new bits are set to 0.Stack inputshift: number of bits to shift to the left.value: 32 bytes to shift.Stack outputvalue << shift: the shifted value. If shift is bigger than 255, returns 0.SINCE	GROUP
Constantinople	Comparison & Bitwise Logic Operations
Index 1 is top of the stack. See [PUSH](https://www.evm.codes/#60).
Notes
Shift the bits towards the most significant one. The bits moved after the 256th one are discarded, the new bits are set to 0.
Stack input
shift: number of bits to shift to the left.
value: 32 bytes to shift.
Stack output
value << shift: the shifted value. If shift is bigger than 255, returns 0.
Reproduce in playground.
Error cases
The state changes done by the current context are reverted in those cases:
Not enough gas.
Not enough values on the stack.

Down to try this for funzies :)


Add number of disposable diapers purchased to annual report

Summary
Add the number of disposable diapers purchased to the annual survey
Add the number of disposable diapers donated to the annual survey
Details
These should go just above the % purchased and % donated, respectively
I believe the values are being calculated already as part of the calculations for the percentages, so it should be easy to get them.
the Annual surveys are at Reports | Annual Survey when signed in as<EMAIL_ADDRESS>Source note
per request from Judy in slack Feb 13 2024 14:23
Criteria for completion
-[ ] # of disposable diapers purchased added to annual survey
-[ ] # of disposable diapers donated added to annual survey
-[ ] tests to demonstrate the above

Can I be assigned to this?

It's yours


Enums instead of Booleans

Overview
This plugin comes with a ton of checks and a lot of these checks use booleans.
This is getting to the point where it is very confusing to keep track of which checks should continue, if the naming convention matches the functionality / expected boolean, and if the check needs to be inverted to match the name.
JavaDocs may help but will still require a lot of back and forth.
This issue was addressed after this method had the incorrect return value (from MobDamageListener).
    private boolean checkForBlockedLineOfSight(EntityDamageByEntityEvent event, Entity entity, PersistentDataContainer mobPDC, double damageAmount) {
        if (!(entity instanceof LivingEntity)) return false;
        // The line below returns true, stopping all nerf checks after this failed.
        if (!configToggles.get(ConfigParser.ConfigToggles.REQUIRE_LINE_OF_SIGHT)) return true;
        Entity damager = event.getDamager();
        boolean lineofsight = ((LivingEntity) entity).hasLineOfSight(damager);
        if (!lineofsight) {
            NerfFarms.debugMessage("Adding " + damageAmount + " to " + entity.getName() + "'s PDC because they do not have a valid line of sight to the damager"
                        + "\nCurrent PDC amount is: " + mobPDC.getOrDefault(blacklistedDamage, PersistentDataType.DOUBLE, 0.0));
            addPDCDamage(mobPDC, damageAmount);
            checkIfPassedBlacklistedDamagePercent(mobPDC, entity);
            return true;
        }
        return false;
    }

Suggestion
Replace boolean return values for an Enumerated Value in the checks.
For example,
public enum CheckCondition { CONTINUE, END }

if (check() == CheckCondition.END) return;

This should reduce the need to keep glancing back and forth to ensure the logic of the plugin is correct given all these checks.
JavaDocs would also be great.

I want to do this :D

Issue no longer relevant, recode in progress.


WIP: Feature/add symbol support

@maxcleme just chucking my Draft PR in here for visibility. Been toying with where I should store my symbols and decided the generation folder might be the best place. I am also tempted by storing cache csvs in there too which will contain any existing symbols assignments so they don't change with each new version. If we think this won't matter too much I am happy to omit that feature to save on the amount of csv's we will need and the extra complexity it provides.
Not sure if this will over complicate things with far too many CSV's, feels a little like that at the moment so feel free to drop your input in if you have any suggestions. 😄

@jaymeh
Thanks again for taking time of doing a PR for beadcolors.
Since we have only one csv and it will only be used by the v3 generator I would better make all of them a const in the v3.go file to be honest. Avoiding reading/parsing even if there is no issue in here.
About the association between a color and a symbol, I don't think we need to keep track of all of them. First idea that come to mind would be to simply assign symbol in order while going through the palette. The only thing that could break is adding new colors randomly in the middle of a palette, which could easily be avoided by making new added colors at the end as a mandatory rule, of even keep track of a created/updated_date in the /raw folder and sort by this before doing association.
I do also think that allowing use of duplicated symbol but in different palette may be a good trade off. Thus we could reduce the symbols set, or at least put the most relevant ones at the beginning. 3rd party apps using symbols features could easily distinguish symbols by using a colors,prefix,whatever for each palettes usage.

@maxcleme thanks for your notes above.
Firstly I had an issue with making them a const since go can't set maps, arrays or slices as const values. I instead added them into the output of a function and used that instead.
Secondly I had to change the overall signature of the gen function in order to pass a row index into them. This is because the gen functions don't keep any track of the current csv row (but they never had to). I was reluctant to do this at first but I feel like we are still maintaining support for all versions and all the tests pass. Let me know if you need me to change anything.

Hey, obviously I said const but var is ok, I'm alright with it, we can't define const slice or map in Go (and it makes sense).
Also alright to change the gen method signature, however I would better call the new parameter index instead of i, small comment over the method would be nice as well (previous declaration using r,g,b parameters was obvious and self-explaining).

Thanks @maxcleme that is a good point. Made those changes now.

LGTM, is it still WIP for you ?

Excellent 😊. No, I'm happy for you to merge when ready.
On Mon, 27 Jul 2020, 06:56 maxcleme<EMAIL_ADDRESS>wrote:

LGTM, is it still WIP for you ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/maxcleme/beadcolors/pull/17#issuecomment-664135945,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AELKNLHSGY4FYWKOGQOX7LDR5UJIFANCNFSM4PGE4X4A
.


.svs files from GT450 scanner don't have labels removed

The new GT450 scanner creates SVS files that have slide labels, but the deidentification tool fails to remove them.

Fixed (for Windows) in 0.9.1 (https://github.com/pearcetm/svs-deidentifier/commit/fbc33a74637bbaf30e224a6299cee02d4369f2f2)

Fixed (for Windows) in 0.9.1 (https://github.com/pearcetm/svs-deidentifier/commit/fbc33a74637bbaf30e224a6299cee02d4369f2f2)


'WebSocket Connection Closed' after asking Alexa to play

Describe the bug
A clear and concise description of what the bug is.

Websocket is closed following a JSON TypeError after asking (by voice) any Echo to play something, be it an Audiobook (from Audible) or Music (from Amazon Music) or the Radio (from Tunein). (Note: not tried other sources).
As a result the state is no longer correctly pulled into HA (i.e. Developer Tools). This is also reflected in the logs as they remain in 'Standby', 'Paused' or 'Idle' state regardless of what they are actually doing.
Alexa Media Player falls back to polling for data every 5 minutes - however polling is disabled for all Echoes [2021-02-23 19:27:24].

Note: Pushing commands to the Echoes from NodeRed still works - e.g. the wake up routine of announcing the time (notify) and reading the weather/news (daily brief skill) still works just fine.
However, due to the crash, the automations that checks whether my daughters turn their Echoes back on after 'Lights Out' no longer works as the STATE is not reported. As such the sleep timer no longer functions.
The only data that appears to change is 'last_called'.
Also, FireTV state is not recognised before or after the crash but I suspect that this is not in scope even though it is included in the detected devices list.
To Reproduce
Steps to reproduce the behavior:
In the example from the attached logs ...

Have music playing on one Echo device
Restart HA
HA boots up and Alexa Media Player connects to Amazon and brings in the states of all Echo devices okay (e.g. the one echo shows 'Playing', all others 'Standby', 'Paused' or 'Idle') [2021-02-23 19:27:13].
FireTVs turn on and play Netflix. (STATE in HA and logs is NOT updated.) [2021-02-23 19:31:56 - 2021-02-23 19:32:03]
Node Red Automation runs that stops my daughters FireTVs playing. Alexa says 'Goodnight' on one Echo and 'See you in the morning' on another. [2021-02-23 19:45:00]
One daughter asks 'Alexa, Resume' to listen to her audiobook. [2021-02-23 19:45:13]
Errors immediately thrown and Websocket closed. STATE of any Echo is no longer updated into HA [2021-02-23 19:45:1]
Three further errors thrown every 5 minutes as can no longer communicate with the API [2021-02-23 19:45:20]
Further 'Good night' message sent by a different automation I thought I had turned off [2021-02-23 20:00:00] and communication resumes
... until the next polling when it cannot communicate with the API again [2021-02-23 20:05:35 and 2021-02-23 20:10:38]
... but then starts polling again continuously. [2021-02-23 20:15:38]

Expected behavior
A clear and concise description of what you expected to happen.

Echo states to be pushed to Alexa Media Player and therefore reflected in HA (Developer Tools).
Alexa Media Player/alexapy handles the malformed JSON better? (or is it the other way around?)
Auto re-enable the websocket?
If websocket fails, re-enable the disabled polling?

Screenshots
If applicable, add screenshots to help explain your problem.
System details

Home-assistant (version): 2021.2.3
Hassio (Yes/No): (Please note you may have to restart hassio 2-3 times to load the latest version of alexapy after an update. This looks like a HA bug).      I think so, yes. I'm running it on a Raspberry Pi 3b (Many, many, many restarts have happened and the issue persists).
alexa_media  (version from const.py or HA startup): 3.8.2
alexapy (version from pip show alexapy or HA startup): 1.24.2
Amazon 2FA is enabled (y/n). We will not debug login issues if unanswered: Yes

Logs
Please provide logs. We'll be most likely asking for them anyway.
home-assistant<PHONE_NUMBER> red.log
Additional context
Add any other context about the problem here.
This does not appear to be a log in issue as I can send commands to the Echoes and, at least initially, receive push notifications from Amazon.

Thanks. I'll look at the disconnect. There is supposed to be an attempt to reconnect the websocket. It will give up after 5 attempts and switch back to polling. Polling behavior is designed not to slam Amazon so it will only poll devices that it knows are actually playing. So you need to interact with HA for it to figure this out, otherwise it won't poll.
Can you please visit https://alexa.amazon.co.uk/api/activities?startTime=&size=10&offset=1 and let me know what you're seeing? The log is indicating the page is blank which is causing the alexaapi.get_last_device_serial function to break.

Hi,
Thank you for taking a look at this.
I restarted HA so I had a working HA and Alexa Media Player. I then asked 'She Who Must Not Be Named' to read my book triggering Alexa Media Player/Alexapy to fall over. I loaded the suggested link (probably about 10-20 seconds later after confirming the crash in the logs) and I've attached the output.
Amazon - Activities output.txt
Note: The restart happened between the 3rd and 4th objects.
I tried it again after this to see if I could 'catch it out' and found that on the third or fourth F5 it came back with
{"message":"Rate exceeded"}
After a few seconds break from refreshing, pressing F5 once more returned the JSON data.
I think this was because I was refreshing too frequently as even after another restart (and not crashing Alexa Media Player/Alexapy first) I got the same message by refreshing the page about 4 or 5 times within a couple of seconds. Guess this is me slamming Amazon! Oops!
I'm not sure where alexaapi.get_last_device_serial gets it's data from, but if it is the 'UtteranceId' then the most recent object has a 'null' here, as does the 9th object where a crash was triggered. Every other object has an 'UtteranceId' value.
I did also wonder if the 'Follow-Up mode' may be at play here but I don't think so as it is consistent across Echoes that have this on (my echo) and others that have it off (my daughters echo).

Thanks for the details. I think it's probably the rate exceeded message. We're not looking at the utterance_id.  I recently made changes to allow seeing the last utterance but that had the side effect of querying that page more often. I think you just managed to trip it with your use case. I can probably have it ignore the message, but it may mean in your use case the last called device and utterance info may be out of date when that happens.

Can you please test the linked PR to see if it resolves the issue for you?

@alandtse do you still need the PR tested? If so, I can try to do so in the morning

Yes please. Once tested I can release it.

Hi,
Apologies for the delay in replying. I only saw this this morning.
Not being sure how to properly use the change you made I literally copied and pasted it into the file on my RPi, restarted HA and tried to break it!
Having requested my audiobook to be read, HA showed the correct status (yeay!) and the log showed the following lines :
home-assistant<PHONE_NUMBER> abr red.log
I checked the Activities page and compared it to the info on the Developer Tools page. The 'last_called_timestamp' for when I asked 'her' to read my book was for the 3rd object (first three were all part of the same request, "Alexa"(3rd), "Read my book"(2nd), "nothing"(1st)) and when I asked her to stop, it was timestamped with the 1st object (first two were all part of the same request, "Alexa" (2nd), "Stop" (1st)) - as you predicted , a bit inconsistent.
I'll keep an eye on it this evening to see if it wobbles again when the automations are also active.
If it is too many requests too quickly that is causing it, is it worth (on capturing this problem) waiting a couple of seconds and trying again as it quickly started responding again after a short pause during my rapid manual refreshes? (or is that what the await bit does?)
Thank you!

I copied the change to my pi and restarted via server management. I'll keep an eye on it today.

Hi,
Evening update ... The error message (as per the log from earlier today) is only appearing with Audiobooks (Audible) now. I cannot reproduce the issue with Music or Radio anymore.
The Websocket is not closed as it was before, the state in Developer Tools is updating and the automations run as expected.
Thank you.

Working good so far

I don't think audible is breaking mine fwiw. @alandtse I am happy to share the activities.json with you but don't really want to post it publicly... any preference on how I get it to you?

Resume does not trigger the error (nor does resume forever!).
When I have some examples in the activities log and HA log I will upload something. It seems Amazon only make the last 50 utterances available and most of these when I checked are ‘resume’ which did not trigger the error.
If it is possible to post this data privately/directly to you, I agree with @genebean, that would be preferable.  It takes ages to sanitise the files! (Especially when I’m not sure what many of the random character strings represent!)
Many thanks

Evening update:
The keywords play, resume or continue do not trigger the error.
The keyword read does seem to trigger the error regardless of what I put after it (e.g. read my book or read [name of the book] or just read.
My lay person observation: If a book is read or played it always seems to generate an extra object which has a null 'Description' but a 'domainAttributes' value instead (note: only reading the book generates the error).
Music by comparison does not do this and combines the 'Description' and 'domainAttributes' name/key pair in the same object.

Are you having it read kindle books (aka narrated by Alexa) or play audible books? I ask because we always tell the echo to “play” a book from audible but read ones that are nit audio books to start with? Also, does “read me a story” or “tell me a story” also cause the problem?

Hi,
We are only listening to books from Audible, e.g. the Harry Potter series as read by Stephen Fry (not Jim Dale ;o).
You make an interesting point about reading vs playing Audible books. Audio generated on the fly verses pre-recorded. Although at the end of the day, should it matter who is 'reading' it? A computer algorithm or a professional narrator?
read me a story does trigger the error and plays the last audiobook that was playing.
tell me a story does not trigger the error but asks whether we want to listen to a story from Alexa (The Potion) or Audible (an extract from Revolting Rhymes by Roald Dahl). A seemingly random selection from Amazon's catalogue.
I noted down the times of this evenings errors and compared them all to the Amazon Activities to see what Alexa was being asked at the time of the errors. I've attached the summary.
Todays error summary.xlsx
I appreciate this is somewhat less useful than the actual logs but shows a trend of sorts. Two strikes each time we ask Alexa to read something and one when it appears to be doing a scheduled refresh every 5 minutes.

Thanks for the help. It was the null description. New build should resolve it and also should populate something in the summary.


feat: allow boto3 to use its native credential finding functionality

Description
Removes hardcoded AWS credential and region information, allowing boto3 to run through its order of credential precedence. This allows for more secure configurations of AWS credentials to be used, making wider adoption possible.
Fixes #1525
Type of change
Please delete options that are not relevant.

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

How Has This Been Tested?
I tried running tests locally but every one of them failed. There was also a missing test dependency I had to add (sqlalchemy) before I could even attempt to run them. I'm hoping there is something in CI that was missing locally, but I'm happy to run tests locally and add some in if I can get some guidance on how to get them to run.

[ ] Unit Test
[ ] Test Script (please provide)

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules
[x] I have checked my code and corrected any misspellings

Maintainer Checklist

[ ] closes #1525 (Replace xxxx with the GitHub issue number)
[ ] Made sure Checks passed

Hey @mikejgray can you please resolve the conflicts so that we can test the PR?

Hi @Dev-Khant , the CI failures don't appear related to the changes in this PR. How would you like me to handle them?

Hey @mikejgray Those failing CI tests are from a separate issue, it's because poetry is not up to date with libraries mentioned.
Could you please add a test script for this and ensure that the approach works for all the different ways the user might pass the credentials?

Hey @mikejgray Those failing CI tests are from a separate issue, it's because poetry is not up to date with libraries mentioned.
Could you please add a test script for this and ensure that the approach works for all the different ways the user might pass the credentials?

I took a look at the proposed changes and there isn't anything in there to test besides what boto3 already provides in their test suite. The way I'm setting up the client is the same way they suggest in the documentation, which also explains the order of credential precedence: https://boto3.amazonaws.com/v1/documentation/api/latest/guide/credentials.html#configuring-credentials
It would also be difficult to test this without having different kinds of credentials, which would need to become a permanent part of your repo secrets and validated in your pipelines to be secure.
If you'd like to provide an OIDC AWS role for this repository and an IAM User, with the secrets stored in the repository secrets, I can write an action workflow to do this. It doesn't seem necessary, however, given that boto3 tests this functionality upstream.


add: *.svg linguist-vendored to .gitattributes

Tried to make GitHub code frequency stats more realistic. At first glance looks like this does not have any effect. The next commit (https://github.com/fohrloop/wakepy/pull/106) included only svg file changes and changed the code frequency diagrams roughly +100 and -150.


Custom registry

Description
Creating optional custom local registry option.

moving registry vars to shared common
skipping deploy registry if a custom one has been configured
fix sync registry to use from shared utils
validating Pull Secret
adding certificate to trusted root

Type of change
Please select the appropriate options:

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

Testing
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[ ] Test A
[ ] Test B

Test Configuration:

Versions:
Hardware:

Checklist

[ ] I have performed a self-review of my own code
[ ] If a change is adding a feature, it should require a change to the README.md and the review should catch this.
[ ] If the change is a fix, it should have an issue. The review should make sure the comments state the issue (not just the number) and it should use the keywords that will close the issue on merge.
[ ] A change should not be merged unless it passes CI or there is a comment/update saying what testing was passed.
[ ] PRs should not be merged unless positively reviewed.

@eifrach is ready to review? I'm not sure if this is which you said "ready to review"


Cannot install with pip 23.1 (bdist_wheel behavior change)

rpm-py-installer became uninstallable with pip 23.1.  This seems to be due to the following change mentioned in the pip 23.1 changelog: https://pip.pypa.io/en/stable/news/#v23-1

Remove setup.py install fallback when building a wheel failed for projects without pyproject.toml. (#8368)

Steps to reproduce
$ virtualenv ~/clean-env
created virtual environment CPython3.11.2.final.0-64 in 141ms
  creator CPython3Posix(dest=/home/rmcgover/clean-env, clear=False, no_vcs_ignore=False, global=False)
  seeder FromAppData(extra_search_dir=/usr/share/python-wheels,download=False, pip=bundle, setuptools=bundle, wheel=bundle, via=copy, app_data_dir=/home/rmcgover/.local/share/virtualenv)
    added seed packages: pip==23.0.1, setuptools==67.6.0, wheel==0.40.0
  activators BashActivator,CShellActivator,FishActivator,NushellActivator,PowerShellActivator,PythonActivator

$ . ~/clean-env/bin/activate

(clean-env) $ pip install pip==23.1
Collecting pip==23.1
  Downloading pip-23.1-py3-none-any.whl (2.1 MB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 2.1/2.1 MB 5.9 MB/s eta 0:00:00
Installing collected packages: pip
  Attempting uninstall: pip
    Found existing installation: pip 23.0.1
    Uninstalling pip-23.0.1:
      Successfully uninstalled pip-23.0.1
Successfully installed pip-23.1

(clean-env) $ pip install rpm-py-installer
Collecting rpm-py-installer
  Using cached rpm-py-installer-1.2.0.tar.gz (21 kB)
  Preparing metadata (setup.py) ... done
Building wheels for collected packages: rpm-py-installer
  Building wheel for rpm-py-installer (setup.py) ... error
  error: subprocess-exited-with-error

  × python setup.py bdist_wheel did not run successfully.
  │ exit code: 1
  ╰─> [23 lines of output]
      running bdist_wheel
      Traceback (most recent call last):
        File "<string>", line 2, in <module>
        File "<pip-setuptools-caller>", line 34, in <module>
        File "/tmp/pip-install-rwp295ut/rpm-py-installer_7082f2e96ca0418fadeafe6e48c32bc1/setup.py", line 94, in <module>
          setuptools.setup(
        File "/home/rmcgover/clean-env/lib/python3.11/site-packages/setuptools/__init__.py", line 108, in setup
          return distutils.core.setup(**attrs)
                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
        File "/home/rmcgover/clean-env/lib/python3.11/site-packages/setuptools/_distutils/core.py", line 185, in setup
          return run_commands(dist)
                 ^^^^^^^^^^^^^^^^^^
        File "/home/rmcgover/clean-env/lib/python3.11/site-packages/setuptools/_distutils/core.py", line 201, in run_commands
          dist.run_commands()
        File "/home/rmcgover/clean-env/lib/python3.11/site-packages/setuptools/_distutils/dist.py", line 969, in run_commands
          self.run_command(cmd)
        File "/home/rmcgover/clean-env/lib/python3.11/site-packages/setuptools/dist.py", line 1221, in run_command
          super().run_command(command)
        File "/home/rmcgover/clean-env/lib/python3.11/site-packages/setuptools/_distutils/dist.py", line 988, in run_command
          cmd_obj.run()
        File "/tmp/pip-install-rwp295ut/rpm-py-installer_7082f2e96ca0418fadeafe6e48c32bc1/setup.py", line 91, in run
          raise Exception('bdist_wheel is not supported')
      Exception: bdist_wheel is not supported
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
  ERROR: Failed building wheel for rpm-py-installer
  Running setup.py clean for rpm-py-installer
Failed to build rpm-py-installer
ERROR: Could not build wheels for rpm-py-installer, which is required to install pyproject.toml-based projects

Actual behavior
Installation command fails, as in above example.
Expected behavior
Installation command succeeds.
Additional info
Installing with older versions of pip works, but it does generate a related warning:

DEPRECATION: rpm-py-installer was installed using the legacy 'setup.py install' method, because a wheel could not be built for it. pip 23.1 will enforce this behaviour change. A possible replacement is to fix the wheel build issue reported above. Discussion can be found at https://github.com/pypa/pip/issues/8368

Also, for users running into this issue with tox.  virtualenv v20.21.1 increased the bundled version of pip to 23.1 (see: https://virtualenv.pypa.io/en/latest/changelog.html#v20-21-1-2023-04-19 ).  In some cases then, to work around the issue, just installing an older pip is not enough - one must do for example pip install tox 'virtualenv<20.21.1'.

Hello,
This failure is now a blocking issue for our manufacturing system. I cannot update the software. Is there a plan to release an update that addresses this issue any time soon?
Thanks, your help is appreciated.

Hi, I'm afraid this requires significant code changes and I don't think there is a plan to release an update, certainly not any time soon. I'm curious, what is your use case for rpm-py-installer?


Build: cleanup and address deprecations

Some obvious review fixes and updates
Enable some warnings and deprecations in build scripts
Fixed various compiler warnings

Looks good! Just one question.
Do you want to add anything more here?

I want to migrate to Mill 0.11, but this is still a bit far, so a merge now makes sense. I'll remove the draft state once I think I'm done.


how Health Check Status is calculated?

[Enter feedback here]
Hi,
I am trying the Health Check feature on Web app.
I can see the instance got rebooted, but I never see it got removed from the load balancer.
Also how Health Check Status metric is calculated? When 1 of my 3 instances is reporting 500 on the health check end point, I don't see the metric drops.
Thanks.

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

ID: 20c42a13-f6cb-1e60-0438-f7f32b86a336
Version Independent ID: 79e7dd15-24aa-2e6d-cee7-7714e1940c11
Content: Monitor the health of App Service instances - Azure App Service
Content Source: articles/app-service/monitor-instances-health-check.md
Service: app-service
GitHub Login: @msangapu-msft
Microsoft Alias: msangapu

@ethanli83 Thanks for the feedback! We are currently investigating and will update you shortly.

Never mind, I figured it out. thanks


Story Embeds Project: Gutenberg Blocks, Customizer, Shortcodes

Task Description
Tracking ticket for the ongoing work to expand the capabilities of the plugin for developers and site owners.

Richer functionality for Gutenberg block(s)
Customizer API
Shortcodes
TinyMCE button for Classic Editor

Hi, I am also looking to create carasouel of web Stories ?
Can you please let me know if this block is developed...

@4jassikingg A carousel block is currently in development, stay tuned!

Sure @swissspidy Many thanks. Can you pl let me know when we might expect it to be released.
I am just looking to show a slider for all stories on my blog. If there is any turaround solution, please help me otherwise I will wait until the release of carousel block.
Kudos for the hard work you people have been doing with this new plugin.!!!

Thanks for the kind words. Early in the new year / Q1 we should have this ready :) Nothing to share just yet unfortunately.


Converting Tensor to Numpy array extremely slow in TF 2.0

Current Issue / Bug Report
I tried to convert a list of tensors to a numpy array.
This was no issue in Tensorflow 1.13.x but is now orders of magnitudes slower in Tensorflow 2.0.0-dev20190405
I linked a piece of code to reproduce the issue in Colab (no HW acceleration required) and to also show the difference in execution time between 1.13.x and 2.0.0/nightly
I tested the issue with
np.array(LIST_OF_TENSORS)
np.shape(LIST_OF_TENSORS)
System information
Colab w.o. HW acceleration and TF 2.0.0-dev20190405
Code to reproduce the issue
Code Example
.

Wow, this is really bad. Thanks for bringing it up!

The underlying issue, after running your repro through a profiler, seems to be that Tensor's array conversion is done through slicing instead of the fastpath.

Interestingly, if you do np.array(tensor) there's no slowdown, it's only if you do np.array(list_of_tensors).

@superbobry do you have any idea why converting a tensor to an ndarray goes through array and is fast while converting a list of tensors goes through iter and is slow?

Not sure, will have a look at PyArray_FromAny.
Also, here's a smaller reproducer:
t = tf.random.uniform([256])
%timeit np.array(t)
%timeit np.array([t])
100000 loops, best of 3: 5.21 µs per loop
10 loops, best of 3: 22.1 ms per loop

I'll see if it's easy to generalize setArrayFromSequence to arbitrarily nested sequence of array-like objects.
In the meantime, I wonder if there is a way to detect np.array([t, t, t...]) calls and error or emit or warning?

There seem to be two distinct issues:

PyArray_FromAny copies its argument to a list when it does not implement __array_interface__, that is exactly numpy/numpy#8562 and it does affect Tensor which only implements __array__.
setArrayFromSequence ignores __array_*__ and __array__ and copies its argument (again!) unless it's an array (or an array subtype).

I have sketched a fix for both issues. Will post an update once I do more testing.

You can wrap each item in the list with np.array() first then wrap the whole list once again with np.array(). I know it does not looks good, but at least it is works for now.

The TF side of the fix has been merged. Closing the issue as the remaining part is on the NumPy side.


[bug]: Promise.allSettled and Promise.any is undefined

Description
seems to be a hermes problem, but I think it's worth mentioning in the documentation
Web3Modal SDK version
1.0.0-alpha.13
Output of npx react-native info
System:
OS: macOS 14.1
CPU: (8) arm64 Apple M1
Memory: 792.44 MB / 16.00 GB
Shell:
version: 3.6.1
path: /opt/homebrew/bin/fish
Binaries:
Node:
version: 18.17.1
path: ~/.volta/tools/image/node/18.17.1/bin/node
Yarn:
version: 4.0.0-rc.48
path: ~/.volta/tools/image/yarn/4.0.0-rc.48/bin/yarn
npm:
version: 9.6.7
path: ~/.volta/tools/image/node/18.17.1/bin/npm
Watchman: Not Found
Managers:
CocoaPods:
version: 1.14.2
path: /opt/homebrew/bin/pod
SDKs:
iOS SDK:
Platforms:

DriverKit 23.0
iOS 17.0
macOS 14.0
tvOS 17.0
watchOS 10.0
Android SDK: Not Found
IDEs:
Android Studio: Not Found
Xcode:
version: 15.0.1/15A507
path: /usr/bin/xcodebuild
Languages:
Java:
version: 21.0.1
path: /opt/homebrew/opt/openjdk/bin/javac
Ruby:
version: 2.6.10
path: /usr/bin/ruby
npmPackages:
"@react-native-community/cli": Not Found
react:
installed: 18.2.0
wanted: ^18.2.0
react-native:
installed: 0.72.6
wanted: ^0.72.6
react-native-macos: Not Found
npmGlobalPackages:
"react-native": Not Found
Android:
hermesEnabled: true
newArchEnabled: false
iOS:
hermesEnabled: true
newArchEnabled: false

Expo Version (if applies)
49.0.16
Steps to reproduce

Snack, code example, screenshot, or link to a repository

this will help you
`import { polyfillObjectProperty } from 'react-native/Libraries/Utilities/PolyfillFunctions';
function allSettled(promises) {
let mappedPromises = promises.map(p => {
return p
.then(value => {
return {
status: 'fulfilled',
value,
};
})
.catch(reason => {
return {
status: 'rejected',
reason,
};
});
});
return Promise.all(mappedPromises);
}
polyfillObjectProperty(Promise, 'allSettled', () => allSettled);`

@nikitapilgrim could you share a public repository or a code sandbox with the reproducible issue?

@nikitapilgrim could you share a public repository or a code sandbox with the reproducible issue?

https://github.com/WalletConnect/web3modal-react-native/tree/V1/apps/native
have you tried running it with hermes?

@nikitapilgrim hermes is turned on by default in that project, and i didn't have the issue

I get it, the Sentry library patches the Promise global

@nikitapilgrim if Sentry library is patching then what could be the solution for this?


Unable to install GRSISort with MacOs Monterey 12.4

I have recently updated my MacOs to Monterey 12.4 and installed root 6.26/02 using Mac ports.
Previously I had issues with my root installation where I was unable to draw objects or launch a TCanvas but that appears to be resolved.
I am using the /bin/bash shell rather than the zsh shell.
I have tried to install GRSISort but have ran into problems.
First when I try to source grsisort I get the following

source thisgrsi.sh
drop_from_path: needs 2 arguments
drop_from_path: needs 2 arguments
drop_from_path: needs 2 arguments

I am not sure if this part is an issue so I have also tried to run "make" but I get the following
Building  grsirc                                                      [OK]
Compiling GVersion.h                                                  [OK]
Compiling .build/util/AddToChannelNumber.o    [ERROR]
In file included from util/AddToChannelNumber.cxx:1:
In file included from /Users/pietrospagnoletti/software/GRSISort/include/TChannel.h:40:
In file included from /opt/local/libexec/root6/include/root/TTree.h:46:
/opt/local/libexec/root6/include/root/TBranch.h:101:7: warning: declaration shadows a variable in the global namespace [-Wshadow]
kDoNotProcess = ::kDoNotProcess, ///< Active bit for branches
^
/opt/local/libexec/root6/include/root/TBranch.h:56:13: note: previous declaration is here
const Int_t kDoNotProcess = BIT(10); // Active bit for branches
^
/opt/local/libexec/root6/include/root/TBranch.h:102:7: warning: declaration shadows a variable in the global namespace [-Wshadow]
kIsClone      = ::kIsClone,      ///< To indicate a TBranchClones
^
/opt/local/libexec/root6/include/root/TBranch.h:57:13: note: previous declaration is here
const Int_t kIsClone      = BIT(11); // to indicate a TBranchClones
^
/opt/local/libexec/root6/include/root/TBranch.h:103:7: warning: declaration shadows a variable in the global namespace [-Wshadow]
kBranchObject = ::kBranchObject, ///< Branch is a TObject*
^
/opt/local/libexec/root6/include/root/TBranch.h:58:13: note: previous declaration is here
const Int_t kBranchObject = BIT(12); // branch is a TObject*
^
/opt/local/libexec/root6/include/root/TBranch.h:104:7: warning: declaration shadows a variable in the global namespace [-Wshadow]
kBranchAny    = ::kBranchAny,    ///< Branch is an object*
^
/opt/local/libexec/root6/include/root/TBranch.h:59:13: note: previous declaration is here
const Int_t kBranchAny    = BIT(17); // branch is an object*
^
In file included from util/AddToChannelNumber.cxx:1:
In file included from /Users/pietrospagnoletti/software/GRSISort/include/TChannel.h:42:
In file included from /Users/pietrospagnoletti/software/GRSISort/include/TMnemonic.h:10:
/Users/pietrospagnoletti/software/GRSISort/include/Globals.h:78:10: fatal error: 'sys/prctl.h' file not found
#include <sys/prctl.h>
^~~~~~~~~~~~~
4 warnings and 1 error generated.
make: *** [.build/util/AddToChannelNumber.o] Error 1
I have check and found that sys/prctl.h does not exist on my machine but from what I could find on google this is something for linux but not Mac.

I fixed that issue. The error you get when sourcing thisgrsi.sh is weird, I don't get that on my Mac. What is the result of
echo $GRSISYS
echo $PATH
echo $LD_LIBRARY_PATH

before and after sourcing the file?

So I realised I left a 'source path/thisgrsi.sh' in my .profile from a previous grsisort installation. That has been removed and now when I perform 'source thisgrsi.sh' the comment
drop_from_path: needs 2 arguments
is no longer returned.
Before doing 'source thisgrsi.sh' I get the following

echo $GRSISYS

(blank line)
echo $PATH
/opt/local/bin:/opt/local/sbin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin:/opt/X11/bin:/Library/Apple/usr/bin:/Users/pietrospagnoletti/MiniballCoulexSort/bin:/opt/local/Library/Frameworks/Python.framework/Versions/3.6/bin:/Users/pietrospagnoletti/gosia
echo $LD_LIBRARY_PATH
(blank line)

After performing 'source thisgrsi.ssh' I get the following

echo $GRSISYS

/Users/pietrospagnoletti/software/GRSISort
echo $PATH
/Users/pietrospagnoletti/software/GRSISort/bin:/opt/local/bin:/opt/local/sbin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin:/opt/X11/bin:/Library/Apple/usr/bin:/Users/pietrospagnoletti/MiniballCoulexSort/bin:/opt/local/Library/Frameworks/Python.framework/Versions/3.6/bin:/Users/pietrospagnoletti/gosia
echo $LD_LIBRARY_PATH
/Users/pietrospagnoletti/software/GRSISort/lib

I have tried running 'make' again but I am still get the same error message.

I have downloaded the most recent version of grsisort and it compiled and is running without issue.
Thanks.


Disallow implicits

Add to tsconfig:
        "noImplicitAny": true,
        "noImplicitReturns": true,
        "noImplicitThis": true

And make tsc happy.

Done! The project does not use these, but at least I added these to the test repos.


Deleting a row after insert

I came to this problem: how can I delete a row that I just inserted in the react table. Why? because it failed to be saved in the database. Here it is my code:

function handleInsertedRow(row) {
// ...
let newRowStr = '';
const userInfo = {};
for (const prop in row) {
newRowStr += prop + ': ' + row[prop] + ' \n';
userInfo[prop] = row[prop];
}
delete userInfo.rowNumber;  /*eliminate dummy column :-) */
console.log('JSON data = ' + userInfo);
axios.post(
'http://'+ window.location.hostname + ':9000/addNewUser', userInfo
)
.then(function (response) {
console.log(response);
})
.catch(function (error) {
console.log(error);
});
I could not find an event like "before insert new row" as with row edition (exist?). But, if this possibility does not exist then I have to find away to delete the just inserted row through the catch section of my axios call back. Can you please tell me how can I do that?
Thx

If your data need to sync with backend, I recommend you to use the remote mode which let you can handle or manipulate the data by yourself instead of react-bootstrap-table itself.
For some of case, people will use remote mode only for some functionality, like paging, load, delete, update, insert etc. but for filter, sort, search they hope react-bootstrap-table to handle it on client side.
so after v2.11.1, we allow remote mode can configure via each different functionality. for you can, maybe I guess you can only enable the insert mode for remote.
check this and this to learn how to use it.  BTW here are all the examples for remote mode


chore(master): Release 0.46.0

:robot: I have created a release beep boop
aztec-package: 0.46.0
0.46.0 (2024-07-09)
Features

VK tree (#6914) (8631237)

barretenberg.js: 0.46.0
0.46.0 (2024-07-09)
Miscellaneous

barretenberg.js: Synchronize aztec-packages versions

aztec-packages: 0.46.0
0.46.0 (2024-07-09)
⚠ BREAKING CHANGES

constant inputs for blackbox (#7222)

Features

Add more slice methods to the stdlib (https://github.com/noir-lang/noir/pull/5424) (6137a10)
Constant inputs for blackbox (#7222) (9f9ded2)
Detect subgraphs that are completely independent from inputs or outputs (https://github.com/noir-lang/noir/pull/5402) (c7b1ae4)
Lsp "go to definition" for modules (https://github.com/noir-lang/noir/pull/5406) (c7b1ae4)
Lsp rename/find-all-references for traits (https://github.com/noir-lang/noir/pull/5409) (c7b1ae4)
Lsp rename/find-all-references for type aliases (https://github.com/noir-lang/noir/pull/5414) (6137a10)
Sync from aztec-packages (https://github.com/noir-lang/noir/pull/5408) (c7b1ae4)
Sync from aztec-packages (https://github.com/noir-lang/noir/pull/5432) (6137a10)
TXE docs and usability (#7305) (6b2a351)
Unquote multiple items from annotations (https://github.com/noir-lang/noir/pull/5441) (6137a10)
Use BB code hash for vk caching (#7396) (b03f1a2)
VK tree (#6914) (8631237)

Bug Fixes

Added a comment to clarify how to launch TXE (#7402) (1ca48a4)
Added missing dep (#7405) (1cb968a)
Allow importing notes from other contracts and inject them in the macros (#7349) (586c0b0)
Change panic to error in interpreter (https://github.com/noir-lang/noir/pull/5446) (6137a10)
Complete call stacks with no_predicates (https://github.com/noir-lang/noir/pull/5418) (c7b1ae4)
Correct range for overlfowing/underflowing integer assignment (https://github.com/noir-lang/noir/pull/5416) (c7b1ae4)
e2e snapshots: Be specific about ipv4 localhost (#7350) (e2bbf06)
Fields fromstring not working as intended (#7365) (633eb6b)
Included argshash computation in public call_interfaces and cleanup (#7354) (13e9b94)
Lsp find struct reference in return locations and paths (https://github.com/noir-lang/noir/pull/5404) (c7b1ae4)
Lsp struct rename/reference difference (https://github.com/noir-lang/noir/pull/5411) (6137a10)
Privacy leak in private refund (#7358) (e05f6f0), closes #7321

Miscellaneous

Add remaining slice methods to the interpreter (https://github.com/noir-lang/noir/pull/5422) (c7b1ae4)
Add test and benchmarks for poseidon2 (https://github.com/noir-lang/noir/pull/5386) (c7b1ae4)
avm: Avoid including flavor where possible (#7361) (dbdffd6)
avm: Better log_derivative_inverse_round (#7360) (6329833)
avm: Make stats thread safe (#7393) (894ac3b)
avm: Smaller prover (#7359) (7d8c833)
avm: Smaller transcript (#7357) (3952a44)
bb: Do not instantiate Relation (#7389) (d9cbf4c)
Counters (#7342) (819f370)
docs: Fix link to docker (#7395) (ad4a401)
docs: Update token tutorial (#7241) (0414eb5)
Dummy workflow (https://github.com/noir-lang/noir/pull/5438) (6137a10)
Extracts refactoring of reading error payloads from #5403 (https://github.com/noir-lang/noir/pull/5413) (6137a10)
Generate declaration files for VKs (#7391) (7c96636)
Merge BarretenbergVerifierBackend and BarretenbergBackend (https://github.com/noir-lang/noir/pull/5399) (6137a10)
Move SSA checks to a new folder (https://github.com/noir-lang/noir/pull/5434) (6137a10)
Notify Noir team on changes to the stdlib (#7387) (061aac3)
Optimize private call stack item hash for gate count (#7285) (783d9b6)
Optimize public call stack item hashing (#7330) (4a5093c)
Refactor logic around inlining no_predicates functions (https://github.com/noir-lang/noir/pull/5433) (6137a10)
Replace relative paths to noir-protocol-circuits (8510269)
Replace relative paths to noir-protocol-circuits (9eb8d1c)
Sync external contributions (#6984) (a265b29)

Documentation

Making private refunds smooth-brain friendly (#7343) (533c937)
Update overview.md (fix link text) (#7344) (79e5856)

barretenberg: 0.46.0
0.46.0 (2024-07-09)
⚠ BREAKING CHANGES

constant inputs for blackbox (#7222)

Features

Constant inputs for blackbox (#7222) (9f9ded2)

Miscellaneous

avm: Avoid including flavor where possible (#7361) (dbdffd6)
avm: Better log_derivative_inverse_round (#7360) (6329833)
avm: Make stats thread safe (#7393) (894ac3b)
avm: Smaller prover (#7359) (7d8c833)
avm: Smaller transcript (#7357) (3952a44)
bb: Do not instantiate Relation (#7389) (d9cbf4c)
Counters (#7342) (819f370)

This PR was generated with Release Please. See documentation.

Benchmark results
Metrics with a significant change:

avm_simulation_time_ms (GasToken:set_portal): 27.9 (+127%)
avm_simulation_time_ms (Token:mint_public): 61.8 (+34%)

Detailed results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
Proof generation
Each column represents the number of threads used in proof generation.

Metric
1 threads
4 threads
16 threads
32 threads
64 threads

proof_construction_time_sha256_ms
5,765 (+1%)
1,566 (+1%)
709 (+1%)
758 (+1%)
780 (+1%)

proof_construction_time_sha256_30_ms
11,938 (+2%)
3,166 (+1%)
1,409
1,428
1,475 (+1%)

proof_construction_time_sha256_100_ms
44,209 (+1%)
11,808 (+1%)
5,425
5,438 (+1%)
5,365

proof_construction_time_poseidon_hash_ms
78.0
34.0
34.0
58.0 (+2%)
86.0 (-2%)

proof_construction_time_poseidon_hash_30_ms
1,529 (+1%)
419 (+1%)
201 (+1%)
234 (+5%)
266 (+1%)

proof_construction_time_poseidon_hash_100_ms
5,786 (+1%)
1,583 (+2%)
725 (+1%)
782 (+1%)
797 (+1%)

L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
4 txs
8 txs
16 txs

l1_rollup_calldata_size_in_bytes
1,412
1,412
1,412

l1_rollup_calldata_gas
9,476
9,466
9,476

l1_rollup_execution_gas
611,215
611,356
611,517

l2_block_processing_time_in_ms
752 (-1%)
1,423 (+1%)
2,705

l2_block_building_time_in_ms
20,776
41,695
81,339

l2_block_rollup_simulation_time_in_ms
20,776
41,695
81,339

l2_block_public_tx_process_time_in_ms
17,764
38,474
78,173

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 8 txs.

Metric
3 blocks
5 blocks

node_history_sync_time_in_ms
7,080 (+1%)
9,893 (+1%)

node_database_size_in_bytes
12,419,152
16,130,128

pxe_database_size_in_bytes
16,254
26,813

Circuits stats
Stats on running time and I/O sizes collected for every kernel circuit run across all benchmarks.

Function
time_ms
bytecode_size_in_bytes

GasToken:_increase_public_balance
67.6 (-6%)
13,790

GasToken:set_portal
:warning: 27.9 (+127%)
3,339

Token:constructor
91.1 (-3%)
23,692

FPC:constructor
61.6 (+19%)
13,592

GasToken:mint_public
55.7 (+11%)
10,158

Token:mint_public
:warning: 61.8 (+34%)
19,034

Token:assert_minter_and_mint
180 (-2%)
12,925

AuthRegistry:set_authorized
27.5 (-10%)
7,812

FPC:prepare_fee
105 (-9%)
15,062

Token:transfer_public
28.1 (-21%)
31,218

FPC:pay_refund
131 (-4%)
25,260

Benchmarking:increment_balance
2,198
15,267

Token:_increase_public_balance
52.0 (-10%)
15,006

FPC:pay_refund_with_shielded_rebate
108 (-11%)
26,347

Public DB Access
Time to access various public DBs.

Function
time_ms

get-nullifier-index
0.153 (-4%)

Tree insertion stats
The duration to insert a fixed batch of leaves into each tree type.

Metric
1 leaves
16 leaves
64 leaves
128 leaves
256 leaves
512 leaves
1024 leaves

batch_insert_into_append_only_tree_16_depth_ms
10.4
16.7
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_count
16.8
31.7
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_ms
0.601
0.512
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_32_depth_ms
N/A
N/A
47.7 (-1%)
75.5 (-1%)
131 (-2%)
244
469 (-1%)

batch_insert_into_append_only_tree_32_depth_hash_count
N/A
N/A
95.9
159
287
543
1,055

batch_insert_into_append_only_tree_32_depth_hash_ms
N/A
N/A
0.488 (-1%)
0.464 (-1%)
0.449 (-1%)
0.443
0.438 (-1%)

batch_insert_into_indexed_tree_20_depth_ms
N/A
N/A
58.9 (-1%)
112
182 (-1%)
353
691 (-1%)

batch_insert_into_indexed_tree_20_depth_hash_count
N/A
N/A
109
207
355
691
1,363

batch_insert_into_indexed_tree_20_depth_hash_ms
N/A
N/A
0.498 (-1%)
0.500
0.482 (-1%)
0.477
0.474 (-1%)

batch_insert_into_indexed_tree_40_depth_ms
N/A
N/A
72.3
N/A
N/A
N/A
N/A

batch_insert_into_indexed_tree_40_depth_hash_count
N/A
N/A
133
N/A
N/A
N/A
N/A

batch_insert_into_indexed_tree_40_depth_hash_ms
N/A
N/A
0.515
N/A
N/A
N/A
N/A

Miscellaneous
Transaction sizes based on how many contract classes are registered in the tx.

Metric
0 registered classes
1 registered classes

tx_size_in_bytes
74,057
667,841

Transaction size based on fee payment method
| Metric |  |
| - |  |

:robot: Release is at https://github.com/AztecProtocol/aztec-packages/releases/tag/aztec-packages-v0.46.0 :sunflower:

:robot: Release is at https://github.com/AztecProtocol/aztec-packages/releases/tag/barretenberg.js-v0.46.0 :sunflower:

:robot: Release is at https://github.com/AztecProtocol/aztec-packages/releases/tag/barretenberg-v0.46.0 :sunflower:


Update versions (mimic dependabot)

[x] Bump GraphQL versions

Only patches and minors. Tested building and running (both dev and prod), which works.


Hot fix for #157

What did I do?

Updated redis version in pubspec.yaml.

Result
minnu@Minnus-Mac hello_server % serverpod run
Starting Serverpod.
 • Automatic generate and reload are enabled.
 • Running Postgres and Redis in Docker container.

Spinning up serverpod generate (this can take a few seconds).
Starting Docker (for Postgres and Redis).
Waiting for Postgres on localhost:8090.
Waiting for Redis on localhost:8091.
Setup complete. Starting the server.

SERVERPOD version: 0.9.7 mode: development
Insights listening on port 8081
Server id 0 listening on port 8080

Fix #157

This looks correct, but will need to test before I can merge.

I closed this because I merged similar changes (there were some even newer versions out now) on the dev branch.
Also, make sure that when changing package dependencies, edit the template files then run util/update_pubspecs. That ensures all packages are correctly defined everywhere. :)


Prototype gz header migration

🎉 New feature
Prototype part of https://github.com/ignition-tooling/release-tools/issues/698
Summary
The big diff is not so easy to read, so I recommend going commit-by-commit:

https://github.com/ignitionrobotics/ign-utils/commit/b2e10989c84dc238af8a89d9768514d5dcc84b10: move the current ignition header folder to gz
https://github.com/ignitionrobotics/ign-utils/commit/7c6e7e82a504cbab9c257cd0cd20ac9da20f61ba: make several changes needed to support the gz headers
https://github.com/ignitionrobotics/ign-utils/commit/6fb3171bd3600feed20cd0528fd220797dcb5d4a: add dummy ignition/* headers that simply redirect to their gz/ counterpart

This takes a very minimal approach, just to change the include path without changing any namespaces or even the IGNITION_* variables in config.hh. The only macros changed are for header guards of the new gz/* headers.
Test it

Checklist

[ ] Signed all commits for DCO
[ ] Added tests
[ ] Added example and/or tutorial
[ ] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[ ] codecheck passed (See contributing)
[ ] All tests passed (See test coverage)
[ ] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

How do you think we should do the deprecation warnings? Using a #pragma message() in every ignition header?

I'm not sure if #pragma message() works with gcc. I think we could define a macro that uses different directives depending on the compiler and include that in each ignition/* header just before the redirect include of the corresponding gz/ header.

Did you use a script or just prototyped it manually?

I did this manually, but it shouldn't be too hard to script

It would be good to move the files from ignition to gz so we keep their commit history.

I did a git mv command in https://github.com/ignitionrobotics/ign-utils/pull/51/commits/b2e10989c84dc238af8a89d9768514d5dcc84b10 and you can confirm with a git blame that the history is still there:

https://github.com/ignitionrobotics/ign-utils/blame/6fb3171bd3600feed20cd0528fd220797dcb5d4a/include/gz/utils/Environment.hh

It doesn't show up because github is a little confused, but I think our history is preserved with this approach.

When this PR is ready for merging, we should update examples and tests

we can decide how to split the work up into phases, but I believe this approach can be merged independently without breaking anything. We could optionally update the #include <ignition/*> calls in src, test, and examples as you say, but I was trying to see what's the minimal amount that needs to be changed for this first PR.
Some possible steps:

Move header files to gz/* and add redirect headers to ignition/* (included in this PR)
Replace all #include <ignition/*> statements in header files with #include <gz/*> (included in this PR)
Replace all #include <ignition/*> statements in src, test, and examples source code with #include <gz/*> (not included in this PR)
Replace all #include <ignition/*> statements in downstream code
Change ign-cmake3's default value of PROJECT_INCLUDE_DIR from ignition/${IGN_DESIGNATION} to gz/${IGN_DESIGNATION} (or use ${GZ_DESIGNATION} if that has been defined) in IgnConfigureProject.cmake
Remove the REPLACE_IGNITION_INCLUDE_PATH parameter override once its no longer needed
Add using namespace ignition to the ignition/ headers
Add portable #warning equivalent to ignition/ headers

Because GitHub gets confused by the diff in this pull request, I like making a narrow PR like this to add the gz/ headers in a backwards-compatible way. This allows flexibility in deciding when to change the c++ namespaces and add deprecation warnings

we could arguably backport this to fortress since I think it's backwards compatible, but I'm not sure if it's worth it

we could arguably backport this to fortress

Something to keep in mind for later 👍🏽

I'm not sure if #pragma message() works with gcc.

It prints a message for me, not sure what's missing? I think the only downside to it is that it's not an actual warning.

I did a git mv command in https://github.com/ignitionrobotics/ign-utils/commit/b2e10989c84dc238af8a89d9768514d5dcc84b10 and you can confirm with a git blame that the history is still there:

Ah perfect, indeed GitHub gets confused in the PR diff

we can decide how to split the work up into phases

I'm fine with that 👍🏽 We just have to make sure we don't introduce warnings in the intermediate steps.

#53 was merged instead


fractals-ewoud

My code draws a triangle, the biggest one in the fractal, then calculates the midpoints of all sides of the triangle and draws a triangle at a midpoint and the sides, and does that 3 times for all sides of the starting triangle. This is one iteration, and the code randomly generates the number of iterations to generate fractals. This is my first time coding with Javascript, but i have some experience with Python, C#/Unity, and C++/Arduino.

[x] I have read the steps to getting a blot
[x] I am submitting art that...

[x] is algorithmically generated (will meaningfully change each time the program is run)
[x] is your own work that is not copied from somewhere else
[x] doesn't call Math.random() (See the documentation on randomness)
[x] is drawable on Blot itself (doesn't have lines overlap more than 5 times)

[x] Doesn't overlap lines more than 5 times
[x] Fits in the working area of 125x125mm
[x] Doesn't rely on colour to look good

Lastly, link a picture of your art piece here:

This is a great start into generative art! Unfortunately, your submission does not meet the requirements found in the submission rules as it is quite simple (Under what not to include).

I will be closing this PR due to inactivity! Please make a new PR when you have updated your piece.


Looking for screenshots or examples

I'm looking at the repo and it looks like it has potential but there are no photos or pictures of what the app looks like or examples of a gallery etc.
Can you post a few, we are looking at using your code in an app we are looking to build to help disabled children but I have no idea what your code generates.

Thanks for the suggestion - but I'm not sure what we'd screenshot. This template produces an Xcode project - the same as any other project you'd produce in Xcode. It doesn't look like anything in particular - the appearance of the app is up to whatever code you write for your app.
The tutorials for Toga, on the other hand, do include screenshots, as that is a widget toolkit. The tutorial for Briefcase also have screenshots, as it's a tool for deploying code as an app. Briefcase uses this project as part of deploying to iOS.
If you've found this project as a result of a search for Python on iOS support, it's probably the wrong place to start - the README points you in the direction of a more helpful starting point.


Spark etl support es upsert

Spark etl support es upsert

Please handle the third dependency check problem.

Please handle the third dependency check problem.

Ok, thank you


ROX-23714: Lazy load registry repo list

Description
Currently when Central starts all integrations are loaded into memory sequentially.  For each integration a list of associated repositories is created.  Populating this list takes time and delays Central startup (ref).
The repo list is used when determining if a registry integration is a match for an image.
Instead of loading the initial repo list when creating a new image integration, load it lazily upon first call to Match (and then as usual on subsequent calls).
This will result in faster Central startup and reduced initial memory consumption.
In the future repo list should be removed entirely, ACS may be making a wrong assumption that if a repo is not in the list then it is not in the registry, which is contrary to this doc:

... a missing entry does not mean that the registry does not have the repository. More succinctly, the presence of a repository only guarantees that it is there but not that it is not there.

Checklist

[ ] Investigated and inspected CI test results
[X] Unit test and regression tests added
[X] Evaluated and added CHANGELOG entry if required
[X] Determined and documented upgrade steps
[X] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed
Unit tests as well as an infra OCP cluster with a mock registry deployed (broken-image-reg) configured to add a 10 second delay to the /v2/_catalog repo list calls - and 20 pull secrets with unique routes to the broken registry created in the OCP cluster.
Without the fix, Central startup is delayed just under two minutes:
2024/05/21 00:52:14.621790 orchestrator.go:231: Info: Successfully fetched 0 Istio CVEs
2024/05/21 00:53:56.624237 instance_config.go:113: Info: Phonehome telemetry collection disabled.

A followup roxctl image scan returns instantly (error expected):
$ time rctl image scan --image=broken1-broken-image-reg.apps.dc-05-20-1.ocp.infra.rox.systems/broken/error:500 --retries 0
ERROR:	scaning image failed after 0 retries: could not scan image: "broken1-broken-image-reg.apps.dc-05-20-1.ocp.infra.rox.systems/broken/error:500": rpc error: code = Internal desc = image enrichment error: error getting metadata for image: broken1-broken-image-reg.apps.dc-05-20-1.ocp.infra.rox.systems/broken/error:500 error: getting metadata from registry: "Autogenerated https://broken1-broken-image-reg.apps.dc-05-20-1.ocp.infra.rox.systems for cluster remote": failed to get the manifest digest: Head "https://broken1-broken-image-reg.apps.dc-05-20-1.ocp.infra.rox.systems/v2/broken/error/manifests/500": http: non-successful response (status=500 body="")

real	0m0.432s
user	0m0.109s
sys	0m0.046s

With the fix, Central starts up faster:
2024/05/21 00:56:22.685922 orchestrator.go:231: Info: Successfully fetched 0 Istio CVEs
2024/05/21 00:56:23.409195 singleton.go:43: Info: Injecting 86 policies into detectors.

The initial repo list load is instead made during the first call to match, which is demonstrated via a roxctl image scan. Notice how it returns after about 10 seconds (the timeout set on the /v2/_catalog endpoint):
$ time rctl image scan --image=broken1-broken-image-reg.apps.dc-05-20-1.ocp.infra.rox.systems/broken/error:500 --retries 0
ERROR:	scaning image failed after 0 retries: could not scan image: "broken1-broken-image-reg.apps.dc-05-20-1.ocp.infra.rox.systems/broken/error:500": rpc error: code = Internal desc = image enrichment error: error getting metadata for image: broken1-broken-image-reg.apps.dc-05-20-1.ocp.infra.rox.systems/broken/error:500 error: getting metadata from registry: "Autogenerated https://broken1-broken-image-reg.apps.dc-05-20-1.ocp.infra.rox.systems for cluster remote": failed to get the manifest digest: Head "https://broken1-broken-image-reg.apps.dc-05-20-1.ocp.infra.rox.systems/v2/broken/error/manifests/500": http: non-successful response (status=500 body="")

real	0m10.391s
user	0m0.114s
sys	0m0.047s

A second call to roxctl returns immediately indicating the repo list call is/was not made again:
$ time rctl image scan --image=broken1-broken-image-reg.apps.dc-05-20-1.ocp.infra.rox.systems/broken/error:500 --retries 0
ERROR:	scaning image failed after 0 retries: could not scan image: "broken1-broken-image-reg.apps.dc-05-20-1.ocp.infra.rox.systems/broken/error:500": rpc error: code = Internal desc = image enrichment error: error getting metadata for image: broken1-broken-image-reg.apps.dc-05-20-1.ocp.infra.rox.systems/broken/error:500 error: getting metadata from registry: "Autogenerated https://broken1-broken-image-reg.apps.dc-05-20-1.ocp.infra.rox.systems for cluster remote": failed to get the manifest digest: Head "https://broken1-broken-image-reg.apps.dc-05-20-1.ocp.infra.rox.systems/v2/broken/error/manifests/500": http: non-successful response (status=500 body="")

real	0m0.405s
user	0m0.119s
sys	0m0.047s

Rebases gone bad... closing

https://github.com/stackrox/stackrox/pull/11186 <- new PR


This app doesn't work. (Google drive)

The app worked yesterday, but somehow it is broken now.
How to reproduce:

import rclone.conf (This config file is okay, I imported the file on my other computer and it works)

I tried to connect to Google drive profile, saw "Error retrieve directory content".

I tried to add new remote in the app itself, saw "Error creating remote" despite the browser says "Success!"

Tried to clear app data and do step 1~3 (or just 3) again, nothing changes, error happens.

I've tried:

Switch between Wi-Fi and mobile.

Clear app data.

Two Google accounts.

My Android is using Oreo (8.1)

I figured out what was happening.
I used DNS66 to block AD traffics, and somehow one of the rules stopped this APP to work, so I whitelisted this APP, add remote and it works.


Add link to Georgian translation

I've created a Georgian translation of the guide.

@lencioni
Hi, I noticed that you merged some pull requests in the past,
So can you please overview my translation (or just add a link to it)?
Thank you!

Hi! We should be able to take a look at this next week, we are waiting on a colleague with knowledge of Georgian to take a look. 🇬🇪

Hi! We should be able to take a look at this next week, we are waiting on a colleague with knowledge of Georgian to take a look. georgia

It looks like something didn't go right...
If this is enough, I will share a file with a list of my translations.

Apparently I don't have write access to this repo - @backwardok do you still have access? Mind approving if so? @lencioni is out on recharge. 🙇 🙏

I'll land it, thanks


Fix safari mobile mouseover #149

fix issue #149

Coverage increased (+0.01%) to 90.672% when pulling f226ec6e455cdf7206739005bbd621824ceed769 on jmoulin-vp:fix-safari-mobile-mouseover into 9e936fa3083777b07b003038b6b0db3306f571e5 on furqanZafar:master.

Coverage decreased (-0.1%) to 90.559% when pulling 8a388e65949e13e6b1a2c3ceb725f2741f782075 on jmoulin-vp:fix-safari-mobile-mouseover into 9e936fa3083777b07b003038b6b0db3306f571e5 on furqanZafar:master.

Coverage decreased (-0.08%) to 90.582% when pulling 2268f2e8001242b38c5a23ce311e3f29e4a41e4b on jmoulin-vp:fix-safari-mobile-mouseover into 9e936fa3083777b07b003038b6b0db3306f571e5 on furqanZafar:master.

Hi, can you check this pull request please ?

Thanks 👍


[RFC] Middleware API for cross-environment organization and end-user extensibility

Discuss!

 //Stdout would be default console.log middleware
import debug, { StdoutMiddleware } from 'debug';
const log = debug('log');
// Add some sort of new middleware element
log.use((data, next) => ... )
// replace Stdout (console.log) with console.info at same position in middleware array
log.replace(StdoutMiddleware, (data, next) => {
  console.info.apply(console, data);
  next();
});

// maybe create a stderr only log for failed uploads:
const appUploadLog = debug('app:upload');

const appUploadErrLog = appUploadLog
  .child('error')
  .replace(StdoutMiddleware, (data, next) => {
    console.error.apply(console, data);
    next()
  });

Why not just use EventEmitters? This module doesn't need to be super over-enterprised with middleware and the like.

Maybe not with event emitters, but #556 has a newer and more relevant discussion on this.


Todoist all tasks not working

Hi - great app man, thanks for making this!
I've been trying to pull all tasks for last couple of days but it isnt' working.
Pulling today's tasks has been fine though.
After I type the command the cursor just continues blinking and nothing happens whereas today's tasks results in the relevant tasks appearing under "inbox" with the deadline and name of task appearing accordingly.
Anything I might be doing wrong or a potential bug?

Apologies I think there was a bug that was introduced somehow. Please update to the latest version and try again.
Thank you.

I just tried again - today's works, but still no good for all tasks post update.

Hey @hkgnp - just heads up think the new update may have broke the plugin. I tried updating to the newest version and it's still not working

It's a bit finicky - I just managed to get "Today's" tasks to come through, but not the overdue ones.
I also just moved the overdue ones to "today" and tried pulling again but it didn't work.

Please try v5.0.3.

It works thanks man!
On Mon, Dec 19, 2022, 05:10 hkgnp @.***> wrote:

Please try v5.0.3.
—
Reply to this email directly, view it on GitHub
https://github.com/hkgnp/logseq-todoist-plugin/issues/48#issuecomment-1356847345,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AW4VWXRGZ2BLWOP6BVAYH2DWN5HP3ANCNFSM6AAAAAARY6YXKA
.
You are receiving this because you authored the thread.Message ID:
@.***>


Dark mode switching while Poedit is running leaves artifacts

In macOS 10.15.3, Poedit 2.3's sidebar will stay in light mode when the macOS has switch to dark mode. However, when the sidebar stays in light mode, the text in the sidebar has changed to white, which makes it impossible to see translation suggestions as well as the notes for translators.

No, it is that way temporarily when you switch the theme. Restart Poedit.
There are more such discrepancies. I'm aware of them. It's a rather minor issue.


Please update the package on npm

Thanks for the component!
The npm package is a bit outdated and doesn't have https://github.com/sharu725/youtube-embed/pull/16 applied. I would like to request a bump of the version with the new code!

Thank you @henrikvilhelmberglund. Updated.


[Suggestion] Nitor within Garden Latern

I know you can use the candles from Thaumcraft but what about using the Nitor as a light source within the garden latern?

Seconding this.

How is nitor typically used?  I've never actually played Thaumcraft.

Nitor is typically a light source on its own, just a free floating flame.  However, it would also look quite nice caged inside the Garden Lanterns.
You can see how it looks here - http://i.imgur.com/YXFbhix.gif

OMG that would look amazing!


Auto-generate documentation, and host it somewhere

First off - Thanks for doing this.  I'm wondering where the class/function/API documentation went? A copy of the old would be close enough?

It's in the back of my mind, but I don't have a ticket for it. I would like to:

Register a domain icaldotnet.net or something
Have NodaTime-like API documentation

I looked into it a little while ago, and there didn't seem to be any obvious winners in the turn-my-XML-into-reasonable-docs department, but Sandcastle(?) seemed to be the most powerful.

GhostDoc is another solution, it generates MSDN style documentation and allows you give code examples. I believe you have the option to output as html.

Any progress on this?