add rocketmq-client-java-5.x plugin

[x] Add a test case for the new plugin, refer to the doc

[x] Update the CHANGES log.

rocketmq-scenario is there, and have tested for 5.1.0. What is new of this one?

This is another official client of Rocketmq, which is the gRPC protocol. Better to use than Rocketmq's client

Please be clear about, it seems also from ASF RocketMQ. What do you mean better than RocketMQ client.

Please be clear about, it seems also from ASF RocketMQ. What do you mean better than RocketMQ client.

The gRPC protocol was newly introduced in version 5.0, aimed at evolving a more lightweight, standardized, and easily extensible client-server communication protocol with mainstream cloud native technologies. The gRPC protocol SDK evolves as an independent repository RocketMQ Clients , supporting languages such as Java/C++/.NET/Go/Rust. There is a relatively decoupled relationship between the client and the server, following the RocketMQ API protocol interface agreement. the doc
I think rocketmq-client-java is easier to use.

Thanks for the clearance.

Then please rename your scenario to rocketmq-5-grpc-scenario

Then please rename your scenario to rocketmq-5-grpc-scenario

modify completed，check please

You need to fix tests and make sure all could pass locally, before you asked for our review.

You need to fix tests and make sure all could pass locally, before you asked for our review.

I'm sorry，but i test locally bash ./test/plugin/run.sh -f rocketmq-5-grpc-scenario ，no code style exceptions occurred

Remove the commented codes, that is unnecessary.
Code style will be checked when it runs on CI.

Remove the commented codes, that is unnecessary. Code style will be checked when it runs on CI.

i'm sorry, i check style pass locally

[INFO] Scanning for projects...
[INFO]
[INFO] ----< org.apache.skywalking.apm.testcase:rocketmq-5-grpc-scenario >-----
[INFO] Building skywalking-rocketmq-5-grpc-scenario 1.0.0
[INFO] --------------------------------[ jar ]---------------------------------
[INFO]
[INFO] --- maven-checkstyle-plugin:3.1.1:check (validate) @ rocketmq-5-grpc-scenario ---
[INFO] Starting audit...
Audit done.
[INFO] You have 0 Checkstyle violations.

testing encountered an issue. rocketmq-client not support autoCreateTopicEnable , look here

so Plugin Test occurred exception: CODE: 17  DESC: No topic route info in name server for the topic: TopicTest. can you give  some advice @wu-sheng
but i'm not create topic by rocketmq-client .  when i test pass locally , access the rocketmq docker I started myself

What kind of ways do they support to create a topic?  A CLI or using legacy client lib?
This test/plugin/scenarios/rocketmq-5-grpc-scenario/bin/startup.sh script would be called.

What kind of ways do they support to create a topic? A CLI or using legacy client lib?
This test/plugin/scenarios/rocketmq-5-grpc-scenario/bin/startup.sh script would be called.

can use admin tool, but I don't know how to use it appropriately in a plugin test case.

This seems a shell, you could download it and run the command in the startup script.
Anyway, this is mostly RocketMQ thing. I have limited knowledge for that.

This seems a shell, you could download it and run the command in the startup script. Anyway, this is mostly RocketMQ thing. I have limited knowledge for that.

ok, i will try it

Why are you adding so many binaries and scripts for only setting a topic? The binaries are not allowed.

i test pass locally

No binary is accepted by us, as they are not source codes. I believe this would not be so hard to just create a topic for RocketMQ server. If you don't know how to do this, I would suggest you talk with their community, to find out What is the quickest way to create a topic through CLI?.

Why are you adding so many binaries and scripts for only setting a topic? The binaries are not allowed.

tool dependent on this binaries ,I'm not sure which ones can not be used, so I uploaded all of them

None of them is acceptable. People would not like to have huge binaries in the source codes tar. Also, this is not allowed by the ASF.
This tool should be downloaded through shell, rather than like this.
Like I said, this should not be this hard. Please reach RocketMQ community about the question I raised.

Why are you adding so many binaries and scripts for only setting a topic? The binaries are not allowed.

tool dependent on this binaries ,I'm not sure which ones can not be used, so I uploaded all of them

None of them is acceptable. People would not like to have huge binaries in the source codes tar. Also, this is not allowed by the ASF.
This tool should be downloaded through shell, rather than like this.
Like I said, this should not be this hard. Please reach RocketMQ community about the question I raised.

Why are you adding so many binaries and scripts for only setting a topic? The binaries are not allowed.

tool dependent on this binaries ,I'm not sure which ones can not be used, so I uploaded all of them

None of them is acceptable. People would not like to have huge binaries in the source codes tar. Also, this is not allowed by the ASF.
This tool should be downloaded through shell, rather than like this.
Like I said, this should not be this hard. Please reach RocketMQ community about the question I raised.

like download from RocketMQ Releases URL ，after execute shell ？

Maybe. My point is

Binaries are not allowed
If you need any binary, download it when the tests run
Ask the upstream for their suggestions. How do they set up test env? Do they always need so huge admin rather than a simple one-file CLI? I don't know, I am just guessing.

Another question is, why can't you use your demo codes to create the topic? I think /health request is the entrance point for your env preparation.

Another question is, why can't you use your demo codes to create the topic? I think /health request is the entrance point for your env preparation.

rocketmq-client-java not provided create topic and group Apis, can i use other client to create topic and group

Another question is, why can't you use your demo codes to create the topic? I think /health request is the entrance point for your env preparation.

rocketmq-client-java not provided create topic and group Apis, can i use other client to create topic and group

That is my point. You should have a simpler way to do so.

Supported-list.md should be updated as well.

Supported-list.md should be updated as well.

This is still missing. @xyyz150 And why do you add 5.1.0 and 5.1.1, are they different?

Supported-list.md should be updated as well.

This is still missing. @xyyz150 And why do you add 5.1.0 and 5.1.1, are they different?

i not sure the difference about 5.1.0 and 5.1.1, i just add test pass version
should i add Supported-list.md like this :

RocketMQ 5.x

Supported-list.md should be updated as well.

This is still missing. @xyyz150 And why do you add 5.1.0 and 5.1.1, are they different?

i not sure the difference about 5.1.0 and 5.1.1, i just add test pass version

Once tests are passed. Please remove 5.1.0. We don't want to cost too many CI resources.

should i add Supported-list.md like this :

RocketMQ 5.x

You should indicate 5.x gRPC client, and update previous RocketMQ to include 5.x as well. I think we have supported the legacy client?


Consider not reversing the task ID for the HostName

The task builder code calls toHostname which takes the path like structure of a task id, reverses it, and joins it back into a string.
This turns a marathon task id such as:
/linux/hastebin/redis
Into a mesos task with a name such as redis.hastebin.linux, and a hostname such as redis-hastebin-linux.marathon.mesos from mesos-dns.
Since we're using the first bit as the team that is running the application (and enforcing this via external systems), it would be nicer if there was a way to make the hostname from mesos-dns something more like: linux-hastebin-redis.marathon.mesos.
I realize this is relatively deep in the marathon code, but it would be excellent if there was a way to make this configurable.

@SEJeff thanks for the suggestion!

I am also interested in this since it makes any service discovery system confused.

Changing the task name seems easy to do in the code but the migration will be probably quite painful. How could we solve that ?

Note: This issue has been migrated to https://jira.mesosphere.com/browse/MARATHON-3008. For more information see https://groups.google.com/forum/#!topic/marathon-framework/khtvf-ifnp8.


Added checks effects interactions in security section

#5 Added section for checks effects interactions pattern in solidity guidelines.

@sarvesh-ost your commit message (or now the GitHub PR description) should include text like fixes #5 or resolves #5 or the likes to auto-close the related ticket. See also https://help.github.com/en/articles/closing-issues-using-keywords


Fix text and screenshots on welcome page

Fixed typos and added screenshots and gifs.

@rabia-pullflow @srzainab I fixed more issue with the welcome text.

Fixes #35

In assets/pr-chat.gif - can we focus on the PR thread on the slack side? Zoom in further?
On assets/pr-dashboard.png - can we show atleast one PR where the user is requested review?

When I zoomed in on the side it kept disappearing completely, a slack bug maybe

You can use this button to make it a separate window and then make it as big as you want
image.png

For the iconography image, we need to add this visual guide to the right for better visibility


ClusterLoader - Adding test, step, phase execution implementations

Simple implementations for ExecuteTest, ExecuteStep, ExecutePhase.

/lgtm


Mypy stubs and general maintenance

Adding testing for Python 3.10-dev
Changing to use tomli instead of toml by default as it is faster and toml 1.0.0 compatible
Fixing #189 by adding mappings for mypy

About tomli, Python 3.11 should add tomllib which is basically tomli but built-in.
Maybe in future toml could be dropped entirely and the import changed to something like:
try:     import tomllib except ModuleNotFoundError:     import tomli as tomllib

https://docs.python.org/3.11/library/tomllib.html

This module does not support writing TOML.

Will have to wait until it works for both reading and writing before it would be worth implementing IMO. And then will have to do a check for versions like this 3.11 for when it doesn't support writing. Them releasing it this early without writing abilities is quite a shortsighted headace.


Additional cli commands pin/search/find

It may be useful to add pin + find/search functionality

https://docs.chocolatey.org/en-us/choco/commands/pin
https://docs.chocolatey.org/en-us/choco/commands/find

not going to do pin, as it would require changes to chocolatey.config (we don't want to break stuff) - search on the other hand would be a non-invasive, query-only command that could benefit from similar optimizations as outdated.


Initialization of global pointer variable seems suspect with our current use of Renode.

@tensorflow/micro
I hit this issue while working on https://github.com/tensorflow/tensorflow/pull/46904:

If I create a global pointer variable (explicitly initialized to nullptr) and check for the variable == nullptr in my factory function, the check always returns false.

AFAICT, this behavior is specific to our use of Renode. I have not been able to reproduce on Linux or with the Xtensa simulator.
I have a workaround that should allow #46904 to be merged and I will then update this issue with a cleaner way to reproduce this error.

The reason is in _zero_initialize_bss_data (tensorflow/lite/micro/tools/make/downloads/stm32_bare_lib/source/startup.c). It initializes the BSS data with DEADBEEF.
To verify that:
Build with debug info
make -f tensorflow/lite/micro/tools/make/Makefile TARGET=bluepill BUILD_TYPE=debug test_renode -j`nproc`

Run in Renode:
include @tensorflow/lite/micro/testing/bluepill_nontest.resc; sysbus LoadELF @tensorflow/lite/micro/tools/make/gen/bluepill_cortex-m3_debug/bin/test_renode; machine StartGdbServer 3333

Start GDB:
tensorflow/lite/micro/tools/make/downloads/gcc_embedded/bin/arm-none-eabi-gdb tensorflow/lite/micro/tools/make/gen/bluepill_cortex-m3_debug/bin/test_renode

In GDB:
tar rem :3333
watch *0x20000008
mon s
c

It will break in the code that actually writes this data.
Now why do they go to bss instead of data? I don't know.
Here's an excerpt from the objdump:
Disassembly of section .data:

20000000 <a>:                        <- OK
20000000:   0000002d    andeq   r0, r0, sp, lsr #32

20000004 <init_to_true>:                        <- OK
20000004:   00000001    andeq   r0, r0, r1

Disassembly of section .bss:

20000008 <init_to_false>:                        <- NOT OK
20000008:   00000000    andeq   r0, r0, r0

2000000c <init_to_nullptr>:                        <- NOT OK
2000000c:   00000000    andeq   r0, r0, r0

20000010 <g_tick_count>:                        <- NOT OK
20000010:   00000000    andeq   r0, r0, r0

Disassembly of section ._user_heap_stack:

20000014 <._user_heap_stack>:
    ...

For the record, this is HAL specific, so it should behave the same way on HW, not only in Renode.

Changing 0xDEADBEEF to 0 of course fixes the problem

Your example @advaitjain has similar results on stm32f4 also built with stm32_bare_lib that causes the problem.
The -559038737 number that is printed as init_to_nullptr is exactly a signed int value for 0xDEADBEEF. The 239 in init_to_false is the ending 0xEF byte.
I remember stumbling upon an issue with this BSS initialization when code from such an if was never executed:
static TYPE* some_pointer = nullptr;
if (!some_pointer){
    // Code never called
}

The compiler simply assumed a static pointer is always nullptr (0x0) right after declaration and skipped the "additional" nullptr initialization while it was 0xDEADBEEF instead.
Now I can see there's no such problem anymore so perhaps the compiler has been fixed.
Nevertheless, this issue made me wonder again about what the purpose of that 0xDEADBEEF initialization is. The binaries work well without it. Perhaps it could be simply disabled @aselle @petewarden? https://github.com/google/stm32_bare_lib/blob/55bf49816f1a9dc7d9e35951c135e852ce7a98df/source/startup.c#L114

Thanks for the debugging @PiotrZierhoffer.
@ajelinski, I'm not sure about the thinking behind the decision to go with not zero initializing the .bss -- it does seem non-standard.
I have made https://github.com/tensorflow/tensorflow/pull/47382 that fixes this issue independent of upstream changes to STM32 Bare Lib and also answers @PiotrZierhoffer's comment about why the variables are ending up in .bss instead of .data
This link has some useful info as well: https://stackoverflow.com/q/8721475

Talked to @petewarden and we decided that changine STM32 Bare Lib is the way to go.

I have made #47382 that fixes this issue independent of upstream changes to STM32 Bare Lib and also answers @PiotrZierhoffer's comment about why the variables are ending up in .bss instead of .data
This link has some useful info as well: stackoverflow.com/q/8721475

#47382 originally added -fno-zero-initialized-in-bss and verified that the globals were in the .data section instead of .bss using the tips from https://stackoverflow.com/q/8721475
It has since been updated to simply pull in an updated version of STM32 Bare Lib (with the zero initialization of .bss)


Update README.md

Import example was missing the iterable import

Pull Request Test Coverage Report for Build 321

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-29.7%) to 50.693%

Totals

Change from base Build 307:
-29.7%

Covered Lines:
366

Relevant Lines:
722

💛  - Coveralls

Pull Request Test Coverage Report for Build 321

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-29.7%) to 50.693%

Totals

Change from base Build 307:
-29.7%

Covered Lines:
366

Relevant Lines:
722

💛  - Coveralls

Pull Request Test Coverage Report for Build 321

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-29.7%) to 50.693%

Totals

Change from base Build 307:
-29.7%

Covered Lines:
366

Relevant Lines:
722

💛  - Coveralls

Thanks @olegsu


Need more details on Hyper-V Isolation

So If I understood properly docker will create a virtual machine for each container and then runs a container inside?
If yes, Isn't this worse than before, where docker was running a single Linux kernel in virtual machine and then all the containers were sharing this kernel? This defeats the purpose of having lightweight containers. Secondly, why "Windows Server Containers" are deemed insecure while similar containers are highly trusted and popular in Linux world (LXC)?
I believe more emphasis needs to be done on the design on Hyper-V Isolation as namespace based containers are already popular and well understood by professionals

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

ID: 07b1cd55-0f91-e97c-06d3-afe54fb909d4
Version Independent ID: db485580-9c27-da7d-26f9-a60449cb5773
Content: About Windows Containers
Content Source: virtualization/windowscontainers/about/index.md
Service: windows-containers
Product: windows-containers
GitHub Login: @taylorb-microsoft

We hugely appreciate your contribution, feedback, and support to our technical documentation.
We’ve phased out GitHub Issues on this repository as the feedback mechanism for content and have replaced it with a new feedback system. For more information see, Content User Feedback. As part of this rollout, we are migrating your issue where our team will use your comments to improve the documentation experience. Unfortunately, this does mean you won't be able to access this issue after the migration is complete.
All issues will be migrated by Thursday 29th August 2024.
Once again, thank you for your valuable feedback, we hope you’ll continue to contribute using our new feedback system.


ci: Add conventional commits check and codeowners

I accidentally also pushed the docs change here. I'll open another PR for that.


Missing YAML Configuration File (Error 404)

Issue
In the Important Endpoints to Explore section, the documentation states that you should check all exposed services/ports in the Docker compose YAML definition.
Hyperlink Docker Compose YAML is Error 404.
Proof
Please find a screenshot of the missing reference:

Fixed by https://github.com/digital-asset/daml-platform-observability-example/pull/31


uavcan start enforcing code style

@davids5 @pavel-kirienko any objections? This brings the uavcan, uavcannode, and uavcanesc modules in line with the rest of PX4 formatting, and doesn't apply to libuavcan.

No objections, as long as this change doesn't affect the logic.
On Aug 25, 2017 17:03, "Daniel Agar"<EMAIL_ADDRESS>wrote:

@davids5 https://github.com/davids5 @pavel-kirienko
https://github.com/pavel-kirienko any objections? This brings the
uavcan, uavcannode, and uavcanesc modules in line with the rest of PX4
formatting, and doesn't apply to libuavcan.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/PX4/Firmware/pull/7856#issuecomment-324929361, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ADJUZKS6keMqGfU1FQSprw3eIYYKNAfNks5sbtQegaJpZM4PAzAx
.


Make server port configurable with .env

It is currently hard-coded in api/index.js

It is configurable
const LISTEN_PORT = process.env.PORT || 3000


Adjust the height of output graphs

Hi.  I have a graph with many nodes and I'd like to be able to see it in full.  by changing
out=widgets.Output(layout={'width': '800px',
'height': '800px','max_height':'100%',
'border': '1px solid black'})
out.display(cyto)
I get a bigger box for my graph but when I move the graph inside the box it gets cut in half (meaning the graph is limited to 400px no matter what).  Is there a way to adjust this?
Thank you,

The cytoscape object defines it's own dimensions in terms of pixels independent of it is parent element (in this case an output widget). Because the output widget and the cytoscape object are separate widget objects that don't really know how to talk to eachother changes will no propogate from one to the other. So you also need to modify the width and height. YOu can do that like so:
cyto.layout.height = '800px'
cyto.layout.width = '800px'

This strategy of setting object.layout.height I think will work for also most everything that is derived from an ipywidgetas the layout attribute is inherited from the widget superclass

Hi ianhi.  Works great now - Thanks a lot for the tip!


Build logs series of warning messages when downloading codehaus artifacts

the build tries to download from the codehaus repo which has shut down.
[WARNING] Could not transfer metadata org.codehaus.groovy:groovy-eclipse-batch/maven-metadata.xml from/to codehaus.org (http://repository.codehaus.org): Failed to transfer file: http://repository.codehaus.org/org/codehaus/groovy/groovy-eclipse-batch/maven-metadata.xml. Return code is: 410 , ReasonPhrase:Gone.
This does not fail the build luckily, but does look weird.

Done.


mongoid 7.1.0 issue with arguments

something stopped working with mongoid 7.1.0 gem release. It was still fine with 7.1.0.rc0
irb(main):001:0> Address.last
Traceback (most recent call last):
        4: from (irb):1
        3: from app/models/address.rb:3:in `<main>'
        2: from app/models/address.rb:8:in `<class:Address>'
        1: from app/models/address.rb:8:in `include'
ArgumentError (wrong number of arguments (given 7, expected 1))

address.rb
# frozen_string_literal: true

class Address
  # == Inclusions ===========================================================

  include Mongoid::Document
  include Mongoid::Timestamps
  include Mongoid::Locker

Maybe Ruby version-related? Either way try to PR something?

I will debug more tomorrow. It only happens when include Mongoid::Locker is included. And also only when upgrading from mongoid 7.1.0.rc0 to 7.1.0. We would have to check what changes they made from rc0 to 7.1.0

I created a new application
rails new mongoid-locker-test --skip-active-record --skip-bundle --api

added the following gems
bundle add mongoid mongoid-locker

added a test model
class Test

  include Mongoid::Document
  include Mongoid::Locker

  field :address, type: String

end

getting the same error
irb(main):002:0> Test
Traceback (most recent call last):
        5: from (irb):1
        4: from (irb):2:in `rescue in irb_binding'
        3: from app/models/test.rb:3:in `<main>'
        2: from app/models/test.rb:7:in `<class:Test>'
        1: from app/models/test.rb:7:in `include'
ArgumentError (wrong number of arguments (given 7, expected 1))

repository is here: https://github.com/SimonHausdorf/mongoid-locker-test

Hello.

Hey :)

@SimonHausdorf began write and pressed ctrl+enter to go to new line. I have finished the comment.

@SimonHausdorf this is known issue, see workarounds here

About strange querying behavior, something similar
Strange querying behavior in mongoid v7.0.5

👋 I'm also getting this error with mongoid 6.4.5 and mongoid-locker 2.0.1. I couldn't find any workaround to make it work.
SomeModel.first

ArgumentError: wrong number of arguments (given 3, expected 1)
from /Users/pablo.cantero/.asdf/installs/ruby/2.5.3/lib/ruby/2.5.0/forwardable.rb:132:in `instance_delegate'

    mongoid (6.4.5)
    mongoid-locker (2.0.1)

I found a workaround using Module.delegate instead of delegate to ensure it uses the Rails delegate helper.


cursor too large when focus in textfield

in safari when is textfield focus, the cursor is as large as the wrapper div ,that's not beautiful
and when i delete the line-height in inputBase , the cursor the same height as input.

[ ] This is not a v0.x issue.
[ ] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior 🤔

Current Behavior 😯

Steps to Reproduce 🕹

Link:

Context 🔦

Your Environment 🌎

Tech
Version

Material-UI
v3.6.1

React
16.6.3

Browser
safari

TypeScript

etc.

@FionaWong I can't reproduce:

@FionaWong Could you provide more details?


Erroneous tab space calculation with static function prototypes

Describe the bug
When tab-aligning two function prototypes, one static and the other not, the space used by the static keyword seems to be ignored.
Erroneous code
Take this example code.
static double	calculate(double nb, double approx, double prev_approx)
{
	if (approx == prev_approx)
		return (approx);
	return (calculate(nb, (approx + nb / approx) / 2, approx));
}

double			ft_sqrt(double nb)
{
	if (!nb)
		return (0);
	return (calculate(nb, 50, 0));
}

In order to visually align the two functions prototypes in  a properly configured editor (ie. not in github visualisation 🙃), there has to be one tab between static double and calculate, and three tabs between double and ft_sqrt, as shown here.

However, the norminette raises the following error about the second prototype :
Error: TOO_MANY_TABS_FUNC   (line:   8, col:  13):      extra tabs before function name
The error persists until there's only one tab left in the declaration of the second function, which leaves the result unaligned.
This result would however work if the keyword static wasn't present in the first prototype.
This behaviour has corresponded to all my tests around this issue, so I'm pretty confident that's it.
Additional infos

OS: MacOS Mojave
python --version: Python 3.7.4
norminette -v: norminette 3.3.2

I have noticed the same behaviour for the const keyword.

Hello,
Functions no longer need to be tab aligned. you simply need to put a single tab after a func proto/decl


Bundle of paratext projects

Build a list of paratext projects in a private repo that we can test parsing and usfm generation - and run it for every release automatically.

This has become less on an issue, as USFM issues have (finally) moved to the background.


Realtime functionality cannot be used for insert many operations.

Currently the realtime module doesn't work if for the insert many operation. We should get that fixed.

Fixed in v0.12.0


fix python3 and pytorch.4 compatibality issues

Hi
I fix some python3 and pytorch.4 incompatibility issues. Beside it used cuda as default which is fixed by simple if statements.
hope it help.

I use the same version python and pytorch to run codes for detecting face from multi-images, but GPU memory usage is increasing as image processing, it seems that this version of code still have some bugs, I would appreciate if you can give advices, thank you.


Zooming with sync charts makes yaxis borders appear

Codesandbox: https://codesandbox.io/s/react-apex-chart-83wbh
I have two synced charts, one of those has the borders hidden for multiple series. The main issue is that when I zoom in any of the charts, the hidden property seems to disappear and the borders are shown.
This is how the chart initially looks:

If I use the zoom in the bigger chart it ends up like this:

If I use the zoom in the smaller chart it ends up like this:

It also breaks using the pan tool.
Funny thing is that this issue wasn't happening in the 3.6.8 version.
Is there any workaround for this?

Thanks for reporting.
I will soon look into it.

I've noticed that in version 3.6.8 the same issue happens only when using the pan tool.


Need Mic/Video permissions

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Due to many complaints about privacy this permission was removed in 0.2.2


SlotInvalidRangeException: Slot 8 not in valid range - [0,8)

SlotInvalidRangeException: Slot 8 not in valid range - [0,8)
https://crashy.net/f8c20513-dafb-4bca-9a0c-19ee4b2dcb24

Hi @m0thknight,
Thanks for reporting this issue, it will be fixed by the implementation of fabric transfert api : #104.


No Ping/IP showing after updating to v2.0

Hi, I just updated to 2.0 on windows 10, no VPN (steam version) and I always see 0ms ping now and no IP address in connected to:
I'm also on the crystal datacenter
I found this stack trace in my dalamud log:
2022-01-28 17:13:08.421 -05:00 [ERR] Unobserved exception in Task.
System.AggregateException: A Task's exception(s) were not observed either by Waiting on the Task or accessing its Exception property. As a result, the unobserved exception was rethrown by the finalizer thread. (Object reference not set to an instance of an object.)
---> System.NullReferenceException: Object reference not set to an instance of an object.
at PingPlugin.PingTrackers.PingTracker.AddressUpdateLoop(CancellationToken token) in K:\arashiiro\PingTrackers\PingTracker.cs:line 68
--- End of inner exception stack trace ---
2022-01-28 17:13:08.429 -05:00 [INF] LASTEXCEPTION:eyJXaGVuIjoiMjAyMi0wMS0yOFQxNzoxMzowOC40MjY5OC0wNTowMCIsIkluZm8iOiJTeXN0ZW0uQWdncmVnYXRlRXhjZXB0aW9uOiBBIFRhc2sncyBleGNlcHRpb24ocykgd2VyZSBub3Qgb2JzZXJ2ZWQgZWl0aGVyIGJ5IFdhaXRpbmcgb24gdGhlIFRhc2sgb3IgYWNjZXNzaW5nIGl0cyBFeGNlcHRpb24gcHJvcGVydHkuIEFzIGEgcmVzdWx0LCB0aGUgdW5vYnNlcnZlZCBleGNlcHRpb24gd2FzIHJldGhyb3duIGJ5IHRoZSBmaW5hbGl6ZXIgdGhyZWFkLiAoT2JqZWN0IHJlZmVyZW5jZSBub3Qgc2V0IHRvIGFuIGluc3RhbmNlIG9mIGFuIG9iamVjdC4pXHJcbiAtLS0+IFN5c3RlbS5OdWxsUmVmZXJlbmNlRXhjZXB0aW9uOiBPYmplY3QgcmVmZXJlbmNlIG5vdCBzZXQgdG8gYW4gaW5zdGFuY2Ugb2YgYW4gb2JqZWN0LlxyXG4gICBhdCBQaW5nUGx1Z2luLlBpbmdUcmFja2Vycy5QaW5nVHJhY2tlci5BZGRyZXNzVXBkYXRlTG9vcChDYW5jZWxsYXRpb25Ub2tlbiB0b2tlbikgaW4gSzpcXGFyYXNoaWlyb1xcUGluZ1RyYWNrZXJzXFxQaW5nVHJhY2tlci5jczpsaW5lIDY4XHJcbiAgIC0tLSBFbmQgb2YgaW5uZXIgZXhjZXB0aW9uIHN0YWNrIHRyYWNlIC0tLSIsIkNvbnRleHQiOiJVbm9ic2VydmVkIGV4Y2VwdGlvbiBpbiBUYXNrLiJ9
2022-01-28 17:13:08.429 -05:00 [ERR] Unobserved exception in Task.
System.AggregateException: A Task's exception(s) were not observed either by Waiting on the Task or accessing its Exception property. As a result, the unobserved exception was rethrown by the finalizer thread. (Value cannot be null. (Parameter 'address'))
---> System.ArgumentNullException: Value cannot be null. (Parameter 'address')
at System.Net.NetworkInformation.Ping.CheckArgs(IPAddress address, Int32 timeout, Byte[] buffer, PingOptions options)
at PingPlugin.PingTrackers.ComponentModelPingTracker.PingLoop(CancellationToken token) in K:\arashiiro\PingTrackers\ComponentModelPingTracker.cs:line 23
--- End of inner exception stack trace ---

Fixed, will release an update in a minute 👍


Item with direct include inside target causes the metadata not getting evaluated

Try out below simple project file
<?xml version="1.0" encoding="utf-8"?>
<Project DefaultTargets="AugmentItemGroup" xmlns="http://schemas.microsoft.com/developer/msbuild/2003" ToolsVersion="4.0">

  <Target Name="AugmentItemGroup">
    <ItemGroup>
      <ItmGrp Include="E:\temp\**\*.txt">
        <Dest>%(RecursiveDir)%(FileName)%(Extension)</Dest>
      </ItmGrp>
    </ItemGroup>

    <Message Text="%(ItmGrp.FullPath) to %(ItmGrp.Dest)" />
  </Target>
</Project>

Above Dest metadata doesn't get evaluated.

That is a confusing discrepancy between items inside targets and outside. I bet @cdmihai will be interested/annoyed by this.
As a workaround you can add the items, then update them in place:
    <ItemGroup>
      <ItmGrp Include="**\*" />
      <ItmGrp>
        <Dest>%(RecursiveDir)%(FileName)%(Extension)</Dest>
      </ItmGrp>
    </ItemGroup>

I have already known this workaround. I asked this inside Microsoft as well. This issue was opened after I asked inside Microsoft. You guys suggested filing issue in github. But it was half year passed. No response/action;

Sorry for the slow response, @cppcraze! This is on our radar. :)
Please do keep filing bugs on here when/if you run into them.
-MSBuild team

@rainersigwald any update on this issue? We just hit it somewhere in corefx and it is pretty annoying to diagnose

Any plans to have a look at this?


Allow loading scene file from custom NSBundle

Currently all scene files are loaded from the main bundle:
https://github.com/tangrams/tangram-es/blob/master/platforms/ios/src/TangramMap/platform_ios.mm#L35
But, in the sdk, as we move from using submodules to pods for the dependent stylesheets we will want to pass in custom bundles to load the scenes from. Could we provide a version of resolvePath which takes a bundle parameter?
NSString* resolvePath(const char* _path, NSBundle bundle)

https://developer.apple.com/reference/foundation/bundle
https://guides.cocoapods.org/syntax/podspec.html#resource_bundles
https://www.cocoanetics.com/2012/05/resource-bundles/

Yep it is be possible to add fallback resolution that would target the iOS SDK bundle. We had a similar way to handle framework resources for fonts that were bundled into the Tangram framework, until we added platform system fonts where we moved away from it.
What we would need is an optional setter on TGMapViewController to add a bundle fallback, if the resource hasn't been found from the main bundle, then we would try url resolution from this fallback bundle.

For this we might provide an option to set the root to which resources should be resolved against. If empty we would fallback to the main bundle (which would be the mainBundle we already use), otherwise it would use it as the root. @sarahlensing how do you feel about this?
From the iOS SDK that would look like that:
// com.mapzen.iosSDK is the name ios SDK framework
NSBundle* sdkFramework = [NSBundle<EMAIL_ADDRESS>
[tangramViewController setResourceRoot:[sdkFramework resourceURL]];

Bundles can be treated as file paths right? Would this issue be solved by using the absolute file path of a resource in a bundle? Resources that specified in the scene with relative paths will be correctly resolved against the scene's absolute path.

Would this issue be solved by using the absolute file path of a resource in a bundle?

Yes, in the proposed example setResourceRoot would be setting an absolute path to the resource in the bundle to use.

@karimnaaji I think that will work well. One thing we might want to consider is coupling its exposure with loading a scene file since the only time we care about resource bundles is when we are loading scene files.
So something like:
NSBundle* sdkFramework = [NSBundle<EMAIL_ADDRESS>
[tangramViewController loadSceneFile:sceneFile resourceUrl:resourceUrl]


Logger integration?

Most of the time I want to debug something in Joken I end up adding some logging. The Logger framework is awesome and I it is easy to integrate it in an application's config.
When disabling it in config, since all its functions are macros, they are not even included in the final AST. This is real neat!
It includes a per application configuration so we could assume a default to ERROR only and advise in the README that while developing it is good to enable it in a :dev config.
Thoughts?

I agree we can adding logging. It will not only help us, but also anyone debugging their applications. The only reason I never put in logging is because I think it can clutter up code, but that's not a very good reason if it's done right.

I'd add Logging as debug on verifying and signing for now. That is as simple as:
require Logger
Logger.debug fn -> "Verifying  #{inspect compact} using #{inspect signer}" end
Logger.debug fn -> "Signing #{inspect token} using #{inspect signer}" end

Fixed in #81.
All we need is to discuss if we should log the key or not.


fix: accept cid encoded peer ids in peerIdFromString

Description
Fixes https://github.com/libp2p/js-libp2p/issues/2772
Change checklist

[x] I have performed a self-review of my own code
[x] I have made corresponding changes to the documentation if necessary (this includes comments as well)
[x] I have added tests that prove my fix is effective or that my feature works

@wemeetagain @achingbrain this could use a peek from one of you

Closing as detailed in https://github.com/libp2p/js-libp2p/issues/2772#issuecomment-2428941432


I have problem in. running filament

Package
filament/filament
Package Version
v2.14.5
Laravel Version
v8.0.0
Livewire Version
v2.0.0
PHP Version
PHP 8.0.0
Problem description
I got this error
Package filament/filament has requirements incompatible with your PHP version, PHP extensions and Composer
version:
- filament/filament v2.14.5 requires php ^8.0 which does not match your installed version 7.3.29.
my mac have already php 8 version and the project is also on php 8 but i dont know why I got this problem. If you please give me detailed steps
Expected behavior
my mac have already php 8 version and the project is also on php 8 but i dont know why I got this problem. If you please give me detailed steps
Steps to reproduce
my mac have already php 8 version and the project is also on php 8 but i dont know why I got this problem. If you please give me detailed steps
Reproduction repository

Relevant log output
No response

I have installed it on my Mac already
Sent from my iPhone

On Jul 25, 2022, at 6:09 PM, Zep Fietje @.***> wrote:

@zainabalanjawy, are you sure that you're running PHP 8.0+, because the error is quite clear?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.

You may have installed it, but your CLI is probably still running an old version

How should I let my cli run a specific version of php?
Sent from my iPhone

On Jul 25, 2022, at 6:38 PM, Dan Harrin @.***> wrote:

You may have installed it, but your CLI is probably still running an old version
Run php -v
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.

How should I let my cli run a specific version of php?
…

Depends on how you've installed PHP (using Homebrew for example). If you perform a Google search for your exact setup, I bet you'll find it! 😄


feat: add replace-row-column-with-grid codemod

Proposed behaviour
The Row and Column components have been deprecated in favour of GridContainer and GridItem components respectively. This codemod converts all Row and Column instances to GridContainer and GridItem
The codemod will also replace Row props with their GridContainer equivalents:

gutter will be changed to gridGap
columns will be changed to gridTemplateColumns

and Column props with their GridItem equivalents:

columnSpan will be changed to gridColumn, only literal values will be changed, e.g.:columnSpan={3}, columnSpan="3", {...{ columnSpan: 3 }}, {...{ columnSpan: "3" }}
columnAlign will be changed to justifySelf only literal values will be changed, e.g.: columnAlign="left", {...{ columnAlign: "left" }}

- import { Row, Column } from "carbon-react/lib/components/row";
+ import { GridContainer, GridItem } from "carbon-react/lib/components/grid";

- <Row gutter="medium" columns="10">
-   <Column columns="10" columnAlign="left" columnSpan="3" columnOffset="1">col1</Column>
-   <Column columns="10" columnAlign="center" columnSpan="2">col1</Column>
-   <Column columns="10" columnAlign="middle" columnSpan="2">col3</Column>
-   <Column columns="10" columnAlign="right" columnSpan="2">col3</Column>
- </Row>
+ <GridContainer gridGap="15px" gridTemplateColumns="repeat(10, 1fr)" columnDivider>
+   <GridItem justifySelf="start" gridColumn="2 / span 3">col1</GridItem>
+   <GridItem justifySelf="center" gridColumn="span 2">col1</GridItem>
+   <GridItem justifySelf="center" gridColumn="span 2">col3</GridItem>
+   <GridItem justifySelf="end" gridColumn="span 2">col3</GridItem>
+ </GridContainer>

Current behaviour
NO codemod exists
Checklist

[x] Release notes (Conventional Commits)
[x] Unit tests
[x] Readme updated

Additional context
FE-3569
Testing instructions
Follow instructions in README to run npx carbon-codemod replace-row-column-with-grid <target>

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Snackbar can only have one child element

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Current behavior 😯

Expected behavior 🤔
should show both elements in the snackbar
Steps to reproduce 🕹
Steps:

create a snackbar with more then one child element
run

Context 🔦
have many alert notification stacking as they come in
Your environment 🌎
your example here https://codesandbox.io/s/0l66jg?file=/demo.js

I have find solution about this. we need to wrap up multiple component inside snackbar into one div. like this
<Snackbar open={open} autoHideDuration={6000} onClose={handleClose}>
           <Alert             onClose={handleClose}             severity="success"             sx={{ width: "100%" }}           >             This is a success message!           </Alert>
kop

       </Snackbar>

Yes, this a good temporary fix but it should still be able to handle
multiple children or display a better error to let people know what the
problem is
On Thu, Jun 9, 2022, 12:50 PM smitgol @.***> wrote:

I have find solution about this. we need to wrap up multiple component
inside snackbar into one div. like this

<Alert onClose={handleClose} severity="success" sx={{ width: "100%" }} >
This is a success message!  kop
—
Reply to this email directly, view it on GitHub
https://github.com/mui/material-ui/issues/33082#issuecomment-1151367318,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABWSM444OQ3L63HXIIWR763VOIODXANCNFSM5YKLWNJA
.
You are receiving this because you authored the thread.Message ID:
@.***>

May I ask for an exemplary scenario in which you would like to pass multiple children? You can simply pass 1 component by wrapping them around with some thing. e.g.,

    <Alert ... />

Maybe we can improve description of children prop in API documentation?

My use case was as notifications got pushed I would simply loop and add
them to the snake bar
On Wed, Jun 15, 2022, 3:43 AM Benny Joo @.***> wrote:

May I ask for an exemplary scenario in which you would like to pass
multiple children? You can simply pass 1 component by wrapping them around
with some thing. e.g.,

Maybe we can improve description of children prop in API documentation?
—
Reply to this email directly, view it on GitHub
https://github.com/mui/material-ui/issues/33082#issuecomment-1156109788,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABWSM47C26TNE33PX3JEFNTVPGCSBANCNFSM5YKLWNJA
.
You are receiving this because you authored the thread.Message ID:
@.***>

also called children as in multiple not child. React does that on purpose .

My use case was as notifications got pushed I would simply loop and add
them to the snake bar.

You can simply add a div element inside the Snackbar component, and then do the loop / add inside the div?


Is this project dead?

Should this project be transferred to another group of supporters?

It looks dead, now, it won't even start in ubuntu 22.04 snap...


You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Typo in documentation for is function

The very start of the [is](https://docs.tsafe.dev/is) function's documentation says:
is is meant to be used in conjonction with assert and enable you to tell the compiler:
"Trust me this `valule` is of type `T`" or "Trust me this `value` is not of type `T`"

This should be:
`is` is meant to be used in conjunction with `assert` and enables you to tell the compiler:
"Trust me, this `value` is of type `T`" or "Trust me this `value` is not of type `T`"

This is one of a few typos I have noticed, as well as missing commas in the import line on assert:
// The website says
import { assert AssertionError } from "tsafe/assert";
// Should say
import { assert, AssertionError } from "tsafe/assert";

If it is possible, I would be happy to submit a pull request to modify these typos, however I cannot seem to find where these files are stored. If that is not a possibility, I can collect all the typos noticed into a single issue, such as this one, for easy reference and fixing.

I found the docs_v1 branch, which appears to be where the documentation files are stored, but there are many duplicate files, so I'm not sure which files need to be edited and which are auto-generated. A pointer on the workflow of editing the docs page would be greatly appreciated!

@garronej What exactly is the anegdotal and main folders for? It seems that the website lives at the root directory, and if I attempt to navigate to the files in either folder, the site redirects to the root directory.

@aarondill,
They are legacy pages.
The MD files are generated automatically by GitBook.
I just sent you an invite by email for if you want to edit directly on GitBook.
In any case, thank you for your help!


Forbid unsafe code

On the tin.

Codecov Report

Merging #104 (f7a52c5) into dev (b0f6f27) will increase coverage by 0.07%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##              dev     #104      +/-   ##
==========================================
+ Coverage   54.10%   54.18%   +0.07%
==========================================
  Files          34       34
  Lines        6825     6814      -11
==========================================
- Hits         3693     3692       -1
+ Misses       3132     3122      -10

Impacted Files
Coverage Δ

cli/src/lib.rs
0.00% <0.00%> (ø)

cli/src/main.rs
0.00% <ø> (-50.00%)
:arrow_down:

cli/src/regtest.rs
0.00% <0.00%> (ø)

config/src/lib.rs
36.80% <ø> (ø)

lib/src/lib.rs
0.00% <ø> (ø)

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


Process get_software_versions fails for trinity

When running in a conda environment on macOS (10.14.6) the get_software_versions fails when trying to get the software version for trinity.

Trinity has a --version flag, but returns an exit code of 1 when run with that, which makes the process fail. That's why I resorted to the current "peeking into the script" solution. Can you, @emnilsson, try to run parts of the command to see where it fails for you?

Fixed in c4a1a3e


[Log in] The ‘Відновити пароль’ page does not meet the design

Environment: Win10 v21H2 64-bit, Firefox v 97.0
Reproducible: always
Build found: frontend: gcp-0.0.53, backend: gcp-0.0.112
Preconditions

Go to OoS (https://oos.dmytrominochkin.cloud/#/)
Click on the ‘Увійти/Зареєструватися’ button

Steps to reproduce
Click on ‘Забули пароль?’ button
Actual result
A user is redirected to the page ‘Відновити пароль’ with English and Ukrainian names of the fields and buttons, no footer and header are displayed

Expected result
A user is redirected to the page ‘Відновити пароль’ that meets the design
https://www.figma.com/file/YRYP4zAAchOyYMgdoeP782/MoE-|-Dev-Team?node-id=4006%3A722
Note: there is a typo on the design – instead ‘Відновити паролю’ it shall be ‘Відновити пароль’
User story and test case links
User story #313
User story #45
Test case OS-299
Labels to be added
"Bug", "pri: medium", "severity:minor"), Type "UI"

@Olya-web7 The text is inconsistent with the design (highlighted on the screen below), could you please fix

Verified and closed


Request: LerpAngle on MathUtils.

I would write a PR for this, since I found myself needing it more than once this week, but can't guarantee the solution I found to the problem is performant enough.
For more info, this is what I'm requesting.
And this is the best way that I found to do it . Anyone know a better way of doing it?

Looks useful, would you mind to send a PR for it?

Make sure to handle degrees and radians!


[berkeley] Make libp2p_keypair CLI argument optional again

This PR makes the --libp2p-keypair CLI argument optional, to avoid the irritating setup steps when first running a Mina node.
This option was added to ensure persistence will interact correctly with bitswap: if a node restarts, the network's knowledge of its persisted data will be preserved, and it doesn't need to re-advertise the contents of its store. However,

bitswap is currently enabled, so this annoyance is for nothing, and
if persistence isn't enabled for the node, re-using the libp2p keypair will mislead the network into believing that it has data that it doesn't.

After some discussion, it appears that

converting the flag back to optional immediately does no harm and improves UX considerably, and
we can solve the issues properly by persisting the libp2p keypair alongside the data, so that we have the correct behaviour both with and without persistence enabled.

It would be best to make this change for the mainnet berkeley release candidate, to ensure that the upgrade is as smooth and easy for node operators as possible.

!ci-build-me

!ci-build-me

!approved-for-mainnet


feat(openapi): added example value support for param/schema

Example Code

struct Api;

fn hello_world() -> String { String::from("Hello World") }

#[OpenApi]
impl Api {
    #[oai(path = "/", method = "get")]
    async fn get(
        &self,
        #[oai(example = "hello_world")] name: Query<String>,
    ) -> PlainText(String) {
        PlainText(name.0)
    }
}

I have also added test for it in poem-openapi/tests/operation_param.rs

fmt failed~

let me fix it. there should be a git pre commit hook for this.

thanks! 🙂


USSD registration validate day and date

Affecting current states 6 and 14.
Add additional state for invalid date.

@gsvr please review

@NicolaasDieBaas Done for now.

@gsvr Hope it's good now :)

@NicolaasDieBaas Please run npm test again - I don't think the compiled files were updated for the comment removal. :+1: when that's done.


[$500] Search - LHN displays search results when searching for contacts in New Chat modal

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

Version Number:  1.4.24-0
Reproducible in staging?: Y
Reproducible in production?: Y
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
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:

Log in with HT account (so that it loads slower during login)
White the app is loading, open new chat modal via CMD+SHIFT+K
Search for a user that has IOU, group chat, other history

Expected Result:
While the search results appear in New Chat modal, LHN in the background should display the chats
Actual Result:
White the search results appear in New Chat modal, LHN in the background displays search results too which include IOU reports, group chats etc
Workaround:
Unknown
Platforms:

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

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/f0afb221-16df-475d-bb91-cf734fb38a8f
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~013d4b0c23b2adb57d
Upwork Job ID:<PHONE_NUMBER>483092992
Last Price Increase: 2024-01-10

Waiting on proposals.

Still waiting on proposals.

@c3024 are you able to reproduce this bug?

Closing.


Delete_similar() not working in 10.1.X

Describe the bug
A script using .delete_similar() worked against 9.1.X but was recently discovered to not be working against 10.1.7. The script has not changed. Only the PAN-OS upgrade.
Expected behavior
Cycling through a dict of DeviceGroups as keys and a list of SecurityRule objects as the value:
for dg in dictRules:
    dictRules[dg][0].delete_similar()

This would do an atomic delete per Device Group
Current behavior
Traceback (most recent call last):
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/panos/base.py", line 3878, in method
    super_method(self, *args, **kwargs)
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/pan/xapi.py", line 733, in delete
    self.__type_config('delete', query, extra_qs)
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/pan/xapi.py", line 805, in __type_config
    raise PanXapiError(self.status_detail)
pan.xapi.PanXapiError: The request could not be handled

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "<pyshell#54>", line 2, in <module>
    dictDisabled[dg][0].delete_similar()
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/panos/base.py", line 1993, in delete_similar
    dev.xapi.delete(
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/panos/base.py", line 3899, in method
    raise the_exception
panos.errors.PanDeviceXapiError: The request could not be handled

Possible solution
Steps to reproduce
Screenshots
Context
This is part of an automation script that is used to disable a csv list of rules and then on a designated day of the month, the script is used to delete those rules if still disabled
Your Environment
Tested with multiple versions of lxml, pan-os-python, and pan-python (0.16.0, 0.17.0).

I am seeing this same issue on 10.1.10.

This is not linked to pan-os-python but to PAN-OS itself.
After getting some feedback from the TAC, the bulk delete XML API calls have been removed voluntarily starting 10.X because of some wrong behaviour on some objects caches.
It has been restored (PAN-179059) on the following releases (not tested on my side) :

10.0.10
10.1.5
10.2.2-h2

I can confirm that .delete_simillar() works as expected on 10.2.4-h3.

This issue seems to have returned in 10.2.7.

delete_simillar has stopped working again on 10.2.5 and greater in the 10.x.x release train. It also does not work on 11.0.3.

I got this feedback from Palo Alto support.
"The support for XML API requests to delete multiple security policies at one shot by, passing the policy names separated by the 'or' operator in the x-path is no longer available."


Confirmation from client on content

We need to have confimation from client on a few things in order to progress and finish other tasks.
Recieve confirmation from client on:

[x] Logo design
[x] Proof reading in malagasy
[ ] QA confirms information provided in comment on task

Once these are confirmed add a comment here with the information provided from the client and move this to QA review.

Client wants to use the original logo.

They have changed a couple things in the text on Malagasy and will send it on e-mail ASAP.

Text added to the MFF-Fed-Jan24 notebook under Developing>Languages.


ng-Class styling and md-row-select incompatible

Implementing a mouseover function which turns table rows into another color as the user moves the cursor around the table. Only works when md-row-select is omitted OR "class" is used instead of "ng-class". Is there a way to keep both ng-class and md-row-select? (Need ng-class to enable conditional styling)

I needed this too and found a pretty simple way to do this:
In the toggleRow method of the md-table-row directive, add and remove the md-selected class to the row element. Next, in the md-select-row directive, comment out the line that adds the ng-class directive in the template function.
With this change, the md-table-row directive handles the class add/remove without the need for the ng-class which leaves it free for you to use.

+1 I would like to use ng-class in a tr balise

In v0.9.x the .md-selected class is toggled by the directive so you are free to use ng-class on a table row.


Ive held down the self cast key and it refuses to work

ive even used it on other keys like Mouse 3 and it refuses to work.
help?

Some of the spells can't be self casted. As for those that can, check this to see what your bind is. If you are on a server, ask what they have as the default bind.

or;

I know there is a file to change the key but try this for now.


Immutable (vs mutable) types defined

I'm seeking a C# Guide definition for "immutable type" for a Blazor article, where I would like to link the text "immutable" to a formal definition. However, I only see "immutable (type)" used to describe types in passing and in other general passing remarks, not a formal definition of "immutable type" versus "mutable type." When "immutable (type)" is used, it's never linked to a definition, including to any non-MS page. Is this by-design, or is it missing coverage that should be added (somewhere, not necessarily to this particular topic)? If by-design, can you recommend a link destination for "immutable (type)" for the Blazor article?
The context for the cross-link is the term "immutable types" in the following bullet point ...

All of the parameters are of known immutable types or any [primitive type](/dotnet/api/system.type.isprimitive), whether nullable or nonnullable, and haven't changed since the previous set of parameters were set.

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

ID: 1991c89a-9728-ea31-a999-79b3cb331372
Version Independent ID: 9aa4f01b-a5a0-3cb1-3c22-db333c0d9f11
Content: The C# type system
Content Source: docs/csharp/fundamentals/types/index.md
Product: dotnet-csharp
Technology: csharp-fundamentals
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

I also need to figure out what to do with the word "known," another cryptic word here for newbies. "Known" by who? What is "known" about the type?
Since I also don't see a clean C# definition for "known type," I wonder if I can remark that a known type implements System.Type.
I don't think the sentence works well with many embedded parenthetical cross-links. I'm inclined to handle this with ye 'olde dagger ...

All of the parameters are known immutable types† or any [primitive type](/dotnet/api/system.type.isprimitive), whether nullable or nonnullable, and haven't changed since the previous set of parameters were set.
†An *immutable type* is a [`readonly`](/dotnet/csharp/language-reference/keywords/readonly) type that prevents a member from changing any property or field value after the type is constructed. A known type is a type that implements <xref:System.Type?displayProperty=fullName>.

Is that † remark accurate?

Are those † remarks accurate?

No. I should update that language.  I meant that the type is one the compiler can verify as immutable. Those are a few types:

The builtin unmanaged types (int, char, float, and so on).
readonly struct types.
Types where every member field is a type the compiler can verify as immutable and is marked as readonly.

That third rule is the one that gets involved. For examples:
// This is verifiably immutable: All members are `readonly` and are themselves verifiably readonly:
public struct Point
{
    public readonly double X;
    public readonly double Y;
    public Point(double x, double y) => (X, Y) = (x, y);
}

// This is not verifiably immutable. Its members are readonly, but they are not verifiably immutable
public class Stuff
{
    // OK, the variable `messages` can't change, but the list itself isn't immutable (its contents could change.)
    private readonly List<string> messages = new List<string>();
}

Excellent ... thx. Then, you'd feel good if I went with ...

All of the parameters are [verifiably immutable types](/dotnet/csharp/language-reference/builtin-types/unmanaged-types) or any [primitive type](/dotnet/api/system.type.isprimitive) that hasn't changed since the previous set of parameters were set.

That's good by me @guardrex

I'm confused: What does immutability have to do with unmanaged? References of the unmanaged types are all mutable.

I read through dotnet/AspNetCore.Docs#25776 and I'm not sure the best resolution.
It might make the most sense to not link between the C# docs and the Blazor docs for this concept. I had been answering from the perspective of the C# language rules. In the language, readonly struct types are the only types guaranteed to be immutable. (To your side note, the runtime team has added the readonly modifier on may of the unmanaged types (Int32, etc).
If I understand correctly, and it's a set of known types for the Blazor ComponentBase, it's probably simplest just to list them. That way you avoid any confusion.
Would that fix the issue?

Perhaps .....

* All of the parameters are known† [verifiably immutable types](/dotnet/csharp/language-reference/builtin-types/unmanaged-types) or any [primitive type](/dotnet/api/system.type.isprimitive) that hasn't changed since the previous set of parameters were set.
†Blazor's framework uses a set of built-in rules and explicit parameter types for change detection. These rules and the types are subject to change at any time. For more information, see the [`ChangeDetection` API in the ASP.NET Core reference source](https://github.com/dotnet/aspnetcore/blob/main/src/Components/Components/src/ChangeDetection.cs).

I'll close this.  I think we've reached the right resolution on the Blazor docs, and the C# language allows, but does not have general syntax to enforce immutable types . (readonly struct does enforce immutable struct types, but that's not general to other types).


Add to tensor definition

Add a small snippet to the tensor definition for the sake of correctness. People always ask about how these are related to tensors as physicists and mathematicians use.

Can one of the admins verify this patch?

I signed it!

Closing.


[bug] MacOS titlebar height inclusive while Linux and Windows are exclusive

Describe the bug
The height of a window being set in the Tauri config is not respected across all platforms. MacOS considers the title-bar as inclusive of the window height, while Windows and Linux do not.
tauri.conf.json
"windows": [
  {
    "title": "Icon Frame",
    "theme": "Dark",

    "width": 900,
    "maxWidth": 900,
    "minWidth": 900,

    "height": 632,
    "maxHeight": 632,
    "minHeight": 632,

    "fullscreen": false,
    "resizable": false
  }

Linux window breakdown

Windows window from dev-tools

MacOS window with the inclusive title-bar

Changing the height inside the tauri.conf.json from 632 to 660 to compensate for the 632 - 604 difference does seem to resolve the issue for the moment.
Reproduction

Open the application on MacOS

Expected behavior
A consistent window height across all 3 supported platforms
Platform and versions
Environment
› OS: Fedora 37.0.0 X64
› Node.js: 19.3.0
› npm: 9.2.0
› pnpm: Not installed!
› yarn: Not installed!
› rustup: 1.25.2
› rustc: 1.67.1
› cargo: 1.67.1
› Rust toolchain: stable-x86_64-unknown-linux-gnu
Packages
WARNING: no lock files found, defaulting to npm
› @tauri-apps/cli [NPM]: 1.2.3
› @tauri-apps/api [NPM]: Not installed!
› tauri [RUST]: 1.2.4,
› tauri-build [RUST]: 1.2.1,
› tao [RUST]: 0.15.8,
› wry [RUST]: 0.23.4,
App
› build-type: bundle
› CSP: default-src 'self'; img-src 'self' asset: https://asset.localhost
› distDir: ../frontend
› devPath: ../frontend
package.json not found
App directory structure
├─ .git
├─ .vscode
├─ frontend
├─ .github
├─ backend
└─ release-builds
Stack trace
No response
Additional context
There seems to also be an issue with opening the dev-tools on linux. Because the dev-tools open as part of the window itself, it seems to force it to resize in order to properly fit inside the window even with "resizable": false and an equal height/maxHeight/minHeight setting. See dev-tools console output. Expected 632, got 656

Not really an issue because the dev-tools are disabled inside of the build, just an observation

I am having the same issue. Is it possible to know how much you should compensate for on the height? I can't seem to find a way to query for the titlebar height.

@tedsteen I got it working by adding 28px to the height of the window. That was the default window decoration size on MacOs at the time of opening the issue

I had the same issue reported some time ago but it was closed because I think it was wrongly interpreted. The issue is still there like described above. Please investigate since this is a real issue.
The bug I reported can be found here: https://github.com/tauri-apps/tauri/issues/4226


Expose unfurl parameters in SlackRtmClient

Expose unfurl option parameters added in https://github.com/slack-scala-client/slack-scala-client/pull/162

I guess you would want a version released with this too?

I guess you would want a version released with this too?

Yes, would be great if you had the time to do so at some point, thanks! 🙇

Yes, would be great if you had the time to do so at some point, thanks! 🙇

I have just released 0.2.13

I have just released 0.2.13


Some thoughts compared to graphqlgen

So I just tried out https://github.com/prisma/graphqlgen and this library with the following template: graphql-codegen-typescript-template to create Typescript types for resolvers.
I wanted to give some feedback on some things I liked better about graphqlgen.

If a query/mutation doesn’t have any arguments, graphqlgen generates the args Typescript type as {} which I like better than the template which uses any.

graphqlgen let’s me set what the Context type is upfront, so I don’t have to do QueryResolvers.Resolvers<Context>, I can just do QueryResolvers.Resolvers.

I’m not sure how to handle this case:

So my schema is this:
type Post {
  id: ID!
  title: String!
  content: String!
  published: Boolean!
  author: User!
}

type User {
  id: ID!
  name: String
}

type Query {
  post(id: ID!): Post
}

So in the post resolver it expects me to return an author, but I really want to return a authorId and have the Post.author resolver use the authorId to fetch and return an author. graphqlgen solves this by allowing you to create your own typescript types and it will use those.

Regarding number 3 I suggest you to have a look at the graphql-codegen-typescript-mongodb-template template: https://github.com/dotansimha/graphql-code-generator/tree/master/packages/templates/typescript-mongodb
It allows you to define the following schema:
type Post @entity {
  id: ID! @id
  title: String! @column
  author: User! @column(overrideType: "string") @map(path: "createdBy")
}

It will generate the following types:
export interface Poll {
  id?: string | null;
  title?: string | null;
  user?: User | null;
}

export interface PostDbObject {
  _id: ObjectID | null;
  title: string | null;
  createdBy: string | null;
}

So you can use Poll for the object returned by the resolver and PostDbObject for the object returned by the database.

@darkbasic thanks, I'll check it out that looks good
@kamilkisiela sweet

Thank you @benawad !
Regarding #1, @kamilkisiela already did a fix for it: https://github.com/dotansimha/graphql-code-generator/pull/741  . We loved this feedback but actually thought to take it a step further and to use never in this case instead of {}, because this way it will enforce that args is not in use, because it’s empty for sure.
Regarding #2, As @kamilkisiela said, we have support for this use case, any you can override it easily. If you wish to change the default value, we added support for this in this PR: https://github.com/dotansimha/graphql-code-generator/pull/743
Regarding #3, as @kamilkisiela mentioned above, you can override every type because we are using generics in the generated resolvers and we think it’s a nicer solution to that issue.
But! I also want to add that think this approach of returning the id from a query resolver might not be that efficient and good practice, because this way you have to fetch the whole object by id in each Post. field, instead of fetching it only once.  we would encourage you not to do it and same goes for our users

@benawad If you are using dataloader to batch requests then it's great :) Then your resolver should expect to get then full object and not just the id, but the linked resolvers should return string in order to fetch it from the dataloader.
Then you should be able to use the generics to override the return value of each resolver, and set it to string. And with the latest version you can override the Parent value (using generics).
Also, @kamilkisiela added a mappers features in this PR, so now you can also custom mapping between GraphQL types and TypeScript interfaces, and this way it will override it in the whole generated file (and then you don't have to use generics each time).


A Matchbox-node-controller for Kubernetes ?

Hi all,
I would like to make a suggestion about Matchbox and Kubernetes.
Context:
How to automate Kubernetes cluster scaling with an on-premise bare-metal solution ? When new nodes are shipped, we need to scale-up. When a node is outdated, we need to scale-down.
Current solution:
A good way (the best actually!) to provision physical boxes is to use Matchbox.
There are many ways to configure Matchbox: config files, via the API, or using Terraform module...
For now, scale-up means:

add a new node definition to the Terraform file with: hostname=node-44 mac-address=XX:XX:XX:XX:XX template=dell-worker
terraform plan and terraform apply

Suggested solution:
What about from within Kubernetes itself? I would like to suggest a "matchbox-node-controller" .
We could manage our metal-infrastructure / kubernetes nodes, to scale-up:

kubectl create node node-44 --label provisioner=matchbox --annotation matchbox.mac-address=XX:XX:XX:XX:XX --annotation matchbox.template=dell-worker
or with a Kubernetes manifest

This solution could be implemented with a "matchbox-node-controller" watching "nodes" objects and applying changes to Matchbox. This could reflect the API node definitions in Matchbox. A similar example is "nginx-ingress-controller" which watches "ingress" objects via the Kubernetes API and applies changes to the Nginx configuration.
NOTE: Using Cluster Autoscaler on cloud providers, Kubernetes clusters are somehow able to scale themselves by asking the provider for new instances/nodes. Here, "auto" is possible because instances are virtually unlimited on clouds. It makes sense to scale-up to increase resource capacity  or scale-down to save money. But on-premise, boxes are physical and static, "auto" concept doesn't apply.
NOTE2: Cherry on the cake, Matchbox templates could be store in a config-map :)

The proposal here is that Matchbox add Kubernetes-specific integrations to watch for CRDs representing Groups and/or Profiles as an alternative to the usual TLS gRPC API.
Its been discussed internally and its not without merit. It still lacks some clarity about how it really solves the problem at hand, but its vaguely aligned with shifts to add kubernetes-native integrations. I suspect something along these lines will appear in some form someday.
Concerns
First, the Match deployment must be run on a Kubernetes cluster, the CRD must be sufficiently descriptive to create groups and profiles, and appropriate access policies must be in place.
Some setups run a Matchbox per site/datacenter, while others run a single redundant Matchbox service to PXE boot and provision sites globally (simpler in many cases). With that setup diversity, there is some oddity to the idea of creating a CRD directly on a Kubernetes cluster to govern machine matching. Its odd given Matchbox is a general bare-metal Container Linux provisioning tool so the new machines might be provisioned into anything (e.g. Mesos) but relying on a CRD. Its odd given you'd be asking for kubectl access to a Kubernetes cluster (maybe not yours) just to match machines at a different site. Meanwhile, Matchbox has an API you can expose over ingress and you can mint separate client certs (only for Matchbox) for each admin. So I think this would be of uses to admins managing a single Kubernetes cluster and running Matchbox on that cluster.
Next there is the issue of "where to go from here". Admins still need to hook into their systems to remotely power on, power off, and possibly tweak boot devices at a minimum. There is an amount of hardware and operation diversity here. CRDs would extend the possible options, but don't fully address  bringing a new piece of metal online.
Matchbox is a general bare-metal Container Linux cluster provisioning system, it powers different use cases, not just Kuberntes. If we do add this integration, requests for other platform integrations (Mesos, VMWare) will creep up. We need to have a good answer for those cases.
Finally, there is secret distribution. Worker nodes in most Kubernetes setups (including Matchbox examples) still require secret credentials be placed on them to join the cluster. This isn't a Matchbox concern per-se, but I'd like to avoid any scenario where folks are tempted to place secrets into Ignition directly. Right now, most setups treat the one doing the provisioning as the root of trust, relying on scp to place a generated kubeconfig or terraform to do the same via SSH. Some folks do have root of trust systems that are more or less based on hardware identities and this won't be a concern for them, but for many it will.
Notes
You can actually get some of what you want today without this being implemented.

Matchbox templates could be store in a config-map

If you're running Matchbox on Kubernetes already, you're picking one of the Kubernetes volumes to back /var/lib/matchbox data (host, EBS, ceph, etc). You can define group or profile files in a ConfigMap and have them setup to be mounted in the directory. You'll have the issue where the pod doesn't pickup the new ConfigMap after you edit it, until you kick the pod, but its pretty close to the same usage flow without new object types.
https://github.com/coreos/matchbox/blob/master/contrib/k8s/matchbox-deployment.yaml#L49

kubectl create node node-44

I don't think node creation would not be in scope, its not a Matchbox concern. Kubernetes workers register themselves with the control plane on bringup. Also, Matchbox has no business being authenticated to create nodes directly in Kubernetes.
Scope
If this were taken on, its scope should be limited to in-cluster usage, watching for CRDs, and creating groups / profiles that are observed in a reconciliation loop. Plenty of complexity there already. Anything related to reaching out to nodes or Kubernetes worker setup specifically is not a Matchbox concern.

Refinement
CRDs are useful for defining new cluster-scoped Kubernetes objects that support CRUD/list-style interactions (e.g. is this a thing you'd "list") and may be application specific. Examples might include defining cluster network policies, prometheus alerting policies, or cluster users.
Matchbox profiles and groups are application-specific types we interact with in a CRUD-style way, but the main mismatch with CRDs is that Matchbox types represent physical inventory, which is global. Matchbox types are not cluster-scoped like a cluster network policy CRD would be, for example.
An analogous question would be, should a database run in-cluster allow data (global rather than cluster-scoped) to be created via a CRD. In that light, I'm concerned building Matchbox CRD support isn't right.

Hi @dghubble,
Thank you for your detailed answer.
I was more thinking about a new binary running as a "controller", called "matchbox-nodes-controller" for Kubernetes watching node definition and applying the config to Matchbox, like "nginx-ingress-controller" watches ingress definition and apply changes to Nginx.
But actually, I think you're right. It's definitely not the job to Matchbox to connect and watch Kubernetes API (for nodes definition or even CRDs definition), but an Infrastructure manager (Terraform?).
Since we have objects such as pods, deployments, services, ingresses, network policies, and even self-hosted feature, we can configure all the infrastructure via Kubernetes API and manifest.
IMO, it would be amazing to be able to manage the Kubernetes nodes cluster via Kubernetes.
Terraform can manage many type of infrastructures (Matchbox included). What do you think about a Terraform-kubernetes-controller ?
Kubernetes Nodes <-watching-> Terraform-controller <-terraform apply-> Infrastructure <-provider-> Node.
Infrastructure could be: VMware-provider to deploy a new node as a VM, or Matchbox-provider to deploy a new node as a Metal.
Thank you.


[SPARK-33578][CORE] enableHiveSupport is invalid after sparkContext t…

What changes were proposed in this pull request?

add setSparkConf method in SparkContext
2.set the newest sparkconf in SparkContext#getOrCreate

Why are the changes needed?
reproduce as follow code:
    SparkConf sparkConf = new SparkConf().setAppName("hello");
    sparkConf.set("spark.master", "local");
    JavaSparkContext jssc = new JavaSparkContext(sparkConf);
    spark = SparkSession.builder()
            .config("spark.serializer", "org.apache.spark.serializer.KryoSerializer")
            .config("hive.exec.dynamici.partition", true).config("hive.exec.dynamic.partition.mode", "nonstrict")
            .config("hive.metastore.uris", "thrift://hivemetastore:9083")
            .enableHiveSupport()
            .master("local")
            .getOrCreate();
   spark.sql("select * from hudi_db.hudi_test_order").show();

it will produce follow Exception     AssertionError: assertion failed: No plan for HiveTableRelation [hudi_db.hudi_test_order … (at current master branch)
org.apache.spark.sql.AnalysisException: Table or view not found: hudi_db.hudi_test_order;  (at spark v2.4.4)
The reason is SparkContext#getOrCreate(SparkConf) will return activeContext that include previous spark config if it has
but the input SparkConf is the newest which include previous spark config and options.
enableHiveSupport will set options (“spark.sql.catalogImplementation", "hive”) when spark session created it will miss this conf
SharedState load conf from sparkContext and will miss hive catalog
Does this PR introduce any user-facing change?
No
How was this patch tested?

Can one of the admins verify this patch?


BUG: Calendars raising unexpected errors for some start dates

Probing bounds of calendars' start parameter has found following unexpected errors:
'XNYS' with start date prior to 1956-12-25 raises ValueError("Special dates [Timestamp('1956-12-24 19:00:00+0000', tz='UTC')] are not trading days."). Suspect related to issue https://github.com/quantopian/trading_calendars/issues/196 for which there was a pending PR https://github.com/quantopian/trading_calendars/pull/204/commits/6f06484404fd32f1a687d34c258f71cc5bdc0d88.
'XKLS' with start date prior to 1984-02-02 raises ValueError("Special dates [Timestamp('1984-02-01 04:30:00+0000', tz='UTC')] are not trading days."). Possibly same cause as 'XNYS'
'XPHS' with start date prior to 1901-12-14 raises NonExistentTimeError('1901-12-13 09:30:00').
Within #31 I'll investigate / incorporate https://github.com/quantopian/trading_calendars/pull/204/commits/6f06484404fd32f1a687d34c258f71cc5bdc0d88. Going to ignore 'XPHS' error given the date (exchange didn't open until 1927!).

'XKLS' with start date prior to 1984-02-02 raises ValueError("Special dates [Timestamp('1984-02-01 04:30:00+0000', tz='UTC')] are not trading days."). Possibly same cause as 'XNYS'

This is happening because of a conflict between adhoc special close 1984-02-01 (which isn't really adhoc but rather evaluted from ChineseNewYearsHalfDay here) and regular holiday FederalTerritoryDay. The same happens for 2049-02-01.
Similarly, calling XISTExchangeCalendar with an end date after 2029-04-23 raises ValueError: Special dates [Timestamp('2029-04-23 09:30:00+0000', tz='UTC')] are not trading days. Again this appears to be a conflict between an 'adhoc' special close here (which isn't really adhoc) and a regular holiday.
For both these calendars the special_closes_adhoc method provides for 'collisions'. I suspect the reason these regular special closes are included as adhoc is to be able to take out these holiday conflicts.
The list of collisions could be extended and maintained although it wouldn't address 'XNYS'.
Occurs to me that all these confilicts could be addressed by allowing special closes / opens to be defined for dates that are holidays and then filter these conflicts out in _special_dates when evaluating special_closes / special_opens(?).   I'll offer a PR.


Plot translation does not seem to work

the plot is not translated despite installing the relevant package and the following configuration:
"sort.metadata.nfo.translatedescription": 1,
"sort.metadata.nfo.translatedescription.language": "en",

Install googletrans 4.0.0-rc1. The current 3.0.0 version is bugged.
pip install googletrans==4.0.0-rc1


feat(model-client): let the user set useRoleIds in the initRepository call

The initRepository endpoint of the model server has an optional useRoleIds parameter. Its default value is true on the server.
ModelClientV2 has an initRepository method that does not accept the useRoleIds parameter, so it sends the request without it. It may cause problem for the user, if they create a repository via the modelClient V2, and then load/read the model assuming it has role names instead of role IDs.
This PR adds the (optional) useRoleIds parameter to that method. The default value is true.

:tada: This PR is included in version 7.5.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add video.json scaler_slot to APF_BRIDGE [WIP]

Files with non-derived changes: docs/control.md, litex/verilog_platform.py, litex/csr.py, target/pocket/core_top.sv
How I built this:
(cd litex && PATH=$PATH:/home/mcc/work/r/v/usr/bin make)
(cd lang/rust/crates/litex-pac/ && make)
(cd projects && ~/usr/intelFPGA_lite/22.1std/quartus/bin/quartus_sh --flow compile openFPGA-RISC-V_pocket)
(export REVERSE=../../../../../f/quartus/reverse/; (rm -f $REVERSE/bitstream.rbf_r && $REVERSE/a.out projects/output_files/openFPGA-RISC-V_pocket.rbf $REVERSE/bitstream.rbf_r && cp $REVERSE/bitstream.rbf_r /media/mcc/3930-6663/Cores/agg23.RISCV-scrolltest/riscv.rev))
sync && umount /media/mcc/3930-6663

Problems:

This should probably go in VIDEO_FRAMEBUFFER or APF_VIDEO. Instead I put it in VIDEO_BRIDGE because I could not find the equivalent of litex/verilog_platform.py for those. More importantly, however:
It does not work. In my testing, the litex-pac change "works" (there is now a register I can write to from Rust with peripherals.APF_BRIDGE.scaler_slot.write(|w| w.bits(delete_me_row % 2));) and the scaler change "works" (if I replace (apf_bridge_scaler_slot|24'h0) in core_top.sv with 24'h1 I successfully switch to video.json slot 1) but it appears values written to the litex-pac value are not relayed all the way to apf_bridge_scaler_slot in core_top.sv.

I don't currently know what I'm missing.

New change addresses all above requests, but (I think the changes were all style-related fundamentally…) it did nothing, the patch still doesn't have any effect when the new register is written to.

Made an attempt to move the scaler_slot register to VIDEO but did not succeed.
 Now the scaler slot register is 4 bit, bits 0-3 are the scaler slot, bit 4 when asserted means "disable the 1-pixel horizontal borders". That is a slightly odd interface but it is so it can default to 268x240 despite the 266 pixel problem. In the docs, I documented this with "4th pixel is set", I don't know how you'd prefer to set bitmasks, or if I should break the bits down into fields. I think setting all bits at once atomically would be ideal.


Add Reset Password

Currently, the ezXSS application only provides options for signing up and logging in. There is no "Forgot Password" or "Reset Password" functionality. This means users who forget their passwords are permanently locked out unless manual intervention is provided by an administrator.
Expected Behavior
A "Forgot Password" option should be added, allowing users to securely reset their password via email or another recovery method.

I'm not sure there is a (secure) way for ezXSS to allow password resets.
The admin can update passwords in case any user forgot the password. It is expected from a ezXSS admin to have some password manager and doesn't forget the password, but in case it does happen they can reset it through the database.
Closing this for now.


feat: Add new storage account and related new resources.

Description

If you haven't already, read the full contribution guide. The guide may have changed since the last time you read it, so please double-check.
Enable GitHub Worksflows in your fork to enable auto-generation of assets with our GitHub Action.
To trigger GitHub Actions after auto-generation, add a GitHub PAT as a secret in your forked repository called PAT.
Adding a new module

[ ] A proposal has been submitted and approved.
[ ] I have included "Closes #{module_proposal_issue_number}" in the PR description.
[ ] I have run brm validate locally to verify the module files.
[ ] I have run deployment tests locally to ensure the module is deployable.

Updating an existing module

[ ] This is a bug fix:

[ ] Someone has opened a bug report issue, and I have included "Closes #{bug_report_issue_number}" in the PR description.
[ ] The bug was found by the module author, and no one has opened an issue to report it yet.

[x] I have run brm validate locally to verify the module files.
[x] I have run deployment tests locally to ensure the module is deployable.
[x] I have read the Updating an existing module section in the contributing guide and updated the version.json file properly:

[ ] The PR contains backwards compatible bug fixes, and I have NOT bumped the MAJOR or MINOR version in version.json.
[ ] The PR contains backwards compatible feature updates, and I have bumped the MINOR version in version.json.
[ ] The PR contains breaking changes, and I have bumped the MAJOR version in version.json.

[x] I have updated the examples in README with the latest module version number.

@VaishnaviLende please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.
@microsoft-github-policy-service agree [company="{your company}"]

Options:

(default - no company specified) I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer.

@microsoft-github-policy-service agree

(when company given) I am making Submissions in the course of work for my employer (or my employer has intellectual property rights in my Submissions by contract or applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer. By signing below, the defined term “You” includes me and my employer.

@microsoft-github-policy-service agree company="Microsoft"

Contributor License Agreement

@microsoft-github-policy-service agree company="Nuance"

@dciborow - can you take a look at this one? I don't have enough context on the Storage RP to review this one effectively

@dciborow - can you take a look at this one? I don't have enough context on the Storage RP to review this one effectively

Overall, the improvements look good, but they need to be merged into the existing module a little bit more.
@VaishnaviLende ,

try to avoid adding any required parameters, by providing default values.
Merge 'destinationStorageAccount' into 'storageAccount'
Use the nested resource notation for the children resources

Thanks for the review comments. I am working on point 1 and 3.
On point 2 - destinationStorageAccount is created only if object replication policy is enabled. Are you suggesting to create it as nested resource under storage account?

Thanks for the review comments. I am working on point 1 and 3.
On point 2 - destinationStorageAccount is created only if object replication policy is enabled. Are you suggesting to create it as nested resource under storage account?

The changes for points 1 and 3 have been completed and committed. Could you kindly review it once and let me know if any changes are necessary. Thanks

Thanks for the review comments. I am working on point 1 and 3.
On point 2 - destinationStorageAccount is created only if object replication policy is enabled. Are you suggesting to create it as nested resource under storage account?

Let me take a fresh pass at this with this understanding.

try to avoid adding any required parameters, by providing default values. Dedup things like "destLocation", and just use the existing 'location' instead.
Merge 'destinationStorageAccount' into 'storageAccount'
Use the nested resource notation for the children resources

I addressed all of the review comments. Could you kindly review the changes once and approve them?

@dciborow - Hope you are doing well! Request you to approve this PR.

shenglol

@shenglol - Is it possible for you to review and approve this PR in absence of @dciborow ?

Could you resolve the conflicts for the PR?

Could you resolve the conflicts for the PR?

@shenglol - Thank you for your response. I have resolved all the conflicts. Could you please review and merge this pull request?

@VaishnaviLende Looks like the REAME file needs a regeneration.

@VaishnaviLende Looks like the REAME file needs a regeneration.

Updated readme file.

@VaishnaviLende Looks like the REAME file needs a regeneration.

@shenglol - Updated readme file.

@shenglol - I updated bicep version and also readme, main.json.

@VaishnaviLende Looks like the REAME file needs a regeneration.

@shenglol - Updated readme file.

@shenglol - I updated bicep version and also readme, main.json.

Can you also update brm with dotnet tool update --global Azure.Bicep.RegistryModuleTool? I think there are some improvements that may affect README generation.

@VaishnaviLende Looks like the REAME file needs a regeneration.

@shenglol - Updated readme file.

@shenglol - I updated bicep version and also readme, main.json.

Can you also update brm with dotnet tool update --global Azure.Bicep.RegistryModuleTool? I think there are some improvements that may affect README generation.

@shenglol - I updated brm also.


"Display As" for data variables in Linear Disassembly

In structures defined in the "Linear Disassembly" view, there is no option to change the display type of defined values. It would make sense to enable the same "Display As" available in the graph view for structure fields.
https://github.com/Vector35/binaryninja-api/issues/1349#issuecomment-520244474

Currently the actual metadata that is required to keep track of an integer's display type state is attached to the function object itself. We need to add the ability to track this bit of metadata on data variables in linear view.

I support that feature request, as I use BN a lot for RE-ing file formats and being able to e.g. quickly view signed numbers in decimal would help a lot. It would also help if individual variables could be displayed as little or big endian individually.

pushed to dev in 2.1.2403


Support Poller or PeriodicTrigger config in PollableMessageSource

Currently the MessageHandler must provide this logic. It would be nice to be able to autowire a Poller or maybe a simple trigger.

Turning it into documentation issue as @garyrussell  example from the old PR to use Scheduled task makes all the difference. We just need to document it


Registryui

Notes for Reviewers
This PR fixes

Rounded and Square corners. ✅
Long vertical line. ✅
Remove animation. ✅
Each branch of the tree lazily loaded. ✅
Eliminate the need for the scrollbar - restyle it when it does need to be displayed.✅
Expand all / Collapse all to simply be standard icons. ✅
Eleminate X✅
Search doesn't work. ✅

Signed commits

[x] Yes, I signed my commits.

at certain width when the search expands hide the collapse icons and switcher and again visible as the search closes

remove the count from tabs after certain width you will get some extra space

Make the tabs horizonal scroll when they start squezing

Have you thought any thing for this @ptdatta ?    // @ritiksaxena124

@ritiksaxena124 after certain width can be remove overview tab and instead display a back arrow icon ? // @ptdatta

@sudhanshutech actually there is another PR which contains the registry mobile view which not yet merged.

#28 216.8 Module not found: Can't resolve 'styled-jsx/server' in '/ui/pages'

@ritiksaxena124 after certain width can be remove overview tab and instead display a back arrow icon ? // @ptdatta

We are completely removing the Overview tab. It will only be shown when the user clicks the Registry tab.

Resyncing fork.

While the Registry may still have bugs that @ptdatta will immediately fix, we might be able to accept the current set of fixes so that existing users can stop suffering with the current implementation in the meantime. @ptdatta hurry up.

@ptdatta done?

@ptdatta done?

Yes✅


feat(core): raise MachineError only for internal-errors

I change certain e xceptiions from MachineError either to:

ValueError, e.g. when they are thrown for contructor arg errors),
AttributeErrors, when invoking a non-existing trigger on the machine

The purpose is for a Machine clients to handle "dynamic" errors that
denote inability to process transition,
differently from "static" setup/configuration errors.

Coverage remained the same at 99.735% when pulling 595161fb9ded9661e3a64b652ef9ed55906ef8d5 on ankostis:raisevalerr into 9dc9f0fbed0ee50bff3d1e94f69feec3b6c77b4e on tyarkoni:master.

I have no strong oppinion about this. @wtgee, @tyarkoni, what about you?

Could break some current implementation code but other than that it sees like a sensible change.
:+1:

Coverage decreased (-0.04%) to 99.694% when pulling 7cc578aa49676607556256e6a7bbdc69539e3522 on ankostis:raisevalerr into 1843e512e105104f5708bfa9735009a82ae034e5 on tyarkoni:master.

Rebased on top of #178.

Coverage remained the same at 99.694% when pulling 51cbcd97d08032cf5b938a2c644e438d81a44ce9 on ankostis:raisevalerr into 607b6dd477011006637cea3f77e106464be8dca8 on tyarkoni:master.

Rebased on latest master.

Hi,
I just want to give you an update why I haven't pulled this request yet. As @wtgee pointed out this might break current implementations catching MachineError. I will wait a bit longer to see if @tyarkoni wants to give feedback. I'd assume that the adaptions for the users are bearable and the change clarifies the cause of errors in a reasonable manner. If no one opposes I would merge the changes in a week.


Making this package an Express 4 Router

Hello @arthurtsang, I'm making this PR mostly to thank you for this awesome fork and inspiration.
I made this fork into an Express Router, so people can mount it in their development server on an arbitrary endpoint (ex: localhost:8080/edit-raml)
I'm making this PR and a full repository dedicated to it, since I think could use way more documentation.

Hi,
I would like to grab your changes but also see about updating the raml-api-designer to the latest. There is a fresh UI for the side-bar generated documentation that would be nice to see in the designer.

I appreciate your feedback, but I'm doubtful this outdated PR is still working.
Since i published it, I had to make a full-fledged rework of your work to make it work smoothly...
Just double-check it in case you are interested in merging, and ping me here to fix it

hi @mrgamer ,  sorry, somehow the notification is buried in my tons of notification (from other repo that i followed) and thanks for your PR, i'd apprepricate the update.  Thanks.


Added Euler Totient function in cpp

@Jawed-byte I wanna work on this! Please assign the issue to me.

@namannimmo10 Nothing is described by him. I implemented this code. Go to repository and check it out.

@fhlasek Can you assign this to me so that I can work on it?

Please don't ask to be assigned. See the contributing guidelines.
If you want to help with this, submit a PR or contact the admins to work on it.
Thank you. 🙂

would love to take a stab at this :)

Please don't ask to be assigned. See the contributing guidelines. If you want to help with this, submit a PR or contact the admins to work on it.
Thank you. 🙂

would love to take a stab at this :)

Please don't ask to be assigned. See the contributing guidelines.
If you want to help with this, submit a PR or contact the admins to work on it.
Thank you. 🙂

I would like to contribute

This was already implemented by #714
and should probably be closed.


Update to consider newest features

Update to consider newest features in ADS and SSMS

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


Update for PureScript 0.11

Bumped dependencies and changed toList to toUnfoldable. No other changes required.

Thanks!

My pleasure!


Add link to npm page in details

https://www.npmjs.com/package/{packageName}/v/{packageVersion}

Implemented on master


Correctly destroying the colorpicker instance

This fixes the issue when $input.colorpicker('destroy'); throws Error: TypeError: Cannot read property 'destroy' of undefined as soon as the AutoForm is destroyed.

@Pagebakers Merging in, thanks a lot!

Sweet!


fix(build): inject css into dynamic module if it is imported

fix #530

Can actually simply prepend the insertion - see a502399