inshellisense: 0.0.1-rc.12 -> 0.0.1-rc.14

Description of changes
Update to latest version.
Diff: https://github.com/microsoft/inshellisense/compare/refs/tags/0.0.1-rc.12...0.0.1-rc.14
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

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

[x] 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
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.


Broken followProgress()

It looks like the recent release which updated JSONStream from 0.10.0 to 1.3.2 has broken followProgress().
With version 1.0.5 onFinished, onProgress callbacks are no longer being called.

Possibly related to https://github.com/apocas/dockerode/issues/439 ?
If so, then<EMAIL_ADDRESS>will fix this.

Correct :)


Update contribute-to-a-github-company-project.md

Fixed broken images for contribute to a github company project

fixed broken images on contribute to a github company project, and contribute to a gerrit project


DumpMachine output uses different ket symbols depending on the relative phase

Describe the bug
DumpMachine uses different ket symbols for states with real and complex amplitudes.
This happens both for full state simulator and for sparse simulator.
To Reproduce
The following code produces the following output on full state simulator:
use qs = Qubit[2] {
    H(qs[0]);
    Controlled Rx([qs[0]], (1.56, qs[1]));
    DumpMachine();
    ResetAll(qs);
}

The output for sparse simulator is the same, minus the basis state with zero amplitude.
Expected behavior
I expect DumpMachine to use the same ket symbol always, preferably the one that renders in the output (the one currently used for basis states with relative phase).
System information

OS: Windows 11
.NET Core Version: 6.0.400
QDK version: 0.24.201332, 0.25.222597

Please note that if you copy/paste this from the console to notepad, the symbols will appear correctly. This may be an issue with the way symbols are displayed.

Interesting. It does appear to be an issue with the old Windows console host. If I use the new Windows Terminal, I don't see an issue. (See screen shot below with Terminal app in the background, and old Console in the foreground, both running exactly the same project). I assume there is no issue on other OSes? (i.e. Mac or Linux)
I thought it might just be the non-Unicode code page used by default (e.g. see https://stackoverflow.com/questions/57131654/using-utf-8-encoding-chcp-65001-in-command-prompt-windows-powershell-window), but even changing that didn't help in the old console.
Being that the new Terminal app is the default on Windows going forward (see https://devblogs.microsoft.com/commandline/windows-terminal-as-your-default-command-line-experience/), I'm not sure how much time we should spend on this one.

One last comment. Directing the output to file and examining in a hex editor, I can see the correct UTF-8 chars are in the output stream for all places the right angle bracket is emitted (i.e. e2 9f a9 - see table at the bottom of https://unicode-table.com/en/27E9/ ). As Dmitry already mentioned, pasting the text from the console where it is displaying incorrectly into notepad makes it appear correctly. (The fact the right bracket on |3> is showing correctly proves it is not a font issue). So looks like a bug in the old Windows Console, which will almost certainly not get fixed. Either we move to using a regular ASCII right bracket by default, or just recommend people use the new Windows Terminal for console execution.

(Oh, and it looks fine in the VS Code terminal on Windows too)


Feature Request: Allow Convert-Path to accept literal paths as pipeline inputs

Add a parameter like -InterpretPipeLiterally such that
$Values | Convert-Path -InterpretPipeLiterally

is equivalent to
$Values | ForEach-Object {Convert-Path -LiteralPath $_}

You can do this with scriptblock binding syntax:
$Values | Convert-Path -LiteralPath { $_ }

Oooh, wow, didn't know that! Thanks

Wait. Where is the documentation for this? I can't seem to get it to work.
@vexx32

Searching "scriptblock binding syntax" only leads to scriptblock documentation

Documentation is here: https://docs.microsoft.com/en-us/powershell/module/microsoft.powershell.core/about/about_script_blocks?view=powershell-7#using-delay-bind-script-blocks-with-parameters
Yeah, it's a bit confusing that it's buried in the scriptblock article, but there's not really a much better place to put it, it's a general purpose syntax available to all commands.
If you can share the code you're running into issues with, I can try to point out what's going wrong. The important limitations with the scriptblock binding for parameters are:

It can't be used with [scriptblock]-type parameters
The parameter must be defined as accepting pipeline input either by value or by property name.

Assuming I have a parameter that accepts any Object, if I want to pass a ScriptBlock, I can just write ({…}), right?

The issue I ran into is that this apparently requires specifying an explicit parameter. I was doing Get-PSDrive | Set-Location {"${_}:"} as an experiment. Get-PSDrive | Set-Location -Path {"${_}:"} worked.

Ah yeah, it has to be a specific named parameter. I don't know the full implementation details but yeah, that's a hard requirement.

Probably not for your specific use case, but you can also construct the objects with property names, for example:
[pscustomobject]@{ LiteralPath= "c:"} | Convert-Path
Does require the cmdlet supports ValueFromPipelineByPropertyName, but they typically do when they have distinct inputs with the same underlying type

Yep! Very often (if not always) -LiteralPath parameters also support a PSPath alias both directly and on the property, because providers' objects in PS have a PSPath property that they'll use when piped into a provider cmdlet. 🙂


Create postfix_evaluation.c

A C program to evaluate user input Postfix expression that can be separated by a blank or a comma.

Merged


desktop.Remove(widget) & Widget remains on screen, becomes unresponsive

Just what it says on the tin. The widget in question is a sfg::Window and I'm using the Signal OnCloseButton that comes with it. The signal definitely works (console prints out text when I click the close button), and clearly something happens since the Window becomes unresponsive afterwards, but the Window still remains on-screen.
I'm pretty new to both C++ and SFGUI, so I'd appreciate some clarification. So far as I can tell from the desktop.cpp example, I'm using the functions the same way as shown.
Thanks.

Don't remove the widget when your are inside the signal handler. This will lead to undefined behaviour. Remember what you have to remove and remove it after your call to HandleEvent() returns.

Is there an example available that shows how to do this? (Desktop.cpp seems to 'remove' inside the signal handler, unless I'm misunderstanding the terminology?)

In the case of Desktop.cpp, we first create another shared_ptr to the widget we want to remove, remove it and return from the handler. We also make sure not to try to manipulate the object too much. Once we remove it we must assume it is no longer safe to access. The example was written this way to stay short, and given its simplicity we could also manage the dangers that are associated with this method. We wouldn't recommend doing this in anything more complex than this scenario however.

Sorry to bother you again, but what do you mean by "Remember what you have to remove and remove it after your call to HandleEvent() returns."?
Does this mean storing the widget to be removed inside of a class or global variable and then deleting it inside of the while (mainWindow.pollEvent(event)) code block?

Does this mean storing the widget to be removed inside of a class or global variable and then deleting it inside of the while (mainWindow.pollEvent(event)) code block?

Yes.


Documentation for some queries.

I love that this exists, it should make my life scripting teamspeak tasks much easier. I'm having trouble figuring out how I should form some of the commands. It looks like they might not be implemented, but I want to make sure I'm just not doing it wrong. For example, I want to use "clientdbfind." This has the parameter "pattern" that has to be passed and the option "uid" that I want to also pass. It appears from your documentation that I should use:
@query.clientdbfind :pattern => "tsuid string" do |opt|
     opt.uid
end

It seems some commands like "clientdbfind" throw an error because the script is waiting for something else. I can't find a pattern to why some commands work and others don't.
Net::ReadTimeout: timed out while waiting for more data
	from /System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/net/telnet.rb:557:in `waitfor'
	from /System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/net/telnet.rb:694:in `cmd'
	from /Library/Ruby/Gems/2.0.0/gems/ts3query-0.7.0/lib/ts3query/ts3_connection.rb:38:in `method_missing'

I think the issue is when you run a command that doesn't return a list delimited with the '|' character. Things work fine when TS returns one of its goofy pipe delimited lists, but if it returns data with no pipe character, it hangs. Since 'clientdbfind' only returns "cldbid=173637" there's no pipe. It also explains why you can't use this to send commands to the server.
I'm not positive that this is what is going on but it's the best I can come up with. I'm kind of a newb.


Exponentiated tick labels not produced correctly

Using Matlab 2015a.
The tick labels are not being produced correctly when the strings are of the format '10^3', as produced by default with logarithmically scaled axes. A simple test program reveals this:
loglog(10.^(0:3),10.^(0:3).*rand(1,4))
plot2svg('test.svg')

same here...

Same here. I am desperate. I really wanted to use svg on my thesis for the great quality. But these+inkscape bugs are sabotaging everything.  I think I will have to go back to the old raster image figures. Truly  a pity.

Okay, I raised the issue a while ago, so I'm going to be a good citizen and propose a fix. More from me in due course.

FYI, I have proposed a fix in a pull request: #14
If you need this functionality quickly, you can clone my Fork.

just wanted to let you know about the first release of fig2svg, which includes your fix among many other new features:
https://github.com/kupiqu/fig2svg


Data loading

currently the software only reads CSV files
can we read instead numpy 3d array data stored as "hdf5"?

this will be opened as an issue in the python version, python will be used to do the conversion


Reduce size of docker image

Do not repack the war
From 930MB to 814MB
Add a Dockerfile-dev to build image with locally build maven artifacts

It should land today
On Mon, Feb 11, 2019 at 11:23 AM Carlos Sanchez<EMAIL_ADDRESS>wrote:

@carlossg commented on this pull request.
In Dockerfile
https://github.com/jenkinsci/jenkinsfile-runner/pull/66#discussion_r255445975
:

COPY plugins.txt /usr/share/jenkins/ref/plugins.txt
-RUN /usr/local/bin/install-plugins.sh < /usr/share/jenkins/ref/plugins.txt
+# empty war file to workaround https://github.com/jenkinsci/docker/pull/786
+RUN echo UEsFBgAAAAAAAAAAAAAAAAAAAAAAAA== | base64 -d > /usr/share/jenkins/jenkins.war && /usr/local/bin/install-plugins.sh < /usr/share/jenkins/ref/plugins.txt && rm -f /usr/share/jenkins/jenkins.war

that was just merged, can use in the next release
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/jenkinsci/jenkinsfile-runner/pull/66#discussion_r255445975,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AC3IoFonZ4_-eDxyEUWakjBFrxgds6_Mks5vMUSngaJpZM4az1Vo
.


Add book "chapter" markings and table of contents

It would be very nice to have a chapter marking or similar, and then have a table of contents or similar, showing the current chapter number at the top of the page perhaps.
When reading long texts, I sometimes want to know how many pages until I reach the end of the chapter.

It seems like this might be a good candidate to integrate with the Bookmarks feature.
@jzohrab were you just imagining this as a separate slideout panel or more like a simple status component at the top that just listed the number of pages (and probably title) of the previous and next bookmarks?

Not sure, @cblanken -- it was a vague idea of perhaps a table of contents page.  The current bookmarks listing page would be totally sufficient.
As for showing the current chapter at the top of the page, that's not necessary.  I have an ereader, and it doesn't show the current chapter at the top, and I don't miss it.  To have the current chapter show at the top of the page, or to somehow indicate it in the page slider, we'd need to differentiate "chapter" bookmarks from regular bookmarks, and it doesn't feel worth it to me.
Ah "previous and next bookmarks" is kind of interesting, but also feels like overkill, given that people can have different types of bookmarks, e.g. topics that they want to chat with a tutor about.
Another vague idea I had was auto-creation of chapter markings ...  Maybe that would be another bookmark action -- "auto-generate bookmarks", e.g., and the user could enter some pattern indicating how a chapter is indicated (e.g., "line starts with 'Chapter'"), but that has so many exceptions and edge cases that it will definitely be more headache than it's worth.

Automatic chapter markings definitely sounds like it'd be tough to make user friendly. You'd have to either give the user full control with a regex field or something, otherwise it'd turn into a huge form with a hundred options to select the right text.
It might be possible to setup automatic bookmark imports from epubs though. I think they store bookmarks in the same file. Not sure how it's stored though, so it might end up doing a search for the next x words/lines to find the right position after it converts everything into Lute pages which could be finicky.

Yeah finicky is the word here, and it could easily backfire and make a mess.  Maybe for imported text files where users can pre-edit the files to add special markers, that's ok, but for a general thing it's a drag.  Still a nice thing to backlog though (maybe :-P )

I kind of like the idea of using Markdown (ref #168) to general the table of contents, distinguishing it from bookmarks which are special user things.  This way, users could add "#" or "##" etc to the start of lines, and those would get converted into rich text, and a TOC could be generated from that relatively easily.


Generate LionWeb chunk deserializers for Javascript/Typescript

When a client receives a tree serialized in LionWeb format, it must be deserialized back to classes to facilitate data manipulation.
This requires duplicating the language's data model in the client, creating all the classes and their relationships again.
Consider generating these data model classes and deserializers from an export of the Language from Kolasu, and the possible integrations with Tylasu.

Yes, that should be done but in the StarLasu Tools. I will copy this issue there

https://github.com/Strumenta/starlasu-tools/issues/102 has been created, closing this one


enabled download data as csv button

I added a button to allow download the current paragraph data as csv

Hi,
Thanks for the contribution, I will review your PR.
For PR that modifies Zeppelin UI, it is better to join a screenshot so that people can see the change without building from your code.
On another note, it seems your branch is not up to date with master branch.

The change is to the right corner, a cloud download image. I would suggest to update documentations for how to contribute and how to build. It was quite a code diving.
And thank you for your review.

We are currently in the process of refactoring with the PR#56
So it should get better afterwards :)
Let me play with it a bit, we got a similar PR before, but I would rather use a non-library approach if possible.

Here is my test so far:
Test

Works for table data to CSV
Kinda works for non-csv type of data, but the file is still .csv and while my test didn't make the stack/text separated like a CSV, it could easily happen.
There is tooltip on the button

Cross-browser

Opera - Launch the download but no default name and format for the file
Chrome - works and download as data.csv
Firefox - Not working
Safari - Open the file in the Browser instead of download
IE 10 - Well somehow I can't make Zeppelin run on it anymore :p

Cross-browser for this kind of download is really a pain in the ass, that's why we tried using jquery-datatables and tabletools before.
Discussion

Position of the button? (I actually like it there)
Should the download data works only for table type of data?
Should the CSV be quoted? Should we use a library for the transformation?

@corneadoug Thank you for your review.
Test

Works for table data to CSV
Kinda works for non-csv type of data, but the file is still .csv and while my test didn't make the    stack/text separated like a CSV, it could easily happen.
There is tooltip on the button

I implemented intended for csv download. I believe if the user want to download any plain text output, they can simple copy and paste.

Cross-browser
I think we should establish a baseline -- allow user to be able to download via right click "save as" or directly launch a download prompt. With some many browsers out there, it is hard to make every feature work. Zeppelin project should have "recommended" or "supported browsers". And unfortunately I don't have IE anymore, so I can't test for that.

Position of the button? (I actually like it there)
Feel free to move it around, i set it to show up when paragraph status == "FINISHED"
Should the download data works only for table type of data?
I implemented intended for csv download. I believe if the user want to download any plain text output, they can simple copy and paste. Maybe when data is not in table format, the button should download the msg value and let end user to do the transformation.
Should the CSV be quoted? Should we use a library for the transformation?
I think we should, I only had numeric value at the time of implementation. If the values are string with white space, it could cause problem.

only 1000 rows download, if i can config more rows for daowload

@qinzl1 , I don't think the limitation of how many rows to download is specified by my download button. The download button only converts existing JSON response retrieved from Zeppelin server and converted in to CSV format with UI button. You should look at which specific interpreter you are using. Most example set the max row to display for dataframe or result for 1000.
For example, [file] (https://github.com/apache/zeppelin/blob/5afd7cacdb6ac35d19ebf63bac9f2d20298cfc5c/python/src/main/resources/interpreter-setting.json)
[
  {
    "group": "python",
    "name": "python",
    "className": "org.apache.zeppelin.python.PythonInterpreter",
    "properties": {
      "zeppelin.python": {
        "envName": null,
        "propertyName": "zeppelin.python",
        "defaultValue": "python",
        "description": "Python directory. It is set to python by default.(assume python is in your $PATH)"
      },
      "zeppelin.python.maxResult": {
        "envName": null,
        "propertyName": "zeppelin.python.maxResult",
        "defaultValue": "1000",
        "description": "Max number of dataframe rows to display."
      }
    }
  },
  {
    "group": "python",
    "name": "sql",
    "className": "org.apache.zeppelin.python.PythonInterpreterPandasSql",
    "properties": { }
  }
]```

However, be aware of how many rows you are requesting. The more rows you request, the more memory it uses to build the JSON response and the longer it takes for the browser to do the file format conversion.

I am not entirely sure how optimized is the JSON parsing and encoding library used by Zeppelin or what library it is. Based on what I have seen, it seems the default library and direct toString() conversion.


Pass open connection?

Hey there,
Looking to see if there is a was to use an existing connection. While using in tests in a nestjs app I have a case like this ->
beforeAll(async () => {
    const module = await Test.createTestingModule({
      imports: [
        ConfigModule.forRoot({ isGlobal: true }),
        MyTestModule,
        TypeOrmModule.forRoot({
          type: 'postgres',
          host: process.env.TYPEORM_HOST,
          port: Number(process.env.TYPEORM_PORT),
          username: process.env.TYPEORM_USERNAME,
          password: process.env.TYPEORM_PASSWORD,
          database: process.env.TYPEORM_DATABASE,
          logging: false,
          entities: ['src/**/*.entity{.ts,.js}'],
          synchronize: false,
        }),
      ],
    }).compile()
    app = module.createNestApplication()
    await app.init()

    // Is it possible to use the connection 'default' created by the test application
    await useRefreshDatabase()
  })

Basically looking to avoid AlreadyHasActiveConnectionError: Cannot create a new connection named "default", because connection with such name already exist and it now has an active connection session.
Is there a way to use an existing connection, or do I just need to make a new one for typeorm-seeding? If so where would I specify a name for that.

It would be nice to be able to pass connection options directly to this library to override any values that may be set in the ormconfig.json or in the event this file doesn't exist.
await useSeeding({ ...connectionOptions }) <--- passing ConnectionOptions directly

Hi @IanPirro  & @abettke

with setConnectionOptions you can override the configs.
now useRefreshDatabaseuses already existing connection.

Example
  let connection: Connection
  beforeAll(async (done) => {
    setConnectionOptions({
      type: 'sqlite',
      database: ':memory:',
      entities: ['sample/entities/**/*{.ts,.js}'],
    })
    connection = await useRefreshDatabase()
    await useSeeding()
    done()
  })

Are these changes helpful for you guys?

@hirsch88 Thanks for the quick reply, PR looks good this is what I did on a local fork in createConnection to get this to work, the connection options override is nice as well.

Hi @hirsch88,
Does your solution still require the presence of ormconfig file, right?


fixes https://github.com/wso2/docs-apim/issues/5658

Purpose

fixes https://github.com/wso2/docs-apim/issues/5658
PR sync https://github.com/wso2/docs-apim/pull/5659

@suksw , I added your suggestion. Pls review, approve and merge.


Unable to generate trace and memory file using cairo-vm-cli 1.0.1

Describe the bug
When attempting to run the following commands, I get the respective errors.
Command 1: cairo-vm-cli a.cairo
Error 1:
expected value at line 1 column 1
Error: Runner(Program(Parse(Error("expected value", line: 1, column: 1))))

Command 2: cairo-vm-cli a.sierra
Error 2:
expected ident at line 1 column 2
Error: Runner(Program(Parse(Error("expected ident", line: 1, column: 2))))

Command 3: cairo-vm-cli a.casm
Error 3:
expected value at line 1 column 2
Error: Runner(Program(Parse(Error("expected value", line: 1, column: 2))))

To Reproduce

Create a file called a.cairo.
Place the following into a.cairo.

use core::felt252;

fn main() -> felt252 {
    let n = 2 + 3;
    n
}

In the same directory, create a Dockerfile.
Add the following into the Dockerfile.

FROM debian:stable-slim

COPY *.cairo .
COPY *.sh .

RUN chmod +x prepare-env.sh

ENTRYPOINT [ "bash", "-c", "./prepare-env.sh && tail -f /dev/null" ]

In the same directory, create a file called prepare-env.sh.
Add the following into prepare-env.sh.

#!/bin/bash

# Update and upgrade packages.
apt-get update -y
apt-get upgrade -y

# Install dependencies
apt-get install apt-utils -y
apt-get install curl -y
apt-get install git-all -y

# Clean up
apt autoremove

# Install Starkli
curl https://get.starkli.sh | sh
PATH=$PATH:$HOME/.starkli/bin
starkliup -v v0.3.4
echo starkli --version

# Install rust
curl --proto '=https' --tlsv1.2 -sSf https://sh.rustup.rs | sh -s -- -y
echo "export PATH=\"$HOME/.cargo/bin:$PATH\"" >> ~/.bashrc
source $HOME/.cargo/env
cargo test

# Install Scarb
curl --proto '=https' --tlsv1.2 -sSf https://docs.swmansion.com/scarb/install.sh | sh
echo "export PATH=\"$HOME/.local/bin:$PATH\"" >> ~/.bashrc

# Install Starknet Foundry
curl -L https://raw.githubusercontent.com/foundry-rs/starknet-foundry/master/scripts/install.sh | sh
echo "export PATH=\"$HOME/.local/bin:$PATH\"" >> ~/.bashrc
snfoundryup

source ~/.bashrc

# Ensure the installation path is in /root/.local/bin, since it's installed for root
if ! command -v scarb &> /dev/null
then
    echo "scarb installation failed or PATH is not correctly set."
    echo "Current PATH: $PATH"
    exit 1
fi

# Verify Scarb installation
scarb --version

# Install Cairo-VM
git clone https://github.com/lambdaclass/cairo-vm.git
cd cairo-vm
make build-cairo-1-compiler

# Return to original directory.
cd /

# Compile Cairo to Sierra to CASM
cairo-compile a.cairo a.sierra
sierra-compile a.sierra a.casm

# Execute the main container command.
exec "$@"
source ~/.bashrc

In the same directory, run docker build -t proof-generator:latest .
In the same directory, run docker run proof-generator:latest

Expected behavior
I expect the output of the program to be shown, which is 5 in this case.
What version/commit are you on?
I am using cairo-vm-cli v1.0.1, cairo-lang-compiler v1.1.1, and cairo-lang-sierra-to-casm v1.1.1.
Additional context
I have tried different compiler versions, the different make commands, different Cairo codes and none of them have worked yet. Also, a.casm matches what I expected given the code on the hosted cairo-vm.

Update: I have managed to get a JSON in a similar format to the one shown in cairo_programs/manually_compiled, however it still does not return a trace and memory proof using cairo-vm-cli.

Create a file called starknet_sample.cairo
Copy the following into the newly created file.

#[contract]
mod Ownable {
    use starknet::ContractAddress;
    use starknet::get_caller_address;

    #[event]
    fn OwnershipTransferred(previous_owner: ContractAddress, new_owner: ContractAddress) {}

    struct Storage {
        owner: ContractAddress,
    }

    #[constructor]
    fn constructor(init_owner: ContractAddress) {
        owner::write(init_owner);
    }

    #[view]
    fn get_owner() -> ContractAddress {
        owner::read()
    }

    #[external]
    fn transfer_ownership(new_owner: ContractAddress) {
        only_owner();
        let previous_owner = owner::read();
        owner::write(new_owner);
        OwnershipTransferred(previous_owner, new_owner);
    }

    fn only_owner() {
        let caller = get_caller_address();
        assert(caller == owner::read(), 'Caller is not the owner');
    }
}

Run starknet-compile starknet_sample.cairo starknet_sample.sierra
Run starknet-sierra-compile starknet_sample.sierra starknet_sample.json
Run cairo-vm-cli starknet_sample.json

Running step 5 gives the following error:
root@8a59b77272b2:/# cairo-vm-cli starknet_sample.json
invalid type: sequence, expected a map at line 1067 column 11
Error: Runner(Program(Parse(Error("invalid type: sequence, expected a map", line: 1067, column: 11))))

Note that the software versions have not changed.

ups

Hi @anonymouspanda-23
Were you able to make it work? I was following you instructions you've passed to replicate the error, but starknet-compile fails to compile the .cairo to .sierra. I think the example code in starknet_sample.cairo might not be correct.

The error you are getting is due to the fact that cairo-vm-cli only works with cairo 0. This explains why you are getting parse errors. You should try using cairo1-run instead.


Explore changes to driver used in Rewrite project

Based on conversation with @javfg and @d0choa, we should explore updating the driver used on the disease profile page as it has a number of technical use cases:

Server-side pagination table (Known Drugs)
Third-party API call (LINK)
Loading static file (Ontology)
Simple GraphQL API call (Phenotypes)

Also tagging @forus because part of this work can address the need to ensure customisations are flexible and easy to implement

@fedde-s @forus I've put together a quick guide to help you with the changes I've made so far. This is related to the sections using platform-api. It should help you in case you port any sections to get familiarized with the new structure.

Updating platform-api sections
Structure changes
Sections have 4 base files now:

index.js: Contains the definition and exports the summary and body of the section. Comes from the old index.js file in current sections.

Summary.js: Summary component. Comes from the old Summary.js file.

Body.js: Body of the section. Comes from the old Section.js file.

Description.js: Summary description shown in the body header. Comes from the old Section.js file.

index.js
Must contain a definition object with id, name, shortName (optional) and hasData fields. The last one describes how to know if the section has any data or is empty and its body should not render.
Example:
export const definition = {
  id: 'relatedDiseases',
  name: 'Related Diseases or Phenotypes',
  shortName: 'RD',
  hasData: (data) => data.relatedDiseases.count > 0,
};

export { default as Summary } from './Summary';
export { default as Body } from './Body';

Summary.js
The main differences with the old version are:

There must be a fragment fetching the data required for display. That fragment should be added as a field to the component after. Inside the component, the data from the fragment can be accessed by using the usePlatformApi hook. This will return a { loading, error, data } object just like Apollo's useQuery.

The return value should be a <SummaryItem> component. It should have definition, request, and renderSummary props.

definition comes from index.js.
request is an object including { loading, eror, data } fields.
renderSummary is a function from request's data to a React Component. It renders the content of the summary.

Note: The fragment can be loaded from a .gql file.

Example:
const SUMMARY_FRAGMENT = gql`
  fragment RelatedDiseasesFragment on Disease {
    relatedDiseases {
      count
    }
  }
`;

function Summary({ definition }) {
  const request = usePlatformApi(SUMMARY_FRAGMENT);

  return (
    <SummaryItem
      definition={definition}
      request={request}
      renderSummary={(data) => (
        <>
          {data.relatedDiseases.count} diseases
          <br />
          (through shared targets)
        </>
      )}
    />
  );
}

Summary.fragments = {
  RelatedDiseasesFragment: SUMMARY_FRAGMENT,
};

export default Summary;

Description.js
This file does not have any changes.
Body.js
The only change in the body file with respect to the old Section.js, is the same as number 2 in Summary.js: The return value should be wrapped in a component. This time in a <SectionItem> component.
It should have definition, request, and renderDescription and renderBody props.

definition comes from index.js.
request is an object including { loading, eror, data } fields.
renderDescription is a function from request's data to a React Component. It renders the content of the description (usually Description.js, but it could just be here).
renderBody is a function from request's data to a React Component. It renders the content of the section's body.

Note: If the component uses the same data for the summary and body, we can use usePlatformApi here again, and save a request to the API.


first commit for Lion Token

Please note: This repository is being rebuilt to accept the new volume of token additions and modifications. PR merges will be delayed.

I agree to not ping anybody on Discord/Twitter/email about this pull request. Instead I will inquire by posting a new comment in the pull request if needed.

PRs are reviewed in bulk and and can take up to two weeks to be merged.
This repository is managed using an auto merge action. Please ensure your PR has no deleted lines, and it will be merged.
Please provide the following information for your token.
Please include change to the src/tokens/solana.tokenlist.json file in the PR.
DON'T modify any other token on the list.
At minimum each entry should have

Token Address:
Token Name:
Token Symbol:
Logo: (logo should be uploaded under assets/mainnet//*.<png/svg>)
Link to the official homepage of token:
Coingecko ID if available (https://www.coingecko.com/api/documentations/v3#/coins/get_coins__id_):

Auto merge requirements
Your pull request will be automatically merged if the following conditions are met:

Your pull request only adds new tokens to the list. Any modification to existing
tokens will require manual review to prevent unwanted modifications.

Your pull request does not touch unrelated code. In particular, reformatting changes to unrelated
code will cause the auto merge to reject your PR.

Any asset files added correspond to the token address you are adding. Asset files
must be PNG, JPG or SVG files.

Your change is valid JSON and conforms to the schema. If your change failed validation,
read the error message carefully and update your PR accordingly.

No other tokens shares the same name, symbol or address.

For example, this change would be rejected due to unrelated changes:

The bot runs every 60 minutes and bulk-merges all open pull requests to prevent conflicts.
This means that you need to wait up to 60 minutes for your pull request to be merged or reprocessed.

I'd love to launch the Lion Token to devnet.
On Sun, Feb 6, 2022 at 12:34 PM token-list-automerger[bot] <
@.***> wrote:

Closed #17496 https://github.com/solana-labs/token-list/pull/17496 via
1f2442d
https://github.com/solana-labs/token-list/commit/1f2442d9d618803f5d0964a3bcb848b08ce91498
.
—
Reply to this email directly, view it on GitHub
https://github.com/solana-labs/token-list/pull/17496#event-6015509381,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXF5MUYPVVPDNNOGBKXQ5P3UZ3LONANCNFSM5NVXSYYQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>


Pull to refresh in official example is not work

My flutter sdk version is 1.5.4-hotfix.2. When I view flutter_gallery example,I found that the pull to refresh is not work on my android device, but it works well on ios simulator.Is it a bug?

it doesnt work unless u enable cupertino platform mechanics
duplicate of https://github.com/flutter/flutter/issues/31709

This seems to have been fixed on the master branch with https://github.com/flutter/flutter/pull/30129/, with an explanation as to why this used to not work with Android. I will close this out, but feel free to re-open if there is an alternate issue!


EVA-2469 - Make assembly report download more reliable

Infeasible due to eUtils rate limiting. Will investigate later...


[FLINK-25792][connectors] Only flushing the async sink base if it is …

…possible to do it in a non blocking fashion or if the buffer is full

What is the purpose of the change
Only flushing the async sink base if it is possible to do it in a non blocking fashion or if the buffer is full
Brief change log
Added conditions to flushing logic in AsyncSinkBase
Verifying this change
Unit tests.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)n
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)n
The serializers: (yes / no / don't know)n
The runtime per-record code paths (performance sensitive): (yes / no / don't know)y
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (yes / no / don't know)n
The S3 file system connector: (yes / no / don't know)n

Documentation

Does this pull request introduce a new feature? (yes / no)n
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)n

@flinkbot run azure

@flinkbot run azure

@flinkbot run azure

@flinkbot run azure

@flinkbot run azure

@flinkbot run azure

@flinkbot run azure

@flinkbot run azure

@flinkbot run azure

@flinkbot run azure

@flinkbot run azure

@flinkbot run azure

@flinkbot run azure

@flinkbot run azure

@flinkbot run azure

@flinkbot run azure

@flinkbot run azure

@flinkbot run azure

@flinkbot run azure

@flinkbot run azure

@pnowojski Thank you again for your comments. I've had a thorough think through and I would like to propose the following changes. I have annotated my code with my thoughts and responses to your review.

I am currently working on a final test to assert the flush() blocking and interruption behaviour, will add in an additional commit shortly.

@fapaul Thanks. The purpose of the PR is to reduce the number flushes in case there are already too many in flight requests (blocking). Specifically this is when

there exists space in the buffer to add an element
there are no free in flight requests slots are left, and,
where the number of in flight requests that have not yet completed (and are waiting on completeRequest) is strictly smaller than the number of in flight requests that need to be freed in order to process the current flush in a non-blocking fashion.

It's an interesting suggestion and would save some mailbox time on the write path. Another benefit is if checkpoints are frequent, we would save operator time in write. However, I'm just cautious about tying the sink behaviour to how checkpoints have been set up. My feeling is that if I can accurately determine whether the current write & flush is not blocking, it would be prudent to flush that batch as soon as I know (with the added side benefit that failed request entries have also been completed and added sooner). The other worry is if I leave more flushing responsibility with the checkpointing mechanism, fewer flushes will result in a build up of elements in the buffer and cause the bufferedRequestEntries.size() >= maxBufferedRequests condition to trigger in write() which would shift the completion of in flight request cycles back to the mailbox thread. This is okay, but I feel the buffer would spend most of its life being full, introducing an artificial latency especially if the buffer is large.

Sorry not sure why it didn't push my last comment, maybe because I commented inline. I pasted it again here for continuity. :-)
@pnowojski I think this is completely wonderful. I believe then the code for completeRequest() should be:
    private void completeRequest(List<RequestEntryT> failedRequestEntries, long requestStartTime) {
        // do completeRequest stuff including reducing the inFlightRequestsCount, etc.
        mailboxExecutor.tryYield();
        nonBlockingFlush();
    }

We would get all the previously alluded to benefits.
My only question (and maybe worry) would be if we had a very large number of in flight requests that have all completed (say 100+? since it's customisable by user). Once one completeRequest is triggered, the mailboxExecutor.tryYield() would repeatedly yield to the next completed request in a daisy chain of length equal to the number of completed in flight requests.
I imagine the state of the mailbox thread would have to live alongside the others' states during that time and I was wondering would that risk us using more stack/heap than otherwise and thereby risking an overflow?

@pnowojski Thank you for all your recommendations. I have gladly implemented the agreed upon changes. Please let me know if you have any final comments.

@flinkbot run azure

@flinkbot run azure

@flinkbot run azure

@flinkbot run azure

Thanks @pnowojski and @fapaul for your deep reviews. Seems like all feedback has been addressed. I will merge this at 17:00 UTC today unless I hear any objections.

@flinkbot run azure

@flinkbot run azure


Is the Application ID same as the Client ID?

In the section, "Authorize the application to use the key or secret", it refer to the "Client ID".  However, this term did not appear in this article prior to that point.  Is the "Application ID" in section, "Register an application with Azure Active Directory" same as "Client ID"?

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

ID: 259ba844-605a-90de-611b-86be48852d5a
Version Independent ID: 093cdacd-b58d-bb93-61b2-0bf731f14648
Content: Get started with Azure Key Vault
Content Source: articles/key-vault/key-vault-get-started.md
Service: key-vault
GitHub Login: @barclayn
Microsoft Alias: barclayn

@meiyueilee Yes, you are correct in your understanding here.

@meiyueilee We will now proceed to close this thread. If there are further questions regarding this matter, please open a new issue and we will gladly continue the discussion.


Add support for TypeScript

Someone who knows more about .d.ts files and packages should probably take a look at what I have set up. I’ll ask on Mastodon.

I tested the setup locally via a TypeScript repo and npm add ../regex and the types worked.
dist/regex.min.js should maybe be included in the package exports in some way.
You may want to clear dist/ before "build" and types/ before "types". I’m using package shx for cross-platform shell commands.
Maybe:

Rename "build" to "esbuild"
Define "build" as "npm run esbuild && npm run types"

I don’t know whether or not "typescript" should become a devDependency.
@overload is another option for overloading regex but I’m not sure in which of the multiple JSDoc comments the actual documentation would go.

Benefit of @overload: It may help with specifying this for regex. I don’t know how to best do that. I avoid using this as an implicit parameter in my code.

Command for checking if the types are correct (potential addition to package.json "scripts"):
npx --package=@arethetypeswrong/cli attw .

Closes #4.

Thanks for the PR, @rauschma!! I'm traveling for the next couple days but will follow up on this shortly after.

Can you share the TypeScript repo you're using so I can exactly reproduce your test setup? I'm new to TypeScript, so every bit helps.

You may want to clear dist/ before "build" and types/ before "types". I’m using package shx for cross-platform shell commands.

Good idea, and thanks for the package suggestion! Package rimraf also looks popular. Maybe e.g. "prebuild": "rimraf dist && rimraf types".

I don’t know whether or not "typescript" should become a devDependency.

I think yes.

I avoid using this as an implicit parameter in my code.

I'd also prefer to avoid using this. However, it seems important to support RegExp subclasses if regex were to be used as a starting point for a native regex template proposal. I think accepting an alternative constructor as an argument or as a property of the options object would likely be worse (I don't want to encourage alternative constructors) and less idiomatic for JS.

Can you share the TypeScript repo you're using so I can exactly reproduce your test setup? I'm new to TypeScript, so every bit helps.

https://github.com/rauschma/regex-ts-demo

Package rimraf also looks popular.

Benefit of shx: If you need more commands (I often need chmod) then you don’t need another package. Other than that, both are fine.

regex.bind() works but isn’t very restrictive. Maybe that’s good enough.
It’d be great to have a single file index.d.ts. But I’m not sure if that’s possible: https://fosstodon.org/@rauschma/112798192886997742

dist/regex.min.js should maybe be included in the package exports in some way.

I'd be happy to. I don't know a ton about packages and bundlers, etc., though. I could include "browser": "./dist/regex.min.js" in package.json but I'm not sure what specific situations or tools this would help with, since src/index.js doesn't rely on anything that doesn't work in browsers. Happy to see any PR that adds dist/regex.min.js to package.json in a way that solves any meaningful use case. I'd also be fine with switching from esbuild to another bundler if that helps with anything.

Benefit of shx: If you need more commands (I often need chmod) then you don’t need another package. Other than that, both are fine.

Cool. I used rimraf for now (in https://github.com/slevithan/regex/commit/ebd85aad23e0a387ade15309af9a9a2ffb82faf0) since shx is using it anyway--so this cuts out the middleman. But I'd be happy to switch if helpful in the future.

regex.bind() works but isn’t very restrictive. Maybe that’s good enough.

Yeah. Beyond binding to RegExp subclasses, it allows some creative use that I'm okay with (such as using it for extensions, or using it with alternative constructors like XRegExp, node-re2, rregex, etc.), but I'd have no problem making it more restrictive in future major versions if it seems better to do so, even if that prevents these (rare) possibilities.

Currently there are multiple .d.ts files:

I asked and people replied with how to get a single file index.d.ts.
On the other hand, the current approach is simple and works.

K, I'll go with the current approach for now, but I'd be happy to see this cleaned up into a single file if anyone wants to submit a PR!

If you do the following, then users should be able to import {regex} from 'regex/min':
"./min": {
  "types": "./dist/regex.min.js",
  "import": "./src/index.js"
}

I haven’t done this myself yet, though. Package exports are documented here

K, I'll go with the current approach for now, but I'd be happy to see this cleaned up into a single file if anyone wants to submit a PR!

Check out the replies in the Mastodon thread. I’d document them (Mastodon posts are not guaranteed to stay around for long) for whoever eventually gets around to it – this kind of stuff is not common knowledge.
Downside of all of the suggestions: You need additional tooling and/or an additional build step.
Also: The current approach scales well to more package exports like regex/min – e.g. if you want to expose another module in the future.

One more thing you can do (but that’s completely a matter of taste and not really “better”):

Rename “index.js” to “regex.js” (file name and the 4 occurrences of “index” in package.json).

I've documented the Mastodon replies in #8 (thanks for posting the question!), and I've switched to using @overload in https://github.com/slevithan/regex/commit/6318669dbf0cd8b668cad75bf5932f9a3be6798b.

One more thing you can do (but that’s completely a matter of taste and not really “better”):

Rename “index.js” to “regex.js” (file name and the 4 occurrences of “index” in package.json).

I've now done this as well in 83c131225a860e5cca6ac599ef6360cfdd2680e2.

I've switched to using @overload

I’ve not seen @overload being used multiple times within the same JSDoc comment but it looks like the best way to use it because you don’t have to split up the English prose.
@overload vs. type union (|) operator:

It’s more verbose, but it looks cleaner.
It may enable you to specify a type for this in the future.

If that’s an option, I’d eliminate any. I’m now also always using Array<T> and not T[] in TypeScript because that’s more similar to e.g. Set<T> and because it still looks good when T becomes more complex.
Maybe:
/**
[...]
@overload
@param {TemplateStringsArray} template
@param {...(string | RegExp | PartialPattern)} substitutions
@returns {RegExp}

@overload
@param {string} [flags]
@returns {(template: TemplateStringsArray, ...substitutions: Array<string | RegExp | PartialPattern>) => RegExp}

@overload
@param {RegexTagOptions} options
@returns {(template: TemplateStringsArray, ...substitutions: Array<string | RegExp | PartialPattern>) => RegExp}
*/

I’ve not seen @overload being used multiple times within the same JSDoc comment but it looks like the best way to use it because you don’t have to split up the English prose.

Yeah, that's the only way I've figured out to avoid repeating the prose in source (it gets repeated in the generated .d.ts files).

If that’s an option, I’d eliminate any (which allows too much and should really only be used as a last resort). I’m now also always using Array<T> and not T[] in TypeScript because that’s more similar to e.g. Set<T> and because it still looks good when T becomes more complex.

I switched to any because template substitutions really do allow any (TS's own definitions for String.raw also use any) and I need to account for that in the implementation code. But I've now heeded your advice and switched back to (string | RegExp | Pattern) (and only allowing string for function pattern) in 3ba75ca43d22ee98812e88bda97887d146fb0463.
Question: In 0ca089311f12cad35440d16e4f917f442bfc7d92 I started creating an ESM bundle that can be used in browsers, and I additionally made the bundle the exports target. Do you see any issues with making it the exports target?
Aside: Since I'm now creating an ESM bundle anyway, that seems to open the door to fully moving to TS in the future rather than adding types through JSDocs.

I switched to any because template substitutions really do allow any

TS doesn’t always reflect JS semantics 100% – e.g., this is a type error:
const x = 3 * '5';

If other types of values are accepted but never make sense, I think it’s OK to be more restrictive.
Optional but maybe useful for debugging: Doing runtime type checks. Given that the regex is only created once, performance is not that important(?)

Do you see any issues with making it the exports target?

Looks OK. You can use my demo TS project and see if the code works after compiling with TypeScript.
regex.min.js should also be a package export(?)

Aside: Since I'm now creating an ESM bundle anyway, that seems to open the door to fully moving to TS in the future rather than adding types through JSDoc comments.

I enjoy working with TypeScript – it catches many small errors and has saved me many times. This is my most recent tsconfig.json: https://github.com/rauschma/markcheck/blob/main/tsconfig.json

I have written this chapter that teaches the essentials of TypeScript and should be a fairly quick read: https://exploringjs.com/tackling-ts/ch_typescript-essentials.html

You can do a lightweight TS mode via a jsconfig.json (js, not ts).

Upside: You can type-check during editing but don’t have to build. Svelte does this.
Downside: I’m not sure it’s worth the hassle. Many TS things will take longer to figure out and will be more verbose.

If other types of values are accepted but never make sense, I think it’s OK to be more restrictive.

Integers can be quite convenient to use in regex/pattern substitutions (e.g. regex`.{1,${num}}`, and {…} quantifiers have their own interpolation context that makes this work). But of course not all JS numbers are integers, which is why I left it out of the allowed types for now.

Looks OK. You can use my demo TS project and see if the code works after compiling with TypeScript.

Thanks again for sharing the project. I'd already used it 😊 so I had a base level of confidence, but was wondering if there were other considerations I wasn't taking into account, given the complex ecosystem of bundlers, build tools, direct ESM module usage in different environments, etc. Thanks for looking at it, and glad to hear it looks OK.

regex.min.js should also be a package export(?)

In what situations would this be used? I'm assuming you'd have to be in a browser environment to want a minified version. And if you had the ability to import, wouldn't you want the ESM bundle that doesn't create global variables? I think I'm missing some important insight/context.

I enjoy working with TypeScript [...]

Thanks for walking me though all of this (!) and for the details about jsconfig.json and your tsconfig.json. I'll definitely read your chapter that you linked.

In what situations would this be used?

Good point. I missed that this bundle becomes available globally and won’t be imported – thus package exports indeed make no sense.

Now I’m wondering if regex.min.js should be included in the package or be made available elsewhere.
Similarly: Do you need src if the package already includes regex.mjs?
As an aside: npm’s “Code” tab is useful for seeing what’s actually in the registry: https://www.npmjs.com/package/regex?activeTab=code

I'll definitely read your chapter that you linked.

My own guess is that it shouldn’t take you longer than one hour to read. If you do get around to reading it, I’d be curious whether my estimate is correct or not.


webchat improved things

:white_check_mark: Build TimeHACK AppVeyor-79 completed (commit https://github.com/TimeHACKDevs/TimeHACK/commit/5e3ad7cea1 by @lempamo)


Address kernel startup and connection problems (okr)

Interpreter is known to be conda, and no activation commands were found
Interpreter is known to be conda, and activated environment variables were not generated
Interpreter is known to be conda, and generating env variables failed
[x] microsoft/vscode-jupyter#6739
[x] microsoft/vscode-jupyter#6425
[ ] microsoft/vscode-jupyter#6414
[x] microsoft/vscode-jupyter#5061
[x] microsoft/vscode-jupyter#1129
[x] microsoft/vscode-jupyter#4593
[ ] microsoft/vscode-jupyter#450
[x] microsoft/vscode-jupyter#1125
[x] microsoft/vscode-jupyter#1131
[x] microsoft/vscode-jupyter#1124
[x] microsoft/vscode-jupyter#1128
[x] microsoft/vscode-jupyter#1253
[x] microsoft/vscode-jupyter#1257
[x] microsoft/vscode-jupyter#1269
[x] microsoft/vscode-jupyter#1698
[x] microsoft/vscode-jupyter#1644
[x] microsoft/vscode-jupyter#1591
[x] microsoft/vscode-jupyter#421
[x] microsoft/vscode-jupyter#1440
[x] microsoft/vscode-jupyter#617
[x] microsoft/vscode-jupyter#1184
[x] microsoft/vscode-jupyter#4244
[x] microsoft/vscode-jupyter#5211
[x] microsoft/vscode-jupyter#5293
[x] microsoft/vscode-jupyter#5394
[x] microsoft/vscode-jupyter#5459
[x] microsoft/vscode-jupyter#5494
[x] microsoft/vscode-jupyter#5736
[ ] microsoft/vscode-jupyter#5897
[ ] microsoft/vscode-jupyter#5998
[x] microsoft/vscode-jupyter#6087
[ ] microsoft/vscode-jupyter-internal#40
[ ] microsoft/vscode-jupyter-internal#77
[ ] microsoft/vscode-jupyter-internal#79
[ ] microsoft/vscode-jupyter-internal#89
[ ] microsoft/vscode-jupyter-internal#61
[ ] microsoft/vscode-jupyter#5314
[x] microsoft/vscode-jupyter#5650

Closing this issue, as this is old and outdated now.


Monitoring improvements

This issue focuses on improvements addressing the main issues users are seeing with the current implementation and functionality provided by Rancher monitoring.
Primary Issues:

Can’t add metadata to alerts for integration into other monitoring systems.
Users can not ship metrics to other systems.
Users can not scrape custom endpoints at the cluster level.

To address:
Need to add the following capabilities to Rancher monitoring:

Ability to specify custom configuration to the prometheus rules file.

This needs to support alert rules/groups
This needs to support recording rules/groups
Users need to be able to create new rules for their own use
Users need to be able to edit Rancher provided rules, so they can add labels / metadata etc.

Custom Grafana dashboards, leveraging both configmaps and dashboard list.

Ability to specify custom scrape configs to the prometheus config yaml.

Allow specifying remote read/write configurations to prometheus so that data can be written to remote syncs for other monitoring tools or long term storage.

Deploy AlertManager with HA configuration.

Allow enabling Thanos side car for users looking to aggregate. rancher/system-charts#95

A user should be able to monitor etcd in the local HA cluster, or imported RKE cluster. (#18619)

Constraints:

Leverage the prometheus operator for deployments.
Allow users to leverage operator CRDs.
Rancher installation of the Prometheus operator should not use the same api groups as the upstream operator.
A user must be able to leverage monitoring in a CIS compliant cluster. This may require the user to deploy additional exporters.

Yes please.  These above objectives would be great.

I have a PR open to enable thanos support (would be great to get that merged)
Support for additionalScrapeConfigs is already in place technically, but the problem is that it gets formatted weird due to monitoring only being able to be configured by form, need to please add configure via YAML to the UI.

Ultimately what I'm doing in the mean time is customizing the system-charts and hosting it myself to meet my own needs, but this in practice is tricky each time that Rancher does an update. I have conflicts in the version number since 0.0.x is being used, I have to redo my patches with all the changes that Rancher makes, etc ...

@ekristen - any quick workaround to force cluster level monitor to honor custom metrics from project level monitoring?

if you are looking to just enable auto scraping of pods that announce metrics via prometheus.io/scrape then simply go to the system project and secrets and locate the Prometheus additional scrape config entry and edit it. Inside find the Prometheus-is-scrape entry and under the Kubernetes_sd_config remove the namespace restrictions that limit it to the ingress and kube-system and save, the operator will redeploy and it will start discovering in all namespaces.

@ekristen  - thanks! I'll try to change this into rancher2 terraform provided answer!

For dashboards, I think using the same workflow as the stable grafana helmchart would be preferable.
Basically a grafana watches namespaces and detects configmaps (secrets) labeled with a specific label. The configmap contains the dashboard/dashboards, and this get mounted into the grafana pods.
This enables developers to easliy bundle their dashboards with the rest of the yaml for their application.
The same goes for Datasources. We run loki in our clusters and using config maps / secrets is way more convenient than patching the current configuration.
https://github.com/rancher/rancher/issues/19125

@LinAnt as an admin is there an easy way to figure out where dashboards/configs are coming from? I see that the searched namespaces can be restricted so there can be some controls put around the access.

Will we be passing custom labels to Prometheus?
If I test by adding a label to a pod (test: "test") and then try running a query that should return the pod in Prometheus (kube_pod_container_status_restarts_total{test="test"}) , it does not.
If I run the a wider query that returns the pod, I do not see the label in it's definition.
kube_pod_container_status_restarts_total
kube_pod_container_status_restarts_total{container="agent",endpoint="http",host_ip="<IP_ADDRESS>",instance="<IP_ADDRESS>:8080",job="expose-kubernetes-metrics",namespace="cattle-system",node="wk1",pod="cattle-node-agent-9pl5c",service="expose-kubernetes-metrics"}

@ekristen

Support for additionalScrapeConfigs is already in place technically, but the problem is that it gets formatted weird due to monitoring only being able to be configured by form, need to please add configure via YAML to the UI.

What do you mean by "technically"? How to set this field to make it work?

@marcinbojko @adampl I've just had to fork the system charts to make it work the way I want for now unfortunately. :(
https://github.com/ekristen/system-charts/tree/release-v2.3
This has a modified version of 0.0.5, specifically designed to enable thanos and change the default scrape config for the prometheus.io/scrape label to target all namespaces.
Set this variable prometheus.scrapeConfigPodNamespaces to nothing and it'll scrape all namespaces for pods that expose metrics.

@ekristen Well, scraping custom pod metrics is already supported, just add an annotation to the pod (or pod template) like this: field.cattle.io/workloadMetrics: '[{"path":"/metrics","port":12345,"schema":"HTTP"}]'. I'm looking for a way to scrape metrics from outside of the Kubernetes cluster.

@adampl that's interesting, I wasn't aware of that method.  I would still prefer standard working out of the box. As for getting additionalScrapeConfigs working, if you write the YAML then copy and paste it into the value field, I've had success with that.

@ekristen Yes, and as you said, the UI breaks the format. I created an issue for this: #24274
Regarding the annotation, you can set it through the UI too, though it didn't quite work for me:
https://rancher.com/docs/rancher/v2.x/en/project-admin/tools/monitoring/#example
If I recall correctly, it messed the service assigned to the workload, so I prefer the YAML method.

@ekristen  Were you able to overcome 0.0.5/0.0.6 templates problems? Looks like my manually created ingress to prometheus cluster monitoring stopped to works (502, bad gateway). Did i miss something or in 0.0.6 we still cannot add ingress to our custer-prometheus?

I'd love the ability to easily do remote writes to my VictoriaMetrics server for long term storage and central dashboarding.  I hate running multiple Prometheuses, but love the direct integration into Rancher - as do our Devs.  Glad it's a planned feature improvement.

There are two issues addressing remote monitoring:
[Enhancement] Prometheus Monitoring Remote Write #20624
Support external prometheus in rancher monitoring #17370

config

not work...

if you are looking to just enable auto scraping of pods that announce metrics via prometheus.io/scrape then simply go to the system project and secrets and locate the Prometheus additional scrape config entry and edit it. Inside find the Prometheus-is-scrape entry and under the Kubernetes_sd_config remove the namespace restrictions that limit it to the ingress and kube-system and save, the operator will redeploy and it will start discovering in all namespaces.

Same as I do now, but I don't like this workaroud while the prometheus "lpv persistent solution" is not working for me, currently the config file located at /etc/prometheus/config_out  is a volatile empty dir in k8s.

Closing this ticket in favor of the new 2.5 scope is available here.


"types" missing in package.json "exports"

When trying to import the xstate to my project I get an TS error
Could not find a declaration file for module 'zustand-middleware-xstate'. '...' implicitly has an 'any' type.   There are types at '...', but this result could not be resolved when respecting package.json "exports". The 'zustand-middleware-xstate' library may need to update its package.json or typings.ts(7016)
Tried cleaning and re-installing node_modules, but it's probably because of the missing "types" in "exports" in package.json according to this SO:
https://stackoverflow.com/questions/76211877/the-xxxx-library-may-need-to-update-its-package-json-or-typings-ts

You're right! Apparently that was a breaking change in TypeScript 4.7, just released a new version that fixes it


Adding Bag Of Bond

What does this implement/fix? Explain your changes.
This PR introduce a method to compute Bag of Bonds of a list of molecules.
The need two parts:
extract the BoB from the list of molecules
compute the BoB vector for a given molecule in this list
Any other comments?

I'm not comfortable accepting this PR with the current API because I don't think it fits the usual RDKit descriptor interface. I do think that this functionality would be useful to have, but it needs to be in a different form.
The current RDKit descriptor calculators all take a molecule and return either a value or a vector of values. The Bag of Bonds calculation, on the other hand, requires a molecule along with the dict storing the number of bins for each "bond type".
A sketch of a concrete suggestion for how I could imagine integrating this:
We define a new class: a BagOfBondsCalculator for want of a better name at the moment.
This class has two important methods:

BagOfBondsCalculator::determineBags() which takes a set of molecules and figures out the bond types and counts used to define the bags (and their sizes)
BagOfBondsCalculator::calculate() which takes a single molecule and returns the vector of values.
The class can be serialized (pickled) so that I can store one and retrieve it for later usage.

I'm happy to talk through this in order to come to an agreement on what would make sense to do (and who should do it).

it makes sense. we will do after the new release!

Closing due to lack of activity. Feel free to reopen if anyone has the time and inclination to revive this one.


added compact Docker Image for build

:warning: This is a Pull Request Template :warning:

Checkout all the things you've done by placing an x within the square braces below.

Proposed Changes

[X] I've read the CODE OF CONDUCT and abide to it.
[X] I've read the CONTRIBUTING.md
[X] I've forked the repository.
[X] I've created a branch and made my changes in it.
[X] I understand opening a PULL REQUEST doesn't mean it will be merged for sure.

It includes latest CMake and Conan.

I've added a compact Docker image that can be build as
docker build -t devenv:llvm6 -f Dockerfile.compact .
Featuring latest CMake and Conan, removing for the moment the need of all packages installed and then removed in the main Dockerfile. When a package is added in one layer, the file removed in another layer do not cleanup space*.
The goal of this compact image is to be used as reference environment for a CI/CD purpose (e.g. Travis CI can run docker images)

Thanks!


Source units available

All the good and positive
please tell me what is the problem
hanging for a week and can not sync
He writes - Source units available
How is it treated?
Thanks to all

It can take several weeks on weak hardware to sync the whole blockchain
from scratch.
For 0.14 we have reworked how to display the time remaining until the
initial block download is complete. For 0.13 the only indicator is the
progress bar in the status bar.

Many thanks

"Source units unavailable" sounds like "No block source available...", which used to be the message while not connected to a peer yet. If it's completely stuck that sounds like a network problem?

laanwj , Yes, it seems the problem with the network
those Internet priozheniya all work perfectly and that "suffocate" I suspected our local secret services because they do not want this to Projects developed in Belarus
And how can you check on what ports goes synchronization maybe they are banned or worse provider at a local pit mix everything on the firewall
Once again, thank you so much for assisting Guys


Needs a sqrt before the norm?

https://github.com/HansRen1024/Face-Tracking-Using-CNN-and-Optical-Flow/blob/1bb71d9b133f560916334ec3859f917abcbd38d0/include/TLD.cpp#L61
Hi, to calculate the ratio between new length and old length, I think it needs a sqrt before the norm? As the norm calculates the sum of square value only. This issue exists also in the python version, thank you!

@hyx07 Thank you for pointing this out. A sqrt would make the ratio curve be smoother and more stable, which is helpful to stabilise the bounding box. You are more than welcome to make a pull request.

pull request submitted for both versions, thank you for your work!


Idiomatic way to implement a desktop app heartbeat

Previously asked at https://github.com/open-telemetry/community/discussions/1598
Is there a documented pattern or generally accepted method of implementing a heartbeat in a desktop application? That is, a periodic message send to a collector for the purpose of determining an active session count? Or perhaps a general pattern of reporting back Boolean states?
I was going down the path of using a Counter with a 0|1 value and just letting that contact a Collector periodically, but wanted to double-check.
My notes thus far:

Metrics reader can be configured non-cumulative for fire once
https://github.com/open-telemetry/opentelemetry-dotnet/issues/2524

On-going work related to establishing conventions for service uptime
https://github.com/open-telemetry/oteps/pull/185

Prior art:

Microsoft App Insights Live Metrics https://azure.microsoft.com/en-us/blog/live-metrics-stream/

We're looking at kicking off a working group around self-observability of our SDKs/APIs.  I think a general health check / heartbeat recommendations should go along with that.
WDYT @carlosalberto @yurishkuro ?   cc @reyang

Not seeing a strong motivation for SDK heartbeat. If SDK produces a metric indicating how much data it exports, that is equivalent to a heartbeat.
On the other hand, this issue seems to be about the application's heartbeat, not the SDKs. Either way, the motivation is not clear to me. I am assuming this is all independent of a health check endpoint an application may provide.

My question was whether, as part of SDK self-o11y health, we provide general health metric support.

This was closed by mistake by the stale bot. Re-opening

There is some place where the discussion about this feature is happening right now?


Scope a specific Azure Repo in oAuth 2.0 token

Hi,
I see various scopes in this page, however, is it possible to specify r/rw access to a repo when granting authorisation?
Essentially specifying a resource in the scope.
Looking to do vso.code_full limited to a repo.
Thanks,

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

ID: ac47cbcf-45ed-81aa-b532-e7ad0046658e
Version Independent ID: 7aa1f60e-069f-3d62-5cac-b65e4b1116ef
Content: Authorization using OAuth 2.0 - Azure DevOps
Content Source: docs/integrate/get-started/authentication/oauth.md
Product: devops
Technology: devops-ecosystem
GitHub Login: @chcomley
Microsoft Alias: chcomley

It's a bit of both really. Anyhow, will try out Stack Overflow and feedback here if there's a breakthrough.


iOS crashing on release version

Experiencing this silent crash with this error on XCode:
Unhandled JS Exception: TypeError: undefined is not an object (evaluating 'this._propsAnimated.setNativeView')
This error is located at:
in n
in t
in n
in RCTView
in RCTView
in t
in RCTView
in n
in r
in Unknown
in t
in a
in RCTView
in t
in Styled(t)
in Unknown
in g
in RCTView
in l
in RCTView
in RootSiblingsWrapper
in RCTView
in t',
Environment
Environment:
OS: macOS High Sierra 10.13.3
Node: 8.4.0
Yarn: 0.27.5
npm: 5.5.1
Watchman: 4.7.0
Xcode: Xcode 9.3 Build version 9Q107o
Android Studio: 3.0 AI-171.4443003
Packages: (wanted => installed)
react: ^16.3.0-alpha.1 => 16.3.0-alpha.1
react-native: ^0.54.2 => 0.54.2

I managed to solve this issue. When I upgrading React Native, libRCTAnimation.a went missing. Replaced it and it now works :)


Cannot specify specify audience parameter in token request

I'm trying to use Duende.AccessTokenManagement to retrieve an Auth0 Management API token, and I need to specify the audience parameter.
If I were using IdentityModel directly, I could use ClientCredentialsTokenRequest.Parameters to set "audience", but there is no way I can see to specify "audience" when configuring a ClientCredentialsClient. I can specify "resource", but Auth0 doesn't accept that.
Is there any problem adding a Parameters property onto ClientCredentialsClient to allow this use case?

Thanks. We will look into it.


Error: You should not call create on a model

Error: You should not call create on a model. Instead, call store.createRecord with the attributes you would like to set.

I'm seeing the above error after converting some of our model tests.

Awesome. We need to fix the fixtures and update the output.

We need to fix the fixtures and update the output.

to what exactly?

For moduleForModel we need to emit:
let subject = this.owner.lookup(‘service:store’).createRecord(options);


CLOUDP-28022: Private Endpoint translation layer

All Submissions:

[x] Have you signed our CLA?
[ ] Put closes #XXXX in your comment to auto-close the issue that your PR fixes (if there is one).
[ ] Update docs/release-notes/release-notes-template.md if your changes should be included in the release notes for the next release.

@josvazg

Conversion coverage is a bit low

It will increase to a complete level as tests from the controller join the code.


Import in Eclipse

Hello, I've been trying to import this project in eclipse and I can't figure out how. I had imported over a year ago but that wasn't the gradle build. How can I import it now?

Hi @libathos it should be easy to import this library in Eclipse, just adding the library folder, or using the Eclipse Gradle Plugin to allow you import the library from mavencentral or jcenter.
And btw, may I ask, why don't you use Android Studio? It has amazing cool features that boost your development, most of all Android developers use it, and most important, the Android Development Toolkit (ADT) that you need to develop Android in Eclipse is not maintained anymore by Google (http://android-developers.blogspot.de/2015/06/an-update-on-eclipse-android-developer.html).
So yeah, you should give Android Studio a try ;-)
Cheers mate.


[Bug] Default function parameters are not type checked

from beartype import beartype

@beartype
def f(x: str = 1) -> None:
    print(f"called with type {type(x)}")

f()
f(1)

Produces the following:
$ python3 main.py
called with type <class 'int'>
Traceback (most recent call last):
  File "/home/loot/code/py/beartype/main.py", line 26, in <module>
    f(1)
  File "<@beartype(__main__.f) at 0x7fa35e986050>", line 22, in f
  File "file/.venv/lib/python3.10/site-packages/beartype/_decor/_error/errormain.py", line 301, in raise_pep_call_exception
    raise exception_cls(  # type: ignore[misc]
beartype.roar.BeartypeCallHintParamViolation: @beartyped f() parameter x=1 violates type hint <class 'str'>, as 1 not instance of str.

I would expect that both calls would fail, but alas, the first one is allowed through.

Oh, Gods. You're right. It's true. @beartype is failing us all here.
Happily, I've just realized that we can (and should) be type-checking default parameters at decoration time. Rather than wastefully recheck default parameters at call time (i.e., on every repeated call to a decorated callable), @beartype can instead check the default parameters of a decorated callable exactly once when that callable is initially decorated. This has a number of profound benefits, including:

It's ludicrously fast. After all, isn't that why we're all here? Yes. Yes, it is.
It's ludicrously early (i.e., at app startup rather than during app execution). So, @beartype will raise type-checking errors for default parameters immediately. That pushes @beartype even closer to the behaviour of a static type-checker, which seems mostly beneficial.

Thanks for the tremendous heads up, @dosisod. This is a critical oversight on our part. We'll bang this out in time for our upcoming beartype 0.11.0 release.

...lolbro. I literally cannot believe it took a year-and-some-change for @beartype to type-check defaults. Actually, what am I saying? Of course I can believe it. It's just that... I don't want to believe it. :disappointed:
In other words, @beartype now efficiently type-checks parameter defaults at decoration time. This:
from beartype import beartype

@beartype
def f(x: str = 1) -> None:
    print(f"called with type {type(x)}")

f()

...now raises the expected violation:
beartype.roar.BeartypeDoorHintViolation: Function __main__.f() parameter "x"
default value 1 violates type hint <class 'str'>, as int 1 not instance of str.

For the sake of my fragile ego, let's pretend I did this within days rather than years. Rejoice, everybody! :smiling_face_with_tear:

I've noticed (in some other responses) you tentatively laying out a roadmap / feature map for 0.19.0, 0.20.0, ...
While taken in jest (who knows what kind of crazy breaking high priority issue is going to suddenly appear?), perhaps organizing open issues & hanging features into github projects might help?
Since there's only one developer (rip), there doesn't need to be much planning (perfect communication + bottleneck), and the number of issues is fairly small, this is probably excessive.
But we stopped having a great idea of what's broken, not implemented (since so much is implemented), what is the great stream of TODOs and false release date promises (add 1.5 years), etc etc etc since ye olde megathread (several GB) was (thankfully) closed.

there's only one developer (rip)

Do you ever feel like the emaciated spectre of Death is looming over your shoulder while grinning cynically at your cats?

what's broken

Nuffin'. This is the answer, @JWCS. Nothing is broken, because you can't break something that never worked in the first place. Galaxy-brain 4D checkmate. <points_knowingly_at_forehead/>

false release date promises (add 1.5 years)

In all seriousness, the @beartype timetable is basically now just:

2024: deeply type-check everything in O(1) time.
2025: deeply type-check everything in O(n) time under BeartypeConf(strategy=BeartypeStrategy.On) configurations.

Of course, we could GitHub Project that. Of course, I'd blindly ignore that GitHub Project and then chaotically do whatever I wanted without rhyme or reason. Despite planning, nothing changes.
@beartype is more of a "random walk across the heterogenous typing landscape" than a coherent project at this point. Actually, wasn't @beartype always pure unadulterated chaos? I'm pretty sure it was. It's a marvel that this creaky zeppelin never ignited under the sheer weight of its unplanned hubris. How is this thing still flying!?
Also, I'm pretty sure we now have the working title for my first paper:

"Beartype: A Random Walk Across the Heterogenous typing Landscape (5 QA Tips Guido Doesn't Want You to Know)."

Resolved by 0fccdb4cb14038a. This is how the beartype 0.18.0 cycle ends.  Praise be to Fat Cat. Do not fear Fat Cat.  Much like that "adorable" cat, this didn't quite work out as expected. Let's just quietly reopen this feature request and pretend that @leycec's initial attempt actually worked rather than blowing up the entire @beartype ecosystem. See also related issue #354.
I weep for that cat. I weep! :sob:


Improved camera, count-tracking and 3D model loader for yoga

This PR fixes the orientation and scale multiplier for all yoga 3D models in both 360 deg and AR viewer.
Also, solves that error because of which some yoga pages were coming up blank.

Last week's [12/06 - 17/06] progress added in this PR is as follows,

Improved camera and the counting algorithm along with feedback for yoga to make it more realistic in the user's environment
Refactored the code structure of the yoga model loader for better readability and scalability.


Trained model

Thank you for your great work.
Can you provide the trained model?
Looking forward to your reply.

We train a single GPU (12GB) on CIFAR100 and miniImageNet from scratch without using any pre-training model.


Introduce Rocky Linux 9.4 support for main branch and add enhanced support for 1.5.X releases.

This PR consists of two commits which

Enhance run.sh to be able to create a Cloudberry DB main branch 000-cbdb-sandbox container on Rocky Linux 9.
Add enhance support for 1.5.X releases. As long as the Dockerfile.RELEASE.centos7supports 1.5.x releases, a developer only needs to replace previous release zip file with new one.
Adjust Santbox CBDB cluster to consist of three primaries and segments so failure scenarios can be tested.

Please review comments in the commits for additional details.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Great work! Thanks @edespino.
Just left some comments, FYI.


Inverse order of attachers when passed in array

Looking at index.js, it seems that
mdast.use([plugin1, plugin2, plugin3])

is equivalent to
mdast.use(plugin3).use(plugin2).use(plugin1)

which is counter-intuitive if you ask me.
Why is it so?

No idea why that’s so. Weird... Will work on that soon...


Hare various test scenarios

We should add three tests for the hare:

A test in which the consensus does not end in the first iteration but in second/third iteration
A test that runs multiple consensus instances (different layers) simultaneously
A test that runs multiple consensus instances in which a previous instance keeps running during a later instance.

Should be unit-test like (non-automation).
A. The first test can be done by manipulating the p2p simulator inbox (just wrapping the inbox by another inbox to make it throw the status/proposal messages of the first iteration).
B. The second test can simply start multiple instances with some offset
C. The third can combine both A & B

Done with first


Extend supported Python versions to 3.09, 3.10, and 3.11

Codecov Report
Patch and project coverage have no change.

Comparison is base (61ea078) 78.67% compared to head (8520c1c) 78.67%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main       #4   +/-   ##
=======================================
  Coverage   78.67%   78.67%
=======================================
  Files          17       17
  Lines         755      755
=======================================
  Hits          594      594
  Misses        161      161

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


merge from template-arlac77-github

.github/workflows/update_readme_api.yml

chore: add<EMAIL_ADDRESS>(jobs.update_readme_api.steps.uses)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
112

Relevant Lines:
119

💛  - Coveralls


Bump to urllib3 = "<3.0"

Resolves #36

Thanks - did you do this with poetry?  I believe we need to update the lock file as well

thanks!


Extended View Dashboard

Not gonna happen, but we'll improve the UI now that we've moved from popup to sidepanel.


Fix TLS custom (couch) dist for Erlang 20

Summary

Fix quoting so that it works with all OTP versions 20 through 24 [1].

Dist API in 20 [2] did not have a listen/2 [3] callback. Implement
listen/1 so we're compatible with all the supported OTP versions.

[1] https://github.com/apache/couchdb/issues/3821#issuecomment-985089867
[2] https://github.com/erlang/otp/blob/maint-20/lib/kernel/src/inet_tcp_dist.erl#L71-L72
[3] https://github.com/erlang/otp/blob/master/lib/kernel/src/inet_tcp_dist.erl#L79-L80
Testing

Follow the nice instructions by @jiahuili430 here. Try with Erlang 20 and then 23, for example.

My setup with 20 looked like:
vm.args:
-proto_dist couch
-couch_dist no_tls<EMAIL_ADDRESS>-couch_dist no_tls<EMAIL_ADDRESS>-ssl_dist_optfile "/...../asf-3/dev/couch_ssl_dist.conf"

remsh to node 2
./dev/remsh-tls 2
(node2@<IP_ADDRESS>)1> net_kernel:nodes_info().
<EMAIL_ADDRESS>         [{owner,<0.399.0>},
          {state,up},
          {address,{net_address,{{127,0,0,1},53431},"xxxx",proxy,inet}},
          {type,normal},
          {in,123},
          {out,137}]},
<EMAIL_ADDRESS>         [{owner,<0.369.0>},
          {state,up},
          {address,
              {net_address,{{127,0,0,1},53429},"<IP_ADDRESS>",tcp,inet}},
          {type,normal},
          {in,177},
          {out,145}]},
<EMAIL_ADDRESS>         [{owner,<0.1095.0>},
          {state,up},
          {address,{net_address,{{127,0,0,1},53431},"xxxx",proxy,inet}},
          {type,hidden},
          {in,8},
          {out,10}]}]}

There are two minor things to watch out for when testing:

In Erlang 20 TLS dist works a bit differently -- there is a proxy driver module, so TLS connections in net_kernel:nodes_info/0 show up as proxy instead of tls. That's expected.

Because connections between nodes are bi-directional, when we set node1 to be TCP, we're not guaranteed to always get a TCP connection to it, as, for example, node1 could first connect to node2  and node2 doesn't have a no_tls flag so it will be a TLS connection. But after a few tries we should get a TCP connection.

Just FYI @jiahuili430's instructions are actually here


[Bug] Meshery isn't starting when resource already exists in k8s cluster and namespace is deleted

Current Behavior

Due to some issues, I had to remove Meshery by deleting namespace in my k8s cluster, plus the .meshery folder too. But when I reinstalled, the manifests got downloaded but couldn't start up because some resources (Clusterroles, Clusterrolebinding) already exists in the cluster.
Logs:
admin-pc@LAPTOP-RROSUK8P:/mnt/d/XCodes/meshery/mesheryctl$ ./mesheryctl system start
Starting Meshery...
⢿ Deploying Meshery on Kubernetes Error: failed to start Meshery
See https://docs.meshery.io/reference/mesheryctl/system for usage details
: cannot start Meshery: rendered manifests contain a resource that already exists. Unable to continue with install: ClusterRole "meshery-metrics-reader" in namespace "" exists and cannot be imported into the current release: invalid ownership metadata; label validation error: missing key "app.kubernetes.io/managed-by": must be set to "Helm"; annotation validation error: missing key "meta.helm.sh/release-name": must be set to "meshery"; annotation validation error: missing key "meta.helm.sh/release-namespace": must be set to "meshery"
Usage:
  mesheryctl system start [flags]
.......

Do note that I deleted the namespace, turns out that some resources are left uninstalled. I had to remove them manually
Expected Behavior

An uninstall option should be present (just a suggestion), so that if users face problem they can reinstall Meshery without issues. Plus, this note must be considered as we're moving to Helm charts for install from version v0.6.0
Screenshots/Logs

Environment

Host OS: Linux (Ubuntu 20.04) via WSL2
Platform: Kubernetes
Meshery Server Version: stable-v0.5.71
Meshery Client Version: stable-v0.5.71

Contributor Guides and Resources

🛠 Meshery Build & Release Strategy
📚 Instructions for contributing to documentation

Meshery documentation site and source

🎨 Wireframes and designs for Meshery UI in Figma
🙋🏾🙋🏼 Questions: Layer5 Discussion Forum and Layer5 Community Slack

@panyuenlau @Aisuko Another issue with deploying Meshery.

I wondering, did we already change the deployment function to meshery-operator while use mesheryctl command?

@DelusionalOptimist

Hi guys, Could we confirm this issue already be fixed?

IG this issue is sorted via the solution provided in the Discourse forum. So are we good to close this issue then?

@piyushsingariya, do you know?

This is solved with the recent addition in stop command via --force flag. Closing this one!


Initializer fix and Compatibility with Ubuntu Xenial 16.04 LTS and Kinetic Kame

This PR brings in a range of updates:

It fixes the initializers for non-install spaces and thereby properly resolves #57
It removes some non-required dependencies that were spewing warnings
It makes Exotica compatible with 16.04 and Kinetic Kame.

Fully tested with both catkin_make_isolated and catkin build on Xenial + Kinetic. Travis confirmed that all changes work with Trusty + Indigo.

Can you move the:
install(DIRECTORY include/ DESTINATION ${CATKIN_PACKAGE_INCLUDE_DESTINATION})
install(DIRECTORY ${CATKIN_DEVEL_PREFIX}/include/ DESTINATION ${CATKIN_PACKAGE_INCLUDE_DESTINATION})

into the AddInitializer.cmake ?
I'm supportive of removing boost all around in favour of std. Especially since ROS is going the same direction.


event grid overview page

@lisawong19,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-mapping/reference.yml
:white_check_mark:Succeeded

docs-ref-services/event-grid.md
:white_check_mark:Succeeded

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


Need to add a step for the gem installation.

The code as it stands fails because the gems are not installed (unless you have a global rails installation). Also, cd hello-world\bin isn't valid, it should be cd hello-world/bin with a forward slash (that one gets me too).
Finally: I suggest using bundler for handling the gems and, to avoid collisions, install locally.

#sign-off

@robconery Thanks for the PR! Great suggestions, I have moved ahead with merging the content. Thank you for the amazing contribution!

You're welcome :). I'm working with this now and I've found a few more niggling little bugs - I can do a separate PR or you can keep this open.


Remove glob-all dependency and added gzip plugin

Added esbuild gzip plugin, removed glob-all dependency and ability to pass npm scripts arguments to solid jsx.


linux: enable FORTIFY_SOURCE

Motivation for this change
CONFIG_FORTIFY_SOURCE adds compile-time and run-time checks for buffer overflows.
This option is enabled by every major distro kernel  (Arch, Debian, Fedora/RHEL).
https://outflux.net/blog/archives/2017/09/05/security-things-in-linux-v4-13/
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
[x] 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.

nix-build -E '(with import ./. {}; linkFarm "linux-configs" (lib.mapAttrsToList (n: v: { name = n; path = v.configfile; }) (lib.filterAttrs (n: v: (v.configfile or null) != null && !v.meta.unsupported) linuxKernel.kernels)))'
/nix/store/qdczas9ic14720s6hydbmq0jy25ygpi3-linux-configs

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/437

Any opinions on whether or not to backport this? (I.e. is there a reason against it that I'm perhaps missing? Because I'm inclined to do so :) )

Maybe some testing period on unstable would be nice?


[SE-0313] Implicit hop to an actor for an "isolated" parameter.

This implements the remaining semantics for SE-0313, by "hopping" to the isolated actor parameter when semantically required:

In an async function, in the function prolog
In an asynchronous call to a synchronous function with an isolated parameter, at the call site

Fixes rdar://80907464.

@swift-ci please test

@swift-ci please nominate


Xcode 6.2 Broken

Xvim isn't available when upgrading to Xcode 6.2.  Building a new version doesn't fix the issue.  If I switch to Xcode 6.1.1 or 6.3 beta 2 then get it back.

+1 on this. After updating Xcode to 6.2 XVim is broken. I have tried rebuilding as well.

See the change made in the referenced commit above, make that change in the XVim project, then manually build

Yeap, just saw that commit. I tried it out and it works now. Thanks.


Recovery Services Vault zone

In https://learn.microsoft.com/en-us/azure/site-recovery/azure-to-azure-how-to-enable-replication-private-endpoints#enable-the-managed-identity-for-the-vault and Azure Portal, zone used is without region.

Learn Build status updates of commit a5bc7d9:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/private-link/private-endpoint-dns.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

@asudbring
Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off

@juanandmsft @asudbring - just noting that you made this change in the Government table, but not in the Azure services DNS zone configuration table. If that also needs to be changed, please submit another PR. I will merge this one.


Regex::bytes Emoji?

I'm trying to create a crate that checks if a string is a single emoji or not (so people don't have to hunt down the way to do that in Rust) but I'm having a hard time following the unicode docs.
What I have looks like this:
use lazy_static::lazy_static;
use regex::bytes::Regex;

pub fn is_emoji(candidate: &str) -> bool {
    lazy_static! {
        static ref REGGIE: Regex = Regex::new(r"^[\p{Emoji}]$").unwrap();
    }
    REGGIE.is_match(candidate.as_bytes())
}

#[cfg(test)]
mod tests {
    #[test]
    fn it_knows_when_you_emoji() {
        assert!(crate::is_emoji("✌🏻"));
    }

    #[test]
    fn it_knows_when_you_are_not() {
        assert!(crate::is_emoji("h"), false);
    }
}

unfortunately, both of these tests fail 😢
Would love some pointers on this! Thanks 😄

This passes:
use lazy_static::lazy_static;
use regex::Regex;

pub fn is_emoji(candidate: &str) -> bool {
    lazy_static! {
        static ref REGGIE: Regex = Regex::new(r"^\p{Emoji}+$").unwrap();
    }
    REGGIE.is_match(candidate)
}

#[cfg(test)]
mod tests {
    #[test]
    fn it_knows_when_you_emoji() {
        assert!(crate::is_emoji("✌🏻"));
    }

    #[test]
    fn it_knows_when_you_are_not() {
        assert!(!crate::is_emoji("h"));
    }
}

Your second assert was just ill-stated. I assume you meant to use assert_eq!, but it's just as well to negate the condition.
Your first assert is a bit more subtle. Many emoji are graphemes, which are single visual characters that are made up of one or more Unicode codepoints. Your example has multiple codepoints. But the regex engine only matches one codepoint at a time. So when you ask for ^\p{Emoji}$, you are saying, "only match strings that contain exactly one codepoint that is in the Emoji class." But the string you provided contains multiple emoji codepoints. So I just changed the regex to ^\p{Emoji}+$.
Also, it would be super helpful if you could explain why you were using regex::bytes::Regex instead of regex::Regex so that I can fix the docs that led you to that. In your code, regex::bytes::Regex was totally unnecessary, so I removed it.

I'm trying to create a crate that checks if a string is a single emoji or not

Some advice: if the extent of your crate is this single small function, then I would urge you to avoid publishing such micro-crates. I don't want to get into it here, but there's a thoughtful post on it here: https://lucumr.pocoo.org/2019/7/29/dependency-scaling/

@BurntSushi - thanks for the very quick response.
re: assert v assert_eq... 🤦♂
re: where did i find docs on regex::bytes, I'm... not actually sure? I trekked through my browser history and couldn't find the culprit, it was probably a GitHub issue or something, can't find anything on legit docs pages.
re: micro-crate - the only reason it looks like this is because i separated it out for this contrived example. I'm trying to make a terminal utility crate that prints out a box with information (think boxen and I want to find the "visual width" of a given string, and calling .len() on an &str with an emoji on it varies wildly. This is reason that I left out the + at the end - since I only want to test the "visual width" of a given string, while this given emoji may have two emoji codepoints, it's rendered as one. I guess the way to test that out is to test if there are two emoji codepoints right next to each other and if the second codepoint is a modifier of the first one... Yeesh.

re: where did i find docs on regex::bytes, I'm... not actually sure? I trekked through my browser history and couldn't find the culprit, it was probably a GitHub issue or something, can't find anything on legit docs pages.

Well, the docs for regex::bytes are here: https://docs.rs/regex/1.3.4/regex/bytes/index.html
What I meant was, what led you to using regex::bytes::Regex instead of regex::Regex? That's the thing I'd like to correct. If you can't remember, no worries.

and I want to find the "visual width" of a given string

I wonder if unicode-width gets this right? If not, it might be worthy of a bug report, although I'm not sure if Unicode itself has a method for this that covers emojis. Discovering the actual visual width of text is generally quite tricky, and as far as I'm aware, is not possible in the general case without an actual font renderer.

Well, the docs for regex::bytes are here: https://docs.rs/regex/1.3.4/regex/bytes/index.html

I think I was led there because I was trying to create a regex with \u and was getting an error, so I switched to regex::bytes when I saw " (To relax this restriction, use the bytes sub-module.)" in this section of the docs. Actually reading the documentation would have prevented me from using it, I just generally don't know what I'm doing - wouldn't read too much into it 😄
also i'm gonna go ahead and say that crate is good enough for what I want and that you're right that this problem is maybe a bit too difficult for a general purpose solution.
This has been fun! Thanks so much for your help 😄 see you around

All righty, great, thanks! Always happy to hear a beginner's perspective on how crate docs can be improved. Thanks.

fwiw i have filed an issue with unicode_width here