Android Basics: Viewmodel and State in Compose

On the 5th step on this pathway link below;
https://developer.android.com/codelabs/basic-android-kotlin-compose-viewmodel-and-state?continue=https%3A%2F%2Fdeveloper.android.com%2Fcourses%2Fpathways%2Fandroid-basics-compose-unit-4-pathway-1%23codelab-https%3A%2F%2Fdeveloper.android.com%2Fcodelabs%2Fbasic-android-kotlin-compose-viewmodel-and-state#7
The game UI state is given as this;
data class GameUiState(
val currentScrambledWord: String = "",
val currentWordCount: Int = 0,
val score: Int = 1,
val isGuessedWordWrong: Boolean = false,
)
but it should be this; just as step 5 says - "Add another variable for the count in the GameUiState. Call it currentWordCount and initialize it to 1"
data class GameUiState(
val currentScrambledWord: String = "",
val isGuessedWordWrong: Boolean = false,
val score: Int = 0,
val currentWordCount: Int = 1
)

Duplicate of #55

Hello @uosamuel,
Thank you for reaching out to us and reporting this typo. The code snippet is updated:


Update README.md

Update summary.  Add TOC that only takes up one line on desktop screens.  Mention new toarray struct tag feature. Mention CBOR tags (type 6) is under consideration as new feature.

Codecov Report

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

@@           Coverage Diff           @@
##           master      #32   +/-   ##
=======================================
  Coverage   97.24%   97.24%
=======================================
  Files           6        6
  Lines        1269     1269
=======================================
  Hits         1234     1234
  Misses         24       24
  Partials       11       11

Continue to review full report at Codecov.

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

Thanks again!  I especially like the TOC as one-liner!


[BC Breaking] Replace sox_effects init/list/shutdown with TS binding

This PR replaces sox effects init/list/shutdown functions with Torch Script binding. (SoxEffectsChain is not changed yet.)
I also moved error checking to C++, so the function signature is changed -> BC breaking.
The previous implementation returned sox_errot_t enum to Python, which is practically useless so I think this BC breaking does not have an impact.

This means that sox_effects would be torchscriptable, right? If so, could you add a test for this?

This means that sox_effects would be torchscriptable, right? If so, could you add a test for this?

Nope. the main sox_effects are more complicated so it's not included in this PR, and there is not really way to test init/shutdown/list in the current framework.

thanks


Add [workspace] to forc manifest

Description

This PR adds a [manifest] to the build-forc-test-project job's cargo manifest

Testing Steps

[ ] CI should pass and build-forc-test-project specifically should have no errors during any steps

@adlerjohn Hey yea I see. Currently away, but I’ll address it when I get back 👌🏽

Filed #1464 to follow up on the above-noticed bug.


Feature request?

Hi Carl,
I have a suggestion or a 'feature' request, not sure how complex it would be to get this implemented or maybe you know a way around this already?
I'm looking to use Scrapli to logon to a device which has literally been pulled out of a box brand new with no config on. It would therefore be stuck in the configuration dialog i.e.:
Would you like to enter the initial configuration dialog? [yes/no]:
Scrapli times out as it can't find a prompt.
Thank you

This can be handled already.
TL;DR: disable auth (auth_bypass: true), then use read callback and/or just a while true and read off the channel and handle sending input however you want.
You can look at boxen for examples (its go but its functionally the same)

Thank you, I think I'm being thick but I can't figure out how to use
read_callback() in this instance :| I do have authentication on the console
server so I wouldn't need to disable it. I guess you wouldn't have a dumbed
down example on how i'd string match "dialog" either from the channel or
callback method and send "no" ? :)
On Thu, 22 Sept 2022 at 17:35, Carl Montanari @.***>
wrote:

This can be handled already.
TL;DR: disable auth (auth_bypass: true), then use read callback
https://github.com/carlmontanari/scrapli/blob/main/scrapli/driver/generic/sync_driver.py#L446-L525
and/or just a while true and read off the channel and handle sending input
however you want.
You can look at boxen for examples (its go but its functionally the same)
—
Reply to this email directly, view it on GitHub
https://github.com/carlmontanari/scrapli/issues/259#issuecomment-1255277535,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AD2CGUA6CQ4BAMZZUD2IZBDV7SDFDANCNFSM6AAAAAAQTGOSHU
.
You are receiving this because you authored the thread.Message ID:
@.***>


remove installing ginkgo using "go get"

Remove install ginkgo using go get as the current supported version is 2.0 and 2.0 still has some conflicting changes. We can rely on the ginkgo binary compiled in kubernetes repo from the current version.
Fixes: https://github.com/kubernetes/kubernetes/issues/108856

/test pull-npd-e2e-kubernetes-gce-gci-custom-flags

https://prow.k8s.io/pr-history/?org=kubernetes&repo=node-problem-detector&pr=657 tests are passing now


MaxAttempts and error handler

Add MaxAttempts and offload to an error handler if things are really backed up. This is a critical state for most applications but beats dropping them on the floor.

We had a case that there was a validation error for "PartitionKey" (bogger than 256 chars) and it got stuck in an infinite loop of retries.


Download video

Use "download video Song_Name". It will be created on the desktop. You should have "pytube" installed. I have included it in the requirements.txt.

A much appreciated PR! One of the best in this project!


Add missing RenderContentMetadata schema to Linkable Entities spec

Bug/issue #, if applicable: rdar://87100087
Summary
This adds a missing RenderContentMetadata schema to LinkableEntities spec, making it a valid OpenAPI spec.
Note: like the RenderNode spec, this one is recursive because RenderInlineContent can be Image which has a RenderContentMetadata which has a RenderInlineContent. This is valid for OpenAPI specs.
Dependencies
n/a
Testing
n/a
Checklist
Make sure you check off the following items. If they cannot be completed, provide a reason.

[ ] Added tests
[ ] Ran the ./bin/test script and it succeeded
[ ] Updated documentation if necessary

@swift-ci please test


Cardputer Keyboard?

https://docs.m5stack.com/en/core/Cardputer

74HC138 CMOS?
Seems to be a rust implementation here: https://github.com/syurazo/cardputer/blob/master/src/keyboard.rs
Pins as seen here: https://github.com/m5stack/M5Cardputer/blob/889cbd7b9d8c395205e22e924a383bd07089b14c/src/utility/Keyboard.h#L28C1-L29C62
const output_list = [8, 9, 11];
const input_list  = [13, 15, 3, 4, 5, 6, 7];

I've tried everything in the docs. I'm new to keyboards, serial, and i2c I have no clue to get button presses to register a console log.
Is this something currently supportable by the project? I'm just worried that somewhere down the line devicescript will just not work for some piece of hardware on the cardputer. Can someone working on the team confirm that keyboard, speakers, bluetooth, sd card, and mic can all work? I got the LED on the board to work and the LCD so far.
I'm a bit confused I know this device can be used to type within the device, and be used when plugged in or via bluetooth can act as an external keyboard over USB (is this what a HID Keyboard is). Is it possible to do both via devicescript?

Let's see:

you will need to add i2s support in a similar fashion to i2c to support the mic.
you should be able to port the Rust driver to devicescript using the gpio module and detect the keystrokes.

can act as an external keyboard over USB (is this what a HID Keyboard is)

Yep.

Is it possible to do both via devicescript?

We do have hid keyboard driver in jacdac (https://github.com/microsoft/jacdac-c/blob/main/services/hidkeyboard.c) but i don't think it is fully wired up for devicescript. @mmoskal
Anyhow, it's gonna require work and somewhat getting deep in building devicescript itself.

@pelikhan thanks for this, I'm digging into the source. I'd down to contribute to the project.
All I want right now is to be able to press any key on the device and console.log. This is not HID, just normal internal keyboard.
I can't find this code: https://microsoft.github.io/devicescript/api/clients/keyboardclient
When I search the project for KeyboardClient nothing shows up 🙃
Where do I start?

All I want right now is to be able to press any key on the device and console.log. This is not HID, just normal internal keyboard.

For that you would need to implement the pin scanning code, just like the rust sample you mentioned.

@pelikhan I got this working so far for this device I got the following working:
[x] Onboard LED
[x] LCD 16bit colors
[x] Keyboard
I'd like to put all this in a module that people can reuse. I think I'm interested in making something more custom.
However modules like @devicescript/drivers aren't in the npm (or github) registry.
How can I make my code into a package?

Note that we'll be happy to take your drivers contributions back into @devicescript/drivers as well!

Thanks done and done
https://www.npmjs.com/package/cardputer
https://github.com/reggi/cardputer


libnmstate: Support ovs-bridge

OVS bridges are now reported with their supported properties and
connected ports/intefaces.
Note: OVS bonds and internal interfaces are not supported yet.

Coverage decreased (-11.7%) to 80.146% when pulling 4faadd338a16a65e09fa0e69d7d3c6231231e256 on EdDev:ovs-bridge into b338149853ba8797024e18e5e880ef42e0499c64 on nmstate:master.

Known problems:

Removing a bridge is possible, but it will not delete the port profiles and will detach the current interface profiles.
The port properties, like 'vlan-tag' have not been tested at all.

The schema does not match, not sure how I got here:
nmstatectl show
Traceback (most recent call last):
  File "/root/.local/bin/nmstatectl", line 10, in <module>
    sys.exit(main())
  File "/root/.local/lib/python2.7/site-packages/nmstatectl/nmstatectl.py", line 44, in main
    return args.func(args)
  File "/root/.local/lib/python2.7/site-packages/nmstatectl/nmstatectl.py", line 90, in show
    state = netinfo.show()
  File "/root/.local/lib/python2.7/site-packages/libnmstate/netinfo.py", line 25, in show
    validator.verify(report)
  File "/root/.local/lib/python2.7/site-packages/libnmstate/validator.py", line 33, in verify
    js.validate(data, validation_schema)
  File "/root/.local/lib/python2.7/site-packages/jsonschema/validators.py", line 541, in validate
    cls(schema, *args, **kwargs).validate(instance)
  File "/root/.local/lib/python2.7/site-packages/jsonschema/validators.py", line 130, in validate
    raise error
jsonschema.exceptions.ValidationError: {'state': 'down', 'type': 'ovs-interface', 'name': 'testbvr', 'ipv4': {'enabled': False}} is not valid under any of the given schemas

Failed validating u'oneOf' in schema[u'properties'][u'interfaces'][u'items']:
    {u'oneOf': [{u'allOf': [{u'properties': {u'type': {u'enum': [u'unknown'],
                                                       u'type': u'string'}}},
                            {u'$ref': u'#/definitions/interface-rw'},
                            {u'$ref': u'#/definitions/interface-ro'}]},
                {u'allOf': [{u'$ref': u'#/definitions-ethernet/interface'},
                            {u'$ref': u'#/definitions/interface-rw'},
                            {u'$ref': u'#/definitions/interface-ro'}]},
                {u'allOf': [{u'$ref': u'#/definitions-bond/interface'},
                            {u'$ref': u'#/definitions/interface-rw'},
                            {u'$ref': u'#/definitions/interface-ro'}]},
                {u'allOf': [{u'$ref': u'#/definitions-ovs-bridge/interface'},
                            {u'$ref': u'#/definitions/interface-rw'},
                            {u'$ref': u'#/definitions/interface-ro'}]},
                {u'allOf': [{u'$ref': u'#/definitions-dummy/interface'},
                            {u'$ref': u'#/definitions/interface-rw'},
                            {u'$ref': u'#/definitions/interface-ro'}]}],
     u'required': [u'name'],
     u'title': u'Interface',
     u'type': u'object'}

On instance[u'interfaces'][4]:
    {'ipv4': {'enabled': False},
     'name': 'testbvr',
     'state': 'down',
     'type': 'ovs-interface'}

The commit message of the last commit starts with bnmstate instead of libnmstate.

I created the testbvr with ovs-vsctl add-br testbvr.

You used an interface type which is not supported yet: "ovs-interface". It is used for OVS internal interface and other OVS created devices. For the moment, only system interfaces are supported.
For bridges, you need to use "ovs-bridge".

It happens with nmsate show, so it is reporting something that it does
not support in the schema.
Edward Haas<EMAIL_ADDRESS>schrieb am Do., 5. Juli 2018, 13:29:

You used an interface type which is not supported yet: "ovs-interface". It
is used for OVS internal interface and other OVS created devices. For the
moment, only system interfaces are supported.
For bridges, you need to use "ovs-bridge".
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/nmstate/nmstate/pull/45#issuecomment-402692470, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAMEWDzalbtw18XkP_2bNQxdwplvNpHZks5uDfhigaJpZM4U-Thk
.

Ok, I see it now. It can be reported but cannot be defined and it is not appearing in the schema.
I will add it to the schema and raise if it is detected on set.
Thanks.

@tyll , I pushed the changes. Could you try it now?

@EdDev looks better now, thank you!


[Super Token Listing Request] USDbCx on Base

Project infos and contact (Website, Email, Telegram and/or Twitter handle):
https://help.coinbase.com/en/coinbase/getting-started/crypto-education/usd-base-coin
Link to the verified Super Token contract on a Block Explorer:
https://basescan.org/address/0x5eacf6a1256d71182df319b38fddec30e23c5851#code
Link to a logo of the token in SVG (preferred) or PNG format:
https://basescan.org/token/images/usdbc_ofc_32.png
Link to the coingecko entry of the token:
https://www.coingecko.com/en/coins/bridged-usd-coin-base/

Hey @bertux I'm afraid there is already a listed USDbC on base https://console.superfluid.finance/base-mainnet/supertokens/0x4db26c973fae52f43bd96a8776c2bf1b0dc29556?tab=streams


Integrated editor in webview

Summary
Allow to create our own "Interactive Playgrounds" mixing HTML and integrated editors.
Description
VS Code has a nice interactive playground, which is a webview with integrated editors.
Help > Interactive Playground

There editors embed all the features of a default text editor:

syntax highlighting
autocompletion
extension contributions
...

I would like to create a webview containing editors like that.
Currently, we can only create webviews with an HTML textarea, like this.
The content of the Interactive Playground is here
https://github.com/microsoft/vscode/blob/72c700cf5419663c7bf833308233cc760709bcaf/src/vs/workbench/contrib/welcome/walkThrough/browser/editor/vs_code_editor_walkthrough.ts#L132-L143
Relates to #77131 even if this can apply to webviews that are not custom editors

Duplicate of #93265


v1.4 backports 2019-02-10

For after tagging; PRs:

#7013
#7012
#7009
#7002
#6986
#6973
#6915

This change is

test-me-please

test-docs-please

test-missed-k8s

Hit timeout, retesting.

test-me-please

test-missed-k8s

Merging so we can install new version into testing cluster


Splitting pane with shortcut opens wrong terminal session

Windows Terminal version
Version: 1.19.11213.0
Windows build number
Microsoft Windows [Version 10.0.22631.3447]
Other Software
No response
Steps to reproduce

open terminal, open any session that is not configured as the default
use shortcuts to split pane alt + shift + -/=

Expected Behavior
It opens up a new pane using the profile/tab I split on
Actual Behavior
It creates a new pane with the default profile

1.19

We changed this behavior to be what you’re expecting in 1.21. It turns out that roughly half of people expect it to be one way and the other half the other way!
You can rebind your split pane keys in the Actions page of settings while you wait for 1.21 to come out 🙂


VSCode Telemetry non-compliance

Before submitting your bug report

[X] I believe this is a bug. I'll try to join the Continue Discord for questions
[X] I'm not able to find an open issue that reports the same bug
[X] I've seen the troubleshooting guide on the Continue Docs

Relevant environment info
doesn't matter

Description
Dear team,
Continue is a great free and open-source extension, however there's a small compliance issue regarding telemetry. According to Microsoft's "Telemetry extension authors guide":

Extension authors who wish not to use Application Insights can utilize their own custom solution to send telemetry. In this case, it is still required that extension authors respect the user's choice by utilizing the isTelemetryEnabled and onDidChangeTelemetryEnabled API. By doing this, users will have one centralized place to control their telemetry settings

Digging through a codebase I haven't found any reference to env.isTelemetryEnabled and env.onDidChangeTelemetryEnabled.
PS Gitlens does a good job on that with settings' note:

References:
https://code.visualstudio.com/docs/getstarted/telemetry#_for-extension-authors
https://code.visualstudio.com/api/extension-guides/telemetry
To reproduce
No response
Log output
No response

I did not know about this, thank you for pointing it out. Fix made here: https://github.com/continuedev/continue/commit/99f50fc5f3999ee683477f9de44220c1dc4ef9eb


Include path node_modules folder

Hi
Is there a way i can include a node modules/@progress scss file in my base.scss?
When i try to reference it in my base.scss file and see the output i get an error:
SassCompilationException: Error: Can't find stylesheet to import. at wwwroot/css/base.scss:45:9 -> @import<EMAIL_ADDRESS>

@htodep This might be an issue with it referencing it from a relative path from wwwroot so it could be looking for  wwwroot/node_modules which wouldn't exist. Maybe copying the files on build/publish time might be an option?

Unfortunately not using WebOptimizer.Sass anymore. It didn't work for me. thanks for responding

What are you using instead?

From: htodep @.>
Sent: Monday, April 25, 2022 9:06:06 AM
To: ligershark/WebOptimizer.Sass @.>
Cc: Benjamin Stern @.>; Comment @.>
Subject: Re: [ligershark/WebOptimizer.Sass] Include path node_modules folder (Issue #44)
Unfortunately not using WebOptimizer.Sass anymore. It didn't work for me. thanks for responding
—
Reply to this email directly, view it on GitHubhttps://github.com/ligershark/WebOptimizer.Sass/issues/44#issuecomment-1108112647, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAJVIJ6KE5M46QUUTLTTNV3VGYY45ANCNFSM5N2CCZ5A.
You are receiving this because you commented.Message ID: @.***>

Webcompiler for VS 2022. It's updated and supports new scss/css functions.


blocked by CORS policy

loadImage(       'https://st.grafia.ink/media/upload/46b38f192dfeb27362d03bd8234e28a5.jpg,       (img, data) => {         console.log(data);       },      { meta: true }, )
response - "Access to fetch at 'https://st.grafia.ink/media/upload/46b38f192dfeb27362d03bd8234e28a5.jpg' from origin 'http://localhost:3100' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource. If an opaque response serves your needs, set the request's mode to 'no-cors' to fetch the resource with CORS disabled"
How do I put mode: "no-cors" in loadImage?

loadImage will try to fetch an image provided via URL as Blob object if it determines that meta data (e.g. EXIF) is required, e.g. by setting the option meta:true as in your example:
https://github.com/blueimp/JavaScript-Load-Image/blob/5dcae765fe7638328e22f7378f408160c4528247/js/load-image.js#L169-L173
It will fall back to load the img directly via its URL if the Blob cannot be loaded - so in many cases, you can ignore such an error message:
https://github.com/blueimp/JavaScript-Load-Image/blob/5dcae765fe7638328e22f7378f408160c4528247/js/load-image.js#L141-L149
To set the no-cors option for the fetch request. you can provide it as option to loadImage, as all options are passed on to the fetch call:
loadImage(url, callback, {
  mode: 'no-cors',
  meta: true
})

However, this will very likely lead to an error accessing the Blob and you won't be able to load the image at all.
So my recommendation is to either fix the CORS headers on the server-side from where you want to load the image, or accept that you won't be able to load meta data and fall back to loading the image only.


problems with new version

Hello Eric,
The new version 0.2.0 gives me the following error term.
Do you have an idea what went wrong?
The previous version seems to run fine.
King regards
Lars Svensson
C:\Python27\python.exe C:/Users/lay/PycharmProjects/test/test.py
Traceback (most recent call last):
File "C:/Users/lay/PycharmProjects/test/test.py", line 2, in
import gtrends
File "C:\Users\lay\AppData\Roaming\Python\Python27\site-packages\gtrends.py", line 12, in
from _login import Downloader
File "C:\Users\lay\AppData\Roaming\Python\Python27\site-packages_login.py", line 1, in
import six
ImportError: No module named six
Process finished with exit code 1

Hmm. So I used the package 'six' to help with the different string representations between python 2 and python 3. I must not have included that with the other dependencies.

I just added six as a dependency and pushed a new version. It should now work fine.

Perfect! Thank you!
is also possible to check for expressions with more than one word? For example "great depression".

Yes, you can check for multi-word terms. The space will get encoded as %20
automatically.
Good luck,
Eric
On Sat, Jun 18, 2016 at 6:09 AM, Lsvensson<EMAIL_ADDRESS>wrote:

Perfect! Thank you!
is also possible to check for expressions with more than one word? For
example "great depression".
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/ecsalina/gtrends/issues/3#issuecomment-226933787, or mute
the thread
https://github.com/notifications/unsubscribe/AHnG6pxlwDkGhq_wqIHOx9LwkQwXIVcSks5qM8PKgaJpZM4I4cuY
.


Typo: form -> from

I know typo-pr's are lame, but PR is quicker than sending email ;)

don't worry I try to accept all PRs if it makes sense :+1: thanks!


A question

IS this just a trigger or can i access the info in the SQS queue as input to my build project?
I want to pull the InstanceID from the SQS message run a script to find out some info such as IP address and used these are variable with in the build process. but I can't see how to do this?
Thank you

+1 - I was hoping that the payload would be inserted as environment variable? Then any underlaying script can do further processing?

@lifeofguenter thanks, closing this one.


Local filenames are not URL encoded

Describe the bug
When using a local source, filenames are not URL encoded and cause 404s
To Reproduce
Steps to reproduce the behavior:

Create a local file source
Create a label template that includes displaying an image, like <Image name="image" value="$url" />
Copy an image file with a + into the file store
Try to label the image
The image will not load because the URL is not correctly encoded

Expected behavior
The image should be URL encoded so it can be displayed.
Environment (please complete the following information):

OS: Docker on Ubuntu 2022.04
"Latest" tag on Docker

@brettp could you please record a video?

@makseq - Here's a video showing my setup with two files that have characters that should be URL encoded but aren't, so cause 404s. When I change the characters to their correctly encoded version in the URL, they load as expected.
https://user-images.githubusercontent.com/43375/166114826-0e458665-997a-4948-8b29-552d2c4ebe29.mp4

Thanks for this great video, we will fix it soon.


Don't require login from the CLI if generating code locally

fern login is for monitoring and controlling access to premium features (like publishing and API documentation generation).
If a user wants to try Fern's free tier without logging in, they should be able to. The --skip-login flag may be a good option.

@mjperrone @aevitas @kikones34 @MarcSalvat, I'm closing this issue in favor of this one: https://github.com/fern-api/fern/issues/2003.
If you have any comments on the proposal in the issue, let me know!


due to keepalive expired

I keep getting the following logs for a client
client [warning] <>@vmq_mqtt_fsm:connected:420 client {[],<>>} with username <<"">> stopped due to keepalive expired
Is this due to configuration on my end or does the client for some reason just lose connection? How can I see what keeplive values each client is using when they connect
Thanks

Hi @yama0615 thanks, I think it's either the client dropping the connection or the client not honoring it's own keepalive window (ie not sending pings).
@larshesel can you think of a way to show the keepalive of a session? (I think it's not in the session command, see vmq-admin session show --help)

@larshesel can you think of a way to show the keepalive of a session?

No, the keepalive isn't exposed via the command line. The only way I know of seeing the keepalive is if a trace is active when the client connects, see: https://docs.vernemq.com/administration/tracing

Is this due to configuration on my end or does the client for some reason just lose connection?

Hard to say - the only fact known at this point is that the client set some keepalive timeout but no keepalive PINGREQ was received before 1.5*keepalive time expired.


Access to Rate Limit headers

Hello, I noticed the ApiError has access to the response headers. However, I was wondering if there is a way to access the headers on a success response too?
On any given successful response there is also rate-limit information passed back in the headers.
https://dev.twitter.com/rest/public/rate-limiting
X-Rate-Limit-Limit: the rate limit ceiling for that given request
X-Rate-Limit-Remaining: the number of requests left for the 15 minute window
X-Rate-Limit-Reset: the remaining window before the rate limit resets in UTC epoch seconds

If this is currently not available let me know if you have thoughts on where this could fit in the library. I'd be happy to help build it in.

Yeah, I've actually been thinking about adding (optionally) better logging in general. The trick is that we don't want to pollute people's logs when they don't want this, so it has to be optional. The straightforward way is to have a logger that only outputs values if a certain flag is set. This is an interface that's pretty easy to implement wrapping the stdlib's logger.
Another option is to use a channel that can feed on logging values emitted by the API client. This is similar to what Go lexers (including the template package, IIRC) do. There are some advantages to doing this, but it may be overkill here, especially if this is the only place that this sort of logging is needed.
Could you elaborate a bit more on the use case here (the specific problem you're running into, in which the headers would be helpful)? That may help us to figure out the best way to add this while maintaining the same interface for API calls that people are used to.

OK sounds good. Thanks for the quick reply.

Could you elaborate a bit more on the use case here (the specific problem you're running into, in which the headers would be helpful)?

I was thinking it would be nice to have access to the rate limit information (for successful queries too). As I think about it some more though the Throttling functionality should suffice, and then we could log when we hit our limits.

@harlow This is fixed, as of bdb4b76. I didn't realize how simple it would be; I forgot we already had the logger from the streaming API (which is separate on Twitter's end, but included in this library).
Let me know if this satisfies what you're looking for - at this point, it'd be easy to tweak it as needed.


Add cosign aggregate transactions command

Description
At the moment, it is quite challenging to cosign transactions while testing the Catapult Developer Preview. Every time an account has to cosign an aggregate bonded transaction, a developer should develop and run a custom program.
https://nemtech.github.io/guides/transaction.html#signing-announced-aggregate-bonded-transactions
Proposed solution

Add transaction hash when listing  transactions using nem2-cli:

nem2-cli account aggregatebonded
nem2-cli monitor aggregatebonded
..

Include a new command  nem2-cli transaction cosign.

USAGE
nem2-cli transaction cosign [...options]
OPTIONS
--profile          - (Optional) Select between your profiles, by providing a profile name
-h, --hash  - Transaction hash to be cosigned

Added in v0.9.5


webstuff

subdcel got its own file
in between commit :D
add new testgraph for mergetest
merge dcel vertices part 1 ( broke subDcelBuilder :( )
merge dcel vertices part 2
merging vertices propably done
improved visualizer (broke donut triangulation visualization:D )
fixed selfreferencing arcs after vertexmerge and added fake_root to donut
fixing error on merge_vertex example graph + a lot of debugprints
some stuff that did not fix our problems
some test data folders
work on vertex merge
some test files
still trying to merge the vertices correctly
main stuff
test data
main
approxtd from sub dcel is gettingg better
logging sub td building to file
hopefully a working SubApproxTd
no fake root in bags + print to log
merging independent sets
whole graph working
Add new command line option for complete td solve.
WIP
remove compound reductions
Diagnostics modal
use input reductions for graph
also work when no layout file available
Add All-With-TD option
fix
fix
fix visualization
no need to move executable
Add extra log folder for all logs.
show verts in donuts and rings
didnt wanna commit this
Add the logs folder to git ignore.
Add find mis exhaustive.
can show mis
try to support windows
Add exhaustive maximum independent set search.
stuff
fix restoring of indices
Update find mis with exact tree decomposition.


bug: not working

Describe the bug

To Reproduce
not bind port 80
Expected behavior
1234
bancho.py Version
last
Python Version
3.9.X (Default)
Relevant log output
[03:48:39PM] Fetching channels from sql.
[03:48:39PM] Fetching clans from sql.
[03:48:39PM] Fetching mappools from sql.
[03:48:39PM] Initializing housekeeping tasks.
[03:48:39PM] Startup process complete.
[03:48:39PM] Listening @ <IP_ADDRESS>
ERROR:    [Errno 13] error while attempting to bind on address ('<IP_ADDRESS>', 80): permission denied
[03:48:39PM] -> Cancelling 3 housekeeping tasks.
make: *** [Makefile:30: run] Error 1

Additional context
No response

you're supposed to use something else than port 80

only root (admin) can bind to port 80
you likely want to use something like nginx or caddy to bind to 80 (something very battle-tested in industry) rather than using bancho.py - which is what we recommend in our documentation
it is much more likely there are bugs in bancho.py than something like nginx, and as a general practice you should avoid running things as admin, whenever possible (minimize surface area of administrative privileges)


[TorchBench] Performance Signal Detected

TorchBench CI has detected a performance signal.
Affected Tests:

eval-cuda-fp32:

dlrm[disc (latency)] 1.764 -> 1.861, -5.4989%
drq[dynamo-blade (latency)] 1.296 -> 1.4, -8.0247%
functorch_maml_omniglot[dynamo-disc (latency)] 0.547 -> 0.663, -21.2066%
hf_Bert_mini[blade (latency)] 0.539 -> 0.653, -21.1503%
hf_Bert_mini[dynamo-disc (latency)] 1.441 -> 1.566, -8.6745%
hf_GPT2[disc (latency)] status changed, AttributeError -> 18.078
hf_GPT2[blade (latency)] status changed, AttributeError -> 32.259
hf_GPT2[disc (compiled)] status changed, N/A -> 2228.0
hf_GPT2[disc (clusters)] status changed, N/A -> 1.0
hf_GPT2_large[disc (latency)] status changed, AttributeError -> 102.772
hf_GPT2_large[blade (latency)] status changed, AttributeError -> 190.692
hf_GPT2_large[disc (compiled)] status changed, N/A -> 6596.0
hf_GPT2_large[disc (clusters)] status changed, N/A -> 1.0
phlippe_resnet[disc (latency)] 2.463 -> 2.214, +10.1096%

detail data can be seen in oss://bladedisc-ci/TorchBench/gpu/full/20230505-15
created by TorchBench CI automatically

Update baseline


react-native launch fail after changing app name

VSCode Version: 1.6.1
OS Version: mac 10.11.6

Steps to Reproduce:

Create new react-native app
Build it with vscode
Change the app name via Xcode
Try building with vscode
build succeed.
launch fail
I get error:

Touch Build/Products/Debug-iphonesimulator/NewMyApp.app
cd /Users/myUser/git/myApp/ios
export PATH="/Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/usr/bin:/Applications/Xcode.app/Contents/Developer/usr/bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin"
/usr/bin/touch -c /Users/myUser/git/myApp/ios/Build/Products/Debug-iphonesimulator/NewMyApp.app
** BUILD SUCCEEDED **
// vscode is trying to install my old app name -->
Installing build/Build/Products/Debug-iphonesimulator/OldMyApp.app
An error was encountered processing the command (domain=NSPOSIXErrorDomain, code=2):
Failed to install the requested application
An application bundle was not found at the provided path.
Provide a valid path to the desired application bundle.
Print: Entry, ":CFBundleIdentifier", Does Not Exist
Command failed: /usr/libexec/PlistBuddy -c Print:CFBundleIdentifier build/Build/Products/Debug-iphonesimulator/OldMyApp.app/Info.plist

Please file this against the vscode react extension here https://github.com/Microsoft/vscode-react-native
Thanks


Fix broken index in Config array.

Was receiving index error on Laravel 4.2. The api key in the config array had a key of 'key', not 'api_key'. - How's that for a tongue twister.
I'm surprised nobody else was receiving this issue. Unless there's something glaringly obvious I've skipped over.

You should be using api_key in your config file, are you sure you are doing that?

Yeap, I checked that. Either I'm the only one running 4.2 using this package, or something seriously crazy is going on. I've tried isolating everything, but that config array contains the key with 'key'.

Thanks for letting us know, @kyranb! We'll try to add some documentation that helps clear this up for people.


Usage of the BLS signature scheme should mandate a "proof-of-possession" check

there is a well-known attack on the BLS signature scheme called a "rogue public key" attack
you can read more about it here: https://hackmd.io/@benjaminion/bls12-381#Rogue-key-attacks
the mitigation is straightforward: publish a "proof of possession" along w/ the public key.
given that this spec current requires builders to sign over their messages, we should also specify that builders publish a "proof-of-possession" alongside their public key and any other configuration info required to connect.
concretely, the "proof-of-possession" can just sign over the message that is the encoding of the builder's BLS public key according to the SSZ spec defined in this repo: https://github.com/ethereum/consensus-specs

note: this attack is only relevant in the context of aggregation in the BLS scheme and there is no usage of aggregation in the builder-specs for now so resolution of this issue is not urgent but should also not be forgotten :)


Add concept of widgets for interactive graphs

This PR adds a concept of custom widgets that let users create their own interaction with the interactive graph. The implementation is pretty straightforward, it only relies on passing objects here and there to check for registered keys and call the related function/method.
Here is an example of how to create a simple "Exapand" widget that zoom or unzoom only on the x axis and how to register it on the plot:
from uniplot import Widget
import math
import os

class ExpandView(Widget):
    def keymap(self):
        return {
            "o": self.expand_left,
            "p": self.expand_right,
            "b": self.reset_view,
        }

    def expand_left(self, options):
        self.expand(options, factor=-1)

    def expand_right(self, options):
        self.expand(options, factor=1)

    def expand(self, options, factor: int) -> None:
        step: float = 0.1 * factor * (options.x_max - options.x_min)
        options.x_min = options.x_min + step
        options.x_max = options.x_max - step

    def reset_view(self, options):
        options.reset_view()

# Sin wave
x = [math.sin(i/20)+i/300 for i in range(600)]
columns, lines = os.get_terminal_size()

# Plotting with the widgets
plot(
    x,
    title="Sine wave",
    interactive=True,
    height=lines - 10,
    width=columns - 10,
    widgets=[ExpandView()]
)

As other example, here is how to consider WASD keypresses along with the HJKL already existing (as answer to #23)
class WASD(Widget):
    def keymap(self):
        return {
            "a": self.left,
            "s": self.down,
            "d": self.right,
            "w": self.up,
        }

    def left(self, options):
        options.shift_view_left()

    def down(self, options):
        options.shift_view_down()

    def right(self, options):
        options.shift_view_right()

    def up(self, options):
        options.shift_view_up()

Here is another example of a simple widget, and how it can be composed into another one to make a "line marker", where you can use the "+", "-" keys to go forward/backward and mark some zones. The "line marker" widget is built on another one "line widget", which can be used without the line marker one, showing that it's possible to create a set of widgets and reuse them (more or less)
class LineWidget(Widget):
    def __init__(self, start=0.0, selected=False):
        self.x = start
        self.selected = selected

    def draw(self, options, layer_factory):
        return layer_factory.render_vertical_gridline(x=self.x, options=options)

    def keymap(self):
        return {
            "+": self.inc,
            "-": self.dec,
        }

    def compute_step(self, options, factor=0.1):
        return 0.1 * factor * (options.x_max - options.x_min)

    def inc(self, options):
        self.x += self.compute_step(options)

    def dec(self, options):
        self.x -= self.compute_step(options)

class LineMarker(Widget):
    def __init__(self, start):
        self.marker = LineWidget(start)
        self.lines = {}

    def draw(self, options, layer_factory):
        return [self.marker.draw(options, layer_factory)] + [line.draw(options, layer_factory) for line in self.lines.values()]

    def keymap(self):
        return {
            "m": self.mark,
            "+": self.marker.inc,
            "-": self.marker.dec,
        }

    def mark(self, options):
        x = self.marker.x
        self.lines[x] = LineWidget(x)

The following video shows an small PoC I made loading a MP3, displaying the two channels waveform thanks to uniplot and navigating/marking zones. I didn't include the code of this PoC here as it relies on other modifications I made on uniplot for non-blocking keypress and another kind of widget that do not require a key to be pressed to activate/redraw/update.
https://github.com/olavolav/uniplot/assets/2317394/75624c8f-ce88-4d44-b340-f65132040c56

Thanks @aranega for the PR ❤ This is something that would benefit issue #23
I'll take a detailed look later this week

@olavolav Sure no problem, I think I had the same kind of problems in the past with Firefox (video that refuses to load). I re-encoded it, it should work this time (hopefully)
https://github.com/olavolav/uniplot/assets/2317394/5c67a606-6d37-4e71-8fe7-dd394733eca9

@aranega Apologies for the slow response. I love what you built here! 😄
Having thought about the best way to do this, I think I would like to have a slightly more generic system of lifecycle hooks. I'll have a go at it in the next weeks.

Hi @aranega finally getting back to this. Still love the PoC/video you had, and thinking about the simplest way to enable that. So your proposal adds a couple of features to the library:

Change the key mapping in interactive mode
Change the key binding, including to arbitrary functions
Add new plot layers
Colored grid lines

Let me comment on each one:

For (1.) and (2.) I have 3 solutions actually: You can either manually set this up by catching the keys yourself and calling update on the plot_gen object, or on a branch I have a keypress callback function as an option. The third option, that I might still get to, is to simply make the zoom & pan keys configurable – so this is #23 .
This is not presently possible, but you definitely can add grid lines as you had in your script when using the plot_gen path above.
This is also not presently possible. I'll make this a new feature request.

If you have feedback on the different options for (1.) and (2.) above, let me know.


Move compression handling from Command to Protocol

Since the protocol knows about snappy support, instead of the command, maybe the protocol should do the decompression.
Like:
unpacked_message = msg_proto.decompress(msg)  # this is just a no-op if msg_proto.snappy_support=False
cmd = command_type.decode(unpacked_message)

Originally posted by @carver in https://github.com/ethereum/trinity/pull/1189

Also maybe there's an opportunity to move the command ID handling outside of the command, too. So the protocol would be the one handling it. I seem to remember Piper saying there was a good reason we can't do this, but someday I'd like to take another crack at it.
I think this could be really helpful when managing many versions of the same subprotocol, where the same command might exist with a different command_protocol_id. It would be nice to do something like:
ETHProtocol65 = ETHProtocol.with_commands(
  Status,
  NewBlockHashes,
  ...
)

ETHProtocol66 = ETHProtocol.with_commands(
  Status,
  SomeCrazyNewCommand,
  NewBlockHashes,
  ...
)

Where the only thing different about NewBlockHashes between the versions is the command id, so we should be able to re-use the command definition.

I think all of this can be done, but the snappy stuff would be the biggest win.  :+1:
Thanks for making an issue for this.  I decided not to do it in #1189 because it was already so huge but forgot to make the issue as to not lose the task.


Unhandled Rejection (TypeError): Failed to fetch

When enter to localhost:3000 in windows 10 with chrome, appears
return fetch(apiRoot + api/user/?uuid=${uuid}, {
I show a screenshot, any solution? Thanks

I answer to myself the problem was backend, you must be opened backend and frontend, now works perfect

Thanks for updating this - I'm sure it will help someone else in the future who's seeing the same thing :)


commitment: add asset type field to asset commitment structure

All assets included within a given asset commitment must have the same asset type. This change helps to reinforce that idea.

quickly funnel assets into their proper commitment space


Check that the URL is valid before attempting to download

In that case, return a nice error to the callback saying that the node version you were trying to download was not found.

Closing since this repo is not maintained anymore.


Reorganize the documentation journey

We had duplicated info in a number of places, it was hard to know the difference between a guide and an integration.
I've consolidated into this flow:

Understand at a high level what runme is and why it exists
Quickly find the thing you want to install
Use the thing you installed
Configure and expand
All the peripheral info people might look for.

Guides become tutorials, which I have removed a bunch of unnecessary introduction, these need more work to make it clear why using them with runme adds value, like with Dagger.

@admc are you on the latest version? Why do we have conflicts?

@sourishkrout  Not sure what the deal was there, but I merged main, prob had to do with switching computers and pulling the branch without getting main first. But I think all the things are there and all links are fixed now.

@sourishkrout okay, woke up with a bunch more changes in mind, I think I'm at a stopping point now if you wanna take a look 🙏

What I'm noticing, unclear if they are showstoppers or not:

[ ] The Getting Started content under /getting-started/vscode is really old - perhaps edit it down to what's relevant and put the Quickstart guide at the top?
[ ] Badges are gone, i.e. no more Runme frontmatter in the docs. This is okay, if the document itself is not runnable.
[ ] Cutover will break links. We should make sure we have redirects in place.

Just checked in an updated vercel.json based on sitemap diff.


Request feature: Scatter3D

I've noticed that the code has already been implemented and merged to dev branch https://github.com/igiagkiozis/plotly/pull/66. But in version 0.7.0, there is still no Scatter3D available. Please merge dev to master, thanks!

Indeed, the recent work is currently an sitting on the Dev branch at the moment. I've got some free time next week thati was planning on allocating to getting a new release out.

From: IchHabeKeineNamen @.>
Sent: Friday, July 29, 2022 12:38:28 PM
To: igiagkiozis/plotly @.>
Cc: Subscribed @.***>
Subject: [igiagkiozis/plotly] Request feature: Scatter3D (Issue #92)
I've noticed that the code has already been implemented and merged to dev branch #66https://github.com/igiagkiozis/plotly/pull/66. But in version 0.7.0, there is still no Scatter3D available. Please merge dev to master, thanks!
—
Reply to this email directly, view it on GitHubhttps://github.com/igiagkiozis/plotly/issues/92, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AHSVKWAVYX2BNHJPJC6PZHTVWO7DJANCNFSM55AO7Z6Q.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

I've just initiated a discussion with the crate owner to see about cutting a 0.8.0 release. In my opinion, now is a good time to push out a new release before the next phase of work on the library.


Exclude columns from results

The idea to add special parameter to SELECT clause to remove columns from results, like"
    var data = [{a:1,b:10},{a:2,b:20}];
    var res = alasql('SELECT * REMOVE b FROM ?',[data]);

returns [{a:1},{a:2}]

:+1:

You can remove unused columns with REMOVE COLUMN clause
The syntax is:
SELECT selected-columns REMOVE [COLUMNS] columns-to-be-removed

like in this example:
alasql('SELECT * REMOVE COLUMNS b,c FROM ?',[data]);

You can also remove by mask:
    SELECT * REMOVE COLUMNS LIKE "b%" FROM one


Status View Component

Issue Description
As an application developer, I need "Status View" card which can fulfill the needs of both the Error View Component, and the No Data Component
http://engineering.cerner.com/terra-clinical/#/site/no-data-view
http://engineering.cerner.com/terra-clinical/#/site/error-view
Questions
I also want to figure out if terra-core just containers a Custom Type view which takes any text and icon to display, allowing terra-consumer and terra-clinical to wrap it providing proper types. Or to bake all the common types into terra-core, theme the icons, and also provide a custom type for consumer to use.
Issue Type

[X] New Feature
[ ] Enhancement
[ ] Bug
[ ] Other

Expected Behavior
This should work similar to terra-clinical No Data View component  with theming option to satisfy terra-consumer and Cerner OCS and Error view e.g Not Authorized, No results found, Error Loading etc.
Tech Design:
Considerations:

Meets specification of No Data View component  in terra-clinical.
Meet specification of terra-consumer e.g having divider, heading / subtext alignment options
Covers most of the Cerner one style No Data view component ( Where can I get the callout for OCS component ?) @alex-bezek @mhemesath
Meets specification of Error view - terra-clinical  and terra-consumer

Responsiveness:

TBD

Bidirectionality

TBD

Accessibility

TBD

Approach 1  - Custom
React Props:

Prop
Type
Default
Description

icon
element
undefined
The icon component displayed on the top. It can be any icon consumer wants

heading
string
required
The title of the no data view.

subtextAlignment
string
center
Set the alignment of subtext (description). The value should be one of 'Left' and 'Center'

subtext
string
undefined
The subtext description of the no data view.

subtextContent
element
undefined
The content to be rendered last. The consumer has total control on what to display inside this.

isIconHidden
bool
false
Whether or not the Icon should be displayed.

Question : If icon is not passed then Icon will not be rendered then I believe we don't need isIconHidden. Let me know if I am missing any use case for this prop.
Approach 2  - Common
React Props:

Prop
Type
Default
Description

type
string
required
The objective of the status view component is defined by consumer by selecting type. As per type we internally decide which icon and heading will be rendered. See more details on type below

subtextAlignment
string
center
Set the alignment of subtext (description). The value should be one of 'Left' and 'Center'

subtext
string
undefined
The subtext description of the no data view.

subtextContent
element
undefined
The content to be rendered last. The consumer has total control on what to display inside this.

isIconHidden
bool
false
Whether or not the Icon should be displayed.

Prop - type:

value
Icon
Heading

CLINICAL
TBD
TBD

NO_DATA
folder icon
No Data

NO_RESULT
Magnifying glass
No Matching Results

NOT_AUTHORISED
lock
Not Authorized

SENSITIVE_DATA
lock
Sensitive Data

ERROR
triangle
Error

NO_INTERNET
triangle
No Internet Connection

ERROR_LOADING
triangle
Error Loading

I feel this list will go long with time. Being this component at terra-core level it would be nice if we keep the icon and heading configurable as mentioned in Approach 1.
CSS Classes:

Selector
Description

.status-view
The root class for the CSS component. It will be applied under all theme. It will have common css across all the theme like left and right padding for card component.

.icon
The class for the icon to set the size of icon and alignment

.heading
The class for the heading to set text size and color and if required line height

.subtext
The class for the subtext to set text size and color and if required line height.

.subtext-content
The class for the subtext to set alignment.

.divider
The class will decide whether horizontal rule will be rendered or not

Note: Themed variable will be used to set the values wherever required for different themes (terra-clinical, terra-consumer and OCS). Be default terra-clinical theme will be applied.
Theme CSS
StatusView.scss
 @include cerner-consumer-theme {
   --terra-statusview-icon-size: TBD ;
     --terra-statusview-heading-text-size: xxRes-Large (30 mobile /36 px desktop);
     --terra-statusview-heading-text-color: text-Light;
     --terra-statusview-subtext-text-size: Body ;
     --terra-statusview-subtext-text-color: Text-Dark;
     --terra-statusview-subtext-aligment: Left/center ;
     --terra-statusview-divider: block;
     --terra-statusview-heading-line-height: TBD;
     --terra-statusview-subtextcontent-aligment: center;
  }

 @include cerner-default-theme {
     --terra-statusview-icon-size: 170x170 px;
     --terra-statusview-heading-text-size: 1.714 rem;
     --terra-statusview-heading-text-color: #c8cacb;
     --terra-statusview-subtext-text-size: 1 rem;
     --terra-statusview-subtext-text-color: #6f7477;
     --terra-statusview-subtext-aligment: center;
     --terra-statusview-divider: none;
     --terra-statusview-heading-line-height: 1.28rem;
     --terra-statusview-subtextcontent-aligment: N/A;
   }

  @include cerner-one-style-theme {
     --terra-statusview-icon-size: 100x100px ;
     --terra-statusview-heading-text-size: 24 px;
     --terra-statusview-heading-text-color: --black-60 ;
     --terra-statusview-subtext-text-size: 14 px ;
     --terra-statusview-subtext-text-color: --black-40;
     --terra-statusview-subtext-aligment: center ;
     --terra-statusview-divider: none;
     --terra-statusview-heading-line-height: 28px;
     --terra-statusview-subtextcontent-aligment: center;
   }

Note : I am assuming the padding between the elements e.g. heading, subtext, divider and content will be common across the theme.

@amf9t2
approach 1
I'm assuming we would then also have to create terra-consumer and terra-clinical wrappers to provide the correct icons in an easily usable fashion.
I think we decided in the war room to not support subtextAlignment . @neilpfeiffer can confirm if thats where we landed.

Question : If icon is not passed then Icon will not be rendered then I believe we don't need isGlyphHidden. Let me know if I am missing any use case for this prop.

I agree. If we do keep it (or in the other option) we also want to rename Glyph to Icon

Responsiveness
Did we determine from Sarah that the medium/large components are different, or up to the user to not provide subtextContent. Not that its responsive to show/not-show the subtextContent?
Also I think our mockups have responsive Heading font size and icon size

Approach 2
If we provide a type, i think we would still need to expose a custom type that consumer could use to make ones with different headers. That or it can also take a header prop to override any type.

subtextContent of type node.
If we take the second approach, I feel it would still be valuable to include the heading prop. I think default headings are appropriate as indicated above, but I'm not sure the heading should be strictly locked to the icon.

i like approach 1. It provides the flexibility for applications requiring custom icon and heading.

if a specific status view is widely used in an application, the view can be defined in one place and reused across the app.

StatusView seems a little generic and there is already a terra-status component. I'm wondering if rather than View a better term is Template being this this is really used as such. E.g. ResourceStatusTemplate which can then be used to create ErrorView and NoDataView

Based on Friday's war room discussion, for approach 2 the terra-core component should only contain the types that are common between clinical and consumer with the additional custom type (as well as an icon and heading prop that can be used for the custom type) so that terra-consumer can expand on it (I believe clinical doesn't need any types beyond the common ones)

Looking back at Neil's comments on the old issues the common types are: (1)No Data, (2)No Matching Results, (3)Not Authorized, (4)Error

@alex-bezek  Yes as per discussion with Sarah we decided to have independent component for medium category. We could achieve that without passing content and icon as per approach 1 but issue is with divider. As divider by default is added for consumer theme whereas in the medium case we don't want divider so it is better we have independent component inside same package.
For approach second : If we take heading as prop, we will still need type to decide icon from mapping and given heading will take the priority. I guess you are saying the same.

@mhemesath  how about ResponseStatusViewTemplate ?
All the variation(NO_DATA, NO_RESULT, SENSITIVE_DATA, ERROR etc) is connected to a response of a user action.

Thanks for the feedback.
The above discussion direct us to combine the approach 1 and approach 2 where consumer either has to set the type or provide icon and heading. One of these will be mandatory.
Combine Approach   - Common - Custom
React Props:

Prop
Type
Default
Description

type
string
required
The objective of the status view component is defined by consumer by selecting type. As per type we internally decide which icon and heading will be rendered. See more details on type below

icon
element
undefined
The icon component displayed on the top. It can be any icon consumer wants. If type is not provided then this becomes required with heading.

heading
string
undefined
The title of the status view. If type is not provided then this becomes required with icon. If type is provided then it will take precedence over the heading value from Type-Icon mapping.

subtextAlignment
string
center
Set the alignment of subtext (description). The value should be one of 'Left' and 'Center'

subtext
string
undefined
The subtext description of the no data view.

subtextContent
element
undefined
The content to be rendered last. The consumer has total control on what to display inside this.

isIconHidden
bool
false
Whether or not the Icon should be displayed.

Let me know if this fulfills all our use-case.
@jmsv6d @alex-bezek

I you changed subtextContent to be of type node I don't think the subtext prop is needed. And I'm not sure if we want the heading string to be over-ridden for the built-in types, I think that the heading prop should only be used when the type is custom. But I would want @neilpfeiffer to confirm both of these.

This is one of the use-case for terra-consumer where message will be subtext and buttons will be subtextContent. So we will need subtext as a separate prop. Same applies for OCS No data component.
The type prop will be taking one of these type CLINICAL, NO_RESULT, ERROR etc (se above Prop - type table). My understanding is we want consumer to pass heading text  If they doesn't like the predefine heading for a particular type.

It's fine if you guys need both, I was just thinking that in our legacy stuff the two were mutually exclusive. As far as the heading goes, the heading messages are specific to the types, if a consumer needs something that displays a different message, then perhaps there's a need for a new type to be defined.

Discussed with @mhemesath about the component name from StatusView to ResponseStatusView.
Any one has any thoughts or concern on ResponseStatusView name ?

@jmsv6d  how about adding new type called CUSTOM and honoring icon and heading given via props. So if consumer doesn't like our mapping then they will have to set type CUSTOM and provide icon and heading. With this type prop become mandatory and if selected type is 'CUSTOM' then given  icon and heading will be rendered.

Adding label "Needs Design Input" until UX delivers finalized API wireframe and theme variations. I will remove label when needed documentation is added.

I just wanted to confirm that the only types that should be in the terra-core component are NO_DATA, NO_RESULT, NOT_AUTHORIZED, ERROR, and CUSTOM. There will need to be another component made in terra-consumer that wraps the terra-

I just wanted to confirm that the only types that should be in the terra-core component are NO_DATA, NO_RESULT, NOT_AUTHORIZED, ERROR, and CUSTOM. There will need to be another component made in terra-consumer that wraps the terra-core component and uses the custom type to create the types that are specific to consumer. Just like what we've done with alert for clinical specific alert types.

Correct. The current goal is for terra-core to implement the component that matches those types. From there @amf9t2 is looking into this issue on theming the icons for each type based on consumer vs clinical https://github.com/cerner/terra-core/issues/587

Awesome. I think this looks good then +1

+1 combined approach looks good to me. I like we are following a similar pattern to alert by using CUSTOM to handle the differences between core, clinical and consumer needs.
In response to changing the name of the component, IMO StatusView is fine. To me, RepsonsiveStatusView makes me think this status would be updating on the fly or adjusting its layout for page resizing due to concept associated with responsive element.

+1 to combined approach. I'm cool with naming it ResponseStatusView. In regards to StatusView.scss and the mixins, the default theme values should be written as fallback syntax. e.g. border-radius: var(--terra-badge-border-radius, 0.25em);, no need for a cerner-default-theme mixin. The cerner-one-style-theme will be named cerner-clinical-theme.

@neilpfeiffer any update on UI design input ?

@neilpfeiffer any update on UI design input ?

Duplicate to https://github.com/cerner/terra-core/issues/1028


Deleting entries (for good)

@ascott @dethe @jesicarson
I asked about this a while ago, but we've had a situation come up with one of our partners that necessitates that we create a way to either

delete entries for good or
re-route non-admin users that click on a link or enter the exact URL of a 'hidden' entry to a page that says 'this entry has been deleted'

'Hidden' cases are still accessible if they were entered in any of the database-search fields (eg. components, specific methods, organizing entity) or if they were hyperlinked to in the narrative body. They should only be accessible by admins.

Good point and this should get implemented. Probably not super high priority though, right Scott?
At least we can hide all the tests and messy stuff, and chances are if it was entered in error it may not be linked to too many other entries (as a method/component etc) right?
Hiding is the MVP, properly hiding is nice to have. But correct me if it’s more urgent. Thanks!

@jesicarson I'm not sure how urgent it is, I'm waiting to hear back from Julien and Mark regarding the 'situation'. I'll forward you the email. I think the person whose work was plagiarized is going to be none-too-pleased that she can still access it by clicking a link

I can return an error if a non-admin tries to access a hidden article easily enough. Will need some front-end work to handle the error and show a nice message. The message itself should be part of our UI localizable text.

@dethe great! thank you!

do we need a page that says 'this entry has been deleted' or can we just return a general 404 page?
@jesicarson do you have a design for a 404 page?

@ascott I don't have a mockup, but would be nicer to have a page, or what about our standard modal/popup/warning format? Could that work?  And yes use the text: "This entry has been removed"

@scottofletcher combatting plagiarism does feel urgent.

@ascott are you ok without a mock if we just use the standard modal?

Sounds good all!

@jesicarson and i just discussed via slack, and we decided on a general 404 page to handle requests to hidden pages as well as invalid url's. @dethe i can implement the 404 response from the server if the article is hidden as well.

ok, but I think this is going to leave users wondering why they can't access an entry in which case they might email us and we'll have to explain. which reminds me: we never finished creating a work-around to the internal messaging system. unlike on .net, if users have a question about an entry, they can't contact the original submitter (unless they provide contact info on their profile page). we started on this here: http://bit.ly/2EtcbR4


NIFI-5740 Ensuring permissions are restored after test completes in T…

…estFetchFile
Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[ ] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[ ] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[ ] Has your PR been rebased against the latest commit within the target branch (typically master)?

[ ] Is your initial contribution a single, squashed commit?

For code changes:

[ ] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[ ] Have you written or updated unit tests to verify your changes?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.

+1 LGTM, verified I couldn't delete the directories manually without this change, and I can afterwards. Thanks for the fix! Merging to master


#270 title extension

added QAI-Export of the new properties, therefore moved to another PR to include the latest commits.


Allow reflectable 0.2.1

or is there a reason it is limited to 0.2.0 in the rc1 branch?

It just hasn't been tested yet, I don't foresee any issues though.


property override fix #2474

Description
Minimal solution to fix the issue of overriding runner properties by JVM args.

Relevant Issues : #2474
Relevant PRs : #2479
Type of change :

[ ] New feature
[x] Bug fix for existing feature
[ ] Code quality improvement
[ ] Addition or Improvement of tests
[ ] Addition or Improvement of documentation

@ptrthomas this PR removes setting the System property from the systemProperty method on the runner to fix this issue. Otherwise the other fix I considered of "syncing" the karate properties with the System properties ended up breaking other tests. Since it lead to a case where, in a multi-runner execution, one runner setting a property would override all other runners that needed to set the same property - e.g., if server.port is set on runner 1, then subsequent runners could not set the same property anymore and it would be forced to use whatever the first runner set it to unless it was explicitly cleared first.. I don't think that would be desirable, especially for mocks where the runners can use this method to set server port details.
If setting the System property is still needed in the systemProperty method for something else, then this issue may become more challenging to deal with. My first thought would be to cache the System properties initially, so that we could have a reference of what properties came from JVM arguments and use that to override each time the runner needs to reference it, while still maintaining an actual sync across runner and system properties elsewhere. Let me know if you have any thoughts on which direction we should take here.


Treat sphinx warnings as errors

This pull request re-enabled features which treats sphinx warnings as errors which cause build to fail.
We had this feature enabled in the past, but at some point we disabled it (probably when we enabled sphinx auto doc) so we accumulated a bunch of documentation and docstring related debts since warnings didn't cause build to fail and people simply ignored those issues.
Future TODO (up for grabs):

[ ] Fix all the docstring related issues
[ ] Fix all the cross reference related issue (more than one target found for cross-reference messages)

Alright, merged.
Going forward, all the sphinx warnings are not treated as error and will fail the build - this means they need to be fixed before merging changes into trunk.


Run with Chef <12

source_url and issues_url are Chef 12 attributes that do not have backwards compatibility. Will submit a PR for this.
Please see e.g. here for more info, to save me repeating everything: https://github.com/logentries/le_chef/pull/30

This is resolved with if respond_to? in the metadata

:+1:


Update tinyxml2 to 8.1.0

Whoops... some errors when building for Android, and... this is for Ubuntu 18.04:
CMake 3.15 or higher is required.  You are running version 3.10.2

I'm confused that we don't have fseek on some platforms.

I'm confused that we don't have fseek on some platforms.

I think it's more that on android-x86_64, it goes into the defined(__x86_64__) condition and tries to use fseeko64, which apparently does not exist.

I tried with a fork of tinyxml2 that sets cmake 3.10.2 (Ubuntu 18.04) and uses this PR for Android.
I already see that tinyxml2 seems to require a version older than 3.10.2, let's see what Android says about my other patch.


create todo list copy for review

This pull request contains a copy of my Todo list for review for javascript, Html and Css best practices

Type checks

Good use of type checks 👍🏾

Naming

Standard naming convention used. camelCase 💯

Simplicity

Clear and simple Algorithms 🥇

DRY

There is no code repetition 👍🏾

KISS

Clear and simple Algorithms 🥇

YAGNI
-No unnecessary use of technologies 👍🏾
All best practices followed, good effort. Please mantain this.


Fix package.json

I changed dependencies to devDependencies, as those dependencies are for development, so devDependencies is what you want. I also did a couple of other simplifications/corrections.

Cool, thank you!


Fix: mailer copyright current year

Fixes #3008
Describe the changes you have made in this PR -
Update mailer copyright text to be with current year

Coverage remained the same at 81.785% when pulling 50e3c7802bbb7552374703a8173d4a38e0c5e6be on ahmedibrahim404:mailer-copyright#3008 into fa7cc17055a03a5eb979922803205d5b5e525dfb on CircuitVerse:master.

@ahmedibrahim404 I am sorry for the misunderstanding but the PR for this is already being raised and under review #3010

@vedant-jain03
I think the other PR is not active, and requires some changes
Could we consider this instead?

@tachyons should we consider this, I closed this PR due to duplicate but the other PR is closed due to inactivity.
and there is a new PR for the same issue https://github.com/CircuitVerse/CircuitVerse/pull/3216 .


Ability to pull stemcell images from a private image repository

Is your feature request related to a problem? Please describe.
We are hosting our stemcell images from a private image repository and at the moment there is no configuration in kubecf we can set that allows the helm installation process to pull from that private image repository.
Describe the solution you'd like
We would like to have a parameterised configuration we can set in kubecf that allows the helm installation process to pull images from a private image repository.
Describe alternatives you've considered
Keep hosting the stemcell images in a public image repository.
Additional context
N/A

@jbuns Sorry, I forgot that we had this issue already. In the interest of keeping the issues list manageable I'm closing this issue because #1421 has more details about the proposed implementation. Feel free to re-open if you disagree!

Fine by me!


Fix empty $route (error routes) in YAML DB

This fixes the problem where error routes end up in the YAML DB as '': 1. Uses the 404 route setting for the error plugin, or /error as fallback.

Sorry for the delay! GitHub needs to surface these better!


new build script proposal

main is now transpiled unminified code
provide minified code to unpkg
add source maps to prod minified code

Hi @pahund , not exactly what you have proposed here https://github.com/technology-ebay-de/react-prebid/issues/20, but this is what I was thinking. Let me know what you think?

Pull Request Test Coverage Report for Build 105

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 94.483%

Totals

Change from base Build 104:
0.0%

Covered Lines:
207

Relevant Lines:
213

💛  - Coveralls

This is perhaps better than my proposal, as it doesn't introduce breaking changes for folks who are using the index-esnext.js module. I'll take a closer look later today (CET).
@sundy001 What do you think?


Gradle 6.9, Android Gradle Plugin 4.2.1

Summary
Bump Gradle to 6.9 which supports Apple Silicon, also Android Gradle Plugin 4.2.1 which defaults to Java 1.8 so no additional config required.
Changelog
[Android] [Changed] - Bump Gradle to 6.9, Android Gradle Plugin to 4.2.1
Test Plan
rn-tester builds and runs as expected

@fkgozali @mdvacca please review and merge

it's almost green 😅

@facebook-github-bot import

@mdvaccaplease help with a failing test. Anything I can help?

The test seems to be unrelated, I will update it separately
There are some internal things we need to update before merging, I'll try to look into them soon-ish :)


could use head as print method

Since data is the star of the show, printing a ddf should not return some odd abstraction-killing object information, but the actual data. Since printing all the data is in general a bad idea , printing some is a better one. A sample or the first few lines are both acceptable options (the former is more statistically motivated, the latter faster to implement and the R tradition

@binhmop can we do a better .toString() per @piccolbo's suggestion?


StructuredList: Keyboard Accessibility

Description
When I made a StructuredListItem interactive by specifying an href or onClick property, I expected to be able to navigate to and trigger those items using the keyboard but it seems like they only become accessible via keyboard if they're first interacted with using the mouse.
Link to Reproduction
https://codesandbox.io/p/sandbox/elegant-microservice-8sjlcy
Steps to reproduce

Go to https://codesandbox.io/p/sandbox/elegant-microservice-8sjlcy
Click to focus on the preview pane
Use tab, shift+tab to navigate the document with the keyboard, notice that list items cannot receive focus
Click on one of the list items (triggering its onclick)
Try using keyboard navigation again, see that the items can now receive focus.

Saas UI Version
2.7.4
Chakra UI Version
2.8.2
Browser
Firefox 124.0.2 (64-bit), Chrome Version 123.0.6312.124 (Official Build) (x86_64), Safari Version 15.4 (176<IP_ADDRESS>.6)
Operating System

[X] macOS
[ ] Windows
[ ] Linux

Additional Information

Thanks for the reproduction. It seems like the tab indexes on the items are set incorrectly. Will look for a fix tomorrow.

@bhainesva Fixed keyboard navigation in the PR above. This will be released shortly.


Form on_change handle not registering for dynamic number of forms

Describe the bug
I want to create a large number of forms that contain the same fields, which I want to add to some collection. I would expect to be able to register on_change handlers for each form, but this seems like it cannot be done.
Environment
{
"marimo": "0.3.10",
"OS": "Windows",
"OS Version": "11",
"Processor": "Intel64 Family 6 Model 126 Stepping 5, GenuineIntel",
"Python Version": "3.12.2",
"Binaries": {
"Browser": "116.0.5845.188",
"Node": "v14.16.0"
},
"Requirements": {
"click": "8.1.7",
"importlib-resources": "missing",
"jedi": "0.19.1",
"markdown": "3.6",
"pymdown-extensions": "10.7.1",
"pygments": "2.17.2",
"tomlkit": "0.12.4",
"uvicorn": "0.29.0",
"starlette": "0.37.2",
"websocket": "missing",
"typing-extensions": "4.9.0",
"black": "24.3.0"
}
}
Code to reproduce

import marimo

__generated_with = "0.3.10"
app = marimo.App()

@app.cell
def __():
    import marimo as mo
    return mo,

@app.cell
def __(mo):
    get_state1, set_state1 = mo.state(None)
    get_state2, set_state2 = mo.state(None)
    get_state3, set_state3 = mo.state(None)
    return (
        get_state1,
        get_state2,
        get_state3,
        set_state1,
        set_state2,
        set_state3,
    )

@app.cell
def __(mo):
    def new_form(form_name: str, on_change):
        form_ = mo.md("""
        Form

        {field}

        """).batch(
            field=mo.ui.text("input")
        ).form(
            on_change=on_change
        )

        return form_
    return new_form,

@app.cell
def __(new_form, set_state1):
    set_state1("state 1 not set")
    def on_change_callback_works(*args):
        set_state1("state 1 set")

    new_form_ = new_form("test", on_change=on_change_callback_works)
    new_form_
    return new_form_, on_change_callback_works

@app.cell
def __(get_state1):
    print(get_state1())
    return

@app.cell
def __(new_form, set_state2):
    set_state2("state 2 not set")
    def on_change_callback_doesnt_work(*args):
        set_state2("state2 set")

    new_form("test2", on_change=on_change_callback_doesnt_work)
    return on_change_callback_doesnt_work,

@app.cell
def __(get_state2):
    print(get_state2())
    return

@app.cell
def __(mo, new_form, set_state, set_state3):
    # Following https://docs.marimo.io/recipes.html#create-a-table-column-of-buttons-with-on-change-handlers

    set_state3("state3 not set")
    def on_change_callback_doesnt_work_too(*args):
        set_state3("state3 set")

    arr = mo.ui.array([new_form("test3", on_change=on_change_callback_doesnt_work_too)])
    return arr, on_change_callback_doesnt_work_too

@app.cell
def __(arr):
    arr[0]
    return

@app.cell
def __(get_state3):
    print(get_state3())
    return

if __name__ == "__main__":
    app.run()

@mrdobalina2k , just wanted to say thanks for opening this bug report. We haven't gotten a chance to fix it yet, but hopefully will get to it this week.

You're very welcome. It's nice to be able to contribute even if it is only with bug reports. Keep up the good work.


Harmonising terminology used on EPP

I think we should change the class article-furniture to article-details?
What does everyone think?


Chunk Texture Management (global texture arrays?)

@beeperdeeper089, please fill out what needs to be done here then add Todo tag

We need to figure out a way to effeciently manage textures within the Chunk system. We currently load in the textures again and again for EACH chunk, which is inefficient and we should keep like a global texture array.

This is now obselete.


Wrong parsing of an incoming request's body

Hello!
I'm writing a program that uses pistache as a rest server and I noticed that the incoming Pistache::Rest::Request object's body() function returns a string which is missing the first CRLF. If I send a request which contains a form-data parameter (multipart/form-data content type) then the first boundary starts with "--" instead of "\r\n--". By rfc7578 the boundary consists of "\r\n--" plus the text specified in the header parameter "boundary". Im not getting the first CRLF, it starts with "--" instead.
The sending side correctly sends the two sets of CRLF., so my assumption is that the parser in pistache cuts request into header and body at the wrong place. The first CRLF is the separator from between the header and the body, but the second one is part of the body.

We're all a little tied up with the V0.1 release presently, but this looks like it should be fairly straight forward for anyone who wants to submit a PR.


Import errors with BlockBlobService

Which service(blob, file, queue) does this issue concern?
Only the BlockBlobService importing issue. Rest are being correctly imported
Which version of the SDK was used? Please provide the output of pip freeze.
azure-core==1.6.0
azure-storage-blob==12.3.1
What problem was encountered?
On trying to run:
from azure.storage.blob import BlockBlobService
Results in:
ImportError: cannot import name 'BlockBlobService' from 'azure.storage.blob' (C:\Users..\azure\storage\blob_init_.py)
Have you found a mitigation/solution?
Can't solve the import error

Hi @kmair
you are looking at a wrong repo. https://github.com/Azure/azure-sdk-for-python/tree/master/sdk/storage/azure-storage-blob is the correct repo

please refer to the wiki https://github.com/Azure/azure-storage-python/wiki#frequently-encountered-problem-no-module-named-azurestorageblob
I'll close this issue for now, if you still have the problem please feel free to reopen it!


Add support for interrupts to DMA API

This pull request is based on #14. I recommend reviewing/merging that pull request first, before this one.
I have additional commits to test this against the Tlera Corp Gnat here:
https://github.com/braun-embedded/stm32l0xx-hal/compare/dma-interrupts...braun-embedded:dma-interrupts-gnat
cc @lthiery

Rebased, ready for review.


DevTools pinning customization

Allow pinning individual Dev Tools like Network, Performance etc., as it is leading to accidental closure sometimes while switching.

Thanks @aadhilrf! If I understand correctly, while switching tabs in DevTools, you sometimes accidentally close them. And to avoid this, you'd like a way to pin certain panels. Is that correct?
We've had that feedback in the past too. One thing we did was give the tabs a bit more padding so it would be harder to accidentally close, but seems like it's still happening for you. Let me ping @hxlnt who will be interested in hearing about this.
Additionally, 2 related things come to mind:

You can re-open closed tabs easily by clicking the + button in the toolbar, or by using the command menu. You might already know this, but just making sure that you do, as we've heard feedback where people accidentally closed tools and could not find them again.
We have an experiment at the moment that moves the tool tabs into an activity bar, which you might be interested in trying out. You might prefer this over the default toolbar. Give it a try if you want and let us know what you think. Learn more here.

Thanks for the feedback, @aadhilrf.
Following up on @captainbrosset's suggestion above, I'm curious if our experimental UI would be of interest to you. The close buttons are not present, and tools are pinned through the + menu. Additionally, the toolbar can be configured in a horizontal or vertical orientation, pictured below.
We've updated the feature and docs recently; take a look and let us know if this would address your concern!


Fixes in en locale

Updated the description for "PARAMS_CAMERA_TOOLTIP_ROT_CLOCKWISE"; corrected spelling errors.

Seems like good fixes, any reason why the pull request was closed?


chore(deps): bump TypeScript to v4.9.5

Summary
Upgrading to TypeScript v5 is currently blocked by @microsoft/api-extractor, because it throws if the ignoreDeprecations compiler option is around. This makes it impossible to use the option with TypeScript 4.9.5 too.
ignoreDeprecations will also be a problem when testing types on older versions of TypeScript. This PR is changing tsconfig.json used for type testing. I think it makes sense to extend from node14 base instead on tsconfig.json which is used for build.
The compiler config for type testing was mostly unsetting some build options like composite. That was nice shortcut, but I think type testing shouldn’t be somehow too much related with build config. It has to work with older versions, so I am trying to reshape it here.
Test plan
Green CI

Hm.. Can’t understand why TS v5.0.2 is used for typechecks:

api-extractor sees TS v4.9.4 during build:

Actually tsc should complain about the importsNotUsedAsValues option as well if repo would be build using TS v5.0.2, but it does not.

Seems like I got through that glitch by moving typechecks to separate job. I was thinking to suggest this at some point, simply to make the TypeScript Compatibility job lighter. The build:ts step is needed, but perhaps that’s fine?
Note that in the new job I skipped ts:bundle, because it removes tsconfig.tsbuildinfo files. The typechecks run through tsc --build, which would rebuild packages if tsconfig.tsbuildinfo files are not found. Another approach could be to make the bundle step take some flag, e.g. ts:bundle --skip-cleanup.

https://github.com/facebook/jest/pull/14055 fixes this without having to split jobs and should be landed regardless since it'll fix bugs when you run old compat tests locally.


Light3D's cull does not affect shadows

Godot version
v4.0.beta10.mono.official [d0398f62f]
System information
Windows 11
Issue description
The Cull Mask option in the Light node does not affect/filter shadows.
In the following screenshot, you can see that the Box (on visual layer 2) does not receives light (cull mask excluding layer 2), but still cast shadows.

Steps to reproduce

Add a Plane to the scene with VisualInstance3D.layers sets to 2
Add a Box to the scene with  VisualInstance3D.layers sets to 1
Add any type of 3D Light to the scene and uncheck layer 2 from Light.light_cull_mask
Remark the lighting does not affect the Box, but still project its shadow

Minimal reproduction project
SampleProject.zip

Possible duplicate of #58322

Duplicate of https://github.com/godotengine/godot/issues/58322.

I have downloaded both beta12 and the project I originally uploaded and the lights are all still casting shadows even tho none of them are lighting up the cube.
I checked with beta11 (in which 58322 got fixed), same result.
Am I missing something? #58322 seems to be fixed.

This behaviour is the same in Godot 3.x and is even highlighted in the documentation:

Cull Mask: Objects that are in the selected layers below will be affected by this light. Note that objects disabled via this cull mask will still cast shadows. If you don't want disabled objects to cast shadows, adjust the cast_shadow property on the GeometryInstance to the desired value.

I'm not saying that the way it works now is the way that it should work, but its clearly currently working as intended, so this is not a bug per se.

This issue is different from bug #58322 which was about shadows not removed on an per-camera cull basis. Or more precisely: VisualInstances (VI) which were culled from a scene due to non-matching camera cull_mask and VI layers still left shadows in the scene.
This issue, on the other hand, is about shadows of non-interacting lights and VI (due to non-matching light_cull_mask and VI layer). So no additional camera involved here.
I know the docs announce this behavior but I still would call it a bug as it doesn't make sense, neither on a physical nor on an "intuitive" basis. Just as in #58322.
To be honest, probably the docs were rather written around this behavior than the original implementation conformed to the docs.

This behaviour is the same in Godot 3.x and is even highlighted in the documentation:

Cull Mask: Objects that are in the selected layers below will be affected by this light. Note that objects disabled via this cull mask will still cast shadows. If you don't want disabled objects to cast shadows, adjust the cast_shadow property on the GeometryInstance to the desired value.

The documented solution doesn't work if you want an object to cast shadows for some light sources but not others.

The documented solution doesn't work if you want an object to cast shadows for some light sources but not others.

This is being tracked in https://github.com/godotengine/godot-proposals/issues/3606, which has a pull request implementing it: https://github.com/godotengine/godot/pull/85338


How to json-patch with real entities?

I've been following the example of the json-patch (https://github.com/eugenp/tutorials/tree/master/spring-web-modules/spring-rest-http), but then using Quarkus and Pananche with a real database and entity.
I did the same as in the example code, but then with my entity:
Lookup the entity from the repository based on id, then apply the JsonPatch and persist it to the repository again. The persisting doesn't go as expected (because of a detached entity since the apply just returns a new entity??). How can I make this work with real entities without having to do manual mapping to the retrieved entity from the db?
@Transactional
public MyModel updateEntity(final Long id, final JsonPatch jsonPatch) {
    return repository.findByIdOptional(id)
            .map(e -> updateEntity(e, jsonPatch))
            .orElseThrow(() -> new ResourceNotFoundException(id));
}

private MyModel updateEntity(final MyEntity entity, final JsonPatch patch) {
    final var entityPatched = applyPatchToMyEntity(entity, patch);
    repository.persist(entityPatched);  // this is where it goes wrong, since entityPatched is not the retrieved entity but becomes a new detached entity?
    return modelMapper.map(entityPatched, MyModel.class);
}

private MyEntity applyPatchToMyEntity(MyEntity targetMyEntity, JsonPatch patch) {
    try {
        final var patched = patch.apply(objectMapper.convertValue(targetMyEntity, JsonNode.class));
        return objectMapper.treeToValue(patched, MyEntity.class);
    } catch (JsonPatchException | JsonProcessingException e) {
        throw new MyException("Error patching/updating entity", e);
    }
}

Hey, @Manfred73.
Thanks for the feedback. We'll add an example using a real database, but it might take a while.
This issue will remain open until then.

Hi,
I found the solution was to do a merge instead of a persist.
Panache doesn't have a merge (like persist) on the repository itself, but you can do it via the EntityManager as follows:
repository.getEntityManager().merge(entityPatched)


Fix caching of Wikibase suggest results

Closes #5190.
This bug was happening because we were storing the results of Wikibase requests incorrectly in the cache. They needed to be translated to a common format supported by the library.

Hi, i don't know how start one conversation in github, can you help me?


About some attribute data files

Thank you for your great work!
Recently I am running your codes, and I found some of the data files are missing.
For example, I don't know where is the "attrib2instance.pkl" in the line 101 of misc/gen_coco_attribute_objs.py.
Also the "attrib_detections.json" in the line 26 of misc/prep_coco_att_data.py seems to be missing so I cannot run the code.
Could you provide such files or provide the ways to get these files?
Thank you very much!

Have you solved it yet

Not yet, I gave up on running this repository.


Dashboard Native Filter: cannot filter null values

I've realized there is an option of "null" for the select filter in the Dashboard's Native Filter. However, when null was clicked as a filter, it does not show the correct output on the charts. Whereas when I try selecting null in the Filter Box created using Explore, it works. I'm using the same dataset for all the charts, filter box, and dashboard native filter.
Expected results
I expect that the Dashboard Native Filter works the same as a Filter Box. In this case, when you click the "null" option shown in the Dashboard Native Filter, it should apply the condition = null for all the charts in the dashboard if they have the specific column, just like how a filter box would do it.
Actual results
When you click "null" option in the Dashboard Native Filter, it just shows "No Data" / count(*) is 0 in the charts, as if there's no null values for the column.
what actually happens.

Screenshots
This screenshot is from a Filter Box created through Explore, and it shows the correct result.

How to reproduce the bug

Display a chart in dashboard where there are null values for a column
Select the column for both Native Filter and Filter Box in the dashboard
Choose "null" in Dashboard Native Filter to simulate "where column is null", you will see that the result of the chart would be "No Data", which is wrong.
Remove the null filter in the Dashboard Native Filter.
Choose "null" in the Filter Box and click apply. The results of the chart is correct.

@junlincc

Hi! I see @mironovmeow created pull request with fixing this issue.
@zhaoyongjie @junlincc will you review it? Or we wait for realization from superset team?


Add minutes to wait_for_tokens when remaining tokens is negative

The current method incorrectly calculated wait time when AsyncKeepa().tokens_left was a negative number.

Seems have to been accounted for in #89.  Merging anyway.

Seems have to been accounted for in #89. Merging anyway.

Ah, my bad. I should have rebuilt my fork. Sorry to muddy up the history!


Invalid files in IOC Editor

IOC Editor fails load these files.
"ioctool 1.0.2100.0
Found 5 files
Checked 0 indicators (5 failed) and 0 were not Unencapsulated OpenIOC format"

@marius0000 what version of IOCe are you using to view these files?  What files are you having issues with?
I just verified that IOCe v <IP_ADDRESS>, available at http://www.mandiant.com/resources/download/ioc-editor/ opens all of the files which have been published in this repository.

recently I met the same problems as marius0000 mentioned, was there any method to solve it?
"ioctool 1.0.2100.0
Found 3 files
Checked 0 indicators (3 failed) and 0 were not Unencapsulated OpenIOC format"


Using program gives error: "FileNotFoundError: [Errno 2] No such file or directory: 'ffmpeg'"

I have verified that I have the prerequisite packages installed. I also installed ffmpeg using pip. I am using python version 3.9.10

To fix that you need to install ffmpeg using brew or apt

you must add ffmpeg to your environmental variables

you must add ffmpeg to your environmental variables

https://www.gyan.dev/ffmpeg/builds/


how to use

Hello, dear
How do I use this beauty?

This is a very old "product" that I released back when there wasn't much like it around, but this is basically a DNS enumerator.
You can download the latest release (from 2008) to on my site: https://www.networkdls.com/Software/View/IPLister.


Add debug logging functionality for GPT interactions

Introduced methods to enable and disable debug logging in UserActions class.
Added Talon commands to toggle debug logging.
Updated modelHelpers.py to print request data when debug is enabled.
Modified GPTState class to include debug_enabled flag and methods to toggle debug state.

I am fine with this, but if there are only two states, enabled or disabled, and the methods don't have other side effects, I think we should just have one toggle method. And then the command would be model toggle debug. Could you make that change?

To be honest I prefer to have explicit commands to enable and disable in this case because it's not easy to view the current debug state so it would be easy to get confused about which state it is in. I would be okay with changing the commands to be 'start' and 'stop' though if that's better.

OK let's use start and stop then just to keep things consistent. Then you can merge after that

Looks good now, thanks!


CB-8727 Fix failing WebSQL test on Android 5.1

scrollByLines is an unknown function in Android 5.1
removed because it causes the exception and does not affect the test itself

Can someone take a look at this? I think it should be a simple merge

What was the purpose of the call before?

I think it was to make sure the most recent additions were seen.  Seems someone magically assumed it would always be less than 20000 lines.

Oh, so it's mostly a convenience for a human tester?

yes. I believe that was the intent, although it obviously failed.
This is an easy merge. +1

Merging.

I done goofed and forgot to say "This closes #124". Merged though.