Add HDFS StorageBackend implementation

This PR adds support for HDFS as a StorageBackend implementation. It also provides Kerberos authentication through the use of a provided keytab and supports asynchronous metric collection based on HDFS client file system statistics.
Users can provide HDFS client configuration in two ways: either by using traditional XML files, specifying their location in the hdfs.core-site.path and hdfs.hdfs-site.path options, or by passing the configuration options as regular Kafka options with the hdfs.conf. prefix.

Thanks @tigrulya-exe! This looks like a great addition and quite complete coverage of the storage back-end. However, I'm hesitant to move forward on the review as I lack experience on HDFS to be useful on anything apart from the APIs usage.
I'd like to leave this PR open in the meantime to gather feedback and let others to chime in around how to proceed with adding a new back-end.
There are still some work on the project we would like to prioritize before on-boarding a new back-end as well: preparing for Tiered Storage becoming prod-ready in 3.9 or later, and adding docs and release process, etc.
A couple of alternatives while this is open for discussion is to point to your fork (or a separate repo with just HDFS) from our README to let users know there's an HDFS implementation.
Let me know wdyt, and thanks again for your contribution!

@jeqo Hi! Thank you for the feedback! I think it's a nice idea to point to our fork with the HDFS storage implementation in your README while this PR is open for discussion :) I don't think we need to create a separate repository just for HDFS, as it could complicate porting future features from the main repository


stop early when there are no candidate hats for a grapheme

No functional changes.
However it prevents a bunch of pointless runs through the
scoring functions with an empty set of candidates.
(All the early stopping conditions check for a
unique winner, not the empty set.
Those aren't worth changing, when we can trivially
avoid calling the function at all.)
The main benefit is reduced noise when logging
inside the scoring functions when debugging.
Checklist

[/] I have added tests
[/] I have updated the docs and cheatsheet
[/] I have not broken the cheatsheet

I'd be tempted to short circuit at the start of maxByFirstDiffering

yeah, I wavered back and forth on that exact point. done.


ERR_CONNECTION_REFUSED under http://debian/

Unable to reach site. Connection denied by debian.
Try to:
Check the connection
Check your proxy and firewall
ERR_CONNECTION_REFUSED

fixed with ip adress


Log sampler migrate from qps to rpm

[x] If this pull request closes/resolves/fixes an existing issue, replace the issue number. Closes #. NO
[x] Update the CHANGES log.

And you should update LAL script, right?

This patch doesn’t change the public API / script, it used qps * 60 as RPM internally.

This patch doesn’t change the public API / script, it used qps * 60 as RPM internally.

I know. I am afraid this could be misleading. Because 30qps * 60 is different with 1800rpm. In the worth case, you could get over 500qps sampled easily in some certain cases.

I think we should break the API in this release.


CAO Karaf Commands to CRUD data

cao:get
cao:list
cao:update
cao:action-list
cao:action-execute
cao:ds-list
cao:ds-create
cao:ds-delete

CAO is no more supported since 7.0.0


[glslang] update to 11.11.0

Library name: glslang
New version number: 11.11.0
Other information that may be useful (release notes, etc...)
A new version of glslang came out 22 days ago, and I'd love to use it! I specifically wanted the TShader::setOverrideVersion method

I have been having some issues trying to update this port. I made a draft PR (#26671) but I was wondering if I could get some assistance as to what I should do to finish the update of this package. I added some extra notes as to the issue I'm having in the comments of that draft PR


Compilation error in v1.7

Getting this error when building with v1.7:
Compiling 28 files (.ex)

== Compilation error on file web/views/page_view.ex ==
** (CompileError) web/views/page_view.ex:3: module Scrivener.SEO is not loaded and could not be found

It looks like the using introduced in #59 is referring to Scrivener.SEO but the module is actually named Scrivener.HTML.SEO?

v1.7.1 released on hex, thanks for the issue!

Thank you for the great utility! :bow:


Client Capabilities: Verifiable Credential Proofs vs. Special discoverable prefix in storage

From #59

@zenomt: a document with a URI in an approved (according to the user's profile) location is just as good as a credential signed by an approved (according to the user's profile) issuer, but with no extra cryptography required (and it's a lot simpler).

If DPoP mechanism and verifiable credentials proof use the same crypto, relying on VC proofs doesn't seem like additional implementation complexity. I actually consider it simpler than having some magic location in the storage which can't be listed etc. Anyways I think we can consider separately how capabilities presented by the client can be verified
EDIT: reference to @zenomt's proposal: https://github.com/solid/authorization-and-access-control-panel/issues/48

The URI for the App Authorization document MUST be in (at a sub-path of) an acl:appAuthorizations in the user's profile (otherwise the resource server MUST ignore it):

https://solid.github.io/data-interoperability-panel/specification/ represents most advanced work on authorizing solid apps (clients)


python3: fix wrong platform libs when cross-compiling

see https://github.com/NixOS/nixpkgs/pull/169475#issuecomment-1129517328
patch by adisbladis
Co-authored-by: adisbladis<EMAIL_ADDRESS>Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

Maybe we should also update the comment?


Define ColorSwatch.lerp() function

Use case
ThemeExtension has been added as a way to support custom colors or theme properties.
It forces you to implement two methods: copyWith() and lerp().
lerp() method is used to linearly interpolate between two values.
A lot of classes already define their own lerp function like Size.lerp, Color.lerp, RRect.lerp, Radius.lerp, etc.
A common way of defining related colors in custom themes is using a ColorSwatch. However, it does not define its own lerp() function at the moment, what makes implement ThemeExtension.lerp() function a difficult task.
Proposal
Define a static lerp() function for ColorSwatch:
static ColorSwatch<T>? lerp<T>(ColorSwatch<T>? a, ColorSwatch<T>? b, double t)

It could delegate in Color.lerp() for every color in the swatch table.

I've issued a PR with a proposal https://github.com/flutter/flutter/pull/103701


USB drives always generate with removable=false

Steps to reproduce:

Create a VM
Go to config => drives
Create a new removable drive, hit done
Change the new drive's Image Type to Disk Image and Interface to USB
Save
Go to Config => QEMU and see the issue.

Issue:
The generated QEMU parameter is as follows:
-device usb-storage,drive=driveX,removable=false
When it should be:
-device usb-storage,drive=driveX,removable=true
This issue prevents the VM from starting because QEMU rejects the config (error "Device needs media, but drive is empty" regardless of if an image has been selected with the Browse option). I have verified that manually specifying the removable=true resolves the issue.

Is this a new bug introduced in .21? I’ve definitely had it working before.

I'm not sure when it was introduced but I've experienced this for a while. Only happens with drives set to removable in the UI. If you import a disk image and change it to USB it works fine.


[CHALLENGE SUBMISSION] 25

Please provide the following information for your submission:
What is your name? (First, Last)
Marcus Turewicz
Where is your GitHub Repo ?(With your challenge solution)
https://github.com/marcusturewicz/25-days-of-serverless-challenge/tree/master/Day-25
What Challenge is this for? (Challenge number between 1 and 25)
25
(Optional) Anything else we should know? e.g., language used, location, blog post?
C# Azure Function, Speech to Text and Blob Storage.

The unthinkable has happened: after weeks of community members all over the globe chipping in to help solve people's problems with serverless technology, the evil grinch's heart is starting to soften! He's still not quite ready to give up his server-stealing plot, but our whole Azure Cloud Advocacy team is flying up to the grinch's lair to help convince him to change his ways.
As such, the team behind 25 Days of Serverless is OOF (Out Of Facility/Office) between Dec 24, 2019 and January 1, 2020. We'll not review Issues in this period of time, but encourage you to catch up on any challenges you haven't had the time to work on yet. 😄 Our Discord Server is a perfect place to reach out to for community support during this time. After our return, we'll add contributors who submitted a working solution as an Issue to CONTRIBUTORS.md file until January 15, 2020.


Update HygieiaArtifactPublishStep.java

adding artifact status feature to get the exact status of artifact. it can be of following values [active,inactive,inprogress]

@irfanjs please address the comments above, if we do not hear from you we will go ahead and close this pull request.

@irfanjs we are closing this PR as there is no response from the submitter and code is not functional. When you are ready with the fully functional code please resubmit a PR and we will get this reviewed and merged. Please see the review comments in case you need direction on how to proceed.


Portrait display orientation, half of Brackets is missing

### Description
When using Brackets on a desktop display in portrait orientation, half of the Brackets window is missing.
### Steps to Reproduce

Open Brackets
Set a single display to be in portrait orientation
Drag / move brackets to this display to replicate.

Expected behaviour:
Brackets to take over the full display height.
Actual behaviour:
It would appear a max-height type setting is applied and crops the Brackets application window at just over 50% of the portrait displays height.
### Versions

Windows 10 Pro
Version 1703
OS Build 15063.413
Brackets
-- Release 1.9 build 1.9.0-17312 (release 189f6d39a)
-- build timestamp: Tue Mar 21 2017 10:29:45 GMT+0000

I hope this helps.

I tried the latest pre-release on Win8 and Win10, I am not able to reproduce the error, it is working fine

@saurabh95 I removed my version entirely, tried to re-run the install but same outcome.
I then re-downloaded the .exe and installed again, works perfectly. (Corrupted download potentially)
Second attempt to fresh install has resolved the issue.
Thanks for investigating.


IGNITE-13952 Cache metrics without description

Co-authored-by: a-polyakov<EMAIL_ADDRESS>(cherry picked from commit 57c53ceb94c8a399c48bdde87489a118909d54ed)

This PR is associated with a resolved ticket. Closing the PR, please reopen it if you still need it.


AMBARI-24939. Refactor logsearch_metadata schama and usage for logsearch portal

What changes were proposed in this pull request?
refactor logsearch_metadata schema, and make logsearch metadata api work with specific types dynamically (by any kind of types)
How was this patch tested?
with docker env

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-LogSearch-Github-PR-Builder/86/
Test PASSed.


Fixing a condition that cause upgrade failure.

An error would occur every time ansible pass by this task as "needs_cordoning" is treated as a string " true " which is not boolean! and so next 2 tasks in the same file fail and shows an error

Gonna close/reopen to see if CI will check in.

ci check this

Can you rebase off current master to get the necessary CI fixes

Looks like this passed CI when looking at the pipeline directly. Good work @Abdelsalam-Abbas, thank you!


Waddle Penguins Island API Maintenance

We'll be bringing down the Waddle Penguins Island API to fix login bugs. The service will be unavailable.

We have taken Waddle Penguins Island down for maintenance.


error in serialization of Transformer result

When a transformer creates a dictionary where the values are numpy scalars,
Seamless says "Transformer exited without result"

Fixed in 0.1.1


Change naming convention from -webpack-plugin to webpack-plugin-

webpack has established an odd convention of suffixing plugin names with -webpack-plugin as opposed to a prefix webpack-plugin-. webpack 2 seems like a good opportunity to change this.

Obviously, it makes sense to define naming convention similar to @angular and @types. So that plugins will be grouped together in config file and in node_modules directory.

Personally I like the *-webpack-plugin naming convention. It matches the format similarily to core plugins. But I don't have time to :bike: shed

There is one benefit using common prefix. It might provide better autocomplete

it allows to have them grouped in package.json too


about rendering pdf file from shared location

HI I am new to PDF.js and i want to load pdf file from shared location instead of from local application context . Please suggest me approach to load pdf file from shared location in linux like /opt/temp

Based on the very limited information provided, it doesn't sound like this is actually a PDF.js specific issue, but rather a more general question regarding file handling in web applications.
Closing as invalid/incomplete.


Godot CLI: Run a project + run a specific scene

Hi,
I would like to use Godot as a CLI. Let's say I have a project.godot file which defaults to MainScene.tscn when run through the editor, and I have an additional RunAllTestsScene.tscn.
I would like to be able to invoke Godot to:

Run the project from the CLI, without opening the editor (something like godot --run <project.godot>, which is like hitting F5 in the editor.
Run RunAllTestsScene.tscn from the CLI, eg. godot --run Scenes/RunAllTestsScene.tscn

I couldn't find a way to do this. Is it possible? If not, can you add support for this?  The latter allows me to integrate with Travis and friends to get proper CI (eg. failure notification when I break tests which break the build).

It is possible: https://docs.godotengine.org/en/3.1/getting_started/editor/command_line_tutorial.html

Thanks, don't know how I missed that page.


Cannot grant non-admin shell access any more after #2154

#2154 has changed how shell access is granted. In my use case I wanted to grant a non-admin user ssh access to the firewall.
This does not seem to be possible any more, either I put the user in "admins" and permit SSH, or the user cannot log in through SSH.
This pops up in system.log: Aug 20 13:52:51 XXX-XXXXXX-XXXX sshd[xxxxx]: error: PAM: authentication error for illegal user xxxxxxxx from xxx.xxx.xxx.xxx

@pv2b just to be sure, what did you configure in "Login Group"?
The release notes do have a mention about this, but it's easy to overlook,

SSH access can be set for an arbitrary group as well under System: Administration for non-members of "admins" group.  However, in both cases only SCP works due to a request in the forum to be more proactive regarding yielding of shell access rights.  If you want a user to gain true SSH access you need to change their shell from "nologin" to an installed shell in their respective settings.

I did see this in the release notes but to me this is cryptic to the point of not being understand what is said.

SSH access can be set for an arbitrary group as well under System: Administration for non-members of "admins" group.

How? The appropriate path is System: Settings: Administration, but under "Login Group" only "wheel" or "wheel, admins" are possible to pick.
The "wheel" group is not accessible through the WebUI, I'm guessing this is the underlying FreeBSD group that permits root access(?).

To clarify, in my case, login group is set to "wheel" which still somehow seems to permit my user to log in even if I just add him to the admin group confused

Wheel is always preselected for clarity. You can choose a second arbitrary group if you create one. The admin group has right page-all which gives wheel access to users directly. Since we don’t handle wheel from the gui it’s a bit hard to follow but not unreasonable.

PS: also set an user shell for each user as stated in the same notes.

That doesn't seem to explain why I am able to log in to the device when "wheel" has been selected as the user group, but I do get to log in as soon as I add the user to the admins group.

You’re not providing enough data to reproduce.

On 20. Aug 2018, at 13:50, pv2b<EMAIL_ADDRESS>wrote:
That doesn't seem to explain why I am able to log in to the device when "wheel" has been selected as the user group, but I do get to log in as soon as I add the user to the admins group.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or mute the thread.


Support Additional Text Feature data

character position tracking #236
number of tokens #236
string length #236


Add high fidelity stylus support.

This change makes UberSignature use high fidelity input from the Apple Pencil when provided. This allows for smoother strokes.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@myhrvold could we potentially merge this?


SMTP configuration mismatch between Gitea and Postmark

Description
Issue:
When trying to configure Gitea to send mail via a Postmark SMTP server, the system is unable to send emails, the test end point in /admin/config does not even complete with an error - it simply spends forever trying to complete.
Details:
I've taken a number of steps to narrow this down. If I configure Gitea to any other SMTP server, I manage to get it working without too much trouble. If I use other ways of testing the Postmark SMTP server, it too manages to work. It's only when I put the two together that I get the broken behavior.
There are two methods of setting up an SMTP server with Postmark. One uses the API token for a Postmark server as both the user and password of the SMTP configuration. The other generates a special "access key" / "secret key" for a particular message stream. Through experimentation I found that only the first method is broken. (And again, it is only broken when I put it together with Gitea - the first method works just fine when tested in any other way).
I took a look at the debug log after trying to run the test email and this is the last line in the log
2023/09/14 22:11:27 ...eb/routing/logger.go:68:func1() [W] router: slow      POST /admin/config/test_mail for ***.***.***.***:0, elapsed 3847.4ms @ admin/config.go:30(admin.SendTestMail)

The page itself simply goes into a never ending loading state.
Gitea Version
1.20.3
Can you reproduce the bug on the Gitea demo site?
No
Log Gist
No response
Screenshots
No response
Git Version
No response
Operating System
No response
How are you running Gitea?
I am running Gitea from the binary installation instructions on an Ubuntu on systemd. No docker. Ubuntu version 22.04.2
Database
SQLite

Without leaking any of your credentials, could you share your mailer config for postmark?

Yes. I've tried a few different configurations - all that seem like they should work when I test postmark other ways.
[mailer]
ENABLED = true
PROTOCOL = smtp+starttls
SMTP_ADDR = smtp.postmarkapp.com
SMTP_PORT = 25
USER = *** Server API-Token ***
PASSWD = *** Server API-Token ***
FROM =<EMAIL_ADDRESS>
I have tried with smtp and with smtp+starttls and I have tried with port 25 and port 587. All had the same behavior.


Grid frame not working properly in Safari

Reported by @zurbrandon via twitter. Reproduced it on https://codepen.io/kball/pen/vZBeLy

So this appears to be due to a difference in the way Safari is handling height calculations of flex children.... the grid is being sized appropriately to fit in the space, but the child cells with .medium-cell-block-y are not, they end up too big and thus don't scroll internally, and we've deliberately disabled externals scroll in this situation (that's what
I think it is tied into this bug: https://github.com/philipwalton/flexbugs#11-min-and-max-size-declarations-are-ignored-when-wrapping-flex-items in that the max-height declaration is being ignored.
A workaround is to disable flex wrapping on the grid in this situation. We can't do that globally because the automatic wrapping to the next line is a key function of the grid, but in this usecase that behavior would actually be more "expected" because we're explicitly laying out our space in the viewport and saying we want these blocks to handle scrolling internally... thats what the .cell-block-container class is for.  So I think that if we add this as one of the effects of the .cell-block-container on its immediate child grid, we will both resolve this problem and not have any unexpected side effects.
I've put together a quick PR at https://github.com/zurb/foundation-sites/pull/10124 but would love it if someone checked my thinking above @brettsmason and @zurbrandon

Tagging @corysimmons based on his extensive flexbug experience to also review my thinking and if there's another way to address this that he happens to know

Sorry man, I need to focus on getting a job. Unsubscribing from all this stuff.
fwiw, I have cross-browser success with the panel stuff via padding grid.

I've looked this over and the logic behind this makes sense. I think we can assume this would be for app style layouts, and setting this to nowrap for the child grid sounds perfectly reasonable for this sort of layout. 👍


chore: Doc Touchups Mostly

Description
Mostly touches up cargo-hoist docs. Also performs a path version bump to test the workflow overlords.

Current dependencies on/for this PR:

main

PR #1   👈

This comment was auto-generated by Graphite.


Return correct status codes in departures backend when handling EIS router responses

Based on: https://github.com/hmrc/transit-movements-trader-at-destination/pull/154 and https://jira.tools.tax.service.gov.uk/browse/CTDA-803. Needs to wait till John is back.

test this please

test this please

test this please

@jhkldev can you please make sure there are tickets to address using the new update endpoint more broadly, even if it is simply a spike ticket so that the declaration of "depreciated" repository endpoints isn't left by the wayside.
Also, whilst I appreciate the technical value of the change, and I'll let it be merged in, I'd like you to highlight that the spike should obfuscate the database-level considerations from the rest of the system - something that would typically be called DepartureService (we might need to rename our current DepartureService to reclaim that name in the spike - perhaps to ModelBuilder?). Can you please create this ticket and add it to the PR in a reply comment, and then I'll approve.

https://jira.tools.tax.service.gov.uk/browse/CTDA-1025


Add threshold for rds.go codecov

Test is flakey, saying it has droppped coverage when it has not due to
it being nondeterministic.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: howardjohn
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: linsun
If they are not already assigned, you can assign the PR to them by writing /assign @linsun in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@howardjohn: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
c59a6c7f676457c92068c1fc1412772b7880f2c6
link
/test istio-integ-k8s-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
c59a6c7f676457c92068c1fc1412772b7880f2c6
link
/test istio-integ-k8s-tests

prow/istio-pilot-multicluster-e2e.sh
c59a6c7f676457c92068c1fc1412772b7880f2c6
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Send it to master?
1.1 development is over.

This isn't an issue on master branch so I thought it should go to the branch where it applies


feat: scf support publish version and traffic config

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Improve snapshot dir error message

Users upgrading to v2.3 don't always know to configure ledger.snapshots.rootDir.
In cases where peer doesn't have write access to this directory,
the error message now mentions the property to configure.
Signed-off-by: David Enyeart<EMAIL_ADDRESS>

@Mergifyio backport release-2.3


GFItemsCarousel

how to using GFItemsCarousel auto play?

You just need to set  autoPlay: true, in GFCarousel widget.
For code please check this
https://docs.getwidget.dev/gf-carousel/


Change CI from Travis to GitHub Actions

Add a first GitHub workflow CI. I expect that the PHP "extensions" list will need to be expanded. But I  think that I have to merge first, in order to get the first run.
I will do these in the coming hours:

event
skel (I may as well keep the "template" up-to-date there)
uri
xml-atom
Baikal

@staabm please remove the required status for Travis on those.

@phil-davis done


Implement PYI048 for flake8-pyi plugin

Summary
Adds PYI048 rule to flake8-pyi.
Open question is how to flag this. Rule currently triggers a violation for every offending line. Which clearly communicates the relevant lines but could lead to a lot of clutter for a large multi-line func. Alternatives considered were highlighting the function name or the last line of a given function, both felt a little unclear, if concise. Any input on this is welcome
Test Plan
Test files (both stub and non-stub) added. Scenarios with and without dosctrings considered to not overlap with PYI021: DocstringInStub as done in other rules and confirmed in: https://github.com/charliermarsh/ruff/pull/4634#issuecomment-1561419718

Open question is how to flag this. Rule currently triggers a violation for every offending line. Which clearly communicates the relevant lines but could lead to a lot of clutter for a large multi-line func. Alternatives considered were highlighting the function name or the last line of a given function, both felt a little unclear, if concise. Any input on this is welcome

I'd vote to use the name of the function. We have a utility for this (helpers::identifier_range(stmt, self.locator)), which will "do the right thing" if you pass in the function statement.


@embroider/macros causing builds to fail > v2.2

After upgrading our dependencies builds started crashing with the below error messages:
Build Error (broccoli-persistent-filter:Babel > [Babel: @embroider/macros]) in @embroider/macros/runtime.js

<EMAIL_ADDRESS>The "path" argument must be of type string. Received an instance of Object

and
Build Error (broccoli-persistent-filter:Babel > [Babel: @embroider/macros]) in @embroider/macros/runtime.js

<EMAIL_ADDRESS>@babel/template placeholder "1000": Expected string substitution

Our investigation lead us to the bump from mirage version 2.2 to 2.4, by locking it to 2.2 builds weren't crashing anymore (2.3 is confirmed to also crash).
I tried reproducing the error in a standalone app but couldn't get it to crash. My guess it has something to do with dependencies from different addons on different versions of the same package (I noticed ember-power-select also has a dependency on @embroider/macros in our app, but that was not causing the issue).
Nevertheless, the issue stands, perhaps someone can find this with similar problems, and we can narrow it down even further.

I tried reproducing the error in a standalone app but couldn't get it to crash.

I guess this resurface some issues that exist in older @embroider/macros version ember-cli-mirage > v2.2
is using.
The problem with bumping @embroider/macros is that it requires Node.js 12+ and ember-cli-mirage v2 supports Node.js 10.
Upgrading @embroider/macros to 1.0.0 would be a breaking change as it would ultimately require Node.js 12+.
I would to try<EMAIL_ADDRESS>as it upgrades @embroider/macros and should be pretty stable (although you should update all the import paths remove in #2314).
Another option is to use resolutions and enforce<EMAIL_ADDRESS>I'm hesitant to bump @embroider/macros to v1 in ember-cli-mirage as it's breaking change.
Another option to move forward is to release stable v3 with<EMAIL_ADDRESS>and rollback #2297 in v2 (which is not great option as well IMO).
Ultimately and unfortunately, issues is not in ember-cli-mirage but in older version of @embroider/macros used in v2.

I'm up for bumping to v3.0.0-alpha.1, but how do import paths need to be updated? Currently having imports like this:
import { Request, Response } from "miragejs";
import { Server } from "miragejs/server";

Cheers for the response.

We are also talking about the imports in your mirage models, adapters, etc
For example: many people have
import { Model, belongsTo } from 'ember-cli-mirage';

which should be changed to
import { Model, belongsTo } from 'miragejs';

Anything that was extracted to MirageJS (almost everything) should be imported frommiragejs
The things left that ember-cli-mirage exports is setupMirage and things used in defining the server as below.
Previously your routes were the only thing defined in the config.js, now your config.js should look like this
import {
  discoverEmberDataModels,
} from 'ember-cli-mirage';
import { createServer } from 'miragejs';

export default function (config) {
  let finalConfig = {
    ...config,
    models: { ...discoverEmberDataModels(), ...config.models },
    routes,
  };

  return createServer(finalConfig);
}

function routes() {
// define your routes here
}

This now makes our definition of a server look like MirageJS documentation.

v3.0.0-alpha.1 still supports the routes only config, but deprecated, will be releasing an alpha.2 that removes that soon

v3 isn't working for me.
Can't even get ember-cli to build.
Build Error (broccoli-persistent-filter:Babel > [Babel: @embroider/macros]) in @embroider/macros/runtime.js

<EMAIL_ADDRESS>@babel/template placeholder "APP": Expected string substitution

stack:
(node:349084) UnhandledPromiseRejectionWarning: Error: @babel/template placeholder "APP": Expected string substitution
    at applyReplacement (<snip>/node_modules/@babel/template/lib/populate.js:74:13)
    at <snip>/node_modules/@babel/template/lib/populate.js:44:7
    at Array.forEach (<anonymous>)
    at populatePlaceholders (<snip>/node_modules/@babel/template/lib/populate.js:42:43)
    at <snip>/node_modules/@babel/template/lib/string.js:20:51
    at <snip>/node_modules/@babel/template/lib/builder.js:75:14
    at Object.buildLiterals (<snip>/node_modules/ember-element-helper/node_modules/@embroider/macros/src/babel/evaluate-json.js:377:73)
    at Object.inlineRuntimeConfig (<snip>/node_modules/ember-element-helper/node_modules/@embroider/macros/src/babel/get-config.js:107:55)
    at PluginPass.enter (<snip>/node_modules/ember-element-helper/node_modules/@embroider/macros/src/babel/macros-babel-plugin.js:77:38)
    at newFn (<snip>/node_modules/@babel/traverse/lib/visitors.js:177:21)
    =============
    at Object.buildLiterals (<snip>/node_modules/ember-element-helper/node_modules/@embroider/macros/src/babel/evaluate-json.js:377:34)
    at Object.inlineRuntimeConfig (<snip>/node_modules/ember-element-helper/node_modules/@embroider/macros/src/babel/get-config.js:107:55)
    at PluginPass.enter (<snip>/node_modules/ember-element-helper/node_modules/@embroider/macros/src/babel/macros-babel-plugin.js:77:38)
    at newFn (<snip>/node_modules/@babel/traverse/lib/visitors.js:177:21)
    at NodePath._call (<snip>/node_modules/@babel/traverse/lib/path/context.js:53:20)
    at NodePath.call (<snip>/node_modules/@babel/traverse/lib/path/context.js:40:17)
    at NodePath.visit (<snip>/node_modules/@babel/traverse/lib/path/context.js:100:31)
    at TraversalContext.visitQueue (<snip>/node_modules/@babel/traverse/lib/context.js:103:16)
(Use `node --trace-warnings ...` to show where the warning was created)
(node:349084) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag `--unhandled-rejections=strict` (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 1)
(node:349084) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

Interestingly I get that same error when I try to update ember font-awesome and staying on ember-cli-mirage 2.4. I had thought it was a font-awesome problems.
Could be another embroider migration problem.
If your app using auto-import V1 or V2?
Feel a lot of addons are probably encountering these mirgration issues
From: PJ Carly @.>
Date: Thursday, February 3, 2022 at 6:32 AM
To: miragejs/ember-cli-mirage @.>
Cc: Gantzler, Brian @.>, Comment @.>
Subject: Re: [miragejs/ember-cli-mirage] @embroider/macros causing builds to fail > v2.2 (Issue #2343)
    External Email – Please use caution before opening attachments or clicking links

v3 isn't working for me.
Can't even get ember-cli to build.
Build Error (broccoli-persistent-filter:Babel > [Babel: @embroider/macros]) in @embroider/macros/runtime.js
@.***/macros/runtime.js: @babel/template placeholder "APP": Expected string substitution
stack:
(node:349084) UnhandledPromiseRejectionWarning: Error: @babel/template placeholder "APP": Expected string substitution
at applyReplacement ***@***.***/template/lib/populate.js:74:13)

at ***@***.***/template/lib/populate.js:44:7

at Array.forEach (<anonymous>)

at populatePlaceholders ***@***.***/template/lib/populate.js:42:43)

at ***@***.***/template/lib/string.js:20:51

at ***@***.***/template/lib/builder.js:75:14

at Object.buildLiterals ***@***.***/macros/src/babel/evaluate-json.js:377:73)

at Object.inlineRuntimeConfig ***@***.***/macros/src/babel/get-config.js:107:55)

at PluginPass.enter ***@***.***/macros/src/babel/macros-babel-plugin.js:77:38)

at newFn ***@***.***/traverse/lib/visitors.js:177:21)

=============

at Object.buildLiterals ***@***.***/macros/src/babel/evaluate-json.js:377:34)

at Object.inlineRuntimeConfig ***@***.***/macros/src/babel/get-config.js:107:55)

at PluginPass.enter ***@***.***/macros/src/babel/macros-babel-plugin.js:77:38)

at newFn ***@***.***/traverse/lib/visitors.js:177:21)

at NodePath._call ***@***.***/traverse/lib/path/context.js:53:20)

at NodePath.call ***@***.***/traverse/lib/path/context.js:40:17)

at NodePath.visit ***@***.***/traverse/lib/path/context.js:100:31)

at TraversalContext.visitQueue ***@***.***/traverse/lib/context.js:103:16)

(Use node --trace-warnings ... to show where the warning was created)
(node:349084) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag --unhandled-rejections=strict (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 1)
(node:349084) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.
—
Reply to this email directly, view it on GitHubhttps://github.com/miragejs/ember-cli-mirage/issues/2343#issuecomment-1028899450, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AALVLRQ5VO72MNN4AARZCK3UZJRUTANCNFSM5M2XSMRQ.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you commented.Message ID: @.***>

This message is for the designated recipient only and may contain privileged, proprietary
or otherwise private information. If you have received it in error, please notify the sender
immediately and delete the original. Any other use of the email by you is prohibited.
Dansk - Deutsch - Espanol - Francais - Italiano - Japanese - Nederlands - Norsk - Portuguese - Chinese
Svenska: http://www.cardinalhealth.com/en/support/terms-and-conditions-english.html

using ember-auto-import v2 in my dependencies however there is still a v1 present in my lock file (from ember-intl).

using ember-auto-import v2 in my dependencies however there is still a v1 present in my lock file (from ember-intl).

@pjcarly could you please check if it's EAI 1.12+ as it has @embroider/* in dependencies.
Regarding<EMAIL_ADDRESS>@babel/template placeholder "APP": Expected string substitution
this issue happens when you have outdated @embroider/* packages somewhere in transitive dependencies.
Can you try to re-roll your lock file to get latest deps?
If issue persists with v3, then there is some another dependency that may need a bump to use v1 of @embroider/* across the board

ember-auto-import v1 is resolved to v1.12.1. (coming from dependency ember-intl v6.0.0-beta.3)
After further investigation it seems that there are still @embroider/* dependencies under v1 (coming from ember-power-select and ember-basic-dropdown.
However the weird thing is, that everything is building just fine (with EPS and EBD), as long as ember-cli-mirage isn't added to the dependencies
¯_(ツ)_/¯

We're running into this same issue with our applications when trying out the v3-alpha release. I've tried forcing an @embroider/macros resolution in both directions (down to 0.50.x and up to 1.x) and neither is compatible.
If ember-cli-mirage releases a breaking v3 that requires<EMAIL_ADDRESS>then it seems that it's going to be a huge blocker to adoption. Maybe consider keeping<EMAIL_ADDRESS>until there's a clear upgrade path that doesn't break everyone's builds?

This is an issue everyone is having. It shows different error messages but it is the same error. Please see https://github.com/FortAwesome/ember-fontawesome/issues/194 and https://github.com/embroider-build/embroider/issues/1077
The second link thinks that the error actually happened with .41 of macros (and not sure why it didnt start being an issue til .50)
This issue states the problem happened in this repo on version 2.3 Looking at the yarn lock of 2.3 we included .41 of macros. https://github.com/miragejs/ember-cli-mirage/blob/v2.3.0/yarn.lock#L1821 It was updated as a result of updating the package.json. If this was the catalyst, its still pre .50.  Would have to test a failing build with a fork of 2.3 and revert macros back to .40
I personally am using ember-cli-mirage 2.4 and not having the issue, but thats not the case for the person who opened the issue. Im currently fighting this exact issue, but with the update to ember-fontawesome.
Much like the ember-cli-babel 7.26 issue, It would seem the way forward is to get every addon to update to macros 1.0 as soon as possible.
Im really looking to embroider/macros to tell what what the way forward is.

@cah-briangantzler My point is that ember-cli-mirage opted into forcing this issue by upgrading the @embroider/macros dependency when it worked fine before as it was. If that upgrade is rolled back, it could open the door to many more people opting into the 3.0-alpha series and being able to provide feedback on the changes the addon is making itself.
I agree that this is on @embroider/macros to provide a path forward, but ember-cli-mirage isn't required to force the pain point in the meantime by upgrading that dependency.

@kpfefferle @cah-briangantzler more thinking of this, more I convinced we should rollback @embroider/macros in v2 to not cause churn for people with < 1.0.0 version we use in ember-cli-mirage@2.
I'll open PR once have a chance and release patch release v2.4.1 without @embroider/macros.
ember-cli-mirage@3 will stay on @embroider/macros@1.
Note that we can't use @embroider/macros@1 in ember-cli-mirage@2 due to Node.js version requirements.

For anyone else struggling with this, I was able to get things working by forcing @embroider/macros to resolve to a single version in my package.json:
// package.json
{
  // ...
  "resolutions": {
    "@embroider/macros": "1.2.0"
  },
  // ...
}

This seems to work with a wide range of @embroider/macros versions in my app as long as I rm -rf node_modules before yarn install to ensure conflicting versions are not hanging around and causing issues.

@SergeAstapov if you are rolling back macros in V2 (and I think you should), I would stick with .40 as that was in 2.2 and people are reporting when we went to .41 in 2.3 is started causing issues
@kpfefferle I had tried this for my upgrading to ember-fontawesome and it did not work for me. However macros was at 1.0.1 at the time. Perhaps 1.2 has fixed the issue, will try it again.

@kpfefferle I tried the resolutions again and this time it worked. I think I neglected to do the rm -rf node_modules last time. Thanks for the tip.

Honestly. I don't understand anything about embroider/macros and how it is causing the issues.
I have many years of Ember experience, but this is a complete mystery to me. Is there any documentation I can read up on to get a better understanding?

I've just run into this both when installing ember-cli-mirage in a fresh 4.2.0 project, and when installing the latest ember-fontawesome.
Is there any path forward without requiring npm-force-resolutions?

fwiw I got past the ember-fontawesome issue by removing regex from fastboot hostwhitelist per https://github.com/ember-fastboot/ember-cli-fastboot/issues/889

For anyone visiting this issue, I was having the reported issue with a fresh new project using Node 20.5.1, Ember 5.3.0, and ember-cli-mirage 2.4.0:
$ ember new mwe
$ cd mwe
$ ember install ember-cli-mirage
$ ember serve

Upgrade ember-cli-mirage to 3.0.0-alpha resolved the issue.

Upgraded from 2.1.0 (I had this problem in 2.2.0) to 3.0.0 and it works to me (https://www.ember-cli-mirage.com/docs/getting-started/upgrade-guide#3-0-upgrade-guide)

Path forward for anyone having such issue - upgrade to v3.


Query optimization: remove inner join from subquery navigation contains

When a sub query uses Contains on a navigation, like the following:
        [ConditionalFact]
        public virtual void Where_subquery_on_navigation()
        {
            using (var context = CreateContext())
            {
                var query = from p in context.Products
                            where p.OrderDetails.Contains(context.OrderDetails.FirstOrDefault(orderDetail => orderDetail.Quantity == 1))
                            select p;

                var result = query.ToList();

                Assert.Equal(1, result.Count);
            }
        }

The SQL contains an inner join on the same table:
SELECT [p].[ProductID], [p].[Discontinued], [p].[ProductName], [p].[UnitsInStock]
FROM [Products] AS [p]
WHERE (
    SELECT CASE
        WHEN EXISTS (
            SELECT 1
            FROM (
                SELECT [o].[OrderID], [o].[ProductID]
                FROM [Order Details] AS [o]
                WHERE [p].[ProductID] = [o].[ProductID]
            ) AS [t]
            INNER JOIN (
                SELECT TOP(1) [orderDetail].[OrderID], [orderDetail].[ProductID]
                FROM [Order Details] AS [orderDetail]
                WHERE [orderDetail].[Quantity] = 1
            ) AS [t0] ON ([t].[OrderID] = [t0].[OrderID]) AND ([t].[ProductID] = [t0].[ProductID]))
        THEN CAST(1 AS BIT) ELSE CAST(0 AS BIT)
    END
) = 1

In cases where the Inner Join is on the same table, it may be possible to eliminate the join
SELECT [p].[ProductID], [p].[Discontinued], [p].[ProductName], [p].[UnitsInStock]
FROM [Products] AS [p]
WHERE (
    SELECT CASE
        WHEN EXISTS (
            SELECT 1
            FROM (
                SELECT TOP(1) [orderDetail].[ProductID]
                FROM [Order Details] AS [orderDetail]
                WHERE [orderDetail].[Quantity] = 1
            ) AS [t]
            WHERE [p].[ProductID] = [t].[ProductID])
        THEN CAST(1 AS BIT) ELSE CAST(0 AS BIT)
    END
) = 1

@smitpatel Duplicate of https://github.com/aspnet/EntityFrameworkCore/issues/3419?

We need to see what is SQL generated in new pipeline.

The test is blocked on #15260


[Word] Publish requirement set 1.4

Docs Build status updates of commit d20d0b9:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.publish.config.json
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 46df7ad:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.publish.config.json
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Add icons for editor settings tabs

This morning I tried finding the editor shortcuts tab and actually overlooked it:

How about adding icons here?

Please lets not get into the adding icons everywhere trend, I tried to avoid this since forever

I prefer without too.

I have to admit it's actually cleaner without icons.

Cleaner and simpler to maintain, since people won't have to go looking for/making a fitting icon every time they add a new tab anywhere!

OK, I agree, closing this :-)

The tab theme could be improved though, I agree that it's hard to notice the non active tabs currently.

maybe we could avoid dimming the text or dim less


Static choices in menu.

Hello guys. I have a question: i recently made a societies script for vrp that can let you update the paycheck of each rank. I was wondering if is there a way to create a sub-menu inside the society menù (that i have already built) to show just the paycheck of that rank:
example:
Captain: 2000
Liutenent: 1000
ecc....
without any function triggered on click.

oh and i'm using an old version of vrp (dunko edited).

https://github.com/ImagicTheCat/vRP/blob/master/doc/README.adoc#issues--features--help


Endpoint list with render-styles 'focused' and 'read' is not displayed correctly

We have an API with a lot of Endpoints and use the render-style 'focused'. We noticed that the endpoint list on the left is cut at the bottom and not all endpoints are listed (see picture).
The issue only occurs when initialy loading the specification, when collapsing and expanding the list, it is displayed correctly.
We had problems with that, because some endpoints could not be found at first sight.
The issue also occurs for render-style 'read'.
I added a sample json-file for reproduction.
OpenAPI_Rapidoc_Demo.json

@mrin9 The commit referenced fixed this issue, should this issue be auto-closed?


Feature Request: Mirror Client

We need an implementation of the MirrorClient & RemoteClusterUtils classes to enable consumer offset synchronization across clusters.
Need to be able to 1) translate offsets from local to remote topics and 2) seek the consumers to those offsets (KIP-382)
This is to support active/standby disaster recovery scenarios since MM2 does not currently automatically synchronize these translated offsets (KIP-545).

thank for the suggestion. using this issue to track interest (it's not currently on our backlog).

thank for the suggestion. using this issue to track interest (it's not currently on our backlog).

Can this be done via the command line and if so is there documentation on how to do it?


Update to the latest version of Azure Cosmos DB JavaScript SDK

“Update to the latest version of Azure Cosmos DB JavaScript SDK
You’re receiving this message because you've used version 1.x of the JavaScript SDK for Azure Cosmos DB SQL in the last 12 months.
We've recently released updates to the JavaScript SDK for Azure Cosmos DB to provide performance, stability, and feature improvements.
As a result, we'll be retiring and no longer supporting version 1.x of the JavaScript SDK on August 30, 2020. After that date, you’ll no longer be able to use it to access Azure Cosmos DB. As with all changes of this type, we’re providing 12 months’ notice so you have adequate time to adjust.
Required action
To continue using the JavaScript SDK for Azure Cosmos DB, update to version 2.x or 3.x by August 30, 2020.
“Cosmos DB account has one or more client applications using version(s) of the Nodejs SDK older than 2.0.0”
The Node JS version that is used officially by Cosmos at this moment is version 3.1.0 as seen in our official documentation below lynk:
https://docs.microsoft.com/en-us/azure/cosmos-db/sql-api-sdk-node

[bug]
[javascript]

Support for the Bot Builder V3 is ending January 1st, 2020 ( https://blog.botframework.com/2019/06/07/v3-bot-broadcast-message/ ).  Since the CosmosDb end date is after this, there is no need to address this issue.
Note: CosmosDb will continue to function after August 30th 2020, as will Bot Builder V3 SDK.


bridges | way to remove animals, add removed animal to inventory

Motivation
You should be able to remove animals if you dont want to place them, and the inventory should update accordingly.
This should be done after #172 or in tandem with it
Implementation
Implementation should be as follows:
Use the Buttons introduced in #172

you should be able to drag the animal back into the button to get rid of it
deleting animal should update inventory
(ideally: drag them to box where that animal is contained, in line with the scroll idea to place animals (see game design docs))

Besides this specific functionality:

you should be able to drag the animal back into the button to get rid of it

removing animals works by using the button on the upper right ( reset / revert last step )


responsive layout

now the heading looks good both on default desktop as well as on smaller screens
pls merge this PR

@Priyanshi662 pls review


User management and git

Hello! First off, I do really like your configuration layout and have adopted it when switched to flakes.
I was wondering how exactly you manage git credentials (username and email) for different users?
I have two different users that both have different git credentials. My idea was to add new fields to
user-manager that are git-specific. But it seems that it's not possible to change the config of
home-manager modules inside of a NixOS module.
I've found this like but can't figure out where the username comes from: https://github.com/lukebfox/nix-configs/blob/5432ca7dead6d39fca9b9772fc3c10914ba6969f/profiles/home-manager/developer/default.nix#L4

Hey sorry for the late reply. If you're still interested or for anyone else reading, user data is contained in ./data/shared.toml. This data is converted from toml into nix:
https://github.com/lukebfox/nix-configs/blob/60c64f14c2212b181fc2819c438b7cfe696bc86d/flake.nix#L41-L42
At the nixos config level, user profiles are imported:
https://github.com/lukebfox/nix-configs/blob/60c64f14c2212b181fc2819c438b7cfe696bc86d/configs/nixos/medusa.nix#L14-L20
User profiles mainly configure the nixos user-manager module from this repo.
https://github.com/lukebfox/nix-configs/blob/60c64f14c2212b181fc2819c438b7cfe696bc86d/profiles/nixos/users/lukebfox/default.nix#L3-L10
The user-manager module implementation takes configuration and defines both the nixos and home-manager user configuration:
https://github.com/lukebfox/nix-configs/blob/60c64f14c2212b181fc2819c438b7cfe696bc86d/modules/nixos/user-manager/default.nix#L78-L104
The username in nixos/home-manager user configs comes from the attrname of the config e.g lukebfox (this is vanilla nixos/home-manager user module stuff). In the home manager developer profile I set some user specific git config:
https://github.com/lukebfox/nix-configs/blob/5432ca7dead6d39fca9b9772fc3c10914ba6969f/profiles/home-manager/developer/default.nix#L1-L7
https://github.com/lukebfox/nix-configs/blob/5432ca7dead6d39fca9b9772fc3c10914ba6969f/profiles/home-manager/developer/default.nix#L20-L25

If you want to use user data more often when writing home-manager configurations, you can add data to shared.toml, and then you can refer to shared.users.${config.home.username}.whatever when you are configuring whatever home-manager options you like.

Thank you for the detailed explanation. This helped out a lot.


Website agent refuses to create event if extraction is empty

Problem:
I have an XML that I scrape via xpath for a certain key. Sometimes that key/value doesn't exist, thus the website agent returns [] according to the log. The website agent is triggered by another event and is set to merge, so it will pass the incoming values to its own event.
Expected behaviour:
Create an event  when "merge" is set. Even if the exctraction is empty.
What I unsuccessfully tried so far to circumvent this behaviour:

scrape and additional (throwaway) value that is always present in the xml
Result: No event was triggered because one of the extractions returned []

scrape as text with regexp instead of xml/xpath. Use a template with {{ value | default: 'none' }} to return the extracted value.
Result: No event was triggered because regexp found nothing (as expected because the value doesn't exist). Default value also wasn't used.

{
  "expected_update_period_in_days": "1",
  "url": "fooredacted",
  "type": "xml",
  "mode": "merge",
  "extract": {
    "torrenturl": {
      "xpath": "/rss/channel/item[1]/link",
      "value": "string(.)",
      "hidden": "true"
    }
  },
  "template": {
    "thumb": "{{thumb}}",
    "pv_url": "{{pv_url}}",
    "name": "{{name}}",
    "altimage": "{{altimage}}",
    "torrenturl": "{{torrenturl | default: 'none' }}"
  }
}

Template is optional, I tried with and without. Only when the value is present in the xml an event is triggered.

After struggling for hours I finally managed to find a workaround via xpath:
Failing to use if/else in the xpath (which exist according to documentation) I used [this trick ] (https://stackoverflow.com/questions/971067/is-there-an-if-then-else-statement-in-xpath) but it wasn't easy to figure out how to use in in huginn. I hope this helps someone else with the same problem:
{
  "expected_update_period_in_days": "1",
  "url": "foo",
  "type": "xml",
  "mode": "merge",
  "extract": {
    "torrenturl": {
      "xpath": "/rss/channel",
      "value": "concat(substring(./item[1]/link, 1, number(contains(./item[1]/link,'h'))      * string-length(./item[1]/link)), substring('none', 1, number(not(contains(./item[1]/link,'h'))) * string-length('none')))",
      "hidden": "true"
    }
  },
  "template": {
    "thumb": "{{thumb}}",
    "pv_url": "{{pv_url}}",
    "name": "{{name}}",
    "altimage": "{{altimage}}",
    "torrenturl": "{{torrenturl}}"
  }
}

Take note of the "xpath" and "value". The key to success is to use an xpath that will always exist, even if the element further down the tree doesn't exist. This will trigger huginn to read the "value" which is also an xpath.
In the "value" we can then apply the if/else workaround.
Where you "split" the xpath is not important, you could use / too probably.
Regardless of my success, this is only a workaround. I still think the website agent should have an option to always create an event, even when empty.


TypeScript definitions

Looks good to me, thanks!

I am not a typescript user, but after some testing I found a few issues with these definitions:

Nothing.length = 2 // Cannot assign to 'length' because it is a constant or a read-only property. - it should be assignable though, this is related to all props
Nothing.anyProperty = {} // Type '{}' is not assignable to type 'INothing'. - any property should allow assigning to anything
(p1: any, p2: any, p3: any, ...): INothing - all these declarations can be replaced by (...args: any[]): INothing
Nothing().call() // Expected at least 1 arguments, but got 0. - should allow any number of arguments

As a summary, I think it's better to replace all TS declarations with something like this:
export declare const Nothing: any
export declare function toBool (p: any): boolean
export declare function isNothing (p: any): boolean
export declare function isSomething (p: any): boolean
export declare function serialize (p: object): string
export declare function deserialize (p: string): object

Otherwise, it's too restrictive and has a lot of edge-cases.

1-4 cases good catches and should be fixed.
export declare const Nothing: any

I think it's bad idea. any so weak. It's almost equal to case, where no definitions totally.
Better to pick Nothing as particular nominal type and operate with it.
export declare function toBool (p: any): boolean
export declare function isNothing (p: any): boolean
export declare function isSomething (p: any): boolean

This has the meaning for older TypeScript versions without conditional types. Now, with conditional types and Nothing as particular nominal type, you can achieve more robust behavior from compiler.

Ok, I investigate 1-4 problems:

It's native lib.d.ts TypeScript definition and can't be fixed:

interface Function {
    readonly length: number;
}

Interface INothing extended from Function interface, because it callable.

Seems TypeScript not support co/contr-variance of interface properties, like FlowType. It assume "mono-type" on input/output. In our case it INothing. This can be easy fixed via Nothing.anyProp = {} as any. So popular practice.

It's understand, ok.

This related to (1). Also lib.d.ts, thisArg required.

interface Function {
    call(this: Function, thisArg: any, ...argArray: any[]): any;
}

My opinion, a little embarrassing (2) case. But it's not problem for extended TypeScript user.
If it discouraged you, feel free to use own suggestion with any

Thanks for the explanation! I guess I'll do the change as the current definitions have too strict restrictions in some cases, IMHO.


Create cred_theft_suspicious_invoice_w_missing_or_image_only_attachme…

Description
new rule to detect messages using invoicing/payment language, mentioning an attachment and only image or no attachments found. also inspects the links for display text mimicking common invoice extensions and several NLU invocations.
Associated samples
Link to samples that are affected by your change.
For example, samples you are negating, samples you are including, or samples your new insight should fire on.

Sample 1
Sample 2

Associated hunts
If you ran any hunts with your rule, please link them here.

Hunt 1

Screenshot (insights)
For new insights only: Insert a screenshot of the insight firing. Remove this section if not applicable.

/update-test-rules


ci: cross compile the arm tarball for tests

This makes the ARM tarball ci test cross compile then run it on an ARM machine
close https://github.com/cilium/tetragon/issues/1955

Thanks @mtardy for the review, I fixed the most of them , except for the upload_path as it's not really an issue and it is easy for me to grep for it


Why MySQL

Could you please share reasons why you decided to use MySQL for this project?

I didn't have any specific reasons. I use MySQL/Postgres most of the time, so it seemed natural to include one of those.
For some time now I'm actually thinking about refactoring that part in a way that supports multiple database backends.
I also want to try Facebook's ent: https://github.com/facebookincubator/ent
For the time being it's MySQL only.

Thank you, solid response!
Not related, but could you please suggest how to implement jwt auth and protect some endpoints. Any references to the projects that implemented that are welcome.  micromdm project is good reference to get into go-kit, but i not found any jwt related code there. Btw they are using sqlx and squirrel and boltdb for some operations.

Go-kit actually comes with JWT package and docs about using it: https://github.com/go-kit/kit/tree/master/auth/jwt
Unfortunately I don't have an example at hand that actually uses it.

Pro tip: look for applications with go-kit topic on GitHub: https://github.com/topics/go-kit


Local Administration takes a long time to load on Apple Silicon

When opening the administration locally via http://localhost/admin, the administration just displays a white page for the first minute or so. After that, the administration is decently usable.
I couldn't figure out why this differs so much when using Apple Silicon, compared to Linux, where the administration is usable in an instant.
Specs:
MacBook Pro (16-inch, 2021)
Chip: Apple M1 Pro
Memory: 32 GB
I'm pretty sure this is a Mac problem, since I couldn't find any problems on the exact same Dockware setup on Linux or even Windows (running a subsystem).
I thought that this stems from xDebug, that's why I disabled it, but it didn't really help.
What do you need in order to investigate further?
Thanks in advance.
Edit: this occurred to me in the dev:<IP_ADDRESS> image, but has happened in earlier releases as well.

Hi
so I have good news, this is not a real problem, I'm also running it on a M1 in a really fast way :)
I think the origin might be due to bind mounting everything, could that be?
on a MAC you need to pay a bit more attention when doing this compared to Linux
here is a guide that should help you
https://docs.dockware.io/tips-and-tricks/how-to-use-bind-mounting
in general, an easy thing to figure out if that is the case, just completely turn off mounts and just "use" it.
if its then fast, then its 100% due to that mount
also...if its not because of that...i had very very weird lags since a longer time now in the admin
meaning the dropdowns were very slow and more...this also happend on server hosted administrations...
but it worked on SAFARI...so something in my chrome screwed it up...dont know what, but got a new Mac anyway, and now its working
i hope this helps

Thank you for your response.
This seems to have resolved.
I didn't really change anything, but all of a sudden it seems pretty fast.
Thank you.

Thank you for your suggestion. Yes, I experienced this several times, too. This seems to be a problem with Mac OS in general, though, not Docker in particular.
Either way, this issue seems to resolved and I hope this won't change any time soon.
Thanks again.

always happy to help :)


[anchors] HTLC update fee clamp

Mitigation for anchor channel attack brought up by @ariard in https://lists.linuxfoundation.org/pipermail/lightning-dev/2020-September/002796.html
We check that the maximum HTLC timeout fees paid (and can potentially be siphoned off as described in the attack), is always less than the channel reserve of the party. This should ensure that whatever state the channel currently is in, the party won't have an incentive to breach using this "siphoned state", since it can gain at most the channel reserve.
There is a case where the party still hasn't received a balance above the channel reserve (this is the case if the party is channel responder). But we can note that these "HTLC timeout fees" always comes from the offering party's balance, and the party cannot offer HTLCs as long as the balance is below the channel reserve.
With this change a party won't attempt to send an update_add or update_fee that would violate this check on the receiver side.
Since this new condition makes the channel close to unusable in certain fee conditions (small reserve, high fee rate, many HTLCs), we add a new option --max-anchor-commit-feerate that defaults to 10 sat/vbyte. This ensures the initiator won't ever try to send a fee update above this.
Since for anchor channels the  important thing is to get the channel to propagate, we could possibly default this to an even lower value.

However, I'm worried that this can cripple smaller channels that regularly make payments (or forward payments) that are above the local dust limit.

Yes, this can definitely cripple smaller channels, however it will only be for anchor channels, so we can expect no existing channels to be affected
A way to mitigate this before zero-fee 2nd level transactions, is to require channels to be of a certain size, or decrease the default max fee rate even more.
We can do a sanity check at channel opening, something like chanSize * reserve / (htlcFee*defaultMaxFeeRate) > X number of HTLC slots.

Lingering linter error:
contractcourt/htlc_success_resolver_test.go:87:15: unlambda: replace `func(c ContractResolver,
	r ...*channeldb.ResolverReport) error {
	return testCtx.checkpoint(c, r...)
}` with `testCtx.checkpoint` (gocritic)
		Checkpoint: func(c ContractResolver,

So thought about this a bit more, and I don't think we need it after all given:

The HTLC fee leak can only be exploited by the responder of a channel funding given it wants to use the fact that the initiator pays for fees to steal funds from them.
The attacker sits and waits until a commitment has a very high fee rate (say 150 sat/byte, etc), then later in the future breaches back to that state. This lets them siphon off the funds from HTLCs to themselves since the initiator is paying for fees.
With the current version of anchors we're shooting for, we'll always clamp the fee rate at ~10-20 sat/byte.
Therefore, assuming an lnd node is the initiator, they're protected from this attack, as they'll never create a commitment with a 500 sat/byte fee rate or w/e.

OK so I was wrong with that scenario above, in that ofc the fee for the HTLC output on the commitment is paid by the initiator, but the fees for the second level are taken from the HTLC value itself.

Replaced by https://github.com/lightningnetwork/lnd/pull/4855 and the new channel type introduced in #4840


Mac OS Compatibility

Is there anything stopping this from becoming cross-platform to Mac OS X and iOS? Seems it's mostly Foundation classes.

Probably not—honestly, I just hadn't had the time to make sure it would all work on OS X.


Register

@JuliaRegistrator register()

Registration pull request created: JuliaRegistries/General/99172
Tip: Release Notes
Did you know you can add release notes too? Just add markdown formatted text underneath the comment after the text
"Release notes:" and it will be added to the registry PR, and if TagBot is installed it will also be added to the
release that TagBot creates. i.e.
@JuliaRegistrator register

Release notes:

## Breaking changes

- blah

To add them here just re-invoke and the PR will be updated.
Tagging
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.0 -m "<description of version>" 5e825b4a6ec38e51c8951b9930afab075d1d90f6
git push origin v0.1.0


OpenType Tag padding

I discovered a strange discrepancy in the OS/2 achVendID value between a font generated through the fontmake toolchain (fontmake -> ufo2ft -> fontTools) and other font editors. When the vendor ID is shorted than 4 characters, fontTools pads it with null (0x00) bytes, whereas other editors pad the value with a space (0x20).
The OpenType specification has this to say on padding (emphasis mine):

A Tag value is a uint8 array. Each byte within the array must have a value in the range 0x20 to 0x7E. This corresponds to the range of values of Unicode Basic Latin characters in UTF-8 encoding, which is the same as the printable ASCII characters. As a result, a Tag value can be re-interpreted as a four-character sequence, which is conventionally how they are referred to. Formally, however, the value is a byte array. When re-interpreted as characters, the Tag value is case sensitive. It must have one to four non-space characters, padded with trailing spaces (byte value 0x20). A space character cannot be followed by a non-space character.

Based on that, I believe the fontTools behaviour to be incorrect. It appears the vendor ID is packed with the following settings:
	achVendID:     4s      # font vendor identification

The Python documentation for struct.pack explains that values are padded with null bytes:

For packing, the string is truncated or padded with null bytes as appropriate to make it fit.

There are a couple of other places in the code where this happens from what I can tell:

The sfnt table directory entries
The fvar tags
The sbix glyph data
The meta table

What is the preferred fix here (if any)? I could open a PR to fix just the vendor ID (by pre-padding the value with a space), but this feels like a larger problem.

Thanks. I think only the vendor id can ever be smaller than 4 bytes, the other tags are all 4 bytes so need no padding.
Yes please send a PR to pad vendor ID with trailing spaces

it's better if ufo2ft ensures the achVendID gets padded to 4 bytes with spaces. It's a easy fix, I can take care of it thanks

Thanks!


Remove extra double-dash in error message

Error message should read --main_process_port, not ----main_process_port.  Users who copy and paste the message as it was, will get this error message:
Accelerate CLI tool: error: unrecognized arguments: ----main_process_port

What does this PR do?
See above. It fixes a typo in the error message printed when there is a process port mismatch
Fixes # (issue)
N/A
Before submitting

[x ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you read the contributor guideline,
Pull Request section?
[ ] Was this discussed/approved via a Github issue or the forum? Please add a link
to it if that's the case.
[ ] Did you make sure to update the documentation with your changes? Here are the
documentation guidelines, and
here are tips on formatting docstrings.
[ ] Did you write any new necessary tests?

Who can review?
Anyone in the community is free to review the PR once the tests have passed. Feel free to tag
members/contributors who may be interested in your PR.
@muellerzr  may be interested as this is a CLI issue

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


I want to add or Remove Objects?

The app is hardcoded for 20 classes and for the tiny-yolo network final output layer. You can check the following code if you want to change this:
https://github.com/natanielruiz/android-yolo/blob/master/app/src/main/java/org/tensorflow/demo/TensorflowClassifier.java
The code describes the interpretation of the output.
The code for the network inference pass is written in C++ and the output is passed to Java. The output of the network is in the form of a String which is converted to a StringTokenizer and is then converted into an array of Floats in line 87 of TensorflowClassifier.java
You can work from there and read the papers to transform the new yolo model output into something that makes sense. (I did it only for one bounding box and also obtained the confidence of this bounding box). This part of the code is commented by me so you can understand what I did. Also read the paper here: https://arxiv.org/abs/1506.02640

Hi Nataniel Ruiz,
Thanks for your response.your app is so useful but i want to train it
according to my needs.where is dataset in the source code like(images of
these 20 objects)? I want to reduce its size as well.
Another main point the app crashes on many android device how we can fix
it. Looking forward.thanks
Best regards Haris
On 24 Mar 2018 5:09 am, "Nataniel Ruiz"<EMAIL_ADDRESS>wrote:
The app is hardcoded for 20 classes and for the tiny-yolo network final
output layer. You can check the following code if you want to change this:
https://github.com/natanielruiz/android-yolo/blob/master/app/src/main/java/
org/tensorflow/demo/TensorflowClassifier.java
The code describes the interpretation of the output.
The code for the network inference pass is written in C++ and the output is
passed to Java. The output of the network is in the form of a String which
is converted to a StringTokenizer and is then converted into an array of
Floats in line 87 of TensorflowClassifier.java
You can work from there and read the papers to transform the new yolo model
output into something that makes sense. (I did it only for one bounding box
and also obtained the confidence of this bounding box). This part of the
code is commented by me so you can understand what I did. Also read the
paper here: https://arxiv.org/abs/1506.02640
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/natanielruiz/android-yolo/issues/31#issuecomment-375829356,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Ag78H0m6XbHDG7SgsC69YGjQAdm_4lYSks5thY6qgaJpZM4S4_sN
.


ActionEvent emission

What problem does this solve?
It would avoid doing a state check in each system to know the current state of the action (just_pressed(), just_released(), pressed for a certain duration, ...).
I don't think it is ideal to have the trigger conditions scattered around your code, particularly if you want the player to be able to configure it in the options of your game.
What solution would you like?
The first idea that comes to my mind is listening to `EventReader<(Entity, Actionlike)> instead of fetching and checking the ActionState and its owner in each system.
How could this be implemented?
We could configure under what circumstance the event is triggered (PressedFor(seconds), JustReleased, PressedTimes(nth), ...). Maybe at the same place we configure the mapping but I'm not sure about that.
I had in mind a struct looking like this :
struct ActionEventConfig<A: Actionlike> {
    action: A,
    trigger: ActionTrigger
}

enum ActionTrigger {
    Pressed,
    JustPressed,
    Released,
    JustReleased,
    PressedForSeconds(f32),
    PressedTimes(i32)
}

The system emitting the events iterate through the Actionlike::variants() and their state then emit if the condition is met.
What alternatives have you considered?
As of today, I am fetching the action state of a given entity and reacting to it in many systems, so it's feasible.

We could optionally enable emitting events via observers these days.


Frequent carbonlink errors

I am seeing a number of errors similar to the following:
[2015-11-19 13:56:51] D [carbonlink] read carbonlink request from <IP_ADDRESS>:32800: Can't read message length: read tcp <IP_ADDRESS>:7002-><IP_ADDRESS>:32800: i/o timeout
[2015-11-19 13:56:51] D [carbonlink] read carbonlink request from <IP_ADDRESS>:32998: Can't read message length: read tcp <IP_ADDRESS>:7002-><IP_ADDRESS>:32998: i/o timeout
I assume this is my fault. Can you guide me any on how to fix this issue?

I am getting same error. @johnseekins Have you fix this ?

I have not. For the most part, I've just been ignoring them since my
install is small enough I don't completely need the carbonlink data.
Are you also running graphite-web 0.9.15?
On Fri, Dec 18, 2015, 4:54 PM Wei Wu<EMAIL_ADDRESS>wrote:

I am getting same error. @johnseekins https://github.com/johnseekins
Have you fix this ?
—
Reply to this email directly or view it on GitHub
https://github.com/lomik/go-carbon/issues/18#issuecomment-165922959.

No, I am trying to use Graphite API 1.1.2 instead of Graphite Web.

This is not error and not problem, error level is "debug". go-carbon closing old unused connections from graphite-web after 30 seconds timeout.

Set log-level = "info" and forget about this message :)

That's certainly a solution...
But maybe you could make the message a little more clear? Perhaps don't tag it as an error in the log?

Excuse me...it isn't tagged as an error. But maybe the message could say something about disconnecting, rather than simply "i/o timeout".


Sum of time dead in performance tab

Not really a bug per se but in the performance section, the summation of a team's total "time dead" is displayed as the total number of seconds dead. I feel like it would make more sense if it was in MM:SS format.

fixed in #1129


is support on windows?

error[E0432]: unresolved import std::os::fd
--> E:\rust.cargo\registry\src\rsproxy.cn-0dccff568467c15b\foyer-storage-0.3.0\src\device\fs.rs:17:9
|
17 |     os::fd::{AsRawFd, BorrowedFd, RawFd},
|         ^^ could not find fd in os

Hi @hmfms , foyer is aimed to support both unix-like system and windows. But it is still under development now. So the support on Windows may not be fullfilled now.
Let me mark this issue as a bug, and I'll fix it later.

recently , I want test it on Windows , Do you have a schedule to solve it ?

recently , I want test it on Windows , Do you have a schedule to solve it ?

Sorry, @hmfms . I cannot give a fixed schedule for it now. Currently, I'm working on the in-memory cache component of foyer, which is also part of my task of my full-time job. Plus, I don't have a windows environment now.
I'll fix the issue after the in-memory cache is merged. (And it will happen soon for I've finish the PR, test and received approves, FYI #270 and related PRs). I'll setup a windows environment with my PVE to fix the bug on Windows.

And I think I need to setup a CI pipline on Windows and MacOS. Let me open an issue to track.

Hi, @hmfms . Now I'm investigating the build on Windows with my PVE. I've not been developed on a Windows machine for years and never write Rust on Windows. May I know what's your setup of your Rust environment and how I like to develop with foyer?


Fix Testimonial stories with flexible container

Summary
Fixes Testimonial stories to properly enable the component to reflow on smaller viewports.
List of notable changes:

Ensure width is set on wrapper div and set max-width: 100%

Supporting resources (related issues, external links, etc):

Fixes https://github.com/github/primer/issues/3777

Contributor checklist:

[ ] All new and existing CI checks pass
[ ] Tests prove that the feature works and covers both happy and unhappy paths
[ ] Any drop in coverage, breaking changes or regressions have been documented above
[ ] New visual snapshots have been generated / updated for any UI changes
[ ] All developer debugging and non-functional logging has been removed
[ ] Related issues have been referenced in the PR description

Reviewer checklist:

[ ] Check that pull request and proposed changes adhere to our contribution guidelines and code of conduct
[ ] Check that tests prove the feature works and covers both happy and unhappy paths
[ ] Check that there aren't other open Pull Requests for the same update/change

Screenshots:

Please try to provide before and after screenshots or videos

 Before  After

@joshfarrant Agree. We should review and update these stories on a follow up PR.


SVG file thumb is invisible

The SVG just does not display anymore since their  tag does not define any size attribute and dont have any size in CSS, resulting in an image of 0x0 pixels.
This may not occurs with all SVG but not sure. However it sure indeed reproduced with generated images from demo, which use common options so this is to fix.
We need to enforce a specific size to ensure every image formats is correctly displayed.

Done in 0.3.0-pre.2


Control caching behavior of contract_type and proxy_info on live networks [APE-911]

Overview
When fetching contract data from a block explorer on a live network, the results are cached in .ape/{ecosystem_name}/{network_name}/contract_types/{address}.json, which is very helpful if you intend to use the contract object again.
However for contracts instantiated with a limited ABI (a minimal ERC-20, for example), the cache remains and must be manually removed and re-fetched to restore full ABI functionality, or to clear storage space.
Specification
An existing JSON file may be passed as the contract_type using Contract(address,contract_type=ContractType.parse_file(path)) where ContractType is imported from ethpm_types
The caching behavior could be controlled with an optional argument cache that defaults to True so existing scripts behave as expected.
Example call that would disable caching to storage:
Contract(address=POOL_ADDRESS, contract_type=ContractType.parse_file(FILE_PATH), cache=False)
Dependencies
None

I will redo the PR after reviewing CONTRIBUTING.md and reformatting

contract_type already overrides whatever the cache has (I believe), so we can add a kwarg ignore_cache or update_cache (boolean defaulted to False) that can be set true to force re-downloading the type via explorer. I think this will improve the UX around purging the cache value in cases where it's not set correctly
Alternatively, there could be some algorithm that merges what's in the cache with whatever the explorer provides (if available), or whatever a user provides via contract_type input. This could be used to solve some of the issues with proxies where the proxy has additional methods from the implementation contract (such as upgradeable proxies)

I had not considered the case of a forced update, I will work on this and wrap into the next PR.

I had not considered the case of a forced update, I will work on this and wrap into the next PR.

Thanks for taking this on!


[YUNIKORN-126] Add execute permission to executables

@kingamarton so you changed execution permission for all scripts here?

@kingamarton so you changed execution permission for all scripts here?

I set executable permission for  build-docker-images.sh and build-release.py and removed the execute permission from LICENSE and NOTICE files

https://github.com/apache/incubator-yunikorn-release/pull/2#issuecomment-630121812
Thanks @kingamarton
Apart from the comment from @adamantal , I dnt have any other comments.
+1 barring that point.


[FLINK-7748][network] properly use the TaskEventDispatcher for subscribing to events

What is the purpose of the change
ResultPartitionWriter currently implements the EventListener interface and is used for event registration, although event publishing is already handled via the TaskEventDispatcher.
Instead of using two different places, this should be unified by using TaskEventDispatcher only which is done by this PR.
~Please note that this PR builds upon #4759.~
Brief change log

make TaskEventDispatcher more generic to register result partitions via ResultPartitionID with one TaskEventHandler per partition (handled by TaskEventDispatcher, not inside ResultPartitionWriter)
remove the EventListener<TaskEvent> implementation from ResultPartitionWriter
add the TaskEventDispatcher to the Environment information for a task to be able to work with it (only IterationHeadTask is using this as of now)
adapt all places to use TaskEventDispatcher instead of ResultPartitionWriter

Verifying this change
This change added tests and can be verified as follows:

a new TaskEventDispatcherTest verifies that TaskEventDispatcher works as expected
indirectly, tests under org.apache.flink.runtime.iterative may verify the use of the IterationHeadTask which is the only user so far

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (JavaDocs)

Could you resolve conflicts before I start reviewing?

alright - didn't even need to do anything here since git auto-merged them :)

merging.


Unable to launch the Mainframe Terminal via RobotFramework on Mac

Hello Team, need one help running Mainframe scripts on Mac platform…i am using MacOS Monterey(v 12.2.1) and want to run my Mainframe automation script created using RobotFramework keywords…the same script is perfectly working fine on Windows machine…in my Mac, I have installed Xquartz (https://www.xquartz.org/), now when I run the script, it is throwing error of not able to launch the terminal…can anyone let me know what extra settings needs to be done please?

If it works on Windows and not on MacOS, it's clearly not a problem with RF itself.


first draft

@cryptodamsky I need to add more- is this a good start and what else should I explain?

ignore, deleted yarn.lock by accident

I understand it correctly that this page is not yet linked from the main page/navbars?


transferMyPlayback not working

According to the type definition of the TransferPlaybackOptions one of the arguments is called device_ids, but in the implementation the argument is called deviceIds. So changed the function transferMyPlayback so that the argument is called device_ids too.

Hm, I actually don't see the issue here. The options properties are camelcased in the implementation, but sent to Spotify's API in snake case. In this case, the wrapper wants options.deviceIds and translates it to device_ids. Similar translations are done for other functions, like snapshotId to snapshot_id or even trackIds to ids. Please reopen if you think I'm incorrect here.
Did notice that the documentation seems a bit off though. And since device_ids is actually required for the endpoint, the wrapper should accept it as its first argument.


[Inspector] Prevent crashing when a mesh's name is of the wrong type.

A user can mistakenly set a mesh's name with the wrong type (not string), like this example: https://playground.babylonjs.com/#XT7L5S, where trying to open the Inspector makes it crash. This PR just adds some extra type checking to avoid it crashing, and also adds a quick check to warn the user when adding to a scene.

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Snapshot stored with reference name:
refs/pull/13893/merge
Test environment:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/13893/merge/index.html
To test a playground add it to the URL, for example:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/13893/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/13893/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/13893/merge
https://gui.babylonjs.com/?snapshot=refs/pull/13893/merge
https://nme.babylonjs.com/?snapshot=refs/pull/13893/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/13893/merge#BCU1XR#0

I am not sure we should do this. This is adding extra code and why doing it here and not for all the other APIs we have ?

This is not the first time this problem has been reported to us and it's not always easy to find that it comes from a bad type in the mesh names, so dealing with the problem on our side would be nice, but as you say, why do it only for this one and not for everything else that could go wrong... It's a bug on the user side, after all, not in Babylon.js. So, after a second though, I would also vote not to do it.

I don't see a warning in the scene as something critical, but I do think the solution for the inspector is important.
This happens from time to time. We can stay defensive, at least in our tools. This is obviously not "our issue", but warning the user they did something wrong is not a bad thing :-)
If you feel it's not the right place, maybe remove the scene changes. The playground changes should stay, as this breaks the inspector.

Love the solution of doing the inspector only :-)


SyntaxError: Invalid or unexpected token in nextjs

hello sir
I'm trying to use React-Numeric-Keyboard in nextjs and it fails with the message

i have checked backspace.png and saw that the file was invalid
can you help me fix this issuse?

Hey there,
Can you provide the code that you are using that you are getting this error?

here sir i use npm create next-app to setup project and npm install react-numeric-keyboard to try to use NumericKeyboard
`import Head from 'next/head'
import styles from '../styles/Home.module.css';
import { NumericKeyboard } from 'react-numeric-keyboard';
export default function Home() {
return (

Create Next App

      Welcome to Next.js!

    <NumericKeyboard isOpen={true} onChange={(val)=>{console.log(val)}}  />

)
}`

This might me the known issue in nextjs. please visit this link:
https://github.com/vercel/next.js/discussions/12410
@song23284


Das_Geek: Watch bookatrekking.com

Das_Geek requests the watch of the watch_keyword bookatrekking\.com. See the MS search here and the Stack Exchange search here.

bookatrekking\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Makyen in Charcoal HQ


Why is the bot offline?

On Discord, the bot is offline for no reason.

Why?

can you just stop going online and offline for 3 hours for no reason. please go auto.

The server it's being ran on isn't in my control, literally he disconnects it from the internet whenever he plays games because it causes lag and will often forget to plug it back in. Once I have an actual hosting solution the uptime will be more consistent and I'll push a big-ish update to VEB I have saved


Add German translation

@schwemmer, danke :smile:

@thomaspark, gern geschehen (you're welcome)! Thanks for the great project!


Use NugetDownloader in the Nuget Catalog

Nuget Catalog currently contains all the logic for communicating with the Nuget feeds. We should remove this logic and instead use NugetDownloader.

PR: https://github.com/weikio/PluginFramework/pull/38


Review and update Ember code in "Modern web app architecture" series

See https://bugzilla.mozilla.org/show_bug.cgi?id=1463837 for full details.

See also #382


Simplify synchronized syntax

Motivation:
We often used synchronized(this) while the whole method was synchronized, which can be simplified by just mark the whole method as synchronized.
Modifications:
Replace synchronized(this) with synchronized on the method
Result:
Cleaner code

@nmittler @trustin @Scottmitch FYI ... just some simplification

LGTM

Cherry-picked into 4.0 (bdea94d80763e6ec1c9dcf51dc9ed8098197d760)  and 4.1 (bf2a99518c3ebc484464fc35279452ddd24c0ea9)


Subsequent apply runs cause subnet and vpc tags to get resetted

I'm submitting a...

[x] bug report

What is the current behavior?
When running terraform apply after initial cluster setup, it resets the subnet annotations. There is a plan run illustrating this:
  ~ module.eks.module.vpc.aws_subnet.private[0]
      tags.%:                                  "2" => "1"
      tags.kubernetes.io/cluster/xx-default: "shared" => ""

  ~ module.eks.module.vpc.aws_subnet.private[1]
      tags.%:                                  "2" => "1"
      tags.kubernetes.io/cluster/xx-default: "shared" => ""

  ~ module.eks.module.vpc.aws_subnet.private[2]
      tags.%:                                  "2" => "1"
      tags.kubernetes.io/cluster/xx-default: "shared" => ""

  ~ module.eks.module.vpc.aws_vpc.this
      tags.%:                                  "2" => "1"
      tags.kubernetes.io/cluster/xx-default: "shared" => ""

If this is a bug, how to reproduce? Please include a code sample if relevant.
Deploy cluster, run plan again.
What's the expected behavior?
It should either set those itself or ignore the attribute.
Environment details

Affected module version: 2.3.1
OS: linux
Terraform version: v0.11.11

I think you need to add the tag tags.kubernetes.io/cluster/xx-default to your module at module.eks.module.vpc?

Ah yeah, that should solve it. Thanks again!

Thanks for this answer. But I'm not very clear as how to implement this. I've my infra structure as below

main.tf
vpc.tf (Uses terraform-aws-vpc module to create vpc and other networking stack)
eks.tf (Creates eks cluster).

Re running terraform apply removes tag kubernetes.io/cluster/${var.name} = "shared". I don't follow what you mean by

I think you need to add the tag tags.kubernetes.io/cluster/xx-default to your module at module.eks.module.vpc?

Can you please share one example.

Can you please share one example.

There are examples in this repo, for example here: https://github.com/terraform-aws-modules/terraform-aws-eks/tree/master/examples/basic


I have another equal sign on a line after the first and syntax highlighting is kind of broken

See the screenshot:

version 1.0.0

I think there is no issue with the library, @capaj you should use quotes

Dotenv works fine without having quotes. Quotes are not required in this case. This library should handle same use case that dotenv does.


Slather unable to find coverage files?

I've tried to set up two different projects with Slather today (https://github.com/lazerwalker/MTLParseAdapter and https://github.com/intentkit/IntentKit – the former is much smaller/simpler, if you're looking for a test case). Both are failing for the same reason:
/Library/Ruby/Gems/2.0.0/gems/slather-1.7.0/lib/slather/project.rb:49:in `coverage_files': No coverage files found.
Are you sure your project is setup for generating coverage files?
Try `slather setup your/project.pbxproj`
(StandardError)

Both of these are failing both locally and on Travis (e.g. https://travis-ci.org/lazerwalker/MTLParseAdapter/builds/59264653).
Poking around existing issues, it seems that's common if you've manually changed where the gcda files get placed. However, I haven't changed that for either of these projects. It also doesn't appear to be a local system-wide Xcode config issue, as I have other projects with similar setups that will let themselves be Slathered just fine.
Any thoughts?

@lazerwalker
Your source_directory must be a directory of files that is included in your xcodeproj directly.
As of now, they are internal Pods that are bundled in your workspace.
(https://github.com/lazerwalker/MTLParseAdapter/blob/master/.slather.yml#L4)
You may need to reorganize your project so that your source files have their own framework and tests, that is separate from your example project.

Ah, gotcha. Thanks!
Might be worth massaging that error message a bit – I'd assume it wouldn't be hard to manually check the case where no coverage reports are generated, but the specified Xcode project has been set up to generate coverage.

Actually, I think I initially misguided you. You can use local Pods, but you need to add plugin 'slather' to your Podfile.

Hah, I was literally in the middle of trying that for IntentKit when I saw your comment. Yeah, just adding it to the Podfile works like a charm.

Sorry about that! Glad you got it working :star2:

For anyone from The Future following along: using Cocoapods 0.36.4, one of my projects needed me to explicitly tell Travis to gem install activesupport to stop it from blowing up when it tried to interpret the Podfile plugin line (see https://github.com/CocoaPods/CocoaPods/issues/3076)