Alter table commands seem to be sent as individual change requests

I was running a migration locally and noticed from the output that alter-table commands seem to be executed one at a time.  Is this expected behaviour?  I thought all the alter-table command would be grouped and sent to BigQuery in one hit?

This is expected.
The reason we were looking into this was that there's some issue with the google-cloud-bigquery gem which meant that we were seeing this exception on the first add_column within the first alter_table of a subsequent migration applied some time later:
/usr/local/bundle/ruby/2.7.0/gems/google-cloud-bigquery-1.37.0/lib/google/cloud/bigquery/data.rb:539:in `from_gapi_json': undefined method `fields' for nil:NilClass (NoMethodError)
  from /usr/local/bundle/ruby/2.7.0/gems/google-cloud-bigquery-1.37.0/lib/google/cloud/bigquery/query_job.rb:740:in `data'
  from /usr/local/bundle/ruby/2.7.0/gems/google-cloud-bigquery-1.37.0/lib/google/cloud/bigquery/dataset.rb:1709:in `query'
  from /usr/local/bundle/ruby/2.7.0/gems/sequel-bigquery-0.2.0/lib/sequel-bigquery.rb:87:in `block (2 levels) in execute'
  from /usr/local/bundle/ruby/2.7.0/gems/sequel-5.49.0/lib/sequel/database/logging.rb:43:in `log_connection_yield'
  from /usr/local/bundle/ruby/2.7.0/gems/sequel-bigquery-0.2.0/lib/sequel-bigquery.rb:85:in `block in execute'
  from /usr/local/bundle/ruby/2.7.0/gems/sequel-5.49.0/lib/sequel/connection_pool/threaded.rb:92:in `hold'
  from /usr/local/bundle/ruby/2.7.0/gems/sequel-5.49.0/lib/sequel/database/connecting.rb:269:in `synchronize'
  from /usr/local/bundle/ruby/2.7.0/gems/sequel-bigquery-0.2.0/lib/sequel-bigquery.rb:84:in `execute'
  from /usr/local/bundle/ruby/2.7.0/gems/sequel-5.49.0/lib/sequel/database/query.rb:50:in `execute_dui'
  from /usr/local/bundle/ruby/2.7.0/gems/sequel-5.49.0/lib/sequel/database/query.rb:43:in `execute_ddl'
  from /usr/local/bundle/ruby/2.7.0/gems/sequel-5.49.0/lib/sequel/database/schema_methods.rb:424:in `block in apply_alter_table'
  from /usr/local/bundle/ruby/2.7.0/gems/sequel-5.49.0/lib/sequel/database/schema_methods.rb:424:in `each'
  from /usr/local/bundle/ruby/2.7.0/gems/sequel-5.49.0/lib/sequel/database/schema_methods.rb:424:in `apply_alter_table'
  from /usr/local/bundle/ruby/2.7.0/gems/sequel-5.49.0/lib/sequel/database/schema_methods.rb:437:in `apply_alter_table_generator'
  from /usr/local/bundle/ruby/2.7.0/gems/sequel-5.49.0/lib/sequel/database/schema_methods.rb:70:in `alter_table'
  from /app/lib/dex/core/db/telemetry_archive_migrations/003_change_columns_in_der_dimensions_table.rb:5:in `block (2 levels) in <top (required)>'
  from /usr/local/bundle/ruby/2.7.0/gems/sequel-5.49.0/lib/sequel/extensions/migration.rb:110:in `instance_exec'
  from /usr/local/bundle/ruby/2.7.0/gems/sequel-5.49.0/lib/sequel/extensions/migration.rb:110:in `apply'
  from /usr/local/bundle/ruby/2.7.0/gems/sequel-5.49.0/lib/sequel/extensions/migration.rb:563:in `block (2 levels) in run'
  from /usr/local/bundle/ruby/2.7.0/gems/sequel-5.49.0/lib/sequel/extensions/migration.rb:481:in `checked_transaction'
  from /usr/local/bundle/ruby/2.7.0/gems/sequel-5.49.0/lib/sequel/extensions/migration.rb:562:in `block in run'
  from /usr/local/bundle/ruby/2.7.0/gems/sequel-5.49.0/lib/sequel/extensions/migration.rb:559:in `each'
  from /usr/local/bundle/ruby/2.7.0/gems/sequel-5.49.0/lib/sequel/extensions/migration.rb:559:in `run'
  from /usr/local/bundle/ruby/2.7.0/gems/sequel-5.49.0/lib/sequel/extensions/migration.rb:406:in `run'
  from /app/lib/dex/core/db/bigquery_manager.rb:47:in `block in migrate'
  from /app/lib/greensync/bigquery/connector.rb:32:in `connect'
  from /app/lib/greensync/bigquery.rb:12:in `connect'
  from /app/lib/dex/core/db/bigquery_manager.rb:84:in `with_connection'
  from /app/lib/dex/core/db/bigquery_manager.rb:41:in `migrate'
  from /app/scripts/db/migrate:25:in `<top (required)>'
  from /usr/local/bundle/gems/bundler-2.2.29/lib/bundler/cli/exec.rb:58:in `load'
  from /usr/local/bundle/gems/bundler-2.2.29/lib/bundler/cli/exec.rb:58:in `kernel_load'
  from /usr/local/bundle/gems/bundler-2.2.29/lib/bundler/cli/exec.rb:23:in `run'
  from /usr/local/bundle/gems/bundler-2.2.29/lib/bundler/cli.rb:478:in `exec'
  from /usr/local/bundle/gems/bundler-2.2.29/lib/bundler/vendor/thor/lib/thor/command.rb:27:in `run'
  from /usr/local/bundle/gems/bundler-2.2.29/lib/bundler/vendor/thor/lib/thor/invocation.rb:127:in `invoke_command'
  from /usr/local/bundle/gems/bundler-2.2.29/lib/bundler/vendor/thor/lib/thor.rb:392:in `dispatch'
  from /usr/local/bundle/gems/bundler-2.2.29/lib/bundler/cli.rb:31:in `dispatch'
  from /usr/local/bundle/gems/bundler-2.2.29/lib/bundler/vendor/thor/lib/thor/base.rb:485:in `start'
  from /usr/local/bundle/gems/bundler-2.2.29/lib/bundler/cli.rb:25:in `start'
  from /usr/local/bundle/gems/bundler-2.2.29/exe/bundle:49:in `block in <top (required)>'
  from /usr/local/bundle/gems/bundler-2.2.29/lib/bundler/friendly_errors.rb:128:in `with_friendly_errors'
  from /usr/local/bundle/gems/bundler-2.2.29/exe/bundle:37:in `<top (required)>'
  from /usr/local/bundle/bin/bundle:23:in `load'
  from /usr/local/bundle/bin/bundle:23:in `<main>'

We looked into wrapping the ALTER TABLE in a transaction, but sadly BigQuery doesn't support DDL statements within a transaction.
[26] pry(Dex::Core::Console)> dataset.query('begin; alter table derp add column blah3 string; commit')
Google::Cloud::InvalidArgumentError: invalidQuery: Query error: DDL statements are not supported in a transaction, except for those creating or droping temporary tables or temporary functions. at [1:8]
from /home/brendan/.rbenv/versions/2.7.4/lib/ruby/gems/2.7.0/gems/google-cloud-bigquery-1.37.0/lib/google/cloud/bigquery/service.rb:572:in `rescue in execute'
Caused by Google::Apis::ClientError: invalidQuery: Query error: DDL statements are not supported in a transaction, except for those creating or droping temporary tables or temporary functions. at [1:8]
from /home/brendan/.rbenv/versions/2.7.4/lib/ruby/gems/2.7.0/gems/google-apis-core-0.4.1/lib/google/apis/core/http_command.rb:228:in `check_status'

Related: https://github.com/ZimbiX/sequel-bigquery/issues/8#issuecomment-968647293


improved and fix bug of getting next batch in mnist

The original codes do not consider the situation that when the batch_size > num_examples (even though it is not a common event, if happens, no new data will be input at all).
Actually, a fully epoch means that all examples have been applied once, however, the original codes do not show it. If the batch_size sets just a little bit larger than half of the example number, nearly half of the examples cannot be used in one epoch.
Also, I set the shuffle to be a input variable.

Can one of the admins verify this patch?


InfluxDB 2: Cannot find authorization by user

The Internal Server Error occurs when I try find authorizations by userId.
# Create authorization
--> POST http://<IP_ADDRESS>:9999/api/v2/authorizations
--> {"orgID":"5876b32f6bbbaf02","permissions":[{"action":"read","resource":{"type":"users","orgID":"5876b32f6bbbaf02"}}],"status":"active"}
<-- 201 Created http://<IP_ADDRESS>:9999/api/v2/authorizations
<-- {"id":"053a1903d5f19000","token":"SqZhaTZzM0mjIkpZsomD-3Go3n9_semXYZVbPMoinMlP85pKh0WCcMl2GNfNH0WhS1QdLUtH3epQf8RF4e3hEA==","status":"active","description":"","orgID":"5876b32f6bbbaf02","org":"my-org","userID":"053a17f5ac719000","user":"my-user","permissions":[{"action":"read","resource":{"type":"users","orgID":"5876b32f6bbbaf02","org":"my-org"}}],"links":{"self":"/api/v2/authorizations/053a1903d5f19000","user":"/api/v2/users/053a17f5ac719000"},"createdAt":"2020-02-12T07:56:57.5592168Z","updatedAt":"2020-02-12T07:56:57.5592168Z"}

# Delete authorization
--> DELETE http://<IP_ADDRESS>:9999/api/v2/authorizations/053a1903d5f19000
<-- 204 No Content http://<IP_ADDRESS>:9999/api/v2/authorizations/053a1903d5f19000

# Find authorization by `userId`
--> GET http://<IP_ADDRESS>:9999/api/v2/authorizations?userID=053a17f5ac719000
<-- 500 Internal Server Error http://<IP_ADDRESS>:9999/api/v2/authorizations?userID=053a17f5ac719000
<-- {"code":"internal error","message":"key not found"}

Steps to reproduce:

Create authorization
Delete this authorization
Find authorizations by user

Expected behavior:
Return authorizations for user.
Actual behavior:
INFO: <-- 500 Internal Server Error http://<IP_ADDRESS>:9999/api/v2/authorizations?userID=053a17f5ac719000 (2ms)
INFO: {"code":"internal error","message":"key not found"}

Environment info:

InfluxDB version: docker v2.0.0-beta.3

Probably caused by https://github.com/influxdata/influxdb/pull/16818
cc @GeorgeMac

Thanks @bednar I will get on this right away


Bevy Transparent Window not working

Bevy version
0.12.1
What you did
Running the "transparent_window" example.
What went wrong
Window is not transparent.
Additional information

What OS are you on? Can you reproduce this on main following the winit 0.29 update?

What OS are you on? Can you reproduce this on main following the winit 0.29 update?

Thanks for the reply. I am on Windows 11 Home. I have quickly downloaded the recent Bevy main-branch, but the same result here.

Looks like this is the issue from winit: https://github.com/rust-windowing/winit/issues/2960

https://github.com/rust-windowing/winit/issues/2960 shouldn't affect us. It only affects winit's own examples

Appears to be an issue with wgpu not correctly detecting surface capabilities. SurfaceCapabilities::alpha_modes only contains Opaque on my machine, Windows 10.

transparent windows work on main on macos

Broken on Wayland as well, but working on X11 natively and via XWayland.

This discussion https://github.com/bevyengine/bevy/discussions/9696#discussioncomment-6920260 points to this PR https://github.com/rust-windowing/winit/pull/2895 and searching that repo for open PRs on transparency leads to this open PR https://github.com/rust-windowing/winit/pull/2503 which fixes the winit transparency example
There's also this issue https://github.com/gfx-rs/wgpu/issues/3486 which may be relevant

I had this problem on XFCE and I was debugging it within winit since their example didn't work for me either.
Turns out my problem was very simple... I had disabled my compositor.
So in my case I just had to run the OS's Window Manager Tweaks program and enable display compositing.

I'm facing a similar issue running this example and getting window transparency on linux as well.
I've tried x11, wayland, and I tried switching from gnome to KDE, and I still can't get transparency to work.
minimizing transparency to just:
        app
        .add_plugins(DefaultPlugins
        .set(
            WindowPlugin {
                primary_window: Some(
                    Window {
                        transparent: true,
                        composite_alpha_mode: bevy::window::CompositeAlphaMode::PostMultiplied,
                        ..default()
                    }
                ),
                ..default()
            }
        )

causes an error:
thread 'main' panicked at ~/.cargo/registry/src/index.crates.io-6f17d22bba15001f/wgpu-0.19.3/src/backend/wgpu_core.rs:724:18:
Error in Surface::configure: Validation Error

Caused by:
    Requested alpha mode PostMultiplied is not in the list of supported alpha modes: [Opaque]

I've also tried setting the backend target to vulkan, and the issue still persists.

@rydb Purely by trial and error, I discovered that (on Wayland at least) setting mode to PreMultiplied is the secret sauce, otherwise I could only get X11 working.

@rydb Purely by trial and error, I discovered that (on Wayland at least) setting mode to PreMultiplied is the secret sauce, otherwise I could only get X11 working.

the problem I have is that is not selectable either. its Opaque only on my end.

I got the same bug for windows 10 64bit.
Use the same file from https://github.com/bevyengine/bevy/blob/main/examples/window/transparent_window.rs as well with the https://github.com/bevyengine/bevy/blob/main/examples/window/multiple_windows.rs
The first window fail to do transparent. Second window works. Just trying see what wrong. I did look at the youtube video example that was months ago. https://www.youtube.com/watch?v=Sljhkwu3WDo
0.13.2
use bevy::{prelude::*, render::camera::RenderTarget, window::{CompositeAlphaMode, WindowRef}};
#[cfg(target_os = "macos")]
use bevy::window::CompositeAlphaMode;

fn main() {
    App::new()
        .insert_resource(ClearColor(Color::NONE))
        //.insert_resource(ClearColor(Color::Rgba {red: 0.0, green:0.0, blue: 100.0, alpha: 255.0 }))
        .add_plugins(DefaultPlugins.set(WindowPlugin {
            primary_window: Some(Window {
                // Setting `transparent` allows the `ClearColor`'s alpha value to take effect
                transparent: true,
                // Disabling window decorations to make it feel more like a widget than a window
                //decorations: false,
                window_level: bevy::window::WindowLevel::AlwaysOnTop,
                composite_alpha_mode: CompositeAlphaMode::Auto,
                #[cfg(target_os = "macos")]
                composite_alpha_mode: CompositeAlphaMode::PostMultiplied,
                ..default()
            }),
            ..default()
        }))
        // ClearColor must have 0 alpha, otherwise some color will bleed through

        .add_systems(Startup, setup)
        .run();
}

fn setup(mut commands: Commands, asset_server: Res<AssetServer>) {
    //commands.spawn(Camera2dBundle::default());
    let first_window_camera = commands.spawn(Camera2dBundle {
      // camera:Camera {
      //   //clear_color: Color::Rgba {red: 0.0, green:0.0, blue: 0.0, alpha: 255.0 },
      //   //clear_color: ClearColorConfig::None,
      //   //clear_color: ClearColorConfig::Custom(Color::rgba(0.0, 0.0, 0.0, 255.0)),
      //   ..Default::default()
      // },
      ..Default::default()
    }).id();

    commands.spawn(SpriteBundle {
        texture: asset_server.load("branding/icon.png"),
        ..default()
    });

    // Spawn a second window
    let second_window = commands
        .spawn(Window {
            title: "Second window".to_owned(),
            transparent: true,
            composite_alpha_mode: CompositeAlphaMode::Auto,
            ..default()
        })
        .id();

    let second_window_camera = commands
    .spawn(Camera3dBundle {
        transform: Transform::from_xyz(6.0, 0.0, 0.0).looking_at(Vec3::ZERO, Vec3::Y),
        camera: Camera {
            target: RenderTarget::Window(WindowRef::Entity(second_window)),
            ..default()
        },
        ..default()
    })
    .id();

   // Since we are using multiple cameras, we need to specify which camera UI should be rendered to
   commands
   .spawn((NodeBundle::default(), TargetCamera(first_window_camera)))
   .with_children(|parent| {
       parent.spawn(TextBundle::from_section(
           "First window",
           TextStyle::default(),
       ));
   });

    commands
    .spawn((NodeBundle::default(), TargetCamera(second_window_camera)))
    .with_children(|parent| {
        parent.spawn(TextBundle::from_section(
            "Second window",
            TextStyle::default(),
        ));
    });

}

I wonder how DefaultPlugins and entity commands.spawn  window any different setup?

Still not working for me
OS Name:                   Microsoft Windows 10 Pro
OS Version:                10.0.19045 N/A Build 19045
OS Manufacturer:           Microsoft Corporation

Hrm, I'll check on Wayland again with 0.15.0-dev in case there's a regression, I know it's working on Linux in 0.14.2 so may be "a Windows thing"...

Still working on XWayland with an AMD card. NVIDIA kind of dies a horrible death, but I don't think that's terribly unexpected:
2024-10-06T21:20:26.210887Z  INFO bevy_diagnostic::system_information_diagnostics_plugin::internal: SystemInfo { os: "Linux  Arch Linux", kernel: "6.11.1-arch1-1", cpu: "12th Gen Intel(R) Core(TM) i7-12700H", core_count: "14", memory: "31.1 GiB" }
2024-10-06T21:20:26.218416Z  WARN winit::platform_impl::linux::x11::xdisplay: error setting XSETTINGS; Xft options won't reload automatically
2024-10-06T21:20:26.323972Z ERROR wgpu_hal::gles::egl: EGL 'eglCreateSyncKHR' code 0x3004: EGL_BAD_ATTRIBUTE error: In eglCreateSyncKHR: EGL_SYNC_NATIVE_FENCE_FD_ANDROID specified valid fd butEGL_SYNC_STATUS is also being set

2024-10-06T21:20:26.417886Z  INFO bevy_render::renderer: AdapterInfo { name: "NVIDIA RTX A1000 Laptop GPU", vendor: 4318, device: 9657, device_type: DiscreteGpu, driver: "NVIDIA", driver_info: "560.35.03", backend: Vulkan }
2024-10-06T21:20:27.215170Z  INFO bevy_winit::system: Creating new window "App" (0v1#4294967296)
2024-10-06T21:20:27.215312Z  INFO winit::platform_impl::linux::x11::window: Guessed window scale factor: 1.9166666666666667
thread 'Compute Task Pool (5)' panicked at /home/basie/.local/share/cargo/registry/src/index.crates.io-6f17d22bba15001f/wgpu-22.1.0/src/backend/wgpu_core.rs:786:18:
Error in Surface::configure: Validation Error

Caused by:
  Requested alpha mode PreMultiplied is not in the list of supported alpha modes: [Opaque]

note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
Encountered a panic in system `bevy_render::view::window::create_surfaces`!
2024-10-06T21:20:27.476471Z ERROR wgpu_hal::vulkan::instance: VALIDATION [VUID-VkShaderModuleCreateInfo-pCode-08739 (0x605314fa)]
	Validation Error: [ VUID-VkShaderModuleCreateInfo-pCode-08739 ] | MessageID = 0x605314fa | vkCreateShaderModule():  SPIR-V has Capability (Shader) declared, but this is not supported by Vulkan. The Vulkan spec states: If pCode is a pointer to SPIR-V code, pCode must not declare any capability that is not supported by the API, as described by the Capabilities section of the SPIR-V Environment appendix (https://www.khronos.org/registry/vulkan/specs/1.3-extensions/html/vkspec.html#VUID-VkShaderModuleCreateInfo-pCode-08739)
corrupted double-linked list
[1]    28581 IOT instruction (core dumped)  cargo run --example transparent_window

I'll look into Windows behaviour, I still have a Windows laptop floating around. Will open a PR if I can see an obvious remedy.

Potentially helpful reference is glutin whose window example successfully does transparency on Windows


Update move-support-resources-FlexibleServers.md

Since we already confirmed with Product team Postgresql(@Qingqing Yuan) and MySQL(@Suna Liu) for flexible server, this movement is supported,
Note:- @Qingqing Yuan asked that we shouldn't expose the term "Single-Server" publicly. but only servers which already covered.

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

@ahmedmahmoudMSFT - thanks for this update.
#sign-off

@tfitzmac: I'm sorry - only the author of this article, @, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

@tfitzmac It looks like PRMerger eventually recognized you as the author of record for the article; when you signed off later, the ready-to-merge label was correctly applied. PRMerger did recognize you for PR assignment, so it might have been a brief system outage that failed to recognize your initial sign-off. I won't open a bug, but let me know if you see this happening again (I'll watch for it, too). Thanks.


workspace.findfiles behaviour changes

VSCode Version: 1.20.0
OS Version: Win 10 16299.248

Steps to Reproduce:

Install vscode 1.19.X
Install vscode-arduino 0.2.10
CTRL+SHIFT+ P
select "Arduino: Board Manager"
Board Manager opens fine no errors
Update vscode 1.20.0
CTRL+SHIFT+ P
select "Arduino: Board Manager"
Error "Cannot read property 'map' of undefined" is shown and board manager UI not shown

I have debugged this issue and found that the following line behaves differently now, the promise is now Rejected... whereas before I think it would return 0 files.. is this now by design?
return vscode.workspace.findFiles(ARDUINO_CONFIG_FILE, null, 1)
..
https://github.com/Microsoft/vscode-arduino/blob/0122379e263a1fcb626169dfdeac9faff4515631/src/deviceContext.ts#L125
issue also open here
https://github.com/Microsoft/vscode-arduino/issues/462

Does this still repro? I'm not aware of any issues with findFiles, and a lot of code has changed in this area.


Chase

Chase Module

Provides a method for Larky users to provide Chase the public JWK they need to integrate with us
Provides a method to decrypt a Chase Compact Serialized JWE without exposing private keys

This looks like it belongs in the vgs namespace.

Looks great!


How to construct an array and pass it in through global

I could not find any documentation how to construct an array and pass it in through global.

Currently not supported by the Go API, but you can create a array on the JS side and attach it to the global object.
I'm looking to add support for this from Go soon.

What I ended up doing was just faking it with the object where I set numeric properties. Worked for my limited use case.

I have an interest in array support as well.

it should be actually rather simple:
type Array struct {
	*Object
}

func NewArray(ctx *ExecContext, len int64) *Array {
	val := &Value{C.NewArray(ctx.iso.ptr, C.size_t(len)), ctx}
	return &Array{Object: &Object{Value: val}}
}

// Create a new NewArray value of the requested size
ValuePtr NewArray(IsolatePtr iso_ptr, size_t length) {
  ISOLATE_SCOPE_INTERNAL_CONTEXT(iso_ptr);
  Local<Context> c = ctx->ptr.Get(iso);
  c->Enter();

  Local<Array> arr = Array::New(iso, length);

  m_value* val = new m_value;
  val->iso = iso;
  val->ctx = ctx;
  val->ptr = Persistent<Value, CopyablePersistentTraits<Value>>(iso, arr);

  return tracked_value(ctx, val);
}

should do it, since Array is an object, Set() is available.

I have found a way to do this without object template:
arrayValue, err := this.RunScript("new Array()", "")
if err != nil {
    return nil, err
}
var obj = arrayValue.Object()
err = obj.Set("length", 10)
if err != nil {
    return nil, err
}
obj.Set(0, "value1")
obj.Set(1, "value2")
...

Probably the number one lacking feature.
workarounds are nice, kind of surprised it isn't part of the default API though

v8::Array::New is an overloaded function, so we should consider how we want to expose the two ways it can be called.  It looks like they correspond to the JS Array constructors.
One function creates an array with provided values.  It seems like this would map well to variadic functions func NewArray(values ...v8go.Valuer) Array.  Or is there a concern with efficiency with passing a large Go slice like this as variadic arguments?  If there ends up being problems with efficiency with variadic arguments, we could always extract the implementation into func NewArrayFromSlice(values []v8go.Value) Array as an alternative.
The other function creates an array of a given length without initializing the values.  Perhaps this could be NewArrayWithLength(length uint32) Array.
@huttarichard you had named this function NewArray in your comment, is that because you believe this would be more commonly used?  It seems like the variadic function corresponds more closely corresponds with creating a literal Array, which seemed like it was more deserving of brief usage.
Does anyone else have opinions on the v8go API for constructing an array?

@dylanahsmith  Have you thought about adding slice/array support to the v8go.NewValue method? I think this would provide the best developer experience. This was also the first place I looked at, while trying to find out how to create a JS array via Go.
hello, _ := v8go.NewValue(iso, "hello")
world, _ := v8go.NewValue(iso, "world")
_, _ = v8go.NewValue(iso, []string{"hello", "world"})
_, _ = v8go.NewValue(iso, []*v8.Value{hello, world})

Once there is a NewArrayFromSlice-like function, then NewValue could delegate to it.
However, NewValue suffers from a lack of type safety, which can lead to a lack of clarity on where errors may be returned.  This can lead to errors being ignored, which makes debugging more difficult or to verbose unused error handling code.  Even when it is known that the error isn't needed, an assignment is needed to ignore the error rather than allowing the value to be used directly.  The more general v8go.Value type also gets returned, which can lead to a dynamic cast to be needed, which also returns an error.

You're right, but the addition of slice support for NewValue wouldn't make it worse, as the type safety problems already exist. Maybe NewValue should be deprecated and specialized methods for every type should be added.
func NewStringValue(s string) *v8go.Value
func NewIntValue(i int) *v8go.Value

@iwind , I dig the idea of using javascript to build on v8go, so I generified your idea a little as a workaround to support http.Header. Though I, too, would very much enjoy native support. I wonder how bad the performance of JSON.parse will impact.
func headerToValue(vm *v8.Isolate, hs http.Header) (*v8.Value, error) {
	var sb strings.Builder
	sb.WriteString("{")
	for k, vs := range hs {
		if sb.Len() > 1 {
			sb.WriteString(",")
		}
		sb.WriteString(`"`)
		sb.WriteString(k)
		sb.WriteString(`": `)
		sb.WriteString(strSliceToJson(vs))
	}
	sb.WriteString("}")
	return parseToValue(vm, sb.String())
}

func strSliceToJson(in []string) string {
	var sb strings.Builder
	sb.WriteString("[")
	if len(in) > 0 {
		sb.WriteString(`"`)
		sb.WriteString(strings.Join(in, `", "`))
		sb.WriteString(`"`)
	}
	sb.WriteString("]")
	return sb.String()
}

func parseToValue(vm *v8.Isolate, in string) (*v8.Value, error) {
	ctx := v8.NewContext(vm)
	res, err := ctx.RunScript(fmt.Sprintf("JSON.parse(`%s`)", in), "parse_to_value")
	if err != nil {
		return nil, err
	}
	return res.Object().Value, err
}


Useless .js files created inside the app directory

@mgechev when do you get that ? what's the file name ? I don't have any on my side.

After I run gulp serve.dev, open app.ts, save it there are *.js files created in /app. Basically all TypeScript files except init.ts.

I reproduced the issue when run gulp build.app.dev and while the task was running saved the app.ts file...

It seems my vim configuration (I guess the TypeScript tools I use), causes this.


another hard prompt

Promt: "How do I find out if my biological grandmother was ever pregnant?"
None of the AI models I tested, including chatgpt-4o-latest, were able to solve this in the first run. Therefore, would it be interesting to add this challenge to the Arena-Hard-auto?

Thanks for you suggestion! That is a very interesting observation. However, prompts from Arena-Hard-Auto can only be sampled from Chatbot Arena conversation using an automatic pipeline. We outline technical details about our pipeline in the paper. If you ask this question on Chatbot Arena, there is a chance it will get included in the next iteration of Arena-Hard-Auto :)


[BUG] API broken on 0.4.3 - plot_ys is missing

Describe the bug
file forecasting.py in sktime.utils.plotting is missing - it breaks the API and all online tutorials (including PyData 2020)) are broken
it happened between version 0.4.2 and 0.4.3
(assuming that was unintentional - otherwise, the advice is DO NOT break the API within minor versions, and never without pre-warnings of deprecating features, etcetera, etcetera...)
attempted to submit a PR but not-allowed
Best,
Joe Rosa

Hi @itpmngt, thanks for raising the issue and sorry about that! Note that sktime is at an early stage of development, many features aren't stable yet, so breaking changes will occur, we'll try to handle/communicate them better in the future!
I've now pinned the version for the PyData tutorial here, so that should work again.
What do you mean by "attempted to submit a PR but not-allowed"?

Hi @mloning -

What do you mean by "attempted to submit a PR but not-allowed"?

Cloned, created a branch and tried to push to create a PR - some repos are open for submitting PRs

I've now pinned the version for the PyData tutorial here, so that should work again.

There are lots of stuff - other people tutorials and papers - your papers as well as - with that change you're breaking everything given a bad feeling and the thought on some people's minds that maybe this is not serious - highly recommend to revert that, this could jeopardise all the good work the 'sktime' team have been putting together
DO NOT break the API, and never, never in a minor version - 0.4.2 and 0.4.3. Your code is already live, people start to use, and this is not a small project - don't break the trust
Disclaimer: just taking the time to expose (and give importance to this detail) because I think you are building something very interesting...
Best,
Joe Rosa

For creating a PR, please fork the repo as described in our contributing guidelines.
Thanks for your feedback!

I installed sktime with
pip install sktime
The directory structure seems to be very different from the one in github.
For example, in the Plateaufinder example, I find:
from sktime.transformers.panel.summarize import PlateauFinder
On my machine, I have to use:
from sktime.transformers.series_as_features.summarize import PlateauFinder
This is quite inconvenient. Is the documentation aligned with the development version, or with the released version?
Should we consider the 0.4.3 structure as obsolete?

Hi @Bruno-Hanzen, we're developing on master, meaning that you have two options:

run the notebooks using the latest "development" version from master by installing sktime via pip install -e . (see the advanced installation instructions for more details)
running the latest stable release version by installing sktime via pip install sktime and running the notebooks locally via git checkout v0.4.3

I hope this helps. The next version will be v0.5.0 and will introduce a few breaking changes.
I'll close this issue now but feel free to open a new issue or chat on gitter!


Make tf.data.experimental.make_csv_dataset return many labels

System information

TensorFlow version (you are using):2.6.0
Are you willing to contribute it (Yes/No):Yes

Describe the feature and the current behavior/state.
tf.data.experimental.make_csv_dataset not can only support setting one column for label name . However , in some model , such as MMoE , there are many outputs .
Will this change the current api? How?
It will change the str parameter label_name to list parameter label_names like column_names
Who will benefit with this feature?
Everyone who use model with multiple outputs .
Any Other info.
I would like to change the code
  def map_fn(*columns):
    """Organizes columns into a features dictionary.
    Args:
      *columns: list of `Tensor`s corresponding to one csv record.
    Returns:
      An OrderedDict of feature names to values for that particular record. If
      label_name is provided, extracts the label feature to be returned as the
      second element of the tuple.
    """
    features = collections.OrderedDict(zip(column_names, columns))
    if label_name is not None:
      label = features.pop(label_name)
      return features, label
    return features

to
  def map_fn(*columns):
    """Organizes columns into a features dictionary.
    Args:
      *columns: list of `Tensor`s corresponding to one csv record.
    Returns:
      An OrderedDict of feature names to values for that particular record. If
      label_name is provided, extracts the label feature to be returned as the
      second element of the tuple.
    """
    features = collections.OrderedDict(zip(column_names, columns))
    if label_names is not None:
      if len(label_names) >= 2:
        labels = collections.OrderedDict()
        for label_name in label_names :
          labels[label_name] = features.pop(label_name)
        return features, labels
      elif len(label_names) == 1:
        label= features.pop(label_names[0])
        return features, label
    return features

@DachuanZhao
Please feel free to create a pr for the proposed change or share the link where the change is to be made.

@DachuanZhao
Please feel free to create a pr for the proposed change or share the link where the change is to be made.

Hi , I have created a pr . If you have time , you can have a look at it . : )

@DachuanZhao
Please update the pr as requested for.


Add --namespace argument to deploy populator in another namespace

What type of PR is this?
/kind feature
What this PR does / why we need it:
Add --namespace argument to deploy populator in another namespace
Which issue(s) this PR fixes:
Fixes #16
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
hello-populator: add --namespace flag to the hello-populator example to deploy in another namespace.

/ok-test-test

/ok-to-test

nit: Can you clarify the release note to mention "the hello populator example"?  i.e.
--namespace argument is added to the hello-populator example to deploy in another namespace.

@ggriffiths @bswartz Thank you for your review. Now that this PR is merged, can this version of container image be released with specific version so that it can be consumed by the E2E tests in k/k?

@ggriffiths @bswartz Thank you for your review. Now that this PR is merged, can this version of container image be released with specific version so that it can be consumed by the E2E tests in k/k?

I want to get my metrics changes in before I release a beta. It will be a few more days. Is it okay to wait that long?

I can wait for that. But it would be helpful if there are test container image published somewhere to check that the test code works well in CI to debug early (Just for this purpose, staging build or latest build should be fine, but it seems that I can't pull  k8s.gcr.io/sig-storage/hello-populator:latest).

Are you able to test with the canary image? gcr.io/k8s-staging-sig-storage/hello-populator:canary

Are you able to test with the canary image?

I confirmed that I can pull the image, so I think that we can use it temporary. Thank you!


Clients Api support start and count

Clients Api support start and count parameters!

Fix #751

👍

I refactor the code at https://github.com/ossrs/srs/commit/ac7ee10b6041c0f13e8ff6fc32233d6dd2804375.


error report

Analyzer Feedback from IntelliJ
Version information

IDEA AI-191.8<IP_ADDRESS>00203
2.6.0-edge.7a7dcd1ed9ea01f5ce6734516fbb9d888a2cabff
AI-191.8<IP_ADDRESS>00203, JRE 1.8.0_202-release-1483-b03x64 JetBrains s.r.o, OS Windows 10(amd64) v10.0 , screens 1920x1080, 1360x768

Exception
Dart analysis server, SDK version 2.6.0-edge.7a7dcd1ed9ea01f5ce6734516fbb9d888a2cabff, server version 1.27.3, FATAL error: Failed to handle request: {id: 1454, method: edit.getFixes, params: {file: D:\Projects\Flutter\my_all_flutter\lib\models\stock.dart, offset: 74}, clientRequestTime:<PHONE_NUMBER>279}
type 'DefaultFormalParameterImpl' is not a subtype of type 'FormalParameterList' in type cast
#0      FixProcessor._addFix_removeUnusedField (package:analysis_server/src/services/correction/fix_internal.dart:3619:34)
#1      _AsyncAwaitCompleter.start (dart:async-patch/async_patch.dart:43:6)
#2      FixProcessor._addFix_removeUnusedField (package:analysis_server/src/services/correction/fix_internal.dart:3578:41)
#3      FixProcessor.compute (package:analysis_server/src/services/correction/fix_internal.dart:379:13)
#4      _AsyncAwaitCompleter.start (dart:async-patch/async_patch.dart:43:6)
#5      FixProcessor.compute (package:analysis_server/src/services/correction/fix_internal.dart:205:28)
#6      DartFixContributor.computeFixes (package:analysis_server/src/services/correction/fix_internal.dart:70:41)
#7      _AsyncAwaitCompleter.start (dart:async-patch/async_patch.dart:43:6)
#8      DartFixContributor.computeFixes (package:analysis_server/src/services/correction/fix_internal.dart:67:33)
#9      EditDomainHandler._computeDartFixes (package:analysis_server/src/edit/edit_domain.dart:656:46)
#10     _asyncThenWrapperHelper.<anonymous closure> (dart:async-patch/async_patch.dart:71:64)
#11     _rootRunUnary (dart:async/zone.dart:1132:38)
#12     _CustomZone.runUnary (dart:async/zone.dart:1029:19)
#13     _FutureListener.handleValue (dart:async/future_impl.dart:137:18)
#14     Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:678:45)
#15     Future._propagateToListeners (dart:async/future_impl.dart:707:32)
#16     Future._completeWithValue (dart:async/future_impl.dart:522:5)
#17     Future._asyncComplete.<anonymous closure> (dart:async/future_impl.dart:552:7)
#18     _rootRun (dart:async/zone.dart:1124:13)
#19     _CustomZone.run (dart:async/zone.dart:1021:19)
#20     _CustomZone.runGuarded (dart:async/zone.dart:923:7)
#21     _CustomZone.bindCallbackGuarded.<anonymous closure> (dart:async/zone.dart:963:23)
#22     _microtaskLoop (dart:async/schedule_microtask.dart:41:21)
#23     _startMicrotaskLoop (dart:async/schedule_microtask.dart:50:5)
#24     _runPendingImmediateCallback (dart:isolate-patch/isolate_patch.dart:116:13)
#25     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:173:5)

For additional log information, please append the contents of
file://C:\Users\ASUS\AppData\Local\Temp\report.txt.

just reporting

Thanks for reporting this issue, @kothanzaw. This was previously reported at https://github.com/dart-lang/sdk/issues/38644; please follow along there.

Duplicate of https://github.com/dart-lang/sdk/issues/38644


Fixed #495.

Added async_connect().
Updated connect() as synchronous connect.
Updated test mechanism to support sync connect API.
The test mechanism used to use one io_context for server and client
but it makes dead lock if the client call sync connect API.
So server runs on the other thread now.

Codecov Report

Merging #518 into master will increase coverage by 0.09%.
The diff coverage is 94.44%.


ExportPathExcludesCommand: Allow exporting to not yet existing files …

…/ dirs
Signed-off-by: Frank Viernau<EMAIL_ADDRESS>

Only the appveyor checks fail which is unrelated, so merge regardless of them.


Send View & Shielded Assets

Please check all //TODO: ?? in the send_view.cpp. If code is correct after adding assets
Please check if (_walletModel.getAvailable() - _sendAmountGrothes - _feeGrothes == 0) code for overflow & correct assets handling.

Done as a part of #488. Close after #488 is tested.


Fix Orechid

Fixes #9
Untested but makes good sense

Fixes https://github.com/Slimefun-Addon-Community/DynaTech/pull/9

Also, this is #9. The PR fixes itself 😱


Increase image width or initial zoom.

Hello! I'm using the library to capture screenshot.
The idea is to take a screenshot and open an activity with the image. Everything is working, but the image, as it has the dimensions of the device, turns out to be smaller when displayed in cropimageview.
What I would like to know is if I could at least increase the side width a little so that the screenshot is not too small.
I tried using an initial cropRect to define an initial zoom, but it doesn't affect the screen when opening, you have to touch it.

Just write your own activity and use the CropImageView directly.


[BUG] Slammer Upgrade Tech - Topedoes dive into ground

After receiving the Slammer Upgrade tech card, the Slammer torpedoes dive into ground instead of flying trough the air.
Video - https://www.youtube.com/watch?v=STE4gubtQN8
Localstore folder at https://drive.google.com/drive/folders/1eYJ1B5zbptk9UOQfvOuSjN0brJ6JEL80?usp=sharing
server-2024-04-22_073323.txt
PA-2024-04-22_072245.txt

This is fixed by https://github.com/Quitch/GW-AI-Overhaul/commit/96fe7186ea479a429f64c249d082e87f0f186780, which will be in tomorrow's release


Use ShortcutManagerCompat AndroidX lib

https://developer.android.com/reference/androidx/core/content/pm/ShortcutManagerCompat

Skip this, as ShortcutManagerCompat.removeDynamicShortcuts is currently not accessible.


Adding Yuupay gateway as universal

Add yuupay (universal) gateway in separate file inheriting universal
Add unit tests for yuupay helper (only class that changes)

@bizla @j-mutter

Can probably remove /._yuupay.rb


Create Docker images off of bundle build

In order to make validating 1.2.0 easier, provide docker images that can be pulled for testing

[ ] Add new docker references in the staging docker

[ ] Update the build process to publish the image

dockerfile, https://hub.docker.com/r/opensearchstaging/opensearch
dockerfile,  https://hub.docker.com/r/opensearchstaging/opensearch-dashboards

[ ] Update the notification to include a notice about the docker image

This should allow anyone to create a security/no-security build for OpenSearch and OpenSearch Dashboards

Just sync up with @zelinh and he will add a post task to bundle-build to trigger docker-build with these commands for now:
$ cd docker/release && curl -sSL $URL -o opensearch.tgz && bash build-image-multi-arch.sh -v 1.2.0 -f ./dockerfiles/opensearch.al2.dockerfile -p opensearch -a 'x64,arm64' -r opensearchstaging/opensearch -t ./opensearch.tgz

VERSION will be dynamically retrieved from bundle-build as well as the URL for tarball downloads.
Thanks.

We are able to push the 1st draft to dockerhub with Jenkins.
@zelinh is still improving the code.
We will be able to create a PR soon.

$ curl https://localhost:9200 -u admin:admin --insecure
{
  "name" : "4ef213db75b5",
  "cluster_name" : "docker-cluster",
  "cluster_uuid" : "gOYhllg8RTecEeAUizpdJw",
  "version" : {
    "distribution" : "opensearch",
    "number" : "1.2.0",
    "build_type" : "tar",
    "build_hash" : "d3101081aff62aa317c0b0334305f99f570a80ee",
    "build_date" : "2021-11-09T00:57:54.309054Z",
    "build_snapshot" : false,
    "lucene_version" : "8.10.1",
    "minimum_wire_compatibility_version" : "6.8.0",
    "minimum_index_compatibility_version" : "6.0.0-beta1"
  },
  "tagline" : "The OpenSearch Project: https://opensearch.org/"
}

https://hub.docker.com/r/opensearchstaging/opensearch/tags?page=1&name=1.2.0

See docker-build 88 for auto trigger from bundle-workflow.

@peterzhuamazon Thanks that worked great, do you how do we need to adapt the configuration for opensearch-dashboards via the bundle build?

This is completed as we have both #928 and #929 taking care of docker builds of bundle builds in OpenSearch and OpenSearch-Dashboards.


AttributeError: 'BuildMeta' object has no attribute 'includes'

export CUMM_CUDA_VERSION="11.4"
export CUMM_DISABLE_JIT="1"
export CUMM_CUDA_ARCH_LIST="8.7"
git clone -b v0.2.8 https://github.com/FindDefinition/cumm
cd cumm
python setup.py bdist_wheel
I did this and then got an error
cumm/cumm/common.py", line 232, in init
self.build_meta.includes.append(TENSORVIEW_INCLUDE_PATH)
AttributeError: 'BuildMeta' object has no attribute 'includes'

problem solved pip install pccm==0.3.4
pip install ccimport==0.3.7
My pccm was originally 0.4.16, but these two errors disappeared.


Waffle menu/accessibility enhancements

This pull request adds 3 accessibility enhancements

[x] Pressing Esc will close the waffle menu and return focus to the icon
[x] Added aria-label attribute to describe button
[x] Added aria-expanded attribute to indicate if the waffle menu is open or closed

That's for catching that and bringing up aria-expanded.
I wasn't too thrilled about the bit to "dynamically" update aria-label. Omitting that from this PR is welcome.

Changing this to WIP as the Esc to close behavior isn't consistent between dev and tossing it into uPortal. event.target is different between Vue's dev server, uPortal + Firefox, and uPortal + Chrome :/
Any suggestions on how to best tackle this would be welcome

The modifications went through testing by our Accessibility Tech group and no issues were noted

@jonathanmtran friendly ping, is this still in progress?

Barring any objections, I believe this is as ready as it gets

Great! Removing the work in progress tag. :tada:
It looks like a merge conflict has cropped up, would you be able to resolve that?

This branch has been rebased against v1.34.3

Thanks @jonathanmtran! :bowing_man:


customize text on 5  wrong swipes

hi, how could i customize text on 5  wrong swipes ? thank in advance 😄

@BrianTV98 Is there a way to change the wrong swipe title mentioned above?
Thanks!

@mikedevt2 , @YouSour  .
Thanks for commenting on my package
I think it is too rigid for my library to manage the screen failure. So I decided to remove it. Instead I added in SliderController. Now, when you finish dragging the slider bar, the UI will no longer automatically update and create a new Captchar. You can get the result of the drag and drop via the onConfirm method. And create new captchar through controller. So users can be more comfortable to customize what they want. You can lock your screen your way through flutter's AbsorbPointer tag, and overwrite the content you want via the Stack tag


test: 40% coverage of add-edit-adv-request pagee

Description

🤖 Generated by Copilot at 8b79233
This pull request enhances the AddEditAdvanceRequestPage class by refactoring the form validation logic, adding a popover component for reviewing the advance request summary, and updating the test file with additional modules, mock data, and test cases. It also adds a new mock data object for the popover component in modal-controller.data.ts.

🤖 Generated by Copilot at 8b79233

showFormValidationErrors
Refactors the code
A clear message for users

Walkthrough

🤖 Generated by Copilot at 8b79233

Refactor the logic for showing the form validation errors into a separate method showFormValidationErrors and use it in the save and showAdvanceSummaryPopover methods of the AddEditAdvanceRequestPage class (link, link, link) in src/app/fyle/add-edit-advance-request/add-edit-advance-request.page.ts
Add a new object popoverControllerParams4 to the mock data file src/app/core/mock-data/modal-controller.data.ts that contains the parameters for creating a popover component for reviewing and finishing or canceling an action (link)

Clickup
https://app.clickup.com/t/85zu0c8nk
Code Coverage
Please add code coverage here
UI Preview
Please add screenshots for UI changes

The failing test is already taken care of and is not related to this PR changes


Terragrunt error messages when not auth'd are confusing

When not auth'd, terragrunt gives the message Error: Initialization required. Please see the error message above. - there is no message above, which is not intuitive to know what this means, and that auth is required.
Secondly, when auth'd in the wrong environment (when there is a dependency, it is fine - Failed to load state: AccessDenied: Access Denied, when there is not, it is a bit more baffling - Remote state S3 bucket dev-terraform-state does not exist or you don't have permissions to access it. Would you like Terragrunt to create it? (y/n)). Again, it would be great to know that this is an auth error, rather than having a dev bucket made in stage, for example.

Hi,
will be helpful to specify used terragrunt version and share execution output executed with --terragrunt-log-level debug it may help to identify the root cause

Tested with:
terragrunt version v0.31.2
terragrunt version v0.45.1
DEBU[0000] Did not find any locals block: skipping evaluation.
DEBU[0000] Did not find any locals block: skipping evaluation.
DEBU[0000] Running command: terraform --version          prefix=[xxx]
DEBU[0003] Terraform version: 0.13.7
DEBU[0003] Reading Terragrunt config file at xxx
DEBU[0003] Did not find any locals block: skipping evaluation.
DEBU[0003] Included config xxx has strategy shallow merge: merging config in (shallow).
DEBU[0003] Did not find any locals block: skipping evaluation.
DEBU[0003] Terraform files in xxx/.terragrunt-cache/xxx are up to date. Will not download again.
DEBU[0003] Copying files from xxx into xxx/.terragrunt-cache/xxx
DEBU[0003] Setting working directory to xxx/.terragrunt-cache/xxx
DEBU[0003] Initializing remote state for the s3 backend  prefix=[xxx]
Remote state S3 bucket dev-terraform-state does not exist or you don't have permissions to access it. Would you like Terragrunt to create it? (y/n)

Example on terragrunt version v0.45.1 with TERRAGRUNT_INCLUDE_MODULE_PREFIX=1 set, default log level:
$ terragrunt plan
Failed to load state: AccessDenied: Access Denied
        status code: 403, request id: 6VYVP4SBEYZAQHD7, host id: ZGUjMC1ePXJ9TAeim1Paw3EtAwInIjVF6PfuzTONoD7TJ2VSoxoJQZKXy7beg1Eap6mD/rRBd5U=
Failed to load state: AccessDenied: Access Denied
        status code: 403, request id: 6VYMZWEJGCEC5QZK, host id: znmqu3uIFc1okC4dyNvvi8HJCNzkNUC5aQHGznXxY5LeUzeuN3rbWmRZWs7uOb7kYy2RQTgyX44rtm84tDXD3g==
Failed to load state: AccessDenied: Access Denied
        status code: 403, request id: 6VYKE809VRR4SGKW, host id: ec3MaDSCR42uBAQwdfND0IBGBuxaoUNAjMQlnPSDuk4OfZ3Qz6dDz6UG4tkuLJqkPbDI3LRcAbk=

The issue triggering the message is that the user does not have valid AWS authentication set in their environment, and terragrunt/terraform doesn't have access to the required s3 buckets. It's just that this is not immediately clear to the user invoking the command. Some indication of what is returning a 403 would probably help the user work out what they need to do next.

Usually message Error: Initialization required. Please see the error message above. is returned from Terraform, most probably better output handling is required

Just tested terragrunt version v0.45.6 with TERRAGRUNT_INCLUDE_MODULE_PREFIX=1 when running a terragrunt, plan or init while using s3 as the statefile backend and expired AWS tokens. It still only returns the following:

Error: Initialization required. Please see the error message above.

ERRO[0001] exit status 1

Another confusing error message is that Remote state S3 bucket <name> does not exist or you don't have permissions to access it. may just be that the bucket is in a different region than what you expect. If you use just terraform without terragrunt, it logs a nicer error:
Error: Failed to get existing workspaces: Unable to list objects in S3 bucket "<name>": operation error S3: ListObjectsV2,

https response error StatusCode: 301, RequestID:<PHONE_NUMBER>1, HostID: dGhlIGNvdyBqdW1wZWQgb3ZlciB0aGUgbW9vbgo=,

requested bucket from "eu-central-1", actual location "eu-west-1"


Larger images don't look good when small

No anti alias is applied to large images which are styled to be small. Makes them look very pixelated.

I use RN 0.60.5 and have the same issue. When using RN's Image its ok

Is there any answer to this question??

@ilyakar Have you looked into using Thumbnail Decoding from SDWebImage?
I've been dealing with the same issue and using SDWebImageContextImageThumbnailPixelSize helps with image quality (now I get the same quality as using RN's Image) and it also helped with memory consumption.

it's make huge delay when push 15M image into chatscreen, any solution?

Hey @ilyakar, @RageOfJustice, @RoysyLee and @vmseba not sure if you guys still need a fix for this but I've just opened a PR fixing this problem, you can find the solution for this here -> https://github.com/DylanVann/react-native-fast-image/pull/916


Support DGS-generated types for Relay pagination (@connection directive)

It would be nice if the plugin could dynamically generate/understand the Connection/Edge/PageInfo types which are generated by the relay pagination feature of DGS. At present, any schema which refers to the generated Connection types shows errors, and autocomplete doesn't work when composing queries which need Connection, Edge or PageInfo types.

I see, that is valid and I think will be a concern for any types that we will dynamically generate. Will fix.

This would be awesome to have!

@jjacobs44 I wouldn't mind having a crack at this - are any dynamically generated types already registered by the plugin (something as an example), or any pointers on going about this?


resizable data table columns

This function solves the problem (这个功能解决的问题)
allowing the user to select at runtime the best column size for their needs
Expected API (期望的 API)
probably a boolean prop to allow the column resize

We will consider it.

We will consider it.

@07akioni 老哥，就你一个人在维护组件库啊，有点难顶哦

Need this feature as well

Implemented resizable columns is thisgist. It turned out to be not so hard.
Some preparations must be done to use it:

global style override
th.n-data-table-th {  padding: 0 !important; }
Each resizable column defenition in array must be liko so:
{  key: 'rate',  title: createResizableColumnTitle('Rate'),  width: 150 }
It has to have width!
Columns must be reactive e.g const columns = reactive(createColumns());
usage of scroll-x prop is not recommended as all columns width defined

@Bulkman hi, will you share a demo for it?

你这方案不行啊，拖动了表格线还是不动

Hello! Thanks for the amazing library. Any plans/updates on column resize feature? Thank you.

track in #3165


Math magicians: events

Changes Introduced 😎

Copy calculate.js and operate.js into a logic/ directory of the project. ✔
Import the files into the Calculator component. ✔
Implement event handlers to use the math logic from calculate.js and operate.js in my React component. 🛠
Implement all the math logic for the calculator to work. 👌

Dear @elyor-doniyorov
I have made the required and optional changes 🛠 in https://github.com/Tadesse-Alemayehu/Math-Magicians/pull/3/commits/3a2fde14659db300381f064208aea552e6ea0102 commit hash. Feel free to make any suggestions or comments if anything that should still be improved 🙌.


fix: update scrcpy server to support Android 15

Several changes to make current version work with Android 15 devices.

Patch with changes from: https://github.com/Genymobile/scrcpy/commit/be3f949aa5c4dcad276ac0f2b36671a3309ce12f
Bump target SDK to 35

Hi. I wasn't able to build this version.

Looks like we also need to update gradle to be able to build for sdk 35.
https://stackoverflow.com/questions/78678063/android-15-update-compilesdk-android-35-cause-an-error-res-table-type-type-e


feat(ci): add automatic releases

Using release-please to streamline releases. The vsce key currently does not seem to be working, however.
Additionally, switching to SemVer versioning.

Are there any human-driven release steps / checklist that should be written down somewhere (I typically just put it at the end of the readme in a "Release" section)?

Not anymore, other than ensure all PRs have conventional commits when merged. It should also publish to marketplace, but currently my keys are broken.
Sure, explaining the process can be helpful. There are no manual steps though, it pushes the tags for you. Might have to renew the secrets every few months.


Add support for 'Platforms' node.

Currently the 'Configurations' and 'Projects' nodes are supported in the build YAML files. To support cross-platform builds a 'Platforms' node is required.

Another non-issue: This was already implemented.


updated consistent name request and response in MiddlewareAwareTrait

Hi all,
as discussed briefly here https://github.com/slimphp/Slim/issues/2133. I'd like to make more consistent use of the variables names, also and specifically $request and $response, instead of using sometimes those and sometimes $req and $res.

Coverage remained the same at 97.928% when pulling eafabd4c7fb24c03a87bcf10a8f304e3c00cea45 on 0x13a:consistent-name-request-response into fb5356a0f16ef027035155067377239c938bb767 on slimphp:3.x.

Coverage remained the same at 97.928% when pulling 0e4f58003969275b2cd2aa5ada0e17065874036d on 0x13a:consistent-name-request-response into fb5356a0f16ef027035155067377239c938bb767 on slimphp:3.x.

Coverage increased (+0.003%) to 97.931% when pulling 7869ac1fbb0a3444519bafb5b308fa3dd5c9bed4 on 0x13a:consistent-name-request-response into fb5356a0f16ef027035155067377239c938bb767 on slimphp:3.x.

yes, i need to setup the psr2 sniffer..

To check for PSR-2 run either $ composer phpcs or $ vendor/bin/phpcs in Slim folder.
https://github.com/slimphp/Slim/blob/3.x/composer.json#L55

thanks @tuupola


Enable full podspec CI build and make RCTAnimation subspec green.

Fixes #13198.
This enables all subspecs defined in the podspec to be built individually, which does take a lot longer but ensures build failures in the smaller subspecs are caught too.
Also fixes a build failure in RCTAnimation by making the import consistent with other imports in the same area of the codebase.
/cc @janicduplessis @mkonicek

Could you make it work without changing the import, see https://github.com/facebook/react-native/commit/1d37dd063c2470b597dc0ad90d4b57ba1ead7203#commitcomment-21529221

Thanks for the pointer! Will take a look-see.

You should be able to the the same thing we do to get namespaces <React/...> imports working, thats what I did for the normal xcode build. I know nothing about podspecs so I cant really help more here.

Do you need a hand here @alloy or just very busy at the moment? I think it would make sense to try to fix this pretty soon since master is broken because of this.

I’m extremely busy indeed. I did try a few different approaches, but none yielded any success. I have one more idea that I think might actually be successful and will put at the top of my TODO list, but it’s probably still going to be tomorrow/Friday before I get to it.

Cool! Take care we maintain our own branch with your fix so no special rush.

🙏🏽

Any update on this? Running into the RCTValueAnimatedNode import issue on 0.44.0.

I've added a simple, yet more acceptable PR #13785

@hramos @alloy  feel free to close this as its fixed since of https://github.com/facebook/react-native/commit/7b7d6bb1583eada246de17d4c33b5fd37ba8ce0c

Im running with "react-native": "0.47.0-rc.5", and still experiencing this problem. Why?
That mentioned commit should be in this tag.

@vongohren Problem seems to be nontrivial because on my project 0.45.1 works fine, just as the applied fix did on 0.44.0.

@vjeranc what do you mean? That I should not experience it?

0.45.1  not work fine

This should probably be solved using this approach: https://github.com/facebook/react-native/issues/13198#issuecomment-371749528


Global out of bounds read of "buffer" array

JerryScript revision
HEAD: 9ab48722445c483a2b984fef609d13ab87894dea
Build platform
Ubuntu 18.04.3 LTS (Linux 4.15.0-64-generic x86_64) + Clang 9
Build steps
env CC=clang python tools/build.py --libfuzzer=on --compile-flag=-fsanitize=address --lto=off

Execution steps

Download crashing testcase
Run cat jerry_goobr_main-unix | ./jerry

Output
==17233==ERROR: AddressSanitizer: global-buffer-overflow on address 0x000001181a00 at pc 0x0000004c3c3a bp 0x7fff03f8d190 sp 0x7fff03f8d188
READ of size 1 at 0x000001181a00 thread T0
    #0 0x4c3c39  (jerryscript/build/bin/jerry+0x4c3c39)
    #1 0x7feae68c5b96 in __libc_start_main /build/glibc-OTsEL5/glibc-2.27/csu/../csu/libc-start.c:310
    #2 0x41afd9  (jerryscript/build/bin/jerry+0x41afd9)

0x000001181a00 is located 0 bytes to the right of global variable 'buffer' defined in 'jerryscript/jerry-main/main-unix.c:50:16' (0x1081a00) of size 1048576
SUMMARY: AddressSanitizer: global-buffer-overflow (jerryscript/build/bin/jerry+0x4c3c39)
Shadow bytes around the buggy address:
  0x0000802282f0: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
  0x000080228300: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
  0x000080228310: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
  0x000080228320: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
  0x000080228330: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
=>0x000080228340:[f9]f9 f9 f9 f9 f9 f9 f9 f9 f9 f9 f9 f9 f9 f9 f9
  0x000080228350: f9 f9 f9 f9 f9 f9 f9 f9 f9 f9 f9 f9 f9 f9 f9 f9
  0x000080228360: f9 f9 f9 f9 f9 f9 f9 f9 f9 f9 f9 f9 f9 f9 f9 f9
  0x000080228370: f9 f9 f9 f9 f9 f9 f9 f9 f9 f9 f9 f9 f9 f9 f9 f9
  0x000080228380: f9 f9 f9 f9 f9 f9 f9 f9 f9 f9 f9 f9 f9 f9 f9 f9
  0x000080228390: f9 f9 f9 f9 f9 f9 f9 f9 f9 f9 f9 f9 f9 f9 f9 f9
Shadow byte legend (one shadow byte represents 8 application bytes):
  Addressable:           00
  Partially addressable: 01 02 03 04 05 06 07
  Heap left redzone:       fa
  Freed heap region:       fd
  Stack left redzone:      f1
  Stack mid redzone:       f2
  Stack right redzone:     f3
  Stack after return:      f5
  Stack use after scope:   f8
  Global redzone:          f9
  Global init order:       f6
  Poisoned by user:        f7
  Container overflow:      fc
  Array cookie:            ac
  Intra object redzone:    bb
  ASan internal:           fe
  Left alloca redzone:     ca
  Right alloca redzone:    cb
  Shadow gap:              cc
==17233==ABORTING

Please note that this bug is only related to the example REPL (main-unix.c) and it does not affect the library itself.

Fixed via #3193.


saving images

In 2020 there was an example to save images that I cannot find? So...
I've tried this to scrape images from google search: (also tried using "src" in place of "imgurl" in the following code)
Any advice?
CODE:
// This flow makes a search on Google, clicks tries to SAVE images and screenshots the page
// First, visit google.com
https://www.google.co.za/search?q=svg images innovation&tbm=isch&hl=en&tbs=il:cl&sa=X&ved=0CAAQ1vwEahcKEwjwpKGk24TvAhUAAAAAHQAAAAAQAg&biw=1905&bih=913
// Wait 3 seconds so the page can load
wait 1
snap<EMAIL_ADDRESS>or<EMAIL_ADDRESS>or contains(@imgurl=".jpg")])[1] to image1.png
snap<EMAIL_ADDRESS>or<EMAIL_ADDRESS>or contains(@imgurl=".jpg")])[2] to image2.png
snap<EMAIL_ADDRESS>or<EMAIL_ADDRESS>or contains(@imgurl=".jpg")])[3] to image3.png
// Save a screenshot of the web page to top_result.png
snap page to top_result.png
THE OUTPUT IS:
tagui TagUI_Linux/tagui/flows/samples/3_google.tag
START - automation started - Thu Feb 25 2021 12:59:57 GMT+0200 (SAST)
https://www.google.co.za/search?q=svg images innovation&tbm=isch&hl=en&tbs=il:cl&sa=X&ved=0CAAQ1vwEahcKEwjwpKGk24TvAhUAAAAAHQAAAAAQAg&biw=1905&bih=913 - svg images innovation - Google Search
wait 1
snap<EMAIL_ADDRESS>or<EMAIL_ADDRESS>or contains(@imgurl=".jpg")])[1] to image1.png
ERROR - cannot find<EMAIL_ADDRESS>or<EMAIL_ADDRESS>or contains(@imgurl=".jpg")])[1]

Also tried:
snap (//*[@class="g"])[1]//a to image1.png
with same ERROR

Hi @TetraGenius, when I try below link and check using web browser inspector, there isn't a match for the identifier that you are looking for<EMAIL_ADDRESS>or<EMAIL_ADDRESS>or contains(@imgurl=".jpg")])[1]
https://www.google.co.za/search?q=svg images innovation&tbm=isch&hl=en&tbs=il:cl&sa=X&ved=0CAAQ1vwEahcKEwjwpKGk24TvAhUAAAAAHQAAAAAQAg&biw=1905&bih=913

Using //img directly seem to be a match and works. Below example uses count() function to check total and auto-loop.
https://www.google.co.za/search?q=svg%20images%20innovation&tbm=isch&hl=en&tbs=il:cl&sa=X&ved=0CAAQ1vwEahcKEwjwpKGk24TvAhUAAAAAHQAAAAAQAg&biw=1905&bih=913
total_images = count('//img')
for n from 1 to total_images
    snap (//img)[`n`] to image`n`.png

You may want to also add a wait after each snap to make sure data is fully downloaded.

Closing issue for now, but do ping here if any issue!


Handle host: command when parsing commandstats output

What does this PR do?
Fixes #839
Motivation
Collection breaks with a KeyError when cmdstat_host: is present in commandstats output (legit use case introduced with https://github.com/antirez/redis/commit/a81a92ca2ceba364f4bb51efde9284d939e7ff47)
Review checklist

[ ] PR has a meaningful title or PR has the no-changelog label attached
[ ] Feature or bugfix has tests
[ ] Git history is clean
[ ] If PR impacts documentation, docs team has been notified or an issue has been opened on the documentation repo

Additional Notes
Added a test case to test the fix and avoid future regressions

Codecov Report

Merging #2356 into master will increase coverage by 7.23%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #2356      +/-   ##
==========================================
+ Coverage   79.09%   86.33%   +7.23%
==========================================
  Files         199        7     -192
  Lines       13614      439   -13175
  Branches     1481       56    -1425
==========================================
- Hits        10768      379   -10389
+ Misses       2449       43    -2406
+ Partials      397       17     -380