Creating directories in explode_init takes 15min

Might be nice to parallelise this and add progress

On very large dataset

Yes sorry, should have been specific - this is when I have ~4000 partitions, resulting in about 150,000 directories.

Did you have 4000 workers on this??? (See #89)

This should be much improved in latest version btw - only the top level dirs are created by init, the partition dirs are made as part of explode-partition


Add workflow for GH Actions

@gwynne doing this as a quick fix for now, then we can investigate what's going on later


feat(new): Added Azure.Firewall.AvailabilityZones

PR Summary
Fixes #2909
Added Azure.Firewall.AvailabilityZones.
PR Checklist

[x] PR has a meaningful title
[x] Summarized changes
[x] Change is not breaking
[x] This PR is ready to merge and is not Work in Progress
Rule changes

[x] Unit tests created/ updated
[x] Rule documentation created/ updated
[x] Link to a filed issue
[x] Change log has been updated with change under unreleased section

Other code changes

[ ] Unit tests created/ updated
[ ] Link to a filed issue
[ ] Change log has been updated with change under unreleased section

Thanks for the great feedback, @BernieWhite! Both the static method and the functions you created earlier were very helpful in developing this rule.
I’ve made some updates based on your input. Could you take a look now and let me know your thoughts?

@BernieWhite,
We pushed at the same time and I pushed with force, saw the push notification from GitHub now.
Can you push it again?

@BenjaminEngeset thanks for the updates. I made a few tweaks and added a test case for a region without AZ support. If you are happy with the changes we should be good to merge.
Great work!

Looks good! Let's merge.


ISPN-11532 Replace HotRod worker thread pool

Revert test change as the netty event loop is no longer replaced

https://issues.redhat.com/browse/ISPN-11532

Merged, thanks Will!


Add Patent Entry

A new Patent Entry would be helpful. It would be similar to the Publication Entry but would have the following fields:

Status (Pending or Approved)
Year submitted/accepted
Application/Patent number

RenderCV's entry types are not data-oriented but rendering-oriented. We would only want to create a new entry type if we require a new rendering type.
You can showcase your patents with one of the seven available entry types. Maybe NormalEntry would be the best?
I am open to discussing any proposals for new rendering types if you have suggestions.


BA - Expensify crashes when entering test transactions

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Go to staging.new.expensify.com or start Expensify App
Login with Expensifail account
Create new or go to existing workspace without BA
Go to Workspace > Connect bank account
Select the Manual method to add a bank account
Follow the flow to connect Pending State Bank account' until 'Test transactions' page
Enter any numbers in the test transaction fields

Expected Result:
User can enter test transactions
Actual Result:
App / Website crashes
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

Version Number: 1.3.56-2
Reproducible in staging?: Yes
Reproducible in production?: n/a
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
logs.log
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/78819774/98280ece-8f49-4381-ab57-fb9eabdacf97
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub
logs.log

:wave: Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open StagingDeployCash deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

Identify the pull request that introduced this issue and revert it.
Find someone who can quickly fix the issue.
Fix the issue yourself.

well, it doesn't re-trigger auto-assignment...
anyway, per this slack discussion, I'm assigning myself
gonna find someone in #engineering-chat

I'm trying to verify that this is not already broken in production...
Meanwhile, this is the error:

Apparently we call Validation.filterInput with all values from the form, and that include some boolean values. The function crashes when we pass true to it.
I think the function crashing may be because of this PR:
https://github.com/Expensify/expensify-common/pull/567
cc @Li357

QA passed:
https://github.com/Expensify/App/assets/87341702/1a86521a-d27d-4205-aaff-5f00c7feb36b


add use eds cache run time flag

This allows envoy to use eds cache when control plane response times out and helps to finish warming without a eds update after cds update

[ ] Ambient
[ ] Configuration Infrastructure
[ ] Docs
[ ] Dual Stack
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

@ramaraochavali: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
77546a52fee58bbc66a92141f21a7211dcc08773
link
true
/test release-notes

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.

It is worth adding a release note

@ramaraochavali: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-telemetry-istiodremote_istio
128b47bae4cfe53c3d48bcd7239d7492e8bf263f
link
true
/test integ-telemetry-istiodremote

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.

@ramaraochavali: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-telemetry-istiodremote_istio
128b47bae4cfe53c3d48bcd7239d7492e8bf263f
link
true
/test integ-telemetry-istiodremote

integ-telemetry_istio
128b47bae4cfe53c3d48bcd7239d7492e8bf263f
link
true
/test integ-telemetry

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.

@ramaraochavali: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-telemetry-istiodremote_istio
128b47bae4cfe53c3d48bcd7239d7492e8bf263f
link
true
/test integ-telemetry-istiodremote

integ-telemetry_istio
128b47bae4cfe53c3d48bcd7239d7492e8bf263f
link
true
/test integ-telemetry

integ-telemetry-discovery_istio
128b47bae4cfe53c3d48bcd7239d7492e8bf263f
link
true
/test integ-telemetry-discovery

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.

/retest

@ramaraochavali: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-telemetry_istio
128b47bae4cfe53c3d48bcd7239d7492e8bf263f
link
true
/test integ-telemetry

integ-telemetry-discovery_istio
128b47bae4cfe53c3d48bcd7239d7492e8bf263f
link
true
/test integ-telemetry-discovery

integ-telemetry-istiodremote_istio
128b47bae4cfe53c3d48bcd7239d7492e8bf263f
link
true
/test integ-telemetry-istiodremote

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.

@ramaraochavali: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-telemetry-istiodremote_istio
128b47bae4cfe53c3d48bcd7239d7492e8bf263f
link
true
/test integ-telemetry-istiodremote

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.

/test integ-telemetry-istiodremote

need https://github.com/istio/istio/pull/48874

@ramaraochavali: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-telemetry-istiodremote_istio
128b47bae4cfe53c3d48bcd7239d7492e8bf263f
link
true
/test integ-telemetry-istiodremote

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.

/test integ-telemetry-istiodremote


Add support for a Pushover alerter

Adding support for notifications via https://pushover.net/.

docker_run.bash's command line arguments are getting pretty complicated.. how do you feel about requiring users to configure this type of alerter via a YAML alerter config?

docker_run.bash's command line arguments are getting pretty complicated.. how do you feel about requiring users to configure this type of alerter via a YAML alerter config?

docker_run.bash's command line arguments are getting pretty complicated.. how do you feel about requiring users to configure this type of alerter via a YAML alerter config?

I think it would make sense to restrict alerter configuration to a yaml config 🤔The YAML config could be made generic, something like:
alerters:
  - name: pushover
    configure:
      - api_token: APITOKEN
      - user_key: USERKEY

You could also auto-generate documentation by dynamically loading the concrete alerters, and creating a yaml template based off the required kwargs.

I would find use out of this too. I was able to replicate the changes from kylemart:kylemart/pushover-support into a local copy (have yet to create a fork) I made of EricJMarti:main while avoiding conflicts (such as adding new args getopts to docker_run.bash for user_key and api_token, I used "u" and "y", respectively).
I was not able to get this to run with either inline arguments or using the alerters.yaml file. In both cases, I'm getting an:
Exception: the "pushover" alerter type does not exist in the registry
Is there something obvious I'm missing? Would love to help implement this feature if possible (though I'm a bit elementary in Python and YAML).
Happy to create a fork and point to the code I have for easier debugging. Though, I mainly implemented @kylemart's existing work here and just changed some things around to avoid conflicts.


Update Docker Image

The Docker image you provide has been quite useful. However it seems to be a bit behind compared the releases.
Would you mind updating it?
Or do you recommend people just use it as a base in building their own Dockerfile. If so, I think the docs should be updated to suggest that route.

Sure! I started it as kind of a toy when I began learning about Docker, and have let it slide since. The inexperience really shows (running nginx and uwsgi inside the same container? Gross!). I'll trigger some new builds now and also look into refactoring the format to be more in-line with what you would expect from a docker container.

If you were to constrain the docker configuration to an S3 store/CDN setup would there be much static file handling left for PyPI? If that is the case you could consider dropping nginx all together and exposing uWSGI directly.

Okay, I've refreshed the setup and moved all the docker files to be in a separate repo over at https://github.com/stevearc/pypicloud-docker so that the images can be updated in a way that isn't tied to the pypicloud release cycle. Let me know if there's anything else you'd like to see from a pypicloud docker image!

Great, I hope to try this out this week. Thanks!

Worked well for me.


The timestamp of xml, report and log is equal

Environment:
OS: Windows10
Robot Framework: Robot Framework 3.1.2 (Python 2.7)
Description:
When ride finish the execution the log, the report and the xml files have the same timestamp for all files. The 3.0.4 version set the timestamp with end of execution to report and log.
Output: C:\LogsRide\output-20191113-155339.xml
Log: C:\LogsRide\log-20191113-155339.html
Report: C:\LogsRide\report-20191113-155339.html

This was explicitly fixed by #2945. Do you have reasons why the old behavior was better?

No reasons why we should return to the old behavior provided. Closing.


Breguet Range Updates

is this ready for review, or in progress?

In progress want to clean a few more things up

@whoburg ready for a review. I made a vectorized SP model that is a little more detailed and has the transonic drag rise you made for D8 as well as new GP model.

also, I forgot exactly what data you fitted for the transonic drag rise. was it xfoil data? I assumed ti was, so I kept a small profile drag coefficient for the fuselage and used parabolic drag model to account for induced drag


Should RG accept ellipse params as well as an ellps=?

While testing various output from projinfo I've come across an example where the ellipse is described in terms of rf and a parameters instead of ellps=<name>. For example a transformation between EPSG:2001 and EPSG:3857 looks like:
+proj=pipeline
  +step +inv +proj=tmerc +lat_0=0 +lon_0=-62 +k=0.9995 +x_0=400000 +y_0=0 +a=6378249.145 +rf=293.465
  +step +inv +proj=longlat +a=6378249.145 +rf=293.465 # Note: I've implemented longlat as a noop in geodesy-wasm.
  +step +proj=push +v_3
  +step +proj=cart +a=6378249.145 +rf=293.465
  +step +proj=helmert +x=-255 +y=-15 +z=71
  +step +inv +proj=cart +ellps=WGS84
  +step +proj=pop +v_3
  +step +proj=webmerc +lat_0=0 +lon_0=0 +x_0=0 +y_0=0 +ellps=WGS84

# With the equivalent RG pipeline being:

  | tmerc inv lat_0=0 lon_0=-62 k=0.9995 x_0=400000 y_0=0  a=6378249.145 rf=293.465
  | noop inv a=6378249.145 rf=293.465
  | push v_3
  | cart a=6378249.145 rf=293.465
  | helmert x=-255 y=-15 z=71
  | cart inv ellps=WGS84
  | pop v_3
  | webmerc lat_0=0 lon_0=0 x_0=0 y_0=0 ellps=WGS84

When converting the same coordinate with PROJ and RG I get differences in the order of 10's of meters. The a=6378249.145 rf=293.465 parameters describe the clrk80 ellipse, so if I update both pipelines to use ellps=clrk80 instead of using a and rf the results are the same.
The question is, should RG be able to handle ellipse input via the rf and a parameters? If yes, I'm happy to attempt a PR with some guidance on where to do this and what else needs to be covered.

Ok thanks for the guidance @busstoptaktik. I'll have a think and give it a whirl as time permits.


Problem with health check repository - invalid author/committer line - missing space before email

Gitea version (or commit ref): 1.13.6
Git version:
Operating system:

Database (use [x]):

[x] PostgreSQL
[ ] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[ ] No

Log gist:

Description

I have an old mirror on my Gitea instance. This mirror was converted to local repo. But after update to Gitea 1.13.6 (???) I get this error messages
Failed to health check repository (cache/libmemcached-old): exit status 4 - error in commit aaa58bfca87a6e47d11c4efbc5f0c0637091c5bb: missingNameBeforeEmail: invalid author/committer line - missing space before email
error in commit 237c14119ce4a784a0cbeba4cf674c4af8aa314d: missingNameBeforeEmail: invalid author/committer line - missing space before email
error in commit 3f2aab5c24ea76d37d4c1ef59f2277d1ea2d265e: missingNameBeforeEmail: invalid author/committer line - missing space before email
error in commit c41e93402bbafd6a9393c0589b8b01650049749f: missingNameBeforeEmail: invalid author/committer line - missing space before email
error in commit 4aac578b259d8ce082a9de428636799da1add598: missingNameBeforeEmail: invalid author/committer line - missing space before email
error in commit 5fc74e1abc81b936f089556ba8edd686846173f9: missingNameBeforeEmail: invalid author/committer line - missing space before email
error in commit 7df00ced05e7e4b447009c77704acbe9bda24a98: missingNameBeforeEmail: invalid author/committer line - missing space before email
error in commit c6f7a55231552beca528c83d0999988bceaa70c1: missingNameBeforeEmail: invalid author/committer line - missing space before email

Screenshots

What's the log from? It seems it's not Gitea's log.

I see this in Gitea

grep -R missingNameBeforeEmail /home/andrew/src/go/gitea
returns nothing.
Gitea is not doing this.

This is from Git. There is an invalid commit in this repository. Gitea will not have made this commit.

@zeripath as I wrote I get this since  Gitea 1.13.5/6. It's old mirror.
This issue can stay closed.


CMake version

Hi,
I was trying to compile with CMake 3.5.2, but I got this error:
CMake Error at CMakeLists.txt:69 (list):
  list does not recognize sub-command FILTER

Apparently FILTER was introduced in version 3.6, but according to the CMakeLists.txt file, only version 2.8 is required.
https://github.com/google/marl/blob/master/CMakeLists.txt#L15
Can we remove FILTER to make this work with older versions of CMake?

Hi @facontidavide,
Thank you for your feedback. I've put together a change that should fix this (#16).
Please can you check to see if this works for you?
Many thanks,
Ben


Docs fixes

keywords
architecture graphic
broken "edit this page" links

Pull Request Test Coverage Report for Build 7f3ea6a8ac9d11f32ac757be1d167dc4495ca2b6-PR-911

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.06%) to 67.457%

Files with Coverage Reduction
New Missed Lines
%

apps/fz_http/lib/fz_http/release.ex
1
93.75%

Totals

Change from base Build be565a6897611875f99f38fa009ad0ec0f8e8cad:
-0.06%

Covered Lines:
1053

Relevant Lines:
1561

💛  - Coveralls


Double reports if the two sensors are updated at different times

I am occasionally seeing double reports from the component within a few milliseconds of each other.
I wonder if this is caused by the 2 sensors being updated at slightly different times?
Does the component wait for both sensors to be updated before doing the calculations?

@dckiller51 I have worked with the author of the Passive BLE Monitor integration such that the weight and impedance sensors update almost simultaneously.  https://github.com/custom-components/ble_monitor/issues/617
However, I am still getting 2 updates of this component. All I can think of is that the few milliseconds between the 2 sensors update is sufficient to trigger the 2 updates.
Can a delay be added in while the component waits for both sensors to update?

Currently the component subscribe for states updates for each sensor.
On each sensor update all values are calculated.
https://github.com/dckiller51/bodymiscale/blob/d5c03bffad048fb431af9485fd58173e7f385082/custom_components/bodymiscale/__init__.py#L237-L244
To avoid the second report, we need to change how we subscribe. Probably introduce a delay

Precisely and you cannot calculate the correct details until both the weight and the impedance are available.


https://github.com/pixie-io/pixie/blob/main/DEVELOPMENT.md - issue with the cloud secrets

Describe the bug
Need addition to the documentation. Following the steps blindly doesn't work. After running the following command:
./scripts/deploy_cloud_prereqs.sh plc-dev dev
getting the following error:
Credentials path "/private/credentials/k8s/dev" does not exist. Did you slect the right secret type?
To Reproduce
Steps to reproduce the behavior:

Checkout pixie repo
Follow the steps mentioned here: https://github.com/pixie-io/pixie/blob/main/DEVELOPMENT.md
Under the step titled 'Load the config maps and secrets.', run the command:
./scripts/deploy_cloud_prereqs.sh plc-dev dev
We get the following error:
Credentials path "/private/credentials/k8s/dev" does not exist. Did you slect the right secret type?

Expected behavior
The setup steps should work seamlessly.

Thanks for reporting this! We've just updated DEVELOPMENT.md with more up-to-date instructions for how you can run the dev version of cloud using Skaffold.


[Bug] Disabling entry filters isn't possible when allowOverrideEntryFilters=true and entryFilterNames is specified in broker.conf

Search before asking

[X] I searched in the issues and found nothing similar.

Version
3.0.x, master
Minimal reproduce step
Disabling entry filters isn't possible when allowOverrideEntryFilters=true and entryFilterNames is specified in broker.conf.
What did you expect to see?
It should be possible to disable entry filters completely at topic level when allowOverrideEntryFilters=true
What did you see instead?
The entryFilterNames in broker.conf are used when the list of entry filters for the topic is empty. Please see the code:
https://github.com/apache/pulsar/blob/50b9a93e42e412d9f17b1637287d1a4c7c7ab148/pulsar-broker/src/main/java/org/apache/pulsar/broker/service/EntryFilterSupport.java#L45-L51
Anything else?
No response
Are you willing to submit a PR?

[ ] I'm willing to submit a PR!

I asked for this feature,  because in a shared environment the amministrator wants to enforce some filters without allowing users to disable them.
I think that the current behavior is correct

I asked for this feature, because in a shared environment the amministrator wants to enforce some filters without allowing users to disable them.
I think that the current behavior is correct

@eolivelli Yes. However, when allowOverrideEntryFilters=true, shouldn't it be possible for users to disable them?


Customizable frame_id for aikido_rviz InteractiveMarkers

Right now, aikido_rviz is hard-coded to publish in the map frame in two separate places.
https://github.com/personalrobotics/aikido/blob/master/aikido_rviz/src/FrameMarker.cpp#L62
https://github.com/personalrobotics/aikido/blob/master/aikido_rviz/src/BodyNodeMarker.cpp#L27
We should consolidate this to one location, and allow it to be changed to publish markers in another frame.  This would handle situations where something else in TF corresponds to the kido root frame.

I believe this is resolved by #183.


Swap symbols for PS5 Create and Options buttons

Hi! Love these icons 👍 just wanted to make a small adjustment - the symbols on the PS5 Create and Options buttons seem to be swapped judging from the angle of the button outlines. The Create button is the left-leaning one and the Options button is the right-leaning one:

Here's the change:

I've only changed the source svg file for now, let me know if all looks good and I can prepare the individual svg and png files. Thanks! 😁

I seem to have issues with PS5 prompts... #12
This looks great, thank you for making a fix!
To merge though, I'll need updates to

previews/preview.svg and previews/ps5_preview.svg (and then exports to png for both). This can be done by copy/pasting the new icons over the olds ones in the svg, and
the corresponding /png and /svg icons.

Ok, I've updated all but the preview pngs. I'm missing the Megusto font so the "B" in "Mr. Breakfast" isn't quite right:

If that's the only place the font is used, could you convert that "B" to a path instead of text (like with the "M") so that Megusto isn't a dependency? I found some sites offering a free download of the font but I don't necessarily trust them.

Yes I can do that! /meta/center_title is a path version of the "Mr. Breakfast," but I'll go through this fork and make them paths

And thank you for your README copy edits and editions 😁

I mistakenly edited main and not this PR. Let me get that fixed. Sorry for the inconvenience

I mistakenly edited main and not this PR. Let me get that fixed. Sorry for the inconvenience

Done 👍

Awesome, thank you!


Import AS not supported

Since Haxe 3.2 in = as (in import).
E.g.: import any.any.Thing in Foo; is equal import any.any.Thing as Foo;

http://haxe.org/manual/type-system-import.html#since-haxe-3.2.0

Ok, see, they added some docs, nice. Seems like we would have to allow both keywords.

Thanks.

Thank you!

Looks like @fzzr- recieved and tested the fix.  Closing.  Re-open if it's not fixed or recurs.


Error: The settings file could not be opened

Environment

OSX 13.1
Sublime Text Build 4143

Steps to Reproduce

Install OpenAI-sublime-text from package control

Added to /Installed Packages/OpenAI completion.sublime-package

Sublime Text → Settings... → Package Settings → OpenAI completion → Settings

Current Behavior
The settings file "res://Packages/openAI/openAI.sublime-settings" could not be opened

There is no openAI directory in /Packages.
An error also occurs if "Key Bindings" is chosen from the menu.
When I start Sublime Text, there are no errors in the console.

Expected Behavior
User settings should be opened.

Main menu uses the wrong package name openAI. A package name must match the install name of the package, which is OpenAI completion.
https://github.com/yaroslavyaroslav/OpenAI-sublime-text/blob/09dafc198716ea144c42dde477e3ed9847f1ac64/Main.sublime-menu#L19

@deathaxe is there spaces allowed within a name? Like should i rename it directly to as you said above: OpenAI completion.sublime-settings?

is there spaces allowed within a name?

Package names can contain spaces. The only limitation is that plugins located within such packages can't be imported by other packages, but thats a rather rare use case with other implications anyway.

Like should i rename it directly to as you said above: OpenAI completion.sublime-settings?

At least the package nme must be renamed to fix the issue. Whether the settings file should be renamed as well, is a rather strategic question. It is a common design pattern to use the package name for settings files as well, just to avoid any confusion.

I take it this still needs to be submitted to the Sublime package repo or something right?

@mattlinares It's been published at v1.1.3.


MediaDriver Unaligned 8-byte access in StatusMessageFlyweight

I  have encountered a repeatable JVM crash issue when starting the MediaDriver on a solaris 10 / sparc9 box.  This appears to be due to an unaligned write in an UnsafeBuffer made by the StatusMessageFlyweight.  I have reproduced this with aeron 1.2.5 and 1.3.0.
To diagnose I ran with the -javaagent:agrona-agent-0.9.6.jar as per the wiki and it returns the following (on windows 7 x64 and solaris 10/sparc9):
WARNING: /var/tmp/aeron-someuser already exists.
TRANSFORM org.agrona.concurrent.UnsafeBuffer
TRANSFORM io.aeron.protocol.DataHeaderFlyweight
[GC (Allocation Failure)  65536K->5467K(251392K), 0.0627118 secs]
TRANSFORM io.aeron.protocol.HeaderFlyweight
TRANSFORM io.aeron.protocol.SetupFlyweight
TRANSFORM io.aeron.protocol.RttMeasurementFlyweight
TRANSFORM io.aeron.protocol.NakFlyweight
TRANSFORM io.aeron.protocol.StatusMessageFlyweight
Exception in thread "main" org.agrona.agent.BufferAlignmentException: Unaligned 8-byte access (Index=28, Buffer Alignment Offset=0)
        at org.agrona.agent.BufferAlignmentInterceptor$Verifier.verifyAlignment(BufferAlignmentInterceptor.java:35)
        at org.agrona.concurrent.UnsafeBuffer.putLong(UnsafeBuffer.java:344)
        at io.aeron.protocol.StatusMessageFlyweight.receiverId(StatusMessageFlyweight.java:179)
        at io.aeron.driver.media.ReceiveChannelEndpointThreadLocals.<init>(ReceiveChannelEndpointThreadLocals.java:70)
        at io.aeron.driver.MediaDriver$Context.concludeNullProperties(MediaDriver.java:724)
        at io.aeron.driver.MediaDriver$Context.conclude(MediaDriver.java:539)
        at io.aeron.driver.MediaDriver.<init>(MediaDriver.java:170)
        at io.aeron.driver.MediaDriver.launch(MediaDriver.java:285)
        at io.aeron.driver.MediaDriver.launch(MediaDriver.java:274)
        at io.aeron.driver.MediaDriver.main(MediaDriver.java:144)

Is this something that can be looked at and resolved in the flyweight?
I appreciate this target platform is unusual but in this instance I have a requirement to deploy a new server to legacy kit running Solaris 10 sparc9 architecture for a transient period.  More generally should we consider sparc/solaris as a platform generically supported if run with a 1.8 JVM but not specifically verified?

Thanks. We have not done much testing on Sparc but it should work and this is one we missed. We'll look at how best address this and will check all the protocol codecs for alignment in case we might have missed others.

@neekolashall  I've committed a fix for this. Can you please build and test to see if it works for you?

@mjpt777 Thanks for the quick response.  I can verify that the fix to the StatusMessageFlyweight has stopped it causing a JVM crash.  I did however hit one further alignment issue when starting the MediaDriver in the CncFileDescriptor which also causes a JVM crash.  It looks like the CLIENT_LIVENESS_TIMEOUT_FIELD_OFFSET needs a 4 byte pad, but this would break the Cnc compatibility and would need to follow your version strategy I expect.
TRANSFORM org.agrona.concurrent.UnsafeBuffer
TRANSFORM io.aeron.protocol.DataHeaderFlyweight
TRANSFORM io.aeron.protocol.HeaderFlyweight
TRANSFORM io.aeron.protocol.SetupFlyweight
TRANSFORM io.aeron.protocol.RttMeasurementFlyweight
TRANSFORM io.aeron.protocol.NakFlyweight
TRANSFORM io.aeron.protocol.StatusMessageFlyweight
Exception in thread "main" org.agrona.agent.BufferAlignmentException: Unaligned 8-byte access (Index=20, Buffer Alignment Offset=0)
        at org.agrona.agent.BufferAlignmentInterceptor$Verifier.verifyAlignment(BufferAlignmentInterceptor.java:35)
        at org.agrona.concurrent.UnsafeBuffer.putLong(UnsafeBuffer.java:370)
        at io.aeron.CncFileDescriptor.fillMetaData(CncFileDescriptor.java:159)
        at io.aeron.driver.MediaDriver$Context.conclude(MediaDriver.java:557)
        at io.aeron.driver.MediaDriver.<init>(MediaDriver.java:164)
        at io.aeron.driver.MediaDriver.launch(MediaDriver.java:274)
        at io.aeron.driver.MediaDriver.launch(MediaDriver.java:263)
        at io.aeron.driver.MediaDriver.main(MediaDriver.java:137)

I attempted a local patch just to see how far I could get and manged to start the MediaDriver start without ByteBuddy errors and errors reported by ErrorStat.  Unfortunately none of the client's can seem to connect to the media driver.  They get a timeout talking to the driver when creating a publication (I just ran the sample program io.aeron.samples.StreamingPublisher).  The same patch works without issue on Windows, so I wonder if I'm now encountering some other unique platform isue with Solaris/sparc.  I'm not sure what the next steps to debug would be, or if I'm starting to enter rabbit hole territory here.
Exception in thread "main" io.aeron.exceptions.DriverTimeoutException: No response from driver within timeout
        at io.aeron.ClientConductor.awaitResponse(ClientConductor.java:478)
        at io.aeron.ClientConductor.addPublication(ClientConductor.java:187)
        at io.aeron.Aeron.addPublication(Aeron.java:197)
        at io.aeron.samples.StreamingPublisher.main(StreamingPublisher.java:79)

We are bumping the CnC version on the next release so this is a good time to address this. Can you try the latest build? I've just made another change.

Thanks again for the quire response time.  The media driver now operates without any alignment issues or crashes.
I have a residual issue where the clients are still not able to talk to the media driver (embedded or run in a separate JVM).  After tracing system calls and pmap/pfiles the memory mapped files appear to be created without issue but the DriverProxy times out waiting for the response when creating a publication.  The MediaDriver doesn't see the publication message (or any messages) and I experience the DriverTimeoutException as above.
At this point I'm looking at retreating and finding another strategy to share some Redhat x86_64 Kit.  I get the feeling there are a range of other platform issues unique to sparc waiting to be found.  I'm happy to simply accept and close this ticket as the alignment fixes are generally applicable and useful.

Please close this one and investigate the other issue.
It may be helpful to use the logging media driver and set logging to all. Also worth using the LogInspector to view the created publications or images.
https://github.com/real-logic/Aeron/blob/master/aeron-samples/scripts/logging-media-driver
https://github.com/real-logic/Aeron/wiki/Monitoring-and-Debugging

@neekolashall I think I've tracked this to a big endian issue that will be fixed in the next release.  Please let us know if the a release after 1.10.4 works for you.

Thanks, I’ll give this a test tomorrow.  Out of interest what other big endian architecture did you resolve this for?

@mjpt777 I ran against a current snapshot (f53dab6195a1147dea720543cec92404172ff831) and encountered a jvm crash.  It seems that there are some new byte alignment issues that have crept into the codebase.
`$ java -cp aeron-all-1.10.6-SNAPSHOT.jar io.aeron.driver.MediaDriver
WARNING: /var/tmp/aeron-adm_nh7 already exists.
INFO: Aeron CnC file exists: /var/tmp/aeron-adm_nh7/cnc.dat
INFO: Aeron toDriver consumer heartbeat is (ms): 38815
WARNING: Existing errors saved to: 2018-08-22-09-17-26-145+1000-error.log

A fatal error has been detected by the Java Runtime Environment:

SIGBUS (0xa) at pc=0xffffffff7dd77d8c, pid=18330, tid=0x0000000000000046

JRE version: Java(TM) SE Runtime Environment (8.0_151-b12) (build 1.8.0_151-b12)
Java VM: Java HotSpot(TM) 64-Bit Server VM (25.151-b12 mixed mode solaris-sparc compressed oops)
Problematic frame:
V  [libjvm.so+0xd77d8c]  Unsafe_SetInt+0x168

Core dump written. Default location: /tmp/aeron-dist/core or core.18330

An error report file with more information is saved as:
/tmp/aeron-dist/hs_err_pid18330.log
Compiled method (nm)   24553  543     n 0       sun.misc.Unsafe::putInt (native)
total in heap  [0xffffffff6a54db10,0xffffffff6a54def0] = 992
relocation     [0xffffffff6a54dc38,0xffffffff6a54dc58] = 32
main code      [0xffffffff6a54dc60,0xffffffff6a54dee8] = 648
metadata       [0xffffffff6a54dee8,0xffffffff6a54def0] = 8
#`
I ran with the media driver with the agrona-agent and it picked up a couple of violations running the BasicPublisher and BasicSubscriber.  There may be more that have been introduced over the past year.
Subscribing to aeron:udp?endpoint=localhost:40123 on stream Id 10 Exception in thread "main" io.aeron.exceptions.RegistrationException: errorCodeValue=0 BufferAlignmentException : Unaligned 8-byte access (Index=26, Buffer Alignment Offset=2)         at io.aeron.ClientConductor.onError(ClientConductor.java:169)         at io.aeron.DriverEventsAdapter.onMessage(DriverEventsAdapter.java:85)         at org.agrona.concurrent.broadcast.CopyBroadcastReceiver.receive(CopyBroadcastReceiver.java:120)         at io.aeron.DriverEventsAdapter.receive(DriverEventsAdapter.java:56)         at io.aeron.ClientConductor.service(ClientConductor.java:716)         at io.aeron.ClientConductor.awaitResponse(ClientConductor.java:752)         at io.aeron.ClientConductor.addSubscription(ClientConductor.java:469)         at io.aeron.ClientConductor.addSubscription(ClientConductor.java:443)         at io.aeron.Aeron.addSubscription(Aeron.java:269)         at io.aeron.samples.BasicSubscriber.main(BasicSubscriber.java:71)
Publishing to aeron:udp?endpoint=localhost:40123 on stream Id 10 Exception in thread "main" io.aeron.exceptions.RegistrationException: errorCodeValue=0 BufferAlignmentException : Unaligned 8-byte access (Index=63, Buffer Alignment Offset=7)         at io.aeron.ClientConductor.onError(ClientConductor.java:169)         at io.aeron.DriverEventsAdapter.onMessage(DriverEventsAdapter.java:85)         at org.agrona.concurrent.broadcast.CopyBroadcastReceiver.receive(CopyBroadcastReceiver.java:120)         at io.aeron.DriverEventsAdapter.receive(DriverEventsAdapter.java:56)         at io.aeron.ClientConductor.service(ClientConductor.java:716)         at io.aeron.ClientConductor.awaitResponse(ClientConductor.java:752)         at io.aeron.ClientConductor.addPublication(ClientConductor.java:387)         at io.aeron.Aeron.addPublication(Aeron.java:241)         at io.aeron.samples.BasicPublisher.main(BasicPublisher.java:65)
This isn't a critical issue but next cnc update it may be worth resolving the alignment issues.
Cheers.

@neekolashall do you happen to have the error logs from the media driver? Will be in the current directory.

Sure, here are the jvm crash logs for a few runs.
hs_err_pid2241.log
hs_err_pid2595.log
hs_err_pid3391.log
hs_err_pid18330.log

Thanks! I see the same cause for those. The ImagesBufferReadyFlyweight is possibly misaligned on the sourceIdentity field length depending on filename length.
That is one we kind of know about. @mjpt777 and I have discussed changes for this. In trying to keep compatible with SBE, this is an issue as SBE has no way to prevent misaligned varData length fields. Although, we do know the charset and can use the fact that the field can be padded with nulls. Sadly, it's not a general fix for SBE, but would be something to consider.
However, not sure why that one would fire on a BasicPublisher, though.

There is a change in the SBE 2.0 RC spec that allows for var data and repeating group headers to be aligned. We could take a similar approach and align the var data length fields. I can find two sources: ImagesBufferReadyFlyweight  and CounterMessageFlyweight.

It will happen on the basic-publisher when the new publication matches an existing subscription and it is filtering the broadcast events and the error happen in the driver.

@neekolashall Can you try the latest build?

@mjpt777 I'm happy to report the latest commit 277929dea1afcd4cb16c71aeaf6473f2df17635a seems to have resolved the issues.  I verified with the streaming performance test and the ping/pong latency test.  Hopefully this exercise has contributed some value to Aeron.  Very impressive turnaround.
I've attached the cpuinfo from kstat and the sample console notes with the notes/console results.
cpu_info.txt
aeron_solaris_sparcv9.txt
Environment:

$ kstat -p cpu_info | head -17
cpu_info:0:cpu_info0:brand      SPARC64-VII+
cpu_info:0:cpu_info0:chip_id    1024
cpu_info:0:cpu_info0:class      misc
cpu_info:0:cpu_info0:clock_MHz  2660
cpu_info:0:cpu_info0:core_id    0
cpu_info:0:cpu_info0:cpu_fru    hc:///component=/MBU_B/CPUM0
cpu_info:0:cpu_info0:cpu_type   sparcv9
cpu_info:0:cpu_info0:crtime<PHONE_NUMBER>
cpu_info:0:cpu_info0:current_clock_Hz<PHONE_NUMBER>
cpu_info:0:cpu_info0:device_ID<PHONE_NUMBER>3639
cpu_info:0:cpu_info0:fpu_type   sparcv9
cpu_info:0:cpu_info0:implementation     SPARC64-VII+ (portid 1024 impl 0x7 ver 0xc1 clock 2660 MHz)
cpu_info:0:cpu_info0:pg_id      1
cpu_info:0:cpu_info0:snaptime   1595164.635855
cpu_info:0:cpu_info0:state      on-line
cpu_info:0:cpu_info0:state_begin<PHONE_NUMBER>
cpu_info:0:cpu_info0:supported_frequencies_Hz<PHONE_NUMBER>
$ cat /etc/release
Oracle Solaris 10 1/13 s10s_u11wos_24a SPARC
Copyright (c) 1983, 2013, Oracle and/or its affiliates. All rights reserved.
Assembled 17 January 2013
$ java -version
java version "1.8.0_151"
Java(TM) SE Runtime Environment (build 1.8.0_151-b12)
Java HotSpot(TM) 64-Bit Server VM (build 25.151-b12, mixed mode)

Best run (with defaults and 1 million messages):

Pinging 1000000 messages
Histogram of RTT latencies in microseconds.


Kuberhealthy chart 60 uses deprecated API apiextensions.k8s.io/v1beta1

https://github.com/jx3-gitops-repositories/jx3-kubernetes/blame/7d75ad00401eb3449bfc35f183f0452ad217b234/helmfiles/kuberhealthy/helmfile.yaml#L14
customresourcedefinition.apiextensions.k8s.io/tasks.tekton.dev serverside-applied
resource mapping not found for name: "khchecks.comcast.github.io" namespace: "" from "config-root/customresourcedefinitions/kuberhealthy/kuberhealthy/khchecks.comcast.github.io-crd.yaml": no matches for kind "CustomResourceDefinition" in version "apiextensions.k8s.io/v1beta1"
ensure CRDs are installed first
resource mapping not found for name: "khjobs.comcast.github.io" namespace: "" from "config-root/customresourcedefinitions/kuberhealthy/kuberhealthy/khjobs.comcast.github.io-crd.yaml": no matches for kind "CustomResourceDefinition" in version "apiextensions.k8s.io/v1beta1"
ensure CRDs are installed first
resource mapping not found for name: "khstates.comcast.github.io" namespace: "" from "config-root/customresourcedefinitions/kuberhealthy/kuberhealthy/khstates.comcast.github.io-crd.yaml": no matches for kind "CustomResourceDefinition" in version "apiextensions.k8s.io/v1beta1"
ensure CRDs are installed first

encountering this as well

looks like the jx3-kubernetes template is one of the few templates (the only one?) that uses kuberhealthy. Perhaps it can simply be removed but don't know the implications of this. Looks like all it adds is health monitoring with prometheus metrics. It is also installing a very old version of the kuberhealthy helm chart.
Can any one confirm that kuberhealthy is safe to remove? And perhaps also a good idea to check other versions. Is there another template that is better to use as a starting point?

See also this pull request: https://github.com/jx3-gitops-repositories/jx3-kubernetes/pull/14, but that gives different errors.
Not a really good experience if you are just trying it out.


What is the meaning of pem in "curl pem https://localhost:4567/logs | jq " in page 86?

This from Chap 3 , when I execute this getting the error ,what is missing here
curl -k --cacert "$(./mkcert -CAROOT)/rootCA.pem" -d ‘{"username":"demo","password":"password"}’ -H ‘Content-Type:application/json’ https://localhost:4567/users
curl: (3) URL using bad/illegal format or missing URL
Bad Message 400reason: Illegal character OTEXT=0xe2


[Question] Can you please let me know how to retrieve the values of the form fields(text boxes/Password fields) created in the bot.

Pre-flight checklist
By the reference of https://github.com/Microsoft/BotBuilder/issues/2785#issuecomment-302429892
created login form in BOT.
System Information (Required)

SDK Platform: Node
SDK Version:
Development Environment: localhost

Issue Description
I am trying to create login(html) form in the BOT ,I am able to create the form with some html controls like text boxes(input fields) please find the screenshot given below.
Can you please let me know how to retrieve the values of the form fields(text boxes/Password fields) created in the bot.
Example Code
Please find the sample snippet

var message = {
    type: 'message',
    textFormat: 'xml',
    text: tableHTML
};
session.send(message);
session.endDialog();
}

]).triggerAction({
matches: [/representative form/i, /loginform/i]
});

function check(form) { /*function to check userid & password*/
            /*the following code checkes whether the entered userid and password are matching*/
            if(form.userid.value == "myuserid" && form.pswrd.value == "mypswrd") {
			alert("success")
                //window.open('target.html')/*opens the target page while Id & password matches*/
            }
            else {
                alert("Error Password or Username")/*displays error message*/
            }
        }

Expected Behavior
In the above function check(form) needs to get the form userid and pwd and after validation need to give success message in the emulator

@MuraliMohanEdiga What you are trying to do is not currently a feature of Bot Framework.
To accomplish your goal, you would need to setup a waterfall dialog that prompts the user for the required information using builder.Prompts.text(). For more information see: Prompt users for input.

@nwhitmont Thank you for your time


ignore-names doesn't seem to work for me

I have code similar to this:
def f(C: float) -> None:
    print(f"cost parameter: {C}")

And I tried to exclude C from the naming check with
[tool.ruff.pep8-naming]
ignore-names = ["C"]

But I still get
naming_test.py:1:7: N803 Argument name `C` should be lowercase
Found 1 error.

I might be doing something wrong, but I can't figure out what.

It looks like we're only checking this for N802. Might just be an oversight? I can fix it.


Feature/add optional support

closes #431
spin off of #474

Looks like you're running afoul of the DCO. See https://github.com/envoyproxy/envoy/blob/main/CONTRIBUTING.md#fixing-dco

The build action runs with Bazel, which uses a separate dependencies list than the Go build. Can you update the PGS dep here?

@rmichela, if you have a chance to take a look?

Any update here? ✌

is this awaiting final review? @rodaine would you know who can approve this?

✌

✌

@akonradi @rodaine just a gentle reminder to catch up on this

When will this feature be merged in? @akonradi

tests passed! 🎉

Awesome job! Thank you for all the hard work to get this over the finish line 😁


having issues setting framerate

Hello,
I am trying to experiment with the framerate property, but I can't seem to change the framerate.
upon reading the docs, I see that it mentions 'This feature is dependent on the tick event object (or an object with an appropriate 'delta' property) being
passed into {{#crossLink 'Stage/update'}}{{/crossLink}}.`, which is the same comment from CreateJS's MovieClip class, here: http://www.createjs.com/docs/easeljs/files/easeljs_display_MovieClip.js.html
It's confusing that the comment is the same between pixi-animate and createJS, is this intentional?
so, since nothing I have done seems to change the framerate of my clips, I assume that I'm not passing the delta into the stage/update method, is there an example someplace of this working I could reference?
[UPDATE] I was able to increase the framerate, but I need to set clip.framerate recursively across all children.
thank you

Actually I do see an issue in my test here.
In my test app I have a button that will double the fps, from 30 to 60 to 120. It stops at 120.
if I press the button, as many times as I like, before triggering the timeline animations, it play as expected. Everything works!
However...
if I play t he movieclip animation and then, either during the animation or well after it has stopped, I then press my double speed button. In this case, where the animation has played already, pressing the speed button will break the animation. The transitions are no longer timed properly with the fps, and then do not animate properly any more. Is there an update call I need to do or something for the timeline animations to be updated in this case?

There is currently no API for dynamically changing the framerate in the way you are suggesting. This definitely could be improved (and is a good feature to add!). We might even consider a convenience speed property which specifically designed to be changed dynamically.
Right now, the framerate is set when you add a new MovieClip, if it doesn't have a framerate set or if there's not a framerate provided in the MovieClip's constructor, it inherits it from the parent's framerate. However, there could definitely be issues with this, as you found out, when changing it after-the-fact.
One thing to note, if you are changing framerate recursively, you should make sure you loop through _timedChildTimelines not children property. Since children MovieClips are added and removed during playback they might not be all on the children property.

oh, well this is interesting then. _timedChildTimelines is always empty in my project, both before and after triggering the animation

Could you possibly make a jsfiddle or upload a version of what you're building? It would be useful to see what's breaking.

I will when I have a bit of spare time, as I don't actually require runtime framerate changes just yet. I was just reporting a bug for the most part.

I tried to make a JSFiddle today, but I got stuck trying to provide the shapes.json file.
do you have an example for showing a published scene in JSFiddle?


Location Is Undefined for Default Albums

When the default photo albums (presumably other default albums as well) are created, sys_albums.Location is set to Undefined.

It's also not possible to clear the values for Location and Description even though they aren't mandatory fields. If you empty the fields and save, the changes are not saved. Some modules give no message and others give an error message. The message is simply "Error" which really is not helpful.


--env-file is not a valid option for docker-compose up

recreate
docker-compose up --env-file=.env
issue
readme.md specifies using  --env-file as method of passing .env to docker compose.
Docker compose returns valid options list: --env-file is not a valid option  for docker-compose up (only for docker run)
sugged fix
remove option from readme.md

I had the same issue originally due to my version of docker-compose (1.24 or something). It should work with the version recommended in the pre-reqs - 1.25+.

untested, but assume this works in v. > 1.25


Add 4 potential SCA tools to INVENTORY_SOURCES.md

Addresses #19:
Add a few more tools that can be potential sources of dependency information.

Thanks!


JOB_MASTER journal quorum info breaks

Alluxio Version:
Alluxio v2.6.2
Describe the bug
./bin/alluxio fsadmin journal quorum info -domain JOB_MASTER command doesn't work anymore.
To Reproduce
Following admin command fails
$ ./bin/alluxio fsadmin journal quorum info -domain JOB_MASTER
Failed to connect to master (<IP_ADDRESS>:20001) after 8 attempts.Please check if Alluxio master is currently running on "<IP_ADDRESS>:20001". Service="JournalMaster"

with following error logs
2021-10-08 21:57:40,377 INFO  Reflections - Reflections took 279 ms to scan 1 urls, producing 69 keys and 375 values
2021-10-08 21:57:40,667 INFO  NettyUtils - EPOLL_MODE is available
2021-10-08 21:57:45,566 ERROR AbstractShell - Error running journal quorum info -domain JOB_MASTER
alluxio.exception.status.UnavailableException: Failed to connect to master (<IP_ADDRESS>:20001) after 8 attempts.Please check if Alluxio master is currently running on "<IP_ADDRESS>:20001". Service="JournalMaster"
        at alluxio.AbstractClient.connect(AbstractClient.java:279)
        at alluxio.AbstractClient.retryRPCInternal(AbstractClient.java:403)
        at alluxio.AbstractClient.retryRPC(AbstractClient.java:372)
        at alluxio.AbstractClient.retryRPC(AbstractClient.java:361)
        at alluxio.client.journal.RetryHandlingJournalMasterClient.getQuorumInfo(RetryHandlingJournalMasterClient.java:69)
        at alluxio.cli.fsadmin.journal.QuorumInfoCommand.run(QuorumInfoCommand.java:76)
        at alluxio.cli.AbstractShell.run(AbstractShell.java:137)
        at alluxio.cli.fsadmin.FileSystemAdminShell.main(FileSystemAdminShell.java:71)
Caused by: alluxio.exception.status.UnavailableException: Failed to handshake with master <IP_ADDRESS>:20001 to load cluster default configuration values: UNIMPLEMENTED: Method not found: alluxio.grpc.meta.MetaMasterConfigurationService/GetConfiguration
        at alluxio.util.ConfigurationUtils.loadConfiguration(ConfigurationUtils.java:504)
        at alluxio.ClientContext.loadConf(ClientContext.java:135)
        at alluxio.ClientContext.loadConfIfNotLoaded(ClientContext.java:159)
        at alluxio.AbstractClient.beforeConnect(AbstractClient.java:176)
        at alluxio.AbstractClient.connect(AbstractClient.java:224)
        ... 7 more
Caused by: io.grpc.StatusRuntimeException: UNIMPLEMENTED: Method not found: alluxio.grpc.meta.MetaMasterConfigurationService/GetConfiguration
        at io.grpc.stub.ClientCalls.toStatusRuntimeException(ClientCalls.java:262)
        at io.grpc.stub.ClientCalls.getUnchecked(ClientCalls.java:243)
        at io.grpc.stub.ClientCalls.blockingUnaryCall(ClientCalls.java:156)
        at alluxio.grpc.MetaMasterConfigurationServiceGrpc$MetaMasterConfigurationServiceBlockingStub.getConfiguration(MetaMasterConfigurationServiceGrpc.java:424)
        at alluxio.util.ConfigurationUtils.loadConfiguration(ConfigurationUtils.java:498)
        ... 11 more

Expected behavior
should be function similar to -domain MASTER
./bin/alluxio fsadmin journal quorum info -domain MASTER
Urgency
Break job-master replacement by command line managing (add/remove) quorum member.
Additional context
This bug is probably introduced by commit #13893
Build with commit 601272157bf8558c2e9bf0315ba10a3815e585b6 reverted, the command works fine.
Need a revisit on commit 601272157bf8558c2e9bf0315ba10a3815e585b6 code change.

all nodes are v2.6.2, I run the command on the same node where MASTER leader runs.
$ ./bin/alluxio version
2.6.2

With default log configuration, job master server side log doesn't verbose any logline.

@kevincai do you mind joining our community slack https://alluxio.io/slack so we can discuss about this issue more interactively? I feel there is a lot of overhead in tagging each other on github...

If you are convenient, please help double check that. The original change is trying to divide the client into two categories:

Know only Master
Know both Master and JobMaster
The rest semantic is same.

In reality, it does talk to the two. I am preferring to keep a simple fix. The client side is quite complex, a lot of clients are there. At the same time, we need make sure the convention following the current one.


Train Carts, made my TPS go down to 10.

BkCommonLib version:
TrainCarts version:
Spigot version:

Problem or bug:
It completely killed the perfomance of my server, check the timings for more info
https://timings.spigotmc.org/?url=qolasadoqa
Expected behaviour:
Not kill the perfomance of my server
Steps to reproduce:
I am not quite sure, I installed hte mod a few days ago, wanting to use it for some minecart system, the server however is still in developement, so did not get around to it yet. I quickly noticed that the plugin, while not even having done any commands with it yet, used 4% of the server tick.
I just shrugged it of, and hoped it would not become an issue in the future, however sadly it did.
I would appreciate a lot if you would be able to solve this, as this plugin do seem pretty interesting.

These issues have actually been fixed in the development builds thanks to someone else reporting it to me a couple weeks ago. The problem is/was that, when chunks unload, train carts did a whole lot of processing for every single sign in that chunk to find the power state and rail information. It also did this for non-TC signs. You can imagine this wreaks havoc on some servers.
There's a whole lot more that I fixed though, and since a new version is coming up soon I'd welcome you to test out the latest development builds to see if that resolves these and other performance problems.
https://ci.mg-dev.eu/job/BKCommonLib/328/
https://ci.mg-dev.eu/job/TrainCarts/207/


Overcounting of step usage in All Steps

Under the "All Steps" tab of the report the "Step Summary Result Chart" y-axis is marked as "Number of Scenarios". This would imply that it is showing how many scenarios a step is used in. However, if a step is used multiple times in a scenario it is counted multiple times, leading to unexpectedly skewed data.
Possible Solutions:
Each step should be counted once for each scenario
The y-axis should be relabeled to "Number of Usages"

I renamed the y axis to number of usages since a step could potentially be included multiple times in a scenario with different results.

Will be included in 2.2.0

@laxersaz Thanks man !! Tentative time frame for 2.2.0 ?

Releasing now :-)

hahah...great thanks buddy !!


openqa-investigate: Improve curl HTTP status handling

Issue: https://progress.opensuse.org/issues/95830
If a HTTP request doesn't return 200, we do not want to continue, and we want
to know the actual URL and status.
Because investigate is called with || do-something, -e is not
in effect, so we check each runcurl call explicitly.
I did the same for the runjq calls.

@perlpunk just a quick niptick: investigate instead of ivestigate :)

@perlpunk just a quick niptick: investigate instead of ivestigate in the commit message (and title) :)

oops. fixed in title and commit :)


[AutoPR adhybridhealthservice/resource-manager] ADHybridHealthService: "type" should be "array" when "items" is present.

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/3887

(message created by the CI based on PR content)

Codecov Report

Merging #3330 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #3330      +/-   ##
==========================================
+ Coverage   56.33%   56.33%   +<.01%
==========================================
  Files        9129     9131       +2
  Lines      179795   179797       +2
==========================================
+ Hits       101295   101297       +2
  Misses      78500    78500

Impacted Files
Coverage Δ

azure-mgmt-adhybridhealthservice/azure/__init__.py
100% <100%> (ø)

...-mgmt-adhybridhealthservice/azure/mgmt/__init__.py
100% <100%> (ø)

Continue to review full report at Codecov.

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


[Feature]: Create dedicated pages for the blockchain subdomains

Is your feature request related to a problem? Please describe.

create separate pages for all the blockchain subdomains and connect them with the cards
https://www.nsccbpit.tech/blockchain
Describe the solution you'd like
No response
Describe alternatives you've considered
No response
Additional context
No response

@pradeeptosarkar can you assign me, i want to work on this issue.

sure go ahead @ronisarkarexe . feel free to tag in case of any doubts.!

sure go ahead @ronisarkarexe . feel free to tag in case of any doubts.!

sure, thanks

@ronisarkarexe are you still working on this issue?

@pradeeptosarkar I am currently participating in a hackathon that started last night and will continue until Monday.

@ronisarkarexe okay cool. so you will work on this after that all shall we reassign?

@ronisarkarexe okay cool. so you will work on this after that all shall we reassign?

you can. Thanks

okay cool @ronisarkarexe

Hey @pradeeptosarkar , i think I can work on this issue. Can you please assign me this task.

cool @ayush4028 go ahead

Hello @pradeeptosarkar , do I have to add any content to those pages or just have to create separate pages and connect them with the cards?

@ayush4028 ya it would be better if you could add the content :)

resolved


feat: add GitHub providers

Relates to:
GitHub Plugin Enhancement
Risks
Low - Changes are contained within the GitHub plugin package and don't affect core functionality.
Background
What does this PR do?
This PR enhances the GitHub plugin functionality by:

Improving file handling capabilities (testFiles.ts, documentationFiles.ts, workflowFiles.ts)
Enhancing repository operations (createCommit.ts, createPullRequest.ts, initializeRepository.ts)
Updating source code and release management (sourceCode.ts, releases.ts)
Refining GitHub service integration (github.ts)

What kind of change is this?
Features (non-breaking change which adds functionality)
Improvements (misc. changes to existing features)
Documentation changes needed?
My changes require a change to the project documentation to reflect new GitHub plugin capabilities.
Testing
Where should a reviewer start?

Start with packages/plugin-github/src/services/github.ts to understand the core service changes
Review the provider implementations in:

testFiles.ts
sourceCode.ts
documentationFiles.ts
workflowFiles.ts

Examine the plugin implementations in:

createMemoriesFromFiles.ts
createPullRequest.ts
createCommit.ts
initializeRepository.ts

Detailed testing steps

Initialize a new repository using the plugin

Verify repository creation
Check initial file structure

Create and commit changes

Test file creation/modification
Verify commit creation

Create a pull request

Verify PR creation with correct files
Check PR template and description

Test memory creation from files

Verify correct parsing of different file types
Check memory creation accuracy

Verify release management

Test release creation
Validate release assets

Deploy Notes
No special deployment steps required. Standard package deployment process applies.

Awesome let me have a look rn :)


Co3k patch 1

Summary

Fixes: List issue numbers here
Other Information

Sorry, didn't mean to send a pull request. I'm just trying to figure out how svelte-adapter-firebase works, so I can also work on improving / sending PRs / making sure it works on the latest sveltekit.


Tooltip not hiding when clicking a link on new tab.

Hi, got an issue with the tooltip as it retains or not destroy when opening a link on a new tab?
Tried the autoHideOnMousedown = true but it doesn't help.

We're experiencing the same thing. This library doesn't seem to be maintained anymore. We're switching to something else.

@lukaszarpak have you found any alternative library?


SystemJS - Incorrect import 'jspdf'

In my case, I am using systemjs to load ES modules. In the test module, I import the jsPDF type from the jspdf module as shown below:
import jsPDF from 'jspdf';

However when you attempt to instantiate a 'jsPDF' object, the "jsPDF is not a constructor" exception has occured.
Here is a runnable example
How to import the 'jsPDF' module correctly?

<EMAIL_ADDRESS>does not yet support ES style module imports/exports. We are currently working on a new release with modern module formats (see e.g. #2804). Your example should work with the files that are currently in the dist folder of this repo.


Shader errors should not necessarily block app startup

Is your feature request related to a problem? Please describe.
See #3737 for an example. In that specific case, the graphics driver has optimized away a uniform variable. The shader presumably would still operate correctly, but the error thrown essentially terminates the application on startup.
Describe the solution you'd like
We do want to be notified of such errors during unit tests and possibly in development builds, but they should not break production builds. We do probably want to log them in production builds.
There may be other, more severe errors that produce the same result. Even if a shader error would cause that shader to fail to operate correctly, that would probably be preferable to terminating the app.
Describe alternatives you've considered
None
Additional context
To unblock the user as quickly as possible, complete and backport either this issue or #3737 - whichever is faster.

@pmconne
@DStradley thinks this would be the fastest way to unblock the user.
Questions:

How do we know what mode we are running in?  Production / Unit Test / Development
How do we log the errors in production builds?


Unable to register new user: "NoneType object has no attribute id"

This looks like a regression of an error from a previous version ( #319 #194 maybe )
Fresh install of version 0.2.1 under CentOS 7.  Python 3.6, gunicorn, systemd, postgresdb
When trying to register first user, all correct data are entered, but it fails to work and gives the red error message:
'NoneType' object has no attribute 'id'
No errors are logged in the logfile to indicate the cause.

This appears to be because the database has an empty "Role" table, and the creation of the first account assumes that the Administrator role will already exist.
There does not appear to be any installation instructions or tool to create the initial data?  Even information that you need to run 'flask db migrate' to initialise the tables is missing form the Postgres instructions, and this fails because it always uses "default_config.py" for its configuration rather than config.py

The "flask db" utility should use config.py if available
The install instructions should mention clearly how to initialise the database
The database initialisation should include creating the required basic records (roles, templates etc) as in previous versions
The initial user creation should not fail if there are no Administrator, Operator and User roles defined; instead these should be automatically created by the system
A failed user creation should log a more detailed trace to the logfile to help with debugging

You faced those issues because the database wasn't initialized properly.
The new version of PDA keeps the same migration scripts as the previous one so the basic records (roles, templates) will be created.
Before running flask db upgrade, we need to set the following environment variables:

FLASK_APP=powerdnsadmin/init.py
FLASK_CONF=

If you don't set the FLASK_CONF variable, it will use the default_config.py file.
Btw, I am going to add more docs in the repo to keep it up-to-date. The docs in the WIKI page contributed by several users so it might be outdated.

I have been trying to create a .repo file for building an RPM of PowerDNS-Admin, so that it can be installed automatically by our system here.  Unfortunately the use of Flask, Pip, and nodejs has made it rather difficult - not to mention trying to find some way to automate database initialisation post-install.  Currently, the 'flask assets build' is the main part causing me issues as for some reason the 'assets' command is not recognised when flask is run within rpmbuild.

Hello,
Can we close this issue ? What is the status about building RPMs ?
Regards


IndexError: redis_graph.commit()

Running this code
import redis
import redisgraph
print(redis.__version__)
from redisgraph import Node, Edge, Graph
r = redis.Redis(host='localhost', port=6379)
redis_graph = Graph('social', r)
john = Node(label='person', properties={'name': 'John Doe', 'age': 33, 'gender': 'male', 'status': 'single'})
redis_graph.add_node(john)
japan = Node(label='country', properties={'name': 'Japan'})
redis_graph.add_node(japan)
redis_graph.commit()

raises:
3.2.1
Traceback (most recent call last):
  File "./test.py", line 11, in <module>
    redis_graph.commit()
  File "/Users/acortis/my_codes/testbed/redisgraph/.env/lib/python3.7/site-packages/redisgraph/client.py", line 136, in commit
    return self.query(query)
  File "/Users/acortis/my_codes/testbed/redisgraph/.env/lib/python3.7/site-packages/redisgraph/client.py", line 158, in query
    statistics = response[1]
IndexError: list index out of range

redis-cli monitor shows
redis-cli monitor
OK

1560173120.370804 [0 [::1]:58173] "GRAPH.QUERY" "social" "CREATE (rhevungnwv:person{name:\"John Doe\",age:33,gender:\"male\",status:\"single\"}),(wtqgmdvmfi:country{name:\"Japan\"})"

The redisgraph.so library was built successfully from commit:
commit 728080f8d6bf72dc12fd90a98f741dbb0f04b74b (HEAD -> master, origin/master, origin/HEAD)

Could you please help?
Thanks
Andrea

Please install redisgraph-py directly from git:
pip install git+https://github.com/RedisGraph/redisgraph-py.git
RedisGraph 2.0 (728080f8d6bf72dc12fd90a98f741dbb0f04b74b)
is yet to be officially released, it introduces a change to the result-set structure which isn't compatible with RedisGraph-py 1.7

Thank you this issue is resolved

For the ones using Docker Images the same problem will appear if you're using the latest image.
To comply with redisgraph-py:1.7, you should use redislabs/redisgraph:1.2.2.

Hey, This issue is still there. @swilly22 's solution solves the problem.


Fix unassignable properties by adding undefined with exactOptionalPropertyTypes

Provide a specific error and codefix for missing undefined in target types with exactOptionalPropertyTypes: true.

The codefix skips paths starting with lib. and containing node_modules
Codefixes cannot use an elaboration as a trigger code. And the codefix logically needs to fix all of the properties in the target at once. So the resulting error message is a little vague. It still relies on reading the elaboration to find out the first erroneous property.

Fixes #44403

@iisaduan you might be interested in using this in your codefix tool when it's ready.

@typescript-bot pack this

I was surprised that neither of these cases are covered:
// @exactOptionalPropertyTypes: true

interface User {
    name: string;
    email?: string;
}

const user: User = {
    name: "Andrew",
    email: undefined,
};

user.email = undefined;

This is ready to review again; it was a surprising amount of work to add the two other cases that @andrewbranch pointed out.

seem like perfect candidates for related diagnostics of '{0}' was declared here.

I'll take a look at how complicated that is. The first draft has some add-related-span code that runs afterward already. The two newer errors are separate from all the drill-down error code, though.

, and your codefix could actually pull the related diagnostic location out instead of calculating it there.

Codefixes only get an error code and a span, not the actual diagnostic. I could change that but I don't have the time right now.

@typescript-bot perf test this

@weswigham can you provide an opinion on the way I've added errors in/around the dril-down error code? I'm not super happy with the way it works.

I didn't see any changes in the perf tests.

@typescript-bot pack this

The failure was on an error here: https://github.com/microsoft/playwright/blob/3e05d8e9fa54ea11def912b69228116b3a431f42/src/server/snapshot/snapshotterInjected.ts#L124
The actual error node is the whole LHS of the assignment:

but the code assumes that the errorNode is the Identifier ensureCachedData. Consequently, errorNode.parent.arguments.indexOf(errorNode) is -1, so crashes at n.valueDeclaration.parameters[i].name.

I went poking through other codefix's code and found getFixableErrorSpanExpression, so I moved that to services/utilities and started using it.

@typescript-bot pack this

@sandersn it now crashes here: https://github.com/microsoft/playwright/blob/869f8d541b60fae388637e83243a642b0f225e4b/src/server/network.ts#L352

errorNode in your codefix is the ObjectLiteralExpression. i is 0. n.valueDeclaration is the PropertyDeclaration:
private _finishedPromiseCallback: (arg: { error?: string }) => void = () => {};

I wrapped each individual request for codefixes in a try/catch and there was one other place in playwright that failed with the same crash. Otherwise, what became apparent is there are still a lot of assignability errors that this codefix doesn’t cover, because we started out with 253 errors (8 were due to me failing to run some codegen as part of the build, I think), and after applying this codefix across all files repeatedly until it was no longer offered (except for the two places that crashed), we actually went up to 404 errors.
Possible conclusions:

We should attempt to identify a lot more assignability errors as fixable with this codefix
#44419 is probably not feasible for large/complex projects (which is where it would be valuable)

If you want to see what unfixable errors remaining after running the tool, I’ve committed them to a fork (one commit per run, and each run actually does two passes of codefixes when it discovers overlapping changes): https://github.com/andrewbranch/playwright/tree/exactOptionalPropertyTypes

Another possible conclusion:
We need more analysis of the new errors to figure out how the codefix causes the new errors. I thought that adding undefined wouldn't change anything since that used to be the semantics, but it sounds like I was wrong.
In the meantime, we need to decide between shipping a half-baked codefix or not. We've done this before! But for a new pedantic feature, I don't think we'll do much better than codemods written by the experts who adopt new features. So I'm leaning toward not shipping it.
@DanielRosenwasser do you have an opinion about this?

I thought that adding undefined wouldn't change anything since that used to be the semantics, but it sounds like I was wrong.
I believe the deal is that if you naively add | undefined to every optional property, you wind up with no errors, minus the ones you couldn’t change because they were in node_modules etc. But the codefix doesn’t want to add | undefined to every optional property; it just does the ones that it identifies as needing to change. Here’s an example of a contrived program with one fixable error that turns into two unfixable ones upon being fixed.
// @exactOptionalPropertyTypes: true

interface A1 {
  a?: string | number;
}

interface A2 {
  a?: string | boolean;
}

interface B extends A1, A2 {
  a?: string;
}

declare let b: B;
b.a = undefined;
//^^ Type 'undefined' is not assignable to type 'string' with 'exactOptionalPropertyTypes: true'. Consider adding 'undefined' to the type of the target.

Fixing the property assignment adds an “incorrectly extends” error for each of B’s extended interfaces.

I took a look at your branch and got 443 errors (!). 213 of those used the new errors, but I guess didn't offer a codefix.
I'm looking at the errors right now.

I think this is the wrong way to fix exactOptionalPropertyTypes errors. Applying the fix produces errors that (1) occur a long way from the true cause (2) penalises slightly-bad code like having structurally related types with no extends clause, or combining options types with a bunch of type arithmetic.
I think the better fix is to unconditionally add | undefined to all optional properties, and then have a human look at the diff and decide which places should not allow undefined.
The new error messages are often correct, so it might be worthwhile to leave them. However, I'm not sure their advice is clear enough without a codefix to back them up.

@typescript-bot pack this

I think this is ready now. I simplified the fix to add | undefined to all properties in a type, regardless of whether the source could assign undefined to it, because that's more likely to result in a working build afterward.
And I got rid of the fix-all, since it's not a good idea. Thanks @amcasey for suggesting this.

@typescript-bot pack this one last time please 🙃