Count option is not refreshed Server side rendering V2.1.0

I've got some issues with count options with server side rendering.
Count is not updated when I update my options parameter.

Count is initialized with 0 in first render
Then count is fetch with async method
Options is updated with right count
MuiDatatable is not refresh (as options is setup in componentWillMount method) It uses only number of element of my data array

async componentDidMount() {
        // Start fetching data
        manufacturersCollection.fetch();
        const value = await phonesCollection.fetchCount(this.componentName)
        runInAction(() => this.count = value);
        phonesCollection.fetch({data: {category: this.componentName, page: 0}});
        const navigationData = this.props.location.navigationData;
        NavigationUtils.showNavigationMessage(navigationData, this.snackBar);
    }

// Prepare data from collection Transform list of object to simple array with order
    @computed
    get preparedData() {
        return DataTableUtils.createDataArray(phonesCollection, this.columns);
    }
render() {

        const {classes, breadcrumbs} = this.props;

     // Options for basic grid
        // TODO: Options in custom component
        const options = { selectableRows: false,
                        serverSide: true,
                        count: this.count,
                        download: true,
                        print: false,
                        viewColumns: false,
                        responsive: "scroll",
                        filterType: 'dropdown',
                        rowsPerPage: 15,
                        onTableChange: (action, tableState) => {
                            switch (action) {
                            case 'changeRowsPerPage':
                            case 'changePage':
                                phonesCollection.fetch({data: {category: this.componentName, page: tableState.page, size: tableState.rowsPerPage}});
                              break;
                            case 'search':
                                if(tableState.searchText && tableState.searchText.length > 1) {
                                    phonesCollection.fetch({data: {category: this.componentName, page: tableState.page, size: tableState.rowsPerPage, searchText: tableState.searchText}});
                                } else if(tableState.searchText === null) {
                                    phonesCollection.fetch({data: {category: this.componentName, page: tableState.page, size: tableState.rowsPerPage}});
                                }
                                break;
                            default:
                                break;
                          }
                        },
                        downloadOptions: {separator: ';'}
                    };

        // Display normal data on page
        return (
                <React.Fragment>
                    {breadcrumbs}
                    <div style={{ margin: 20 }}>
                    {/* Page header + toolbar  */}
                    <Typography classes={{root: classes.headLineDataTable}} variant="h4" color="initial" noWrap>
                        {this.componentName}
                        <Button variant="contained" className={classes.button} size="small" onClick={this.startCreation} data-cy="create-user-button">
                            <AddIcon classes={{root: classes.iconThemeColorGreen}} className={classes.leftIcon} />
                            Create {this.componentName}
                      </Button>
                    </Typography>

                    {/* Datatable to display results */}
                    <MUIDataTable
                            data={this.preparedData}
                            columns={this.columns}
                            options={options}
                    />
                    <SimpleSnackbar ref={this.snackBar}/>
                    </div>
                </React.Fragment>
        );
    }

And as a workaround, I just put this, that is working haha ! :)
 {/* Datatable to display results */}
<MUIDataTable key={this.count}
            data={this.preparedData}
             columns={this.columns}
             options={options}
/>

same in V2.2.0

And as a workaround, I just put this, that is working haha ! :)
 {/* Datatable to display results */}
<MUIDataTable key={this.count}
            data={this.preparedData}
             columns={this.columns}
             options={options}
/>

Works in v2.2.0

This is also the case for filterList in the column options

I think it's quite similar with my issue #658 . I return to version 2.0.0 and it works well

I have some regressions with v2.3.0 on columns parameters too..
I change this object but mui-datatables till have the old columns settings put in first place.
I did this by the past to make it work but it doesn't now with or without this :
// Change array reference so that datatable API knows sthing changed
this.columns = this.columns.slice(0);

...

 return (
                <MUIDataTable
                title={<Typography variant="subtitle1" component="span" color="textSecondary" noWrap>{this.props.componentName}</Typography>}
                data={this.data}
                columns={this.columns}
                options={this.options}
                />
                );

It was working before with old version  I had : 2.0.0-beta-54
I think we are in the same kind and related situation.

@jkeruzec We have a separate issue to track column props more generally: https://github.com/gregnb/mui-datatables/issues/643.

The solution of using the prop key fixes the problem but creates another.
When u are filtering the table and the server returns a different count value the filters menu closes and, besides, if u try to clear the filter deleting the chip the 'tableState.filterList' don't change, he keeps always the same array of values.

@VagnerNico  Yes I saw that too on my side, sorry for that, Solution I proposed was only a workaround waiting for a proper fix...

I checked the code and found the problem. The count options are passed to the component only one time on componentWillMount lifecycle method, so I created a little tricky code that solved the problem, that solution consists of add count to the state of the table and on componentDidUpdate u compare props received with the component state.
I've created a branch with the fix and one working example with filters.

@VagnerNico This fix is already in PR. Make sure to check up on the PRs first before duplicating effort!

My mistake I have not read all the comments kkkk

No worries, I appreciate the initiative! :) Just wanted to save you some effort.
Something we do need at this point is a temporary fix for column props updating, if you want to take a look: https://github.com/gregnb/mui-datatables/issues/643. There is also a PR open for that, but I've given feedback around needed fixes.

Not all work was in vain, I believe that the example I put in my PR could help to test the bug fix proposed by @bndynet. The example simulates a server-side rendering with data length equals to rowsPerPage, but with the count value coming from the server, that represents all the length of the database for a given query.
About the issue #643 I will give a try to fix.

@VagnerNico Can you submit a PR with those tests? Would be much appreciated!

@gabrielliwerant PR #682 ready, u can test with the code from @bndynet, tested here and all works fine incoming data and filters.

Should be fixed in 2.4.0. Please open a new ticket with reproducible steps if there are additional issues.


Using instance of Robot() in a separate node

After starting the robot instance using
from rae_sdk.robot import Robot robot = Robot()
Can that instance of robot be called from another node to, for example stop that instance or add a publisher, nav command or make another api call to Robot? Creating a second instance of Robot() disrupts the first instance.

Hi, do you mean if it's possible to create ROS interfaces (subscriptions, timers) etc. via some external service call? That is currently not available. I also did not test it yet but I think it should be possible to create a ROS node that has Robot class in it. Regarding multiple 'Robot' class instances, you can use RobotOptions to indicate whether new instance should start the hardware controllers and/or controllers, is that more in line of what you need?

Here's an example: robot_test_1.py brings up rae, and robot_test_2.py seems to have no effect. This would be the same if a new Robot object is instantiated in a node, for example to add a publisher or test the audio using the python api:
robot_test_1.py:
from rae_sdk.robot import Robot
from rae_sdk.robot.robot_options import RobotOptions

## brigup rae
options = RobotOptions(
    name='rae_sdk_1',
    namespace='',
    launch_controllers=True,
    start_hardware=True,
    launch_mock=False
)
robot = Robot(options)
robot.led.set_leds('#FF0000', 100, 'solid', 1)  # set color to red

and
robot_test_2.py:
from rae_sdk.robot import Robot
from rae_sdk.robot.robot_options import RobotOptions

options = RobotOptions(
    name='rae_sdk_2',
    namespace='',
    launch_controllers=False,
    start_hardware=False,
    launch_mock=False
)
robot = Robot(options)
robot.led.set_leds('#0000FF', 100, 'solid', 1)  # set color to blue

Hi, sorry for the delay, I didn't get the notification. If you set launch_controllers to true in robot_test_2.py you should be able to use the interfaces.


Add support for building docker-engine RPM on Oracle Linux 7.

This adds support for building the upstream docker-engine on Oracle Linux 7, using the oraclelinux:7 base image available on the Docker Hub Registry.
Resolves issue #14481
Signed-off-by: Avi Miller<EMAIL_ADDRESS>

I will try this, thanks

can you also add closes # whatever that issue was to your first comment so the issue is closed when we merge this

ok these are running now, but in the meantime ping @tianon to make sure everything checks out with him

:+1: I'll be submitting a similar pull request for Oracle Linux 6, once I fix up a few internal things in the oraclelinux:6 image so that we can use a single .spec file.

do we support that version because we are also dropping support of rhel 6, what kernel does oracle linux 6 use?

Oracle Linux 6 ships with both a RH-compatible 2.6.32 kernel which will not be supported, as well as an Oracle-built 3.8 kernel which will. The changes I need to make internally will allow me to add a Requires: to the specfile specifically for OL6 to require that 3.8 kernel.

ah cool awesome

LGTM ping @tianon

LGTM :+1:

cool!


Could not compile:

Hello i get this error when i want to compile for my NodeMCU: (i changed nothing in source.)
In file included from /Users/Juergen-S/Downloads/McLighting-master/Arduino/McLighting/McLighting.ino:218:0:
sketch/request_handlers.h: In function 'void onMqttDisconnect(AsyncMqttClientDisconnectReason)':
request_handlers.h:1132: error: 'TLS_BAD_FINGERPRINT' is not a member of 'AsyncMqttClientDisconnectReason'
if (reason == AsyncMqttClientDisconnectReason::TLS_BAD_FINGERPRINT) {
^
request_handlers.h:1146: error: 'ESP8266_NOT_ENOUGH_SPACE' is not a member of 'AsyncMqttClientDisconnectReason'
} else if (reason == AsyncMqttClientDisconnectReason::ESP8266_NOT_ENOUGH_SPACE) {
^

AMQTT of enabled by default, so have you installed async mqtt libraries?
See wiki for other requirements:
AMQTT integration (enabled by default)
Async MQTT by @marvinroger (tested with 0.8.1)
Only when you have activated AMQTT in definitions.h.
https://github.com/marvinroger/async-mqtt-client
Needs ESPAsyncTCP: https://github.com/me-no-dev/ESPAsyncTCP
(Download the library repository as ZIP. Load that library into to Arduino software by: "Sketch > Include Library > Add .ZIP Library".)

Ohoh... now it works!
Thank you


Document list triggers error in change request after a schema change

I experience a "GraphQL query error" in a document list in the following case:

Create a change request CR1 for an existing database
Exit the change request CR1
Make changes to a document schema and commit them
Activate the change request CR1
Open the document list for the modified document type

The error "GraphQL query error" is displayed. In the network pane the affected resource returns 404.

fixed


optimize the speed of tests with stop sleep container

Signed-off-by: Ye Sijun<EMAIL_ADDRESS>Stop a sleep container always need to wait for 10s, it's so slow.
Before:
$ go test -v ./cmd/nerdctl/... -exec sudo -run TestSharedNetworkStack
test target: "nerdctl"
=== RUN   TestSharedNetworkStack
    run_network_linux_test.go:512: container nerdctl-testsharednetworkstack is now running
--- PASS: TestSharedNetworkStack (22.34s)
PASS
ok      github.com/containerd/nerdctl/cmd/nerdctl       22.369s

After:
$ go test -v ./cmd/nerdctl/... -exec sudo -run TestSharedNetworkStack
test target: "nerdctl"
=== RUN   TestSharedNetworkStack
    run_network_linux_test.go:512: container nerdctl-testsharednetworkstack is now running
--- PASS: TestSharedNetworkStack (2.62s)
PASS
ok      github.com/containerd/nerdctl/cmd/nerdctl       2.656s

Thanks!


Improve diffing implementation

To diff the set of signatures we need to fold over a pair of maps representing each a version of the interface.
Our current implementation does by folding over both maps sequentially to be able to detect added and modified values in the first pass and removed values in the second pass.
This is not very efficient and folding over both is in fact possible using Map.merge which signature correspond exactly to what we want.

CC @Siddhi-agg

@NathanReb Commenting to be assigned.


Latest MOLOv5 MergeCMS(inprogress) - No merge with Develop

The article page does not update comment counter e.g.
There's more that 2 replies

@projectmushroom @codiebeulaine please review
Nofitication:

Comments & Reply thread:

View more comments:

Reply & Reply list:


Add shuffle_trials functionality

 @gcroci2 cloned issue ANNUBS/neurogym#23 on 2024-08-28:

We need to add the shuffle_trials functionality to the class/module we will develop.

 @gcroci2 commented on 2024-10-15:

In the original annubes repo, we predefined the number of trials in a sequence, allowing us to directly create it (shuffled or not). However, in the current setup, we define a general environment where sequences are generated dynamically, with stimuli types shuffled by default based on probabilities at each trial.
While this isn't an issue for plotting (we can simply modify the plotting function if we want an unshuffled sequence), it presents a challenge for the learning process. To control the sequence during training, we would need to modify the step() method to track the number of steps, along with a counter that keeps track of the total steps required. I’m unsure how to implement this total step count effectively.
My main question is whether training an agent on non-shuffled trials (e.g., first 1000 visual stimuli trials, followed by 1000 auditory trials) is even necessary. Wouldn't it make more sense to train separate agents for each modality (visual and auditory)? If training a single agent on both modalities is desired, sequential training (first visual, then auditory) could be implemented, which aligns with the approach in #21.
Please let me know if replicating the original annubes behavior is essential @jfmejias, and we can explore it further.


Broken link

[Enter feedback here]
Link
Learn more about Microsoft 365 Defender
https://docs.microsoft.com/en-us/microsoft-365/security/defender/microsoft-threat-protection
doesn't work
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 881d6a41-be6c-430e-d4f3-26e08fa1bb50
Version Independent ID: 5bb5aaaa-8d13-f3e1-52b1-3a043f7ff4fe
Content: Integrate Microsoft Defender for Endpoint with other Microsoft solutions
Content Source: microsoft-365/security/defender-endpoint/threat-protection-integration.md
Product: m365-security
Technology: mde
GitHub Login: @mjcaparas
Microsoft Alias: macapara

@gianlucadardia Thank you for your feedback. We have made changes to the document via PR#https://github.com/MicrosoftDocs/microsoft-365-docs/pull/5267
Thanks
Sri

@msbemba Thank you very much for the contribution and updating the documentation with PR. @gianlucadardia Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


TAJO-1480: Kafka Consumer for kafka strage.

Scanner of kafka storage needs to implement a Kafka Consumer for to fetch data from kafka.
The Kafka Consumer have methods like this,

getPartitions(): Get partition id list from specific topic.
poll(): Fetch data from kafka.
getEarliestOffset(): Get earliest offset from specific partition.
getLatestOffset(): Get latest offset from specific partition.
etc..

Thanks you for your contribution! I left trivial comment

@jinossy Thanks for your reviews. I modified sources.
Please, check again.
Thanks.

+1 Looks great to me!
Thank you for contribution!


Update p256-cortex-m4 to 0.1.0-alpha.6

This patch updates the p256-cortex-m4 dependency to version
0.1.0-alpha.6.  This version includes a fix for signatures that are not
exactly 72 bytes long, see https://github.com/ycrypto/p256-cortex-m4/pull/4.
Fixes https://github.com/Nitrokey/nitrokey-3-firmware/issues/31

Hi!
I see this passes updates through Cargo.lock, but I do not see relevant change in the Cargo.toml (specifically trussed dependency). Can you check?

https://github.com/trussed-dev/trussed/blob/main/Cargo.toml#L48

The version = "0.1.0-alpha.5" in Trussed’s Cargo.toml is a shorthand for version = "^0.1.0-alpha.5", where ^0.1.0-alpha.5 translates to something like >= 0.1.0-alpha.5, < 0.2.0.  This is the range of supported versions.  At compile time, we chose one of these versions.  This selection is stored in our Cargo.lock.  So if we want to force all Trussed users to use the new version, we would have to update its Cargo.toml.  This is typically only done for breaking changes that are required to compile the crate.  Updating it for every bug fix would create too much maintenance effort.  In our case, we only want to make sure that the new version is used in our project, so it is sufficient to update the Cargo.lock file for our crate.


[Fix #2008] Check for too many spaces in SpaceBeforeModifierKeyword

Previously we only checked for missing space.

:+1:


programs scenes souliss app

I'm using Soulissapp now for a while, but some instructions doesn't work and others are rather limited.
Programs :

I receive the feedback when a program takes some action, but there is no action on the output. f.e. When I say to put a light on, I receive the message that the light is activated, but the light stays OFF.
Should it be possible to modify the timed action so that this function can be used to make a eek program ?
Couldn't find how to set a temperature for the thermostat.
Scenes :
Scenes seems limited to 8 typicals. When you more than 8, the first action is repeated.
Could it be possible to select also programs in a scene ?

I'm using : arduino IDE 1.6.5, Souliss V7.1, Soulissapp 1.8.3

You should use the integration with Automate or Tasker to have more
flexibility.
Dario.

Dario,
Please don't see my comments as remarks. It is only a feedback to improve the Souliss system after using it for a while.
My choice for Souliss is based on following strong points of the system :+1:

intelligent front ends who can work stand alone.
redundancy with the network by using a backup gateway.
no central server needed
users friendly APP. User need no knowledge of software to configurate the app.

When you need to use a second app, like Automate or Tasker, you are loosing the last benefit of the system.

Your comments are appreciated and I mostly agree with you, but IfThenElse
app are really complex piece of software and we have decided to integrated
with those defacto standard for such task instead of rebuild the whole from
scratch.
SoulissApp hasn't a big enough audience to justify a such big effort.

I receive the feedback when a program takes some action, but there is no action on the output. f.e. When I say to put a light on, I receive the message that the light is activated, but the light stays OFF.

This is because we're on UDP I guess. Have you tried incrementing the scenario's command delay (in the app)?

Should it be possible to modify the timed action so that this function can be used to make a eek program ?

what does 'eek' programs mean? Could you elaborate pls?

Couldn't find how to set a temperature for the thermostat.

well, T51 has a long and sad history. Anyway, the setpoint should be working... @fazioa what do you say?
Scenes :

Scenes seems limited to 8 typicals. When you more than 8, the first action is repeated.

Really? well, I think this is an actual bug, I'll check in the code

Could it be possible to select also programs in a scene ?

No, the two things are separated. A scene is a plain list of commands executed in sequence, while a program is a command triggered by some action/value/movement. The opposite is possible tho, meaning that, as a program's target, you may specify a scene so that a list of commands is executed when something happens


Slots inprovment

improve the slots interface:

[x] make the fields collapsible
[x] display quantity with image
[x] let user search for and select occupants as well as change quantity
[x] try to make it as compact as possible (slot filter?)
[x] pageify if number of slots is large (vending machine)?

@ComputerDruid @Emilgardis
next update is pretty much ready
if either of you want / can help out with a review that would be great.

I should have some time to take a look today, sure!

I tried the demo and came away with some notes:

You probably want to write patch notes for this still
I can't change the ReferenceId of a IC10 chip I added to a second CircuitHousing. Not that I know why it would matter what the id of a slot occupant is, but maybe it would matter if it was using an ID I wanted to use for something else?
wow, adding a stack of coal ore to the input slot of the sorter, and seeing it rendered with the icon and the little 1/50 looks really good, very impressive.
I can add a second circuit housing and a chip to it, but it still doesn't appear in the drop-down for setting the active IC. Presumably that's just not implemented yet? Also changing the ID of the main Housing makes it so that it doesn't run anymore, but I think that was true before this PR.
wow, the device search is fast now! Presumably helped by the "too many results" page.

I tried it out with a random file I had open in a tab and was able to add stuff to the import slots and see it react (turning on the silo), so that's a good sign. link to my test script
Overall the new stuff feels really nice to use. I'm planning to look at the code later today, too.

I tried the demo and came away with some notes:
* You probably want to write patch notes for this still

absolutly

* I can't change the ReferenceId of a IC10 chip I added to a second CircuitHousing. Not that I know why it would matter what the id of a slot occupant is, but maybe it would matter if it was using an ID I wanted to use for something else?

the slot occupant Id change will need to wait for a bigger rewrite I'm planning

* wow, adding a stack of coal ore to the input slot of the sorter, and seeing it rendered with the icon and the little 1/50 looks really good, very impressive.

(^^)

* I can add a second circuit housing and a chip to it, but it still doesn't appear in the drop-down for setting the active IC. Presumably that's just not implemented yet? Also changing the ID of the main Housing makes it so that it doesn't run anymore, but I think that was true before this PR.

I think I can fix the housing Id change, in theory I can enable multiple IC's at this point but I'd rather wait till after that bigger rewrite.

* wow, the device search is fast now! Presumably helped by the "too many results" page.

Yeah, turns out rending that many dom nodes at once is slow, who knew? (I did, I just didn't consider how slow)

Overall the new stuff feels really nice to use. I'm planning to look at the code later today, too.

I did read through the rust code but most of this change wasn't rust (and I'm not familiar enough with js to be able to offer much insight, so I didn't try reading that). The bits I read all look good to me!


Stylize info table

The information table for each point is now stylized and is much more readable.

Looks Great! Much clearer to see now and we can better utilize Point-Info


ADDON-34278: single select component

Jira: https://jira.splunk.com/browse/ADDON-34278

Add singleSelect component

Jira: https://jira.splunk.com/browse/ADDON-36031

Made mapping an optional entity for heading and moreInfo
Add mapping support for table rows (heading in globalConfig)
Fix the precedence for autoCompleteFields and endpointUrl in singleSelect and multiSelect components

I will update the single select component and react-ui version once it is released with support for label-value mapping in ComboBox.


ensure private key and certificate match

Adding an ingress tls secret with a non matching certificate and private key break at least the nginx-controller permanently until the offending secret is deleted.
In that case nginx refuses to start/reload with an error like this:
Error: exit status 1
2017/06/13 12:16:53 [emerg] 51#51: SSL_CTX_use_PrivateKey_file("/ingress-controller/ssl/tls-baremetal-3-example-com.pem") failed (SSL: error:0B080074:x509 certificate routines:X509_check_private_key:key values mismatch)
nginx: [emerg] SSL_CTX_use_PrivateKey_file("/ingress-controller/ssl/tls-baremetal-3-example-com.pem") failed (SSL: error:0B080074:x509 certificate routines:X509_check_private_key:key values mismatch)
nginx: configuration file /tmp/nginx-cfg728491545 test failed

This change is

Coverage increased (+0.02%) to 46.5% when pulling 8304feb497ae7793b88d1b2eed29d33acb648c59 on databus23:patch-1 into eb618737308fa78f154259f96edabae3a2c375ea on kubernetes:master.

/lgtm

@databus23 thanks!


Migrating azure-monitor-opentelemetry to azure-sdk

Description
Please add an informative description that covers that changes made by the pull request and link all relevant issues.
If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

Thanks, @jeremydvoss.
Noted some items needed for getting this ready for making this repository its home 😃 . Most of it just adding missing files (see the template project for the general idea):

Add a dev_requirements.txt file which is used for ci/testing. I think the following contents are probably fine for now.
-e ../../../tools/azure-sdk-tools
-e ../../../tools/azure-devtools

along with the rest of your test requirements.

Add a CHANGELOG.md file. Formatting guidance can be found here, but pretty much formatted like:
# Release History

## 1.0.0b12 (Unreleased)

### Features Added

### Breaking Changes

### Bugs Fixed

### Other Changes

Add an sdk_packaging.toml file. Can just contain:
[packaging]
auto_update = false

Add a pyproject.toml file with a section like:
[tool.azure-sdk-build]
type_check_samples = false
verifytypes = false
pyright = false

Can also do pylint = false if you want to disable that too, but better to see if you can get it to pass first. Similarly with mypy = false.

Add an empty py.typed file to the project (i.e., ./azure/monitor/opentelemetry/py.typed)

Add a MANIFEST.in file. I think contents would be:
recursive-include tests *.py
recursive-include samples *.py
include *.md
include LICENSE
include azure/__init__.py
include azure/monitor/__init__.py
include azure/monitor/opentelemetry/py.typed

Can remove setup.cfg as it's not needed.

Add the project to the Artifacts list in ci.yml

Update https://github.com/Azure/azure-sdk-for-python/blob/main/shared_requirements.txt with package names that are included in your setup.py dependencies but not listed there.

Done.

PR for vendoring: https://github.com/microsoft/ApplicationInsights-Python/pull/280

/azp run python - monitor - ci

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-monitor-opentelemetry

Ubuntu tests failing seems to stem from the following error:
.tox/mindependency/lib/python3.11/site-packages/wrapt/decorators.py:34: in <module>
    from inspect import ismethod, isclass, formatargspec
E   ImportError: cannot import name 'formatargspec' from 'inspect' (/opt/hostedtoolcache/Python/3.11.4/x64/lib/python3.11/inspect.py)
whl: OK ✔ in 44.51 seconds
sdist: OK ✔ in 49.91 seconds
  whl: OK (44.51=setup[37.01]+cmd[0.12,4.88,0.24,2.25] seconds)
  sdist: OK (49.91=setup[40.60]+cmd[7.46,0.23,1.63] seconds)
  mindependency: FAIL code 4 (115.93=setup[26.96]+cmd[87.16,1.11,0.23,0.13,0.34] seconds)
  evaluation failed :( (116.01 seconds)
ERROR:root:Command '['/opt/hostedtoolcache/Python/3.11.4/x64/bin/python', '-m', 'tox', 'run-parallel', '-p', 'all', '--root', '.', '-c', '/mnt/vss/_work/1/s/eng/tox/tox.ini', '-e', 'whl,sdist,mindependency', '--', '--no-cov', '--ignore', 'azure_monitor_opentelemetry.egg-info']' returned non-zero exit status 255.

cannot import name 'formatargspec' from 'inspect'

https://github.com/GrahamDumpleton/wrapt/issues/196


no-unused-vars: Type[] doesn't mark Type as used

Well, I spent some time on this. Here is a PR with a failing test case.
I would really like to see the solution to this, because from my debugging, the identifier Type does get marked as used. So why the fail?

Alright, I have a solution for this.
I don't have push access to your repo, so I will just submit a new PR.
You can close this PR as I included your test case in mine, no need to keep this one open...
Let me know if you find any other case, I've been submitting PRs improving this particular rule, but I'm pretty sure there are still cases that I haven't taken into account...

@weirdpattern thanks for the fix. Next time, you could cherry pick the failing test commit. Or some other way to credit me or the next guy in the git log.

@weirdpattern Or rebase your PR on top of this branch.


How to make power of 10 in superscript in optical spectra

Hello,
I am trying to plot the optical spectra of some material. I am not getting the power in superscript at the y-axis.
I am attahing my plot here.

How can I get rid of this?

That is odd, can you let us know what versions of Numpy and Matplotlib you are using?

Numpy= 1.20.2  and 1.22.3
matplotlib=3.4.1 and 3.6.2
On Sat, Jan 21, 2023 at 2:58 PM Adam J. Jackson @.***>
wrote:

That is odd, can you let us know what versions of Numpy and Matplotlib you
are using?
—
Reply to this email directly, view it on GitHub
https://github.com/SMTG-UCL/sumo/issues/196#issuecomment-1399215845, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ADZPLT2PBNWQRWVSUBWA4DDWTOT4XANCNFSM6AAAAAAT4OZXT4
.
You are receiving this because you authored the thread.Message ID:
@.***>

Fixed in #203


JSON_BuildTests fail when JSON_DisableEnumSerialization is set to ON

Description
Setting JSON_DisableEnumSerialization to ON when building JSON for Modern C++ with testing enabled (JSON_BuildTests set also set to ON), the tests do not build and the build fails. Tests that rely on that setting should not be built for library clients who do not want EnumSerialization and the build should succeed.
Reproduction steps
From a fresh clone of nlohmann/json, I ran the following commands to configure and build the library using CMake:
/usr/bin/cmake -DJSON_DisableEnumSerialization=ON -DJSON_BuildTests=On -DJSON_Install=ON -DJSON_SystemInclude=ON -DCMAKE_INSTALL_PREFIX=/workspaces/json/install/gcc12-release -DCMAKE_EXPORT_COMPILE_COMMANDS=true -DCMAKE_BUILD_TYPE=Release -DCMAKE_CXX_COMPILER:FILEPATH=/usr/bin/g++-12 -S/workspaces/json -B/workspaces/json/build/gcc12-release -DCMAKE_BUILD_TYPE:STRING=Release -DCMAKE_C_COMPILER:FILEPATH=/usr/bin/gcc-12 -DCMAKE_CXX_COMPILER:FILEPATH=/usr/bin/g++-12 -DCMAKE_CXX_STANDARD:STRING=20 -DCMAKE_CXX_STANDARD_REQUIRED:BOOL=ON -DCMAKE_CXX_EXTENSIONS:BOOL=OFF -DCMAKE_CXX_FLAGS_INIT:STRING='-fPIC -O3 -mavx2 -mavx'
/usr/bin/cmake --build /workspaces/json/build/gcc12-release --parallel 17 --target all

Expected vs. actual results
I expect successful building and linking of all executable tests with no errors or warnings. In practice, the compiler produces errors and warnings as detailed below in the Error messages field in this form.
Minimal code example
N/A - just building the tests of the library

Error messages
/workspaces/json/tests/src/unit-conversions.cpp:1272:53: error: expected primary-expression before ‘)’ token
 1272 |         CHECK(json(cpp_enum::value_1).get<cpp_enum>() == cpp_enum::value_1);
      |                                                     ^
/workspaces/json/tests/thirdparty/doctest/doctest.h:2427:24: note: in definition of macro ‘DOCTEST_ASSERT_IMPLEMENT_1’
 2427 |                     << __VA_ARGS__) DOCTEST_CLANG_SUPPRESS_WARNING_POP
      |                        ^~~~~~~~~~~
/workspaces/json/tests/thirdparty/doctest/doctest.h:2969:20: note: in expansion of macro ‘DOCTEST_CHECK’
 2969 | #define CHECK(...) DOCTEST_CHECK(__VA_ARGS__)
      |                    ^~~~~~~~~~~~~
/workspaces/json/tests/src/unit-conversions.cpp:1272:9: note: in expansion of macro ‘CHECK’
 1272 |         CHECK(json(cpp_enum::value_1).get<cpp_enum>() == cpp_enum::value_1);
      |         ^~~~~
gmake[2]: *** [tests/CMakeFiles/test-conversions_cpp11.dir/build.make:76: tests/CMakeFiles/test-conversions_cpp11.dir/src/unit-conversions.cpp.o] Error 1
gmake[1]: *** [CMakeFiles/Makefile2:1763: tests/CMakeFiles/test-conversions_cpp11.dir/all] Error 2
gmake: *** [Makefile:146: all] Error

Compiler and operating system
gcc12, Ubuntu 22.04.1 x86_64
Library version
3.11.2
Validation

[X] The bug also occurs if the latest version from the develop branch is used.
[ ] I can successfully compile and run the unit tests.

That's a very specific use case, but the error should be easily fixable. I will have a look.

I propagated the JSON_DisableEnumSerialization flag to the test cases to exclude those that rely on enum conversion. Please take a look @madamsrc @efesx

I propagated the JSON_DisableEnumSerialization flag to the test cases to exclude those that rely on enum conversion. Please take a look @madamsrc @EfesX

Thanks @nlohmann - this fix did the trick!


New version: LoopVectorization v0.12.52

Registering package: LoopVectorization
Repository: https://github.com/JuliaSIMD/LoopVectorization.jl
Created by: @chriselrod
Version: v0.12.52
Commit: 8c8cda9e30151b78dcc922f49b13ed6220e81771
Reviewed by: @chriselrod
Reference: https://github.com/JuliaSIMD/LoopVectorization.jl/commit/8c8cda9e30151b78dcc922f49b13ed6220e81771#commitcomment-53631994

[noblock]
@chriselrod Did you intend to skip a version?

[noblock]
@chriselrod Did you intend to skip a version?

No, that was a mistake.


Add support for insets on Android 9 and below

Thanks for this really useful plugin! I use it in an app which has both a transparent status bar and navigation bar, so it's very useful to make sure buttons are not hidden under them.
Currently on Android 9 and below, the plugin always returns 0 for all insets, which was causing buttons to be rendered below the status and navigation bars in my app. This is because the insets are only calculated for Build.VERSION_CODES.Q and above.
This PR adds an else branch to this if check to ensure the insets are returned correctly on Android 9 and below. I have tested it in my app, and the height of the transparent status and navigation bars are returned correctly. getSystemWindowInsets is only supported on Android 10 and above, but the individual calls like getSystemWindowInsetTop are supported down to Android 4.4, which I why I've used those.

@Menardi Thanks for the really useful PR, merging it, also apologies for the long delay!

Published with v2.1.2


Error converting fir to verilog

Hello:
I have write hardware module with Chisel, and converted it to verilog code.
However, it seems odd behavior when convert fir to verilog.
As below:
.fir
module TLBridgeDMA_bridgedma :
input clock : Clock
input reset : UInt<1>
...................
reg sdma_state : UInt<3>, clock with : (reset => (reset, UInt<3>("h00"))) @[TLDMA.scala 92:42]
.................

.v
module TLBridgeDMA_bridgedma(
input         clock, //<EMAIL_ADDRESS>input         reset,
.............

I cannot find something as "reg sdma_state" in verilog code and it disappears.
I am not sure why such odd behavior occurs.
Is it bug or my Chisel implementation code violate some coding style rule?

Is it possible that your register might have gotten ConstProped away?

Sorry about that I am not familiar with firrtl end.
Could you describe ConstProp detailed?

In my Chisel implementation, there is another state register ddma_state: val ddma_state              = RegInit(ddma_idle). I also write two state List in single module.
val sdma_idle :: .... :: Nil
val ddma_idle :: .... :: Nil

I can find ddma_state register in output verilog code, but sdma_state is missing.
reg [2:0] ddma_state; // @[TLBridgeDMA.scala<EMAIL_ADDRESS>

Can I write multiple state List as above in single module?
Or it cannot. Therefor I get such error.

Sorry for being nagging again.
I have traced out the fir file and verilog file.
Some register as "sdma_state" and fsm operation blocks exist certainly in fir fie.
However, they disappear in out verilog file.
I still cannot figure out the reasons behind.

As @edwardcwang gets at, this is likely either due to one of two transforms optimizing away parts of your code through either:

Dead Code Eliminataion
Constant Propagation

Consider the following FIRRTL code. This has three modules: A, B, C. All three have an internal 1-bit register called sdam_state. Dead code will remove this register in A (because sdma_state is never used), constant propagation will remove B (and the entirety of module B because sdam_state is set to zero and never changes), and C will be not modified by either of these two passes.
circuit Top:
  module A:
    input clock: Clock
    input reset: UInt<1>
    input in : UInt<1>
    output out: UInt<1>

    reg sdma_idle : UInt<1>, clock with : (reset => (reset, UInt<1>("h00")))

    out <= in

  module B:
    input clock: Clock
    input reset: UInt<1>
    input in : UInt<1>
    output out: UInt<1>

    reg sdma_idle : UInt<1>, clock with : (reset => (reset, UInt<1>("h00")))

    out <= sdma_idle

  module C:
    input clock: Clock
    input reset: UInt<1>
    input in : UInt<1>
    input switch : UInt<1>
    output out: UInt<1>

    reg sdma_idle : UInt<1>, clock with : (reset => (reset, UInt<1>("h00")))

    when switch :
      sdma_idle <= xor(UInt<1>("h01"), sdma_idle)

    out <= or(in, sdma_idle)

  module Top:
    input clock: Clock
    input reset: UInt<1>
    input in : UInt<1>
    output out_a : UInt<1>
    output out_b : UInt<1>
    output out_c : UInt<1>

    inst a of A
    inst b of B
    inst c of C

    a.clock <= clock
    a.reset <= reset
    a.in <= in
    out_a <= a.out

    b.clock <= clock
    b.reset <= reset
    b.in <= in
    out_b <= b.out

    c.clock <= clock
    c.reset <= reset
    c.in <= in
    c.switch <= asUInt(clock)
    out_c <= c.out

I then run this through the FIRRTL compiler with:
firrtl/utils/bin/firrtl -ll Debug -i test.fir -o test.v

While all this is flying by, you see the following which provides information about what dead code is being eliminated.
======== Starting Transform DeadCodeElimination ========
[DCE] port A.clock
[DCE] port A.reset
[DCE] reg A.sdma_idle
[DCE] port B.clock
[DCE] port B.reset
[DCE] port B.in
[DCE] port B.out
[DCE] reg B.sdma_idle
[DCE] module B
[DCE] inst Top.b
--------------------------------------------------------

Time: 11.1 ms
Form: UnknownForm
Annotations:
targetDir
.
EmitCircuitAnnotation(class firrtl.VerilogEmitter)
TargetDirAnnotation(.)
======== Finished Transform DeadCodeElimination ========

I then wind up with the following Verilog (where I've manually removed the randomization stuff to make it less cluttered):
module A( //<EMAIL_ADDRESS>  input   in, //<EMAIL_ADDRESS>  output  out //<EMAIL_ADDRESS>);
  assign out = in;
endmodule
module C( //<EMAIL_ADDRESS>  input   clock, //<EMAIL_ADDRESS>  input   reset, //<EMAIL_ADDRESS>  input   in, //<EMAIL_ADDRESS>  input   switch, //<EMAIL_ADDRESS>  output  out //<EMAIL_ADDRESS>);
  reg  sdma_idle; //<EMAIL_ADDRESS>  reg [31:0] _RAND_0;
  wire  _GEN_0; //<EMAIL_ADDRESS>  assign _GEN_0 = switch ? 1'h1 ^ sdma_idle : sdma_idle; //<EMAIL_ADDRESS>  assign out = in | sdma_idle;
  always @(posedge clock) begin
    if (reset) begin
      sdma_idle <= 1'h0;
    end else begin
      if (switch) begin
        sdma_idle <= 1'h1 ^ sdma_idle;
      end
    end
  end
endmodule
module Top( //<EMAIL_ADDRESS>  input   clock, //<EMAIL_ADDRESS>  input   reset, //<EMAIL_ADDRESS>  input   in, //<EMAIL_ADDRESS>  output  out_a, //<EMAIL_ADDRESS>  output  out_b, //<EMAIL_ADDRESS>  output  out_c //<EMAIL_ADDRESS>);
  wire  a_in; //<EMAIL_ADDRESS>  wire  a_out; //<EMAIL_ADDRESS>  wire  c_clock; //<EMAIL_ADDRESS>  wire  c_reset; //<EMAIL_ADDRESS>  wire  c_in; //<EMAIL_ADDRESS>  wire  c_switch; //<EMAIL_ADDRESS>  wire  c_out; //<EMAIL_ADDRESS>  A a ( //<EMAIL_ADDRESS>    .in(a_in),
    .out(a_out)
  );
  C c ( //<EMAIL_ADDRESS>    .clock(c_clock),
    .reset(c_reset),
    .in(c_in),
    .switch(c_switch),
    .out(c_out)
  );
  assign out_a = a_out;
  assign out_b = 1'h0;
  assign out_c = c_out;
  assign a_in = in;
  assign c_clock = clock;
  assign c_reset = reset;
  assign c_in = in;
  assign c_switch = $unsigned(clock);
endmodule

Usually, DCE/ConstProp are doing the right things and they likely are in your case. If you want to mark specific signals as not getting modified by these types of passes, you can use a dontTouch annotation in your Chisel code: https://chisel.eecs.berkeley.edu/api/#chisel3.core.dontTouch$

Thanks for your tips. I just fix out partial problem.
However, there are still some problems and condition regarded  as  Dead Code Eliminataion from FIRRTL compiler.
Code:
class FrontendModule(c: FrontendModuleParams)(implicit p: Parameters) extends TLRegisterRouter(
    c.address, "FrontendConfig", beatBytes = c.beatBytes)(
      new TLRegBundle(c, _) with DeviceCtrlBundle)(
      new TLRegModule(c, _, _) with FrontendTLModule)

trait DeviceCtrlBundle extends Bundle {
  implicit val p: Parameters

  val front = new ControllerIOBundle  //Output
  val back  = new ResponseIO             //Input
}

Top module:
class TLTOP (address: BigInt, beatBytes: Int)(implicit p: Parameters) extends LazyModule {

  val frontend = LazyModule(new FrontendModule(FrontendModuleParams(address, beatBytes)))

  lazy val module = new LazyModuleImp(this) {

   val source_address  = RegInit(0.U(xLen.W))
   source_address   := frontend.module.io.front.source_address
   ......

  }
}

[DCE] port FrontendModule.io_front_source_address
[DCE] port TLTop_tltop.io_front_source_address

It always return as above message with using FIRRTL in spite of
applying donotTouch API or modifying coding style.

Dead Code Elimination doesn't remove things because they aren't used, it removes them because they do not affect the output. In the partial snippet given, source_address only updates itself, it must affect the output for DCE to keep it. You can mark it dontTouch (eg. dontTouch(source_address)) to ensure it is kept.

If this is still an issue, feel free to reopen.


use correct shader

Do you have a case when that does not work? Texture can be changed, trimming will be changed too. The idea is that both shaders have the same input and thus same location numbers

"Do you have a case when that does not work?"  :  No. If no this PR, it could work too. But in logic , it's not perfect.
So , this pr is not for fixing bugs, just  cosmetic.

By the way , the mesh in dev branch has been changed (about tint) , This branch also need to update.

Lets not put this shader into glData then.

Lets not put this shader into glData then.

I can't understand this sentence. Could you tell me more ?

@ivanpopelyshev , I've remove the shader in glData


adding example to demo the use of 'params' in transaction

this frustrated me for about 3 hours due to lack of documentation. so let's fix it.

Hi. Sorry about the delay.
Thank you for the PR. For legal reasons we need a signed Contributor License Agreement (CLA) before we can merge your code into the project.
You can download the form from our website: https://www.arangodb.com/documents/cla.pdf
Once you've filled out and signed it, please send it to us via e-mail<EMAIL_ADDRESS>or fax: +49-221-2722999-88.

We don't seem to have a CLA from you on file. Could you please follow the above instructions? We can't merge the PR without a signed CLA, sadly.

sorry but just saw the email. in the middle of something. will sign ASAP.

@pluma just sent the signed CLA.

@pluma CLA received, @ctmakro thanks a lot


Animation fails to repeat when used in AnimationTreePlayer

Godot version:
master / 68f277477b5a5f3003405e36c6b6853720cb91de
OS/device including version:
Manjaro Linux 17.1
Issue description:
When I export animations from Blender and activate the repeat option for each of them from Godot, I can see them looped correctly from the animation panel.
But when I use them from AnimationTreePlayer they just play once and stop. Strangely, if I duplicate such an animation from the animation panel and replace the original animation with it, it repeats correctly even when played by AnimationTreePlayer.
I used the latest version of Better Collada addon to export my animations. I'm not too sure if it's a bug of the engine or the addon, or maybe there's something I'm completely missing in the process.
(If you need more information to investigate the problem, please let me know how I can help. Thanks!)

After reimporting and recreating everything, I can no longer reproduce the problem. I'll close this issue and will reopen it if it ever happens again.

Not sure if I'm shouting into the void commenting on an archived issue, but I'm also seeing this exact same behaviour with v3.0.2 (Steam).
I also note that the animations loop correctly when manipulating the AnimationTree in the editor. It's only when the blend amounts are changed in code, that the triggered animations don't loop.
For me, creating a new inherited scene and recreating my animation changes didn't make any difference. However, the workaround to duplicate the animations did work.
I tried saving both the original and duplicated animations as .tres files, but there were no differences.

Hi. I had the same problem with animationTree in Godot 3.1 or animationTreePlayer in Godot 3.06. I think I found the problem and a not too elegant solution.
Here is the explanation I copied from https://godotengine.org/qa/41611/looping-animations-with-animationtree-animationtreeplayer :

In some objects exported from blender, when you create a "New Inherited" scene, the root object is created as "Scene Root" and the rest as "Inherit Scenes" (you can see this because the first object in the scene tree has "white" letters and the rest has dark gray letters).

After all this, your secondary objects are no longer inheritance and the "animationtreeplayer" and "animationtree" works!!
I don't know if there is an easier option to do this, It would be nice if there is.

Hi. I had the same problem with animationTree in Godot 3.1 or animationTreePlayer in Godot 3.06. I think I found the problem and a not too elegant solution.
Here is the explanation I copied from https://godotengine.org/qa/41611/looping-animations-with-animationtree-animationtreeplayer :

In some objects exported from blender, when you create a "New Inherited" scene, the root object is created as "Scene Root" and the rest as "Inherit Scenes" (you can see this because the first object in the scene tree has "white" letters and the rest has dark gray letters).       After all this, your secondary objects are no longer inheritance and the "animationtreeplayer" and "animationtree" works!!   I don't know if there is an easier option to do this, It would be nice if there is.

+1 - Found the same issue today and the linked post describes the likely problem and a solution perfectly.

I am experiencing the issue as well when attempting to loop an animation. Hopefully, this can be fixed soon.

This still happens in Godot 3.2.2

@Griefchief

Hi. I had the same problem with animationTree in Godot 3.1 or animationTreePlayer in Godot 3.06. I think I found the problem and a not too elegant solution.
Here is the explanation I copied from https://godotengine.org/qa/41611/looping-animations-with-animationtree-animationtreeplayer :

In some objects exported from blender, when you create a "New Inherited" scene, the root object is created as "Scene Root" and the rest as "Inherit Scenes" (you can see this because the first object in the scene tree has "white" letters and the rest has dark gray letters).       After all this, your secondary objects are no longer inheritance and the "animationtreeplayer" and "animationtree" works!!   I don't know if there is an easier option to do this, It would be nice if there is.

+1 - Found the same issue today and the linked post describes the likely problem and a solution perfectly.

I am experiencing the issue as well when attempting to loop an animation. I managed to get it looped by adding the suffix "-loop" to the name of the animation, as illustrated below.

Hope this helps

I worked for me too.


Minecraft Launcher Bug

Describe the bug
When pressing the play button on the launcher for java edition Minecraft, the launcher appears to go through the whole progress bar and launch but then crashes and says exited with code 0.
Desktop (please complete the following information):
OS & version [e.g. Windows 10, 1822]: Windows 10
MSFT Build of OpenJDK Version:
openjdk 11.0.12 2021-07-20
OpenJDK Runtime Environment Microsoft-25199 (build 11.0.12+7)
OpenJDK 64-Bit Server VM Microsoft-25199 (build 11.0.12+7, mixed mode)
To Reproduce:
Steps to reproduce the behavior:

Open Minecraft launcher
Go to Minecraft: Java Edition
Click on Play button
Wait for progress bar to load fully
See error

Expected behavior
I expect that the game would run normally as nothing has changed from the last time I played. I was able to play earlier this week just fine and have installed no new programs since.
Screenshots, Logs etc
Additional context
N/A

This may be a case of you needing to have Java 17.0.1 to run Minecraft 1.18


Defining common terms

OPTiMaDe specification has a section for term definitions. Terms defined there are supposed to be used throughout the document to refer to the same concepts. In discussions with @CasperWA (e.g. in #104) the following issues were stressed:

Do we typeset the terms specifically throughout the document to distinguish them (as is done in legal papers by capitalization)? For example, there is a need to distinguish 'Property' (~ an attribute of a Resource object) from 'property' (~ generic word). We already use special style to distinguish 'MUST' from 'must'
Should we repeat term definitions, or try to avoid that?

Reason to repeat:

reduces the need of jumping between sections of the specification document

Reasons not to repeat:

bloat
term definition changes force one to update the definition in many places
menace of out-of-sync definitions

@CasperWA please add more arguments if you feel like.

Thanks for opening this issue @merkys.
I feel strongly for section 6. Entry List being a look-up "encyclopedia", where all fields and properties are definitively defined. Hence, I would want, when going through this section, not to have to scroll back and forth in the document to understand the requirements and description of said fields and properties.
The main issue has been involving the term ID and the field id, since this is a special case, where the value of the field id is the term ID. In this context, the question came up whether to define the requirements of what the value of the field id under section 6. Entry List or under the initial Term Definitions at the top of the document.


Can I upload the html files online within sketch-measure?

Currently, sketch-measure can save sketch aboard into html files, so we can inspect all the design details offline.
But inspecting offline doesn't meet our common use, alternatively, we often need to share the files online with single link.
If it's possible to upload the generated output immediately? Or if can we implement it by ourselves, any guides?

I've got it, thx!


DOMElement  is incomplete

A dom element has sets/slices of attributes not just one. An attribute can have different keys than "id",  "class" and value" (e.g. data-xyz, hidden, label etc..) along with other properties.
I would propose to bring it more in line with x/html to better represent the structure of a html node. We may exclude the relationships (Parent, FirstChild, LastChild, PrevSibling, NextSibling *Node) if  murlokswarm is not interested to serialize the html(though I think it would be quite handy).

type Node struct {
    Parent, FirstChild, LastChild, PrevSibling, NextSibling *Node
    Type      NodeType
    DataAtom  atom.Atom
    Data      string
    Namespace string
    Attr      []Attribute
}

type Attribute struct {
    Namespace, Key, Val string
}

This is a bunch of javascript to handle and it's pretty tricky to serialize dom objects.
I have got some very strange results when i was just trying to JSON.stringify the attributes in order to have a full attribute map.
Right know, I prefer to focus on other things.
But if someone is open to clone this and try to make it, would be glad to review the changes.

Did you consider the Objective-C API instead of JS?
https://developer.apple.com/library/content/documentation/Cocoa/Conceptual/DisplayWebContent/Tasks/DOMObjCBindings.html


Rewrite

Any updates on this?

Hi @simc are there any updates on this ?

Any updates on this @simc ?


Remove dependency on setuptools for docs

This removes the direct import of setuptools_scm from doc/conf.py.
This is the suggested approach in the setuptools_scm project: https://github.com/pypa/setuptools_scm#usage-from-sphinx
This should also get us one step closer to removing the ignore_import flag for mypy

@cidrblock look at https://ansible-navigator--1001.org.readthedocs.build/en/1001/changelog/ — it does not have the tags fetched. IIRC this is fixable via the RTD config and needs to be included.
Also, the tox env needs to be changed to always recreate the tox virtualenv so that locally the version is updated.

What is this about?

Originally, yes, simply avoid a type ignore[import], but it doesn't appear this has ever worked correctly. So now it's a testing ground to see if it can be fixed.

Brilliant! Thanks for taking care of this!

Couldn't have done it w/o your help.


Fix containerd template render

Signed-off-by: x893675<EMAIL_ADDRESS>
What type of PR is this?

/kind bug
What this PR does / why we need it:
containerd sanbox image mirror should same with localRegistry rather than insecureRegistry
Which issue(s) this PR fixes:

Fixes #
Special notes for reviewers:

Does this PR introduced a user-facing change?

None

Additional documentation, usage docs, etc.:

/cc @zhuzhenfan @qinyer

Codecov Report

Merging #58 (59bc509) into master (84faa6f) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #58   +/-   ##
=======================================
  Coverage   11.33%   11.33%
=======================================
  Files          95       95
  Lines       14710    14710
=======================================
  Hits         1667     1667
  Misses      12822    12822
  Partials      221      221


use tini to run every container

fixes #1101
additional context:

this was introduced in docker 1.13 (1/2017), we require docker 17.10
(10/2017), this should not have any issues dependency-wise, as docker-init
is in the docker install from that point in time. unless explicitly removed,
it should be in the dind container we use as well...
the PR that introduced this to docker is
https://github.com/moby/moby/pull/26061 for additional context
it may be wise to put this through some paces, if anybody has any...
interesting... function containers. the tests seem to work fine, however, and
this shouldn't be something users have to think about (?) at all, just
something that we are doing. this isn't the default in docker for
compatibility reasons, which is maybe a yellow flag but I am not sure tbh
i've defaulted this to 'on' with the config bit to turn it off, in case it runs into
issues it should be easy to flip it back off without having to deal with a build, etc.
we need to test on osx and win and we can default it to 'off' and turn it on
for any 'serious' deployments if there are issues to keep the out of the box xp working,
it does seem to work on osx (per linked issue above), but i haven't tested that or win myself.
it seems to add about 50ms to the start time, which seems worth it.

Works fine on mac , LGTM

thanks for review... see if any windows people come out of the woodwork we can flip it relatively easily

Docker4Windows uses basically the same LK  VM images as d4m so I "expect it will be fine"

"expect it will be fine"

famous last words!


Existing API Re-queue Function Does Not Appear to Work

Within the API, we have a requeue mechanism at /api/requeue/send that should send an existing file to a receiver. The invocation should look like this:
curl -X POST "http://localhost:7071/api/requeue/send?reportId=15a67edd-0913-4f1c-81ea-42cc34064826&receiver=de-dph.elr"

Unfortunately this no longer appears to work, and it does not provide a response or an error message indicating why. We will fix it. It is possible that the error is related to issues with authentication when attempting to invoke. If this is the case, it would be better to return an error informing the invoker that they are not authorized to initiate this action instead of the current result which is it returns nothing.
Acceptance Criteria

The endpoint accepts either POST or GET. Currently it only accepts POST but with an empty body. Let's make it more flexible and accept either
The endpoint should return a result in either an error or on success
The endpoint should return 401 if the user is not authorized to conduct this action
Verify with the pipeline team that the requeue actions are tracked in the submission history, and if they are not, coordinate the creation of a new ticket for them to do that. We should be logging these actions so we know when a file is resent.

Goals

Once we have this working more clearly, the intent is to wire this into the web UI so we can invoke it for files that need to be resent. To that end, please coordinate your activities on this ticket with @TomNUSDS

Oh, I forgot to mention. I tested this using a POST API and it seemed to work?
The API returned:
Will resend all (30) items in d05d17f7-acaa-4f0e-bda1-135dbd81d477
d05d17f7-acaa-4f0e-bda1-135dbd81d477 has been queued to resend immediately to ignore.HL7_BATCH_OFFSET_TIME_UTC

I got the params from the task table and found a row that was marked send_error and used the reportId (d05d17f7-acaa-4f0e-bda1-135dbd81d477) and receiver_name (ignore.HL7_BATCH_OFFSET_TIME_UTC) columns.
We still need to sanity check that the resend is actually queuing correctly.

yeah, it works.
Current syntax for calling it as of this writing is:
curl -X POST -H "content-length:0" -H "x-functions-key:" "https://prime.cdc.gov/api/requeue/send?reportId=861f31ec-276c-489c-a733-b55f701bcb4a&receiver=nm-doh.elr"
The secret is for the requeue azure function; NOT for any of the regular submission endpoints.
Because it operates within the existing code, any data re-sent gets correctly linked in the auditing tables.
Things to fix:

move to using Okta auth
get rid of the bizarre content-length header requirement.

Longer term:

add a Resend button on the Receiver's Daily Data of Daily Data Details page.   Would need to hide this button for receivers that have no automated transport defined.


added missing flash import

flash used in lines 65-67 of app.py, but not imported from flask
 if social_id is None:
        flash('Authentication failed.')
        return redirect(url_for('index'))

Thanks!


[release-1.3] Don't set request.host as label for ingress

This is an automated cherry-pick of #18821

@googlebot I consent

@istio-testing: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-new-install-k8s-tests_istio_release-1.3
b49dd1f8bcfb58f3cc7d4f747986ef1dbc028430
link
/test integ-new-install-k8s-tests_istio_release-1.3

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

@rlenglet @brian-avery is it too late for getting this in 1.3.5?

1.3.5 is already built
On Sat, Nov 9, 2019 at 2:48 PM Neeraj Poddar<EMAIL_ADDRESS>wrote:

@rlenglet https://github.com/rlenglet @brian-avery
https://github.com/brian-avery is it too late for getting this in 1.3.5?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/pull/18823?email_source=notifications&email_token=AAEYGXLFQG4MR7JVGPCYGUTQS444RA5CNFSM4JLJW3YKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDUQ6CY#issuecomment-552144651,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEYGXNTFRSXMYZFW43FB43QS444RANCNFSM4JLJW3YA
.


Error when I start agent-loop

Hi
I am able to use Twitter API if I do
agent-action twitter post-tweet TEST1
But when I do
chat or agent-loop I get an error Error in agent loop: Twitter username is required
How can I fix it?

im fixing it right now

@tezer
same here.
Is it fixed? I get: Error in agent loop: User Name ist required
Where do I have to add the user name?


imagesetthickness() in DrawLineModifier has no effect in certain environments

Describe the bug
The width parameter for the drawLine method was implemented in v 3.3.3. It worked correctly upon upgrade. For reasons unknown, it has stopped working and all drawn lines are 1 px, no matter what the width parameter is. This is so effing weird!
Code Example
public function drawLine($fromX, $fromY, $toX, $toY, $color, $width): void
    {
        ray($width)->blue();
        $this->image->drawLine(function ($line) use ($fromX, $fromY, $toX, $toY, $color, $width) {
            $line->from($fromX, $fromY);      // starting point of line
            $line->to($toX, $toY);            // ending point
            $line->color($color);             // color of line
            $line->width($width);             // line width in pixels
        });
    }

Expected behavior
In the above example, $width was set to 4 (and verified by ray()) - image produced has a 1 px line
Images

Environment (please complete the following information):

PHP Version: 8.3
OS: MacOS Sonoma 14.3
Intervention Image Version: 3.3.3
GD or Imagick: GD

That's really strange. Unfortunately, I can't reproduce it right now.
According to a very old comment in the PHP documentation, the function imagesetthickness() does not work if imageantialias() is activated. I can't say whether this is still relevant (my tests show that it works with both options). But maybe you can comment out the antialias line in the vendor code and see if it has an influence.

@olivervogel Thanks for this response. THAT ACTUALLY FIXES THE PROBLEM! Is there a way for me to override this one function? I tried to just override that one class, but it's buried so deep inside Image Intervention that my override was not being recognized. Thank you for your help!

@olivervogel Thanks for this response. THAT ACTUALLY FIXES THE PROBLEM! Is there a way for me to override this one function? I tried to just override that one class, but it's buried so deep inside Image Intervention that my override was not being recognized. Thank you for your help!

On the one hand, this is good news. On the other hand, it is also bad news, as the current implementation does not seem to work on certain systems. What particularly surprises me is that I can't detect any problems and I'm even using the same setup (MacOS, PHP 8.3.2, GD). And it is not really clear what triggers the behavior and how to handle this.
As a quick solution for the moment, I would suggest to use a custom modifier that replicates the function of the DrawLineModifer without antialiasing.
It could look like this:
use Intervention\Image\Interfaces\ModifierInterface;
use Intervention\Image\Interfaces\ImageInterface;

class DrawPixelLine implements ModifierInterface
{
    public function __construct(
        protected int $fromX,
        protected int $fromY,
        protected int $toX,
        protected int $toY,
        protected mixed $color,
        protected int $width
    ) {
    }

    public function apply(ImageInterface $image): ImageInterface
    {
        $lineColor = $image->driver()->colorProcessor($image->colorspace())->colorToNative(
            $image->driver()->handleInput($this->color)
        );

        foreach ($image as $frame) {
            imagealphablending($frame->native(), true);
            imagesetthickness($frame->native(), $this->width);
            imageline(
                $frame->native(),
                $this->fromX,
                $this->fromY,
                $this->toX,
                $this->toY,
                $lineColor
            );
        }

        return $image;
    }
}

I'm sorry that I can't offer you a better solution at the moment.

Here are the specs of my local GD installation for comparison. Maybe there are differences that you can recognize.

@olivervogel wow - thank you for all the great information. After some further research on my end, I am pretty sure this has something to do with my system, not your code. One of my co-workers pulled my code to his machine (actually a windows box running Sail) and everything worked as expected. I've been trying to get Laravel Herd to work successfully so my guess is something in that process caused this issue. I'm going to say thank you (big time!) and close this - I don't think you need to spend any more time on it unless someone else encounters the same thing.


add type, isBasedOn, variable and numberOfOptions

What
Added new Cantabular model fields to dataset client to match dp-dataset-api models
How to review
Make sure code makes sense, tests pass.
Can check relative values in dp-dataset-api/models/ (type Instance and type Dimension) to see they match in type and JSON tags
Who can review
Anyone

Code looks good to me but I do have a general question:
You may have already considered this and have a plan but Ill ask anyway - have we considered how backward compatibility will work for existing datasets / instances?
If we are adding new fields to these models will that affect existing data stored in Mongo and the graph dbs? Do we need to update existing data or use a default for these new fields?

The omitempty field keeps these changes backwards compatible. The related fields in Mongo also have the omitempty bson tag, so existing data should pass through unaffected. The tests in dp-dataset-api confirm this. Additionally these changes are to the v2 of dp-api-clients go which is still in beta, but the directly affecting changes have already been made in dp-dataset-api few weeks ago.


PLAT-9 Rid ansible python legacy deps. Remove redundant playbook 01

This changes the installer to abandon Ansible requirement on legacy openstack-sdk (for image upload only) by moving the image upload to terraform instead. Also the old 01 playbook is not relevant anymore, so it was deleted and the the other ones re-numbered: 02->01, 03->02, 04->03 .
Also removed the dependency on boto3 aws library by using "aws (cli)" commands from playbook rather than the aws module because of strange behavior of certficate validation. We anyway need to use aws command for presigning the boot ignition file so we might a well use the aws command for creating the bucket and uploading the ignition file to it.
PS: Installation of Openshift enterprise will likely break with this change. This will be addressed later.
PS2: Docs needs updating to reflect new reality.

LGTM!!


[AGE-160] Hide cost and usage in playground and evaluation when these are not specified

Context:
Some users are confused that mistral/mistral-medium for instance is showing cost 0. They ask: "if I use model "mistral/mistral-medium" in the Playground, I get $0.0 cost for each prompt. Does this mean it would be zero cost?"
Solutions
When the cost is either unknown or not specified, we should display a "-" or something else that clearly indicates that the cost is unknown, not free. Whenever possible, we should also add the cost. We should explore libraries that can calculate this cost.
From SyncLinear.com | AGE-160

@mmabrouk I think if it's 0 we should display 0 not -
I also think this should be covered in backend in case there is no cost at all we return null not 0.
also we need to consider displaying 0.000001$.

@aakrem
Yes, if it is a 0 then we should display a 0, I totally agree. We just need to make sure that we have some kind of undefined value when we don't have that value (cc @aybruhm how is it done now?)

@aakrem
Yes, if it is a 0 then we should display a 0, I totally agree. We just need to make sure that we have some kind of undefined value when we don't have that value (cc @aybruhm how is it done now?)

Same way you just described. If the cost is not passed as a response by the user (just the way we have it in our example apps), it defaults to None. Otherwise, it uses whatever the cost is.

@mmabrouk what about 0.000001$ I think it makes sense to display it as it is tbh.
Otherwise we are giving the idea that it's "free".


Inline: Issue with git commit message

please complete the following information:

OS version: Win10
Editor: VSCode
Editor version: 1.60.2
Programming language: (Git commit message)
TabNine extension version:3.4.27
Engine version (could be located within Tabnine Hub): 3.7.5-40

Issue Details:
Inline completion appears in editor while I'm typing git commit message.
https://user-images.githubusercontent.com/59691627/134654866-f9c456cc-40c7-45ff-a18c-b5bdcf46526f.mp4
gz#6872

Hi @sevenc-nanashi
Thanks for submitting the issue! https://github.com/codota/tabnine-vscode/pull/644/files should fix this.
Make sure your plugin is updated to the latest version - 3.4.29.
Let us know should you have any other issues!


modifications for api gw client

Modifications:

Use configured inline model resolver instead of new instance
Extend form-param check to check respective content-type headers of request bodies. The Swagger Coverter packages form-params into request bodies having respective content-type params (multipart/form-param,  application/x-www-form-urlencoded) removing them as parameter objects ( see class SwaggerConverter#convertFormDataToRequestBody (line 672), openapi-generator 7.7.0)
One parameter name of code gen config was updated - the code was adapted accordingly.

I am using the google codestyle plugin (google-java-format Settings) in intellij which modified imports.


Document the AWS_DEFAULT_REGION trick

Hi,
Thanks for this great tool. We adopted entirely in our team as soon as we found a way to point ssm-diff to a specific region. As you may know, you have to run:
AWS_DEFAULT_REGION=<your region> ssm-diff ...
I think this should be in the README, do you agree?

Hi Sylvain,
Glad that it works for you, I find the tool surprisingly convenient as well, despite its relative simplicity.
A couple of enhancements were introduced, and known bugs were fixed in a latest merged pull, thanks to @AndrewChubatiuk; will release a new version in a while, but you might want to give em a try if you spotted any problems.
Added an extra bit of info on how AWS SDK (thus, ssm-diff) figures out its configuration to README.


V2 remove fpe from anonymizer

Please notice, this branch is over the v2_anonymizer_improvements branch so the loop change is included here as well.
Remove FPE from anonymizer since we currently do not support it.

@shiran - can u do this change over the main branch, so it will not include other changes? as we want to introduce this change now


Added StatefulIterators_ module with parameterized type

Instead of having two or more leaf types, there is now only one parameterized type.
Performance is much better than the current IterIterable type. For the ArrayIterable it is mixed. When ArrayIterable uses pure indexing it is a few times faster, but when using the iteration protocol it is >10x slower then the proposed type.

i haven't forgotten this, but work has been hell - will likely look this weekend.  thanks.


fix https://github.com/storybooks/storybook/issues/2367

Issue:
This story has tests configured, but no file was found
What I did
fixing bug This story has tests configured, but no file was found when add-on can't load results from jest mentioned in https://github.com/storybooks/storybook/issues/2367
How to test
check @storybook/addon-jest with recent version of jest
Is this testable with Jest or Chromatic screenshots?
no
Does this need a new example in the kitchen sink apps?
no
Does this need an update to the documentation?
no
If your answer is yes to any of these, please make sure to include it in your PR.
For maintainers only: Please tag your pull request with at least one of the following:
["cleanup", "BREAKING CHANGE", "feature request", "bug", "documentation", "maintenance", "dependencies", "other"]

hi @andreyantipov.
please see this comment on the same thread: https://github.com/storybooks/storybook/issues/2367#issuecomment-348253768
the correct usage is withTests({ results })
Hope that helps!


Remove page reload in pwa.js.

/CC @Lyricalz: I think this should be fine, i.e. leaving it up to the browser to do its job.
Fixes #24868

That will work yeah, but do you still want me to write a "proper" solution?

If this solution works out of the box, without any side-effects, the only thing I'd like it's just a console.log. Feel free to improve it later when you have some time :)

You'd like a console.log or you'd prefer to remove it?

Now that I think about it lets keep it simple as long as it works without
any side effects.
Feel free to improve the whole experience and ping me :)
Thanks!
On Dec 1, 2017 18:20, "Michael Olukoya"<EMAIL_ADDRESS>wrote:

You'd like a console.log or you'd prefer to remove it?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/twbs/bootstrap/pull/24923#issuecomment-348537684, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAVVtd53_87vJEgjujNKMW6A9regwI9Gks5s8CcsgaJpZM4Qw4Gi
.

@mdo @Johann-S please approve


One Half Light man page highlighting issue

Environment
Windows Terminal version (if applicable): 10.0.18362.356

Steps to reproduce
Execute man page command like:
user@xps:~$ man ls

Actual behavior
Some text in man page are invisible.

Trying to search -a in the man page:

The default One Half Light scheme:
{
    "name": "One Half Light",
    "foreground": "#383A42",
    "background": "#FAFAFA",
    "black": "#383A42",
    "red": "#E45649",
    "green": "#50A14F",
    "yellow": "#C18301",
    "blue": "#0184BC",
    "purple": "#A626A4",
    "cyan": "#0997B3",
    "white": "#FAFAFA",
    "brightBlack": "#4F525D",
    "brightRed": "#DF6C75",
    "brightGreen": "#98C379",
    "brightYellow": "#E4C07A",
    "brightBlue": "#61AFEF",
    "brightPurple": "#C577DD",
    "brightCyan": "#56B5C1",
    "brightWhite": "#FFFFFF"
}

Similar problem happen in less page. Searched texts are not highlighted in One Half Light scheme.

/dup #2864


No openaiapirc file being created with bash_setup.sh

Hello,
I am trying to install this on a bash terminal on mac using the following fork to run with gpt-3.5-turbo: https://github.com/Lukas-LLS/Codex-CLI
The setup script does not seem to be creating the openaiapirc file in /src, leading to the following error. No errors are thrown on setup. Has anyone else experienced this? Thank you!
Open a new Bash terminal, type '#' followed by
your natural language command and hit Ctrl + G!
***********************************************
(base) ~/Documents/Code Projects/CodexCli/scripts> # what's my ip?
wc: /Users/jordanbowman-davis/Documents/Code: open: No such file or directory
wc: Projects/CodexCli/src/openaiapirc: open: No such file or directory
Codex CLI configuration is missing, try reinstalling```

This problem most likely appears because there is a space in the name of your Code Projects folder, the path handling of the setup script most probably breaks because of that.


Adding more platforms and multiple sonos systems

First off - thanks for the plugin. It’s great to be able to control the Arc and Beam by voice and that you can trigger voice enhancement and night mode.
Unfortunately every time I need to restart my homebridge the plugin connects with other speakers of mine (one, sub, play 5). I need to do several restarts to connect it with the Arc again. Is there anyway that you could have it only recognize and Arc or Beam.
Also, any chance you could add multiple platforms for those that have more than one Arc or Beam?
Thanks in advance.

Same issue here.
I need several restarts to select te Arc again. Sometimes I get the sub instead.
For the rest this plugin is exactly what I was looking for!
Keep up the good work!

Hopefully I’ll get a chance to look at this over the next day or two. I’ve only got the Beam as my only Sonos product so hadn’t tested with multiple devices. Shouldn’t be too difficult to fix!

Would it be possible to include the Playbar in the soundbar category? Thanks in advance

@jfkung I've added Playbar to the list of discoverable devices but as I don't have one myself I can't know for 100% that it works.
If it doesn't work for you open an issue and I'll have a look at it at the weekend! This one has been closed for over a month now.

@jfkung I've added Playbar to the list of discoverable devices but as I don't have one myself I can't know for 100% that it works.
If it doesn't work for you open an issue and I'll have a look at it at the weekend! This one has been closed for over a month now.

Looking good! Thank you very much. Updated and other devices were removed and playbar was added from cache


case sensitivity issue in cmakelists.txt

This line in Cmakelists.txt -
file (RENAME "${CMAKE_BINARY_DIR}/cpack/README.md" "${CMAKE_BINARY_DIR}/cpack/Readme.md")

causes problems on recent macOS versions because the RENAME command complains that it is invalid to rename a file such that only case differs.
I thought maybe the rename could be done on the COPY in the previous line, but no.

After all this time, it's not clear to me why we change case here. I'm totally fine with it staying README.md. Can you think of a reason why we should try to do anything more complicated? Was it required to be "Readme.md" for some reason?

It looks historical. Later in the file, a CPACK_RESOURCE file name is set to the non-shouting version of Readme, and the text Readme is not referenced elsewhere in the source tree. So I think if the later reference is changed to should README.md it'll all be cool.

Fix proposed in #1687


Barriers should not show colors of the previously answered question.

Resolved with commit: does not show previous colors in answer options anymore but resets design


Serveless Function crash on Vercel

Describe the bug
When deploying to Vercel, no error occurs on build, but no OG Images are available and I receive a Serveless Function error when I try to access the image:

This is the complete log of the error:
Unhandled Promise Rejection 	{"errorType":"Runtime.UnhandledPromiseRejection","errorMessage":"TypeError: Body is unusable","reason":{"errorType":"TypeError","errorMessage":"Body is unusable","stack":["TypeError: Body is unusable","    at specConsumeBody (node:internal/deps/undici/undici:6539:15)","    at Response.arrayBuffer (node:internal/deps/undici/undici:6434:18)","    at Object.transform (file:///var/task/chunks/rollup/provider.mjs:94:41)","    at async walkSatoriTree (file:///var/task/chunks/rollup/provider.mjs:60:11)","    at async walkSatoriTree (file:///var/task/chunks/rollup/provider.mjs:62:7)","    at async walkSatoriTree (file:///var/task/chunks/rollup/provider.mjs:62:7)","    at async walkSatoriTree (file:///var/task/chunks/rollup/provider.mjs:62:7)","    at async Object.createVNode (file:///var/task/chunks/rollup/provider.mjs:208:5)","    at async Object.createSvg (file:///var/task/chunks/rollup/provider.mjs:220:20)","    at async Object.createPng (file:///var/task/chunks/rollup/provider.mjs:194:17)"]},"promise":{},"stack":["Runtime.UnhandledPromiseRejection: TypeError: Body is unusable","    at process.<anonymous> (file:///var/runtime/index.mjs:1189:17)","    at process.emit (node:events:525:35)","    at emit (node:internal/process/promises:149:20)","    at processPromiseRejections (node:internal/process/promises:283:27)","    at process.processTicksAndRejections (node:internal/process/task_queues:96:32)"]}
Unknown application error occurred
Runtime.Unknown

Maybe I'm doing something wrong, but I can't identify it.
Reproduction
https://github.com/ojvribeiro/vulmix-docs
System / Nuxt Info
Nuxi 3.4.3

RootDir: _omitted_
Nuxt project info:

------------------------------
- Operating System: Windows_NT
- Node Version:     v16.16.0
- Nuxt Version:     3.5.2
- Nitro Version:    2.4.1
- Package Manager<EMAIL_ADDRESS>- Builder:          vite
- User Config:      modules, content, postcss, css, ogImage
- Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>- Build Modules:    -
------------------------------

Hey @ojvribeiro, thanks for the issue.
I've looked into this and pushed up a fix for the bug. I also looked at implementing the custom tailwind config.
At the moment it won't integrate with your current config so you'll need to provide the config like so:
export default {
ogImage; {
  satoriOptions: {
      tailwindConfig: {
        theme: {
          extend: {
            colors: {
              'vx-grey-500': '#181818',
            }
          }
        }
      }
    }
  }
}

Let me know if you have any further issues! The site looks amazing btw

Ok, let's go:

Are you using vercel or vercel edge?

I'm using Vercel.

Do the images work locally for you?

Yes, they do. Even the local playground works fine:

Do you have any more of the stack trace?

The only thing I have is this info from Vercel Logs:

Unhandled Promise Rejection

{
  "errorType": "Runtime.UnhandledPromiseRejection",
  "errorMessage": "TypeError: Body is unusable",
  "reason":
    {
      "errorType": "TypeError",
      "errorMessage": "Body is unusable",
      "stack":
        [
          "TypeError: Body is unusable",
          "    at specConsumeBody (node:internal/deps/undici/undici:6539:15)",
          "    at Response.arrayBuffer (node:internal/deps/undici/undici:6434:18)",
          "    at Object.transform (file:///var/task/chunks/rollup/provider.mjs:94:41)",
          "    at async walkSatoriTree (file:///var/task/chunks/rollup/provider.mjs:60:11)",
          "    at async walkSatoriTree (file:///var/task/chunks/rollup/provider.mjs:62:7)",
          "    at async walkSatoriTree (file:///var/task/chunks/rollup/provider.mjs:62:7)",
          "    at async walkSatoriTree (file:///var/task/chunks/rollup/provider.mjs:62:7)",
          "    at async Object.createVNode (file:///var/task/chunks/rollup/provider.mjs:208:5)",
          "    at async Object.createSvg (file:///var/task/chunks/rollup/provider.mjs:220:20)",
          "    at async Object.createPng (file:///var/task/chunks/rollup/provider.mjs:194:17)",
        ],
    },
  "promise": {},
  "stack":
    [
      "Runtime.UnhandledPromiseRejection: TypeError: Body is unusable",
      "    at process.<anonymous> (file:///var/runtime/index.mjs:1189:17)",
      "    at process.emit (node:events:525:35)",
      "    at emit (node:internal/process/promises:149:20)",
      "    at processPromiseRejections (node:internal/process/promises:283:27)",
      "    at process.processTicksAndRejections (node:internal/process/task_queues:96:32)",
    ],
}

Unknown application error occurred
Runtime.Unknown

You should have more luck with the latest beta version 58: https://vulmix-docs-fork.vercel.app/api/og-image-svg?path=/
Will post some more details and close this in the next couple of days.

It seems to be working now. I noticed that I had to replace Tailwind's h-[...] class with a HTML height attribute in order to keep the logo proportions. But that should be just fine for now.

Will post some more details and close this in the next couple of days.

I'll keep an eye on it. 😄

Thank you very much for your great support on this issue and for this amazing module, Harlan! Have a great one!

The Bug
This was a Vercel-specific bug in their Node 18.x runtime, related to how Nitro mounts the global fetch instance.
For whatever reason it would attempt to read the body twice when accessing .arayBuffer(), instead we have ofetch return the data as an arrayBuffer as part of the request params. (see)
We also provide a requestOrigin instead of relying on Nitro's internal fetch of public files, which is buggy on certain environments.
Additionally, after fixing this it was discovered Satori wouldn't render production images without dimensions. To get around this, automatic image dimensions/aspect ratios were implemented.
Image Sizing
To address the issue you had @ojvribeiro, when you only provide an h-[...], it will compute the width based on image width and height based on tailwind, so it skews it. I have a solution to this but I will come back to it later as it involves some complexity around resolving the tailwind classes up front. (It's mainly an issue in Satori and I'm hoping they solve it on their end soon).
End
Thanks again for the help and best of luck with Vumix!
I'm still quite deep into Laravel so will need to check it out. I actually may have a conference talk around Vue DX in Laravel coming up, so may be interested in contributing to it.


Replay Test: Test support for Record & Replay

Contains
Testing support to Record & Replay through the creation of a new type of test: Replay Tests. It is now possible to write asserts to check game states while a replay is playing. There is also an example test and a template to help in the creation of Replay Tests. For more information, check https://github.com/MovingBlocks/Terasology/wiki/Replay-Tests
How to test
Run one of the tests in ExampleReplayTest or follow the tutorial in the new wiki page about Replay Tests to create your own replay test.
Outstanding before merging

[ ] Since replay tests are "headed" they cannot be run by Jenkins, so it is important that the annotation "@Ignore" is present in every replay test.
[ ] It is known that selecting more than one replay test to run causes issues in the tests that come after the first one, which causes the second test to not run.

Hooray Jenkins reported success with all tests good!

We talked over in the slack and decided that Replay Tests should be moved to a new module: ReplayTestModule.
Therefore, the things that were moved to the new module was deleted here. There are still some small refactors and an addition of an important method in EventSystemReplayImpl!

Would you mind if we close this and restart the PR with just the few final changes? There's also a (minor) conflict now that I merged the refactor PR :-)
Rebase would be an alternative (and you can do that into another branch to squash the reverted commits) but since the nature of the PR changed it seems more proper to just do a new PR and write it up fresh.
The wiki page might also be a bit out of date now vs the module approach?

Closed this PR and opened this one: https://github.com/MovingBlocks/Terasology/pull/3445


Use H64 for Block Nonce

Motivation
geth and other libraries represent the block nonce as a 8 byte fixed array. ethers-rs uses an unsigned 64 bit integer to represent the nonce. This causes the nonce to be serialized without leading 0's as specified by the JSON-RPC spec, which therefore may cause deserialization problems with other libraries that expect a fixed byte array. This is the case with web3 for example which fails to deserialize the nonce because it uses H64
Solution
Use H64 instead of U64 for nonce
PR Checklist

[ ] Added Tests
[ ] Added Documentation
[x] Updated the changelog

Did we encounter this as a bug in someone's code, or is this a proactive change?

I just double checked the RPC docs
for blocks, nonce is indeed DATA whereas only for tx it is QUANTITY...
so @jpopesculian you're totally right, and we should change this to H64,
geth also returns block nonces like that and so does hardhat
 "nonce": "0x0000000000000042",
ref https://github.com/foundry-rs/foundry/issues/2088

@gakonst we can merge this, the failing tests are due to installed Anvil which uses the old nonce format


No module named manim

Hi ! So, I tried to install manim but ran into an issue: I have successfully installed pip, cairo, ffmpeg and sox (and updated latex), but when I try to run a file, I get /usr/bin/python: No module named manim so I guess manim didn't install where it should have (and with further investigation, it looks like it didn't install at all). So, I'm kinda lost, I have tried many things, but I'm a beginner so I don't know what I should be looking for. I've read that the installation instructions are outdated, so I've tried installing some other ways around but still nothing. As I've said, I'm lost, and don't really even what to say to get some help.
I hope I've been clear enough, thanks in advance !
PS: I've also tried with python3 and python3.7 in the command line, same problem.
EDIT: Forgot to precise I'm on Ubuntu !

I have the same problem on macOS. I am still trying various things but suspect there may be a regression that was introduced recently.

@Planeuge Somebody on the Discord server suggested you need to cd into the manim folder if you are running it as a Python module. You’ll have to find that folder, of course.

I've (finally) found that folder where manim is installed, but I guess I now have to indicate it, because it's not where it's supposed to be installed ? And if I'm right, how do I do that ?
I've also tried (as suggested) to cd into that folder and running python3 -m manim filename.py -pl (the file I tested was an example file, I just can't remember the name) but I still get the same error message, so maybe I just didn't understand what you/they meant ?

I really don’t know exactly what to do, as I do not have Ubuntu myself. Maybe you could join the Discord server and ask people in the manim-help channels?

Well, it was my "rescue plan", I wasn't sure I could manage to install manim with the cloning method, but I gave it a try after uninstalling my first installation, and it worked like a charm ! I spent more time trying to make the first method work than just using the second method, but I guess I got some experience. Thanks anyway !

Get an example file, like this one: https://github.com/zimmermant/manim_tutorial/blob/master/manim_tutorial_P37.py
I tried using "python3 -m manim pymanim_tutorial_P37.py Shapes -pl" and got the "No module named manim" error
Next I ran just "manim pymanim_tutorial_P37.py Shapes -pl" and it worked fine.
I guess if you install via pip3 you don't need the "python3 -m" before it (compared to cloning the project)?
Running in Manjaro with kde.

I am running into this issue at the moment. Stupid question but does the venv folder should be placed in the manim folder or outside in a general folder (let's say animation)?

I have faced similar issues and this worked for me
cd manim
python -m pip install -r requirements.txt manim
Hope it helps

This is happening because in the latest updates the python file 'manim' is in some other subfolder.
Try using the following code instead:
python -m manimlib example_scenes.py


erlcloud_s3 module now uses signv4

Problem

Amazon s3 uses some different approach to sign their requests

Solution

Amended to work with Amazon s3

this is part and preparation for #215

I think it is better to move  s3-related changes into separated s3-related PR.
I've created  https://github.com/gleber/erlcloud/pull/219 removing them and also other things you've commented here.
@lilrooness please close this PR


Add HS256 algorithm to decode default options

Things Done

Added algorithm: 'HS256' to the JWT.decode default options list
Added spec tests

Addresses Issue: #227

As mentioned in #227. This is a documentation issue. This beahvior is intended as of #184.


Improve docs regarding manifest usage

Document SPRING_CLOUD_CONFIG_ENABLED=false to suppress WARN messages when
you don't want to use the config server

Fixes #326
Fixes #307

Edited and merged as 0968e35


Upgrade to bootstrap 5.1.1

Should we need to upgrade to bootstrap 5.1.1 ?

Good suggestion @Unviray  ,
If you can please go ahead .


DBeaver Voluntary Product Accessibility Template (VPAT)

US Government Agencies are requesting that software and tools, like DBeaver, complete and publish a Voluntary Product Accessibility Template (VPAT). The presence of a VPAT for a tool, like DBeaver, makes it easier for the tool to be approved for use and distributed internally.
More info for regarding the Voluntary Product Accessibility Template (VPAT) here: https://www.section508.gov/sell/vpat/
My ask/idea is that the DBeaver community produce a Voluntary Product Accessibility Template (VPAT) and publish it!

Thanks for the suggestion.
It has already been done for Enterprise products. And it is not planned for the Community edition.


Support match applications options beside name

Is your feature request related to a problem? Please describe.
Currently image-updater seems to come with option to filter or match applications by name e.g --match-application-name staging-*. It's common to use k8s labels to filter applications which is currently not supported
Describe the solution you'd like
It would be great if image-updater can match applications by labels
Describe alternatives you've considered
We considered using ignore-tags option but it's not suitable in this use case.
Additional context
Add any other context or screenshots about the feature request here.

Oh, this issue must have been slipped through, sorry.
I agree, this is a great idea to filter applications by labels and would be nice to have.

Hey @jannfis could you assign this to me?
Thanks

Thanks @jaideepr97 - let me know any questions you may have during implementation :+1:

Implemented by #368


Issue 60

documentation fix for #60

Coverage remained the same at 90.862% when pulling 38782a3b4b0792408c2ffda7663a15705118a18b on tsaiyl:issue-60 into 42f91f603864c7dd8988cd7b02cfdf00d937c439 on IBM-Bluemix:master.

Coverage remained the same at 90.862% when pulling f295514eae56531e2d2510f2e7a2a211a538b614 on tsaiyl:issue-60 into 42f91f603864c7dd8988cd7b02cfdf00d937c439 on IBM-Bluemix:master.

Coverage remained the same at 90.862% when pulling f295514eae56531e2d2510f2e7a2a211a538b614 on tsaiyl:issue-60 into 42f91f603864c7dd8988cd7b02cfdf00d937c439 on IBM-Bluemix:master.

Coverage remained the same at 90.862% when pulling f295514eae56531e2d2510f2e7a2a211a538b614 on tsaiyl:issue-60 into 42f91f603864c7dd8988cd7b02cfdf00d937c439 on IBM-Bluemix:master.

thanks @tsaiyl


🐛 bug fix:fix issue in entitypage tabs with replacing nav link tabs

Hey, I just made a Pull Request!
fix #19912

:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[x] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

prev

after

just to get the maintainers' view
clean up required this is not the final commit.

@tudi2d thought on changes? help with tests?

@npiyush97 can you add a test for this please? :pray:

👉 @benjdlambert


fixed "add to album" error on uploads

Fixed the way error when adding album is being handled.

I used "file(s)" since technically speaking you can upload more than one file at a time from the API (right?) :thinking:
Btw, previously whenever a file could not be added to album, the error would never be shown to user, and it would throw an error in terminal about being unable to set headers and whatnots (https://github.com/WeebDev/lolisafe/issues/91).

I just did some review, and I realized that this part isn't where it's supposed to "add" the files to album, it's only the part where it updates the "editedAt" property, so the warning message being sent to user is wrong.
I'll push another commit to restore the previous error message tomorrow.