LTSS error

Hi @Expaso,
Also facing this error:
Log Details (ERROR)
Logger: custom_components.ltss
Source: custom_components/ltss/__init__.py:167
Integration: ltss (documentation)
First occurred: 4:06:44 PM (10 occurrences)
Last logged: 4:07:12 PM

Error during connection setup: (psycopg2.errors.UndefinedFile) could not open extension control file "/usr/share/postgresql/extension/postgis.control": No such file or directory [SQL: CREATE EXTENSION IF NOT EXISTS postgis CASCADE] (Background on this error at: http://sqlalche.me/e/e3q8) (retrying in 3 seconds)

I think this occurs when this is executed:
con.execute(
                text("CREATE EXTENSION IF NOT EXISTS postgis CASCADE"
                ).execution_options(autocommit=True))

Hi @le-andreazevedo,
Thanks! Did not encouter this one, because currently ltss seems broken somehow (i get a db_url not found, while it's right there in my config).
I've put out a new release, can you try if this works for you?

Should be working!

I've also suggested a fix for LTSS, but I suspect you are not affected as you did not receive that error in the first place.

Yes everything seems ok
Thanks a lot!


⚠ Mobitopup has degraded performance

In 8dfd62c, Mobitopup (https://napcard.voz.tech) experienced degraded performance:

HTTP code: 200
Response time: 3380 ms

Resolved: Mobitopup performance has improved in 74c5c72.


Use eval('require') instead of require to avoid bundler error

Use eval('require') instead of require to avoid bundler error.
This allow this package continue work with Node.js and can avoid a bundler error.
This PR fixes papnkukn/qrcode-svg#11

I think make sense to avoid Webpack errors

It makes more sense to avoid eval() at any cost.

Can't we provide a build for browsers without the save function and one for node?
NPM provides the feature to automatically import the correct version using
"exports": {
    "browser": {
      "default": "./qrcode.browser.min.js",
    },
    "default": "./qrcode.min.js"


[BUG] azure-storage-blob library with azure-core-http-okhttp client prevents application from termination for 60 seconds.

Describe the bug
azure-storage-blob library with azure-core-http-okhttp client prevents application from finishing for 60 seconds.
To Reproduce
Execute the provided code snippet. Instead of instantly terminating after printing 'File uploaded successfully', it holds for a 60-second delay before concluding execution.
Code Snippet
build.gradle
plugins {
    id 'java'
    id 'org.springframework.boot' version '3.2.3'
}

apply plugin: 'io.spring.dependency-management'

group = 'org.example'
version = '1.0-SNAPSHOT'

repositories {
    mavenCentral()
}

dependencies {
    implementation('org.springframework.boot:spring-boot-starter')

    implementation("com.azure:azure-identity:1.11.2") {
        exclude module: "azure-core-http-netty"
    }
    implementation("com.azure:azure-storage-blob:12.25.1") {
        exclude module: 'azure-core-http-netty'
    }
    implementation("com.azure:azure-core-http-okhttp:1.11.18")

    testImplementation platform('org.junit:junit-bom:5.9.1')
    testImplementation 'org.junit.jupiter:junit-jupiter'
}

test {
    useJUnitPlatform()
}

application code:
import java.io.ByteArrayInputStream;
import java.nio.charset.StandardCharsets;
import java.util.UUID;

import com.azure.core.credential.TokenCredential;
import com.azure.identity.ClientSecretCredentialBuilder;
import com.azure.storage.blob.BlobServiceClientBuilder;
import org.springframework.boot.CommandLineRunner;
import org.springframework.boot.SpringApplication;
import org.springframework.boot.autoconfigure.SpringBootApplication;

@SpringBootApplication
public class AzureBlobApp implements CommandLineRunner {

    public static void main(String[] args) {
        SpringApplication.run(AzureBlobApp.class, args);
    }

    @Override
    public void run(String... args) {
        var client = new BlobServiceClientBuilder().endpoint(System.getenv("BLOB_ENDPOINT"))
            .credential(getClientSecretTokenCredential())
            .buildClient();

        var containerClient = client.getBlobContainerClient(System.getenv("CONTAINER_NAME"));

        var blobClient = containerClient.getBlobClient("test_do/" + UUID.randomUUID());
        blobClient.upload(new ByteArrayInputStream("Hello, Azure!".getBytes(StandardCharsets.UTF_8)));
        System.out.println("\nFile uploaded successfully");
    }

    protected TokenCredential getClientSecretTokenCredential() {
        return new ClientSecretCredentialBuilder()
            .tenantId(System.getenv("TENANT_ID"))
            .clientId(System.getenv("CLIENT_ID"))
            .clientSecret(System.getenv("CLIENT_SECRET"))
            .build();
    }

}

Expected behavior
Blob client will not block application from finishing
Screenshots
possible thread that cause waiting

Setup (please complete the following information):

OS: Windows 10 / alpine:3.18.6 docker image
IDE: IntelliJ / -
Library/Libraries: see code snippet
Java version: 17
App Server/Environment: -
Frameworks: Spring Boot

Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[X] Bug Description Added
[X] Repro Steps Added
[X] Setup information Added

Thank you for filing this question @if-of.
Did some investigation, this appears to be the default behavior of OkHttp's Dispatcher class, which in turn by default we don't modify when OkHttp is being used leaving that configuration to the application to handle.
Looking at Dispatcher it uses a ThreadFactory that creates non-daemon threads for the ExecutorService backing the Dispatcher (https://github.com/square/okhttp/blob/master/okhttp/src/main/kotlin/okhttp3/Dispatcher.kt#L103, https://github.com/square/okhttp/blob/master/okhttp/src/main/kotlin/okhttp3/internal/-UtilJvm.kt#L65).
The good news, our OkHttp HttpClient builder allows for configuration of the Dispatcher being used. So, a simple work around could be supplying your own Dispatcher that doesn't use daemon threads, therefore prevents the Dispatcher from preventing JVM shutdown while the threads time out in the ExecutorService.
https://github.com/Azure/azure-sdk-for-java/blob/main/sdk/core/azure-core-http-okhttp/src/main/java/com/azure/core/http/okhttp/OkHttpAsyncHttpClientBuilder.java#L298

Thank you, @alzimmermsft. Your workaround effectively solved my issue, eliminating the 60-second wait time.
P.S: It's quite unfortunate that this isn't the default behavior. However, considering the simplicity of the fix, it's not much of an inconvenience.


Can’t transmit on DVB-T2 on VATEK output plugin - Suntechtv U3 USB Modulator

Bug description:
Trying to do a screen share with Open Broadcaster Software sending it to FFmpeg then to TSDuck.
Selecting DVB-T2 modulation with the tsp vatek output plugin and sending it to FFmpeg and TSDuck hangs and freezes up at around 30 seconds. Also, to note even before if freezes up no signal comes out at all. Tested this with different computers and still the same results.
How to reproduce:
Note this is a code snippet:
tsp -O vatek --bandwidth 7 --bandwidth-extension --device 0 --fft-mode 16K --frequency 585500000 --modulation DVB-T2 --pilot-pattern 7 --plp0-code-rate 5/6 --plp0-fec-type 64K --plp0-high-efficiency --plp0-modulation 256-QAM --plp0-rotation --t2-guard-interval 1/128 --t2-l1-modulation 64-QAM
This command does work however no signal is generated and FFmpeg and TSDuck freezes at around 30 seconds.
Expected behavior:
Should be able to transmit on DVB-T2 and get a signal on my TV / Set top box and FFmpeg and TSDuck should not to freeze up at all.
Errors and logs:
None: FFmpeg and TSDuck freezes up at around after 30 seconds. No DVB-T2 signal is generated at all.
Environment:
OS: Windows 10 21H2
TSDuck full version: 3.35-3258
Installation type: official binary portable
Additional information:
Besides DVB-T2 modulation also tested it with DVB-T and DVB-C (J83A). With DVB-T and DVB-C no problems were encountered. Everything went perfectly fine (this is with FFmpeg and TSDuck).
I also have the VATEK SDK installed on the computer and tried to transmit DVB-T2 with their application. No problems. Works perfectly fine. So therefore, at this current moment I cannot get a DVB-T2 signal generated from FFmpeg connected with TSDuck.
Would like to know if I am missing something or this is a TSDuck bug?

Hi @WinSta0,
Your tsp command only includes the vatek output plugin. Any other plugin? Do you simply pipe ffmpeg output into that tsp command with nothing else than the output plugin?
Richie, @a83912a, could you have a look at this one please?

Thanks for your response.
Here are the commands I use:
Method 1)
ffmpeg.exe -reconnect 1 -i http://<IP_ADDRESS>:8000/live/alex.flv -c:v copy -c:a ac3 -b:a 640k -ac 6 -channel_layout 5.1(side) -ar 48000 -mpegts_service_id 0x400 -streamid 0:0x0462 -streamid 1:0x0463 -muxrate 40000000 -f mpegts -listen 1 http://<IP_ADDRESS>:1234
The command above is FFmpeg grabbing a http FLV stream from a server and converting it to a mpegts http stream.

This is the exact commands that I use with FFmpeg and TSDuck. It dosent matter which FFmpeg method I use: Method 1) or Method 2) TSDuck and FFmpeg freezes.

Hi @lelegard,
Thanks for your response.
Forgive me if I went too detail in my previous post. This time I will answer the questions in a general manner.
Your tsp command only includes the vatek output plugin. Any other plugin? Yes: http, filter, inject, timeref, pcrverify.
Do you simply pipe ffmpeg output into that tsp command with nothing else than the output plugin? No. The output plugins I use when I pipe ffmpeg into that tsp command are: http, filter, inject, timeref, pcrverify and of course vatek plugin.

When a problem occurs in the presence of multiple factors, the first thing to do is to eliminate those factors one by one, until the problem disappears, which identifies the problematic one. Alternatively, start with the simplest possible configuration and, assuming it works, add the other factors one by one, until the problem appears.
Therefore, I suggest to start with the simplest structure, a simple pipe with ffmpeg and tsp on the same machine, without any additional processing:
ffmpeg ... -f mpegts | tsp -O vatek ...

At a prerequisite, make sure that the requested muxrate in ffmpeg is exactly the expected DVB-T2 bitrate, computed from your modulation parameters. 40000000 seems a bit too "round" to be credible. Use an online DVB bitrate calculator if you are unsure about the value.
If the simplest command does not work, this is probably an issue with the TSDuck vatek output plugin.
It it works, only add the intermediate HTTP service. If it works, add other plugins, one by one.

Thanks for your response.
I have just already done a test with FFmpeg and TSDuck with the simplest structure, a simple pipe with ffmpeg and tsp on the same machine, without any additional processing and still the same results.
The FFmpeg muxrate has now been set to 40907500. This value comes from the original Vatek SDK application. Though I have tried with other values in the past before posting the bug in GitHub, I still get the same results (FFmpeg and TSDuck freezing up and no signal coming out at all). I have used multiple DVB bitrate calculator in the past.
It appears from what I have just tested that the simplest command does not work. I have tested this multiple times and still get the same results.

OK, so this must be a vatek plugin issue. @a83912a, could you please have a look ?

OK, I will test this issue.

Hi @WinSta0 ,
Can you please changing the DVB-T2 FFT to 8K and see if it successfully outputs? If all other modulations are working fine and only DVB-T2 is experiencing issues, it might be a problem with our plugin.

Hi @a83912a,
Can you please changing the DVB-T2 FFT to 8K and see if it successfully outputs? Done that and got some good news, works perfectly fine with all the TSDuck plugins. Got signal coming out from the modulator and Video and Audio on the TV.
The 16K FFT however still does not work. I can Imagine that this option is malfunctioning. Probably plugin related. The VATEK SDK default application (vatek_toolkit.exe) can transmit in 16K FFT without any problems, so it’s definitely not the modulator.

@WinSta0
OK, please give me some time to fix this issue. If the fix is completed and deployed, I will notify you. Thank you!

Richie, @a83912a, is there anything to fix in the TSDuck plugin? Or in the libvatek?
I plan a new release for December, maybe January. If you have a fix, I will wait for it. Please let me know.

Hi Thierry , @lelegard,  I plan to update libvatek to improve the stability and bugs in A3. There are currently no modifications needed for the TSDuck plugin. I anticipate the update to be released in mid-December. Thank you.

Thanks Richie, @a83912a. Let us know here when you publish the next binary release of libvatek so that the next release of TSDuck can embed the right version.

Hi Richie @a83912a. Does the recently released VATek SDK 3.10 solve the issue? Is this correct to assume that building a new release of TSDuck with libvatek 3.10 fixes the problem?

Hi Thierry @lelegard. Yes, this version has addressed the issue with DVB-T2 FFT 16K not working, and it includes some modifications to enhance the stability of A3. Thank you.

OK, great, I will probably release TSDuck version 3.36 within a week or so.


How to dedup schemas?

If you take a look at my example project, it generates 26 times the same schema for Foo.
The behavior I would expect is to generate a single schema for Foo in /components/schemas and then use $ref to use it.
Looking at the code, I see the opportunity in ReferenceOr::Reference, but I can't find a place to generate it, the code always uses the other variant

You can enable that behaviour with gen::extract_schemas.
https://github.com/tamasfe/aide/blob/556227c330481f960e99e9ee3889cf06a09965d4/crates/aide/src/gen.rs#L54
It's disabled by default because schemars does no name mangling nor detects collisions so foo::Foo will be overwritten by bar::Foo.

Thank you
Il gio 8 dic 2022, 17:16 Ferenc Tamás @.***> ha scritto:

You can enable that behaviour with gen::extract_schemas.
https://github.com/tamasfe/aide/blob/556227c330481f960e99e9ee3889cf06a09965d4/crates/aide/src/gen.rs#L54
It's disabled by default because schemars does no name mangling nor
detects collisions so foo::Foo will be overwritten by bar::Foo.
—
Reply to this email directly, view it on GitHub
https://github.com/tamasfe/aide/issues/27#issuecomment-1342966991, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABZXINMSS6T7DDDRFLTN3MDWMICUJANCNFSM6AAAAAASW7ZXAM
.
You are receiving this because you authored the thread.Message ID:
@.***>


Required parameter check.

Closing as we do not intend to continue investment in the C# code generator.


The audio file temp_recording.wav was not found. error message has occured.

My environment:
Windows 10, rtx 4090, 32GB ram
Error Message:
Recording saved to audio_files\temp_recording.wav
Transcribing audio file: audio_files\temp_recording.wav
An error occurred during transcription: The audio file temp_recording.wav was not found.
I performed setup items 1-8 and set all configs to local.
(TRANSCRIPTION_API = "whisperx", COMPLETIONS_API = "ollama")
What I said is well saved in 'AlwaysReddy\audio_files\temp_recording.wav'.
I'm not sure why that error message appears.

I had the same problem. For me it was a problem with the ffmpeg installation.
conda install -c conda-forge ffmpeg
fixed it.

I had the same problem. For me it was a problem with the ffmpeg installation.
conda install -c conda-forge ffmpeg
fixed it.

i dont use conda. then how can i resolve that?

`# on Ubuntu or Debian
sudo apt update && sudo apt install ffmpeg
on Arch Linux
sudo pacman -S ffmpeg
on MacOS using Homebrew (https://brew.sh/)
brew install ffmpeg
on Windows using Chocolatey (https://chocolatey.org/)
choco install ffmpeg
on Windows using Scoop (https://scoop.sh/)
scoop install ffmpeg`

That did not help me .....Anyone else have this issue also ?
f:\ZONE PROJ\CHAT-TTS\AlwaysReddy\areddy\lib\site-packages\pyannote\audio\core\io.py:43: UserWarning: torchaudio._backend.set_audio_backend has been deprecated. With dispatcher enabled, this function is no-op. You can remove the function call.
torchaudio.set_audio_backend("soundfile")
Lightning automatically upgraded your loaded checkpoint from v1.5.4 to v2.2.3. To apply the upgrade to your files permanently, run python -m pytorch_lightning.utilities.upgrade_checkpoint C:\Users\pscho\.cache\torch\whisperx-vad-segmentation.bin
Model was trained with pyannote.audio 0.0.1, yours is 3.1.1. Bad things might happen unless you revert pyannote.audio to 0.x.
Model was trained with torch 1.10.0+cu102, yours is 2.3.0+cu118. Bad things might happen unless you revert torch to 1.x.Using WhisperX model: tiny and device: cuda
Press 'ctrl + shift + space' to start recording, press again to stop and transcribe.
Double tap to give the AI access to read your clipboard.
Press 'ctrl + alt + x' to cancel recording.
Press 'ctrl + alt + f12' to clear the chat history.
use_clipboard: False
Starting recording...
Recording started...
use_clipboard: False
Stopping recording...
Recording saved to audio_files\temp_recording.wav
Transcribing audio file: audio_files\temp_recording.wav
Transcription:
Who are you today?
Running TTS: I am your mischievous assistant today.
Adding to queue
Error reading file f:\ZONE PROJ\CHAT-TTS\AlwaysReddy\audio_files\tmp0igjac9k.wav: Error opening 'f:\ZONE PROJ\CHAT-TTS\AlwaysReddy\audio_files\tmp0igjac9k.wav': Format not recognised.

@sirPhoebus Looks like you are facing a different issue. Your transcription is fine - the issue is with TTS. Assuming you are using piper, I'd check if everything is ok with it. Try running a command to check if it's generating audio files properly:
echo 'Hello. This is a test.' | \   ./piper --model en_US-amy-medium.onnx --output_file welcome.wav
If it generates the output file and doesn't throw any error, open the wav file and see if it plays the audio.

Thanks @kaminoer !! In the meantime I found the post on alltalk_tts .
I tried it ...working like a charm and I am very happy with it !!
I would advise to try it also ;)

oooh wow...thanks for the tip...Will look into it right away !


⚠ Bofrost.de has degraded performance

In 28cd067, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1618 ms

Resolved: Bofrost.de performance has improved in d33ef5b after 15 minutes.


instrument.js:109 web3-shh package will be deprecated in version 1.3.5 and will no longer be supported.

Describe the bug
when loading the site, there are many warning on this:
instrument.js:109 web3-shh package will be deprecated in version 1.3.5 and will no longer be supported.
reproduce
go to protected tab

Web3 is displaying this message, nothing we can do
See this


IR sealed subclasses prototype

Proposing this as a contribution to https://github.com/JetBrains-Research/reflekt/issues/71, would productionize it further to pick the single-arg listOf() call if there's only one subtype, emptyList for no subtypes, etc etc.
One open question would be whether or not to support this on untyped KClass types, like KClass<*> or KClass<Any>. I suspect they'd all fall under the general category of error handling where we say the type visible to the compiler is not sealed


The help function includes the -ShowWindow parameter on non-Windows platforms

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
help is a proxy function for Get-Help that is loaded into the initial session state when PowerShell starts. As a proxy function, it defines all the parameters that Get-Help has. The definition of the proxy function was created for Windows PowerShell but never adjusted for non-Windows.
See source code - https://github.com/PowerShell/PowerShell/blob/73630960efa0a84475731bfbee4b46c046be72ad/src/System.Management.Automation/engine/InitialSessionState.cs#L4185-L4187
The proxy function includes the -ShowWindow parameter for all platforms.
Expected behavior
PS /home/sdwheeler> help Get-Command -ShowWindow
help: A parameter cannot be found that matches parameter name 'ShowWindow'.

Actual behavior
PS /home/sdwheeler> help Get-Command -ShowWindow
Get-Help:
Line |
  64 |      $help = Get-Help @PSBoundParameters
     |                       ~~~~~~~~~~~~~~~~~~
     | A parameter cannot be found that matches parameter name 'ShowWindow'.

Error details
No response
Environment data
PS /home/sdwheeler> $PSVersionTable

Name                           Value
----                           -----
PSVersion                      7.3.5
PSEdition                      Core
GitCommitId                    7.3.5
OS                             Linux <IP_ADDRESS>-microsoft-standard-WSL2 #1 SMP Fri Jan 27 02:56:13 UTC 2023
Platform                       Unix
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

PS /home/sdwheeler> lsb_release -a
No LSB modules are available.
Distributor ID: Ubuntu
Description:    Ubuntu 20.04.6 LTS
Release:        20.04
Codename:       focal

Visuals
No response

Thank you @sdwheeler for reporting this issue.Yes, the WG agrees this is a bug.  We will mark this up-for-grabs or look forward to your pr if you'd like to contribute.


armnn Parser for Caffe deconvolution.

I am trying to write a parser for caffe deconvolution to execute through armnn. I have a few questions.

What difference will be there in comparision to caffe convolution parser with respect to weights?
Should I reshape the weights in any specific manner before passing to Deconvolution configure?
What will be the values of inner_border_top & inner_border_right? and
Are there any other things that I need to take care of? for getting the same answer as in caffe deconvolution layer.
Thanks in advance!

Hi @ssj4gogeta,
Thanks for showing interest in ArmNN and sorry for the late response.
We haven't analyzed the Caffe deconvolution layer, so I'm afraid I won't be able to answer some of the questions. You could look into Compute library's implementation, as mentioned below:

Weights should be the same as in the Convolution layer: [W, H, IFM, OFM] and not [W, H, OFM, IFM]
https://github.com/ARM-software/ComputeLibrary/blob/master/arm_compute/runtime/NEON/functions/NEDeconvolutionLayer.h#L84
inner_border_right & inner_border_top have the same value.
https://github.com/ARM-software/ComputeLibrary/blob/master/arm_compute/runtime/NEON/functions/NEDeconvolutionLayer.h#L57

I hope this is helpful.

Hi @ssj4gogeta,
Since we haven't analyzed the Caffe deconvolution layer. I'm marking this ticket as future work and closing. Let us know if you agree or find any other issues.
Telmo


correct fix for logging

Please accept this full fix for the logging issue and push it to pypi (new version 1.2.4)
Thanks

Thanks, can you please publish this version on PyPi ?

Done. Thanks for the PR

Done. Thanks for the PR

hum, for me PyPi is still in 1.2.3 Released: Jul 22, 2021

maybe it will take a second to show up


[Bug] Fix scenarios where --empty flag pushes limit 0 into metadata queries

resolves #1033
Problem
We use {{ relation }} to render the relation name in metadata queries. However, we implemented the --empty flag to add limit 0 when {{ relation }} is used. We wind up with queries like:
describe table (select * from my_relation limit 0)

which is not valid sql.
Solution

Use {{ relation.render() }} in these scenarios (there are many)
Add a test case that confirms we can use these macros while using the --empty flag

Implementation

Merge this PR demonstrating that these tests work (we`re here)
Move the bulk of this PR into dbt-tests-adapter
Create a final PR that removes most of this and uses the dbt-tests-adapter tests

Checklist

[x] I have read the contributing guide and understand what's expected of me
[x] I have run this code in development and it appears to resolve the stated issue
[x] This PR includes tests, or tests are not required/relevant for this PR
[x] This PR has no interface changes (e.g. macros, cli, logs, json artifacts, config files, adapter interface, etc) or this PR has already received feedback and approval from Product or DX

@mikealfare When might one expect these change in a release?  Or is there a way to test these changes on our system before a release?

I will ask my question to a larger audience: @colin-rogers-dbt @McKnight-42 @jtcohen6  When might one expect these change in a release? Or is there a way to test these changes on our system before a release?


Windows 10 animations

Version 1.6 brought up better looking frame, but minimize/maximize animation isn't working now.

I am aware of this issue, (not really an issue actually more of a style change).
It is due to the FormBorderStyle.None property that is set to customize the window frame.  Unfortunately the animation is due to the "default" / "standard" window frame and isn't actually a result of calling minimise on the form.
The only way to make the animation work again is something to do with WinProc calls.  I'll look into this but no promises.  C# is hardly by language of choice

Hope this might help https://github.com/Soundnode/soundnode-app

@kybarg Unfortunately not, that app linked is written in node (javascript).  The framework behind it might be more helpful, I'll keep looking around


a weblet or terraform file to deploy/add a validator

It should be very easy for anyone to deploy a validator

https://github.com/threefoldtech/grid_weblets/issues/631


Fix Python3 compatibility

Yo,
This PR can be able to fixes #5 and https://github.com/avelino/vim-bootstrap/issues/156 issues.

+1


Does extension support EMR 6.1.0?

When choosing emr version, there seem to be two entries for 6.2.0

That's a bug, 6.2.0 is the earliest supported 6.x EMR version as EMR on EKS images first started being published with 6.2.0. I missed 6.3.0. :)


mesh2d_manual example doesn't render correctly

Bevy version
main, since #15524
Relevant system information
SystemInfo { os: "MacOS 14.5 ", kernel: "23.5.0", cpu: "Apple M1 Max", core_count: "10", memory: "64.0 GiB" }
AdapterInfo { name: "Apple M1 Max", vendor: 0, device: 0, device_type: IntegratedGpu, driver: "", driver_info: "", backend: Metal }
Also tested win11+4080
What you did
cargo run --example mesh2d_manual
What went wrong
Example should display a yellow star. But instead it displays some wispy blue lines.

Same issue on Linux

i have isolated the commit that caused this issue. its because of the commit 54006b107b921c7339df4e87a407b348effc4009

should be fixed in #15862

should be fixed with that pr


Prefix Management

Test and add OpenAPI docs to the views.py file for each of the Prefix Management API endpoints:

[ ] api/prefixes/create Issue biocompute-objects/bco_api#61
[ ] api/prefixes/delete Issue biocompute-objects/portal#103
[ ] api/prefixes/permissions/set Issue biocompute-objects/portal#104
[ ] api/prefixes/token Issue biocompute-objects/portal#105
[ ] api/prefixes/update Issue biocompute-objects/bco_api#62

Repurposed as a PR


Plain function's usage of computed property doesn't get considered as dependency

This is resolved now


Enhancement: format (u)int8 types correctly to number, if formatted with %d, &i, %u

If I try to format a (u)int8 number, Boost.Format considers this as (unsigned) char, even if I use %d, %i or %u.
This is a known and rather old issue.
The workaround is to cast the number to (u)int32, before formatting.
Nevertheless the current behavior leads to very subtle bugs, because the developer has to check the type of the number to be printed very carefully.
Plain printf handles (u)int8 numbers in combination with %d, %i or %u correctly.
Boost version: 1.68 (32bit)
Compiler: MSVC 15.8

Just faced with the same issue trying to format GUID to its string representation. I use %02X to format the last bytes, but unexpectedly it was shown as non-US-ASCII character =(.
It there any plans to fix this?

I don't think this is an enhancement. This just bit me and imho it's a plain bug: i specifically ask for some kind of numeric formatting and it simply doesnt. If I specify '%x' and the argument is a string it should just throw.


pick up intersection points using ray

I want to pick up intersection points using ray like follows
  window.addEventListener('mousedown', (e) => {
    if (e.button === 1) {
      const raycaster = new THREE.Raycaster()
      pointer.x = (e.clientX / container.clientWidth) * 2 - 1
      pointer.y = -(e.clientY / container.clientHeight) * 2 + 1
      raycaster.setFromCamera(pointer, camera)
      if (scene) {
        // let points = scene.children.find((item) => item.isPoints)

        console.log('viewer,scene', e, viewer, scene) //scene.children
        const intersects = raycaster.intersectObjects(viewer.children)
        const geometry = new THREE.BoxGeometry(0.01, 0.01, 0.01)
        const material = new THREE.MeshBasicMaterial({ color: 0x00ff00 })
        const cube = new THREE.Mesh(geometry, material)
        // points.raycast(raycaster, intersects)
        console.log('intersects', intersects[0], intersects)
        // let intersects1 = intersects.filter((item) => item.object.isPoints && item.distance >= 0.00001)
        // console.log('intersects1', intersects1)
        intersects[0] && cube.position.copy(intersects[0].point)
        scene.add(cube)
      }
    }
  })

But I can only get intersection points in a invisible sphere at the origin like in the following image. And it is not possible to get intersect point outside the sphere area.

Ah, right now raycasting with a splat mesh doesn't work with three.js' raycaster, because raycasting against a splat mesh works differently (it returns intersections with splats, not three.js meshes).
Here is an example of how to use the raycaster that I made for this library specifically for raycasting against splats:
const viewer = new GaussianSplats3D.Viewer();
viewer.addSplatScene(path)
.then(() => {
   viewer.start();
   viewer.renderer.domElement.addEventListener('pointerdown', onMouseClick, false);
});

const onMouseClick = function(mouse) {
  raycastSplatMesh(mouse.offsetX, mouse.offsetY);
}

const raycastSplatMesh = function() {

    const renderDimensions = new THREE.Vector2();
    const toNewFocalPoint = new THREE.Vector3();
    const mousePos = new THREE.Vector3();
    const outHits = [];

    return function(x, y) {
        viewer.getRenderDimensions(renderDimensions);
        mousePos.set(x, y);
        outHits.length = 0;
        viewer.raycaster.setFromCameraAndScreenPosition(viewer.camera, mousePos, renderDimensions);
        viewer.raycaster.intersectSplatMesh(viewer.splatMesh, outHits);
        if (outHits && outHits.length > 0) {
          console.log(outHits[0])
        }
    };

}();

This solves my problem, thank you.

Great, glad that fixed it!


Make BQM able to be deep copied when not writeable

Closes #524

Codecov Report

Merging #527 into master will increase coverage by 0.91%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #527      +/-   ##
==========================================
+ Coverage    92.7%   93.62%   +0.91%
==========================================
  Files          49       49
  Lines        3443     3452       +9
==========================================
+ Hits         3192     3232      +40
+ Misses        251      220      -31

Impacted Files
Coverage Δ

dimod/binary_quadratic_model.py
95.63% <100%> (+0.31%)
:arrow_up:

dimod/utilities.py
96.29% <100%> (+0.25%)
:arrow_up:

dimod/sampleset.py
94.84% <0%> (+0.51%)
:arrow_up:

dimod/higherorder/polynomial.py
93.68% <0%> (+0.97%)
:arrow_up:

dimod/serialization/format.py
97.42% <0%> (+1.03%)
:arrow_up:

dimod/decorators.py
71.32% <0%> (+1.47%)
:arrow_up:

dimod/views/bqm.py
91.93% <0%> (+1.61%)
:arrow_up:

dimod/testing/asserts.py
96.9% <0%> (+2.06%)
:arrow_up:

dimod/reference/composites/scalecomposite.py
95.55% <0%> (+2.22%)
:arrow_up:

... and 7 more

Continue to review full report at Codecov.

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


Images not being pushed to docker hub

The last update on docker hub seems to be 2 years ago. The GitHub actions seems to skip over the docker hub image push.
Context: I need the WHOAMI_PORT_NUMBER to be set, but the latest image on docker hub does not support that.

Hello,
the image is available on the Docker Hub https://hub.docker.com/repository/docker/traefik/whoami
The images are pushed to Docker Hub when a tag is created.

Ahh, looks like I missed that piece of info. Thanks so much!


[Question] - project.json RC2 - tools and dependencies section

I have noticed that some tools are listed just in the tools section, and others are listed in the tools section and the dependencies section of the project.json file.
e.g - this is just in the tools section
"Microsoft.AspNetCore.Server.IISIntegration.Tools": {
      "version": "1.0.0-preview1-final",
      "imports": "portable-net45+win8+dnxcore50"
    },

Where as razor tools is in the tools and the dependencies section:
 "Microsoft.AspNetCore.Razor.Tools": {
      "version": "1.0.0-preview1-final",
      "imports": "portable-net45+win8+dnxcore50"
    },

and
 "Microsoft.AspNetCore.Razor.Tools": {
      "version": "1.0.0-preview1-final",
      "type": "build"
    },

I thought perhaps this was basically the difference between tooling that is used purely at development time (i.e just in the tools section), and tooling which is also distributed as a runtime dependency with your application (i.e in the dependencies section)?
However - I noticed the packages in the dependency section have a "type" set to "build" - which I am guessing is the indicator that it shouldn't be treated as a runtime dependency?
so - why are some tools packages present in both sections and others only in one?

@dazinator it's more about whether a tool needs "special" handling during restore. If for example its dependencies aren't all aware of netstandard1.x TFMs, the tool must also be described in the dependencies section. A lot of this boilerplate will be removed as we move to the new netstandard1.0 Json.NET version (9.0.1-beta1). But the capability of overriding a tool's dependencies will remain.

This issue is being closed because it has not been updated in 3 months.
We apologize if this causes any inconvenience. We ask that if you are still encountering this issue, please log a new issue with updated information and we will investigate.


Recreate media player slide for communion

The previous slide has gone AWOL.

I found the existing slide and put it here: C:\Users\livestream.CCW2301\ATEM Media Pool\ATEM Communion.gif


Justified Nav example is broken

nav-tabs its come to down

Sorry, but I don't understand your description. Please give more details. See https://github.com/twbs/bootstrap/blob/master/CONTRIBUTING.md#bug-reports for how to write a better bug report.

Might be referring to the broken clearfixes on v4 navs?

![menubar](https://cloud.githubusercontent.com/assets/3354478/10187930/4dc82b06-6779-11e5-93e3-b686fbc0b3a7.jpg
enclosed the issue image

Same behavior in Chrome browser (Version 45.0.2454.101 m)

using a display: inline-block on both the <a> and the <li> wil fix this issue somewhat.
Now I run into the problem that there is spacing between the list items in the HTML file which I removed by using comments. Link to Stack Overflow
Also it seems there is no border-bottom on the last <li> item.

Can't dig to it deeper for now. Someone can follow up on this?

X-Ref: #17631

@mdo Could we get an explanation for why justified navs were removed?

Justified navs and button groups were super bugged out in Safari when you'd resize your browser.; things would break until you refreshed. Rather than work around that, I'm pushing for us to implement these options in flexbox for more control, albeit with less browser support (sorry IE9).
Example usage in the docs coming in #21201, but not in a separate /examples instance.


Change disabled tests to use [TestProperty("Ignore", "True")]

Change disabled tests to use [TestProperty("Ignore", "True")] instead of commenting out the value so that they are still detectable.

/azp run

I think you forgot to update the following tests:
https://github.com/microsoft/microsoft-ui-xaml/blob/b24dcb014522cc32ec4a1b459c4fb2afdd539ca2/dev/Materials/Reveal/APITests/RevealTests.cs#L27
https://github.com/microsoft/microsoft-ui-xaml/blob/b24dcb014522cc32ec4a1b459c4fb2afdd539ca2/dev/RadioMenuFlyoutItem/InteractionTests/RadioMenuFlyoutItemTests.cs#L58
https://github.com/microsoft/microsoft-ui-xaml/blob/b24dcb014522cc32ec4a1b459c4fb2afdd539ca2/dev/PullToRefresh/ScrollViewerIRefreshInfoProviderAdapter/InteractionTests/ScrollViewerAdapterTests.cs#L43

Thanks @chingucoding ! My search method didn't find tests in test classes which had 0 tests.

/azp run


Added a table for Hangman words

Added an entity, and migration for the database and modified the game to not rely on the hardcoded wordlist. Also removed that list and interface that are not used anymore. This implements issue #69
Again this has an empty merge commit (02f5c17) from my local repo. Hope you wont mind

Not sure if this was what you mean, but I changed this branch to rebase onto the other pull request. After al this I figure what happens If you reject the previous pull request. Then this one can't be accepted also. Chips, this wasn't a good idea, I misunderstood?
So what is the best approach if you work on 2 separate issues. Base them both on an up to date master, as I did initially?

And to answer your question, if you're 90% sure that the first change will get merged in, doing it like this is fine. This just forces an order to the merges, so this approach is fine when the change you're depending on is a simple, one-line change.
If you aren't sure, just go back to master and branch off again. Then when one gets approved, you merge/rebase the new master branch. In a team environment, if disciplined, you can get things merged in quickly. In teams I mentor, we try to get it so that PRs never sit more than an hour before merging or requiring changes.


[VS Code] secret_password_store_sync() failed with error code 2: Cannot create an item in a locked collection

Error:
secret_password_store_sync() failed with error code 2: Cannot create an item in a locked collection
Steps to Reproduce:

signing in in linux with github account
pasting the code in vs code

Version Data

extensionName
VSLS

extensionVersion
0.2.405

protocolVersion
2.1

applicationName
VSCode

applicationVersion
1.22.2

platformName
Linux

platformVersion
4.13.0-38-generic

@axyz What distro are you using?  Most likely you have not installed the needed gnome-keychain and libsecret pre-requisites.

@Chuxel I'm using xubuntu, don't use gnome, but gnome-keyring is installed

@axyz You should be getting a password prompt when VS Code starts up asking you to unlock a keyring. This will unlock the login keyring.  If you are not seeing that prompt then something is up.
First, go to "Session and Startup" and ensure "Launch GNOME services on startup" is checked. If not check it and restart.
If it is checked, install seahorse and then go to "Passwords and Keys".  You should see a "Login" keyring.  Try to unlock it.
Unclear why this is a problem in your situation as I've got an install of Xubuntu 16.04 that works fine once gnome-keyring is installed.


Fix credentials may break Flink engine launch command

Why are the changes needed?
Currently, Flink engine doesn't use delegation tokens and these tokens need to be filtered out from the Flink engine launch command, or the command may be corrupted because the credentials could contain new lines.
How was this patch tested?

[x] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[ ] Run test locally before make a pull request

Was this patch authored or co-authored using generative AI tooling?
No.

Test failure seems related

@pan3793 Lots of tests failed at the uncompress in job setup stage, is that a known issue?

Should be a transient issue of GitHub.

Codecov Report

Merging #5238 (5e2403a) into master (8c512f4) will not change coverage.
Report is 9 commits behind head on master.
The diff coverage is 0.00%.

@@          Coverage Diff           @@
##           master   #5238   +/-   ##
======================================
  Coverage    0.00%   0.00%
======================================
  Files         590     590
  Lines       33344   33345    +1
  Branches     4403    4403
======================================
- Misses      33344   33345    +1

Files Changed
Coverage Δ

...ache/kyuubi/engine/flink/FlinkProcessBuilder.scala
0.00% <0.00%> (ø)

... and 1 file with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Merged to master/1.8


Modify storage path structure

Hi,
I'm using multisite. Is it possible change path structure like WordPress's structure.
For example:
WordPress: /(site-id)/(year)/(month)/file.jpg
Top level folder will dynamically create site-id folder.
Thanks!

 If so, seems like this idea has been raised before at https://github.com/wpCloud/wp-stateless/issues/149

Yes, I shared bucket across the entire network.
Are you still working on it?
Thanks!

Not yet, but please share your interest in the original issue. I'm thinking we can bump the priority since others have requested it. Thanks


⚠ Bofrost.at has degraded performance

In 9b377ca, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 3221 ms

Resolved: Bofrost.at performance has improved in c883f0d.


How to generate Access token dynamically?

Here, with provided sample, I can see the embedded token is statically added. How can we generate it dynamically?

Use one of the Embed Token REST APIs of PowerBI
https://docs.microsoft.com/en-us/rest/api/power-bi/embedtoken


update calc_open_short to match solidity

Resolved Issues
https://github.com/delvtech/hyperdrive-rs/issues/29
https://github.com/delvtech/hyperdrive-rs/issues/121
Description
This PR updates the Rust open short and derivative implementations to more closely match Solidity.
Lower tolerances on tests 🎉

After the fix, I was able to completely remove the error tolerance on fuzz_calc_open_short.
I also updated the epsilon & tolerance values for the fuzz_short_deposit_derivative and fuzz_short_principal_derivative tests. I use a larger epsilon and smaller tolerance, which I think is what we prefer (more distant points should match linearity assumptions better due to the lack of monotonicity coming from pow).

Increased tolerances on tests 😭

I had to increase error tolerance for fuzz_calculate_max_short_no_budget from 1e12 to 1e16. I compared the rust & solidity implementations, but they're too different from each other to find any bugs by pattern matching. I'm not sure what is going on here, and would suggest we open a new issue to investigate. (more on this below)
I was hoping that this PR would allow me to reduce the additional bond amount in fuzz_error_open_short_max_txn_amount (per this comment), but could not (it is still 100M). This is probably related to lingering issues with calculating the max short.
I had initially thought this PR fixed fuzz_calculate_max_short, but I think the more likely scenario is the test wasn't set up to consistently fail for it's condition. I modified the test in my last commit and now it is consistently passing.

(relatively) Inconsequential updates

I updated some of the derivative tests to use an equivalent, but simpler method for numerical differentiation that only applies the epsilon once and therefore makes the distance between points easier to understand from the empirical_derivative_epsilon parameter. I also changed the tolerance check to be <= instead of < for the same reason.
I removed the use of calc_max_short in the preamble & open short tests. While on working this PR I was getting frustrated because the open short tests were failing due to errors in max short. This is a better setup in general, so that our tests behave more like unit tests that are easier to debug than integration tests.
I renamed long_amount_derivative to calculate_open_long_derivative and short_deposit_derivative to calculate_open_short_derivative.
I moved some arguments out of fn signatures and instead use state member variables. This is a departure from Solidity but (imo) justified because the Rust fns are all implemented on the state struct. The preferred pattern (and what we do elsewhere) is to update state and call the fn instead of passing different values as fn args.

Important items to check

My changes to the short deposit derivative and calc open short were not algebraic -- I believe my function is entirely different from what was there. My guess is calc open short now matches solidity, but the derivative does not. I think it's worth comparing this against solidity to make sure there aren't errors in the solidity implementation of the derivative. It could explain the next point:
The error in fuzz_calculate_max_short_no_budget might be due to a difference in max_short_guess or an error in the solidity short derivative fn. I still think this is best investigated in a follow-up issue, but I'm noting it here as a clue for the future.

Math outline
The below formulation follows the logic outlined in HyperdriveShort.sol. I made some slight changes to variable names and ordering of presentation.

The fuzz_calculate_max_short is failing again after I pushed up a rebased version of the branch. So that problem is not fixed. I'm going to remove it from the PR desc as "solved".
Looking at the current failure & the one reported in https://github.com/delvtech/hyperdrive-rs/issues/121 I'm thinking the amount it is off is quite large -- not just a matter of bumping some error tolerance.

closing in favor of https://github.com/delvtech/hyperdrive-rs/pull/152


append identity file with local-ssh

What this PR does / why we need it:
Appends the -i <identity-file-path> when custom identity-file is specified with --local-ssh option for virtctl.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #7071
Special notes for your reviewer:

I am still familiarizing myself with the kubervirt's flow. Please let me know, in case this change leads to an unexpected behavior. :)

Release note:
Don't ignore --identity-file when setting --local-ssh=true on `virtctl ssh`

/test all

/lgtm
would be nice to have a unit test but I can see why you didn't add one -- your commit is based on a tree that's a few months old so the unit tests for this file don't exist in it.
Let's see if I can also write a release note in a reply:
Fix issue #7071: apply --identity-file when setting --local-ssh=true

/lgtm
/approve
Thanks! Sorry for the delay.

/retest

/ok-to-test
/retest

/retest

/retest


[Issue]: ODR Violations Due to Missing inline Specifiers in bfloat16 Conversion Functions

Problem Description
Starting from ROCm v5.7, the introduction of certain bfloat16 conversion functions in the header files has led to One Definition Rule (ODR) violations when building projects. This is due to some host functions not being specified as inline or static, resulting in linkage errors across multiple translation units.
Temporary Workaround
A temporary workaround involves manually modifying the header file to add the inline keyword to the __HOST_DEVICE__ macro definition. Specifically, changing line 96 in /opt/rocm/include/hip/amd_detail/amd_hip_bf16.h to:
#define _HOST_DEVICE_ _host_ _device_ inline

This resolves the linkage issue but is not a sustainable solution.
Expected Behavior
The host functions should be defined with inline or static specifiers to prevent ODR violations and ensure that the header files can be safely included across multiple translation units without causing linkage errors.
Additional Context
It appears that there is an ongoing effort to fix this issue, as seen in the commit 86bd518981b364c138f9901b28a529899d8654f3. However, this fix does not seem to be included in any of the ROCm releases.
Users attempting to install vLLM on ROCm, specifically after vLLM-rocm is merged into the mainline vLLM, may encounter issues due to the aforementioned ODR violations. It would be beneficial for the community if such fixes were included in an official ROCm release to avoid the need for manual intervention and to ensure clean and maintainable codebases.
Operating System
Ubuntu 22.04.3
CPU
AMD EPYC 7763 64-Core Processor
GPU
AMD Instinct MI250, AMD Instinct MI210
ROCm Version
ROCm 6.0.0, ROCm 5.7.1
ROCm Component
No response
Steps to Reproduce

Build a project on ROCm that includes the bfloat16 conversion functions from the header file /opt/rocm/include/hip/amd_detail/amd_hip_bf16.h.
Observe the ODR violations in the build process when linking multiple translations units including such header.

(Optional for Linux users) Output of /opt/rocm/bin/rocminfo --support
No response
Additional Information
No response

Its already added
https://github.com/ROCm/clr/blob/c398c75512f099227244a3d13e32d79be18fdce3/hipamd/include/hip/amd_detail/amd_hip_bf16.h#L102
Should reflect in upcoming release

Thanks for the update. Is there a possibility of having a maintenance release for ROCm-5.7 like ROCm-5.7.x?

Hi @tanpinsiang, can you please check if your issue is resolved with the latest ROCm 6.2? If resolved, please close the ticket. Thanks!

Hi @tanpinsiang, I am closing the ticket as I have confirmed that the update is in the newest release. Please feel free to reopen it or ask follow-up questions should further assistance be required. Thanks!


Host parsing

Refactor host parsing logic. Allow wildcard through.
Closes #1855 and #2323

Coverage decreased (-0.003%) to 93.102% when pulling 1fb06e6a085d52e7e237a0b3d8f833739a016cfe on host-parsing into d82423788cf372134a9c7d055c8293b281e61803 on master.


Do Screenshots on a Qt based Distro

Screenshots promoted by argos are token from a GNOME display manager. Gnome itself has not the best integration with Qt. Gnome uses Gtk.
Maybe take these screenshots on a display manger with a better Qt integration like KDE Plasma.

Can you make a pull request, I'll look at it? One reason to potentially keep Gnome is that most of the users current are from Snapcraft/Ubuntu which defaults to Gnome

Can you make a pull request, I'll look at it? One reason to potentially keep Gnome is that most of the users current are from Snapcraft/Ubuntu which defaults to Gnome


Make normalize function more resilient.

Probably not worth merging, but I thought I would share my findings.
In reference to https://github.com/jonathantneal/flexibility/issues/34, https://github.com/jonathantneal/flexibility/issues/23, & https://github.com/jonathantneal/flexibility/issues/61.
Basically the library was calculating whack styles on ie8 for some reason where the values would be something like "auto-5" or "0auto-255". Then when piped through the normalize function in index.js just "px" would be appended to the end. Then when trying to set a style of "0auto-255px" for anything it would barf. Now I'm checking to see if we're dealing with an actual number, if not then I just give it "0px".

Is this being considered?


Broadcaster periodic creation of secret

Description
This PR fixes a problem in the Advertisement broadcaster.
When 2 clusters are peered, if a network error occurs the broadcaster continues to work, logging an error when trying to create the Advertisement.
If the Advertisement expires it is deleted, and the secret containing the kubeconfig for the virtualKubelet is deleted as well. However, when the connection between the clusters is re-established, the Advertisement is recreated but the secret isn't, therefore the VirtualKubelet cannot start.
Now the secret for the VirtualKubelet is periodically created (or updated) along with the Advertisement.
Other changes

added a backoff when the creation of Advertisement fails (to reduce the logging)
in the Advertisement operator, when the Advertisement expires try to gracefully delete it before removing it directly

Pull Request Test Coverage Report for Build 264950214

4 of 50 (8.0%)  changed or added relevant lines in 2 files are covered.
88 unchanged lines in 5 files lost coverage.
Overall coverage decreased (-0.3%) to 51.065%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

internal/advertisement-operator/controller.go
0
5
0.0%

internal/advertisement-operator/broadcaster.go
4
45
8.89%

Files with Coverage Reduction
New Missed Lines
%

internal/advertisement-operator/controller.go
6
13.25%

internal/kubernetes/endpoints.go
8
73.96%

internal/kubernetes/pod.go
12
9.21%

internal/liqonet/tunnelEndpointCreator-operator.go
23
65.5%

internal/advertisement-operator/broadcaster.go
39
48.1%

Totals

Change from base Build 261773825:
-0.3%

Covered Lines:
6858

Relevant Lines:
13430

💛  - Coveralls

/rebase

/rebase

/rebase

/rebase


theme-color is not updated when the PWA is inactive.

VS Code Version: 1.97.0-insider
OS Version: PWA (Mac OS, but should apply to all)

Steps to Reproduce:

Set the colour titleBar.inactiveBackground to something significantly different to the activeBackground to easily distinguish the focused app.
Focus on any other app.

Notice that even when the PWA is inactive, it still takes the theme-color from the active titlebar background.
https://github.com/microsoft/vscode/blob/main/src/vs/workbench/browser/style.ts#L27-L41
I think that ideally the theme-color would match the titlebar colour, regardless of whether the app is active or not.

I was able to fix the reported problem, and the title bar renders correctly when merged together with the PWA title bar. When the title bar is not merged with the PWA title bar, there seems to be a border with a different color...
 https://github.com/user-attachments/assets/df6651a3-bce0-45bd-88ce-96cc949aa23d


Documentation: Changing "combinatorial" to "combinational"

Resolves #1301
The first two commits update the documentation and are simple find-and-replace in python docstrings and rst, respectively.
When rebasing, an additional search may be needed.
The third commit adds additional content and should be reviewed by project maintainers more closely.

@whitequark , ready for review

@whitequark , heh you are too quick for me. Will take a look at #1333 and rebase.

@whitequark rebased and ready for review


Several issues

Hi team,
Shot in the dark maybe, but trying to reach help.
Outdated docs
Here: https://learn.microsoft.com/en-us/entra/external-id/microsoft-account you guys show outdated documentation
It shows B2C images, while you advertise with External ID:

Invite external federated Microsoft account with MS Entra Extranal Id
When I try to invite a federated Microsoft account (private<EMAIL_ADDRESS>account) I can accept the invitation (got an email), awesome.
When I finish the user flow (accept permissions and filled in personal stuff), I land at : https://account.activedirectory.windowsazure.com/ ??

How to solve this, how to configure?
But In the users section of entra id, the user has been marked as "Accepted" invitation.
Then I try to login to our own application with the invited user email address.
But I can NOT login afterwards with the same email address.

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

ID: a8c91376-cbbb-ec1b-4318-bc870622e9e1
Version Independent ID: 2354aa2d-0271-f7c8-6639-06d65250effe
Content: Add Microsoft account (MSA) as an identity provider - Microsoft Entra External ID
Content Source: docs/external-id/microsoft-account.md
Service: active-directory
Sub-service: b2b
GitHub Login: @msmimart
Microsoft Alias: mimart

Hi @icazemier
I'm sorry to hear that you are experiencing issues with the Microsoft External Id feature.
Regarding the outdated documentation, thank you for bringing this to our attention. I will make sure to pass this feedback along to the appropriate team to update the documentation.
Regarding the issue with the user flow, it is possible that there is a misconfiguration in your Azure AD tenant that is causing the issue. I recommend checking the following:
Make sure that the user is added to the correct Azure AD tenant and that the tenant is configured to allow external users to sign in.
Check the Azure AD sign-in logs to see if there are any errors or issues with the user's sign-in attempts.
Make sure that your application is configured to allow external users to sign in and that it is properly configured to use the Microsoft External Id feature.
If you continue to experience issues, I'd recommend posting your question in https://azure.microsoft.com/en-us/support/community/. They will be able to help you troubleshoot the issue and provide guidance on how to resolve it.


Enhance osiris build for windows & linux environment

make it windows available to run in cmake
cmake one line build command for windows & linux
option LINKER:--no-undefined for linux only

I don't think we should include building with CMake on Windows in the README because it has no advantage over building from VS project and requires additional software (CMake).
CMake configuration is different from the build command: configuration is done once when changing build type or compiler while the build command is execute multiple times without reconfiguration. These two commands together wastefully perform reconfiguration before every build.
if(NOT CMAKE_CXX_COMPILER_FRONTEND_VARIANT STREQUAL "MSVC") already prevents LINKER:--no-undefined under MSVC.


UI improvements to QR code page

Tweaks and improvements to the QR code page. See here for inspiration.

Resolved by #116

Resolved by #116


Update CONTRIBUTING.md

Description

Update contributing to make note of the use of default values.

Issue ticket number
Closes #xxxx

Checklist before requesting a review

[x] I have performed a self-review of my code

[x] Conforms to the Contributing guidelines

[ ] Proposed changes are described in the CHANGELOG.md

[ ] I have tested my code with viash ns test --parallel -q <name or namespace>

Check the correct box. Does this PR contain:

[ ] Breaking changes
[ ] New functionality
[ ] Major changes
[ ] Minor changes
[x] Documentation
[ ] Bug fixes

Fixed by #82


OLS-319 sprint4 schema

Hi,
These are the changes on the Schema to support the Sprint-4 features of OLS.
The various details about the requirements, Schema definitions, and review comments during the course of planning and sprint development phases are captured in the Confluence feature links of OLS 3.2 here:
https://telecominfraproject.atlassian.net/wiki/spaces/WIFI/pages/2234810369/Release+3.2+Items
If there is something that is missing on what we intend to cover in 3.2 which is not in the Schema or any errors, please do let me know.  Request you to review and approve, try by 20-September please.
Thanks
Binny..

Hi @binnyjeshan, @mikehansen1970
I've left some comments, but these are mostly to-start-a-discussion, rather than critique comments (changes look fine by me +1; but i'd like to have all inlined comments discussed)

Hi @mikehansen1970
I have made changes and pushed in.
Please take a look again now, and let me know if i missed anything. Thanks.

Hi @mikehansen1970 I have made changes and pushed in. Please take a look again now, and let me know if i missed anything. Thanks.

Hi @binnyjeshan
Approved, thank-you.


unittest: support ztest_new_api

Add support for the new Ztest API when using unittest.
Fixes #47420
Signed-off-by: Yuval Peress<EMAIL_ADDRESS>

Looks good, but I need a few more hours for testing this more extensively


Detect(single|all)Face intermittently only returns undefined on certain browsers

I've been experiencing this for a while, in my case I can replicate on Chrome on Ubuntu.. I can load the page 10 times, and maybe 4 out of those 10 times, Detecting faces doesn't work and returns undefined. I can lean into the camera, etc but it never detects a face (When its running in a loop at 1 second intervals)
However, I can refresh, and then it works fine, same code.. I can keep refreshing and it stops again. So far I have only ever seen it happen in chrome but due to its intermittent nature its hard to replicate. On the same PC I switch to FireFox and cannot replicate.
There are no errors in the console, and literally no clues!
I'm posting in the hope someone else may have experienced, or any pointers of what to check / where to debug.
I'm using ReactJS

Perhaps you're facing the some problem I had. Depending on your face position inside the picture, the library will not return a valid result. Try to duplicate the image width, leaving the original picture at left.

I am not seeing this as ur use case is something that we do as well. Capture face every 1 second. We have done this for upto 5 mins and have not seen a loss of data as mentioned. Maybe it is Ubuntu + Chrome + camera combination. I don't use Ubuntu but have tested it on Mac and windows. No issues as you mentioned.

I'm thinking something in React is causing this. I think I'll pull down the dev version of faceAPI and put debug statements everywhere!


Adds retraction status

This PR checks if a doi has been retracted.
Reaction data downloaded on 09-94-24 is added to paperqa/clients/client_data
Will update the retraction dataset every 30 days.
Pulled from the spetember-branch
Closes #313

Test fail due to: ModuleNotFoundError: No module named 'pandas'
Use dict instead?

Let's not open into main, open into september release

Let's not commit the CSV @geemi725 - it's pretty big

Not really sure why test_title_search failing.
Possible fix:   instead of ALL_CLIENTS we could specify the clients [CrossrefProvider, SemanticScholarProvider, JournalQualityPostProcessor]  in the test.  Avoid calling RetrationDataPostProcessor.
https://github.com/Future-House/paper-qa/blob/99a4a63325af5167e90bc07b28cf9b2d74624ad4/tests/test_clients.py#L101

Removing redundant RetrationDataPostProcessor calls in test_client.py worked!


Support ES256 algorithm for JWT plugin

Currently the JWT implementation supports RS256. It would be nice if ECC signatures were supported. ES256 and ECC in general are more modern crypto algorithms, and the keys and signatures are much more compact.

I'm willing to take a stab at PR on this, but I wanted feedback on the idea, first to see if it would be accepted.

Sure!, looking forward to it!

fixed by #1920


Cannot set mode for executable files via mode on pkg_tar

Noticed after updating from 0.8.1 to 0.10.1, and narrowed down to most likely being as a result of https://github.com/bazelbuild/rules_pkg/pull/756, specifically the following line, https://github.com/bazelbuild/rules_pkg/blob/main/pkg/private/pkg_files.bzl#L411, its no longer possible to set a mode for executable files via the mode attribute on pkg_tar, as it gets forced to 0o755.
Non-executable files can still be set, as their values don't end up in the manifest (and the manifest takes preference over flags to build_tar.py) due to the default_mode being set to None.
This makes it impossible for us to set a more restrictive mode of 0o555 on executable files.

Does setting the mode work with pkg_files | pkg_tar?


APIv2: decide reset behavior of unmarshal

There are 3 unmarshal functions:

proto.Unmarshal
encoding/protojson.Unmarshal
encoding/prototext.Unmarshal

Currently, proto.Unmarshal does not reset the message when unmarshaling, while the later 2 do. I believe these should be consistent.
If the message is not reset, then there needs to be well-defined merge semantics for the serialization. Such semantics exist for wire and text serialization. However, JSON lacks any defined merge semantics and we should not invent any.
Given that JSON has no merge semantics, it seems to me that all 3 Unmarshal functions should default to reseting the message (to be consistent within our module). To provide merge-semantics, we should provide a AvoidReset option for wire and text unmarshaling.
\cc @neild @cybrcodr

V1 text unmarshaling does reset the message, while interestingly V1 JSON unmarshaling does not.
Having consistency in V2 would be nice, and I like your suggestion.

Something to investigate: The C++ JsonStringToMessage function takes in a *Message. Investigate whether they clear this message or merge into it. If it merges, then there is existing precedence of merge-semantics in JSON.

On head, JsonStringToMessage source shows it calls into JsonToBinaryString which constructs a serialized binary and then calls Message::ParseFromString which resets the message.

Not resetting by default has the downside of being a change from v1 and not being entirely obvious. (C++/Java put "Merge" in the name of the non-resetting functions, which makes it very clear that they preserve existing data in the message.) I think this makes me lean slightly toward reset-by-default as being the best approach.
Defaulting to the better performing version is appealing, though.

Not resetting by default has the downside of being a change from v1

Another downside is how a migration tool would work to make a semantically equivalent transform from v1 to v2.
If the tool sees:
protoV1.Unmarshal(b, m)

It would have to re-write it to:
protoV2.Reset(m)
protoV2.Unmarshal(b, m)

unless it can prove m is empty. A cursory glance over code at Google shows that a majority of call-sites have something like:
var m foopb.MyMessage
err := proto.Unmarshal(b, &m)

or
m := new(foopb.MyMessage)
err := proto.Unmarshal(b, &m)

in which case, the logic to prove emptiness may be fairly easy for this common case.

The original posted suggestion by @dsnet does provide a solution for v1 wrapping v2 via an option for binary and text unmarshaling, i.e. making the default for all v2 APIs consistent while having the v1 wrapper use the option to preserve v1 compatibility in its API.  JSON unmarshaling can implement such option at later point when a spec is defined.
I would suggest a different option name though, say Merge or AllowMerge or something.
When you mentioned that there's an existing merge semantic for text proto serialization, I take it you meant whatever the C++ lib does.

I'm going through Google code to clean up uses of RequiredNotSetError, and one thing which strikes me is that almost every call to Unmarshal really should be UnmarshalMerge. I'm sprinkling in Merge: true as I do this cleanup, but it seems clear that the vast majority of users going forward are going to get a reset that they don't really need.
Perhaps this is irrelevant; it adds a minor performance cost to unmarshal operations, but that's it.
But I think we should consider one last time what we want for a default. The arguments in favor of reset-by-default were, I believe:

Easier automatic rewrites from APIv1 to APIv2.
protojson has no merge semantics.

We can fix the automatic rewrite problem by adding a Reset field to the options; then an automatic v1->v2 rewrite becomes:
err := proto.Unmarshal(b, m) // old

err := (proto.UnmarshalOptions{Reset: true}).(b, m) // new

The new version is more verbose, but that's okay--it encourages someone to go remove the Reset: true if they don't want it.
The lack of JSON merge semantics could be addressed by returning an error if a merge would be required, as suggested by @dsnet. For efficiency, we can probably just consider fields that are set in the JSON. That's a form of merge semantics, but it seems like a harmless one.
Or perhaps we should stick with what we have as good enough. Most people won't care about the performance cost, and adding Merge: true to a few extreme hot-path functions (RPC packages, etc.) will probably be enough.

I don’t know that having Merge: true is so bad. Similar to having it alternatively Reset: true it might just encourage people to actually think about if they really need the other functionality or not.
Make the common case simple, and the rare case hard.

It makes me a bit sad, because most Unmarshal calls are going to be a little bit less efficient than they should be, but sticking with the current semantics seems like the best option. The only downside is a small performance cost, avoidable by setting Merge: true.


[Keyboard Navigation - Make Code - Blocks]: There is no keyboard access for "Input Section" controls under "Blocks"

User Experience:
Users who depend on keyboard navigation will be impacted if controls are not accessible with keyboard and they will not be able to perform any desired action with the keyboard.
Repro Steps
Open Microsoft MakeCode for micro:bit (microbit.org).
Select the 'Blocks' option in header.
Now verify that user is able to insert the different category blocks in editor area or not without drag and drop feature.
Actual Result
There is no keyboard access for "Input Section" controls under "Blocks"
Observation:
While focus is on "Input" control , after pressing tab , right/left arrows and ctrl+arrows focus is not moving to the Right section of input controls.
Expected Result:
"Input Section" controls should be accessible through the keyboard.
MAS Reference
MAS 2.1.1 – Keyboard
Reference Links:
Accessibility Insights! - Identify accessibility bugs before check-in and make bug fixing faster and easier.
External Bug Process: If this bug belongs to external team, mark it as resolved/done and assign it back to the tester with notes on where to file/route the bug. For more information, please use this External Bug Process (sharepoint.com). “HCL Staff should not log any third-party external bugs. Should be routed to EDAD team”.
Please reach out to C&AI Teams channel for any process-related queries.
Test Environment:
OS Version: 22H2(OS Build: 22621.819)
Browser Microsoft Edge Version 109.0.1495.2(Official build) dev (64-bit)
URL: https://makecode.microbit.org/#editor
https://user-images.githubusercontent.com/117158108/207312611-3f55d19e-7a6b-4b46-8270-54030f02edd6.mp4

#A11yMAS;#HCL; #Accessibility;#A11ySev2;#HCL-MakeCodeArcade-Sep21;#WCAG2.1.1;#A11yeDAD

We have an exemption for Blocks.  Closing.


Testing against MongoDB

If we are going to test queries, this should be done against a MongoDB instance (an integration test, not a mocked out interface).
We need a helper to handle instantiating the mongo instance and tearing it down (also in loading test data).
There's a basic implementation in the specs for Runtime.Events.MongoDB that uses Mongo2Go.  This is too manual and (while working locally on all configs) hangs on the CI server.
We need a base MSpec spec that handles the mongo stuff.  This might be on a per spec basis of a per assembly basis (using IAssemblyContext).
┆Issue is synchronized with this Asana task

An example of setting up mongodb to use the in-memory version (ephemeralForTest storage engine)
http://thomasb.fr/tag/mongodb/


No image thumbnails

When I have a StreamField with an ImageChooserBlock, I get thumbnail previews when I'm adding new images. But when I later edit the page, there are no thumbnails for any image. This is particularly inconvenient for image carousels (ListBlock containing an ImageChooserBlock) because I can't figure out which image I need to change or delete. Thanks for any advice!

Could you confirm that you’re using Wagtail >= 2.2, wagtail-react-streamfield 0.8.6 with wagtail-react-streamfield before wagtail.images in INSTALLED_APPS?

wagtail 2.2.2
Successfully installed wagtail-react-streamfield-0.8.6
INSTALLED_APPS = [

    'search',
    'department', # Delete this after transferring data
    'department2',
    'wagtail_react_streamfield',

    'wagtail.contrib.forms',
    'wagtail.contrib.redirects',
    'wagtail.contrib.styleguide',
    'wagtail.contrib.postgres_search',
    'wagtail.embeds',
    'wagtail.sites',
    'wagtail.users',
    'wagtail.snippets',
    'wagtail.documents',
    'wagtail.images',
    'wagtail.search',
    'wagtail.admin',
    'wagtail.core',

    'modelcluster',
    'taggit',
    'osm_field',
    'wagtailosm',
    'wagtailaccessibility',

    'django.contrib.admin',
    'django.contrib.auth',
    'django.contrib.sessions',
    'django.contrib.messages',
    'django.contrib.staticfiles',
    'django.contrib.contenttypes',
]

I see you have a visual glitch next to the delete icon. Did you make sure the browser cache is cleared, using ctrl+F5 on non-Apple OSes?

Did you make sure the browser cache is cleared

That did it. Thanks! The little square next to the delete icon is still there, though. I'm using Safari 12.0 on macOS 10.13.6.
On a side note, when do you anticipate this will be available for inlines?

@grasshoppermouse Great!
About the little square, could you share the HTML source of that square please?
I’m planning to release the equivalent for inlines within two weeks.


Extract MappingRule from Configuration

Minor refactoring.
This PR extracts MappingRule from Configuration. There were many local methods in Configuration that were related only to mapping rules, so I think this makes sense.
Just to be clear, there are no changes in the implementation nor the test coverage.
I think that some of the code from Service should also be in this new module, but let's go step by step.
Let me know what you think @mikz . If you agree I'll add some documentation for the public methods.

@davidor definitely agree! This is a good path to split the configuration.lua. The .new(...) looks like has too many params, so maybe I'd not expose it just yet. But looks good anyway!


Adding new .state save files?

Issue summary
I used FCEUX emulator to get save files from a level I want the agent to start, but the states files are saved in format ".fc", not ".state".
I tried changing their types to .state and put them into place where roms are placed(\data\stable\KirbysAdventure-Nes). And it's not working.
Every emulator uses different save formats and they are not compatible to each other.
"Getting started" guide just tells me the state files are specific to each emulators, ending with ".state", but gzipped. I don't get this. Emulators use different formats I think it's impossible to make them save with ".state". This is nonsense to me.
How to I get save files, with format ".state"??
System information

Windows10
Python 3.7.7
Gym-retro 0.7.1

Figured out bizhawk emulator can do it.


[os_net_setup] Add --availability-zone-hint to network creation

This PR adds the option to configure external networks for certain AZs. In order to do this, the list availability_zone_hints should be configured as port of cifmw_os_net_setup_config.

/approve

(non-blocking) suggestion: I see some vars used in the template that are missing of some context and format.
Unsure what's the format of availability_zone_hints but I might have a lot of missing context.
WDYT about adding that to the default file: https://github.com/openstack-k8s-operators/ci-framework/blob/main/roles/os_net_setup/defaults/main.yml to see an example?
I'm thinking if it might make sense to rework eventually in the README file to briefly explain each of the possibilities of:  cifmw_os_net_setup_config
Apart of this: lgtm

recheck

recheck


Inconsistent "Send Logs" message

Summary:
The translatable string "log_collection_started" mentions a "Send Logs" button. I couldn't find such a strings elsewhere. The closest string is probably send_log_file, which says "Send log file" in English. The translatable strings must be consistent.
Commons app version:
This is correct for Git revision 2af57fc59421047c58a5bb9df007c170a570821d .
Would you like to work on the issue?
It's a trivial issue, so I can submit a patch as soon as somebody who takes responsibility for the design and the UI strings phrasing tells me what the strings should be :)

"Send log file" is probably more technically correct (the users are sending a file via email), although we do colloquially use "send logs" to talk about it, which probably explains why the mix up happened. Thanks @amire80 . :)


feat: related content

as titled
ref:

https://gohugo.io/content-management/related/
https://discourse.gohugo.io/t/using-regularpages-related-gives-me-keywords-index-support-error/35657/7

@HEIGE-PCloud I just realized that it should be configurable, tried to add that now :)

Hi @wey-gu, thanks for the pr!
I have added some styles to the related content section, check it out here and tell me what you think :)
https://do-it-git-fork-wey-gu-relatedcontent-pcloud.vercel.app/related-tests/

Hi @wey-gu, thanks for the pr!
I have added some styles to the related content section, check it out here and let me know what you think :)
https://do-it-git-fork-wey-gu-relatedcontent-pcloud.vercel.app/related-tests/

holy sh*t this is amazingly elegant😅, thanks!!!


Joins

Support the ability to join records from other collections providing an output where the selected/join result is the document value.

Could you elaborate more about the feature? It would be better if you provide one example to illustrate it.

Sure, for instance in mongo with spring you can use dbref for subdocuments. In spring if you annotate a class with @dbref it will automatically join that document as a property of the annotated document. So for example:
public class Person{ private String id; private String name; private String addressId; }
public class Address{ private String id; private String street; }
Now obviously this is regarding POJOs however if i do a query to select a document by id, It would be awesome if i could join the address document to the returning document where the address document id is equal to the addressId field.
Another example could be that if i have 3 documents that represent portions of an object but in a very normalized way.
public class Point{  public String id; }
public class AbsolutePoint extends Point{ public String id; public Double distance; }
public class GeographicalPoint extends AbsolutePoint{ public String id; public Double latitude; public Double longitude;
It would be awesome if i could create a aggregated document where i can query the three stores/tables by id and create a single result that includes the fields/attributes from the three source documents where the id matches.

By design I decided not to include aggregation in an embedded database. The detailed reason I described here in issue #24 . If you are asking for something else and I did not understand it, please let me know.

Anindya,
Thanks, I can understand the reasoning, however my thoughts regarding my
description of aggregation is not really pertaining to analytics or
calculations, but more so for merging multiple documents into a single
master document (aggregating fields). If you are familiar with JPA or how
other ORMs store records in a relational db, the fields that are also
entities represent a table of its own with a joint table linking the two.
When you query based on the parent entity the ORM will pull the objects
from the tables necessary to make up the parent object. I see this as the
foundation of the request, however it would be nice if i could do it based
on arbitrary documents as sometimes i need to work with relatively dynamic
data.
On Mon, Nov 13, 2017 at 8:54 AM, Anindya Chatterjee <
<EMAIL_ADDRESS>wrote:

By design I decided not to include aggregation in an embedded database.
The detailed reason I described here in issue #24
https://github.com/dizitart/nitrite-database/issues/24 . If you are
asking for something else and I did not understand it, please let me know.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dizitart/nitrite-database/issues/33#issuecomment-343839754,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFgDOPBLy4y048j3TfSP_RnccyNoM9zYks5s1_XAgaJpZM4QbB3I
.

Does projection serve your purpose? - http://www.dizitart.org/nitrite-database/doc/2.0.0/index.html#projection

Projection seems to be based on a single document where you want to only
have certain properties associated with the returned document. What i am
thinking would be projection where a field could be a document returned
from another collection based on a foreign key. For example.
https://drive.google.com/file/d/1XAoVo-B8r0uUTdmBIR6XdHkaEWD7I2Pt/view?usp=sharing
On Mon, Nov 13, 2017 at 9:36 AM, Anindya Chatterjee <
<EMAIL_ADDRESS>wrote:

Does projection serve your purpose? - http://www.dizitart.org/
nitrite-database/doc/2.0.0/index.html#projection
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dizitart/nitrite-database/issues/33#issuecomment-343848067,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFgDOPSZQx_HOgokillVAFM80hpF-_N7ks5s1_91gaJpZM4QbB3I
.

It does look like you are thinking something similar to lookup operation in mongo. So if I understand correctly, if you have 2 repositories like Address, Person
public class Address {
      private String id;
      private String personId;
      private String streetAddress;
}

public class Person {
    private String id;
    private String name;
}

public class PersonDetails {
    private String id;
    private String name;
    private List<Address> addressList;
}

you want to get a result of type PersonDetails where person.id = address.personId

yes, pretty much. Lookup is definitely a better explanation. Ultimately it
is just joining the address and person details into a result of person
details.
On Mon, Nov 13, 2017 at 9:55 AM, Anindya Chatterjee <
<EMAIL_ADDRESS>wrote:

It does look like you are thinking something similar to lookup
operation in mongo. So if I understand correctly, if you have 2
repositories like Address, Person
public class Address {
private String id;
private String personId;
private String streetAddress;
}
public class Person {
private String id;
private String name;
}
public class PersonDetails {
private String id;
private String name;
private List addressList;
}
you want to get a result of type PersonDetails where person.id =
address.personId
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dizitart/nitrite-database/issues/33#issuecomment-343852591,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFgDOOygkPDJBCOPi4lN2I43Qmj8KPUYks5s2APjgaJpZM4QbB3I
.

Just out of curiosity, what is stopping you to create a repository of PersonDetails in the first place and get the Person and Address objects out of it using projection? Unlike sql databases, here you can persists embedded object directly to the database.

Good question, So i'm working on a project where the source data model is
really normalized. And the data is transmitted via database replication.
Each object has its own table and FKs represent the attribute children
objects. There are also levels of inheritance where Object A extends Object
B but instead of just receiving Object B that contains the fields of Object
A, i get the entire inheritance hierarchy. Object B does not contain the
fields of Object A.  So my thoughts were that i could use Nitrite initially
as a staging db for these records, then using the querying, joining and
projection to create the denormalized representation and create a new
denormalized domain model. I cannot control the source and by having a
staging environment such as describe would save me from having to
explicitly mapping each table to a new domain model... or at least reduce
the amount necessary.
On Mon, Nov 13, 2017 at 10:19 AM, Anindya Chatterjee <
<EMAIL_ADDRESS>wrote:

Just out of curiosity, what is stopping you to create a repository of
PersonDetails in the first place and get the Person and Address objects
out of it using projection? Unlike sql databases, here you can persists
embedded object directly to the database.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dizitart/nitrite-database/issues/33#issuecomment-343858697,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFgDOBNsu5Glk0a3psy3pxSO7Gr_cCWBks5s2AmmgaJpZM4QbB3I
.

I think that a lookup would be great and very helpful in my case. I think
that others could use it as well, so it would not be specific to my use
case. I'm attempting to build the source now, will see what else i can come
up with.
Ultimately, the same thing could be achieved manually, however if it could
be automated via the code that would be awesome.
On Mon, Nov 13, 2017 at 10:47 AM, Anindya Chatterjee <
<EMAIL_ADDRESS>wrote:

Interesting use case. If the idea of look up helps you, I'll try to add
the feature
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dizitart/nitrite-database/issues/33#issuecomment-343865870,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFgDOMjfEnUgGdTlIf2qRcSTaxgSd6A5ks5s2BAhgaJpZM4QbB3I
.

I have come up with a basic design as follows:
Lookup lookup = new Lookup();
lookup.localField = "id";
lookup.foreignField = "personId";
lookup.targetField = "addressList";

Cursor cursor = collection1.find().join(collection2.find(), lookup); // for documents
Cursor<PersonDetails> cursor = repo1.find().join(repo2.find(), lookup); // for objects

Let me know your thoughts

Looks good to me if the output is as expected.... Perhaps using a builder
pattern may make it flow better
Join join = new
Join.Builder().where("id").eq("otherId").from(collection1.find()).applyTo("addressList");
Cursor cursor = collection1.find().join(join)
Ideally it should work for both collections and single objects/documents.
Thanks for your effort!
On Mon, Nov 13, 2017 at 1:05 PM, Anindya Chatterjee <
<EMAIL_ADDRESS>wrote:

I have come up with a basic design as follows:
Lookup lookup = new Lookup();
lookup.localField = "id";
lookup.foreignField = "personId";
lookup.targetField = "addressList";
Cursor cursor = collection1.find().join(collection2.find(), lookup); // for documentsCursor cursor = repo1.find().join(repo2.find(), lookup); // for objects
Let me know your thoughts
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dizitart/nitrite-database/issues/33#issuecomment-343899630,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFgDOM3LqRrVM1GQvTsrrB6KBnkDgQ-vks5s2DCagaJpZM4QbB3I
.

I am not inclined to introducing builder patterns right now, as other constructs like query, projection does not have any corresponding builders yet. May be later when I introduce builders for other constructs, I'll definitely add one for it. But for the first step I want to keep it as simple as possible.

Understand completely. Thanks for your efforts!
On Mon, Nov 13, 2017 at 1:33 PM, Anindya Chatterjee <
<EMAIL_ADDRESS>wrote:

I am not inclined to introducing builder patterns right now, as other
constructs like query, projection does not have any corresponding builders
yet. May be later when I introduce builders for other constructs, I'll
definitely add one for it. But for the first step I want to keep it as
simple as possible.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dizitart/nitrite-database/issues/33#issuecomment-343905620,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFgDONYgvPJfJHCHxtXvrbp4Kaz_Feshks5s2DcngaJpZM4QbB3I
.

I have made a commit on master. Please build it locally and test. Let me know if it serves your purpose. Once confirmed, I'll do a release, then you will be able to refer it from maven central.

Anindya,
Thanks, i will do that as soon as possible.
On Tue, Nov 14, 2017 at 7:50 AM, Anindya Chatterjee <
<EMAIL_ADDRESS>wrote:

I have made a commit on master. Please build it locally and test. Let me
know if it serves your purpose. Once confirmed, I'll do a release, then you
will be able to refer it from maven central.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dizitart/nitrite-database/issues/33#issuecomment-344162580,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFgDOMO9PTY0ScYqyEFLpGdy5z6pApG2ks5s2TgcgaJpZM4QbB3I
.

Let me know the outcome so I can close this issue.

I’ve tested the issue. It works as expected. Thanks!
Sent from my iPhone

On 16. Nov 2017, at 06:45, Anindya Chatterjee<EMAIL_ADDRESS>wrote:
Let me know the outcome so I can close this issue.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Thanks for confirmation. Closing this issue.


Craft 3 support

Hey @matthiasott,
I'm working on integrating my Craft 3 site into the micro.blog world and the last piece of the puzzle is to get replies on my posts back to Craft via Webmentions. Is there any chance this plugin will support Craft 3?

Hey Rhys, although I answered on Twitter in the meantime, I'll leave this ticket open as a reminder. ;)

Damn, does this not support Craft 3? I was just about to start using this and Craft. 😭

Hi Matthias,
I’m jumping into this conversation as well. On your website, I read about a plan to update this plugin for Craft 4. I was wondering if you're anywhere close to finishing that effort since I'm considering creating a plugin myself. Let me know if it's still a priority for you, as I'd like to avoid overcrowding Craft's plugin ecosystem with too many plugins that do the same thing.


command error

The first command should be modified into the following form:
./MimicNet/run_0_setup.sh CPU|GPU

Thanks. Fixed.


Update WCSS to use new size tokens

Is your feature request related to a problem? Please describe.
Auro Design Tokens are being updated https://github.com/AlaskaAirlines/AuroDesignTokens/issues/71
Describe the solution you'd like
Post the update and release of work from the linked issue above, while the existing tokens are not removed from the lib, they have been marked as deprecated. These tokens need to be replaced in WCSS for the new ones released with this update to Auro Design Tokens.
Exit criteria
This work will be considered complete once all the deprecated size tokens have been replaced with the new scale.

Since this PR was created, a new version of AuroDesignTokens was released. This needs to be updated once more.

Also update the Auro Doc site to use latest WCSS.


Installed ManagedClusterAddon not removed after unregistering the ManagedCluster

After deleting a managed cluster from the hub, the cluster namespace still remains and the ManagedClusterAddons under that cluster namespace are also not stripped from the hub cluster.
I'm not sure if this is the expected behavior, but clarifying the deletion order of these per-cluster resources in the hub data-plane will be helpful.

/assign @zhiweiyin318
I think we should create a proposal to define the deletion behavior.

@zhiweiyin318 is this fixed already?

@qiujian16 no, the deletion proposal still needs to be revised and there are some details which are unclear.

@zhiweiyin318 Is this fixed already? I am meeting the same problem now. ManagedClusterAddons created by ClusterManagementAddons such as cluster-proxy, cluster-gateway,service-account still exist event if managedCluster has been deleted. And I try to delete these ManagedClusterAddon by hand, it success, but their manifestworks still remain and cannot be deleted, until deleting the manifestworks' finalizer.

@USER0308 , not yet. we have an enhancement https://github.com/open-cluster-management-io/ocm/issues/140 to follow up the enhancement of mcl deletion process including how to cleanup the addon and manifestwork.
currently ,suggest deleting addon before deleting managedcluster.
it will be great if you can add your use cases to that enhancement issue. thanks .

Use case:
Three managedClusters(mcl-1,mcl-2,mcl-3) join into one hub cluster and cluster-proxy, cluster-gateway, service-account these three clusterManagementAddons are installed in hub cluster.
Now I want to detach mcl-1 from hub cluster.
If I firstly delete managedClusterAddon in mcl-1 namespace, it will be recreated by ClusterManagementAddon and it is not work.
If I firstly delete the ClusterManagementAddon, the managedClusterAddon in all of three managedClusters might be deleted, which against my will.
So, after several try, I find that it is work when firstly deleting the managedCluster mcl-1, then deleting the managedClusterAddons in mcl-1 namespace, which will not be recreated since the managedCluster mcl-1 is already not exist.
But this method still has a shortcoming that manifestworks in mcl-1 namespace are still remain and cannot be deleted until removing their's finalizer.

https://github.com/open-cluster-management-io/ocm/issues/140 has completed. Now we support to delete addon and manifests after cluster is deleted.

/close


fixes #1617 Use bitmap font texture from spritesheet

Great !
Does it manage the resolution of the base texture ? I had extra code for that but I may be missing something you did.
Btw about the following line :
texture: core.utils.TextureCache[charCode] = new core.Texture(texture.baseTexture, textureRect)
It exists in PIXI too but do you see the point of "core.utils.TextureCache[charCode]" ? Because it can't manage multiple fonts and I don't see where it can be useful.

I think resolution should work fine, but I'm not very experienced with that.

Because it can't manage multiple fonts and I don't see where it can be useful.

I think that was a tag along from v2, we should just remove that assignment.

:+1:


TP-85249: Github actions WebUI test execution fix

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Implement auto update via Websockets

Closes #13.

Before this can be closed, the app should only start these requests if there is a live tournament. The ping system would use too much bandwidth otherwise.

We can probably disregard the above comment given the new ping system. #whitewine


Memory and FLOPs concern?

Hi! very interesting work!
How is Params calculated? Do you use profile ?
I have noticed that you use a script to calculate memory and flops. Can you share the script? Many thanks goes to the author~

Hi, thanks for your interest in our work. You can use the script provided in our HorNet repo to compute FLOPs, which is also compatible with GFNet models. For memory usage, we use torch.cuda.max_memory_allocated to measure the max GPU memory usage during training.

Very thanks for the nice response!
I have another question about the parameters~
Do you use "flops, params = profile(model, inputs=(input, ))" to calculate the params (number of parameters)?
Maybe the parameter K (learnable weight) is not taken into consideration?


Don't allow guardians to choose a password but auto-generate it

Humans are really bad at choosing high-entropy passwords, we should instead generate a password, require the guardian to write it down/save it and the request it again at the next step. 16 alphanumeric, non-case sensitive characters should be more than enough (83bit of security).

Let's close in favor of https://github.com/fedimint/ui/issues/393


Add config support

The configuration always makes it easier to use the tool, especially if you do it frequently and don't want to write a colossal command every time. I want to implement the functionality that would add the --config flag and use the config.json to configure the tool

Fixes #11


NetDuke32 for Duke Nukem 3D

Project Link
https://voidpoint.io/StrikerTheHedgefox/eduke32-csrefactor/-/releases/
Project License
GPLv2
Steam App Ids

https://steamdb.info/app/434050/
https://steamdb.info/app/225140/
https://steamdb.info/app/359850/

Additional Context
https://forums.duke4.net/topic/11826-netduke32-enhanced-duke3d-netplay/
Also, I noticed that original DN3D release on Steam isn't supported by Luxtorpeda. Would it be possible to provide EDuke32, BuildGDX, Raze and NetDuke32 for it? Here is file structure;
https://steamdb.info/app/359850/depots/

@rea987
The original release isn't on the store anymore, so I would have no way to test it. Do you have that one?
I'll look into this engine though, thanks

The original release isn't on the store anymore, so I would have no way to test it. Do you have that one?

Unfortunately I don't have that one. By the looks of depots, it utilizes DOSBox on Linux, Windows and macOS.

@rea987
I added this for megaton edition here: https://github.com/luxtorpeda-dev/packages/pull/632
eduke32 does not seem to work with World Tour, at least the version that this project is using.

@d10sfan It looks like NetDuke32 has been added for Megaton Edition but not for World Tour. As Megaton is no longer on sale, would it be possible to add NetDuke32 fpr World Tour? Thank you!

@rea987 As I mentioned above, it does not seem like this supports world tour. If you find out if there's any fixes for that, please let me know.
When I tried to run it, it came up with errors that it did not understand the con files.

Yeah, it looks like those who wish to play that port and happened to not own Megaton should go for Atomic Edition in Zoom Platform. Thanks for the explanation.
https://www.zoom-platform.com/product/duke-nukem-3d-atomic-edition


flutter beta (1.19.0-4.3.pre -> 1.20.0-7.2.pre), dev (1.20.0-3.0.pre -> 1.21.0-1.0.pre) update

Motivation for this change
Flutter update
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Since this PR only updates beta and dev versions, it is probably not urgent. Especially since the stable version was updated recently.

@ofborg build flutterPackages.beta flutterPackages.dev

I think a more clear message to use for your commit would be:
flutterPackages.beta: 1.19.0-4.3.pre -> 1.20.0-7.2.pre, flutterPackages.dev: 1.20.0-3.0.pre -> 1.21.0-1.0.pre

What is important is knowing which attribute has changed.

@worldofpeace I see. Should I change it and force-push again?

@worldofpeace I see. Should I change it and force-push again?

Yep.

@worldofpeace Done

Thanks @kolaente, merged :sparkles: