Question on free look camera

Hi i have just a question.
In your readme it is stated that you have a free look camera.
It is not implemented yet?
Do you have a example showing of how to use?
Thanks
Sebastian

Hi Sebastian,
so far I had not yet had time to further joml-camera and to design and assess a possible free look camera implementation.
Much work has been going into the underlying JOML math library recently, which will enable an easy implementation of the free look camera now.
Also, thanks for your post! It gives valuable input to this project that the free look camera is actually desired, and I think a first prototype, which is not much effort thanks to JOML, will be done in the next days. :)

Thanks for your reply,
i really like your joml library. Great work!
I will look forward to your free look camera implementation.
Regards
Sebastian

:+1: for free look camera
I'm going to spend some time tinkering with the ArcBallCamera anyway, it sounds fun :smile_cat:

Btw, if there is a way I can buy you a coffee or something please let me know. JOML is a great piece of work.

Hi,
hehe, thank you!
Coffee would be nice right now. Preparing the 1.4.0 release of JOML due to tomorrow.
This will be the first version of JOML to be on Maven Central!
Since JOML grew substantially over the last time and contains some rather utility methods/classes, that aren't necessary for the majority of 3D OpenGL applications, there is now a bare-bone joml-mini artifact in the 'mini' branch of JOML. This will only contain the most essential classes and methods. Have a look at it.
It will also be deployed as version 1.4.0 tomorrow in Maven Central.
After that, I will take more care of joml-camera. :)

So it's been two weeks again since the last update that I would work on joml-camera more, but I didn't. :(
During the last two weeks there was just again so much going on with JOML, since that's where the priorities and use cases are right now. I hope I can make a free look camera happen sometime.

No worries man. :)
We're just happy to have someone donating such great work to build a foundation for the rest of us!
I have a couple of ideas I'm tinkering with that might help on that front.
Once I prove them out for myself I'll send them to you to see if you can make use of them.

It's been quite some time, but it's finally there! A FreeCamera. ;)
Have fun with it! :)

Also there is an accompanying demo.


[dev-tool] Drop NodeJS v12 and add v18

v12 has reached end-of-life in April 2022 and our support for it expires in October 2022. This PR will be merged after the release week, in the week of October 17th.

Looks good! Did you also want to tackle updating the syntax transforms in util/samples/syntax.ts?

Will took care of that in https://github.com/Azure/azure-sdk-for-js/pull/24148 so I believe this can be merged now.


Error when build with Firebase

Hi friend, i want to use the package but I have not been able due to some problems when making the build:

The first problem was with the minimum version of the sdk, which should be 24, this is easily solved by changing the minimum version in android/app/build.gradle

defaultConfig {
    // TODO: Specify your own unique Application ID (https://developer.android.com/studio/build/application-id.html).
    applicationId "com.example.chat_app"
    minSdkVersion 24
    targetSdkVersion flutter.targetSdkVersion
    versionCode flutterVersionCode.toInteger()
    versionName flutterVersionName
}

The second error has the following message: "More than one file was found with OS independent path 'lib/arm64-v8a/libavutil.so'. If you are using jniLibs and CMake IMPORTED targets, see  https://developer.android.com/studio/preview/features#automatic_packaging_of_prebuilt_dependencies_used_by_cmake". Searching the internet, in StackOverflow someone publish the following solution: add the following line in android/app/build.gradle

android {
    // ...
    packagingOptions {
        pickFirst '**/*.so'
    }
}

With the line above the error message no longer appeared.

The third error is the main reason for this issue, when I add the line in point 2, and I am using firebase services (firebase_core and others), the app builds successfully but the following error appears as soon as it tries to start the app: "MissingPluginException (MissingPluginException (No implementation found for method Firebase#initializeCore on channel plugins.flutter.io/firebase_core))"

void main() async {
  WidgetsFlutterBinding.ensureInitialized();

  await Firebase.initializeApp();

  runApp(const MyApp());
}

I don't know if the video_trimmer package is the cause of this error or the line added in point 2, I would be grateful if you could give me an idea of what is happening and how I could solve it since I need to use firebase services.
Thanks!

This is related to firebase_core package, please open an issue there if you are facing this.


[SPARK-39776][SQL][FOLLOW] Update UT of PlanStabilitySuite in ANSI mode

What changes were proposed in this pull request?
The current verbose string of LogicalPlan didn't contain join type, we should add this
(10) BroadcastHashJoin [codegen id : 4]
Left keys [1]: [ws_sold_date_sk#7]
Right keys [1]: [d_date_sk#9]
Join type: Inner
Join condition: None

Why are the changes needed?
Add missed join type in verbose plan
Does this PR introduce any user-facing change?
User can see join type in verbose plan string
How was this patch tested?
Existed UT

ping @HyukjinKwon @cloud-fan

Merged to master.


bug: Rust LazyVim Extra package not working properly out of box

Did you check docs and existing issues?

[X] I have read all the LazyVim docs
[X] I have searched the existing issues of LazyVim
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
v0.9.4
Operating system/version
Linux indell 6.6.3-200.fc39.x86_64
Describe the bug
Rust language package from the extra configurations doesn't work properly out of box, it doesn't mark errors or autocomplete methods.
Steps To Reproduce
Default install

mv ~/.config/nvim{,.bak}
mv ~/.local/share/nvim{,.bak}
mv ~/.local/state/nvim{,.bak}
mv ~/.cache/nvim{,.bak}
git clone https://github.com/LazyVim/starter ~/.config/nvim
rm -rf ~/.config/nvim/.git
nvim

Install Rust lang package
1 .nvim
2. Press x (to enter Lazy Extras menu)
3. Search for lang.rust and enable it using x
Create empty rust project

cargo init abc
nvim abc
Open main.rs
add an extra invalid line inside of the main function  like "hello".asdf();
Save

No error is displayed, when you type "hello". you should get methods available such as trim or chars, instead you see only Snippets
Expected Behavior
Error should be displayed on the invalid line  and autocomplete should suggest valid methods not only Snippets
Repro
-- DO NOT change the paths and don't remove the colorscheme
local root = vim.fn.fnamemodify("./.repro", ":p")

-- set stdpaths to use .repro
for _, name in ipairs({ "config", "data", "state", "cache" }) do
  vim.env[("XDG_%s_HOME"):format(name:upper())] = root .. "/" .. name
end

-- bootstrap lazy
local lazypath = root .. "/plugins/lazy.nvim"
if not vim.loop.fs_stat(lazypath) then
  vim.fn.system({ "git", "clone", "--filter=blob:none", "https://github.com/folke/lazy.nvim.git", lazypath, })
end
vim.opt.runtimepath:prepend(lazypath)

-- install plugins
local plugins = {
  "folke/tokyonight.nvim",
  "folke/LazyVim",

require("lazy").setup({
  spec = {
    { "LazyVim/LazyVim", import = "lazyvim.plugins" },
    { import = "lazyvim.plugins.extras.lang.rust" },
    { import = "plugins" },
  },
})

vim.cmd.colorscheme("tokyonight")
-- add anything else here

The issue seems to come in different forms, and I don't know why: #2345. Something's definitely broken, yes.

@bubuntux I cannot reproduce this. I get methods such as trim and chars when I type "hello".
How did you install Rust in your OS? There was another issue that had problems with Rust installed via the distro's package manager (see this comment).
Could you try installing Rust via rustup as per the official Rust docs and see if you still have a problem?

Try to do cargo clean. This is what helped me fix a big issue with Rust.

Pretty sure it has something to do with the Rust installation itself. OP also created a Discussion in AstroNvim and it seems that he uses some kind of container and uses the distro's package manager to install Rust. Based on the comment I linked in my previous post, I believe it has something to do with that. That's why I asked if he could install Rust via rustup and check again.

@bubuntux I saw in the other thread in AstroNvim repo, that I linked above, that you use docker containers to run Neovim. I tried it and it seems it has something to do with the Rust version that the Alpine package manager distributes. If you install Rust with the method that is suggested from the official Rust docs, then it works fine (at least from the testing that i did).
The following command is what I used
docker run -w /root -it --rm alpine:edge sh -uelic '
     apk add curl bash git lua nodejs npm lazygit bottom python3 go neovim ripgrep alpine-sdk --update
     curl --proto \'=https\' --tlsv1.2 -sSf https://sh.rustup.rs | sh
     source ~/.cargo/env
     git clone --depth 1 https://github.com/LazyVim/starter ~/.config/nvim
     sleep 5
     sed -i \'/{ "LazyVim\/LazyVim", import = "lazyvim.plugins" },/a { import = "lazyvim.plugins.extras.lang.rust" },\'  ~/.config/nvim/lua/config/lazy.lua
     cargo init abc
     nvim abc/src/main.rs
   '

With this I get the methods that you mentioned.
@folke This doesn't seem to be a LazyVim bug to me from the testing that I did, rather a problem with the Rust version that the Alpine package manager distributes. I don't know the internals being used from Neovim LSP about how it works with LSP servers installed via distro package managers and why they might be incompatibilities, but it is what it is.

Hello @dpetka2001 ,
I don't know if it is related, but I have almost similar problem with rust-analyzer. First, I thought it might be pure rust-analyzer, but it works there.
All problems started happening, when I updated to latest rust-analyzer version.
It doesn't show all error messages, but clippy warnings are showed.
When I disable/enable diagnostics, I spot three different behaviors:
1/ It enables normally => but Rust errors are not shown
2/ It stuck and need to kill a pane
3/ It shows this error:
E5108: Error executing lua: .../nvim/2819/usr/share/nvim/runtime/lua/vim/diagnostic.lua:1031: Invalid 'line': out of range
stack traceback:
	[C]: in function 'nvim_buf_set_extmark'
	.../nvim/2819/usr/share/nvim/runtime/lua/vim/diagnostic.lua:1031: in function 'show'
	.../nvim/2819/usr/share/nvim/runtime/lua/vim/diagnostic.lua:1223: in function 'show'
	.../nvim/2819/usr/share/nvim/runtime/lua/vim/diagnostic.lua:1179: in function 'show'
	.../nvim/2819/usr/share/nvim/runtime/lua/vim/diagnostic.lua:1173: in function 'show'
	.../nvim/2819/usr/share/nvim/runtime/lua/vim/diagnostic.lua:1595: in function 'enable'
	...ocal/share/nvim/lazy/LazyVim/lua/lazyvim/util/toggle.lua:48: in function 'diagnostics'
	...l/share/nvim/lazy/LazyVim/lua/lazyvim/config/keymaps.lua:119: in function <...l/share/nvim/lazy/LazyVim/lua/lazyvim/config/keymaps.lua:119>

I am on latest Lazyvim update with Rust extra lang enabled (no manual tweaking).

Yeah, seems like related.
As of temporary workaround, I disabled extra rust:
-- { import = "lazyvim.plugins.extras.lang.rust" },

And enabled new plugin separately:
return {
  {
    "simrat39/rust-tools.nvim",
    enabled = false,
  },
  {
    "mrcjkb/rustaceanvim",
    version = "^3", -- Recommended
    ft = { "rust" },
  },
}

It fixed the issue on my side for now. Hope you can replace rust-tools soon :pray:

@Verisana I don't have anything to do with the project. Just a simple user like you who tries to help. I'm sure the maintainer of the project will look into this when he has time. There's also an open PR that tries to do that. All that remains is for it to be merged.

closing this since we now use rustaceanvim


haskellPackages.pthread: mark as broken on darwin

Description of changes
Preprocessing library for pthread-0.2.1..
compiling dist/build/System/Posix/Thread_hsc_make.c failed (exit code 1)
rsp file was: "dist/build/System/Posix/hsc2hscall43249-0.rsp"
command was: /nix/store/sa6hywsm1mqfyd1xakyzv4ljjsb3hawh-clang-wrapper-11.1.0/bin/clang -c dist/build/System/Posix/Thread_hsc_make.c -o dist/build/System/Posix/Thread_hsc_make.o --target=arm64-apple-darwin --target=arm64-apple-darwin --target=arm64-apple-darwin -F/nix/store/bd2l2sf1wh63cbc4g0ma30ryq0026kxz-apple-framework-CoreFoundation-11.0.0/Library/Frameworks -D__GLASGOW_HASKELL__=904 -Ddarwin_BUILD_OS=1 -Daarch64_BUILD_ARCH=1 -Ddarwin_HOST_OS=1 -Daarch64_HOST_ARCH=1 -I/nix/store/xzgz4yl47qjadcjwqcll08dx930mr324-libiconv-50/include -I/nix/store/4rxw7fvi574i13xdg183gc3aam89fhin-libcxx-11.1.0-dev/include -I/nix/store/0y21fgdr09v4fc9p3xbwqfzyjbm51j99-libcxxabi-11.1.0-dev/include -I/nix/store/h6jcfmqap8rg503vhydxr2kjj5yrsd2z-compiler-rt-libc-11.1.0-dev/include -I/nix/store/cyz92bd5vaws21yj01vw0k695xkc58h7-libobjc-11.0.0/include -I/nix/store/xzgz4yl47qjadcjwqcll08dx930mr324-libiconv-50/include -I/nix/store/4rxw7fvi574i13xdg183gc3aam89fhin-libcxx-11.1.0-dev/include -I/nix/store/0y21fgdr09v4fc9p3xbwqfzyjbm51j99-libcxxabi-11.1.0-dev/include -I/nix/store/h6jcfmqap8rg503vhydxr2kjj5yrsd2z-compiler-rt-libc-11.1.0-dev/include -I/nix/store/cyz92bd5vaws21yj01vw0k695xkc58h7-libobjc-11.0.0/include -Idist/build/autogen -Idist/build/global-autogen -include dist/build/autogen/cabal_macros.h -I/nix/store/xzgz4yl47qjadcjwqcll08dx930mr324-libiconv-50/include -I/nix/store/4rxw7fvi574i13xdg183gc3aam89fhin-libcxx-11.1.0-dev/include -I/nix/store/0y21fgdr09v4fc9p3xbwqfzyjbm51j99-libcxxabi-11.1.0-dev/include -I/nix/store/h6jcfmqap8rg503vhydxr2kjj5yrsd2z-compiler-rt-libc-11.1.0-dev/include -I/nix/store/cyz92bd5vaws21yj01vw0k695xkc58h7-libobjc-11.0.0/include -I/nix/store/xzgz4yl47qjadcjwqcll08dx930mr324-libiconv-50/include -I/nix/store/4rxw7fvi574i13xdg183gc3aam89fhin-libcxx-11.1.0-dev/include -I/nix/store/0y21fgdr09v4fc9p3xbwqfzyjbm51j99-libcxxabi-11.1.0-dev/include -I/nix/store/h6jcfmqap8rg503vhydxr2kjj5yrsd2z-compiler-rt-libc-11.1.0-dev/include -I/nix/store/cyz92bd5vaws21yj01vw0k695xkc58h7-libobjc-11.0.0/include -I/nix/store/xzgz4yl47qjadcjwqcll08dx930mr324-libiconv-50/include -I/nix/store/805ppsk5vx9haxjrsg20q64ykxdwl3mq-ghc-9.4.7/lib/ghc-9.4.7/base-<IP_ADDRESS>/include -I/nix/store/agw6nf5hcmsyaihi80cyxj975iqwkrk8-gmp-with-cxx-6.3.0-dev/include -I/nix/store/805ppsk5vx9haxjrsg20q64ykxdwl3mq-ghc-9.4.7/lib/ghc-9.4.7/ghc-bignum-1.3/include -I/nix/store/805ppsk5vx9haxjrsg20q64ykxdwl3mq-ghc-9.4.7/lib/ghc-9.4.7/rts/include -I/nix/store/m2ci1q4vy264dp2j7fj0rd847k2gs4y4-libffi-3.4.4-dev/include -I/nix/store/805ppsk5vx9haxjrsg20q64ykxdwl3mq-ghc-9.4.7/lib/ghc-9.4.7/include/
error: Thread.hsc:59:5: error: operand of type 'double' cannot be cast to a pointer type
    hsc_type (pthread_t);
    ^~~~~~~~~~~~~~~~~~~~
/nix/store/805ppsk5vx9haxjrsg20q64ykxdwl3mq-ghc-9.4.7/lib/ghc-9.4.7/template-hsc.h:70:20: note: expanded from macro 'hsc_type'
    if ((t)(int)(t)1.4 == (t)1.4)                           \
                   ^~~
Thread.hsc:59:5: error: operand of type 'double' cannot be cast to a pointer type
    hsc_type (pthread_t);
    ^~~~~~~~~~~~~~~~~~~~
/nix/store/805ppsk5vx9haxjrsg20q64ykxdwl3mq-ghc-9.4.7/lib/ghc-9.4.7/template-hsc.h:70:30: note: expanded from macro 'hsc_type'
    if ((t)(int)(t)1.4 == (t)1.4)                           \
                             ^~~
2 errors generated.

Fails on double -> pthread_t in generated C code. Maybe there's some CFLAG that can make it work but not exactly clear if that'd be a safe choice without closer knowledge of the cast intent.
Let's mark package as broken on darwin for now.
ZHF: #265948
Things done

Built on platform(s)

[X] x86_64-linux
[X] aarch64-linux
[X] x86_64-darwin
[X] 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

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

[ ] Fits CONTRIBUTING.md.

I think you're right, there are some dedicated scripts at https://github.com/NixOS/nixpkgs/blob/master/pkgs/development/haskell-modules/HACKING.md#mark-broken-packages
And it will produce commits like 537bedd19e26d88609ea9a72bf12161c651e6407 - so also updating broken.yaml list with reference to specific hydra build
Will close this one as it may be outdated by now


Resource thousandeyes_label apply ends with Error: Failed call API endpoint. HTTP response code: 403. Error: 403 Forbidden

Snippet of configuration, many different approaches tried.
resource "thousandeyes_label" "vpn_underlay" {
name = "vpn_underlay"
type = "tests"
tests {
test_name = "cph_customer_overlay"
agents {
agent_id = data.thousandeyes_agent.underlay.agent_id
}
}
}

Apologies for the time it took us to address this. This should be fixed in the new version of the provider:
https://github.com/thousandeyes/terraform-provider-thousandeyes/releases/tag/v2.0.0
Please give it a try and let us know if you're still facing these issues


CON 158: Enabled finding VPC and subnet in elect leader cluster

@pauldallen Let me know if this looks okay now.


feature xz-static has correctness issues within compression library

$ cargo build --release --bin unsquashfs --features xz-static
   Compiling backhand v0.9.1 (/home/wcampbell/projects/wcampbell/backhand)
    Finished release [optimized] target(s) in 2.73s

$ ./target/release/unsquashfs -f -d out-rust test-assets/test_openwrt_netgear_ex6100v2/bytes.squashfs
thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: Custom { kind: InvalidInput, error: Options }', src/bin/unsquashfs.rs:132:48
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

squashfs-tools/unsquashfs is just fine
$ unsquashfs test-assets/test_openwrt_netgear_ex6100v2/bytes.squashfs
Parallel unsquashfs: Using 8 processors
1249 inodes (1049 blocks) to write

create_inode: could not create character device squashfs-root/dev/console, because you're not superuser!
[================================================================================================================================================================================================================================================================================================================================================| ] 2297/2298  99%

created 1043 files
created 122 directories
created 205 symlinks
created 0 devices
created 0 fifos
created 0 sockets
created 0 hardlinks

Also linked: https://github.com/alexcrichton/xz2-rs/issues/105


Add class to yield results form onnx model and computes differences between two runs

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

Comparison is base (ad22d16) 81.80% compared to head (d616c58) 81.78%.
Report is 1 commits behind head on main.

Files
Patch %
Lines

onnx_array_api/reference/evaluator_yield.py
87.23%
6 Missing :warning:

onnx_array_api/graph_api/graph_builder.py
66.66%
5 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #71      +/-   ##
==========================================
- Coverage   81.80%   81.78%   -0.02%
==========================================
  Files          56       57       +1
  Lines        7452     7512      +60
==========================================
+ Hits         6096     6144      +48
- Misses       1356     1368      +12

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


Adds support for receiving multiple protocols/sub-protocols

I wrote this a while ago but it's still necessary if you ask me, for example:
new WebSocket("ws://example.com/socket", ['echo','messages']
https://developer.mozilla.org/en-US/docs/Web/API/WebSockets_API/Writing_WebSocket_client_applications
as you can see in those documents specifying multiple protocols using an array should be supported.
My little commit just takes the first protocol in that array and uses that.

I don't think I agree with this. This patch updates the server component, but your example here references usage of the client API.
The client is allowed to specify multiple protocols that it's willing to speak, but it's the server's responsibility to choose exactly one to accept. The way in which that preferred protocol is selected is application-specific and should not be implemented in this library.
Your application should inspect the requestedProtocols array and select the one it agrees to when calling request.accept().

Agreed.


[Bug]Duplicate actions in on page load linked list hash set

Description
If a user turns on page load for an action, and then binds it into a widget, the page load actions set would contain this action twice.
Server needs to implement toposort (parallel DAG) to find out sets of actions which can be executed parallely and sets which have to be executed sequentially.

Closing as duplicate of #2562

Closing as duplicate of #2562

Both are two separate issues.

Both are two separate issues.

@ofpiyush : These are two symptoms of the same problem.

@ofpiyush : These are two symptoms of the same problem.


locked dependencies aren't subsequently resolvable

./gradlew --write-locks build
Persisted dependency lock state for configuration 'annotationProcessor'
Persisted dependency lock state for configuration 'spotbugs'
Persisted dependency lock state for configuration 'spotbugsPlugins'
Persisted dependency lock state for configuration 'checkstyle'
Persisted dependency lock state for configuration 'testAnnotationProcessor'

> Task :compileJava UP-TO-DATE
Persisted dependency lock state for configuration 'compileClasspath'
Persisted dependency lock state for configuration 'annotationProcessor'

> Task :processResources NO-SOURCE
> Task :classes UP-TO-DATE
> Task :jar UP-TO-DATE
> Task :assemble UP-TO-DATE

> Task :checkstyleMain
Persisted dependency lock state for configuration 'checkstyle'

> Task :spotbugsMain
Persisted dependency lock state for configuration 'spotbugs'
Persisted dependency lock state for configuration 'spotbugsPlugins'
Scanning archives (9 / 9)
2 analysis passes to perform
Pass 1: Analyzing classes (42 / 42) - 100% complete
Pass 2: Analyzing classes (1 / 1) - 100% complete
Done with analysis

> Task :compileTestJava UP-TO-DATE
Persisted dependency lock state for configuration 'testCompileClasspath'
Persisted dependency lock state for configuration 'testAnnotationProcessor'

> Task :processTestResources NO-SOURCE
> Task :testClasses UP-TO-DATE

> Task :test UP-TO-DATE
Persisted dependency lock state for configuration 'testRuntimeClasspath'

> Task :check
> Task :build

BUILD SUCCESSFUL in 6s
6 actionable tasks: 2 executed, 4 up-to-date
[xeno@freyja json-schema]$ ./gradlew build

FAILURE: Build failed with an exception.

* What went wrong:
Could not determine the dependencies of task ':spotbugsMain'.
> Could not resolve all dependencies for configuration ':spotbugs'.
   > Dependency lock state for configuration 'spotbugs' is out of date:
       - Did not resolve 'org.ow2.asm:asm-commons:6.1.1' which is part of the lock state
       - Did not resolve 'net.jcip:jcip-annotations:1.0' which is part of the lock state
       - Did not resolve 'org.ow2.asm:asm-analysis:6.1.1' which is part of the lock state
       - Did not resolve 'org.ow2.asm:asm-xml:6.1.1' which is part of the lock state
       - Did not resolve 'commons-lang:commons-lang:2.6' which is part of the lock state
       - Did not resolve 'com.google.code.findbugs:jFormatString:3.0.0' which is part of the lock state
       - Did not resolve 'org.apache.bcel:bcel:6.2' which is part of the lock state
       - Did not resolve 'org.ow2.asm:asm-util:6.1.1' which is part of the lock state
       - Did not resolve 'com.google.code.findbugs:jsr305:3.0.2' which is part of the lock state
       - Did not resolve 'org.dom4j:dom4j:2.1.0' which is part of the lock state
       - Did not resolve 'com.github.spotbugs:spotbugs-annotations:3.1.3' which is part of the lock state
       - Did not resolve 'org.ow2.asm:asm-tree:6.1.1' which is part of the lock state
       - Did not resolve 'com.github.spotbugs:spotbugs:3.1.3' which is part of the lock state
       - Did not resolve 'org.ow2.asm:asm:6.1.1' which is part of the lock state
       - Did not resolve 'jaxen:jaxen:1.1.6' which is part of the lock state

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 0s

source/sha here https://bitbucket.org/xenworks/json-schema/commits/0cf82198def60ed64e4a881015d2453770aee445

Thanks for the report, there is indeed a bug there. Fix is on its way and should be in Gradle 4.9


Routing system backward incompatibility

Maybe due to the #7011 , Router::register must return a string or a Response instance.
I think this is a backward incompatibility.
$router = $app->make('router');

// Doesn't work, but same as the example in the app.php
$router->register('/test1', function () {
    print 'This is a contrived example.';
});

// Works
$router->register('/test2', function () {
    return 'This is a contrived example.';
});

// Works
$router->register('/test3', function () {
    return (new \Concrete\Core\Http\Response('This is a contrived example.'));
});

Confirmed on 8.5.1 release and develop branch.

Thoughts on what this affects and how we might fix it? I haven't come across any code that is affected by this.

This example code in app.php affects.
 * ## Pass some route parameters
 *
 * Route::register('/test/{foo}/{bar}', function($foo, $bar) {
 *  print 'Here is foo: ' . $foo . ' and bar: ' . $bar;
 * });

Thoughts on what this affects and how we might fix it? I haven't come across any code that is affected by this.

I have many "tools" scripts (some 3rd party) includes that this breaks :(


fix: Update java-client, update tests, and remove support for predicate expression

Fix tests for the upcoming 6.1 release.  A couple of server responses were changed from "ERROR:: . . ." to "ok".  Also, extra latency was added to scans which required an additional latency for the test to succeed.  I also needed to remove support for predicate expressions because of the java-client update from 5.1 to 6.0.1.
This PR should not be merged until the next java-client release with a bugfix relevant to BatchGet tests.

See the tests are failing.


Jschiller/python

Current coverage is 65.24% (diff: 71.42%)

Merging #166 into master will increase coverage by 0.06%

@@             master       #166   diff @@
==========================================
  Files            63         63
  Lines          2929       2932     +3
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           1909       1913     +4
- Misses          893        894     +1
+ Partials        127        125     -2

Powered by Codecov. Last update c81d4cc...f50c3d4

Can you extract all the python files that will be in the wheel into a separate dir? A layout like this:
scoot/
client/
python/
bin/
pyscoot/
tests/
setup.py
...
I think it will be more clear and easier to build a wheel from.
What do you think?

Without knowing much about python best practices, I think this change is overkill for a largely non-python project. I'm not sure why bin/ or tests/ is important for creating a wheel (are wheels supposed to contain tests?) and the current structure, which puts all the required files in daemon/protocol/python, seems sufficient.
Can you elaborate on why this change would be impactful?

I don't think wheels contains tests dir.
The structure I am suggesting does not have functional impact but will have structural benefits.
It is very common for a multi-language project to isolate files of different languages if possible.
You put all required files in daemon/protocol/python, which I think is good. But there are 2 things I want to bring up:

Will you have clients for additional languages other than python in the future? If so, scoot/client/ or scoot/daemon/client/ may be a better place for those language specific code. daemon/protocol is more about daemon instead of clients IMHO.
I prefer to have python test files in the same dir with python source files (like I suggested above). It is less messy and developers can easily find what they want if they want to make changes to python code. It will be even better if you can separate go files and python files completely. I understand you have integration_py_test.go as a way to run python test under go test framework, but even if they are not in the same dir, you should still be able to invoke the integration test, right?

I personally feel more strongly about 2 than 1, but neither of them are blockers. Feel free to ignore them if you want.
Thanks for dealing with pypi stuff!


DK's not having tank sheet

I noticed that I couldn't theorycraft my DK tank on here, but Pally and warrior tanks are present, is there soon to be an implement for it?

https://wowsims.github.io/wotlk/tank_deathknight/ - its still very early in development which is why its not visible in the dropdown menu yet


Dynamically show/hide columns based on user selection with respect to react-table : React+Typescript

I am using react-table for as a data grid for an application . I have this scenario where I need to show/hide columns that are listed and based on user selection I need to show/hide them . Is there any way I can achieve this?
Basically need to have a some settings kind of icon , on click of the same I need to display all the available columns and based on the checkbox selection I need to show/hide them.
How can I get all the column values from react-table to show in this dropdown?
How can I add this settings icon as a part of column header(I will be displaying id's under this column), But the header for this column would be a settings icon next to "Edit" label

import * as React from 'react';
import ReactTable from 'react-table';
import 'react-table/react-table.css';

export default class App extends React.Component<{},{}>

    constructor(props:any){
        super(props);
        this.state={
           data : [];
    }

    componentDidMount() {
      //using some fetch Call, I get the data and set it to state data property
      let data = fetch(); // fetch call which is not described here
      this.setState({data})
    }

    render(){
       <ReactTable
           data={this.state.data}
            columns = {[
                    {
                      Header: "Edit",// here i need to add an settings icon next to "Edit" header and on click of the same need to show all the column names, then show/hide them based on selection
                      id: "row",
                      Cell: (row:any) => {
                        return <span>{row.index+1}</span>;
                      }
                    },
                    {
                      id: 'name,
                      Header: 'Name',
                      accessor: 'username',
                    },
                    {
                      id: 'dob,
                      Header: 'DOB',
                      accessor: 'userDOB',
                    },
                    {
                      id: 'status',
                      Header: 'Status',
                      accessor: 'userStatus',
                    }
              ]}
        />

    }
}

Please ask your question on https://spectrum.chat/react-table. Thanks!


bincode breaks Enums

The change from JSON to bincode introduced here breaks Enum transmissions. Upon receiving one gets the error
ERROR bevy_spicy_networking::client: Failed to decode network packet from [<IP_ADDRESS>:9999]: Bincode does not support Deserializer::deserialize_identifier

I set up a small example for verification here.
 Every click broadcasts an MyEnum::A wich cannot be parsed from the client.

You are correct, bincode has some weird restrictions as it is a self-describing format and thus can't work with things like that.
I do consider the use of bincode in that context sub-par, but couldn't find a suitable one. I am open for suggestions!

I don't know whether this is a limitation of bincode, as they support Enums, which I verified here.

Yes, as it is able to differentiate the two. As you can see in this issue, this problem is known and probably will not be fixed: https://github.com/bincode-org/bincode/issues/272#issuecomment-562872027
I am open for alternate defaults. I plan on adding compile-time picking of encoding scheme, but did not get around to do that yet.

I have a rough compile-time encoding scheme plan in my fork, I need to get it into a better place though before I do and then I'll submit a PR for it, been kicking the tires on it with this server mmo project and I've made way to many other changes that I should roll into a separate performance PR


Please update this software.

I dream of finding something similar, but I just can’t do it.  I would be happy to program something like that myself, but I do not know how.  this is the best solution on the Internet of all possible (which are not)

Pray u!

four years ago

@InquisitorSpyke I have this kind of software, if u need contact me.

Dude, give it to me, please, I've been trying to find something similar for weeks now.

@whyynoot Dude, give it to me, please, I've been trying to find something similar for weeks now.


List of all defined keywords in BuildIn library.

How can I extract all defined keywords in BuildIn library.

This issue tracker is for bug reports and enhancement requests. Use Slack, Forum, or other such support channels listed at robotframework.org for questions.


can't evaluate 3>4 with nightly build

TypeScript Version:
Version 2.1.0-dev.20160925
Code
var x = 3>4;

Expected behavior:
no errors
Actual behavior:
error TS2365: Operator '>' cannot be applied to types '3' and '4'.

Duplicate of https://github.com/Microsoft/TypeScript/issues/10120


I hope to be able to participate in the project。

Hi octref, I like vetur.
I want to be involved, it is more perfect.
How can I contact you?
What do I need to pay attention to?

Hey, thanks for your interest in contributing! 👍
Here is the contributing doc:
https://github.com/octref/vetur/blob/master/.github/CONTRIBUTING.md
Maybe you can first try building the project itself. Then participate in some issues, answer people's questions and find smaller ones to fix (ping me first before sending a PR).

Closing this issue since there is nothing I need to do, but feel free to ask more questions here and I'll respond.


Slow data generation is preventing travis checks

Such as : https://travis-ci.org/pyros-dev/pyros-schemas/jobs/341915685

Fixed in #5


Changed to 0.1

Changed from 100 to 0.1 milliseconds

Related Issue(s)

Has Unit Tests (y/n)

Documentation Included (y/n)

Change Description
A description of the changes contained in the PR.
Rationale
A rationale for this change. e.g. fixes bug, or most projects need XYZ feature.
Testing/Review Recommendations
Fill in testing procedures, specific items to focus on for review, or other info to help the team verify these changes are flight-quality.
Future Work
Note any additional work that will be done relating to this issue.

Thanks for fixing this! This looks like the appropriate solution to me. It might be worth checking if this mistake is repeated anywhere else.


[Feature]  add override file.

All users to add a caule-override.yaml file to override manual edits users (currently) make to caule-themes-pack-1.yaml
the override file would be designed to survive updates.
my work laptop, given to me by my corporate overlords, has HORRIBLE contrast ratio.  not your fault... but...
on black theme: primary-background-color, divider-color, and disabled-color, cannot be distinguished between each other if there is sunlight shining into the room with the laptop.
same issue on Samsung Note 10 in direct sunlight... but yeah...
Needless to say, some users like to customize.  an override file would help us do that.

BTW.
love your themes.
I have the theme color change based on the state of my Alarm System
Green : disabled
Yellow : armed home
Orange : armed away
Red : Triggered!!!!!!


Calculate distance from centre for each event

Important pre-requisite before I can integrate with alternative reward functions (see separate issue)

Fixed in v3 codeline


FontRenderer -> TextRenderer

It actually renders text not fonts.

renders text not fonts

Call it GlyphRenderer

We already have GlyphRenderer. That renders a single type of glyph.

Oops. Should have looked it up before dumping the differences between font, typeface and glyph...


Create Code Review page

This page should be similar to the Issues tab page, but only show code reviews. It should alsod default filters to reviews that are active and that you are participating in.

This page is essentially the index. It should be linked to the "Code review" tab.

This page actually shows the code review. The index page is a different issue.


Changing script also changes script of UI elements

Example:

There should probably be a class for Sanskrit elements (I think there's one already?), and script-changing should only act on those elements.
(Actually I would say all Sanskrit elements should have their "original" text in an attribute like data-original-contents or whatever, and script selection should operate on the original contents directly: right now I think it's not invertible?)

Indeed it's not round-trip clean: see "pha઼ont sija઼e" (and just the lower-casing) here:

Fixed in #184.


helm-template function fails with .kube/config

Running the helm-template function
kpt pkg get https://github.com/GoogleContainerTools/kpt-functions-catalog.git/examples/helm-template .
kpt fn run helm-template/local-configs --mount type=bind,src=$(pwd)/helloworld-chart,dst=/source

with the error
[ERROR] Helm template command results in error: W0817 23:32:40.608720      12 loader.go:223] Config not found: /usr/local/google/home/jingfangliu/.kube/config

Error: exit status 1

Unsetting the environment variable by unset KUBECONFIG solves this error, but we should fix the problem from the helm-template image side.

cc @prachirp @Shell32-Natsu

@Shell32-Natsu This issue of helm not finding KUBECONFIG is similar to #879 where kustomize could not find TMPDIR

@prachirp You mean we should ignore KUBECONFIG env?

TMPDIR is implicitly propagated by macOS but obviously KUBECONFIG isn't. I think what makes more sense might be check is path specified by KUBECONFIG is valid and unset it if it isn't in the function.

@Shell32-Natsu kpt has no way of knowing whether any env variable is used in the function so we need more general logic.
How about we expose a --kubeconfig=/x/y/z flag in kpt fn run which takes a path to the kube config as argument, and if no arguments are passed it default uses the local KUBECONFIG env variable. If --kubeconfig flag is not passed then we ignore KUBECONFIG env.

I don't think add a flag for single env is a good idea, otherwise we will have dozens of flags in the future. I am think about using the same way that docker is using. Add a -e, --env flag to ask user to explicitly export env variable instead of blindly export all envs.
@frankfarzan Any input?

That makes sense to me. Now is a good time to make this change since we haven't released kpt v1.0.0 yet. As the biggest drawback is this is backwards-incompatible; it involves changing from passing all local env variables to docker to passing none by default.

I don't think add a flag for single env is a good idea, otherwise we will have dozens of flags in the future. I am think about using the same way that docker is using. Add a -e, --env flag to ask user to explicitly export env variable instead of blindly export all envs.
Frank Farzan Any input?

For kubectl, there is already a --server

That makes sense to me. Now is a good time to make this change since we haven't released kpt v1.0.0 yet. As the biggest drawback is this is backwards-incompatible; it involves changing from passing all local env variables to docker to passing none by default.

I think it might make sense to have this work the same way kubectl works. Users will be familiar with that. Is this possible to always propagate the env var (KUBECONFIG) into kpt fn run (in fact into all kpt commands)?
The ordering for finding kubeconfig in kubectl is:

--kubeconfig flag
KUBECONFIG env var. NOTE: This can have multiple kubeconfig files separated by colon which are merged
Default location for Unix: ~/.kube/config, Windows (forgot)

Also, values in the kubeconfig can be overriden (e.g --server).
See: https://ahmet.im/blog/mastering-kubeconfig/

Sorry: accidentally closed this. I've re-opened.

Thanks @seans3 , the main problem in this issue is KUBECONFIG is exported to the function accidentally and the path/file doesn't exist in the container. If we always export, it's up to user to make sure that the path specified by KUBECONFIG does exist in the container, otherwise they will have issue (like this).
At the same time I still think changing to passing no env by default is better than passing all because currently user has no way to control which env should or should not be passed.
Another thought is not all functions are using kubectl, is it worthwhile to add a flag for this? I prefer the method used in istioctl_analyze function. Flags to istioctl are passed as data and converted to flags by TS wrapper.

I think automatically exporting all host env vars to the container will be problematic in general. Setting env variables should be done explicitly. @Shell32-Natsu This should be tracked in the discussion of revisiting how docker flags are exposed in kpt.

We still haven't had a final decision about the flag name, reopen this.

closed by https://github.com/kubernetes-sigs/kustomize/pull/2988


preg_match_all errors with router in Lumen v5.1.1

Hi,
I just installed a fresh version of Lumen v5.1.1 with the dingo/api dev-master.
I followed the installation instructions and added to app.php:
$app->register(Dingo\Api\Provider\LumenServiceProvider::class);
$api = $app['api.router'];

To my routes.php I added:
$api->version('v1', function ($api) {
    $api->get('number', 'App\Http\Controllers\v1\NumberController');
});

Whatever I fill in in the first or second parameter of the get (or if I do post), I keep getting the following error:
ErrorException in Std.php line 26:
preg_match_all(): Compilation failed: internal error: previously-checked referenced subpattern not found at offset 99

in Std.php line 26
at Application->Laravel\Lumen\{closure}('2', 'preg_match_all(): Compilation failed: internal error: previously-checked referenced subpattern not found at offset 99', '/var/www/service/current/vendor/nikic/fast-route/src/RouteParser/Std.php', '26', array('route' => '/api/number', 'matches' => null))
at preg_match_all('~\{ \s* ([a-zA-Z][a-zA-Z0-9_]*) \s* (?: : \s* ([^{}]*(?:\{(?-1)\}[^{}]*)*) )? \}~x', '/api/number', null, '258') in Std.php line 26
at Std->parse('/api/number') in RouteCollector.php line 30

I tried replacing the code in Std.php with the newest version of nikic/fast-route, but that doesn't help anything. I also tried using API_PREFIX or API_DOMAIN.
I run PHP v5.6.10-1, but before this I ran v5.5.29, and had the same issues.
Am I doing something wrong here or is this a bug between specific versions?
PS. When I run the code just through PHP, it works.
<?php

preg_match_all('~\{ \s* ([a-zA-Z][a-zA-Z0-9_]*) \s* (?: : \s* ([^{}]*(?:\{(?-1)\}[^{}]*)*) )? \}~x', '/api/number', $matches, '258');
var_dump($matches);

Um, you're missing the the method for that route. You specify the controller by not the @method that the route uses.
$api->get('number', 'App\Http\Controllers\v1\NumberController@index');

You'll also need to get the instance of the router in the routes.php file as it's no longer handed between the bootsrapping file and the routes file.

Sorry for not clearing that up earlier. Not having the @method in the examples is just because I almost tried everything. Also with the @method it doesn't work.
The router instance I currently get in the bootstrap/app.php, and that part seems to work. But I started with getting the instance in routes.php, and I switched it up to the app.php to see if that helped.
Also, when change Std.php to not use the preg_match_all, but just do as if the preg_match_all didn't have any results, it all works okay. So it really seems to be some word combination issue.
public function parse($route) {
    return [$route];
}

note: i will edit my original post to make sure everything is clear.

It seems to have been a problem with PHP 5.5.25 in combination with that regex. Swiftmailer had exactly the same problem: https://github.com/swiftmailer/swiftmailer/issues/259


Cannot get Badge Number on Android

I called FCB.getBadgeNumber() and got the following message:
E/ReactNativeJS: undefined is not a function (evaluating 'RNFIRMessaging.getBadgeNumber()')

I'm running RN 0.34.1, react-native-fcm 2.3.0, on a Motorola XT1069 with Android 6.0.
My app was running on the foreground.
I searched the source inside react-native-fcm's android folder, but couldn't find any mention of the word "badge".
Here's my FCM control component:
import React, { Component } from "react";
import FCM from "react-native-fcm";
import uuid from "react-native-uuid";
export default class PushController extends Component {
  componentDidMount() {
    FCM.requestPermissions();
    FCM.getFCMToken().then(token => {
      console.log("TOKEN (getFCMToken)", token);
    });
    this.notificationUnsubscribe = FCM.on("notification", notif => {
      console.log("Notification", notif);
      let { title, body } = notif;
      FCM.getBadgeNumber().then(number => {
        FCM.presentLocalNotification({
              id: uuid.v4(),
              title,
              body,
              sound: "default",
              priority: "normal",
              click_action: "ACTION",
              number: number + 1,
              icon: "appLogo",
              vibrate: 300,
              lights: true,
              show_in_foreground: true
          });
      });
    });
    this.refreshUnsubscribe = FCM.on("refreshToken", token => {
      console.log("TOKEN (refreshUnsubscribe)", token);
    });
  }
  componentWillUnmount() {
    this.refreshUnsubscribe();
    this.notificationUnsubscribe();
  }
  render() {
    return null;
  }
}

@tioback related to https://github.com/evollu/react-native-fcm/issues/145
check my answer here: https://github.com/evollu/react-native-fcm/issues/145#issuecomment-252633192
getBadgeNumber is iOS only

Thanks, @sibelius, I just sent PR #152 that mentions this in the README.
Later I'll see if I can do something about providing this feature for Android, too.


Test2

Hey there and thank you for opening this pull request! 👋🏼
We require pull request titles to follow the Conventional Commits specification and it looks like your proposed title needs to be adjusted.
Details:
No release type found in pull request title "Test2". Add a prefix to indicate what kind of release this pull request corresponds to. For reference, see https://www.conventionalcommits.org/

Available types:
 - fix
 - hotfix
 - feat
 - feature
 - chore


Add integer range patterns

In an effort to consolidate the typechecking primitives::Pat and exhaustiveness::lower::Pat  data structures, integer ranges need to be added to the language:
Questions to answer:

What is the syntax? My proposition would be:

a..b for open range, as in  a <= x < b

a..=b for open range, as in  a <= x <= b

a.. for unbounded range, as in  a <= x <= type_of(a)::MAX with the assumption that a is of an integral type

should we have some kind of stepping variant?

Should we have an equivalent thing for floats, i.e 3.0..=27.3 ?

Things that need to be done:

AST:

New pat variant

Support in parsing
Semantic Analysis (ensure range makes sense)

lower bound is smaller or equal to the upper bound

Support in tc traversal
Support in exhaustiveness checking (needs to be hooked up with existing logic)

@kontheocharis thoughts?

What about a..<b for exclusive end and a..b for inclusive end?

a..+1..b, a..-1..b for steps? Although this is only for iterators, which is a different discussion

for i in 1..+2..end { whatever(i)

end := get_endpoint();

match i {
    1..end => ..,
    _ => ..
}


l

For questions or issues related to using one of Affectiva's SDKs, please use StackOverflow with the
"affdex-sdk" tag.  The Affectiva team monitors StackOverflow for these questions and will respond
there.  This approach also helps other developers with the same questions or problems find
solutions more quickly.
https://stackoverflow.com/questions/tagged/affdex-sdk
For more general discussion about our SDKs, please use our SDK forum:
https://discuss.affectiva.com/c/emotion-sdk
If you have a question or problem directly related to the samples in this repo (not with the SDK
itself), please proceed to create an issue here.  If you create an issue here that is not
related to this repo, we will ask you to move it to one of the locations above, as this helps
ensure it gets to the right people and is addressed most effectively.  Thanks for your
cooperation.

Dunno what happened here, sorry. Delete it if possible :(


[Catalog] No WASM Filters displayed

Description
Visit https://meshery.io/catalog/webassembly or click the WebAssembly radio button on meshery.io and find that no WASM filters are displayed.
Expected Behavior

Contributor Guides and Handbook
The meshery.io website uses Jekyll and GitHub Pages. Site content is found under the master branch.

📚 See the Contributing to Meshery.io Website section of the readme.md and other contributing instructions, too.
🎨 Wireframes and designs for Meshery UI in Figma
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

HEY @leecalcote can I work on this

@nakulbh, we missed you on this one, but thank you for volunteering!

@vishalvivekm "wasm filter" is not a category.

@vishalvivekm "wasm filter" is not a category.

@leecalcote As previously wasm filter was under Technology.
Can I include "wasm filter" in Technology section as a checkbox?

@vishalvivekm "wasm filter" is not a category.

@leecalcote As previously wasm filter was under Technology. Can I include "wasm filter" in Technology section as a checkbox?

Yes, @vishalvivekm + it should be accessible at https://meshery.io/catalog/webassembly

@vishalvivekm, while you're on that,  kindly look at the repetition of "deployment" under categories too

@vishalvivekm @thisiskaransgit please fix today.


Create Copy-AzOperationalInsightsTable.ps1

Added a custom PowerShell function to copy tables in log analytics to a custom log
Required items, please complete
Change(s):

See guidance below

Reason for Change(s):

See guidance below

Version Updated:

Required only for Detections/Analytic Rule templates
See guidance below

Testing Completed:

See guidance below

Checked that the validations are passing and have addressed any issues that are present:

See guidance below

Guidance <- remove section before submitting

Before submitting this PR please ensure that you have read the following sections and filled out the changes, reason for change and testing complete sections:
Thank you for your contribution to the Microsoft Sentinel Github repo.

Details of the code changes in your submitted PR.  Providing descriptions for pull requests ensures there is context to changes being made and greatly enhances the code review process.  Providing associated Issues that this resolves also easily connects the reason.

Change(s):

Updated syntax for XYZ.yaml

Reason for Change(s):

New schema used for XYZ.yaml
Resolves ISSUE #1234

Version updated:

Yes
Detections/Analytic Rule templates are required to have the version updated

The code should have been tested in a Microsoft Sentinel environment that does not have any custom parsers, functions or tables, so that you validate no incorrect syntax and execution functions properly.  If your submission requires a custom parser or function, it must be submitted with the PR.

Testing Completed:

Yes/No/Need Help

Note: If updating a detection, you must update the version field.

Before the submission has been made, please look at running the KQL and Yaml Validation Checks locally.
https://github.com/Azure/Azure-Sentinel#run-kql-validation-locally

Checked that the validations are passing and have addressed any issues that are present:

Yes/No/Need Help

Note: Let us know if you have tried fixing the validation error and need help.

References:

Guidance for Detection checks
General contribution guidance
PR validation troubleshooting

Hello @TheAlistairRoss, Please enter above mentioned PR details

Hello @TheAlistairRoss, Please add PR description and a Readme file for better understanding

Hello @TheAlistairRoss, Please work on the above requested changes.

Hello @TheAlistairRoss, Please reply to the above mentioned comments

Hello @TheAlistairRoss, Can you please fill out PR description template


XMLHttpRequest does not have the right constructor.

When you do new XMLHttpRequest(), it creates an object which has the constructor referencing the native XMLHttpRequest Object instead of the XMLHttpRequest Object introduced by Pretender. So When you do new XMLHttpRequest() instanceof XMLHttpRequest() it returns false. This causing a problem for a third party library we are using, as it validates the object is an instance of XMLHttpRequest.

Can you point us to which library? I imagine their use case is differentiation between XMLHttpRequest on most browsers and ActiveXObject("Microsoft.XMLHTTP") on certain versions of Internet Explorer.
When pretender starts it replaces the value stored at window.XMLHttpRequest, so by defintion running new XMLHttpRequest() instanceof XMLHttpRequest() at that point must return true because the global variable XMLHttpRequest refers to itself.
In the same way that:
function Y(){}

aY = new Y();
ay instance of Y;

MUST be true but
function Y(){}

aY = new Y();

window.Y = Array;
ay instance of Y

Would be false.
I suspect what is happening is they the library is grabbing the value stored in window.XMLHttpRequest before Pretender can replace the global, and they later attempt to check if request objects created after window.XMLHttpRequest has been replaced were constructed with the old value of window.XMLHttpRequest (the who that is [native code] inspector).
For example:
console.log(new XMLHttpRequest() instanceof XMLHttpRequest)
// true, since the value of `var` XMLHttpRequest is, of course, `XMLHttpRequest`

var StashedXHR = XMLHttpRequest;
// store a reference to the native XHR object that is the current value of the `XMLHttpRequest` variable

new Pretender()
// start a new server, which replaces the value of `var XMLHttpRequest` with another object.

console.log(new XMLHttpRequest() instanceof XMLHttpRequest);
// true, since the value of `var` XMLHttpRequest is, of course, `XMLHttpRequest`

console.log(new XMLHttpRequest() instanceof StashedXHR)
// false. The value at `var XMLHttpRequest` is NOT the same as the value of StashedXHR

Here is a JSFiddle if you'd like to play around with how these global vars are handled: https://jsfiddle.net/1g0fb4hz/1/
If the library you're using compares constructors, you'd need to start a Pretender instance before that library tries to grab the current value stored in window.XMLHttpRequest if possible.

Closing for now. We can reopen if the @smk1992 has more info


GH-43633: [R] Add tests for packages that might be tricky to roundtrip data to Tables + Parquet files

Rationale for this change
Add coverage for objects that might have issues roundtripping to Arrow Tables or Parquet files
What changes are included in this PR?
A new test file + a crossbow job that ensures these other packages are installed so the tests run.
Are these changes tested?
The changes are tests
Are there any user-facing changes?
No

GitHub Issue: #43633

@github-actions crossbow submit test-r-extra-packages

@github-actions crossbow submit test-r-extra-packages

@github-actions crossbow submit test-r-extra-packages

@github-actions crossbow submit test-r-extra-packages

@github-actions crossbow submit test-r-extra-packages

@github-actions crossbow submit test-r-extra-packages

@github-actions crossbow submit test-r-extra-packages

@github-actions crossbow submit test-r-extra-packages

@github-actions crossbow submit test-r-extra-packages

@thisisnic @nealrichardson @amoeba I think y'all are probably the most likely to know other extensions of data.frame objects that folks have roundtripped to parquet (or arrow tables) and back. Any other objects / packages that I'm missing? I've set these tests up such that we don't need to add any of these to suggests, but they will run on crossbow runs. On the positive side this also would have caught the altrep bug we caught with revdep checks, so that's nice!

Hi @jonkeane, this is great. The only extension I'm personally familiar with is units. Would that make sense to test here? See:
> library(arrow)
> library(units)
> df <- data.frame(x=1:10)
> units(df$x) <- "s"
> df
        x
1   1 [s]
2   2 [s]
3   3 [s]
4   4 [s]
5   5 [s]
6   6 [s]
7   7 [s]
8   8 [s]
9   9 [s]
10 10 [s]
> class(df$x)
[1] "units"
> # write.csv/read.csv drops units
> tf <- tempfile()
> write.csv(df, tf)
> read.csv(tf)
    X  x
1   1  1
2   2  2
3   3  3
4   4  4
5   5  5
6   6  6
7   7  7
8   8  8
9   9  9
10 10 10
> # write_csv_arrow errors
> tf <- tempfile()
> arrow::write_csv_arrow(df, tf)
Error: Invalid: Unsupported Type: [s]
> # write/read_parquet works great
> write_parquet(df, tf)
> read_parquet(tf)
        x
1   1 [s]
2   2 [s]
3   3 [s]
4   4 [s]
5   5 [s]
6   6 [s]
7   7 [s]
8   8 [s]
9   9 [s]
10 10 [s]

@github-actions crossbow submit test-r-extra-packages

@github-actions crossbow submit test-r-extra-packages

@github-actions crossbow submit test-r-extra-packages

@github-actions crossbow submit test-r-extra-packages

@jonkeane I'm late to adding this, but notably my a team member sent me evidence that the index in data.table can cause problems in reading the parquet back in (Error: IOError: Couldn't deserialize thrift: TProtocolException: Exceeded size limit) and it explodes the size of the file (in her example from 400MB to 2GB with the only change being the index). See reprex below:
library(data.table)
library(arrow)

dt<-data.table(x=c(1:1e8), y = round(runif(n=1:1e8, min=1, max=5)))

#Looking at rows where y == 3
dt[y == 3,]

#Creating a new variable, which is done uniformly across all rows (suggesting the previous row index isn't applicable?)
dt[, z := 1]

#Save the dt
write_parquet(dt, "example.parquet")
gc()

#Cannot open the dt
dt_open<-read_parquet("example.parquet")

#Removing indexing that was created when looking at the y==3 subset before saving allows
#the file to be opened after re-saving.
setindex(dt, NULL)

write_parquet(dt, "example2.parquet")
dt_open<-read_parquet("example2.parquet")

Thanks for this reprex! Would you mind opening a new issue here (I'm also happy to copy / paste, but want to give you github points for reporting this).
I've poked at this a bit and I don't believe that this is new or a regression, but I still would love to see if there's anything we can do to make this smoother. The metadata in the example is both large and in the information-theoretic worst case scenario for compression, so we don't get much compression out of it at all. But we might be able to do something else.


chore: Remove the checking of the number of dlls

Remove the checking of the number of dlls as there are valid NuGet packages that contains no dlls(ex. PowerShell modules)

Sorry it took me this long to get to this PR!
Would you be able to extend this to make not checking for DLL's an opt-in flag?

Happy to do that

LGTM! I seriously need to adjust my notification settings for this repository!.


Change the home page look

Want to make the home page more creative like by adding background etc.

Hi 😄, @tjanhvi Thanks for creating issue at makemathseasy, do read and follow the Code of Conduct and the Contribution Guidelines while contributing.

okay :)

Please assign me this issue

hey i would like to work on this issue plzz assign lgmsoc participant

hey @nitishapiplani we have to follow the first come first serve rule so that's why i have to assign this issue to @tjanhvi .
if she is unable to complete this issue before the deadline then i will assign it to you. :)

go ahead :)
DeadLine 17 june

hey @nitishapiplani we have to follow the first come first serve rule so that's why i have to assign this issue to @tjanhvi .
if she is unable to complete this issue before the deadline then i will assign it to you. :)

Okkay

@nitishapiplani Till then explore the Project, or there are too many issues Open, even if it's labeled as GSSOC are open to work.
or you can create your own issue, and ask for assignment


when choose button not working

when i click choose inside file manager nothing happen
laravel  version 6

I have the same errors,when i click   standalone button, popup but when i click to choose an image,popup can't close,  nothing happen, not like demo
Who can help me, please !
Laravel ver 5.7, file manager ver 1.8

Hello for some reason it's not working with summer note so use creditor instead,, and that file choose not working  because javascript and css file are not loading correctly! so my solution was to put public in the assets url i m not using php artisan serve so i have to put public


Remove redundant pre-write/commit logic in LockResolverTest

What problem does this PR solve?
Fix #1061
What is changed and how it works?
Use prewrite() and commit() APIs of RegionStoreClient in test.
Check List
Tests

Unit test

Related changes

Need to cherry-pick to the release branch

/run-all-tests

/run-all-tests

/run-all-tests

/run-all-tests

/run-all-tests tidb=v3.0.2


Verify upper bits of literal numbers that aren't a multiple of 32 bits.

See
https://github.com/KhronosGroup/SPIRV-Tools/pull/659

@dneto0, I think I can handle this issue.
Shall the verification happen at parsing/operand level, before the call to ProcessInstruction( )?
Since multiple instructions use literals, one single check somewhere in Parser::parseOperand( ) will be easier than individual validations for all those instructions with literals, right?
Does this issue apply to SPV_OPERAND_TYPE_TYPED_LITERAL_NUMBER only?
It seems to me that the validation code should go either in the switch of parseOperand( ) or in setNumericTypeInfoForType().


Added User Registration Flow to UI

Wired up blazor.wa.tbd with a registration page

Started https://github.com/tbd-friends/water-drink-water/discussions/53


Add versor v0.0.1 to BCR

versor doesn't have a release on github. I can only use the tag URL from github.

Hello @bazelbuild/bcr-maintainers, modules without existing maintainers (versor) have been updated in this PR. Please review the changes.

Requesting maintainers to add the label to skip the unstable source URL validation as github.com/wolftype/versor doesn't have a release


Improvement/fwuv 48/reject authentication if client does not match a white list

Hi Lokesh,
please review my code. Thx!
Cheers
Michael

one general comment:
We are using the intension formatter for the java code and xml. Also we have the cleanup configuration for eclipse.
https://intension-de.atlassian.net/wiki/spaces/DEV/pages/455934471/Code-Cleanup+Code-Formatter+und+XML+in+Eclipse+konfigurieren
It would be great if you format the code and xml using this formatter


Certificate-based authorization for the VolumeService HTTPS API

The HTTPS API will be access controlled using client certificates.  Initially this will be limited to granting full access to a client or rejecting all access to that client based on whether the certificate is signed by the correct CA certificate or not.
Later we may want to offer finer-grained access control.  For example, we may want a class of user that has read-only access.  We could do this based on the client certificate we already plan to require.  We could also do it by adding an additional layer of credentials in the form of usernames and passwords.

We are moving our development planning to JIRA. This is issue is now being tracked at https://clusterhq.atlassian.net/browse/FLOC-902. You are welcome to file additional issues in GitHub if that's easier for you.


[#122200041] Add datadog-router job

#122200041
What
Creates a new job to declare the integrations to the datadog-agent for the gorouter via the datadog-router job
Dependencies
Once this is merge, tag the code and update the PR https://github.com/alphagov/paas-cf/pull/521
How to review
The most interesting change is be able to deploy the release to a micro environment:
For this case, we want to propose an alternate approach: use bosh-lite.
You can follow the instructions on the repo or this quick guide:

Check that you have vagrant and Virtualbox installed. I can be run on AWS, but we did not test it. Check the bosh-lite docs for that.
Install bosh_cli with gem install bosh_cli.
 Build this release and upload it:
    ```
bosh create release --with-tarball

bosh upload release dev_releases/datadog-for-cloudfoundry/datadog-for-cloudfoundry-<IP_ADDRESS>+dev.1.tgz # update the version accordingly

Create the manifest below:

cat > /tmp/manifest.yml <<EOF
name: datadog-test
director_uuid: DIRECTOR_UUID
releases:

name: datadog-for-cloudfoundry
version: latest

networks:

name: default
subnets:

range: <IP_ADDRESS>/28
reserved: [<IP_ADDRESS>]
static: [<IP_ADDRESS>,<IP_ADDRESS>,<IP_ADDRESS>]
cloud_properties:
name: random

resource_pools:

name: default
stemcell:
name: bosh-warden-boshlite-ubuntu-trusty-go_agent
version: "3262.2"
network: default
cloud_properties: {}

compilation:
workers: 2
network: default
cloud_properties: {}
update:
canaries: 1
canary_watch_time: 60000
update_watch_time: 60000
max_in_flight: 2
jobs:

name: app
templates:

name: datadog-router
release: datadog-for-cloudfoundry
instances: 1
resource_pool: default
networks:
name: default
static_ips:

<IP_ADDRESS>
properties:
api_key: mykey
hostname: hehe

EOF
  5. Change the bosh uuid: `gsed -i "s/DIRECTOR_UUID/$(bosh status --uuid)/" /tmp/manifest.yml`
  6. Target the deployment and deploy:
     ```
bosh deployment /tmp/manifest.yml
bosh deploy

It should succeed deploying. Now you can ssh to the "vm":
vagrant ssh # ssh into bosh-lite
bosh ssh app

Check that the "process check" is available:
stat /var/vcap/jobs/datadog-router/conf/datadog-integrations/process.yaml

Who can review
Not @keymon or @richardc

Merge commit tagged as 0.0.1.


docs: Update swagger doc

Close: #2690
PR Checklist
Please check if your PR fulfills the following requirements:

[ ] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

If your build fails due to your commit message not passing the build checks, please review the guidelines here: https://github.com/edgexfoundry/edgex-go/blob/master/.github/Contributing.md.
What is the current behavior?

Issue Number: #2690
What is the new behavior?
Using the enum keyword to specify possible values of the model property.
Does this PR introduce a breaking change?

[ ] Yes
[x] No

New Imports

[ ] Yes
[x] No

Specific Instructions
Are there any specific instructions or things that should be known prior to reviewing?
Other information

recheck

Codecov Report

Merging #3576 (74f050a) into master (0092eb6) will increase coverage by 12.47%.
The diff coverage is 61.56%.

@@             Coverage Diff             @@
##           master    #3576       +/-   ##
===========================================
+ Coverage   34.67%   47.14%   +12.47%
===========================================
  Files         139      111       -28
  Lines       11770     8141     -3629
===========================================
- Hits         4081     3838      -243
+ Misses       7488     3936     -3552
- Partials      201      367      +166

Impacted Files
Coverage Δ

internal/core/data/application/subscriber.go
0.00% <0.00%> (ø)

internal/core/data/io/event.go
80.95% <ø> (ø)

internal/core/data/messaging/messaging.go
83.92% <ø> (ø)

internal/core/metadata/controller/http/device.go
93.87% <ø> (ø)

...nal/core/metadata/controller/http/deviceprofile.go
88.82% <ø> (ø)

...al/core/metadata/controller/http/deviceresource.go
100.00% <ø> (ø)

...nal/core/metadata/controller/http/deviceservice.go
96.90% <ø> (ø)

.../core/metadata/controller/http/provisionwatcher.go
93.02% <ø> (ø)

internal/core/command/application/command.go
26.56% <26.56%> (ø)

internal/core/data/application/event.go
61.90% <61.90%> (ø)

... and 199 more

Continue to review full report at Codecov.

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


useCookie doesn't change value of an object stored in cookie

Environment

Operating System: Linux
Node Version:     v18.18.0
Nuxt Version:     3.9.0
CLI Version:      3.10.0
Nitro Version:    2.8.1
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      -
Runtime Modules:  -
Build Modules:    -

Reproduction
https://stackblitz.com/edit/nuxt-starter-4fsr9x?file=app.vue
Describe the bug
Same as https://github.com/nuxt/nuxt/issues/22001
When I store an object in a cookie using useCookie composable and then want to change the value of a key of that object it doesn't effect the cookie in the web browser. The value in the ref is good but value registered in the cookie browser it not updated.
Additional context
No response
Logs
No response

Hi, I tested the repro locally on my mac machine and it seems to be working fine.
Not sure if it's stackblitz behaviour that did not update the cookies correctly in browser, here's a stackblitz repro from another issue related to cookie that I tested.
Attached is a demo of me running the repro locally, can notice that the cookie did update when I click refresh after hitting the button.
Hope it helps!
https://github.com/nuxt/nuxt/assets/18641991/df40d72c-5117-4506-aaa7-120ad8340fa6


Export right private address for DigitalOcean.

If private networking is enabled while creating a DigitalOcean droplet, private ip address is exported in ipv4_address_private.
I also kept the old use-public-address-as-private-address method as a fallback to keep it backward compatible in case of private networking not chosen when creating the droplet.

Awesome, thank you!


Avoid passing empty headers during upgrade

This fixes the proxy for Safari which fails with a cryptic Invalid UTF-8 sequence in header value  error when the upgrade request contains a header with an empty Value e.g. Set-Cookie:
I also removed the http.CanonicalHeaderKey function call which is not needed for Header.Get

Thanks @databus23 :+1:


fill_in a long text results in timeout

Using Cpaybara + Rspec + Selenum .
Trying to fill in a textarea with a very long string:
fill_in 'agreeText', :with => contents

results in:
Failure/Error: fill_in 'agreeText', :with => contents
          Net::ReadTimeout:
            Net::ReadTimeout

any idea how to resolve?

How long is "very long"? What driver are you using? Also, if you can provide a minimal test case that would be great.

Closing due to lack of response


Add redis to dbms

Redis would be a nice addition to the list of dbmss.
Thank you.

Please check the website if the generated REDIS DSN string is as you would expect 😉

I'll check if the database number is working properly with the generated dsn.
Additionally I would add https://book.cakephp.org/5/en/core-libraries/caching.html#redisengine-options at least persistent and timeout as additional params.
I don't know if you wanted to cover the cache engines too though...

I think I will make a separate cache engine DSN form to make it easier to differentiate the 2 types of DSN strings 😁


Migrate from deprecated fields

Some GraphQL fields were recently deprecated in our API:

OrderLine.thumbnailUrl
Page.availableOn
Page.isVisible
ProductVariant.availability
Product.availability -> pricing
Product.availableOn
Product.price
Product.thumbnailUrl
Collection.publishedDate

We should migrate queries in Dashboard 2.0 to use the new fields, so that we'll be able to drop the old ones from the schema.

Field Collection.publishedDate is not used


add script data

Hi, thanks a lot for this wonderful and very useful code!
My main interest is to get the script for a character through UAX#24 (http://www.unicode.org/Public/UNIDATA/Scripts.txt), is it something that could be added? Thanks!

Hi, this is meant to be only a backport of the cpython's built-in unicodedata module, we don't add new features that are not in the upstream copy.
If you're interested in obtaining the script for a unicode character, try using fontTools.unicodedata::script method, i.e.:
https://github.com/fonttools/fonttools/blob/a2eef358be401cfa1c783f24f48a000424457152/Lib/fontTools/unicodedata/__init__.py#L45
The fontTools.unicodedata module works like a shim to the built-in unicodedata (or unicodedata2 if available) and adds a few more additional methods, like this script method:
>>> from fontTools import unicodedata
>>> unicodedata.script(chr(0x6AD))
'Arab'

thanks for your answer, I perfectly understand! I'll look at fontTools
Just for reference, there's a zombie issue for the addition of script to unicodedata in cpython on https://github.com/python/cpython/issues/50580


Split algorithms up into another package

Mentioned before,
but wha about having something like MLDatasets but for
algorithms for EVOLP?

As I mentioned before, this sounds like a good idea but perhaps not as a complete separate package but as an extension. This would require Julia 1.9, but we can definitely do that even without the upgrade (there's packages for that).
My idea would be to add more stuff to EvoLP v1, including the renaming of the types, a deprecation notice for the old types, a few other things, and then v2 would include three major improvements:

The algorithms as an extension. EvoLPBank sounds good!
The island models as an extension.
The new type hierarchy to remove the deprecation notice

We then worry about multi-objective and maybe other stuff 😅


Pico multicore 'sleep_ms' not working on core1 / potential race condition on core1

I've got massive problems getting core1 to work.
I assume there is a problem using 'sleep_ms' on core1.
Code:
#include <stdio.h>
#include <stdint.h>
#include "pico/stdlib.h"
#include "pico/multicore.h"
#include "hardware/gpio.h"

void core1_entry() {
    puts("core1 started");
    while (1) {
        puts("core1");
        sleep_ms(500);
    }
}

int main() {

    stdio_init_all();

    multicore_launch_core1(core1_entry);

    while (1) {
        puts("core0");
        sleep_ms(500);
    }
}

Output:

core0
core1 started
core1
core0
core0
core0
core0
core0
core0
core0
core0
core0
core0
core0
core0

However if i add a short delay before starting core1 everythings works as expected:

int main() {

    stdio_init_all();

    sleep_ms(10);
    multicore_launch_core1(core1_entry);

Output:
core0
core1
core0
core1
core0
core1
core0
core1
core0
core1
core0
core1

Is it possible that the race-condition might be in puts rather than sleep_ms? :shrug:

I can't tell for sure. I tried to "puts" before starting the second core but that didn't help.
It seems that you need to call 'sleep_ms' once before starting the second core. Even 'sleep_ms(1)' helps.

Is it possible that the race-condition might be in puts rather than sleep_ms? 🤷

I cannot reproduce this, can you please
attach the MakeLists.txt and the build output files (i.e. .UF2, .ELF and .DIS)
Also what compiler you are using.

I attached my complete project folder to reproduce:
It's compiled using Windows:

C:\Users\hdo>arm-none-eabi-gcc.exe --version
arm-none-eabi-gcc.exe (GNU Arm Embedded Toolchain 10-2020-q4-major) 10.2.1 20201103 (release)
Copyright (C) 2020 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

CMakelists.txt
cmake_minimum_required(VERSION 3.13)

include(pico_sdk_import.cmake)

set(projname "pico_core1")

project(${projname} C CXX ASM)

set(CMAKE_C_STANDARD 11)
set(CMAKE_CXX_STANDARD 17)

pico_sdk_init()

add_executable(${projname}
    main.c
)

pico_enable_stdio_uart(${projname} 1)
pico_add_extra_outputs(${projname})

target_link_libraries(${projname}
    pico_stdlib
    pico_multicore
)

https://www.dropbox.com/s/ceukqy6ver0a2rp/pico-core1.zip?dl=0

Thanks for the hint. I'm no openocd export but the flags mentioned are only related to loading to RAM aren't they. Up until now i'm using openocd only for flashing the pico withouht using USB UF2. Any tips for correct command doing this?

Yes these flags which is what we say in the documentation are incorrect. See
raspberrypi/pico-feedback#153

No they are wrong for everything.
you must do "monitor reset init" before loading any time of executable (if you want it to work correctly always)

No they are wrong for everything.
you must do "monitor reset init" before loading any time of executable (if you want it to work correctly always)

@kilograham The jimtcl script that implements program/preverify/verify/reset/exit calls "reset init" so the commands below should function successfully, or do you see this differently?
openocd -f interface/picoprobe.cfg -f target/rp2040.cfg -c "program pico_core1.elf verify reset exit" -s c:\dev\openocd\tcl

openocd -f interface/raspberrypi-swd.cfg -f target/rp2040.cfg -c "program pico_core1.elf verify reset exit"

See also OpenOCD User's Guide: Flash Programming.

yeah i wasn't aware that's what program verify reset exit did. seems like it ought to work then

Was this intentionally closed?

i thoguht i had reopened

I think this issue is rather related to openocd so maybe it can be closed?

yeah it only remains open as a reminder to @lurch that the documentation is wrong


Include all (or none) yearly reports in the website menu

I seem to recall that there were at least 2-3 yearly reports we'd put together over the years. Either we should have all of them on the website (and continue preparing them), or get rid of the subpage in the menu and degrade the one we have to a blog note.

I would sat @cjyabraham hide it for now and let's discuss off-line,after launch


compliance reporting nodes counts should NOT change when filtering by status

Describe the bug
When calling the compliance/reporting/nodes API, counts are returned. These counts are used for the filter buttons in the UI.
It appears that these counts are being modified when a status filter is included in the request.
no filters: "total":5,"total_passed":0,"total_failed":5,"total_skipped":0,"total_waived":0}

with filter:
curl -H "api-token: $ACCEPTANCE_TOKEN" -s https://a2-local-inplace-upgrade-acceptance.cd.chef.co/api/v0/compliance/reporting/nodes/search -d '{"filters": [{"type": "status", "values": ["passed"] } ]}'
{"nodes":[],"total":0,"total_passed":0,"total_failed":0,"total_skipped":0,"total_waived":0}

To Reproduce
Steps to reproduce the behavior:

call the compliance reporting nodes api. note counts.
call the same api with a status filter. note difference in counts.

Expected behavior
The counts returned with the API should be unaffected by a status filter.
In other words, if i call the API with no filters, I should get a full list of nodes and the counts.
If I call the API with a status filter, I should get a list of nodes that have the same status as the requested one, but the counts should remain unchanged.
This will allow us to use backend filtering for the status filters on that page.

related: https://github.com/chef/automate/issues/3178


Canvas 3D support

Describe the bug
With Canvas 3D, the tooltip is not displayed next to token but somewhere else on the screen.
There's a similar issue with Health Estimate, with suggestion that it is displayed by 2D camera coordinates.
Might be more of a feature request.
Maybe it is more applicable for the Canvas3D.
What steps did you take before experiencing the problem?
Steps to reproduce the behavior:

Open a Canvas 3D map
Place any token
Hover the token

What did you expect to happen?
Tooltip should display next to the token's projection on the screen from its real position on 3D scene
What happened instead?
Tooltip is displayed somewhere else but not next to the token
Screenshots

Foundry Versions

Version of this module: 3.4.2
Foundry Version: 11.305
System Name and Version: Windows 10
Browser Name and Version: Chrome

What other modules did you have active when experiencing this issue (include both names and versions)?

Canvas 3D 5.3.8

I will not be spending time adding support for a premium module I have no interest in using myself.
If someone wants to submit a PR for this, I would consider included it (as long as doing so would not jeopardize the functionality or maintainability for non-Canvas 3D users).


Need examples

I would be nice if there are some examples so readers can be sure of the formatting. Example: are quotes allowed"
ms-author: johndoe
vs.
ms-author: "johndoe"

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

ID: 1bdd1aa5-a8cb-8c77-1c36-5a9986875825
Version Independent ID: 3c9107fe-1b1d-da60-e697-27a54e481f7b
Content: Metadata for docs.microsoft.com - Contributor Guide
Content Source: Contribute/metadata.md
Product: non-product-specific
GitHub Login: @sarah-barrett
Microsoft Alias: sabarret

@vtieto - Thank you for this feedback! We've opened a PR to add this information and examples to the doc.


replace handleWithGlobalMiddlewares with `handlerWithGlobalMiddlewa…

replace handleWithGlobalMiddlewares with `handlerWithGlobalMiddlewa

- A picture of a cute animal (not mandatory but encouraged)
…res`
Signed-off-by: mgniu<EMAIL_ADDRESS>

ping

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "master" git@github.com:pwzgorilla/docker.git somewhere
$ cd somewhere
$ git rebase -i HEAD~2
editor opens
change each 'pick' to 'edit'
save the file and quit
$ git commit --amend -s --no-edit
$ git rebase --continue # and repeat the amend for each commit
$ git push -f

Ammending updates the existing PR. You DO NOT need to open a new one.

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "master" git@github.com:pwzgorilla/docker.git somewhere
$ cd somewhere
$ git rebase -i HEAD~2
editor opens
change each 'pick' to 'edit'
save the file and quit
$ git commit --amend -s --no-edit
$ git rebase --continue # and repeat the amend for each commit
$ git push -f

Ammending updates the existing PR. You DO NOT need to open a new one.

LGTM

LGTM


Explicit extension

Coverage increased (+0.2%) to 97.26% when pulling 8176dca9b68504461a85a02e63d8333077a06b9e on explicit-extension into c1895f2e9ee1d4e651f7a96957a33b86927534c1 on master.


Is one netty's channel map to a tcp connection and a socket port?

Another question is that can I parallel send data in multi thread over a single tcp connection? If the answer is yes,let's see the case. The first send request from client are blocked in server caused by some code bug. Can other thread's callback listener receive their response successfully?

You should consider Stackoverflow - Netty for questions.
Coming to your question, if you've 4 workers and 1 worker is blocked for some reason. Let's say, heavy processing or something, the other 4 workers will handle the traffic. However, you should not block Netty workers.

@hyperxpro Thanks for your remind. I will move the question to stackoverflow. As what you said, netty server can process multi client thread's request in parallel. How netty ensure their order?

Netty doesn't use threads. It's event-loop based.


System transparency?

I wonder if something like this - https://www.system-transparency.org/ - could be used to increase user trust in the executions

Big fan of these types of solutions. Whereas cloud compute frameworks only allow for 100% trust, these types of solutions allow for much more granularity and unbundling of trust. E.g. a range from "very little trust" (and low cost) to "high trust" (and higher price premium).
If we can build the trust mechanism to be modular/pluggable, then System-Transparency could be one useful approach among many.

This seems to be a system for securing the bootloader on a physical server. How would you see it relating to our userland software?

We could detect it (and/or suggest our Storage Providers build systems with it in).


Add optional env to pass through to xinetd conf of a remote problem

Let's defer this as the immediate problem need has been addressed by other means


misuse of logging lib

This was causing an error for me.
https://github.com/geometalab/geojson2vt/blob/master/geojson2vt/geojson2vt.py#L65
Looks like you swapped print with logging.debug at some point.
  File "REDACTED/python3.7/site-packages/geojson2vt/geojson2vt.py", line 244, in geojson2vt
    return GeoJsonVt(data, options, log_level)
  File "REDACTED/python3.7/site-packages/geojson2vt/geojson2vt.py", line 65, in __init__
    logging.debug('tiles generated:', self.total, self.stats)
Message: 'tiles generated:'
Arguments: (473, {'z0': 1, ...})

Was able to get around it by doing this.
logging.disable(level=logging.DEBUG)
try:
    vector_tile_index = geojson2vt(geojson, vector_tile_options, logging.INFO)
finally:
    logging.disable(level=logging.NOTSET)


QsString for Actix is not working correctly when using url encoded parameters

Just as a preamble, I love this library and I'm using it right now for nested parameter query. It's very nice to be used. Also, mind you that I have no idea whether this is a non-goal or an actual bug, but I'm going to make this issue either way just to be sure.
Issue
I've found a behavior where QsQuery<T> behaves oddly when the url is encoded using the methods described in https://en.wikipedia.org/wiki/Percent-encoding.
For example, this handler and their structs,
#[derive(Debug, PartialEq, Deserialize, Serialize)]
pub struct QueryTest {
    reg_int: Option<i64>,
    reg_string: Option<String>,
}

#[derive(Debug, Deserialize, Serialize, Clone)]
pub struct FlatParam {
    pub text: Option<String>,
    pub number: Option<i32>,
}

#[get("/test/nested-parameter")]
pub async fn nested_parameter(param: QsQuery<NestedParam>) -> impl Responder {
    actix_web::HttpResponse::Ok().json(param.into_inner())
}

If given we give request it using the query parameters,
/test/nested-parameter?text=yes&nested%5Btext%5D=very%20nice%20yeah%20%F0%9F%98%8B&nested%5Bnumber%5D=123

which is below when decoded,
/test/nested-parameter?text=yes&nested[text]=very nice yeah 😋&nested[number]=123

will give the response,
{
  "text": "yes",
  "nested": null
}

I've also tested the behavior of using QsParam using dbg!() and it still lead to the same behavior. It seems that the QsString only take a decoded string by design? I've tested QsString using a unit test and TestRequest and as I've stated, it only works with an already decoded string. I see this as a bug because as far as I've tested, Insomnia  and Axios encode their parameter using percent encoding.
Current Workaround
Below is my workaround that I've used in my project. Note that GeneralResult is just a wrapper for Result<T,E> in my application that uses a custom error that's render-able to the user.
#[derive(PartialEq, Eq, PartialOrd, Ord)]
pub struct NestedQuery<T>(pub T);

impl<T> NestedQuery<T> {
    /// Get query parameters from the path
    pub fn from_query<S>(query_str: S) -> GeneralResult<T>
    where
        T: de::DeserializeOwned,
        S: AsRef<str>,
    {
        serde_qs::from_str(query_str.as_ref()).map_err(GeneralError::QueryStringParse)
    }
}

impl<T> FromRequest for NestedQuery<T>
where
    T: de::DeserializeOwned,
{
    type Error = GeneralError;

    type Future = Ready<GeneralResult<Self>>;

    type Config = ();

    #[inline]
    fn from_request(req: &actix_web::HttpRequest, _: &mut actix_web::dev::Payload) -> Self::Future {
        let query = match urlencoding::decode(req.query_string()) {
            Ok(a) => a,
            Err(e) => return err(GeneralError::QueryStringParse(serde_qs::Error::FromUtf8(e))),
        };

        let parsed = match Self::from_query(query) {
            Ok(p) => p,
            Err(e) => return err(e),
        };

        ok(Self(parsed))
    }
}

Conclusion
If this is a bug that needed to be addressed, I'm more than happy to work on it. If not, well that's ok too. Thank you for the very useful library, I hope to hear from other contributors soon.

Hey @tengkuizdihar!
Currently, serde_qs defaults to using "strict" mode. In which we expect non-percent encoded square brackets to represent any nested keys. You can change this behaviour by setting the configuration flag: https://docs.rs/serde_qs/0.8.5/serde_qs/#strict-vs-non-strict-modes


undefined symbol: _ZN10tensorflow12OpDefBuilder4AttrESs

Firstly, after I finished the compile with adding the "-NDEBUG  " to CFLAGS  ,  i meet the problem " xxxxxxx/correlation.so: undefined symbol: _ZTIN10tensorflow8OpKernelE"   when i want to run the Inference.
Then  i add the " TF_LIB  = python -c "import tensorflow; print(tensorflow.sysconfig.get_lib())" at the begining of Makefile  and " -L $(TF_LIB)  -ltensorflow_framework"  to CGPUFLAGS .  This time it is reported that " xxxxxxx/correlation.so: undefined symbol: _ZN10tensorflow12OpDefBuilder4AttrESs "  when i want to run the inference.
Now i have no idea how to solve this .....
Really need your help   !!!
My Context:
CUDA: 10.0
Tensorflow: 1.13
g++:  5.4
ubuntu: 16.04

And when i use the command "ldd -r correlation.so ",  i found that there are so many undefined symbol

It's not linking against TF. I modified the Makefile as follows to get around it (asking TF what appropriate linking flags to add):
diff --git a/Makefile b/Makefile
index 073c011..202236b 100644
--- a/Makefile
+++ b/Makefile
@@ -1,6 +1,7 @@
 # Makefile

-TF_INC = `python -c "import tensorflow; print(tensorflow.sysconfig.get_include())"`
+TF_CFLAGS=( $(python -c 'import tensorflow as tf; print(" ".join(tf.sysconfig.get_compile_flags()))') )
+TF_LFLAGS=( $(python -c 'import tensorflow as tf; print(" ".join(tf.sysconfig.get_link_flags()))') )

 ifndef CUDA_HOME
     CUDA_HOME := /usr/local/cuda
@@ -9,9 +10,9 @@ endif
 CC        = gcc -O2 -pthread
 CXX       = g++
 GPUCC     = nvcc
-CFLAGS    = -std=c++11 -I$(TF_INC) -I"$(CUDA_HOME)/include" -DGOOGLE_CUDA=1
+CFLAGS    = -std=c++11 $(TF_CFLAGS) -I"$(CUDA_HOME)/include" -I"$(CUDA_HOME)/.." -DGOOGLE_CUDA=1
 GPUCFLAGS = -c
-LFLAGS    = -pthread -shared -fPIC
+LFLAGS    = -pthread -shared -fPIC $(TF_LFLAGS)
 GPULFLAGS = -x cu -Xcompiler -fPIC
 CGPUFLAGS = -L$(CUDA_HOME)/lib -L$(CUDA_HOME)/lib64 -lcudart

@@ -59,23 +60,23 @@ endif


You exceeded your current quota, please check your plan and billing details.

This plugin only for prepaided GPT accounts?
I just install the plugin and add fresh key from my openAI account
But all tie have error
"You exceeded your current quota, please check your plan and billing details."

i was check with both of keys with the same results.

Same here. Can use chat.openai.com without any notification about exceeding any quota, but cannot use this plugin at all.

i try both variants with the same results

chat.openai.com is free with the 3.5 model. If you want to use the API of OpenAI you need to buy credits.

chat.openai.com is free with the 3.5 model. If you want to use the API of OpenAI you need to buy credits.

You are don’t reading previous posts. I have two accounts, one of them is free and next one is prepared. Both of them return the same error.

I have read all posts and you didn't mention that you tried it with an account where you have purchased credits.

type <C-n> to edit a new session and clear the message if you use an account with credits

same here


[VS2012][PTVS 2.1][W8] Neither signature help nor completion / IntelliSense

I have just installed PTVS 2.1-VS2012-Windows8. There is neither signature help nor completion. I have replayed the sequence of task described on the video "PTVS 2.0: A quick overview (2-4 mins per feature)", url : www.youtube.com/watch?v=JNNAOypc6Ek.

If you change _init_ to __init__, do you get the suggestions you expect? (You need two underscore characters at either end of the name.)

:) Thx you Zooba for your help.
Sorry for my dummy question.
I am learning python.
I will make more research before posting another question next time.