Fix and polish artifact dockerfiles (and make them buildable again)

Currently,  apt-get install -y crossbuild-essential-armhf doesn't work on debian jessie because the packages are broken (see https://lists.debian.org/debian-embedded/2018/03/msg00000.html),
which prevents are from rebuilding the artifact docker image.
Steps taken to make the dockerfile buildable again:

move Node and its dependencies to a separate docker image, which builds on top of debian:stretch (Node will likely need to update its artifact building scripts.
other languages (C#, PHP, ruby) continue using the jessie-based image (it's hard to switch all of them to debian stretch, I tried).
while doing this, perform a bit of cleanup as well.

This is needed for some followup work on C# artifact building.

@nicolasnoble  @mehrdada   please give owners approval.

****************************************************************

libgrpc.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

I'm pretty sure we're not actually using those Dockerfiles at all for Node, so those new ones shouldn't be necessary. @nicolasnoble you put together the new Node build process, can you confirm?

[trickle] No significant performance differences

@murgatroid99  ok, I removed the node docker files.

****************************************************************

libgrpc.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

Objective-C binary sizes
*****************STATIC******************
  New size                      Old size
 1,964,802      Total (=)      1,964,802

 No significant differences in binary sizes

***************FRAMEWORKS****************
  New size                      Old size
10,887,470      Total (>)     10,887,469

 No significant differences in binary sizes

[microbenchmarks] No significant performance differences

Objective-C binary sizes
*****************STATIC******************
  New size                      Old size
 1,964,802      Total (=)      1,964,802

 No significant differences in binary sizes

***************FRAMEWORKS****************
  New size                      Old size
10,887,471      Total (>)     10,887,469

 No significant differences in binary sizes

[microbenchmarks] No significant performance differences

The artifact build has migrated to the grpc-node repository, alongside its Dockerfiles.


DE connector - use setName instead of setDisplayName for the software…

… server capability

The setter method was replaced as part as https://github.com/odpi/egeria/pull/5196, when removing some constants classes in DE OMAS. The name of the method was misleading, since DE OMA was actually creating a SoftwareServerCapability and setting the name property for it, and not the display name


Is it possible to rename instances of ViGemBus?

Is your feature request related to a problem? Please describe.
I'm modifying XJoy for better handling of several joycons, but end with a list of same ViGemBus instances:

Describe the solution you'd like
At time ViGemBus client is instanciated, is it possible to set name for it? As I'm able to read joycon data (as colors) I want to create a list like:

red right joycon
blue left joycon
Combined green-lightblue joycons

instead of:

Controller (XBOX 360 for windows)
Controller (XBOX 360 for windows)
Controller (XBOX 360 for windows)

But for shake of simplicity:

xbox 1
xbox 2
xbox 3

Would be just right
Describe alternatives you've considered
Nothing I'm aware of
Additional context
For any third party app using ViGemBus, rename controller so you have a sight of what app creates that would be helpful

I second this if it's possible to set a custom name!

The name "a static text", is created and passed here:
https://github.com/ViGEm/ViGEmBus/blob/master/sys/XusbPdo.cpp#L49
https://github.com/ViGEm/ViGEmBus/blob/master/sys/XusbPdo.cpp#L88
I'm following the device allocation to find how to pass custom text on controller creation PdoCreateDevice, PdoPrepareDevice...

It is possible, although careful with your expectations. These names are "cached" in special registry sections upon device creation and there are some circumstances to be aware of that might be in the way of reliably putting a custom name on virtual devices.

Sure.
Any hint about this would be appreciated, can find the connection between vigem_connect and those lines. I can build/test and provide feedback.

The mentioned sections are not the right spot. I can give you more insight once back at PC.

That would be great, the make a PoC PR and see where we are.

Closed since there is no consistent solution that would not result in lots of wonky work.


Valgrind for the C side

The C side needs to be valground. Write a test case and grind it.

Ran valgrind. Code was solid.


Use of a 'simpler' DOCTYPE

One of the features of inliner is that it uses a 'simpler' DOCTYPE. It is true that it uses a shorter one, but changing the DOCTYPE has no benefits that I know of, and has negative consequences for email development.
The DOCTYPE it forces you to use isn't just a simpler one, as it's also, of course, the DOCTYPE for HTML5 documents. This is the latest DOCTYPE, but in this case there is hardly any gain in using it. In fact, both emailonacid and campaign monitor suggest using the xhtml1 DOCTYPE when it comes to email. But they also leave it open to the developer to choose what they'd like to use, as your choice affects how the email renders in numerous clients.
On the side of web development, why force someone over to a new DOCTYPE if they're not building an HTML5 site? Sure, there's no harm done, but there's also no benefit.
The point I'm trying to make is that switching to a 'simpler' DOCTYPE probably isn't a good idea for this project, given that email developers might be developing around other DOCTYPES, and I can't think of any benefits of this feature.
Suggestion: Make the feature optional, at least. Personally, I wouldn't even have it turned on by default. Just leave the DOCTYPE the way it is.

This is fixed @ 1.0.0.

:trumpet:


Windows installation notes

Per #237 notes about Windows platform support for the install docs.   Assuming this is what was desired will add additional docs to --allowed-modules and update https://github.com/apple/pkl-jvm-examples/pull/9 .  https://github.com/apple/pkl-jvm-examples/pull/9 is mostly not related to Windows, just likely Windows users may run into issues around its usage.
I am not sure what items other than allowed-modules may specifically have path related issues or if there is a better place to discuss Windows paths in more detail.
I have not found a way to sparse-checkout (cone or no-cone) or ignore the two invalid window path areas:

pkl-doc/src/test/files/DocGeneratorTest/output/*
pkl-core/src/test/files/LanguageSnippetTests/input-helper/globtest/*

@mitchcapper had you had a chance to take a look at the comments?


Fix/ndc sdg filter

pivotal task
basecamp thread
Updates dots behaviour on country NDC-SDG's section and tooltip sectors data:

Coloured dots (SDG's selected country has on its NDC commitments) stay big when a filter sector is active only if this sector is under their commitments.
On the tooltip are listed all the sectors under an SDG, but only those who are under a country commitment are underlined.

@simaob, @Bluesmile82 ready to be checked again


Add dependabot

Summary of the Pull Request

References

PR Checklist

[ ] Closes #xxx
[x] CLA signed. If not, go over here and sign the CLA
[ ] Tests added/passed
[ ] Documentation updated. If checked, please file a pull request on our docs repo and link it here: #xxx
[ ] Schema updated.
[ ] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Detailed Description of the Pull Request / Additional comments
I added dependabot for Actions.

Validation Steps Performed

See also:

#4165
#13363

It kinda looks like dependabot was enabled automatically for us - what would this PR change?

@zadjii-msft That's for other packages. This is for GitHub actions.

I'd love to have this if it's going to provide us even better insights! Sorry for the delay.
We currently use Microsoft's "Component Governance" portal, which warns us about unsafe packages detected after a build completes, plus catalogues our entire set of packages for legal purposes. I wonder if there's a solution to this that will both (1) get us Dependabot integration and (2) continue providing security notifications through the MS CG channels...

Fwiw, in theory this will get a pull request suggesting updating check-spelling, it would look something like this:
https://github.com/check-spelling/PowerToys/pull/13
I obviously don't know anything about MS CG.


Makefile fails to maintain template.pot on multi-user system

since Makefile uses template.pot in its process, the parallel use of several
users causes interference on this file(multi-user system).
the actual status of this file could not be maintained to all users if e.g.
makemessages Makefile executed from differnet users.

reproduce the problem:

user A calls makemessages, this changes the template.pot file
when user B calls makemessages too, this leads to have interference on this file (the actual status of this file is lost to some users)

solution:
add the name of the user to template.pot filename so each user has his own file,this keeps the behavior of this file  as it is.
we solved this problem with this commit: https://hg.intevation.de/gemma/rev/dd9d90cae683

One problem is: Because user A created the file, user B cannot write it.


chore: removes unused transition

Removes the transition on App.vue’s router view. The linter for https://eslint.vuejs.org/rules/require-toggle-inside-transition.html started to go off and that made me wonder if we even really use this transition. I’m not sure what effect it actually has on our pages. Does it have any?
Signed-off-by: Philipp Rudloff<EMAIL_ADDRESS>

Also see https://github.com/kumahq/kuma-gui/pull/1687


Workspace swipe on last workspace shaky jitters

When swiping on last workspace, everything looks like it's shaking and stuck there till further input, it's like it's trying to go to non-existent next workspace in infinite loop and going back as there isn't any next workspace...

does it still happen? I'm unable to reproduce.

should be fixed


refactor!: remove 'call' method impl. from Strip classes

Closes #130
BREAKING CHANGES

@rultor merge it.

@rultor merge it.

@rafamizes OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge it.

@rafamizes Done! FYI, the full log is here (took me 2min)


Does the introduction of the "End user protection" policy represent a shift in licensing requirements?

The "End user protection" baseline policy includes functionality that is part of Azure AD Identity Protection:

Requiring MFA for risky sign in events
MFA registration with 14 day skip option

I understand these baseline policies do not require P2 licensing, whereas Identity Protection does. Does this represent a broader shift in licensing requirements for CA policies that evaluate sign in risk? If I have P1 licenses, can I now make use of sign in risk in custom CA policies?
Thanks!

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

ID: ea065ca4-8c74-a1ed-0862-35bcaf6bfb88
Version Independent ID: fd330ca5-ac7c-8824-1381-067f050fa597
Content: Conditional access baseline protection policies - Azure Active Directory
Content Source: articles/active-directory/conditional-access/concept-baseline-protection.md
Service: active-directory
Sub-service: conditional-access
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

@jaffadog  Thank you for your query . We will investigate and update further.

@jaffadog Basically baseline policies are available for everyone to enable 4 baseline level of security without any extra cost.  However, you still need to get P2 licenses for enabling Identity Protection and CA policies require P1 or higher licenses.

@jaffadog We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


Refresh needed

Teams admin is now called "Teams Service Administrator"
Privileged role admin is now called "Privileged Authentication Administrator"
would be good to have role combination examples to highlight potential dependencies (i.E. Teams Admin role + User Admin to provision Users for Teams, etc.)

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

ID: f18d8345-78f3-0b16-f585-1e53f49d138f
Version Independent ID: 015e98be-2db5-11a3-9abc-6f89b44c3f96
Content: About admin roles in the Microsoft 365 admin center - Microsoft 365 admin
Content Source: microsoft-365/admin/add-users/about-admin-roles.md
Service: o365-administration
GitHub Login: @KwekuA
Microsoft Alias: kwekua

Updated the Teams service admin name. Confirming the Privileged auth admin change.

Privileged auth admin is a separate role from the privileged role admin. It's documented here: https://docs.microsoft.com/en-us/azure/active-directory/users-groups-roles/directory-assign-admin-roles. Thanks.


Feature/expand precedence groups

makes PriorityPrecedence lower than AdditionPrecedence to address #22

@ateliercw Reopening against develop and then merging


Non-Native Chat for Cross Cloud Federation

Please include the caveat in this article that cross cloud, for example GCC-High to Commercial is non-native federation. This article states that Teams only to teams only will always be native, but when users are in different clouds, it will use non-native federation.

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

ID: 1ac8f101-bdb5-2b55-d5f4-e070b70768d4
Version Independent ID: 964d836e-d00b-b585-8f02-e03c5ec8dab9
Content: Native chat experience for external (federated) users in Microsoft Teams - Microsoft Teams
Content Source: Teams/native-chat-for-external-users.md
Service: msteams
GitHub Login: @MikePlumleyMSFT
Microsoft Alias: mikeplum

Hi @bradbrill, thank you for your feedback.
@MikePlumleyMSFT, could you help to confirm this information? Between government clouds and commercial, and what about between different government clouds like GCC and Dod? Thanks!

@dariomws Yes, it applies to other clouds. DoD, GCCH, and WW (GCC is considered part of WW)

@bradbrill,
If you have a Microsoft alias, you can link your accounts in GitHub so that a blue "Microsoft submitter" label appears to any PR's or issue you submit. Guidance on it here: https://docs.opensource.microsoft.com/tools/github/index.html
Here is the site you can use to see the mappings between GitHub and Microsoft aliases: https://repos.opensource.microsoft.com/people?q=bradbrill

@dariomws Thank you. I think I am now linked.


Cross colliders without turbo

accelerate in a lage straight part of the road
collide with the rocks for example
sometimes you can cross the wall

This case of funnelling should be fixed. The other ones are still being worked on


exception thrown in constraint

just noticed, error happend during constraint's runtime evalution will crash nools, and no being handle by error callback, supplied through 'session.match'.
use case:
i)   normal constraint - name: Fact name.valid_param == "some value";
-> matched normally
ii)  constraint with undefined/null parameters- name: Fact name. == "some value";
-> matched normally
iii) constraint with undefined property of undefined object: Fact name[undefined parameter][undefined child] == "some value";
-> nools crash....
any idea?.

Yes, you can't use two levels of undefined keys. You can define new facts to represent intermediate data.

we are interested on taking over nools, in case no one else interested on maintaining the project.
how shall we proceed?

Hi @joeel1 I have moved the nools codebase to a new org noolsjs. I am actively working on an update to nools with the first order of business being updating the code base to es6 and to support the latest versions of node. I plan on publishing the new release in a couple of weeks once I have all the documentation updated.
Once I finish the first update I will begin working on getting the outstanding PRs merged then Ill start working on new features. If you are interested in contributing please feel free to issue PRs against this branch.


2.32.1 regression with set_motion_device_intrinsics()

Required Info

Camera Model
T265

Firmware Version
<IP_ADDRESS>6

Operating System & Version
Win 10

Kernel Version (Linux Only)

Platform
PC

SDK Version
2.32.1

Language
C++

Segment
AR

Issue Description
The code below works with 2.31.0 but with the latest it throws: RealSense error calling rs2_set_motion_device_intrinsics(sensor:00000180FA8AB1B0, profile:00000180F8E34828, intrinsics:0000007736F3FD70):     Invalid stream index
rs2::context ctx;
rs2_motion_device_intrinsic read_motion_intrinsics[2] = {};
for (auto dev : ctx.query_devices())
{
    if (auto tm2 = dev.as<rs2::tm2>())
    {
        for (auto sensor : tm2.query_sensors())
        {
            for (auto profile : sensor.get_stream_profiles())
            {
                if (auto mp = profile.as<rs2::motion_stream_profile>())
                {
                    read_motion_intrinsics[mp.stream_type() == RS2_STREAM_GYRO ? 0 : 1] = mp.get_motion_intrinsics();
                }
            }
        }

        for (auto imu : { RS2_STREAM_GYRO, RS2_STREAM_ACCEL })
        {
            tm2.set_motion_device_intrinsics(imu, read_motion_intrinsics[imu - RS2_STREAM_GYRO]);
        }
        tm2.write_calibration();
        std::cout << "Set motion device intrinsics" << std::endl;
    }
}

Thanks.

Hi,
Will you be needing further help with this?
If we don’t hear from you in 7 days, this issue will be closed.
Thanks

Hi,
Thanks for checking.  This isn't something that's currently holding us up so I will close it.  I'll open a new issue if it comes up again.


Create Queue_DataStructure implementation in C++

Please follow the Contributing Guidelines & naming conventions in the guidelines accordingly. 🙂
Following naming guidelines is important to maintain consistency.😊
Please NOTE: In case, same implementation of an algorithm is submitted by someone else and it gets accepted, then your contribution won't be merged ( to avoid duplicates )

I already created a pull request submitting an implementation of queues and stacks in c ++

i would like to help out

Should i take this issue?

Can I take this up? The issue seems to be listed as open.

Sorry this issue has been solved.


Bluez-util no longer used... pinaple is broken

Apparently, bluez got rid of bluez-utils and replaced it with bluetoothcli. I don't know the difference because this is all new to me, but here is the source: https://www.raspberrypi.org/forums/viewtopic.php?f=28&t=133961
The bluetoothcli seems to be part of the bluez package from apt.
I will look and see if there is anything I can change in the script but I have very little programming experience.
Anyway, well here is what happens when I try to run it:
pi@raspberry:~ $ sudo pinaple-agent --pin 1234
Traceback (most recent call last):
  File "/usr/local/bin/pinaple-agent", line 66, in <module>
    adapter_path = manager.DefaultAdapter()
  File "/usr/lib/python2.7/dist-packages/dbus/proxies.py", line 70, in __call__
    return self._proxy_method(*args, **keywords)
  File "/usr/lib/python2.7/dist-packages/dbus/proxies.py", line 145, in __call__
    **keywords)
  File "/usr/lib/python2.7/dist-packages/dbus/connection.py", line 651, in call_blocking
    message, timeout)
dbus.exceptions.DBusException: org.freedesktop.DBus.Error.UnknownMethod: Method "DefaultAdapter" with signature "" on interface "org.bluez.Manager" doesn't exist

I'm having the same issue.

After, I got dbus and gobject imports to work, now I am getting the following error:
pi@raspberrypi:~/pinaple $ sudo pinaple-agent --pin 1234
Traceback (most recent call last):
  File "/usr/local/bin/pinaple-agent", line 66, in <module>
    adapter_path = manager.DefaultAdapter()
  File "/usr/lib/python2.7/dist-packages/dbus/proxies.py", line 70, in __call__
    return self._proxy_method(*args, **keywords)
  File "/usr/lib/python2.7/dist-packages/dbus/proxies.py", line 145, in __call__
    **keywords)
  File "/usr/lib/python2.7/dist-packages/dbus/connection.py", line 651, in call_blocking
    message, timeout)
dbus.exceptions.DBusException: org.freedesktop.DBus.Error.UnknownMethod: Method "DefaultAdapter" with signature "" on interface "org.bluez.Manager" doesn't exist

I am assuming the changes in bluez is too much.


HIVE-27122: Use Caffeine for caching metadata objects in Compactor threads

What changes were proposed in this pull request?

Use Caffeine for caching metadata objects in Compactor threads
Why are the changes needed?

Caffeine is a better library than Guava for caching, hence better to use it.
Does this PR introduce any user-facing change?

No
How was this patch tested?

Existing tests

LGTM +1


feature request: fix rename tab behavior

In case of bug report, please fill this template and remove italic lines
Otherwise, clear all and type :)
Tab Modifier version
Browser: Chrome
Extension version: 0.22.0
Can be found here: chrome://extensions/?id=hcbgadmbdkiilgpifjgcakjehmafcjai
Expected behavior
When I right-click a web page and do Rename Tab, I'd expect:

The text box that comes up should default to containing the current title (regardless if it came from a rule or the default page title)
If I hit cancel, no rule should be created and the page should not reload.
If I hit ok, The rule it creates should have a title that matches the name I selected, rather than "Rule created from right-click (URL)" which is completely unreadable in the rule list and makes no sense (after all, I just typed in the label I wanted for this entity)
The rule it creates should be appended as the new highest priority rule, not lowest. This way I can make low priority rules for an entire website (possibly manually in the rule list) and use "rename tab" to rename more specific pages within that site and get the expected behavior.

Actual behavior
What I see is an empty text box where hitting either OK or Cancel creates a rule with an unreadable name at lowest priority.
With the current behavior I have to rearrange rule priority every time I rename tab to get the desired behavior, which should be the default IMO. If you don't agree, please at least add a setting for where the "rename tab" rules are appended as highest priority.
Thanks.

This could be easily done. Do you still need it?

This could be easily done. Do you still need it?

yes this would be helpful.
However to be honest the more useful fix would be for issue #186, which is very annoying . I believe that issue has no viable workaround (due to the fact that the fix requested by all of #86 #74 #110 for a means to drop unmatched captures was declined).

Perhaps yesterdays Pull Request will have solved this. The title won't be changed if the DOM selctor can't be found. But now that I think about it... you want that the title doesn't change if the matching rule is unsuccessful. Thanks for the reply, and let's see how the new version affect this.


how about replace pickle with dill? or at least make it an option?

Queue object couldn't be pickled by pickle, which would raise:
TypeError: can't pickle _thread.lock objects

luckily, dill is API-compatible with pickle, and it could pickle (almost) anything in python, including Queue
so, could you please add an option for that?

Serializing locks makes no sense. What is your application? This looks more like a programmer error, changing serialization formats wouldn't help.

Serializing locks makes no sense

https://stackoverflow.com/a/25411691/7966259

I know what dill is, but I'm unclear why you need it. Can you describe your motivation?
Serializing objects that are fundamentally unserializable (file handles, open streams, locks…) indicates an error in the application layer, in which case masking the problem at the database library layer by replacing pickle with dill would do more harm than good.

@piskvorky
that stackoverflow answer is not to tell you what dill is, but to show there are some people who need to pickle those so called "un-serializable" objects, and which is practicable.
didn't see anyone asking for a motivation in that stackoverflow answer. why would you like to know my motivation so much, when you already had that motivation from others?
even without your help, the patch is quite simple:
import dill
import sqlitedict

sqlitedict.dumps = dill.dumps
sqlitedict.loads = dill.loads
sqlitedict.PICKLE_PROTOCOL = dill.HIGHEST_PROTOCOL

no big deal
have a nice day

Because we do not include patches without a clear motivation.
If dill works for you, great – but in case anyone stumbles on this ticket in the future, mind the caveats above.

def sqlitedict_with_dill():
    import dill
    import sqlitedict
    sqlitedict.dumps = dill.dumps
    sqlitedict.loads = dill.loads
    sqlitedict.PICKLE_PROTOCOL = dill.HIGHEST_PROTOCOL
    return sqlitedict

import queue

sqlitedict = sqlitedict_with_dill()
q = queue.Queue()
q.put(1)
with sqlitedict.SqliteDict('tmp.db', autocommit=True) as sd:
    sd['q'] = q
with sqlitedict.SqliteDict('tmp.db', autocommit=True) as sd:
    print(sd['q'].get() == 1)


Update abaplint.jsonc

@oblomov-dev merge this, and the rest I think should be fixed via PRs to https://github.com/abaplint/deps

thank you


Add horizontal scalability (sharding)

Add multiplexer for Neuro.rest - a function takes a model and determines which rest implementation should be used for that model.

get rest impl given a model
get all rest impl
get rest impl for query (default is query all)

Additional tests will be done in another issue:

When a shard can't be determined
When a shard fails
When a query is executed


Removes accidental zipkin lib dependency and fixes throwable catching

In #1008, we only fixed one place where semi-implementation of JMS could
raise unexpected throwables. This fixes the rest and also removes the
accidental compile dep on zipkin2.

landing this prior to the osgi thing


Created new .NET 6 Function with identity-based "Function host storage" gets Microsoft.AspNetCore.Connections.ConnectionAbortedException

I'm in the process of creating multiple Azure Functions which only use identity-based connections.
While doing so, I also want to use the Function Host Storage (preview) feature.
I'm also using the RUN_FROM_PACKAGE to a storage account, also identity-based.
However, this doesn't appear to work for me at the moment. The Functions don't
The Diagnostics blade tells me the following:

Timestamp : 2/8/2022 8:16:35 AM
Inner Exception Type: Microsoft.AspNetCore.Connections.ConnectionAbortedException
Total Occurrences: 1
Latest Exception Message: Exception encountered while listening to EventStream
The HTTP/2 connection faulted.
Full Exception :

 System.AggregateException : One or more errors occurred. (The request stream was aborted.)
 The request stream was aborted.
 The HTTP/2 connection faulted.
    at System.Threading.Tasks.Task`1.GetResultCore(Boolean waitCompletionNotification)
    at System.Threading.Tasks.Task`1.get_Result()
    at async Microsoft.Azure.WebJobs.Script.Grpc.FunctionRpcService.<>c__DisplayClass4_0.b__1(??) at /src/azure-functions-host/src/WebJobs.Script.Grpc/Server/FunctionRpcService.cs : 46
    at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
    at async Microsoft.Azure.WebJobs.Script.Grpc.FunctionRpcService.EventStream(IAsyncStreamReader`1 requestStream,IServerStreamWriter`1 responseStream,ServerCallContext context) at /src/azure-functions-host/src/WebJobs.Script.Grpc/Server/FunctionRpcService.cs : 93
 (Inner Exception #0) System.IO.IOException : The request stream was aborted.
 Microsoft.AspNetCore.Connections.ConnectionAbortedException : The HTTP/2 connection faulted.    End of inner exception
    at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
    at System.IO.Pipelines.Pipe.GetReadResult(ReadResult& result)
    at System.IO.Pipelines.Pipe.GetReadAsyncResult()
    at System.IO.Pipelines.Pipe.DefaultPipeReader.GetResult(Int16 token)
    at async Microsoft.AspNetCore.Server.Kestrel.Core.Internal.Http2.Http2MessageBody.ReadAsync(CancellationToken cancellationToken)
    at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
    at System.Threading.Tasks.Sources.ManualResetValueTaskSourceCore`1.ThrowForFailedGetResult(Int16 token)
    at System.Threading.Tasks.Sources.ManualResetValueTaskSourceCore`1.GetResult(Int16 token)
    at System.Runtime.CompilerServices.PoolingAsyncValueTaskMethodBuilder`1.StateMachineBox`1.System.Threading.Tasks.Sources.IValueTaskSource.GetResult(Int16 token)
    at async Grpc.AspNetCore.Server.Internal.PipeExtensions.ReadStreamMessageAsync[T](PipeReader input,HttpContextServerCallContext serverCallContext,Func`2 deserializer,CancellationToken cancellationToken)
    at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
    at System.Threading.Tasks.ValueTask`1.get_Result()
    at async Grpc.AspNetCore.Server.Internal.HttpContextStreamReader`1.g__MoveNextAsync|9_0[TRequest](??)<---

What I've done is the following:

Create Function App - Consumption - Linux - with Managed Identity
Running them from a package stored in a Storage Account
Assigning the Blob Data Reader role to the Managed Identity for this Storage Account containing the Functions package
Create Storage Account acting as the WebJobsStorage
Assigning the Blob Data Owner & Queue Data Contributor roles to the Managed Identity for this Storage Account
Also added the Reader role to the Managed Identity for the Resource Group (done manually, not in the Bicep templates)

My solution can be found over here: https://github.com/Jandev/minifier where the build & deploy workflow sets everything up in Azure.
There's also the error stating:
{
2/8/2022 8:09:40 PM
Function app does not contain any active triggers
Function app does not contain any active triggers. Creating a scale down activation task to ensure the app scales down or does not get assigned to a worker.
}

I guess this is due to the fact the Function Runtime isn't able to start.
Now, this could be a configuration/permission issue, but I think all authorization roles are set up correctly.
This is what I'm seeing in the portal.

For AzureWebJobsStorage, you may need the "Storage Account Contributor" role as well, as that has a management permission on the storage account itself, needed to check the general blob service configuration. This happens early in the host startup sequence and could explain the early failure, but I would still expect you to be able to see the functions list. Also, this role need is missing from the docs - update coming. I believe it should only be needed in the context of a blobTrigger definition within the payload, but it might be needed more broadly.
I can also imagine that the app attempts and fails to resolve the package before the role assignment kicks in. I'm not sure what the retry logic is off-hand, and it could be that some change would be needed to the site, possibly even the package URL itself. You might consider having the config object depend on the role assignment, although that returns quickly and doesn't mean that access is granted yet. Alternatively, a user-assigned identity with the role pre-assigned could be used (along with the WEBSITE_RUN_FROM_PACKAGE_BLOB_MI_RESOURCE_ID setting) to ensure that when the app is created it can resolve the package. I expect that if we looked at your blob logs, you'd see unauthorized access attempts. Similarly, there should be a platform log on the App Service side about the package that would indicate a resolution failure, I think. But this kind of race condition for the role assignment would explain the issue.

Thanks!
I've tried your first suggestion just now by adding the Storage Account Contributor role to both of my managed identities to their corresponding Storage Accounts.
I've also turned on the Diagnostic Settings for all Storage Accounts & Azure Functions, to see if there's something useful in them. I'm not seeing any "unauthorized access"-attempts just yet
I am seeing some requests being made to the 3 storage accounts

The ones to the deploy accounts should/could be the Runtime retrieving the package.
The other ones could/should be some IO the Runtime is doing when starting up.
They are using the AccountKey authentication type, which isn't something I'd expect, but the naming might throw me off over here.
I'll try your second suggestion later on today, using a user-managed-identity, as this requires a bit more work in the templates. This also gives AAD some time to sync all roles, which might or might not help.

I have tested the setup using a User Assigned Identity also now, I think.
It was a bit unclear what value the WEBSITE_RUN_FROM_PACKAGE_BLOB_MI_RESOURCE_ID should get. I had expected it to be the Client Id or Object Id from the managed identity, but seeing the docs really state the 'resource id', I've configured the settings like the following (obfuscated some values):
[
  {
    "name": "APPINSIGHTS_INSTRUMENTATIONKEY",
    "value": "d7f76f2d-3a71-45cd-bece-e31d94175950",
    "slotSetting": false
  },
  {
    "name": "AzureWebJobsStorage__blobServiceUri",
    "value": "https://storageaccountforthisazurefunction.blob.core.windows.net",
    "slotSetting": false
  },
  {
    "name": "AzureWebJobsStorage__queueServiceUri",
    "value": "https://storageaccountforthisazurefunction.queue.core.windows.net",
    "slotSetting": false
  },
  {
    "name": "FUNCTIONS_EXTENSION_VERSION",
    "value": "~4",
    "slotSetting": false
  },
  {
    "name": "FUNCTIONS_WORKER_RUNTIME",
    "value": "dotnet",
    "slotSetting": false
  },
  {
    "name": "RUN_FROM_PACKAGE",
    "value": "https://storageaccountcontainingdeployments.blob.core.windows.net/deployments/632b884c-7bb4-41da-88a6-03863b555697/frontend.zip",
    "slotSetting": false
  },
  {
    "name": "WEBSITE_CONTENTAZUREFILECONNECTIONSTRING",
    "value": "DefaultEndpointsProtocol=https;AccountName=storageaccountforthisazurefunction;EndpointSuffix=core.windows.net;AccountKey=theKey",
    "slotSetting": false
  },
  {
    "name": "WEBSITE_CONTENTSHARE",
    "value": "azure-function",
    "slotSetting": false
  },
  {
    "name": "WEBSITE_RUN_FROM_PACKAGE_BLOB_MI_RESOURCE_ID",
    "value": "/subscriptions/54b9020c-1ce9-4470-909f-555a07a1b6c0/resourcegroups/my-resourcegroup/providers/Microsoft.ManagedIdentity/userAssignedIdentities/the-new-user-managed-identity",
    "slotSetting": false
  }
]

I've also removed (turned off) the System assigned identity and added the new User assigned identity to the Function App.

Before doing so, I have added the following roles to this identity:

Reader for the whole subscription
Storage Blob Data Reader for the storageaccountcontainingdeployments Storage Account (which holds the zip-file)
Contributor for the storageaccountforthisazurefunction which is the account configured in the AzureWebJobsStorage settings
Storage Account Contributor for the storageaccountforthisazurefunction
Storage Blob Data Owner for the storageaccountforthisazurefunction
Storage Queue Data Contributor for the storageaccountforthisazurefunction
Storage Table Data Contributor for the storageaccountforthisazurefunction

For the visual effect:

In the settings above I only have configured AzureWebJobsStorage__blobServiceUri and AzureWebJobsStorage__queueServiceUri. I've also tried adding AzureWebJobsStorage with the value https://storageaccountforthisazurefunction.blob.core.windows.net, but this didn't fix anything.
I've also tried using the Client ID and Object ID (Principal ID) for the WEBSITE_RUN_FROM_PACKAGE_BLOB_MI_RESOURCE_ID value, but this didn't help much either.
There is something new in the Log Stream & Live Metrics though:
Headers:
Date: Wed, 09 Feb 2022 20:15:21 GMT
Server: Kestrel
Transfer-Encoding: chunked
Content-Type: application/json; charset=utf-8
. Azure.Identity: Service request failed.
Status: 400 (Bad Request)

Content:
{"statusCode":400,"message":"Unable to load requested managed identity.","correlationId":"9c1f3cd9-8176-48be-a81f-b8c09f8b7926"}

Headers:
Date: Wed, 09 Feb 2022 20:15:21 GMT
Server: Kestrel
Transfer-Encoding: chunked
Content-Type: application/json; charset=utf-8
.
2022-02-09T20:15:27Z   [Verbose]   Host instance '0000000000000000000000005D42BD33' failed to acquire host lock lease: Azure.Identity: ManagedIdentityCredential authentication failed: Service request failed.
Status: 400 (Bad Request)

This might be a misconfiguration for the configuration setting because the MI is right there in the Identity blade.
Any other ideas or pointing out my (obvious?) mistakes is appreciated!
I've heard from a friend he got this working on the Windows plan, so I'll try that also on another day since I'm working on the Linux Consumption plan at the moment.

I've tried it on a Windows Consumption & Dedicated plan.
Same result, but after adding the AZURE_CLIENT_ID setting with the user-managed identity client id, I'm receiving at least some Informational messages on both Windows Function Apps and my Linux Consumption app
2022-02-10T20:18:48Z   [Information]   Host Status: {
  "id": "minifier-prod-weu-app",
  "state": "Running",
  "version": "<IP_ADDRESS>",
  "versionDetails": "4.1.3+3ed9ce8ebeef5b156badaf203a016a56e819a852",
  "platformVersion": "",
  "instanceId": "4A8D2640-637801201576779466",
  "computerName": "",
  "processUptime": 954957,
  "functionAppContentEditingState": "Allowed",
  "extensionBundle": {
    "id": "Microsoft.Azure.Functions.ExtensionBundle",
    "version": "2.8.3"
  }
}

And the Windows dedicated plan also has a nice Verbose message now, which happens about every minute
2022-02-10T20:14:11Z   [Verbose]   Request successfully matched the route with name '' and template 'admin/warmup'

Adding this value sure did something, the runtime still doesn't do anything it appears.

Well, I hardly dare to admit it, but I found the issue and it was in my configuration.
The observant reader might have caught it already.
[
  {
    "name": "APPINSIGHTS_INSTRUMENTATIONKEY",
    "value": "1eb74289-a1d4-4b01-bfd0-50947ed437b6",
    "slotSetting": false
  },
  {
    "name": "AzureWebJobsStorage__blobServiceUri",
    "value": "https://mystorageaccountforthefunctionapp.blob.core.windows.net",
    "slotSetting": false
  },
  {
    "name": "AzureWebJobsStorage__queueServiceUri",
    "value": "https://mystorageaccountforthefunctionapp.queue.core.windows.net",
    "slotSetting": false
  },
  {
    "name": "FUNCTIONS_EXTENSION_VERSION",
    "value": "~4",
    "slotSetting": false
  },
  {
    "name": "FUNCTIONS_WORKER_RUNTIME",
    "value": "dotnet",
    "slotSetting": false
  },
  {
    "name": "WEBSITE_CONTENTAZUREFILECONNECTIONSTRING",
    "value": "theComplateConnectionStringWhichIsNecessaryForSomeReason",
    "slotSetting": false
  },
  {
    "name": "WEBSITE_CONTENTSHARE",
    "value": "azure-function",
    "slotSetting": false
  },
  {
    "name": "WEBSITE_RUN_FROM_PACKAGE",
    "value": "https://mydeploymentstorageaccount.blob.core.windows.net/deployments/thepathtomy/functionapp.zip",
    "slotSetting": false
  }
]

Do you see it?
I had missed the WEBSITE_-prefix in my earlier configuration. After changing the settings to WEBSITE_RUN_FROM_PACKAGE all worked out well!
For completeness, I have set up the following roles on the storage account necessary for the Function App.


Automatically update iricdev-2019 iriclib version

This is done in .github/workflows/build.yml 'Update iriclib version' and 'Commit and push iricdev' steps


Is there a way to have front end app chatting template like Azure Sampleaoai which consumes the endpoint? Need to do it with APP Service.

Hi @rehat22
Thanks for your feedback, can you please elaborate on your ask? What is the Azure Sampleaoai that is referencced in the titile, and how do you like the prompt flow to support it? It would be great if you can provide some code examples for us to learn about your user scenario, thanks.

Azure Sampleaoai refers to this - https://github.com/microsoft/sample-app-aoai-chatGPT
We have deployed using the code from this repo. We basically wanted a front end like this which consumes the endpoint for users.
Basically we need a front end for consumers for our prompt flow end point which will be hosted on Azure APP service

please let me know. Thank you

Your ask is a bit ambiguous and I'm trying to understand your scenario here.
If you want a front ui for a developed flow then we do have an onging work to trigger one like this:

If you want to share the flow to other customers then you can refer to Distribute flow as executable app — Prompt flow documentation
Please let us know if above work satisfy your needs, otherwise please provide more details about how you would like promptflow to support, thanks.

this solves my issue. The link provided is what i was looking for. Stream-lit will help me share the endpoint with customers. Im sure stream-lit can be replaced by AZURE APP Service to create the web app.


Add wsLogger to Javalin (fixes #407)

Supersedes #410 and this pull request only contains non-merged changes.

@Playacem should enableDebugLogging also add a WebSocket logger?

In my opinion it should. I am just a little unsure on what to log.
I would consider the following quite important

the complete matched path
the session id
all parameters where applicable.

The current debug-logger is quite extensive, so I would just log everything you consider relevant. Add it to the LogUtil.

Found a small bug in WsSession: if the queryString is null then ContextUtil throws an exception.

Thanks! I'll merge this now and make a few adjustments.

I can fix it in this pull request but that feels out of the scope.

Agreed. Better to create an issue and fix it in a new PR :)

@Playacem This change is out as part of 2.4.0 now: https://javalin.io/news/2018/11/04/javalin-2.4.0-released.html


SSH timeout doesn't work when hostname cannot be resolved

Even though -o ConnectTimeout=5 is set with ssh, it takes ages to timeout if hostname cannot be resolved.

The problem was on my local machine. I was using /etc/resolver/<company>.corp with nameserver IP address not reachable from internet. This caused the delay.


Delete files not found in trash

VSCode Version: Code 1.14.2 (cb82febafda0c8c199b9201ad274e25d9a76874e, 2017-07-19T23:26:08.116Z)
OS Version: Linux x64 4.10.0-28-generic
Extensions:

Extension
Author (truncated)
Version

vscode-eclipse-keybindings
alp
0.9.6

java
red
0.8.0

Steps to Reproduce:

Delete a directory using r-click ->Delete
Select Move to Trash
Go to Trash, get disappointed

Reproduces without extensions: Yes

AFAIK electron attempts to do this using various libraries on Linux already.

I have the same problem. After my test, I found the deleted file where
<mounted_disk>/.Trash-1000/files/<deleted_file_name>
you can use commandfind / -name "delete_file_name" to find the deleted file.
this ".trash-1000" floder in the mounted disk.
And, every file that is deleted will be classified under the mount disk where the project is located.
@anandchakru


ROX-13263: Refactor tests to reusue code

Description
Code cleanup
Testing Performed
CI

Current dependencies on/for this PR:

master

PR #3616   👈

PR #3615

This comment was auto-generated by Graphite.

/retest


Unc0ver 3.8.0~b1 stuck on step 1/31 for iPhone XR iOS 12.4.1

Describe the bug
When pushing the “Jailbreak” button it gets stuck on step 1/31 and then reboots.

Me too😓

Out of curiosity, can you send a screenshot of what it says before that? I see "failure" at the top.

Assertion failure, whatever that means...

Assertion failure, whatever that means...

It means it’s injecting code into the kernel and it’s failing

There is no reason to keep talking about this issue pwn20 knows about it and it won’t get fixed any faster then it is now

All I did was answer someone’s question. Who are you to tell someone what they should and shouldn’t be talking about? We aren’t bugging anyone to fix it, we’re merely talking amongst ourselves.

Same issue persists with V4.0 of Unc0ver on the iPhone XR
(iOS 12.4.1)

No problem for me today with unc0ver 4.0.0, always on iXR with iOS 12.1.1

I moved to 13.3 and was fine pwn had suggested moving to 13.3 at the time. I suggest trying 4.0.1.


Fix(FullscreenGrab): Fix the issue where TopPanel cannot be displayed immediately after switching between Freeze and Unfreeze and Sub-screen shift move selection doesn't cover full screen

Fix the issue where TopPanel cannot be displayed immediately after switching between Freeze and Unfreeze
Fix Sub-screen shift move selection doesn't cover full screen

because of this change line 599 will need to have a larger check size to see if the click was a single point. I tested using 5px instead of 3px for detecting of it was a small click and that worked.

previous: bool isSmallClick = (regionScaled.Width < 3 || regionScaled.Height < 3);

new: bool isSmallClick = (regionScaled.Width < 5 || regionScaled.Height < 5);

If you want to check it on your end we should be good.

Since the size of regionScaled is calculated by selectBorder through screen scaling, why not directly use the size of selectBorder to determine whether it is a smallclick? m.M11 and m.M22 I looked at it should be the screen scaling right, and the only thing that is obtained is the scaling of the main screen, and this is impossible to determine, may be 100% may be 200% may be 300%, if it is 300% that smallclick judgment is not satisfied which use regionScaled.Width < 5 || regionScaled.Height < 5 (of course, this situation is generally less), so either in the acquisition of the true scaling ratio calculated back to the original size, to determine whether it is the initial size, that in fact, it is equivalent to the size of the direct use of regionScaled to determine whether it is a small Wouldn't clicking be better?
like this:
bool isSmallClick = (selectBorder.Width < 3 || selectBorder.Height < 3);

Of course this all assumes that I understand your smallclick as a left click and no drag, if that's not the definition of a small click on your side of the design, then it's fine!

I like the idea of what you are describing. On thing to consider is that Windows scaling only goes up to 300%, so while calculating the scaling will work, it might be overkill. ☺ Either way I'm open to the idea.


[prototype] shared-elements with detail & grid view

Experimenting with shared-elements (Grid -> Detail) from ex-navigation...
TLDR: It looks cool, but wayy too much work for the amount of animation code that needs to be written.

Closing in favor of a easier to work with router #32


Some builds are showing up with large duration and queue times

There are several builds (mostly those aborted in the queue)  that are being reported as having a large execution and queue time.  These range from 8694 to 40384 minutes which is impossible since the bitrise elite plan has a limit of 90 minutes. Checking the results via  https://api-docs.bitrise.io/#/builds/build-show  shows something similar below:

{
"data": {
"triggered_at": "2021-03-10T09:57:02Z",
"started_on_worker_at": null,
"environment_prepare_finished_at": null,
"finished_at": "2021-03-10T10:02:16Z",
"slug": "13141942932b35cb",
"status": 3,
"status_text": "aborted",
"abort_reason": "User PXXXX requested to abort this build.",
"is_on_hold": false,
....

}

And the data that arrived to grafana has these

Status
Duration
Queued

aborted
8694
8694

Hi Anuerind,
This is the time the build was waiting not running.
To calculate the running duration use started_on_worker_at instead of triggered_at

@DamienBitrise  thanks for confirming. Would that mean that when visualizing the data the aborted ones should not be included?  Shouldnt the Duration field for aborted statuses remain at zero since the Queued column already contains the duration that the build is waiting? Is there a way  to distinguish the entries that were aborted either due to timeout or manually while running?

I saw an outlier as well that was successful but the Duration and Queued values are also very high. The API returns the following:

{
"data": {
"triggered_at": "2021-02-18T11:58:36Z",
"started_on_worker_at": null,
"environment_prepare_finished_at": null,
"finished_at": "2021-02-18T12:16:39Z",
"slug": "b991a9b3fe69a301",
"status": 1,
"status_text": "success",
"abort_reason": null,
"is_on_hold": false,
"triggered_by": "webhook",
"machine_type_id": "elite",
"stack_identifier": "osx-xcode-11.7.x",
...

}
}

and the data that is shown in grafana is this:

Build #
Status
Duration
Queued

11059
success
37372.4
37372.4

I think what caused this outlier is that while it was successful (run time is around 18 minutes), the started_on_worker _at  field remains set to null.

Caveat that I am not a developer (anymore). I was looking at the builds.js logic and it may be that there should be an initial filtering of completed builds (abort, errored, successful) since the logic in the getBuildTableData function is biased in using the started_on_worker_at  field to determine if the build is still active or unserviced.  The first if-check on an "aborted but never started" build wilkl then use the current timestamp to determine the queue time. This will then result to a very large false-positive value if the retrieval was done several hours after the build request was aborted.


Migrate this code to citrus (instead of ctrcommon)

SteveIce10 has already updated and ctrcommon support will eventually be dropped. This code should therefore be migrated to citrus.

Done!


create a palindromic pattern in c++

this is the output of the program i coded in c++ for palindrome patter assign it to me

Assign it to me and i want to work on this issue

Assign me I’ll do it

I want to contribute on this. plz assign it to me.

@1dharode pls assign this project to me pls pls

please assign to me


[BUG] wrong spell in lock implementation flow chart

Describe the bug
文档配图中用词错误,如图标红部分,应该为 取到
To Reproduce

good job，would you like udpate the document and contribute to community?
contribute document: https://github.com/apache/incubator-dolphinscheduler/issues/310

the dev-db branch dosen't have the docs folder. so which branch shall i pr ?  @dailidong

officical document address: https://github.com/apache/incubator-dolphinscheduler-website, please
read README.md first

thx for reply @dailidong .  i have been fixed the issue , and the pr is here :
apache/incubator-dolphinscheduler-website#47


Update super-linter

Contributor Comments
Update the super-linter

[ ] Account for the gitleaks (allow disabling)
[ ] Keep an eye on mypy (this and this)

Pull Request Checklist
Thank you for submitting a contribution to the goat!
In order to streamline the review of your contribution we ask that you review and comply with the below requirements:

[X] Rebase your branch against the latest commit of the target branch
[X] If you are adding a dependency, please explain how it was chosen
[X] If manual testing is needed in order to validate the changes, provide a testing plan and the expected results
[X] If there is an issue associated with your Pull Request, align your PR
title with this documentation

Need to keep an eye on the following issues, which may help to fix CI:

[ ] https://github.com/python/mypy/issues/10768
[ ] https://github.com/python/mypy/issues/10600

An update happened in https://github.com/SeisoLLC/goat/pull/55
Still need to handle the gitleaks config; put that on backlog


Custom component added in container state array not rendering in release mode

Hi Team,
i want to add component one above another in one screen, so i have maintained one array inside parent container. Whenever user click on button  i will add one component inside state array  and once state update it will automatically render call and render next view, but when i add third component in state , the component will add properly but its not showing on screen. this thing happen only in release mode. in debug mode its work properly, i dont understand whats the problem in release mode.
Below some same code i have added
//First Click
const view = this.getView1();
    const array = this.state.viewStack;
    array.push(view);
    this.setState({
      viewStack: array,
    });

//Second  Click
const view = this.getView2();
    const array = this.state.viewStack;
    array.push(view);
    this.setState({
      viewStack: array,
    });

//Third  Click
const view = this.getView3();
    const array = this.state.viewStack;
    array.push(view);
    this.setState({
      viewStack: array
    });

Please let me know if anyone face same issue.

Please use one of the provided issue templates.

Thanks hrams, I have resolved this issue.


[Enhancement] doc tool generate provides referenced by information on schema

Enhancement
That will be helpful to navigate from schema and its usages to get an overview of a schema

Hey @Peefy, I noticed there's a Ref field already specified within KclOpenAPIType. Are we considering using it for displaying the referenced by information on the schema?

I am not sure that the filed has been used. The basic and related issue is https://github.com/kcl-lang/kcl/issues/1215 You can give a try. Thank you @shruti2522 ❤


'syntax error, unexpected '?'' with Laravel/PHP 5

I know version 5 should not be supported, but I think this is a small change that can help legacy apps so as to not break when updated
I get the following error when installing on a fresh Laravel 5 which uses PHP 5 (any subversion)
'syntax error, unexpected '?'' vendor/nicolaslopezj/searchable/src/SearchableTrait.php:380
This is because line 380 of said file is
if ($this->relevanceField ?? false) {
Which uses the Null Coalescing Operator ?? which was introduced in PHP 7. This change was introduced in 1.11
The "correct" way to fix this would be to use a regular ternary operator, or just specify in all composer.json files from version 1.11 to require PHP 7 and above, instead of the current 5.4 which is unsoported. This way a PHP 5 installation will refuse to update from 1.10 and avoid breaking.

you can fork this repo, and then extend the trait to change the interested method, something like this:
<?php

namespace App\Extended;

trait MySQLSearchableTrait
{
    use SearchableTrait;
    protected function getRelevanceField()
    {
        if ($this->relevanceField ? $this->relevanceField : false) {
            return $this->relevanceField;
        }

        // If property $this->relevanceField is not setted, return the default
        return 'relevance';
    }
 }
```
and then use this trait instead of the original one


[#2111, #3259] Move encumbrance calcs & add multipliers & bonuses

Moves the encumbrance calculation logic into AttributesFields to be called by individual data models.
Also adds bonuses and multiplier fields that can be used to modify encumbrance in a more fine-grained level using active effects. There are formulas targeting the three encumbrance thresholds and one that applies to all three thresholds.
Closes #3259

Should add those fields to ActiveEffect5e.FORMULA_FIELDS as well.


Bundle Julia scripts (DiffEqPy.jl) in diffeqpy Python package?

Since current DiffEqPy.jl is very small, why not bundle it in the Python package?  I've done it in #19 and it's not difficult.
This way, change in Julia code directly trigger the tests here.  Also, we don't need to worry about DiffEqPy.jl-vs-diffeqpy API compatibility this way (although currently there is no complex API).

To be a registered Julia package it needs to have the .jl on the end. But won't that clone weird for Python users?

What I mean is to get rid of DiffEqPy.jl Julia package entirely and add Julia scripts to diffeqpy Python package.  So not registering any additional package.  It means to bundle some Julia files in diffeqpy and include those files, instead of using DiffEqPy.

could you mock it? I think I see what you're getting at but the exact details where what got me. If it's not a package, are you just includeing some path? How do you get that path in Python? Etc.


Fix/move example

Status
READY
Description
Remove example for tune upload and create from examples/user
Impacted Areas in Library
Examples

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1545

Relevant Lines:
1616

💛  - Coveralls


Ocean fishing not working with german language

Even after the fixing PRs/commits for fixing ocean fishing, the fishing overlay is not working correctly in the german language
[29.02.2020 09:03:08] Info: fischer: BrowserConsole: failed to look up place: null (Source: file:///E:/FINAL%20FANTASY%20XIV%20-%20A%20Realm%20Reborn/AdditionalExtern/ACT/Plugins/OverlayPlugin_ngld/cactbot/ui/fisher/seabase.js, Line: 242)
[29.02.2020 09:03:08] Info: fischer: BrowserConsole: failed to look up place: {} (Source: file:///E:/FINAL%20FANTASY%20XIV%20-%20A%20Realm%20Reborn/AdditionalExtern/ACT/Plugins/OverlayPlugin_ngld/cactbot/ui/fisher/seabase.js, Line: 242)

Is this still an issue? Sorry for losing track of this.

On my try today it seams that everything works.
Closed.


Enabling upcoming feature ExistentialAny does not work for standard library protocols.

Description

Enabling upcoming feature ExistentialAny does not work for standard library protocols.
For example, it should produce a compile error, considering Decoder is a bare protocol. However, the compiler does not produce any error or warning, even though for other non-standard library protocols it produces errors and warnings correctly.
init(from decoder: Decoder) throws { /* ... */ }

Steps to reproduce

swift repl -enable-upcoming-feature ExistentialAny
put following;

protocol P {}
extension Int: P {}

put following:

let value: P = 42

It produces an error
expression failed to parse:
error: repl.swift:5:12: error: use of protocol 'P' as a type must be written 'any P'
let value: P = 42
           ^
           any P

put following;

let value: Codable = 42

It works!
Expected behavior
Codable should also produce a compile error, but it doesn't.

Environment

Swift compiler version infoswiftc -version
swift-driver version: 1.75.2 Apple Swift version 5.8 (swiftlang-<IP_ADDRESS>.2 clang-14<IP_ADDRESS>.100)
Target: x86_64-apple-macosx13.0

Xcode version info Xcode 14.3
Build version 14E222b

Deployment target: N/A

As I checked, it seems any P check is not working for standard library protocols without associated types including Codable, Decoder, Encoder, CustomStringConvertible, CustomDebugStringConvertible, ExpressibleBy*Literal, and so on.
Forum thread: https://forums.swift.org/t/any-p-check-doesnt-work-for-standard-library-protocols/64276

Thanks!


Feature/updatecore

You did it @dinakarmeda!
Thank you for signing the Anthem Opensource Software CLA.


chore(teku/src/test): variable fix

minor typo in the a variable name in the tests, nothing critical but correct wording is overridden.
have a nice evening

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

Thank you. Good observation. LGTM. :) I see other instances of the wrong overriden in comments in Spec.java , NetworkingSpecConfigDeneb.java and BeaconNodeCommandTest if you wish to fix those as well as part of this PR.

Sure, I'm glad to help. I'm still learning. It should be good now, have a nice day

Thank you. Looks good.


[Feature] Integration with Bug Tracking systems

Hi @Kiswa,
I wrote a feature that I would like to bring in.  It allows users to integrate links to bug tracking tool(s) into the board's items similarly to TortoiseSVN BugTraq. The tools can be configured on the board's setting/edit page.
All the changes to 'dev' are in this commit commit and seem to behave just fine.
I'm not native in Angular (or any web development at all) so I didn't want to just propose pull request without a review of some sort.

The code looked reasonable from a quick read-through. Go ahead and submit the PR and I'll take a closer look.
Thanks for the contribution!

@kiswa I don't want to reopen an issue unnecessarily, but do you have any idea if this feature is working?

@andrewdmaclean It should be functional, yes. In the settings for a board, the last item let's you setup a way for tasks in your board to automatically link elsewhere. The tooltip provides an example.

Ah yes, that definitely makes sense - but I think perhaps the default RegExp for Github isn't working?
I have tried 4 different expressions and I am still not seeing any issues created in the Github Issues for the repository in question.

Ah, yeah, it's not an integration. It's just a convenience for automatically linking to existing issues. @andrewdmaclean


ehhmmm

This code will be erroneous when merged, you cant call .json on a Promise, no such method exists.

And adding on to that, .json is also async too

Why would you break a perfectly working code.


Removed node_modules folder and added .gitignore file

Generated .gitignore via http://gitignore.io configured for Node.js
projects

Closed - this was by design and this change isn't needed.


Crashes when copying scenarios

I tried copying the House of Lynch scenario as you stated we could. It crashed within seconds of clicking the button. Other scenarios and quests even in D2E have the same issue as well.

Creating a new scenarios/quests also crashes. Editing existing scenarios open fine though.

Please attach log.

output_log.txt

Fixed.


forum link

@justinbmeyer do you have a preferred Forum in mind for DoneJS? The Forum link under Help is currently a broken link.

Decided to use another forum after launch (can't remember the name)

discourse


Fix spec error - there is no 'git lfs sync' command

Wow, you found an oldie :)

Super-trivial, but I thought I'd start with something easy :)


setup electron to a frameless window

<!--
  Is this a bug report?
  If so, go back and select the "Bug report" option or your issue WILL be closed.
--!>
phase 1: frameless window
phase 2: no menubar

test and make sure buttons can still be clicked after making this change to the app.
see Electron Documentation for reference on Frameless windows.


Cookbook should use rubygems_url client config for rubysource attribute.

Cookbook version
4.1.12
Chef-client version
13.6
Platform Details
RHEL7
Scenario:
Cookbook should honor honor the rubygems_url in the chef client config file for air gapped environments.
Steps to Reproduce:
Run cookbook in air gapped environment without specifying default['lvm']['rubysource'], and having rubygems_url in the client configuration.
Expected Result:
Client should use client configured rubygems URL.
Actual Result:
Client does not and fails in air gapped environments.

Why don't you just override that attribute and redirect it to your internal gem source?  This is a normal activity as part of running chef.  (proven to work in my company where we are 'air gapped'.

That is what I do, but feel like the client config should be honored.  Why not use what the client is configured for?  If not, why even configure the client?

Indeed, why even set #{chef_etc_dir}/gemrc, /root/.gemrc, /root/.bundle/config and probably one more when we can have a new time-bomb with every new cookbook we consider importing?
This cookbook needs a break, really, as it's created a lot of heart-ache with the rubygems version lock-step, and it'd be great if this wasn't another shot in the foot.  Can we consider having it look for another system-wide value as well?
default['lvm']['rubysource'] ||= node.read('rubysource') || 'https://rubygems.org'

I know, I know, xkcd.com/927 and all.  But if it listens to a centralized value, it may be an improvement on the way to actually using the config in the client.


Update IDEA project files

@devoncarew @pq
Updates project files for IntelliJ and Android Studio that are generated by 'flutter create'. For all three project types:

Open the Project view (not Android, which is default for AS)
Mark special directories
Do not generate any balloons when opening the project
Add the Android facet if needed
Define the project SDK if needed

Forgot to mention that this fixes a couple plugin issues:
https://github.com/flutter/flutter-intellij/issues/1347
https://github.com/flutter/flutter-intellij/issues/1343


Apple Attestation Object Validation Guide

Hello! First of all, great library 🚀
I am checking it using Apple's Attestation Object Validation Guide sample and I get an error "The attestation's nonce is invalid"
val appleAppAttest = AppleAppAttest(
  app = App("0352187391", "com.apple.example_app_attest"),
  appleAppAttestEnvironment = AppleAppAttestEnvironment.PRODUCTION,
)
val attestationValidator = appleAppAttest.createAttestationValidator(
  clock = Clock.fixed(Instant.parse("2024-04-18T00:00:00Z"), ZoneId.systemDefault()),
)
attestationValidator.validate(
  attestationObject = Base64.decode("o2NmbXRvYXBwbGUtYXBwYXR..."), // get the whole value from the guide
  keyIdBase64 = "bSrEhF8TIzIvWSPwvZ0i2+UOBre4ASH84rK15m6emNY=",
  serverChallenge = "test_server_challenge".toByteArray(),
)

I think the problem is in Apple's documentation sample because in step 2:

Create clientDataHash as the SHA256 hash of the one-time challenge your server sends to your app before performing the attestation, and append that hash to the end of the authenticator data

They really do not do the SHA256 of the server challenge but append its plain bytes instead.
I already submitted this feedback to Apple's documentation.
Anybody else encountered this problem? 🙏

We're using this library to support iOS attestation in WARDEN and we have even started to exploit the fact that only the hash is passed. See here, for a description.

Thanks @JesusMcCloud we will take a look at WARDEN

I think it really depends, whether this makes sense for you. Don't get me wrong: We're always happy about new users, because it is an opportunity to gather feedback and improve, but if you are only interested in iOS, that additional layer of indirection may not be worth it. In this case, it's probably easier to just use this library here.
On a related note: We're reworking the iOS attestation format to emulate key attestation and neither back-end nor iOS samples in the WARDEN repo have been updated yet.
What I showed you was a peek into the next release, to stress that there's indeed hashing going on. Here's the corresponding client code including hashing, from our Supreme KMP crypto provider.


Did you mean to create a tag for this latest version 3.0.1?

And if not is there any chance of releasing the current version? Would love to use the new chefspec matchers :)

Sorry for the delay, 3.0.2 is now available on the Supermarket


[Docs] The ckpt and log links on the README seem to be unavaliable

Branch
main branch  https://mmdetection3d.readthedocs.io/en/latest/
📚 The doc issue
The ckpt and log links on the README seem to be unavaliable; they were still available last week.
I am not sure it's the case on my side, but I can't open it even if I switch to another network.
Suggest a potential alternative/fix
No response

Thank you for your reply and look forward to updating

The links have been updated. Please try to download them again.


Cascading object serialization

Hey Koert,
I implemented this similarly to what you had, but the way it is in Scalding. Have a look at the pull request and let me know your thoughts.

thanks Ivan!


Only run :Startify on DirChanged for changed cwd

Ref: https://github.com/tpope/vim-fugitive/issues/1132

Thanks! ✨


Roll Packages from 84ff11d7ee5a to 947e34ce9fed (1 revision)

https://github.com/flutter/packages/compare/84ff11d7ee5a...947e34ce9fed
2024-02-20<EMAIL_ADDRESS>[flutter_markdown] Replace deprecated API (flutter/packages#6134)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-packages-flutter-autoroll
Please CC flutter-ecosystem<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Added /year/month/slug url to blog posts.

To better mimic wordpress and other blogs URL I added a
/year/month/slug url path.

I made the change, do I close this request and request a new one?


error in cin

I want to use this code for solve a simple sat and I have following error in line 58 of code
could be 'std::basic_istream<_Elem,_Traits> &std::operator >><std::char_traits<char>>

Hi @tuytoosh
What process did you follow?

clone the project
make
run ./sat < input.txt

If so, which is the format of your input file?

Hi @jomsdev , thanks for your answer.
I just copied the sss.cpp code into visual studio`s new project and builded it...
I am working in windows 10.

This code has only been tested in *NIX like systems (MacOS, Arch, Debian and Ubuntu).
I assume that your error is at compile time. (Am I right?)
The dependencies of this project are just stl ones
#include <iostream>
#include <stdlib.h>
#include <algorithm>
#include <vector>
#include <time.h>

line 58 is cin >> aux >> numVars >> numClauses;
std::cin is part of the <iostream> library. If it is a "runtime" error, it's about the file format. Does it match with the format of the other inputs? (check this one for example

@jomsdev My error is at compile time... I am compiling code in microsoft visual studio without using cmake or...

@tuytoosh  did you manage to solve the problem?
if not, try to compile a simple program with the same feature
#include <iostream>
#include <stdlib.h>
#include <algorithm>
#include <vector>
#include <time.h>

using namespace std;
int main() {
    int x;
    cin >> x;
    cout << "your integer was: " << x << endl;
}


Nest the UI directory into a ctl subdir

Closes #952
Code Changes

[x] nest the code down
[x] update code references

Steps to Confirm

[ ] local npm commands run

Pre-Merge Checklist

[x] All CI Pipelines Succeeded
Documentation Updated:

[x] documentation complete, or draft/outline provided (tag docs-team to complete/review on this branch)
[ ] documentation issue created (tag docs-team to complete issue separately)

[x] Issue Requirements are Met
[ ] Relevant Follow-Up Issues Created
[x] Update CHANGELOG.md

Description Of Changes
A quick update that nests the UI down one level so it won't conflict with merging the fidesops UI

known bugs happening with the external tests as well as the docs build, merging regardless of the failures


Using custom data with 1280 by 720 image size; RuntimeError: Sizes of tensors must match except in dimension 1. Expected size 46 but got size 45 for tensor number 1 in the list.

Whenever I try to use one dataset where the image size is 1280 * 720, I kept getting the following error
RuntimeError: Sizes of tensors must match except in dimension 1. Expected size 46 but got size 45 for tensor number 1 in the list.
The error generated in YOLOPAFPN
99   f_out0 = torch.cat([f_out0, x1], 1)  # 512->1024/16
As per my knowledge image size should not be a problem as the model automatically resize it in accordance with its input size.

Yes I am facing the exact same issue could someone check this out?

@FahadMahdi this issue can be solved by setting image size to (640,640) or (416,416). If you want multiscale training and want to specify the range of resize checkout the tutorial to change image size

The input size of image should be multiple of 32 here.


CUDA version: Issue with "best gflop/s" if the validation fails for some sizes

This is for the CUDA version. If the CUDA kernel launch fails, the results will fail validation but still be included in the results, so the "best gflop/s" will be too big since the kernel time was very fast since it failed.
One option to avoid this is the following, which if the validation fails, sets the kernel time to the biggest possible, so that it will not mess up the computation of "best" gflop/s. There are probably cleaner ways, this is just what I was using to get it working.
diff --git a/src/main.cpp b/src/main.cpp
index 32b01be..0eba5e7 100644
--- a/src/main.cpp
+++ b/src/main.cpp
@@ -388,9 +388,11 @@ std::pair<double, std::vector<size_t>> validate(const Sample &sample, const Para
     }
     // flush at the end to make sure errors are clumped together
     std::cerr << std::flush;
+    return {valid, maxDiffPct, s, SummaryStats<double>({ std::numeric_limits<double>::max() }), 0, 0, 0};
   }
-
-  return {valid, maxDiffPct, s, SummaryStats<double>(msWithoutWarmup), gflops, gfinsts, interactions_per_sec};
+  else {
+    return {valid, maxDiffPct, s, SummaryStats<double>(msWithoutWarmup), gflops, gfinsts, interactions_per_sec};
+   }
 }

Thanks, would you like to open a PR?

Sure, I will do it now, thanks!


chore(deps): update dependency mocha to v8

This PR contains the following updates:

Package
Change
Age
Adoption
Passing
Confidence

mocha (source)
7.2.0 -> 8.3.0

@types/mocha
7.0.2 -> 8.2.1

Release Notes

mochajs/mocha
v8.3.0
Compare Source
:tada: Enhancements

#4506: Add error code for test timeout errors (@boneskull)
#4112: Add BigInt support to stringify util function (@JosejeSinohui)

:bug: Fixes

#4557: Add file location when SyntaxError happens in ESM (@giltayar)
#4521: Fix require error when bundling Mocha with Webpack (@devhazem)

:book: Documentation

#4507: Add support for typescript-style docstrings (@boneskull)
#4503: Add GH Actions workflow status badge (@outsideris)
#4494: Add example of generating tests dynamically with a closure (@maxwellgerber)

:nut_and_bolt: Other

#4556: Upgrade all dependencies to latest stable (@AviVahl)
#4543: Update dependencies yargs and yargs-parser (@juergba)

Also thanks to @outsideris and @HyunSangHan for various fixes to our website and documentation.
v8.2.1
Compare Source
Fixed stuff.
:bug: Fixes

#4489: Fix problematic handling of otherwise-unhandled Promise rejections and erroneous "done() called twice" errors (@boneskull)
#4496: Avoid MaxListenersExceededWarning in watch mode (@boneskull)

Also thanks to @akeating for a documentation fix!
v8.2.0
Compare Source
The major feature added in v8.2.0 is addition of support for global fixtures.
While Mocha has always had the ability to run setup and teardown via a hook (e.g., a before() at the top level of a test file) when running tests in serial, Mocha v8.0.0 added support for parallel runs. Parallel runs are incompatible with this strategy; e.g., a top-level before() would only run for the file in which it was defined.
With global fixtures, Mocha can now perform user-defined setup and teardown regardless of mode, and these fixtures are guaranteed to run once and only once. This holds for parallel mode, serial mode, and even "watch" mode (the teardown will run once you hit Ctrl-C, just before Mocha finally exits). Tasks such as starting and stopping servers are well-suited to global fixtures, but not sharing resources--global fixtures do not share context with your test files (but they do share context with each other).
Here's a short example of usage:
// fixtures.js

// can be async or not
exports.mochaGlobalSetup = async function() {
  this.server = await startSomeServer({port: process.env.TEST_PORT});
  console.log(`server running on port ${this.server.port}`);
};

exports.mochaGlobalTeardown = async function() {
  // the context (`this`) is shared, but not with the test files
  await this.server.stop();
  console.log(`server on port ${this.server.port} stopped`);
};

// this file can contain root hook plugins as well!
// exports.mochaHooks = { ... }

Fixtures are loaded with --require, e.g., mocha --require fixtures.js.
For detailed information, please see the documentation and this handy-dandy flowchart to help understand the differences between hooks, root hook plugins, and global fixtures (and when you should use each).
:tada: Enhancements

#4308: Support run-once global setup & teardown fixtures (@boneskull)
#4442: Multi-part extensions (e.g., test.js) now usable with --extension option (@jordanstephens)
#4472: Leading dots (e.g., .js, .test.js) now usable with --extension option (@boneskull)
#4434: Output of json reporter now contains speed ("fast"/"medium"/"slow") property (@wwhurin)
#4464: Errors thrown by serializer in parallel mode now have error codes (@evaline-ju)

For implementors of custom reporters:

#4409: Parallel mode and custom reporter improvements (@boneskull):

Support custom worker-process-only reporters (Runner.prototype.workerReporter()); reporters should subclass ParallelBufferedReporter in mocha/lib/nodejs/reporters/parallel-buffered
Allow opt-in of object reference matching for "sufficiently advanced" custom reporters (Runner.prototype.linkPartialObjects()); use if strict object equality is needed when consuming Runner event data
Enable detection of parallel mode (Runner.prototype.isParallelMode())

:bug: Fixes

#4476: Workaround for profoundly bizarre issue affecting npm v6.x causing some of Mocha's deps to be installed when mocha is present in a package's devDependencies and npm install --production is run the package's working copy (@boneskull)
#4465: Worker processes guaranteed (as opposed to "very likely") to exit before Mocha does; fixes a problem when using nyc with Mocha in parallel mode (@boneskull)
#4419: Restore lookupFiles() in mocha/lib/utils, which was broken/missing in Mocha v8.1.0; it now prints a deprecation warning (use const {lookupFiles} = require('mocha/lib/cli') instead) (@boneskull)

Thanks to @AviVahl, @donghoon-song, @ValeriaVG, @znarf, @sujin-park, and @majecty for other helpful contributions!
v8.1.3
Compare Source
:bug: Fixes

#4425: Restore Mocha.utils.lookupFiles() and Webpack compatibility (both broken since v8.1.0); Mocha.utils.lookupFiles() is now deprecated and will be removed in the next major revision of Mocha; use require('mocha/lib/cli').lookupFiles instead (@boneskull)

v8.1.2
Compare Source
:bug: Fixes

#4418: Fix command-line flag incompatibility in forthcoming Node.js v14.9.0 (@boneskull)
#4401: Fix missing global variable in browser (@irrationnelle)

:lock: Security Fixes

#4396: Update many dependencies (@GChuf)

:book: Documentation

Various fixes by @sujin-park, @wwhurin & @Donghoon759

v8.1.1
Compare Source
:bug: Fixes

#4394: Fix regression wherein certain reporters did not correctly detect terminal width (@boneskull)

v8.1.0
Compare Source
In this release, Mocha now builds its browser bundle with Rollup and Babel, which will provide the project's codebase more flexibility and consistency.
While we've been diligent about backwards compatibility, it's possible consumers of the browser bundle will encounter differences (other than an increase in the bundle size). If you do encounter an issue with the build, please report it here.
This release does not drop support for IE11.
Other community contributions came from @Devjeel, @Harsha509 and @sharath2106. Thank you to everyone who contributed to this release!

Do you read Korean? See this guide to running parallel tests in Mocha, translated by our maintainer, @outsideris.
:tada: Enhancements

#4287: Use background colors with inline diffs for better visual distinction (@michael-brade)

:bug: Fixes

#4328: Fix "watch" mode when Mocha run in parallel (@boneskull)
#4382: Fix root hook execution in "watch" mode (@indieisaconcept)
#4383: Consistent auto-generated hook titles (@cspotcode)
#4359: Better errors when running mocha init (@boneskull)
#4341: Fix weirdness when using delay option in browser (@craigtaub)

:lock: Security Fixes

#4378, #4333: Update javascript-serialize (@martinoppitz, @wnghdcjfe)
#4354: Update yargs-unparser (@martinoppitz)

:book: Documentation & Website

#4173: Document how to use --enable-source-maps with Mocha (@bcoe)
#4343: Clean up some API docs (@craigtaub)
#4318: Sponsor images are now self-hosted (@Munter)

:nut_and_bolt: Other

#4293: Use Rollup and Babel in build pipeline; add source map to published files (@Munter)

v8.0.1
Compare Source
The obligatory patch after a major.
:bug: Fixes

#4328: Fix --parallel when combined with --watch (@boneskull)

v8.0.0
Compare Source
In this major release, Mocha adds the ability to run tests in parallel. Better late than never! Please note the breaking changes detailed below.
Let's welcome @giltayar and @nicojs to the maintenance team!
:boom: Breaking Changes

#4164: Mocha v8.0.0 now requires Node.js v10.12.0 or newer. Mocha no longer supports the Node.js v8.x line ("Carbon"), which entered End-of-Life at the end of 2019 (@UlisesGascon)

#4175: Having been deprecated with a warning since v7.0.0, mocha.opts is no longer supported (@juergba)
:sparkles: WORKAROUND: Replace mocha.opts with a configuration file.

#4260: Remove enableTimeout() (this.enableTimeout()) from the context object (@craigtaub)
:sparkles: WORKAROUND: Replace usage of this.enableTimeout(false) in your tests with this.timeout(0).

#4315: The spec option no longer supports a comma-delimited list of files (@juergba)
:sparkles: WORKAROUND: Use an array instead (e.g., "spec": "foo.js,bar.js" becomes "spec": ["foo.js", "bar.js"]).

#4309: Drop support for Node.js v13.x line, which is now End-of-Life (@juergba)

#4282: --forbid-only will throw an error even if exclusive tests are avoided via --grep or other means (@arvidOtt)

#4223: The context object's skip() (this.skip()) in a "before all" (before()) hook will no longer execute subsequent sibling hooks, in addition to hooks in child suites (@juergba)

#4178: Remove previously soft-deprecated APIs (@wnghdcjfe):

Mocha.prototype.ignoreLeaks()
Mocha.prototype.useColors()
Mocha.prototype.useInlineDiffs()
Mocha.prototype.hideDiff()

:tada: Enhancements

#4245: Add ability to run tests in parallel for Node.js (see docs) (@boneskull)
:exclamation: See also #4244; Root Hook Plugins (docs) -- root hooks must be defined via Root Hook Plugins to work in parallel mode

#4304: --require now works with ES modules (@JacobLey)

#4299: In some circumstances, Mocha can run ES modules under Node.js v10 -- use at your own risk! (@giltayar)

:book: Documentation

#4246: Add documentation for parallel mode and Root Hook plugins (@boneskull)

:nut_and_bolt: Other

#4200: Drop mkdirp and replace it with fs.mkdirSync (@HyunSangHan)

:bug: Fixes
(All bug fixes in Mocha v8.0.0 are also breaking changes, and are listed above)

Renovate configuration
📅 Schedule: At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about these updates again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

⚠ Artifact update problem
Renovate failed to update an artifact related to this branch. You probably do not want to merge this PR as-is.
♻ Renovate will retry this branch, including artifacts, only when one of the following happens:

any of the package files in this branch needs updating, or
the branch becomes conflicted, or
you check the rebase/retry checkbox if found above, or
you rename this PR's title to start with "rebase!" to trigger it manually

The artifact failure details are included below:
File name: packages/opentelemetry-cloud-trace-propagator/package-lock.json
bash: line 0: hash: npx: not found


fixed typo

I found a small typo in nbs 22, I am creating a pull request to fix it and train myself at contributing to the repo.

Thanks a lot!


fix(Beaker): Exception has been thrown as raise missed argument

Exception has been thrown as raise missed argument

David are these internal tests failing because of new way to authenticate ? Is it safe to merge?

David are these internal tests failing because of new way to authenticate ? Is it safe to merge?

@Tiboris Correct, it's failing due to credentials. I will find a wy to fix them, in the meanwhile, i think we can merge this :)


Build mage with bootstrap.go

I noticed you're installing mage with go get. The problem with this is that it doesn't build in all the -version info.  You don't need mage to build mage, there's a bootstrap file that will let you use go run to install mage.
This will download the mage source and install it (if you're using a gopath):
go get -d github.com/magefile/mage
go run $GOPATH/src/github.com/magefile/mage/bootstrap.go install

If you're using modules, you can do it with good old git clone:
git clone git@github.com:magefile/mage
go run ./mage/bootstrap.go install

note that in the second case, the binary will be copied to where go env GOPATH points.

The problem with this is that it doesn't build in all the -version info.

What do you mean by that? Is it a "less functional binary" or is it just cosmetics? (i.e. the output of "mage -version").
I don't care about the cosmetic version info. I do, however, care about getting the correct version (the one in my go.sum file). My tests show that I now at least get the Mage version that I have specified. Do any of your suggestions add any non-cosmetic improvement to that?

If it were me, I would run mage -version after building it to ensure I got the right version (since that has been a problem).   If you don't want to do that, then yeah, there's no difference in the way it's built.


Moved some test codes (mostly Tuple related)

There were two test classes in redis.clients.jedis package which are now moved to redis.clients.jedis.tests package.
import static is used for assertEquals in HostAndPortTest class.
tupleCompare test method is moved into TupleTest class and removed two redundant checks.

LGTM @HeartSaVioR

LGTM. Thanks @sazzad16 for the contribution.

Merged to master and up to 2.9. Thx @sazzad16 for the contribution!.


Prevent fees > some max values

Prevent burning more tokens than intended due to some typo.
Quick solution would be to hard code some reasonable max value
The SDK comes with 2 options to set fees via CLI parameters:

a fix amount : --fees string              Fees to pay along with transaction; eg: 10uatom
a dynamic amount based on TX simulation and these parameters:
 --gas string               gas limit to set per-transaction; set to "auto" to calculate sufficient gas automatically (default 200000)
--gas-adjustment float     adjustment factor to be multiplied against the estimate returned by the tx simulation; if the gas limit is set manually this flag is ignored  (default 1)

Any solution should prevent both cases to exceed a fix maximum value.
The flag names are defined in github.com/cosmos/cosmos-sdk/client/flags

flags.FlagFees
flags.FlagGasPrices
flags.FlagGasAdjustment

The PersistentPreRunE could be extended to parse the flags early. With simulations this is more complicated

Would be vert good to have this implemented but not critical to delay launch


Failed to connect: "config.brokers" threw: brokers is not a function

Hi! I was wondering if you guys could help me understand the schema and solve this issue.
I have a prototype with the following docker-compose (here is the consumer code aswell)
version: '3'
services:
  zookeeper:
    image: wurstmeister/zookeeper:latest
    ports:
      - "2181:2181"
  kafka:
    image: wurstmeister/kafka:2.11-1.1.1
    ports:
      - "9092:9092"
    environment:
      KAFKA_ADVERTISED_HOST_NAME: localhost
      KAFKA_ZOOKEEPER_CONNECT: zookeeper:2181
      KAFKA_CREATE_TOPICS: "topic-test:1:1"
    volumes:
      - /var/run/docker.sock:/var/run/docker.sock

I run docker-compose up and it's working fine.
Now I have an AdonisJS project and I'm trying to make it communicate with another PROJECT (for now, my prototype)
I run npm install kafkajs ("kafkajs": "^1.15.0",)
Then in my Controller:
const { Kafka } = require('kafkajs')
async testKafka ({ request, response }) {
   const kafka = new Kafka({
      clientId: 'api-producer',
      brokers: 'localhost:9092' // Also tried to change to same as composer file 'kafka:9092'
   })
}

const producer = kafka.producer()
request.producer = producer
await request.producer.connect() // ERROR HERE

When I try to make the request using insomnia I get the following ERROR:
{
  "error": {
    "message": "Failed to connect: \"config.brokers\" threw: brokers is not a function",
    "status": 500
  }
}

It was working when I was doing both consumer/producer on the same space LIKE:
/kafka // Main Folder
  /api/src   //API Code
  /microservice/src //simulating consumer

But now that I'm trying to connect them from a real project it's not working.
Also, am I right to think like:

I'll have a server running Kafka
I'll have project A -- Producer
I'll have project B -- Consumer
Whenever I run an http request from project A and trigger kafka code sending a message, KAFKA server will receive this and send the message to ==> Project B (as long as it is listening to the same topic?.
I read documentation, but this part still blurry to me.

Thanks
-------- EDIT
I was reading Kafka files and brokers  MUST BE an Array. My bad!
Although, if you guys could help me with the second question =)

I'm gonna close this issue as you found your problem, which is indeed that config.brokers is an array of strings.
As for your second question, I would suggest that you read up a bit on how Kafka works. There's nothing specific about KafkaJS there. But in general, yes, if you produce a message to a topic, and some consumer is subscribed to that topic, then the consumer will consume the message. That said, there are many caveats to this, such as how messages are distributed across partitions, what happens when you have multiple consumer instances, etc. But again, this is general Kafka stuff, for which there are plenty of resources on the web.


feat: Add Stackable vendor label to recommended labels

Part of https://github.com/stackabletech/issues/issues/188

Added your suggestion.


Support and/or testing of DIGEST

DIGEST support has been a bit flaky, but now Ersatz can actually test it reliably so let's get the DIGEST support working and tested fully.

Need to upgrade to Ersatz v0.5.0+
Add DIGEST testing to all HTTP methods
Ensure that BASIC is also well-tested
May need to add extra library to support in OkHttp client

This has been addressed in the 0.14.0 release (development branch)


Update request: texlive 2022 → 2023

Package name: texlive
Latest released version: 2023 (released on 2023-03-19, see news)

Current version on the unstable channel: 20221227
Current version on the stable/release channel: 2021

[x] Checked the nixpkgs pull requests

Notify maintainers

@veprbl

Note for maintainers: Please tag this issue in your PR.

Sorry, did not see that there's #222142 already...


Throw on null

Currently fails with Proxy error with mewting null
mewt(null)
// TypeError: Cannot create proxy with a non-object as target or handler

Solved similarly to Underscore's _.isObject. Also, there is no need to check if it is an Array since Arrays are already the typeof 'object'. Finally, added tests for the all potential typeofs.

Thanks again! 😄


#172: Added Integer and Decimal as a possible type for all result att…

…ributes in QDM 5.3.

Bryn,
I've requested a review from Chris Moesel.
Best,
Lizzie

Also -- just to confirm, these changes shouldn't affect the XSD?

And yes, these changes don't affect the XSD, I'm not reflecting the choice types in the XSD, only the model info.

Okay, submitted a fix for these issues. This PR will need to be merged first though, otherwise, the component changes from the other PR may be stomped.


Add Travis-CI logo

Adds the Travis-CI logo
Closes #1107 - also discussion on there to decide which logo & color to use

@1.5x
@2x

Doesn't show well on the social style badge, but don't see it ever being used on that.
Should be ready for merge, will leave open for the day in case anyone notices anything that they think should be changed.

Hey @paulmelnikow. Just heard from Travis CI folks that alterations or unapproved uses of their logo are not something they've approved. This is one more nail in the "service logos in Shields badges" coffin for me. Can we make a plan to revert this change or deprecate this badge to replace it with the normal Travis CI build badge so we don't break people's stuff?

In general, I'd much prefer if we stopped including any company logos in badges. Any logo usage like this is likely to be a violation of brand guidelines at the very least. At worst, like in this case, it's a misuse of copyrighted or trademarked assets. I've explained a long time ago why I think logos are at cross-purpose with the idea of Shields.
I've yet to see a usage of a service logo that made me reconsider this: https://github.com/badges/shields/blob/master/spec/SPECIFICATION.md
Aside from the legal/contractual perspective, I still think logos are aesthetically problematic because we have limited range design-wise to accommodate their design within our own badge designs. But beyond the aesthetic consideration focusing on the service companies and their logo design, I think their logos compromise the Shields badge design(s) itself.
@RedSparr0w you can reach out Travis CI folks directly if you want but their policy on the matter is pretty clear: https://travis-ci.com/logo

The fact that they reached out to me directly and chose not to start a conversation here makes me thing that perhaps we should take the hint.

I think we are using their logo to "show that [our] application or product integrates with Travis CI" (by calling their API) and that it is there so that our users can "use the logo to show [they're] running [their] tests on Travis CI". Perhaps if one of us offered to get a tattoo, or bake them some cookies it might smooth things over? :wink:
That said, I don't really care to pursue it myself (or get a tattoo). If one of you wants to fight that battle, feel free. I'd probably just delete the logo and blacklist it so we won't use the simple-icons version.

I'd prefer that someone reach out to Travis and try to get their permission to continue to use the logo, because users who depend on the logo will come back and read this discussion, and it will be harder to reach out to Travis then.
That said, if none of the maintainers want to do that, then let's remove it promptly, and block it from being loaded via simple-icons.
We've never had a company complain before, so I don't see a pressing need for us to act immediately beyond this badge. We've supported the logos for a long time, they're popular, and other badge services provide them, which means (company requests aside) the decision to remove them from Shields should be made by the community. (Given the popularity of logos, would removing them drive users to other services? If so, would the net effect of those users walking away be more uniformity or less?)
Some other things we can do to "reel things in," so to speak:

Document best practices for using badges (#2868)
Incorporate a statement on design philosophy into the project, and create guidelines about how we deal with the badge format.

Vendor in or at least quote Olivier's "rage diamond" blog post might be a good idea, too.

Consider removing styles which are rarely used (like popout* and plastic)

Hello, there. So, the violation here is: "Alter the logo in any way". The logo we encourage others to use are shown on the logo page, and it unfortunately does not include a mono-colored one.
Aesthetically speaking, Mr. T is a little too busy (too many intricate lines) to look really good with a single color in a very tight space such as the badge.
We want to make this easier for our users, of course, but that would take some time. So, in the mean time, we appreciate it if you could remove Mr. T.
Thank you for your understanding and cooperation.

Hi! Thanks so much for the clarification!
One question: Am I correct that you're okay with us using the full color or grayscale versions that are published on that page?

You mean this one, right?

Yes. But, as I said before, this won't look good if you shrink it too much.

It's not great, though I think some users would find it better than nothing until Travis approves something new. I think if we check it in logoColor would be ignored and so it'll always show this color.
What do you think @RedSparr0w @chris48s?

IMO the full color looks better, but the grayscale isn't bad.

1x
2x

🤔 Let me ask that question internally, and get back to you.

@BanzaiMan It seems like using the logos from that page in the posted colors should be in line with the guidelines. Can you confirm? Thanks so much!

Hello, there. I've asked internally, as promised, and came to the conclusion that, among the choices given in @RedSparr0w's comment https://github.com/badges/shields/pull/1276#issuecomment-465034694, the full color version is the best.
I hope this clarifies the situation. Let me know if you have further questions. Thank you!

Fabulous. Glad to hear that! Thanks so much!


Remove unnecessary installation steps

Since everything now runs via the official maven repository, these steps are no longer necessary to use the surrealdb java client.

@phughk Could you take a look, please?

Yes, thanks for spotting! Merging now :)


Configure buildbot to run tests inside vagrant.

#173 introduced a decorator that detects if the tests are running on a vagrant share, to skip ones that deal with permissions. We should run tests on buildbot under these circumstances to detect when we need to add that decorator.

We are moving our development planning to JIRA. This issue is now being tracked at https://clusterhq.atlassian.net/browse/FLOC-180. You are welcome to file additional issues in GitHub if that's easier for you.


demo to run 3Dunet

hello, thanks for you work, would you mind to share you demo file to run this model? Appreciate

I do not have a demo file at this time for this repository. But we have created a docker hub for our new proposed brain tumor segmentation model at https://hub.docker.com/r/pykao/brats2018/.