Old tokens not being deleted

Version: 0.1.43

I currently have config.max_number_of_devices = 6 and config.token_lifespan = 10.weeks but everytime I login it is creating a new token and not deleting the older ones. As a result I have 63 tokens right now which I verified by running User.first.tokens.count
Is there a way to delete old tokens?

@KauNair, hello.
Let's take a look here:
https://github.com/lynndylanhurley/devise_token_auth/blob/72976d8d3a0cd8f24578138f9edb7e7930fb75b2/app/models/devise_token_auth/concerns/user.rb#L249-L256

Set remove_tokens_after_password_reset = true in the gem initialize file.
Call User.first.remove_tokens_after_password_reset where you want.
Or implement if block from remove_tokens_after_password_reset where you need.

Or the second way:
user = User.first
user.tokens = {}
user.save

I didn't check these ways, but you should try them.

With a app people are actually using and generating tokens on every request it seems kind of crazy that it defaults to keeping these around forever (if it actually is)? The gem's domain includes authorization, so I'd assume it would take responsibility for removing these.

It looks like this should be handling it:
https://github.com/lynndylanhurley/devise_token_auth/blob/72976d8d3a0cd8f24578138f9edb7e7930fb75b2/app/models/devise_token_auth/concerns/user.rb#L230-L237
Might be worth looking at the actual expiry and your server time and seeing if it actually meets these conditions?


fix some log noise

This downgrades two of the info! level log messages to debug.
They becomes quite noisy if fuse-backend-rs is used as a library. Logging it at debug! keeps them silent until log levels are increased.

It make sense, perhaps we can provide these info by other method?

You mean, you'd be fine to merge this in case there's ways to get out major, minor, capable, enabled as a consumer of the library, so you can log them out there?
If so, any preferences on the API?
major, minor already are constants:
major: KERNEL_VERSION,
minor: KERNEL_MINOR_VERSION,

... and stored in self.vers.
want is coming from the flags in passed by the user (so probably possible to calculate statically). The enabled flags are already available in InitOut, so maybe they can be stored in self too?

Hmmh, passing these flags up requires touching a lot of things. No code is arguably having a use for these values (otherwise it would have been exposed already), and the only usecase would seems to be logging it to the user (when they're /debugging/ fuse-backend-rs) so I'm thinking increasing the log level in that case is the right call.
The current situation causes every user of the library to get these log statements printed out to their CLI users, even if everything is fine. At least if the binary doesn't do some extra work on explicitly filtering out info log messages for the fuse-backend-rs crate.
Another example: A HTTP client library also doesn't print the source port it uses for the outgoing TCP connection whenever it opens one, at least not in the info/default log level. If users want (and use it in a log framework that allows crate-level log levels), they can crank up the debug level on that specific crate.

Indeed, should avoid output info level logs within a lib, but it's hard to change the log level to debug issue in production, maybe we can store pub capable in self? WDYT cc @jiangliu @bergwolf

Poke :-) Any chance we can land this?


Missing ULINT for read_structure_by_name

Is there any possibility to read a ULINT variables within a structure with read_structure_by_name? The documentation says structure_def doesnt expect pyads.PLCTYPE_ULINT.

I was able to read ULINT variables without any errors after adding PLCTYPE_ULINT: "<Q" to DATATYPE_MAP in constants.py.


chore: WIP update

Changes

upgraded the aragon sdk
changed default plugin for useNewDAO hook.

@pythonpete32 I am having some issues with the createDAO function on line 47.
I think, I have zeroed down the issue to the params I am passing for the TokenVotingInstallPlugin. But I haven't found out the fix to it yet.


Prevent URL displaying in Chrome Ghost drag image

If you have
Drag Me"
  as a dragable element Chrome displays a Ghost image that includes the URL. I've done lots of Google'ing and found various info on Ghost images, but no easy way to not have the URL included.
Interestingly jquery-sortable http://johnny.github.io/jquery-sortable/ doesn't show the URL however after studying the code & css and checcking in the debugger I can't work out how it resolves this issue.
I realize this isn't a html5sortable issue but since that's what I'm using I'm hoping someone here may be able to help.

Should get fixed by #77

This should be fixed by #77 if the issue persists with the new version v0.2.5 please reopen the issue.


Unable to view horizontal scroll bar in results

Issue Type: Bug
I do not see the horizontal scroll bar in the results pane after running a query. I am able to click in a cell and move to the right and see the extra columns, but I cannot scroll to the right either with a mouse or by click on a scrollbar.
Azure Data Studio version: azuredatastudio 1.0.0 (cab8f3e37c9ac4367e0dacef5185da08bd8021c2, 2018-09-20T22:04:35.897Z)
OS version: Windows_NT x64 10.0.16299

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-7300U CPU @ 2.60GHz (4 x 2712)

GPU Status
2d_canvas: unavailable_softwarechecker_imaging: disabled_offflash_3d: unavailable_softwareflash_stage3d: unavailable_softwareflash_stage3d_baseline: unavailable_softwaregpu_compositing: unavailable_softwaremultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: unavailable_softwarevideo_decode: unavailable_softwarevideo_encode: unavailable_softwarewebgl: enabled_readbackwebgl2: unavailable_off

Memory (System)
31.88GB (17.28GB free)

Process Argv
C:\Program Files\Azure Data Studio\azuredatastudio.exe

Screen Reader
no

VM
0%

Extensions: none

We've made several improvements to scrolling since this issue was opened.  Including improving some of the cases with multiple resultsets and double-scrollbars and increasing the size of the scrollbars.  Please let us know if you're still impacted by this issue.


Provide API to allow the capability to extend the trace extension

Currently, there is no way for another extension to customize the Trace Viewer and add features onto the existing views. Adding this capability would be very beneficial for cases where someone wants to add another view to the trace viewer that is specific to their use case and would not be acceptable to add it to the base extension. For example, some trace types may want to show raw bytes of an event; They can contribute a Raw Bytes view to the trace viewer from their extension, and using the API provided by vscode-trace-extension, it can communicate the selected experiment, selected event, etc. The calling extension can then update their view accordingly.
Adding an API for an extension is quite simple with the VSCode API:
https://code.visualstudio.com/api/references/vscode-api#extensions
The alternative is to use VSCode commands, but it may not be scalable as the API grows larger.
For the initial API, I am interested in obtaining the experiment UUID of the current panel open in Trace Viewer. This issue can track the discussion of this feature, and the work for providing an API.

I think it would be good to have some APIs to allow to other extension to add custom functionality on top of the base trace extension. If every customization had to be implemented to the base trace extension then the TSP would have to be often modified to allow custom behaviours driven by the server back-ends.
I'm wondering if the experiment UUID is all that is needed. Maybe the whole Experiment object would be useful when retrieving the current active panel.
Also, would it be useful to provide a callback mechanism, when an experiment  is selected? The experiment selected change is already propagated inside the base trace extension.
I think the extension API should also provide the sever connection settings (URL, port etc),  a way to get notification about server status, update server status etc. I'm also wondering if the base extension should allow adding custom menu items at different locations in the base trace extensions as well as providing custom query parameters for a domain specific data provider.
The latter suggestions for the extension APIs I see as future additions and not in scope of the initial extension API for the currently open panel. The initial API, however, should be a data structure where we can add additional APIs later on.


Add require for FT and NFT contracts regarding gas for transfer-and-call

A user from Telegram was reporting an odd error that I believe will confuse folks in the future.

This person was using NEAR CLI to call ft_transfer_call, which I believe will happen again in the future. My mistake was that I thought NEAR CLI had the default gas of 300 Tgas, when it's actually 30 Tgas.
It just so happens that 30 Tgas is the perfect amount for the first Promise to attempt to do subtraction, leading to zero attached gas for the *_on_transfer cross-contract call.
This PR simply adds a require statement that will provide feedback to the end user / contract that they'll need to attach more gas.
I've tested this and here are some perhaps helpful snippets to modify and test. You can build with the current master branch and then with this PR and test out the functionality.
near delete ft.demo.testnet demo.testnet
near create-account ft.demo.testnet --masterAccount demo.testnet
near deploy ft.demo.testnet --wasmFile res/fungible_token.wasm --initFunction new_default_meta --initArgs '{"owner_id": "ft.demo.testnet", "total_supply": "1000000"}'
near call ft.demo.testnet storage_deposit '' --accountId demo.testnet --amount 0.00125

near delete ft-new.demo.testnet demo.testnet
near create-account ft-new.demo.testnet --masterAccount demo.testnet
near deploy ft-new.demo.testnet --wasmFile res/fungible_token.wasm --initFunction new_default_meta --initArgs '{"owner_id": "ft-new.demo.testnet", "total_supply": "1000000"}'
near call ft-new.demo.testnet storage_deposit '' --accountId demo.testnet --amount 0.00125

near call ft.demo.testnet ft_transfer_call '{"receiver_id": "demo.testnet", "amount": "1", "msg": "merry xmas"}' --accountId ft.demo.testnet --depositYocto 1
near call ft-new.demo.testnet ft_transfer_call '{"receiver_id": "demo.testnet", "amount": "1", "msg": "merry xmas"}' --accountId ft-new.demo.testnet --depositYocto 1

Then I also fixed a small copy/paste issue where a constant was FT instead of NFT

Seeing as this is a teeny change and Phuong is on break, I feel good about merging this. 🎅🏼

Seeing as this is a teeny change and Phuong is on break, I feel good about merging this. 🎅🏼

Yup! I was just going to wait until the CI passed anyway :) 🎅🏼


Placement of Help Button in sos report

This placement of Help button looks quite abrupt .It can be relocated to a better place.

go for it!

please assign this task to me under gssoc'24

I have recently completed my javascript , html and css course i can help you with this issue . could you assign me with this task?
I'm under GSSOC'24


Show a warning when a subsystem fails to initialize.

Users won't know what's going on if we shows a message only in debug mode.
Signed-off-by: David Calavera<EMAIL_ADDRESS>

Thanks @calavera
LGTM.

@calavera come to think of it, the reason it was added under Debug is to make sure we dont throw this warning message in Docker Daemon when not using the experimental channel.
Since the multi-host networking is only part of experimental and the kv-store configuration is required only for multi-host networking, we didnt want to alarm the user for a lack of configuration which is not even required for non-experimental docker.
Having said that, If am not wrong, we will not see this message in the non-experimental version of docker daemon due to the mere fact that none of the cfgs are initialized by default. But, I think that is just a side-effect than by design.
Hence, It will be good to move this message only for experimental version.

Now that we've shipped 1.7, this code should not be released again until 1.8, or a experimental version. I think it's fare to assume that these features will land in the final 1.8 release. There is no need to make the log special, it should be a warning every time.

@calavera from an user standpoint, Warn is a stronger message. my concern with changing the logging level is that we will be unnecessarily warning the user even if the user intention is to NOT use the data-store.
BTW, we will fail with appropriate message anyways if the user tries to create a network or service using overlay driver or any other driver that actually has a GlobalScope compared to LocalScope that default bridge driver belongs to.
So, I think having a Warn message for a use-case that the customer really dont want to try seems excessive for me. The only reason we have a Debug message here is for debugging purposes. But I wont mind even removing this message during the init, because we will anyways throw the error message when the user tries to use the kv-store without proper configuration.
WDYT ?


fediplay could write out a playlist during a session

We already leave media lying around, why not also output a playlist? Then you could relive a fediplay session anytime.

I don't have any plans to implement this. If someone thinks it would be cool, we can reopen this issue.


Bug report: the rest-sidebar can not be edited by multiple lines

When I use cmd+enter, can not jump to a new line
0.8.7

It was fixed by Logseq Team in Logseq v0.8.9.


EmrEtlRunner: add support for shredding from Kinesis S3 Loader's enriched event output

The issue here is the absence of a run= folder to work with.

Hey @alexanderdean, @BenFradet, here's what I come up with regarding this feature. Would like to brain-storm it and change approach if you have objections as it looks like many EER UX decisions were made.
Basically, this commit:

Adds new buckets.enriched.stream path in config.yml
Makes buckets.raw optional as with this "stream mode" buckets.enriched.stream is source of data
If buckets.enriched.stream is defined: enrich step doesn't happen at all; instead pipeline starts from special stage-enriched-stream step. staging and archive_raw also never happen.
So instead staging,enrich,shred... we now have stage-enriched-stream,shred..., where stage-enriched-stream just copies data to shredder's HDFS directory

This also affects new RDB Loader in many ways mostly because we cannot use etl_tstamp anymore.

This sounds great @chuwy. Let's discuss with @BenFradet straight after our weekly.


failed calling webhook

Describe the bug:

Expected behaviour:

Steps to reproduce the bug:

Anything else we need to know?:
Environment details:: AWS ubuntu

Kubernetes version: 1.21
Cloud-provider/provisioner: AWS
cert-manager version:  v1.5.3
Install method: e.g. helm/static manifests: static

/kind bug

Duplicate of #4425


Step 8 Doesn't Work

It looks like there's nothing to click or edit, it's just text.

Tried again. It works. Thanks


Namechange for Atom

Hey! I was wondering if you could change the name of the package to follow the standard of language-NAME on the package manager

Do you by any change know if this can be done without impact already installed instances? In other words does this mean people will have to reinstall the packages due to the fact it would be renamed?

As far as I know it should do it automatically, though I haven't looked too
much into it
On Mon, 13 May 2019, 10:42 way2muchnoise<EMAIL_ADDRESS>wrote:

Do you by any change know if this can be done without impact already
installed instances? In other words does this mean people will have to
reinstall the packages due to the fact it would be renamed?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/way2muchnoise/ZenScript-Highlighter-Atom/issues/3#issuecomment-491752578,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABSAYSYFC3EVD73XI4GV4Z3PVEZXVANCNFSM4HMMAF2Q
.


Release contains Java 17 bytecode

hi! first off - thanks for releasing this great library!
I was wondering if it'd be possible to publish using JDK8 or JDK11? Our builds fail when ran under JDK8:
https://github.com/theiterators/kebs/actions/runs/5800590124/job/15723015359?pr=302#step:6:419
Thanks!

Sure, no problem. I just had forgotten about that aspect when configuring the project.
I'll publish a new release sometime within the next 24 hours.


Issue in tracking of enabled/disabled upgrades

Issue:
We have Better Scanner and Night Vision unlocked.
We decided to do some configs for Night Vis and disable Portable TP.
After loading back in, our upgrades became an entire different set.
Before: Better Scanner + Night Vision
After: Back Muscle + Discombobulator
Expected:
Loading a game should be able to keep track which config is enabled or not.
I didn't read the codebase but I'm thinking the possible cause for this is some sort of array-related behavior
Replication Steps:

Get Better Scanner and Night Vision
Save at orbit and exit
Set Portable TP to false
load the save
type unlocks in terminal, displays Back Muscle and Discombobulator
attempt use Night Vision, not accessible.
exit
reconfig Portable TP to enabled
load the save
type unlocks in terminal, we have Better Scanner and Night Vision again

Note: We've made sure that its indeed the same save we're playing.

I've read the last part of readme.md so I'll add some more info.
I am the host and I was the one who initiated both of the upgrades purchase.

Hi,
This is a great bug report, thank you. I think you're correct that disabling them and removing them from the list of registered network prefabs is causing the ships item saving system to now be holding onto irrelevant indices.
I am about to put up a nightly build for v1.5.0 on here and it should go live on thunderstore tomorrow. I'll make sure to test this case to ensure it is fixed and close this issue.
Thanks again.


Match substrings with text=<regex> and id=<regex>

It would be nice to be able to have tapOn: button find a button with text == My button with a long name. Today we need to write tapOn: .*button.*. Same request for the id matcher.

Perhaps adding regex example in the documentation will help as well.
For example to tap on complex buttons, e.g. button with image: "Click Me 😃" we need to use:
tapOn: .*Click Me.*

I'd be happy to write the docs that explain that regex is supported. I found this very confusing at first.
Maybe an addition to:
https://maestro.mobile.dev/api-reference/selectors


The biggest 3 things that golang should do first.

android/ios
bigdata
GUI
I think these 3 things should have the highest priority.
If golang doesn't notice this, it will lost the time.

Thanks for the comment.
Works are already underway for the first and 3rd point.
x/mobile and x/exp/shiny, respectively.
bigdata is too board a term for me to comment.
This issue is not quite actionable, so closing.
If you want to propose the addition of a specific feature
or package, please elaborate.

Do you even bigdata?

I need to know if golang is webscale ASAP.


Feature/3054 update hero comp to use muiv5 style engine

Fixes # BLUI-3054
Changes proposed in this Pull Request:
Update styles to use Mui-v5 Styled components
Update component to extend sx prop
Update component docs for Hero
Test:
There is a showcase branch feature/3050-channel-value-mui-v5-styles-update that you can use to test this.
When testing, ensure that there are no breaking changes and that the old API properties still work the same and that you can still style things with inline styles as well as using our classes prop.
Also, test out the new sx prop extension. You can apply styles to the root by using sx directly on the component as well as by targeting nested items by the generated classnames.
Here is an example component that you can take parts from to test:
<Hero sx={{ margin: '1rem' }} icon={<GradeA fontSize={'inherit'} />} label={'Efficiency'} style={{ flex: 1 }} />

The icon styles aren't working as they were before.

Yep just noticed this; icon styles again.

So yesterday I was testing out the nested style rules can came across this observation:
This code snippet applies a red font onto a Hero's icon & works in the pre-MUI5 style migration:
<Hero
    classes={{
        icon: classes.red
    }}
    icon={
        <GradeA />
    }
    label={'Efficiency'} />

However after moving over to MUI5 styles, these overrides are ignored since the MUI5 styles introduce an additional layer of specificity to their style definitions.  The font isn't red anymore & the style hierarchy now looks like this:

Moving the icon's style override directly onto the component fixes the issue:
icon={<GradeA className={classes.red} fontSize={'inherit'} />

Unfortunately this problem is not only applied to the Hero component; it seems user-provided style-overrides via classes prop will be ignored within all non-root elements.

After talking with Joe I'll make some follow-up stories to revisit the ChannelValue, Hero, EmptyState component so we can make sure these nested overrides still work via the classes prop.


Add man-db to Debian common package list

man-db isn't included in the Docker node images:

https://github.com/nodejs/docker-node/issues/209#issuecomment-234426404
You'll probably find that man is not installed in a majority of the official Docker images, particularly ones that build off the official debian image. Presumably the rationale is that Docker containers are a form of app encapsulation and not something you typically log into and use interactively.

Since a dev container is used interactively, it would be nice if man-db was installed out of the box. git --help doesn't work without it, among other things.

Thanks for the feedback! Yes, historically this has been excluded to keep image sizes down. However, I do find myself wishing it was there as well and it looks like it's about a 6mb, so this is a reasonable request. We can do that I think.

Updated!

Thanks! (Thanks also for the new javascript-node:12-buster - between that and this, I no longer need my own Dockerfile at all. Loving the continued improvements to the developer experience.)


sharee does not receive a mail notification when sharer changes permission to a shared resource

Describe the bug
User does not get mail notification when shared file/folder's permission is changed
Steps to reproduce
Steps to reproduce the behavior:

Alice creates a resource textfile.txt
share the resource to Brian using command curl -kv -X POST https://localhost:9200/ocs/v1.php/apps/files_sharing/api/v1/shares -d 'shareType=0&path=/testfile.txt&permissions=17&role=viewer&shareWith=brian' -u alice:123456
Brian accepts the share
Alice changes permission of the resource

curl -kv -X PUT https://localhost:9200/ocs/v1.php/apps/files_sharing/api/v1/shares/<space-id> -d 'permissions=19&role=editor' -u alice:123456

Expected behavior
Brian should receive mail notification when shared file/folder's permission is changed by Alice
Actual behavior
Brian does not receive mail notification when shared file/folder's permission is changed by Alice
Setup
Ocis : latest master
Additional context
Add any other context about the problem here.

the behaviour is same for the project spaces too now

According to https://github.com/owncloud/ocis/issues/6649#issuecomment-1707919956 this comment this is a feature request and not an issue so closing this
cc: @ScharfViktor


Entering the URL of the Jupyter server is too deep

I want to connect to the remote Jupyter server. I press the Select Kernel button:

I'm presented with a text field that looks like a good place to enter the server URL but that doesn't work. Instead, I have to press Select Another Kernel.

I'm presented with another text field that looks like a good place to enter the server URL but that also doesn't work. Instead, I have to press Existing Jupyter Server.

I'm presented with yet another text field that looks like a good place to enter the server URL but that again doesn't work. Instead, I have to press Enter the URL of the running Jupyter server.

Finally, I'm presented with a text field that will accept the URL.

Overall, to get to the field that accepts the URL I had to skip another 3 text fields. I think it would be nice if any of these intermediate text fields would directly accept the URL.

@kryptan thanks for filing this issue,
we have removed the last step from the kernel picker, that should reduce the number of steps to 3 steps instead of 4.
Plese update to the latest version and let me know how that works.


rework migrations between differerent databases

This is a follow-up issue for the migration to Hiqlite.
It will take care of the migrations between different databases. The whole MIGRATE_DB_FROM will be reworked internally to make it less error prone and work with Hiqlite.

The following migrations have been tested and are working now:

sqlite -> hiqlite
sqlite -> postgres
hiqlite (sqlite) -> postgres
postgres -> hiqlite

The migrations are working fine.
At the current stage, Rauthy will always create (and need) a Postgres connection, even when using Hiqlite.
This is due to the design from before and the connection pool being inside the AppState.
This needs to be dropped and moved out, which is already prepared, but dropping it completely will result in many things that need fixing, so this will be done in another PR again.


#4607 Add Tailscale to SSH menus

adresses #4607

tested to be working also after a recover/update - merging


Fix duplicate System logging issue (#747)

Fix for https://github.com/Azure/azure-webjobs-sdk-script/issues/747

:shipit: !


Compiled list of material that could be reuse

I've moved the SPARK books to the list of skipped items.


Returned 204 status code for no snippet content

Overview
This PR fixes #1200
Returns 204 HttpStatusCode

Returns 200 HttpStatusCode

Testing Instructions

Checkout the branch associated with this PR
Run the project locally.
Using postman to call the PowerShell snippet Api with request bodies expected to return either 204 or 200 http status codes. For example: POST /v1.0/external/connections/contosohr/schema HTTP/1.1 Host: graph.microsoft.com Accept: application/json is expected to return a 204 while GET /v1.0/external/connections/contosohr/schema HTTP/1.1 Host: graph.microsoft.com Accept: application/json is expected to return a 200

While this fixes the response status code from DevX API, it doesn't address why the snippet for POST /external/connections/{externalConnection-id}/schema is missing. The API path doesn't exist in the OpenAPI document due to missing annotation in the metadata.
As per the API reference doc, POST https://graph.microsoft.com/v1.0/external/connections/contosohr/schema is valid path that should be supported - https://learn.microsoft.com/en-us/graph/api/externalconnectors-schema-create?view=graph-rest-1.0&tabs=http#example-register-custom-schema-asynchronously.
Please open an issue in the metadata repo to track this.


Game Over

For now, the game cannot be lost. It could be interesting to add different ways to lose the game based on the choices made by the player.

There is also no ending screen. When you win, the game closes.


Immediate Playback using WebAudio Backend

Wavesurfer.js version(s):
2.0.5
Browser version(s):
Chrome Mac 66.0.3359.181 (Official Build) (64-bit)
Description of the Issue:
When loading a large audio file (20mb+) using the default WebAudio backend, the playback can only begin after the whole audio file has been downloaded. When using the MediaElement backend, the playback is immediate, however, the timing information returned by wavesurfer.getCurrentTime() is imprecise, for long audio files (45min +) the offset is up to a few seconds. Four our purposes, an application that synchronizes text with the audio being played back, we need immediate playback, but precise timing information.
How can we achieve this? Any help would be greatly appreciated!
What we have tried:

we have precomputed the peaks files and load them with wavesurfer.load('audioFiles/' + this.props.recordId, peaksData);. This does not change the problem that we have to wait until the whole file has finished downloading until the playback can begin.
we tried to correct the offsets by reading the timing information directly out of wavesurfer.backend.media.currentTime in case wavesurfer.backend.params.backend=='MediaElement'. This did not work, consistently. I.e. sometimes the timings were close to the true values, most of the times, they were off quite a bit.
loading a small version of the audio initially, only to replace it with the full version of the audio file by calling wavesurfer.load(url_to_large_file) immediately after calling wavesurfer.load(url_to_small_file). This did not work, as the second call to wavesurfer.load() aborts the playback of the small file.
same as above, but this time by downloading the audio using ajax, before trying to load it with wavesurfer.loadBlob(). Sadly, loadBlob() did not read the audio data we provided and resulted in the error message: Uncaught (in promise) DOMException: Unable to decode audio data (code below)

         loadInBackground(jsroutes.controllers.SpeechFileController.getAudioFile(this.props.callId))
            .done( (data, result, response) => {
                //console.log(this.state.wavesurfer);
                let blob = new ArrayBuffer(data);
                this.state.wavesurfer.loadArrayBuffer(blob);
            })
            .fail( (response) => {
                console.log("problem loading audio file:", response);
            })

What we're looking to try next:

try to down sample the mp3 we need to load, with the side effect of having lower quality audio in our application.

Is the wavesurfer.getCurrentTime() returning the wrong time, or is the playhead on the waveform at the wrong spot? I'm building something that needs to work with long files (up to a couple hours) as well, so both of these things are really important.
In other words, is the time being returned by wavesurfer.getCurrentTime() actually changing at a rate that's not consistent with the specified playback rate, or is the waveform visualization just not lining up with what you're hearing?


⚠ ClamAV has degraded performance

In 482c18b, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 69 ms

Resolved: ClamAV performance has improved in 653c218.


Tests missing!

Thanks man! Exactly what I needed.
Has this been used in production?
What's Goyoo OEM project?

It's very nice that this project can help you.
Goyoo is the company I'm woking in. OEM is the project I'm working on in Goyoo, that's not publicly open yet.
Tests are coming soon, thanks for reminding.

Test is added now


Remove hardcoded dependencies from install instruction

Motivation: installation instruction is often updated at the same time
when SDC moves to new version of dependencies, and while building SDC
from source works, installing from pre-built binaries might fail since
package is uploaded to intel/label/beta only after all validation
checks.
Fixes #968

Need to verify that pip is working too..

As I remember, these changes "pyarrow=2.0.0 pandas=1.2.0" were added to fix issue when old sdc version is installed instead of fresh because old sdc required less changes in environment then new...
If now latest version is installed using updated commands - it is fine.
Regarding pip - maybe we need to remove these instructions while SDC new wheel packages is ready, because they can install only old version which I think not up-to-date.

Hello @kozlov-alexey! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file buildscripts/test_conda_install.py:

Line 40:41: W605 invalid escape sequence '\s'


Enable python 3.11 for unit tests

close for now since ray 2.4 is not compatible with python 3.11
https://github.com/ray-project/ray/issues/33864


interactively execute multiple commands

On my computer, it takes a long time to boot Ruby and do all the requires.
So I made this interactive loop thing that lets you run multiple commands.

This might be an interesting use case, but right now all the tests fail. Is this still worth pursuing? Or should this be closed?

I no longer have the problem that this change solves.


bug: When side menu is disabled, expose-aside-when="large" still triggers

Type: bug
Platform: all
In the 'ion-side-menu' when is-enabled="false", expose-aside-when="large" is still triggering. So in some sections I am disabling the menu but if the width is greater than 768px the menu is still appearing.
Codepen: http://codepen.io/matiastucci/pen/oHapg

The expose-aside-when feature is pretty poorly thought out/configured IMO... There needs to be an easier way to override when to show/hide the exposed side menu, beyond using media queries -- We need to be able to control this easily with JS as well! I think the directive needs another attribute that takes a true/false expression and shows or hides itself based upon that.
As an example: If a user hasn't logged into my app yet, I may not want to have the side menu options exposed (they might be dynamically set according to the user). In this case, having expose-aside-when set to true will unfortunately ensure that the side menu is always shown which means I can't use it without writing my own code for fighting against the way it currently works.

I'm also having the problem. I didn't found a proper way to close / toggle a side menu which have the expose-aside-when directive. Using $ionicSideMenuDelegate.isOpenLeft() always returns false. This is inconsistent and should be changed.
Thanks

noKid
http://pain-advice.com/betonmarkets-front
24.03.2015, 14:12, "noKid" notifications@github.com:I'm also having the problem. I didn't found a proper way to close / toggle a side menu which have the expose-aside-when directive. Using $ionicSideMenuDelegate.isOpenLeft() always returns false. This is inconsistent and should be changed.
Thanks
—Reply to this email directly or .

it's still visible even though disabled

solution of "When side menu is disabled, expose-aside-when="large" still triggers" this problem is in Latest stable version 1.0.0 is replace below line in ionic.bundle.js lines "   sideMenuCtrl.exposeAside($window.matchMedia(mq).matches); sideMenuCtrl.close();" with " if ($attr.isEnabled == true) {sideMenuCtrl.exposeAside($window.matchMedia(mq).matches);} else {sideMenuCtrl.close();}" . it will resolved this problem

Tis bug is still present in the v1.1.0

I also have this bug when using expose-aside-when and it's really annoying. Did someone found a workaround for this?

hi,
in version 1.3 I am still getting the same behavior (exposed-aside-when seems to override is-enabled)
@lordgreg approach solves my use case, but this is still a bug / inconsistency in my opinion.


Draft: fix(alias): removed pointer for array alias types

Pointers were used on aliased arrays. This made casting difficult because after the cast handler expected pointer to the cast type.
Because slices are still passed as references and there is almost no allocating happening under the hood I thought it would be OK to change them to nonpointer.
Example before:
	apiPets := conv.Map(pets, func(pet pet.Pet) api.Pet {
		return api.Pet{
			ID:   pet.ID,
			Name: pet.Name,
			Tag:  api.NewOptString(pet.Tag),
		}
	})

	return conv.ToPointer(api.FindPetsOKApplicationJSON(apiPets)), nil

Example after:
	apiPets := conv.Map(pets, func(pet pet.Pet) api.Pet {
		return api.Pet{
			ID:   pet.ID,
			Name: pet.Name,
			Tag:  api.NewOptString(pet.Tag),
		}
	})

	return api.FindPetsOKApplicationJSON(apiPets), nil

If you prefer the pointer I could create a new helper func to be generated along with alias?
Your call.
Thanks!

Closing this, found out I'm overthinking this.

Actually this would be pretty helpfull, pointers for aliased types are annoying.

Ok, I'll reopen this MR after I fix something.
Wasn't sure if you even want this change and I didn't want to "force" personal opinions.


Add stack operations in Java using ArrayList

Add stack operation using ArrayList like push and pop

Created the PR

@georgioupanayiotis  Please create another file as this maintains the repository that can be understood by new comers


Weather calculation issue (Eureka)

Weather calculations aren't working correctly for me in Pagos nor Anemos.  For example, zoning into Anemos with rain as current weather, cactbot's eureka ui is displaying sun for the next 26 minutes with gales in 64 minutes.  Using Garlandtools skywatcher, the weather is showers for the next 15 minutes and double gales after.  This weather issue existed at least a few days before the latest patch from what I remember.

Thanks, I'll take a look.  The weather had been working just fine for me in pagos before the patch.  I'll double check if they modified the weather tables.

Thank you!

Also in Pagos


compatibility with symfony 2.4 or 2.5

Is it planned ? that prevents to do it right now?

It is tested with Symfony 2.4 and 2.5, on PHP 5.3 to 5.6 and HHVM. I even use it successfully in a 2.6 project.
Have you found a bug that is not covered by the test suite? Is there an integration problem?

If it works, you're should change composer.json require section:
  "require": {
        "symfony/symfony": "~2.6",
    }

Since 2.3 (LTS) will be maintained until May 2016, I don't see the point in making it impossible for everyone under 2.6 to install the library. The version constraint ~2.6 means >=2.6.0,<3.0.0, there's just no gain.
If there's a bug then I'm all ears, but I won't bump the minimum version for no reason.


Duplicate serviceBroker calls causing excessive work / delays

Traced to transition hook onEnter, which fires after the header has loaded.  This causes the cache to be emptied and subsequent calls to be remade.  Switching to onBefore resolves the issue (https://ui-router.github.io/guide/transitions#lifecycle-events).
Also, the cache was seeing duplicates as sometimes entity references had superflous fields which causes the serialized versions of them not to match up.


Send negotiated config as JSON, apply client-side foveation

Currently the client-side foveation doesn't seem to be working.

Client-side foveation is still not working, merging as it doesn't cause any issues.


The average time per job in the status is wrong

It decreases with every request and seems to be wrong (the print takes on average ~10 seconds but the shown time is 2,5 seconds).
Hopefully we will get rid of the average time in the status with #243, but we should make sure that the average time is calculated correctly for the metrics.

I'm not sure that wrong but the first job take many more time than the others we should probably remove it from the stats ?


Add property method to extract object properties

Hello @mmkal! 👋
Thank you for your hard work on the expect-type library! I really appreciate the effort you put into maintaining such a valuable tool for the community.
As I’ve been using the library, I noticed that it could be even more powerful with the addition of a new method. This pull request proposes the addition of a property method that allows users to extract specific properties from an object and perform type assertions on them.
The property method will enable users to easily check the types of individual properties within an object, enhancing the library's functionality and improving its usability.
For example:
const user = {
  name: 'Alice',
  age: 30,
};

expectTypeOf(user).property('name').toBeString();
expectTypeOf(user).property('age').toBeNumber();

Let me know if you have any questions or need further changes!

Have you tried the toHaveProperty method that we have? It seems like they might be very similar if not identical.

Have you tried the toHaveProperty method that we have? It seems like they might be very similar if not identical.

I’ve had a moment of clarity and realized I might have been a bit too eager with my pull request. 😅 I apologize for the oversight regarding the toHaveProperty method!
I’ll go ahead and close this pull request. Thank you for your understanding, and keep up the fantastic work on the library!

Closing this for now - thank you for taking the time to create a PR! And if there's anything you think could be added to the docs to make it clearer that you can use .toHaveProperty(...) in this way, please feel free to submit another one (this might be in the form of a usage.test.ts change which updates as you've seen in this PR).


Trouble creating transaction

I'm trying to create a dynamic version of this query so that I can delete multiple items with a single transaction. It appears to render properly, but returns an error Invalid argument (sql): Had an unexpected trailing statement
BEGIN TRANSACTION;

DELETE FROM table_set
WHERE name="name" AND type="MyBuiltEnum" AND value="FOO";

DELETE FROM table_set
WHERE name="name" AND type="MyBuiltEnum" AND value="BAR";

DELETE FROM table_set
WHERE name="name" AND type="MyBuiltEnum" AND value="BAZ";

COMMIT;

Here's what I have so far
    final table = 'table_set';
    final values = ['FOO', 'BAR', 'BAZ'];

    execute('''
      BEGIN TRANSACTION;

      ${values.map((_) => 'DELETE FROM ${table.set} WHERE name=? AND type=? AND value=?;').join('\n')}

      COMMIT;
      ''', [
      for (final value in values) ...[name, type, value]
    ]);

However, it produces this error
  Invalid argument (sql): Had an unexpected trailing statement.: "      BEGIN TRANSACTION;\n\n      DELETE FROM main_set WHERE name=? AND type=? AND value=?;\nDELETE FROM main_set WHERE name=? AND type=? AND value=?;\nDELETE FROM main_set WHERE name=? AND type=? AND value=?;\n\n      COMMIT;\n      "
  package:sqlite3/src/ffi/impl/database.dart 242:11  DatabaseImpl._prepareInternal
  package:sqlite3/src/ffi/impl/database.dart 109:19  DatabaseImpl.prepare
  package:sqlite3/src/ffi/impl/database.dart 97:20   DatabaseImpl.execute
  package:kv_sqlite/src/extensions.dart 101:5        SetDatabaseExtensions.deleteSetElements
  package:kv_sqlite/src/kv_set.dart 64:12            KvSet.removeAll
  test/kv_sqlite_test.dart 128:13                    main.<fn>.<fn>
  test/kv_sqlite_test.dart 85:33                     main.<fn>.<fn>

Extracting from there, the query generated is:
      BEGIN TRANSACTION;

      DELETE FROM main_set WHERE name=? AND type=? AND value=?;
DELETE FROM main_set WHERE name=? AND type=? AND value=?;
DELETE FROM main_set WHERE name=? AND type=? AND value=?;

      COMMIT;

and the generated arguments are
["name", "MyBuiltEnum", "FOO", "name", "MyBuiltEnum", "BAR", "name", "MyBuiltEnum", "BAR"]

which should render as
BEGIN TRANSACTION;

DELETE FROM table_set WHERE name="name" AND type="MyBuiltEnum" AND value="FOO";
DELETE FROM table_set WHERE name="name" AND type="MyBuiltEnum" AND value="BAR";
DELETE FROM table_set WHERE name="name" AND type="MyBuiltEnum" AND value="BAZ";

COMMIT;

sqlite3's API can only prepare one statement at a time (and a PreparedStatement instance from this package wraps one such statement too). In your case though, there are five statements (begin, 3x delete, commit).
If you give sqlite3 multiple statements in a prepare call, it will only scan the first statement and stop there (so the native sqlite3 APIs would only give you BEGIN TRANSACTION and silently ignore the rest). This is super confusing, so this package attempts to check that the query you issued ends with the first statement (so you get an exception instead of silently ignored statements, an improvement IMO).
Now, how to solve this. A neat thing about prepared statements is that you can prepare (= compile) them once and run them multiple times, which comes in handy for your deletes. So I'd probably write it like this:
final delete =
    db.prepare('DELETE FROM table_set WHERE name = ? AND value = ?;');

db.execute('BEGIN;');
delete.execute(['MyBuiltEnum', 'FOO']);
delete.execute(['MyBuiltEnum', 'BAR']);
delete.execute(['MyBuiltEnum', 'BAZ']);
db.execute('COMMIT;');

delete.dispose();

Obviously you probably need something more dynamic with loops for delete.execute, but I think you'll get the point :D Statements without any parameters can just be directly executed on the database (doesn't make too much sense to prepare those).
Alternatively, you could use prepareMultiple to get a list of statements from a single string, but using that will probably lead to a more complicated implementation for your case.


[UnderlineNav]: VRT Snapshot and interaction Tests (Playwright)

This PR is a follow up on @joshblack's playwright PR
and covers UnderlineNav related next steps.

Introduce primer layout primitives
Tests for states of UnderlineNav on available Primer layout breakpoints and themes (5 breakpoint and 9 themes)
Interaction tests

Take a look at the What we look for in reviews section of the contributing guidelines for more information on how we review PRs.

Closing this because we have the e2e interaction test in this PR


browserpass: 2017-04-11 -> 1.0.5

Motivation for this change
See https://github.com/rycee/home-manager/pull/16#discussion_r121245588.
This also updates the NixOS module.
Things done

[x] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Looks good to me but I've never used this so cannot say for certain ☺
CC @pajowu

Any opposition to a merge of this PR as it is now?

ping @fpletz


Install with ES 5 on Windows

Hello
I am trying to get this installed on ES 5.0 on a Windows server. I try the install command line before but cannot get it to work.

Where do I copy the files to? I tried putting a jettro folder within the bin folder and ran it from there but I keep getting the following error:

Error: unknown plugin jettro\elasticsearch-gui
Help would be appreciated.

Version 5 is not yet supported. In version 5 they have disabled the
standard site plugins. So you cannot run it as a plugin.
On Fri, Oct 28, 2016 at 7:08 PM, arnoldvilleneuve<EMAIL_ADDRESS>wrote:

Hello
I am trying to get this installed on ES 5.0 on a Windows server. I try the
install command line before but cannot get it to work.

Where do I copy the files to? I tried putting a jettro folder
within the bin folder and ran it from there but I keep getting the
following error:

Error: unknown plugin jettro\elasticsearch-gui
Help would be appreciated.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/jettro/elasticsearch-gui/issues/109, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AAJIt5qXQ-Hgv3MijJqX0AS-ga2nBMhKks5q4iwAgaJpZM4Kjp44
.

--
Jettro Coenradie
http://www.gridshore.nl

Are there any plans to support version 5?

I have the plans, but I am not sure how I am going to do it. Have to change
a few things as we cannot use it as a plugin any longer. Also not sure
about creating it as a Kibana plugin or different.
On Thu, Mar 2, 2017 at 3:16 PM, MetalArend<EMAIL_ADDRESS>wrote:

Are there any plans to support version 5?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/jettro/elasticsearch-gui/issues/109#issuecomment-283664422,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAJIt0P0HwKBo-KWi7Fbc2uyuFRLsAi2ks5rhs9agaJpZM4Kjp44
.

--
Jettro Coenradie
http://www.gridshore.nl

Just ran into this also. Will keep an eye out for updates. Thanks.

I hope my small toy can help a little
https://github.com/xzer/elasticsearch-plugin-hosting
which is a old style site plugins hosting tool.
so the current issue is, how we can make the plugin work with the changes from es 5.


Helpful Count

The scraping of helpful isnt working and is always returning 0 as the output as mentioned in the exception

@bhavanal Can you provide more info about the exception you're seeing? Is it related to #3?

replace your selector with this
helpful = review.find_element_by_class_name(
'voteHelpful').find_element_by_class_name(
'count').find_element_by_tag_name('span')


server: rename some functions and variables in server

The server.go function names are confusing - With the introduction of new DNS support the earlier names used for securegRPC server with DNS certs are confusing. This PR renames them to secureGrpc*** to avoid conflicting with the DNS support.

@ramaraochavali: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio
6437bd3bc55c7b4063975794dbc639637db2cbc5
link
/test lint_istio

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


Install Class Diagrams Component Missing Step

In section "Add class diagrams to projects", steps to install the component are missing a step: user must click on Modify after opening the VS Installer if they have already installed Visual Studio (this is why the screenshot in the documentation says "Modifying" at the top). (There's probably a different step missing if the component is being installed as part of a fresh VS install.)

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

ID: 1a723ca4-ee71-386d-aa7a-461dd565bc84
Version Independent ID: fa9856a9-4bab-805d-c5a1-bd8e506e4fb0
Content: Design, visualize, & refactor with Class Designer - Visual Studio (Windows)
Content Source: docs/ide/class-designer/designing-and-viewing-classes-and-types.md
Product: visual-studio-windows
Technology: vs-ide-general
GitHub Login: @ghogen
Microsoft Alias: ghogen

Thanks, it looks like this is fixed now.


GeoJSON aoi upload should validate coordinate system is lat/lon

According to http://geojson.org/geojson-spec.html#coordinate-reference-system-objects a geojson object without a 'crs' key must be in geographic coordinates. So instead of inspecting the uploaded coordinates to -180,180 bounds we should just specify that only geographic coordinate geojsons are allowed.


aerial mapper building error, Eigen version mismatched.

Hi,
I am compiling aerial mapper and faced following compile error.
Errors     << aerial_mapper_dsm:make /home/yourID/workspace/indigo_catkin_ws/logs/aerial_mapper_dsm/build.make.001.log
/home/yourID/workspace/indigo_catkin_ws/src/aerial_mapper/aerial_mapper_dsm/src/dsm.cc: In lambda function:
/home/yourID/workspace/indigo_catkin_ws/src/aerial_mapper/aerial_mapper_dsm/src/dsm.cc:172:31: error: no match for 'operator=' (operand types are 'Eigen::internal::enable_if<true, Eigen::IndexedView<Eigen::Matrix<float, -1, -1>, double, double> >::type {aka Eigen::IndexedView<Eigen::Matrix<float, -1, -1>, double, double>}' and 'double')
         layer_elevation(x, y) = idw_height;

It looks Eigen version issue and I am using Eigen 3.3.90 on Ubuntu 14.04.5 ROS-indigo.
Have you encounter similar issue before?
Best,

Hi @inkyusa
The framework is tested with https://github.com/ethz-asl/eigen_catkin (see https://github.com/ethz-asl/aerial_mapper/blob/master/install/dependencies.rosinstall#L4) which currently uses Eigen 3.2.10: https://github.com/ethz-asl/eigen_catkin/blob/master/CMakeLists.txt#L88
I haven't seen this issue, does layer_elevation(x, y) << idw_height; fix it?
Best


Dark theme

Added Dark Theme
Changes made

Added theme icon at the home page app bar for changing theme
Added dark theme same as Hacktober dark theme
Added small padding on the container on The home page that contains search bar to beautify it.

How does the solution address the problem
This PR will add an dark theme to the app same as the official dark theme of Hacktoberfest website.
(You can delete the screenshots folder that contains screenshots for now after accepting PR)

Great work @AwsmAsim. Thanks for your contribution


IndexError: tuple index out of range

Getting the following error. Any suggestion to solve it?

2018-06-04 02:49:35,663 [MainThread  ] [INFO ]  Writing summaries.
2018-06-04 02:49:35,664 [MainThread  ] [INFO ]  Processing summaries. Saving system files to /tmp/tmp2x4ur084/system and model files to /tmp/tmp2x4ur084/model.
2018-06-04 02:49:35,664 [MainThread  ] [INFO ]  Processing files in ./data/system_summaries.
2018-06-04 02:49:35,664 [MainThread  ] [INFO ]  Processing test.txt.tgt.tagged.
2018-06-04 02:49:35,717 [MainThread  ] [INFO ]  Saved processed files to /tmp/tmp2x4ur084/system.
2018-06-04 02:49:35,718 [MainThread  ] [INFO ]  Processing files in ./data/model_summaries.
2018-06-04 02:49:35,718 [MainThread  ] [INFO ]  Processing predicted_summary.txt.
2018-06-04 02:49:35,777 [MainThread  ] [INFO ]  Saved processed files to /tmp/tmp2x4ur084/model.
Traceback (most recent call last):
File "seq_ats/test_pyrouge.py", line 9, in
output = r.convert_and_evaluate()
File "/home/wasiahmad/software/anaconda3/lib/python3.6/site-packages/pyrouge-0.1.3-py3.6.egg/pyrouge/Rouge155.py", line 367, in convert_and_evaluate
rouge_output = self.evaluate(system_id, rouge_args)
File "/home/wasiahmad/software/anaconda3/lib/python3.6/site-packages/pyrouge-0.1.3-py3.6.egg/pyrouge/Rouge155.py", line 334, in evaluate
self.write_config(system_id=system_id)
File "/home/wasiahmad/software/anaconda3/lib/python3.6/site-packages/pyrouge-0.1.3-py3.6.egg/pyrouge/Rouge155.py", line 318, in write_config
self._config_file, system_id)
File "/home/wasiahmad/software/anaconda3/lib/python3.6/site-packages/pyrouge-0.1.3-py3.6.egg/pyrouge/Rouge155.py", line 271, in write_config_static
id = match.groups(0)[0]
IndexError: tuple index out of range

This error means that pyrouge cannot find document IDs in your file names when trying to match system and gold summaries. Your filenames should follow a pattern as explained here:
https://github.com/bheinzerling/pyrouge#i-have-summaries-in-plain-text-format-and-want-to-get-the-rouge-scores

What is document id? Do you mean we need to provide each document/summary in separate files? I have all the summaries in one file and all the gold summaries in another file. Isn't it enough?

Do you mean we need to provide each document/summary in separate files?

yes


Responsible entity

In order to properly capture the entity responsible for a test/set of tests, we may use http://purl.org/vocab/frbr/core#responsibleEntity

responsible entity is dcat:contactPoint


Is it possible to add ignoring the display of such 'Errors'? [FEATURE]

FS19 will never be loaded again, but mods from it are still being looked at (for reference, etc.).

Is it possible to add the "Ignore this":

item to the context menu, by right-clicking the mouse?

Hmm.  let me think some more on this, but I can at least have the little flag cleared when you open the debug log.


⚠ Bofrost.lu has degraded performance

In eb2cf5e, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1168 ms

Resolved: Bofrost.lu performance has improved in 9ee991c after 8 minutes.


Chess: Can't start a game or set options with keyboard

https://developer.microsoft.com/en-us/microsoft-edge/testdrive/demos/chess/
Suffers from buttons being divs.

Closing in favor of #255.


feat: remove cipher update

As discussed in #20, this PR proposes to remove the cipher.update(b"\x11") in the decryption code for BTHome V2. This will remove the need to apply this in the sensor firmware code as well.
In V1, this cipher.update(b"\x11") will be remained as it was, to keep backward compatibility. V2 isn't used most likely at the moment, as Home Assistant will be updated in the December release with V2 support.

Fine with us!
Although, now that I look at it, it's a change for the sake of change itself. I necessitates additional if-s to handle, so it may be simpler to just leave it as is. That way, encrypt/decrypt functions can be reused verbatim from v1.

v2 shold be about 'this is the better way to do this' and not 'we made it work like somebody else'. Remove IMHO, or explicitly explain why it is like that in documentation


TCP ports on Kubernetes

Please see this ticket.

Closing after discussing with @taojing2002 and @mbjones. Explanation:
This issue originally arose from testing with metacat outside and the indexer inside the cluster, which meant we had to expose solr and rabbitmq ports through the ingress.
However, since we now have a helm chart for Metacat that includes the indexer as a sub-chart, the two most likely deployment scenarios will now be "everything inside the cluster" or "not kubernetes-based" (i.e. a legacy deployment). This, in turn, means that we no longer need to expose indexer-related ports externally, thus rendering this Issue (and Issue #38) moot.


Add 'Institute of Chartered Accountants of Scotland' to institutions

Context
https://trello.com/c/9qNctNxM/4652-register-trainee-teachers
A provider cannot find the closest match, so a request has been made by support to add it to the list of institutions.
Changes proposed in this pull request

Add The Institute of Chartered Accountants of Scotland to INSTITUTIONS

@jhenderson can you confirm whether this institution has degree awarding powers in Scotland or not? They don't seem to be in this list, but possibly that's not up-to-date? https://www.gov.scot/policies/universities/
If they don't award "degrees" but do award equivalent qualifications, we can still add them to the list, adding 'has_never_awarded_degrees' => true. This helps us validate degree qualifications (in theory).

I don't think we should add this. This list aims to have all UK degree awarding institutions - which I don't think this is.
This has come about because a provider has attempted to add an accounting qualification to Register. But I suspect we don't want to know about the accounting qualification as it isn't relevant to their teacher training.
What this does highlight is that Register (and this list) don't have any generic options for where an institution is missing - we should add something for this.

To summarise some offline conversations:
This institution isn't degree awarding, so arguably doesn't belong in this list. However, the list already contains many non-degree awarding institutions as a result of HESA including them - so one more doesn't make a big difference. We have a flag that can mark something as non-degree awarding. As this is blocking a user from proceeding, we'll add the item. Services can choose to suppress non-degree awarding institutions from appearing if they choose.
In short term we will also look at adding a 'catch all' generic option of "Other institution" - which could be used in cases like this.


In the front-office, whenever I select "The license is not listed here" for datasets, it shows a specific message

"A specific license has been chosen by the rights holder. Get in touch with the rights holder for reuse rights"

Not needed because we have logic


Training error when image resolution is no multiple of 32

Output of train.py
Sanity Checking DataLoader 0:   0%|                                                                                | 0/2 [00:00<?, ?it/s]Traceback (most recent call last):   File "/home/jbogaert/keypoint-detection/keypoint_detection/train/train.py", line 115, in <module>    main(hparams)   File "/home/jbogaert/keypoint-detection/keypoint_detection/train/train.py", line 60, in main     trainer.fit(model, data_module)   File "/home/jbogaert/miniconda3/envs/keypoint-detection/lib/python3.9/site-packages/pytorch_lightning/trainer/trainer.py", line 608, in fit     call._call_and_handle_interrupt(   File "/home/jbogaert/miniconda3/envs/keypoint-detection/lib/python3.9/site-packages/pytorch_lightning/trainer/call.py", line 38, in _call_and_handle_interrupt     return trainer_fn(*args, **kwargs)   File "/home/jbogaert/miniconda3/envs/keypoint-detection/lib/python3.9/site-packages/pytorch_lightning/trainer/trainer.py", line 650, in _fit_impl     self._run(model, ckpt_path=self.ckpt_path)   File "/home/jbogaert/miniconda3/envs/keypoint-detection/lib/python3.9/site-packages/pytorch_lightning/trainer/trainer.py", line 1103, in _run     results = self._run_stage()   File "/home/jbogaert/miniconda3/envs/keypoint-detection/lib/python3.9/site-packages/pytorch_lightning/trainer/trainer.py", line 1182, in _run_stage     self._run_train()   File "/home/jbogaert/miniconda3/envs/keypoint-detection/lib/python3.9/site-packages/pytorch_lightning/trainer/trainer.py", line 1195, in _run_train     self._run_sanity_check()   File "/home/jbogaert/miniconda3/envs/keypoint-detection/lib/python3.9/site-packages/pytorch_lightning/trainer/trainer.py", line 1267, in _run_sanity_check     val_loop.run()   File "/home/jbogaert/miniconda3/envs/keypoint-detection/lib/python3.9/site-packages/pytorch_lightning/loops/loop.py", line 199, in run     self.advance(*args, **kwargs)   File "/home/jbogaert/miniconda3/envs/keypoint-detection/lib/python3.9/site-packages/pytorch_lightning/loops/dataloader/evaluation_loop.py", line 152, in advance     dl_outputs = self.epoch_loop.run(self._data_fetcher, dl_max_batches, kwargs)   File "/home/jbogaert/miniconda3/envs/keypoint-detection/lib/python3.9/site-packages/pytorch_lightning/loops/loop.py", line 199, in run     self.advance(*args, **kwargs)   File "/home/jbogaert/miniconda3/envs/keypoint-detection/lib/python3.9/site-packages/pytorch_lightning/loops/epoch/evaluation_epoch_loop.py", line 137, in advance     output = self._evaluation_step(**kwargs)   File "/home/jbogaert/miniconda3/envs/keypoint-detection/lib/python3.9/site-packages/pytorch_lightning/loops/epoch/evaluation_epoch_loop.py", line 234, in _evaluation_step     output = self.trainer._call_strategy_hook(hook_name, *kwargs.values())   File "/home/jbogaert/miniconda3/envs/keypoint-detection/lib/python3.9/site-packages/pytorch_lightning/trainer/trainer.py", line 1485, in _call_strategy_hook     output = fn(*args, **kwargs)   File "/home/jbogaert/miniconda3/envs/keypoint-detection/lib/python3.9/site-packages/pytorch_lightning/strategies/strategy.py", line 390, in validation_step     return self.model.validation_step(*args, **kwargs)   File "/home/jbogaert/keypoint-detection/keypoint_detection/models/detector.py", line 319, in validation_step     result_dict = self.shared_step(val_batch, batch_idx, include_visualization_data_in_result_dict=True)   File "/home/jbogaert/keypoint-detection/keypoint_detection/models/detector.py", line 220, in shared_step     predicted_unnormalized_maps = self.forward_unnormalized(input_images)   File "/home/jbogaert/keypoint-detection/keypoint_detection/models/detector.py", line 169, in forward_unnormalized     return self.unnormalized_model(x)   File "/home/jbogaert/miniconda3/envs/keypoint-detection/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1130, in _call_impl     return forward_call(*input, **kwargs)   File "/home/jbogaert/miniconda3/envs/keypoint-detection/lib/python3.9/site-packages/torch/nn/modules/container.py", line 139, in forward     input = module(input)   File "/home/jbogaert/miniconda3/envs/keypoint-detection/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1130, in _call_impl     return forward_call(*input, **kwargs)   File "/home/jbogaert/keypoint-detection/keypoint_detection/models/backbones/unet.py", line 102, in forward     x = block(x, x_skip)   File "/home/jbogaert/miniconda3/envs/keypoint-detection/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1130, in _call_impl     return forward_call(*input, **kwargs)   File "/home/jbogaert/keypoint-detection/keypoint_detection/models/backbones/unet.py", line 59, in forward     x = torch.cat([x, x_skip], dim=1) RuntimeError: Sizes of tensors must match except in dimension 1. Expected size 620 but got size 621 for tensor number 1 in the list.
Image input size was 2208x1242. Error was resolved when image was rescaled to a multiple of 32.

This is a known issue, most models are sensitive to this. You can try my PR which allows you to dynamically rescale during training without having to change the data itself.


[Actions] extend docs

As a user of ZITADEL I want to know which actions exist, when they are invoked, which parameters are set, what i have to return and what libraries can be used. This improves my understanding of how to use actions in general.

comment issues/discussions:

https://github.com/zitadel/zitadel/discussions/4083#discussioncomment-3296548

check if this can be done as well: https://github.com/zitadel/zitadel/issues/2485

seperate branch: https://github.com/zitadel/zitadel/pull/4377

the docs will follow this structure:
docs/
    apis/
        actions/
            overview
            modules
            login flow
            register flow


Move CI to pdm

Using pdm run all in place of tox. Tox is not adding much in this case since we are not creating a PyPI package.


WIP Lazy routing

Description
Lazy-loading routes
remaning tasks

import the create resource forms directly from the files and delete the index registry
rewrite remaining routes in app.js (the ones deemed suitable)
Remove ComponentFor, ComponentForList and its usages (otherwise components will be batch-loaded anyway).
a) in CustomResourceDefinitionVersions
b) verify ApplicationMappings in namespace details, such component should't exist

Related issue(s)
Resolves #1006

/test pre-busola-smoke-test-docker

/test pre-busola-smoke-test-docker

/test pre-busola-integration-test-k3s

/test pre-busola-smoke-test-docker


feat/dtfs2-6113/add optional pagination to tfm deals endpoint

Introduction :pencil2:
The api doesnt paginate data at all. This should be introduced optionally without breaking existing behaviour, including for external uses of dtfs-central-api
Resolution :heavy_check_mark:

add page and pagesize parameters to the GET tfm deals endpoint. Return a pagination object containing metadata.
pass this data through trade-finance-manager-api

Misc ➕
Refactored trade-finance-manager-api mock api call implementations

Codecov Report
Attention: 29 lines in your changes are missing coverage. Please review.

Comparison is base (6da84ce) 21.88% compared to head (47aa08a) 6.44%.
Report is 150 commits behind head on main.

Files
Patch %
Lines

...1/controllers/tfm/deal/tfm-get-deals.controller.js
0.00%
23 Missing and 5 partials :warning:

dtfs-central-api/src/drivers/db-client.js
0.00%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #2550       +/-   ##
==========================================
- Coverage   21.88%   6.44%   -15.45%
==========================================
  Files        1132     522      -610
  Lines       27048   10565    -16483
  Branches     4447    2124     -2323
==========================================
- Hits         5920     681     -5239
+ Misses      18403    8280    -10123
+ Partials     2725    1604     -1121

Flag
Coverage Δ

api-test
0.48% <ø> (ø)

ui-test
?

unit-test
7.39% <0.00%> (+3.60%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

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


[Test] Add range-info test driver to swift-ide-test. NFC

Per discussion with @akyrtzi , swift-ide-test is a better test driver than sourcekitd-test for language-specific APIs, which range-info resolver belongs to. This patch teaches swift-ide-test to test range info resolver.

@swift-ci Please smoke test and merge


[receiver/lightprometheus] OTL-2117: Limit resource attributes

Limit resource attributes added by the lightprometheus receiver by default and make them configurable.
Make enabling/disabling particular attributes possible and disable the following attributes by default:

net.host.name:
net.host.port:
http.scheme:

To enable them back the attributes add the following configuration:
resource_attributes:
  net.host.name:
    enabled: true
  net.host.port:
    enabled: true
  http.scheme:
    enabled: true

The configuration interface is aligned with receivers created by the mdatagen tool

Linting issues - looks like they predate your changes, sorry. Please apply fixes.

@rmfitzpatrick added eeadme, PTAL https://github.com/signalfx/splunk-otel-collector/pull/3049


[vxl] build failure

Host Environment

OS: [Windows 10 Pro Version 1909 (OS Build 18363.657) ]
Compiler: VS 2017 v15.9.19 and VS2019 v16.4.4

To Reproduce
Steps to reproduce the behavior:
.\vcpkg.exe install vxl[core,core_imaging]:x64-windows
Building package vxl[core,core_imaging]:x64-windows...
CMake Warning at ports/vxl/portfile.cmake:8 (message):
Can't build VXL CORE_IMAGING features with non built-in OpenJpeg.  Please
remove OpenJpeg, and try install VXL again if you need them.
Call Stack (most recent call first):
scripts/ports.cmake:90 (include)
-- Using cached E:/Git/vcpkg.downloads/vxl-vxl-dac1c7ed8e183e9c6de8b928c8b0294a7bd1d8ee.tar.gz
-- Extracting source E:/Git/vcpkg.downloads/vxl-vxl-dac1c7ed8e183e9c6de8b928c8b0294a7bd1d8ee.tar.gz
-- Applying patch fix_dependency.patch
-- Using source at D:/Git/vs2019.prod/buildtrees/vxl/src/4a7bd1d8ee-c80aa455d1
-- Configuring x64-windows
-- Building x64-windows-dbg
CMake Error at scripts/cmake/vcpkg_execute_build_process.cmake:136 (message):
Command failed: "C:/Program Files/CMake/bin/cmake.exe" --build . --config Debug --target install -- -v
Working Directory: D:/Git/vs2019.prod/buildtrees/vxl/x64-windows-dbg
See logs for more information:
D:\Git\vs2019.prod\buildtrees\vxl\install-x64-windows-dbg-out.log
Call Stack (most recent call first):
scripts/cmake/vcpkg_build_cmake.cmake:91 (vcpkg_execute_build_process)
scripts/cmake/vcpkg_install_cmake.cmake:24 (vcpkg_build_cmake)
ports/vxl/portfile.cmake:50 (vcpkg_install_cmake)
scripts/ports.cmake:90 (include)
Error: Building package vxl:x64-windows failed with: BUILD_FAILED
Elapsed time for package vxl:x64-windows: 43.74 s
Failure logs
Attached "install-x64-windows-dbg-out.log"
install-x64-windows-dbg-out.log
Additional context
I first got the message about "fix_dependcy.patch" needing to be updated:
-- Applying patch fix_dependency.patch
CMake Error at scripts/cmake/vcpkg_apply_patches.cmake:52 (message):
Applying patch failed.  Patch needs to be updated to work with source being
used by vcpkg!
Call Stack (most recent call first):
scripts/cmake/vcpkg_extract_source_archive_ex.cmake:134 (vcpkg_apply_patches)
scripts/cmake/vcpkg_from_github.cmake:139 (vcpkg_extract_source_archive_ex)
ports/libmariadb/portfile.cmake:8 (vcpkg_from_github)
scripts/ports.cmake:90 (include)
So I created a new patch to work with the new source - attached as fix_dependency.patch.txt.
fix_dependency.patch.txt

Related issue #3911


Decreace log level

Would it be possible to decrease the log level fpr the requests from INFO to DEBUG?
Also for some other log messages (i.e. calculated time value in minute and second).

I will change this in the next versions.

done


Fix regression in statistics test

See rust-lang/rust#43680 for more details

This is resolved by #e728aac. The regression only affected empty slices and can be resolved by pinning the type of the slice to [f64; 0]. I do not believe this warrants any API changes


This application failed to start because it could not find or load the Qt platform plugin "xcb"

edison@amax-server:~/labelImg-master$ make qt5py3
pyrcc5 -o resources.py resources.qrc
edison@amax-server:~/labelImg-master$ python3 labelImg.py
This application failed to start because it could not find or load the Qt platform plugin "xcb"
in "".
Available platform plugins are: eglfs, linuxfb, minimal, minimalegl, offscreen, vnc, xcb.
Reinstalling the application may fix this problem.
Aborted (core dumped)

**OS:  Ubuntu 16.04
*PyQt 5

please give me a help, thank you

There are a number of threads on stackoverflow regarding this issue (an example), have you been able to try any of the solutions?

Closing due to inactivity.

Had the same problem even though I followed the install instructions. In my case it was because I had a custom installation in /opt/Qt/. After deleting paths to that folder from LD_LIBRARY_PATH and PATH, it works. For some reason, exporting the missing plugin path does not work export QT_QPA_PLATFORM_PLUGIN_PATH=/opt/Qt/5.12.3/gcc_64/plugins/platform. Not sure what I'm doing wrong here.

sudo apt-get install pyqt5-dev-tools install dev tools

@krishnaupadhyay3 I used sudo apt-get install pyqt5-dev tools and it worked :) Thank you

OS:windows
set QT_QPA_PLATFORM_PLUGIN_PATH=C:\Python36\Lib\site-packages\PyQt5\plugins

I was using Conda on Windows. Uninstalling and reinstalling worked as suggested here:
conda remove qt
conda remove pyqt
conda install qt
conda install pyqt


Link to the IDR preprint

From Trello:

For now add link in the IDR home page to the pre-print of the paper for description of the resource and how to browse in supplementary info.

See screenshot on https://trello.com/c/5dIZXs5V/94-short-help-page-on-how-to-browse-idr

All happy.


Footer Decoration - wrldbaems 62

Please always provide the GitHub issue(s) your PR is for, as well as test URLs where your change can be observed (before and after):
Fix WRLDBAEMS-62
Test URLs:

Before: https://main--world-bank--aemsites.hlx.live/
After: https://wrldbaems-62--world-bank--aemsites.hlx.live/

RTL Desktop View

LTR Desktop View

RTL Tablet View

LTR Tablet View

RTL MWeb View

LTR MWeb View

@thomsebastin :

global rules should be outside any media query, also px size should be corrected : Done
instead of using data-block-name="columns", use class ".columns" : Done
Round this to 22. There won't be any difference. : Done
If this is not part of their design system library, I would suggest naming it entirely differently to avoid confusions. : Done
Variable names should not start in capitals unless they are a constructor in JavaScript. Can you make changes? : Done
Indentation seems off : Linting passed
In general, I would avoid rems on padding. When the font size increases, the padding might also increase leaving less and less space for content. : Done
Can leverage dom-helpers that we have added. - Added some code in that function.
We have don-utils that you can use. It will simplify some of the code. - Added some code in that function.

@mkbansal1 :

variable name should start with small case. : Done
Hover state is missing for all links : Done
On desktop, its consuming 100% width : Done
Footer does not match the figma (eg. top spacing) : Done

@adobe-rpal Could you please fix this portion

As per designs, they shouldn't wrap.

@adobe-rpal
In tablet, the text and button can be aligned a little better. See if you can add some gap so the wrapping happens a little early

On Mobile, the social-icons should wrap. Can you please have a look?

Footer links wrap : Done
In tablet, the text and button can be aligned a little better. See if you can add some gap so the wrapping happens a little early : Done
On Mobile, the social-icons should wrap. Can you please have a look? : Done


issue-5586-added-tests

Added tests in response to issue #5586

Thanks for the PR and your first contribution to rustfmt 🎉
Can you condense these inputs into a single file please. Also, please add a // rustfmt-version: Two comment to the start of the file to configure this test to run with version=Two. Details on this can be found in the Create test case docs.


Build script fails

Right now we don't have any fake secrets.yaml file so that one need to be added.

Closed bd69a9f628ba9132fd965bc77bd03a3b82dccc29


ci: add github actions. fix #1

lint is failing, but I'll address that later when I'll have a better idea what's going on.


L/XL Icon Additions

Component Requirements: napari hub created 5 new icons for use on our new homepage (currently underdevelopment). All Icons are size L/XL:

rocket
Read
Widget
4.Grid with three dots
Updated
Want to share them to see if they could/should be added to the SDS figma and component library.
I have reviewed them with Connor for consistency in stroke, size, and positioning.
Figma: https://www.figma.com/file/pAyebvFjfRkOsfcCRvUR1D/New-Icon-Workspace?type=design&node-id=15%3A271&t=4WuGB6EHIReDhcJL-1
MUIv4 Base Component (if applicable): Icon
Team(s) Requesting: Imaging
Component Needed By: next week or two?

https://github.com/chanzuckerberg/sci-components/pull/525#issuecomment-1619240377


LUCENE-9849: Don't cross-link between modules for interim snapshot builds.

This isn't ideal - I think the root property luceneDocUrl should be moved to render-javadoc, but it's a start?

Hey, @uschindler do you want to take a peek? Or should I just commit this in and we can follow-up with other improvements later on?


[CI] Hot fix for actions/upload-artifact and actions/download-artifact

a hot fix for actions/upload-artifact and actions/download-artifact
ref

https://github.com/sustainable-computing-io/kepler-metal-ci/actions/runs/11146969903/job/30980143377?pr=284#step:1:27
https://github.blog/changelog/2024-02-13-deprecation-notice-v1-and-v2-of-the-artifact-actions/

thanks @KaiyiLiu1234, as the CI passed, merge this PR for hot fix.


feat(ts/analyzer): report invalid override class method

Description:

BREAKING CHANGE:

Related issue (if exists):

Thank you!


feat(version): option to add commit login username on each changelog entry, closes #248

Description
Add new lerna version --changelog-include-commits-client-login [msg] option that will add commit remote client login username to each changelog commit line entry. This option will fetch all commits from the client remote server (only GitHub for now) and from that list it will find associated user login with commit changelog entry and will add them at the end of each changelog entry (when found), it uses the GitHub Graphql API to fetch all recent commits (since last release) from GitHub with only the fields we're interested in.
Motivation and Context
closes #248, which is to add user login on each changelog commit line and that will indirectly create a Contributors list when creating a new GitHub Release
Example with default format
**deps:** update dependency git-url-parse to v12 ([978bf36](https://github.com/.../978bf36)) (@renovate-bot)

or a custom format (use any of %a and/or %l)
lerna version --changelog-include-commits-client-login " by @%l, %a"

# result
**deps:** update dependency git-url-parse to v12 ([978bf36](https://github.com/.../978bf36)) by @renovate-bot, Whitesource Renovate

How Has This Been Tested?
added necessary unit tests and tested in git-dry-run mode to see changelog being updated as intended
Types of changes

[ ] Chore (change that has absolutely no effect on users)
[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

Hello @MiguelSOliveira
I saw you commenting on Lerna project and didn't want to comment directly in Lerna, because I spent quite a lot of time to get it working in Lerna-Lite, but basically the feature you are asking about in Lerna (which is to have commit contributors in github releases) is available in Lerna-Lite (this repo) through the above PR. The way it works is that unfortunately GitHub username aren't available directly in git, however what we can do is to query the GitHub GraphQL API for all the commits since the last the last release and then loop through all the commits and add associated username through git hash mapping, this mapping is executed when the changelogs get updated (which is also used by the github release)
Lerna-Lite is a smaller/lighter version of Lerna, you can read more about it on its readme page
The full documentation on how to use it is available in the version --changelog-include-commits-client-login docs and below are 2 print screen sample of this feature used in Lerna-Lite project and another project of mine
last release of Lerna-Lite with contributors

another one of my project


Support CUDA 12.2

Description

switches to CUDA 12.2.2 for building conda packages and wheels
adds new tests running against CUDA 12.2.2

Notes for Reviewers
This is part of ongoing work to build and test packages against CUDA 12.2.2 across all of RAPIDS.
For more details see:

https://github.com/rapidsai/build-planning/issues/7
https://github.com/rapidsai/shared-workflows/pull/166

Planning a second round of PRs to revert these references back to a proper branch-24.{nn} release branch of shared-workflows once https://github.com/rapidsai/shared-workflows/pull/166 is merged.
(created with rapids-reviser)

This is ready to merge. All CUDA 12 Python test jobs are being skipped currently by the CI scripts, but I think that could be fixed if the wholegraph developers want to install pytorch now that it supports CUDA 12. We shouldn't block this PR on that since builds are passing.

/merge

/merge


Modals ignore clicking backdrop to close, if mousedown happened on dialog element, reflecting original bootstrap.js behaviour

In bootstrap's source code, it tries to not close the modals if the
mouse down event was inside of the modal and mouse up event was outside
of the modal. With these changes we are copying the same behaviour from
it. We are also using similar variables to reflect it's source code.
Here is the click event of bootstrap: https://github.com/twbs/bootstrap/blob/master/js/src/modal.js#L361
Here is the handlers for mouse down and up events: https://github.com/twbs/bootstrap/blob/master/js/src/modal.js#L154

@eyupatis thanks for working on this!
So far this looks good! But I would love to have a test covering that behaviour. Would you be able to add one? Something that basically simulates what is described in #934:

This prevents closing the modal when user holds down the button on a modal, drags the cursor to the backdrop and releases the button.

@simonihmig I added two test cases that covers existing behavior and newly added behavior.

Thank you for your contribution, @eyupatis!

It was a pleasure to contribute and work with you, thank you for accepting and merging pr :)


connecting to lmstudio vllm?

Hi,
would it be possible to add support for these or is it possible at the moment?
I tried combing throught the code with success.

try pop this at the top of your example...
os.environ["OPENAI_API_BASE"]="http://localhost:1234/v1"

didn't even think that would so simple, thank you.


Enable SDK E2E test on Windows.

Description
Enable SDK E2E test on Windows.
Issue reference
We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.
Please reference the issue this PR will close: #2695
Checklist
Please make sure you've  completed the relevant tasks for this PR, out of the following list:

[ ] Code compiles correctly
[ ] Created/updated tests
[ ] Unit tests passing
[ ] End-to-end tests passing
[ ] Extended the documentation / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Specification has been updated / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Provided sample for the feature / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]

/ok-to-test

/ok-to-test

/ok-to-test

/ok-to-test

/ok-to-test

/ok-to-test

/ok-to-test

/ok-to-test

/ok-to-test

/ok-to-test

/ok-to-test

/ok-to-test

/ok-to-test

/ok-to-test

/ok-to-test

/ok-to-test

/ok-to-test

/ok-to-test

/ok-to-test


contrib: add qatzip compressor

Commit Message: add qatzip compressor as contrib extension #30771
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

@giantcroc would you add more description about this extension and the evidence of the user using this extension.

@giantcroc would you like to add more description about this extension and the evidence of the user using this extension.

OK, I have added user usage in #30771. Thanks!

Time for an Intel specific project to slim down vendor and platform specific features in core Envoy?

I talked with @giantcroc. Today we get customer adoption, which is good news for us, and then we want to see if we reach the requirement to contribute it to the upstream.
From my understanding, the purpose of contributing to the open-source project is that it is hard for us to reach more customers, the upstream project helps us to get wider adoption and feedback.  Just like this one https://github.com/envoyproxy/envoy/issues/21531#issuecomment-1790551233
Also, contribute open-source project, and we get some reviews and feedback from the community, that help us to integrate our stuff correctly with the upstream project.
That is all we appreciate what we get from the community.
We know it will give a lot of burden to the community, that is why we submit to the contrib extension not the Envoy core (accept anything can help on common use-case)
But yes, it is also hard for us to change the whole intel to match the upstream requirement, but just try our best. It is also fine if we are really far from the least requirement, we accept that, and hope we can come back in the future when we get better.

Time for an Intel specific project to slim down vendor and platform specific features in core Envoy?

I talked with @giantcroc. Today we get customer adoption, which is good news for us, and then we want to see if we reach the requirement to contribute it to the upstream.
From my understanding, the purpose of contributing to the open-source project is that it is hard for us to reach more customers, the upstream project helps us to get wider adoption and feedback. Just like this one #21531 (comment)
Also, contribute open-source project, and we get some reviews and feedback from the community, that help us to integrate our stuff correctly with the upstream project.
That is all we appreciate what we get from the community.
We know it will give a lot of burden to the community, that is why we submit to the contrib extension not the Envoy core (except anything can benefit for common use-case)
But yes, it is also hard for us to change the whole intel to match the upstream requirement, but just try our best. It is also fine if we are really far from the least requirement, we accept that, and hope we can come back in the future when we get better.

Yeah, agree with @soulxu, we are very glad to have customer's usage case now,  and we really hope this could meet  community's requirements, if not, we are prepared to do better.

@mattklein123 Hi Matt, do you think there is an opportunity for us to contribute qatzip compressor to the community? And it's fine if we don't match the requirement, we will try in the future.

Following up with maintainers

Please resolve Windows build error before I can merge it.
/wait

/retest

/retest

/retest

If this PR is merged I have issues compiling on MacOS. Mainly bazel build @envoy//bazel/foreign_cc:lz4
bazel build @envoy//bazel/foreign_cc:lz4 --verbose_failures
INFO: Analyzed target //bazel/foreign_cc:lz4 (0 packages loaded, 0 targets configured).
INFO: Found 1 target...
ERROR: /Users/jonas.brunsgaard/code/strimzi-wip/envoy2/envoy/bazel/foreign_cc/BUILD:63:5: Foreign Cc - Make: Building lz4 failed: (Exit 2): bash failed: error executing command (from target //bazel/foreign_cc:lz4)
  (cd /private/var/tmp/_bazel_jonas.brunsgaard/b87ffa8d691acaee9dfa9a389da51f08/sandbox/darwin-sandbox/19226/execroot/envoy && \
  exec env - \
    PATH=/opt/homebrew/bin:/opt/local/bin:/usr/local/bin:/usr/bin:/bin \
  /bin/bash -c bazel-out/darwin-fastbuild/bin/bazel/foreign_cc/lz4_foreign_cc/wrapper_build_script.sh)
# Configuration: 293240611eb29bd26a1d18e2f6bc646c3721901cd716a4ec57297f597907ae3e
# Execution platform: @local_config_platform//:host

Use --sandbox_debug to see verbose messages from the sandbox and retain the sandbox build root for debugging
rules_foreign_cc: Build failed!
rules_foreign_cc: Keeping temp build directory and dependencies directory for debug.
rules_foreign_cc: Please note that the directories inside a sandbox are still cleaned unless you specify --sandbox_debug Bazel command line flag.
rules_foreign_cc: Printing build logs:
_____ BEGIN BUILD LOGS _____

Bazel external C/C++ Rules. Building library lz4

Environment:______________
XCODE_VERSION_OVERRIDE=<IP_ADDRESS>C65
BUILD_SCRIPT=bazel-out/darwin-fastbuild/bin/bazel/foreign_cc/lz4_foreign_cc/build_script.sh
EXT_BUILD_ROOT=/private/var/tmp/_bazel_jonas.brunsgaard/b87ffa8d691acaee9dfa9a389da51f08/sandbox/darwin-sandbox/19226/execroot/envoy
BUILD_LOG=bazel-out/darwin-fastbuild/bin/bazel/foreign_cc/lz4_foreign_cc/Make.log
PWD=/private/var/tmp/_bazel_jonas.brunsgaard/b87ffa8d691acaee9dfa9a389da51f08/sandbox/darwin-sandbox/19226/execroot/envoy
SDKROOT=/Library/Developer/CommandLineTools/SDKs/MacOSX.sdk
BUILD_WRAPPER_SCRIPT=bazel-out/darwin-fastbuild/bin/bazel/foreign_cc/lz4_foreign_cc/wrapper_build_script.sh
TMPDIR=/var/folders/yp/sg57x5597798plt6918hpzjr0000gn/T/
CMAKE_OSX_ARCHITECTURES=x86_64
EXT_BUILD_DEPS=/private/var/tmp/_bazel_jonas.brunsgaard/b87ffa8d691acaee9dfa9a389da51f08/sandbox/darwin-sandbox/19226/execroot/envoy/bazel-out/darwin-fastbuild/bin/bazel/foreign_cc/lz4.ext_build_deps
APPLE_SDK_PLATFORM=MacOSX
APPLE_SDK_VERSION_OVERRIDE=14.2
BUILD_TMPDIR=/private/var/tmp/_bazel_jonas.brunsgaard/b87ffa8d691acaee9dfa9a389da51f08/sandbox/darwin-sandbox/19226/execroot/envoy/bazel-out/darwin-fastbuild/bin/bazel/foreign_cc/lz4.build_tmpdir
SHLVL=3
ZERO_AR_DATE=1
INSTALLDIR=/private/var/tmp/_bazel_jonas.brunsgaard/b87ffa8d691acaee9dfa9a389da51f08/sandbox/darwin-sandbox/19226/execroot/envoy/bazel-out/darwin-fastbuild/bin/bazel/foreign_cc/lz4
PATH=/private/var/tmp/_bazel_jonas.brunsgaard/b87ffa8d691acaee9dfa9a389da51f08/sandbox/darwin-sandbox/19226/execroot/envoy:/opt/homebrew/bin:/opt/local/bin:/usr/local/bin:/usr/bin:/bin
DEVELOPER_DIR=/Library/Developer/CommandLineTools
__CF_USER_TEXT_ENCODING=0x1F5:0x0:0x0
_=/usr/bin/env
__________________________
+ ASFLAGS='-D_FORTIFY_SOURCE=1 -fstack-protector -fcolor-diagnostics -Wall -Wthread-safety -Wself-assign -fno-omit-frame-pointer -O0 -DDEBUG DEBUG_PREFIX_MAP_PWD=. -isysroot __BAZEL_XCODE_SDKROOT__ -F__BAZEL_XCODE_SDKROOT__/System/Library/Frameworks -F__BAZEL_XCODE_DEVELOPER_DIR__/Platforms/MacOSX.platform/Developer/Library/Frameworks -no-canonical-prefixes -pthread -no-canonical-prefixes -Wno-builtin-macro-redefined -D__DATE__=redacted -D__TIMESTAMP__=redacted -D__TIME__=redacted -target x86_64-apple-macosx14.2 -DABSL_MIN_LOG_LEVEL=4'
+ CFLAGS='-D_FORTIFY_SOURCE=1 -fstack-protector -fcolor-diagnostics -Wall -Wthread-safety -Wself-assign -fno-omit-frame-pointer -O0 -DDEBUG DEBUG_PREFIX_MAP_PWD=. -isysroot __BAZEL_XCODE_SDKROOT__ -F__BAZEL_XCODE_SDKROOT__/System/Library/Frameworks -F__BAZEL_XCODE_DEVELOPER_DIR__/Platforms/MacOSX.platform/Developer/Library/Frameworks -no-canonical-prefixes -pthread -no-canonical-prefixes -Wno-builtin-macro-redefined -D__DATE__=redacted -D__TIMESTAMP__=redacted -D__TIME__=redacted -target x86_64-apple-macosx14.2 -DABSL_MIN_LOG_LEVEL=4'
+ CXXFLAGS='-D_FORTIFY_SOURCE=1 -fstack-protector -fcolor-diagnostics -Wall -Wthread-safety -Wself-assign -fno-omit-frame-pointer -O0 -DDEBUG -std=c++11 DEBUG_PREFIX_MAP_PWD=. -isysroot __BAZEL_XCODE_SDKROOT__ -F__BAZEL_XCODE_SDKROOT__/System/Library/Frameworks -F__BAZEL_XCODE_DEVELOPER_DIR__/Platforms/MacOSX.platform/Developer/Library/Frameworks -no-canonical-prefixes -pthread -no-canonical-prefixes -Wno-builtin-macro-redefined -D__DATE__=redacted -D__TIMESTAMP__=redacted -D__TIME__=redacted -target x86_64-apple-macosx14.2 -DABSL_MIN_LOG_LEVEL=4 -std=c++17'
+ LDFLAGS='-lc++ -fobjc-link-runtime -headerpad_max_install_names -no-canonical-prefixes -target x86_64-apple-macosx14.2 -Xlinker -no_deduplicate -lc++ -target x86_64-apple-macosx14.2'
+ AR=/private/var/tmp/_bazel_jonas.brunsgaard/b87ffa8d691acaee9dfa9a389da51f08/sandbox/darwin-sandbox/19226/execroot/envoy/external/local_config_cc/libtool
+ CC=/private/var/tmp/_bazel_jonas.brunsgaard/b87ffa8d691acaee9dfa9a389da51f08/sandbox/darwin-sandbox/19226/execroot/envoy/external/local_config_cc/wrapped_clang
+ CXX=/private/var/tmp/_bazel_jonas.brunsgaard/b87ffa8d691acaee9dfa9a389da51f08/sandbox/darwin-sandbox/19226/execroot/envoy/external/local_config_cc/wrapped_clang_pp
+ RANLIB=:
+ CPPFLAGS=
+ /private/var/tmp/_bazel_jonas.brunsgaard/b87ffa8d691acaee9dfa9a389da51f08/sandbox/darwin-sandbox/19226/execroot/envoy/bazel-out/darwin-opt-exec-2B5CBBC6/bin/external/rules_foreign_cc/toolchains/make/bin/make lib MOREFLAGS=-fPIC BUILD_SHARED=no PREFIX=/private/var/tmp/_bazel_jonas.brunsgaard/b87ffa8d691acaee9dfa9a389da51f08/sandbox/darwin-sandbox/19226/execroot/envoy/bazel-out/darwin-fastbuild/bin/bazel/foreign_cc/lz4
make[1]: Entering directory '/private/var/tmp/_bazel_jonas.brunsgaard/b87ffa8d691acaee9dfa9a389da51f08/sandbox/darwin-sandbox/19226/execroot/envoy/bazel-out/darwin-fastbuild/bin/bazel/foreign_cc/lz4.build_tmpdir/lib'
compiling static library
error: /Library/Developer/CommandLineTools/usr/bin/libtool: no output file specified (specify with -o output)
Usage: /Library/Developer/CommandLineTools/usr/bin/libtool -static [-] file [...] [-filelist listfile[,dirname]] [-arch_only arch] [-sacLT] [-no_warning_for_no_symbols]
Usage: /Library/Developer/CommandLineTools/usr/bin/libtool -dynamic [-] file [...] [-filelist listfile[,dirname]] [-arch_only arch] [-o output] [-install_name name] [-compatibility_version #] [-current_version #] [-seg1addr 0x#] [-segs_read_only_addr 0x#] [-segs_read_write_addr 0x#] [-seg_addr_table <filename>] [-seg_addr_table_filename <file_system_path>] [-all_load] [-noall_load]
make[1]: *** [Makefile:99: liblz4.a] Error 1
make[1]: Leaving directory '/private/var/tmp/_bazel_jonas.brunsgaard/b87ffa8d691acaee9dfa9a389da51f08/sandbox/darwin-sandbox/19226/execroot/envoy/bazel-out/darwin-fastbuild/bin/bazel/foreign_cc/lz4.build_tmpdir/lib'
make: *** [Makefile:57: liblz4.a] Error 2
_____ END BUILD LOGS _____
rules_foreign_cc: Build wrapper script location: bazel-out/darwin-fastbuild/bin/bazel/foreign_cc/lz4_foreign_cc/wrapper_build_script.sh
rules_foreign_cc: Build script location: bazel-out/darwin-fastbuild/bin/bazel/foreign_cc/lz4_foreign_cc/build_script.sh
rules_foreign_cc: Build log location: bazel-out/darwin-fastbuild/bin/bazel/foreign_cc/lz4_foreign_cc/Make.log

Target //bazel/foreign_cc:lz4 failed to build
INFO: Elapsed time: 4.034s, Critical Path: 3.37s
INFO: 2 processes: 2 internal.
FAILED: Build did NOT complete successfully


Setting "max" of genes in optimization.json not working as expected

The maximum value of a gene needs to be set to n+1 to work as would be expected.
For Example if I set the "min" to 1 and the "max" to 10, the actual range that will be tested will be between 1 and 9.

Great spot. The max was just passed into standard Random.Next(min, max) which has an exclusive upper bound. This was fixed with latest commit. This issue will not have affected genes with precision.


Support Context Path changing

First, I want to thank you for sharing this tool. My clients, who are data scientists working in French hospitals, have the exact same need that you have addressed here.
I need to support a scenario where a reverse proxy exposes Nexus within a shared domain, in a dedicated directory. For example:https://your_url.domain/nexus/
I propose introducing a new optional environment variable: NEXUS_PATH.
If NEXUS_PATH is not defined, the URI remains unchanged to ensure backward compatibility and avoid breaking existing configurations.

I'm happy you accept my contribution. I followed the linter's suggestions, I hope this will pass. Feel free to fix the PR if not.

Hi @JimMadge, can you help me with the failing test, please ? I don't see how the 403 can relate with my changes.

@Jbaudon Yes let me have a look. The tests seem to pass fine for me locally.

I think the failure just means that the Nexus container hasn't finished setting up yet. I couldn't think of a better way to establish they are all ready so it is just a sleep command in the job.

Actually, the problem is here,
usage: nexus-allowlist [-h] --admin-password ADMIN_PASSWORD
                       [--nexus-host NEXUS_HOST] [--nexus-port NEXUS_PORT]
                       [--nexus-path NEXUS_PATH] [--version]
                       {change-initial-password,test-authentication,initial-configuration,update-allowlists} ...
nexus-allowlist: error: argument {change-initial-password,test-authentication,initial-configuration,update-allowlists}: invalid choice: '--admin-password password --nexus-host nexus --nexus-port 8081' (choose from 'change-initial-password', 'test-authentication', 'initial-configuration', 'update-allowlists')

@Jbaudon OK, got it working but unfortunately I've had to undo the good work to collect the arguments. I'd quite like to get this feature merged though so I think that can be an issue for another time.
Thanks again :pray:.


Add SSL policy for all HTTPS listeners by default

Issue-id
PR Checklist

[X] I have added example(s) inside the [./examples/] folder
[x] I have added the example(s) to the integration test list for normal (~30 minutes) or long runner >30 minutes
[X] I have checked the coding conventions as per the wiki
[X] I have checked to ensure there aren't other open Pull Requests for the same update/change?

Description
Currently, there is no option of defining SSL policy for listeners by default, whether choosing from one of the predefined policies or creating a custom security policy. The PR configures a default listener SSL policy which is applied to all HTTPS listeners unless they are overridden by listener specific SSL Policy under SSL settings.
Does this introduce a breaking change

[ ] YES
[X] NO

Testing

thanks for your contribution to Cloud Adoption Framework @ianlim-cldcvr


Table 37 Sales Line - fct. CalcPlannedShptDate new publisher

Hi,
for our application, we need a new Publisher in fct. CalcPlannedShptDate() as follows:

thanks in advance,
best regards,
Kathrin

Should Rec be passed by reference (with VAR) or by value (without VAR)?

The Rec should be passed by reference (with VAR)
Thanks a lot.

And would it be possible to have an Parameter IsHandled?


Updated transaction application model: replayer tool

Part of https://github.com/MinaProtocol/mina/pull/11760.

merged into https://github.com/MinaProtocol/mina/pull/12548