refactor(tangle-dapp): Change homepage to account page

Summary of changes
Provide a detailed description of proposed changes.

Moved account page to /.
Nomination page is now under /nomination.
Tested most links I could reasonably access.

Proposed area of change
Put an x in the boxes that apply.

[ ] apps/bridge-dapp
[ ] apps/hubble-stats
[ ] apps/stats-dapp
[x] apps/tangle-dapp
[ ] apps/testnet-leaderboard
[ ] apps/faucet
[ ] apps/zk-explorer
[ ] libs/webb-ui-components

Reference issue to close (if applicable)
Specify any issues that can be closed from these changes (e.g. Closes #233).

Closes #2092

Screen Recording
If possible provide a screen recording of proposed change.

Code Checklist
Please be sure to add .stories documentation if any additions are made to libs/webb-ui-components.

[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

Can you deal with merge conflicts @yurixander so we can merge this too


BaseStringHelper::truncateHtml calls HTMLPurifier lexer with null $context

What steps will reproduce the problem?
Calling StringHelper::truncateWords($model->body, 20, '...', true) on server without php-xml package installed (DOMDocument class missing in php).
What is the expected result?
StringHelper truncates $model->body to 20 words
What do you get instead?
PHP Fatal Error – yii\base\ErrorException
Call to a member function register() on null in /basepath/vendor/ezyang/htmlpurifier/library/HTMLPurifier/Lexer/DirectLex.php at line 79
Additional info
BaseStringHelper::truncateHtml line 157 calls HTMLPurifier lexer->tokenizeHTML with $context set to null. It works in case of DOMLex (which depends on DOMDocument class) but in case it uses DirectLex, it tries to call register method on null context. I assume DirectLex is poorly programmed here but I think a fix should be also considered on the yii2 side, maybe something like
$lexer = \HTMLPurifier_Lexer::create($config);
$context = new \HTMLPurifier_Context;
$tokens = $lexer->tokenizeHTML($string, $config, $context);

in truncateHtml() would be sufficient.
@samdark The same issue is reported here, btw.

Q
A

Yii version
2.0.9

PHP version
5.6/any

Operating system
Linux/any

@mattheobjornson Are you able to check if the tests still pass when providing a $context object?
I had a quick look and it doesn't get used much outside of DirectLex so shouldn't cause any issues.

Seems that phpunit tests also depend on DOMDocument class (phpunit.xml.dist cannot be parsed) so they do not run at all without it. This may be as well the reason why this issue was not discovered earlier.
For test purposes I forced HTMLPurifier to load DirectLex in all cases.
Without fix:
mattheo@e350m1-lubuntu:~/development/projects/git/yii2$ php vendor/bin/phpunit --group=helpers
PHPUnit 4.8.27-9-g7b58ff0 by Sebastian Bergmann and contributors.

...............................................................  63 / 248 ( 25%)
............................................................... 126 / 248 ( 50%)
.......................................................EE...... 189 / 248 ( 76%)
...........................................................

Time: 684 ms, Memory: 26.00MB

There were 2 errors:

1) yiiunit\framework\helpers\StringHelperTest::testTruncate
Error: Call to a member function register() on null

/home/mattheo/development/projects/git/yii2/vendor/ezyang/htmlpurifier/library/HTMLPurifier/Lexer/DirectLex.php:79
/home/mattheo/development/projects/git/yii2/framework/helpers/BaseStringHelper.php:159
/home/mattheo/development/projects/git/yii2/framework/helpers/BaseStringHelper.php:108
/home/mattheo/development/projects/git/yii2/tests/framework/helpers/StringHelperTest.php:110
/home/mattheo/development/projects/git/yii2/vendor/phpunit/phpunit/phpunit:47

2) yiiunit\framework\helpers\StringHelperTest::testTruncateWords
Error: Call to a member function register() on null

/home/mattheo/development/projects/git/yii2/vendor/ezyang/htmlpurifier/library/HTMLPurifier/Lexer/DirectLex.php:79
/home/mattheo/development/projects/git/yii2/framework/helpers/BaseStringHelper.php:159
/home/mattheo/development/projects/git/yii2/framework/helpers/BaseStringHelper.php:131
/home/mattheo/development/projects/git/yii2/tests/framework/helpers/StringHelperTest.php:127
/home/mattheo/development/projects/git/yii2/vendor/phpunit/phpunit/phpunit:47

FAILURES!
Tests: 248, Assertions: 922, Errors: 2.
mattheo@e350m1-lubuntu:~/development/projects/git/yii2$

With fix (providing valid context object):
mattheo@e350m1-lubuntu:~/development/projects/git/yii2$ php vendor/bin/phpunit --group=helpers
PHPUnit 4.8.27-9-g7b58ff0 by Sebastian Bergmann and contributors.

...............................................................  63 / 248 ( 25%)
............................................................... 126 / 248 ( 50%)
............................................................... 189 / 248 ( 76%)
...........................................................

Time: 696 ms, Memory: 26.00MB

OK (248 tests, 938 assertions)
mattheo@e350m1-lubuntu:~/development/projects/git/yii2$


Hide Lines

After a order is placed the lines should auto Hide so we get a sense of confirmation that the order got placed

Do you mean like automatically?

yeah meaning after the order is placed, the lines should hide so we can see the order is placed and a sense of confirmation, what do you think ?

I see. I think it would be an overkill to add such a feature considering that the lines can now be hidden with a single click or with a single press of a key.

ok


wip:fix: included all accessed changed slots

closes https://github.com/paradigmxyz/reth/issues/8202
we need to manually track reverted changed slots in case they're handled gracefully

I would close this PR in the favor of this change: https://github.com/bluealloy/revm/pull/1437
This would preserve any account that got loaded inside revm and would make it easier to construct full access list

nice!


Update dependencies

excoveralls to 0.10.1
credo to 0.10.2

Coverage remained the same at 100.0% when pulling 3a828a1ea43e1708bae607db316bf25ebcc9597b on update_deps into 4a0df962b9bf7d04d3ba9552df81e17212e979bd on master.

Coverage remained the same at 100.0% when pulling 3a828a1ea43e1708bae607db316bf25ebcc9597b on update_deps into 4a0df962b9bf7d04d3ba9552df81e17212e979bd on master.

Coverage remained the same at 100.0% when pulling 3a828a1ea43e1708bae607db316bf25ebcc9597b on update_deps into 4a0df962b9bf7d04d3ba9552df81e17212e979bd on master.


[3.x] curvetexture and gradientexture default to 64 as a size

Sister PR for #47410

Sister PR is closed too


#6946 Run hooks on merge/edit and cope with protected branches

Fix #6946 by setting and checking PullRequest ID on pushing if we're pushing from a PR.
Fixes #6003.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@7dd9837). Click here to learn what that means.
The diff coverage is 32.5%.

@@            Coverage Diff            @@
##             master    #6961   +/-   ##
=========================================
  Coverage          ?   41.46%
=========================================
  Files             ?      441
  Lines             ?    59694
  Branches          ?        0
=========================================
  Hits              ?    24751
  Misses            ?    31721
  Partials          ?     3222

Impacted Files
Coverage Δ

models/branches.go
43.33% <ø> (ø)

routers/private/branch.go
13.11% <0%> (ø)

routers/private/internal.go
57.14% <100%> (ø)

models/helper_environment.go
87.5% <100%> (ø)

models/pull.go
49.71% <50%> (ø)

Continue to review full report at Codecov.

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

OK two things:

I'm getting a deadlock on merges which is interesting because I don't think that should be happening.
I'm also getting an error on the tests.

Right I think I'll leave this as WIP but open a second PR to stop running the push hooks on merge.

OK I've discovered the issue here, TestSearchRepos is broken.

Now working after applying #7004 to fix that broken test

Please resolve conflicts

Conflicts resolved.

So the next step once we have this PR is to enforce that we have to have an appropriate Gitea environment before allowing pushing to gitea repositories - this will prevent the numerous issues that we get whereby people abuse Gitea repositories by pushing outside of Gitea.

glad to see this fixed in gitea
thank you @zeripath :taco:
related gogs issue #5120


/v2/api-docs outputs JSON not XML, causes parsing error

In my Spring MVC config, I have:
    @Override
    protected void configureContentNegotiation(ContentNegotiationConfigurer configurer) {
        configurer
                .favorPathExtension(true)
                .defaultContentType(MediaType.APPLICATION_JSON);
    }

which is required by my application. However, this breaks Swagger when loaded into the same DispatcherServlet; it returns JSON instead of XML to the browser, resulting in:
XML Parsing Error: not well-formed Location: http://localhost:8623/v2/api-docs Line Number 1, Column 1:
{"swagger":"2.0","info":{"description":"Api Documentation", ...

Is the SwaggerController missing a @Produces to force JSON to serialize this as XML or something? Is there an easy way to configure that Controller despite me not really having access to it? I only added @EnableSwagger2 and have no other dialogue with that code.

Actually, removing defaultContentType() does not fix this, so that's a red herring. I still get JSON. :-(

Additional versions:
Firefox 42.0, nothing special.
Jetty 9.2.3.v20140905
Spring 4.1.7.RELEASE (with a couple of 4.1.6 bits left over)

Closing due to misunderstanding of relationship between the static HTML and the Controller.


feat(dynamic-input): add disabled prop

#4055

Codecov Report
Base: 62.55% // Head: 62.55% // Decreases project coverage by -0.00% :warning:

Coverage data is based on head (621ed07) compared to base (2131cdc).
Patch coverage: 66.66% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4114      +/-   ##
==========================================
- Coverage   62.55%   62.55%   -0.01%
==========================================
  Files        1069     1069
  Lines       22380    22385       +5
  Branches     5862     5867       +5
==========================================
+ Hits        13999    14002       +3
  Misses       6863     6863
- Partials     1518     1520       +2

Impacted Files
Coverage Δ

src/dynamic-input/src/PairPreset.tsx
72.72% <ø> (ø)

src/dynamic-input/src/DynamicInput.tsx
54.08% <60.00%> (+0.19%)
:arrow_up:

src/dynamic-input/src/InputPreset.tsx
100.00% <100.00%> (ø)

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Machinelearningservices microsoft.machine learning services 2022 12 01 preview

ARM API Information (Control Plane)
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Azure 1st Party Service can try out the Shift Left experience to initiate API design review from ADO code repo.  If you are interested, may request engineering support by filling in with the form https://aka.ms/ShiftLeftSupportForm.
Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

[ ] new service onboarding
[x] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month.
When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month.
By default, Azure SDKs of all languages (.NET/Python/Java/JavaScript for both management-plane SDK and data-plane SDK, Go for management-plane SDK only ) MUST be refreshed with/after swagger of new version is published. If you prefer NOT to refresh any specific SDK language upon swagger updates in the current PR, please leave details with justification here.

Contribution checklist (MS Employees Only):

[ ] I commit to follow the Breaking Change Policy of "no breaking changes"
[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

Applicability: :warning:
If your changes encompass only the following scenarios, you should SKIP this section, as these scenarios do not require ARM review.

Change to data plane APIs
Adding new properties
All removals

Otherwise your PR may be subject to ARM review requirements. Complete the following:

[ ] Check this box if any of the following apply to the PR so that the label "ARMReview" and "WaitForARMFeedback" will be added by bot to kick off ARM API Review.  Missing to check this box in the following scenario may result in delays to the ARM manifest review and deployment.

Adding a new service
Adding new API(s)
Adding a new API version
-[ ] To review changes efficiently, ensure you copy the existing version into the new directory structure for first commit and then push new changes, including version updates, in separate commits. You can use OpenAPIHub to initialize the PR for adding a new version. For more details refer to the wiki.

[ ] Ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If you have any breaking changes as defined in the Breaking Change Policy, request approval from the Breaking Change Review Board.
Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Additional details on the process and office hours are on the Breaking Change Wiki.
NOTE: To update API(s) in public preview for over 1 year (refer to Retirement of Previews)
Please follow the link to find more details on PR review process.

@ivaliyivanov - Your PR contains breaking changes. You will need to either make sure there are no breaking changes across versions or reach out to Azure Breaking Changes Reviewers<EMAIL_ADDRESS>for the approval.

"/providers/Microsoft.MachineLearningServices/operations": {

201 is a valid response code for PUT. 202 can be removed

Refers to: specification/machinelearningservices/resource-manager/Microsoft.MachineLearningServices/preview/2022-12-01-preview/machineLearningServices.json:37 in a27c53e. [](commit_id = a27c53e305b3652d40966064c854bbee6fb15942, deletion_comment = False)

"/subscriptions/{subscriptionId}/resourceGroups/{resourceGroupName}/providers/Microsoft.MachineLearningServices/workspaces": {

All Tracked resources must implement List calls at Subscription and Resource Group scopes

Refers to: specification/machinelearningservices/resource-manager/Microsoft.MachineLearningServices/preview/2022-12-01-preview/machineLearningServices.json:265 in a27c53e. [](commit_id = a27c53e305b3652d40966064c854bbee6fb15942, deletion_comment = False)

      "type": "string"

can this be an enum ?

Refers to: specification/machinelearningservices/resource-manager/Microsoft.MachineLearningServices/preview/2022-12-01-preview/machineLearningServices.json:2335 in 43a9ffb. [](commit_id = 43a9ffbee26e6128d4db6d84115c8ce469cade3d, deletion_comment = False)

        "format": "int32"

this is a breaking change , you would need to support both top and count

Refers to: specification/machinelearningservices/resource-manager/Microsoft.MachineLearningServices/preview/2022-12-01-preview/mfe.json:6186 in 43a9ffb. [](commit_id = 43a9ffbee26e6128d4db6d84115c8ce469cade3d, deletion_comment = False)

        "format": "int32"

this is a breaking change , you would need to support both top and count
Refers to: specification/machinelearningservices/resource-manager/Microsoft.MachineLearningServices/preview/2022-12-01-preview/mfe.json:6186 in 43a9ffb. [](commit_id = 43a9ffb, deletion_comment = False)

it's a change in a preview version, so it should not be considered as a breaking one.

      "type": "string"

can this be an enum ?
Refers to: specification/machinelearningservices/resource-manager/Microsoft.MachineLearningServices/preview/2022-12-01-preview/machineLearningServices.json:2335 in 43a9ffb. [](commit_id = 43a9ffb, deletion_comment = False)

not a blocking comment

@ivaliyivanov Please fix the validation errors for LintDiff, Avocado and ModelValidation, details can be found in https://github.com/Azure/azure-rest-api-specs/pull/21761#issuecomment-1338332880

/azp run


Tray with libdbusmenu-gtk3 is buggy

Describe the bug
I'm upgrading the Gentoo package to the new version but I've noticed that the tray behaves strangely. When I click the tray icon then most of the time, nothing happens. I have these warnings/errors in the stdout:

2024-11-25T05:06:30.093663Z  WARN GTK: 112: [LIBDBUSMENU-GLIB] Unable to get session bus: Operation was cancelled
2024-11-25T05:06:32.540958Z ERROR GTK: 111: [LIBDBUSMENU-GLIB] CRITICAL: dbusmenu_menuitem_send_about_to_show: assertion 'DBUSMENU_IS_MENUITEM(mi)' failed

But on other times it works fine and the menu is shown. Sometimes it also works once and then stops working.
To Reproduce
Steps to reproduce the behavior:

Click on a tray icon (I'm using the one from Steam)
Nothing happens
Check the stdout

Expected behavior
The menu opens all the time.
System information:

Distro: Gentoo
Compositor: niri
Ironbar version: 0.16.1

It seems odd that'd be tempermental, but this to me reads like an environmental issue. Perhaps a race condition between Ironbar starting and your dbus session (env vars) loading. There have been other Niri-specific tray issues recently too.
Are you able to confirm if this is still an issue for yourself on another compositor please?

Interesting thought!
I gave it a try with Hyprland. Observations:

The warning message from my initial post is always there and apparently has nothing to do with the issue
At first, I was not able to reproduce the issue. But it seems when I kill ironbar, freshly start it and directly go for the tray menu, it will fail. The error message was the same on Hyprland
After ironbar is running for a while, it seems the tray menu fixes itself and it starts working (permanently?). This is where it seems niri behaves differently than Hyprland. With niri it felt like on/off randomly whereas in Hyprland it was first off, then on

On Sway I often get issues when turning my monitors back on.
For the tray it happens that old apps are still shown and I have to reopen ironbar.


Feature request: Export/Import IDs

Hi,
great APP.
Most Flarm aircrafts just publish their ID. So on our airfield we have many manually configured IDs to quickly identify the known aircrafts. And everyone has to type them into yxour App manually.
A simple Import/Export/Sharing option would be great.

Hi @Beejai,
this feature is already in development but will take more time than expected.
Thanks for joining GitHub and creating this feature request.
Progress will be tracked here.
Dominik

Hello @Beejai,
we added a csv import/export and released a beta version. Could you please check out and comment:
https://play.google.com/apps/testing/com.meisterschueler.ognviewer
But please be aware, that the csv format is not fixed, so the final release could have a slightly different format (e.g. separator ";" instead of "," or additional columns...)

Thanks. But it does not work on my phone.
"Error during Export."
Samsung S5, Android 6.0.1


Syslog messages don't show up in Visual Syslog Server nor WinSyslog

Using WireShark I noticed unprintable characters and a wrong timestamp:

Possibly related: lora-aprs/LoRa_APRS_iGate#227

what you see after CRC-ERROR are the data not correctly decoded by the LoRa Radio, it include any kind of characters as it is not parsed. These data are useful for debuging. If you notice, the station you are receiving is IK2UUJ and the level is -142dBm, very marginal. This CRC-ERROR is due to the location and distance, in other cases if you see the level higher it may be due to disturb or colision.

Alright, that's what I thought.
According to your screenshot you're receiving the messages just fine in Visual Syslog Server. Is there anything that you had to change in the configuration? Perhaps the filters?

Alright, that's what I thought.
According to your screenshot you're receiving the messages just fine in Visual Syslog Server. Is there anything that you had to change in the configuration? Perhaps the filters?

I guess the problem with timestamp is on the syslog server more than on the igate firmware as it just uploads the log but the log does not push time to it. Its the syslog server that saves his own timestamp into the log.

as far as I've tested and others told me, the timestamp is added on the syslog server side.
not on yours?

Right now I don't see any messages from iGate, but I'm pretty sure the timestamp is indeed added by the Syslog server.
However, appending the timestamp from the firmware's side would be useful as a reference, especially when there's high latency involved.

Right now I don't see any messages from iGate, but I'm pretty sure the timestamp is indeed added by the Syslog server.
However, appending the timestamp from the firmware's side would be useful as a reference, especially when there's high latency involved.

how much latency are you experimenting?

Nothing too extreme, but in certain environments it may be useful to know exactly when an event happens.
It's just a suggestion, not something I actually need.

will save this idea for later (when I add NTP server time to igate)
meanwhile I'll close it.

Sounds good!


Create a SECURITY_CONTACTS file.

As per the email sent to kubernetes-dev[1], please create a SECURITY_CONTACTS
file.
The template for the file can be found in the kubernetes-template repository[2].
A description for the file is in the steering-committee docs[3], you might need
to search that page for "Security Contacts".
Please feel free to ping me on the PR when you make it, otherwise I will see when
you close this issue. :)
Thanks so much, let me know if you have any questions.
(This issue was generated from a tool, apologies for any weirdness.)
[1] https://groups.google.com/forum/#!topic/kubernetes-dev/codeiIoQ6QE
[2] https://github.com/kubernetes/kubernetes-template-project/blob/master/SECURITY_CONTACTS
[3] https://github.com/kubernetes/community/blob/master/committee-steering/governance/sig-governance-template-short.md

Required SECURITY_CONTACTS file still does not exist. Please resolve as soon as possible.

kubernetes/contrib still needs a SECURITY_OWNERS file.
/assign @eparis @k8s-merge-robot @mwielgus @aledbf @lavalamp @brendandburns


Revamp UI

1)Fresh Screen Added for Get Started
2)Ask permissions in Get Started screen itself
3)Add Animations to hanging Friend Pendant
4)On Permission deny, throw a dialog
5)Redirect user to system settings to grant permission for bluetooth, notifications permission is not mandatory
6)Update App with New Logo

Rename App to Friend from FriendInWearable


Add gnuplot scripts

Fix #744

Does it work OK for you? (workingDirectory thing)

Yes, completely. Travis failure seems unrelated too.

May I ask you to add a Readme entry too?

Done !


Circular Gradient on waves ripple

In the new materialize CSS update, there's a small gradient on the waves ripple.
Here's what I mean:

In the previous version, it looked like this:

Notice the difference?
It becomes a bit more clear when you add the waves-effect class to a large element:

This does not follow material design guidelines

Video example: https://fast.wistia.net/embed/iframe/gzr1k22mil?videoFoam=true

We updated waves.js with 1.1.0 (https://github.com/materializecss/materialize/commit/fce7b8be7b94cb8d0792f560694aa72cc12abcdb) so this will be due to that. The original project seems fairly dead so we may have to patch this ourselves.

Ok, thanks Allie for pointing that out.
I look in the Project and its fairly small, so it makes sense as you said to maintain the code ourselves.
I looked in the project in the sass folder an i think i already found the gradient which causes this behavoir.
As i tested it on their project-page, it does not ooks too bad to me, so I would suggest that we should discuss here if we should change it according to the google specs? What are the benefits of changing the design? Is it "faster" to use no gradient?
Thanks

https://github.com/materializecss/materialize/blob/a088f1480879fcb5da1b6daab3512821258e259e/sass/components/_waves.scss#L49 basically this should be a hard gradient, not smoothed over

Thats awesome! Thank you 🙏

Not really sure why but it also sticks around after using until the user clicks off (using the undocumented btn-flat to make it clear)
https://user-images.githubusercontent.com/13716824/117042254-a209f200-ad03-11eb-9a42-c833a44cb121.mp4

Hm, maybe some focus or class related issue?

Also, could there be a class for a centered ripple?

new implementation in v2-dev


Milestone 0.1.10

Coverage remained the same at 100.0% when pulling 935d7d1735fda275aaff1795494e03bc0e8f7391 on milestone-0.1.10 into 31f974874d42e555913bd7cc1c2f2cc4a2d8d273 on master.

Coverage remained the same at 100.0% when pulling 3c9734a7a8bc6c0892985502000b3ed0ebe4bdfc on milestone-0.1.10 into 31f974874d42e555913bd7cc1c2f2cc4a2d8d273 on master.

Coverage remained the same at 100.0% when pulling 3c9734a7a8bc6c0892985502000b3ed0ebe4bdfc on milestone-0.1.10 into 31f974874d42e555913bd7cc1c2f2cc4a2d8d273 on master.


Set port to 8053 in BIND configuration example, fix typos

Thank you for this submission @elnappo

I've changed it based on https://github.com/digitalocean/bind_exporter/blob/master/bind_exporter.go#L438 and https://github.com/digitalocean/bind_exporter/issues/24#issuecomment-290298767
I'm not sure if there is something like a "default" port. BINDs documentation mentions port 8888 (https://ftp.isc.org/isc/bind9/cur/9.10/doc/arm/Bv9ARM.ch06.html#statistics), some how-tos/tools are using 8080 (https://kb.isc.org/article/AA-00769/0/Using-BINDs-XML-statistics-channels.html) or 8053 (https://collectd.org/wiki/index.php/Plugin:BIND).
It does not matter for me which port is the "default" port, but it should be consistent between README.md and the actual default value for the port flag.

lgtm


Custom cookie loaders

This change allows for the use of custom functions for loading and saving cookies by the relog utility.
loadCookie(): ?{{string} cookie, {?number} time}
saveCookie(cookie: string): void
It includes documentation in README.md and 3 sample loaders for popular databases (Postgres, Mysql and Msql).
Notes
Automated tests have been run whoever I did not notice any unit tests. If all of the dynamic tests are manual, do I need to run or schedule them?
closes #41
fixes #48

At this point, I have finished everything I plan to add.

fixes #48

What is preventing this from being merged?

I have experienced difficulty contacting the owner to assist in working on this framework. As far as I am aware no changes have been requested.

The only apprehension I have about this is the unreliability of Roblox's API. I haven't seen many requests for this feature either.

I would recommend to merge.

I'm honestly surprised this hasn't been closed by now. I made this back in February 2019, 14 months ago. Early on a minor correction was requested but I never received any requests for changes after that.
I've since moved on from Roblox entirely (due to disagreements with Roblox's privacy policy) so I won't be making any updates to it beyond this simple merge conflict resolution today.

This could be closed now thanks to v4.6.0


Feature Request: Include folders

I have a monstrous Angular app.  The component hierarchy errors out due to stack size.  There's a way to exclude folders, could we have a way to 'include' folders?  If set, then only those folders are scanned.

Thank you for the feedback :thumbsup:
I will look into it.

@ratsey in my opinion this IncludeFolder-setting is a setting specific for each project. The only way I think this could be implemented, is by introducing a settings-file for the vscode-AngularTools extension that is located in your project folder.
I'm thinking something like the .vscodeignore and the .gitignore. I could make the extension read a .vscodeangulartools json settings file where you could have the list of include directories. By specifying the included directories setting, then only those directories in your project would be scanned.
Any objections to this idea?

No objections, sounds perfect!
On Thu, Jun 15, 2023, 10:41 PM Allan Simonsen @.***>
wrote:

@ratsey https://github.com/ratsey in my opinion this
IncludeFolder-setting is a setting specific for each project. The only way
I think this could be implemented, is by introducing a settings-file for
the vscode-AngularTools extension that is located in your project folder.
I'm thinking something like the .vscodeignore and the .gitignore. I could
make the extension read a .vscodeangulartools json settings file where you
could have the list of include directories. By specifying the included
directories setting, then only those directories in your project would be
scanned.
Any objections to this idea?
—
Reply to this email directly, view it on GitHub
https://github.com/CoderAllan/vscode-angulartools/issues/439#issuecomment-1594010651,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABVTXGG2S4IOXVNPWYLLOW3XLPBUPANCNFSM6AAAAAAZFGPCJE
.
You are receiving this because you were mentioned.Message ID:
@.***>

@ratsey I have now uploaded a new version 1.11.0 of the AngularTools extension. Please add a .vscodeangulartools file as described in the README.md file and test if it has solved your issue. There may be other if your directory structure is very big, but let me hear if the current solution to the issue has worked.


Using the Access Token

Question
I want to use the access token generated from Add-PodeAuth in other WebPAges.
I cant find how to do this

Look at the examples and the tutorial. You should be able to find your answer

I cant find the answer :(
I have the following code:
Add-PodeWebPage -Name 'Subscriptions' -Icon 'Settings' -ArgumentList $accesstoken -ScriptBlock {
        param($Accesstoken)

        New-PodeWebCard -Content @(
            New-PodeWebTable -Name 'Subscriptions' -ArgumentList $accesstoken -ScriptBlock {
                param($Accesstoken)
                $config = Get-PodeConfig

                $tenantid = $config.tenantId
                $Uri = "https://login.microsoftonline.com/$tenantid/oauth2/v2.0/token"
                $Headers = @{'Content-Type' = 'application/x-www-form-urlencoded' }
                $Method = 'POST'
                $Body = @{
                    grant_type          = 'urn:ietf:params:oauth:grant-type:jwt-bearer'
                    scope               = 'https://management.azure.com/.default'
                    client_id           = $config.clientId
                    client_secret       = $config.clientSecret
                    assertion           = $accessToken
                    requested_token_use	= 'on_behalf_of'
                }

                $token = (Invoke-WebRequest -Uri $Uri -Method $Method -Headers $Headers -Body $body).Content | ConvertFrom-Json -Depth 10

                $uri = "https://management.azure.com/subscriptions?api-version=2022-12-01"

                # Set up the headers, including the authorization token
                $headers = @{
                    "Authorization" = "Bearer $($token.access_token)"
                    "Content-Type"  = "application/json"
                }

                (Invoke-RestMethod -Uri $uri -Method Get -Headers $headers).value | Select-Object DisplayName, SubscriptionId, State
            }
        )
    }

I am looking for a way to not have access tokens as parameters. I guess I could save them as secret, but it is only for a session.

This is a Pode.Web question. You posted on the wrong forum.
It looks like Add-PodeWebPage is missing the Authentication parameter.
I suggest to post this question in Discord

Hey @Dylan-Prins,
It looks like you're using OAuth with AAD? There's a write-up of  it here: https://badgerati.github.io/Pode/Tutorials/Authentication/Inbuilt/AzureAD/#full-example
The $accessToken will be available as a parameter to the scriptblock supplied to Add-PodeAuth. You can either add the access token to the $user object returned and be able to retrieve it via $WebEvent.Auth.User in your page/table. Or, you could add it to the user's session in the Add-PodeAuth script via $WebEvent.Session.Data.AccessToken = $accessToken, and the retrieve accordingly as well.
Hope that helps :)


Child workflows should be visible from the parent workflow in the UI

Summary
When I kick off a child workflow using a WorkflowTemplate, like this:
    - name: create-and-trigger-workflow-template
      serviceAccountName: foo
      inputs:
        parameters:
          - name: domain
          - name: snowflake-env
      resource:
        action: create
        manifest: |
          apiVersion: argoproj.io/v1alpha1
          kind: Workflow
          metadata:
            generateName: "{{inputs.parameters.domain}}-domain-{{inputs.parameters.snowflake-env}}-workflow-"
          spec:
            serviceAccountName: foo
            arguments:
              parameters:
              - name: domain
                value: "{{inputs.parameters.domain}}"
              - name: snowflake-env
                value: "{{inputs.parameters.snowflake-env}}"
            workflowTemplateRef:
              name: foo-domain-templates
        successCondition: status.phase == Succeeded
        failureCondition: status.phase in (Failed, Error)

I should be able to see the child workflow and all that goes on inside of it from the parent level workflow in the UI. Instead, I can only see the status / exit code of the child workflow:

Use Cases
When would you use this?
Any time someone wants to use a WorkflowTemplate from within a dag or steps, they must instantiate a child workflow in this fashion.

Message from the maintainers:
Love this enhancement proposal? Give it a 👍. We prioritise the proposals with the most 👍.

This is a nice idea, but it is not it works. There is no formal concept of "workflow-of-workflows". It would not be possible to add this feature without a massive redesign.

@alexec what about something simple like when a template invokes resource: action: create and the kind is a workflow, we add a label or annotation to it that keeps the identifier of the parent, so the UI has something to connect them together?

(disclaimer, I know next-to-nothing about how the innards of argo / argo workflows works. I am a naïve user making a blind suggestion 😄  )

I'm not saying it is not possible, just that it is a great deal of work.

closing, stale, also no longer need

Duplicate of #6922, which is fairly popular and still desired.


HIVE-27621: Backport of HIVE-25697: Upgrade commons-compress to 1.21

JIRA link - https://issues.apache.org/jira/browse/HIVE-27621

@sankarh @dengzhhu653 @vihangk1 Can you please review and merge this. A clean cherry pick


Update ja/docs/concepts/services-networking/service to v1.19

Fix: https://github.com/kubernetes/website/issues/26892
Update ja/docs/concepts/services-networking/service.md

/assign @nasa908

/assign @nasa9084

/approve

/lgtm


nockback: ability to invalidate recorded fixtures using 'before' hook or by time expiration

hi, is there a way I can pass a hint to nockback which will make it ignore recorded fixtures and re-record them?
some kind of cancelation token passed back by the 'before' hook,
in there I can add my custom invalidation logic - e.g. check if original request time has exceeded...
current context load method checks if fixture exists and and sets the isLoaded flag,
in case it will be set to false, flow should work as expected (consider fixture not existing -> record will take place)
relevant code section:
if( fixture && fixtureExists(fixture) ) {
    var scopes = nock.loadDefs(fixture);

    // original
    // applyHook(scopes, options.before);

   // general idea \ suggestion
   var isInvalidated = applyHook(scopes, options.before);

    scopes = nock.define(scopes);
    applyHook(scopes, options.after);

    context.scopes = scopes;
    context.isLoaded = !isInvalidated;
  }

i can workaround it by creating time fingerprinted fixture files (myfixture_20160221.json),
but it'll generate a lot of garbage data.

any comment?


[e2e] enable cluster dns for tests to work

last 2 failures in e2e tests seem to indicate we need the kube-dns installed and running

oops. looks like already dine in #133


NFC Wifi and Phone Number Scripts

Hello!
I created two python scripts based on your code to generate a WiFi NFC tag as well as a Phone Number NFC tag for the Flipper Zero.
nfc_gen_phone.py - Tap your phone to the emulated tag on the Flipper Zero and your device will prompt you to call the number you've encoded.
nfc_gen_wifi.py - Tap your phone to the emulated tag on the Flipper Zero and your device will join the WiFi network you've encoded.

Hello!

Looks good...

Should probably eventually merge all these together (someday)


Simple Objects - Shape Rectangle: The error appears when drawing a rectangle of small height

Steps to Reproduce

Open Ketcher.
Click to 'Shape Rectangle' button in the tool palette.
Draw a rectangle of small height.

Expected behavior
There's no error in console.
Actual behavior
The error "Error:  attribute height: A negative value is not valid. ("-X.00000000")"  appears.
https://user-images.githubusercontent.com/90637536/134469094-86b93b2e-c1d5-4ad1-a864-0e9c96e9526f.mp4
Desktop:
OS: Windows
Browser: Chrome Version 93.0.4577.63, Microsoft Edge Version 93.0.961.47

Release/2.4 Closed as fixed.


Fix handling of unconventional variable names

If you have a variable named something["z"][()] the loo* functions will fail, because it tries to split, but will get the wrong part, changing this to rsplit and limiting the number of elemetns to 2 resolves this and will always take the rightmost [] which contains the index.

IIRC there are quite a few suboptimal implementations in ParetoSmooth (even a few more in the function alone that is touched by this PR), but I haven't really used the package a lot, so there hasn't been a strong incentive from my side to improve things. IMO it would be good to make the code more robust and better tested. Or maybe just join forces with https://github.com/arviz-devs/PSIS.jl which (AFAIK) has better code quality and is better maintained.

Regarding this PR more concretely: Can't we avoid parsing strings completely and just work with VarNames (as shown in https://github.com/TuringLang/DynamicPPL.jl/blob/3b8dceab76ef37d57d0abdecb205a5c46550679f/src/loglikelihoods.jl#L201-L206)? And do we actually have to sort, given that (nowadays) DynamicPPL.pointwise_loglikelihoods returns an OrderedDict?

Regarding this PR more concretely: Can't we avoid parsing strings completely and just work with VarNames (as shown in https://github.com/TuringLang/DynamicPPL.jl/blob/3b8dceab76ef37d57d0abdecb205a5c46550679f/src/loglikelihoods.jl#L201-L206)? And do we actually have to sort, given that (nowadays) DynamicPPL.pointwise_loglikelihoods returns an OrderedDict?

I'm happy to change to this if that's the better option. I just made the PR based on the minor change I made to get it working. I'm not entirely sure yet how much work it would be to convert it to using VarName as I'm not familiar with the interface.

@sethaxen, apart from the integration with PPLs, are there any other differences between ParetoSmooth and PSIS?

It's been a long time since I made a comparison, but besides the differences mentioned in https://github.com/TuringLang/ParetoSmooth.jl/pull/97#issuecomment-2296668010 (scope, package weight, API, code quality, test coverage), there are a few differences in functionality:

they use different ESS estimates. ParetoSmooth's is more pessimistic, while PSIS's uses the one from the paper, which is connected with MCSE like the usual ESS.
PSIS also includes a diagnostic plot recipe: https://julia.arviz.org/PSIS/stable/plotting/
PosteriorStats includes Pseudo-BMA, bootstrapped Pseudo-BMA, and stacking model weighting methods for model comparison (see https://julia.arviz.org/PosteriorStats/stable/api/#Model-comparison). Stacking is currently the recommended approach in the literature. IIRC ParetoSmooth only implements Pseudo-BMA.
PosteriorStats also includes loo_pit for posterior predictive checks: https://julia.arviz.org/PosteriorStats/stable/api/#PosteriorStats.loo_pit

Both packages are outdated in their use of the diagnostics. PSIS is shortly getting updates from the recent PSIS paper revision.

But is it clear whether the sorting is actually needed? https://github.com/TuringLang/ParetoSmooth.jl/pull/97#issuecomment-2285464006


Update PUDL to use pandas 2.2

Overview

Bump pandas version to 2.2
Bump ferc-xbrl-extractor to 1.3.2 which allows use of pandas 2.2
Switch to faster calamine Rust Excel parser
Fix some tests that relied on old incorrect merge sort ordering in pandas.
Fix a test where generator_id was getting cast sometimes as a float, and other times as a string, making it impossible to merge two dataframes together using that column as one of the merge keys.

# Failing Tests
- [x] `test/unit/helpers_test.py::test_annual_attribute_merge`
- [x] `test/unit/extract/excel_test.py::TestGenericExtractor::test_read_excel_calls`
- [x] `test/unit/analysis/plant_parts_eia_test.py::test_one_to_many`
- [x] `test/unit/transform/ferc1_test.py::test_calculate_values_from_components`
- [x] `test/unit/transform/glue_test.py::test_epacamd_eia_subplant_ids`

# Before Merging
- [x] Make sure all unit tests pass
- [ ] Fix failing `plant_parts_eia` asset
- [ ] Make sure all integration tests pass
- [ ] Run the full ETL
- [ ] Run integration tests against full DB
- [ ] Run data validation tests
- [ ] Update the [release notes](../docs/release_notes.rst): reference the PR and related issues.
- [ ] Review the PR yourself and call out any questions or issues you have

Okay that record_id definitely appears in the core table core_ferc1__yearly_steam_plants_sched402
steam = pd.read_sql("core_ferc1__yearly_steam_plants_sched402", pudl_engine)
{col: val[28340] for col, val in steam[steam.record_id == "f1_steam_2019_12_454_2_2"].to_dict().items()}

{'record_id': 'f1_steam_2019_12_454_2_2',
 'utility_id_ferc1': 362,
 'report_year': 2019,
 'plant_name_ferc1': 'roy s. nelson 6',
 'plant_type': 'steam',
 'construction_type': 'outdoor',
 'construction_year': 1982.0,
 'installation_year': 1982.0,
 'capacity_mw': 113.6,
 'peak_demand_mw': 222.0,
 'plant_hours_connected_while_generating': 5519.0,
 'plant_capability_mw': nan,
 'water_limited_capacity_mw': 550.0,
 'not_water_limited_capacity_mw': 550.0,
 'avg_num_employees': 53.0,
 'net_generation_mwh': 727960.234,
 'capex_land': 1717.0,
 'capex_structures': 49605194.0,
 'capex_equipment': 250931302.0,
 'capex_total': 300914091.0,
 'capex_per_mw': 2648891.6,
 'opex_operations': 1043624.0,
 'opex_fuel': 18828461.0,
 'opex_coolants': nan,
 'opex_steam': 1389184.0,
 'opex_steam_other': nan,
 'opex_transfer': nan,
 'opex_electric': 946619.0,
 'opex_misc_power': 823046.0,
 'opex_rents': 11314.0,
 'opex_allowances': 3679.0,
 'opex_engineering': 184156.0,
 'opex_structures': 443928.0,
 'opex_boiler': 3943577.0,
 'opex_plants': 1191624.0,
 'opex_misc_steam': 252868.0,
 'opex_production_total': 29062080.0,
 'opex_per_mwh': 39.9,
 'asset_retirement_cost': 375878.0}

But... there no longer appears to be an out_ferc1__yearly_steam_plants_sched402 table at all.  Is that intentional? I see there are a couple of other steam plants output tables with fuel_by_plant, and the fuel table itself. This seems odd.  If we're combining all the FERC1 plants into a single table as the only plant outputs, there will be a lot of nulls, including in the plant_id_ferc1 column that we assign, since those IDs only apply to the steam plants, which could be confusing.
The "missing" record_id does appear in the out_ferc1__yearly_all_plants table though.
steam = pd.read_sql("out_ferc1__yearly_all_plants", pudl_engine)
{col: val[30757] for col, val in steam[steam.record_id == "f1_steam_2019_12_454_2_2"].to_dict().items()}

{'report_year': 2019,
 'utility_id_ferc1': 362,
 'utility_id_pudl': 107.0,
 'utility_name_ferc1': 'Entergy Louisiana, LLC',
 'plant_id_pudl': 517.0,
 'plant_id_ferc1': 238.0,
 'plant_name_ferc1': 'roy s. nelson 6',
 'asset_retirement_cost': 375878.0,
 'avg_num_employees': 53.0,
 'capacity_factor': 0.731518339201878,
 'capacity_mw': 113.6,
 'capex_annual_addition': 5040091.0,
 'capex_annual_addition_rolling': 5557459.0,
 'capex_annual_per_kw': 44.366998239436626,
 'capex_annual_per_mw': 44366.99823943662,
 'capex_annual_per_mw_rolling': 48921.29401408451,
 'capex_annual_per_mwh': 6.923580114130245,
 'capex_annual_per_mwh_rolling': 7.6342892653117085,
 'capex_equipment': 250931302.0,
 'capex_land': 1717.0,
 'capex_per_mw': 2648891.6,
 'capex_structures': 49605194.0,
 'capex_total': 300914091.0,
 'capex_wo_retirement_total': 300538213.0,
 'construction_type': 'outdoor',
 'construction_year': 1982.0,
 'installation_year': 1982.0,
 'net_generation_mwh': 727960.234,
 'not_water_limited_capacity_mw': 550.0,
 'opex_allowances': 3679.0,
 'opex_boiler': 3943577.0,
 'opex_coolants': nan,
 'opex_electric': 946619.0,
 'opex_engineering': 184156.0,
 'opex_fuel': 18828461.0,
 'fuel_cost_per_mwh': 25.864683427199402,
 'opex_misc_power': 823046.0,
 'opex_misc_steam': 252868.0,
 'opex_nonfuel_per_mwh': 14.057936851534118,
 'opex_operations': 1043624.0,
 'opex_per_mwh': 39.9,
 'opex_plant': 1191624.0,
 'opex_production_total': 29062080.0,
 'opex_rents': 11314.0,
 'opex_steam': 1389184.0,
 'opex_steam_other': nan,
 'opex_structures': 443928.0,
 'opex_total_nonfuel': 10233619.0,
 'opex_transfer': nan,
 'peak_demand_mw': 222.0,
 'plant_capability_mw': nan,
 'plant_hours_connected_while_generating': 5519.0,
 'plant_type': 'steam',
 'record_id': 'f1_steam_2019_12_454_2_2',
 'water_limited_capacity_mw': 550.0,
 'fuel_cost_per_mmbtu': nan,
 'fuel_type': None,
 'license_id_ferc1': nan,
 'opex_maintenance': nan,
 'opex_total': nan,
 'capex_facilities': nan,
 'capex_roads': nan,
 'net_capacity_adverse_conditions_mw': nan,
 'net_capacity_favorable_conditions_mw': nan,
 'opex_dams': nan,
 'opex_generation_misc': nan,
 'opex_hydraulic': nan,
 'opex_misc_plant': nan,
 'opex_water_for_power': nan,
 'ferc_license_id': None,
 'capex_equipment_electric': nan,
 'capex_equipment_misc': nan,
 'capex_wheels_turbines_generators': nan,
 'energy_used_for_pumping_mwh': nan,
 'net_load_mwh': nan,
 'opex_production_before_pumping': nan,
 'opex_pumped_storage': nan,
 'opex_pumping': nan}

So it seems like it must be getting lost somewhere between that output table and the record linkage process.

Going another step into the record linkage process, we still seem to have this record available in the prepared FERC 1 plants:
from pudl.etl import defs
input_manager = pudl.analysis.record_linkage.eia_ferc1_record_linkage.InputManager(
    plants_all_ferc1=defs.load_asset_value(AssetKey("out_ferc1__yearly_all_plants")),
    fbp_ferc1=defs.load_asset_value(AssetKey("out_ferc1__yearly_steam_plants_fuel_by_plant_sched402")),
    plant_parts_eia=defs.load_asset_value(AssetKey("out_eia__yearly_plant_parts")),
)
prepared_plants_ferc1 = input_manager.get_plants_ferc1()
prepared_plants_ferc1.loc["f1_steam_2019_12_454_2_2"]

Doesn't this error also potentially mean that the associated EIA plant part record for FERC record f1_steam_2019_12_454_2_2 isn't in the plant parts list? In the training data, the associated EIA record is 1393_1_2019_plant_unit_total_11241, is that record in the plant parts list?
My other guess is that it's a weird index setting error. In recent times record_id_eia has gone back and forth from being the index of the plant parts list to just being a column, and that once caused an issue. But not sure why pandas 2.2 would cause it to change. I can take a look later today but I guess I'd probably need to recreate all the dependencies to this step which takes a while for me.

Okay yes you're right @katie-lamb it's the PPE record that's missing:
ppe = defs.load_asset_value(AssetKey("out_eia__yearly_plant_parts"))
ppe[ppe.record_id_eia == "1393_1_2019_plant_unit_total_11241"]
# yields an empty dataframe

Are there any validations that we do on the PPE? I can open it up but I don't know what I should be looking for in there for sanity checks.

a naive debugging thing i do sometimes when this kind of issue occurs is write a stupid little log function with a string name of where in the process it is being run and then i just add it in many places and re-run the cram
def stupid_log(df, where_am_i):
    logger.info(f"{where_am_i}: {df[df.record_id_ferc1 == "yer_record"]}")

oh no its a unit.... @zaneselvans  did you say that something about the merging does something different with the order...?? this could finally require us to fix this #1105

Look at PPE for the plant and year of the missing EIA record_id.

It seems like the error in the FERC-EIA record linkage model is because .map(str) maybe isn't behaving the same.
We create a column called plant_id_report_year_util_id as the blocking column in the FERC-EIA record linkage. To create this column, we do plant_id_report_year_util_id=lambda x: (x.plant_id_pudl.map(str) + "_" + x.report_year.map(str) + "_" + x.utility_id_pudl.map(str)). When there are null values in the plant_id_pudl and utility_id_pudl columns (which are there because of the bug fix I haven't merged into this branch yet), then .map(str) first maps the integer columns to floats, and then to strings. Is this expected? I also don't see any mention of this changing in the release notes, but it might be an indirect cause of some other change?
Thus, when we create plant_id_report_year_util_id, we get strings like "16_20_43.0", and the subsequent merge on this blocking column isn't effective.
I temporarily fixed this by changing the .map(str) calls to astype(str). Maybe this isn't robust enough? I'm tempted to say this isn't a big issue because the Splink model doesn't use this plant_id_report_year_util_id blocking column, and it seems like we'll replace the existing model with the Splink model very soon.

Great! I think that astype() is the more appropriate (and probably more performant) option. I found one other instance where we were doing map(str) and switched it over too.
If you have or could easily create a minimal example of the change in the behavior of map(str) it would be good to create an issue in the pandas repo.  I searched and didn't see any open or closed issues that seemed to be referring to this.
I notice that we're defining plant_id_report_year_util_id in two separate places, with a not entirely identical process. Is that expected / desired?
There's pudl.analysis.plant_parts_eia.py
def plant_parts_eia_distinct(plant_parts_eia: pd.DataFrame) -> pd.DataFrame:
    plant_parts_eia = plant_parts_eia.assign(
        plant_id_report_year_util_id=lambda x: x.plant_id_report_year
        + "_"
        + x.utility_id_pudl.astype(str)
    ).astype({"installation_year": "float"})
    ...

and also pudl.analysis.record_linkage.eia_ferc1_record_linkage.InputManager.get_ferc1_plants()
...
                .assign(
                    installation_year=lambda x: (
                        x.installation_year.astype("float")
                    ),  # need for comparison vectors
                    plant_id_report_year=lambda x: (
                        x.plant_id_pudl.astype(str) + "_" + x.report_year.astype(str)
                    ),
                    plant_id_report_year_util_id=lambda x: (
                        x.plant_id_report_year + "_" + x.utility_id_pudl.astype(str)
                    ),
                    fuel_cost_per_mmbtu=lambda x: (x.fuel_cost / x.fuel_mmbtu),
                    unit_heat_rate_mmbtu_per_mwh=lambda x: (
                        x.fuel_mmbtu / x.net_generation_mwh
                    ),
                )
...

Is that expected? Are these just the left & right dataframes getting their respective columns defined for the linkage?

@bendnorman that's interesting about the pd.read_excel() call that's not using calamine still being there and... still working. I wonder if it's using calamine automatically because the default openpyxl engine isn't installed?  That would be nice!

Oh pudl.helpers.convert_df_to_excel_file creates a pd.ExcelFile() object that specifies the engine. https://github.com/catalyst-cooperative/pudl/blob/72ad0ecc671ded6e560b4764726f12e95f09e18b/src/pudl/helpers.py#L1597-L1611
I wonder if pandas have plans for pd.ExcelWriter to support calamine.


Use --no-fail when --dockerfile=warn

This fixes a bug where --dockerfile=warn still exits with error code 1, by using the correct flag.

Tests are failing due to a bug:
out/linters/golangci-lint-v1.42.0-x86_64 run
19
out/linters/hadolint-v2.7.0-x86_64./test/test.Dockerfile
20
make: out/linters/hadolint-v2.7.0-x86_64./test/test.Dockerfile: Command not found
21
make: *** [Makefile:23: lint] Error 127
22
Error: Process completed with exit code 2.

Moving PR to my fork


Potential copyright issue with idleness.txt

Hi,
I am developer from Apache Dubbo(incubating) project, one of our project has a transitive dev dependency to this project:
<EMAIL_ADDRESS>./dubbo-admin-ui
├─┬<EMAIL_ADDRESS>│ └─┬<EMAIL_ADDRESS>│   └──<EMAIL_ADDRESS>└─┬<EMAIL_ADDRESS>  └─┬<EMAIL_ADDRESS>    └─┬<EMAIL_ADDRESS>      └─┬<EMAIL_ADDRESS>        └─┬<EMAIL_ADDRESS>          └──<EMAIL_ADDRESS>
When reviewing our release candidate, we found that the test/idleness.txt contains text with copyright[1]:
"In Praise of Idleness and Other Essays (1932) – Copyrighted in the
United States until 2027 due to Renewal R320904"
which might against ASF copyright policy. While I am confirming with ASF legal team, I am wondering that if you could replace this text with something else?
Thanks in advance!
[1]  https://en.wikisource.org/wiki/Author:Bertrand_Arthur_William_Russell

I noticed that the file in question is in a tests folder. Can this be fixed by just adding a files or ignore field in package.json so that only the files necessary for the library are exported in the npm package?

This would likely be fixed by #22.


[Bug] Incorrect mongodb driver link for Go in adapter documentation

Describe the bug
It's a documentation issue. In the supported adapters table, under Go's description section for mongoDB:

For MongoDB based on MongoDB driver for Go

The driver link points to the old and unofficial mongo driver whereas the adapter uses the latest and official mongo driver
To Reproduce
Steps to reproduce the behavior:

Go to adapters
Click on Go under supported adapters.
Scroll down to MongoDB Adapter
See error in the description section

Expected behavior
The driver link of the description should point to the correct link of the mongo driver used by the adapter.

Fixed in: https://github.com/casbin/casbin-website/commit/17f980954b7828ab6b198ef9cd4bb148a36a40a2


Stdout/stderr should be surfaced by default

I've noticed while training recently that the trainer prints essentially nothing to stdout once training is underway. Even print statements in the trainer code no longer get printed to the console. This is a serious UX regression that must be addressed before v0.6.0.

Fixed in #893 and #917


Python bh jet integration

Hi Matteo,
These changes allow BHJet to be called from python using a ctypes C interface without changing core BHJet code
MakeBHJet now builds an addition library "pyjetmain"  which is a simple pass through to jetmain method in BHJet
DemoPyJetMain.py calls pyjetmain with parameters and plots the spectral output
BHwrap.x works as previously and outputs test files
These changes are needed for BHJet integration with model fitting in gammapy and the MWL/MM analysis prototype
best
Sheridan

This is fantastic, thank you so much!

Good stuff !


chore: fix migration notes

Docs Preview
Hey there! 👋 You can check your preview at https://664c6ce5d68552007f1601e9--aztec-docs-dev.netlify.app

Benchmark results
Metrics with a significant change:

app_circuit_proving_time_in_ms (Test:emit_nullifier): 1,691 (-37%)
app_circuit_proving_time_in_ms (FPC:fee_entrypoint_public): 9,146 (-45%)
protocol_circuit_witness_generation_time_in_ms (base-parity): 241 (-73%)
protocol_circuit_witness_generation_time_in_ms (public-kernel-setup): 413 (+18%)
protocol_circuit_witness_generation_time_in_ms (public-kernel-teardown): 510 (+51%)

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.
L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
8 txs
32 txs
64 txs

l1_rollup_calldata_size_in_bytes
772
772
772

l1_rollup_calldata_gas
6,832
6,844
6,832

l1_rollup_execution_gas
587,372
587,384
587,372

l2_block_processing_time_in_ms
1,348 (-2%)
5,052 (-1%)
10,059 (-1%)

l2_block_building_time_in_ms
33,594
131,927
263,049

l2_block_rollup_simulation_time_in_ms
33,413
131,288
261,798

l2_block_public_tx_process_time_in_ms
15,678 (+1%)
67,067 (+1%)
134,910

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

Metric
5 blocks
10 blocks

node_history_sync_time_in_ms
15,164 (-1%)
28,423 (-1%)

node_database_size_in_bytes
21,401,680
38,056,016 (-1%)

pxe_database_size_in_bytes
29,868
59,425

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

Metric
1 leaves
16 leaves
64 leaves
128 leaves
512 leaves
1024 leaves
2048 leaves
4096 leaves
32 leaves

batch_insert_into_append_only_tree_16_depth_ms
11.3
18.3
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_count
16.7
31.8
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_ms
0.658
0.562
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_32_depth_ms
N/A
N/A
52.3
81.8 (+1%)
262 (-1%)
505
985 (-1%)
1,953 (-1%)
N/A

batch_insert_into_append_only_tree_32_depth_hash_count
N/A
N/A
95.9
159
543
1,055
2,079
4,127
N/A

batch_insert_into_append_only_tree_32_depth_hash_ms
N/A
N/A
0.535
0.505 (+1%)
0.477
0.471 (-1%)
0.467
0.466 (-1%)
N/A

batch_insert_into_indexed_tree_20_depth_ms
N/A
N/A
62.4
119 (-1%)
376 (-1%)
741 (-1%)
1,467
2,923 (-1%)
N/A

batch_insert_into_indexed_tree_20_depth_hash_count
N/A
N/A
106
208
692
1,363
2,707
5,395
N/A

batch_insert_into_indexed_tree_20_depth_hash_ms
N/A
N/A
0.543
0.535
0.511 (-1%)
0.510
0.508
0.508
N/A

batch_insert_into_indexed_tree_40_depth_ms
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
67.6

batch_insert_into_indexed_tree_40_depth_hash_count
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
108

batch_insert_into_indexed_tree_40_depth_hash_ms
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
0.592

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
84,743
673,221

Transaction size based on fee payment method
| Metric |  |
| - |  |
Transaction processing duration by data writes.

Metric
0 new note hashes
1 new note hashes
2 new note hashes

tx_pxe_processing_time_ms
28,772
3,111
98,218

Metric
0 public data writes
1 public data writes
2 public data writes
4 public data writes
8 public data writes

tx_sequencer_processing_time_ms
1,346
2,142
1,868
2,754
1,767


k8up backup doesn't support backing up openebs storage class PVs with RWO access

Description
Hello Team,
I was trying to use k8up to backup RWO PVs for openebs, iscsi and nfs storage classes each. It worked well for iscsi and nfs backed PVs but the backup pod failed with below error while mounting openebs backed PV for backup:
MountVolume.SetUp failed for volume "pvc-e67547c6-fd91-445b-8223-98694b15655e" : rpc error: code = Internal desc = verifyMount: device already mounted at [/var/lib/kubelet/pods/d0e858c5-8bce-4d1d-8d8e-07cf4abf8334/volumes/kubernetes.io~csi/pvc-e67547c6-fd91-445b-8223-98694b15655e/mount /var/lib/kubelet/pods/d0e858c5-8bce-4d1d-8d8e-07cf4abf8334/volume-subpaths/pvc-e67547c6-fd91-445b-8223-98694b15655e/yelb-db/0]
Kindly let us know, if k8up supports backing up openebs backed PVs. If yes, then please share the settings.
Thanks & Regards,
Sunil Kumar
Additional Context
No response
Logs
No response
Expected Behavior
k8up should support backing up openebs backed PVs.
Steps To Reproduce
No response
Version of K8up
2.7.1
Version of Kubernetes
v1.24
Distribution of Kubernetes
Rancher

Hi @vistasunil
In theory it should not make a difference for K8up what provider is behind the RWO PVCs. K8up tries to place the backup pods in such a way that, if they are mounted, it will run on the same node, thus allowing access to the data.
Openebs also seems to be using ISCSI under the hood, so it should behave the same as your other RWO PVCs. However we have not explicitly tested it with openebs. So there might be a chance of some implementation specific issue at hand.
One thing I could think of that lead to this issue: Is it possible that the pod mounting the openebs PVC has moved to another k8s node while the backup was running? If that happens, then the scheduled backup pod can't access the PVC anymore.

@Kidswiss I was also expecting same but backup did not work for openebs PVCs. Also, there was no pod movement during the backup run. So last statement do not stands valid case here. Can you please test the same at your end and let us know what is missing here?

@Kidswiss  Any update on this?


Service 'mitmproxy' failed to build : Build failed

I started looking into Phishpond today and had an issue building the image in Docker for Windows.
The error was:
=> => extracting sha256:00eb45722879c184549e05dc998f20acdb8d1c628ffbbe1dfc68fab3c67002e5                                   0.1s
=> => extracting sha256:97bfbf48a93798d26ebd24e2ce38363db807ec1a1afb8fc852892c6716e60d1f                                   1.8s
=> => extracting sha256:5b8302d2fac67ebd00245c192f10f439ab8079104b0514f133f6e3352204a9c0                                   0.0s
=> [internal] load build context                                                                                0.1s
=> => transferring context: 1.98kB                                                                                0.0s
=> [2/5] RUN apk add bash vim                                                                                5.1s
=> [3/5] RUN mkdir /home/mitmproxy/logs                                                                                0.5s
=> ERROR [4/5] RUN chmod -R 777 /home/mitmproxy/.mitmproxy /home/mitmproxy/logs                                            0.4s
------
> [4/5] RUN chmod -R 777 /home/mitmproxy/.mitmproxy /home/mitmproxy/logs:
#7 0.376 chmod: /home/mitmproxy/.mitmproxy: No such file or directory
------
executor failed running [/bin/sh -c chmod -R 777 /home/mitmproxy/.mitmproxy /home/mitmproxy/logs]: exit code: 1
ERROR: Service 'mitmproxy' failed to build : Build failed

I was able to get the build working by adding the following line to phishpond/docker/mitmproxy/Dockerfile before line 5:
RUN mkdir /home/mitmproxy/.mitmproxy

Is there a reason this line isn't included? If not, I can submit a PR to fix this issue.

The statement "RUN mkdir /home/mitmproxy/.mitmproxy" will cause problem in Linux

Thanks to the extra RUN command I got this running. Was about to give up [female dog]in' that this does not work.


Integration image is broken

Before today it was working fine.

Version 3.0.1 pushed to rubygems. Thx!

Thanks for fixing it so quickly 😄


Requests and observables are not unsubscribed when the view is gone

Network requests and observables are loaded with onLoad() method when the CurrenciesActivity view is created but when the views are paused or destroyed, the requests and subscriptions are not canceled/un-subscribed because method CurrencyListViewModel.dispose() is not being called.

Good catch. Thanks for review :)
Fix applied.

No problem.
I'm super curious because another side effect of this with MVP would be that you'll try change a view that is not visible and Android will throw a fatal exception. Your example is using DataBinding, so I'm curious if you notifyPropertyChanged while the view is gone and if the DataBinding is already protected to avoid calling the view in this case.

In this case, it would cause small short time memory leak. View Model would stay in memory little longer than it needs to be - until the background operation is finished.
It's not a problem for Activity. It will be garbage collected anyway, because ViewModel does not hold any reference to the View. That is another advantage of MVVM over MVP.
No crash is expected because ViewModel knows nothing about observers.  And when observer is gone, it won't be notified of the change.


Use action vars for go versions

Switches go version to use workflow variables. Makes it easier to update the go version in workflows.

Should work based on https://github.com/orgs/community/discussions/46785 but somethings borked. Closing for now. Will revisit later.


Descriptions for SkipDomainValidationForXXX missing

For parameters
-SkipDomainValidationForMailContact
-SkipDomainValidationForMailUser
-SkipDomainValidationForSharedMailbox
The description is just:
{{ Fill SkipDomainValidationForMailContact Description }}
{{ Fill SkipDomainValidationForMailUser Description }}
{{ Fill SkipDomainValidationForSharedMailbox Description }}

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

ID: a0de94ad-58e2-3ba9-e854-8acd7591fb2c
Version Independent ID: 691f3292-81aa-6367-8b67-bf37aef2deb5
Content: Remove-RecipientPermission (ExchangePowerShell)
Content Source: exchange/exchange-ps/exchange/Remove-RecipientPermission.md
Service: exchange-powershell
GitHub Login: @chrisda
Microsoft Alias: chrisda

@New-Authentication, empty parameter descriptions are a known commodity. Availability vs. functionality are two different things. You can read more about it here.
Are you being directed to use these parameters in other content?

@chrisda Thank you for your reply. I'm currently troubleshooting a problem with the Remove-RecipientPermission and these three parameters could possible do what I need, but I really don't know what they're doing. I have not been directed to use them - their names just look interesting.

@chrisda Thank you for that. And in the mean time I solved my issue (user error). Closing.


⚠ Bofrost.be has degraded performance

In 828492c, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1296 ms

Resolved: Bofrost.be performance has improved in 37ed520 after 6 minutes.


Problem replicating the matching chip-color-test.py's libswscale test (ie. the one where the output matches the original)

I have cloned the repository and run chip-color-test.py.  I do not have an FFmpeg with VFR, as I only cared about matching the results visually.  I am using FFmpeg 6.0.  I am comparing the images with my viewer (mrv2 and with iv) and they look the same:
mrv2 chip-chart-yuvconvert/chip-chart-1080-noicc-spline444out_color_matrix.png ../sourceimages/chip-chart-1080-noicc.png
However, if I compare the actual movie file to the original -noicc.png, I don't get a match:
mrv2 chip-chart-yuvconvert/spline444out_color_matrixfull.mp4 ../sourceimages/chip-chart-1080-noicc.png
or:
iv chip-chart-yuvconvert/spline444out_color_matrixfull.mp4 ../sourceimages/chip-chart-1080-noicc.png
I am trying to understand why.
### Tasks

Sorry I missed this issue, I'll take a look.

I know this ticket is rather old, did any of the changes we saw a month ago make a difference here?

Quite frankly, I forgot about this issue.  It is probably not relevant now.  I will be closing it.

Yep.  I just checked with mrv2 and it matches perfectly now.  Thanks for your help!


Version upgrade

Can you please upgrade the version to v0.9.24

@PankajChaudhary5 we use nimbus which is not updated onto v0.9.24. we are keeping diora in line with the latest release of nimbus.


Integration testing

Please try out the documentation: https://docs.human-connection.org/nitro/workflow/testing/integration-testing
Todo`s

[x] setup basic cypress authentication testing
[ ] get feedback
[ ] write more tests
[ ] how to integrate into our continious delivery pipeline?

@appinteractive yes, but it's not only about local testing where the developer is fully in control of putting the correct repositories together. It's about our build server. Consider you have a change in the backend repository. How does the build server know how to check out the backend repo at the right commit?


Can't create instance after upgrade from 4.2.1 to 4.11.2

ISSUE TYPE

Bug Report

CLOUDSTACK VERSION
CS version 4.11.2 (both manager and agents)
CONFIGURATION
basic networking
OS / ENVIRONMENT
OS Ubuntu 16.04
Java 8
SUMMARY

After successful upgrade from 4.2.1 to 4.11.2, can't create new instances.
Error: "Error while starting VM"
agent.log
manager.log

Can you describe your env, hypervisors, networking etc? Did you seed the 4.11.2 systemvmtemplate? Did you perform in place mgmt server upgrade or used a new mgmt server host?

Because of 4.11.2 requirements, I had to upgrade OS of the mgmt and agents. I did clean Ubuntu 16.04 installation on mgmt, restored database and /etc/cloudstack directory and installed 4.11.2 packages after that. On agents, OS upgrade with clean Ubuntu 16.04 installation, CS agent 4.11.2 packages and restored agent.properties! Everything is working (start/stop instance, migrate instance between hosts) except create instance!
Environment is: basic networking, 1 mgmt Ubuntu 16.04 with CS version 4.11.2 with 1 cluster KVM - 4 hosts with Ubuntu 16.04 and CS agent 4.11.2 java 8. Virtual router and SSVM and CPVM have been updated with  4.11.2 systemvmtemplate.

From your logs, seems there's a conflict in your host and cluster setup
2019-07-12 14:55:30,721 WARN  [o.a.c.alerts] (API-Job-Executor-5:ctx-c22c483d job-4033 ctx-d82ec66b) (logid:c73f95c5) AlertType:: 8 | dataCenterId:: 1 | podId:: 1 | clusterId:: null | message:: Failed to deploy Vm with Id: 3206, on Host with Id: null
The reason why start/stop instance, migrate instance between hosts seems to work is because they can find a cluster and host to migrate to as you can see
2019-07-12 14:53:25,926 DEBUG [c.c.v.VmWorkJobHandlerProxy] (Work-Job-Executor-8:ctx-e53ceb96 job-4033/job-4034 ctx-4b496990) (logid:c73f95c5) Done executing VM work job: com.cloud.vm.VmWorkStart{"dcId":1,"podId":1,"clusterId":1,"hostId":37,"rawParams":{"VmPassword":"rO0ABXQADnNhdmVkX3Bhc3N3b3Jk"},"userId":2,"accountId":2,"vmId":3206,"handlerName":"VirtualMachineManagerImpl"}
Maybe try and check if your service offerings or maybe storage offering could be limited to a certain zone/cluster or host

@velinbudinov can you post the result of mysql queries below ?
select broadcast_uri from cloud.networks where id=<network_id>;
select broadcast_uri from cloud.nics where instance_id=<instance id>;

@velinbudinov can you post the result of mysql queries below ?
select broadcast_uri from cloud.networks where id=<network_id>;
select broadcast_uri from cloud.nics where instance_id=<instance id>;

both returns NULL

@velinbudinov can you post the result of mysql queries below ?
select broadcast_uri from cloud.networks where id=<network_id>;
select broadcast_uri from cloud.nics where instance_id=<instance id>;

@ustcweizhou both returns NULL

@velinbudinov can you change broadcast_uri of network to "vlan://untagged" and retry ?

@velinbudinov can you post the result of mysql queries below ?
select broadcast_uri from cloud.networks where id=<network_id>;
select broadcast_uri from cloud.nics where instance_id=<instance id>;

@ustcweizhou both returns NULL

@velinbudinov can you change broadcast_uri of network to "vlan://untagged" and retry ?

@ustcweizhou thanks a lot! That fixed it! :)


[DOC]: Set up prettier and pre-commit hook to prettify codebase

Implementation:

Setup husky: https://typicode.github.io/husky/get-started.html
Install prettier as dev dependency
Add .prettierrc file with the below configurations.{
	"semi": false,
	"tabWidth": 4,
    "printWidth": 100,
    "singleQuote": true,
    "trailingComma": "none",
    "bracketSameLine": false,
}

Add dist and node_modules to .prettierignore file
Run prettier on all files.

@sstephanyy: This might be an even better first issue. Let me know if you want to claim this one instead. I have also updated the contributing[https://github.com/cdrani/chorus/blob/develop/contributing.md) file with steps for local development.

@cdrani thanks for the contribution guide, will be very helpful. Yes, I wanna claim this one :)

@cdrani  sorry to bother again, but I'm facing this error again and again everytime I git commit. Do you have an ideia of what am I doing wrong?

Solved in #212.


Resolution supported

Which kind of resolution are supported?
##For now, i know that full hd and ultra hd on pc are avaiable only on windows through edge.
with this plugin, can i see (at least) 1080p content on linux?

There´s no absolute answer to this question, as this is always hardware dependent when using Widevine to decrypt & decode the video, as it can be done through software decoding only.
Rule of thumb, if your CPU is powerful enough & enough RAM is available, 1080 is possible, regardless of the OS used.
What do I mean with powerful enough? At the moment an Odroid C2 (1.5 GHz Quad Core with 2gb RAM) is not able to do more than 720 reliably. But this is subject to change, as the inputstream.adaptive dependency (which basically is the bridge between Widevine & Kodi) is still work in progress. It will eventually be possible to have 1080 on the RPi 3 sometime in the future, but this is based on educated guesses, not on hard facts.
Regarding 4k support: You probably need to be a wizard to know if it will someday work or not. As Widevine does not (seem) to support it at the moment. But Netflix announced recently that they´re working with browser vendors (foremost Google, when it comes down to Widevine support) to get 4k working in the Browser. How this will be done, no Idea. I´d guess you need some kind of hardware acceleration (e.g. hardware decoding) to make it work...
TL:DR; If you´ve got enough CPU power & RAM, 1080 work, 4k doesn't & there´s hope to make 1080 work on "low power" machines like the RPi 3.

there is a way to check live streaming resolution?

@napgab You can press the "o" key on your keyboard in Kodi when a video is running, that´ll give you all the information inlcuding the resolution.
Also, you can set the max & min resolution in the inputstream.adaptive settings

The problem is most of the movies are streamed at 480p with a bitrate under 2 Mbps. With the Shield app, I can see the same movie at 1080p with a 7 Mbps bitrate. Would it be possible to "simulate" an app instead of a web browser?

See here for 1080p support on linux https://github.com/truedread/netflix-1080p

With the chrome plugin I see that I need to use super netflix to enforce a better bitrate because a movie starts with 1080p and in the middle starts caching 720p and goes to 720p. But yes, some movies don't have 1080p just 720p or less like "Friends" I think it is just the way Netflix ripped them. Thanks for the great plugin @asciidisco! Chrome can't do 5.1 sound but Kodi can!

There was a bug in inpustream.adaptive where if you selected 720p, you would get a lower quality stream. And if you selected 1080p you would get the 720p stream. It is fixed in peak3d/inputstream.adaptive#259


merge package template from arlac77/npm-package-template

package.json

chore(devDependencies): update<EMAIL_ADDRESS>from template

.travis.yml

chore(travis): merge from template .travis.yml

Coverage remained the same at 85.014% when pulling 29646cbbf556bec1dbf3d9a23514d65dc8f81c22 on template-sync-1 into 5a773083ea69c789c7b418d1b8b1db07dbd1f822 on master.

Coverage remained the same at 85.014% when pulling 29646cbbf556bec1dbf3d9a23514d65dc8f81c22 on template-sync-1 into 5a773083ea69c789c7b418d1b8b1db07dbd1f822 on master.

Codecov Report

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

@@           Coverage Diff           @@
##           master      #60   +/-   ##
=======================================
  Coverage   84.21%   84.21%
=======================================
  Files          19       19
  Lines         304      304
=======================================
  Hits          256      256
  Misses         48       48

Continue to review full report at Codecov.

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


Any plans to get this upstream to 'necolas/normalize.css' ?

Hello,
Great work, Do you have any plans (or discussions) to get some (or all) of your changes as part of the upstream 'normalize' project ? It makes sense doesn't it ?
Thanks for the nice defaults.

See https://github.com/necolas/normalize.css/issues/343

Thanks for your interest in the project, and @briankoser for providing the answer.
I carefully developed this project to make merging it into Normalize.css possible—the testing framework, code, and comment style all referenced the original project. But I was pretty sure it wouldn’t get merged, which is why I didn’t do it as just a pull request from the beginning.
Necolas kindly gave it a try, but wasn’t interested in merging it, which is a totally fair response—it does go beyond the scope of browser defaults and well into typographic conventions.
I will hopefully be doing another pass on the whole thing in the near future, with some tests I can automate. Any changes at this point are likely to move further away from having it merged, but I think that’s okay.
Thanks for opening an issue!


WIP: Try to install MariaDB without uninstalling MySQL first.

Closing this one because we went the route to extract the MySQL packages instead: https://github.com/youtube/vitess/pull/890


The codeflare-operator e2e tests are currently failing

WHAT
The codeflare-operator e2e tests are currently failing.

@VanillaSpoon Can this issue be closed? Or is there something else to address?


infinite world (no map limits)

The worldedge mod teleports the player to the other side of the world when they reach the edge.
When the player walks past 30000 in any direction they will be teleported to the other side of the world as if they walked around the world. (when you teleport to the other side you will be at ground height)
So, no worldmap limits can be reached ! :)
mod Link : https://forum.minetest.net/viewtopic.php?f=9&t=10753
(PS : We have the time for this mod, see below the actual worldmap)

(PS : We have the time)

Just enough time for me to add it, tomorrow. This mod seams great, I would like to see it. Does it work with every axis (x, y, and z)?

https://github.com/Ombridride/minetest-minetestforfun-server/commit/4d30bdcd61796d579d0fa5c031b33ca6eac3467e


Support "Legacy" (Idiomatic) version files

This would be really useful for those who prefer the idiomatic version files
https://github.com/jdx/rtx#legacy-version-files

do they not work? they should if you have them in your repo


Disable broken sticky-sidebar in Firefox

Fixes https://github.com/refined-github/refined-github/issues/6834
Fixes https://github.com/refined-github/refined-github/issues/6408

Due to lack of has selector support

Hotfixed in 23.8.24


Add ~Common~ only when partition name not provided

This should fix https://github.com/f5devcentral/terraform-provider-bigip/issues/82
@scshitole

Another approach would be to instruct the users to always use the fullPath when describing resource names, i.e. /<partition_name>/<resource_name> and remove this check all together.
Reason I mention this is because from what I see other resources are expected to provide the fullPath or the object is failed to be retrieved, so this creates an exception.

Ping @scshitole 😄

@scshitole will you consider this PR?


I have a  question ， Can you give me a simple explanation about the difference between rye and uv? I think that   these two tools  are  so  similar

I have a  question ， Can you give me a simple explanation about the difference between rye and uv? I think that   these two tools  are  so  similar  ,  thank  you  very  gratefully  !

See https://github.com/astral-sh/rye/discussions/1342

👍 I recommend checking out https://github.com/astral-sh/rye/discussions/1342.


[Feature] YouTube add to playlist

Would be great if you could implement that the video exported to youtube would get automatically added to a specified playlist.

Auto exporter with extra settings for every streamer so you can set a specific playlist like the Season theme you already did with naming would be great.

creating playlists might be a bit overkill, i'll make it so you have to create your own and input them in the config for now

creating playlists might be a bit overkill, i'll make it so you have to create your own and input them in the config for now

You write as tooltip that Use this format: channelname=ABC123;secondchannel=DEF456
example thiseguy=playlist ID or playlist name?

the playlist id yes, i'll make the id visible in the export dialog too

does it add the video to the playlist? i have no idea

does it add the video to the playlist? i have no idea

just tried to upload a video to a playlist, the video appears in the youtube manager but the job is kinda funky in the gui:

[11/Oct/2022:09:29:38 +0000] "GET /api/v0/youtube/playlists HTTP/1.1" 200 6466 "-" "Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:105.0) Gecko/20100101 Firefox/105.0"
Create exporter youtube vod {
title_template: '[{login}] {title} ({date})',
directory: '',
host: '',
username: '',
password: '',
description: 'VOD',
tags: 'Gaming VOD',
category: '20',
file_source: 'segment',
privacy: 'private',
vod: 'thiseguy (2022) - s202210e04',
remote: '',
playlist_id: 'PLqBFPPTpZzdGtIZzC2wVwrligyBBfC24l'
}
2022-10-11 11:29:45.372 | YouTubeExporter  Uploading /usr/local/share/twitchautomator/data/storage/vods/thiseguy/Season 02/thiseguy (2022) - s202210e04.mp4 to YouTube...
2022-10-11 11:29:45.374 | job  Save job YouTubeExporter_thiseguy (2022) - s202210e04.mp4 with PID undefined to /usr/local/share/twitchautomator/data/cache/pids/YouTubeExporter_thiseguy (2022) - s202210e04.mp4.json
Broadcasting data to 1 clients: {"server":true,"action":"job_save","data":{"job_name":"YouTubeEx...

does it add the video to the playlist? i have no idea

just tried to upload a video to a playlist, the video appears in the youtube manager but the job is kinda funky in the gui:

[11/Oct/2022:09:29:38 +0000] "GET /api/v0/youtube/playlists HTTP/1.1" 200 6466 "-" "Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:105.0) Gecko/20100101 Firefox/105.0" Create exporter youtube vod { title_template: '[{login}] {title} ({date})', directory: '', host: '', username: '', password: '', description: 'VOD', tags: 'Gaming VOD', category: '20', file_source: 'segment', privacy: 'private', vod: 'thiseguy (2022) - s202210e04', remote: '', playlist_id: 'PLqBFPPTpZzdGtIZzC2wVwrligyBBfC24l' } 2022-10-11 11:29:45.372 | YouTubeExporter Uploading /usr/local/share/twitchautomator/data/storage/vods/thiseguy/Season 02/thiseguy (2022) - s202210e04.mp4 to YouTube... 2022-10-11 11:29:45.374 | job Save job YouTubeExporter_thiseguy (2022) - s202210e04.mp4 with PID undefined to /usr/local/share/twitchautomator/data/cache/pids/YouTubeExporter_thiseguy (2022) - s202210e04.mp4.json Broadcasting data to 1 clients: {"server":true,"action":"job_save","data":{"job_name":"YouTubeEx...

btw not added to playlist so far

hmm ok, no playlist add errors?

Nope none it's just not added if I edit the video file on YouTube I see that there is no playlist selected.

i don't know if i accidentally fixed it but it seems to be working on my end now at least


Native events with EventTarget

Have you considered just implementing native events with EventTarget? It's in Node ever since v14, and once is an option on addEventListener. TypeScript natively supports it as well.
https://developer.mozilla.org/en-US/docs/Web/API/EventTarget
Pulled from discussion on Reddit from jessepence

No planning on using native events for this for now.


resolve_unc_paths="True" causes System.IO.IOException

When attempting to ship a local file via a UNC path using the new resolve_unc_paths="True"  ( #104  ) feature, as configured below:
<LogSearchShipper data_folder="C:\Apps\LogSearchShipper-DEV\data" resolve_unc_paths="True"

			...
				<watch files="\\LON-WS01186\Logs\LogSearchShipper-DEV\EDB_expected_event_sources.log" type="json">
					...

LogSearchShipper fails to start; throwing the following "System.IO.IOException","Message":"The process cannot access the file 'C:\\Logs\\LogSearchShipper-DEV\\EDB_expected_event_sources.log' because it is being used by another process. error:
{"@timestamp":"2015-06-03T14:29:53.109Z","ClassName":"System.IO.IOException","Message":"The process cannot access the file 'C:\\Logs\\LogSearchShipper-DEV\\EDB_expected_event_sources.log' because it is being used by another process.","Data":null,"InnerException":null,"HelpURL":null,"StackTraceString":"   at System.IO.__Error.WinIOError(Int32 errorCode, String maybeFullPath)\r\n   at System.IO.FileStream.Init(String path, FileMode mode, FileAccess access, Int32 rights, Boolean useRights, FileShare share, Int32 bufferSize, FileOptions options, SECURITY_ATTRIBUTES secAttrs, String msgPath, Boolean bFromProxy, Boolean useLongPath, Boolean checkHost)\r\n   at System.IO.FileStream..ctor(String path, FileMode mode, FileAccess access, FileShare share)\r\n   at LogSearchShipper.Core.NxLog.NxLogProcessManager.HasReadAccess(String folderPath)\r\n   at LogSearchShipper.Core.NxLog.NxLogProcessManager.PrepareFilePaths(String val)\r\n   at LogSearchShipper.Core.NxLog.NxLogProcessManager.GenerateNormalFileWatchConfig(FileWatchElement inputFile, Int32 i)\r\n   at LogSearchShipper.Core.NxLog.NxLogProcessManager.GenerateInputFilesConfig()\r\n   at LogSearchShipper.Core.NxLog.NxLogProcessManager.SetupConfigFile()\r\n   at LogSearchShipper.Core.NxLog.NxLogProcessManager.Start()\r\n   at LogSearchShipper.Core.LogSearchShipperProcessManager.Start()\r\n   at LogSearchShipper.LogSearchShipperService.Start(HostControl hostControl)\r\n   at Topshelf.Builders.ControlServiceBuilder`1.ControlServiceHandle.Start(HostControl hostControl)\r\n   at Topshelf.Runtime.Windows.WindowsServiceHost.OnStart(String[] args)","RemoteStackTraceString":null,"RemoteStackIndex":0,"ExceptionMethod":"8\nWinIOError\nmscorlib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089\nSystem.IO.__Error\nVoid WinIOError(Int32, System.String)","HResult":-2147024864,"Source":"mscorlib","WatsonBuckets":null,"logger":"Topshelf.Runtime.Windows.WindowsServiceHost","level":"FATAL"}

@mrdavidlaing, isn't there an extra instance of LSS or nxlog process running?

Yes, but none that is attempting to ship this file

@mrdavidlaing, unfortunately I can't reproduce this, but I've fixed one possible case. Could you try this? https://github.com/cityindex/LogSearchShipper/pull/111


Fix dialyzer errors, add all apps to dialyzer analysis

Given that we get rid of 1.10 in another PR I will just ignore that failure.


⚠ GitLab Server has degraded performance

In c0f1c5c, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 500
Response time: 9381 ms

Resolved: GitLab Server performance has improved in af018d7.


cake build 0.26.1 support

Updates the solution to support multi targeted output, including .net standard 2.0 required for cake build.
Hopefully fixes problems reported at https://github.com/cake-contrib/Home/blob/master/Audit_for_Cake_0.26.0.md
Unsure how the cake.build ILRepack works.
No unit tests added, but https://github.com/cake-contrib/Cake.FileHelpers/tree/master/Cake.FileHelpers.Tests includes an interesting pattern that might be followed to ensure everything behaves.
Closes #58.

@sergeyzwezdin Just curious if you had a chance to review this change? There are a few downstream projects using cake build and your addin, so it would be nice to bump the edition of cake.build that's supported.

@sergeyzwezdin bump. Would you mind reviewing the PR?

hi there, looking to use MagicChunks but noticed that latest stable version (<IP_ADDRESS>) is not compatible with Cake 0.26.
any chance to get this PR merged?

Would realy love to see this PR merged! @sergeyzwezdin can you please have a look at it?

@Alex-Witkowski I've merged it, but not actively working on the project right now.

@sergeyzwezdin is there anyway that we can help with this project to get a new release shipped? Happy to assist with this, just might need some pointers. Thanks again for this amazing library!

@gep13 Thank you! Well, there are problem with assembly (#71) and updating the project for Azure DevOps (#97). I planned to create separate organization and write it using .NET Standart + Node, and also implement it for all popular CI frameworks, but didn't have a time because of work. If you're ready to help at this stage (to resolve the current issues and make it working again), ping me by email/skype/telegram.

@sergeyzwezdin @gep13 I could also help maintaining it, especially the Cake addin part.

@sergeyzwezdin that is great news!  I will drop you an email just now.


Improve documentation

How to add the dependency to Spek (in gradle)
Maven version should be written in the future

How about your concerns about this pull request ?

I reflect reviewer's comment and rebased this branch.

We've addressed this in the new docs. (Maven still pending). Thanks.


Cisco ISE parser optimization regarding IcM

Required items, please complete
Incident 371233670 : [CRI] <2302220050001383> –
Change(s):

See guidance below

Reason for Change(s):

See guidance below

Version Updated:

Required only for Detections/Analytic Rule templates
See guidance below

Testing Completed:

See guidance below

Checked that the validations are passing and have addressed any issues that are present:

See guidance below

Guidance <- remove section before submitting

Before submitting this PR please ensure that you have read the following sections and filled out the changes, reason for change and testing complete sections:
Thank you for your contribution to the Microsoft Sentinel Github repo.

Details of the code changes in your submitted PR.  Providing descriptions for pull requests ensures there is context to changes being made and greatly enhances the code review process.  Providing associated Issues that this resolves also easily connects the reason.

Change(s):

Updated syntax for XYZ.yaml

Reason for Change(s):

New schema used for XYZ.yaml
Resolves ISSUE #1234

Version updated:

Yes
Detections/Analytic Rule templates are required to have the version updated

The code should have been tested in a Microsoft Sentinel environment that does not have any custom parsers, functions or tables, so that you validate no incorrect syntax and execution functions properly.  If your submission requires a custom parser or function, it must be submitted with the PR.

Testing Completed:

Yes/No/Need Help

Note: If updating a detection, you must update the version field.

Before the submission has been made, please look at running the KQL and Yaml Validation Checks locally.
https://github.com/Azure/Azure-Sentinel#run-kql-validation-locally

Checked that the validations are passing and have addressed any issues that are present:

Yes/No/Need Help

Note: Let us know if you have tried fixing the validation error and need help.

References:

Guidance for Detection checks
General contribution guidance
PR validation troubleshooting

Hi @v-rucdu, please provide your feedback. Thanks

Hello @v-rucdu please look into this

Hi @v-rucdu, please provide your feedback. Thanks

Hi @v-rucdu, could you please provide your feedback. Thanks

Hi @v-rucdu, please look into this and provide your feedback. Thanks

Hello @v-rucdu, please provide your feedback on this. Thanks

Hello @v-rucdu, please provide your feedback on this. Thanks

Hi
Currently we are having issues with parser performance testing tool. Dropped an email to Prathibha, once it's fixed, will test the parser performance and merge it.
Thanks

Hi
Started dumping data for parser testing, once data dumping will complete, we will test the parser performance and after that we will merge the PR.
Thanks


Double check if manage stored access policies is possible without keys for files OAuth

It is my understanding this API requires keys. So will not expose when no key credentials are present (same as blob).


Update quickstart-linux.md

Also, it appears the deployment template is stale.  It produces errors that asks for 'require ssh' policies for creating a vm.  Please try it out to confirm if this is valid observation.  Essentially, "Deploy the IoT Edge device" section is not working.

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

@eustacea Reiterating my comment from your other PR, that show-connection-string has been deprecated in favor of connection-string show. I double checked the CLI release notes and it appears that show-connection-string has been officially removed as of 0.10.2: https://github.com/Azure/azure-iot-cli-extension/blob/dev/HISTORY.rst#01012
I will take a look at the deployment template today to see if I can reproduce/resolve those errors.
#please-close

Perhaps something is amidst somewhere @@.***>?
[Graphical user interface, text  Description automatically generated with medium confidence]
Thanks for looking into it.
Eustace
From: Kelly Gremban @.>
Date: Monday, June 14, 2021 at 12:15 PM
To: MicrosoftDocs/azure-docs @.>
Cc: Eustace Asanghanwa @.>, Mention @.>
Subject: Re: [MicrosoftDocs/azure-docs] Update quickstart-linux.md (#76841)
@eustaceahttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Feustacea&data=04|01|Eustace.Asanghanwa%40microsoft.com|9d5f9f02333149312b6d08d92f68bbe3|72f988bf86f141af91ab2d7cd011db47|1|0|637592949165916966|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=y%2Fzd4%2FT6R2A6hbmROUyvQjU7%2FuIi09I5DqU3W0bOBKA%3D&reserved=0 Reiterating my comment from your other PR, that show-connection-string has been deprecated in favor of connection-string show. I double checked the CLI release notes and it appears that show-connection-string has been officially removed as of 0.10.2: https://github.com/Azure/azure-iot-cli-extension/blob/dev/HISTORY.rst#01012https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fazure-iot-cli-extension%2Fblob%2Fdev%2FHISTORY.rst%2301012&data=04|01|Eustace.Asanghanwa%40microsoft.com|9d5f9f02333149312b6d08d92f68bbe3|72f988bf86f141af91ab2d7cd011db47|1|0|637592949165927016|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=ZIQGbTBz7SEZFwMBBF3cP8iIoU1Vgmt5PvwhkhN3IWY%3D&reserved=0
I will take a look at the deployment template today to see if I can reproduce/resolve those errors.
#please-close
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fpull%2F76841%23issuecomment-860928628&data=04|01|Eustace.Asanghanwa%40microsoft.com|9d5f9f02333149312b6d08d92f68bbe3|72f988bf86f141af91ab2d7cd011db47|1|0|637592949165927016|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=hN%2FvafJNgfwQF1mZPREWc4%2BdYJlPr6jAXuNxZSft%2Fak%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAGWFHQHFDVXK3GCW7YUUGWDTSZIMBANCNFSM46RSKUZQ&data=04|01|Eustace.Asanghanwa%40microsoft.com|9d5f9f02333149312b6d08d92f68bbe3|72f988bf86f141af91ab2d7cd011db47|1|0|637592949165936905|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=WJvGZfCKa%2Fy4oahx8U1pbnEdf2BLZCjYtyrownYCaz8%3D&reserved=0.