When create a  with a hash only parameter I got 'undefined' on  output

I open this issue on react-router https://github.com/reactjs/react-router/issues/3187 but seens it belog to here
I was trying anchor based navigation with some  like this:
<Link to={{hash: "#home"}} className="nav-link" onClick={e => scroll(e)}>Home</Link>
<Link to={{hash: "#sample-code"}} className="nav-link" onClick={e => scroll(e)}>Sample</Link>
<Link to={{hash: "#features"}} className="nav-link" onClick={e => scroll(e)}>Features</Link>

but the HTML output is something like this:
<a class="nav-link" href="undefined#home" data-reactid=".<IP_ADDRESS>.<IP_ADDRESS>.0">Home
<a class="nav-link" href="undefined#sample-code" data-reactid=".<IP_ADDRESS>.<IP_ADDRESS>.0">Sample

It would be nice not having to add: pathname:"/" or pathname:""  to <Link to={{
The issue maybe is on createHref function.

This has absolutely nothing to do with this library.

@taion createHref it is not a history function ?

@taion @mjackson
The problem is on this function: https://github.com/mjackson/history/blob/v2/modules/createHistory.js#L162
Can you fix this and publish a v2 update before release v3??
thanks.

This is an incorrect usage of the API on your end. Please follow up on an appropriate support forum.


Fix list-files for disk

Fix the list-files implementation for the disk driver so it lists all files (like find), not just only the top-level directories.  This makes it consistent with the rest of the drivers on list-files.  Update the tests to test this new behavior.

Good catch @jcnelson -- merging to develop now.


Add AWS Glue Service to browser ServiceMap

https://us-west-2.console.aws.amazon.com/glue/ for example

test failed


chore: update<EMAIL_ADDRESS>

Types are not compatible when importing with latest 0.47.0

@kianomg nice! Thanks for the quick merge, do you mind publishing?

Just published, 1.0.3 should be there now.

@kianomg working, thank you!


AKS support for SLB+VMSS

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR has modified HISTORY.rst describing any customer-facing, functional changes. Note that this does not include changes only to help content. (see Modifying change log).

[x] I adhere to the Command Guidelines.

Looks good in general, but a few comments:

could we also support public ip name if the same resource group for instance? I think passing is too complex
I think it would be good to improve documentation and define what's the relationship between new parameters

I think it's ok to merge as is, some improvements could be made in the future

Thanks a lot for the comments, @zikalino . Will take your suggestion into the consideration in the next release(should not be too far).

@qwordy , can you take a look ?


bashdb: 4.4-0.94 -> 4.4-1.0.0

Motivation for this change
Things done

[X] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
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 nox --run "nox-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)
[X] Fits CONTRIBUTING.md.

.. accidentally got pull req. as parent; rinse and repeat.


Leverage WebWorker for clustering

Clustering a large amount of probes can take ~1 seconds by itself. All of this time is currently spent on the main thread, which yields a poor user experience.
We could move this computationally intensive logic inside a web worker to free up the main thread.
This issue has already been opened in the past, but it was previously closed: https://github.com/Leaflet/Leaflet.markercluster/issues/131

Hi @Etienne-M this indeed sounds like a significant amount of refactoring as previously mentioned. Perhaps we can use this issue as a poll for such improvement. If we got a sufficient amount of attention we can def prioritize it. In the meantime, feel free to open a PR if you had any solution as well we can review it.


Why did you automatically delete old versions of packages?

Brief description of your issue
I cannot use winget to install older versions of Mailbird.Mailbird. Why did you make a package manager if you don't allow the users to install any avaiable versions?
https://github.com/microsoft/winget-pkgs/commit/479ff745fedb4bf5d47e02a71cb734105679fae4
Steps to reproduce
The current version of Mailbird.Mailbird is 2.9.68 as of today.

winget install --id Mailbird.Mailbird --version 2.9.67 (2.9.67 is the previous version).
See errors.

Expected behavior
Install 2.9.67 version without any error.
Actual behavior
No version found matching: 2.9.67

Environment
Windows Package Manager v1.3.2691
Copyright (c) Microsoft Corporation. All rights reserved.

Windows: Windows.Desktop v10.0.22621.674
System Architecture: X64
Package: Microsoft.DesktopAppInstaller v1.18.2691.0

Logs: %LOCALAPPDATA%\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\DiagOutputDir

Links
---------------------------------------------------------------------------
Privacy Statement   https://aka.ms/winget-privacy
License Agreement   https://aka.ms/winget-license
Third Party Notices https://aka.ms/winget-3rdPartyNotice
Homepage            https://aka.ms/winget
Windows Store Terms https://www.microsoft.com/en-us/storedocs/terms-of-sale

One possible cause is that the installers are no longer being offered by the application's site or place of distribution

This is something you'll have to talk to the publisher.
Mailbird.Mailbird currently only has a Download URL that always points to the latest version which prevents historical versions from being in the repository.

Dear @ItzLevvie, that's good to know. I am going to point this out to the developers.


FPORT: Fix for sbt/sbt#2435

Forward port of Fix for sbt/sbt#2435 (sbt/sbt#2513)

@Duhemm I think this is failing because it needs https://github.com/sbt/sbt/pull/2574

Closing since these were added by https://github.com/sbt/sbt/pull/2911


[CT-1064] [Bug] Dbt incremental: unique_key with null values generates duplicates row

Is this a new bug in dbt-core?

[X] I believe this is a new bug in dbt-core
[X] I have searched the existing issues, and I could not find an existing issue for this bug

Current Behavior
I'm using dbt(v1.1.2) with Redshift. This is scheduled from Airflow. I have an incremental model with two unique_key (type, name). When I run the model for day 1 it presents null values for type but for name it does not.  The generated table looks like this:
type | name | count_values
NULL onion 5
fruit apple 10
When I run the model for day 2, it presents again null values for type but not for name, this generates again another row in the final table and does not append to the existing one.
type | name | count_values
NULL onion 5
fruit apple 10
NULL onion 7
Expected Behavior
What I expect to happen is:
type | name | count_values
NULL onion 12
fruit apple 10
Steps To Reproduce
SQL Incremental Model:
select
type,
name,
count(*) as count_values
from another_table
where date >= '{ds}' and
group by type, name
Relevant log output
No response
Environment
- Python:3.10
- dbt: 1.1.2
- Airflow: 2.3.2

Which database adapter are you using with dbt?
redshift
Additional Context
No response

@AndresSalvatierra Thanks for opening!
In cases where multiple columns are needed to uniquely identify a record (= unique_key is a list), none of those columns can be null. The equality check for upserts / merges doesn't have handling for null values built in—nor do I think it should, since this obfuscates more logic in an already difficult-to-debug place (materialization logic, rather than transformation logic).
If there's any risk of one or more columns being null, you're better off creating a single surrogate (hash) key. The dbt_utils.surrogate_key macro handles null values, and will return the same hashed result for the same null input(s).
It looks like we don't do a good job of documenting this today: https://docs.getdbt.com/docs/building-a-dbt-project/building-models/configuring-incremental-models#defining-a-uniqueness-constraint-optional
I'm going to close this issue as a wontfix, and open a new issue in https://github.com/dbt-labs/docs.getdbt.com to update that documentation.

Hi Jeremy!
Thank you for your quick response!
I solved my problem using dbt_utils.surrogate_key, thank you very much!


Remove tag

Feature request: Remove tag from history table
Submitted by: Phyer#3136
Server: SHMEAT

Removed tag in add_field output.


Override AbstractProvider->httpBuildQuery() in Square provider

Square appears to perform a superfluous redirect during the OAuth
workflow that does two things: 1) sorts query string parameters; 2)
removes an approval_prompt parameter automatically added by
AbstractProvider->getAuthorizationUrl().
At best, this adds a needless request to the OAuth workflow, making it
take longer. It appears that it may also be causing the workflow to fail
in Square->prepareAccessTokenResult() within a local development environment.

@ameech You have access to kick Scrutinizer and/or just merge this?

@elazar I kicked it, still have access somehow. I think @ameech and @dolfelt have access to the scrutinizer org.

Well, I tried to kick it. Didn't seem to work. Looks like I added them without linking to an org, so I just removed all of them and someone else can add them back in.


Add .travis.yml to run ESLint

Add .travis.yml in order to run ESLint on PRs.
I'm not that familiar with Travis, so LMK if I'm doing anything wrong.
to: @lelandrichardson @christopherdro

It seems to work!

@spikebrehm  (:

woot 👍


Bug: Able to create records with invalid id

Example:
const db = new Surreal(...);

... use ns, signin

const myObject = {
    id: '',
    name: 'test',
};

const res = await db.create('channel', myObject);

This successfully creates unaccessable (and somewhat undeletable) record with id channel:<>. Only way i figured how to delete this record was to delete the whole channel table, which isn't very optimal solution.
How it looks in Surrealist: Screenshot

Hiya, nicely spotted! I will move this issue to the main SurrealDB repository as it's rather an issue of SurrealDB accepting such a message in the first place.
For easy reproduction, here's a raw message you can send over a websocket to reproduce:
{
    "id": 123,
    "method": "create",
    "params": ["example", { "id": "" }],
}


Endless scanning sometimes.

Using following version
"react-native": "0.62.2",
"react-native-ble-plx": "^1.1.1",
I want to know reason of below scenario:
When Mobile app is in connected status with ble device,
and I turn off the device app goes to disconnected mode and re-scan and then connect. (I repeat this)
It happens regular 2-3 time successfully, after this sometime app goes to endless scanning (Scanning...) mode so in this case I have to kill and re-launch the app.
And sometime  (after 2-3 time) it continuously says BleError : can not start scanning operation
so in this case I have to kill and re-launch the app or need to again turn-off or turn on the device.
My code below
this.manager.startDeviceScan(null,{ScanMode : "Balanced" }, (error, device) => {     console.log("Scanning...");     if (null) {         console.log('null');         return;     }     if (error) {         console.log("Error in scan=> "+error)         this.scanAndConnect()         return     }     if(device.name == 'mydevice'){          this.manager.connectToDevice(device.id, {autoConnect:true}).then((device) => {             (async () => {                             //listener for disconnection                 device.onDisconnected((error, disconnectedDevice) => {                     console.log('Disconnected ', disconnectedDevice.name);                     this.scanAndConnect(); // to start scanning again                 });             })();             });     } })
may I know the reason or any mistake if I made here in code ?

You do not stop the scan it seems?

I do just after once mydevice got found. Sorry for above code mistake.
this.manager.startDeviceScan(null,{ScanMode : "Balanced" }, (error, device) => { console.log("Scanning..."); if (null) { console.log('null'); return; } if (error) { console.log("Error in scan=> "+error) this.scanAndConnect() return } if(device.name == 'mydevice'){ this.manager.stopDeviceScan();this.manager.connectToDevice(device.id, {autoConnect:true}).then((device) => { (async () => { //listener for disconnection device.onDisconnected((error, disconnectedDevice) => { console.log('Disconnected ', disconnectedDevice.name); this.scanAndConnect(); // to start scanning again }); })(); }); } })

here is the error {console.log(JSON.stringify(error));}
[Sat Jun 06 2020 21:26:23.869]  LOG      {"message":"Cannot start scanning operation","errorCode":600,"attErrorCode":null,"iosErrorCode":null,"androidErrorCode":null,"reason":"Undocumented scan throttle (code<PHONE_NUMBER>), suggested retry date is Sat Jun 06 21:26:21 GMT+05:30 2020","name":"BleError","line":26121,"column":28,"sourceURL":"http://localhost:8081/index.bundle?platform=android&dev=true&minify=false"}

You scan too often. Android allows only 5 scans in every 30 seconds window.

same is for ios ?

No

hmm, I checked with stop watch, continuous =>scan,connect,disconnect (Repeat) work for 2 time only, in 3rd round it goes endless scanning-> {"message":"Cannot start scanning operation"} and this error.
Check with stopwatch 3rd round was under 30seconds window.

Full working file
`import React,{Component} from 'react';
import {BleManager} from "react-native-ble-plx";
import {PermissionsAndroid, Platform} from 'react-native';
export default class Ble_test extends Component {
constructor() {
super();
if (Platform.OS === 'android') {
PermissionsAndroid.request(PermissionsAndroid.PERMISSIONS.ACCESS_COARSE_LOCATION)
.then(granted => {
if (granted !== 'granted') {
console.log('BLUETOOTH denied')
}
else {
this.initializeManager();
}
})
} else {
this.initializeManager();
}
}

initializeManager() {
    this.manager = new BleManager({
        restoreStateIdentifier: 'testBleBackgroundMode',
        restoreStateFunction: bleRestoredState => {
        }
    });
    const subscription = this.manager.onStateChange((state) => {
        if (state === 'PoweredOn') {
            this.scanAndConnect();
        }
    }, true);
}

scanAndConnect() {
    this.manager.startDeviceScan(null, null, (error, device) => {
        console.log("scanning");
        console.log(device);

        if (error) {
            console.log(JSON.stringify(error));
            this.scanAndConnect()
            return
        }

        if(device.name == 'TAPP_noljal'){
            console.log("found ", device.name);
            this.manager.stopDeviceScan();
            this.manager.connectToDevice(device.id, {autoConnect:true}).then((device) => {
                    (async () => {
                        console.log("connected to ",device.name);
                        //listener for disconnection
                        device.onDisconnected((error, disconnectedDevice) => {
                            console.log('Disconnected ', disconnectedDevice.name);
                            this.scanAndConnect();
                        });

                    })();
                })
     }
    });
}

render() {
    return null;
}

}`

Correct me if I am wrong but returning an error is not an endless scan? Is it? It is an error while starting a scan?

Yes,
So  I am calling this.scanAndConnect() again if error comes.

Mhm... And what do you recon then?

Yes, I want reconnect then with same device.
Because user who is using BLE device can regular switch off and switch on the device. This can be a scenario.

There is nothing you can do. There may be only that many scans during a specific time frame. You have to deal with it

Ahh, Okay,
Well Thanks for responding so fast. I am closing it now, If in case further scanning query I will reopen it.
I don't know if i am referring wrong here for replying on https://github.com/Polidea/react-native-ble-plx/issues/669 please

Any solution found? I m facing the same issue. After connecting and disconnecting the Peripheral two times, it doesn't start the scan again.

FWIW the quick start instructions pretty much guarantee a new developer will encounter this throttle error. It would be nice if the instructions guided 'best practice' instead of 'rough and ready'.


Fix bugs in Index Statistics for Multi-Column case

What changes were proposed in this pull request?
In Multi-Column case, an interval has more than one columns and the last one
maybe DUMMY_KEY. For example interval.start = (1, 100, "a", DUMMY_KEY_START)
But, interval.start != DUMMY_KEY_START, and compare it may still encounter
ArrayIndexOutOfBoundException. This PR fixes this problem
Based on PR #257
How was this patch tested?
Unit Test

Can you update this?


Correlation of Logs and Spans via trace context does not seem to work

Bug Report
The logs created via the OpenTelemetry bridge do not seem to get their trace_id and span_id fields set based on the currently-active span.
Version
My Cargo.toml:
[package]
name = "otel-rust-tracing"
version = "0.1.0"
edition = "2021"

[dependencies]
opentelemetry = "0.26.0"
opentelemetry-appender-tracing = "0.26.0"
opentelemetry-stdout = "0.26.0"
opentelemetry_sdk = "0.26.0"
opentelemetry-semantic-conventions = "0.26.0"
tracing = "0.1.40"
tracing-opentelemetry = "0.26.0"
tracing-subscriber = "0.3.18"

Platform
Mac OS X
Description
In the following, the Log should have additionally the TraceId: d260f7f85b4f5e5c6bfc651318c16b66 and SpanId: a47bb75086adb5cf fields set.
Logs
Resource
         ->  telemetry.sdk.language=String(Static("rust"))
         ->  telemetry.sdk.name=String(Static("opentelemetry"))
         ->  telemetry.sdk.version=String(Static("0.26.0"))
         ->  service.name=String(Static("unknown_service"))
Log #0
         EventName: "event src/main.rs:35"
         Target (Scope): "otel_rust_tracing"
         Observed Timestamp: 2024-10-15 08:13:53.456706
         SeverityText: "INFO"
         SeverityNumber: Info
         Body: String(Owned("yak shaving completed."))
         Attributes:
                 ->  number_shaved: Int(3)
Spans
Resource
         ->  service.name=String(Static("unknown_service"))
         ->  telemetry.sdk.version=String(Static("0.26.0"))
         ->  telemetry.sdk.language=String(Static("rust"))
         ->  telemetry.sdk.name=String(Static("opentelemetry"))
Span #0
        Instrumentation Scope
                Name         : "dash0"

        Name        : app_init
        TraceId     : d260f7f85b4f5e5c6bfc651318c16b66
        SpanId      : a47bb75086adb5cf
        ParentSpanId:<PHONE_NUMBER>000000
        Kind        : Internal
        Start time: 2024-10-15 08:13:53.456575
        End time: 2024-10-15 08:13:53.456891
        Status: Unset

This is the code that generates the output above:
use opentelemetry::global::{self, ObjectSafeSpan};
use opentelemetry::trace::Tracer;
use opentelemetry_appender_tracing::layer;
use opentelemetry_sdk::logs::LoggerProvider;
use opentelemetry_sdk::trace::TracerProvider;
use opentelemetry_stdout as stdout;
// use opentelemetry_semantic_conventions as semconv;
use tracing::{info};
use tracing_subscriber::layer::SubscriberExt;
use tracing_subscriber::Registry;

fn main() {
    // Create a new OpenTelemetry trace pipeline that prints to stdout
    let tracer_provider = TracerProvider::builder()
        .with_simple_exporter(stdout::SpanExporter::default())
        .build();
    global::set_tracer_provider(tracer_provider);

    let tracing_layer = tracing_opentelemetry::layer();

    let log_provider = LoggerProvider::builder()
        .with_simple_exporter(stdout::LogExporter::default())
        .build();
    let log_layer = layer::OpenTelemetryTracingBridge::new(&log_provider);

    let subscriber = Registry::default().with(tracing_layer).with(log_layer);

    // Trace executed code
    tracing::subscriber::with_default(subscriber, || {
        // Spans will be sent to the configured OpenTelemetry exporter
        let tracer = global::tracer("dash0");
        let mut root = tracer.start("app_init");

        let number_shaved = 3;
        info!(
            number_shaved,
            "yak shaving completed."
        );

        root.end();
    });

    global::shutdown_tracer_provider();
}

Unfortunately, with the current implementation, it is not possible to associate traces with logs when using tracing-opentelemetry. For more details, please refer to this issue
From what I have investigated, when opentelemetry_sdk generates logs, it looks for the trace_id from the active span, but it references opentelemetry's Context during this process. However, tracing manages its own Context (active span), and opentelemetry does not take tracing's context into account.
The specific implementation can be found here:
https://github.com/jpramosi/opentelemetry-rust/blob/11d01e29d64f578bee987f49b8823463cd8a6371/opentelemetry-sdk/src/logs/log_emitter.rs#L258.
One developer has published an example of the necessary implementation. As shown in the example, a logic that allows opentelemetry to reference tracing's span is needed.

Thanks for the help :-)


Application Gateway under AGKI control retains tls certificates from deleted endpoints.

Describe the bug
Application Gateway under AGKI control retains tls certificates from deleted endpoints.
This leads to to broken update state when number of tls certificates goes above 100
To Reproduce
Create 100 kubernetes endpoints, e.g. service1.contoso.com service2.contoso.com ... service100.contoso.com
Delete all endpoints from kubernetes cluster, and try to deploy service101.contoso.com
AG update will fail due to lingering tls certificates from previous deployments.

@gregory-j-baker I ended up cleaning up my gateway and stopping usage of certificates that are uploaded from kubernetes secrets. Instead i did a manual upload of a single domain certificate to the app gateway and reference it from ingress annotation like this:
`
annotations:
  kubernetes.io/ingress.class:  azure/application-gateway
  appgw.ingress.kubernetes.io/appgw-ssl-certificate: "yourcertificate"

`

Is it possible to know if it will be fixed or not?

It is not fixed. We are using ingress with SSL and have hit this limit several times now.


Automate deployment (GitHub Actions)

https://github.com/orgs/community/discussions/26323

Still need unitypackage deployment


beacon: speed up slow test

Motivation
TestBeacon_proposalPassesEligibilityThreshold is a test that takes ~ 10 seconds to complete on my machine. The fastest GH runner (ubuntu) needs ~ 70 seconds for it. If we want to also execute it on slower runners (linux/arm64 and darwin/arm64), we should speed it up so it doesn't time out.
Changes
Reduce amount of proposals that are processed in test from 10k to 1k thereby speeding up the test by a factor of ~ 10.
Test Plan
n/a
TODO

[x] Explain motivation or link existing issue(s)
[x] Test changes and document test plan
[x] Update documentation as needed

DevOps Notes

[x] This PR does not require configuration changes (e.g., environment variables, GitHub secrets, VM resources)
[x] This PR does not affect public APIs
[x] This PR does not rely on a new version of external services (PoET, elasticsearch, etc.)
[x] This PR does not make changes to log messages (which monitoring infrastructure may rely on)

bors merge


github and google analytics graphs

This PR includes graphs for the github metrics

Have added web traffic overtime for the main OCS website as well as each case study and engagement metric graphs for each case study and a table with metrics for the main OCS website and pooled case studies

@carriewright11 the checks pass now and there's a preview!

Would appreciate feedback on output sizes for the tables and plots and if you'd like any adjustments for those

This looks great! I wonder if we could do a map instead of the list of countries?

Would appreciate feedback on output sizes for the tables and plots and if you'd like any adjustments for those

I noticed some of the case study names are getting cut off in the facets, maybe we could adjust to allow more of the name to fit? can maybe remove ocs-bp from names if that helps

I think we could merge this though and start a new PR on maps (I will send resources on slack)

It might be worth hiding the plots for github except the first one for now


Add debugger extension tests

This PR adds tests for #2219. The VS Code extension has not had any tests yet. The tests only cover code that was touched by me.
I have added the tests to the compilation_on_android_ubuntu.yml workflow. The first run will take about two hours, since LLDB is built from scratch. Later, the build should be cached and the whole job should not take more than three minutes.
Core of the PR is an integration test that boots up vs code and lets it debug a test WASM file.
Run Tests locally
Download the LLDB build for your platform from my pipeline.
cd ${WAMR_ROOT}/test-tools/wamr-ide/VSCode-Extension/resource/debug
wget ${LLDB_FOR_YOUR_PLATFORM}.tar.gz -O lldb.tar.gz
tar -xvf lldb.tar.gz
cp -R wamr-lldb/* ./${YOUR_PLATFORM}

Make sure wasm32-wasi target is available.
rustup target add wasm32-wasi

Build iwasm with source debugging feature
cd ${WAMR_ROOT}/product-mini/platforms/${YOUR_PLATFORM}
mkdir build && cd build
cmake .. -DWAMR_BUILD_DEBUG_INTERP=1
make

Note: On MacOS M1 environment, pass the additional -DWAMR_DISABLE_HW_BOUND_CHECK=1 cmake configuration.

Install npm dependencies and run tests.
cd ${WAMR_ROOT}/test-tools/wamr-ide/VSCode-Extension
npm install
npm run test

To run tests locally, the node version has to be 16.x+(GitHub Actions use 18.16.0), is there a way to add it in package.json to ensure the user's node meets the requirement?
What I can think of is to add the engines field in your package.json file like:
    "engines": {
        "vscode": "^1.59.0",
        "node": "^16.0.0"
    },

and add engine-strict option in the new .npmrc file:
engine-strict=true

This will force npm to err for node versions lower than 16.x. I think it's better to err by setting engines rather than warn by setting devDependencies
What do you think? Is there any other way to achieve this?

Hi @TianlongLiang,
Thanks for the suggestions suggestions. All Done!

does it worth adding the same tests for macOS CI job if it's supported? @MrSarius

In general, of course, my integration tests could also find major problems in the LLDB build (crashes etc.). That could be helpful for all platform that are suppported.
But in the context of the extension, that doesn't add much value compared to the extra effort it means (building and caching LLDB for Macos)

Hi, @MrSarius, after the recent update of VS Code, for version above 1.86.0, it seems that the format conversion in formatters/rust.py failed, therefore it returns the raw format and assertion fails during the value check process.
      The Deque summary string looks different than expected
      + expected - actual

      -alloc::collections::vec_deque::VecDeque<int, alloc::alloc::Global> @ 0xfff1c
      + (5) VecDeque[1, 2, 3, 4, 5]

For fixing the CI error, could you please let me know your opinion? Do you think this is mainly a VS Code bug? If so, maybe we can safely ignore it for a while and use the fixed VS Code version. Or is it a major interface change that needs modification on our side?


Stop setting setContent if value is the same

I had the problem, that form is still made dirty (similar to #48), because ngModel.$setPristine() does not make the root form prestine.
Please merge this PR and tag a new version.
Thanks :-)

I made some modifications to the original PR due to recent changes - this should do the trick though.


What's the time-ago filter?

I want to display the time ago (e.g. "a few seconds ago") in a tooltip when an icon is hovered, like so:

How can I do this? I can't find anything like this in the docs.

We do have an amTimeAgo filter. It is not documented (as it should be), but you can still use it.
https://github.com/urish/angular-moment/blob/9680d6f1d21585e40f32e1de8f002e64daf20906/angular-moment.js#L589

I found it shortly after opening this issue, thanks for your time and apologies for not looking more thorough before posting!


Handle multipolygons with empty polygons

Fixes #13856.

This error occurs again when using webGL in openlayers 8.2.0.


add passkey support.

Modern times speak of using modern authentication systems. Do not need full oauth2 etc.. just need to let people use passkey's for their account (have to switch an existing account from using a password to a passkey, though.)

See: https://github.com/mkalioby/django-passkeys


Animation glitch when minimizing editor by Show desktop on Windows 11 (caused by low processor mode)

Godot version
v3.4.beta3.official [8db0bd442] and v4.0.dev.custom_build [0a001afec]
System information
Windows 11, Vulkan Clustered
Issue description

Steps to reproduce
Minimize the editor by clicking on Show desktop on Windows 11.
Minimal reproduction project
No need.

windows 11 is not even officially available as stable ... I don't think it is necessary to correct anything at the moment.

Alright then.

On the contrary, early tech previews are mostly targeted at devs so that they can check if their apps are working properly. This is worth reporting.

I would not pay attention to it, maybe it is something that will change in windows later and then we will have to modify Godot again.

That's not a hard thing to do, and can boil down to reverting a single PR. It doesn't mean issues shouldn't be reported, or that we shouldn't prepare for new releases beforehand. There is no harm in logging it and if someone wants to investigate it, it's okay as well.

@Torguen I kind of doubt it's Windows' problem as I haven't met such an issue with other apps.

Can you reproduce this if you enable Update Continuously in the Editor Settings?
Can you reproduce this if you comment out this line then recompile the editor?

@Calinou The first solution works while the second doesn't.

Is this still reproducible on newest master?

@KoBeWi No.


Emit pipeline variables as outputs and variables

This updates the calls to setVariable to set both as variable and output. This is intended to add backward compatibility for the output changes released with 0.6.24
Resolves #45
Resolves #46
Still testing this in pipeline to ensure azure pipelines allows both to be set so opening as draft for now.

Unfortunately, it appears azure pipelines does not allow you to set a variable and output with the same name. If I set TERRAFORM_PLAN_HAS_CHANGES twice, once with isOutput=false and once with isOutput=true, the same issue still occurs. So likely last one wins.
Will be reverting the changes the output changes so everyone is unblocked while I figure out a different strategy.


feat: add ApiDefinitionAnnotationProcessor

This is a PoC for #2834
:heavy_check_mark: Checklist

[ ] All tests are passing yarn test
[ ] Screenshots attached (for UI changes)
[ ] Relevant documentation updated
[ ] Prettier run on changed files
[ ] Tests added for new functionality
[ ] Regression tests added for bug fixes
[ ] Added a changeset (more info)

Another possibility:
spec:
  implementsApis:
    - location: ./openapi.spec
      name: my-openapi-thing
      ...etc

@freben Thanks for your feedback
Yeah, after the discussions about annotation, the variant with the complex implementsApis entries feels appealing. I'm just wondering how to document this. I would keep the spec (validation) of the component as-is, but allow users to add complex objects there. The processor evaluates them and replaces them with the simpler links to the API entity. So only the simple links are stored. And here is probably were the relation system comes into play. Therefore it's fine for me to postpone this for a week and see how the relations come along.

Exactly, that does beg the question, should one really rewrite the spec at all. It sounds more appealing to eat the format as is and spit out relations and tell people to read those instead of consuming the spec.

I will close this for now, but might revisit this later

I will close this for now, but might revisit this later

For reference we built a somewhat similar processor to add API entities automatically. Except the difference is it does a lookup for a published API spec at a known location. This means it doesn't require any changes in provides by component catalog file. We don't have a consumes equivalent yet though.

For reference we built a somewhat similar processor to add API entities automatically. Except the difference is it does a lookup for a published API spec at a known location. This means it doesn't require any changes in provides by component catalog file. We don't have a consumes equivalent yet though.


[ADDON] - Thanos

Add support for Thanos as an add-on.
Ex: https://github.com/thanos-io/thanos/tree/main/tutorials/kubernetes-helm

@bryantbiggs, I think you can close this issue. Thanos can be installed in this Addon:
https://github.com/aws-ia/terraform-aws-eks-blueprints/tree/main/modules/kubernetes-addons/kube-prometheus-stack


Unnecessary API calls in console

Unnecessary API calls noticed in console
Groups page

Users page

Roles page

No longer reproducible.


Feature: DynamoDB: Support GSI projectiontype KEYS_ONLY

Fixes #3055
GSI's can be configured to only store a subset of the data, to save space/money, by specifying a ProjectionType.
This PR adds support for the projection type KEYS_ONLY.
The ProjectionType 'ALL' is supported by default.
The ProjectionType 'INCLUDE' is not yet supported, but can be easily added later on.

Only just realized this change currently breaks tables with an LSI. Will fix it soon.

Coverage decreased (-0.01%) to 94.381% when pulling 4e0d5883073b3c12c27e888b978dba530708035a on bblommers:dynamodb-gsi-projectiontype into 385c78a996f07dd3d603acb2a2c9bdc799670fa4 on spulec:master.

Thank you!


ScanCode cannot detect jar dependencies as well as syft

ScanCode:

Full output: https://gist.github.com/Hritik14/c476ede36d0b486375ce60f404da3ea1
Syft: https://gist.github.com/Hritik14/8f5300e0dba08d15886f9f64fb2daad0
Jar file scanned: gs-spring-boot-gradle.zip/initial/build/libs/spring-boot-0.0.1-SNAPSHOT.jar

@Hritik14 Somehow this was missed. Thanks for this... we are improving with the addition of the PurlDB!


Can this model be used for human keypoint detection？

Can this model be used for human keypoint detection？

Hi @Xjmengnieer, are you referring to direct testing our model on human, or you will train on a small set of human images first?


Support static linking with /MT /MTd

Maybe it is already possible, but the docs so far appear to imply code gets compiled into DLLs.
Please also support compiling all code into a .lib with /MT /MTd runtime libraries for those of use who prefer their executables to be self-contained (or document how it is done).

DLLs are the hardest, so we've focused on them first, but static linking is definitely a needed feature!
Our current plan (open for discussion) is to have /MT be configured in the triplets files. So, you would add a single line to triplets\<triplet>.cmake like set(VCPKG_VS_CRT_LINKAGE STATIC). This would likely need to be combined with a separate set of triplets (like x86-winstatic).
In Visual Studio, this triplet would automatically get picked whenever /MT is selected.

If we're discussing static linking, it might be worthwhile asking how to handle LTCG (primarily for the purposes of PGO), as not all libraries can be compiled as static libraries. (there are probably a few out there that don't play nice with LTCG, but those should be significantly rarer)
Are there any plans to also support individual libraries as statically-linked but built against the dynamic runtime?

I have created a new branch: staticlibs.
In this branch we introduce an initial approach to supporting static libs.
Static libs are treated as a separate triplet. For example, in addition to x86-windows there is now x86-windows-static.
The triplet definitions in triplets\ specify some variables that can be used by the portfiles to configure the build appropriately. Currently, we have modified zlib only to take them into account and build either dlls or static libs.
To install zlib dlls or static libs you need the following commands, respectively:
vcpkg install zlib:x86-windows
vcpkg install zlib:x86-windows-static

We welcome feedback on the approach we have taken. In particular, one of the points that was raised is the name of the triplets. Some alternatives are:
x86-win (for dlls, instead of x86-windows)
x86-winstatic
x86-win-static

Is this issue resolved via #97 and therefore can be closed or is there still something missing needing to be addressed @aardappel

@adam4813 : thanks, this is very helpful.
One issue that worries me is that now each package needs to explicitly support static linking in its portfile, and if portfile authors are generally omitting this, it will be some pain for those using static linking getting that fixed.
Since most open source libraries have no particular requirements on what their code gets compiled into, a default solution that automagically is able to link statically would have been even better.
But this is already entirely workable, so feel free to close the issue.

To fill in a few more details about the support we've added with #97: these settings are controlled by the variables VCPKG_CRT_LINKAGE and VCPKG_LIBRARY_LINKAGE inside the triplet file [1]. These will then be used by vcpkg_configure_cmake to automatically set the appropriate compiler options and BUILD_SHARED_LIBS, so cmake-based ports will generally get support for free :) (@aardappel).

Are there any plans to also support individual libraries as statically-linked but built against the dynamic runtime?

@Orvid Yes, this is supported! We don't currently ship with a triplet that has the appropriate settings, but you can  create your own triplet or modify the existing ones.
Finally, due to unfortunate interactions with the core of C++'s MSBuild, it is exceedingly difficult to reliably switch triplets based on the project's CRT linkage so we have removed that particular feature. <arch>-windows will always be selected when targetting Windows desktop and if you'd like to use an alternative triplet you will need to override the automatic detection.

MSBuild uses the property VcpkgTriplet which can be overridden with /p:VcpkgTriplet=<triplet> on the command line
CMake uses the macro VCPKG_TARGET_TRIPLET which can be overridden with -DVCPKG_TARGET_TRIPLET=<triplet> on the configure line
Other buildsystems are already encoding the triplet in their include dir and link dir, so just change those :)

[1] https://github.com/Microsoft/vcpkg/blob/master/triplets/x86-windows.cmake#L2

Thanks, that's even better :)


Not Sure This Works

I could be totally using this wrong.
Here is the link I'm trying to use:
https://platform.openai.com/docs/quickstart
What are your results? It does not seem to scrape anything. Whether I use API or Browser method.

link didn't crawl for me either, could be link... I have no issues with any other sites yet


[REFACTOR] Move Hardcoded Value to Constant for Max Invitations

Feature Description
Here:
https://github.com/juspay/hyperswitch/blob/b2364414edab6d44e276f2133bc988610693f155/crates/router/src/core/user.rs#L501
In the current implementation, the value 10 is hardcoded when checking the number of invite requests in the code. To improve maintainability and avoid magic numbers, this value should be moved to a constant in crates/router/src/consts/user.rs.
Possible Implementation

Create a new constant in crates/router/src/consts/user.rs to represent the maximum number of allowed invite requests.
Refactor the code to replace the hardcoded value 10 with this new constant.

Have you spent some time checking if this feature request has been raised before?

[X] I checked and didn't find a similar issue

Submission Process:

Ask the maintainers for assignment of the issue, you can request for assignment by commenting on the issue itself.
Once assigned, submit a pull request (PR).
Maintainers will review and provide feedback, if any.
Maintainers can unassign issues due to inactivity, read more here.

Refer here for Terms and conditions for the contest.

Hi @ThisIsMani,
I would like to work on this issue. Can you assign it to me?

@nabeel001, seems like @ramkumargs is already working on this issue. Can you please choose any other issue.

Hey @ramkumargs, can you comment here, so I can assign this issue to you.

Hey @ThisIsMani I am already working on this.


Elasticsearch health check on CI

Add Elastic health check to execute tests when Elastic is ready.

Pull Request Test Coverage Report for Build 334

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

Totals

Change from base Build 332:
0.0%

Covered Lines:
517

Relevant Lines:
542

💛  - Coveralls


Fetching pull requests failing

Extension version: v0.0.14
VSCode Version: 1.53.2
OS: macOS 11.2.2

Steps to Reproduce:

Log in
Try to fetch pull requests

The PAT has all the required scopes, even tried giving it full access. I am the project administrator of the project I am trying to fetch pull requests from.
[Info] AzdoRepository> Fetching pull requests for search: {"status":1} failed: Error: {"$id":"1","innerException":null,"message":"A potentially dangerous Request.Path value was detected from the client (:).","typeName":"System.Web.HttpException, System.Web","typeKey":"HttpException","errorCode":0,"eventId":0}

orgUrl: https://dev.azure.com/akvagroupsoftware
projectName: Plan

Weird can you paste the entire log.

[Info] Looking for git repository
[Info] Git repository found, initializing review manager and pr tree view.
[Info] Review> Validate state in progress
[Info] Review> Validating state...
[Info] No remote with name 'upstream' found.
[Info] Found GitHub remote
[Info] Unable to resolve remote: Error: {"$id":"1","innerException":null,"message":"A potentially dangerous Request.Path value was detected from the client (:).","typeName":"System.Web.HttpException, System.Web","typeKey":"HttpException","errorCode":0,"eventId":0}
[Info] Review> no matching pull request metadata found for current branch master
[Info] AzdoRepository> Fetching pull requests for search: {"sourceRefName":"master"} failed: Error: {"$id":"1","innerException":null,"message":"A potentially dangerous Request.Path value was detected from the client (:).","typeName":"System.Web.HttpException, System.Web","typeKey":"HttpException","errorCode":0,"eventId":0}
[Info] AzdoRepository> Fetching pull requests for search: {"status":1} failed: Error: {"$id":"1","innerException":null,"message":"A potentially dangerous Request.Path value was detected from the client (:).","typeName":"System.Web.HttpException, System.Web","typeKey":"HttpException","errorCode":0,"eventId":0}
[Info] AzdoRepository> Fetching pull requests for search: {"status":1} failed: Error: {"$id":"1","innerException":null,"message":"A potentially dangerous Request.Path value was detected from the client (:).","typeName":"System.Web.HttpException, System.Web","typeKey":"HttpException","errorCode":0,"eventId":0}
[Info] Review> Queuing additional validate state

Can you please set azdoPullRequests.logLevel to debug, reload the window and share the log once again.

[Debug<PHONE_NUMBER>.968s] AzdoRepository> Auth> Successful: Logged userid: c8667a92-e38a-63ac-bdf5-3a8f6fe52443
[Debug<PHONE_NUMBER>.973s] Looking for git repository
[Debug<PHONE_NUMBER>.973s] Git repository found, initializing review manager and pr tree view.
[Debug<PHONE_NUMBER>.311s] Review> Validate state in progress
[Debug<PHONE_NUMBER>.311s] Review> Validating state...
[Debug<PHONE_NUMBER>.314s] No remote with name 'upstream' found.
[Debug<PHONE_NUMBER>.314s] FolderRepositoryManager> Displaying configured remotes: origin, upstream
[Debug<PHONE_NUMBER>.322s] Found GitHub remote
[Debug<PHONE_NUMBER>.323s] AzdoRepository> Fetch metadata - enter
[Debug<PHONE_NUMBER>.513s] Unable to resolve remote: Error: {"$id":"1","innerException":null,"message":"A potentially dangerous Request.Path value was detected from the client (:).","typeName":"System.Web.HttpException, System.Web","typeKey":"HttpException","errorCode":0,"eventId":0}
[Debug<PHONE_NUMBER>.523s] Review> no matching pull request metadata found for current branch master
[Debug<PHONE_NUMBER>.523s] AzdoRepository> Fetch pull requests for branch - enter
[Debug<PHONE_NUMBER>.523s] AzdoRepository> Fetch metadata - enter
[Debug<PHONE_NUMBER>.776s] AzdoRepository> Fetching pull requests for search: {"sourceRefName":"master"} failed: Error: {"$id":"1","innerException":null,"message":"A potentially dangerous Request.Path value was detected from the client (:).","typeName":"System.Web.HttpException, System.Web","typeKey":"HttpException","errorCode":0,"eventId":0}
[Debug<PHONE_NUMBER>.489s] AzdoRepository> Fetch pull requests for branch - enter
[Debug<PHONE_NUMBER>.489s] AzdoRepository> Fetch metadata - enter
[Debug<PHONE_NUMBER>.696s] AzdoRepository> Fetching pull requests for search: {"creatorId":"c8667a92-e38a-63ac-bdf5-3a8f6fe52443","status":1} failed: Error: {"$id":"1","innerException":null,"message":"A potentially dangerous Request.Path value was detected from the client (:).","typeName":"System.Web.HttpException, System.Web","typeKey":"HttpException","errorCode":0,"eventId":0}
[Debug<PHONE_NUMBER>.686s] AzdoRepository> Fetch pull requests for branch - enter
[Debug<PHONE_NUMBER>.686s] AzdoRepository> Fetch metadata - enter
[Debug<PHONE_NUMBER>.872s] AzdoRepository> Fetching pull requests for search: {"status":1} failed: Error: {"$id":"1","innerException":null,"message":"A potentially dangerous Request.Path value was detected from the client (:).","typeName":"System.Web.HttpException, System.Web","typeKey":"HttpException","errorCode":0,"eventId":0}

I have updated the extension to v0.0.15 with additional logging which may help me to narrow down the issue. Could you please update the extension to above version and share the log again. My current  guess is there is a colon (:) somewhere in REST request sent to AZDO. Need to narrow it down further.

I figured it out! I had set the projectName to include the organization URL so it was essentially requesting https://dev.azure.com/akvagroupsoftware/https://dev.azure.com/akvagroupsoftware/Plan. I didn't see it when I answered your first question either. 🤦  The additional logging helped me realize it though. 😁
So sorry!


Update bevy_egui to 0.5

I noticed this was behind the current bevy_egui release because I kept seeing multiple versions of bevy_egui being compiled so I decided to upgrade it.
I ran most examples as a sanity check and they all seemed to work. All the test also pass.
I tried updating other outdated crates, but it required actual change in the code.

Thanks! I'll update to the new rapier version when I get to it and then release a new version.


Dev

Deleted seaborn

Coverage decreased (-24.0%) to 54.742% when pulling 665d212ceadc0d5c9d5334f38f3644a13f50b349 on konchokdolma:dev into fa366671b191b25236c00cc2c61be79560784be4 on autonomio:master.

Coverage decreased (-22.2%) to 56.529% when pulling 583405323a2c4a88d92856c3f3c7ad083db6a9e2 on konchokdolma:dev into fa366671b191b25236c00cc2c61be79560784be4 on autonomio:master.

Coverage decreased (-22.3%) to 56.469% when pulling 32db4c4ade15cd5e2f53bc996059269bf57a2959 on konchokdolma:dev into fa366671b191b25236c00cc2c61be79560784be4 on autonomio:master.

Coverage decreased (-8.9%) to 69.844% when pulling 6e120faa1c8c355fe687c42bdb02dd927d705fbd on konchokdolma:dev into fa366671b191b25236c00cc2c61be79560784be4 on autonomio:master.

Coverage increased (+1.9%) to 80.589% when pulling 10190de0a0ba271eacaa3341aa3af71737686b89 on konchokdolma:dev into fa366671b191b25236c00cc2c61be79560784be4 on autonomio:master.

Coverage decreased (-8.9%) to 69.844% when pulling 10190de0a0ba271eacaa3341aa3af71737686b89 on konchokdolma:dev into fa366671b191b25236c00cc2c61be79560784be4 on autonomio:master.

Coverage increased (+3.8%) to 82.496% when pulling 30bce3e9b490fe5699588a6120dd1ade5fd91002 on konchokdolma:dev into fa366671b191b25236c00cc2c61be79560784be4 on autonomio:master.

Coverage increased (+1.9%) to 80.589% when pulling 93e4ea77f1abc88fafd945588ac02d35e6cd7c9f on konchokdolma:dev into fa366671b191b25236c00cc2c61be79560784be4 on autonomio:master.

Coverage increased (+3.8%) to 82.496% when pulling 390f42edeedfe27c1221c99e65304816d24773a6 on konchokdolma:dev into fa366671b191b25236c00cc2c61be79560784be4 on autonomio:master.

Coverage increased (+4.8%) to 83.536% when pulling a577faff2ca0e66a5603f5d58f2c85866a0e918f on konchokdolma:dev into fa366671b191b25236c00cc2c61be79560784be4 on autonomio:master.

Coverage increased (+6.7%) to 85.391% when pulling aa3d69a8c15e1b7e7d9c25328e90b27952a37e58 on konchokdolma:dev into fa366671b191b25236c00cc2c61be79560784be4 on autonomio:master.


Add Florentiner Recipe

Maybe consider adding a Florentiner recipe. It is very delicious and great for Christmas season.

Thank you for the comments! I implemented them in the changes!

Thanks again @martinaengel, I will merge this PR now!

@allcontributors please add @martinaengel for bug, content, code


Dont have Windows processes the interrupt signal

Changed proccessed_input mode so that we receive the 0x03 byte
rather than the program just exiting.  This way we can still restore
the console mode in Windows.
The corresponding change in *nix is unncessary since terminal.MakeRaw
already does this.

@howeyc You may want to try this out just to make sure I didn't have an issue when manually testing it.  I made a little test program that went and just got the password but had some fmt.Print statements temporarily in the code to see when I was setting/resetting the console.
Even when the problem existed the console mode was a little less obviously changed than on *nix when it can make your screen look like it exploded.

Addresses the Windows side of https://github.com/howeyc/gopass/issues/18 .  I dont think there is an existing problem on the *nix side.

Would be made obsolete by https://github.com/howeyc/gopass/pull/26  If we are merging that (and I hope we do) then this can be closed.  It should address the same issue (manually confirmed that it works and confirmed that in the ssh/terminal logic it is also changing the processed input mode like I do here).

I'm closing this as you suggest.


No instruction on how to actually give the account permission if it's missing. Nice.

No where on this page does it say how to give the account permissions. Really? The entire page goes through how to check it, then no example of how to fix it. Nice one.

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

ID: a6c7b290-3f3e-70c9-7524-80d7364dd763
Version Independent ID: d12b324f-57b2-9fd8-a433-b25644424171
Content: Troubleshoot self-service password reset writeback - Microsoft Entra
Content Source: articles/active-directory/authentication/troubleshoot-sspr-writeback.md
Service: active-directory
Sub-service: authentication
GitHub Login: @Justinha
Microsoft Alias: justinha

@crshovrd41
Thanks for your feedback! We will investigate and update as appropriate.


clients/transmission.py: fixed from_transmission plugin

Very simple fix of broken from_transmission plugin by removing leftovers of transmissionrpc (transmission-rpc uses another structure).
Addressed issues:

Fixes #2794 .

Thanks!

Thanks!


Getting a hold of injected Config outside of Controllers

In version 1.5.x we used to grab the pac4j session id in various places throughout our app, by doing the following:
sessionId = StorageHelper.getOrCreationSessionId(context.session());

When upgrading to version 2.0.1-SNAPSHOT we changed this into:
@Inject
Config config;

final WebContext webContext = new PlayWebContext(context, config.getSessionStore());
sessionId = config.getSessionStore().getOrCreateSessionId(webContext);

This new manner of grabbing (or creating) the sessionId only seems to work in Controllers, is that right? Elsewhere (models, services) config remains null. As I'm still quite unfamiliar with Dependency Injection in Play 2.4.x I may be missing something here.
Any comments or best practices?

Injection should work in all components using @Inject although I didn't test it outside controllers.
In all cases, you need the web context so I'm not sure how you can / could use that in models and services. Can you elaborate on this?

Sure. We use Deadbolt for role management. As such, we have an AbstractDeadboltHandler class that is used to perform certain checks on requests before they even get to Controllers. That class gets a context. We used to check for a pac4J session id using the StorageHelper.getOrCreationSessionId() function. Now it seems we have no way of getting to the session store from AbstractDeadboltHandler :cry:.

I'm surprised the config injection does not work. Though, if you want to share your configuration, you can certainly use the ConfigSingleton class.
Moreover, pac4j now handles authorization so you can certainly remove Deadbolt and use the authorizerName parameter.

I will try the ConfigSingleton approach. Thanks.
And we are definitely planning on migrating our Deadbolt logic to Pac4J ;). One adventure at a time.

When I use the ConfigSingleton class to obtain the pac4j config like so:
Config authConfig = ConfigSingleton.getConfig();

I get an empty config.
Is there a special way to get a hold of the config this way?

You must first do a: ConfigSingleton.setConfig(config); in your SecurityModules for example.

Understood. I've added it to the SecurityModule.
Thanks!

Keep us posted about the authorizations part...


Merge upstream packagist repository

We should merge the upstream repo regularly. The logic around package providers was improved a few days ago.

I posted a PR, so the Packagist WebBundle can be subtree splitted: https://github.com/composer/packagist/pull/478
With https://github.com/derhasi/packagist-webbundle I prepared a substree split that can be triggered by Travis CI.

@webflo, can you rebase the current master on composer/packagist:master? So we could see what we changed for Drupal Packagist in a linear history. There where to many rebase conflicts, and as I did not write the code, I think it would be best you review that at the first place ;)
After that I could try to add composer/packagist-webbundle on my own I guess.

Merged and deployed last week.


This PR:

adds a new constant recentEras :: Set AnyRecentEra

this constant defines the complete set of recent eras that are supported for transaction construction.

uses this constant to simplify the construction of the following API errors:

ApiErrorNodeNotYetInRecentEra
ApiErrorUnsupportedEra

Issue
ADP-3359
Follow-on from #4595.

👍 (I think the api is the wrong place for computations, but this is not relevant for the refactoring)

I agree with you!
Though hopefully this refactoring goes in the right direction: the definition of the set of recent eras is now in the Write layer (and no longer in the api layer).


[BugFix] fix failed ut about DdlException

What type of PR is this：

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

I will remove all unreasonable ddlExceptions in the final pr.
https://github.com/StarRocks/starrocks/issues/11349
Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] I have added user document for my new feature or new function

run starrocks_fe_unittest

[FE PR Coverage Check]
:heart_eyes: pass : 0 / 0 (0%)


Is it safe to create an NSDateFormatter inside of the mapping function? Will it be called many times if it is trying to map an array?

Is it safe to create an NSDateFormatter inside of the mapping function? Will it be called many times if it is trying to map an array?

The mapping function is called once per mapped object. I will leave it up to you whether you think this is safe or not, however IMO NSDateFormatter objects are light weight enough to be created on the fly


Bump kubernetes version

Bumping kubernetes version from ab2ca04 to bed16d2
Signed-off-by: Joaquim Moreno<EMAIL_ADDRESS>

Hit the build flake #222, rerunning jobs


Adding new observer+subscriber pair

goal is to edit the lift sequence listed in rmf_rxcpp/design.md document.
eg. item1 after lift arrives, I wish to trigger a new next subscriber to get information of lift crowdedness and depending on the result,

either carry on to item2 move robot into lift,
or trigger next chain of observer_subscriber pair (i.e. cancel lift request -> request for lift again).
this is for the use case where robot and passengers are allowed to co-share a lift.

Would need advice on which files to modify the lift sequence, create new type of observer to lift crowdedness module (within RMF) and whether the existing templates available in observer class is sufficient for the described feature above.

I would suggest putting that logic into the lift_supervisor instead. The lift_supervisor is responsible for managing the "lift sessions" that determine which agent is using the lift at any given time.
If the lift supervisor is aware of the crowdedness of a lift, it could simply refuse to initiate a session with the AMR until people clear out of it.
A very simple prototype would be: Add a condition to this if-statement that evaluates to false if the lift is too crowded to allow an AMR to enter.
I have plans to develop a "smart lift adapter", but I don't anticipate I'll have time to begin developing in earnest that until January.


OrientDB(2.0.5): Error in restoring the Database through plocal

Having an issue in V2.0.5, I'm able to take back up using backup command through plocal successfully. While restoring database with zip file its giving an error that "uncompromising database.ocf file :
The process cannot access the file because another process has locked the file.".  I'm checked that any other process is locking that datafbase.ocf file. Found no other process is locking that.
I have uploaded the screen shots to know exactly what is issue.

Seems a configuration problem of permissions on directories and running OrientDB console or servers.


Cannot get passed loading screen

Eth-Hot-Wallet
Please direct redux-saga related questions to stack overflow:
http://stackoverflow.com/questions/tagged/redux-saga
Issue Type

[X ] Bug
[ ] Feature

Description
When running code from download I cant seem to get it passed the loading screen.
Any tips?
(Add images if possible)

Steps to reproduce
download -> npm install -> npm run start
Versions

ETH-Hot-Wallet (see package.json):
Node/NPM: "node": ">=7"
Browser: true

I have the same issue

Turns out this issue is related to issue #24
Resolve that, and it loads.


The answer to question #6 of chapter 13 seems wrong.

Hello:
I think the answer to question #6 (c,d) of chapter 13 (https://danhalligan.github.io/ISLRv2-solutions/multiple-testing.html) is wrong.
The FDP is defined with V/(V+S), which is (false positive)/(false positive+true positive),
For Bonferroni procedure the false discovery rate will be 0/7, 0/7, 0/3.
For Holm procedure the false discovery rate will be 0/7, 0/8, 0/8.

Agreed -- this looks like an error thanks. Do you want to make a pull request and I'll merge in? If not, I can make the edits.

@danhalligan you can make the edits, thanks


Fix misspell in ja/index.html

Summary
Fixed misspell on https://jestjs.io/ja/ page
Test plan
'でｓ' would change into 'です'

signing done. @facebook-github-bot

Thanks for the PR @shin1ohno! However, the file you've changed is generated - the translations lives on crowdin: https://crowdin.com/project/jest/ja. Maybe https://crowdin.com/translate/jest/1221/en-ja? I'm sad to admit I don't really know how Crowdin works, but that looks correct.
Once you update there, a new deploy of the website will pull in that change.


add required files for Oracle Database 11g Release 2 Enterprise Edition

basic copy of setup/startup scripts, and simple modification of some values.
Sufficient to get a running database, can probably be much improved

I've added @gvenzl to review but it's unlikely this will be merged as I don't believe we support 11gR2 running inside a Docker container.
I'll wait for his response before reviewing myself.

@Djelibeybi, you are correct. 11gR2 nor <IP_ADDRESS> are supported in a Docker environment and therefore won't be made available in this repository.


chore: upgrade test should use v1.1.0 by default

What type of PR is this?

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #

/retest

looks like we have a legitimate v1.1 => main upgrade issue

/retest


[core] Fix input object generation

packages used in type arguments were not included in input objects

Note that there was another issue that came up during the incident where if the package for a type argument does not exist, a validator could end up waiting forever for it, so holding off on the accept until we know what the implications of that are,

That should be covered? though I'm not sure exactly how that bug came about.
This would add the type arguments as an input object, so the certificate should not get generated if the object does not exist for that validator?

That should be covered? though I'm not sure exactly how that bug came about.

Yes this will be covered by this PR.

Does this mean we could get rid of this error type since these should be impossible now? https://github.com/MystenLabs/sui/blob/main/crates/sui-types/src/messages.rs#L1550

Does this mean we could get rid of this error type since these should be impossible now? https://github.com/MystenLabs/sui/blob/main/crates/sui-types/src/messages.rs#L1550

Ya know... this should have been a big hint to us in retrospect


Early exist to the notebook

User feedback

one thing that would be nice is to be able to exit the notebook (without throwing) (like a return or signal that the notebook is done running the cells), perhaps this is already documented though

Originally posted by @bneigher in https://github.com/alexandreCameron/boar/issues/50#issuecomment-961381144
Comment
The feature does not exist for the moment.
Feedback seems to indicate that there may issues with the documentation, feel free to ask other questions in the issues.
Proposition
Adapt the output of the running.run_notebook function to allow give cell by cell execution status

@bneigher , thank you for your feedback, it's really nice to see that people enjoy this tool.
This package started like a side project and since I've been busy with personnel issues.
I've opened the issue but I won't be able to give you any schedule for the next release.


テスト用 Issue

Please provide us with the following information:

This issue is for a: (mark with an x)
- [ ] bug report -> please search issues before submitting
- [ ] feature request
- [ ] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce

Any log messages given by the failure

Expected/desired behavior

OS and Version?

Windows 7, 8 or 10. Linux (which distribution). macOS (Yosemite? El Capitan? Sierra?)

Versions

Mention any other details that might be useful

Thanks! We'll be in touch soon.

テスト用の Issue で今後使用しないためクローズ


4351 device management library

Added "Fragments" for adding device management library data to ManagedObjects.
Isn't implemented in "Operations" yet (see devicecontrol).

Coverage of package devicemanagementlibrary is now at 89%, but all remaining lines are trivial getters.
But I spotted a semantic bug, while looking at all uncovered lines, so thanks for the hint :)


Added Random movie Name generator

Fixes #1692 @iamrahulmahato I have added random movie name generator
Before

After

@PoojaKanagaraj06 resolve the conflict

@PoojaKanagaraj06 not solved

@PoojaKanagaraj06 if you are getting issue please update the fork then add your code then make a pr request

@iamrahulmahato okay

@iamrahulmahato corrected merge conflicts


New Mecanum Wheel drive chain

Develop and prototype a modular chassis for new mecanum wheels.

Abandoned early last semester. Wheels are in the cabinet in case of future interest.


refactor: allow more reuse of nodejs_binary starlark lib

I'm using this for a new jest_test rule which wants to return a similar launcher binary but control the args

@alexeagle Do you have an example of what your jest_test rule looks like? Trying to achieve the same in rules_js.

@bazaglia I have some code started in a private rules_jest repo, but haven't made progress on it in a while. I guess I could make it public, or just invite you for now

@alexeagle That'd be great. If you invite me I might be able to help you with a PR.


Mobile notification problem

Description
I m using rocket chat on my own Linux server and i have downloaded android latest version mobile apps in mobile, but when someone messaged me i didn't get notify like other apps. all my users facing same problem.  but we get notify on desktop version only mobile app not get any notification.
Devices and Versions

Your Rocket.Chat.Android version: (e.g. 2.1.0)
Your Rocket.Chat Server version: (e.g. 0.63.1-develop)

Mobile device model and OS version: (e.g. "Nexus 7 - Android 6.0.1")
Steps to reproduce

Logs

Same for us. Not working since about Monday 30th Jul.

Yes we do.

This might be related: https://github.com/RocketChat/Rocket.Chat.Android/issues/1568

@AmShaegar13 Can you confirm if you're receiving the notifications on the iOS app?

I m using gateway. rocketchat. Com and I have problem with both ios and the android app, both mobile notifications not working

@himel2017 I see... I just tested here and works for me. Can you confirm if that happens only to your user or to all users?

I had multiple complaints from many users. We just rolled out mobile access to our instance so I cannot say if it's really all users affected but I do not know of any user that got notifications since Monday.
Both Android and iOS affected.

@AmShaegar13 Thanks! It was working before? Wondering if it's related to some setting on the server that was changed. It doesn't work even for DMs?

It was working before but suddenly stopped work..

I can give you a my admin login info

Yup. Was working before and does not even work for DMs.

@AmShaegar13 @himel2017 Just check with the back-end guys and they recommended you guys to try restarting your server. There's a bug that happens sometimes and the database gets stuck, and the library that's being used to send the push notification stop sending it at all. Can you try that and let me know please?

OK I will....

@rafaelks Restarting the server fixed the problem for us. Is this something we can expect to happen frequently or was that issue related to a configuration change on your end?

@AmShaegar13 It may happen frequently, yes... it's a bug in MongoDB that wasn't fixed yet.


using  Illuminate\Foundation\Testing\DatabaseTransactions

Sorry to open an issue without running any code but i think i have enough info to ask what others may be wondering.
Mysql seems to commit open transactions when a new one is open. http://stackoverflow.com/questions/1490846/mysql-transactions-within-transactions
So how does the Illuminate\Foundation\Testing\DatabaseTransactions trait in this package work if the code called in the test starts a new transaction and commits it. In this case could a rollback work?
If not does that mean this trait can only be used to test code that doesn't use transactions?
Would a problem develop if someone used it wrongly i.e. could wrong use of this feature corrupt the db for later tests?

What laravel vesion?

I'm sure we fixed this?

I think this applies to at least  5.2 and the current tree. If you say you think we fixed this does that mean it's a sounds something like valid issue? Has it been discussed somewhere?


feat (DOCS-CODE-013): [api] canonical protected api access another protected resources on behalf user

closes: #508177

@mmacy -- This is ready for your review.  Feel free to review the README and anything else you wish before initial merge.

thanks a lot for your feedback/improvements over here @mmacy 💪
@ckittel, it sounds we've got green light to squash!


Added section on updating controllers to avoid deprecation notices

Added material from a comment I made regarding the new deprecation notices, and how to update your code to prevent them.

I'm starting a new project and encountered this error.
As there will be loads of controllers and methods I don't want to have misunderstood anything and incorrectly start this project.
According to what I read on the web and your solution I undestand that we should create a factory per controller. Is that correct?
I previously used a single method to get the Doctrine EntityManager and called it whenever I needed it. I used:
$this->getServiceLocator()->get('Doctrine\ORM\EntityManager')
This method now triggers a deprecated warning. Does it mean that I should move it to a service and pass it to every controller through specific factories?
As I might have more than 100 controllers this seems to be quite redundant and long to develop... Would there be any better and clean solution?

@aspic10

Would there be any better and clean solution?

Sure, use an abstract factory.

Thanks! I had never used abstract factories and it seems to be a simple and clean solution. Again, to be sure I start correctly I need your comments on what I did.
I created an abstract factory that will instantiate the controllers and pass the EntityManager as an argument.
In the module.config.php I removed the controllers from the "invokables" and "factories" sections" and added my abstract factory in the "abstract_factories" section.
If a controller needs different arguments I will add it to the "factories" and create a specific factory.
Is this correct?

@aspic10 Yes, it's correct. Service Manager will use factory at first if it is given.
If there is no factory for your service then SN goes through all abstract factories.
Please read secion about Factories in documentation. Everything is there explained with some important "Best practices"

Everything is there explained with some important "Best practices"

Right!

Thank you very much. I understand factories much better now and will improve my code to make it cleaner and more performant.

One quick note: with zend-servicemanager v3 (which MVC v3 requires), abstract factories are also compatible for use as factories. This allows you to map multiple controller names to the same factory, making it more explicit how they will be created.

From what I read in the link provided by webimpress and weierophinney's answer the best practice would be to use a common factory instead of an abstract factory.
I'm still not sure yet that I have got it all and that I'm doing it right though. Is this example correct:
In module.config.php:
'controllers' => array(     'factories' => array(         Controller\Index::class => Factory\DefaultControllerFactory::class,         Controller\Other::class => Factory\DefaultControllerFactory::class,         Controller\Third::class => Factory\DefaultControllerFactory::class,         Controller\User::class => Factory\UserFactory::class,     ), ),
And in the default factory:
`public function __invoke($container, $name, $requestedName, array $options = null) {
$serviceManager = $container->getServiceLocator();
$entityManager = $serviceManager->get('doctrine.entitymanager.orm_default');
$controller = $requestedName . "Controller";
if(class_exists($controller)){
   return new $controller($entityManager);
}
else {
    return false;
}

}`
I had previously defined an abstract factory that implemented AbstractFactoryInterface and did pretty much the same, except controllers related to it weren't defined under "invokables" nor "factories" in the module.config.php file.

@aspic10 you can use abstract factory if you'd prefer, it would be nothing bad with it, and as you said then you don't need define "invokable" nor "factories" in config. One abstract factory is not going make your application dramatically less efficient :) Please notice that this abstract factory will be defined only for controllers (controller manager) - so it will be checked only on creating controller.
If you decide use common factory, then you need define these relations in module.config.php as you did. In your default factory code probably $requestedName is already controller name, so you don't need append "Controller" and you don't need return false if controller is not found.
Personally, I would go with abstract factory in that case.


The "template card" does not work as expected

The usual "fields" process templates perfectly. However, no templates are interpreted in the "Actions". I would like to use the action "URL" for an action type and a template should serve as the target. Unfortunately, template is not converted in actions.
Will this be possible in a future version?
To Reproduce
Steps to reproduce the behavior:

create a mushroom teplate card
select any action choose action type "URL"
insert as target http://{{ states('sensor.any_sensor_ip') }}
test the selected action, the target is: "http://%7B%7B%20states%28%27sensor.any_sensor_ip%27%29%20%7D%7D/"

Expected behavior
Open a browser-tab with the IP-Address inside the template.
Additional context
Home Assistand Core Version: 2022.4.5
Mushroom Version: 1.6.2

The action doesn't support template and we can not add template support because it will be to complicate. Also Mushroom only standard action editor so the action editor of home assistant should be modified.

OK, that makes sense!
Thank you for your great work!


Application State Machine

Can we change this epic to something else? It's cluttering up our backlog.


[9.x] Add Arr::nth and Arr::trim

This PR adds two small functions to the Arr class that I now often solve with ugly code.

Convert empty strings to null, can also be a nice addition to Arr class

I think I've seen a similar PR before for the nth function. The thing is that if you try to access the array:
$arr = ['foo' => 'bar', 'k' => 'v']

using a numerical index, it becomes ambiguous when the array is defined as:
$arr = ['k' => 'v', 'foo' => 'bar']

Arguably, those two arrays are the same (internally they have a different order though).
I'm sure there are cases though to access the keys by index but at first sight it looks like an antipattern. If all else fails, you could do it with raw php using:
$second = $arr[array_keys($arr)[1]];

Or the collection variant:
$collection = collect($arr);
$second = $collection[$collection->keys()[1]];

Also note that there is an nth function in the Collection class which "Creates a new collection consisting of every n-th element.". So there's a bit of a meaning/naming collision there.


⚠ Samsung has degraded performance

In 3325b80, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2666 ms

Resolved: Samsung performance has improved in d10d990.


brew language not applied to preview

Renderer
v3
Browser
Chrome
Operating System
MacOS
What happened?
Loaded a saved brew, opened the properties panel, changed the language to es, saved the brew, opened the preview, inspected the rendered brew.

Code
No response

linked to #3294


Problems with intronic_splicing_enhancer and exonic_splicing_enhancer

Brought to my attention by Colin Logie
What is the SO term name and accession?
intronic_splicing_silencer (SO:0002056)
exonic_splicing_silencer (SO:002058)
Both are children of silencer (SO:0000625)
Also relevant, splice_enhancer (SO:0000344)
Describe what you would like to change.
There are multiple issues with the current ‘cis-regulatory module’ (CRM, SO:0000727) child term ‘silencer’ (SO:0000625) and its current two child terms SO:0002056 and SO:0002058.
Firstly, in the field, a transcription silencer has been equated to a ‘negatively acting ‘enhancer’, a CRM that is bound by DNA-binding repressors of transcription, such as those that nucleate the formation of heterochromatin to silence the transcription of a gene.
Second, the potential confusion between SO:0002056 and 2058 with RNA splicing control region terms that are in the SO:0001056-SO:0000344-SO:0000683-SO:0000320 branch.
Thirdly, the current molecular scenario captured by the SO:0002056 and SO:0002057 terms that may have to be accommodated, whereby unspliced mRNA binding proteins would influence DNA transcription by a DNA-template-dependent RNA polymerase. Notably, to the best of my knowledge, as of 2020, no, or very scant peer-reviewed experimental data-based literature has documented such a scenario.
Fourth, the converse scenario from 2056 and 2058, namely whereby a DNA-bound transcription factor that promotes DNA looping, influences splice site selection has been reported (PMID: 29199022). It may therefore be good to include that in SO.
Specific issues and possible solutions

The definition of SO:0000625 silencer is correct. The issue are the current children SO:0002056 and SO:0002058, that are very special cases of RNA sequence-dependent DNA transcription regulation.

The ancestor terms’ (SO:0000625 and SO:0000727) definitions include transcription factors, which are DNA binding proteins that influence the rate, frequency and/or efficiency of RNA biosynthesis by RNA polymerases, whilst the two child terms are about RNA elements and RNA binding proteins that influence RNA splicing by the spliceosome if they were to be referred to as splicing factors (this is explicit in the literature reference PMID: 23241926 . This can be solved by moving the ontology branch consisting of the two current children of SO:0000625 to another parent term that would represent CRMs that act as RNA elements that are bound by RNA binding proteins to influence transcription, contrary to the enhancer (SO:0000165) and the locus control region (SO:which are CRMs that act at the DNA sequence element level by virtue of being bound by DNA binding proteins.

Note that the SO:0000344 branch is correct as a child of splicing_regulatory_region (SO:0001056). But, the parent of SO:0001056 should not be transcription_regulatory_region (SO:0001679) as these are not transcription regulatory elements. It is therefore misplaced. SO:0001056 should be a child of SO:000831.

There is a parental CRM term that encompasses RNA and DNA cis-regulatory modules; those that respectively act at the RNA level and those that act at the DNA level. It is SO:0000831. A new child for this term, next to transcription (SO:, translation and recombination control regions, would be a splicing control region (currently splicing_regulatory_region (SO:0001056)).

The converse scenario from 2056 and 2058, namely whereby a DNA-bound transcription factor that promotes DNA looping, influences splice site selection has been reported (PMID: 29199022). It may therefore be good to include that in SO. So a spicing regulation branch that depends on DNA-bound proteins that influence mRNA splicing.

Relevant Publications
PMID:23241926 speaks about silencers on RNA affecting the splicing of RNA as opposed to silencers on DNA that affect transcription.

Hi Colin,
I think this can be fixed without changing that many terms. Below is my proposal:
Move splicing_regulatory_region:
id: SO:0001056; name: splicing_regulatory_region; def: "A regulatory_region that   modulates splicing."
Change to is_a: SO:0000831 !   gene_member_region
Create a new child term:
name: splice_silencer; def: "An RNA splicing regulatory   element that functions to recruit trans-acting splicing factors suppress   splicing." [PMID:23241926, SO:ke]
is_a: SO:0001056 !   splicing_regulatory_region
Move the two misplaced terms:
id: SO:0002058; name: exonic_splicing_silencer; def: "An exonic splicing regulatory   element that functions to recruit trans acting splicing factors suppress the   transcription of the gene or genes they control." [PMID:23241926, SO:ke]
is_a: New_Term !   splice_silencer (obviously I will add the ID once it is created)
is_a: SO:0000852 ! exon_region
id: SO:0002056; name: intronic_splicing_silencer; def: "An intronic  splicing regulatory element that functions   to recruit trans-acting splicing factors suppress the transcription of the   gene or genes they control." [PMID:23241926, SO:ke]
is_a: New_Term !   splice_silencer
relationship: part_of SO:0000188 ! intron
I think this will cover the issues. Colin, do you agree?
@keilbeck, do you agree with these changes?
Thanks,
Dave

I concur Dave, except for the literature reference for SO:0002056 and SO:0002058 as
PMID:23241926 does not mention transcription regulation at all. It is a review about splicing control and appears to me to be about the terms in the SO:0001056-SO:0000344-SO:0000683-SO:0000320 branch. Quote: "These SREs are conventionally classified as exonic splicing enhancers (ESEs) or silencers (ESSs), and intronic splicing enhancers (ISEs) or silencers (ISSs)." I do not know literature about non-viral eukaryotic RNA sequences and the proteins they would recruit to act as transcription regulator activity related to that of the sequence-specific DNA-binding Transcription Factors (GO:0003700). A viral example is TAR (GO:1990970) and that is activating rather than silencing transcription.
Small thingie: "... to recruit trans-acting splicing factors suppress the transcription of the gene or genes they control ..." should read "... and suppress ...", shouldn't it?
Having though more about this, I propose to erase SO:0002056 and SO:0002058, as well as SO:0002057 since the latter goes nowhere and appears to me to be entirely redundant with the SO:0001056-SO:0000344-SO:0000683-SO:0000320 branch for splicing regulation.
The alternative is to keep SO:0002056 and SO:0002058 for transcription repression and to create 2 more terms for transcription activation. However, in Wikipedia the TAR RNA is referred to as a microRNA.  I am therefore unsure that it could be referred to as as either an intron or an exon. Hence, the 2056 -2058 branch appears to need more work, since it lacks a transcription activation sub-branch and since exons and introns may not be sufficient nor appropriate to describe the example of RNA-directed transcriptional control in eukaryotic cells that is known to me, namely the TAR-TAT system.
Best, Colin
@davidwsant
@keilbeck

Hi Colin,
I think if the terms are moved to under SO:0000831 (gene_member_region) that they would no longer be in an area of the ontology that is related to transcription and this would solve the problem. I think the terms are needed as the comments in the obo file tell us that they were requested by someone, so I do not think they should be obsoleted unless the definitions change dramatically. I think at the time they were created they meant terms that affect splicing, but were mistakenly put under terms related to transcription. I think moving them still seems ideal.
Following the proposal above, I still think it would be good to move splicing_regulatory_region (SO:0001056) to is_a gene_member_region(SO:0000831), and then create a new child term splice_silencer which would be the parent of 2056 and 2058. This will mean no obsoleting of terms. If new terms that affect the splicing and transcription are made, those terms would need a new ID. I do not think such terms are necessary, however.
Do you think this would be ok?
Thanks,
Dave
@colinlog
@keilbeck

I accidentally clicked "close and comment" instead of "comment". It is not re-opened.


Slot is not working in rare and unpredictable cases

Version
2.6.14
Reproduction link
https://stackblitz.com/edit/vuejs-starter-ipzkcs?file=app.js
Steps to reproduce
Please look at the field "additionalArguments", it contains a link.
Press ALT M to load new data, instead of rendering a link here, the default slot content for that named slot is rendered instead.
You can press ALT J to load the original data, again no link.
What is expected?
Instead of rendering the default slot content defined by the child, the passed down slot from orderDataCard should be rendered instead (the link).
What is actually happening?
The slot can not be manipulated by collapsableCard.

So I have this weird problem where a certain slot doesn't work properly in very rare, but unknown conditions. What happens is that instead of rendering the slot according to the grandparent (OrderDataCard), the grandchild's (HeadLessTable) slot renders the default slot instead.
If you look at the render window in stackblitz, you will notice a table where a row contains the label "additionalArguments". Here, instead of rendering a string, a link shall be rendered instead. And this is obviously already working for this example. However, if you change the data via ALT M, the slot stops working and the default content will be rendered.
What's funny is that, if I use vue-devtools to copy over the new values one by one, there will be no issues.
CollapsableCard wraps the slot of HeadlessTable and passes it to OrderDataCard in a way that is described here: https://github.com/vuejs/vue/issues/11478#issue-647344835
But the issue here is not that this is done in a reusable way - it also happens if you pass the slots manually (EDIT: see here)
What I noticed is that if you uncomment line 19 in app.js, the issue does not occur. But in production the issue arises e.g. when invoices is set (line 88 app.js).

Your reproduction is not minimal, please read and follow https://new-issue.vuejs.org/?repo=vuejs/vue-next#why-repro when reporting a bug.

Your reproduction is not minimal, please read and follow https://new-issue.vuejs.org/?repo=vuejs/vue-next#why-repro when reporting a bug.

I minimalized it, please open the issue again. Check Reproduction link section or click here for minimalized code.


Android Toast callback on touch

Expected behavior
Receive args.event data on Android like it works on iOS
Actual behavior
I receive only args = "OK"
I'm seeing this behaviour on

[ ] iOS device
[ ] iOS sim
[x] Android device
[ ] Android sim

Hardware models
iPhone 5 (works) - Samsung SM-T355 (Android 5.1.1)
OS versions
(Android 5.1.1)
I'm not a dummy, so I've checked these

[ ] It happens on a fresh Cordova CLI project as well.
[ ] I'm waiting for deviceready to fire.
[x] My JavaScript has no errors (window.onerror catches nothing).

So how can we reproduce this?
window.plugins.toast.showWithOptions(
                {
                    message: message,
                    duration: "long",
                    position: "bottom",
          styling: {
            backgroundColor: '#66B2FF',
            textColor: '#000000',
          }
                },
                //Success callback
                function(args) {
          console.log('cordovaToast: '+JSON.stringify(args));
          if (args && args.event == "touch") {
            //next function
          }
                },
                function(error) {
                    log('cordovaToast error: ', error);
                }
    );

Thanks,
Luca

I just tested this code on a 5.1.0 simulator which works fine..
08-11 09:21:48.433 1996-1996/io.cordova.hellocordova I/chromium: [INFO:CONSOLE(37)] "cordovaToast: {"event":"touch","message":"hey there"}", source: file:///android_asset/www/index.html (37)

Which plugin version is this?
Does it work correctly on other Android models (or simulators)?

This appears to be a duplicate of #88

I've just found the same issue. In my case (maybe this is some hint) the touch event doesn't work when the toast code is inside event listener registered using `document.addEventListener(). For example:
document.addEventListener('onAdDismiss', function(data) {
$timeout(function() {
if (window.plugins.toast) {
window.plugins.toast.showWithOptions({
message: 'Tap to remove ads',
duration: "long", // 2000 ms
position: "center",
styling: {
opacity: 0.75, // 0.0 (transparent) to 1.0 (opaque). Default 0.8
backgroundColor: '#FF4848', // make sure you use #RRGGBB. Default #333333
textColor: '#FFFBF0', // Ditto. Default #FFFFFF
textSize: 16, // Default is approx. 13.
}
},
function(result) {
console.log(angular.toJson(result));
if (result && result.event) {
if (result.event === 'touch') {
$scope.showaPage('BuyMe', 'TOAST AFTER AD');
}
}
})
}
}, 1000);
});
``
In that case the touch event handler doesn't work, but the "hide" event comes and the console outputs it. properly.
If I use the same code from outside an event listener, touch works OK.
Maybe this is the reason for other people as well?


Offline favorites no pages

As of version 2.3.1, all of my offline favorites are said to have 0 pages. this is only seen when accessing the doujin from the offline favorites menu, accessing the same doujin from the online favorites, view history, and the main page/home works normally.
A50 running on android 10

+1, i also find the problem that i can use the online tag to filter the doujin(i have logged in ). the local tag function works.

If I understand the problem there is an option in settings to disable the online tags


important config breaks @apply

When in config:
{
  important: '.wnd',
  ...
}

windi classes are al scoped in .wnd but
@apply 'returns' nothing.
without the important config @apply works as expected.

Have to check later..

Is this svelte or sveltekit?

The issue is with svelte-windicss-preprocess and sveltekit.
Was using the npx svelte-add windicss which is using svelte-windicss-preprocess instead of vite-plugin-windicss.

👍 then we do suggest using vite-plugin-windicss for svelte-kit for now

Yes. I also added a comment to the svelte-add repo, hopefully preventing more issues here.

#65


[Feature Request] Graph Axes

Rename the LHS search result graph (raw numbers) as "# of Grades"
and the RHS compare graph (percents) as "% of Grades"

changing names to "# of Students" and "% of Students", as the numbers are referring to the number of students that have received each grade on the graph, rather than the grades themselves.


Filter duplicate Insertion

Is there moor way to automatically filter duplicate data insertion , especially at moor files ?

You could add a unique constraint and then use INSERT OR IGNORE INTO ... to avoid inserting data twice.

Like this INSERT OR IGNORE INTO user VALUES :userCompanion ?

I mean you can write into(user).insert(userCompanion, mode: InsertMode.insertOrIgnore) in Dart. I can take a look at supporting companions inside moor files too, but it'll probably be some work.

This is clean too. Thx for ur help. This issue can be closed now

Wait! sadly this does not filter duplication here. How insertOrIgnore works?

How can I detect the duplication by a column ? Example, detect email column of incoming data for filtering.

How can I detect the duplication by a column

If you want that column to be unique, you can add a unique column constraint. That should work together with INSERT OR IGNORE.
Or do you want to allow duplicate values in general, but avoid them when inserting data in a batch?

Cool if I can define table like this in moor file and InsertMode.insertOrIgnore automatically detect it
CREATE TABLE user (
  id INTEGER PRIMARY KEY NOT NULL,
  name TEXT,
  email TEXT,
  UNIQUE(id,email)
);

The UNIQUE(id, email) key won't do much - id is the sole primary key, so it's unique on its own. You can just use a UNIQUE (email) constraint if you want them to be unique.

if you also insert the primary key (id), you can do that on the table level:
CREATE TABLE user (
  id INTEGER,
  name TEXT,
  email TEXT,
  primary key ("id") on conflict replace
);


JSON file is invalid (has markdown appended)

When specifying the JSON format and saving to a file, an extra bit of markdown text is appended, causing the JSON to be invalid.
Using the action like this
      - name: Link Checker
        id: lychee
        uses<EMAIL_ADDRESS>        env:
          GITHUB_TOKEN: ${{secrets.GITHUB_TOKEN}}
        with:
          args: '--verbose ./_posts/**/*.md --exclude-mail --scheme http https'
          format: json
          output: ./lychee/links.json
          fail: false

eg. here's the end of the resulting JSON file
    "./_posts/2011/2011-11-01-springtime-in-garden.md": [
      {
        "url": "http://msmvps.com/blogs/bill/archive/tags/Life/default.aspx",
        "status": "Failed: Network error"
      }
    ]
  }
}[Full Github Actions output](https://github.com/flcdrg/myproject/actions/runs/2089026665?check_suite_focus=true)

I think the culprit is https://github.com/lycheeverse/lychee-action/blob/master/entrypoint.sh#L28
I don't think this should be appended if the format is JSON

You're right. We're missing an "if" condition here.


read/2 cause segmentation fault

Following code cause segmentation fault:
:- initialization(main).

main :-
    open('foo.pro', read, Stream),
    read(Stream, S),
    close(Stream),
    write(S),
    halt.

Backtrace:
Core was generated by `tpl foo.pro'.
Program terminated with signal SIGSEGV, Segmentation fault.
#0  0x000000000040c1db in xref_rule (l=0x7ffd4116f2e0, n=0x0) at src/trealla.c:2519
2519            node *tmp = NLIST_FRONT(&n->val_l);
(gdb) bt
#0  0x000000000040c1db in xref_rule (l=0x7ffd4116f2e0, n=0x0) at src/trealla.c:2519
#1  0x000000000041507c in bif_iso_read2 (q=0x6c253d0) at src/bifs_iso.c:1192
#2  0x0000000000432484 in call (q=0x6c253d0) at src/jela.c:461
#3  0x000000000040cc31 in query_run (self=0x6c253d0) at src/trealla.c:2718
#4  0x000000000040e6dc in trealla_run_query (self=0x6b08010, src=0x6c040e0 "main") at src/trealla.c:3251
#5  0x000000000040e338 in trealla_consult_file (self=0x6b08010, name=0x7ffd411713a0 "foo.pro") at src/trealla.c:3148
#6  0x00000000004043ba in main (ac=2, av=0x7ffd41170308) at tpl.c:328

Yes, that area is a mess. Am re-doing it.

Working in my few simple tests, may need more work.

Cool!
Now the output is:
Trealla v0.1alpha, 64-bits (int=128-bits), Dec 16 2016
initialization(main) :-
        Halted
DEBUG: orphaned=5

There are two problems here:

Order of initialization(main) and :- is reversed.
orphaned=5 debug info is lefted.

Also issue #33 seems to be effected by this change, I'll reopen that issue.


Improve README, especially Mapbox auth instructions

See commit messages for more details.

Additionally, I know it's not the scope of this PR, but "Configure credentials" for map box here should be also changed.

Thanks! I've replaced those instructions with a pointer to the earlier ones.


AudioStreamPlayer2D not working inside a Subviewport no sound is played.

Tested versions
v4.3.stable.mono.official [77dcf97d8]
v4.4.dev4.mono.official [36e6207bb]
System information
Godot v4.4.dev4.mono - Windows 10.0.19045 - Multi-window, 1 monitor - Vulkan (Forward+) - dedicated NVIDIA GeForce RTX 3060 Ti (NVIDIA; <IP_ADDRESS>94) - AMD Ryzen 7 3700X 8-Core Processor (16 threads)
Issue description
Audio does not play when inside a Subviewport.

https://github.com/user-attachments/assets/dc6ba1f2-d617-4c1b-a7f6-6d17e721974e
Steps to reproduce
Enable playing and autoplay in the AudioStreamPlayer2D and move it inside SubViewport and then move it outside.
Minimal reproduction project (MRP)
audiobug.zip

Searching I found this was a bug before:
https://github.com/godotengine/godot/pull/76713
Maybe it can help finding the solution.

The problem was that there is an option to enable sound in the viewport. Just enabling it and everything worked.