dnd-drop event is not getting called in Safari browser

I am going through documentation and provided demo's and noticed that dnd-drop event is not getting called in safari browser.
See link - http://marceljuenemann.github.io/angular-drag-and-drop-lists/demo/#/advanced
I am trying implement this functionality in all browser. It is working fine in Chrome but causing issue in safari browser.
Any help would be apriciated

I am also getting this issue. dnd-drop is not getting called in Safari. no console errors.

I can confirm this issue with the Brave browser (Chrome-based), while Chrome itself does trigger the event.

seems like "dragend" event is being treggered instantly after you start dragging an element.


Clarify expectation of trust_bundle with regards to mTLS

When confiuring an SSL proile on the switch for mTLS there is generally two thing in particular done

A particular CA is marked as trusted. This will be used to authenticate the clients certificate and does not necessarily have to be part of the server certificate chain.
A certificate chain is formed on the server with a leaf certificate and 0 or more intermediate certs.

The proto comments contains the following
// Target (as seen from gNSI.certificate microservice point of view)
// |
// +-+ certificate (used by all gRPC services)
// | +- certificate (with public key)
// | +- private key
// |
// +-+ trust bundle (Certificate Authority certificates)
// | +- CA Root certificate
// | +- CA Intermediate Certificate

This would indicate to me that all CA's as part of the trust bundle are only to be marked as trusted and the server will only have one certificate used in it's chain.
I think a reasonable change to this would be to change
  oneof entity {
    Certificate certificate = 3;
    CertificateBundle trust_bundle = 4;
    CertificateRevocationListBundle certificate_revocation_list_bundle = 5;
  }

to
  oneof entity {
    Certificate leaf_certificate = 3;
    CertificateBundle intermediate_certificates = 4;
    CertificateBundle trust_bundle = 5;
    CertificateRevocationListBundle certificate_revocation_list_bundle = 6;
  }

The private key would be optional for all certs in the chain except for the leaf certificate. This is already marked as optional so we are good here.

@tomek-US
I believe our view was the trust_bundle contains the CA root as well as all intermediates
I am not sure why we would seperate it out like this?

It sounds like trust_bundle is the chain including the RootCA that created the leaf server certificate.
This means that there is no option to provide a distinct list of CA's that may have signed a client certificate which would be used in mTLS.

Yes, the assumption behind this API design decision is that the trust bundle includes the whole chain of CA certificates that are needed to verify the incoming connection from clients. It might be but does not have to be the one that is used to sign the servers certificate. The assumption is that when the server (host) certificate is provided using the Rotate() operation it is from a trusted source and there is no need to verify it.

Understood, to a trust_bundle is a list of trusted certificates that the server can use to verify a client during mTLS.
The issue still stands with the client validating the server. The server must return its chain to the client, at the moment the message can only include the leaf certificate, but if there is an intermediate CA in the chain there is no way to provide this.
      ┌───►serverInterCA─────►serverLeaf
      │
rootCA│
      │
      │
      └───► clientInterCA────►clientLeaf

in this case the rootCA would be the trusted cert for the client and server, (trust_bundle). Ther client would give clientInterCA and clientLeaf. This would be validated by the server fine. The server should give serverInterCA and serverLeaf to the client but the spec only allows for serverLeaf to be passed and therefore returned. It should not be the servers job to figure out what CA's in the trust bundle are also part of the servers chain by trying to validate it itself, instead the client should be able to be specific on what the servers chain is and what the trusted certs are.

Hmm. I guess I am missing something but to my best knowledge the server does not send the CA trust bundle to the client. The client has to have it locally and use it to verify the received server certificate.
Could you point me to some text describing the scenario with the server providing the CA?

The server should not send the trusted certificates, but should send any intermediate CA that is part of its own chain. This may or may not include the root CA.
this is discussed here https://www.rfc-editor.org/rfc/rfc5246#section-7.4.2

OK. If I understood correctly a single leaf as a server certificate is not enough. Then making it a bundle should solve the problem, right?
  oneof entity {
    CertificateBundle certificate_bundle = 3;
    CertificateBundle trust_bundle = 4;
    CertificateRevocationListBundle certificate_revocation_list_bundle = 5;
  }

With this approach one can specify either a single/leaf server certificate or a chain of certificates that are needed by the client to verify that the server uses a valid certificate.

That is correct. It should be clear that a private key is only needed on the leaf certificate and the leaf certificate should be clear, either first or last certificate.

I like the idea of introducing the concept of a parent in the chain of certificates. Both options look good to me.


NodeOrderFn return float value

What this PR does / why we need it:
As I said in issue #708, changing the return value of NodeOrderFn from float to int would be better.
The ability of NodeOrderFn remains unchanged now, because functions like leastReq still returns int type and int type could change to float type safely.
But the code after this would benefit from it.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #708
Special notes for your reviewer:

Just change the return value

Release note:

/lgtm

/approve


added input keywords

Did you test if cumpleaños doesn't create encoding issues?
b'day - does the quote need to be escaped?


Add advanced event handling RFC

For https://github.com/uber/deck.gl/issues/2099
Background

See RFC

Change list

RFC catalog
Advanced event handling RFC

@Pessimistress I'm late to the party, but the other thing that'd be great to include is the ability for a layer to interrupt or prevent certain interactions (e.g. dynamically prevent map panning).


logo isnt visible on chrome iphone5

Is this still an issue? Can you provide more details?

@mccraveiro  I dont think its an issue now, @Dogfalo  any updates on this?

@Dogfalo and @acburst Can you confirm if there is an issue? If not mark this closed? Or atleast add some details regarding the issue

This is no longer an issue


Hooks: UI for information after commit

Successful or failed commit because of an Actions execution should enable the end-user to understand what was running and or failed from on the last commit.

This is done - the action's hooks and logs are available to the user.


add prometheus support

This proposed patch integrates metrics exporting for prometheus.
It eliminates the need for the prometheus graphite-exporter (or any other exporter) between the monitored application and prometheus.
In order to do that, it opens its own HTTP listener, which can optionally encrypted using a self-signed certificate. (More SSL support could be added on top of that.)
The patch integrates both into yammer and codahale modules.
An example reporter/exporter config is included for Apache Cassandra and has been tested with versions 2.1.x and 3.x.
A big section of the configuration (as in the example config) is dedicated to map yammer/codahale metric names to prometheus names and labels.

Thanks for your pull request. Prometheus support would make a great addition.
I got a maven build error because of protobuf toolchain is not setup locally. I haven't had much experience using protobuf in my projects, so it took me a bit to figure it out. I think the dependency on this extra setup outside of the project/pom is not ideal, especially given that it is only needed for one of the reporters, so I'd like to explore alternatives:

Include the compiled java class directly in the project. It does require extra manual work for devs when making related changes in the future, but it reduces confusion and external dependencies for majority of the users.
Include the protoc executable in the project and use the protocExecutable config of the protobuf-maven-plugin, but there's the os issue, not to mention the bad taste of checking in an executable in source control
Something rather involved like this: http://vlkan.com/blog/post/2015/11/27/maven-protobuf/

To me #1 seems to be the best option. Although it may go against the best practices of using protobuf in general, it's probably the best compromise for this project.
Yuesong

I have a few questions. Why are you rolling your own metrics transformation code when prometheus provides a decorator class for metrics3? The prometheus devs also provide a push library, so why no use that? Reusing that code seems like a better approach than rolling our own.
https://github.com/prometheus/client_java
Please see pushgateway & the dropwizard modules.

If you would like to stick with the protobuf support it seems the prometheus devs provide a compiled java class for us here

@snazy any updates on this? This is actually something we will find very useful soon, so if you don't have the time I can take over in another PR.

@jhorwit2, I'm currently maxed out with other activities and unfortunately have very little time for this one. Happy to hand this over to you!
Well, the decorator approach only works for metrics3 but not the old metrics code, which is still used in the wild (e.g. Apache Cassandra 2.x).
Protobuf: I've a preference for the protobuf version as it is cheaper to handle than the text format.
Pushgateway: I assume your use case is about short running (batch) jobs?

Hi,
would be great to have this! In general it probably makes sense to use the prometheus client lib decorators but not the push library. Prometheus encourages to use push only for short running jobs. The right way is to have your service provide the prometheus metrics on a endpoint so the prometheus server can scrape it at whatever interval is configured.

@discordianfish That would require an api change to support passing the environment into the reporter config to add a servlet. If you want to expose the endpoint I would suggest you use the dropwizard decorator in combination with the metric servlet. Should only be a couple LOC to make it work.

To be exact, you need these two modules
            <dependency>
                <groupId>io.prometheus</groupId>
                <artifactId>simpleclient_servlet</artifactId>
                <version>0.0.16</version>
            </dependency>

            <dependency>
                <groupId>io.prometheus</groupId>
                <artifactId>simpleclient_dropwizard</artifactId>
                <version>0.0.16</version>
            </dependency>

and to add this to your application run.
CollectorRegistry registry = new CollectorRegistry();
registry.register(new DropwizardExports(state.getMetricRegistry()));
environment.admin()
    .addServlet("prometheus-metrics", new MetricsServlet(registry))
    .addMapping("/prometheus-metrics");

Actually after playing with that it's not great since it doesn't let you specify labels/values. I'll see about incorporating the servlet option into this.

Closing this in favor of https://github.com/addthis/metrics-reporter-config/pull/33.
@discordianfish I added the ability to do servlet type instead of push gateway; however, you will need to register the servlet yourself. I didn't want to pull in that dependency (dropwizard). You can look at the readme for an example.


Add TagCellLayout by @riteshhgupta

TagCellLayout
Its an UICollectionView layout class that takes care of all the logic behind making tags like layouts using collection-view. It allows you to adjust the alignment of your layout i.e Left || Centre || Right. Now the developer just has to take care of tag UI and nothing else.

Thanks


Upstream fix for sidekiq 6 5 0

Thanks for this! I will follow up with fixing the busted CI setup and expand the test coverage to cover the range of Sidekiq 6.x versions before cutting a release.


Háček is rendered incorrectly as a circumflex

This is how the famous Czech pangram looks like:

It should look like that:
Příliš žluťoučký kůň úpěl ďábelské ódy
Also the ď and ť are buggy in that font. Even though they technically are with háček, an apostrophe is used instead for the readability. That's not a problem for ľ and Ľ though (which are used in Slovak but not Czech)

Some fixes have been implemented in a PR against a fork of this repo, and also with Nerd Fonts:

https://github.com/jesusmgg/comic-shanns-mono/pull/7
https://github.com/Rodrigodd/comic-shanns-mono/pull/1
https://github.com/ryanoasis/nerd-fonts/pull/1546


[Bug]: Validation Error in Bm25 Retreiver

Bug Description
Getting this error
ValidationError: 1 validation error for NodeWithScore
node
Can't instantiate abstract class BaseNode with abstract methods get_content, get_metadata_str, get_type, hash, set_content (type=type_error)
Version
0.10.12
Steps to Reproduce
bm25_retriever = BM25Retriever.from_defaults(docstore=index.docstore, similarity_top_k=3)
response_synthesizer = get_response_synthesizer()
bm25_query_engine = RetrieverQueryEngine(
retriever=bm25_retriever,
response_synthesizer=response_synthesizer,
)
bm25_query_engine.query("query").response
Relevant Logs/Tracbacks
No response

@arnavroh45 It did work perfectly for me. Did you follow index creation as mentioned here ?
Otherwise I don't see any issue with the code you shared.

I did this
s3 = s3fs.S3FileSystem()
persist_dir = "s3_directoryt"
storage_context = StorageContext.from_defaults(persist_dir=persist_dir, fs=s3)
index = load_index_from_storage(storage_context)
query_engine = index.as_query_engine()

It is working fine with llama_index==0.9.39

@ravi03071991 I am confused is it not working with bm25 retriever?

@arnavroh45 in v0.10.x, there was a bit of a change to the codebase. It seems like you might have been mixing llama_index.legacy imports with newer imports
Either all imports are from .legacy, or none are


community[patch]: Add retrieval params to neo4j vector

Sometimes, you want to use various parameters in the retrieval query of Neo4j Vector to personalize/customize results. Before, when there were only predefined chains, it didn't really make sense. Now that it's all about custom chains and LCEL, it is worth adding since users can inject any params they wish at query time. Isn't prone to SQL injection-type attacks since we use parameters and not concatenating strings.

Thank you!


[Feature Request] broadcast an intent during a flow

Is your feature request related to a problem? Please describe.
Some of my use cases are initiated from a broadcast  (i.e. implements a BroadcastReceiver). I would like to automate the tests for these receivers.
Describe the solution you'd like
Some mechanism to initiate or simulate a broadcast (Local or System), or call adb to do it and check the result.
I would like to test when the following are sent:

correct broadcast (data/params)
incorrect broadcast ,
multiple concurrent broadcast (attempt to overload)
Also
no ANR
What happens when app is in non-interrupt state (e.g. user filling in a form) and intent is broadcast
Behaviour under Doze, standby or process death.
Test order of broadcast receipt (if dependent on order).

I'm sure I'm leaving something out, but that should cover a lot of it.
Describe alternatives you've considered
I'm currently just using scripts calling into ADB with a variety of known values and logging the interactions.

Hey @andre-artus can you explain the detailed use case you are looking at? What type of behavior do you want to verify in the maestro test?

We use broadcast Intents to simulate the behaviour of the physical scan engine on enterprise devices (as Zebra TC58) on emulators:

Example: Send Barcode "test" to our application:
adb shell am broadcast -a de.guettlerlogistik.hallenapp.scan --es barcode "test" de.guettlerlogistik.hallenapp
That should also be supported with maestro. :)
Have a great week.
Max

Sorry, @amanjeetsingh150 , I failed to respond to you (I don;t know if I missed the notification) but Max's examples is very similar to ours (scanner or other accessories on enterprise devices).

@andre-artus How did you solved it with scripts calling into ADB. Can you share an example?

Hey all,
This seems to a bit of a more niche use case, and we're a small team busy with work, so I cannot share any timeline on when we will implement it. That said if anyone decided to contribute this feature, we're always happy to review PRs.

Yes, this feature would be extremely useful for my workflow too, as I need to test an application on a Zebra scanner 🙏🏻


A fatal error has been detected by the Java Runtime Environment: SIGSEGV (0xb) at pc=0x00007f28889c36f7, pid=3403, tid=3404

Please provide a brief summary of the bug
Java version: 21.0.3+9-LTS (also seems to apply to 17.0.11+9-LTS)
OS: Debian GNU/Linux 12 (bookworm)
hs_err_pid3403.log
#
# A fatal error has been detected by the Java Runtime Environment:
#
#  SIGSEGV (0xb) at pc=0x00007f28889c36f7, pid=3403, tid=3404
#
# JRE version:  (21.0.3+9) (build )
# Java VM: OpenJDK 64-Bit Server VM (21.0.3+9-LTS, mixed mode, sharing, tiered, compressed oops, compressed class ptrs, g1 gc, linux-amd64)
# Problematic frame:
# V  [libjvm.so+0x7c36f7]  G1CollectedHeap::populate_archive_regions_bot_part(MemRegion)+0xe7
#
# No core dump will be written. Core dumps have been disabled. To enable core dumping, try "ulimit -c unlimited" before starting Java again
#
# An error report file with more information is saved as:
# /root/hs_err_pid3403.log

hs_err_pid3403.log is attached.
Did you test with the latest update version?

[X] Yes

Please provide steps to reproduce where possible
Install Temurin on Debian 12 Bookworm and run java -version
Expected Results
Display Java version without SIGSEGV
Actual Results
Above error
What Java Version are you using?
JRE version:  (21.0.3+9) (build ) # Java VM: OpenJDK 64-Bit Server VM (21.0.3+9-LTS, mixed mode, sharing, tiered, compressed oops, compressed class ptrs, g1 gc, linux-amd64)
What is your operating system and platform?
Debian GNU/Linux 12 (bookworm)
How did you install Java?
Temurin Debian repository.
Did it work before?
Java 8 seems to work without problem. Java 17 and 21 do not, but this is the first time I've tried Java 17 and 21.

Did you test with other Java versions?
No response
Relevant log output
No response

Can you try Java 11?  I'm trying to narrow this down between LTS's :-)

Certainly, I'll have that tested in a few minutes.

0 ✓ debfactory [rw] ~ # java --version
openjdk 11.0.23 2024-04-16
OpenJDK Runtime Environment Temurin-11.0.23+9 (build 11.0.23+9)
OpenJDK 64-Bit Server VM Temurin-11.0.23+9 (build 11.0.23+9, mixed mode)

Seems to work!

More information discovered during troubleshooting:
The application of Java here is on kiosk-like hardware. Java (Debian's Java  included) is selected and installed by Puppet config management. This same Puppet config management installed Temurin 8 and 11 "correctly".
If I manually install or reinstall the temurin-21-jre package, it seems to work.
0 ✓ debfactory [rw] /etc/apt/sources.list.d # java --version
openjdk 21.0.3 2024-04-16 LTS
OpenJDK Runtime Environment Temurin-21.0.3+9 (build 21.0.3+9-LTS)
OpenJDK 64-Bit Server VM Temurin-21.0.3+9 (build 21.0.3+9-LTS, mixed mode, sharing)

Oh I see OK, your crash is with the OpenJDK that Ubuntu packages, not temurin, so you'll want to post your original bug there.

No, Temurin 17 and 21 crash too. Debian's OpenJDK 17 crashes as well. So while I don't believe the Temurin builds are at fault, they do crash.
It seems like something may have changed for LTS versions higher than 11 that isn't compatible with something Puppet is doing to install OpenJDK (including Temurin). Not really sure what yet.
Cheers

Is puppet installing it in a different location?

No, it doesn't. In fact it should be installing almost identically. I can get you the exact flags it passes to apt shortly.

Debug: Executing: '/usr/bin/apt-get -q -y -o DPkg::Options::=--force-confold --force-yes install openjdk-17-jre'
This is what Puppet is executing under the hood if I specify openjdk-17-jre as an example.
The only thing that gave me a little pause was --force-confold, but as far as I can tell both Temurin and Debian's Java don't really ship with files marked as configuration?

After a lot of reworking the build process for our images and rebuilding a base image, things seem to work fine now.
It's still a mystery to me how it could have broken in this very specific way (especially with Temurin 8 and 11 working fine...) but it seems like there was something wrong in the base image. The base image is what get Puppeted to build specific "final" images. It's a head-scratcher mostly because installing OpenJDK (Temurin or otherwise) manually "fixed" the problem, but OpenJDK isn't part of the base image... So I don't know how "fixing" the base image could have fixed this.
Anyway, I appreciate the time you've had to spend on this. If I stumble upon the why for this problem I'll let it be known here for future reference.


How to add a new link on to the report

For exmaple ,
I want to add a link "google" on to the allure report which should direct me to the respective reference .

@msp248 sorry for the delay. Could you please explain more clearly what exactly do you need? Is it link for whole report or for each suite/case/feature you have in mind?
In case link for whole report there is few possibilities for you:
The report top-left link
Usually it is link to test execution system (CI). But if you want you can override it to whatever you need.

The environment variable

No , actually I want a link on top of the report near allure logo which can
direct me to the link which I want , for. example to Google.com . currently
am replacing the index.HTML file using bat command post site execution .
let me know if you have any other ideas for this
On 9 Nov 2015 17:45, "Dmitry Baev"<EMAIL_ADDRESS>wrote:

@msp248 https://github.com/msp248 sorry for the delay. Could you please
explain more clearly what exactly do you need? Is it link for whole report
or for each suite/case/feature you have in mind?
In case link for whole report there is few possibilities for you:
The report top-left link
Usually it is link to test execution system (CI). But if you want you can
override it to whatever you need.
[image: 2015-11-09 15 03 16]
https://cloud.githubusercontent.com/assets/2149631/11033291/0f514ad0-86f3-11e5-924d-ae8556d3cc5d.png
The environment variable
[image: 2015-11-09 15 12 06]
https://cloud.githubusercontent.com/assets/2149631/11033462/59cfcda6-86f4-11e5-9df3-9559156f063f.png
—
Reply to this email directly or view it on GitHub
https://github.com/allure-framework/allure-core/issues/701#issuecomment-155048279
.

I want a link on top of the report near allure logo

As I say you can add link on top-left near the allure logo. All you need if write environment.xml file to your results folder with the following structure:
<qa:environment xmlns:qa="urn:model.commons.qatools.yandex.ru">
    <name>Your google link name</name>
    <url>https://your-google-link.com</url>
</qa:environment>

closed as answered, if you have other questions - you can create a new one


[HELP] I can't open gtk::FileDialog with given "initial-folder".

gtk::FileDialog provides a way to set initial-folder, but it does not work for me no matter what I would do.
Could you please, let me know what I do wrong or it is a bug in gtk::FileDialog?
Are there any settings that gtk::FileDialog may depend on which turn on that kind of behaviour? I use GNOME 3.38.5, so I assume I could be missing some settings which normally present for latest GNOME versions.
I'm not very comfortable with C, so I haven't tried to rebuild same example in C and check whether it is a bug in GTK or in gtk4-rs bindings for Rust.
Thank you in advance for any help!
P.S.: initial-file and initial-name does not work for me as well.
Steps To Reproduce
cargo new file-dialog
cargo add gtk4 --rename gtk --features v4_10
cd file-dialog
cat <<'EOF' > src/main.rs
use gtk::{
    self,
    prelude::*,
    Application,
    ApplicationWindow,
    FileDialog,
    gio::{
        self,
        prelude::*,
        Cancellable,
    },
    glib::{
        self,
        prelude::*,
        clone,
    }
};

const APP_ID: &str = "info.zekefast.FileDialogTest";

fn main() -> glib::ExitCode {
    let app = Application::builder()
        .application_id(APP_ID)
        .build();

    app.connect_activate(build_ui);

    app.run()
}

fn build_ui(app: &Application) {
    let opened_dir = gio::File::for_path("./");

    let file_dialog = FileDialog::builder()
        .title("Open File")
        .modal(true)
        .initial_folder(&opened_dir) // THIS IS THE PART THAT DOES NOT WORK
        .build();

    let window = ApplicationWindow::builder()
        .application(app)
        .title(APP_ID)
        .build();

    file_dialog.open(Some(&window), Cancellable::NONE, clone!(@weak window => move |result| {
        if let Ok(Some(pathbuf)) = result.as_ref().map(|file| file.path()) {
            println!("{}", pathbuf.display());
        } else {
            eprintln!("{:?}", result);
        }
    }));

    window.present();
}
EOF

Once again source of src/main.rs with Rust syntax highlighted:
use gtk::{
    self,
    prelude::*,
    Application,
    ApplicationWindow,
    FileDialog,
    gio::{
        self,
        prelude::*,
        Cancellable,
    },
    glib::{
        self,
        prelude::*,
        clone,
    }
};

const APP_ID: &str = "info.zekefast.FileDialogTest";

fn main() -> glib::ExitCode {
    let app = Application::builder()
        .application_id(APP_ID)
        .build();

    app.connect_activate(build_ui);

    app.run()
}

fn build_ui(app: &Application) {
    let opened_dir = gio::File::for_path("./");

    let file_dialog = FileDialog::builder()
        .title("Open File")
        .modal(true)
        .initial_folder(&opened_dir) // THIS IS THE PART THAT DOES NOT WORK
        .build();

    let window = ApplicationWindow::builder()
        .application(app)
        .title(APP_ID)
        .build();

    file_dialog.open(Some(&window), Cancellable::NONE, clone!(@weak window => move |result| {
        if let Ok(Some(pathbuf)) = result.as_ref().map(|file| file.path()) {
            println!("{}", pathbuf.display());
        } else {
            eprintln!("{:?}", result);
        }
    }));

    window.present();
}

Expected Behaviour
File dialog  opens given directory ./ as a current directory.
Actual Behaviour
File dialog always opens "Recent Files".
Environment

Rust: rustc 1.68.2 (9eb3afe9e 2023-03-27)
gtk4 crate: 0.6.4
GTK: 4.10.0
GNOME: 3.38.5
OS: Debian GNU/Linux 11 (bullseye)

Notes on setup

GTK4 was compiled from sources and was installed in parallel with system-provided version of GTK3. I let cargo know about it with the following flags:
#!/usr/bin/env bash

export LD_LIBRARY_PATH="/opt/gtk/lib/x86_64-linux-gnu:/opt/adwaita/lib/x86_64-linux-gnu"
export CPPFLAGS="-I/opt/gtk/include -I/opt/adwaita/include"
export LDFLAGS="-L/opt/gtk/lib/x86_64-linux-gnu -L/opt/adwaita/lib/x86_64-linux-gnu"
export PKG_CONFIG_PATH="/opt/gtk/lib/x86_64-linux-gnu/pkgconfig/:/opt/adwaita/lib/x86_64-linux-gnu/pkgconfig/"

Sounds very much like an upstream issue, the bindings don't do anything special here.

@bilelmoussaoui Should I report it somewhere else or it will be automatically recreated based on assigned label?

Please open the issue here gitlab.gnome.org/gnome/gtk/


Resolve yarn check errors

yarn check is throwing a bunch of errors, mostly related to polaris. We should make sure the lockfile is properly up to date to avoid these.
Ideally we would have this check run in CI.

Closing because this project is now deprecated.


PerThing: from_rational and mul odd behaviour

PS probably fixed by https://github.com/paritytech/substrate/pull/13660
A benchmarking failure in https://github.com/paritytech/polkadot/pull/6780 prompted an investigation of the arithmetics of PerThing.
There are two observations:

The mathematical sentence $\lfloor {Amount \over Denom} * Denom \rfloor <= Amount$ always holds.
Now implementing this with Perquintill:

assert!(Perquintill::from_rational(amount, denom).unwrap().mul_floor(denom) <= amount);

This is not the case for these numbers:
#[test]
fn regress() {
	let amount =<PHONE_NUMBER>28156970819068829320184091938_u128;
	let denom =<PHONE_NUMBER>85736967680636164088585662015_u128;
	let approx_amount = Perquintill::from_rational_with_rounding(amount, denom, Rounding::Down)
		.unwrap().mul_floor(denom);

	assert!(approx_amount <= amount, "approx: {}, amount: {}", approx_amount, amount);
}

So it sometimes overshoots although it should round Down.

One problem seems to already be in the from_rational, since it rounds Up instead of Down. Wolfram Alpha output.

#[test]
fn regress() {
	let amount =<PHONE_NUMBER>28156970819068829320184091938_u128;
	let denom =<PHONE_NUMBER>85736967680636164088585662015_u128;
	let parts = Perquintill::from_rational_with_rounding(amount, denom, Rounding::Down)
		.unwrap().deconstruct();

	assert_eq!(parts,<PHONE_NUMBER>35475778_u64); // Fails with<PHONE_NUMBER>35475779
}

Extended tests to check these requirements.

Test 1
/// Test that this is never true:
/// from_rational_with_rounding(amount, denom, Rounding::Down) * denom > amount
#[test]
fn brute_force() {
	use f128::f128;
	use rand::{Rng, SeedableRng};
	use rayon::prelude::*;
	use sp_arithmetic::{Perquintill, Rounding};
	use sp_runtime::PerThing;

	(0..320).into_par_iter().for_each(|i| {
		let mut rng = rand::rngs::SmallRng::seed_from_u64(i);

		for _ in 0..10_000_000 {
			let amount: u128 = rng.gen::<u128>().saturating_add(1);
			let denom: u128 = rng.gen::<u128>().saturating_add(amount);

			let approx_ratio =
				Perquintill::from_rational_with_rounding(amount, denom, Rounding::Down)
					.expect(format!("{:?} / {}", amount, denom).as_str());
			let approx_amount = approx_ratio.mul_floor(denom);

			if approx_amount > amount {
				panic!(
					"TOO LARGE amount: {:?}, denom: {:?}, approx_amount: {:?}, diff: {:?}",
					amount,
					denom,
					approx_amount,
					approx_amount - amount,
				);
			}
		}
	});
}

Test 2
/// Test that the generated parts are correct.
#[test]
fn brute_force_amount_check() {
	new_test_ext().execute_with(|| {
		use f128::f128;
		use num_traits::{float::Float, ToPrimitive};
		use rand::{Rng, SeedableRng};
		use rayon::prelude::*;
		use sp_arithmetic::{Perquintill, Rounding};
		use sp_runtime::PerThing;

		(0..320).into_par_iter().for_each(|i| {
			let mut rng = rand::rngs::SmallRng::seed_from_u64(i);

			for _ in 0..10_000_000 {
				let amount: u128 = rng.gen::<u128>().saturating_add(1);
				let denom: u128 = rng.gen::<u128>().saturating_add(amount);

				let perfect_ratio = f128::from(amount) / f128::from(denom);
				let perfect_parts =
					(f128::from(Perquintill::ACCURACY) * perfect_ratio).floor().to_u128().unwrap();

				let approx_ratio =
					Perquintill::from_rational_with_rounding(amount, denom, Rounding::Down)
						.expect(format!("{:?} / {}", amount, denom).as_str());
				let approx_parts = approx_ratio.clone().deconstruct();

				if approx_parts != perfect_parts as u64 {
					panic!(
						"WRONG PARTS amount: {:?}, denom: {:?}, perfect_parts: {:?}, approx_parts: {:?}",
						amount, denom, perfect_parts, approx_parts
					);
				}
			}
		});
	});
}

fixed in #13660


Web - Amount sent message not shown after we send the money and remain in the chat

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Create a chat with any user
Press the + button and send any amount of money to that user
The message or the amount sent is not shown in the chat
When we change chats and come back to the original chat, only then the message is shown

Expected Result:
Message should be shown after we send the amount to user
Actual Result:
Message is not shown until after we change the chat and come back again
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.3.28-5
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/93399543/e0b91d59-67ac-434e-a9d1-86c1780782ea
Expensify/Expensify Issue URL:
Issue reported by: @MohsinAliTanvir
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1687176371429829
View all open jobs on GitHub

reviewing

I cannot reproduce this

Pinged in slack to get more reproduction steps/pre-reqs


Finding matching slot even when time is booked - Updated UI?

Testing this locally and Cypress is firing fine on the reservation page. I'm seeing odd behavior though where, for example, it says there's an open match at 5:00pm yet I can see in the Cypress screenshot it is "Sold Out". I'm wondering if something with the Tock layout has changed that prevents the matching logic to work like it did before?

Thanks for the report! Just to be clear, are you looking into this or are you asking me to? ;)
AFAICT your hunch about layout is likely correct. Happy to take a stab at it when I can assuming you’re not already doing so. Clear steps to reproduce would be helpful.

On Jun 14, 2024 at 12:53 PM,  <Pejman Pour-Moezzi @.***)>  wrote:
Testing this locally and Cypress is firing fine on the reservation page. I'm seeing odd behavior though where, for example, it says there's an open match at 5:00pm yet I can see in the Cypress screenshot it is "Sold Out". I'm wondering if something with the Tock layout has changed that prevents the matching logic to work like it did before?
book.reservation.--.for.first.available.time.preference.failed.png (view on web) (https://github.com/azoff/tockstalk/assets/481729/f8dd4f7c-f46d-43bc-8129-efc54c25a084)
@.*** (view on web) (https://github.com/azoff/tockstalk/assets/481729/e2a74629-05cf-4142-92ba-e5c2b4d37b4c)
—
Reply to this email directly,  view it on GitHub (https://github.com/azoff/tockstalk/issues/6), or   unsubscribe (https://github.com/notifications/unsubscribe-auth/AABHFRE2K2XE6POONRCICVTZHMU3NAVCNFSM6AAAAABJKYR44GVHI2DSMVQWIX3LMV43ASLTON2WKOZSGM2TGOBTGI4DANY).
You are receiving this because you are subscribed to this thread.Message ID:  @.***>


Add a physical close button to side panel

yeah, i wanted to get it in the top left with the same dimensions/location as the open button, but i couldnt easily figure out how to space it properly without making the title look weird, so i decided to do more useful things instead :P

added by 58dda5f68ca22fab97a52cc89ec86a1dea3f9550


Add ghidra 11 support

While not released yet, this allows to compile the plugin against latest ghidra master.
Only change is that getRelocation() was replaced by getRelocations()

The dev branch should be buildable with the Ghidra master branch.

Thank your for pointing this out, I completely missed this branch. Sorry for the noise!

Thank your for pointing this out, I completely missed this branch. Sorry for the noise!

No problem. Any thoughts on how this can be made clearer?


Model component Local Impact

Made changes to model component, as well as created a new Command called removetag.

Dont accept the PR. Not required to do so.


Correctly detect if KeyManager is not supported by OpenSSL version

Motivation:
We did incorrectly catch Error and not Exception which resulted in some probes not been executed when the OpenSSL version did not support using a KeyManager. This could result in using incorrect defaults in this case
Modifications:
Correct catch Exception
Result:
Use correct defaults even if KeyManager is not supported

Noticed this while working on some other SSL code 🤦


Provide docs about how to update default streams.

Automated message from Dropbox CLA bot
@mrdrozdov, it looks like you've already signed the Dropbox CLA. Thanks!

Are we rebase/squashing?

Yes the Zulip commit model is the rebase based model; see http://zulip.readthedocs.org/en/latest/code-style.html#commit-discipline

Following up on this comment from the original code:

I needed to rely on this command to update the 30ish users that had registered prior to us setting up default streams. It was too labor intensive to add each user for each stream. That being said, I see where you're coming from. It was maybe too specific for our use case to be included in this part of the docs.

Hmm, yeah, I think we probably should have a good solution in the web UI for that use case, but given your point I think it's probably worth leaving in your docs but just add a note at the start that you can add folks using the streams page normally but in some cases when adding default streams it can be handy to do the bulk operations on the command line and this is how to do it.

I went ahead and merged this after squashing the commits to match the Zulip commit style; we can always optimizing the text further in a later change.
Thanks for your contribution!

No problem. Thank you!


Bug: Time::createFromTimestamp() uses default timezone, not UTC for timestamp

Describe the bug
UNIX timestamps are all based on the UTC timezone. However, Time::createFromTimestamp() uses the default timezone. This means that created dates can be off by several hours
CodeIgniter 4 version
4.0.4
Affected module(s)
I18n\Time
Expected behavior, and steps to reproduce if appropriate
The timestamp should assume it is UTC, then create the date using the supplied timezone. Instead the time is offset. For example, the following code should output "2020-12-01 12:00:00". Instead it outputs "2020-12-01 06:00:00" (due to the -6 offset for Chicago).
use CodeIgniter\I18n\Time;
date_default_timezone_set('America/Chicago');
$date = Time::createFromTimestamp(1606824000, 'utc');
echo $date->format('Y-m-d H:i:s');

I suspect the problem is that the timestamp is being converted to a date format by just using date()
Context

OS: macOS 10.15.7 locally & Linux web server
Web server: Apache 2.4.46 locally, 2.4.43 on server
PHP version: 7.4.9 locally, 7.4.5 on server

The Time::createFromTimestamp() method does not convert the timestamp to the new time zone.
It just returns a DateTime instance with new timezone.
To convert use DateTime::setTimezone()
$date = Time::createFromTimestamp(1606824000);
$date->setTimezone(new DateTimeZone('utc'));

The Time::createFromTimestamp() method has a timezone argument. Going
$date = Time::createFromTimestamp(1606824000, new DateTimeZone('utc'));

should be the same as
$date = Time::createFromTimestamp(1606824000);
$date = $date->setTimezone(new DateTimeZone('utc'));

(turns out setTimezone() doesn't set the time zone of date but creates a new instance with the new timezone).
However, they give different results (the latter does give a correct result by converting the result back to UTC from the default timezone, but the former just sets the timezone property to supplied value without converting the actual value).


Change Thrift mirror

http://www.eu.apache.org/dist/thrift/0.9.1/thrift-0.9.1.tar.gz isn't working any more.
Maybe we should upgrade to 0.9.3? That is available here: http://www.eu.apache.org/dist/thrift/0.9.3/thrift-0.9.3.tar.gz
There doesn't seem to be breaking changes in the Thrift changelog.

At this rate should we maybe host it ourselves on Bintray?

There are some other alleged 0.9.1 mirrors linked here: http://www.filewatcher.com/m/thrift-0.9.1.tar.gz.3402353-0.html
But hosting it ourselves may be the most reliable option, as long as we remember to keep up with new releases.


flet build macos with numpy - built app crashes

Description
Building Flet app with numpy completes with no error on mac, but built app crashes right after opening a window.
Code example to reproduce the issue:
import flet as ft
import numpy as np

def main(page: ft.Page):
    page.add(
        ft.Text(f'Value: {np.random.random_sample()}')
        )

ft.app(target=main)

Build command:
flet build macos

Describe the results you received:
Built app crashes right after rendering a window.
Running the app by python main.py or flet run main.py works fine.
After commenting import numpy as np out, built app renders a window with no content. It won't crash.
Describe the results you expected:
Random float something like Value: 0.8875341508950007 appears on a window.
Additional information you deem important (e.g. issue happens only occasionally):
App built by flet pack main.py won't crash and works as expected.
Flet version (pip show flet):
Name: flet
Version: 0.21.2
Summary: Flet for Python - easily build interactive multi-platform apps in Python
Home-page:
Author: Appveyor Systems Inc.
Author-email<EMAIL_ADDRESS>License: Apache-2.0
Location: /Users/handsome/Documents/Python/fletAudioPlayback/.venv/lib/python3.11/site-packages
Requires: cookiecutter, fastapi, flet-runtime, packaging, qrcode, uvicorn, watchdog
Required-by:

Give your requirements.txt file (don't pip freeze, instead give direct packages):
numpy

Operating system:
ProductName:            macOS
ProductVersion:         14.3.1
BuildVersion:           23D60
M2 Max 12 core CPU + 30 core GPU
32GB RAM
Additional environment details:
% python --version
Python 3.11.7

% pip list|grep numpy
numpy                     1.26.4

% flutter --version
Flutter 3.19.3 • channel stable • https://github.com/flutter/flutter.git
Framework • revision ba39319843 (2 weeks ago) • 2024-03-07 15:22:21 -0600
Engine • revision 2e4ba9c6fb
Tools • Dart 3.3.1 • DevTools 2.31.1

% brew list cocoapods
Warning: Treating cocoapods as a formula. For the cask, use homebrew/cask/cocoapods or specify the `--cask` flag.
/opt/homebrew/Cellar/cocoapods/1.15.2/bin/pod
/opt/homebrew/Cellar/cocoapods/1.15.2/bin/xcodeproj
/opt/homebrew/Cellar/cocoapods/1.15.2/libexec/bin/ (5 files)
/opt/homebrew/Cellar/cocoapods/1.15.2/libexec/cache/ (34 files)
/opt/homebrew/Cellar/cocoapods/1.15.2/libexec/doc/ (11485 files)
/opt/homebrew/Cellar/cocoapods/1.15.2/libexec/extensions/ (4 files)
/opt/homebrew/Cellar/cocoapods/1.15.2/libexec/gems/ (2166 files)
/opt/homebrew/Cellar/cocoapods/1.15.2/libexec/specifications/ (34 files)

% xcodebuild -version
Xcode 15.3
Build version 15E204a

Numpy is a "native" package, please read here: https://flet.dev/docs/guides/python/packaging-app-for-distribution#native-python-packages

I can reproduce this issue by simply adding the numpy import to a hello world file.
Works:
import flet as ft

def main(page: ft.Page):
    page.add(ft.Text("Hello World!"))

ft.app(main)

Fails with crash after build:
import flet as ft
import numpy as np

def main(page: ft.Page):
    page.add(ft.Text("Hello World!"))

ft.app(main)

As the reporter noted, this causes a crash, after which point, all apps in this project will render a blank page (no crash), even if you remove the dependency.
The app works with flet run and even flet pack suggesting there may be an issue with flet build.

Related: https://github.com/flet-dev/serious-python/issues/85

There is an explanation and workaround for "numpy on macos" issue: https://github.com/flet-dev/serious-python/issues/85#issuecomment-2065000974
Either re-build the app with flet build macos (both 0.21.2 and 0.22.0 build templates where updated to set OPENBLAS_NUM_THREADS variable or implement a fix in your own python app.

Great! Thank you, @FeodorFitsner !
I just tested building my app with 0.21.1 and 0.22.0 without modifying my code, and both built apps worked with no issue. Awesome!


APK Release checklist for both  Covax app and ANC MVP

Covax Demo app

[x] User authentication
[x] Sync in progress in Navigation drawer
[x] Patient list with sync progress bar
[x] On sync complete alert user
[x] Pagination in patient list
[x] Filter by overdue immunization
[x] Search First name, Last name
[ ] Scan Barcode (Barcode ID as Patient Identifier)
[ ] If search not found, create new patient and add scanned bar code as the patient identifier
[x] View Patient profile with Immunization details /status
[ ] Record vaccine with correct alert dialog for Doses 1 & 2
[ ] One receive dose of vaccine refresh the patient profile page
[ ] VIew patient profile if partially vaccinated and fully immunized

ANC MVP

[ ] User authentication
[ ] Family list with sync progress bar
[ ] On sync complete alert user
[ ] Add a family
[ ] Assign head of household
[ ] Add members of the family
[ ] View Family Profile
[ ] View Family member profile
[ ] Search for a Family by Family name
[ ] Search for an individual ANC client i.e. by  First name, Last name

Is it however noted that the Barcode implementation is not yet merged on the Andsropif FHIR SDK thus may be missing from the generated APK

Covax review

Questionnaire with Comorbidity risk flagging should only be visible for JnJ use case/workflow
Remove Comorbidity Questionnaire and related-person details  for WHO Covax demo and leave only patient details
Integrate barcode functionality - still missing
Record vaccine questionnaire takes a while to load
Language switching not supporting radio buttons

ANC Review


Revise file sort in archive

I'm on macOS and the IC (ignore case) flag doesn't appear to work. The last file ends up first in order and the first file ends up (usually) second to last. I combined the original lower() function with the natsort. I also tried updating natsort >=4.0, which should allow locale() or humansort() to work on macOS, but I had no luck. However, sort() works just on the files I tested.

Can you provide a file that reproduces this issue? You can download the CBZ at this site and just rename the pages. I'm just trying to find out what caused this as I can't find any filenames that produce the issue your having.
Here is a trivial script which lists all the pages in a comic archive. It needs either comictagger installed or to be placed next to comictagger.py
list.txt
I've tested this both on the latest commit with every version of natsort between 5.1.0 and 8.1.0 and they all report the same output and the macOS binary for 1.3.0 and all report the correct sort order for ascii text.
Also this pull request cannot be merged as the signed keyword argument was removed in version 6.0.0 https://github.com/SethMMorton/natsort#dropped-deprecated-apis
Can you report your environment?
locale:
$ defaults read -g AppleLocale # if on macOS
$ locale # if on linux

or in python
>>> import locale
>>> locale.getlocale()

OS and version info:
$ cat /etc/os-release # if on linux
$ sw_vers # if on macOS
$ python -VV
$ pip list

I got the current version working when natsort is set to 8.1.0 but the minimum version in the requirements.txt isn't specified. It seems like it would be better to specify in the requirements, unless that screws something else up.
Here's my specs FYI.

import locale
locale.getlocale()
('en_US', 'UTF-8')

(venv) ➜  comictagger git:(natsort_fix) ✗ sw_vers
ProductName:    macOS
ProductVersion: 12.2.1
BuildVersion:   21D62
(venv) ➜  comictagger git:(natsort_fix) ✗ python -VV
Python 3.9.10 (main, Jan 15 2022, 11:48:04)
[Clang 13.0.0 (clang-13<IP_ADDRESS>)]
(venv) ➜  comictagger git:(natsort_fix) ✗ pip list
Package                   Version

altgraph                  0.17.2
beautifulsoup4            4.10.0
certifi                   2021.10.8
cffi                      1.15.0
charset-normalizer        2.0.7
configparser              5.0.2
idna                      3.3
macholib                  1.15.2
natsort                   3.5.2
Pillow                    8.4.0
pip                       21.3.1
pycparser                 2.20
pyinstaller               4.3
pyinstaller-hooks-contrib 2021.3
PyPDF2                    1.24
PyQt5                     5.15.5
PyQt5-Qt5                 5.15.2
PyQt5-sip                 12.9.0
PyQt5-stubs               <IP_ADDRESS>
requests                  2.26.0
setuptools                57.0.0
soupsieve                 2.2.1
unrar                     0.3
unrar-cffi                0.2.2
urllib3                   1.26.7
wheel                     0.36.2
Thanks for your work on this BTW. Since the slow demise of ComicRack, this project has been a great organizational resource.

Could you squash your commits and then force push this branch? It will make this change easier to follow once merged.
Examples can be found here https://stackoverflow.com/questions/5189560/squash-my-last-x-commits-together-using-git


feat: add an option for repository-local caching

This PR adds a feature to store Deno installation to repository-local cache with @actions/cache, which generally speeds up the workflow.
I believe it should be the default behavior, but it's turned off by default in this PR for safety.
I also modified log messages to make the difference between runner's cache of tools and repository-local caching clear, since I find it a little bit confusing that setup-deno always downloads Deno from remote regardless of saying "Cached Deno to ...".
I refered to setup-zig for implementation: https://github.com/goto-bus-stop/setup-zig
Not sure the best way to test the feature, but I turned on the feature only for matrix.deno == "~1.32", since adding cache = [true, false] to the matrix seems too much.
You can see the result of tests on my forked repository: https://github.com/hasundue/setup-deno/actions/runs/6634339100
The first run is without cache in the first place, and the second run hits the caches. We can see some difference in the step duration as expected.

We can see some difference in the step duration as expected.

Maybe not... :sweat_smile:

.@kt3k kindly pointed me out that actions/cache fetches repository-local cache from remote anyway, and thus it doesn't speed up the workflow as expected.
The feature is, however, still useful when you use a custom runner, like act, for example,  where requests to the cache server may be redirected to a local server.
But the complexity introduced here may not be acceptable for those limited benefit, so please close this PR in that case.

As you pointed, we are not in favor of adding these complexities and also many dependencies because the benefit of this feature looks too limited. Closing without merge.
Thanks for your contribution anyway @hasundue


ADAPT-000: Bump for release.

READY FOR REVIEW
Summary

Version bump
We already have a 1.0.0 tag...

Ahh makes sense!

Green lights all around. Thanks.


Vim.js Integration

https://github.com/coolwanglu/vim.js
Any chance to integrate vim keybinds for the keyboard-fluent? If you can point me the files to modify I can probably submit a pull request.

Also this could be really useful for remote collaboration if there's a text chat.

Theoretically this is something that could be 100% encapsulated by
Codemirror right?  Blindpad delegates pretty much the entire editing
experience to that logic.  Maybe there's a codemirror vim plugin that you
can try to integrate?  Totally happy to merge a PR that does this.
On Mon, Apr 17, 2017 at 12:04 PM, Ricky Han<EMAIL_ADDRESS>wrote:

Also this could be really useful for remote collaboration if there's a
text chat.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/blindpad/blindpad/issues/6#issuecomment-294561503,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEK18HxXT5rQOe0-mVw0IprrYyJEcuaYks5rw7fLgaJpZM4M_cio
.

--
Ryan Noon
http://rmnoon.github.io/ http://rmnoon.github.io/

Yes CodeMirror has Vim, Emacs and Sublime key integration. I got stuck on displaying dropdown.
If you can take a look it would be perfect.

cool!  thanks for contributing.  i'm a little busy this week but I'll take a look soon and see if I can make it work.

@rmnoon it's working now. Could you please test it? Thanks.

There's a PR open here #7 waiting for code review if you're still interested in this feature!


Width calculation only is correct when the view takes up the whole width of the screen.

Nice work on the bottom modal.
There is an issue with the width calculation anytime the view doesn't take up the full screen  (landscape tablets or web implementations).   line 91 in giphy_tab_detail.dart is using (MediaQuery.of(context).size.width which is simply the width of the screen and not the width of the view.  Using a LayoutBuilder to get the constraints maxwidth would fix this issue.

thank's, can you make a pull request fixing this issue?

hello @m-j-g can you test this version if this issue has fixed?

fixed in version 3.0.1


Please wait a bit while pyV8 binary is being downloaded

Hello.
I get the "Please wait a bit while pyV8 binary is being downloaded" message, but Emmet not working.

I am also facing the same issue(using ctrl+E) on fedora21 x64 and ST3 x64 build 3065.
Emmet.sublime-settings's "disable_pyv8_update" is false:
PyV8: Creating new thread
Emmet: Creating thread
Emmet: Loading https://api.github.com/repos/emmetio/pyv8-binaries/contents
Package Control: Skipping automatic upgrade, last run at 2015-01-05 10:51:55, next run at 2015-01-05 11:51:55 or after
Emmet: Loading PyV8 binary from https://raw.github.com/emmetio/pyv8-binaries/master/pyv8-linux64-p3.zip
Exception in thread Thread-5:
Traceback (most recent call last):
  File "/home/chenhao/.config/sublime-text-3/Installed Packages/Emmet.sublime-package/emmet/pyv8loader.py", line 424, in download
    return self.execute(command)
  File "/home/chenhao/.config/sublime-text-3/Installed Packages/Emmet.sublime-package/emmet/pyv8loader.py", line 330, in execute
    raise error
emmet.pyv8loader.NonCleanExitError: 22

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "./threading.py", line 901, in _bootstrap_inner
  File "/home/chenhao/.config/sublime-text-3/Installed Packages/Emmet.sublime-package/emmet/pyv8loader.py", line 575, in run
    package = self.download_url(url, 'Unable to download package from %s' % url)
  File "/home/chenhao/.config/sublime-text-3/Installed Packages/Emmet.sublime-package/emmet/pyv8loader.py", line 535, in download_url
    return downloader.download(url.replace(' ', '%20'), error_message, timeout, 3)
  File "/home/chenhao/.config/sublime-text-3/Installed Packages/Emmet.sublime-package/emmet/pyv8loader.py", line 427, in download
    code = re.sub('^.*?(\d+)\s*$', '\\1', e.output)
  File "./re.py", line 170, in sub
TypeError: can't use a string pattern on a bytes-like object

Emmet.sublime-settings's "disable_pyv8_update" is true:
PyV8 auto-update is disabled
Package Control: Skipping automatic upgrade, last run at 2015-01-05 10:51:55, next run at 2015-01-05 11:51:55 or after


C++ "Hello World" Project

Hi,
I would appreciate it if you can provide the C++ configuration details e.g example "Hello World" project for C++.
This would help fast track myself and others with C++ development on the ESP32.
Keep up the good work.
BR,
Brent

@GarageGadget
Great idea ... thank you for that sir.   Here is a first pass ... have a go and lets see if this meets the need.   Please post back with comments/results.
https://github.com/nkolban/esp32-snippets/blob/master/skeletons/cpp/main_helloworld.cpp

I replaced the main.c with main_helloworld.cpp file a project based on esp-idf-template project.
I had to add the include directory to the include path to resolve ESP_LOGD declarations.
The code compiled and flashed without issue but unfortunately none of the application messages were sent to the uart.
A full sample project with compiler configuration etc. would help.

Ahhh ... yes ... sorry ... by using ESP_LOGD() that logs at the debug level which is a lower level logging than default.  But it sounds like you got it figured.
Close this issue when you are happy.

@GarageGadget I'm having the same issue, will you guide me how did you adjust the log level to get the output ? Thanks in advance.
and @nkolban , Great tutorial and awesome effort writing the book about esp32. Really appreciated.

@mumairanwar92 The logging is documented here ...
http://esp-idf.readthedocs.io/en/latest/api-reference/system/log.html
At a high level, and assuming your are using ESP-IDF, run "make menuconfig" and navigate through the menus to the log settings.  Set the default log output level to debug or lower.  If you need further assistance, don't hesitate to post back.

@nkolban thanks for the comment. It has been resolved. you and your tutorials have been a great help throughout.

hi
I am unable to import includes file c_includes.xml.
after giving path. and finish nothing happens

Hi all,
I am too unable to import includes from the c_includes.xml. Exactly the same situation as arunbm123.
Has any solution been found?
Thank you.

https://github.com/nkolban/esp32-snippets/blob/master/eclipse/c_includes.xml

@nkolban
I ma using using esp idf environment on esp32 board. I want to decode url.
URL>>file:///F:/s?ssid=cabin+336&Passkey=Cabin336%40efc
In above url, 'Cabin336%40efc' includes '%40' which has its decoded value '@'
so how to decode this?
I tried to use base64.h library file from esp idf but gives an error..
fatal error: base64.h: No such file or directory

Check you esp-idf installation:
https://github.com/espressif/esp-idf/blob/master/components/wpa_supplicant/include/wpa2/utils/base64.h

@chegewara
yes, this file is on this location but when i used    #include "base64.h"  i gives error..

Try like this:
https://github.com/espressif/esp-adf/blob/master/examples/cloud_services/google_translate_device/main/google_sr.c#L31

@chegewara
hey Its working.
thank you so much for help and quick response..


#16887 Expand/collapse individual cells

https://user-images.githubusercontent.com/35821147/176406400-2e0d03d7-8ece-4763-918e-0aaf86333be4.mp4
Please ensure that both normal and record views work against relational and document databases.

Custom data types elements in pg are shown as n/a for now and cannot be expanded in record mode

Also, custom data types are broken in Oracle

Only arrays can be expanded at the moment


Prepare initial release.

research what's needed, read some about best practices
update the readmes for all our crates
check our public api and add missing docs (and maybe deny(missing_doc))
rename the tendermint fork and add a note in it's readme with what is it
some CI publishing job?

#69


chore: add feature requests to contributing.md

Changes proposed in this pull request

At @huijing 's request, I've added feature requests to the contribution guidelines.
Context

Checklist

[ ] Related issues linked using fixes #number
[ ] Tests added/updated
[ ] Documentation added
[ ] Make sure that all checks pass
[ ] Bruno collection updated

Temporarily blocked by https://github.com/interledger/rafiki/pull/2510. Build fails because the Netlify environment has been updated to node 20 in preparation for merging that in.
After which, all open PRs need to be rebased.


Wmi_Exporter 0.9.0 doesn't start on Windows 2008R2

Hello, wmi_exporter version 0.9.0 doesn't start on Windows 2008R2 in my environment.
It works find on later versions (20012 and 2016).
I was trying to downgrade to the version 0.8.3 but there's no luck as well.
The main problem I don't see any logs which explains what happens, it just freezes on the "starting server" stage.
Please assist

Hi @basift,
What are you expecting to happen after the "Starting server" log entry? What happens if you try to access localhost:9182?

Hi @carlpett,
Thanks for your answer.
I'm sorry there was a firewall issue, so it works fine on localhost.


ui: added search support for schema page

Fixes #8480
integrated search support for the schema page.

https://github.com/open-metadata/OpenMetadata/assets/71748675/c9653549-bd35-4e24-8c10-4e4d28ba145f

Type of change:

[ ] Bug fix
[ ] Improvement
[ ] New feature
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation

Checklist:

[x] I have read the CONTRIBUTING document.
[ ] My PR title is Fixes <issue-number>: <short explanation>
[ ] I have commented on my code, particularly in hard-to-understand areas.
[ ] For JSON Schema changes: I updated the migration scripts or explained why it is not needed.

@ShaileshParmar11 do we have cypress tests for these

@ShaileshParmar11 do we have cypress tests for these

I have in my local, need to make some modifications before pushing it.


Github actions for more connectivity!

We can implement github workflows for the following things:

Auto comment on issue Closed and Opened
Auto comment on PR mergs
Auto comment on PR raises

Using this workflows makes you'r repo more interactive ! @gauravsingh1281
i'm expecting a level3 with gssoc-ext and hacktoberfest-accepted label

Hey @smog-root two workflows have already added to the repo .
1 ) Auto comment on issue Opened
2 ) Auto comment on PR raises
you can work on closed Issue and PR merge workflows.


Fix nil error in +ivy/jump-list for empty buffers

For empty buffers which don't have any lines, (thing-at-point 'line)
will return nil. This broke +ivy/jump-list.

Looks good to me. Thanks for the help!


AKAmplitudeTracker forces sample rate of input signal to 44.1k. AudioKit needs variable sample rate support.

Routing an audiofile at 48k through AKAmplitudeTracker will result in that file's audio being played at 44.1k.
This isn't a problem specific to the tracker class, but to the underlying implementation.
AudioKit needs to support a true variable sample rate that conforms to AKSettings.audioFormat.

Update from Dave: (in progress) It’s still a huge project though. We need to refactor AudioKit to not use singletons and magic numbers.

I'll just close this as Dave clearly has it in progress.


use trtexec

i run this command
python optimize.py --input weights/superpoint_lightglue.onnx
and i get superpoint_lightglue_fused.onnx and superpoint_lightglue_fused_fp16.onnx
but when i use
trtexec --onnx='/home/sy/sy/lightglue_ws/src/LightGlue-ONNX/weights/my/sim/superpoint_lightglue_fused_fp16.onnx'  --fp16 --saveEngine=superpoint_lightglue_fused_fp16.trt --warmUp=500 --duration=10
to trans them into .trt file, the following error appears
[10/10/2023-10:53:57] [E] [TRT] 3: getPluginCreator could not find plugin: MultiHeadAttention version: 1
[10/10/2023-10:53:57] [E] [TRT] ModelImporter.cpp:771: While parsing node number 272 [MultiHeadAttention -> "/transformers.0/self_attn/Reshape_5_output_0"]:
[10/10/2023-10:53:57] [E] [TRT] ModelImporter.cpp:772: --- Begin node ---
[10/10/2023-10:53:57] [E] [TRT] ModelImporter.cpp:773: input: "Transpose_0_out"
output: "/transformers.0/self_attn/Reshape_5_output_0"
name: "MultiHeadAttention_0"
op_type: "MultiHeadAttention"
attribute {
  name: "num_heads"
  i: 4
  type: INT
}
domain: "com.microsoft"

[10/10/2023-10:53:57] [E] [TRT] ModelImporter.cpp:774: --- End node ---
[10/10/2023-10:53:57] [E] [TRT] ModelImporter.cpp:777: ERROR: builtin_op_importers.cpp:5404 In function importFallbackPluginImporter:
[8] Assertion failed: creator && "Plugin not found, are the plugin name, version, and namespace correct?"
[10/10/2023-10:53:57] [E] Failed to parse onnx file
[10/10/2023-10:53:57] [I] Finished parsing network model. Parse time: 0.0864084
[10/10/2023-10:53:57] [E] Parsing model failed
[10/10/2023-10:53:57] [E] Failed to create engine from model or file.
[10/10/2023-10:53:57] [E] Engine set up failed

my tensorrt version is <IP_ADDRESS>,  onnxruntime = 1.16.0

Hi @kajo-kurisu, thank you for your interest in LightGlue-ONNX.
The fused models are intended to be run via ONNXRuntime as they use contrib operators (under the com.microsoft domain).

thank u , i'll close this issue

@kajo-kurisu 请问下，您转trt引擎转成功了吗

@kajo-kurisu 请问下，您转trt引擎转成功了吗

我后来没用他的optimize.py ，用的onnx.simplify自己对onnx简化之后就能转化成功


Allow Base Color and Alpha Map textures to be combined on export

Currently, it appears that alpha transparency in textures can only be used effectively when it's included in the Base Color texture.
It would be amazing if the exporter was capable of combining the Base Color texture with an Alpha Map/Mask on export, possibly via Add Shader or Mix Shader + Transparent BSDF (or whichever makes the most sense).
Example:

This would greatly benefit project workflows where separate Alpha Maps/Masks are used.
Thanks for the consideration and all the great work.

Maybe a prerequisite question is, should Transparent BSDF, mixed with other nodes, be a supported way to export transparency? Without that the node graph above will not work anyway, and you'd need to connect the textures in a new node before connecting to the Base Color socket.
^My preference would be to get the Principled BSDF node's alpha channel reflected better in the Blender preview window, and then we wouldn't need to go after the Transparent BSDF node, which does not translate as well to glTF. The Blender devs have mentioned they want this too, but that change is probably more than I can manage.
In the meantime, I'm not sure what I think about supporting Transparent BSDF... we'd have to guess whether it's using 'CLIP' or 'MASK' modes I think. 😕

This would greatly benefit project workflows where separate Alpha Maps/Masks are used.

Is there a particular situation where separate alpha maps are preferable? I've certainly seen it in e.g. some COLLADA models, but I'm not sure why.

Hi, @donmccurdy.

Maybe a prerequisite question is, should Transparent BSDF, mixed with other nodes, be a supported way to export transparency?

I'm not partial to Transparent BSDF in any way; I was just providing a vague example of what I thought could be a possibility in order to combine the Base Color and Alpha texture on export.

My preference would be to get the Principled BSDF node's alpha channel reflected better in the Blender preview window

I actually believe there have been improvements here. Last I checked, the Base Color alpha channel was being reflected correctly in the viewport, which wasn't the case until somewhat recently.

Is there a particular situation where separate alpha maps are preferable? I've certainly seen it in e.g. some COLLADA models, but I'm not sure why.

In any case where the Base Color texture doesn't included alpha, say, when using a tiled base texture and want to map alpha to that texture without creating more faces, similarly to how emissive maps are used.
I've seen multiple instances of texture packs containing alpha maps/masks with their base textures. When trying to use them in Blender, I realized that I couldn't figure out how to export the alpha since they weren't combined.
I hope that makes sense.

Ok, I see. In that case we can focus on the alpha map question here and leave Transparent BSDF for another thread, it should be totally possible to support separate Alpha Map textures without it.

In any case where the Base Color texture doesn't included alpha, say, when using a tiled base texture and want to map alpha to that texture without creating more faces, ...

The big caveat here is that if your base color is tiled and alpha is not, or vice versa, merging means baking the tiling and wasting a lot of texture space. Dealing with UV transforms when merging textures is probably out of scope for this exporter, but...
... I would totally support having the exporter be able to merge color and alpha channels of two textures into a single base color texture. That seems analogous to what it needs to do already for occlusion/roughness/metalness, and presumably can rely on mostly the same code.

The big caveat here is that if your base color is tiled and alpha is not, or vice versa, merging means baking the tiling and wasting a lot of texture space. Dealing with UV transforms when merging textures is probably out of scope for this exporter, but...

Ah, yes, Fair point. I can see that potentially being an issue, especially since they need to share the same UVs/transforms, if I understand correctly.

... I would totally support having the exporter be able to merge color and alpha channels of two textures into a single base color texture. That seems analogous to what it needs to do already for occlusion/roughness/metalness, and presumably can rely on mostly the same code.

Cool. Yeah, that was my thinking, to combine them similarly to how the ORM texture is handled. I think this would be really useful in cases where they're provided separately. It's likely a matter of convenience, but a big one at that; having to combine them elsewhere and re-import them is a bit tedious for something that can likely be done automatically within Blender/exporter.
Thanks for your insight, @donmccurdy.

@donmccurdy It appears I may have been wrong about improvements in regard to Principled BSDF shader's Alpha preview in Blender viewport, unless there has been a recent regression. I was using the Background shader, which previews as expected, but still not the case for Principled BSDF. Sorry for the confusion.

This does not work for me!
@scurest I tried like in your screenshot but no texture is exported for me.
My shader setup:

Can you please verify if it works for you?
Model link :
test-alpha-map.zip
Using blender 2.81
Tested with addon version 1.2.17 and 1.2.33 (installed manually)
Regards

Never mind, I solved it. Problem was that alpha map size was different from the color map.

b36_b41_color_alpha_combine_tests.zip
Hey, sorry for reviving this, but I noticed some incosistent behaviour between versions. I am currently using Blender 4.1 and the color/alpha texture combination only works if the alpha texture includes an alpha channel, even if you use it's color output for alpha. On the other hands, while testing this more, I noticed that in Blender 3.6 it only works if the alpha texture DOES NOT have an alpha channel - the exact opposite of 4.1. I have included a zip file with .blend and .glb files for both occasions.

Hello,
This should be fixed in 4.2
A beta build will be available today or tomorrow.
Please test with 4.2 beta, and open a new ticket if any issue


can't convert "leaf-list" with different dimensional arrays

Hi all,
I am currently trying to use the uproot to convert my root/tree data files to NumPy or Pandas data format. My data structure looks like this:

And I tried using the “array” to convert the anaToF to an array, but it didn’t work and gave me this error message:

How can I deal with it?

This is failing because the leaves in this branch all have different numbers of dimensions. There isn't a good error message telling you that because I didn't even know it was possible and hence, didn't write one.
More fundamentally, this is a mismatch between ROOT's data model and NumPy's: ROOT's leaf-list describes contiguous data, data that can't be read separately because it's interleaved, as does NumPy's record arrays, so we convert leaf-lists into record arrays. However, ROOT apparently allows each element of the leaf-list to have its own dimensions, but NumPy shapes contiguous data into multiple dimensions after fixing its dtype (where the fields of a record array are defined).
As for how you can deal with it, if you made this file by serializing a C struct, try turning on ROOT's splitlevel feature. This should put each leaf in a separate branch. Not only would they not all be glued together in one NumPy array (where you have problems with not being able to give each field a different number of dimensions), but you'd be able to read them separately. Splitting data is good for offline analysis in many ways.

I'm assuming that I can close this. Let me know if I'm wrong.


remove template variables from apache license appendix

I believe the appendix is actually supposed to have the brackets there and no year+name substitution already done.
I think it might also be okay to omit the appendix? But better not if github includes it by default
Close #428

Thanks! Didn't realize this was still there.


[FR] Automatic pasting into note after copying

This feature would be amazing: If a text is highlighted with a marker, the text will be automatically inserted in the corresponding annotation note file. This corresponding annotation file is glued to the PDF file. So when opening the PDF, the note with the glued annotations will open as well in an adjacent pane.

How should the pasting position be determined? Sorted by the order in which the annotated texts appear in the PDF file?

It's possible to add this if it suffices to just append copied links one by one at the end of the associated markdown file.

Yes, thank you very much, that would be wonderful.

@N3C2L I'm wondering how a markdown file should be associated with a PDF file.
Currently, I have two options:

Specify a PDF via a property, like so

---
PDF: "[[file.pdf]]"
---

When you copy a link for the first time, you need to paste it manually to a markdown file. Then, PDF++ remembers the markdown file, and the next time on, it automatically pastes links to that markdown file.

The first option is adopted by many tools like the Annotator plugin, Zotero-related plugins, Logseq, ... and it will allow us to paste links even when the markdown is not opened.
However, a drawback is that it won't allow us to auto-paste links to other markdown files, i.e. we can use this feature for only one note per PDF file.
On the other hand, the second option will allow us to flexibly change which markdown to target.
It will be also useful when we want to put annotations about various PDF files in a single markdown file.
Its drawbacks include:

we have to manually paste links at the first time
we need to open the markdown file that links are pasted to

What's your thoughts?

I think the second option would be more versatile.
Let's say my e-book has 20 chapters. I decide to read Chapter 12 and open the corresponding note file for Chapter 12 of that book. If later I want to read Chapter 8, I open the note file corresponding to Chapter 8 of the book. The advantage here is that you always have your automatically inserted notes in the right place, matching each chapter of the book.
This wouldn't be possible with option one. In this case, the text would always be automatically inserted at the end of the note file, regardless of which chapter you are currently on. This would result in the links from the automatically inserted notes being mixed up, sometimes Chapter 8, then Chapter 12, depending on where you read and marked.
There could also be an additional feature in the second option where the last used note file is always opened together with the PDF.
But perhaps in this second option, there might still be a way to skip the manual insertion of links at the beginning? For example, when the automatic insertion function is activated, a modal popup opens where you select the note file into which the notes will be automatically inserted.

Thanks for the feedback!

But perhaps in this second option, there might still be a way to skip the manual insertion of links at the beginning? For example, when the automatic insertion function is activated, a modal popup opens where you select the note file into which the notes will be automatically inserted.

I see, but when you want to auto-paste, the corresponding markdown file is usually opened, right?
Then, a manual paste will not be that tedious IMO.

It's now released as the "Copy & auto-paste link to selection or annotation" command in 0.26.0.
See the "Copying links via hotkeys" section in the plugin setting for the details.
I'm closing this for now, but feel free to re-open it or create a new issue if you want propose improvements including the manual paste thing.

Thank you very much for the implementation!
There is a button in your plugin: "toggle select text to copy mode". May I ask you to implement another button with the function "toggle select text to copy & paste mode"? When this button is enabled, the highlighted text will be copied & pasted in the annotation file. I really would need this function, so that I don't have to push another button, only highlight the text with the mouse.
I would be very glad about this addition.

Hmm. Sorry, can't reopen this issue by myself. Because if the repository owner closes an issue, a normal user like me isn't allowed to reopen the issue.

It's also possible, but isn't it a bit annoying if a link is appended even when you make a mistake in selecting an appropriate range of text?
For example, suppose you want to select the word "Hello" but mistakenly select "Hell" only.
If either auto-paste or select-to-copy is disabled, you can avoid the failure link to "Hell" to be pasted into your note right away. However, if both of them are enabled, you will have to do manual deletion every time you make a mistake in text selection, which will be distracting.
So it seems to me that at least one of the auto-paste and select-to-copy is disabled. What do you think?

It's also possible, but isn't it a bit annoying if a link is appended even when you make a mistake in selecting an appropriate range of text?

Could happen sometimes, yes, But with a double click on the highlighted text you will jump to the appended note and can delete the mistake there by deleting the linked text.

So it seems to me that it's important that at least one of the auto-paste and select-to-copy is disabled. What do you think?

Yes, when one button is active, the other button should be deactivated. Otherwise, it would be confusing as to which function is currently active.

Closing this as merged into #33.


[added]: pass overlay and content element references to onAfterOpen fn

Fixes #739 .
Changes proposed:

Pass overlay and content references to onAfterOpen prop fn

Acceptance Checklist:

[x] The commit message follows the guidelines in CONTRIBUTING.md.
[ ] Documentation (README.md) and examples have been updated as needed.
[x] If this is a code change, a spec testing the functionality has been added.
[ ] If the commit message has [changed] or [removed], there is an upgrade path above.

Coverage remained the same at 86.621% when pulling 78447d5e16b46abfda5648089d926af76ca1365c on baruchvlz:feat/reference-after-open into a2838bbfd11e6203ffb7df1b76575da328d0d3fd on reactjs:master.

Hi @baruchvlz, you can access the overlay and content from the modal reference. Can you provide an example showing where this can be helpful?

@diasbruno To be honest with you, I've never used this library. Just saw that #739 was an open request and I figured, hey couldn't bother to have a hook for this.
Obviously, the best practice would be to take of this with references since it's already provided by React and it makes for cleaner code, now more so with hooks.
With this said, I don't have an example of why this would be a useful addition to the library nor why it should be added.

Oh, of course. Totally missed the reference to the issue.
Any other approach to this problem? Like adding to the list of props...
I'll try to find some time to review this.

@diasbruno hey, sorry to disturb you. Is this pull request going to be merged?

Thanks for ping me, @JamesJGoodwin.


Support for 9.0?

Adding this to the requirements.txt for the Docker compose  onbuild fails with error:
Could not find xmlsec1 config. Are libxmlsec1-dev and pkg-config installed?
So my question is, does this work with 9.0? If so, what am I doing wrong.
I'm adding the master.zip to requirements.txt and running docker-compose build. I also tried to setup sentry on an ubunutu server and I ran pip install https://github.com/getsentry/sentry-auth-saml2/archive/master.zip  with the same error.

Hi @MikeBairdRocks, the docker image is missing the libxmlsec1-dev and pkg-config package. I've added it to our docker build here https://github.com/getsentry/docker-sentry/pull/147 so it should now work once the docker image rebuilds.


get location input to work for create/edit profile

What does this pull request aim to achieve? 💯
This will get the location input working as before for Create User Profile. It also is working with Edit Account:

This is not ideal use of react & react-hook-form but we can refactor later and I we need these working so people can end-to-end test.
But, there is another issue I realized about react-hook-form. The other places using location filter, the onboarding & feed filter - I don't they they are using react-hook-form anyways so we might need to wrap the address input (in a controller?) anyways so it can be used with/without react-hook-form.
@indirabhowmick if your husband has any suggestions based on his experience with react here is the item to look at.
Please be concise and link any related issue(s):
🚨Before submitting this pull request🚨:
Please do NOT submit this PR if you have not done the following:

I have checked that no one else is working on similar changes.
I have read the latest README and understand the development workflow.
The name of this branch is: feature/<branch_name> (need to have cloned the repo not forked).
This branch is rebased/merged with the latest master.
There are no merge conflicts.
There are no console warnings and errors.
On the right hand side of this PR, I have linked the appropriate issue(s) under "Issues" and added a project progress under "Projects".

I'm going to go ahead and merge so other e2e stuff can continue. I'll work on improving this tomorrow morning.


4.5.1: Build hangs on artifactoryPublish task for a WAR

Version 4.5.1: Build hangs on artifactoryPublish task for a WAR.
Works with 4.5.0.

@isobretatel,
I was not able to reproduce this issue.
Can you pleas share your build log?
Which gradle version are you using?
Are you using the --parallel gradle option when running your build?
Additionally, will you be able to share a sample project through github, which demonstrates this?

Sorry, I cannot.
Gradle 3.5.

Are you using the --parallel gradle option when running your build?

No.

We have a large WAR file (360MB).
Not sure if that is related.


Passing no-sandbox but still getting error 'Running as root without --no-sandbox is not supported.`

I was able to get it working using the node user and setting up a sandbox using --cap-add=SYS_ADMIN but AWS ECS Fargate does not support adding SYS_ADMIN as a  linux parameter. Therefore, I am trying to pass no--sandbox to puppeteer so I can run as root but am still getting the error Running as root without --no-sandbox is not supported.

# Install app dependencies
RUN npm install

# Tried running as node user
#USER node

# Provide google authentication credentials to your application code
ENV GOOGLE_APPLICATION_CREDENTIALS=/usr/src/app/google/keys.json

CMD [ "npm", "start" ]

Puppeteer.js
const puppeteer = require('puppeteer');

/**
 * Initializes and returns a puppeteer instance
 * @name {getPuppeteerInstance}
 * @returns {Promise} resolves with puppeteer instance
 */
module.exports = async () => {
  try {
    const options = {
      headless: true,
      args: ['--no-sandbox', '--disable-setuid-sandbox'],
      defaultViewport: {
        width: 1440,
        height: 900,
      },
      timeout: 0,  // 0ms timeout is no timeout
    };
    const browser = await puppeteer.launch(options);
    const page = await browser.newPage(); ``

    page.setDefaultTimeout(0);
    page.setDefaultNavigationTimeout(0);

    return { browser, page };
  } catch (error) {
    console.log(`error`, error);

    return {};
  }
};

What is the expected result?
By passing --no-sandbox I would expect that would be able to launch puppeteer as root without setting up a sandbox with no issues.
What happens instead?
error Error: Failed to launch chrome!
blackbox-app | [0113/220603.530554:ERROR:zygote_host_impl_linux.cc(89)] Running as root without --no-sandbox is not supported. See https://crbug.com/638180.

Same here

For me too

Reopening the issue. It still persist


Support block operation

WIP

Codecov Report

Merging #56 into dev-0.2 will decrease coverage by -1.91%.
The diff coverage is 47.05%.

@@             Coverage Diff             @@
##           dev-0.2      #56      +/-   ##
===========================================
- Coverage      100%   98.09%   -1.91%
===========================================
  Files           18       18
  Lines          669      684      +15
===========================================
+ Hits           669      671       +2
- Misses           0       13      +13

Impacted Files
Coverage Δ

pila/stack.go
100% <100%> (ø)
:white_check_mark:

pilad/conn.go
94.29% <40%> (-5.71%)
:x:

Continue to review full report at Codecov.

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


[C++] Fix pattern consumer's deadlock for topics auto discovery

Fixes #7168
Motivation
When a pattern consumer is blocked by receive(), the mutex_ will be held until new messages arrived. If the auto discovery timer task found new topics and tried to subscribe them, mutex_ must be acquired first, then the deadlock happened.
Modifications

Release the mutex_ after the consumer's state was verified.
Change unit tests to verify that new topics could be subscribed when the consumer's blocked by receive(Message&) or receive(Message&, int) methods.

Verifying this change

[ ] Make sure that the change passes the CI checks.

This change is already covered by existing tests, such as BasicEndToEndTest.testPatternMultiTopicsConsumerAutoDiscovery.

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks


key_max_age check

https://github.com/GoogleCloudPlatform/keyrotator/blob/fdd7448f0e021056169cf7461c647971aa33ab19/keyrotator/cleanup.py#L57
Would it make more sense to use ">=" as opposed to ">" when comparing the difference in key time vs. max age?
I have a process which automatically rotates keys daily, and so I am using "--key-max-age 1", only keys are not deleted until after 2 days.  Today is May 2, and using a max age of 1 was failing to delete keys created on April 30.  I have to use a max age of 0 to rotate keys daily.

Thanks for opening the issue @NFollett89 . I'll get started on making this change as the issue you raise makes sense.


TODO: Come up with a better Ip address system

As of right now the local IP is hard coded. Either refactor to take this as an argument or grab it from the system

I'm thinking create a config.rs module... handle the config in there exposing a global config const from a config struct.
Using config-rs if you want a config file/ env vars driven configuration or clap for command line args driven. Personally I prefer config-rs. Let me know your thoughts, I'll see if I can PR it at the weekend.

I'm thinking create a config.rs module... handle the config in there exposing a global config const from a config struct.
Using config-rs if you want a config file/ env vars driven configuration or clap for command line args driven. Personally I prefer config-rs. Let me know your thoughts, I'll see if I can PR it at the weekend.

I'm thinking create a config.rs module... handle the config in there exposing a global config const from a config struct.
Using config-rs if you want a config file/ env vars driven configuration or clap for command line args driven. Personally I prefer config-rs. Let me know your thoughts, I'll see if I can PR it at the weekend.

config-rs would be nice for the stream key as well. It just needs to be configurable from docker as well

I'm thinking create a config.rs module... handle the config in there exposing a global config const from a config struct.
Using config-rs if you want a config file/ env vars driven configuration or clap for command line args driven. Personally I prefer config-rs. Let me know your thoughts, I'll see if I can PR it at the weekend.

config-rs would be nice for the stream key as well. It just needs to be configurable from docker as well

Closing as this has been remedied by this commit! @aberrios85 I'm sorry you didn't get to take a crack at this one but that you for being willing and there will definitely be more things in the future!

Closing as this has been remedied by this commit! @aberrios85 I'm sorry you didn't get to take a crack at this one but that you for being willing and there will definitely be more things in the future!

Closing as this has been remedied by this commit! @aberrios85 I'm sorry you didn't get to take a crack at this one but that you for being willing and there will definitely be more things in the future!

Closing as this has been remedied by this commit! @aberrios85 I'm sorry you didn't get to take a crack at this one but that you for being willing and there will definitely be more things in the future!


for #211: fix deb package dependency on libasound2

Hi,
Thanks for this PR, I will merge it.
Best Regards,
Michel.


Replace homepage "Find a job" promo

With a new "Right to Rent" promo.
Requested by Jon Sanger.

I'm waiting for confirmation from Jon that this can be merged and deployed.

I've received confirmation that this can be merged.