Double photoelectron emission modeling

This implements double photoelectron emission (dpe) modeling using a binomial process instead of the previous gaussian approximation making the model more realistic for low energy S1s.
The probability of the number of dpe's is modeled as: n_pe - n_photons ~ Binom(n_photons, p=p_dpe) where n_pe is the number of photoelectrons detected, n_photons is the number of photons detected and p_dpe is de dpe fraction.
To implement this an extra tensor of shape (n_events, n_photoelectron detected, n_photon detected) is added to the core flamedisx model and an extra inner dimension photoelectron_detected is added. Bounds estimation and simulation is added as well.
The changes have been validated using the models_check notebook
This implements a solution for issue #68

Thanks for reviewing and further improvements! Looks good to me, merging.


Initial site

http://cibox.tools static site content

@podarok Do we really need this in CIBox repo? I this we should use another repository.


Cannot remove unused config nodes

The problem
When the package of a Node-RED node does not installed any more, but it has saved configuration node for a flow, then flow execution is stopped with the message below.

Flows stopped due to missing node types. Check logs for details.

The solution
If I select Configuration nodes from main menu I can see the unused node, but cannot remove it to make the flow operational again.
I would like to see a way of removing configuration nodes regardless of it's state.
Example

Environment
Node-RED version: 0.17.5
NodeJs version: 6.11.3

Dbl click on it to open its edit dialog then click delete.
We have an item on the backlog for improving the general usability of the config sidebar.

The tab however remains open - even if you delete the config file

Wondering if it safe to remove a hidden node in the attached example?
This was a node I used in the flow and I found out it was the wrong one so I deleted it.  Now it appears as a hidden node.

Yes - if it is used 0 times, it is safe to delete.

@cranksV477 do not use closed issues to ask general questions, please use Slack, Stack Overflow or the forum.
As Dave said, yes it is safe to remove

Thanks gents!  Point noted hardillb.  Apologies!


IPC::System::Simple missing from non-standard module list

Docs should add this module..

hi @bsdzilla. I just double-checked (because I'm forgetful), but I don't see any place in my code where that's used. Could you explain?

./git-hub
IPC::System::Simple required for Fatalised/autodying system() at ./git-hub line 12.
main::BEGIN() called at ./git-hub line 12
eval {...} called at ./git-hub line 12
BEGIN failed--compilation aborted at ./git-hub line 12.
It does not get pulled in via the other dependencies you mentioned..

Thank you. It's now added to the README.md.


[PE-D][Tester C] Unable to View Important Details during Resize

Able to resize the application window to the point where the important fields and contact details are all truncated and unable to be viewed. This is important since users may want to resize the window to suit their operations or screen size.

Labels: severity.Low type.FeatureFlaw
original: kvernnn/ped#15

If we make it smaller we only see the like the categories we want if we change to what you suggest. So whats the use.


[staging-next] go_1_21: 1.21.3 -> 1.21.4

Description of changes

go_1_21: 1.21.3 -> 1.21.4
Release notes: https://go.dev/doc/devel/release#go1.21
Issues that comprise the milestone

Going off of the discussion on #259329 it looks like targeting staging-next might be preferable here, but happy to revise!

Things done

Built on platform(s)

[ ] x86_64-linux
[x] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

That's security issues only for parsing Windows paths?  I.e. I assume only relevant for executables running on Windows?
If so, maybe nixpkgs go can be used that way, but surely that's not a major use case.  Therefore I'd rather target just staging, as I wouldn't see that as urgent.


feat(gui): replace material tailwind with shadcn, and support dark theme

I have went ahead and migrated every material tailwind component to the equivalent shadcn components. There are certain material tailwind components that I migrated to regular html elements such as material's Typography, ButtonGroup, NavBar, Spinner, and List components due to shadcn not having an equivalent.
One note is that shadcn's Radio component is currently untested as I was not able to figure out how to call the SelectUnityVersionDialog function. Other than that, I've tested the rest of shadcn's components to make sure data was being processed properly. Since this is a big PR, there could be possibilities of some things I've missed.

Before

After

shadcn allowed the project to be initialized with a dark theme template with all the shadcn components supporting the css variables. I've left the current theme to the Zinc theme you can find over at shadcn's site with the exception of modifying the destructive css variable for dark theme since it was too hard to read. To compensate for the blue, green, and yellow colors in the GUI, I've added info for blue, success for green, and warning for yellow. I've avoided explicitly naming the variables after the name of colors for the convenience of having the ability to change the css variables to any color without having to rename the variables. This can be useful for having a feature to load a user's custom css in the future!

Since there needed to be a way to swap between the two themes, I added a dropdown in settings underneath the language settings to swap between light and dark. These settings will persist since I've simply added them to the gui-config.json file appropriately.
However, I am directly interacting with the DOM in order to apply the theme with document.documentElement.setAttribute() as I haven't found a better way to be able to modify the html element on the client side. Let me know if nextjs has a better way of handling something like this!
Toastify's theme doesn't entirely match shadcn's theme and I haven't seen a way to properly configure the toasts, so I've set toastify's theme to colored to match the theme as close as I could.
Closes #704, #601, #431

I found a bug that breaks the displayed package version.

Open Project Management.
Select any package version to install/update, and cancel in the dialog.
The package version appears to have changed even though I canceled it.
Refresh Packages does not change the situation.
Reloading the page restores it

And could you add to changelog?

The hint messages for creating project is too small.
Current:

This PR:

The red "Legacy" is a little hard to see when using dark theme.
It may be necessary to lighten the color a little and make it a little bolder.

Found a bug with the language setting dropdown where the default value will be set to the wrong language. I'll play around with this to see if I can fix it. I can reproduce the bug with the following steps:

Select an item from the dropdown
Change to a different page from the sidebar
Change back to the settings page

2. The red "Legacy" is a little hard to see when using dark theme.
   It may be necessary to lighten the color a little and make it a little bolder.

Let me know if the text needs to be brighter or need to be bold still! I tried my best to gauge the sweet spot for the color.

2. The red "Legacy" is a little hard to see when using dark theme.
   It may be necessary to lighten the color a little and make it a little bolder.

Let me know if the text needs to be brighter or need to be bold still! I tried my best to gauge the sweet spot for the color.

It looks good!

I couldn't try some features so I'll try tomorrow. sorry.

I couldn't try some features so I'll try tomorrow. sorry.

Take your time, no rush!

See changelog button is on the next line. it was same line as package name

The project location and unity version line, and manage packages line is not vertically centered.
The mergin size of search box and buttons might be differ?

Line with both button and input are misaligned. something like margin for input field is wider than buttons?

Thank you for the reviews! Let me know if there's any further changes required otherwise everything looks good to go, and I'm ready for merge whenever you are!


[Feature Request] Dead players speak directly

Hello,
Is there any possibility when a player is dead, He can speak directly with other dead people and not wait for an Emergency call / find body ... then when game starts again he can talk to other.
Thanks.

Duplicate of https://github.com/denverquane/amongusdiscord/issues/83 but yes, this will be in the next update


"No deserializable introspection present for type:" When adding "io.micronaut.mqtt:micronaut-mqttv5" dependency

Expected Behavior
micronaut-serde should not be used at all, since I already have jackson-databind in use
This is the same problem as https://github.com/micronaut-projects/micronaut-problem-json/issues/257
It can be fixed by adding the following to the build.gradle.kts
configurations.all {
  exclude(group = "io.micronaut.serde")
}

Actual Behaviour
The error "No deserializable introspection present for type:" is thrown as soon as anything in the project tries to serialize or deserialize json.
Steps To Reproduce
No response
Environment Information
No response
Example Application
No response
Version
4.0.5

This also applies when using the io.micronaut.mqtt:micronaut-mqttv3 dependency. Builds are broken for clients that use micronaut-jackson-databind instead of micronaut-serde. Only when excluding the transitive micronaut-serde-jackson dependency does it build.

Fixed in MQTT 3.1.1 👍


No proper check for interface type

Bug Report

🔎 Search Terms

🕗 Version & Regression Information

Any version

⏯ Playground Link

Playground link with relevant code
💻 Code

interface IPort {
  notify(data: 'a' | 'b' | 'c' | 'd'): void;
}

class A {
  constructor(public serviceA: IPort) {
    serviceA.notify('a');
  }
}

class ServiceA {
  notify(data:  'b') {

  }
}

const serviceA = new ServiceA();

new A(serviceA);

🙁 Actual behavior
I am not getting the error
🙂 Expected behavior
I expect to get a mismatch error between the type of the expected interface and the actual implementation

https://github.com/Microsoft/TypeScript/wiki/FAQ#why-are-function-parameters-bivariant
Use properties instead of methods.

This works also in common:
Playground
For example here:
type FnX = (val: 'a' | 'b') => void;
type FnY = (val: 'a' | 'b' | 'c') => void;

const x: FnX = (val: 'a' | 'b') => {};

const y2: FnY = x;

We are not able to assign FnX method to FnY.

Type 'FnX' is not assignable to type 'FnY'.
Types of parameters 'val' and 'val' are incompatible.
Type '"a" | "b" | "c"' is not assignable to type '"a" | "b"'.
Type '"c"' is not assignable to type '"a" | "b"'.

because otherwise, we will able to:
 y2('c');

that did not look valid.
But with implementing interface in class we able to use a subset of parameter type (instead of same type or superset) without any error:
interface ISome {
  create(variant: 'a' | 'b' | 'c'): void;
}

class Some implements ISome {
  create(variant: 'a' | 'b') { }
}

const some: ISome = new Some();
some.create('c');

This looks not valid from my point of view. I guess there should be caused an error in the implementation stage or on assigning Some to ISome.

As mentioned before, this is working as intended and a known unsoundness. You will have the same behaviour with functions if you disable strictFunctionTypes.
If you use properties instead you have the behaviour you want:
interface ISome {
  create: (variant: 'a' | 'b' | 'c') => void;
}

class Some implements ISome {
  create: (variant: 'a' | 'b') => { }
}


Handle opts.cwd correctly when using lintText

@jcorbin @Matt-Esch @malandrew @Raynos

@lucasmarshall please look at how feross/standard solves this issue (I believe they had this bug too and resolved it) and either cherry-pick that commit or manually apply those changes and submit as a new PR.
https://github.com/feross/standard/blob/master/index.js

@malandrew Looks like feross/standard doesn't have this particular issue fixed, actually. I just tested the latest from their master and it definitely doesn't work as expected.

Can you give me more details on the issue this is fixing and how to reproduce it?

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


Multi address family support

I know the changes are huge but give it some consideration...
Would it make sense to filter iface.ip() for the address family of the socket that received a query?

Thanks, I generally like how this looks.
I made some comments, mostly just not picking.
One issue is that registering a service can in theory fail (though because the implementation is not fully compliant is does not currently).
With this new design, the responder cannot notify the caller of this, whereas the future could have be used for this.
This is not a real blocker, just something to consider.
Also I'm not sure I get your question.

Thank you very much for the useful feedback. My commits address it all.

One issue is that registering a service can in theory fail (though because the implementation is not fully compliant is does not currently).
With this new design, the responder cannot notify the caller of this, whereas the future could have be used for this.
This is not a real blocker, just something to consider.

There still are the txs_notifiers to communicate with the FSM threads. register() could block until it receives the reply to a future Command::SendUnsolicited(mpsc::Sender<bool>)

Also I'm not sure I get your question.

We get the local host's addresses from iface.ip() - but does it make sense to include its IPv4 addresses when sending over IPv6 and vice versa?

There still are the txs_notifiers to communicate with the FSM threads. register() could block until it receives the reply to a future Command::SendUnsolicited(mpsc::Sender)

"Send unsolicited" is too low level to deal with problems like name collision and al. In theory, we would need to send out a request and wait some time to ensure the name isn't already in use. Also even after registering, the responder has to send a few more unsolicited packets.
All this requires the responder logic to be in the FSM rather than the main thread.
But like I said, none of it is implemented, so there's no need to worry about it for now.

We get the local host's addresses from iface.ip() - but does it make sense to include its IPv4 addresses when sending over IPv6 and vice versa?
I don't think how the packet was received should have any impact on the responses sent.

Merging, thanks !


PluginLoader shouldn't log stuff

The caller is perfectly capable of doing their own logging before and after calling it if they desire logging
(double-check everything else to see if anything else is doing redundant logging as well)

Fixed in 1.17.x / Databuddy 2


Add a "happy to help out" in Contributing.md

Not sure if this is the best place to put it, but thought we might want to add something like this. Especially this sentence seemed a little "strict" to me, but I think it shouldn't scare people away. 😃
Thoughts, @jessesquires?

good catch @BasThomas !
I added a bit more. 😊

👍


fix(styling): Viewing Room Artwork spacing

The type of this PR is: fix

This PR resolves CX-938
Description

before

after

PR Checklist (tick all before merging)

[x] I have included screenshots or videos to illustrate my changes, or I have not changed anything that impacts the UI.
[x] I have tested my changes on iOS and Android.
[x] I have added tests for my changes, or my changes don't require testing, or I have included a link to a separate Jira ticket covering the tests.
[x] I have added a feature flag, or my changes don't require a feature flag. (How do I add one?)
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added an app state migration, or my changes do not require one. (What are migrations?)
[x] I have added a CHANGELOG.yml entry or my changes do not require one.

Changelog updates
Changelog updates

Cross-platform user-facing changes

iOS user-facing changes

Fix spacing on Viewing Room Artworks - pavlos

Android user-facing changes

Dev changes

#squashongreen


fix(seed-db): preprod

:tada: This PR is included in version 1.94.4 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Some notes about current master branch

Hi Team,
I wanted to share some comments about some recipes on current master branch, tested on Yocto Sumo:

libguvc/gst-uvc-sink has some warnings/errors related to AUTOREV

WARNING: /home/gfallas/work/devdirs/customers/ridgerun/yocto-imx6/sources/meta-ridgerun/recipes-multimedia/libguvc/libguvc_1.4.0.bb: Exception during build_dependencies for AUTOREV
WARNING: /home/gfallas/work/devdirs/customers/ridgerun/yocto-imx6/sources/meta-ridgerun/recipes-multimedia/libguvc/libguvc_1.4.0.bb: Error during finalise of /home/gfallas/work/devdirs/customers/ridgerun/yocto-imx6/sources/meta-ridgerun/recipes-multimedia/libguvc/libguvc_1.4.0.bb
WARNING: /home/gfallas/work/devdirs/customers/ridgerun/yocto-imx6/sources/meta-ridgerun/recipes-multimedia/gstreamer/gst-uvc-sink_1.0.bb: Exception during build_dependencies for AUTOREV
WARNING: /home/gfallas/work/devdirs/customers/ridgerun/yocto-imx6/sources/meta-ridgerun/recipes-multimedia/gstreamer/gst-uvc-sink_1.0.bb: Error during finalise of /home/gfallas/work/devdirs/customers/ridgerun/yocto-imx6/sources/meta-ridgerun/recipes-multimedia/gstreamer/gst-uvc-sink_1.0.bb
ERROR: ExpansionError during parsing /home/gfallas/work/devdirs/customers/ridgerun/yocto-imx6/sources/meta-ridgerun/recipes-multimedia/gstreamer/gst-uvc-sink_1.0.bb

GstPerf requires a license file in the repo and a couple of changes on the recipe

Error related to missing license

ERROR: gst-perf-1.0-r0 do_populate_lic: QA Issue: gst-perf: Recipe file fetches files and does not have license file information (LIC_FILES_CHKSUM) [license-checksum]
ERROR: gst-perf-1.0-r0 do_populate_lic: Fatal QA errors found, failing task.
ERROR: gst-perf-1.0-r0 do_populate_lic: Function failed: populate_lic_qa_checksum
ERROR: Logfile of failure stored in: /home/gfallas/work/devdirs/customers/ridgerun/yocto-imx6/build_x11/tmp/work/armv7at2hf-neon-mx6qdl-fslc-linux-gnueabi/gst-perf/1.0-r0/temp/log.do_populate_lic.20675
ERROR: Task (/home/gfallas/work/devdirs/customers/ridgerun/yocto-imx6/sources/meta-ridgerun/recipes-multimedia/gstreamer/gst-perf_1.0.bb:do_populate_lic) failed with exit code '1'

I tested adding the same recipe used by GstQtOverlay in the sources of GstPerf, I modified the source gst-perf.bb as follows:

SUMMARY = "GStreamer Perf 1.0"
DESCRIPTION = "GStreamer element to measure fps and performance"
HOMEPAGE = "https://github.com/RidgeRun/gst-perf-autotools"
SECTION = "multimedia"
LICENSE = "LGPLv2"
LIC_FILES_CHKSUM = "file://COPYING;md5=2282fc857e14ce7b17c1c9d810504ac5"
DEPENDS = "gstreamer1.0 gstreamer1.0-plugins-base gstreamer1.0-plugins-bad"
SRCBRANCH ?= "master"
SRCREV = "939eee167c6e95ed1c2f2013246b1f0b671a378b"
SRC_URI = "git://github.com/RidgeRun/gst-perf.git;protocol=https;branch=${SRCBRANCH}"
S = "${WORKDIR}/git/"
INSANE_SKIP_${PN} = "dev-so"
FILES_${PN} += "${libdir}/gstreamer-1.0/libgstperf.so
${libdir}/gstreamer-1.0/libgstperf.so.0
${libdir}/gstreamer-1.0/libgstperf.so.0.0.0
"
inherit autotools pkgconfig gettext
do_configure() {
${S}/autogen.sh
oe_runconf
}

Changes:
1- Added the license section
2- Adjusted 'S' to correct folder

Regarding libguvc and uvc-sink, the problem is that the URI is pointing to an invalid URL and that makes AUTOREV to fail. Once SRC_URI has the correct value the error should disappear.
Also, I would suggest to use the yocto-sumo branch when working with sumo.

This uvc error causes your readme step to fail. https://github.com/RidgeRun/meta-ridgerun#building-gst-shark-and-gstreamer-deamon
I ended up reverting to origin/feature/add-gst-motiondetect-recipe since this was right before uvc was added.

This uvc error causes your readme step to fail. https://github.com/RidgeRun/meta-ridgerun#building-gst-shark-and-gstreamer-deamon
I ended up reverting to origin/feature/add-gst-motiondetect-recipe since this was right before uvc was added.


Update netlink library to support 4.20+ kernels

This PR updates the netlink library to add a patch required for kernel 4.20+
See this PR for more detail:  https://github.com/vishvananda/netlink/pull/498
Symptoms of the problem:

kubelet updating node status to NotReady because runtime is considered unresponsive. Log message: "container runtime is down"
runtime is failing the CRI Status call because of CNI (can be checked with crictl info)
CNI delete calls fail (can be checked with cnitool)

Closing for now as I've found a regression in another call
Will reopen when I'm sure I have a patch

Thanks. We are running this on 5.5/5.6/5.7 kernels with no issues so curious what you end up seeing.

@theatrus interesting. For us Delete CNI calls fail on kernel 5.3 with the delete code from v0.6.1
Steps to reproduce:
sudo ip netns add testing
sudo CNI_PATH=/opt/cni/bin cnitool add cni-ipvlan-vpc-k8s /var/run/netns/testing
sudo CNI_PATH=/opt/cni/bin cnitool del cni-ipvlan-vpc-k8s /var/run/netns/testing

The del command hangs and never returns (behavior actually depends on the exact go version used for compilation which I found very misleading until I saw the patch from Daniel which fixes a stack corruption in vethPeerIndex (https://github.com/lyft/cni-ipvlan-vpc-k8s/blob/v0.6.1/plugin/unnumbered-ptp/unnumbered-ptp.go#L579)
Note that the version we run does not include https://github.com/lyft/cni-ipvlan-vpc-k8s/pull/83
Which means its calls the buggy netlink.vethPeerIndex which master doesn't use anymore (see: https://github.com/containernetworking/plugins/blob/master/pkg/ip/link_linux.go#L258).
So I think the issue is only present in versions up to v0.6.1 (included).
We are currently testing the netlink update with an additional small patch which I can PR (upgrading the library should be good in any case as we all move to much more recent kernels)


Broken in latest update

Maps fail to load when trying to use it in the latest release (2020.923.0)

Known, will look at it today.

Please update to Rush! 2020.924.0 and let me know if you have any more problems.


fix 17616 - makeIndex cannot fully use range of index type

Would look clearer and better as:
foreach(size_t i; 0 .. r.length)

I just saw this right now & appended it to the PR as well. FWIW editing PRs is really awesome and saves a lot of time :)
Btw thanks for being so fast.

Thanks for making the changes, and adding the uint unit test, looks great!


Consider adding Verifier::new_with_extra_roots implementation to other platforms

The functionality of new_with_extra_roots is primarily useful for Linux/WASM/BSD platforms that don't have a consistent source of trusted CA root/anchors available. However, many private/internal applications often use their own private CAs instead of publicly issued ones. This seems like a use case we could support without much burden, even if those users might be better off making their own webpki-based verifier instead.
Implementation details
It's worth noting that the Apple and Windows code for this already exists in a near-drop in form. Android would require more work to make a TrustManager that combined certificates.
macOS/iOS
We can call SecTrustSetAnchorCertificates to add additional roots to the evaluation, and then call SecTrustSetAnchorCertificatesOnly with false to trust both the custom roots and the default OS-provided ones.
Windows
We can create a custom CERT_CHAIN_ENGINE_CONFIG and set cAdditionalStore to a custom, in-memory certificate store. This engine and store are then included in our call to CertGetCertificateChain.
Android
I believe this can be done by using a PKIXParameters. I think the best idea is to create a custom TrustManager class that considers the system Keystore's trustmanager and then a custom Keystore containing the user-provided roots. I'm not yet sure about the exact implementation strategy though since the Android X509 and PKIX APIs are a handful. This and this blog may be helpful references.

I second that. This also makes development easier since when using MacOS I've spent some time figuring out why the new_with_extra_roots method is missing :)

Note that this is blocking (built-in) support for rustls-platform-verifier in reqwest: https://github.com/seanmonstar/reqwest/pull/2286#discussion_r1609846678.

@stormshield-gt Based on https://github.com/rustls/rustls-native-certs/issues/3#issuecomment-2298675787, I believe this is the issue would be good to help with to help towards reqwest integration.
If you'd like to take my existing work in https://github.com/rustls/rustls-platform-verifier/tree/shared/extra-roots-additions and update it to use the new types provided in https://github.com/rustls/webpki-roots/pull/75, that would make progress on the iOS side (as macOS and iOS share a verifier impl).

Thanks for the hint, I will start here so.

I'm adding tests for the Verifier::new_extra_roots and I wonder if I should just replace the existing tests that override one CA. Does it make sense to keep them both ?

This is something I was also thinking about when working on the branch and I was undecided 😅. If its cleaner with tests to merge together the mock CA and extra roots, go ahead and do so.
The one important part we need to keep around though is the ability to tell the system verifier to only use those roots during testing, and not consider the wider system state. This isn't wanted in production but during testing its very valuable to keep things reproducible given the background-managed nature of trust roots on platforms.

I think we should reopen this while we don't support android and windows

I wonder if we should consider adding an option to configure if the extra roots must be exclusive or not. I mean if we should consider only the extra root or the extra root and the OS root store.
Reqwest has this flag in its client builder and if we want to integrate with it, I think we want to expose this functionality.
For instance, on MacOS, when reqwest use the native-tls feature, this flag is just propagated to secure transport crate which is then just used as an argument to the call of the SecTrustSetAnchorCertificatesOnly function.
We could easily replicate this in our implementation. Besides it will remove the need for having special field inside the Verifier only for testing.

It seems like an edge case, honestly, but maybe we do need to support it for reqwest to be happy.
IMO that flag doesn't make much sense to have in a platform verifier: why would you use this crate and only custom roots? You could just use rustls on its own with the webpki-based verifier if the roots are known to be good ahead-of-time. Usually the platform verifiers are better at handling "messy" certificate chains because of how much compat work the OSes put in since their code has to "just work."

I see your point in the case you want to use the extra root also on production, I also think you better off using the webpki verifier.
But in a case you use extra roots only for testing, I think it can be valuable to have the same verifier for testing and production, as the behavior might be slightly different.
As for this crate, when doing the test you might want only the extra root for reproducibility.
In any case, as you said, we might just need this for reqwest support.

The callout about end user testing is something I hadn't considered, thanks for mentioning that. I can see how people would want to maybe do similar to us and isolate unit test to an exclusive root store and not let the OS do any certificate fetching on-the-fly etc for reproducibility.

cpu closed this as completed in #135 now

GitHub misunderstood "partially fix 135" as "fix 135" - I believe all that remains now is support for this in the Android verifier.

Has this non-Android functionality been released yet?

No: https://github.com/rustls/rustls-platform-verifier/compare/v/0.3.4...main

I can put up a PR to start the release process. I think the new APIs have sat for long enough in main without anyone finding an obvious limitation right away.

That would be awesome, I'm very excited for reqwest to start using this!

Maybe before publishing we can take a look one more time at this https://github.com/rustls/rustls-platform-verifier/pull/133#discussion_r1740770593 ?
As we now also iterate in the macOS/IOS case, maybe impl IntoIterator<Item = pki_types::CertificateDer<'static>> would be slightly more flexible ?

Maybe before publishing we can take a look one more time at this #133 (comment) ? As we now also iterate in the macOS/IOS case, maybe impl IntoIterator<Item = pki_types::CertificateDer<'static>> would be slightly more flexible ?

Sounds good to me, want to submit a PR?

That's done with https://github.com/rustls/rustls-platform-verifier/pull/145


Is it possible to disable the functionality of the mod for nether portals?

Hello. I want to use this mod for some custom dimensions that I do with datapacks. But I want to keep the nether portal functionality completely vanilla. Is it possible to do this? Perhaps there is some sort of dimension blacklist to make the mod ignore the nether portals... I found this parameter in the config, set it to vanilla, but it doesn't seem to work as it should. When I activate a portal, it is created as vanilla, but after entering it, the purple overlay breaks and the portals become immersive.
#Allowed Values: normal, vanilla, adaptive, disabled  nether_portal_mode = "vanilla"
I am using the latest version of the mod for Forge, Minecraft 1.16.5

The example datapack has one to convert vanilla nether portals. You need to delete that one https://github.com/qouteall/ImmersivePortalsMod/wiki/Datapack-Based-Custom-Portal-Generation#convert_vanilla_nether_portaljson-convent-vanilla-nether-portals-into-see-through-portals-if-the-shapes-are-compatible


Compile fails, shows up with file not found error

I tried compiling this from the source code, but I can't compile it because whenever I try to run the make command, I get this error:
g++ -I"/home/cadendurden/smooth/classes/backends/xlib"/../../../include   -fvisibility=hidden -fPIC -pthread -DSMOOTH_DLL  -c backendxlib.cpp -o backendxlib.o
backendxlib.cpp:11:10: fatal error: gtk/gtk.h: No such file or directory
11 | #include <gtk/gtk.h>
|          ^~~~~~~~~~~
System Specs If Needed:
System Name: Dell Latitude D630
RAM: 2GB DDR2
WiFi Card: Broadcom BCM4311
Operating System: Debian Sid (Originally Debian 12 Bookworm)

You will need to install GTK 3 development packages in order to build smooth.
Additionally, you will need the following development packages:

libbz2
libcurl
libfribidi
libjpeg
libxml2

There are some other dependencies, but those should already be included as dependencies of GTK 3.


Change the default log level from InfoLevel to WarnLevel to fix #3835.

In #3759, it changed pkg/log/log.go to use default logging options which sets to InfoLevel by default.

I'm trying to pick some starter issues to help to ramp up on Istio, please let me know if you think we should not change the default logging level, I'll try to fix the issue in a different way.
Thank you!

/lgtm, but i will defer the approval to Mandar/Martin

@yangminzhu: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
10360f700da991bc89b5e3eb1c27e51d34f21ee1
link
/test istio-presubmit

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

This change is not appropriate, the default log level must stay at Info for the product.
What specific issue is this trying to fix?

This is to fix #3835.

@yangminzhu the proper fix is probably to move the spew to Debug rather than change the default to Warning. That or decide because it's a one time spew it is actually appropriate and useful to see at startup/once
What we want to make sure is we don't log anything at Info for every request.

The solution to #3835  is definitely just to change the simple log statement in mixs and not to change the system's default log level globally.
Thanks.

@geeknoid Thanks for the suggestion!
These logging information are generated from the calling to runtime.KindMap() in mixer/cmd/mixs/cmd/validator.go:39. And it's KindMap() that calls log.Infof() which in turn produces these information. (https://github.com/istio/istio/commit/ea7303f7103a8d8c2f9afa408cae4ef37c2c3739#diff-9476892d25087e7c68a6c9343c536b64R72)
@mandarjog Could you let me know if it's ok to change the log.Infof() in KindMap() to Debug level to fix the issue?
Thanks!

Yes, that's totally appropriate.
On Fri, Mar 2, 2018 at 7:19 PM, Yangmin<EMAIL_ADDRESS>wrote:

@geeknoid https://github.com/geeknoid Thanks for the suggestion!
These logging information are generated from the calling to
runtime.KindMap() in mixer/cmd/mixs/cmd/validator.go:39. And it's
KindMap() that calls log.Infof() which in turn produces these information. (
ea7303f#diff-9476892d25087e7c68a6c9343c536b64R72
https://github.com/istio/istio/commit/ea7303f7103a8d8c2f9afa408cae4ef37c2c3739#diff-9476892d25087e7c68a6c9343c536b64R72
)
@mandarjog https://github.com/mandarjog Could you let me know if it's
ok to change the log.Infof() in KindMap() to Debug level to fix the issue?
Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/pull/3925#issuecomment-370112663, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AVucHes1A0etrskGtMBaJD1dh1iVd2FCks5tagukgaJpZM4SatUq
.

@yangminzhu: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
10360f700da991bc89b5e3eb1c27e51d34f21ee1
link
/test istio-presubmit

prow/istio-unit-tests.sh
caa27ddfd12bfb5df9faf96463335ec369cbb320
link
/test istio-unit-tests

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

@yangminzhu: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
caa27ddfd12bfb5df9faf96463335ec369cbb320
link
/test istio-unit-tests

prow/istio-presubmit.sh
caa27ddfd12bfb5df9faf96463335ec369cbb320
link
/test istio-presubmit

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

/approve

/lgtm

@yangminzhu: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
aae4079409ee1ab7e8f344c4ed617b1762cc5568
link
/test istio-presubmit

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


Unexpected behaviour when registering multiple Model::creating() event callbacks

Laravel Version: 9.18.0
PHP Version: 8.1.6
Database Driver & Version: MySQL 8

Description:
Applications may register several Eloquent model callbacks for the same event. In my app, I'm automatically assigning certain values to models throughout their lifecycle. When registering multiple creating callbacks, they cause an unexpected early INSERT if any of the callbacks return a "truthy" value.
This might be a bug, or it might just be inconsistent behaviour, I'm not sure. Either way it doesn't seem to be documented anywhere, and works the opposite way to how we'd normally stop event propagation. If I register several static::creating() handlers on the same model class, I'd expect them to run sequentially, and for them all to complete, before the model can be considered created.
I was caught out by this because of using PHP's short function syntax to define my callback, as shown in the example below.
Steps To Reproduce:

Create the database schema:

<?php

use Illuminate\Database\Migrations\Migration;
use Illuminate\Database\Schema\Blueprint;
use Illuminate\Support\Facades\Schema;

return new class extends Migration {
    public function up(): void
    {
        Schema::create('my_models', function(Blueprint $table) {
            $table->increments('id');
            $table->string('optional_column')->nullable();
            $table->string('required_column');
            $table->timestamps();
        });
    }
}

Create the model:

<?php

namespace App\Models;

use Illuminate\Database\Eloquent\Model;
use Illuminate\Support\Facades\Log;

class MyModel extends Model
{
    protected $fillable = [
        'optional_column',
        'required_column',
    ];

    protected static function booted(): void
    {
        // 1: Log a message with the Model's initial (empty) attributes when it's being created
        // (This behaves "normally", it logs and continues execution to the next callback as we'd expect.)
        static::creating(static fn(Model $model) => Log::info('Log 1', $model->getAttributes()));

        // 2A: Assign an optional value to the Model, if not already set, using arrow syntax.
        // (🧨 This causes an SQL INSERT error. Returning the model in the callback caused an immediate insert.)
        static::creating(static fn(Model $model) => $model->optional_column === null
            ? $model->optional_column = 'hello world'
            : $model
        );

        // 2B: Assign an optional value to the Model, if not already set, WITHOUT using arrow syntax.
        // (✅ This works as expected. The model attribute is updated and the next callback is handled.)
        static::creating(static function(Model $model) {
            $model->optional_column === null
                ? $model->optional_column = 'hello world'
                : $model;
        });

        // 4: Assign a required value to the Model.
        // After this callback executes, the model will be saved successfully.
        static::creating(static function(Model $model) {
            $model->required_column = 'lorem ipsum';
        });
    }
}

In Artisan, call \App\Models\MyModel::create().

Hey @JackWH, it happens because the creating model event, dispatched inside the performInsert method, is fired using event halting:
https://github.com/laravel/framework/blob/4d46bda6f19d490080512fa053065081f737bf2f/src/Illuminate/Database/Eloquent/Model.php#L1168
This is the signature of fireModelEvent:
https://github.com/laravel/framework/blob/4d46bda6f19d490080512fa053065081f737bf2f/src/Illuminate/Database/Eloquent/Concerns/HasEvents.php#L169
So, when you register a listener for that event calling static::creating, if your listener returns any not null values then the Dispatcher class will instantly return that response, and the performInsert method will try to insert the model to the database.
https://github.com/laravel/framework/blob/4d46bda6f19d490080512fa053065081f737bf2f/src/Illuminate/Events/Dispatcher.php#L254-L256
@driesvints I don't know if you'll consider this a bug, but I kind of agree with @JackWH on this one. I wouldn't expect for the model to be inserted into the database before all the listeners defined for creating (which means before inserting the model) are executed. Please let me know your thoughts on this and I can make a PR if it is indeed a bug.

I also agree I would expect a different behavior, specially now when arrow functions that auto-return a value are more common.
@JackWH in the meanwhile, you could convert your arrow functions to regular closures without a return statement.
The only thing I kind of don´t agree is this would be bug.
It is surely unexpected behavior, but changing this behavior that is around for so long looks more like a feature request, at least in my opinion.
But it would be nice if someone could better explain this behavior on the documentation.
Note all "before" events (the ones ending in -ing, such as creating) are dispatched using Dispatcher@until, which will stop in the first non-null response from a listener, while the "after" events (the ones ending in -ed, such as created) are dispatched using Dispatcher@dispatch which will only halt on the first response equals to false. So using arrow functions on these "after" events is fine, as long they don't explicitly return false.
This difference could be better explained in the docs.

Applications may register several Eloquent model callbacks for the same event.

Afaik we don't document this anywhere and I don't think we support that. What's the reason you can't use a single callback?

@driesvints Traits might want to register their own callbacks in the bootTrait() method. For example:
trait HasUuids
{
    public static function bootHasUuids(): void
    {
        static::creating(static fn(WithUuid $model) => $model->{$model->getUuidField()} = Str::uuid()->toString());
    }

    // ...
}

A quick search of my codebase shows 3 different packages registering their own model callbacks using traits.

Hah, I forgot we had support for that. We should document that really.

It looks like this will require a change in behavior so for now, we're not categorizing this as a bug. A PR to attempt to change this behavior has been opened at master. Let's see how that goes.


Added Square_Root_under_Modulo_p in Number theory

Great work please give this repo a star and follow me for follow back 😊


S3 Buckets support

Any recommendation on how I could set up a connection to a S3 bucket for data catalog in WhereHows?

@rilma Have you got any recommendation or have you tried anything ? Please share your thoughts, thank you.

Dear issue owner,
Thanks for your interest in WhereHows. We have recently announced DataHub which is the rebranding of WhereHows. LinkedIn improved the architecture of WhereHows and rebranded WhereHows into DataHub and replaced its metadata infrastructure in this direction. DataHub is a more advanced and improved metadata management product compared to WhereHows.
Unfortunately, we have to stop supporting WhereHows to better focus on DataHub and offer more help to DataHub users. Therefore, we will drop all issues related to WhereHows and will not accept any contribution for it. Active development for DataHub has already started on datahub branch and will continue to live in there until it's finally merged to master and project is renamed to DataHub.
Please check the datahub branch to get familar with DataHub.
Best,
DataHub team


Macbook m1 chip  php5.6.31 wrong

MacBook m1 build successed，but can't work!

I would suggest to use Google's official implementation which has PHP support for sometime now. php-protobuf won't be developed anymore.

I would suggest to use Google's official implementation which has PHP support for sometime now. php-protobuf won't be developed anymore.


fix(gemini): unthemed home page

🔧 What does this fix? 🔧
as said in the title

🗒 Checklist 🗒

[X] I have read and followed Catppuccin's contributing guidelines.
[X] I have updated the version appropriately in the ==UserStyle== header of the catppuccin.user.css file.

i'll fix it dw


[SPARK-12972] [CORE] Update org.apache.httpcomponents.httpclient

What changes were proposed in this pull request?
(Retry of https://github.com/apache/spark/pull/13049)

update to httpclient 4.5 / httpcore 4.4
remove some defunct exclusions
manage httpmime version to match
update selenium / httpunit to support 4.5 (possible now that Jetty 9 is used)

How was this patch tested?
Jenkins tests. Also, locally running the same test command of one Jenkins profile that failed: mvn -Phadoop-2.6 -Pyarn -Phive -Phive-thriftserver -Pkinesis-asl ...

Test build #58607 has started for PR 13117 at commit bc1720e.

Test build #58607 has finished for PR 13117 at commit bc1720e.

This patch fails build dependency tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/58607/
Test FAILed.

Test build #58608 has started for PR 13117 at commit 1ff05ba.

Test build #58608 has finished for PR 13117 at commit 1ff05ba.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/58608/
Test PASSed.

I'm going to try merging this in master only at first. This again passes the PR builder, and for me, passes locally with the same build config that one previously-failing Jenkins job used. I also have a pretty clear reason to believe the failure was connected to the HtmlUnit / Selenium version. Fingers crossed and apologies in advance if this doesn't fly.

Well, it turns out this is entirely fine, except for the Hadoop 2.2 build:
*** RUN ABORTED ***
  java.lang.NoClassDefFoundError: org/apache/commons/io/Charsets
  at com.gargoylesoftware.htmlunit.util.EncodingSniffer.<clinit>(EncodingSniffer.java:414)
  at com.gargoylesoftware.htmlunit.WebResponse.getContentCharsetOrNull(WebResponse.java:152)
...

I'm going to attempt to fix this forward by investigating and harmonizing the commons-io dependency version, as I kind of suspect this is something that will otherwise bite us later in 2.x. If it's not working, I'll have to revert.


OSF-4265 Fix strange behavior of user settings page expand/collapse caret

Purpose
User settings page's section for configuring addons have strange behaviors about the expand/collapse caret of each addon. (see https://openscience.atlassian.net/browse/OSF-4265)
Changes
@caneruguz 's original PR(https://github.com/CenterForOpenScience/osf.io/pull/4236) on this bug was supposed to deal with the fact that different addon's are loaded differently as

There are two types of data load methods for addons

Those that get added by backend and mako file addon_permissions.mako
-- For these the document.ready function was used to apply
Asyncronous loading with ajax after load
-- For these the initialization was added where nodes have been applied

and, judging from QA's feedback in the ticket, the fix did work at first. The ticket was re-opened during regression testing. I checked the current develop and the changes made in the PR are not showing. It is possible that some later PR reverted the merge.
Thus I replicate the changes made in the PR by Caner. (All such changes are included in the first commit) Somehow the expand/collapse caret's behavior for all addons are still unpredictable on each refresh of the page, and the fix does not seem to work on Safari at all (No carets will show). As QA have noticed, the caret's will show upon resizing the page. Thus, in the "profile-settings-addon-page.js", a line is added to programmatically trigger a resize event of the browser after the page is fully loaded. (The second commit)
Side effects
Safari seems to block the resize event so that the fix still does not work on it. Works on Chrome and Firefox. Need more tests on IEs.
Don't know why the fix in the original PR worked before but not now. Maybe other components related to this bug changed between then and now. And the fix in the second commit is quite hackish I'll admit.
Ticket
https://openscience.atlassian.net/browse/OSF-4265

@adlius, can you look into how to fix this on Safari?


Trained Model and Code?

Hi,
Thank you for providing the code corresponding to the paper; the outcomes are noteworthy, especially when compared with TinEye. I have observed that there is some code missing, as highlighted in a separate issue. Could you also share the trained model and the associated code, please?

Hi
the code in here is outdated .... there's the another repo that has even more functionalities ...
it has a better color  counting methods which I took some time to discover ... as well as some other functions ... have a look  and let me know what you think ....
FashionColor

Hi,
Thank you so much for sharing the code! I will check it out. I have another thing to clarify with you, so from the paper, it seems you fine-tuned one pretrained masked-rcnn model using that 50k modanet dataset, is this correct?

@ysong10 I've found a pre-trained model here: https://github.com/cad0p/maskrcnn-modanet/blob/38551c4ae684000a4c6f31c46c6912db5fdbe2c3/maskrcnn_modanet/download.sh#L141
The new codebase doesn't clearly show how he created the pre-trained models. I'm going through the new repo after seeing this comment. I also think there was some training on additional clothing classes as well. I'll update or modify if I have additional insights

@ysong10 I've found a pre-trained model here: https://github.com/cad0p/maskrcnn-modanet/blob/38551c4ae684000a4c6f31c46c6912db5fdbe2c3/maskrcnn_modanet/download.sh#L141
EDIT: Looks like you can train model using get_prediction_method from his custom segmentation functions: https://github.com/morawi/FashionColor/blob/174ea1d83af48a4afc48e564b3440e42ef44de40/draw_image.py#L28
You can see something more detailed in one of the pytorch documents he references: https://pytorch.org/tutorials/intermediate/torchvision_tutorial.html
From there, I'm assuming you can train the saved model to the pth file using pickle, etc. I'm also going through this on my own for a project and will post an article when done

Hi, do you know how can I use the trained model in google colab?
Thanks for your help.

@nadia886  @ysong10
I'll make my notebook open. I don't have the original 50k dataset, but found a much smaller subset here:
https://universe.roboflow.com/gizem/modanet-97ezb
I'm using YoLo to train it which seems more up-to-date: https://colab.research.google.com/drive/1lSO8p7i-oYaXUpouJD-XnyN9qTmi_Oh7?usp=sharing
Link to the pre-trained model here (YoLo v11 only):
https://drive.google.com/file/d/1Ny0ZFOvw8YTq1xXYOujvGzH9S2P07Hkz/view?usp=sharing
One important note is that I used vast.ai to train the model instead of Google Colab. The processor for Google Colab is quite slow. I wound up spending about $1 to rent an RTX 4090 for an hour, though it took like 10 minutes to train this small dataset. There are other larger datasets I will train on once I get the other pieces of my personal project working.


fix:add gaspremium

Codecov Report

Merging #218 (a9e698b) into master (2bea615) will decrease coverage by 0.00%.
The diff coverage is 13.04%.

@@            Coverage Diff             @@
##           master     #218      +/-   ##
==========================================
- Coverage   15.45%   15.45%   -0.01%
==========================================
  Files          57       57
  Lines        5414     5417       +3
==========================================
  Hits          837      837
- Misses       4497     4500       +3
  Partials       80       80

Flag
Coverage Δ

unittests
15.45% <13.04%> (-0.01%)
:arrow_down:

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


txn-mempool-conflict (code 18)

When I try to add liquidity to the btc pool this error occurs.

Ether pool works fine .

already worked!

already worked!

What was the solution? I have the same issue.

What was the solution? I have the same issue.

I have the same problem. I need at least 2 attempts each time

I have the same problem. I need at least 2 attempts each time


fixthe common type of date and decimal should be  double

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run p0


Updating plugins and dependencies

Any chance on getting this in as well as getting a release since the prior PR I pushed reworked some dependency usage?

@stanislawosinski I've made some additional dependency updates.  Any chance of getting this applied and released?

Thanks for the PR, @hazendaz! I'm trying to get this released, but running Maven (mvn -Prelease clean verify site), I'm getting an error like this:
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-site-plugin:3.5:site (default-site) on project smartsprites: Error generating versions-maven-plugin:2.2:property-updates-report: only whitespace content allowed before start tag and not \u10f (position: START_DOCUMENT seen \u10f... @1:1)

(full log here: http://builds.carrot2.org/browse/SMARTSPRITES-SOFTWARE-DIST-20/log)
I'm not too much of a Maven expert, any idea how to fix this? I don't even seem to be able to find the path to the offending file.

Looks like a problem with pom where it switched to bom.    It's usually a bad non UTF 8 character in the pom.  Only way I know to fix it is to create a completely new pom and copy the contents from original into new one.  Or disable the version plugin.  I'll try to track down the root cause tonight.
Sent from Outlook Mobile
On Mon, Mar 7, 2016 at 3:28 AM -0800, "Stanisław Osiński"<EMAIL_ADDRESS>wrote:
Thanks for the PR, @hazendaz! I'm trying to get this released, but running Maven (mvn -Prelease clean verify site), I'm getting an error like this:
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-site-plugin:3.5:site (default-site) on project smartsprites: Error generating versions-maven-plugin:2.2:property-updates-report: only whitespace content allowed before start tag and not \u10f (position: START_DOCUMENT seen \u10f... @1:1)

(full log here: http://builds.carrot2.org/browse/SMARTSPRITES-SOFTWARE-DIST-20/log)
I'm not too much of a Maven expert, any idea how to fix this? I don't even seem to be able to find the path to the offending file.

Reply to this email directly or view it on GitHub:
https://github.com/carrotsearch/smartsprites/pull/13#issuecomment-193213671

Good point, thanks! Removing BOM from pom.xml did the trick. I've just made the release, it should propagate soon.

Awesome!  Thanks
Sent from Outlook Mobile
On Mon, Mar 7, 2016 at 4:33 AM -0800, "Stanisław Osiński"<EMAIL_ADDRESS>wrote:
Good point, thanks! Removing BOM from pom.xml did the trick. I've just made the release, it should propagate soon.

Reply to this email directly or view it on GitHub:
https://github.com/carrotsearch/smartsprites/pull/13#issuecomment-193232417


Are you planning to fix the shader import not working?

I am working on a project and I would love if I could import ninjarip shaders into blender and this is the only tool I have found, but as the readme says it doesn't work are you planning to update it?

Not in the foreseeable future. It will be very difficult, and I don't even use NinjaRipper anymore.


Disable select for events from external calendars

Short description

This Pr disables the select button in the event form for events that have been imported from external calendars.
Proposed changes

Disable dropdown when it's from an external calendar

Side effects

This has been worked on in collaboration with @KateGithub12
Resolved issues

Fixes: #3204

Pull Request Review Guidelines

@david-venhoff Thank you very much for the explanation. The suggestion is applied now, could you please check if everything is okay now? Thank you :)


feat(java): Support copy capabilities for some classes without no-argument constructors

What does this PR do?
Some classes with no-argument constructors will report an error when calling copy().
This pr:

implement the copy method for the no-argument constructor serializer
add test case

Related issues
https://github.com/apache/fury/issues/1777
https://github.com/apache/fury/issues/1679
Does this PR introduce any user-facing change?

[ ] Does this PR introduce any public API change?
[ ] Does this PR introduce any binary protocol compatibility change?

Benchmark

Very nice work, thank you @zhaommmmomo

LGTM, this is not an easy work, thanks for your contribution @zhaommmmomo

This is what I should do


Centralize Girder version

Fixes #293
@mgrauer If this approach is fine, I can use it for romanesco_version as well.

Go for it.
This looks great so far.

Looks great @danlamanna

Good doc changes.  I'll test with a fresh ansible vm.

LGTMAA. merging.
AA == and ansible


Windows compatibility

Added platform check
Use selector according to the platform (only select is available on Windows #37)
Platform compatible tests
Tested on Windows and Linux

Thanks for this. Merged.
Closes #37


pyrealsense2 support for Python 3.11

Hi,
Ref the Python 3.11 support request in this issue.
At what point do you consider Python 3.11 to no longer be in a pre-release state? Can we expect the next pyrealsense2 release to include Python 3.11 support?

Hi @pasandal  Previously, Python 3.11 was categorized by Python themselves as a release preview as of August 8 2022.
https://www.python.org/downloads/release/python-3110rc1/
On October 24 2022, a final stable release of 3.11 was made.
https://www.python.org/downloads/release/python-3110/
In light of the availability of a stable release of 3.11, I will discuss the issue with my Intel RealSense colleagues.

Ok. Thanks for the update.

As Python 3.11 support has been merged into the SDK, I will close this case.  Thanks, everyone!

When will the SDK with Python 3.11 support be published?

Hi @luigipacheco  I do not have a time estimate for release of the next SDK version.  The source code of the in-progress next version, which has already had Python 3.11 support added, is available on the SDK's development branch though.
https://github.com/IntelRealSense/librealsense/tree/development

Hi,
is there any plan to release for linux_aarch64 platform on python 3.11 version?

Hi @lgjonathan  You can compile the pyrealsense2 wrapper from source code with modern Python versions.
In regard to support for pip install pyrealsense2 installation for aarch64, at the time of writing this the newest Python version supported with aarch64 is Python 3.9.  Support for pip install for Python 3.11 was enquired about recently and my Intel RealSense colleagues provided a response at https://github.com/IntelRealSense/librealsense/issues/12188#issuecomment-1740576882


How to have iOS version use notifications not widgets

Our current version of the Physical Web on iOS was forced to use a widget because we
a) wanted a simpler way to scan for beacons (only when the widget is shown)
b) Had issues getting things to work properly with notifications
It's clear that the UX for the our iOS is less than idea as most people just don't use widgets in the TodayView. We'd like feedback from any iOS developers out there on ways we could get this to work without a widget. Here is what we need:

a way to scan for beacons that doesn't use much power. We'd like to only scan when the screen is first turned on. We think scanning constantly while in the pocket would just waste battery.

Show a notification when a beacon is found. We need to remove the notification if the beacon goes away.

Imagine waking up the phone at 1:00pm and finding a beacon and posting a notification.
Assume the user does nothing and puts the phone to sleep
at 2:00pm the user wakes the phone and no beacons are found
Clear the old beacon as it is no longer valid

Curious if anyone has any suggestions on best ways to do this.

Why the implementation of physical web in chrome not possible in iOS for chrome just like in chrome for Android.

They have different operating systems so different constraints

I am not an iOS developer so please excuse me if I make any assumptions
that are unfounded.
Looking at how iOS handles lock screen notifications there is a time since
the arrival of the notification. Is it not fair to give users some credit
to know that if the notification says 1hr and given that they will know
that they have moved that the notification may not be valid?
Let's assume anyway that they still click on the notification and it opens
Chrome to a Physical Web scan page, a rescan should be initiated to show
the latest information from the beacon anyway, so if they have moved out of
range nothing will show up. Would it be possible then to show a message for
old notification clicks to say that the Physical Web object is no longer in
range?
On Wed, Feb 17, 2016 at 8:26 AM scottjenson<EMAIL_ADDRESS>wrote:

They have different operating systems so different constraints
—
Reply to this email directly or view it on GitHub
https://github.com/google/physical-web/issues/592#issuecomment-185046494
.

--
Regards,
Russell M Morton

Currently the BeaconSage app in the app store has lockscreen notifications.  When a new beacon comes within range it will update the notifications.  There are 2 notifications: 1 for what's closest to you (when you swipe it takes you to the beaconsage app and takes you to that page), the other is telling you how many beacons are currently around you (when you swipe it takes you to the beaconsage app and shows you a list of beacons based on proximity).
We also included the ability to favorite beacons to view them later and a no-follow meta tag that developers can use to not allow people to favorite a particular page.

So, i'm thinking that if you were to have a beacon broadcasting Eddystone-URL with either a UID or iBeacon frame intermingling somewhere amongst the intervals, you would then be able to have a either the Chrome app or Physical Web app wake up, either if the app is closed and phone is sleeping or just on the lock screen, and generate a notification similar to what is seen in the BeaconSage app. Or, what I think to be the better approach, just put a small physical web icon in the lower left corner indicating the presence of beacons in the area which is achievable through the inclusion of the interweaving of the additional protocol type.  The user swipes up on the lower left corner icon on their lock screen, thereby unlocking the device and opening the app where they are taken directly to a physical web scanner which is populating all nearby beacons.  I'm wondering if this could be triggered by the Eddy-URL alone but I'm unsure as to the ability to generate this specific type of notification based on this alone.  I feel this is a good approach as it's still providing the passive experience, while allowing users to get some sort of visual lock screen indication of the existence of beacons. Something like this but physical web logo, of course

Great to see the physical web team considering user experience that will help physical web adoption grow rather than just be a cool concept. The digital marketer inside me is excited to see what solution you guys come up with.

I think that to have stuff that work as expected on iOs, it's often necessary to go the Apple way. So I agree with @rochforp on the fact that it could work with an intermittent iBeacon signal (I don't recommand Eddystone-UID there, as from my knowledges they can't wake up an iOs app in background yet).
More precisely, Chrome or any Physical Web app would monitor a specific iBeacon region (that needs to be decided in specifications). Then, whenever an iBeacon of this region will be detected, the app will wake up, scan and detect that there are one or many Eddystone-URL beacons in range and could then show a notification.
The constraint of this approach is that it would requires beacons emitting both iBeacon / Eddystone-URL signals (therefore forcing manufacturers to have firmware updates for existing beacons), or to set up some iBeacons emitting the specified iBeacon region next to Eddystone-URL beacons. But if it allows to have a good user experience on iOs, it's worth it.

@sebblanc I think I might have a work around and it would just use the Eddy-URL frame.  It's probably what the BeaconSage app is doing... maybe. I think your sdk in the app would be searching for Eddy-URL's, either at all times or only when the phone is awake (preferably awake to increase battery life). When an Eddy-URL is discovered a generic object like a CLBeacon is created which is used to trigger notifications and prompt the user to open the app.  Once the app is opened, the pw scanner handles the rest for consuming the real meat of the Eddystone-URL broadcast.    You're right about the iBeacon needing to be intermingled though in order to actually wake the device up, perhaps something like once every 10 frames so it doesn't drain the device battery or the beacon itself by throwing in too many iBeacon broadcasts and losing out on your real, desired Eddy-URL.

Unfortunately that's not a practical option as it would mean that every Eddystone-URL beacon in the world would ALSO need to broadcast an iBeacon frame. That has a huge impact on battery performance.
We we were hoping to do was do a low power scan using the ServiceIDList (which is a standard BLE parameter) in the ad packet and one which Apple does actually support.
We're not worried about scanning for the packet, I think we have some good ideas around that. Our biggest issues are:

Being able to use notifications (adding/removing them as needed)
Waking up at clever times to scan and not use too much power.

If we can use the Apple beacon scanning library in a way that makes this easier, great.
Scott

Quick question about this:

most people just don't use widgets in the TodayView

Is that general usage, or specifically from the implementation of PW in Chrome? That is, is it generally something that users don’t do at all, or a question of educating / demonstrating how to do it for Chrome?

From my own personal experience, the only reason I enabled Today View (or
frankly installed Chrome on iOS) was for physical web support.
My own opinion is that the infrastructure in both platforms is fairly
sufficient as is for a LOT of use cases.  I'm not sure that a lot of
changes need to be made (or any) until some early wins occur that drive
adoption forward.
Consider that when device manufacturers (whether IOT or otherwise) begin
shipping devices with no meaningful UI and/or documentation, instead
relying on in_built beacons advertising their interface, Android will
support this nicely out of the box with what is in Chrome Beta today.
Mass -> in-built <- adoption from manufacturers and not post-shipment
external beacon placement is likely what will eventually draw all mobile
platforms into the physical web camp.  The idea of having a highly mobile
battery-backed physical web beacon seems like a novelty with far fewer
practical use cases by comparison to this big elephant.
So, in short I think retail/marketing will eventually get the experience
they want from the UI/responsiveness, but there's not going to be as big a
clamor to get there as there would likely be if driven from other angles.
On Thu, Feb 18, 2016 at 9:28 AM, Peter Gasston<EMAIL_ADDRESS>wrote:

Quick question about this:
most people just don't use widgets in the TodayView
Is that general usage, or specifically from the implementation of PW in
Chrome? That is, is it generally something that users don’t do at all, or a
question of educating / demonstrating how to do it for Chrome?
—
Reply to this email directly or view it on GitHub
https://github.com/google/physical-web/issues/592#issuecomment-185744873
.

I've demoed the physical web to at least 200 iPhone users live in the past few months, my experience is:

about 50-60% have Chrome installed (this was a surprise)
hardly anyone knows what widgets are and how to use them
whenever I mention Chrome everyone instantly launches it (before I get to the part about widgets)

Because a lot of corporate executives are iPhone users, improving the current onboarding would be big help to companies like us as we have several major corporate prospects that are not seriously looking into this because of the complicated onboarding on iOS (at least compared to Android).

We are aware of this issue ;-) What would you suggest?

@scottjenson I'm no developer.

Consistency in behavour: how beacons are discovered (you're going to do that with filtering out HTTP on iOS for example)
If you can't have the same kind of notification-based experience as you have in Android (because of OS limitations), try to have it directly in the App (Like the Opera Dev build)

Our next release of Chrome for iOS uses the same filtering as Android so the same beacons will show up in both.
Not every iOS user uses Chrome
We are moving to a more in app experience for iOS
We hope the new iOS10 widget experience improves the experience

I believe you can cancel a local notification via UIApplication cancelLocalNotification:.  The hard part is knowing when the user has left the area where the PW beacon is so that you can cancel the notifications appropriately.  You could use a background task and before it expires remove notifications if the beacons aren't detected anymore.  But you only get 3 minutes, and that may not be long enough for the user to physically leave the area.
Instead of firing a local notification for each beacon, you could fire a notification that says something like "PhysicalWeb object(s) detected". Selecting this notification would take you into a table view in Chrome that shows the list.  If you aren't able to remove that notification when the beacon goes away, at least you've only added one notification.  IMHO, this would be preferable to the current setup where very few people are able to use PW on iOS because they don't know about the widget.

There are battery concerns about scanning on the background. Given the improvements with Widgets in iOS10 we hope that our current widget approach will be easier to setup and use  (it is not very easy to setup to be honest)
We're also realizing that while notifications are universal and easy to launch, they do have a downside: the can be very easily dismissed. They also can just not scan at the right time and not show the user the right beacon in front of them. The huge advantage of a widget is that it is inherent in it a 'scan now' request. Every time you open it, it WILL scan which has strong benefits.
We're trying to figure out ways making this happen on android. There is a 'nearby' quick setup icon that will help (but let's be honest, many won't find it) There are other things we're experimenting with.

Hi scottjenson,
without chrome i could not able to notify url to customer in ios. There is no alternative like safari to push eddystone url

The only app that we offer is Chrome. It is not supported by Safari (yet) If you want, you can install other physical Web scanners on ios. THere is Beacon Sage and BKON. Both are excellent


New, possibly benign, error with Homebridge beta

I seem to be getting a new error message regularly from the plugin since I installed Homebridge beta v1.3.0-beta.53:
[2/7/2021, 7:35:47 PM] Received warning for the plugin 'homebridge-harmony' from the characteristic 'Active Identifier': characteristic was supplied illegal value: number -1 exceeded minimum of 0.
[2/7/2021, 7:35:47 PM] Received warning for the plugin 'homebridge-harmony' from the characteristic 'Active Identifier': characteristic was supplied illegal value: number -1 exceeded minimum of 0.
Just thought I'd mention it! I can't see if it affects anything just yet...

Hi, it is a known issue #315 :)

Apologies - I should have seen that. Are there any ramifications to the problem?


Enhance setup_oud.sh to read subfolders

Enhance setup_oud.sh to read subfolder during installation. This should allow to organise base software, patches etc into several folders. Same should apply for https://github.com/oehrlis/oudbase/blob/master/local/oudbase/bin/setup_oud_patch.sh.

update function get_software to read packages from subfolder

add first release in setup_oud.sh and setup_oud_patch.sh to read package from subfolder


Updated Readme.md

Hi @nikhita : please review and approve the changes at the earliest !

@sudharshan-govindan wow, thanks for adding so many conferences! Just wanted to let you know that I've seen this PR and will review it by today. :+1:

@nikhita : Thanks for your lightning response !
I am a Developer Advocate in IBM, Bengaluru. We use your repo to communicate to our developers about the list of conferences, workshops etc happening through out the year.
Fantastic initiative 👍

@sudharshan-govindan Awesome 👍 Great list of conferences to know.
But, most of them are not having an option to submit a CFP. And one devops conference is from a company.
@nikhita May be we should create a field separating community and organization driven conferences?

Hi Admin(s),
Can you please review and approve the PR at the earliest ?

But, most of them are not having an option to submit a CFP. And one devops conference is from a company.

Thanks for pointing this out, I was not aware of this. :+1:

@nikhita May be we should create a field separating community and organization driven conferences?

I would rather remove the organization driven conferences. I don't want to include commercially driven conferences in this repo. It opens a huge can of worms, which I don't want to deal with. :)
If someone wants to include them, they are free to open a new repo for it.

Can you please review and approve the PR at the earliest ?

Sorry, I'm busy at $DAYJOB but I'll get to it soon.

2018 is almost over, so I'll close this. Thanks for creating the PR, much appreciated!


token holder compensation

is there something token holders can get back who suffered from the hack?

You can redeem funds at https://redeem.mango.markets
Check out Discord for further support


Lord of xen patch comment support

Adds support for generating functions, enums, structs and handles with comments.

This is how comments look like on generated result:


Home Page

Describe the feature you'd like

Currently , the entire home screen is not being used . We can increase the grid size of those cards , We can cover the entire screen , and then add a bottom navigation bar containing some basic features.
Additional context

Screenshots

Would you like to work on the issue?
I want to work on this , Both the bottom navigation bar implementation and the home screen layout.

This screen will be taking for now....in issue #43 and #42


[Feat]: Adding  my profile

What feature?
I have an idea
Add screenshots
Add screenshots
Code of Conduct

[ ] I agree to follow this project's Code of Conduct

@shyamtawli please assign me the isseue

You can raise PR adding you profile without opening a issue.


Dates before 2000-01-01 not handled correctly

Ecto appears to not decode dates correctly that are before 2000-01-01.
For example, modifying the date={...} line in the test "test update all with casting and dumping" in ecto/integration_test/cases/repo.ex causes a test failure with any date component before 2000-01-01.
From tests with the code that originally showed the problem the date appears to be correctly written to the database but not decoded correctly when read-back.
Running the code in Ecto.Adapters.Postgres.DateTime.encode, 1999-12-31 encodes as <<255, 255, 255, 255>>.  Using this as an input to Ecto.Adapters.Postgres.DateTime.decode results in the date {11761221, 1, 19}, which corresponds with what I get in my own code when I use this date.
I've tried this with Erlang 17.4.1, Elixir 1.0.3, Ecto 0.10.0, Postgrex 0.8.0 and PostgreSQL 9.4.1 on Mac OS X 10.10.2.

Thank you for the report. If you can send a pull request, it would be appreciated, otherwise I will take a look at it later. This problem likely exists in Postgrex too /cc @ericmj


Add band/point invertExtent.

Adds invertExtent method for band and point scales, which can be useful for brushing and linking over ordinal domains. Separated from earlier PR #60.

I’d like the documentation to be clearer. Given this representation of bands:

Some questions: Is a band included if [r0, r1] has a non-empty intersection with the band, or does [r0, r1] have to completely cover the band? And are r0 and r1 both treated as inclusive bounds? Is a point scale treated as zero-width bands? If [r0, r1] falls within the padding between bands, does it return an empty array? It’d be helpful to have tests which explicitly state the expected outcome for these cases.

Non-empty intersection with a band is sufficient for inclusion; r0 and r1 are treated as inclusive bounds. Related cases are included in the submitted tests (e.g., band-test.js#L309).
A point scale is treated as having zero-width bands. Zero bandwidth scales are not explicitly included in the tests, but band boundary tests are.
If the queried range falls entirely within padding, invertExtent returns undefined. This case is included in the submitted tests (e.g., band-test.js#L340, though more detailed comments might be helpful).

was there any update on merging this or whether there would be .invert() support for band/point scales? would <3 this functionality.

@williaster If it is useful to you, we have an updated version of band/point scales included in the vega-scale repo. It includes an invert function in addition to modifications to improve layout consistency. We'd be happy to have those merged here if there is interest.

I’ve implemented a demo of ordinal brushing here:
https://observablehq.com/@d3/ordinal-brushing
I used bisectRight as in this PR. One point of awkwardness is x.domain().map(x) to access the underlying ordinal scale’s range. My preference for this type of interaction would be to only select ordinal values based on the middle of the band (in effect, treating band and point scales the same way). I wonder whether the proposed band.invertExtent here could be flexible enough to support different intersection logic, or if some other API is desired?

After a little more thought, I switched my demo to use a point scale. I think it makes sense for the point scale to consider something selected only if the point is inside the interval, while the band scale would consider something selected if the band intersects the interval, as implemented here. So, I’m down with this approach—I just need to think through the API a little more.

Any update on this? It'll be really useful to have an invert function for band scales.


[FEATURE] Custom Ping

Currently it's possible to set a custom pointer, but not a custom ping. I would love to theme both the ping and the pointer to match my game's UI.

this is already possible:

Just check the box for "ping" instead of "pointer".

this is already possible:

Just check the box for "ping" instead of "pointer".


Build failure in auto-builder

----------
1. WARNING in src/main/java/org/apache/log4j/xml/DOMConfigurator.java (at line 37)
	import org.apache.log4j.spi.OptionHandler;
	       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
The import org.apache.log4j.spi.OptionHandler is never used
----------
----------
2. WARNING in src/main/java/org/apache/log4j/chainsaw/DetailPanel.java (at line 37)
	class DetailPanel
	      ^^^^^^^^^^^
The serializable class DetailPanel does not declare a static final serialVersionUID field of type long
----------
3. WARNING in src/main/java/org/apache/log4j/chainsaw/DetailPanel.java (at line 42)
	private static final Logger LOG =
	                            ^^^
The field DetailPanel.LOG is never read locally
----------
----------
4. WARNING in src/main/java/org/apache/log4j/chainsaw/MyTableModel.java (at line 37)
	class MyTableModel
	      ^^^^^^^^^^^^
The serializable class MyTableModel does not declare a static final serialVersionUID field of type long
----------
5. WARNING in src/main/java/org/apache/log4j/chainsaw/MyTableModel.java (at line 142)
	private Priority mPriorityFilter = Priority.DEBUG;
	                                            ^^^^^
The field Priority.DEBUG is deprecated
----------
----------
6. WARNING in src/main/java/org/apache/log4j/chainsaw/ExitAction.java (at line 29)
	class ExitAction
	      ^^^^^^^^^^
The serializable class ExitAction does not declare a static final serialVersionUID field of type long
----------
----------
7. WARNING in src/main/java/org/apache/log4j/chainsaw/Main.java (at line 44)
	public class Main
	             ^^^^
The serializable class Main does not declare a static final serialVersionUID field of type long
----------
----------
8. WARNING in src/main/java/org/apache/log4j/chainsaw/ControlPanel.java (at line 40)
	class ControlPanel extends JPanel {
	      ^^^^^^^^^^^^
The serializable class ControlPanel does not declare a static final serialVersionUID field of type long
----------
9. WARNING in src/main/java/org/apache/log4j/chainsaw/ControlPanel.java (at line 42)
	private static final Logger LOG =
	                            ^^^
The field ControlPanel.LOG is never read locally
----------
----------
10. WARNING in src/main/java/org/apache/log4j/chainsaw/LoadXMLAction.java (at line 40)
	class LoadXMLAction
	      ^^^^^^^^^^^^^
The serializable class LoadXMLAction does not declare a static final serialVersionUID field of type long
----------
----------
11. ERROR in src/main/java/org/apache/log4j/net/SMTPAppender.java (at line 1)
	/*
	^
The type javax.mail.internet.MimeBodyPart cannot be resolved. It is indirectly referenced from required .class files
----------
12. ERROR in src/main/java/org/apache/log4j/net/SMTPAppender.java (at line 45)
	import javax.mail.internet.MimeBodyPart;
	       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
The import javax.mail.internet.MimeBodyPart cannot be resolved
----------
13. ERROR in src/main/java/org/apache/log4j/net/SMTPAppender.java (at line 299)
	MimeBodyPart part = new MimeBodyPart();
	^^^^^^^^^^^^
MimeBodyPart cannot be resolved to a type
----------
14. ERROR in src/main/java/org/apache/log4j/net/SMTPAppender.java (at line 299)
	MimeBodyPart part = new MimeBodyPart();
	                        ^^^^^^^^^^^^
MimeBodyPart cannot be resolved to a type
----------
----------
15. WARNING in src/main/java/org/apache/log4j/lf5/Log4JLogRecord.java (at line 32)
	public class Log4JLogRecord extends LogRecord {
	             ^^^^^^^^^^^^^^
The serializable class Log4JLogRecord does not declare a static final serialVersionUID field of type long
----------
----------
16. WARNING in src/main/java/org/apache/log4j/lf5/LF5Appender.java (at line 20)
	import org.apache.log4j.lf5.util.Resource;
	       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
The import org.apache.log4j.lf5.util.Resource is never used
----------
----------
17. WARNING in src/main/java/org/apache/log4j/lf5/util/LogFileParser.java (at line 156)
	LogFactor5ErrorDialog error = new LogFactor5ErrorDialog(
	                      ^^^^^
The local variable error is never read
----------
18. WARNING in src/main/java/org/apache/log4j/lf5/util/LogFileParser.java (at line 165)
	_loadDialog.hide();
	            ^^^^^^
The method hide() from the type Dialog is deprecated
----------
----------
19. WARNING in src/main/java/org/apache/log4j/lf5/util/AdapterLogRecord.java (at line 33)
	public class AdapterLogRecord extends LogRecord {
	             ^^^^^^^^^^^^^^^^
The serializable class AdapterLogRecord does not declare a static final serialVersionUID field of type long
----------
----------
20. WARNING in src/main/java/org/apache/log4j/lf5/LogLevel.java (at line 39)
	public class LogLevel implements java.io.Serializable {
	             ^^^^^^^^
The serializable class LogLevel does not declare a static final serialVersionUID field of type long
----------
----------
21. WARNING in src/main/java/org/apache/log4j/lf5/viewer/categoryexplorer/CategoryImmediateEditor.java (at line 90)
	CategoryNode node = (CategoryNode) path.getLastPathComponent();
	             ^^^^
The local variable node is never read
----------
22. WARNING in src/main/java/org/apache/log4j/lf5/viewer/categoryexplorer/CategoryImmediateEditor.java (at line 91)
	boolean rv = false;
	        ^^
The local variable rv is never read
----------
----------
23. WARNING in src/main/java/org/apache/log4j/lf5/viewer/LogFactor5Dialog.java (at line 31)
	public abstract class LogFactor5Dialog extends JDialog {
	                      ^^^^^^^^^^^^^^^^
The serializable class LogFactor5Dialog does not declare a static final serialVersionUID field of type long
----------
24. WARNING in src/main/java/org/apache/log4j/lf5/viewer/LogFactor5Dialog.java (at line 58)
	super.show();
	      ^^^^^^
The method show() from the type Dialog is deprecated
----------
----------
25. WARNING in src/main/java/org/apache/log4j/lf5/viewer/FilteredLogTableModel.java (at line 39)
	public class FilteredLogTableModel
	             ^^^^^^^^^^^^^^^^^^^^^
The serializable class FilteredLogTableModel does not declare a static final serialVersionUID field of type long
----------
----------
26. WARNING in src/main/java/org/apache/log4j/lf5/viewer/LogFactor5ErrorDialog.java (at line 33)
	public class LogFactor5ErrorDialog extends LogFactor5Dialog {
	             ^^^^^^^^^^^^^^^^^^^^^
The serializable class LogFactor5ErrorDialog does not declare a static final serialVersionUID field of type long
----------
27. WARNING in src/main/java/org/apache/log4j/lf5/viewer/LogFactor5ErrorDialog.java (at line 55)
	hide();
	^^^^^^
The method hide() from the type Dialog is deprecated
----------
----------
28. WARNING in src/main/java/org/apache/log4j/lf5/viewer/LogFactor5LoadingDialog.java (at line 31)
	public class LogFactor5LoadingDialog extends LogFactor5Dialog {
	             ^^^^^^^^^^^^^^^^^^^^^^^
The serializable class LogFactor5LoadingDialog does not declare a static final serialVersionUID field of type long
----------
----------
29. WARNING in src/main/java/org/apache/log4j/lf5/viewer/LogBrokerMonitor.java (at line 24)
	import org.apache.log4j.lf5.util.StreamUtils;
	       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
The import org.apache.log4j.lf5.util.StreamUtils is never used
----------
30. WARNING in src/main/java/org/apache/log4j/lf5/viewer/LogBrokerMonitor.java (at line 795)
	JMenuItem menuItem = result;
	          ^^^^^^^^
The local variable menuItem is never read
----------
31. WARNING in src/main/java/org/apache/log4j/lf5/viewer/LogBrokerMonitor.java (at line 914)
	JMenuItem exitMI;
	          ^^^^^^
The local variable exitMI is never read
----------
32. WARNING in src/main/java/org/apache/log4j/lf5/viewer/LogBrokerMonitor.java (at line 1066)
	LogFactor5ErrorDialog error = new LogFactor5ErrorDialog(
	                      ^^^^^
The local variable error is never read
----------
33. WARNING in src/main/java/org/apache/log4j/lf5/viewer/LogBrokerMonitor.java (at line 1246)
	fonts = tk.getFontList();
	           ^^^^^^^^^^^^^
The method getFontList() from the type Toolkit is deprecated
----------
34. WARNING in src/main/java/org/apache/log4j/lf5/viewer/LogBrokerMonitor.java (at line 1428)
	LogFactor5ErrorDialog error = new LogFactor5ErrorDialog(
	       ERROR: Function failed: do_compile (see /home/otavio/hacking/ossystems/embedded-linux/tmp-eglibc-eglibc/work/x86_64-linux/log4j1.2-native-1.2.15-r0/temp/log.do_compile.10130 for further information)
               ^^^^^
The local variable error is never read
----------
35. WARNING in src/main/java/org/apache/log4j/lf5/viewer/LogBrokerMonitor.java (at line 1475)
	LogFactor5ErrorDialog error = new LogFactor5ErrorDialog(
	                      ^^^^^
The local variable error is never read
----------
36. WARNING in src/main/java/org/apache/log4j/lf5/viewer/LogBrokerMonitor.java (at line 1538)
	LogFactor5ErrorDialog error = new LogFactor5ErrorDialog(
	                      ^^^^^
The local variable error is never read
----------
37. WARNING in src/main/java/org/apache/log4j/lf5/viewer/LogBrokerMonitor.java (at line 1555)
	LogFactor5ErrorDialog error = new LogFactor5ErrorDialog(
	                      ^^^^^
The local variable error is never read
----------
----------
38. WARNING in src/main/java/org/apache/log4j/lf5/viewer/LogFactor5InputDialog.java (at line 38)
	public class LogFactor5InputDialog extends LogFactor5Dialog {
	             ^^^^^^^^^^^^^^^^^^^^^
The serializable class LogFactor5InputDialog does not declare a static final serialVersionUID field of type long
----------
39. WARNING in src/main/java/org/apache/log4j/lf5/viewer/LogFactor5InputDialog.java (at line 88)
	hide();
	^^^^^^
The method hide() from the type Dialog is deprecated
----------
40. WARNING in src/main/java/org/apache/log4j/lf5/viewer/LogFactor5InputDialog.java (at line 96)
	hide();
	^^^^^^
The method hide() from the type Dialog is deprecated
----------
41. WARNING in src/main/java/org/apache/log4j/lf5/viewer/LogFactor5InputDialog.java (at line 103)
	hide();
	^^^^^^
The method hide() from the type Dialog is deprecated
----------
----------
42. WARNING in src/main/java/org/apache/log4j/lf5/LogLevelFormatException.java (at line 30)
	public class LogLevelFormatException extends Exception {
	             ^^^^^^^^^^^^^^^^^^^^^^^
The serializable class LogLevelFormatException does not declare a static final serialVersionUID field of type long
----------
----------
43. WARNING in src/main/java/org/apache/log4j/lf5/LogRecord.java (at line 33)
	public abstract class LogRecord implements java.io.Serializable {
	                      ^^^^^^^^^
The serializable class LogRecord does not declare a static final serialVersionUID field of type long
----------
----------
44. WARNING in src/main/java/org/apache/log4j/spi/NOPLogger.java (at line 59)
	void closeNestedAppenders() {
	     ^^^^^^^^^^^^^^^^^^^^^^
The method NOPLogger.closeNestedAppenders() does not override the inherited method from Category since it is private to a different package
----------
44 problems (4 errors, 40 warnings)

This "The import javax.mail.internet.MimeBodyPart cannot be resolved" looks like fail compilation auf gnumail, please check the jar for classes with 0 bytes.

It did fix this however we have this build failures. Is it a concurrency issue?

Not really I think,
we have observed with gnumail sometimes the compiling of some classes goes wrong and they end up with 0 bytes. But I didnt see quite for a while on my systems. Thats why I asked to look up for 0 Bytes class inside of the gnumail jar's

I have this in classpath and gnumail and this ofthen happens when I do auto-rebuild of all code from scratch.

Then make some statistics do cleansstate build in a loop and check for 0 Byte classes.

a better heuristic would be in do_install or do_compile and abort the build if it encounters such 0 byte files. It is a build error after all.

I'm thinking on disabling the parallel install/build and check if "fix" this.

@koenkooi agreed.

Yes Yes
I thought about it too and will come up with a while loop which checks for 0 byte classes.


Add support for UseSSL configuration (needed to run bugsnag-agent)

If you set an endpoint that starts with http:// it should do this already with a need for this change.
Let me know if that doesnt work for you.
Reference

That works for me, thanks.
The PR would bring bugsnag-laravel implementation on par with the reference implementation bugsnag-php (even though UseSSL is redundant in that lib too).
Unfortunately there is another PR pending on that is still blocking me from using this repo (since it depends on bugsnag-php) https://github.com/bugsnag/bugsnag-php/pull/115. Do you think you could have review that one for merge?

@aleemb We've merged the cURL fix, and tagged a release over on our PHP package.


Replace BGzipCodec with equivalent BGZFCodec from Hadoop-BAM

The idea is to consolidate on the BGZF code in Hadoop-BAM that other projects (like GATK, ADAM) use too. It's not ready for merge yet as the write side (and support for the .bgz suffix) is not in Hadoop-BAM yet: https://github.com/HadoopGenomics/Hadoop-BAM/pull/106.

I've updated this with the code from Hadoop-BAM (in 7.6.0), so it's ready for review. Can you take a look please @cseed?

I'll take a look.  Thanks Tom!

jenkins


"customCollapse" is not definied

Hi,
if I use the a11ytree with the example from your site I got the error "customCollapse" is not definied.
What I'm doing wrong?
http://jsbin.com/vuvoki/2/edit

I'll take a look this weekend, and get back to you.
On May 18, 2015 12:14 AM, "Jan"<EMAIL_ADDRESS>wrote:

Hi,
if I use the a11ytree with the example from your site I got the error
"customCollapse" is not definied.
What I'm doing wrong?
http://jsbin.com/vuvoki/2/edit
—
Reply to this email directly or view it on GitHub
https://github.com/longmatthewh/jquery-a11ytree/issues/9.

Hi,
have you found time to take a look in the meanwhile?

looking at it now. sorry about that.

Take a look at http://jsbin.com/guzifadeyi/edit?html,js,output. customCollapse and customExpand are dummy functions. The onCollapse and onExpand options allow to provide a function to be executed on expand and collapse if you want to. In my example, I added the expand/collapse function to swap the font awesome icons (ie. + to - on collapse, and - to + on expand). I'm most using CSS to do the visual expand/collapse. Take a look at my example js source at https://github.com/longmatthewh/longmatthewh.github.io/blob/master/js/a11yTree.init.js and the less at https://github.com/longmatthewh/longmatthewh.github.io/blob/master/css/a11yTree.less.
Let me know if you any further questions.

Hi,
thanks for your explaination and example. But why do the collapse function of the menu not work?

customCollaspe and customExpand are just dummy functions for the example code. They're not defined anywhere. Perhaps, I'll just remove them from the example to avoid confusion.


Bug:

Describe the Bug
Sometimes, whitespace is not compacted.
How to Reproduce
I am using Linter 1.12.0 with Obsidian 1.2.3 on Windows.
And the following settings:

Remove multiple spaces: on

You can also set these, but they don't seem to matter here:

Space after list markers: on
Unorderderd list style: on
Convert bullet list markers: on

Copy the following Markdown code:
- foo
-     bar    baz
-    bar    baz

Note that on the second line, there are 5 blanks before "bar" and 4 blanks before "baz". On the third line there are 4 blanks before both words.
Run "Lint the current file"
Only the two occurences of multiple whitespace in the third line will be compacted, none of those in the second line:
- foo
-     bar    baz
- bar baz

Expected Behavior
On the second lines, the two occurences of multiple whitespace should be compacted as well.
Expected output:
- foo
- bar baz
- bar baz

Thanks for reporting this. I can reproduce with the example provided. The regex matches just fine, but I need to check why it is not actually updating the data as expected. It could be that one of the ignores is catching the second foo bar. It could be getting considered to be a code block or something like that. If that is the case, I am not sure if I can do anything about it.

So the problem is that 4 spaces like you have in the example is an indicator of a code block. Thus the space cannot be removed as the rule ignores code blocks to prevent issues with spacing in code blocks.
Here is what it looks like before the regex runs due to the ignores:
 - foo
 - {CODE_BLOCK_PLACEHOLDER}
 -    bar    baz

As far as I can tell, this is expected. I will just double check with the online AST parser and see if that still comes out to be a code block reference. If it is, then this is expected behavior.

The AST does interpret that block as a code block:

So I am not sure this can be fixed if we preserve spacing in code blocks.
I can add an example or more documentation around this if you would like @Cito . But other than that, I plan to close this as a non-issue as it is expected behavior for the amount of spaces present.

Feel free to respond on the issue, but I am closing it as it worls as intended.


RLOS in 3D in Pro- Input points have too much vertical exaggeration

When running RLOS in a scene, the tool will run, and the input points show up in the contents pane as 3D layers. This default is ok, except that there appears to be too much vertical exaggeration of the points. I don't see any vertical exaggeration, cartographic offset, or any other default appearence settings enabled when the tool runs, which is why this is confusing. For now, you can just turn it into a 2D layer, but the 3D layer should accurately reflect the offset you enter in the parameters of the add-in.

With the updates to Military Tools in the way that we draw the points in 3D this is no longer an issue:


Bluetooth: SMP: Fix minor coding style issues

commit 9b6ad4067be40b8c8aa1da67fc39eb53fb50ace5 introduced some minor
coding style issues related to line splitting. Fix these.
Signed-off-by: Johan Hedberg<EMAIL_ADDRESS>

@jli157 FYI

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@9b6ad40). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #9159   +/-   ##
=========================================
  Coverage          ?   52.48%
=========================================
  Files             ?      200
  Lines             ?    25156
  Branches          ?     5258
=========================================
  Hits              ?    13202
  Misses            ?     9834
  Partials          ?     2120

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 9b6ad40...7c65f2c. Read the comment docs.


playbackDevice.writeSamples eating CPU

On my system which is a 64-bit Linux box using Alsa for sound, the call to writeSamples on a AudioDevice maxes out a CPU core if the samplerate is 30khz (30000) or higher; however hardly any at all ( < 10% CPU) is used when the samplerate is 20khz (20000) or lower. 44.1khz and 48khz also maxes out a CPU.
Android seems to have no problem whatsoever.

Issue is not related to audio driver but to java code (see my PR that fix it)