test

test

test

Here is what I did:
from IPython.utils.traitlets import link
from IPython.html.widgets import *

And I ended up with the widget link without knowing it...I agree that
namespaces are a great idea, but in this case, I think a bit more
distinction for the user would be helpful (link versus jslink).

Here is what I did:
from IPython.utils.traitlets import link
from IPython.html.widgets import *

And I ended up with the widget link without knowing it...I agree that
namespaces are a great idea, but in this case, I think a bit more
distinction for the user would be helpful (link versus jslink).
On Mon, Jan 26, 2015 at 4:14 PM, Jonathan Frederic<EMAIL_ADDRESS>
wrote:

Is there a strong chance they'll be imported at the same time? It seems to
be like you'd use one or the other, but not a combination of both.
Also, like @SylvainCorlay https://github.com/SylvainCorlay says "names
spaces are one honking great idea". Isn't Python's from X import Y as Z
designed to allow the users to get around this problem?
—
Reply to this email directly or view it on GitHub
https://github.com/ipython/ipython/pull/7468#issuecomment-71566171.

--
Brian E. Granger
Cal Poly State University, San Luis Obispo
@ellisonbg on Twitter and GitHub
<EMAIL_ADDRESS>and<EMAIL_ADDRESS>

@ellisonbg Strange, seems to work when I post your comment as-is through the web interface.  Must be something wrong with the email parser...


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/50684

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/50794

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/96822

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/96823

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/108608


stumpwm-contrib: e13988 -> 9bebe3

Things done

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

[x] NixOS
[ ] OS X
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

this contains a fix for controlling volume with hardware keys

Just a detail, but there doesn't seem to be any versioned release of this package.
Shall it be renamed to stumpwm-contrib-git-YYYY-MM-DD like stated in the manual or to something like stumpwm-contrib-YYYY-MM-DD like proposed in #14572?

stumpwm-contrib isn't a real package - we just fetch it in case the user activates it. No need for a version.
I'll have to test this PR with a friend who uses stumpwm (I moved to herbstluftwm some time ago) to check if the new way of handling contrib is compatible with our packaging approach.

Merged as 008ba8f


Allow for searching by taxonomic rank through /taxa route

Something like this...
/data1.2/taxa/list.json?rank=family
/data1.2/taxa/list.json?rank_no=25
Currently rank can be used to further filter a query for a specific taxnomic group. What I'm looking for is not a filter, but rather being able to pull a list of all families/genera etc.

Never mind this is my mistake. We can already do this.


Refactor authentication for servlet HTTP header handler

I'm working on integrating gitblit behind a reverse web proxy which performs authentication and provides group (team) membership via HTTP headers. As such, I'm trying to write a custom authentication plugin which pulls the user and team membership from the request headers. I wouldn't mind using the users.conf mechanism (ConfigUserService) for the storage of the gitblit specific authorization settings.
My goal is similar to current X509 handling, in that it will need to be verified on every request. I started by doing a proof of concept by hard-wiring it in AuthenticationManager, however I then followed it up with modularizing as in option C below, which I think is a better structured approach, because of the extensibility provided. Both steps are available as separate in sequence commits in this pull request.
The mechanism that I'm envisioning would be very similar to the following that we're using for XWiki:
https://github.com/xwiki-contrib/xwiki-authenticator-headers
However, I was a bit unsure on the best location hooks to use to wire up a new auth plugin, where I have access to the HTTP request header values. After looking at the code base for a bit, I saw a few options:
    A. Write extension of UsernamePasswordAuthenticationProvider, but ignore the parameter values and pass back a UserModel based just on the request context headers.

I haven't found a way to access the request HTTP headers to perform the auth and verify information from within the AuthenticationProvider.authenticate(String username, char[] password) structure. Furthermore, it seems that this is only used for handling the login form submission, after which the session cookie is used for further requests. I'd like to avoid requiring a cookie at all, and build everything for a stateless session with a lifetime only for the duration of request processing.
    B. Modify existing AuthenticationManager direct handling of servlet authentication

The servlet authenticate() version seems to be called from SessionPage on every load, so that appeared to be a viable option. Wicket pages that inherit from SessionPage call GitblitManager, which in turn calls the servlet authenticate method on IAuthenticationManager. The direct handling of the servlet authenticate request within the AuthenticationManager could be updated to add the header authentication.
        Note that the javadoc for the IAuthenticationManager.authenticate() username/password method includes the following, but IUserService doesn't appear to have any authenticate methods (anymore?).
            * @see IUserService.authenticate(String, char[])

    C. Refactor AuthenticationManager so that the authenticate(HttpServletRequest httpRequest) flavor is pluggable like the username password mechanism. This would seem better than B for flexibility. Even adding the header auth into the existing flow in AuthenticationManager, it doesn't allow an installer to select the order or enabling/disabling of various mechanisms. For example, I want to configure it to only trust my custom headers managed by the proxy, not try X509 or pull values from the REMOTE_USER (as getUserPrincipal does).

It seems straight forward to add the servlet authenticate() overload to AuthenticationProvider, and implementations may provide a simple null returning implementation if they don't support a given authentication path. For now that would mean implementing it in my new plugin, and adding a simple null return to UsernamePasswordAuthenticationProvider.
The AuthenticationProvider provides user and team helper methods that check the checksum before calling the UserManager user/team update methods, but I'm just always blinding calling them directly from AuthenticationManager for every request (several per page load). That works but is quite inefficient on I/O, and making servlet authentication handling pluggable would enable using the helper methods as they currently stand within AuthenticationProvider.
This would also provide the benefit of each servlet mechanism being able to indicate their own support for support{DisplayName,Credential,EmailAddress,TeamMembership}Changes, whereas now it is required to be somewhat awkwardly handled in various permutations in the AuthenticationManager itself. The findProvider behavior of returning the NULL AuthenticationProvider if none other is found gets in the way here.
I'd appreciate any comments, there are a few items that I wasn't sure on, so I've submitted the pull request more as an RFC.

Should a specific AccountType be created for each provider method (what I've done), or should the existing EXTERNAL type by used?

The existing handling for AuthenticationManager uses flagSession which requires the AuthenticationType, so I added that to the AuthenticationProvider abstract, however it wasn't clear which values should be mapped through and how its handling differs from AccountType.

I haven't yet converted the existing servlet auth mechanisms to AuthenticationProvider yet, but I think for this to be complete that would need to be done as well. If you're okay with the approach I put together I will do that final conversion.

The existing requiresCertificate variant of the authenticate(httpRequest) method becomes a bit odd with the new construct. My thought is that this option ends up more cleanly supported as requiring only the (future reworked) X509AuthProvider loaded in order for the realm.authenticationProviders config value.
 For example, setting git.requiresClientCertificate previously would now look like:
         realm.authenticationProviders = x509client

 But optionally providing client cert, but not requiring it, looks something like:
         realm.authenticationProviders = x509client basicauth

 The 1.6.2 default processing order would look like:
         realm.authenticationProviders = container x509client sessioncookie basicauth

I'd also like to disable local auth entirely for my installation. The easiest way to do that seems to be move AuthenticationManager.authenticateLocal() away from an internal always executed call, and into an AuthenticationProvider module. Are there some assumptions baked in which I haven't noticed about auth providers being always external vs. local/internal authentication?

I'm ok with the approach you have coded so far to support HTTP header-sourced users.  I would prefer not to use the config settings of other realms in your new realm (e.g. realm.container.autoCreate).
In point 4 you have implicitly redefined an AuthenticationProvider to mean mechanism of authentication rather than provider of user authentication  (e.g. cookie, basic, certificate, etc VS ldap, htpasswd, PAM, Windows, etc).  If you want to make the supported mechanisms  configurable this will need to be a new set of classes/interfaces - like AuthenticationMechanism or something to that effect - and a different PR.
In point 5 you'd like to make local a separate authentication provider.  Conceptually I'm ok with that idea, but it will introduce even more permutations too an already complex system.  e.g. you can't manually add/create accounts without local.  You may (or may not) need local to administer teams.  We'd have to identify all the points local is implicitly relied on and handle them.  This would also need to be a separate PR.
As for your hope to remain stateless.... that'll be hard.  Wicket is a stateful framework and there are pages where this is the required behavior.  Gitblit does make an effort not to trigger all of Wicket's stateful record keeping behavior and generally succeeds, but you'll find that trying to avoid a Session is going to cause you pain.

I've done some further work on the refactoring and turning everything into an AuthenticationProvider and ran into the issues you expected with mechanism vs. provider. I think it's still overall worth doing, but will take a bit more work. I'll break that off into a separate branch/PR, hopefully within a week or two. Meanwhile, I'll do some cleanup on the in-line HTTP header auth as is.
As for a separate realm autocreate flag, I had just used that one initially since it was there and easy. Creating a separate realm.X.autoCreate flag would seem a bit like proliferation of config options. One thing that I appreciate about the gitblit config file is that it's well documented, as easy to read through all of the options when doing an initial setup. Adding a bunch more options injudiciously would detract from this.
One may, however, legitimately want to auto create accounts for HTTP headers, LDAP, and PAM authentication, but not X509 authentication, so it does provide that flexibility. Perhaps a top-level value of realm.autoCreate which is used by each of the individual realms unless they are specifically defined. Let me know how you'd prefer this to be done.
On the statelessness, I think my biggest gripes aren't the state, but how wicket adds the JSESSIONID to the URL as well as cookie until the cookie is sent back. It ends up being ugly to map on the proxy side, so I'd like to just be able to disable that - but again, a separate PR.

Also, what is the correct approach for AccountType for the HTTP header auth? The semantic difference between AccountType and AuthenticationType isn't clear.
The definition of AccountType in Constants.java contains EXTERNAL which would seem to include CONTAINER, LDAP, HTPASSWD and others, since they're all listed separately. However, the only use of AccountType.EXTERNAL is in ConfigUserService.java, with the following snippet:
if (Constants.EXTERNAL_ACCOUNT.equals(user.password) && user.accountType.isLocal()) {     user.accountType = AccountType.EXTERNAL; }

Note that this PR is currently incompatible with https://github.com/gitblit/gitblit/pull/908, once that PR is addressed, I'll update this one to change flagSession to flagRequest.

Rebased pull source branch against v1.7.1 tag (cleanly applies against master) which includes merge of #908


Implement Core Dump for modeling app state

Since this greatly increases the size of the JSON generated by core dump, this changes the handling of core dump data from being inlined into a GitHub issue into a uploaded attachment. A user facing toast message is added when the core dump is triggered.
Gathering XState data has been deferred for now pending the upgrade to XState 5.
Closes https://github.com/KittyCAD/modeling-app/issues/2338 https://github.com/KittyCAD/modeling-app/issues/2316 https://github.com/KittyCAD/modeling-app/issues/2650
Updates https://github.com/KittyCAD/modeling-app/issues/1573

Added e2e test

@jessfraz says ignore codecov


PR label test

Introduction
In the introduction, please describe the type of change introduced by the pull request and its purpose and link the GitHub issue that this pull request addresses if possible, or add any reference that would provide more context to the reviewer.
Changes
Please list the changes introduced by the pull request and the important design decisions that were made.
Behavior
How the new behavior is different from the current behavior, how you tested and how the reviewer can test it.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
:loudspeaker: Thoughts on this report? Let us know!.

test successful


Results viewer: horizontal scroll with mouse wheel - does not work on windows 10

Version: 5.0.6
OS: windows 10
General information:
Tested with postgreSQL and mssql.
Tested to use the mouse pad on my laptop also on one machine which does not read it. Also tested with a standard mouse with no effect change.
Vertical scroll works perfect.
Tested the scroll on macOS also but there it works with the mousepad on the laptop. seems to be isolated to windows for some reason.

You need to press SHIFT to perform horizontal scroll

Verified that it works with Shift + mouse on windows 10
Trackpad on laptop from when I tested did not work with Shift though (windows 10).
Shouldn't this be the same binding on macOS btw? Where shift is the key to use for doing vertical horizontal scroll?

Trackpad works a bit different in SWT (not like mouse wheel).
It works on MacOS but with trackpad it makes both vertical+horizontal scrolling (diagonally) with shift. A bit weird :)


MAINT: Fix CubicSplinerInterpolator for pandas

Follow up of #10021. Should fix pandas-dev/pandas/issues/26189 I have tested it locally.
But I must say that their support of scipy interpolators is quite fragmentary.
@rgommers

Straightforward enough, thanks @nmayorov!

Agree with no explicit test for this - testing structurally with datetime/timedelta seems a bit out of scope.


ConstShapedNeighborhoodIterator doesn't loop over all possibilities

In a region of 12×12 pixels, an iterator is expected to hit all 144 possibilities.  However it gets only 142 of them.
Steps to Reproduce
#include "itkImage.h"
#include "itkConstShapedNeighborhoodIterator.h"
#include "itkConstantBoundaryCondition.h"

int man(int argc, char * argv[])
{
    using PixelType = unsigned char;
    constexpr unsigned int Dimension = 2;
    using ImageType = itk::Image<PixelType, Dimension>;
    using ImagePointer = typename ImageType::Pointer;
    using RegionType = typename ImageType::RegionType;
    using IndexType = typename RegionType::IndexType;
    using SizeType = typename RegionType::SizeType;
    using OffsetType = typename RegionType::OffsetType;
    ImagePointer imageTest = ImageType::New();
    imageTest->SetRegions(RegionType{ IndexType{ 0, 0 }, SizeType{ 10, 10 } });
    imageTest->Allocate();
    using BoundaryConditionType = itk::ConstantBoundaryCondition<ImageType>;
    using SquareIterator = itk::ConstShapedNeighborhoodIterator<ImageType, BoundaryConditionType>;
    using RadiusType = typename SquareIterator::RadiusType;
    SquareIterator sqIt(RadiusType{ 1, 1 }, imageTest, RegionType{ IndexType{ -1, -1 }, SizeType{ 12, 12 } });
    sqIt.ActivateOffset(OffsetType{ 0, 0 });
    sqIt.ActivateOffset(OffsetType{ 0, 1 });
    sqIt.ActivateOffset(OffsetType{ 1, 0 });
    sqIt.ActivateOffset(OffsetType{ 1, 1 });
    int numberOfIterations{ 0 };
    for (sqIt.GoToBegin(); !sqIt.IsAtEnd(); ++sqIt)
    {
      ++numberOfIterations;
    }
    std::cout << "numberOfIterations = " << numberOfIterations << " but should be 144." << std::endl;
}

Expected behavior
Prints numberOfIterations = 144 but should be 144.
Actual behavior
Prints numberOfIterations = 142 but should be 144.
Reproducibility
100%
Versions
Master branch (currently 25dea2f73d974855fbdfb13f5f2d7f25e4ec8a4e).
Environment
Ubuntu 20.04.  gcc 10.2.0.
Additional information.
I am not completely sure by any means, but maybe the problem is that the region for the square iterator includes values with x < 0.  Because of this, the .end() value that is calculated as the left-most position of the row after the region can be misinterpreted as being near the right end of the previous row.  If that's true, maybe it is also true that a computed WrapOffset is negative when it is implicitly assumed to always be positive.

The above testing code will be added as Test 6 to itkContourExtractor2DImageFilterTest.cxx with Pull Request #2387. That pull request will fail Tests 5 and 6 until this issue is addressed.

The Test 6 has been commented out.  When this issue is addressed that test should be reinstated, though not necessarily in that file.  Test 5 no longer depends upon this issue being resolved; its implementation was changed to make use of a range rather than an iterator and that bypasses this issue.


New data

I'm going to push this up to a special branch to see if it's something we want to pursue....


Menu icon overlaps with title in modeladmin on mobile

The menu icon overlaps with page title in model admin on mobile.
Steps to Reproduce:

Open a model admin.
Create a small 'mobile' viewport.
See the overlap.

Expect that there is a margin between menu icon and title.
Technical details: Wagtail 2 / Google Chrome

Thanks for reporting! Remember seeing something similar while back: #3703
Dropping this here as a reference.

Thanks! Merged with 3bcefe3


Upgrade Vertex integration google-cloud-aiplatform minimum required version to 1.34.0

Describe changes
Following an issue raised in this slack conversation it appears that using the native Vertex scheduling capacity for scheduling pipelines with the Vertex orchestrator needs google-cloud-aiplatform>=1.34.0
Pre-requisites
Please ensure you have done the following:

[X] I have read the CONTRIBUTING.md document.
[ ] If my change requires a change to docs, I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] I have based my new branch on develop and the open PR is targeting develop. If your branch wasn't based on develop read Contribution guide on rebasing branch to develop.
[ ] If my changes require changes to the dashboard, these changes are communicated/requested.

Types of changes

[X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Other (add details above)

Thanks, @francoisserra !


Client gets stuck on boot

After building using build-ouinet.sh (using commit 2cda0c05), I run:
$ ouinet-build/client --repo ouinet/repos/client \
  --injector-ep INJECTOR_IP:INJECTOR_PORT \
  --injector-ipns INJECTOR_IPNS

The client shows this and then gets stuck at that point:
Default RLIMIT_NOFILE value is: 1024
RLIMIT_NOFILE value changed to: 4096

netstat shows no open ports for the process (which according to PS has reserved 20000 TiB virtual space), it uses no CPU and attaching to it with GDB shows:
#3  0x000056406e886ccb in boost::asio::detail::posix_event::waity<boost::asio::detail::scoped_lock<boost::asio::detail::posix_mutex> > (this=0x611000000318, lock=...)
    at /usr/include/boost/asio/detail/posix_event.hpp:106

The injector is working (it replies to proxy requests).

The problem went away when running the GNUnet services, I'll improve the docs.


"Collective" synonymous to "Organization"?

Want to make sure we are clear on this ... mainly because most (at least in the U.S.) will not equate "collective" with a nation-state or large organization.
If we use term "collective"...must make sure it is equated to "organization" clearly.

I'm using 'collective' since it's the way I refer to this in the codebase of Sovereign. But if it makes more sense to name this as 'Organization' happy to do it. I am in fact aware of the connotation that collective might have and it could be wise to change this. You think its better to go with 'Organization'?

i'd vote for organization and agree about challenge of the word 'collective'

great to see you here dwight! way to go. will change to 'organization'.


fix(menu): unable to bind to xPosition and yPosition

Fixes not being able to use data bindings on the xPosition and yPosition properties.
Note: These changes could be considered breaking, if somebody was implementing the MdMenuPanel interface.

Fixes #4169

Addressed the feedback @kara.

Also fixes #1329

There are some teams in google that extend MdMenuPanel, we will need to update them before merging


Use a content security policy

This adds the GOV.UK content security policy to the demo version of this app and adds a nonce to all inline javascript. This will then allow other apps to add a nonce generator which disables unsafe inline scripts and the components won't have violations.
This also adds unsafe_eval to the Content Security Policy on component-guide routes as the axe-core script uses a dependency that makes use of eval.  This will resolve reports like:

I've found I've got a CSP issue to resolve in Firefox before I can merge this 😭

So it turns out that since we still use jQuery 1.12.* we will get CSP violations if unsafe-inline is switched off. This is due to some jQuery code which sets inline onsubmit / onchange / onfocusin attributes to an element. Looks like this code is removed on later jQuery versions.
Annoyingly this only raises an issue in Firefox, with Chrome and Safari not having a problem.
cc @tijmenb this is relevant if you're wondering how we can work towards turning unsafe-inline off for GOV.UK

@kevindew what would the minimum version of jquery need to be to avoid these issues? Could we upgrade? (I realise the enormity of this suggestion)

@kevindew what would the minimum version of jquery need to be to avoid these issues? Could we upgrade? (I realise the enormity of this suggestion)

jQuery 2 no longer has that so a version of 2.2 would suffice. I believe historically we've not upgraded from jQuery 1.x due to it supporting IE8 however I assume that time has moved on and that's not the same concern it once was.
I think @alex-ju is working towards removing jQuery altogether as a dependency for these components.

Replaced by https://github.com/alphagov/govuk_publishing_components/pull/1145


Fixes #18.

Previously, iterating over zero-length matches did not advance
correctly at UTF-8 character boundaries.
(The empty_match_unicode_* tests are regression tests.)

Thanks!


Flipping orientation, then dismissing vc causes strange sizing issues

Here is a video of it in action: https://www.dropbox.com/s/l96bjv49s522erd/rotation issues.mp4?dl=0
If I comment out:
func animationController(forDismissed dismissed: UIViewController) -> UIViewControllerAnimatedTransitioning? {
    return FTBAnimationController(displayType: .dismiss, direction: .down, speed: .moderate)
  }

It works perfectly, but the segue animation is mega lame!

Thank you for the feedback. The code currently does not account for changes in orientation. As mentioned in the readme, it is designed and intended for portrait orientation only.


Curious build failure with Python 3.7 on Windows

Running python -m build . in GitHub CI ends with this error:
+ meson install --destdir C:\Users\RUNNER~1\AppData\Local\Temp\build-via-sdist-x9mu6s_a\siphash24-1.1.dev0\.mesonpy-wgvn__mt\install
ninja: Entering directory `C:\Users\RUNNER~1\AppData\Local\Temp\build-via-sdist-x9mu6s_a\siphash24-1.1.dev0\.mesonpy-wgvn__mt\build'
ninja: no work to do.
Installing siphash24.pyd to C:\Users\RUNNER~1\AppData\Local\Temp\build-via-sdist-x9mu6s_a\siphash24-1.1.dev0\.mesonpy-wgvn__mt\install\hostedtoolcache\windows\Python\3.7.9\x64\Lib\site-packages
Installing siphash24.lib to C:\Users\RUNNER~1\AppData\Local\Temp\build-via-sdist-x9mu6s_a\siphash24-1.1.dev0\.mesonpy-wgvn__mt\install\hostedtoolcache\windows\Python\3.7.9\x64\Lib\site-packages
    return mesonpy._tags.StableABITag(tag)

ERROR Backend subprocess exited when trying to invoke build_wheel
  File "C:\Users\RUNNER~1\AppData\Local\Temp\build-env-0_b2_gmq\lib\site-packages\mesonpy\__init__.py", line 310, in _calculate_file_abi_tag_heuristic_windows
    return mesonpy._tags.StableABITag(tag)
  File "C:\Users\RUNNER~1\AppData\Local\Temp\build-env-0_b2_gmq\lib\site-packages\mesonpy\_tags.py", line 37, in __init__
    raise ValueError(f'Invalid PEP 3149 stable ABI tag, expecting pattern `{self._REGEX.pattern}`')
ValueError: Invalid PEP 3149 stable ABI tag, expecting pattern `^abi(?P<abi_number>[0-9]+)$`

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "C:\hostedtoolcache\windows\Python\3.7.9\x64\lib\site-packages\pep517\in_process\_in_process.py", line 351, in <module>
    main()
  File "C:\hostedtoolcache\windows\Python\3.7.9\x64\lib\site-packages\pep517\in_process\_in_process.py", line 333, in main
    json_out['return_val'] = hook(**hook_input['kwargs'])
  File "C:\hostedtoolcache\windows\Python\3.7.9\x64\lib\site-packages\pep517\in_process\_in_process.py", line 250, in build_wheel
    metadata_directory)
  File "C:\Users\RUNNER~1\AppData\Local\Temp\build-env-0_b2_gmq\lib\site-packages\mesonpy\__init__.py", line 987, in build_wheel
    return project.wheel(out).name
  File "C:\Users\RUNNER~1\AppData\Local\Temp\build-env-0_b2_gmq\lib\site-packages\mesonpy\__init__.py", line 935, in wheel
    wheel = self._wheel_builder.build(self._build_dir)
  File "C:\Users\RUNNER~1\AppData\Local\Temp\build-env-0_b2_gmq\lib\site-packages\mesonpy\__init__.py", line 578, in build
    wheel_file = pathlib.Path(directory, f'{self.name}.whl')
  File "C:\Users\RUNNER~1\AppData\Local\Temp\build-env-0_b2_gmq\lib\site-packages\mesonpy\__init__.py", line 189, in name
    python_tag=self.python_tag,
  File "C:\Users\RUNNER~1\AppData\Local\Temp\build-env-0_b2_gmq\lib\site-packages\mesonpy\__init__.py", line 244, in python_tag
    selected_tag = self._select_abi_tag()
  File "C:\Users\RUNNER~1\AppData\Local\Temp\build-env-0_b2_gmq\lib\site-packages\mesonpy\__init__.py", line 375, in _select_abi_tag
    tags = self._files_by_tag()
  File "C:\Users\RUNNER~1\AppData\Local\Temp\build-env-0_b2_gmq\lib\site-packages\mesonpy\__init__.py", line 361, in _files_by_tag
    tag = self._calculate_file_abi_tag_heuristic(file)
  File "C:\Users\RUNNER~1\AppData\Local\Temp\build-env-0_b2_gmq\lib\site-packages\mesonpy\__init__.py", line 346, in _calculate_file_abi_tag_heuristic
    return self._calculate_file_abi_tag_heuristic_windows(filename)
  File "C:\Users\RUNNER~1\AppData\Local\Temp\build-env-0_b2_gmq\lib\site-packages\mesonpy\__init__.py", line 312, in _calculate_file_abi_tag_heuristic_windows
    return mesonpy._tags.InterpreterTag(tag)
  File "C:\Users\RUNNER~1\AppData\Local\Temp\build-env-0_b2_gmq\lib\site-packages\mesonpy\_tags.py", line 83, in __init__
    f'Unknown Python implementation: {self.implementation}. '
NotImplementedError: Unknown Python implementation: mesonpy. Please report this to https://github.com/FFY00/mesonpy/issues and include information about the Python distribution you are using.

ERROR Backend subprocess exited when trying to invoke build_wheel

Funny enough, the same builds on macOS and Linux and on Windows with Python 3.8 and later. I have absolutely no clue about what may be wrong.

Investigating a little, I see that the first issue is that the extension filename on Python 3.7 is siphash24.pyd while on Python 3.8 and later is something like siphash24.cp38-win_amd64.pyd. This seems a Meson bug probably triggered by some Python standard lib difference between Python 3.7 and later versions.
The second issue is that the unexpected file path confuses the regular expression here https://github.com/mesonbuild/meson-python/blob/5dba6571bf89f89ba08840e75ec99c20214363cd/mesonpy/init.py#L102
_WINDOWS_NATIVE_MODULE_REGEX = re.compile(r'^(?P<name>.+)\.(?P<tag>.+)\.pyd$')

Maybe the name and tag groups could accept a narrower charset, something like:
_WINDOWS_NATIVE_MODULE_REGEX = re.compile(r'^(?P<name>[^.].+)\.(?P<tag>[^.]+)\.pyd$')

and the match be done on the filename part only (now I have the impression that the whole file path is matched).

If the problem is somewhere in the standard library, it may already be fixed, however, GitHub Actions are stuck at Python 3.7.9 on Windows while the current release is 3.7.15.

Meson names the extension module that way because it uses what the sysconfig module tells him to use, see https://github.com/mesonbuild/meson/blob/e68fcac919b332c7f9469672a243d2aab1bfea0a/mesonbuild/modules/python.py#L444
In Python 3.7.9 on Windows:
python -c 'import sysconfig; v = sysconfig.get_config_vars(); print((v.get("EXT_SUFFIX"), v.get("SO"), v.get(".so")))'
('.pyd', '.pyd', None)

While with Python 3.8.10 on Windows:
python -c 'import sysconfig; v = sysconfig.get_config_vars(); print((v.get("EXT_SUFFIX"), v.get("SO"), v.get(".so")))'
('.cp38-win_amd64.pyd', '.cp38-win_amd64.pyd', None)

Digging through the Python changelogs, I found that sysconfig was fixed to behave correctly on Windows in Python 3.8.7, see https://docs.python.org/3.8/whatsnew/changelog.html#python-3-8-7-final :

bpo-39825: Windows: Change sysconfig.get_config_var('EXT_SUFFIX') to the expected full platform_tag.extension format. Previously it was hard-coded to .pyd, now it is compatible with distutils.sysconfig and will result in something like .cp38-win_amd64.pyd. This brings windows into conformance with the other platforms.

Therefore it seems that Meson gets the extension filename wrong on Python versions smaller than 3.8.7 and meson-python trips on that. Most likely something to fix in Meson.

With mesonbuild/meson#10961 Meson generates the extension module with the correct file name: siphash24.cp37-win_amd64.pyd for Python 3.7 vs siphash24.cp38-win_amd64.pyd for Python 3.8. However, pip does not like the wheel that meson-python produces:
ERROR: siphash24-1.1.dev0-cp37-cp37-win_amd64.whl is not a supported wheel on this platform.

despite the fact that the wheel filename is in line with what works just fine on Python 3.8: siphash24-1.1.dev0-cp38-cp38-win_amd64.whl

The correct wheel filename for CPython 3.7 is siphash24-1.1.dev0-cp37-cp37m-win_amd64.whl

Thank you for looking into it, it is really appreciated! It seems like we may need to implement a workaround for this bug.

I don't think a workaround is necessary. The extension module filename can be fixed in Meson, see the linked PR. The wheel ABI tag determined by meson-python is just wrong and need fixing, but I would not call this a workaround.
There is no way to get the correct ABI tag from just the extension filename. However, I think meson-python can assume that the interpreter used to run itself is the one it is building for, thus it can use introspection to get the correct ABI tag.
For reference, here is what wheel does to compute the ABI tag https://github.com/pypa/wheel/blob/7b9e8e1022b9c850756ef34bc1a4a88042988a86/src/wheel/bdist_wheel.py#L71


What is the "type" parameter on RTCDataChannel.send

Ordinarily, that method only takes one argument, for the data. In this implementation, there's a "type" string parameter there too, and I'm not sure what exactly is supposed to go there. I can't find anything about it in the documentation or even any information on what it does in the source code. What is that string supposed to signify?

According to source type param must be 'text' or 'binary'

Ah, thank you.


(chores) Cleanup unused throws

components: components/camel-bean, components/camel-bindy,
components/camel-caffeine, components/camel-consul,
components/camel-cxf-transport, components/camel-docker

There may be warnings about formatting issues, but they should be fixed by #5715.


Upgrade the CLI to use Go 1.12.

This does not change our supported operating systems, Go 1.12 being the
last release supporting macOS 10.10 Yosemite.

@mesosphere-ci run integration tests

Waiting for https://github.com/docker-library/golang/pull/264 to be merged.

The new ProcessState.ExitCode method returns the process's exit code.

We should be able to simplify this by using exitErr.ExitCode():
https://github.com/dcos/dcos-cli/blob/b4a7d91628ecec93c4f4dc4ab7cb9ae7b4ccb29f/pkg/cmd/dcos.go#L234-L244
Wanna give it a try in this PR?

@mesosphere-ci run integration tests

@mesosphere-ci run integration tests

For Appveyor: https://github.com/appveyor/ci/issues/2875

@mesosphere-ci run integration tests

@mesosphere-ci run integration tests


[Support Request]picture upload problems after let's encrypt certificate enabled

Attempted Debugging

[X] I have read the debugging page

Searched GitHub Issues

[X] I have searched GitHub for the issue.

Describe the Scenario
I have a problem witch I cant find a solution for it. maybe someone can put me in the right direction. I have installed Bookstack on Ubuntu server LXC 20.04 on Proxmox. For reverse proxy I'm using NginxProxyManager. So Evrything is working fine when im usuing local on an local Ip, when I go thru NginxProxyManager the Http is working fine, but after I request an  Let's Encrypt certificate after login in I can't upload picuters. Im using the latest version BookStack v21.11.1
It asking for a login and password ,and after I put it again it just crashes. I can find a solution for it.
Exact BookStack Version
BookStack v21.11.1
Log Content

PHP Version
7.4
Hosting Environment
Ubuntu LXC 20.04

Hi @theonlyone54,
Have you updated the APP_URL variable in the .env file to match your new https:// address?

Hi @theonlyone54, Have you updated the APP_URL variable in the .env file to match your new https:// address?

thank you, after changing this to https everything works now.

Great! will therefore close this off.


Do not restrict to canonical values

RFC7643 §2.3.1 indicates that string types with canonical values defined can or cannot have other values, depending on the server configuration:
   Additionally, when "canonicalValues"
   is specified, service providers MAY restrict accepted values to the
   specified values.

In §7 it is indicated that values can be ignored:

      canonicalValues  A collection of suggested canonical values that
         MAY be used (e.g., "work" and "home").  In some cases, service
         providers MAY choose to ignore unsupported values.  OPTIONAL.

Currently scim2_models defines custom enums, but that makes pydantic enforce those values only.
https://github.com/yaal-coop/scim2-models/blob/56287ad73027f6e6825d41e0b0c6ef0a339f4f18/scim2_models/rfc7643/user.py#L44-L48
https://github.com/yaal-coop/scim2-models/blob/56287ad73027f6e6825d41e0b0c6ef0a339f4f18/scim2_models/rfc7643/user.py#L56-L57
In that situation scim2-models should accept both arbitrary and enum values.
Also, when #6 is implemented, scim2-models should generate dynamic enums.

This test should pass:
def test_canonical_values():
    class Model(Resource):
        class Foo(str, Enum):
            bar = "bar"
            baz = "baz"

        schemas: List[str] = ["urn:example:2.0:Extension"]
        foo: Optional[Foo]

    valid = Model(foo="bar")
    assert valid.foo == "bar"

    valid = Model(foo="invalid")
    assert valid.foo is None


chore: fix broken links

What this PR changes/adds
The links in the main readme were updated as they were broken. To fix some broken links, the docs/developer/decision-records directory had to be created. A readme was placed in the latter to populate the folder.
Why it does that
To make links work again.
Further notes
No further notes.
Linked Issue(s)
No issues was created.

please update the dependencies file, then we'll be ready to go

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (0519c97) 64.54% compared to head (ce8551b) 64.54%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #146   +/-   ##
=======================================
  Coverage   64.54%   64.54%
=======================================
  Files          26       26
  Lines         629      629
  Branches       28       28
=======================================
  Hits          406      406
  Misses        218      218
  Partials        5        5

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

@rafaelmagalhaes-ctw stil not working, my suggestion would be to copy and paste the output of the failed job: https://github.com/eclipse-edc/Technology-Aws/actions/runs/6980195692/job/18995724372?pr=146


Add table with enumerations links to enumerations-excel.md

The enumerations-excel.md file is missing documentation for available enumerations -- the pages / files can only be accessed on the sidebar on the official site. I've added a Markdown table with links to all of the files.

 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.:x: paskhaver sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

VBA/Excel-VBA/articles/enumerations-excel.md
:white_check_mark:Succeeded

VBA/Excel-VBA/articles/pagesetup-orientation-property-excel.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.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

VBA/Excel-VBA/articles/enumerations-excel.md
:white_check_mark:Succeeded

VBA/Excel-VBA/articles/pagesetup-orientation-property-excel.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.


Is there any documentation for this somewhere?

I'd love to see a basic example or a documentation of using this

Thanks for asking! There isn't a completely worked example or docs yet, but there's a very tiny app in the test suite based on Flask's app factory pattern:
https://github.com/SteelPangolin/flask-oidc/blob/master/tests/app.py
https://github.com/SteelPangolin/flask-oidc/blob/master/tests/test_flask_oidc.py#L66
http://flask.pocoo.org/docs/0.10/patterns/appfactories/

I've gotten it where it calls back after prompting for authentication.
How to fetch the user details though...?

I have taken over maintenance of this project, and will write documentation in the next few days.
I have now added a basic example in the main directory as example.py.
This will show how to get the users' email address, which at this moment is the only piece of information retrieved.
I'm working on generalizing this.

There is now documentation for v1.0: http://flask-oidc.readthedocs.io/en/latest/.


merge from template-sync/template

.github/workflows/ci.yml

chore: (jobs.test-node.steps)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>0
Details

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 95.161%

Totals

Change from base Build<PHONE_NUMBER>2:
0.0%

Covered Lines:
54

Relevant Lines:
56

💛  - Coveralls


Build failure: beets

Steps To Reproduce
Steps to reproduce the behavior:

build beets

Buiid attempted on master. Looks like a sphinx issue?
Build log
https://gist.github.com/lunik1/f99cb57de250fab7a6a21a4f3b7d5bfa
Additional context
Add any other context about the problem here.
Notify maintainers
@aszlig @doronbehar @lovesegfault @pjones
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
 - system: `"x86_64-linux"`
 - host os: `Linux 6.6.0, NixOS, 23.05 (Stoat), 23.05.20231118.decdf66`
 - multi-user?: `yes`
 - sandbox: `relaxed`
 - version: `nix-env (Nix) 2.18.1`
 - channels(root): `"nixos"`
 - channels(lunik1): `""`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

This was fixed on beets master in beetbox/beets@2106f47 (not available in any tagged releases yet). I've applied this as a patch and it fixes the issue, although I'm now seeing build issues with test_aura.py which I'm looking into.


Problem with displaying photos on the server

Laravel Version: 5.4.25
Voyager Version: 0.11.14
PHP Version:
Database Driver & Version:

Description:
Then I load photos with new user via admin panel everything is good. I see photo in preview in Users folder and in Media/Users. But then I've published project on server at first time then I've sign in admin panel I saw notification "Lost Simylink" so I've fixed it by submiting notification. After that I tried to sing up a new user but photo is not loaded. In Media/User I saw two photo was uploaded the first one is good an I saw it but the second was damaged and this one was in  <img src="">.
Could any body help with it?
Steps To Reproduce:

Labels: help wanted, question


Zomato UI Clone

issue no. #644

🛠 Issue (Number)

Closes #644
Description
Please include a summary of the change. Please also include relevant motivation and context. List any dependencies that are required for this change.
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist:

[x] I have made this from my own
[ ] I have taken help from some online resourses
[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] The title of my pull request is a short description of the requested changes.

ATTACH SCREEN-SHOTS / DEPLOYMENT LINK

Make it responsive

 Make it responsive

Sir It is Responsive, pls Kindly Elaborate what I have to implement!!

 Make it responsive

Sir It is Responsive, pls Kindly Elaborate what I have to implement!!

Go and search "What is responsiveness?" and then implement that
And push all the changes


The variable $from should be the variable (parameter) $conn

Duplicate of #413. Sorry that took me a while to merge in. Thanks for the PR and reminder, it's been fixed!

I saw that MR, this is another instance of you using $from where it should be $conn though.
Check the onError method. (line 169)

Whoops! My apologies. Thanks for catching this!


Redox Wireless receiver LEDs not lighting up

I'm having an issue with the receiver: I can program it by header, but when I connect it to the Arduino Pro Micro, no lights appear. I've assembled several units at this point, since I ordered some redundant parts, and they all have the same behavior. If pin  validated the continuity (testing for shorts as well) but the lights still do not turn on.
I record a voltage drop over the LED/resistor as well, so it seems odd that nothing is visible. Do you have any thoughts?

https://photos.app.goo.gl/ZBaQ3kaNugr6mHov6
https://photos.app.goo.gl/jb4UTSCqM7cQC6G86
I've confirmed with Falba that their PCB's are behaving correctly. The above pictures are of one of the assembled boards.

Testing the voltage on my pins, it's becoming clear that none of them are reaching a sufficient low for current to pass. They seem to be floating. It's possible that the arduinos are no good. At least one of the ones I was testing with turned out to be the 3.3V Pro Micro

It's possible that the arduinos are no good. At least one of the ones I was testing with turned out to be the 3.3V Pro Micro

Yup. You should use 5V Pro Micros.

Sorry, this is not closed. The 5V Pro Micro has not solved my issue

From #94

I think I've confirmed that the problem is on the Arduino/QMK side of things. It's an arduino micro clone, but it's still using an Mega32U4-MU. Arduino IDE identifies it as a leondardo, which I understand is fairly common for micro clones.

Did you pinpoint the issue? What resistor values did you use for the LEDs? Does the receiver works (i.e. keystrokes are registered?) Does this happen for all the LEDs?

The issue hasn't been pinpointed, but at least one receiver is working. Only the first one appears to have been bricked, which led me down a bit of a goose chase. In fact I'm writing on it now.
I'm referring to the LEDs on the receiver board. Now that it's transmitting, it appears that the bottom LEDs do function. I'm using the default configuration for the LED/resistor pairs. Red, Blue, Green, White in that order top to bottom. 330, 220, 330, 220.

I'm referring to the LEDs on the receiver board. Now that it's transmitting, it appears that the bottom LEDs do function. I'm using the default configuration for the LED/resistor pairs. Red, Blue, Green, White in that order top to bottom. 330, 220, 330, 220.

The LEDs light up when the layer is switched. To trigger a layer switch you need to press the following keys in the default keymap:

Refer to the default keymap for details.
Please note that the fourth LEDs is not mapped to any function in the default keymap and thus won't light up.

I'm using the default configuration for the LED/resistor pairs. Red, Blue, Green, White in that order top to bottom. 330, 220, 330, 220.

You should be good then.

I am having similar issues, I am using the SparkFun Pro Micro. However, I do not have any LEDs lighting even after following this issue. I have checked the leads and all seems proper, and my reset button works, but the LEDs do nothing. Any ideas for that (or should I make a new issue)?

@TheCatster At this point I’m suspecting something broke in QMK. If you burn the pre compiled QMK hex files available here do the LEDs light up? That file (qmk_redox_w_default.hex) was built using an older version of QMK and should work.

I'll try it out as soon as I'm done flashing the left hand here. To add to your possible suspicion, I haven't actually tried your QMK file yet, I have only compiled my own keymaps!

Just tested it with the latest version of QMK firmware. Everything is working fine.
@TheCatster If you can't connect the transmitters to the receiver the LEDs won't light up as they're triggered by key presses. You won't get keypresses until the firmware for the trasmitters MCUs, the firmware for the receiver MCU and the QMK firmware for the Arduino are all burned on their MCUs as described here. If you still have issues, please open a new issue.
@Goldielocks Since I didn't receive any update I'm closing this. Feel free to reopen this if you still have issues.

Hi, I'm currently running into this issue. I can only get the LEDs to work if I use the default firmware from the redox repository. If I try to compile it myself (using 0.15.25 due to https://github.com/qmk/qmk_firmware/issues/16553) everything seems to work fine except the LEDs on the arduino. Is there anything in QMK that could have changed to break this? Or a setting that needs to be changed from default?

Just to sanity check, I tried with the version of QMK that would have been latest at the time of your last comment, 0.12.32, and found the same behavior.

Okay, sorry for the triple post here, but I wanted to update you to say I found the solution.
The root cause is definitely that I am a noob, but I think there is a serious lack information here. It took me many hours to find that the LED behavior is implemented in the keymap at the very bottom. The very limited information about this I can find online never mentions this, and in fact seems to imply that it's built into the firmware.
All the guides to first time users creating their first firmware suggests to use the online configurator, which of course creates a keymap file from scratch, and won't have the above code in it. Is there no way this could be implemented into the keyboard-specific firmware code in QMK, rather than the keymap?
Again, sorry for all the comments about this, but I wanted to get all this info out there for others in the future, and in the hope it might be improved. Thank you for all your work on this keyboard, it really is great to use. This issue is minor in the grand scheme, after all.

That's quite interesting. I actually gave up on the LED, thinking it was something I had done wrong (I guess in a way it was). This would be nice to get in the firmware, but as @tterrag1098 said it's a minor issue overall.


cetz-venn:0.1.2

I am submitting

[ ] a new package
[x] an update for a package

Update to make cetz-venn work with cetz >= 0.3.1.
I have read and followed the submission guidelines and, in particular, I

[ ] selected a name that isn't the most obvious or canonical name for what the package does
[ ] added a typst.toml file with all required keys
[ ] added a README.md with documentation for my package
[ ] have chosen a license and added a LICENSE file or linked one in my README.md
[ ] tested my package locally on my system and it worked
[ ] excluded PDFs or README images, if any, but not the LICENSE

[ ] ensured that my package is licensed such that users can use and distribute the contents of its template directory without restriction, after modifying them through normal use.

Thank you!


[Errors] error recovery when /// or import cannot be resolved

Expected: continue checking the rest of the file
Actual: compiler stops processing the file immediately

I don't think this is still true, is it?

Nope. Fixed.


MNT: monitor fixes

Current coverage is 82.17% (diff: 100%)

Merging #542 into master will increase coverage by <.01%

@@             master       #542   diff @@
==========================================
  Files            21         21
  Lines          4043       4044     +1
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           3322       3323     +1
  Misses          721        721
  Partials          0          0

Powered by Codecov. Last update 4521828...b7d724f

👍


Null of comlex type in List and Dictionary<TKey, TValue> deserialized as non-null object

hi,
i have some problems to unpack list with null elements.
how to fix it please
public class SimpleItem
{
    public int Id;
    public SimpleItem()
    {
}
}
var items= new List<SimpleItem>();
                    items.Add(null);
items[0] = new SimpleItem();

var stream = new MemoryStream();
var context = new SerializationContext();
var serializer = context.GetSerializer<SimpleItem>();

serializer .Pack(stream, items);
stream.Position = 0;
var cloneItems = serializer .Unpack(stream);

cloneItems null item in list is not null, why?

Sorry, it is a critical bug. Thank you for reporting!

hi, thank, i can investigate if you help me, i want to fix it quilkly.
i use build dotNet3.5.
I have meet order dysfonctionnement ,I write other issue after.
but good job is very fast serializer.

Thank you very much for your reactivity to this problem.  i try it now. good job

Thank you. I will release it as 0.8.1 soon.

BTW, your "order dysfonctionnement" is related to #71?

Fixed in 7e73aff


Add comment on why gradle action supports iOS also

          1 Warning

:warning:
Please provide a changelog summary in the Pull Request description @KrauseFx

  Generated by :no_entry_sign: Danger


Add a separate Docker stack with Prometheus

The following steps need to be performed:

[x] Create a docker-compose.yml file that runs all the Prometheus dependencies
[x] Configure the Prometheus to collect information from Actuator - update application.yml files to export metrics in the Prometheus format

Implemented in #48, so, closing the ticket.


Add back button to the settings page

User can not close settings page if the app was collapsed

Oped settings in the app
Press back button -> settings are closed because previously main app screen was opened
Open settings again
Collapse the app
Press back button on Android

Actual Result: App is closed by back button because it was collapsed previously
Expected Result: Add back button that can always return user to the main screen of the app

https://user-images.githubusercontent.com/85628357/127322186-5a9c5c50-d25c-43e6-becb-c3067baa0148.mp4

Tested:

back button was added that returns user to the main screen of the app
also issue was fixed: default android back button now returns user to the main screen of the app


mat: improve mat element shadowing detection

Please take a look.
Updates #377.

Is there utility in trying to add a test of this to list_test (or something)? It's easy to review what is here, but harder to review what's missing. Maybe we could automatically check that functions correctly panic with overlap.

I think so - this is why I wrote "Updates" rather than "Fixes".


feat(container): add sysctls setting in the container view

This is picking up #3974 , next up is resolving merge conflicts :)
Edit: Whew, merging a lot of changes + pre-commit hooks make for some confusing changes
Fixes https://github.com/portainer/portainer/issues/2756

@pull-dog up

Thanks for the contribution @Transmitt0r , I'll take care of starting the CI process.

/azp run

@deviantony can you see what the issue is? I'm having trouble running tests locally

Looks like the CI is not properly triggered here. We'll have a look.

/azp run

Great news @deviantony, hope we can get this merged soon then 🥳

@deviantony what's the status on the problem you mentioned above?

@Transmitt0r all our engineers are fairly busy on other topics at the moment, this item is to be reviewed by our QA team soon though.
Do you mind rebasing it?

Hi @Transmitt0r, the issue has been fixed in develop, could you please rebase this pr so I can test the security settings 🙂

Hey @nzbasic @deviantony , sorry for the delay. I merged develop into this PR because rebasing onto develop turned out to be very cumbersome because some of the commit from the originl PR-Author are quit old. But i think you can test the changes now :)

/azp run

🎉

Thanks again for the contribution @Transmitt0r 👍


Empty 'com' folder doesn't get deleted for Android project if package name doesn't start with 'com'

As the title suggests empty com folder should get deleted if the package name doesn't start with com for eg. org.test. This is happening only for Android project.
It should be deleted from three places main test and androidTest
In Android Studio it looks something like this

Easiest way to solve this is  to update https://github.com/theapache64/compose-android-template 's package name from com.theapache64.composeandroidtemplate to com.theapache64


Plot a simple graph with the price of a stock

Question
How do I plot a simple graph with the value of a stock?
This simple code below doesn't work.
Thanks
Code example
import pandas as pd
import yfinance as yf
from lightweight_charts import Chart, JupyterChart

df_ticker = yf.download('MSFT')['Close'].to_frame()
tvchart = JupyterChart()
tvchart.set(df_ticker)
tvchart.load()

# line = tvchart.create_line(df_ticker.columns[0])
# line.set(df_ticker)
# tvchart.add_line(line)
# tvchart.load()

The column names and order matter I believe
Make sure your datarame has ['date', 'open', 'high','low','close'] in that order when your use tvchart.set(df_ticker)
Best to start with the working example and then make sure the input you are supplying the the exact same

Thank you for your comment. But this way the chart is a candlestick (OHLC), but I just want a line with the stock price. The code below works, but it's not what I want.
import pandas as pd
import yfinance as yf
from lightweight_charts import Chart, JupyterChart
df_ticker = yf.download('MSFT')
tvchart = JupyterChart()
tvchart.set(df_ticker)
tvchart.load()

You need to look at the create_overlay example I think
tvchart = JupyterChart()
line = tvchart .create_line(name='Close')
line.set(df_ticker[['Close']])
df_ticker index must be date or datetime index

The code below worked. Thank you very much @karunkrishna !
import pandas as pd
import yfinance as yf
from lightweight_charts import Chart, JupyterChart
df_ticker = yf.download('MSFT')
tvchart = JupyterChart()
line = tvchart.create_line(name='Close')
line.set(df_ticker['Close'].to_frame())
tvchart.load()

The code below worked. Thank you very much @karunkrishna !
import pandas as pd
import yfinance as yf
from lightweight_charts import Chart, JupyterChart
df_ticker = yf.download('MSFT')
tvchart = JupyterChart()
line = tvchart.create_line(name='Close')
line.set(df_ticker['Close'].to_frame())
tvchart.load()


gradient accumulation for stage 2

ZeRO stage 2 does not support gradient accumulation, if you need gradient accumulation please use stage 1

Stage 2 significantly reduces the memory footprint. Is there a plan to support gradient accumulation for stage 2? Thanks.

Thank you for you interest in DeepSpeed! We just added support for gradient accumulation in ZeRO-2. Please let us know if you run into any issues!

hi, I see in ZeRO-2, communicates during non gradient accumulation boundaries as well

Is it for some reason? could you help me understand this?
Looking forward to your reply

I have the same question. why zero-2 need communicate gradient whether it is on the grad acc boundary?


Dacpac Extract crashes SqlToolsService if invalid version is used

Open dacpac wizard and choose extract
Set version to <IP_ADDRESS>abcd
Click Next and kick off the wizard

Actual:
Get error SQL Tools Service component exited unexpectedly. Please restart Azure Data Studio.

Expected:
a) Validation on page blocks going Next until a valid version is defined
b) the SqlToolsService code needs an overall try/catch to avoid crashing if anything goes wrong with the dacpac export. This breaks the entire app

Documented as Known Issue

@kisantia , could you please take a look?

this was fixed in SqlToolsService https://github.com/Microsoft/sqltoolsservice/pull/789


fixed the script.

Closes: #1
Updated validations.py python script.
Fixed the behaviour of validate_user function in validations.py.
Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

[ ] Tests pass
[ ] Appropriate changes to README are included in PR

Fixed username behaviour.

Fixed username behaviour


fix: use tailwind supports syntax to enable backdrop blur

@AnilSeervi This is now fixed by https://github.com/shadcn-ui/ui/pull/1628. Thanks for being the first with the fix. Appreciate it.


log error messages and clean up monitor when indexing doc level queri…

…es or metadata creation fails (#900)

log errors and clean up monitor when indexing doc level queries or metadata creation fails
refactor delete monitor action to re-use delete methods
Signed-off-by: Surya Sashank Nistala<EMAIL_ADDRESS>
Issue #, if available:
Description of changes:
CheckList:

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

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

CI is failing
REPRODUCE WITH: ./gradlew ':alerting:integTest' --tests "org.opensearch.alerting.DocumentMonitorRunnerIT.test execute monitor generates alerts and findings with per alert execution for actions" -Dtests.seed=D611385684093386 -Dtests.security.manager=false -Dtests.locale=lt -Dtests.timezone=America/Paramaribo -Druntime.java=17
org.opensearch.alerting.DocumentMonitorRunnerIT > test execute monitor generates alerts and findings with per alert execution for actions FAILED
    java.util.NoSuchElementException: List is empty.
        at __randomizedtesting.SeedInfo.seed([D611385684093386:28F5267A0603560E]:0)
        at kotlin.collections.CollectionsKt___CollectionsKt.first(_Collections.kt:212)
        at org.opensearch.alerting.DocumentMonitorRunnerIT.test execute monitor generates alerts and findings with per alert execution for actions(DocumentMonitorRunnerIT.kt:259)

Will rerun once

Same test failed again (there's also a number of other failures)
org.opensearch.alerting.DocumentMonitorRunnerIT > test execute monitor generates alerts and findings with per alert execution for actions FAILED
    java.util.NoSuchElementException: List is empty.
        at __randomizedtesting.SeedInfo.seed([340552357EBF6165:CAE14C19FCB504ED]:0)
        at kotlin.collections.CollectionsKt___CollectionsKt.first(_Collections.kt:212)
        at org.opensearch.alerting.DocumentMonitorRunnerIT.test execute monitor generates alerts and findings with per alert execution for actions(DocumentMonitorRunnerIT.kt:259)
REPRODUCE WITH: ./gradlew ':alerting:integTest' --tests "org.opensearch.alerting.DocumentMonitorRunnerIT.test execute monitor generates alerts and findings with per alert execution for actions" -Dtests.seed=340552357EBF6165 -Dtests.security.manager=false -Dtests.locale=hr-HR -Dtests.timezone=Australia/Darwin -Druntime.java=17

Backports for 2.7 were previously completed. Closing this PR


BossBar acting weird.

What is the output link of /viaversion dump?
Bungee:
https://gist.github.com/25740bcc259abcd4c4b77f50226a3918
Bukkit:
https://gist.github.com/47ec2d5795e31545152105f00f2c82c9
Are you using any additional software like BungeeCord? If so, what software and version? (Please list your plugins as well)
I'm using HexaCord normally. Im using ServerListPlus with it, The Viaversion plugin and thats it
How does this error happen? login? Using an item?
It just appears when the bossbar goes on. Im using a plugin named AirBar that creates a BossBar, and it just gives 'Wither' on version higher then 1.8.8 (because thats my main server version).
The BossBar acting weird: https://gyazo.com/c33ab2645d262b235024683379525356
Is there an error in the console? Use pastebin.com. Is there a kick message?
No kick message, no nothing, only when i dont use the ViaVersion plugin in the BungeeCord it gives me this error when using VIaVersion and ProtocolSUpport at the same time.
Kick Message when not using ViaVersion in bungee:
https://gyazo.com/bbccea2c6fabcd2dd937b6a0c1777afe

Its because you're using a leaked version of airbar

Nope, I bought it from spigot. Its all legit.

Custom named withers work fine for me on 1.8. Sadly can't try to reproduce it with that plugin since it's premium.

We sadly can't help you with issues you have with premium plugins. If you have a way to reproduce the issue, I'd love to hear it. Spawning a custom named wither / creating a BossBar with the ViaVersion API works correctly for us.


TypeError: Bad argument

Hello,
When I add jpegoptim in the use array, I get an error, but I can use the rest of the imagemin plugins.
This is the code:

var gulp = require('gulp');

var imagemin = require('gulp-imagemin');
var pngquant = require('imagemin-pngquant');
var jpegoptim = require('imagemin-jpegoptim');
var jpegtran = require('imagemin-jpegtran');

gulp.task('optimg', () => {
    return gulp.src('public/assets/img/*')
        .pipe(imagemin({
            progressive: true,
            optimizationLevel: 4,
            svgoPlugins: [{removeViewBox: false}],
            use: [pngquant(),jpegoptim()] // <-- with jpegtran() there is no problem
        }))
        .pipe(gulp.dest('public/assets/img'));
});

This is the error:

internal/child_process.js:274
  var err = this._handle.spawn(options);
                         ^

TypeError: Bad argument
    at TypeError (native)
    at ChildProcess.spawn (internal/child_process.js:274:26)
    at exports.spawn (child_process.js:339:9)
    at Through2._transform (/usr/local/lib/node_modules/imagemin-jpegoptim/index.js:51:12)
    at Through2.Transform._read (/usr/local/lib/node_modules/imagemin-jpegoptim/node_modules/through2/node_modules/readable-stream/lib/_stream_transform.js:184:10)
    at Through2.Transform._write (/usr/local/lib/node_modules/imagemin-jpegoptim/node_modules/through2/node_modules/readable-stream/lib/_stream_transform.js:172:12)
    at doWrite (/usr/local/lib/node_modules/imagemin-jpegoptim/node_modules/through2/node_modules/readable-stream/lib/_stream_writable.js:237:10)
    at writeOrBuffer (/usr/local/lib/node_modules/imagemin-jpegoptim/node_modules/through2/node_modules/readable-stream/lib/_stream_writable.js:227:5)
    at Through2.Writable.write (/usr/local/lib/node_modules/imagemin-jpegoptim/node_modules/through2/node_modules/readable-stream/lib/_stream_writable.js:194:11)
    at Through2.ondata (/usr/local/lib/node_modules/imagemin-pngquant/node_modules/readable-stream/lib/_stream_readable.js:581:20)

Am I doing something wrong, or is it a bug?
Thank you very much.

Can you show us the result of:
node --version
npm --version
npm ls imagemin imagemin-jpegoptim

And what is your OS?

Never mind. Fixed in 4.1.0. https://github.com/imagemin/imagemin-jpegoptim/commit/4781cbee8d1b1e0af6d0d4853348da14260aa0fd

Thank you very much @shinnn for your fast reply.
I have updated the package but now I get another problem: the optimization works, but it reduces the images at 100%, so I get 0kb images as a result.
Should I change something on the code? I have tried with the max and size options but I get the same result.

If you want, I can open a new Issue with this problem.
Thank you very much

If you want, I can open a new Issue with this problem.

No thanks. Create a small repository by which we can reproduce the problem instead.


install docker on ubuntu

user needs to be added to the docker group:
sudo usermod -aG docker <yousername>

Yep. Gonna close this as this is part of the official Post-installation steps for Linux documentation :)


TypeError: Cannot read property 'querySelector' of undefined

I have the ionic content banner in a side menu content view. And it throw error:
TypeError: Cannot read property 'querySelector' of undefined
It regard to this line of code: getActiveView(body).querySelector('.scroll-content').appendChild(element[0]); line 149
What is going wrong? Could you fix that?

Thanks @hughred22 !! I'll try to check this out soon and if everything is good merge it in

Awesome!

Also encountered this.

FYI: This can also happen when you try to show a banner before the active view (ion-view[nav-view="active"]) is in the DOM.

@davej What do you mean "before" the active view?  The banner should show at the active view in order to be seen.

I have $ionicContentBanner.show() in a controller but ion-view[nav-view="active"] isn't created yet when $ionicContentBanner.show() is called and I got the error described above. The error goes away if I put the call inside of a setTimeout.

Instead of setTimeout, put your call in timeout() command. You need to wait till the ion-view is created before calling show(). timeout() with fix the digest circle for you and make sure show() is called last.

Yeah I understand the issue. I fire the function call on the $viewContentLoaded event now so it's not an issue anymore. Perhaps the plugin could wait for the loaded event on the view too?

I have the same issue and even a timeout is not working for me.
Any help?

+1

The same error. $timeout/setTimeout does not help. Any news on this issue @djett41 ?

Also for non-scrollable content areas:
getActiveView(body).querySelector('.scroll-content-false').appendChild(element[0]);
getActiveView(body).querySelector('.scroll-content').appendChild(element[0]);

$timeout( ..., 200) helped me.

I had to set timeout to be 1000 to fix the problem. It's better to wait until your view becomes fully active to call show();


xlights: 2024.11 -> 2024.12

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/smeighan/xLights/releases.
meta.description for xlights is: xLights is a sequencer for Lights. xLights has usb and E1.31 drivers. You can create sequences in this object oriented program. You can create playlists, schedule them, test your hardware, convert between different sequencers
meta.homepage for xlights is: https://xlights.org
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2024.12 in filename of file in /nix/store/97jbslxp6xcgpbqw61wrl2wri41ryfx0-xlights-2024.12

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

xlights

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/97jbslxp6xcgpbqw61wrl2wri41ryfx0-xlights-2024.12 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A xlights https://github.com/r-ryantm/nixpkgs/archive/2ea3d91d462a43516cec888bd8b2d2e09b8d07ec.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/2ea3d91d462a43516cec888bd8b2d2e09b8d07ec#xlights

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/97jbslxp6xcgpbqw61wrl2wri41ryfx0-xlights-2024.12
ls -la /nix/store/97jbslxp6xcgpbqw61wrl2wri41ryfx0-xlights-2024.12/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

xlights

Maintainer pings
cc @kashw2 for testing.

@NixOS/nixpkgs-merge-bot merge


Don't install openjdk

The source image already includes a version of openjdk-8. Removing this command as it fails due to the package not being available from the software repos.

Hi Geoff,
I believe this is an alternative solution to the problem @tbouron was
fixing.
Applying both would result in openjdk not being available.
Paul
On Thu, Feb 20, 2020 at 10:15 AM Geoff Macartney<EMAIL_ADDRESS>wrote:

@geomacy commented on this pull request.
In Dockerfile
https://github.com/apache/brooklyn/pull/44#discussion_r381904448:

@@ -19,9 +19,6 @@

and PhantomJS
FROM maven:3.6.3-jdk-8
-# Install the non-headless JRE as some tests requires them
your branch is a couple of commits behind master, how will this interact
with 667083e
https://github.com/apache/brooklyn/commit/667083e648d52c0aba7f4da600ff17e5510d9b1b
by @tbouron https://github.com/tbouron?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/apache/brooklyn/pull/44?email_source=notifications&email_token=AAIYGZM6GNZSNXLPTMSGYM3RDZJ27A5CNFSM4KXWZ3UKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCWIFKQY#pullrequestreview-361780547,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAIYGZLFV447R5U6ZHUZNH3RDZJ27ANCNFSM4KXWZ3UA
.

--
Paul [W] Campbell

Hi Geoff, I believe this is an alternative solution to the problem @tbouron was fixing. Applying both would result in openjdk not being available. Paul
…
On Thu, Feb 20, 2020 at 10:15 AM Geoff Macartney @.> wrote: @.* commented on this pull request. ------------------------------ In Dockerfile <#44 (comment)>: > @@ -19,9 +19,6 @@ # and PhantomJS FROM maven:3.6.3-jdk-8 -# Install the non-headless JRE as some tests requires them your branch is a couple of commits behind master, how will this interact with 667083e <667083e> by @tbouron https://github.com/tbouron? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#44?email_source=notifications&email_token=AAIYGZM6GNZSNXLPTMSGYM3RDZJ27A5CNFSM4KXWZ3UKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCWIFKQY#pullrequestreview-361780547>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAIYGZLFV447R5U6ZHUZNH3RDZJ27ANCNFSM4KXWZ3UA .
-- Paul [W] Campbell

So is this PR needed given that @tbouron's is already there?

@geomacy I think we want to be going forward, and using the updated maven 3.6.3 image, which this PR allows. The PR to roll-back to maven 3.5.4 by @tbouron was added to enable the Jenkins builds to start working again. Now we know what the problem was, this PR removes the need for that version roll-back.

sounds good to me, merging.


MoreMenu.js: hides core Menu if empty

Hides div#p-cactions which used to contain “Move” link and which is now empty.
Fixes #6

Unfortunately it's not this simple. There's no guarantee that some other script will initialize after MoreMenu and try to add a link, for example.
Instead, we use a MutationObserverer to automatically show/hide the More menu based on whether it's empty. This should be working now.
In addition, MoreMenu tries to intelligently learn if any other scripts change the More menu, and based on that it will decide whether it should leave the More menu unhidden. For that logic, see https://github.com/wikimedia-gadgets/MoreMenu/blob/master/src/MoreMenu.js#L878-L902
That said, are you still seeing the empty More menu, @pols12 ?

Unfortunately it's not this simple.

😢

That said, are you still seeing the empty More menu, @pols12 ?

Yes, I do. However, I used to enable PurgeTab gadget which added a “Purge” link in More menu. So observeCactionsMenu() may have populated mmNativeMenuUsage local storage key. I just removed it (it was valued at 5) and now More menu is correctly hidden (thanks for pointing this!).
Do you think it would interesting/feasible  to make this counter automatically decrease with time?

Do you think it would interesting/feasible to make this counter automatically decrease with time?

Yes, I think that makes sense. Maybe it should have an expiry, say a week or so. I'll try to look into this soon, unless you care to rework this PR :) Thanks for your feedback either way!

Hello MusikAnimal,
I have reworked the code. Do you have any time for reviewing it? (sorry for bad commit history)

Dang, I'm so sorry it took so long to get back to this! I'm now cleaning up all the PRs and realized this one has been around for years. it's just hard to test.
Also, things have changed... now we have a setExpires() via mw.storage, so there's no need to use an object. We'll just set an expiry and when it expires, it's as if there was no localStorage item to begin with, effectively doing what we want.
I'll rework the code to use this new approach, but in the spirit of collaboration, I'm going to merge your PR as-is so you at least get a spot in the git history :)
Thanks for contributing, and apologies again for never getting back to you!


c3 js chart text dubbling issue

Hi,
I am trying to convert c3js chart to png using html2canvas. But i am getting all the chart text doubling.

This guy found a fix, but you are going to have to modify the source code: https://github.com/niklasvh/html2canvas/issues/734#issuecomment-158303134

Yes, you can modify the code. Or just run build task and find the result from dist dir. I think it's easier.


bug: [Rider] Cody is disabled on this file, by an admin.  This is an enterprise feature but I am a Pro User (rider-cody-nightly-6.0.12)

IDE Information
JetBrains Rider 2024.1.4
Build #RD-241.18034.76, built on June 24, 2024
Runtime version: 17.0.11+1-b1207.24 amd64
VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o.
Windows 11.0
.NET Core v8.0.1 x64 (Server GC)
GC: G1 Young Generation, G1 Old Generation
Memory: 2048M
Cores: 32
Registry:
ide.experimental.ui=true
ide.mac.bigsur.window.with.tabs.enabled=false
ide.new.project.model.index.case.sensitivity=true
Bug Description
Cody is warning with
"Cody is disabled on this file"
This feature has been restricted by an Admin.

First occurrence of this was seen after installing Cody AI 6.0.12-nightly
I have a Pro subscription, Not enterprise. Therefore there can be no enterprise policies/restrictions in place.
This warning seems incorrect

Additional context
No response

I'm getting those as well in Idea. Disabled the notification for whole project, it was too annoying.

thanks for reporting this, we'll take a look!
@jafin does Cody work at all or is it just the notifications that pop up?

In my case (Idea) Cody works just fine, I do not see any difference. It is just the popup.

thanks @Nopik! does it only happen on start up and eventually goes away?

It was happening for me for an hour or so, randomly - but usually when I opened new file. It was annoying enough and I eventually disabled that notification completely, not sure if it would go away after a longer time.

@kalanchan Mostly at startup from my observation.

Same on PyCharm 2024.1.4 with Cody 6.0.14. Disabled it after a few minutes of the startup because of too many notifications, so I don't have any long term observations.

Same for me, on every file I open. Maybe it's because I'm opening files from a private repo of an organization?

@cbrwizard The PR I linked should fix this for people who see it only occasionally - if this is happening on every file, it's likely a different issue.
Are you an enterprise user? Has this been happening to you consistently even after restart?

@cbrwizard Okay gotcha.
To confirm: This is happening on EVERY file you open, even after restarting extensions?
If you can provide console logs, more info about what 'private repo' refers to, and if you've modified any settings that would be useful.

should be fixed by above please re-file if it's still a problem

This has indeed been fixed for me, thank you!


Add trace and multi cq implementations/tests for WH Resnet

Ticket
https://github.com/tenstorrent/tt-metal/issues/10020
Problem description
Adding trace and multi cq versions of Resnet for WH
What's changed
Refactored out logic for padding/sharding for first conv to allow for preprocessing needed for trace/multi cq
Checklist

[x] Post commit CI passes https://github.com/tenstorrent/tt-metal/actions/runs/9845333911
[ ] Model regression CI testing passes (if applicable)
[ ] New/Existing tests provide coverage for changes

@tt-aho Will these tests also checks functional correctness?

@tt-aho Will these tests also checks functional correctness?

The perf tests (E2E + device) were not made to test correctness, but there are correctness tests for each of the variants in models/experimental/resnet/tests/test_ttnn_resnet50_performant.py which I added to nightly ci (device perf technically does do correctness tests since it's a script wrapping this test).

@tt-aho Will these tests also checks functional correctness?

The perf tests (E2E + device) were not made to test correctness, but there are correctness tests for each of the variants in models/experimental/resnet/tests/test_ttnn_resnet50_performant.py which I added to nightly ci (device perf technically does do correctness tests since it's a script wrapping this test).

Got it, Thanks. I was adding the resnet test for new conv api in model test (tests/ttnn/integration_tests/resnet/test_ttnn_functional_resnet50_new.py). Is it already covered under nightly CI?


Plans for a Triton implementation?

Thank you guys for open sourcing this amazing work, I was curious though if there are any plans for a Triton implementation for a higher level implementation. I would like to experiment with this project in tandem with a library I have been working on to accelerate diffusion models but I am not entirely familiar with CUDA yet.
Looking forward to your response 🙂

We may try if we have free time, but it's not in the pipeline at the moment. We welcome community contributions!
What changes are you interested in making that Triton would be helpful for?

Mainly looking for an implementation I can easily play around with, hopefully stuff like bias and activation fusion, extension to 2D, etc. Is there a reference pytorch implementation anywhere I can look at?


Added deep links

Added the deep links key to the WebResponse object.
This could be a very useful feature if the library is used for spidering.

The line:
packaged_results = [WebResult(single_result_json) for single_result_json in json_results.get("webPages", {}).get("value", [])]
in py_ms_cognitive_web_search.py replaced a similar line that had json_results["webPages"]["value"]
While this was correct most of the time, it caused an unhanded error when encountering empty results.

Right, those changes make sense!
Thanks for accepting the pull request. Hope it helps!
Cheers!

Np, thanks for contributing 👍


Make public/plugins JSON

This is a file that shall not contain code and therefore does not have to be a JavaScript file.

Comes with open-scd-core


Use only "string" reference

It is weird that Nexus provides two ways for the same task. Sooner or later schema has to be split to files, and then non-string reference does not make sense.
export const SignInErrors = objectType({
  name: 'SignInErrors', // Allow only string here.

Does it make sense?

The main idea here was that by allowing object references it provides a better experience for IDEs that support features like jump-to-definition, unfortunately I haven't been able to get that to work a similar way with just strings.
So really it's up to however you want to use it I suppose, if you want to only use strings you're more than welcome to do it that way.

I see. Thank you for the answer, that makes sense.


training process freeze suddenly

Hi dear
Thank you for your project.
when I run train.py script, it start for training and prints these lines then go to pause and don't print anything.
`
1/Unknown - 13s 13s/step - loss: 3.1658 - acc: 0.0616 - mIoU: 0.01282020-11-04 11:56:42.866834: I tensorflow/core/platform/default/device_tracer.cc:641] Collecting 1278 kernel records, 234 memcpy records.
  2/Unknown - 13s 7s/step - loss: 3.1030 - acc: 0.0730 - mIoU: 0.0139
  3/Unknown - 14s 5s/step - loss: 3.0520 - acc: 0.0800 - mIoU: 0.0149
  4/Unknown - 14s 4s/step - loss: 3.0151 - acc: 0.0868 - mIoU: 0.0158
  5/Unknown - 15s 3s/step - loss: 2.9743 - acc: 0.0991 - mIoU: 0.0177
  6/Unknown - 15s 2s/step - loss: 2.9408 - acc: 0.1135 - mIoU: 0.0196
  7/Unknown - 15s 2s/step - loss: 2.9064 - acc: 0.1288 - mIoU: 0.0217
  8/Unknown - 16s 2s/step - loss: 2.8769 - acc: 0.1451 - mIoU: 0.0240
  9/Unknown - 16s 2s/step - loss: 2.8478 - acc: 0.1617 - mIoU: 0.0262
 10/Unknown - 16s 2s/step - loss: 2.8215 - acc: 0.1782 - mIoU: 0.0285
 11/Unknown - 17s 2s/step - loss: 2.7944 - acc: 0.1972 - mIoU: 0.0311
 12/Unknown - 17s 1s/step - loss: 2.7659 - acc: 0.2168 - mIoU: 0.0337
 13/Unknown - 18s 1s/step - loss: 2.7429 - acc: 0.2321 - mIoU: 0.0358
 14/Unknown - 18s 1s/step - loss: 2.7188 - acc: 0.2492 - mIoU: 0.0382
 15/Unknown - 18s 1s/step - loss: 2.6954 - acc: 0.2673 - mIoU: 0.0409
 16/Unknown - 19s 1s/step - loss: 2.6758 - acc: 0.2847 - mIoU: 0.0432
 17/Unknown - 19s 1s/step - loss: 2.6534 - acc: 0.3008 - mIoU: 0.0455
 18/Unknown - 19s 1s/step - loss: 2.6323 - acc: 0.3146 - mIoU: 0.0474
 19/Unknown - 20s 1s/step - loss: 2.6122 - acc: 0.3281 - mIoU: 0.0494
 20/Unknown - 20s 1s/step - loss: 2.5933 - acc: 0.3395 - mIoU: 0.0512
 21/Unknown - 21s 978ms/step - loss: 2.5727 - acc: 0.3521 - mIoU: 0.0529
 22/Unknown - 21s 951ms/step - loss: 2.5521 - acc: 0.3646 - mIoU: 0.0546
 23/Unknown - 21s 925ms/step - loss: 2.5342 - acc: 0.3750 - mIoU: 0.0561
 24/Unknown - 22s 902ms/step - loss: 2.5147 - acc: 0.3862 - mIoU: 0.0577
 25/Unknown - 22s 880ms/step - loss: 2.4969 - acc: 0.3958 - mIoU: 0.0589
 26/Unknown - 22s 861ms/step - loss: 2.4781 - acc: 0.4054 - mIoU: 0.0602
 27/Unknown - 23s 843ms/step - loss: 2.4590 - acc: 0.4152 - mIoU: 0.0617
 28/Unknown - 23s 826ms/step - loss: 2.4409 - acc: 0.4242 - mIoU: 0.0630
 29/Unknown - 23s 810ms/step - loss: 2.4240 - acc: 0.4320 - mIoU: 0.0640`

Hi,
With the GPU: GeForce GTX 1080 (Driver Version: 440.100, CUDA Version: 10.2), I am able to train completely for 50 epochs.
If you have adopted multi-worker training with keras, it might be this issue. So remove it and try doing.
Else while creating batches of data in preprocessing/augment_data/get_data() set num_parallel_calls = None and try training.
Hope this helps.

thanks. I dont use multi-worker.
also I set num_parallel_calls = None but it doesnt help.


Fix typo in nf-winsock2-bind.md

As the title said, fixed a typo in "bind" function documentation, in the error codes description section.

Thanks!


chaining: source security ID in hairpin might lead to policy denies in >=v1.7.4

Bisecting blame 60b42109c43f9e34872263fea93d4c97f735b5c9
xx drop (Policy denied) flow 0x53352ab to endpoint 694, identity 4417->56410: <IP_ADDRESS>:53662 -> <IP_ADDRESS>:80 tcp SYN

The security ID in the endpoint is different from the one being generated by the datapath:
ENDPOINT   POLICY (ingress)   POLICY (egress)   IDENTITY   LABELS (source:key[=value])                                       IPv6   IPv4             STATUS
           ENFORCEMENT        ENFORCEMENT
366        Disabled           Disabled          4428       k8s:class=tiefighter                                                     <IP_ADDRESS>   ready
                                                           k8s:io.cilium.k8s.policy.cluster=default
                                                           k8s:io.cilium.k8s.policy.serviceaccount=default
                                                           k8s:io.kubernetes.pod.namespace=default
                                                           k8s:org=empire

Is this a potential collision in the mark value?

It looks like some bits changed:
4417<PHONE_NUMBER>001
4428<PHONE_NUMBER>100

@aanm How did you get into this state?

In that case yes, but it is more common to be off by 1

This could be a --set 1/0xF or similar where 4 bits get cleared and one bit gets set.

To close out discussion, this was caused by a conflicting mark value. So added a PR to opt-out of setting identity field in the mark. This will force the ingress path to do another lookup and obfuscate the trace logs some, but will allow policy to work correctly.

[Only thing left here is a doc follow-up to update the chaining guide.]

Fixed via #12185 and #12194.


feat: add upgrade handler 2.2.0

Description
Closes: #XXXX

Author Checklist
All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.
I have...

[ ] included the correct type prefix in the PR title
[ ] added ! to the type prefix if API or client breaking change
[ ] targeted the correct branch (see PR Targeting)
[ ] provided a link to the relevant issue or specification
[ ] followed the guidelines for building modules
[ ] included the necessary unit and integration tests
[ ] added a changelog entry to CHANGELOG.md
[ ] included comments for documenting Go code
[ ] updated the relevant documentation or specification
[ ] reviewed "Files changed" and left comments if necessary
[ ] confirmed all CI checks have passed

Reviewers Checklist
All items are required. Please add a note if the item is not applicable and please add
your handle next to the items reviewed if you only reviewed selected items.
I have...

[ ] confirmed the correct type prefix in the PR title
[ ] confirmed ! in the type prefix if API or client breaking change
[ ] confirmed all author checklist items have been addressed
[ ] reviewed state machine logic
[ ] reviewed API design and naming
[ ] reviewed documentation is accurate
[ ] reviewed tests and test coverage
[ ] manually tested (if applicable)

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@9e4acae). Click here to learn what that means.
The diff coverage is n/a.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #127   +/-   ##
=======================================
  Coverage        ?   54.91%
=======================================
  Files           ?       24
  Lines           ?     2278
  Branches        ?        0
=======================================
  Hits            ?     1251
  Misses          ?      952
  Partials        ?       75


Add different buttons for inserting and manipulating text in document area

add buttons for inserting paragraph in textarea, button for finding and replacing text , to apply style, to change font, to apply custom styles, to find and replace all similar texts,
This will help in formatting the templates that we will import in future from AP template.
Also updated it with changes suggested.

Agree. First we need a plan/design.


[BUG] Containers do not start when applying a docker-compose project

Describe the bug
When a docker-compose.yml file is imported, exo creates containers for each of the services as well as (at least one) network to attach the containers to.  If the containers end up getting created before the network is available, they will not start automatically. Similarly, when destroying a project, we try to destroy all components in parallel, and destroying the network fails because there are still containers attached to it.
Components should have some sort of dependency ordering that should be respected when starting/stopping/removing everything in a workspace.

See also #171


Add docs about configuration structure

Q
A

Is bugfix?
❌

New feature?
❌

Breaks BC?
❌

Fixed issues
#11

Do I need to add documentation about the config package?

Yes if you think it is necessary.

Yes if you think it is necessary.

Сheck

There is docs for application confuguration (config groups, environments) in Yii Config Docs, DI container configuration in Yii DI and Yii Definitions.
Suggest instead this long documentation add short note with links to other packages docs.

@DAGpro do you have time to apply @vjik suggestion?


django login screen

I thought this could be a useful feature, if we have a nice login page for django example app.
common credentials are
User: chatterbot
Pass: chatterbot

@gunthercox can i submit PR for this request?

My original intention for the example django app was to keep it as simple as possible so that new developers could easily see the changes they need to make to add ChatterBot to their Django project. I'm not opposed to the idea of making additions, but what would the benefit of a login page be?

Usually the web services are provided using login credentials only. That's the reason i am looking into

Since ChatterBot is now supporting concurrent chat sessions, I can see the ability to log into the Django example being useful. I think a good use case might be that someone wants to test out the chat bot having two different conversations. They could make two user accounts and sign in on different browsers.

Agreed with you


[v1.x] Use ArmPL as BLAS for MXNet AArch64 wheels

Description
(Brief description on what this PR is about)
Checklist
Essentials

[ ] PR's title starts with a category (e.g. [BUGFIX], [MODEL], [TUTORIAL], [FEATURE], [DOC], etc)
[ ] Changes are complete (i.e. I finished coding on this PR)
[ ] All changes have test coverage
[ ] Code is well-documented

Changes

[ ] Feature1, tests, (and when applicable, API doc)
[ ] Feature2, tests, (and when applicable, API doc)

Comments

If this change is a backward incompatible change, why must this change be made.
Interesting edge cases to note here

@mxnet-bot run ci [unix-cpu, unix-gpu]

@mxnet-bot run ci [unix-cpu, unix-gpu]


Exclamation point (!) in block comment unapplies hydrated CSS

Reproduction
This repo demonstrates the issue:
https://github.com/exogen/styled-components-comment-hydration-bug
Steps to reproduce
See above repo.
When a styled component is created with CSS like the following:
const Styled = styled.div`
  /*! hello */
  .selector { color: red; }
`;

…then the style will seemingly be applied correctly during SSR, but upon hydration it will be unapplied.
The descendant selector makes a difference.
Additional details:

Removing the ! will make it start to work.
If SSR is skipped so that it only renders client-side, it works fine.
It seems similar to this issue: https://github.com/styled-components/styled-components/issues/888, except I tried both minify: true and minify: false and it's broken both ways.
Broken in both production and development mode.

Expected Behavior
Valid CSS syntax should not cause issues.
I noticed Stylis had a release some time ago with “support legal block comments /*! */” in it.
That makes me suspect it's not a Stylis parsing issue but rather something to do with SC's hydration.
Actual Behavior
The style is unapplied when hydrated.
Environment
System:

OS: macOS 10.14.5
CPU: (8) x64 Intel(R) Core(TM) i7-8559U CPU @ 2.70GHz
Memory: 326.80 MB / 16.00 GB
Shell: 3.2.57 - /bin/bash

Binaries:

Node: 8.9.4 - ~/.nvm/versions/node/v8.9.4/bin/node
Yarn: 1.16.0 - /usr/local/bin/yarn
npm: 6.9.0 - ~/.nvm/versions/node/v8.9.4/bin/npm

npmPackages:

babel-plugin-styled-components: ^1.10.0 => 1.10.0
styled-components: ^4.2.1 => 4.2.1

So, stylis.js definitely has a test in place for this:

Will need to investigate further where this is happening. Thanks for submitting a full repro 🙇

@exogen can you try with styled-components@canary-sheet? We reworked the core stylesheet infrastructure for v5, I would love to know if that fixes it.

It seems fixed with the alpha when I pulled down the repo and upgraded. Super weird issue.


reboot at wifi

My T display S3 has started to develop a bug. As soon as it connect to wifi, it reboots. I have reuploaded my sketch with no improvement.
I have used the flash tool and the firmware available on the github and same behaviour, I have the Lilygo logo, then scan wifi and it reboots.
Same behaviour on various USB cable and power source...
I suppose it is a harware issue?
Thanks for your support!
Eric

If it restarts when connected to wifi, please replace the data cable and power supply for testing. Test it under several power supplies. If the same result is obtained, it can be determined that there is a problem with the PCB power supply.

Ok, so it is a defective PCB. It has been tested with different cables, power sources and even with external battery through the battery port.
Is there any warranty on this item? It is less than 18 months old.

I don't know if there is a warranty, you need to ask where you bought it


Add Boise State logo and changes to About page

Add Boise State logo to the left of the fire emoji and 2F+
Make changes requested by Moji in the first section of this document.

Added BSU logo to the navbar and made it a link to Boise State's website.
{t:15}

Added new "brief" content. Might need to go redo some formatting later.
{t:30}

Added the rest of the requested about page.
{t:30}


[release-1.9] set MaxEjectionTime to BaseEjectionTime if it is greater than 300s

Manual cherrypick required.
#30602 failed to apply on top of branch "release-1.9":
Applying: set MaxEjectionTime to BaseEjectionTime if it is greater than 300s
Using index info to reconstruct a base tree...
M	pilot/pkg/networking/core/v1alpha3/cluster.go
M	tests/integration/pilot/locality_test.go
Falling back to patching base and 3-way merge...
Auto-merging tests/integration/pilot/locality_test.go
Auto-merging pilot/pkg/networking/core/v1alpha3/cluster.go
CONFLICT (content): Merge conflict in pilot/pkg/networking/core/v1alpha3/cluster.go
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 set MaxEjectionTime to BaseEjectionTime if it is greater than 300s
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

@hzxuzhonghu I think this can be closed?


Bump version. Added chart version for helm_release

Closing this as this was handled in another PR https://github.com/ministryofjustice/cloud-platform-infrastructure/pull/734