Long duration hevc file not muxing

Hevc file with long duration like 1hr or more not muxing and output file is 0bytes and mp4muxer.exe running where read disk activity occur for few seconds but no write disk activity.
For hevc clips like 2min duration it works fine.
Kindly solve the error, thanks.

The purpose of dlb_mp4base is to help Dolby's customers/partners to implement theirs mux tool, not use the tool directly. The executable tool provided is just for testing purpose, so it has limited quality.
We recommend to use Bento4 which Dolby has finished the contribution, and make it support Dolby codec.

The purpose of dlb_mp4base is to help Dolby's customers/partners to implement theirs mux tool, not use the tool directly. The executable tool provided is just for testing purpose, so it has limited quality. We recommend to use Bento4 which Dolby has finished the contribution, and make it support Dolby codec.

Thanks for information, is bento4 support dlb_mp4demux also?
Is mp4extract in bento4 is same as dlb_mp4demux?

Have a try with Mp42Hevc.

Have a try with Mp42Hevc.

Ok thanks but it is for extract video only right, then how to extract audio like audio.ec3 ?

Have a try with Mp42Hevc.

Ok thanks but it is for extract video only right, then how to extract audio like audio.ec3 in bento4 ?

You can use dlb_mp4demux for ac3, ec3 and ac4 demuxing.

read disk activity occur for few seconds but no write disk activity.

It writes stuff in RAM.

@Learner00100
Have you fix this problem already?
I was using this command:
mp4muxer --dv-profile 5 -i video.hevc -i audio.ec3 --media-lang eng -o outputfile.mp4
Do you know something similar and what command should i use?

It's working fine, just wait 5-20 minutes and it will be ready.

It's working fine, just wait 5-20 minutes and it will be ready.

Yeah i know now.  It seems like nothing happen , but i just have to wait. Thanks for answering.

It's working fine, just wait 5-20 minutes and it will be ready.

Thank for your answer.
May I  know for mp4 demuxing which tool should I use in bento4 that function as mp4demuxer of Dolby.

Bento4 has mp42aac, mp42avc, mp42hevc, that's all codecs they've supported.

May I know for mp4 demuxing which tool should I use in bento4 that function as mp4demuxer of Dolby.

Bento4 has mp42aac, mp42avc, mp42hevc, that's all codecs they've supported.

May I know for mp4 demuxing which tool should I use in bento4 that function as mp4demuxer of Dolby.

Bento4 has mp42aac, mp42avc, mp42hevc, that's all codecs they've supported.

But for ac3,ac4 what to choose?
Mp42aac only for aac right.
I need for ac3,ac4. So kindly tell what to choose.


Refused to set unsafe header "User-Agent" in Chrome

Package Name: @azure/cosmos
Package Version: 3.15.1
Operating system: Win 10
[ ] nodejs

version: 16.13.1

[x] browser

name/version: Chrome 98.0.4758.82 (Official Build) (64-bit)

[ ] typescript

version: 4.5.5

Is the bug related to documentation in

[ ] README.md
[ ] source code documentation
[ ] SDK API docs on https://docs.microsoft.com

Describe the bug
Console error in browser: Refused to set unsafe header "User-Agent" thrown in xhrHttpClient.js:50 during any operation to an otherwise (seemingly) correctly set up Cosmos DB resource
To Reproduce
Steps to reproduce the behavior:
I used an Angular v13 project, but any project consuming @azure/cosmos making calls via above noted version of Chrome would presumably have the same issue
Expected behavior
No console error. Possibly, package should have a config option to disable setting of user-agent header.

Also running into this issue, any way to fix this?

@selenecodes can you attach a sample?  I have tested this with the version 3.15.1 on both edge and chrome. It does not throw the error on console

Certainly, I created a mockup using the node.js quickstart on https://observablehq.com/@selenecodes/cosmosdb-user-agent-header-issues.
It shows two things:

An error in the console: Refused to set unsafe header "User-Agent"
ChangeFeeds not working as they should (e.g. iterator.hasMoreResults never returns false and iterator.fetchNext() continues providing repeat values.

In other SDKs we use x-ms-useragent in the browser for this exact reason. It seems like Cosmos is setting the User-Agent header explicitly though: https://github.com/Azure/azure-sdk-for-js/blob/2214c6a4845c5ba134f2dcb9388a06da1734ac64/sdk/cosmosdb/cosmos/src/CosmosClient.ts#L88
I think the right fix here is to migrate Cosmos to using our centrally provided userAgent policy instead of manually setting the header: https://github.com/Azure/azure-sdk-for-js/blob/main/sdk/core/core-rest-pipeline/src/policies/userAgentPolicy.ts

Sounds great, I was wondering though. Should I open a new issue for the ChangeFeed issue?

@selenecodes yes please, we will investigate that separately. Thanks.

#21309 For anyone dealing with the ChangeFeed issues please head to this new issue.

@selenecodes wrote:
This was entirely due to me not setting up CORS in Azure Portal.
If you're running into this issue be sure to:
Disable any CORS bypass extensions on your web browser
You should now not be able to request any data.
To solve this follow this documentation page
Closed Issue:
https://github.com/Azure/azure-sdk-for-js/issues/21309

Ehm @jay-most I believe you've closed this issue prematurely. #21309 needed to be closed as this was a user error. However, the User-Agent bug described here persists and needs to be fixed.
See:

In other SDKs we use x-ms-useragent in the browser for this exact reason. It seems like Cosmos is setting the User-Agent header explicitly though:
https://github.com/Azure/azure-sdk-for-js/blob/2214c6a4845c5ba134f2dcb9388a06da1734ac64/sdk/cosmosdb/cosmos/src/CosmosClient.ts#L88
I think the right fix here is to migrate Cosmos to using our centrally provided userAgent policy instead of manually setting the header: https://github.com/Azure/azure-sdk-for-js/blob/main/sdk/core/core-rest-pipeline/src/policies/userAgentPolicy.ts

Reopening to follow up on @selenecodes comment


Unable to run amplify push via management console due to missing file

How did you install the Amplify CLI?
N/A
If applicable, what version of Node.js are you using?
N/A
Amplify CLI Version
Latest
What operating system are you using?
Amazon Linux
Did you make any manual changes to the cloud resources managed by Amplify? Please describe the changes made.
No
Describe the bug
2023-04-27T18:10:00.408Z [INFO]: 🛑 ENOENT: no such file or directory, open '/codebuild/output/src715423107/src/PROJECTFOLDER/amplify/backend/api/hydrus/schema.graphql'
2023-04-27T18:10:00.408Z [INFO]: Resolution: Please report this issue at https://github.com/aws-amplify/amplify-cli/issues and include the project identifier from: 'amplify diagnose --send-report'
Learn more at: https://docs.amplify.aws/cli/project/troubleshooting/
2023-04-27T18:10:00.415Z [WARNING]: ✖ There was an error initializing your environment.
2023-04-27T18:10:00.415Z [INFO]: 🛑 ENOENT: no such file or directory, stat '/codebuild/output/src715423107/src/PROJECTFOLDER/amplify/backend/api/hydrus/schema.graphql'
2023-04-27T18:10:00.415Z [INFO]: Learn more at: https://docs.amplify.aws/cli/project/troubleshooting/
2023-04-27T18:10:00.421Z [INFO]:
2023-04-27T18:10:00.421Z [INFO]: Session Identifier: 5f388968-5dec-41ed-811e-6962ff34ceac
Expected behavior
The amplify backend build should finish normally.
I did nothing but believe the amplify cli had updated a few days ago. Right now, the backend build is failing because the schema.graphql file isn't found but I don't use a single file. I put all my graphql documents into the schema folder.
Reproduction steps

Running amplify push / running a deployment

Project Identifier
2023-04-27T18:10:00.421Z [INFO]: Session Identifier: 5f388968-5dec-41ed-811e-6962ff34ceac
Log output

# Put your logs below this line
2023-04-27T18:10:00.408Z [INFO]: 🛑 ENOENT: no such file or directory, open '/codebuild/output/src715423107/src/**PROJECTFOLDER**/amplify/backend/api/hydrus/schema.graphql'
2023-04-27T18:10:00.408Z [INFO]: Resolution: Please report this issue at https://github.com/aws-amplify/amplify-cli/issues and include the project identifier from: 'amplify diagnose --send-report'
                                 Learn more at: https://docs.amplify.aws/cli/project/troubleshooting/
2023-04-27T18:10:00.415Z [WARNING]: ✖ There was an error initializing your environment.
2023-04-27T18:10:00.415Z [INFO]: 🛑 ENOENT: no such file or directory, stat '/codebuild/output/src715423107/src/**PROJECTFOLDER**/amplify/backend/api/hydrus/schema.graphql'
2023-04-27T18:10:00.415Z [INFO]: Learn more at: https://docs.amplify.aws/cli/project/troubleshooting/
2023-04-27T18:10:00.421Z [INFO]:
2023-04-27T18:10:00.421Z [INFO]: Session Identifier: 5f388968-5dec-41ed-811e-6962ff34ceac

Additional information
N/A
Before submitting, please confirm:

[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
[X] I have removed any sensitive information from my code snippets and submission.

Hey @georgeleeventures , 👋  thank you for reaching out. This issue is currently being tracked on https://github.com/aws-amplify/amplify-category-api/issues/1074
recently reported on https://github.com/aws-amplify/amplify-category-api/issues/1385
Please feel free in subscribing to the issue.
From the issue, could you try downgrading to CLI version to 11.0.5? Do let us know if you are still experiencing an issue on push or deployment.

transferring the issue to API category.


Add support php 8.0

Phony needs to be upgraded ^7.1

@shadowhand maybe it makes sense for PHP 8.0 via ci to update this dependency? To still leave a larger range of support PHP version? In my opinion, test dependencies should in no way affect the number of versions of support

@Yozhef dropping PHP 7.2 support is fine, it is EOL.

@shadowhand okay -)

@shadowhand done =)

@shadowhand are we ready to make a new release?

Release 4.0.0 just tagged.


InBuilt support for AT25XE041B

Added support for new device AT25XE041B (Adesto - Atmel)
Tests passed - speeds are taken from tests.

Hi,
Thanks for the patch.

Can you use a datasheet URL from the manufacturer rather than from a distributor (maybe https://www.adestotech.com/wp-content/uploads/DS-AT25XE041B_062.pdf, not sure).
Use a generic device syntax for FTDI (ftdi:///1) so that no specific HW is defined

Thanks.

Can you have a look at https://github.com/eblot/pyspiflash/pull/20#issuecomment-729057705 Thanks

Sorry for the delay. I was really busy last days. Fixed according you feedback...

Merged, thanks.
I reverted the changes in the unit test, as I'm not sure they would not break with other flash devices.

Merged, thanks.
I reverted the changes in the unit test, as I'm not sure they would not break with other flash devices.


Installable module

Is there any way to make a module installable, something like packages? if so what are the steps to do that, any doc available?

I have been wanting this for a while. So something like October CMS or more famously Word Press correct?
I am still doing research on this and most people don't really say how they did it.
I think it is a combination of composer/installers and using satis as your repository. You will possibly need to register with composer/installers for your own setup but you probably could use a preset type like asgard:modules.
I was able to use composer/installers from a regular github repo to pull my module into the Modules directory. I am still trying to figure out the admin controller side to do this. I really would like to be able to pull down a list of available modules and let users install their own modules with having to run composer from the command line.
I hope this helps and would appreciate if you get further to let me know how you went about it.

@illuminate3  Do you have any which explains how the Asgard CMS works, like how can i build an application which can be used to supports modules?

Asgard uses their own module system. I prefer the simplicity of Caffeinated modules.
Asgard uses composer/installer ( https://github.com/composer/installers ) and then sets the type. Asgard has a module called workbench which lists and let's you enable/disable the modules but you have to install several other modules too. Building a workbench like module based on Caffeinated is not that hard.
But like you, I am still looking for system that let's me get a list of supported modules, download them and install them. This API side is what led me to look into Satis ( https://composer.github.io/satis/ ).
Sadly, I haven't had time to progress beyond this point because I can't get Caffinated to move config files with vendor:publish with laravel 5.6. Road block.
Like i said before, you might want to look into October cms and see how they do it. I have been looking for a package to do the API side but haven't seen anything.
Before, with laravel 5.1. I used the merge-plugin and then just used composer to call my modules. it wasn't fancy. It just sorted my code into logical units (modules) instead of creating packages in the vendor directory. So, don't over think it!
How far have you got and what kind of discovery have you made?

Asgard Usage the https://github.com/nWidart/laravel-modules package for building modules. Each module can be a separate composer package and you can install from the terminal (though it requires an another package to do that). For updating those individual packages composer-merge-plugin could be an option though i have not tried that yet.
I was searching for a install/uninstall option for Caffeinated  module but did not find any and that is the reason of crating this issue.
Satis is the best option to maintain private packages and there is an another approach named VCS Repo

We are on the same page here I think.
I haven't found a module package that allows direct install without having that module already in composer or using composer required from the command line. You would have to create a function that finds the modules (why I suggested satis) and then run php artisan from within a function to composer require it. I still haven't had a chance to install Satis so I am not familiar with what kinds of APIs can be built.
I am porting https://github.com/rappasoft/laravel-5-boilerplate into a module system at the moment. Almost done but have an issue with moving the USER model into a module. I will link back after I sort that out or if I give up tonight I will just push to my github account. Either way, I will let you know.

@nWidart thanks for mentioning the command module:install. I did not find anything related to that and also on publishing module page you mentioned the package joshbrw/laravel-module-installer. Can you please revisit the doc and update if necessary.


Android support

Bellow list is exploring list, not implementation. I'll start implementing once all are feasible.

[x] Init scripts
[x] IPC
[x] custom protocol

Evaluate script is meaningless until we can provide some Android window (activity/fragment) library. See this comment for more info.

Android has its own webview, available in java / kotlin.
Is this a feature request to have the same rust wry api available as jni? I once created a webview with an additional nim / c++ jni, based on android studio https://github.com/marcomq/nimview/tree/main/examples/android but I guess you are looking for something different here...

@marcomq We are actually looking for ways to build from its webview jni too! Because the inner webview library ABI is not exposed afaik.
I'm still busy on other stuff but I hope I can resume my research soon. Basically I would like to use android-ndk-rs
 and combine with cargo-mobile. So it can make dev easier to setup the project and even run it.
Do you think this is the right way to do so?

Do you think this is the right way to do so?

Actually no idea. My android knowledge is pretty basic. I was just interested on how this is planned.
But this sounds good. Great to know that there already is some library that can run rust on android and great to know that there is a plan :)

It shouldn't be hard to support android, maybe should consider more priority on this issue

@Mohsen7s
It literally can't get any higher on our priority list. Mobile support is our main feature for tauri v2!
Btw, we appreciate every helping hand we can get, especially for Android support.

Hey guys wanna give some update on current progress.
Here's my attempt to support Android so far: https://github.com/wusyong/android_webview.rs

It doesn't contain lots of things because I'm focussing on how to make it more structural and ergonomic for users to do everythings they want. The sad thing is we can't create a NDK and use NativeActivity because it doesn't have Looper implemented by default and WebView will reject it upon creating. So the minimum we can offer is EmptyActivity and this also comes down to multi-windows support. We really can't use winit nor tao because they literally just some code implementation in NativeActivity. We will have to implement a new window management library if we want to offer richer window features on android.
This also affect after webview creation because we could only write FFI function to communicate with koltin code. We can't control the event loop after onCreate. That means the timing to call evaluate_script will be difficult.
So for the foreseeable future, the most reasonable todo list I could come up with is following:

Building wry android support with one Activity with one WebView tightly coupled. And you won't be able to configure anything after creating it.

Add init/eval_script method (though eval will be useless until window library is done)
Add IPC handler
Add custom protocol

Implement a some sort of Android window management library using Application.ActivityLifecycleCallbacks

...I don't know what will be the blockers yet lol

Do you think this will be the reasonable progress? Any feedback is welcome!

Afaik, Activities on android has to be declared in the manifest and can't be created at runtime so I don't think multi-window or multi-activity WRY app is possible and probably shouldn't.
I guess we could make a macro the defines an entry point for an activity and we create these activities manifest at compile time.
I think Mobile support in general should be one window and use a client-side router or any kind of web navigation.

@amrbashir Yes, Activity has to be registered to manifest and cannot be created dynabmically during runtime.
But searching a bit, I still saw lots of issue about multiple webviews on android development. So I just want to take this into consideration just in case.
One possibility is running build script macro to edit the AndroidManifest.xml
Or if that's not desirable, perhaps we can narrow to one activity with multiple webviews.

Or if that's not desirable, perhaps we can narrow to one activity with multiple webviews.

We can probably use fragments, they can be dynamically created and I think multiple activities are more resource hungry than using multiple fragments in on activity. Jetpack Compose, which is the new modern toolkit to build native android app using kotlin, also uses fragments.

Quick update for some great news. Init_script and IPC handling are possible!
https://github.com/wusyong/android_webview.rs/commit/2b221c065f51a0005521d036f6fd0dad0e1775e8
The last thing would be custom protocol. It should also be possible sine android has a guide explicit for it.
I'll give it a try next week.

Continued in..

#510

What's the current status of the issue? Is anyone working on it? What would a complete newbie to the project need to know to help with the issue?

@Adhalianna we're actively working on it, see https://github.com/tauri-apps/cargo-mobile and the next branch in tauri. Also check out our mobile channel on Discord.

The sad thing is we can't create a NDK and use NativeActivity because it doesn't have Looper implemented by default and WebView will reject it upon creating.

Can you elaborate on this? I don't quite understand why you can't subclass NativeActivity and add a webview plus some additional methods to interface with the webview. Saddly the way it's currently handled there is no way to build without a complex build system like gradle, even after building a maven clone in rust and shelling out to kotlin/r8, since the AppCompatActivity requires support for res/drawables/etc.
This is how far I've got with trying to get xbuild to build a wry app for android:
 x run --device adb:16ee50bc
[1/3] Fetch precompiled artefacts
info: component 'rust-std' for target 'aarch64-linux-android' is up to date
[1/3] Fetch precompiled artefacts [1564ms]
[2/3] Build rust
    Finished dev [unoptimized + debuginfo] target(s) in 0.12s
[2/3] Build rust [159ms]
[3/3] Create apk [21007ms]
/home/dvc/rosetta-wallet/target/x/debug/android/dioxus-wall...le pushed, 0 skipped. 92.7 MB/s (114760019 bytes in 1.181s)
Success
Starting: Intent { act=android.intent.action.RUN cmp=com.example.dioxus_wallet/.TauriActivity }
pid of com.example.dioxus_wallet is 10822
--------- beginning of main

10822 10822 I e.dioxus_walle: Late-enabling -Xcheck:jni
10822 10822 I e.dioxus_walle: Unquickening 18 vdex files!
10822 10822 W e.dioxus_walle: Can't mmap dex file /data/app/~~Te3ceWAfiF2TYGsUCY8lRA==/com.example.dioxus_wallet-Dr1fKS2N2lPyWiRie5_ZFA==/base.apk!classes.dex directly; please zipalign to 4 bytes. Falling back to extracting file.
10822 10822 I Perf    : Connecting to perf service.
10822 10822 D NetworkSecurityConfig: No Network Security Config specified, using platform default
10822 10822 D NetworkSecurityConfig: No Network Security Config specified, using platform default
10822 10822 D AndroidRuntime: Shutting down VM
--------- beginning of crash

10822 10822 E AndroidRuntime: FATAL EXCEPTION: main
10822 10822 E AndroidRuntime: Process: com.example.dioxus_wallet, PID: 10822
10822 10822 E AndroidRuntime: java.lang.NoClassDefFoundError: Failed resolution of: Landroidx/appcompat/R$drawable;
10822 10822 E AndroidRuntime: 	at androidx.appcompat.widget.f0.<init>(Unknown Source:6)
10822 10822 E AndroidRuntime: 	at androidx.appcompat.widget.g0.i(Unknown Source:26)
10822 10822 E AndroidRuntime: 	at androidx.appcompat.app.r0.<init>(Unknown Source:94)
10822 10822 E AndroidRuntime: 	at androidx.appcompat.app.r0.<init>(Unknown Source:1)
10822 10822 E AndroidRuntime: 	at androidx.appcompat.app.N.i(Unknown Source:2)
10822 10822 E AndroidRuntime: 	at androidx.appcompat.app.K.u0(Unknown Source:4)
10822 10822 E AndroidRuntime: 	at androidx.appcompat.app.K.attachBaseContext(Unknown Source:0)
10822 10822 E AndroidRuntime: 	at android.app.Activity.attach(Activity.java:7899)
10822 10822 E AndroidRuntime: 	at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:3384)
10822 10822 E AndroidRuntime: 	at android.app.ActivityThread.handleLaunchActivity(ActivityThread.java:3596)
10822 10822 E AndroidRuntime: 	at android.app.servertransaction.LaunchActivityItem.execute(LaunchActivityItem.java:85)
10822 10822 E AndroidRuntime: 	at android.app.servertransaction.TransactionExecutor.executeCallbacks(TransactionExecutor.java:135)
10822 10822 E AndroidRuntime: 	at android.app.servertransaction.TransactionExecutor.execute(TransactionExecutor.java:95)
10822 10822 E AndroidRuntime: 	at android.app.ActivityThread$H.handleMessage(ActivityThread.java:2067)
10822 10822 E AndroidRuntime: 	at android.os.Handler.dispatchMessage(Handler.java:106)
10822 10822 E AndroidRuntime: 	at android.os.Looper.loop(Looper.java:223)
10822 10822 E AndroidRuntime: 	at android.app.ActivityThread.main(ActivityThread.java:7715)
10822 10822 E AndroidRuntime: 	at java.lang.reflect.Method.invoke(Native Method)
10822 10822 E AndroidRuntime: 	at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:592)
10822 10822 E AndroidRuntime: 	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:952)
10822 10822 E AndroidRuntime: Caused by: java.lang.ClassNotFoundException: Didn't find class "androidx.appcompat.R$drawable" on path: DexPathList[[zip file "/data/app/~~Te3ceWAfiF2TYGsUCY8lRA==/com.example.dioxus_wallet-Dr1fKS2N2lPyWiRie5_ZFA==/base.apk"],nativeLibraryDirectories=[/data/app/~~Te3ceWAfiF2TYGsUCY8lRA==/com.example.dioxus_wallet-Dr1fKS2N2lPyWiRie5_ZFA==/lib/arm64, /data/app/~~Te3ceWAfiF2TYGsUCY8lRA==/com.example.dioxus_wallet-Dr1fKS2N2lPyWiRie5_ZFA==/base.apk!/lib/arm64-v8a, /system/lib64, /system_ext/lib64]]
10822 10822 E AndroidRuntime: 	at dalvik.system.BaseDexClassLoader.findClass(BaseDexClassLoader.java:207)
10822 10822 E AndroidRuntime: 	at java.lang.ClassLoader.loadClass(ClassLoader.java:379)
10822 10822 E AndroidRuntime: 	at java.lang.ClassLoader.loadClass(ClassLoader.java:312)
10822 10822 E AndroidRuntime: 	... 20 more
10822 10822 I Process : Sending signal. PID: 10822 SIG: 9

@wusyong is this the issue you encountered? https://stackoverflow.com/questions/72767602/creating-android-webview-in-nativeactivity-with-jni-in-rust

well, there are problems, but at least the one in the thread can be fixed by calling Looper.prepare() before creating the WebView. will see if I can get it working, if not I guess the proposed approach might be necessary.
// Copyright 2020-2022 Tauri Programme within The Commons Conservancy
// SPDX-License-Identifier: Apache-2.0
// SPDX-License-Identifier: MIT
use super::{WebContext, WebViewAttributes, RGBA};
use crate::{application::window::Window, Result};
use jni::objects::JObject;
use ndk::looper::ThreadLooper;
use std::rc::Rc;

pub(crate) struct InnerWebView {
  #[allow(unused)]
  pub window: Rc<Window>,
}

impl InnerWebView {
  pub fn new(
    window: Rc<Window>,
    attributes: WebViewAttributes,
    _pl_attrs: super::PlatformSpecificWebViewAttributes,
    _web_context: Option<&mut WebContext>,
  ) -> Result<Self> {
    let context = ndk_context::android_context();
    let vm = unsafe { jni::JavaVM::from_raw(context.vm().cast()) }?;
    let env = vm.attach_current_thread()?;
    let ctx = unsafe { JObject::from_raw(context.context().cast()) };

    let class = env.find_class("android/os/Looper")?;
    env.call_static_method(class, "prepare", "()V", &[])?.v()?;
    println!("Looper.prepare()");

    let class = env.find_class("android/webkit/WebView")?;
    let webview = env.new_object(class, "(Landroid/content/Context;)V", &[ctx.into()])?;
    println!("WebView webview = new WebView(ctx)");

    if let Some(url) = &attributes.url {
        let url = env.new_string(url)?;
        env.call_method(
            webview,
            "loadUrl",
            "(Ljava/lang/String;)V",
            &[url.into()]
        )?;
        println!("webview.loadUrl(url)");
    }

    /*env
      .call_method(
        ctx,
        "setContentView",
        "(Landroid/view/View;)V",
        &[webview.into()],
      )?
      .v()?;
    println!("setContentView(webview)");*/
    Ok(InnerWebView { window })
  }

  pub fn print(&self) {}

  pub fn eval(&self, _js: &str) -> Result<()> {
    Ok(())
  }

  #[cfg(any(debug_assertions, feature = "devtools"))]
  pub fn open_devtools(&self) {}

  #[cfg(any(debug_assertions, feature = "devtools"))]
  pub fn close_devtools(&self) {}

  #[cfg(any(debug_assertions, feature = "devtools"))]
  pub fn is_devtools_open(&self) -> bool {
    false
  }

  pub fn zoom(&self, _scale_factor: f64) {}

  pub fn set_background_color(&self, _background_color: RGBA) -> Result<()> {
    Ok(())
  }
}

pub fn platform_webview_version() -> Result<String> {
  Ok("1".into())
}

@dvc94ch I believe we have done this before. The result is Webview implementation doesn't work well with NDK.
What you see we do now is using regular Activity. Winit itself is also pushing an android-activity implementation that could work well with any kind of activity.

Got it working with gradle. Is the android-activity winit backend going to be adopted by tao? Currently tao does it's own weird thing and seems to be tightly coupled with and built specifically for wry. That would make experimenting with the wry android backend easier and tooling to not have to add special support for tao/wry.

Yes, itwill be the backend after it's merged in winit. The setup from tao is to connect to our project template from cargo-mobile fork.

@dvc94ch got a repo/example anywhere of an Android Gradle project that uses wry?

not really, xbuild handles this all in the background. you can have a look at target/x/debug/android/gradle after running the following commands:
cargo install --git https://github.com/cloudpeers/xbuild
x new template
cd template
x build --platform android --arch arm64

is there a reason why on android doesn't support custom protocols? can they be supported?

I'm not quite following what you implemented and why. I registered a custom asset handler that works on all platforms except android. what I'd like to do is this:
use include_dir::{include_dir, Dir};

pub static ASSETS: Dir<'_> = include_dir!("$CARGO_MANIFEST_DIR/assets");

pub fn asset_handler(request: &Request<Vec<u8>>) -> Result<Response<Vec<u8>>> {
    let path = request.uri().to_string().replace("asset://", "");
    let mime = match path.rsplit_once('.') {
        Some((_, "css")) => "text/css",
        Some((_, "png")) => "image/png",
        _ => "application/octet-stream",
    };
    let file = if let Some(file) = ASSETS.get_file(path) {
        file
    } else {
        return Response::builder()
            .status(StatusCode::NOT_FOUND)
            .body(b"Not Found".to_vec())
            .map_err(From::from);
    };
    Response::builder()
        .header("Content-Type", mime)
        .body(file.contents().to_vec())
        .map_err(From::from)
}

sorry for being thick. I managed to get it working like this:
macro_rules! img {
    ($file:literal) => {{
        #[cfg(target_family = "wasm")]
        let url = concat!("img/", $file);
        #[cfg(target_os = "android")]
        let url = concat!("https://asset.localhost/img/", $file);
        #[cfg(not(any(target_family = "wasm", target_os = "android")))]
        let url = concat!("asset://localhost/img/", $file);
        url
    }};
}

It should work. On tauri we do request.uri().strip_prefix("tauri://localhost").

Closing as android support have been out for a while now.


waydroid: 1.3.4 -> 1.4.0

Description of changes
Waydroid update to 1.4.0.
Things done

Built on platform(s)

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

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

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

[x] 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
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

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

[x] Fits CONTRIBUTING.md.

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

1 package built:

waydroid

https://github.com/waydroid/waydroid/commit/574405d056f08aae90769c7515d8cccf78b1fec9 replace the alsolute icon path with a relative one. The patch on the desktop files can be removed.

https://github.com/waydroid/waydroid/pull/701 which is removed.

@linsui, thank you! I remove unnecessary code.

Hi, @AtaraxiaSjel in the mean time, Waydroid 1.4.1 has been released. Could you squash the following patch and modify the PR title? (I already tested it):
diff --git a/pkgs/os-specific/linux/waydroid/default.nix b/pkgs/os-specific/linux/waydroid/default.nix
index aa348ba5062..2cf6e0c47e4 100644
--- a/pkgs/os-specific/linux/waydroid/default.nix
+++ b/pkgs/os-specific/linux/waydroid/default.nix
@@ -18,14 +18,14 @@

 python3Packages.buildPythonApplication rec {
   pname = "waydroid";
-  version = "1.4.0";
+  version = "1.4.1";
   format = "other";

   src = fetchFromGitHub {
     owner = pname;
     repo = pname;
     rev = version;
-    sha256 = "sha256-zm5CIJd4FBWHRVNT4ObuznI6+8+ACqunQ1g35OcESno=";
+    sha256 = "sha256-0AkNzMIumvgnVcLKX72E2+Eg54Y9j7tdIYPsroOTLWA=";
   };

   buildInputs = [

@CajuM, thank you! Done and rebased.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/2248


Run autocannon twice to let V8 optimizing compiler do its job

As titled

You said that, we should work twice test, am I right?

yes, and discard the first results.


Can't change mainOverlay opacity in tweenHandler

I want the main content to go dark when the drawer is open by setting the mainOverlay background to black and opacity to 0 and in tweenHandler tried to change the opacity but it doesn't seem to work:
<Drawer
        ref="navigation"
        type="displace"
        content={<SideMenu />}
        tapToClose={true}
        openDrawerOffset={0.2}
        panCloseMask={0.2}
        styles={{mainOverlay: {backgroundColor: '#000000', opacity: 0}}}
        negotiatePan={true}
        tweenHandler={(ratio) => ({
          mainOverlay: { opacity: ( ratio / 2 )}
        })}
      >

Is this a bug or am I doing it wrong?
Thanks.

This works.
tweenHandler={(ratio) => ({
    mainOverlayProps: { opacity: ( ratio / 2 )}
})}

Ah, a bug ideed. Short term workaround: enable captureGestures

I cut a pre-release with a fix for this and some other improvements as well available via npm i react-native-drawer@next
Let me know if that works for you

Zack,
It’s perfect. Thank you.
Yong

On Apr 23, 2016, at 12:19 PM, Zack Story<EMAIL_ADDRESS>wrote:
I cut a pre-release with a fix for this and some other improvements as well available via npm i react-native-drawer@next
Let me know if that works for you
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub https://github.com/root-two/react-native-drawer/issues/145#issuecomment-213812065

:+1:

Don't know if anyone is still needing help, but I figured out how to make the area behind the drawer go darker with opacity.
                tweenHandler={(ratio) => ({                   mainOverlay:{ backgroundColor:"rgba(71, 71, 71,"+ (ratio)/1.5+")"},                 })}
I just used rgba (which is supported by react native. Runs on both iOS and Android) and used the ratio to control the opacity. Hope that helps! @rt2zz @Qizly

@garrettbland Thank you very much :)


[Feature Request] Add a contact form to the contact widget

A form, to contact the owner could be a nice addition to the contact widget. And although we cannot do this on a static site without a backend, it's possible to use third-party services (like Formspree)
I have modified the contact.html partial to implement this and this is how it looks on my website
The form element that I added in the partial contact.html:
{{ with $.Site.Params.email }}

Let's get in touch. Send me a message:

    <form action="https://formspree.io/{{ . }}" method="POST">
        <input class="form-element-input" type="text" name="name" required="required" placeholder="Name">

        <input class="form-element-input" type="email" name="email" required="required" placeholder="Email">

        <textarea class="form-element-input" rows="3" name="message" placeholder="Message" required="required"></textarea>

        <input class="form-element-submit" type="submit" value="Send">
        <input type="hidden" name="_subject" value="New mail from Site" />
        <input type="text" name="_gotcha" style="display:none" />
    </form>

{{ end }}

For further details, visit the vendor's site
And this is the css for the form:
.form-element-input {
  width: 90%;
  padding: 8px 14px;
  margin: 6px 0;
  display: inline-block;
  border: 1px solid #ccc;
  border-radius: 4px;
  box-sizing: border-box;
}

.form-element-submit {
  width: 90%;
  background-color: #1d2731;
  color: #fff;
  padding: 10px 16px;
  margin: 6px 0;
  border: none;
  border-radius: 4px;
  cursor: pointer;
}

I believe that a form like this could be a unique feature to have, and it could be implemented easily by using the underlying features.

Thanks for the suggestion. This feature is on our roadmap.
The downside with Formspree is that the feature is not much different to just provided your email address in the normal way as Formspree by default still requires your email address in the page code, which bots can then harvest as usual. You would currently have to pay them $120/yr just to hide your email address from bots...

I totally agree with you, this form feature could not be categorized as a 'must have' requirement. Although it is good to know that it's development is being planned. 👍

FYI it seems that there exists a feature from netlify to do that https://www.netlify.com/docs/form-handling/ may be it can be useful.


RHost: Replace font width feature for XAML plots

X11 XTextWidth may be?

Since we don't officially support XAML plots, we can probably postpone this for later.

No plans to support them


Updated donations link

Our application to the Gratipay was rejected for now. Unfortunately, I don't currently have the time to perform a rename of all our assets (e.g. repo, screenshots, Twitter account).
As a temporary stop-gap (and to start seeing how many people are interested in donating), we are adding in a bit.ly link to a Flattr page for now.
In this PR:

Updated donation link to use bit.ly pointing to Flattr

/cc @rpdelaney

Going to cowboy land this for now

Works for me.

Cool, thanks for the speedy approval =)


Add support for an equiv to GUI Extension's customconfiguration element.

Add ability for a plugin to have custom configuration that is equivalent to the GUI Extension editor's customconfiguration element.
Should be easily readable on client as well as server.

Documentation of implementing this feature in your plugins can be found at:
http://a4t-api.alchemywebstore.com/html/a155bb48-95ab-49c9-9b61-ebcc119dc2dd.htm


Is there is type error.

Thank for sharing the code. when i run your code in spyder IDE it give me error in the "class ChannelAttention" class. How to remove this error. Thank you.
Bellow is the visual of your code.

Hi @Altaf-hucn,
I'm really sorry for that little bug. I'm gonna correct it immediately.
If you refer to the CBAM architecture, you notice that the input goes through two parallel paths: the Global Max Pooling and Global Average Pooling ones. So, in the notebook, x should be input.

Ok, now it should work just fine!
Good luck for your work!


WIP: Controlled instancing

This is a proposal for conversation: What if we were able to strictly control which schema types could be instanced, and which could not?

I can see that one might choose not to allow instancing of certain types of objects, but might it not depend on where the instancing is?  Specifically, you might want to say a SerializableCollection has to hold distinct objects.  But maybe it's useful to allow metadata, which is unstructured, to be able to point at a particular object?   In which case, controlling instancing based on the type, rather than whom the referer is, is not what you want.   Should metadata continue to be allowed to be the wild wild west?  Offhand, I'd say yeah, that's what metadata is for...

Actually, I'd like to see a case of where instancing needs to be held in check, because allowing it creates badness.  Understanding the reasons for needing to control it (if in fact we do) might suggest a targeted path to how to do it.

Yeah, I'm not sure if restricting it at the type level is the right place. We'll have to think about that more.
Here are use cases where it probably makes sense:
MediaReferences: Several people have asked for instanced MediaReferences. This makes sense in that all of the clips that point to the same media could share the same MediaReference in the OTIO data structure and you don't have to worry about divergent metadata on each one.
Inputs to video effects: When we eventually add support for video effects that perform compositing, we may want an Effect to point to other Clips or MediaReferences that are used as elements of the composite (e.g. keys, channels, etc.) This might be a case where the source of the reference, not the type pointed to is important. Modeling effects like this is on the roadmap, but we haven't done the research to propose anything specific yet.
Here are cases where it would be problematic:
Composable: For example a Clip that appears in a Track twice, or a single Clip that appears in multiple Tracks. Since a Clip has a reference to its parent, the first case might sort of work (they both have the same parent), but the second does not. If we allowed multiple parents, then it could be made to work, but a variety of common use cases become more difficult to handle generally. For example, making a Clip longer will make the other instances longer - something that many users of OTIO might not expect. It also complicates any questions we ask about what range of time a Clip occupies in a parent Composition since we might have to disambiguate the two (or more) instances.
Instanced RationalTime or TimeRange: They are immutable, so this is relatively safe, but what would this mean and why would you want them instanced?
Instanced Effects or Markers: What would this mean and why would you want them instanced?
The argument for most of these is just "it would be complicated and confusing". Adding the extra burden of "what if this object is used in multiple places?" to many run of the mill use cases would make working with OTIO more complex - something we're trying to avoid unless there is a clear gain.

Composable: For example a Clip that appears in a Track twice, or a single Clip that appears in multiple Tracks.

The composition mechanism currently screens this out, right?  I can’t put the same clip in more than one track because it’s an error at the Composition level already when I go to insert it as a child.
I’m saying that the explicit parent/child mechanism we have for Composition catches many of the cases you want to rule out.  What’s a case not caught by Compsition that also needs to be disallowed?

Since a Clip has a reference to its parent, the first case might sort of work (they both have the same parent), but the second does not. If we allowed multiple parents, then it could be made to work, but a variety of common use cases become more difficult to handle generally. For example, making a Clip longer will make the other instances longer - something that many users of OTIO might not expect. It also complicates any questions we ask about what range of time a Clip occupies in a parent Composition since we might have to disambiguate the two (or more) instances.
Instanced RationalTime or TimeRange: They are immutable, so this is relatively safe, but what would this mean and why would you want them instanced?

FYI, instancing is only for SerializableObjects.  There is no instancing of RationalTime, TimeTransform, or TimeRange because these are value types now (in C++).  Instancing them would be plain dumb, and unsupportable in general.  You’ll notice that these types don’t pump out a REF_ID field.
Instanced Effects or Markers: What would this mean and why would you want them instanced?
not sure.  if effects and markers must be “owned” by someone, maybe they want an owner field, so they can’t be given to multiple owners?
If I’ve some sort of custom effect, and I want to use it multiple times, but not keep track of changing it multiple times because I had to copy it, is that a valid use case?

The argument for most of these is just "it would be complicated and confusing". Adding the extra burden of "what if this object is used in multiple places?" to many run of the mill use cases would make working with OTIO more complex - something we're trying to avoid unless there is a clear gain.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub https://github.com/PixarAnimationStudios/OpenTimelineIO/pull/509#issuecomment-486384513, or mute the thread https://github.com/notifications/unsubscribe-auth/ADAWJJMBXE5ZDDJCMXOFJTDPSCVJPANCNFSM4HIFOSTQ.


Include the overlay part in the tab focus order

Fixes #68

This change is

src/vaadin-overlay.html, line 374 at r1 (raw file):

          Polymer.RenderStatus.afterNextRender(this, () => {
            if (this.focusTrap) {
              this._cycleTab(0, 0);

If you add tabindex="1" to an element inside the overlay, it still gets focused (instead of the overlay):
<vaadin-overlay focus-trap id="overlay2" >
  <template>
    <input tabindex="1"></input>
  </template>
</vaadin-overlay>

Comments from Reviewable

Review status: 0 of 2 files reviewed at latest revision, 1 unresolved discussion.

src/vaadin-overlay.html, line 374 at r1 (raw file):
Previously, tomivirkki (Tomi Virkki) wrote…
If you add tabindex="1" to an element inside the overlay, it still gets focused (instead of the overlay):
<vaadin-overlay focus-trap id="overlay2" >
  <template>
    <input tabindex="1"></input>
  </template>
</vaadin-overlay>

This is a feature. A tabindex="1" is expected to focus before default tabindex="0".
Consider:
<vaadin-overlay focus-trap id="overlay2" >
  <template>
    <input tabindex="1"></input>
    <button tabindex="0">button</button>
  </template>
</vaadin-overlay>

All the elements are in the same DOM scope, hence all tabindex values have same level of effect.
After the user opens the overlay, if the overlay is focused instead of input, then the next tab will focus the button, because it’s a next element of the same tabindex of 0. This would be unexpected, because with tabindex="1" the input is expected to gain focus before everything else of tabindex="0".
Btw the HTML spec says: “Developers should use caution when using values other than 0 or −1 for their tabindex attributes as this is complicated to do correctly.”

Comments from Reviewable

src/vaadin-overlay.html, line 374 at r1 (raw file):
Previously, platosha (Anton Platonov) wrote…
This is a feature. A tabindex="1" is expected to focus before default tabindex="0".
Consider:
<vaadin-overlay focus-trap id="overlay2" >
  <template>
    <input tabindex="1"></input>
    <button tabindex="0">button</button>
  </template>
</vaadin-overlay>

All the elements are in the same DOM scope, hence all tabindex values have same level of effect.
After the user opens the overlay, if the overlay is focused instead of input, then the next tab will focus the button, because it’s a next element of the same tabindex of 0. This would be unexpected, because with tabindex="1" the input is expected to gain focus before everything else of tabindex="0".
Btw the HTML spec says: “Developers should use caution when using values other than 0 or −1 for their tabindex attributes as this is complicated to do correctly.”

Ok :+1:

Comments from Reviewable

Reviewed 2 of 2 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Reviewed 2 of 2 files at r1.
Review status: :shipit: all files reviewed at latest revision, all discussions resolved, all commit checks successful.

Comments from Reviewable


Review: Status.jsx and DispenseMedication.jsxReview:

Review Branch

review-79

Files to be Reviewed

Status.jsx
DispenseMedication.jsx

Checklists

Architecture
Design
JS
React
UI

Specific Requests for this request
Reviewing the Status page as there should now be pagination, filtering, and sorting.
Reviewing the Dispense Medication page as there should now be filtering.
Due Date
12:00 pm Tuesday, November 2 2021
For more information
The review process is documented at: http://courses.ics.hawaii.edu/ics414s21/morea/review/reading-idpm-review.html

Status.jsx
Architecture

AR-01: Meets Requirement
AR-02: Meets Requirement
AR-03: Meets Requirement

Design
DE-01: Meets Requirement
DE-02: L75-L123 Nested if statement, maybe you can do an “||” operator to narrow it down to one “if statement”
DE-03: Meets Requirement
DE-04: Meets Requirement
DE-05: Meets Requirement
DE-06: Meets Requirement
DE-07: Meets Requirement
DE-08: Meets Requirement

Javascript and ESLint Checklist
Javascript
JS-01: Meets Requirement
JS-02: Not Applicable
JS-03: Meets Requirement
JS-04: Meets Requirement
JS-05: Meets Requirement
JS-06: Meets Requirement
JS-07: Meets Requirement
ESLint
ES-01: Meets Requirement
ES-02: Meets Requirement
React Checklist
RE-01: Meets Requirement
RE-02: Not Applicable
RE-03: Meets Requirement
RE-04: Meets Requirement
RE-06: Meets Requirement
RE-07: Meets Requirement
RE-08: Meets Requirement
RE-09: Meets Requirement
RE-10: Meets Requirement
UI
UI-01: Meets Requirement
UI-02: Meets Requirement
UI-03: Meets Requirement
UI-04: Meets Requirement
UI-05: Meets Requirement
UI-06: Meets Requirement
UI-07: Meets Requirement
==============================================================================
DispenseMedication.jsx
Architecture
AR-01: Meets Requirement
AR-02: Meets Requirement
AR-03: Meets Requirement
Design
DE-01: Meets Requirement
DE-02: Meets Requirement
DE-03: Meets Requirement
DE-04: Meets Requirement
DE-05: Meets Requirement
DE-06: Meets Requirement
DE-07: Meets Requirement
DE-08: Meets Requirement
Javascript and ESLint Checklist
Javascript
JS-01: Meets Requirement
JS-02: Not Applicable
JS-03: Meets Requirement
JS-04: Meets Requirement
JS-05: Meets Requirement
JS-06: Meets Requirement
JS-07: Meets Requirement
ESLint
ES-01: Meets Requirement
ES-02: Meets Requirement
React Checklist
RE-01: Meets Requirement
RE-02: Meets Requirement
RE-03: Meets Requirement
RE-04: Meets Requirement
RE-06: Meets Requirement
RE-07: Meets Requirement
RE-08: Meets Requirement
RE-09: Meets Requirement
RE-10: Meets Requirement
UI
UI-01: Meets Requirement
UI-02: Meets Requirement
UI-03: Meets Requirement
UI-04: Meets Requirement
UI-05: Meets Requirement
UI-06: Meets Requirement
UI-07: Meets Requirement

Status.jsx
Should this be return false? 0 is falsey, but it isn't clear.
https://github.com/minerva-medical/minerva-matrp/blob/bca125e307e80f9a8414d58028e73346b63f494c/app/imports/ui/pages/Status.jsx#L82
Can you filter by more than one thing at a time? It doesn't look like
it.
DispenseMedication.jsx
Why are we updating the name, brand, etc. Can't we just set quantity
to 0?
https://github.com/minerva-medical/minerva-matrp/blob/bca125e307e80f9a8414d58028e73346b63f494c/app/imports/ui/components/DispenseMedication.jsx#L38
Use consts in withTracker
https://github.com/minerva-medical/minerva-matrp/blob/bca125e307e80f9a8414d58028e73346b63f494c/app/imports/ui/components/DispenseMedication.jsx#L220

##Status.jsx
Architecture
OK
Design
DE-02: L73-149: use switch statements instead of nested if statements
DE-09: L73-149: filtering should be handled in a function
JS
OK
React
L46-47: variables should be moved to state
UI
Dropdown search query is squished
##DispenseMedication.jsx
Architecture
OK
Design
OK
JS
OK
React
RE-06: define L222-227 as const in withTracker()
UI
OK

Status.jsx:
Architecture
AR-01: file name is not in camelCase
Design
Fulfills all design requirements
Javascript/ESLint
Fulfills all JS/ESLint requirements
React
Fulfills all react requirements
UI
UI-02: Layout does not work as expected on mobile-sized screen (table doesn’t look right)
UI-03: the dropdowns do not make the mouse give the clicker icon (they give a cursor for typing instead)
DispenseMedication.jsx:
Architecture
AR-01: file name is not in camelCase
Design
Fulfills all design requirements
Javascript/ESLint
Fulfills all JS/ESLint requirements
React
Fulfills all react requirements
UI
Fulfills all UI requirements

Status.jsx
DE-02: 73-78 Is there a better way to filter this?
UI-01: The goal of the page is clear, but its kind of long. Maybe we can add comments briefly explaining some the functions on this page?
DispenseMedication.jsx
UI-01: The goal of the page is clear, but its kind of long. Maybe we can add a few more comments briefly explaining some the functions. Other than that, comments are helpful in differentiating the purpose of each section of code

Status.jsx
Architecture:
AR-01 - AR-3: Meet Requirements
Design:
DE-01, DE-03 - DE-07:  All good, requirements met.
DE-02:
- Lines 75 - 123: nested if statement can be rewritten
JavaScript & ESLint:
JS-01- JS-07: All good, requirements met.
ES-01/02: All good, requirements met.
React:
RE-01 - RE-10: All good, requirements met.
UI:
UI-01, UI-03 - UI-07:  All good, requirements met.
UI-02: Different screen sizes could still be adjusted
DispenseMedication.jsx
Architecture:
AR-01 - AR-03:  All good, requirements met.
Design:
DE-01 - DE-08:  All good, requirements met.
JavaScript & ESLint:
JS-01- JS-07: All good, requirements met.
ES-01/02: All good, requirements met.
React:
RE-01 - RE-10: All good, requirements met.
UI:
UI-01 - UI-07:  All good, requirements met.

Architecture:

All good

Design:

nested if statement

Javascript:

All good

React:

All good

UI:

All good

DispenseMedication.jsx
Architecture:

All good

Design:

All good

Javascript:

All good

React:

All good

UI:

All good


Latest version update issue

The CDN URL returns an application/octet-stream content type (e.g.: ��88.0.705.81\r\n) the download of latest version fails, since it can't create a tmp folder with this name. So instead of a simple trim, we need to sanitize the returned data.

Hello,
The msedgedriver does not download the latest version of msedge driver. I have edge version at 89.0.774.63 but during install it always seems to download 87.0.664.66. How do I set it to download the version that's been installed on the machine?
I have also tried installing using npm install msedgedriver --edgechromiumdriver_version=LATEST. I am getting error.
C:\Work\AWeb>npm install msedgedriver --edgechromiumdriver_version=LATEST
<EMAIL_ADDRESS>install C:\Work\AdvantageWeb\node_modules\msedgedriver
node install.js
:\Users\harish\AppData\Local\Temp\��8 9 . 0 . 7 7 4 . 6 3
\msedgedriver is not writable: The argument 'path' must be a string or Uint8Array without null bytes. Received 'C:\Users\harish\AppData\Local\Temp\��8\x009\x00.\x000\x00.\x007\x007\x004\x00.\x006\x003\x00\r\x00\n\x00\msedgedriver'
:\Users\harish\AppData\Local\Temp\��8 9 . 0 . 7 7 4 . 6 3
\msedgedriver is not writable: The argument 'path' must be a string or Uint8Array without null bytes. Received 'C:\Users\harish\AppData\Local\Temp\��8\x009\x00.\x000\x00.\x007\x007\x004\x00.\x006\x003\x00\r\x00\n\x00\msedgedriver'
tmp\��8 9 . 0 . 7 7 4 . 6 3
\msedgedriver is not writable: The argument 'path' must be a string or Uint8Array without null bytes. Received 'C:\tmp\��8\x009\x00.\x000\x00.\x007\x007\x004\x00.\x006\x003\x00\r\x00\n\x00\msedgedriver'
:\Work\AdvantageWeb\node_modules\msedgedriver\tmp\��8 9 . 0 . 7 7 4 . 6 3
\msedgedriver is not writable: The argument 'path' must be a string or Uint8Array without null bytes. Received 'C:\Work\AdvantageWeb\node_modules\msedgedriver\tmp\��8\x009\x00.\x000\x00.\x007\x007\x004\x00.\x006\x003\x00\r\x00\n' +
...
Can not find a writable tmp directory, please report issue on https://github.com/uex-io/node-msedgedriver/issues/ with as much information as possible.
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>install: node install.js
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
Looks like there is an issue with the code.
Thanks and Regards,
Harish Rao

Would be good to get this merged if it's working, as the package being downloaded is quite out of date at this point.


Random Shuffle

Done. c.f. https://github.com/alvations/komorebi/commit/04df24cce68a47de667d8b869720063233efb83f

Usage
>>> from komorebi import ParallelData
>>> sgfile = 'data/kopitiam/kopitiam.sg'
>>> enfile = 'data/kopitiam/kopitiam.en'
>>> x = ParallelData(sgfile, enfile)
Building source vocab and counter... populate_dictionary took 00:00:00
Building target vocab and counter... populate_dictionary took 00:00:00
Filtering least frequent words in vocab. >>>
>>>
>>> for sg, en in x.shuffle():
...     print(sg)
...     print(en)
...     break
...
Variable containing:
 0
 3
 5
 1
[torch.LongTensor of size 4x1]

Variable containing:
    0
    3
    4
    6
    9
    8
    1
[torch.LongTensor of size 7x1]


Add python launch files and modfiy default_ad_api

Add python launch files for packages as follow: external_velocity_limit_selector, mission_planner, motion_velocity_smoother,  planning_validator,  scenario_selector and rtc_auto_mode_manager
Modify default_ad_api package (only keep routing node)

@cszeng There are conflicts between your changes and the target branch (drive). Please rebase your branch on drive to resolve these conflicts:
git checkout drive
git pull
git checkout fix-planning-incompatibility
git rebase drive
... fix conflicts ...
git rebase --continue
... continue to fix and --continue until all are resolved ...
git push -f

This branch is based upon PR #6 rather than MR #5. I think that's what causes the conflicts. If I rebase drive and fix the conflicts, it seems that the branch will be based upon MR #5.

@cszeng I reverted the drive branch to before #5. I think this is ready to go. Thanks!


[Bug] The 'class-modifiers' language feature is disabled for this library

Describe the bug
Im not sure what's wrong, all builds are successful, but when I run command flutter run, it throws an error:
    Finished dev [unoptimized + debuginfo] target(s) in 34.26s
ERROR: lib/bridge_generated.io.dart:214:1: Error: The 'class-modifiers' language feature is disabled for this library.
ERROR: Try removing the package language version or setting the language version to 3.0 or higher.
ERROR: final class _Dart_Handle extends ffi.Opaque {}
ERROR: ^^^^^
ERROR: lib/bridge_generated.io.dart:216:1: Error: The 'class-modifiers' language feature is disabled for this library.
ERROR: Try removing the package language version or setting the language version to 3.0 or higher.
ERROR: final class wire_uint_8_list extends ffi.Struct {
ERROR: ^^^^^
ERROR: Target kernel_snapshot failed: Exception
Building Linux application...
Exception: Build process failed

Codegen logs with RUST_LOG=debug environment variable
RUST_LOG=debug flutter_rust_bridge_codegen --rust-input native/src/api.rs --dart-output lib/bridge_generated.dart --c-output ios/Runner/bridge_generated.h --extra-c-output-path macos/Runner/ --dart-decl-output lib/bridge_definitions.dart --wasm

To Reproduce
No response
Expected behavior
No response
Generated binding code
No response
OS
Arch Linux - Kernel: Linux 6.4.7-arch1-1
Version of flutter_rust_bridge_codegen
1.79.0
Flutter info
[✓] Flutter (Channel stable, 3.10.6, on Arch Linux 6.4.7-arch1-1, locale en_US.UTF-8)
    • Flutter version 3.10.6 on channel stable at /home/thaodt/projects/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision f468f3366c (3 weeks ago), 2023-07-12 15:19:05 -0700
    • Engine revision cdbeda788a
    • Dart version 3.0.6
    • DevTools version 2.23.1

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /home/thaodt/Android/Sdk
    • Platform android-34, build-tools 34.0.0
    • Java binary at: /opt/android-studio/jbr/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b829.9-10027231)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • CHROME_EXECUTABLE = /usr/bin/google-chrome-stable

[✓] Linux toolchain - develop for Linux desktop
    • clang version 15.0.7
    • cmake version 3.27.1
    • ninja version 1.11.1
    • pkg-config version 1.8.1

[✓] Android Studio (version 2022.3)
    • Android Studio at /opt/android-studio
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b829.9-10027231)

[✓] Connected device (2 available)
    • Linux (desktop) • linux  • linux-x64      • Arch Linux 6.4.7-arch1-1
    • Chrome (web)    • chrome • web-javascript • Google Chrome 114.0.5735.198

[✓] Network resources
    • All expected network resources are available.

• No issues found!

Version of clang++
15.0.7
Version of ffigen
8.0.2
Additional context
No response

flutter_rust_bridge was updated to use exclusively Dart 3 in c5cf71c in v1.76.0, so you can either update your project to match or pin the version of flutter_rust_bridge to an older one.

oops! thank you for pointing it out, just back to upgrade the old project, maybe I missed that important information. Lemme try again.


" gmail.tracker**.xhr_open** " what does this line do?

I don't have any idea what does this line of code does.
Can somebody please help me with this?

Considering this a question and question answered. Closing :)


Refactors literal input.

closes #3011
Why was this change made?
More usable inputs.
How was this change tested?
Unit
Which documentation and/or configurations were updated?
README

Code coverage drop is because I made copies of some components/hooks. When the refactor completes, the originals will be gone and code coverage will be happy.


win: remove MAX_PATH limitations

Since Windows 10 1607 some WinApi functions no longer have a MAX_PATH limit on the filenames length. This PR removes this hard-coded path length limit from various places in libuv, switching to dynamically allocating string buffers.
Fixes: https://github.com/libuv/libuv/issues/2331

CI: https://ci.nodejs.org/view/libuv/job/libuv-test-commit/1848/

Old CI 404, new CI: https://ci.nodejs.org/view/libuv/job/libuv-test-commit/1881/

Green CI except for two unrelated failures in osx1015:
not ok 48 - fs_event_error_reporting
# timeout
# Output from process `fs_event_error_reporting`: (no output)

not ok 131 - hrtime
# timeout
# Output from process `hrtime`: (no output)

ping @libuv/collaborators

Landed in https://github.com/libuv/libuv/commit/da7e50bbd843a9278c57fc1f90cb4cbbd5fba169

I'm not sure how things work in this repo, but is there a reason this isn't on master and was merged in the v1.x branch?

Node.js uses v1.x so most of the development happens here.

Are PR's welcome that would push this onto master to sync up with v1.x?

@musm You mean a PR that merges v1.x into master? Yes, but it'll be a ton of hurt.


[PE-D] Find event command fail when input single character for info/

Command used:

find -e info/s

When find on a single character, the command will fail and the error message does not seem to match the error.
Expected output: Return the list of events with info containing 's'.

Labels: severity.Medium type.FunctionalityBug
original: hanhuiice/ped#1

fixed! issue was in the regex of information


FRAME log level does not work

Trying to hunt down why the latest doesn't work with the Allpixel anymore.... but found another bug.
At the very least for script run animations, the frame log level doesn't work:
import bibliopixel
# causes frame timing information to be output
bibliopixel.log.setLogLevel(bibliopixel.log.FRAME)

# Load driver for the AllPixel
from bibliopixel.drivers.serial import *
# set number of pixels & LED type here
driver = Serial(num = 3, ledtype = LEDTYPE.WS2811)

# load the LEDStrip class
from bibliopixel.layout import *
led = Strip(driver)

# load channel test animation
from bibliopixel.animation import StripChannelTest
anim = StripChannelTest(led)

try:
    # run the animation
    anim.run()
except KeyboardInterrupt:
    # Ctrl+C will exit the animation and turn the LEDs offs
    led.all_off()
    led.update()

I think it's something to do with the top_level property on Animation() but I have no clue what that does. There's no comments.

Closing. This is #1084


GH-3012: Non-blocking retries support @KafkaListener on class level

Fixes: #3012

Non-blocking retries support @KafkaListener on class level, include @RetryableTopic, @DltHandler, RetryTopicConfiguration, RetryTopicConfigurationSupport.
Doc and test Non-blocking retries support @KafkaListener on class level.

Part 1:  Support process annotation @RetryableTopic and @DltHandler when @RetryableTopic annotated on class, see #3107.
Part 2:  RetryTopicConfigurer#processAndRegisterEndpoint support MultiMethodKafkaListenerEndpoint, see #3112.
Part 3:  @EnableKafka integration feature that non-blocking retries support @KafkaListener on class level,  see #3105.

 Can you split the PR into multiple ones? That way, it is easy for us to review; maybe you can isolate the components needed as a prerequisite for the feature into a separate one. And when that is merged, perform the feature changes in another one. However, if it is difficult for you to split the PR, please let us know, and we will try our best to review this as is.

That way, it is easy for us to review; maybe you can isolate the components needed as a prerequisite for the feature into a separate one. And when that is merged, perform the feature changes in another one.

Good advice, let me try! And please allow me keep this PR, until complete prerequisite feature.

Thanks, @Wzy19930507, for this excellent new feature. Just a reminder: limit the commit message titles to only 50 or fewer characters. Commit message descriptions can be as long and descriptive as you want.


Refactor Flux types to use new semantic.MonoType

With the introduction of Algo W we have new semantic.MonoType that should be used in place of any semantic.Type instances.

I was going to keep this open until we have the feat/use-algo-w passing in CI in influxdb.
Currently there are two failures:

the integration tests for downstream projects
there's issues with the Flux queries issued by the notification package, which is this issue: https://github.com/influxdata/influxdb/issues/16809

@jsternberg got the pkg-config build working, so I'm closing this.
https://github.com/influxdata/influxdb/pull/17036


Update .eslintrc.js

Certainly! Here’s a polished explanation of the changes made to the ESLint configuration that you can use for approval:
Summary of ESLint Configuration Updates
We have made several improvements to the ESLint configuration to enhance clarity, maintainability, and overall effectiveness. Below are the key changes:
Organized Rules by Category:
Import Rules: Grouped all import-related rules together for better readability and management. React Rules: Consolidated React-specific rules, including those related to prop handling, component definitions, and performance optimizations. TypeScript Rules: Centralized TypeScript-related rules, covering type assertions, method signatures, and readability improvements. Simplified Configuration:
Removed Deprecated Rules: Eliminated obsolete rules related to Flow and other outdated configurations to streamline the setup. Consolidated Overrides: Used overrides to apply specific rules only to relevant files, such as documentation and Storybook setups, ensuring that general rules do not conflict with specialized requirements. Enhanced Readability:
Structured Rule Definitions: Organized rules in a logical order and grouped similar rules together to improve the ease of understanding and modification. Removed Redundant Rules: Cleaned up redundant and unnecessary rules to reduce complexity and potential conflicts. These updates aim to improve the maintainability of the configuration, ensure consistency across the codebase, and optimize performance and readability.
Please review the changes and let me know if there are any further adjustments needed.
Feel free to adjust any details or add more context based on your specific requirements!

🟡 Heimdall Review Status

Requirement
Status
More Info

Reviews

🟡
0/1

Denominator calculation

Show calculation

1 if user is bot
0

1 if user is external
0

From .codeflow.yml

1

Additional review requirements

Show calculation

Max
0

0

From CODEOWNERS

0

Global minimum
0

Max

1

1

1 if commit is unverified
0

Sum
1


cob_extern: 0.6.17-1 in 'melodic/distribution.yaml' [bloom]

Increasing version of package(s) in repository cob_extern to 0.6.17-1:

upstream repository: https://github.com/ipa320/cob_extern.git
release repository: https://github.com/ipa320/cob_extern-release.git
distro file: melodic/distribution.yaml
bloom version: 0.9.7
previous version for package: 0.6.15-1

cob_extern

No changes

libdlib

No changes

libntcan

No changes

libpcan

No changes

libphidgets

No changes

opengm

No changes

just to keep versions in sync with noetic - #25078


Files.isWritable returning false for network paths

Please provide a brief summary of the bug
We are using openjdk version "17.0.6" 2023-01-17.  OpenJDK 64-Bit Server VM Temurin-17.0.6+10 (build 17.0.6+10, mixed mode, sharing) in windows machine.
For a network samba shared path, Files.isWritable is returning false. Actually I have full access to the path. I am able to create subdirectory and files using command prompt / explorer.
I also observer path.toFile().canWrite() returns true for that shared path.
We have issues wherver we use Files.isWritable for network shared paths
Please provide steps to reproduce where possible
package Testing;
import java.nio.file.Files;
import java.nio.file.Path;
import java.nio.file.Paths;
public class FilesIsWritableTest {
public static void main(String[] args) {
	String f1 = "x:/yylmssysdm/zzzz";
	Path p = Paths.get(f1);

	System.out.println(Files.isWritable(p));
	System.out.println(p.toFile().canWrite());

}

}
Expected Results
true
true
Actual Results
false
true
What Java Version are you using?
OpenJDK Runtime Environment Temurin-11.0.17+8 (build 11.0.17+8)
What is your operating system and platform?
Microsoft Windows [Version 10.0.19045.2604]
How did you install Java?
MSI
Did it work before?
No

Did you test with the latest update version?
Yes

Did you test with other Java versions?
Tested with Java 17 also. Fails there

Relevant log output
No response

Looks like https://bugs.openjdk.org/browse/JDK-8154915?jql=text ~ "Files.isWritable"

https://bugs.openjdk.org/browse/JDK-8154915 has this ask:
It would be useful if the submitter could try Files.readAttributes(path, BasicFileAttributes.class) and report back so to whether it succeeds or throws exception. If there is an exception then it might reveal an issue in the environment.

@nsrg ^^

https://bugs.openjdk.org/browse/JDK-8154915 has this ask:
It would be useful if the submitter could try Files.readAttributes(path, BasicFileAttributes.class) and
report back so to whether it succeeds or throws exception. If there is an exception then it might
reveal an issue in the environment.

Tried  Files.readAttributes(path, BasicFileAttributes.class) ;
It does does not throw exception.

System Information:
Java version:    17.0.5+8-jvmci-22.3-b08
Virtual Machine: OpenJDK 64-Bit Server VM  17.0.5+8-jvmci-22.3-b08
[20]-> Path.of("\\server\tasks\test2.txt").getParent().toFile().canWrite();
$83 ==> true
[21]-> Files.isWritable(Path.of("\\server\tasks\test2.txt").getParent());
$85 ==> false
Looks like not Samba issue: \server is Windows Server 2019 Standard


[Automation] Generate Fluent Lite from Swagger billingbenefits#package-2022-11-01

[Automation] Generate Fluent Lite from Swagger billingbenefits#package-2022-11-01

/azp run java - billingbenefits - ci

API change check
APIView has identified API level changes in this PR and created following API reviews.
com.azure.resourcemanager:azure-resourcemanager-billingbenefits


Solid mutations are never isSuccess but directly isIdle

Describe the bug
We're trying to write some tests for a hook using a mutation with solid-query, it seems that when you use a mutation if the mutation succeeds, we always get an isIdle status
Your minimal, reproducible example
https://github.com/gabrieldemian/wagmi/blob/poc-tests/packages/solid/src/hooks/accounts/useConnect.test.ts
Steps to reproduce

use createMutation
Check the status if the mutation succeeds, you can even log it into a onSuccess callback, the status will be isIdle

Expected behavior
I would expect the status to be isSuccess if the mutation succeeds
How often does this bug happen?
Every time
Screenshots or Videos
No response
Platform

OS: Macos

TanStack Query version
"@tanstack/solid-query": "^4.22.0",
TypeScript version
"typescript": "^4.9.3",
Additional context
No response

@ardeora can you have a look?

I just found out the problem: on our code, we were destructuring reactive variables from the return object of the createMutation. And so breaking the reactivity... I was debbuging and used a single variable to get the data and it works normally! sorry about that, you can close this issue @TkDodo

Closing this issue! Issue was resolved


Name and socials rendering repetitively on navigation

Description
On switching between the tabs on a user's profile the name and socials get rendered again and again. It's just a little bit distracting.
Screenshots

https://www.awesomescreenshot.com/video/15158002?key=0dfc8065ede6a2e6c6928ea4396ac472
Additional information
No response

I would like to contribute to this issue. It would be my first open source contribution. But i have worked with React, Next for over 3 years now. So i am confident that it's a good first issue for me. It would be great if you can assign me this issue. Thanks. Soon  i will also share a video of the solution.

https://www.loom.com/share/c0b4d8c001734d86931668b43d53e51f
I have resolved this issue. It's a performance improvement too. As profile data doesn't get changed too frequently and it is a also a UI heavy component with images, QR Code and stuff. So using a memoized component instead.
Please provide suggestions if any.
And would be great if you can assign me this issue. If yes i'll create a PR for the same.

https://www.loom.com/share/c0b4d8c001734d86931668b43d53e51f
I have resolved this issue. It's a performance improvement too. As profile data doesn't get changed too frequently and it is a also a UI heavy component with images, QR Code and stuff. So using a memoized component instead.
Please provide suggestions if any.
And would be great if you can assign me this issue. If yes i'll create a PR for the same.
Sorry, i was an enthusiast here, and worked on the issue before resolution of triage, but excited for open-source contributions.

Looks awesome @psavan1655 I am not a maintainer so I can't assign this issue to you but soon someone will look into it and give you the required directions. 😊

https://www.loom.com/share/c0b4d8c001734d86931668b43d53e51f
I have resolved this issue. It's a performance improvement too. As profile data doesn't get changed too frequently and it is a also a UI heavy component with images, QR Code and stuff. So using a memoized component instead.
Please provide suggestions if any.
And would be great if you can assign me this issue. If yes i'll create a PR for the same.
Sorry, i was an enthusiast here, and worked on the issue before resolution of triage, but excited for open-source contributions.

Looks awesome @psavan1655 I am not a maintainer so I can't assign this issue to you but soon someone will look into it and give you the required directions. 😊

Thanks man. And thanks for this amazing feeling, i've been working in companies since a long now. but the excitement of working on open-source is something different, and would try doing more and more, Also develop my skills more in the process and help others do the same.

Happy to hear that 🌈

@Ananya2001-an Thank you for raising but this is actually a duplicate of an existing Issue: https://github.com/EddieHubCommunity/LinkFree/issues/5003 so I will be closing this.
@psavan1655 We are pleased that you have chosen EddieHub to make your first open source contribution. In future we would be grateful if you could please follow the procedures we have in place for working on Issues as these have been designed by the maintainers to make sure that the project runs effectively, avoids confusion and also prevents contributors from doing unnecessary or work which will not be used:

Wait until an Issue has the label 'status: ready for dev'
In the comments to be assigned this Issue.
If the Issue is assigned to you then please make a Pull Request.

As it appears you have fixed this problem raised in this Issue, please ask for it to be assigned to you here: https://github.com/EddieHubCommunity/LinkFree/issues/5003 and I will assign it to you.

Thanks man.

@psavan1655 I would suggest you join the EddieHub Discord so that you can learn how we use inclusive language in this community and refrain from calling anyone man guy girl etc. https://discord.com/invite/jZQs6Wu


Multiple core runnable

We can now sandbox the core in it's own context, the .mesg folder contains now a folder for each core based on the name. The core now has all the docker configurations (service, container, network) based on its name and we can now so run multiple core with some env variable.
./dev-core
MESG_CORE_NAME=core2 MESG_CORE_PATH=$(pwd)/xxx MESG_SERVER_ADDRESS=:50051 MESG_CLIENT_ADDRESS=localhost:50051 ./dev-core

and access with the cli
./dev-cli service list
MESG_CLIENT_ADDRESS=localhost:50051 ./dev-cli service list

Please merge https://github.com/mesg-foundation/core/pull/477 before

This close #457

To do:

[ ] the command stop is stopping mesg services from any core. We should use another system than listing all docker service that have a mesg.hash (add the name of the core as well?)
https://github.com/mesg-foundation/core/blob/bf15f80105e8c1b212098ba7de5175a8c584989f/service/status.go#L72-L87

@mesg-foundation/core ready to review again, I added the MESG_CORE_PATH to run the core with different config/db repository like that we can run multiple core with their own database or even choose to run multiple core with the same database


Added a more detailed description into the README

as discussed in https://github.com/phpstan/phpstan-webmozart-assert/issues/12
Closes #12

Thank you! Can you send a similar PR to beberlei-assert extension as well?

sure


Cannot delete files

Deleting files in the assets folder anywhere causes EIP-bot to throw with a "not found" message, resulting in the PR failing to merge.

This is now blocking two PRs: ethereum/EIPs#5054 and ethereum/EIPs#5017

Any particular reason why has been this "reopened" ?

The fix apparently didn't work.

That fix as you mentioned, was addressed in EIPW ? (Question)

No, it wasn't.

Deleting EIP asset file == modifying EIP asset files ? (Question), if this is the case then it would be covered with Issue
#95.

No, this is different. If any file is deleted, EIP-bot throws

is anyone working on it ?

No, there is not.

I will take a look on it.

I'm not looking at this one.

Hi.
I did several tests and PRs in the testing environment for this.

There is no bug with any of the bots: EIP-Bot and EIPW.
The test: https://github.com/JEAlfonsoP/EIPs/actions/runs/3093930099/jobs/5006818557
EIPW fails because the file pretending to be merged into the EIP repo does not follow EIP-1 rules. e.g:
Error: error: first line must be --- exactly
--> EIPS/eip-5255.md
|
1 | Delete EIP-5000
|

This is the first line for: https://github.com/ethereum/EIPs/pull/5255/commits/bdaf424ed9c178d462cc4c3ba71c3b6812bb2b89

Try to merge and delete an EIP: (Please I just try to delete last EIP-merged)
https://github.com/JEAlfonsoP/EIPs/pull/16
Actions: https://github.com/JEAlfonsoP/EIPs/actions/runs/3098269137 (No failure for any BOT).
There is a failure from CI: https://github.com/JEAlfonsoP/EIPs/actions/runs/3098269137/jobs/5015970668 (CodeSpell due to the fact that this empty case is not considered).

Sam, may be the messages from EIPW could be a more explicative (a suggestion).
PandaPip1,  I presume that this empty check might be included in CI.
General Observation: Any file to be merged must follows EIP1 rules.
Beside this I think that the case for deleting any file from the EIPs repo is solved.
Let me know if  you need something else or if it's Ok so this issue could be closed.

Your particular test didn't work because of an API rate limit: https://github.com/JEAlfonsoP/EIPs/actions/runs/3098245342/jobs/5015914203

Your particular test didn't work because of an API rate limit: https://github.com/JEAlfonsoP/EIPs/actions/runs/3098245342/jobs/5015914203

? What is this Panda ?
I do not understand it.

I am stating, as a fact, that deleting any file incorrectly throws a "not found" error due to the github API giving a 404 error. This is a bug, and is separate from any other CI elements correctly failing.

You need to consider the fact that deleting an EIP's repo file does not activate EIP-Bot.

Yes, it does.

You need to consider the fact that deleting an EIP's repo file does not activate EIP-Bot.

Yes, it does.

Interesting. If you have a test case with the actual ./workflow please share it if you like, so I can take a look on it.
Regardless, please, I ask you again that you advice if:
a.- You are working on this issue or any other one from this repo. So I avoid time consumption on it.
Thank you !


Feature request: explain how it works

I have not found in readme of on docs.rs main page an explanation how it works to quickly understand if it is suitable or not. In particular, it is unclear, how expensive it is. E. g. what would be overhead of picking among million items (on screen/total) for example?

There is a lengthy description of how the plugin works at the top level documentation: https://docs.rs/bevy_mod_picking/latest/bevy_mod_picking/index.html
The expensiveness of picking is completely dependent on the backend you decide to use. If the provided backends are not performant enough, it is incredibly simple to plug your own backend in - this is a core feature of the plugin.


[IOTDB-2205] Optimize unaligned int/long read/write functions in BytesUtils

注意到在现有的代码中，BytesUtils中非对齐的int/long的读写函数一次循环只能处理一个比特，效率较低，可以对其进行优化，使得一次循环可以处理一个字节。
该优化改动了下面的函数：

void longToBytes(long srcNum, byte[] result, int pos, int width)
void intToBytes(int srcNum, byte[] result, int pos, int width)
long bytesToLong(byte[] result, int pos, int width)
int bytesToInt(byte[] result, int pos, int width)

通过对用例的查找，发现该优化将会对使用TS_2DIFF编码的数据读写产生影响。
单元性能测试表明，该优化可以提高函数接近5倍的性能。总体性能测试表明，对于采用TS_2DIFF编码且不压缩的INT32/INT64数据，该优化可以减少约40%的读取时间开销。

In the current codes, the read and write functions of unaligned int/long in BytesUtils can only process one bit per loop, which is inefficient. It can be optimized to one byte per loop.
The following functions are optimized:

void longToBytes(long srcNum, byte[] result, int pos, int width)
void intToBytes(int srcNum, byte[] result, int pos, int width)
long bytesToLong(byte[] result, int pos, int width)
int bytesToInt(byte[] result, int pos, int width)

Through the use case search, the optimization influences the reading and writing of data encoded by TS_2DIFF.
In the unit performance test, the optimized functions are about 5x faster the current ones. In the overall test, about 40% time cost is reduced in the query of the uncompressed TS_2DIFF-coded INT32/INT64 data due to the optimization .

With the help of IoTDB-benchmark, I measured the performance improvement. Due to the difference in test data, the results are slightly different from those previously reported.

The write throughput before improvement is 232,268 pt/s and 237,046 pt/s after improvement. The gap is within the fluctuation range because the write bottleneck is not the optimized function.
The query throughput before improvement is 5,648,345 pt/s and 6,767,949 pt/s after improvement. Thers is a 20% improvement.

The modified config of IoTDB-benchmark is listed as follows while others are still the default.
LOOP=1000
SENSOR_NUMBER=4
CLIENT_NUMBER=1
POINT_STEP=1000
INSERT_DATATYPE_PROPORTION=0:1:1:1:1:0
COMPRESSOR=UNCOMPRESSED
ENCODING_INT32=TS_2DIFF
ENCODING_INT64=TS_2DIFF
ENCODING_FLOAT=TS_2DIFF
ENCODING_DOUBLE=TS_2DIFF
LINE_RATIO=0
SIN_RATIO=0
SQUARE_RATIO=0
RANDOM_RATIO=1
CONSTANT_RATIO=0
BATCH_SIZE_PER_WRITE=10000
QUERY_INTERVAL=100000000000

OPERATION_PROPORTION=1:0:0:0:0:0:0:0:0:0:0 (for write)
OPERATION_PROPORTION=0:0:1:0:0:0:0:0:0:0:0 (for query)

Hi, plz run mvn spotless:apply to fix the format issue.

Hi, plz run mvn spotless:apply to fix the format issue.

Thanks. The format issue is fixed in the new commit.

Note: There was a bug in the code changes submitted here, IOTDB-4633, which has been fixed in #7669.
byte y = (byte) (srcNum >> width);

has been corrected to
byte y = (byte) (srcNum >>> width);

Note: There was a bug in the code submitted here, IOTDB-4633, which has been fixed in #7669, where
byte y = (byte) (srcNum >> width);

has been corrected to
byte y = (byte) (srcNum >>> width);


[Feature Req] Flux Power Spectrum

Is your feature request related to a problem? Please describe.
No. It's a new physical feature that might be nice, suggested by @andreimesinger: getting the Flux PS (eg. https://academic.oup.com/mnras/article-pdf/400/3/1461/12387835/mnras0400-1461.pdf Sec 2.3).
Describe the solution you'd like
In principle, this is already achievable, by implementing a 1/r^2 profile, and setting the normalisation of the flux properly, similar to HI. It may be best implemented as an extension (eg. halomod-ionization or something), which just implements the necessary extension components, and sets reasonable defaults for this problem. Putting it into halomod itself is also possible.

So a quick comment on this. The 1/r^2 with an exponential cut is already included in the HI branch (which is a much more general one but a special case would be that flux density profile), and once I get that tested we can just use it. I think in terms of setting the normalisation, since it is just a constant I would say just add a unit conversion function in Constant hod class? And we set the default to the one in their paper.
If that makes sense, I'll also do that and maybe add a few lines in a tutorial somewhere mentioning we can use halomod to calculate flux power spectrum

Awesome! Yeah, I think a small separate tutorial, where we can point out the use-case would be useful.


Remove supports metaproperty (Unsupported in Chef 13)

https://docs.chef.io/deprecations_supports_property.html

Seeing this issue too. Would be good to get this merged...

+1

+1

Can this be merged? Seems it also needs a version bump in metadata unless CI handles that?


Allow symlink of packages

This is useful when developing packages. This is like yarn link.
CLI command zigmod link <name> <local dir> and unlink would be used to override package with local version. A config file at "~/.config/zigmod/links.yml" would store the config.

Since Zigmod is designed to be fairly decentralized, the concept of just names don't hold as much sense as they do in a npm/crates universe. The recommended workflow for the local development of packages is laid out here https://github.com/nektro/zigmod/blob/master/docs/tutorial.md#contributing-to-dependency-upstream. If the subpackage you're working on is your own, the flow is even more streamlined as you do not have to make a fork.

But I want to work locally. Now I have to manually symlink the local directory to the one zigmod uses.

the folder path i described above is local

By "local" I mean "on my machine".

yes

Oh, I understand: tell Zigmod that the package is a git hosted dependency, let it fetch it, and then work on it from the .zigmod/deps/git directory (after adding a remote)
That's similar to how I would work with PHP Composer packages.
Perhaps put this somewhere easily accessible in the docs? It's a common thing to want to do :)