Improve argparser

Some improvements to the argument parser. This PR greatly improves the readability of the help command and introduces Enums for arguments.
Current help text

Changes

\n in help strings are now rendered correctly
Added argument groups and rearranged argument positions
Updated readme to match changes
Updated max_help_position to avoid unnecessary line breaks
Added choices as additional help for from, order and format
Added enums for from, order and format

--
I've split these changes into multiple commits to make reviewing them a bit easier.
On a related note, what is your opinion to squash merge for PRs? Although it might seem counterintuitive at first, I think it makes the git log much easier to read and the Github integration is better. The squash commit message includes the PR number which will be automatically be linked (for git blame).

@cdce8p Wow! Thank you. It's so easy to read!
I didn't know that you can do this hack with add_argument_group and self defined HelpFormatter. It's so cool.

what is your opinion to squash merge for PRs?

 That helps me in my review as well. Thanks.
I don't always squash merge approved PRs, but there is nothing wrong with squash merging this PR.

@cdce8p Wow! Thank you. It's so easy to read!
I didn't know that you can do this hack with add_argument_group and self defined HelpFormatter. It's so cool.

what is your opinion to squash merge for PRs?

 That helps me in my review as well. Thanks.
I don't always squash merge approved PRs, but there is nothing wrong with squash merging this PR.

I just remembered that I missed implementing a few small checks. I'll add them later today.
Those checks should cover specifying --no-license-path and --with-notice-file without also adding --with-license-file and --filter-code-page without --filter-strings.

I just remembered that I missed implementing a few small checks. I'll add them later today.
Those checks should cover specifying --no-license-path and --with-notice-file without also adding --with-license-file and --filter-code-page without --filter-strings.

Just pushed all remaining changes. In addition to extending args verification, I've also added some type hints for args. Those allow for much better autocompletions in IDEs. Lastly, I added a basic config for isort in setup.cfg. With that it's pretty easy to sort all imports and make them look nice.
After this PR is finished, I would like to propose moving from a single source file to a package structure. piplicenses.py is already quite large with almost 900 lines and it would only continue to grow in the future. The move could improve the readability. What do you think?

Just pushed all remaining changes. In addition to extending args verification, I've also added some type hints for args. Those allow for much better autocompletions in IDEs. Lastly, I added a basic config for isort in setup.cfg. With that it's pretty easy to sort all imports and make them look nice.
After this PR is finished, I would like to propose moving from a single source file to a package structure. piplicenses.py is already quite large with almost 900 lines and it would only continue to grow in the future. The move could improve the readability. What do you think?

After this PR is finished, I would like to propose moving from a single source file to a package structure. piplicenses.py is already quite large with almost 900 lines and it would only continue to grow in the future. The move could improve the readability. What do you think?

Yes, I agree with your proposal.
We have a lot of unit test code. It will also help when changing to a package structure. The test code you added is one of them.

After this PR is finished, I would like to propose moving from a single source file to a package structure. piplicenses.py is already quite large with almost 900 lines and it would only continue to grow in the future. The move could improve the readability. What do you think?

Yes, I agree with your proposal.
We have a lot of unit test code. It will also help when changing to a package structure. The test code you added is one of them.

That's also where squash merge has the most benefits.
Just to clarify, how the whole process would look like:

Squash merge every new PR into dev
For new release
2.1. Push new commit updating the version string
2.2. Create new PR, dev against master, this will run all CI on last time
2.3. If CI passes, normal merge PR into master
2.4. Since master now contains an additional merge commit that dev doesn't have, merge master back into dev. A normal push is enough
Go back to step 1

Some additional notes:

If you set the default branch to dev, all new PRs will target it by default
All changes, except version updates, should only be squash merged via a PR. This guarantees that all commits which show up in git blame reference a PR with additional information
If all changes have a PR, it also makes sure that all checks have passes and you don't accidentally committed something which causes a bug.
You could add the requirement that all checks must pass before a PR can be merged
Optional add that every PR needs at least one approving review
I believe it's also possible to set branch protections in Github, eg. for master don't allow push commits

--
Please let me know if you have any questions

That's also where squash merge has the most benefits.
Just to clarify, how the whole process would look like:

Squash merge every new PR into dev
For new release
2.1. Push new commit updating the version string
2.2. Create new PR, dev against master, this will run all CI on last time
2.3. If CI passes, normal merge PR into master
2.4. Since master now contains an additional merge commit that dev doesn't have, merge master back into dev. A normal push is enough
Go back to step 1

Some additional notes:

If you set the default branch to dev, all new PRs will target it by default
All changes, except version updates, should only be squash merged via a PR. This guarantees that all commits which show up in git blame reference a PR with additional information
If all changes have a PR, it also makes sure that all checks have passes and you don't accidentally committed something which causes a bug.
You could add the requirement that all checks must pass before a PR can be merged
Optional add that every PR needs at least one approving review
I believe it's also possible to set branch protections in Github, eg. for master don't allow push commits

--
Please let me know if you have any questions

To express my personal opinion:
I am against having a permanent dev branch. You are right, it is profitable for big projects to develop several new features in parallel. But I do not see this tool as a big project. It is better to keep the flow simple, as shown in the GitHub flow.
Next, let's talk about the release plan.
Improve argparser is a simple and beautiful advantage for many users.
On the other hand, in the process of embarking on the move to the package structure, it is possible that some of the most recent issues that have arrived can be resolved. (e. g. #82 #81 )
It seems to me that this effort is worth working on the base branch as you suggest. For example, create a base branch such as dev-4.0.0 or release-4.0.0. However, as I mentioned earlier, I am against a permanent branch. So, when the goal of the package structure is achieved and shipped, this development branch will end its lifecycle.
I'm waiting for your opinion.

To express my personal opinion:
I am against having a permanent dev branch. You are right, it is profitable for big projects to develop several new features in parallel. But I do not see this tool as a big project. It is better to keep the flow simple, as shown in the GitHub flow.
Next, let's talk about the release plan.
Improve argparser is a simple and beautiful advantage for many users.
On the other hand, in the process of embarking on the move to the package structure, it is possible that some of the most recent issues that have arrived can be resolved. (e. g. #82 #81 )
It seems to me that this effort is worth working on the base branch as you suggest. For example, create a base branch such as dev-4.0.0 or release-4.0.0. However, as I mentioned earlier, I am against a permanent branch. So, when the goal of the package structure is achieved and shipped, this development branch will end its lifecycle.
I'm waiting for your opinion.

That works for me. My intention behind the suggestion was simply to allow for a faster development pace. Implementing all changes would otherwise either require one huge PR, which would basically be unreviewable, or take months to complete. Having a separate branch thus allows for a faster turnaround once you approve a PR. I don't really have a name preference.
Regarding the issues you mentioned, it's definitely worth keeping them in mind. I also mentioned some other topics in my last PR #86 which I would like to work on. Targeting 4.0 would certainly make things easier, regarding breaking changes.
I plan on opening a new issue later to track all topics for an upcoming release. This would also be a good place to have general discussions if topics pop up.

That works for me. My intention behind the suggestion was simply to allow for a faster development pace. Implementing all changes would otherwise either require one huge PR, which would basically be unreviewable, or take months to complete. Having a separate branch thus allows for a faster turnaround once you approve a PR. I don't really have a name preference.
Regarding the issues you mentioned, it's definitely worth keeping them in mind. I also mentioned some other topics in my last PR #86 which I would like to work on. Targeting 4.0 would certainly make things easier, regarding breaking changes.
I plan on opening a new issue later to track all topics for an upcoming release. This would also be a good place to have general discussions if topics pop up.

While I was working on #88, I noticed some small issues. The last commit address these.

While I was working on #88, I noticed some small issues. The last commit address these.

@cdce8p Am I correct in understanding that your work on the improve-argparser branch is complete?
Then I will start the next task. Please let me know if there is any miscommunication.

Merge this P-R (with squash)
Version 3.3.0 will be shipped based on the master branch.
Create a remote branch dev-4.0.0 .

@cdce8p Am I correct in understanding that your work on the improve-argparser branch is complete?
Then I will start the next task. Please let me know if there is any miscommunication.

Merge this P-R (with squash)
Version 3.3.0 will be shipped based on the master branch.
Create a remote branch dev-4.0.0 .

@raimon49 That is correct. This PR is complete.

@raimon49 That is correct. This PR is complete.

@raimon49 Something went wrong with the squash merge. Did you do it with Github?
To be honest, I would like to see it fixed. I could post some commands that should do it if you agree.

@raimon49 Something went wrong with the squash merge. Did you do it with Github?
To be honest, I would like to see it fixed. I could post some commands that should do it if you agree.

Looks like it worked. Thanks!

Looks like it worked. Thanks!

@raimon49 Just one last thing: Do you want to release this as 3.3.1? That way we could guarantee this issue shouldn't pop up anywhere in the future.

@raimon49 Just one last thing: Do you want to release this as 3.3.1? That way we could guarantee this issue shouldn't pop up anywhere in the future.

I have released 3.3.0 to PyPI based on commit 4ace27c31978f42f77b979f90549277f94986ac3.
As far as I can see, there is no difference from the master branch that was forced to be pushed.
$ git diff origin/master 4ace27c31978f42f77b979f90549277f94986ac3

I have determined that users who have the package installed will not be affected. So I decide that there is no need to release 3.3.1.

I have released 3.3.0 to PyPI based on commit 4ace27c31978f42f77b979f90549277f94986ac3.
As far as I can see, there is no difference from the master branch that was forced to be pushed.
$ git diff origin/master 4ace27c31978f42f77b979f90549277f94986ac3

I have determined that users who have the package installed will not be affected. So I decide that there is no need to release 3.3.1.


Segmentation fault: 11 - Can't create more than two child pages

Issue Summary
When creating more than two child pages, python causes a segmentation fault. Restarting wagtail works, but as soon as you access the third child page in the admin interface, the segmentation fault is caused again. Deleting the problematic page in /django-admin also causes the same error.
Steps to Reproduce

Create new project wagtail start example
python managa.py migrate
python managa.py createsuperuser
python managa.py runserver
Go to admin page
Create two new child pages under Home

Creating a third child page causes the segmentation fault.
Technical details

Python version: 3.7.2 (on macOS Mojave, installed using brew)
Django version: 2.1.7
Wagtail version: 2.4

Using virtualenenv for all dependencies (including wagtail).

Hi @jnsdrtlf - thanks for the report. A segmentation fault would indicate something fundamentally failing within Python or a native extension such as Sqlite, so it's highly unlikely that this is something we can fix within Wagtail - however, I'll keep this issue open in case anyone can track it down further.

Hi @gasman, that's what I've already been afraid of. I was wondering if anyone can give me advice on how to debug this error.

@gasman, I was facing the same issue. I upgraded sqlite and it seems it is working now. I am using Python 3.7.2, and I regularly install dependencies using brew.

@lmiguelvargasf updating didn't help. However, I'm now using PostgreSQL which works without a problem. So it most certainly is an issue with sqlite.


App login issue

🐞 bug report
Describe the bug
After going through with the login and getting back into the app, nothing changes, still asks for login.

To Reproduce
Login, close web window and get back to app.

Expected behavior
After the process the app should see I'm logged in and let me operate changes like switching leagues.

Screenshots

🌍  Your Environment

OS:  Win10 64
Version:  0.8.10 portable
PoE:  e.g. Kakao 3.17 kor

Can you check for any errors in the application log?
(The log can be found here: %appdata%\poe-overlay\logs\main.log - feel free to attach the entire log file)

This does not remain in the log because it does not appear as an error.
Instead, I attach a video about the symptoms.
https://www.youtube.com/watch?v=AbwY7oLwoqs

Ah, i see. The video does make it clear now.
Poe overlay cf was never tested using the kakao login method. It looks like it simply doesn't work. I don't understand enough of the Korean language to look into this and fix it, sorry.

I've looked into some code of poe overlay cf related to logging in, and it seems I've only covered the official poe website and didn't add the domain names for other languages.
I'll apply a fix, so perhaps that solves the login issue. The fix should be available in the next release (v0.8.11)

Thank you for solving it.
I hope there will always be good things.


[BUG]  Navbar buttons not working on click

Is there any existing issue for this?

[X] I have searched the existing issues

Description
When clicking on nav items, the page doesn't scroll to that section, I want to add the navigation feature to smoothly scroll to the particular section when clicked.
Can I solve this issue under GSSOC '23?
Screenshots
No response
Any additional information?
No response
What browser are you seeing the problem on?
Chrome

Hey there some one is already working on this issue you can find another
You can refer to #74 for the same


fix: GitHub Star & Fork count

Fixes #327

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 76.842%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
219

Relevant Lines:
285

💛  - Coveralls

This is off topic, but it seems there is no cache for GitHub data.
There is $expires in the Config, but not used.
https://github.com/codeigniter4projects/website/blob/376eadaf29ff0567bfce7dbd9a9ace12e08b22e5/app/Config/GitHub.php#L14

Re $expires the whole API library is kind of odd. I tried to refactor it but really it should have been rewritten. It's not that big, so something like cache oversight is surprising. Maybe a good confidante for another take on it.


Add categories?

Do you feel the need to make media and collections categorizable using Zikula Categories? If so, could you please explain why?

It could be useful for enhanced integrations. Let's say you have another module (for example a webshop) and you want to include media based on the current category. But not urgent imo.

Content Module has built in category by page functionality which might be a use case here.

If there is a content plugin for a collection the category is not needed here.
But if we have a category by media and do have a block/content plugin we would be able to filter by category inside a collection. Nice possibility but I am still looking for its use case.
I think this is not needed for 1.x. Let us see if we find later a use case.


Moves SDWebImage dependency to a newer version with fewer crashes

I've tested this in circumstances that regularly led the older version to crash, and I haven't seen problems in the newer version.

Already fixed with another PR


Including french and brazilian CakePHP community sites

Why on cake3 and not master?

For some absence of mind probably! :smile:
I just forgot to send it too.


[Silicon Labs Platform Example] Virtual Devices - Light

Problem
<what's wrong or missing, please include any applicable:

expected behavior
actual behavior
steps to reproduce
system configuration
screenshots, images, logs, etc.

Proposed Solution
<suggested fix, suggested enhancement>

demo completed


Help understanding syntax error with non-terminal, ok with terminal

Here's a complete repro if the issue I'm trying to resolve. The output will be:
Error:Syntax error, expected: > ((1:10))
If all you do is comment out the line marked fails and uncomment the line marked succeeds, the expression parses successfully.
There's an ambuguity with "<" in that it might be part of an 'infix' expression or the beginning of a part of a 'componentAccess' expression, but I expected Irony to work it out since the componentAccess rule won't be met, whereas infix is. It seems to be able to do that when the infixOp is removed as a non-terminal and a pure terminal used instead.
Either this indicative of a bug or I just need help understanding this behavior, and whether or not its possible to have a grammar parsed successfully this way. I've played with grammar hints to no avail. This is a simplified version of the actual grammar I'm using, for demonstration purposes.
    class Program
    {
        static void Main(string[] args)
        {
            Try("abc < xyz");

            Console.ReadLine();
        }

        private static void Try(string expr)
        {
            var p = new Parser(new ExpressionGrammar());
            var result = p.Parse(expr);
            Console.WriteLine(result.Status + ":" + string.Join("\n", result.ParserMessages.Select(m => $"{m.Message} ({m.Location})")));
        }

        public class ExpressionGrammar : Grammar
        {
            public ExpressionGrammar() : base(false)
            {
                var formula = new NonTerminal("formula");
                var infix = new NonTerminal("infix");
                var infixOp = new NonTerminal("infixOp");
                var componentAccess = new NonTerminal("componentaccess");
                var identifier = new NonTerminal("identifier");

                Root = formula;

                formula.Rule =
                    identifier
                    | componentAccess
                    | infix
                    ;

                identifier.Rule = ToTerm("abc") | ToTerm("xyz");
                infixOp.Rule = ToTerm("<");
                // e.g.: abc<xyz>.abc
                componentAccess.Rule = formula + "<" + identifier + ">" + "." + identifier;

                // fails
                infix.Rule = formula + infixOp + formula;
                // succeeds (inlining infixOp)
                //infix.Rule = formula + ToTerm("<") + formula;
            }
        }
    }

For what it's worth I found a workaround, which is to have a special version of the infix rule that is specifically for the < case. The code that handles the parse tree just has to account for this possibility. Would really really love to know of a more correct way, or know if this is a bug.
    class Program
    {
        static void Main(string[] args)
        {
            Try("abc < xyz");
            Try("abc > xyz");
            Try("abc<xyz>.abc");
            Try("abc<xyz>.abc > abc<xyz>.abc");
            Try("abc<xyz>.abc < abc<xyz>.abc");

            Console.ReadLine();
        }

        private static void Try(string expr)
        {
            var p = new Parser(new ExpressionGrammar());
            var result = p.Parse(expr);
            Console.WriteLine(result.Status + ":" + string.Join("\n", result.ParserMessages.Select(m => $"{m.Message} ({m.Location})")));
        }

        public class ExpressionGrammar : Grammar
        {
            public ExpressionGrammar() : base(false)
            {
                var formula = new NonTerminal("formula");
                var infix = new NonTerminal("infix");
                var infixLt = new NonTerminal("infixLt");
                var infixOp = new NonTerminal("infixOp");
                var componentAccess = new NonTerminal("componentaccess");
                var identifier = new NonTerminal("identifier");

                Root = formula;

                formula.Rule =
                    identifier
                    | componentAccess
                    | infixLt // workaround
                    | infix
                    ;

                identifier.Rule = ToTerm("abc") | ToTerm("xyz");
                infixOp.Rule = ToTerm("<") | ToTerm(">");
                componentAccess.Rule = formula + "<" + identifier + ">" + "." + identifier;

                // 2 rules to work around it
                infixLt.Rule = formula + ToTerm("<") + formula;
                infix.Rule = formula + infixOp + formula;
            }
        }
    }

Small world ...

@sebastienros Didn't know you were here! HAI!

@InfinitiesLoop
This: https://github.com/sebastienros/fluid
and this: https://github.com/OrchardCMS/OrchardCore/blob/dev/src/OrchardCore.Modules/OrchardCore.Queries/Sql/SqlGrammar.cs


fix: redefine Nat.mod to match leanprover-community/lean

In mathlib, our definition of 0 : fin n exploits the fact that 0 % n = 0 is true by definition.
This is desirable because it means that ⟨0, zero_pos⟩ = ofNat 0 is true by definition.
On the assumption that Lean4 core doesn't care about the underlying definition, this adjusts it to match the one used in Lean3, with the expectation that this will make porting things about fin easier.
The proofs are loosely inspired by https://github.com/leanprover-community/lean3port/blob/master/Leanbin/Init/Data/Nat/Lemmas.lean#L1318-L1345; althought these used syntax that is not yet available.
See the Zulip discussion here.
This is not the only solution to the problem, but is certainly the one that requires the least change in mathlib.

FWIW, if we're messing around with defeqs then I think we should be explicit about it.
protected def modCore (x y : Nat) : Nat :=
  if 0 < y ∧ y ≤ x then
    Nat.modCore (x - y) y
  else
    x
decreasing_by apply div_rec_lemma; assumption

@[extern "lean_nat_mod"]
protected def mod : @& Nat → @& Nat → Nat
  | 0, _ => 0 -- special case so that `0 % n` reduces to `0`
  | x, y => Nat.modCore x y

Oh, I guess that's a much smaller fix!

Alternatively, we could also define OfNat (Fin (no_index (n+1))) (nat_lit 0) and OfNat (Fin (no_index (n+1))) (no_index (i+1)) separately.

Yes, that is the approach I describe in this message

Superceded by #2020


cx:chartSpace is Missing

cx:chartSpace is Missing
When trying to create a histogram chart (an example of an extended chart) in office the cx:chartSpace xml element is required. A C# class for this exists in an uncompiled test which is named DocumentFormat.OpenXml.Office2016.Drawing.ChartDrawing.ChartSpace. For some reason this class isn't included in the sdk.
Information

.NET Framework
DocumentFormat.OpenXml Version: 2.10

Repro
DrawingsPart drawingsPart = ...;
var extendedChartPart = drawingsPart.AddNewPart<ExtendedChartPart>();

\\I want the below line to work, but the ChartSpace element doesn't exist. Also it would be nice to be able to call extendedChartPart.ChartSpace = rather than extendedChartPart.RootElement =
extendedChartPart.RootElement = new DocumentFormat.OpenXml.Office2016.Drawing.ChartDrawing.ChartSpace();

Observed
It doesn't compile and there doesn't seem to be an easy workaround

Thanks @ScottHaney for the report. Any of the new Office 2016 related schema entries were added as best effort knowing that there were some gaps, but figured getting what we had out would be better than postponing until we had full support. Seems like you found one of the gaps.
@tomjebo This looks like it may be related to the issue you are already investigating.

@twosouthwick - I don't mind doing some updates to the code to help with the missing features. I am in need of them to get the histogram plot working. How do I update the code to add the missing items? For instance in the Chart class with SchemaAttr value 80 there is an Id attribute missing. If I just add a property named Id to Chart with an Index attribute it won't serialize out the Id value to the xml when the document is saved. Where in the code base do you go to make changes to the generated files and have those changes show up in the xml produced?

Unfortunately, the source has a code generator we can't make public (hopefully someday). I've been working towards removing those magic numbers, but it's a slow process... right now, you can't just add things out. @tomjebo has found the issue and is looking to submit a fix.

@twsouthwick - Thanks for the response! I found that I was able to use the public method OpenXmlElement.SetAttribute to add attributes that would serialize out. Also in the meantime I added a custom implementation of the ExtendedChartPart which allows me to hack my way into creating histograms finally.

FYI there's a fix for this in #687 that @tomjebo is working through

@ScottHaney #687 was merged, can you please retest and let me know if you're not able to get the histogram chart working?


UTCTiming is ignored when using SegmentTimeline even when autoCorrectDrift is off

Have you read the FAQ and checked for duplicate open issues?
Yes
What version of Shaka Player are you using?
v2.5.9
Can you reproduce the issue with our latest release version?
Yes
Can you reproduce the issue with the latest code from master?
Yes
Are you using the demo app or your own custom app?
The demo app
If custom app, can you reproduce the issue using our demo app?
N/A
What browser and OS are you using?
Chrome, macOS Catalina
For embedded devices (smart TVs, etc.), what model and firmware version are you using?
N/A
What are the manifest and license server URIs?

https://live.unified-streaming.com/scte35/scte35.isml/.mpd
What did you do?

Open the demo page
Uncheck Auto-Correct DASH Drift config
Start playing the manifest url above on the demo player.

What did you expect to happen?
It will call a request to https://time.akamai.com/?iso
What actually happened?

No UTCTiming request is called

I created a pull request here.
https://github.com/google/shaka-player/pull/2412

Thank you for the report and the PR!  After having analyzed the code a bit, I think you are correct that this is a bug.


Switch to using analysis framework natively with fact export

As noted in golangci/golangci-lint#4158, this linter is incompatible with the analysis framework because it strongly relies on state across packages, and this state is not exported using the facts API.
The analyzer shim present in golangci-lint seems insufficient in dealing with this limitation. This entire module would likely need to be re-written as an analyzer.

Ah makes sense, I'll take a look.

Thanks for taking a look. As far as in-memory caching goes, my recommendation is the following strategy:

store a cache which maps packages to their own caches; access to this cache must be synchronized since drivers may execute independent packages in parallel
in each package, only perform writes to the cache of the same package and reads from other packages; these caches do not need to be synchronized because no well-behaved driver will perform concurrent passes over dependent packages or multiple passes against the same package
at the beginning of an analysis pass, delete and re-initialize the cache for the package in question to avoid any risk of staleness

Another thing to worry about with using the facts API: once you add analyzer facts, drivers will start running the analyzer against the dependency closure rather than just the packages in question. Sadly, this will drop performance of the singlechecker considerably. There's not really a workaround aside from adding some config option to exclude reporting and fact gathering on certain packages.

What caching are you referring to? I've used the facts API before and looking at how it would be used here, it seems sufficient for this use case.

It would be acceptable to use no in memory caching and purely rely on the facts API.


Multidomain config as a default in Apache

Current Apache configuration has this kind of domain-to-folder mapping:
Level 1:
{FOLDER}.<IP_ADDRESS>.nip.io -> /var/www/html/{FOLDER}
Level 2:
{SUB_FOLDER}.{FOLDER}.<IP_ADDRESS>.nip.io -> /var/www/html/{FOLDER}/{SUB_FOLDER}
The idea is to allow multi-domain setups enabled and configured by default in this way:
Level 1:
{FOLDER}.<IP_ADDRESS>.nip.io -> /var/www/html/{FOLDER}
Level 2:
{WEBSITE}.{FOLDER}.<IP_ADDRESS>.nip.io -> /var/www/html/{FOLDER}
This configuration is more beneficial for day-to-day issues reproduction.
Together with this, we can include magento-vars.php as default, like in a Cloud environment (directly in php.ini)

Released in 3.0.1


fix Utf8 encoder

Added test case for currently failing scenarios.
Fixes #2386

Updated the algorithm for decoder. /cc @boingoing @jianchun

Added some more tests.

@dotnet-bot test Ubuntu ubuntu_linux_debug_static please


Move to using release bundles

Rather than using multiple remote downloads.
Resolves #617
Resolves #615

Coverage remained the same at 74.597% when pulling 76824f80a333b3f79212c2f8f9c9bdc37bad39c5 on use-release-tools into a24f1313ce4725126ac8dc179d34a22cd3a6aa9f on master.


Firewall log download returns an empty file of 0Kb

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[x] I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md
[x] I am convinced that my issue is new after having checked both open and closed issues at https://github.com/opnsense/core/issues?q=is%3Aissue

Describe the bug
Describe the bug
OPNsense Version: 23.1.11
RAM 16GB
Disk size: 115GB
CPU size: Intel(R) Core(TM) i7-3612QE CPU @ 2.10GHz (4 cores, 8 threads)
Description:
In situations where the OPNsense box is in high traffic network generating many logs (Approx. 300MB daily; while trying to download firewall log files via the web GUI, an empty file of size 0kb is returned. Since the circular log files were replaced by the number of preservation days for logs to be stored, it becomes difficult to retrieve log files via the web GUI because of the size of the individual daily logs.
After editing different parameters in the backend and this issue not being resolved, there might be a hardcoded limit on the file size of logs that can be downloaded or retrieved via the web interface.
Steps to reproduce:

Go to Firewall.
Click on “Log Files.”
Then click on “Plain view.”
Scroll down to the end of the page and click the cloud download widget and the bottom right.
The download process takes a while to complete, but an empty file is returned once completed.

Expected behavior:
The expected behavior is that the log files are downloaded successfully via the web GUI.
Alternative considered / Remediation.
Edited php.ini file in the (root/usr/local/opnsense/service/templates/OPNsense/WebGui) directory. Increasing the memory limit to 4GB & 8GB, respectively, increasing the upload_max_filesize to 2GB, increasing the post_max_size to 2GB, and changing the execution time and the max_input _time as “indefinite.” However, the same behavior was noticed despite changes to those php download parameters.

If I'm not mistaken, the filter applies on the download too. I'm not sure we can change this easily to stream the data to the browser, which would prevent memory exhaustion.

Provided you have the memory resources. What are the options to prevent this behavior. Is there a hard coded limit in the source that can handle this exception ?

I would expect it crashes on a php memory limit, but it could also be crashing due to insufficient memory in the python script being passed through configd. You likely have to investigate more in order to know where it stops. Short term there's not much I can offer, I can label this ticket "feature request" and put it on the stack for future improvements. There is a way to offer this type of functionality, but it will easily cost a couple of working days to mature and develop.
For us it's currently not a super high priority, there are other means to export large sets as well.

requires https://github.com/opnsense/core/commit/cee33f5393154bb459150fd4639a6a8cd77456ee in combination with https://github.com/opnsense/core/commit/42fd27df777ee85a0e42767a814bed5e93659f0c and https://github.com/opnsense/core/commit/28b99101d45779c7f8e75e9c6817c942e2d0dfee


[bug]: Unhandled exception: Bad state: The "HOME" environment variable is not set. This package (and POSIX) requires that HOME be set.

Describe the bug
Hello, I've been unable to get the stacked_cli to work on Windows 10. Whenever I run the cli with stacked ... it gives an error:
Unhandled exception:
Bad state: The "HOME" environment variable is not set. This package (and POSIX) requires that HOME be set.

I seen this and this, but as you can see below trying the local version didn't help and my Dart version shouldn't be a problem: Dart SDK version: 3.2.0 (stable) (Tue Nov 14 18:26:59 2023 +0000) on "windows_x64"
Have any ideas what is causing the issue?
To reproduce
Following the docs, I ran the command to activate the cli. However, after the error I continued like so:
PS C:\Users\Acer> dart pub global activate stacked_cli
Package stacked_cli is currently active at version 1.12.1.
The package stacked_cli is already activated at newest available version.
To recompile executables, first run `dart pub global deactivate stacked_cli`.
Installed executable stacked.
Activated stacked_cli 1.12.1.
PS C:\Users\Acer> stacked
Unhandled exception:
Bad state: The "HOME" environment variable is not set. This package (and POSIX) requires that HOME be set.#0      _getDirectory (package:xdg_directories/xdg_directories.dart:120:5)
#1      _directoryFromEnvironmentWithFallback (package:xdg_directories/xdg_directories.dart:109:12)
#2      configHome (package:xdg_directories/xdg_directories.dart:152:5)
#3      PathService.configHome (package:stacked_cli/src/services/path_service.dart:41:35)
#4      PosthogService.init (package:stacked_cli/src/services/posthog_service.dart:42:31)
#5      setupLocator (package:stacked_cli/src/locator.dart:27:24)
#6      main (file:///C:/Users/Acer/AppData/Local/Pub/Cache/hosted/pub.dev/stacked_cli-1.12.1/bin/stacked.dart:18:9)
#7      _delayEntrypointInvocation.<anonymous closure> (dart:isolate-patch/isolate_patch.dart:295:33)
#8      _RawReceivePort._handleMessage (dart:isolate-patch/isolate_patch.dart:184:12)
Unhandled exception:
Bad state: The "HOME" environment variable is not set. This package (and POSIX) requires that HOME be set.#0      _getDirectory (package:xdg_directories/xdg_directories.dart:120:5)
#1      _directoryFromEnvironmentWithFallback (package:xdg_directories/xdg_directories.dart:109:12)
#2      configHome (package:xdg_directories/xdg_directories.dart:152:5)
#3      PathService.configHome (package:stacked_cli/src/services/path_service.dart:41:35)
#4      PosthogService.init (package:stacked_cli/src/services/posthog_service.dart:42:31)
#5      setupLocator (package:stacked_cli/src/locator.dart:27:24)
#6      main (file:///C:/Users/Acer/AppData/Local/Pub/Cache/hosted/pub.dev/stacked_cli-1.12.1/bin/stacked.dart:18:9)
#7      _delayEntrypointInvocation.<anonymous closure> (dart:isolate-patch/isolate_patch.dart:295:33)
#8      _RawReceivePort._handleMessage (dart:isolate-patch/isolate_patch.dart:184:12)

Then I downloaded the source from the stacked_cli repo and tried to run it:
PS C:\Users\Acer\Downloads\cli-main\cli-main> dart pub global activate --source path .
Resolving dependencies... (2.9s)
+ _fe_analyzer_shared 65.0.0
+ analyzer 6.3.0
+ ansicolor 2.0.2
+ args 2.4.2
+ async 2.11.0
+ boolean_selector 2.1.1
+ build 2.4.1
+ build_config 1.1.1
+ build_daemon 4.0.1
+ build_resolvers 2.4.1
+ build_runner 2.4.7
+ build_runner_core 7.2.11
+ built_collection 5.1.1
+ built_value 8.8.0
+ checked_yaml 2.0.3
+ code_builder 4.8.0
+ collection 1.18.0
+ convert 3.1.1
+ coverage 1.7.1
+ crypto 3.0.3
+ dart_style 2.3.4
+ file 7.0.0
+ fixnum 1.1.0
+ freezed 2.4.5
+ freezed_annotation 2.4.1
+ frontend_server_client 3.2.0
+ functional_data 1.1.1
+ get_it 7.6.4
+ glob 2.1.2
+ graphs 2.3.1
+ hive 2.2.3
+ http 1.1.2
+ http_multi_server 3.2.1
+ http_parser 4.0.2
+ io 1.0.4
+ js 0.6.7
+ json2yaml 3.0.1
+ json_annotation 4.8.1
+ json_serializable 6.7.1
+ lints 3.0.0
+ logging 1.2.0
+ matcher 0.12.16
+ meta 1.11.0
+ mime 1.0.4
+ mockito 5.4.3
+ mustache_template 2.0.0
+ node_preamble 2.0.2
+ package_config 2.1.0
+ path 1.8.3
+ plain_optional 1.0.0
+ platform 3.1.3
+ pool 1.5.1
+ process 5.0.1
+ pub_semver 2.1.4
+ pub_updater 0.4.0
+ pubspec_parse 1.2.3
+ pubspec_yaml 3.1.0
+ recase 4.1.0
+ shelf 1.4.1
+ shelf_packages_handler 3.0.2
+ shelf_static 1.1.2
+ shelf_web_socket 1.0.4
+ source_gen 1.4.0
+ source_helper 1.3.4
+ source_map_stack_trace 2.1.1
+ source_maps 0.10.12
+ source_span 1.10.0
+ stack_trace 1.11.1
+ stream_transform 2.1.0
+ string_scanner 1.2.0
+ sum_types 0.3.5
+ term_glyph 1.2.1
+ test 1.24.9
+ test_api 0.6.1
+ test_core 0.5.9
+ timing 1.0.1
+ typed_data 1.3.2
+ vm_service 13.0.0
+ watcher 1.1.0
+ web 0.4.0
+ web_socket_channel 2.4.0
+ webkit_inspection_protocol 1.2.1
+ xdg_directories 1.0.3
+ yaml 3.1.2
Changed 85 dependencies!
Package stacked_cli is currently active at version 1.12.1.
Installed executable stacked.
Activated stacked_cli 1.12.1 at path "C:\Users\Acer\Downloads\cli-main\cli-main".
PS C:\Users\Acer\Downloads\cli-main\cli-main> stacked
Building package executable... (6.4s)
Built stacked_cli:stacked.
Unhandled exception:
Bad state: The "HOME" environment variable is not set. This package (and POSIX) requires that HOME be set.
#0      _getDirectory (package:xdg_directories/xdg_directories.dart:120:5)
#1      _directoryFromEnvironmentWithFallback (package:xdg_directories/xdg_directories.dart:109:12)
#2      configHome (package:xdg_directories/xdg_directories.dart:152:5)
#3      PathService.configHome (package:stacked_cli/src/services/path_service.dart:41:35)
#4      PosthogService.init (package:stacked_cli/src/services/posthog_service.dart:42:31)
#5      setupLocator (package:stacked_cli/src/locator.dart:27:24)
#6      main (file:///C:/Users/Acer/Downloads/cli-main/cli-main/bin/stacked.dart:18:9)
#7      _delayEntrypointInvocation.<anonymous closure> (dart:isolate-patch/isolate_patch.dart:295:33)
#8      _RawReceivePort._handleMessage (dart:isolate-patch/isolate_patch.dart:184:12)

Expected behavior
To be able to run the cli. :)
Screenshots
No response
Additional Context
No response

Hey,
Is it possible for you to set a HOME environment and see if it works?
I don't want to delete my home env. Mine is /Users/danemackier on my mac.

I get the same error.
OS:
Win 11
Dart version:
Dart SDK version: 3.3.0-91.0.dev (dev) (Thu Nov 2 09:10:03 2023 -0700) on "windows_x64"
Flutter version:
Flutter 3.17.0-1.0.pre.3 • channel beta • https://github.com/flutter/flutter.git
Framework • revision 12b47270b7 (5 days ago) • 2023-11-27 17:21:11 -0600
Engine • revision dff66925dc
Tools • Dart 3.3.0 (build 3.3.0-91.0.dev) • DevTools 2.29.0

Stacked error:
PS D:\dev\workspace\flutter> stacked
Unhandled exception:
Bad state: The "HOME" environment variable is not set. This package (and POSIX) requires that HOME be set.
#0      _getDirectory (package:xdg_directories/xdg_directories.dart:120:5)
#1      _directoryFromEnvironmentWithFallback (package:xdg_directories/xdg_directories.dart:109:12)
#2      configHome (package:xdg_directories/xdg_directories.dart:152:5)
#3      PathService.configHome (package:stacked_cli/src/services/path_service.dart:41:35)
#4      PosthogService.init (package:stacked_cli/src/services/posthog_service.dart:42:31)
#5      setupLocator (package:stacked_cli/src/locator.dart:27:24)
#6      main (file:///C:/Users/oppah/AppData/Local/Pub/Cache/hosted/pub.dev/stacked_cli-1.12.3/bin/stacked.dart:18:9)
#7      _delayEntrypointInvocation.<anonymous closure> (dart:isolate-patch/isolate_patch.dart:295:33)
#8      _RawReceivePort._handleMessage (dart:isolate-patch/isolate_patch.dart:184:12)
Unhandled exception:
Bad state: The "HOME" environment variable is not set. This package (and POSIX) requires that HOME be set.
#0      _getDirectory (package:xdg_directories/xdg_directories.dart:120:5)
#1      _directoryFromEnvironmentWithFallback (package:xdg_directories/xdg_directories.dart:109:12)
#2      configHome (package:xdg_directories/xdg_directories.dart:152:5)
#3      PathService.configHome (package:stacked_cli/src/services/path_service.dart:41:35)
#4      PosthogService.init (package:stacked_cli/src/services/posthog_service.dart:42:31)
#5      setupLocator (package:stacked_cli/src/locator.dart:27:24)
#6      main (file:///C:/Users/oppah/AppData/Local/Pub/Cache/hosted/pub.dev/stacked_cli-1.12.3/bin/stacked.dart:18:9)
#7      _delayEntrypointInvocation.<anonymous closure> (dart:isolate-patch/isolate_patch.dart:295:33)
#8      _RawReceivePort._handleMessage (dart:isolate-patch/isolate_patch.dart:184:12)

I remember it working like a week or two ago.
Where should this "HOME" environment variable point to? I am confused, why do I need it now?

@oppahansi it's a requirement for another package.
We store global configuration in the directory that the operating system recommends. That directory is usually relative the the HOME directory, so when we look for the Document directory that path is built from the Home directory.

We can definitely have a better message for that issue.

can you help me, how to set HOME enviroment variable. i quite on this.
Thank you.

Set a "C:\Users\xxxx" (xxxx is u user name) work to me, but, u can need create u project at same folder, u need open a Terminal at "C:\Users\xxxx" folder and create u project here

Create a HOME environment variable and set its value to C:\Users\your_username. That's all.
This worked for me...


Allow for backup of CRD resources

I tried adding cert-manager CRD resources to the backup-list:
- name: GLOBALRESOURCES
              value: "namespace storageclass pv clusterrolebinding clusterrole clusterissuer"
            - name: RESOURCETYPES
              value: "ingress deployment configmap svc rc ds networkpolicy statefulset\
                \ pvc cronjob secret sealedsecret rolebinding role certificate issuer"

but still it won't backup certificates, issuers or cluster-issuers.
Should it work OOTB?

nvm
PEBKAC


.

update_comment@350 aka 20200110.3 vs master ewma over 40 builds from 129 to 346

ID of first self comment None

update_comment@351 aka 20200110.4 vs master ewma over 40 builds from 129 to 346

ID of first self comment 573228086


'Changed Records' events are missing before 11/1/2022

https://pulibrary.slack.com/archives/C031U4XCUH2/p1673012458251949
I was reviewing the bibdata events page and deleted three Full Partner Recap Records from 2021. I scrolled down to review the events and decide what dates and events to clear, as part of a maintenance ticket.
I noticed that the event ‘Change Records’ which is the one that the incremental publishing job creates daily exists in the events page after 2022-11-01 which is odd.
Based on my calendar a full reindex ran on 10/31/2022 .

Is there a history table where we can see any kind of changes?
Was there any code update in bibdata before 11/1/2022 that could have caused this bug?
Create in bibdata-staging changed records and partner events and see if you can replicate this.
Did someone delete Changed records before 11/1/2022?

Could it be this cron job? https://github.com/pulibrary/bibdata/blob/main/config/schedule.rb#L42-L44
This part seems to remove all incremental Events from the past two months: https://github.com/pulibrary/bibdata/blob/main/lib/tasks/marc_liberation.rake#L5
If the cron job was last run on Sunday, Jan. 1, that would explain why everything before November is missing.

@sandbergja you are right! I totally forgot about this job and the date that these events were cleared made me think that something else was going on. I'm closing this as resolved. Thanks !


Show only whitelisted attributes on technology form

This is note quite what was asked for in #1292, because it hides COP – rather than disables it – for technologies on which it is not supported. I think that's an acceptable compromise in the name of simplicity, since you can still tell that efficiency is 100% by comparing the input and output capacities. If you feel strongly that it should be visible, let me know. I intend to revisit this with "Input Validation".
I have also hidden demand, volume, and capacity attributes in some cases. For example, technologies which belong to heat buffers should not allow the visitor to set a demand or volume (those are set on the buffer). Other technologies, such as Households, have their load set by demand and I felt that also showing a capacity was confusing (and the two attributes would conflict).

Removed demand and volume from all heating technologies.

Removed COP from all technologies except:

Heat pumps
Hybrid heat pumps
Power-to-heat

From base loads, removed capacity and volume.

From P2G, removed demand.

From the neighbourhood battery, removed demand.

From power-to-heat, removed demand.

P2P batteries, removed demand.

From solar PV, removed demand and volume.

From the electric vehicle, removed demand.

@ChaelKruip I'd appreciate a critical eye over these changes; I don't think they'll be controversial, but just in case...


[SPARK-47289][SQL] Allow extensions to log extended information in explain plan

What changes were proposed in this pull request?
This addresses SPARK-47289 and adds a new section in explain plan where Spark extensions can add additional information for end users. The section is included in the output only if the relevant configuration is enabled and if the extension actually adds some new information
Why are the changes needed?
Extensions to Spark can add their own planning rules and sometimes may need to add additional information about how the plan was generated. This is useful for end users in determining if the extensions rules are working as intended.
Does this PR introduce any user-facing change?
This PR increases the information logged in the UI in the query plan. The attached screenshot shows output from an extension which provides some of its own operations but does not support some operations.

How was this patch tested?
Unit test and manual testing
Was this patch authored or co-authored using generative AI tooling?
No

@dongjoon-hyun github actions are enabled in my repository and the branch is based on the latest commit in master. In my repo the ci checks are shown as passing.

I've never encountered this situation in github before. Any suggestions?

Rebased on latest master and tests are passing now

CI test failure appears to be unrelated. Not re-running, because this PR will change.

@cloud-fan I've update this PR. Can you please take a look?
I also moved the unit test to a more appropriate test suite.
(Sorry it took a while; I was making sure that with this approach extensions would be able to publish their extended info and also be able to build and work with older versions of Spark which do not have the trait.)

BTW, connect-client-jvm seems to start to fail in CIs.
Do connect-client-jvm module mima check ...
finish connect-client-jvm module mima check ...
ERROR: Comparing Client jar: /__w/spark/spark/connector/connect/client/jvm/target/scala-2.13/spark-connect-client-jvm_2.13-4.0.0-SNAPSHOT.jar and Sql jar: /__w/spark/spark/sql/core/target/scala-2.13/spark-sql_2.13-4.0.0-SNAPSHOT.jar
problems with Sql module:
interface org.apache.spark.sql.ExtendedExplainGenerator does not have a correspondent in client version
Exceptions to binary compatibility can be added in 'CheckConnectJvmClientCompatibility#checkMiMaCompatibilityWithSqlModule'
connect-client-jvm module mima check failed.

@dongjoon-hyun thank you for pointing the ci failure out. I've addressed it by excluding the newly added trait from the compatibility check (it is not needed by the client).

thanks, merging to master!

Thank you @cloud-fan @dongjoon-hyun  for the review!

Thank you, @parthchandra and @cloud-fan .
This is much better. :)


WeCrossProxy/proxy.go: modify the variable name of hasInfo to isLocke…

…d on line 280 and 282.
The variable name of hasInfo gives the misleading impression that the return value of fuction getLockedContract is a hash value. In addition, the fuction of getLockedContract calls return value variable names remain the same.
Signed-off-by: yjwxfq<EMAIL_ADDRESS>

LGTM


Update canvas helm chart brandable_css

No longer need to move the .dist contents as everything is going onto S3 now

Just kidding, we still need to move the .dist directory. Closing PR


[Disco] Support ShapeTuple in Disco Protocol

ShapeTuple is an essential item used in Relax stack as the runtime representation of shapes. It is also used as a current workaround to represent integers (1-d shape) given standalone non-constant integers are currently absent in Relax.
This PR introduces formal support for ShapeTuple in Disco's communication protocol based on the recent enhancement of TVM RPC system to support TVM Objects: https://github.com/apache/tvm/pull/15631.

CC @jinhongyii @LeshengJin @tqchen


bump nodejs client to 2.0.0

Check List

[ ] New functionality includes testing.

[x] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Can we update this https://github.com/opensearch-project/opensearch-js/blob/main/COMPATIBILITY.md in this PR?


add schema for BUS_TRIP_UPDATES configuation

This adds configuration details and testing for RT Bus Trip Update files.
Asana Task: https://app.asana.com/0/1202413225616901/1202518897679077

this LGTM


Improve backend memory usage

Summary
The problem
I've been noticing lately that our tomcat server has been shutting down unexpectedly due to an OutOfMemoryError. So I set up memory profiling with a program called yourkit and I noticed that every time a request is made to a servlet which does any db operations (a.k.a DBServlet), the memory usage of the application would increase by like 300kb and this memory would not get freed up over time/by garbage collection. I haven't been able to directly reproduce the OutOfMemoryError but I assume it is related to this issue.
The solution
I checked out the type of java objects that were making up the majority of this extra memory usage and it was mostly HashMaps used under the hood by MongoDB (see screenshot below). This led me to re-read the mongodb docs where I found that the normal usage is to create only one MongoClient object instance to be reused by all servlets in your application. This is different from what we're currently doing which is creating a unique MongoClient object for each servlet. I followed a guide on how to share a single reference to a MongoClient object accross all servlets and I made the necessary changes.
The version of java we have running on our server is 1.7 so I configured our maven build to use the same version. Also, to be able to use annotations in our backend code I needed to bump up the version of the javax.servlet library from 2.5 to 3.1.0
After this, I found that the problem of memory being used then not freed up on every db request went away. If we ever end up getting an OutOfMemoryError again, I've configured the JVM to do a memory dump when that occurs, in which case we can take a look at what's taking up the most memory in the application.

Java memory profiling
If you want to try out the memory profiling yourself, you can profile our server remotely by installing yourkit and clicking Connect to remote application then for the hostname put conucourseplanner.online.

tbh I was playing around a lot with the java version on the server when trying to get memory profiling to work. We can probably upgrade to java 8 easily so we should probably do it at some point.

according to this thread (from March 2017), 4.7% of all servers now use Java 7


Prepare some vocab YAML files

[ ] @rhysrevans3 provide a YAML template for Ag to populate with CMIP5 and CMIP6 vocabs
[ ] @agstephens: populate those YAML files

I've just checked the code and it looks like it's not yet using YAML but I believe it should look something like this for the cache creation where each workflow is a different namespace:
workflows:
  cmip6:
    concept_schemes:
      - name: source_id
        prefLabel: source_id
        altLabel: Source ID
        location: ../../CMIP6_source_id.json
      - name: source_type
        prefLabel: source_type
        altLabel: Source Type
        location: ../../CMIP6_source_type.json

and this for the general vocab:
concept_schemes:
  - name: model
    prefLabel: model
    description: A model is a ...
    sub_schemes:
      - name: source_id
        namespace: 'http://test.org/cmip6/'
      - name: model
        namespace: 'http://test.org/cmip5/'

Which URI formula to use:

cmip6:UKESM1-0-MMh
http://vocab.ceda.ac.uk/cmip6/UKESM1-0-MMh

Yaml example for CMIP6:
https://github.com/cedadev/stac-vocab/blob/85e62983fcb8ceac6b4e105e084465ec16455ac8/stac_vocab/vocab_descriptions/external_vocabs/cmip6.yml

We will come back to this when the time is right, so closing.


Ability to set default branch?

Maybe I missed it, but is there a way to have gitlabform set the default branch for a repository?
We created new "main"/"develop" branches in many of our repositories, but the default branch still points to "master".  Being able to update the default branch from gitlabform would make it considerably easier to automate these changes.

This one is working for me:
project_settings:
  default_branch: production

Odd, I'm not seeing that setting anywhere in the code or the example config.

Ahh, disregard.  I see it falls under project_settings which has a link to the available values.  I'll open a quick PR to add default_branch to the example config as it's a setting some people might be looking for.

I assume that this is fixed by #183. Please reopen if needed. :)


Train SqueezeNet from scratch ?

Hello guys, I want to train SqueezeNet  from scratch with my own data instead of using ImageNet dataset. I would like to ask is that possible or not. I am new to Caffe, I already know Tensorflow but seems like there is no SqueezeNet Tensorflow version support.

Try this:
https://github.com/mrgloom/kaggle-dogs-vs-cats-solution/tree/master/learning_from_scratch/SqeezeNet_v1.1

@mrgloom I have the same question, but specifically for building for iOS. Are there any other resources you know of that could help with this?

Not sure of caffe2 is compatible with caffe models, in my repo you can find all .prototxt that needed to train model. Also I was using caffe+DIGITS to train model, DIGITS simplifies DB creation, if we are talking about classification.


Profiler takes redundant 'quiet' option

I notice that there is both a verbose  and a quiet  kwarg when instantiating the profiler. I think if verbose  is False, quiet should be False.

verbose controls printing of status messages, quiet allow vrt rebuilding without showing any printout.

quiet takes the value of verbose now.

Make that: quiet takes the value of ~verbose.


Add Findstemmer.cmake to find lib stemmer

Lib stemmer can be installed through setup script. See #10984.
This pr allows cmake to find installed lib stemmer.

@kgpai, @assignUser, could you take a look? The CI failure is not related.


Matroska, handling of files with Tracks element after Cluster elements

Will also fix https://github.com/MediaArea/MediaConch_SourceCode/issues/612.

Hi Jerome, I'm not entirely sure how to go about compiling this so not sure if I can test, sorry!

No stress, it will be in the snapshot tomorrow.

Lovely, thanks Jerome.


(fleet) make repositories stateless

This PR makes the repositories struct stateless, following the pattern set in repository. This ensures we always load the underlying state on disk, making sure we can't get de-synced with it.

/merge

/status


rdrDayDisabled first class and last class

With css it is not possible to get the first element and the last one by class, so it would be useful to give these elements the appropriate classes.

Together with this solution, you can give classes in groups of days so that you can round off corners:


go client Compile error

etcd 2.3.6 client compile error
..\..\..\golocal\src\github.com\coreos\etcd\client\keys.generated.go:136: z.EncSendContainerState undefined (type codec.genHelperEncoder has no field or method EncSendContainerState)
..\..\..\golocal\src\github.com\coreos\etcd\client\keys.generated.go:136: too many errors

go version 1.6.2

@evildecay it seems the code is building against an old version of the github.com/ugorji/go/codec package, try updating or using etcd's vendored packages?

@heyitsanthony update codec from github.com/ugorji/go to compile success! thanks! etcd v3.6.2 release source package compile error, please update release version source.


cargo lambda build failed due to missing zig

cargo lambda build
   Compiling tokio v1.36.0
   Compiling futures-util v0.3.30
   Compiling tracing v0.1.40
   Compiling ring v0.17.8
   Compiling serde v1.0.196
   Compiling pin-project v1.1.4
   Compiling darling_macro v0.20.6
   Compiling async-stream v0.3.5
The following warnings were emitted during compilation:

warning<EMAIL_ADDRESS>Error:   × Failed to find zig
warning<EMAIL_ADDRESS>
warning<EMAIL_ADDRESS>  Was this error unexpected?
warning<EMAIL_ADDRESS>  Open an issue in https://github.com/cargo-lambda/cargo-lambda/issues
warning<EMAIL_ADDRESS>

error: failed to run custom build command for `ring v0.17.8`

  cargo:warning=Error:   × Failed to find zig

  cargo:warning=

  cargo:warning=  Was this error unexpected?

  cargo:warning=  Open an issue in https://github.com/cargo-lambda/cargo-lambda/issues

  cargo:warning=

  exit status: 1

  --- stderr
...

warning: build failed, waiting for other jobs to finish...

Hi @calavera  Thanks for the help.
It is installed via brew on linux.

Same issue. I installed with pip3.


Download option

This issue relates to the download option for query results, which is not working.

Actually, this works for certain features, but not all.

Actually, this works for certain features, but not all.

This option still does not work reliably, neither does the copy-to-notepad option. I thought this was an issue on my end, but that does not appear to be the case.

This is a Swagger UI issue.


BU Support: NPC standing animation

NPC standing animation isn't working as expected.
Expected:

Actual:

Per Cirras this is an effect packet that needs to be implemented. This packet is not sent by vanilla eoserv.
(EFFECT, AGREE)
{
    char x
    char y
    short effect_id
}


Theia electron installer on windows has a hard coded default path

Bug Description:

Download the installer from here https://download.eclipse.org/theia/1.9.1/windows/
Install
See the default path

Steps to Reproduce:

Additional Information

Operating System:
Theia Version:

@kittaakos I transferred the issue to the theia-example repository, if it’s incorrect please let me know 👍

Thanks for reporting this, that needs to be fixed for the beta.

I confirmed the issue on Windows. However, it only happens when I chose the option to install for all users. When installing only for me, the path is correctly customized based on my machine.

This is taken from the electron-app package.json's author field.
What should we take as the organisation name? Some suggestions:

Eclipse
Eclipse Theia
Eclipse Foundation

I suggest "Eclipse Theia"


[Bug]: Profane warning for 'Class'

Describe the bug
The bot sends a profance word warrning for the word 'Class'.
To Reproduce
Type 'Class' with Community Monitor in your server.
Expected Behavior
No response
Screenshot/ Video
No response
Desktop (please complete the following information)
No response
Additional context
No response

How can I fix this issue?

Issue Update: I think the error is with any word using with 'ss' eg: class, glass.

@krishguptadev I think the issue is in the restrictedwords.js. Maybe some files need to be updated there but I cannot see any code lines there which might be causing the issue

I think the issue is being caused due to the word 'ass'. since the words 'class' and 'glass' have a substring of 'ass' It is throwing a warning when those workds are invoked. I think in order to solve this issue an excpetion file may have to be made that excludes 'ass' if it is a substring

How about using alexjs for the purpose? Just like EddieHubCommunity/EddieBot does that

I dont know about alexjs, could you implement it please?

I am talking about this: https://alexjs.com/ Can I get assigned to implement alexjs instead of the old method?

I'd really appreciate it if you could implement alexjs. I do not have the assignment permission but I will ask the maintainer to assign you this issue :)


[CSBindings] Try to infer bindings from related type variables

Attempt to use potential bindings inferred for related types variables
discoverable through 'subtype' constraints, this helps to build a
more precise bindings domain for each type variable.
Resolves: rdar://problem/38159133

/cc @rudkx this is the only way I see to fix this at this point... Maybe with a better join implementation we would not need this...

@swift-ci please test

@swift-ci please test source compatibility

@xedin I’ll pull this down and take a look.

@rudkx Thanks!


Custom Login/Register Pages

The Login and Register forms in the template seem to come completely pre-made from IdentityServer. The html for the angular login and logout components consist entirely of one line: “{{ message | async }}”.
Is there a way to change these forms (add more fields, change design) or to use my own login page in Angular?

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

ID: 21cb7455-3305-9316-5440-bdc22947a82d
Version Independent ID: 137d4b94-7b26-3911-b22d-42c754a95fc1
Content: Introduction to authentication for Single Page Apps on ASP.NET Core
Content Source: aspnetcore/security/authentication/identity-api-authorization.md
Product: aspnet-core
Technology: aspnetcore-security
GitHub Login: @javiercn
Microsoft Alias: scaddie

You can customize them through identity
https://docs.microsoft.com/en-us/aspnet/core/security/authentication/scaffold-identity?view=aspnetcore-2.2&tabs=visual-studio

Not quite sure how to do that. Adding an example to the doc would be quite useful.

@nkaenzig  https://docs.microsoft.com/en-us/aspnet/core/security/authentication/scaffold-identity?view=aspnetcore-2.2&tabs=visual-studio has examples.


WebSockets over HTTP/2

We've heard some interest express for WebSockets support over HTTP/2 in YARP. This would require support in ASP.NET Core (Kestrel) and HttpClient. The protocol is quite different from original WebSockets.
RFC: https://datatracker.ietf.org/doc/html/rfc8441
Kestrel: https://github.com/dotnet/aspnetcore/issues/7801
HttpClient: https://github.com/dotnet/runtime/issues/69669
We'd also need to worry about protocol upgrades/downgrades between front-end and back-end.

Triage: We have interested partner team, but not yet committed. It helps with COGS.
We would need feature work in .NET Runtime (7.0+).
Forwarding is directly through SocketsHttpHandler, so the API pattern might be different from current ClientWebSocket.

This should be happening for .NET 7 on both the server and client side, so let's make YARP take advantage of that in 2.0

@greenEkatherine as the first step here, can you test YARP 1.1 on .NET 7 with an Http/2 WebSocket client and capture the error here in this issue?

I'm running YARP 1.1 with .NET 7, attempting to proxy the Angular CLI development server's WebSocket connection. Here's some information that might be helpful:
Headers:
{
  "method": "GET",
  "url": "wss://localhost:5001/ng-cli-ws",
  "httpVersion": "",
  "headers": [
    {
      "name": "Pragma",
      "value": "no-cache"
    },
    {
      "name": "Origin",
      "value": "https://localhost:5001"
    },
    {
      "name": "Accept-Encoding",
      "value": "gzip, deflate, br"
    },
    {
      "name": "Host",
      "value": "localhost:5001"
    },
    {
      "name": "Accept-Language",
      "value": "en-GB,en-US;q=0.9,en;q=0.8"
    },
    {
      "name": "User-Agent",
      "value": "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36 Edg/107.0.1418.42"
    },
    {
      "name": "Upgrade",
      "value": "websocket"
    },
    {
      "name": "Cache-Control",
      "value": "no-cache"
    },
    {
      "name": "Connection",
      "value": "Upgrade"
    },
    {
      "name": "Sec-WebSocket-Version",
      "value": "13"
    },
    {
      "name": "Sec-WebSocket-Extensions",
      "value": "permessage-deflate; client_max_window_bits"
    }
  ]
}

Here's the request as logged in ASP.NET Core:
[09:44:51 INF] Request starting HTTP/2 CONNECT https://localhost:5001/ng-cli-ws

Here's a stack-trace from YARP:
System.NotSupportedException: Unsupported request method 'CONNECT'.
   at Yarp.ReverseProxy.Forwarder.RequestUtilities.GetHttpMethod(String method)
   at Yarp.ReverseProxy.Forwarder.HttpForwarder.CreateRequestMessageAsync(HttpContext context, String destinationPrefix, HttpTransformer transformer, ForwarderRequestConfig requestConfig, Boolean isStreamingRequest, ActivityCancellationTokenSource activityToken)
   at Yarp.ReverseProxy.Forwarder.HttpForwarder.SendAsync(HttpContext context, String destinationPrefix, HttpMessageInvoker httpClient, ForwarderRequestConfig requestConfig, HttpTransformer transformer)

Let me know if I can provide more information. If I've misunderstood and this isn't what you're after, I can delete this comment.

@serpent5 we know, we haven't implemented that support for YARP yet.
Some workarounds in the meantime:

Disable HTTP/2 in Kestrel.
Add a middleware to reject CONNECT requests with HTTP_1_1_REQUIRED (0xd) (I need to try this...)

https://learn.microsoft.com/en-us/aspnet/core/fundamentals/servers/kestrel/http2?view=aspnetcore-6.0#reset
You can also try returning a 400 (I need to try this too...)

Ok, thanks. I understand it's a known issue. I was just trying to provide more information around the exception to see if it helped. Thanks for the workarounds.

In my testing, rejecting the request with HTTP_1_1_REQUIRED works well. Returning a 400 causes the client to keep retrying the request, and so it keeps failing.

@serpent5 we know, we haven't implemented that support for YARP yet.

I know it's a hard question but: do you have any estimated release date for Websockets over HTTP/2?

I know it's a hard question but: do you have any estimated release date for Websockets over HTTP/2?

Not specifically, but we should start working on it soon. Once it's merged you'll be able to try it from nightly builds and preview releases.
What's your interest/dependency on this feature?

We would need it for Server-Side-Blazor.

To safe people some time: a temporary middleware implementing the fix from above: https://gist.github.com/Lakerfield/0ea3a687974df5192d66a375d78ce0e6
if (context.Request.Method == HttpMethods.Connect && context.Request.Protocol != HttpProtocol.Http11)
{
  var resetFeature = context.Features.Get<IHttpResetFeature>();
  if (resetFeature != null)
  {
    //https://www.rfc-editor.org/rfc/rfc7540#page-51
    //HTTP_1_1_REQUIRED (0xd):  The endpoint requires that HTTP/1.1 be used instead of HTTP/2.
    resetFeature.Reset(errorCode: 0xd);
    return;
  }
}

Hi
Will this http2 support added recently means that it will support also an incoming http1.1 ws connection request from client while the outgoing (downstream service) can be http2 wss?

Yes, you can do any combination of [1.1, 2.0] incoming to [1.1, 2.0] outgoing.

Yes

Since which yarp version?
I think it's not in latest published nuget 1.1
בתאריך יום ג׳, 17 בינו׳ 2023, 20:53, מאת Miha Zupan <
@.***>:

Yes, you can do any combination of [1.1, 2.0] incoming to [1.1, 2.0]
outgoing.
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/reverse-proxy/issues/1375#issuecomment-1385893835,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGGVMYDBGH257E7LRV6RCALWS3TEBANCNFSM5H3SHTWA
.
You are receiving this because you commented.Message ID:
@.***>

This will be in the upcoming 2.0-rc.1 release.


Upgrade dolphin to v4.0-7010

Thank you for the contribution. It was merged directly as 88e484923235937d712b9c475ac1e4b38bdf5313 to keep commit history cleaner. Your contribution is still credited to you.


Remove erlydtl and replace with mustache

I think I did it...

We should also open up the erlydtl compiler as a plugin to provide a better migration path. Hopefully the templates people used won't explode. What does mustache allow as far as features go for conditionals and such?

Basic conditionals and stuff https://github.com/mojombo/mustache.erlhttps://github.com/mojombo/mustache.erl
Basic templates will work the same. And this means old rebar2 templates people made work the same.
I'll move the erlydtl provider to a plugin under my account soon.

@marianoguerra do you think this could break your efene templates or it should be compatible? You're the biggest user of these templates I know.

https://github.com/tsloughter/rebar3_erlydtl_plugin

I just use variable substitution and in case it breaks I can fix them when it's merged :)

Doc updated: http://www.rebar3.org/v3.0/docs/using-templates
Going through the commands to drop the ErlyDTL stuff.


refactor: i18n update it

Description

Related Issues

Checklist:

[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
[x] Have you linted your code locally prior to submission?
[x] Have you successfully ran app with your changes locally?

Are there more updates? If not, I will merge.

Are there more updates? If not, I will merge.

Hi,
no more updates.thanks!


Rendering problems

There is a problem with rendering where the class table overlaps with the text below it, only correcting itself once another character is typed.
Here is an example:
Additional Details
Brew code to reproduce : Click to expand

##### The Elementalist
| Level | Proficiency Bonus | Strike | Rushes | Features | Spells Known | 1st | 2nd | 3rd | 4th | 5th |
|:----:|:--:|:----:|:-:|:------------------------------------|:---:|:---:|:---:|:---:|:---:|:---:|
| 1st  | +2 | 1d4  | 2 | Unarmed Strike, Elemental Rush      | —  | — | — | — | — | — |
| 2nd  | +2 | 1d4  | 2 | Spellcasting, Weaponless Charge     | 2  | 2 | — | — | — | — |
| 3rd  | +2 | 1d4  | 2 | Elemental Origin, Selective Channel | 3  | 3 | 2 | — | — | — |
| 4th  | +2 | 1d4  | 3 | Ability Score Improvement           | 3  | 3 | 2 | — | — | — |
| 5th  | +3 | 1d6  | 3 | Elemental Weapon                    | 3  | 4 | 3 | — | — | — |
| 6th  | +3 | 1d6  | 3 |                                     | 4  | 4 | 3 | — | — | — |
| 7th  | +3 | 1d6  | 3 | Origin Feature                      | 4  | 4 | 3 | — | — | — |
| 8th  | +3 | 1d6  | 3 | Ability Score Improvement           | 5  | 4 | 3 | — | — | — |
| 9th  | +4 | 1d8  | 4 | —                                   | 5  | 4 | 3 | 2 | — | — |
| 10th | +4 | 1d8  | 4 |                                     | 5  | 4 | 3 | 2 | — | — |
| 11th | +4 | 1d8  | 4 | Origin Feature                      | 6  | 4 | 3 | 3 | — | — |
| 12th | +4 | 1d8  | 4 | Ability Score Improvement           | 6  | 4 | 3 | 3 | — | — |
| 13th | +5 | 1d10 | 4 |                                     | 7  | 4 | 3 | 3 | 1 | — |
| 14th | +5 | 1d10 | 5 |                                     | 7  | 4 | 3 | 3 | 1 | — |
| 15th | +5 | 1d10 | 5 |                                     | 8  | 4 | 3 | 3 | 2 | — |
| 16th | +5 | 1d10 | 5 | Ability Score Improvement           | 8  | 4 | 3 | 3 | 2 | — |
| 17th | +6 | 1d12 | 5 | Origin Feature                      | 9  | 4 | 3 | 3 | 3 | 1 |
| 18th | +6 | 1d12 | 5 |                                     | 9  | 4 | 3 | 3 | 3 | 1 |
| 19th | +6 | 1d12 | 5 | Ability Score Improvement           | 10 | 4 | 3 | 3 | 3 | 2 |
| 20th | +6 | 1d12 | 5 |                                     | 10 | 4 | 3 | 3 | 3 | 2 |

Creating a Elementalist
A core part to what an elementalist is is what their origins are. Most of them end up stumbling into their power without direction. There isn't usually guilds or gatherings of elementalists, as most don't even know of their power as being categorized. Although the powers granted by the elemental origin don't come until 3rd level, you should plan ahead on which element they succeed with and how that ties into who they are. Why did Mother Earth reach out to you? Did you find her? Where were you before you end up discovering your powers? Most elementalists are every-day no bodies who have had their life thrown sideways by the disovery of their powers.
How did you discover your powers? Was it accidental or did you actively search it out. Many elementalists were those searching for power, abandoning whatever powers they had before to uptake the new life they have found in front of them.
Quick Build
You can make an Elementalist quickly by following these steps. First, make Strength your highest ability score, followed by Wisdom. Second, choose the urchin background.

Class Features
As an elementalist, you gain the following class features
Hit Points

Hit Dice: 1d8 per elementalist level
Hit Points at 1st Level: 8 + your Constitution modifier
Hit Points at Higher Levels: 1d8 (or 5) + your Constitution modifier per fishmongerer level after 1st

Proficiencies

Armor: Medium armour, Light Armour
Weapons: Simple weapons, fists
Tools: Alchemist's tools

Saving Throws: Strength, Wisdom
Skills: Your choice of two from: Acrobatics, Athletics, Arcana, Intimidation, Perception, Stealth, Survival

Equipment
You start with the following equipment, in addition to the equipment granted by your background:

(a) a martial weapon or (b) two simple weapons
(a) leather armour or (b) dragon scale armour
(a) explorer's pack or (b) dungeoneer's pack

Duplicate of #295


Appium Driver java.lang.NullPointerException

Hi , i've just cloned your repo and try running on Java 1.8 . is there something im missing ?
[ERROR] Tests run: 3, Failures: 2, Errors: 0, Skipped: 1, Time elapsed: 3.535 s <<< FAILURE! - in com.eliasnogueira.TipTest
[ERROR] com.eliasnogueira.TipTest.preConditionnull, null, null  Time elapsed: 2.899 s  <<< FAILURE!
java.lang.NullPointerException

It seems the three required parameters are null.
Did you run the tests using the test suite?

Me also facing same issue, update IP address and run test_suite.xml
do I need to run anything manually ?


MCS-889 Replaced NOT_RECEPTACLE with NOT_INTERACTABLE in some scenarios and updated docs.

Corresponding Unity PR: https://github.com/NextCenturyCorporation/ai2thor/pull/162

@cedorman - Definitely needs to be aware of this ticket for the scorecard work I believe.


Parallelize decoding

Parallelizing decoding is straight-forward with TIFFs since they are composed of separately-encoded strips or tiles
One might say this should be done at the client/file level, but for large files it may be convenient to have it done at the strip level, especially in an interactive context
Here are results I get for a variety of LZW-compressed images--from tiny to huge, for white noise and solid color--showing a win for all but the smallest, simplest images
white noise

serial (s)
parallel (s)

25.06848
3.21034

2.06287
0.30021

0.27048
0.07222

0.06873
0.02438

0.00174
0.00331

solid color

serial (s)
parallel (s)

1.35969
0.88756

0.08941
0.05104

0.00777
0.01789

0.00193
0.00331

0.00012
0.00023

:100: This would be great to have. I agree that the TIFF format lends itself nicely to parallelization. I just haven't had the bandwidth to tackle it, but would review any PR if you want to tackle it (looks like you might have a first draft already?)


[feat] [C API] Expose Get/Set Listener Name in C API

Fixes #369
Motivation
The NodeJS client relies on the C API, which does not yet include the getListener and setListener methods. This means that using the NodeJS client, one is unable to choose which listener to create a client for.
Modifications
Exposed getListener and setListener via the C API.
Verifying this change

[ ] Make sure that the change passes the CI checks.

Added test for newly exposed methods

Documentation

[ ] doc-required
(Your PR needs to update docs and you will update later)

[x] doc-not-needed
(Please explain why)

[ ] doc
(Your PR contains doc changes)

[ ] doc-complete
(Docs have been already added)

PR validation / Build the Wireshark dissector (macos-12) is very flaky due to the upstream (homebrew) issue. Let's ignore this test for now and merge it.

Thanks all for the quick review, and for looking into those checks! Do you know if there's an expected timeline for 3.5.0?

I updated the timeline with 2023/01/19 just now.

https://github.com/apache/pulsar-client-node/issues/214 is currently being blocked due to this changes no being released. What's the expected timeline for a release?


progress ?

I was wondering if this project is progressing .

The project is not being worked on but feel free to fork it.


The Windows Subsystem for Linux instance has terminated.

Version
Microsoft Windows [Version 10.0.19043.1266]
WSL Version

[X] WSL 2
[ ] WSL 1

Kernel Version
shadycuz-> wsl.exe --status
Default Distribution: Ubuntu
Default Version: 2

Windows Subsystem for Linux was last updated on 7/22/2021
WSL automatic updates are on.

Kernel version: 5.10.16

Distro Version
Ubuntu
shadycuz-> lsb_release -r
Release:        20.04

Other Software
Docker Desktop (Windows) 3.5.1 (66090)
Visual Studio Code 1.61.0
Repro Steps
This is an intermittent issue that happens to me everyday but under different circumstances. The common denominator seems to be the use of vscode.
While using vscode in the wsl backend, the editor will suddenly disconnect. This is around the time that Vmmem has exhausted the 6GB of memory I have given to wsl, even though I can't seem to see what is using the 6GB of ram. In wsl it shows very little usage. In docker desktop I will see 2/3GB if I have a running container. It seems like their is a memory leak, since I will be coding for 2 or 3 hours before the crash but not opening any new processes. Except maybe browser tabs outside of wsl.
Once wsl has crashed I can not get back into it or use docker for windows.
C:\Users\work>wsl
The Windows Subsystem for Linux instance has terminated.

C:\Users\work>wsl -l -v
  NAME                   STATE           VERSION
* Ubuntu                 Running         2
  docker-desktop         Running         2
  docker-desktop-data    Running         2

but I can shut it down and restart it
C:\Users\work>wsl --shutdown

C:\Users\work>wsl
[/mnt/c/Users/work]
shadycuz->

Expected Behavior
It would be nice if it gave some info about why it has shutdown so I could fix/prevent it. It would also be nice to have some detailed logs somewhere like in the WSL container or in windows? Maybe the event viewer?
Actual Behavior
wsl crashes without explanation for why/what caused it.
Diagnostic Logs
I will try and provide next time it crashes.
Update
I was working inside a container using the remote-container extension for about an hour and all seemed fine. I then used the workspace search feature in vscode to search for Class I wanted to rename in multiple files. The ram usage for my docker container grew by over 1GB and the 1 or 2GB of free ram I had in WSL using free -h -t went from around 1.5Gi available to 36Mi.
I had first noticed that the search was taking forever when it's usually instant. Then the vscode window disconnected.

Note: In the search it says 1 result but that is not correct.
The container that vs-code was running then disconnected with exit code 137 which is from OOM kill. I had a second wsl window open running watch free -h -t and it froze.

My wsl2 config:
[wsl2]
memory=6GB # Limits VM memory in WSL 2 to 4 GB
processors=3 # Makes the WSL 2 VM use two virtual processo

I wonder if making the swap bigger could prevent these crashes. I could also give more memory?

Me and my team have similar issue. Wsl crashes multiple times a day. I need to restart wsl and docker to work again. We are using intellij solutions, Webstorm and Phpstorm. We have standard more wsl and memory.
[wsl2]
memory=8GB   # Limits VM memory in WSL up to 8GB
processors=4 # Makes the WSL use 4 virtual processors

I tried to increase memory for wsl, it does not resolve our issues.

@benhillis
@OneBlue
or anyone else from team. Can some of you please look into the issue? It's occurring so often and issue is here for over year without any action. Please stop ignoring it. Thank you!

Hi guys. Sorry this is crashing for you. Do you generally see this with Docker also being deployed? we're trying to find what might be causing this.

/logs

@keith-horton
Yes, we use Docker Desktop connected with WSL2. But I also heard a colleague who uses Rancher alternative and is facing the same issue.
@OneBlue
Thanks! Will try it and paste when the WSL crashes again. Stay tuned.
Executing second command:
Set-ExecutionPolicy Bypass -Scope Process -Force

outputs:
Set-ExecutionPolicy : Windows PowerShell updated your execution policy successfully, but the setting is overridden by a
 policy defined at a more specific scope.  Due to the override, your shell will retain its current effective execution
policy of RemoteSigned. Type "Get-ExecutionPolicy -List" to view your execution policy settings. For more information p
lease see "Get-Help Set-ExecutionPolicy".
At line:1 char:1
+ Set-ExecutionPolicy Bypass -Scope Process -Force
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : PermissionDenied: (:) [Set-ExecutionPolicy], SecurityException
    + FullyQualifiedErrorId : ExecutionPolicyOverride,Microsoft.PowerShell.Commands.SetExecutionPolicyCommand

unsure whether it might affect the log output, but I am afraid I won't be able to change that as policies are managed by company.

WslLogs-2022-11-30_09-58-22.zip
Thisi is what I could see in console.
Directory: C:\Users\z0034zpz\wsl.logs

Mode                 LastWriteTime         Length Name
----                 -------------         ------ ----
d-----        11/30/2022   9:58 AM                WslLogs-2022-11-30_09-58-22
The operation completed successfully.
The operation completed successfully.
The operation completed successfully.

        The profiles are already running.

        Profile Id: WSL.Verbose.File

        Error code: 0xc5583001
Log collection is running. Please reproduce the problem and press any key to save the logs.
Saving logs...

        There are no trace profiles running.

        Error code: 0xc5583000

Another crash log retrieved:
WslLogs-2022-12-01_09-20-42.zip
Est. time of crash 13:03

I have since moved to 64GB of ram and don't have problems anymore. 32GB might also work.

@keith-horton
@OneBlue
Hi guys, any update please?

Thank you for the logs @Lkopo.
It looks like the WSL service (lxssmanager) is either being restarted, or crashing.
Can you please follow the '9) Collect WSL logs with Feedback hub
' section and share the resulting URL ?
With that we should be able to see service crashes

@OneBlue
This is what I see in WinDbg:
(6fd4.2198): Security check failure or stack buffer overrun - code c0000409 (first/second chance not available)
Subcode: 0x7 FAST_FAIL_FATAL_APP_EXIT
For analysis of this file, run !analyze -v
ucrtbase!abort+0x4e:
00007ffb`26fd286e cd29            int     29h

*******************************************************************************
*                                                                             *
*                        Exception Analysis                                   *
*                                                                             *
*******************************************************************************

KEY_VALUES_STRING: 1

    Key  : Analysis.CPU.mSec
    Value: 640

    Key  : Analysis.DebugAnalysisManager
    Value: Create

    Key  : Analysis.Elapsed.mSec
    Value: 2758

    Key  : Analysis.IO.Other.Mb
    Value: 73

    Key  : Analysis.IO.Read.Mb
    Value: 1

    Key  : Analysis.IO.Write.Mb
    Value: 102

    Key  : Analysis.Init.CPU.mSec
    Value: 1921

    Key  : Analysis.Init.Elapsed.mSec
    Value: 639149

    Key  : Analysis.Memory.CommitPeak.Mb
    Value: 141

    Key  : FailFast.Name
    Value: FATAL_APP_EXIT

    Key  : FailFast.Type
    Value: 7

    Key  : Timeline.OS.Boot.DeltaSec
    Value: 182167

    Key  : Timeline.Process.Start.DeltaSec
    Value: 1

    Key  : WER.OS.Branch
    Value: vb_release

    Key  : WER.OS.Timestamp
    Value: 2019-12-06T14:06:00Z

    Key  : WER.OS.Version
    Value: 10.0.19041.1

    Key  : WER.Process.Version
    Value: 10.0.19041.1566

FILE_IN_CAB:  wsl.exe.28628.dmp

NTGLOBALFLAG:  0

PROCESS_BAM_CURRENT_THROTTLED: 0

PROCESS_BAM_PREVIOUS_THROTTLED: 0

APPLICATION_VERIFIER_FLAGS:  0

CONTEXT:  (.ecxr)
rax=0000000000000001 rbx=00007ffb26fd1f80 rcx=0000000000000007
rdx=000000000000000f rsi=00007ff7d164b7c0 rdi=0000000000000000
rip=00007ffb26fd286e rsp=000000230027b760 rbp=000000230027fce0
 r8=0000000000000001  r9=000000230027b708 r10=0000000000000012
r11=0000200880000020 r12=ffffffffffffffff r13=00007ffb26ec69a8
r14=0000000000000001 r15=000000230027b9b0
iopl=0         nv up ei pl nz na pe nc
cs=0033  ss=002b  ds=002b  es=002b  fs=0053  gs=002b             efl=00000202
ucrtbase!abort+0x4e:
00007ffb`26fd286e cd29            int     29h
Resetting default scope

EXCEPTION_RECORD:  (.exr -1)
ExceptionAddress: 00007ffb26fd286e (ucrtbase!abort+0x000000000000004e)
   ExceptionCode: c0000409 (Security check failure or stack buffer overrun)
  ExceptionFlags: 00000001
NumberParameters: 1
   Parameter[0]:<PHONE_NUMBER>000007
Subcode: 0x7 FAST_FAIL_FATAL_APP_EXIT

PROCESS_NAME:  wsl.exe

ERROR_CODE: (NTSTATUS) 0xc0000409 - The system detected an overrun of a stack-based buffer in this application. This overrun could potentially allow a malicious user to gain control of this application.

Thank you @Lkopo, that's very interesting.
Can you write a dump file and share it with us (.dump /f) ?

Oh wait, looking at the stack, I think I know what it is.  It's an issue that should be fixed with this update. Let me know if installing the update resolves the issue.

Hi @OneBlue,
adding the following update. So the suggested update seems to be not automatically available for us as it's a Preview version, but after installing December update https://support.microsoft.com/en-us/topic/december-13-2022-kb5021233-os-builds-19042-2364-19043-2364-19044-2364-and-19045-2364-44e774aa-60c4-4e38-b7e7-c886d210db3b, WSL terminal suggested to update WSL via wsl.exe --update. I did that yesterday, and since then I didn't experience any crashes. It might be too soon, so I'll keep you posted whether the issue is still present.
On the contrary, everytime I start up my computer, I receieve following error message for "RemoteApp" which I have to accept at least 3-4 times to disappear.

This started happen directly after wsl.exe --update command finishes.

Interesting, that's not expected.
@hideyukn88: Do you know what could cause this ?

@Lkopo, it's most likely your organization deployed some group policy? See https://github.com/microsoft/wslg/issues/841, thanks!

Hey I'm facing the same issue


add module docs and man page for pychroot utility

Probably use sphinx and rst files to add some initial docs.

Initial sphinxification done, man page will come later.


Directory is ignored on linux (xfce)

When I use "open terminal here" it opens a terminal but completely ignores the directory. The terminal always starts at home directory which is not the desired behavior.
It doesn't matter if I use it from the tree view or from an open file.
I'm on Linux Mint 17.1 with XFCE desktop.

Which version of the plugin are you running?
I've just released v2.0.0 yesterday which fixes this issue on Ubuntu Linux.

I've updated to v2.0.0 earlier but still had the issue. Now - after closing and starting again - it's gone. Thanks!
I noticed that the update didn't finish correctly after hitting the button - it kept going on and on but the package manager later said it was on v2.0.0 (maybe that caused the delay). I don't know if that's an issue with the packages (happens sometimes with different packages, first time with yours) or with atom itself.
But I'm happy that it works now because that saves me a lot of time.

Thanks @phellmes!
I'm not sure what caused the delay in the update.
As a package developer, updating a package consists only of running a publish command which pushes the updates to GitHub and registers the new package in the Atom registry.


test raftstore::test_stale_peer::test_node_stale_peer_out_of_region ... thread 'main' panicked

raftstore::test_stale_peer::test_node_stale_peer_out_of_region
https://internal.pingcap.net/idc-jenkins/blue/organizations/jenkins/tikv_ghpr_test/detail/tikv_ghpr_test/18994/pipeline/230 Integration Part16

This issue has become out-of-date. Given that no further feedback regarding this issue has been reported, it can be safely closed.


dbt does not appropriately quote column names for incremental models and seed files.

Describe the bug
dbt doesn't cooperate with lower- or mixed-case column names on Snowflake, regardless of quoting settings currently. While sometimes this "just works", there are two areas of the product that I've run into in testing that exhibit issues when mixed casing is used:

dbt seed does not respect quotes in column names inside of a CSV file, creating the columns as unquoted.
Both incremental materialization strategies (merge and delete+insert) list out the columns using introspection and do not quote the column list. Therefore, if any of the columns in an incremental model are mixed- or lower-case, incremental updates will always fail with an invalid identifier error.

I also tested snapshotting on top of the same test dataset I was using, and in all cases I could identify, column-level quoting was applied correctly and dbt produced the intended results.
Steps To Reproduce
Here's my test scenario:
create or replace table analytics.DBT_JTHANDY."QuotingSource" ("col_A" integer,"col_B" integer);
insert into analytics.DBT_JTHANDY."QuotingSource" ("col_A", "col_B") values
(1, 2),
(3,4),
(5,6)

--quoting_test.sql
{{config(
    materialized = 'incremental',
    unique_key = '"col_A"',
    incremental_strategy = "delete+insert"
    )}}

select "col_A", "col_B"
from {{ref('QuotingSource')}}

"col_A","col_B"
1,2
3,4
5,6

Expected behavior
I would expect that all columns that are quoted when their column name is provided to be referenced by dbt using their quoted column name.
System information
Which database are you using dbt with?

[ ] postgres
[ ] redshift
[ ] bigquery
[x] snowflake
[ ] other (specify: ____________)

The output of dbt --version:
installed version: 0.14.2
latest version: 0.14.3

The operating system you're using: OSX
The output of python --version: Python 2.7.10

Two quick editorial notes:

of the two problems (seeds and incremental models) identified here, I believe that incremental models are a significantly bigger problem. Fixing seeds would be good, but the fact that this is broken on incremental models is a much bigger problem IMO.
I haven't looked at the source code at all, but I attempted to test the areas of the product where I could anticipate that dbt might be writing column-level SQL statements out. I may have missed some. It seems like this is an area that we haven't looked at explicitly in the past, so it would be useful for someone actually working on this to determine whether or not there are any other potential areas of the code base that this could impact that I missed.

I'm actually running into the same issue, also documented in this SO question. My table is built flawlessly but the incremental runs fail because the engine "uppercases" a column name that's not fully uppercased in my database:
The query:
{{
  config(
    materialized='incremental',
    unique_key='event_ID'
  )
}}

SELECT
    {{ dbt_utils.star(from=ref('staging_pg_ahoy_events'), relation_alias='events', prefix='event_') }},
    {{ dbt_utils.star(from=ref('staging_pg_ahoy_visits'), relation_alias='visits', prefix='visit_') }}
FROM
    {{ ref('staging_pg_ahoy_events') }} AS events
LEFT JOIN {{ ref('staging_pg_ahoy_visits') }} AS visits ON events.visit_id = visits.id
{% if is_incremental() %}
    WHERE "event_ID" >= (SELECT max("event_ID") FROM {{ this }})
{% endif %}

The error:
14:07:57    000904 (42000): SQL compilation error: error line 4 at position 12
14:07:57    invalid identifier 'DBT_INTERNAL_SOURCE.EVENT_ID'

The only fix I've found so far is by upper casing the prefix via dbt_utils, but that's more of a workaround…

Experiencing the same invalid identifier error with snowflake and dbt incremental run, as described by @coisnepe.
@drewbanin Would it be possible to reopen this ticket or would you advise creating a new one - if there is none already?

@dvd-p would you be able to open a new issue please? It just would make it easier to look at this with your specific scenario and details to make sure we are able to reproduce the error correctly

Came here as this bug is still active using the Snowflake adapter.  I did have success by implementing this workaround in the config (using @coisnepe example).
unique_key='"event_ID"'


Switch all Reporting Platform components to Apache 2.0 license

I'm starting to think about what needs to happen to make the Reporting Platform fully public — by which, for clarity, we'll say means pushing semantically versioned releases to major public package repositories (e.g., PyPi, npm, Docker Hub). As the product of a commercial venture itself, and in the interest of wider adoption, the copyleft requirements of the GPL aren't appropriate. Among non-copyleft, permissive, "pushover" licenses, Apache 2.0 is the recommendation of the FSF and their reasoning looks sound to me. I also notice that it seems to be popular among the caliber and type of projects that I'd like to be playing among, — viz. Luigi, Superset, &c.

I'm a fan of Apache 2.0

That sounds like a good idea to me as well.

https://github.com/OAODEV/nerium/commit/88fb89f5398cb7c59b2162a1d041541f1e5a1cce


The new lobby captcha solving

Hey.
Presenting new issue with the new option you just added:
`The interactive captcha has been presented.
Do you want to solve it via Telegram? (Y/n)

y
Traceback (most recent call last):
File "ikabot\command_line.py", line 208, in
File "ikabot\command_line.py", line 197, in main
File "ikabot\command_line.py", line 185, in start
File "ikabot\web\session.py", line 36, in init
File "ikabot\web\session.py", line 205, in __login
File "ikabot\helpers\botComm.py", line 48, in sendToBot
File "ikabot\helpers\botComm.py", line 128, in checkTelegramData
File "ikabot\helpers\botComm.py", line 74, in telegramDataIsValid
File "ikabot\web\session.py", line 580, in getSessionData
AttributeError: 'Session' object has no attribute 'cipher'
[43380] Failed to execute script command_line

C:\Users\user\Desktop\ikabot>`

you are most likely not using the most up to date version of ikabot.
are you using 6.5.8?

you are most likely not using the most up to date version of ikabot.
are you using 6.5.8?

Indeed, I got the update

great, does it work? can I close this?

great, does it work? can I close this?

No, the problem exists in the new version, that's the point

I'm not sure why but the package binary for 6.5.8  did not get generated.
Installing ikabot with pip will install the latest version just fine.

Ok I'm closing this


improvement to Contact us page

The contact us page has some text that is a relic from when there was no events page:
Upcoming events
If you'd like to find out more about our monthly upcoming events please drop us a mail at
<EMAIL_ADDRESS>Would be better to get rid of this section completely.

Rather than removing it I have added a link to the events page.
The text now reads: "Upcoming events
If you'd like to find out more about our upcoming events visit our
events page."
This change will be visible on dev shortly.


online doc: the rendering of the tables is broken

https://docs.rubocop.org/rubocop/cops_layout.html for instance shows as:

(Firefox and Safari)
A bit farther:

Cheers!

seems to be due to this commit: https://github.com/rubocop/rubocop/commit/52a56f0be2332d61718597261a15860201d1fc3c

While at it, you might want to fix the example of AbcSize (https://docs.rubocop.org/rubocop/cops_metrics.html#metricsabcsize):
# `model` and `current_user`, refenced 3 times each,

I suppose s/refenced/referenced/.

The rendering of the tables will be recovered by https://github.com/rubocop/rubocop/pull/11136. But not yet applied to the doc site. And s/refenced/referenced/ has been fixed by https://github.com/rubocop/rubocop/commit/af68c6fc420fcbe7d6bfca1db90df3a93a21938b.

RuboCop 1.39 has been released. Please see the latest docs. Thank you.