[Toolchain] Run compile using default toolchain

This commit runs compile with cfg using default toolchain.
ONE-vscode-DCO-1.0-Signed-off-by: Jiyoung Yun<EMAIL_ADDRESS>

Running this PR failed in my test setting. 😢
This test setting works well with code in main branch

log
[6/27/2022, 1:48:11 PM][ToolchainProvider][info] Compile toolchain: triv2-toolchain-latest-1.1.0~22062620
[6/27/2022, 1:48:11 PM][Env][info] Done building WorkFlow on Env: [{"jobType":9,"name":"config","valid":true,"tool":"onecc","toolArgs":["--config","/home/eric/src/model/ONE-vscode-res/inception_v3/2020-02-03/inception_v3_conv2d.0to128.cfg"],"root":false}]
[6/27/2022, 1:48:11 PM][Helpers][info] obtainWorkspaceRoot: /home/eric/src/model/ONE-vscode-res
[6/27/2022, 1:48:11 PM][JobRunner][info] Run tool: onecc args: ["--config","/home/eric/src/model/ONE-vscode-res/inception_v3/2020-02-03/inception_v3_conv2d.0to128.cfg"] cwd: /home/eric/src/model/ONE-vscode-res root: false
[6/27/2022, 1:48:11 PM][ToolRunner][info] Running: config
tflite2circle: ERROR: Failed to open file: ./inception_v3_conv2d.0to128.tflite
[6/27/2022, 1:48:12 PM][ToolRunner][info] child process exited with code 255
[6/27/2022, 1:48:12 PM][ToolRunner][info] Build Failed: 255

my setting:
$ pushd /home/eric/src/model/ONE-vscode-res/inception_v3/2020-02-03/
~/src/model/ONE-vscode-res/inception_v3/2020-02-03 ~/src/model/ONE-vscode-res

eric@labs:~/src/model/ONE-vscode-res/inception_v3/2020-02-03  ( ) (13:49:53)
$ ll
total 350212
drwxr-xr-x 2 eric eric     4096  5월 25 17:21 ./
drwxr-xr-x 4 eric eric     4096  5월  6 12:16 ../
-rw-r--r-- 1 eric eric      552  6월 27 13:45 inception_v3_conv2d.0to128.cfg
-rw-r--r-- 1 eric eric 95307124  5월  3 19:29 inception_v3_conv2d.0to128.tflite
-rw-r--r-- 1 eric eric      617  6월 24 14:42 inception_v3_conv2d.0to128-wrongcfg.cfg
-rw-rw-r-- 1 eric eric 95307140  6월 27 13:45 model.circle
-rw-rw-r-- 1 eric eric       84  6월 27 13:45 model.circle.log
-rw-rw-r-- 1 eric eric 95303824  6월 27 13:45 model.opt.circle
-rw-rw-r-- 1 eric eric       96  6월 27 13:45 model.opt.circle.log
-rw-rw-r-- 1 eric eric 24358864  6월 27 13:45 model.q8.circle
-rw-rw-r-- 1 eric eric      642  6월 27 13:45 model.q8.circle.log

eric@labs:~/src/model/ONE-vscode-res/inception_v3/2020-02-03  ( ) (13:49:56)
$ more inception_v3_conv2d.0to128.cfg
[onecc]
one-import-tf=False
one-import-tflite=True
one-import-bcq=False
one-import-onnx=False
one-optimize=True
one-quantize=True
one-pack=False
one-codegen=True

[one-import-tflite]
input_path=./inception_v3_conv2d.0to128.tflite
output_path=./model.circle

[one-optimize]
input_path=./model.circle
output_path=./model.opt.circle
remove_redundant_transpose=True

[one-quantize]
input_path=./model.opt.circle
output_path=./model.q8.circle
granularity=channel

[one-codegen]
backend=tv2
command=-o model.tvn --DSP-quota 8K --save-temps ./model.q8.circle

@jyoungyun
When I try this PR, there was an error.
My company system currently blocks me from uploading gif file again. So I'd like to leave a log.
Here you can find password input window is acting weird.

pw dialog shows from 4:28:09 and 4:28:18
[6/30/2022, 4:28:09 PM][Env][info] Done building WorkFlow on Env: [{"jobType":11,"name":"install","valid":true,"tool":"aptitude","toolArgs":["install","-o Aptitude::ProblemResolver::SolutionCost='100*canceled-actions,200*removals'","triv2-toolchain=1.1.0","-q","-y"],"root":true}]
[6/30/2022, 4:28:09 PM][Helpers][info] obtainWorkspaceRoot: /home/eric/m/src/model/ONE-vscode-res
[6/30/2022, 4:28:09 PM][Env][info] Showing password prompt
[6/30/2022, 4:28:09 PM][JobRunner][info] Run tool: aptitude args: ["install","-o Aptitude::ProblemResolver::SolutionCost='100*canceled-actions,200*removals'","triv2-toolchain=1.1.0","-q","-y"] workdir: /home/eric/m/src/model/ONE-vscode-res root: true
[6/30/2022, 4:28:09 PM][ToolRunner][info] Running: install
[sudo] password for eric: Sorry, try again.
[sudo] password for eric:
sudo: 1 incorrect password attempt
[6/30/2022, 4:28:11 PM][ToolRunner][info] child process exited with code 1
[6/30/2022, 4:28:11 PM][ToolRunner][info] Build Failed: 1

[6/30/2022, 4:28:11 PM][ToolchainProvider][err] Installation is failed
[6/30/2022, 4:28:18 PM][Env][info] Password dialog canceled

When I try this PR, there was an error. I removed out, node_modules and tried to run once more but I got the same result. 😿

@hyunsik-yoon I fixed this issue. Could you please test it again?

/cc @hyunsik-yoon

Merged. Done!


[Feature] Bug explanation

Describe the bug
/ss and /socialspy on disables socialspy no matter what
To Reproduce
Steps to reproduce the behavior:

run /ss or /scoialspy on
look at chat

Expected behavior
Should enable social spy not disable it
Username:
_Conj
Server name (all applicable):
All

This has been fixed, closing


Can't communicate with container from vm using same physical interface

In my setup my server has two network interfaces: one for the server to communicate which resides on one network and another for vms and now docker containers to communicate on which connect to a different subnet. The server(host) has only got an ip assigned to to one interface and the other is left up but without ip assigned.
I'm starting my container with the command:
pipework enp4s6 $(docker start bind) 10.0.x.y/24 && pipework route bind add default via 10.0.x.1
This works perfectly to start my container and let it communicate on the second network interface. I also have a few VMs running which use the second NIC and my problem is that they can't reach the docker container communicating on the same interface.
I tried running the following commands inside my vm but that didn't help:
ip addr del 10.0.x.z/24 dev eth0
ip link add link eth0 dev eth0m type macvlan mode bridge
ip link set eth0m up
ip addr add 10.0.x.z/24 dev eth0m
route add default gw 10.0.x.1
The network interface of the vm is of type pcnet and has Promiscuous Mode Allow All.
Am I missing something or would my scenario not work e.g. letting my vms communicate with the container using the same NIC?

Which VM system are you using? I know that some systems will hook themselves at a specific layer in the NIC driver, preventing communication with other sub-interfaces ...
You can try two things:

enabling hairpin switching if your network hardware supports it
instead of using macvlan, create a bridge, attach the containers to the bridge, and put the NIC in the bridge as well

I'm using Virtualbox for my VMs.
What would hairpin switching mean in this case? Do you mean using a trunk port from the server to my router and using subinterfaces for different vlans? E.g. traffic from docker container goes out on one vlan to the router and is then routed to another subinterface and takes the trunk back to the server but this time on a differnt vlan.
How does a bridge differ from macvlan? Would it still support my setup such that the Host system(the server) has not got an IP assigned to the interface, and multiple docker containers as well as virtual machines might use the interface with their own IP assigned to it on the same network?

Hairpin switching would let the switch re-forward frames to your machine (potentially on the same subinterface, not requiring a trunk port). I know that it can help in this scenario but I haven't used it directly myself.
Using a bridge is generally more "compatible" but also more costly (in terms of latency and CPU overhead). To phrase things differently — macvlan (as well as the mechanism used by VMware or VirtualBox) are more efficient, but designed to work for one specific use-case, and don't always play nice with each other.
If your machine as an extra NIC (and you have an extra switch port available), you could use one NIC for the VMs and one NIC for the containers. This would let you use fast, efficient mechanisms, at the expense of one extra round-trip to the switch. In my experience, the extra round-trip to the switch is not an issue, especially on gigE networks; and at gigabit/s speeds, the CPU overhead of the internal bridge gets significant.
If your machine doesn't have an extra NIC, you could try to use VLANs, and put the containers on one VLAN, the VMs on the other VLAN, and bridge the VLANs together (if your switch supports it).
I hope this helps!


positionning stacked bars

hello
i would like to ask if is it possible to draw stacked bars not from the graph base but from the end of the previous bar like image below

thanks in advance

Hi i have changed the option name for the overlaying bars feature to overlayBars (https://github.com/leighquince/Chart.js/commit/54896333c6126fca043249ed388d5c1af9aca8e4)  now to avoid the misleading title of stacked and what it was previously doing. This feature can now be called stacked( or something like) that but going to have a think about the best way to do it as it might make sense to have this as a new chart rather than an option.

i've realized there is actually a community extension that does this - https://github.com/Regaddi/Chart.StackedBar.js


Fix + should not concat strings

For https://github.com/witheve/Eve/issues/631 finally.
I've gone for the more conservative option of only filtering if one or both items cannot be converted to a number.
Let me know if you want this code to filter and I'll change
search
a = "1"
b = "2"
x = a + b
commit @browser
[#div text:x]
Which will currently set x to 3.

Thanks for putting this together Cormac. On the topic of implicit conversion (treating a string as a number silently), I think that should be a failure as well.

Done.

Done
Regards,
Cormac

Thanks Cormac!


python3Packages.mailchecker: 4.1.10 -> 4.1.11

Motivation for this change
https://github.com/FGRibreau/mailchecker/blob/master/CHANGELOG.md#v4111-20220201-0918-0000
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 157815 run on x86_64-linux 1

6 packages built:

dvc
dvc-with-remotes
python310Packages.mailchecker
python310Packages.python-benedict
python39Packages.mailchecker
python39Packages.python-benedict


Use v1 Channelable in NATSS Channel

Problem
NATSS Channel still uses v1alpha1 Channelable which is scheduled to be removed in 0.18.

https://github.com/knative/eventing/pull/3789#pullrequestreview-469565624 depends on this


[AppBar] Dark theme doesn't change AppBar color

When using the dark theme, the AppBar is still blue, which doesn't mix with the rest of the theme.

#926

Teal 200 doesn't mix well either. I've been using Grey 800, like this:
import DarkTheme from "material-ui/lib/styles/themes/dark-theme";
import {Styles} from "material-ui";

// modified dark theme from mui to add missing stuff
let Dark = {
	getPalette: DarkTheme.getPalette,
	getComponentThemes(palette)
	{
		let themes = DarkTheme.getComponentThemes(palette);
		themes.appBar = {
			color: Styles.Colors.grey800
		};

		return themes;
	}
}

export default Dark;

@thesbros Thanks for the feedback - we defaulted it to what was on the sticker sheet here:
https://www.google.com/design/spec/resources/sticker-sheets-icons.html
Of course, you can override this to be whatever you want - which it looks like you've done. :+1:


Save Favorite Broadcaster/Save to Broadcaster folder

Hello
Wonderful piece of software here!
Could you add an option to save a list of favorite broadcasters and also when saving it auto generates a folder specific to the broadcaster

Cool idea. I like 😄

A "Save favorite broadcaster seems to be what I want too, so I'm posting here.
OP mentions a "list", which is more than 1. 10 should be perfectly fine (enough), but the very simple way I imagined it first does not permit for a user-chosen amount - so maybe 5?
The way I thought of it first was basically like this: https://vgy.me/7NgXRn.png - Doing it this way there's no fuss about adding more buttons to the top/resizing them so an addition fits neatly etc, and the Search tab is displayed by default on opening so the buttons are there anyway vs having to go to a different pane between search and favorites etc. Clicking one of the buttons would simply launch a channel search for the fav.
Maybe there's a better, cleaner way to do this (the UI of this is gorgeous). I had just imagined it like this because it seems really simple/easy to add. Cheers!

Nice attempt... don't like it :D
Here's mine:
In the preferences you will be able to click some +/- buttons next to the "channel name" textbox. This way you can add unlimited favourite streamers to a list. The Textbox will be changed to a dropdownbox. This way you are able to type in the control like it was and in addition you can choose one of your favourite streamers.
The same applies to the textbox on the search panel. Just click the dropdown, choose a favourite streamer and it "search".
Pros:

Not many additional controls
Unlimited favourites
Current behaviour isn't changed
Ability to choose a favourite streamer as the "search on application startup" thingy.

Cons:

3
2
None


Huffman Coding 💯  lossless data compression algorithm.

Huffman Coding Problem
Problem Description
Huffman Coding is a lossless data compression algorithm. The idea is to assign variable-length codes to input characters based on their frequencies. The more frequent a character appears, the shorter its code. The goal is to reduce the total size of data while preserving all information.
You are given a set of characters and their corresponding frequencies. Your task is to build a Huffman Tree and find the Huffman codes for each character.
Input

A set of unique characters.
A corresponding frequency for each character.

Output

The Huffman codes for each character, represented as binary strings.

Constraints

The number of characters, n, will be such that 1 ≤ n ≤ 100.
Frequency values are positive integers.

Approach

Create a Min Heap:

Each character and its frequency are stored as nodes in a priority queue (min-heap).

Build the Huffman Tree:

While there is more than one node in the heap:

Extract the two nodes with the lowest frequency.
Create a new internal node with these two nodes as children, and the frequency as the sum of their frequencies.
Insert this internal node back into the heap.

The remaining node in the heap is the root of the Huffman Tree.

Generate Huffman Codes:

Perform a depth-first traversal of the Huffman Tree, assigning a binary digit to each left (0) and right (1) edge.
When a leaf node is reached, the path taken from the root to the leaf forms the character's code.

Code Implementation in C++
#include <iostream>
#include <queue>
#include <unordered_map>
#include <vector>

using namespace std;

// A Huffman tree node
struct Node {
    char ch;
    int freq;
    Node* left;
    Node* right;

    Node(char c, int f) : ch(c), freq(f), left(nullptr), right(nullptr) {}
};

// Comparison object to order the heap based on frequency
struct Compare {
    bool operator()(Node* left, Node* right) {
        return left->freq > right->freq;
    }
};

// Function to print the Huffman Codes from the root of the tree
void printHuffmanCodes(Node* root, string str) {
    if (!root)
        return;

    // If this is a leaf node, then it contains one of the input characters
    if (!root->left && !root->right)
        cout << root->ch << ": " << str << "\n";

    printHuffmanCodes(root->left, str + "0");
    printHuffmanCodes(root->right, str + "1");
}

// Main function to build Huffman Tree and decode characters
void huffmanCoding(const unordered_map<char, int>& freqMap) {
    // Create a priority queue (min heap)
    priority_queue<Node*, vector<Node*>, Compare> pq;

    // Create a leaf node for each character and add it to the priority queue
    for (auto pair : freqMap)
        pq.push(new Node(pair.first, pair.second));

    // Iterate until the heap contains only one node (root of the Huffman tree)
    while (pq.size() != 1) {
        // Remove the two nodes of highest priority (lowest frequency)
        Node* left = pq.top();
        pq.pop();
        Node* right = pq.top();
        pq.pop();

        // Create a new internal node with these two nodes as children
        // The frequency of the new node is the sum of their frequencies
        Node* internalNode = new Node('$', left->freq + right->freq);
        internalNode->left = left;
        internalNode->right = right;

        // Add this node to the heap
        pq.push(internalNode);
    }

    // Print the Huffman codes by traversing the tree
    printHuffmanCodes(pq.top(), "");
}

int main() {
    // Input: Frequency of characters
    unordered_map<char, int> freqMap = {
        {'a', 5}, {'b', 9}, {'c', 12}, {'d', 13}, {'e', 16}, {'f', 45}
    };

    // Build Huffman Tree and display the codes
    huffmanCoding(freqMap);

    return 0;
}

PLEASE ASSIGN ME THIS ISSUE AS HACKTOBERBER ACCEPTED AND ACCEPT MY PULL REQUEST


No module named 'inference'

The code that is publicly available now is missing the inference.py.
Your work is very inspiring to me, may I have your complete code, my email is<EMAIL_ADDRESS>Thanks!

The code that is publicly available now is missing the inference.py. Your work is very inspiring to me, may I have your complete code, my email is<EMAIL_ADDRESS>Thanks!

Sorry...The missing inference file has been uploaded. Please check.


Update SQL Feature Support reference docs

Fixes DOC-5814
Fixes DOC-2158
Informs closing of old issue DOC-299
Summary of changes:

In a few cases, we have docs for workarounds to standard SQL features, so I added links

There were a few things missing from the SQL feature support that were in e.g. the Data Types page, so I added them

Also updated our PG compat page with the lack of foreign data wrappers

Took the opportunity to add some linkage between keywords/identifiers and SQL name resolution docs that could be useful, since they are related

This change is


The extension goes to a different page when I am scrolling an element with a horizontal overflow

Describe the bug
When I scroll code comments on github.com or stackoverflow.com the extension navigates back or forward. When my mouse is inside such code comments, I simply want to scroll them horizontally, not go to a different page.
To Reproduce
Steps to reproduce the behavior:

Go to any page which has elements with overflow-x:  auto, for example on github
Try to scroll that element to see it with a touchpad
Actual behaviour: page navigates

Expected behavior
A clear and concise description of what you expected to happen.
The extension allows horizontal scrolling on overflow-x:  auto elements and doesn't call window.history.back();
Instead it only navigates back when the element.scrollLeft == 0  and only navigates forward when element.scrollLeft == element.scrollLeftMax
Desktop (please complete the following information):

OS: Windows 10
Browser Firefox
Version 99 Developer edition

I already have fixed it, will publish a PR soon

Yes, had some issues updating the add-on due to "Unsafe assignment to innerHTML on options/options.js line 5 and line 7".
Are they needed?

Yes, had some issues updating the add-on due to "Unsafe assignment to innerHTML on options/options.js line 5 and line 7". Are they needed?

If the addons team has issue with that, you can swap it to innerText just fine


no issue

haha the coding assistant with voice and codesandbox is very nice!

Thanks! Hoping to flush it more

I’ve been trying to send a photo of CodeSandbox render to the AI for feedback, but the photo always shows up as white. Maybe you know a trick ?

Could you share screenshot or steps to repro?
It is a little buggy.

Sorry for the misunderstanding. I’m working on a similar project with GPT and CodeSandbox, not your repository.
I’ve tried different React libraries to take screenshots in order to build a feedback pipeline after the AI codes something.
maybe you can help. No code yet but its the Reactor project on my gh profile

Oh I see. Puppeteer is pretty good for taking web screenshot programmatically

Checkout my repo sketch2app and scroll up to previous commits, it demonstrates how to dynamically show react apps / react native apps with codesandbox rendered inside the app. It uses a package called sandpack which lets you pass json full of code.

I'm facing an issue: when I try to capture a screenshot, the iframe showing the rendered preview is always blank (white).

@cameronking4
Thank you for you work with VAPI, this repo has been a huge help.
I was trying to add the coding assistant demo to my app but I am unable to do it as mentioned in the docs, I have setup the basic component but have no idea on how you setup the assistant for it? Is there any guidance you could provide on that, it would be of great help.
Thank you.

Ah Yes please email me I'll send you my setup my Vapi dashboard.
<EMAIL_ADDRESS>


INT96 is marked as deprecated

Hi everyone,
According to https://issues.apache.org/jira/browse/PARQUET-323 type INT96 is deprecated.
However, DateTime and DateTimeOffset are serialized to INT96.
Are there any plans to mitigate this problem?
Should I use DateTimeOffset.ToUnixTimeMilliseconds() and serialize value as INT64 as a recommended workaround?
Thanks in advance.

I had the same issue.  You can create a DateTimeDataField rather than a DataField and specify DateTimeFormat.DateAndTime to use a 64-bit timestamp.


Document the ownership of buffer holding request object pointers

This is necessary for the asynchronous backends like TRT.

Can you create a ticket for that, we can continue the discussion there.

DLIS-2752


Daedalus 1.1.4883 still has error with non-latin characters in APPDATA folder`s name

When I try to run I get error

cardano-launcher.exe
Couldn't pack log files, reasons: can't pack log file pub\node.pub because it doesn't exist or it's not a file

http://prntscr.com/iojuv9

on windows?

yes, on windows 10

Sadly it's not trivial to fix - please monitor https://github.com/facebook/rocksdb/issues/3408

To catch issues like this in the future, is it possible to have one of the CI run on a non-ASCII locale? @domenkozar

For previous version of Daedalus 1.0.3769 I created symlink AnyLatinName to my real account NonLatinName and changed all %APPDATA% to C:\Users\AnyLatinName in the bat file, and all was fine. But is no bat file in last version, and I can run Daedalus only if I create another account with latin characters only

We are actively working on a fix - but don't expect this soon as we have to upstream fixes.

at least im not the only one with the problem
howcome the update was automatically without warning upfront? would have been nicer so everyone can first make sure they have absolutely saved all keys and recovery logs

eh i just fixed my issue (also had the pub\node.pub error)
i first installed the old wallet again (1.10 version or something) then i downloaded the new wallet from the cardano website, installed it & was able to open it without issue after a short sync


Use sys.maxsize as upper bound for randrange

It's possible to never exit the while loop if there are more than 1000 items that need to be bound. This PR mitigates it by using sys.maxsize, which is Python 2 and 3 compatible.
There is no real performance penalty by increasing the upper bound of random.randrange:
%timeit random.randrange(1, 1000)
# 773 ns ± 28.4 ns per loop (mean ± std. dev. of 7 runs, 1000000 loops each)
%timeit random.randrange(1, sys.maxsize)
# 822 ns ± 3.58 ns per loop (mean ± std. dev. of 7 runs, 1000000 loops each)

Reproducing error:
import string
import random

def generate_sql(items=None):
    sql_template = """
    SELECT
      col
    FROM
      table
    WHERE
      1
      {% if items %}
      AND table.col IN {{ items | inclause }}
      {% endif %}
    """

    j = JinjaSql()
    data = dict(items=items)
    sql, bind_params = j.prepare_query(sql_template, data)
    return sql, bind_params

def id_generator(size=6, chars=string.ascii_uppercase + string.digits):
    return ''.join(random.choice(chars) for _ in range(size))

# works
generate_sql(items=[id_generator() for _ in range(1000)])
# infinite loop!
generate_sql(items=[id_generator() for _ in range(1001)])

ran into this too: https://github.com/hashedin/jinjasql/pull/17. though not sure we why wouldn't use hash, are we worried about performance?

@benrudolph hasing is better. closing my PR in favor of yours.


benchmark, writes: etcd, Zookeeper, Consul test metrics

Here's the new test cases to be implemented, or just to be configured, for better performance comparison.
At the very high level:

Throughput; scale the clients to find the best throughput T of each database ()

For graph, x-axis is the number of clients
y-axis is the average throughput for the corresponding client number k

raw data should be aggregated by client

y-axis should also show the minimum and maximum throughput for that client number
test case #1: 256-byte key, 1024-byte value, 1-million entries, total 1.3 GB, variable client numbers

e.g. client numbers could be 1, 3, 5, 10, 50, 100, 500, 700, 1000, ...

Latency; measure the latency with the best-possible throughput

Choose the client number K with the best throughput T, for each database
No rate limit
For graph, just show the latency distribution with box plots
test case #2: 256-byte key, 1024-byte value, 1-million entries, total 1.3 GB, client number K

Disk; measure sector written

sector written should be greater than disk writes in /proc/diskstats
make sure numbers match up, there could be race in metrics collector
For graph, x-axis is the number of clients
y-axis is the number of sectors written per clients, per database

this would be the rough estimates from 3 nodes' system metrics
interpolate each CSV, and get the average value by unix second

use the test case #1
need to improve data visualization (TODO: gnuplot)

Disk utilization; measure the total database size on filesystem

du -sh $HOME/etcd.data
just explain with plain table with sums
use the test case #1

Network; measure the total network overhead between peers, clients

use the netstat data in system metrics
just explain with plain table with sums
use the test case #1

CPU; measure the overall system load

CPU numbers per PID might not be accurate
use the overall system load numbers from top command
For graph, x-axis is the number of clients
y-axis is the average system load numbers for each database

this would be the rough estimates from 3 nodes' system metrics
interpolate each CSV, and get the average value by unix second

use the test case #1
need to improve data visualization (TODO: gnuplot)

Memory; measure the memory usage as number of keys grow

For graph, x-axis is the cumulative number of keys (estimated)

if database A has average memory M and 'cumulative' throughput 32,000, at second x, the x-value would be 30,000 and y-value would be M

y-axis is the memory being used, by the number of keys
test case #3: 256-byte key, 1024-byte value, 1-million entries, total 1.3 GB, 1,000 clients

Latency with too many keys; keep writing until the last database breaks

For graph, x-axis is the cumulative number of keys (estimated, same as case 7)
y-axis is the average latency, by the number of keys
use the best-possible throughput of each database from the case 1
test case #4: 256-byte key, 1024-byte value, X-million entries, total X GB, client number K

FYI. These are the test cases that me and Anthony discussed today, for the comparison benchmarks.
We need to add a few more features to run those.
/cc @heyitsanthony @xiang90


Add a few more typescript decls

There are certainly more that could be added but these are a start.

Thanks @garyo!


Freestanding Expression macro code that uses try fails to compile with 'Errors thrown from here are not handled' when used in a throwing closure

Description
When using freestanding expression macros, if the macro generates code that uses the try keyword in the returned expression, the following compile error is produced when the macro is used inside a throwing closure:

Errors thrown from here are not handled

Steps to reproduce

Download the following sample project: ThrowingInFreestandingMacroReproProj.zip
Build the ThrowingInFreestandingMacroReproProjClient target
See Errors thrown from here are not handled compile error

Expected behavior
The project should compile successfully because errors thrown from inside the closure are in fact handled.
Environment
$ swiftc -version
swift-driver version: 1.82.2 Apple Swift version 5.9 (swiftlang-<IP_ADDRESS>.10 clang-15<IP_ADDRESS>)
Target: arm64-apple-macosx13.0

$ xcodebuild -version
Xcode 15.0
Build version 15A5161b

Closures infer their throwing-ness based on the presence of try in the source code. Because macro expansion occurs after type checking, the closure is inferred to be non-throwing. This is the expected behavior for the system.

Because macro expansion occurs after type checking, the closure is inferred to be non-throwing.

Thanks for explaining this @DougGregor. Does this mean that use of throwing functions inside an expression macro is not supported then? I can't quite wrap my head around how you would write on that supports this, but maybe that's because you can't?

It would be great if this issue could be re-considered. The current handling leads to an error message when using an#expect(try …) macro in Swift Testing within a closure:
import Testing

class Foo {
	init() {
	}

	func doSomething() throws -> Bool {
		throw TestError("Error")
	}
}

struct FooTest {
	@Test func throwingClosure() async throws {
		let foo = Foo()

		try Self.perform {
			#expect(try foo.doSomething())
		}
	}

	static func perform(_ block: () throws -> Void) throws {
		try block()
	}
}

In this case an "Errors thrown from here are not handled" is emitted. While using the same #export(try …) macro directly from a throwing function works.
If this can't be easily fixed, it would be already very helpful if the error message would hint on the throwing closure (because it takes quite time to understand the context…


Performance vs Newtonsoft.Json.Schema

Documentation

[X] I have consulted the documentation, and my question isn't answered there.

Nuget Package
JsonSchema.Net
Package Version
7.1.2
How can I help?  Please provide as much context as possible.
I recently came across this library and am interested in using it instead of Newtonsoft.Json.Schema, I am running some very basic performance tests and it appears to be ~3x slower against NSJ and also allocates more, is this typical of the performance one can expect, or am I missing something very basic?
This is the BenchmarkDot.Net test that I am using, its just using a very basic Person schema and request:-
using BenchmarkDotNet.Attributes;
using BenchmarkDotNet.Columns;
using BenchmarkDotNet.Configs;
using BenchmarkDotNet.Jobs;
using BenchmarkDotNet.Reports;
using BenchmarkDotNet.Running;
using BenchmarkDotNet.Toolchains.InProcess.NoEmit;
using Json.Schema;
using Newtonsoft.Json.Linq;
using Newtonsoft.Json.Schema;
using System.Text.Json.Nodes;

License.RegisterLicense("YOUR_LICENCE_KEY_HERE");

BenchmarkRunner.Run<JsonSchemaValidatorBenchmark>();

[Config(typeof(Config))]
[MemoryDiagnoser]
public class JsonSchemaValidatorBenchmark
{
    private const string _personRequest = """
                {
          "firstName": "John",
          "lastName": "Doe",
          "age": 30,
          "email"<EMAIL_ADDRESS>        }

        """;
    private const string _personJsonSchema = """
                {
          "$schema": "https://json-schema.org/draft/2020-12/schema",
          "title": "Person",
          "type": "object",
          "properties": {
            "firstName": {
              "type": "string",
              "description": "The person's first name."
            },
            "lastName": {
              "type": "string",
              "description": "The person's last name."
            },
            "age": {
              "type": "integer",
              "description": "Age in years which must be equal to or greater than zero.",
              "minimum": 0
            },
            "email": {
              "type": "string",
              "format": "email",
              "description": "The person's email address."
            }
          },
          "required": ["firstName", "lastName", "age", "email"],
          "additionalProperties": false
        }

        """;
    private Json.Schema.JsonSchema? _personSchema;
    private JSchema? _nsjPersonSchema;

    [GlobalSetup]
    public void Setup()
    {
        _personSchema = System.Text.Json.JsonSerializer.Deserialize<Json.Schema.JsonSchema>(_personJsonSchema);
        _nsjPersonSchema = JSchema.Parse(_personJsonSchema);
    }

    [Benchmark(Baseline = true)]
    public void NewtonSoftJsonParsePersonRequestAndValidateSchema()
    {
        JObject instance = JObject.Parse(_personRequest);
        bool r = instance.IsValid(_nsjPersonSchema!);
    }

    [Benchmark]
    public void JsonEverythingParsePersonRequestAndValidateSchema()
    {
        JsonNode? instance = JsonNode.Parse(_personRequest);
        EvaluationResults r = _personSchema!.Evaluate(instance);
    }

    private class Config : ManualConfig
    {
        public Config()
        {
            SummaryStyle = SummaryStyle.Default.WithRatioStyle(RatioStyle.Trend);

            AddJob(Job
                .MediumRun
                .WithLaunchCount(1)
                .WithToolchain(InProcessNoEmitToolchain.Instance));
        }
    }
}

The results on my machine are as follows:-

Method
Mean
Error
StdDev
Ratio
RatioSD
Gen0
Allocated
Alloc Ratio

NewtonSoftJsonParsePersonRequestAndValidateSchema
2.846 us
0.0320 us
0.0299 us
baseline

1.2550
5.13 KB

JsonEverythingParsePersonRequestAndValidateSchema
10.490 us
0.0593 us
0.0463 us
3.68x slower
0.05x
2.7618
11.31 KB
2.20x more

I am using the following package versions:
<PackageReference Include="BenchmarkDotNet" Version="0.13.12" />
<PackageReference Include="JsonSchema.Net" Version="7.1.2" />
<PackageReference Include="Newtonsoft.Json.Schema" Version="4.0.1" />

I am running on .NET8
I might be very wrong in setting it all up, this is as basic as I could make it without using our schemas and requests.
Code of Conduct

[X] I agree to follow this project's Code of Conduct

I thnk you've identified one of the primary advantages in your first line of code:  JsonSchema.Net is free.
Secondly, I encourage you to rewrite the test so that it runs multiple validations against the same schema.  JsonSchema.Net will perform some initial analysis of the schema on the first run, which is likely what's pushing the numbers up.  Subsequent validations will be considerably quicker since it has to do less work.
Finally, JsonSchema.Net uses System.Text.Json, which is inarguably the way forward for JSON in .Net.
Let me know how you manage with the updated benchmarks.

Thanks for the reply, yes I agree with both of your points entirely. I was meant to add on the original post this is not meant to be a criticism about a free library vs a paid for library.
I am not sure I understand when you say run multiple validations against the same schema, are you suggesting I just put 2 or more calls to Evaluate back to back and re-run the tests, or are you suggesting something more involved?

If you look at the current JSON Schema benchmark, you'll see that it runs the test suite once and then 10 times.  The scaling isn't linear.
This is due to the preprocessing that happens the first time.  For the benchmarks, all of the schemas of the test suite (and there are thousands of them) are retained in memory, so when the test is run again, it doesn't have to do thta preprocessing again.  This produces a noticeable improvement in subsequent runs of the same schema.

Update the benchmark to run this:
[Params(1, 10, 100, 500)]
public int Count { get; set; }

[Benchmark]
public void NewtonSoftJsonParsePersonRequestAndValidateSchema()
{
	for (int i = 0; i < Count; i++)
	{
		JObject instance = JObject.Parse(_personRequest);
		bool r = instance.IsValid(_nsjPersonSchema!);
	}
}

[Benchmark]
public void JsonEverythingParsePersonRequestAndValidateSchema()
{
	for (int i = 0; i < Count; i++)
	{
		JsonNode? instance = JsonNode.Parse(_personRequest);
		EvaluationResults r = _personSchema!.Evaluate(instance);
	}
}

[Benchmark(Baseline = true)]
public void NewtonSoftJsonValidateSchema()
{
	JObject instance = JObject.Parse(_personRequest);
	for (int i = 0; i < Count; i++)
	{
		bool r = instance.IsValid(_nsjPersonSchema!);
	}
}

[Benchmark]
public void JsonEverythingValidateSchema()
{
	JsonNode? instance = JsonNode.Parse(_personRequest);
	for (int i = 0; i < Count; i++)
	{
		EvaluationResults r = _personSchema!.Evaluate(instance);
	}
}

and let me know.  I don't have a license for it, so I can't run this.

Thanks for the detailed response!
Yes, I've seen that a lot of time is spent during the BuildEvaluation stage, from my very limited understanding of the code it seemed difficult to do really significant improvements here for this particular use case without really changing the fundamental approach.
I see you have a branch "functional" and one "functional-low-alloc". Would it already help if I gave one of those branches a try?
BTW: I've seen some surprisingly large improvement for my use-case by simply removing the building of the validation messages log, basically just not creating the Fail message in ConstKeyword and RequiredKeyword.
I guess it helps reducing the large number of allocations I've seen:

I've just replaced the errormessage with "null", I think the proper way would be to check if the OutputFormat is set to OutputFormat.Flag and then not create the messages -- they are thrown away anyway later.
It's not really helping for my use case, even large improvements here are wiped out by just adding another $ref, but maybe it is worth doing for some other use cases.

    I don't think Newtonsoft supports the output formats.


/ru/docs/Web/API/ParentNode/append

Request type

[] Please close this issue, I accidentally submitted it without adding any details
[ ] New documentation
[x] Correction or update

Details
In English version example of string which is used for appending is "Some text".
But in the Russian version:

In section Appending text is "Вася ест ноздри" which translates as: "Ivan eats nostrils"
In the section Appending an element and text is "Вася уже съел ноздри" which translates as "Ivan ate nostrils".

I don't think that eating nostrils is a good idea for learning program development :)
Also, it is a not good idea in general, so I believe that examples should be changed to "Любой текст", which is translation of "Some text".

Fixed. No more nostril eating here!
Thanks for reporting ;-)

Thx!


Set-TeamPicture doesn't work if you are not owner of the team

I have try the new cmdlet Set-TeamPicture, but it doesn't work if you are not the owner of the teams.
Is it normal ?
I don't see the owner of the team entering PowerShell command line to modify the picture team !
Best regards
Yannick

@officedocsbot assign @yogkumgit

Hello @koxo How are you?
I could verify what you said. For feature requests, I need to kindly suggest Microsoft Teams User Voice that way, your idea can reach the product team to change the current behavior of the cmdlet.
Thanks

Hi Andres !
I’m going to ask MS Teams to modify the cmdlet.
Best regards
Yannick
Yannick LANCHEC
Developpement & I.T.
Infrastructure & Directories
infos@koxo.netmailto:infos@koxo.net?subject=Demande d'informations
GSM : +33 (0)6 43 25 20 80
KoXo Développement
2 rue de Pen Ar C’hleuz
29000 Brest
[Logo-KoXo-Dev-PHPBB]
[msregisteredpartner]
De : Andres Mariano Gorzelany<EMAIL_ADDRESS>Envoyé : mardi 30 juin 2020 18:07
À : MicrosoftDocs/office-docs-powershell<EMAIL_ADDRESS>Cc : Informations KoXo Dev<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Objet : Re: [MicrosoftDocs/office-docs-powershell] Set-TeamPicture doesn't work if you are not owner of the team (#6009)
Hello @koxohttps://github.com/koxo How are you?
I could verify what you said. For feature requests, I need to kindly suggest Microsoft Teams User Voicehttps://microsoftteams.uservoice.com/ that way, your idea can reach the product team to change the current behavior of the cmdlet.
Thanks
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/office-docs-powershell/issues/6009#issuecomment-651892449, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AGR5PYLM5UCWV2BVQ25IQIDRZIER7ANCNFSM4OJABFYQ.

@koxo  Thank you very much for your kind feedback. Considering your valuable feedback, as advised please log this issue as a User Voice. Concerning team will evaluate and take the appropriate action, as per their schedule. We proceed here to close it. Thanks again for taking out some time and sharing your feedback.


How Do I Insert Change Objects Captured In Another Codemirror Document

I am building a realtime collab and am using codemirror. How do i insert a list of change objects into another editor. Both editors started with the same content and am trying to replicate what happens on one editor to the other. Thank you in advance for any help you will give.
@marijnh if you have the time, could you take a quick second and look at this? Help would be much appreciated. I really love codemirror and I think this issue has a really simple answer but all of my research has resulted in nothing informative. Thanks!

This works. Weird not included in documentation.

This isn't the case anymore in recent versions of the library, so of course the current docs don't talk about it.

oh, I am using a old version. Thanks


feat: add friendly tips for helmfile config order

feat: add kindful tips for helmfile config order.
check bases environments releases key order. And show more friendly tips.

/cc @mumoshu @xiaomudk

Great job!
I think the code can be simplified, here is my description of my ideas in python:
#!/bin/python3
topkeysPriority = {
	"bases": 0,
	"environments": 1,
	"releases": 2
}

def check(keys):
	if len(keys) == 1:
		return True

	for i in range(1, len(keys)):
		key1 = topkeysPriority[keys[i-1]]
		key2 = topkeysPriority[keys[i]]
		if key1 >= key2:
			return False
	return True

print(check(['bases']))
print(check(['bases', 'releases']))
print(check(['bases','environments' ,'releases']))
print(check(['environments', 'bases']))
print(check(['bases', 'bases']))
print(check(['releases', 'bases']))

/cc @mumoshu @xiaomudk

@yxxhero Hey! Would you mind clarifying why you need this? Since Helmfile v1 we are going to force environments and releases to be in separate helmfile.yaml parts via --- so there should be no need to force a specific key order at least for helmfile v1.
Also, this seems like a backward incompatible change. Can we turn this on only for v1, or make it just a warning(don't force-exit helmfile) for v0?

@mumoshu thanks for your option. let me show an example:
repositories:
  - name: bitnami
    url: https://charts.bitnami.com/bitnami
releases:
  - name: test
    chart: bitnami/nginx
    namespace: {{ .Values.ns }}
---
environments:
  default:
    values:
      - ns: test2

it will has errors.
err: error during a.yaml.gotmpl.part.0 parsing: template: stringTemplate:7:25: executing "stringTemplate" at <.Values.ns>: map has no entry for key "ns"
changing working directory back to "/root"
in ./a.yaml.gotmpl: error during a.yaml.gotmpl.part.0 parsing: template: stringTemplate:7:25: executing "stringTemplate" at <.Values.ns>: map has no entry for key "ns"

if we change the helmfile to the below:
environments:
  default:
    values:
      - ns: test2
---
repositories:
  - name: bitnami
    url: https://charts.bitnami.com/bitnami
releases:
  - name: test
    chart: bitnami/nginx
    namespace: {{ .Values.ns }}

all is ok.
So I think we should add the tips for the helmfile uses. reduce some of the confusion.
/cc @mumoshu

/cc I will update the code as you advised.

@yxxhero Ah, that makes sense! Thanks for clarifying.
Could you also somehow make this warning to be emitted only when there's --- in-between?
I think this is ok-ish in v0 and not allowed in v1:
releases:
  - name: test
    chart: bitnami/nginx
    namespace: {{ .Values.ns }}

environments:
  default:
    values:
      - ns: test2

while this is wrong in both v0 and v1:
releases:
 - name: test
   chart: bitnami/nginx
   namespace: {{ .Values.ns }}

---

environments:
 default:
   values:
     - ns: test2

If you could somehow differentiate the above 2 cases in code and emit a warning only for the latter case, it looks perfect.

@mumoshu ok. I will update code.

@mumoshu warn in v0, error in v1.


metric karpenter_pods_state shows wrong data

Version
Server Version: version.Info{Major:"1", Minor:"19+", GitVersion:"v1.19.16-eks-a05fdea", GitCommit:"a05fdea9a04815782f9c4b08eed73bb2d78b07e3", GitTreeState:"clean", BuildDate:"2022-06-09T21:50:47Z", GoVersion:"go1.15.15", Compiler:"gc", Platform:"linux/amd64"}
Expected Behavior
Hello,
We are using karpenter v0.16 and are utilizing the metrics for monitoring purposes.
We have noticed that we have a pod that is in ContainerCreating state but the karpenter_pods_state shows it as pending.
for example we will use a pod named: sleep-db77db456-drf92
the pod is in ContainerCreating:
kubectl get pods -A | grep sleep-db
devops-apps-01                  sleep-db77db456-drf92                                             0/2     ContainerCreating   0          2d3h

But the metric shows it as Pending:

We want the metric to be fixed to show the current phase of the pod.
Actual Behavior
Metric shows wrong status as described in Expected Behaviour
Steps to Reproduce the Problem
to reproduce the problem, i think you need to have a pod in ContainerCreating phase and watch the metric karpenter_pods_state
Resource Specs and Logs
nothing in the logs of karpenter.

This is by design. The phase parameter matches the .status.phase of each of the pods. In the case of a pod that is in a ContainerCreating state, the .status.phase will be pending
apiVersion: v1
kind: Pod
metadata:
  annotations:
    kubernetes.io/psp: eks.privileged
  creationTimestamp: "2022-09-13T18:14:23Z"
  generateName: inflate-6d4b6466f8-
  labels:
    app: inflate
    app.kubernetes.io/name: inflate
    pod-template-hash: 6d4b6466f8
  name: inflate-6d4b6466f8-68wss
  namespace: default
  ownerReferences:
  - apiVersion: apps/v1
    blockOwnerDeletion: true
    controller: true
    kind: ReplicaSet
    name: inflate-6d4b6466f8
    uid: 19e3f68c-3843-4399-8b91-ccc97348b30a
  resourceVersion: "12125880"
  uid: 4d5a8967-a04f-4f0b-a228-3bc4f2402ded
spec:
  containers:
  - image: public.ecr.aws/eks-distro/kubernetes/pause:3.2
    imagePullPolicy: IfNotPresent
    name: inflate
    resources:
      requests:
        cpu: "1"
    terminationMessagePath: /dev/termination-log
    terminationMessagePolicy: File
    volumeMounts:
    - mountPath: /var/config
      name: config-vol
    - mountPath: /var/run/secrets/kubernetes.io/serviceaccount
      name: kube-api-access-qn8jn
      readOnly: true
  dnsPolicy: ClusterFirst
  enableServiceLinks: true
  nodeName: ip-172-31-4-156.us-west-2.compute.internal
  preemptionPolicy: PreemptLowerPriority
  priority: 0
  restartPolicy: Always
  schedulerName: default-scheduler
  securityContext: {}
  serviceAccount: default
  serviceAccountName: default
  terminationGracePeriodSeconds: 0
  tolerations:
  - effect: NoExecute
    key: node.kubernetes.io/not-ready
    operator: Exists
    tolerationSeconds: 300
  - effect: NoExecute
    key: node.kubernetes.io/unreachable
    operator: Exists
    tolerationSeconds: 300
  volumes:
  - configMap:
      defaultMode: 420
      items:
      - key: log_level
        path: log_level
      name: log-config
    name: config-vol
  - name: kube-api-access-qn8jn
    projected:
      defaultMode: 420
      sources:
      - serviceAccountToken:
          expirationSeconds: 3607
          path: token
      - configMap:
          items:
          - key: ca.crt
            path: ca.crt
          name: kube-root-ca.crt
      - downwardAPI:
          items:
          - fieldRef:
              apiVersion: v1
              fieldPath: metadata.namespace
            path: namespace
status:
  conditions:
  - lastProbeTime: null
    lastTransitionTime: "2022-09-13T18:14:23Z"
    status: "True"
    type: Initialized
  - lastProbeTime: null
    lastTransitionTime: "2022-09-13T18:14:23Z"
    message: 'containers with unready status: [inflate]'
    reason: ContainersNotReady
    status: "False"
    type: Ready
  - lastProbeTime: null
    lastTransitionTime: "2022-09-13T18:14:23Z"
    message: 'containers with unready status: [inflate]'
    reason: ContainersNotReady
    status: "False"
    type: ContainersReady
  - lastProbeTime: null
    lastTransitionTime: "2022-09-13T18:14:23Z"
    status: "True"
    type: PodScheduled
  containerStatuses:
  - image: public.ecr.aws/eks-distro/kubernetes/pause:3.2
    imageID: ""
    lastState: {}
    name: inflate
    ready: false
    restartCount: 0
    started: false
    state:
      waiting:
        reason: ContainerCreating
  hostIP: <IP_ADDRESS>
  phase: Pending
  qosClass: Burstable
  startTime: "2022-09-13T18:14:23Z"

Closing since this is expected behavior based on the Pod phase


Optimize the PathRenderer in MonumentSwapBlock to cull offscreen textures

Not a huge fan of the complexity of doing a bounds check for every texture draw, but I don't think people are going to need to touch this code again so it should be fine lol.


Fix a bug with syncing services

Avoid showing "there is no service in the cluster" when syncing a
service for the cluster if the operator has been restarted after
the cluster had been created.

Coverage increased (+0.002%) to 11.329% when pulling 37caa3f60b9a92aa2f94e65510fbf687130e04eb on assign_existing_service_definition_during_sync into 8f08bef67cac54503c4794145f63421d0446b0c4 on master.

👍

:+1:


Cannot see any server in server list

System Information (please complete the following information):

Model Builder Version: Date Installed: 9/1/2020 Version: <IP_ADDRESS>1102
Visual Studio Version: Microsoft Visual Studio Professional 2019 Version 16.7.2

Describe the bug
On the Choose Data Source dialog the list of servers is never populated.
To Reproduce
Steps to reproduce the behavior:
When I add a machine learning value prediction, I select Local CPU environment, then I select SQL Server option for input. When I select the  option I select Microsoft SQL Server option:

The Connection properties show no SQL Servers:

When I select the Refresh option it still finds no servers. However, when I enter the server IP, the SQL Server Authentication, I can select the table. I can also select Test Connection and I get a Test connection succeeded message.

If there is no IP entered and I select Refresh, I get the message that a list of servers are being retrieved:

Once this message goes away there are still no servers listed even I select the drop down.

Should I not see the server in the list?

Well, since in the second attempt, your server name is empty. So SQL server connection service won't know which server to connect to and therefore no table/server will be listed.

I think you missed the point, if you look above, after selecting refresh there is a dialog that shows "Please wait while we retrieve the list of servers ...". When this dialog is closed there are no servers in the list. Yet if you look above that I have connected to a server by giving the IP and it works. This proves there is a server on the network and the refresh fails to find that server.
This is the issue: at no time is a live SQL Server put into this dropdown. Either there is an issue with the server discovery, or the dialog should be changed to force the user to input the server IP/host. It is very confusing to a user to have a dropdown and refresh button that does not work.

Oh I see your point now, The sql connect tool is from Visual studio, and model builder simply calls it from VS SDK. (Tools -> Connect to Database -> Microsoft SQL Server). Why the team which owns this tool puts a dropdown list and refresh button that doesn't work is unclear though, they may miss that, or they might have a specific reason to design that we don't know. We will deliver your feedback to them to find the answer

This is most likely by design. The server list (which will be repopulated if you click the Refresh button) includes all available instances of SQL Server within the local network. The list is returned by System.Data.Sql.SqlDataSourceEnumerator.Instance.GetDataSources(). The list may be empty if there is not SQL Server instance in your local network.

@ddemland Does the response above explain the difference?

Closing this. If the problem persists please ping me an re-open the issue. Thanks!


Undo/Redo ignores transactions.

Can fix this by grouping mutations by the start and end events the spy function catches.

#45


GMX-1968 remove poisonous Hashie

@jstoup111 @nankrah still need to check any potential downstream effects, but the existing specs are all green
cc @asurin

Love it!

Think it makes sense to use HashWithIndifferentAccess.
I know it's an ActiveSupport dependency, but might be worthwhile looking into.

Looks good otherwise.


Refactor and add error codes

Ok, big commit here, apologies for this. I initially started out by trying to just add the error codes but that meant I needed to touch a lot of code in order to do so. Regardless I think it was a good idea to do this now rather than later on, before we run into any issues.
Firstly, I added an enum which describes all the error codes which we might run into, including internal errors. This allows functions to indicate if it succeeded (i.e it returned E_NONE which is equal to zero) or if it failed (it returned another tx_err value as defined in err.h).
I renamed the error handling functions because I didn't believe they were accurate: tx_gl_error_callback now becomes tx_glfw_error_callback, tx_gl_geterr becomes tx_geterr again (because it handles GLFW and OpenGL errors while also checking to see if the error log is clear alltogether) and tx_gl_print_errors now becomes tx_print_errors again too (because it prints any sort of error that might find itself in the error log).
Secondly, I extracted a couple of FreeType functions into font.c, because we're probably going to need that eventually even if not right now.
One current thing to note is that the path to the font at /usr/share/fonts/corefonts/arial.ttf isn't vaild on either of the systems I tried (OSX, Ubuntu). Obviously it is just temporary but it still means that I can't test things out, so we may want to change that.
Sorry for the wall of text, I should be able to get started on the OpenGL rendering stuff today or tomorrow.
Let me know if there are any issues.

Looks like now this PR has unresolved merging conflicts.

@Bfgeshka probably my fault when running make format on everything in one of my commits.

Let me just check that actually compiles now...

@desolt  It's mostly my fault because I didn't run it after changing astyle config.

Compiles fine here (with a couple of unused variable warnings from @desolt's PR, but nothing to worry about).


Fix changeLog generation test under Windows

Under Windows generated changeLog contains mix of \n and \r\n line endings. It should be normalized before comparison with expected value.

P.S.
Test diff output:
\n# [(2).0.0](https://github.com/tschulte/gradle-semantic-release-plugin/compare/v1.0.0...v2.0.0) (2018-02-19)\n(\r)\n...
\n# [(2).0.0](https://github.com/tschulte/gradle-semantic-release-plugin/compare/v1.0.0...v2.0.0) (2018-02-19)\n(-~)\n...

(former is generated changeLog, latter is expected).
If you believe this is a bug then we should fix implementation instead.

Coverage remained the same at 91.727% when pulling 0e8c24df6ace995f78fae0d1b64d4b28629fb937 on grv87:patch-1 into e5a442c91e7437c233adec1f0b73ceee32681b65 on tschulte:master.


[CLOSED] Run sidekiq when running with bowl

 Issue by tommyp
Tuesday 12 August 2014 at 12:40 GMT
Originally opened as https://github.com/alphagov/specialist-publisher/pull/223

Now with sidekiq workers, we need to run the App with foreman to run sidekiq and the web when using bowl. This commit adds foreman and the web worker to the Procfile.

tommyp included the following code: https://github.com/alphagov/specialist-publisher/pull/223/commits

 Comment by bradwright
Tuesday 12 August 2014 at 12:47 GMT

Should foreman be included in this project's Gemfile? It's not in any other projects - we expect that to be installed in the development environment.

 Comment by tommyp
Tuesday 12 August 2014 at 15:16 GMT

@bradleywright Because it's not in the bundle, bowl can't run it.

 Comment by tommyp
Tuesday 12 August 2014 at 12:49 GMT

I wasn't sure if I should include it or not. Shall I remove it?


[IndexStore] Enable blocks support on Linux builds

This enables block support for Linux builds of IndexStore. The motivation is to have the various _apply functions be present (ex indexstore_store_units_apply). Without this, only the function pointer variants are available (ex indexstore_store_units_apply_f).

Note that the Swift 5 Linux release includes a libIndexStore.so that is missing those symbols.

cc @benlangmuir


update zemu image

Zemu docker images was updated. Zeroize RAM can be reproduced now on Speculos.

@ftheirs this will need an update of the test elf


fix: support < v10.1.0-next.6 library projects

Describe the bug
Starting from Angular v10.1.0-next.6, the library builder has changed:
-"builder": "@angular-devkit/build-ng-packagr:build",
+"builder": "@angular-devkit/build-angular:ng-packagr",

This causes library builds versions prior to v10.1.0-next.6 to fail since we're targeting a builder that does not exist.
Failing pipeline:
https://github.com/chihab/ngx-access/actions/runs/703138676
ng-packagr builder update: v10.1.0-next.6
https://github.com/angular/angular-cli/commit/da9825ceb279807eb65f6cff65e5da83209d9d2a#diff-15ae6659598d4085a6e89c51399950e056dddc736a2c78a0affd515d9feae072
To Reproduce
Besides the action execution above, I run the action locally  (replaced the @actions/core with static values) on a Angular 11.x project
-  const angularVersion: string = core.getInput('angular-version');
+ const angularVersion: string = process.argv[2];

node ../angular-versions-action/dist/index.js 10.0.x
package.json dependencies update looks okay.

npm install && npm run build

Expected behavior
I expect the library to build successfully 😀
Possible solutions
In Lumberjack you seem to have the deprecated builder name as a dependency which explains why it works with the action. So either:
Solution1 (for any Angular version):

Add the legacy builder dependency in package.json
Replace the library builder in angular.json

Solution2 (for "legacy" Angular versions):

Add the legacy builder dependency in package.json
Update angular.json according to the target angular version.

By the way, to use core.getInput('angular-version') locally, I think you need to set an environment variable such as INPUT_ANGULAR_VERSION=10.0.x. Do you remember if this is right, @NachoVazquez?

By the way, to use core.getInput('angular-version') locally, I think you need to set an environment variable such as INPUT_ANGULAR_VERSION=10.0.x. Do you remember if this is right, @NachoVazquez?

I think that's the way, it varies on Windows, Mac, and Linux I think

We have come up with a plan for fixing this. As we need to touch angular.json, we're going to introduce breaking changes and release this bugfix as part of v4.0.

@chihab
Please use the v4 tag to fix this issue.
- uses: ngworker/angular-versions-action@v4
  with:
    # or any other supported version
    angular-version: 12.0.x
    # (optional parameter) the workspace root folder containing `package.json` and `angular.json`
    root-path: ./

Thank you @NachoVazquez!

Thank you @NachoVazquez!

Thank you @chihab ! Now the Action is better after your suggestion. Awesome!


WDYT about defining your schema in an objective way

I have created this library https://github.com/simPod/GraphQL-Utils to power up creating schema definitions (and some other things like DateTime scalar etc.)
I'm using it for a while on several projects and I'd love to know what do you think about it or even better, if you could try it. Current state is the best I alone could come up with, there are no more tweaks I can think of so when other people had a look, it would help.
I built it because I hate when there are huge/many arrays in PHP and defining schema just with arrays is very vulnerable to human errors (typos+need to remember docs) / cannot be statically tested.
If you find it handy, would be cool if it could be added to "suggested" dependencies. If not, just ignore.
Thanks for feedback! Sorry if OT

This is awesome! We actually have related issue https://github.com/webonyx/graphql-php/issues/70 but having such tools in the separate repo makes total sense to me. So I'll close it and recommend using your library instead.
Please post a PR with a link to your project in complementary tools doc.


WIP - Streams

Fixes #

[ ] Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR).
[ ] Title summarizes what is changing.
[ ] Has a news entry file (remember to thank yourself!).
[ ] Appropriate comments and documentation strings in the code.
[ ] Has sufficient logging.
[ ] Has telemetry for feature-requests.
[ ] Unit tests & system/integration tests are added/updated.
[ ] Test plan is updated as appropriate.
[ ] package-lock.json has been regenerated by running npm install (if dependencies have changed).

Codecov Report

Merging #11154 (9b8c429) into main (bc5b95c) will decrease coverage by 0%.
The diff coverage is 66%.

@@           Coverage Diff           @@
##            main   #11154    +/-   ##
=======================================
- Coverage     63%      62%    -1%
=======================================
  Files        485      485
  Lines      34100    34103     +3
  Branches    5561     5562     +1
=======================================
- Hits       21586    21482   -104
- Misses     10452    10554   +102
- Partials    2062     2067     +5

Impacted Files
Coverage Δ

...c/kernels/execution/cellExecutionMessageHandler.ts
70% <66%> (-8%)
:arrow_down:

src/notebooks/outputs/errorRendererComms.ts
26% <0%> (-29%)
:arrow_down:

src/notebooks/controllers/kernelSelector.ts
28% <0%> (-15%)
:arrow_down:

...notification/pythonExtensionRestartNotification.ts
90% <0%> (-10%)
:arrow_down:

src/kernels/execution/helpers.ts
61% <0%> (-7%)
:arrow_down:

src/kernels/installer/productInstaller.node.ts
72% <0%> (-7%)
:arrow_down:

src/kernels/kernelDependencyService.node.ts
80% <0%> (-5%)
:arrow_down:

src/standalone/api/kernelApi.ts
69% <0%> (-5%)
:arrow_down:

src/notebooks/notebookCommandListener.ts
49% <0%> (-4%)
:arrow_down:

... and 18 more


Error: error: src refspec main does not match any

The following error is shown when the versioning command is run,
nx run button:version --releaseAs=prerelease --preid=beta --skipRootChangelog --push

Error: error: src refspec main does not match any
error: failed to push some refs to 'https://**.git'

It works as expected when the same command from the log is run.
✔ bumping version in libs/button/package.json from 0.0.1-beta.1 to 0.0.1-beta.2
✔ outputting changes to libs/button/CHANGELOG.md
✔ committing button/package.json and libs/button/CHANGELOG.md and all staged files
✔ tagging release button-0.0.1-beta.2
ℹ Run git push --follow-tags origin feature/versioning` to publish

Running git push --follow-tags origin feature/versioning in CLI works as expected.

@maheshambiga What is your default branch? main branch seems not to be your default branch.

@edbzn - main is not my default branch. I am expecting the changes to be pushed to the current upstream branch.

@edbzn  Setting --baseBranch seems to have solved the issue.
Closing the issue. Thank you.


Can we integrate this framework with VUE JS.

I love this framework. Very good job. And how can we help you ?

I don't really understand how you want do to that, i mean, VueJS is made for front-end and this library made for back-end, i never really used Vue but if you can add global function you can make something like that https://gist.github.com/felixdorn/c192d3b21a0ccc41d230655bd5def0e4

In future version i'll add support for twig and maybe VueJS if you explain more clearly what's your problem.


BugFix: Update useWorkPoolLastPolled to not accept a filter

Description
The useWorkPoolLastPolled composition accepted a filter which was being used to limit the work queues. But this defeats the point of knowing then the work pool was last polled. Removing that filter to get an accurate time

I think you'll want to also cleanup the filter being passed in here, otherwise lgtm! Thanks for catching that.

Nice catch. Removed!


When updating the security stamp, tokens for confirm email is invalid

I'm not sure if this is by design or not, but I think it's broken.
We had a user which created a new account, got an email from the system to confirm his email address.
Then he logged out, asking to log out from all devices, so we needed to use UpdateSecurityStampAsync and then he tried to confirm his email address which failed because the security stamp has changed.
Something in the framework IMHO needs to be changed to support this (not bizarre) scenario.

This behavior is currently by design, whenever the security stamp is invalidated, the default email token provider generates tokens with the security stamp embedded for verification.  Log out everywhere invalidates all security tokens associated with the account, including confirmation/forgot password/etc.
Any concerns with this @blowdart ?

It obviously makes sense.
But I think there should be a way of not invalidating email tokens, probably should be an opt-in and not the default behavior.

You can always implement your own email token provider that does not use the security stamp as part of its validation.

The other approach is to set RequireConfirmedEmail to be true. Or they just request another password reset and the stamp will be regenerated and finally work.
It's weird, but it is secure :) As you can do it yourself and it's by design, I'm going to close this as wontfix.

@blowdart I didn't understand what you meant by.

The other approach is to set RequireConfirmedEmail to be true.

https://github.com/aspnet/Identity/blob/dev/src/Microsoft.AspNetCore.Identity/SignInOptions.cs#L15

Basically, if you require users to have confirmed emails before allowing them to log in, that would mitigate the issue somewhat.

@HaoK That's a no go for us (and most applications I know to be honest).
Anyway, any chance you will consider adding a flag for the EmailTokenProvider in the future into the framework in the future?


Application-specific Comparsion

$ scryer-prolog -v
v0.9.4-46-g42a0d686
$ tpl -v
Trealla Prolog (c) Infradig 2020-2023, v2.51.8-5-gbcb2
$ scryer-prolog -f
?- use_module(library(lists)).
   true.
?- [user].
[a,b,c].
   error(permission_error(create,static_procedure,nameless_procedure),load/1).
?- a.
   error(existence_error(procedure,a/0),a/0).
?- halt.

$ tpl
?- use_module(library(lists)).
   true.
?- [user].
[a,b,c].
   true.
?- a.
   true.
?- halt.

% And
$ tpl
?- [user].
[[a]].
   true.
?- [user].
[[a]].
Warning: overwriting '.'/2 % Meaning?
   true.
?-

Where am I? Where are you off to?

I believe it appeared with term_expansion, which can return a list of terms. It's acceptance here is of course is unintended, though cute.

SWI accepts it with a warning...
$ swipl
Welcome to SWI-Prolog (threaded, 64 bits, version 9.3.3-40-g6d3b1fa0b-DIRTY)
?- [user]. [a,b,c].
Warning: user://1:7:
Warning:    Clauses of a/0 are not together in the source-file
Warning:    Earlier definition at user://1:7
Warning:    Current predicate: c/0
Warning:    Use :- discontiguous a/0. to suppress this message
Warning: user://1:7:
Warning:    Clauses of b/0 are not together in the source-file
Warning:    Earlier definition at user://1:7
Warning:    Current predicate: a/0
Warning:    Use :- discontiguous b/0. to suppress this message
Warning: user://1:7:
Warning:    Clauses of c/0 are not together in the source-file
Warning:    Earlier definition at user://1:7
Warning:    Current predicate: b/0
Warning:    Use :- discontiguous c/0. to suppress this message
|:
% user://1 compiled 0.00 sec, 6 clauses
true.
?- a.
true ;
true.
though how it is a redefinition elludes me.

Due to weakness of brain I am inclined to the view that that is that is that.

$ swipl # 9.2.4
?- [user].
|: [[[[[[a]]]]]].
|: ^D% user://1 compiled 0.01 sec, 64 clauses
true.

?- a.
true ;
true ;
true ;
:
:
?-

Et cum spíritu tuo. 64 clauses?


Definition jumps to local functions that should not be able to be referenced

localFunction() is a local function that can only be referenced in someFunction().
it should not be referenced in main.m, but you can jump to its definition.
I think the error should be shown, but even MATLAB editor does not show it, so if you could at least prevent me from jumping to the definition, it would help prevent the bug.

This should be resolved in v1.1.1.


Cold performance of ResourceManagerStringLocalizer is atrocious

Consider a localized version of the home page in the standard ASP.NET starter web template. You get around 50 localized strings. This results in 50 calls to ResourceManager.GetString that will throw MissingManifestResourceException when using the base language (i.e. one that doesn't have a specific culture assembly). That takes about 6 seconds. The next call is super fast as everything is then cached (~15ms).
Need to explore how we could possibly make that initial render faster.

I have added a new issue, could be this in relationship with my issue? Thanks!
https://github.com/aspnet/Localization/issues/252

@ryanbrandenburg can you take a look to see where the time is being spent at startup?

To be clear, this is about an app that is "localizable" but not "localized." But it would also be interesting to see what differences there are when the app is localized.

I don't see this terrible performance when running a test app from the console, but it IS really slow when run from VS (whether the debugger is attached or not). Console gives us ~10 requests/ms vs VS taking 289ms per request. Given that info, this sounds more like a VS bug than a Localization problem.

Bear in mind the date this was reported, the underlying systems may have been improved. For instance if you were to attach a debugger and verify that those exceptions are no longer thrown at all (first chance) then that's a sign that the underlying issue was fixed.

@ryanbrandenburg how many localized string callsites were there in your test app?

I set up a double for loop going through 5 StringLocalizers looking for 100 different strings (1-100), so in total it looked for 500 strings which shouldn't have been in the cache.
Yeah, I'm keeping in mind that this could have already been resolved for the console case by another fix (even inadvertently) but I've debugged this and we definitly still throw the exceptions here.

Thanks. Seems it no longer repros then.

My best guess on the difference here is that the version of ResourceManager used in netcoreapp1.1 seems to be ~13x faster than the one in net451. The project I'm using to test this is both net451 and netcoreapp1.1, running 'dotnet run' was giving me netcoreapp and VS was defaulting to net451 and I didn't notice. I think we can close this one out as having been fixed.


Ldap configuration for Kafka

How to add ldap configurations using cp-ansible role ?

Can be added using a variable - https://docs.confluent.io/ansible/current/ansible-authorize.html#optional-settings-for-rbac
We don't have a role for configuring an ldap server. That's owned by user.
Our docs are quite helpful. I'd highly suggest going through the docs properly before opening an issue.


Streaming function and ddb_to_es.py generate different _id in elastic search.

Describe the bug
This conversation was started on https://github.com/aws-amplify/amplify-cli/issues/3705.
Relevant comments are:
https://github.com/aws-amplify/amplify-cli/issues/3705#issuecomment-605118467
https://github.com/aws-amplify/amplify-cli/issues/3705#issuecomment-605125513
https://github.com/aws-amplify/amplify-cli/issues/3705#issuecomment-605128083
https://github.com/aws-amplify/amplify-cli/issues/3705#issuecomment-605151484
https://github.com/aws-amplify/amplify-cli/issues/3705#issuecomment-605160134
https://github.com/aws-amplify/amplify-cli/issues/3705#issuecomment-605202981
https://github.com/aws-amplify/amplify-cli/issues/3705#issuecomment-605206802
https://github.com/aws-amplify/amplify-cli/issues/3705#issuecomment-605209743
The ddb_to_ess.py (from https://github.com/aws-amplify/amplify-cli/pull/3712) causes the IDs for elasticsearch to be different from what the autogenerated streaming lambda creates.
When I ran the ddb_to_ess.py script to delete duplicates it did indeed delete duplicates. However, it made the "_id" for the items to be formatted as ID|SLUG.
However, the streaming lambdas create the "_id" as just ID. This is happens after updating, even with the lambda that checks for the key.
So if I use the ddb_to_ess.py to try and migrate up, then any time I update an item it creates a new elastic search record with the _id being just ID.
Example difference:
For an item with the ID of D-436-36 and slug of d-436-miniseal-crimp-splice-26-20-awg-gauge-wire-d-436-36.

ddb_to_ess.py ID - D-436-36|d-436-miniseal-crimp-splice-26-20-awg-gauge-wire-d-436-36
Streaming Lambda ID - D-436-36

I am using a @key for a search by slug. But it should not be the primary key. The ID is still a valid primary key.
Amplify CLI Version
Amplify v4.17.1
To Reproduce
Have a key for the slug, but this is just to create a query by slug @key(name: "BySlug", fields: ["slug"], queryField: "productsBySlug")
Run the ddb_to_es python script to remove duplicates. This makes all items create with the _id of ID|SLUG.
Update one of the items. This creates a new item with the _id being ID
Expected behavior
The _id generated byt the ddb_to_es.py function should be the same as the ones created by the streaming function.
Desktop (please complete the following information):

OS: macOS Catalina 10.15.3
Node Version. Node 10.15.3 (Yup, happens to be the same as the OS.)

An email was sent to the amplify team with the pertinent code.

@SwaySway

@simeon-smith  I am experiencing the same issue as you do.
DdbToES Lambda function as the same with v4.17.2's.
The _id generated by the DdbToEsFn together with the ddb_to_es.py are not aligned with the lambda trigger.
For now, i am hardcoding my ddb_to_es.py and DdbToEsFn to my table's primary key.

For those who need's a quick fix:
in ddb_to_es.py change:
ddb_keys_name = [a['AttributeName'] for a in table.attribute_definitions]
to
ddb_keys_name = [a['AttributeName'] for a in [{ 'AttributeName': "<YOUR TABLE PRIMARY KEY HERE" }]]
e.g.
ddb_keys_name = [a['AttributeName'] for a in [{ 'AttributeName': "id" }]]
This solution will only work if:

You are using v4.16.1 and up
Your DdbToEsFn is updated

Any updates on this?

Hello @simeon-smith
Looking at the other linked issue it is possible that you saw different behavior do a change in stream arn. Was that the case in the other issue you listed?

No, this caused the troubleshooting that ended up in the stream arn getting changed.

@simeon-smith @SwaySway quick question - is it possible to modify ddb_to_es.py locally and it be included when i do amplify push? I want to do some custom bits in it

@SwaySway Having the same issue
generated script _id:
2020-07-21T18:56:00.468Z|P-ABC-123123123123123|05d6d41b-81c0-4287-a1b4-6a0d10e00a27
if I update an item in dynamo, generates new entry with _id:
2020-07-31T07:32:02.795Z|05d6d41b-81c0-4287-a1b4-6a0d10e00a27
This appears to be due to a secondary index @key(name: "bySomeKey", fields: ["someFIeld", "createdAt"]) on this model.
Any user updates triggers duplicate entries

Looking at the Key portion, which is what the script determines as the primary id for the record, includes keys from given GSI/LSI. As of now @searchable only supports @key when changing the primary KeySchema. Labeling this as a bug to prioritize.
There are a few ways to address this

Check against table to get KeySchema
Pass some sort of mapping into the lambda to know which Keys to use
Will review with the team and provide an update on this thread.

Closing this issue as the fix is merged. The latest migration script will work on tables with LSI/GSIs.


Usage of includeXxx parameters

In Fractal it's possible to give includeXxx methods a ParamBag containing parameters (even if it isn't very clear about it).
Even in the documentation of laravel-responder there is an example which takes in an array (instead of a ParamBag), but it isn't explained how to use them in any way...
I tryed calling the with() method with the param syntax of fractal like
->with('roles:param(value1|value2)')
but I got an error in response.
Is there a way to actually provide parameters to an includeXxx method on a transformer class when using a return responder()->success($data)->respond() chain in a controller method?

Ahh, just tested this out and realised this doesn't work as intended. Basically, Fractal's parseIncludes method will do the parsing of parameters. However, the package calls this method after setting available relations on the transformer(s), meaning it will try to include relations in the transformer with the full parameter query which obviously fails. I think the easiest way of solving it is to strip out the parameters only when setting available relations, then send the untouched relation with parameters to Fractal for parsing. Sorry for the issues, I'll get this sorted as soon as possible!

Also, the package converts the ParamBag to an array before passing it the includeXXX methods in an effort to stay as decoupled from Fractal as possible.

Also, the package converts the ParamBag to an array before passing it the includeXXX methods in an effort to stay as decoupled from Fractal as possible.

Good idea!

Sorry for the issues, I'll get this sorted as soon as possible!

You've done a great job doing this and releasing it under MIT license, I don't think you shall feel "sorry" in any way :P
I'll just wait for the fix (I'll start contributing to the library myself when I finish exploring it in depth probably, just to avoid to mess everything up), the one you proposed seems simple and straight so it must be the right one :)

Also, I would add a small paragraph about this feature (and the fact that it returns an array and not a ParamBag) in the docs if possible

Appreciate the kind words! And yeah, I'll get it fixed and added to the documentation, must have slipped my mind when writing it

I fixed this following the hint you gave.
I added a function at the end of TransformBuilder like this
/**
 * Remove parameters from relations that must be loaded.
 *
 * @return array
 */
protected function relationsWithoutParameters(): array
{
    return array_map(function($value) {
        return explode(':', $value)[0];
    }, $this->with);
}

And here i changed
$data->load($this->with);

with
$data->load($this->relationsWithoutParameters());
Now it works fine.
If you don't have time to do it, next week I'll probably clone the repository, add tests and create pull requests


Switch from nbstripout to nbclean

Seems that nb-clean is a good alternative to nbstripout.

It can also remove empty cells
It removes empty tags (?) -- see #350
It seems to have more frequent releases

See https://github.com/ComPWA/gluex-nstar/pull/91#issuecomment-2304754891


Hash table variants and optimizations

Introduces an alternative F14 style interleave layour for HashTable. This layout alternates tags and pointers to payload. Each group takes two cache lines: 16 bytes of tags, 8 * 48 bytes of pointers in the first cache line and 8*48 bits of pointers in the second cache line. A hash table that is expected to hit frequently, i.e. a group by table, will benefit from this layout since the tags and pointers most often share one cache miss and always share the TLB lookup.
Adds optional counters for hash lookups and collisions.
simplifies the hashProbe concept by passing the table as an argument as opposed to different fields of the table.
Uses SIMD for an array hash table probe. Previously this was limited to group by tables.

The loop conditionmeans that I + 7 is in range. It is not clear what you mean by batch sizes 2 or 4. Less than 8 is not considered here because it is considered in the next loop.  The test cases have every kind of non-multiple of 8 number of inputs.

@oerling On AVX2 we issue 2 gathers to cover the 8 words.  On SSE or NEON you would need 4 gathers.


Document UseDataLoader()

Is your feature request related to a problem?
UseDataLoader was added in https://github.com/ChilliCream/hotchocolate/issues/2505 but there doesn't seem to be any documentation around it.
The solution you'd like
Document it maybe here? https://chillicream.com/docs/hotchocolate/fetching-data/dataloader
Product
Hot Chocolate

Do you want to do a pull-request?

Hi, I'm new to HotChocolate, that's why I'm asking about this documentation :slightly_smiling_face:

I am curious about the DataLoader middleware too. The graphql-workshop  repository mentions (just above this) in session 6 that using a resolver alongside a dataloader causes paging to happen in memory:

There is one caveat in our implementation with the TrackType. Since, we are using a DataLoader within our resolver and first fetch the list of IDs we essentially will always fetch everything and chop in memory. In an actual project this can be split into two actions by moving the DataLoader part into a middleware and first page on the id queryable. Also one could implement a special IPagingHandler that uses the DataLoader and applies paging logic.

Would UseDataLoader() middleware resolve this inefficiency?

I believe this is still relevant. I have been unable to find documentation on UseDataLoader, which seems like a key component in stacking middlwares for custom resolvers.
@michaelstaib, can you shed some light on how this method should be used?


iOS project   Reference missing

Hi,
When i try to build/deploy on iOS, the project throws an exception :
error : Can't resolve the reference 'CoreLocation.CLLocationCoordinate2D Mapbox.MGLCoordinateBounds::sw', referenced from the method 'System.Threading.Tasks.Task1<Naxam.Controls.Mapbox.Forms.OfflinePack> Naxam.Controls.Mapbox.Platform.iOS.OfflineStorageService::DownloadMap(Naxam.Controls.Mapbox.Forms.OfflinePackRegion,System.Collections.Generic.Dictionary2<System.String,System.String>,Naxam.Controls.Mapbox.Forms.IOfflineStorageDelegate)' in 'Naxam.Mapbox.iOS, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null'._
My android project works fine.
Best Regards.

Hi @Fraguelrok
Sorry for the late reply. Please try our latest package to see if the problem remains.
Regards.

Plz reopen if you still get it, @Fraguelrok


SSO Setup

Is your feature request related to a problem? Please describe.
It is not clear if the SSO is setup correctly and how to do that.
Describe the solution you'd like
There should be a prod and a local configuration. The prod configuration uses the ldap, the local one does not. Local contains for it testuser in an own realm.
Additional context

create sso folder in monorepo
create configuration files
update example and prod docker-compose yaml

Should the whole Keycloak app be inside the sso folder or just the config files? And is the config file supposed to be a config for the whole keycloak inside our app or just for the sso topic?

@dominikhorn93


New package: Hnsw v0.0.1

Registering package: Hnsw
Repository: https://github.com/jean-pierreBoth/hnsw-jl
Created by: @jean-pierreBoth
Version: v0.0.1
Commit: 2b104b4497236377b5d006d82f8f52f1a97de452
Git reference: master

I will change the name of the repository


Register package

@JuliaRegistrator register branch=main

Registration pull request created: JuliaRegistries/General/116674
Tip: Release Notes
Did you know you can add release notes too? Just add markdown formatted text underneath the comment after the text
"Release notes:" and it will be added to the registry PR, and if TagBot is installed it will also be added to the
release that TagBot creates. i.e.
@JuliaRegistrator register

Release notes:

## Breaking changes

- blah

To add them here just re-invoke and the PR will be updated.
Tagging
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.0 -m "<description of version>" 911e2b2390580a44d19d8b725472686beed0f0bd
git push origin v0.1.0

@JuliaRegistrator register branch=main

Registration pull request created: JuliaRegistries/General/116899
Tip: Release Notes
Did you know you can add release notes too? Just add markdown formatted text underneath the comment after the text
"Release notes:" and it will be added to the registry PR, and if TagBot is installed it will also be added to the
release that TagBot creates. i.e.
@JuliaRegistrator register

Release notes:

## Breaking changes

- blah

To add them here just re-invoke and the PR will be updated.
Tagging
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v1.0.0 -m "<description of version>" 0d0b3460194aca83459bcd98639fa2c09308ce5e
git push origin v1.0.0

Registration pull request created: JuliaRegistries/General/117431
Tip: Release Notes
Did you know you can add release notes too? Just add markdown formatted text underneath the comment after the text
"Release notes:" and it will be added to the registry PR, and if TagBot is installed it will also be added to the
release that TagBot creates. i.e.
@JuliaRegistrator register

Release notes:

## Breaking changes

- blah

To add them here just re-invoke and the PR will be updated.
Tagging
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v1.0.1 -m "<description of version>" 449d76accd9551aa216f82b0a75555c6b9a91851
git push origin v1.0.1

@JuliaRegistrator register branch=main

Registration pull request created: JuliaRegistries/General/117433
Tip: Release Notes
Did you know you can add release notes too? Just add markdown formatted text underneath the comment after the text
"Release notes:" and it will be added to the registry PR, and if TagBot is installed it will also be added to the
release that TagBot creates. i.e.
@JuliaRegistrator register

Release notes:

## Breaking changes

- blah

To add them here just re-invoke and the PR will be updated.
Tagging
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v1.0.2 -m "<description of version>" d367740459440ecebfd0fecd383d26842745817f
git push origin v1.0.2

@JuliaRegistrator register branch=main

Registration pull request created: JuliaRegistries/General/120077
Tip: Release Notes
Did you know you can add release notes too? Just add markdown formatted text underneath the comment after the text
"Release notes:" and it will be added to the registry PR, and if TagBot is installed it will also be added to the
release that TagBot creates. i.e.
@JuliaRegistrator register

Release notes:

## Breaking changes

- blah

To add them here just re-invoke and the PR will be updated.
Tagging
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v1.1.0 -m "<description of version>" 6056e601cf04f180c96eebb5916def91d0128133
git push origin v1.1.0

@JuliaRegistrator register branch=main

Registration pull request created: JuliaRegistries/General/120790
Tip: Release Notes
Did you know you can add release notes too? Just add markdown formatted text underneath the comment after the text
"Release notes:" and it will be added to the registry PR, and if TagBot is installed it will also be added to the
release that TagBot creates. i.e.
@JuliaRegistrator register

Release notes:

## Breaking changes

- blah

To add them here just re-invoke and the PR will be updated.
Tagging
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v1.1.1 -m "<description of version>" d362b31a1a68335abcbea0f752b7de5a61b85ed4
git push origin v1.1.1


ShowSimpleChildWindow in VB.Net

Hi,
I've a usercontrol named setupchildwindow, I've imported MahApps.Metro.Controls but I'm not able to use
"Me.ShowChildWindowAsync(new setupchildwindow())" how to make my childwindow appear ?
Thanks !

@furiousbuilder can you explain a little bit more? is there any bad output? or exception or something else?

ShowChldWindowAsync is not defined, I work with VS 2013 and net framework 4.5. Is there any other way to show a simplechildwindow in VB.Net ?

@furiousbuilder any news for this? can you provide a full stack trace?

Hi,
I haven't code for 2 months due to some exams and I just formatted my drive to dual boot W10 and Mac OS X, so I need to reinstall visual studio but I think that the problem has been corrected, when VS will be reinstalled I'll provide you some code :)

MsgBox are now working in 1.2.0 but I'm still not able to use ShowMessageAsync, because it seems to be not declared

Make sure you import MahApps.Metro.Controls.Dialogs, as it provides the override methods.


Type error when I build with resizable

Type error when I build with resizable in NextJs
I didnt edit anything

@rhs419 Curious to know, how did you resolve this? Getting the same issue, and forcing version on 1.0.5 seems to fix it.

@efleury1 we solved it by upgrading typescript version 4.95 to 5.0.0. sorry for rate checking


Unnecessary addition to the rate limit for event grid domain

I feel that this sentence "5,000 events of 64 KB size per second or 5 MB per second (whichever comes first)" is misleading. I would say that this part "of 64 KB size" should be removed as later on there is information which events are counted as more than one. Here it looks like we can send 5000 events each 64 kb or 5MB so the second limit will be always first so what is the point of the first one? Probably sounds like a not important and obvious thing (once you get it) but actually to me it was really confusing...

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

ID: 7e29b6cc-f1c7-fb30-f0da-e043ec5903d6
Version Independent ID: f47e660a-b09e-10c5-8f6b-c8c51f25bf4c
Content: Quotas and limits - Azure Event Grid - Azure Event Grid
Content Source: articles/event-grid/quotas-limits.md
Service: event-grid
GitHub Login: @spelluru
Microsoft Alias: spelluru

@AdiuTankT72
Thanks for your feedback! We will investigate and update as appropriate.

@spelluru Can you share your thoughts on this.

#reassign:robece

Thank you. Removed "of 64 KB size" after discussing with the product team. The update will go live in a few hours.
#please-close


Add simulated head position data

Noticed it was missing. Using the OVR head model. Matches standard Oculus Unity integration/plugin where enabled by default so think makes sense to do the same.
Test: readily apparent in a sample with near field objects. (Or stop by.)
Minor, but could rename head_rotation_provider.h/cpp to head_pose_provider.h/cpp to be more accurate.
Signed-off-by: m.blix<EMAIL_ADDRESS>

@danke-sra, please review.  thanks.

@mkeblx We have a head-model implementation in place. Do you think we can use that one instead of the head pose provided by Oculus?
In the current change, it looks like both our head model and Oculus head model are applied. Our head model holds the CameraRig and already handles the position. However, we can review it too.

@danke-sra Interesting, I can notice no head model in effect, only seeing rotating around center-eye point. It should be quite noticeable when looking for it.
I see now head-model-parms in VR app settings XML, but could not see effect when setting to various values.

It may be broken due to other changes. I plan to check it today or tomorrow.

Sorry I just got a chance to look at this. The code looks good. +1


config(reindex): create reindex timeout configuration

disable reindex timeout by default

Checklist

[ ] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
[ ] For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

Per https://github.com/datahub-project/datahub/issues/10557


Pattern: Content block - with cards visual design & design specs provided by Reboot designers

Notes

The Reboot team is responsible for creating the visual designs and design specs for most of the patterns included in the Solutions page.  This issue is to track the transition of the Reboot design work to the DDS design & engineering team members.

Additional Information

Visual Designs & Design Specs Box folder
Links to the visual designs and design specs will be updated once received and reviewed by the Reboot designers.

Acceptance Criteria

[ ] Visual designs and specs received from Reboot designers
[ ]  A pdf file is created and placed in the Box folder with the updated Sketch file
[ ] Meetings conducted with DDS team members to review the designs

Design review meeting with DDS designers and developers conducted April 21, 2020.  Lila to post sketch and pdf files into Box.

@RobertaJHahn This one was also not completed yet. The visual designs and specs in sketch and PDF are now posted in this Box folder: https://ibm.ent.box.com/folder/111084561933