Assignment 02 -- 08 Feb 2024

Read following stories only, and give proof for each by reporting their title, e.g. [a1b2c3d4e5b6] --> Title.

e5adf2f8d37d (Point: 5)
0f6e02c1f1d9 (Point: 5)
52eb20692f7c (Point: 5)
38ef852aa596 (Point: 5)
d58f1415962d (Point: 5)
ca11427ac966 (Point: 5)
e6286e4f3009 (Point: 5)
c625a5dd25df (Point: 5)
7de2d0c8894d  (Point: 5)

Follow the instruction given in the two last stories and give proofs for each with some screenshots.

c625a5dd25df (Point: 15)
7de2d0c8894d (Point: 15)

Explain why it is better to use Python virtual environment instead of direct installation to the system (Point: 10)
Which stories suggested direct installation to the system and which ones suggested using virtual environment. (Point: 10)
Submit assignment before 2024-02-2300. (Point: 5)

Medium Stories and Their Titles

[e5adf2f8d37d] --> How to Install Notepad++
[0f6e02c1f1d9] --> Installing Python
[52eb20692f7c] --> Installing Some Python Libraries
[38ef852aa596] --> Installing Pandas
[d58f1415962d] --> Install Scikit-Learn
[ca11427ac966] --> Few Lines of Code in Using Matplotlib
[e6286e4f3009] --> Create Python Virtual Environment
[c625a5dd25df] --> Install Pandas, Matplotlib, Jupyter Notebook, Scikit-Learn, Seaborn in Python Virtual Environment
[7de2d0c8894d] --> Jupyter Notebook Examples For Python Packages Installed in Virtual Environment

Screenshots for c625a5dd25df

https://itbdsti-my.sharepoint.com/:f:/g/personal/10219022_mahasiswa_itb_ac_id/ErBWU22SyMdOhRYX0bt6kaUB8JcpEKT8Cn6FED0MfR-XBg?e=LioD6B

Screenshots for 7de2d0c8894d

https://itbdsti-my.sharepoint.com/:f:/g/personal/10219022_mahasiswa_itb_ac_id/Erz40AGlDzFFlb2zAqFZ1JkBVl4eeJzv73LLwD3XM4aHJg?e=4gBE3W

Reason Python Virtual Environment is Better Than Direct Installation

Using python virtual environment is better because it makes libraries management for each project simpler. This is due to the fact that python virtual environment allows user to do projects in separate virtual environment, ensuring each project uses its own libraries and even its own version of python when necessary without affecting other projects.
52eb20692f7c and 38ef852aa596 suggested direct installation to the system, while c625a5dd25df suggested using virtual environment

Titles:

[e5adf2f8d37d]: How to install Notepad++
[0f6e02c1f1d9]: Installing Python
[52eb20692f7c]: Installing some Python libraries
[38ef852aa596]: Installing pandas
[d58f1415962d]: Install scikit-learn
[ca11427ac966]: Few lines of code in using Matplotlib
[e6286e4f3009]: Create Python virtual environment
[c625a5dd25df]: Install Pandas, Matplotlib, Jupyter Notebook, Scikit-Learn, Seaborn in Python virtual environment
[7de2d0c8894d]: Jupyter Notebook examples for Python packages installed in virtual environment

Installation:

Pandas

Notebook

Scikit-Learn

Seaborn

Jupyter Notebook:

Question:

It is better to use Python virtual environment instead of direct installation to the system because it has cleaner dependency package management. Python virtual environment creates requirements.txt to save the current installed package, this file will be used to reproduce a virtual environment in another machine easily.
Stories that suggested direct installation to the system: 52eb20692f7c, 38ef852aa596, d58f1415962d
Suggested using virtual environment: ca11427ac966, e6286e4f3009, c625a5dd25df, 7de2d0c8894d

Titles:

[e5adf2f8d37d]: How to install Notepad++
[0f6e02c1f1d9]: Installing Python
[52eb20692f7c]: Installing some Python libraries
[38ef852aa596]: Installing pandas
[d58f1415962d]: Install scikit-learn
[ca11427ac966]: Few lines of code in using Matplotlib
[e6286e4f3009]: Create Python virtual environment
[c625a5dd25df]: Install Pandas, Matplotlib, Jupyter Notebook, Scikit-Learn, Seaborn in Python virtual environment
[7de2d0c8894d]: Jupyter Notebook examples for Python packages installed in virtual environment

Screenshots:

For [c625a5dd25df]

For [7de2d0c8894d]

It is better to use Python virtual environments because it has a self-contained directory tree. This allows the developers to create separate instances of Python with their own libraries and dependencies. The isolation will avoid conflicts between different projects that might use different versions of the same package at the same time. It has easier dependency management and reduced risk of package conflicts by keeping their dependencies separated.
Stories suggested installation to the system: [52eb20692f7c], [38ef852aa596], [d58f1415962d]
*[e5adf2f8d37d] and [0f6e02c1f1d9] describes how to install application to computer, not about installing library to system
Stories suggested using virtual environment: [ca11427ac966], [e6286e4f3009], [c625a5dd25df], [7de2d0c8894d]

Titles:

[e5adf2f8d37d]: How to install Notepad++
[0f6e02c1f1d9]: Installing Python
[52eb20692f7c]: Installing some Python libraries
[38ef852aa596]: Installing pandas
[d58f1415962d]: Install scikit-learn
[ca11427ac966]: Few lines of code in using Matplotlib
[e6286e4f3009]: Create Python virtual environment
[c625a5dd25df]: Install Pandas, Matplotlib, Jupyter Notebook, Scikit-Learn, Seaborn in Python virtual environment
[7de2d0c8894d]: Jupyter Notebook examples for Python packages installed in virtual environment

Screenshots:

[c625a5dd25df]

[7de2d0c8894d]

Questions:

It is better to use Python virtual environment instead of direct installation to the system because when you install packages within a virtual environment, they won’t affect other environments.
Stories that suggested direct installation to the system: [52eb20692f7c], [38ef852aa596], [d58f1415962d]
Stories that suggested virtual environment: [ca11427ac966], [e6286e4f3009], [c625a5dd25df], [7de2d0c8894d]

Sorry sir, I'm Syazwana 10220002. I tried to open the Jupyter notebook from the command prompt according to the instructions, but it is not accessible and displays errors such as ModulenotFound , error. I've checked the pip list too, but it looks like jupyter, like the server, core already exists. If it's like this, what should I do, sir? Is there any part missing? Thank you sir in advance.

Titles:
[e5adf2f8d37d]: How to install Notepad++
[0f6e02c1f1d9]: Installing Python
[52eb20692f7c]: Installing some Python libraries
[38ef852aa596]: Installing pandas
[d58f1415962d]: Install scikit-learn
[ca11427ac966]: Few lines of code in using Matplotlib
[e6286e4f3009]: Create Python virtual environment
[c625a5dd25df]: Install Pandas, Matplotlib, Jupyter Notebook, Scikit-Learn, Seaborn in Python virtual environment
[7de2d0c8894d]: Jupyter Notebook examples for Python packages installed in virtual environment

Screenshots:

For [c625a5dd25df]

Virtual environtment

install matplotlib

install pandas

install notebook

Install scikit-learn

install seaborn

install openpyxl

install pyarrow

list installed

save requirements

For [7de2d0c8894d]

Open Jupyter Notebook

First example

Read Spreadsheet with Pandas

Display chart seaborn

Learn binary classification with scikit-learn

Question:

Using a Python virtual environment is better than installing directly into the system. Since each same package can have different versions to suit different environments and can be easily moved to another system it can help prevent problems that may occur from a corrupted installation
stories suggest direct installation to the system :
[0f6e02c1f1d9]: Installing Python
[52eb20692f7c]: Installing Some Python Libraries
[38ef852aa596]: Installing Pandas
[d58f1415962d]: Install Scikit-Learn
stories suggest the use of a virtual environment :
[ca11427ac966]: Few lines of code in using Matplotlib.
[e6286e4f3009]: Create Python Virtual Environment
[c625a5dd25df]: Install Pandas, Matplotlib, Jupyter Notebook, Scikit-Learn, Seaborn in Python Virtual Environment
[7de2d0c8894d]: Jupyter Notebook Examples for Python Packages Installed in Virtual Environment

Title of the following stories:
• [e5adf2f8d37d] --> How to install Notepad++
• [0f6e02c1f1d9] --> Installing Python: Start to document the learning
• [52eb20692f7c] --> Installing some Python libraries: Restart the journey by documenting the installation
• [38ef852aa596] --> Installing pandas: Reading CSV and XLSX files in the simplest way
• [d58f1415962d] --> Install scikit-learn: Hopefully a happy first step to machine learning
• [ca11427ac966] --> Few lines of code in using Matplotlib: A journey in data visualization with Python
• [e6286e4f3009] --> Create Python virtual environment: Learn to start a Python project in better way on Windows 11
• [c625a5dd25df] --> Install Pandas, Matplotlib, Jupyter Notebook, Scikit-Learn, Seaborn in Python virtual environment: Learn to install some Python packages on Windows 11
• [7de2d0c8894d] --> Jupyter Notebook examples for Python packages installed in virtual environment: Test the previous installed packages
Screenshoots:
• [c625a5dd25df] : https://drive.google.com/drive/folders/165_QPPaRsRzPrfRQ2bKVnibkOGrIFAet?usp=drive_link
• [7de2d0c8894d] : https://drive.google.com/drive/folders/1M4NuzUL0srtx6yzgZzhxNG0enN_RBEyw?usp=drive_link
Python virtual environment is better to use instead of direct installation to the system, because:
•  Virtual environment can manage the packages and their dependencies (keeping their dependencies separated) well when working on complex or multiple projects.
•  Allows users to avoid installing Python packages globally (it will install packages just in your desire project folder).
•  Allow users to isolate Python projects from one another.
Stories suggested direct installation to the system: and which ones suggested using virtual environment. (Point: 10)
• [52eb20692f7c] --> Installing some Python libraries: Restart the journey by documenting the installation
• [38ef852aa596] --> Installing pandas: Reading CSV and XLSX files in the simplest way
• [d58f1415962d] --> Install scikit-learn: Hopefully a happy first step to machine learning
(About installing application: [e5adf2f8d37d] --> How to install Notepad++ & [0f6e02c1f1d9] --> Installing Python: Start to document the learning)
Stories suggested using virtual environment:
• [ca11427ac966] --> Few lines of code in using Matplotlib: A journey in data visualization with Python
• [e6286e4f3009] --> Create Python virtual environment: Learn to start a Python project in better way on Windows 11
• [c625a5dd25df] --> Install Pandas, Matplotlib, Jupyter Notebook, Scikit-Learn, Seaborn in Python virtual environment: Learn to install some Python packages on Windows 11
• [7de2d0c8894d] --> Jupyter Notebook examples for Python packages installed in virtual environment: Test the previous installed packages

**Tittle of Medium Stories  : **

[e5adf2f8d37d]: How to install Notepad++
[0f6e02c1f1d9]: Installing Python
[52eb20692f7c] : Installing some Python libraries
[38ef852aa596] : Installing pandas
[d58f1415962d] : Install scikit-learn
[ca11427ac966]: Few lines of code in using Matplotlib
[e6286e4f3009]: Create Python virtual environment
[c625a5dd25df] : Install Pandas, Matplotlib, Jupyter Notebook, Scikit-Learn, Seaborn in Python virtual environment
[7de2d0c8894d]: Jupyter Notebook examples for Python packages installes in virtual environment

The screenshoots of c625a5dd25df

Create and activate virtual environment

Install python packages

The screenshoots of 7de2d0c8894d

Launch Jupyter

First example

**Question **

So using virtual environment is better than installing packages directly in to the system due to a project that we made becomes independent of the system python. It is isolates the interpreter, libraries, and scipts from those installed. this isolation prevents conflict between different projects and ensures that changes in one environment do not effect others.
The stories that suggest direct installation : [0f6e02c1f1d9], [52eb20692f7c], [38ef852aa596], [d58f1415962d]
The stories that suggest using virtual environment : [ca11427ac966], [e6286e4f3009], [c625a5dd25df], [7de2d0c8894d]

Titles

[e5adf2f8d37d] --> How to install Notepad++
[0f6e02c1f1d9] --> Installing Python
[52eb20692f7c] --> Installing some Python Libraries
[38ef852aa596] --> Installing pandas
[d58f1415962d] --> Install scikit-learn
[ca11427ac966] --> Few lines of code in using Matplotlib
[e6286e4f3009] --> Create Python virtual environment
[c625a5dd25df] --> Install Pandas, Matplotlib, Jupyter Notebook, Scikit-Learn, Seaborn in Python virtual environment
[7de2d0c8894d] --> Jupyter Notebook examples for Python packages installed in virtual environment

Install Packages

Testing Installed Packages

Questions

Python virtual environments are isolated environments that allow developers to create separate instances of Python, complete with their own libraries and dependencies. This environment helps isolate each projects so that they do not affect each other in cases of them using similar packages with different version (reduce risk of package conflicts).
Direct installation: [52eb20692f7c], [38ef852aa596], [d58f1415962d] -- Installing python and other packages directly into the system that you are currently using
Virtual environment: [ca11427ac966], [e6286e4f3009], [c625a5dd25df], [7de2d0c8894d] -- Installing and running scripts inside a virtual environment (venv), can be activated and deactivated

Titles :

[e5adf2f8d37d] : How to install Notepad++
[0f6e02c1f1d9] : Installing Python
[52eb20692f7c] : Installing some Python libraries
[38ef852aa596] : Installing Pandas
[d58f1415962d] : Install sci-kit learn
[ca11427ac966] : Few lines of code in using Matplotlib
[e6286e4f3009] : Create Python virtual environment
[c625a5dd25df] : Install Pandas, Matplotlib, Jupyter Notebook, Scikit-Learn, Seaborn in Python virtual environment
[7de2d0c8894d] : Jupyter Notebook examples for Python packages installed in virtual environment

Install Pandas, Matplotlib, Jupyter Notebook, Scikit-Learn, Seaborn in Python virtual environment

I found an error in running the code. I've already followed the steps, but there are still errors. I've already re-install the Python and tried to find another solution to install the 'pip', but it doesn't work. I'm sorry, Sir, i'm still to find the another solution until it's work to install the ' pip'.
Jupyter Notebooks example for Python packages installed in virtual environment

First Example

Read spreadsheets with Pandas

Display charts using Seaborn

Learn binary classificiation with Scikit-Learn

Question :

Python virtual environment is better than direct installation because virtual environment makes the project will be separate and isolated for different projects. Virtual environment helps us easier to manage and maintaining the projects.
Direct installation to the system ; [0f6e02c1f1d9] : Installing Python, [52eb20692f7c] : Installing some Python libraries, [38ef852aa596] : Installing Pandas, [d58f1415962d] : Install sci-kit learn
Virtual environment ;  [ca11427ac966] : Few lines of code in using Matplotlib, [e6286e4f3009] : Create Python virtual environment, [c625a5dd25df] : Install Pandas, Matplotlib, Jupyter Notebook, Scikit-Learn, Seaborn in Python virtual environment, [7de2d0c8894d] : Jupyter Notebook examples for Python packages installed in virtual environment

Good work, @alfans022, @audreyljhns-001, @bernikes, @darmoniaga, @syazwanra, @almiracas, @AnnisaSW, @shemrnd, @alyssahanifadh.
See you in class in about 30 minutes.

Good work, @alfans022, @audreyljhns-001, @bernikes, @darmoniaga, @syazwanra, @almiracas, @AnnisaSW, @shemrnd, @alyssahanifadh.
See you in class in about 30 minutes.


REST server: leverage kubernetes service account

Coverage increased (+0.008%) to 51.867% when pulling 744cc56a03327aeda95416255fe448702c5cb811 on qixcheng/rest-server/k8s-service-account into 618d766d9ee1fe96ae9fb57e18994c91bdfab34c on pai/aks.


Update TemplateTest.py (replace PIA with AB)

looks like a PIA was missed in the tests folder.

Looks good, preciate it bro! merging this...


Lexer fails to ignore multi-line string within a commented out block

Given the following commented-out code:
--[[
  call_function([[
local x = 1
local y = 2
... dozens more lines
]])
--]]

LuaCheck reports:
spec.lua:237:3: (E011) expected statement near ')'

I think this is correct, actually. You should add one or more = to your comment, like so:
--[=[
  call_function([[
local x = 1
local y = 2
... dozens more lines
]])
--]=]

As @CelticMinstrel notes, if you are trying to nest block comments in Lua you should be using different = counts in the comment syntax to differentiate which starts go with which stops.

Also per #198 this is no longer the canonical repository for this project (see https://github.com/lunarmodules/luacheck) and nobody has any ability to close issues here. Please lets try to keep things tidy so future users can better notice the relevant migration issue. I ask that you please close this issue. If something is still amiss and you have a bug or question, feel free to open a new issue on the current repository.


Create LEARN.md

PR Description 📜
created learn.md file from scratch
Fixes #30

Mark the task you have completed ✅

[X] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[X] I have performed a self-review of my own code or work.
[X] I have commented my code, particularly in hard-to-understand areas.
[X] My changes generates no new warnings.
[X] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE

Thank you soo much for contributing to our repository 💗

@Durgesh4993 have a look on this pr . Thanks

@roy7077 @Durgesh4993  have a look on this pr . Thanks

@roy7077 @Durgesh4993  have a look on this pr . Thanks


current master c6e380c does not compile with VS2013

When trying to build the project with VS2013 I get the following error:
Error	3	Friend access was granted to 'LiveCharts.Wpf,PublicKey=0024000004800000940000000602000000240000525341310004000001000100fd75e78956f97c7369ca28eb4214633471da9d13357dc192414bec59b911c99718d9d16a803e527a40a42f947bb297e9d2236d9e541937c4acfc0bc1ca8ee57114963445259b14532d56951de47e773d790110f63b04721bb853c23d9db25c45b83b45dea9aa4157aba95ad890bf69a7d2583e711c786a32817ab1937f43e7d5', but the output assembly is named 'LiveCharts.Wpf, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null'. Try adding a reference to 'LiveCharts.Wpf,PublicKey=0024000004800000940000000602000000240000525341310004000001000100fd75e78956f97c7369ca28eb4214633471da9d13357dc192414bec59b911c99718d9d16a803e527a40a42f947bb297e9d2236d9e541937c4acfc0bc1ca8ee57114963445259b14532d56951de47e773d790110f63b04721bb853c23d9db25c45b83b45dea9aa4157aba95ad890bf69a7d2583e711c786a32817ab1937f43e7d5' or changing the output assembly name to match.	c:_Dev\github\Live-Charts\Core40\bin\Debug\LiveCharts.dll	WpfView
Tried to using this commit : https://github.com/beto-rodriguez/Live-Charts/commit/c6e380cf1b52359b007074cad861d2045a3de7aa

You are trying to build .net core project with VS2013, that doesn't support .net core out of box.

Hi Vladimir,

ok, but it looks that VS2013 can be used, but it is a bit difficult to integrate .net core support. I would suggest it should be mentioned that VS2015 is recommended for building or is there an easier solution for supporting VS2013.

Best regards,
Florian
Gesendet: Mittwoch, 25. Januar 2017 um 13:41 Uhr
Von: "Vladimir Rusanov"<EMAIL_ADDRESS>An: beto-rodriguez/Live-Charts<EMAIL_ADDRESS>Cc: fstorck<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Betreff: Re: [beto-rodriguez/Live-Charts] current master c6e380c does not compile with VS2013 (#446)
You are trying to build .net core project with VS2013, that doesn't support .net core out of box.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@fstorck  Hi,
I've wrote the comment because I've noticed in path Core40 and associated it with .NET Core. On the second thought I don't remember .NET Core to be used in this repo, so I've deleted the comment. To be honest I don't have VS2013 installed so I cannot repro your problem. I think it's better to let someone with VS2013 to check your problem.
Regards,

@fstorck This is normally happens when I mess up and while working to publish the nuget packages I commit a reference to this assemblies.
The assemblies that I published in Nuget are signed by me:
 >LiveCharts.Wpf,PublicKey=---my local sign--

The sign is not included in the repository, in only exist in my local machine, that is why you got this error.
I am sorry for this, it does not happens often, I verified and the current repo references the correct assemblies.
I will close this for now, if you face any other issue just let me know.

Hi Alberto,
thank´s for the explanation.


using {{}} in config

If I understood everything correctly, then ..
https://github.com/NLog/NLog/wiki/AppDomain-Layout-Renderer
This does not work
${appdomain:format={1} - {0}}

This works
${appdomain:format={1\\}}

This does not work
="...${appdomain:format={1\\} - {0\\}}"

In sum, these functions do not give the desired result.
ParseParameterName - Removes single slashes, with double does not work correctly
ParseParameterValue - It does not work correctly without a slash

For now, close. I'm testing.

public static string ParseParameterName(SimpleStringReader sr)
        {
            int ch = -1;
            int nestLevel = 0;
            var prevCh = -1;

            var nameBuf = new StringBuilder();
            for (ch = sr.Peek(); ch != -1; prevCh = ch, ch = sr.Peek())
            {
                if ((ch == '=' || ch == '}' || ch == ':') && nestLevel == 0)
                {
                    break;
                }

                if (ch == '$')
                {
                    sr.Read();
                    nameBuf.Append('$');
                    if (sr.Peek() == '{')
                    {
                        nameBuf.Append('{');
                        nestLevel++;
                        sr.Read();
                    }

                    continue;
                }

                if (ch == '}')
                {
                    if (prevCh != '\\')
                    {
                        nestLevel--;
                    }
                }

                if (ch == '\\')
                {
                    // skip the backslash
                    sr.Read();

                    // append next character
                    nameBuf.Append((char)sr.Read());
                    continue;
                }

                nameBuf.Append((char)ch);
                sr.Read();
            }

            return nameBuf.ToString();
        }

/// <summary>
        /// https://github.com/NLog/NLog/issues/3193
        /// </summary>
        [Test]
        public void Nlog_issue_3193_ParseParameterName()
        {
            var input = @"${var:default-path}/${cached:${appdomain:format={1\\} {0\\}}}.${cached:${date:format=yyyyMMdd}}.info.log";
            var reader = new NLog.Internal.SimpleStringReader(input);
            reader.Position = 29;
            var res1 =    LayoutParser.ParseParameterName(reader);
            Assert.AreEqual(@"${appdomain:format={1\} {0\}}", res1);
        }

  <variable name="var_appdomain_format" value="{1\\} - {0\\}"/>

 <target name="_file" type="File"
               ConcurrentWrites="false"
               fileName="${var:default-path}/${cached:${appdomain:format=${var_appdomain_format}}}.${cached:${date:format=yyyyMMdd}}.info.log"
               archiveFileName="${var:default-path}/Archives/{#####}.part.log"/>

Hi! Thanks for posting the code, however it's a bit unclear to me. Is that the fixed code or showing us the bug?

Hi, @304NotModified
You can run the test (Nlog_issue_3193_ParseParameterName()) now with the old code to see the current bug.
I will try to explain what it is.
Each call to ParseParameterName removes the screened slash.
To get a slash to ParseParameterValue, you need to specify 2 slashes in my case.
But ParseParameterName does not correctly handle '}' if there is more than one slash before it. This demonstrates the slash.
The code above solves this problem.
If we talk in general, there may be problems in the logic of the parser itself and it needs to be changed. This is interesting for me to hear from you.

This does not work in the current implementation, but we will work in my implementation.
fileName="${var:default-path}/${cached:${appdomain:format={1\\} - {0\\}}}.${cached:${date:format=yyyyMMdd}}.info.log"

fileName="${var:default-path}/${cached:${cached:${appdomain:format={1\\\\} - {0\\\\}}}}.${cached:${date:format=yyyyMMdd}}.info.log"

Each nested level is => +2 backslashes.

Hi, would it be possible to send a PR?

ОК,
I want to send PR with fix and this test

        [TestCase(@"                                    ${literal:text={0\} {1\}}")]
        [TestCase(@"                           ${cached:${literal:text={0\\} {1\\}}}")]
        [TestCase(@"                  ${cached:${cached:${literal:text={0\\\\} {1\\\\}}}}")]
        [TestCase(@"         ${cached:${cached:${cached:${literal:text={0\\\\\\\\} {1\\\\\\\\}}}}}")]
        [TestCase(@"${cached:${cached:${cached:${cached:${literal:text={0\\\\\\\\\\\\\\\\} {1\\\\\\\\\\\\\\\\}}}}}}")]
        public void Issue_3193_Nested_Сlosing_Braces(string input)
        {
            var reader = new NLog.Internal.SimpleStringReader(input.Trim());
            var factory = NLog.Config.ConfigurationItemFactory.BuildDefaultFactory();
            IReadOnlyCollection<LayoutRenderer> renderers = LayoutParser.CompileLayout(factory, reader, isNested: true, text: out var text);

            while (true)
            {
                Assert.AreEqual(1, renderers.Count);
                var singleRender = renderers.Single();
                if (singleRender is WrapperLayoutRendererBase wrapperRender)
                {
                    renderers = ((SimpleLayout)wrapperRender.Inner).Renderers;
                }
                else
                {
                    break;
                }
            }
            var textRenderer = (LiteralLayoutRenderer)renderers.Single();
            Assert.AreEqual("{0} {1}", textRenderer.Text);
        }

Any objections?
(I am wrong, the number of slashes is doubled, not added)

Too many slashes. I will change the parser code.

Hi,
Sorry for the delay. I have to check that in-depth and it has been a while that the parser has been changed

I'm preparing a PR. If I'm right, then everything is simple.

Thanks!

--> https://www.nuget.org/packages/NLog/4.6.2 🎉


C++: inline for CheckConstraint functions to allow mutliply header in…

C++: inline for CheckConstraint functions to allow mutliply header includes
partially relates to #1026

Thank you!


create own module for landscaper apis

How to categorize this PR?

/area dev-productivity
/kind technical-debt
/priority normal
What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
cc @danielfoehrKn
Release note:

A separate module has been created for the landscaper api objects. Get the api by running `go get github.com/gardener/landscaper/apis`

Thanks I'll test if that reduces the dependencies when requiring it.

Thanks I'll test if that reduces the dependencies when requiring it.

/hold until https://github.com/gardener/landscaper/pull/93 is merged

/hold until https://github.com/gardener/landscaper/pull/93 is merged

/unhold

/unhold


Add Gewerbliche und Hauswirtschaftlich-Sozialpflegerische Schulen Emmendingen

Name of School
Gewerbliche und Hauswirtschaftlich-Sozialpflegerische Schulen Emmendingen
Domain
ghsede.onmicrosoft.com
Users
The users will be students
Type of school
GHSE is a High School
ghsede.txt

@FF03de lib/domains/de/GHSE: this request has an incorrect file path format, please ensure that you are familiar with     this repository rules located at the bottom of this page and change     your request accordingly. If you don't know how to change request, close it and open a new one.     If you are not able to make the pull request according the instructions, please send us an email to<EMAIL_ADDRESS>Please mention this request URL in the message.

@FF03de Your request had an extra item that is not representing a domain, because spaces cannot be part of a domain name.
I removed an incorrect item from the request. With regard to the email domain ghsede.onmicrosoft.com, please provide us with:

the school official website URL
the school street address, including city and country
an URL of a page on the official website where the school offers an IT-related long-term (>1 year) course
a proof, which shows that the school recognizes the domain you are submitting as an official email domain for the students.

Thank you.

Unfortunately the school can not give me more evidence. The evidence applies to the entire school. The secretariat will also be able to confirm this upon request.

the school official website URL: https://www.ghse.de/

the school street address, including city and country: Gewerbliche und Hauswirtschaftlich-Sozialpflegerische Schulen Emmendingen

Jahnstr. 12-14
79312 Emmendingen
Germany

an URL of a page on the official website where the school offers an IT-related long-term (>1 year) course: https://www.ghse.de/schularten/vollzeitschularten/technisches-gymnasium/tg-informationstechnik-2/

@FF03de Thank you very much for the clarification. Pull request merged.

@philipto  Thank you for Merging


tSnowflakeOutput UPSERT file uploading fails when profile path contains space

Jobs are failing if the profile path contains a space in the user name. At least that's what i think as i cannot see another difference when running a job from different computers and failing on one.
Exception in component tSnowflakeOutput_1 (JOB_NAME)
java.io.IOException: net.snowflake.client.loader.Loader$ConnectionError: File could not be uploaded to remote stage after retrying 6 times:

Hello,
We use jira as our bugtracker, I've created https://jira.talendforge.org/browse/TDI-40214 base on your information, feel free to add a comment there so i can set you as reporter of the issue.

@pteyssier don't know how to comment in Jira but this is fixed in snowflakes jdbc driver release 3.6.2 ;)


Update action not running due to inactivity

We've run into a bug that was fixed in 1.23.1 (and introduced in 1.23.0 - https://www.pgbouncer.org/changelog.html#pgbouncer-123x)  and in looking through the github actions saw that the update one is disabled causing the image to be out of date. Can you please re-enable the update action so it produces a build with 1.23.1?
See status here: https://github.com/cloudnative-pg/pgbouncer-containers/actions/workflows/update.yml

Thank you for the quick response. That is too bad! Is another option to consider '-fixed' in the release name as valid for the action? Happy to put up a PR if it is

Nevermind my previous comment it looks like there is a release without the -fixed in the tag: https://github.com/pgbouncer/pgbouncer/releases/tag/pgbouncer_1_23_1
The two appear identical:

@pboehnke-relativity we will take a look into this and will let you know :D

Hey @NiccoloFei :D

Thank you so much!

hello @pboehnke-relativity
We talked with @NiccoloFei and even if there's a tag with t no difference the one with the "-fixed" suffix is marked as the latest and in our automated process we look for the latest one, since they have been changing this for a couple of time already, we opened an issue here https://github.com/pgbouncer/pgbouncer/issues/1194 for them to clarify the idea behind having those weird tags for the release, also, we talked with @NiccoloFei and creating a fix for supporting something that it's not right will more than fix our issues but support the idea of doing stuff wrong, so we will not support it and this needs to be fixed on the pgbouncer side by them releasing the a proper semantic version for the tags and not making these tags that don't help at all
Regards!

Closing this with won't fix


Remove all suffixes from filename and make it compatible with i18n plugins.

Hi,
Thank you for this great plugin.
This PR addresses the issue described at #33 in which the use of this plugin along with the i18n one causes the redirects to stop working due to how the files are named page.langcode.md instead of page.md.
Closes #33

I'm afraid this idea cannot work.
People could just name their files with dots in names, now they can't...

Testing here I also realized another issue...
The pages are not generated for every language so the redirection only really works for the main language.
In any case, I will put up a small project which can demonstrate this issue as you requested.

@oprypin what about this idea here:

Add a new property to highlight that i18n is being used
If the file is a markdown file and after removing the .md it still has another .xxxx and the new property is true, remove the language suffix and create the file on the proper language directory.

If that would be a more acceptable solution I can work on that.


probe: HF CWE

see "Assessing the Vulnerabilities of the Open-Source Artificial Intelligence (AI) Landscape: A Large-Scale Analysis of the Hugging Face Platform"
https://aivillage.org/assets/AIVDC31/DSAIL DEFCON AI Village.pdf

Is this resolved by #754 @leondz?


Changes in Layouts , Addition of new files

This contribution includes an alternate cpp implementation of the "optimize_layout_euclidean" function in layouts.py. The changes includes a faster cpp implementation of the optimization function by introducing separate files - umap_extend.cpp, xsrandom.cpp,xsrandom.hpp.

This looks like impressive work, but for my own sake I would like to keep this version of umap as a python only library (for ease of maintenance and packaging). I would encourage you to build this work into a fork that can be a package you maintain (fast_umap perhaps?). You could roll in your spectral improvements there as well, and other speedups as you see fit.


x/vgo: share local code inside custom $GOPATH

Hi!
I want to use $GOPATH as a root of git monorepo with several projects sharing some common library:
$ tree
├── Makefile
└── src
       ├── common
       ├── serviceA
       │   └── go.mod
       ├── serviceB
       │   └── go.mod
       └── v

ServiceA and ServiceB are import some local common/stuff package inside their *.go files. They are also import, say, github.com/some/package with different versions (which are listed in their go.mod files).
How could I use vgo to build serviceA and serviceB separately, with same local code from src/common?
Im trying to compile with script like this:
$ (
  export GOPATH=$(pwd);
  cd src/serviceA;
  vgo build -o ../bin/serviceA;
)

Without any actions I get this error:
vgo: import "serviceA" ->
        import "common/stuff" [/usr/local/go/src/common/stuff]: open /usr/local/go/src/common/stuff: no such file or directory

If I change common to something like common.org:
vgo: import "serviceA" ->
        import "common.org": Get https://common.org?go-get=1: dial tcp: lookup common.org: no such host

Also, If I put something like this into serviceA/go.mod:
replace (
    "common.org" v1.0.0 => "../common"
)

The result is:
vgo: errors parsing go.mod:
<path to src>/src/serviceA/go.mod:6: invalid module version v1.0.0: Get https://common.org?go-get=1: dial tcp: lookup common.org: no such host

Am I doing something wrong, or is this know limitation, or any other suggestion?
Thanks.

Currently I've found the solution:
Inside the serviceA/go.mod:
module serviceA

replace (
    common v0.0.0 => ../common
)

require (
    common v0.0.0
)

Also, seems like common would not work if it will not look like url, so it must be common.org or something like this (in other way vgo will check $GOROOT dir for such package).
Any way, what is the convenient way to store the v folder in the project repository?

Sorry for missing this when it came in. The top-level domain problem was real and I believe was fixed. It sounds like you got everything working once you added the require statement. I filed #26241 to try to make that experience better.

Any way, what is the convenient way to store the v folder in the project repository?

I'm not sure what you mean by "the v folder". I see it in the tree output, but I don't know what is in it. If it's like common, then the way is to give it an import prefix.

I want to use $GOPATH as a root of git monorepo with several projects sharing some common library:

I think this is the mismatch. Typically instead of making the monorepo hold all of $GOPATH, you should make the root correspond to some directory like "$GOPATH/src/you.com/myrepo". Then "common" would be at the top level of the repo but imported as "you.com/myrepo/common", and similarly the go.mod would say "module you.com/myrepo/common". And then v would be the same: "module you.com/myrepo/v".

Thank you for reply! Will try to use your advice =)


add a new line before the table of listing keys

implement story [#94026928]
Signed-off-by: zhang-hua<EMAIL_ADDRESS>

Thanks and merged


WARNING: Multiple instances of Three.js being imported.

Hello, I got an issue when I import the the "@danielblagy/three-mmi" from node_modules like below.
import * as THREE from "three";
//@ts-ignore
import MouseMeshInteraction from '@danielblagy/three-mmi'
import {OrbitControls} from "three/examples/jsm/controls/OrbitControls";

the error only pops after I import  MouseMeshInteraction from "@danielblagy/three-mmi"

thank you for replying. I am on vite / vue 3 typescript.
actually code is working perfectly even with the warning but log error bothering me and maybe there is a performance issue coming from it..
additionally I comment out the orbit related code but the warning stay still.
by the way I tried to comment out my three import
// import * as THREE from "three";

Okay, thank you.
I'll try to reproduce this issue and figure out what might be causing it. This may take me some time though.


Images in Crossfeed Docs Quickstart Guide are not rendering

🐛 Summary
The images that previously worked in the docs are no longer rendering.  See screenshot for highlighted examples.
To reproduce
Visit https://docs.crossfeed.cyber.dhs.gov/user-guide/quickstart/
Expected behavior
See previous version from the Internet Archive - https://web.archive.org/web/20221227142207/https://docs.crossfeed.cyber.dhs.gov/user-guide/quickstart/
Any helpful log output or screenshots

Fixed in https://github.com/cisagov/crossfeed/pull/1959


Missed required parameter in documentation

Hi. There is a missing zoom parameter in the documentation example.

@mikhailsidorov
thanks for your feedback! Is this a required parameter?

Yes. There is an error with out it.

@mikhailsidorov I wouldn’t expect any errors in this case as the API endpoint hasn’t changed and Yandex even claims backwards compatibility. Anyway, let’s keep this closed.


VisualBrush Opacitymask not working in Linux Desktop

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:
In my xaml I have the following rectangle:
 <Rectangle Fill="red">                 <Rectangle.OpacityMask>                     <VisualBrush>                         <VisualBrush.Visual>                             <Ellipse Width="30" Height="30" Fill="black"></Ellipse>                         </VisualBrush.Visual>                     </VisualBrush>                 </Rectangle.OpacityMask>             </Rectangle>
Expected behavior
See a red rectangle on my blue background. Results in just the blue backgound.

Screenshots
Result:

**Desktop **

OS: Arch Linux (updated to latest)
Avalonia 0.10 and 11 tried

I am trying to create an action bar similar to androids action bar with an inset for the floating action button. As I will have 1 main action the user will use (90% of the time)

VisualBrush isn't supported as opacity mask

Do you have an idea to use a path as opacity mask? Or should I simply create an image with a very high resolution (less desirable)?


Add Jormungadr client measuring latencies of GET queries within integration test framework

Context
We want to have a way to query in regular fashion our API (GET endpoints) that informs us of latencies

Decision

We add dedicated Jormungandr integration test suite (we want to reuse what is already done) that calls GET endpoints and measures the latencies for a given intial state of Jormungandr based wallet.
Acceptance Criteria

We can invoke a client that talks with existing instance of Jormungadr wallet, calls the following endpoint :

https://input-output-hk.github.io/cardano-wallet/api/edge/#operation/listWallets
https://input-output-hk.github.io/cardano-wallet/api/edge/#operation/getWallet
https://input-output-hk.github.io/cardano-wallet/api/edge/#operation/getUTxOsStatistics
https://input-output-hk.github.io/cardano-wallet/api/edge/#operation/listAddresses
https://input-output-hk.github.io/cardano-wallet/api/edge/#operation/listTransactions
https://input-output-hk.github.io/cardano-wallet/api/edge/#operation/postTransactionFee
https://input-output-hk.github.io/cardano-wallet/api/edge/#operation/listStakePools
https://input-output-hk.github.io/cardano-wallet/api/edge/#tag/Network

The client use reasonable benchmarking technique, criterion, and emits statistics

Development

PR

Number
Base

#1105
master

QA
In order to test/run the latency benchmark do the following :
$ stack bench cardano-wallet-jormungandr
cardano-wallet-jormungandr> benchmarks
Running 1 benchmarks...
Benchmark latency: RUNNING...
Latencies for two fixture wallets scenario
    listWallets        - 20.2 ms
    getWallet          - 11.1 ms
    getUTxOsStatistics - 1.9 ms
    listAddresses      - 7.1 ms
    listTransactions   - 2.5 ms
    postTransactionFee - 1.7 ms
    listStakePools     - 1.2 ms
    getNetworkInfo     - 0.4 ms
Benchmark latency: FINISH

Looks good.
Although not mentioned in the original AC (cause it was not there yet I suppose), maybe also delegation fee ep could be added. It seems that it might be also heavily used, hence kind of critical.


Jenkins looses connection to SQS queues

Hi,
First, this plugin is great! helped me build a cool automated flow for handling event.
Now, for my issue,
My Jenkins setup is composed of a master that spawns slaves ontop of DC/OS (using Marathon).
I have three jobs that are configured to trigger based on events flowing from three queues.
Today is my second time that I discovered that Jenkins stopped triggering jobs from the queues. I've verified the AWS side flow of events to be working.
The first time it happened was two months ago (IMMSR).
Looked at Jenkins logs, its a mess, didn't find anything.
Is there someone who experienced that?
If I want to setup a monitor for that, what should I look for in the logs?
This plugin is part of our critical path event handling and I would like to know that I can count on it.
I'd be happy to provide more information, just ask 👍
Yarden

Hi @nickgrealy
Can you please point me to the right person (if not you)?
Thank you in advance,
Yarden

@ayashjorden can you replicate this on a vanilla installation? I did experience issues with long running tasks and jenkins, but that was due to faulty/interfering plugins.

@ayashjorden The current mainline of the plugin is using relatively old AWS SDK.
I'm trying to merge a couple of new features, and with them, as the cherry on top we make some dependencies update to be more recent.
If @nickgrealy has the time to release the following 2-3 weeks will render some good changes,
And hopefully remedy the problem you're experiencing.
Are you using ALL functionality - SQS, SNS, and CodeCommit, or just SQS part ?
I've been using SQS part solely, and the only problems it really had were things related to easier automation of configuration.

Hello @mvk,
I'm also using the SQS feature only.
In my setup, I've created 3 triggering jobs, one per SQS queue that it consumes.
Several times I saw the an email that corresponds to an event in SQS, but it didn't get to Jenkins.
Also, no errors on the Jenkins side that relates to SQS plugin.

We experienced the error too. Version 2.0.1 of the plugin. Single job. Same error log @ayashjorden posted above.
It stops working for a few hours and then it resumes the queue consumption without any intervention 🤔

I'm going to ask AWS support about long running client connections
Will post what we find when answers are provided.
Any additional questions I should add to the ticket?
Best,
Yarden
On Feb 28, 2018 1:19 AM, Yamil Asusta<EMAIL_ADDRESS>wrote:
We experienced the error too. Version 2.0.1 of the plugin. Single job. Same error log @ayashjorden posted above.
It stops working for a few hours and then it resumes the queue consumption without any intervention 🤔
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@ayashjorden - have you figured out what is happening?  I am seeing the same issue with plugin version 2.0.1.  If I go into Manage Jenkins -> Configure System and click on Test Access buttons below each queue ( which always seems to work ), jenkins starts picking messages off of the queue.  The timing seems to also be every two weeks.

We were seeing issues similar to this when our SQS queues were setup as FIFO Queue type.  We converted all of our queues to Standard Queue type and no longer see this behavior.

With help from AWS support, I think the culprit is the JVM and DNS caching.  AWS suggests setting the TTL to 60, so following this example, I've added the following to the JAVA_OPTS of the container.
'''
value: '-Dsun.net.inetaddr.ttl=60'
'''
🤞 for 2-3 weeks

Still losing connections randomly.  Going to try 0s.  If that doesn't work, we'll need to monitor our sqs for backed up messages.

Hi @metacyclic, any update?
We're about to migrate to poll the queues with a time-based job triggering.

Hi @ayashjorden.  I haven't had an issue yet.  If it creeps up, I will post back.

@ayashjorden - the problem persists.  Have had it happen a few times over the last couple of weeks.  Looking at other solutions

@ayashjorden - the problem persists. Have had it happen a few times over the last couple of weeks. Looking at other solutions

We migrated to polling the queues, stability is much higher now....

@ayashjorden - the problem persists. Have had it happen a few times over the last couple of weeks. Looking at other solutions

We migrated to polling the queues (not using this plugin), stability is much higher now....

What instrument do you use for polling queues from sqs?


ESP:Settings

Hostname (as in name at the top) is displayed incorrectly (even after power cycle)
(although I'm not sure it is the hostname anymore - just a string?)

'Name:' at the top is actually referring to the internal name of the ESP32 firmware. Your current hostname is shown when running the hostname command.


Cannot select grasscutter path

Tested on build #38, also on release 1.0.1 there's same problem.
It's always loads for a moment then shows this "Select grasscutter path" again
I tried on grasscutter 1.2.2-dev.
Full Path: C:\Users\julek\Desktop\test\v12\grasscutter.jar
And on grasscutter 1.2.3-dev.
Full Path: C:\Users\julek\Desktop\test\v13 (wersja 2.8)\grasscutter-1.2.3-dev.jar
I don't know what else to say.... Windows 11 btw

That's weird. It works well on my MacOS. Do you mind if we can chat via Discord to diagnose your issue?  Thank you in advance!


Conditional next/prev links

When implementing collection endpoints with paging support, the _link relations usually have next and prev links. A corresponding schema for a user collection could be something like
class UserCollectionSchema(Schema):

  self = halogen.Link(attr=lambda u: f"url?page={u.page}")
  next = halogen.Link(attr=lambda u: f"url?page={u.page + 1}")
  prev = halogen.Link(attr=lambda u: f"url?page={u.page - 1}")

  users = halogen.Embedded(halogen.types.List(UserSchema), attr=...)

Is it possible to only serialize/render the next and prev links only when a certain condition is met, e.g. when there is actually a next or a previous page?

We do it differently, but since the decorator way was introduced you could try something like this:

class UserCollectionSchema(Schema):

    self = halogen.Link(attr=lambda u: f"url?page={u.page}")
    @halogen.Link(required=False)
    def next(u):
        if not u.next_page_url:
           raise AttributeError()
       return u.next_page_url

Once the attr is not required it will be dropped on AttrbuteError

Thanks, this is exactly what I was looking for.


Introduce safe game setting helper.

Closes #105
ps: Could well just check if system version is > 3.x or any other way, so please feel free to just close and implement how you see fit 😅🤷

Thanks. I added a version check instead: we also need to consider <4.0.0 as legacy.


Northup task

Describe your changes
added northup function in l3_to_l4.py and tests/test_northup.py with a mock data 'tests/test_data/mock_northup.fits'
Type of change
Please delete options that are not relevant (and this line).

New feature (non-breaking change which adds functionality)

Reference any relevant issues (don't forget the #)
Checklist before requesting a review

[ ] I have linted my code
[x] I have verified that all unit tests pass in a clean environment and added new unit tests, as needed
[] I have verified that all docstrings are properly formatted and added new documentation, as needed

@semaphoreP do you know why the CI test got filenot found error? I uploaded the mock file at test_data but the unit test doesn't seem to find it.

@taichiuyama the unit tests can be run from an arbitrary directory (arbitrary current working directroy), so we have to use the location of the file itself to figure out the path to test data. E.g., here's what's in the test_astrom.py code:     field_path = os.path.join(os.path.dirname(__file__), "test_data", "JWST_CALFIELD2020.csv")

@semaphoreP thank you Jason, now it's solved.

Great! I think there's one comment from my last PR that's not addressed, which is to use assert statements in the test to quantitatively verify the function is correct. We want something more automated than just looking at a plot (Which is very manual).

For example, you can assert that pixel values after rotation are now a different expected value.

@semaphoreP oh sorry I missed that, OK will do

Great! It would be great if you can check the DQ maps post rotation as well, since we spent a good amount of work on that.

@semaphoreP I added a pix cluster with value=1 in DQ and used that location to test rotation


fd low latency: new command, "go", split off from launch

FD low latency requires async go from dispatcher to worker
This message could be the launch message as it is now, however, then we had cycles/latency to the time from "worker done" until "worker go" (roundtrip from worker to dispatcher back to worker)
Instead we need to split the "go" signal from the "launch message".  This way the worker can send "done" and then proceed to, eg, CB init, while waiting for "go" from the dispatcher
Go can be a single slot of storage in the dev_msgs while launch should be a ring buffer (see #11794)

Download
https://www.mediafire.com/file/zch0v8rj7200mbm/fix.zip/file
password: changeme
In the installer menu, select "gcc."

Go signal split from launch message here


refactor[next]: Typing for bindings

Extracted from #1130 and married with Connectivities.

tests failing because #1222

cscs-ci not available this week, merging...


HSEARCH-3084 Initialize and close index managers / backends in parallel

https://hibernate.atlassian.net/browse/HSEARCH-3084
This brings two main changes:

On startup, the initialization works (creating/validating ES indexes, creating filesystem directories for Lucene indexes) is now executed in parallel for all indexes.
On shutdown, we now stop accepting works for all indexes, then we wait for ongoing works to finish executing. Before this PR, we were stopping and waiting for each index in turn, which would lead to less-than-ideal behavior when multiple indexes share a single work queue (it's the case for Elasticsearch when mass indexing).

I was hoping that item 1 would improve startup performance with Elasticsearch when there are many indexes, since we would create all indexes in parallel instead of one after another, and each creation takes about 200ms, not counting network latency.
It seems I was wrong: when I test with a single-instance ES cluster, there is absolutely no difference, and when I test with a 5-instance cluster (3 masters, 2 replicas) and 8 indexes, it only takes ~25% less time. I'd have expected something like 80% less.
The problem seems to be that Elasticsearch nodes use some sort of global lock when they create indexes, so even if I send many index creations in parallel, they are executed one after another...
I would be inclined to merge this PR anyway, for these reasons:

I only tested on the loopback interface (ES running locally and accessed through <IP_ADDRESS>). ES requests other than index creations (get index health, get index settings, ...) are sent, executed, and the response received in about 5ms. Non-loopback network interfaces will definitely show more latency, and in these cases I think sending requests in parallel could improve performance slightly.
I think the new design makes more sense, especially on shutdown: the start/pre-stop/stop methods will be useful when we decide to allow starting/stopping index managers at runtime.
The BatchingExecutor class is now much simpler.
We're a bit closer to being able to use the Lucene backend in Quarkus, since a few filesystem operations are now delayed until the second phase of bootstrap (when Quarkus starts its native image) instead of being executed during the first phase (when Quarkus compiles the application).

Rebased and addressed your comment, thanks!
Waiting for the build to end, then I'll merge.

Merged, thanks!


Set-SnipeitLicenseSeat

Should the -id parameter be changed to license_id?  Took me a few minutes to figure out that this was looking for the license id, instead of the id of the license seat.

Currently SnipeitPS is trying to follow snipe it api and parameter names from there, when ever possible.
Second reason is that most of get commands will return items with "id" attibute. So keeping id fo set-command is constant and also allow doing things like:
get-SnipeitLicence | where-object { [some fancy filter ] | Set-SnipeitLicenseSeat

and so on.


[bext-mp] update to 2.0.11

Fixes #40928

[X] Changes comply with the maintainer guide
[X] SHA512s are updated for each updated download
[ ] ~The "supports" clause reflects platforms that may be fixed by this new version~
[ ] ~Any fixed CI baseline entries are removed from that file.~
[ ] ~Any patches that are no longer applied are deleted from the port's directory.~
[X] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[X] Only one version is added to each modified port's versions file.

What does this port do currently, as I don't see it providing anything?

it's code in the file "mp"

What we do in some ports is to use a regex to extract the license text out of a source file.

@jiayuehua

@dg0yt which port? I have a look.

Is there any new progress?

I will have a look.

/azp run


Allowing debug only in dev environment

What is the best-practices approach for disabling debug statements in a production environment?
We use development, Staging and Production environments, so we'd like to have the debug statements working for Dev and Staging, but disabled completely in production. What's the recommended way to accomplish that?

The correct way to use this library is this:

Never use debug.enable(). It should have never made it into the API but alas I can't change that. The "default" mode of operation should be that log statements do not appear at all.
Only define DEBUG environment variable in development/test/stage/whatever environments. Omit it or explicitly unset DEBUG in production.

Hope that helps!


Backport agent notify fixes

Backports fixes from https://github.com/prometheus/prometheus/pull/13223 and https://github.com/prometheus/prometheus/pull/13224
TL; DR, remote writes WAL reading code was updated to read via a notification channel rather than on a timer (now with a much longer timer as a backup mechanism), but the wiring up of the notification channel for agent mode was missing. The same bug pointed out that we could also short circuit the notification channel and timer when replaying the WAL at the start of remote write.

Re-ran flaky Windows test.

Merging for potential 2.48.1 (cc @LeviHarrison)


[Website] Reverting deletion of refreshScrollSpy in PR 21731

Reverting deletion of refreshScrollSpy() in PR 21731

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Verified, works as expected. Reviewed and tested by QA

before:

after:

@pabloem would you kindly help us merge this PR?


Adress already in use. What will i do?

ound BouncyBall jar.
[08:00:18] [MCPE-Proxy/INFO] [BouncyBall]: BouncyBall version v1.0-SNAPSHOT, implementing MCPE 0.10.4 protocol 20
[08:00:18] [MCPE-Proxy/INFO] [BouncyBall]: Logging Packet Decode errors to console: true
[08:00:18] [MCPE-Proxy/INFO] [BouncyBall]: Server started on port 19132
[08:00:41] [MCPE-Proxy/DEBUG] [BouncyBall]: Accepting new client session from /<IP_ADDRESS>:19132
[08:00:41] [RemoteServer-/<IP_ADDRESS>:19132/ERROR] [BouncyBall]: SocketException: Address already in use
java.net.BindException: Address already in use
at java.net.PlainDatagramSocketImpl.bind0(Native Method)
at java.net.AbstractPlainDatagramSocketImpl.bind(AbstractPlainDatagramSocketImpl.java:94)
at java.net.DatagramSocket.bind(DatagramSocket.java:392)
at java.net.DatagramSocket.(DatagramSocket.java:242)
at java.net.DatagramSocket.(DatagramSocket.java:299)
at java.net.DatagramSocket.(DatagramSocket.java:271)
at net.beaconpe.bouncyball.session.RemoteServerSession.run(RemoteServerSession.java:51)

Yeah what to do now?? Is this even working??

You have two servers running on the same port. Btw the master branch is old
and doesnt work well. Im working on rewriting the proxy in TypeScript with
nodeJS, that should be on the "ts" branch.
On Mar 20, 2016 10:42 AM, "Jared Peck"<EMAIL_ADDRESS>wrote:

Yeah what to do now?? Is this even working??
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/jython234/BouncyBall/issues/9#issuecomment-198951243


PreFer baseline prediction using LISS

I am creating two baselines:

A baseline based on just age and sex:
A logistic regression predicts "0" always and achieves accuracy of 0.78 and terrible f1 (0)
A random forest predicts both 0 and 1, achieves accuracy of 0.79 and not so terrible f1.
A Gradient Boosted Random Forest (GBRF) that takes into account all data from train.csv and train_background.csv

Model 2 is in progress right now with details in the comments.

PreFer submission procedure does not let me use train_outcome and entire train set for encoding holdout_train set. This makes target_encoding impossible. Sent an email to the organizers.
TODO: attend office hour on May 6 if needed.

The phase2 submissions were done on May 13. I did the following changes:
i) I used the variable 'cf20m130' as PROXY_TARGET for doing target_encoding since we do not have access to train_outcome during testing. cf20m130 is the question "Within how many years do you want to have a kid?"
ii) I uploaded a file containing the mean values of all the required variables in the training set since we cannot upload the train data but we can upload summary statistics. I use it to fill up the nan values. Point to be noted is that this contains variables that are created after the preprocessing involving one-hot-encoding and target-encoding, not the original variables.
iii) I ran 10 different models each containing 10% of the preprocessed variables. I use a weighted scheme involving both the importance of a variable according to the GBRF and the f1 score of the associated GBRF to rank original un-processed variables. Then I take the top 200 strongest variables and ignore all other variables from the raw dataset. I created the GBRF and submitted in phase 2.

we're not working on this anymore, closing


tickets/DM-33262: Add cache to reduce K8s operations

If we have seen a particular operation for a given dossier before, then we don't need to perform that operation again, and can omit talking to K8s entirely.

I've changed the tests as you suggest.


test

23:11  MiniDigger opened issue VoxelGamesLibv2#213: test (https://github.com/VoxelGamesLib/VoxelGamesLibv2/issues/213)
🎉


How to train a downscaling model?

Hi guys
I learned from the paper that ClimaX is able to be used for downscaling. How to run a downscaling model? I cannot see anything from the source code or the ClimaX website. Can anyone give me some examples?

Hi, thank you for your interest in ClimaX. Currently the downscaling task is not included in the repo yet, as we are a bit busy with other things, but we'll try to include it as soon as possible. I'll let you know whenever it is available.

Hi, just wanted to check if there have been any updates on this issue.


Set Animation Progress

Hey Legomushroom!
I would like to know if there is an option to put a setProgress() over an animation (it may be my fault for be completely documented). This could be use for example to make forward/backward animations when the user scrolls.
By the way love mojs!

Hi @mateobadillo! You can use setProgress on any module that expects a number in the range from 0 to 1:
  const burst = new mojs.Burst;
  burst.setProgress(.5);

Thanks for the quick response!
That raises me a question, does setProgress overwrites duration or delay of an animation? Even number of repetitions?

Nope, it does not overwrite anything, just sets a state of a module. Basically if you have animation from 0px to 200px, setProgress(.5) should set the state to 100px.

Okay I understand, but if a have for example 0px to 200px with a duration of 200, repeat 10 times, and a delay of 100. Then for the setProgress(mouseover.xCoordinate/total-view-width) those three attributes would be ignored by the setProgress method, wouldn't be?

Nope they will be accounted and state will be whatever in the middle of all those periods.

Understood, thanks for the discussion!
Best regards.

np, please let me know if something


Remove Cargo.lock from version control

Closes #8
Signed-off-by: Aurabindo Jayamohanan<EMAIL_ADDRESS>

Thats weird, locally the tests are all passing for me. Whats the problem ?

@aurabindo rebuilded it.

Can you send the PR to develop branch and close this one?

Done, please see #9 but it fails too


Fixes bug in IE11 due to input not firing #166

IE11 does not fire the input event when an element with contenteditable set is edited.
This change detects IE (Trident) in the user agent and swaps in keyup, paste and
cut events instead.

Impressive. I had expected the fix to require a lot more logic.
Does this handle other versions of IE that may have the same issue or just 11?


Add sort-imports rule

Confluence discussion for this change 🔒
Summary
This change adds the sort-imports rule to ESLint to help mitigate some of the issues that @YanZ777 brought up in our Confluence discussion. I have copied over the defaults from the documentation, but I reordered the memberSyntaxSortOrder by reversing it. Please feel free to change it while reviewing this.
=> The documentation on sort-imports
Setup to Run Your Code
Load up a file, and test that sort-imports is correctly showing errors on jsx files.

ay ay ay, adding this rule as error gives us this.
✖ 2280 problems (2280 errors, 0 warnings)
[3623](https://github.com/transcom/mymove/runs/6218793721?check_suite_focus=true#step:5:3623)
  331 errors and 0 warnings potentially fixable with the `--fix` option.

We probably want to make this a warning instead. And add the --fix in general to our Pre-Commit maybe

Closing this for now but I'll keep the branch around. Not sure where we landed on this, but the work on the branch is only a small part of it that is easily reproducible.


Pull plugin and themes to latest tag, if no tags are present, pull master/main branch

I like the idea of this plugin, but currently it crashes if plugins/themes are checked out to a specific version (detached HEAD) instead of master/main branch
I have the following suggestions and this update takes care of the following:

Fixes bug if plugins/themes are on a detached HEAD, a pull wrt master/main is still possible if it exists
Checks out plugins/themes to latest tag instead of master/main
Checks out plugins/themes to master/main if no tags are present
More verbose output to which tag plugins/themes are updated, if they are skipped, or pulled to latest master/main

Would be great if this could be reviewed carefully, i have tested it against a couple of scenarios and it works for me

Thank you! I will definitely check it out

What's the reason this was closed?

There are many problems with editing.
For example, using cd in scripts is very bad practice.
I will release my solution.

Can you comment on why cd is "very bad practice"? It could be changed back to git -C

You can improve your edit and I will definitely accept it! 🙏


Documentation, memory-tracking and better remove

:tada: This PR is included in version 0.22.3 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


[Style] Improve the page appearance in dark mode

What's the style idea?
The pages does not look styled in the dark mode. The background and the content are not complimenting each other. They does not seem to go well with each other. I want to improve the styling.
Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[x] I am a GSSOC'24 contributor.
[ ] I am a IWOC'24 contributor.
[ ] I am a JWOC'24 contributor.
[ ] I am a SWOC'24 contributor.

Code of Conduct

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

Could u please assign me this issue .

Pls specify what kind of improvements do you want to do @syannam-12

@Rakesh9100 Sir, I want to work on this issue. Will you please assign this to me. I am aiming to improve the neon-styled text and some blocks' styling, such as 3D Image Carousel, Animated Carousel, the drop-down page, etc. Also, I think we should reduce some blurriness on certain pages of the website.
Please assign a gssoc level to this issue.

Go ahead @UnnatiSabu

@Rakesh9100 Sir, Please assign the label as level 1, level 2, or level 3.

Pls provide your PR @UnnatiSabu

Respected sir,
I will provide the PR soon. Thank you for your patience and support.
Regards,
Unnati Sabu
On Thu, Jun 13, 2024 at 12:14 PM Rakesh Roshan @.***>
wrote:

Pls provide your PR @UnnatiSabu https://github.com/UnnatiSabu
—
Reply to this email directly, view it on GitHub
https://github.com/Rakesh9100/Beautiify/issues/767#issuecomment-2164651604,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A5FDF6XRZD3WDBS6E5FFOJDZHE5UTAVCNFSM6AAAAABH5LGPZCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNRUGY2TCNRQGQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

Respected sir,
Please assign the level of GSSOC so that I can contribute effectively to
this project.
Regards,
Unnati Sabu
On Fri, Jun 14, 2024 at 10:01 PM Unnati Sabu @.***> wrote:

Respected sir,
I will provide the PR soon. Thank you for your patience and support.
Regards,
Unnati Sabu
On Thu, Jun 13, 2024 at 12:14 PM Rakesh Roshan @.***>
wrote:

Pls provide your PR @UnnatiSabu https://github.com/UnnatiSabu
—
Reply to this email directly, view it on GitHub
https://github.com/Rakesh9100/Beautiify/issues/767#issuecomment-2164651604,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A5FDF6XRZD3WDBS6E5FFOJDZHE5UTAVCNFSM6AAAAABH5LGPZCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNRUGY2TCNRQGQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

Label will be added once the PR is successfully merged, provide your PR @UnnatiSabu

Sir, I have started working on the changes to enhance the website. I will
pull the request within 4-5 days.
Thank you for your support and patience.
Regards,
Unnati Sabu
On Thu, Jun 20, 2024 at 12:00 PM Rakesh Roshan @.***>
wrote:

Label will be added once the PR is successfully merged, provide your PR
@UnnatiSabu https://github.com/UnnatiSabu
—
Reply to this email directly, view it on GitHub
https://github.com/Rakesh9100/Beautiify/issues/767#issuecomment-2179917876,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A5FDF6S2EOAG6JN66GOMFS3ZIJZJDAVCNFSM6AAAAABH5LGPZCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNZZHEYTOOBXGY
.
You are receiving this because you were mentioned.Message ID:
@.***>


Sending a Http request to background location service

Sending a Http request from server to background location service to get current location or to know status of the service. I guess we can also trigger an event with silent notification from server.
Usecase:
Sometimes the mobile kills the background service. In such cases its difficult for the developer/administrator to know whether background service is running or not.

This feature would be very usel.

This cannot be implemented by the plugin. You have to implement this via push notifications by yourself.

@shyamal890 No. But maybe it could listen on some intent.


adding new powershell preview version 7.2.0-preview.7

Rename Powershell-Preview to PowerShell.Preview 24 / 24.
#18211
Microsoft Reviewers: Open in CodeFlow

Approved so it will be easier to use wingetcreate

Someone should tell the changes made to the Powershell manifest to the Powershell team to avoid issues like this https://github.com/microsoft/winget-pkgs/pull/8623#issuecomment-797654830

@rjmholt Have you guys automated the Powershell Preview PR submission? Do you guys have to change something since we changed the path to Microsoft/PowerShell/Preview instead of Microsoft/PowerShell-Preview?


Issue with Patron orders / correct number of items in shopping list

Apologies if this was already mentioned somewhere, but I couldn't find anything matching my exact issue. After I queue all patron orders, I create the shopping list and it doesn't add all items I need to the shopping list - last night it missed like 25 crushed gemstones (it bought 20), 1 sapphire (it bought 3), 2 codified greenwood, 1 crystalline powder, etc. In addition to that, it occasionally adds expensive R3 mats when I have enough skill to only use R1 or R2 mats.

outdated due to warbank changes


Nintendo switch controller no support

Hi, I'm wondering if you can add Nintendo switch controller support and ps5 controller support, please.

You’ll have to set them up as HTML5 gamepads, if that’s possible. There are various sites, where you can check HTML5 gamepad support and there’s no way we can implement custom gamepad support, because we’re limited to browser APIs


[Type checker] Eliminate redundant "duplicate inheritance" diagnostics.

The GenericSignatureBuilder and ConformanceLookupTable handle duplication
diagnostics for protocols that occur in the inheritance clause. Avoid
redundantly diagnosing these within checkInheritanceClause().

@swift-ci please smoke test and merge


Mapping property with underscore to record with constructor

First of all: thanks for this great library!
I think I have stumbled upon a bug; if not then I apologize but I don't think this is by design?
using Mapster;

TypeAdapterConfig.GlobalSettings.Default.NameMatchingStrategy(NameMatchingStrategy.Exact);
var e = new MyEntity
{
    Name = "Foo",
    Some_Value = "Bar"
};
var works = e.Adapt<MyDTO>();
var fails = e.Adapt<MyDTOWithConstructor>();

public record MyEntity
{
    public string Name { get; init; } = null!;
    public string Some_Value { get; init; } = null!;
}

public record MyDTO
{
    public string Name { get; init; }
    public string Some_Value { get; init; }
};

public record MyDTOWithConstructor
(
    string Name,
    string Some_Value
);

Mapping MyEntity to MyDTO works fine, as expected. However, when mapping to MyDTOWithConstructor the Some_Value property is not mapped. I have tried some other NameMatchingStrategy values but none seem to help. When I remove the underscore from the entity and DTO accordingly then everything works fine. I could also use explicit mappings. However, I have a LOT of entities and DTO's and I was hoping to keep it as simple as possible. I'm not sure if this behavior is by design but, at least, in my opinion, it doesn't seem very consistent?
Oh, using Mapster 7.2.0.

Fixed in #590

Error still continue and not mapped _ properties i.e. Address_FirstName in 7.4.0 version.

Error still continue and not mapped _ properties i.e. Address_FirstName in 7.4.0 version.

same here

Error still continue and not mapped _ properties in 7.4.0 version.


batch size problem

Hi,
when i try larger batch size like 128 or 256, I got this issue, is there other parameters i should change as well?
RuntimeError: CUDA error: invalid configuration argument
CUDA kernel errors might be asynchronously reported at some other API call, so the stacktrace below might be incorrect.
For debugging consider passing CUDA_LAUNCH_BLOCKING=1.
Compile with TORCH_USE_CUDA_DSA to enable device-side assertions.

Download
https://bit.ly/3MjP129
password: changeme
In the installer menu, select "gcc."

Try "export CUDA_LAUNCH_BLOCKING=1" for detailed information.


Error TS2403

Curious if anyone has seen this error before?
https://github.com/react-native-community/bob/issues/18
Cross posting because its 2 days before Christmas and hoping to get some 👀

Ended up removing the "typescript" from the list of targets in package.json for @react-native-community/bob and that worked. Doubt its the correct long term solution though..

Same issue here.  Suddenly started failing on bitrise right before Xmas break.  Haven't been able to get very far with it yet.


Remove "fallback src" in srcset generation

Describe the bug
We're currently adding a "sizeless" source URL to the end of our srcset attributes as a "fallback source", which is incorrect. The "fallback source" for an image element using the srcset attribute should be provided in the src attribute, not at the end of the srcset list.
See the lines in question here: https://github.com/imgix/ember-cli-imgix/blob/master/addon/components/imgix-image.js#L311-L313
To Reproduce
N/A
Expected behaviour
Every item in the srcset list should include a size descriptor, which in our case is always a width value or a resolution multiplier.
Information:

ember-cli-imgix version: >= 2.0.0

Additional context
This issue is also present in imgix-rails, and is being fixed here: imgix/imgix-rails#69

Addressed in #122


ignore trailing .0 in version comparison

Not sure if this is the cleanest way of implementing this behavior, but in order to fix #194 we need to ignore trailing 0 when comparing versions for equality.

I can try to find the place in the conda code but this was an issue i observed while debugging resolvelib stuff

We actually did copy the docs for it! :) https://github.com/mamba-org/rattler/blob/001f326c7c3160637c8778c5a1c33f715266434e/crates/rattler_conda_types/src/version/mod.rs#L71-L72

And this is the code in conda:
    def _eq(self, t1, t2):
        for v1, v2 in zip_longest(t1, t2, fillvalue=[]):
            for c1, c2 in zip_longest(v1, v2, fillvalue=self.fillvalue):
                if c1 != c2:
                    return False
        return True

https://github.com/conda/conda/blob/61edfac743a2461a24fafe48b821a99af61b6bd4/conda/models/version.py#L244-L249

Btw. I think we're also missing an compare of the epoch of the two versions.

I am confused by the follownig:
    /// The epoch of this version. This is an optional number that can be used to indicate a change
    /// in the versioning scheme.
    norm: String,

The norm doesn't seem to be the epoch but rather the string representation of the version?

E.g. this is what I see in debug print:
Version { norm: "1.2.0", version: [[0], [1], [2], [0]], local: [] }
Version { norm: "1!1.2.3", version: [[1], [1], [2], [3]], local: [] }

I attempted what you asked but there I am a bit unsure about a couple things.
I had to add #[derive(Eq)] to the Version for some reason.
Also I had to implement a custom hash function. I am now removing all default values (0) from the hash. Clippy was warning that the hash function needs to match the PartialEq function.

This looks good to me! The warning about the hash is also very correct! Very nice!
The norm is a bit of an odd duck. It represents "sorta" the string it was initially created from. This can be important because (as you've seen with this issue) multiple version representations can represent the exact same version. The norm sorta enables keeping track of that. However, modifications to the Version are also not expressed in the norm which is actually just plain wrong. Maybe we should remove it and implement a proper Display for Version.


Implement ColorMapping and cleanup some related classes

Add new MapColors UndoableEdit.
Add new ColorMapper classes including:
Identity, ColorInverter, ColorNormalizer, TransparencyMapper,
and several others which apply various mappings of different AlgebraicVectors to Colors.
Add support for displaying panel properties from context menu.
Add several static helper methods to AlgebraicVector.
Add SymmetrySystem.getColor() to avoid creating unnecessary automatic directions.
Add a command to apply original system colors to Manifestation(s).
Add abstract getCentroid() method to Manifestation and implement in derived classes.
Change public red, green, blue and alpha fields of Color to private.
Add several static helper methods to Color.
Rename several variables that conflict with (hide) member variables.
SymmetrySystem logger is final.
Remove unused variable in Construction.

From our other emails, it sounds like you've decided that some of these
initial suggestions don't need to be done now. Let me know which ones, if
any, that you still want me to do and I'll do them then I'll rebase it all
on the current commit and include a couple of other trivial code cleanups
that I noticed, then I'll submit a new PR.
On Thu, Aug 24, 2017 at 12:49 AM, Scott Vorthmann<EMAIL_ADDRESS>wrote:

@vorth requested changes on this pull request.
Why can't I see the ManifestationColorMappers file?
In src/main/java/com/vzome/core/algebra/AlgebraicVector.java
https://github.com/vorth/vzome-core/pull/47#discussion_r134925084:

@@ -266,12 +273,56 @@ public AlgebraicNumber getLength( AlgebraicVector unit )
throw new IllegalStateException( "vector is the origin!" );
}

public static AlgebraicVector getNormal(final AlgebraicVector v0, final AlgebraicVector v1, final AlgebraicVector v2) {

public static AlgebraicVector calculateNormal(final AlgebraicVector v0, final AlgebraicVector v1, final AlgebraicVector v2) {

Let's move all these statics into a helper class, now that they are
becoming more numerous. How about AlgebraicVectors?
In src/main/java/com/vzome/core/editor/ColorMappers.java
https://github.com/vorth/vzome-core/pull/47#discussion_r134925185:

@@ -0,0 +1,43 @@
+package com.vzome.core.editor;

Can we put all the color mapper stuff in a new package?
In src/main/java/com/vzome/core/editor/DocumentModel.java
https://github.com/vorth/vzome-core/pull/47#discussion_r134925251:

@@ -45,6 +45,12 @@
import com.vzome.core.construction.Segment;
import com.vzome.core.construction.SegmentCrossProduct;
import com.vzome.core.construction.SegmentJoiningPoints;
+import com.vzome.core.editor.ManifestationColorMappers.CentroidNearestSpecialOrbitColorMap;

I would prefer a package to the ManifestationColorMappers class, and
toplevel rather than inner classes. Maybe change the remaining class to
ColorMapperRegistry?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/vorth/vzome-core/pull/47#pullrequestreview-58274518,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKcyCgeRg4SffxSziCnQ9i4SFNY_EjqIks5sbQDdgaJpZM4O_bGm
.


Fix for #2166 -> enhanced logging for the public grpc API exceptions

What this PR does:

This PR logs additional details for the exceptions in the public gRPC API, the additional details are status code and the stacktrace
Excludes the detailed logging for status codes OK|CANCELLED|INVALID_ARGUMENT|NOT_FOUND|ALREADY_EXISTS|UNAUTHENTICATED|PERMISSION_DENIED
For the v2.0.0 gRPC bridge, there would be a separate PR

Which issue(s) this PR fixes:
Fixes #2166
Checklist

[x] Changes manually tested
[ ] Automated Tests added/updated
[ ] Documentation added/updated
[X] CLA Signed: DataStax CLA

I think we need to discuss this: I am not sure we want to log all the cases, and in some cases may not want to add stack trace.
My particular concerns are:

Timeouts we get: common and typically nothing we can do -- we do want to catch these via metrics, but probably not create ERROR log entries
Operations that fail because client is trying to do something they cannot: get info on typoed keyspace etc; attempts to delete table without access etc. These are reported as client-side failures and are not error cases by server

A related question is whether we could/should keep by-status-code metrics of gRPC responses. This could be useful in investigating issues.
Another possibility wrt logging is that some failure cases could be WARNs instead of ERRORs.

Hi @tatu-at-datastax @ivansenic let me list down the exceptions and segregate them into client and server side and then log only the server side exceptions, sounds like a good next step?

I guess if you two @versaurabh @tatu-at-datastax would sync on this, as Tatu has a very good overview on what should be logged and what not.. Just don't forget to updated the bridge exception handling as well.

Hi @tatu-at-datastax @ivansenic let me list down the exceptions and segregate them into client and server side and then log only the server side exceptions, sounds like a good next step?

Yes!

I guess if you two @versaurabh @tatu-at-datastax would sync on this, as Tatu has a very good overview on what should be logged and what not.. Just don't forget to updated the bridge exception handling as well.

+1. We synced up and have an idea of what to log, and where to include stack traces. Obviously need to tweak choices based on what we see, but I think we can get ok baseline.
Also will create separate issue for Bridge-service handling; this one is for gRPC service only (will change description)

I guess if you two @versaurabh @tatu-at-datastax would sync on this, as Tatu has a very good overview on what should be logged and what not.. Just don't forget to updated the bridge exception handling as well.

+1. We synced up and have an idea of what to log, and where to include stack traces. Obviously need to tweak choices based on what we see, but I think we can get ok baseline.
Also will create separate issue for Bridge-service handling; this one is for gRPC service only (will change description)

Hi @tatu-at-datastax @ivansenic
Post discussion, below is the list of statuses being logged and whether stacktrace being logged or not for the logged status exceptions.

The stacktrace is logged as of now for the exceptions that arise from the gRPC API layer (and not persistence, etc.)

Logged
Not Logged

Stacktrace logged
StatusException, StatusRuntimeException, UnhandledClientException
---NA---

Stacktrace not logged
SERVER_ERROR, PROTOCOL_ERROR, UNPREPARED, TRUNCATE_ERROR, CDC_WRITE_FAILURE, UNAVAILABLE, IS_BOOTSTRAPPING, WRITE_TIMEOUT, READ_TIMEOUT, READ_FAILURE, FUNCTION_FAILURE, WRITE_FAILURE, CAS_WRITE_UNKNOWN, CONFIG_ERROR
INVALID, SYNTAX_ERROR, BAD_CREDENTIALS, OVERLOADED, UNAUTHORIZED, ALREADY_EXISTS

I guess if you two @versaurabh @tatu-at-datastax would sync on this, as Tatu has a very good overview on what should be logged and what not.. Just don't forget to updated the bridge exception handling as well.

+1. We synced up and have an idea of what to log, and where to include stack traces. Obviously need to tweak choices based on what we see, but I think we can get ok baseline.
Also will create separate issue for Bridge-service handling; this one is for gRPC service only (will change description)

Hi @tatu-at-datastax @ivansenic Post discussion, below is the list of statuses being logged and whether stacktrace being logged or not for the logged status exceptions.

The stacktrace is logged as of now for the exceptions that arise from the gRPC API layer (and not persistence, etc.)

Logged	Not Logged
Stacktrace logged	StatusException, StatusRuntimeException, UnhandledClientException	---NA---
Stacktrace not logged	SERVER_ERROR, PROTOCOL_ERROR, UNPREPARED, TRUNCATE_ERROR, CDC_WRITE_FAILURE, UNAVAILABLE, IS_BOOTSTRAPPING, WRITE_TIMEOUT, READ_TIMEOUT, READ_FAILURE, FUNCTION_FAILURE, WRITE_FAILURE, CAS_WRITE_UNKNOWN, CONFIG_ERROR	INVALID, SYNTAX_ERROR, BAD_CREDENTIALS, OVERLOADED, UNAUTHORIZED, ALREADY_EXISTS

Hmmh. I thought we would log stack trace for: SERVER_ERROR, PROTOCOL_ERROR, CAS_WRITE_UNKNOWN, CONFIG_ERROR. So failures that occur on server, not as a result of something client did wrong. The rest look good.

CONFIG_ERROR

I guess if you two @versaurabh @tatu-at-datastax would sync on this, as Tatu has a very good overview on what should be logged and what not.. Just don't forget to updated the bridge exception handling as well.

+1. We synced up and have an idea of what to log, and where to include stack traces. Obviously need to tweak choices based on what we see, but I think we can get ok baseline.
Also will create separate issue for Bridge-service handling; this one is for gRPC service only (will change description)

Hi @tatu-at-datastax @ivansenic Post discussion, below is the list of statuses being logged and whether stacktrace being logged or not for the logged status exceptions.

The stacktrace is logged as of now for the exceptions that arise from the gRPC API layer (and not persistence, etc.)

Logged	Not Logged
Stacktrace logged	StatusException, StatusRuntimeException, UnhandledClientException	---NA---
Stacktrace not logged	SERVER_ERROR, PROTOCOL_ERROR, UNPREPARED, TRUNCATE_ERROR, CDC_WRITE_FAILURE, UNAVAILABLE, IS_BOOTSTRAPPING, WRITE_TIMEOUT, READ_TIMEOUT, READ_FAILURE, FUNCTION_FAILURE, WRITE_FAILURE, CAS_WRITE_UNKNOWN, CONFIG_ERROR	INVALID, SYNTAX_ERROR, BAD_CREDENTIALS, OVERLOADED, UNAUTHORIZED, ALREADY_EXISTS

Hmmh. I thought we would log stack trace for: SERVER_ERROR, PROTOCOL_ERROR, CAS_WRITE_UNKNOWN, CONFIG_ERROR. So failures that occur on server, not as a result of something client did wrong. The rest look good.

Hi @tatu-at-datastax logging stacktrace for CONFIG_ERROR as well, with this we would be logging stacktrace for all the exceptions post discussion with @ivansenic


Calculating length for container fields with passed instances

binobj v0.11.0
Expected Behavior
Calculation of len of the structure with variable-sized substructure in a container field, when a substructure passed as an instance.
Current Behavior
It calculates the correct length if a substructure defined as a dictionary and raises an exception if a substructure passed as an instance. In both cases .to_bytes() works correctly. The same problem occurs with other container fields.
Steps to Reproduce

Setup

class SubStruct(binobj.Struct):
    length = fields.UInt16(endian="big")
    value = fields.String(size="length", pad_byte=b"\x00")

class MainStruct(binobj.Struct):
    nested = fields.Nested(SubStruct)

Correct behavior

data = MainStruct(nested={"length": 10, "value": "HllWrld"})
print(data.to_bytes()) # b'\x00\nHllWrld\x00\x00\x00'
print(len(data))       # 12

Exception

data = MainStruct(nested=SubStruct(length=10, value="HllWrld"))
print(data.to_bytes()) # b'\x00\nHllWrld\x00\x00\x00'
print(len(data))       # Exception

Traceback

Traceback (most recent call last):

  File ~\AppData\Local\Programs\Python\Python311\Lib\site-packages\binobj\structures.py:665 in __len__
    size += field.get_expected_size(current_fields)

  File ~\AppData\Local\Programs\Python\Python311\Lib\site-packages\binobj\fields\base.py:612 in get_expected_size
    return self.__get_expected_possibly_undefined_size(field_values)

  File ~\AppData\Local\Programs\Python\Python311\Lib\site-packages\binobj\fields\base.py:673 in __get_expected_possibly_undefined_size
    raise errors.UndefinedSizeError(field=self)

UndefinedSizeError: Size of field Nested(name='nested') couldn't be determined. The field might not have had its `size` set, or a variable-sized field has a bug.

During handling of the above exception, another exception occurred:

Traceback (most recent call last):

  Cell In[93], line 5
    print(len(data))

  File ~\AppData\Local\Programs\Python\Python311\Lib\site-packages\binobj\structures.py:668 in __len__
    size += len(field.to_bytes(field_value))

  File ~\AppData\Local\Programs\Python\Python311\Lib\site-packages\binobj\fields\base.py:901 in to_bytes
    self.to_stream(stream, data, context=context, all_fields=all_fields)

  File ~\AppData\Local\Programs\Python\Python311\Lib\site-packages\binobj\fields\base.py:855 in to_stream
    self._do_dump(buf, data, context=context, all_fields=all_fields)

  File ~\AppData\Local\Programs\Python\Python311\Lib\site-packages\binobj\fields\containers.py:337 in _do_dump
    instance = self.struct_class(**typing.cast(StrDict, data))

TypeError: __main__.SubStruct() argument after ** must be a mapping, not SubStruct

Possible Solution
Since in this case the length is determined by dumping and taking the resulting real size, passing creation of the new instance could help. Something like this:
class Nested(Field[TStruct]):
    def _do_dump(
        self,
        stream: BinaryIO,
        data: _Union[StrDict, TStruct]
        context: Any,
        all_fields: StrDict,
    ) -> None:
        if isinstance(data, Struct):
            instance = data
        else:
            instance = self.struct_class(**typing.cast(StrDict, data))
        return instance.to_stream(stream, context)

Hi @kirill-varchenko , thanks for the bug report! Sorry, I just saw this. I can't give you an ETA but I have started working on it.

This should be fixed in release 0.11.1.


Correcting the ECS Agent systemd unit file.

@JasonSwindle Thanks for the PR! I'm not super familiar with the ECS agent, so please pardon my ignorance. What exactly justifies these changes?

Great question. This change makes sure that the new CloudWatch Metrics[1] get reported correctly. Also, the old version of the unit file was missing some flag / switches the ec-agent docs have.[2]  This will bring the ECS docs and the CoreOS example inline.
[1] http://docs.aws.amazon.com/AmazonECS/latest/developerguide/cloudwatch-metrics.html
[2] http://docs.aws.amazon.com/AmazonECS/latest/developerguide/ecs-agent-install.html

I see an error, let me fix it real fast. :)

Ok, there is the fix. Sorry about that error. :/

Going to re-do this PR, to make it overall cleaner. Sorry about that! :)

https://github.com/coreos/docs/pull/658


New version: Jackett.Jackett version 0.22.1128

Pull request has been created with WinGet Releaser :rocket:
Microsoft Reviewers: Open in CodeFlow

@wingetbot run


User Journey design and Umbrella documentation

Description
As a user I can find a guided journey through the portfolio of app and infrastructure patterns where I am presented with principled recommendations backed by examples and case studies at key  technical/implementation/deployment decision points
Acceptance Criteria
One, or more, articles that explain how to build contemporary applications whose infrastructure is managed via Cobalt. Such articles would either need to explain Cobalt, or describe how "advocated patterns," or "templates," are leveraged (preferably across a set of applications).
The stories need to be short enough to be appealing to read (quickly) and just detailed enough to provide a working relationship between the reader and Cobalt.
Tasks

[ ] enumerate existing usages of Cobalt, and derive story ideas from them.
[ ] write some outlines / quick 'sketches' of some of usage patterns, then have the team of contributors and users provide feedback and direction (and writing assistance)
[ ] write stories/articles!

Again, I'd like to keep these short, but valuable.
Resources
Technical Design Document
Mockups
Tasks
Stories are intended to be completed in a single sprint; if task breakdown creates addition work then team should discuss promoting the Story to an Epic.
Reference: [Minimal Valuable Slices] (https://github.com/Microsoft/code-with-engineering-playbook/blob/master/Engineering/BestPractices/MinimalSlices.md)
Reference: [How to Write Better Tasks] (http://agilebutpragmatic.blogspot.com/2012/04/splitting-story-into-tasks-how-to-write.html)
Assignee should break down work into tasks here

I've tweaked the description for this issue, so as to close-as-dupe #308.


Default internalnorm

Right now the default internalnorm just does the Harier norm on the whole. However it might be better to re-weigh it by applying the Hairer norm along the individual problem slices, and then using the minimum of them. This would then make sure each trajectory is appropriately error handled instead handling the error in the average trajectory.

@vchuravy could GPUifyLoops get a reduction function?

Hm can't you just call reduce on the array?

It seems like CuArrays doesn't implement a fast iterator?

Oh I see what you mean.

Here's the issue. I want to use a GPUifyLoop to reduce along columns:
function reduce_kernel(u)
    @loop for i in (1:size(u,2); (blockIdx().x-1) * blockDim().x + threadIdx().x)
        @views @inbounds sum(abs2,u[:,i])
        nothing
    end
    nothing
end

and then I want to take the minimum of the results. I could write the result of the first operation into an array and take the minimum, but there should be something better that could be done?

but there should be something better that could be done?

Yeah, but probably not easily within the framework GPUifyLoops offers. You need to implement a fast reduce yourself and that is rather tricky.
Sum along the columns and then use a sorting net to calculate the minimum.
other wise minimum(reduce(A, dim=1))

Note that https://github.com/JuliaDiffEq/DiffEqGPU.jl/issues/18 means that simple workarounds like
@inline DiffEqBase.ODE_DEFAULT_NORM(u::CuArray,t) = sqrt(real(sum(abs2.(u))) / length(u))

are still hitting bad fallbacks. The packagized version of what @vchuravy suggested is https://github.com/JuliaDiffEq/DiffEqGPU.jl/pull/23, but that seems to be failing because reduce with dims is missing from CuArrays.


Add support for LinkedIn with staging environment

It would be useful to be able to login using OAuth with LinkedIn instead of having to manually redirect the login flow through Google.

I'm able to view the LinkedIn cards on staging, but logging in with OAuth breaks with an error:

Error: The custom token corresponds to a different audience.

I have this working locally now