Raspberry Pi requires multi-planar buffer type

Raspberry Pi only supports the multi-planar API.
Could we get another method (or modify the current v4l::output::device::Device.enum_formats method) that allows us to specify the buffer type?
We need to allow
https://github.com/raymanfx/libv4l-rs/blob/2ab400dc4cfca4aafc7b278529eece85381d1992/src/output/device.rs#L97
to be
 v4l2_fmt.type_ = v4l2_buf_type_V4L2_BUF_TYPE_VIDEO_OUTPUT_MPLANE;

Reference v4l::buffer::BufferType::VideoOutputMplane.

I should also point out that this may only be true on Raspberry Pi 4. That is what I'm currently testing on.

Hi, I'm very interested in getting multi-planar format support going. Unfortunately I don't have a camera device which supports/needs it. What device are you testing this with? I'm pretty sure standard USB webcams will work using the single-planar (aka packed) v4l2 layer even on the RPi 4, no?
From your first comment I read that you need multi-planar support for the output device, is that correct? Maybe we can tackle it for input and output at the same time even.
Apart from the format change, I can envision some more edits to be required, e.g. a new Buffer struct to represent planar buffers. Perhaps using a data structure like Vec<&[u8]>.

I'm using /dev/video10 on a raspberry pi 4 (kernel: 5.4) in an attempt to decode h264. It seems this device only supports multi-planar buffers. With the v4l2_buf_type_V4L2_BUF_TYPE_VIDEO_OUTPUT buffer type you get Invalid Argument errors.
Using v4l2-ctl I still haven't been able to get a h264 stream decoded, keep running into DECODER STOP messages.
However it seems I've taken a few steps back. I tried to install aarch64 arch linux (kernel: 4.8.9) and now I don't get any v4l2 devices at all.
Anyway,

From your first comment I read that you need multi-planar support for the output device, is that correct? Maybe we can tackle it for input and output at the same time even.

Yes. If I can help I'll try.

Apart from the format change, I can envision some more edits to be required, e.g. a new Buffer struct to represent planar buffers. Perhaps using a data structure like Vec<&[u8]>.

I'm very new to this space and I'm not really even sure what multi-planar is. If I understand correctly it's buffers for a device that needs a discontiguous buffer? (going off https://www.kernel.org/doc/html/latest/userspace-api/media/v4l/planar-apis.html#planar-apis)

I'm using /dev/video10 on a raspberry pi 4 (kernel: 5.4) in an attempt to decode h264. It seems this device only supports multi-planar buffers. With the v4l2_buf_type_V4L2_BUF_TYPE_VIDEO_OUTPUT buffer type you get Invalid Argument errors.

Oh I see. So you're not working with actual video I/O devices, but trying to use the hardware memory-to-memory codec on the RPi (which happens to be implemented as a v4l2 module).

I'm very new to this space and I'm not really even sure what multi-planar is. If I understand correctly it's buffers for a device that needs a discontiguous buffer? (going off https://www.kernel.org/doc/html/latest/userspace-api/media/v4l/planar-apis.html#planar-apis)

Yes, that's about right. Decoding a video frame will usually give you "raw" frames, probably some YUV format instead of the more familiar RGB formats. YUV data captures luminance and chrominance. While YUV pixels can be packed, video codecs usually prefer them to be laid out in memory planes, e.g. one plane for the luminance (Y) and one for the chrominance components (UV).

FYI I'm planning to tackle this tomorrow. Are you available for testing this weekend?

Yeah, I should be free sometime this weekend to test. Sorry, meant to reply sooner. Went out to buy a C7000, gonna try setting up virtual desktops, encode their screens, and decode on Pi's.
I got the PI back to having the v4l2 devices online.
Would also note that the PI seems to fail v4l2-compliance tests.
v4l2-compliance SHA: not available
, 32 bits, 32-bit time_t

Compliance test for bcm2835-codec device /dev/video10:

Driver Info:
	Driver name      : bcm2835-codec
	Card type        : bcm2835-codec-decode
	Bus info         : platform:bcm2835-codec
	Driver version   : 5.4.51
	Capabilities     : 0x84204000
		Video Memory-to-Memory Multiplanar
		Streaming
		Extended Pix Format
		Device Capabilities
	Device Caps      : 0x04204000
		Video Memory-to-Memory Multiplanar
		Streaming
		Extended Pix Format
Media Driver Info:
	Driver name      : bcm2835-codec
	Model            : bcm2835-codec
	Serial           : 0000
	Bus info         : platform:bcm2835-codec
	Media version    : 5.4.51
	Hardware revision: 0x00000001 (1)
	Driver version   : 5.4.51
Interface Info:
	ID               : 0x0300000c
	Type             : V4L Video
Entity Info:
	ID               : 0x00000001 (1)
	Name             : bcm2835-codec-decode-source
	Function         : V4L2 I/O
	Pad 0x01000002   : 0: Source
	  Link 0x02000008: to remote pad 0x1000004 of entity 'bcm2835-codec-decode-proc': Data, Enabled, Immutable

Required ioctls:
	test MC information (see 'Media Driver Info' above): OK
		fail: v4l2-compliance.cpp(694): found_bit
	test VIDIOC_QUERYCAP: FAIL

Allow for multiple opens:
	test second /dev/video10 open: OK
		fail: v4l2-compliance.cpp(694): found_bit
	test VIDIOC_QUERYCAP: FAIL
	test VIDIOC_G/S_PRIORITY: OK
	test for unlimited opens: OK

	test invalid ioctls: OK
Debug ioctls:
	test VIDIOC_DBG_G/S_REGISTER: OK (Not Supported)
	test VIDIOC_LOG_STATUS: OK (Not Supported)

Input ioctls:
	test VIDIOC_G/S_TUNER/ENUM_FREQ_BANDS: OK (Not Supported)
	test VIDIOC_G/S_FREQUENCY: OK (Not Supported)
	test VIDIOC_S_HW_FREQ_SEEK: OK (Not Supported)
	test VIDIOC_ENUMAUDIO: OK (Not Supported)
	test VIDIOC_G/S/ENUMINPUT: OK (Not Supported)
	test VIDIOC_G/S_AUDIO: OK (Not Supported)
	Inputs: 0 Audio Inputs: 0 Tuners: 0

Output ioctls:
	test VIDIOC_G/S_MODULATOR: OK (Not Supported)
	test VIDIOC_G/S_FREQUENCY: OK (Not Supported)
	test VIDIOC_ENUMAUDOUT: OK (Not Supported)
	test VIDIOC_G/S/ENUMOUTPUT: OK (Not Supported)
	test VIDIOC_G/S_AUDOUT: OK (Not Supported)
	Outputs: 0 Audio Outputs: 0 Modulators: 0

Input/Output configuration ioctls:
	test VIDIOC_ENUM/G/S/QUERY_STD: OK (Not Supported)
	test VIDIOC_ENUM/G/S/QUERY_DV_TIMINGS: OK (Not Supported)
	test VIDIOC_DV_TIMINGS_CAP: OK (Not Supported)
	test VIDIOC_G/S_EDID: OK (Not Supported)

Control ioctls:
	test VIDIOC_QUERY_EXT_CTRL/QUERYMENU: OK
	test VIDIOC_QUERYCTRL: OK
	test VIDIOC_G/S_CTRL: OK
	test VIDIOC_G/S/TRY_EXT_CTRLS: OK
	test VIDIOC_(UN)SUBSCRIBE_EVENT/DQEVENT: OK
	test VIDIOC_G/S_JPEGCOMP: OK (Not Supported)
	Standard Controls: 2 Private Controls: 0

Format ioctls:
	test VIDIOC_ENUM_FMT/FRAMESIZES/FRAMEINTERVALS: OK
	test VIDIOC_G/S_PARM: OK (Not Supported)
	test VIDIOC_G_FBUF: OK (Not Supported)
	test VIDIOC_G_FMT: OK
	test VIDIOC_TRY_FMT: OK
	test VIDIOC_S_FMT: OK
	test VIDIOC_G_SLICED_VBI_CAP: OK (Not Supported)
	test Cropping: OK (Not Supported)
	test Composing: OK
	test Scaling: OK (Not Supported)

Codec ioctls:
	test VIDIOC_(TRY_)ENCODER_CMD: OK (Not Supported)
	test VIDIOC_G_ENC_INDEX: OK (Not Supported)
		fail: v4l2-test-codecs.cpp(123): ret != 0
	test VIDIOC_(TRY_)DECODER_CMD: FAIL

Buffer ioctls:
	test VIDIOC_REQBUFS/CREATE_BUFS/QUERYBUF: OK
	test VIDIOC_EXPBUF: OK
	test Requests: OK (Not Supported)

Test input 0:

Streaming ioctls:
	test read/write: OK (Not Supported)
	test blocking wait: OK
		fail: v4l2-test-buffers.cpp(1353): ret == 0
	test MMAP (select): FAIL
		fail: v4l2-test-buffers.cpp(1353): ret == 0
	test MMAP (epoll): FAIL
	test USERPTR (select): OK (Not Supported)
	test DMABUF: Cannot test, specify --expbuf-device

Total for bcm2835-codec device /dev/video10: 51, Succeeded: 46, Failed: 5, Warnings: 0

Anyway, shoot me any tests you want me to run.

It looks like getting your usecase working is going to be a lot more involved than I initially thought. I've started the work in the next branch. Basically, there's two (currently unsupported) features that are required:

Streaming output (MMAP in particular) - right now, only plain write() is supported for output devices.
Multi-planar buffer support

Those 2 sound about right. I found some more interesting information at https://www.raspberrypi.org/forums/viewtopic.php?f=68&t=281296&p=1737303&hilit=v4l2+%2Fdev%2Fvideo10#p1737303 that might be useful.
I'm a beginner in rust but if there's anything I can do to help I'd love to.

Some good news - I was able to implement MMAP output streaming today (tested using v4l2loopback). My WIP stuff is on the next branch if you want to check it out.

I just pushed some preliminary multi-planar buffer support to the next branch. Could you please change this: https://github.com/raymanfx/libv4l-rs/blob/next/src/output/device.rs#L157 to return VideoOutputMplane and see if that makes it work for you? I'm still thinking about how to best incorporate multi-plane support into the API.

Cool. I'll give it a test in the next couple hours.

Looks golden.
Source:
use std::io::Read;

use v4l::prelude::DeviceExt;
use v4l::output::device::Device as OutputDevice;

// use nix::sys::ioctl;

const DECODE_DEVICE: &str = "/dev/video10";

fn main() -> std::io::Result<()> {
    let mut file = std::fs::File::open("/home/alarm/short.h264")?;
    let mut contents = vec![];
    file.read_to_end(&mut contents)?;

    let dev = OutputDevice::with_path(DECODE_DEVICE)
        .expect(&format!("Unable to open decode device {}", DECODE_DEVICE));

    let capabilities = dev.query_caps().expect("could not get capabilities");

    dbg!(capabilities);

    let formats = dev.enum_formats()?;
    println!("Number of formats: {}", formats.len());
    for fmt in formats {
        print!("{}", fmt);
    }

    Ok(())
}

Output:
[alarm@alarmpi framebuffer_decode]$ ./target/debug/framebuffer_decode
[src/main.rs:21] capabilities = Capabilities {
    driver: "bcm2835-codec",
    card: "bcm2835-codec-decode",
    bus: "platform:bcm2835-codec",
    version: (
        5,
        4,
        75,
    ),
    capabilities: VIDEO_M2M_MPLANE | EXT_PIX_FORMAT | STREAMING,
}
Number of formats: 2
index       : 0
type:       : 10
flags:      : COMPRESSED
description : H.264
fourcc      : H264
index       : 1
type:       : 10
flags:      : COMPRESSED
description : Motion-JPEG
fourcc      : MJPG

Also, I hit up the raspberry pi forums about the compliance issue: https://www.raspberrypi.org/forums/viewtopic.php?f=67&t=291227&p=1760878#p1760878
Some interesting input on v4l2-ctl/v4l2-compliance in there.

Sounds good. Now we need to check whether the actual streaming I/O works with the new code. Did you try writing some H.264 or JPEG frames to that device (by creating a MmapStream and using the io::stream::Output trait)?

Not sure if I'm doing this correctly, but output it Invalid Argument.
Source:
use std::io::Read;

use v4l::{Timestamp, buffer::{Buffer, Flags, Metadata}};
use v4l::io::stream::Output;
use v4l::output::device::Device as OutputDevice;
use v4l::prelude::{DeviceExt, MmapStream};

// use nix::sys::ioctl;

const DECODE_DEVICE: &str = "/dev/video10";
const VIDEO_PATH: &str = "/home/alarm/short.h264";

fn main() -> std::io::Result<()> {
    let mut dev = OutputDevice::with_path(DECODE_DEVICE)
        .expect(&format!("Unable to open decode device {}", DECODE_DEVICE));

    let capabilities = dev.query_caps().expect("could not get capabilities");

    dbg!(capabilities);

    let formats = dev.enum_formats()?;
    println!("Number of formats: {}", formats.len());
    for fmt in formats {
        print!("{}", fmt);
    }

    let mut stream = MmapStream::with_buffers(&mut dev, 1).expect("Failed to create buffer stream");

    let mut file = std::fs::File::open(VIDEO_PATH)?;
    let mut contents = vec![];
    file.read_to_end(&mut contents)?;

    let metadata = Metadata {
        bytesused: contents.len() as u32,
        flags: Flags::empty(),
        field: 0,
        timestamp: Timestamp::new(0, 0),
        sequence: 0
    };

    let buffer = Buffer {
        planes: vec![contents.as_ref()],
        meta: metadata
    };

    Output::next(&mut stream, buffer)?;

    Ok(())
}

Output:
[alarm@alarmpi ~]$ ./framebuffer_decode/target/debug/framebuffer_decode
[src/main.rs:19] capabilities = Capabilities {
    driver: "bcm2835-codec",
    card: "bcm2835-codec-decode",
    bus: "platform:bcm2835-codec",
    version: (
        5,
        4,
        75,
    ),
    capabilities: VIDEO_M2M_MPLANE | EXT_PIX_FORMAT | STREAMING,
}
Number of formats: 2
index       : 0
type:       : 10
flags:      : COMPRESSED
description : H.264
fourcc      : H264
index       : 1
type:       : 10
flags:      : COMPRESSED
description : Motion-JPEG
fourcc      : MJPG
thread 'main' panicked at 'Failed to create buffer stream: Os { code: 22, kind: InvalidInput, message: "Invalid argument" }', src/main.rs:27:60
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

Okay, so this will require some more debugging. Can you add some logs to this function: https://github.com/raymanfx/libv4l-rs/blob/next/src/io/mmap/arena.rs#L54 and see where exactly it fails?
I just looked at some Raspberry Pi 4 bundles today, but the 4GB models are still rather expensive, I don't think I can justify getting one just for this project. Anything less than 4GB is not worth paying for though.
Apart from that, I'm still wondering about how the H.264 decoder works here - you're basically just passing the entire file buffer in one go. From what I read in the link you provided (https://www.raspberrypi.org/forums/viewtopic.php?f=67&t=291227&p=1760878#p1760878), I think you're supposed to pass framed data - so you'd need to split the H.264 bytestream into frames before passing them to the decoder? Then again, they also they it should work with unframed data as well, disregarding a latency increase.

I'll add some logs to that function and see where it's failing.
Yeah, from what the forum post explains, the decoder should work on unframed data (with performance degradation). But honestly I don't really know if passing in the file buffer is the right way to do it either.
Can I just give you ssh access to my PI? I leave it on 24/7 anyway. If that works just email me at ayrton AT sparling DOT us.

Oh .. I see it now. The H.264 decoder device on the Raspberry Pi 4 requires using the VIDEO_M2M_MPLANE API, but this crate right now only implements capture and output (not memory-to-memory codecs). That's going to involve more work. I think I want to restructure some things first before implementing M2M streaming.

Hey @raymanfx, what is the status for it ?

Hi @patrickelectric, I prepared the multi-planar API support a while ago: https://github.com/raymanfx/libv4l-rs/tree/mplane but was not happy with the API at the time. Since it is not required for my projects, I did not finish the work. If you are interested in picking it up or collaborating, let me know.

Hi @raymanfx , it looks that the mplane- branch has already been compatible with a VIDEO_CAPTURE_MPLANE device, just by looking into the data structures defined and capture procedures. But you said you did not finish the work. Would you mind pointing out the unfinished part or giving any guidelines on its accomplishment?

Hi @patrickelectric, I prepared the multi-planar API support a while ago: https://github.com/raymanfx/libv4l-rs/tree/mplane but was not happy with the API at the time. Since it is not required for my projects, I did not finish the work. If you are interested in picking it up or collaborating, let me know.


Addition of Boyer-Moore Algorithm in String Algorithms

I would like to add a new algorithm i.e., Boyer-Moore Algorithm (String Algorithms) that skips sections of the text, resulting in faster search times than naive approaches.

@pratheekv39 You can start


e16: 1.0.23 -> 1.0.24

Motivation for this change
Update to version 1.0.24
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 146625 run on x86_64-linux 1

1 package built:

e16


fix release date format on missing games

displayed without dashes which makes it harder to read

can you explain how to download new build ?
why dont you provide new release ?

@larg000 its under the Action tab here on GitHub, then select the latest build and the files are there
reason for this is that i havent had much time at the computer this summer due to having the kids home for a long period of times and the plan was to fix some more bugs and maybe add a feature before a new release

fixed with release 1.7.0


Support from-to token direct select by URI query

app.terraswap.io?from=tokenContract1orNativeTokenDenom&to=tokenContract2orNativeTokenDenom

To keep the existing input value when push to Trying again button.

Also you'll be able to share the link with predefined tokens which is a very nice feature to have 👀


Do not ban peers after block validation fails for intermittent or internal errors

Right now, we ban peers on every failed block validation ("sent invalid consensus value"). Since #4428, it is possible to be more nuanced in our reaction to these events, as validation failures can also come from internal and temporary failures.
The blocking logic should be updated to not ban peers if one of these failures happens.

Changes mentioned in the above comments were merged into feat-1.6 and are not present in feat-2.0 branch.

Potential 2.1.0 as this can be implemented as a non-breaking change.


Fix a typo

Typo: with -> within.
I guess within would be better.

@niebayes : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Not possible to exit settings when Sidebar is hidden

Search before asking

[X] I had searched in the issues and found no similar issues.

Operating System

[ ] macOS
[X] Windows
[ ] Linux

Operating System Version
Windows 10, Windows 11
It happens on the web browser too?
No, it just happens on the Desktop app
Rocket.Chat Desktop App Version
3.9.14
Rocket.Chat Server Version
6.6.6
Describe the bug
If the Sidebar is hidden there's no visible way to exit the Settings menu.
How to Reproduce

Launch the 'Rocket.Chat Desktop Client'
Go to 'Top Menu' -> 'Window' -> 'Settings'
Turn off 'Sidebar'
-> No visible or logical way to exit Settings

Describe your Expected behavior
A X-button or back-arrow to leave the Settings-menu
Anything else
A way I've found to exit is to:

Go to 'Top Menu' -> 'Window'
Choose the server you'd like to go to
However it isn't really useful for users as they won't find it. It makes it even harder to find if the Menu bar is hidden aswell.

Are you willing to submit a code contribution?

[ ] Yes, I am willing to submit a Pull Request!

Can you help me

Ctrl+1 ?


remove duplicate records from dataset

Sometimes datasets contain duplicate records. We would love to see them removed automatically.
MVP
from argilla_plugins.datasets import delete_duplicates

delete_duplicates(name="dataset")
delete_duplicates.start()

Stretch
filtering variables like query could be added to limit the sync.

This would be a very useful feature.
In particular, when I want to add additional records to an existing dataset I would like any duplicate entries in the rg.log() to be ignored. I currently have a unique identifier stored in record["metadata"] which would make the de-deduplication very straightforward. Actually, would it make sense to require a unique identifier (or at least make one optional) on the text record level?

Hi @zachschillaci27, internally we actually use a unique identifier, which is automatically generated for each record during database insertion. However, you can also set this manually when creating records by setting the 'id' parameter. This approach is also about removing duplicate string that might not have a duplicate id.

Hi @zachschillaci27, internally we actually use a unique identifier, which is automatically generated for each record during database insertion. However, you can also set this manually when creating records by setting the id parameter. This approach is also about removing duplicate strings that might not have a duplicate id.

Got it, that's very useful to know! Thanks for the quick feedback.
It could also be useful to delete duplicate strings, though in cases where these are "true" duplicates (e.g. identical news articles published by different sources) it perhaps makes sense to instead propagate the annotations (if any) and allow the user to deduplicate themselves "offline".


Enabling Auditing - More clarity needed

Q1 : There is reference to "Server Blob Auditing" throughout this page. Does this refer to enabling auditing where the Audit Log Destination = Azure Storage" ?  (The UI shows no reference to "Server Blob Auditing")
Q2 : This docpage says : "If server blob auditing is enabled, it always applies to the database. The database will be audited, regardless of the database auditing settings."  Does this ALSO apply to the scenario where I am NOT sending audit log to storage, but instead I am sending Audit logs to "Log Analytics Workspace"?  (i.e. if I am only auditing to Log Analytics at server level, does this setting also apply to every DB on the server?)
Q3 : When sending audit logs to Log Analytics, can you tell us the name of the Log Analytics table where this information is sent to?
Q4 : The section titled "Analyze Audit Logs and Reports" takes you step by step to opening in Log Analytics a pre-built query.  This query fails with the following syntax error:
'project' operator: Failed to resolve scalar expression named 'event_time_t'
Support id: eeff5f01-af3c-477d-9d95-0c390587d2ec
For reference, the pre-built query at fault is:
search *
| where Category == 'SQLSecurityAuditEvents'
| where ResourceId == '/SUBSCRIPTIONS/82931E73-05C6-4DA8-A666-BC4A7DD1BD3E/RESOURCEGROUPS/FABRIKAMLTD-PAAS-RG/PROVIDERS/MICROSOFT.SQL/SERVERS/FABRIKAMLTDTODODBSERVER/DATABASES/MASTER'
| project event_time_t ,statement_s , succeeded_s,affected_rows_d,server_principal_name_s
,client_ip_s,application_name_s,additional_information_s
| order by event_time_t desc
| take 100
Thanks... JD

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

ID: 0c27fe13-a824-84a5-e314-892a27c1dec5
Version Independent ID: 510b5427-c927-84df-563f-31dfe13fc771
Content: Get started with Azure SQL database auditing
Content Source: articles/sql-database/sql-database-auditing.md
Service: sql-database
Sub-service: security
GitHub Login: @vainolo
Microsoft Alias: arib

@julesman101 Thank you very much for taking the time to bring this issue to our attention. We are investigating and will update this issue when we have accurate information.

@julesman101 In looking into this, the tutorial walks you through defining Server-level versus Database-level audit policies and configuring the destination for the related audit data to either an Azure Storage account, an Azure Log Analytics workspace, or an Azure Event Hub instance.
In the case of the specific audit policy implemented, the choice is to set a server-level or a database-level audit based upon the desired data. You need to be selective here as you could end up with a ton of data that isn't useful. As for the desired output destination, you could select one or all three currently supported audit log destinations.
I am currently looking into the Log Analytics issue.

@julesman101 YOu need to generate some audit activity. If you set a server-level or database-level default policy and access the SQL instance via SSMS or the Query editor, you will populate the tablespace and your query will not receive a syntax error:
Just after creating a server-level audit policy and attempting to access the Log Analytics log data:

After accessing the Azure SQL Database instance via SSMS and the Query editor via the Azure Portal:

And after waiting for audit event data, you can see the query return log activity:


Fix make bundle command

Summary
Remove the client.js (version 3) from the webpack.config.js.
Checklist

[x] Ran make check-style to check for style errors (required for all pull requests)
[x] Ran make test to ensure unit tests passed
[x] Ran make flow to ensure type checking passed

Test Information
This PR was tested on: [Arch Linux]

Just one thing should this line in readme be updated?
`This will generate lib/mattermost.client.js, lib/mattermost.client4.js, and lib/mattermost.websocket.js which can be loaded in a browser. Also note that babel-polyfill is required``

@sudheerDev very good point

Fixed, and after the test passes I'll merge it :)


Cleanup comments and implement ask function in more places

This PR does the following:

Modifies comment formatting in this repo’s scripts to be less obtrusive.
Implements the ask function in scripts that ask questions that hadn’t yet implemented the new function.
Makes slight adjustments to formatting of code to resolve shellcheck, EditorConfig, and shfmt linter violations.
Changes the default answer from Y to N when asked if a conflict-practice repo should be recreated for a user who already has a conflict-practice repo. This will make it easier to re-run the script without accidentally performing a potentially destructive operation.

I have tested these script locally and verified that the most frequently used scripts are working as intended.

@rwnfoo, check if you have the temp directory in place from a previous run.  Wondering if you ran into the .com version of #244.  Also, I'm going to be doing some test runs today against GHES, so I can let you know if I run into a similar issue as well.

Thank you @amyschoen for the insight! For future reference, I ran git clean -f -d to remove all untracked files and directories.
I tested out the script against a repo, and here are the results:

Seems to be working fine ✨

You just taught me something new and very useful @rwnfoo.

I noticed in testing the PDF script that we have an extra dependency introduced.

I'll file an issue and then do a quick update to the GitBook directions since this falls outside the scope of this PR.

Thanks everyone! 😊


Update iOS Object Detection project setting

Hi,
I update Object Detection project.

Add Pofile.lock and Pod setting to pbxproj
Upgrade Swift version 4.2 to 5

Thanks.

Thanks @daisuke-t-jp! Please run pod deintegrate to remove the extra pod references. Otherwise, this change LGTM.

@morganchen12
Thanks your review.
I did it.


repeated builds in emacs

Using ghcide 0.2 from command line or emacs, I get a weird behavior where things start rebuilding the same few dependencies for, I think, every file in the project?
I get a "Consulting the cradle for ..." and then "resolving dependencies" and then it builds them and then that happens again for lots of files, maybe all of them.
I did have a (one-line, for cabal) hie.yaml.  Same behavior after deleting that.

Impossible to make any progress without any reproduction or log output.

@adamConnerSax i think some improvements for emacs and file handling was added (#831), could you try last ghcide version to checkit that is reproduced (or even better switch to haskell-langiage-server)

Swicthed to haskell-language-server and this behavior did indeed cease.  Still crashes a bunch from emacs but I can't pin it down to anything small and reproducible...


Wrong parsing method when first single character in sentence

Related to #16
The parsing method which escapes characters seems to not fully working. For instance, when the sentence starts with a single character that has to be escaped, actually it is not.
Example:
a las 07:30 gives am las 07:30
It seems to come from the regex here which is not matching a first single character and gives - for example - this:

Fixed in #28


Loaded texture is not being set in shader

phaser: "3.55.2"
import Phaser from "phaser";

class Test extends Phaser.Scene {
    constructor () {
        super('Test');
    }

    preload() {
       this.load.glsl('test', 'public/assets/shaders/test.glsl');
       this.load.image("reflection", "public/assets/shaders/reflection.jpg");
    }

    create() {
        const center = this.centerScreen();

        this.test = this.add.shader("test", center.x, center.y , window.innerWidth, window.innerHeight);
        this.test.setChannel0("reflection");
    }

    centerScreen(){
        return {x: window.innerWidth * 0.5, y: window.innerHeight * 0.5 }
    }
}

export default Test;

Shader (test.glsl):
uniform sampler2D iChannel0;

void main() {
    gl_FragColor.rgba = vec4(texture2D(iChannel0, gl_FragCoord.xy).xyz, 1.0);

}

I can't debug what's going on here, this should work right?

In fact, modifying this sample to use a different texture, does not work!
https://labs.phaser.io/edit.html?src=src/game objects/shaders/change shader texture.js&v=3.55.2
What's going on here? Is it Chrome caching or something?

Tried it on firefox too, doesn't function at all. As far as I can tell I've followed the instructions precisely


Download post images

People are taking screenshots to share images with other people, which its annoying to have to do.
Let's investigate if we can have the native save image functionality.
This will be an interesting one with copyright. There is a feature request to be able to mark posts under different licenses, meaning we could make it something to be enabled/disabled per post license.

Slightly related, but is it possible to implement an animation with the image? Like pop the image out and allow the image to be moved around, and tucked back into the feed?
I found this in the Flutter cookbook https://flutter.dev/docs/cookbook/navigation/hero-animations


update

Docs Build status updates of commit b01a49f:
:warning: Validation status: warnings

File
Status
Preview URL
Details

msteams-platform/messaging-extensions/how-to/create-messaging-extension.md
:warning:Warning
View
Details

msteams-platform/bots/how-to/create-a-bot-for-teams.md
:white_check_mark:Succeeded
View

msteams-platform/messaging-extensions/how-to/create-messaging-extension.md

Line 127, Column 177: [Warning-file-not-found] Invalid file link: '../../tabs/how-to/authentication/auth-aad-sso'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit c9d52d9:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/bots/how-to/create-a-bot-for-teams.md
:white_check_mark:Succeeded
View

msteams-platform/messaging-extensions/how-to/create-messaging-extension.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit b6ab881:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/bots/how-to/create-a-bot-for-teams.md
:white_check_mark:Succeeded
View

msteams-platform/messaging-extensions/how-to/create-messaging-extension.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


All files are necessary, including tests. npm explore tape 'npm install && npm test' should work.

example/ and .travis.yml are not necessary.
what if you will have 100 MB fixtures in test/? it shouldn't be included in published package.

I agree that fixtures perhaps don't need to be included - but few packages have fixtures at all, let alone large ones.
I agree that "example" and the travis config aren't necessary - but to exclude them, we'd have to add .npmignore, which means all the .gitignore entries have to be duplicated - that's nonzero extra complexity, just to save a few hundred K on disk.

there is no need to add .npmignore, see details of my commit.

also there are much more fixtures than people who ever knows about npm explore and especially npm explore package 'npm install && npm test' – why do I want to do this? we have dependencies and devDependencies here for a reason, so I'll clone the repo, install deps and run tests if I want to contribute.

I agree with @deepsweet. Why would someone need to run npm test for the package in their node_modules folder? What is the use case?
Loading unnecessary files is one of the reasons npm installs take so darn long. Anton Rudeshko very well put it in his article:

Speaking in terms of object-oriented design, your npm package should be highly cohesive and loosely coupled. Your package should do only one thing and do it well.

Corey Butler also wrote a nice article about why it's important to minimize module footprints, including test suite:

In the rare situation a developer actually wants to run your test suite on their own local computer, they’ll likely clone or fork it.

$ du -sh node_modules/less/test/

2.9M	node_modules/less/test/

@deepsweet if we wanted everything included by default, and only certain files excluded, we would need .npmignore. A whitelist like you have here is insanely dangerous - it's very easy to add a new file to your package but forget to add it to the whitelist.
@mistadikay My use case is, when I want to run tests for a module i've installed. I do this occasionally, and as part of automated tests for some of my modules.
Package size is npm's problem to solve - package authors should not be trying to solve it.

ok, I'm going to say goodbye to this TAP implementation because it looks like you are missing the whole point of NPM registry as a package delivery system.

The package includes its tests. That is what I want delivered.

a whitelist like you have here is insanely dangerous

I beg to differ. Blacklists might be more dangerous since you can forget to exclude something and introduce a potential security threat if there is a sensitive information. If you forgot to include something into whitelist, at worst you'll have to release a patch version.

My use case is, when I want to run tests for a module i've installed

I still don't quite understand why would you test someone else's packages. Isn't it what package maintainers should do? What if they didn't write any tests?

Package size is npm's problem to solve - package authors should not be trying to solve it.

How npm can solve this problem? By making some precarious assumptions on what should be loaded and what should not?

Let's stay kind to the maintainers here - they see value in tests/ and examples/ being sent, saying that they miss the point of npm... is not particularly nice.

It's not nice to chase my PRs even in not related to this maintainer
projects. There are still no strong arguments to include tests.
On Wednesday, 17 February 2016, Yoshua Wuyts<EMAIL_ADDRESS>wrote:

Let's stay kind to the maintainers here - they see value in tests/ and
examples/ being sent, saying that they miss the point of npm... is not
particularly nice.
—
Reply to this email directly or view it on GitHub
https://github.com/substack/tape/pull/260#issuecomment-185033087.

The strong argument is "I want to be able to run tests in my dependencies when possible". You're creating a huge amount of FUD and future functionality risk by submitting tons of PRs all over the place that use a whitelist approach - files are only downloaded once per version, and are cached. This just isn't a big problem - install time isn't where speed matters.

that's why I said that you are missing something very important. npm
install time IS  the big problem since npm@3 and especially for the last
few months, that's why projects like https://github.com/rstacruz/pnpm are
here now, and so on.
I already answered you why blacklist can be more dangerous than whitelist.
sorry but "I want to be able to run tests in my dependencies" is absolutely
uncommon bad pattern.
your point to include tests (even with 100 MB fixtures, because w/o it
tests will be broken) to npm package is totally wrong for me.
On Wednesday, 17 February 2016, Jordan Harband<EMAIL_ADDRESS>wrote:

The strong argument is "I want to be able to run tests in my dependencies
when possible". You're creating a huge amount of FUD and future
functionality risk by submitting tons of PRs all over the place that use a
whitelist approach - files are only downloaded once per version, and are
cached. This just isn't a big problem - install time isn't where speed
matters.
—
Reply to this email directly or view it on GitHub
https://github.com/substack/tape/pull/260#issuecomment-185039419.

And look, projects like pnpm are solving it and package authors don't have to.
It is better to release a working package with sensitive data than a broken package.

@deepsweet don't throw out a strawman, test + examples in this package results in 220kb with no gzip - that's two orders of magnitude off from your statement of 100mb.
There's probably more important things to worry about than <200kb in overhead for a package that is already cached. Visiting pretty much ANY website on the internet will download more assets than that. I agree with @ljharb, file size optimization should be solved by package managers - not by package authors.

$ curl -s https://registry.npmjs.org/tape/-/tape-`npm view tape version`.tgz | wc -c
19957

20 kilobytes.


lastMatch should depend on isExact as well

In situation when url matched for nested routes, url is always the same but isExact is not.

I'm merging it and will add a comment later.


Publish 02/07/2022 3:30 PM PT

Docs Build status updates of commit 9d09ec9:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-online/Set-SPOTenant.md
:white_check_mark:Succeeded
View (sharepoint-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Missing Pro 8 support

From this discussion, data for the Pro 8 is missing, effectively rendering my Slim Pen 2 useless. Touch screen gestures are also a problem although I am unsure of correct installation, missing packages, or that I am simply dumb.
"libwacom-list-local-devices" output:

/dev/input/event15 is a tablet but not supported by libwacom
/dev/input/event11 is a tablet but not supported by libwacom
Failed to find any devices known to libwacom.

Knowing how to contribute would be of great help.

Can you post the list of devices that shows up when you run sudo evtest?
We essentially need the device id and the rest can probably be copied from
the sp7 or spx.
You could also have a look at some other issues where I think I posted some
instructions on how to compile and test (might be on
linux-surface/libwacom). I only have my phone with me right now, but I can
probably post some more detailed instructions later when I'm back home.
On Sat, Jun 10, 2023, 18:53 YosypC @.***> wrote:

From this
https://github.com/linux-surface/linux-surface/discussions/1157#discussioncomment-6136733
discussion, data for the Pro 8 is missing, effectively rendering my Slim
Pen 2 useless. Touch screen gestures are also a problem although I am
unsure of correct installation, missing packages, or that I am simply dumb.
"libwacom-list-local-devices" output:
/dev/input/event15 is a tablet but not supported by libwacom
/dev/input/event11 is a tablet but not supported by libwacom
Failed to find any devices known to libwacom.
Knowing how to contribute would be of great help.
—
Reply to this email directly, view it on GitHub
https://github.com/linux-surface/libwacom-surface/issues/13, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ADG7W7QOLALXPEB6N4VMRBDXKSQ7TANCNFSM6AAAAAAZBY6BWI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

FYI, the list of devices won't be enough for ITHC devices, you need to input the number of the device and the post this line from the output:
Input device ID: bus 0x0 vendor 0x45e product 0x21 version 0x0

Or look at the output of systemctl status $(iptsd-find-service) and post the line that says something like
[22:13:26.813] [info] Connected to device 045E:0021

Ah, I somehow thought evtest would contain the ID.

Might be a good idea to modify ITHC and add the ID to the name.

Or I should just remember to ask for scripts/diagnostics.sh, which should also pick that up.

Or I should just remember to ask for scripts/diagnostics.sh, which should also pick that up.

@qzed Done! diagnostics.tar.gz

Okay, the device ID is 045E:0C37.
I've added a branch with the new data file. Can you git clone https://github.com/linux-surface/libwacom and git checkout feature/sp8, then build and test it by running the following commands?
meson setup build
meson compile -C build
./build/libwacom-list-local-devices --database ./data

All touch device IDs for ithc-based Surface devices are documented here: https://github.com/quo/ithc-linux/issues/5

@quo neat, thanks!

I've install all the other dependencies and gcc compiler, I can't find this package other than: Package libgudev-237-4.fc38.x86_64 is already installed.

You need the development variant since libwacom wants to compile against it and not just use it as runtime-dependencie, i.e. libgudev-devel (in general, the libraries you need for developing/compiling applications use the -devel suffix).

Anyways, this probably will work fine without much testing. Apart from the SP8, we're missing the SP9 and SLS. I'll try to add those as well in a bit and then I'll push an updated package. Feel free to test once I've released that.

I'd be great if I could get a compiled package without installing dependencies one at a time, thanks.

@YosypC new packages (v2.7.0-3) should be available ~5 minutes from now.

I have a libwacom setting page now. Thank you.


Minor correction to LocalService enum value

Following the doc and using 'localServices' results in MakeAppx failing to create a package due to an invalid manifest. Enabling verbose output produces the following:
MakeAppx : error: Error info: error C00CE169: App manifest validation error: The app manifest must be valid as per schema: Line 49, Column 12, Reason: 'localServices' violates enumeration constraint of 'localSystem localService networkService'. The attribute 'StartAccount' with value 'localServices' failed to parse.
As such, made a minor update to the doc to correct the value (localServices->localService).

Docs Build status updates of commit da8dc40:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

winrt-related-src/schemas/appxpackage/uapmanifestschema/element-desktop6-service.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

@drewbatgit

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@TiberiusDRAIG - Thanks for catching this. I confirmed that the token is "localService" in the schema.
<xs:simpleType name="ST_StartAccount">
<xs:restriction base="xs:string">
<xs:enumeration value="localSystem"/>
<xs:enumeration value="localService"/>
<xs:enumeration value="networkService"/>
</xs:restriction>
</xs:simpleType>

#sign-off


Backups are not performed regularly

Releases: 2.0.1
yml
models:
  db_backup:
    schedule:
      corn: "30 13 * * *"
    compress_with:
      type: tgz
    storages:
      local:
        type: local
        keep: 60
        path: /home/xx/db-backups
    databases:
      my_db:
        type: mysql
        host: xx
        port: 3306
        database: xx
        username: root
        password: xx
        exclude_tables:
          - xx

gobackup start
no effect

Show me the ~/.gobackup.gobackup.log

Here is a typo:
corn -> cron


Ci/#195 setup ci using GitHub actions

Issue Fix
Fixes #195

[x] Run the unit tests with ./gradlew check to make sure you didn't break anything

[x] If you have multiple commits please combine them into one commit by squashing them.

@abhijitramesh Can you please review it?

@shiv07tiwari Can you please review it?

@Sparsh1212 Please squash them into a single commit.

@Sparsh1212 Please squash them into a single commit.
Done!


Fix spelling error in traffic-analytics.md

Thank you @chunderbolt!
#sign-off

#sign-off

Thanks @chunderbolt


lib/pq driver instrumentation PoC

Depends on #88

Codecov Report

Merging #92 (2b57b09) into main (1d5d9b3) will decrease coverage by 9.37%.
The diff coverage is 56.41%.

@@            Coverage Diff             @@
##             main      #92      +/-   ##
==========================================
- Coverage   88.55%   79.17%   -9.38%
==========================================
  Files          18       20       +2
  Lines         568      802     +234
==========================================
+ Hits          503      635     +132
- Misses         47      134      +87
- Partials       18       33      +15

Flag
Coverage Δ

Linux
79.17% <56.41%> (-9.38%)
:arrow_down:

Windows
?

macOS
?

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

Impacted Files
Coverage Δ

...n/github.com/lib/pq/splunkpq/internal/connector.go
50.25% <50.25%> (ø)

instrumentation/github.com/lib/pq/splunkpq/sql.go
91.42% <91.42%> (ø)

Continue to review full report at Codecov.

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


Environment directory to move outside of App

The Application should be the product of its environment.  Wouldn't it be better to move the environment directory one level up?

Tracking this in https://github.com/angular/angular-cli/issues/1603.


fixes valid time fluency

Before this commit, adding a DatalogTransaction with valid time or valid time range had to be done:
Transactions::builder().put_with_valid_time_range(...);

This PR changes this approach to something more fluent:
Transactions::builder().put(..).valid_time(...).end_valid_time(...);

Codecov Report
Patch coverage: 78.94% and project coverage change: +3.60 :tada:

Comparison is base (544d85b) 67.82% compared to head (08d0142) 71.42%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #54      +/-   ##
==========================================
+ Coverage   67.82%   71.42%   +3.60%
==========================================
  Files           6        6
  Lines         230      224       -6
==========================================
+ Hits          156      160       +4
+ Misses         74       64      -10

Impacted Files
Coverage Δ

gxtdb-rs/src/transactions/mod.rs
84.12% <78.94%> (+13.11%)
:arrow_up:

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


llvm_{5,6,8..11}: explicitly pass lib to openmp.nix

this was a fallout of the stdenv.lib -> lib changes
actually llvm/7/ was fixed already.

Motivation for this change
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

@GrahamcOfBorg instantiate llvmPackages_11 llvmPackages_10 llvmPackages_9 llvmPackages_8 llvmPackages_7 llvmPackages_6 llvmPackages_5

@GrahamcOfBorg instantiate llvmPackages_11 llvmPackages_10 llvmPackages_9 llvmPackages_8 llvmPackages_7 llvmPackages_6 llvmPackages_5

@SuperSandro2000 this is the PR against  master, I'll apply you suggestions here.

@SuperSandro2000 this is the PR against  master, I'll apply you suggestions here.

@SuperSandro2000 PTAL

@SuperSandro2000 PTAL


Update documentation for users using intelliJ

The default profile, local-test causes DFSMiniCluster to not be downloaded. To avoid having errors showing, the hdfs-test profile should be used.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/2830/
Test PASSed.


Firefox requires the membarrier system call

Getting this error for every tab

Yes, instructions are provided here: https://github.com/jlesage/docker-firefox#allowing-the-membarrier-system-call
Let met know if you still have problems after that.

Thanks, it is working

Hello,
Is there a way to implement it using a docker-compose file ?
I don't find any lead on that... And I create my container using a docker-compose.yml file with Portainer CE v2.1.1.
Thanks in advance.

You can add something like this:
    security_opt:
      - seccomp:unconfined

Or you can update your Docker.  The membarrier system call is not a problem with recent versions of Docker.


Adds missing options to readme file

Added theme and allowGlobalShortcuts to readme Options section
PR Checklist
Please check all that apply to this PR using "x":

[x] I have checked that this PR is not a duplicate of an existing PR (open, closed or merged)
[x] I have checked that this PR does not introduce a breaking change
[ ] This PR introduces breaking changes and I have provided a detailed explanation below

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting)
[ ] Refactoring (no functional changes)
[x] Documentation changes
[ ] Other - Please describe:

Fixes
Fixes: #181

There is already an pull request for this: https://github.com/btzr-io/Villain/pull/183
Feel free to check other issues: https://github.com/btzr-io/Villain/issues


Fix #16183 when yii\helpers\BaseFileHelper sometimes returned wrong value

Q
A

Is bugfix?
yes

New feature?
no

Breaks BC?
no

Tests pass?

Fixed issues
#16183

Merged. Thanks!


How to install a custom brush?

Hello! How to use your pattern as a brush?

No there is no option as such I guess


No install documentation, unable to add as integration

I don't know how to properly state that I cannot for the life of me understand how this is added and works. I have two BMS attached via USB and have downloaded the repository via HACS and nothing stands out as to how I can add the sensor to the energy dashboard or anywhere for that matter. Can you provide a version number of HA this worked with and if this will work with the current version? Maybe a small step by step of what to look for?

Hi, sorry i didn't test this with a new HA version for a long time. Please try if the new version v1.0.2 works for you.

I removed it and reinstalled everything. Still getting unknown. I do see this error in the logs. Could it be the naming convention of the BMS?
bmstools: Not adding entity with invalid device info: Invalid device info {'default_name': 'Battery Management System', 'hw_version': '2021.6.25', 'identifiers': {('bmstools', 1)}, 'manufacturer': 'JBD', 'model': 'JBD-SP10S009-L8S-100A-B-K 1ST', 'sw_version': 22} for 'bmstools' config entry: device info needs to either describe a device, link to existing device or provide extra information.

Also, there are two BMS connected to this system. Unsure if that makes any difference as well.

This should be fixed now: https://github.com/ElD4n1/home-assistant-bms-tools-integration/releases/tag/v1.1.0-beta.2

Also, there are two BMS connected to this system. Unsure if that makes any difference as well. I did disconnect the one above and tried the second alone and got the same style error for the second one alone. For some reason HA does not want to make this a device properly.

For multiple devices it is mandatory that you allow the integration to write a unique serial number to the BMS in the last step.


Allow client to not sleep/disconnect while in forced passthrough

Currently, when you leave boundaries, or enable passthrough, it will disconnect (and sleep) alvr.
This is slightly inconvenient when you just want to reach out for something for a second, and/or sometimes you may still want to talk with someone while you're out of boundaries, in passthrough mode.
It would be nice to have toggle for this:

False - Disconnect client on boundaries leave or forced passthrough (default behavior in current alvr)
True - Don't disconnect or sleep client and allow for tracking/mic/everything as usual when you leave boundaries or enable passthrough.

This would be similar to how currently ALXR works.
I also noticed that on ALXR it sort of throttles tracking while in that mode (lower hz), but it may be runtime-specific (not that important if it still works)

Im also unsure if enabling passthrough manually and through boundaries leave is pretty much the same thing for app, but i think this toggle could work for both of them.

So in VD you are not able to talk with some one when you are outside of the boundaries.
Its just that the pc still thinks everything is connected so games, steamvr etc are still running while with ALVR as the connection gets closed you have to restart everything every time you go out of the boundaries.
That means if your drink is outside of the boundaries and you make a training you have to finish the training before your able to drink something otherwise you close the game and ruine your progress xD Having the drink inside the boundaries risks the drink itself.
What i mean is only close the connection if someone is closing the ALVR application on the headset or choose to disconnect not when going into passthrough :D


Could not connect to Local Rancher Server through EC2 host.

Rancher versions:
rancher/server:1.6.8
I have setup a Rancher Server on my local machine.
Added a custom host on same machine -- IT Worked
I was trying to add EC2 hosts by both ways (Through Wizard & manually by custom command through ssh from those machines)
I knew that my local IP would not be reachable so I set up a tunnel at "http://myhost.*****.me", changed the rancher server URL in UI.
If I curl "http://myhost.*****.me" from EC2 instances, it is accessible.
But somehow I am getting an error while running custom host command on those hosts.
Results:
sudo docker run -e CATTLE_AGENT_IP="EC2_INSTANCE_IP"  --rm --privileged -v /var/run/docker.sock:/var/run/docker.sock -v /var/lib/rancher:/var/lib/rancher rancher/agent:v1.2.6 http://myhost.*****.me/v1/scripts/5D8150914C11A1D35B64:1483142400000:18lFKUzZS1yDpuOMsaNgxI7m6Tc
INFO: Running Agent Registration Process, CATTLE_URL=http://myhost..me/v1
INFO: Attempting to connect to: http://myhost..me/v1
INFO: http://myhost.****.me/v1 is accessible
INFO: Inspecting host capabilities
INFO: Boot2Docker: false
INFO: Host writable: true
INFO: Token: xxxxxxxx
INFO: Running registration
Traceback (most recent call last):
File "./register.py", line 11, in
secret_key=os.environ['CATTLE_REGISTRATION_SECRET_KEY'])
File "/usr/local/lib/python2.7/dist-packages/cattle.py", line 45, in from_env
return gdapi.from_env(prefix=prefix, factory=Client, **kw)
File "/usr/local/lib/python2.7/dist-packages/gdapi.py", line 613, in from_env
return _from_env(prefix=prefix, factory=factory, **args)
File "/usr/local/lib/python2.7/dist-packages/gdapi.py", line 632, in _from_env
return factory(**result)
File "/usr/local/lib/python2.7/dist-packages/cattle.py", line 12, in init
super(Client, self).init(*args, **kw)
File "/usr/local/lib/python2.7/dist-packages/gdapi.py", line 197, in init
self._load_schemas()
File "/usr/local/lib/python2.7/dist-packages/gdapi.py", line 315, in _load_schemas
response = self._get_response(self._url)
File "/usr/local/lib/python2.7/dist-packages/gdapi.py", line 266, in _get_response
self._error(r.text)
File "/usr/local/lib/python2.7/dist-packages/gdapi.py", line 255, in _error
raise ApiError(self._unmarshall(text))
File "/usr/local/lib/python2.7/dist-packages/gdapi.py", line 300, in _unmarshall
object_pairs_hook=self.object_pairs_hook)
File "/usr/lib/python2.7/json/init.py", line 352, in loads
return cls(encoding=encoding, **kw).decode(s)
File "/usr/lib/python2.7/json/decoder.py", line 364, in decode
obj, end = self.raw_decode(s, idx=_w(s, 0).end())
File "/usr/lib/python2.7/json/decoder.py", line 382, in raw_decode
raise ValueError("No JSON object could be decoded")
ValueError: No JSON object could be decoded
Is there any way I can achieve this?
By "this", I mean I want my EC2 hosts to be connected to my local Rancher Server just like I have locally connected hosts.

ngrok is a great tool for doing this. I use it alsmost daily.
Note that you will have to create an account and save your authtoken locally via ngrok authtoken, otherwise ngrok will rate limit your requests and the hosts wont be able to connect.

The root cause could be that your tunnel is somehow not handling the traffic properly. In particular, we make use of websockets, which we've known to cause problems for proxies/tunnels that are not configured properly to handle then.
I'm going to close this. If you feel your question wasn't answered sufficiently, go ahead and reopen.


Speed has decreased (BGSG mode)

Hi!
In the latest beta version 0.2.211012 in BGSG mode, the speed decreased exactly 2 times compared to version 0.2.211007. The command line is the same.

It's strange, but I don't see such a speed difference right now. But I recreated the Bloom filter in the old and new versions.
I'll check it tomorrow on a more powerful computer.
Thanks.
$ ./keyhunt -m bsgs -t 4 -f tests/120.txt -n<PHONE_NUMBER>842624 -s 10 -S -k 52 -r<PHONE_NUMBER>00000000000000000000:819999999999999999999999999999 -B random [+] Version 0.2.211007 Chocolate ¡Beta!, developed by AlbertoBSD [+] Threads : 4 [+] Stats output every 10 seconds [+] K factor 52 [+] Mode BSGS random [+] Opening file tests/120.txt [+] Added 1 points from file [+] Range  [+] -- from : 0x800000000000000000000000000000 [+] -- to   : 0x819999999999999999999999999999 [+] N = 0x3ffffc8000000 [+] Bloom filter for<PHONE_NUMBER> elements : 5981.00 MB [+] Bloom filter for 87241524 elements : 299.05 MB [+] Allocating 1331.00 MB for 87241524 bP Points [+] Reading bloom filter from file keyhunt_bsgs_0_1744830464.blm .. Done! [+] Reading bloom filter from file keyhunt_bsgs_1_87241524.blm .. Done! [+] Reading bP Table from file keyhunt_bsgs_2_87241524.tbl .. Done! [+] Total<PHONE_NUMBER>33808384 keys in 10 seconds: ~11 Pkeys/s<PHONE_NUMBER>3380838  [+] Total<PHONE_NUMBER>69572352 keys in 20 seconds: ~11 Pkeys/s<PHONE_NUMBER>8478617  [+] Total<PHONE_NUMBER>70699264 keys in 30 seconds: ~11 Pkeys/s<PHONE_NUMBER>5689975  [+] Total<PHONE_NUMBER>73781760 keys in 40 seconds: ~11 Pkeys/s<PHONE_NUMBER>6844544  [+] Total<PHONE_NUMBER>09545728 keys in 50 seconds: ~11 Pkeys/s<PHONE_NUMBER>6190914  [+] Total<PHONE_NUMBER>45309696 keys in 60 seconds: ~11 Pkeys/s<PHONE_NUMBER>2421828  ^C] Thread 0x804464b4082eb857a7178b6d84b3ad
$ ./keyhunt -m bsgs -t 4 -f tests/120.txt -n<PHONE_NUMBER>842624 -s 10 -S -k 52 -r<PHONE_NUMBER>00000000000000000000:819999999999999999999999999999 -B random [+] Version 0.2.211012 Chocolate ¡Beta!, developed by AlbertoBSD [+] Threads : 4 [+] Stats output every 10 seconds [+] K factor 52 [+] Mode BSGS random [+] Opening file tests/120.txt [+] Added 1 points from file [+] Range  [+] -- from : 0x800000000000000000000000000000 [+] -- to   : 0x819999999999999999999999999999 [+] N = 0x3ffffc8000000 [+] Bloom filter for<PHONE_NUMBER> elements : 5981.00 MB [+] Bloom filter for 87241524 elements : 299.05 MB [+] Allocating 1331.00 MB for 87241524 bP Points [+] Reading bloom filter from file keyhunt_bsgs_3_1744830464.blm .. Done! [+] Reading bloom filter from file keyhunt_bsgs_1_87241524.blm .. Done! [+] Reading bP Table from file keyhunt_bsgs_2_87241524.tbl .. Done! [+] Total<PHONE_NUMBER>62578688 keys in 10 seconds: ~10 Pkeys/s<PHONE_NUMBER>6257868  [+] Total<PHONE_NUMBER>25157376 keys in 20 seconds: ~10 Pkeys/s<PHONE_NUMBER>6257868  [+] Total<PHONE_NUMBER>87736064 keys in 30 seconds: ~10 Pkeys/s<PHONE_NUMBER>6257868  [+] Total<PHONE_NUMBER>17633280 keys in 40 seconds: ~10 Pkeys/s<PHONE_NUMBER>0440832  [+] Total<PHONE_NUMBER>47530496 keys in 50 seconds: ~10 Pkeys/s<PHONE_NUMBER>4950609  [+] Total<PHONE_NUMBER>77427712 keys in 60 seconds: ~10 Pkeys/s<PHONE_NUMBER>4623795  ^C] Thread 0x813ed8bb220280f1de2ae67e4643b3

here are the speed compare with JLP bsgs. All are single thread.
BSGS v1.2
BabyStep:0x1000000 (2^24.00)
Start:60000000000000
Stop :80000000000000
Keys :1
Number of CPU thread: 1
BabyStep Thread 0: 0x1 -> 0x1000000
[2.46 MKey/s][Cnt 2^24.00][06s][787.1MB]
Sort Thread 0: 00000000 -> 02000000
[60.80 MSort/s][Cnt 2^25.00][00s][787.1MB]
Warning, range is not a multiple of nbThread*1024
GiantStep Thread 0:<PHONE_NUMBER>0000
[3.57 MKey/s][Cnt 2^27.42][50s][787.1MB]
Key# 0 Pub:  0x0385A30D8413AF4F8F9E6312400F2D194FE14F02E719B24C3F83BF1FD233A8F963
Priv: 0x6ABE1F9B67E114
Done: Total time 58s
time ./keyhunt -m bsgs -t 1 -f tests/55.txt -r<PHONE_NUMBER>0000:80000000000000 -n 0x1000000000000 -s 0
[+] Version 0.2.211012 Chocolate ¡Beta!, developed by AlbertoBSD
[+] Thread : 1
[+] Turn off stats output
[+] Mode BSGS secuential
[+] Opening file tests/55.txt
[+] Added 1 points from file
[+] Range
[+] -- from : 0x60000000000000
[+] -- to   : 0x80000000000000
[+] N = 0x1000000000000
[+] Bloom filter for 16777216 elements : 57.00 MB
[+] Bloom filter for 838861 elements : 2.88 MB
[+] Allocating 12.00 MB for 838861 bP Points
[+] processing 16777216/16777216 bP points : 100%
[+] Sorting 838861 elements... Done!
[+] Thread Key found privkey 6abe1f9b67e114
[+] Publickey 0385a30d8413af4f8f9e6312400f2d194fe14f02e719b24c3f83bf1fd233a8f963
All points were found
real    2m0.006s
user    1m59.191s
sys     0m0.070s
I use same number baby steps(2^24) to test.
It is about 2x slower than JLP BSGS.

Off-topic question: Alberto, do I need to recreate the bloom filter if I changed the search range and the base public key? (parameters k n unchanged)

No, you don't need to change the files, it is independent of the publickey, just if you change the n or k values the files are going to be created if you have the -S option

I can say to @albertobsd that on version012 I have speed around 800pkeys from 018 till now speed increases to 1,8Ekeys

@albertobsd
Same command but my output is a bit different. Could you explain this diff?, are there any change in speed ? Thank you

time ./keyhunt -m bsgs -t 1 -f tests/55.txt -r<PHONE_NUMBER>0000:80000000000000 -n 0x1000000000000 -s 0 -q -S -k 10
[+] Version 0.2.211012 Chocolate ¡Beta!, developed by AlbertoBSD
[+] Thread : 1
[+] Turn off stats output
[+] Quiet thread output
[+] K factor 10
[+] Mode BSGS secuential
[+] Opening file tests/55.txt
[+] Added 1 points from file
[+] Range
[+] -- from : 0x60000000000000
[+] -- to   : 0x80000000000000
[+] N = 0xfffffa000000
[+] Bloom filter for 167772160 elements : 575.00 MB
[+] Bloom filter for 8388608 elements : 28.76 MB
[+] Allocating 128.00 MB for 8388608 bP Points
[+] Reading bloom filter from file keyhunt_bsgs_3_167772160.blm .. Done!
[+] Reading bloom filter from file keyhunt_bsgs_1_8388608.blm .. Done!
[+] Reading bP Table from file keyhunt_bsgs_2_8388608.tbl .. Done!
[+] Thread Key found privkey 6abe1f9b67e114
[+] Publickey 0385a30d8413af4f8f9e6312400f2d194fe14f02e719b24c3f83bf1fd233a8f963
All points were found
real    0m10.609s
user    0m10.389s
sys     0m0.220s

My output:
Allocating is 2.00 MB
[+] Version 0.2.211117 SSE Trick or treat ¡Beta!, developed by AlbertoBSD
[+] Thread : 1
[+] Turn off stats output
[+] Quiet thread output
[+] K factor 10
[+] Mode BSGS secuential
[+] Opening file tests/55pub.txt
[+] Added 1 points from file
[+] Range
[+] -- from : 0x60000000000000
[+] -- to   : 0x80000000000000
[+] N = 0xfffffa000000
[+] Bloom filter for 167772160 elements : 575.10 MB
[+] Bloom filter for 5242880 elements : 17.97 MB
[+] Bloom filter for 163840 elements : 0.88 MB
[+] Allocating 2.00 MB for 163840 bP Points
[+] processing 167772160/167772160 bP points : 100%
[+] Making checkums .. ... done
[+] Sorting 163840 elements... Done!
[+] Writing bloom filter to file keyhunt_bsgs_4_167772160.blm .... Done!
[+] Writing bloom filter to file keyhunt_bsgs_6_5242880.blm .... Done!
[+] Writing bP Table to file keyhunt_bsgs_2_163840.tbl .. Done!
[+] Writing bloom filter to file keyhunt_bsgs_7_163840.blm .... Done!
[+] Thread Key found privkey 6abe1f9b67e114
[+] Publickey 0385a30d8413af4f8f9e6312400f2d194fe14f02e719b24c3f83bf1fd233a8f963
All points were found
real    2m0.252s
user    3m51.257s
sys     0m0.878s

Can you run the same again? The last version remade some files so if you notice the b Points were processed again.

[+] processing 167772160/167772160 bP points : 100%

If youn run the command again it will take less time

The different is  Allocating 2.00 MB for 163840 bP Points (Version 0.2.211117) vs Allocating 128.00 MB for 8388608 bP Points (0.2.211012)
2M vs 128M.
Is this important?
I have pull and build from newest version:  Version 0.2.211117

The number of bP points only change if you change the k or n values. The new newest version use less memory

The new version use less memory


scripts: Allow using quarantine mechanism with test_plan.py script

In twister, applying quarantine is a part of apply_filters() function. However, this function is not called when --load-test is used. Therefore, if one wants to use quarantines in combination with dynamic scope from the test_plan.py script, one has to pass such info through the script.

I found that this didn't work. I added an arg to quarantine, but didn;t propagate it when Filter() method is called. The only difference with the previous version (not seen due to rebase) is a modification from line 461

ping @nashif

@nashif rebased

another rebase. @nashif @hakehuang @gchwier please, re-approve


[fix][broker] Fix getMessageById throws 500

Motivation
When using  bin/pulsar-admin topics get-message-by-id public/default/my-topic-1 -e 0 -l 1024 with a non-existent ledger, I expect to throw the 400 error instead of 500.
CLI log:
$ bin/pulsar-admin topics get-message-by-id public/default/my-topic-1 -e 0 -l 1024

 --- An unexpected error occurred in the server ---

Message: Message not found, the ledgerId does not belong to this topic or has been deleted

Stacktrace:

org.apache.bookkeeper.mledger.ManagedLedgerException$LedgerNotExistException: Message not found, the ledgerId does not belong to this topic or has been deleted

Reason:
 --- An unexpected error occurred in the server ---

Message: Message not found, the ledgerId does not belong to this topic or has been deleted

Stacktrace:

org.apache.bookkeeper.mledger.ManagedLedgerException$LedgerNotExistException: Message not found, the ledgerId does not belong to this topic or has been deleted

Broker log:
2024-01-18T18:03:46,824+0800 [pulsar-web-47-14] INFO  org.eclipse.jetty.server.RequestLog - <IP_ADDRESS> - - [18/1月/2024:18:03:46 +0800] "GET /admin/v2/persistent/public/default/my-topic-1/ledger/1024/entry/0 HTTP/1.1" 500 340 "-" "Pulsar-Java-v3.2.0" 29

Modifications

In the internalGetMessageById, when readEntryFailed returns LedgerNotExitException, convert this exception to 400 error.
When the admin receives the 400 error, skip handle recovery.

Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

/pulsarbot rerun-failure-checks


GCR Compare with RoG

Hi,
Both RoG and GCR are inspiring works on KGQA tasks. Furthermore, it is very interesting to see how the trie structure built from KG can be used to constrain the output of LLM for reasoning paths.
I am trying to find the subtle differences between RoG and GCR since they are mostly following the same pipeline. In my understanding, GCR has one more component, which is graph constrained decoding, compared to RoG during planning. Is this the only major difference? I wonder if I missed any other significant part that boosts GCR performance over RoG.
Thanks

Hi,
Thank you for your interest in our work. GCR builds on our previous project, RoG. Unlike RoG’s planning-retrieval framework, GCR uses graph-constrained decoding to directly generate KG-grounded reasoning paths, integrating the KG reasoning process seamlessly into the LLM decoding.
Best.


finished events now appear in the correct order (newest first)

"Tidligere arrangementer" events now appear in the correct order (newest first)
Pull request checklist
Please check if your PR fulfills the following requirements:

[ ] Tests for the changes have been added (for bug fixes / features)
[ ] API docs on Codex have been reviewed and added / updated if needed (for bug fixes / features)
[ ] The fixtures have been updated if needed (for migrations)

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

'Expired' va en property i Event klassen og ikke en boolean attributt, så kune ikke bruke ORM filtrering på den her, så måtte bruke if.. else her som en liten workaround


Add Deployment manifest for Function Controller

The controller itself should run in the cluster as a deployment so it will be managed (and so it won't require the use of kubectl proxy as it currently does when running locally).

we now have 3 components (topic-controller, topic-gateway, and function-dispatcher) that all run in the cluster, as described in the README


add KCP watcher controller and tests

Integration with lifecycle manager
Reiterate implementation and refactor test (improvement areas could be covered in follow-up PRs)
KCP Watcher can be enabled via flag enable-kcp-watcher, which is by default set to true
SKR watcher chart moved to operator/internal/charts
Necessary implementation added to Dockerfile

/hold

Was this working before? https://github.com/kyma-project/lifecycle-manager/pull/224/files#diff-caa6bb0b6f058fb6a0a3d3421cac26d99698d1edd6aec7757dd8b45c1b87ac67R386

/unhold


Enable all services in deploy-EdgeX.sh

Signed-off-by: Jeremy Phelps<EMAIL_ADDRESS>

recheck

recheck

recheck

recheck

recheck

recheck

@tsconn23 +2?


Added settings folder with settings files for specific build environments

Using a single settings file for all environments (development, testing, production) is inefficient since many variables and parameters vary according to the current environment. For example, you'd want to set the 'DEBUG' option to 'True' during local development, but it must set to to 'False' in a production environment. This hierarchy lets you do just that. Additional instructions to configure your vitualenv to automatically source respective settings will be provided in the wiki.
Resolves #25
Signed-off-by: Nitin Paul<EMAIL_ADDRESS>

Current coverage is 0.00%

Merging #34 into master will not affect coverage as of 615d932

No diff could be generated. No reports for #34 found.
Review entire Coverage Diff as of 615d932

Powered by Codecov. Updated on successful CI builds.

Looks good to me!

Looks good. Merging this in.


Files missing from the pdf.js folder structure

I have a Xamarin solution that has pdf.js as a sub-module. I'm trying to build it through VSTS and it failed so I pulled it down to my PC.
Steps to reproduce the problem:

Pull the code down in Visual Studio
npm install
gulp generic
Build the solution

Error MSB3030: Could not copy the file   "Assets\pdf.js\build\generic\web\compatibility.js" because it was   not found.
Error MSB3030: Could not copy the file   "Assets\pdf.js\build\generic\web\l10n.js" because it was not found.
Error MSB3030: Could not copy the file "Assets\pdf.js\build\pdf.js" because it was not found.
Error MSB3030: Could not copy the file   "Assets\pdf.js\build\pdf.worker.js" because it was not found.
The gulp file doesn't create these files, although they do get put in other places in the folder structure. If I manually copy them the solution builds in Visual Studio but the VSTS build still fails.

It looks like you're using an old version of PDF.js because web/compatibility.js was removed quite a while ago and I'm not that web/l10n.js ever existed. I would suggest upgrading to the latest release and changing your build scripts accordingly.

Sorry for the n00bness but how do I update the version? All I can see in the submodule reference is this:
[submodule "src/Investor.Droid/Assets/pdf.js"]
path = src/Investor.Droid/Assets/pdf.js
url = https://github.com/mozilla/pdf.js.git
As you can tell this is the first time I've worked with all this stuff,

I followed the steps on this page:
https://stackoverflow.com/questions/8191299/update-a-submodule-to-the-latest-commit
but I still get the same errors.

I deleted and re-added the submodule and it still gets the same errors.

You need to build PDF.js after you have pulled it. Use e.g., gulp generic for that; refer to https://github.com/mozilla/pdf.js/wiki/Setup-pdf.js-in-a-website#build-pdfjs-from-source.

It turns out that the problem was in the Visual Studio project, not pdf.js. The project file had AndroidAsset entries that expected files in the locations of the old version. Once I updated it to where they are in the new version it was all good.
Thanks for your help, Tim.


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-master" workflow

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

Project coverage is 98.27%. Comparing base (e547e42) to head (160d4a7).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #118   +/-   ##
=======================================
  Coverage   98.27%   98.27%
=======================================
  Files           1        1
  Lines          58       58
  Branches       10       10
=======================================
  Hits           57       57
  Misses          1        1

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


Error while inference: RuntimeError: expected scalar type Half but found Char

Hi,
I am getting the following error, while replicating the quickstart notebook: https://github.com/facebookresearch/llama-recipes/blob/main/quickstart.ipynb with model "meta-llama/Llama-2-13b-hf"
The error occurred when running inference for eval prompt given in notebook.
Please help to resolve the issue.
Thanks

@ShreyansDhankharTR Please share the detailed error message that you are seeing?
Also include the GPU / machine details on which you ran the tests, include the output of python -m torch.utils.collect_env

llama-recipes/inference# python chat_completion.py --model_name "/media/root/sdc/data/model/Llama-2-13b-chat-hf" --prompt_file chats.json  --quantization
File "/media/root/ssd2t/data/pro/tmp/ol/new/llama-recipes/inference/chat_completion.py", line 129, in
fire.Fire(main)
File "/root/anaconda3/envs/openai/lib/python3.10/site-packages/fire/core.py", line 141, in Fire
component_trace = _Fire(component, args, parsed_flag_args, context, name)
File "/root/anaconda3/envs/openai/lib/python3.10/site-packages/fire/core.py", line 475, in _Fire
component, remaining_args = _CallAndUpdateTrace(
File "/root/anaconda3/envs/openai/lib/python3.10/site-packages/fire/core.py", line 691, in _CallAndUpdateTrace
component = fn(*varargs, **kwargs)
File "/media/root/ssd2t/data/pro/tmp/ol/new/llama-recipes/inference/chat_completion.py", line 96, in main
outputs = model.generate(
File "/root/anaconda3/envs/openai/lib/python3.10/site-packages/torch/utils/_contextlib.py", line 115, in decorate_context
return func(*args, **kwargs)
File "/root/anaconda3/envs/openai/lib/python3.10/site-packages/transformers/generation/utils.py", line 1588, in generate
return self.sample(
File "/root/anaconda3/envs/openai/lib/python3.10/site-packages/transformers/generation/utils.py", line 2642, in sample
outputs = self(
File "/root/anaconda3/envs/openai/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
return forward_call(*args, **kwargs)
File "/root/anaconda3/envs/openai/lib/python3.10/site-packages/accelerate/hooks.py", line 165, in new_forward
output = old_forward(*args, **kwargs)
File "/root/anaconda3/envs/openai/lib/python3.10/site-packages/transformers/models/llama/modeling_llama.py", line 806, in forward
outputs = self.model(
File "/root/anaconda3/envs/openai/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
return forward_call(*args, **kwargs)
File "/root/anaconda3/envs/openai/lib/python3.10/site-packages/transformers/models/llama/modeling_llama.py", line 693, in forward
layer_outputs = decoder_layer(
File "/root/anaconda3/envs/openai/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
return forward_call(*args, **kwargs)
File "/root/anaconda3/envs/openai/lib/python3.10/site-packages/accelerate/hooks.py", line 165, in new_forward
output = old_forward(*args, **kwargs)
File "/root/anaconda3/envs/openai/lib/python3.10/site-packages/transformers/models/llama/modeling_llama.py", line 408, in forward
hidden_states, self_attn_weights, present_key_value = self.self_attn(
File "/root/anaconda3/envs/openai/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
return forward_call(*args, **kwargs)
File "/root/anaconda3/envs/openai/lib/python3.10/site-packages/accelerate/hooks.py", line 165, in new_forward
output = old_forward(*args, **kwargs)
File "/root/anaconda3/envs/openai/lib/python3.10/site-packages/transformers/models/llama/modeling_llama.py", line 295, in forward
query_states = [F.linear(hidden_states, query_slices[i]) for i in range(self.pretraining_tp)]
File "/root/anaconda3/envs/openai/lib/python3.10/site-packages/transformers/models/llama/modeling_llama.py", line 295, in
query_states = [F.linear(hidden_states, query_slices[i]) for i in range(self.pretraining_tp)]
RuntimeError: expected scalar type Half but found Char

8*3090 24G
Please help to resolve the issue.
Thanks

Experienced the same issue in both 13b-chat-hf & 70b-chat-hf with --quantization.
The chat_completion.py ran smoothly in 7b-chat-hf with --quantization, and all 3 chat-hf without --quantization.


incorporate into Q estimator Bill Premerlani's fast rotation handling from MatrixPilot

not yet flight tested
small subset of https://github.com/PX4/Firmware/pull/4989

@kd0aij What is the status of this?

I'm waiting till https://github.com/PX4/Firmware/pull/5498 and https://github.com/PX4/jMAVSim/pull/36 are merged. Once those are in I'll be able to fly automated trajectories (attitude sequences) to thoroughly analyze performance, first in SITL with ground truth, then on a real vehicle.

Can one of the admins verify this patch?

rebased and tested in SITL and flight tested on QAV250. Significant reduction in attitude estimation errors during coordinated turns: (roll/pitch in degrees, note the good agreement with ground truth, especially after the turn stops)

This is now ready to merge.

Jenkins test this please.

for comparison, this is the same flight sequence without this commit: (not the significant pitch error after the turn stops)

flight tested and rebased on master

Looks good to merge to me.

Looks good to merge to me.

I believe you, but I think there are two significant changes here and I don't swe any justification why we now only estimate gyro bias when we are rotating below 10 deg/s. I would be happy with just another sitl plot  vs ground truth to compare with those above when we set it back to 1 rad/s as the gyro bias estimation cut off.

Just out of curiosity, where did the 1 rad/sec value come from, originally? I don't remember seeing any testing that justified it.

@kd0aij I settled on adding the logic after doing some flight test and an a b test to see that it did help in high g turns. At the time, we didn't have any such logic so I wanted to be conservative. If we can afford to lower it to this value I'm fine with that.

OK, I'll repeat the testing I thought I had done earlier and document it properly this time. It's possible that the results could be different than with MatrixPilot anyway. One caveat I just remembered is that this needs to work with no GPS available for acceleration compensation.  I know I had the GPS disconnected during actual flight testing, but maybe not for SITL.

@jgoppert baseline Q estimator at yaw rate of about 0.5 rad/sec (significant bias error):

and at about 0.2 rad/sec:

Nice, I'm good to merge now.
On Nov 3, 2016 2:29 PM, "Mark Whitehorn"<EMAIL_ADDRESS>wrote:

@jgoppert https://github.com/jgoppert baseline Q estimator at yaw rate
of about 0.5 rad/sec (significant bias error):
[image: yawrate_0 5]
https://cloud.githubusercontent.com/assets/2300221/19979314/9efcaf7a-a1bf-11e6-93d1-b6e96d41923b.png
and at about 0.2 rad/sec:
[image: yawrate_0 2]
https://cloud.githubusercontent.com/assets/2300221/19979714/0dd4c83c-a1c1-11e6-9f76-0fe8aaa3bcb7.png
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/PX4/Firmware/pull/5275#issuecomment-258232742, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAc6rMh-hHKVhRzDNaBc7C2eLFxrm0PVks5q6igHgaJpZM4JfI7S
.

@jgoppert I guess you have to do "another review approving the changes in the pull request" before this can be merged. That's actually pretty cool.


Error: Attribute 'password' is longer than platform limit of 2560 chars

Can't connect my Account. Trying via email shows the error code Error: Attribute 'password' is longer than platform limit of 2560 chars
Also can not type something when trying to input the token.

What platform are you on? mod.io tokens are very long, and I only confirmed that I could store them using Apple Keychain. Also, you should be able to type when entering a token manually, but it intentionally does not show what you are typing (think of it like when you see •••••••• on websites).

Oh, so does copy pasting the code via CTRL + V not do anything? I‘m on Windows 11.

Pasting in your token is expected, it would be insane to have someone type it in manually lol
There is a bug with a dependency, however, at least on macOS, that might freeze the program when you try to paste in your token.
I just about have a fix ready for the error it gave when you did the email login method, but I don't have a Windows computer. Would you be willing to download it and see if it fixes your issue?

I dont know when I‘ll be back home, will test and respond back as soon as I‘m back home.

Also, the program doesn‘t freeze or anything, it just won‘t let me Type anything. When pressing enter it always shows five or six ****** and shows the error code i mentioned. Dont know if this is maybe helpful.

Also, the program doesn‘t freeze or anything, it just won‘t let me Type anything. When pressing enter it always shows five or six ****** and shows the error code i mentioned. Dont know if this is maybe helpful. Will post a screenshot if the error is still there.

Oh, good! That probably means the warning message about the prompt only applies to Mac, which is okay because I'm probably going to be the only person involved in this that's on a Mac lol
If the fix (that I'm about to upload the build to) works, then the error should go away for both the email method and the token method. You should be able to paste your token (nothing will appear at first), press enter (the stars will appear), and then you should be logged in.

v0.1.1 has been pre-released, you can download it here when you're ready to try it.

Getting the same issue even with the v0.1.1 pre-release. I think the windows password vault has a password length limit

Getting the same issue even with the v0.1.1 pre-release. I think the windows password vault has a password length limit

It does. In v0.1.1 I tried working around that by splitting the token into multiple entries, but that hasn't worked apparently... I might need to include some debug code in a build and see what it outputs.

Would it be an idea just to have an encrypted file sitting somewhere in the appdata directory that stores the token?

Would it be an idea just to have an encrypted file sitting somewhere in the appdata directory that stores the token?

I used to actually just store the token in the app_data file, which is the file that the program creates and uses to keep track of installed mods and their versions and such, but I'm not really familiar with how I can encrypt that in a way that isn't easy to decrypt from a malicious program or person. That's why currently the program just relies on platform-specific password storing APIs.

I don't think encrypting it would be necessary, at least not for now. Don't think there's anyone targeting mod.io tokens and especially not when they're read only

Tokens created with the email code method might not be read only, I'm honestly not sure, I don't think there's a way to specify with the modio Rust library I'm using. I'm about to upload a new build and make a new release, but if I can't figure this out by the end of the day then yeah, I guess I'll just store it with the rest of the app data. Would prefer to not do that though, so I'll keep trying for now.

Sorry for the delay, v0.1.1-beta.1 is now up.

Using the email method:
(Entered code and stuff)
[src/authentication.rs:53] remaining_password = "1613 characters long token starting with eyJlbm and ending with 09JsLVQ"
Error: Attribute 'password' is longer than platform limit of 2560 chars
Press q to quit

Whoops- sorry for accidentally asking you to tell me your token lol, meant to do dbg!(remaining_password.len()) not dbg!(remaining_password).
Anyway... yeah I've got pretty much no idea, it clearly isn't going over the limit but the error's still there. Might be time to give up and kindly ask that nobody goes snooping in the app_data file, or at least on Windows, I don't technically have to do the same on Mac and Linux where it works fine.

v0.1.1-beta.2 is ready, the Windows build shouldn't depend on keyring at all.

Login with Token works now, got in first try.

Great, thank you everyone who participated for your assistance!


affinity filters have to be explicitly enabled in OSP9

In OSP9 packstack deployments nova scheduler doesn't include by default following filters: "ServerGroupAntiAffinityFilter,ServerGroupAffinityFilter" and these have to be added explicitly.
It would worth adding a note into prerequisities about this.

@javierpena seems it's in the docs. Worth closing?


Restructure docs to match the new structure

This doesn't add any new content or modifies existing one, it just moves things around to match the new docs structure.

IMO just go for it and we can spot-check improvements to the structure in future PRs. This PR will get out of date immediately if we leave it open for very long :-)

YAY! Let's try this out and modify as we go along.


License?

I've looked through your project but I can't seem to find any documentation on it's licensing. Would love to  verify if it had an MIT or Apache 2.0 license so that I could know whether or not I could use this project. Great project btw!

Resolved, Thanks for reporting


Insert fails if body includes _id in string

Real application is using Pydantic.json(), but this is a short example...
    tst = {}
    tst["HTML"] = "<!DOCTYPE html>
link_id=\"fred\"
"

    meta = nodes.insert(json.dumps(tst))

  File "./main.py", line 321, in parse_object_as_message
    meta = nodes.insert(json.dumps(tst))
  File "/ENV/lib/python3.8/site-packages/arango/collection.py", line 2335, in insert
    vertex = self._ensure_key_from_id(vertex)
  File "/ENV/lib/python3.8/site-packages/arango/collection.py", line 205, in _ensure_key_from_id
    doc_id = self._validate_id(body['_id'])
TypeError: string indices must be integers

This is a data issue - other nodes inserted ok.
I'll look at using a custom JSON encoder to handle the date times...

Hi @nhi-vanye,
It seems like the value of body['_id'] was somehow invalid? Could you check what it was?

body is a string - so "_id" is doing a substring search, but can't deference body.
Worked around it by dumping to string and reloading - the alternate would have been to use a custom JSON encoder. Keeps the POC moving forward until I write the custom encoder.
meta = nodes.insert(json.loads(msg.json()))
( I think the problem is the intersection between Pydantic structs (and their json() method ) and Arango. Prior approach worked (using msg.json() directly) - but is a data issue if there a "_id" substring in the result and zi can't control the data being added...)

Collection.insert method expects a Python dictionary. But I can see the use case of wanting to insert a JSON string directly. I will see if I can add this feature in the next version. Thanks.

Hi @nhi-vanye,
Due to lack of time, I've decided not to add the feature (document CRUD via raw JSON strings) for the time being. You will have to handle the data sanitization on your end, and ensure that the input documents are valid python dictionaries. Thanks.


bug/OP-14540-auditDownlaodParams adding noofdays for downlaod audit

https://devopsmx.atlassian.net/browse/OP-14540
We prefer small, well tested pull requests.
Please refer to Contributing to Spinnaker.
When filling out a pull request, please consider the following:

Follow the commit message conventions found here.
Provide a descriptive summary for your changes.
If it fixes a bug or resolves a feature request, be sure to link to that issue.
Add inline code comments to changes that might not be obvious.
Squash your commits as you keep adding changes.
Add a comment to @spinnaker/reviewers for review if your issue has been outstanding for more than 3 days.

Note that we are unlikely to accept pull requests that add features without prior discussion. The best way to propose a feature is to open an issue first and discuss your ideas there before implementing them.

This PR  triggered this build in this execution, please wait for it  and child pipelines to succeed before merging:  https://isd.prod.opsmx.net/deck/#/applications/gcp-gitprtrigger/executions/details/01GDJDPXJ8832MGVZFZW19MBS3

This PR  triggered this build in this execution, please merge to master only if this succeeds:  https://isd.prod.opsmx.net/deck/#/applications/gcp-gitprtrigger/executions/details/01GDJDQFZBZZN66NHAEZXRR0G0

SUCCESSFUL RunTests RESULT: Tests run: 0, Failures: 0, Errors: 0, Skipped: 0 Reviewers ,  this PR did not cause the failures in  https://jenkins.opsmx.net:8181/jenkins/job/Dev-run-tests-on-gitpr-parallel/ws/emailable-report-523.html


Update JvmWebSocketClientFactory.kt

Codecov Report

Merging #10 into master will not change coverage.
The diff coverage is 0%.

@@            Coverage Diff            @@
##             master      #10   +/-   ##
=========================================
  Coverage     21.75%   21.75%
  Complexity      500      500
=========================================
  Files           177      177
  Lines          7580     7580
  Branches       2015     2015
=========================================
  Hits           1649     1649
  Misses         5495     5495
  Partials        436      436

Impacted Files
Coverage Δ
Complexity Δ

...m/soywiz/korio/net/ws/JvmWebSocketClientFactory.kt
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

Continue to review full report at Codecov.

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


OAK-10526 : fix test regression for mongo 70 split doc

fixes test regression introduced by https://github.com/apache/jackrabbit-oak/pull/1190


[jnigen] Update links and paths to use dart-lang/native

Final (?) step to merge jnigen here.

coverage: 90.813% (-7.3%) from 98.152%
when pulling 1257bf7b51162b5704f14f6963914c61d4739cbf on jnigen-doc-update
into 5dca10ea324bb19f104126d72472280091829873 on main.


Fix switch quality video loop bug.

When video has ever switched quality, and looping is enabled, the deleted video object will receive the play event and starts to play in the background. The callback for play event should use player as reference instead of video, which is already deleted from DOM.
There might be a garbege collection problem in the switch quality part, if the reference to the old video tag is kept somewhere in the event.

but then you will hear two videos playing simultaneously, and likely starts at different time.

Merged, thanks!


remove PropertyAssignament from ObjectLiteralExpression

Hello, before throwing myself to the water and implement it, I would like to know if there is some way of removing a PropertyAssignament from an ObjectLiteralExpression, example:
given an object literal {name: 'seba', lastname: 'gurin'}
and I want to remove property 'name' so the resulting object is {lastname: 'gurin'}
Thanks in advanced

Nope. Looks like that was never implemented (same for ShorthandPropertyAssignment and SpreadAssignment). It looks like that would be a good one to implement and add .remove() methods on them. Thanks for checking 😄

thanks! nice to know - will PR then.
Idea / suggestion - from my point of view (user manipulating the AST) exporting removeChildren() (/src/manipulation/manipulations/removal.ts) would give a lot of flexibility. I know is low level - and I know is one more signature to maintain backwards compatibility - but this would be very very useful for advanced users . These things could be exported in a "lowLevel" namespace or something like that to not disturb common users :P .. the same applies to other utilities in manipulations/ my two cents.

aside question - I feel like there should be a couple extra interfaces for this  kind of artificialities. For example, Removable (for this case), StructureFillable (fill), StructureGetable (getStructure)  (and a couple more that I can't remember now), Also typeguards for those would be nice. I wonder if you agree with this....

I'd rather implement it for the scenario and have people submit pull requests/report an issue when something's not implemented. That way everyone can benefit. Additionally, there can be a lot of edge cases and exposing removeChildren would just lead to people opening bug reports when it doesn't work the way they expect it to.

https://github.com/dsherret/ts-simple-ast/pull/343

I missed a few things in the review so did a new commit. This should be good to go so I'm doing a release now.


Add matrix badge

Also remove CEH badge as it is superseded by Intersect policy.

Windows build seems broken at solving time, not related to anything in here?

Yes, this is unrelated.