Build errors in esp-idf-hal-0.32.10

   Compiling esp-idf-hal v0.32.10
error[E0433]: failed to resolve: could not find `Operation` in `spi`
   --> /home/brian/.cargo/registry/src/github.com-1ecc6299db9ec823/esp-idf-hal-0.32.10/src/spi.rs:575:50
    |
575 |                 embedded_hal_0_2::blocking::spi::Operation::Write(write) => {
    |                                                  ^^^^^^^^^ could not find `Operation` in `spi`

error[E0433]: failed to resolve: could not find `Operation` in `spi`
   --> /home/brian/.cargo/registry/src/github.com-1ecc6299db9ec823/esp-idf-hal-0.32.10/src/spi.rs:578:50
    |
578 |                 embedded_hal_0_2::blocking::spi::Operation::Transfer(words) => {
    |                                                  ^^^^^^^^^ could not find `Operation` in `spi`

error[E0405]: cannot find trait `Transactional` in module `embedded_hal_0_2::blocking::spi`
   --> /home/brian/.cargo/registry/src/github.com-1ecc6299db9ec823/esp-idf-hal-0.32.10/src/spi.rs:563:38
    |
563 |     embedded_hal_0_2::blocking::spi::Transactional<u8> for Master<SPI, SCLK, SDO, SDI, CS>
    |                                      ^^^^^^^^^^^^^ not found in `embedded_hal_0_2::blocking::spi`
    |
help: consider importing one of these items
    |
23  | use embedded_hal::i2c::blocking::Transactional;
    |
23  | use embedded_hal::spi::blocking::Transactional;
    |

error[E0412]: cannot find type `Operation` in module `embedded_hal_0_2::blocking::spi`
   --> /home/brian/.cargo/registry/src/github.com-1ecc6299db9ec823/esp-idf-hal-0.32.10/src/spi.rs:569:60
    |
569 |         operations: &mut [embedded_hal_0_2::blocking::spi::Operation<'a, u8>],
    |                                                            ^^^^^^^^^ not found in `embedded_hal_0_2::blocking::spi`
    |
help: consider importing one of these items
    |
23  | use embedded_hal::i2c::blocking::Operation;
    |
23  | use embedded_hal::spi::blocking::Operation;
    |

Some errors have detailed explanations: E0405, E0412, E0433.
For more information about an error, try `rustc --explain E0405`.
error: could not compile `esp-idf-hal` due to 4 previous errors

Wondering if I somehow picked up the wrong version of something. But it looks fine to me...
Later: Oh, I see, it was getting an old version of embedded_hal, the listed version isn't precise enough. This solves it:
diff --git a/Cargo.toml b/Cargo.toml
index f353d1c..67f42b6 100644
--- a/Cargo.toml
+++ b/Cargo.toml
@@ -69,7 +69,7 @@ esp-idf-sys = { version = "0.30.3", features = ["binstart"] }
 esp-idf-svc = "0.36.9"
 esp-idf-hal = "0.32.9"
 embedded-svc = "0.16"
-embedded-hal = "0.2"
+embedded-hal = "0.2.7"
 embedded-graphics = "0.7"
 display-interface = "0.4"
 display-interface-spi = "0.4"

This is fixed in the upcoming esp-idf-hal.


Need help for ACPI table, rsdt and xsdt entries are aberrant

I want to print all acpi headers. I can detect the rsdt and xsdt tables and their checksums are correct. But the entries in xsdt all have a length of 0 and those in rsdt a length of more than 1 billion.  All the signatures are not ascii characters.
my code
#![no_main]
#![no_std]

use core::str::from_utf8_unchecked;

use log::info;
use uefi::{prelude::*, table::cfg::ACPI2_GUID};

#[repr(C, packed)]
#[derive(Debug)]
struct RSDP {
    signature: [u8; 8],
    checksum: u8,
    oem_id: [u8; 6],
    revision: u8,
    rsdt_address: u32,
    length: u32,
    xsdt_address: u64,
    extended_checksum: u8,
    reserved: [u8; 3],
}
/// acpi spec 5.2.7
#[repr(C, packed)]
#[derive(Debug, Clone, Copy)]
struct ACPISDTHeader {
    signature: [u8; 4],
    length: u32,
    revision: u8,
    checksum: u8,
    oem_id: [u8; 6],
    oem_table_id: [u8; 8],
    oem_revision: u32,
    creator_id: u32,
    creator_revision: u32,
}
impl ACPISDTHeader {
    #[inline]
    fn get_entries_len(&self, entry_ptr_size: usize) -> usize {
        let entries_size = self.length as usize - size_of::<Self>();
        entries_size / entry_ptr_size
    }

    #[inline]
    fn get_entries_start(&mut self) -> *mut ACPISDTHeader {
        unsafe { (self as *mut Self).offset(1) }
    }

    #[inline]
    fn get_signature(&self) -> &str {
        unsafe { from_utf8_unchecked(&self.signature) }
    }

    fn compute_checksum(&self) -> u8 {
        let mut sum: u8 = 0;

        let mut start = self as *const Self as *const u8;

        unsafe {
            for _ in 0..self.length {
                sum = sum.wrapping_add(*start as _);
                start = start.offset(1);
            }
        }

        sum as _
    }
}

unsafe fn print_entries(rsdt_or_xsdt: *mut ACPISDTHeader, entry_ptr_size: usize) {
    info!(
        "xsdt or rsdt: signature {}, checksum : {}",
        (*rsdt_or_xsdt).get_signature(),
        (*rsdt_or_xsdt).compute_checksum()
    );

    let mut start = (*rsdt_or_xsdt).get_entries_start();
    let len = (*rsdt_or_xsdt).get_entries_len(entry_ptr_size);

    for i in 0..len {
        let entry = if entry_ptr_size == 4 {
            (start as u32) as *mut ACPISDTHeader
        } else {
            start
        };

        let a = (*entry).signature;
        let length = (*entry).length;

        info!("entry {i} : signature {a:?}, length {length}");
        start = start.byte_add(entry_ptr_size);
    }
}

#[entry]
fn main(_image_handle: Handle, mut system_table: SystemTable<Boot>) -> Status {
    uefi::helpers::init(&mut system_table).unwrap();

    for config_entry in system_table.config_table().iter() {
        match config_entry.guid {
            ACPI2_GUID => {
                let rsdp = config_entry.address as *mut RSDP;
                unsafe {
                    let xsdt = (*rsdp).xsdt_address as *mut ACPISDTHeader;
                    print_entries(xsdt, 8);

                    let rsdt = (*rsdp).rsdt_address as *mut ACPISDTHeader;
                    print_entries(rsdt, 4);
                }
            }
            _ => {}
        }
    }

    system_table.boot_services().stall(20_000_000);
    Status::SUCCESS
}

my results

I think you're missing a level of pointer indirection. get_entries_start should look something like this:
    #[inline]
    fn get_entries_start(&self) -> *const *const ACPISDTHeader {
        let start = unsafe { (self as *const Self).offset(1) };
        start.cast()
    }

Each entry is a pointer (either 32-bit or 64-bit depending on the table), so you need to deference a second time.
        let a = (*(*entry)).get_signature();
        let length = (*(*entry)).length;

With that change in place I can see what looks like valid XSDT data:
[ INFO]: uefi-test-runner/src/main.rs@088: xsdt or rsdt: signature XSDT, checksum : 0
[ INFO]: uefi-test-runner/src/main.rs@108: entry 0 : signature "FACP", length 244
[ INFO]: uefi-test-runner/src/main.rs@108: entry 1 : signature "APIC", length 144
[ INFO]: uefi-test-runner/src/main.rs@108: entry 2 : signature "HPET", length 56
[ INFO]: uefi-test-runner/src/main.rs@108: entry 3 : signature "MCFG", length 60
[ INFO]: uefi-test-runner/src/main.rs@108: entry 4 : signature "WAET", length 40
[ INFO]: uefi-test-runner/src/main.rs@108: entry 5 : signature "BGRT", length 56

You'll need to adjust the code a little for the 32-bit pointers.

Thanks a lot. But for me "addresses that point to other DESCRIPTION_HEADERs" means that the pointers point to the a DESCRIPTION_HEADER and not a pointer that point to a DESCRIPTION_HEADER.

You're correct, but the reason there's a double pointer involved is because the "outer" pointer is pointing to the list of pointers. So each inner pointer (the entries in the list) is indeed pointing to a DESCRIPTION_HEADER, and the outer pointer (the double pointer) only exists within the code, not part of the data structure as such.
TOP-LEVEL HEADER
entry_pointer -> HEADER  <-- get_entries_start() returns a pointer to this pointer
entry_pointer -> HEADER
entry_pointer -> HEADER


Forward multi-media messages

https://github.com/tucnak/telebot/blob/0edece47c90fc697ea89edf00f9b7e1330db3d9c/bot_test.go#L170-L175
How to forward multi-media messages like picture, voices, and videos?
I'm doing:
bot.Handle(tb.OnText, app.ForwardHandler)
bot.Handle(tb.OnAudio, app.ForwardHandler)
bot.Handle(tb.OnContact, app.ForwardHandler)
bot.Handle(tb.OnDocument, app.ForwardHandler)
bot.Handle(tb.OnLocation, app.ForwardHandler)
bot.Handle(tb.OnPhoto, app.ForwardHandler)
bot.Handle(tb.OnVenue, app.ForwardHandler)
bot.Handle(tb.OnVideo, app.ForwardHandler)
bot.Handle(tb.OnVideoNote, app.ForwardHandler)
bot.Handle(tb.OnVoice, app.ForwardHandler)

and then,
app.bot.Forward(chat, message)

But that's not working for me for picture, voices, and videos. How to fix it? Thx!

Following code:
b.Handle(tb.OnPhoto, func(m *tb.Message) {
    b.Forward(m.Chat, m)
})

works perfectly for me, maybe something is broken inside your ForwardHandler?

Oh, indeed! Thanks.


Crashing iplSimulatorRunReflections with baking

System Information

Steam Audio version: 4.5.2
Operating System and version: Windows 11

Issue Description
Reflections work fine in my project but it's compute-heavy, so I want to bake my reflections. I have one source that moves around in a circle. My listener is in the middle of circle. I want to perform a static listener bake for the reflections. I've generally followed the programmer's guide for baking, but changed it up a bit to use static listener instead of static sources. However, when I call iplSimulatorRunReflections after baking, it crashes with an "Access violation reading location 0x0000000000000000." error.
Steps To Reproduce

I have one probe array, one probe batch, and one identifier.
When I run the simulation, I set the simulator inputs baked flag and baked data identifier to the same one I used to initially bake.
Crashes when I call iplSimulatorRunReflections

One thing I noticed is that my call to iplReflectionsBakerBake is instantaneous. In the Steam Audio docs, it says that this function will block, so I assumed that it would take a while to simulate the reflections. However, that doesn't seem to be the case for me. I'm wondering if this may be the reason why running reflections is crashing after I bake.
I have also tested with the static source variation but I still get the exact same crash.

After debugging, I noticed that using the UNIFORMFLOOR generation type doesn't populate my probe array.
IPLMatrix4x4 boxTransform = {
       {{100.0f, 0.0f,   0.0f,   0.0f},
	{0.0f,   100.0f, 0.0f,   0.0f},
	{0.0f,   0.0f,   100.0f, 0.0f},
	{0.0f,   0.0f,   0.0f,   1.0f}}
};

IPLProbeGenerationParams probeParams {};
probeParams.type = IPL_PROBEGENERATIONTYPE_UNIFORMFLOOR;
probeParams.spacing = 2.0f;
probeParams.height = 1.5f;
probeParams.transform = boxTransform;

iplProbeArrayCreate(myContext, &myBake.probeArray);
iplProbeArrayGenerateProbes(myBake.probeArray, scene, &probeParams);
int count = iplProbeArrayGetNumProbes(myBake.probeArray); // Returns 0

On the other hand, if I use CENTROID, the probe array has a size of 1 (which is expected since CENTROID only generates one probe in the center of the box) and iplSimulatorRunReflections does not crash.
IPLMatrix4x4 boxTransform = {
	 {{100.0f, 0.0f,   0.0f,   0.0f},
	  {0.0f,   100.0f, 0.0f,   0.0f},
	  {0.0f,   0.0f,   100.0f, 0.0f},
	  {0.0f,   0.0f,   0.0f,   1.0f}}
};

IPLProbeGenerationParams probeParams {};
probeParams.type = IPL_PROBEGENERATIONTYPE_CENTROID;
probeParams.transform = boxTransform;

iplProbeArrayCreate(myContext, &myBake.probeArray);
iplProbeArrayGenerateProbes(myBake.probeArray, myScene, &probeParams);
int count = iplProbeArrayGetNumProbes(myBake.probeArray);  // Returns 1 and no crash

Wondering if this may be the root cause? The API wasn't populating the probes after generating them and so it wasn't baking anything?

Below are three main causes when no probes are generated:

Scene has no geometry added to it.
The Box doe not include any of the geometry in the scene.
All the probes generated (i.e. height of 1.5 above the floor/geometry in the box) lie outside the box itself.

All that said, we should probably do graceful failure here.

Thank you for your response.
I did a bit more testing, and my scene does have geometry in it. I don't think I'm understanding how the probes are generated for UNIFORMFLOOR. Could you explain to me how probe placement works in this example?
IPLMatrix4x4 boxTransform = {
	 {{100.0f, 0.0f,   0.0f,   0.0f},
	  {0.0f,   100.0f, 0.0f,   0.0f},
	  {0.0f,   0.0f,   100.0f, 0.0f},
	  {0.0f,   0.0f,   0.0f,   1.0f}}
};

IPLProbeGenerationParams probeParams {};
probeParams.transform = boxTransform;
probeParams.type = IPL_PROBEGENERATIONTYPE_UNIFORMFLOOR;
probeParams.spacing = 0.2f;
probeParams.height = 0.15f;

IPLBakedDataIdentifier identifier;
identifier.type = IPL_BAKEDDATATYPE_REFLECTIONS;
identifier.variation = IPL_BAKEDDATAVARIATION_STATICLISTENER;
identifier.endpointInfluence.center = listenerPosition; // (0, 0, 0)
identifier.endpointInfluence.radius = 100.0f;

Where is the center of the probe box in this example?
Does the box only extend in the positive axis directions in this example? i.e. You cannot have negative values for the box.

A 4 x 4 matrix basically can represent - scaling, rotation, and transform. This link has some basic information that might be helpful on specifying translation in a matrix.
To answer your questions.

Probe box center can be specified through the boxTransform.
The box is centered around the center and its size is defined by the scale.
Imagine dropping probes and them falling on geometry, wherever they hit the geometry is the floor. Height is the distance above the floor at which we place the probe.
As long as the triangles are part of the scene, this should be fine.

Thank you very much.

It seems like vertical rectangles can't be detected in the probe box because they have no z-component, assuming the rectangle only has x and y components. If I angle the rectangle so that it's slightly slanted and I make the probe spacing small enough, it'll detect it. Is this expected behavior?
Ideally, I would like both the source and listener to be able to move around. I tried using IPL_BAKEDDATAVARIATION_DYNAMIC for baking, but then my bakes don't seem to reflect anymore. Is there an additional parameter I should toggle? I am specifically trying to bake reflections, not paths.

If you have some minimal viable repro, I can take a quick look.  Especially, for issue#2 you mentioned above.
For issue#1 - We assume +Y is up and down is where we let probes fall to the ground. That can be changed by your boxTransform though. Here is how we do it internally, if that helps. So, you can adjust the matrix to rotate -Y to whatever your down vector is.

That said, it can be tricky to get this right. If possible, having a debugging tool for visualizing your geometry and probe box could help.

Thank you again for all the help.

Further elaboration on the vertical rectangle issue. My positive y is also up. The four points of the rectangle are (0, 0, 0), (1, 0, 0), (1, 1, 0), and (0, 1, 0) and I've added these vertices to the scene as two triangles. My probe box is centered at the origin and has a scale of (100, 100, 100). However, the geometry for the rectangle is not being detected during probe generation even though it exists in the scene. I know it exists in the scene because I am able to apply occlusion on this geometry.

Your spacing between probes is 2.0f and your rectangle is basically of size 1. You can either reduce the spacing between probes or increase the size of rectangle.

I've tested this with spacing as 0.1f. I've also tried changing the size of the rectangle to have side length of 10 for both sides. Neither worked.

Looks like your quad is in X-Y axis. Can you have a quad in X-Z axis and make sure it is 10 across in size and try again. If that does not work, feel free to share a repro code and I can take a look.

Yes, if I have a horizontal (i.e. X-Z) quad, I am able to detect probes for it. My root question was if it's possible to have vertical (i.e. X-Y or Y-Z) planes.

Are you trying to place probes on that thin edge?  Your down vector is -Y and if there is a just a thing plan, no probe is sort of going to land on that thin edge.
You can manually add probes using iplProbeBatchAddProbe  to a probe batch before baking reflections if iplProbeArrayGenerateProbes does not generate probes that you want.

Regarding my other question about dynamic baking, is dynamic baking only for path and not for reflections? Because I see in the source code that there is an assert for reflection baking.
https://github.com/ValveSoftware/steam-audio/blob/master/core/src/core/reflection_baker.cpp#L54

Oh yes, Nice catch. That is true, primarily because storing N^2 IRs requires a lot of memory.

I see, thank you very much for all the help.


Saved Power Feature

Checklist

[X] I have filled out the template to the best of my ability.
[X] This only contains 1 feature request (if you have multiple feature requests, open one feature request for each feature request).
[X] This issue is not a duplicate feature request of previous feature requests.

Is your feature request related to a problem? Please describe.
Usually you use a home assistant to calculate electricity costs.
I wanted to know what I was saving.
Describe the solution you'd like
It only works with fixed consumers!
An example:
I have a SAN DSM from Synology.
When switched on, it consumes 15 watts. I automatically shut down this device off remotely and then back on again with Wake on Lan. I usually use  a time scheduler and the @home function for this.
This is how I calculate the power savings by configuring off and unreachable with 15 watts and On with 0 watts. I call this entity powersaved.
Exactly the opposite of what Powercalc is for.
Describe alternatives you've considered
My idea is that this can be done 'nicely' as a feature in the GUI of powercalc.
Additional context
Another aspect would be standby, which I am not currently using. So the difference from on to standby to off
Quick note:
I've only been using Powercalc for a short time, so please bear with me if there is already a more elegant way to do this.

It's a nice idea, thought about it, but I don't think it's really feasible to implement.
Because it can only work when using power strategy fixed, and then even not when using a template or states power option.
This and all other strategies are dependant on more state than only on/off, so we cannot really know what we are actually saving in this scenarios. So it's not just a revert calculation behaviour.
So I don't think I'm going to implement this, sorry.

Thanks for your answer.
Yes, you are absolutely right, there would be too many dependencies, which would actually be a new plugin.
It was more of an idea that I wanted to share, which is what I used your great plugin for.
So you can closed it and thank for your great work, for the community.


[bug 1247219] Capitalize first letters in group edit submit buttons.

Let's fix this one as well:
https://github.com/akatsoulas/mozillians/blob/edit-groups-fix-typo/mozillians/jinja2/groups/includes/group_edit_access.html#L174

Coverage remained the same at 92.434% when pulling f13f40c2bf90c2a4ca175cb834b86fb2fda83ec7 on akatsoulas:edit-groups-fix-typo into cc6035640cd813588f854f3cf7e5cbc65ba963f9 on mozilla:master.

If we want to capitalize first letters on submit buttons I suggest we do it everywhere. Besides the one @johngian already pointed there is this: https://github.com/akatsoulas/mozillians/blob/edit-groups-fix-typo/mozillians/jinja2/groups/includes/group_edit_admin.html#L36

@johngian pr updated

r+

Coverage remained the same at 92.434% when pulling 82775fdb04873b8336ba6c3e0a7c186dd2f8a1ea on akatsoulas:edit-groups-fix-typo into cc6035640cd813588f854f3cf7e5cbc65ba963f9 on mozilla:master.


MAP-1458 PrisonService findByPrisonFilter without textSearch without Active

As I am working reducing gaps found in the Kover report.
Class overall coverage is increased by 85.7% to 86.1% with this commit
Before

After

added together PR 260


Find PETSc fails

I'm trying to use the CMake module to find the PETSc installation that I built on Ubuntu 19.04. I am setting both environment and CMake variables for PETSC_DIR and PETSC_ARCH but I still get the following error message:
-- PETSC_DIR: /petsc
-- PETSC_ARCH: mpi-real-double
CMake Error at /usr/share/cmake-3.13/Modules/FindPackageHandleStandardArgs.cmake:137 (message):
  PETSc could not be found.  Be sure to set PETSC_DIR and PETSC_ARCH.
  (missing: PETSC_INCLUDES PETSC_LIBRARIES PETSC_EXECUTABLE_RUNS) (found
  version "3.11.1")

Any ideas?

Please check CMakeFiles/CMakeOutput.log and CMakeFiles/CMakeError.log.
You might also prefer to use CMake's pkg-config support instead of FindPETSc.cmake unless you need legacy support (to very old versions of PETSc before PETSc.pc was distributed).


building tensorflow from source / general issue

Dear Tensorflow team,
I was facing a problem described here:
https://stackoverflow.com/questions/45859371/error-building-tensorflow-on-centos-7
In short words: In the documentation there is no any single word about bazel version need for specific version of Tensorflow. In particular, I could not build a Python package with the newest version of Bazel. Experimentally I found out that that the master version (r1.3) can be built successfully with Bazel not newer than 0.5.1. (with very old versions it also doesn't work).
So I am kindly asking to mention a Bazel version (as well as other dependencies, such as gcc, etc) to have successful build for the next releases.
I spent a lot of time to find a solution. Probably it will be useful experience for others.
Cheers,
Sergey

gunan@ is our bazel version check not working again?
@srvoin If the check is working properly, it should tell you something like "you need at least bazel X.X.X".
I think otherwise this is an unfortunate one-off, since the 0.5.3 bazel release was broken.
Leaving open only to check whether our check works.

@yifeif are you aware of any issues with bazel version check?

configure checks bazel against a minimum version of 0.4.5.
afaik Bazel 0.5.3 issue was resolved with #11871. I haven't seen the swig error mentioned in the SO link.

@martinwicke as @yifeif said it were checking against a minimum version and nothing about maximum. In my case it didn't work with any version higher than 0.5.1
It looks like there are some differences in environment settings within bazel scope... could be this a problem:
Release 0.5.3
Incompatible changes:

deprecate --python2_path and --python3_path flags for the
new --python_path flag; introduce --python_top flag for using
the new py_runtime rule.
....

I can't reproduce test with version 0.5.2 at the moment, but 0.5.3 and 0.5.4 don't work for python package generation.

Maybe we should fix the bazel version for releases, since it's breaking relatively quickly.
This should be fixed at head though. @srvoin can you verify that building head with the latest bazel works for you?

@srvoin we added a check to switch to --python_path for bazel >=0.5.3 at head, but it was not in 1.3 branch.
https://github.com/tensorflow/tensorflow/commit/6252d296040e8a2ff0c6c8668db55abe3e6f1416#diff-ade1d3e4b7c35655f854151d899df62b
Not sure if the python_path change is the cause of the issue you are seeing, but might be good to try the configure from master.

I am able to build clean the pip wheel target from head of master (as of 9.30a (pacific) today) via bazel build --config=opt //tensorflow/tools/pip_package:build_pip_package - no non-default values in the build configuration setup.
I'm using 0.5.3 Bazel:
autogenic:tensorflow loki$ bazel version
Build label: 0.5.3-homebrew
Build target: bazel-out/darwin_x86_64-opt/bin/src/main/java/com/google/devtools/build/lib/bazel/BazelServer_deploy.jar
Build time: Sun Aug 6 12:54:10 2017<PHONE_NUMBER>)
Build timestamp:<PHONE_NUMBER>
Build timestamp as int:<PHONE_NUMBER>
autogenic:tensorflow loki$

Thanks @quaeler. I will close this issue, since it will be resolved in 1.4 then.


deep nesteed yaml added for better usage

Hey @dl-tg ,
just added support for deep nesting in yaml file. earlier we could only do 1-2 level deep to have say nested folder within folder and a file.
We couldn't have folder within folder within folder and then file.
This PR solves all those issues making it a more useful tool for the developers

@dl-tg created a draft release ,feel free to make changes as deemed fit

Thank you for still maintaining the project. I don't really do programming anymore and ever since the last release my life changed significantly, which made me forget that I even had a github account in the first place.

Ahh i see, well i hope the significant change is in a positive direction!! 😄


Preloading links on hover or mousedown

Hello Alpine AJAX,
Following on from #99, SPA navigation via hx-boost is deliberately not provided by Alpine AJAX due to browser improvements and View Transistions which I am now convinced is a reasonable position to take.
However, would it be possible for Alpine AJAX to provide a link preloading mechanism to make page to page navigation even speedier? Something like:

HTMX Preload Extension
Turbo Prefetching Links on Hover

HTMX provides preloading on either mousedown or mouseover events, defaulting to mousedown. I think this is a very sensible default and to be honest is just about all this is required since mouseover could end up in disaster (too many requests, many not necessary).
The Alpine AJAX library author posted this on X:

There’s so many accessibility & state management foot guns in SPA navigation I’m not sure it’s the best approach. There’s cool alternative browser APIs like Speculative Loading & View Transitions that are easier to implement without so many downsides.

Speculative Loading whilst interesting does not seem as immediately useful is mousedown Link preloading.
I would be interested to know what position Alpine AJAX takes on this.
Best regards.

Hello @imacrayon,
See this comment in #99.

@imacrayon,
The speculationrules API  works in Chrome to preload links on hover.
I just added this to my app's layout:
    <script type="speculationrules">
    {
      "prerender": [{
        "source": "document",
        "where": {
          "href_matches": "/*"
        },
        "eagerness": "moderate"
      }]
    }
    </script>

Details are provided here.
Note, this does not work in Brave browser as noted here.
Safari and Firefox, not yet, but I do envisage this eventually being done.
I suggest do nothing, let browsers worry about preloading.
Best regards.


User LP Typing Event Models

Enhancement to add LP models for all events. Moreover, debugging options for themselves

Resolved. Closed


How to show img with gt boxes after Data Augmentation

Hi, for some reason. I want to show the img with gt infos after data augmentation, is there any func for this?
Thanks~

You can use this script.


no completion for try

i see foreach  is able to complet to
foreach ($variable as $key => $value) {
 code...
}

but not see try

duplicate of https://github.com/felixfbecker/php-language-server/issues/200


sluggish performance after reconnecting for a couple of times

I have a ViewController in which I'm using the WebSocket.
there is also a UITableView in myViewController.
I only initialize the WebSocket once, and then close and open if needed.
but every time I close the WebSocket, and open it again(without re creating any object) the tableView gets more sluggish. as if after 5 times it gets really slow.
the cpu doesn't show anything and ram usage is normal.
but still everything is super slow, even the OS it self
I'm using ipad mini2 and iOS 9.3
any guess what this could be?

What do you mean by sluggish? Is tableview scrolling slow, or the entire app? I would like to reproduce this issue. Could you provide the steps to create this problem?
If I were to guess then I would say that it has to do with some type of blocking occurring when the main runloop is in UITrackingRunLoopMode.

yes, the tableview becomes slow, any other tableviews that I add afterwards, by using the addSubVIew and addChildController still work smooth but the tableView inside the main UIViewController is really slow.
to reproduce the issue, I would suggest to just create a normal tableView with limited rows(mine had 5) and the have the websocket initialized inside the UIViewController that is also the datasource and the delegate for the tableView. then have the socket closed and opened several time(as much as it gets sluggish, for me it took 4-5 times) and it should to the trick.
like I said, I had the websocket initialized in my UIViewController which also was the dataSource and the delegate for table view.
if you close and open the websocket several times, the tableView will be really slow when dragging.
thanks for your time :-)

I'm not able to reproduce the issue. I created a new app with a UIViewController. I added a UITableView and set the delegate and dataSource to the UIViewController. I then displayed 1,000 table cells and set the textLabel to 1...1000.  I then started a websocket (I used the echo service in the README). Finally I set the ws.open(); ws.close() actions to the didSelectRowAtIndexPath.
I was able to close and reopen dozens of connections without seeing any sluggishness.
Could you create a simple test project that presents this issue and attach it to this thread?
Thanks a ton.

sure I'll create a sample project
and try to see at what point, and after what action exactly the table view goes sluggish
I'll post the results ASAP
thanks

I'm closing this issue because it's not reproducible.


Navigating to different tab in Docker Desktop loose context in OpenShift extension

Ways to reproduce:
1- Start the process to deploy an image to openshift
2- Jump into another tab from Docker Desktop (like containers)
3- Navigate back into the OpenShift Extension
Actual behaviour:

Context is lost and the user is back at the landing screen (intro to openshift). All selected information in the "Deploy to OpenShift" screen are lost.

Expected behavior:

The context is not lost and user is finding the information already filled previously

There is not API for saving state. We can try to save it to file and restore state from file when extension is activated, but there is no API to access to file system ether. Workaround is to use Exec and read/change file using platform specific commands like cat/echo. Am I missing something @fbricon?
IIRC Docker Dashboard even kills exec commands started by extension if extension is deactivated. In this case starting (1) and doing (2) while (1) is still in progress, will kill (1) without waiting for it to finish, potentially leaving resources create in OpenShift cluster in error state.

So #22 shows there's a really simple way to keep state during the session. As for DD killing exec commands, this orthogonal to this issue IMO, well have this problem no matter what

Resolved by using localState. The state persists between sessions. Navigate back to welcome page by clicking on OpenShift logo. SessionStorage persists values only when switching between extensions, activation of any non extension page resets the sessionStorage content.


Determine what post processing of geostrings should be.

Things like removing the string "block of ", or appending "Chicago Illinois" to the geostring if it doesn't already have it.
This will be somewhat geotagging provider dependent, but there's probably a lot of overlap in what transformations are beneficial for geotagging.

We have a first pass completed (see merged PR above).


Brew installation is very slow

Environment

ggshield version:
Operating system (Linux, macOS, Windows): Ubuntu & MacOS
Operating system version: Ubuntu 22.04 / macOS 13
Python version: 3.11.6 (for both linux and macos)

Describe the bug
When installing ggshield using brew, the process is very slow both on linux and macos. It can take up to 20 minutes on some machine. The average is around 4 to 5 minutes.
Steps to reproduce:

Remove package if already installed
Run command brew install gitguardian/tap/ggshield

Of course, I am ready to help you diagnose / improve the process if needed !

This is because brew rebuilds all dependencies, and that now includes the cryptography package, which takes a long time to build.
I don't think there is a way around this when using Homebrew, though I would be happy to be proven wrong.
Using pipx to install and update ggshield is faster because pipx uses pre-compiled wheels for all ggshield dependencies.

Closing this as we can't do much for now.


Serialize arrays and maps with known length

This is benchmark.sh for a few fixtures, before left and after
right, in operations per second:
canada packb:       423,  978
citm_catalog packb: 1064, 1839
github packb:       2533, 5007
numpy float64:      20,   41
twitter packb:      2620, 4463

Thank you so much!


Toggle for filtering results from known buckets

Hi!
I'm here from https://github.com/ScoopInstaller/Scoop/issues/4627#issuecomment-1012173950, and I wanted to ask whether a toggle/checkbox that could be added to only show search results from known buckets, somewhere in the middle here:

(I'm aware that the results are already sorted to show known buckets first, but still.)
Cheers!

Hey @rashil2000 ,
Thanks for your request. The information about the bucket source type is already stored in the index so it should be quite straight-forward. I will try to take a look and add this toggle.

Thanks for the update! Looking forward to it :)

I just pushed a new version with a toggle to search only in the official buckets

I'm open to your suggestions as I'm not fully happy with the placement/wording.

Hmm, I have a couple of thoughts:

Maybe we can put it in the center (above pagination links)
I think "Search only known buckets" is a more descriptive label

I tried to put the switch in the center but it doesn't look nice and is too distractive. I ended up with a placement on the right with your label and it looks way better.

What are your thoughts?

To me it looks asymmetrical (sparse on left and congested on right). But if there's no other way, it's understandable.

Yep, I prefer to keep the sorting/filtering options in the same area even if it makes the display slightly asymmetrical. It doesn't look too disturbing to me 👍

No worries, thanks for the quick turnaround on implementing this request ❤
Should I close this?

 I let you close this request...


New Defer API does not work with blues-stack

Have you experienced this bug with the latest version of the template?
Yes
Steps to Reproduce
Setup the project
Create account and sign in
Update loader in notes.tsx to return defer instead of json
Expected Behavior
I would expect that just changing to defer while still resolving the promise in the loader would still work without any other changes, since the data is being awaited in the loader.
Actual Behavior
The page errors with TypeError: Cannot read properties of undefined (reading 'length')
When looking at the value of data from const data = useLoaderData<typeof loader>();, it looks like this:
DeferredData {
  pendingKeysSet: Set(0) {},
  subscribers: Set(0) {},
  deferredKeys: [],
  abortPromise: Promise { <pending> },
  controller: AbortController { signal: AbortSignal { aborted: false } },
  unlistenAbortSignal: [Function (anonymous)],
  data: { noteListItems: [] },
  init: { headers: { 'content-type': 'application/json; charset=utf-8' } }
}

This shape seems wrong to me. You should have to access the real data via data.data, right?

Repro repo: https://github.com/justinwaite/blues-stack-defer

switching from json to defer is not enough, you have to wrap the deferred data in <Await />
See our guide on streaming: https://remix.run/docs/en/v1/guides/streaming#the-solution

@machour I would encourage you to reopen – If you look at the repo (and my instructions above), I'm awaiting the data in the loader, so it should be "instantly" available.
I did find the issue – it's the --bundle flag on the build:server script. If you remove it, everything works as expected.

If you want, feel free to remove await in the loader and add Suspense and Await in the component. You just don't need to go that far to see the error. The result is the same.

🤦🏼♂ sorry @justinwaite, reopened.

Any new info regarding this? I'm running into this as well but removing --bundle is not an option for me.

@justinwaite @JDouven Does it work with the default Remix template?
https://github.com/remix-run/remix/tree/main/templates/remix

Yes, that works. In my case I serve the app with express instead of remix-serve.
I used the following code:
import { defer } from '@remix-run/node';
import { Await, useLoaderData } from '@remix-run/react';
import { Suspense } from 'react';

export async function loader() {
  const testPromise = new Promise<string>((resolve) => {
    setTimeout(() => {
      resolve('Hello with a delay!');
    }, 3000);
  });

  return defer({
    instantly: 'foo',
    testPromise,
  });
}

export default function Index() {
  const data = useLoaderData<typeof loader>();
  return (

Welcome to Remix

Instant data:

{data.instantly}

Streamed data:

      <Suspense fallback={
Suspense fallback
}>
        <Await
          resolve={data.testPromise}
          errorElement={
Await errorElement
}
        >
          {(testPromise) =>
{testPromise}
}
        </Await>
      </Suspense>

  );
}

So I used the express template, and that works too. Then I changed the following to make it more closely match the relevant parts of blues-stack:

Rename server.js to server.ts and fix the imports (and npm install missing types)
Update package.json to:

"scripts": {
    "build": "remix build",
    "build:server": "esbuild --platform=node --format=cjs ./server.ts --outdir=build --bundle",
    "dev": "run-p dev:*",
    "dev:remix": "remix watch",
    "dev:build": "cross-env NODE_ENV=development npm run build:server -- --watch",
    "dev:server": "cross-env NODE_ENV=development node --inspect --require ./node_modules/dotenv/config ./build/server.js",
}

(inspired by the blues-stack package.json)
When I now run npm run dev my example stops working. The frontend stops receiving loader data completely. Neither the 'instant' data nor the streamed data is visible.
Also in this small example, removing --bundle from the build:server script is a workaround. However as I said, that is not an option in the larger app I want to use this in.
Let me know if you need more info.

This looks to be due to a failing instanceof check across a CJS/ESM boundary when using --bundle.  We'll get a fix added for this but in the meantime if you'd like to get past it via patch-package you just need to replace the single check in @remix-run/router:
  // before
  if (result instanceof DeferredData) {

  // after
  if (result &&
      typeof result === "object" &&
      typeof result.data === "object" &&
      typeof result.subscribe === "function" &&
      typeof result.cancel === "function" &&
      typeof result.resolveData === "function") {

You should patch this in all 3 build files to be safe:

@remix-run/router/dist/router.js
@remix-run/router/dist/router.cjs.js
@remix-run/router/dist/router.umd.js

@brophdawg11 I tried this and I can confirm this fixes the issue! Thanks!

The fix for this is out in 1.15.0-pre.1 if anyone wants to give that a spin.  We should have a stable 1.15.0 out soon as well.


fix: ui to 0.413.7

fixes action scorer entity substitution

:tada: This PR is included in version 0.344.1 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


feat(safe-kit): Multisig

What it solves
Resolves https://github.com/safe-global/safe-core-sdk/issues/868, https://github.com/safe-global/safe-core-sdk/issues/880
How this PR fixes it
Transactions and on-chain messages support

send({ transactions, options }) / sendMessage(message): Creates, signs and execute transactions or on-chain messages

1/N new Safe: Deployment and transaction execution
1/N existing Safe: Transaction execution
M/N new Safe: Deployment and transaction proposal
M/N existing Safe: Transaction proposal

confirm(safeTxHash): To confirm pending transactions

M/N existing Safe: Add signature and transaction execution if the threshold is matched

getPendingTransaction(): Get the list of transactions that must be confirmed

Off-chain messages support

sendMessage({ transactions, options }): Creates, signs and stores off-chain messages using the Transaction service

1/N new Safe: Deployment and save message
1/N existing Safe: Save message
M/N new Safe: Deployment and save message
M/N existing Safe: Save message

confirmMessage(messageHash): To confirm pending off-chain messages

M/N existing Safe: Add signature to the off-chain messages. If the threshold is matched the message is confirmed and valid

getPendingMessages(): Get the list of messages that must be confirmed

SafeOperations support

sendSafeOperation({ transactions, options }): Creates, signs and stores SafeOperations using the Transaction service. When the SafeOperation has enough signatures it sends it to the bundler. Supports paymaster usage.

1/N new Safe: Deployment and send UserOperation to the bundler
1/N existing Safe: Send UserOperation to the bundler
M/N new Safe: Deployment and stores the SafeOperation
M/N existing Safe: Save SafeOperation.

confirmSafeOperation(safeOperationHash): To confirm pending SafeOperations

M/N existing Safe: Add signature to the SafeOperation. When the threshold is reached the SafeOperation is being sent to the bundler

getPendingSafeOperation(): Get the list of messages that must be confirmed

All the flows above were tested by using the included playgrounds:

send-transactions
send-on-chain-message
send-off-chain-message
send-safe-operation

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

2 of 9 (22.22%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.4%) to 77.298%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

packages/safe-kit/src/SafeClient.ts
2
9
22.22%

Totals

Change from base Build<PHONE_NUMBER>:
-0.4%

Covered Lines:
763

Relevant Lines:
920

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

2 of 9 (22.22%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.4%) to 77.298%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

packages/safe-kit/src/SafeClient.ts
2
9
22.22%

Totals

Change from base Build<PHONE_NUMBER>:
-0.4%

Covered Lines:
763

Relevant Lines:
920

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

2 of 33 (6.06%)  changed or added relevant lines in 1 file are covered.
4 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-2.9%) to 74.826%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

packages/safe-kit/src/SafeClient.ts
2
33
6.06%

Files with Coverage Reduction
New Missed Lines
%

packages/safe-kit/src/SafeClient.ts
4
8.0%

Totals

Change from base Build<PHONE_NUMBER>:
-2.9%

Covered Lines:
754

Relevant Lines:
939

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

2 of 36 (5.56%)  changed or added relevant lines in 1 file are covered.
4 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-3.1%) to 74.653%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

packages/safe-kit/src/SafeClient.ts
2
36
5.56%

Files with Coverage Reduction
New Missed Lines
%

packages/safe-kit/src/SafeClient.ts
4
7.55%

Totals

Change from base Build<PHONE_NUMBER>:
-3.1%

Covered Lines:
754

Relevant Lines:
942

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

10 of 50 (20.0%)  changed or added relevant lines in 2 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-3.0%) to 74.676%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

packages/safe-kit/src/SafeClient.ts
3
43
6.98%

Files with Coverage Reduction
New Missed Lines
%

packages/safe-kit/src/SafeClient.ts
1
8.77%

Totals

Change from base Build<PHONE_NUMBER>:
-3.0%

Covered Lines:
762

Relevant Lines:
953

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

16 of 56 (28.57%)  changed or added relevant lines in 2 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-2.8%) to 74.572%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

packages/safe-kit/src/SafeClient.ts
3
43
6.98%

Files with Coverage Reduction
New Missed Lines
%

packages/safe-kit/src/SafeClient.ts
1
8.77%

Totals

Change from base Build<PHONE_NUMBER>:
-2.8%

Covered Lines:
780

Relevant Lines:
976

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

16 of 59 (27.12%)  changed or added relevant lines in 2 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-3.1%) to 74.296%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

packages/safe-kit/src/SafeClient.ts
3
46
6.52%

Files with Coverage Reduction
New Missed Lines
%

packages/safe-kit/src/SafeClient.ts
1
8.06%

Totals

Change from base Build<PHONE_NUMBER>:
-3.1%

Covered Lines:
780

Relevant Lines:
979

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

16 of 59 (27.12%)  changed or added relevant lines in 2 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-3.1%) to 74.296%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

packages/safe-kit/src/SafeClient.ts
3
46
6.52%

Files with Coverage Reduction
New Missed Lines
%

packages/safe-kit/src/SafeClient.ts
1
8.06%

Totals

Change from base Build<PHONE_NUMBER>:
-3.1%

Covered Lines:
780

Relevant Lines:
979

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

16 of 59 (27.12%)  changed or added relevant lines in 2 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-3.1%) to 74.296%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

packages/safe-kit/src/SafeClient.ts
3
46
6.52%

Files with Coverage Reduction
New Missed Lines
%

packages/safe-kit/src/SafeClient.ts
1
8.06%

Totals

Change from base Build<PHONE_NUMBER>:
-3.1%

Covered Lines:
780

Relevant Lines:
979

💛  - Coveralls

There is an issue with the confirmations when using the safeOperations extension => https://github.com/safe-global/safe-transaction-service/pull/2119
The extension only works with threshold 1 Safe account right now until the issue is fixed

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

22 of 64 (34.38%)  changed or added relevant lines in 2 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-3.0%) to 74.465%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

packages/safe-kit/src/SafeClient.ts
3
45
6.67%

Files with Coverage Reduction
New Missed Lines
%

packages/safe-kit/src/SafeClient.ts
1
8.2%

Totals

Change from base Build<PHONE_NUMBER>:
-3.0%

Covered Lines:
786

Relevant Lines:
984

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

22 of 64 (34.38%)  changed or added relevant lines in 2 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-3.0%) to 74.465%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

packages/safe-kit/src/SafeClient.ts
3
45
6.67%

Files with Coverage Reduction
New Missed Lines
%

packages/safe-kit/src/SafeClient.ts
1
8.2%

Totals

Change from base Build<PHONE_NUMBER>:
-3.0%

Covered Lines:
786

Relevant Lines:
984

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

22 of 64 (34.38%)  changed or added relevant lines in 2 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-3.0%) to 74.465%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

packages/safe-kit/src/SafeClient.ts
3
45
6.67%

Files with Coverage Reduction
New Missed Lines
%

packages/safe-kit/src/SafeClient.ts
1
8.2%

Totals

Change from base Build<PHONE_NUMBER>:
-3.0%

Covered Lines:
786

Relevant Lines:
984

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

22 of 64 (34.38%)  changed or added relevant lines in 2 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-3.0%) to 74.465%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

packages/safe-kit/src/SafeClient.ts
3
45
6.67%

Files with Coverage Reduction
New Missed Lines
%

packages/safe-kit/src/SafeClient.ts
1
8.2%

Totals

Change from base Build<PHONE_NUMBER>:
-3.0%

Covered Lines:
786

Relevant Lines:
984

💛  - Coveralls


[Research] Smart Contract Audit questions and logistics

Audit questions

How much does one cost?
What do we need to prepare?
Who are good auditors to use?

How would adding a Migration feature that would allow an admin to migrate funds to another contract for a new round or contract update.


[rds]: Add missing selectors

Description of your changes

Add dbParameterGroupNameSelector to ClusterInstance
Add parameterGroupNameSelector to Instance

I have:

[x] Run make reviewable test to ensure this PR is ready for review.

How has this code been tested
uptest below

/test-examples="examples/rds/instance.yaml"

It looks like parameterGroupNameSelector to Instance was added in https://github.com/upbound/provider-aws/pull/1037 so I am rebasing this PR leaving only dbParameterGroupNameSelector to ClusterInstance part

/test-examples="examples/rds/clusterinstance.yaml"

/test-examples="examples/rds/clusterinstance.yaml"


[WIP] Fix broker_url parameter for Update

The update only works with broker_url. The parameter broker-url does not work.
Pull Request Template
Closes #78
Motivation
Wrong Documentation
Approach
Correct documentation
Pull Request status

[x] Documentation Changed
[ ] Code for documentation generation changed

 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.

Coverage remained the same at 84.051% when pulling b6634af6b468bf137dbac81768094c7a30a2981c on thwiegan:patch-1 into f59c3d4f641f392e55b74a19c677e8669487bd1d on Peripli:master.


jocode limitations

so ive noticed some weird jocode bugs with branches that extend from thick trunks. it seems to be when the tapering value for growth parameters are set fairly high.
in this case, im playing around with baobab trees from dt forestry (ive also been modifying some growth parameters for some other trees where i began to notice this)
is there any way to fix this? since generation relies on jocodes, all baobab trees (or anything ive made that uses rather thick branches) look really ugly in the world unless theyre grown by a seed manually.
heres an example, i copy the jocode with the wand from a fully grown baobab tree, the trunk and branches look fine.

paste it elsewhere and it looks like this:

sometimes branches on the tree will also look odd, you can see the heart of the tree on some branches which causes a lot of z-fighting. this to me seems like some issue with tapering where if the branches are too thick, the jocode doesnt like them.
example, before being a grown tree and after being the jocode pasted tree:

edit: ive talked to the devs for dynamic trees and maxhyper has said he fixed this in 1.16 but probably wont be implemented for 1.12 due to no longer being supported. perhaps it could be backported?
edit 2: this might be of some use

ive managed to fix this, see pull request: https://github.com/TerraFirmaCraft-The-Final-Frontier/DramaticTrees/pull/13


🐛 can't refresh the stack if an EnvironmentVariable is removed

Hi,
I've provisioned an envvar via pulumi-vercel, then I removed it manually via the Vercel console
When I pulumi refresh, I get this error:
error: Preview failed: Expected an Object, got: {<nil>}

I've noticed this as well - it's an issue with the upstream provider I'll try and fix it there


HVAC status output processed wrong on Captur

Using this wonderful project, I discovered this anomaly on a Captur:
KamereonVehicleHvacStatusData(raw_data={'socThreshold': 20.0, 'hvacStatus': 'off', 'internalTemperature': 21.0, 'lastUpdateTime': '2023-09-23T14:41:08Z'}, lastUpdateTime='2023-09-23T14:41:08Z', externalTemperature=None, hvacStatus='off', nextHvacStartDate=None, socThreshold=20.0)
I observe that the raw_data contains different items than the parsed data.
If you confirm this is not as intended - maybe this is just a difference between models of cars - then I can work on a PR.
Please DO give some guidance what would be the intention.

Renault API is fully undocumented, and the features for different cars vary.
In this particular case, I suggest that you add a PR to add the missing fields (don't remove the absent fields as they are present on other cars) together with a "real" fixture file and adjusted tests.

Thanks for the advice. I'll have a go at it some time in the future.

Not likely, sorry about that...


Documentation or UI: Unclear purpose of raw and default views

I have been using neurosift for a while and I still don't understand what is the purpose of each of the windows on the top:

I think that widgets, specification and annotations are kind of clear enough by themselves but default vs raw is not.
Perhaps a tooltip with the purpose as documentation would be useful?

I have added tooltips.

The new ones look great. Thanks a bunch!


[5.1] Add some fluent methods to routes (again)

Resubmit of #10200 against 5.1
I'm back!
This is another attempt at #5859, now with new and improved name lookups.
This enables us instead of doing this:
$router->get('/', ['uses' => 'Controller@method', 'as' => 'route.name', 'middleware' => ['SomeMiddleware']]);

to do this:
$router->get('/', 'Controller@method')->name('route.name')->middleware('SomeMiddleware');

Which the folks in #5859 all seemed to agree is much nicer.
This time I've added a one time refresh of the name look-up table in RouteCollection, to ensure that it's up-to-date and still blazingly fast, even if some routes had names defined fluently. This refresh, in my testing, adds ~0.7ms, or ~0.0007 of a second, for 1,000 routes.

:dancer: :dancer:  :dancer:

:dancer: :dancers:

Nice work! It always bothered me to use that ugly array syntax for defining route names and middleware! It's much more readable now!


JkLog loose level offst

Only the first logs loose offset then it goes ok.
Just build jerkar itsell within the ide to reproduce
|  |   \ Done : 27 file(s) copied. in 0.291 seconds.
|  |  Run JUnit tests ...
Resolving dependencies with specified scopes [compile] ...
|  |  |  |  31 modules leading to 30 artifacts.
|  |  |   \ Done in 1.335 seconds.
|  |  |  Resolving dependencies with specified scopes [test] ...
|  |  |  |  31 modules leading to 30 artifacts.

It appears to be related to test run
|  |   \ Done : 27 file(s) copied. in 0.216 seconds.
|  |  Run JUnit tests ...
Creating zip file : C:\Users\ANGIBA~1\AppData\Local\Temp\oooo5029913572735843950.zip ...  \ Done in 0.282 seconds.
|  |  |  Resolving dependencies with specified scopes [compile, provided] ...
|  |  |  |  3 modules leading to 2 artifacts.
|  |  |   \ Done in 0.229 seconds.


Task: Finalize the publish registry API MVP

There has been some work put in publish registry API. What still needs to be done to finalize this and start implementing an MVP?

Blocking #39 #38 #34 #33

A draft by @lann available at https://hackmd.io/@lann/rk1r8alfj

Closing, since there's already tooling doing this (https://github.com/bytecodealliance/registry/issues/44)


rendering overlays = multiple canvas with the same id

not sure if this is configurable, couldn't track it down but seems like a bug

the recommended practice would be to create one canvas with multiple layers sharing the same gl context for the rendering.

@gnavvy Does there need to be an ID at all? React's refs should allow you to reference the DOM element, and it seems like it isn't being used anyways. I removed the ID on my fork and everything works great.

by ID were you referring to the layer ID? it is used for:

matching layers so as to check if there's a change in data/viewport, if not, skip the calculation;
distinguish programs and geometries for multiple layers of the same type.

@gnavvy RE: canvas element ID

you can specify canvas ID and style for each DeckGLOverlay for customized styling now, from v2.4.4.


Non-Enum types require ActiveEnum trait when used a primary keys.

Description
See here: https://github.com/SeaQL/sea-orm/discussions/1807
It's been unanswered for more than a year and it isn't really a discussion and more like a bug.
In my case I have a wrapper type (named Snowflake) around i64 rust type.
So for every trait that needs to be implemented I can just call the appropriate method on i64 but ActiveEnum is not implemented for it but I can still use it as a primary key?
Steps to Reproduce

Create a custom type and implement all necessary traits mentioned here
Use that type inside a sea-orm entity as a primary key
try cargo build

Expected Behavior
I dont need to implement ActiveEnum traits for non-enum types.
Actual Behavior
It requires me to do so.
Reproduces How Often
Everytime
Workarounds
Non / Custom types are unuseable here.
Reproducible Example
See discussion
Versions
1.1.2

I would like work on this, if that's okay.


Allowing lodash style selectors for findFunction and findParam

Can now specify a callback function or minDate/maxDate by passing a set of
object keys into the picker, allowing lodash style selectors for findFunction and findParam.

For example, pass a callback function such as 'objectA.objectB.functionName' to search scopes for an object
called objectA, containing an object called objectB, containing a
function called functionName
Resolves #211

Can now specify the first day of the week for the month view. See my demo (bottom left column) for the new feature in action.

Can select values 0 (Sunday) - 6 (Saturday)
Defaults to Sunday.
Uses default if provided value is outside the valid range.
Resolves #214

Extra commits should resolve #214


feat: add libp2p perf blog post

@mxinden are you still interested in getting this published? I'm sorry that this got lost in the shuffle. I'm going through the blog backlog now and getting all of these PRs landed or closed. Let me know what you'd like to do with it.

I will not continue to work on this.
Obviously, anyone please feel free to continue the work https://github.com/libp2p/test-plans/tree/master/perf.
In the meantime, I am closing here.


HGET implementation in Redisson

Apologies if this is something simple that I've missed out
Have been trying to figure this out for a while but can't.
Trying to simply get the value associated with field in the hash stored at key as described in the documentation for HGET
The code used is
RedissonClient redisson = Redisson.create();
RMap<String, String> map =  redisson.getMap("myhash");
System.out.println(map.get("a"));

However, it seems like this implementation fetches all values stored in all fields in 'myHash' instead of just the value stored in field 'a'. Unsure if this fetching of values happens during redisson.getMap ( More likely ) or map.get()
Expected behavior
Redisson issues only one HGET and retrieves only the value of field 'a' in hash 'myhash'
Actual behavior
Redisson seems to be fetching values of all fields in 'myhash'
Steps to reproduce or test case
redis-cli
HSET myhash a valueofa
HSET myhash b valueofb
HSET myhash c valueofc

java
RedissonClient redisson = Redisson.create();
RMap<String, String> map =  redisson.getMap("myhash");
System.out.println(map.get("a"));

Redis version
4.0.9
Redisson version
3.10.7
Redisson configuration
Default ( Testing on localhost )

Redisson seems to be fetching values of all fields in 'myhash' How did you manage to observe this behaviour, may I ask?

Redisson seems to be fetching values of all fields in 'myhash' How did you manage to observe this behaviour, may I ask?

I inspected the value of map using IntelliJ's debugger and found all fields and their values in it

In your case, it is the debugger initiated the call and retrieved all the value from redis.

Why redisson not suport  fetach the value stored in field?

@MingJunDuan
Which command do you mean? You can find it here https://github.com/redisson/redisson/wiki/11.-Redis-commands-mapping

@MingJunDuan
Which command do you mean? You can find it here https://github.com/redisson/redisson/wiki/11.-Redis-commands-mapping

hget, RMap.get("fieldKey").getAsync(), it retrieved all the hash field.

RMap.getAsync() uses HGET command


Remove "Keptn on microk8s" docs

In my opinion, the docs regarding Installing Keptn on Microk8s are misplaced for multiple reasons.

This does not have anything to do with "Operating Keptn", but it's more about "how to setup Microk8s" (on MacOS and Ubuntu).
Our installation instructions provide several different options (e.g., for K3s, Minikube, GKE, ...) - but not for Microk8s.

The following instructions should really be all that's needed (from Keptn core perspective):
sudo snap install microk8s --classic
sudo iptables -P FORWARD ACCEPT
sudo microk8s.enable dns
sudo microk8s.enable dns ingress
sudo microk8s.enable storage
sudo /snap/bin/microk8s.config > kubeconfig
export KUBECONFIG=./kubeconfig
kubectl apply -f https://raw.githubusercontent.com/google/metallb/v0.8.3/manifests/metallb.yaml

followed by
keptn install ...

Suggestion

[ ] Remove https://keptn.sh/docs/0.7.x/operate/microk8s/ for 0.7.x and 0.8.x
[ ] Add simple instructions (see above) into https://keptn.sh/docs/0.7.x/operate/install/ for 0.7.x and 0.8.x

Like the proposed idea since it reduces the menu items under "Operate Keptn".

This has already been done for 0.8.x.
I don't see a reason to touch the docs for 0.7.x as it has already been deprecated.


have different configurations for different filetypes

I may be weird, but have different preferences for how to show indentation on different languages (say, as vertical bars on something like Rust, but as dots on something like Haskell)
I at least tried with filetype and filetype_exclude and couldn't really get it to work
Is there a way to do it right now, or would it need to be an addition? If so, would be interesting being something like
require('indent_blankline').setup_ft({
    rust = { --[[...]] },
    haskell = { --[[...]] },
    default = { --[[ everything else ]] }
})

or maybe
require('indent_blankline').setup({
     --[[ normal config ]],
    ft_override = {
        rust = { --[[...]] },
        haskell = { --[[...]] }
    }
})

You can do this by setting buffer local variables.
eg. put vim.b.indent_blankline_char = "H" in ftplugin/lua.vim and the indent char for lua files will be H


Can't copy 80080 bytes of a tensor into another with 40040 bytes buffer. [[{{node _arg_input_1/BottleneckInputPlaceholder_0_0}}]]

I'm trying to run this solution
https://github.com/dotnet/machinelearning-samples/tree/master/samples/csharp/getting-started/DeepLearning_ImageClassification_Training
with GPU , but im getting this error.
I have tried with solution created from scratch, but after i got the same error i tried with Microsoft's example from link above.
"Can't copy 80080 bytes of a tensor into another with 40040 bytes buffer.
[[{{node _arg_input_1/BottleneckInputPlaceholder_0_0}}]]"
Byte size depends from BatchSize value.
This is for BatchSize = 10
in this line of code
ITransformer trainedModel = pipeline.Fit(trainDataView);
This are versions of nuget packages that I'm using

I have setup environment following this article
https://github.com/dotnet/machinelearning/blob/master/docs/api-reference/tensorflow-usage.md
I have CUDA 10.0 and CUDDN cudnn-10.0-windows10-x64-v<IP_ADDRESS>
GPU geforce GTX 950M with 2GB GDDR5
OS Windows 10 pro 64bit
framework core 3.1
Does anyone have any idea why this might happening?
Thanks

I have fix my issue by updating to at this moment latest version of nuget packages


NOISSUE - Migrate gocoap library from v2 to v3.3

What type of PR is this?
This is a feature because it migrates the version of the coap library from v2 to v3.3

What does this do?

Migrates the plg-dev\go-coap library from v2 to v3.3
Improves CLI by using spf13/cobra

Which issue(s) does this PR fix/relate to?

Resolves https://github.com/absmach/magistrala/issues/2203
Have you included tests for your changes?
Yes, I have included tests for my changes.

Did you document any new/modified feature?
No,

Notes
No,

Please resolve conflicts.

Server disconnection idle client leads to new problem The client doesn't know the disconnection, and client again send disconnection request to server So we are getting error in server like {"time":"2024-05-16T10:17:46.594334142Z","level":"WARN","msg":"Unsubscribe failed to complete successfully","duration":"31.172803ms","channel_id":"dff59205-90d6-430d-9903-a611cd401bb3","error":"not subscribed"}

It looks like the problem is that the subscribed client isn't aware of the broken subscription, which makes sense due to the connectionless nature of the underlying UDP. We can merge this one, but @arvindh123 please open a ticket to remind us to double-check this.


URL encoding issue when trying to navigate to a resource from Query Lists

This issue only occurs in FIREFOX and is created from the fetch resource action in the RESOURCE VIEW (not the URL location change)

this turns out not to be an encoding issue (Firefox displays all urls as decoded in the devtools)
instead its related to a 500 graph error on the nexus server-side https://github.com/BlueBrain/nexus/issues/652

proof cuRL:
{"@context":"https://bluebrain.github.io/nexus/contexts/error.json","@type":"InternalError","reason":"The system experienced an unexpected error, please try again later."}🌺¸ pirman@bluebrain370 ~ $ curl 'https://sandbox.bluebrainid.ac%2Finstitutes%2Fgrid.1007.6' -H 'User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.14; rv:66.0) Gecko/20100101 Firefox/66.0' -H 'Accept: text/vnd.graphviz' -H 'Accept-Language: en-GB,en;q=0.5' --compressed -H 'Referer: https://sandbox.bluebrainnexus.io/web/demo/kenjinp/resources/http%3A%2F%2Fwww.grid.ac%2Finstitutes%2Fgrid.1007.6'

The payload contains some empty objects, like:
{
   "field": ""
}

This causes the DOT format of the resource to crash (Id was empty). DOT does not support empty objects.


[Tooltip] Issue

The example in the documentation doesn't work on click, but works on keyboard focus.
I tried this in Safari, Chrome, and Firefox on MacOS Ventura 13.5.

That's the expected behavior see https://github.com/radix-ui/primitives/issues/955#issuecomment-960610209

Ah makes sense, I was conflating the expected behavior with that of a popover. Thanks!
HOWEVER (sorry for not mentioning in my original post), it also doesn't respond to mouse hover either. Wouldn't this be a problem?
From the ARIA Authoring Practices guide:

A tooltip is a popup that displays information related to an element when the element receives keyboard focus or the mouse hovers over it

HOWEVER (sorry for not mentioning in my original post), it also doesn't respond to mouse hover either. Wouldn't this be a problem?

Hm, seems to work here. There is a slight delay to open the tooltip, so keep your mouse over long enough

Got it!
I think what was happening is that my clicking wasn't merely ignored - the clicking was actively blocking the hover from being understood. Verrryyy interesting. Thanks!
Closing since this is more user error than anything, though I think it'd be a nice improvement if clicks could somehow be ignored without it preventing the hovering from working.


Make file changes watchable

As per docs I added addDependency to ensure the files are watched when using webpack --watch. Not sure if more is needed, but it works for me with this change.

@WearyMonkey  can you accept this PR?

@Anthropic @feitzi Not sure why this is required. If the resource is being processed by the loader chain webpack must of already added it as a dependency correct?
Can you please provide more details in what is failing for you?

@WearyMonkey did you check the link I added? It says a loader must call it for --watch to work. Without it, I can update my HTML templates as much as I like and it won't compile again. Webpack won't watch any file you include only from within the loader as it assumes most of the files are for working on the load and not part of it I suspect.

@Anthropic

If a loader uses external resources (i. e. by reading from filesystem), they must tell about that. This information is used to invalidate cacheable loaders and recompile in watch mode.

The loader doesn't read from the filesystem are use any external resources. It simply transforms an existing dependency on the loader chain. If what you're saying is true, every loader would need to call addDependency.
Willing to take a look if you send me a sample project?

@WearyMonkey If you include the file from within the webpack config file, no issue, if I include it from a file, loaded by the config, it fails to see changes making --watch worthless when updating templates.
It may be due to webpack being a bit late to the es6 import syntax party with webpack 2, I don't know, I haven't tried it since without the line to be honest. But based on @feitzi 's recent comment I'm assuming it still doesn't work without it.
Believe me, --watch totally works with this single line addition and totally doesn't work without it in certain cases. It also can't do any harm at all having it regardless as all it does is keeps track of dependencies, if it is already in the dependency list it wont change anything anyway.
my usage relying on the change
Within the project I am working on: https://github.com/json-schema-form/angular-schema-form-bootstrap
// in script src/bootstrap-decorator.js
import actionsTemplate from './bootstrap/actions.html';
import arrayTemplate from './bootstrap/array.html';

// in webpack.config.js
      rules: [{
        test: /\.html$/,
        use: [{
          loader: 'ngtemplate-loader',
          options: {
            relativeTo: path.join(__dirname, 'src')
          }
        }, 'html-loader'],
        exclude: /(index)/
      }]

addDependency usage
TypeStrong TypeScript
https://github.com/TypeStrong/ts-loader/blob/master/src/index.ts#L130
Here's a few other random ones that use it from the loader list on their site
https://github.com/bendman/soy-loader/blob/master/index.js
https://github.com/akim-mcmath/ejs-html-loader/blob/master/src/cache.js
https://github.com/yuri-karadzhov/glsl-template-loader/blob/master/index.js

Thanks, I'll take a deeper look. I've been using with webpack-dev-server and webpack 1, it's possible webpack 2 --watch behaves differently.

Ok I tried to reproduce your problem here: https://github.com/WearyMonkey/ngtemplate-watch-test
but was not able too.
The typescript loader manually adds dependencies because typescript can detect dependencies webpack cant.
The soy loader is adding a dependency to the extra soyUtils.js dependency, which is external to the original loaded dependency.
Looks like the glsl and ejs-html loaders are resolving their own import graphs which webpack would not be able to detect.
Take a look at html-loader. It has no addDependency because just like ngtemplate-loader it is transforming an existing dependency on the dependency chain.
The fact that you brought up ts-loader suggests you are using it? ts-loader Clears dependencies and manages them manually. It's likely that the html dependency is being lost by ts-loader.
Can you send me a sample project that is not working?

@WearyMonkey I'm pleased to report that I upgraded from Webpack 2.0 to 2.2 which wasn't out when I made this PR, and without the function call it works, must have been a webpack issue that they resolved. I guess the function call was like an emergency backup to force it into the cache when they had a bug that missed it. Sorry for the trouble mate, love your work!
@feitzi which Webpack version are you on where it isn't working, 2.0 also?

@Anthropic great! glad it worked out. Thanks for taking the time on this PR, greatly appreciated.


Latest Changes to work with aws-ddk 1.0.0b1

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@anmolsgandhi Ok if we close this and keep everything at release-1.0.0 until the major version release? I'll add a link to the current readme to the branch for people interested in using the prereleases.

Yes this works actually ! lets close this for now


Obsoletion request: polysome binding & regulation children

Please provide as much information as you can:
Following up on https://github.com/geneontology/go-ontology/issues/24651

GO term ID and Label
'polysome binding' http://purl.obolibrary.org/obo/GO_1905538
'regulation of polysome binding' http://purl.obolibrary.org/obo/GO_1905696
'negative regulation of polysome binding' http://purl.obolibrary.org/obo/GO_1905697
'positive regulation of polysome binding' http://purl.obolibrary.org/obo/GO_1905698

Reason for deprecation Put an x in the appropriate box:
Outside the scope of GO; this is a high level complex of ribosomes, and doesn't provide any additional information compared to 'GO:0043022 ribosome binding'

"Replace by" term (ID and label)

GO:1905538 polysome binding >> replace by GO:0043022 ribosome binding
regulation of binding need to be looked at; I cannot suggest a replacement

"Consider" term(s) (ID and label)
Suggestions for reannotation

Are there annotations to this term?

How many EXP: 20

Are there mappings and cross references to this term? (InterPro, Keywords; check QuickGO cross-references section)

Is this term in a subset? (check the AmiGO page for that term)

Any other information

Checklist for ontology editor
Check term usage and metadata in Protégé

[ ] check term usage in the ontology
[ ] check internal mappings: RHEA, EC, MetaCyc
[ ] check subset usage
[ ] check taxon constraints

Check annotations

[ ] create Annotation Review spreadsheet

Annotation Review template: https://docs.google.com/spreadsheets/d/1izV3q1jnq_Ybg5MJ53O_b5BCTkGtjl_Fi-cOg3-7eBE/edit#gid=0

Instructions: https://wiki.geneontology.org/Annotation_Review_Request

[ ] create annotation review ticket: https://github.com/geneontology/go-annotation/issues/new?assignees=&labels=annotation+review&template=annotation-review-for-obsoletion.md&title=Review+annotations+to+[INSERT+GO+TERM]
[ ] find external mappings (via IEAS), include in obsoletion notice
[ ] tag curators in Annotation Review ticket. Responsible annotator for each group: https://github.com/geneontology/go-site/blob/master/metadata/group-contacts.csv

Notification

[ ] create obsoletion announcement
[ ] announce to GO friends<EMAIL_ADDRESS>[ ] paste the text in the ontology ticket

Shouldn't this be included in the pending discussion? https://github.com/geneontology/go-annotation/issues/4750

2 TAIR annotations removed.

Dear all,
The proposal has been made to obsolete
GO:1905538 polysome binding
GO:1905696 regulation of polysome binding
GO:1905698 positive regulation of polysome binding
GO:1905697 negative regulation of polysome binding
The reason for obsoletion is that the terms are outside the scope of GO.

This term is not present in any subset.
Annotations can be viewed here: https://docs.google.com/spreadsheets/d/1ETWpaqztdAmlvRhUOv-MqUsK0bGMkVYAKQT3VkEtols/edit#gid=0
GO:1905538 polysome binding is being replaced by GO:0043022 ribosome binding.
Any comments can be added to the issue: https://github.com/geneontology/go-ontology/issues/26170
Thanks,


Fix: Twin Shield Tweaks

Hi there,
I made some tweaks to the Guardian Twin Shields ability as the book says that 'you gain the benefits of both items' (197) but in the compendium the bonus was only being applied once. I also added a Twin Shields item for when the player has one runic and one bronze shield
Changes

Twin Shields > set def bonus to 4
Twin Shields (Runic) > set def to 4, set magic def to 4
Twin Shields (Runic & Bronze) > set def to 4 and magic def to 2

Sorry if I've misunderstood the conventions on how you're building items. Thanks for all your hard work, the system is looking great.

I just noticed I didn't update the uuid inside the new item json, and only put it in the filename. I'm not sure what the best way to generate an id is, I just made one in foundry by creating a copy of one of the other items. Will that cause problems?

No worries, i just generated a new id and edited the id in the json.

I thought the idea was that you would have the twin shields item only replace one of the shields ands still have the other equipped on your offhand.

I thought the idea was that you would have the twin shields item only replace one of the shields ands still have the other equipped on your offhand.

I guess the issue with this is that twin shield counts as a two handed weapon so it’ll take up both main and off hand. I’m holding off doing anything with shields until the equipment system is overhauled, where equipment data is stored in the actor instead of items. From there we could work on equipment validation.
Maybe adding a checkbox that’ll allow you to equip two handed weapons in one slot. So you could equip another shield and gain their benefits.

To update everyone, in V2.4.0, with the new implementation of EquipmentHandlerV2, the other twin shields wouldn't be necessary anymore since you can now equip two shields and also equip the twin shield item into a "phantom slot".


[W6.5][W11-B2]Oh Han Yi

Add label to GUI

@slothhy
Some comments added. Please ack & close the PR after reading comments.

Acknowledged


chore(br,dm): delete jenkins/pipelines/ci/br and jenkins/pipelines/ci/dm folder

dm and br were archived.
Fixes #3051
Fixes #3052
Signed-off-by: wuhuizuo<EMAIL_ADDRESS>

/hold

/unhold


Distinguishing split antecedents versus discontinuous mentions

As noted in the README, the corefud scorer does not score split antecedents (SplitAnte) but does score discontinuous mentions. Although I'm somewhat confused when a coreference is considered a discontinuous mention versus a split antecedent.
For example, consider entity e127 in the following two sentences included from the training set of English ParCorFull in CorefUD 1.1. "[The Comite River] near Baton Rouge and [Amite River]" is a discontinuous mention that corefers with "[both rivers]". So if I am right this would be scored by the corefud scorer whereas it seems in spirit to be a split antecedent.
I'm not sure what a good solution to this ambiguity would be. 		PUNCT	.	_	4	punct	_	_

"[The Comite River] near Baton Rouge and [Amite River]"

I agree this should be a split antecedent of "both rivers", i.e. "The Comite River" should be one entity and "Amite River" should be another entity. (I would even say the span should include the whole dependency subtree in this case: "The Comite River near Baton Rouge" and "Amite River near Denham Springs", but that is another controversial issue, unrelated to this one).
We (authors of CorefUD) may try to fix this in the new version of CorefUD, but in general there are many such problems in the original datasets and we cannot fix them all. So far, we focused on unifying the format and validation, but there are still many differences in the annotation guidelines of the original datasets, which we had not time to unify.
I've created an issue in our private ufal/corefud repo for this problem, but the two ParCorFull datasets are very small, so this is not the highest priority for us.

I'm somewhat confused when a coreference is considered a discontinuous mention versus a split antecedent.

There are clear cases of discontinuous mentions, e.g. in "fresh apples and oranges" when it means that both apples and oranges are fresh, the mention "fresh oranges" is discontinuous. In some coreference annotation styles, if two mentions in coordination are next to each other, the coordination (e3) is annotated in addition to the two mentions (e1 and e2):
fresh    (e3(e1(e2[1/2])
apples   e1)
and
oranges  (e2[1/2])e3)

I would say e3 is not needed (and should be omitted from the annotation). But again, we didn't have time to unify these differences across datasets in CorefUD.

That makes sense; thanks for the explanation!
In case it helps to know: in English ParCorFull there are only five examples (all in the training set) of discontinuous mentions, and as far as I can tell they should all similarly be SplitAnte.


Generating with preset: 'client' is not working correctly

My config:
import { TheCodegenConfiguration } from '@the-codegen-project/cli'
const config: TheCodegenConfiguration = {
	inputType: 'asyncapi',
	inputPath: '../../definitions/asyncapi.json',
	language: 'typescript',
	generators: [
		{
			preset: 'client',
			outputPath: 'src/__nats__/channels',
			language: 'typescript',
			protocols: ['nats'],
		},
		{
			preset: 'payloads',
			outputPath: 'src/__nats__/payloads',
			serializationType: 'json',
			language: 'typescript',
			enum: 'enum',
			map: 'record',
			useForJavaScript: true,
			rawPropertyNames: false,
		},
		{
			preset: 'parameters',
			language: 'typescript',
			outputPath: 'src/__nats__/parameters',
			serializationType: 'json',
		},
	],
}
export default config

Is generating the following error:

It seems like the dependencies is getting a default array of 1 item: channels-typescript which the code might then expect to be present all the time. In my case I don't have any dependencies.

Update on this. I figured out it was because it adds the channels-typescript as a dependency but doesn't generate a default channels generator. That's the core issue here. Fixed it with this:

		{
			preset: 'channels',
			outputPath: './src/__nats__/channels',
			language: 'typescript',
			protocols: ['nats'],
		},


Server error for<EMAIL_ADDRESS>

This worked just a couple of days ago so I'm not sure what changed. I know that the package got updated recently but older versions of the package don't seem to work either anymore.
Cheers!

I invalided the entry, it works again.

Thanks!