re #149 support event streaming

I'm happy to contribute this small, but very helpful improvement - event stream listener.
Starting from Marathon 0.9.0 it provides /v2/events endpoint, which is implementation of Server-sent Events. It's much better compared to event callbacks since there is no need to register Bamboo as callback on Marathon.
Implementation is backward compatible with 0.7.0 because this mode is disabled by default. It could be enabled in config like that:
{
  "Marathon: {
    "Endpoint": "...",
    "UseEventStream": true
  }
}

NB: if "UseEventStream" is enabled then callbacks are not auto-registered.
When Bamboo will drop support for Marathon < 0.9.0 then this property could be true by default.
Implementation was tested with Marathon 0.9.0, covered cases:

Listening for events
Re-connection when Marathon is not responding
Bad data from Marathon
Many events at a time

Looks good for the most part, I've added a couple of comments. On a broader note, I'd love to see some of this broken out into smaller functions; a function to parse a single message and return an event would be great, as it'd allow for very easy testing. At the moment, this is a rather had function to test.

@bluepeppers I've made some changes, including:

Reuse code from EventSubscriptionAPI for parsing and dispatching events
Use Ticker for retrying
Log every error

Currently listenToEventStream contains only SSE handling logic. Should I split it more?

I like this. I'd like to split it up more, and break the whole thing out behind an interface, but this is a move in the right direction. Seeing as this is opt in, I'd be happy merging it now, but will probably wait till Monday, and confer with @activars about the best way to test this mode with some decent traffic.

@bluepeppers ok, lets wait what @activars will say :)

Looks good  :+1:
We will test this branch on our dev cluster.

don't worry about the abstraction too much, we will refactor some common features in next release.

docker build with this was broken for me-- use of the http.Client timeout requires go 1.4, the dockerfile uses go 1.2

You also may want to wait until the fix for https://github.com/mesosphere/marathon/issues/1926 is included in a marathon release (probably 0.11). Otherwise there could in some cases be a delay for bamboo to receive events.

@bsideup hey, sorry we haven't done anything on this. I've finally automated some bamboo integration testing stuff, and so I'll give this a shot on the staging cluster in the next week.

Hi @bluepeppers ! Sounds great :) Ping me if anything will not work for you :)

One many thing; we have a username and password for marathon configurable: https://github.com/QubitProducts/bamboo/blob/master/configuration/marathon.go#L10-L15. The event stream request should probably support HTTP basic auth also

@bluepeppers it was introduced after this PR :) But I can support it if you want

@drewrobb good catch. Looks like ubuntu base image comes with Go 1.2 by default. Any idea how to fix it?

@bluepeppers rebased on top of last master

@bluepeppers implemented basic auth

The go vet message at https://travis-ci.org/QubitProducts/bamboo/builds/80229954 was very useful. Without that change, the goroutine was listening to the same (last) marathon always, which causes the problem @drewrobb linked with marathon to be much more of an issue. I was having problems getting event streaming to work with this patch, but I think that go vet may have found the problem.

@bluepeppers yes, already fixed :)

@bluepeppers any news?

+1 to getting this in, I have been using it for a while now successfully

+1

Appologize we haven't got chance to test this out because there are a lot of other tasks we need to work on internally.
If anyone has time to test it out & sign-off on the status (with 3 thumbs up), we can speed up the release process.
We will deploy this on an internal test cluster to test it out in November.

Since this is an optional mode I think it's pretty safe to merge, I know at
least 2 production deployments running my fork so it's battle tested:)
On Tue, 27 Oct 2015 at 6:59 PM Jing Dong<EMAIL_ADDRESS>wrote:

Appologize we haven't got chance to test this out because there are a lot
of other tasks we need to work on internally.
If anyone has time to test it out & sign-off on the status (with 3 thumbs
up), we can speed up the release process.
We will deploy this on an internal test cluster to test it out in
November.
—
Reply to this email directly or view it on GitHub
https://github.com/QubitProducts/bamboo/pull/151#issuecomment-151569035.

I'll merge this today (absent any issues). Sorry it took so long

Awesome, thanks!
On Thu, 12 Nov 2015 at 8:58 PM Laurie Clark-Michalek <
<EMAIL_ADDRESS>wrote:

Merged #151 https://github.com/QubitProducts/bamboo/pull/151.
—
Reply to this email directly or view it on GitHub
https://github.com/QubitProducts/bamboo/pull/151#event-462793867.

Yeah, thanks guys!

@bluepeppers was it released btw?

No. The next release will probably be 0.3, with this and #180 being the features I want to test the most. I'm running master + #180 in dev atm, though there's still some extra work to be done around the UI for #180.

@bluepeppers ok, keep me in touch then :) Thanks!


[BUG] Delete All rows in table not working.

When I try to select all rows and delete the rows on provided GUI it does not perform the required action. The reason is it prepends a comma(,) before the first id which makes an invalid SQL syntax. You can see from the image below.

OS INFO : Ubuntu 18.04

Update to 2.5.3 fix.

Great! It is fixed


Azure CLI to open many ports

Is your feature request related to a problem? Please describe.
Need to open a bunch of ports, not just one, to my VM using Azure CLI
Describe the solution you'd like
Able to open a bunch of ports, not just one, to my VM using Azure CLI
Describe alternatives you've considered
being able to declare what you want for your VM in a single operation, like #2011
Additional context
I'm following the following to use Azure CLI to open ports
https://docs.microsoft.com/en-us/azure/virtual-machines/linux/nsg-quickstart
I.e., the command,
az vm open-port --resource-group myResourceGroup --name myVM --port 80

works fine. However, when I try to open up the next port, 21, I got this:
Security rule open-port-80 conflicts with rule open-port-21. Rules cannot have the same Priority and Direction.

So

how do I open a bunch of ports (not just one) to my VM using Azure CLI?
how do I add more ports to the already opened set of ports to my VM using Azure CLI?

thx

Arguments
    --port   [Required]: The port or port range (ex: 80-100) to open inbound traffic to. Use '*' to
                         allow traffic to all ports.
    --priority         : Rule priority, between 100 (highest priority) and 4096 (lowest priority).
                         Must be unique for each rule in the collection.  Default: 900.

So you have a couple options:

Specify a range of ports and issue one command
Specify each port or port range with separate calls and specify the priority (you can omit on the first call if you are okay with the default of 900).

Thanks @tjprescott,

what's the implication for the priority? Does it determine the order Azure ... hmm... still can't think of any reason port number need to be associated with priority.
when I was creating the VM using to portal, I used to open the following port, is there any way to know how the portal assign the priorities to each of them?

22
21
25
53
80
110
443
587
993
995

also, with the portal, the Endpoints can choose either TCP or UDP. How to choose TCP or UDP for Azure CLI?

thx

The priority is related to the NSG rule. Rules are executed in priority from high to low and once a rule is satisfied, lower priority rules are not processed. Priority really doesn't have to do with ports--it has to do with conflicting the rules (which is the mechanism through which you can allow or deny ports).
If you run network nsg show on the network security group for your VM, it should list all the rules and show the priorities that the portal designated. We do not try to apply a heuristic.
vm open-port is a convenience command only and thus it doesn't allow you to specify UDP/TCP for the resulting rule. From the extended help text description, if you want more fine grained control that the open-port command allows, you need to use the underlying network commands. In your case it would az network nsg rule create.

thanks, appreciate the detailed answer.

Just to be 100% sure, vm open-port open but UDP/TCP ports, right, or it is just TCP?

Yes, it opens both.


CO2 alarm

We need to order a CO2 alarm for the lab.
Here is some reporting on why CO2 is bad.

[x] Research vendors (Danielle)
[x] Pick item (get the Extech on GSA)
[x] put together purchasing info for kathleen
[x] put purchase info on google an link
[x] purchase when we have $
[ ] delivered Nov 29
[ ] Mount on the wall under white board

Some options--all have audible alarms, but none say anything about being approved by FDA:
Extech (available on GSA)
Grainger
Gain Express/Amazon
Forensics Detectors/Amazon
GZAIR/Amazon
Langkou Store/Amazon

Thanks for tracking these down. Maybe certification is from OSHA. What do
you recommend?
-paul
On Mon, Sep 13, 2021 at 5:11 PM dmperez89 @.***> wrote:

Some options--all have audible alarms, but none say anything about being
approved by FDA:
Extech (available on GSA) http://www.extech.com/products/CO220
Grainger
https://www.grainger.com/product/21RV54?ef_id=CjwKCAjw7fuJBhBdEiwA2lLMYaR5oSnKBuEAWZoHa6YJLteO6c88rDXR7yKuajZoyReTfpgiBEdIxBoCEycQAvD_BwE:G:s&s_kwcid=AL!2966!3!496359978193!!!g!438000512349!&gucid=N:N:PS:Paid:GGL:CSM-2295:4P7A1P:20501231&gclid=CjwKCAjw7fuJBhBdEiwA2lLMYaR5oSnKBuEAWZoHa6YJLteO6c88rDXR7yKuajZoyReTfpgiBEdIxBoCEycQAvD_BwE&gclsrc=aw.ds
Gain Express/Amazon
https://www.amazon.com/Temperature-Humidity-Mountable-Detector-0-2000ppm/dp/B00MB93CK8/ref=sr_1_13?dchild=1&keywords=Co2%2BAlarm&qid=1631575963&sr=8-13&th=1
Forensics Detectors/Amazon
https://www.amazon.com/Dioxide-Monitor-Forensics-Compliant-Classrooms/dp/B08XH5ZG71/ref=sr_1_5?dchild=1&keywords=carbon+dioxide+alarm&qid=1631576023&sr=8-5
GZAIR/Amazon
https://www.amazon.com/GZAIR-Temperature-Humidity-Tabletop-Mountable/dp/B0986Y7PVS/ref=sr_1_8?dchild=1&keywords=carbon+dioxide+alarm&qid=1631576023&sr=8-8#customerReviews
Langkou Store/Amazon
https://www.amazon.com/Detector-Pollution-Temperature-Humidity-Professional/dp/B08Q7YMX9X/ref=sr_1_14?dchild=1&keywords=carbon+dioxide+alarm&qid=1631576023&sr=8-14
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/NWFSC-OA-lab/asw/issues/8#issuecomment-918678409, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AEOOQSIFLJP677WUJZTI75DUB2HLZANCNFSM5DMGGP6A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

I didn't see anything about OSHA, or really any approval. The one from GSA might be good if it makes things easy for Kathleen. Do we care about wall vs battery powered, or connecting to Wifi? Or do we really just want something that's going to beep really loud if we go above a certain CO2 level?

mainly beeping. Which one is gsa?
On Tue, Sep 14, 2021 at 1:07 PM dmperez89 @.***> wrote:

I didn't see anything about OSHA, or really any approval. The one from GSA
might be good if it makes things easy for Kathleen. Do we care about wall
vs battery powered, or connecting to Wifi? Or do we really just want
something that's going to beep really loud if we go above a certain CO2
level?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/NWFSC-OA-lab/asw/issues/8#issuecomment-919475432, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AEOOQSKFRBKUQTLFJHW7ER3UB6TN5ANCNFSM5DMGGP6A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Extech on GSA
Apparently it's on sale! But only until September 30

I like that it has a data logger. Too bad we will miss the sale!
-paul
On Tue, Sep 14, 2021 at 1:26 PM dmperez89 @.***> wrote:

Extech on GSA
https://www.gsaadvantage.gov/advantage/ws/catalog/product_detail?gsin=11000038532310
Apparently it's on sale! But only until September 30
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/NWFSC-OA-lab/asw/issues/8#issuecomment-919487646, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AEOOQSIYQ75YTYIA2RBDBYDUB6VVVANCNFSM5DMGGP6A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Sounds good, I'll put in a request to Kathleen

Can't make any purchases until we can spend money again in October. It is
good to have found an option that will work so we are read to go when the
funds arrive.
On Tue, Sep 14, 2021 at 2:25 PM dmperez89 @.***> wrote:

Sounds good, I'll put in a request to Kathleen
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/NWFSC-OA-lab/asw/issues/8#issuecomment-919524343, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AEOOQSIV225FWJQN3EINMVDUB64TBANCNFSM5DMGGP6A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Google Doc purchase request for Kathleen
Ocean Acidification Drive -> 06. OA Lab Logistics -> 01. Purchase Orders, Receipts, & Packing Slips -> Purchase Orders

installed


Update duplicate packages insight

close: https://github.com/relative-ci/bundle-stats/issues/2831

closed in favor of https://github.com/relative-ci/bundle-stats/pull/2933


[incubator][VC] fix register compile error

Signed-off-by: zhuangqh<EMAIL_ADDRESS>

/lgtm
/approve


datePicker autoOk auto accepts date when picking year

Hi,
I recently upgraded to 3.2.9 version and was facing a problem with the DatePicker :
In previous version, when using DatePicker with autook and views in static container, I was able to pick the year, then the month, and only when I was picking the day, the date was accepted.
In new version, onAccept is triggered as soon as I pick a year in year view (and the month view is briefly appearing, before my custom dialog closes because of acceptation)
Here is the relevant config of my date picker (at least, I think...) :
<DatePicker
  variant="static"
  openTo="date"
  views={['year', 'month', 'date']}
  autoOk
  value={this.state.date}
  onChange={this.handleChange.bind(this)}
  onAccept={this.handleAccept.bind(this)}
  />

I was able to recover the previous behavior by downgrading to 3.2.0 and wonder if my problem is related to this commit : https://github.com/mui-org/material-ui-pickers/commit/41fa5373397b5a6b01a690d9db5b861ca6be4d1a
Is there a new way to achieve this in latest versions (autoOk while enabling ability to choose by year, then month, then day) ?
Thanks in advance for your help

This was also fixed in v4, checkout this demo https://next.material-ui-pickers.dev/demo/datepicker#responsiveness


Trying to get in touch regarding a security issue

Hey there!
I belong to an open source security research community, and a member (@haxatron) has found an issue, but doesn’t know the best way to disclose it.
If not a hassle, might you kindly add a SECURITY.md file with an email, or another contact method? GitHub recommends this best practice to ensure security issues are responsibly disclosed, and it would serve as a simple instruction for security researchers in the future.
Thank you for your consideration, and I look forward to hearing from you!
(cc @huntr-helper)

Report is located here https://huntr.dev/bounties/50996581-c08e-4eed-a90e-c0bac082679c (only maintainers with write permissions can view)
This issue is different from the known issues listed here https://github.com/shelljs/shelljs/wiki/Security-guidelines

Thanks for reaching out. I sent an email to @Haxatron earlier today. I'll review this report over the next few days and decide on next steps (if any).

Thanks for the report. I believe this is a valid issue in ShellJS, so I've created a fix and pushed a release.

The patch for this is https://github.com/shelljs/shelljs/pull/1060. That PR landed on tip-of-tree (master branch), however I also cherrypicked this onto the 0.8-release branch.
This fix was released in 0.8.5. The patch for this bug is the only change between 0.8.4 and 0.8.5.
I'm trying to figure out a nice way to edit the CHANGELOG.md to explain what went into these two versions, but this doesn't work well with the tool we use to automate changelog generation. I'll need to think a bit more about how to mix manual edits with autogenerated changelog.
I added a SECURITY.md in https://github.com/shelljs/shelljs/pull/1061. For future reference, the best way to get in contact with me for security issues is with the email address in that page.

If you believe this patch is insufficient, please let me know privately via email and I'll gladly investigate further.

@nfischer - thank you for such an in-depth follow-up of the report.
Would it be possible to confirm the fix commit SHA against the report?
You can also bag a bounty for your work as well! ❤

The report should be public now. I recommend folks upgrade to ShellJS 0.8.5 to ensure you have the fix.
This bug only impacts the synchronous version of shell.exec(). All other ShellJS methods (including the async usage of shell.exec()) should not be impacted, however it's of course perfectly safe to update ShellJS anyway.


ARTEMIS-1188: Update sevntu to 1.24.0 available in maven central

remove custom repo
update groupid to match artifact in maven central.
bump version also to that now deployed to maven central.
bump checkstyle version to 7.7 to make compatible.
updated checkstyle.xml to ignore existing issues which are prolific
which are now flagged in latest version as some bugs in previous meant they we'ren't detected e.g. checkstyle/checkstyle#3320
fixing some violations which are not too prolific.

FYI - I'm happy to sort prolific check style violations now picked up on upgrade of checkstyle to latest (key ones are redundant final modifiers and also do while loop formatting) , but will do this later, key to this is just to get sevntu updated to the maven central version, so builds are more stable :)

I see... hash without equals

@clebertsuconic yeah, and also the other non-prolific one was braces for try catch.
Im just working on the prolific ones, almost done actually.


Error with Textfield when try input

When iam try to input text to textfield application return to last page on android emulator device but on ios emulator is working normally
https://user-images.githubusercontent.com/101683175/201195452-47b6825b-f60b-4264-8a9f-908983061a06.mp4

Hi @salahaldinalhajduzce,
Can you please provide your flutter doctor -v and a minimal complete reproducible code sample?
Thank you


Sprite picking support

Support picking non-meshes. This is probably just adding an extensible API to #3.

Would be really nice to have 2D support and be able to pick sprites.

I think the only way to do this will be with a color picking shader, which is still way over my head. Because sprites don't have a mesh that match the shape of the underlying sprite, I don't think there is any way to use the current ray casting method that won't have problems with alpha and z-ordering. That said, the color picking shader is really the last big thing I want to tackle.

I did not take a look at the code, but would the approach used for the current bevy ui be feasible? It can correlate the mouse cursor / screen coordinates to the sprite of a button. I'm just thinking out loud and poking around in the dark...so ¯\_(ツ)_/¯

This is now working, to some extent, with 2d Ortho cameras. However this still only works with actual meshes.

This is now working, to some extent, with 2d Ortho cameras. However this still only works with actual meshes.

Hi there!
Not sure because I just landed in the Bevy's community but I think this issue may be related to my use case.
I am using bevy_prototype_lyon to create a tile bundle:
#[derive(Bundle, Default)]
pub struct TileBundle {
    pub position: XY<f32>,
    pub walkable: Walkable
    #[bundle]
    sprite: ShapeBundle
}

impl TileBundle {
    pub fn new(position: XY<f32>, walkable: Walkable) -> TileBundle {
        let tile_size = 108.;
        let tile_half_width = tile_size.clone() / 2.;
        let tile_half_height = &tile_half_width / 2.;

        let transform = Transform::from_xyz(
            (position.clone().x - position.clone().y) * &tile_half_width,
            (position.clone().x + position.clone().y) * &tile_half_height,
            0.
        );

        TileBundle {
            position,
            walkable,
            sprite: ShapeBundle {
                path: {
                    let mut path = PathBuilder::new();

                    path.move_to(Vec2::new(tile_half_width.clone(), 0.));
                    path.line_to(Vec2::new(tile_size.clone(), tile_half_height.clone()));
                    path.line_to(Vec2::new(tile_half_width.clone(), tile_half_width.clone()));
                    path.line_to(Vec2::new(0., tile_half_height.clone()));
                    path.close();
                    path.build()
                },
                transform,
                colors: ShapeColors {
                    main: Color::RED,
                    outline: Color::rgba(255.,255.,255.,0.5)
                },
                mode: DrawMode::Fill(Default::default()),
                visible: Visible::default(),
                ..Default::default()
            }
        }
    }
}

But when I insert the PickableBundle, it throws me a panic:

thread 'Compute Task Pool (0)' panicked at 'Mesh does not contain vertex positions', /home/maz/.cargo/registry/src/github.com-1ecc6299db9ec823/bevy_mod_raycast-0.2.2/src/lib.rs:361:41
note: run with RUST_BACKTRACE=1 environment variable to display a backtrace
thread 'main' panicked at 'task has failed', /home/maz/.cargo/registry/src/github.com-1ecc6299db9ec823/async-task-4.0.3/src/task.rs:368:45
Segmentation fault (core dumped)

I just wanted to know if my issue was related to this one? Just to stop to hit my head on a wall haha :)
I would love to help but this is definitely out of my knowledge so all I can ask is the basic "any ETA on this one?"
Thanks :)

If lyon is generating meshes, it should work. Note this is unrelated to sprites, as far as I can tell. :)
The error you see comes from the mesh not having the Mesh::ATTRIBUTE_POSITION attribute. It's possible this is happening because the mesh is empty? My first guess would be to try adding the PickableBundle after the mesh is initialized. If that works, we can make a small update to bevy_mod_raycast.
If you follow up with this, could you post it in an issue over here: https://github.com/aevyrie/bevy_mod_raycast/issues
Thanks!

Thanks for the details, I don't fully understand all of that stuff works atm.
I already had the advice to add PickableBundler after the mesh is initialized, this didn't solve the problem:
impl Plugin for GridPlugin {
    fn build(&self, app: &mut AppBuilder) {
        app
            .add_plugin(ShapePlugin)
            .add_startup_system(setup.system())
            .add_stage("GridStage", SystemStage::parallel().with_system(add_pickable_bundle.system()));
    }
}

fn setup(mut commands: Commands) {
    commands.spawn_bundle(TileGridBundle::default())
        .with_children(|parent| {
            for (x, &row) in GridPlugin::GRID_MATRIX.iter().enumerate() {
                for (y, &walkable) in row.iter().enumerate() {
                    if walkable == 1 {
                        parent.spawn_bundle(TileBundle::new(
                            XY { x: x as f32, y: y as f32},
                            Default::default()
                        ));
                        println!("Spawn {};{}", x, y)
                    }
                }
            }
        });
}

fn add_pickable_bundle(mut commands:Commands, query:Query<Entity,(With<Walkable>,With<Uninitiated>)>) {
    println!("init pickable");
    for entity in query.iter(){
        commands.entity(entity).insert_bundle(PickableBundle::default()).remove::<Uninitiated>();
    }
}

The console confirm that I'm adding the PickableBundle after all my TileBundles are added.
The problem may come from the lyon mod not adding the meshes at the place that the picking mod is looking?
Here is the ShapeBundle struct, if you have see some stuff that could be modified to be compatible, let me know I'll make a PR into the lyon repo.

I attempted this as well, expecting this to work:
use bevy::prelude::*;
use bevy::sprite::MaterialMesh2dBundle;
use bevy_mod_picking::{DefaultPickingPlugins, PickableBundle, PickingCameraBundle, PickingEvent};

fn main() {
    App::new()
        .add_plugins(DefaultPlugins)
        .add_plugins(DefaultPickingPlugins)
        .add_startup_system(setup)
        .add_system_to_stage(CoreStage::PostUpdate, print_events)
        .run();
}

pub fn print_events(mut events: EventReader<PickingEvent>) {
    for event in events.iter() {
        match event {
            PickingEvent::Selection(e) => info!("A selection event happened: {:?}", e),
            PickingEvent::Hover(e) => info!("Egads! A hover event!? {:?}", e),
            PickingEvent::Clicked(e) => info!("Gee Willikers, it's a click! {:?}", e),
        }
    }
}

fn setup(
    mut commands: Commands,
    mut meshes: ResMut<Assets<Mesh>>,
    mut materials: ResMut<Assets<ColorMaterial>>,
) {
    commands
        .spawn_bundle(MaterialMesh2dBundle {
            mesh: meshes.add(Mesh::from(shape::Quad::default())).into(),
            transform: Transform::default().with_scale(Vec3::splat(128.)),
            material: materials.add(ColorMaterial::from(Color::PURPLE)),
            ..Default::default()
        })
        .insert_bundle(PickableBundle::default());
    commands
        .spawn_bundle(OrthographicCameraBundle::new_2d())
        .insert_bundle(PickingCameraBundle::default());
}

But no such luck, currently debugging to figure out why.

Note that lyon support has been enabled upstream by https://github.com/aevyrie/bevy_mod_raycast/pull/38. The tracking issue for 2d meshes (not sprites) for mod_picking is here: https://github.com/aevyrie/bevy_mod_picking/issues/130

Is there any WIP on this, or anyone have an idea how to tackle it? I'm digging around in bevy_sprite but I'm kinda new to bevy, so not exactly sure how to approach this.
Do sprites somehow get their quads generated that aren't assigned as a mesh component?

I'm working on a refactor right now that should make it much easier to do this. There is no ETA, though I'm hoping to get this library upstreamed for bevy 0.9.

Note that there is now a sprite backend in the beta branch: https://github.com/aevyrie/bevy_mod_picking/tree/beta
This will only check the bounds of the sprite, there isn't really a good way to check the alpha at a specific pixel in an texture on the CPU. This should work with alpha channel once we have GPU picking support in-engine: https://github.com/bevyengine/bevy/pull/6991

I'm getting some issues with the sprite backend in the beta branch, it seams to only work with one sprite.
I modified the example for the sprite backend while testing, I imagine I am doing this as intended:
main.rs
As a workaround if you clone the beta branch and comment out backends/bevy_picking_sprite/src/lib.rs Line 60 to disable the focus policy stuff. I imagine this breaks other behaviour though and is in no way a permanent fix but it works for my use case.
60: // blocked = focus != Some(&FocusPolicy::Pass);

Thanks for the heads up. I'll take a look.

I think this could be closed as the sprite picking works fine in the currently published version?


Allow to instrument client pool sizes

LGTM, thanks.
Let me know if you find that default limit of 10 connections is not enough...

...and feel free to merge this and release new version :)

Closing in favour of https://github.com/go-redis/redis/pull/241


a

create a custom source code generator for definitions, that creates a globals using redirect for Spr and other Enums, to a custom class, we create at runtime casting the enum values referenced to Enum.Parse<>() and handles casting thus allowing us to rewrite the code.

https://martinjt.me/2022/05/03/evil-monkeypatching-in-c-with-rosyln-source-generators/


Feature: add a phone number field to onboarding and profile settings

Admins need information about volunteer phone numbers to keep track of people and fill roles in the event of cancellations.
Subtasks:

[ ] Update the database User model to track phone numbers
[ ] Make a reusable input schema for phone number validation
[ ] Integrate input fields for phone numbers into the signup process
[ ] Update the admin users table to display phone numbers
[ ] Update volunteer info cards in the event management routes to display phone numbers

@parkerdavis1 , as we've discussed, there ends up being a number of different parts of this task, and there may be more unanticipated complexity. Please feel feel to break work on this into different pull requests at your discretion

All of these tasks have now been completed with the following PRs
https://github.com/opportunity-hack/evs/pull/34
https://github.com/opportunity-hack/evs/pull/35
https://github.com/opportunity-hack/evs/pull/37


[enhancement] Implement API for Search Autocomplete, Suggestion

Let figure out the design of API in terms of Input & Output before we look at implementation

@jitendravarma this reference might help


Update version number

Coverage remained the same at 94.945% when pulling 27265f40ae05d40b1c964730f9be8d364c7c4433 on ConorGriffin37:patch-1 into dcbeb36e3470f82e57477201fbd4ae081ad60034 on Roblox:master.


Checkout tiny button incorrectly rendered

The checkout tiny pill/rect button is incorrectly rendered in
Firefox and Opera.

Can you re-check this with the latest changes? Looks good in firefox to me. Haven't checked opera yet.

Opera is fine.
Used http://bluesuncorp.co.uk/icv4/allbutton.htm
to check with latest changes.


TCP2: Handling of RST flag from server makes poll() call unable to return indefinitely

Describe the bug
The poll()/zsock_poll() call hangs when a TCP message with a RST flag is received.
I have looked into the TCP2 code and added some debug statements to determine whether the message is passed to the TCP2 stack, which is not the case. This same was done to the zsock_poll() call where I can see that the z_pend_curr() in z_impl_k_poll() doesn't return.
To Reproduce
Run the supplied code example
tcp_rst.zip
Expected behavior
I expect the TCP message to be handled by the TCP2 code and the poll() call to return with a POLLHUP event on my socket.
Impact
Showstopper as it completely blocks the thread.
Logs and console output

Text log from zephyr OS
*** Booting Zephyr OS build v2.4.0-ncs1  ***
TCP_RST problem code started
[00:00:00.038,757] <err> eth_ksz8851: Memory self test: 0x1010
[00:00:00.040,222] <err> eth_ksz8851: KSZ8851 driver thread id: 0x200107f8
[00:00:00.040,496] <inf> eth_ksz8851: KSZ8851 Initialized
[00:00:00.040,679] <dbg> net_conn.conn_register_debug: (0x20011bf8): [0x200124b4/17/1/0x1d] remote -/67
[00:00:00.040,710] <dbg> net_conn.conn_register_debug: (0x20011bf8):   local <IP_ADDRESS>/68 cb 0x00015155 ud 0x00000000
[00:00:00.093,322] <err> module_Tcp: Tcp thread id: 0x200103b8
[00:00:00.093,353] <dbg> module_Tcp.TcpInit: TcpInit start
[00:00:00.093,383] <inf> module_Tcp: Pointer to iface ETH_0 0x20010348
[00:00:00.093,475] <dbg> module_Tcp.NetworkMgmtHandler: callback_handler start
[00:00:00.093,475] <err> module_Tcp: Received Network management event:<PHONE_NUMBER>
[00:00:00.093,505] <err> module_Tcp: Network management event: IPV4_ADDR_ADD
[00:00:00.093,688] <err> module_Tcp: Interface 0x20010348: ip=<IP_ADDRESS>, netmask=<IP_ADDRESS>, gateway=<IP_ADDRESS>
[00:00:00.093,688] <dbg> module_Tcp.NetworkMgmtHandler: callback_handler end
[00:00:00.093,719] <dbg> module_Tcp.TcpInit: TcpInit end
[00:00:02.170,104] <inf> eth_ksz8851: Link status changed
[00:00:02.170,166] <inf> eth_ksz8851: Port Status: status link up 1, an done 1, speed: 100, duplex: half
[00:00:02.170,166] <inf> eth_ksz8851: Partner Capability: 10BT half 0, 10BT full 0, 100BT half 4, 100BT full 0
[00:00:02.170,196] <err> eth_ksz8851: Clear Wake-Up Event Indication
[00:00:10.093,811] <dbg> module_Tcp.TcpOpenSocket: NetworkingSocketOpen start
[00:00:11.139,953] <dbg> net_tcp.tcp_conn_ref: (0x200103b8): conn: 0x20024890, ref_count: 1
[00:00:11.139,984] <dbg> net_tcp.tcp_conn_alloc: (0x200103b8): conn: 0x20024890
[00:00:11.186,920] <dbg> net_sock.zsock_socket_internal: (0x200103b8): socket: ctx=0x20012190, fd=0
[00:00:11.232,940] <dbg> net_tcp.net_tcp_connect: (0x200103b8): context: 0x20012190, local: <IP_ADDRESS>, remote: <IP_ADDRESS>
[00:00:11.233,032] <dbg> net_tcp.net_tcp_connect: (0x200103b8): conn: 0x20024890 src: <IP_ADDRESS>, dst: <IP_ADDRESS>
[00:00:11.233,154] <dbg> net_conn.conn_register_debug: (0x200103b8): [0x20012494/6/1/0x3f] remote <IP_ADDRESS>/4433
[00:00:11.233,215] <dbg> net_conn.conn_register_debug: (0x200103b8):   local <IP_ADDRESS>/43581 cb 0x00017d99 ud 0x20012190
[00:00:11.233,215] <wrn> net_tcp: Received flags:0xffffffff, and len=-1|-1
[00:00:11.233,306] <dbg> net_tcp.tcp_in: (0x200103b8):  [LISTEN Seq=2183506434 Ack=0]
[00:00:11.233,520] <dbg> net_tcp.tcp_out_ext: (0x200103b8): SYN Seq=2183506434 Len=0
[00:00:11.233,612] <dbg> net_tcp.tcp_send_process_no_lock: (0x200103b8): SYN Seq=2183506434 Len=0
[00:00:11.233,795] <dbg> net_tcp.tcp_send: (0x200103b8): SYN Seq=2183506434 Len=0
[00:00:11.234,558] <err> eth_ksz8851: Packet (0x20023908) transmitted. bytes: 42 (2006)
[00:00:11.234,741] <err> eth_ksz8851: Finished transmit routine
[00:00:11.235,076] <inf> eth_ksz8851: Transmitted frame. TXQ space:(6144)
[00:00:11.235,809] <dbg> net_tcp.tcp_in: (0x200103b8): LISTEN->SYN_SENT
[00:00:11.236,877] <err> eth_ksz8851: Packet (0x20023950) transmitted. bytes: 54 (2006)
[00:00:11.237,060] <err> eth_ksz8851: Finished transmit routine
[00:00:11.237,396] <inf> eth_ksz8851: Transmitted frame. TXQ space:(6144)
[00:00:11.238,616] <dbg> net_ipv4.net_ipv4_input: (0x200119a0): IPv4 packet received from <IP_ADDRESS> to <IP_ADDRESS>
[00:00:11.238,677] <dbg> net_conn.net_conn_input: (0x200119a0): Check TCP listener for pkt 0x200235f0 src port 4433 dst port 43581 family 1
[00:00:11.238,677] <dbg> net_conn.net_conn_input: (0x200119a0): [0x20012494] match found cb 0x00017d99 ud 0x20012190 rank 0x3f
[00:00:11.238,739] <wrn> net_tcp: Received flags:0x12, and len=44|0
[00:00:11.238,952] <dbg> net_tcp.tcp_in: (0x200119a0): SYN,ACK Seq=1762929497 Ack=2183506435 Len=0 [SYN_SENT Seq=2183506435 Ack=0]
[00:00:11.238,952] <dbg> net_tcp.tcp_options_check: (0x200119a0): len=4
[00:00:11.238,983] <dbg> net_tcp.tcp_options_check: (0x200119a0): opt: 2, opt_len: 4
[00:00:11.238,983] <dbg> net_tcp.tcp_options_check: (0x200119a0): MSS=1460
[00:00:11.239,013] <dbg> net_tcp.tcp_in: (0x200119a0): Lowering send window from 14600 to 1536
[00:00:11.239,105] <dbg> net_tcp.tcp_send_timer_cancel: (0x200119a0): SYN Seq=2183506434 Len=0
[00:00:11.239,379] <dbg> net_tcp.tcp_out_ext: (0x200119a0): ACK Seq=2183506435 Ack=1762929498 Len=0
[00:00:11.239,501] <dbg> net_tcp.tcp_send_process_no_lock: (0x200119a0): ACK Seq=2183506435 Ack=1762929498 Len=0
[00:00:11.239,654] <dbg> net_tcp.tcp_send: (0x200119a0): ACK Seq=2183506435 Ack=1762929498 Len=0
[00:00:11.239,685] <dbg> net_tcp.tcp_in: (0x200119a0): SYN_SENT->ESTABLISHED
[00:00:11.239,715] <err> net_tcp: TCP_ESTABLISHED flags=0x00
[00:00:11.240,203] <dbg> net_tcp.net_tcp_connect: (0x200103b8): conn: 0x20024890, ret=0
[00:00:11.240,386] <err> eth_ksz8851: Packet (0x20023998) transmitted. bytes: 54 (2006)
[00:00:11.240,570] <err> eth_ksz8851: Finished transmit routine
[00:00:11.235,076] <inf> eth_ksz8851: Transmitted frame.
Using the Nordic SDK v1.4.0

I just fixed one RST packet handling in commit b519ba54f5ddfc51ddbc24d8264fa1dde5186d2d. Could you try to revert that commit and report did it help?

The RST handling patch introduced in #29835 was missing unit tests. The tests can be found in #30142

I have updated to #30142 and the problem remains. I did some additional testing where i discovered the following:
Calling poll() after the socket connection was established, results in the thread blocking on poll() while the RST message is never passed from the ethernet driver to the upper layers.
Stop calling poll after the connection was established, results in the RST message to be passed to the upper layers and the TCP connection is closed.
So maybe the fault doesn;t occur in the TCP2 code?
[00:01:15.462,646] <inf> eth_ksz8851: RX interrupt with framecount: 1
[00:01:15.463,806] <err> eth_ksz8851: read_fifo: length=64
[00:01:15.464,355] <inf> eth_ksz8851: Forwarded packet to higher layer: 64
[00:01:15.464,752] <dbg> net_ipv4.net_ipv4_input: (rx_q[0]): IPv4 packet received from <IP_ADDRESS> to <IP_ADDRESS>
[00:01:15.464,813] <dbg> net_conn.net_conn_input: (rx_q[0]): Check TCP listener for pkt 0x20026090 src port 4433 dst port 56724 family 1
[00:01:15.464,874] <dbg> net_conn.net_conn_input: (rx_q[0]): [0x20014610] match found cb 0x0001ca71 ud 0x2001430c rank 0x3f
[00:01:15.465,209] <dbg> net_tcp.tcp_in: (rx_q[0]): ACK,RST Seq=3791526107 Ack=4097630127 Len=0 [ESTABLISHED Seq=4097630127 Ack=3791526107]
[00:01:15.465,240] <dbg> net_tcp.tcp_in: (rx_q[0]): ESTABLISHED->CLOSED
[00:01:15.465,362] <dbg> net_tcp.tcp_conn_unref: (rx_q[0]): conn: 0x20029730, ref_count=1
[00:01:15.465,393] <dbg> net_conn.net_conn_unregister: (rx_q[0]): Connection handler 0x20014610 removed
[00:01:15.465,423] <dbg> net_sock.zsock_received_cb: (rx_q[0]): ctx=0x2001430c, pkt=0x00000000, st=-54, user_data=0x00000000
[00:01:15.465,454] <dbg> net_sock.zsock_received_cb: (rx_q[0]): Marked socket 0x2001430c as peer-closed

By giving the logging thread a higher priority to my thread calling pol() I figured out what my thread was doing. Seems that I misunderstood the behavior of the thread. The RST flag is properly received by the TCP2 code regardless of whether I'm calling poll().
When the connection is closed the poll() call keeps returning with a POLLIN event on the socket and the recv() call returns 0 bytes.
Is this how the poll() call should return when the connection is closed?

When the connection is closed the poll() call keeps returning with a POLLIN event on the socket and the recv() call returns 0 bytes.
Is this how the poll() call should return when the connection is closed?

Yes, if the connection is closed, any read will return 0 bytes. This is the normal BSD socket API convention.

Ok, thanks for your help.


[FR] Mixing compiled NNF formulas with bauhaus statements

Willingness to contribute
The Bauhaus Community encourages new feature contributions. Would you be willing to contribute an implementation of this feature?

[ ] Yes. I can contribute this feature independently.
[ ] Yes. I would be willing to contribute this feature with guidance from the Bauhaus community.
[x] No. I cannot contribute this feature at this time.

Proposal Summary
Ability to use compiled and simplified NNF statements in future constraints.
Motivation
I attempted to use bauhaus in an iterative algorithm, where the formulas needed to be compiled to NNF and simplified each time for use in the next iteration. However, I would run into the issue in the next iteration where I would need to combine a formula with bauhaus propositions with a compiled NNF formula, and as it stands you can only add constraints using bauhaus propositions. Below is a simple example of the error:
a = Bauhaus("a")
b = Bauhaus("b")
e.add_constraint(a | b)
# get NNF
f = e.compile()
# attempt to use it again, results in the error: 'And' object has no attribute 'compile'
e.add_constraint(f & (a | b))
e = e.compile()
Describe the solution you'd like
Ideally, the functionality to use either NNF statements or statements with bauhaus propositions (or both) as needed when adding constraints.
Why is it currently difficult to achieve this with Bauhaus?
Bauhaus currently only allows the user to add constraints using statements with bauhaus propositions.

Not exactly sure what the right solution is, but do agree that an issue exists for iterative processing. It's more than just adding constraints dynamically, as this would kind of do the same as what's suggested above (but work now):
a = Bauhaus("a")
b = Bauhaus("b")
e.add_constraint(a | b)
e.add_constraint(a | b)
e = e.compile()

I mean, the example on the line e.add_constraint(f & (a | b)) makes it look a little odd (same constraint twice), but the point is you can iteratively add constraints. Where it runs into trouble is when we want to compile, processing, then augment. Specifically, if you just keep combining things, the theory blows up by the end. So you need to simplify at every iteration:
a = Bauhaus("a")
b = Bauhaus("b")
e.add_constraint(a | b)
theory = e.compile()
theory = theory.simplify()
# ?? add some more constraints to theory using bauhaus ??


Duplicated tensorboard output

I'm seeing duplicate Tensorboard runs for any guild run that happened as part of a batch. What seems to be happening is that guild is creating symlinks to each individual run folder, as well as a symlink to the parent (batch) folder, which in turn has symlinks to each of the individual runs. Given that the direct links to individual runs have a better name (it includes the run label), perhaps the thing to do is to stop adding symlinks to batch folders for Tensorboard?
Happy to provide more details of my setup if this is hard to reproduce.

Thank you, yes you're right! This is fixed and available to test in the 0.6.5.dev6 pre-release. You can install that version by running pip install guildai --upgrade --pre. Otherwise 0.6.5 will go out later this week.

This seems to still be happening in 0.6.5, including for runs performed after I upgraded.

In my tests on 0.6.5, I'm only seeing the non-batch runs. Just to confirm we're seeing the same thing (at least as a control), could you clone Guild examples and run this?
$ cd examples/noisy
$ guild run noisy.py x=[1,2,3] -y
$ guild tensorboard

On my environment, I'm seeing only the three noisy.py runs - not the batch run. So the values reported in TensorBoard are not duplicated.
For this test, there is the matter of ensuring that you're not looking at any other runs by accident. You can filter using guild tensorboard -o noisy.py but that could spoil the test. The safest way would be to either run this test in a new virtual env or to temporarily move existing runs into an archive before running the code above. I've provided steps for either approach below.
Option 1: Create a virtual env:
$ virtualenv /tmp/guild-issue-32
$ source /tmp/guild-issue-32/bin/activate
$ pip install guildai
<run test steps above>
$ deactivate && rm -rf /tmp/guild-issue-32

Option 2: Temporarily move existing runs:
$ guild export --move guild-issue-32-archive
<run test steps above>
$ guild import --move guild-issue-32-archive
$ rmdir guild-issue-32-archive

I created a project to help recreate the current behavior:
https://github.com/guildai/issues/tree/master/issue-32
I'm not able to see batch runs anymore in TensorBoard as of 0.6.5. If you're seeing something different, please update here so we can investigate!

I see what's going on here - I'm only seeing duplicated runs when i go to TensorBoard through guild view, not directly. So the issues script provided works as expected, but if you change the last line to guild view instead then click through to TensorBoard you still see the duplicates. Not sure how you want to handle this, for my purposes at least I'm happy with using guild tensorboard directly so I don't see further action as essential.

Still an issue in 0.7 :(
Will move to high priority to fix after 0.7 release.
Screeshot:

I'm not seeing this anymore in 0.7.2. I'm going close. We can reopen if I've missed something.
To create I ran the recreate script in https://github.com/guildai/issue-resolution/tree/master/32-duplicated-tensorboard-output.

I'm not seeing this anymore in 0.7.2. I'm going close. We can reopen if I've missed something.
To create I ran the recreate script in https://github.com/guildai/issue-resolution/tree/master/32-duplicated-tensorboard-output.


[Enhancement]Add qianfan Auth

添加千帆的鉴权，
测试（api type均为qianfan）均已通过
1、环境变量设置ak sk
2、直接传参ak sk
3、环境变量设置access_token
4、直接传参access_token
5、环境变量设置ak sk access_token
麻烦review～ @sijunhe

Codecov Report
Attention: 18 lines in your changes are missing coverage. Please review.

Comparison is base (8937548) 64.77% compared to head (24d9caa) 64.40%.

:exclamation: Current head 24d9caa differs from pull request most recent head dd74c08. Consider uploading reports for the commit dd74c08 to get more accurate results

Files
Patch %
Lines

...t-agent/src/erniebot_agent/chat_models/erniebot.py
5.55%
17 Missing :warning:

...nt/src/erniebot_agent/utils/config_from_environ.py
75.00%
1 Missing :warning:

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #211      +/-   ##
===========================================
- Coverage    64.77%   64.40%   -0.37%
===========================================
  Files           57       57
  Lines         2947     2967      +20
===========================================
+ Hits          1909     1911       +2
- Misses        1038     1056      +18

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

修复如果环境变量设置了access token但同时传入ak sk的报错


feat(section-Image-flush-bottom ): add image-flush-bottom section.

Issue
Fixes #MERATIVE-782
Description
To create a new section and variations for this called "Image band - flush bottom"
New

Design Specs

If applicable, add the direct link to the design specs of the component/feature that's part of this PR.

Figma Link - https://www.figma.com/file/vw24IPXXt4vCh9BNumka9A/Web--Merative-Digital-Design-System-2.0?type=design&node-id=1459-19541&mode=design&t=moJo2KW9FFraRerT-0

Test URLs

Before (Changes from main): https://main--merative2--nimithshetty17.hlx.page/block-library/sections/section-image-band-flush-bottom
After (Changes from this PR): https://section-image-flush-bottom--merative2--nimithshetty17.hlx.page/block-library/sections/section-image-band-flush-bottom

Testing Instruction
To test the section image flush bottom with brand logo and its responsiveness with respect to the design across all devices.

HI,

Image outline is missing in all breakpoints except the gradient one.

REst is good to go.
@proeung @nimithshetty17 @keith-kaplan @Shalini-SB

@sachinmesh - Please check now.

@proeung - have done the changes for the same.


Support for deleting EC2 Mac Instances in AWS

Can you add support for deleting EC2 Mac Instances in AWS?

@mvinkler Sounds interesting!
Just looked at the link you shared in the doc. With the first look I believe there is no vital difference between a non-mac instance (VM) and Mac instance (VM).
So the current implementation for removing the EC2 instance/VM from AWS should be good enough. It is just that you need to remember the instance prepended and the running time should match the cleanup criteria from https://github.com/RedHatQE/cloudwash/blob/master/settings.yaml.template settings file.

Hi @jyejare,
it's actually not that simple.
Process for running Mac instances:
aws ec2 allocate-hosts allocates HW machine (Mac Mini).
aws ec2 run-instances installs selected macOS on the Mac Mini.
Process for teardown:
aws ec2 terminate-instances deletes the macOS from the Mac Mini
aws ec2 release-hosts releases the HW machine.

HW machine cannot be released if there is a running instance bound to it.
there is a 24-hour minimum allocation period to comply with the Apple macOS Software License Agreement
At the end of the 24-hour minimum allocation period, the host can be released at any time (provided there is no running instance bound to it)


[Bug]: Why explicitly remove MetaMask?

Current Behavior
When a user has multiple injected web3 providers(e.g. {CoinbaseWallet, MetaMask}), web3-onboard explicitly hides the MetaMask option. Why was this logic implemented? Why hide MetaMask, specifically? Some of our users have MetaMask, plus another injected wallet, and want the ability to use MM.
https://github.com/blocknative/web3-onboard/blob/be9e6a6819fd623038cb0d6300391ec72acb85ef/packages/injected/src/index.ts#L67
Expected Behavior
No response
Steps To Reproduce
No response
Onboard Version
2
Node Version
14.7.0
What browsers are you seeing the problem on?
No response
Relevant log output
No response
Anything else?
No response

That logic is there to remove MetaMask in the case that a provider has the isMetaMask property set to true, but is not MetaMask.
In the case that you are referring to, when a user has two extensions installed that both sit on the window.ethereum namespace, one will overwrite the other, making the first extension (usually MM) unavailable to the app. In the case of Coinbase wallet they keep the reference to the MM provider and add it to a providers property. We are working on an enhancement that checks that parameter and includes all providers that are included in that array. We have a working branch here that we are working on testing further before releasing.

Of course our users just so happened to have found the one pair that does work right :)
Thanks for the detailer. Will watch out for the next releases...

That logic is there to remove MetaMask in the case that a provider has the isMetaMask property set to true, but is not MetaMask.
In the case that you are referring to, when a user has two extensions installed that both sit on the window.ethereum namespace, one will overwrite the other, making the first extension (usually MM) unavailable to the app. In the case of Coinbase wallet they keep the reference to the MM provider and add it to a providers property. We are working on an enhancement that checks that parameter and includes all providers that are included in that array. We have a working branch here that we are working on testing further before releasing.


Add integration tests

This introduces a fake server that can be replaced by an actual server if the environment variables TSURU_TEST_HOST and TSURU_TEST_TOKEN. The commands in the Makefile ensure that all resources required by the integration tests exist in the tsuru API.
How to test this PR:

make sure you have latest version of tsuru-client (brew install tsuru/tsuru/tsuru, apt-get install tsuru-client or checkout and build)
run tsuru install
run TSURU_TEST_HOST=http://<IP_ADDRESS>:8080 TSURU_TEST_TOKEN=$(tsuru token-show | awk '{print $3}') make integration

Example:
% TSURU_TEST_HOST=http://<IP_ADDRESS>:8080 TSURU_TEST_TOKEN=$(tsuru token-show | awk '{print $3}') make integration
TSURU_TARGET="${TSURU_TEST_HOST}" TSURU_TOKEN="${TSURU_TEST_TOKEN}" tsuru-admin platform-add python || tsuru-admin platform-update python
Error: Duplicate platform
Step 1 : FROM tsuru/python
Pulling from tsuru/python
Digest: sha256:defc498232ab374d3ff7636c2b291fb37a88e3b2407b054b3b6d6c976d910a11
Status: Image is up to date for tsuru/python:latest
 ---> 9c42a1304b7f
Successfully built 9c42a1304b7f
Platform successfully updated!
TSURU_TARGET="${TSURU_TEST_HOST}" TSURU_TOKEN="${TSURU_TEST_TOKEN}" tsuru-admin pool-add -p 'dev\dev.example.com' || true
Pool successfully registered.
TSURU_TARGET="${TSURU_TEST_HOST}" TSURU_TOKEN="${TSURU_TEST_TOKEN}" tsuru-admin pool-add -p 'qa\qa.example.com' || true
Pool successfully registered.
TSURU_TARGET="${TSURU_TEST_HOST}" TSURU_TOKEN="${TSURU_TEST_TOKEN}" tsuru-admin pool-add -p 'stage\stage.example.com' || true
Pool successfully registered.
TSURU_TARGET="${TSURU_TEST_HOST}" TSURU_TOKEN="${TSURU_TEST_TOKEN}" tsuru-admin pool-add -p 'prod\example.com' || true
Pool successfully registered.
TSURU_TARGET="${TSURU_TEST_HOST}" TSURU_TOKEN="${TSURU_TEST_TOKEN}" tsuru-admin plan-create -c 128 small || true
Plan successfully created!
TSURU_TARGET="${TSURU_TEST_HOST}" TSURU_TOKEN="${TSURU_TEST_TOKEN}" tsuru-admin plan-create -c 256 medium || true
Plan successfully created!
TSURU_TARGET="${TSURU_TEST_HOST}" TSURU_TOKEN="${TSURU_TEST_TOKEN}" tsuru-admin plan-create -c 512 huge || true
Plan successfully created!
TSURU_TARGET="${TSURU_TEST_HOST}" TSURU_TOKEN="${TSURU_TEST_TOKEN}" tsuru team-create myteam || true
Team "myteam" successfully created!
TSURU_TARGET="${TSURU_TEST_HOST}" TSURU_TOKEN="${TSURU_TEST_TOKEN}" tsuru team-create superteam || true
Team "superteam" successfully created!
TSURU_TARGET="${TSURU_TEST_HOST}" TSURU_TOKEN="${TSURU_TEST_TOKEN}" tsuru-admin user-quota-change $(tsuru user-info | grep Email: | awk '{print $2}') -- -1
Quota successfully updated.
go test
PASS
ok  	github.com/ef-ctx/tranor	3.976s

Current coverage is 94.48% (diff: 94.11%)

Merging #5 into master will decrease coverage by 1.18%

@@             master         #5   diff @@
==========================================
  Files             6          6
  Lines           761        761
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
- Hits            728        719     -9
- Misses           18         24     +6
- Partials         15         18     +3

Powered by Codecov. Last update 5917cdb...69a4557


feat: add logging to deploy process

add logging into deploy process so developer can track progress & debug when errors are thrown

closed by https://github.com/nomad-xyz/monorepo/pull/141


chore: Add travis

@nikit19 @dreadpool2 @simarsingh24 Guys, please don't compromise with code quality. I saw some strange stuff in the app. I know you're on tight schedule, but GSOC is about learning as well. You must also learn how to design and architect applications, not just write code that technically works.
If you spend 12 hours writing good code, you'll spend 20 hours fixing that code or maybe not as you move forward, you'll forget about it and it'd be written in stone. The greatest benefit of a greenfield project is that you get to do things right and make it correct from start. Coding this way will only make things worse for you guys in future and increase technical debt.
Just see the way API token is handled in each class, this is loitering. I'll be creating an isolated component tomorrow to encapsulate the logic in it. Please follow suit in thinking and designing before coding and you'll see your code shine after these 4 months

@iamareebjamal Should I send the PR about the creation of the Login activity Viewmodel now. I had sent it to the previous repo. Or should I do it after you share your details tomorrow?

@dreadpool2 It's not about MMVM for now. Even after introducing MVVM, it the architecture is not correct, things will be bleak. MVVM MVP is presentation technology. The backend services should be well made first. This is something you should do as you code. Hopefully, I will create an example tomorrow on how to handle encapsulation of logic. Some of the stuff has been done in this PR as well

@iamareebjamal Great :+1:


feat: create StoreBadge component

Could u add your name to this file? :)

Could u add your name to this file? :)

of course yes, done)


[FLINK-23243][docs-zh]Translate "SELECT & WHERE clause" page into Chinese

What is the purpose of the change
Translate "SELECT & WHERE clause" page into Chinese
The page url is https://ci.apache.org/projects/flink/flink-docs-release-1.13/zh/docs/dev/table/sql/queries/select/
The markdown file is located in docs/content.zh/docs/dev/table/sql/select.md
Brief change log

Translate 'flink/docs/content.zh/docs/dev/table/sql/select.md'.

Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving):  no
The serializers:  no
The runtime per-record code paths (performance sensitive):  no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper:  no
The S3 file system connector:  no

Documentation

Does this pull request introduce a new feature?  no
If yes, how is the feature documented? not documented

@95chenjz
Ready to review

cc @wuchong @xccui @klion26 @PatrickRen @becketqin . Could you review my PR when you're free? No hurry, looking for your feedback.

cc @RocMarshal , would you like to review this translation?

@edmondsky Thanks for your contribution. I made a few comments. Please take a look.

Tthanks for your comments. I think it's all set. Please review that again. @RocMarshal

Thanks again :)  @RocMarshal

All of the style of "Catalog" have been changed to "Catalog" style. Is that okay?      @RocMarshal


Facilitate migration from shapefiles with .DBF

So in order to facilitate the migration of shape-files but still maintain functionality, we might consider using only .dbf files instead of .shp files. (excl. buildings geometry, terrain and zone of study).
It will be faster to read by other formats such as CityGML in the future. It will avoid using geopandas potentially. Which is kind of hard to get running.

why?

also, .dbf cannot be directly read from pandas as far as i could find out...

well....i already did it. it is faster by 0.25 sec than reading a shapefile

but... why?

can you specify?

seriously, jimeno, i must have missed the issue with shapefiles. what was the problem again? (since we still need them for the building geometry as mentioned above...) what is the natural format for arcgis? it's not dbf right?

why not sqlite for that matter? or any other format?

so...we need just one shapefile not eight to store geometry.
dbf can be read in arcgis too and work with other shafiles. dbf can be open from excel too.

does sqlite offers those options? if so, i believe it will be a better option than dbf

good. thank you for explaining 😊 (i just needed a motivation for the change because i didn't understand) still... i would like this kind of change (that requires changing the whole documentation and the test cases etc. to have more discussion before we do them - we still need to fix up the code base to be more readible etc...)

totally agree. I also do not know if in terms of interoperability it is better to keep the shapefiles, so it easier for the user to map directly all data. However i found that the way we are doing the shapefiles might be mixing the data in properties.py :-(

right at the time of creating a shapefile and saving it. (this was code that i made up) @daren-thomas coudl you take a look if this is the right way?. then we can leave this issue open for another time

@daren-thomas from the workshop, i see this change does not make sense anymore. if we exchange the shapefiles for .dbs, the users will not be able to select polygons and change the databases in arcgis. it will drastically reduce user experience. i close the issue.


Add a default config for the haskell language server.

Hi folks! First PR here.
Added a simple config for the Haskell Language Server (https://github.com/haskell/haskell-language-server).
Avoided installation as it gets pretty messy.

do we need https://github.com/haskell/haskell-ide-engine then? i.e. can we remove this : https://github.com/neovim/nvim-lspconfig#hie

AFAIK haskell-language-server (AKA hls) is supossed to be the integration point of haskell-ide-engine(AKAhie) with other language servers, and it's the project around which the Haskell community is unifying.  However, hlsandhiewill continue to co-exist side-by-side, withhiegetting mostly bug fixes and new features being added tohls`.
So I think it makes sense to keep both configurations here, let people choose what they want, and only remove hie later when it becomes clearly outdated.

I agree with @joaopizani.
hls does not yet completely replace hie, so I think it makes sense to keep it for now.


chore: replace Redux configuration with Redux Toolkit

@jonnyn Thank you for your PR! Oh great, you update README...! Looks great and merged thanks


BodyTrackingException, Missing Library

Hi all
I am a newbi regarding Kinect. I created a Visual Studio project and was able to use my kinect device. Then I added the body tracking stuff. During the runtime I get the following runtime exception:
K4AdotNet.BodyTracking.BodyTrackingException
HResult=0x80131500
Message=Version 1.0.0 of Body Tracking runtime is expected.
Source=K4AdotNet
In my project file I have:
<ItemGroup>
    <PackageReference Include="K4AdotNet" Version="1.3.5" />
    <PackageReference Include="Microsoft.Azure.Kinect.BodyTracking" Version="1.0.0" />
    <PackageReference Include="Microsoft.Azure.Kinect.Sensor" Version="1.3.0" />
</ItemGroup>

I also installed the Kinect BT package version 1.0.0 (https://docs.microsoft.com/bs-latn-ba/azure/Kinect-dk/body-sdk-download). The sample body tracking application (executable) is working fine!
Any ideas?

Created new project from scratch solved the issue.


Update Tv3play sites to new Viafree site.

Tv3play, Tv6play and Tv8play have moved to Viafree.se. This updates to the new site layout and removes the obsolete files.

Sorry for lateness, been unwell.


[T4A5][T09-B4] Ze Xuan

Ready to review

Some comments added. Feel free to comment if you have any questions. Once done, you may the PR after reading comments.


How can I eject the styles from styled component?

I have a styled component like this:
const Container = styled.div`
  width: 100%;
  max-width: 1200px;
`;

export default Container;

How can I eject the styles from the Container to have it in another constant?
For example,:
let containerStyles = `
  width: 100%;
  max-width: 1200px;
`;

I have to honestly say, I’m a little disappointed after having posted this: https://github.com/styled-components/styled-components/issues/3014#issuecomment-585012306

please post usage questions on StackOverflow or our Spectrum as listed in our issue template.

Again, we’re happy to help and answer questions on GitHub, but It’s still mainly intended to file bugs and other source code related requests. We’re not a large team and face a lot of work just triaging and sorting through issues. 😔
You can use the css helper to reuse blocks of CSS https://styled-components.com/docs/api#css


ERPNext Report not working due to call of get_content on file in S3

The built in "Stock Balance" report  (and likely other reports) in ERPNext is broken when using S3 for attachments.
I have DFP External Storage configured to put all Attachments in S3 storage. When I open the Stock Balance report and click "Generate New Report", I get the error: "Cannot access file path /file/c6238a107c/2023-43-18-13:12.json.gz". Now that file does exist as a "File" record, and is stored in S3, as it should be.
I dug into the code for the report. When you generate a report it runs the query in the background and then displays the content. The content is saved as a file (the one mentioned above), and this file is attached to a "Prepared Report" record type. When the report wants to display the query results, it fetches this Prepared Report (frappe.core.doctype.prepared_report.prepared_report.py) and calls get_prepared_data() on it. This function fetches the file attached to it (the above json file) as a "File" record, and then calls get_content() on it. This fails because the file does not exist locally, and the get_content() function is only defined on "File", but not on "DFPExternalStorageFile". Thus it does not know how to fetch the remote file contents and throws an exception.
A quick search of frappe and erpnext shows many places where get_content() is called, so this could be a more widespread issue.
I'm not sure what the best way to fix this is, hence this issue. It seem like it might be necessary to override get_content() in DFPExternalStorageFile to catch this case and fetch the content for remote files correctly.

Hi @khoran thank you again for let me know your issues :). I just commit a fix for it (thanks to the extended info you gave me!), but not tested at my side (https://github.com/developmentforpeople/dfp_external_storage/commit/f7575dd46d0dd37aebc77fa596091aafe4b65b80). Please check that commit to understand how I resolved it. It just ignores using DFP External File Storage for files attached to "specified" doctypes. But I am with you and I will try to review all code and to find a global solution (if not at least manage what doctypes should be ignored,... 🤔). I will check asap, and let you know :) Stay in touch Kevin ;)

@khoran just checking the code and after the report is generated it is saved as json gzipped and attached as File here: https://github.com/developmentforpeople/frappe/blob/75c79925575e418e82083d6b1f22279c71d71a8f/frappe/client.py#L353
I will do my best ;)

@khoran hi again, resolved in different branch with this commit :) https://github.com/developmentforpeople/dfp_external_storage/commit/1c2d97a622745c07a1c334612b30072add1c517b
I will merge to develop probably this week after some more tests ;)

Great, that fixes the report issue! Thanks for the quick response.


Comma two freeze and reboot while engaged

Describe the bug

I recently had an incident on the interstate where my comma two froze completely (while engaged) and rebooted. The video froze, Comma's steering torque turned off, then after about five seconds in this state, the device rebooted.
How to reproduce or log data
I don't know. I've driven about 5k miles at this point and this is the first time this has happened. So it seems like quite an anomaly.
Expected behavior
Comma should not freeze/reboot while driving :)
Device/Version information (please complete the following information):

Device: comma2
Dongle ID: 75516b36ec7b5bde
Route: I don't see any routes on my useradmin?
Timestamp: I don't see any routes on my useradmin?
Version: v0.8.1-release
Car make/model: Toyota Corolla 2017

Additional context
I was on a very slight curve on an interstate and had to intervene to prevent going out of lane. I was paying attention, but it took a second to realize what was happening and could have been much worse had it been a sharper curve. Is there any way to have the data/telemetry examined by someone at Comma to make sure there's no obvious defect that caused it? I'm on the latest official release (i.e. no custom fork) and I know the time that it happened. Love the product and appreciate any feedback on this issue.

Almost the exact same issue just happened to me tonight. Slight curve, C2 totally locked up, then rebooted. My only indication was that the path didn't get tighter with the turn, then I got a couple LKA warnings on the dash, before I realized what was going on. It was quite disorienting, and probably the scariest thing my C2 has ever done.
For me, it felt more like 15-30 seconds of being frozen before rebooting.
Device: comma2
Dongle ID: 903210ecc0cb706b
Route: Thu, 28 Jan, 2021 @ 19:43 | 903210ecc0cb706b|2021-01-28--20-37-09--43
Timestamp: 21:21:30
Version:  openpilot v0.7.9-VW-release Community Fork
Car make/model: 2019 VW Golf R Manual

Not sure how to pull logs or if there is a core-dump or panic log I can pull, but more than willing to help out with this however I can.

Almost the exact same issue just happened to me tonight. Slight curve, C2 totally locked up, then rebooted. My only indication was that the path didn't get tighter with the turn, then I got a couple LKA warnings on the dash, before I realized what was going on. It was quite disorienting, and probably the scariest thing my C2 has ever done.
For me, it felt more like 15-30 seconds of being frozen before rebooting.
Device: comma2
Dongle ID: 903210ecc0cb706b
Route: Thu, 28 Jan, 2021 @ 19:43 | 903210ecc0cb706b|2021-01-28--20-37-09--43
Timestamp: 21:21:30
Version:  openpilot v0.7.9-VW-release Community Fork
Car make/model: 2019 VW Golf R Manual

Not sure how to pull logs or if there is a core-dump or panic log I can pull, but more than willing to help out with this however I can.

Update: Comma support's response is to return/exchange the unit due to presumed hardware failure. It would be nice to know what exactly happened but I get you can't thoroughly investigate every anomaly. Folks at @commma feel free to close this issue.

Same thing happened to me. Just logging it here in case it's useful to refer to later.
Description
Was driving and the the vehicle unexpectedly veered out of the lane during a highway curve (in the overtaking/left lane).
Speed was steady but the lateral control went out, followed by an 'LKA error', then followed by the Comma hanging and rebooting (at this point the car displayed a 'Cruise Control Malfunction' error). When I stopped, turned off the car and rebooted the device and it was fine afterwards.
How to reproduce or log data
Unable to reproduce immediately under same external conditions. Data was synced via explorer. There are before and after video segments with a few seconds showing all trace lines disappearing from the OP Comma display prior to veering out of lane.
Expected behavior
Maintaining lateral control (and ideally warning of a loss from the device; only the stock warning chimed).
Device/Version information:
Device: Comma2
Dongle ID: 74fc73caf41697fa
Route: Wed, Feb 24 @ 06:49 to 10:12
Timestamp: Approximately at 10:11 (with post-device-reboot video in following segments)
Version: OpenPilot v0.8.1
Car make/model: 2021 Toyota Rav4
Additional context
My thoughts - it was a bit mucky (snow/slush mist); might have been the radar that threw a code (from being covered in dust). I wiped down the logo where the radar is housed and it didn't seem to happen again. Could also have been the (Toyota LKA) camera. Weather was fair with direct sun from driver's side. Prior to using OP, some highway drives would result in the Toyota throwing an error when the radar/camera would get obstructed by ice/dirt. Could be a possible link.

Interesting, looks like it's being worked on (copying from linked ticket):

Also wanted to let you know we are actively working on this. We're looking at it from different angles. We will be increasing the CPU core voltage on devices that are factory tuned to run lower, and we're also looking into any issues caused by the modem driver locking things up.

In my case the incident also followed closely after driving through snowy conditions for a couple hours. But I've driven through plenty of other snowy conditions around New England where comma is just fine 🤷♂ .

So this happened to me one other time since the last time. By far not common, but twice in a month is way more than none in like 6 months of previous ownership.
One thing I was thinking is that I did upgrade NEOS to the latest version, to try OP base, vs the VW fork I'm on right now. When I went back to the VW fork, I did have to downgrade NEOS. Not sure if that is the cause or coincidence.

Update: Just happened again to me (2nd instance in less than 2 weeks) running the newest version.
Weather conditions were good; driving late night (just before midnight). Windshield and radar/camera were clean. It was a moderate, sustained right curve and I was in the middle lane of a 3 lane highway. The lane markings had some gaps but otherwise it was exactly the same as before; noticed an error on the dash with 'cruise control malfunction' along with loss of steering inputs, then the device hanging for a few seconds and just rebooting. Same follow-up; for the remainder of the drive the Comma device was on and recording but the 'cruise malfunction' would come on any time I re-enabled the cruise control and wouldn't engage. Uploaded the drives.
Route: Sat, Mar 6 @ 23:12 to 23:41
Timestamp: Approximately at 23:41 (new segment starts at 23:42)
Version: OpenPilot v0.8.2

I did have to downgrade NEOS. Not sure if that is the cause or coincidence.

Just a much delayed update.
After updating NEOS again, the issue went away for a long while, but now there are other issues that are mentioned in #21057
No clue if they are related at all or not.

Hasn't reoccured on mine since the latest update, however I do have a suspicion that it might be linked to the automatic sway warning that chimes in on some Toyota, VW, and Hyundai (as far as I know have them) "time for a break?" popups during long periods of cruise control being engaged (usually has a little picture of a coffee cup). Just saw your #21057 mention and thought it might help!
That being said, I did have one instance of the "camera malfunction - contact support" issue which went away after about 2-3 minutes on its own (only thing I did that was noteworthy was driving in direct sunlight).

Also has not occurred for me since getting a new C2. The original drive where it happened on my previous unit was quite long in total, but I don't remember how long the particular stretch was. I've been on 5 or 6 ~2 hr drives since getting the new unit.

Hasn't reoccured on mine since the latest update, however I do have a suspicion that it might be linked to the automatic sway warning that chimes in on some Toyota, VW, and Hyundai (as far as I know have them) "time for a break?" popups during long periods of cruise control being engaged (usually has a little picture of a coffee cup). Just saw your #21057 mention and thought it might help!

Well, for what it's worth, in my ticket #21057, after lots of testing it's looking to be the same issue as described in #20872.
So those two issues might get merged. Thanks for the update!

Tracking this in #21539. A fix should ship in 0.8.13.

Hi.. my comma2 freeze today... Just start the car... move a few metters and then freeze at 6km/h winth the video frozen too...the través 30 minuts and still frozen.. so I unplugged and plugged the USB... then never turn on... no vibration at plug... and only the blue Led in the back.... what should I do?? Thanks for your suppot and help.


Configurable drag start threshold for CDK drag-drop

After a week with the drag and drop CDK I have chosen to go with SortableJS. I would like to propose some features for the CDK for future use.
SortableJS has a thresholds feature which is what I was looking for in the CDK's version. Not only can adding threshold tuning be an excellent upgrade but will pave the way for nested cdkDropLists.
Recap 2 related features:

Arrangement threshold to control when a dragging item triggers movement
Nesting cdkDropLists within cdkDrag is a missing feature driving developers away from the CDK and likely requires some type of threshold to integrate.

There's already support for thresholds, although you can only use it by going through the DragRef since it hasn't been exposed as an input. As for nesting, I think it should be covered by https://github.com/angular/components/issues/16671.

Hey Kristiyan, thanks for the link. I searched the issues quickly before opening this but did not find it.
After reading through, it does appear that threshold deserves an input and it also would be very useful if nesting got a section in the documentation.

Renaming this to only cover thresholds since we have a different issue for nested dragging.


BitVec.count_ones seems wrong

I'm having a hard time with count_ones. It does not seem to reflect the actual ones in the slice returned by as_slice. If I use the same slice to reconstruct a BitVec<Lsb0, u8>, I get a different count.
let buffer = mask.as_slice(); // mask is a BitVec<Lsb0, u8>
// buffer is [240, 1, 127, 240, 31, 255, 231, 255, 252, 159, 255, 255, 95, 255, 227, 127, 248, 31, 255, 247, 255, 254, 159, 255, 225, 63, 248, 3, 127, 252]
let mask2: BitVec<Lsb0, u8> = BitVec::from_vec(buffer.to_vec());
println!("{} {}", mask.count_ones(), mask2.count_ones());
// 175 181

The correct count is 175. Am I doing something wrong?

Check that assert_eq!(mask, mask2); succeeds. I strongly suspect that it does not.
The total count of one-bits in buffer is 181, which you can test with mask.as_slice().iter().copied().map(u8::count_ones).sum::<u32>().
However, BitSlice::count_ones only counts within the subset of bits in memory that it considers alive. mask.len() is probably 234 or so, and the six high bits of the last byte are not being included in the count.
The count is expanding from 175 to 181 because constructing a new bit-vector over a memory buffer sets it to cover every bit in the buffer, including any garbage bits that mask had been ignoring. You can test this by calling mask.set_uninitialized(false); before viewing it as a slice; I would bet that this sets the last byte in buffer to 0.
At this point, I would ask how you are constructing mask; I try to have all garbage bits in the buffer set to zero by default, not one, and it generally takes some deliberate fiddling to get a buffer that has garbage ones.

@myrrlyn I am initializing with bitvec![Lsb0, u8; 1; (size.w * size.h) as usize], thus the extra bits were set to 1.
Since I'm working with immutable data, I placed the mask.set_uninitialized(false) right after the initialization.
Now works as I expected. Thanks for the explanation and your time!


improve sort no fields error

Add a description to SortByList in the grammar, and tweak the cli's handling of errors at the end of input to emit better errors when the user forgets to add a field for the sort:
# bin/juttle -e 'emit | sort'
<input>:1:12:
   1:emit | sort
                ^
Error: Expected sort field(s) but end of input found. (SYNTAX-ERROR-WITH-EXPECTED)

# bin/juttle -e 'emit | sort | head'
<input>:1:13:
   1:emit | sort | head
                 ^
Error: Expected option or sort field(s) but "|" found. (SYNTAX-ERROR-WITH-EXPECTED)

@rlgomes @dmajda @mstemm

+1

I pushed a commit which adds a description also for ByList.


environment variables to enable certain modules e.g mod_rewrite?

is it possible to build this into the official image such that one can call certain environment variables to enable certain popular modules? like mod_rewrite?
Thanks

BTW, what is the current state to enable the mod?

Sorry for the delay.  The configuration provided by this image is entirely stock from httpd upstream -- we don't provide any script to help with modification to it.
To enable additional modules, you'll need to update /usr/local/apache2/conf/httpd.conf which includes a number of commented-out LoadModule lines that will need to be enabled.
Since this is a configuration issue, I'm going to close.  If enabling modules via an environment variable is something you need, I'd recommend creating a short Dockerfile which is FROM httpd:X.Y and COPYs an appropriate script which can then be used when running the image to modify the configuration at runtime instead of at build-time. :+1:


Zitadel client id type

Preflight Checklist

[X] I could not find a solution in the existing issues, docs, nor discussions
[ ] I have joined the ZITADEL chat

Describe the docs your are missing or that are wrong
Hello
I simply cannot find anywhere what is the id type, is it bigint / bigserial? What is the type?
Additionally, there is no mention how it is randomised (algorithm).
Additional Context
Would also join ZITADEL chat but invite isn't working–it's crashing at the moment.

Hei @graforlock
Our ids are stored as strings and are generated with sonyflake its an id generator inspired by twitters snowflake.

Thank you for information @hifabienne, I think it's important to include this info in the docs!


Event when rendering is complete?

I need to calculate the vertical position of text sections when the render is complete. This is mostly fine with text only but when I have embedded media, e.g. images, these take longer to render, and push other content down the page when they are displayed. Is there a 'rendering complete' event I can listen for, to know when the content is fully rendered?

All methods are synchronous. No events required.


Fix inconsistency in extracting schema and cause redoc to crash

Closes #95
I just realized it does the same thing as #94, yet this PR avoids duplicate code like in #94

released in 0.13.1


gorm preload not supported count()

Your Question
For some special reasons, I need to query count() on the query with preload. What should I do?
My idea is to make a deep copy of qurey and set the statement.Preloads of the copy to nil
func Paginate(c *gin.Context, originDb *gorm.DB) {
	countQuery := originDb
	if len(originDb.Statement.Preloads) > 0 {
		countQuery.Statement.Preloads = nil
	}
	var totalCount int64
	err := countQuery.Count(&totalCount).Error
}

After this modification, the preloads of originDb are also removed, but I want to keep it. Modifying the preloads of countQuery does not affect originDb. What should I do?
The document you expected this should be explained
Expected answer
Make a copy of originDb, modify the preloads of the copy but not affect originDb

same issue


RL4J : Converted all line endings of rl4j-core to LF

What changes were proposed in this pull request?
Changed all line endings to LF (was a mix of CRLF and LF) of files under rl4j-core.
No other change has been made.
How was this patch tested?
N/A
Quick checklist
The following checklist helps ensure your PR is complete:

[X] Eclipse Contributor Agreement signed, and signed commits - see IP Requirements page for details
[X] Reviewed the Contributing Guidelines and followed the steps within.
[X] Created tests for any significant new code additions.
[X] Relevant tests for your changes are passing.

@aboulang2002 thanks!


[WIP]: screen-pipe: 0.1.48 -> 0.1.93; switch remote.

Changes

Bump tag to v0.1.93
Change repo slug to https://github.com/mediar-ai/screenpipe (owner transferred it to an org)

Things done
(Don't know how to locally test a build, happy to do so if someone could point me in the right direction)

Built on platform(s)

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

[ ] 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/)
24.11 Release Notes (or backporting 23.11 and 24.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.

Add a :+1: reaction to pull requests you find important.

@SomethingGeneric Your PR title should be screen-pipe: 0.1.48 -> 0.1.93; switch remote.

Old maintainers doesn't update for so long.

Would be happy to!

@Bot-wxt1221 commit message should be the same as the title. Read CONTRIBUTION.md for detailed information.

@Bot-wxt1221 what do you need from me to get this going? :sweat:  Sorry if I'm missing something obvious

@SomethingGeneric Fix ofborg first. Look at CI.

@SomethingGeneric Leave hash as blank and build it. nix will tell you the right hash.

@SomethingGeneric Leave hash as blank and build it. nix will tell you the right hash.

Locally build? Or with OfBorg?

@SomethingGeneric build it with nix-build /path/to/nixpkgs -A pkgs-name or nix build /path/to/nixpkgs#pkgs-name locally.

Seems like there's more going on here. Probably going to be stale while upstream figures out a cargo dep (https://github.com/mediar-ai/screenpipe/issues/442)

Should I also update the package to use vendored crates?

@SomethingGeneric Please fix it first.

@SomethingGeneric Leave hash as blank and build it. nix will tell you the right hash.

I was actually just checking to update this dependency. They dont commit the Cargo.lock from their repo, so you have to generate it and copy it over to nixpkgs. The process is not super straightforward. Ive open an issue upstream to ask them to commit the lockfile. Then updating the dependency should just be a nix-update away.
Just saying this is not standard, hopefully it doesnt discourage you from contributing in the future.


Refactor Logic Around Blank Line Being Put Around Content

This makes the empty line around content logic more robust and reusable.
Changes Made:

Converted the logic over to get the start of the line ignoring non-whitespace and greater than characters (>)
Made sure the logic would decide which empty line value to use when blockquotes were involved to make things more uniform
Updated several examples to show the change in logic

This was started on because it seemed related to https://github.com/platers/obsidian-linter/issues/560 at the time. Now it is just more of an enhancement.


Added Annotation Base Class

Reference Issues/PRs

With regards to #798
What does this implement/fix? Explain your changes.

Creates a Base Class for Annotations.
Does your contribution introduce a new dependency? If yes, which one?

What should a reviewer concentrate their feedback on?

If the Base Estimator contains the proper code and is in the proper directory.
Any other comments?

PR checklist

For all contributions

[ ] I've added myself to the list of contributors.
[ ] Optionally, I've updated sktime's CODEOWNERS to receive notifications about future changes to these files.
[ ] I've added unit tests and made sure they pass locally.

For new estimators

[ ] I've added the estimator to the online documentation.
[ ] I've updated the existing example notebooks or provided a new one to showcase how my estimator works.

@satya-pattnaik, @mloning, great work!
I would, however, suggest to think about the following:

I would strongly suggest that the annotated time series z does not need to have the same indices!
I don't think we need exogeneous data - can we not just assume that the series to segment is multivariate to start with? If the annotator insists to focus on one variable, then this can be passed as info to init.
I would suggest to cover both supervised and unsupervised annotators, that should be easy with a "meta-base-class". Supervised ones accept a Y which are training annotations.
I would suggest to call the object being annotated X, not Z.
I would suggest to adopt the fit/_fit design from forecasters - this will make input checks etc easier. We'll probably move there too with TSC, so let's make this uniform!
for segmentation, I would suggest the convention returns a series with an pandas.IntervalIndex; more generally, the docstrings should describe the convention of the input or return objects, if any - implement tests accordingly.
I strongly suggest that we also think about the panel case - this is important, and covered ,methodologically by key annotation packages in the segmentation domain, like hmmlearn. The format I'd suggest is "list-of-X", and for supervised learners "list-of-Y" as a label. Such annotators should also accept the "single-sample" case.

Closed in favour of #1021


Upgrade to socket.io 2.0.x

Hi Simon,
It seems socket.io has come out with 2.0.0 version recently.
Have you planned to upgrade and if so, then when?
Thanks,
Janno

+1

Updated to socket.io v2 in 0.2.0

Thanks Simon!


SAK-30384 - Fixed chat width conflict

Resolution of bug 30384 on (Jira).

I didn't know you were working on this, I did #1979 yesterday because I saw JIRA unassigned. Sorry if it bothers you.

Thanks for your work on this Zachary!

That's my fault, I didn't assign myself on Jira. Thanks!