Consider switching to SocketsHttpHandler

Description
The two examples to configure the http client handler and to disable cookies create new instances of HttpClientHandler. Should these be switched to SocketsHttpHandler?
Page URL
https://learn.microsoft.com/en-us/aspnet/core/fundamentals/http-requests?view=aspnetcore-8.0#cookies
Content source URL
https://github.com/dotnet/AspNetCore.Docs/blob/main/aspnetcore/fundamentals/http-requests.md
Document ID
3d356334-e672-7e8b-1fef-7944ebee99eb
Article author
@stevejgordon
Related Issues

That request doesn't really make sense since HttpClientHandler is using SocketsHttpHandler internally already
https://github.com/dotnet/runtime/blob/main/src/libraries/System.Net.Http/src/System/Net/Http/HttpClientHandler.cs#L13-L29


Image Hover for Stream View

I record my sessions using Stream View and I wanted the images to show on the Stream View client when I hover in the DM screen, is there a way to do that?

Hi Conrad,
Currently the easiest way to do that would be to have a game master set a keybind in the image hover settings (show art to all users). Then as a game master hovering over a token and pressing that keybind. This would show the artwork to all users. Assuming I understand the stream view module from a quick read.


Support GraphQL logic-endpoint series tag(s)

Please answer these questions before submitting your issue.

Why do you submit this issue?
[ ] Question or discussion
[ ] Bug
[ ] Requirement
[ ] Feature or performance improvement

Question

What do you want to know?

Bug

Which version of SkyWalking, OS and JRE?

Which company or project?

What happen?
If possible, provide a way for reproducing the error. e.g. demo application, component version.

Requirement or improvement

Please describe about your requirements or improvement suggestions.

#5030


Optional sync_attributes?

Laravel Version: 5.4.28
Adldap2-Laravel Version: Current
PHP Version: 7.x

Description:
As a simple way of describing administrators in my application, we are using an unused field (otherpager) in our AD and setting it to 1. On all other users in the directory, it's empty.
This, however, causes issues when I specify it in my adldap_auth.php under sync_attributes, causing it to only import users with this value set (administrators), ignoring everyone else.
Is there an way to have optional sync_attributes which could, perhaps, default to a specified value if they are not provided by LDAP?

Hi @byanke-pcn,

This, however, causes issues when I specify it in my adldap_auth.php under sync_attributes, causing it to only import users with this value set (administrators), ignoring everyone else.

When you enter a sync attribute that might not exist on the users LDAP model, it should still import the account without issue.
Each attribute is retrieved using the getFirstAttribute method on the model:
https://github.com/Adldap2/Adldap2-Laravel/blob/master/src/Auth/Importer.php#L103-L115
https://github.com/Adldap2/Adldap2/blob/master/src/Models/Model.php#L289-L319
For this field your syncing on your local SQL table, is it nullable? Otherwise it will generate an exception trying to save the model.

Worst case, you can create an AttributeHandler to handle this situation:
https://github.com/Adldap2/Adldap2-Laravel/blob/master/docs/auth/syncing.md#attribute-handlers
Example:
namespace App\Handlers;

use App\User as EloquentUser;
use Adldap\Models\User as LdapUser;

class LdapAttributeHandler
{
    /**
     * Synchronizes ldap attributes to the specified model.
     *
     * @param LdapUser     $ldapUser
     * @param EloquentUser $eloquentUser
     *
     * @return void
     */
    public function handle(LdapUser $ldapUser, EloquentUser $eloquentUser)
    {
        $eloquentUser->name = $ldapUser->getCommonName();

        if ($pager = $ldapUser->getFirstAttribute('otherpager')) {
            $eloquentUser->otherpager = $pager;
        }
    }
}

Fantastic, thank you.
After some additional digging and thanks to your suggestion, I went a similar route: implementing in the user model Mutator.
// /app/user.php

class User extends Authenticatablec
{
// [...]
  public function setIsadminAttribute($value)
   {

       // This is required to properly handle null/empty values from AD
       if(is_null($value) || $value != 1) {
         $value = 0;
       }

       $this->attributes['isadmin'] = $value;
   }
// [...]
}

Great! No problem at all :).


Feature/#21 range component

_ No description provided. _

hi @Shinji22 I have already tested and your component is amazing! thanks for the contribution
before merging can you please rebase your git history?

Hi @vczb,
Glad you like it :)
Sorry I messed up with my branch...
It should be OK now


[TypeScript][Lib] Remove TSLint configuration

Purpose
What is the context of this pull request? Why is it being done?
Changes
Are there any changes that need to be called out as significant or particularly difficult to grasp? (Include illustrative screenshots for context if applicable.)
Tests
Is this covered by existing tests or new ones? If no, why not?
Feature Plan
Are there any remaining steps or dependencies before this issue can be fully resolved? If so, describe and link to any relevant pull requests or issues.
Checklist
General

[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have added or updated the appropriate tests
[ ] I have updated related documentation

Bots

[ ] I have validated that new and updated responses use appropriate Speak and InputHint properties to ensure a high-quality speech-first experience
[ ] I have replicated language model changes across the English, French, Italian, German, Spanish, and Chinese .lu files and validated that deployment is successful

Deployment Scripts

[ ] I have replicated my changes in the Virtual Assistant Template and Sample projects
[ ] I have replicated my changes in the Skill Template and Sample projects

FYI @Ezecriis - I updated the package.json and npm-shrinkwrap file to remove the TSLint dependencies. Please check those changes.
Also, can you point this PR to next branch?
LGTM 😎


Shouldn't this just compile, or am I missing something?

Here's what I get(see pic)

The app is tokenized. Before building the solution you must configure the tokens with the right values in the AppSettings.cs file.
You should deploy the backend services in your azure subscription and get the patientId and tenantId values. In the wiki you can find information about how to deploy the ASPNET5 app.

Thanks.


Add supprt for request.session cookies or passing referer

I have a url. curl -H "Referer: Url" Link
How to implement it in pysmartdl
tried
request_args = {"headers": {"Referer": "url", "User-Agent": "Mozilla/5.0"}}
but not working says 410 gone. same error when curl is run without referer

when i normally run session.get(link) it download file but with pysmartDL IT SAYS 410 gone


Early loop detector: dependency loop remover from cache

Related to: https://bugs.gentoo.org/show_bug.cgi?id=391945
This detector just calculates loops, solves weak loops (with at least one use or after dependency), and print information about all found loops (including unsolvable).
Solving is just breaking the loop with removing one or few use/after dependencies.
This detector cannot solve loops fully based on "need" dependencies. To prevent hang up in this case, it's need to use the later loop detector (https://github.com/OpenRC/openrc/pull/13) or solve it manually (sysadmin will be notified about any loop).
Presentation in PDF: https://github.com/xaionaro/documentation/raw/master/openrc/earlyloopdetector/early-loop-detection.pdf

@xaionaro:
Can you meet with myself or @dwfreed sometime on irc to chat again about this code? I'm not sure where we left it. We both agree that there is a problem we need to solve, so let's get together again and chat about it. :-)

@xaionaro, Hey Dimitry, long time no see.  MIssed you a lot.  Glad to tell that we have OpenRC 0.18.3 in Debian.  Your updates has resolved all the build issues on ppc64el and kFreeBSD.
@williamh, William, sorry for being idle for long.  The point of breaking a loop is not only for Debian, but to make OpenRC more robust against configuration mistakes.  Such a mistake as dependency loop can be fetal (unable to boot).  Dimitry has made the point that the loop solver breaks the weakest dependence in the loop, which makes sense.
The only concern for me is the code complexity, which I think, yes, it should be addressed cooperatively.

Hello. Sorry that I still didn't fix the coding style of the patch as @williamh requested long time ago. I just have no time. It's quite bad years in my university right now (partly due to geopolitical problems), so I have to work a lot and more. :(

I suppose we should do something with this pull request.

I'm against trying to remove loops behind the sceens; we have no way of knowing for sure what the relationships of the services should be. I would rather just warn about them and allow users to fix them.

I would rather just warn about them and allow users to fix them.

For me it's ok but I worry about "rc_parallel=no" by default. It's not normal for modern init system.
OpenRC must push users to write strict and right rules.

I can add an option to not break loops automatically, today if required.
On the on hand this patched was written for Debian and it works there well enough, so as for me mission accomplished. On the another hand, I'd like to be able to say people that OpenRC can safely boot in parallel mode (and do not hang), so I'd like to see this patched be merged to the upstream code.

I can add an option to not break loops automatically, today if required.

At least it's good functional. But I don't know what thinking @williamh about it.

we have no way of knowing for sure what the relationships of the services should be.

If init system don't know about relationships then who knows? My main goal enables parallel by default.
What should we do for that?

@xaionaro I don't want to break loops at all, so please just have this patch print error messages if loops are detected; I think loops indicate bugs which should be fixed by the init script authors. Also, please rebase the patch on current master. I'm definitely interested in this, so I'll take a look once it is updated.
@stalkerg What I meant above is, if we get into a deadlock situation, there is not really a smart way to resolve it automatically. We don't really know what services do, so we can't really decide which service to start first to break the deadlock. I think it is a better choice to warn that there is a loop when we are building the deptree but let the script author take care of it.

Maybe it's destiny but I have catch deadlock with parallel=on after upgrade to 0.24.

I am uncomfortable with this pr because we should warn admins about loops and make them resolve the loops themselves rather than randomly drop dependencies. If you want to do that, feel free to update this and re-open it. Also, please avoid breaking the API. Thanks much.


SIMD optimization

.NETCore already has hardware accelerated types (System.Numerics.Vectors) which should work cross platform - I believe functions like Salsa will be able to take advantage of that, correct me if I am wrong, I am currently looking to optimize it once I get time.
Also .NETCore has a using System.Runtime.CompilerServices package for unsafe code which allows copying bytes.

That would be cool, I'm curious to see how fast it is.
Let me know if you find out something.

Any progress? I really need this optimization.

I have no plans to implement that, but a PR is very welcome.


Add Solar weather data

A user on Twitter requested an addition to the weather app. This should add details on space weather to support ham radio enthusiasts. https://twitter.com/AndreiBulu/status/1690533688667512832
Aim : New feature pulls solar weather data from xml files hosted at https://www.hamqsl.com/solarxml.php

Need to check the terms of service there before using this code, but the XML format provided looks like it is designed for being downloaded to scrapers in this way.

Could either add this as a second page of info, or as a standalone.

Interesting idea to add a chart of atmospheric conditions like this one https://prop.kc2g.com/fof2/

Created a new app “space”  which adds lots of space weather data.


Documentation Request for Episode Format

Hi, what are the semantic meanings of the fields in the episode JSON files at the HuggingFace OVMM dataset page?
For instance, target goals seem to be defined as a list of 4-tuples, and I wasn’t able to understand if they refer to xyz coordinates and a single angle w along the z-axis, or something else.
"targets": {
        "Olive_Kids_Dinosaur_Land_Lunch_Box_:0000": [
            [
                0.70621,
                0,
                -0.708,
                -15.78765
            ],
            [
                0,
                1,
                0,
                0.87444
            ],
            [
                0.708,
                0,
                0.70621,
                0.66923
            ],
            [
                0,
                0,
                0,
                1
            ]
        ]
    },

Thanks for raising this!
This targets field is not used in Ovmm task. It is used for specifying a target goal (target transformation for the object) in the GeoGoal rearrange task.
Added an initial description of the OVMM task-relevant fields here. This will be visible on the huggingface dataset page soon.

Sounds good, thanks for the updated documentation @yvsriram!
As a follow-up, and apologies about digressing a bit from the OVMM challenge -
Is there a recommended way to use episode generation code in the habitat-lab repo to generate longer horizon tasks that might involve open/close actions or a sequential sequence of pick/place actions, like decluttering/tidying a living room?
(I realize this would be a departure from the specification for OVMMDatasetV0 class, but in general looking for how to perform more complex task sequences.)

For multiple pick/place interactions: You can look at the configs from hab2 tasks. You will have to modify this script for retrieving categories of multiple goals.

For open/close actions: Currently OVMM uses HSSD scenes, which do not have articulated receptacles. So open/close actions cannot be performed on them.
But you can create episodes using ReplicaCAD scenes: Look at rearrange.yaml here. rearrange_easy.yaml is the one which does not have open/close interactions.

I think in the fall we're hoping to add articulations, but yeah not available yet.
We can also continue this discussion on Slack @prasoonvarshney

So we're probably not going to add docs to the episode setup as it currently is, will hopefully have a better version soon. Closing the issue


dataportals.org

make sure we're not missing anything from http://dataportals.org

of the 15 links pertain to (Vancover BC, Oregon and Washington), the first 10 are not in nwdata.org. I can add first nine under British Columbia section, and the global health under Organizations.  Please advice

Qualicum Beach, British Columbia
URL: http://data.civicinfo.ca/organization/qualicum-beach-town

Nanaimo, British Columbia
URL: http://data.nanaimo.ca/

Township of Langley, British Columbia
URL:http://www.tol.ca/ServicesContact/OpenData/OpenDataCatalogue.aspx

Chilliwack, British Columbia
URL: http://www.chilliwack.com/main/page.cfm?id=2329

Stanch, British Columbia
URL: http://www.saanich.ca/data/catalogue/index.php

Victoria, British Columbia
URL: http://www.victoria.ca/EN/main/city/open-data-catalogue.html

District of North Vancouver, British Columbia
URL: http://geoweb.dnv.org/data/

Vancouver, British Columbia
URL: http://data.vancouver.ca/datacatalogue/index.htm

Surrey, British Columbia
URL: http://www.surrey.ca/city-services/658.aspx

GHDx- Global Health Data Exchange
URL: http://ghdx.org

Civic Apps for Greater Portland
URL: http://www.civicapps.org/datasets

Oregon
URL: http://data.oregon.gov/

Washington
URL: http://data.wa.gov/

Seattle
URL: http://data.seattle.gov/

King County, WA
URL: http://www.datakc.org/


adding latest tag to per commit image builds

What type of PR is this?

Uncomment only one  /kind <> line, hit enter to put that in a new line, and remove leading whitespaces from that line:

/kind bug

/kind cleanup
/kind design
/kind documentation
/kind failing-test
/kind feature
/kind flake

What this PR does / why we need it:
We need 'latest' tag on every image produced
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

/approve

/approve
/lgtm


set EnableNatGateway boolean to true for Azure, AzureLite and AzureHA for upgrades

As part of enabling NAT Gateway for Azure clusters, this PR sets the EnableNatGateway boolean to true for Azure, AzureLite, and AzureHA plans.
See for more information: https://github.com/kyma-project/kyma/issues/13553
When a partial payload is provided, Provisioner gets the missing information from the database with previously used data, and therefore only values that are different are changed and only providing EnableNatGateway should be sufficient.

/retest pre-control-plane-components-kyma-environment-broker

/retest

We MUST NOT reconfigure existing cluster to use NAT gateway as it would involve change of the outbound IPs used for the exiting runtimes. Who requested this change?

https://github.com/kyma-project/kyma/issues/13553#issuecomment-1167053459


Create exporter based on agent endpoint configuration

See https://github.com/owncloud/ocis/issues/2453

needs changelog


Add renovate

Configure renovate to manage dependencies automatically.

[ ] merge this PR.
[ ] go to  https://github.com/marketplace/renovate to get a "free for OS" plan of renovate

What it will do?
see my initial PR: https://github.com/MichaelSp/docker-mailserver-helm/pull/1

What to Expect
With your current configuration, Renovate will create 8 Pull Requests:
Update Helm release kubernetes-ingress to v1.22.4
Update dependency mailserver/docker-mailserver to v11.1.0
Update helm/chart-releaser-action action to v1.4.0
Update helm/chart-testing-action action to v2.2.1
Update helm/kind-action action to v1.3.0
Update actions/checkout action to v3
Update actions/setup-python action to v4
Update azure/setup-helm action to v2

I like pulling the image tag from Chart.AppVersion so it only has to be updated in one place. So merging this (there was already a renovate.json file, just not fleshed out).


ETA for Fedora

Sorry, if this has been mentioned already soemwhere, but is there an rough ETA on Fedora? And will it be available for normal Windows FCU or only in insider?

When Fedora is ready it will be available on Fall Creators Update.

Se this comment: https://github.com/Microsoft/BashOnWindows/issues/8#issuecomment-212116902

Fedora is not available in the Fall Creators Update.  I too have searched far and wide.  I have come up empty on any official wording from the Fedora Community.

Fedora seems to be advertised as being available when searching for it on the Microsoft Store, and yet it's not actually there. I imagine it must be coming soon if it's mentioned on the store.

It's been a month or so, any update?

@cwawak I put the question in the Fedora forums under an already existing thread called 'Fedora on the Windows Store' . Still waiting for official response. Feel free to check it out along with the other forums as we'll need to know what issues arise specific to Fedora when it is finally available. This will help ensure the WSL team can focus on all the big things we the community have been pinging them on.

Why not you install Fedora from rootfs tarballs (tar.gz)? Those really works in WSL!

@Biswa96  I'm looking for an officially supported version to be released so any issues I have can be officially supported either here or by Fedora. The same will go for any other distro that gets released into the Store. Since 2 of my target platforms are CentOS and RHEL, having Fedora officially available would be immensely helpful and make things much easier instead of making conversions from Ubuntu.

Yes, the docker rootfs tarball provided in fedora official website: https://download.fedoraproject.org/pub/fedora/linux/releases/ and the procedure is as similar as Windows store provided distros. See my repo.

Is it just me, or does it seem really odd that Microsoft advertises Fedora so prominently on the Microsoft Store, when it still doesn't exist?

@albeauregard No, you're not the only one. If you saw this article New distros coming to Bash/WSL via Windows Store, in that image @bitcrazed showed Fedora in build 10.0.16188. So, Fedora may be available for the MS programmers. 🤫

@All: As per Fedora Lead @Mattdm (on Twitter):

We're working on resolving some non-technical issues. I'm afraid I don't have any more than that right now.

Source:

Please do ping Matt and ask him for updates - he uses your asks as ammo' ;)

I don't use Twitter but if somebody here does and pings him on this, you can add a +1 from me for getting this to the Store or at the least the offline .appx that was shown (barely visible) on the Flock 2017 Youtube video.

Yeah, it's odd that it's been three months and the store still lists Fedora even though it's not actually available.

As I suggested above, the ball is in Fedora's court.  I suggest letting them know this is important to you.

Is there any suggested channel for people without twitter?

@MikeGitb - https://fedoraproject.org/wiki/Contact

@mattdm is this something you can update us on?

@benhillis, can you comment on above? Apparently ball is back in your court?

@bitcrazed @tara-raj are more up to date on specifics.

Should this ticket be closed? Just conscious that it's still being updated and I'm acutely interested in seeing this happen (I'm trying not to grow attached to WSL Ubuntu since I'll be dropping it as soon as this lands).

You do realize you can run both of them right? You may even find yourself in a situation in which you'd want to run more than one distro for testing out scenarios.

I did not know that, that is useful information though thank you. I'd typically only use a single distro though - usually I would just be happy with any Linux terminal but a major project that I work on also requires rpmbuild / mock setup for generating CentOS rpms which is more of a red hat thing so having fedora on WSL would be really handy.

Trust me, I know your pain. Considering I have CentOS and RHEL in my environment makes this a requirement. If you haven't tried to though, you can install rpm on Ubuntu (and yum as well oddly enough). So you don't have to toss the syntax on the back burner entirely. I do have to admit I prefer the apt method with Ubuntu (syntax is more user friendly but that could just be my view point) but Ubuntu isn't entirely ideal as I already said. I would keep Ubuntu installed for one of those "just in case" situations. Making something that can work on both would be more ideal because it can then be shared more easily if it's not something proprietary.

Hey all - back in the office after a few days OOF-sick.
Appreciate all the enthusiasm re. Fedora and thanks for your patience while we work through a few things.
Rest assured that we're keenly working to resolve the blocking issues and will do our utmost to get Matt & team unblocked and able to get Fedora into the store.
When we do, you'll hear A LOT of noise on Twitter from @Mattdm, @tara-raj and I, and on our blog at https://aka.ms/cliblog.
Reopening for now to reflect the issue's actual status, but until it's closed, assume that we're working hard to move forward.

Wow there are a lot of great things on that blog post list you guys really are making Windows usable for people like me thank you! On the fedora thing I'll wait with bated breath...

Okay, seriously, what's going on with this? What's the hold up for Fedora on WSL now?
I just saw Debian enter the fold, and Fedora was a "launch" partner for the initial multi-distro effort for WSL. How is it that it's still not available?

Debian? I just checked and I don't see Debian available. Kali Linux is now available though! It does make one wonder if it literally took 2 months for Kali to come to the Store why Fedora has been so much longer. Really puts the emphasis on "non-technical issues"

@DarthSpock It was just announced on the blog...

It's now available on the Store and I see the blog. That's crazy. Kali was released yesterday and now Debian! We're on a roll here. C'mon Fedora, let's keep the wheels moving.

I wonder if we will ever learn what exactly the holdup was about.

There's nothing super-dramatic; it's just slow to get the legal details worked out. That progress is continuing. I'll let you know if I give up. :)

Thanks @mattdm

The Microsoft Store's splash for WSL has me a bit concerned:

That real estate used to be occupied by Fedora!

@mmmmmmbeer - We can't win...
Fedora is in the list of available distros - "WHERE IS FEDORA"
We remove Fedora from the list - "WHY DID YOU REMOVE FEDORA FROM YOUR LIST"

@benhillis I think it’s safe to assume that @mmmmmmbeer’s comment was solely directed at the Fedora side of this issue.

I just wanted to ensure it didn't portend some sort of falling-out between the two parties! Hope to see Fedora (or any RPM-based distro, for that matter) available soon!

My non-technical contribution to this discussion is to point out that SUSE is rpm-based.
I am anxiously awaiting Fedora, and I hope these issues get sorted out sooner than later.

Still no updates about Fedora coming to the store? Seeing all these distro's coming to the store it does seem like a long wait for fedora. I really hope these "legal details" aren't starting to become something major.

We are making progress.

@mattdm will the appx be open source like the debian did?

Everything Fedora does is always open source / free software.

@mmmmmmbeer - absolutely no falling-out or anything else worthy of daytime soap-opera drama. Though it's quite likely I owe @mattdm at least a couple of beers - hope he doesn't hold it against me :)
We're just working-through some paperwork stuff while various parties shipping products, being in and out of the office attending events etc. eating into available time.
Again, please bear with us - we're working on this as quickly as we can.

Thanks for the updates @bitcrazed and @mattdm. Really looking forward to seeing this go live.

@calebmeyer - When the Fedora store is in the store it will be available in the current stable build (Fall Creators Update).

any updates? really looking forward to have fedora on my laptop (which I require to run with windows10)

@sagyvolkov Nothing to announce yet, but please do follow @tara-raj (tara_msft) and myself (richturn_ms) on Twitter and/or keep an eye on our blog - we'll be quite noisy about Fedora when it lands ;)

Is the open sourcing of the WSL sample at all related to the non-technical details required to get Fedora released for WSL? https://blogs.msdn.microsoft.com/commandline/2018/03/26/wsl-distro-launcher/

Looking forward to Fedora on Windows!

Just my opinion in form of a reality check: this very long delay in releasing fedora on WSL store is passing the point where "we are working on non-technical/legal issues" is cutting it.  You might at least want to give more details if you expect the community to stay with you. Again, just my opinion, no offense. Personally I'm very excited for the day I can use fedora in my day to day development activities.

Agreed. It would seem to be in the spirit of the Fedora community to have more transparency as to what is going on here.

Again, echoing what I said above:
Appreciate all the enthusiasm re. Fedora and thanks for your patience while we work through a few things.
Rest assured that we're keenly working to resolve the last remaining non-technical blockers.
When we do, you'll hear A LOT of noise on Twitter from @mattdm, @tara-raj and I, and on our blog at https://aka.ms/cliblog.

Rest assured that we're keenly working to resolve the last remaining non-technical blockers.

What are the remaining "non-technical" blockers?

Just boring paperworkey-things.

We hope the paper work can be sorted out soon!
After all, it’s been 6 months and the rest of the distros are out! Missing only Fedora!

We hear ya: We've all been eager to see Fedora in the store and on WSL since we announced it!!
But doing the Corp' stuff properly is important for everyone. Alas, that can sometimes take time.
Thanks for your continued patience.

I guess the thing everyone is wondering is what makes fedora's paperwork so different from the other distros?
I'm certainly not asking to reveal  any details about ongoing negotiations (or whatever you are doing right know), but will we at least see at the end, what kind of legal Agreement (If any) was reached between MS and Fedora?

@mattdm can we get an update from your side of things?

@MikeGitb I think something everyone here needs to realize is Fedora (and CentOS) fall under the Red Hat umbrella (and have Red Hat employees actively contributing to both projects). The paperwork required is most likely between Red Hat and Microsoft, which as you see not surprisingly takes a decent bit of time (think bureaucracy). I wouldn't be surprised to see after all is said and done, we get not only Fedora but CentOS and RHEL available in the Store, which would be great. At least this is my hope.

Why not you guys suppress your appetite temporarily with docker images? Both Fedora and CentOS docker images can be installed in WSL.

I'll check with RH legal about what they feel comfortable with me saying — I don't want to cause more difficulty. As @DarthSpock notes, Red Hat is both the primary sponsor and trademark holder for Fedora, and is the entity which takes on legal risk. (And yes, it's true that if we resolve this for for Fedora, that will clear the runway for CentOS.)

@mattdm would RHEL be paved open as well? OpenSuse has their Enterprise version available, so it would make logical sense for Red Hat to do the same. They offer a free one-year license subscription for use on WSL (probably because WSL has limitations real Linux doesn't have). It would great to see Red Hat offering the same.

@DarthSpock Ah, here we are in clear territory, because I can definitively say that I have absolutely no ability to say anything whatsoever about RHEL plans. :)

Gotcha, hope something is in the works.

@Biswa96 Unfortunately that doesn't work...ish. Currently WSL distributions are installed like any  windows store application, and on file system get installed in Packages under %APPDATA% in its own directory. (This is why the distro switcher on GitHub doesn't work anymore by the way.) To do what you suggest would mean to install one of the other distros as a dummy package, replace its files with those of a Fedora Docker image, handle some user issues (passwd/shadow/group). While this is doable (I did it), you will end up with a very limited Fedora setup which doesnt even have sudo, has problems with networking (which means you cant just dnf install stuff) etc.  So... doesn't work... ish.

@mattdm Thank you very much for the RH legal trip, it would really help to know at least in part of what is going on.

@tgoncuoglu Biswa is referring to https://github.com/Microsoft/WSL-distrolauncher/ which is the official method of side-loading distros. Is this what you used? It was only open-sourced quite recently.

@DarthSpock I believe he was referring to https://github.com/RoliSoft/WSL-Distribution-Switcher , which automated the process I described above with docker images, but yes, you are right, I should check that one too. Thank you.

While I'm looking forward to see fedora on Windows I don't care that much about which distro I'm running in WSL anyway, as it is neither my desktop, nor server OS and most Linux tools work on any distro anyway.

@MikeGitb While I agree in general, I know that there are often differences between distro's when it comes to installing dependencies, and configuring things, and that its generally less error prone to build packages, etc. on the platforms you aim to deploy onto.

Any updates? 😃

@mattdm did RH legal get back to you?

F28 out now, would be super awesome to play with it in WSL. Any update on the legal ballyhoo?

Also waiting eagerly at the moment. No more XAMPP for my local development needs.

fedora should be have been done a long time ago. even second right after ubuntu and then all the other...

@hkrware The thumbs down could be because your commend adds absolutely nothing to the discussion.
(of course, now I'm equally guilty)

Anybody on Twitter want to ping @mattdm on the reply from legal? @bitcrazed anything new from your end?

I'm following this. I promise updates when I have something I can share.

Is the re any shell script to make Fedora tar.gz file? Like this from Debian by @rhaist: https://salsa.debian.org/rhaist-guest/WSL/blob/master/create-targz.sh

@Biswa96  if you've got cygwin, you can use the tar utility:
tar -xf fedora-27-x86_64-20180307.tar.xz -C outdir/ cd outdir tar -czf ../fedora-27-x86_64-20180307.tar.gz --hard-dereference *

I know that. I've posted that process before. But many things don't work with that docker image.

Can't wait for this to land so I can have easier access to mingw tooling.

@Biswa96 If the Docker image doesn't work so well, have you found an image that doesn't have any issues?

@DarthSpock I was thinking of to make a tar.gz from Fedora ISO (I've done that with Antergos, CentOS). But it is 3.6GB (including Desktop Env). So I asked if there is a script to make a workable tar.gz for WSL.

Sounds like we'd have to strip down the Docker image (or whatever image is the smallest) to as bare-bones as possible just like Kali does it. We could always create a script from there to install anything that was stripped.

@Biswa96 Was CentOS a successful attempt?

@Biswa96 Oh, and did you try unxz the tar.xz into a tar and then gzip ?

I installed Fedora successfully 💯. Don't ask me that again. And that procedure works for CentOS well.

You asked for a script, I tried to help you. You don't need to be rude. If you don't like it, GitHub is not the place to take it out. Everyone can ask whatever they want.

Sorry, if this has been mentioned already soemwhere, but is there an rough ETA on Fedora?

No.

So why close this now? 🤔

Most likely because now it's a RedHat issue, not a WSL issue.

Most likely because now it's a RedHat issue, not a WSL issue.

Has something changed?

(which disregards the first two sentences of CONTRIBUTING.md)
Sorry for that.

was simply answered is all. There is no ETA right now.
Sad, but good enough for me (actually,I was fine with it being closed back in December 2017).

Yawn 🌵

Hello, check this: https://github.com/WhitewaterFoundry/WSLFedoraRemix


IE Horizontal Scrollbar covers last row.

I have autoheight set to true (which I need because I need all DOM elements).  When there is a horizontal scrollbar it covers the last row.  I implemented the fix from 414 which fixed firefox... but I still have the problem in all versions of IE.  Is this a known problem or should it work?  Thank you.

if you add the option enableAddRow an extra blank row will be added to the bottom

this repo is dead. use https://github.com/6pac/SlickGrid


Pre-release for v 1.4.0

This PR adds functionality to correct and update grant data in the PASS repository to conform with new semantics as to what these data represent. The earlier semantics reflected only the latest iteration of a multi-award grant. In response to user feedback, we are adopting a different view of the semantics of some fields on the PASS grant object: fields like startDate awardDate, grantNumber and name, which might change for different iterations of a grant, will be tied to the initial award. endDate is the exception - it will apply to the latest  iteration. In addition, any investigator who was a pi or co-pi on any iteration will be reflected in the co-pi list, except for the latest pi.
The correction for existing data may be accomplished by performing a pull and load with a start date the same as for the initial pull and load for the existing PASS data , with the -init option specified. this option directs the loader to overwrite fields in the PASS data which would not be overwritten in a load without this option. This is necessary because with the correction applied, some fields in the PASS data will be considered more authorittive than incremental update pulls, as update pulls will not aggregate the historical data needed to reflect the new semantics.

@jrmartino i tested locally and the build passed, but Travis seems unhappy...

@emetsger travis always fails before merging because encrypted credentials are not available to forks - i expect it to pass once the PR is merged (famous lst words)

Updated this PR for release 1.4.0


Automated translation updating

Extract translations from the spreadsheet and apply to relevant files.
Follows and automates all steps from the HOWTO doc explaining updating translations.

extracts the Legal and Main sheets first and third columns
runs auto-translate.xsl against the jurisdiction/survey translation
runs auto-translate.xsl against the main/site translation
runs surveyor.xsl to create the surveyor translation .yml and .rb files

Coverage decreased (-0.89%) to 79.89% when pulling c8c038bb54a5bfe9492939cbbb22db9953839c98 on feature-cz-translation into 719996ea9da325f531d8c631c19f78aaf38a3b96 on master.

Coverage decreased (-0.89%) to 79.89% when pulling c35265a782f1560c2db2e48b7bbd886a75dab744 on feature-cz-translation into 719996ea9da325f531d8c631c19f78aaf38a3b96 on master.

Coverage decreased (-0.96%) to 79.82% when pulling 0a1dda07d4327f19b1017e485234df7dd02a15a5 on feature-cz-translation into 719996ea9da325f531d8c631c19f78aaf38a3b96 on master.

Coverage decreased (-0.96%) to 79.82% when pulling 90417352d7d740bf18321a244368650af01f2743 on feature-cz-translation into 719996ea9da325f531d8c631c19f78aaf38a3b96 on master.

Coverage decreased (-0.96%) to 79.82% when pulling b64e5e7c1c1fc2b9e3894c012a6922e36cadf199 on feature-cz-translation into 719996ea9da325f531d8c631c19f78aaf38a3b96 on master.

Coverage decreased (-0.96%) to 79.82% when pulling b64e5e7c1c1fc2b9e3894c012a6922e36cadf199 on feature-cz-translation into 719996ea9da325f531d8c631c19f78aaf38a3b96 on master.

Coverage decreased (-1.0%) to 79.78% when pulling 51cecdbc4e0898d5f3d61a7628a95c4abe2e6d20 on feature-cz-translation into 719996ea9da325f531d8c631c19f78aaf38a3b96 on master.

Coverage decreased (-1.0%) to 79.78% when pulling b9952b34be4d674dd00b2fc431afc1a9c5be0504 on feature-cz-translation into 719996ea9da325f531d8c631c19f78aaf38a3b96 on master.


"Folder" image will continue to find different file size + Pixiv Sketch error

Prerequisites

[x] Did you read FAQ section in readme.md?
[x] Did you test with the latest releases or commit ?
[x] Did you search for existing issues in Issues?

Description
Getting images from this user (65619146) will result in a few problems. If you try to download multiple times, it will constantly redownload the "folder" image because of a file size issue.
Note: After running it about 3 times, the 4th time finally said the file size was accurate.
Also, if you select "y" to include Pixiv Sketch, you get this error:

Processing Artist Id: user_wxgx2788 for PixivSketch
Traceback (most recent call last):
File "PixivSketchHandler.pyc", line 46, in process_sketch_artists
File "PixivBrowserFactory.pyc", line 1054, in sketch_get_posts_by_artist_id
File "PixivBrowserFactory.pyc", line 1092, in getPixivSketchPage
File "PixivBrowserFactory.pyc", line 209, in open_with_retry
File "mechanize_mechanize.pyc", line 257, in open
File "mechanize_mechanize.pyc", line 313, in _mech_open
mechanize._response.get_seek_wrapper_class..httperror_seek_wrapper: HTTP Error 404: Not Found
Error at process_sketch_artists(): user_wxgx2788
Exception: (<class 'mechanize._response.get_seek_wrapper_class..httperror_seek_wrapper'>, <httperror_seek_wrapper (urllib.error.HTTPError instance) at 0xf96a90 whose wrapped object = <HTTPError 404: 'Not Found'>>, <traceback object at 0x04EEC7E8>)

Steps to Reproduce

Choose option 1 with user 65619146. Select "y" to include Pixiv Sketch
When it finishes. Run the same options again and the "folder" image will redownload.
Run it as many times as you want and it will make a new "folder" image each time.

Expected behavior: "Folder" image should only download once.
Actual behavior: "Folder" image downloads a new image each time.
Log file: [Attach the pixivutil.log file in the application folder, recommended to delete the old file, reproduce the issue, and upload the newly generated file here]
Versions
You can get this information from executing PixivUtil2.py --help.
Latest version available in https://github.com/Nandaka/PixivUtil2/releases

either the downloaded folder.jpg is corrupted so it try to redownload again, or you can set alwaysCheckFileSize = False in config.ini
- alwaysCheckFileSize

Check the file size, if different then it will be downloaded again, set False to disable. This will override the image_id checking from db (always fetch the image page to check the remote size).

``


Extend AI Service metrics

At the moment, we only have 2:
# HELP langchain4j_aiservices_TriageService_triage_seconds
# TYPE langchain4j_aiservices_TriageService_triage_seconds summary
langchain4j_aiservices_TriageService_triage_seconds_count 1.0
langchain4j_aiservices_TriageService_triage_seconds_sum 2.285523708
# HELP langchain4j_aiservices_TriageService_triage_seconds_max
# TYPE langchain4j_aiservices_TriageService_triage_seconds_max gauge
langchain4j_aiservices_TriageService_triage_seconds_max 2.285523708

Collecting the number of requests, successes, and failures would be interesting.

Definitely

I would like to get https://github.com/quarkiverse/quarkus-langchain4j/pull/145 in before working on this as it could have implications on how it's implemented


Initial commits to the website

Please describe the changes this PR makes and why it should be merged:
This addition adds the start of the website. It is no where close to complete, but seeing as I am out of town attending to family matters, I thought I would commit the beginnings to hasten the process. I do plan to work on the doc page for #194. As mentioned there, some commands in the repo don't include them, which in the site are fixed.
This add a file called yagpdb-cc-website which holds all code for the website. Seeing as this changes none of the ccs, I am not sure if the status is needed for this.
As always, the fastest way to reach me is Discord. Feel free to @ me or DM me.
If you plan to help out, the steps to installing docusaurus is here. Make sure to cd into yagpdb-cc-website first to run the development server (which you can do by running npm run start or yarn start).
Status

[x] Code changes have been tested on an instance of YAGPDB, or there are no code changes
[x] I have read and followed the contribution guide

So, just thinking further ahead here but we can add a search feature using Algolia DocSearch. We can do our own search feature, but I don't see any downsides to using Algolia as it is free.
We can also use i18n for translations of the site. Although we don't offer much translated support, maybe a good idea to have a German translation at least.
Potentially get rid of all the fluff folders/code (blog, start page, extra pages, etc), and make it purely docs only so the other pages are not encountered randomly.

@jo3-l what do you think?

Unrelated commits are gone, feature/website didn't have some commits from master as I suspected. Slapped a git merge on it and this looks to go after weird spacing is fixed.

smh Joe

:shipit:

:shipit:


Custom NPC Skin

[FCNPC] Error: NPC 'Bad Dog' not created. Name 'Bad Dog' is invalid or the maxnpc limit in server.cfg has been reached.
maxnpc 100

Already known:
http://forum.sa-mp.com/showpost.php?p=3997724&postcount=3529
#125

Space isn't allowed in nick names.


KAFKA-8546: Call System#runFinalization to avoid memory leak caused by JDK-6293787

This PR prevents the surge of Finalizer objects from [Inflater, Deflater] instances, by calling System#runFinalization on closing GZip[Input, Output]Stream.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

I am not convinced that this is actually needed. The OpenJDK issue seemed to imply that this was needed when the Stream was not closed.

@ijuma I see. For this issue is not an urgent one, let's wait and see the feedback from the community. Thanks for your instant response!

@ijuma Kafka definitely close the Stream correctly. However, the heap dump shows that it is 98% used by Finalizer object. After eliminating other impossible causes, JDK-6293787 is the only one left.
https://dzone.com/articles/debugging-understand-finalizer explains that close has no bearing on when finalizer is called. So even if "close" is called, "finalizer" is not guaranteed to be called.
https://docs.oracle.com/javase/9/docs/api/java/lang/Object.html#finalize-- in Java 9 explains this in the deprecation note for "finalize" method: "The finalize method might be called on a finalizable object only after an indefinite delay, if at all."

@ijuma I found that a similar problem may occur with lz4. To summarize:

KafkaLZ4BlockInputStream, KafkaLZ4BlockOutputStream calls XXHashFactory.fastestInstance() internally.
However, XXHashFactory.fastestInstance() returns the native instance factory if lz4 classes are loaded by the system class loader.
The native instance factory returns StreamingXXHash32JNI instance, which has finalize method, like Gzip's Inflater and Deflater.

From then on, a similar situation (i.e., the surge of Finalizer objects) described by @badaiaqrandista may occur.
It seems like the developers of lz4-java are also aware of this problem: lz4/lz4-java#131

Retest this please.

Here is the fix. GZIPInputStream is basically the copy of JDK 8's GZIPInputStream, with the following differences:

It uses ThreadLocal Inflater.
Resets Inflater instance after #close() for later use.

(It also applies to GZIPOutputStream, which replaces GZIPOutputStream)

retest this please

Retest this please

retest this please

retest this please

Rebased against the latest trunk. cc/ @badaiaqrandista @ijuma

Rebased against the latest trunk. cc/ @badaiaqrandista @ijuma

Rebased against the latest trunk. cc/ @badaiaqrandista @ijuma

Retest this please.

Retest this please.


Is version 2 still beta?

 Issue by RobSmyth
Monday Oct 30, 2017 at 04:35 GMT
Originally opened as https://github.com/gsscoder/commandline/issues/496

Hi,
Is there a stable v2 release?
It looks to me that the last stable release was 1.9.71 (2013?) and 2.1.1-beta was published on NuGet Nov last year.
Thanks

 Comment by nemec
Tuesday Oct 31, 2017 at 17:37 GMT

No, there is not a stable 2.x release, but the beta is pretty "stable" from a random-crashing perspective (and it contains a lot of bug fixes that the 1.x doesn't)

 Comment by jmecosta
Wednesday Nov 01, 2017 at 08:22 GMT

@nemec any estimates when 2.x is release. would like to take that into use

 Comment by ohadschn
Wednesday Nov 01, 2017 at 16:45 GMT

See https://github.com/gsscoder/commandline/issues/488

@RobSmyth @jmecosta @ohadschn formal release of v2.2 is eminent.


ERROR in example $getRecord(key)

if we look at the documentation (https://www.firebase.com/docs/web/libraries/angular/api.html#angularfire-firebasearray-getrecordkey) the examples use function $getRecord(key) so see
var list = $firebaseArray(ref);
var rec = list.$getRecord("foo"); // record with $id === "foo" or null

BUT it's not work
this implementation that works
var list = $firebaseArray(ref);
list.$loaded()
    .then(function (arr)
    {
        console.log(arr.$getRecord(_id));
    });

Thanks for this. I'll see what we can do to clear that up.
The assumption here is that you'd already have the list loaded if you know a key you want to get out of it (otherwise, where did the key come from?). If you're trying to load a single record, you don't need $firebaseArray(); you can reference the record directly and load it into $firebaseObject.
Furthermore, $loaded() is generally only useful as a debugging tool, so we try not to include it in the examples because it tends to confuse things more than help. We do cover asynchronous ops and a bit on debugging (i.e. using $loaded()) in the guide.
Again, appreciate the feedback and I'll ponder this.


RTL documentation missing

Breaking of #4367:
proposal:
RTL support exists but I could find how to use it in the documentation, I found an issue, linked to a changeset leading to the relevant code.

+1

I think a guide in the guides/ folder would be enough. cc. @EladBezalel

@HarelM @DevVersion well, i think the basic assumption is that it works out of the box, we can use a guide that says use dir="rtl" on html/body and automatically all the components will support it

True, but material 2 also supports dynamic content direction change using [dir]=... which is a great feature that IMHO deserves a guide/page/paragraph somewhere.

Yeah I agree that we should briefly mention that RTL support is there & how you could use it.
Also import to mention is that RTL mode doesn't work when specifying the direction from CSS.

@EladBezalel @DevVersion Well, I tried to add dir="rtl" on html and most of the components appear as expected, but when I opened a dialog it still had dir="ltr" on the cdk-overlay-pane, which means that unfortunately not all the components support it.

I use a variable on a service so that I can change it in one place to change all the components.
I manually add direction to all dialogs to support this, since they are not part if the "main" DOM it didn't feel weird to me.

Then, at least, it has to be well documented, currently there is no a single word about this property on the API reference page,

When #4044 will get in it would support html/body, I agree we should address RTL on the docs.. next thing to do!

By the way, as per W3 https://www.w3.org/International/tutorials/bidi-xhtml/

Add a dir attribute to the html tag to set the default base direction of your page if it is right-to-left. Use the dir attribute on block elements within the page only where you need to change the base direction.

So in my, may not so humble, opinion, MdDialogConfig.direction should not have a default value...


controller: assertion in ble_ll_utils_csa2_remap2chan

While going through GATT/CL/GAD/BV-06-C test case where connection with PTS is established multiple times, an failed assertion occurs randomly in ble_ll_utils_csa2_remap2chan(): g_ble_ll_data.chan_map_used is set to 0

To reproduce this, use these settings to debug nrf5340 netcore

FYI @KKopyscinski


503 on function download_notification_report for report format JSON

When generating an JSON report in the Black Duck UI the HUB will display the error "Unable to read report contents due to an unarchive operation error" when the generated report is too large to be read in the application. Error: 503
The provided fix patches with the broken function download_notification_report by replacing the download link from "content" to  "download". The patched function always downloads the report as ZIP file like it was before for TEXT format only. Now the return value is consistent for all type of reports.
We provide the attached patch in the hope that it is useful for all and merged to master.
001-Reporting.patch

Closing as stale - feel free to reopen if still needed

What does stale mean? We are using the patch in production to do reporting. As far as I know the python blackduck client has still the same issues, hence it is not compatible with the current REST API. Should we try to contact Synopsys support?


What are the best practices for ord as a service?

In case of errors, database files often get corrupted and need to be re-indexed.

We use systemd you can have a look at our service files in deploy directory. You just have to watch out to properly shut down ord before attempting to move or rename the index.redb file.

I confirm that it crashes and recovery takes very long time. The weird thing is that even if its at the tip, doing nothing, just querying bitcon node, it corrupts the db if it crashes.
I am wondering if its related on how the index is opened even there is nothing to do.

How does it crash? How are you shutting it down? Or does it just randomly crash?

Its not the shutdown scenario, if I recall correctly it crashed because of OOM, which definitely means memory settings need to be tweaked on my end, but the thing I wanted to raise is that indexer should not corrupt the database if its just noop-ing querying bitcoin node.

What is the appropriate memory size to set? I want to know how many concurrent queries an ord with 32g memory can handle.

What is the appropriate memory size to set? I want to know how many concurrent queries an ord with 32g memory can handle.

Only the initial indexing takes a lot of memory. Once the index is built it can handle quite a lot of requests. We have a server that handles 5 million requests per day and sits at less than 5% CPU usage normally. And at about 70% RAM usage with 128gb of RAM.

Its not the shutdown scenario, if I recall correctly it crashed because of OOM, which definitely means memory settings need to be tweaked on my end, but the thing I wanted to raise is that indexer should not corrupt the database if its just noop-ing querying bitcoin node.

To build the full index with --index-sats you need at least 64gb of memory. Once the initial indexing is done corruption is very unlikely since that can only happen while flushing the cache to disk and in idle mode (following chain tip) that takes less than a second.

What is the appropriate memory size to set? I want to know how many concurrent queries an ord with 32g memory can handle.

Only the initial indexing takes a lot of memory. Once the index is built it can handle quite a lot of requests. We have a server that handles 5 million requests per day and sits at less than 5% CPU usage normally. And at about 70% RAM usage with 128gb of RAM.

Can systemd automatically restart in case of unexpected situations such as oom? Currently, the db cannot be recovered after the unexpected exit we have encountered, and can only be re-indexed

For @emilcondrea 's point about the index being corrupted, this has happened to us a few times. The issue is that the index updater requires a write transaction to run the update function:
pub(crate) fn update_index(&mut self, mut wtx: WriteTransaction) -> Result

This is required even if the Ord index is already at tip. Since the updater runs quite often, if there is a critical crash of any kind, the index will be corrupted due to that write transaction being open.
Instead, the updater could get the tip indexed height and the current tip height from the btc node (and hashes to check for reorg) before it opens the write transaction. If the index is already at tip and no reorg occurred, then the loop can just continue without ever opening the write transaction and risking the index corruption.


[Feature Request] Don't hide unfinished videos

If possible, I'd like to be able to avoid hiding videos that are only partially watched - a few seconds of watch progress will cause it to be hidden right now.

This is configurable via the settings. Set it to 100%, to completely disable


Hi, about d = || x - y_C - y_R ||^2

hi buddy，
Awesome code and article， i hava a question； In this code d = || x - y_C - y_R ||^2  turn to d = || x - y_C ||^2 - || y_C ||^2 + || y_C + y_R ||^2 - 2 * (x|y_R)； I want to know why not calc d in the way the same as in Faiss IVFPQ，first calc R = x - y_C on the fly, and then calc R and  y_R‘s l2 distance table,then find this table and Cumulative sum for then candidate indexed docs？In this way do not need store term2（ || y_C + y_R ||^2） save memory，and d is the exactly same as calc above || x - y_C ||^2 - || y_C ||^2 + || y_C + y_R ||^2 - 2 * (x|y_R).
Thank you for helping me

Hi,
Thanks for your question. Let's look at FAISS decomposition:
d = || x - y_C ||^2 + || y_R ||^2 + 2 * (y_C|y_R) - 2 * (x|y_R)
Here, || y_R ||^2 + 2 * (y_C|y_R), indeed, can be precomputed and needs nlist * M * ksub storage.
In our work, the coarse codebook contains 2^20 centroids => nlist=2^20. Let's assume that M=2^4 and ksub=2^8. As a result, we have to store 2^32 floats and hence 2^34 bytes ~= 16Gb. Our decomposition demands storing || y_C + y_R ||^2, which is just ~1Gb after the quantization.

hi dbaranchuk，
Thanks for your reply. I think i have got your point, very make sense~
Then I have another confusion about the purpose of the decomposition in your code and Faiss; At the very beginning i know ivfpq， people do the  d = || x - y_C - y_R ||^2 calculation as follow steps：
I. use x find the nearest y_C, the calculate residual x_R=x - y_C;
II. then use x_R precompute L2 distance table with y_R；
(see:https://hal.inria.fr/file/index/docid/514462/filename/paper_hal.pdf page 8 and 9 C. Search algorithm)
I am wondering what benefits can we get from the decomposition no matter in your code or Faiss code，save one vector minus step or turn distance table calculation from l2 to inner production?leading to higher recall or faster query performance?
Bad english but also sincerely thanks  for this again ~ ^_^


Filename Timestamp Formatting

Do some nicer string formatting for the timestamp filenames

merged straight into master -- commit 070213af


Batch processing for trained SOM

The current process for running the trained SOM is fairly slow (#345). This is in part because we have to perform individual operations for every pixel. I think we could do some linear algebra to create a batch version of this, and run multiple pixels through the trained SOM at once. This should substantial improvements in speed.

I asked the author of MiniSOM if there was an option to process MiniSOM in batches rather than one-by-one and he basically said he would, only if I provided a solid research paper to back up doing it that way. Like speed optimization requires a paper to back it up. Guess we'll have to do some work for him.

I believe he was referring to batch processing during training. I was thinking about that as well, could provide a nice speedup. But it would fundamentally change the algorithm, since you'd need to average the updates somehow.
However, for batch-processing the trained version, there's no theory required, because you aren't actually modifying anything.

@ngreenwald OK, I missed the "trained SOM" part of this issue. Yeah, that would be even easier to do. We would have to modify the winner function to be able to assign multiple winning coordinate values at once for the clustering, which shouldn't be a huge deal.

Yes, you can either modify winner to have a batch mode, or make a
batch_winner function, whichever is easier.
On Thu, Dec 3, 2020 at 11:06 AM alex-l-kong<EMAIL_ADDRESS>wrote:

@ngreenwald https://github.com/ngreenwald OK, I missed the "trained
SOM" part of this issue. Yeah, that would be even easier to do. We would
have to modify the winner function to be able to assign multiple winning
coordinate values at once for the clustering, which shouldn't be a huge
deal.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/angelolab/ark-analysis/issues/363#issuecomment-738231089,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADJB47MK7VHFFCL4Y6SZ2Z3SS7OUJANCNFSM4UML4HDA
.

Closing this issue since we've switched to a full R subprocess call implementation.

Closing this issue since we've switched to a full R subprocess call implementation.


Revelation 6:9 tan yucan / τὰς ψυχὰς

Regarding Revelation 6:9
tav 3588 {T-APF} | yucav 5590 | <yucav> 5590 VAR: yucav 5590
https://github.com/byztxt/byzantine-majority-text/blob/6bcbfd664991db05f3b915afafa0b844c5d8a418/parsed/RE.UB5#L967
https://github.com/byztxt/byzantine-majority-text/blob/6bcbfd664991db05f3b915afafa0b844c5d8a418/textonly-beta-code/RE.CCT#L107
Should it be tan yucan? Maybe it is a typo? The "CNTTS Apparatus" in accordance doesn't list a single manuscript with the ν ending. Does anyone know what is going on here?

The encoding of the parsed text is not BETA encoding, but (I believe) Online Bible encoding.
In this latter encoding, "v" means "final sigma", not "nu".


Fix a few typos

Awesome guide! I just noted a few typos.

@snird ahm ahm

Thank you! This looks great.


Exposing PoppperJS trough the library

Hi everyone,
Hi recently opened #224 that i realize is a poor solution to the problem.
The problem is that you may need to access to popper defaults in order to build an object that is compatible with the propTypes.
Matching your Popper with the one from react-popper is complicated and if often get a double boundling with webpack.
I would like to suggest to add PopperJS to the exports of react-popper in order to access defaults.
I would submit a green PR but i have flowJS test failing and i have no idea how to fix them

For anyone interested in a solution, I expressed my opinion in #244.


Update constants.py

@offish I have updated this package to work with Selenium 4 for my needs, please review.

Reopened - confirmed to be working all correctly :)


<EMAIL_ADDRESS>error.

I want to clarify right away that there was no such error and the music was playing calmly, the error appeared by itself and refers to<EMAIL_ADDRESS>

I turn on the music in the usual way.

And I get the following error: TypeError [ERR_INVALID_ARG_TYPE]: The first argument must be of type string or an instance of Buffer, ArrayBuffer, or Array or an Array-like Object. Received undefined

As I understand it, YouTube does not return a response and<EMAIL_ADDRESS>does not handle such an exception.

environment

package.json

{
    "discord.js": "^14.14.1",
    "distube": "^4.1.1",
    "@distube/yt-dlp": "^1.1.3",
    "@discordjs/opus": "^0.9.0",
    "@discordjs/voice": "^0.16.1",
}

system

This was fixed on latest version of ytsr, I will update DisTube later.
Temporary solution:

Disable ytsr update check https://github.com/distubejs/ytsr?tab=readme-ov-file#update-checks
Override ytsr version (package.json)

{
    "overrides": {
        "@distube/ytsr": "2"
    }
}

Fixed on<EMAIL_ADDRESS>you can use npm update to update it

Thanks for the promptness!


Issue/963

As discussed, @Meisolsson @hzsweers.
Api key has been regenerated as well.

Looks good to me!

lgtm :+1:

Thanks guys!


'React/RCTConvert.h' file not found

Hello,
I am getting error as 'React/RCTConvert.h' file not found after adding firebase dependency in pod file and doing pod update.
I am getting above error in class RNSVGBrush

react-native-svg - ^6.3.1
react-native - ^0.50.1
react - 16.0.0-alpha.12

I am stuck here from last few hours.

Have same issue. Have you foung a solution?

I have the same issue.

Hello is there any solution to this?

Seems like a linking issue. Hard to give any suggestions with the current amount of information, as I haven't seen/resolved the issue myself, and am not even using cocoapods for anything atm. Could you make a minimal reproduction? Start with initialising a new react-native project and git repository, configure cocoapods, make sure everything works, commit. Add react-native-svg and firebase as dependencies, check if it still fails. Then commit, push to some public git repo and report back with a link to the git repo.

Fixed by making the following changes to Podfile

pod 'React', :path => '../node_modules/react-native', :subspecs => [
    'Core',
    'CxxBridge',
    'DevSupport',
    # the following ones are the ones taken from "Libraries" in Xcode:
    'RCTAnimation',
    'RCTActionSheet',
    'RCTBlob',
    'RCTGeolocation',
    'RCTImage',
    'RCTLinkingIOS',
    'RCTNetwork',
    'RCTSettings',
    'RCTText',
    'RCTVibration',
    'RCTWebSocket'
  ]

  pod 'yoga', :path => '../node_modules/react-native/ReactCommon/yoga'
  pod 'DoubleConversion', :podspec => '../node_modules/react-native/third-party-podspecs/DoubleConversion.podspec'
  pod 'Folly', :podspec => '../node_modules/react-native/third-party-podspecs/Folly.podspec'
  pod 'glog', :podspec => '../node_modules/react-native/third-party-podspecs/GLog.podspec'

pod 'RNSVG', :path => '../node_modules/react-native-svg'

post_install do |installer_representation|
    installer_representation.pods_project.targets.each do |target|
        if target.name == "React"
            target.remove_from_project
        end
    end
end

In my case react-native-svg installed React pod which's already there in node_modules, hence creating conflict.
So declared React pod, referencing to pod exists in node_modules/react-native

@saggiyogesh I have tried this solution and I think I'm getting one step closer, however  I am getting an error:
ld: 63 duplicate symbols for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)
Do you have any idea what this could be?

I think the error is with react-native-video. Try to clear everything & then reinstall.

rm -rf node_modules/
npm / yarn install
pod update

Hope this works.

same @jaygould

@Zurab I ended up spending a few hours and running in to issue after issue, so decided to download a fresh install of RN 0.57 and copying over my git folder, src, and all settings etc, re installing via NPM and re-linking. All works like a charm now and runs a bit faster.

This is probably an issue with react-native & cocoapods, not this library. Closing...

Hi there, I fixed this by removing RNSVG completely from my Podfile and then manually linking RNSVG into the "Libraries" group in Xcode.
The cause was as mentioned above, React is declared as a dependency in this Pod and we already have it linked outside Cocoapods. In fact we are almost 100% Pod-free!
NB: I had to add it to "Link Binary With Libraries" for all the targets in my app.
I think the podspec for this library has been written badly so it's best to avoid it.

@nabilfreeman thanks, man. This one works!


Route jump

Why does the push of the route not destroy the current component？What should I do when I want to push the route and destroy the current component？

Hey, you can use pushReplacement method like this
Navigator.pushReplacement(context, MaterialPageRoute(builder: (context)=>MyNewPage()));

Closing as this is neither a bug nor a feature request.


http proxy support is broken in 7.21+

After switching HTTP adapter from Net::HTTP to Net::HTTP::Persistent proxy support is broken:
WARN: TypeError: Parameter 'http': Expected type Net::HTTP, got type Net::HTTP::Persistent with hash<PHONE_NUMBER>967444236
Caller: /opt/vendor/bundle/ruby/3.3.0/gems/vonage-7.26.0/lib/vonage/namespace.rb:18
Definition: /opt/vendor/bundle/ruby/3.3.0/gems/vonage-7.26.0/lib/vonage/http.rb:25 (Vonage::HTTP::Options#set)

Hi @profforg can you provide some more detail about this issue? E.g. the specific steps which led to the error? Thanks!

The http proxy credentials are passed when the client instance is initialized (since the system proxy settings from ENV are ignored) and when I try to send SMS I get this error
example code:
    client = Vonage::Client.new(
      http: {
        proxy_address: ENV.fetch('VONAGE_PROXY_ADDR', nil),
        proxy_port: ENV.fetch('VONAGE_PROXY_PORT', nil) },
      api_key: ENV.fetch('VONAGE_API_KEY', nil),
      api_secret: ENV.fetch('VONAGE_API_SECRET', nil)
    )
    client.sms.send(from:, to: phone:, text:, type:, callback:)

@profforg I think I've identified the cause of the issue (actually, there are two separate, but related, underlying issues). I just need to investigate/test a bit more before pushing a fix. I'm aiming to release an update with a fix for this issue early next week.

@profforg setting HTTP options is fixed in this release.
Note: some of the available options on Net::HTTP::Persistent are different from the ones on Net::HTTP (event though Net::HTTP::Persistent uses Net:HTTP under the hood). For example, there is a single setter for proxy rather than separate setters for proxy_address and proxy_port. See Rubydoc for more info.


Add template for feature request issues

Created a template for the feature request issues.

Closed with #10


Modify cache query reponse

I have a feed query that loads initial 40 data items from null position cursor from network and caches( sqlite chained with in-mem-cache)  the response and after time T(mins) the users refreshes and loads 40 more from last most recent cursor, and  after time T(mins)  the app process is killed and restarted with no network connection. and the first null cursor position Feed data is loaded and the user has to manually load the second cache cursor position. loading more than the required data for without network connection.
Could i modify the query cache result so just the latest 20 Feed data items are initially loaded with no network connectivity?
Our two possible solutions as of now are

Query replaying and filtering by timestamp to get the user to most recent data.

This give rise to complex replaying strategies or just calling notfityDataSetChanged which blinks the screen and not very UI friendly

Replaying the first query with limits to 20 and filtering each time users refreshes the page so when offline the query cache results reflects the most recent data.

This will defeat the our main goal of switching to graphql which was to save bandwidth by sending only what is need when its is need. and replaying the first query over and over pushes redundant data over the wire

I'm hoping there is at least a better way to get the latest feed data on first load either from network or cache without UI or Network side effects

Unfortunately you can't as for the paginated queries the normalized cache stores response page by page with a cache key calculated based on cursor and page size values. Why not store the cursor of the latest page somewhere outside of Apollo like shared pref and then load only the latest page? But unfortunately you can't change the the page size, you will get all initially requested 40  items. If you need less then you need trim the list before showing it on the UI

Can i query the cache for a fragment with __typename and get all fragments of that type in the cache store?

Normalized cache is key - value cache (map data structure). The cache key is defined by com.apollographql.apollo.cache.normalized.CacheKeyResolver and  the only way to find smth in the cache is to provide the exact key that in most cases is globally unique ID.

Closing due to inactivity.


Hotfix/i1421

Fixed: #1421

Codecov Report

Merging #1432 (e0c4f7c) into master (a2c5c57) will decrease coverage by 0.04%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master    #1432      +/-   ##
==========================================
- Coverage   69.26%   69.22%   -0.05%
==========================================
  Files         381      381
  Lines       37930    37930
==========================================
- Hits        26273    26257      -16
- Misses      10006    10017      +11
- Partials     1651     1656       +5

Flag
Coverage Δ

go-1.14
?

go-1.15
69.22% <0.00%> (+<0.01%)
:arrow_up:

go-1.16
?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

os/glog/glog_logger.go
69.79% <0.00%> (ø)

net/gtcp/gtcp_pool.go
34.09% <0.00%> (-4.55%)
:arrow_down:

os/glog/glog_logger_rotate.go
67.31% <0.00%> (-1.47%)
:arrow_down:

container/gtree/gtree_avltree.go
92.03% <0.00%> (-1.20%)
:arrow_down:

container/gtree/gtree_redblacktree.go
86.68% <0.00%> (-0.47%)
:arrow_down:

Continue to review full report at Codecov.

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

Aaavvvaaarrrriiiid

@danvinhe 有冲突请解决一下。

@danvinhe 有冲突请解决一下。

使用goimports格式化文件，导致包引入顺序变化了。建议还是要排序下，比较规整


Scoped account signing keys

Enhancement to add scoped account keys to the account.
e.g.
            claimsAccount.Account.SigningKeys.Add(new NatsAccountScopedSigningKey
            {
                 Key = kpScopedAccountSigningKey.GetPublicKey(),
                 Role = "chat_user",
                 Template = new NatsUser()
                 {

                 }
            });

The list of signing keys still also supports strings via NatsAccountSigningKey which has implicit string operators.
Tested using $SYS.REQ.CLAIMS.UPDATE to push the new account, and then using the new SetScoped method from NatsUserClaims signing with the scoped account signing key.

Quick note about UserPermissionLimits type used in Go implementation: I realise the Template property type differs from the Go implementation which is UserPermissionLimits however we don't have that implemented in this library. UserPermissionLimits is a subset of the NatsUser with only IssuerAccount string property missing. I don't think it's worth creating the UserPermissionLimits intermediate type here since the only meaningful usage seems to be in this Template property. We can live with the extra property not being used in this context.


About trained model and data

Thanks for your work,
Can you please also share trained model and dataset..

Thank you for your comment,
Sorry for the late replay
I can provide you with a sample of trainned data but I can not with the model
Data.zip
but for the record
I do not save the model while training. I only save its weight

Thank you


Suggest adding safe termination of emulationstation to save metadata

I've modified your script in retropie so emulationstation is gracefully terminated and metadata is saved
#functions that handle button events
def when_pressed():
led.blink(.2,.2)
os.system("sudo killall emulationstation && sleep 5s && sudo shutdown -h now")
def when_released():
led.on()
def reboot():
os.system("sudo killall emulationstation && sleep 5s && sudo reboot")
This was at the suggestion of an amazon user named Crysknife007
There may be a better way to accomplish this but this method definitely works. Obviously something more universal that does the same if you're using attractmode or some other front end would also be ideal.

This is a bad method!
Use python handler
import subprocess
output = subprocess.check_output(['./multi_switch.sh', '--es-pid'])
to get PID from ES binary if value>0 then EmulationStation is running
and now use os.system("./multi_switch.sh --es-shutdown")
to prober close system... and save metadata in every usecase
If value 0 is returnd use regular shutdown
os.system("sudo shutdown -h now")
multi_switch.sh can be used from my github here
https://github.com/crcerror/ES-generic-shutdown
This is the swiss knife for ES shutdowns.
Use as you want, but give credits to the original author!

I don't doubt you're correct but could you elaborate on why this is a bad method?
I'm all for a better method of colurse!

That makes sense


[Enhancement]: Passing the url of the image to the pipeline directly instead running wget first

The ERROR Read image towhee_logo.png failed has been reported several times, many users might forget to run the prep work in quick start. Since the file directory and url are supported, shall we consider using the url in quick start?

Made some changes to the quick start guide in https://github.com/towhee-io/towhee-docs/pull/132. If that look sufficient, please close this issue.


Fix/cluster tags

Issue #, if available:
#631
Description of changes:
Tags for GenericClusterProvider and FargateClusterProvider.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/do-e2e-tests


Ensure sorting between most db queries requests

This PR ensures that the result in the "Most Database Queries" is actually sorted in descending order. The previous code could result in the results being out of order since the ordering was applied seperately per view_name.

@auvipy could you hit the button to approve the workflow run?

done

can you check the build failures? also shouldn't we update unit tests for the change?

The build failures aren't related to the code changes in the PR. In fact, they're also present in https://github.com/jazzband/django-silk/pull/479/checks which doesn't do any code changes at all. It looks like master is broken.
Regarding updating the unit tests: I agree, that was a bit of an oversight on my end. I'll update them asap.

but test needed for all the proposed changes

@auvipy ref: https://github.com/psycopg/psycopg2/issues/1293 looking to see if anyone else has a solution in jazz band discussion. If not, then we can merge #486


Adding helm testing script for e2e tests

Completes https://github.com/kubernetes/test-infra/issues/1887
cc @michelleN @viglesiasce

Does this need to wait until we have tests in all the charts?
No, I think it provides value right away by testing that install and purge work as expected against a cluster built from HEAD kubernetes.

Thanks for the review @viglesiasce @prydonius
Are there specific items we need to address before merging this for CI to start running it?

LGTM as well. @viglesiasce can we reuse this script for our CI as well?

I think so. Will need to figure out if there is full parity and fill any gaps.


Issue 1871: Upgrade the maven-resources-plugin from 3.0.2 to 3.2.0

Upgrade the maven-resources plugin from 3.0.2 to 3.2.0

Pull Request Description
This pull request closes #1871
Acceptance Test

[x] Building the code with mvn clean install -Dintegration.tests still works.
[x] Running mvn spring-boot:run in the strongbox-web-core still starts up the application correctly.
[x] Building the code and running the strongbox-distribution from a zip or tar.gz still works.
[x] The tests in the strongbox-web-integration-tests still run properly.

Questions

Does this pull request break backward compatibility?

[ ] Yes
[x] No

Does this pull request require other pull requests to be merged first?

[ ] Yes, please see #...
[x] No

Does this require an update of the documentation?

[ ] Yes, please see strongbox/strongbox-docs#{PR_NUMBER}
[x] No

Code Review And Pre-Merge Checklist

[x] My code follows the coding convention of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code in hard-to-understand areas.
[x] My changes generate no new warnings.
[x] I have added tests that prove my fix is effective or that my feature works.
[x] New and existing unit tests pass locally with my changes.

Thank, @SolangeUG ! :)
I will now close this pull request, as the testing has passed.


No response from sent commands

Dear, thank you very much for this great project.
I have the following problem: when I send an alarmArm command the command is sent correctly but does not resize the alerts from the device when the alarm is activated.
I do not resent any alerts when I send commands.
I am using the 3.7 version with a gps tk300
Do you know what can be happening

What protocol are you using?

I'm using gps103 protocol
El 16 nov. 2016 21:31, "Anton Tananaev"<EMAIL_ADDRESS>escribió:

What protocol are you using?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tananaev/traccar/issues/2544#issuecomment-261117814,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AWNFrDNn1uc-6_aq4rAQuTxgN-rD2MhPks5q-6B7gaJpZM4K0ltM
.

How do you know if your device accepted the command. I guess it just doesn't understand or doesn't support it.

The command to stop the engine works correctly but the server does not show
the message of confirmation
(StopEngine Ok)
El 16 nov. 2016 21:47, "Anton Tananaev"<EMAIL_ADDRESS>escribió:

How do you know if your device accepted the command. I guess it just
doesn't understand or doesn't support it.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tananaev/traccar/issues/2544#issuecomment-261120616,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AWNFrAfmsEPCePUjvawBV1ZikapW8gthks5q-6QpgaJpZM4K0ltM
.

If you have the documentation, I can check if the other commands are correct.

I have the documentation physically. But I send you some photos
http://subefotos.com/ver/?aded740f9e89ef7fae476c71a7b55852o.jpg
http://subefotos.com/ver/?c21f6c0968896188282affa4a6b91856o.jpg
El 16 nov. 2016 22:40, "Anton Tananaev"<EMAIL_ADDRESS>escribió:

If you have the documentation, I can check if the other commands are
correct.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tananaev/traccar/issues/2544#issuecomment-261129375,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AWNFrMQkpF2cybvLHA7oJddERD7LjDHFks5q-7CggaJpZM4K0ltM
.

That's just device manual. I need documentation for communication protocol.

I found this documentation. It's enough?
El 17 nov. 2016 0:15, "Anton Tananaev"<EMAIL_ADDRESS>escribió:

That's just device manual. I need documentation for communication protocol.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tananaev/traccar/issues/2544#issuecomment-261143621,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AWNFrCy8Nx_LAWrZBIlUQdU065-IfBXxks5q-8bKgaJpZM4K0ltM
.

I don't know if you realize, but you are sending email directly to GitHub. It doesn't handle attachments. You can do it through GitHub website directly.

You're right, sorry I send you the file link.
https://www.google.cl/url?sa=t&source=web&rct=j&url=http://www.spadamar.com/wp-content/uploads/2011/11/GPRS-Data-Protocol.xls&ved=0ahUKEwjiqc7r6a7QAhXKEZAKHaSvA-UQFghLMAQ&usg=AFQjCNEhMNSn2CjW9H0FSTSxEOn11ob7qg
El 17 nov. 2016 0:23, "Anton Tananaev"<EMAIL_ADDRESS>escribió:

I don't know if you realize, but you are sending email directly to GitHub.
It doesn't handle attachments. You can do it through GitHub website
directly.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tananaev/traccar/issues/2544#issuecomment-261144741,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AWNFrDzeGJDz-yWpSLaYh9peh4heOLObks5q-8jGgaJpZM4K0ltM
.

There is nothing about commands for engine stop/resume or arm/disarm.

That means that gps does not support those commands via gprs only sms?

It probably means that documentation is not complete.

para quien trabjas omar??

Hola eduardo,
por que lo preguntas?
2017-01-06 22:56 GMT-04:00 Eduardo Vizcaino<EMAIL_ADDRESS>
para quien trabjas omar??
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tananaev/traccar/issues/2544#issuecomment-271058081,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AWNFrFt0zpmTubjayH3XHrQdjpO-s1Kcks5rPv7sgaJpZM4K0ltM
.

Hola eduardo,
por que lo preguntas?

pq un desarrollador que pregunta eso...
deberia ser el dueño...
y no trabajar para alguien

no trabajo aun estoy estudiando


'dict object' has no attribute 'stdout' when running in --check mode

We are seeing an error that appears only when --check mode is added.
ansible 2.9.13
  config file = /Users/name/dev/website/ansible/ansible.cfg
  configured module search path = ['/Users/name/.ansible/plugins/modules', '/usr/share/ansible/plugins/modules']
  ansible python module location = /usr/local/Cellar/ansible/2.9.13/libexec/lib/python3.8/site-packages/ansible
  executable location = /usr/local/bin/ansible
  python version = 3.8.5 (default, Jul 21 2020, 10:48:26) [Clang 11.0.3 (clang-1<IP_ADDRESS>)]

With --check
TASK [ansistrano.deploy : ANSISTRANO | RSYNC | Rsync application files to remote shared copy] ************************************************************************************************************************************************
fatal: [thehost-uat]: FAILED! => {"msg": "The task includes an option with an undefined variable. The error was: 'bool object' has no attribute 'stdout'\n\nThe error appears to be in '/Users/name/.ansible/roles/ansistrano.deploy/tasks/update-code/rsync.yml': line 6, column 3, but may\nbe elsewhere in the file depending on the exact syntax problem.\n\nThe offending line appears to be:\n\n\n- name: ANSISTRANO | RSYNC | Rsync application files to remote shared copy\n  ^ here\n"}

And without --check, i.e. running normally:
TASK [ansistrano.deploy : ANSISTRANO | RSYNC | Rsync application files to remote shared copy] ************************************************************************************************************************************************
changed: [thehost-uat]

I've just seen this issue exists: https://github.com/ansistrano/deploy/issues/320

Closing as the issue already exists


Blank page when accessing /admin

$ go run main.go
Failed to find configuration config/application.yml
Failed to find configuration config/smtp.yml, using example file config/smtp.example.yml
Failed to find configuration config/database.yml, using example file config/database.example.yml
Failed to create unique index for translations key & locale, got: Error 1170: BLOB/TEXT column 'key'
used in key specification without a key length
(Error 1170: BLOB/TEXT column 'key' used in key specification without a key length)
[2017-08-26 18:30:21]
[warning] duplicated callback qor_admin:composite_primary_key from /Users/alexander/go/src/github.com/qor/admin/composite_primary_key_callback.go:31
[warning] duplicated callback qor_admin:composite_primary_key from /Users/alexander/go/src/github.com/qor/admin/composite_primary_key_callback.go:32
[warning] duplicated callback qor_admin:composite_primary_key from /Users/alexander/go/src/github.com/qor/admin/composite_primary_key_callback.go:32
Listening on: 7000
2017/08/26 18:31:27 Finish [GET] /admin Took 0.02ms
failed to find template: auth/login
failed to find template: auth/login

Are you using the latest auth? maybe try to get latest qor libs with $GOPATH/src/github.com/qor/qor/update_all_qor_repos.sh?


Component loses typings with HOC

Steps to Reproduce
Describe how to reproduce this issue.
component1.tsx:
import Query from './query.graphql';

interface IProps {
  foo: string;
  bar: number;
}
const Component: (props: IProps) => JSX.Element = ({ foo, bar }) => {
  // return JSX
}

export default @graphql(Query)(Component)

component2.tsx:
import Component from './component1';

const SimpleWrapper: (props: {}) => JSX.Element = () => (
  <Component foo={false} />
)

Buggy Behavior
The above will compile, because react-apollo has typings:
export default function graphql(document: Document, operationOptions?: OperationOption): (WrappedComponent: any) => any;

Expected Behavior
This shouldn't compile, because prop bar is missing and foo is of the wrong type.
I'm thinking that the graphql HOC can take generics like:
interface InjectedProps<T> = {
  data?: T;
  loading: boolean;
  // etc.
}

interface ComponentDecorator<OriginalProps, AllProps> {
  (component: React.ComponentClass<AllProps> | React.StatelessComponent<AllProps>): React.ComponentClass<OriginalProps>;
}

graphql<QueryProps, ComponentProps>(document: Document, operationOptions?: OperationOption): ComponentDecorator<ComponentProps, ComponentProps & InjectedProps<QueryProps>>

(did this pretty quickly, probably not 100% accurate, see react-redux typings)
Version

<EMAIL_ADDRESS><EMAIL_ADDRESS>
Notes
I searched the issues and didn't see anything about this.
I'd be willing to make a PR if such a thing would be acceptable

Sure, this sounds reasonable if you can make it work.

Cool, I'll take a look and open a PR

I've got it working with Stateless Components and regular classes, but it fails when using a decorator.
Been through a bunch of variations and can't seem to make it work properly.
export interface InjectedGraphQLProps<T> {
  data?: T;
  loading?: boolean;
}

export interface FixedComponentClass
 extends ComponentClass
 {
  new (props?: P, context?: any): React.Component
;
}

export type ComponentClass
 = React.ComponentClass
;
export type StatelessComponent
 = React.StatelessComponent
;

export interface WrapWithApollo<QueryProps, ComponentProps> {
  (
    WrappedComponent: (
      ComponentClass<ComponentProps & InjectedGraphQLProps<QueryProps>> |
        StatelessComponent<ComponentProps & InjectedGraphQLProps<QueryProps>> | any
    )
  ): (FixedComponentClass<ComponentProps>);
}

export default function graphql<QueryProps, ComponentProps>(/* args */) {
  const wrapWithApolloComponent: WrapWithApollo<QueryProps, ComponentProps> = WrappedComponent => {
     /* body */
  }
}

and this works with:
export interface ITest1Props {
  x: boolean;
}

export interface ITest1QueryProps {
  a: string;
  b: number;
}

export type IProps = ITest1Props & InjectedGraphQLProps<ITest1QueryProps>;

export class Test1 extends Component<IProps, {}> {
  componentWillReceiveProps(props) {
  }
  render (): JSX.Element { return
{this.props.data.a}
; }
}

export const Test1Out = graphql<ITest1QueryProps, ITest1Props>(null!)(Test1);

class Test1Wrapper extends Component<{}, {}> {
  componentWillReceiveProps(props) {
  }
  render (): JSX.Element {
    return <Test1Out x={true} />;
  }
}

@graphql<ITest1QueryProps, ITest1Props>(null!)
export class Test2 extends React.Component<IProps, {}> {
  render (): JSX.Element {
    return
{ this.props.data.a }
;
  }
}

class Test2Wrapper extends Component<{}, {}> {
  render (): JSX.Element {
    return <Test2 x={false} />;
  }
}

const Test3: (props: ITest1Props) => JSX.Element = (props: ITest1Props) =>
;

const Test3Out = graphql<ITest1QueryProps, ITest1Props>(null!)(Test3);

class Test3Wrapper extends Component<{}, {}> {
  render (): JSX.Element {
    return <Test3Out x={false} />;
  }
}

but it fails with:
@graphql<any, any>(null!)
class TestFail extends React.Component<any, any> {
  componentWillReceiveProps(props) {
  }

  render() {
    return null;
  }
}

because

Unable to resolve signature of class decorator when called as an expression.
Type 'FixedComponentClass' is not assignable to type 'typeof TestFail'.
Type 'Component<any, any>' is not assignable to type 'TestFail'.
Property 'componentWillReceiveProps' is missing in type 'Component<any, any>'.

i'm sure something like https://github.com/DefinitelyTyped/DefinitelyTyped/blob/types-2.0/react-redux/index.d.ts#L26 is needed for the decorator, but that breaks it for the class/stateless component 😞
i'll keep looking, but any help would be appreciated

actually can just take the thing from react-redux and make the props from react-apollo optional and able to be generated with the InjectGraphQLProps interface


Support CLI interface

Is there way to run mockoon without UI ? I want to use mockoon on UNIX-like operating systems without GUI.

Dear people interested in the CLI. You will be happy (we hope 🤞) to learn that we just published a beta version of it. 🥳https://www.npmjs.com/package/@mockoon/cli
It's quite clean and stable, but still, it's a beta version, and we will need your feedback to improve it. 😃
After several months of work, we will need to give some love to the main application, but we will closely follow the evolution of this tool.
We will announce its availability next week. Until then, do not hesitate to tinker with it and tell us what you think.
Thanks for your patience. 🚀


ValueError: output of generator should be a tuple (x, y, sample_weight) or (x, y). Found: None

Hi, Thanks for this open sourcing this lipnet. I'm still new to ML and still learning.
However I'm trying to train 10 videos (from grid) for now(just for testing) using the "unseen_speakers" and getting an error about validation steps.
Also enabled vtype='face', face_predictor_path=FACE_PREDICTOR_PATH
Below is the complete log
Using all available GPUs.
Using TensorFlow backend.

Loading dataset list from cache...
Found 10 videos for training.
Found 10 videos for validation.

_________________________________________________________________
Layer (type)                 Output Shape              Param #
=================================================================
the_input (InputLayer)       (None, 75, 100, 50, 3)    0
_________________________________________________________________
zero1 (ZeroPadding3D)        (None, 77, 104, 54, 3)    0
_________________________________________________________________
conv1 (Conv3D)               (None, 75, 50, 25, 32)    7232
_________________________________________________________________
batc1 (BatchNormalization)   (None, 75, 50, 25, 32)    128
_________________________________________________________________
actv1 (Activation)           (None, 75, 50, 25, 32)    0
_________________________________________________________________
spatial_dropout3d_1 (Spatial (None, 75, 50, 25, 32)    0
_________________________________________________________________
max1 (MaxPooling3D)          (None, 75, 25, 12, 32)    0
_________________________________________________________________
zero2 (ZeroPadding3D)        (None, 77, 29, 16, 32)    0
_________________________________________________________________
conv2 (Conv3D)               (None, 75, 25, 12, 64)    153664
_________________________________________________________________
batc2 (BatchNormalization)   (None, 75, 25, 12, 64)    256
_________________________________________________________________
actv2 (Activation)           (None, 75, 25, 12, 64)    0
_________________________________________________________________
spatial_dropout3d_2 (Spatial (None, 75, 25, 12, 64)    0
_________________________________________________________________
max2 (MaxPooling3D)          (None, 75, 12, 6, 64)     0
_________________________________________________________________
zero3 (ZeroPadding3D)        (None, 77, 14, 8, 64)     0
_________________________________________________________________
conv3 (Conv3D)               (None, 75, 12, 6, 96)     165984
_________________________________________________________________
batc3 (BatchNormalization)   (None, 75, 12, 6, 96)     384
_________________________________________________________________
actv3 (Activation)           (None, 75, 12, 6, 96)     0
_________________________________________________________________
spatial_dropout3d_3 (Spatial (None, 75, 12, 6, 96)     0
_________________________________________________________________
max3 (MaxPooling3D)          (None, 75, 6, 3, 96)      0
_________________________________________________________________
time_distributed_1 (TimeDist (None, 75, 1728)          0
_________________________________________________________________
bidirectional_1 (Bidirection (None, 75,<PHONE_NUMBER>
_________________________________________________________________
bidirectional_2 (Bidirection (None, 75,<PHONE_NUMBER>
_________________________________________________________________
dense1 (Dense)               (None, 75, 28)            14364
_________________________________________________________________
softmax (Activation)         (None, 75, 28)            0
=================================================================
Total params: 4,572,156.0
Trainable params: 4,571,772.0
Non-trainable params: 384.0
_________________________________________________________________
W tensorflow/core/platform/cpu_feature_guard.cc:45] The TensorFlow library wasn't compiled to use SSE4.1 instructions, but these are available on your machine and could speed up CPU computations.
W tensorflow/core/platform/cpu_feature_guard.cc:45] The TensorFlow library wasn't compiled to use SSE4.2 instructions, but these are available on your machine and could speed up CPU computations.
W tensorflow/core/platform/cpu_feature_guard.cc:45] The TensorFlow library wasn't compiled to use AVX instructions, but these are available on your machine and could speed up CPU computations.
Epoch 0: Curriculum(train: True, sentence_length: -1, flip_probability: 0.5, jitter_probability: 0.05)
Process Process-1:
Traceback (most recent call last):
  File "/usr/local/Cellar/python/2.7.13/Frameworks/Python.framework/Versions/2.7/lib/python2.7/multiprocessing/process.py", line 258, in _bootstrap
Epoch 1/5000
Traceback (most recent call last):
  File "/Users/rad182/Documents/rizkiarm-LipNet/training/unseen_speakers/train.py", line 76, in <module>
    train(run_name, 0, 5000, 3, 100, 50, 75, 32, 10)
  File "/Users/rad182/Documents/rizkiarm-LipNet/training/unseen_speakers/train.py", line 72, in train
    pickle_safe=True)
  File "/usr/local/lib/python2.7/site-packages/keras/legacy/interfaces.py", line 88, in wrapper
    return func(*args, **kwargs)
  File "/usr/local/lib/python2.7/site-packages/keras/engine/training.py", line 1851, in fit_generator
    str(generator_output))
ValueError: output of generator should be a tuple `(x, y, sample_weight)` or `(x, y)`. Found: None

Hi @rad182, I'm also trying to do a simple example training using unseen_speakers.  It sounds like I'm taking the same steps as you, though I'm using 17 training videos and 17 validation videos.  I also encountered this error and will post here if I figure anything out.
I see you changed the title of the issue.  Does that mean you figured out the problem?  If so, could you post how you resolved it?

@michiyosony i just changed the minibatch_size param in train() to the number of videos to train and it should work but I'm getting a new error which is updated above.

@rad182 I just saw that, thanks.  For the benefit of future users, I think it would be good to revert this above description and open a new issue with the new stack track.

Please tell me if the the following is the correct way of making changes in train.py, to include vface and face_predictor_path ?
lip_gen = BasicGenerator(dataset_path=DATASET_DIR, vtype = "face", face_predictor_path="/home/deepakgupta1313/Desktop/Deepak/Programs/Github/LipNet/common/predictors/shape_predictor_68_face_landmarks.dat", minibatch_size=minibatch_size, img_c=img_c, img_w=img_w, img_h=img_h, frames_n=frames_n,

FYI  if the validation_steps == 0 will raise this ValueError

I have edited fit_generator and assign a value to validation_step, is this the correct way to so it
@rizkiarm @shensheng27
    lipnet.model.fit_generator(generator=lip_gen.next_train(),
                        steps_per_epoch=lip_gen.default_training_steps, epochs=stop_epoch,
                        validation_data=lip_gen.next_val(), validation_steps=1,
                        callbacks=[checkpoint, statistics, visualize, lip_gen, tensorboard, csv_logger],
                        initial_epoch=start_epoch,
                        verbose=1,
                        max_q_size=5,
                        workers=2,
                        pickle_safe=True)