Replace encoders and decoders with kotlinx.serialization format

Definition.encodeToByteArray(definition)
Definition.decodeFromByteArray(byteArray)
Instead of string tags like Json or Yaml use int tags to represent opcodes see TaggedDecoder

[x] Primitives
[x] Multiple values per opcode
[x] Multiple opcodes per value

[x] Index offset
[ ] Different logic (obj low detail models)

[x] Collections
[x] Unsigned primitives
[x] Nullable fields
[x] Custom primitives (smarts, mediums)
[ ] Modifications (<< 2, * 5) - KSerializer template/polymorphism/contextual?
[x] Operations which don't read just set values

How will these custom types be decoded?

~Unsigned types~ ~Kotlins experimental types~ Not supported, @Unsigned annotation for now?
~Smart types~ Custom methods/inline class? Waiting on https://youtrack.jetbrains.com/issue/KT-28791
Arrays which start with length as Byte/Short/Int
~Nested arrays for models~ Custom data classes
~Parameters which use Map<String, Any>~ Custom data classes
~Single opcodes which have multiple mixed type values within (array?, tuples)~ Custom data classes
Opcodes which don't read anything just set values (0, 1, 2, true, false)
~Single opcodes which have multiple arrays (colours, textures)~ Custom data classes
~Values which are normally modified (<< 2, * 5)~ Don't modify/modify on read
~Opcodes which set an index in an array~ Custom property serializer

What about other formats?

Maps <-> ByteArray
Locations <-> ByteArray
XteaKeys (IntArray) <-> String
Xteas (Map<Int, IntArray>) <-> ByteArray
CS2 (CS2) <-> ByteArray
Interfaces <-> ByteArray
Sprites <-> ByteArray/BufferedImage?
WorldMap/Icon<-> ByteArray

Working prototype implemented however consistently running into kotlinx.serialization limitations (no inline class support, collections annotations not working). Worth attempting the same with Jackson.

With Kotlin's 1.4.30 release they announced support for serialization of inline classes using snapshots and their IR compiler. With that I got the prototype fully working with item definitions.
There are some tradeoffs though:

The correct primitive types have to be used, (Short, UShort, Medium etc..) as current implementation favour Int this would introduce a lot of additional casting to existing usages. And a lot of unsigned fields like to default to -1 which doesn't work when using unsigned primitives, they'd have to be changed to nullables.
Default values don't persist, when values are read to an array the default values aren't carried across. It could potentially be supported with a lot of boilerplate for each array type.

Serialized values aren't as optimised as the originals as empty values currently are written.
Some fields have to be combined into data classes, depending on serialization format

Will need to decide if the tradeoffs of not having to write manual codecs are worth the performance cost.
Might be worth looking again into auto generating decoders which should perform much faster.

Not worth the time it would take to implement.
It would remove at most 59 decoder class files, and instead double the length of the data classes making them harder to read.
The main motivation for this idea was for a flat cache to allow easy modification and addition parameters. However the definition extra's extension is currently working fine and cache modifications aren't needed atm. The flat cache idea also didn't consider that packing custom fields into the cache would need custom opcodes (limited to 256), as well as bulk the cache, and would crash the client if they were sent.


Obsolete deprecation warning

Describe the bug:
The following deprecation warning is no longer accurate:

Deprecated behaviour! Other logging resources exist with the same loggingRef: example. This is going to be an error with the next major release.

Expected behaviour:
This deprecation should only be a notice, as there are certain use cases, where this behaviour is otherwise appropriate.
Steps to reproduce the bug:
Create two logging resources without specifying a loggingRef or by specifying the same loggingref and check their status.

fixed in #1699


Update to Unicode 9 beta files as of February 24, 2016.

Various updates, but most importantly, removal of two map symbols
from Unicode 9 repretoire and more additions to script extensions.

LGTM


Removes use of jQuery library

There's much that is very good about jQuery.  I'm taking it out because I've concluded that the purpose of demonstrating the capacity of Python + Transcrypt is better served by showing that the native JS DOM calls fit very nicely into the Python idioms.

I think you're right, this is an improvement.

Thanks, Jacques. I got motivated to undertake the change when I stumbled across You Might Not Need jQuery. Have you seen it? It's got side-by-side examples of a large subset of jQuery calls with replacement code using only vanilla JS.

I hadn't seen the document, but in many cases I try to circumvent jQuery, as it is a bit too 'magical' for me and makes debugging sometimes difficult.


AMBARI-24618. Ambari generates wrong zookeeper connection string for KMS HA configuration

What changes were proposed in this pull request?
Ambari generates wrong zookeeper connection string for KMS HA configuration
How was this patch tested?
manually
Please review Ambari Contributing Guide before opening a pull request.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/3907/
Test FAILed.
Test FAILured.

retest this please

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/3921/
Test FAILed.
Test FAILured.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/3924/
Test FAILed.
Test FAILured.


Improve documentation/error reports for concept macros

when trying to create concepts, i get messages "error: like no applicable concept macro found"
this is not very helpful and I don't even get, how to define the correct concept-macros
I'm not even able to (instantiate Array float 2)
isn't there a simpler way than concepts for such things? I thought about macro-dispatching on generic types like (def x (macro ((a (array-of 0 \))))), which may also be
but maybe this won't be easyly implementable

On Thu, Sep 01, 2016 at 02:14:57PM -0700, Fabio Krapohl wrote:

when trying to create concepts, i get messages "error: like no applicable concept macro found"
this is not very helpful and I don't even get, how to define the correct concept-macros
I'm not even able to (instantiate Array float 2)

The error messages have been improved, and now show the concepts
implemented by the arguments (including refinements), as well as the
concepts expected by the concept macros that exist with that name.
For example:
(import concepts)
(import array)

(def mys (struct intern ((a int) (b int))))
(std.concepts.implement Struct mys)
(std.concepts.instantiate Array mys 5)

yields:
test.dt:6:33: error: no applicable concept macro found (arguments implement the following concepts: Struct (refines Type); Value; current candidates accept: EqualityComparable, Value) (see macro at 6:1)
test.dt:6:1: error: macro expansion error (see previous)

When a concept macro is asserting the implementation of a concept
outside the dispatch context, the required concept will now be
included in the error message, too:
(import concepts)
(import array)
(import derivations)

(def mys (struct intern ((a int) (b int))))
(std.concepts.implement Struct mys)
(std.concepts.instantiate relations mys)
(std.concepts.implement EqualityComparable mys)
(std.concepts.instantiate Array mys 5)

yields:
test.dt:9:33: error: type does not implement concept LessThanComparable (see macro at 59:21)
.../array.dt:59:21: error: macro expansion error (see previous) (see macro at 1173:31)

isn't there a simpler way than concepts for such things? I thought about macro-dispatching on generic types like (def x (macro ((a (array-of 0 \))))), which may also be
but maybe this won't be easyly implementable

Hopefully the error messages now are decent enough that alternatives
don't seem so necessary, but I'm not inclined to add this sort of
functionality into the core.  Although there is a fair amount of
ceremony required when defining new types, instantiating algorithms
for types, and so on, this is necessary to some extent in any
language.  It seems like a relatively small cost for allowing the
implementation to be completely library-based, given how much more
complicated the core language would become if it had to handle
functions with generic types and concept (or type class or similar)
definitions.


Light version

Just wondering... if this is a dark version, there will be a light one? Or not compatible at all with conservative 7-color palettes?

A light version will require some changes, but it should be possible while still using the conservative 7 color palette. I will start work on it shortly.


DataPlane Integration Tests (#752)

Signed-off-by: Stepan Anokhin<EMAIL_ADDRESS>Description
Write integration tests for data-plane
Motivation and Context
See #752
How Has This Been Tested?

[ ] Covered by existing integration testing
[x] Added integration testing to cover
[x] Tested locally

Types of changes

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

Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

@edwarnicke please take a look.
According to @haiodo, a motivation behind #752 is to perform required groundwork to troubleshoot #753. That means we need a test setup in which dataplane is programmed directly (e.g. nsmd is not involved) and a way to easily test different scenarios with cross-connect create/update. I think current PR solves this problem and could be merged.
I can easily create memif cross connections, but not sure what is the best way to verify it. Memif-tests are optional for above mentioned goal, so I think we can basically skip it for now and merge PR as is. I've created a new PR #827 to cover and fix #753 so I may include memif-related tests in it later (memif is not a priority now).
What do you think?

Let's merge #795 first, then you can merge master and we'll consider it.

There is no conflict with #795 , so merged.


Adding OCSF Finding Info missing types

As per discussion, adding better types to work with to model Finding Info.
Minor:

Stop gen legacy code from legacy protos
Moving to buf workspaces for local gen
Ignoring annoying Mac specific gen files

Needed to finish up https://github.com/smithy-security/smithy/pull/487

Pull Request Test Coverage Report for Build<PHONE_NUMBER>7
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 40.769%

Totals

Change from base Build<PHONE_NUMBER>2:
0.0%

Covered Lines:
3489

Relevant Lines:
8558

💛  - Coveralls


setup action not working with 0.24.3

I am trying to use the github action tree-sitter/setup-action for my tree-sitter grammar, but it is failing:
Run config="$(tree-sitter init-config | cut -c32- | tr '\\' '/')"
 config="$(tree-sitter init-config | cut -c32- | tr '\\' '/')"
printf '{"parser-directories":["%q"]}' "${GITHUB_WORKSPACE%/*}" > "$config"
shell: /usr/bin/bash --noprofile --norc -e -o pipefail {0}
env:
 TREE_SITTER_REF: v0.24.3
 TREE_SITTER_REPO: https://github.com/tree-sitter/tree-sitter
/home/runner/work/_temp/c8ba34f7-7dcc-4b97-92e5-509eba40c2a0.sh: line 2: ree-sitter field has been automatically migrated to the new tree-sitter.json` config file
ps://tree-sitter.github.io/tree-sitter/creating-parsers
/home/runner/.config/tree-sitter/config.json: No such file or directory
Error: Process completed with exit code 1.
``

I  am using the same workflow file as other grammars in github and they don't seem to have that problem, but I realized those were using trees-sitter 0.23.0.

Changed tree-sitter-ref to v0.23.0 and then it works.

Should be fixed now.


Remake the model

Clean up model to use preprocess fx instead of undersample.
Add SMOTE
Use ddof=0
Don't convert date to int64 for derivative, just assume dx = 1
Don't use PCA for final

https://github.com/suchak1/hyperdrive/tree/feature/update-model-2


Fix sentence fragment

The sentence fragment and the next sentence are combined because they're probably meant to be together.

Hello @greyhillman ... It's not a fragment, and the author did intend this explanation in this format. However, I may have said "... is only ..." in the first line. We also generally avoid future tense, so "the item will never expire" would go to "the item never expires." Also, it doesn't quite make sense to me to say something can be accessed more frequently than ... [an] interval, which is singular. Seems like it should say within the interval forever/repeatedly or forever/repeatedly before the end of the internal is reached.
Ping @Rick-Anderson to help sort this out. 😄

@guardrex  not crazy about Oh-Red as in

are logically ORed

@guardrex  one last look

Yeah ... except it's not just within the period, right ... it's like accessed forever/repeatedly over and over within the period ... that will never expire it.

Hello @greyhillman ... It's not a fragment, and the author did intend this explanation in this format. However, I may have said "... is only ..." in the first line.

I don't know grammar too well, so maybe sentence fragment isn't the right one. When I read the sentence, it has the feeling of needing more. I think the confusion is what "only" is modifying.

I read it as "only" modifying "at risk of becoming stale". The sentence ends there but I'm expecting there to be more: "When is at risk of becoming stale?"
What I think y'all are seeing is "only" modifying "a sliding expiration". With that view, it's a proper sentence.

I'll take a crack at re-wording it in a suggested change comment.

@greyhillman this PR was merged some time ago. We don't want to invest anymore time on it, it's in good shape now. You've had two writers on this.

🤨

We don't want to invest anymore time on it, it's in good shape now.

I understand it's not that big of an issue, so not investing anymore time on it is fine, but it's not in good shape though:

"internal" was used instead of "interval" in a correction, and
the first sentence is still confusing.

You've had two writers on this.

I never asked for two writers, so I don't see how that's my fault or justifies dismissing me.
Also, it was coincidental that I made my comments at roughly the same time as the merge: I didn't have time to respond until then.


Temporarily disabled cache usage for v2

Temporarily disabling usage of pre-cached Node.js versions

Temporarily disabling usage of pre-cached Node.js versions

I just got a Renovate bot PR that wants to update action/setup-node from v2.3.0 to v2.3.1, and I'm really confused on why I as a end-user should update to the newer version.
Can somebody explain in a general sense what's going wrong right now, and how the temporary fix helps with this problem?
I've read the PR description, and checked the latest release notes, but they don't have a explanation at all.


Notification and Date Pannel are outside of the screen

Describe the bug
The panel is cut off and outside of the screen
To Reproduce
open any panel that goes to the right side
Expected behavior
The right side of the display is set wrong
Screenshots

Desktop (please complete the following information):

Distribution: [e.g. Arch Linux] Arch Linux
Window Manager/Desktop Environment: [e.g. Hyprland] Hyprland

Additional context
I have rebooted the system and only problems on the right side

Could you provide your screen resolution and any scaling/resolution related config that you have in your hyprland.conf?
And the output of echo $GDK_SCALE if you have any custom scaling.

I can replicate this, only I additionally have my "main menu" (top left arch logo button) also cut off at the bottom of my screen, left/right position is perfect.
I have no $GTK_SCALING parameter set, BUT I use plasma 6 side-by-side with hyprland and have plasma set to 150% zoom as I have a HiDPI monitor (1920x1080, scaling as if it was 1280x720). The only scaling parameter in hyprland.conf as far as I can see is for XCURSOR_SIZE=24, as the mouse was not scaled by default.

I'm getting the same. I tinkered around with the code but couldn't find anything. I put in some console.log()'s at DropdownMenu.ts and got the below output when opening the Dashboard on the left, vs the Notifications on the right. ("MarginRight/Left 1" is called right after being assigned, and "MarginRight/Left 2" is right after the "self.set_margin" calls.
Opening Dashboard
Gjs-Console-Message: 00:50:29.565: marginRight 1: 2322.5
Gjs-Console-Message: 00:50:29.565: marginLeft 1: -147.5
Gjs-Console-Message: 00:50:29.566: gdkScale:
Gjs-Console-Message: 00:50:29.566: value[0]: 45
Gjs-Console-Message: 00:50:29.566: monWidth: 2560
Gjs-Console-Message: 00:50:29.566: currentWidth: 385
Gjs-Console-Message: 00:50:29.566: marginRight 2: 2175
Gjs-Console-Message: 00:50:29.566: marginLeft 2: 0

Opening Notifications:
Gjs-Console-Message: 00:50:36.832: marginRight 1: 862.5
Gjs-Console-Message: 00:50:36.832: marginLeft 1: 1386.5
Gjs-Console-Message: 00:50:36.832: gdkScale:
Gjs-Console-Message: 00:50:36.832: value[0]: 1542
Gjs-Console-Message: 00:50:36.832: monWidth: 2560
Gjs-Console-Message: 00:50:36.832: currentWidth: 311
Gjs-Console-Message: 00:50:36.832: marginRight 2: 862.5
Gjs-Console-Message: 00:50:36.833: marginLeft 2: 1386.5

I can see that the out-of-bounds logic is working for the Dashboard as it corrects the negative margin-left, but the same doesn't work for margin-right.
NixOS if it matters.

@dringwald Thank for all that info, it's greatly appreciated. I'll have to look into why this is happening. If you wouldn't mind, can you log the global mouse position value?
You can do this by dropping this console.log(value) anywhere within the globalMousePos.connect scope.
The margins on both are being calculated properly, but I'm guessing that the position of the mouse click is being calculated incorrect so the correct margins are being applied on the wrong coordinates.

echo $GDK_SCALE

none, its empty

@Jas-SinghFSU
console.log(value) gets me 1542,20.
A console.log(clickPos) in modules/bar/utils.ts in openMenu gets me 1547,20. So nothing too different there.
Logging value after clicking on the media module (middle) and dashboard (left) gets me 800, 19 and 45, 24 respectively. So it seems to be scaling appropriately, it just thinks the monitor is ~1600 pixels wide.

Yeah, that's really odd... the click coordinate being gathered is assuming a different screen width than the margin setting logic...

@JulianTurner @dringwald @smathles Could you pull the branch menu_test  and let me know if that had any change?
Instead of using event.get_root_coords(), I instead get the cursor position using hyprctl cursorpos; this may hopefully fix it.

@Jas-SinghFSU  Hey, the issue still persist even on the new branch.

@Jas-SinghFSU I have the same issue as Julian, no change in this branch unfortunately. If I export GDK_SCALE=1.6, I do have the menus on the right hand side of the screen successfully be inside the screen, but they're still scaled much too big and I cannot see the bottom of the menus (also true for the scrollable elements inside the settings widget, they "finish" scrolling before I can actually see the bottom).
So the GDK_SCALE does indeed make the notification + date panel fall inside the screen on the right hand side, but it still doesn't affect the menu's scaling, which may or may not be the same problem.

Update: I tried Julian's solution  of setting monitor=,preferred,auto,auto to monitor=,preferred,auto,1 to set the scaling of the monitor to 1 instead of hyprland's automatic scaling of 1.5 (didn't realise hyprland was clever enough to sense HiDPI monitor and scale accordingly, learn something new everyday I guess). When scaled to 100% instead of the automatic 150%, it works (with the exception of the icons + spacing still being slightly off-center)!

The overall environment though is now a bit small for my liking, so IMO scaling for the ags widgets might be better? No idea how/if that could be implemented sorry.

This is super useful information. Thank you for sharing that; I will see if I can replicate it and handle the positioning better.

Update: I was able to replicate this give your awesome info. Debugging now!

@JulianTurner @dringwald @smathles Could you please git pull and verify if this Merge fixed your issue?

@Jas-SinghFSU Unfortunately it hasn't, at least not on the main branch. I have not set gdk_scaling to anything, menus still appear the same size as before (150%, desired 100% unlike the system environment).

@smathles Could you run hyprctl monitors and give me your output please?
Additionally if you could run git show and provide the commit id as well.

hyprctl monitors
Monitor eDP-1 (ID 0):
<EMAIL_ADDRESS>at 0x0
	description: InfoVision Optoelectronics (Kunshan) Co.Ltd China 0x057D
	make: InfoVision Optoelectronics (Kunshan) Co.,Ltd China
	model: 0x057D
	serial:
	active workspace: 3 (3)
	special workspace: 0 ()
	reserved: 0 36 0 0
	scale: 1.50
	transform: 0
	focused: yes
	dpmsStatus: 1
	vrr: 0
	activelyTearing: false
	disabled: false
	currentFormat: XRGB8888
	availableModes<EMAIL_ADDRESS>
git show
commit 089395207ccf5ddb70faa008850bb6134a95447f (HEAD -> master, origin/master, origin/HEAD)

Though I just read the section in the README regarding scaling based off of font size, manually set that to 0.8 instead of 1.2 and it works fine! Had an initial teething issue where the notifications/clock widget were too far to the right/left, but it sorted itself out after a few seconds (probably still initialising).
Not sure if that's best practice, to change the font size vs. hyprland global scaling, but it worked great for me! Still a few offsets in the icons, might make a separate issue for that though (no offset issues on my non-HiDPI main pc).

@smathles  Font size scaling your panel is very much intentional, hence why the default units are em and the entire application scales in such a way :). Happy that you've got that nicely scaled. As for the off-center fonts,  make sure that you're using a mono-space fonts; otherwise that just the downside of nerd-fonts since they use more than one cell unlike regular text :(.
How about the menu spawning? Is that still spawning your menus off screen or were you able to git pull the latest branch ad fix that aspect?

Menu spawning is perfect now, thanks!
Wasn't aware that mono-space fonts had a purpose like that, probably should have guessed from the name lol. I'll give that a go 👍.

Awesome :smile: ! Happy we got it all sorted!

@Jas-SinghFSU i alo can confirm it is working now, thanks buddy :-)


[IDEA] Privacy and security

Have you read the code of conduct?

[X] I have read the code of conduct

Has this idea already been presented?

[X] My idea has not been presented yet

Description of your idea

Tools

Services

[x]  Intro

[x] Messengers

[ ] Why defaults matter

[ ] Email

[ ] Big Tech and monopolies

[ ] Passwords/MFA

[ ] Browsers

[ ] Centralised, Decentralised and Distributed networks

[ ] VPNs and Tor

Links
https://privacyguides.org/
https://madaidans-insecurities.github.io/
https://grapheneos.org/
https://calyxos.org/
https://techlore.tech/
https://thenewoil.org/
https://privsec.dev/
https://qua3k.github.io/
https://wonderfall.dev/
Anything else?
No response

Web3 as part of Centralised, Decentralised and Distributed networks post


Reproducable OutOfMemory error on an open source Typescript repo analysis on google colab

Describe the bug
When I run the appinspector analyze as described below, on the public repository, I receive the following error:

=[09:42:06 INF] Progress bar is enabled so console output will be suppressed. No LogFilePath has been configured so you will not receive log messages
6n2;1H2;1H[09:42:16 ERR] Uncaught exception: ArgumentOutOfRangeException:Length cannot be less than zero. (Parameter 'length').    at System.String.Substring(Int32 startIndex, Int32 length)
at ShellProgressBar.StringExtensions.Excerpt(String phrase, Int32 length)
at ShellProgressBar.ProgressBar.ProgressBarBottomHalf(Double percentage, DateTime startDate, Nullable1 endDate, String message, Indentation[] indentation, Boolean progressBarOnBottom, Boolean showEstimatedDuration, TimeSpan estimatedDuration, Boolean disableBottomPercentage, String percentageFormat)    at ShellProgressBar.ProgressBar.UpdateProgress()    at ShellProgressBar.ProgressBar.Dispose()    at Microsoft.ApplicationInspector.Commands.AnalyzeCommand.GetResult() in D:\a\_work\1\s\AppInspector\Commands\AnalyzeCommand.cs:line 1026    at Microsoft.ApplicationInspector.CLI.Program.RunAnalyzeCommand(CLIAnalyzeCmdOptions cliOptions) in D:\a\_work\1\s\AppInspector.CLI\Program.cs:line 310    at Microsoft.ApplicationInspector.CLI.Program.VerifyOutputArgsAndRunAnalyze(CLIAnalyzeCmdOptions options) in D:\a\_work\1\s\AppInspector.CLI\Program.cs:line 124    at Microsoft.ApplicationInspector.CLI.Program.<>c.<Main>b__1_1(CLIAnalyzeCmdOptions cliAnalyzeCmdOptions) in D:\a\_work\1\s\AppInspector.CLI\Program.cs:line 41    at CommandLine.ParserResultExtensions.MapResult[T1,T2,T3,T4,T5,TResult](ParserResult1 result, Func2 parsedFunc1, Func2 parsedFunc2, Func2 parsedFunc3, Func2 parsedFunc4, Func2 parsedFunc5, Func2 notParsedFunc)
at Microsoft.ApplicationInspector.CLI.Program.Main(String[] args) in D:\a_work\1\s\AppInspector.CLI\Program.cs:line 40

To Reproduce
!sudo apt-get update && \
  sudo apt-get install -y dotnet-sdk-6.0
  # sudo apt-get install -y dotnet-sdk-8.0
!dotnet --list-sdks
!dotnet --version
!dotnet tool install --global Microsoft.CST.ApplicationInspector.CLI

!export PATH="$PATH:/root/.dotnet/tools"
! ls /root/.dotnet/tools/appinspector
!git clone https://github.com/lucgagan/auto-playwright
!ls -al /content/auto-playwright/src/
!/root/.dotnet/tools/appinspector analyze -s /content/auto-playwright/src/

Optionally you may add the following in a new cell and run it before running the appinspector analyze:
%%writefile ~/.bash_profile
# Add .NET Core SDK tools
export PATH="$PATH:/root/.dotnet/tools"

Operating Environment (please complete the following information):

Application Inspector Version: ApplicationInspector.CLI 1.9.18+00ad48b230
OS: Linux Ubuntu 22.04.3 LTS
Dotnet version 6.0.127 but also persists in version 8.0.102(restart and run the notebook by changing the commented out line).

This doesn't appear to be an out of memory error - it seems like an issue with the progress bar. I can't reproduce this on a normal terminal in Ubuntu 22.04.3, so this is likely caused by the unique environment you're running in - I'm not familiar with google colab but if it doesn't have an interactive terminal that may not play nicely with the redraws required to render the progress bar.
Since this seems to be a scripted run, you probably want to disable the progress bar anyway with the -N or --no-show-progress argument.

Thank you


fix protocol detection for secure sockets

@matthew342 so if I run document.location.protocol on github (https) chrome gives me 'https:'

sorry, read it wrong, nevermind

merged!  Thanks


Bot Analytics Platforms

Would like to know more about available Bot Analytics platforms that are compatible with MSFT BF.

@alokraj68 you can check https://www.dashbot.io/

http://botanalytics.co/
https://bot-metrics.com/

@chetanmekha , @slidenerd the integration parts of this platform is a pain.

You people are using any of this?

I have tried botanalytics but wasnt able to do so, here s working code for bot metrics
'use strict';
const
    constants = require('./constants'),
    request = require('request');

let botmetrics = require('node-botmetrics')(constants.BOT_METRICS);

botmetrics.trackIncoming = function (userId, text, platform) {
    if (platform !== 'emulator') {
        botmetrics.track({
            text: text,
            message_type: 'incoming',
            user_id: userId,
            platform: platform
        });
    }
}

botmetrics.trackIncomingSession = function (session) {
    if (session.message.address.channelId !== 'emulator') {
        botmetrics.track({
            text: session.message.text,
            message_type: 'incoming',
            user_id: session.message.user.id,
            platform: session.message.address.channelId
        });
    }
}

botmetrics.trackOutgoing = function (userId, text, platform) {
    if (platform !== 'emulator') {
        botmetrics.track({
            text: text,
            message_type: 'outgoing',
            user_id: userId,
            platform: platform
        });
    }
}

module.exports = botmetrics;

UPDATE
I am using dashbot now and its truly awesome. It integrates directly with the bot framework
Just specify the app id, password and facebook page access token. havent set slack, kik, skype and the other one in the code below. it works like a charm
const
    connector = new builder.ChatConnector({
        appId: constants.APP_ID,
        appPassword: constants.APP_PASSWORD
    }),
    bot = new builder.UniversalBot(connector),
    // only include tokens for the platforms that you support
    dashbotApiMap = {
        facebook: constants.DASHBOT_API_KEY_FACEBOOK,
        slack: process.env.DASHBOT_API_KEY_SLACK,
        kik: process.env.DASHBOT_API_KEY_KIK,
        webchat: process.env.DASHBOT_API_KEY_GENERIC,
        skype: process.env.DASHBOT_API_KEY_GENERIC
    },
    dashbot = require('dashbot')(dashbotApiMap).microsoft,

@alokraj68 Did you try dashbot? Still have questions? I'll close the issue but feel free to reopen if you have follow ups!

@slidenerd , @carlosscastro  I could not do the same because My bot framework nuget update ended up in crashing bot project. #2395

@alokraj68 If it isn't too late, please take a look at Ibex dashboard. Its built on top of Application Insights, so you'll need an Azure subscription.
To get analytics sent from your bot, you can use one of:

Node.js Instrumentation
C# Instrumentation (Work in-progress)

Already tried it, but bit hard to implement. and could not succeed the
deployment.
Thanks and Regards,
[image: Astro-Vision Future Tech] https://htmlsig.com/t/000001C1E7PD
Alok Rajasukumaran / Developer - Innovation Team
<EMAIL_ADDRESS>/<PHONE_NUMBER>7654
Astro-Vision Future Tech
Kuthappadi, Thamannam, ErnakulamKerala, India - 682032
alokraj68.in
[image: Twitter]  https://htmlsig.com/t/000001BVYZJE [image: Facebook]
https://htmlsig.com/t/000001BX9HG8 [image: Google +]
https://htmlsig.com/t/000001BV7H1J [image: LinkedIn]
https://htmlsig.com/t/000001C065D9 [image: Youtube]
https://htmlsig.com/t/000001C06RDJ [image: Github]
https://htmlsig.com/t/000001BYKTTT [image: Soundcloud]
https://htmlsig.com/t/000001BX996X [image: Stack Overflow]
https://htmlsig.com/t/000001BVE8RR
On 3 July 2017 at 03:26, Syed Hassaan Ahmed<EMAIL_ADDRESS>wrote:

@alokraj68 https://github.com/alokraj68 If it isn't too late, please
take a look at Ibex dashboard
https://github.com/CatalystCode/ibex-dashboard. Its built on top of Application
Insights
https://azure.microsoft.com/en-us/services/application-insights/, so
you'll need an Azure subscription.
To get analytics sent from your bot, you can use one of:

Node.js Instrumentation
https://github.com/CatalystCode/botbuilder-instrumentation
C# Instrumentation
https://github.com/CatalystCode/bot-sample-telemetry (Work
in-progress)

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/BotBuilder/issues/2012#issuecomment-312519472,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIyDm3KRlDg61EB3-WOShMR8IrmCJie-ks5sKBIigaJpZM4LgSvu
.

aha .. May I ask if you tried with Node.js or C# version? Which specific part was hard to implement? What was the error/issue you faced during the deployment?

I was working with C# version of bot and node js version of IBEX, the
analytics was not connecting from my application insights.
Thanks and Regards,
[image: Astro-Vision Future Tech] https://htmlsig.com/t/000001C1E7PD
Alok Rajasukumaran / Developer - Innovation Team
<EMAIL_ADDRESS>/<PHONE_NUMBER>7654
Astro-Vision Future Tech
Kuthappadi, Thamannam, ErnakulamKerala, India - 682032
alokraj68.in
[image: Twitter]  https://htmlsig.com/t/000001BVYZJE [image: Facebook]
https://htmlsig.com/t/000001BX9HG8 [image: Google +]
https://htmlsig.com/t/000001BV7H1J [image: LinkedIn]
https://htmlsig.com/t/000001C065D9 [image: Youtube]
https://htmlsig.com/t/000001C06RDJ [image: Github]
https://htmlsig.com/t/000001BYKTTT [image: Soundcloud]
https://htmlsig.com/t/000001BX996X [image: Stack Overflow]
https://htmlsig.com/t/000001BVE8RR
On 3 July 2017 at 17:44, Syed Hassaan Ahmed<EMAIL_ADDRESS>wrote:

aha .. May I ask if you tried with Node.js or C# version? Which specific
part was hard to implement? What was the error/issue you faced during the
deployment?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/BotBuilder/issues/2012#issuecomment-312629950,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIyDm7zrmqjJcBCYLEBgjse2VjA_i2K3ks5sKNs6gaJpZM4LgSvu
.

Ah right! .. As I mentioned above, C# instrumentation is a WIP .. We're tracking it here .. Hopefully it will resolve your issue once its done.

Wow, Thanks a lot @syedhassaanahmed am looking forward to it.

@slidenerd Can you share your project please. Thanks

Checked both https://bot-metrics.com/ and https://dashbot.io/ i and they only support nodejs for MSFT BF. Ibex dashboard looks really great as I saw on channel9 however I am looking for an option that doesn't involve App Insights yet because of its pricing. Are there other analytics platform for MSFT BF C#?

Better use  https://chatbase.com/overview

New update people, has any of you tried Chatbase from Google? for bot analytics


A11y: actions within the tree are not read by screen readers

VSCode Version: 1.0.1 - Insider
OS Version: Windows 10, NVDA

Steps to Reproduce:

Put workbench in tab mode
Tab into the working files section in the explorer
Observe: the x button to remove a file gets focus but nothing useful is read.

Seems to be true for any action in trees, I am quite sure this used to work but no longer is. No clue why so far...

I found out that the fact we assign role: tree seems to cause this and changing it to role: listbox seems to fix it. However I am not feeling comfortable making such a change because the role should actually be tree.
@zersiax any clue what might be going on? we have a tree with treeitem and each treeitem can have one or more button that have a tabindex to navigate to. Once I change the role from tree to listbox the screen reader (NVDA) correctly reads out the button when I navigate to it.

May have something to do with chromium bug 601933  https://bugs.chromium.org/p/chromium/issues/detail?id=601933
Have a look around the accessibility tree if something similar might be going on. I am guessing that changing the role to listbox breaks navigation among levels?

@zersiax interesting, though I see this even with Chrome 45. I have not tested any other a11y feature after changing to listbox. I still keep role treeitem for the elements within and that seems to at least let NVDA read out the collapsed/expanded state and hierarchy level of the tree :)

To keep the number of issues in our inbox on a manageable level, we're closing issues that have been on the backlog for a long time but haven't gained traction: We look at the number of votes the issue has received and the number of duplicates issues filed.
Thanks for your understanding. Happy coding!


Auto Scroll Down and a shortcut key to enable / disable it

Description of the new feature / enhancement
Please add the option to auto scroll the current chapter in scroll speed (with keys) column.
Scenario when this would be used?
To auto scroll through the current reading chapter in epub and comic.
Supporting information
Please add an option to enable / disable auto scroll.
The auto scroll px value should be enterable / changeable even when the option is disabled.
Because if auto scroll is enabled it will scroll through automatically before entering the desired value without this.
The enterable scroll value in auto scroll is of px, the same as scroll speeds.

You can use middle mouse button. With it you can choose direction as well as speed.


Load automatically jest.[json|yaml|etc...]

Do you want to request a feature or report a bug?
Feature
What is the current behavior?
Currently, Jest support json and other formats with the --config jest.json option.
What is the expected behavior?
Loads the configuration file automatically following the pattern jest.[format] or jest.config.[format].
Please provide your exact Jest configuration and mention your Jest, node, yarn/npm version and operating system.
Jest 20.0.4
NPM 4.1.2
Windows 10 x64

We won't be supporting other formats at this point. We already support jest.config.js.

@cpojer The yaml|etc... was just an extrapolation, you already support json the idea is to load it without requiring --config like eslint does.


suspected crash bug: hitting debug assert in tcp_proxy

I set up Envoy with a single listener that has a tcp_proxy filter, with TCP tunneling enabled, where the upstream cluster has http2_protocol_options set, meaning that upsream codec is expected to be HTTP/2.
The problem I'm facing is that when there's a high throughput load tunneled through the proxy filter, the following line causes the process to crash in debug build:
https://github.com/envoyproxy/envoy/blob/0ea68d2eb56a59be4216fcf57eefe243d5968e19/source/common/tcp_proxy/tcp_proxy.cc#L287
This was also mentioned in #24839, but I think that now after some digging and learning the code, I have better insights, therefore opened a new issue to escalate this. After reading the flow_control#http2-implementation-details to try and understand how the flow control works, I noticed that for HTTP/2 streams it's possible that there are two flow control blocks - one at the network level and one at the H2 level. Considering that if I comment out the assert calls - the tunneling flow works correctly, I think that this is what's happening in this case (will be explained through logs below), and the fix to the code would be either removing the assertions, or changing the assertion so multiple watermark calls may be called.
The following trace logs are provided for the when the assert calls are not commented out (later I commented out the assert calls):
... Removed lines about socket ready/write/ssl etc.. (too many to include here) ...
... Removed lines about socket ready/write/ssl etc.. (too many to include here) ...

connection_impl.cc:530       [C2] onAboveReadBufferHighWatermark
connection_impl.cc:362       [C2] readDisable: disable=true disable_count=0 state=0 buffer_length=1048585
tcp_proxy.cc:673             [C2] downstream connection received 1048585 bytes, end_stream=false
connection_impl.cc:523       [C2] onBelowReadBufferLowWatermark
connection_impl.cc:362       [C2] readDisable: disable=false disable_count=1 state=0 buffer_length=0
codec_impl.cc:586            [C3] send buffer over limit
upstream.cc:119              HttpUpstream::onAboveWriteBufferHighWatermark
tcp_proxy.cc:338             [C2] Filter::UpstreamCallbacks::onAboveWriteBufferHighWatermark
connection_impl.cc:362       [C2] readDisable: disable=true disable_count=0 state=0 buffer_length=0

... Removed lines about socket ready/write/ssl etc.. (too many to include here) ...

codec_impl.cc:593            [C3] send buffer under limit
upstream.cc:124              HttpUpstream::onBelowWriteBufferLowWatermark
tcp_proxy.cc:349             [C2] Filter::UpstreamCallbacks::onBelowWriteBufferLowWatermark
connection_impl.cc:362       [C2] readDisable: disable=false disable_count=1 state=0 buffer_length=0
connection_impl.cc:483       [C3] writing 16393 bytes, end_stream false
connection_impl.cc:548       [C3] onAboveWriteBufferHighWatermark
upstream.cc:119              HttpUpstream::onAboveWriteBufferHighWatermark
tcp_proxy.cc:338             [C2] Filter::UpstreamCallbacks::onAboveWriteBufferHighWatermark
connection_impl.cc:362       [C2] readDisable: disable=true disable_count=0 state=0 buffer_length=0

... Removed lines about socket ready/write/ssl etc.. (too many to include here) ...

connection_impl.cc:537       [C3] onBelowWriteBufferLowWatermark
upstream.cc:124              HttpUpstream::onBelowWriteBufferLowWatermark
tcp_proxy.cc:349             [C2] Filter::UpstreamCallbacks::onBelowWriteBufferLowWatermark
connection_impl.cc:362       [C2] readDisable: disable=false disable_count=1 state=0 buffer_length=0

... Removed lines about socket ready/write/ssl etc.. (too many to include here) ...

connection_impl.cc:530       [C2] onAboveReadBufferHighWatermark
connection_impl.cc:362       [C2] readDisable: disable=true disable_count=0 state=0 buffer_length=1048585
tcp_proxy.cc:673             [C2] downstream connection received 1048585 bytes, end_stream=false
connection_impl.cc:523       [C2] onBelowReadBufferLowWatermark
connection_impl.cc:362       [C2] readDisable: disable=false disable_count=1 state=0 buffer_length=0
codec_impl.cc:586            [C3] send buffer over limit             ### Here the high watermark is called by codec_impl
upstream.cc:119              HttpUpstream::onAboveWriteBufferHighWatermark
tcp_proxy.cc:338             [C2] Filter::UpstreamCallbacks::onAboveWriteBufferHighWatermark
connection_impl.cc:362       [C2] readDisable: disable=true disable_count=0 state=0 buffer_length=0

... Removed lines about socket ready/write/ssl etc.. (too many to include here) ...

connection_impl.cc:548       [C3] onAboveWriteBufferHighWatermark    ### Here the high watermark is called by connection_impl
upstream.cc:119              HttpUpstream::onAboveWriteBufferHighWatermark
tcp_proxy.cc:338             [C2] Filter::UpstreamCallbacks::onAboveWriteBufferHighWatermark
tcp_proxy.cc:339             assert failure: !on_high_watermark_called_.

backtrace.h:104 Caught Aborted, suspect faulting address 0x19
backtrace.h:91 Backtrace (use tools/stack_decode.py to get line numbers):
backtrace.h:92 Envoy version: 10604c0031a7ab1b0fdbe1aeced568a6aa41ad4f/1.26.1-dev/Modified/DEBUG/BoringSSL
backtrace.h:96 #0: Envoy::SignalAction::sigHandler() [0x556624937288]
backtrace.h:96 #1: __restore_rt [0x7f9f80bdde30]

ConnectionImpl 0x466bf893450, connecting_: 0, bind_error_: 0, state(): Open, read_buffer_limit_: 1048576
socket_:
  ListenSocketImpl 0x466bf7ab780, transport_protocol_: raw_buffer
  connection_info_provider_:

Also marked inline - the first onAboveWriteBufferHighWatermark is called at codec_impl.cc:586, then called at connection_impl.cc:548, and both are calling back HttpUpstream::onAboveWriteBufferHighWatermark at tcp_proxy/upstream.cc:119.
The following trace logs are provided for the when the assert calls are commented out:
... Removed lines about socket ready/write/ssl etc.. (too many to include here) ...

tcp_proxy.cc:673             [C2] downstream connection received 119289 bytes, end_stream=false
codec_impl.cc:586            [C3] send buffer over limit
upstream.cc:119              HttpUpstream::onAboveWriteBufferHighWatermark
tcp_proxy.cc:338             [C2] Filter::UpstreamCallbacks::onAboveWriteBufferHighWatermark
connection_impl.cc:362       [C2] readDisable: disable=true disable_count=0 state=0 buffer_length=0
connection_impl.cc:483       [C3] writing 16393 bytes, end_stream false
codec_impl.cc:1177           [C3] sent frame type=0, stream_id=1, length=16384
codec_impl.cc:593            [C3] send buffer under limit
upstream.cc:124              HttpUpstream::onBelowWriteBufferLowWatermark
tcp_proxy.cc:349             [C2] Filter::UpstreamCallbacks::onBelowWriteBufferLowWatermark

... Removed lines about socket ready/write/ssl etc.. (too many to include here) ...

connection_impl.cc:530       [C2] onAboveReadBufferHighWatermark
connection_impl.cc:362       [C2] readDisable: disable=true disable_count=0 state=0 buffer_length=1048585
tcp_proxy.cc:673             [C2] downstream connection received 1048585 bytes, end_stream=false
connection_impl.cc:523       [C2] onBelowReadBufferLowWatermark
connection_impl.cc:362       [C2] readDisable: disable=false disable_count=1 state=0 buffer_length=0
codec_impl.cc:586            [C3] send buffer over limit
upstream.cc:119              HttpUpstream::onAboveWriteBufferHighWatermark
tcp_proxy.cc:338             [C2] Filter::UpstreamCallbacks::onAboveWriteBufferHighWatermark
connection_impl.cc:362       [C2] readDisable: disable=true disable_count=0 state=0 buffer_length=0

... Removed lines about socket ready/write/ssl etc.. (too many to include here) ...

connection_impl.cc:548       [C3] onAboveWriteBufferHighWatermark    ### This call would cause assert hit, if not commented
upstream.cc:119              HttpUpstream::onAboveWriteBufferHighWatermark
tcp_proxy.cc:338             [C2] Filter::UpstreamCallbacks::onAboveWriteBufferHighWatermark

... Removed lines about socket ready/write/ssl etc.. (too many to include here) ...

codec_impl.cc:593            [C3] send buffer under limit
upstream.cc:124              HttpUpstream::onBelowWriteBufferLowWatermark
tcp_proxy.cc:349             [C2] Filter::UpstreamCallbacks::onBelowWriteBufferLowWatermark
connection_impl.cc:362       [C2] readDisable: disable=false disable_count=2 state=0 buffer_length=0

... Removed lines about socket ready/write/ssl etc.. (too many to include here) ...

connection_impl.cc:537       [C3] onBelowWriteBufferLowWatermark
upstream.cc:124              HttpUpstream::onBelowWriteBufferLowWatermark
tcp_proxy.cc:349             [C2] Filter::UpstreamCallbacks::onBelowWriteBufferLowWatermark
connection_impl.cc:362       [C2] readDisable: disable=false disable_count=1 state=0 buffer_length=0

... Removed lines about socket ready/write/ssl etc.. (too many to include here) ...

connection_impl.cc:530       [C2] onAboveReadBufferHighWatermark
connection_impl.cc:362       [C2] readDisable: disable=true disable_count=0 state=0 buffer_length=1048585
tcp_proxy.cc:673             [C2] downstream connection received 1048585 bytes, end_stream=false
connection_impl.cc:523       [C2] onBelowReadBufferLowWatermark
connection_impl.cc:362       [C2] readDisable: disable=false disable_count=1 state=0 buffer_length=0
codec_impl.cc:586            [C3] send buffer over limit
upstream.cc:119              HttpUpstream::onAboveWriteBufferHighWatermark
tcp_proxy.cc:338             [C2] Filter::UpstreamCallbacks::onAboveWriteBufferHighWatermark
connection_impl.cc:362       [C2] readDisable: disable=true disable_count=0 state=0 buffer_length=0

In this case there are alternating connection_impl.cc:548 - onAboveWriteBufferHighWatermark and codec_impl.cc:586 - send buffer over limit but each one also gets the lowWatermark call too.
I'll appreciate help verifying if this is the case, before opening a PR to fix/remove the ASSERT calls. cc @alyssawilk, @KBaichoo, @ggreenway

cc @alyssawilk

At first glance I think the assert is correct, and we need to change the TCP proxy filter to handle multiple calls and unwind, like the HTTP stack does, or the stack can get stalled.  @ohadvano would you be game for crafting an e2e test to exploit this and trying the fix or would you prefer kevin or I tackle. it?

if nothing else wouldn't that result in the stats being off for number of disables?  generally those track actual disabling, not the number of times the disable call is called.

arguably we could audit all the endpoints to make sure they're safe, and update the stats more frequently.  cc @KBaichoo for thoughts

if nothing else wouldn't that result in the stats being off for number of disables? generally those track actual disabling, not the number of times the disable call is called.

That's a good point, seems like indeed stats would be reported x2 times in such cases, but seems easily tweakable in:
https://github.com/envoyproxy/envoy/blob/c0dce839eda044eaccf219896809a0d8bbd1bb53/source/common/tcp_proxy/tcp_proxy.cc#L298
By remembering if it was already called with disable==true before, wdyt?

generally those track actual disabling, not the number of times the disable call is called.

+1
Since the connection itself is already tracking the number, ISTM that modifying readDisable
https://github.com/envoyproxy/envoy/blob/c0dce839eda044eaccf219896809a0d8bbd1bb53/source/common/tcp_proxy/tcp_proxy.cc#L296
To return status and increment the stat after that based on the output of that status.
Statuses could be: TransitionedToReadEnabled, TransitionedToReadDisable, ReadDisable.

I'm less familiar with HCM code, but when I think about it, isn't this problem also relevant to HCM?
https://github.com/envoyproxy/envoy/blob/fc5361a35bcfeeae8603e5d3957fefddf9d68a4d/source/common/http/conn_manager_impl.cc#L1752
Alternatively asking, is onDecoderFilterAboveWriteBufferHighWatermark called only when the H2 stack fires the high watermark buffer, or also when connection stack calls it, similar as what's happening for tcp_proxy?
Also @KBaichoo I guess your suggestion also makes sense, just asking out of curiosity - what benefit does it have over managing this Boolean state in the tcp_proxy itself rather than modifying readDisable() signature?

The HTTP stack can unwind multiple read disable.

Also @KBaichoo I guess your suggestion also makes sense, just asking out of curiosity - what benefit does it have over managing this Boolean state in the tcp_proxy itself rather than modifying readDisable() signature?

The readDisable call is effectively doing this work already so we're just no longer throwing away useful information
Since we're now returning a superset of returned values (prior was null set as this was void), we can just ignore it in places it's not being used and update the places where it's useful e.g. TCP proxy.


Allow merging of IFDs with different offset types.

With the caveat that I barely know anything about tiff images or this code, this change fixes an error I was having trying to load a file.
Stack trace for said error:
ERROR: LoadError: MethodError: no method matching merge(::TiffImages.IFD{UInt64}, ::TiffImages.IFD{UInt32})
Closest candidates are:
  merge(::TiffImages.IFD{O}, ::TiffImages.IFD{O}) where O<:Unsigned at /home/dcjones/.julia/dev/TiffImages/src/ifds.jl:71
  merge(::NamedTuple, ::Any) at namedtuple.jl:277
Stacktrace:
  [1] (::TiffImages.var"#3#4")(x::Tuple{TiffImages.IFD{UInt64}, TiffImages.IFD{UInt32}})
    @ TiffImages ~/.julia/dev/TiffImages/src/types/dense.jl:22
  [2] iterate
    @ ./generator.jl:47 [inlined]
  [3] collect(itr::Base.Generator{Base.Iterators.Zip{Tuple{Vector{TiffImages.IFD{UInt64}}, Vector{TiffImages.IFD{UInt32}}}}, TiffImages.var"#3#4"})
    @ Base ./array.jl:681
  [4] map(f::Function, A::Base.Iterators.Zip{Tuple{Vector{TiffImages.IFD{UInt64}}, Vector{TiffImages.IFD{UInt32}}}})
    @ Base ./abstractarray.jl:2323
  [5] TiffImages.DenseTaggedImage(data::Array{ColorTypes.Gray{FixedPointNumbers.N0f16}, 3}, ifds::Vector{TiffImages.IFD{UInt64}})
    @ TiffImages ~/.julia/dev/TiffImages/src/types/dense.jl:22
  [6] load(tf::TiffImages.TiffFile{UInt64, FileIO.Stream{FileIO.DataFormat{:TIFF}, IOStream, String}}; verbose::Bool, mmap::Bool)
    @ TiffImages ~/.julia/dev/TiffImages/src/load.jl:30
  [7] load(io::IOStream; verbose::Bool, mmap::Bool)
    @ TiffImages ~/.julia/dev/TiffImages/src/load.jl:7
  [8] #8
    @ ~/.julia/dev/TiffImages/src/load.jl:3 [inlined]
  [9] open(f::TiffImages.var"#8#9"{Bool, Bool}, args::String; kwargs::Base.Iterators.Pairs{Union{}, Union{}, Tuple{}, NamedTuple{(), Tuple{}}})
    @ Base ./io.jl:330
 [10] open
    @ ./io.jl:328 [inlined]
 [11] #load#7
    @ ~/.julia/dev/TiffImages/src/load.jl:2 [inlined]
 [12] load(filepath::String)
    @ TiffImages ~/.julia/dev/TiffImages/src/load.jl:2

Oof. What kind of image/condition triggered this error?
My guess is that it should only happen for images that very close to max size of the TIFF container.

The file was from a public CODEX dataset. I don't know that there's anything exceptional about it. It's about 1GB if you'd like to try to reproduce the issue:
https://assets.hubmapconsortium.org/2f8d43199ca3167e991b320038024b77/output/extract/expressions/ome-tiff/R001_X001_Y001.ome.tiff

Ooo. That's interesting. It's a 1GB BigTIFF, which tricks TiffImages.jl's logic. Usually people don't use BigTIFFs (w/64bit offsets) unless the file is larger than 4GB since BigTIFFs use twice as much space for each offset versus normal TIFF's 32bit offsets. I haven't seen that before.

Codecov Report

Merging #69 (6b08c28) into master (a1f00c1) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master      #69   +/-   ##
=======================================
  Coverage   90.87%   90.87%
=======================================
  Files          12       12
  Lines         603      603
=======================================
  Hits          548      548
  Misses         55       55

Impacted Files
Coverage Δ

src/ifds.jl
98.40% <100.00%> (ø)

Continue to review full report at Codecov.

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

Thanks @dcjones. I'm still a little disappointed by TiffImages.jl speed at opening the file, but that seems to be related to #19, we really need to decompress separately from reading to speed up the process.

Thanks! Glad I could provide an interesting tiff specimen.


Option to use SqlDelightStatement instead of SQLiteStatement for insert/update

We use https://github.com/pushtorefresh/storio for DB operations and don't use SQLiteDatabase directly, so new binder for insert/delete/update is not quite convenient.
Can you please add some option to switch insert/delete/update operations to select-like (i.e. SqlDelightStatement) binding (which is great indeed)?

Does storio not have the ability to use compiled statements? The goal is to actively discourage users from not precompiling insert/updates/deletes since its much less efficient.

Not yet: https://github.com/pushtorefresh/storio/issues/427
The goal is pretty clear for sure, but a little option would be nice.


API review changes to ChangeTracker

Use IAccessor for StateManager
Remove all but most basic TrackGraph methods

Includes removal of related infrastcuture
Made the KeyValueEntityTracker public so it can be easily used with TrackGraph

:shipit:


ensure $HOME and Dir.tmpdir are writable

Fixes #5518
What was the end-user problem that led to this PR?
A user had issues installing gems due to a permissions issue on the temp dir, because
Bundler was not checking for proper permissions on the temp directory or $HOME.
What was your diagnosis of the problem?
After discussing the issue with @colby-swandale, the solution was to check permissions on $HOME and the tmpdir.
What is your fix for the problem, implemented in this PR?
The creation of the temp dir is now wrapped in the block passed to filesystem_access, so filesystem_access will rescue the Errno::EACCES exception which is thrown if the effective user of the bundler process doesn't have sufficient permissions to create the temp dir.
Why did you choose this fix out of the possible options?
I chose this fix because it's what was discussed in the original issue thread.

No worries, I will remove the validation on $HOME, and just leave the validation on the temp directory.

Travis failed here, because the Ruby interpreter exploded. The spec passed locally for me; is this possibly a Travis issue?

Travis failed here, because the Ruby interpreter blew up. I tried to recreate the failure locally using Ruby 1.9.3-p551, but the specs all pass. It looks like a Jenkins specific issue, and I'm not sure how to fix it.

Awesome!
@bundlerbot r+


Crash when dereferencing invalid string

Reproduce:

Open the game project
Open resources
Add a scene by right-clicking resources
Select the added scene
Add a model by right-clicking resources
Crash in SceneEditor::Show, it seems that scene is a dangling pointer.

Probable cause: resources are stored in a vector in ResourceFolder in ResourceList. When adding a scene, the vector is resized and so the resource's scene string is no longer valid.


Add importance indicator to WOF neighborhoods

It's hard to tell if one neighborhood is more important than another and labels are crowding. We can use area as a rough proxy, and also number of contributing photos or some direct indicator from Who's On First like min_zoom / scale rank.
#14/37.7583/-122.4419

We're only pulling down the meta file at the moment, so we're limited in terms of the information we have available to us. The bounding box is included there, and it should mostly be able to serve as a proxy for calculating the area. The edge case would be a neighborhood that is a thin diagonal.
Or do you think that we'll need to gain access to the raw files themselves eventually anyway, in which case we should start going down that path sooner rather than later. Would we want to crawl the data every time we import? Or is there some sort of other api planned for this sort of thing? There are about 50k neighbourhoods at the moment.

We can keep the hash column in the database as a "version" or ETag, so we only need to re-fetch those for which the meta file tells us the hash has changed. But still, it's a lot of extra work to pull those down, parse them, etc...
Is area a good enough indicator of importance to justify this? misc:photo_sum has been mentioned as an alternative, but does that have universal coverage?

Let's do tile_rank using photo counts from WOF/Quattroshapes and tie break based on area. Also include area in tile properties.

Are we inventing scalerank: 12 on all these features? Is that a legacy from the OSM sourced neighborhoods?

Are we inventing scalerank: 12 on all these features? Is that a legacy from the OSM sourced neighborhoods?

It is. I think it started coming from the ne dataset, and then we added some default values in a transform.
Should we put the new values calculated from the WOF values into this existing scalerank property instead of making a new tile_rank attribute?

No, these new WOF features shouldn't get a scalerank property. Instead we should expose the min_zoom and max_zoom properties (sourced from mz:min_zoom and mz:max_zoom), as well as a is_aoi(sourced frommz:is_aoi`).
The tile_rank should be kind_tile_rank (pending other discussion in the station issue), and should be sorted first on min_zoom and then on photos_sum.

This tile doesn't make sense. All the features with min_zoom of 12 have a high kind_tile_rank, and features with min_zoom of 13 have a low kind_tile_rank. The opposite should be true.

http://vector.dev.mapzen.com/osm/all/12/654/1583.topojson?api_key=vector-tiles-HqUVidw

This feature has a min_zoom of 13, but kind_tile_rank of 1.
{
type: "Point",
properties: {
kind: "neighbourhood",
name: "Inner Richmond",
area: 2314979,
mz_n_photos: 53065,
max_zoom: 19,
kind_tile_rank: 1,
source: "whosonfirst",
min_zoom: 13,
id: 85865919
},

and
{
type: "Point",
properties: {
kind: "neighbourhood",
name: "Richmond District",
area: 1623917,
mz_n_photos: 3082,
max_zoom: 13,
kind_tile_rank: 10,
source: "whosonfirst",
min_zoom: 12,
id: 85869003
},

Hmm... I think this is because the ranking function assigns ranks in order; the first matching item gets 1, the second 2, etc... But I think we sometimes sort items in the layer so that the more important ones come last, which would be the opposite order.
For "painters algorithm", having more important things last is the right way to do it, but (IIRC) all drawing in Tangram is based on assigned z-index, which is independent of the order of the data in the tile. Also, when drawing things with collision/exclusion boxes (i.e: any icons or text), the things which are drawn first are the most important.
We should make sure that we know, for each layer, which order is "correct", and that we have that documented somewhere, even if only as a comment in a YAML file. My thought is that we should order any "label" layer most-to-least, and any "polygon" or "line" layer as least-to-most, but we do have a bunch of layers where we mix labels and other features.
@nvkelso what do you think?

In this case it looks like I just never added the min_zoom to the places layer sort. I'll make a pr to address that.

Works now :)


Crash on restart after changing Icon theme

Old icon theme was not perfect, but the new one is just ugly. YMMV anyway. As long as I switch to the Spyder 2 theme, everything is fine, but Spyder 3.0.2 (stable one from Ubuntu16.10) crashes while trying to restart itself after changing the setting.
What steps will reproduce the problem?

Segmentation fault (core dumped)
$ Traceback (most recent call last):
  File "/usr/lib/python2.7/dist-packages/spyder/app/restart.py", line 279, in <module>
    main()
  File "/usr/lib/python2.7/dist-packages/spyder/app/restart.py", line 232, in main
    restarter.launch_error_message(error_type=CLOSE_ERROR)
  File "/usr/lib/python2.7/dist-packages/spyder/app/restart.py", line 152, in launch_error_message
    raise RuntimeError(message)
RuntimeError: It was not possible to close the previous Spyder instance.
Restart aborted.

Parent window becomes closed, but both preferences and restart dialog stay on screen while new splash screen appears with a "closing spyder" message.

Ubuntu bug reporting window is not resizeable and doesn't allow to copy contents, but that's the only clue there.
Versions and main components

Spyder Version: 3.0.2 (Ubuntu 16.10)

What happens if you press No and manually restart Spyder?

If I press No and manually restart, it work as expected - icons change without crashes.

I wasn't able to reproduce this error 😞

Closing because we were unable to reproduce this error.

I am not using Spyder anymore.


CB-9011 Adding Buildbot feature support in medic

Following are notable changes.

Added commit-triggered builds.
Now using Buildbot Codebases for each codebase; no more downloading repo configs.
Made code more modular and less repetitive.
Added GitHub hook support for test installations of Medic (the Apache Infrastructure installation of Medic has a different master.conf).
Periodic builds no longer run if no changes are present.

This is a significant improvement in our BuildBot capability! Let's get this in and up on Apache CI.

+1


Unable to install Clang.jl on Julia master

julia> Pkg.build("Clang")
    Building Clang → `~/.julia/scratchspaces/44cfe95a-1eb2-52ea-b672-e2afdf69b78f/461757522a8e312a2394efbf83668cef58469ca4/build.log`
ERROR: Error building `Clang`:
┌ Warning: Platform `arm64-apple-darwin23.2.0` is not an officially supported platform
└ @ BinaryProvider ~/.julia/packages/BinaryProvider/U2dKK/src/PlatformNames.jl:450
ERROR: LoadError: KeyError: key "unknown" not found
Stacktrace:
  [1] getindex
    @ Base ./dict.jl:510 [inlined]
  [2] parse_dl_name_version
    @ BinaryProvider ~/.julia/packages/BinaryProvider/U2dKK/src/PlatformNames.jl:511 [inlined]

Perhaps the same as #234?

I'm using a very old Clang.jl, it turns out.

The issue seems to be the lack of availability of an updated libLLVM_jll, which is forcing Clang.jl down to an old version.
(@v1.11) pkg> add<EMAIL_ADDRESS>   Resolving package versions...
ERROR: Unsatisfiable requirements detected for package libLLVM_jll [8f36deef]:
 libLLVM_jll [8f36deef] log:
 ├─possible versions are: 16.0.6 or uninstalled
 └─found to have no compatible versions left with Clang_jll [0ee61d77]
   └─Clang_jll [0ee61d77] log:
     ├─possible versions are: 9.0.1 - 17.0.6 or uninstalled
     └─restricted by compatibility requirements with Clang [40e3b903] to versions: 12.0.0 - 15.0.7
       └─Clang [40e3b903] log:
         ├─possible versions are: 0.7.0 - 0.17.8 or uninstalled
         └─restricted to versions 0.17.8 by an explicit requirement, leaving only versions: 0.17.8


xrmBrowser.Navigation.OpenSubArea("Sales", "Opportunities")

This works some times and most of the time it gives error
: System.NullReferenceException: 'Object reference not set to an instance of an object.'
No Inner Exception: : Null
Stack Trace:
at Microsoft.Dynamics365.UIAutomation.Api.XrmNavigationPage.<>c__DisplayClass3_0.b__0(IWebDriver driver)
at Microsoft.Dynamics365.UIAutomation.Browser.DelegateBrowserCommand1.ExecuteCommand(IWebDriver driver, Object[] params)    at Microsoft.Dynamics365.UIAutomation.Browser.BrowserCommand1.Execute[T1,T2,T3,T4,T5,T6,T7,T8,T9](IWebDriver driver, T1 p1, T2 p2, T3 p3, T4 p4, T5 p5, T6 p6, T7 p7, T8 p8, T9 p9)
at Microsoft.Dynamics365.UIAutomation.Browser.BrowserCommand1.Execute(IWebDriver driver)    at Microsoft.Dynamics365.UIAutomation.Browser.BrowserPage.Execute[TResult](BrowserCommandOptions options, Func2 delegate)
at Microsoft.Dynamics365.UIAutomation.Api.XrmNavigationPage.OpenSubArea(String area, String subArea, Int32 thinkTime)

Another error I am getting while using xrmBrowser.Navigation.OpenSubArea:
System.TypeLoadException occurred
HResult=0x80131522
Message=Could not load type 'OpenQA.Selenium.Support.UI.WebDriverWait' from assembly 'WebDriver.Support, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null'.
Source=Microsoft.Dynamics365.UIAutomation.Browser

It looks like you may be missing the Selenium Support assembly.    Can you check that the Nuget Package is installed and on the correct version?  You may have updated another package and it's looking for 3.8 version of WebDriver.Support

Thanks for your help Daren; this is resolved.
Project I have created has latest Nuget package but "Microsoft.Dynamics365.UIAutomation.Api" and "Microsoft.Dynamics365.UIAutomation.Browser" referring to older versions of Selenium Support so I updated and its started working.


x/tools/internal/lsp: consider rethinking the details field of completion candidates

Currently, the details field of completion candidates is of limited value in editors (e.g the preview window in Vim -- not the candidates list).
I'd like to propose that the details field should contain as much information as the label field of completion candidates plus more. Probably, it should be the full type definition for types and the declaration for variables and constants. For functions and methods, the full declaration would be useful: name (along with receive for methods), parameters, and return values.
Contrast that with the current details. Here are two examples: the details field only contains the type name for constants and only has return values for functions and methods.
Here's a screenshot of completing constants in vim-go. The details field of the completion candidates is shown in the top window:
https://user-images.githubusercontent.com/1958864/57467163-e6f30f00-7236-11e9-8ce3-fe6a685b763d.png
Here's a similar screenshot of completing a method in vim-go. Again, the details field of the completion candidates is shown in the top window:
https://user-images.githubusercontent.com/1958864/57467081-b90dca80-7236-11e9-964c-e1f1e3f5cbdb.png
Relates to #31923

According to the LSP spec, detail is meant to contain "additional information about this item, like type or symbol information", which is why the detail is done this way.
An example in VSCode:

It seems to me that maybe Vim should behave differently in this case? Detail is meant to augment the information provided in the label, not entirely replace it.

That's understandable.
Changing vim itself isn't likely to happen, but I can certainly use the details in vim-go more like you suggest. The one complication is methods and functions with more than one return value: parsing the details string correctly to decide when it should be wrapped with parentheses is tricky. Could we have gopls wrap the details with a set of parentheses when the function or method has multiple return values since gopls actually knows how many return values exist?

Yep, gopls does that.

(Depending on the length of the label and detail, VSCode displays things differently.)

Sounds good to me. Perhaps that changed recently? I've not seen the parentheses wrapping multiple return values in the details field...


CODE128 barcode displays incorrectly

As you can see from the image, the top/black barcode is one generated using JsBarcode. The bottom/red one is what I'm actually trying to generate. They are both CODE128, but it seems that the CODE128 Auto setting slightly misrepresents a small portion of the barcode (indicated by the green arrows). The project BWIP-JS generates it as expected, but I'd prefer to use JsBarcode instead, as I've been using it for a little over a year now. What would I need to do to solve the minor discrepancy in the generated barcode?

thanks! BWIP-JS  works fine

These barcodes both represent the same value.
The difference is because the number is not an even number of digits.
So the first 18 digits are encoded using CODE 128C and the last digit needs to be encoded using CODE 128A or CODE 128B.
The choice of whether to use 128A or 128B is arbitrary because character being encoded (1) is a single symbol in both.
The symbol to switch to 128A is<PHONE_NUMBER>0. This is what the "Generated" barcode is using.
The symbol to switch to 128B is<PHONE_NUMBER>0. This is what the "Expected" barcode is using.
The symbol for 1 is the same in both 128A and 128B<PHONE_NUMBER>0), so both barcodes look the same there.
The next symbol is the checksum. Because one barcode is using the 128A symbol before the last 1 digit and the other is using the 128B symbol, the checksum is different. That is why the barcodes differ after the 1 digit.
The stop symbol and final bar at the end are the same for both again.
So if you scan either of these barcodes you will get the same value.


feat(sass-lint): added border-zero sass linting rule

PR Checklist
Please check if your PR fulfills the following requirements:

[ ] The commit message follows our guidelines: https://github.com/graycoreio/daffodil/blob/master/CONTRIBUTING.md#commit
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?

Fixes: N/A
What is the new behavior?
Does this PR introduce a breaking change?
[ ] Yes
[ ] No

Other information

This needs a rebase on develop.


CapricornConsultingGmbH.Vigoon version 2.4.2261

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.0 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Ah, no i understand. You mean the version of the .msi file. Okay, i uploaded a new pull request with correct version and will close this one. thanks


OPENABC2_DATASET contains only step0 files ?

After downloading and unzipping OPENABC2_DATASET.zip (Torch tensor format), I see only step0 files in the processed/ directory. Dont we have AIGs for all the 20 steps ?

Hi @udaymallappa,
Sorry, the zenodo only allowed dataset upto 50GB at the time when I uploaded the dataset. The original dataset do have all the AIGs processed for each step. The reason for keeping this dataset short was to predict the QoR using synthesis recipe and the starting AIG only.
If you face the issue downloading and unzipping the original dataset, let me know.

Thanks for your quick response.
I am particularly interested in the ML-ready torch dataset and the original dataset has a lot of other information that I do not need.  In any case, I was trying to load the *pt files corresponding to step0, but run into data loading issues as a result of torch version.
Also, does "step0"  correspond to unoptimized start AIG circuits ? If so, does 1500 synthesis recipes for each step0 flavor would be identical right ?

Yes, you're correct.
The step0 pt will have the same original unoptimized AIG, however, the labels will be different as those will capture the post synthesis results.

Okay. Is there a way to host the 50GB torch-only dataset with all the 20 steps ?

Let me see what can be done. Can you explain what is your requirement?
The one which is already hosted on the zenodo has consumed 18GB of space. Now,  for each nth step AIG, there are 1500 (20-n) length recipes across all designs. So it will be even difficult to host all the steps with all the recipe labels.

We are looking to learn representations for AIG circuits. Because all step0 circuits correspond to the same circuit, all we have is just 29 unique circuits. If you could generate 20 steps for each  circuit, with 1500 synthesis recipes, that would help us obtain more aig circuits, for the training purpose.

Hi @udaymallappa, the data you’re asking for are the processed pt format dataset of all 870k aigs. It’s part of the original dataset however the entire data is too much to be uploaded on zenodo since even the compressed version need more than 500GB.
Let me figure out a way to host original dataset in such a way that anyone do not have to download the entire data but only the processed dataset.

@udaymallappa, give me some time by end of this week. I need to coordinate with NYU-IT dept. to manage this. The requirement from their end is to evenly chunk the entire dataset and host it. That was the main reason why the entire dataset was zipped and chunked.
I will try to find a way via which only processed pt files are grouped together and rest are zipped together and chunked. This will take some time on my side.
Also, I believe the pyg version used to dump the dataset was older version. For compatibility, with newer version, please follow the following thread:
https://github.com/pyg-team/pytorch_geometric/discussions/5528
I plan to migrate the entire dataset to new pytorch version but it will take some time. Thank you for your patience.


Instance port allocation for CloudFormation backend

Addresses https://github.com/remind101/empire/pull/803#discussion_r61543193
This adds instance port allocation to the CloudFormation scheduling backend by using CloudFormation custom resources.
In a nutshell, when we provision a CloudFormation stack for an app, every container port in a task definition gets a Custom::InstancePort resource (e.g. web8080InstancePort). CloudFormation sends the provisioning request to an SNS topic, and Empire pulls these off of an SQS queue and provisions the ports (or releases them) using the existing port allocator.
This also sets up a nice little framework for adding additional custom resources.

Also, note that this is getting merged into the cloudformation branch, not master.

On a side note, this might make for a good blog post, since this is a pretty nice general way of dynamically handling the instance port limitations of ECS.

👍 this is really cool.
This relies on an SNS topic and an SQS topic that is subscribed to the SNS topic correct?

@mhahn yep. I should update the example CloudFormation stack to create the subscription automatically.

I'll merge this into the cloudformation branch and continue with any changes there.


[9.4-stable] Backport timeout improvements

Backport to 9.4-stable the following PRs:
https://github.com/lf-edge/eve/pull/3428
https://github.com/lf-edge/eve/pull/3518
https://github.com/lf-edge/eve/pull/3509
In order to avoid unwanted triggers from watchdog.

@rene Which change in this PR is fixing the watchdog that was observed: "msg": "file /run/volumemgrmetrics.touch was not changed in 506 seconds (more than 500)"

@rene Which change in this PR is fixing the watchdog that was observed: "msg": "file /run/volumemgrmetrics.touch was not changed in 506 seconds (more than 500)" ?

It won't fix @milan-zededa , I'll open another PR to fix that, anyways this PR is needed to improve the timeouts, specially the qemu img convert...


Coverity: fix error handling issues

Fixing Coverity 1520505 and 1520506.

[ ] documentation is added or updated
[x] tests are added or updated

This pull request is ready to merge

Merged, thanks for the reviewing.


Download & clean campaign finance data for city council members

[ ] Kaur (D1)
[ ] McKee-Rodriguez (D2)
[ ] Viagran (D3)
[ ] Castillo (D5)
[ ] Cabello Havrda (D6)
[ ] Alderete Gavito (D7)
[ ] Whyte (D10)


changefeedccl: support updated and resolved timestamps in avro

PR: https://github.com/cockroachdb/cockroach/pull/34095
From release notes:

The CHANGEFEED experimental_avro format is now backward- and forward-compatible with adjacent schemas for the same table. [#34095][#34095] {% comment %}doc{% endcomment %}

@lhirata, is this done already or covered by another issue? If so, feel free to close.


Stale

Stale bot for issues

oh i like this, should we talk about it tomorrow? @yossi-r

oh, issues related to people working on solutions. I don't know why that would be open for longer then 60 days, but could be a need.
so maybe another label to the list:
exemptLabels:

pinned
security
solution_building**


Recompose does not pass through static class objects

class Component extends Component {
  static get options() {
    return {
      test: true
    }
  }
}

expect(Component .options).toBeDefined() // PASSES

const ComposedComponent = compose(
  connect(mapStateToProps, mapDispatchToProps),
  withProps(),
)(Component)

expect(ComposedComponent.options).toBeDefined() // FAILS

Applys to all recompose functions i tried it with. React native navigation V2 relies on options to be passed through to the higher order components and recompose breaks the flow.

Fixed with hoistStatics.
class Component extends Component {
  static get options() {
    return {
      test: true
    }
  }
}

expect(Component .options).toBeDefined() // PASSES

const ComposedComponent = hoistStatics(
  connect(mapStateToProps, mapDispatchToProps),
  withProps(),
)(Component)

expect(ComposedComponent.options).toBeDefined() // PASSES


Set NfcGuideView: Set nfc_device programmatically

It would be great if we could set nfc_device of NfcGuideView programmatically. Currently, this is only possible in XML.

Agree! We're open for any PR regarding this 😃 Would be great to be able to change the shown device depending on which is in use :)

This is addressed and released with version 0.1.21: https://github.com/fidesmo/nordpol/tree/v0.1.21


Add more per-class information during MAP calculations

In my scenario, to properly judge the quality of training I need to know precision, accuracy, and ideally "false negatives" on a per-class basis.  This can be reported out during any MAP calculations along with the TP & FP).
 detections_count = 4410, unique_truth_count = 2855
class_id = 0, name = Object, ap = 97.34%         (TP = 2783, FP = 125, FN = 72, Pr = 0.96, Rc = 0.97)  <===

 for conf_thresh = 0.25, precision = 0.96, recall = 0.97, F1-score = 0.97
 for conf_thresh = 0.25, TP = 2783, FP = 125, FN = 72, average IoU = 79.83 %

Added many new fields to the map output.  Example output:
using 4 threads to load 8824 validation images for mAP% calculations
processing #8820 (100%)
 detections_count = 190784, unique_truth_count = 57108
 rank = 190700 of ranks = 190784

  Id Name             AvgPrecision     TP     FN     FP     TN Accuracy ErrorRate Precision Recall Specificity FalsePosRate
  -- ----             ------------ ------ ------ ------ ------ -------- --------- --------- ------ ----------- ------------
   0 vehicle               91.2495  32648   3903   5826   3903   0.9095    0.0905    0.8486 0.8932      0.9179       0.0821
   1 motorcycle            80.4499   2936    513    569    513   0.8850    0.1150    0.8377 0.8513      0.9046       0.0954
   2 bicycle               89.0912    570    124    104    124   0.9475    0.0525    0.8457 0.8213      0.9715       0.0285
   3 person                76.7937   7072   1727   2574   1727   0.8894    0.1106    0.7332 0.8037      0.9145       0.0855
   4 many vehicles         64.3089   1068    509    733    509   0.9087    0.0913    0.5930 0.6772      0.9390       0.0610
   5 green light           86.8118   1969    239    510    239   0.8904    0.1096    0.7943 0.8918      0.8898       0.1102
   6 yellow light          82.0390    126     38     30     38   0.9525    0.0475    0.8077 0.7683      0.9764       0.0236
   7 red light             94.1033   3449    217    451    217   0.9237    0.0763    0.8844 0.9408      0.9115       0.0885


falcosidekick-ui  shows "Please send with post http method" message only

Describe the bug
don't see anything in falcosidekick webui
How to reproduce it
helm install falco -n falco  ./falco -f falco-values.yaml
kubectl run event-generator -n falco --image myprivaterepo/falcosecurity/event-generator -- run syscall --loop
image:
  registry: myprivaterepo
tty: true
driver:
  kind: ebpf
falcosidekick:
  enabled: true
  webui:
    enabled: true
    image:
      registry: myprivaterepo
  image:
    registry: myprivaterepo
  ingress:
    enabled: true
    hosts:
      - host: falcosidekick-ui.falco.mydomain
        paths:
        - path: /

Expected behaviour
Expect to see falcosidekick webui with security events
Screenshots

Environment

Falco version:
0.34.1

System info:

Cloud provider or hardware configuration:
baremetal
OS:
rhel 8 like OS
Kernel:
5.15.87-1.el7.3.x86_64
Installation method:
helm chart

Additional context

Hi,
Which version of the webui are you hosting in your private registry? It seems to me you use an old one, the path should be /ui then.

default version installed by helm
falcosecurity/falcosidekick:2.27.0
falcosecurity/falcosidekick-ui:v2.1.0

it's strange but ok. as it's not an issue in falcosidekick-ui, could use close this issue please?

Do you mean that it'is problem with incorrect ingress ?


fix dead links to api pages

fix dead links to api pages

Good catch!  The docs needs to be fixed too... https://raw.githubusercontent.com/jubatus/jubakit/master/doc/source/architecture/service.rst

Coverage decreased (-0.08%) to 89.256% when pulling 0b654f029f99899ac54843f9a4c77d4c92848ec9 on fix-deadly-links into 1d10f284990357f243c2219054f0990c4cb2decd on develop.

Coverage decreased (-0.08%) to 89.256% when pulling 0b654f029f99899ac54843f9a4c77d4c92848ec9 on fix-deadly-links into 1d10f284990357f243c2219054f0990c4cb2decd on develop.

@kmaehashi Thanks for review! I fix other api links too.

Coverage remained the same at 89.336% when pulling f82caf3edf2625a52e3e97767652a2421b20bfb5 on fix-deadly-links into 1d10f284990357f243c2219054f0990c4cb2decd on develop.

Coverage remained the same at 89.336% when pulling f82caf3edf2625a52e3e97767652a2421b20bfb5 on fix-deadly-links into 1d10f284990357f243c2219054f0990c4cb2decd on develop.

Coverage remained the same at 89.336% when pulling f82caf3edf2625a52e3e97767652a2421b20bfb5 on fix-deadly-links into 1d10f284990357f243c2219054f0990c4cb2decd on develop.

Coverage remained the same at 89.336% when pulling f82caf3edf2625a52e3e97767652a2421b20bfb5 on fix-deadly-links into 1d10f284990357f243c2219054f0990c4cb2decd on develop.

👍

💯


Timeline-Folders in the timeline picker of the dialog node

Is your feature request related to a problem? Please describe.
The timeline picker of the timeline property of the dialog node doesn't show the folders like the pickers inside the dialogic tab.
Describe the solution you'd like
Have a similar picker to the change timeline event
Describe alternatives you've considered
I personally don't even instance the node directly often. Mostly I do it by code so if this isn't implemented soon i'm not sad. But I think if this option is provided, this feature should come one day.

Added a partial fix to the timeline pick:

Dialogic.start() will now use paths for specific timelines. A fallback is in place, but specificity is preferred. For instance: Dialogic.start('my-timeline') will search any timeline with that name; Dialogic.start('/chapter-1/my-timeline') will open the timeline namde my-timeline inside the folder chapter-1.

But the inspector picker should be updated next.

Not using this system any more


Loading Permissions > Namespaces & Rules > Permission rules fails

Preflight checklist

[X] I could not find a solution in the existing issues, docs, nor discussions.
[X] I agree to follow this project's Code of Conduct.
[X] I have read and am following this repository's Contribution Guidelines.
[X] This issue affects my Ory Network project.
[X] I have joined the Ory Community Slack.
[X] I am signed up to the Ory Security Patch Newsletter.

Describe the bug
Loading the permission rules in the Ory console freezes the page.
Reproducing the bug

Go to https://console.ory.sh/projects/7da84a50-eb0b-4e69-a58e-da0667326303/permissions/configuration
Switch to the tab "Permission rules"
The tab freezes

Relevant log output
No console output ...

Relevant configuration
No response
Version
latest
On which operating system are you observing this issue?
Ory Network
In which environment are you deploying?
Ory Network
Additional Context
We seem to be using the new permission language ?!

Hi! Thanks for the report, I can confirm. We will take a look.


Update PR template

Pull Request

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

#141  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @JackPlowman and the rest of your teammates on  Graphite


knife wildcard search for attrs with values containing spaces doesn't work

Description
Knife search for attrs with spaces gives unexpected behaviour, like issue 2467
Chef Version
chefdk-2.0.28-1.el7.x86_64
chef-server-core-12.17.15-1.el7.x86_64 on server
Platform Version
client: Enterprise Linux 7 (rhel/centos)
server: Enterprise Linux 7 (rhel/centos)
Replication Case
cd ~/chef-repo/knife/prod/
on 12 nodes
knife node attribute set testbox updates.sched '0200 sun'
on 8 other nodes:
knife node attribute set nodeXX updates.sched '0200 wed'
and search

knife search node  'updates_sched:0200*sun' -a updates.sched
knife search node  'updates_sched:0200?sun' -a updates.sched
knife search node  'updates_sched:0200' -a updates.sched
knife search node  'updates_sched:0200*' -a updates.sched
knife search node  'updates_sched:0200?' -a updates.sched

Client Output
(vv-output included for one; I can include for the others.  I'm also taking some license in truncating the output, but I can always repeat the searches and add then in and I'm hoping I made the output clear)

search with '' to cover the space: no results
:
DEBUG: Initiating POST to https://chef.our.org/organizations/org/search/node?q=updates_sched:0200sun&start=0
:
0 items found

Search with '?' to cover the space.  No hits
:
0 items found

Search for the first 4 letters and find all the test subjects - test pass.
:
tstel7-01:
updates.sched: 0200 sun
tstel7-02:
updates.sched: 0200 wed
:
20 items found

Search for 0200* and find the same expected list - test pass
:
20 items found

Search for 0200? to cover the space again
:
0 items found
The debug output (chef-client -l debug) may be useful, but please link to a gist, or truncate it. -->

Stacktrace

I'm not sure if there's chef-server-style solr-ish flags which let one do this or not, so I'm unclear if this is a bug or just a how-to-chef request.


[dynamic finetuning] Adding dynamic-finetuning code

Commit 1 : [dyanmic training] Adding dynamic-training code …
Added dynamic-training code with both max and l2norm mode
Verified working with existing examples given the threshold
Commit 2: [dynamic-training] Add dynamic training using derivatives …
Added dynamic training using derivatives where the decision to
apply the gradient is calculated using the derivative received
without calculating the gradient itself.
Self evaluation:

Build test: [x]Passed [ ]Failed [ ]Skipped
Run test: [x]Passed [ ]Failed [ ]Skipped

Signed-off-by: Parichay Kapoor<EMAIL_ADDRESS>

:memo: TAOS-CI Version: 1.4.20191203. Thank you for submitting PR #843. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://nnsuite.mooo.com/.

:memo: TAOS-CI Version: 1.4.20191203. Thank you for submitting PR #843. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://nnsuite.mooo.com/.

:octocat: cibot: @kparichay, One of the module of prebuild group is failed. If you want to get a hint to fix this issue, please go to https://github.com/nnstreamer/nntrainer/wiki/.

:octocat: cibot: @kparichay, One of the module of prebuild group is failed. If you want to get a hint to fix this issue, please go to https://github.com/nnstreamer/nntrainer/wiki/.

:octocat: cibot: @kparichay, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://nnsuite.mooo.com/nntrainer/ci/repo-workers/pr-checker/843-202101042103400.86273097991943-320e1914d0c3d625b92b3d6b065e8a6d7e685a6a/.

:octocat: cibot: @kparichay, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://nnsuite.mooo.com/nntrainer/ci/repo-workers/pr-checker/843-202101042103400.86273097991943-320e1914d0c3d625b92b3d6b065e8a6d7e685a6a/.

:octocat: cibot: @kparichay, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://nnsuite.mooo.com/nntrainer/ci/repo-workers/pr-checker/843-202101051828150.62125897407532-f2900fae6889c10c4fefd2389bb3ef1b13168772/.

:octocat: cibot: @kparichay, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://nnsuite.mooo.com/nntrainer/ci/repo-workers/pr-checker/843-202101051828150.62125897407532-f2900fae6889c10c4fefd2389bb3ef1b13168772/.

:octocat: cibot: @kparichay, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://nnsuite.mooo.com/nntrainer/ci/repo-workers/pr-checker/843-202101060020570.45308494567871-f698b0fea6f93675c36f13d6cbe89c735701112b/.

:octocat: cibot: @kparichay, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://nnsuite.mooo.com/nntrainer/ci/repo-workers/pr-checker/843-202101060020570.45308494567871-f698b0fea6f93675c36f13d6cbe89c735701112b/.

:octocat: cibot: @kparichay, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://nnsuite.mooo.com/nntrainer/ci/repo-workers/pr-checker/843-202101060201150.080136060714722-8f463a8071db8a2f04eb06ad7e7f5751c9065b88/.

:octocat: cibot: @kparichay, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://nnsuite.mooo.com/nntrainer/ci/repo-workers/pr-checker/843-202101060201150.080136060714722-8f463a8071db8a2f04eb06ad7e7f5751c9065b88/.

:octocat: cibot: @kparichay, One of the module of prebuild group is failed. If you want to get a hint to fix this issue, please go to https://github.com/nnstreamer/nntrainer/wiki/.

:octocat: cibot: @kparichay, One of the module of prebuild group is failed. If you want to get a hint to fix this issue, please go to https://github.com/nnstreamer/nntrainer/wiki/.


docs(plugins): improvise content

Follow up of #3342

Hi @jamesgeorge007
I was creating a follow up PR here based on my suggestions from #3342
I dont understand why you'd try to copy them over to this PR

closing in regard to #3433

Hi @jamesgeorge007
I was creating a follow up PR here based on my suggestions from #3342
I dont understand why you'd try to copy them over to this PR

Apologies, it just caught my eye and I went on to shoot one.


Academy updates for MR Spatial 220, Spatial Sound

This updates the Toolkit to the latest version and cleans up the documentation.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/holograms-210.md
:white_check_mark:Succeeded
View

mixed-reality-docs/holograms-211.md
:white_check_mark:Succeeded
View

mixed-reality-docs/holograms-212.md
:white_check_mark:Succeeded
View

mixed-reality-docs/holograms-220.md
:white_check_mark:Succeeded
View

mixed-reality-docs/images/comehere.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/gocharge.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/gohide.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/hideunderworld.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/holograms220-taptoplace.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/showunderworld.png
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.