Use content-tag so that we can work on Safari / iOS

Pending:

https://github.com/embroider-build/content-tag/issues/11
https://github.com/embroider-build/content-tag/pull/18

I've made a babel version of content-tag if that helps you.
https://github.com/patricklx/ember-template-tag

3.9KB is certainly more attractive than 741Kb:
https://bundlejs.com/?q=ember-template-tag&treeshake=[*]&config={"esbuild"%3A{"external"%3A["%40babel%2Fcore"%2C"%40babel%2Ftypes"%2C"%40babel%2Ftraverse"%2C"%40babel%2Fgenerator"%2C"%40glimmer%2Fsyntax"%2C"%40babel%2Fparser"]}}
Though, I already have all of babel in the repl (which is approaching 5 MB). I wonder if there there is any benefit to moving all of my infra to SWC for parsing / compiling via WASM instead of babel.
Right now, 3.9KB is fantastic. (Keeps me under 5 MB in transpilation assets).
I need to look in to switching to SWC though -- maybe it's doable -- would need to figure out if they support legacy decorators -- or wait until @ef4's work with supporting both legacy decorators and spec-decorators... and then just have less transpilation in general.

@patricklx it was not so easy :upside_down_face: I opened a couple issues on your repo, and opened this PR for ember-template-tag: https://github.com/NullVoxPopuli/limber/pull/1168


nixos-shell: init at 0.1.1

Motivation for this change
Based on popular demand. fixes https://github.com/Mic92/nixos-shell/issues/2
Things done

[ ] 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.

This still causes infinite recursion. Need to find out why.


make python fails on Debian 10 (python:3.8.12-slim-buster base image)

What version of OR-Tools and what language are you using?
Version: master
Language: Python
Which solver are you using (e.g. CP-SAT, Routing Solver, GLOP, BOP, Gurobi)
I am planning to use the CP-SAT solver in Python but got stuck already in the build process
What operating system (Linux, Windows, ...) and version?
I am trying to install or-tools in docker container based on the python:3.8.12-slim-buster image.
cat /etc/os-release outputs:
PRETTY_NAME="Debian GNU/Linux 10 (buster)"
NAME="Debian GNU/Linux"
VERSION_ID="10"
VERSION="10 (buster)"
VERSION_CODENAME=buster
ID=debian

However, I got this very same error when I tried to build or-tools on my virtualbox running with ubuntu 20.04 LTS and also in a container based on the latest alpine-edge image.
What did you do?
I try to build or-tools from source and use the official installation guide as a reference.
Steps to reproduce the behavior:

Strictly follow the instructions on the installation page of or-tools
Install all prerequisites
Run make third_party
run make python

Until and including step 3, installing all third party tools, everything runs without problems.
What did you expect to see
I expected a successful build. On my virtualbox running on ubuntu, I actually succeeded in the third or fourth attempt. However, it is hard for me to say why it finished successfully because I did small changes here and there of which I was not sure if they have an impact. For sure, I can say that I had to explicitly install the python extensions.
But for sure, for the apline-edge and the python slim-buster images, I also installed these python extensions.
What did you see instead?
The build process always exits at 96 % with the following error message:
make[3]: Entering directory '/or-tools/dependencies'
[ 96%] Building CXX object ortools/util/python/CMakeFiles/sorted_interval_list.dir/sorted_interval_list.cc.o
make[3]: Leaving directory '/or-tools/dependencies'
[ 96%] Generate Python 3 protocol buffer for ortools/constraint_solver/assignment.proto
make[3]: Entering directory '/or-tools/dependencies'
[ 96%] Building CXX object ortools/init/python/CMakeFiles/pywrapinit.dir/__/__/__/python/ortools/init/initPYTHON_wrap.cxx.o
protoc-gen-mypy: program not found or is not executable
Please specify a program using absolute path or make sure the program is available in your PATH system variable
--mypy_out: protoc-gen-mypy: Plugin failed with status code 1.
make[3]: *** [CMakeFiles/Pyortools_proto.dir/build.make:90: python/ortools/constraint_solver/assignment_pb2.py] Error 1
make[3]: Leaving directory '/or-tools/dependencies'
make[2]: *** [CMakeFiles/Makefile2:2105: CMakeFiles/Pyortools_proto.dir/all] Error 2
make[2]: *** Waiting for unfinished jobs....
[ 96%] Building CXX object CMakeFiles/flatzinc.dir/ortools/flatzinc/checker.cc.o
[ 96%] Building CXX object CMakeFiles/flatzinc.dir/ortools/flatzinc/cp_model_fz_solver.cc.o
In file included from /or-tools/dependencies/python/ortools/init/initPYTHON_wrap.cxx:3167:
/or-tools/ortools/base/python-swig.h: In function ‘bool PyObjAs(PyObject*, T*)’:
/or-tools/ortools/base/python-swig.h:59:1: warning: no return statement in function returning non-void [-Wreturn-type]
 }
 ^
/or-tools/ortools/base/python-swig.h: In function ‘PyObject* PyObjFrom(const T&)’:
/or-tools/ortools/base/python-swig.h:63:1: warning: no return statement in function returning non-void [-Wreturn-type]
 }
 ^
[ 96%] Linking CXX shared library ../../../lib/_pywrapinit.so
make[3]: Leaving directory '/or-tools/dependencies'
[ 96%] Built target pywrapinit
[ 96%] Building CXX object CMakeFiles/flatzinc.dir/ortools/flatzinc/model.cc.o
[ 96%] Building CXX object CMakeFiles/flatzinc.dir/ortools/flatzinc/parser.cc.o
[ 96%] Linking CXX shared module ../../../lib/sorted_interval_list.cpython-38-x86_64-linux-gnu.so
/or-tools/ortools/flatzinc/model.cc: In member function ‘bool operations_research::fz::Argument::IsArrayOfValues() const’:
/or-tools/ortools/flatzinc/model.cc:668:1: warning: control reaches end of non-void function [-Wreturn-type]
 }
 ^
[ 96%] Building CXX object CMakeFiles/flatzinc.dir/ortools/flatzinc/parser.tab.cc.o
[ 96%] Building CXX object CMakeFiles/flatzinc.dir/ortools/flatzinc/parser.yy.cc.o
make[3]: Leaving directory '/or-tools/dependencies'
[ 96%] Built target sorted_interval_list
[ 96%] Building CXX object CMakeFiles/flatzinc.dir/ortools/flatzinc/presolve.cc.o
[ 96%] Linking CXX shared library lib/libflatzinc.so
make[3]: Leaving directory '/or-tools/dependencies'
[ 96%] Built target flatzinc
make[2]: Leaving directory '/or-tools/dependencies'
make[1]: *** [Makefile:166: all] Error 2
make[1]: Leaving directory '/or-tools/dependencies'
make: *** [makefiles/Makefile.cpp.mk:45: lib/libortools.so.9] Error 2
root@406f1f5c37c1:/or-tools#

Let me know, if you need further information

Thanks. We need to to document that you need to install mypy-protobuf

Many thanks for the fast reply. Apparently, I have mypy-protobuf already installed. Does this makes sense to you?
root@406f1f5c37c1:/or-tools# which python
/usr/local/bin/python
root@406f1f5c37c1:/or-tools# python --version
Python 3.8.12
root@406f1f5c37c1:/or-tools# python -m pip install mypy-protobuf
Requirement already satisfied: mypy-protobuf in /root/.local/lib/python3.8/site-packages (3.2.0)
Requirement already satisfied: protobuf>=3.19.3 in /root/.local/lib/python3.8/site-packages (from mypy-protobuf) (3.19.4)
Requirement already satisfied: types-protobuf>=3.19.5 in /root/.local/lib/python3.8/site-packages (from mypy-protobuf) (3.19.12)

make sure that is script is accessible on the path
Laurent Perron | Operations Research | @.*** | (33) 1 42 68 53
00
Le ven. 25 févr. 2022 à 09:50, koljaLanger @.***> a
écrit :

Many thanks for the fast reply. Apparently, I have mypy-protobuf already
installed. Does this makes sense to you?
@.:/or-tools# which python
/usr/local/bin/python
@.:/or-tools# python --version
Python 3.8.12
@.***:/or-tools# python -m pip install mypy-protobuf
Requirement already satisfied: mypy-protobuf in /root/.local/lib/python3.8/site-packages (3.2.0)
Requirement already satisfied: protobuf>=3.19.3 in /root/.local/lib/python3.8/site-packages (from mypy-protobuf) (3.19.4)
Requirement already satisfied: types-protobuf>=3.19.5 in /root/.local/lib/python3.8/site-packages (from mypy-protobuf) (3.19.12)
—
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/issues/3145#issuecomment-1050655362,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACUPL3OM5LAXIIMCZKDKZ33U447FFANCNFSM5PJURRLQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you modified the open/close state.Message
ID: @.***>

@koljaLanger you should check using:
which protoc-gen-mypy

ref: https://github.com/nipunn1313/mypy-protobuf/blob/3444fed9ac123c2e0d71d13b440665784a7950fc/setup.cfg#L22-L24

Many thanks for your support! I added /root/.local/bin to the path and this solves all problems.
Sorry for the inconvenience


custom note 0.4.2

What is your question?
I added a custom note and the game ends if I die I just added it as a source

0.4.2 isn't supported anymore


Tutorial: tracing code

Here is a trace of the add and edit commands:

Thanks! Am closing this issue since no further follow-up is required


Permission denied to access object on Firefox

When I try to use post-robot with firefox I get this error:
Error: postrobot_message_ack postrobot_ready error:
  - postrobot_post_message: Permission denied to access object

Things works fine with Chrome.
Any idea how I could debug that?
Thanks

Here's where I use post-robot:
https://github.com/deskpro/apps-libs/blob/master/packages/apps-sdk/src/Widget/WidgetWindowBridge.js#L202

Not sure if this is related, but I've found that iframe for Cross-Domain localStorage require to have CORS enabled on Firefox, I have no idea why. I've written that in this article: Cross-Domain LocalStorage.

@jducro Have you tried enabling CORS as @jcubic has mentioned?  It doesn't surprise me that Firefox is requiring that.  Let me know if it works for.  If it doesn't, we can dig a bit more.

@mnicpt the project this was used in is deprecated, I can't test that anymore, I'll close the issue.


Consider removing EnableWindowsAuth flag on QS AccountOptions

I think it's now redundant with AspNetCore 2.

Agreed.

done


Filestream

This PR fixes file streaming so that file uploads and downloads are chunked.  Also added support for the get command to download files straight to disk.  Fixes #2 and #28.

Putting this up here for some CR.  I got frustrated trying to get the file streaming working with 2.6, but I will give it a try later this week.

Why did you re-add all the dynamic function constructors? I thought we were getting away from those?
https://github.com/ipfs/python-ipfs-api/pull/30/files#diff-33e409056abcd2472116a6860bf25cafR43

I didn't re-add them, I just made the client class methods wrap calls to those functions so that we could add helpful docstrings.  Its basically the same thing the decorators did, but I agree with you that decorators are too confusing.


Make hosted cluster syncing explanation generic

This change moves the syncing explanation out of the EKS section into
its own section, and rewords it to apply to both EKS and GKE.
https://github.com/rancher/docs/issues/3105

I see this is partly addressed in https://github.com/rancher/docs/pull/3210 I'll wait for that to merge before making any other changes

rebased


The settings panel alignment is off

Can you please post a screenshot!

In red in the screenshot. I don't understand how panel panel placement works in swing, I took took me way too much time to do this and it's still not aligned.
Also I'm unable to set a fixed width for the FileMaskPanel and IdeSupportPanel (the last two) so their size is defined by their content and it annoys me.


Some OG tags not found

Some OG tags are not found if the tags are in the body, not the header. Currently the code only checks doc.html.header. Will provide a fix to search all HTML (maybe this should be configurable?)

I think that it need fork.


Support Ubuntu 14.04+

Use update-motd in Ubuntu 14.04+
Also don't spit out empty roles in the motd

Two more thoughts on this:

Putting that file in place might be problematic if for some reason people want to use the resource twice
In ubuntu 14.04, /etc/motd behaves like /etc/motd.tail in previous versions (I haven't tested 16.04 yet, but I expect it's similar), so we might want to just change the filename rather than add the extra update-motd stuff.


Please provide a class summary table like MSDN had

Example:
https://docs.microsoft.com/en-us/uwp/api/Windows.UI.Xaml.Controls.Grid
All the inherited members are just a giant unformatted sea of hyperlinks.  And for everything implemented by the class it's a giant scrolling page trying to find anything.  Old MSDN had a nicely formatted summary table listing all of the cl***'s members, properties, and events, with hyperlinks and a brief description of the member.  It's incredibly difficult to now to get an "at a glance" view of what a class actually does.

⚠ Idea migrated from UserVoice
Created By: Anonymous
Created On: 2017/05/19 21:57:06 +0000
Votes at Migration: 18
Supporters at Migration: 18

@dend What is the status of this issue?

Isn’t this fixed now @dend?

@mairaw yes, this is fixed!

@dend @mairaw Is this something that needs to be configured?  How would I show a class summary table for a class such as Map?


n_jobs is not accepted as an argument of XGBClassifier

According to its documentation, xgboost has an n_jobs parameter. However, when I attempt to set n_jobs, I get this error:
TypeError: __init__() got an unexpected keyword argument 'n_jobs'

Same issue for some other parameters like random_state. I assumed this might be an update issue, but it seems I have the latest version (0.6a2, installed with pip).
Operating System: Linux
Package used (python/R/jvm/C++): python

The Pypi xgboost package is over a year old. Please install from the source.

still have the same issue on version 0.7.1 any fix for this?

n_jobs should be accepted by XGBClassifier, according to line 397 of sklearn.py, version 0.7.1. Make sure you have the correct version installed.


Build fails with these errors 'folly/folly-config.h' file not found and Could not build Objective-C module 'React'

How were you trying to build the app?
After running bun ios to build the app I am getting the errors(PLEASE see the full build logs section).
 To address this issue, either add output dependencies to the script phase, or configure it to run in every build by unchecking "Based on dependency analysis" in the script phase. (in target 'React-Fabric' from project 'Pods')
▸ Running script '[CP-User] [RN]Check rncore'
▸ Running script '[CP-User] Generate Specs'
▸ Compiling RCTVibration.mm
▸ Compiling RCTSettingsManager.mm
▸ Compiling RCTNetworking.mm
▸ Compiling RCTLinkingManager.mm
▸ Compiling RCTBlobManager.mm
▸ Compiling RCTNativeAnimatedTurboModule.mm
▸ Compiling RCTImageEditingManager.mm
▸ Compiling RCTNativeAnimatedModule.mm
▸ Compiling FBReactNativeSpec-generated.mm
▸ Compiling RCTAccessibilityManager.mm
▸ Compiling RCTImageStoreManager.mm
▸ Compiling RCTImageLoader.mm
▸ Compiling RCTFileReaderModule.mm
▸ Compiling RCTWebSocketModule.mm
▸ Compiling RCTTiming.mm
▸ Compiling RCTStatusBarManager.mm
▸ Compiling RCTSourceCode.mm
▸ Compiling RCTRedBox.mm
▸ Compiling RCTPlatform.mm
▸ Compiling RCTLogBox.mm
▸ Compiling RCTKeyboardObserver.mm
▸ Compiling RCTModulesConformingToProtocolsProvider.mm
▸ Compiling NitroModulesSpecJSI-generated.cpp
▸ Compiling NitroModulesSpec-generated.mm
▸ Compiling FBReactNativeSpecJSI-generated.cpp
▸ Compiling RCTI18nManager.mm
▸ Compiling RCTExceptionsManager.mm
▸ Compiling RCTDeviceInfo.mm
▸ Compiling RCTDevSettings.mm
▸ Compiling RCTDevMenu.mm
▸ Compiling RCTDevLoadingView.mm
▸ Compiling RCTClipboard.mm
▸ Compiling RCTAppearance.mm
▸ Compiling RCTAppState.mm
▸ Compiling RCTAlertManager.mm
▸ Compiling RCTActionSheetManager.mm
▸ Building library libReactCodegen.a
▸ Building library libReact-RCTVibration.a
▸ Building library libReact-RCTSettings.a
▸ Building library libReact-RCTNetwork.a
▸ Building library libReact-RCTLinking.a
▸ Building library libReact-RCTAnimation.a
▸ Running script '[CP-User] [RN]Check rncore'
▸ Compiling NativeMicrotasks.cpp
▸ Compiling NativeIdleCallbacks.cpp
▸ Compiling NativeReactNativeFeatureFlags.cpp
▸ Compiling NativeDOM.cpp
▸ Compiling RCTActivityIndicatorViewComponentView.mm
▸ Building library libReact-RCTImage.a
▸ Building library libReact-RCTBlob.a
▸ Compiling DefaultTurboModules.cpp
▸ Compiling RCTAppSetupUtils.mm
▸ Building library libReact-CoreModules.a
▸ Compiling RCTScrollViewComponentView.mm
▸ Compiling RCTSafeAreaViewComponentView.mm
▸ Compiling RCTPullToRefreshViewComponentView.mm
▸ Compiling RCTParagraphComponentView.mm
▸ Compiling RCTParagraphComponentAccessibilityProvider.mm
▸ Compiling RCTUnimplementedViewComponentView.mm
▸ Compiling RCTInputAccessoryComponentView.mm
▸ Compiling RCTThirdPartyFabricComponentsProvider.mm
▸ Compiling RCTTextInputComponentView.mm
▸ Compiling RCTSwitchComponentView.mm
▸ Compiling RCTFabricComponentsPlugins.mm
▸ Compiling RCTDebuggingOverlayComponentView.mm
▸ Compiling RCTComponentViewFactory.mm
▸ Building library libReact-RCTFabric.a
▸ Building library libReact-idlecallbacksnativemodule.a
▸ Building library libReact-microtasksnativemodule.a
▸ Building library libReact-featureflagsnativemodule.a
▸ Building library libReact-domnativemodule.a
▸ Running script 'Copy generated compatibility header'
▸ Building library libReact-defaultsnativemodule.a
▸ Building library libReact-RCTAppDelegate.a

⚠  /Users/pat/Documents/Projects/NitroInAppBrowser/ios/Pods/Headers/Public/React-Core/React/RCTSurfaceHostingView.h:63:11: parameter 'disabled:' not found in the function declaration [-Wdocumentation]

#import "React-Core-umbrella.h"
        ^

❌  /Users/pat/Documents/Projects/NitroInAppBrowser/ios/Pods/Headers/Public/RCT-Folly/folly/portability/Config.h:20:10: 'folly/folly-config.h' file not found

#include <folly/folly-config.h>
         ^

❌  /Users/pat/Documents/Projects/rn-browser/ios/HybridInAppBrowser.swift:2:8: could not build Objective-C module 'React'

import React
         ^

    Run script build phase 'Bundle React Native code and images' will be run during every build because it does not specify any outputs. To address this issue, either add output dependencies to the script phase, or configure it to run in every build by unchecking "Based on dependency analysis" in the script phase. (in target 'NitroInAppBrowser' from project 'NitroInAppBrowser')
    Run script build phase '[CP-User] [Hermes] Replace Hermes for the right configuration, if needed' will be run during every build because it does not specify any outputs. To address this issue, either add output dependencies to the script phase, or configure it to run in every build by unchecking "Based on dependency analysis" in the script phase. (in target 'hermes-engine' from project 'Pods')
    Run script build phase '[CP-User] [RN]Check rncore' will be run during every build because it does not specify any outputs. To address this issue, either add output dependencies to the script phase, or configure it to run in every build by unchecking "Based on dependency analysis" in the script phase. (in target 'React-FabricComponents' from project 'Pods')

info 💡 Tip: Make sure that you have set up your development environment correctly, by running npx react-native doctor. To read more about doctor command visit: https://github.com/react-native-community/cli/blob/main/packages/cli-doctor/README.md#doctor

node:internal/process/promises:288
            triggerUncaughtException(err, true /* fromPromise */);
            ^

[UnhandledPromiseRejection: This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). The promise rejected with the reason "Error: Failed to build ios project. "xcodebuild" exited with error code '65'. To debug build logs further, consider building your app with Xcode.app, by opening 'NitroInAppBrowser.xcworkspace'.".] {
  code: 'ERR_UNHANDLED_REJECTION'
}

Node.js v18.20.2

Project dependencies
"devDependencies": {
    "@react-native/eslint-config": "^0.75.2",
    "@types/jest": "^29.5.12",
    "@types/react": "^18.3.4",
    "eslint": "^8.57.0",
    "eslint-config-prettier": "^9.1.0",
    "eslint-plugin-prettier": "^5.2.1",
    "nitro-codegen": "^0.11.0",
    "prettier": "^3.3.3",
    "react": "^18.3.1",
    "react-native": "^0.75.2",
    "react-native-nitro-modules": "0.11.0",
    "typescript": "^5.5.4"
  }

Nitro Modules Version
0.11.0
Nitrogen Version
0.11.0
Target platforms
iOS
Operating system
MacOS
Can you build the Nitro Modules Example app?
Yes, I can successfully build the Example app here
Additional information

[ ] I am using Expo
[X] I am using Nitrogen (nitro-codegen)
[X] I searched for similar issues in this repository and found none.

Hm, this doesn't look like a Nitro error. If you can build the example app here, maybe there's something wrong with the app you created.
Does it build if you uninstall nitro?

Hm, this doesn't look like a Nitro error. If you can build the example app here, maybe there's something wrong with the app you created.
Does it build if you uninstall nitro?

Hi @mrousavy, yes it does build once I uninstall nitro.
I managed to resolve this build error with the following change in the .podspec file
require "json"

package = JSON.parse(File.read(File.join(__dir__, "package.json")))

Pod::Spec.new do |s|
  s.name         = "<<moduleName>>"
  s.version      = package["version"]
  s.summary      = package["description"]
  s.homepage     = package["homepage"]
  s.license      = package["license"]
  s.authors      = package["author"]

  s.platforms    = { :ios => min_ios_version_supported }
  s.source       = { :git => "https://github.com/mrousavy/nitro.git", :tag => "#{s.version}" }

  s.source_files = [
    # Implementation (Swift)
    "ios/**/*.{swift}",
    # Autolinking/Registration (Objective-C++)
    "ios/**/*.{m,mm}",
    # Implementation (C++ objects)
    "cpp/**/*.{hpp,cpp}",
  ]

  s.pod_target_xcconfig = {
    "GCC_PREPROCESSOR_DEFINITIONS" => "$(inherited) FOLLY_NO_CONFIG FOLLY_CFG_NO_COROUTINES"
  }

  load 'nitrogen/generated/ios/<<moduleName>>+autolinking.rb'
  add_nitrogen_files(s)

  if respond_to?(:install_modules_dependencies, true)
      install_modules_dependencies(s)
  else
    s.dependency "React-Core"
  end
end

Do you think https://github.com/mrousavy/nitro/pull/235 maybe fixed this as well?


Improve entity linking

Updating the entity linking code with the new and improved TFIDF/BERT based entity linking. The old version used functions from spacy and had stopped being able to be run due to needing an older version of spacy than is currently in wellcomeml.
The code in entity_linking.py was previously in the datalabs repo here, but I am moving it over to live here. See PR https://github.com/wellcometrust/datalabs/pull/684/files for the work of removing it from datalabs. The only difference is that the class was called TfidfLinker in datalabs but here it is called EntityLinker, which is more suitable anyway since it has the ability to use BERT embeddings instead of TFIDF.
Note:
Because EntityLinker imports BertVectorizer the order of importing these 2 in the init file is important (BertVectorizer before EntityLinker).

A small limitation of the current approach is that it does not learn much, i.e. it basically fits TFIDF or uses BERT to embed. This is not necessarily a problem but in the future you can look into what Antonio built for reranking, basically given some training data you can rerank the retrieved entities using a neural model or other model.

Another comment is that you should consider whether you want to treat this problem as a 2 step problem as it was before. Step one fitting a knowledge base, in which the embedding would happen, either from scratch with TFIDF or some a neural embedding layer or with some pre trainined embeddings like frozen bert. Step two fitting an entity linker, in which you get a text with entities and correct entities and then you train to differentiate which entity is from the knowledge base (also an input, possibly in the init). At the moment you are doing both steps in one. This might get slighly confusing as your entity linker behaves a bit more like a knowledge base, i.e. you need to pass information about entities into fit in order to get predictions in predict. But it could be that you believe that this is more intuitive and on purpose. In order to decide you might want to think what happens in a future point where you get annotated entities with links, how do you use them? currently you need to find information on the entities and pass them to fit and then use the annotations to evaluate (but not train).

@nsorros I do agree that it might be good to do the knowledge base curation separately, it does feel like this class is trying to do 2 separate things.
However, the purpose of this PR was really just to move the code from datalabs to wellcomeml (on my to do list for ages!), I don't intend to work on entity linking for a while. So if the code is ok I would prefer to get it merged as is, and then when I readdress the entity linking (which could be in a few months) I will probably want to restructure things then anyway.

@nsorros I do agree that it might be good to do the knowledge base curation separately, it does feel like this class is trying to do 2 separate things.
However, the purpose of this PR was really just to move the code from datalabs to wellcomeml (on my to do list for ages!), I don't intend to work on entity linking for a while. So if the code is ok I would prefer to get it merged as is, and then when I readdress the entity linking (which could be in a few months) I will probably want to restructure things then anyway.

Fair enough, do you think it might be worth keeping the code for SpacyEntintyLinking and SpacyKnowledgeBase and name this somehow else? For example SimilarityEntityLinking

@nsorros yep good idea, i've made the name changes and added back in the spacy scripts


Add .NET 8 support

Change target framework in projects from .NET 6 to .NET 8 or add it as second target framework.

@NikiforovAll What do you think, would it be ok to change target framework of all projects to .NET8 and update nuget packaegs to latest stable version?

This is great idea

Please feel free to update sample apps and dependencies.
Ref: https://github.com/NikiforovAll/keycloak-authorization-services-dotnet/pull/148

@NikiforovAll sorry, my bad. I've checked Visual Studio nuget search and for some reason it do not show references to .NET8:

But in nuget.org reference exists. Will update sample apps.


⚠ ClamAV has degraded performance

In c1c001f, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 80 ms

Resolved: ClamAV performance has improved in 8d92371.


#86 material3 date picker

Good morning,
Added the Material3 DatePicker as per issue 86, please let me know if you are happy or if I should change anything?

Hey @MJJacobs01!
It's important merge your branch with main branch of project. I have made some changes on Dialog (#85) and your changes conflicts with this!


Re-export wgpu_types or all types used in Bevy APIs

What problem does this solve or what need does it fill?
Certain Bevy APIs (such as RenderDevice::poll and other low-level graphics APIs) use types from wgpu_types which are not re-exported by Bevy. Using these methods requires linking the same version of wgpu_types as Bevy, which I feel is an unnecessary annoyance for a modular game engine like Bevy.
What solution would you like?
All types which appear as arguments or public fields in any public Bevy API should be publicly re-exported - aka, I should be able to call any Bevy method or construct any public type without ever needing to include a specific version of another library.
To accomplish this, we should audit the types used in public APIs and re-export those in an organized way, such as in a new render_resource::raw or render_resource::wgpu module.
What alternative(s) have you considered?
Re-export wgpu_types wholesale. It's easier, but obviates a perhaps more thoughtful organization of only those wgpu_types types which are actually necessary for Bevy.
Additional context
Types like these also provide bad user experience in docs.rs, being types which you can't click to visit docs for. Perhaps there's also a way to use rustdoc to easily list the problem APIs?

Having thought about this a little, I think exporting whole crates is the better and more conservative plan. We can always provide additional organization later, but trying to carve out exactly what part of wgpu-types someone might want to use seems pointless. The primary goal should be to simplify dependency management when writing a game with Bevy, which is solved just by a simple re-export.


docs: udpate the link of pointcloud_divider

Description
udpate the link of pointcloud_divider
related PR
https://github.com/autowarefoundation/autoware.universe/pull/8823

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[ ] I've confirmed the contribution guidelines.
[ ] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The Reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

@YamatoAndo This looks OK to me but is there any way that I can view this PR as Autoware Documentation (like this photo)? I want to make sure that the new URL can be accessed from the web page of the documentation.

@anhnv3991 please look ↑ URL


Can´t add Mr. Mime to the list using /add

Can´t add Mr. Mime to the list using /add

This is a known bug because the pokemon list is space separated, and Mr. Mime breaks everything.
You can use /add 122 to add by pokedex number as a workaround.


Event Synchronization using Negentropy

Implement https://github.com/hoytech/strfry/blob/next/docs/negentropy.md

Use https://github.com/hoytech/negentropy#javascript


Chartero version 2.9.3 in Zotero 7.0 64-bit cannot import legacy history or import pasted history

Error information of the Chartero 2.9.3 when using the buttons Automatically search and import legacy history and Import pasted history.

I have tried other earlier versions, but get the same error (cannot open Chartero settings) as described in https://github.com/volatile-static/Chartero/issues/116.
Thanks for your time.

#123


Update CI in if-plugins and if-unofficial-plugins to match if

What
Replicate this in if-plugins and if-unofficial-plugins
Why
These changes to the CI make it easier to create a new release because they remove some fiddly manual steps and enable the creation of beta releases for testing. This will make the releases faster and less error prone.
Context
We have experienced our release process being slow and quite frustrating, especially because it has to be repeated across multiple IF repositories. It is quite brittle, and we don't currently know that a release was successful until after it is already published and this has led to some late-night rapid re-releasing on two occasions. These CI changes make the process more robust and easy to test, providing the core team with a smoother process and the community with a more reliable release schedule.
Prerequisites/resources
none
SoW (scope of work)

[ ] In .github/workflows, nodejs-ci.yml, npm-publish.yml, release-commit.yml and release-publish.yml should be created/upodated to match the equivalent files in if.  In the top level directory, .commitlintrc.js, .release-it./json and package.json should be created/updated to match their equivalents in if.

Acceptance criteria

[ ] Given (Setup): a core team member creates a tagged release in the Github UI
When (Action): The CI triggers an automated relkease commitand updates the version numbers and poushes directly to main
Then: no additional manual steps are required before main can merge into release

[ ]  Given a core team member merges main into release
When the CI triggers a release to be created automatically on npm
Then no further manual actions are required in order to download and test the latest release

[ ] Given a core team member wants to create a test release
When they include beta in the release name
Then an interim beta release is created that can be used for testing. The subsequent release has the same version numbers as the beta release but no beta tag.

@manushak please review the PRs

@manushak please review the PRs

done


We don't need placeholder edit permissions checking in view mode

Currently we calculated edit permissions every time a placeholder is rendered, but that does not make any sense if we're not in edit mode.
Probably we can avoid putting placeholders in the request in that case, but this needs a few more changes, that need to be investigated
Thanks @vad for spotting this

@divio/django-cms-core any comment?

Changes Unknown when pulling 230bf5cf0078d66d15830a981afcfd2f86e755dc on yakky:feature/no_perms_check_in_view_mode into ** on divio:release/3.3.x**.

Well, we were effectively checking perms twice, which is no good. So, nice one!

Changes Unknown when pulling 76981caa24b72a0453614b2c3aa9a4c2e8903761 on yakky:feature/no_perms_check_in_view_mode into ** on divio:release/3.3.x**.

Minor comment. Looks good, thanks :)

Ok to merge.

Changes Unknown when pulling 3d8e764f6d93bc3d1395ed3b15d7ae191346a6d7 on yakky:feature/no_perms_check_in_view_mode into ** on divio:release/3.3.x**.

Coverage increased (+0.6%) to 89.508% when pulling 24aef5f2988f7a3957a5c7244b929ba2bd408b97 on yakky:feature/no_perms_check_in_view_mode into d65517e0229f53979280d0229046fdca8fe6bcb0 on divio:release/3.3.x.

@mkoistinen @czpython rebased

@mkoistinen @czpython any more fixes needed to merge  this?

@yakky Nope. Go for it.

@czpython :+1:


tinytex with linux alpine

Can you help me to install tinytex into a alpine linux?

I'm afraid I can't. I have no experience with Alpine Linux. Sorry.


README Update

Screen captures and reports verified.

Feature branch was merged, pulled, and verified successfully.

README file was updated, checked and approved by team.


🎨 Expect auths [][]string in validate.Validate, not [][][]byte

Summary
Converting from [][][]byte to [][]string simplifies encryption & decryption in validatorctl.
Acceptance Criteria

validate.Validate signature refactored
validatorctl refactored accordingly, HTTPFileAuthBytes deleted

It was changed to map[string][][]byte and can just stay that way.


Refactor to update Kotlin Coroutines

This PR should:

[x] Update gradle dependencies
[ ] Use CoroutineScope instead of GlobalScope on ViewModels
[ ] Fix #11

We should consider upgrading to Retrofit 2.6.0 before merging this.

Closing this, since I'm starting the 2020 challenge.


Can future versions of cargo break our digests?

cargo does some stuff to the original Cargo.toml and directory where it downloads the crate. If anything about it changes in the future, digest that we've calculated could change, and all the existing Project Review Proofs would stop working.
Can we do anything about it? Will the cargo team agree to keep the download directory immutable? https://github.com/rust-lang/cargo/issues/6340
Right now when calculating the checksum cargo-crev will skip the .cargo-ok file.
I wish cargo would just leave the whole directory alone, and any additional files or modifications happened in another place, letting the crate source stay immutable.

It looks that except for .cargo-ok (which we ignore-list), packages are immutable.


Subscription Sign In Issue

In the section "Create blank logic app": After opening selecting the context menu item 'Open With Logic App Designer' the Logic App Properties box, where you enter in the subscription information, is disabled. There is no error message and I have tried two separate Azure accounts.

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

ID: a4e4c182-ccd7-0ee5-a993-9f4b4febf8d5
Version Independent ID: a02db181-a2e8-2a13-5932-d93dadacbae9
Content: Create logic apps that automate workflows with Visual Studio - Azure Logic Apps
Content Source: articles/logic-apps/quickstart-create-logic-apps-with-visual-studio.md
Service: logic-apps
GitHub Login: @ecfan
Microsoft Alias: estfan

Hi @aaronralls, thanks for the feedback! We are currently investigating and will update you shortly.

@aaronralls What role you have in your subscription ?
To check the role in your subscription browse to https://portal.azure.com --> All Services --> Subscriptions --> Select your Subscription --> Access Control (IAM) --> Role Assignments
Try giving Owner Role to yourself and check it in Logic App Properties box again after restarting your Visual Studio Instance.

@aaronralls I am not able to reproduce the issue at my side.
Can you confirm that you are logged in with the correct account in Visual Studio.
If yes, try removing all the accounts from Visual Studio and LogIn again.

If the issue still persist than it will require some deeper analysis as this is specific to your environment  , I would recommend you to raise a support request. If you don't have access to a support plan, please reach out to azcommunity at microsoft dot com with a link to this issue as well as subscription ID and we can help you to get one time free support for this issue.

@aaronralls Any update on this ?


Update about checking comment length with non-ASCII characters

Coverage remained the same at 51.126% when pulling 9e73abf8f6b92c7192bb63cc831a89d163069af8 on wknet123:dev into ead0985d20e3deff277f2d8084602bf47e38125a on vmware:dev.


Cannot resolve keyword 'title' into field issue

Using Q function, I do simple searching and without parler (single language, normal django model) all work fine. But with parler, 'title & summary' fields are not resolved. What should I look into?
Below is what I used in vews.py
    def get_queryset(self):
        result = super(ArticleListView, self).get_queryset()

        query = self.request.GET.get('q')

        if query:
            query_list = query.split()
            result = result.filter(
                reduce(operator.and_,
                       (Q(title__icontains=q) for q in query_list)) |
                reduce(operator.and_,
                       (Q(summary__icontains=q) for q in query_list))
            )
        return result

@sunjoomoon when using normal queryset methods/operators the translations__field syntax must be used, to allow Django ORM to traverse the join

@sunjoomoon @yakky , you can also use MyModel.objects.translated(name="..", title="..") which handles the translations__ prefix on the fly for you.
Note that parler stores the translated fields in a separate table. We don't try to mask that fact as it would increase complexity. Instead, your application will have to deal with this fact when performing queries.

Thank you for the guide, learnt that now.  All multilanguage simple search works just great now.


Feature - Lambda Delete by arn

Existing tests only tested whether Lambdas could be deleted when passing in a name.
According to the docs, Lambdas can also be deleted by passing in an ARN: https://boto3.amazonaws.com/v1/documentation/api/latest/reference/services/lambda.html#Lambda.Client.delete_function
The list of mock-URLs had to be adjusted to account for the fact we're passing a HTML-safe ARN in the URL, which can contain ampersands

Coverage decreased (-0.01%) to 93.523% when pulling ce4bd5d64c2caf734069339928e5d6c075e5b90c on bblommers:feature/lambda-delete-by-arn into b433399cb582cb5084c88e8d70ba08e9395c1d07 on spulec:master.

Looks like a test failed that's unrelated to my changes (test_kms.test_generate_random(1, {})) - will look into why/how that happened

I re-ran the failing test and is passed. Looks like we've got a flaky test in KMS. That's unrelated to this PR so I'll merge it. Nice work on this!


refactor: use artifact instead of artefact everywhere

rg -l artefact | xargs -I {} sed -i 's/artefact/artifact/g' {}
rg -l Artefact | xargs -I {} sed -i 's/Artefact/Artifact/g' {}

find . -type d -name '*artefact*' | grep -v '.git' | xargs -I {} rename artefact artifact {}
find . -name '*artefact*' | grep -v '.git' | xargs -I {} rename artefact artifact {}

Related to: https://github.com/open-component-model/ocm/issues/183

Have you added/registered the appropriate additional legacy types for type constants used in existing component versions?

I'm not sure I understand that. Add/register where?

Copying content from Slack so it's tracked and others can see it when doing review or looking back in the future.

There are type constants, like the artifact type ociArtefact which are placed in component descriptors, which are already in some repositories. If such values are just changed, all the existing CVs are corrupted. Therefore we have to

introduce new legacy type values for those cases and register them together with the new ones with the appropriate registration methods.
Code, which checks an value against such a constant must be adapted to accept both, the old and the new ones. Best, use an appropriate IsXYZ function defined at the place where the constants are defied (instead of checking against the constants directly)
There are for sure handlers registered using such constants. They must be registered for the old ones, also. May be, we could use the same mechanism, providing a function returning the values for registration (like TypeSet() []string, then further changed would be easier.

Source: Slack (@mandelsoft)

Can't we ignore as we did not have official release and in that "beta"/"alpha" stage, breaking changes are expected, we don't really have much components yet, and for those we would pollute the system with "legacy" code which sounds extremely  weird on a system that's did not reach 1.0.0 yet (currently v0.1.0-alpha.1).


Dependency build instructions: Cryptopp

Cryptopp uses makefile. Should a link to https://github.com/noloader/cryptopp-cmake also be added to the readme?

Ok, cmakelists file is available in release 5.6.5 of Cryptopp, though may want to update to latest release versions which no longer have a cmake lists file by default: https://www.cryptopp.com/wiki/CMake

Indeed the support dor cmake in ceyptopp was recently dropped. Last time I checked there had not been a release since though.


Fixed add to cart button is not coming completely on compare page #4086

BUGS:
Link: https://github.com/bagisto/bagisto/issues/4086

Done


Calling feed.trips fails

I am trying to analyse the german GTFS feed from the following website: https://gtfs.de/en/feeds/
Whenever I run the following code I get an ValueError. The error occurs when patterns are being computed. I have tried other GTFS feeds as well and still get the same ValueError. How can I fix this issue?
feed = Feed("data/germany-gtfs.zip")
feed.trips

Output:
INFO:root:computing patterns

ValueError: Cannot set a DataFrame with multiple columns to the single column pattern_id

hi @johanngrobe my guess is that there's something strange in this GTFS file. The latest version should give you better error messages as of why this is happening. Can you try updating the package and trying to run it again?
You can also try Feed(gtfs_path, patterns=False) if you are not particularly interested in the patterns.

Closing issue. You can re-open with the latest info if needed


⚠ Bofrost.es has degraded performance

In 5060853, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1469 ms

Resolved: Bofrost.es performance has improved in d297020 after 8 minutes.


Missing IP in Office sharing functionallity because of missing IP <IP_ADDRESS>

Hey, for being able to share e.g. a word file the IP <IP_ADDRESS> is required to be opened on the firewall/proxy. This IP is not mentioned in the lists for Office365 nor Azure. If the IP is not reachable the error message "Unfortunately, something is preventing us from sharing right now" appears. As soon as the IP is configured on the firewall the function is working as expected. We tried to address the issue of this missing IP in the Office365-IP range list or Azure IP range list by opening a Microsoft case, but were denied by the support. They cannot address this IP address to the corresponding teams that the IP is added to the list.
So I try it this way, in the hope that everyone gets something out of it.
Cheers

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

ID: 077406d8-bff1-496e-3ea2-b38558fd9355
Version Independent ID: 57db3dec-2250-2dee-fd1a-ec6656911798
Content: Office 365 URLs and IP address ranges
Content Source: Enterprise/urls-and-ip-address-ranges.md
Service: o365-administration
GitHub Login: @JoeDavies-MSFT
Microsoft Alias: josephd

@TorstenLaier - thanks for the feedback, I'll forward this to the Enterprise content team for topic update consideration.

@TorstenLaier I have forwarded this issue to the product team that owns these endpoints for their investigation. Thanks for letting us know. Closing this issue for now.


What is missing for a Mac app ?

Just see the title of the issue. ;-)

In theory, it should out of the box since Qt is supposed to be cross platform. In practice though... Can't say for sure. I don't have any Mac to test on and not enough hard disk space for a VM (will have soon) so I don't know if it works. I guess the release packaging scripts should be updated to build .app files and maybe the file I/O functions to handle Mac paths and dialog boxes.
Will update this issue asap.

Ok.

@projetmbc Test build on OS X: https://mega.nz/#!KN52FCDb!Nk-xnU2VwJkPITHen-G6_Kr7ScarF-HeVuK7Wb_c66A

Thanks for the "building". It seems to work.


Basic Arm Functionality

Will not close this until basic moveit trajectory execution (movegroups) are integrated onto our robotic arm.


Use shared RouteCollection among nested routes and apply prefix to route...

Use shared RouteCollection among nested routes and apply prefix to route path before generating unique route name
Fixes #1106

This fix seems good, why hasn't it been merged yet? The current route name generation is not working.. just tested it.

Closing in favor of #1205 where I've cherry-picked your changes and simplified the patch and avoided to expose an internal state.


Error after "Configure UI"

Describe the bug:
Error after exiting Configure UI. Requires reload to fix.
Config YAML:

HA and/or Browser Error:
Uncaught TypeError: Cannot read property 'querySelector' of null
    at styleButtons

HA, Card, and Browser Versions:

I am still facing this issue after update to 1.2.0
Uncaught TypeError: Cannot read property 'querySelector' of null


block handler filters not working: handleBlockWithCallToContract

Do you want to request a feature or report a bug?
bug
What is the current behavior?
blockHandlers:
- handler: handleBlock
- handler: handleBlockWithCallToContract
filter:
kind: call
handleBlockWithCallToContract never got called
If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem.
just deploy subgraph with block handlers
      blockHandlers:
        - handler: handleBlock
        - handler: handleBlockWithCallToContract
          filter:
            kind: call

What is the expected behavior?
handleBlock get called for all blocks while handleBlockWithCallToContract only get called when the block contains txs to the contract.

Hi! Looking at this - just to verify, the problem here is that the second blockhandler isn't run, rather than that blockhandlers with call filters are not working?

I'm experiencing the same problem with only a single block handler defined. When the block handler definition includes the call filter it appears to never be run.
Here is the relevant snippet from my subgraph.yaml file.
      blockHandlers:
        - handler: handleBlock
          filter:
            kind: call

Removing the call filter will cause the handler to be run. Adding the call filter again will cause the handler to not be run.
I would say this means blockhandlers with call filters are not working.

Same here for me. The docs don't describe supported filters. And the schema manifest seems outdated https://github.com/graphprotocol/graph-node/blob/master/docs/subgraph-manifest.md#1524-blockhandler

Seems we are also experiencing this with only one block handler defined.
Removed the filter for now, but triggering for every block makes syncing a magnitude slower.
Looking forward for the fix!

I've hunt down this one and right now, my investigation leads me to a problem with the extend
On a data source with call handlers, after the extend call, the original one is not modified.
I'll make some unit tests and share a PR when ready.


Issue with mobile Chrome's 'Reduce data usage'

Chrome's 'Reduce data usage' option breaks the CSS of bootstrap-social. Chrome is removing a space from .btn-social-icon :first-child selector, checkout this SO issue.
I've fixed it by adding a child selector everywhere where :first-child is used, it became
> :first-child

Maybe it would make sense to make this change in the main lib itself? I think, it would make bootstrap-social more stable.

But I have it like that.. no?
https://github.com/lipis/bootstrap-social/blob/gh-pages/bootstrap-social.less#L22

Oh I see.. in the generated CSS the space is gone..

Yes, Chrome modifies the generated CSS in a wrong way.

I have the same issue.

"Reduced" css:

@snovity can you post the changes you did? Thx

I made a pull request with my changes


test: tests for component FaveStar

SUMMARY

moving FaveStar.tsx to FaveStar folder
switching FaveStar to function component
tests for FaveStar component

TEST PLAN
No change in behavior. All tests must pass.

Codecov Report

Merging #13320 (5bf1b42) into master (29d6420) will decrease coverage by 19.06%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           master   #13320       +/-   ##
===========================================
- Coverage   77.20%   58.14%   -19.07%
===========================================
  Files         872      480      -392
  Lines       45101    16004    -29097
  Branches     5435     4130     -1305
===========================================
- Hits        34820     9305    -25515
+ Misses      10158     6699     -3459
+ Partials      123        0      -123

Flag
Coverage Δ

cypress
58.14% <100.00%> (-0.40%)
:arrow_down:

javascript
?

python
?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...uperset-frontend/src/components/FaveStar/index.tsx
100.00% <100.00%> (ø)

...uperset-frontend/src/dashboard/util/dnd-reorder.js
0.00% <0.00%> (-100.00%)
:arrow_down:

...rset-frontend/src/dashboard/util/getEmptyLayout.js
0.00% <0.00%> (-100.00%)
:arrow_down:

...dashboard/components/resizable/ResizableHandle.jsx
0.00% <0.00%> (-100.00%)
:arrow_down:

...tersConfigModal/Footer/CancelConfirmationAlert.tsx
0.00% <0.00%> (-100.00%)
:arrow_down:

...ConfigModal/FiltersConfigForm/FilterScope/state.ts
0.00% <0.00%> (-100.00%)
:arrow_down:

...odal/FiltersConfigForm/FilterScope/ScopingTree.tsx
6.25% <0.00%> (-93.75%)
:arrow_down:

.../src/dashboard/util/getFilterScopeFromNodesTree.js
0.00% <0.00%> (-93.48%)
:arrow_down:

...src/dashboard/components/gridComponents/Header.jsx
10.52% <0.00%> (-86.85%)
:arrow_down:

superset-frontend/src/components/IconTooltip.tsx
13.33% <0.00%> (-86.67%)
:arrow_down:

... and 735 more

Continue to review full report at Codecov.

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


Controling signs of context variables in Prophet

Project Robyn
Describe issue
Hi everyone,
I have a question about controlling the sign of context variables. We do have context_signs, which control it in the model itself. However, when a context_variable is also a factor variable, prophet uses them as additional information. Then, in the prophet decomposition we can see how the effect from this variable looks.
Provide reproducible example
I have two factor variables, which are simply 1 in case of event, 0 when there is no event. For factor 1 I expect to have a positive effect on revenue, while for factor 2 I expect a negative effect. Inside robyn_inputs I specify:
context_vars = c("other_variable","factor_1",'factor_2'),
context_signs=c('positive','positive','negative'),
This works well in the model itself - in the waterfall chart I have the contribution of factor 2 as negative.
However, this is what the Prophet decomposition for factor 2 looks like - despite specifying the context_sign as negative, Prophet attributes positive effect to it.
c
My question is: is there a way to specify the sign, so that Prophet would always estimate the effect of a variable as negative? Also, do I understand correctly that these context variables are used in both Robyn and Prophet as predictors?
Environment & Robyn version
Make sure you're using the latest Robyn version before you post an issue.

Robyn 3.9.0.
R version 4.2.2

Check ?robyn_inputs. You can specify prophet_signs to be negative. No context vars are not used in prophet as predictors.

I tried using these, but these only work for trend, holiday and season. If I try specifying my factor variable in there, it gives:
'prophet_signs' must have same length as 'prophet_vars'
And in prophet_vars I can't specify anything else, except for holiday, season etc.
Am I missing something or does it indeed not work for factor variables?

Ok I got it. The beta sign for your context_vars from the model is more important, right? Do you get negative for factor 2 in the final decomp?

If I specify context variables sign, then the sign is what I need it to be in the final decomp. I specify negative sign for factor 2, then in the model it is negative, but still shows as positive in prophet decomposition. I wanted to get rid of that inconsistency, but if it is not influential then I can leave it.
As far as I understand, all we get from prophet is trend + seasonality + holidays as predictors, but factor variables inside prophet just help with getting those values from the dependent variable. Is that correct?
In other words, the question I'm asking is: how influential is the wrong sign for a factor variable in the prophet decomposition, considering that this variables has a correct sign in the model itself?

You can ignore the sign from prophet decomp


report popup window unification

Version used

AET 3.0.

Context of the issue/feature
It looks like every single report pop-up windows differ from one another a bit:

the history popup looks completely different from the shortcuts or note editor popup windows
the shortcuts "X" icon (to close the window) on the history window differs from the "X" icon on the history window while it is not available on the note-editor window.

For details see the image attached.

I would also align the wording in the popup - the third line with "use" verb seems bit odd in the context of all the "press"es


The zap-baseline.py runs on my site but scans only the target url in passive mode

I ran the zap baseline on my site which is a redirection so it redirects to another url. I see it scans only the target url, the python script I modified to login and it gave me the logged in access token also. But for some reason the warnings are only for the target url and not other spider crawled urls. Please help.
I ran this command. zap-baseline.py -t https://qa-pinpoint.ddiworld.com/psg2 -r testreport.html --hook=/zap/auth_hook.py -z "auth.loginurl=https://qa-pinpoint.ddiworld.com/psg2 auth.username="*****" auth.password="****" auth.username_field="Username" auth.password_field="Password" auth.submit_field="logon-button" auth.auto=1" -m 4 -j
The warnings came only from the target whereas I am expecting all the logged in api calls also.
WARN-NEW: Cookie No HttpOnly Flag [10010] x 16
https://qa-pinpoint.ddiworld.com/psg2 (200 OK)
https://qa-pinpoint.ddiworld.com/robots.txt (200 OK)
https://qa-pinpoint.ddiworld.com/sitemap.xml (200 OK)
https://qa-pinpoint.ddiworld.com/variables.css (200 OK)
https://qa-pinpoint.ddiworld.com/ (200 OK)
WARN-NEW: Cookie Without Secure Flag [10011] x 16
https://qa-pinpoint.ddiworld.com/psg2 (200 OK)
https://qa-pinpoint.ddiworld.com/robots.txt (200 OK)
https://qa-pinpoint.ddiworld.com/sitemap.xml (200 OK)
https://qa-pinpoint.ddiworld.com/variables.css (200 OK)
https://qa-pinpoint.ddiworld.com/ (200 OK)
WARN-NEW: Incomplete or No Cache-control and Pragma HTTP Header Set [10015] x 9
https://qa-pinpoint.ddiworld.com/psg2 (200 OK)
https://qa-pinpoint.ddiworld.com/robots.txt (200 OK)
https://qa-pinpoint.ddiworld.com/sitemap.xml (200 OK)
https://qa-pinpoint.ddiworld.com/variables.css (200 OK)
https://qa-pinpoint.ddiworld.com/ (200 OK)
WARN-NEW: Content-Type Header Missing [10019] x 1
https://qa-pinpoint.ddiworld.com/config (200 OK)
WARN-NEW: X-Frame-Options Header Not Set [10020] x 6
https://qa-pinpoint.ddiworld.com/psg2 (200 OK)
https://qa-pinpoint.ddiworld.com/robots.txt (200 OK)
https://qa-pinpoint.ddiworld.com/sitemap.xml (200 OK)
https://qa-pinpoint.ddiworld.com/variables.css (200 OK)
https://qa-pinpoint.ddiworld.com/ (200 OK)
WARN-NEW: X-Content-Type-Options Header Missing [10021] x 16
https://qa-pinpoint.ddiworld.com/psg2 (200 OK)
https://qa-pinpoint.ddiworld.com/robots.txt (200 OK)
https://qa-pinpoint.ddiworld.com/sitemap.xml (200 OK)
https://qa-pinpoint.ddiworld.com/variables.css (200 OK)
https://qa-pinpoint.ddiworld.com/ (200 OK)
WARN-NEW: Content Security Policy (CSP) Header Not Set [10038] x 6
https://qa-pinpoint.ddiworld.com/psg2 (200 OK)
https://qa-pinpoint.ddiworld.com/robots.txt (200 OK)
https://qa-pinpoint.ddiworld.com/sitemap.xml (200 OK)
https://qa-pinpoint.ddiworld.com/variables.css (200 OK)
https://qa-pinpoint.ddiworld.com/ (200 OK)
WARN-NEW: Cookie Without SameSite Attribute [10054] x 16
https://qa-pinpoint.ddiworld.com/psg2 (200 OK)
https://qa-pinpoint.ddiworld.com/robots.txt (200 OK)
https://qa-pinpoint.ddiworld.com/sitemap.xml (200 OK)
https://qa-pinpoint.ddiworld.com/variables.css (200 OK)
https://qa-pinpoint.ddiworld.com/ (200 OK)
FAIL-NEW: 0     FAIL-INPROG: 0  WARN-NEW: 8     WARN-INPROG: 0  INFO: 0 IGNORE: 0       PASS: 42

Does your site work with session cookies or auth tokens? Auth tokens are not supported atm. I am currently building support for it.

Please use the latest version with Authorization header support: #22
Please open a new issue if you are still having problems.


MongoError

An error was deployed using the stateful method,I don't know the wrong reason.
 message: 'failed to connect to server [<IP_ADDRESS>:27017] on first connect [MongoError: connect ECONNREFUSED <IP_ADDRESS>:27017]' }
Error in workloop { MongoError: failed to connect to server [<IP_ADDRESS>:27017] on first connect [MongoError: connect ECONNREFUSED <IP_ADDRESS>:27017]
    at Pool.<anonymous> (/opt/cvallance/mongo-k8s-sidecar/node_modules/mongodb-core/lib/topologies/server.js:328:35)
    at emitOne (events.js:96:13)
    at Pool.emit (events.js:191:7)
    at Connection.<anonymous> (/opt/cvallance/mongo-k8s-sidecar/node_modules/mongodb-core/lib/connection/pool.js:274:12)
    at Object.onceWrapper (events.js:293:19)
    at emitTwo (events.js:106:13)
    at Connection.emit (events.js:194:7)
    at Socket.<anonymous> (/opt/cvallance/mongo-k8s-sidecar/node_modules/mongodb-core/lib/connection/connection.js:177:49)
    at Object.onceWrapper (events.js:293:19)
    at emitOne (events.js:96:13)
  name: 'MongoError',

I found the wrong reason, because the disk space is full

@post707 can you elaborate on this? We are seeing the same issue, but it doesn't look like our disk space if full.


HotKeys Map to String

📝 Provide a description of the new feature
What is the expected behavior of the proposed feature?  What is the scenario this would be used?
For example, I can press a HotKeys  to quickly input a preset string.
There are two Mode:

input word by word. ( simulates a typewriter that keyboard input)
direct paste. (Similar to copy and paste, print directly)

If continue to dev, can also specify a group of strings: Increase the index after each input or press a key to increase.

If you'd like to see this feature implemented, add a 👍 reaction to this post.

Duplicate of https://github.com/microsoft/PowerToys/issues/1207


enhance detection of blocked requests

Intent
To provide a more solid functionality to capture unexpected terminations of the XHR communication such as CORS violation.
Solution

We test the XHR object against whether its readyState is updated as per the defined steps
If we detect that the readyState skips any of the steps and gets settled to 4 we mark the XHR object as prematurely terminated (xhr.prematurelyTerminated).
The assumption is that if xhr.prematurelyTerminated is truthy and onerror is invoked, it will be a result of CORS violation.

LGTM

LGTM. Did you test on IE?

Yeah, we tested IE11. The previous solution also worked in IE but this looks more robust and general :)

LGTM


Add support for application/json-patch+json media type

Environment Details

Helidon Version: 3.2.7
Helidon SE and Helidon MP
JDK version: 17
OS: n/a
Docker version (if applicable):

Problem Description
Add support for application/json-patch+json media type.
Steps to reproduce
n/a

This is do-able with JSON-P.
@Path("/patch")
public class JsonPatchResource {

    private static final JsonProvider JSON_PROVIDER = JsonProvider.provider();

    private JsonObject jsonObject = JsonValue.EMPTY_JSON_OBJECT;

    @GET
    public JsonObject get() {
        return jsonObject;
    }

    @PUT
    public void put(JsonObject jsonObject) {
        this.jsonObject = jsonObject;
    }

    @Consumes("application/json-patch+json")
    @POST
    public void patch(JsonArray operations) {
        JsonPatch patch = JSON_PROVIDER.createPatch(operations);
        this.jsonObject = patch.apply(jsonObject);
    }
}

# verify the empty initial state
curl http://localhost:8080/patch
{}

# set the object
curl -X PUT -H "Content-Type: application/json" http://localhost:8080/patch -d '{"foo": "bar"}'

# verify the object
curl http://localhost:8080/patch
{"foo":"bar"}

# patch the object
curl -X POST -H "Content-Type: application/json-patch+json" http://localhost:8080/patch -d '[ {"op":"replace","path":"/foo","value": "123"} ]'

# verify the result
curl http://localhost:8080/patch
{"foo":"123"}

We should double-check if this also works with Helidon SE. If this works, we should create examples for both.


Some confusion by reading about install-update -a command.

I am reading this page now.
At the bottom there is cargo install-update -a command suggestion.
I don't understand why is there Minimum Rust Version 1.5 badge? Rust nightly is 1.38 at the moment.
I am not native speaker and it is a little bit confusing to me. I did not seen this sentence This will allow you to use this command
For me it were better like that:

This will allow you to use cargo install-update -acommand ...

If it is ok dkotrada:patch-1


add RSA rustsec warning to ignores, we don't use that dependency

Interestingly there is no issue created.
Also I don't seem to find the reason why cargo audit warns here,
Crate:     rsa
Version:   0.9.4
Title:     Marvin Attack: potential key recovery through timing sidechannels
Date:      2023-11-22
ID:        RUSTSEC-2023-0071
URL:       https://rustsec.org/advisories/RUSTSEC-2023-0071
Severity:  7.4 (high)
Solution:  No fixed upgrade is available!
Dependency tree:
rsa 0.9.4
└── sqlx-mysql 0.7.2
    ├── sqlx-macros-core 0.7.2
    │   └── sqlx-macros 0.7.2
    │       └── sqlx 0.7.2
    │           └── docs-rs 0.6.0
    └── sqlx 0.7.2

( while th mysql feature is not active from our side)

cargo audit checks the lockfile, which lists all dependencies that might be activated if you activate (transitive) dependencies features, it doesn't limit itself to checking just the dependencies which are used.


PostgreSQL at version 12 not support column hasoids in pg_catalog.pg_class

Dbeaver 7.1.4
PostgreSQL 12
https://postgresql.verite.pro/blog/2019/04/24/oid-column.html
Can't get column 'relhasoids': The column name relhasoids was not found in this ResultSet.

Using checkbox "hasoids" in Table Properties results in an error.
Need to remove the reading of this column for version 12

checkbox "hasoids" in table editor removed for Postgres version >= 12

verified


"Share App Usage" label and switch control are not programmatically associated [Android]

The "Share App Usage" label is announced separately from the switch control. TalkBack announces the switch control as "switch, on"

Remediation Guidance:
Programmatically associate the label by including it in the accessibilityLabel attribute on the switch. Hide the visible text label from VoiceOver users via the accessibilityElementsHidden attribute, as not to announce the label twice to said users.
<View accessibilityElementsHidden={true}>
  <Text>Share App Usage</Text>
</View>
<Switch accessibilityLabel='Share App Usage' />

@jonnyjames This has now been addressed.

@floridemai I am confirming this issue is fixed. Closing this issue.


Upgrade of cstor pool fails due to image tag parsing

Description
Upgrade of cstor pool using job as described on https://github.com/openebs/openebs/blob/master/k8s/upgrades/README.md failed.
It seems to be due to parsing of image tag based on :
we run in an airgap env where we go through internal docker proxy, hence there will be two : in image: ourserver:port/openebs/theimagename:thetag
Expected Behavior
Should set correct images and tags
Current Behavior
image will be set to quay.io/openebs/fsdepot.evry.com:1.5.0
Possible Solution
tried using - "--to-version-image-prefix=fsdepot.evry.com:8085/openebs/" arg
For the parsing of tag, rather split string on : and use last segment as tag. Leave elements left of last : as they were.
Steps to Reproduce
have a cstor pool using images named as explained above
Screenshots

Your Environment
N/A
OpenEBS 1.4.0 -> 1.5.0
control-plane was successfully upgraded with helm prior to attempting data-plane
See thread on slack https://openebs-community.slack.com/archives/C3NPGQ6G3/p1576341369008300

The 1.5 upgrade image was repushed with the above fix. Thanks @davidkarlsen for reporting this issue.


Add Getting started doc

This PR adds GettingStarted.md which is a combination of Interoperability.md and TranslationProxy's readme. I've co-author initial commit crediting credited both @daricksee and @rrybarczyk.
The formatting of the code is still very bad, we'd have to work on better visualizing code, but that's beyond the scope of this PR.
Also the images aren't really responsive so we'd need someone who knows these things to help 😄

I will merge this one so we can easier finish the remaining tasks. Let's continue to iron it out though.


WIP: add execute to gs.provider

@tshemsedinov @belochub PTAL.

Closed in favor of https://github.com/metarhia/globalstorage/pull/318.


Server Down

Hi - The server appears to have been down since yesterday morning. I get the following message when running disaggregation in HazardCurveGUI-1.5.2:
Steps to reproduce: (PLEASE FILL IN)
Other info: (PLEASE FILL IN)
Application: HazardCurveLocal
Version: 1.4.0
Bulid Type: Development
Java Version: 1.8.0_241 (Oracle Corporation)
Operating System: Windows 10 (arch: amd64, version: 10.0)
Exception:
{{{
java.lang.RuntimeException: Server is down , please try again later
at org.opensha.sha.calc.disaggregation.DisaggregationCalculator.openServletConnection(DisaggregationCalculator.java:1113)
at org.opensha.sha.calc.disaggregation.DisaggregationCalculator.getDisaggregationPlotUsingServlet(DisaggregationCalculator.java:848)
at org.opensha.sha.gui.HazardCurveApplication.getDisaggregationPlot(HazardCurveApplication.java:2353)
at org.opensha.sha.gui.HazardCurveApplication.showDisaggregationResults(HazardCurveApplication.java:1553)
at org.opensha.sha.gui.HazardCurveApplication.computeHazardCurve(HazardCurveApplication.java:1461)
at org.opensha.sha.gui.HazardCurveApplication.run(HazardCurveApplication.java:882)
at java.lang.Thread.run(Unknown Source)
}}}

I can't reproduce this issue, are you sure you're using a recent build? You mention that you're using the most recent version (1.5.2) but the bug report lists a development build of version 1.4.0, which would indeed no longer work. Can you re-download the latest release and try again? https://github.com/opensha/opensha-apps/releases


[bug] Typescript: Could not find a declaration file for module "@tauri-apps/api/xxx"

Describe the bug
I updated my tauri-apps dependency, and I now can't build successfully.
It tells me that Typescript could not find corresponding declaration file for all my tauri apis.
When I added "type" to the path, it works, like the image below:

Here are my dependencies:
  "dependencies": {
    "@tauri-apps/api": "^1.5.2",
    "@vueuse/core": "^10.7.0",
    "@vueuse/integrations": "^10.7.0",
    "@web-types/vuetify": "^2.0.15-1",
    "change-case": "^5.3.0",
    "crypto-js": "^4.2.0",
    "dayjs": "^1.11.10",
    "fuse.js": "^7.0.0",
    "ip-address": "^9.0.5",
    "monaco-editor": "^0.45.0",
    "pinia": "^2.1.7",
    "prettier": "3.1.1",
    "qrcode": "^1.5.3",
    "roboto-fontface": "^0.10.0",
    "tauri-plugin-store-api": "https://github.com/tauri-apps/tauri-plugin-store#v1",
    "terser": "^5.26.0",
    "ua-parser-js": "^1.0.37",
    "uuid": "^9.0.1",
    "vue": "^3.3.12",
    "vue-i18n": "9.8.0",
    "vue-router": "4.2.5",
    "vuetify": "^3.4.7",
    "webfontloader": "^1.6.28"
  },

and my tsconfig.json:
{
  "compilerOptions": {
    "target": "ESNext",
    "useDefineForClassFields": true,
    "module": "ESNext",
    "moduleResolution": "Node",
    "strict": true,
    "jsx": "preserve",
    "sourceMap": false,
    "resolveJsonModule": true,
    "isolatedModules": true,
    "esModuleInterop": true,
    "lib": ["ESNext", "DOM", "DOM.Iterable", "ScriptHost"],
    "skipLibCheck": true,
    "baseUrl": ".",
    "paths": {
      "@/*": ["./src/*"]
    },
    "allowSyntheticDefaultImports": true,
    "forceConsistentCasingInFileNames": true
  },
  "include": [
    "src/**/*.ts",
    "src/**/*.d.ts",
    "src/**/*.tsx",
    "src/**/*.vue",
    "build/**/*.ts"
  ],
  "references": [{ "path": "./tsconfig.node.json" }],
  "exclude": ["node_modules"]
}

Is is because of an mismatch version in my dependencies? Or should I downgrade it? Or shold I add "type" to everywhere I use a tauri api?
Reproduction
No response
Expected behavior
No response
Platform and versions
[✔] Environment
    - OS: Windows 10.0.19045 X64
    ✔ WebView2: 120.0.2210.77
    ✔ MSVC: Visual Studio Community 2022
    ✔ rustc: 1.73.0 (cc66ad468 2023-10-03)
    ✔ cargo: 1.73.0 (9c4383fb5 2023-08-26)
    ✔ rustup: 1.26.0 (5af9b9484 2023-04-05)
    ✔ Rust toolchain: stable-x86_64-pc-windows-msvc (default)
    - node: 20.8.0
    - yarn: 1.22.19
    - npm: 10.1.0

[-] Packages
    - tauri [RUST]: 1.5.3
    - tauri-build [RUST]: 1.5.0
    - wry [RUST]: 0.24.6
    - tao [RUST]: 0.16.5
    - @tauri-apps/api [NPM]: 1.5.2
    - @tauri-apps/cli [NPM]: 1.5.8

[-] App

Stack trace
No response
Additional context
No response

I made a mistake, I should change "type" to "types", that was a typo.

Another error occured when I added "types" to "tauri-apps/api/types/shell":
[commonjs--resolver] Missing "./types/shell" specifier in "@tauri-apps/api" package
Image as below:

In tauri-apps/api version 1.5.1, the directory looks like this:

You can see that type definitions are placed in the same folder with source files.
And this is what it looks like when using version 1.5.2:

You can see that type definitions are placed in a separate folder.
Mostly it's because of this.

This error can be fixed by downgrading @tauri-apps/api to version 1.5.1.

For version 1.5.2, it can be fixed in this way(using clipboard as an example):
import { clipboard } from "@tauri-apps/api/types";

await clipboard.writeText("Hello, world!");

But the documentation is still not updated, so should I use this way?

Note that this also works:
import { clipboard } from "tauri-apps/api";

I suggest altering the tsconfig.json setting from "moduleResolution": "Node" to "moduleResolution": "bundler". This modification resolved the issue for me.

This is fixed and will be available in the next release.


fix DateTime case typo in data-modeling.md

looks good, thanks!

Hey @fiws
Would you mind signing our CLA for this project? Since our repos are new, we didn't have cla assistant setup at that time.
Please sign it here: https://cla-assistant.io/prisma/prisma2-docs?pullRequest=7
If you don't sign this, we will, unfortunately, have to remove your commit.


Remove references for ESP32

This actually bit colliding with some upstream patches, but I will take care of it.
Thank you very much for your pull request!
For receiving the free PineCone, please sign up at this link.
(If there will be any issues with signing up, please let me know here).


IBM Z Open Editor support for mainframe Multi Factor Authentication via SAF (RACF etc)

We use Multi Factor Authentication for all our mainframe access  - whether it be via TSO/IPSF, z/OSMF, Eclipse etc. This is done via the standard IBM SAF exit supported by RACF, ACF2 and Top Secret.   z/OSMF itself does allow a userid and password to be entered and it is the presence of a value in the password field that triggers the MFA process.
To make use of IBM Z Open Editor, we need the ability to perform this Multi Factor Authentication.  We cannot use hardcoded passwords saved in clear text which appears to be the current requirement.

@davidkjackson54 for Z Open Editor we want to follow the work of the Zowe CLI and Explorer teams that we are also part of. See this article describing the current direction: https://medium.com/zowe/zowe-cli-token-authentication-mfa-and-sso-b88bca3efa35
In Zowe Explorer 1.9 we are not quite there, yet, but making good progress. On the Z Open Editor side we are trying to adopt these items as much as we can. Sometimes there are hick-ups as you observed with secure credential plugin support. However, it is a top priority to add it very soon. We are working also on supporting other aspects of the paper linked above.


Outdated data

The data in this document seems to be outdated as some products don't have all services. Where can I get the updated reference?

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

ID: 7c245452-161d-89a1-c026-99a0721cf071
Version Independent ID: 16dec59a-eb83-1837-4efd-4645e7aeaed2
Content: Product names and service plan identifiers for licensing in Azure Active Directory
Content Source: articles/active-directory/active-directory-licensing-product-and-service-plan-reference.md
Service: active-directory
GitHub Login: @piotrci
Microsoft Alias: piotrci

@BukeMan Thanks for the feedback. We are actively investigating and will get back to you soon.

This doc lists only the most popular SKUS and Service Plans in AAD and not all.

Sorry guys, but I do not get it. I opened at least three premier cases in the past years to have SKUs and service plans properly documented. I also talked personally with Alex Simons about that ...
It always ended up with a single documentation that was never updated.
As a customer you suddenly see strange service plans appearing in your SKUs with no documentation what it actually is (Best example: BPOS_S_TODO_FIRSTLINE).  We required a full up-to-date list of SKUs, ServicePlans and explanation of the service. Why is that so difficult??

Fully agree - we need up to date list
And if possible some API to make it easier when we run PowerShell script to take most up to date list.

@MicrosoftDocs/azure-cxp-triage please close this issue as a dupe for this issue.

@mtillman this is not a dupe as the other issue was opened 9 days after this one.
@MohitGargMSFT please reopen this one and close another one. thanks!

#please-close I have updated the page with latest information on 8/17/2018. Note - These tables are for reference purposes and are accurate only as of the date when this article was last updated. Microsoft does not plan to update them for newly added services periodically.


Initialize Persistent Storage

Major Changes:

Implement GET API calls as real database lookups.
Implement /reset-data servlet (As a GET request, so one can do this from a browser -- this is temporary anyway) to delete and reinitialize test data.

Minor Changes:

Change serialization and deserialization of entity keys, to solve problems with Key.toUrlSafe() format

TODO:
1. Current serialization and deserialization methods assume the default database instance is always to be used. However, in a testing environment, this will not be the case. (Done)
2. Write integration tests on either an in-memory or online test database. (Perhaps on separate PR)

@codyjrivera: let me know when its ready for a rereview!


Add support for nullish values in points

Initial checklist

[x] I read the support docs
[x] I read the contributing guide
[x] I agree to follow the code of conduct
[x] I searched issues and couldn’t find anything (or linked relevant results below)
[x] If applicable, I’ve added docs and tests

Description of changes
Sometimes these are produced by vfile-message.

1.0.3!


chore(deps-dev): upgrade scripts and docker version and solc version

Looks like the workflow is not passed @mask-pp

Looks like the workflow is not passed @mask-pp

Ok, I will fix it.


Add button to navigate back to Templates and Guidance pages

Currently, the only way to return to the main Templates or Guidance page when editing this content is to select them from the main Admin dropdown menu.
DMPonline has a button in the upper right hand corner to "View all templates" or "View all guidance" - we should reintroduce this button for Template editing, Guidance group editing, and Guidance editing pages (screenshots below).

button is back from View all templates, but we still need them in the upper right for Guidance group and Guidance pages @briri (screenshot below of roadmap stage)

now we have all the buttons! closing issue.


Report constraint violations for invalid solutions

Fixes #8

This also includes all the commits in #7 as its branch was created from there. Might be worth ignoring this until #7 is merged.

This makes changes that will affect any outstanding work on #2

Merge conflict after #7 went in.


tensorrt_llm llama Engine build error

System Info
trtllm-build --checkpoint_dir ./tllm_checkpoint_1gpu_bf16
--output_dir ./tmp/llama/7B/trt_engines/bf16/1-gpu
--gpt_attention_plugin bfloat16
--gemm_plugin bfloat16
An error occurs when the command is entered.
It may not be optimal to set max_num_tokens=max_batch_size*max_input_len when remove_input_padding is enabled, because the number of packed input tokens are very likely to be smaller, we strongly recommend to set max_num_tokens according to your workloads.
Traceback (most recent call last):
File "/usr/local/bin/trtllm-build", line 8, in
sys.exit(main())
File "/usr/local/lib/python3.10/dist-packages/tensorrt_llm/commands/build.py", line 514, in main
parallel_build(source, build_config, args.output_dir, workers,
File "/usr/local/lib/python3.10/dist-packages/tensorrt_llm/commands/build.py", line 441, in parallel_build
with ProcessPoolExecutor(mp_context=get_context('spawn'),
File "/usr/lib/python3.10/concurrent/futures/process.py", line 611, in init
raise ValueError("max_workers must be greater than 0")
ValueError: max_workers must be greater than 0
Who can help?
No response
Information

[X] The official example scripts
[ ] My own modified scripts

Tasks

[X] An officially supported task in the examples folder (such as GLUE/SQuAD, ...)
[ ] My own task or dataset (give details below)

Reproduction
trtllm-build --checkpoint_dir ./tllm_checkpoint_1gpu_bf16
--output_dir ./tmp/llama/7B/trt_engines/bf16/1-gpu
--gpt_attention_plugin bfloat16
--gemm_plugin bfloat16
Expected behavior
Build Complete
actual behavior
[TensorRT-LLM] TensorRT-LLM version: 0.9.0.dev2024022000
/usr/local/lib/python3.10/dist-packages/torch/cuda/init.py:626: UserWarning: Can't initialize NVML
warnings.warn("Can't initialize NVML")
[02/26/2024-09:18:30] [TRT-LLM] [I] Set bert_attention_plugin to float16.
[02/26/2024-09:18:30] [TRT-LLM] [I] Set gpt_attention_plugin to bfloat16.
[02/26/2024-09:18:30] [TRT-LLM] [I] Set gemm_plugin to bfloat16.
[02/26/2024-09:18:30] [TRT-LLM] [I] Set lookup_plugin to None.
[02/26/2024-09:18:30] [TRT-LLM] [I] Set lora_plugin to None.
[02/26/2024-09:18:30] [TRT-LLM] [I] Set moe_plugin to float16.
[02/26/2024-09:18:30] [TRT-LLM] [I] Set context_fmha to True.
[02/26/2024-09:18:30] [TRT-LLM] [I] Set context_fmha_fp32_acc to False.
[02/26/2024-09:18:30] [TRT-LLM] [I] Set paged_kv_cache to True.
[02/26/2024-09:18:30] [TRT-LLM] [I] Set remove_input_padding to True.
[02/26/2024-09:18:30] [TRT-LLM] [I] Set use_custom_all_reduce to True.
[02/26/2024-09:18:30] [TRT-LLM] [I] Set multi_block_mode to False.
[02/26/2024-09:18:30] [TRT-LLM] [I] Set enable_xqa to True.
[02/26/2024-09:18:30] [TRT-LLM] [I] Set attention_qk_half_accumulation to False.
[02/26/2024-09:18:30] [TRT-LLM] [I] Set tokens_per_block to 128.
[02/26/2024-09:18:30] [TRT-LLM] [I] Set use_paged_context_fmha to False.
[02/26/2024-09:18:30] [TRT-LLM] [I] Set use_context_fmha_for_generation to False.
[02/26/2024-09:18:30] [TRT-LLM] [W] remove_input_padding is enabled, while max_num_tokens is not set, setting to max_batch_sizemax_input_len.
It may not be optimal to set max_num_tokens=max_batch_sizemax_input_len when remove_input_padding is enabled, because the number of packed input tokens are very likely to be smaller, we strongly recommend to set max_num_tokens according to your workloads.
Traceback (most recent call last):
File "/usr/local/bin/trtllm-build", line 8, in
sys.exit(main())
File "/usr/local/lib/python3.10/dist-packages/tensorrt_llm/commands/build.py", line 514, in main
parallel_build(source, build_config, args.output_dir, workers,
File "/usr/local/lib/python3.10/dist-packages/tensorrt_llm/commands/build.py", line 441, in parallel_build
with ProcessPoolExecutor(mp_context=get_context('spawn'),
File "/usr/lib/python3.10/concurrent/futures/process.py", line 611, in init
raise ValueError("max_workers must be greater than 0")
ValueError: max_workers must be greater than 0
additional notes
If anyone has solved the problem, please comment.

This answer solved my problem!

I seem to find the reason! It is because no GPU was found and the number of workers was set to 0 here. Check if your container or host sees GPU by running nvidia-smi. In my case, the container was not connecting to CUDA resulting in the error. I have restarted the container and the problem was resolved.

@limes22 Do you still have the question? If not, we will close it soon.


Missing pubsub messages between client disconnect and reconnect

<Copied from https://docs.microsoft.com/en-us/answers/questions/482237/missing-pubsub-messages-between-client-disconnect.html>
We are using Azure web pubsub to push events realtime to devices. The client connects to the websocket url and the server pushes event messages to clients through pubsub using azure pubsub python sdk. We found that the pubsub resends unsent event messages to clients and sometimes it does not.
There are two scenarios:

When the device stays unlocked.

When the client app disconnects and reconnects after sometime, the pubsub resends the missing event messages from server during this time. The unsent event messages are even resent to client app when it goes to background and comes to
foreground(reconnects to pubsub) after 1 hour!

When the device is locked and unlocked in between

The problem occurs when the device is locked. When its locked we found that the websocket connection gets immediately
disconnected and when the screen is unlocked, the client app does not get the unsent event messages during this period.
The client app, though, keeps on getting the new messages after the device is unlocked.
We want to understand the working of pubsub's retry mechanism in case of unsent event messages and its guarantees. We could not find any documentation related to this in the microsoft docs.

Web PubSub itself does not have a message delivery guarantee, it acts as a broker that delivers real-time messages to connected WebSocket clients. So if the WebSocket connection is disconnected, the client does not get the messages sent during this period. if you want to guarantee message delivery, consider supporting it in your application protocol or using a protocol having such mechanism, e.g. MQTT.

"When the client app disconnects and reconnects after sometime, the pubsub resends the missing event messages from server during this time." => No, Web PubSub does not have a mechanism to "resend" messages, it does not preserve messages. Could you describe more about how your server-side sends event messages, is there any retry logic from your server-side?

hey @vicancy
with Azure Web PubSub Reliable JSON WebSocket subprotocol is there a way to accomplish message delivery guarantee?
for example if Client A sends message to server -> server reads target client Id from the message -> sends message to Client B.
Client A would get ACK from the server that message was delivered to Client B?

hey @vicancy with Azure Web PubSub Reliable JSON WebSocket subprotocol is there a way to accomplish message delivery guarantee?
for example if Client A sends message to server -> server reads target client Id from the message -> sends message to Client B.
Client A would get ACK from the server that message was delivered to Client B?

Client A gets ACK from server when server successfully received the message from A. When using reliable subprotocol, Client B is guaranteed to receive the message from A even if Client B is disconnected and reconnected during this period. But if clientA is expecting some ACK from B, then B need to send some application layer Ack message to A, such ack is not supported in the reliable subprotocol level.


Add option to import CSV files in VMWare controller UI

This change is

@floreks
Please check travis failure. UT failed due to missing copyright header in ConfigurationType.java.
[junit] Test failed due to the following files don't contain the specified copyright headers:
[junit] /home/travis/build/servicecatalog/oscm/oscm-common-unittests/../oscm-app-vmware/javasrc/org/oscm/app/vmware/importer/model/ConfigurationType.java

@GoebelL Yes, it's my class, I will update it tomorrow.


Snatch some of the Apple foundation tests to cover edge cases

Codable tests in the foundation repo:

https://github.com/apple/swift-corelibs-foundation/blob/main/Tests/Foundation/Tests/TestCodable.swift
https://github.com/apple/swift-corelibs-foundation/blob/main/Tests/Foundation/Tests/TestJSONEncoder.swift

These should help re-create some of the more difficult to hit scenarios like — when do you need a nested unkeyed container in an unkeyed container? Trying to have a [[Int]] property on a Codable struct doesn't seem to hit that codepath in the encoding container 🤷🏽♂

I really like the test helper and the naming (via https://github.com/apple/swift-corelibs-foundation/blob/main/Tests/Foundation/Tests/TestCodable.swift):
func expectRoundTripEquality<T : Codable>(of value: T, encode: (T) throws -> Data, decode: (Data) throws -> T) throws where T : Equatable  {
    do {
        let data = try encode(value)
        let decoded: T = try decode(data)
        if value != decoded {
            throw NSError(domain: "Decode mismatch", code: 0, userInfo: ["msg": "Decoded \(T.self) <\(decoded)> not equal to original <\(value)>"])
        }
    }
}

func expectRoundTripEqualityThroughJSON<T : Codable>(for value: T) throws where T : Equatable {
    let inf = "INF", negInf = "-INF", nan = "NaN"
    let encode = { (_ value: T) throws -> Data in
        let encoder = JSONEncoder()
        encoder.nonConformingFloatEncodingStrategy = .convertToString(positiveInfinity: inf,
                                                                      negativeInfinity: negInf,
                                                                      nan: nan)
        return try encoder.encode(value)
    }

    let decode = { (_ data: Data) throws -> T in
        let decoder = JSONDecoder()
        decoder.nonConformingFloatDecodingStrategy = .convertFromString(positiveInfinity: inf,
                                                                        negativeInfinity: negInf,
                                                                        nan: nan)
        return try decoder.decode(T.self, from: data)
    }

    try expectRoundTripEquality(of: value, encode: encode, decode: decode)
}


CONDA_AUTO_ACTIVATE_BASE=true sets base

Testing #20538
In the TPI's Case II & III, I notice that base gets activated anyways

@andreamah This seems expected as auto activate base is configured to true.
Manual activation isn't required in those cases.

Didn't case II and III have something like
Open a Python file to activate the extension, ensure the base environment is not auto-selected at startup.

?
So then I wouldn't expect an automatic activation in that case?

Ah, Yes, I think what's happening is that base environment was already selected for the workspace. In which case it was auto activated.
If some other environment was pre selected, we would not be auto selecting and activating base in that case. Thanks for pointing it out!


Add pre-commit and fix its complaints

See #49

Please follow the PR Submission Checklist.
For example, some changes from this PR's files are removed empty lines (bullet point number 5). While these stylistic changes aren't inherently bad, we would want these type of changes to be in another PR to individualize problems and goals.
Additionally, it is strongly recommended to give a good PR description so other contributors can have a clear idea of what the PR is doing. The following format is what I use:

Previous Issue
What previous problems in the project that causes the need for this PR?

Purpose of this PR
How does the PR solve the problems mentioned?

Testing
What are the ways that this PR was tested?

Issues
If there are existing issues that this PR might introduce, state them here. (The solution to the problem might not be related to the goal of the current PR, which is a valid reason to not have it fixed in the current PR)

Also, please update CHANGELOG.md as well. Thank you.


Not able to connect

Installed the bluetooth chat in two devices(Android M & Android N) & when I m trying to search for nearby device its not detecting

I expect this is due to lack of proper permission handling (required for Android M+). You will need to add permission request to manifest & manually enable for the app. See this for similar problem with Google BluetoothLeGatt sample. It may seem odd, but ACCESS_COARSE_LOCATION is required for bluetooth scanning (because data gathered might allow someone to extrapolate your location).

https://github.com/googlesamples/android-BluetoothChat/issues/20


Cannot perform actions on elements bound to names containing 'of' literal

Performing an action on a web element that is bound to a name containing the 'of' literal incorrectly calls I <click|check|uncheck> <element> of <context> instead of I <click|check|uncheck> <element>.
For example, the following should locate and click the end of page button but it instead attempts to locate page button (the text that appears after the 'of' literal) and fails.
Given the end of page button can be located by id "endofpage"
 When I click the end of page button

This was detected in gwen-web v1.2.0

Solution was to perform the I <click|check|uncheck> <element> logic if  I  <click|check|uncheck> <element> of <context> results in a LocatorBindingException.


Remove cargo-install-all.sh hack for solana-genesis

just found this while working on #2867

🫶

:heart:
let's see this aged bug (https://github.com/solana-labs/solana/issues/5826) still reproduces before landing #2867. :)

seems both windows and linux aren't experiencing the original build stuck issue. (macos tests are still running though)


Allow to specify withdraw address

Due to the fact that Keplr does not support different Ledger apps, it would be nice to be able to specify the withdraw address when withdrawing funds from Osmosis. This way, people that are using addresses generated with different SLIP-44 values can safely withdraw their funds to such addresses without having to perform the following steps:

withdraw the funds to the address generate by Keplr
send the funds to their real account

This was intentionally done to:

Prevent users accidentally withdrawing their tokens to an exchange address. (Which we've seen happen quite a few times on Keplr).
Prevent confusion around 'same mnemonic but different address' stemming from derivation path confusion.

Additionally, 'memo' field is not sent over IBC–therefore it'll be impossible to actually use IBC for exchange withdrawals.
That being said, I'll explore this with the product team (cc @victortokaito) to see if we can provide an 'advanced' option to allow manually setting the withdrawal address.

You can also add an address book or something similar directly into osmosis.

[ ] Non 118 bip44 config
[ ] CS20


fix: fixed a few issues found during end-to-end testing

we were calling the /auth endpoint, whereas we should have been calling the /webhook endpoint.
we should not be caching annotations if there are none.

Whoops, fat lot of good my other code reviews have been :P
LGTM

Coverage increased (+0.3%) to 84.375% when pulling 8921b9ef2e3d206ee50780b21e3e4f2bba17a109 on testing-bug-fixes into 45cf849407ee784914c184e01bc3ea12631cdbca on master.

Coverage increased (+0.3%) to 84.375% when pulling 8921b9ef2e3d206ee50780b21e3e4f2bba17a109 on testing-bug-fixes into 45cf849407ee784914c184e01bc3ea12631cdbca on master.

Coverage increased (+0.3%) to 84.375% when pulling 8921b9ef2e3d206ee50780b21e3e4f2bba17a109 on testing-bug-fixes into 45cf849407ee784914c184e01bc3ea12631cdbca on master.


Improve slider components

Description
Added label for the rangeslider and some improvements in the component
How Has This Been Tested?
Ran existing tests
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have updated the CHANGELOG document.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

Would it be possible to integrate a fix for https://github.com/SwedbankPay/design.swedbankpay.com/issues/447 with this PR?
I can help if needed


refactor: modularize backends

Part of #1388
Checklist:

[ ] Check if underline-color and scrolling-regions flags are still working
[ ] Add crate documentation/metadata
[ ] Do not compile termion on Windows (target.cfg didn't work as workspace dependency)
[ ] Other bits

There's still a problem with refs in the docs in ratatui-* to ratatui:: types. Turns out that while examples can reference the libraries in dev-dependencies, the docs cannot. This seems odd to me.


[DO NOT MERGE] AzureDDoSStandardProtection workbook update for ICM

Required items, please complete
Change(s):

See guidance below

Reason for Change(s):

See guidance below

Version Updated:

Required only for Detections/Analytic Rule templates
See guidance below

Testing Completed:

See guidance below

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

See guidance below

Guidance <- remove section before submitting

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

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

Change(s):

Updated syntax for XYZ.yaml

Reason for Change(s):

New schema used for XYZ.yaml
Resolves ISSUE #1234

Version updated:

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

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

Testing Completed:

Yes/No/Need Help

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

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

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

Yes/No/Need Help

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

References:

Guidance for Detection checks
General contribution guidance
PR validation troubleshooting

@NikTripathi  and @v-rucdu : Please review and provide your feedback.

This workbook is with customer and we are waiting on there confirmation.
Thanks

We have a call scheduled on 21 sep

This workbook is with customer, and we are waiting on their confirmation. Thanks

@NikTripathi : Could you please work on it as suggested by author

@NikTripathi : Could you please have a look and provide an update

This workbook is with customer, and we are waiting on their confirmation. Thanks!!

@NikTripathi, can you please check and confirm if customer has responded, thanks.

We had a call scheduled with customer. But he didn't joined. So we are still waiting on customer.

This workbook is with customer, and we are waiting on their confirmation. Thanks!!

Raised a new PR with latest update AZ DDOS workbook update
#6377 Closing this one.