trace: allow enabling packet trace at runtime

If the -t, --test-mode flag is used, enable packet tracing instead of
relying on an undocumented compilation flag. The performance impact
should be minimal when it is not enabled.

On the principle, it looks good, but I suspect it has a non negligeable impact on performance.

Out of curiosity, I ran a simple IP forwarding performance test, before and after this patch.
before: 23.56 MPPS

GROUT: main: starting grout version v0.1-1-gbed15f4ef48e

NODE        CALLS  PACKETS  PKTS/CALL  CYCLES/CALL  CYCLES/PKT
ip_input     1988   315693      158.8       8367.9        52.7
port_rx      2176   315693      145.1       5573.3        38.4
port_tx      1988   315693      158.8       4400.9        27.7
eth_input    2176   315693      145.1       3263.6        22.5
ip_output    1988   315693      158.8       3168.4        20.0
eth_output   1988   315693      158.8       3135.6        19.7
ip_forward   1988   315693      158.8       1741.1        11.0

after: 23.56 MPPS

GROUT: main: starting grout version v0.1-2-ge3133c7c1c80

NODE        CALLS  PACKETS  PKTS/CALL  CYCLES/CALL  CYCLES/PKT
ip_input     2354   315196      133.9       7055.5        52.7
port_rx      2624   315196      120.1       4626.7        38.5
port_tx      2354   315196      133.9       3678.2        27.5
eth_input    2624   315196      120.1       2708.8        22.6
ip_output    2354   315196      133.9       2681.7        20.0
eth_output   2354   315196      133.9       2621.4        19.6
ip_forward   2354   315196      133.9       1455.5        10.9

If this patch has any impact, it is not noticeable in this particular test.


ROX-13259: graphInit called during init time

Description
A detailed explanation of the changes in your PR.
Feel free to remove this section if it is overkill for your PR, and the title of your PR is sufficiently descriptive.
Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added
[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed
TODO(replace-me)
Use this space to explain how you tested your PR, or, if you didn't test it, why
you did not do so. Valid reasons include, for example, "CI is sufficient",
"No testable changes". Feel free to attach JSON snippets, curl commands,
screenshots.
In addition to reviewing your code, reviewers must also review your testing
instructions and make sure they are sufficient.

/test all

/retest

Going to wait on #3727 to go in so that the tests pass


Fixes #3574. Enabling trim mode breaks program at runtime (v2 only)

Fixes

Fixes #3574

Proposed Changes/Todos

[ ] Add SourceGenerationContext

Pull Request checklist:

[x] I've named my PR in the form of "Fixes #issue. Terse description."
[x] My code follows the style guidelines of Terminal.Gui - if you use Visual Studio, hit CTRL-K-D to automatically reformat your files before committing.
[x] My code follows the Terminal.Gui library design guidelines
[x] I ran dotnet test before commit
[ ] I have made corresponding changes to the API documentation (using /// style comments)
[x] My changes generate no new warnings
[x] I have checked my code and corrected any poor grammar or misspellings
[x] I conducted basic QA to assure all features are working

I think @dodexahedron should review this before we merge.

@BDisp would it be possible (needed?) to fold the work @tznind did on

#2423

into this PR?

FYI - tests are failing because of work i'm doing on trying to fix #3052. JFYI

@tig, can you test the Analyzers unit tests on Linux to see if they are failing assertions due to the newline format, please? Thanks.

@tig, can you test the Analyzers unit tests on Linux to see if they are failing assertions due to the newline format, please? Thanks.

But in the VS2022, please.

@tig, can you test the Analyzers unit tests on Linux to see if they are failing assertions due to the newline format, please? Thanks.

But in the VS2022, please.

Yep.

So I was right about the failure of these tests. The test files are in CRLF format, which means that on Windows the tests do not fail. But on Linux it fails because it uses the /n format for newlines. Changing the format to LF tests pass on Linux. As the CI is now using linux-ubuntu, the tests must be failing but it does not detect the source of the failure well. I am increasingly convinced that Analyzers are not well prepared to overcome certain unpleasantness.

So I was right about the failure of these tests. The test files are in CRLF format, which means that on Windows the tests do not fail. But on Linux it fails because it uses the /n format for newlines. Changing the format to LF tests pass on Linux. As the CI is now using linux-ubuntu, the tests must be failing but it does not detect the source of the failure well. I am increasingly convinced that Analyzers are not well prepared to overcome certain unpleasantness.

Interestingly, the Analyzers unit tests are not failing here (lines 183-294):
https://github.com/gui-cs/Terminal.Gui/actions/runs/9830843782/job/27137474915

I confirm that the jobs are not failing.
But why doesn't it pass in VS2022. It's not just problems with my configuration because it's happening to you too.

I confirm that the jobs are not failing. But why doesn't it pass in VS2022. It's not just problems with my configuration because it's happening to you too.

I dunno. something with WSL?

I dunno. something with WSL?

Maybe because even running the test directly in WSL also fails. Naturally there is some configuration needed in WSL to map /r/n to /n automatically. But I have no idea. We already know that WSL uses many Windows dependencies and naturally is not separating them properly, just as the dockerfile does because it is independent of the host OS.

The reason unit tests from the Analyzers project are not failing to execute actions is because the repository files are automatically written with the LF format on new lines when it is pushed to the respective operating system.
In Visual Studio 2022 it is using WSL to run unit tests but using the Windows 11 directory (/mnt/{drive}/{repo}) whose newline format is CRLF. Ideally, Visual Studio 2022 would first push the code to WSL, thus transforming the format of new lines as LF, and then run the unit tests. I wonder if this is possible.

This dude right here, absolute mad lad 😎

@Mersid did you already test this PR? Say if it's working with you. Thanks.

@dodexahedron - Can we please get your review of this?

It's failing again because the JETBRAINS_ANNOTATIONS.

I have to remove JETBRAINS_ANNOTATIONS from the Terminal.Gui project to avoid this error.
╰─ .\SelfContained.exe
Unhandled exception. System.IO.FileNotFoundException:
File name: 'JetBrains.Annotations, Version=4<IP_ADDRESS>, Culture=neutral, PublicKeyToken=1010a0d8d6380325'
   at System.ModuleHandle.ResolveTypeHandle(Int32, RuntimeTypeHandle[], RuntimeTypeHandle[])
   at System.Reflection.RuntimeModule.ResolveType(Int32, Type[], Type[])
   at System.Reflection.CustomAttribute.FilterCustomAttributeRecord(MetadataToken, MetadataImport& , RuntimeModule, MetadataToken, RuntimeType, Boolean, ListBuilder`1&, RuntimeType& , IRuntimeMethodInfo& , Boolean& )
   at System.Reflection.CustomAttribute.AddCustomAttributes(ListBuilder`1&, RuntimeModule, Int32, RuntimeType, Boolean, ListBuilder`1)
   at System.Reflection.CustomAttribute.GetCustomAttributes(RuntimeModule, Int32, Int32, RuntimeType)
   at System.Reflection.CustomAttribute.GetCustomAttributes(RuntimePropertyInfo, RuntimeType)
   at System.Reflection.RuntimePropertyInfo.GetCustomAttributes(Type, Boolean)
   at System.Attribute.InternalGetCustomAttributes(PropertyInfo, Type, Boolean)
   at System.Attribute.GetCustomAttributes(MemberInfo, Type, Boolean)
   at System.Attribute.GetCustomAttribute(MemberInfo, Type, Boolean)
   at System.Attribute.GetCustomAttribute(MemberInfo, Type)
   at System.Reflection.CustomAttributeExtensions.GetCustomAttribute(MemberInfo, Type)
   at Terminal.Gui.ConfigurationManager.<>c.<Initialize>b__48_8(PropertyInfo prop) in E:\Repos\CSharp\gui-cs\Terminal.Gui\Terminal.Gui\Configuration\ConfigurationManager.cs:line 526
   at System.Linq.Enumerable.Any[TSource](IEnumerable`1, Func`2)
   at Terminal.Gui.ConfigurationManager.<>c.<Initialize>b__48_2(<>f__AnonymousType1`2 <>h__TransparentIdentifier0) in E:\Repos\CSharp\gui-cs\Terminal.Gui\Terminal.Gui\Configuration\ConfigurationManager.cs:line 524
   at System.Linq.Enumerable.WhereSelectEnumerableIterator`2.MoveNext()
   at Terminal.Gui.ConfigurationManager.Initialize() in E:\Repos\CSharp\gui-cs\Terminal.Gui\Terminal.Gui\Configuration\ConfigurationManager.cs:line 533
   at Terminal.Gui.ConfigurationManager.Reset() in E:\Repos\CSharp\gui-cs\Terminal.Gui\Terminal.Gui\Configuration\ConfigurationManager.cs:line 342
   at Terminal.Gui.ConfigurationManager.get_Settings() in E:\Repos\CSharp\gui-cs\Terminal.Gui\Terminal.Gui\Configuration\ConfigurationManager.cs:line 163
   at Terminal.Gui.ConfigurationManager.Load(Boolean reset) in E:\Repos\CSharp\gui-cs\Terminal.Gui\Terminal.Gui\Configuration\ConfigurationManager.cs:line 269
   at Terminal.Gui.Application.InternalInit(ConsoleDriver driver, String driverName, Boolean calledViaRunT) in E:\Repos\CSharp\gui-cs\Terminal.Gui\Terminal.Gui\Application\Application.cs:line 248
   at Terminal.Gui.Application.Run[T](Func`2 errorHandler, ConsoleDriver driver) in E:\Repos\CSharp\gui-cs\Terminal.Gui\Terminal.Gui\Application\Application.cs:line 702
   at SelfContained.Program.Main(String[]) in E:\Repos\CSharp\gui-cs\Terminal.Gui\SelfContained\Program.cs:line 13

It's failing again because the JETBRAINS_ANNOTATIONS.

Yep. I mentioned that somewhere. All that constant does is exactly that - sets the condition that makes them transitive.
It's fine to remove it, especially since I would have ended up taking it out in #3558 anyway.
I thought I did already take it out in one of the PRs that just merged though?

I thought I did already take it out in one of the PRs that just merged though?

See this comment https://github.com/gui-cs/Terminal.Gui/pull/3600/files#r1675696417 please.

I removed the Terminal.Gui.Analyzers.Internal project from the Terminal.Gui project, deleted the Directory.Build.* and added the JetBrains.Annotations PackageReference to the UICatalog and UnitTests projects. Here the commit https://github.com/gui-cs/Terminal.Gui/pull/3582/commits/abdde3ab6c59b9ed66a5d21530d1bcdd1353f64d.

@BDisp Ready for me to merge this now?

@BDisp Ready for me to merge this now?

Yes.

I could also suggest publishing an update in the nuget and then testing in the Release version with the new version, if possible. Thanks.

I removed the Terminal.Gui.Analyzers.Internal project from the Terminal.Gui project, deleted the Directory.Build.* and added the JetBrains.Annotations PackageReference to the UICatalog and UnitTests projects. Here the commit abdde3a.

A rebase on v2_develop after the merges would have taken care of that too.

I could also suggest publishing an update in the nuget and then testing in the Release version with the new version, if possible. Thanks.

@tig @bdisp @tznind
For anyone wanting to test nuget packaging:
You can dotnet pack locally and just add a local nuget source so you can install the package via standard nuget workflows. Then, multiple projects can make use of it and it doesn't disappear on you. It's the way you test before deploying/publishing to nuget.org or another public repository.
To set up a local feed, just make two directories in a common path outside of any git repos (this only needs to be done once):

One for the output of dotnet pack -o path/to/pack/output/directory for any projects you build packages from.
One to be your local nuget repository feed (path/to/feed/directory).

To update your feed with your latest builds, run nuget init path/to/pack/output/directory path/to/feed/directory
To use the feed, just add that feed as a nuget source.
Note that our current nuget.config in the repository root will nuke your custom feed because of the clear element, so either remove that line or just add your feed in that file instead.
Sourcelink warnings or errors are resolved by adding a mapping for your feed.
I prefer to do it by just explicitly specifying a custom nuget.config file on the command line when testing the nuget restore/dotnet restore, so it only uses the local feed when I explicitly want it to and doesn't affect anything otherwise.

A similar but simpler procedure can also be used to set up a local powershell gallery, for ease of installing/importing powershell modules without needing to navigate to the containing directory.
I have several such custom local psrepositories on all of my dev machines, as well as PSDrives for solution directories as PSDrives that are created when powershell launches, as part of my powershell profile.

Hm... You know what...
Yeah, a LOT of stuff reverted after that, including stuff from other PRs.
There's a strong possibility of regressions beyond just the project references and that PR.

About the root cause of that sort of thing:
Almost definitely happened during merge conflict resolution on a branch.
Be extra careful to analyze conflicts that are not just edits on each side, and are instead add/edit/remove on one side and a different operation on the other side.
Those merge errors are a lot easier to catch with proper rebasing/merging procedures.

Oh yeah.
There's also https://int.nugettest.org/ for when you want to test it over the internet but not live on nuget.org.

To testing nuget packages locally I use the following:
In the nuget.config:
<add key="Terminal.Gui" value="{Drive}:\{Path}\Terminal.Gui\Terminal.Gui\bin\Release" />
I created a Package.ps1 file with:
$Version = "2.0.0"

# Delete previous package
rm -r $env:USERPROFILE\.nuget\packages\terminal.gui\$Version

# Install dependencies
dotnet restore

# Build Release
dotnet build --no-restore -c Release

# Pack
dotnet pack -c Release --include-symbols -p:Version=$Version

Yep. That's the way. 🙂 At least to pack it. Still need to test the install/restore and build in both visual studio and dotnet cli afterward.
Side note:
Another option for anyone who doesn't want to bother with the nuget.config is you can call dotnet restore --source path/to/directory/containing/the/package --force --force-evaluate to restore from just that path, ignoring other sources. You can specify --source multiple times in the command, if you want to also have it consider other sources, too.


feat(integrations): add fastify support

TODO:

[x] Update CHANGELOG.md with your change (include reference to issue & this PR)
[x] Make sure all of the significant new logic is covered by tests
[x] Rebase your changes on master so that they can be merged easily
[x] Make sure all tests and linter rules pass

Hey @phra, thanks a lot for your PR! We really appreciate all the hard work you put in, but on further consultation with the team we've come to the conclusion that fastify isn't yet established enough to warrant including it in the core repo, so I'm going to close the PR for the time being. However, we'll be happy to revisit this decision in the future should anything change. In the meantime you should be able to host the fastify package on your own github and release it on npm. I think all the core functionality needed from graphql-server are exported, except for the tests (you could just copy those for now).
Thanks again, and let us know if we can help with anything!

i hope to find time to do it. right now i'm working on some R&D  tasks for a brand new OSS project. 👍

I think this deserves re-opening as Fastify has since seen a lot of improvements and ecosystem growth. Recently version 1.0 was released, finalizing and improving much of the API. It has many features that Express or other frameworks lack (HTTP/2 for instance) while having extremely low overhead and deserves to be in the core Apollo-server repo in my opinion.


syncthing: 1.1.1 -> 1.1.3

Motivation for this change
Upstream release
Things done

[X] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[X] Assured whether relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

tested with nix-review, syncthing runs, but syncthing-gtk throws:
[nix-shell:~/.cache/nix-review/pr-61314-1]$ syncthing-gtk
Traceback (most recent call last):
  File "/nix/store/czb6i7j5bv0602djxz478yqbbh0h9kcq-syncthing-gtk-0.9.4/bin/..syncthing-gtk-wrapped-wrapped", line 38, in <module>
    from syncthing_gtk.app import App
  File "/nix/store/czb6i7j5bv0602djxz478yqbbh0h9kcq-syncthing-gtk-0.9.4/lib/python2.7/site-packages/syncthing_gtk/app.py", line 9, in <module>
    from gi.repository import Gtk, Gio, Gdk, GLib, GdkPixbuf
  File "/nix/store/fdb5b6s3ns0mc9m843qpjg4j9l7d785i-python2.7-pygobject-3.32.1/lib/python2.7/site-packages/gi/importer.py", line 140, in load_module
    raise ImportError(e)
ImportError: Typelib file for namespace 'Pango', version '1.0' not found

@Lassulus I get that same error, but it is not a regression due to this PR. Building with syncthing-1.1.1 also has that issue.

Indeed, well syncthing-gtk needs some updating anyway.
thanks!


V1.0.0

Version 1.0.0
New features

Prepared for PersistentData features (not used so far)

Improvements

Requires userlevel "Service" to provide UI

Bugfix

Download of first preselected file did not work after app reboot

Following tasks were checked

[x] Module is coded and structured according to the "Developing guideline for modules" described within the CSK documentation
[x] All functions/events/parameters are documented within the manifest documentation
[x] The manifest description of the main CROWN includes main information about the purpose of the module and how to use it in general
[x] API docu based on the manifest was created and stored within the "docu"-folder of the repository
[x] Internal LUA code documentation exists for variables and non served functions
[x] All relevant infos are logged via the SharedLogger 'ModuleLogger'
[x] Module supports persistent data feature based on 'CSK_Module_PersistentData'
[x] Module supports user management based on 'CSK_Module_UserManagement'
[x] No open "ToDos" within the code or at least clearly explained comments why they exist...
[x] "Version" key in app manifest was updated following semantic versioning (and use '0.x.y' for test / experimental modules which are not yet ready to be officially released)
[x] Meaningful IDs used for UI elements
[x] Module was tested on an AppSpace device (at least on SICK AppEngine) with no error message
[x] README.md is up to date (incl. info of device + firmware the module was tested with)
[x] CHANGELOG.md is up to date

@kogliinSICKAG Thx for your review! I merged the relevant parts into this repo.


Directive collision with ui-select on attribute 'sortable'

Can you prefix the sortable attribute to prevent this collision?

Sure, dt-sortable should work?

Sure, that works. =)
On Tue, Aug 4, 2015 at 10:53 AM, Austin<EMAIL_ADDRESS>wrote:

Sure, dt-sortable should work?
—
Reply to this email directly or view it on GitHub
https://github.com/Swimlane/angular-data-table/issues/39#issuecomment-127691798
.

plz test on 0.2.0?


Spike login flow change

change login flow to interject a confirmation page for profiles that could be duplicates.

The FE and CSS are good to go -  I did inline comment on a couple of things though


Add security notice about using CX at runtime to the overview page

Docs Build status updates of commit db7037b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

prosepy/docs-ref-conceptual/intro.md
:white_check_mark:Succeeded

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


(fix): Wayback machine - Upgrade to 3.0.6

Overview
Fixes https://github.com/sissbruecker/linkding/issues/256
The current version of waybackpy is no longer supported. This PR upgrades to the latest with the appropriate changes. A docker image is available for quick manual QA.

@sissbruecker I was getting some odd results attempting to run tests locally. Would you be able to trigger CI for this PR, please?

Thanks for the version bump, the tests seem to run fine, they are just a bit noisy ATM with (expected) error messages. Added a commit to reuse the user agent from the website loader.


Linting, Syntax and Unit livestream

There should be an option to have the results of the lint, syntax and unit phases streamed to the output, even when it succeeds.  Looks like it is just missing the live_stream: STDOUT

In addition to seeing the output this would allow for the coverage report for code reviews:
ChefSpec Coverage report generated...
Total Resources:   6
Touched Resources: 3
Touch Coverage:    50.0%
Untouched Resources:


Chapters for videos

Would like to have a list of timestamp links on each video, linking to different sections/chapters of the video. Similar to Rascal Two's timestamps on Youtube.

Link to Rascal's work please?

https://www.youtube.com/watch?v=qEj0pXGVwjY Second comment, by Rascal_Two. Although maybe not that exhaustive.
As you commented in #60, pretty sure we're talking about the same thing, I misread your post. In order to keep the discussion in one spot, I'm closing that one as a duplicate and we'll go from here.
The way I'm thinking of it, however, does not rely on #58. What I'm thinking of is a section on each video page that has a list of timestamp links that change the position of the video on the current page. Since the links are "internal", the implementation does not require timestamps in the query parameters of the page. Because I'm not using any CSR, that would force the page to reload.

discussion moved to #61

Argh closed the wrong one.

So, do you envision a single text input that requires a specific pattern and then parses that data to create youtube timestamp links?
Input:
20:32 - Spicy rant
39:12 - Don't tell on yourself
45:16 - JavaScript review

Output:
20:32 - Spicy rant
39:12 - Don't tell on yourself
45:16 - JavaScript review


Adjust the universal flow-level padding for narrow static-sized dimensions

This is part-solution, part-reframing for #11632. The immediate motivation is that https://github.com/nod-ai/SHARK/issues/1581 is about a model that is all vector-times-matrix matmuls, and at the moment we are padding everything to the next multiple of 16 in Flow (which is the topic of #11632). To get good performance on mat-vec we need to stop doing that.
There was some existing logic in MaterializeEncoding to adjust tile sizes to narrow static dimensions (adjustTileSizesToNarrowStaticShape) but it was framed as a local implementation detail, which prevented letting Flow take advantage of it to say: "Since MaterializeEncoding will never generate tiles greater than this along this narrow dimension, I also don't need to pad more than this".
This PR changes that into a real contract between Flow (SetEncoding) and HAL (MaterializeEncoding).
At the moment, there is an e2e matmul test failure specifically only with the VMVX backend with ukernels, with 1x1 matrices: now that they aren't padded anymore to 16x16, the ukernel is being called with the same data pointer for the LHS and RHS matrices. (See printfs intentionally left in for now). @benvanik @stellaraccident any idea?

I know this is a draft, but we this is probably not the right path. The easiest thing to do would be to add a new operation
%hi:2 = iree_linalg_ext.encoding_padding_value <encoding Attr>

and use %hi#0, %hi# as the padding value in the tensor.pad operation created.
%pad = tensor.pad %unpacked_tensor low[0, 0] high[%hi#0, %h1#1]

During MaterializeEncoding we have all the information we need to convert the %hi#0 and %hi# into a scalar value and then we will not overpad....

I know this is a draft, but we this is probably not the right path. The easiest thing to do would be to add a new operation
%hi:2 = iree_linalg_ext.encoding_padding_value <encoding Attr>

and use %hi#0, %hi# as the padding value in the tensor.pad operation created.
%pad = tensor.pad %unpacked_tensor low[0, 0] high[%hi#0, %h1#1]

During MaterializeEncoding we have all the information we need to convert the %hi#0 and %hi# into a scalar value and then we will not overpad....

Thanks @MaheshRavishankar for the suggestion. Implementing it in https://github.com/openxla/iree/pull/14349 . Closing this as superseded by that.


Question about Filters

Currently a Controller Filter have this skelleton:
<?php namespace App\Filters;

use CodeIgniter\HTTP\RequestInterface;
use CodeIgniter\HTTP\ResponseInterface;
use CodeIgniter\Filters\FilterInterface;

class MyFilter implements FilterInterface
{
	public function before(RequestInterface $request)
	{
		//
	}

	public function after(RequestInterface $request, ResponseInterface $response)
	{
		//
	}
}

What about to update for something like:
<?php namespace App\Filters;

use CodeIgniter\HTTP\IncomingRequest;
use CodeIgniter\HTTP\Response;
use CodeIgniter\Filters\FilterInterface;

class MyFilter implements FilterInterface
{
	public function before(IncomingRequest $request)
	{
		//
	}

	public function after(IncomingRequest $request, Response $response)
	{
		//
	}
}

Motive: We will can use all methods of the IncomingRequest and Response. I think that the interfaces only are a little limited in this place.

You can still use all of the methods in the IncomingRequest and Response class as is. By specifying the Interfaces, it just means that ANY class that implements that interface can be passed in. This is needed to make sure a Filter can be used for CLI requests, also. Or that you can extend the IncomingRequest class to customize something, and still have that work.

Hahah, of course.
This is what happens when one depends a lot on code completion.
I was trying here but the methods that appear are just from the RequestInterface. But the IncomingRequest class are there, too.

Many thanks, @lonnieezell .

Yeah, I thought it might have been something like that :) No worries.


Status of demes-slim

Hi Graham,
I'd like to use a model in the demes format with slim (in my case, I can ignore migration). Is demes-slim mature enough to use it for this purpose?
Thanks,
Vince

Hi Vince,
I suspect that this has suffered some bitrot. The demes-slim project was never polished, in large part because there didn't seem to be anyone interested in using it (you're the first person that has asked about it!). It was targeting SLiM v3.7 or so, and should work with that version (subject to the open issues regarding migrations/pulses). Some small fixes are likely needed to work with multispecies versions of SLiM because various object properties got shuffled around (SLiM >= 4.0?).
In any event, it probably would be a good idea to look at output from the DRY_RUN option to the initialise.slim file, which prints the script blocks that are scheduled to run and then exits. Honestly, I was never really happy with how the Demes events were mapped to SLiM generations (I forget the details, but some things like migrations happen only in the generation after you ask for them), but it should be fine unless you're doing weird things with rescaling and also have short time intervals between events.


Teradici+Windows no Blender

When installing the CFN using public VPC/Teradici/Windows, Blender is not installed.

solved


Add memory constraints in conftest and change unit logging to debug in test models

Issue

We are not setting any constraints on the mysql units (which results in mysql setting innodb_buffer_pool_size from calculations of the entire host machine). This may lead to some failures (OOM) in integration tests
We are not printing out debug log in the integration tests, making it very difficult to see what was happening when a test failed

Solution

Set a memory constraint of 1G
Print unit debug logs in the output

Closing, due to the introduction of a testing profile in a separate PR


Add capfruit, a gatfruit mutation.

About the PR
This PR adds two gatfruit mutations; fake capfruit and real capfruit. While they both look identical, they have equal chance of mutating with one giving harmless "toy" capfruit when eaten; and the other with the syndicate item.
Why / Balance
It's always fun when botany ends up making a bunch of pythons and spreads it around the station. This will hopefully make gatfruit a more viable syndicate purchase and cause more confusion from security as they're struggling to find out who has contraband and who has a bunch of harmless toys. It will definitely put them on edge. Also, this technically introduces botany gambling considering they have to try to roll the correct plant with rng.
Media

Requirements

[X] I have read and I am following the Pull Request Guidelines. I understand that not doing so may get my pr closed at maintainer’s discretion
[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog
:cl: Ubaser

add: Add two gatfruit mutations, one fake and one real capfruit which spawns cap guns when eaten.

this is just... evil

This stands for Reduced Metagaming Mechanics this PR https://github.com/space-wizards/docs/pull/214
You can make two mutated capfruits, toy and real one - "toy" capfruits you give to sec to check, and thay say "It's okay", another one you use by yourself or give your teammate, or just flood the station with "real one guns"

All these approves yet no one merges it


Add --override to env targets cli

To bring them in line with env set and env add operations on override environments.
closes #857

Pull Request Test Coverage Report for Build 1342

14 of 14 (100.0%)  changed or added relevant lines in 4 files are covered.
4 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.01%) to 70.487%

Files with Coverage Reduction
New Missed Lines
%

pkg/app/base_app.go
4
63.95%

Totals

Change from base Build 1339:
-0.01%

Covered Lines:
12546

Relevant Lines:
17799

💛  - Coveralls


:bug: add proper path to pythonpath if step is a module

With this, you'll be able to import files without relative imports. Absolute imports let you use VSCode code interpreter.

Maybe revert the changes in population, or otherwise it will trigger a bit ETL run

Thanks for spotting it, that was close 😅


comment not working , going live issue

Issue Type: Feature Request
comments like '//' and /* not working in my vs code and also i am facing issue in going live
VS Code version: Code 1.55.2 (3c4e3df9e89829dce27b7b5c24508306b151f30d, 2021-04-13T09:35:57.887Z)
OS version: Windows_NT x64 10.0.19042

Do you mean the colors aren't updating when you comment out code? Can you describe what's happening in more detail, and what programming language you're using?


Removes redundant sparkleOn

miqAjaxButton already includes the sparkle!

@miq-bot add_label euwe/yes

Agreed completely (took the changes right out of my mouth ;)) :+1:
Merging..

Euwe Backport details:
$ git log -1
commit 39955b79342dc677009ab634af7747c6b85fd8dd
Author: Martin Hradil<EMAIL_ADDRESS>Date:   Fri Oct 14 08:40:03 2016 +0000

    Merge pull request #11898 from gildub/remove-some-sparkle

    Removes redundant sparkleOn
    (cherry picked from commit 92a6c872eeaed53834a21914547482da5c0fe81b)


bug: onClick override href attribute behavior when specified

Currently, if one were to specify a onClick handler inside an a tag that also has an href attribute, the onClick would override the link redirection behavior. It appears for a tags, event.preventDefault() is being used by default, and we should explicitly make the developers call this themselves. Also, one other thing to note, no matter where you put the onClick, in a parent tag, or in a child tag, the onClick will override any link redirection.
This might go into the territory of breaking changes for all the widgets out there, but more than likely not since this is just constrained to an a tag.

@ChaoticTempest Could you please describe it more precisely and provide an example?

For example, the following widget when you click on it will not send you to the profile page, but only logs the event:
const profileUrl = `/#/calebjacob.near/widget/ProfilePage?accountId=chaotictempest.near`;

return (
   {
      console.log("event", event);
    }}
  >
    Click

);

I believe this is due an event.preventDefault() being inserted and used somewhere inside the VM which prevents navigating to the new page.

I think this is the culprit
https://github.com/NearSocial/VM/blob/9d7ecff7bf50a114a59306b79f7d5a09706f06fe/src/lib/vm/vm.js#L1354

If anyone else runs into this issue, a creative workaround is to use the onPointerUp event (which fires for a mouse and touch):
 console.log("asdf")}>Click

@ChaoticTempest I double checked, and onMouseDown doesn't actually seem to fire on mobile after all, but onPointerUp should be just what you need.

The original issue was addressed in 2.5.0.


String Compatibility in Send Message

As a user of the message sending interface, I would greatly appreciate the ability to use strings to send information downstream. This allows for enhanced information between subsystems without having to translate down to floats or int, where info could be lost

I will be looking at adding this next week. Stay tuned.

We found a solution for our purpose using bitshifts and ints but this would be much simpler

Functionality has been added, closing this issues. Please message me if any bugs occur that I may have missed.


Consider adding http://www.apews.org

http://www.apews.org/?page=test&C=813&E=453575&ip=<IP_ADDRESS>

Seems outdated/unreliable.


Enforcing Pod Security Standards For CAPM3

I am currently working on adding a new feature of enforcing security standards for CAPI currently set to restricted level in the following PR
https://github.com/metal3-io/cluster-api-provider-metal3/pull/950
Regarding this PR, where should the security definitions go i.e. in some kustomization or directly applied from the manifest in the root?  Is there any specific way such file should be applied to the repository? Any other comments regarding this would also be appreciated.

Hi! This seems to be a question rather than a bug. To get help, please contact us on Slack: https://metal3.io/community-resources.html. You can also stop by our weekly meeting: https://groups.google.com/g/metal3-dev/c/sAsc9msfRZo.


fix: do not pass in ember-cli's registry object into Less compiler.

Resolves #65
Avoids passing in ember-cli's registry into the Less compiler.  Not needed and breaks when Less tries to access it.

I don't believe my changes are responsible for CI failing.  Let me know if you think otherwise.


Check strictly increasing values when clipping contrast limits to a new range

Description
When we change contrast_limits_range, we clip contrast_limits to fit in that new range. If the current limits have no real overlap with the new range, then we set them to the new range. We previously had some logic to handle that, but missed the case when the overlap between the current limits and new range had no length (i.e. the clipped limits were equal and therefore not strictly increasing). In this PR, I fixed and simplified that logic to make it clearer.
The example described in #5257 shows one way of reaching this buggy case, though it is caused by some fairly complex logic around how we automatically determine contrast_limits on Image initialization. In short when passing any data that is not an ndarray without contrast_limits, we set the contrast_limits to be [0, 1] if the inferred dtype is not an integer. In that case, the data had a min of -1 and a max of 0, so when automatically calculated the limits on slicing, we failed.
I added a test to cover the example in #5257, then added a finer grained unit test to check what happens when setting a new range around the boundary of the current limits.
Type of change

[x] Bug-fix (non-breaking change which fixes an issue)

References
Closes #5257
How has this been tested?

[ ] added tests to cover buggy behavior pass
[ ] all tests pass with my change

I added the v0.4.17 milestone as this fixes a regression from v0.4.16. Alternatively, we could revert #5036 in the v0.4.17x branch, though I'll need to check there are so side-effects of that.


chore: fork sitemap generator

Renames this module to be scoped to the BBC and gives it a base version.

I made this PR against the wrong base, apologies.


Curriculum/lesson page is broken

Going to https://www.c0d3.com/curriculum/2 results in error screen. Something changed Session type so it can return null under submission field hence _.get(session, 'submissions', []) does not work anymore.

TLDR: Our typescript types don't work because of manual type assertions X as Y and X:Y.
We switched from incorrect manual types to almost useless codegen ones, case in point pages/curriculum/[lesson].tsx file:
 const userSubmissions: Submission[] = (_.get(session, 'submissions', []) as Submission[])

This breaks production but it passes typescript checks because we manually added type instead of letting typescript to infer it. I'm not really know where to start with fixing it, initial typedefs are still not strict enough.

Not causing issues atm, but part of overall problem to fix types.

replaced by #841


l10n: Update Spanish translation, part 1

Translations made using wiki's Localization guide.
Part 1 of many, which I will be adding in separate PRs.

Thanks for the contribution!
I rill review it after 7.3.2 release.

Thanks for the contribution!
I rill review it after 7.3.2 release.

Merge it now to avoid future merge conflicts as we going to do several refactorings.
I'll make Early Access version later today. Please verify the translation in UI.

Merge it now to avoid future merge conflicts as we going to do several refactorings.
I'll make Early Access version later today. Please verify the translation in UI.

@serge-rider, are you pushing a new Early Access version for the Community Edition? I can see a new Enterprise Edition version was submitted in https://dbeaver.com/files/ea/update/dbeaver/, but not yet for the Community Edition in https://dbeaver.io/files/ea/.

@serge-rider, are you pushing a new Early Access version for the Community Edition? I can see a new Enterprise Edition version was submitted in https://dbeaver.com/files/ea/update/dbeaver/, but not yet for the Community Edition in https://dbeaver.io/files/ea/.

@adamantike Please check CE EA now https://dbeaver.io/files/ea/.

@adamantike Please check CE EA now https://dbeaver.io/files/ea/.

Please ask to reopen if something was not applied

Please ask to reopen if something was not applied

Everything looks good, thank you!

Everything looks good, thank you!


whats default username and password?

Good work, as always. Whats default username and password?

There is no default username and password; just register, you'd get an email to continue

As a workaround you can register and modify EmailConfirmed property of a user directly in DB

Email confirmation works without manually modifying the DB


Add a new comman-line entrypoint symver-smap

This is useful specially for distributions where the name 'smap' conflicts with existing applications.
This also makes the name of the command-line application to be used in the output files instead of the hard-coded name 'smap'. The tests have been updated accordingly.

Pull Request Test Coverage Report for Build 149

3 of 4 (75.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.2%) to 98.293%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/smap/main.py
2
3
66.67%

Totals

Change from base Build 142:
-0.2%

Covered Lines:
731

Relevant Lines:
732

💛  - Coveralls

Codecov Report

Merging #73 into master will decrease coverage by 0.26%.
The diff coverage is 50%.

@@            Coverage Diff             @@
##           master      #73      +/-   ##
==========================================
- Coverage   97.53%   97.26%   -0.27%
==========================================
  Files           4        4
  Lines         729      732       +3
  Branches      187      188       +1
==========================================
+ Hits          711      712       +1
- Misses          0        1       +1
- Partials       18       19       +1

Impacted Files
Coverage Δ

src/smap/symver.py
97.48% <100%> (ø)
:arrow_up:

src/smap/main.py
81.81% <33.33%> (-18.19%)
:arrow_down:

Continue to review full report at Codecov.

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


World save/load improvement

According to the, https://github.com/egordorichev/LastTry/pull/40 we must impove save system for the world. Added to todo list https://trello.com/c/SG1TeEc6/26-make-world-file-size-smaller-a-ton.
Probably, we should implement chunks system. If we will use chunks method, will we need to create infinite worlds?

I've been wondering, why is the world stored in memory as a linear array rather than just making it two dimensional so blocks can be accessed directly with their x and y?

Because to alloc 2d array you need to call new width times. But when you use 1d array, you need only one alloc call.

And in the new system I plan to use the same technic, but splited into chunks. But will help you, we always can make a array wrapper.

A 2d array in java is done like this:
int[][] data = new int[height][width]

So here is the development plan:

[ ] 1D array class wrapper If we need it
[ ] Chunk Manages constant amount of blocks
[ ] ChunkProvider Loads/saves chunks, updates corruption
[ ] ChunkManager Stores all of the chunks, calls Chunk Provider if needed
[ ] Update the world class

Also, will we need infinite worlds?

Apparently terraria doesn't have an infinite world because they keep the entire map in memory so that they can make trees grow and stuff so we don't really need one but I think that we could have one if we wanted.
Instead of keeping all of the map in memory we could just increase a time variable on each chunk that isn't loaded and when the chunk gets loaded it updates everything that changes with time like trees.

Ok, sure. Also, we should flip the y-axis of the game logic. When we moved to libgdx, we inverted the drawing y-axis, but not the logic one. So now, the world 0:0 should be at bottom right for simplicity of calculations.

Done in https://github.com/egordorichev/LastTry/pull/58 .


Support Symfony 4.4

The 7.2 version is unusable for Projects depending on Symfony LTS (4.4).
Can you maybe consider supporting Symfony 4.4 and 5?
https://github.com/ongr-io/ElasticsearchDSL/blob/7.2/composer.json#L15

v5 is already supported in 7.2.2:
https://github.com/ongr-io/ElasticsearchDSL/blob/c0789c35e8738c2b1138c8d33ec9fbcd740c909d/composer.json#L15


how to use the php version of this starter template?

Hello,
Im trying to integrate Codyframe with Kirby by following this https://codyhouse.co/ds/docs/framework#gulp-php however after changing the gulp.js file to this one and changing the index.html file to index.php there is nothing that appears in the browsersync.
could someone please let me know is the gulp.js is correct?

Hi there, could you please provide more info about the issue? For example, do you get an error message in the console, is your project page showing up but not refershing? Also, could you share the structure of your project? Thank you.

I'm closing this issue since no additional info was provided. Feel free to reopen it if you are still having problems. Cheers!


step3: add project status check target

Codecov Report

Merging #1 (f1635b2) into main (642acca) will increase coverage by 39.65%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             main        #1       +/-   ##
============================================
+ Coverage   60.34%   100.00%   +39.65%
============================================
  Files           3         2        -1
  Lines          58        38       -20
============================================
+ Hits           35        38        +3
+ Misses         23         0       -23

Impacted Files
Coverage Δ

api/tests/test_calculator.py
100.00% <100.00%> (ø)

api/app.py

api/calculator.py
100.00% <0.00%> (+9.09%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Add navbar with static pages

Is your feature request related to a problem? Please describe.
currently, there is no navbar on the top of the application for the web app and android app of the project. we can add a navbar with logo and link to static pages like about us, contact, etc.
screenshots
web app

Android through chrome inspect

Describe the solution you'd like
make a new navbar/header component and use that for all pages.

@KeenWarrior @Abhishek-kumar09 can I work on this?

@Abhijay007 The app is more for mobile, We can add some web functionalities once the mobile 1st version is completed. I guess bottomnavbar will suffice all the requirements for mobile screens.
There will be some screens which will not be having bottomnavabar, there we would be adding topbar.
The topbar's code is already here: https://github.com/codeforcauseorg/edu-client/tree/main/src/layouts/MainLayout/TopBar
It needs only alterations adhering to the currenting application needs.

@Abhijay007 The app is more for mobile, We can add some web functionalities once the mobile 1st version is completed. I guess bottomnavbar will suffice all the requirements for mobile screens.
There will be some screens which will not be having bottomnavabar, there we would be adding topbar.
The topbar's code is already here: https://github.com/codeforcauseorg/edu-client/tree/main/src/layouts/MainLayout/TopBar
It needs only alterations adhering to the currenting application needs.

ok, no problem, and thanks for letting me know about it. I will try to focus more on the mobile version from now and try to contribute to that. you can close this issue.

Thanks @Abhijay007 , We will be first completing the mobile view because the motive is to create a cross platform hybrid application with capacitor, once done will be focusing on adding some extra features to make it compatible for Web versions too.
Thanks for opening the issue.


Add UI_APPEARANCE_SELECTOR to linkAttributes, activeLinkAttributes and inactiveLinkAttributes properties.

I'd like to add UI_APPEARANCE_SELECTOR to linkAttributes, activeLinkAttributes and inactiveLinkAttributes properties. I'm active user of TTTAttributedLabel and it's a bit strange, that I have to overwrite default settings for these properties in every place I'm using TTTAttributedLabel.
Are you ok we such improvement? If yes, I'll create a PR with changes/test.
Thanks

This library doesn't currently support the appearance proxy, but a PR to add that would be welcome.


Add missing fusion-cli test fixtures

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


OAK-9111: oak default analyzer for elastic

The current test verifies that query uses the provided analyzer. Can we also add test for the change in ElasticIndexHelper ? There is already a test class for that.


two bug fixes

fixed https://github.com/vmalyi/adb_android/issues/23
fixed print statement for python3

also, is there something wrong with the Travis CI? It fails for some reason.

Awesome, thanks!


DND Apply Function Signature from TypeDef

Is your feature request related to a problem? Please describe.
I'm always frustrated when I lookup a typedef for an unnamed function and then have to clear the filter, wait for the tree to finish, right click on the typedef, choose show base type, wait and then drag the function type onto the function in the listing.
Describe the solution you'd like
It would be nice if the function signature could be applied from the typedef. This should only apply if the typedef is to a FunctionDefinition and not a pointer to one.
Describe alternatives you've considered
See initial description.
Additional Context
Iirc it currently attempts to set the return type to the typedef and then fails because it the type has no size.

Great suggestion!

Great suggestion!

Yes

Great suggestion!

So it appears that attempting to create a typedef to a FunctionDefinition through the ui results in a typedef to a pointer to said FunctionDefinition. I honestly would like to be able to apply the definition even if the typedef is a function pointer. It seems weird to me that attempting to drag a type onto a function name results in setting the return type unless the type is a FunctionDefinition.
@dev747368 the typedefs in question were the result of DWARF information. I'm thinking that the parser should have created a typedef to a pointer to the FunctionDefinition. It does use a pointer to that typedef where used though.


Multiple models

This pull request allows for the successive loading of pre-trained models. The use case looks like this:
from saber import Saber

sb = Saber()

# load as many or as little models as you want
sb.load('CHED')
sb.load('PRGE')
sb.load('DISO')

ann = sb.annotate('some text)

When annotate is called, every avaliable model is used to annotate the text, and the resulting annotations are hamonized and returned in a single dictionary (ann).

TODO

[ ] Check that Saber.save() still works (I suspect it won't).
[ ] Try to refactor how multiple models works under the MultiTaskLSTMCRF class.
[ ] As always, write unit tests for any new functionality.
[ ] Consider writing Saber.load() such that it can accept a list of arguments.

Pull Request Test Coverage Report for Build 338

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

Totals

Change from base Build 336:
0.0%

Covered Lines:

Relevant Lines:
0

💛  - Coveralls


Page loading indicator covers the window manager

Describe the bug
The page loading indicator covers the window manager tabs
To Reproduce

Open up any window manager, for instance a shell in to a kube cluster
Navigate anywhere the page loading indicator is shown

any page in the explorer that loads a resource not seen before (tweaking the dev tools network tab connection speed may help to increase the time the loading indicator is shown)

Result

Note - From memory there are different contexts of the loading indicator. Some probably should appear over the window manager (global)... some might not (context).

Validated on Rancher version: v2.10-5d66d49d7a72b4c439a33043007e9bb0223c8693-head
UI: master cf8c6d1
This isn't reproducible on this Rancher version, I've tried throttling the network speed and the adding resources to trigger a slow loading of a page after opening a pod's logs.


Typings error when using onEnter in state declaration

I'm creating a state in Typescript like this:
 state: {
            name: 'mainState.import',
            url: '/import',
            reloadOnSearch: false,
  onEnter: (['$uibModal', (uibModal: ng.ui.bootstrap.IModalService) => {
                let modalOptions: any = {
                    headerText: 'Import State goes here'
                };
                ...
                uibModal.open(modalConfig);

            }])
}

While it works fine I get a typescript compilation error:
[0] (14,14): error TS2322: Type '({ name: string; icon: string; keyboardShortcut: string; state: { name: string; component: string...' is not assignable to type 'ISidebarPage[]'.
[0]   Type '{ name: string; icon: string; keyboardShortcut: string; state: { name: string; component: string;...' is not assignable to type 'ISidebarPage'.
[0]     Type '{ name: string; icon: string; accessibleForRoles: string[]; state: { name: string; url: string; r...' is not assignable to type 'ISidebarPage'.
[0]       Types of property 'state' are incompatible.
[0]         Type '{ name: string; url: string; reloadOnSearch: boolean; onEnter: (string | ((uibModal: IModalServic...' is not assignable to type 'StateDeclaration & { component?: string; template?: string; }'.
[0]           Type '{ name: string; url: string; reloadOnSearch: boolean; onEnter: (string | ((uibModal: IModalServic...' is not assignable to type 'StateDeclaration'.
[0]             Types of property 'onEnter' are incompatible.
[0]               Type '(string | ((uibModal: IModalService) => void))[]' is not assignable to type 'TransitionStateHookFn'.
[0]                 Type '(string | ((uibModal: IModalService) => void))[]' provides no match for the signature '(transition: Transition, state: State): boolean | TargetState | void | Promise<boolean | TargetState | void>'

Is this some sort of bug with the typings?

that should be fixed by #3076
What version of angular-ui-router are you using?

I'm using 1.0.0-rc.1 which should be the latest right? And I still see the error

Yep rc.1 is latest.  I'll take a look and see if there's still a problem

#3076 did not add typings for inline-annotated functions - those are still missing


Openapi emitter should fail on duplication operationId

It is possible for operations in different namespaces but the same "service" to have the same operationId. This is an error in OpenAPI. The openapi3 emitter should detect this and flag it as an error in the Cadl file.
Playground repro
Alternate repro

est: 3


Only 2 entities

Hi Lukas,
looks like I couldn't integrate SWS12500 correctly, because I'm getting only 2 entities, with unknown status :(

On a HA CZ site I saw a test via browser:
http://<IP_ADDRESS>:8123/weatherstation/updateweatherstation.php?ID=xxxxxx&PASSWORD=xxxxxx&action=updateraww&realtime=1&rtfreq=5&dateutc=now&baromin=29.99&tempf=22.2
this gives back the "OK" string.
Could you please help?
Regards

@bgeza What is your station's firmware? In 1.0.0 firmware is a bug.
Can you see any errors in HA's logs?

Hi... the fw is 2.0.6
As for the log, only a few related rows, no indication of error:
2024-06-01 02:46:34.286 WARNING (SyncWorker_0) [homeassistant.loader] We found a custom integration sws12500 which has not been tested by Home Assistant. This component might cause stability problems, be sure to disable it if you experience issues with Home Assistant
2024-06-01 02:46:38.485 INFO (MainThread) [homeassistant.bootstrap] Setting up stage 2: {'energy', 'weather', 'google_translate', 'asuswrt', 'trace', 'smartlife', 'anniversaries', 'lovelace', 'script', 'input_number', 'scene', 'intent', 'conversation', 'onvif', 'default_config', 'mobile_app', 'zha', 'ffmpeg', 'backup', 'hikvision_next', 'var', 'input_datetime', 'stream', 'google_assistant', 'browser_mod', 'person', 'multiscrape', 'blueprint', 'input_boolean', 'dlna_dms', 'zone', 'wake_on_lan', 'stt', 'tts', 'media_player', 'ariston', 'adguard', 'history', 'config', 'diagnostics', 'upnp', 'logbook', 'my', 'sws12500', 'schedule', 'tag', 'input_select', 'input_text', 'hikconnect', 'systemmonitor', 'number', 'hardware', 'system_health', 'google', 'mqtt', 'homeassistant', 'midea_ac', 'sensor', 'analytics', 'filesize', 'cast', 'persistent_notification', 'media_source', 'switch', 'image_upload', 'file_upload', 'wake_word', 'rpi_power', 'counter', 'tod', 'select', 'raspberry_pi', 'automation', 'fully_kiosk', 'webostv', 'panel_custom', 'map', 'spook', 'onboarding', 'timer', 'sun', 'hacs', 'assist_pipeline', 'homeassistant_alerts', 'input_button', 'device_automation', 'application_credentials', 'search'}
2024-06-01 02:46:53.289 INFO (MainThread) [homeassistant.setup] Setting up sws12500
2024-06-01 02:46:53.289 INFO (MainThread) [homeassistant.setup] Setup of domain sws12500 took 0.00 seconds
2024-06-01 02:46:53.290 INFO (MainThread) [custom_components.sws12500] Registered path to handle weather data: {'path': '/weatherstation/updateweatherstation.php'}
2024-06-01 02:47:05.303 INFO (MainThread) [homeassistant.components.sensor] Setting up sws12500.sensor

@bgeza that seems, that SWS component is installed correctly. There will be misconfigured station. Try to add only IP address with port to your station setup.
Is your HA running under https?

in the sencor itself I have have set <IP_ADDRESS>:8123
I don't know what exactly mean "under https", never did anything intentionally regarding this. In HA settings I have Home Assistant URL Internet being http://mysitename.duckdns.org:8123

@bgeza is your station connected to AP. Be aware that if you modify some fields for posting data. It might loose AP information. If it is connected ad your IP is right, then you should prove, that is posting data to correct port.
Lets go to chat over Discord http://discordapp.com/users/1224371660027596961

thank you very much Lukas for your kind and prompt help! Now everything is working!


Firefox window doesn t display with Go 1.10.1 on Ubuntu 18.04

After updating from Ubuntu 17.10 (Go 1.8) to Ubuntu18.04 AND recompiling my project with go 1.10;
the firefox window doesn t appear.
The app run fine and I get all the logs from my application;  it is really clicking and so on.
But no window
When I run the previous compiled (Go 1.8) project binary the window appears; this makes me think it comes from GO 1.10.1
go version go1.10.1 linux/amd64
geckodriver-v0.20.1-linux64
selenium-server-standalone-3.11.0
firefox 60.0.2
EDIT:
I have now update to selenium-server-standalone-3.11.2
But I get the same behaviour
LOG :
08:35:04.315 INFO [GridLauncherV3.launch] - Selenium build info: version: '3.12.0', revision: '7c6e0b3'
08:35:04.317 INFO [GridLauncherV3$1.launch] - Launching a standalone Selenium Server on port 7070
2018-06-09 08:35:04.416:INFO::main: Logging initialized @425ms to org.seleniumhq.jetty9.util.log.StdErrLog
08:35:04.585 INFO [SeleniumServer.boot] - Selenium Server is up and running on port 7070
OS Macos
08:35:10.262 INFO [ActiveSessionFactory.apply] - Capabilities are: Capabilities {browserName: firefox, browser_version: 59.0, moz:firefoxOptions: {prefs: {intl.accept_languages: fr}}, os: OS X, os_version: Snow Leopard, plateform: MAC}
08:35:10.263 INFO [ActiveSessionFactory.lambda$apply$11] - Matched factory org.openqa.selenium.remote.server.ServicedSession$Factory (provider: org.openqa.selenium.firefox.GeckoDriverService)
1528526110309	geckodriver	INFO	geckodriver 0.20.1
1528526110327	geckodriver	INFO	Listening on <IP_ADDRESS>:13721
1528526110672	mozrunner::runner	INFO	Running command: "/usr/bin/firefox" "-marionette" "-profile" "/tmp/rust_mozprofile.1Ns6Xm9z71YH"
1528526112622	Marionette	INFO	Listening on port 45083
08:35:12.752 INFO [ProtocolHandshake.createSession] - Detected dialect: W3C
08:35:12.810 INFO [RemoteSession$Factory.lambda$performHandshake$0] - Started new session d1ea2567-b311-42d0-bfea-f45b4d824858 (org.openqa.selenium.firefox.GeckoDriverService)

Lately i have reinstall everything on another ubuntu 18.04 machine with go 1.8 (instead of 1.10) and it shows the firefox windows.
I can confirm GO 1.10 compiled Binaries doesn t show the FF window


Updater fails battery checks on a machine without a battery

On my bluefin:38 machine:
Oct 04 18:33:37 exo systemd[1]: Failed to start ublue-update.service - Universal Blue Update Oneshot Service.
Oct 04 18:33:37 exo systemd[1]: ublue-update.service: Failed with result 'exit-code'.
Oct 04 18:33:37 exo systemd[1]: ublue-update.service: Main process exited, code=exited, status=1/FAILURE
Oct 04 18:33:37 exo ublue-update[14453]:  - Battery less than 50.0%
Oct 04 18:33:37 exo ublue-update[14453]: Exception: update failed to pass checks:
Oct 04 18:33:37 exo ublue-update[14453]:     raise Exception(f"update failed to pass checks: \n - {exception_log}")
Oct 04 18:33:37 exo ublue-update[14453]:   File "/usr/lib/python3.12/site-packages/ublue_update/cli.py", line 96, in hardware_inhibitor_checks_failed
Oct 04 18:33:37 exo ublue-update[14453]:     hardware_inhibitor_checks_failed(
Oct 04 18:33:37 exo ublue-update[14453]:   File "/usr/lib/python3.12/site-packages/ublue_update/cli.py", line 247, in main
Oct 04 18:33:37 exo ublue-update[14453]:              ^^^^^^
Oct 04 18:33:37 exo ublue-update[14453]:     sys.exit(main())
Oct 04 18:33:37 exo ublue-update[14453]:   File "/usr/bin/ublue-update", line 8, in <module>
Oct 04 18:33:37 exo ublue-update[14453]: Traceback (most recent call last):
Oct 04 18:33:37 exo ublue-update[14453]: [2023-10-04 18:33:37,034] ublue_update.cli:INFO | No updates are available.
Oct 04 18:33:37 exo ublue-update[14453]: [2023-10-04 18:33:37,034] ublue_update.update_checks.system:INFO | No system update available.

Do you by any chance use a wireless mouse?
My machine without battery returns None by the following command. Can you run this command and post the output?
echo -e "import psutil\nprint(psutil.sensors_battery())" | python

I do have a wireless mouse!
echo -e "import psutil\nprint(psutil.sensors_battery())" | python
sbattery(percent=23, secsleft=<BatteryTime.POWER_TIME_UNKNOWN: -1>, power_plugged=False)

and
ls /sys/class/power_supply/*
capacity      model_name  scope          type
device        online      serial_number  uevent
hwmon7        power       status         voltage_now
manufacturer  powers      subsystem      wakeup78

Can you do a grep -r . /sys/class/power_supply/ too? Problem with this stuff is that there quiet a few of these kind of devices with a battery, that can show up.

I have one item in there:
❯ ls -l
lrwxrwxrwx@ - root  5 Oct 13:11 hidpp_battery_0 -> ../../devices/pci0000:00/0000:00:02.1/0000:05:00.0/0000:06:0c.0/0000:10:00.0/usb5/5-4/5-4:1.2/0003:046D:C539.0008/0003:046D:4079.0011/power_supply/hidpp_battery_0

For example TLP is doing the same, it's ignoring exactly the one you are posting
https://github.com/linrunner/TLP/blob/40c44d108182fd57c2ff44d685a8f908a9c782d3/tlp-func-base.in#L51C1-L52
# power supplies: ignore MacBook Pro 2017 sbs-charger and hid devices
readonly RE_PS_IGNORE='sbs-charger|hidpp_battery|hid-'

Maybe we should skip the battery thing in bluefin entirely and just avoid having to special case a bunch of hardware?

Can you verify that ublue-update runs correctly if you set min_battery_percent = 0 in the config toml?

unfortunately this is an upstream issue with psutil, the relevant PR is here:
https://github.com/giampaolo/psutil/pull/2305


build: use node lts version

We cannot bump the required node version to 18 yet as the docker image mcr.microsoft.com/playwright is still running v16 due to issues described here

We cannot bump the required node version to 18 yet as the docker image mcr.microsoft.com/playwright is still running v16 due to issues described here

Does that mean we can't get new versions of nebula in?

We cannot bump the required node version to 18 yet as the docker image mcr.microsoft.com/playwright is still running v16 due to issues described here

Does that mean we can't get new versions of nebula in?

No, but circleci needs to install and build using v18 and locally you should use v18 as well


[Feat]: Validation add on Contact form to enhance UI

What feature?
Add empty validation on contact form
Validation add for valid email
Error message shown below of fields
Highlighted that field by red border to enhance user experience..

@PiyushKalyanpy I would like to contribute on this project under GSSOC'23. Please assign this to me.
Show us the magic with screenshots
here is demo:

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSoC'23 contributor
[X] I want to work on this issue

We are closing this issue as you have created two same issues.


Accessibility Issues (Medium Important)

Accessibility Issues
What Is the Issue About?
This issue will be helpful for those who use screen-readers.
What Will Be Fixed?
The examples below are going to be fixed by this issue:

Pictures whose alts are not filled
Semantic HTML that's not in the website

Notes
This issue will be updated, so please wait for the next update of the issue. Also, this will be included in the next 0.2.0 update.

Will fix the issue today.

Will fix the issue this week. Got some problems...

Closing issue for the time:
https://github.com/TheRobotCodingMan/LittleNews/discussions/11


Feature request: meta-learning baselines

Hi! Thank you for your work!
It would be great to have some baselines that allow for a fast start with Metaworld (e.g. code used for the paper). Or at least to mention some repositories that use Metaworld.

@Guitaricet thanks for the FR. I agree that this would be an excellent contribution to the meta-RL community, but it's definitely out-of-scope for this repo (which intentionally includes the environments only).
We believe it's essential to separate benchmarks from implementations. If you'd like strong baseline implementations, I recommend you take a look at our sister project garage.

Since it's out-of-scope for this repo, I'm going to close this issue. Please don't hesitate to open a new one with a different problem.


Dynamically changing the localization

Hi,
I am out of options on where to ask, and fairly new to Ionic.
When I use app.run() to set the localization data in my app.js, it works.
However, when I try the same thing in my controller, it never gets executed.
How am I supposed to change the localization if my user changes his prefered language while using the application?

Hi,
the localization factory works fine for me, although it's limited - it does not change locale data of already opened picker, but this should be no issue. When locale data are changed and new picker is opened, it should use the new data. See codepen to see it working in action.


fix ObjectBeforeCollectCallback unittests

nullptr should clear out existing callbacks
explicitly make oom when merging callback lists unrecoverable (we were asserting that OOM was unhandled)

Rerunning win7


Adds a File abstraction

This is for swapping out the underlying writing implementation, for example for writing directly to google cloud storage.

Made the changes discussed offline.


WIP: YJDH-364

Description :sparkles:
TODO
Issues :bug:
YJDH-364
Testing :alembic:
Screenshots :camera_flash:
Additional notes :spiral_notepad:

Codecov Report

Merging #1054 (a106018) into develop (ca0ca00) will increase coverage by 18.76%.
The diff coverage is 59.09%.

@@             Coverage Diff              @@
##           develop    #1054       +/-   ##
============================================
+ Coverage    64.58%   83.34%   +18.76%
============================================
  Files          905       67      -838
  Lines        25098     2210    -22888
  Branches      2546        0     -2546
============================================
- Hits         16209     1842    -14367
+ Misses        8500      368     -8132
+ Partials       389        0      -389

Impacted Files
Coverage Δ

backend/shared/shared/vtj/vtj_client.py
59.09% <59.09%> (ø)

backend/shared/shared/ytj/ytj_client.py
27.58% <0.00%> (-65.52%)
:arrow_down:

backend/shared/shared/common/decorators.py
37.93% <0.00%> (-58.63%)
:arrow_down:

...nd/shared/shared/service_bus/service_bus_client.py
27.65% <0.00%> (-57.45%)
:arrow_down:

backend/shared/shared/yrtti/yrtti_client.py
39.39% <0.00%> (-51.52%)
:arrow_down:

backend/shared/shared/audit_log/viewsets.py
45.56% <0.00%> (-50.64%)
:arrow_down:

backend/shared/shared/common/lang_test_utils.py
60.00% <0.00%> (-40.00%)
:arrow_down:

backend/shared/shared/models/mixins.py
66.66% <0.00%> (-33.34%)
:arrow_down:

backend/shared/shared/common/utils.py
75.00% <0.00%> (-25.00%)
:arrow_down:

backend/shared/shared/audit_log/audit_logging.py
76.59% <0.00%> (-23.41%)
:arrow_down:

... and 840 more

Continue to review full report at Codecov.

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


BUG:  not following pwd after restoring session with Resession

Did you check docs and existing issues?

[X] I have read all the docs.
[X] I have searched the existing issues.
[X] I have searched the existing discussions.

Neovim Version (nvim -v)
0.10.0 Release
Operating System / Version
Arch Linux
Describe the Bug
When I restore a session with resession.nvim which in turn changes the current working directory (:pwd), neo-tree doesn't reflect the new current working directory when I open it if it was previously open
Screenshots, Traceback
This shows Neo-tree opened not in the current working directory which is displayed in the bottom when I run :pwd

Steps to Reproduce

create a directory with the repro.lua file
mkdir test, make a directory in there as a "scratch directory
echo "SESSION LOADED" > test/test, make an example file
nvim -u repro.lua, start up neovim
:cd test, change directory into the test directory
:e test, open our example file
:lua require("resession").save("Test"), save a session
:q, quit neovim
nvim -u repro.lua, start up neovim again
:Neotree, open neo-tree and see that we are in our repro folder
q, close neo-tree
:lua require("resession").load("Test"), load our session
:pwd, check the current working directory and see that it is in our test/ folder
:Neotree, open neo-tree and see that it has us in our repro folder and not in the test/ folder
:pwd, check the current working directory even in our Neo-tree buffer and see that it has us in the test/ directory

Expected Behavior
Neo-tree should be following the current working directory no matter what. I'm not sure what would be causing this issue. My initial thought is the issue is on the neo-tree side just because neo-tree should be respecting the current working directory no matter what. Please let me know if I can provide any other information to help debug this!
Your Configuration
-- DO NOT change the paths and don't remove the colorscheme
local root = vim.fn.fnamemodify("./.repro", ":p")

-- set stdpaths to use .repro
for _, name in ipairs({ "config", "data", "state", "cache" }) do
	vim.env[("XDG_%s_HOME"):format(name:upper())] = root .. "/" .. name
end

-- bootstrap lazy
local lazypath = root .. "/plugins/lazy.nvim"
if not vim.loop.fs_stat(lazypath) then
	vim.fn.system({ "git", "clone", "--filter=blob:none", "https://github.com/folke/lazy.nvim.git", lazypath })
end
vim.opt.runtimepath:prepend(lazypath)

-- install plugins
local plugins = {
	"folke/tokyonight.nvim",
	{
		"nvim-neo-tree/neo-tree.nvim",
		branch = "v3.x",
		dependencies = {
			"nvim-lua/plenary.nvim",
			"nvim-tree/nvim-web-devicons", -- not strictly required, but recommended
			"MunifTanjim/nui.nvim",
			-- "3rd/image.nvim", -- Optional image support in preview window: See `# Preview Mode` for more information
		},
		opts = {},
	},
	{
		"stevearc/resession.nvim",
		opts = {},
	},
	-- add any other plugins here
}
require("lazy").setup(plugins, {
	root = root .. "/plugins",
})

vim.cmd.colorscheme("tokyonight")
-- add anything else here

Neo-tree works if you refresh (by using the R key).
Neo-tree needs to automatically refresh.


upgrade go to 1.21.0

Description
Upgrades Go version to 1.21.0 and updates GH actions to use go.mod instead of a hard-coded Go version.
Fixes #998
Quick checks:

[x] There is no other pull request for the same update/change.
[ ] I have written unit tests.
[x] I have made sure that the PR is of reasonable size and can be easily reviewed.

Seems like the test can't reach haris-redshift-cluster-2.cricdtjk6gta.us-east-1.redshift.amazonaws.com. @hariso do you know what could be the issue?

Seems like the test can't reach haris-redshift-cluster-2.cricdtjk6gta.us-east-1.redshift.amazonaws.com. @hariso do you know what could be the issue?

The instance has been deleted, it wasn't supposed to be used in these tests. I think I added it temporarily to see if an upgrade was OK.

Hmm, we have an issue then with automatically merging dependabot upgrades. We rely on our tests to tell us if a dependency upgrade broke anything, so if tests are green we assume we can merge the upgrade automatically. So creating a temporary instance would help us only for this specific PR 😕

@simonl2002

Closing in favor of https://github.com/conduitio-labs/conduit-connector-redshift/pull/95.


[extractor/vk] playlist fix for new API requirements

IMPORTANT: PRs without the template will be CLOSED
Description of your pull request and other information
As reported here, VK playlist downloads stopped working. The reason is, that their API has been changed yet again and now requires "playlist_ID" instead of just "ID" as a request parameter. With this fix everything works again
Fixes  https://github.com/yt-dlp/yt-dlp/pull/4930#issuecomment-1409576726
PLEASE FOLLOW THE GUIDE BELOW
Before submitting a pull request make sure you have:

[X] At least skimmed through contributing guidelines including yt-dlp coding conventions
[X] Searched the bugtracker for similar pull requests
[X] Checked the code with flake8 and ran relevant tests

In order to be accepted and merged into yt-dlp each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[X] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[X] Fix or improvement to an extractor (Make sure to add/update tests)
[ ] New extractor (Piracy websites will not be accepted)
[ ] Core bug fix/improvement
[ ] New feature (It is strongly recommended to open an issue first)

Yes, the other branches are for URLs, that either provide their own valid section id, or download an entire channel 'all'. Both of these are still what VK uses in their requests unchanged.
I did however just notice, that I forgot to update the playlist test, so I fixed that.
When doing so I also noticed, that most of the single video (VKIE) tests either fail for various reasons, or get skipped because the video has been removed. I think they were already failing months ago, but I am not sure. If you want me to, I can just update all of the tests in the extractor (either in this PR, or another one?)  to make sure nothing there has been broken by this or an older API change

If you want me to, I can just update all of the tests in the extractor (either in this PR, or another one?) to make sure nothing there has been broken by this or an older API change

Sure, you can do it in same PR

I updated all the tests and it seems like they did in fact change their site somewhere else.
They changed their wallposts audio data to JSON and removed the url encryption, which crashed the previously already non-functioning audio extraction completely. I fixed that, but downloading still returned an error from the 'FFmpegFixupM3u8PP', because the audio stream is mp3 and not the expected aac. I changed the container to mov to hide the error for now. The files would however actually need a simple fixup, as they are currently only playable with a handful of media players. A real fix for this would be to have a url check in the fixup, to ommit the incompatible stuff:
if all(self._needs_fixup(info)):
    if 'vkuseraudio.net' in info['url']:
        self._fixup('Fixing MPEG-TS in MP4 container', info['filepath'], [
            *self.stream_copy_opts(), '-f', 'mp4'])
    else:
        self._fixup('Fixing MPEG-TS in MP4 container', info['filepath'], [
            *self.stream_copy_opts(), '-f', 'mp4', '-bsf:a', 'aac_adtstoasc'])

but that seems like an inelegant way to do this and I am not familiar enough with the codebase to come up with a better alternative. For now you would either have to play the files in something like mpv, or use -x to extract the actual audio manually.
VK also lifted a lot of restrictions to being registered, so a lot of tests for edge cases are no longer required and some don't need to be skipped anymore.
All tests succeed now

The files would however actually need a simple fixup, as they are currently only playable with a handful of media players.

If it's mp3, what kind of fixup does it need?

Great, the files are playable and have no corruptions.
I just changed the author extraction class, because vk changed it within the last few days and updated the tests.
Only thing left would be merging the audio & video entries again when returning, but I couldn't figure out how to append the audio entries to the playlist_from_matches without basically reverting that change from your cleanup back to using 'playlist_result'

Only thing left would be merging the audio & video entries again when returning, but I couldn't figure out how to append the audio entries to the playlist_from_matches without basically reverting that change from your cleanup back to using 'playlist_result'

My bad. Reverted

When doing final testing, I noticed, that irrelevant reaction clips in the comments will also be picked up. That seemed like a bug to me, so I fixed that by only searching for videos in the actual post body. Otherwise everything should be good


Add interfaces for neuropixel data

Add SpikeGLX interfaces and metadata handling for Neuropixel data

example file here: https://drive.google.com/file/d/1rVlEja713onTChw9qoqypr1d7yuNbT_n/view?usp=sharing

@felixp8 That shared example conversion seems to only be for the trials data, not NeuroPixels

ah whoops, this should be the right one: https://drive.google.com/file/d/1tF11XP5J4HrMQKdaY4PLIhGxYG8jrCOa/view?usp=sharing


PomBase slim update  ADD "organophosphate biosynthetic process"

"organophosphate biosynthetic process"

moved up to metabolic process


Clarify License

Very nice image!
Where did you get the license from?
This seem to be the original post: https://www.reddit.com/r/linuxmasterrace/comments/lzdwl4/nixowos/
It does not have a license.
It is referenced here as to have CC BY: https://github.com/TheDarkBug/uwufetch/blob/main/res/COPYRIGHT.md#nixos-nix-snowflake
With a CC license that has BY, you have to name the original author "ant-arctica".
When the license is clarified, please add a LICENSE file to the repo.

I had a second look at the copyright a license situation. I don't see it as necessary to name the artist as long as there is a link (or other direct reference) to the original work where all of that information can be found.
I used the CC-BY-SA here because the NixOS website has that license.
To make it more clear I will add the section from the COPYRIGHT.md to the readme of this repo.

@davidak Do you consider this issue solved?

tl;dr yes, it's solved. thanks!

I don't see it as necessary to name the artist as long as there is a link (or other direct reference) to the original work where all of that information can be found.

The site says:

You must give appropriate credit

And as explanation:

If supplied, you must provide the name of the creator and attribution parties, a copyright notice, a license notice, a disclaimer notice, and a link to the material.

That's pretty clear i think.

I used the CC-BY-SA here because the NixOS website has that license.

That is a good license for a final content piece, so modifications have to have the same license, but could be too limiting here. For example i don't think it would be allowed to use this logo in NixOS, because it's not CC BY-SA, but that would be fun.
But you have CC BY now, so everything is fine.

To make it more clear I will add the section from the COPYRIGHT.md to the readme of this repo.

Looks very good!
It's still unclear what "@timbertson" means. Is it a Twitter handle or GitHub user? But i think it's fine unless they complain.
(It's actually both)
https://github.com/timbertson
https://twitter.com/timbertson
uwu

I don't know what this repo does but the logo's cute 😀

@timbertson it's just an extra cute version of the logo (as vector graphic, SVG). Someone else did draw it.


Create CODEOWNERS

Hello!
We're making maintainers explicit. I've guessed the maintainers here, let me know if this is not right @emilk. Thanks!
Checklist

[x] I have read the Contributor Guide
[x] I have read and agree to the Code of Conduct
[x] I have added a description of my changes and why I'd like them included in the section below

thanks!


Make install fails

Requirement already satisfied: greenlet!=0.4.17 in ./venv/lib/python3.11/site-packages (from SQLAlchemy==2.0.19->-r requirements.txt (line 67)) (3.0.1)
/usr/local/bin/docker-compose up -d
[+] Running 4/4
 ✔ mysql 3 layers [⣿⣿⣿]      0B/0B      Pulled                                                                                2.2s
   ✔ c1a182859289 Already exists                                                                                0.0s
   ✔ 5d2fb452c483 Already exists                                                                                0.0s
   ✔ 1773f3c7aa95 Already exists                                                                                0.0s
Error response from daemon: content digest sha256:d43bab9d9bd18d3770f6156bdb7c5364cac797c6a906e67cf548b0a439ff1d2d: not found
make: *** [install] Error 1

Could be caused by the MySQL resource (https://hub.docker.com/layers/library/mysql/8/images/sha256-d43bab9d9bd18d3770f6156bdb7c5364cac797c6a906e67cf548b0a439ff1d2d) not existing; see the screenshot below:

Ran docker pull mysql as a separate command.
However the following new error has surfaced.
/usr/local/bin/docker-compose up -d
image with reference mysql:8 was found but does not match the specified platform: wanted linux/arm64/v8, actual: linux/amd64
make: *** [install] Error 1

Just realized there are two yaml files in the dj_backend_server directory.
I just needed to rename to suit my architecture:
cp docker-compose.yaml docker-compose.arm.yaml
mv docker-compose.linux.yaml docker-compose.yaml

@tpmccallum glad you found the solution


Avoid fade out/in on updateDom when many calendars are used

related to #3185

I have limited updated dom to one update
-> updateDom() is activated by a timer which resets when a new event arrives (CALENDAR_EVENTS)

I have set no speed to self update. I think it's not necessary
-> update it directly

If somebody can test and tell me result
In all case, I will patch my prod mirror for testing

Unfortunately, I see this morning that it doesn't work.
The problem remains the same.
I think it's because of the time taken to fetch  the data.
I'll see how to do it differently

Really if node 18 is strong too...
$ npx jest tests/e2e/modules_empty_spec.js
 PASS   e2e  tests/e2e/modules_empty_spec.js
  Check configuration without modules
    ✓ shows the message MagicMirror² title (203 ms)
    ✓ shows the url of michael's website (202 ms)

Test Suites: 1 passed, 1 total
Tests:       2 passed, 2 total
Snapshots:   0 total
Time:        1.682 s

$ node -v
v18.18.0

v2: I clear all and start again.

added updateOnFetch  feature:

true: (default) like actually update DOM on each calendar data received
false: delayed update DOM and automaticaly updated every minutes

basicaly, I see an internal timer every minutes for update DOM.

this internal timer is called on start()
I move it to a function called : selfUpdate()
I have added this rule: if updateOnFetch is not activated, So updatedDom with animationSpeed

---> +/- no change: just add animation speed or not when DOM Updated

When CALENDAR_EVENTS received (socketNotificationReceived)

if updateOnFetch is activated (default): no change, it's updateDom() on ach notification received
if updateOnFetch is NOT activated:

updateDom() on first data received for each calendars
When others data received (next cycle): don't update DOM and use selfUpdate() rules for update DOM

---> Final result must be, when updateOnFetch disabled: ONLY one update DOM every minute
I will patch my prod mirror with this new file for testing (with updateOnFetch disabled)
I will see tomorrow morning

Good news; It works as requested!

so, its ready for review @bugsounet ?

yes, it's ready but something is really not clear for me
I have done some test

On my pi4 (without  animate and customCSS )

Tested with my prod version: I have the main issue (#3185)
Tested with only clock / calendar: I have the issue too
Tested with export ELECTRON_DISABLE_GPU=1: I have the same issue

On my dev desktop (without animate, customCSS, ....)
OS Ubuntu: 23.10
Same prod config or only clock / calendar
I have not this main issue !

maybe pi take many time for change CSS ?
So, I really don't understand
@rejas: what do you think about this ?


Trial date not being set from Plan from Stripe

I made a new plan in Stripe, giving it a 30 day trial period.
I have then subscribed a user to this plan, and whilst the subscription table now has a new record, it's missing the trial_ends_at date.
I see in the docs I can set the number of trial days, but surely that defeats the purpose of setting it in Stripe?
Which one takes precedence? If my plan in Stripe is set to 30 days, and I set it to 10 or even 60 in Laravel, which one is actually used?
What am I missing?

Trial dates in Cashier take precedence over Stripe trial days because Cashier allows you to work with a trial period without a credit card up front. That's why a custom way of working with trial days was built in. You should just leave your trial days on plans untouched when creating them.
When a subscription is created, the trial_ends_at parameter on the subscription will be set from whatever trial days you're passing with the trialUntil method.

I've sent in a PR to add a note about this to the docs: https://github.com/laravel/docs/pull/5271

So if I make a price plan on Stripe with no trial set, and I set one in Cashier, how does it know when to bill the customer?
If it subscribes the customer to that plan in Stripe, its going to start billing them right away?

So if I make a price plan on Stripe with no trial set, and I set one in Cashier, how does it know when to bill the customer?

Like I said above, it overwrites the trial_ends_at parameter when it creates a subscription. See https://stripe.com/docs/api/subscriptions/create#create_subscription-trial_end

@driesvints sorry, so just to be clear, even if a plan has a trial period set, individual customers can have a custom trial period? (ignoring Cashier, is this something that Stripe does?)

@paul-gene the trial_ends_at parameter is meant to overwrite the plan's trial period so each subscription can have its custom trial ending. So basically, yes.

Thank you for clarifying :)

Could Cashier not have a method trialFromPlan() ? Stripe sets trial_from_plan to false by default, it means you will always need to specify it. I don't want to have it only in my env variables

@tquiroga this isn't supported unfortunately.

@driesvints That could be easily added I believe and would allow people to not have to match their code with what they set in Stripe.


⚒ refacto layouts

PR type

[x] Bugfix
[x] Refactoring

What's new?

[x] PR closes #1360
[x] moved KeyboardShortcutsModal in Footer
[x] remove code from layouts files
[x] remove isShareMode from layouts

Before submitting Pull Request, please make sure:

[x] My contribution builds clean without any errors or warnings
[x] I've merged recent default branch -- main and I've no conflicts
[x] I've tried to respect high code quality standards
[ ] I've didn't break any original functionality

Had issue bounty label?

[x] Fill up your KSM address: Payout

LGTM 🚀

pay 100 usd

😍 Perfect, I’ve sent the payout
💵 $100 @ 157.01 USD/KSM ~ 0.637 $KSM
🧗 HguGafrfk8nRP9a4tgJf7hLvSSFie8GtkDjyWFHQ2kDUAMc
🔗 0x3de110b6df5564312aa928585332e738b804c6931f2bcc0a1ee637e454c1c19e
🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues

updated #2641


[2020-10-05] Deployment

Gallery
Changes:
make attribute id value unique(#8210)
Darken the orange buttons (#8219)
Darken social buttons for accessibility (#8220)
[Cookie Compliance] The new cookie compliance pipeline. (#8209)
Delete unused Grunt scripts and add .eslintignore (#8227)
Disable flaky test (#8232)
Revert "[Cookie Compliance] The new cookie compliance pipeline. (#8233)
Update NuGet client libraries to 5.8.0 prerelease (#8229)
add more description for learn more (#8226)
[Cookie Compliance] Revert revert and fix the functional test (#8236)
path fix in scritp point to vs2019 (#8244)
Revert "[Cookie Compliance] Revert revert and fix the functional test (#8246)"
Progress:

[x] Merge RI: https://github.com/NuGet/NuGetGallery/pull/8239, https://github.com/NuGet/NuGetGallery/pull/8254
[x] Build: https://devdiv.visualstudio.com/DevDiv/_build/results?buildId=4144387&view=results
[x] Release: https://devdiv.visualstudio.com/DevDiv/_releaseProgress?_a=release-pipeline-progress&releaseId=844058
[x] DEV
[x] INT
[x] PROD

Let's temporarily freeze the PROD deployment, wait until the cookie work is stabilized and then move to PROD together. @skofman1

This deployment is complete.


Separate the programming settings for C++, Python and JavaScript

Currently everything is mixed up together on Programming Settings tab of Application Settings. In the future if you managed to implement https://github.com/vinadevs/VinaText/issues/16 then your text editor will go the direction like of Sublime Text, you will need to refactor all of this primitive support for programming languages into plugins.

Thanks, i noted.


46: Documented instructions to setup project on developer machines

46: Documented instructions to setup project on developer machines

Looks good


Custom reply queue naming

Provide an option (and better defaults) for naming the reply queues.

My thoughts it should be smth meaningful and unique so that's why I added this to #13:
var replyQueue = queueName + '-reply-' + corrId

But anything that can be used to remove it is fine to me.


Make nobelium as template repo

Is your feature request related to a problem? Please describe.
Nope. It's just the semantics of Use the template are more proper than fork.

Describe the solution you'd like
https://docs.github.com/en/repositories/creating-and-managing-repositories/creating-a-template-repository
Describe alternatives you've considered
Nope
Additional context
Nope

Thanks for the advice. However, I have concerns about how users will be able to sync Nobelium updates if they use the template repo feature to create new sites. Updating git branches manually seems difficult for non-programmers.


Discover / Creator Page

[x] Show list of uploaded scenes
[x] Add option to Delete, Edit and Feature(Admin only) scenes in creator page
[x] Show list of featured scenes in Admin page
[x] Show list of featured room/world in Admin page

Fixes #457

@ajbura looks like there's an error when featuring rooms/worlds after switching over to the new repository room

 Uncaught (in promise) TypeError: i is undefined
    _getEventFromHomeserver hydrogen.es.js:13614

We should feature flag this using local storage until we're ready for launch.


Proposal: release the push lock when the SSH connection is severed

Summary
This issue if for detecting when the SSH connection is severed, and releasing the push lock when that happens.
If You Are Reporting a Bug
If the user doing a git push cancels the git push (by doing a ctrl+c, for example), the builder still holds the push lock until the builder pod finishes. Instead, it should immediately release the lock
Replication Steps
Please provide a detailed list of steps to reproduce it.

Create a Deis Cluster
Register an app
Start a git push
Cancel it
Try to do another git push

After #5, you'll notice that the git push will fail with an error message until the builder pod finishes.
Related Issues
Note that https://github.com/deis/builder/pull/339 fixes the issue where the lock is released when the builder pod errors.

fixed in #339


Block format

This is amazing. It is wonderfully fast and great with files.
It is possible to change the formatting of the blocks. I don't need user name or time signature.
Also, is it possible to send emails to the Telegram bot? This would open new possibilities.
Many thanks.

You can change the formatting of the blocks by editing the code directly, although it would be nice if this were made easier to do. For instance, for text blocks look at line 259  block: { uid, string: [[${name}]] at ${hhmm}: ${text} }` You can edit what is between the quotes.


Update docs, experimental and samples

Updates the documentation, scripts in experimental and the samples to reflect the renaming of the Python packages.

LGTM once we
1. land the version change

2. land the package name change

3. publish our first nightly and stable releases of the new packages

These steps are complete. Can you sync this?


Column/Bar chart does not respect axis max

See http://jsfiddle.net/tff4w6jz/ (column chart) vs http://jsfiddle.net/tff4w6jz/1/ (line chart)
It is arguably somewhat nonsensical to limit the column chart this way when your data is explicitly higher, but it should behave equivalent to the line-chart for consistency.
@jaimedp

The difference is not really on the type of visualization but on the type of axis that is used, the default x-axis type for line charts is "linear" which implies a linearly increasing numeric series, the column really only makes sense in an "ordinal" axis type, which is not based on an increasing numeric series, but on discrete categorical values (ie. strings or numbers or labels). In the ordinal axis it has no meaning to have a max since the categories can be strings or whatever. Makes sense?
You can see this also if you take the line chart example and add a type: 'ordinal' to the xAxis options, and that would make it behave the same as the column case.

Thanks that makes sense, thanks.


az login failed

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az login
Errors:
request failed: Error occurred in request., RetryError: HTTPSConnectionPool(host='api-dogfood.resources.windows-int.net', port=443): Max retries exceeded with url: /subscriptions?api-version=2019-06-01 (Caused by ResponseError('too many 500 error responses',))

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az login

Expected Behavior
Environment Summary
Windows-10-10.0.18362-SP0
Python 3.6.6

azure-cli 2.2.0 *

Extensions:
aks-preview 0.4.42
interactive 0.4.4
spring-cloud 0.2.2

Additional Context

hi @arrownj could  you pls have a look? thanks

plus 1 one this
On G.Actions:
Run az login --service-principal -u $AZURE_SP -p $AZURE_SP_PASSWORD --tenant $AZURE_SP_TENANT
az login --service-principal -u $AZURE_SP -p $AZURE_SP_PASSWORD --tenant $AZURE_SP_TENANT
shell: sh -e {0}
env:
AZURE_SP: ***
AZURE_SP_PASSWORD: ***
ARM_CLIENT_SECRET: ***
AZURE_SP_TENANT: ***
ARM_TENANT_ID: ***
SUBSCRIPTION_ID: ***
ARM_SUBSCRIPTION_ID: ***
AZURE_STORAGE_ACCOUNT: navtestapimstorage
AZURE_STORAGE_KEY: ***
ARM_CLIENT_ID: ***
ARM_ACCESS_KEY: ***
APIM_NAME: test-apim-nav
RESOURCE_GROUP: nav_test_apim
ACTUAL_SIZE: 12
GENERATED_SIZE: 17
Traceback (most recent call last):
File "/usr/lib/python3.7/runpy.py", line 193, in _run_module_as_main
"main", mod_spec)
File "/usr/lib/python3.7/runpy.py", line 85, in _run_code
exec(code, run_globals)
File "/usr/local/lib/python3.7/dist-packages/azure/cli/main.py", line 33, in
az_cli = get_default_cli()
File "/usr/local/lib/python3.7/dist-packages/azure/cli/core/init.py", line 628, in get_default_cli

Hi @frankliu20 , why the endpoint in your log is api-dogfood.resources.windows-int.net ? Is there any special settings in your case ? Maybe you can share more about your environment to help us investigate the problem. Thanks

Close this first for no feedback for a long time.


Version v0.14.4 Analytics in Docker Container won't work

Originally posted by @marquicus in https://github.com/NREL/api-umbrella/issues/284#issuecomment-391607706

After some struggeling with the docker container. I checked the elastics process. I can't query any elastics status.
curl http://<IP_ADDRESS>:14002
curl: (56) Recv failure: Connection reset by peer
Elastics log looks good.


Proposal: Add strict comparison flag

A simple flag to enforce strict comparisons. When set, if the two operands of == have different types, the compiler should raise an error, regardless of if they are comparable in JavaScript.
Why not just use ===? Because then I'd have to go back through my whole codebase to replace every == with ===, and because it's a pain when writing in more than one language to remember that in TypeScript == is broken and I should be using === instead. It would be a lot simpler if == just wasn't broken to begin with.

One of the main design goals of TypeScript is to align with JavaScript (see https://github.com/Microsoft/TypeScript/wiki/TypeScript-Design-Goals#goals). == has useful use cases, e.g. comparing against null and undefined. disallowing it or changing its semantics would go against this design goal.
As for the issue of using == I would recommend adding a tslint rule for using == to your build.

TypeScript already doesn't align with JavaScript when JavaScript made decisions that were broken, for instance, dynamic typing. I don't see how this is any different.

@markboyall typescript fully supports JavaScript's dynamic typing and in fact it is the default. variables without an inferred type are given the dynamic type any. Furthermore, TypeScript's type system was specifically designed to statically model JavaScript's dynamic one.
As @mhegazy noted, tslint is the answer here. I have exactly such a rule in all of my projects.

Having a completely broken comparison operator and having to arse around endlessly with linting tools and unnecessary language-changing mental overhead would remain fully supported and the default.

Having a completely broken comparison operator

Mostly broken.