How to access query params?

Is there a way to retrieve query params passed to a child component?
I get $route undefined. Also tried with this.$route.
In parent:
<inertia-link :href="route('posts.create', { custom_param: myParam })">
    Add new post
</inertia-link>

In child:
export default {
    props: {
        myParam: $route.query.myParam
    }
}

Edit: I've found this in docs, but same problem I can't retrieve data.
<inertia-link href="/endpoint" method="post" :data="{ foo: bar }">Save</inertia-link>

So, $route and route are not properties that Inertia itself makes available, although we have used the Ziggy library in the Ping CRM demo for generating URLs, and there we make the this library available as a mixin.
Also, I'm not really sure what you mean that you can't access the query params in the child component. Do you mean you are creating a child element/component within the <inertia-link> component? Or do you mean that you're trying to access the query string once you visit a page? Because in that case, you can just use window.location.search to get these values.

Yes, I'm  trying to access the query string once you visit a page. I used window.location.search as you suggested. I tough Inertia makes $route available in views.

Understood. As noted, $route is not a "thing" in Inertia. You can totally use window.location.search (and maybe even the qs library to parse). However, I personally don't ever do this, and instead simply passing the query string values from my response to my view, and then access them as props.
For example, if I have a user search endpoint (/users?search=Jonathan), I will pass the search value as a prop back to my users page to display back on that page. You can see an example of this in the Ping CRM app here and here. I prefer this approach, since props are reactive.
Also, if you pass data to an <inertia-link> that uses post for the method, it's passed down as the body params. If you pass data to an <inertia-link> that uses get for the method (the default), it's passed down in the query string.
Not totally sure what you mean by this:

Does it make a data object available on the target view?

But no, Inertia doesn't pass link data directly to some view. Rather, it submits it server-side, and you can then choose to make it available to your page components by passing it as props (response data), as suggested above.
Hope that helps!

Thanks @reinink ! That's helps a lot.
My goal was to pass data from view A to view B, knowing that I use <inertia-link> to navigate between the 2 views. What would you recommend to use in this case?
Thanks again for your help!

Inertia links are always used for making visits. That's it. So, if an Inertia link navigates between two views, then send the data down from the link in the first view, and pass it back in the response as a prop to the second view. 👍

@reinink sorry for being dumb but I tried that method and I got undefined.
View A:
<inertia-link :href="route('comments.create')" :data="{ post: post.id }">
    Create new comment
</inertia-link>

View B: (http://mysite.test/comments/create?post=1)
export default {
    props: {
        post: String
    },
}

@wassim Happy to hear 👍

HandleInertiaRequests.php
```

public function share(Request $request)
{
return array_merge(parent::share($request), [
'ziggy' => function () use ($request) {
return array_merge((new Ziggy)->toArray(), [
'location' => $request->url(),
'query'=>$request->query()
]);
},
]);
}

utils.ts

import {usePage} from "@inertiajs/inertia-vue3";
export function getQuery() {
return (usePage().props.value.ziggy as any).query;
}

can't use window.location.search in ssr. because node server doee not have window variable.

HandleInertiaRequests.php
public function share(Request $request)
  {
      return array_merge(parent::share($request), [
          'ziggy' => function () use ($request) {
              return array_merge((new Ziggy)->toArray(), [
                  'location' => $request->url(),
                  'query'=>$request->query()
              ]);
          },
      ]);
  }

utils.ts
import {usePage} from "@inertiajs/inertia-vue3";

export function getQuery() {
    return (usePage().props.value.ziggy as any).query;
}

can't use window.location.search in ssr. because node server doee not have window variable.

This has saved me quite some time, I was thinking about going with the window.location.search and then parsing the link for param approach.

just share props with PageProps where it need ?
like
Intertia::render('SearchItemPage', ['itemId' => $itemId])

@santutu provides a great helper, here's an updated version for React, where you can specify a key and fallback value
import { usePage } from "@inertiajs/react";

export function getQuery(key: any = null, fallback: any = null) {
  let query = (usePage().props.ziggy as Record<any, any>).query;
  query = query.length == 0 ? {} : query; // cuz when empty Laravel return an array - can change it in HandleInertiaRequests.php!

  if (key) {
    return query.hasOwnProperty(key) ? query[key] : fallback;
  }

  return query;
}

To get the query data add  'query' => $request->query(), in HandleInertiaRequests

public function share(Request $request): array
    {
        return [
            ...parent::share($request),
            'auth' => [
                'user' => $request->user(),
            ],
            'ziggy' => fn () => [
                ...(new Ziggy)->toArray(),
                'location' => $request->url(),
                'query' => $request->query(),
            ],
        ];
    }

just share props with PageProps where it need ? like
Intertia::render('SearchItemPage', ['itemId' => $itemId])

Yeah, keep it simple. This is better overall unless the query parameters are useful on every page of the app and not needed on the backend.


What's the reasoning behind UserManager.GetUserIdAsync(TUser user) ?

If I have the TUser instance, I can simply use user.Id, can't I?
Why does this method exist?
BTW, What is the most efficient way of getting the current logged in user 's Id in ASP.NET Core?
Thanks,
Doron

User.Id exists in the identity EF implementation. It's not necessarily required as custom identity  implementations do not have to expose an id property. The tradeoff is between having the manager abstract this operation or directly going against an implementation property that the store backs against.

@HaoK
A. So might be a good idea changing the template which does use EF and still uses this method to get the id (didn't check if it was changed in 1.0.0 but I guess it wasn't).
B. Any tip about What is the most efficient way of getting the current logged in user's Id in ASP.NET Core? because currently I'm using this awkward code to get the user with a navigation property:
var user = await UserManager.GetUserAsync(HttpContext.User);
user = await Context.Users.Where(x => x.Id == user.Id).Include(x => x.Follows).SingleAsync();

There must be a better way of doing this!

Any tip about What is the most efficient way of getting the current logged in user's Id in ASP.NET Core

Put the user's id as a claim in the cookie when you sign them in.


consolidate the information in requirements.txt, PLAN.md

Currently there is some duplication, some conflicting information, and some outdated information in requirements.txt and PLAN.md.
I'd like us to consolidate that information such that it is DRY and accurate. Probably this means that we'll end up with one file, README.md

Define goal of software

Done in PR #112


feat: add types

Fixes #20

What is the Subject type please, it seems to be blocking build locally ?

Since the project is in pure JS, I fear this PR will never be merged.. Or maybe in some documentation section


Proposed my own subdomain project

[x] My submissions follows the Submission Rules
[x] I have read and accepted the Terms and Conditions

shyngys is my name and I am going to use this as my portfolio link. SalemAlem means Salem = Hello, Alem = World in Kazakh


No vcs provider found for [ hg4idea ]

Got this error after I clicked "precommit":

after i click precommit:
server url is not configure
What should I do?


Proxy: support compressed blacklists

ie:http://dsi.ut-capitole.fr/blacklists/download/ via @gitmod

@jschellevis just skimming through packets.. this was always a good addition being requested :)

@fichtner I will try to add the compressed downloads before 16.1, it's not forgotten :)

Thanks!


Shift-clicking items into the Redstone Logic Control GUI locks the game

Discovered that if you shift-click items into the Redstone Logic Control GUI, the game locks and just stalls as is. Found this out with my modpack, but I tested this in a fresh instance with just this mod in it, and confirmed it's still happening.
latest.log
This is the log, but no errors seem to appear. It just... freezes.

Hi serfina, ty for the hint, I could reproduce and fix the issue - pushed to Curse (redstonepen-1.18.2-1.0.10). Cheers,-


rule explanation should show a code example

Every rule explanation should show show a code example:
e.g.
$ ruff rule SIM210
if-expr-with-true-false

Code: SIM210 (flake8-simplify)

Autofix is always available.

Message formats:

* Use `bool({expr})` instead of `True if {expr} else False`

Example:
-a = True if b else False
+a = bool(b)

I agree. I'd consider this a duplicate of #1467.


"-- --output=elm.js" not working as expected in PowerShell

I'm currently going through this chapter on elmprogramming.com.
Half-way through, it instructs me to run the following command:
elm-live post-app/Main.elm --pushstate -- --output=elm.js

When doing this in PowerShell (tried both with version 5.1.19041.610 and 7.0.0), I get the following output:
----------------------
|| ERROR IN COMMAND ||
----------------------

Usage: <elm-main> [options] [--] [elm make options]

-- Output In Wrong Location -----------------

You have used the elm make flag --output in the wrong location. As seen in the usage example about, all elm make flags must be added to your command after the -- separator.

The workaround seems to be to add double-quotes around the -- separator like so:
elm-live post-app/Main.elm --pushstate "--" --output=elm.js

I've tried the regular Command Prompt as well, and that just works as expected.
Versions:

elm-live: 4.0.2
elm: 0.19.1
OS: Windows 10 Pro, (Version: 2004, OS build: 19041.867)

saved my day, thank you for the workaround!


Fix issue with missing rename of Self_

Summary
See #489.
The issue is that serde's snake case for enum variants differs from heck's snake case, especially for Self_ (and one more unexpected case, see the diff).
This was fixed by including the function used by serde, which is sadly not exported so I copy pasted it (serde is under MIT license).
The issue here is obviously that serde may change the internal function but I think this is very unlikely since it breaks all parsing code in existence and serde is no longer in alpha :)
Checklist

[x] ran cargo make fmt
[x] using conventional commits to hightlight user-facing fixes and features

@arlyon

Running CI now, thanks!

Closes #489

Closes #499

So just to make sure I've mentioned everything: The two implementations differ in:

Trailling _
_ before upper case character in the name
We could patch the implementation from heck, but at this point this patch is as large as just copying the implementation from serde.

Furthermore the code relies on the two snake cases (what serde does and what we do to determine whether a rename is needed) aligning, otherwise we will output wrong renames or unneeded renames.

Did some more digging and happy to merge this. Thanks!

Apologies for the delay, we have a few very kind people who participate sometimes but usually when I am busy PR reviews stall. Thank you for taking the time.

Is there anything broken with the release pipeline? The crates.io version is stuck at 0.31 and we are at 0.33 in this repository already since three weeks? @arlyon

Yeah, I need to manually push them. One of many things with this repo I've been meaning to get around to.


Ncalc error on second of stacked Transform using if

Describe the bug
Mobiflight reports ncalc error on the second of two stacked Transformation modifier.  Disabling the second modifier makes the error disappear.  However, inverting the two modifiers, again generates the error on the second modifier (formerly first).
Steps To Reproduce

Create stacked Transform modifiers like this

Hit run and see ncalc error

Error icon says "Transform Error".

Note that if the second modifier is disabled, the error disappears.

The red icon on the second output config is just complaining that there is no Simconnect connection.  I am not really using any Simconnect variable.  Only MF variables for this test.

Note that if I invert the modifiers, first goes to second and second comes to first, Mobiflight again reports there is an Ncalc error and disabling the SECOND modifier makes the error disappear.

Both modifiers work as expected inside the editing dialog and using the test function.

Expected behavior
Both modifiers work as expected in Run mode without ncalc errors.
Actual behavior
Ncalc error on the second modifier (regardless of which of the two is second) prevents the configuration from working appropriately.
Details

MobiFlight Version: 10.1.0
FlightSim: irrelevant
AddOn: irrelevant

Additional context
Maybe related to this bug, the display of the formulas when the modifiers are in list mode are not shown correctly.  See images below.  Note the && is shown as one single &

In Run mode, the error appears for whatever the condition of the second modifier is true.
This is the input config used to set the MF variable with a button toggling between 200 and 0.5.

I think I figured out why the Ncalc error. The string '-. - -' value cannot be used as $ in the second if statement for a numeric comparison.  So need to use a different config reference with the correct value.


Option to disable during voiced cutscenes

See title. For this to happen we need some list of voiced cutscenes and a method of detection.

I was using Anna's Sound Log And it seems like all voice lines come from cut/{ex{1-3]/sound/voicem/..(ex number varies by expansion the voice is coming from) and they fire right as the text box comes up. So you could scan for those voice files firing while the current condition is Current ID: 35 Enum: OccupiedInCutSceneEvent and not trigger TTS if a sound from that directory fires when the dialogue box pops up.

Oh, this could work. I might be able to pull file load detection from Penumbra or something.

Well I checked how the files load with reslogger, and it loads al the voices at the beginning of the cutscene during the loading screen and fire at the appropriate time. Wonder if Anna's Sound Logger would be a good place to start.

Oh, I just woke up 🥴 Will check that.

Actually, where is that? Not seeing it on her plugin repo.

It's part of her Sound Filter plugin

Oh not sure if you know this, but I remember reading something about sound file streaming, and how Anna's sound logger disables it so it can see when certain sounds are playing. I think they said Penumbra does the same thing. So I was able to see the sound paths for cutscene text that way.
Hope that's helpful!

Oh, this could work. I might be able to pull file load detection from Penumbra or something.

I think Penumbra does disable sound streaming so this could work too.
With sound streaming disabled it loads the voice audio file right as the NPC talks.

Oh this would be awesome for us controller players who stream to TV and are far from their keyboards 😀!


XAML code is invalid

ShowError="bool" should either be ShowError="True" or "False".

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

ID: c1ee8cca-c633-aa59-9602-e6b4a6e5e0e8
Version Independent ID: 45391825-e86e-a791-da06-9b815e12fb39
Content: ProgressBar.ShowError Property (Windows.UI.Xaml.Controls) - Windows UWP applications
Content Source: https://cpubwin.visualstudio.com/DefaultCollection/windows-uwp/_git/winrt-api-build?path=%2fwinrt-api-build%2fwindows.ui.xaml.controls.progressbar.yml&version=GBlive&_a=contents
Product: uwp
Technology: winrt-reference
Git Login: jwmsft
Microsoft Alias: jimwalk

I think the pattern for these sections is to show a declaration rather than example code, so it shows the type of the value. For example, FrameworkElement.Width is a double type, so this section shows
<frameworkElement MinWidth="double"/>

But it's confusing because in some cases it is example code, like FrameworkElement.Style:
<frameworkElement Style="{StaticResource styleResourceKey}"/>

This is a systemic problem. I think the docs team should advise on how to fix.

There'a a few issues here. One seems to be using a placeholder value, "bool", in place of a literal value or a set of values.  Is it not clear that "bool" is a placeholder? The properties values section that follows on that page describes the possible bool values, True and False.
We also have an optional xaml values section that will appear when the system moves to the new rendering engine. The xaml values is used when the possible values for a xaml attribute are more complex, like in the case of the Path shape.

Is the intent that this section be sometimes placeholder, sometimes example, sometimes both? Maybe indicate the placeholder case, for example italicize.

The intent is that it is a placeholder. Examples would be in the Examples section, not in the Syntax section. Italicizing is a good idea, but we can't do that because it's in a markdown code block.

Maybe brackets? I've found these sections confusing in the past too.
I see where people have made this section into example code too, like FrameworkElement.Style:
<frameworkElement Style="{StaticResource styleResourceKey}"/>

(Also, is there a reason why the "f" in  is lower case?)

We can test some different options like brackets and see if anything works well.
The lowercase "f" is to indicate that you'd be setting this property on something that inherits from FrameworkElement, but you would not be putting <FrameworkElement .../> in your XAML.
The FrameworkElement.Style example you give isn't intended to look like a copy-and-paste example, but there's important contextual information that is currently not shown (even though it still exists in the markdown). We're working to fix this. This archived version of the doc shows what it's meant to look like. Notice the 'Syntax' header and the 'XAML Values' section.

Thanks for taking the time to let us know about this issue and we sincerely apologize for the delayed response.
As this issue is a result of auto-generated global syntax from our publishing system, we've created an internal work item to triage and address across all WinRT API docs.


Websocket binaryType setting

Websockets have a property called binaryType that takes a value of blob (default) or arraybuffer. In my app, I have to change it to arraybuffer, but there is no simple mechanism in WebSocketSubject to do this.
For now, I'm setting the property once the socket has been opened. For instance:
  let subject = Observable.webSocket({
    url: '',
    openObserver: {
      next: e => { e.currentTarget.binaryType = 'arraybuffer' }
    }
  })

This seems to work, but I'm worried there could be a race condition where data is sent or received before the binaryType is set. When this value is not set correctly, TypeError: First argument to DataView constructor must be an ArrayBuffer is thrown.
I propose that a binaryType property be accepted in the config object passed to Observable.webSocket. It will immediately set the property on the socket instance as soon as it is created:
        try {
            socket = this.protocol ?
                new WebSocketCtor(this.url, this.protocol) :
                new WebSocketCtor(this.url);
            this.socket = socket;
            if (this.binaryType) {
                this.socket.binaryType = this.binaryType
            }
        }

Thoughts?

It seems like we should expose this via configuration object.

Awesome, that was fast!


Make truncating the debug.log file an option.

Useful for developers who need to refer to futher back in debug.log history, but who don't want to
enable the -debug option and all the verbosity that comes with that.
This implements 1396

Gitian built successful and works exactly as expected. LGTM


rename pontusx_test to pontusx

As discussed on slack, the _test is indicative of the paratime being on testnet, and pontusx alone might be preferable

Hm I'd still prefer to get this merged; the Nexus issues have been resolved and this PR will allow Nexus to support a mainnet pontusx runtime with a simple config change when that time comes.


One problem about the definition of Vendor ID in Matter Specification Version 1.3 documentation.

Build issue(s)
Documentation issues
Page 828 in Matter Specification Version 1.3 documentation has the definition in the figure for AdminVendorID Field:

I would like to know what is mentioned here "This value SHALL NOT be one of the reserved Vendor ID values defined in Table 1, 'Vendor ID Allocations '."

How to understand it?
And why can't I use the Vendor ID in Table 1?

Page 53 in Matter Specification Version 1.3 documentation has the Table 1, and all Vendor IDs in Table 1 are "reserved" as described in the documentation:

I would like to know what happens if the Vendor ID from Table 1 is used in the AdminVendorID Field?

Thank you very much for your answering!
Platform
No response
Anything else?
No response
Platform
No response
Anything else?
No response

I used the CHIP Tool's --commissioner-vendor-id parameter for the test of Vendor ID definition and I can specify any value for the Vendor ID. So I don't understand what the documentation means by "SHALL NOT be one of the reserved Vendor ID values defined in Table 1".

I think the intent is that it can't be 0 or one of the test values; it has to be an actual vendor id identifying the commissioner....   But either way, spec issues need to go in the spec repository (unfortunately not public).
I filed https://github.com/CHIP-Specifications/connectedhomeip-spec/issues/9212 to track this; thank you for raising it!

Thank you! If there is any progress, I would appreciate it if you could reply here with an explanation of the documentation because of the not public spec issues. :)

I'll try, but I am not following the spec closely...


How to click on an element by name?

Hello,
How can I click on a element by name, for example

I want to click on button with data="success"
Thanks,

You can use Attribute selectors, like [data="success"]. Example with clicking on a link:
'use strict';

const puppeteer = require('puppeteer');

(async function main() {
  try {
    const browser = await puppeteer.launch({ headless: false });
    const [page] = await browser.pages();

    await page.goto('https://example.org/');

    const element = await page.$('[href="http://www.iana.org/domains/example"]');
    await element.click();
  } catch (err) {
    console.error(err);
  }
})();


when scheduler changes temperature, while windows is open - thermostat just sheepishly listens.

Pretty simple.
Create a phoney window sensor you can control by hand (or use a spare door sensor) and assign it in configuration.
Set the temp on the thermostat to what ever you want.
Set the schedule to change the temperature / set preset in next few minutes.
"open the window"
Now when the schedule kicks in one would assume that the window override will be still preserved, but nope - the schedule just sets the temp and radiator can successfully heat the neighbourhood :)
After you close the window, the old temp gets set back.
Tested with with frost protection preset and OFF.
Yeah, I know, me and my silly issues.

Hello @tomtom13 ,
You didn't respect the template for issue. So sad, I cannot see your parameter. I need to know what is your window tuning ?

Default values.

Turn to OFF when window is open ?

I suppose yes. When changing preset or target temp when the window is open, the preset or target temp change but the VTherm is still turned off.
When you close the window, the Vtherm turns on and then the new target or preset is used.
This is totally wanted and a good behavior on my opinion: the scheduler does its life, the VTherm also and when VTherm turns on, the last Scheduler command is applied.
Agree ?

Disagree, because radiator start immediately heating even thou the window is open.
The other parts are "nice to have" but the fact that scheduler can override the open window thing is a bit daft.

OK, my presets are setup to do "Auto @ desired_temp". IF that is undesired, I might have missed that in the documentation (no, it's not a snarky "it's not there", your documentation is pretty big and I might have missed that part).
But at least you've pointed out the culprit.


[defaultLocale] Cannot switch language to defaultLocale

Description
Config
export const routing = defineRouting({   locales: ["en", "es"],   defaultLocale: "en",   localePrefix: {     mode: "as-needed",   }, });

open the site not under "/es" and try to switch to "en". Basically whatever you will set to default, user can't to switch to lang that in under defaultLocale

Here the swither
....
  const currentLocale = useLocale();
  console.log("currentLocale", currentLocale);
  console.log("pathname", pathname);

<Select
...   onChange={(e) => {
          const newLocale = e.target.value as Locale;
          console.log("newLocale", { from: currentLocale, to: newLocale });
          console.log("locale", { locale: newLocale });

          router.replace(pathname, { locale: newLocale });
        }}
...

Verifications

[X] I've verified that the problem I'm experiencing isn't covered in the docs.
[X] I've searched for similar, existing issues on GitHub and Stack Overflow.
[X] I've compared my app to a working example to look for differences.

Mandatory reproduction URL
https://d10cir56sw4n5d.cloudfront.net/de
Reproduction description
Config
export const routing = defineRouting({   locales: ["en", "es"],   defaultLocale: "en",   localePrefix: {     mode: "as-needed",   }, });

open the site not under "/es" and try to switch to "en". Basically whatever you will set to default, user can't to switch to lang that in under defaultLocale

Here the swither
....
  const currentLocale = useLocale();
  console.log("currentLocale", currentLocale);
  console.log("pathname", pathname);

<Select
...   onChange={(e) => {
          const newLocale = e.target.value as Locale;
          console.log("newLocale", { from: currentLocale, to: newLocale });
          console.log("locale", { locale: newLocale });

          router.replace(pathname, { locale: newLocale });
        }}
...

"next-intl": "^3.24.0",
Expected behaviour
User can switch to defaultLocale

          router.replace(pathname, { locale: newLocale });
          router.refresh(); // this one fix the issue

Please refer to the examples, which have working locale switchers. This is also covered in the docs.
I'm moving this to a discussion since it's a usage question.


Possible to login into website?

Hi!
What I'm asking is, I'm making a small REST API for a website, and I was wondering, is it possible to do a POST to a login page, and save the session cookie to that website, and keep being logged in for the rest of the session?
Hope it makes sense!

Hey there. I'm pretty sure it is possible in this way:
cloudscraper.post('http://site.com/login', {login:'test', password:'test'}, function(err, body, response) {
  console.log(response.request.headers.cookie);
});

Let me know if this works fine for you. If not, we will find some other solution :)


[Extensibility Request] Table 472 "Job Queue Entry" - Procedure from Local to Global

Hello, we need the following currently Local procedure to be set as Global. Can you please check this? Thank you!
procedure CleanupAfterExecution()

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 45938.


Use layout-name in new-project

Current: ./proton_cli new-project <name> <layout-id>
Desired: ./proton_cli new-project <name> <layout-name>
Reason: We want to reference layouts by name, not id, to make it easier for the user. Update command and functionality

The SQL commands might need to wait to be changed until the database overhaul happens


Facebook Graph API v2.4

Facebook updates it API recently,
it seems that the way to get extra scope is not working now.
e.g. setting SOCIAL_AUTH_FACEBOOK_SCOPE = ['email']
does not return the email.
here's the link to graph API v2.4
https://developers.facebook.com/blog/post/2015/07/08/graph-api-v2.4/?ref=hp

+1

+1

+1

I found solution:
you need to request email field explicitly to retrieve it from Facebook, so just add this to your settings:
SOCIAL_AUTH_FACEBOOK_PROFILE_EXTRA_PARAMS = {
    'fields': 'id,name,email', # needed starting from protocol v2.4
}

Thanks for your solution.
How I can recover the email users who are already registered before the solution?

Thanks highcat. Your solution seems to be working.

Thanks!

Thanks!

:+1:

Thanks! Please, update the docs.

and twitter? email = "" ?

I tried the SOCIAL_AUTH_FACEBOOK_PROFILE_EXTRA_PARAMS, didn't work for me.
Has the API been changed again?

If I work with facebook to twitter the issue is not cast me email

I updated SOCIAL_AUTH_FACEBOOK_PROFILE_EXTRA_PARAM, but doesnt seem to work.
Do we need to update API version in https://github.com/omab/python-social-auth/blob/master/social/backends/facebook.py ?

SOCIAL_AUTH_FACEBOOK_PROFILE_EXTRA_PARAMS works for me on Facebook, but doesn't work for Twitter. Any ideas on how to get the e-mail?

As I remember, there was NO way to get email from twitter, Twitter API won't return it (and everybody was frustrated about that).
Check it's API first, maybe things changed.

@highcat. I found this: http://stackoverflow.com/questions/22627083/can-we-get-email-id-from-twitter-oauth-api
It migh be worth trying. I have submitted a request. Let's see what happens.

@christianmls the next time user logs in with facebook their email will be added to profile.

Hi!
A work arround using Django 1.10, Python Social Auth 0.2.21 and Requests 2.11.1 to get email of user with facebook strategy:
@partial
def require_email(strategy, details, user=None, is_new=False, *args, **kwargs):
    if user and user.email:
        return
    elif is_new and not details.get('email'):
        if strategy.request_data().get('email'):
            details['email'] = strategy.request_data().get('email')
            return

        if strategy.request.session.get('email', '') != '':
            details['email'] = strategy.request.session['email']
            return

        fbuid = kwargs['response']['id']
        token = kwargs['response']['access_token']
        url = 'https://graph.facebook.com/{0}/?' \
              'fields=email&access_token={1}'.format(fbuid, token)
        resp = requests.get(url)
        if resp.json().get('email'):
            details['email'] = resp.json().get('email')
        else:
            return redirect('baid:email-required')

And in settings:
SOCIAL_AUTH_PIPELINE = (
        #...
        'social.pipeline.social_auth.social_user',
        'myapp.pipeline.require_email'
        #...
)

It works for for facebook but not for LinkedIn. What is the solution for LinkedIn?

@harun1393 Hi Harun, could you provide the stack trace?

Hi, My problem solved for LinkedIn also. I have done some mistake to write settings

Facebook API is already updated, currently version is 2.8.

Thank you
2016-12-27 3:03 GMT+06:00 Matías Aguirre<EMAIL_ADDRESS>
Facebook API is already updated, currently version is 2.8.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/omab/python-social-auth/issues/675#issuecomment-269241641,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJGe41m-jxSSiKwgNAxRmXwYLRlgUKLAks5rMCuRgaJpZM4FWB96
.

--
হারুন অর রশিদ
স্বেচ্ছাসেবক, এফওএসএস বাংলাদেশ

Hi, How get email in Facebook API v3.1?
Didnt work (((
SOCIAL_AUTH_FACEBOOK_PROFILE_EXTRA_PARAMS = {
'fields': 'id, name, email',
}


Remap controls for standalone mupen64.

I need assistance remapping the controls for the standalone mupen64 emulators. I can't find any place that provides instructions on doing so.

https://github.com/christianhaitian/arkos/wiki/Key-codes-and-Global-Hotkey-config-file-locations#global-hotkey-config-file-locations-6


fixing base path link

fixing base path link in yaml

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


How to generate lua code according to ast ？

Recursively. Walk depth-first through the AST, generate code for each node and glue it together. Figure something out.


Improved copying and selection behaviour of array editor

This replaces https://github.com/spyder-ide/spyder/pull/2579 (so close that request if you're happy with this one).
The justification remains the same:
The reason for this change is that if you went to the array editor and used the top left square to select everything, it only selected the currently loaded data. For large arrays that was a subset of the whole thing (because extra data is fetched on demand). It wasn't obvious that it had done this unless you scrolled to check what it had selected, or copied the selection and found it was smaller than expected.
I've changed two things:

When an array is copied and the selected ranges reach the edges of the currently loaded data, the range copied goes to the end of the array, not the end of the loaded data.
When a whole row or column is selected and more data is loaded into the array editor, the selection is expanded too (which I think is intuitive).

The advantage over the original pull request is that it no longer suddenly loads the whole array into the editor (which could use large amounts of memory)

Thanks a lot for this and for taking the time to improve on your previous idea.
I'm very +1 to your contribution, and I'll try to review it during the weekend :-)


documentation: switch to m2r2 for Markdown support; update requirements.txt

Summary
This change allows including Markdown files outside of the Documentation directory. To do so they can be included in the toc or inline with: mdinclude directive.
Impact
Documentation
Testing
Tested including a Markdown file and displayed correctly

Can you update the pipenv file as well.

I was about to ask how to do that

pipenv install <package name> should add it.
Otherwise recreate it from the requirements.txt pipenv install -r requirements.txt

Ok, I just force pushed the Pipfile change.
What about the .python-version file? Should this be pulled in or added to .gitignore?

What about the .python-version file? Should this be pulled in or added to .gitignore?
We should drop that.  The rest looks good though.

Ok, just added to gitignore.


Update constants section to talk about inlining

https://gleam.run/book/tour/constants.html

When a constant is referenced the value is inlined by the compiler

This is true for Erlang, but not for JavaScript, and it would probably make more sense to update this documentation than the change the JavaScript generation, because there are actually a lot of reasons why JavaScript constant inlining would be bad.

Yup yup, this is outdated documentation. Thank you!


Test

in this pull request, i wrote test to cover all endpoints in the application.

Everything seems to be good, we can merge now


Lime-210, Redundant Calculation of Collateral Tokens

Description
The liquidate in CreditLine computes twice the amount of collateral tokens, in function calls calculateCurrentCollateralRatio and calculateTotalCollateralTokens.
Integrations Checklist

[x]  Have any function signatures changed? If yes, outline below.
[ ]  Have any features changed or been added? If yes, outline below.
[ ]  Have any events changed or been added? If yes, outline below.
[ ]  Has all documentation been updated?

Changelog

Returning totalCollateralTokens along with currentCollateralTokens to avoid multiple calculations.

Event Signature Changes
Function Signature Changes

function calculateCurrentCollateralRatio(uint256 _id) public ifCreditLineExists(_id) returns (uint256, *uint256*)

Features
Events

Codecov Report

Merging #235 (6566354) into master (6aaf59a) will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##           master     #235   +/-   ##
=======================================
  Coverage   15.95%   15.95%
=======================================
  Files          16       16
  Lines        1736     1736
  Branches      321      321
=======================================
  Hits          277      277
  Misses       1459     1459

Impacted Files
Coverage Δ

contracts/CreditLine/CreditLine.sol
9.82% <0.00%> (ø)

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 6aaf59a...6566354. Read the comment docs.


P2G?

I believe old versions of Phonetisaurus had a --swap flag that could be used for P2G. Is this still possible?
http://wiki.phonetisaurus.googlecode.com/hg-history/01f8ea06f6591cf74d8ef4fd156cd27728ec9bd5/QuickStartExamples.wiki

Although the main focus in on Grapheme-to-Phoneme conversion, the approach that Phonetisaurus employs is symmetric with regard to the input and output symbols, thus it is relatively trivial to also construct Phoneme-to-Grapheme models using the same approach with albeit swapped inputs and outputs.
== Forward P2G ==
A Phoneme-to-Grapheme model can be generated from the same input dictionary by simply flipping the '--swap' flag, which will swap the positions of the pronunciations and words in the training data.
$ ./train-model.py \
         --dict ../data/g014a2.train.bsf \
         --order 9 \
         --prefix test/test \
         --delX --maxX 2 --maxY 2 \
         --maxFn joint \
         --smoothing FixModKN
         --swap

and this model can be similarly tested with the following command.
$ ../phonetisaurus-g2p \
         -m test-swap/test-swap.fst \
         -t ../data/g014a2.clean-swap.test \
         --sep " " | ./eval-test.pl
Total: 4998
Corr:  2975
WER:  0.404761904761905

For the moment the best bet would be to use the actual older version.  These are still available in the defunct downloads section in googlecode.  We can see about adding them back in a bit later.

https://code.google.com/archive/p/phonetisaurus/


How can i invite someone?

I saw some functions with invite and comment: " Invite users to app. but this only works when you have a leftover invitation.".
So what I need to do?

Well, you need to have unused invitations
client.me()  ->  'num_invites': 0,
If you have some invites, you just call client.invite_to_app(name='Alex',phone_number='+79681112233')


Timestamp Component Overflows Card in IssueItem in recrutadores

Towards #116

is this an legitimate issue because those small and minor changes were done to rectify small imperfections in the ui? @guibranco

LGTM! Approved.
Thanks for the effort @AnuravModak


Information outdated

Unable to find the Principle listed when trying to use my own certificate.  It says "No results found".

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

ID: beac0740-af6e-61ce-bcfa-e783ca181b7d
Version Independent ID: d97885ea-b051-5fde-bd14-ec977e34760e
Content: Tutorial: Configure HTTPS on an Azure CDN custom domain
Content Source: articles/cdn/cdn-custom-ssl.md
Service: azure-cdn
GitHub Login: @duongau
Microsoft Alias: duau

@demuire, Thanks for your feedback. We will review this and get back to you shortly!

Thanks for the reply - it turns out I missed a step, and I've managed to get it working now!Sent from my iPhoneOn 12 Aug 2022, at 2:00 pm, Kapil Ananth @.***> wrote:
@demuire, Thanks for your feedback. We will review this and get back to you shortly!
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>

Thanks for the update @demuire.
I shall now archive this thread.


Don't really push silence to the ring buffer

This implements your idea, only by removing code it turns out.

I've tested it on a few devices on my macbook 2018: built-in speakers/mic, an hd25 plugged in the TRRS socket(it doesn't have a mic), and a Sony WH-1000 xm3, that has a mic, via bluetooth, and trying all sorts of combinations and switches.

Add a reference: This fixes BMO 1629839.


[searchguard branch] elasticsearch/config/sg should be mounted?

Shouldn't the searchguard config dir (elasticsearch/config/sg) mounted from host? Spent hours updating the permissions and running init_sg.sh only to realize that it's not taking effect since it's not mounted

It should be possible to mount that directory instead of baking it inside the image indeed.

So, mount via compose file under elasticsearch service as most effective solution?

Yes that sounds reasonable. If you believe it makes things easier let's go for it.


config: Require strictly-positive timeout values

If the timeout value was zero, the hook would always error, and there doesn't seem to be much point to that.  And I'm not sure what negative timeouts would mean.  By adding this restriction, we do not limit useful hook entries, and we give the validation code grounds to warn users attempting to validate configs which are poorly defined or have useless hook entries.
Removing the pointer is not strictly required, but keeps up with our anti-pointer zero-value style now that Go's zero-value is clearly invalid.

LGTM

I don't think the pointer should be removed here

I don't think the pointer should be removed here

Because?  We removed similar in unnecessary pointers in #653.

This was done for similar reasons (which I presume are nulls being supported) in image-spec recently, it seems like a valid place here: https://gist.github.com/eef405c950a123c1b178aa706e935139
Just seagulling, hope it's ok.

On Thu, Apr 13, 2017 at 08:26:40AM -0700, Erik Hollensbe wrote:

This was done for similar reasons (which I presume are nulls being
supported) in image-spec recently, it seems like a valid place here:
opencontainers/image-spec#633

In that case (and also in opencontainers/image-spec#625 and
opencontainers/image-spec#626), you need a pointer for the optional
property, because of golang/go#11939.  However, in this case ‘Timeout’
is an int, and (unlike structs) those do have zero values that
omitempty supports.  So the non-pointer int property works, and is in
line with our current policy 1 (since #653 / #656).  Maybe the
policy is closer to:
New properties should not have pointer Go types when the zero value
works with omitempty and is a no-op or invalid.  For existing
properties, maintainers will decide on a case-by-case basis with no
recognizable pattern, so don't worry too much about trying to guess
what they want.
;)

You cannot apply just blanket rules to everything, its not all 1 or 0s.  If you remove the pointer, and a 0 is invalid, you make every config invalid because 0 is the default value.  So how are we supposed to validate this at all without a pointer on the int when it will always be 0?

LGTM

LGTM


GetConn waiting queue as FIFO queue

Hi, we noticed that under higher load that get_conn() requests get served out-of-order resulting in high tail latencies. This happens when a queued waker gets called, but before the GetConn is polled a newer GetConn grabs the available db connection. On top of that, the previously queued GetConn gets pushed to the back of the waiting queue, adding more latency.
Would you consider turning the waiting queue in Exchange into a FIFO queue? I have a patch that we tested in prod and it shows much lower tail latencies for us. See #213.

I wonder if this could be done more easily by marking connections as reserved. Because the main problem is that woken GetConns don't get the expected connections. So maybe we can simply move a connection from available: Vec to a new reserved: Vec and woken GetConns from there.

Hi. Hmm, hard to say, really.. Reserved connection may end up being dead, this could add to complexity..
Btw, I've updated your PR, could you please look at it? The main idea is to keep queue position for caller whose connection end up being dead and to tidy up the Future impl.

Yes, I looked at adding a reserved: Vec to Exchange earlier, but keeping track of where things are makes this pretty complex, true.
I looked at your commit. Looks great! Much simpler and covers the case when a connection is bad.
What are the next steps? Do you need me to do anything?


Use registry from PackageBundleController

The packages should come from where the bundle comes from by default.

/lgtm


Set/Unset Before/After steps

Description
Ability to modify before/after steps through the CLI
Fixes/Implements: #166
Checklist

[✅] I have added tests that prove my fix is effective or that my feature works
[✅] I have update README (If needed)

Please also check Codacy - bot reviews

@gopinath-langote where can I see codacy review?

https://app.codacy.com/manual/gopinath-langote/1build/pullRequest?prid=4307405

@gopinath-langote should be fixed now ;)

@duderman LGTM.


Another way of handling ZEPPELIN-547

@prabhjyotsingh How about this way? i think it's more simple way to fix ZEPPELIN-547. Please take a look.

Yes, this is definitely a better fix.

Yes, this is definitely a better fix.


Incorrect Endpoint Generated

Good day,
I have this configuration:
var nodemailer = require('nodemailer');
var ses = require('nodemailer-ses-transport');

var tp = nodemailer.createTransport(ses({
  accessKeyId: 'XXXXX',
  secretAccessKey: 'YYYY',
  region: 'us-west-2'
}));

I might be doing something wrong, but that gives me an error:
The Canonical String for this request should have been
'POST
/

host:email.us-west-2.amazonaws.com
x-amz-content-sha256:xxxxxxxxxxxxxxxxxxxxxxx
x-amz-date:20160916T122437Z

host;x-amz-content-sha256;x-amz-date
xxxxxxxxxxxxxxxxxxxxxxx'

In my AWS SES account, the endpoint is actually email-smtp.us-west-2.amazonaws.com, which is different from host value generated above.
I tried adding this line in my transport config:
ServiceUrl: 'email-smtp.us-west-2.amazonaws.com',

but the result is the same.
Hoping to get some advise on this. Thanks.
In the meantime, I'm just using the standard nodemailer (no plugin) format of 'smtps://user:pass@endpoint', which works fine.

I ran into this too, and in order to get things functional ended up using:
var transport = nodemailer.createTransport({
        transport: "smtp",
        host: "email-smtp.us-west-2.amazonaws.com",
        secureConnection: true,
        port: 465,
        auth: {
            user: config.app.ses_accessKey,
            pass: config.app.ses_secretKey
        }
    });

Is the SES transport just fundamentally broken?

@eberls Thanks for the updated. I was facing the same issue. Your solution worked for me


[BC Break] Drop built-in code scanners for 4.0

There is a discussion in the PR #123 about possible removing of low-level scanners in the Zend\Code\Scanner subnamespace for 4.0. Reason for that is to avoid keeping support for low-level code analysis via tokens inside scanners, this job can be performed by external libraries, such as nikic/php-parser, roave/better-reflection, goaop/parser-reflection, etc.
Some facts about Scanner code usage in the project https://github.com/zendframework/zend-code/pull/123#issuecomment-382443064:

There are only dependencies on Scanner namespace in the Reflection namespace, eg. Zend\Code\Reflection\DocBlockReflection::reflect uses DocBlockScanner class, as well Zend\Code\Reflection\FileReflection::reflect uses CachingFileScanner. All remaining integration parts could be removed as they unused.
Classes from the Reflection namespace adds some public API to the PHP internal classes, thus Reflection namespace could not be easily dropped if we want to support *Generator::fromReflection() methods that accept only Zend\Code\Reflection classes and use specific methods from reflection.
If we decide to drop fromReflection() method from all generators or replace typehint in them with built-in PHP classes, then whole Scanner and Reflection namespace could be dropped, but it will be significant BC break as we remove all functionality that are not related directly to the code generation.

@Ocramius suggestions from https://github.com/zendframework/zend-code/pull/123#issuecomment-382547482:

Whole Scanner namespace could be removed, there are only dependencies in Reflection namespace, eg. Zend\Code\Reflection\DocBlockReflection::reflect uses DocBlockScanner class, as well Zend\Code\Reflection\FileReflection::reflect uses CachingFileScanner. All remaining integration parts could be removed as they unused.

Yeah, this stuff can be dropped. At this point, pointing to roave/better-reflection for most of the "brute-forcing the filesystem, looking for symbols"

Classes from the Reflection namespace adds some public API to the PHP internal classes, thus Reflection namespace could not be easily dropped if we want to support *Generator::fromReflection() methods that accept only Zend\Code\Reflection classes and use specific methods from reflection.

Not sure about this one. I think the initial idea was that ext-reflection had huge potholes, and the library tried to fill them. I'd keep this for a separate issue, to figure out if the Zend\Code\Reflection namespace actually does anything more than ext-reflection these days. Considering that Zend\Code\Reflection is the entry point to all of the usages of this library (including the generators), I think that would make the migration path too painful at first.

If we decide to drop fromReflection() method from all generators or replace typehint in them with built-in PHP classes, then whole Scanner and Reflection namespace could be dropped, but it will be significant BC break as we remove all functionality that are not related directly to the code generation.

The ::fromReflection() constructors are extremely useful to mimick API without rewriting all of it manually, so I'd probably keep them in place. Instead, I think that allowing core reflection classes as parameters would be a better solution here.

I like the idea to drop both Scanner and Reflection namespaces from the code.

Let's start with the Scanner only for now.

Originally posted by @lisachenko at https://github.com/zendframework/zend-code/issues/155

Yeah, this stuff can be dropped. At this point, pointing to roave/better-reflection for most of the "brute-forcing the filesystem, looking for symbols"

@Ocramius So, we are going to drop both DocBlockReflection and FileReflection classes, right? Or keep them but use external libs? For example, DocBlockReflection contains sensitive logic for accessing tags, however we can mimic this logic via optional dependency phpdocumentor/reflection-docblock that provides good API for that.

The ::fromReflection() constructors are extremely useful to mimick API without rewriting all of it manually, so I'd probably keep them in place. Instead, I think that allowing core reflection classes as parameters would be a better solution here.

Ok, I will check if it's possible to switch to the core classes without pain.

Let's start with the Scanner only for now.

Ok.

Originally posted by @lisachenko at https://github.com/zendframework/zend-code/issues/155#issuecomment-382705649

So, we are going to drop both DocBlockReflection and FileReflection classes, right?

I think so, yes.

Originally posted by @Ocramius at https://github.com/zendframework/zend-code/issues/155#issuecomment-382767977

Handled in #58 - DocBlockReflection will live on for some time, but probably dead by 5.0.0


refactor!(dialog): change type field in JS to kind

Fix #944
Before
await invoke("plugin:dialog|confirm", {
	message: "1",
	type_: 'error',
});

After
await invoke("plugin:dialog|confirm", {
	message: "1",
	type: 'error',
});

I think the better choice here is to rename the type field to kind to match the Rust API.

Yes, if we don't mind breaking the old API, I'm fine with both ways 😀

Thanks for your contibution btw.


[Quick Start with Kubernetes] Use Helm and Tiller to manage the Istio deployment fails

Describe the bug
When following option 4: Use Helm and Tiller to manage the Istio deployment. step 3 fails.
$ kubectl apply -f install/kubernetes/helm/helm-service-account.yaml
serviceaccount/tiller created
clusterrolebinding.rbac.authorization.k8s.io/tiller created

$ helm init --service-account tiller
$HELM_HOME has been configured at /usr/local/google/home/yinjie/.helm.
Warning: Tiller is already installed in the cluster.
(Use --client-only to suppress this message, or --upgrade to upgrade Tiller to the current version.)
Happy Helming!

$ helm install install/kubernetes/helm/istio-init --name istio-init --namespace istio-system
Error: release istio-init failed: namespaces "istio-system" is forbidden: User "system:serviceaccount:kube-system:default" cannot get namespaces in the namespace "istio-system"

Expected behavior
Install the istio-init chart to bootstrap all the Istio’s CRDs should succeed.
Steps to reproduce the bug
{{ Minimal steps to reproduce the behavior }}
Version
$ istioctl version
version.BuildInfo{Version:"1.1.0-rc.0", GitRevision:"3632361b439ae27962f0af472064d452fc63273b", User:"root", Host:"9efe8d69-34c2-11e9-b805-0a580a2c0506", GolangVersion:"go1.10.4", DockerHub:"docker.io/istio", BuildStatus:"Clean", GitTag:"1.1.0-snapshot.6-26-g3632361"}

$ kubectl version
Client Version: version.Info{Major:"1", Minor:"13", GitVersion:"v1.13.3", GitCommit:"721bfa751924da8d1680787490c54b9179b1fed0", GitTreeState:"clean", BuildDate:"2019-02-01T20:08:12Z", GoVersion:"go1.11.5", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"11+", GitVersion:"v1.11.6-gke.2", GitCommit:"04ad69a117f331df6272a343b5d8f9e2aee5ab0c", GitTreeState:"clean", BuildDate:"2019-01-04T16:19:46Z", GoVersion:"go1.10.3b4", Compiler:"gc", Platform:"linux/amd64"}

Installation
istio-1.1.0-rc.0-linux.tar.gz.sha256
Environment
Google Kubernetes Engine

@ostromart thinks he can handle this easily, but if it proves to be more complicated we will probably reassign to @howardjohn

It could be a case of the RBAC not yet having taken effect when you run helm install. Could you please try the last command again if your cluster is still in the same state, or repeat the install and wait for a while before helm install?
You should also run helm init --upgrade to update your tiller pod as it looks to be quite a long way behind the client.

Warning: Tiller is already installed in the cluster

If tiller is already present, these instructions do not need to be run.  This is more or less working as intended.

The warning is expected but the main complaint here is the error further down:
$ helm install install/kubernetes/helm/istio-init --name istio-init --namespace istio-system
Error: release istio-init failed: namespaces "istio-system" is forbidden: User "system:serviceaccount:kube-system:default" cannot get namespaces in the namespace "istio-system"

@ostromart that means tiller was not installed with an appropriate service account.  Many people don't instantiate tiller properly.  This is not really a code problem - perhaps this could be handled with documentation.

@sdake I wonder if this should be reassigned to you to send a doc PR?   Got the bandwidth?

Just did some direct debugging with @JimmyCYJ and I think we are good. He will update.

I solve this problem by running
$ helm init --service-account tiller --upgrade
$HELM_HOME has been configured at /usr/local/google/home/yinjie/.helm.

Tiller (the Helm server-side component) has been upgraded to the current version.
Happy Helming!

$ helm install istio.io/istio-init --name istio-init --namespace istio-system
NAME:   istio-init
LAST DEPLOYED: Mon Mar  4 14:55:58 2019
NAMESPACE: istio-system
STATUS: DEPLOYED

$ kubectl get crds | grep 'istio.io\|certmanager.k8s.io' | wc -l
59

$ helm install istio.io/istio --name istio --namespace istio-system
NAME:   istio
LAST DEPLOYED: Mon Mar  4 14:56:35 2019
NAMESPACE: istio-system
STATUS: DEPLOYED

Thanks for the help @ostromart

Doc instructions say this:
Verify that all 58 Istio CRDs were committed to the Kubernetes api-server using the following command:

$ kubectl get crds | grep 'istio.io\|certmanager.k8s.io' | wc -l
58

I think this is now a doc bug, albeit not a high priority one.

This is now fixed with latest stuff.

I solve this problem by running
$ helm init --service-account tiller --upgrade
$HELM_HOME has been configured at path-to-helm.helm.

Tiller (the Helm server-side component) has been upgraded to the current version.
Happy Helming!

$ helm install istio.io/istio-init --name istio-init --namespace istio-system
NAME:   istio-init
LAST DEPLOYED: Mon Mar  4 14:55:58 2019
NAMESPACE: istio-system
STATUS: DEPLOYED

$ kubectl get crds | grep 'istio.io\|certmanager.k8s.io' | wc -l
59

$ helm install istio.io/istio --name istio --namespace istio-system
NAME:   istio
LAST DEPLOYED: Mon Mar  4 14:56:35 2019
NAMESPACE: istio-system
STATUS: DEPLOYED

it also solved my problem


Build errors in Xcode b5

Hello, thanks for the report. Looks like there've been yet more API changes in Swift/ASL interoperability.
Anyway, I've got an xcode7-beta5 branch available that should resolve the issue.
Please let me know how it goes, and I'll merge it into master once it's confirmed that the build works.

Building the branch xcode7-beta5 produces the following:
*** Fetching CleanroomLogger
*** Fetching Result
*** Fetching CleanroomASL
*** Checking out CleanroomLogger at "3afe5b31d7d90c315363bab3f093d5bf01337254"
*** Checking out CleanroomASL at "1.2.3"
*** Checking out Result at "0.5"
*** xcodebuild output can be found in /var/folders/pg/2jn2s0qn5dsc56cb_l0l_flr0000gn/T/carthage-xcodebuild.MUKMUC.log
*** Building scheme "CleanroomASL iOS" in CleanroomASL.xcodeproj
** BUILD FAILED **

The following build commands failed:
	CompileSwift normal x86_64
	CompileSwiftSources normal x86_64 com.apple.xcode.tools.swift.compiler
(2 failures)
/Users/glebd/git/cloner/Cloner/Carthage/Checkouts/CleanroomASL/Code/ASLObject.swift:405:143: error: value of optional type '[CChar]?' not unwrapped; did you mean to use '!' or '?'?
/Users/glebd/git/cloner/Cloner/Carthage/Checkouts/CleanroomASL/Code/ASLClient.swift:129:44: error: value of optional type 'String?' not unwrapped; did you mean to use '!' or '?'?
/Users/glebd/git/cloner/Cloner/Carthage/Checkouts/CleanroomASL/Code/ASLClient.swift:129:66: error: value of optional type 'String?' not unwrapped; did you mean to use '!' or '?'?
A shell task failed with exit code 65:
** BUILD FAILED **

The following build commands failed:
	CompileSwift normal x86_64
	CompileSwiftSources normal x86_64 com.apple.xcode.tools.swift.compiler
(2 failures)

Thanks for pointing that out, Gleb. I neglected to update the Cartfile. Please try again with the xcode7-beta5 branch. It should be good to go now.

Seems to work fine now, thanks!


Faster-Whisper

Any really good reason to use whisper module instead of faster-whisper?
faster-whisper have auto language recognition, and it's 5x faster for Base model in CPU (I didn't calculate speed for other models, but it's around 20 seconds against 70 seconds for Medium size also in CPU)
I can fork and send a PR tomorrow, but I would be satisfied to hear why you didn't choose faster-whisper as first option to this project.

https://github.com/Uberi/speech_recognition/pull/800/commits/e19727b05aa56808250c6082130f4d27b01a5d52


When uninstalling an extension, ask to also uninstall extensions that depend on it

A user reported that he "Cannot uninstall 'Language Support for Java(TM) by Red Hat' extension. 'Java Test Runner' extension depends on this.", but it's only one extension among many that depend on Java.
Rather than manually hunting down  those extensions to uninstall first (which itself is a daunting task, see #119242), the warning popup should provide a button to uninstall all those other extensions (and their dependants) as well.
See https://github.com/redhat-developer/vscode-java/issues/1837

I‘d like to handle this enhancement

For me the notification lists dependants.
#104997 added the ability to disable the dependants directly from the notification that appears when disabling (rather than uninstalling). Sounds like we want the same on the uninstall.

the notification lists 2 dependents at most. This is insufficient data.

the notification lists 2 dependants at most. This is insufficient data.

Fair point. But as a workaround you can presumably uninstall the two shown, then try again on the main extension and find out which two (or possibly one) to uninstall next.

Yes, but I get @fbricon‘s point. Installing every single extension by hand, and rechecking dependencies after removing the two removed extensions by may be kind of annoying.
You‘re trying to make the Pop-up show all dependent extensions, rather then just showing only two, correct?

You‘re trying to

I am unclear who "you" is.

I was referring to @fbricon

I think we provide Disable All button when user tries to disable an extension and there are dependents. We shall do the same for Uninstall

I‘d love to work on this then! :)

We closed this issue because we don't plan to address it in the foreseeable future. If you disagree and feel that this issue is crucial: we are happy to listen and to reconsider.
If you wonder what we are up to, please see our roadmap and issue reporting guidelines.
Thanks for your understanding, and happy coding!

To verify:

Install Java Test Runner vscjava.vscode-java-test extension
Uninstalling Language Support for Java extension should show a dialog asking to also uninstall the Java Test Runner Extension


Regression in 5.x , paket is failing to resolve prerelease dependency

Description
We encountered following issue we have two packages that have a common dependency on of them using release and one is using prerelease of this dependency.
This an example:
ServiceA.Interface (1.0.0)
Common.Contracts (>= 2.4.10)
ServiceB.Interface (1.0.0)
Common.Contracts (>= 2.5.0-pre01)
In paket 4.8.8  for Common.Contracts version 2.5.0-pre01 is taken but in paket 5.1.8 we geting a conflict it unable to find a package that satisfies both condition

actually that seems like paket 5 is correct here. ServiceA.Interface doesn't allow prerelease.
I think you can Common.Contracts prerelease to deps file to allow that.

I try setup like in attached archive, but can't reproduce problem.
paket2459.zip

ls *.template | % { paket pack output ..\nuget templatefile $_.Name } in \paket2459\packages\ folder
paket install in \paket2459\solution\ folder

In \paket2459\solution\paket.lock file i get:
NUGET
  remote: ..\nuget
    common (1.2.0-pre01)
    packageA (1.0.0)
      common (>= 1.1.1)
    packageB (1.0.0)
      common (>= 1.2.0-pre01)

o_O

I will try to make a repro

I think this is kind of solved now. At least we improved the situation a bit via https://github.com/fsprojects/Paket/issues/2475 and https://github.com/fsprojects/Paket/issues/2471
Let us know if this is not the case. Short story: You need to tell paket that you want to allow the prerelease in the dependencies file. This is expected at the moment.


Unable to resolve ForwardRefs

Hi! I've been trying to use pydantic, ray, and confection all together (in PY3.9) for a project, and am running into a problem resolving "forward refs" by way of both from __future__ import annotations and using string names as type annotations. According to pydantic, this should "just work", but unfortunately that has not been my experience in this usage. Hopefully I'm just doing something wrong...
Here's a minimal reproducible example, with packages
catalogue == 2.0.7
confection == 0.0.3
pydantic == 1.10.2
ray == 2.1.0

import catalogue
import confection
import pydantic as pyd
from ray.air.config import RunConfig

confection.registry.test = catalogue.create("test")

@confection.registry.test.register("foo")
class FooSchema(pyd.BaseModel):
    bar: str
    run: RunConfig

config_str = """
[foo]
@test = "foo"
bar = "bar"

[foo.run]
name = "test"
"""
config = confection.Config().from_str(config_str)
confection.registry.resolve(config)

That last line raises ConfigError: field "callbacks" not yet prepared so type is still a ForwardRef, you might need to call RunConfig.update_forward_refs(). I'm not actually sure how to follow the suggestion of the error, since RunConfig does not have that method, but calling FooSchema.update_forward_refs() before resolving raises the same error.
The problematic field's type annotation here is Optional[List["Callback"]].

Hi @svlandeg , no worries! This issue never got resolved -- seems to be a fundamental limitation of the tool(s)? -- but iirc I worked around it by specifying that config as a dict and then manually validating it against that RunConfig schema elsewhere in the code. Not very elegant, and a bit of a pain, but doable.


[release/v1.4] Update vSphere CCM to v1.23.0

What type of PR is this?
/kind feature
What this PR does / why we need it:

Update vSphere CCM to v1.23.0 for Kubernetes 1.23 clusters
Add support for Kubernetes 1.23 on vSphere

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #1685
Does this PR introduce a user-facing change?:
Update vSphere CCM to v1.23.0 for Kubernetes 1.23 clusters. Add support for Kubernetes 1.23 on vSphere

/assign @kron4eg

/approve
/lgtm


Snips 0.55.2

Updated to latest snips, 0.55.2
Added customtts option which uses HA providers
Added german and french assistants

Thanks for maintain and update it 👍 please set next time the correct version on build.json too. In build it was 0.53.17 and in changelog it was 0.55.1 and in the description you had use 0.55.2

This PR has been clearly merged untested.
Since there is an error in the configuration schema, the add-on won't even be loaded by Hass.io.
I've opened an PR containing the hotfix. #289

@TheZoker Pls start an issue and feel free to tag me. Nothing changed in the mosquitto configuration. When you do can you post your config options and the start of the log that shows mosquitto trying to start?


Set a time interval for each location request

I realize that the location service has a random time, how and where could a time interval be configured to request location? For example, every 10 or 20 seconds? I couldn't find where this configuration would be.

This is not a bug. I'm closing it. Updates of the location are fixed, not random. If you want to use this code for another project, I suggest you go through it, debug it, and you will find where the interval is set.


Allow scheduled events to be scheduled in a disabled state

While Zappa currently allows event_source events (SQS, etc) to be created in a disabled state, it does not offer the same ability for cloud_watch events.  This MR adds the ability to schedule them in disabled mode.  See README changes for more details/example.
Relevant boto docs: https://boto3.amazonaws.com/v1/documentation/api/1.9.42/reference/services/events.html#CloudWatchEvents.Client.put_rule

Coverage increased (+0.007%) to 73.529% when pulling d1c785694331f65de9a406e771f4055414cbfb62 on javulticat:jav/disabled-events into 86a471ca90ebe86cd103ab1fcaadcf9b099abc9b on zappa:master.

Any plans on releasing this changes with a bumped up version number => 0.55.1 ?


When is the next planned release?

I'm curious as to when you'll be releasing the next version from v0.12.0; specifically, I'm interested in the change introduced in PR #27
Thanks!

Just released 0.12.1 :)

great, thanks!


Added CI/CD workflow

#177
provided a high-level description for a CI/CD setup, likely integrating tests, builds, or deployments to streamline the development workflow using GitHub Actions. This is essential for automating processes like:
Running tests on code pushes and pull requests.
Building and deploying the application on merges.
Ensuring consistency and quality of the project through automated checks.

@ananyag309 great work!!


Update time comparison between asf datasketches and datasketch

I am concerned that this is not a very accurate way to measure average update time. Specifically, starting and stopping
the timer is not cost-free, so measuring each update by doing that on each sketch seems like it'll distort things. Ideally it's just adding a noise floor in all cases, but I dunno if it'll interact with things like instruction caching in the CPU.
The preferred approach is generally to time adding a bunch of data at once in order to minimize the timer overhead. By using the sequence generator, I think the idea would be to load a bunch of data from there into a local array and then update from the array so that every sketch sees the same inputs in the same order.

I think the comment about timing was fair criticism.  I have simplified the notebook to test fewer stream lengths but incorporated the timing in the manner that you suggested.  It should be quicker to run and simpler to parse.

in general for for library we'd expect 8 < 6 or 4.  6 and 4 are harder to predict since they both involve bit twiddling; for 6 we often need to cross byte boundaries while 4 never does but has exceptions.
That they bounce around very close to each other makes me think a fair amount of it is python overhead overwhelming the actual sketch.

Are you referring to the line update_data = self.data_bytes[trial][:_n] ?
This was the only way I could avoid generating new random variables for each trial.
If not this, then I'm unsure what data prep you are referring to as the data is generated once in the class instantiation and all trials are executed on the same dataset.

Also regarding the relative timings between 8 vs 6 and 4: I could run more trials to get closer to your expected outcome, but I think the purpose of this experiment is to show relative performance vs datasketch.  We may need many many more trials to get to your expectation in which case I imagine a cpp profile is more suitable anyway.


Can't escape strings with ', " or \ in Array

When I try to insert record which have escaped char for single quote, double quote or backslash in some Array field, I got an error from database.

What input FORMAT do you use ?
Could you also provide error message and an example reproducing the problem?

I'm using VALUES format, and I can insert single quote with escape like ' but string is written in database with that escape, not like single quote.

It actually writes to DB single character. To prove this I print length(str) in addition to str:
:) drop table if exists test.quoting;

DROP TABLE IF EXISTS test.quoting

Ok.

0 rows in set. Elapsed: 0.000 sec.

:) CREATE TABLE test.quoting (str String) ENGINE = Memory;

CREATE TABLE test.quoting
(
    str String
) ENGINE = Memory

Ok.

0 rows in set. Elapsed: 0.007 sec.

:) insert into test.quoting VALUES ('\'') ('\"') ('\\') ('"') ('0');

INSERT INTO test.quoting VALUES

Ok.

5 rows in set. Elapsed: 0.001 sec.

:) select str, length(str) from test.quoting;

SELECT
    str,
    length(str)
FROM test.quoting

┌─str─┬─length(str)─┐
│ \'  │           1 │
│ "   │           1 │
│ \\  │           1 │
│ "   │           1 │
│ 0   │           1 │
└─────┴─────────────┘

5 rows in set. Elapsed: 0.001 sec.

:) select str, length(str) from test.quoting FORMAT TabSeparatedRaw;

SELECT
    str,
    length(str)
FROM test.quoting
FORMAT TabSeparatedRaw

'	1
"	1
\	1
"	1
0	1

5 rows in set. Elapsed: 0.001 sec.

:)

Many output formats add escape symbols to the result.
Default formats TabSeparated and Pretty do that.
To avoid escaping you can use one of "raw" formats, for example TabSeparatedRaw as in my example.


Rationale for ajaxing lectures/new etc.

When creating a simple lecture creation test, cypress warns
This element <input.btn.btn-primary> is not visible because its parent <div#new-course-area.bg-green-lighten-4.p-3.mb-3.rounded> has CSS property: display: none

Fix this problem, or use {force: true} to disable error checking

, when trying to click on the "save lecture" button, after selecting the plus button.
When forcing to click, it kind of works, but no result/creation is displayed afterwards.
I can not tell, whether this issue is caused by cypress, because the hot plugged elements are displayed in their preview at some point.
I believe, that loading code and executing it as a "side-effect" causes this problem
I think the initial reason to load it via ajax was to automatically load the modules. Maybe migrating to an API approach, where fetching data, not code, could resolve this.

The problem is caused by the redirect after submission via Turbolinks (not a classical HTTP 302). still investigating whether it is a problem with cypress.

Rationale is this by DHH himself.

ok thanks. tried to click the wrong (course) save button 🤦, which  has the same content/meta, is hidden, and therefore couldn't be clicked. Shouldn't be the course creation be side loaded the same way?

Hm, it seems indeed that the courses/_new partial is already rendered on the page, but hidden. I do not remember why there is this difference.


Smoke-testing IPFS Desktop v0.13.0

We now have new a brand CI/CD pipeline responsible for building IPFS Desktop (#1657, #1659).
Before we ship v0.13, let's have more eyes on platform-specific packages.

Help us by doing a quick test of IPFS Desktop v0.13.0 (RC1)
If you have spare time between now and Monday (~5min):

Install IPFS Desktop using package listed below
Go to Files and Import a folder with some cat pictures
You should be able to see successfully imported files in your Files screen
Comment on this issue with package name and if it went smoothly or  if you had any issues.

macOS

[x] IPFS-Desktop-0.13.0.dmg

Windows

[x] IPFS-Desktop-Setup-0.13.0.exe

Linux
If you happen to run Linux, make sure to include the name of distro and Desktop Environment you are running.

[x] ipfs-desktop-0.13.0-linux-x64.tar.xz (@lidel: unpacked standalone version runs as expected)
[x] ipfs-desktop-0.13.0-linux-amd64.deb
[ ] ipfs-desktop-0.13.0-linux-x86_64.rpm
[ ] ipfs-desktop-0.13.0-linux-x86_64.AppImage
[x]  ipfs-desktop-0.13.0-linux-amd64.snap
[ ] ipfs-desktop-0.13.0-linux-x64.freebsd

Our team will be back on Monday,
ready to squash any bugs blocking the release

Mac here. All worked as expected for importing both via Import > Folder and just drag/drop, except the drawer gave me an "NaN" while uploading:

Tested .deb on Ubuntu 20.04 stock version.
Import button and dragging on Files screen both work as expected, except I got cheese NaNed like @jessicaschilling .

Snap: same story

Worth noting: My previously addded files and folders are not available.
I think it's because snap sets up its own repository, isolated from other apps...

IPFS-Desktop-0.13.0.dmg - it went smoothly.

Importing a directory worked but showed Imported NaN Items as mentioned above, for both import via menu option and import via file drop.

I managed to get hold on Windows 10 laptop  and both  installing for single user and globally on the OS seems to work as expected. Binary is signed.

Worked fine on Windows 10 (1809). Got a message saying Imported NaN items after adding a file, but apart from that everything was fine.

Thank you all! :pray:
I've filled https://github.com/ipfs-shipyard/ipfs-webui/issues/1659 for the NaN thing. It is not related to Electron, but  a ux bug in webui (confirmed it in Firefox).
While NaN looks bit awkward, it is shown only when more than one file was imported.
I don't think this small UX nit should block the release that ships go-ipfs 0.7.0.
@jessicaschilling Let's discuss this later today during triage

Just tested .AppImage on Ubuntu 20.04
It worked once perfectly with import. Even without the NaN problem:

But most of the time there is the NaN/ 0/ 1 problem:
One dragged file:

Sometime it displays this :

... then this after a few seconds :

Other than that it works.

Update: @Gozala is troubleshooting - ideally we'd patch this before releasing a new Desktop, as it's a highly visible UX flaw even if it's not breaking functionality.

@Gozala  fixed theNaN bug (https://github.com/ipfs-shipyard/ipfs-webui/pull/1660), and we switched to webui v2.11.3 (https://github.com/ipfs-shipyard/ipfs-desktop/pull/1666).
@jessicaschilling @johnnymatthews @bertrandfalguiere @olizilla
please repeat the test using RC2 version from ipfs-desktop-0.13.0-rc2/ :pray:

IPFS Desktop-0.13.0.dmg: NaN issue is gone.
Note that I've filed https://github.com/ipfs-shipyard/ipfs-desktop/issues/1668 for less-than-ideal placement of Mac window controls vis-a-vis IPFS cube.

AppImage: OK
.deb: OK
.snap: must be installed with the --dangerous flag (I guess it won't be needed once on the snapstore?). Files are in a different place than with other installs, and the peer ID is different. Otherwise, OK

@bertrandfalguiere thank you! Yes, --dangerous won't be needed when it lands on snapstore. Snap has its own philosophy regarding paths, so those discrepancies are expected.
@jessicaschilling  I've switched to webui v2.11.4 with fix for https://github.com/ipfs-shipyard/ipfs-desktop/issues/1668. Electron had a bunch of security fixes, so I've also updated us to 9.3.2 and created RC3:
Once more, with feeling..

Install IPFS Desktop using package  from ipfs-desktop-0.13.0-rc3/
Go to Files and Import a folder with some cat pictures
You should be able to see successfully imported files in your Files screen
Comment on this issue with package name and if it went smoothly or  if you had any issues.

Release checklist

[ ] Windows
[x] Linux @lidel
[ ] macOS

@lidel tested on MacOS and looks good 👍

@lidel @rafaelramalho19 Confirmed OK on MacOS here as well.

Thank you all for help! Shipping it to autoupdate channel.

v0.13.0 :ship: :dash:


Added sokol_nuklear.h utility header

sokol_nuklear.h implements the initialisation, rendering and event-handling code for Nuklear on top of sokol_gfx.h and (optionally) sokol_app.h. The design of it is heavily based on the existing sokol_imgui.h header.
See also this PR for the example in the sokol-samples repo: https://github.com/floooh/sokol-samples/pull/81

FYI: I'm going to look into this PR after the "API breaking merge" I'll do sometime this week (probably tomorrow). I'll need to fix a few things because of the API changes, but I can do this during the merge.

Ok, I'm starting to look into this now.

BTW I just saw this:
https://github.com/floooh/sokol/blob/9d518545fcdd6e775f6fe5ff35e099bfd2c4d8fd/util/sokol_nuklear.h#L1662
Note that SOKOL_ASSERT() will be compiled out in release builds (or more specifically, when NDEBUG is defined), so putting logic into SOKOL_ASSERT() isn't a good idea (I'll fix that stuff during the merge - I'll also try updating to the latest Nuklear, and put the header right into sokol-samples/libs)

I'm going to use the sample code from here:
https://github.com/Immediate-Mode-UI/Nuklear/blob/master/demo/overview.c
This demonstrates more Nuklear features similar to the ImGui samples.

...I also added an overflow-check for the vertex- and index-data:
https://github.com/floooh/sokol/blob/1ddb55b177ab64667d389715717fa15c4170b962/util/sokol_nuklear.h#L1899-L1905
This is also caught by the sokol_gfx.h validation layer, but catching this higher up is a bit cleaner.

Oh, and I added a preprocessor check and a comment that the implementation must be compiled in C, this is a bit special because most other headers can be either compiled in C or C++, but I think for Nuklear it makes sense, and the C99 designated initialization makes it "worth it" (I'm actually thinking about moving some other header-implementations from the common C/C++ subset to C99 as well).

...and merged. Thanks! I'll update the CHANGELOG and add the sample to the sample webpage next.

Awesome. I’m glad it’s in (and thank you for Sokol in general!)


Found a few more http->https for forecast.weather.gov

Sorry, I missed a few calls on the initial PR.

Thanks again @nivex awesome work!


[SPARK-26001][SQL]Reduce memory copy when writing decimal

What changes were proposed in this pull request?
this PR fix 2 here:

when writing non-null decimals, we not zero-out all the 16 allocated bytes. if the number of bytes needed for a decimal is greater than 8. then we not need zero-out between 0-byte and 8-byte. The first 8-byte will be covered when writing decimal.

when writing null decimals, we not zero-out all the 16 allocated bytes. BitSetMethods.set the label for null and the length of decimal to 0. when we get the decimal, will not access the 16 byte memory value, so this is safe.

How was this patch tested?
the existed test cases.

cc @mgaido91, @dongjoon-hyun , @cloud-fan , @kiszk

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

I have two questions.

Is this PR already tested with "SPARK-25538: zero-out all bits for decimals"?
How does this PR achieve performance improvement? This PR may introduce some complication. We would like to know the trade-off between performance and ease of understanding.

@mgaido91
thank you for review it.  I added a test case to test "write a decimal with 16 bytes and then one with less than 8".  then the current change the remaining 8 bytes would not dirty.

@kiszk thank you for review it.

when writing null decimals：

OpenJDK 64-Bit Server VM 1.8.0_163-b01 on Windows 7 6.1
Intel64 Family 6 Model 94 Stepping 3, GenuineIntel
iter length 1048576:                     Best/Avg Time(ms)    Rate(M/s)   Per Row(ns)   Relative
------------------------------------------------------------------------------------------------
before PR (input == null)                       51 /   56         20.4          49.0       1.0X
after PR (input == null)                         8 /    9        125.2           8.0       6.1X

when writing non-null decimals

OpenJDK 64-Bit Server VM 1.8.0_163-b01 on Windows 7 6.1
Intel64 Family 6 Model 94 Stepping 3, GenuineIntel
iter length 1048576:                     Best/Avg Time(ms)    Rate(M/s)   Per Row(ns)   Relative
------------------------------------------------------------------------------------------------
before PR (input != null)                       52 /   53         20.3          49.2       1.0X
after PR (input != null)                        54 /   56         19.3          51.7       1.0X

I think this is wrong. We have to zero out the bytes even writing a null decimal, so that 2 unsafe rows with same values(including null values) are exactly same(in binary format).

yes, I agree with @cloud-fan , this can create wrong results with nulls...


100% CPU Usage

I realized I have 100% CPU usage using this library.

I think we need more input on your setup, for me executing examples/hello-world.php is not consuming any CPU time. HTTP requests also work properly.

PHP 7.2.11 on Ubuntu 18.04 => 100% CPU Usage, but it works.
If I install lib-uv => 0% CPU Usage, so it has something to do with the native driver.

Okay, that I can reproduce; was using ext/uv here - will investigate.

:+1: thanks


Azure WebJob: There is not enough space on the disk

I am getting an error on my Azure WebJob about not being enough space on the disk. I had it earlier, so I moved it to Standard 1 which says it comes with 50GB.
Repro steps.
App runs fine on my local machine, and sometimes runs fine when manually triggered as an Azure WebJob, but runs out of space.
Project structures.
The code for the app is not large, but it does download some big files from an FTP site, then uncompress them on disk and send them up to the S3 cloud.
So I can see storage being an issue on local but Home should have plenty.
Console reports 87,402,242,048 bytes free
Kudu reports 112,640 MB total; 83,353 MB free on D:\home and 11,264 MB total; 11,259 MB free on d:\local
I can see the files download and uncompress on the Home directory in the Console, but does all this happen in the background on local first and then get copied over after the app runs or something?

This is a triggered WebJob that is scheduled to run once a day with CRON
It is being deployed by publishing directly from inside Visual Studio Pro 2017
It is written in C#.net Core 2.2 as a Console Application
I am NOT using the WebJobs SDK

The log/error given by the failure.
[08/29/2019 05:19:04 > cd1c7c: SYS ERR ] Failed to copy job files: System.IO.IOException: There is not enough space on the disk.

   at System.IO.__Error.WinIOError(Int32 errorCode, String maybeFullPath)
   at System.IO.File.InternalCopy(String sourceFileName, String destFileName, Boolean overwrite, Boolean checkHost)
   at Kudu.Core.Infrastructure.FileSystemHelpers.CopyDirectoryRecursive(String sourceDirPath, String destinationDirPath, Boolean overwrite) in C:\Kudu Files\Private\src\master\Kudu.Core\Infrastructure\FileSystemHelpers.cs:line 241
   at Kudu.Core.Infrastructure.FileSystemHelpers.CopyDirectoryRecursive(String sourceDirPath, String destinationDirPath, Boolean overwrite) in C:\Kudu Files\Private\src\master\Kudu.Core\Infrastructure\FileSystemHelpers.cs:line 235
   at Kudu.Core.Infrastructure.FileSystemHelpers.CopyDirectoryRecursive(String sourceDirPath, String destinationDirPath, Boolean overwrite) in C:\Kudu Files\Private\src\master\Kudu.Core\Infrastructure\FileSystemHelpers.cs:line 235
   at Kudu.Core.Jobs.BaseJobRunner.<>c__DisplayClass49_0.<CacheJobBinaries>b__0() in C:\Kudu Files\Private\src\master\Kudu.Core\Jobs\BaseJobRunner.cs:line 188
   at Kudu.Core.Infrastructure.OperationManager.<>c__DisplayClass2_0.<Attempt>b__0() in C:\Kudu Files\Private\src\master\Kudu.Core\Infrastructure\OperationManager.cs:line 17
   at Kudu.Core.Infrastructure.OperationManager.Attempt[T](Func`1 action, Int32 retries, Int32 delayBeforeRetry, Func`2 shouldRetry) in C:\Kudu Files\Private\src\master\Kudu.Core\Infrastructure\OperationManager.cs:line 42
   at Kudu.Core.Infrastructure.OperationManager.Attempt(Action action, Int32 retries, Int32 delayBeforeRetry) in C:\Kudu Files\Private\src\master\Kudu.Core\Infrastructure\OperationManager.cs:line 19
   at Kudu.Core.Jobs.BaseJobRunner.CacheJobBinaries(JobBase job, IJobLogger logger) in C:\Kudu Files\Private\src\master\Kudu.Core\Jobs\BaseJobRunner.cs:line 179
[08/29/2019 05:19:06 > cd1c7c: SYS INFO] Status changed to Failed
[08/29/2019 05:19:06 > cd1c7c: SYS ERR ] WebJob run failed due to: System.InvalidOperationException: Missing working directory
   at Kudu.Core.Jobs.BaseJobRunner.InitializeJobInstance(JobBase job, IJobLogger logger) in C:\Kudu Files\Private\src\master\Kudu.Core\Jobs\BaseJobRunner.cs:line 232
   at Kudu.Core.Jobs.TriggeredJobRunner.<>c__DisplayClass10_1.<StartJobRun>b__0(Object _) in C:\Kudu Files\Private\src\master\Kudu.Core\Jobs\TriggeredJobRunner.cs:line 78

Debug your Azure website remotely.
Subscription is... 095ca...88d23
Initially posted Stack Overflow, posting here after recommendation...

We took a look at the site "webjo...lein" and, unfortunately, it was moved to another VM.  All space looked good - please confirm if still issue.   Given it was moved, we could not investigate the root cause.   We have improved telemetry recently to help pinpoint (ETA 4 weeks out).   In the meanwhile, if you ran into the issue again, do re-open this and let us know.

i actually ended up tweaking the code to delete the files, so no longer running into this specific issue.
definitely seemed like there is some sort of "hard limit" on disk usage by a webjob, regardless of the tier you use.
or, it could be something unrelated to disk space and instead CPU usage or something, but as you state, the reporting is pretty minimal. not sure why we can't get some sort of error code or something, rather than simply "aborted" which is more or less useless. hopefully the telemetry update will include a little more info.


Winelib compilation failing

Essentially the same issue as this:
https://github.com/doitsujin/dxvk/issues/1179
https://bugs.winehq.org/show_bug.cgi?id=47710
Which dxvk managed to fix with these changes:
https://github.com/doitsujin/dxvk/commit/5ea5d2a81660480ba553b8b98e8682dfc0153d20
https://github.com/doitsujin/dxvk/commit/c57e63f7d297c3f29ca0b34d93960fb414cf633c
However, I tried applying the same changes to the d9vk project and didn't get it working, a good reason for this could be that I'm dumb though.
winebuild.patch.txt

I rebased, can you try again?

Had the same issue, works great now, thanks!


Mix

Codecov Report

Merging #94 into master will decrease coverage by -0.31%.
The diff coverage is 95.23%.

@@            Coverage Diff             @@
##           master      #94      +/-   ##
==========================================
- Coverage     100%   99.69%   -0.31%
==========================================
  Files          54       55       +1
  Lines         303      324      +21
==========================================
+ Hits          303      323      +20
- Misses          0        1       +1

Impacted Files
Coverage Δ

src/helpers/mix.js
95.23% <95.23%> (ø)

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 18ea7aa...05e56e0. Read the comment docs.


Deblur trim length reported inaccurately

If you select a BIOM artifact from a files network, it shows in the "processing parameters" used and they include a value for "trim-length". However, that value is not incorrect as the trimming is performed upstream. This is misleading.

This is also confusing within the API as incorrect values are reported in multiple places, including in the parent.
In [21]: t = biom.load_table(a.filepaths[0][1])

In [22]: len(t.ids(axis='observation')[0])
Out[22]: 150

In [23]: a.processing_parameters.values['trim-length']
Out[23]: 100

In [24]: a.id
Out[24]: 25483

In [28]: p = a.parents[0]

In [30]: p.processing_parameters.values
Out[30]: {'input_data': u'25331', 'length': 150}

In [32]: p.processing_parameters.command.parameters
Out[32]: {'input_data': ['artifact', None], 'length': ['integer', '100']}

Good point, however, that's a behavior due to the parameters used for debluring, which has (can't remember which) a parameter to ignore the trim-length. Not sure which will be the best behavior? Always check all params (easy, most accurate) for downstream analysis? Inherit the parameters (can lead to inconsistencies)? Other?

I understand the issue with deblur, but I do not understand the issue with
the parent artifact reporting the wrong trim length in the code pasted, any
idea there? Checking all params doesn't resolve confusion.
As to what to show in the UI, perhaps a clear statement in the artifact
network itself of what the actual trim length is?
On Apr 11, 2017 05:30, "Antonio Gonzalez"<EMAIL_ADDRESS>wrote:
Good point, however, that's a behavior due to the parameters used for
debluring, which has (can't remember which) a parameter to ignore the
trim-length. Not sure which will be the best behavior? Always check all
params (easy, most accurate) for downstream analysis? Inherit the
parameters (can lead to inconsistencies)? Other?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/biocore/qiita/issues/2105#issuecomment-293245174, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAc8svq1lcUGlnG1i8o2cu3HgzjcaaKbks5ru3JZgaJpZM4M5RxI
.

Sent (a version of) this via chat but to keep it here for reference: Got it! p.processing_parameters.values will retrieve the parameters used to generate that artifact, p.processing_parameters.command will retrieve the parameter that was used to generate that parameter (in other words, the Command object that generated that artifact), and p.processing_parameters.command.parameters the default parameters of that Command.

Can that last one be denoted as "default_parameters"?
On Apr 11, 2017 07:09, "Antonio Gonzalez"<EMAIL_ADDRESS>wrote:

Sent (a version of) this via chat but to keep it here for reference: Got
it! p.processing_parameters.values will retrieve the parameters used to
generate that artifact, p.processing_parameters.command will retrieve the
parameter that was used to generate that parameter (in other words, the
Command object that generated that artifact), and p.processing_parameters.
command.parameters the default parameters of that Command.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/biocore/qiita/issues/2105#issuecomment-293275056, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAc8skK3pTjW-rKftoAl8ylUFVB0ZjaGks5ru4mQgaJpZM4M5RxI
.

I guess, we will just need to change the method name and the tests ... can we rename the issue?

If others think it should be changed as well? No concern on renaming the
issue
On Apr 11, 2017 07:51, "Antonio Gonzalez"<EMAIL_ADDRESS>wrote:
I guess, we will just need to change the method name and the tests ... can
we rename the issue?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/biocore/qiita/issues/2105#issuecomment-293288191, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAc8ssDfZA1A-u36ZqCL5UC5pWvgDBMhks5ru5M5gaJpZM4M5RxI
.


Add .gitignore

Added a .gitignore file

To clarify, this is a smaller update and will also probably be my last as I have my own projects to work on.
def say_goodbye() -> int:
    print("Goodbye!")
    return 0

if __name__ == "__main__":
    main()

Uhhh I was hoping this would get accepted so that I can delete my fork and live in peace, but fine, I'll wait.

Okay then I'll just close this I don't care.


Sparkta working over a standalone cluster

Description
Sparkta working over a standalone cluster

This PR will go to a common branch for SparkStandalone, Mesos and YARN


skip test for blkio.weight when kernel does not support it

Signed-off-by: baude<EMAIL_ADDRESS>

/approve

Sure, LGTM

Change LGTM, but gofmt isn't happy with rmi.go.

Can we change the validate code to only run with golang 1.11, this is what we did with buildah and it eliminated all of this noise.
diff --git a/Makefile b/Makefile
index bc70bff2..6cd8f53a 100644
--- a/Makefile
+++ b/Makefile
@@ -5,8 +5,8 @@ BINDIR := $(PREFIX)/bin
 BASHINSTALLDIR=${PREFIX}/share/bash-completion/completions
 BUILDFLAGS := -tags "$(AUTOTAGS) $(TAGS)"
 GO := go
-GO111 := 1.11
-GOVERSION := $(findstring $(GO111),$(shell go version))
+GO110 := 1.10
+GOVERSION := $(findstring $(GO110),$(shell go version))

 GIT_COMMIT := $(if $(shell git rev-parse --short HEAD),$(shell git rev-parse --short HEAD),$(error "git failed"))
@@ -53,7 +53,7 @@ deps: gopath
 validate:
        # Run gofmt only on version 1.11

-ifeq ($(GO111),$(GOVERSION))
+ifneq ($(GO110),$(GOVERSION))
        @./tests/validate/gofmt.sh
 endif
        @./tests/validate/whitespace.sh

Can we change the validate code to only run with golang 1.11, this is what we did with buildah and it eliminated all of this noise.
diff --git a/Makefile b/Makefile
index bc70bff2..6cd8f53a 100644
--- a/Makefile
+++ b/Makefile
@@ -5,8 +5,8 @@ BINDIR := $(PREFIX)/bin
 BASHINSTALLDIR=${PREFIX}/share/bash-completion/completions
 BUILDFLAGS := -tags "$(AUTOTAGS) $(TAGS)"
 GO := go
-GO111 := 1.11
-GOVERSION := $(findstring $(GO111),$(shell go version))
+GO110 := 1.10
+GOVERSION := $(findstring $(GO110),$(shell go version))

 GIT_COMMIT := $(if $(shell git rev-parse --short HEAD),$(shell git rev-parse --short HEAD),$(error "git failed"))
@@ -53,7 +53,7 @@ deps: gopath
 validate:
        # Run gofmt only on version 1.11

-ifeq ($(GO111),$(GOVERSION))
+ifneq ($(GO110),$(GOVERSION))
        @./tests/validate/gofmt.sh
 endif
        @./tests/validate/whitespace.sh

Yes, I know exactly what to do and we are in the middle of doing it; as I said, we need the origin group to merge a PR I have to make this correct.

/retest

/approve
and
/lgtm


[Darwin] Ensure that fetch is executed on the NSManagedObjectContext …

…private queue instead of chip dispatch queue
Problem
Darwin KVS is creating an NSManagerObjectContext using  NSPrivateQueueConcurrencyType but tries to fetch data from the main chip dispatch queue.
Change overview

Use performBlockAndWait to dispatch message to the private queue

Fast tracking platform changes that are impacting CI as well.


Deal with duplicated experiment name in querying

We try to limit the assumptions made in the database, so experiment names (like 01deg_jra55v13_ryf9091) aren't themselves unique — rather, the combination of experiment name and its root directory is. However, in functions like cc.querying.getvar, we only ask for the experiment name. This could lead to some unexpected behaviour, pulling files from two experiments with the same name, but a different root directory.

Good point, sorry. Actually, that particular experiment name is unique it is just that (for a short time) there is a duplicate copy on both hh5 and ik11. So, I will delete the hh5 copy as soon as I can convince myself that the rsync went well.

Actually, this was because I had too versions of the same folder listed on hh5, one with /g/data/hh5/ and one with /g/data3/hh5/. This should be easy to avoid...

Should be easy to avoid, but I don't think it should break so horribly in these cases!

The idea of a unique experiment id could address this I think. In the case where there is match with multiple IDs the operation could error and the user warned that manual disambiguation is required. The same thing could be done if there are multiple root paths I guess.
In any case, definitely a related issue
https://github.com/payu-org/payu/issues/191