Add several new watchers

This PR extends the dynamic watchers & auditors framework from https://github.com/Netflix/security_monkey/pull/405 to support several new watchers.
List of added watchers:
* CloudTrail
* AWSConfig
* AWSConfigRecorder
* DirectConnect::Connection
* DirectConnect::VirtualGateway
* EC2::EbsSnapshot
* EC2::EbsVolume
* EC2::Image
* EC2::Instance
* ENI
* KMS::Grant
* KMS::Key
* Lambda
* RDS::ClusterSnapshot
* RDS::DBCluster
* RDS::DBInstace
* RDS::Snapshot
* RDS::SubnetGroup
* Route53
* Route53Domains
* TrustedAdvisor
* VPC::DHCP
* VPC::Endpoint
* VPC::FlowLog
* VPC::NatGateway
* VPC::NetworkACL
* VPC::Peering
Additional changes:

Move rds[security_group] to rds/ directory.
Update vpc/route_table to use boto3 lib.
Add tests for tech types supported by moto
Initialize name to avoid UnboundLocalError
Update RDS watcher ephemeral values
Use boto3.session.get_available_regions in select watchers
Convert routetable watcher to decorator pattern
Convert route53 domains to decorator pattern
Handle the case where the aws principle is a string
Catching assume role exception in decorator

Do we have the list of new IAM permissions that will be required for all these new watchers?
On first run I get this:

2016-09-12 23:18:29,487 INFO: 'Problem Connecting to peering/demo_account/None:\nAn error occurred (UnauthorizedOperation) when calling the DescribeVpcPeeringConnections operation: You are not authorized to perform this operation.'

The specific error you're seeing should've been addressed by the setup script already in develop - https://github.com/Netflix/security_monkey/blob/develop/scripts/secmonkey_role_setup.py#L62 .
We've changed our deployment process to separate the role setup into a different provisioning script, so I'll paste it here but we could bake it into secmonkey_role_setup.py if you'd rather us push a new commit to the PR. We use the managed SecurityAudit policy plus a delta, attached as an inline policy to the role SecurityMonkey uses:
{
    "Version": "2012-10-17",
    "Statement": [
        {
            "Effect": "Allow",
            "Action": [
                "acm:ListCertificates",
                "acm:DescribeCertificate",
                "es:ListDomainNames",
                "es:DescribeElasticsearchDomainConfig",
                "lambda:ListFunctions",
                "ses:ListVerifiedEmailAddresses",
                "ses:GetIdentityVerificationAttributes",
                "sns:ListSubscriptionsByTopic"
            ],
            "Resource": [
                "*"
            ]
        }
    ]
}

@monkeysecurity should submit the fixes above and resubmit? There is also an additional bug we have found internally and fixed since this PR was made

Hey @aebie - if you just want to push a new commit to your new-watchers branch, I can rebase that on top of the other changes I've made.

Hey @aebie,
Which items were you thinking about updating?

direct_connect/virtual_gateway.py
boto3/531 TODO
config AVAILABLE_REGIONS

I've already fixed the region=kwargs['account_name'] in kms.py and added ARN to all applicable watchers in my Bridgewater-new-watchers branch: https://github.com/Netflix/security_monkey/commits/Bridgewater-new-watchers

@monkeysecurity I have a fix for the TODO/531 - changing to the updated iter_account_region decorator tag which involves encapsulation of get_account_regions in the decorator
The other fix is problems we found on our side with the new decorator that cause logging to be messed up and find_changes not to run multiple accounts.
Hopefully Aaron will push these changes today.

pushed the decorator tag fixes for cherry-pick to #411

Thanks @zollman.  I merged the update to decorators.py and config_recorder.py into my Bridgewater-new-watchers branch.
SO I guess my last question before merging is whether direct_connect/virtual_gateway.py is misnamed.

does not call describe_virtual_gateways() as you might expect with a 'directconnect' boto3 client
instead calls describe_vpn_gateways() with an ec2 boto3 client

@monkeysecurity are you suggesting we simply rename an move the virtual_gateway file?

Hey @aebie,
I'm not super familiar with those services so I just wanted to bring up what looked like a problem and get your opinion on what should be done.  Are you saying you think it's okay the way it is?

@monkeysecurity To be honest I didn't write this watcher and the guy who did has moved on to another project so I am not sure what this thinking behind the placement was. I would guess that the thought was that VPN was similar to direct connect in in concept so the two should be grouped together. I can also see the logic of grouping it with the vpc especially since that's where it lives in the AWS console. Personally I'd like to align with the AWS console so would be happy to move it and rename the watcher Virtual Private Gateway or something to the effect.

Awesome. Thanks @aebie .

I'm going to remove direct_connect/virtual_gateway.py from this PR so I can merge it.  We can add it back later with a different PR.

Merged in #411


Table colspec nodes using px instead of %

https://github.com/sphinx-contrib/confluencebuilder/blob/57ab6c4d1fd2df8c16953b97d1aa843667722dc0/sphinxcontrib/confluencebuilder/translator/storage.py#L811
Not sure if this is intentional, but it seems inconsistent with how the standard html is built with docutils. With html colspec uses % by default, but here px is used.

@r-dgreen, thanks for reporting this issue. A change (#461) has been added into the main branch that should address this issue and should be made available next stable release (v1.5).

v1.5 is now available on PyPI -- marking as closed.


weird output....

Not sure what is going on here:

This is related to this known issue. You must add a space before a question mark. Otherwise, it will output information about image:;

This is a feature of ipython, not a bug ;-)
Also please copy&paste your prompts into github issues, so that I don't have to type them from your screenshot ;-)


Fix ratelimit issues

For alpha, we need a plan to fix the ratelimit interactions with mixer.
Proposed plan:

hardcode in a name in the mixerclient config (maybe change it from RequestCount to RateLimit) (see: istio/manager#527)
add fields to the mixer quota aspect config that allows specification of a limit that should be applied within that scope/subject (DONE - istio/mixer#549)
remove max_amount from the descriptor of RequestCount/RateLimit (no sense in setting an upper bound on rate limits at this point) (DONE - istio/api#77)
set limit: 5 in the aspect config for the selector of "ratings*"

That should allow mixerclient to call Quota() on all requests, but have mixer just return status.OK unless the quota aspect is configured and linked to the RateLimit descriptor for a scope/subject.  And it should allow different limits based on scope/subject.

Do we still need this issue?

Yes, it's still broken AFAIK. @mandarjog was running down the latest issue.

This is done now.


Updates captchas template

Describe your changes
This adds the WorkInProgress component to the captchas template
Screenshots - If Any (Optional)
Issue ticket number and link - If Any
Issue #95
Checklist before requesting a review

[X] I have performed a self-review of my code.
[X] Followed the repository's Contributing Guidelines.
[X] I ran the app and tested it locally to verify that it works as expected.
[ ] I have checked my code with an automatic accessibility tool such as Axe Dev Tools or Wave
and it shows no errors.

Additional Information (Optional)
Any additional information that you want to give us.

Thanks so much for your contribution!


tiktok

Hi guys,
Firs of all what a great library. We are dropping everything else and only using this from no on.
Thank you for making available to the public!
Any chance to create a TikTok icon to go along the other social media icons? It's the only icon we are missing right now.
Best regards,
Pablo

Thanks so much, Pablo! tiktok-logo was just added in v1.3.1!

You guys are amazing!
Pablo Monzon
CEO, WeFit
+6592726420  | @.***
www.wetime.io  | Skype: pablomonzon

On Jul 8, 2021, at 2:31 PM, Tobias Fried @.***> wrote:

Closed #48.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


Missing documentation on using BUILDKITE_SPLITTER_IDENTIFIER when running multiple split steps

My use case:

Run a step with test-splitter parallelism for regular unit tests
Run a step with test-splitter parallelism for integration tests

I'm attempting to achieve this with the first step using BUILDKITE_SPLITTER_TEST_FILE_EXCLUDE_PATTERN to exclude the integration tests, and the second step using BUILDKITE_SPLITTER_TEST_FILE_PATTERN to only include the integration tests.
However in practice, the second step only picks up specs that should have been part of the first step!
ie. running BUILDKITE_SPLITTER_TEST_FILE_PATTERN=spec/foo/**/*_spec.rb test-splitter results in running tests from spec/bar which were definitely not part of that pattern.
It looks like our fix was to set BUILDKITE_SPLITTER_IDENTIFIER to a different unique value per step, however I note that this doesn't seem to be documented yet. Currently the documentation reads as follows:

Hi @TJC Thanks for reaching out!
Please try setting BUILDKITE_SPLITTER_IDENTIFIER to a different unique value for those two steps (eg BUILDKITE_SPLITTER_IDENTIFIER: "${BUILDKITE_BUILD_ID}_${BUILDKITE_STEP_ID}".
I think what happens with providing same identifier is that the test plan is cached after first time making request to our test plan API. In you second step, even you provide exclude/include pattern the filter won't be applied due to caching.
Let us know how you go with using a different identifier.

Yes, as I noted in the description, our fix did seem to be setting BUILDKITE_SPLITTER_IDENTIFIER.
The issue I'm raising is that this isn't really documented.

Thanks for providing us the feedback 💚
I've created an internal ticket to improve the documentation.

Is the buildkite splitter identifier truncated to a certain length in your systems?
I ask because when I tried using ${BUILDKITE_BUILD_ID}_${BUILDKITE_STEP_ID} I saw that the id was different on separate steps, but still encountered the problem that the second step re-used the plan from the first.
Switching back to my original method, things work as expected again.

Hey @TJC! Thanks for this additional feedback, and this also helps contextualise the other issue that you raised.
Just for background information, we cache the plan on unique combinations of BUILDKITE_SPLITTER_IDENTIFIER and BUILDKITE_SPLITTER_SUITE_TOKEN, so you can ensure uniqueness by either changing the value of BUILDKITE_SPLITTER_IDENTIFIER or BUILDKITE_SPLITTER_SUITE_TOKEN
Changing the value of BUILDKITE_SPLITTER_IDENTIFIER to ${BUILDKITE_BUILD_ID}_${BUILDKITE_STEP_ID} should work, and while we do have truncation for this field it doesn't kick in until 1024 characters so this is unlikely to be the issue you're seeing! It may be to do with how you're interpolating and passing through this value? If you're interpolating the value of BUILDKITE_STEP_ID on your pipeline definition, then it will have the step id of the upload step, rather than the step that is actually running the rspec specs. You might need to change your interpolation to happen within the testing environment so the correct step id value is used. Would appreciate if you could double check this and give that a go!
Alternatively, you could set up a second Test Suite to run your integration tests. That way, the data for your regular unit tests and the data for your integration tests are collected in different Test Suites, with different suite api tokens. This is what we do on our pipeline, as there are actually differences between the rspec setup and teardown for the two suites, and having the isolated data gave us better test splitting results. Does this sound like a use case for you as well? This would also solve the uniqueness issue :)
Sorry about our lack of documentation about this, we're still trying to figure out what other people's use cases are like, and what is sensible to recommend during the set up process!

Ah, thanks for explaining that aspect of the Step Id -- now I check the logs, it does rather look like the BUILDKITE_SPLITTER_IDENTIFIER ended up being the same everywhere, I think due to the interpolation happening too early.

Hi @TJC
We just released a new version 0.4.0 of the test splitter.
One of the update in this new release is to set theBUILDKITE_SPLITTER_IDENTIFIER default to BUILDKITE_BUILD_ID/BUILDKITE_STEP_ID.
If BUILDKITE_SPLITTER_IDENTIFIER is not provided, as long as we make BUILDKITE_BUILD_ID and BUILDKITE_STEP_ID accessible by the test splitter, it will use the default value and the uniqueness of the identifier can be guaranteed across different test steps.
As part of the release we also updated the documentation of the identifier. Please let us know if there's anything unclear about the docs 😊

One of the update in this new release is to set theBUILDKITE_SPLITTER_IDENTIFIER default to BUILDKITE_BUILD_ID/BUILDKITE_STEP_ID.
If BUILDKITE_SPLITTER_IDENTIFIER is not provided, as long as we make BUILDKITE_BUILD_ID and BUILDKITE_STEP_ID accessible by the test splitter, it will use the default value and the uniqueness of the identifier can be guaranteed across different test steps.

I tested this out, but it didn't seem to work for us when I removed the setting of BUILDKITE_SPLITTER_IDENTIFIER.
Perhaps I should check what exactly you mean by "as long as we make BUILDKITE_BUILD_ID and BUILDKITE_STEP_ID accessible by the test splitter" -- is there something specific required to enable that?

Perhaps I should check what exactly you mean by "as long as we make BUILDKITE_BUILD_ID and BUILDKITE_STEP_ID accessible by the test splitter" -- is there something specific required to enable that?

@TJC no action required to enable that in the tool. To clarify, the BUILDKITE_BUILD_ID and BUILDKITE_STEP_ID environment variables must be present in the environment in which the test-splitter is being run.
Can I confirm that these two environment variables are present when the BUILDKITE_SPLITTER_IDENTIFIER environment variable is removed?

Looking at the "Environment" tab of the Buildkite steps involved, I see that in both steps, those values are present.
The build id is the same for both, and the step id is separate to each.

Thanks for confirming.
Are you using the docker-compose plugin? If so, are the environment variables being passed through to the docker container?

Are you using the docker-compose plugin? If so, are the environment variables being passed through to the docker container?

We are using the docker-compose plugin. Looks like we needed to set propagate-environment: true. That has resolved the issue.

Good to hear!
I’ll add another clarification to the README.md to explicitly call out the environment in the wording.
I’ll close this issue now, much appreciated for your input @TJC!

Thanks for your help; glad I could help improve the docs.


feat: update tar pipeline

This PR is paired with #105. The goal is to remove the date from everywhere but the URLs. We don't need them in the tar file, as we have the build date encoded into the binary.

According to offline discussions, it would add complexity for nothing. It will  be solved directly in the script instead of in the pipeline.


Add a tag similar to discord-attachments but for "autplayed looped no-control" videos

The idea is to play tenor gifs in video format, like the original client:

The styling is almost similar to message attachments.
Link to above gif

<img src="https://c.tenor.com/Qeom_gC-4XYAAAAC/haikyuu-kageyama.gif" />

Just get the actual gif URL and add it in an image tag in the current attachment component

Yh but the link comes as the video property of a MessageEmbed

I added the tag in my fork, its working fine now :)


OAK D PRO - MAXIMUN RESOLUTION IMAGES ARE NOT SENT

I have the OAK-D PRO camera model. In theory it is a 12MP (4056x3040) camera. On the one hand, I'm not able to set the camera to 4056x3040px. When I indicate the resolution of 4056x3040 I get the following error:
ColorCamera(0) - 'video' width or height (4056, 3040) bigger than maximum at current sensor resolution (3840, 2160)
My configuration file is the following:
/oak:
  camera_i_enable_imu: true
  camera_i_enable_ir: true
  camera_i_floodlight_brightness: 0.0
  camera_i_ip: ''
  camera_i_laser_dot_brightness: 1000.0
  camera_i_mx_id: ''
  camera_i_nn_type: none
  camera_i_pipeline_type: RGBD
  camera_i_usb_port_id: ''
  camera_i_usb_speed: SUPER_PLUS
  imu_i_max_q_size: 30
  nn_i_nn_config_path: depthai_ros_driver/mobilenet
  rgb_i_board_socket_id: 0
  rgb_i_calibration_file: ''
  rgb_i_enable_preview: false
  rgb_i_fps: 1
  rgb_i_height: 3040
  rgb_i_interleaved: false
  rgb_i_keep_preview_aspect_ratio: true
  rgb_i_low_bandwidth: false
  rgb_i_low_bandwidth_quality: 50
  rgb_i_max_q_size: 30
  rgb_i_preview_size: 300
  rgb_i_publish_topic: true
  rgb_i_resolution: '12MP'
  rgb_i_set_isp_scale: false
  rgb_i_output_isp: false
  rgb_i_width: 4056
  rgb_r_exposure: 1000
  rgb_r_focus: 1
  rgb_r_iso: 800
  rgb_r_keep_preview_aspect_ratio: true
  rgb_r_set_man_exposure: false
  rgb_r_set_man_focus: false
  rgb_r_set_man_whitebalance: false
  rgb_r_whitebalance: 3300
  rgb_i_isp_num: 2
  rgb_i_isp_den: 3

For this reason I have to change the resolution to 3840x2160 px.
On the other hand, at the resolution 3840x2160px, the maximum that tells me it is possible, it gives me the following error and only provides me with one image:
[StereoDepth(3)] [error] Out of memory while creating pool for 'rectifiedLeft' frames. Number of frames: 3 each with size: 921600B
How can I receive images at the maximum resolution with a certain frequency?

Hi, few comments:

Thanks for notifying on the video size warning - there is a minor bug here, and although images are publishing at 12MP this will cause publishing of invalid CameraInfo data, this will be updated in the next release.
While you can set the large resolutions individually, there are limitations when it comes to the device resources - memory pool is one limitation, the other one would be the input size for Spatial Location calculator - currently the limit input size for Spatial Location Calculator is FHD resolution. With NN disabled you can run both depth and RGB at 4K resolution.

Yes, you can output 12MP images, stereo output gets automatically scaled up to match RGB resolution, but you can override that by setting for example:
stereo:
      i_width: 1280
      i_height: 720


Case for a "Notebook Subproject" Github organization

I'd like to make a case for a separate "Notebook" Github org.
I know we discussed that we'd rather aim to consolidate things (in today's notebook meeting), but because there is a unique governing body that will manage this subproject and a specific set of projects under these team's management (see the list below), I believe there is a decent case for a separate org.
Further, I don't think it's the responsibility of the JupyterLab team to maintain nbclassic and the (new) notebook_shim package; rather it's the responsibility of the notebook team.
Here is a list of repos we would immediately move to this new org:

jupyterlab/nbclassic
jupyterlab/notebook_shim
jupyter/notebook
jupyter/notebook-team-compass

Pinging @tonyfast @afshin @jtpio.

I think it is very important to keep jupyter/notebook as the home of the codebase a user is installing when they run pip install notebook. This does not prevent us, however, from having that repo redirect to a new one. I would propose waiting until after the the dust settles from the Jupyter Notebook v7 launch and the activity that will undoubtedly generate.


Fixed reference link to canvas lesson in main readme and translations

Fixed reference link to canvas lesson in main readme and translations
Originally posted by @ThesllaDev in https://github.com/microsoft/Web-Dev-For-Beginners/pull/285

Complete, I think


32TLn1WLcu8LtfvweLzYUYU6ubc2YV9eZs

hay
i have wrong send to wrong address
32TLn1WLcu8LtfvweLzYUYU6ubc2YV9eZs

i want the money back to binance to this
address 1EkF3H6v4LQUDewaStcu4hhN1v7ZXKDBy2
please replay and help me


npm install not running with slsart monitor/config command

The load generator lambda gives following error. Workaround is to run npm install manually.
Unable to import module 'handler': Error at require (internal/module.js:20:19) at Object.<anonymous> (/var/task/task.js:1:79) at Module._compile (module.js:570:32) at Object.Module._extensions..js (module.js:579:10) at Module.load (module.js:487:32) at tryModuleLoad (module.js:446:12) at Function.Module._load (module.js:438:3)

@ashmibhanushali Which branch are you seeing this on?

monitoring-bigscript2

Thanks! Having a look.

I cannot reproduce this problem using the current branch. I've privately provided a transcript of my testing.
Please feel free to re-open with new information.
TYVM,
~Greg

Current branch is which branch?

monitoring-bigscript2


Hide Column

Feature
Hide or show Columns.
Tasks

[x] Added property hidden (type: bool) to hide a Column

Hey @difelice, thanks for the PR. I think this usecase is already covered by the user being able to determine if a column should be rendered at all:
<Table>
  <Column />
  {hidden ? null : <Column />}
</Table>

Yes, that would do it. Thanks.


Mark this repository as archived so that people are aware this project is dead

See title.

In case you don't know: There is active development at the community fork Fomantic-UI since 2018
https://github.com/fomantic/Fomantic-UI

wait it's dead?!

No new releases since 2018, it even starts to stink

No new releases since 2018, it even starts to stink

I was soooo excited when I found a semantic ui last night. It is everything I have been looking for. Looks amazing and now I find out it's not being maintained. :( my luck!

I was soooo excited when I found a semantic ui last night. It is everything I have been looking for. Looks amazing and now I find out it's not being maintained. :( my luck!

@JayBit33 The FomanticUI fork is alive and well: https://github.com/fomantic/Fomantic-UI

@JayBit33 The FomanticUI fork is alive and well: https://github.com/fomantic/Fomantic-UI

There's also JupiterUI as an alternative.

why it dead?

It had only a single maintainer who didn't allow anybody else access to maintain the code in his stead, and then he went on some soul-searching journey across the world or something like that. That means there was nobody to maintain this repo, and there still doesn't seem to be anybody to do so.
Hence FomanticUI which is actually actively maintained.


Feature Request: Download List Discovery

Hi Everybody,
Love the work that is in progress and already done on brewcask. However I think the site is a little underwhelming. There is a search function on the website, which is pretty neat. However it only shows the command to install if you find the specific app. It would be great to discover apps that can be installed through brewcask. Doing this by showing a summary of what the app does, its website, its logo, active installs, etc. Maybe catagorise the search function. Reviews is a little too much, but a rating is not bad. etc (I would not put screenshots on it though)
what do you think?

Homebrew Cask is not an app discovery platform.


Publish for MacOS and ARM too

It would be great to have binaries available on all platforms.
Maybe ask the Netty guys how they do that? ping @normanmaurer
FYI, I have the Netty decoder working nicely based on jvmbrotli (just sad the brotli copy is very old) without having to fork anything, on contrary to your original PR :)

GitHub Actions supports macOS and ARM. I think that's the easiest way to build cross-platform binaries.

GitHub Actions supports macOS and ARM. I think that's the easiest way to build cross-platform binaries.

Can you share the code of Decoder? I'm curious to know more about it.

Can you share the code of Decoder? I'm curious to know more about it.

It's still a WIP in a private branch I can't open source yet.
I can push a public WIP branch as soon as you have a release available for MacOS and w/ Apache 2 license.
I'll then add tests and contribute upstream to Netty.
Note: I'll only contribute the Decoder (needed for Gatling), not the Encoder.

It's still a WIP in a private branch I can't open source yet.
I can push a public WIP branch as soon as you have a release available for MacOS and w/ Apache 2 license.
I'll then add tests and contribute upstream to Netty.
Note: I'll only contribute the Decoder (needed for Gatling), not the Encoder.

Perfect makes sense.

Perfect makes sense.

It's done. v1.2.0 uses Apache License 2 and supports macOS, Can we work on Netty Brotli support now?

It's done. v1.2.0 uses Apache License 2 and supports macOS, Can we work on Netty Brotli support now?

see https://github.com/gatling/gatling/commit/934885be192afba56af3ec3e388e269a04b2ed1c

see https://github.com/gatling/gatling/commit/934885be192afba56af3ec3e388e269a04b2ed1c

I forgot to change the License type in 1.2.0. Fixed in 1.2.1, sorry for the inconvenience.

I forgot to change the License type in 1.2.0. Fixed in 1.2.1, sorry for the inconvenience.

My logic behind the decoder was: 1 Chunk of Brotli can be up to 16 MB. Whenever I get a ByteBuf, I will pass it to Decoder and Decoder will tell if decoding was SUCCESSFUL or NEEDS_MORE_INPUT.
For NEEDS_MORE_INPUT, we'll keep aggregating ByteBuf and pass it to Decoder until decoding was SUCCESSFUL or aggregation exceeded 16 MB.
If a file of 2 GB is compressed with Brotli, each chunk could be up to 16 MB only. So we're not the aggregating whole data but chunks.
Does this sound good?

My logic behind the decoder was: 1 Chunk of Brotli can be up to 16 MB. Whenever I get a ByteBuf, I will pass it to Decoder and Decoder will tell if decoding was SUCCESSFUL or NEEDS_MORE_INPUT.
For NEEDS_MORE_INPUT, we'll keep aggregating ByteBuf and pass it to Decoder until decoding was SUCCESSFUL or aggregation exceeded 16 MB.
If a file of 2 GB is compressed with Brotli, each chunk could be up to 16 MB only. So we're not the aggregating whole data but chunks.
Does this sound good?

I'm really not a Brotli expert but my use case is mostly web where I would never see such large payloads. So I would be more inclined to use the same model as the existing ZlibDecoders that pull decompressed data as soon as it's available. Moreover, IIRC, your own take did a lot a memory copies, such as https://github.com/netty/netty/pull/10626/files#diff-04ab86c9d58745be5ec7455a7a9e7a011426adb5e556a8c206d132e212e7b037R35-R38.

I'm really not a Brotli expert but my use case is mostly web where I would never see such large payloads. So I would be more inclined to use the same model as the existing ZlibDecoders that pull decompressed data as soon as it's available. Moreover, IIRC, your own take did a lot a memory copies, such as https://github.com/netty/netty/pull/10626/files#diff-04ab86c9d58745be5ec7455a7a9e7a011426adb5e556a8c206d132e212e7b037R35-R38.

If you can open PR in Netty, we can work with others and see the best possible way to handle these issues. I am also running out of ideas. :p

If you can open PR in Netty, we can work with others and see the best possible way to handle these issues. I am also running out of ideas. :p

Will do.
However, sad news: the MacOS jar is actually empty: https://repo1.maven.org/maven2/com/aayushatharva/brotli4j/native-macos_x86-64/1.2.1/

Will do.
However, sad news: the MacOS jar is actually empty: https://repo1.maven.org/maven2/com/aayushatharva/brotli4j/native-macos_x86-64/1.2.1/

Life is unfair. :( Lemme debug this now.

Life is unfair. :( Lemme debug this now.

Just asking? Are you sure the file you're generating has a dylib extension? The Netty files have a jnilib one (which is not what System#mapLibraryName returns, so they have to hard code.

Just asking? Are you sure the file you're generating has a dylib extension? The Netty files have a jnilib one (which is not what System#mapLibraryName returns, so they have to hard code.

Yes, binary is generated with dylib.

Yes, binary is generated with dylib.

@slandelle Deployed 1.2.2 on Central after checking each file carefully. It's been a BrotliDay today. :D

@slandelle Deployed 1.2.2 on Central after checking each file carefully. It's been a BrotliDay today. :D

Awesome, thanks a lot! I confirm it works now.

Awesome, thanks a lot! I confirm it works now.

This ticket mentions ARM but I can see only x86_64 binaries at https://search.maven.org/search?q=g:com.aayushatharva.brotli4j
Is there any chance for Linux aarch64 binary too ? Should I open a new ticket ?
Thanks!

@martin-g I've plans for ARM in future but I'm blocked with tons of work these days.
However, if you can do PR for it, I can merge and publish ARM binaries.

Awesome! I'll send you a PR soon!


UI: add an option to have multiple views for single data source

Therr is an option to "undock view" which will open new window with the messages

Undock view and undock view per process were updated in latest commit


Crash when use payment view

Crash when push the view from payment builder
This is the log for error

This the code

hey @mrahmed017, we're checking this and will be back to you soon

sent a sample project through Slack. Have you received that? @mrahmed017

No I don't
I have downloaded the simple project from Github, I followed the steps but crash still show when push or present the payment view

Can you please have a look at this project that's created by the sample code that you provided above? It does not show any errors so, if you can reproduce the error on it and send it back, we can have a detailed look. @mrahmed017
Error-237495.zip

@okhan-okbay-cko project you sent is working, and the same code when used in my project crash
Checkout.zip
this archive contains video, images and crash log
and if you want to meet on(TeamViewer Or Meet) just let me know

I had already investigated the screenshots and the logs you provided.
The project that I sent here is the only code and integration that we currently provide and it works when done as suggested. There is no property called as tableView inside the PaymentViewController, so, I'll assume this is an error that comes from your implementation. @mrahmed017

@okhan-okbay-cko What do you suggest that we do now to work properly, I removed the old version (3,0) permanently and downloaded the new version (4.2) twice

If you can send a project that reproduces this issue, then I can have a look at the problem. @mrahmed017

Oky no problem, but I prefer send it by email ( or any private channel)

sure, you can send it through Slack as well. @mrahmed017

We already with checkout with Slack

Hey, thanks for your reply, let's take this offline and loop in with your customer success manager and/or implementation engineer to discuss further. @mrahmed017


Add example for timers

Summary
Adds an example for https://github.com/mattermost/mattermost-plugin-apps/pull/375.
Ticket Link
https://mattermost.atlassian.net/browse/MM-48342

This PR should not be merged until a new release of the apps plugin is cut.


Calculation entry standardization

We need to standardize mandatory and optional the fields in the calculation entry type.

Here is an attempt at a concrete suggestion:
{
  ... <other response items> ...
  "data": [
    {
      "type": "calculation",
      "id": "example.db:calcs:0001",
      "attributes": {
        "local_id": "example.db:calcs:0001",
        "url": "http://example.db/calcs/0001",
        "immutable_id": "http://example.db/calcs/0001@123",
        "last_modified": "2007-04-05T14:30Z",
        "calculation_description" = "relaxation",
        "related_structures_descriptions": ["initial", "final"],
        "related_structure_ids" = ["example.db:struct:0042", "example.db:struct:4711"],
      }
    }
  ]
}

It was said during the discussion that this probably cannot be decided without another meeting.

Should we think about how we make a query ? For example, if I want to get the data of BAND GAP, should we enter the keyword  "band_gap" , or " BandGap" , or "BGap", ... etc. Is it discussed somewhere else ?   Or we have already had the specification formulated ?

Looking at other parts of the specification, e.g., #23, makes it seem likely to become band_gap over the other alternatives you suggest. But, we presently have no agreement of a common definition of band_bap. I suspect we will go down the road of, e.g., band_gap_kohn_sham, band_gap_optical, band_gap_fermi_liquid_quasiparticle, etc. But I expect that we'll need a discussion in a meeting to sort this out.


introducing spanning media query + various JS API and examples updates

Not a fan of the word chosen for the media query. Thinking it through, and talking with others I’d prefer “spread”. Or anything other then spanning. ”Span” already existing, and adding a “spanning“ media query could be confusing to new, and even existing developers. I’m still reviewing the doc. I was waiting for this to happen, and it’s exciting to see! Nice work @zouhir

FWIW, I understood what ‘spanning’ meant and the same would not have been true of spread, at least not without an additional qualifying word (‘viewport-spread’?). I’ve heard the construction ‘the desktop spans two monitors’ but never ‘the spread of the desktop is two monitors’ — which, although technically coherent, doesn’t seem like natural English and sort of evokes mayonnaise.


enhance: add retry helper method for Trylock

Signed-off-by: Wei Fu<EMAIL_ADDRESS>
Ⅰ. Describe what this PR did
The ctrd maybe hold the lock for a while. In the concurrent request, it
might fail to serve request frequently. For this case, pouchcontainer
should retry in the specific duration to reduce the number of failed
request.
Ⅱ. Does this pull request fix one issue?

fix #2281
Ⅲ. Why don't you add test cases (unit test/integration test)? (你真的觉得不需要加测试吗？)
Added
Ⅳ. Describe how to verify it
Check case
Ⅴ. Special notes for reviews
5 seconds is enough? or use request context?

LGTM until some minor request updated.

@allencloud @allencloud I used context as timeout. If the request doesn't set the timeout, it will retry like spin-lock.

next PR I will remove the TryLock....

LGTM


Calling OSC commands from GDSCripts

What's the proper way to call an OSC command that needs a sender as argument from GDScript?  Should it send an OSC message to itself?  I had this in prior iterations, but I'm sure there's a proper way to do it.
A use for this is loading custom command files and running them at startup, for example with config files (#12 )

The answer is: use $Main.evalOscCommand(address, args, null). Note that sender is set to null.
From within an actor node, $Main can't be used, but it can be accessed with get_parent().get_parent() (sic: it's 2 levels above)


feat: updating to Style Dictionary v4

Description
This PR is an attempt to update the Style Dictionary v3 library to Style Dictionary v4 (including refactoring the design tokens to the DTCG format).
Merge Checklist

[ ] If your update includes visual changes, have they been reviewed by a designer? Send them a link to the Netlify deploy preview, if applicable.
[ ] Does your change adhere to the documented style conventions?
[ ] Do any prop types have missing descriptions in the Props API tables in the documentation site (check deploy preview)?
[ ] Were your changes tested using all available themes (see theme switcher in the header of the deploy preview, under the "Settings" icon)?
[ ] Were your changes tested in the example app?
[ ] Is there adequate test coverage for your changes?
[ ] Consider whether this change needs to reviewed/QA'ed for accessibility (a11y). If so, please request an a11y review for the PR in the #wg-paragon Open edX Slack channel.

Post-merge Checklist

[ ] Verify your changes were released to NPM at the expected version.
[ ] If you'd like, share your contribution in #show-and-tell.
[ ] 🎉 🙌 Celebrate! Thanks for your contribution.

Thanks for the pull request, @PKulkoRaccoonGang!
What's next?
Please work through the following steps to get your changes ready for engineering review:
:radio_button: Get product approval
If you haven't already, check this list to see if your contribution needs to go through the product review process.

If it does, you'll need to submit a product proposal for your contribution, and have it reviewed by the Product Working Group.

This process (including the steps you'll need to take) is documented here.

If it doesn't, simply proceed with the next step.

:radio_button: Provide context
To help your reviewers and other members of the community understand the purpose and larger context of your changes, feel free to add as much of the following information to the PR description as you can:

Dependencies

This PR must be merged before / after / at the same time as ...

Blockers

This PR is waiting for OEP-1234 to be accepted.

Timeline information

This PR must be merged by XX date because ...

Partner information

This is for a course on edx.org.

Supporting documentation
Relevant Open edX discussion forum threads

:radio_button: Get a green build
If one or more checks are failing, continue working on your changes until this is no longer the case and your build turns green.
:radio_button: Let us know that your PR is ready for review:
Who will review my changes?
This repository is currently maintained by @openedx/paragon-working-group.  Tag them in a comment and let them know that your changes are ready for review.
Where can I find more information?
If you'd like to get more details on all aspects of the review process for open source pull requests (OSPRs), check out the following resources:

Overview of Review Process for Community Contributions
Pull Request Status Guide
Making changes to your pull request

When can I expect my changes to be merged?
Our goal is to get community contributions seen and reviewed as efficiently as possible.
However, the amount of time that it takes to review and merge a PR can vary significantly based on factors such as:

The size and impact of the changes that it introduces
The need for product review
Maintenance status of the parent repository

:bulb: As a result it may take up to several weeks or months to complete a review and merge your PR.

TODO

correct display of the header of generated files
checking the correctness of variable generation (for some variables you can notice an extra indication of the unit of measurement, for example 0rem)
...

ES6 modules
Since the updated Style Dictionary v4 API is based on ES6 modules, an open question is finding a solution for the correct operation of Style Dictionary v4 together with the existing CommonJS functionality of Gatsby. According to the Gatsby documentation, support for ES6 modules is implemented in more recent releases of the framework. Most likely, for the current version of Gatsby (4.23.1), a potential solution could be to use asynchronous imports or update the Gatsby version.

Thank you so much for putting this together! Considering the task of "do some discovery around v4" this is truly above and beyond!

checking the correctness of variable generation (for some variables you can notice an extra indication of the unit of measurement, for example 0rem)

I think it's reasonable to update the .json files to include units where we don't yet have them as a solution here.

Since the updated Style Dictionary v4 API is based on ES6 modules, an open question is finding a solution for the correct operation of Style Dictionary v4 together with the existing CommonJS functionality of Gatsby. According to the Gatsby documentation, support for ES6 modules is implemented in more recent releases of the framework. Most likely, for the current version of Gatsby (4.23.1), a potential solution could be to use asynchronous imports or update the Gatsby version.

Could you elaborate on this a bit? I worry that moving from Gatsby v4 to v5 would be quite a heavy lift, and I wonder how that compares to some workarounds people have tried https://github.com/gatsbyjs/gatsby/discussions/31599#discussioncomment-1239988

@brian-smith-tcril @adamstankiewicz In this commit we removed the brand-openedx theme. Do we want to remove the sidebar toggle button and any mention of the brand in Paragon?


ML 3.2: k-means Clustering (A)

Description 📜
k means clustering algo, Steps
Elbow method
Disadavantages
Applications
Fixes # <4109>

Type of change 📝

[x] Au#dio (Should be in mp3 format Includes speech clarity, Concise ,Low distortion)

Domain of Contribution 📊

[x] Machine #Learning

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm GWOC'21 contributor

Screenshots / Gif (Optional) 📸

Audio content:
k means clustering algo, Steps
Elbow method
Disadavantages
Applications
@abhisheks008 , @geeythree please review on this

@geeythree  please check it now :D

Renamed.

Merged @ron352 🎉


fix: https://github.com/kubeovn/kube-ovn/issues/1271#issue

What type of this PR

Bug fixes

Which issue(s) this PR fixes:
Fixes #(issue-1271)

@oilbeater pls check this pr?

LGTM, thanks!


Fix article list sometimes not appearing on start

Under certain circumstances, the article list would be empty after app launch.
We make sure the app is ready to handle notifications before setting up the display of article list.
Issue #1623

Mmmh… This breaks old browser.
At launch, the main window does not appear and I get messages like :
-[ArticleListView activateWebView]: unrecognized selector sent to instance 0x…

The problem is seen when :

old browser is used
a single tab (the main tab) is set


fix(validateBulk): Handling corner case in SLP Utils validateBulk

:tada: This PR is included in version 1.24.3 :tada:
Your semantic-release bot :package::rocket:


Problem when using @Ignored keyword

I used this model.

But I can't binding value for String type (number) variable using @Ignored keyword.
How to binding data when using @Ignored keyword?

Hi @nielsenko,
Thank you for your answer.
Can I know I can add that number field to mobile realm local database?
Thank you.

Hi Salitha Shamood
Thank you for taking an interest in our new Flutter / Dart SDK.
I assume, you are missing the constructor argument to set number. We do not generate this currently, so you will need to set the property after construction, something like
final item = Item(ObjectId(), 'Salitha Shamood', 'item#2')..number = '2';

Be aware that this property will not be persisted when adding the item to the realm, and you will have to set it on all Items you read from realm, including whenever traversing a link to an Item.

I'm not sure I understand your question correctly, but if you remove the @Ignored() attribute, the number field will be stored in the realm, along with the rest of the fields.

Yes. I know it. But I want to that number field store in local database only. I didn't want to add that field in mongoDb database. Can I do it?

Only synchronizing a subset of the local database is not supported, unfortunately.

You can have two separates realms, one that is synced, and one that is non-synced and stays local. You can store different data in each


E2E test timed out for sending coins

UI Version: 0.8.2
Description:
See CircleCI run: https://circleci.com/gh/cosmos/voyager/4976

Should be fixed by: https://github.com/cosmos/voyager/pull/1014/commits/6fd704e988b2ee82377b31f461348ce2c5d3d4e7


Remove redundant type conversion

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

Needs approval from an approver in each of these files:

pilot/OWNERS

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

Hi @sevenNt. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
I understand the commands that are listed here.

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

/assign @andraxylia

/test

Hi, @andraxylia some test scripts executed failed, is there something wrong with this PR?

/ok-to-test

The test were flaky, nothing wrong with the PR. I re-triggered them. If they fail again, could you please merge with latest master? If it still fails, @wattli is on call and can help to get it in.

OK, thank you andraxylia.

@wattli , would you mind merge the PR into master?

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: andraxylia, sevenNt
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/OWNERS [andraxylia]

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


Feature: custom secrets

I would like to be able to add custom secrets in values that can be loaded into custom volume mounts on st2actionrunner and related pods. The custom volume mounts feature is covered by #212 (I have a branch prepared for this, PR will come later). So this feature request is only about adding custom secrets.
I have a branch prepared to add this feature: https://github.com/cognifloyd/stackstorm-ha/tree/custom-secrets
I will submit a PR later.

I've seen custom secrets for different use cases normally created outside of the chart scope and then could be referenced in Helm values where possible.


Unable to use fractional Membership fee

Describe the bug
While creating DAO I was unable to use 0.1 as membership fee
To Reproduce
Steps to reproduce the behavior:
I was unable to create DAO due to fact I am using fractional numbers
How to repeat:

Go to https://beta.gamedao.co/app/organisations
Leave everything unchanged
Enter 0.1 as Membership fee
Click Create Organization

Development Console shows Error:
Uncaught (in promise) Error: createType(Call):: Call: failed decoding gameDaoControl.create:: Struct: failed on args: {"creator":"AccountId","controller":"AccountId","treasury":"AccountId","name":"Bytes","cid":"Bytes","body":"u8","access":"u8","fee_model":"u8","fee":"Balance","gov_asset":"u8","pay_asset":"u8","member_limit":"u64"}:: Struct: failed on fee: Balance:: Balance: Negative number passed to unsigned type
Expected behavior

If fractional fee not supported I expect clear JS warning about it.
0.1 is clearly not negative number so underlying level incorrect.
UI should prevent from entering and submitting invalid values.

Screenshots

Desktop (please complete the following information):

OS: Ubuntu 20
Browser Firefox 94 64bit

Additional context
I got enough ZERO on my wallet 39

values need to be properly converted, e.g. with the polkadot utils

seems to work as expected now @5-mark @DarkNebula0


Question: How to create a route file to run a scenario for a specific route?

Describe our question or idea
I want to create a route to test a scenario using: python scenario_runner.py --route <route_file_path> <scenarios_file_path> <route_name> --agent <agent_file>, but i have not much idea how to generate that route is it through rosbag?
Desktop (please complete the following information):

OS: [e.g. Ubuntu 18.04]
CARLA Version [e.g. CARLA 0.9.10]
Python version [e.g. 2.7]
Version [e.g. 0.9.10]

Hey @ParthAD3, routes are generally created manually, so they require some work. As you pointed out, they need a route file (example here), scenario file (example here) and your agent.
For routes, you need an xml with at least one route. Each route needs id (to differentiate between them), town (name of the map where they'll be run) and a list of waypoint. These serve as reference and SR will search for the shortest path between them, creating waypoints every meter (by default). Done here. Optionally, you can use weather to set a specific weather (you can just set one of the weather parameters, all of them, or any in between).
For the scenario file, you can set as many scenarios as you want, and SR will filter those your route passes through. This is probably the most useful and confusing feature routes have. The correspondence between the scenario names at the scenario file and their python class is done here. This is probably the second most confusing feature.
Hope this helps, and yeah, everyone gets confused by routes at first.


[Bug Report]: Registration role initial state does not align with settings view

Describe the Bug
On a fresh setup, the "Registration Role" option will typically show "Admin" whereas this settings is actually empty in the system. Admin is only selected due to the setting control defaulting to the first role in the system. When registered this role would not be used unless the registration settings box was saved to actually set that option.
Exact BookStack Version
<= v21.12.3

Closed in 17215431caec61334cd653e030ba3b42c7ef2ca1


Difference between layers

Hello,
Could you please tell me what is the difference between "GaborSmallConv2d", "GaborConv2d" and "GaborCurvedConv2d"? from the math point of view.

Hi. Sorry for the delay.
I updated repo: created package and added usage guide!
Also I complitely removed  GaborSmallConv2d and GaborCurvedConv2d due to the fact that their expediency has not been proven in the article.


Unable to install alchemy in spree (3.4.4) project

Hi,
I am unable to install alchemy (4.0-stable) in existing spree (3.4.4) with rails (5.1.6) solution.
getting below error-

Bundler could not find compatible versions for gem "cancancan":
In snapshot (Gemfile.lock):
cancancan (= 2.2.0)
In Gemfile:
alchemy_cms was resolved to 4.0.1, which depends on
cancancan (~> 1.9)
spree (~> 3.4.4) was resolved to 3.4.5, which depends on
  spree_core (= 3.4.5) was resolved to 3.4.5, which depends on
    cancancan (~> 2.0)

Duplicate of https://github.com/AlchemyCMS/alchemy_cms/issues/1417


fix: include ajv in devDependencies for Expo Router and npm

Please verify the following:

[ ] yarn test jest tests pass with new tests, if relevant
[x] yarn lint eslint checks pass with new code, if relevant
[x] yarn format:check prettier checks pass with new code, if relevant
[x] README.md (or relevant documentation) has been updated with your changes
[x] If this affects functionality there aren't tests for, I manually tested it, including by generating a new app locally if needed (see docs).

Describe your PR

This would resolve one of the issues in https://github.com/infinitered/ignite/issues/2840, although there are additional problems with the Expo Router template.

This is a little bit of a sledgehammer approach, but I wanted to start with the simple solution and we can enhance it if that seems necessary.
In https://github.com/infinitered/ignite/pull/2823, we added the --legacy-peer-deps flag for npm install commands. This fixed a problem upstream from https://github.com/standard/eslint-config-standard/issues/412, but I believe it has opened up a new problem for users who choose to use Expo Router and npm. Based on the issues that Expo Router + ESLint has experienced with how bun resolves peer dependencies, I am guessing the legacy peer dependency resolution falls victim to the same problem.
The very simplest way to solve this is if the generated Ignite project includes ajv at the right version number in its own devDependencies. This version "wins" across the board.
The drawback to this approach is that if Expo Router changes its required ajv version, there may be some new errors for the project to resolve.
Another drawback to this PR is that it would add ajv to all projects, including other package managers that don't suffer from this problem, and including projects that do not opt in to Expo Router. So we could alternatively run npm install ajv@^8 --legacy-peer-deps in the install command right after https://github.com/infinitered/ignite/blob/e59d880c951a8ec729ca65f39ba0dae1de27a729/src/tools/packager.ts#L149, and maybe even do a check for if Expo Router is enabled.
Screenshots (if applicable)
No screenshots, but running node ~/ignite/bin/ignite new project-name and choosing npm and Expo Router successfully works with this change.

16e30eb is the more targeted approach, but we can easily drop that commit if simple is preferred.

Yeah, eslint-plugin-n needs to fix their peer dependencies, as in https://github.com/standard/eslint-config-standard/issues/412 (this is why we added --legacy-peer-deps for our own issue over in https://github.com/infinitered/ignite/issues/2818). Once eslint-plugin-n resolves their issue, we should be able to remove --legacy-peer-deps from our commands here and then remove this patch as well.

@coolsoftwaretyler ah yeah I had a discussion about this in Slack. config-standard is abandoned I think for neostandard in that issue thread https://infinitered.slack.com/archives/C0X0P5LP7/p1730325079888599
should we look to swap it out for neo and see if n just works or cut back on all these extra config / plugins?

@frankcalise - if we swap in neostandard and it Just Works, I think that's ideal. I can take a look sometime this week to make sure it doesn't cause any changes while linting a freshly ignited project.
Want to merge this one for now to patch up the ajv issue in the short term and I'll get back to ya in a few days on neostandard?

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Adds a standalone job for running decrypt private membership queries

Adds a standalone decrypt job to the panel exchange tools.

This change is


Memory game

I want to create a Memory game, please assign me under DCP'21

@Naman-mahi It's already there.

But this game was different

@Naman-mahi This can be added as an improvement to the Flip-me game. If that's good with you, I'll reopen the issue.

Okay

@Naman-mahi go ahead


data augmentation by rotating the image

When implementing data augmentation by rotating the image, How to adjust the target position information accordingly?

Where you able to get an answer or figure this out?


fby3.5: rf: Initialize rainbow fall BIC

Summary:

Initialize Rainbow Falls BIC.
Create project config, dts files, file header, and CMake file.
Add project code in meta-facebook/yv35-rf/.

Test plan:

Build code: Pass
Can access BIC console: Pass

Log:

Can build pass.
openbic$ west build -p auto -b ast1030_evb meta-facebook/yv35-rf
-- west build: generating a build system
...
[244/244] Linking C executable zephyr/Y35BRF.elf
Memory region         Used Size  Region Size  %age Used
SRAM_NC:         12 KB       320 KB      3.75%
FLASH:          0 GB         0 GB
SRAM:      343752 B       448 KB     74.93%
IDT_LIST:          0 GB         2 KB      0.00%
248284+0 records in
248284+0 records out
248284 bytes (248 kB, 242 KiB) copied, 0.759969 s, 327 kB/s
0+0 records in
0+0 records out
0 bytes copied, 5.4698e-05 s, 0.0 kB/s

Can access BIC console.
*** Booting Zephyr OS build v00.01.04-5-g70b0b6ecc605  ***
Hello, welcome to yv35 Rainbow Falls 2022.20.e3
Function pal_fix_full_sdr_table is not implemented
The SDR number is equal to 0
ipmi_init

With Rainbow Falls moving to a BIC-less design, are these changes still necessary?

Hi Michael,
For the next version Rainbow Falls board, they are not necessary but can be reference code.
For the first version (current version) RF, we still need to complete the hardware validation and send the boards to Meta. We need a versioned BIC firmware to record it.

Thanks Delphine for confirming.  We should keep track of the PRs that are temporary so we can pull them back out once we move to the new BIC-less design.

Hi @GoldenBug ,
Force update to reabse main and fix the conflicts in common/service/ipmi/ipmi.c.
Please help to review again, thank you.


[Feature Request] Closing menu with Escape key

Hello,
Thanks a lot for your super useful alt-tab dwm patch.
It would be nice to have the menu close using Escape key like it does in other alt-tab implementations.
Thanks in advance!

Hi,
Apologies for the late response. College has kept us quite busy lately.
I've found your feature request really interesting, so I went ahead and implemented and I've created a new patch for the repository.
The patch will come to the suckless webpage later.
You can checkout the changes and leave me feedback. If everything is okay we can close issue and I will add patch to the suckless webpage.

Yeah, Everything works just fine, and there doesn't seem to be any issues.
May Allah bless you with success in your college, and thank you Villiam so much for your time.