Create a sample to show taking input attachments on a waterfall dialog

currently as i see , the handling attachment bot sample is more of a bot sample jumping different functions , it would be a better implementation if the same can be implemented in a waterfall-dialog setting .
attachments are a key part of the chat bot experience , the one i am developing for my company's project requires this .
currently i am trying to find a way to switch bots with a waterfall ,
any guidance will help . thank you .
EDIT : Multi attachment handling isnt very clear in the handling attachment sample .

any updates on how to do this ?

@v-kydela, can you please provide an example

i have described this issue here : https://stackoverflow.com/questions/58336861/how-to-access-iturn-context-functions-through-a-waterfall-dialog-in-bot-framewor
got no support there .
need to make a prototype to show my team before monday.
its meant to handle jpeg/png images . there is just a lack of samples on handling attachments . The attachmentprompt too has sparse documentation and literally no usage examples.

@vyomanaut - I have answered your question on Stack Overflow. Is it the same as your issue here or does this GitHub issue need to be solved separately?

Just saw your answer , what i am requesting is if a sample can be made for the next release of 4.6 .
Attachments is a very critical part of the chat-bot experience .
Also very sparse documentation on attachment prompt with no examples .

@vyomanaut, it's unlikely this meets the bar for 4.6 given the time frame of that release. @cleemullins to confirm.

can we add more examples/samples of handling attachements ?
the handling attachements sample is honestly not an ideal use case . Customers will likely want attachement funstionality in component dialogs .
also as i mentioned earlier , example demonstrating AttachmentPrompt is missing .

try this solution here :
https://github.com/microsoft/botframework-sdk/issues/5312

I'm open to improving the Attachment Sample. Certainly as part of the Teams work, that's on our radar. This would happen in our R7 or R8 time frame.
@vyomanaut, do you have a specific set of criteria you would like to see included?

@cleemullins - My recommendation would be to update the multi-turn prompt sample to include attachment prompts, since it has most other prompts already

@v-kydela I'm good with that. I've traiged to R7.

definitely what Kyle has said is recommended .
Apart from that nothing else so far .
btw , thats a cool profile picture though !, you seem to be one of ours ....

@v-kydela I've assigned you here. Can you take this with your suggested approach?

@cwhitten - Yes, you can expect a PR tomorrow. Thank you, Chris.

@cwhitten - I have submitted a PR for .NET and will look into doing Node as well

@cwhitten - I've made the changes to the Node sample now. I've requested your review.


fix: stableswap joinPoolSharesInternal

Closes: #XXX
What is the purpose of the change
Stable swap joinPoolSharesInternal doesn't updatePoolForJoin for the case of single asset join
Brief Changelog
I change joinPoolSharesInternal so that it updatePoolForJoin for the case of single asset join
Documentation and Release Note

Does this pull request introduce a new feature or user-facing behavior changes?  no
Is a relevant changelog entry added to the Unreleased section in CHANGELOG.md? no
How is the feature or change documented? not documented

Seems that @catShaark is working on unit tests in: https://github.com/osmosis-labs/osmosis/issues/1726

Seems that @catShaark is working on unit tests in: #1726

should I add the unit test in this PR or in another PR ?

Seems that @catShaark is working on unit tests in: #1726

should I add the unit test in this PR or in another PR ?

I think in this PR would be best if that's okay with you. It would allow us to compare the change to what unit tests try to achieve.

Seems that @catShaark is working on unit tests in: #1726

should I add the unit test in this PR or in another PR ?

I think in this PR would be best if that's okay with you. It would allow us to compare the change to what unit tests try to achieve.

I'll include the test in this PR


fix(Weaver): Don't delete the PDB after Weaving

This has caused us a lot of pain, mostly in the form of breakpoints and ASMDEF-related workarounds to get breakpoints working. Direct fix for #632.
Please test, I only tried the Mirror Examples, not a complete project.

My name is Chris Burns .... and I endorse this message (or PR or whatever)
The only thing pdb delete is solving is an issue where cecil will error to console when symbol files mismatch the assembly. Deleting pdb file is much worse, as all debug is pulled from it (eg. line numbers for back tracing and break points)
Existing projects on master will need to apply this fix, close out, nuke Library folder, then open up (force rebuild of all libs and pdbs)

we need to find the pr (and the reason) why we removed it in the first place to be sure we don't fix this here and open an older bug there.

Makes sense, but I already found that @vis2k. That particular bit of code was from HLAPI source Weaver. I don't think we had an equivalent before then.

I know I'm obviously one for wild ideas, but let's fix the issue of literally no one being able to debug or output line numbers in backtraces right now because we can. Weaver has been thoroughly tested without this code several times by several different people, among them myself. How it keeps managing to find its way back in I don't even want to know, it just needs to go.

Ok I'm retarded and my screen somehow hadn't updated and this had already merged. Everyone carry on here ... good work
Backs away slowly while pretending to be relevant

:tada: This PR is included in version 3.3.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Size of build output error, although nextjs build output is way smaller

Before opening, please confirm:

[X] I have checked to see if my question is addressed in the FAQ.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting questions.
[X] I have removed any sensitive information from my code snippets and submission.

App Id
dqdk4n5ewneid, d2vwdfz9609eox
AWS Region
us-west-1
Amplify Hosting feature
Frontend builds
Question
Hi all!
I've been having issues with a specific nextjs app in a monorepo with two other nextjs apps.  It's the smallest nextjs app of three with only one page and one API route. The build output is fairly large (172M total, 73M of .next/cache) but that's far less than the reported file size from the build process (238471689). I've got a hunch this error started being thrown since a different app (but from the same monorepo) actually had a too large build size (lots of static resources were accidentally pushed and deployed). The weird part is that the error is also being thrown when hosting the problematic app with a newly created amplify app.
appID dqdk4n5ewneid is an app created today, same problem.
appID d2vwdfz9609eox is the existing app which has been stuck now for days (maybe weeks).
So the other two monorepo apps are way, way bigger (e.g. ~30 pages with 4 locales, lots of static resources) and have no issues with being build. One of those has a build output of 458M, but 316M of it is .next/cache.
I'm pretty much stuck at the moment, and been trying to fix this issue for the last couple of days. I understand monorepo's, nextjs build output is not the Amplify's team responsibility, but 🤷♂  no idea what to do next.
Error being thrown is:
!!! CustomerError: The size of the build output (238471689) exceeds the max allowed size of230686720 bytes. Please reduce the size ofyour build output and try again.
What I've tried

Creating a new app.
Adding - envCache --set stackInfo "" to amplify build phase.
Adding rm -rf .next/cache/ as a build and postbuild command.
Different frontend.artifacts.build locations in amplify.yml.
Tons of tinkering with nextjs versions, amplify cli versions, nextjs configurations, etc.

What I haven't tried yet

amplify CLI  command invalidate-project-cache command  https://awscli.amazonaws.com/v2/documentation/api/latest/reference/codebuild/invalidate-project-cache.html. I'm not sure which codebuild project I should use as it managed by amplify. Also a newly setup app shouldn't have build cache already available right?

This issue might be related https://github.com/aws-amplify/amplify-hosting/issues/3601 because of the build size exceeding  error since a deployment with a build actually being too large. Difference is a different monorepo-amplify-app had the actual exceeding build output, and the app mentioned in this issue is still throwing the error.

@jbreemhaar could you check what is taking most of the size on your build output? You can do this by running the following commands locally:

Run a build first using standalone mode (this will produce node_module folders inside .next/standalone).

export NEXT_PRIVATE_STANDALONE=true
npm run build # or yarn build

Now check the size of these folders

Static files
du -h .next/static

Public folder
du -h public

and the standalone folder
du -h .next/standalone

@ferdingler sure!
.next/static
2.5M    .next/static
public
4.2M    public
96M    .next/standalone

Hi, we have the same problem with a full NextJS 13.4.10
46M .next/static
12M public
126M .next/standalone

Same issue here, even when simply triggering a build that passed before: worked fine in July 21st and fails today when retriggering the same build with same commit, same dependencies, same build pipeline etc.
When checking on size, both locally and on the node with du, the size is about 159M:
2023-07-26T11:56:37.462Z [INFO]: ℇ  (Streaming)  server-side renders with streaming (uses React 18 SSR streaming or Server Components)
                                 λ  (Server)     server-side renders at runtime (uses getInitialProps or getServerSideProps)
                                 ○  (Static)     automatically rendered as static HTML (uses no initial props)
2023-07-26T11:56:37.684Z [INFO]: # Executing command: du -h -d1 .next
2023-07-26T11:56:37.687Z [INFO]: 133M	.next/cache
2023-07-26T11:56:37.688Z [INFO]: 30M	.next/server
2023-07-26T11:56:37.688Z [INFO]: 28M	.next/static
2023-07-26T11:56:37.688Z [INFO]: 44K	.next/types
2023-07-26T11:56:37.702Z [INFO]: 99M	.next/standalone
                                 291M	.next
2023-07-26T11:56:37.702Z [INFO]: # Executing command: rm -fr .next/cache
2023-07-26T11:56:37.719Z [INFO]: # Executing command: du -h -d1 .next
2023-07-26T11:56:37.720Z [INFO]: 30M	.next/server
2023-07-26T11:56:37.720Z [INFO]: 28M	.next/static
2023-07-26T11:56:37.721Z [INFO]: 44K	.next/types
2023-07-26T11:56:37.734Z [INFO]: 99M	.next/standalone
                                 159M	.next
2023-07-26T11:56:37.734Z [INFO]: # Completed phase: build
2023-07-26T11:56:37.735Z [INFO]: ## Completed Frontend Build
2023-07-26T11:56:37.736Z [INFO]: ## Build completed successfully
2023-07-26T11:56:37.738Z [INFO]: Setting NEXT_PRIVATE_STANDALONE=true to produce .next/standalone directory
2023-07-26T11:56:37.740Z [INFO]: Found customHttp.yml, applying custom headers...
2023-07-26T11:57:18.260Z [ERROR]: !!! CustomerError: The size of the build output (323613118) exceeds the max allowed size of230686720 bytes. Please reduce the size ofyour build output and try again.

Hey everyone 👋
We just released some changes to improve how we bundle Next.js apps. These changes don't need the standalone directory anymore. That should help eliminate the problem of reaching the 200MB bundle limit. These changes require a new deploy in your application to take effect. The expected outcome is that your app bundles should be smaller, and not bump into the size limit.
If you have comments or questions don't hesitate to reach us via AWS support.


[Bug] Headers files for macos/ios are not generated as expected when following the multiple files article

Describe the bug
http://cjycode.com/flutter_rust_bridge/article/generate_multiple_files.html
Following that article i created two rust files and tried to do code gen. but the generated headers file only have functions that was present in the last file to be provided to the argument
Codegen logs with RUST_LOG=debug environment variable
Tools • Dart 2.18.2 • DevTools 2.15.0
[2022-10-14T03:00:54Z INFO  lib_flutter_rust_bridge_codegen] Picked config: Opts { rust_input_path: "/Users/jawadkalia/Downloads/flutter_rust_bridge_template-main/native/src/release_mode.rs", dart_output_path: "/Users/jawadkalia/Downloads/flutter_rust_bridge_template-main/lib/bridge_generated_releasemode.dart", dart_decl_output_path: None, c_output_path: ["/Users/jawadkalia/Downloads/flutter_rust_bridge_template-main/ios/Runner/bridge_generated.h", "/Users/jawadkalia/Downloads/flutter_rust_bridge_template-main/macos/Runner/bridge_generated.h"], rust_crate_dir: "/Users/jawadkalia/Downloads/flutter_rust_bridge_template-main/native", rust_output_path: "/Users/jawadkalia/Downloads/flutter_rust_bridge_template-main/native/src/generated_releasemode.rs", class_name: "ReleaseModeClass", dart_format_line_length: 80, skip_add_mod_to_lib: false, llvm_path: ["/opt/homebrew/opt/llvm", "/usr/local/opt/llvm", "/usr/lib/llvm-9", "/usr/lib/llvm-10", "/usr/lib/llvm-11", "/usr/lib/llvm-12", "/usr/lib/llvm-13", "/usr/lib/llvm-14", "/usr/lib/", "/usr/lib64/", "C:/Program Files/llvm", "C:/msys64/mingw64"], llvm_compiler_opts: "", manifest_path: "/Users/jawadkalia/Downloads/flutter_rust_bridge_template-main/native/Cargo.toml", dart_root: Some("/Users/jawadkalia/Downloads/flutter_rust_bridge_template-main"), build_runner: true, block_index: BlockIndex(1), skip_deps_check: false, wasm_enabled: false, inline_rust: false }
[2022-10-14T03:00:54Z INFO  lib_flutter_rust_bridge_codegen] Phase: Parse source code to AST, then to IR
[2022-10-14T03:00:54Z WARN  lib_flutter_rust_bridge_codegen::source_graph] Skipping unresolvable module web (tried /Users/jawadkalia/Downloads/flutter_rust_bridge_template-main/native/src/web.rs, /Users/jawadkalia/Downloads/flutter_rust_bridge_template-main/native/src/web/mod.rs, /Users/jawadkalia/Downloads/flutter_rust_bridge_template-main/native/src/bridge_generated/web.rs, /Users/jawadkalia/Downloads/flutter_rust_bridge_template-main/native/src/bridge_generated/web/mod.rs, )
[2022-10-14T03:00:54Z WARN  lib_flutter_rust_bridge_codegen::source_graph] Skipping unresolvable module io (tried /Users/jawadkalia/Downloads/flutter_rust_bridge_template-main/native/src/io.rs, /Users/jawadkalia/Downloads/flutter_rust_bridge_template-main/native/src/io/mod.rs, /Users/jawadkalia/Downloads/flutter_rust_bridge_template-main/native/src/bridge_generated/io.rs, /Users/jawadkalia/Downloads/flutter_rust_bridge_template-main/native/src/bridge_generated/io/mod.rs, )
[2022-10-14T03:00:54Z WARN  lib_flutter_rust_bridge_codegen::source_graph] Skipping unresolvable module io (tried /Users/jawadkalia/Downloads/flutter_rust_bridge_template-main/native/src/io.rs, /Users/jawadkalia/Downloads/flutter_rust_bridge_template-main/native/src/io/mod.rs, /Users/jawadkalia/Downloads/flutter_rust_bridge_template-main/native/src/generated_api/io.rs, /Users/jawadkalia/Downloads/flutter_rust_bridge_template-main/native/src/generated_api/io/mod.rs, )
[2022-10-14T03:00:54Z WARN  lib_flutter_rust_bridge_codegen::source_graph] Skipping unresolvable module io (tried /Users/jawadkalia/Downloads/flutter_rust_bridge_template-main/native/src/io.rs, /Users/jawadkalia/Downloads/flutter_rust_bridge_template-main/native/src/io/mod.rs, /Users/jawadkalia/Downloads/flutter_rust_bridge_template-main/native/src/generated_releasemode/io.rs, /Users/jawadkalia/Downloads/flutter_rust_bridge_template-main/native/src/generated_releasemode/io/mod.rs, )
[2022-10-14T03:00:54Z INFO  lib_flutter_rust_bridge_codegen] Phase: Transform IR
[2022-10-14T03:00:54Z INFO  lib_flutter_rust_bridge_codegen] Phase: Generate Rust code
[2022-10-14T03:00:54Z INFO  lib_flutter_rust_bridge_codegen] Phase: Generate Dart code
[2022-10-14T03:00:54Z INFO  lib_flutter_rust_bridge_codegen] Phase: Generating Dart bindings for Rust
Running in Directory: '/Users/jawadkalia/Downloads/flutter_rust_bridge_template-main'
Input Headers: [/var/folders/s7/1kvs565j4r566jwg00xl3d4w0000gp/T/.tmpSuivVd.h]
[SEVERE] : Header /var/folders/s7/1kvs565j4r566jwg00xl3d4w0000gp/T/.tmpSuivVd.h: Total errors/warnings: 180.
 Just run just
Expected behavior
bridge_generated.h file should have function signature from all provided rs files
Generated binding code
flutter_rust_bridge_codegen \
        --rust-input native/src/api.rs native/src/release_mode.rs \
        --dart-output lib/bridge_generated.dart lib/bridge_generated_releasemode.dart \
        --rust-output native/src/generated_api.rs native/src/generated_releasemode.rs \
        --class-name ApiClass ReleaseModeClass \
        --c-output ios/Runner/bridge_generated.h \
        --c-output macos/Runner/bridge_generated.h

OS
MAcOS
Version of flutter_rust_bridge_codegen
1.49
Flutter info
lutter doctor -v
[✓] Flutter (Channel stable, 3.3.4, on macOS 12.3 21E230 darwin-arm, locale en-PK)
    • Flutter version 3.3.4 on channel stable at /opt/homebrew/Caskroom/flutter/3.3.4/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision eb6d86ee27 (9 days ago), 2022-10-04 22:31:45 -0700
    • Engine revision c08d7d5efc
    • Dart version 2.18.2
    • DevTools version 2.15.0

[!] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
    • Android SDK at /Users/jawadkalia/Library/Android/sdk
    • Platform android-33, build-tools 33.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)
    ! Some Android licenses not accepted.  To resolve this, run: flutter doctor --android-licenses

[✓] Xcode - develop for iOS and macOS (Xcode 13.4.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 13F100
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2021.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • 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 11.0.13+0-b1751.21-8125866)

[✓] VS Code (version 1.72.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.50.0

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-arm64   • macOS 12.3 21E230 darwin-arm
    • Chrome (web)    • chrome • web-javascript • Google Chrome 106.0.5249.119

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

Version of clang++
Apple clang version 13.1.6 (clang-13<IP_ADDRESS>.5) Target: arm64-apple-darwin21.4.0 Thread model: posix
Version of ffigen
ffigen: ^6.0.1 in project pubspec.yaml
Additional context
No response

/cc @dbsxdbsx

i just wanted to make sure if im doing it properly as well. i would appreciate it very much if you could download the zip file attached and take a look at it?
Is there a workaround in the meantime? do i seperate the command into two parts and then wire it up in ffi.dart ?
do i creat two seperate headers files?

Understood! that looks like an interesting project! i wish you good fortune!
My question is around what would be the way to do run codegen commands when im not using the multiple files technique. is single file api.rs the way to go?

Thanks :)

when im not using the multiple files technique...is single file api.rs the way to go?

Yes, just one api.rs.
Maybe refer to .github/workflows/ci.yaml
And ./justfile

Thank you! ill be honest i didnt see the pure dart multi stuff. ill look into that and see if it resolves my issue and if it does ill close the ticket

pure dart is about generating .so files. so i dont think that would work in a flutter context. still looking for advice. for now the single api.rs file is good for my needs but i may expect the project getting larger

Maybe mimic the pure_dart_multi to configure the flags?
Anyway, I suspect this may be a bug. Feel free to PR!

https://github.com/fzyzcjy/flutter_rust_bridge/blob/a62dcf0c84/frb_example/pure_dart_multi/rust/build.rs this might be something.

So i tried a build.rs file like so
use lib_flutter_rust_bridge_codegen::{
    config_parse, frb_codegen, get_symbols_if_no_duplicates, RawOpts,
};

/// Path of input Rust code
const RUST_INPUT_1: &str = "src/api_1.rs";
const RUST_INPUT_2: &str = "src/api_2.rs";
/// Path of output generated Dart code
const DART_OUTPUT_1: &str = "../dart/lib/bridge_generated_api_1.dart";
const DART_OUTPUT_2: &str = "../dart/lib/bridge_generated_api_2.dart";
/// Path of output Rust code
const RUST_OUTPUT_1: &str = "src/generated_api_1.rs";
const RUST_OUTPUT_2: &str = "src/generated_api_2.rs";
/// Class name to use in dart, corresponding to each Rust block
const CLASS_NAME_1: &str = "ApiClass1";
const CLASS_NAME_2: &str = "ApiClass2";

fn main() {
    // Tell Cargo that if the input Rust code changes, to rerun this build script.
    println!("cargo:rerun-if-changed={}", RUST_INPUT_1);
    println!("cargo:rerun-if-changed={}", RUST_INPUT_2);
    // Options for frb_codegen
    let raw_opts = RawOpts {
        // Path of input Rust code
        rust_input: vec![RUST_INPUT_1.to_string(), RUST_INPUT_2.to_string()],
        // Path of output generated Dart code
        dart_output: vec![DART_OUTPUT_1.to_string(), DART_OUTPUT_2.to_string()],
        // Path of output Rust code
        rust_output: Some(vec![RUST_OUTPUT_1.to_string(), RUST_OUTPUT_2.to_string()]),
        // Class name of each Rust block of api
        class_name: Some(vec![CLASS_NAME_1.to_string(), CLASS_NAME_2.to_string()]),
        // c outputs for macos headers
        c_output: Some(vec![
            "macos/Runner/bridge_generated_1.h".to_string(),
            "macos/Runner/bridge_generated_2.h".to_string(),
        ]),
        // for other options use defaults
        ..Default::default()
    };
    // get opts from raw opts
    let configs = config_parse(raw_opts);

    // generation of rust api for ffi
    let all_symbols = get_symbols_if_no_duplicates(&configs).unwrap();
    for config in configs.iter() {
        frb_codegen(config, &all_symbols).unwrap();
    }
}

(I also tried with a single element for the c output vec!)
contents of api_1.rs
pub fn sum_1(a: u32, b: u32) -> u32 {
    a + b
}

contents of api_2.rs
pub fn subract_1(a: u32, b: u32) -> u32 {
    a - b
}```
Generated file 1
```rust

#include <stdbool.h>
#include <stdint.h>
#include <stdlib.h>
typedef struct WireSyncReturnStruct {
  uint8_t *ptr;
  int32_t len;
  bool success;
} WireSyncReturnStruct;
typedef int64_t DartPort;
typedef bool (*DartPostCObjectFnType)(DartPort port_id, void *message);
void free_WireSyncReturnStruct(struct WireSyncReturnStruct val);
void store_dart_post_cobject(DartPostCObjectFnType ptr);
void wire_subract_1(int64_t port_, uint32_t a, uint32_t b);
static int64_t dummy_method_to_enforce_bundling(void) {
    int64_t dummy_var = 0;
    dummy_var ^= ((int64_t) (void*) wire_subract_1);
    dummy_var ^= ((int64_t) (void*) store_dart_post_cobject);
    return dummy_var;
}```
Generated file 2 is an exact copy of the above.

@AhyaanTech ,  actually, after I finished the multi file feature since version 1.37.1, I have had little time on frb. And I did notice that I can't compile my project with multiple files with frb with recent version ^1.48.
It needs some time to figure the issue out.

@dbsxdbsx Well, maybe next time we should ensure to add enough tests. If there is a test, CI will ensure we never have such regression today :/

one

@dbsxdbsx Well, maybe next time we should ensure to add enough tests. If there is a test, CI will ensure we never have such regression today :/

@fzyzcjy Maybe just add an argument to c_output, which like https://github.com/fzyzcjy/flutter_rust_bridge/issues/781#issuecomment-1279861204

two
From this sentence, I understand that c file does not need to be decomposed, it should be all mixed together.
https://github.com/fzyzcjy/flutter_rust_bridge/blob/master/frb_codegen/src/config.rs#L180-L181

// c output path(s) (only 1 list is needed, nothing to do with number of rust_input_paths)

Is my understanding correct?

two section two
https://github.com/fzyzcjy/flutter_rust_bridge/blob/master/frb_codegen/src/lib.rs#L115-L121
do what I say above

for output in &config.c_output_path {
        fs::create_dir_all(Path::new(output).parent().unwrap())?;
        fs::write(
            output,
            fs::read_to_string(&temp_bindgen_c_output_file)? + "\n" + &c_dummy_code,
        )?;
    }

three
So it doesn't have to be executed

let exclude_symbols = generated_rust.get_exclude_symbols(all_symbols);

just use all_symbols?

three section two
If we want exclude_symbols. We need to split the header. It could be as follows

for output in &config.c_output_path {
        fs::create_dir_all(Path::new(output).parent().unwrap())?;
        fs::write(
            format!("{}_{}",output,id), // id is from rust_input_index
            fs::read_to_string(&temp_bindgen_c_output_file)? + "\n" + &c_dummy_code,
        )?;
    }

@huang12zheng Not exactly sure about what you want to ask, but feel free to try it by making a PR. If CI passes and also no bug is introduced, we can surely utilize the approach :)

@fzyzcjy
What I want to say is that I didn't find enough information about the C header file. Such as doc, example dir, and issues.
So I'm a little confused about what to expect from the C header file.
I would try a pr with The way I mentioned in three section two.
It is expected to generate multiple.h files in each Runner, such as xx_1.h, xx_2.h

@AhyaanTech，

headers file only have functions that was present in the last file to be provided to the argument

for this, you are right. It is a bug, but it doesn't effect compiling app. Once it is fixed, you would be noticed here.
And For the "[SEVERE] :" log issue,  I've test it your zip project on win10 with latest frb(version 1.56.0), it is workable for win10 and Android APP, with some modification on your project, then there is no "[SEVERE] :" logging any longer:
Specifically:

I update frb for dart to  flutter_rust_bridge: ^1.56.0,
I add "--wasm" for the justfile, since you used templates where there is a "ffi_web.dart"
I modified some code in ffi.dart and ffi_web.dart, it seems that you think there only need one api to handle all rust blocks, which is not true, and you don't change name correctly according to flag class-name from justfile.
I deleted all generated files for both dart and rust, which you could generate them by run ning just.

new.zip
Could you please test on your APPLE machine, to see if the annoying "[SEVERE] :" log still appears, and more importantly, if it succeeded in producing APP for your iOS or MacOS?
@fzyzcjy, at present, just like what in template, ffi_web.dart and ffi.dart are files that need users to write manually, would it be a good idea to let frb to generate them automatically?

is this directed towards me? do i need to download the zip file and run it my local

is this directed towards me? do i need to download the zip file and run it my local

Yes, just download it and ,run just, and flutter run to check.  But I am not sure if there is some issue with MacOS or IOS, seems you are coding on the platform.

@fzyzcjy, at present, just like what in template, ffi_web.dart and ffi.dart are files that need users to write manually, and somehow it is a little complicated to write it correct, just like what I said in point 3. Would it be a good idea to let frb to generate them automatically?

@dbsxdbsx Can we extract some common function to frb_dart to simplify it?

So i ran the just file. and it gave me an error about having that dart declarations output.

ill add that command to just and continue. but wanted to see if you have any recommendations @dbsxdbsx

@dbsxdbsx
Hi! multiple files are being generated
but the SEVERE logs are still there

when im running the app i get this

im guessing i have an issue on my setup but im coming up blank. please help

Also this

@AhyaanTech, thanks for your feedback. But frankly, I have no idea why these logs occur. Hopefully, this issue could be eased or totally solved once https://github.com/fzyzcjy/flutter_rust_bridge/pull/862 is solved.

https://github.com/fzyzcjy/flutter_rust_bridge/pull/937 is merged

@AhyaanTech, now since frb 1.59, whenever you want to generate the same c-output file in more than two places, please use flag extra-c-output-path (or e for short)  to define extra path without the c header file name. The example is refined here.
And the number of fields used in flag c-output(or c for short) should be consistent with that of rust-input if the flag is not omitted.
Hopefully, this release could alleviate or solve your issue.


Time for a new release?

The latest release, 1.0.0 doesn't handle image rotation correctly. This has been fixed in master, but pulling the latest tag will give you broken or old behaviour. I just spent quite a long time debugging inconsistent behaviour between my project and running the examples, only to find that dep will default to the latest tag, so I had to switch to the master branch manually.

Good idea. Just added v1.1.0 release. Thanks.


Command History Not Working on Windows

I'm not sure if this is because bash is not fully implemented on Windows, but using the Up Arrow to get my command history does not work when using bash through PlatformIO.

Same here. Tab key is not working too.

Same issue here.  Looking into it to see if I can fix, but this is probably beyond what I know how to work on. :(

any updates on this, I have the same issue.

same.

can try ctrl+p or other  key combination

It's not a problem with bash on windows, since the up and down commands work fine when bash is executed from powershell. Identical to issues #231 and #221

Please retest again with platformio-atom-ide-terminal 2.9.0 is using node-pty-prebuilt instead unmaintained and outdated pty.js


Version 6.8.0 release

Changed

Added support for version v2.1-<IP_ADDRESS> of the DocuSign ESignature API.
Updated the SDK release version.

closing to regenerate w/ fixed swagger


Pool getOutputAmount calculation incorrect for some cases

I've noticed for some pools, the getOutputAmount produces different results than quoter v2
https://github.com/z-hao-wang/v3-core-debug/pull/1
this branch can reproduce the error I found
The quote results are same for config like this
{
    description: 'high fee, 1:1 price, default config',
    feeAmount: FeeAmount.HIGH,
    tickSpacing: TICK_SPACINGS[FeeAmount.HIGH],
    startingPrice: encodePriceSqrt(1, 1),
    positions: [
      {
        tickLower: getMinTick(TICK_SPACINGS[FeeAmount.HIGH]),
        tickUpper: getMaxTick(TICK_SPACINGS[FeeAmount.HIGH]),
        liquidity: expandTo18Decimals(2),
      },
    ],
    swapTests: [{
      zeroForOne: true,
      exactOut: false,
      amount0: expandTo18Decimals(1),
    }],
  }

but it doesn't work for config like this
{
    description: 'high fee, 1:1 price, buggy swap calculation',
    feeAmount: FeeAmount.HIGH,
    tickSpacing: TICK_SPACINGS[FeeAmount.HIGH],
    startingPrice: BigNumber.from('3905891053926514387903925684'), // tick at -60200
    positions: [
      {
        tickLower: -60200,
        tickUpper: 138200,
        liquidity: '1505426792435356595487',
      },
    ],
    swapTests: [{
      zeroForOne: true,
      exactOut: false,
      amount0: expandTo18Decimals(1),
    }],
  }

I found this is due to amount remaining isn't returned.
The pool doesn't get enough liquidity.
see below pr that fixes it.
https://github.com/QuantSatoshi/v3-sdk/pull/1/files


Speed up execution of hooks

Don't know if this will be accepted but based on the results of hyperfine this reduces overall time by ~ 10 ms

Not that much of an improvement to justify potential unexpected behavior.


Improve adding custom action to buttons.

Improve adding custom action to buttons.
For example I want add a button with custom action but this package not supported that. After this pull request, everybody can button with custom action using via like following code:
File: app/Datatables/UsersDataTable.php
...
public function html()
    {
        return $this->builder()
            ->setTableId('usersdatatable-table')
            ->columns($this->getColumns())
            ->minifiedAjax()
            ->dom('Bfrtip')
            ->orderBy(0)
            ->buttons(
                (new Button)->text('Button Test')->action('alert("clicked!")')
            );
    }
...

Btw, I supported before this PR codes. So this PR not broke older codes.

Released on v4.20.0 🚀

Hi, i have install 4.20 in 7.0.33 php environment (it's need...) but
(1/1) FatalThrowableErrorCall to undefined function Yajra\DataTables\Html\substring()
Can you help me?

@matteocostantini I send PR for your problem. Details: https://github.com/yajra/laravel-datatables-html/pull/116


lookup and lookupAsync should not rely on incoming 'contract' param

https://github.com/EOSIO/eosjs/blob/affaa4b478d06102f73ff36b4a3b04fb772899f3/src/asset-cache.js#L16
s.a. https://eosio.stackexchange.com/questions/544/does-eosjs-only-support-eosio-and-eosio-token-contract
I create a transaction like this:
var eos = Eos.Localnet(options);
eos.transaction({
actions: [
{
account: "adv",
name: "publish",
authorization: [{
actor: issuer,
permission: "active"
}],
data: {
"issuer": issuer,
"commission": commission,
"schema1": 0,
"schema2": 0,
"link": link
}
}
]
})
but get_currency_stats will query currency stats from 'adv' contract like this
curl http://<IP_ADDRESS>:8888/v1/chain/get_currency_stats -X POST -d '{"code":"adv","symbol":"ADV"}'
It is  obviously wrong. I suppose the 'lookup' and 'lookupAsync' s 'contract' param is redundant and harmful. once I removed this param and set  'contract' to 'eosio.token', the push_transaction operation will  work fine.
Wish eosjs team could help to confirm, thanks!

The asset and symbol types were pointing to the actions current account.  Agreed,  more common to default them to eosio.token, safer too.

published<EMAIL_ADDRESS>

I will ask about that..

Haha，I'm joking!
But it's so exciting if I can got even 1$
(^_^)


Fixed indent in Python code

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


Workbench fails to connect to extension host

Issue Type: Bug
I'm having all sorts of issues with Visual Studio Code and I'm unable to find the issue.

Git doesn't detect the repository even though a .git folder does exist.
If I run a search it never completes. It looks like it's trying, but never returns anything.
If I open the Terminal, it closes immediately.
If I have an extension that adds something to the status bar, those don't show.

I've tried:

Re-installing Visual Studio Code.
Using AppCleaner to remove all the Visual Studio Code files it could so I start fresh.
Removing the .vscode folder from my project.
Tried opening different repos.
Tried restarting my Mac.
Tried running Visual Studio Code  with extensions diabled.
Tried putting any settings back to their default values.

Typically the only thing that works is a Mac restart. Then on initial launch it typically works fine, but any launch after that it's back to broken.
In the Developer Tools it says
ERR Error received from starting extension host (kind: LocalProcess)
ERR Error: listen EADDRINUSE: address already in use /var/folders/2g/wjwbbs397xv7grs6070l2l540000gp/T/com.apple.WorkflowKit.BackgroundShortcutRunner/vscode-ipc-dad0e606-eb7c-41a7-9adb-41aceb20ada6.sock
    at Server.setupListenHandle [as _listen2] (net.js:1301)
    at listenInCluster (net.js:1366)
    at Server.listen (net.js:1463)
    at workbench.desktop.main.js:2984
    at new Promise (<anonymous>)
    at q._tryListenOnPipe (workbench.desktop.main.js:2984)
    at q.start (workbench.desktop.main.js:2984)
    at new es (workbench.desktop.main.js:1557)
    at y._createInstance (workbench.desktop.main.js:574)
    at y.createInstance (workbench.desktop.main.js:574)
    at d (workbench.desktop.main.js:1557)
    at workbench.desktop.main.js:2982
    at Array.forEach (<anonymous>)
    at jl._startExtensionHosts (workbench.desktop.main.js:2982)
    at jl._initialize (workbench.desktop.main.js:2982)
    at workbench.desktop.main.js:2985

ERR undefined
All errors map to log.ts:313
I also see warnings.
WARNING: IPC handle "/var/folders/2g/wjwbbs397xv7grs6070l2l540000gp/T/com.apple.WorkflowKit.BackgroundShortcutRunner/vscode-ipc-dad0e606-eb7c-41a7-9adb-41aceb20ada6.sock" is longer than 103 chars, try a shorter --user-data-dir ipc.net.ts:753
WARN Settings Sync: Accounts are not initialized
log.ts:313
VS Code version: Code 1.63.2 (899d46d82c4c95423fb7e10e68eba52050e30ba3, 2021-12-15T09:37:28.172Z)
OS version: Darwin x64 21.2.0
Restricted Mode: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8850H CPU @ 2.60GHz (12 x 2600)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
2, 2, 2

Memory (System)
16.00GB (0.31GB free)

Process Argv
--crash-reporter-id e3482bcd-f302-46d4-afba-cecee3442a64

Screen Reader
no

VM
0%

Extensions: none
A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
pythonvspyl392cf:30425750
pythontb:30283811
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
pythondataviewer:30285071
vscod805:30301674
pythonvspyt200:30340761
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593cf:30376535
vsc1dst:30428979
pythonvs932:30410667
wslgetstarted:30426333
vscop804:30404766
vscop453:30404998
vsrem710:30416614
vscop841:30429141
vsbas813:30426126

The root cause is this being your extension host IPC handle: /var/folders/2g/wjwbbs397xv7grs6070l2l540000gp/T/com.apple.WorkflowKit.BackgroundShortcutRunner/vscode-ipc-dad0e606-eb7c-41a7-9adb-41aceb20ada6.sock.
Two things here:

We should not be using UUIDs here, especially on *nix where there's a hardcoded limit on the size of these files. I suggest to come up with a shorter random string generator.
The user's TEMP dir appears to be /var/folders/2g/wjwbbs397xv7grs6070l2l540000gp/T/com.apple.WorkflowKit.BackgroundShortcutRunner. I wonder why.

We should not be using UUIDs here, especially on *nix where there's a hardcoded limit on the size of these files. I suggest to come up with a shorter random string generator, as it used to be. cc @jrieken

Well, before it was 21 bytes as hex and a uuid is 16 bytes as hex (plus a few formatting dashes). So, I would say it used to be worst. Also, we made this change because of really bad observed performance of the old solution.
Overall, the approach of tweaking the length of the random portion seems pretty arbitrary. Not saying it shouldn't be shorter but the overall length needs to be taken into account and we should stop when it's too long.

/duplicate https://github.com/microsoft/vscode/issues/139570

UUID is 36 characters long. 😕

Overall, the approach of tweaking the length of the random portion seems pretty arbitrary. Not saying it shouldn't be shorter but the overall length needs to be taken into account and we should stop when it's too long.

Correct. But given that we don't know how long the TMP folder is going to be (on some systems it depends on the username... and some people have pretty long usernames), we should do anything we can to minimize the file name. And I don't see us changing the default location from TMP, since that will open up a brand new door of issues. I'm certainly not brave enough to do that. I've been in this fight for years, ever since we had our first IPC implementation. There's only one way to win: minimize the socket file name.

UUID is 36 characters long. 😕

Yes, and require('crypto').randomBytes(21).toString('hex').length is 42 characters long

Ah so! Yeah that makes sense. I'm fine with dropping crypto, we know it's slow. But instead of taking the entire UUID, we could just take the first 8 chars and call it enough, or?

Sure, sure. We can use 8 or 12 or 4 characters but it is a band aid because the total length is important. I looks like we already check the overall value but maybe more than console.warn is needed? https://github.com/microsoft/vscode/blob/7b3474abff37d9ebbe11e48ed8aa38bd077bdf2e/src/vs/base/parts/ipc/node/ipc.net.ts#L749

Yeah... maybe something more dramatically is called for here. But it needs to strike a good balance since it's still possible to open, save and close files.


Update cmake + use new mechanism to deliver nanodiamond NCMAT file

Hi @nicriz
So here are the latest updates concerning how to share your nanodiamond ncmat file with users of your plugin. It requires NCrystal 2.4.0, and your datafile is once again just a physical file. It is placed in <root>/data/ncplugin-SANSND_nanodiamond.ncmat. Long story short: no matter how the plugin is used, it will be available and can be referred to by its filename ncplugin-SANSND_nanodiamond.ncmat. If you want to share more files in the future, just add more files in the same directory.
I notice that this data file and testcode/data/custom.ncmat has almost identical contents. To avoid confusion, I suggest you remove testcode/data/custom.ncmat and change all code referring to it to simply refer to the ncplugin-SANSND_nanodiamond.ncmat file now. You can of course have files in testcode/data, but it should be for when their contents are different - and when the file is not supposed to be used by other users.
Cheers,
Thomas

Hi Thomas, everything clear from my side 👌 I will update the SANSND plugin to be more in line with the new release. Thank you  for this new feature!


Hosting Error:ErrorNotAuthorized

I have built the CloudARPin example to both android and iOS, but I am getting a Hosting Error:ErrorNotAuthorized on both devices when I try to host. This happens right after I place the pin. I added an API key to the android build, but not the ios build, should I add the same key to both, or remove the API key from android build? I'm not too new to AR dev, but I am kind of new to the cloud side of AR dev.
UnrealEngine-4.20-arcore
Samsung s9 version 8.0
iPhone 11 version 11.4

Yeah, you should add API key to both iOS and Android field. Whether you can use the same API key depends on how you setup your API key. If it is an unrestricted API key, you can use the same key for both Android and IOS. Otherwise, you need two different keys for different platform.

Thanks, that makes sense. I have added my API key, which is unrestricted, to both android and ios ... but I still receive the Hosting Error:ErrorNotAuthorized.
I see nothing in my Output log that would indicate any issue or direct me down a path to solve the issue.

Ahh ... that was it. I originally enabkes it, I thought, but just checked and it was disabled.
Working now. Issue resolved.
Thank you.

Before this get closed, and maybe I should open a new issue for this ...
I am able to host a session from each device, but I am not able to join the session from the other (tried both ways). What would be causing this failure to join?

I have only tried android to ios, I can try android to android as well as ios to ios tmrw at the office. I am on the same network, not 100% if my network allows p2p, I was just looking into that, do you know how I can confirm that it does or not?

You can try ping other phone using its ip address using "adb shell ping" and see if it can success.

Yes, deff allows p2p ... I was able to ping my device.

Got it to work on iOS to iOS, so I will close this issue. If you can point me toward some documentation and/or example projects that have a work around for connecting android and iOS devices, that would be appreciated.
Thanks for your help.

I think the connection issue between Android and iOS is only an issue in 4.20, which I believe is introduced by Unreal upgrade iOS ip address to ipv6. You can try use some other network solution to pass around the Cloud ARPin ID. Or if you want to keep using local network, you can try use the 4.19-arcore branch which doesn't have the issue but 1 version behind.

https://github.com/google-ar/arcore-unreal-sdk/issues/35#issuecomment-419526790
@bopangzz will this be fixed in a future version of the plugin out of interest?

The issue is not in the Google ARCore Plugin but a Unreal Engine networking issue. Epic will have to fix the issue in future Engine releases.


Fixed Magma cube sound and spawn size

Based from the Slime itself.
I spawned a few of them to test the sound and realized the big one couldn't spawn. The size seems to be on par with vanilla Minecraft.

This looked good, why did you close? Squash the commits into a single one and it can be merged.


python312Packages.{webassets, flask-assets}: remove nose; modernize

Follows up and includes the work from #334483. Fixes some broken subsitutions in flask-assets and enables some tests in webassets.
Part of #326513.
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

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

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

[x] Fits CONTRIBUTING.md.

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

Currently running nixpkgs-review, will add output of that below once I've finished running it.

Getting some build errors, trying to isolate the issue now.


Missing logout icon

Can't find this icon in @material-ui/icons 4.11.2:
https://material.io/resources/icons/?search=logout&icon=logout&style=baseline

My bad, it's been added into MUI v5


CUDA 11.0RC

Closes #302

Fixes the test suite for CUDA 11.0.167 under linux
Upgrades jitify to the latest version (commit 2a015bb) which works with CUDA 11.0 RC
Adds CUDA 11.0RC to the Ubuntu and Windows github action worflows as build matrix options.

Tested generated executables locally under linux.
Windows executables untested (will likely wait until a non RC to install CUDA 11.0 on windows locally).

Can this be checked to ensure tests run on windows before merge. @Robadob

Tested with CUDA11, found tests failed to run due to changes to jitify::demangle method.
I've re-implemented a minimal implementation of what jitify was previously doing for Windows. Will test again and merge if successful.


Not working with chartjs 3.5.1

I need to use this plugin for styling tooltip boxShadow but in latest version of chart.js all the options are different so it's not working with latest version :(
Any Solutions / Fix?

I think its mostly due to the helpers classes that changed. Any ideas how to fix it?

Need help with the same. Please update this library to support the latest chartjs versions.

same here

+1

Me too

I guess someone has to rollout new version for this because v3 of chartjs has changed a lot, I will try to update this one but not sure if new version could be made compatible with older versions because I don't have any idea in writing open source


Issue Bulk Upserting Records and Associating with an External ID

Good Morning Folks,
We're developing an implementation for a client with DeveloperForce.Force v2.1.0 and are hopeful that we can get some help on an issue we can't seem to resolve regarding relationships with other objects.
We're following the SF documentation provided here for Bulk API and also in the section entitled "Upserting Records and Associating with an External ID" here for REST API which seem to have the same format.
We are calling the library with batches like this:
var result = await forceClient.RunJobAndPollAsync("Opportunity", "TransactionID__c", BulkConstants.OperationType.Upsert, sObjectList);
This is the actual batch JSON recordlist provided to the method where we are trying to create the relationship between the Opportunity and the Account on the account's external ID field:
[
  [
    {
      "RecordTypeId": "XXXXXXXXXXXXXXXX",
      "TransactionID__c": "1",
      "Account": {
        "Station_ID__c": "47551"
      },
      "Dealer__c": null,
      "Amount": 0,
      "Description": null,
      "Date_Time__c": "2010-04-27T14:41:09Z",
      "Status__c": "Approved",
      "Card_Type__c": "Test Card",
      "Member_Number__c": "111111111",
      "License_Plate__c": null,
      "License_Plate_State__c": null,
      "Service__c": "Full Service",
      "StageName": "Closed Won",
      "CloseDate": "2021-09-09T14:37:43.0371832Z",
      "Name": "Station Transaction"
    }
  ]
]

The response we get back from the server is:
{"Fields":["Account"],"Message":"Opportunity: bad field names on insert/update call: Account","StatusCode":"INVALID_FIELD_FOR_INSERT_UPDATE"}
But when we test using UpsertExternalAsync with the same payload of the first batch item, the record is created and linked successfully:
var result = await forceClient.UpsertExternalAsync("Opportunity", "TransactionID__c", 1);
We need to use batches because we have hundreds of thousands of records to sync. Any assistance you can provide would be greatly appreciated!

We managed to resolve this issue. The library turns the SObject into an XML formatted payload within the code. We had been using strongly typed classes similar to the one below for the Account relationship lookups, which was causing the issue when it was being converted into XML format:
public class OpportunitySObject
{
    public string RecordTypeId { get; } = "000000000000000";
    public int External_ID__c { get; set; }
    public KeyValuePair<string, string> Account { get; set; }
    ...
}

When the KV pair for Account was being passed into the RunJobAndPollAsync method in the SObjectList it resulted in the error we were seeing above because it wasn't being formatted properly in XML.
What we had to do was create a new class to hold the relationship lookup by the Account's external id similar to this:
public class OpportunityAccountRelationshipSObject
{
    public string External_ID__c { get; set; }
}

And modify the base OpportunitySObject by adding a new SObject with the OpportunityAccountRelationshipSObject to it.
We hope that will help anyone else that may encounter it.


slowing down issue, during realtime face detection

I m trying to change this code to run with real-time video by using thread.
but It tends to slow down like half of its speed after several minutes.
so I ran this code as thread with static photo, but had same result.
any suggestion would be Thankful.
TIA !

I haven't tried thread，you can refer to this section  https://github.com/jackweiwang/Android-FaceDetection-UltraNet-MNN/blob/6f4e5e0184bbdfe6ed6bcf9b993d393bc03f18a2/app/src/main/cpp/net.cpp#L117-L121

tried to change the code that you pointed out, but I couldn't fix any of it.
could you explain it more specific ? cause Im not good at handling with C :(


Ensure API Gateway has X-Ray Tracing enabled

Possible check to target the following resource

[ ] aws_api_gateway_stage

closing, tracking in spreadsheet

closing, tracking in spreadsheet


Refine image search prototype

The image search prototype of https://github.com/wellcometrust/wellcomecollection.org/pull/4898 is a first pass, next we need to:

~Add related images to the panel~
Ensure copy and link to work/item is correct in panel
Add tracking for panel expansion etc
~Remove the separate image search page~ ~and toggle~ ~and add an "opt-in" to the works search to go to an images-only search.~
~Open in same tab and remember search position on back to results~

cc @jennpb

Before we make this available to everyone, I'd like to:

change preview panel background to charcoal / use white for text
Change primary CTA button label to View image
Change secondary CTA to LinkLabels style; label text reads About this work
Change Related images to Visually similar

Further refinements can be made later.
cc @GarethOrmerod @davidpmccormick @gestchild


How use Confirm notification ?

You have feature Confirm notify ?
I want to use "Confirm" button in notification box alert.

There is no Confirm button feature but you can do I easily using custom body:
<notifications :close-on-click="false">
  <template #body="{ close, item }">
      <span>
        {{ item.title }}
      </span>

      <button @click="close">
        Confirm
      </button>
    </div>
  </template>
</notifications>


Wrong redirection after removing myself as project admin in project collaboration

Describe the bug
When I (user flaurens - system admin) want to remove myself as project admin in one project (e.g. Anything), I am redirected to the project description page, no toolbar menu displayed (http://<IP_ADDRESS>:4200/project/0001/info). Probably user permission as project admin involved.
To Reproduce
Steps to reproduce the behavior:

Go to 'http://<IP_ADDRESS>:4200/project/0001/collaboration'
Add yourself as project admin
Click on Remove as project admin
See error --> redirection to http://<IP_ADDRESS>:4200/project/0001/info

Expected behavior
It should reload the content of the collaboration page - no redirection.
Desktop (please complete the following information):

OS: MacOS CAtalina
Browser: Chrome

Additional context
It works well with other users than my own account on other projects.

I'm not able to reproduce the issue. But I still have knora-api v10.1.1
I'll update the version and test it again

...no, I'm not able to reproduce your issue.

It should reload the content of the collaboration page - no redirection.

There's no redirection defined. The refresh is defined as follow:
refresh(): void {
  // referesh the component
  this.loading = true;
  this.initList();
}

But I see, there's an issue in "session" in the localStorage. The logged-in user should have an array of project IRIs, where she/he's project admin. But in my case the list is empty.

I will fix all issues here. You're right...in another part in users-list is a redirection to project page. I don't remember why

project description page, no toolbar menu displayed

a „normal“ user without system or project admin rights does not have access to collaboration, groups, data model an lists in the admin interface of project. @flavens we can discuss it today whether this is the right approach.


How to close parameter server with python?

In the TensorFlow documentation, the tf.train.Server.join() will block forever, but I would like to close it after training. Are there any possible solutions for it?
Thanks.

Maybe you can create a Supervisor which you then monitor from a separate thread? If you get a should_stop you exit the entire process. I have not tried it myself but could be worth trying.

This should be an easy question for @mrry to answer.

There's no built-in API for shutting down a TensorFlow server remotely; the assumption is that whatever mechanism (e.g. a cluster manager) you used to start the processes would also kill them after the job completes.
If you're feeling adventurous, you could try using TensorFlow coordination mechanisms, like a tf.FIFOQueue to signal when training is complete. The basic idea would be to (1) create a shared queue on one of the PS tasks; (2) on each of the PS tasks, run a dequeue op on that queue instead of calling `server.join(); (3) at the end of training enqueue_many N elements into the queue (where N is the number of parameter servers). If you did that, the PS tasks would all unblock when training was done, and exit.

This looks like a satisfactory answer to me, so I'm going to close this out. Let me know if you want me to reopen this @LiamHe.

Would you reopen this issue, please?  In our cluster setup (LSF+MPI shared by many different groups), it requires extra efforts to monitor a TF job in order to stop it after training.  A simple wait-then-close method for parameter server would be much appreciated because it increases ease-of-use a lot, especially in a shared cluster where parameter servers should be closed after every single job.  Thank you.
For your reference, this issue is brought up again here: https://github.com/tensorflow/ecosystem/issues/19.

There's a work-around -- instead of server.join() have your parameter servers do sess.run(queue.dequeue) k times if there are k workers. Then each worker does an enqueue at the end of it's work. Here's a full example, search for create_done_queues here. An alternative solution was started in this pr -- https://github.com/tensorflow/tensorflow/pull/6185

Hmm, that's a nice option. @martinwicke FYI, possibly worthwhile doing in tf.learn by default?

@jhseu this is not a reasonable default since workers can in some settings restart unpredictably during training

@alextp @jhseu How about adding it as an option, disabled by default?  The user may turn it on if desired.

Stop ps server gracefully is a requirement when run distributed training with kubernetes batch job. Anyone can write a detail demo base on mnist_replica? @mrry @jart

@hustcat http://stackoverflow.com/questions/39810356/shut-down-server-in-tensorflow/40186129#40186129

I write a demo for MNIST_data, and it seems run OK. See dist_fifo.


MINOR: distinguish between missing source topics and internal assignment errors

Minor followup to KAFKA-10559
I noticed that we were converting any and all TaskAssignmentException to the INCOMPLETE_SOURCE_TOPIC_METADATA error code to shut down all the clients. Since these errors are typically fatal, it does seem appropriate to propagate the shutdown command. But we should do so with a new AssignorError instead of piggy-backing on the INCOMPLETE_SOURCE_TOPIC_METADATA, which would be pretty confusing for users who do in fact have all their source topics.
Changes in this PR:

Add new AssignorError.ASSIGNMENT_ERROR for generic assignment errors
Missing source topics --> throw/catch MissingSourceTopicException --> INCOMPLETE_SOURCE_TOPIC_METADATA
Internal assignment errors --> throw/catch TaskAssignmentException --> ASSIGNMENT_ERROR

Should be cherry-picked to 2.7

heads up @mjsax @guozhangwang

Merged to trunk

Cherry-picked to 2.7 (had to fix up a test in StreamsPartitionAssignorTest that was relying on the new ReferenceContainer which is only in trunk)


Make Foundation compile in Xcode again

The code is still using version 3 syntax so pass the
-swift-version 3 flag

@phausler This allowed foundation to compile in Xcode with latest toolchain

It seems like we're not too far away from being Swift 4 compliant, so maybe hold off on this?

@swift-ci test and merge

Actually let's just merge this since it's Mac only.


Add current status or implementation history of proposals

What is the URL of the document?
Each proposal doc at https://github.com/aws-controllers-k8s/community/tree/main/docs/design/proposals

Which section(s) is the issue in?
Missing section

What needs fixing?
Add current Status or Implementation History to each proposal
Ex: https://github.com/kubernetes/enhancements/tree/master/keps/provider-aws/629-alb-ingress-controller#implementation-history

Additional context

/lifecycle frozen


[KFNBC]: Launcher can only launch Jupyter-based images

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
The logic for the launcher is at the moment purely focused on Jupyter Notebooks. It introduces many things in the Notebook definition (and then the Pods) that may not be relevant for every custom images, like some env vars, or make them simply unusable like readiness and liveness probes (because of course there is nothing to answer at the specified address).
As a concrete example, a perfectly legit image for the Notebook Controller may not work because the readiness/liveness probes injected by our launcher/dashboard check for a non-existent API, therefore preventing the Pod to reach running state. At the moment, the probes are hard-coded there: https://github.com/opendatahub-io/odh-dashboard/blob/14a2e2f33e2439c9fe143be3dfaf878b8a39e586/frontend/src/services/notebookService.ts#L112
Expected Behavior
In a more general way, here are the items that should be customizable per type of image if we want to enable the full power of custom images (or simply let users work with their own material, as the Notebook Controller was designed for):

Readiness and Liveness probes: using different services or mechanisms.
User PVC mount point (could be different than /opt/app-root/src which is the current norm only because of the S2I support in our general images). Some images may even want to disable this user PVC.
Supplementary mounts: other volumes and mount points should be allowed. It was the case previously with the single-user profile mechanism that allowed to do this through a simple ConfigMap.
Environment variables: different of supplementaty ones should be configurable.
As there may be too many parameters to take into account, including fool-proofing future ones, a simple method could be to allow a ConfigMap with the same name as the ImageStream to keep all of these configurations. Then instead of hard-coding the Notebook configuration here , it could be retrieved on a per-case bases. The existing value could be the default, replaced by the content of the relevant ConfigMap if indicated as such in the ImageStream (maybe an annotation with the name of the CM to use for Notebook stub).

Using different types of images (RStudio, VSCode,...) also has impact on the UX. Currently, the launcher has only one tile "Jupyter", leadind to the selection of the image to launch. Which may not be Jupyter at all... We have different choices here:

have different tiles for the different tools. Drawback is that it forces to duplicate/customize the Start page for each of those images (that we may not know in advance).
change the launcher entry name for "Notebooks" or "Servers" (original KF naming), and keep the Start page identical, putting all the "notebooks" images accessible from there. Drawback: change of tile Icon, name, and description to include the fact that it may be more than Jupyter images.

Steps To Reproduce
No response
Workaround (if any)
No response
OpenShift Version
No response
Openshift Version
No response
What browsers are you seeing the problem on?
No response
Open Data Hub Version
No response
Relevant log output
No response

Moved to kind/enhancement  since this is a new feature that wasn't initially planned in https://github.com/opendatahub-io/odh-dashboard/issues/213

Compatible images can be created without modifications, as done here: https://github.com/opendatahub-io-contrib/workbench-images/releases/tag/2023b_2023-02-01
Some points are still valid (custom mountpoints, dedicated launch tiles,...), but they are already followed in other issues.


Update to memcached?

Let's see if it's possible to do this. https://devcenter.heroku.com/articles/memcachedcloud#using-memcached-from-python I don't know if it uses up a "dyno" is the issue. If it doesn't, then let's do it.

Basically, the issue is the simplecache seems to be storing multple values for the same key.
So when that happens it cycles through the cache, e.g. manually update a flag from blue to red, and every time you refresh it alternates between the before and after state. It's really weird.
They put a warning on it that simplecache is good for dev, and I guess that's what they mean. At the moment this is what I have in the admin panel as a temporary fix.

Basically, the issue is the simplecache seems to be storing multple values for the same key.
So when that happens it cycles through the cache, e.g. manually update a flag from blue to red, and every time you refresh it alternates between the before and after state. It's really weird.
They put a warning on it that simplecache is good for dev, and I guess that's what they mean. At the moment this is what I have in the admin panel as a temporary fix.

OK, actually we can use Redis for free on Heroku. Neat! #162


Deleting bookmarks doesn't exist

IIRC I don't have it implemented for the server yet, but even if that's not the issue, when using the CLI, trying to confirm a deletion just repeats the text again and says "No bookmarks deleted."

Should be fixed now.


Themes cannot be deleted.

Once a theme is created, it can't be deleted, only edited.

Reopened because 86d1070 closes #66, not this issue.


Error in upload trigger

I tried deploying the code and tested out by uploading a document.
The document was uploaded to S3 but the upload trigger function throws error, as head object not found
Logs:
[ERROR] ClientError: An error occurred (404) when calling the HeadObject operation: Not Found
Traceback (most recent call last):
  File "/opt/python/aws_lambda_powertools/logging/logger.py", line 441, in decorate
    return lambda_handler(event, context, *args, **kwargs)
  File "/var/task/main.py", line 31, in lambda_handler
    s3.download_file(BUCKET, key, f"/tmp/{file_name}")
  File "/var/task/boto3/s3/inject.py", line 190, in download_file
    return transfer.download_file(
  File "/var/task/boto3/s3/transfer.py", line 326, in download_file
    future.result()
  File "/var/task/s3transfer/futures.py", line 103, in result
    return self._coordinator.result()
  File "/var/task/s3transfer/futures.py", line 266, in result
    raise self._exception
  File "/var/task/s3transfer/tasks.py", line 269, in _main
    self._submit(transfer_future=transfer_future, **kwargs)
  File "/var/task/s3transfer/download.py", line 354, in _submit
    response = client.head_object(
  File "/var/task/botocore/client.py", line 535, in _api_call
    return self._make_api_call(operation_name, kwargs)
  File "/var/task/botocore/client.py", line 980, in _make_api_call
    raise error_class(parsed_response, operation_name)

Make sure the filename you are uploading doesn't contain spaces.

Thanks, that fixed it. Could you please mention it in README file or rename the file and remove spaces before uploading to S3.

Now fixed e8a5cbf


Incorrect Header Values Displayed for Non-Existent Addresses on Teloscan

Describe the bug
When searching for a non-existent address on Teloscan, the header values (balance and nonce) from the previously searched address are displayed instead of showing the correct values for a non-existent address.
To Reproduce
Steps to reproduce the behavior:
Go to 'Teloscan'
Search for the address '0x4DaEFE5943b29940b0c55C1410A61633b9048e66'
Next, search for the non-existent address '0xA308816024FE8ddbCD202406b57F27AC580c42ce'
Observe that the balance and nonce values from the first address are still displayed
Expected behavior
When searching for a non-existent address, the expected behavior should display a 0 balance and 0 nonce, similar to how Etherscan handles non-existent addresses. Etherscan shows 0 balance, 0 nonce, and no matching entries in the data tables. The displayed values should not be carried over from a previous search.

Fixed in #368, close to confirm


v5.0.0 tracking issue

This release's main addition is the upgrade to Cosmos SDK 0.46.
Milestones
v5.0.0 milestone
IBC spec compatibility
v1.0
QA
Backwards compatibility

Fungible token transfers

Over a non-incentivised channel:

Successful token transfer:

[ ] chain A (with ibc-go v5) -> chain B (ibc-go v4)
[ ] chain A (with ibc-go v5) -> chain B (ibc-go v3)
[ ] chain A (with ibc-go v5) -> chain B (ibc-go v2)
[ ] chain A (with ibc-go v4) -> chain B (ibc-go v5)
[ ] chain A (with ibc-go v3) -> chain B (ibc-go v5)
[ ] chain A (with ibc-go v2) -> chain B (ibc-go v5)

Failed token transfer (e.g. receiver address is not valid)

[ ] chain A (with ibc-go v5) -> chain B (ibc-go v4)
[ ] chain A (with ibc-go v5) -> chain B (ibc-go v3)
[ ] chain A (with ibc-go v5) -> chain B (ibc-go v2)
[ ] chain A (with ibc-go v4) -> chain B (ibc-go v5)
[ ] chain A (with ibc-go v3) -> chain B (ibc-go v5)
[ ] chain A (with ibc-go v2) -> chain B (ibc-go v5)

Over an incentivised channel:

Successful token transfer:

[ ] chain A (with ibc-go v5) -> chain B (ibc-go v4)
[ ] chain A (with ibc-go v4) -> chain B (ibc-go v5)

Failed token transfer (e.g. receiver address is not valid)

[ ] chain A (with ibc-go v5) -> chain B (ibc-go v4)
[ ] chain A (with ibc-go v4) -> chain B (ibc-go v5)

Interchain Accounts

Over a non-incentivised channel:

Successful interchain account registration AND successful bank transfer on the host (from the interchain account):

[ ] controller chain (with ibc-go v5) / host chain (ibc-go v4)
[ ] controller chain (with ibc-go v5) / host chain (ibc-go v3)
[ ] controller chain (with ibc-go v4) / host chain (ibc-go v5)
[ ] controller chain (with ibc-go v3) / host chain (ibc-go v5)

Successful interchain account registration BUT failed bank transfer (e.g. receiver address is not valid) on the host (from the interchain account):

[ ] controller chain (with ibc-go v5) / host chain (ibc-go v4)
[ ] controller chain (with ibc-go v5) / host chain (ibc-go v3)
[ ] controller chain (with ibc-go v4) / host chain (ibc-go v5)
[ ] controller chain (with ibc-go v3) / host chain (ibc-go v5)

Over an incentivised channel:

Successful interchain account registration AND successful bank transfer on the host (from the interchain account):

[ ] controller chain (with ibc-go v5) / host chain (ibc-go v4)
[ ] controller chain (with ibc-go v4) / host chain (ibc-go v5)

Successful interchain account registration BUT failed bank transfer (e.g. receiver address is not valid) on the host (from the interchain account):

[ ] controller chain (with ibc-go v5) / host chain (ibc-go v4)
[ ] controller chain (with ibc-go v4) / host chain (ibc-go v5)

Migration

Checklist

[x] Bump go package version.
[x] Change all imports starting with github.com/cosmos/ibc-go/v4 to github.com/cosmos/ibc-go/v5.
[x] Branch off main to create release branch in the form  of release/v5.0.x.
[x] Add branch protection rules to new release branch.
[x] Add backport task to mergify.yml
[x] Upgrade ibc-go version in ibctest.
[ ] Check Swagger is up-to-date.

Post-release checklist

[ ] Update CHANGELOG.md
[ ] Update the table of supported release lines (and End of Life dates) in RELEASES.md.
[ ] Update version matrix in RELEASES.md.
[ ] Update docs site:

[ ] Add new release branch to docs/versions file.
[ ] Add label and key to versions array in config.js.

[ ] After changes to docs site are deployed, check ibc.cosmos.network is updated.

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate contributors tagged/assigned

We should add any automated tests that get added as a result of this work to #1607

v5.0.0 is released.


Update the docs

After recent changes the docs are no longer up-to-date.

Closing as a duplicate of #299 .


Support passing room to useIsEncrypted

I am aware this is alpha, so I am not sure how much support this gets.
I would find it useful if the hook could also get the room passed directly as other hooks so it can be used in a place where Room is manually created, where the room context is not available. :pray:
I was not sure whether the hook should only get options where both participant and room could be. I can rewrite it to that if that makes more sense. :pray:

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

this makes sense, thanks for your contribution!


Implement a mark function so we're compaction friendly

If we don't manually mark members of the struct, they can move out from
underneath us.  For now lets just mark them so everything stays stable.

Oh we can totally delete it I just forgot to hit the button


How to delete a comment in issue

Thanks your feedback!
Currently not supported, but will be added.

+1

Patch has pushed to fix this issue, please test on develop branch or https://try.gogs.io.


Remove assertion of 'root_certificates' value.

As previously documented in README.md, client can work with Tls=True without setting 'root_certificates' if grpc lib can verify server cert using locally installed CA certs.
Previously mentioned in #18.

Fixed in de3143c.


Add Angular  project Example

Hi, it would be great an angular example as starter
Thanks

Hi @jromerob fill free to provide a Pull Request.
You can try to migrate this example: mxGraph integration with Angular 13 https://github.com/Dhawaldeep/mxgraph-angular-tteen
The example uses mxGraph with TypeScript support provided by typed-mxgraph


do nothing in ssl_test() when build with -DDISABLE_SSL=1

@byronhe Thank you for finding this! Instead of making the test always succeeded, can we skip the test itself like this?
#if !SSL_LIBRARY_NOT_FOUND && (defined(__linux__) || defined(__APPLE__))
    ts.doTest( "ssl test",
               ssl_test );
#endif


PLT - Aggregate similar solvers in legend

This enhances the plotting part:

Aggregate solvers in legend under the same name
enable hiding/showing aggregated solvers in legend and plots

Feature preview

Resnet benchmark
Lasso Benchmark

next step

enhance styling
more complete filtering tool

P.S: similar solvers means the same solver run with different parameters

@tomMoral, I uploaded the feature preview instead.
Have a look at https://github.com/benchopt/benchopt/pull/474#issue-1440491550

@tomMoral, I consider your remarks in the last changes
pls have a look on the feature preview https://github.com/benchopt/benchopt/pull/474#issue-1440491550

When only one is available, I would not create a group no?

I think it is better to keep the group structure to preserve homogeneity. Otherwise, the result looks a bit weird (see the Lasso benchmark in #474)

I think it is better to keep the group structure to preserve homogeneity. Otherwise, the result looks a bit weird (see the Lasso benchmark in #474)

I managed to construct groups for solvers that are run with multiple configurations of parameters
@bmalezieux, could you check whether it renders well for you?

Resnet benchmark
Lasso Benchmark

I think it is better to keep the group structure to preserve homogeneity. Otherwise, the result looks a bit weird (see the Lasso benchmark in #474)

I managed to construct groups for solvers that are run with multiple configurations of parameters @bmalezieux, could you check whether it renders well for you?

Resnet benchmark
Lasso Benchmark

Actually, I preferred the last version with one block per solver, even with a single configuration. What disturbs me is that the solvers should be represented in the same way in the legend, and this is not the case here in my opinion.

WDYT @tomMoral?

I agree that we should find a consistent way to do this but I don't like the fact that we have to scroll to have the full legend, even when all legend labels could be put on 2/3 lines. This is something we changed 6 months ago so it feels that we are going back.
We need to find a better way to present this than big blocks that stack over each other.
Maybe simply make the parent block also a float, that do not take the full screen. That way, solvers with one config will jsut have an extra gray box above with the name of the solver and multiple solver will have a box around grouping them together?

WDYT @tomMoral ?

@bmalezieux any thoughts about https://github.com/benchopt/benchopt/pull/474#issuecomment-1320404474?

It looks good ! I think it would be even better to add the name of the solver above solo solvers (it is what Thomas meant if I understand the drawing correctly).

Yes I like the style!! :)
As @bmalezieux  said, I would keep the box above the solo solvers.
Also, could you make it so that if we fall a box, it stay the same width?

By the way, I find it clearer to put the style in the css file and not in the class name.

I'm late to the party, but instead of graying the unselected solvers, would it be possible to collapse the unselected block, displaying only its title ? If there are many solver sin a block and they're unselected, one still has to croll a lot to go to the next block


INT - B-22083 AOA Packet Won't Download Due to Deleted Order Record

B-22083
Summary
This fixes the issue where the AOA packet and SSW would not download when a uploaded order had been deleted. The issue was due to pop query not filtering out soft deleted files correctly when fetching docs to build pdfs. Solved by filtering out the userUploads with deleted_at. Tried to filter it using pop, but could not get it it work.
How to test

As a customer -> Create a PPM move and choose yes when asked Would you like to request an advance on your incentive? in Avances page and put in any amount.
As a counselor -> Click edit the shipment and continue to 2nd page to approve the AOA. Edit orders and fill in the required entries -> submit the move.
Download AOA Paperwork (PDF) link should appear in the shipment card. Click and verify that it downloads without an error.
As the customer -> upload ppm docs and submit.
As the closeout counselor -> review and approve all the docs. Verify you can download the payment packet after without an error.

Screenshots

sorry for the delay. lgtm 👽

No worries, thanks!

Test coverage on server failed.
Last run: Wed 11 Dec 2024 10:34:41 PM UTC
Refer to this report to see coverage details for the server app.
Documentation for test coverage can be found at this link.


Fix README.md: --offsetStorage is not a valid flag

Thanks so much for creating this tool, I discovered it today and it was super helpful in debugging some issues.
I noticed --offsetStorage is not a valid flag and had to figure out the right usage. I've updated your README.md accordingly.

@Mortimerp9

@Mortimerp9 Just noticed you approved this, thanks! Since I'm not the owner / a developer of this project, I can't merge this pull request. Can you please merge it or let me know who can? Thank you! This tool was very useful in debugging some issues I was having

Ping -- this tool is really useful and this simple doc update will allow it to be useful to more people.


Laravel 5

This seems like a great package and I regret that haven't found it in my previous projects. With the release of Laravel 5, I started my new projects with it too. It would be great too if you would release the version that supports 5. I know you planned it, but is there any new news? Like how's the progress and such and when you expect to release it? If you've not done anything yet I might find some time at home to do it.

+1

+1

+1

+1

:+1:

+1

:+1: :+1:

+1

It appears @ollieread already responded to this issue and closed it.
https://github.com/ollieread/multiauth/issues/82
I guess its up to another user to do this. I'd volunteer, but I just started using laravel so I wouldn't even know where to begin.

Check the bottom of this form post on laracsts, maybe it will help some of you. I haven't tried it yet but I'm about to. Perhaps @ollieread can update this git using the information?
https://laracasts.com/discuss/channels/general-discussion/multiple-authentication-in-laravel-5-ldap-and-default-auth

It's more that I haven't had much time to do this yet as I've been working a lot. I will do when I get chance, but others are welcome to make PRs.

Well that stinks, I guess I'll have to work around that part of the system I'm creating as best as I can. I think this issue should remain open however so other users trying to figure out why this won't work on laravel 5 can see it. I wouldn't have thought to look in the closed section of the issues area when I first realized it and would have made a duplicate post. I see the closed being a solved/declined rather than a pending/on hold. That's just my opinion.

Check my fork https://github.com/sboo/multiauth and post on laracasts. maybe that helps
https://laracasts.com/discuss/channels/general-discussion/multiple-authentication-in-laravel-5-ldap-and-default-auth/replies/44260

Hi!
I'm using Laravel 5 native auth to handle multiple auth contexts (database
tables)... It's not difficult, please check this Gist:
https://gist.github.com/danielcoimbra/64b779b4d9e522bc3373
Daniel
2015-03-18 22:40 GMT-03:00 Dubz<EMAIL_ADDRESS>
Well that stinks, I guess I'll have to work around that part of the system
I'm creating as best as I can. I think this issue should remain open
however so other users trying to figure out why this won't work on laravel
5 can see it. I wouldn't have thought to look in the closed section of the
issues area when I first realized it and would have made a duplicate post.
I see the closed being a solved/declined rather than a pending/on hold.
That's just my opinion.
—
Reply to this email directly or view it on GitHub
https://github.com/ollieread/multiauth/issues/86#issuecomment-83253810.

@visualcom Changing the value of auth.model is a clever solution, but the session data still the same for both auth contexts, causing when you call Auth::logout() both "sessions" are closed.
This is because Guard::getName() use the same name for the session data no matter what model we are using... :/
It seems we need a more complex solution like @sboo repo.


x/pkgsite: src/encoding/json/decode.go | add DecodeValue to JSON decoder

The Json decoder doesn´t have a DecodeValue function (like the gob encoder).
This is a feature request for that feature. The DecodeValue function is pretty important in JSON due to being a very dynamic type.

Umarshal can be called with reflection.Value.Interface()


[8.x] Remove useless parenthesis.

This PR ensures that useless parentheses are removed.

Are all these huge changes going to give me merge conflict nightmares soon 😬

👎 for this change, for that reason. StyleCI can also get rid of these brackets, if wanted. That is not in the Laravel preset at the moment.

Are all these huge changes going to give me merge conflict nightmares soon 😬

I would absolutely prefer to avoid this! I also hate conflicts

👎 for this change, for that reason. StyleCI can also get rid of these brackets, if wanted. That is not in the Laravel preset at the moment.

Any reason why this is not enabled?

Any reason why this is not enabled?

Because enabling it would cause conflicts, and applying that code style rule is not urgent. It doesn't do much to improve readability.

Tbvh, there's lots of situations in this PR where parenthesis make the code more clear to read. Take the following for example:
if ($times < 1 || ($when && ! $when($e))) {
    throw $e;
}

vs.
if ($times < 1 || $when && ! $when($e)) {
    throw $e;
}

The former is much more clear and immediately discerns the logic on how the operation is handled while for the latter you'll have to process it more to know how it's actually handled. Both are syntactic correct but there's something to be said for code clarity.


To add a calculator for calculating equation of a plane in intercept form

Is your feature request related to a problem? Please describe.
To add a calculator for calculating equation of a plane in intercept form

Hi, I want to work on this issue I will start working on it as soon as I get assigned!! I am a part of GSSoC'21.
Kindly assign this issue to me!
Discord Username : Abhijeet Sinha(P)
Discord Tag: #4018

Go ahead
Deadline 22 May,2021

@sairish2001 can I work??

@Amit366 go ahead
Deadline 31 May,2021


Make recyclers great again

Really it was just rotating the sprite and adding a tag to a bunch of stuff.
Combining recycled materials into metal or whatever is someone else's concern if they want it.

Tag's cheaper than a component, especially if the entity already has tags.
I don't want spacing to be a thing because it cucks shuttles very hard, and if people really want incineration over just deleting then that's eh.


Why not use govalidator internally?

Hello,
I find this package extremely useful but I was wondering, why not use https://github.com/asaskevich/govalidator internally? it can be pretty powerful and also much simpler to add custom validators.

I have strong opinions about validators -- you're welcome to use it but I don't want to build one into the package.
Thanks for the suggestion though!


Views

This is a plan for introducing views into 0.6.0. Views are a flexible system for grouping widgets together onto a single container, and then laying them out automatically.
A view consists of a freeform rectangle on the screen, as well as extra layout metadata: which side of the screen the view is docked on, whether it's floating, so on and so forth. The exact details will be fleshed out when this is implemented.
Views would be used to implement a few new features, including a settings menu (which is where the dark mode toggle is going to be moved), a hamburger menu in the paint state, where the room ID and save button are going to be moved, a color picker and palette editor, and in future versions, tool menus, for selecting variants of existing tools (such as selection shapes, brush types, among other things), and a menu for managing online users - giving them permission to draw, kicking them, banning them, maybe also a dialog box for users that are trying to join, to let them in, kick them out, or ban them.
Tracking issues for the aforementioned features:

#63
#64
#65
#66
#67

All seems to be done for now, closing.
The settings menu has been removed from the TODO as there isn't really anything to put there yet. Maybe once internationalization (#36) is done we can put language selection as well as keybindings there.


improvements

Hi Amine,
It is nice to meet you.
Please find my pull request for new code I added to your awesome tool.
Many thanks,
Cristiano

thanks for the PR. btw, fixes you made in marker.sh are already present in the master branch, so you can remove them.


Merge react scripts next

This tracks commit https://github.com/facebook/create-react-app/commit/66cc7a903e91777e223780b898122a4dd3491d54
Don't merge yet, I'm testing locally to make sure this works fine.

Were you going to rebase this on top of your other CL?

This is good to go, and I managed to have green CI too.
I'm merging this and releasing under the cra-next tag on NPM.


Don't know how to build task 'jobs:work'

The Heroku logs reveal this error:
2016-04-17T20:58:17.395941+00:00 heroku[worker.1]: State changed from crashed to starting
2016-04-17T20:58:21.999649+00:00 heroku[worker.1]: Starting process with command `bundle exec rake jobs:work`
2016-04-17T20:58:22.537204+00:00 heroku[worker.1]: State changed from starting to up
2016-04-17T20:58:27.742994+00:00 app[worker.1]: rake aborted!
2016-04-17T20:58:27.743014+00:00 app[worker.1]: Don't know how to build task 'jobs:work' (see --tasks)
2016-04-17T20:58:27.743267+00:00 app[worker.1]:
2016-04-17T20:58:27.743268+00:00 app[worker.1]: (See full trace by running task with --trace)
2016-04-17T20:58:28.685267+00:00 heroku[worker.1]: State changed from up to crashed
2016-04-17T20:58:28.687831+00:00 heroku[worker.1]: Process exited with status 1
2016-04-17T21:06:31.084805+00:00 app[web.1]: source=rack-timeout id=a0185057-a69d-45a6-9825-a301a9f9da58 wait=0ms timeout=28000ms state=ready
2016-04-17T21:06:31.086118+00:00 app[web.1]: Started GET "/" for <IP_ADDRESS> at 2016-04-17 21:06:31 +0000

Googling around, it appears that Heroku expects that the "delayed_job" gem is installed as a gem, and that this is a known error in its documentation.  My guess is that the instructions were fine for Rails 2, but we're using Rails 4. See:

https://github.com/collectiveidea/delayed_job/issues/147
https://groups.google.com/forum/#!topic/heroku/Sb1TMwcc0NE
https://github.com/collectiveidea/delayed_job

The generally-recommended solution seems to be to install the "delayed_job" gem.  If anyone has any comments (pro or con), please let me know.

In particular, I would recommend installing https://github.com/collectiveidea/delayed_job_active_record

Hmm, investigating further, it appears that delayed_job_active_record is no longer recommended by Heroku: "We recommend not using delayed job for most applications due to the extra load generated on the database. Instead we recommend a Redis based queuing library." https://devcenter.heroku.com/articles/delayed-job
We may as well look into newer/better replacements now, before we integrate an older thing in.
Sigh.  It's sometimes hard to keep up with the hampster wheel of tech replacements :-).

I've run Redis on Heroku and it's perfectly fine. It is annoying to add an additional requirement for developers.
I suspect we would be fine with delayed_job_active_record for a while, although obviously if the app becomes wildly popular maybe we would start sending tons of email.

It's definitely annoying.  Is there a way to make it easy?  We need to make it easy for new developers to join the cause.  Maybe we need to create something to make it easy to get started (e.g., using Vagrant, Otto, a Docker container, or even a starter VM).

Discourse has invested a lot of effort into using Vagrant to make it easier for new developers. See http://blog.discourse.org/2013/04/discourse-as-your-first-rails-app/
I emulated them on my work project and was not happy with the experience. Vagrant works acceptably for QA, but for developers, there are too many small incompatibilities vs. working in a native environment (either OS X or Linux).
I believe you're on a better track with https://github.com/linuxfoundation/cii-best-practices-badge/blob/master/install-badge-dev-env since Redis is just a Homebrew install away.

As a quick temporary measure, commit 8b91ef15987f045051c450a3055cc4b096ba311e adds a stub "jobs:work" task to stop the whining.  I'm leaving this issue open, because there's a larger question about handling jobs (and simplifying developer installs), but we also want to have cleaner logs now so that any real problems can be quickly noted.

The "temporary" measure actually works pretty well.  Until we need more sophisticated approaches, let's just go with the cleanup of the logs.  We can add Redis or whatever later when we actually need it.


fix(acme): fix ACME renewal bug

Summary
Fix ACME renewal bug.
Using client.renew_certificate directly as the callback function in ngx_timer_at causes the parameter value to not be the plugin's config.
Checklist

[ ] The Pull Request has tests
[ ] A changelog file has been created under changelog/unreleased/kong or skip-changelog label added on PR if changelog is unnecessary. README.md
[ ] There is a user-facing docs PR against https://github.com/Kong/docs.konghq.com - PUT DOCS PR HERE

Issue reference
KAG-4008
Fix #12442

Do we have test case to verify this issue is fixed?

Do we have test case to verify this issue is fixed?

It has already been verified using my personal domain name + EC2 instance.

The test for this ticket requires binding a public IP address to a domain name, and then configuring it according to the instructions in https://docs.konghq.com/hub/kong-inc/acme/how-to/ . Afterward, send a PATCH request (also in this document), and we will see this error log in Kong's logs:
... attempt to index local 'config' (a boolean value), context: ngx.timer

After fixing it, if we execute it again, there will be no error log output, and the API will still return {"message":"Renewal process started successfully"}.

Missing tests, coverted to a draft.


6.2.x Enable portable mode

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Fix typo in README

Any clue about the workflow failure ?

Probably is just some PR permission issues. I'm merging the PR to confirm. Thanks again.

The workflow has run successfully. It's just some PR permission issues


support for windows plugins

This is intended to fix #3.
Here's my first (mostly successful) go at this. I'll add explanatory comments into the changes, but at a high level:

generate_bindings.rs now (on windows) also creates qemu_plugin_api.def
the (new) build.rs script for qemu-plugin-sys (on windows) makes the appropriate delay link .lib file from the .def. This only works in gnu-style toolchains, not -msvc style.
I've dropped in the delay link hook to qemu-plugin. It works without further config on the plugins side.
g_free is a problem - windows doesn't expose functions unless asked to, so we don't have an (easy) way of getting access to g_free. For now I've implemented it as a no-op, and so we leak memory whenever it's used.

The tiny example compiles and runs on my windows machine, as does my internal more complex plugin. tracer doesn't compile because it uses memfd, which doesn't work on windows.
Limitations:

-gnu only, because I'm using dlltool. there may be a way to get this to work on -msvc, but I've not looked into it.
g_free leaks

Wow, this is awesome, quite a productive holiday season 😉 I'll take a look!

Just merged #5 which includes these commits so I'm going to go ahead and close this one.


gateway: add load field to the session probe

This change is

We will add this once we actually use it.


css: Add function for converting css::Rule to string

@robinlinden What do you think about having a widget for showing the stylesheet?

It would be helpful to have a way of displaying the stylesheet(s)! :P
See: https://github.com/robinlinden/hastur/issues/34


Added timeout, fixed request bug, better error handling, fetched other pull requests

Fetched Pull Request https://github.com/hendrikw01/tr-064/pull/13 from @NicolaiSchmid

The XML Implementation for setting TR-064 Action was not correctly implemented.
Fixed XML Building and created a new Function Syntax.
Altered Readme accordingly.
Added Example for toggling WLAN of a FritzBox

Additional functions & Bugfixes:

New timeout function
Fixed bug with request > 3 (authorization error)
Better error handling

I think this package is not maintained anymore. But if you want, we could fork it and publish our own version.

Hey that sounds good

Okay, then I'm gonna set up a new repo, with auto deploy and invite you as a maintainer.

Can you please post the new repo and package then here please?

How times change, now I haven't used this module in three years.
But, the last working version can be found here:
https://github.com/NicolaiSchmid/tr064
Feel free to improve, make PRs or fork