TopicSender unable to recover after timeout

Package Name:
azure-servicebus
Package Version:
0.50.2
Operating System:
Experienced in Docker image python:3.8-slim-buster as well as locally on macOS 10.14.6
Python Version:
3.8

Describe the bug
Trying to send a message after the 240 s of inactivity results in various errors.
To Reproduce
Steps to reproduce the behavior:

Get a sender sender = TopicClient.from_connection_string(connection_string).get_sender()
Send a message (msg1)
Wait 241 s
Send a message (msg2)

Observed behavior
The send operation errors with
ErrorCodes.ConnectionCloseForced: The connection was inactive for more than the allowed 240000 milliseconds and is closed by container

Usually, sender.send() of subsequent messages does succeed, however see Additional context
Expected behavior
Sender reconnects after experiencing the timed out connection, and sends msg2 transparently.
Additional context
The application is a small python program running in a container.
It reads messages from a subscription. Reads some data. Sends a message to another topic.
Both receiver and sender are instantiated on startup and not manually (by us) replaced.
Messages sometimes come in more than 240s apart.
We are not seeing any warnings or errors from the receiver.
From looking through logs, I see that the sender usually recovers after failing the current message, and sends subsequent messages successfully.
I'm am very unsure whether the following is in fact related or another issue, but I include it just in case:
Occasionally the sender enters an untenable state, where all subsequent send attempts fail with
Handler failed: The supplied authentication has already been consumed by another connection. Please create a fresh instance.

This first happened after a few weeks of running continuously, with only occasional errors of the previously mentioned type.
More specifically this started on Jul 14 around 8.00 (Copenhagen time).
The problem went away after restarting the container, but reappeared on Jul 18 around 10.04. This time the error train looks to have begun with the aforementioned timeout error.
Workarounds tried
Catching the error, I have tried both calling sender.reconnect() and then re-sending the message and creating a new sender and resending the message. Both attempts result in errors stating that the message was already settled.
My current – so far seemingly working – workaround is to cache the last timestamp of activity and recreate the sender before sending a message if the last activity was more than 210s ago (240s - 30s margin).

Hey adamal;
Thanks for reaching out, sorry you've run into this trouble.
The 4 minute timeout is a known behavior, this is a service-side timeout to terminate inactive links.  This was also ostensibly something addressed in version 0.50.3, the final pre-release version of Servicebus. (Our GA release currently is in beta, as a note.)  I would first encourage updating to that (pip install azure-servicebus==0.50.3 --upgrade) and seeing if the problem is addressed, as we ostensibly support this scenario (see this test here.)  Do let me know if this doesn't mitigate the issue.
The second error you observed: "Handler failed: The supplied authentication has already been consumed by another connection. Please create a fresh instance." is not familiar to me offhand, will do some digging on what it might imply and get back to you here if I find anything suspicious, but I'd be interested to see if the fix above improves your situation altogether.
Profuse thanks as well for the very detailed issue report!

Thanks. I'll keep that in mind. We haven't experienced issues since implementing the workaround mentioned above. I think we'll wait with the upgrade until the next version comes out of pre-release.

Closing as mitigated; feel free to reopen if there are any further issues or questions on this that we can assist with.


Add VS Code suggested extensions

We should test the new Angular ones, especially Nx Console, in the lab once some of the other changes are in.

I don't know about including Nx Console. It is pretty nice and gives a GUI for a lot of Angular CLI stuff but it might be more unneeded complexity.
@NicMcPhee @kklamberty thoughts?

I'm not sure about the spell checker. Does it think that camel case words are not spelled right? @NicMcPhee and I noticed it caring about things that were not helpful. I think it was that extension - it was one about spelling, and it was not ignoring things we really thought it should iirc.

The spell checker seems to be pretty smart about stuff like camel case. Here is an example I just noticed:

It has dictionaries and specific support for various languages like Java and Typescript (there are more you can turn on in its settings, like Mongo and Gradle).

Also, for certain specific words we are using a lot that are not real words like todos we can add them to a dictionary for the project so they are not shown as misspelled automatically. When you go to fix a word you can select "add to folder dictionary" and it adds it to this section in .vscode/settings.json:
{
    "cSpell.words": [
        "todos"
    ]
}

As @floogulinc and I discussed earlier today in person, @NicMcPhee and I had discussed maybe using a TS Linter. If the spelling is smart on most things and can be trained, I think it would be a great idea.


Move provisioning plugins into VTS

At present, pieces of the TCB associated with the ingestion of endorsements and reference values leak into the provisioning frontend.
To improve cohesiveness of the Veraison TCB we need to move the provisioning plugins into VTS.

Started investigating this work item!


[PE-D][Tester C] Explanation does not match purpose of diagram

Explanation says that it displays an activity diagram that explains roughly what happens when a user tries to add an application but the diagram is for updating application. Also there is a spelling mistake in application.

Labels: severity.Medium type.DocumentationBug
original: Soumyadip-cmd/ped#3

Resolved by #226

@AY2425S1-CS2103T-T09-1/developers fix typo, tries to add -> tries to update


Update server.js

Moved WebSocket emitter require to top of file
Added PORT constant instead of hard coded 3020
const koa to var koa for consistency

@hedgerh Thank you 🙂


Raise focus exit event for last pointer leaving when two or more leave in the same frame

Modified FocusProvider.UnregisterPointer() to keep track of the number of pointers leaving a target in a given frame and raise the pointer left event for the last one, instead of the first.
Fixes #4160

@Alexees , same story, do you mind having a quick look at this PR and at the corresponding issue?


code error

  File "/data1/gaoziqi/anaconda3/lib/python3.8/site-packages/torch/multiprocessing/spawn.py", line 59, in _wrap
    fn(i, *args)
  File "/data1/gaoziqi/PSF-main/train_distill.py", line 402, in train
    x_gen_list = model.gen_sample_traj(new_x_chain(x, 1).shape, x.device, freq=40, clip_denoised=False)
  File "/data1/gaoziqi/PSF-main/train_distill.py", line 171, in gen_sample_traj
    return self.flow.p_sample_loop_trajectory(self._denoise, shape=shape, device=device, noise_fn=noise_fn, freq=freq,
  File "/data1/gaoziqi/PSF-main/train_distill.py", line 69, in p_sample_loop_trajectory
    total_steps =  self.num_timesteps if not keep_running else len(self.betas)
AttributeError: 'Flowmodel' object has no attribute 'num_timesteps'

num_timesteps not define in the class, please check the code

Fixed, thanks for pointing this out!


Release 7.0.0

Potential open PRs to add:

[x] https://github.com/theforeman/puppet-dhcp/pull/193

Related PR merged, please update.


v7.0.0

i updated the references but will need u to help fix positioning of elements

@Moore2021 which version of discord.js is required for this PR? I tried installing v12.2.0 and apparently all commands that using Pistachio.reply() cannot be used, and it gives me error log. Checkout the screenshot.

The node packages all need updates, discord.js will be 12.0.2, snekfetch is node-fetch, canvas-constructor is 4.2. All of these you have to make sure are correct one thing u might have todo is nuke your node modules folder and reinstall with "npm i -d -m"

This version requires version of Node.js to be at least v12.18.3


LG Ultrawide Volume Control

Before opening the issue, have you...?

[X] Searched for existing issues
[X] Read through the Monitor-Troubleshooting Wiki
[X] Updated MonitorControl to the latest version (if applicable)

Describe the issue
Hi there,
I'm having a few issues with the volume control on my LG 34WN80C-B UltraWide Monitor. Using the Mac keyboards to turn the sound up & down does work but it brings up the LG volume indicator overlay on the screen which is a bit in the way & not as sleek as the Mac volume indicator. While the Mac volume indicator still shows up, it is hidden b

ehind the LG volume indicator. Photo attached.
The other issue is that when I mute the system audio (via the Mac keyboard button) it does mute however, when I unmute, the LG monitor stays muted. The only want to unmute is to toggle the nob below the monitor.
Otherwise, this app is awesome & does a great job.
Expected behavior
I'd love for the volume indicator on the Mac to only show up & the LG volume indicator remain inactive. I've tried SoundSource & it works in that regard. Also MonitorControl does work perfectly when adjusting the brightness (no LG monitor overlays).
I'm also hoping muting & unmuting would work as expected without having to toggle LG monitor settings.
Anything else?
No response
Environment Information (please complete the following information)
- macOS version: 11.6.1
- Mac model: 15-inch, 2019
- MonitorControl version: 4.0.2
- Monitor(s): LG 34WN80C-B UltraWide Monitor
- Monitor Cable(s)/Connection(s): USB-C
- Apple Silicon/M1 (yes or no): no

Hi @Duskcyclist!

Using the Mac keyboards to turn the sound up & down does work but it brings up the LG volume indicator overlay on the screen which is a bit in the way & not as sleek as the Mac volume indicator. While the Mac volume indicator still shows up, it is hidden behind the LG volume indicator.

This is expected behaviour. The difference between things like SoundSource is that MonitorControl controls the display itself instead of using a virtual audio channel. We used to have a feature that spammed a certain command that removed the LG OSD (aka overlay), but it didn't work very reliably, so it got removed. There's not much more we can do there.

Also MonitorControl does work perfectly when adjusting the brightness (no LG monitor overlays).

This is LG specific. I have almost the same model as you do. There is no LG OSD for brightness like there is for volume. The reason there's an OSD for volume is because you can use the knob to adjust volume, it's displaying that OSD.

The other issue is that when I mute the system audio (via the Mac keyboard button) it does mute however, when I unmute, the LG monitor stays muted. The only want to unmute is to toggle the nob below the monitor.

Can you try:

Preferences > Displays > Toggle Show advanced settings at the bottom
Toggle Enable Mute DDC command for the display.

Thank you so much for your speedy response! I really appreciate it. I can certainly live with the LG overlay. All good. But that mute toggle suggestion did fix it! Much better. Sorry I didn't see that option to show advanced settings.
Really appreciate your assistance & great app! Have a wonderful weekend.


GL crash

Describe the Bug
Game crashed randomly
Reproduction Steps
It was a random crash
Expected Result
Not to crash
Screenshots and Videos
No response
Crash Report or Log
https://pste.ch/ojuxifopuy.pl
Operating System
Windows 11
What is your GPU?
RTX 3080ti
Mod Version
0.6.0
Minecraft Version
1.18.1
Forge Version
39.0.63
Other Mods
No response
Additional Context
No response

This looks like a duplicate of #56. Please upload the full crash log or update flywheel

FlyingWheel is fully updated

Crash just happened again placing a Mill
https://pste.ch/uqacekaqeq.sql

Again when placing Mill
https://pste.ch/zuwoyibuda.sql

https://pste.ch/basahariga.makefile


feat(client): ImageLayer: derive Debug, Eq, PartialEq; impl Hash trait

Update ImageLayer to:

derive Debug, Eq, PartialEq traits
implement Hash trait

The latter was motivated by the need to deduplicate collections of ImageLayers.
Note: I created this PR against the branch that we currently use in Spin.  Would also be happy to PR upstream (krustlet/oci-distribution) instead or as a follow-up.

Closing in favor of upstream PR; once merged upstream, will update the branch we use in Spin.


Allow CLI arguments to override config file settings

Is your feature request related to a problem? Please describe.
If you specify options other than output formats in the CLI and provide a config file, the CLI options are ignored.
Describe the solution you'd like
It would be useful if the CLI options would instead override the corresponding settings in the config file. Since the CLI only has a single Stage focus, the options passed from the CLI must act like globals, and override the corresponding config file settings for EVERY Stage.

This feature is already in the code base, but untested and undocumented.
Depends on #26.

This feature causes problems when a default config file is present in the current working directory. Might revisit this idea in the future after a CLI refactor but closing for now.


why not just inject IDbConnection instead of ISqlConnectionFactory?

Very great DDD sample.
I have one question confuse me.
why not use IDbConnection  directly but use ISqlConnectionFactory to wrap a connection?
Any concern about use IDbConnection  ?
https://github.com/kgrzybek/sample-dotnet-core-cqrs-api/blob/master/src/SampleProject.API/Customers/DomainServices/CustomerUniquenessChecker.cs#L9
Thank you.

The main reason is to use the same connection during "request" handling. It is solved using IoC container:
builder.RegisterType<SqlConnectionFactory>()
    .As<ISqlConnectionFactory>()
    .WithParameter("connectionString", _databaseConnectionString)
    .InstancePerLifetimeScope();

Request means HTTP request or internal command processing.

The main reason is to use the same connection during "request" handling. It is solved using IoC container:
builder.RegisterType<SqlConnectionFactory>()
    .As<ISqlConnectionFactory>()
    .WithParameter("connectionString", _databaseConnectionString)
    .InstancePerLifetimeScope();

Request means HTTP request or internal command processing.

Thanks for your reply. I think the same connection determined by InstancePerLifetimeScope,
the IDbConnection can be registered the same way.
builder.RegisterType()
.As()
.InstancePerLifetimeScope();

You mean this?
builder.RegisterType<SqlConnection>()
.As<IDbConnection>()
.WithParameter("connectionString", _databaseConnectionString)
.InstancePerLifetimeScope();

This has some issues:

When the connection is opened?
You need to explicitly close connection according to https://docs.microsoft.com/en-us/dotnet/api/system.data.sqlclient.sqlconnection.open?view=netframework-4.8 :

If the SqlConnection goes out of scope, it is not closed. Therefore, you must explicitly close the connection by calling Close.

When the connection is closed, should be reopened if necessary
In factory you can configure other options like timeout etc.

You mean this?
builder.RegisterType<SqlConnection>()
.As<IDbConnection>()
.WithParameter("connectionString", _databaseConnectionString)
.InstancePerLifetimeScope();

This has some issues:

When the connection is opened?
You need to explicitly close connection according to https://docs.microsoft.com/en-us/dotnet/api/system.data.sqlclient.sqlconnection.open?view=netframework-4.8 :

If the SqlConnection goes out of scope, it is not closed. Therefore, you must explicitly close the connection by calling Close.

When the connection is closed, should be reopened if necessary
In factory you can configure other options like timeout etc.

Thanks for your explanation!


Add RainMaker WiFI provisioning

Following examples of https://github.com/espressif/esp-rainmaker add WiFi provisioning in one branch.
It would be the Espressif alternative to send your WiFi credentials to your devices using Android and Iphone devices.

[x] Add all RainMaker required libraries
[x] Do a vector _buffer tests ( test/vector )
[ ] Add some documentation

Note provisioning starts to work. But there is still an error in the provisioning process:
I (31038) app_wifi: Connected with IP Address:<IP_ADDRESS>
I (31038) esp_netif_handlers: sta ip: <IP_ADDRESS>, mask: <IP_ADDRESS>, gw: <IP_ADDRESS>
I (31048) wifi_prov_mgr: STA Got IP
I (31048) app_wifi: Provisioning successful
I (31058) esp_mqtt_glue: Connecting to a1p72mufdu6064-ats.iot.us-east-1.amazonaws.com
I (31058) esp_mqtt_glue: Waiting for MQTT connection. This may take time.
E (34778) BT_SMP: smp_encrypt_data failed unable to allocate buffer

E (34778) BT_BTC: btc_gatts_cb_param_copy_req 2 no mem

I (40028) app_wifi: Disconnected. Connecting to the AP again...
E (40038) esp-tls-mbedtls: mbedtls_ssl_handshake returned -0x2880
E (40038) wifi_prov_mgr: STA Disconnected
I (40038) esp-tls-mbedtls: Certificate verified.
E (40038) wifi_prov_mgr: Disconnect reason : 200
E (40048) esp-tls: Failed to open new connection
E (40048) wifi:sta is connecting, return error
E (40058) TRANSPORT_BASE: Failed to open a new connection
E (40068) MQTT_CLIENT: Error transport connect
E (40068) esp_mqtt_glue: MQTT_EVENT_ERROR
W (40078) esp_mqtt_glue: MQTT Disconnected. Will try reconnecting in a while...

So it connects to WiFi but it does not get listed in the App as a device.

This was merged into this branch and it will eventually land in refactor/oop or wifi-provisioning after being fully tested.

This tickets needs more testing before closing it. Must say that I already shipped some devices built with this branch and it worked pretty good.
But it needs more documentation and also #50 should be implemented otherwise it works until someone decides to change their WiFi credentials and then it just keeps on hooked to the old WiFi.

Let’s reopen this after Espressif launches Rainmaker as a component


Out of sync sequence number between our node and the public lcd.terra.dev when running estimateFee

Trying to run estimateFee fails when run against our own node with the error message: account sequence mismatch, expected 266, got 265: incorrect account sequence: invalid request but against the public lcd.terra.dev it works with the same sequence number fetched from auth.accountInfo.
The error we are seeing is:
message: 'account sequence mismatch, expected 266, got 265: incorrect account sequence: invalid request'
The code triggering the error is:
const account = lcdClient.auth.accountInfo(sourceAddress);
const signerDataArray = [{
  publicKey: account.getPublicKey(),
  sequenceNumber: account.getSequenceNumber()
}];

const feeResult = await lcdClient.tx.estimateFee(signerDataArray, options);

The sequence number returned from auth.accountInfo is the same from both lcd.terra.dev and our own node but only lcd.terra.dev will estimate fees and not throw that error.
The address is terra195kvgyq0ut99a655jpcgdge220afxmgfscnml6
We are running the following version on our node which is in sync, we recently rebooted from a snapshot after the soft fork and thats when this all started.
https://github.com/terra-money/core/releases/download/v0.5.13/terra_0.5.13_Linux_x86_64.tar.gz

Update:
If I try estimateFee against our node or the public LCD with another address then it estimates fees correctly so I'm not sure why its happening with our wallet address.

I downloaded the latest snapshot and then re-synced our node and this is resolved. Almost seems like a race condition where our node half processed a tx and then was rebooted which resulted in corrupted state. Hard to tell. Closing this for now. Thanks for looking into this.

Hey @corymsmith , I see you've reopened this recently. Are you experiencing this again? We also experience that, but surprisingly, against the public lcd

I experienced this on bombay LCD as well. According to https://github.com/terra-money/terra.js/issues/168#issuecomment-1008597347 @hanjukim it seems that the load balancer may be the cause of the problem.
I also experienced the same issue mentioned by the OP that the problem seems to be associated with some addresses - the problem doesn't occur to a different address. (On the public LCD.)

You need a dedicate RPC node to remove sequence error completely.

I ran into this with a dedicated node which is why I was so confused
On Tue, Apr 5, 2022 at 12:15 AM Paul Kim @.***> wrote:

You need a dedicate RPC node to remove sequence error completely.
—
Reply to this email directly, view it on GitHub
https://github.com/terra-money/terra.js/issues/189#issuecomment-1088250187,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAABZE6FMTON3VNNYCDMQPDVDO46LANCNFSM5KOIGCPQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Thanks,
Cory Smith
http://ca.linkedin.com/in/corysmith/
http:/twitter.com/smixx http://twitter.com/smixx


Move some of the common login functionality to a common location

There was some code duplication between invite-user and the splash
screen, this removes the duplication that I was able to find.

Small comment, otherwise :+1:

Review status: 0 of 4 files reviewed at latest revision, 1 unresolved discussion, some commit checks broke.

src/generic_ui/polymer/invite-user.ts, line 184 [r2] (raw file):
Might be good to revert this to the old way, since nothing has really changed here

Comments from the review on Reviewable.io


sbom: Updates to include builder name and CI link

Fixes: https://github.com/adoptium/temurin-build/issues/3533 https://github.com/adoptium/temurin-build/issues/3534
Draft until it's been tested in a build flow but reviews welcome in the meantime :-)

LGTM:
        {
          "name" : "Temurin Build Ref",
          "value" : "https://github.com/sxa/temurin-build/commit/792d4d99c19663d4a7c065d1951ebe016cad7119"
        },
        {
          "name" : "Builder Job Reference",
          "value" : "https://ci.adoptium.net/job/build-scripts/job/jobs/job/jdk21u/job/jdk21u-linux-aarch64-temurin/29/"
        },
        {
          "name" : "Builder Name",
          "value" : "dockerhost-equinix-ubuntu2204-armv8-1"
        },
        {
          "name" : "Build Timestamp",
          "value" : "2023-11-16 13:17:16"
        },

I'll push a fix for the linter and take it out of draft...


feat: Add data residency for eu and global regions

Fixes DII-1230
This PR:

Adds a setter setDataResidency() which allows the users to switch to EU sub region, reidrecting their requests to api.eu.sendgrid.com
Upgrades the testing pipeline to node v10+ (eslint and typescript do not work well on <10 versions)
Upgrades typescript, which was causing the pipleine failure for very long

Checklist

[x] I acknowledge that all my contributions will be made under the project's license
[x] I have made a material change to the repo (functionality, testing, spelling, grammar)
[x] I have read the Contribution Guidelines and my PR follows them
[x] I have titled the PR appropriately
[x] I have updated my branch with the main branch
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have added the necessary documentation about the functionality in the appropriate .md file
[x] I have added inline documentation to the code I modified

If you have questions, please file a support ticket.

Hi, would it be possible to open a separate PR for the version upgrades? A number of people are awaiting #1387 in order to complete their security upgrades.

I think we need to resolve this usability gap on the setApiKey and setDataResidency before we release.

We have resolved the discrepancy in the latest commits. The limitation no longer exists.


default -march=native build cannot be overridden by flags

I would like to use nmslib in python and when installing nmslib by pip install nmslib, -march=native is passed to gcc https://github.com/nmslib/nmslib/blob/829e830c33d1b40f6e59e0ad0e2f5e80bb62483e/python_bindings/setup.py#L82 and so binaries are created that work only on the same architecture. I would like to use nmslib inside docker and these docker containers would run on different types of machines (different generations of aws ec2 instances), but this building setting will cause Illegal instruction in some cases.
It would be great if C++ build flags could be overridden somehow. I tried with CFLAGS="-march=sse4.2", but these options get before -march=native", so the latter overrides the former.
I created my own fork of nmslib, modified setup.py and this solved the problem, but this clearly isn't the best solution, and maybe others have the same problem when using nmslib with docker and ec2, so I hoped someone here can suggest a better solution.
Thanks

I can open a PR for this, but I can't really design a general solution, i would just append an extra option to the extra_compile_args list between these two lines:
https://github.com/nmslib/nmslib/blob/829e830c33d1b40f6e59e0ad0e2f5e80bb62483e/python_bindings/setup.py#L112-L113
If you don't mind this being too specific, I'll create the PR.

@zseder what do you mean by a general solution?

I really don't know :)
I just feel like this solution, a gcc call wil look like this:
gcc ... -march=native ... -march={user_defined}
and this looks very confusing because one has to know that the latter always overrides the former

@zseder I think this makes a lot of sense if you can just specify a "downgraded" version for CXX flags. This is also a good reminder to still support older architectures: although I am a bit surprised, we don't have AVX on most AWS instances yet.

PS: I can't think of a more general solution and your approach sounds just fine.

I tried to make this work, but this setuptools is way too complicated.
The solution you suggested adds a new argument to install but setup.py is implementing build_ext, not install, so I couldn't make it work, arguments passed to install (pip install ... --extra-arg) could not be obtained in build_ext.
After this, I tried https://stackoverflow.com/questions/15725869/pass-build-ext-options-to-pip-install
But with this solution, build_ext arguments got overridden, and all the other dependencies got the argument (like numpy), and they complained they don't understand this extra arg for build_ext.
I'm afraid I'll have to give up here, i have no idea about the whole setuptools and I just don't know what I am doing...

@zseder aaargh, ok. In fact, there seems to be a bug in setuptools.py that we don't have a workaround for yet. setuptools is difficult.

@DemonTPx no, it's not too risky. It's just some work needs to be done by processing extra flags.
For a quick fix: what prevents you from manually changing -march=native in the setup.py script? It's only an issue if you install it through pypi. Will it be too much additional work while creating a docker container?

Another option would be to try this approach: https://stackoverflow.com/questions/677577/distutils-how-to-pass-a-user-defined-parameter-to-setup-py
Yet, I didn't try it yet.

@zseder This is definitely fixed now. If you export CFLAGS for the Linux build and set e.g., -march= or -msse.... it will go with a specified option. For a Windows build you need to specify the ARCH variable. Please, see the code here for a reference: https://github.com/nmslib/nmslib/blob/master/python_bindings/setup.py#L116


[ClickHouse] Add async insert related metrics

What does this PR do?
This PR adds two metrics related to async inserts
Motivation
Async insert is becoming the default way of inserting data in ClickHouse and ClickHouse Users are using it for its ease and performance over normal insert.
Additional Notes
Including just two metrics which are the most useful :) Keeping the same nomenclature for metrics as used for normal inserts.
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] Changelog entries must be created for modifications to shipped code
[ ] Add the qa/skip-qa label if the PR doesn't need to be tested during QA.

Hi Team,
Looking at the test failure, I found that the newly added metrics only get populated when we do an async insert into clickhouse something like since async_insert is not enabled by default:
INSERT INTO <table> .... SETTINGS async_insert = 1
Can you give me pointer how can i set clickhouse configuration async_insert = 1 for tests

We included these changes in https://github.com/DataDog/integrations-core/pull/17218.
Thanks for the PR!


Support for loading platform-specific icons

Per https://github.com/flutter/flutter/pull/7297#pullrequestreview-13884591
Currently, if you want the platform-specific version of the back icon, you need to write something like this:
  IconData _backIcon() {
    switch (Theme.of(context).platform) {
      case TargetPlatform.android:
      case TargetPlatform.fuchsia:
        return Icons.arrow_back;
      case TargetPlatform.iOS:
        return Icons.arrow_back_ios;
    }
    assert(false);
    return null;
  }

Perhaps we should add a utility function to Icons, like Icons.for(TargetPlatform), so you'd write this instead:
Icons.for(Theme.of(context).platform).arrow_back

I'd be wary of adding that much overhead for all icons.
How about a specific method just for this icon, Icons.arrow_back_for(Theme.of(context).platform)?

It would require a bit of tweaking to the script so that it adds breadcrumbs from the relevant real icons to this method.

From a discoverability point of view, it's a little annoying to have the one-off static methods on Icons, though as you say a tweak to the docs mitigates that a bit. Given how few of these I'd expect there to be, agreed that keeping it all consts seems like a fair trade though.
Quick look at Apple's icons page in the HIG and it doesn't look like many other widely-used icons where we'd want to use the iOS version. The main candidates look like the 'action' (share) icon and maybe the delete (trash can) icon.

@abarth may have opinions on this given his work on the Cupertino library.

I'm tentatively closing this. The decision of the icon to use should be in the 'application'-side Material BackButton rather than in the supply side Icons definitions.


Travis broken

It seems the SS4 version broke Travis.
The tests are okay in SS4, it's only the Travis config that's broken

Thanks for reporting @Firesphere - @dnsl48 has fixed this with #15 :)


Issue with consumables/weapons/potions

(Required) 3.3.4
(Required)No console errors show up. I have a quest to deliver a "Water Bottle" to an NPC. When I try to deliver it says in game "Potion x1 is not a required item for this quest!" I have tried different ways of inputing it but it keeps defaulting to "Potion".
(Optional) https://pastebin.com/3XA3gig1

Duplicate of #107


Managed Instance Questions

Managed instances are an interesting and useful development.
(1) Is is possible to pause a manage instance, as with Azure SQL Data Warehouse (DW)?
Azure SQL DW allows the DW to be paused, during which time only storage charges accumulate. One of the weaknesses of Azure SQL DB is that pausing a DB is not possible - which goes against one of the ideals of the cloud (only pay for what you need).
By pausing, I means I can stop it instantly (within a few seconds) and resume it instantly. I don't want to have to export/backup my data every time that I need to pause.
(2) Are there plans to introduce additional sizes for the manage instances before/at GA? 8 cores is too large for some purposes and 24 too small for others.

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

ID: 28ca1f7f-5a34-0f23-10ce-3ff24c76db97
Version Independent ID: 5c4a22c4-78fc-911d-9fa0-8dfab1653b6e
Content: Azure SQL Database Managed Instance Overview
Content Source: articles/sql-database/sql-database-managed-instance.md
Service: sql-database
GitHub Login: @bonova
Microsoft Alias: bonova

@cbailiss Thanks for the feedback! We are currently investigating and will update you shortly.

@cbailiss You are correct in that only Azure SQL Data Warehouse can be paused. Azure SQL Database and Azure SQL Elastic Pools cannot be paused. You have an option with SQL Server installed inside an Azure Windows Virtual Machine (Single Instance), where the SQL Server runs as a Windows Service and can be stopped/started and the Windows VM can be stopped/started as needed to reduce total cost. This might be an option for you, and it might help you with the second question you are asking in regards to # of cores, as you have the ability to select the VCPUs, IOPS, RAM, etc.
As for changes to the pricing model for Managed Azure SQL Server and Elastic Pools, I do not know of any changes to how pricing might change in the future. A good place to investigate Azure pricing can be found here, where you can find a pricing calculator for Azure: https://azure.microsoft.com/en-us/pricing/
Please let me know if you have additional questions.
Thank you,
Mike

@cbailiss Additionally, you may provide product feedback here in regards to your second question about pricing. This information goes directly to the product owners.
Regards,
Mike

Thank you for the reply.  However, my questions relate specifically to the recently announced "SQL Database Managed Instance"
i.e. as described on this page:
https://docs.microsoft.com/en-us/azure/sql-database/sql-database-managed-instance
Your answer does not cover this (i.e. you mention Azure SQL DB and Azure SQL Elastic Pools which are different to "Managed Instances").  You also mention an Azure Windows VM which is presumably SQL IaaS and based on the following Azure service (so also not what I was asking about):
https://azure.microsoft.com/en-gb/services/virtual-machines/
Please could I ask you to answer the questions again (especially the first question), this time specifically about "SQL Database Managed Instances".

@cbailiss SQL Database Managed Instance is part of SQL Database, as are single databases and elastic pools. SQL Database does not currently support pausing.

Thanks.  It is not clear from the documentation that Managed Instances are part of Azure SQL database.  E.g. in the usage scenario table near the top of this page (URL below), (1) Managed Instance, (2) Azure SQL Database and (3) SQL IaaS are listed as three separate items, which suggests Managed Instances are different to Azure SQL Database.
https://docs.microsoft.com/en-gb/azure/sql-database/sql-database-managed-instance

@cbailiss updateed topic to add clarity

@cbailiss Updates to the Azure SQL Database Managed Instance Overview documentation have been made. I am going to move this issue tracker to closed but, if you should have any additional questions or comments please do not hesitate to reopen so as to continue the dialog.
Kind regards,
Mike

Managed instance - Is it planned to have database, managed instance or SQL Agent on managed instance to be offline or pause? More than reducing the cost its much needed feature to quiesce the database activity during code upgrades, patches etc. It contradicts with the cross database query feature if it can not be paused for the database updates.


Add ExprStmt?

Right now the Stmt enum is as follows:
#[ast_node]
pub enum Stmt {
    #[tag("BlockStatement")]
    Block(BlockStmt),

    #[tag("EmptyStatement")]
    Empty(EmptyStmt),

    // ...etc...

    #[tag("*")]
    Expr(Box<Expr>),
}

So since it goes straight from the Stmt enum to the Expr there is no way to get a struct node for the expression statement that would have a span that may include the semi-colon (or I might be missing something... I'm new to Rust).
Proposed Change
#[ast_node]
pub enum Stmt {

    // ...etc...

    #[tag("ExpressionStatement")]
    Expr(ExprStmt),
}

#[ast_node("ExpressionStatement")]
pub struct ExprStmt {
    pub span: Span,

    pub expr: Box<Expr>,
}

I published a new version of ast / parser / codegen with a fix.

Wow, awesome! Thanks again @kdy1!


[Question] Is it possible to optionally include the leading space

I'm trying to achieve the equivalent of the highlighting provided by something like https://github.com/folke/todo-comments.nvim or the following emacs screenshot.
One thing I haven't been able to figure out how to do it is to match the preceding whitespace for a tag if there is any.

This doesn't seem to be part of the actual comment i.e. this parser I don't think covers it, and I'm also unable to use #offset! to extend the range of the injection so I guess my question is, is this currently possible with a query (as far as you can see) or would it require this parser to be able to include it?

Hi, the offset directive currently only works with queries for injections, I thought there was an issue tracking this on neovim, but the closes to this is https://github.com/neovim/neovim/issues/16032.
Once the offset directive is supported on highlights queries, you should be able to write a query and/or directive to include the surrounding spaces.
Another solution would be for the parser to expose the "whitespace" nodes https://github.com/stsewd/tree-sitter-comment/blob/a37ca370310ac6f89b6e0ebf2b86b2219780494e/grammar.js#L1, but I haven't seen any other parser to that.


Help in testing Netlify deployment

I need someone free to help test draft Netlify deployment with me. Please comment below if you can work on it with me.
The testing will just be you creating pull requests when needed.

@shahednasser I would love to help you out.

Ok great @HADES-01 . Please create a pull request with any change in index.html. The change is not important so it can be anything


How to start developing

General Question
Add reference about how to start developing the project.

You may refer to wiki-list

@2019zgs
If you are intereting about  developing, you can first refer
https://github.com/radondb/radondb-mysql-kubernetes/issues/379

at first you should install k8s or kubesphere， then run and test it.

Glad to join us@2019zgs


Add information on how to start syslog-ng daemon

The webpage says:

The syslog daemon is not run automatically. You have to start it yourself.

However I could not find any information on that in the documentation. Could you please add information on how to start the syslog-ng daemon?

Which "webpage" do you mean? Your link doesn't go anywhere.

Sorry, fixed that.

Oh, I think I got it: the quote was part of the motivation for baseimage-docker. Sorry.


fix: implement handle_info/2

When running the project again, I noticed some errors I didn't see before in the output:

This messages correspond to the messages sent by the nodejs for warnings and debugging info.
This PR fixes the problem by implementing the handle_info/2 method, as advised in the error message. In order to avoid the spam, I added a check for Mix.env() to make sure we were only showing this information in dev.
This is the equivalent of the command above with this PR:

FYI @Valian @grossvogel

@grossvogel do you think you could have a look at this ?


include root directory in .classpath (in sbt build)

When following standard conventions, users put .scala files in the root of the project directory in sbt.  However, sbt-eclipse doesn't add a .classpath entry for these .scala files in the root directory.
see stack overflow for an example where the .classpath generated for the sbt build is missing the the root directory, and for a manually generated solution.
It's probably possible to detect this situation automatically by noticing .scala files in the root directory.  But even without an automatic detection, an option like sourcesInRootDirectory=true would be nice.

Any progress on this? I too have a project that has sources in the root folder but nested. That is com/company/lib. So the contents of this folder should have package com.compay.lib
The plugin currently adds
<classpathentry output="C:-Users-User-Projects-Project1" kind="src" path="C:-Users-User-Projects-Project1"/>

that's the path to my project dash separated and it doesn't work
I fix it manually from eclipse ui by adding the project root as a source folder and an inclusion rule and I get the following classpathentry
<classpathentry including="com/**" kind="src" path=""/>

which works
Can this be done automatically? I would gladly create a PR if someone points me to the right direction. :)


ZipZap iOS framework, objective-c, how can I make a single zip file, jpg s all under the Documents folder?

ZipZap iOS framework, objective-c, how can I make a single zip file, jpg s all under the Documents folder?
My Documents directory : Documents directory: ( "1.jpg", "2.jpg", "3.jpg", "4.jpg", "5.jpg", "6.jpg" )

Follow the recipe here: https://github.com/pixelglow/ZipZap/wiki/Recipes -- in particular, use [NSData dataWithContentsOfFile:options:error:] to get the data from each picture.


Fix daa7d7d

I'm sorry. I'm still learning this project's architecture and accidentally broke metadata for some channels on PlutoTV with this patch. Hope to contribute a lot of changes as I'm using it to archive a lot of content.

it should still be merged

Life got busy this year, and I'm going to trust that this works good


chore(chats): make unreads a HashSet ans rename messages functions

What this PR does 📖

rename some functions in compose/messages/mod.rs to make it easier to read.
in preparation for adding infinite scrolling to chats, change unreads from a u32 to a HashSet. This will allow uplink to properly update the "unreads" badge for a chat even when the chat view doesn't contain the message being deleted. unreads was made private to force the UI to use utility functions. This necessitated adding a new() function because the struct syntax could no longer be used to create a Chat.

Which issue(s) this PR fixes 🔨

Resolve #

Special notes for reviewers 🗒
Additional comments 🎤
testing:

receive new message in a chat that isn't selected. unreads should increment.
switching chats should clear unreads
receiving a message in the active chat should not increment unreads
if a conversation has unread messages, quitting uplink and re-starting the app should not affect the unread count.

This is the sort of change that we can expect the state.json to get recreated again and we loose existing settings like favorited users and the people in the chatbar, right?

This is the sort of change that we can expect the state.json to get recreated again and we loose existing settings like favorited users and the people in the chatbar, right?

i'd expect unreads to get changed to zero. I'd expect no other changes regarding state.json.

This is the sort of change that we can expect the state.json to get recreated again and we loose existing settings like favorited users and the people in the chatbar, right?

During testing I can verify that I lost my chats/favorites but other than that the functionality of this PR has been awesome 👍

does this happen in dev too?


feat: increase fast path concurrency

Adopt the same pattern as "slow path". Allow concurrent checks against checkAssociatedObjects instead of serial ones. On large payloads (or multiple object/relation) we don't want to create push back (but not allow it to go unbounded) thus keeping the concurrency to 2.
With the improvements in https://github.com/openfga/openfga/pull/1984/files the throughput of the producer should increase thus needing a faster consumer.
Eventually these consumeUsersets and consumeDispatches could be refactored into a generic function.

Description

References

Review Checklist

[ ] I have clicked on "allow edits by maintainers".
[ ] I have added documentation for new/changed functionality in this PR or in a PR to openfga.dev [Provide a link to any relevant PRs in the references section above]
[ ] The correct base branch is being used, if not main
[ ] I have added tests to validate that the change in functionality is working as expected

Instead of skip changelog, perhaps it will be good to mention this perf improvement


fix: do not add target blank to urls that start with # in markdown

Description
Currently it is not possible to add  internal links starting with # as those get target=_blank added.
Additional context
#234

What is the purpose of this pull request?

[x] Bug fix
[ ] New Feature
[ ] Documentation update
[ ] Other

Before submitting the PR, please make sure you do the following

[x] If it's a new feature, provide a convincing reason to add it. Ideally, you should open a suggestion issue first and have it approved before working on it.
[x] Read the Contributing Guidelines.
[x] Read the Pull Request Guidelines and follow the Commit Convention.
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[x] Ideally, include relevant tests that fail without this PR but pass with it.

I could not run the code directly.
I have tested it in our Project where I have found it with editing the code in node_modules directly.
Here the issue that I have locally when I tried to run the code.
Nodejs: 16.16.0   pnpm: 7.9.0
pnpm run watch
.....
│ dist/bundled/client/app/util.js                                             2.80 KiB / gzip: 0.69 KiB
│ dist/bundled/client/app/Story.js                                            3.01 KiB / gzip: 0.98 KiB
│ dist/bundled/client/app/Variant.js                                          1.56 KiB / gzip: 0.56 KiB
└─ Running...
/home/resubaka/code/histoire/packages/histoire-plugin-nuxt:
 ERR_PNPM_RECURSIVE_RUN_FIRST_FAIL<EMAIL_ADDRESS>build: `rimraf dist && tsc -d`
Exit status 2
 ELIFECYCLE  Command failed with exit code 1.
 ELIFECYCLE  Command failed with exit code 1.

I could not run the code directly.

Could you rebase with main?

After the rebase it works now.
Should I add a new Story with only Markdown and a Link that links to a h1 on the page for example as a test?

Sure!


The Gradle Language Server server crashed 5 times in the last 3 minutes. The server will not be restarted.

Extension Name: vscode-gradle
Extension Version: 3.8.0
OS Version: Windows 10
VSCode version: 1.60.2
Describe the bug
As the title states, the Gradle language server crashes immediately when starting VSCode.  The Gradle Language Server output tab shows the following stacktrace (five times):
Exception in thread "main" java.lang.NoSuchMethodError: java.util.List.of(Ljava/lang/Object;Ljava/lang/Object;)Ljava/util/List;
	at com.microsoft.gradle.GradleServices.<clinit>(GradleServices.java:86)
	at com.microsoft.gradle.GradleLanguageServer.<init>(GradleLanguageServer.java:76)
	at com.microsoft.gradle.GradleLanguageServer.main(GradleLanguageServer.java:54)
[Info  - 9:56:05 AM] Connection to server got closed. Server will restart.

This didn't start happening until this morning (it appears the extension was updated last night).
To Reproduce
Install extension, restart VSCode.
Expected behavior
I would expect the Gradle Language Server to start properly.
Screenshots

Output from "Gradle Tasks"
[info] [gradle-server] Server started, listening on 60802
[info] Gradle client connected to server

Does the bug still exist if you disable all other extensions?
Yes
Additional context
I am using a gradle wrapper with gradle version 4.10.3.

@AHilyard Your workaround can resolve this issue, but there are still some bugs for Java8 users :(. (As we know, List.of() becomes available since Java9), I'll fix it.

Seeing the issue too. But in our case, we can't move off Java8 any time soon. So a fix would be great.


sourav_np

Hey @souravsah  thanks for your contribution ❗. Keep up the good job and leave a star if you liked the repo .
Happy hacking 😉


Thought on the Reducer

Wondering what you think about doing this for your reducer instead of the switch statement? My thoughts were switch statements could get long, plus maintaining a large one could be annoying. Plus the type key/value pair seems redundant if you make the added state object agnostic of static state names.
const reducer = (state, action) => {
  const actionKey = Object.getOwnPropertyNames(action)
  return action ? { ...state, [actionKey] : action[actionKey] } : state
}

instead of dispatching like so,
() => dispatch({ type: 'type', thing: false})
you would dispatch like so,
() => dispatch({thing: false})

Reducer and action can be written in any way in your app.
Of course your solution can work too. Also depends on the use case. If the action passes simple value, your solution is enough. But usually you need to do more complex operations with your state (merge deeper objects, etc.)


some question about documentation

so what is the difference about line and lines
and why the lines、percentage、status.paste dont display in the status line, what's the meaning of them?

line is the current line the cursor is on. lines is the total number of lines in the file. lines and percentage should display in the statusline if you modify the option to include them (they are not in the format by default). status.paste only shows information if the paste option is enabled. Otherwise it will be invisible.

lines and percentage dont work, display nothing.

So what should I do then?

Ah I think it should be status.lines, the docs should be clarified about this.

and what about percentage?
status.percentage dont work as well.


Detailed History

I would be nice to know what was updated along with who updated a record.

Votes: 2
Author: vanslyke
Voters: vanslyke,jlutang

Unfortunately we aren't able to do this because we don't collect or store that information.
Author: postit


Hamming distance implementation

Hi!
I love this library and how it provides such a rapid distance calculation for Python, so I've created a Hamming distance implementation. However I can't seem to create a pull request from my branch. Would it be possible to merge this, as I'd love to have more distance metrics being available as part of this library.
// levenshtein.hpp
template <typename Sentence1, typename Sentence2>
std::size_t hamming(const Sentence1& s1, const Sentence2& s2);

// levenshtein.txx
template <typename Sentence1, typename Sentence2>
std::size_t levenshtein::hamming(const Sentence1& s1, const Sentence2& s2)
{
  auto sentence1 = utils::to_string_view(s1);
  auto sentence2 = utils::to_string_view(s2);

  // Swapping the strings so the first string is shorter
  if (sentence1.size() != sentence2.size()) {
    return std::numeric_limits<std::size_t>max();
  }

  std::size_t hamm = 0;

  for (std::size_t i = 0; i < sentence1.length(); i++) {
      if (sentence1[i] != sentence2[i]) {
          hamm++;
      }
  }
  return hamm;
}

For what it's worth, I've also updated the py_levenshtein.cpp code in the Python repo:
constexpr const char* hamming_docstring = R"(
Calculates the Hamming distance between two strings.

Args:
    s1 (str):  first string to compare
    s2 (str):  second string to compare

Returns:
    int: Hamming distance between s1 and s2
)";

// C++11 does not support generic lambdas
struct HammingDistanceVisitor {
  template <typename Sentence1, typename Sentence2>
  std::size_t operator()(Sentence1&& s1, Sentence2&& s2) const
  {
    return rlevenshtein::hamming(s1, s2);
  }
};

PyObject* hamming(PyObject* /*self*/, PyObject* args, PyObject* keywds)
{
  PyObject* py_s1;
  PyObject* py_s2;
  static const char* kwlist[] = {"s1", "s2", NULL};

  if (!PyArg_ParseTupleAndKeywords(args, keywds, "OO", const_cast<char**>(kwlist), &py_s1, &py_s2))
  {
    return NULL;
  }

  if (!valid_str(py_s1, "s1") || !valid_str(py_s2, "s2")) {
    return NULL;
  }

  auto s1_view = decode_python_string(py_s1);
  auto s2_view = decode_python_string(py_s2);
  std::size_t result = mpark::visit(HammingDistanceVisitor(), s1_view, s2_view);

  return PyLong_FromSize_t(result);
}

@maxbachmann the way I'm catching the length mismatch isn't quite right but would love your thoughts!

Thank you for the contribution @ideasbyjin.
I would change the length mismatch error to:
  if (sentence1.size() != sentence2.size()) {
    throw std::invalid_argument("s1 and s2 are not the same length.");
  }

std::size_t result;
try {
    result = mpark::visit(HammingDistanceVisitor(), s1_view, s2_view);
} catch(std::invalid_argument& e) {
    PyErr_Format(PyExc_ValueError, e.what());
    return NULL;
}

and update the docstring to tell the user, that the lenght has to be similar (otherwise a ValueError is thrown)

I added the implementation. The commits reference you as co author.

@maxbachmann many thanks! This is great. I realised the Python library (py_levenshtein.cpp) still remains unchanged though, can we get that sorted?

I added this in the Python library yesterday aswell: https://github.com/maxbachmann/RapidFuzz/commit/fa17ff98e8dba3d67e21b4264d8eca336a8e401a
I released Version 0.14.0 yesterday, that includes this change.

doh thanks i'm blind


`GLIBC_2.28' not found

Hi there,
First of all, thank you for creating this tool. I cannot find anything that allows me to convert an onnx model to an image which is really annoying when trying visually show the file in anything that is not an ONNX viewer.
I have tried to run the following command:
netron_export --output ./output.svg ./yolo_nas_s.onnx
but I get the following:
(ONNX_to_Image) simon@simon:~/Documents/ONNX_to_PNG/netron-export$ netron_export --output ./output.svg ./yolo_nas_s.onnx
Serving './yolo_nas_s.onnx' at http://<IP_ADDRESS>:8487
/home/simon/anaconda3/envs/ONNX_to_Image/lib/python3.8/site-packages/playwright/driver/node: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.28' not found (required by /home/simon/anaconda3/envs/ONNX_to_Image/lib/python3.8/site-packages/playwright/driver/node)
Exception: Connection closed while reading from the driver
Stopping http://<IP_ADDRESS>:8487
Task exception was never retrieved
future: <Task finished name='Task-4' coro=<Connection.run..init() done, defined at /home/simon/anaconda3/envs/ONNX_to_Image/lib/python3.8/site-packages/playwright/_impl/_connection.py:268> exception=Exception('Connection closed while reading from the driver')>
Traceback (most recent call last):
File "/home/simon/anaconda3/envs/ONNX_to_Image/lib/python3.8/site-packages/playwright/_impl/_connection.py", line 269, in init
self.playwright_future.set_result(await self._root_object.initialize())
File "/home/simon/anaconda3/envs/ONNX_to_Image/lib/python3.8/site-packages/playwright/_impl/_connection.py", line 211, in initialize
await self._channel.send(
File "/home/simon/anaconda3/envs/ONNX_to_Image/lib/python3.8/site-packages/playwright/_impl/_connection.py", line 61, in send
return await self._connection.wrap_api_call(
File "/home/simon/anaconda3/envs/ONNX_to_Image/lib/python3.8/site-packages/playwright/_impl/_connection.py", line 482, in wrap_api_call
return await cb()
File "/home/simon/anaconda3/envs/ONNX_to_Image/lib/python3.8/site-packages/playwright/_impl/_connection.py", line 97, in inner_send
result = next(iter(done)).result()
File "/home/simon/anaconda3/envs/ONNX_to_Image/lib/python3.8/site-packages/netron_export/_netron_export_graph.py", line 23, in save_model_graphs
async with async_playwright() as pw:
File "/home/simon/anaconda3/envs/ONNX_to_Image/lib/python3.8/site-packages/playwright/async_api/_context_manager.py", line 47, in aenter
playwright = AsyncPlaywright(next(iter(done)).result())
Exception: Connection closed while reading from the driver
(ONNX_to_Image) simon@simon:~/Documents/ONNX_to_PNG/netron-export$
I have following:
node version: v16.20.2
nvm 0.39.2
Ubuntu 18.04.6 LTS - Bionic
Do you have any suggestions?
All the best,
Simon

It's been over a year so I am not sure whether this well help you or not but it might help someone else facing the same issue in the future.
The problem you have is not directly related to this package. Rather it seems that Node.js, which is used as a driver by playwright (one of the dependencies of this package), requires a libc version of 2.28. As you are on Ubuntu 18.04 you probably have 2.27 and it is really not easy to upgrade these kinds of system libs without bricking your OS.
Here are two options that might fix the issue:

It seems like older versions of node might work with older versions of libc. Version 16 in particular seems to do the trick (see here)

Upgrade your Ubuntu distro. Newer Ubuntu versions come with newer libc versions. Also 18.04 is EOL anyway so its probably a good time to upgrade regardless.


joined to the friends list

Added to friends list

Your pull request seems good, but must be checked by a human maintainer for the following reason(s):

This contains potentially hazardous tags that need manual review.

Please be patient until a maintainer has time to look over your contribution.
This is an automated response, beep boop.
if you think there is a mistake, contact a maintainer

c5f4914b3106a15fbafa0d10e724857b96d3c272

Make fixes in #358 . Closing as duplicate


Create preferences to hold if trade has occurred and details of the trade for that day.

This is a work around for creating a database.

preference activity added and tested.


Simplify object types when distributing index over them

fixes https://github.com/microsoft/TypeScript/issues/59946

@typescript-bot test it


Add CI Android targets

Adds a CI Workflow to build the testapps against the publicly available packaged SDK.

Borrows heavily from the Firebase C++ SDK build scripts.
Nightly builds at 1am PST.
Builds triggered on PRs.
Some updates to the android build dependencies in order for the build to succeed with the cmake version installed on the GitHub CI runners.
#23

Yes, I was thinking along the same lines!


Metrics from riot application

is there a way to obtain metrics from riot application when it is running other than jconsole or similar application?
I want to run riot as part of a script and I want to monitor if queues are getting overwhelmed while performing scan/dump
Thanks!

I suspect it's too late for the original asker of the question, but it's possible to modify the source code to have Spring Batch export metrics via micrometer.  I tried to use the Spring Boot application.properties file to make this happen, but picocli introduces just enough friction that it's not automatic.

Thanks @corey-cole


Convolution refactor

It is probably smart to refactor the convolution module. Exactly how is left to the following comments which act as the framework for the convolution.

For convolution itself, it should all be based on the Astropy convolution function, the class wraps around it and serves to provide
multiple inputs to it with proper checks for size and shape of arrays.


[feat] Measure the time it took to do validation calls, store in results/history

Description
Measuring the time validation took is a useful feature to use, and it is difficult to measure this from the outside. Even if it is just in some "debug" mode, providing this metric in the validation result is needed
Why is this needed
Debugging pipelines, implementing new validators, production system reliability
Implementation details
Timers can be implemented at the validator level, or similarly in the runner
End result

They can get this today by passing in an OTEL tracer.

Is this for each individual validator? Or the validation process as a whole

Both, we currently construct spans for each "step/reask" in the validation loop, the llm interaction (labeled as "call"), and for each validator that runs.  These are sent to the opentelemetry compatible sink that the user specifies when creating their tracer; otherwise none of this is collected.
Once instrumented the user can visualize their telemetry data with their tool of choice.  OpenSearch Dashboards gives some nice waterfall charts out of the box for the spans in a trace.

For users that want to invest less time in setup though, OpenInference's Phoenix app provides some tree visualizations for the spans in a trace out of the box and lists the latency for each span.


Expandable navigation - NVDA

The Expandable Section navigation immediately reads 'list with 5 items, item 1 visited link' - is there a reason why the Expandable Navigation section behaves differently (you have to explicitly tab to the items)?
When you navigate back to the menu, rather than simply say 'expanded' or 'collapsed' it would be better to say 'Show navigation menu - collapsed' and 'Hide navigation menu - expanded' - similar to gov.uk

@susannah-rogers This is now on Netlify ready for testing.

@mjbp, Apologies, I referenced the wrong pattern in the first part:
The Expandable Section navigation immediately reads 'list with 5 items, item 1 visited link' - is there a reason why the Expandable Navigation section behaves differently (you have to explicitly tab to the items)?
It should be:
The menu on Exclusive Toggles immediately reads 'list with 5 items, item 1 visited link' - is there a reason why the Expandable Navigation section behaves differently (you have to explicitly tab to the items)?
The second part of this is now much better ('Show navigation menu - collapsed' and 'Hide navigation menu - expanded') :-)

@susannah-rogers @sarah-storm updated this so both patterns are the same (you have to tab to the menu). The updated version is on Netlify for you to test at your convenience.

@mjbp This one is working as expected now. Thanks.


Unable to Build on OpenSuse Tumbleweed

Steps to Reproduce

Configured project to build on Linux and installed flutter via snap as recommended in the docs.
Executed flutter build linux and it throws an error with clang++

Expected results:
Compile successfully
Actual results:
Building without sound null safety
For more information see https://dart.dev/null-safety/unsound-null-safety

CMake Error at /snap/flutter/50/usr/share/cmake-3.10/Modules/CMakeTestCXXCompiler.cmake:45 (message):
  The C++ compiler

    "/snap/flutter/current/usr/bin/clang++"

  is not able to compile a simple test program.

  It fails with the following output:

    Change Dir: /home/mhicks/projects/prototype/flutter/lesson1/build/linux/release/CMakeFiles/CMakeTmp

    Run Build Command:"/snap/flutter/current/usr/bin/ninja" "cmTC_65a20"
    [1/2] Building CXX object CMakeFiles/cmTC_65a20.dir/testCXXCompiler.cxx.o
    [2/2] Linking CXX executable cmTC_65a20
    FAILED: cmTC_65a20
    : && /snap/flutter/current/usr/bin/clang++    -L/snap/flutter/current/usr/lib/gcc/x86_64-linux-gnu/8 -L/snap/flutter/current/usr/lib/x86_64-linux-gnu -L/snap/flutter/current/usr/lib/  -L/snap/flutter/current/usr/lib/gcc/x86_64-linux-gnu/8 -L/snap/flutter/current/usr/lib/x86_64-linux-gnu -L/snap/flutter/current/usr/lib/  -L/snap/flutter/current/usr/lib/gcc/x86_64-linux-gnu/8 -L/snap/flutter/current/usr/lib/x86_64-linux-gnu -L/snap/flutter/current/usr/lib/ CMakeFiles/cmTC_65a20.dir/testCXXCompiler.cxx.o  -o cmTC_65a20   && :
    /snap/flutter/current/usr/bin/ld: warning: /usr/lib64/gcc/x86_64-suse-linux/10/../../../../lib64/crt1.o: unsupported GNU_PROPERTY_TYPE (5) type: 0xc0008002
    /snap/flutter/current/usr/bin/ld: /usr/lib64/gcc/x86_64-suse-linux/10/../../../../lib64/crti.o: unable to initialize decompress status for section .debug_aranges
    /snap/flutter/current/usr/bin/ld: /usr/lib64/gcc/x86_64-suse-linux/10/../../../../lib64/crti.o: unable to initialize decompress status for section .debug_aranges
    /usr/lib64/gcc/x86_64-suse-linux/10/../../../../lib64/crti.o: file not recognized: File format not recognized
    clang: error: linker command failed with exit code 1 (use -v to see invocation)
    ninja: build stopped: subcommand failed.

  CMake will not be able to correctly generate this project.
Call Stack (most recent call first):
  CMakeLists.txt:2 (project)

Building Linux application...
Unable to generate build files

Logs
Unable to build, no logs
Analyze:
Analyzing lesson1...

   info • Unused import: 'package:lesson1/main.dart' • test/widget_test.dart:11:8 • unused_import
  error • The function 'MyApp' isn't defined • test/widget_test.dart:16:29 • undefined_function

2 issues found. (ran in 0.8s)

[✓] Flutter (Channel beta, 2.0.1, on Linux, locale en_US.UTF-8)
    • Flutter version 2.0.1 at /home/mhicks/snap/flutter/common/flutter
    • Framework revision c5a4b4029c (5 days ago), 2021-03-04 09:47:48 -0800
    • Engine revision 40441def69
    • Dart version 2.12.0

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /home/mhicks/Android/Sdk
    • Platform android-30, build-tools 30.0.3
    • ANDROID_SDK_ROOT = /home/mhicks/Android/Sdk
    • Java binary at: /home/mhicks/.sdkman/candidates/java/current/bin/java
    • Java version Java(TM) SE Runtime Environment (build 1.8.0_241-b07)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at google-chrome

[✓] Linux toolchain - develop for Linux desktop
    • clang version 6.0.0-1ubuntu2 (tags/RELEASE_600/final)
    • cmake version 3.10.2
    • ninja version 1.8.2
    • pkg-config version 0.29.1

[!] Android Studio
    • Android Studio at /home/mhicks/.local/share/JetBrains/Toolbox/apps/AndroidStudio/ch-0/201.6953283
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    ✗ android-studio-dir = /home/mhicks/.local/share/JetBrains/Toolbox/apps/AndroidStudio/ch-0/201.6953283
    ✗ Unable to find bundled Java version.
    • Try updating or re-installing Android Studio.
    • Consider removing your android-studio-dir setting by running:
      flutter config --android-studio-dir=

[✓] IntelliJ IDEA Ultimate Edition (version 2021.1)
    • IntelliJ at /home/mhicks/.local/share/JetBrains/Toolbox/apps/IDEA-U/ch-0/211.6305.21
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] VS Code (version 1.49.2)
    • VS Code at /usr/share/code
    • Flutter extension can be installed from:
      🔨 https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter

[✓] Connected device (2 available)
    • Linux (desktop) • linux  • linux-x64      • Linux
    • Chrome (web)    • chrome • web-javascript • Google Chrome 88.0.4324.182

! Doctor found issues in 1 category.

FYI, I was able to get this to work after switching to the local installation.


Add deleteStickerSet method

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (38de712) to head (e68433b).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##              master       #50   +/-   ##
===========================================
  Coverage     100.00%   100.00%
- Complexity      1137      1143    +6
===========================================
  Files            243       244    +1
  Lines           4166      4178   +12
===========================================
+ Hits            4166      4178   +12

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


[Bug] Files created are owned by root

Current Behavior
I am trying to extract the subtitles from an MKV and the files are owned by root.
Here is my command:-
docker run --rm --name=mkvextract -e USER_ID=$(id -u) -e GROUP_ID=$(id -g) -v $(pwd):$(pwd):rw jlesage/mkvtoolnix /usr/bin/mkvextract "$(pwd)/input.mkv" tracks 3:"$(pwd)/output.srt"
It runs, with no error shown, but the output file is owned by root:-
-rw-r--r--   1   root   root   131K   Jul 15 12:41   output.srt
What am I doing wrong or is this a bug?
Thanks,
Paully
Expected Behavior
I want the extracted subtitles file to be owned by me!
Steps To Reproduce
docker run --rm --name=mkvextract -e USER_ID=$(id -u) -e GROUP_ID=$(id -g) -v $(pwd):$(pwd):rw jlesage/mkvtoolnix /usr/bin/mkvextract "$(pwd)/input.mkv" tracks 3:"$(pwd)/output.srt"
Environment

OS: Linux
OS version: Ubuntu LTS 22.04.1
CPU: Intel(R) Core(TM) i3 CPU M 380  @ 2.53GHz
Docker version: Docker version 24.0.2, build cb74dfc

Container creation
docker run
Container log
N/A

Container inspect
No response
Anything else?
No response

When you invoke a command inside the container, it is run as root.  You can run as another user by using su-exec.  For example, something like that should work:
docker run --rm --name=mkvextract -v $(pwd):$(pwd):rw jlesage/mkvtoolnix su-exec "$(id -u):$(id -g)" /usr/bin/mkvextract "$(pwd)/input.mkv" tracks 3:"$(pwd)/output.srt"

Thanks Jocelyn,
docker run --rm --name=mkvextract -e USER_ID=$(id -u) -e GROUP_ID=$(id -g) -v $(pwd):$(pwd):rw jlesage/mkvtoolnix su-exec "$(id -u):$(id -g)" /usr/bin/mkvextract "$(pwd)/input.mkv" tracks 3:"$(pwd)/output.srt"
Error: Failed to create the file '/home/plittlefield/output.srt': 13 (open file error)

So close!

Error 13 is permission error.  Make sure output.srt doesn't already exist from a previous execution.

Yay - we have a winner!
Thank yooooouuuu  :-)


why? How? train error 0%| | 0/100000 [00:00<?, ?it/s]

Traceback (most recent call last):
File "/root/anaconda3/envs/CornerNet_Lite/lib/python3.7/multiprocessing/queues.py", line 236, in _feed
obj = _ForkingPickler.dumps(obj)
File "/root/anaconda3/envs/CornerNet_Lite/lib/python3.7/multiprocessing/reduction.py", line 51, in dumps
cls(buf, protocol).dump(obj)
File "/root/anaconda3/envs/CornerNet_Lite/lib/python3.7/site-packages/torch/multiprocessing/reductions.py", line 295, in reduce_storage
fd, size = storage.share_fd()
RuntimeError: unable to write to file </torch_5807_1602858868>
setting learning rate to: 0.00025
training start...
0%| | 0/100000 [00:00<?, ?it/s]

please，help me

if you are using docker, try to run it with  --ipc=host flag

if you are using docker, try to run it with  --ipc=host flag

it works.Thank you.So happy

what if not? I encounter the same problem with him. Thanks.

如果您使用的是docker，请尝试使用 --ipc=hostflag 运行它

hello,where can i add this command？

i met the same problem , but i did not use docker, so how can i solve the problem?


deps: update @octokit dependencies

Octokit dependencies are updated, which include breaking changes such as only supporting Node versions greater than 18. Additionally, it introduces the ability to access new APIs now available on GitHub.
Note: the latest versions are not used because the packages were converted to ESM only, removing support for CommonJS

It's curious how it worked locally but doesn't work on GitHub Actions, I'm going to investigate it.


API.get_direct_messages() not working

I would like to read my direct messages however the call API.get_direct_messages() returns:

AttributeError                            Traceback (most recent call last)
 in
----> 2 API.get_direct_messages()
AttributeError: 'API' object has no attribute 'get_direct_messages'
My permissions are correct as I can send messages to other users. Any ideas of why this might be?
Thanks in advance

My bad, I was looking at old documentation. API.list_direct_messages() is working

API.list_direct_messages was actually renamed to API.get_direct_messages with https://github.com/tweepy/tweepy/commit/ff1186f80b2dbb7f5e4ed3669d90db300e8de36d, and the latest documentation is up to date with that change. API.get_direct_messages should work with the development version on the master branch, and any documentation with API.list_direct_messages would be the "old documentation".


Install seccomp before setting the environment

Successful run here: https://github.com/hakman/containerd/runs/2445821186
Fixes: #https://github.com/containerd/containerd/issues/5425
It is not an ideal fix, but would prefer to change as little as possible to unblock the release.
Signed-off-by: Ciprian Hacman<EMAIL_ADDRESS>

runc without libseccomp is almost useless, so NOTLGTM

Not sure what you mean @AkihiroSuda. libseccomp should still be installed with this.

Sorry if I'm misunderstanding, but this PR seems to disable compiling binaries with CGO, which results in compling runc without libseccomp support, no?

Got it. I think I found a better way by reordering the build steps and installing libseccomp earlier, before setting the environment for cross compiling.

/ok-to-test

/retest

/retest

@dims Any ideas what is wrong in the k8s pre-submit?
unable to create gce instance with running docker daemon for image cos-85-13310-1260-5.
could not create instance tmp-node-e2e-ac19a87d-cos-85-13310-1260-5:
[&{Code:QUOTA_EXCEEDED Location: Message:Quota 'IN_USE_ADDRESSES' exceeded.
Limit: 100.0 in region us-central1. ForceSendFields:[] NullFields:[]}]

Just saw this after pushing an alternative https://github.com/containerd/containerd/pull/5429
Additionally It looks like we don't need to compile libseccomp since we are pulling it in from the package repo. Notably here you are not building seccomp for the target platform anymore 😄

The alternative @cpuguy83 proposed seems more correct as far as what is being built, but if there is some reason to not be performing this "install seccomp" step we should look at that too--can you clarify what that means @cpuguy83?

@estesp We only need libseccomp for building runc, and we are already installing libseccomp for each architecture from the package manager. go build with the right CC (which we have) will find this just fine.

/test pull-containerd-node-e2e

With #5429 merged do we still need any part of this?

No, but maybe be a good idea to fix the btrfs issue addressed https://github.com/containerd/containerd/pull/5434.


Adding justgo 1.0.1

If there’s a checkbox you can’t complete for any reason, that's okay, just explain in detail why you weren’t able to do so.
After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} reports no offenses.
[x] The commit message includes the cask’s name and version.

Additionally, if adding a new cask:

[x ] Named the cask according to the token reference.
[x] brew cask install {{cask_file}} worked successfully.
[x] brew cask uninstall {{cask_file}} worked successfully.
[x] Checked there are no open pull requests for the same cask.
[x] Checked the cask was not already refused in closed issues.
[x] Checked the cask is submitted to the correct repo.

This seems to be a CLI-only open-source tool. Please submit it first to Homebrew/core and only if they don’t accept it, submit here (linking back to the pull request where it was refused).
This policy is documented in faq/rejected_casks:

The app is both open-source and CLI-only (i.e. it only uses the binary artifact). In that case, and in the spirit of deduplication, submit it first to Homebrew/core. If it is rejected there, you may then try again in Homebrew-Cask (link us to the issue on Homebrew so we can see their reasoning for rejection).

@vitorgalvao thank you for such prompt response. This is my first brew submission so I may not understand how to properly do it, but when I read instructions at homebrew/core it explicitely said they do not like binary submissions and any binary submission should be directed to Cask?
What I am submitting is a Go CLI tool and it's important the installation to be binary / not require pre-existing Go environment (otherwise a go get is more than sufficient).
Can you please clarify why a binary submission is redirected to core? Thanks

Why can’t this be built from source? That’s the point.
Unfortunately, I just noticed neither HB nor HBC will accept this. You’re self-submitting a tool with too few stars. We don’t accept those to keep the repos manageable with software other people will actually use.

@vitorgalvao it can be built from source but that defeats the purpose. The only reason anybody would install a Go package with brew is to download a binary (and not have to install Go environment). If somebody already has a working Go environment, or is willing to install one, then go get <githuburl> is both simpler and easier.


Request for Additional Information on Time Metrics and Heatmaps in Tables 1 and 2

Hello Edward,
I am currently reproducing the results presented in your paper, and I have encountered a couple of issues that I'm hoping you can help me with to ensure an accurate comparison with other works.
Firstly, while Table 1 in the paper provides details on Length and Drop, there is no mention of the Time taken for each dataset. Although Table 2 includes the Time, it is not explicitly divided into the time taken for generating the Heatmap and the time taken for the subsequent search process. For the sake of a fair comparison with other papers, could you kindly share the Time details for Table 1 and Table 2, as well as the breakdown of the time consumed for each part?
Secondly, I appreciate that you have shared the checkpoints, which are immensely helpful. However, to fully reproduce your paper's results, could you also share the Heatmaps generated for the datasets in Tables 1 and 2? Access to these Heatmaps would be incredibly valuable for my reproducing.
Thank you for your time and assistance, and for your contributions to the field. I look forward to your response.
Best regards,
Yifan

Hi Yifan,
About time in Table 1, please refer to FAQ A.4 in our camera-ready version of the paper.
About the time taken for each stage (I assume you are referring to the MCTS results), I think you can approximate the neural network time by the greedy decoding time, and MCTS is the total time subtract that.
Currently, we don't have a plan to share the heatmap checkpoints, as they are usually super large, especially for TSP-10000 instances. But I think you can easily reproduce them by just running our inference code with the provided checkpoints.
Thanks

Thank you very much for your assistance. It's been invaluable in advancing my research.


Add unit tests verifying From impls for hashes

It is convenient to write pk.into() to get various hashes. We already have these implementations, add a unit test to verify it.
Fix a code comment in the wpukbey_hash function while we are at it.
Fix: #1909

Also From is defined in key.rs so the test could go there?  I don't really understand why the impl PubkeyHash and the struct are in different files along with the traits.

I don't really understand why the impl PubkeyHash and the struct are in different files along with the traits.

BOOM! Thanks man.

You say new does not make sense for Script since it implies ownership, but both Path and PathBuf have new constructors (and neither have empty one).
I think we should use the name new for both.

ack b798939fde1d9890a93d41abe31eaf23e4a5356e other than the new/empty thing. Also, I'm unconvinced that moving the hash types is a breaking change, but it's fine, since the Script constructor rename definitely will be.

Will fix  emty/new thing as suggested.

I'm unconvinced that moving the hash types is a breaking change

Well after this one can no longer write use bitcoin::hash_types::PubkeyHash and before this they can. That is a breaking change, right? IIRC I mentioned it because I intentionally did not add a re-export in hash_types since we are moving loads of things out of there.

I just realized the new/empty change should not even be in this PR, its totally unrelated to pubkey/script hash - will pull it out into a separate PR.

Added a patch 9216ab11 Move re-export of newtypes::* to stop rustfmt messing with the code layout.

Rebased on master (to pick up #1927). No other changes.

Removed the first two patches (because we now have https://github.com/rust-bitcoin/rust-bitcoin/pull/1925) and rebased.

Pulled master and rebased, woops.

Merging based on #1945. The rebase made no changes.


Add some assertions to cranelift_frontend

cc https://bytecodealliance.zulipchat.com/#narrow/stream/217117-cranelift/topic/Jumping.20to.20an.20earlier.20block.20from.20a.20later.20block.3F/near/198585961
cc @osa1

Thanks for doing this, getting "INVALID encountered" in seal_block or when printing is really confusing. Catching these errors earlier will be very helpful.


Change level 4 sub headers to level 3

Signed-off-by: Kevin Earls<EMAIL_ADDRESS>

@kevinearls Thanks!


[BUG]: Create sample data set for trace analytics and Gantt charts

What is the bug?
Unable to create Gantt chart visualization type in Dashboards using sample data sets. The Event, Start time, and Duration metrics don't exist in the sample data sets. I tested all three Dashboards data sets and I'm unable to create a Gantt chart following the documentation tutorial steps. Also, a sample Gantt chart isn't in the OpenSearch Playground sample visualizations list.
How can one reproduce the bug?
Steps to reproduce the behavior:

See existing documentation https://opensearch.org/docs/latest/dashboards/gantt/

What is the expected behavior?
User should be able to follow the tutorial steps to learn how to create Gantt charts in Dashboards. I'm unable to replicate this tutorial using our current sample data sets.
What is your host/environment?

OS: iOS
Version: 12.6.2
Plugins: NA

Do you have any screenshots?

Do you have any additional context?
See existing documentation https://opensearch.org/docs/latest/dashboards/gantt/

@alicejw-aws This issue is to related to the sample data set request for trace analytics and Gantt charts.


Allow for non-integer user id in the shell adapter.

Under hubot/src/adapters/shell.coffee line 45, the user id is parsed as integer. For generality, can we remove the parseInt and allow for string type id? Many adapters and chat clients use non-integers too.

That is what I'm proposing in https://github.com/github/hubot/pull/1052

@technicalpickles my bad; didn't see that when I was searching for the issue.


Improved the UI of Contact Us Page

#585 Fixed
Please review and merge this PR.
@sunny0625 @khushi-joshi-05

@jency1 solve conflicts

@jency1 solve conflicts

@khushi-joshi-05 Conflicts resolved.
Please review and merge this PR.

@khushi-joshi-05 @sunny0625 Conflicts resolved.
Please review and merge this PR.

@sunny0625   Can you change the level from level1 to level2 ?
For login page UI also I was given level2 that's why I am telling.
If it's possible then do change it.
Thank you.


Dynamic loading and memory optimization

Hi,
I'm planning to use Swiper for a project where I'll have huge and multiple galleries on a single page so I'm a tad bit worried about performance. So my questions:

A gallery contains 600 images, but I need to load 3 to 5 at a time, however, the pagination should show the actual slide/page number as if it was a 600 page gallery, is that possible with Swiper?

Is Swiper memory efficient? I'm sure it must be but since nothing is mentioned on the website so I'm wondering if Swiper also does some of this:

PhotoSwipe keeps only three images in the DOM at once, because each “slide” is a composited layer, which consumes quite a lot of memory.
Almost no memory is consumed until the gallery is first opened.
The script has been tested for memory leaks. Navigating images back and forth and creating or deleting a gallery doesn’t leave a trace in memory.

RequestAnimationFrame is used wherever possible.
Only the transform and opacity properties are animated. Almost no paints occur during animation, making transitions as smooth as possible.

Thanks

PhotoSwipe keeps only three images in the DOM at once, because each “slide” is a composited layer, which consumes quite a lot of memory.

no

Almost no memory is consumed until the gallery is first opened.
The script has been tested for memory leaks. Navigating images back and forth and creating or deleting a gallery doesn’t leave a trace in memory.

RequestAnimationFrame is used wherever possible.

Only the transform and opacity properties are animated. Almost no paints occur during animation, making transitions as smooth as possible.

yes


Clock out of sync with simple calendar by exactly 1 minute

Just a weird bug I'm encountering, and searching through the settings didn't turn up a solution. I've got Simple Calendar installed, and I just installed SmallTime today. For some reason the SmallTime clock is always 1 minute ahead of the Simple Calendar clock no matter what I set the time to. PF2e's built-in world clock agrees with the Simple Calendar time.

Known issue when you're in a PF2e world with a date that's prior to the world's epoch. Nothing I can do (or rather, am interested in doing) about that, sorry.

ah, damn. thanks for the quick response. What exactly do you mean by prior to the world's epoch? Like, if the date is from before the foundry world was created?

When you create a PF2e world, it has an epoch date set. I don't really understand their mechanics around it. But if your date in the world is at any point earlier than this epoch date, there's an off-by-one error that gets introduced somewhere in the calculation chain that I've never been able to track down.

gotcha, that makes sense, thanks. Any chance you know how to change the epoch date in the world? If not I'll just ask around on discord and that should work. Thanks!

Pretty sure the short answer is you don't, but you could ask your system folks yeah.

Managed to figure it out, actually! Had to go into the console, but game.settings.get("pf2e", "worldClock.worldCreatedOn") output the epoch date, and then game.settings.set("pf2e", "worldClock.worldCreatedOn, [new date]") let me change it. [new date] being the output of the first command but just changing the year number from 2024 to 2019


Dynamic configuration

Is it possible for the plugin use a dynamic configuration?
For example, have the match pattern of fluentd specify a type name in elasticsearch? The Fluentd elasticsearch plugin (which this plugin extends?) allows this to happen.
I've had a play around with the source and have not successfully been able to get it working with the dynamic plugin, but I'm not a ruby dev. Does anyone know if this is possible or am I just shooting in the dark?
Many thanks!

hi @tyronedougherty .
Sorry, do not supported a dynamic configuration.
I do not fully understand yet a dynamic configuration.
thanks!

For anyone that comes across this in the future that wants to know how to do a dynamic configuration for fluentd, I managed to solve this problem by installing fluent-plugin-forest, which allows you to use the tag_parts variable in your config.
My fluentd config now looks similar to the following
<match testpattern.**>
        type forest
        subtype "aws-elasticsearch-service"
        <template>
            type_name ${tag_parts[1]}
            logstash_format true
            flush_interval "10s"
            <endpoint>
                url "http:/xxx.ap-southeast-2.es.amazonaws.com:80"
                region "ap-southeast-2"
                access_key_id "xxx"
                secret_access_key "xxx"
            </endpoint>
        </template>
</match>

Hope this solves someones future problems! :smile:


Fix bug with trajectory sampler node where it publishes 0, 0, 0 on st…

…artup.
@SebastianInd FYI :)
Now the timer doesn't start auto-running at startup since it shouldn't do that anyway. Let me know if there's any issues with this fix!

Rikolas is on holidays now so self-LGTM.


support multi display

https://developer.apple.com/library/content/qa/qa1741/_index.html
https://developer.apple.com/library/content/qa/qa1740/_index.html#//apple_ref/doc/uid/DTS40011007

#9


UCP: Reduce the unnecessary updateStoreStatusLocked call counts in heartbeat process

Description

Score

200

Mentor

@nolouch

Recommended Skills

go

Learning Materials
Ref #1846

/pick-up-challenge

thanks!

/pick-up-challenge

@mantuliu try again please~

/pick-up-challenge

Is this issue already finished? @mantuliu

@nolouch already finished.


Responsive Burger menu does not open on click

The responsive burger menu does not respond to the users click on all the pages except Dashboard. The issue can be that the JavaScript file might not be called right in the HTML inside the handlebars file.

Issue was fixed from fixing the signout button burger menu works on all pages and responds to all clicks to navigate back to other pages.


Change get_current_animation() to get_assigned_animation()

Currently several functions check AnimationPlayer.get_current_animation() to e.g. start playback or find the animation to retarget. This function returns an empty String if the animation is not playing rightnow or has ended and is not looped. Since the Inspector in Godot also bugs out on all input field while an animation is running this creates the annoying requirement to constantly stop and start animations manually for retargeting.
Changing to AnimationPlayer.get_assigned_animation() makes it easier as this function also returnes the previous running animation if playback is stopped. It still requires that the animation was loaded and started ones in the AnimationPlayer. If in the editor the animationtrackeditor panel must be opened ones to set an assigned_animation.

Changed with commit d4e970f64d58739faf736c20da357a3b75878219


feat: add bi-directional infinite query support

PR for discussion:
Adds ability to automatically support both directions:
const {
  fetchNextPage,
  fetchPreviousPage,
  hasNextPage,
  hasPreviousPage,
  isFetchingNextPage,
  isFetchingPreviousPage
} = useInfiniteQuery(key, fn, {
  getNextPageParam: lastPage => lastPage.nextId,
  getPreviousPageParam: firstPage => firstPage.prevId
})

return (
  <>
    <button onClick={fetchPreviousPage} disabled={!hasPreviousPage || isFetchingPreviousPage}>Load previous</button>
     ...
    <button onClick={fetchNextPage} disabled={!hasNextPage || isFetchingNextPage}>Load next</button>
  </>
)

I like it. Though, personally the append/prepend option is strange to think about. Technically this means that you can fetchNextPage(), but it would get prepended, correct? (and visa versa)
I realize that might be helpful for like... a news feed concept. But wouldn't that just end up being a visual reversal at render time and not at the data model?

Yeah the append mode is indeed for those kind of scenarios, where new pages appear at the top instead of at the bottom. Reversing the list when rendering would have the same effect. Mostly added it for convenience and because the reporter of https://github.com/tannerlinsley/react-query/issues/1086 was trying to load previous pages instead of reversing the list. Don’t have a strong opinion on this option though

The amount of code to support the option is tiny and it also won’t carry a lot of opinion with it into the future. So I think it’s fine.

This one should be ready. Also includes support for refetching when manual page params are provided

Excellent usage of the select option to avoid the reversal option.

:tada: This PR is included in version 2.24.0-next.6 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Per FlowFile Target/URL

Is it possible to set a target/URL for each inviidual file? Our REST API uses endpoints like /api/file/1/data, /api/file/2/data, etc so we need the ability to set a URL target per file.
Thanks.

How does one set the target on a per file basis?


HTML Snake Game advance project

HTML Snake Game advance project

You haven't added the Project_doc.md file which is required. Fetch upstream before adding the file

Do the needful


[BUG] Dates are not correct when loading files saved in "Strict Mode"

🐛 Bug Report
TL;DR When parsing Strict mode xlsx files, date cells are parsed incorrectly resulting in dates in 1904/1905.
If you go into excel, do "Save As" and select "Strict Open XML Spreadsheet (.xlsx)" it creates a file in the Strict format (rather than the default "Transitional" format).  The strict format has a few differences, but the one I'm running into issues with is the Date formatting.
I found this reference regarding the date formatting: https://www.loc.gov/preservation/digital/formats/fdd/fdd000401.shtml

Textual and serial numeric representations of dates and times: In the Strict variant of XLSX, dates may be stored using textual forms specified in the ISO 8601 standard. Late in the ISO standardization process for OOXML, a proposal was made to adopt the ISO 8601 format for dates and times in spreadsheets. Dates and times in spreadsheets have usually been stored as numbers (sometimes termed "serial date-time" values)

Unfortunately, when loading a Strict mode xlsx file with excelJS, any date cells are improperly treated as floats and you end up with a date in 1904 or 1905 (depending on OS).
Lib version: 4.4.0
Steps To Reproduce

Using the attached file: DateFormatStrict.xlsx
const ExcelJS = require('./excel')
const fs = require('fs/promises')

const fh = await fs.open('../DateFormatStrict.xlsx')
const bufferData = await fh.readFile()

const workbook = new ExcelJS.Workbook();
await workbook.xlsx.load(bufferData)

const cell = workbook.worksheets[0].getCell('A2')

expect(cell.value).to.equal(new Date('02/09/2024'));
// cell.value is actually 1905-07-16T00:00:00.000Z

The expected behaviour:
The date value should be parsed correctly and match what was entered in excel (so a date object representing 02/09/2024).
Possible solution (optional, but very helpful):
I noticed that the t attribute for the cell values is d, so perhaps we could add a handler for that value type to CellXform parseClose here: https://github.com/exceljs/exceljs/blob/master/lib/xlsx/xform/sheet/cell-xform.js#L358

else if (this.t === 'd') {
  model.result = new Date(model.value)
}

It looks like there is other code that also needs updating since that code is just parsing a float (somewhere else is turning it into a date).  If I have time tonight I'll try to figure it out and make a PR

Hii @jec006 👋
Any chance to see the PR merged? What are the next steps?
I would be happy to help if needed


Cannot click other part of the screen when toastr message is shown

I made a screencast for this: http://screencast.com/t/kAxp1ubj4oNV
Is it possible to clear the message when I click any part of the screen? Thanks a lot for your help.

All sorts of things look strange about your attached video.
Can you post a fiddle of this behavior?

I don't know what you mean about a fiddle of this behavior. Essentially, while the messages are shown, I cannot check the options (such as close button, debug) that are on the same line as messages - the messages are getting the focus - nothing on the same height can be clicked till you close those messages. I hope this is clear enough. Thanks!

Hope this following explains what I meant:

I have no way to reproduce the issue that you're reporting.
Can you provide a way for me to do that?

I just tried again using two browsers at http://codeseven.github.io/toastr/demo.html, same issue - make sure you choose top center and open a few messages so that the message will cover a few options.

I understand your issue.
This is not an issue in the demo master or develop branch, but is an artifact of the demo.html file not being updated in the gh-pages branch.

I am using the latest version (CodeSeven-toastr-2.1.2-3-g8bbd8db) and still have this issue - did you mean the demo page was not using the latest version of toastr?

Demo page now updated. Issue no longer present.

@TimFerrell yes. the demo page is ok now - could you please let me know how you fixed it? I updated to the latest version and the problem was still there. Thanks a lot for your help.

Check out the commits on Aug 26: https://github.com/CodeSeven/toastr/commits/gh-pages

problem solved after update the CSS and JS - thanks a lot for your help!!


Fragile attach debugger behaviour in vscode

🐛 Bug Report
I use jest for some of my projects in the monorepo https://github.com/Prettyhtml/prettyhtml the configuration below is used to run jest tests of the webparser package. As soon as I attach a new breakpoint the test output isn't visible. Sometimes the breakpoint works sometimes not.
Sometimes it works, sometimes not.
To Reproduce

Steps to reproduce the behavior:
Vscode configuration:
{
  "version": "0.2.0",
  "configurations": [
    {
      "type": "node",
      "request": "launch",
      "name": "Debug Webparser Tests",
      "cwd": "${workspaceFolder}/packages/webparser",
      "args": [
        "--inspect-brk",
        "${workspaceRoot}/packages/webparser/node_modules/jest/bin/jest.js",
        "--runInBand",
        "--config",
        "${workspaceRoot}/packages/webparser/jest.config.js"
      ],
      "console": "integratedTerminal",
      "internalConsoleOptions": "neverOpen"
    }
  ]
}

Expected behavior
Debugger should be attached and tests executed.
Run npx envinfo --preset jest
Paste the results here:
  System:
    OS: Windows 10
    CPU: x64 AMD Ryzen 5 2600X Six-Core Processor
  Binaries:
    Node: 10.12.0 - C:\Users\deusd\AppData\Local\nvs\default\node.EXE
    Yarn: 1.10.1 - C:\Program Files (x86)\Yarn\bin\yarn.CMD
    npm: 6.4.1 - C:\Users\deusd\AppData\Local\nvs\default\npm.CMD

Try adding --no-cache jest arg: https://github.com/facebook/jest/issues/6683

Yes, using --no-cache solves my issue thank you!


Test that Ticketmaster (and advertising) modules work.

Integrate and test TM and confirm that the behaviour in the new SDK is correct.

[ ] Take campaigns example app, and install the TM presence SDK: https://developer.ticketmaster.com/products-and-docs/sdks/presence-sdk/android/index.html ;
[ ] Install the Rover Ticketmaster module, and integrate it with the Ticketmaster presence SDK .
[ ] Reach out to @orospakr for credentials for a TM team;
[ ] use the Ticketmaster steps in the integration test plan to verify that it is working: https://github.com/RoverPlatform/rover-campaigns-android/wiki/Test-Plan ;
[ ] while doing the test plan, pay extra close attention to the to the ticketmasterhostMemberID and ticketmasterteamMemberID fields (ignore any capitalization problems).  The team member ID is the important one because we actually only support Archtics integration on the server side.
[x] ensure advertising behaves as expected

sent as hostMemberID and teamMemberID
If one is set, it overwrites the other, so only one can be set at a time