Data download features

Allows users to download data in CSV and JSON format from reports
Use cases:

[ ] Can be added to table component for exporting the table as CSV
[ ] Can be added to top of report for exporting the whole function result to JSON
[ ] Can be added to top of report section for exporting a set of bite-size charts/tables to one larger CSV

Requirements:

[ ] UI is a kebab dropdown menu that takes array-based table data and transforms to CSV and JSON as-is
[ ] Standard formatters are included, JSON accepts the data as-is  and stringify's it, CSV accepts arrays for headers and rows.  The component should allow data input to be provided for each formatter, it's up to the caller to transform their data appropriately.

Once we have the table component it would be nice to build the configuration of this button off of the table component configuration, so that an author need only specify { enableCSV: true } to get a button in a standard location. I'm not sure whether JSON needs to be included here.
For JSON output, I was thinking more along the lines of what SeaSketch currently has in the MPATT project with a top-level export option for the entire reporting output of a Sketch or Collection. In legacy SeaSketch this can be accessed from both a context menu action and the report sidebar itself. This is not enabled on all projects, IIRC because the automated output is quite cryptic and oddly organized due to the fact that we don't have a lot of control in ArcGIS Server.

This would be a lot more tricky to implement, since we specify the contents of the report using ReportCard and useFunction hooks. There would have to be some runtime trickery done to collect that info. It also requires some coordination between the report iframe and the surrounding SeaSketch App. Maybe the surrounding app can issue a postMessage request for output? This is something I think we can plan to include for SeaSketch Next, rather than bolt onto current SeaSketch.

Here's an example where I don't think we want a simple export button per table because they are really a group of bite size table. Rather we want an export to CSV for the whole set

Ah interesting. I guess in this case... oh boy yeah that does require some very context specific decisions.

One could argue those bite-sized tables should become one 'group' table if they are being generated from the same function result and have the same data shape.  I'll look closer at it.  Maybe I can start with the simplest and gather feedback.


Use async HTTPClient in BinaryArtifactsManager

Motivation:
To adopt Swift concurrency in ManifestLoader, we need to convert RegistryClient first, which then would require use of HTTPClient instead of LegacyHTTPClient. That would also cascade to BinaryArtifactsManager, so I'm starting there first to make PRs smaller.
Modifications:
Converted (and as a consequence, simplified) code in Workspace+BinaryArtifacts.swift, also adopted HTTPClient in WorkspaceTests to fix tests after the change.
Converted some error diagnostics to conform to Error, which makes control flow with ObservabilityScope.trap wrapper easier to write and read.
Result:
One major previous adopter of LegacyHTTPClient is now compatible with Swift concurrency.
Since BinaryArtifactsManager was blocking Swift concurrency thread pool and also used .sharedConcurrent dispatch queue, this potentially fixes thread explosion/deadlocks that may have been caused by previously broken code.

@swift-ci test

@swift-ci test linux

@swift-ci test linux

@swift-ci test linux

@swift-ci test linux


Incorrect structure on random points for data with small variations. Lack of shuffling?

umap.__version__ == 0.4.6

In the following example, UMAP is applied to random data with tiny variations around a fixed value. Arbitrary labels are assigned to the data in order ('a' for the first 150 samples, 'b' to the next 150, etc.).
When applying UMAP, a structure appears (points with the same label are grouped), as shown in the image below, but it should not (random data).
This issue can potentially lead to incorrect interpretations.
It could be due to a kind of greedy behavior in the construction of the neighborhood graph.
Randomly shuffling the data before construction of the neighborhood graph solves the problem, but might not be an ideal fix.
import numpy as np
from umap import UMAP
import umap.plot

n_obs = 600
n_vars = 16
np.random.seed(42)

fixed_value = np.random.random(n_vars)
X = 10 * fixed_value + 1e-9 * np.random.random((n_obs, n_vars))
labels = np.array(['a'] * (n_obs // 4)
                  + ['b'] * (n_obs // 4)
                  + ['c'] * (n_obs // 4)
                  + ['d'] * (n_obs // 4))

reducer = UMAP(
    metric="cosine",
    min_dist=0.0,
    n_neighbors=10,
    random_state=0,
)
reducer.fit(X)
p = umap.plot.points(reducer, labels=labels, theme="fire")
umap.plot.show(p)

In the sample code, the rows of X are basically identical, at least up to the 8 decimal places that my Jupyter Lab notebook displays, so the small variation in this simulation data might be a bit too small. Is this level of numerical similarity based on real-world data?
Anyway, what you are seeing is the nearest neighbor routine failing in the event that it can't distinguish the distances between any observations (the distances are all zero). What happens under those conditions as to what neighbors get picked is going to be very implementation-dependent. In an ideal world picking at random would be better than what we get, but I don't see this even with other commonly used nearest neighbor methods (e.g. Annoy, HNSW, the exact nearest neighbors R package FNN).
In my experience this scenario doesn't happen with non-simulation data. What I have observed are small subsets of the data being exact duplicates. Degeneracy of distances can be a bigger risk with low-dimensional binary data with integer-based distances because there are a limited number of possible distances. But I don't know how common such datasets are and wouldn't result in the full-on catastrophe of all observations being effectively duplicates.
Assuming this does affect real data, there isn't an easy fix because as mentioned other nearest neighbor routines also fail in a similar way (see https://github.com/jlmelville/uwot/issues/85 for instance). Diagnosing when the issue is likely to occur is probably the best that can be done. Something that might help would be to log the number of times the binary search in smooth_knn_dist fails. I have this in uwot and it invariably indicates the presence of duplicates. It might not be too hard to come up with a PR to implement this here too.
(as an aside for anyone reading this, I recommend the recently released Avonet bird morphology dataset for a fun dataset where you will see some duplicates. Seems there are many parrot species with identical beak/wing measurements. The resulting embedding is not unduly affected however)

Thanks for your answer!
I actually encountered the bug on real data, or, more precisely, on embeddings generated via machine learning from real data.
The learning algorithm was a variant of contrastive learning, which tends to make embeddings of similar samples "collapse" (without being identical). So it is not unusual to have neighborhoods of embeddings in which the different points are extremely close to each other (with distances around 1e-10). It really took me some time to understand what was happening!
Thanks for the information that other nearest neighbor routines would also fail. I'll remember to either diagnose the "dangerous" cases, as you mentioned, or to shuffle the data before computing neighborhood graphs.
However, a warning when very small non-zero distances are detected would, I think, be a way to prevent other people from encountering the same bug.

We typically deal with the nearest neighbour problem with regards to
duplicate data with a UMAP flag unique=True.  This will dedup your data
before computing our nearest neighbour graph and then embed all the
duplicate copies of your data on the same coordinate in your low
dimensional space.  As James correctly pointed out this problem is commonly
encountered with binary data derived from categorical features.  Duplicate
count_vectorized text data was the original use case that tripped me up
with respect to this property.
You could always round your data to some small precision (such as 1e-10)
and then use unique=True.  Of course, that doesn't help with detecting
the problem in the first place.  I'm not sure that this problem is common
enough to warrant a warning but I could be convinced otherwise at which
point a PR would be lovely.
On Thu, Mar 3, 2022 at 1:22 PM Nicolas Perrin-Gilbert <
@.***> wrote:

Thanks for your answer!
I actually encountered the bug on real data, or, more precisely, on
embeddings generated via machine learning from real data.
The learning algorithm was a variant of contrastive learning, which tends
to make embeddings of similar samples "collapse" (without being identical).
So it is not unusual to have neighborhoods of embeddings in which the
different points are extremely close to each other (with distances around
1e-10). It really took me some time to understand what was happening!
Thanks for the information that other nearest neighbor routines would also
fail. I'll remember to either diagnose the "dangerous" cases, as you
mentioned, or to shuffle the data before computing neighborhood graphs.
However, a warning when very small non-zero distances are detected
would, I think, be a way to prevent other people from encountering the same
bug.
—
Reply to this email directly, view it on GitHub
https://github.com/lmcinnes/umap/issues/840#issuecomment-1058351183, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AC3IUWX6IHUJA7LZ65XQUNDU6D7NFANCNFSM5PXT4MBQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


the default taskbar feature for showing the desktop is not working properly

When taskbarx is on the feature for showing the desktop is not working properly (the right most box).

my mouse is on the right down corner of my screen but i cannot click on the square that will show my desktop.
i can move my mouse a bit for the square to flash white meaning that i can press it.
exiting taskbarx and restarting windows explorer from the taks manager fixes the issue as long as you don't open the taskbarx again

I have the same issue. Downloaded from microsoft store. Windows 10 Pro N 21H1 build 19043.
It seems like it's the one pixel right at the corner that it somehow outside the click area. And if you flick your mouse there, that's the pixel you will click.

To be able to focus on major/current bugs I had to close all issues. The issues section has gotten very messy to a point that I couldn't follow it anymore. Most issues are duplicates, very old or are already fixed/solved. Feel free to reopen this issue if you still have this issue on the latest version of TaskbarX to get my attention. All feature requests have been put in a list here: 💡| Feature requests #870
I hope you understand ❤


Button hover effect

looks like the small button still has wrong pink hover color

changes:

bg color to k-accent
text color to black

👋🏻


WSL stops opening in Windows Terminal after a while

Windows Terminal version
1.12.10983.0
Windows build number
10.0.22000.0
Other Software
WSL Ubuntu, ssh
Steps to reproduce

Boot into Windows, open a Terminal tab that runs wsl.exe -d Ubuntu with \wsl.localhost\Ubuntu\home\username as starting directory.  I also have a terminal tab that runs the command "ssh username@myserver" when it's opened.

Wait some amount of time (unclear when it breaks)

Open a new WSL tab

Expected Behavior
Windows Terminal opens a WSL shell tab
Actual Behavior
A new Terminal tab is opened, but it just has a flashing cursor and won't start the WSL process.
I am still able to open WSL through a command prompt or powershell by running "wsl -d Ubuntu", and Terminal still opens my both my elevated and unelevated powershell tabs with no issues.
This just started to happen a couple weeks ago, and I had not made changes to my configuration before it started happening. Windows and WSL are up to date, as are all of my Ubuntu packages.
The solution is to run "wsl --shutdown" from an elevated command prompt or powershell, but eventually the terminal won't open again and I need to keep rerunning "wsl --shutdown".  Rebooting the system also works.  Closing and reopening the Terminal application does not help.

I have not reported it to the WSL repo but I will do that.
Unfortunately, it's more or less impossible to get a more detailed
reproduction of it. It just seems totally random, but always happens. I
always have the same applications running on my PC, so my best guess is a
recent update to one of them made something wonky. But that doesn't make a
lot of sense either.
It's so weird that wsl works when run from a command prompt, but terminal
opening it directly doesn't work. I've gone through syslogs and tried to
track what's happening down, but gremlins don't leave logs. I can't find
anything.
Thanks for the response. I'll get on to reporting it to the WSL repo in a
bit. I hope they can help.
On Thu, Jun 9, 2022, 12:36 PM Mike Griese @.***> wrote:

That makes... no sense. I have no idea what could possibly cause that.
We might need a better repro here, unfortunately. There's not much we can
do with "I do something but I have no idea what, and WSL doesn't work".
Have you reported this on the WSL repo/? You might get better debugging
tips from the WSL team.
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/terminal/issues/13259#issuecomment-1151543065,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAYOKGP7N2OGQDLAORSLO33VOJBSZANCNFSM5YLC7BRQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

When you write:

I am still able to open WSL through a command prompt or powershell by running "wsl -d Ubuntu"

Are you saying that WSL still works if you start it from outside of Windows Terminal?
I can remember that I had similar issues with WSL for a short period of time but it's been at least a year since...

I can start it through Windows Terminal after it breaks, but it needs to be invoked from a powershell or command prompt tab by running "wsl -d Ubuntu".  The issue is with the tabs that directly invoke wsl.exe.  If I run Ubuntu from the start menu, it does open a window that looks like yours, only it doesn't properly apply my zsh themes because it has issues with loading certain fonts.
I had a problem about a year ago when I was running Windows 10 where wsl would shut down at random.  I was running the Ubuntu command from the start menu at the time and the Ubuntu command would quit working at some point during my session.  The solution was to run wsl.exe on a command prompt to restart the service, then Ubuntu would work again.  A bit of a different problem, but similar enough (though unrelated to Windows Terminal).

Are you using the latest WSL version from the store?


conditionally search for masters

Description
Chef search may not be desired due to a large Chef
environment or simply because of user defined servers.
Keep seach by default, but allow to skip it, so search
is not run, and user defined servers are used.
Check List

[ ] All tests pass. See TESTING.md for details.
[ ] New functionality includes testing.
[X] New functionality has been documented in the README if applicable.

Fixed the changelog validator, but I failed to understand Release Label Validator , and did not find anything relevant in the doc. Any hints ? Is it you setting the label ?

Fixed the changelog validator, but I failed to understand Release Label Validator , and did not find anything relevant in the doc. Any hints ? Is it you setting the label ?

@jeremy-clerc we're still working on properly importing this cookbook into Sous Chefs. Once we'll do that, we can look at getting this reviewed/released. I hope to get that done in the next week or so. Thanks!

@jeremy-clerc we're still working on properly importing this cookbook into Sous Chefs. Once we'll do that, we can look at getting this reviewed/released. I hope to get that done in the next week or so. Thanks!

@jeremy-clerc can you please rebase and resolve any conflicts now that we released a new version finally

@jeremy-clerc can you please rebase and resolve any conflicts now that we released a new version finally

Thanks for the quick import in Sous-Chef.
Rebased, still Release Label Validator unhappy, I have not figured out how to fix it.

Thanks for the quick import in Sous-Chef.
Rebased, still Release Label Validator unhappy, I have not figured out how to fix it.

Released as: 1.1.0

Released as: 1.1.0


retained_mem and mimxrt

I'm having trouble configuring retained_mem on mimxrt1050_evk with Zephyr v3.7.0.
I follow documentation example and:

it fails with external SDRAM
mimxrt1050_evk.overlay
/ {
    sram@81ff0000 {
        compatible = "zephyr,memory-region", "mmio-sram";
        reg = <0x81ff0000 DT_SIZE_K(64)>;
        zephyr,memory-region = "RetainedMem";
        status = "okay";

        retainedmem {
            compatible = "zephyr,retained-ram";
            status = "okay";
            #address-cells = <1>;
            #size-cells = <1>;

            boot_info0: boot_info@0 {
                compatible = "zephyr,retention";
                status = "okay";
                reg = <0x0 0x10000>;
                prefix = [CA FE];
                checksum = <4>;
            };
        };
    };

    chosen {
        zephyr,bootloader-info = &boot_info0;
    };
};

/*
 Reduce sdram0 usage by 64 kB

 Original memory range: 0x80000000 - 0x82000000-1
 New memory range: 0x80000000 - 0x81fff000-1
 */
&sdram0 {
    reg = <0x80000000 (DT_SIZE_M(32)- DT_SIZE_K(64))>;
};

build error
/home/benjamin/zephyr-rtos/include/zephyr/arch/arm/cortex_m/arm_mpu_mem_cfg.h:72:2: error: #error "Unsupported sram size configuration"
   72 | #error "Unsupported sram size configuration"

it also fails with OCRAM
mimxrt1050_evk.overlay
/ {
    chosen {
        zephyr,bootloader-info = &boot_info0;
    };
};

/*
 Reduce ocram usage by 1 kB

 Original memory range: 0x20200000 - 0x20240000-1
 New memory range: 0x20200000 - 0x2023fc00-1
 */
&ocram {
    reg = <0x20200000 (DT_SIZE_K(256)- DT_SIZE_K(1))>;
};

&flexram {
    rmem: sram@2023fc00 {
        compatible = "zephyr,memory-region", "mmio-sram";
        reg = <0x2023fc00 DT_SIZE_K(1)>;
        zephyr,memory-region = "RetainedMem";
        status = "okay";

        retainedmem {
            compatible = "zephyr,retained-ram";
            status = "okay";
            #address-cells = <1>;
            #size-cells = <1>;

            boot_info0: boot_info@0 {
                compatible = "zephyr,retention";
                status = "okay";
                reg = <0x0 0x400>;
                prefix = [CA FE];
                checksum = <4>;
            };
        };
    };
};

build error
/home/benjamin/zephyr-rtos/drivers/memc/memc_nxp_flexram.c:66:1: note: in expansion of macro 'BUILD_ASSERT'
   66 | BUILD_ASSERT((OCRAM_TOTAL) == DT_REG_SIZE(OCRAM_DT_NODE), "OCRAM node size is wrong");

Is there any way to declare retain data section and pass memory size check?
I also don't understand documentation example:
/ {
        sram@2003F000 {
                compatible = "zephyr,memory-region", "mmio-sram";
                reg = <0x2003F000 DT_SIZE_K(1)>;
                zephyr,memory-region = "RetainedMem";
                status = "okay";

                retainedmem {
                        compatible = "zephyr,retained-ram";
                        status = "okay";
                        #address-cells = <1>;
                        #size-cells = <1>;

                        boot_info0: boot_info@0 {
                                compatible = "zephyr,retention";
                                status = "okay";
                                reg = <0x0 0x100>;
                        };
                };
        };

        chosen {
                zephyr,bootloader-info = &boot_info0;
        };
};

/* Reduce SRAM0 usage by 1KB to account for non-init area */
&sram0 {
        reg = <0x20000000 DT_SIZE_K(255)>;
};

In this example, sram0 range is [0x20000000 - 0x2003FC00-1].
So why RetainedMem starts at 0x2003F000?
I also don't understand why boot_info0 size is only 256 B and not 1 KB (but it is not really an issue).

Hi @blemouzy ,

it fails with external SDRAM:
/home/benjamin/zephyr-rtos/include/zephyr/arch/arm/cortex_m/arm_mpu_mem_cfg.h:72:2: error: #error "Unsupported sram size configuration"
72 | #error "Unsupported sram size configuration"

That is an MPU error.  ARM's MPU only supports binary sizes (2^x).  The MPU cannot handle a size of DT_SIZE_M(32)- DT_SIZE_K(64).  Instead of reducing the size of sdram0, perhaps you can add your RetainedMem as a child node within sdram0 to retain the original size.

it also fails with OCRAM:
/home/benjamin/zephyr-rtos/drivers/memc/memc_nxp_flexram.c:66:1: note: in expansion of macro 'BUILD_ASSERT'
66 | BUILD_ASSERT((OCRAM_TOTAL) == DT_REG_SIZE(OCRAM_DT_NODE), "OCRAM node size is wrong");

A similar issue, the FlexRAM has specific sizes of OCRAM it can support.  See NXP Application Note AN12077 for more details.  Your size of 255 KB is not a valid option for OCRAM.

So why RetainedMem starts at 0x2003F000?

Good question, that looks like an error to me.

Hi @DerekSnell,

it fails with external SDRAM:
/home/benjamin/zephyr-rtos/include/zephyr/arch/arm/cortex_m/arm_mpu_mem_cfg.h:72:2: error: #error "Unsupported sram size configuration"
72 | #error "Unsupported sram size configuration"

That is an MPU error. ARM's MPU only supports binary sizes (2^x). The MPU cannot handle a size of DT_SIZE_M(32)- DT_SIZE_K(64). Instead of reducing the size of sdram0, perhaps you can add your RetainedMem as a child node within sdram0 to retain the original size.

Understood.
It finally works with the following mimxrt1050_evk overlay:
/ {
    chosen {
        zephyr,bootloader-info = &boot_info0;
    };
};

&sdram0 {
    retainedmem {
        compatible = "zephyr,retained-ram";
        status = "okay";
        #address-cells = <1>;
        #size-cells = <1>;

        boot_info0: boot_info@0 {
            compatible = "zephyr,retention";
            status = "okay";
            reg = <0x1fff000 0x1000>;
            prefix = [CA FE];
            checksum = <4>;
        };
    };
};

Memory looks correctly filled:
x /16x 0x81fff000
0x81fff000:	0x2016feca	0x2000002c	0x01010001	0x00000120
0x81fff010:	0x00012002	0x01200300	0x20040000	0x01020008
0x81fff020:	0x00000000	0x20050000	0x00000004	0x00000040
0x81fff030:	0x00000000	0x00000000	0x00000000	0x00000000

and displayed bootloader version is correct.
Note: with Zephyr main (1a89e1640c1464340dfb1d652fdf9c02130b482c), I encountered an issue with flash SPI partitions using mimxrt1050_evk@hyperflash.
It looks like flashing application erased MCUBoot.

it also fails with OCRAM:
/home/benjamin/zephyr-rtos/drivers/memc/memc_nxp_flexram.c:66:1: note: in expansion of macro 'BUILD_ASSERT'
66 | BUILD_ASSERT((OCRAM_TOTAL) == DT_REG_SIZE(OCRAM_DT_NODE), "OCRAM node size is wrong");

A similar issue, the FlexRAM has specific sizes of OCRAM it can support. See NXP Application Note AN12077 for more details. Your size of 255 KB is not a valid option for OCRAM.

Thanks for the AN link.
I tried to create a second OCRAM but it's clear now that is not possible.

So why RetainedMem starts at 0x2003F000?

Good question, that looks like an error to me.

I think it's a bug too.
With same error in tests/boot/mcuboot_data_sharing/boards/nrf52840dk_nrf52840.overlay.
I will send a PR.

Hi @blemouzy ,

I encountered an issue with flash SPI partitions using mimxrt1050_evk@hyperflash board. It looks like flashing application erased MCUBoot.

Will you be creating a new Issue for this MCUboot topic, and sharing the details to replicate?
Since this Issue about retained_mem seems resolved, can we close it now?  Thanks


TypeError: WeakMap key undefined

Description
JavaScript error: TypeError: WeakMap key undefined must be an object or an unregistered symbol (https://try.gitea.io/ @ 529:36). Open browser console to see more details.

worker port event error
Object { type: "error", data: undefined }
index.js:30:76443
    Tg https://try.gitea.io/assets/js/index.js?v=1.21.4:30

This does not occur in a private window, I have tried to clear cookies and site data, no change.
Screenshots

Gitea Version
1.22.0+dev-575-gd68a613ba
Can you reproduce the bug on the Gitea demo site?

Disabling the dark reader extension for gitea fixed it for me

Goddammit darkreader

On Gitea side, we could do this: Avoid duplicate JS error messages on UI #28873

had this happening to me too. thanks for fixing it!


Calling from visual mode gives "E481: No range allowed"

On neovim nightly on Mac M1 I've configured Silicon using Lazy (in Fennel) with
(lazy :krivahtoo/silicon.nvim
      {:build "./install.sh build"
       :opts {:font "Fira Code=16"
              :theme "Monokai Extended"
              :window_controls false
              :tab_width 2
              :pad_vert 0
              :pad_horiz 0}})

There are no errors in the build process but if I

call :Silicon I get an image successfully created but only containing the first line of the file.
call :%Silicon I get E481
select some text in visual mode and call :'<,'>Silicon I also get E481

I can't see any other error messages so I'm not too sure how to debug the issue further.

What is the output of :command Silicon?

Not sure how to inspect the definition but here's the output of running :command Silicon
    Name              Args Address Complete    Definition
!   Silicon           0                        <Lua 156>

Same here. Also in normal mode, it takes only the first line. Feels like the condition to check the mode is reversed
https://github.com/krivahtoo/silicon.nvim/assets/51330172/493c643a-764e-45b0-9bea-d339cab7d176

I'm also encountering this error in Neovim nightly (541c2d3816cdef9aba7f55f74908fa07d0cfa3ec) on macOS.

silicon.nvim currently does not support neovim-nightly.

Same issue on the new stable

@Demianeen
Thanks for reporting. I didn't realize there was a new version on Neovim.
I have released a new update the should work on v0.10

It started to work after the update! It still crushes neovim after it does screenshot though. Same behaviour on stable and nightly versions. Might you know where the issue is?
https://github.com/krivahtoo/silicon.nvim/assets/51330172/79e1a62b-a84b-40e1-8bdf-d8a74369e1f4
Versions info:
Stable:
NVIM v0.10.0
Build type: Release
LuaJIT 2.1.1713484068
Run "nvim -V1 -v" for more info

Nightly:
NVIM v0.11.0-dev-49+g10601ac5f
Build type: RelWithDebInfo
LuaJIT 2.1.1713484068
Run "nvim -V1 -v" for more info

It still crushes neovim after it screenshots though. The same behaviour on stable and nightly versions. Might you know where the issue is?

I am unable to reproduce the issue in both v0.10 and nightly. I am on Linux so maybe the issue is with Mac builds.

P.S. I already added plans to learn Rust in the next couple of months to help here 😄

That would be awesome 😊


Remove beta prefixes for columns and expander

Got this when starting up so thought I'd do a quick little PR

Thanks for the contribution!


fix: ensure execution on initial sync

Tweaks ExEx execution to ensure that it can run during the node execution pipeline that occurs during initial sync.
Currently, the ExEx panics while executing on pre-merge blocks with the following trace:
thread 'tokio-runtime-worker' panicked at /home/ubuntu/.cargo/git/checkouts/revm-bf744d8ffabcbad9/a28a543/crates/interpreter/src/instructions/host_env.rs:32:61:
called `Option::unwrap()` on a `None` value
stack backtrace:
   0: rust_begin_unwind
   1: core::panicking::panic_fmt
   2: core::panicking::panic
   3: core::option::unwrap_failed
   4: revm_interpreter::instructions::host_env::difficulty
   5: revm::handler::mainnet::execution::execute_frame
   6: core::ops::function::Fn::call
   7: revm::evm::Evm<EXT,DB>::transact_preverified_inner
   8: shadow_reth_exex::execution::ShadowExecutor<DB>::execute_one
   9: <core::iter::adapters::map::Map<I,F> as core::iter::traits::iterator::Iterator>::try_fold
  10: alloc::vec::in_place_collect::<impl alloc::vec::spec_from_iter::SpecFromIter<T,I> for alloc::vec::Vec<T>>::from_iter
  11: core::iter::adapters::try_process
  12: shadow_reth_exex::ShadowExEx::init::{{closure}}::{{closure}}
  13: <core::panic::unwind_safe::AssertUnwindSafe<F> as core::future::future::Future>::poll
  14: <futures_util::future::future::map::Map<Fut,F> as core::future::future::Future>::poll
  15: <tracing_futures::Instrumented<T> as core::future::future::Future>::poll
  16: <futures_util::future::select::Select<A,B> as core::future::future::Future>::poll
  17: reth_tasks::TaskExecutor::spawn_critical_as::{{closure}}
  18: tokio::runtime::task::core::Core<T,S>::poll
  19: tokio::runtime::task::harness::Harness<T,S>::poll
  20: tokio::runtime::scheduler::multi_thread::worker::Context::run_task
  21: tokio::runtime::scheduler::multi_thread::worker::Context::run
  22: tokio::runtime::context::scoped::Scoped<T>::set
  23: tokio::runtime::context::runtime::enter_runtime
  24: tokio::runtime::scheduler::multi_thread::worker::run
  25: tokio::runtime::task::core::Core<T,S>::poll
  26: tokio::runtime::task::harness::Harness<T,S>::poll
  27: tokio::runtime::blocking::pool::Inner::run
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.
2024-06-24T08:16:42.950275Z DEBUG reth_exex::manager: Updating lowest notification id in buffer min_id=1
2024-06-24T08:16:42.950378Z ERROR exex{id="ShadowExEx"}: reth_tasks: Critical task `exex` panicked: `called `Option::unwrap()` on a `None` value`
2024-06-24T08:16:42.952353Z ERROR reth::cli: shutting down due to error
Error: Critical task `exex` panicked: `called `Option::unwrap()` on a `None` value`

Location:
    /home/ubuntu/.cargo/git/checkouts/reth-36d3ea1d1152b20c/d777d5f/crates/cli/runner/src/lib.rs:146:32

This is caused by EvmBuilder::default() using SpecId::Latest. We fix this by re-configuring the evm with the correct spec id for a given block on each execution.

One other thing that would maybe be nice to fix here, we get tx hold warnings during sync as we end up holding the read tx while processing the execution chunk:
024-06-26T12:48:00.887083Z  WARN exex{id="ShadowExEx"}: storage::db::mdbx: The database read transaction has been open for too long. Backtrace:
   0: reth_db::implementation::mdbx::tx::MetricsHandler<K>::log_backtrace_on_long_read_transaction
   1: <reth_db::implementation::mdbx::tx::Tx<K> as reth_db::abstraction::transaction::DbTx>::get
   2: <reth_provider::providers::state::historical::HistoricalStateProviderRef<TX> as reth_storage_api::account::AccountReader>::basic_account
   3: <revm::db::states::state::State<DB> as revm_primitives::db::Database>::basic
   4: revm::journaled_state::JournaledState::load_account
   5: core::ops::function::Fn::call
   6: revm::evm::Evm<EXT,DB>::transact_preverified_inner
   7: shadow_reth_exex::execution::ShadowExecutor<DB>::execute_one
   8: <core::iter::adapters::map::Map<I,F> as core::iter::traits::iterator::Iterator>::try_fold
   9: alloc::vec::in_place_collect::<impl alloc::vec::spec_from_iter::SpecFromIter<T,I> for alloc::vec::Vec<T>>::from_iter
  10: core::iter::adapters::try_process
  11: shadow_reth_exex::ShadowExEx::init::{{closure}}::{{closure}}
  12: <core::panic::unwind_safe::AssertUnwindSafe<F> as core::future::future::Future>::poll
  13: <futures_util::future::future::map::Map<Fut,F> as core::future::future::Future>::poll
  14: <tracing_futures::Instrumented<T> as core::future::future::Future>::poll
  15: <futures_util::future::select::Select<A,B> as core::future::future::Future>::poll
  16: reth_tasks::TaskExecutor::spawn_critical_as::{{closure}}
  17: tokio::runtime::task::core::Core<T,S>::poll
  18: tokio::runtime::task::harness::Harness<T,S>::poll
  19: tokio::runtime::scheduler::multi_thread::worker::Context::run_task
  20: tokio::runtime::scheduler::multi_thread::worker::Context::run
  21: tokio::runtime::context::scoped::Scoped<T>::set
  22: tokio::runtime::context::runtime::enter_runtime
  23: tokio::runtime::scheduler::multi_thread::worker::run
  24: tokio::runtime::task::core::Core<T,S>::poll
  25: tokio::runtime::task::harness::Harness<T,S>::poll
  26: tokio::runtime::blocking::pool::Inner::run
  27: std::sys_common::backtrace::__rust_begin_short_backtrace
  28: core::ops::function::FnOnce::call_once{{vtable.shim}}
  29: std::sys::pal::unix::thread::Thread::new::thread_start
  30: <unknown>
  31: <unknown>
 open_duration=65.715026391s self.txn_id=228648


Clarity core components - Alert not closable

Describe the bug
Cannot close the alert message using the close button (x icon)
How to reproduce
Create a closable alert like this and try to close it by clicking the x icon.
<cds-alert-group status="danger">
  <cds-alert closable="true" cds-i18n='{ "closeButtonAriaLabel": "close danger alert"}'>
	  This example is an alert a user may be able to close inside an alert group with a status of "danger".
  </cds-alert>
</cds-alert-group>

Stackblitz
Expected behavior
Alert should close
Versions
Clarity project:
Clarity Core 5.3.0
Clarity Angular/UI 5.3.0
Clarity version: 5.3.0
Framework: Angular
Framework version:
Angular 11.2.13
Device:
OS: Windows 10
Browser Chrome 91.0.4472.77

I ran in to the same problem.
I had to wire it up manually using the (closeChange) binding. https://clarity.design/core-components/alert/api/#alert-events

@grahamford is correct. The core alerts don't close themselves. They will tell the application that an interaction wants to close an alert. The application can take that information and close the alert. Or not...


Fix for typo in indicator type

Indicator type for domains should be domain
(Thanks for this extension, really useful !)
Signed-off-by: Luigi Mori<EMAIL_ADDRESS>

Superseded by #3


fix: rename entities obfuscated class detection

Previously, it would not detect classes like a$a or aaa$aaa which caused some issues.
I am not sure if a cache would be needed since I am using regex now.
As an additional thought, the regex could be a cli paramater since names could be obfuscated differently.
Fixes point 3 & possibly 2 of #112

Codecov Report

Merging #195 (cc3fd0e) into develop/1.9.0 (1714df9) will increase coverage by 0.00%.
The diff coverage is 85.71%.

@@               Coverage Diff                @@
##             develop/1.9.0     #195   +/-   ##
================================================
  Coverage            77.38%   77.39%
- Complexity            7939     7940    +1
================================================
  Files                  219      219
  Lines                25484    25486    +2
  Branches              6609     6610    +1
================================================
+ Hits                 19722    19724    +2
  Misses                3773     3773
  Partials              1989     1989

Impacted Files
Coverage Δ

...va/decompiler/modules/renamer/ConverterHelper.java
63.63% <85.71%> (+1.73%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

I don't know how unit tests work in quilt sooo

Fair, I realized it's never specified anywhere. I'll add one myself.


Fixed memory usage high in some cases

Fixed metal rendering when presentsWithTransaction is enabled, based on Apple documentation:
Setting this value to YES makes the layer draw its contents synchronously, using whichever Core Animation transaction is current at the time you call the drawable’s present method. To ensure that a transaction is available when you schedule the drawable to be presented, first commit the command buffer containing your Metal rendering commands. Then, call its waitUntilScheduled method to synchronously wait until the command queue schedules the command buffer to execute on the GPU. Finally, call the drawable’s present method.
Source: https://developer.apple.com/documentation/quartzcore/cametallayer/1478157-presentswithtransaction?language=objc.
This will fix correctly also the following issues: https://github.com/maplibre/maplibre-native/issues/2337, https://github.com/maplibre/maplibre-native/issues/2380, https://github.com/maplibre/maplibre-native/issues/2053, so I removed old hacks.
Allocations after the fix:
https://github.com/maplibre/maplibre-native/assets/19795769/8722d529-d974-49e8-aaa4-748a54b25e60

Nice! I can make a release when this is merged.


serialization_struct: Getting merge from stable to compile

I merged the official stable branch into my fork of serialization_struct and is currently getting linker errors (undefined symbols) when running rdmd -unittest -main std\serialization\package.d. I'll keep you updated on my findings.

I'm not sure if you literally mean "merged", but please use rebase instead of merge when updating to latest upstream. A merge adds additional unnecessary commits and the commits specific to the serialization_struct will not be linear.
I have no idea if package.d contains all that it should contain. It was a while since I worked on this :wink:

The problem with rebasing is that the old branch will be invalid. You essentially have to force push which makes collaboration more difficult.
Is there any magic involved in the templates that could make the life hard for rdmd? Generally it should find all the files needed as long as all files import what they need.

The problem with rebasing is that the old branch will be invalid. You essentially have to force push which makes collaboration more difficult.

I think it's enough to rebase once, to get up to date with upstream. It should not be any problem if everyone involved knows about it. I would have rebased it anyway at some point. Then after that commit as usual.
It will be more difficult when it's time to squash the commits to remove all the unnecessary in between commits when the history is not linear.

Is there any magic involved in the templates that could make the life hard for rdmd? Generally it should find all the files needed as long as all files import what they need.

Templates will make life hard if separate compilation is used, but I assume it's not. Yes, i should fin all files. But I have no idea if package.d imports all the necessary files.

I just tried doing a rebase and git wasn't friendly with the merge conflicts, making the whole process very error prone. The merge was instant and only conflicted in the makefile.

Templates will make life hard if separate compilation is used, but I assume it's not. Yes, i should find all files. But I have no idea if package.d imports all the necessary files.

That's all I need to know, thanks!

How do you suggest squashing the commits when it's not a linear history?

git merge --squash works perfectly. Gives a very clean commit with only your changes.

But I see that they don't recommend using merge. It's a one time fix though so I guess it should be OK. If you want to do a rebase correctly I'll use that but I wasn't able to do the rebase in a confident manner so I won't try myself again.

I've rebased and pushed the serialization_struct branch.


enhancement - undo action & impacts

add choice in description while jumping and remove choice if undo the "jump"

i won't implement an undo jump. the description must be updated manually


Shorten ensureHTTP function

Just thought this looks a bit better with a regex. Feel free to decline the merge 😀.

Merged this PR into #19 . Thanks @DylanPiercey!


Do not continue comments after newline!

I love using docblockr, it speeds coding up significantly. However, what does not speed things up is hitting enter after writing a comment, and having to erase the automatically inserted comment characters.
I know with docblockr, if I use  after creating a comment, the next line is a normal code editing line. However, it is much more common for me to need code lines after a comment instead of a new comment line (for this, I just use multiline comments anyways...).
Is there a way (or potential future update) to have the user toggle this behavior?

This does not answer your question, but you can use shift-enter to not continue the comment and avoid deleting the characters, see documentation here under “Comment Extension”. It’s been a while since I’ve developed on docblockr but if other people think it’s worth it to add a way to disable single line comment extension (and it’s not already a feature slated to be added), then I would not be opposed to adding this and submitting a pull request.


element-plus type tools is not working

Vue - Official extension or vue-tsc version
2.1.0
VSCode version
1.92.2
Vue version
3.4.29
TypeScript version
5.4.5
System Info
System:
    OS: Windows 11 10.0.22631
    CPU: (12) x64 12th Gen Intel(R) Core(TM) i5-12400
    Memory: 3.51 GB / 15.75 GB
  Binaries:
    Node: 20.10.0 - ~\AppData\Local\Volta\tools\image\node\20.10.0\node.EXE
    npm: 10.2.3 - ~\AppData\Local\Volta\tools\image\node\20.10.0\npm.CMD
    pnpm: 9.7.1 - ~\AppData\Local\Volta\tools\image\pnpm\9.7.1\bin\pnpm.CMD
  Browsers:
    Edge: Chromium (127.0.2651.74)
    Internet Explorer: 11.0.22621.3527

Steps to reproduce
update Official, element-plus every component type is any
but if i use Official 2.0.28, it's working
What is expected?
element-plus types should woking
What is actually happening?
component type is any
Link to minimal reproduction
No response
Any additional comments?
No response

Same behaviour observed with latest Nuxt with PrimeVue components

Also happened to me, had to downgrade to v2.0.28 due to this issue.


Server fails to start when used in "proxy" mode

I'm looking to use this as a proxy in front of ollama running within a container elsewhere, but it fails because it expects the ollama binary to be in PATH of the process.
The problem can be traced to checkOllamaVersion, which tries to run ollama --version, but if you have provided a ollama-url command line argument it should bypass that check, assuming that the fetchApi passes (like startOllamaServer does).
This would mean that you can't get the version number of the ollama server, and thus, can't verify that the ollama server meets the minimum version required, but that's probably an acceptable trade off to have.

Good catch!
I'll fix that when I'm back

All good @sinedied, PR #4 is created which fixes this (plus one other issue I found when testing).


Added new scaling modes to splash screen

The only way to scale the boot splash image is the option "Application/Boot Splash/fullsize" in the project settings. This PR adds 4 new ways of scaling: keep_width, keep_height, cover and expand and keeps the previous one (calling it full_size).
The old fullsize option looked like this:

And I replaced it for a new option I called "scale":

Here I show some pictures to illustrate how each scaling mode works. In each case an icon with size 64x64 is used:

Scaling Mode
Horizontal (960x540)
Vertical (540x960)

disabled

keep_width

keep_height

cover

expand

full_size

The last one scales in the same way as the previous "fullsize" boolean option.
I updated both rasterizer_gles2.cpp and rasterizer_gles3.cpp. I also realised that the old "fullsize" option was not working when using GLES2.
This is my first PR so any tips, suggestions, recommendations... about anything are very welcome, specially if it's code-related.

Problem with this PR is that it breaks compatibility, so it can't be merged until 4.0.. maybe adding another property like expand_mode ?

Sorry for the late response.
I could keep the "fullsize" property and create a new dropdown called "expand_mode" which would only take effect when the "fullsize" property is true. The dropdown could have all the options listed in the description of this PR, renaming the choice "full_size" to just "keep". Would that be fine and would that keep compatibility?
Also, I removed my fork and now the PR is shown as commited from an "unknown repository". I've been researching and it looks like I will have to make a new PR. Sorry for this! My lack of experience on GitHub... ^^'.

I finally opened a new PR (because as I said I can't edit this PR anymore) here: https://github.com/godotengine/godot/pull/22488. I think that now this one becomes obsolete and can be rejected.

Closing as superseded by #22488.


Unable to view folders due to color scheme problems

This is Angular Project.
i am unable to see few folders properly, dist/ node_modules/

Duplicates #57

Hi @rdandnayak :wave:, thanks for your contribution :+1:
This is a known problem tracked in the pinned issue #57. Please see my comment that describes the current limitation for theme authors as well as comments from other users like the ones that describes how to fix it on your side.


Remove dependabot yarn workflow

Yarn 2 is now supported by dependabot

Codecov Report
All modified lines are covered by tests :white_check_mark:
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Files
Coverage Δ

components/grid-col/src/index.tsx
100.00% <100.00%> (ø)

components/heading/src/heading.tsx
100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes
:loudspeaker: Thoughts on this report? Let us know!.


fix: 'on completion' delete now renders correctly in Live Preview

Types of changes

Changes visible to users:

[x] Bug fix (prefix: fix - non-breaking change which fixes an issue)

Issue/discussion: #3043

Description
This fixes #3043 by removing some code which appears no longer to be necessary, and which caused '🏁 delete' in Live Preview to render the new task as DONE, even though in the markdown it was correctly TODO.
This is a revert of the behaviour added in 73d910490e1882dfb2d22ea0f53bfec42b357c96
Motivation and Context

#3043

How has this been tested?
In the test vault:

By running through the smoke tests in Live Preview to make sure they still work
By running through the reproduction I added in https://github.com/obsidian-tasks-group/obsidian-tasks/issues/3043#issuecomment-2393969888

Screenshots (if appropriate)
Checklist

[x] My code follows the code style of this project and passes yarn run lint.
[ ] My change has adequate Unit Test coverage.

Terms

[x] My contribution follows this project's contributing guide
[x] I agree to follow this project's Code of Conduct

Description
This fixes #3043 by removing some code which appears no longer to be necessary, and which caused '🏁 delete' in Live Preview to render the new task as DONE, even though in the markdown it was correctly TODO.
This is a revert of the behaviour added in 73d9104

...

How has this been tested?
In the test vault:

By running through the smoke tests in Live Preview to make sure they still work
By running through the reproduction I added in Repeated tasks are being added“ 🏁 After the 'delete' field, the task status cannot be reset #3043 (comment)

Some of the above turned out to be incorrect. I don't know whether I ran the smoke tests on an out-of-date build, or whether late-night tiredness and multiple open tabs meant that I didn't spot the incorrect rendering - but it was incorrect.
And the new problem was reported by a user in:

#3148

I've added more smoke tests, and come up with a more precise fix, in:

#3149

And released the improved fix in Tasks 7.12.1.


Include/Exclude Predicate for the "NuGet API"

Currently, IL Repack is coded with the feature parity towards ILMerge command line arguments in mind, but not using IL Repack as dependency.
As such, a good feature would be to be able to pass a predicate/Func<Type, bool>/delegate to the repacking process to determine which types should be included/merged in the process.
Currently, this can only be done with an attribute, which is why I need manual CECIL'ing to flag types that should be dropped in the process. It would be much better to allow the caller to just determine that via their criteria and only using predicates (i.e. NO RegEx or other solutions)

We support the exclude file, read more at this issue:
https://github.com/gluck/il-repack/issues/323
/internalize:"repackExclude.txt"

Keeping the issue open to document the exclude file in the command line help

Can you try this:
  <ItemGroup>
    <InputAssemblies Include="$(OutputPath)*.dll" />
    <InputAssemblies Remove="$(OutputPath)grpc_csharp_ext.x86.dll" />
    <InputAssemblies Remove="$(OutputPath)grpc_csharp_ext.x64.dll" />
  </ItemGroup>

Then build your project from the command line using msbuild /bl and open the resulting msbuild.binlog using the viewer from https://msbuildlog.com.
In the viewer, verify that the two files you want excluded are actually excluded from the InputAssemblies item.


Incorrectly mapping a varchar column to a file input

Environment

Ruby 2.6.5
Rails 6.1.3
Simple Form  5.1.0

Current behavior
I have a varchar column field named "poa" inside a table named "poas". When I use f.input on that field it is being wrapped in a div with a "file" class and has an input type='file'. I have recently upgraded from Rails 4.2 and Simple_Form 4.0 and this wasn't an issue before. I'm not having this problem with any other varchar field. The only things special about this column is it is named the same as the model, and there is also a poa_attachment column that is used for a carrierwave uploader. Not sure if there is some sort of magic happening in the background that is getting some wires crossed or something.
<%= f.input :poa, label: 'POA', wrapper_html: { class: 'columns medium-6' } %>

This is the resulting html

    POA
    <input class="file optional" type="file" name="record[poa_attributes][poa]" id="record_poa_attributes_poa">

Expected behavior
I expect the field to be treated as a text input. I can get around it by adding as: :string to the field, but I shouldn't have to do that.

Hey! Yeah, that's because you have poa_attachment column in your model. You can check how SimpleForm detects file inputs here https://github.com/heartcombo/simple_form/blob/86429bceb950096df3c29616f31bd5a5ce706c06/lib/simple_form/form_builder.rb#L592. So for your case you can use as: :string to avoid it. Unfortunately it's hard to make it work properly for all the case.


Please update the android support libs

The android support libs have been out for a while now. I cannot use this lib with the latest support lib version.
Will you merge a PR that updates this?

1

Well you could add a PR to fix it. @aritchie do you require it on v5 or the master?

I've been evaluating the upgrade for a while.  A pull request is unnecessary at this point as I already have one internal.  I have to finish testing with xamarin forms first as there is a large userbase that uses it that also uses this library.
In any case, I need the upgrade as well for mvvmcross 4 stuff.

@aritchie Thanks for the new version :)!

@aritchie Spoted a small mistake while updating the references. Made a PR for it (without pumping of version numbers, resources etc).

Thanks.  Strange that the nuget updated, but the reference did not :+1:


Binding an implicit implementation with a qualifier

Binding an implicit implementation seems to be possible without a qualifier, but adding a qualifier means the target of the binding must be specified. This caused a little head scratching.

    public static class A {

    }

    @Retention(RetentionPolicy.RUNTIME)
    @BindingAnnotation
    public @interface MyQualifier {

    }

    @Test
    public void test() {
        Guice.createInjector(new Module() {

            @Override
            public void configure(Binder binder)
            {
                binder.requireExplicitBindings();
                binder.bind(A.class); //This is OK.
                binder.bind(A.class).annotatedWith(MyQualifier.class); //Why is this not OK?
                //binder.bind(A.class).annotatedWith(MyQualifier.class).to(A.class); //This is OK.
            }
        });
    }

com.google.inject.CreationException: Unable to create injector, see the following errors:

1) No implementation for test.GuiceTest$A annotated with interface test.GuiceTest$MyQualifier was bound.
  at test.GuiceTest$1.configure(GuiceTest.java:42)

Correct, as mentioned on the wiki: "When specifying binding annotations, you must still add the target binding, even it is the same concrete class." https://github.com/google/guice/wiki/UntargettedBindings
This is because declaring a qualified binding key and then not binding it to something is more often than not a programming mistake. It's also easier to read/follow qualified bindings when they have an explicit target.

This is WAI.


Moving github workflows to python 3.10 & adding backwards compat steps

Github workflows will now default to python 3.10 and torch 1.12
Test plan: unit tests

didn't check the branch LGTM


[REQUEST] Show all icons in system tray

Hello
First, thanks for your work
Can you add an option to always show icons on system tray on win11?

On Windows 10, it was sufficient to add
[HKEY_CURRENT_USER\Software\Microsoft\Windows\CurrentVersion\Explorer]
"EnableAutoTray"=dword:00000000

to the registry to always show all tray icons. On Windows 11, the same task seems to be surprisingly difficult. Some people found the IsPromoted registry value, and a PowerShell script such as
Set-ItemProperty -Path 'HKCU:\Control Panel\NotifyIconSettings\*' -Name 'IsPromoted' -Value 1 -Type 'DWord' -ErrorAction 'SilentlyContinue';
can set this value for all existing icons – but this will not affect icons added afterwards. One could run the script each time a user logs on, but this is still not very reactive, and you would see a PowerShell window pop up every time.
Nevertheless, I have added a new checkbox Always show all tray icons to the form, which runs a PowerShell script as described above.

Can confirm W11 has trashed this.
I had set the script to run once an hour, but quickly moved on to a 3rd party start menu because of this and other start menu changes.
I hate not knowing whats running on my system at all times, and while not everything is in the tray, at least anything that would normally be there shows up and programs with issues don't go unnoticed like it does when it hides them.

Don't worry, thank you for your effort :)

Several users had complained about the PowerShell window popping up at each logon. I have now replaced it with a scheduled task that runs silently in the background and executes a lightweight .vbs script. The task is triggered for each user at logon and
once every minute.


How to check a user's permissions when accessing API resources?

I want to authorize only users with a specified permission to use API resources.
According to the FastEndpoints documentation instructions, I set up claims about the user, role, and permissions to create a token to send to the user. If that person has some access to the resource then it seems reasonable. But if a user has several hundred permissions, the following problems will occur:

Bearer token in the request header will be very heavy.
May lead to security leaks.
Is there a way to secure the token, and check whether a user's access permission is valid or not, without relying on the token?
My idea is to write a service like IUserService to check whether the userId in the incoming request has that permission or not? If so, continue processing in the Handle method, otherwise return an unauthenticated error.

public class CreateNumerologyMapEndpoint(INumerologyService numerologyService, IStringLocalizer<CreateNumerologyMapEndpoint> localizer) : Endpoint<CreateNumerologyMapRequest, ApiResponse<NumerologyMapResponse>>
{
    private readonly IStringLocalizer _s = localizer;

    public override void Configure()
    {
        Post(EndpointConstants.Numerology);
        Permissions("Permissions.Numerology.Create");
    }

    public override async Task HandleAsync(CreateNumerologyMapRequest req, CancellationToken ct)
    {
        // handle logic here
    }
}

This is a sample JWT Token
{
  "id": "69537306-8c44-4d05-914c-2f5d810f8dc1",
  "emailaddress"<EMAIL_ADDRESS>  "givenname": "root",
  "surname": "Admin",
  "imageUrl": "",
  "phone_number": "",
  "username"<EMAIL_ADDRESS>  "type": "Pro",
  "tenant": "root",
  "permissions": [
    "Permissions.Dashboard.View",
    "Permissions.Hangfire.View",
    "Permissions.Users.View",
    "Permissions.Users.Search",
    "Permissions.Users.Create",
    "Permissions.Users.Update",
    "Permissions.Users.Delete",
    "Permissions.Users.Export",
    "Permissions.UserRoles.View",
    "Permissions.UserRoles.Update",
    "Permissions.Roles.View",
    "Permissions.Roles.Create",
    "Permissions.Roles.Update",
    "Permissions.Roles.Delete",
    "Permissions.RoleClaims.View",
    "Permissions.RoleClaims.Update",
    "Permissions.Numerology.View",
    "Permissions.Numerology.Search",
    "Permissions.Numerology.Create",
    "Permissions.Numerology.Update",
    "Permissions.Numerology.Delete",
    "Permissions.Numerology.Export",
    "Permissions.Tenants.View",
    "Permissions.Tenants.Create",
    "Permissions.Tenants.Update",
    ...
    ...
    ...
    "Permissions.Tenants.UpgradeSubscription"
  ],
  "role": "Admin",
  "aud": "https://helloworld.com",
  "iss": "HelloApi",
  "exp":<PHONE_NUMBER>,
  "iat":<PHONE_NUMBER>,
  "nbf":<PHONE_NUMBER>
}

first try the source generated permissions/access control feature. the 3 digit generated permission codes might be able to reduce the size of the jwt payload.
if that's still too big, you can write a session based authentication handler like this, where the client will be sending just a session id and the auth handler will be looking up the users roles/claims/permissions on every incoming request.

Your idea of creating permissions with 3 characters is also great, but perhaps there should still be other options that do not assign permissions to tokens. Is there any other method like what I do below?
In which I have a service IUserService to check whether the user has the specified permission or not?
Previously when using Controller. I write a class like this:
public class MustHavePermissionAttribute(string action, string resource) : AuthorizeAttribute, IAuthorizationRequirement, IAuthorizationRequirementData
{
    public string Permission { get; } = FSHPermission.NameFor(action, resource);

    public IEnumerable<IAuthorizationRequirement> GetRequirements()
    {
        yield return this;
    }
}

internal class PermissionAuthorizationHandler(IUserService userService) : AuthorizationHandler<MustHavePermissionAttribute>
{
    protected override async Task HandleRequirementAsync(AuthorizationHandlerContext context, MustHavePermissionAttribute requirement)
    {
        if (context.User?.GetUserId() is { Length: > 0 } userId &&
            await userService.HasPermissionAsync(userId, requirement.Permission))
            context.Succeed(requirement);
    }
}

In any method in the Controller, I want users to have a specified permission to access the API as follows:
[HttpGet("/test")]
[MustHavePermission("Test", "View")]
public IActionResult Test()
{
    return Ok("Hello world!");
}

@ngoquoctoandev you could potentially look at creating a pre processor to handle it.
Not used them my self, but quick glance it should work - https://fast-endpoints.com/docs/pre-post-processors#short-circuiting-execution

@ngoquoctoandev a pre-processor would work like @IeuanWalker mentioned.
however the way we attach custom attributes to endpoints in FE is like this:
public override void Configure()
{
    ...
    Options(x=>x.WithMetadata(new MustHavePermission("Test", "View")));
}

just doing that should work. if it doesn't, attach a small repro project with your approach and i'll look in to it.
the other, much cleaner alternative would be to retrieve the current user's permissions from the database and stick them in the user principal using a custom claims transformation.
that way you can simply do the Permissions(...) call in endpoint config and everything will work according to the design of FE.

@ngoquoctoandev created an example for you: https://gist.github.com/dj-nitehawk/220363f14e649a2cb850d61f9bd793b5


Documentation on database structure

Is there some documentation on the Firebase database structure? I want to work on updating the PHP debugger agent to work with this new debugger and would need to know the details of the data that is saved there

There isn't any documentation on the structure yet, but we can add that fairly quickly.
It's great to hear that you're interested in updating the PHP debugger agent to use the Firebase backend.  Is the intention to do this work on a fork of the PHP agent or the main repo?  I want to make sure that any teardown related to the deprecation of the Cloud Debugger API will not damage your efforts.
It's also worth noting that there is no firebase-admin sdk for PHP, so you'd likely have to use the REST api.

My plan was to create a fork of https://github.com/googleapis/google-cloud-php-debugger. I will then create a PR for this repo but hopefully if this gets removed the fork will remain.
Regarding access to Firebase I was planning to use https://github.com/kreait/firebase-php which seems to provide the needed functionality

Hi Carlos,
The DB layout is now documented in snapshot-debugger/docs/SCHEMA.md. You may also find the Firebase DB client implementations of the other Snapshot Debugger agents helpful as a reference:

Java Agent Firebase Client
Python Agent Firebase Client
Node.js Firebase Client

If you have any questions along the way or find some of the documentation unclear please let us know.
Regards,
Jason

I created this PR to update the PHP agent so that it now connects to Firebase and is able to work with this tool. https://github.com/googleapis/google-cloud-php-debugger/pull/4
I do not have the time to complete this PR with tests and everything, so just adding this here for reference for now in case someone else needs it. If I find the time to complete the PR and make it mergeable in the future, I'll let you know

Happy to hear that you were able to get it to work.  Hopefully that is useful for others in its current state.  I'll try to find somebody who can look at the PR from my side.

Closing this as the database schema has been documented.


Release v1.2.17

@TomaszAIR log from successful update of NS5x TGL on rc4: https://paste.dasharo.com/?57663b05e9db5404#FRvLVqxX8Hbw3nxXMnwK8pRa285Say1uQ83CcARkhcGW

@mkopec thanks, I will wait for NS7x and then will release v1.2.17


Clean up PROD derived datasets in Error status

These are left over from failed PROD runs and should be removed from the ES index before release.

Done.


Could not find folder with role 'all'.

First step with Nylas Mail on Ubuntu 16.04, using a Gmail account the Inbox just shows a "Hooray! you're done" message, I only can browse the Spam folder. Sending a mail seems ok but a message appears :
"Error, Your message successfully sent; however, we had trouble saving your message, "test", to your Sent folder.
Could not find folder with role 'all'.
"

I have this issue as well, running Nylas Mail 2.0.32 on Ubuntu 16.04. Clicking 'Show details' in the error dialog does nothing.

Yes, I do have the same problem... Nylas Mail 2.0.32 and Ubuntu 16.04

Now I'm using the Windows 10 client and have the same problem. I don't think is a OS problem...


out of memory when juicefs rmr on big dirs

What happened:
when use juicefs rmr to remove several directories, each directory hav 500k files, it will OOM. juicefs process will cost 16.5G memory.
refer to the heap in the attachment
What you expected to happen:
rmr should not cost much memory and should not OOM。
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?
Environment:

JuiceFS version (use juicefs --version) or Hadoop Java SDK version:
Cloud provider or hardware configuration running JuiceFS:
OS (e.g cat /etc/os-release):
Kernel (e.g. uname -a):
Object storage (cloud provider and region, or self maintained):
Metadata engine info (version, cloud provider managed or self maintained):
Network connectivity (JuiceFS to metadata engine, JuiceFS to object storage):
Other
heap.zip
s:

Fixed by #2214


semi: [2, "never"] not erroring when semi-colon is after a static property

I originally filed this issue on the original eslint repo, but I was told that I should be filing it here (eslint/eslint#5366)
ESLint Version: 2.1.0
Babel ESLint Version: 5.0.0
If I have semi: [2, "never"] set, I would imagine this would cause an error:

export class MyClass extends Component {
  static propTypes = {
  }; // <-- expected this to give an error

  render () {
    // code
  }
}

Currently, I am getting 0 output from ESLint about this.

Actual source code
ESLint Config

It's not going to error because the rule itself doesn't know about class properties (it's not in https://github.com/eslint/eslint/blob/master/lib/rules/semi.js#L160-L179).
We would have to fork the rule in https://github.com/babel/eslint-plugin-babel to include class properties and use that rule instead.

@hzoo should i request this change in eslint-plugin-babel?

Yeah although it's basically the opposite of https://github.com/babel/eslint-plugin-babel/issues/43 so I think it's the same issue - you can post a comment there
Basically semi needs to support class properties in general

1 for warnings and 2 for errors
http://eslint.org/docs/developer-guide/working-with-custom-formatters#the-message-object
"semi": 2
{
    "parserOptions": {
        "ecmaVersion": 6,
        "sourceType": "module",
        "ecmaFeatures": {
            "jsx": true
        }
    },
    "rules": {
        "semi": 2
    }
}

http://eslint.org/docs/user-guide/configuring
http://eslint.org/docs/rules/no-compare-neg-zero
"semi": [2, "always"]

"semi": [2, "always"],

http://eslint.org/docs/rules/no-compare-neg-zero


Is it possible to change Icon everyday with Adaptive Icon?

I have seen Google Calendar is Changing it's date everyday within the icon. Is this can be done using Adaptive Icon's or is there is something else to it. I am pretty sure they haven't bundled 31 icons as this would be resource consuming.
PS: I ❤ your plaid app though

Thanks!
No; it's not possible to dynamically change your icon. I'm not certain how Calendar does it but likely by working with the Launcher.


Duration from media item

I am trying to get the duration from the media item itself instead of player.getduration() as player.getduration() gives C.TIME_UNSET until player gets ready

The duration is available as soon as the player know the duration.
If you know the duration ahead of time, then you can add it to the MediaMetadata.extras with a custom key that can define for your app.

@marcbaechinger
Does media metadata have duration?
How to get that from extras ?
Can you please mention some sample code lines

@marcbaechinger Does media metadata have duration? How to get that from extras ? Can you please mention some sample code lines

Can parser be used for it?

The duration is available as soon as the player knows the duration.
If you know the duration ahead of time, then you can add it to the MediaMetadata.extras with a custom key that can be defined by your app.

Does media metadata have duration?
How to get that from extras ?
Can you please mention some sample code lines
Can parser be used for it?

Player.getDuration() shows the duration of the media as known to the player at the earliest moment possible. If Player.getDuration() returns C.TIME_UNSET, then the duration is not known to the player yet. There is not other API that then knows the duration.
If you as the developer know the duration, as for instance you have the duration of your media stored in a database, then you can populate the MediaMetadata with that duration from the database before you pass the MediaItem to the player. Please consult our developer guide on how you can build a media item.


Change representation of ü in the package.json author field

The "ü" character in the "author" field was represented using the
character 0xfc. This is correct for Unicode, but is an invalid character
in UTF-8. From https://en.wikipedia.org/wiki/Specials_(Unicode_block):

The first and last byte are valid UTF-8 encodings of ASCII, but the
middle byte (0xFC) is not a valid byte in UTF-8.

The presence of 0xFC is a problem because it causes tools that are
strict about JSON files having UTF-8 encoding to fail.

Thanks!


env: don't panic when name is empty

All of this is basically to say that the fork only happens in some circumstances

Indeed, on my system it calls clone (which is called by posix_spawnp) according to strace . However, I think the main point of this comment is that it would not be necessary to spawn a child process and that we could keep using the current process. I think it's probably not worth changing, as you pointed out using the standard library is much simpler.

Thanks for the PR! Nice that you added such a detailed comment and a test :)

I spent a lot of time researching what was the correct way to do this. Might as well document it and possibly save someone else's time in the future :)

All of this is basically to say that the fork only happens in some circumstances

Indeed, on my system it calls clone (which is called by posix_spawnp) according to strace . However, I think the main point of this comment is that it would not be necessary to spawn a child process and that we could keep using the current process. I think it's probably not worth changing, as you pointed out using the standard library is much simpler.

Will remove the FIXME and change the comment to specify why fork/clone is called.


Refactor region SST struct to hold an UUID as SST key to reduce overhead

What type of enhancement is this?
Performance
What does the enhancement do?
SST file struct FileMeta uses a String as the file name:
https://github.com/GreptimeTeam/greptimedb/blob/373d4f9b9c55cef46bbac599bab1da45c1e20819/src/storage/src/sst.rs#L294
In fact, SST file names are always UUID v4:
On flush:
https://github.com/GreptimeTeam/greptimedb/blob/e17d5a1c41147fd8033cf87017c5099a53158d34/src/storage/src/flush.rs#L252
On compaciton:
https://github.com/GreptimeTeam/greptimedb/blob/ff48d2706923993b49f150f00d482d9c0586b07f/src/storage/src/compaction/task.rs#L177
So we can simply store an 128 bit UUID struct in FileMeta to alleviate heap allocation.
Implementation challenges
No response

May I take this one?

@vinland-avalon Please go ahead

The names of files that flushed to external storage must still have an extension of .parquet. The improvement is about how struct FileMeta get stored in memory.  Am I right?

The names of files that flushed to external storage must still have an extension of .parquet. The refactoring is about how struct FileMeta get stored in memory -- we can ingnore the extension and just store uuid here. Am I right?

We could add a typedef or a new type FileId to avoid exposing Uuid directly. I'd prefer to use new type pattern as we can add some method to FileId, similar to the ProcedureId
https://github.com/GreptimeTeam/greptimedb/blob/df751c38b4014641da0cf59a9e13ab5d4d44355b/src/common/procedure/src/procedure.rs#L166-L168

Make sense. Thx!

Hi, I've finished my coding. I'm ready for your reviews :) @v0y4g3r @evenyag


add hacbs release-service manager

Signed-off-by: Johnny Bieren<EMAIL_ADDRESS>

Relates to https://github.com/redhat-appstudio/release-service/pull/117

@Michkov can you take a look when you have time?

getting these errors/warnings:
E1014 13:33:11.278571 1 reflector.go:138] pkg/mod/k8s.io/client-go@v0.24.3/tools/cache/reflector.go:167: Failed to watch *v1alpha1.ApplicationSnapshotEnvironmentBinding: unknown (get applicationsnapshotenvironmentbindings.appstudio.redhat.com)
W1014 13:33:12.115780 1 reflector.go:324] pkg/mod/k8s.io/client-go@v0.24.3/tools/cache/reflector.go:167: failed to list *v1alpha1.ApplicationSnapshotEnvironmentBinding: applicationsnapshotenvironmentbindings.appstudio.redhat.com is forbidden: User "system:serviceaccount:release-service:controller-manager" cannot list resource "applicationsnapshotenvironmentbindings" in API group "appstudio.redhat.com" at the cluster scope: access denied
E1014 13:33:12.115812 1 reflector.go:138] pkg/mod/k8s.io/client-go@v0.24.3/tools/cache/reflector.go:167: Failed to watch *v1alpha1.ApplicationSnapshotEnvironmentBinding: failed to list *v1alpha1.ApplicationSnapshotEnvironmentBinding: applicationsnapshotenvironmentbindings.appstudio.redhat.com is forbidden: User "system:serviceaccount:release-service:controller-manager" cannot list resource "applicationsnapshotenvironmentbindings" in API group "appstudio.redhat.com" at the cluster scope: access denied
E1014 13:33:48.419923 1 reflector.go:138] pkg/mod/k8s.io/client-go@v0.24.3/tools/cache/reflector.go:167: Failed to watch *v1alpha1.Release: unknown (get releases.appstudio.redhat.com)
E1014 13:33:51.374803 1 reflector.go:138] pkg/mod/k8s.io/client-go@v0.24.3/tools/cache/reflector.go:167: Failed to watch *v1beta1.PipelineRun: unknown (get pipelineruns.tekton.dev)
W1014 13:33:52.736114 1 reflector.go:324] pkg/mod/k8s.io/client-go@v0.24.3/tools/cache/reflector.go:167: failed to list *v1beta1.PipelineRun: pipelineruns.tekton.dev is forbidden: User "system:serviceaccount:release-service:controller-manager" cannot list resource "pipelineruns" in API group "tekton.dev" at the cluster scope: access denied
E1014 13:33:52.736156 1 reflector.go:138] pkg/mod/k8s.io/client-go@v0.24.3/tools/cache/reflector.go:167: Failed to watch *v1beta1.PipelineRun: failed to list *v1beta1.PipelineRun: pipelineruns.tekton.dev is forbidden: User "system:serviceaccount:release-service:controller-manager" cannot list resource "pipelineruns" in API group "tekton.dev" at the cluster scope: access denied
E1014 13:35:16.296493 1 reflector.go:138] pkg/mod/k8s.io/client-go@v0.24.3/tools/cache/reflector.go:167: Failed to watch *v1alpha1.Component: unknown (get components.appstudio.redhat.com)
W1014 13:35:17.548884 1 reflector.go:324] pkg/mod/k8s.io/client-go@v0.24.3/tools/cache/reflector.go:167: failed to list *v1alpha1.Component: components.appstudio.redhat.com is forbidden: User "system:serviceaccount:release-service:controller-manager" cannot list resource "components" in API group "appstudio.redhat.com" at the cluster scope: access denied
E1014 13:35:17.548912 1 reflector.go:138] pkg/mod/k8s.io/client-go@v0.24.3/tools/cache/reflector.go:167: Failed to watch *v1alpha1.Component: failed to list *v1alpha1.Component: components.appstudio.redhat.com is forbidden: User "system:serviceaccount:release-service:controller-manager" cannot list resource "components" in API group "appstudio.redhat.com" at the cluster scope: access denied

But that can be fixed/investigated later.


Fix field masks used in list queries

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 94.85%. Comparing base (6245289) to head (726f013).
Report is 41 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #25   +/-   ##
=======================================
  Coverage   94.85%   94.85%
=======================================
  Files           6        6
  Lines         369      369
=======================================
  Hits          350      350
  Misses         19       19

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Dump Creation In Blame Mode added

Added option to create dump in blame mode by --blame:dump|/blame:dump

@vagisha-nidhi,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by Microsoft. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

https://github.com/Microsoft/vstest/pull/1458


build: secured CDIR variable, now should work with spaces in path

Generally I think it's a good idea.
But functional test's build is always failing with:
can't load package: package github.com/coreos/fleet/functional: cannot find package "github.com/coreos/fleet/functional" in any of:
	/home/core/go/src/github.com/coreos/fleet/functional (from $GOROOT)
	/home/core/fleet/functional/gopath/src/github.com/coreos/fleet/functional (from $GOPATH)

Interesting.

Ok, the build error happens because both functional/test and build-env try to define a variable ${CDIR}, although functional/test sources build-env. You need to rename ${CDIR} into another name, either in functional/test or build-env. I think a correct fix would be to update build-env, as that's also imported by other files.

fixed in other way. just use CDIR=$(cd dirname "$0/.." && pwd)

LGTM


Merge ThrowableUtils into ThrowableUtil.

Motivation:
We should merge ThrowableUtils into ThrowableUtil as this name is more consistent with the naming of utility classes in netty.
Modifications:
Merge classes.
Result:
More consistent naming

@Scottmitch PTAL

lgtm

Cherry-picked into 4.0 (9d29c50) and 4.1 (9d29c50)


[FEATURE] Automating Dependency Upgrade of LinkFree

Description
Description:
I have been considering automating the process of upgrading dependencies for LinkFree. By doing so, we can eliminate the need for manual updates and avoid using deprecated or outdated dependencies that might pose security vulnerabilities, bugs, or performance issues. This automation would not only reduce the maintenance burden but also save time.
Solution:
In my quest for a solution that offers simple configuration and the ability to schedule automated dependency updates, I stumbled upon the perfect match in the open-source tool called "Renovate." This remarkable tool ticks all the boxes, providing the control I desired and empowering us to effortlessly manage dependency upgrades on our own terms.
Way Forward:
I've taken the initiative to integrate it into my forked repository to gather initial feedback. I invite you to have a look at the following PR: Link.
If someone wants to collaborate and work on this issue. Feel free to ask. Thanks
Screenshots
Version Upgrade one by one:

Version Upgrade in Batch:

Additional information
No response

This looks super interesting and useful 👍

It requires dependabot setup. I can work on the configuration setup.


Error checkPermission() was not implemented by android

I also having the same problem when running on android studio
[Barcode Scanner error: Error: BarcodeScanner.checkPermission() is not implemented on android]
here is my implementation,
    methods: {
        async startScan() {
        try {
            // Check camera permission
            await BarcodeScanner.checkPermission({ force: true });

            // Make background of WebView transparent
            BarcodeScanner.hideBackground();

            const result = await BarcodeScanner.startScan();

            // If the result has content
            if (result.hasContent) {
            console.log(result.content);
            // You can process the scanned content as per your requirements
            // For example, you can display it on the screen or send it to a server.
            }
        } catch (error) {
            console.error("Barcode Scanner error:", error);
        }
    },

However , it works well on webview via ionic serve. The camera get accessed smoothly and scan work nicely.
*I have disable the checkPermission() line , but it comes out with new error for scan
Sending plugin error: {"save":false,"callbackId":"4461136","pluginId":"BarcodeScanner","methodName":"startScan","success":false,"error":{"message":"Attempt to invoke virtual method 'void androidx.camera.view.PreviewView.setScaleType(androidx.camera.view.PreviewView$ScaleType)' on a null object reference"}}

I have same problem. Are there any solutions?


Templating language tags in javascript strings break syntax highlighting

When trying to implement syntax highlighting for Mojolicious templates (highlight.js pull request, mojo pull request), we found that highlight.js can't handle template tags inside javascript strings.

This problem also shows up with the ERB syntax (which I used as reference):

It doesn't happen with PHP tags, which are handled internally in the xml.js language definition:

I suspect this can only be fixed in xml.js, not by modifying an external language definition.

Good catch, thanks! I'll look into it as time allows but it might prove to be a real tough one…

same problem,when i use mustache tag {{  and }}
in markdown:
https://github.com/zicai/zicai.github.com/blob/master/_posts/2015-03-08-mustache-intro.md

with highlight.js
http://zicai.github.io/lessons/2015/03/08/mustache-intro/

the words in {{ and }}  are missing

Removing 8.8 as it is indeed proving to be a tough one.
P.S. @zicai: the issue you mentioned has absolutely nothing to do with the original one. Moreover, it's not a scripting issue at all as those missing words are already missing from your HTML source.

@zicai If you need the mustache tags {{ }} to render, try wrapping them in {% raw %} {% endraw %}.
http://stackoverflow.com/questions/24102498/escaping-double-curly-braces-inside-a-markdown-code-block-in-jekyll

The issue with escaping two curly braces  {{ }} still exists. Anyone know of a solution?

I just stumbled into this one too.
In my case I'm working with a Vue.js app, the fence code that is throwing the error is:
<template lang="pug">
v-app
  v-toolbar
    v-toolbar-title {{ $static.metaData.siteName }}
    v-spacer
    v-toolbar-items
      v-btn(flat exact :to="{ name: 'home' }") Home
      v-btn(flat :to="{ name: 'about' }") About

  v-container
    slot
</template>

<static-query>
query {
  metaData {
    siteName
  }
}
</static-query>

The error I'm seeing when enabling hljs inside markdownit is:
 ERROR  [Vue warn]: Error in render: "TypeError: Cannot read property 'metaData' of undefined"

When hljs is disable then no errors are thrown.

It's as if it was trying to parse and execute the code inside {{ }}.

This is related to Vue.
Solved by changing the hljs invocation from this:
hljs.highlight(lang, str).value

To this:
let highlighted = hljs.highlight(lang, str).value
highlighted = highlighted.replace(/{{(.*)}}/g, '{{ "\\{\\{$1\\}\\}" }}')
return highlighted

There I an alternative Vue language definition that might help  (I've not tried  it):   https://github.com/highlightjs/highlightjs-vue

I think this problem is misstated (or at the very least oversimplified):

Templating language tags in javascript strings break syntax highlighting

The reason this doesn't work is because template tags are NOT Javascript. :-)  I'm not sure you should actually expect this to work IMHO.  I'm not sure it's so great we added automatic PHP support to xml, though I guess that ship has sailed. :-)  There is also a cost here.  Now you can't build a working Hightlight.js with "xml" without bundling "php" as well, although perhaps the parser just quietly ignores missing languages.
But if that's the case I'm not sure how it can ever properly detect the END of the templating, since it can't know about any escaping being using inside it (ie):
<h1>
I'm just a <%= "HAPPY %>" bunch of more Ruby code here that isn't html %>
</h1>

Although I guess if you weren't actually using that templating engine then it wouldn't so much matter if it silently failed and also never matched.

The real problem is many (all?) template engines are generic.  You can use ERB with JS, HTML, XML, JSON, CoffeeScript, and much more... so if we're going to add to one, we need to add to all - or we need to understand clearly why JS is such an exception compared to the others. AFAIK, ERB is more commonly used with HTML than JS - so if JS is an exception I think HTML would be a larger exception.
IE, this is NOT just an issue with the Javascript parser, and shouldn't be though of as such.  Again, unless we first define why it's such an exception vs the other cases.  We made need a better way to think about templating engines in general.
Perhaps it'd be best to flag a snippet as "php/html".... or "erb/js"...

Since most major template engines "trump everything" (ie, the "start" macros can show up literally anywhere) perhaps there is some built-time option we could add to say "compile highlight.js with ERB support"... and then magically you can just toss ERB in the middle of any language - and it will "just work"...  If you look closely you can see even say the built-in support for PHP starts to fail if you look at edge cases, like PHP inside attribute strings... which should definitely be possible... you should be able to break in anywhere:
![Screen Shot 2019-10-06 at 1 25 31 AM](https://user-images.githubusercontent.com/6473/66264688-4fd7b680-e7d8-11e9-8957-4d963c52455e.png)

I forgot (well, actually I never know) that in our own repo we have example of how you'd actually do this correctly, such as VBScript-HTML:
function(hljs) {
  return {
    subLanguage: 'xml',
    contains: [
      {
        begin: '<%', end: '%>',
        subLanguage: 'vbscript'
      }
    ]
  };
}

@dotandimet You still around?  I think what you want is your own language that's defined similar to the VBScript/HTML language.   It's a bad idea (and simply not practical) to add every possibly templating language under the sun to xml.  Adding PHP to xml in the past was probably short-sighted without considering what that would look like long-term.
Also it becomes almost impossible when as soon as have different templating languages that share similar syntaxes... (it looks like ERB and Mojolicious are very similar).  Having a separate language per templating engine removes all the ambiguity.

Closing as asked and answered and lack of activity.


AVRO-3716: No longer build under JDK 8 to allow new plugin versions

https://issues.apache.org/jira/browse/AVRO-3716
Because we build under JDK 8, 11, 17 and 19 some build plugins can no longer run as they require JDK 11.
To allow using these newer plugin versions this drops the support of building with JDK 8.
The final code MUST still be JDK 8 compatible.

Obsolete experiment


feat: add list of rules on main ruleset pages

Thanks for the PR @Whomy09

@Whomy09 thanks for the pr 🙏


Add indentation to logging

Beat logging can be indented.  See conversation https://github.com/perrygoy/screenpy/discussions/21#discussioncomment-403009
Still need to consider allowing aside to also be indented in a way that makes it easy to manage.

submitted PR https://github.com/perrygoy/screenpy/pull/24


Adding basic string types - NonEmptyString, Character

I have found myself declaring simple string types a lot and thought it would be nice to have them here.
I suggest adding the following types:

NonEmptyString to indicate some string cannot be empty
Character type to indicate a string has length of exactly 1.

What do you think? Is it too pedantic? Would you use these types?

Sounds great!
I often define these types myself, it would be really useful.

These must be very common then
I thought I was the only one using these until I've seen your issue!

tbh I think these types would even fit the main pydantic lib. The char is classic and is a built-in in C & Java. The non empty string is also a great idea and would be super useful


Fix grad of variable size on extract_image_patches

This fixes #11651.
The unit test is taken from #17420.

@rthadur Is it possible this PR to be included with 1.14?

Sadly no, 1.14 is about to be released.
On Tue, Jun 18, 2019, 06:51 Vasileios Lioutas<EMAIL_ADDRESS>wrote:

@rthadur https://github.com/rthadur Is it possible this PR to be
included with 1.14?
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/pull/29815?email_source=notifications&email_token=AAABHROMUBPLFHLJMYCD7PTP3DR63A5CNFSM4HYOFJOKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODX6VCXY#issuecomment-503140703,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAABHRM7VWLBHNGQA457T73P3DR63ANCNFSM4HYOFJOA
.

@alextp Okay thank you for letting me know.

Yes
On Tue, Jun 18, 2019 at 1:12 PM Vasileios Lioutas<EMAIL_ADDRESS>wrote:

@lioutasb commented on this pull request.
In tensorflow/python/kernel_tests/extract_image_patches_grad_test.py
https://github.com/tensorflow/tensorflow/pull/29815#discussion_r295003548
:

   in_val = array_ops.placeholder(

     shape=test_shape,

     dtype=dtypes.float32)

   feed_dict = {in_val: np.random.random(in_shape)}

   for padding in ['VALID', 'SAME']:

     out_val = array_ops.extract_image_patches(in_val, test_case['ksizes'],

                                               test_case['strides'],

                                               test_case['rates'], padding)

     out_val_tmp = out_val.eval(feed_dict=feed_dict)

     out_shape = out_val_tmp.shape

     err = gradient_checker.compute_gradient_error(in_val, in_shape,

                                                   out_val, out_shape)

     print('extract_image_patches gradient err: %.4e' % err)

I see the same print used earlier here
https://github.com/tensorflow/tensorflow/blob/e67dc75850fe3a54071fdd9db6ad91906ad51797/tensorflow/python/kernel_tests/extract_image_patches_grad_test.py#L104,
should I remove that as well?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/pull/29815?email_source=notifications&email_token=AAABHRMW2JA4MT2QNLMWME3P3E6SRA5CNFSM4HYOFJOKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOB35P2ZA#discussion_r295003548,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAABHRLV2XFCB5JDAHP7DSTP3E6SRANCNFSM4HYOFJOA
.

--

Alex

@alextp I fixed the pylint issue. Now it should be ok to merge automatically once you approve it again.


visualisation of clusters

I am playing a bag file (Velodyne LiDAR 32 channel). Then I am running this command
rosrun depth_clustering show_objects_node
How to visualize the clusters using rviz?

You would need to write a node that publishes the clusters on some topic to visualize them in rviz. Currently, this is not implemented.

Is there any other way (using your program ) to visualize the cluster in real time (as shown in video)??


Addition of "Fleet Light"

Add a new light-mode theme that is based off of "Fleet Light" in JetBrains Fleet.

Any update on this? They's color palette looks really nice and it would be great to have it in VS Code too. Thanks.

I am also interested in the light theme. Any update?

me too

waiting someone can create a pull request of a rough version of 'Fleet Light' :)


Error MSB3971: The reference assemblies for ".NETFramework,Version=v5.0" were not found. You might be using an older .NET SDK to target .NET 5.0 or higher. Update Visual Studio and/or your .NET SDK. (MSB3971)

/Applications/Visual Studio.app/Contents/Resources/lib/monodevelop/bin/MSBuild/Current/bin/Microsoft.Common.CurrentVersion.targets(3,3): Error MSB3971: The reference assemblies for ".NETFramework,Version=v5.0" were not found. You might be using an older .NET SDK to target .NET 5.0 or higher. Update Visual Studio and/or your .NET SDK. (MSB3971)
I am using visual studio for mac and my version is up to date, if I create a new project with version dotnet 5 all is well but as soon as I try to build the project I have worked on windows I get this error. I have also installed the latest version of SDK

/cc @rainersigwald - can you assist?

What SDK version and VS for Mac version are you running?
Make sure you have the minimum version required for the SDK version:
https://dotnet.microsoft.com/download/dotnet/5.0

In addition to Maira's questions, do you have a global.json file in your repo? If so can you share its contents?

I'm getting the same problem using VS2019 Preview (I'm on Windows 10).  When I click on Help/Check for Updates it comes back with, “Visual Studio is up to date”.
Per your response above I then went to https://dotnet.microsoft.com/download/dotnet/5.0 and chose SDK 5.0.300 for Windows x64.  When finished it reported:
.NET SDK 5.0.203
.NET Runtime 5.0.6
I tried running the same command again that originally raised this error: docker build . -t
Output: the same error:
ERROR [build 6/6] RUN dotnet build "SpecialOffers.csproj" -c Release -o /api/build                                                                                1.6s

[build 6/6] RUN dotnet build "SpecialOffers.csproj" -c Release -o /api/build:
#13 0.627 Microsoft (R) Build Engine version 16.7.2+b60ddb6f4 for .NET
#13 0.627 Copyright (C) Microsoft Corporation. All rights reserved.
#13 0.627
#13 1.163   Determining projects to restore...
#13 1.427   All projects are up-to-date for restore.
#13 1.550 /usr/share/dotnet/sdk/3.1.409/Microsoft.Common.CurrentVersion.targets(1177,5): error MSB3971: The reference assemblies for ".NETFramework,Version=v5.0" were not found. You might be using an older .NET SDK to target .NET 5.0 or higher. Update Visual Studio and/or your .NET SDK. [/src/SpecialOffers.csproj]

This is my Dockerfile:
FROM mcr.microsoft.com/dotnet/aspnet:5.0 AS base
WORKDIR /app
EXPOSE 80
EXPOSE 443
FROM mcr.microsoft.com/dotnet/sdk:5.0 AS build
WORKDIR /src
COPY ["SpecialOffers/SpecialOffers.csproj", "SpecialOffers/"]
RUN dotnet restore "SpecialOffers/SpecialOffers.csproj"
COPY . .
WORKDIR "/src/SpecialOffers"
RUN dotnet build "SpecialOffers.csproj" -c Release -o /app/build
FROM build AS publish
RUN dotnet publish "SpecialOffers.csproj" -c Release -o /app/publish
FROM base AS final
WORKDIR /app
COPY --from=publish /app/publish .
ENTRYPOINT ["dotnet", "SpecialOffers.dll"]

@Richbward
These parts of that output

Microsoft (R) Build Engine version 16.7.2+b60ddb6f4 for .NET
...
/usr/share/dotnet/sdk/3.1.409/Microsoft.Common.CurrentVersion.targets

indicates that you're using .NET Core SDK 3.1.409, rather than .NET SDK 5.0.xxx, which is required to target .NET 5.0.
I tried using your Dockerfile and it seems to work fine to me--the dotnet/sdk:5.0 container has 5.0.300 installed.
One difference I find interesting is that the step number is 7/7 instead of 6/6 as you posted:
 => [build 6/7] WORKDIR /src/SpecialOffers                                                                         0.1s
 => [build 7/7] RUN dotnet build "SpecialOffers.csproj" -c Release -o /app/build                                   2.1s
 => [publish 1/1] RUN dotnet publish "SpecialOffers.csproj" -c Release -o /app/publish                             1.5s

Is that your full, exact Dockerfile?

I have the same issue everytime i run dotnet build even though my sdk is set to the right location.
`Ademoyes-MacBook-Air:MvcMovie ademoyeoluwanifesimi$ dotnet build
Microsoft (R) Build Engine version 16.7.2+b60ddb6f4 for .NET
Copyright (C) Microsoft Corporation. All rights reserved.
Determining projects to restore...
Restored /Users/ademoyeoluwanifesimi/Projects/MvcMovie/MvcMovie/MvcMovie.csproj (in 1.1 sec).
/usr/local/share/dotnet/x64/sdk/3.1.416/Microsoft.Common.CurrentVersion.targets(1177,5): error MSB3971: The reference assemblies for ".NETFramework,Version=v5.0" were not found. You might be using an older .NET SDK to target .NET 5.0 or higher. Update Visual Studio and/or your .NET SDK. [/Users/ademoyeoluwanifesimi/Projects/MvcMovie/MvcMovie/MvcMovie.csproj]
Build FAILED.
/usr/local/share/dotnet/x64/sdk/3.1.416/Microsoft.Common.CurrentVersion.targets(1177,5): error MSB3971: The reference assemblies for ".NETFramework,Version=v5.0" were not found. You might be using an older .NET SDK to target .NET 5.0 or higher. Update Visual Studio and/or your .NET SDK. [/Users/ademoyeoluwanifesimi/Projects/MvcMovie/MvcMovie/MvcMovie.csproj]
0 Warning(s)
1 Error(s)
`

@nifesimii Looks like you are running .NET Core SDK 3.1.416, which did not understand how to target .NET 5.0+, and the error message is accurate.

even though my sdk is set to the right location

How are you setting this?

This problem creates after installing .net 6 for arm architecture and already installed .net 5 for x64. And PATH has only one path used for our SDK.

In case someone has similar issue on the Visual Studio for Mac. After updating to MacOS Ventura all projects started throwing same errors. In my situation the next steps helped:

Open the Visual Studio
Go to Preferences > SDK Locations
For the Location field use /usr/local/share/dotnet/x64/dotnet instead of the /usr/local/share/dotnet/dotnet path.

This one applies for dotnet 6.0 and 7.0, not sure it will help with the 5.0 version.

I've got this same problem - it cannot find reference assemblies for .NET 6, despite the SDKs being found in visual studio.
I'm at a loss here.

@brennhill Please open a new issue with more information, including logs that show the version number of the SDK as above.


Custom sort

Would it be difficult to do so that I could sort the images as I want and not by e.g. date, size etc.

add custom sort  to albums as well.

Related #97

I hope we get this feature in the near future :). I would be an awesome extension for this app.

The ability to pin an item at the top is great, but with many items pinned one might still need to fall back to using prefixes. New items could be placed at top to make them easy to spot, much like new albums are shown today. Also, while showing the custom order, it would be clever to ignore pin status but still be able toggle items as pinned for scenarios where a user switches back and forth between the custom and non-custom sorting order.


What caffe do you use ?

Hello, what caffe does apollo model use ?
Thanks.

caffe？

yes, this model uses caffe, right ?

The model has been converted to a format supported by tensorrt. Since caffe has been merged into pytorch, caffe will not be supported separately in the future.