Move utils

Moved several simple utility classes from xenon.engine.util to xenon.util to allow re-use by applications that use xenon.

Codecov Report

Merging #407 into master will decrease coverage by -35.71%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #407       +/-   ##
============================================
- Coverage     85.51%   49.8%   -35.71%
+ Complexity     2262    1410      -852
============================================
  Files           159     159
  Lines          6806    6806
  Branches       1020    1020
============================================
- Hits           5820    3390     -2430
- Misses          661    3182     +2521
+ Partials        325     234       -91

Impacted Files
Coverage Δ
Complexity Δ

.../xenon/adaptors/scripting/RemoteCommandRunner.java
0% <ø> (-84.85%)
0 <ø> (-8)

...esciencecenter/xenon/engine/util/BatchProcess.java
58% <ø> (-34%)
8 <ø> (-7)

...java/nl/esciencecenter/xenon/util/InputWriter.java
0% <ø> (ø)
0 <ø> (?)

...sciencecenter/xenon/engine/util/CommandRunner.java
0% <ø> (-52.18%)
0 <ø> (-6)

.../nl/esciencecenter/xenon/util/StreamForwarder.java
42.85% <ø> (ø)
6 <ø> (?)

...ava/nl/esciencecenter/xenon/util/OutputReader.java
0% <ø> (ø)
0 <ø> (?)

...ecenter/xenon/adaptors/ftp/FtpDirectoryStream.java
0% <ø> (-100%)
0% <ø> (-3%)

...enon/adaptors/ssh/SshDirectoryAttributeStream.java
0% <ø> (-100%)
0% <ø> (-3%)

...n/adaptors/webdav/WebdavRegularFileAttributes.java
0% <ø> (-100%)
0% <ø> (-2%)

...non/adaptors/webdav/WebdavDirectoryAttributes.java
0% <ø> (-100%)
0% <ø> (-2%)

... and 75 more

Continue to review full report at Codecov.

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


Floating point exception xt::random::randn

This
#include <xtensor/xio.hpp>
#include <xtensor/xrandom.hpp>

int main()
{
  xt::xtensor<int,2> A = xt::random::randn<int>({20,5});

  std::cout << A << std::endl;
}

Results in
[1]    3823 floating point exception  ./test

The internally used std::normal_distribution only works with floating point types.
https://en.cppreference.com/w/cpp/numeric/random/normal_distribution

The result type generated by the generator. The effect is undefined if this is not one of float, double, or long double.

Not sure if we should prevent users from instantiating with <int>...

I see, so
xt::xtensor<int,2> A = 100. * xt::random::randn<double>({20,5});

indeed works 👍


Add an equality check method to ComPlainDate and ComPlainTime

Plain-date and plain-time objects can be compared with <, >=, etc... but not for equality with == because of JS objects, duh.
It'd still be useful to compare them for equality though.
The simplest way would be an is(date: ComPlainDate): bool method that checks that all three properties are equal.
Next level would be isMatch(date: { year?: number | string, month?: number | string, day?: number | string }): bool that would return true when just the given properties match.
Having the latter makes the first one redundant. Would it make sense to implement the latter but calling it simply is, or would that be a confusing name?

Let's consider what the independent utility function would be called!
isDateTimeMatch(pattern: SloppyDateTime)(subject: SloppyDateTime)

Ideally it'd compare two objects with any combination of the common date & time properties.
So subject { year: 2023, hour: 12 } would return true for pattern { hour: 12 }. But should it also match if the arguments were reversed? I.e. would it only compare properties that are present in both objects? If so, the arguments should probably be named something more generic, like:
isDateTimeMatch(a: SloppyDateTime)(b: SloppyDateTime)

And what if there is no overlap at all, would comparing { year: 2023 } to { hour: 12 } return false?
Perhaps the utility function should be stricter and only return true when every present property in both objects match, while the objects' is method match properties present in the pattern only?

I found this for creating a type where at least one of the properties is required: https://stackoverflow.com/questions/48230773/how-to-create-a-partial-like-that-requires-a-single-property-to-be-set

I'm thinking that perhaps an independent utility function for comparing sloppy date-time objects is not really needed.
For now we'll add the methods to ComPlainDate and ComPlainTime. If we ever have ComPlainDuration in the future, perhaps the corresponding method on that object will cover those cases where the independent utility function would be used anyway?


MemberDefinition.TypeName is set to its parent's type name, not itself

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
When using Get-Member to get a property from a class, and then getting TypeName property on it, it returns the parent's type name, instead of the target property.
For example consider this code:
class MyClass {
    [string]$CoolProperty = "Hi"
}

$myValue = [MyClass]::new()
($myValue | Get-Member -MemberType Property)[0].TypeName

The last line will return MyClass, but what we actually want it to do is to return type name of the property "CoolProperty" (which is string). I don't think this is intended (If this was really intended, then TypeName should be renamed to ParentTypeName, so it doesn't make confusion!)
I believe the issue is from here.
Expected behavior
PS> ($myValue | Get-Member -MemberType Property)[0].TypeName
string

Actual behavior
PS> ($myValue | Get-Member -MemberType Property)[0].TypeName
MyClass

Error details
no error.

Environment data
Name                           Value
----                           -----
PSVersion                      7.3.7
PSEdition                      Core
GitCommitId                    7.3.7
OS                             Microsoft Windows 10.0.19042
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Visuals

The .TypeName property by design refers to the type that the member at hand is a part of.
Unfortunately, the Microsoft.PowerShell.Commands.MemberDefinition instances output by Get-Member do not have a property that reflects the .NET type of the member itself, but the information is embedded in the [string]-valued .Definition property:
PS> ($myValue | Get-Member -MemberType Property)[0].Definition
string CoolProperty {get;set;}

It seems to me that Get-Member is primarily used for interactive discovery, so its output is geared toward providing information to the human observer rather than facilitating programmatic use.
As a simple alternative to Get-Member, if you have an instance of a type of interest, you can simply call .GetType() on a property of interest; for methods, simply omit ():
# Property
PS> $myValue.CoolProperty.GetType()

IsPublic IsSerial NameSpace                                Name                 Location                                                               BaseType
-------- -------- ---------                                ----                 --------                                                               --------
True     True     System                                   String               /Users/mklement/.powershell-PREVIEW/System.Private.CoreLib.dll         System.Object

# Method
PS> $myValue.ToString

OverloadDefinitions
-------------------
string ToString()

It is unfortunate that Get-Member doesn't support reflection at the .NET type level (with the limited exception of static members via -Static); see:

#6382

List of all other Get-Member-related feature requests (the above isn't labeled as such, due to its age):

https://github.com/PowerShell/PowerShell/issues?q=is%3Aissue+is%3Aopen+in%3Atitle+get-member+label%3Aissue-enhancement+

@mklement0 Thanks for the complete info!

It seems to me that Get-Member is primarily used for interactive discovery, so its output is geared toward providing information to the human observer rather than facilitating programmatic use.

I understand that. But the thing is, this is way too much confusing! When a normal human uses .TypeName on a property, will it really expect to get the parent's type name?? It took me half an hour to debug my script just to understand each time I'm calling my function type checker, it's checking for same type (yup, I was iterating on output of Get-Member and each time was receiving the wrong type).

Also, thanks for the link (Microsoft.PowerShell.Commands.MemberDefinition) that you provided! And cool fact that inside of the docs, they haven't pointed out .TypeName will be returning parent's type name...

Well, if powershell developers decide to not change this behavior (or at least not make it a little bit clear in the docs), I'll respect their decision and just try sending a pull request on powershell docs and mention in there that .TypeName actually returns property's parent's type name (.....instead of just writing there "type name" lol)

Glad to hear it was helpful, @ALiwoto.
The documentation for the output types emitted by cmdlets isn't part of the main PowerShell documentation, and is notoriously terse, because it is compiled from source-code comments:

https://github.com/PowerShell/PowerShell/blob/0beb4dad4407c689afca3e75c84ff59074f6d69c/src/Microsoft.PowerShell.Commands.Utility/commands/utility/GetMember.cs#L35-L38

As such, you'd have to create a PR here - but I'm not sure there's even a process around making documentation-in-source-code-only changes.

Note that in the for-display output produced by Get-Member the problem is mitigated by the enclosing type name being a grouping caption; e.g.:
PS> $myValue | Get-Member CoolProperty

   TypeName: MyClass

Name         MemberType Definition
----         ---------- ----------
CoolProperty Property   string CoolProperty {get;set;}


animation kit-ml dataset

Hi, i know how to animation the HumanML3D dataset, however i can't animation KIT-ml,the shape is (timesteps, 21,3),

Hi, for python animation, you just need to change the arguments of kinematic chain and fps while calling plot_motion_3d function

Hi HappyPiepie can u please share how did u animate the HumanML3D dataset?

Hi, for python animation, you just need to change the arguments of kinematic chain and fps while calling plot_motion_3d function

hi, which fps i need use?

30
On Mon, Jun 17, 2024 at 4:11 AM ngc7293 @.***> wrote:

Hi, for python animation, you just need to change the arguments of
kinematic chain and fps while calling plot_motion_3d function
hi, which fps i need use?
—
Reply to this email directly, view it on GitHub
https://github.com/EricGuo5513/HumanML3D/issues/36#issuecomment-2172588913,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKRYNB7VFLWAH7QPYB4HSHTZH2K47AVCNFSM6AAAAAAVPI5PDSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNZSGU4DQOJRGM
.
You are receiving this because you commented.Message ID:
@.***>

I downloaded your KIT_ML new_joint dataset and animated this,  but the result is just like this, what error I make?
https://github.com/EricGuo5513/HumanML3D/assets/25836491/246a0cf7-7484-499a-9bda-1774187a3a8a

30
…
On Mon, Jun 17, 2024 at 4:11 AM ngc7293 @.> wrote: Hi, for python animation, you just need to change the arguments of kinematic chain and fps while calling plot_motion_3d function hi, which fps i need use? — Reply to this email directly, view it on GitHub <#36 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AKRYNB7VFLWAH7QPYB4HSHTZH2K47AVCNFSM6AAAAAAVPI5PDSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNZSGU4DQOJRGM . You are receiving this because you commented.Message ID<EMAIL_ADDRESS>
I downloaded your KIT_ML new_joint dataset and animated this, but the result is just like this, what error I make?
00009.mp4

Same here. Did you fix this problem?


RFE: allow filtering per partition in Cluster tab

A useful addition to the "Cluster" view would be the possibility to filter by partition.

Ok, that's a "tricky" one.
The way filtering is done is by concatenating strings from selected columns and then running one regexp across this big string. That way we get away with some nice gimmicks using very little code, like: ability to write multi-column filters like jobname_re.*username_re.*(PEND|RUN)
The challenge with partitions filtering, at least on our site, and that's the reason they're not included in the filter string is that we have single letter partition names (c,g,m), which doesn't work that well with the logic above (but we do have partition letter in the node naming schema, e.g. clip-[gmc], so we get away with partition filtering) :laughing:
I'll have to think a bit about how to tackle this, any ideas/proposals how to solve the partition filtering to the user in this situation?

Done. Single letter partitions can be filtered out with the following form: \.m\..


[hotfix-DTStack#932][chunjun-examples|docs] fix content's type is str…

I fixed the chunjun-example's stream.json and docs quickstart the issue address is [no.932]
{https://github.com/DTStack/chunjun/issues/932}

Thanks a lot for this pr. But i'm sorry that we got the same pr #927 and we just merged.

ok！chunjun is a good data product


Will SMPLX-based version be released?

Having tested PSHuman with some samples, I find some problems that can probably be solved with SMPLX conditioning. According to Table 6 in the paper, SMPLX conditioning is very important for model performance.
Correct me if I'm wrong, but I think the currently released SMPL-Free version and thee SMPLX-based version have different weights. Therefore, I'd like to ask whether the SMPLX-based version of PSHuman will be released?
Thank you again for your great work~

Yes, we will release the SMPLX-based version soon.


Checksum mismatch error while downloading dependencies

Hey,
I am getting the following error while trying to download dependencies for the project:

go: downloading github.com/btcsuite/btcd v0.20.0-beta
verifying<EMAIL_ADDRESS>checksum mismatch
downloaded: h1:PamBMopnHxO2nEIsU89ibVVnqnXR2yFTgGNc+PdG68o=
sum.golang.org: h1:DnZGUjFbRkpytojHWwy6nfUSA7vFrzWXDLpFNzt74ZA=
SECURITY ERROR
This download does NOT match the one reported by the checksum server.
The bits may have been replaced on the origin server, or an attacker may
have intercepted the download attempt.
For more information, see 'go help module-auth'.

This issue should be fixed by upgrading btcd to v0.20.1-beta, here is more info on this:
https://github.com/btcsuite/btcd/issues/1487
I tried to do it myself, however some tests are failing after making this upgrade. Could you look into this? I can't verify if these tests are failing due to my change or not, since I can't run tests with previous version of btcd :)

Thks for the report.  looks like the retag of btcd cause the issue. We will upgrade the version soon.

hi @apralat  you may try release v1.2.1


Docs URL broken

The link in this README and in the pypi entry link to a 404 docs page
https://googleapis.dev/python/cloudfunctionslatest

I don't have triage permissions on this but wanted to update with the needed update

Update docs/README.rst and README.rst with https://googleapis.dev/python/cloudfunctions/latest


Article does not contain customization instructions

This article tells me how to view, search and disable rules. It does not explain how to add new rule groups or compose custom rules.

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

ID: 422b2c33-f7e5-d800-6852-44f35b41dc60
Version Independent ID: ea29c962-8fa1-e6dc-5024-682d7879410d
Content: Customize web application firewall rules in Azure Application Gateway - Azure portal
Content Source: articles/application-gateway/application-gateway-customize-waf-rules-portal.md
Service: application-gateway
GitHub Login: @vhorne
Microsoft Alias: victorh

Investigating...

@nerath
Sorry, this not yet possible. But it is a feature on the to-do list!

Got it. Thanks for the follow-up!
From: Vic<EMAIL_ADDRESS>Sent: Thursday, July 26, 2018 1:36 PM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Erath, Neil R<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: [EXTERNAL] Re: [MicrosoftDocs/azure-docs] Article does not contain customization instructions (#12233)
@nerathhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_nerath&d=DwMCaQ&c=9TnM8cDdIndCtQknbqO0eA&r=zPD-zaqQO5srqh0QDOw7zQ&m=xKM2MHO-3s2osGmmGofv4blNRX32118cgrRHI5BolkY&s=gJeoPNe0d6TB4zC3IYbK3m9APArDkFS4mAa_57klvxA&e=
Sorry, this not yet possible. But it is a feature on the to-do list!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_MicrosoftDocs_azure-2Ddocs_issues_12233-23issuecomment-2D408175451&d=DwMCaQ&c=9TnM8cDdIndCtQknbqO0eA&r=zPD-zaqQO5srqh0QDOw7zQ&m=xKM2MHO-3s2osGmmGofv4blNRX32118cgrRHI5BolkY&s=MUoBWv8ZU5SJ7nkQ0cXB_jXzqlOqrK3HnCx8GD1E-Kc&e=, or mute the threadhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AnqiPmkcBNgEx48GRAe-5F9T-5FTfMvavodHks5uKf3ngaJpZM4VbdVP&d=DwMCaQ&c=9TnM8cDdIndCtQknbqO0eA&r=zPD-zaqQO5srqh0QDOw7zQ&m=xKM2MHO-3s2osGmmGofv4blNRX32118cgrRHI5BolkY&s=KOb9It9aZZS22k43oLVXhOIVDfGlfoTF87yanht9hZ0&e=.

This email (including any attachments) is intended for the designated recipient(s) only, and may be confidential, non-public, proprietary, and/or protected by the attorney-client or other privilege. Unauthorized reading, distribution, copying or other use of this communication is prohibited and may be unlawful. Receipt by anyone other than the intended recipient(s) should not be deemed a waiver of any privilege or protection. If you are not the intended recipient or if you believe that you have received this email in error, please notify the sender immediately and delete all copies from your computer system without reading, saving, printing, forwarding or using it in any manner. Although it has been checked for viruses and other malicious software ("malware"), we do not warrant, represent or guarantee in any way that this communication is free of malware or potentially damaging defects. All liability for any actual or alleged loss, damage, or injury arising out of or resulting in any way from the receipt, opening or use of this email is expressly disclaimed.

#please-close

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

Any time frames on when we can expect the ability to create custom rules?


Patch osx 29 jan2015

This is the pull request following our chats yesterday - http://chat.stackoverflow.com/transcript/message/21261643#21261643

Great, thanks for building!  And hopefully you will be able to get some mileage out of it.  My hope is that even with it in an imperfect state, it can be useful.  And don't tell anyone, but if you want to use "vanilla" Rebol without all my proposals just comment out this bit:
https://github.com/hostilefork/rencpp/blob/bca47c16944b6a37ef53cb078bb49949fe43f4a9/examples/workbench/renconsole.cpp#L455
(I don't want to make it an option yet, because I'd like to get people into the mindset of trying and fixing all those things...!)
...and I wanted to mention to you that when you said not knowing C++ could be a barrier to contribution, that I pushed another piece of functionality out from C++ to Rebol (autocomplete!) and also began the documentation effort for these "helper" scripts:
https://github.com/hostilefork/rencpp/tree/develop/examples/workbench/scripts/helpers
So it shouldn't be too hard to tinker and contribute.  And shows the general plan for how I'd like to start seeing as much of the logic as possible implemented outside the C++ part!

Oh, although don't let me forget... if you have a few minutes sometime, we should try to make it so you don't need to apply any patches to a-lib.c to get it to build...I thought I understand the issues so we wouldn't need to, but see your instructions still mention it!

Unfortunately Earl's Rebol patch (https://gist.github.com/earl/d027d702121323efdfec) is still needed for OS X.
I just tried rebuilding from latest source without the patch I get the following error...
[ 39%] Building CXX object examples/CMakeFiles/function-1.dir/function-1.cpp.o
Linking CXX executable function-1
duplicate symbol _Host_Lib in:
    ../libRenCpp.a(rebol-os-lib-table.cpp.o)
    ../libRenCpp.a(a-lib.o)
ld: 1 duplicate symbol for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)
make[2]: *** [examples/function-1] Error 1
make[1]: *** [examples/CMakeFiles/function-1.dir/all] Error 2
make: *** [all] Error 2

Compiles and run OK with the patch.  NB. Tested on OS X Mavericks 10.9.5
So we would need to get Earl to add this patch to the RebolSources Github repo.


No autocompletion available when installed via Spack

Following the getting started guide https://reframe-hpc.readthedocs.io/en/stable/started.html#enabling-auto-completion it states there should be autocompletion scripts under <install_prefix>/share/completions/reframe.<shell>.
Spack shows:
$ spack find -p reframe
==> 1 installed package
-- linux-centos8-sandybridge /<EMAIL_ADDRESS>------------------------
<EMAIL_ADDRESS>/nopt/spack/opt/spack/linux-centos8-sandybridge/gcc-9.3.0/reframe-3.4-rkbham2eyv3cy7fpot7ljf7eejjnjtc7

But there's no share/ directory I can find in there?

Hi @sjpb auto-completion was introduced in 3.4.1 and updated in 3.4.2. The Spack repo has only up to 3.4, unfortunately. We need to update the recipes there.

@sjpb ReFrame 3.4.1 and 3.4.2 have just been merged into Spack.


Not UTC timezone given when connection is etablished

In newer versions > 19.11 Clickhouse sends non standart timezone and go app in docker image without tzdata fails:
{"level":"info","ts":1566471707.4000757,"msg":"Connection to clickHouse is open"}
panic: Unable to build connClickhouse ||: Unable to ping clickHouse connection: could not load time location: unknown time zone Etc/UTC

GO sources - https://golang.org/src/time/zoneinfo.go?s=7575:7624#L265
GO driver fo CH - https://github.com/kshvakov/clickhouse/blob/master/lib/data/server_info.go#L38

It's not a bug. The reason of that tzdata2019.1.
SELECT timezone()

┌─timezone()─┐
│ Etc/UTC    │
└────────────┘

vi  /etc/default/clickhouse

export TZ=UTC

/etc/init.d/clickhouse-server restart
SELECT timezone()

┌─timezone()─┐
│ UTC        │
└────────────┘

Most probably relates to #5828 and #5922
Now clickhouse resolves symlinks during default timezone detection.
/cc @alexey-milovidov


daemon: Track policy implementation delay by source

We see high variance in the time from a .PolicyAdd in the daemon to when
a revision is finally in the datapath. Quantifying this delay is
valuable when diagnosing certain classes of problems, particularly
transient drops to IPs seen by the DNS Proxy and inserted into node's
policy.
The options to .PolicyAdd now includes a source field that reflects the
source of that specific invocation. It is expected to be one of api,
fqdn or k8s. This information is passed onto a new goroutine that waits
on each revision to be deployed and records the time since the
.PolicyAdd invocation into a new policy_implementation_delay histogram.

This change is

test-me-please

test-me-please

test-me-please

Thinking about this a little more I think I prefer the callback scheme more: https://github.com/cilium/cilium/compare/master...raybejjani:fqdn-policyadd-propagationtime-callback?expand=1
In particular, it does seem better to record this information per endpoint (but aggregated by source) than for all endpoints.

@raybejjani needs rebase.

Coverage decreased (-0.02%) to 42.979% when pulling a9a5ff27520855b9ca30d5587c0eaa122d6c9ebb on raybejjani:fqdn-policyadd-propagationtime into e3abc9c7f1146d3c587bca347d32724ab52f5731 on cilium:master.

@tgraf I switched to the callback based scheme. It's a little cleaner, and tracks the delay per endpoint. This gives us insight into the best- and worst-case scenarios.

@aanm Let me know if the callback thing is ok. I can do it differently if you want (e.g. I can make closing the channel a callback too! but that's probably not useful).

test-me-please

test-me-please

test-me-please

test-me-please


Default ownership choice should be available as a global setting

By default, the ownership field is set to 'private' when creating a volume, container or service. This should be available as a global setting, some users might want  this field to default to public.

That would be a great enhancement!

Are you sure that the ownership is default set to 'private'? My volumes/containers/services are all 'public' by default. I am using Portainer 1.13.3.

@m-a-v yes, when you head into the container/service/volume creation view, access control is enabled and the first selected choice is either private or administrator (if you're an admin).
Point here is to give the ability to change that default value in these views. Of course containers created outside of Portainer will always be declared as public as access control is managed from inside Portainer.

Would it be difficult to provide a mechanism to change the default behaviour for externally created docker volumes/containers/services? Portainer could tag the ressources which are listed for the first time.

Yes I'm afraid that this would be quite hard to implement. In the meantime I'd recommend setting the ACL manually on each resource using an administrator account.

Don't you have a database with all the entries? If the ownership field is not defined you could just set it to private or the default ownership which has to be implemented first :-).

The database only store UAC related information, it does not store any Docker related information (containers, etc...). So that would require parsing all the containers running on boot / when adding an endpoint and automatically add information on each resource. And also a refactor on how the resource control model is currently defined.
You can open another issue for that and we can discuss it more in there.

@deviantony would you close this issue related to #960? Now the default ownership is set to admins. Another more practical option is to set the default ownership using labels #1257.

This issue was originally opened to allow the user to set a default choice in the UI when associating an access control policy on a resource. I believe that it is not really required. Closing.

Well, I actually need this feature... Me and my friend share portainer and I want him to be able to manage whatever is created on his docker host via cli.

@mateuszdrab I'm working on it on #2424 as missing it is real showstopper for us.
In mean while you can get it working by downgrading Portainer to 1.19.1 version.

Thanks @olljanat
I'll await for the next release, hopefully this will make its way soon. :)

FYI, this one is now solved with new RBAC extension. There you can grant user(s) "Endpoint administrator" role which with they can see externally created resources but not change system settings.
@deviantony IMO this can be closed.

I'm keen to close this issue when we'll introduce #1257, I believe that the possibilities available with both #1257 and the RBAC extension should cover the requirements for this one.

So, it seems like a year and a half later this behaviour still hasn't been implemented :/

What's the latest on this?

FYI, this one is now solved with new RBAC extension. There you can grant user(s) "Endpoint administrator" role which with they can see externally created resources but not change system settings.

@deviantony It looks to be that this is not case anymore. Even when "Environment administrator" role description is "Full control of all resources in an environment" it still does not give access to swarm resources which are created outside of Portainer.


chore(blob-explorers): accomodate new blobscan API changes

Blobscan had some more API changes. They added a few things:

a field for tx category, present on both block/tx responses.
added the rollup field for the block response as well.
They renamed the DataStorageReferences fields to be storage and url.

This aligns the new responses to the new format, and another tx test. See their API docs.


Added missing newline to example Post route

Just something I noticed while trying out binding for the first time that bothered me more than it should :+1:

:) Thanks.


[crashtracker] Better way to pass timeouts to receiver

          Right--this one is hard.

Two failed ideas from my side.

The receiver code actually doesn't have control over the acquisition of its own args and it'd be a weird change to the entrypoint to make a new args struct required.  So not a great choice of arg propagation.
Relying on the receiver configuration means waiting a few steps into the IPC sequence, which is not only weird but incorrect.  I think I suggested this at one point, but I see how naive that was.

That really leaves the environment as the dominant way of passing this configuration.
With that in mind, some thoughts.  I think the first two are constructive and the last is speculative.
First, we should probably by the naming that has been determined elsewhere.  All the runtimes are using the CRASHTRACKING label instead of CRASHTRACKER.
Second, how do you feel about marking this "private' (prepending one or two _ to the name)?  End-users probably shouldn't be passing this.  They should be using whatever configuration levers are provided by their library, and the library should pass this.
Third, here's an idea for tightening the interface.  We could take a new option in the ReceiverConfig and then propagate it to the receiver's environment in make_receiver as part of the Prepared Args.
To sketch out that idea,

impl PreparedExecve would have a new fn new_with_env(config: &CrashtrackerReceiverConfig, env: Option<Vec<String, String>>) interface
Merge config.env and env (this other vec) in the PreparedExecve new_with_env, with the env coming last (winner-take-all in case of conflict).

Originally posted by @sanchda in https://github.com/DataDog/libdatadog/pull/702#discussion_r1826256968

Apologies if this has already been discussed and discarded, but it occurred to me that a command-line argument may work as well for this :D


Add support for poll interval and max-reconcile-rate flags

Description of your changes
Add support for poll interval and max-reconcile-rate flags allowing to the users set a custom configuration. In order to use these flags it is necessary update the controller runtime used from k8s-runtime to crossplane-runtime. Also the starting debug log now prints the new configuration on start up:
2024-01-26T08:24:58+01:00	DEBUG	provider-kafka	Starting	{"sync-period": "1h0m0s", "poll-interval": "1m0s", "max-reconcile-rate": 10}
Fixes #61
I have:

[x] Read and followed Crossplane's contribution process.
[x] Run make reviewable test to ensure this PR is ready for review.

How has this code been tested
Tested locally by:

Executing make test
Running the provider in a local cluster without any additional configuration and checking that it still working with the default configuration
Running the provider with --poll=2m --max-reconcile-rate=1  and checking that 2 topics and 2 ACLs are processed one by one and once they are in a ready status, the reconcile loop happened every 2 min

Hi @turkenh,
I am not sure if you were notified of this PR, could you take a look at it?
Thanks :)

Any news about this?
We need it too :)


Add OrderByKey() and OrderBy(field,"desc") and asc for StreamBuilder

bro, please add this feature on next update :(

Without additional information, we are unfortunately not sure how to resolve this issue. We are therefore reluctantly going to close this bug for now. Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.


Fix process spawning in macOS runtime

Currently, the macOS runtime is failing to spawn processes with an EPERM. Fix this.

closed in #148


Update WebView2 release note to add HiddenPdfToolbarItems API

Update WebView2  release note to add HiddenPdfToolbarItems API and its link.

Docs Build status updates of commit 0fbddd1:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/webview2/release-notes.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

@Reezaali this is a quick fix by fei for a link to the api! Good to merge! Thx


Update Video.md

Thank you for your contribution to MagicScript Project.
Before submitting this PR, please make sure:

[ ] Your code builds clean without any errors or warnings
[ ] You are using approved terminology
[ ] You have added unit tests
[ ] You have updated documentation

Need to finish updating viewMode


Enable strict mode for all SDKs but dart

:scroll: Description
#skip-changelog
:bulb: Motivation and Context
https://dart.dev/guides/language/analysis-options#enabling-additional-type-checks
:green_heart: How did you test it?
:pencil: Checklist

[ ] I reviewed submitted code
[ ] I added tests to verify changes
[ ] I updated the docs if needed
[ ] All tests passing
[ ] No breaking changes

:crystal_ball: Next steps
dart package but that would be another PR since its much bigger and more fragile

Codecov Report
Base: 90.03% // Head: 90.14% // Increases project coverage by +0.10% :tada:

Coverage data is based on head (0937223) compared to base (3ad1e5e).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1163      +/-   ##
==========================================
+ Coverage   90.03%   90.14%   +0.10%
==========================================
  Files         110      119       +9
  Lines        3483     3672     +189
==========================================
+ Hits         3136     3310     +174
- Misses        347      362      +15

Impacted Files
Coverage Δ

dio/lib/src/sentry_transformer.dart
94.44% <0.00%> (ø)

dio/lib/src/sentry_dio_client_adapter.dart
100.00% <0.00%> (ø)

logging/lib/src/logging_integration.dart
90.47% <0.00%> (ø)

logging/lib/src/extension.dart
100.00% <0.00%> (ø)

dio/lib/src/dio_event_processor.dart
96.36% <0.00%> (ø)

dio/lib/src/sentry_dio_extension.dart
76.92% <0.00%> (ø)

dio/lib/src/tracing_client_adapter.dart
85.71% <0.00%> (ø)

dio/lib/src/breadcrumb_client_adapter.dart
80.00% <0.00%> (ø)

dio/lib/src/failed_request_interceptor.dart
100.00% <0.00%> (ø)

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Request: levelChange event

I have spent quite some time researching the docs, debugging outputs, posting (no reply :( ), trying to find a reliable method to find out when Marzipano triggers a level change event.
My goal is to be able to display loading indicators (and more) when/if level changes and if textures start loading. The textureLoad event is kinda unusable alone if there is no way to know if level changes and textures starts loading in the first place. Surely a levelChange event or currentLevel property would be useful for extended interfaces?
Is there no existing method to get current level or detect level change?

about level, did you see this : RectilinearView.prototype.selectLevel = function(levelList){ .. return level according height and fov).

@samuelgirardin Thanks, but I can only see that this function is used to manually SET the level to be used to render the view.
http://www.marzipano.net/reference/RectilinearView.html#selectLevel

Select the level that should be used to render the view.

in fact no, this method is always used when zooming , put some console.log :
`for (var i = 0; i < levelList.length; i++) {
// var level = levelList[i];
          if (this._coverFactor * levelList[i].height() >= this._requiredPixels) {

            console.log( levelList[i]) ;
                return levelList[i];
            }
        }`

you will see  sthg like that
CubeLevel {_fallbackOnly: false, _size: 1024, _tileSize: 512} rectilinearView.js:817 CubeLevel {_fallbackOnly: false, _size: 1024, _tileSize: 512} rectilinearView.js:817 CubeLevel {_fallbackOnly: false, _size: 1024, _tileSize: 512} rectilinearView.js:817 CubeLevel {_fallbackOnly: false, _size: 1024, _tileSize: 512} rectilinearView.js:817 CubeLevel {_fallbackOnly: false, _size: 1024, _tileSize: 512} rectilinearView.js:817 CubeLevel {_fallbackOnly: false, _size: 2048, _tileSize: 512} rectilinearView.js:817 CubeLevel {_fallbackOnly: false, _size: 2048, _tileSize: 512} rectilinearView.js:817 CubeLevel {_fallbackOnly: false, _size: 2048, _tileSize: 512}

you can see hre  the switch between level 1024 and 2048

Thanks. I am actually using single-file 'EquirectGeometry' sources, two levels (two files), and I'm trying to figure out what loads when even before there is any zoom. Depending on screen, first level may be preloaded, while second level may be loading, or not even required at all. To determine this, regardless of any zoom (although that could trigger a new level), I need to detect current target level. Do you have  code snippet example of selectLevel in action? It still seems a bit vague for the requirement, is not an event, and I need to feed it with some data?
In the meantime, I have had some progress with textureLoad, which seems to return an object with level data loaded. For EquirectGeometry sources, it  returns z-index of the level and src width amongst other things.

Given the current API, the best bet to detect whether textures are loading is the TextureStore#query method. The selectLevel method is only meant to be used internally.
I agree it makes sense to add the TextureStore#textureStartLoad and Layer#levelChange events.
Although not documented, all event handlers are called with the event name as the first argument. In the case of TextureStore#textureLoad, the second parameter is (as documented) the tile the event pertains to.

I've added a Stage#renderComplete event which should make loading indicators a lot easier to implement.
The usage looks like this:
viewer.stage().addEventListener('renderComplete', function(eventName, complete) {
  // show or hide loading indicator depending on the value of `complete`
});

The API may change in the future, but for now complete is just a boolean (true = loading complete, false = loading in progress).
I will add an example to the demos soon.

Thanks @tjgq! I am very much looking forward to try new renderComplete and textureStartLoad events to achieve desired interface behaviors. Since they are not yet part of a release or NPM, I am personally a unsure how to compile github src for testing. In the meantime, I am using a textureLoad workaround for a beta version, but will test new events soon.

Hi. I have been looking into Stage#renderComplete event, and it does the job. A few observations:
1. It triggers fast. Perhaps as frequent as viewChange event? I can see it has to trigger frequently, since new textures may need to load after view change. I was however expecting it to perhaps only trigger when there was a state change to the complete value, but perhaps that is not logical to implement.
Can add our own logic for that of course, but I'm cautious about including events that trigger unnecessarily. Stage#renderComplete may not be productive for single-file equirect sources, since we are only expecting a single load.
2. The event triggers on window blur, even after viewer.destroyAllScenes(). I noticed this because I have a popup mechanism, and the event persists even after the popup is closed and all scenes are destroyed. Harmless, but I'm a bit paranoid about events leaking once I have destroyed all scenes and closed our panorama modal.
For my panorama 'popup' app, I am creating a persistent instance of Marzipano.Viewer that remains in memory (also after popup close). Then I just create/destroy scenes on popup open/close, to avoid having to recreate the Viewer instance each time, which includes a lot of controls and custom interface events. I'm still evaluating if this is a good idea, but apart from hogging some additional memory I assume, I can't see why not.

The renderComplete event triggers whenever the stage renders, full stop. There's a number of events that cause the stage to render (changes to the view, effects, fixed level, or loading/unloading of textures), but the render rate is capped by requestAnimationFrame (typically 60 fps).
Note that firing the event is very inexpensive compared to the render that has just occurred. If you only care about changes to the "complete" parameter, it should be cheap to check for it on the event handler.
Once you call Viewer#destroyAllScenes, I would expect a couple more render events to draw a blank stage, and maybe react to texture unloading (the latter is technically unnecessary). However, an empty stage should not keep firing renderComplete indefinitely; if you're seeing repeated events after you empty the stage, that definitely looks like a bug, and I'd be interested in looking at a minimal repro. (For what it's worth, I tried calling Viewer#destroyAllScenes in a couple of  demos and the event stops firing.)


Fix general pipeline failures

Update license year
Fix semgrep tests

Static test failures:

Min Platform Version
Action Coverage


Added Compute Replacement API in OPAMP component of OneOps

Added Compute Replacement API in OPAMP component of OneOps

 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.Davinder Singh seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Cancelling this PR , created a new PR # 359


[Simplified Collect][Profile] Implement default description and invite messages

cc @narefyev91 from the Simplified collect design doc we need to make updates to the default description and invite messages.

Hi! I'm Nicolay from Callstack - expert contributor group. I'll work on this issue so please assign me. Thank you.

@luacmartins could you assign me here? thanks

Done

Just pending payment

Waiting on payment

Bump for payment @JmillsExpensify

Bump for payment @JmillsExpensify

Payment summary

Contributor: No payment necessary
Contributor+: @getusha $500

C+ contract paid and regression test created. Closing this one!


[ibm.com/standards/web page] Add indicator when links are internal

The problem
Users who are not working within IBM VPN will get an error page when they try to click on most of the links on this page -> ibm.com/standards/web
Suggested solution
So for all links on this page that send the user to w3 page, need to add to the card an indicator that it is for internal use.
An example is on this page:
https://www.ibm.com/design/language/ibm-logos/8-bar
under the Resources section.  It show message after title  (IBM id required)
Application/website
www.ibm.com/standards/web
Link to repo for changes:  https://github.ibm.com/webstandards/standards-website
This is the page where the edits will be needed: https://github.ibm.com/webstandards/standards-website/blob/master/src/gatsby-theme-carbon/templates/Homepage.js.
Business priority
Low Priority = release date is not dependent on fix or not upcoming
What time frame would this ideally be needed by (if applicable)
Nice to have in next few months.
Examples
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I checked the current issues for duplicate issues

No epic needed.  Create one design issue that is linked to this for a review of Linda's recommendation and refinement.

This is the page where the edits will be needed https://github.ibm.com/webstandards/standards-website/blob/master/src/gatsby-theme-carbon/templates/Homepage.js.

Needs an estimate, bringing back for planning poker.

Hey team! Please add your planning poker estimate with ZenHub @annawen1 @ariellalgilmore @emyarod @IgnacioBecerra @kennylam


Markdown horizontal scrollbar usability

To repro:

ADS
Version: 1.26.1 (user setup)
Commit: 796abbdf5fdd341e3528e30a6b777c93e115eb02
Date: 2021-02-24T23:42:55.085Z
VS Code: 1.48.0
Electron: 9.4.3
Chrome: 83.0.4103.122
Node.js: 12.14.1
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19042

Is your feature request related to a problem? Please describe.

Floating horizontal scrollbar is hard to get to when word wrap is turned off.
At a glance, users won't know that there is a scroll bar.

It only shows when you hover around somewhere in the middle.

Sometimes it's off the screen. This makes it hard for scrolling horizontally while editing the lines that are on the top, There will be a lot of scrolling up and down while scrolling horizontally.

Describe the solution or feature you'd like
The Horizontal scrollbar to show more intuitively.
Describe alternatives you've considered
None.
Additional context
Word wrap is often turned off for users can work with tables.

Added more details and screenshots.

Scroll bar visibility: this seems to be consistent with rest of ADS: only show scrollbars on mouse hover and/or if editor is active. I think if we are to make a change, it needs to be more global and would require alignment with everyone. At first glance, this doesn't seem like a problem to me.
Scroll bar off screen: For large cells, I think that it is problematic that it is off screen. I think we should make it sticky to the bottom of the visible area, similarly how the markdown editor is sticky to the top.
Scroll bar position: It should show up at the bottom of the cell, and not in the middle (end of content). The exception is if the bottom of the cell is lower than visible area. Then, it should be sticky to the bottom of visible area.

Changing this to P2.


Since you had the pcd2las，why you still use the CC viewer？

Hi, pcd2las.py is a script which utilizes CloudCompare software to finish format conversion. To successfully use pcd2las, you have to install the CloudCompare and adjust the path in the script.

Is the operation under the  windows or linux as I see CloudCompare.exe in the order?

It can be done both in windows & Linux as long as you install the CloudCompare and configure the path. In windows, it is CloudCompare.exe.


made signup.html similar to login.html

@deepbeatz ✅

@deepbeatz I have merged the request but check about that three icons appearing in your image near password section

@mrbhatt2348 sir that is just my chrome extension.

avira password manager

avira password manager

Okay 👍


Few questions

Hi,
Thanks for the great library. Works perfectly fine. I had couple of questions:
Is it possible remove the logs that show up on logcat?
What happens when no internet on mobile?
What about the privacy issues here? Do you store locations details of users?
what is the minimum Android version and google play service required?
Let me know!
Thanks!

Yeah sure. I cannot remove them from the core library code as it is handy
for other developers, though you can download the zip code and implement it
manually and remove the logs.
The GPS has nothing to do with internet connectivity. I had used fused
location api so that has an algorithm set to take location first from gps,
then network connection, then wifi, then bluetooth, So no internet
connection wont be a problem.
Regarding privacy, I had made this library public, So you can view the
source code, I am not saving the locations of any user, I am just passing
them to the call back.
On Thu, Oct 27, 2016 at 5:09 PM, vijaykd<EMAIL_ADDRESS>wrote:

Hi,
Thanks for the great library. Works perfectly fine. I had couple of
questions:
Is it possible remove the logs that show up on logcat?
What happens when no internet on mobile?
What about the privacy issues here? Do you store locations details of
users?
Let me know!
Thanks!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/kingsammalik/SamLocationAndGeocoding/issues/4, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AKEddpSwLJHToaes8SmH4O5WSjuIaSLZks5q4I2MgaJpZM4KiPyG
.

--
Sachin Malik
Android Developer -  Caringforyou IT solutions
Call: +91 http://nitinsinhame.net/8802729349  |  Mail:
<EMAIL_ADDRESS>Please consider the environment before printing this email


Update README.rst

modify require python version >= 3.6 to >= 3.6.1
cause 3.6.0 does not include Deque in typing. it was added on 3.6.1
Reference: https://docs.python.org/3/library/typing.html

I help maintain @home-assistant which is a consumer of websockets hence I have an interest in this PR.
I would prefer if instead of updating the min version, websockets drops the Deque type and instead type it with deque imported from collections and keeps supporting Python 3.6.0.

Thanks for the PR and sorry for not replying earlier!
Some significant code changes were required in addition to these documentation changes. I just made all these changes, which supersede this pull request.


Rich Text fields incorrectly set form to modified when clicking into them

Bug Report
That's still not fixed in 1.4?! I think it works in normal fields, but in arrays -> richtext it doesn't. Even just setting the cursor inside the richtext (which is inside an array) instantly reverts it from published to saved
Other
It is by no means a priority to fix, just noticed it...

Hey @johannesschaffer — this is actually something different. I have updated the issue to reflect the actual issue and will investigate shortly!

@johannesschaffer this is now fixed and released in patch 1.4.1.
Good catch. Slate has lots of "gotchas" but this one was relatively straightforward.
Relevant Slate issue:
https://github.com/ianstormtaylor/slate/issues/3567
We want to be as diligent as possible with resolving actual issues so feel free to bring it on 😎

This is a bit unrelated 🙈, but I'm doing a React project in the near future which involves a text editor and I was planning to use Slate, given its popularity. Would you recommend a different editor then, if Slate has so many hickups?


Add disable_skip_errors flag to ConventionChecker API

Added to ConventionChecker.check_source() and ConventionChecker.check().
This allows other systems that integrate pydocstyle,
such as flake8-docstyle, to request errors that are otherwise
skipped due to # noqa comments.
Added test for skip_errors feature in general and with this flag.
Fixes #484
Thanks for submitting a PR!
Please make sure to check for the following items:

[x] Add unit tests and integration tests where applicable.
If you've added an error code or changed an error code behavior,
you should probably add or change a test case file under tests/test_cases/ and add
it to the list under tests/test_definitions.py.
If you've added or changed a command line option,
you should probably add or change a test in tests/test_integration.py.
[x] Add a line to the release notes (docs/release_notes.rst) under "Current Development Version".
Make sure to include the PR number after you open and get one.

Please don't get discouraged as it may take a while to get a review.

Will release a new minor version tomorrow.


Make Float work with no_std (rebased #50)

Use ::core::num::FpCategory when deriving Float to work with no_std
Get rid of unnecessary double-borrowing to fix clippy lint

This is a rebase of #50 without the inlining change that I was not comfortable with.

bors r+


I can't program esp32 adc pins as I want, is there a way to do this? (IDFGH-5371)

I am measuring analog reading voltage with adc_1 39 pin.
I'm measuring interrupt rpm with 36 pins.
34 input 3.3v.
35 input 3.3v.
If 39.34.35 pins are energized. 36 pin produces wrong value.
39.34.35 pins if it connects to gnd it fixes it.
but i can't use 353439
how do i get over this

https://github.com/espressif/esp-idf/issues/1096


[Feature] 建议支持v2ray内核

Verify Steps

[X] Tracker 我已经在 Issue Tracker 中找过我要提出的问题
[X] Need 当前 OpenClash 并不包含该功能特性或者还不完善
[X] Framework 这是 OpenClash 应包含的特性, 并非 Clash 特性
[X] Meaningful 我提交的不是无意义的 催促更新或修复 请求

Describe the Feature
clash所有上游均已停止维护。
Describe the Solution
建议支持v2ray/sing-box内核
Describe Alternatives
No response
Additional Context
No response

那样openclash目前的脚本都要重写了

我也觉得添加协议工作量应该前所未有的大。
转网上哥们发了份统计。

Clash.Meta「已archived」
Clash Core「已删库」
Clash for Windows「已删库」
Tuic协议「已archived」
Helloworld路由器插件「已删库」
Gui.For.Clash「已删库」
ClashForAndroid 「已删库」
homeproxy 「已archived」
ClashMetaForAndroid 「已archived」
Fclash 「已archived」
Clash verge 「已archived」

meta还能打一阵，但独木难支或许只是时间问题，期待singbox早日推出适宜gui。
感谢项目组成员大佬们长久以来的维护，小弟拜谢。
穷鬼一个，谈不起赞助，一行文字，略表谢意。

there's passwall/passwall2 that support xray and sing-box. but I can't get it to work with my router. something about loopback idk. without turning on the main switch the nodes seems to be ok. the ping appears. but when the main switch turned on, the ping timeout. and the issues on that project also closed at the same time with the closing of clash/meta 😭

可以出一个纯防火墙模式，二进制文件自己制定。反正redirect和tproxy大部分软件都支持。

You can create a pure firewall mode and customize the binary file yourself. Anyway, most software supports redirect and tproxy.

give me an example for fw4 with sing-box
I played around with box for root for Android. I think it has potential. but I don't know how to implement forwarding for openwrt. does it even support tun. in android it does support tun.


Error 415 : Unsupported Media Type

Hello
On :
Terraform v0.11.1
+ provider.null v1.0.0
+ provider.opc v0.1.3

Each of the examples (even simple) ends up with :
* provider.opc: 415: {
  "code" : "UnsupportedMediaType",
  "message" : "Unsupported Media Type"
}

The debug doesn't seems to help but i may track it the wrong way, export TF_LOG=DEBUG till ERROR seems pretty verbose but i can't see the json produced ...
Have a nice day.

Does the following work for you:
$ git clone https://github.com/oracle/terraform-provider-compute
$ cd terraform-provider-compute/examples/opc/instance-with-ssh/
$ ssh-keygen -f ./id_rsa -q -N ""

Create a file terraform.tfvars (set with your account details)
user =<EMAIL_ADDRESS>password = "PASSWORD"
domain = "mydomain"
endpoint = "https://compute.uscom-east-1.oraclecloud.com/"

Then run:
$ terraform init
$ terraform plan
$ terraform apply

Thanks, but still the same issue.
(Also variables are well set except var.domain)
var.domain - *terraform.NodeRootVariable
var.endpoint - *terraform.NodeRootVariable
var.password - *terraform.NodeRootVariable
var.ssh_public_key_file - *terraform.NodeRootVariable
var.user - *terraform.NodeRootVariable
2017/12/19 08:27:26 [ERROR] root: eval: *terraform.EvalConfigProvider, err: 415: {
"code" : "UnsupportedMediaType",
"message" : "Unsupported Media Type"

I'm getting the same error.  Tested on multiple versions of new opc provider from 0.1.3 to 1.0.1, so assuming this is a syntax issue, however, lie cledorze said, even my simple examples are not working.  I'm on Win7 with same error in powershell or cmd.

Hold up... I confused the OPC provider with the OCI provider.  OPC is for classic api no?  So classic auth won't work for new endpoints correct?  So in my case, it is probably an auth issue disguised as a resource error.  Using this endpoint will generate the error: https://iaas.us-ashburn-1.oraclecloud.com.  Using this classic endpoint https://compute.uscom-east-1.oraclecloud.com/ does not and I get an auth error as expected.
Error: Error running plan: 1 error(s) occurred:

provider.opc: 401: {"message": "Incorrect username or password"}

OPC provider is for "Oracle Cloud Infrastructure Classic" (previously known as Oracle Public Cloud Compute) and "Oracle Cloud at Customer" (Oracle Cloud Machine)
For the OPC provider authentication you need to set:

endpoint either the Region of Zone endpoint URL e.g.  https://compute.uscom-east-1.oraclecloud.com or https://api-z27.compute.us6.oraclecloud.com
identity_domain the Identity Domain name for traditional accounts e.g. acme, or the Service ID for newer IDCS based accounts, e.g. or 575260584
user
password

More info:

https://www.terraform.io/docs/providers/opc/index.html
https://docs.oracle.com/en/cloud/iaas/compute-iaas-cloud/stcsa/Authentication.html

f.y.i the OPC provider and issues repo has moved to the official terraform providers repo at https://github.com/terraform-providers/terraform-provider-opc, this repo location is deprecated.
The "Oracle Cloud Infrastructure" OCI provider is currently maintained at https://github.com/oracle/terraform-provider-oci

Hi
I have also similar issue.
Is there any update.
curl -i -X POST -H "Cookie: $COMPUTE_COOKIE" -H "Content-Type: application/Compute-707014927-v3+json" -H "Accept: application/Compute-707014927-v3+json" -d<EMAIL_ADDRESS>https://compute.eucom-north-1.oraclecloud.com/launchplan/
HTTP/1.1 415 Unsupported Media Type
Date: Wed, 28 Feb 2018 12:57:31 GMT
Server: nginx
Content-Type: application/json
Content-Length: 37
X-Oracle-Dms-Ecid: 1.005P^L9GMLkFs1c_xh^AyW00010p0061ug;kXjE0ZDLIPJGjCPOpUROcGSRXURB
Expires: Wed, 28 Feb 2018 12:57:31 GMT
X-Oracle-Compute-Call-Id: 180228c62c3674bca62f3b9635b883f1c0cf86
Cache-Control: no-cache
Content-Language: en
{"message": "Unsupported Media Type"}[root@mysql-db1 tmp]

Hi
I have resolve my issue and I can create instance and storage volume now.
Thanks
Sanjay


cargo fix: duplicate diagnostic detection does not handle insert-only replacements

The following example:
macro_rules! foo {
    () => {
        &1;
    };
}

fn main() {
    foo!();
    foo!();
}

generates a suggestion (on beta 1.55) to insert a fix like this:
let _ = &1;

However, since this diagnostic is triggered from macros, rustc emits two separate machine-applicable suggestions at the exact same spot. That causes rustfix to end up changing the code to:
let _ = let _ = &1;

which fails to compile.
rust-lang/rustfix#131 added detection to avoid applying duplicate suggestions, but I believe it does not handle the case when it is an insert_only suggestion.

https://github.com/rust-lang/rustfix/pull/131 seems to work because in the case of duplicate replacements, it is enough to check if a replacement is the same before and after. A duplicate insertion wouldn't be able to do that, and would need a more fundamental fix.


New build uwuw

Add security edit
Build on musl based x86 architectures

should I merge this?

i will close this


Percentile NPE

CrateDB version: 1.0.1
JVM version: 1.8
OS version / environment description: Centos 7
Problem description:
When executing a percentile query, sometimes I would get the results, but other times I would get an NPE.
SQLActionException: INTERNAL_SERVER_ERROR 5000 NullPointerException in io.crate.types.DataTypes.fromStream(DataTypes.java:182)

SELECT PERCENTILE(someColumn, 0.95) FROM someTable;

Steps to reproduce:
Running the SQL statement above via the Crate UI (console) and hitting execute. Sometimes the query would succeed, sometimes it would not.

We're not able to reproduce this issue.
Could it be that you're running nodes with different versions in the same cluster? Do you changed the cluster (size, settings, etc) somehow while running into the NPE compared to a successful query?

It appears one node in our cluster is throwing the error. It's the same version as all of the other nodes 1.0.1.
The log ouputs:
[ERROR][io.crate.types.DataTypes] 5120 is missing in TYPE_REGISTRY
java.lang.NullPointerException
at io.crate.types.DataTypes.fromStream(DataTypes.java:182)
...

can you please provide us the output of select version, os_info from sys.nodes; ?
also the complete stacktrace would be helpful.
what client are you using?
thanks a lot.

We're using the crate HTTP endpoint to execute queries at the moment.
JVM Version: 1.8.0_92
VM Version: 25.92-b14
All nodes say Crate 1.0.1
Stacktrace:
[ERROR][io.crate.types.DataTypes] 5120 is missing in TYPE_REGISTRY
java.lang.NullPointerException
at io.crate.types.DataTypes.fromStream(DataTypes.java:182)
at io.crate.planner.node.dql.AbstractProjectionsPhase.readFrom(AbstractProjectionsPhase.java:128)
at io.crate.planner.node.dql.RoutedCollectPhase.readFrom(RoutedCollectPhase.java:228)
at io.crate.planner.node.ExecutionPhases.fromStream(ExecutionPhases.java:53)
at io.crate.operation.NodeOperation.readFrom(NodeOperation.java:115)
at io.crate.operation.NodeOperation.<init>(NodeOperation.java:62)
at io.crate.action.job.JobRequest.readFrom(JobRequest.java:71)
at org.elasticsearch.transport.netty.MessageChannelHandler.handleRequest(MessageChannelHandler.java:222)
at org.elasticsearch.transport.netty.MessageChannelHandler.messageReceived(MessageChannelHandler.java:116)
at org.jboss.netty.channel.Channels.fireMessageReceived(Channels.java:268)
at org.jboss.netty.channel.Channels.fireMessageReceived(Channels.java:255)
at org.jboss.netty.channel.socket.nio.NiWorker.read(NioWorker.java:88)
at org.jboss.netty.channel.socket.nio.AbstractNioWorker.process(AbstractNioWorker.java:108)
at org.jboss.netty.channel.socket.nio.AbstractNioSelector.run(AbstractNioSelector.java:337)
at org.jboss.netty.channel.socket.nio.AbstractNioWorker.run(AbstractNioWorker.java:89)
at org.jboss.netty.channel.socket.nio.NioWorker.run(NioWorker.java:178)
at org.jboss.netty.util.ThreadRenamingRunnable.run(ThreadRenamingRunnable.java:108)
at org.jboss.netty.util.internal.DeadLockProofWorker$1.run(DeadLockProofWorker.java:42)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
at java.lang.Thread.run(Thread.java:745)

Does this issue still occur? If so, does it always occur on the same node?
We still aren't able to reproduce this, maybe you can provide us some more information (steps) to reproduce?
Also the complete output of this query would be really nice select version, os_info from sys.nodes;

I am also getting the same error but it occurs every time. I created a simple table and everything worked just fine. The table that is failing only has 2k entries and eventually will be > 160M. It has 23 fields, composed of 2 long, 5 timestamp and the rest are string.
I cannot cut/paste from the source system but here is the output from "select version, os_info from sys.nodes". User bputt works in the same place and his info is probably the same. There are 4 entries, I assume for each host in the cluster, that are all the same:
{"version": {"build_snapshot": false, "number": "1.0.1", "build_hash": "227 ...3c4"},
"os_info": {"version": "3.10.0-327.36.3.e17.x86_64", "arch": "amd64", "name": Linux", "jvm": {"vm_name": "OpenJDK 64-Bit Server VM", "version": "<IP_ADDRESS>", "vm_version": "25.111-b15", "vm_vendor": "Oracle Corporation"}, "available_processors": 8}}

@bmwmaestoso @bputt thanks for your reply, we could reproduce it now and will have a look.

@bputt @bmwmaestoso we've fixed it by https://github.com/crate/crate/commit/c6f5a8940e6d862061affad36ab2ed7421fd0f83. it will be released with the next 1.0.2 hotfix release. thanks for reporting (and being patient ;-).

@bmwmaestoso @bputt forgot to mention: as this bug is class loading issue, you could workaround it by sending a percentile containing statement once to every running node inside the cluster. every node which is handling the client request will load the required class. afterwards a percentile statement should work everywhere.

thanks, i got it working


Namespace for k8s metrics

Current behaviour
When collecting k8s metrics with awscontainerinsightreceiver  and exporting using awsemf the namespace is always set to the configured value in the metrics.
Desired behaviour
The metrics regarding pods/services/replicasets etc should contain the k8s Namespace the resources are running in.
It seems that there is no possibility to configure this behaviour. Or am I missing something?

@crigertg can you please clarify if this is same as https://github.com/aws-observability/aws-otel-collector/issues/1712 ?

@vsakaram yes this is the same issue. When opening this I wasn't aware of the cause. Sorry for not updating this issue. I'll close it.


Add a porter explain command

Closes:
What does this change
This PR adds a new explain command that prints usage information for a bundle! Long gone are the days of trying to figure out how to use the bundle by running install until it works!
$ porter explain --tag jeremyrickard/porter-do-bundle:v0.4.1
Name: spring-music
Description: Run the Spring Music Service on Kubernetes and Digital Ocean PostgreSQL
Version: 0.4.1

Credentials:
+------------------+--------------------------------+----------+
|       NAME       |          DESCRIPTION           | REQUIRED |
+------------------+--------------------------------+----------+
| kubeconfig       | Kube config file with          | false    |
|                  | permissions to deploy app      |          |
| do_access_token  | Access Token for Digital Ocean | false    |
|                  | Account                        |          |
| do_spaces_key    | DO Spaces API Key              | false    |
| do_spaces_secret | DO Spaces API Secret           | false    |
+------------------+--------------------------------+----------+

Parameters:
+---------------+--------------------------------+---------+-------------------+----------+-------------+
|     NAME      |          DESCRIPTION           |  TYPE   |      DEFAULT      | REQUIRED | APPLIES TO  |
+---------------+--------------------------------+---------+-------------------+----------+-------------+
| node_count    | Number of database nodes       | integer | 1                 | false    | All Actions |
| porter-debug  | Print debug information from   | boolean | false             | false    | All Actions |
|               | Porter when executing the      |         |                   |          |             |
|               | bundle                         |         |                   |          |             |
| region        | Region to create Database and  | string  | nyc3              | false    | All Actions |
|               | DO Space                       |         |                   |          |             |
| space_name    | Name for DO Space              | string  | jrrportertest     | false    | All Actions |
| database_name | Name of database to create     | string  | jrrportertest     | false    | All Actions |
| helm_release  | Helm release name              | string  | spring-music-helm | false    | All Actions |
| namespace     | Namespace to install Spring    | string  | default           | false    | All Actions |
|               | Music app                      |         |                   |          |             |
+---------------+--------------------------------+---------+-------------------+----------+-------------+

Outputs:
+------------+--------------------------------+--------+-------------+
|    NAME    |          DESCRIPTION           |  TYPE  | APPLIES TO  |
+------------+--------------------------------+--------+-------------+
| service_ip | IP Address assigned to the     | string | All Actions |
|            | Load Balancer                  |        |             |
+------------+--------------------------------+--------+-------------+

No custom actions defined

This also provides support for json and yaml output.
An implementation note: I defined "Printable" types for this that only expose some of the fields of each type AND remove the omitempty on the json/yaml tags. I felt it was important to show values, even when they'd be omitted because they were "empty" (i.e. false...).
What issue does it fix
Closes #635
If there is not an existing issue, please make sure we have context on why this change is needed. See our Contributing Guide for examples of when an existing issue isn't necessary.
Notes for the reviewer

I added a new explain.go file, because bundle is getting rather large and I think I'd eventually like to explode this command more and support like an "explain" command on parameters or outputs themselves, like print the JSON schema in a useful format so you could leverage that info when crafting parameters.

Checklist

[x] Unit Tests
[x] Documentation

Updated this to use our pkg/printer, new table format is thus:
$ porter explain --tag jeremyrickard/porter-do-bundle:v0.4.1
Name: spring-music
Description: Run the Spring Music Service on Kubernetes and Digital Ocean PostgreSQL
Version: 0.4.1

Credentials:
Name               Description                                       Required
do_access_token    Access Token for Digital Ocean Account            false
do_spaces_key      DO Spaces API Key                                 false
do_spaces_secret   DO Spaces API Secret                              false
kubeconfig         Kube config file with permissions to deploy app   false

Parameters:
Name            Description                                                     Type      Default             Required   Applies To
database_name   Name of database to create                                      string    jrrportertest       false      All Actions
helm_release    Helm release name                                               string    spring-music-helm   false      All Actions
namespace       Namespace to install Spring Music app                           string    default             false      All Actions
node_count      Number of database nodes                                        integer   1                   false      All Actions
porter-debug    Print debug information from Porter when executing the bundle   boolean   false               false      All Actions
region          Region to create Database and DO Space                          string    nyc3                false      All Actions
space_name      Name for DO Space                                               string    jrrportertest       false      All Actions

Outputs:
Name         Description                                Type     Applies To
service_ip   IP Address assigned to the Load Balancer   string   All Actions

No custom actions defined


Humanoid Avatar Rig Not Working

Describe the bug 💬
I can't apply the modification to the Humanoid character and create the Avatar element for it. It says that LeftUpperLeg is missing from the model, even though it is correctly configured on both the character and in AvatarUtils.
Error:

AvatarUtils:

GLTF containing L_Hip, which is interpreted as LeftUpperLeg in AvatarUtils:

Steps to reproduce 🔢

Import the GLTF model into Unity.
Verify that the bone names are correctly mapped in AvatarUtils.
Create the Avatar using the Avatar menu in the prefab generated by the GLTF Plugin.
Error: Missing LeftUpperLeg.

Files to reproduce the issue ♻
https://drive.google.com/drive/folders/1d72fK9oWq1OY7NH4RuqtHdd9p2FA4yz7?usp=sharing
Editor Version 🎲
6000.0
Render Pipeline and version
Built-In
UnityGLTF Version
2.14.1
Operating System 👩💻
Windows
When does this problem happen?

[X] Editor Import
[X] Runtime Import
[x] Editor Export
[x] Runtime Export

Additional Info 📜
This exact same error occurs for other users with different plugins, such as GLTFUtility, where I also reported this issue. I’m not sure, but it could be related to Unity and how it handles this:
https://github.com/Siccity/GLTFUtility/issues/70
Validations 🩹

[X] I have searched existing issues: no issue already exist that reports the same problem.
[X] I follow the Code of Conduct
[X] I provided a minimal reproducible example, including files when necessary.

Hey, the name of the node seems to be "L_Hip" but your mapping looks for "riotleague:L_Hip" which is a different string.
Can you try adding the actual name of the node without that prefix?

hey, I also did this test by modifying the bones directly in the GLTF file using the names used in Unity, but it still didn’t work

Can you upload those files and what error message you get as well please? Thanks

yes, are the files I sent mentioned in:
Files to reproduce the issue ♻
https://drive.google.com/drive/folders/1d72fK9oWq1OY7NH4RuqtHdd9p2FA4yz7?usp=sharing


Add support for the default Facebook webhook debug message

When you're looking at your app's webhooks (e.g. https://developers.facebook.com/apps/115612679446387/webhooks/ )
Facebook will try to send this message:
{
  "field": "messages",
  "value": {
    "page_id": "106728097004746"
  }
}

which we throw up on.
Instead, it should get swallowed with some sort of simple "ping" log message

actually, the full message is:
{
  "entry": [
    {
      "changes": [
        {
          "field": "messages",
          "value": {
            "page_id": "1067280970047460"
          }
        }
      ],
      "id": "0",
      "time":<PHONE_NUMBER>
    }
  ],
  "object": "page"
}


Incompatibility - LP Title replaced with backend function

@axelmohnen fyi i think you used the title function.
I deleted the set title function at the frontend:
https://github.com/abap2UI5/abap2UI5-connector_sap_fiori_launchpad/blob/f437eed2c18f2f9f58ed38d849a134e4655306cc/src/02/zabap2ui5.wapa.controller_-view1.controller.js#L12
But it can be set now on app level oin the backend with:
data(lt_params) = client->get( )-t_comp_params.
page->_z2ui5( )->lp_title(  lt_params[ n = `APP_TITLE` ]-v. ).

I changed it to stay consistent with the abap2UI5 approach: keep the frontend as small as possible and put all logic to the abap layer. Hope this works for you.
check out this for more launchpad infos:
https://www.linkedin.com/pulse/abap2ui5-host-your-apps-sap-fiori-launchpad-23-features-abap2ui5-upche/?trackingId=21HxoNP8IR4m1GCD35QzKA%3D%3D
Best regards.

@oblomov-dev . Thanks for the info. I saw already your new blog on LinkedIn. It sounds really good the new features!
I will test it as soon as we will switch to the latest ABAP2UI5 version.

great, i will close this one, just create a new issue if you run into problems.


Getting two responses for single slack command

I am building slack bot which basically takes kubectl command like kubectl get pods. Now, I want to send some message like Your request has been acknowledged immediately after any command is sent from slack and then the result of command. Currently, I have implemented the command result sending process but couldn't figure out how to send the "pre result message". Any idea? I tried implementing this way:
func response(command *bot.PassiveCmd) (string, error) {
   return fmt.Sprintf("Your request has been added to queue"), nil
}
func init() {
	bot.RegisterPassiveCommand(
					"",
					response)
}
func init() {
	bot.RegisterCommand(
 		"deployer",
 		"Kubectl Slack integration",
 		"",
	kubectl)

}

But the problem is when I enter slack message with @deployer get pods, then it triggers kubectl and when I just hit kubectl get pods, it triggers response. But I need to combine both. PS: Here, my cmdprefix is @. Hope you get it. If I am not clear, please feel free to ask on comment. Thanks.

If you are ok with using @deployer get pods then you can register it as a V3 command which allows you to send multiple messages over time by writing them to a channel.
Otherwise, we can extend the passive command framework to offer a V3 equivalent. I can look into that this weekend.

https://github.com/toolateforteddy/bot/blob/master/cmd.go#L175
Proposal for you. What do you think?

Sorry, but I am still not getting it. Here is how I tried:
func kubectl(command *bot.Cmd) (msg string, err error) {
.....
return fmt.Sprintf(okResponse, nickname, output), nil
}

func cmdV3(command *bot.PassiveCmd) (result bot.CmdResultV3, err error) {
	result = bot.CmdResultV3{
	Channel: "#channel",
	Message: make(chan string),
	Done:    make(chan bool)}
fmt.Println("hello")
return result, nil
}

func init() {
bot.RegisterPassiveCommandV2(
		"",
		cmdV3)
		bot.RegisterCommand(
		"deployer",
		"Kubectl Slack integration",
		"",
		kubectl)
}

@dwdraju if I understood correctly, the problem is that you are not writing on the Message channel.
for the V3 command to work properly you need to start a go routine to do the long running work and return the CmdResultV3 imediatelly, try something like this:
func cmdV3(command *bot.PassiveCmd) (result bot.CmdResultV3, err error) {
	result = bot.CmdResultV3{
        Channel: "#channel",
        Message: make(chan string),
        Done: make(chan bool)}

    result.Done <- false

    go func() {
        result.Message <- "Starting heavy work..."
        ... do your stuff
        result.Message <- "Heavy work done... time to grab a beer."
        result.Done <- true
    }()

    return result, nil
}

I did not test this code so the syntax might be wrong.

@fabioxgn, did that way and it works but can we add Args  []string on PassiveCmd as well like on Cmd. I was doing like this:
func kubectl(command *bot.Cmd) (msg string, err error) {
......
output := execute("kubectl", command.Args...)
......
}

And it was working well. Now, I tried like below for passive cmd:
func cmdV3(command *bot.PassiveCmd) (result bot.CmdResultV3, err error) {
.......
go func() {
		result.Message <- "ping"
		output := execute("kubectl", command.MessageData.Text)
		result.Message <- output
		result.Done <- true
	}()
......

Here is execute:

func execute(name string, arg ...string) string {
	rescueStdout := os.Stdout
	r, w, _ := os.Pipe()
	os.Stdout = w
	os.Stderr = w

	cmd := exec.Command(name, arg...)

	stdin, err := cmd.StdinPipe()

	if err != nil {
		fmt.Println(err)
	}

	cmd.Stdout = os.Stdout
	cmd.Stderr = os.Stderr

	if err = cmd.Start(); err != nil {
		fmt.Println("An error occured: ", err)
	}

	stdin.Close()
	cmd.Wait()

	w.Close()
	out, _ := ioutil.ReadAll(r)
	os.Stdout = rescueStdout

	return string(out)
}

Now, the problem is, as I am using string command.MessageData.Text, the cmd := exec.Command(name, arg...) is working like: kubectl 'get pods'(with single quote) if I pass get pods via slack. I think adding Args        []string could solve this.
I might sound off topic, but anyway help would be highly appreciated. Thanks.

@dwdraju why are you using command.MessageData.Text did you try using command.RawArgs like here?
https://github.com/go-chat-bot/plugins/blob/master/cmd/cmd.go#L32


Can't run successfully in my cpu only computer

I have followed the steps in this project but I got the following errors:

I tried to fix this problem by searching the answers in https://github.com/junyanz/pytorch-CycleGAN-and-pix2pix/issues/67 and https://github.com/junyanz/pytorch-CycleGAN-and-pix2pix/issues/360, but the error still exists.
So I really hope that someone could help me to figure it out. My computer is Windows 10 and only has CPU.

you can try  the following code so that unset gpu.
python train.py --dataset mnist --isize 32 --nc 1 --niter 15 --abnormal_class 9 --device cpu

you can try the following code so that unset gpu.
python train.py --dataset mnist --isize 32 --nc 1 --niter 15 --abnormal_class 9 --device cpu

This code actually works for me. Thanks a lot !!!


show map name/author during COTD

Just thought of this today - I think it should be fairly easy to implement just checking whether the gamemode is TM_COTDQualifications_Online or TM_KnockoutDaily_Online and then having the UI element at the correct height for each. Probably have a dropdown setting for showing during neither, one, the other, or both

Anything but simple, but I have it working. did you know there are 2 quali UI elements on different pages?
It's not so simple because they're all different sizes, at different positions, and monitoring whether they're closed or not is not as easy as copy-pasting.

Sorry I thought it would have been simple but thank you! I did not know there were 2 quali UI elements but I also didn't really look for them

It's okay. I refactored the logic to work for arbitrary UI elements. so that's an advantage. there's a lot of refactoring more I'd like to do in general tho (not required for this issue tho)

yep i noticed this yesterday too :/


create a warning or throw an error

https://github.com/yonathanF/Repair-Graph-Compression/blob/98b7026e6707fbe0f25830b249fd4a9e7f77b11f/graphs/graph_generator.py#L27

PR #43 resolves this by throwing an error. Failing silently is dangerous bc it could mess with the data when we do experiments.


[Docs] Adding CCPA under Compliance

GDPR and eiDas are currently listed and CCPA is an upcoming regulation (1st Jan) - quite important to the NA market. Are we planning to add this?

Moved: https://github.com/wso2/docs-is/issues/125


Update copy from source of truth doc

New text document by David for better source of truth for the text in the design
https://docs.google.com/document/d/1BsYWykVzI1hWE8nlzNruAJ7KAXG8UNboaBWQ3fOPhko/edit?pli=1#

Document up above. I am going to see if our content specialist Nayaab can give it another look through

Nayaab is going to do another pass, but if we can get a resync of copy that would be good. Any additional changes I or David can make. @signalnerve

Ready, but can't upload without being added to repo. under local branch raden-text-updates


Past Ops list, sends you to empty list when not inserting number

Past Ops list, will send you to an empty list if you click the go button at the bottom, when not inserting a number in the field.
A quick fix would be to default no input to page 1.

Merged fix

Deployed


Vickel: Watch +1-844-248-9739

Vickel requests the watch of the watch_number +1-844-248-9739. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

Approved by Jeff Schaller in Charcoal HQ


cifar10 speed numbers

In the comments of cifar10_train.py, it says:
Accuracy:
cifar10_train.py achieves ~86% accuracy after 100K steps (256 epochs of
data) as judged by cifar10_eval.py.

Speed: With batch_size 128.

System        | Step Time (sec/batch)  |     Accuracy
------------------------------------------------------------------
1 Tesla K20m  | 0.35-0.60              | ~86% at 60K steps  (5 hours)
1 Tesla K40m  | 0.25-0.35              | ~86% at 100K steps (4 hours)

The math doesn't look right to me:
5 x 60 x 60 / 60K=0.3 sec/batch
4 x 60 x 60 / 100K=0.144 sec/batch
Also, on my machine (K40c, 28 CPU cores, SSD) I can only reach about half the speed (~0.6 sec/batch). Any idea what might be the factor that slows it down? Thanks!

I bet these numbers (both ours and yours) are probably out dated -- re-open if there's still a problem with newer releases.


Cannot open any wsl paths in neither explorer.exe nor PowerShell

Windows build number:
10.0.18334.1
What I'm doing at what's happening:
Upgraded to version 1903 through the Windows Insider Program. Both Debian and Ubuntu were installed prior to the upgrade.
Start Ubuntu or Debian using any one of the commands below:
ubuntu
wsl.exe --distribution Ubuntu
debian
wsl.exe --distribution Debian

A terminal opens with the respective distribution. I leave these terminals running. To confirm the distros are running I run the following command from within cmd.exe:
C:\Users\sean>wsl --list --running
Windows Subsystem for Linux Distributions:
Ubuntu (Default)
Debian

In either of the Linux terminals I run the command below:
explorer.exe .

The command immediately returns but nothing else happens. I have also tried specifying other paths in the linux filesystems and get the same result. However, if I run:
cd /mnt/c
explorer.exe .

...an instance of File Explorer opens to c:\
In PowerShell, I run either of the following commands and get the results shown:
PS C:\Users\sean> cd \\wsl$\Ubuntu
cd : Cannot find path '\\wsl$\Ubuntu' because it does not exist.
At line:1 char:1
+ cd \\wsl$\Ubuntu
+ ~~~~~~~~~~~~~~~~
    + CategoryInfo          : ObjectNotFound: (\\wsl$\Ubuntu:String) [Set-Location], ItemNotFoundException
    + FullyQualifiedErrorId : PathNotFound,Microsoft.PowerShell.Commands.SetLocationCommand

PS C:\Users\sean> cd \\wsl$\Debian
cd : Cannot find path '\\wsl$\Debian' because it does not exist.
At line:1 char:1
+ cd \\wsl$\Debian
+ ~~~~~~~~~~~~~~~~
    + CategoryInfo          : ObjectNotFound: (\\wsl$\Debian:String) [Set-Location], ItemNotFoundException
    + FullyQualifiedErrorId : PathNotFound,Microsoft.PowerShell.Commands.SetLocationCommand

What's wrong / what should be happening instead:
When running explorer . when in a WSL folder, an instance of File Explorer should appear and be browsing the distribution's filesystem as per this blog article:
https://blogs.msdn.microsoft.com/commandline/2019/02/15/whats-new-for-wsl-in-windows-10-version-1903/
Also per the above blog, calling cd \\wsl$\<Distro>, I should get a directory listing.
I also installed SLES-12 after the upgrade to version 1903 and I got the same results as on Ubuntu and Debian.

That article is for Windows 10 insider build 18836 and above. You've to enable "skip ahead" feature in Settings.

That feature is for Windows 10 insider build 18836 and above. You've to enable "skip ahead" feature in Settings.

Ok. I had never enabled skip ahead before and it seems I may not have the option at this point. I tried following the instructions in this article but it doesn't seem to have had any affect (including after a restart). Is there some way to get access to skip ahead at this point?

You'd better wait for the feature to come to the 19H1(1903) branch.

Please note as of right now these features are available in the insider build 18836, which is a 20H1 branch. They will become available in a 19H1 branch build very soon! Since the features are already available to some users we decided to post the blog content early. :)
https://twitter.com/craigaloewen/status/1096522531714940928

Ok, thanks for the clarification. I'm now following @craigaloewen on Twitter too. Closing the issue.


Test on current Node.js versions

Drop the end-of-life versions of Node.js from the test matrix, and add the currently supported versions. (See: Node.js Release schedule)
Also update actions/checkout and actions/setup-node.

Thank you! I should also update the package json to require node >= 12

Hmm, wonder why it's failing on 14 and 16?

So do I! It kind of looks like a test environment problem; apparently it can't connect to the database. It's failing during the beforeEach step, where it tries to connect and set up the initial database environment: https://github.com/joeandaverde/tinypg/blob/e80896d761ab85e99bcb8ca7a4aff07ee890afb7/src/test/hooks.ts#L14-L24
On my machine, it seems to be failing to connect, even though I think I modified the connection string correctly for my environment: https://github.com/joeandaverde/tinypg/blob/e80896d761ab85e99bcb8ca7a4aff07ee890afb7/src/test/helper.ts#L8
I'm logging the client between construction and creation, and it sure looks like the relevant parameters are correct, so I'm not sure why it can't connect.
I also have no idea why that would differ between 12 and 14/16. Do you? If it failed for all three, I might worry about the fact that this PR came from a fork, but... why would that affect only the newer versions?

Oh, maybe it has to do with the devDependency on pg@7? Installing the most recent version of pg fixed it:
diff --git a/package.json b/package.json
index 8c7dc11..cc6f5b4 100644
--- a/package.json
+++ b/package.json
@@ -52,7 +52,7 @@
     "@types/uuid": "^8.3.0",
     "chai": "^3.4.1",
     "mocha": "^7.1.1",
-    "pg": "^7.18.2",
+    "pg": "^8.6.0",
     "rimraf": "^3.0.2",
     "source-map-support": "^0.5.19",
     "typescript": "^3.9.7"

Unfortunately, package-lock.json was written with a more recent version of npm than I have handy, so committing that change would also include this, which I don't think you want:
diff --git a/package-lock.json b/package-lock.json
index 6cd202f..8e21cd2 100644
--- a/package-lock.json
+++ b/package-lock.json
@@ -1,1799 +1,8 @@
 {
   "name": "tinypg",
   "version": "7.0.0-rc.1",
-  "lockfileVersion": 2,
+  "lockfileVersion": 1,
   "requires": true,

@joeandaverde, if you want to upgrade the dev dependency outside of this PR, I can rebase onto that; otherwise, I can see about upgrading npm next week.

Yep, I think that's it: https://github.com/brianc/node-postgres/issues/2170 was fixed in version 8.0.3 of pg.

thanks for investigating!

@jasonaowen has this worked out well for you?

@dynajoe it has! We've been using<EMAIL_ADDRESS>on Node.js 12 in production, testing with 12 & 14, and developing on 14, with no issues. I've also tested<EMAIL_ADDRESS>on Node.js 14 locally to verify the connection string changes, and again everything worked as hoped. :) Thank you!


Pay pattern confirmation in MWSS missing default routing rule

What is the context of this PR?
Schema validator wasn't working for routing rules properly, so a fix revealed invalid schema in 1_0005.json. All that was missing was a default routing rule which has been added.
How to review
MWSS should still work functionally as it was

yeah thats exactly it @nealedj it makes no difference apart from its technically an invalid schema

LGTM - squishy squashy


[firebase_messaging]Update firebase messaging dependencies to 17.3.4

Updating Android firebase dependencies to the latest version

@Vanethos Thanks for the PR, however the Android dependencies of the Firebase plugins should be updated together for now to avoid incompatibility. The Firebase team is working on this however for now we will update the FlutterFire Android dependencies together.


Intro of nitrestful a RESTful API generator

This tool generate a Nit module which implements Action::answer to redirect request to a static Nit method. It checks the presence of args, their types, deserialize objects as needed and calls the target method.
Missing arguments or arguments with errors (wrong type, failed deserialization, etc.) are replaced by null when the corresponding parameter is nullable. If the parameter is non-nullable, or if there is any other error, the generated code calls super from answer for the user code to handle exceptions and errors.
With the restful annotation we can write a normal method with static types such as: (from the example)
# User code
class MyAction
	super RestfulAction

	# Method answering requests like `foo?s=some_string&i=42&b=true`
	fun foo(s: String, i: Int, b: Bool): HttpResponse is restful do
		var resp = new HttpResponse(200)
		resp.body = "foo {s} {i} {b}"
		return resp
	end
...

And nitrestful will generate for use the wrapper extracting the args from the request and call the method foo:
# Generated code by `nitrestful`
redef class MyAction
	redef fun answer(request, truncated_uri)
	do
		var verbs = truncated_uri.split("/")
		if verbs.not_empty and verbs.first.is_empty then verbs.shift
		if verbs.length != 1 then return super
		var verb = verbs.first

		if verb == "foo" then
			var in_s = request.string_arg("s")
			var out_s = in_s

			var in_i = request.string_arg("i")
			var out_i = deserialize_arg(in_i)

			var in_b = request.string_arg("b")
			var out_b = deserialize_arg(in_b)

			if not out_s isa String or not out_i isa Int or not out_b isa Bool then
				return super
			end
			return foo(out_s, out_i, out_b)
		end
		return super
	end
end

This is an early version of this tool. More work is needed to test different types, different usages, error management, and improve the docs and examples.
close #1852

Close #1852

i really like the approach. +1 once the proto passes the tests

Fixed the tests and improve the doc, it should be ready for review and merge. I already have the next version with more options in the works.

+1

+1

please rebase first

More seriously, that's kinda cool :shipit:


microkernel

What is the difference between microkernel here in hypervisor and microkernel in MicroV?

There is no microkernel in MicroV. MicroV itself is an extension to the Bareflank hypervisor, so it is, in a sense, a user-space application that runs on the Bareflank microkernel. MicroV implements the VMM itself.


Refactoring AdaptiveTabs

The component should start using Tabs component from gravity-ui/uikit instead of own Tabs.
DATAUI-1524

I have refactored Tabs to be able to use them in AdaptiveTabs

I'm currently working on merging those two components here:
https://github.com/gravity-ui/rfc/issues/16
Can we close this task? @Marginy605

I'm currently working on merging those two components here: gravity-ui/rfc#16
Can we close this task? @Marginy605

Let''s close this issue after rfc will be done completely


Where's the rest of the documentation?

Like... the output of the API? It's returned several strings, including "LexicalFull", "ITNFull", "MaskedITNFull", and "DisplayFull". They are all very similar. Where is it documented what these things are and which I should use?

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

ID: 73fb7e76-fb80-c420-2efe-e3ca3a27b749
Version Independent ID: 07a42557-deb3-e4b3-3f14-f26a27be49c2
Content: How to use Batch Transcription - Speech Service - Azure Cognitive Services
Content Source: articles/cognitive-services/Speech-Service/batch-transcription.md
Service: cognitive-services
Sub-service: speech-service
GitHub Login: @PanosPeriorellis
Microsoft Alias: panosper

@RyanCantab
Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.
@PanosPeriorellis  Hi Panos, could you please check this issue and update the document if necessary? Thank you very much.

sorry for the delay, the docs have been updated.


bug from faucet

System information
OS & Version: Windows
Browser & Version: Chrome
Commit hash: when using checkout
Expected behavior
bug fix for faucet
Actual behavior
Steps to reproduce the behavior

@himunali Thanks for reporting this!
Do I understand it right that you didn't receive token from the faucet?

The faucet ran out of funds. Sorry for the inconvenience. It's fixed now.


Link to APK download from /apps/android

When we make releases of the Zulip mobile app, we upload them not only to Google's and Apple's app stores, but as an APK posted on a GitHub "release". These are appreciated by users who use Android without Google Play, whether out of privacy concerns or a desire to stick rigorously to free software.
As suggested in comments on zulip/zulip-mobile#1270, it'd be good to mention this fact where we advertise the Android app on the website, at /apps/android.
I just took a look at the code involved and determined this isn't a trivial matter of adding a <p> and an <a> to an HTML template, so I hope someone else will do it. :-)
Here's the outcome that I think would be good:

Let's link to https://github.com/zulip/zulip-mobile/releases/latest . That will always point to the latest version we've released to "production", excluding any releases still in alpha or beta.
A user who just wants "the Android app", and is moving quickly without reading, needs to be taken to the Play Store, just like they are now. (Users who want the APK will know they're looking for something specifically out of the ordinary, and will take a moment to read the choices. Conversely: most users won't really be aware that separately downloading an APK is a thing, and this option isn't for them.)
Specifically, I think this would work well:

The big green button stays as it is.
Below or next to it, add a link with the text "Advanced", or "APK (advanced)". (I don't 100% love that text; suggestions welcome.)
For a model for the style, I like our "Forgot your password?" and "Sign up" links on the login screen:

Separately:

The existing big green button can probably be improved by replacing it with a Google Play badge. Similarly an App Store badge for /apps/ios . See respective upstream marketing guidelines for details: Google, Apple.

Seems valuable.  @gnprice what do you think about just having the text "Or download the APK from GitHub" in smaller text below the button?  I'm not sure "Advanced" really adds much.
Hmm, the other thing to note which might argue for "Advanced" is that folks doing the download not through Google Play would presumably need to know they need to come back for updates.

I'm always wary when the word "advanced" comes up in design discussions. It's not specific and implies that whatever isn't "advanced" is "basic". I do think adding a sentence noting that users downloading the APK will need to manually check back for updates would be a good idea.

@gnprice  Hi, I would like to work on this, could you please provide me the link to this file on the code repository. I am quite lost.

templates/zerver/apps.html is the template; static/js/portico/landing-page.js is the JS code.

I have done it.

But since I didn't knew where the css file was located I have added the inline style for font size.
(I am really new to this project so please guide me.)

@zulipbot claim

@timabbott @gnprice I have opened a PR for this issue.
https://github.com/zulip/zulip/pull/11765

I merged a version of #11765 that I'm reasonably happy with; the big outstanding thing here is doing the play store replacements, which I encourage @borisyankov to just do since it should be reasonably quick (though a not instant, because of details around how we currently construct the button text).

Oops didn't mean to close this.

I guess this issue can be closed as we have a link to manually download APK from GitHub releases section. Or do we still need brand/platform badges in their respective buttons?
Screenshot:

Yes, this is still open since we want to use the standard app/play store buttons.  Might make sense to close this and open a new issue that's clearly just for that to declutter the story.

Yeah, agreed. Filed #14061; closing this one. Thanks again @theredcap!


Creating a catalog is not supported. Clarify HTTP response

Hi,
I spent some time to  create a catalog, until I noticed it wasn't supported. I think the response should be more explicit than "null".
This one line patch clarify the status of the API, but I'm wondering if a 501 status code would be better than a 415.
Thanks.

Thanks. We have combined both unsupported media type and operation not supported under 415. Probably because 501 was not one of the available Status enumerations in jsr311. We should separate out the status response.


Turn on raw-win-handle for most of the clippy checks.

Just doing the PR to test CI. This should fail for now, but succeed once the raw-win-handle fixes are in.

Great minds think alike :laughing:


[Website] Implement breadcrumb on all pages following Schema spec

Resolves #143 by implementing breadcrumbs on all public pages, following the Schema spec for breadcrumb element structure and attributes.
Closes #178 which implements breadcrumbs but does not fully follow the spec.

Conflicts resolved, rebased against latest master.