'react/bridging/LongLivedObject.h' file not found And Could not build module 'ReactCommon'

Description
/Pods/Headers/Public/ReactCommon/ReactCommon/LongLivedObject.h:11:10 'react/bridging/LongLivedObject.h' file not found
/Pods/Headers/Public/React-Core/React/RCTAppSetupUtils.h:30:9 Could not build module 'ReactCommon'

Version
0.70.6
Output of npx react-native info
System:
OS: macOS 13.0.1
CPU: (8) x64 Apple M1 Pro
Memory: 11.91 MB / 16.00 GB
Shell: 5.8.1 - /bin/zsh
Binaries:
Node: 19.2.0 - /usr/local/bin/node
Yarn: 1.22.19 - /usr/local/bin/yarn
npm: 8.19.3 - /usr/local/bin/npm
Watchman: 2<IP_ADDRESS> - /usr/local/bin/watchman
Managers:
CocoaPods: 1.11.3 - /usr/local/bin/pod
SDKs:
iOS SDK:
Platforms: DriverKit 22.1, iOS 16.1, macOS 13.0, tvOS 16.1, watchOS 9.1
Android SDK: Not Found
IDEs:
Android Studio: 2021.2 AI-212.5712.43.2112.8609683
Xcode: 14.1/14B47b - /usr/bin/xcodebuild
Languages:
Java: Not Found
npmPackages:
@react-native-community/cli: Not Found
react: 18.1.0 => 18.1.0
react-native: ^0.70.6 => 0.70.6
react-native-macos: Not Found
npmGlobalPackages:
react-native: Not Found
Steps to reproduce
Integrated RN version 0.70.6 to existing ios apps
Installed pod vial this command RCT_NEW_ARCH_ENABLED=1 fabric_enabled=1 arch -x86_64 pod install
Snack, code example, screenshot, or link to a repository

We're seeing this on 0.68.2
Not opted in to new architecture

Happens to me with new architecture on 0.70.6

Happens to me on 0.69.7, not opted in to new architecture nor Hermes

Happens to me with new architecture on 0.71.1

Also seeing it.
❯ yarn react-native info
info Fetching system and libraries information...
System:
    OS: macOS 13.2
    CPU: (10) arm64 Apple M1 Pro
    Memory: 75.94 MB / 16.00 GB
    Shell: 5.8.1 - /bin/zsh
  Binaries:
    Node: 16.16.0 - /private/var/folders/k9/xm7wlry90z3fjwl909bs1bb80000gp/T/xfs-cc83b2a2/node
    Yarn: 3.4.1 - /private/var/folders/k9/xm7wlry90z3fjwl909bs1bb80000gp/T/xfs-cc83b2a2/yarn
    npm: 9.4.2 - ~/.nvm/versions/node/v16.16.0/bin/npm
    Watchman: 4.9.0 - /opt/homebrew/bin/watchman
  Managers:
    CocoaPods: 1.11.3 - /Users/steve.loveless/.gem/ruby/2.7.6/bin/pod
  SDKs:
    iOS SDK:
      Platforms: DriverKit 22.2, iOS 16.2, macOS 13.1, tvOS 16.1, watchOS 9.1
    Android SDK: Not Found
  IDEs:
    Android Studio: 2022.1 AI-221.6008.13.2211.9514443
    Xcode: 14.2/14C18 - /usr/bin/xcodebuild
  Languages:
    Java: 11.0.18 - /usr/bin/javac
  npmPackages:
    @react-native-community/cli: Not Found
    react: ^18.2.0 => 18.2.0
    react-native: ^0.71.3 => 0.71.3
    react-native-macos: Not Found
  npmGlobalPackages:
    *react-native*: Not Found

@cortinico I'm pretty sure the nature of this issue is the same as what this PR fixes. It sounds like the same error, just for a different header file. I tried naively adding the #ifdef __cplusplus declaration analogous to the PR to the LongLivedObject.h, but it didn't resolve it to me. @janicduplessis Do you maybe have some insight on this?

Hi there, I'm sorry for this problem.
Can anyone provide a repro I can try on my own?
It's not the first time we see such issues. Our CI has tests for both the Old and the New Architecture, to make sure that it builds, so it should be some edge case that we are not handling properly.
For context:

We currently have 2 LongLivedObject.h: one in ReactCommon/react/bridging and one in ReactCommon/react/nativemodule/core/ReactCommon.
The second one is required to make sure that use_frameworks! actually work as expected.
We are excluding the duplicated files here because Xcode can't really build two files with the same name when they are in the same folder.

Now, my gut feeling is that there is something in your configuration that changes the HEADER_SEARCH_PATH of React Native and makes it lose the reference to the ReactCommon base path.
But to verify this, I'll need to reproduce it on a small sample.

Hey @cipolleschi I am also facing this issue after following the guide to create a turbo module.
Here is a reproducer: https://github.com/carlosalmonte04/CxxTurboModulesGuide.
It built successfully after commenting out the code you pointed to.

Hello  @cipolleschi , is this issues has been fixed, we are facing the same issue. ?

Hi @kaushal9808, unfortunately I didn't have much time to go deeper in the repro. :(
This week will be tricky, but I'll try to prioritize it next week.
Thank you all for your patience! 🙇

Facing the same issue, using img.ly's PE.SDK with localization : https://img.ly/docs/pesdk/react-native/guides/user-interface/localization/
including #import <RNPhotoEditorSDK/RNPhotoEditorSDK.h> in AppDelegate.mm
adding CLANG_ALLOW_NON_MODULAR_INCLUDES_IN_FRAMEWORK_MODULES option and select Yes. Then, search for c++ flags and add -fcxx-modules to both Debug and Release.
without localization and these additions, it build well.

Facing the same issue after upgrading to 0.69.x

Thank you @cipolleschi , I have now this error :
[!] There are multiple dependencies with different sources for RNPhotoEditorSDK in Podfile:

RNPhotoEditorSDK (from ../node_modules/react-native-photoeditorsdk)
RNPhotoEditorSDK

EXJSIUtils

Are you using Expo? Or just some libraries from Expo but the main flow is handled by the RN CLI?

Yes expo in bare mode, so RN mode.

@lc3t35 Ok, so, would you mind open an issue on their repo? Expo does some other things on top of the vanilla RN setup, so it could be that they are applying some changes to the Cocoapods setup that I would not be able to repro.
If they says that's a problem on the RN side, we would need a repro for React Native vanilla in order to work on that.

As mentioned by @cipolleschi, let's move this issue over to expo:
https://github.com/expo/expo/issues
I'll be closing this but feel free to reopen if the other repo points back to us as a issue in the core of React Native.

I believe the problem is that the header moved. There's a proxy header created here, but cocoapods seems to not make it available, because there's no implementation file.
If I manually touch up the files that contain the import #import <ReactCommon/LongLivedObject.h> and replace that with #import <react/bridging/LongLivedObject.h> the build succeeds.

Here is the patch-package for react-native 0.71.3 :
create patches/react-native+0.71.3.patch and add "postinstall": "patch-package"in your package.json.
diff --git a/node_modules/react-native/ReactCommon/react/nativemodule/core/ReactCommon/TurboModuleBinding.cpp b/node_modules/react-native/ReactCommon/react/nativemodule/core/ReactCommon/TurboModuleBinding.cpp
index 03c65ef..b41b29d 100644
--- a/node_modules/react-native/ReactCommon/react/nativemodule/core/ReactCommon/TurboModuleBinding.cpp
+++ b/node_modules/react-native/ReactCommon/react/nativemodule/core/ReactCommon/TurboModuleBinding.cpp
@@ -10,7 +10,7 @@
 #include <stdexcept>
 #include <string>

-#include <ReactCommon/LongLivedObject.h>
+#include <react/bridging/LongLivedObject.h>
 #include <cxxreact/SystraceSection.h>

 using namespace facebook;
diff --git a/node_modules/react-native/ReactCommon/react/nativemodule/core/ReactCommon/TurboModuleBinding.h b/node_modules/react-native/ReactCommon/react/nativemodule/core/ReactCommon/TurboModuleBinding.h
index 1791d4e..ed4184a 100644
--- a/node_modules/react-native/ReactCommon/react/nativemodule/core/ReactCommon/TurboModuleBinding.h
+++ b/node_modules/react-native/ReactCommon/react/nativemodule/core/ReactCommon/TurboModuleBinding.h
@@ -9,7 +9,7 @@

 #include <string>

-#include <ReactCommon/LongLivedObject.h>
+#include <react/bridging/LongLivedObject.h>
 #include <ReactCommon/TurboModule.h>
 #include <jsi/jsi.h>

diff --git a/node_modules/react-native/ReactCommon/react/nativemodule/core/platform/ios/RCTTurboModule.mm b/node_modules/react-native/ReactCommon/react/nativemodule/core/platform/ios/RCTTurboModule.mm
index 1ee5908..8aab28f 100644
--- a/node_modules/react-native/ReactCommon/react/nativemodule/core/platform/ios/RCTTurboModule.mm
+++ b/node_modules/react-native/ReactCommon/react/nativemodule/core/platform/ios/RCTTurboModule.mm
@@ -22,7 +22,7 @@
 #import <React/RCTModuleMethod.h>
 #import <React/RCTUtils.h>
 #import <ReactCommon/CallInvoker.h>
-#import <ReactCommon/LongLivedObject.h>
+#import <react/bridging/LongLivedObject.h>
 #import <ReactCommon/TurboModule.h>
 #import <ReactCommon/TurboModulePerfLogger.h>
 #import <ReactCommon/TurboModuleUtils.h>

@cortinico as you can see all modules are react-native 0.71.3 related not expo (version 48.0.19 I currently use)
Can you please check if these imports are corrected in upper version of react-native ?

Can you please create a repro on 0.71 or 0.72 which doesn't use Expo and reproduces this issue? Once you have it please open a new issue and link this one, and we can look into it


allow users to specify # of rows to be duplicated

Instead of duplicate one row at a time, we allow users to specify # of rows to be duplicated.
We used to have that but took this feature out.

There should be a hard maximum limit for the # of rows to be added.. Maybe 200? @jrchudy can you try and see how responsive is if you add 100 rows at once..


Add COPY TO PARTITION BY details

This PR add the new COPY TO PARTITION BY syntax to the docs, according to the original description in the PR (https://github.com/duckdb/duckdb/pull/5964#issue-1552796444).

Looks great, thank you for your help!


Cannot download nuget for PCL Targetting 4.5, Win8 and WP8.1

I am trying to use your really handly nuget, but even with the support claimed on the front page of this repo, it doesnt seem to have a nuget published for anything but normal .net. Can't seem to get this into my portable lib.
Thoughts?

I have tested it for these versions:
.Net Framework 3.5, .Net Framework 4.0, .Net Framework 4.5, Windows 8, Windows 8.1, Windows Phone 8, Windows Phone 8.1, Mono, Unity

I understand that, but have you tested them using a PCL that targets multiple platforms using your nuget? There is not binary published.

https://github.com/Quobject/SocketIoClientDotNet/blob/master/grunt/Gruntfile.js
You seem to be building for all plats, but only copying the release for mono and .net4.5 if i am reading your script properly.

The nuget upload and the zip file  contains dll's for all of the following:
{ "Name": "SocketIoClientDotNet.net35", "NuGetDir": "net35" },
{ "Name": "SocketIoClientDotNet.net40", "NuGetDir": "net40" },
{ "Name": "SocketIoClientDotNet.net45", "NuGetDir": "net45" },
{ "Name": "SocketIoClientDotNet.windowsphone8", "NuGetDir": "windowsphone8" },
{ "Name": "SocketIoClientDotNet.netcore45", "NuGetDir": "netcore45" },
{ "Name": "SocketIoClientDotNet.portable-wpa81+wp81", "NuGetDir": "portable-wpa81+wp81" },
{ "Name": "SocketIoClientDotNet.portable-win81+wpa81", "NuGetDir": "portable-win81+wpa81" }

I dug a little further into my issue. You are correct that these are published (of course you were), but apparently if I have a PCL targetting a combination of the above it doesn't like to work.
I went a looked at the nuget package for another dependency I am using and it formats itself similar but also has:
portable-net45+wp80+win8+wpa81+dnxcore50
portable-net40+sl5+wp80+win8+wpa81
I assume there is no way I could get a build drop for that right? I need to be able to target 4.5, phone and win8 and would prefer to not have to build multiple libs

Sorry, I don't have the time to do any other builds. Please use the code and build yourself.


Stack protector

The stack is descending.
Detect when stack collides with adjacent memory area.

Does it make sense to do this at the (virtual) hardware level or at the software level? In the emulator context, it's slightly more efficient to do it at the hardware level, presumably. In the circuit context, it's not so clear which would be more efficient, since to do this at the hardware level would require adding circuit complexity.

Sized at two weeks.


Add Disclaimer for Developers using Yarn in "Getting Started" Section

Description/ Problem
For developers got used to using yarn in their app projects, running yarn run android throws errors related to JAVA_HOME not set and emulator not found. But it all works fine if simply run npx react-native run-android instead
How can we address it?
Better add a disclaimer for those using Yarn to use npx react-native run-android instead of yarn run android as the latter will not work
Why is it important?
For many developers like me using 'Yarn', the above mistake can be easily made and overlooked with respect to the errors get thrown. And these errors can consume many hours if not days of precious time!
Who needs this?
All developers using Yarn
When should this happen (use version numbers if needed)?
In all the versions using npx to install the latest react native

running yarn run android throws errors related to JAVA_HOME not set and emulator not found.

Which kind of errors? Is this issue still relevant?
I personally have been using yarn run android for months without any issue at this stage. Can you please verify if the issue is still there?

Closing as we haven't got a follow up on the questions.
If the issue is still happening, feel free to re-open


Create k8s cluster failed using sealer0.9.2 and image docker.io/sealerio/kubernetes:v1-22-15-sealerio-2

What happen?
Here is command:
 sudo sealer run docker.io/sealerio/kubernetes:v1-22-15-sealerio-2  --masters <IP_ADDRESS> -n  <IP_ADDRESS> -p ubuntu -d
Relevant log output?
This node has joined the cluster:
* Certificate signing request was sent to apiserver and a response was received.
* The Kubelet was informed of the new secure connection details.

Run 'kubectl get nodes' on the control-plane to see this node join the cluster.

2023-04-11 16:01:36 [INFO] [join_nodes.go:85] succeeded in joining <IP_ADDRESS> as worker

2023-04-11 16:01:36 [INFO] [runtime.go:131] succeeded in creating a new cluster.

2023-04-11 16:01:36 [DEBUG] [hook.go:110] no hooks found at phase: post-init-host

2023-04-11 16:01:36 [DEBUG] [hook.go:149] no hooks found at phase: post-install

The connection to the server localhost:8080 was refused - did you specify the right host or port?
The connection to the server localhost:8080 was refused - did you specify the right host or port?
2023-04-11 16:01:36 [DEBUG] [remove_container.go:66] 48d4dc6c76065ad7b508fc723679ad99b44bd2ec7f49617e1f5d2a9fdd2b0fc0

Usage:
  sealer run [flags]

Examples:

run cluster by Clusterfile:
  sealer run -f Clusterfile
run cluster by CLI flags:
  sealer run docker.io/sealerio/kubernetes:v1.22.15 -m <IP_ADDRESS> -n <IP_ADDRESS> -p Sealer123
run app image:
  sealer run localhost/nginx:v1

Flags:
  -f, --Clusterfile string   Clusterfile path to run a Kubernetes cluster
      --apps strings         override default AppNames of clusterimage
      --cmds strings         override default LaunchCmds of clusterimage
  -e, --env strings          set custom environment variables
  -h, --help                 help for run
  -m, --masters string       set count or IPList to masters
      --mode string          load images to the specified registry in advance (default "apply")
  -n, --nodes string         set count or IPList to nodes
  -p, --passwd string        set cloud provider or baremetal server password
      --pk string            set baremetal server private key (default "/home/ubuntu/.ssh/id_rsa")
      --pk-passwd string     set baremetal server private key password
      --port uint16          set the sshd service port number for the server (default port: 22) (default 22)
  -u, --user string          set baremetal server username (default "root")

Global Flags:
      --color string               set the log color mode, the possible values can be [never always] (default "always")
      --config string              config file of sealer tool (default is $HOME/.sealer.json)
  -d, --debug                      turn on debug mode
      --hide-path                  hide the log path
      --hide-time                  hide the log time
      --log-to-file                write log message to disk (default true)
  -q, --quiet                      silence the usage when fail
      --remote-logger-url string   remote logger url, if not empty, will send log to this url
      --task-name string           task name which will embedded in the remote logger header, only valid when --remote-logger-url is set

2023-04-11 16:01:36 [ERROR] [root.go:75] sealer-v0.9.2: failed to execute command(cd /var/lib/sealer/data/my-cluster/rootfs && bash application/apps/calico/calico.sh) on host(<IP_ADDRESS>): error(exit status 1)

What you expected to happen?
create k8s cluster succeed.
How to reproduce it (as minimally and precisely as possible)?
sealer version: v0.9.2

` sudo sealer run docker.io/sealerio/kubernetes:v1-22-15-sealerio-2  --masters <IP_ADDRESS> -n  <IP_ADDRESS> -p ubuntu -d`

Anything else we need to know?
No response
What is the version of Sealer you using?
{"gitVersion":"v0.9.2","gitCommit":"7bab233","buildDate":"2023-03-08 03:24:20","goVersion":"go1.17.13","compiler":"gc","platform":"linux/amd64"}
What is your OS environment?
Ubuntu 18.04.6 LTS
What is the Kernel version?
Linux version 4.15.0-208-generic
Other environment you want to tell us?

Cloud provider or hardware configuration:
Install tools:
Others:

Find the reason：Should su root before run sealer run xxxx  rather than sudo run xxxx.  So this issue is closed now.


add nim books

Hacktoberfest notes:

due to volume of submissions, we may not be able to review PRs that do not pass tests and do not have informative titles.
please read our contributing guidelines
be sure to check the output of Travis-CI for linter errors
if this is your first open source contribution, make sure it's not your last!

What does this PR do?
Add Resource(s)
For resources
More book on Nim language
Description
This PR add more resource of Nim book, both online and PDF
Why is this valuable (or not)?
New book have sample application created with Nim language, the old one is not
How do we know it's really free?
Specify in resource
For book lists, is it a book? For course lists, is it a course? etc.
Yes, it's book
Checklist:

[x] Not a duplicate
[x] Included author(s) if appropriate
[x] Lists are in alphabetical order
[x] Needed indications added (PDF, access notes, under construction)

Thanks for review, I've change per your comment

seems this failed because another merge broke the build.

Thanks, and welcome to free-programming-books!


feat: implement checkpointing and state persistence for Parquet Reader and FileSource

As part of this issue, we want to add state persistence to the Parquet File Source. Basically, if the Dagger job gets stopped  due to some reasons, the job should resume from a reliable state and should not start processing all the parquet files again from the beginning.

This was done and merged as part of PR: https://github.com/odpf/dagger/pull/163


[questions] i18n routes and sitemap.xml

Hello !
First thanks for the work on this project :)
I'm just wondering if this plugin can handle generation of sitemap.xml file related to generated routes.
Also, it may be interesting to be able to localize URI per locale.
I think <route> SFC block is a good place for these settings.
I can start working on it, but I prefer to ask before as this plugin is growing :)

Sounds good, but this is a pages plugin, so I think create a new plugin to handle it will be better.
We can do something like this to get generated routes. WDYT

Yes it is a better idea to externalize the sitemap generation with an exposed api :).
For the i18n routes, I like the nuxt's behavior, but it means that vue-i18n is installed. Do you think this is a feature that this plugin can handle ?

This is out of scope of this plugin, but provide a resolver with some hooks this plugin have can make it works.


[engsys] Remove unused @types/inquirer dependency from api-management-custom-widgets-tools

Packages impacted by this PR

@azure/api-management-custom-widgets-tools

Issues associated with this PR

Fixes #22842

Describe the problem that is addressed by this PR
Removes type definition dependency that is unused (there's no corresponding dependency on the actual inquirer package and no references to the types in the code.

API change check
API changes are not detected in this pull request.


00## - Check if a number is prime

Description of the Problem
Check that a number is prime
Code
Input: 7
Output: 7 is  a prime number

Input: 8
Output: 8 is not a prime number

Link To The LeetCode Problem
LeetCode

Please Assign this to me, I can do this efficiently.
Thank You.

I have solve this problem . #738

Closed - Does not match Contribution Guidelines! Only LeetCode Problem accepted!


[SMALLFIX] Removed explicit argument type in JournalIntegrationTest

Removed an explicit argument type in the JournalIntegrationTest class.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/10185/
Test PASSed.

LGTM

LGTM


Explore whether there is any legitimate reason to return more than once

One of the current calling convention rules is written as:

Every rule must expand to exactly one macro invocation and nothing else.
Output tokens are expected to be returned through tt_return!. Expanding to nothing, expanding to more than one macro invocation, or expanding to anything other than a macro invocation are not permitted.

There are possibly some interesting reasons to bend this in useful ways.


GEODE-8448: Update README based on user feedback

Update Redis API readme based on user testing feedback.

Acceptance Test failure is unrelated to this PR


Added Punctuation

Closes #95
Added the punctuation codes to the switch case

Thank you very much for your help, @GravityProtogen! 😘


cnijfilter-common: add generic build method for Canon printer drivers (help needed)

Motivation for this change
Older Canon printers need a different driver version for each printer. Currently we have a couple of versions implemented by hand, but it would be much better to have a generic implementation that can build any version. This package is a port of a Gentoo solution found here.
Things done
I've ported the generic framework and mostly got it working. I've also added an instance of a particular driver version. It would be great if someone could help me out with finishing it.

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Right now this fails for me:
gcc  -O2 -L../..//libs_bin64  -o cif bjferror.o bjfilter.o bjfimage.o bjfoption.o bjfpos.o bjfrcaccess.o getipc.o bjflist.o -lcnbpcmcm -lcnbpess -lm -ldl -ltiff -lpng -lcnbpcnclapi -lcnbpcnclbjcmd -lcnbpcnclui -lpopt
/nix/store/sq2b0dqlq243mqn4ql5h36xmpplyy20k-binutils-2.31.1/bin/ld: cannot find -lcnbpcmcm
/nix/store/sq2b0dqlq243mqn4ql5h36xmpplyy20k-binutils-2.31.1/bin/ld: cannot find -lcnbpess
collect2: error: ld returned 1 exit status

Yes, I see the same thing. Oddly, if I build step-by-step in nix-shell it doesn't happen. BTW, for development it's a good idea to comment out all but one printer model, then it configures much faster.

How exactly do you build step by step? (I'd like to reproduce)

BTW, for development it's a good idea to comment out all but one printer model, then it configures much faster.

Oh thanks, that's a good hint :) maybe it's possible to make each printer model its own derivation?

My attempt:
$ nix-shell -A pkgs.cnijfilter-common.cnijfilter-3_80

[nix-shell] $ phases="configurePhase buildPhase" genericBuild
configuring
libs
bash: pushd: libs: No such file or directory

exit

Any idea what I'm doing wrong?

I usually do it like this:
From the nixpkgs directory:
nix-shell default.nix --pure -A pkgs.cnijfilter-common
mkdir /tmp/canon-build
cd /tmp/canon-build
unpackPhase
cd cnijfilter-source-3.80-1
patchPhase
eval "$configurePhase"
eval "$buildPhase"

Oh thanks, that's a good hint :) maybe it's possible to make each printer model its own derivation?

I think initially we should just replicate what the Gentoo overlay does and have packages supporting multiple printers (since most users will just get it from Hydra anyway). We should also have some nixos module that selects the appropriate package for your printer.
Then for v2, we could move to the approach suggested here where the common part is built separately and each printer has its own package.

-lcnbpcmcm

The error must be that some environment variable is not set. The correct library name should be e.g. -lcnbpcmcm402, where 402 corresponds to some model id. I guess this works in the nix-shell because it persists some environment variables differently?

Excellent, thanks for your help. cnijfilter_3_80 now builds, but some stuff seems to end up in $out/usr/libexec/cups/backend/, the cnijnet and cnijusb executables.

I ported the ebuilds for the other driver version, sadly most of them don't build yet.

@terlar does it work for you on this branch?


Add semicolons and other cleanup eslint things

10/10 titling from sleepy mobile Insist right there

Just curious, do you want to integrate this with Pokemon Showdown? I understand the intent to change code style to match Showdown style, however this isn't strictly speaking a part of Pokemon Showdown and as such doesn't have to follow its style (and I will admit, I like not using semicolons in JavaScript myself).
That said, there are some things I would love to see changed in PSDevBot in event you want to contribute somehow, but I'm too busy to change them because they are low priority, in particular modernization of this code to ECMAScript 6. To be exact, PSDevBot could be improved by means of template literals usage, let/const, request promises, use of maps instead of objects. It also would be neat to have some unit testing to ensure things keep working, which I may as well create soon I suppose.

Sure I could do let/const things


[api-minor] Add a new transferBinaryData option to allow transferring more data to the worker-thread (bug 1809164)

Also, removes the initialData-parameter JSDocs for the getDocument-function given that this parameter has been completely unused since PR #8982 (over five years ago). Note that the initialData-parameter is, and always was, intended to be provided when initializing a PDFDataRangeTransport-instance.

This seems to work, but doesn't cover initialData yet.
/botio unittest

/botio unittest

/botio test

Hopefully this should work correctly, and there's now basic unit-tests for all the relevant code-paths.

What about: https://github.com/mozilla/pdf.js/blob/d6f63b5fb7cc63fa5d38ff71a56b8ee46d4f1b33/src/display/transport_stream.js#L66 ?

What about:
https://github.com/mozilla/pdf.js/blob/d6f63b5fb7cc63fa5d38ff71a56b8ee46d4f1b33/src/display/transport_stream.js#L66

There's somewhat similar code found in all of the src/display/fetch_stream.js, src/display/network.js, and src/display/node_stream.js files, w.r.t. basically copying a TypedArray when reviving data, and it's not at all clear to me that it'd be generally safe to start changing that.

Even if we only change the mentioned src/display/transport_stream.js code, since that's probably the only thing that's generally safe, that code has quite limited test-coverage and I'm a bit worried about breaking the Firefox PDF Viewer in some weird edge-case.
Ninja-edit: Although looking the following code, I suppose that this should be safe in Firefox?

https://searchfox.org/mozilla-central/rev/a156a65ced2dae5913ae35a68e9445b8ee7ca457/toolkit/components/pdfjs/content/PdfStreamConverter.jsm#1042-1052
https://searchfox.org/mozilla-central/rev/a156a65ced2dae5913ae35a68e9445b8ee7ca457/toolkit/components/pdfjs/content/PdfStreamConverter.jsm#207-228

/botio unittest

In looking at:
https://searchfox.org/mozilla-central/rev/a156a65ced2dae5913ae35a68e9445b8ee7ca457/toolkit/components/pdfjs/content/PdfStreamConverter.jsm
I noticed that all the postMessage calls don't use the 3rd argument for transferable.
So right now, it means that we get a copy of the data in pdf.js side.
I do wonder if we should it: I think it should be safe (note that I don't have a strong certainty), wdyt ?
Anyway I'm fine to take the risk to break some corner cases: if something is broken it's an opportunity to improve and add some new tests so it isn't that bad :).
Even the initial goal of the bug I filed was mainly to fix a memleak, I think it's not that bad to improve that stuff here: it could help to win few 0.01% of battery on mobile.

I noticed that all the postMessage calls don't use the 3rd argument for transferable.
So right now, it means that we get a copy of the data in pdf.js side.
I do wonder if we should it: I think it should be safe (note that I don't have a strong certainty), wdyt ?

For the data that's coming directly from the PdfStreamConverter.jsm file it really ought to be safe, given how that data is being read:

https://searchfox.org/mozilla-central/rev/a156a65ced2dae5913ae35a68e9445b8ee7ca457/toolkit/components/pdfjs/content/PdfStreamConverter.jsm#1042-1052
https://searchfox.org/mozilla-central/rev/a156a65ced2dae5913ae35a68e9445b8ee7ca457/toolkit/components/pdfjs/content/PdfStreamConverter.jsm#207-228

For the data that's coming from the PdfJsNetwork.jsm file, I suppose that it really depends on how the browser XMLHttpRequest-implementation works. Assuming that multiple onreadystatechange calls don't share one underlying ArrayBuffer, transferring this data ought to be safe as well.
Maybe we can just try updating the relevant postMessage calls with a transfers-argument, and test if that works?
However, we probably want to land this PR in mozilla-central first to not have all changes in the same Nightly-version.

As far as I can tell, it'd be very strange to not have full ownership of the ArrayBuffer returned by xhr because it'd mean that the data could change if not read/copy immediately and in such a case I'd expected to have a big warning in the MDN doc.
And yes, we should change that stuff in 111 (it should be next week iirc).


ERROR: cannot launch node of type [lio_sam/lio_sam_imuPreintegration]: Cannot locate node of type [lio_sam_imuPreintegration] in package [lio_sam]. Make sure file exists in package path and permission is set to executable (chmod +x)

ERROR: cannot launch node of type [lio_sam/lio_sam_imuPreintegration]: Cannot locate node of type [lio_sam_imuPreintegration] in package [lio_sam]. Make sure file exists in package path and permission is set to executable (chmod +x)
ERROR: cannot launch node of type [lio_sam/lio_sam_imageProjection]: Cannot locate node of type [lio_sam_imageProjection] in package [lio_sam]. Make sure file exists in package path and permission is set to executable (chmod +x)
ERROR: cannot launch node of type [lio_sam/lio_sam_featureExtraction]: Cannot locate node of type [lio_sam_featureExtraction] in package [lio_sam]. Make sure file exists in package path and permission is set to executable (chmod +x)
ERROR: cannot launch node of type [lio_sam/lio_sam_mapOptmization]: Cannot locate node of type [lio_sam_mapOptmization] in package [lio_sam]. Make sure file exists in package path and permission is set to executable (chmod +x)

I also encountered this problem. Have you solved it? I started getting errors at 80% of catkin_make. Although it finally reached 100%, there were many compilation errors in the middle.

Can you share how you managed to solve the problem if you did.


Is it possible to disable mixed precision on a layer specific layer?

It seems that mt5/t5 models that have been pre-trained under bfloat16 don't quite work under fp16 mixed precision and require special handling.
I found a workaround by disabling autocast (pytorch native amp) for one layer that caused all the problems:
https://github.com/huggingface/transformers/pull/10956
The gist of the change is:
    def _forward(self, hidden_states):
        forwarded_states = self.layer_norm(hidden_states)
        forwarded_states = self.DenseReluDense(forwarded_states)
        hidden_states = hidden_states + self.dropout(forwarded_states)
        return hidden_states

    def forward(self, hidden_states):
        # many t5/mt5 models are trained in bfloat16 and don't do well under mixed precision (fp16).
        # It appears that it's enough to disable autocast for this FF layer to avoid inf/nan
        # problems for the whole model
        if torch.is_autocast_enabled():
            with torch.cuda.amp.autocast(enabled=False):
                return self._forward(hidden_states)
        else:
            return self._forward(hidden_states)

Is there a way to do the same for DeepSpeed? i.e. continue using fp16 mixed precision for everything but a specific context?
Actually further testing shows that for a simple case, this is enough:
    def forward(self, hidden_states):
        forwarded_states = self.layer_norm(hidden_states)
        with torch.cuda.amp.autocast(enabled=False):
            forwarded_states = self.DenseReluDense(forwarded_states)
        hidden_states = hidden_states + self.dropout(forwarded_states)
        return hidden_states

But I haven't done full testing to know if it covers all bases.
Thank you!

Hi, I met with this problem too. Have you figured out a plausible way to realize that?

Bump. Did this ever get worked out?

Not that I know of.


Deleting a deployment fails

[Env]
Kubernetest V1.4.9
[Descriptions]
on my test env,  etcd/apiserver/control-manage/scheduler pods are deployed on different machines.
i met error information when deleting a deployment.  The deployment will be deleted when repeating the operation twice or three times.
[Log]
[root@localhost tt]# kubectl -s <IP_ADDRESS>:80 delete -f webapp-deployment.yaml --timeout=600s
error: error when stopping "webapp-deployment.yaml": timed out waiting for the condition

[Question]
how to config apiserver or other components to make it pass?  Thanks

@b43646 this issue should've been registered in kubernetes/kubernetes repo

@b43646 Thanks for opening this issue. It looks like you're asking for specific help with your environment, rather than making a feature request for the docs. I encourage you to bring your question to the #kubernetes-users channel in Kubernetes slack.


Refactor daemon to allow testing platform detection

PTAL @SalDaniele

/lgtm

/hold need to think a bit more about naming

/test make-e2e-test

/retest

/test make-e2e-test

/test make-e2e-test

/retest

/unhold

/test make-e2e-test

/test make-fast-e2e-test

/lgtm


logging in doesn't refresh the current page

In backstage on an app such as _space logout of tiddlyspace.
Now login.
_space still displays as if you were logged out.
Expected:
the iframe needs to refresh the parent frame.

This is now addressed and deployed so closing.


Unable to add field during creation for fields that require the model id.

I've overridden a field in my model so that the user can create this model relationship when the user creates the parent model. Instead of using an image directly on the model, I have an Image model that allows me to attach multiple images to the model. The only problem with this is when the user creates the model with the image it fails with the SQL error on the child object as the parent model_id has not yet been created. This is fine when the parent model already exists but fails when the user is creating the parent model.

I think this is a good feature. Since there's already a PR for this, I say let's move the entire conversation there - #925


Snap : joystick

It would be great to be able to use a plug like joystick.
I saw another similar snap that you could run a command like:
snap connect supertuxcart:process-control :process-control
snap connect supertuxcart:joystick :joystick

Perhaps this could even be done by default, if that's an option.
Any chance this could come here?

This is already possible. On any system run:
sudo snap connect supertuxkart:joystick

or on Ubuntu you can use the Software Centre to do it via a GUI:

This issue hasn't had any replies for some time, so I'm going to close it for now. Please reopen if it is still a problem, or open a new issue for other problems.

I'd like to reopen this, or open a new issue specifically about Wiimotes (or maybe Bluetooth gamepads in general).  I've tried to configure STK to use a Wiimote without success: none of mine ever successfully pair, and my guess is this fails because the snap doesn't have permission to access bluetooth devices.
STK handles this controller type via the wiiuse library, which in turn depends on kernel bluetooth support and the BlueZ drivers (both are enabled on my Ubuntu 18.04 system, I use several BT devices).  When STK is run with logging enabled, this is what I see:

wiiuse v0.15.4 loaded.
De-facto official fork at http://github.com/wiiuse/wiiuse
Original By: Michael Laforest <thepara[at]gmail{dot}com> https://sourceforge.net/projects/wiiuse/
[INFO] wiiuse clean up...
hci_get_route: Permission denied

I wonder if this could be resolved by plugging into the bluetooth-control and/or bluez interfaces?


Add support for<EMAIL_ADDRESS>

This project currently only supports<EMAIL_ADDRESS>but now that tslint 6 is out, it would be great to have support.
The only issue I ran in to when I ignored the dependency mismatch warning was that it seems to be finding invalid regex expressions inside of quoted strings (I tried single quotes, double quotes, and back-tick quotes):
The 'export-name' rule threw an error in 'R:/MyProjPath/src/config/index.ts':
SyntaxError: Invalid regular expression: /^{
.
.
.
        prefix: process.env.API_PREFIX || '/api',
.
.
.

}\..+/: Nothing to repeat
    at new RegExp (<anonymous>)
    at validateExport (R:\MyProjPath\node_modules\tslint-microsoft-contrib\exportNameRule.js:146:21)
    at walk (R:\MyProjPath\node_modules\tslint-microsoft-contrib\exportNameRule.js:170:13)
    at Rule.AbstractRule.applyWithFunction (R:\MyProjPath\node_modules\tslint\lib\language\rule\abstractRule.js:40:9)
    at Rule.apply (R:\MyProjPath\node_modules\tslint-microsoft-contrib\exportNameRule.js:28:21)
    at Linter.applyRule (R:\MyProjPath\node_modules\tslint\lib\linter.js:215:29)
    at R:\MyProjPath\node_modules\tslint\lib\linter.js:156:85
    at Object.flatMap (R:\MyProjPath\node_modules\tslint\lib\utils.js:161:29)
    at Linter.getAllFailures (R:\MyProjPath\node_modules\tslint\lib\linter.js:156:32)
    at Linter.lint (R:\MyProjPath\node_modules\tslint\lib\linter.js:111:33)

This project is read-only and archive mode.


Improve suggested tags

We want to add improved weighting that gives better suggestions.
In order of bias where tags are seen:

Bottle
Brand/Distillery
Region
Country

Added randomization to the list so at least selection bias goes away. Currently this is only weighted by Bottle, and likely requires tags to be materialized/indexed to do more.


QUESTION: GUI for Modules Management

Hello
Someone know if exist a component to manage all modules, like, add, disactivate, activate, etc etc with crud included.
Thanks

I do not believe this exists. Could be a nice standalone package I think. Let me know if you're going to implement something like this!

I do not believe this exists. Could be a nice standalone package I think. Let me know if you're going to implement something like this!

Check This, https://laraadmin.com/docs/1.0/module_manager, but  i think that is not support any longer.
I will to check that project and i try to adapt to laravel v8 only the module manager system.
see you soon.

btw this would have been better on the discussion page https://github.com/nWidart/laravel-modules/discussions


Fix Ansible 2.14 unsupported parameters for main branch

Signed-off-by: Rajan Mishra<EMAIL_ADDRESS>

Hi Rajan, looks fine for me !

Suggest to use #718 instead...


deadletter oddities

This morning the deadletter graph in the dashboard looked as pictured below. Since I was not taking any action, (and I assume you were not) its is hard to see how the deadletters could ever go down.
This may be dashboard related but I suspect something is amiss with the server side computation?

Just now I used the command line to get the count and it said 260 while the dashboard itself is reporting 180 and storage explorer was saying 173 (though it was acting a little off so I restarted it and it reported 262).  Ask three mechanisms and get four answers...  may be some funny business on the Azure side?

It looks like it's been happening about every minute just when the value is not cached anymore. Those requests take more than 5 sec to complete when we are polling every 5 sec...

Interesting.  We do cache the deadletter count for 1 minute which might explain why every minute it is longer.  I would not expect it to take longer than 5 sec though.  And i would expect it to return the right value when it does return...
anyway, lets monitor for a couple days and see if the problem resurfaces.

behavior seems to be back to normal.  closing


Add a new update available widget to statusBar

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)

Proposed changes
Part of #4931.
Add a new update available widget to statusBar.
I also modified the statusBar to allow hiding/showing the widget.
Checklist

[x] I have read the contribution guidelines
[x] For non-bugfix PRs, I have discussed this change on the forum/slack team.
[x] I have run npm run test to verify the unit tests pass
[ ] I have added suitable unit tests to cover the new/changed functionality

There are two things left to do here:

add the interval to update the catalog (L631)
refresh the status after each module update

As discussed on slack, I'll take care of pulling apart the catalogue refresh from the ui refresh.

This PR is being replaced by #4948 - contains all the same work, plus the additional refactoring on how catalogues are loaded.


Extension name and description

Is your feature request related to a problem? Please describe.
The name of the extention is the name of the package: https://github.com/Jonghakseo/chrome-extension-boilerplate-react-vite/blob/main/manifest.js#L9
Package name regexp is restricted by pattern:  ^(?:(?:@(?:[a-z0-9-*~][a-z0-9-*._~]*)?/[a-z0-9-._~])|[a-z0-9-~])[a-z0-9-._~]*$
But in réality name is rather a human-readable name (e.g. "MyExtension : best extension ever" , "ExtensionBlack : best ad-blocker in the world"...)
Is the a way to have another package.json field mapped to Extension name.
Or even support multiple languages https://developer.chrome.com/docs/extensions/reference/i18n/#how-to-support-multiple-languages
Describe the solution you'd like
In manifest.js, I'd set name and description like this:
"name": "__MSG_extensionName__",
"description": "__MSG_extensionDescription__",
"default_locale": "en"

and I'd create following files:

_locales/en/messages.json
with "_locales/en/messages.json" file containing such values:

{
  ...
  "extensionName": {
    "message": "Chrome extension boilerplate with React, Vite & Typescript",
    "description": "Extension name"
  },
  "extensionDescription": {
    "message": "Chrome extension boilerplate",
    "description": "Extension description"
  },
  ...
}

What do you think about that?
I can do the PR.
Thanks in advance,

I'm personally using this feature, cause from this template you don't create lib like this or any other like MUI, Nest or anything.
It's useful IMO.
If you need multiple languages you can use e.g i18next, and use this value from package.json as a key.

@Jonghakseo are you interested or I cancel the PR ?

@romaindequidt
In fact, today is the first time I've seen this offer and PR properly. But I've been thinking about it off and on since it came up.
I was thinking that the field MSG_extensionName seems like a bit of an afterthought for developers who aren't thinking about multilingual support, and that it could be a hindrance to understanding the configuration of the extension.
However, I think it's good that the name of the package is strictly limited by the rules, as you suggested, and the problems I mentioned can be compensated through comments inside the JSON.
I'll do a review on PR, thank you so much for your contribution.


[Program Card Images] [Admin] Display the program image in the active & draft version rows on program index page

We want to make sure admins remember to upload images for (preferably) all their programs. Including the current image in the program index page will make it obvious to admins which programs have images and which don't.
Because admins need to make sure they hit Publish in order for applicants to actually see the image, we should include the image in both the Active row and the Draft row. This should highlight that a new image has been added to the draft program, but hasn't been published.
Part of Epic #5676

Oops, realized I created two issues for this.
Duplicate of https://github.com/civiform/civiform/issues/6281


Is there any way to use Webpack 5?

Problem Description
I am using a third-part plugin, i.e. splidejs. This is where the problem comes.

Module parse failed: Unexpected token (3221:2)
You may need an appropriate loader to handle this file type.

Steps To Reproduce

Install the package npm install @splidejs/react-splide
Import the package to the Module - import { Splide, SplideSlide } from '@splidejs/react-splide';

Screenshot and/or gif
Failed to compile.

<EMAIL_ADDRESS>Module parse failed: Unexpected token (3221:2)
You may need an appropriate loader to handle this file type.
|   children: children2,
|   className,
|   ...props
| }) => {
|   return /* @__PURE__ */React.createElement("div", {
```

+1
same error with chart.js


Can't match domains like 17173.com

Hello, You creating a great software.
It's works great but when I trying to crawl some website like 17173.com which domain starts with number, it wouldn't works.
I found the problem is in the submodule "web", the regex that in class AbsUrl would not match such domains. Could you change it for me? Thanks.

Hi,
Thanks for reporting the issue.
I have fixed the regex for url. Although, now, the module 'web' is part of the package 'mutils' (on pypi). If you download and install imagebot(v1.2.1), you should not see this issue.
Regards,
Amol.


Loading models segfaults on x86

I've been having trouble training and testing my models on my desktop PC. Currently, the dev branch (821bea9ec7e1e4fc61c7d7e30832ed53222510dc) and nvmm-disabled (9a94ef5053ccb7c28aa02539180a47fb46a22853) compiles. Other branches like master (9bf55495d6a4ff1946dddbb0e81102d2d05f4952) produce this error when I try to build:
 [  6%] Linking CXX shared library ../x86_64/lib/libjetson-utils.so
/usr/bin/ld: cannot find -lnvbuf_utils

When I compile dev or nvmm-disabled and run the detectnet executable through a script, I get this output:
[TRT]    cache file not found, profiling network model on device GPU
[TRT]    The logger passed into createInferBuilder differs from one already assigned, 0x7ffe5f25b868, logger not updated.

./test.sh: line 36: 47735 Segmentation fault      (core dumped) detectnet --model=$MODEL --labels=$LABELS --input-blob=input_0 --output-cvg=scores --output-bbox=boxes --threshold=$THRESHOLD $INPUT $OUTPUT

I've run this same model on real Jetson hardware with no issues. I was able to run this command a year or so ago when my machine was on Ubuntu 18, cuda 10.X.
Is this issue happening because I'm not on the master branch? What is causing nvbuf_utils to not be found? I'm happy to provide more info if needed.
uname -a output:
Linux XXX 5.4.0-89-generic #100-Ubuntu SMP Fri Sep 24 14:50:10 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
lsb_release -a output:
Distributor ID:	Ubuntu
Description:	Ubuntu 20.04.3 LTS
Release:	20.04
Codename:	focal

nvidia-smi output:
Thu Nov  4 19:34:15 2021
+-----------------------------------------------------------------------------+
| NVIDIA-SMI 465.19.01    Driver Version: 465.19.01    CUDA Version: 11.3     |
|-------------------------------+----------------------+----------------------+
| GPU  Name        Persistence-M| Bus-Id        Disp.A | Volatile Uncorr. ECC |
| Fan  Temp  Perf  Pwr:Usage/Cap|         Memory-Usage | GPU-Util  Compute M. |
|                               |                      |               MIG M. |
|===============================+======================+======================|
|   0  NVIDIA GeForce ...  On   | 00000000:01:00.0 Off |                  N/A |
|  0%   37C    P8     9W / 215W |    337MiB /  8116MiB |      0%      Default |
|                               |                      |                  N/A |
+-------------------------------+----------------------+----------------------+

+-----------------------------------------------------------------------------+
| Processes:                                                                  |
|  GPU   GI   CI        PID   Type   Process name                  GPU Memory |
|        ID   ID                                                   Usage      |
|=============================================================================|
|    0   N/A  N/A      1524      G   /usr/lib/xorg/Xorg                327MiB |
|    0   N/A  N/A      1681      G   /usr/bin/gnome-shell                6MiB |
+-----------------------------------------------------------------------------+

nvcc --version output:
nvcc: NVIDIA (R) Cuda compiler driver
Copyright (c) 2005-2021 NVIDIA Corporation
Built on Mon_May__3_19:15:13_PDT_2021
Cuda compilation tools, release 11.3, V11.3.109
Build cuda_11.3.r11.3/compiler.29920130_0

dpkg -l | grep nvinfer output:
ii  libnvinfer-bin                                              8.0.0-1+cuda11.3                      amd64        TensorRT binaries
ii  libnvinfer-dev                                              8.0.0-1+cuda11.3                      amd64        TensorRT development libraries and headers
ii  libnvinfer-doc                                              8.0.0-1+cuda11.3                      all          TensorRT documentation
ii  libnvinfer-plugin-dev                                       8.0.0-1+cuda11.3                      amd64        TensorRT plugin libraries
ii  libnvinfer-plugin8                                          8.0.0-1+cuda11.3                      amd64        TensorRT plugin libraries
ii  libnvinfer-samples                                          8.0.0-1+cuda11.3                      all          TensorRT samples
ii  libnvinfer8                                                 8.0.0-1+cuda11.3                      amd64        TensorRT runtime libraries
ii  python3-libnvinfer                                          8.0.0-1+cuda11.3                      amd64        Python 3 bindings for TensorRT
ii  python3-libnvinfer-dev                                      8.0.0-1+cuda11.3                      amd64        Python 3 development package for TensorRT

I found the issue. Or at least, part of it. I upgraded using this command:
sudo apt install libnvinfer8*
libnvinfer is now on version 8.2.0.
I still can't compile the master branch. nvmm-disable is working though.


Fix missing permissions for OSX app

This will fix missing permissions for the OSX application
This also includes a refactoring of the CI to be able to test the package built form pull request and catch this kind of bugs at an early stage

It will be necessary to add a new build.yml to solution. I've not found a way to do this with dotnet CLI

Tested.


Axis reordering Jacobian

Context:
We have chosen wrong convention for axis ordering of Jacobians.
f(x, y) -> (a, b, c) should return a Jacobian df = ((ax, ay), (bx, by), (cx, cy))
Description of the Change:
The arguments and return axis is switched.
It changes the shapes of Jacobians.
Benefits:
Better convention.
Todo:

Update finite diff.

[sc-46574]

Thanks @rmoyard, the PR looks great! Just some minor comments from my side, please see the conversations.


[release-4.12] OCPBUGS-20417: syncer: fix nil pointer dereference in log message

https://issues.redhat.com/browse/OCPBUGS-20417
This is a manual backport of https://github.com/openshift/vmware-vsphere-csi-driver/commit/6de63b32bd4308f0ff689b6657a2201bd2dcebd1
This fix already exists in 4.13+
/cc @openshift/storage @duanwei33

/jira refresh

/label backport-risk-assessed

/retest

/hold
For pre-merge testing.

Issue reproduced on 4.12.34:

create pv with volumePath but no .vmdk.
vsphereVolume:
fsType: ext4
volumePath: '[host2-local] test1'
create pvc to bind pv and create pod
wait vmware-vsphere-csi-driver-controller pod crash with panic:
panic: runtime error: invalid memory address or nil pointer dereference [recovered]
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x8 pc=0x1b3c7c5]

Then build pre-merge build with this open PR, and upgrade to the build, do the same thing and panic doesn't happen.

/label qe-approved
/label cherry-pick-approved

/hold cancel


Gaps between blocks

I saw a previous issue saying to solve gaps between blocks to use a subsurface modifier but I'm still getting gaps between blocks I'm not sure if this is an issue with the model I have or something. This is what I've been using. I have downloaded a model off the internet and then increased the scale idk if that would effect things. Thanks for this awesome mod though!

The script that converts the model to voxels only uses the vertices of the mesh. Therefore, if there is too much space between two vertices, no blocks will be placed there. The subsurf modifier works well if your mesh has uniform detail, meaning that the vertex density is roughly the same everywhere. If you have some large faces, consider subdiving them manually or adding some loop cuts. Once the detail is uniform everywhere you can increase the subsurf modifier level until there are no more gaps.
I hope this makes things more clear!

Thank you this worked! I have one other issue and that is that conduits are used in the ingame structure a good amount, and always spawn in water, which then drips down the build. Do you know of a way so that certain blocks can not not be spawned in with the commands?

Sadly, that is still an issue I haven't addressed. Currently, the only way to blacklist certain blocks is to edit the mod's code. If you are able to do that and rebuild the mod afterwards, the line you have to edit is line 71 inside VoxelLoader.java, where you can add a check to make sure the block isn't a conduit.
The other solution, which is easier if you are using vertex colors, is to change the color that is producing conduits by a little bit so that it produces another block instead, but that could be annoying if you are using a texture.
If you are unable to fix it by yourself, I will soon start working on an update to prevent such problems more easily.

Thanks for the help! I'll try editing that line.


Add basic sphinx documentation

This is pretty awesome. How can we get this to be the default documentation? https://readthedocs.org is fairly easy to setup.


CoreLabel does not return the same width in PY2 as in PY3

The figure from the left is in python2 and the one from the right is in python3 the yellow rectangles are the dimensions of the corelabel elements.

@andnovar, could you share a test app for this which does not involve matplotlib?

used six.text_type("{}".format(s)) to format text gives the same width and height in both py2 and py3


FT-710 will not transmit

Hi. Have the latest version of FT8CN, v.0.88
All works as advertised using a Motorola android smartphone, lining to the rig with a (USB-B - to - USB-A cable) and a (USB A-to-C) OTG adaptor. I have tried a (USB A to C) cable but this will not permit the phone to recognise a radio.
The decoding happens without issue, and everything works fine EXCEPT THAT the radio will NOT transmit.
When the time comes to transmit, the rig is activated accordingly but no output ensues.
Without a specific profile for the 710, I have entered the nearest available which is the FT-DX10 profile

I have uploaded a screenshot of my settings
Help!

Oh I do know I have the callsign blank in the screenshot, in use I have my legit callsign in there

Google Translate on text above....
##########################################
It is not clear whether there is a difference in CAT between FT-710 and FT-DX10.
The reasons for no output are generally as follows:

ALC is too low. You can increase the volume of your phone to improve ALC
Audio is not being output to the FT-710. Please check the audio cable, FT-710 should support a virtual sound card, please check the USB Cable, or check the OTG transfer
Radio settings. Some radio settings will also affect audio transmission, please check the radio settings, you can refer to the related settings of JTDX and WSJT-X.
############################################
Thank you confirming that FTDX10 settings apply ti FT710
I have adjusted ALC and 'phone output settings on the app, with no effect.
I have no idea how to check whether any audio is actually coming down the USB cable from the phone to the FT710
I shall check radio settings according to videos available on line, specifically using the FT710 with WSJT-X.

I also read that ferrite beads on the USB cable may help. I have ten beads and placed them all on the USB cable, but still no output.
Many thanks, Ken

Having considered the issues above, I investigated forcing audio to appear at the USB-c connector on my 'phone. It transpires that the availability of audio at the 'phone's USB-c connector has been removed as from Android 12! It cannot be restored in developer mode.
One solution is to use a USB soundcard.
Another technologically easier solution is to make a cable using a 4-pole 3.5mm connector for the 'phone's audio in and out, and connect this directly to the radio's mic and speaker-out connectors, with appropriate attenuators and DC-blocking capacitors.
This should work for any version of Android to work with any radio, with no setup required. Of course changes in frequency at either end of the cable will not be reflected at the other, but this was never crucial.
Of course this will not work for those 'phones that have no sockets at all.
Ken

Many thanks. I have a work- around meanwhile. I was made aware of an ADI to CSV converter at https://ad1c.us/software/ADIF_to_CSV/index.html
which is read by Excel.
A few macros and l have my logbook!
Many thanks,
Ken Ruiz
Sent from my HAL9000 computer

From: bg7yoz @.>
Sent: Monday, May 15, 2023 4:02:04 AM
To: N0BOY/FT8CN @.>
Cc: ZB2ham @.>; Author @.>
Subject: Re: [N0BOY/FT8CN] FT-710 will not transmit (Issue #38)
暂时不支持CSV或XLS格式，以后也许会考虑增加这个选项。
―
Reply to this email directly, view it on GitHubhttps://github.com/N0BOY/FT8CN/issues/38#issuecomment-1547089760, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A7R67UR4GAENC22OTN2SZG3XGGFBZANCNFSM6AAAAAAXTHLJ4I.
You are receiving this because you authored the thread.Message ID: @.***>

I have posted my personal low-tech, low-cost solution to lack of audio at the USB-C connector in Android 12.
See Discussions > Categories > Show & Tell > Audio Link
I hope this is of some use.
Ken


Can install plugin, but autocompletion does not work.

I installed the plugin tern_for_vim. Followed all the instructions from readme but it doesn't work. I don't have any autocompletion for javascript, the commands :TernDef and :TernDoc are not recognized in vim. I installed the tern server globally using npm and I can launch it in the terminal.
However, if I launch :call tern#Enable() it seems that the plugin gets loaded and I can see :TernDef and :TernDoc commands but still no autcompletion for javascript.
I tried to launch tern with the --verbose option but it did not output anything when I tried to use autocompletion in javascript files.
It seems that there is no communication between vim and the tern server if I understand it correctly.
Here are my plugins in .vimrc file:
" let Vundle manage Vundle, required
Plugin 'VundleVim/Vundle.vim'
Plugin 'tmhedberg/SimpylFold'
Plugin 'vim-syntastic/syntastic'
Plugin 'scrooloose/nerdtree'
" Plugin 'jistr/vim-nerdtree-tabs'
Plugin 'kien/ctrlp.vim'
Plugin 'vim-airline/vim-airline'
Plugin 'vim-airline/vim-airline-themes'
Plugin 'tpope/vim-fugitive'
Plugin 'vim-scripts/indentpython.vim'
Plugin 'nvie/vim-flake8'
" Plugin 'Valloric/YouCompleteMe'
Plugin 'davidhalter/jedi-vim'
Plugin 'SirVer/ultisnips'
Plugin 'honza/vim-snippets'
Plugin 'pangloss/vim-javascript'
Plugin 'mxw/vim-jsx'
Plugin 'alvan/vim-closetag'
Plugin 'Valloric/MatchTagAlways'
Plugin 'ternjs/tern_for_vim'
Plugin 'lervag/vimtex'

I also setup a .tern-config file in the home directory:
{
    "libs": [
	"ecmascript",
	"browser",
	"jquery"
    ],
    "plugins": {
        "es_modules": {},
	"node": {}
    }
}

I could run ternjs when I set up YouCompleteMe for python autocompletion, but I didn't like it and chose jedi-vim (which is really nice). Now I want to setup javascript autocompletion but without no luck.
Is it possible that the plugin is in conflict with some of my other plugins?
OS: Xubuntu 16.04 LTS

I had something similar. Tried adding loadEagerly?
http://ternjs.net/doc/manual.html#configuration

Thanks, for the response. How do you trigger autocompletion. What shortcut?

tern hook into omni completition, but your completion plugin may not, you can test whether your omni completion works by use

In hopefully what is considered a related note: I can't seem to confirm if the instance of tern started by the vim/nvim plugin is reading the defs files.  I cannot find the defs directory for the tern instance the plugin starts.  However, I have confirmed that tern does not find the third party libraries (react and ecma6) defs when I start the tern server from my z-shell.
Where should I expect to find the defs and plugins directories reference in the terndocs when I installed tern using tern_for_vim?
Thank you!

Where should I expect to find the defs and plugins directories referenced in the terndocs when I installed tern using tern_for_vim?

They are part of the tern package -- i.e. node_modules/tern/plugin/ under your tern_for_vim if you installed normally.

Perfect. In my case (Mac OS, installing tern the normal, npm i -g tern way that is run by your config spec) the location for both the plugin and defs directories was /usr/local/lib/node_modules/tern/.
I was able to confirm that the globally installed tern server was accessing the required files.  To do so, I manually fired up tern in the shell and got rid of any errors by manually copying the required files.
Specifically, I copied from the ternjs/tern github repo the required lib and plugin sources referenced in my .tern-config file.
Knowing what you shared was helpful because while this plugin takes care of the install, I still wanted to know where to put the extra source files that I'm referencing in the .tern-config file.
Am I defeating some project specific capacity/feature? I do know that my set-up is now working at a useful baseline level.
Thank you for your help.

You can also put definition files and plugins in your project—if their name, interpreted as a filename relative to the .tern-project file, exists, that'll be used.

Sorry for the late reply. My issue was resolved. Here is a link to vim stackexchange - https://vi.stackexchange.com/questions/13348/cannot-setup-tern-for-vim-plugin/13359#13359.


Memory leak in KDD Syncer if we fail to create watchers

From @robbrockbank: "Because we aren't managing to create all of the Watchers, we never hit the part of the syncer loop that starts pulling events off the Watchers that we have created."
Expected Behavior
No leak
Current Behavior
Leaks significantly, if, for example the CRD manifests aren't installed so we fail to create CRD watchers.
Possible Solution
I think the best solution would be to use a goroutine per watcher, then the correct teardown logic could be deferred rather than needing to track a lot of state in the main loop.
Steps to Reproduce (for bugs)

Start felix with CRD-expecting libcalico-go but no CRDs created
Leak!

Context
Found in post 2.5.0 testing.
Your Environment

libcalico-go version: 1.6.0
Operating System and version:
Link to your project (optional):

This is being tracked over here: https://github.com/projectcalico/calico/issues/1057


build and test on arm64 mac

Adds build and test CI for arm64 Mac

Might fix #218 if build and test work on arm64

Could you try this in your own fork first? I don't think this is correct and GitHub also hasn't released free arm64 Mac agents yet.

I don't think this is correct and GitHub also hasn't released free arm64 Mac agents yet.

Au conratire! Just last week: https://github.blog/changelog/2024-01-30-github-actions-introducing-the-new-m1-macos-runner-available-to-open-source/

GitHub also hasn't released free arm64 Mac agents yet.

Au conratire! Just last week: https://github.blog/changelog/2024-01-30-github-actions-introducing-the-new-m1-macos-runner-available-to-open-source/
Has to be macos-14 though, latest still points to 12 (Intel)...

You're right, I totally missed that. That's great!

As for the rest, seems overly complex indeed... Why not use a matrix instead of duplicating all this stuff?

I wasn't able to express all the different cases with a matrix and it seemed that listed every config was more straight forward to manage.

Passing build and test here now: https://github.com/jakevc/rawpy/actions/runs/7879645476

All checks have passed for this

Thanks! :)

please release this version to PyPi when you get the chance


[Enhancement] Add ttl cleanup docs

Description
Add docs related to ttl cleanup:

new flags
supported date formats
reconciliation mecanism
etc...

Relevant PR(s):

https://github.com/kyverno/kyverno/pull/7821

Slack discussion
No response

I suggest waiting a bit longer in the development cycle before embarking on this issue. Your contribution would still be valued.


Emit extensions fix

Fixes emit-extensions and emit-descriptions features not being used by mavlink-bindgen (#246).
This requires a (none breaking) bump in version since fixes in the mavlink crate require newer versions of mavlink-bindgen/mavlink-core crates.
To ensure test work this required adding the path specifications for the dependencies within this crate.
Fixes missing storm32 dialect feature.

Looking forward to this as well, thank you

I was originally under the impression storm32 was just overlocked and since it generated a compiler warning I just added it.
But #165 indicated it was intentionally left out. I also probably should never have been in this unrelated PR. Therefore I removed it.

@pv42 can you rebase over master ? It appears to contain conflicts.

Fixed the conflicts with 0fb0e2e.

Should have be properly rebased now.


Viki authentication 1080p download not working.

The viki is working with this command
youtube-dl.exe -f best --get-url https://www.viki.com/videos/67066v-sungkyunkwan-scandal-episode-1  and the best quality is 480p
but if I use the premium account of viki to fetch HD videos, it error me and this is my command.
youtube-dl --username<EMAIL_ADDRESS>--password **** -f best --get-url https://www.viki.com/videos/67066v-sungkyunkwan-scandal-episode-1 --verbose flag
It return me this.
youtube-dl.exe --username<EMAIL_ADDRESS>--password querty123 -f best --get-url https://www.viki.com/videos/67066v-sungkyunkwan-scandal-episode-1 --verbose
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--username', 'PRIVATE', '--password', 'PRIVATE', '-f', 'best', '--get-url', 'https://www.viki.com/videos/67066v-sungkyunkwan-scandal-episode-1', '--verbose']
[debug] Encodings: locale cp1252, fs mbcs, out cp1252, pref cp1252
[debug] youtube-dl version 2019.04.30
[debug] Python version 3.4.4 (CPython) - Windows-10-10.0.17763
[debug] exe versions: ffmpeg N-93515-g6aeaac3e1c, ffprobe N-93515-g6aeaac3e1c
[debug] Proxy map: {}
ERROR: Unable to download JSON metadata: HTTP Error 404: Not Found (caused by HTTPError()); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpfeq7l785\build\youtube_dl\extractor\common.py", line 626, in _request_webpage
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpfeq7l785\build\youtube_dl\YoutubeDL.py", line 2227, in urlopen
  File "C:\Python\Python34\lib\urllib\request.py", line 470, in open
  File "C:\Python\Python34\lib\urllib\request.py", line 580, in http_response
  File "C:\Python\Python34\lib\urllib\request.py", line 508, in error
  File "C:\Python\Python34\lib\urllib\request.py", line 442, in _call_chain
  File "C:\Python\Python34\lib\urllib\request.py", line 588, in http_error_default

Hello, i have the same issue since 2 days.
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v', '-u', 'PRIVATE', '-p', 'PRIVATE', '--write-sub', '--sub-format', 'srt', '--convert-subs', 'srt', '--sub-lang', 'en,fr', '-o', 'E:\Web\(_) Nouveau\Viki\Download/%(title)s.%(ext)s', 'https://www.viki.com/tv/36575c-angels-last-mission-love']
[debug] Encodings: locale cp1252, fs mbcs, out cp850, pref cp1252
[debug] youtube-dl version 2019.05.20
[debug] Python version 3.4.4 (CPython) - Windows-10-10.0.18362
[debug] exe versions: ffmpeg N-87306-g6743351, ffprobe N-87306-g6743351
[debug] Proxy map: {}
[viki:channel] Logging in
ERROR: Unable to download JSON metadata: HTTP Error 404: Not Found (caused by HTTPError()); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpt31j5y3w\build\youtube_dl\extractor\common.py", line 626, in _request_webpage
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpt31j5y3w\build\youtube_dl\YoutubeDL.py", line 2227, in urlopen
File "C:\Python\Python34\lib\urllib\request.py", line 470, in open
File "C:\Python\Python34\lib\urllib\request.py", line 580, in http_response
File "C:\Python\Python34\lib\urllib\request.py", line 502, in error
File "C:\Python\Python34\lib\urllib\request.py", line 442, in _call_chain
File "C:\Python\Python34\lib\urllib\request.py", line 685, in http_error_302
File "C:\Python\Python34\lib\urllib\request.py", line 470, in open
File "C:\Python\Python34\lib\urllib\request.py", line 580, in http_response
File "C:\Python\Python34\lib\urllib\request.py", line 508, in error
File "C:\Python\Python34\lib\urllib\request.py", line 442, in _call_chain
File "C:\Python\Python34\lib\urllib\request.py", line 588, in http_error_default

I am having this issue now too.
I am wondering if the _APP_SECRET value changed again, and would need to be udated in the viki.py file.
Does anyone know how to find/figure out what the new value is?  When I login to the site and use the Chrome Inspect tool, I do see the dash files for all the resolutions.  I get the same JSON metadata / HTTP 404 error as both of you guys though.

Looking at the login page source:
<input type="hidden" name="authenticity_token" id="authenticity_token" value="cZPvSFgWRCNv+qkQxZm1HER2sjyiGpIyl0mwDO1Sb0PnGrdM3hStQwVEXoB7zAlOfjItY4fhjT/6o0g3e9vFsQ==">
Changing to this value does not seem to fix the issue, same JSON message.

Please help! When using username and password I am also getting the error:
ERROR: Unable to download JSON metadata: HTTP Error 404: Not Found

Authentication can only be done now through HTTPS.
In viki.py need to change
_API_URL_TEMPLATE = 'http://api.viki.io%s&sig=%s'
to
_API_URL_TEMPLATE = 'https://api.viki.io%s&sig=%s'

Yes!  This change did it for me.  Thank you!

Great! If everyone's saying it's working, I will give it a try! Thanks for figuring it out, and please incorporate it into the next release!

i'm waiting for the release change (windows)


Check perms using a userid?

How to check if a user has a certain permission by using the userid?

You do $user.Permissions("permission")
It will return a boolean of true or false


[Event Hubs] Support for local non-TLS

Summary
The focus of these changes is to add unofficial support for the forthcoming Event Hubs local development emulator.  This is intended to support internal testing on very early alpha builds.

/azp run net - eventhub - tests

API change check
API changes are not detected in this pull request.

/azp run net - eventhub - tests


fixes #31 - build Error: Cannot find module 'object.assign'

Hmm I don't think this should be valid - if you don't have Object.assign then you should use the polyfills - could you explain the need for this one?

The exception happens at build time, and this commit (as I understand) is doing exactly that - adds polyfill for Object.assign. @PWKad, are You saying that building works for You and I should install something else globally?


Support generic base classes

Currently this scenario is not supported.
public class BaseClass<T>
{
    public BaseClass(T t) { }
}

[AutoConstruct]
public class ChildClass : BaseClass<int>
{
}

All fixed in v1.1


TfLiteMobileNetSsd issue with rockpro64.

Hi, I have rockpro64(with Ubuntu18.04 OS).
I followed the link for tensorflowlite with armnn-
https://developer.arm.com/solutions/machine-learning-on-arm/developer-material/how-to-guides/configuring-the-arm-nn-sdk-build-environment-for-tensorflow-lite/setup-and-download-libraries
When i ran "TfLiteMobileNetSsd-Armn" test code from armnn, providing tflite model and all parameters, it threw error-
ArmNN v20190800 Armnn Error: No input binding found for subgraph:0 and name:normalized_input_image_tensor. Possible inputs are: ['Preprocessor/sub' ] at function GetNetworkInputBindingInfo [/home/rock64/rahul/armnn-tflite/armnn/src/armnnTfLiteParser/TfLiteParser.cpp:2670]
What can be the issue here?
Please suggest if any specific version to be used with armnn for ACL,tensorflow,flatbuffer,protobuf and boot library to run tensorflowlite inferencing for mobilenetssd detection?
Thanks.

@rahulsharma11 May I recommend to run SSD-MobileNet with ArmNN as described in ArmNN-MLPerf CK workflows?

Hi.
Thanks for the link.
Can you please also tell what was the issue for my above querry?
Thanks.

@rahulsharma11 I'll defer your question to the Arm guys. They must have a better idea.

That error is saying that input layer name given in the test does not match anything in the model file you provided. I believe the model file you need to use is this one:
http://download.tensorflow.org/models/object_detection/ssd_mobilenet_v1_quantized_300x300_coco14_sync_2018_07_18.tar.gz

I think, this update is related with the issue but no reviewer is interested in.
https://review.mlplatform.org/c/ml/armnn/+/2219

from comment I wrote on the review:
if convert tflite model in link below to armnn model, then layerBindingIds of ouput layer are 42752, 43008, 43264, 43520 but graph->outputIds are totally different with those values.
I didn't try but I think mismatch between inputIds,outputIds and bindingIds will cause inference malfunctioning.
http://storage.googleapis.com/download.tensorflow.org/models/tflite/coco_ssd_mobilenet_v1_1.0_quant_2018_06_29.zip
from: https://github.com/tensorflow/examples/tree/master/lite/examples/object_detection/android

But @Tee0125 that change is to do with armnnSerializer only, isn't it? @rahulsharma11 is not using armnnSerializer here. The issue is as described by @Burton2000: the test app expects a particular model, in this case http://download.tensorflow.org/models/object_detection/ssd_mobilenet_v1_quantized_300x300_coco14_sync_2018_07_18.tar.gz

Hi @rahulsharma11 ,
Did you manage to convert the TensorFlow model to TfLite using Richard's instructions above?
Cheers,
James

Closing due to inactivity. If you need more help please open another issue.


Can I specify a list of file extensions limiting selections in FileSelectionDialog?

I want to select from a directory only e,CSV .g. nd JSON files using FileSelectionDialog. How can I limit the display of only the files with specified extensions?

@AndriesSHP Unfortunately it actually not possible, but we can adjust it to make it possible. This period I am a bit busy, but it is a feature that I will implement in a near future. ;-)


feat: implement fuse-t feature

Use fuse-t to replace MacFUSE as the implementation of FUSE. fuse-t is based on NFS and does not require the use of kernel extensions. Installation can be done with lower privileges and it does not impact system stability.

There are a few warnings:
warning: unused import: `vmm_sys_util::tempdir::TempDir`
  --> tests/macfuse_smoke.rs:17:9
   |
17 |     use vmm_sys_util::tempdir::TempDir;
   |         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   |
   = note: `#[warn(unused_imports)]` on by default

warning: unused import: `crate::example::macfuse`
  --> tests/macfuse_smoke.rs:19:9
   |
19 |     use crate::example::macfuse;
   |         ^^^^^^^^^^^^^^^^^^^^^^^

warning: function `validate_hello_file` is never used
  --> tests/macfuse_smoke.rs:21:8
   |
21 |     fn validate_hello_file(dest: &str) -> bool {
   |        ^^^^^^^^^^^^^^^^^^^
   |
   = note: `#[warn(dead_code)]` on by default

warning: function `exec` is never used
  --> tests/macfuse_smoke.rs:38:8
   |
38 |     fn exec(cmd: &str) -> Result<String> {
   |        ^^^^

warning: `fuse-backend-rs` (test "macfuse_smoke") generated 4 warnings (run `cargo fix --test "macfuse_smoke"` to apply 2 suggestions)

Not sure why CI didn't catch it.
Otherwise lgtm!

tests/macfuse_smoke.rs this file only run fuse-t enabled, so has warning in fusedev test.
Add feature = fuse-t  works.


Moved username out of ROS2 section

@luis-camero Does this change how we read the username from the config?

@luis-camero Does this change how we read the username from the config?

No, this does not change anything downstream. It only changes where the ClearpathConfig looks for the username tag.


Value of "BANNED" on the Auto IP

Hey Y'all,
This just happened to me:

After some talks with bermello I'm posting this as an issue in order for us to use this functionality provided by GitHub.
Cheers,
Gus

This was caused due to lack of variable verification on incoming IPs requests. Already fixed on code and will be available on next commit. Thanks for reporting.


Where is the v1.4.5 tag?

Where can I find the v1.4.5 tag?
Are there release notes and/or a list of changes, somewhere?
Thanks in advance!

Also the source code seems to be missing yet, 1.x working branch still has 1.4.4

https://github.com/NancyFx/Nancy/commit/cbb94a811cd4b74139dc99f0a79d45814c7a4b42
It was just a direct commit to fix a bug.

I see but shouldn't SharedAssemblyInfo also be updated then?

Yeah, this was a manual build to fix a bug only in the nancy core dll, I probably didn't push the shared assembly info, or the tag, file after I'd manually created the nuget package.


Unify DB installation from tests and qrlew-datasets DB

Make datasets work in colab where no docker can run

Done


Test whether a PR from a maintainer is executed without label

Description

   if: (github.event_name == 'pull_request' && github.event.pull_request.head.repo.fork == 'false') || (github.event_name == 'pull_request_target' && contains(github.event.pull_request.labels.*.name, 'safe to test'))
Related Issues

https://github.com/recommenders-team/recommenders/issues/1853
References

Checklist:

[ ] I have followed the contribution guidelines and code style for this project.
[ ] I have added tests covering my contributions.
[ ] I have updated the documentation accordingly.
[ ] This PR is being made to staging branch and not to main branch.

it doesn't work o_o

If I add safe to test, it doesn't work either


Upgrade to Manifest V3

Manifest v3 improves security, privacy, and performance:
https://youtu.be/cqAO2xR7lzM?t=27
Migration guide:
https://developer.chrome.com/docs/extensions/mv3/intro/mv3-migration/

Upgraded to support manifest v3 (part of release version 4.0.0).


How to "run the included QuickconvectorV2.ahk"

Firstly there's a typo, it says 'QuickConvectorV2.ahk' instead of 'QuickConvertorV2.ahk'.
Secondly how do I run it?
It says on point 1 "Then run the included QuickConvectorV2.ahk file with AHK V2 Beta".
I've been using V1 for years and am a developer and have no idea how to run this. I installed and ran V2 and don't see a run script option.
Just a small explanation would be super helpful to non expert users.
Thanks.

Thanks, I'll fix the README soon

Download AHK v2.0 (It says beta but I've been using it with full release and it works fine)
Run QuickConvertorV2.ahk using AHK v2.0
On the left Textbox enter in your v1 script and press the green arrow
Run the v2 script, conversion may not be perfect so you'll need to manually fix that

This may sound obvious but I didn't realise you had to just double click the script to run it after installing AHK 2🤡
Please add that to the README 👍

README's been updated in #150, is it more clear now?

Closing this issue now as the changes have been made, If you feel otherwise please reopen

Yes looks great 👍


broken dependency on WebActivator

DataAnnotationsExtensions.MVC3 depends on WebActivator, which is deprecated.
also, after installation RegisterClientValidationExtensions.cs includes a reference to WebActivatorEx which is not installed.
the dependency needs to be updated to WebActivatorEx.

Good to know, I'll change the dependency to WebActivatorEx and see if it breaks/changes anything, and if not I'll push out a new version with that change.

@Spongman ok, the newest version https://www.nuget.org/packages/DataAnnotationsExtensions.MVC3/<IP_ADDRESS> now depends on WebActivatorEx (>= 2.1.0) which should work.  Thanks for pointing this out.


Original

New endpoints

GET /api/v1/statuses/:id/history
GET /api/v1/statuses/:id/source
PUT /api/v1/statuses/:id (update status)

Add EditedAt to Status struct.

Codecov Report
Base: 87.17% // Head: 87.63% // Increases project coverage by +0.45% :tada:

Coverage data is based on head (6a9400f) compared to base (9e1af56).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #159      +/-   ##
==========================================
+ Coverage   87.17%   87.63%   +0.45%
==========================================
  Files          15       15
  Lines         936      954      +18
==========================================
+ Hits          816      836      +20
+ Misses         75       74       -1
+ Partials       45       44       -1

Impacted Files
Coverage Δ

mastodon.go
73.79% <ø> (ø)

status.go
80.75% <100.00%> (+2.80%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

Thank you


Fix path for installation instructions

go-jira/jira isn't a proper path.

Whoops, yeah, moved code around a few weeks back. Thanks!
-Cory


UI Updates for SSO

Please update the UI in the article (and any other aspects of the documentation) to align with the current Power BI behavior. Here's the current UI:

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

ID: bf5c2cbb-f3de-c63e-fc74-a5157d5f5c9c
Version Independent ID: dd280f16-bad8-5f60-f960-208047f6f9b2
Content: Azure Synapse Analytics (formerly SQL Data Warehouse) with DirectQuery - Power BI
Content Source: powerbi-docs/connect-data/service-azure-sql-data-warehouse-with-direct-connect.md
Service: powerbi
Sub-service: pbi-data-sources
GitHub Login: @davidiseminger
Microsoft Alias: davidi

Thanks for your comment, @VitalyMCT. This issue has been assigned to the doc author for review.


Feat/3.17

WHAT

🤖 Generated by Copilot at c55932b
This pull request updates the LanguageServerProtocol library to implement the latest LSP specification. It adds new types, members, and request handlers to the Client.fs, Server.fs, and LanguageServerProtocol.fs files to support new features and capabilities for the client and server. It also updates the Benchmarks.fs and Shotgun.fs files to use the new LSP types and properties in the tests.

🤖 Generated by Copilot at c55932b

Oh we are the coders of the LSP
We update our types with the latest spec
We add new fields and requests to our code
And we pull and push on the git repo

🆕📝🔧

WHY
This PR is trying to add support for LSP 3.17. It should add all support except notebook part.
HOW

🤖 Generated by Copilot at c55932b

Add new abstract members to the ILspClient and ILspServer interfaces to support the latest LSP specification (link, link, link, link, link, link, link)
Add default and concrete implementations of the new abstract members to the LspClient and LspServer types, which inherit from the interfaces (link, link, link, link, link, link, link, link, link, link, link, link, link)
Add new entries to the defaultRequestHandlings function in the Server module in the LanguageServerProtocol.fs file to handle the new requests from the client to the server (link, link, link, link)
Modify the ClientCapabilities type in the MultipleTypesBenchmarks type in the Benchmarks.fs file to add more optional properties and new capabilities that reflect the latest LSP specification (link, link, link, link, link, link, link, link)
Add new fields to the DocumentSymbol type in the Gens module in the Shotgun.fs file to indicate additional information about the symbol (link)
Add a new line at the end of the Benchmarks.fs file (link)

@tcx4c70 anything left for this or should I merge it?

Hi @TheAngryByrd , I think the PR is ready to be merged. What do you think?

This is brilliant @tcx4c70 - thank you for taking the time to get us up to date!


Upgrade to lts-5.0

I updated the stack.yml file to the lts-5.0. The library parseerror-eq-<IP_ADDRESS> is included in this lts, so I think we can remove it from the extra-dependency list in the stack.yml file.

@CristhianMotoche thanks!


Exceptional work! 🤩

As a fellow security engineer who works on GitHub's open source IAM system (entitlements), this is really awesome work!

Thank you! Entitlements is also a great system and we definitely took a look at it for some inspiration when designing Access. 🎉

Right back at ya! Folks on my team are checking out this project as well for inspiration into the entitlements system and future improvements 🎉 🤜 🤛


Move directory prep into dispatcher abstraction

This PR relocates the 'directory prep' functionality from pkg/controllers/vdb/paths.go to the dispatcher abstraction. The 'directory prep' is only used by AT and not by vclusterops. The key benefit here is that it moves some ugly sudo calls into code that is specific to admintools deployments. This gets us a step closer to removing the requirement of sudo access for vclusterops deployments.

I got a strange error message: 'pods "v-enterprise-sanity-sc1-0" is forbidden.' @spilchen , do you have any insights into this?
kuttl-test-complete-koi verticadb-operator-controller-manager-ff7f545d4-k5xj6 manager 2023-09-21T01:38:54.884Z	ERROR	Reconciler error	{"controller": "verticadb", "controllerGroup": "vertica.com", "controllerKind": "VerticaDB", "VerticaDB": {"name":"v-enterprise-sanity","namespace":"kuttl-test-complete-koi"}, "namespace": "kuttl-test-complete-koi", "name": "v-enterprise-sanity", "reconcileID": "d9eca9cb-4e6c-4543-928d-29a0b64d1be8", "error": "could not execute: pods \"v-enterprise-sanity-sc1-0\" is forbidden: User \"system:serviceaccount:kuttl-test-complete-koi:verticadb-operator-controller-manager\" cannot create resource \"pods/exec\" in API group \"\" at the cluster scope"}

Can you try running your change where the create db has to retry? An easy way is to pick a communal path that isn't empty. Be sure you use different paths for depot. I was using the following manifest:
apiVersion: vertica.com/v1beta1
kind: VerticaDB
metadata:
  name: vc
  annotations:
    vertica.com/vcluster-ops: "true"
spec:
  image: vertica/vertica-k8s-private:latest-master
  imagePullPolicy: IfNotPresent
  communal:
    path: "/communal"
    includeUIDInPath: false
  initPolicy: CreateSkipPackageInstall
  kSafety: "1"
  local:
    dataPath: /data
    catalogPath: /catalog
    depotPath: /depot
  subclusters:
    - name: pri
      size: 3
  volumes:
  - name: hostpath
    hostPath:
      path: /host
  volumeMounts:
  - name: hostpath
    mountPath: /communal
    subPath: Linux64/Test/k8s

I saw this error in the logs:
2023-09-21T15:16:45.702Z    ERROR   Reconciler error    {"controller": "verticadb", "controllerGroup": "vertica.com", "controllerKind": "VerticaDB", "VerticaDB": {"name":"vc","namespace":"spilly"}, "namespace": "spilly", "name": "vc", "reconcileID": "98c96e37-745c-4dd5-af9d-2f55bcd33cc9", "error": "execute NMAPrepareDirectoriesOp failed, details: status code 500 returned from host <IP_ADDRESS>: catalog parent directory already exists: /catalog/vertdb

status code 500 returned from host <IP_ADDRESS>: catalog parent directory already exists: /catalog/vertdb

status code 500 returned from host <IP_ADDRESS>: catalog parent directory already exists: /catalog/vertdb
"}

Can you try running your change where the create db has to retry? An easy way is to pick a communal path that isn't empty. Be sure you use different paths for depot. I was using the following manifest:
apiVersion: vertica.com/v1beta1
kind: VerticaDB
metadata:
  name: vc
  annotations:
    vertica.com/vcluster-ops: "true"
spec:
  image: vertica/vertica-k8s-private:latest-master
  imagePullPolicy: IfNotPresent
  communal:
    path: "/communal"
    includeUIDInPath: false
  initPolicy: CreateSkipPackageInstall
  kSafety: "1"
  local:
    dataPath: /data
    catalogPath: /catalog
    depotPath: /depot
  subclusters:
    - name: pri
      size: 3
  volumes:
  - name: hostpath
    hostPath:
      path: /host
  volumeMounts:
  - name: hostpath
    mountPath: /communal
    subPath: Linux64/Test/k8s

I saw this error in the logs:
2023-09-21T15:16:45.702Z    ERROR   Reconciler error    {"controller": "verticadb", "controllerGroup": "vertica.com", "controllerKind": "VerticaDB", "VerticaDB": {"name":"vc","namespace":"spilly"}, "namespace": "spilly", "name": "vc", "reconcileID": "98c96e37-745c-4dd5-af9d-2f55bcd33cc9", "error": "execute NMAPrepareDirectoriesOp failed, details: status code 500 returned from host <IP_ADDRESS>: catalog parent directory already exists: /catalog/vertdb

status code 500 returned from host <IP_ADDRESS>: catalog parent directory already exists: /catalog/vertdb

status code 500 returned from host <IP_ADDRESS>: catalog parent directory already exists: /catalog/vertdb
"}

Wait! you are testing with vcluster ops.

Can you try running your change where the create db has to retry? An easy way is to pick a communal path that isn't empty. Be sure you use different paths for depot. I was using the following manifest:
apiVersion: vertica.com/v1beta1
kind: VerticaDB
metadata:
  name: vc
  annotations:
    vertica.com/vcluster-ops: "true"
spec:
  image: vertica/vertica-k8s-private:latest-master
  imagePullPolicy: IfNotPresent
  communal:
    path: "/communal"
    includeUIDInPath: false
  initPolicy: CreateSkipPackageInstall
  kSafety: "1"
  local:
    dataPath: /data
    catalogPath: /catalog
    depotPath: /depot
  subclusters:
    - name: pri
      size: 3
  volumes:
  - name: hostpath
    hostPath:
      path: /host
  volumeMounts:
  - name: hostpath
    mountPath: /communal
    subPath: Linux64/Test/k8s

I saw this error in the logs:
2023-09-21T15:16:45.702Z    ERROR   Reconciler error    {"controller": "verticadb", "controllerGroup": "vertica.com", "controllerKind": "VerticaDB", "VerticaDB": {"name":"vc","namespace":"spilly"}, "namespace": "spilly", "name": "vc", "reconcileID": "98c96e37-745c-4dd5-af9d-2f55bcd33cc9", "error": "execute NMAPrepareDirectoriesOp failed, details: status code 500 returned from host <IP_ADDRESS>: catalog parent directory already exists: /catalog/vertdb

status code 500 returned from host <IP_ADDRESS>: catalog parent directory already exists: /catalog/vertdb

status code 500 returned from host <IP_ADDRESS>: catalog parent directory already exists: /catalog/vertdb
"}

Wait! you are testing with vcluster ops.

Yeah, we want to be sure that any directory prep needed by the operator is done by the nma. The operator shouldn't be doing this prep manually.

Can you try running your change where the create db has to retry? An easy way is to pick a communal path that isn't empty. Be sure you use different paths for depot. I was using the following manifest:
apiVersion: vertica.com/v1beta1
kind: VerticaDB
metadata:
  name: vc
  annotations:
    vertica.com/vcluster-ops: "true"
spec:
  image: vertica/vertica-k8s-private:latest-master
  imagePullPolicy: IfNotPresent
  communal:
    path: "/communal"
    includeUIDInPath: false
  initPolicy: CreateSkipPackageInstall
  kSafety: "1"
  local:
    dataPath: /data
    catalogPath: /catalog
    depotPath: /depot
  subclusters:
    - name: pri
      size: 3
  volumes:
  - name: hostpath
    hostPath:
      path: /host
  volumeMounts:
  - name: hostpath
    mountPath: /communal
    subPath: Linux64/Test/k8s

I saw this error in the logs:
2023-09-21T15:16:45.702Z    ERROR   Reconciler error    {"controller": "verticadb", "controllerGroup": "vertica.com", "controllerKind": "VerticaDB", "VerticaDB": {"name":"vc","namespace":"spilly"}, "namespace": "spilly", "name": "vc", "reconcileID": "98c96e37-745c-4dd5-af9d-2f55bcd33cc9", "error": "execute NMAPrepareDirectoriesOp failed, details: status code 500 returned from host <IP_ADDRESS>: catalog parent directory already exists: /catalog/vertdb

status code 500 returned from host <IP_ADDRESS>: catalog parent directory already exists: /catalog/vertdb

status code 500 returned from host <IP_ADDRESS>: catalog parent directory already exists: /catalog/vertdb
"}

Wait! you are testing with vcluster ops.

Yeah, we want to be sure that any directory prep needed by the operator is done by the nma. The operator shouldn't be doing this prep manually.

So I mean the error is our expectation, because we don't do this prep manually.

So I mean the error is our expectation, because we don't do this prep manually.

Then we need new nma changes to do this prep then. The operator should not be responsible for that.

So I mean the error is our expectation, because we don't do this prep manually.

Then we need new nma changes to do this prep then. The operator should not be responsible for that.

Jira is created https://jira.verticacorp.com/jira/browse/VER-89413

@spilchen I just merge the latest vcluster repo, and I test with your manifest for create_db and it passed
INFO] [NMAPrepareDirectoriesOp] result from host <IP_ADDRESS> summary SUCCESS, details: {status:0 statusCode:200 host:<IP_ADDRESS> content:{"/catalog/vertdb/procedures":"created","/catalog/vertdb/v_vertdb_node0001_catalog":"created","/data/vertdb/v_vertdb_node0001_data":"created","/depot/vertdb/v_vertdb_node0001_depot":"created","/opt/vertica/config/logrotate":"created"} err:<nil>}
[INFO] [NMAPrepareDirectoriesOp] result from host <IP_ADDRESS> summary SUCCESS, details: {status:0 statusCode:200 host:<IP_ADDRESS> content:{"/catalog/vertdb/procedures":"created","/catalog/vertdb/v_vertdb_node0003_catalog":"created","/data/vertdb/v_vertdb_node0003_data":"created","/depot/vertdb/v_vertdb_node0003_depot":"created","/opt/vertica/config/logrotate":"created"} err:<nil>}
[INFO] [NMAPrepareDirectoriesOp] result from host <IP_ADDRESS> summary SUCCESS, details: {status:0 statusCode:200 host:<IP_ADDRESS> content:{"/catalog/vertdb/procedures":"created","/catalog/vertdb/v_vertdb_node0002_catalog":"created","/data/vertdb/v_vertdb_node0002_data":"created","/depot/vertdb/v_vertdb_node0002_depot":"created","/opt/vertica/config/logrotate":"created"} err:<nil>}
[INFO] [NMANetworkProfileOp] result from host <IP_ADDRESS> summary SUCCESS, details: {status:0 statusCode:200 host:<IP_ADDRESS> content:{"name":"eth0","address":"<IP_ADDRESS>","subnet":"<IP_ADDRESS>/24","netmask":"<IP_ADDRESS>","broadcast":"<IP_ADDRESS>"} err:<nil>}
[INFO] [NMANetworkProfileOp] result from host <IP_ADDRESS> summary SUCCESS, details: {status:0 statusCode:200 host:<IP_ADDRESS> content:{"name":"eth0","address":"<IP_ADDRESS>","subnet":"<IP_ADDRESS>/24","netmask":"<IP_ADDRESS>","broadcast":"<IP_ADDRESS>"} err:<nil>}
[INFO] [NMANetworkProfileOp] result from host <IP_ADDRESS> summary SUCCESS, details: {status:0 statusCode:200 host:<IP_ADDRESS> content:{"name":"eth0","address":"<IP_ADDRESS>","subnet":"<IP_ADDRESS>/24","netmask":"<IP_ADDRESS>","broadcast":"<IP_ADDRESS>"} err:<nil>}


Add an option to hide user from Directory search

Require https://github.com/matrix-org/matrix-android-sdk/pull/397, and the last version of the SDK
Fixes #385

Done


schema.lua support for type coercion

Summary
Currently AFAIK there is no option to coerce a given field value and then force schema to the coerced value (env variables are. strings for instance).
It would be great to support cases like that:
{
  name = "some-plugin",
  fields = {
    { config = {
        type = "record",
        fields = {
          { a_string_number = { type = "string",
                             transform = tonumber
                             required = true,
                             between = { 0, 1 } } },
        },
    }, },
  },
}

Hello @Asafb26,
Good question. The way field transformation is done in Kong is typically via transformers like:
https://github.com/Kong/kong/blob/next/kong/db/schema/entities/ca_certificates.lua#L17-L28
Does this answer your question?

Closing for now, feel free to reopen if you have more questions :)

Closing for now, feel free to reopen if you have more questions :)


Clean up controller code

Removed unnecessary machinery
Fixed misleading comments

This was split from #173

/assign @mattmoor

I can't wait for your PR to merge and then rebase.

s/can't/can

Rebase done.