Cast key value to string when your related table uses string

When working with an database that was managed remote we ran into performance issues. The indexes on columns were not used. After a EXPLAIN EXTENDED we noticed this was a issue with mismatching data types.
The parent table uses INT for the primary key but the related tables uses varchar for their fields. Because this database is not ours we can not change this!
The suggested change fixes this issue. It uses the existing KeyType that is present on the Model instance.
In our case 4 queries before this fix, 4000ms, after the fix. 40ms.

What database driver?

What database driver?

The issue is occurring with driver MySQL. I will update the PR.

I know you can't change it, but this is really not a good database design.
Is the table using the MyISAM engine? I can't reproduce this with InnoDB.
The code should also check whether the relationship is actually referencing the related model's primary key.

I know you can't change it, but this is really not a good database design.
Is the table using the MyISAM engine? I can't reproduce this with InnoDB.
The code should also check whether the relationship is actually referencing the related model's primary key.

I can attach a example sql file and steps how to reproduce. Tables are using InnoDB. As far as i can see all primary keys (and relation referencing columns) are varchars, but the actual content is int.
I don't know what you mean with your last comment, why should there be extra checks?

Yes, please post an example. What version of MySQL are you using?
The relationship doesn't always reference the related model's primary key. It could reference an integer column while the primary key is a string column. Then you shouldn't cast the value to a string.

You can find the export over here: https://gist.github.com/Bartjuh4/126fdd757f660a4f20156e39d85ae720
Query Laravel executes before the fix: SELECT * FROM subfile WHERE file_id IN (7) (20ms)
After fix: SELECT * FROM subfile WHERE file_id IN ("7") (2MS)
When you prepend "EXPLAIN EXTENDED" you can see that the index is not used as expected.

I don't think I plan to merge this on 5.7 at least and without further confirmation from @staudenmeir who helps with a lot of database maintenance.


flowtype conflict with cssx

I got past the eslint errors with the eslint-plugin-cssx!
Next, I also use flowtype.org for my react app, and it chokes on the syntax.  I'm wondering if there is a flow preprocessor/api as well so we can take the same approach.
 ❯❯❯ flow                                                                                Launching Flow server for /Users/kross/projects/my-relay-starter-kit
Spawned flow server (child pid=66702)
Logs will go to /private/tmp/flow/zSUserszSkrosszSprojectszSmy-relay-starter-kit.log
src/App.js:21
 21:     font-family: Roboto, sans-serif;
                    ^ Unexpected token :

Found 1 error

Reproducing

git clone https://github.com/rosskevin/my-relay-starter-kit
npm install
npm run flow

I see. I had similar issues with JSX. The trick was to run CSSX transpiler before the one that transpiles JSX. However, in this case we run flow directly and we can't plug in something in between. As a temporary solution I can suggest:

We use https://github.com/krasimir/cssx/tree/master/packages/cssx-cli to transpile all the files to a temporary folder
We run flow ignoring src
We then delete the temporary folder.

Thanks, I'll use your workaround until we can get a preprocessor in flow.

@rosskevin is the suggested workaround fixed your problem?

This workaround is viable, just not optimal. As-is, I'm not using cssx yet due to this and the ide integration problem (which I filed an issue).

I see. Unfortunately I don't see any other option at the moment. I mean CSSX is kinda new language and without transpiling the code nothing works on top of it. Regarding the IDE, I totally agree. It's still a problem.

If someone still needs this - you can use Flow comments as workaround:
export default useSheet(
  /*::`*/
  <style>
    outroAppInfo {
      display: flex;
      margin-top: 1.5rem;
    }
  </style>
  /*::`*/
)(OutroAppInfo);

(I'm using it with react-jss here)
This code will be treated by Flow as template literal and therefore ignored.


[Bug] Error while importing with containers.

Error while importing a GCS character with containers. The attached zip file has the same character twice, once with containers, which triggered this error:
"An error occured while importing Human Guard, TypeError:Cannot read property 'carried' of undefined"
Removing the containers (the other file in the archive) fixes the issue.
Archive.zip

The issue here is that Foundry has a hard coded limit of 10 levels of nested objects below the "Actor" level.    If a character has 4 sub-containers in Skills, it creates 11 levels (count the periods... plus 1)
data.skills.00000.contains.00000.contains.00000.contains.00000.contains.00000
We can support 3 levels of sub-containers, which have 9 levels (for ads, skills, spells) and 10 levels for equipment (since it has 2 subcategories 'carried' and 'other):
data.equipment.carried.00000.contains.00000.contains.00000.contains.00000
I've created an issue with Foundry:   https://gitlab.com/foundrynet/foundryvtt/-/issues/4679, but until then, we cannot support more than 3 levels.
I will update the import code to provide a warning.


FEATURE: Put cursor in the beginning of line in normal mode

Did you check the docs?

[X] I have read all the docs.

Is your feature request related to a problem? Please describe.
Most of the time, when renaming a file I just want to rename the filename without changing the extension. Currently the cursor placed in the end of line.
Describe the solution you'd like.
I think it would be better to put the cursor in the beginning of the line so we can just use cw or ct. to change the filename.
Describe alternatives you've considered.
No response
Additional Context
No response

https://github.com/nvim-neo-tree/neo-tree.nvim/issues/602

Sorry probably I'm missing something.
I'm aware that the normal mode exists, but when I want to rename the file, the cursor started in the end of the line.
Is there any other config to make the cursor in the beginning of the line instead?

Oh I'm sorry.

Just adding info for anyone looking to pick this up: you might have success getting the nui input/popup window ID (split.winid) and setting the cursor with vim.api.nvim_win_set_cursor().

neo_tree_popup_input_ready event has been added with https://github.com/nvim-neo-tree/neo-tree.nvim/pull/1372.
Please read :h neo-tree-events for more information.

Not sure if this is the right place to ask this, but does anyone know if there's a way to control what happens based on the different input popups?
I want to be able to customize what happens when the input box for the move operation opens, and a different action when the input box for add opens

Also with this new suggested event:
{
  event = "neo_tree_popup_input_ready",
  ---@param input NuiInput
  handler = function(input)
    vim.cmd("stopinsert") -- Enter normal mode
  end,
}

I am unable to enter normal mode after manually entering insert mode with i, since pressing esc even in insert mode just closes the input. Is there a way to make it so that I need to press esc from within normal mode only to exit the input box, as I believe it used to be before?

Not sure if this is the right place to ask this, but does anyone know if there's a way to control what happens based on the different input popups? I want to be able to customize what happens when the input box for the move operation opens, and a different action when the input box for add opens

There's no simple way to do this, but it would be a nice feature. Someone could probably come up with a code snippet to inspect the title displayed in the wrapper window.

I am unable to enter normal mode after manually entering insert mode with i, since pressing esc even in insert mode just closes the input.

That's a bug, or at least an oversight. I made a separate issue out of that.

Isn't it just up to the user to remap using input.map inside the event by themselves if they want this feature?

Isn't it just up to the user to remap using input:map inside the event by themselves if they want this feature?

Maybe, but we make the mapping with Nui so I'm not sure how easy it is to unmap from the event.

Wdym? There's input:unmap or do you mean something else?

Wdym? There's input:unmap or do you mean something else?

input is a Nui object that is not available in the event and also not standard nvim api so I'm not sure its the best user experience even if we could provide a reference to that object.

I'm sorry, I even put it in the docs but forgot to pass input as an argument to fire_event. This is more of a bug I'd say.
I think we should pass it as either input or { input = input }.
Which one would you prefer? I'll make a PR later today.

@uthmanmoh
I've created https://github.com/nvim-neo-tree/neo-tree.nvim/pull/1398. Do you think this method works for you?

Yup being able to control the mappings seems good. Thanks for the fix.
Although im thinking now with the complexity of this event handler increasing, it might have been a good idea to keep the boolean option for enabling normal mode which if set to 'true' would just create this handler. That makes sense as its own option since thats a basic functionality imo.
Let me know if this makes sense, maybe I can create a separate ticket for it

This is my opinion on that https://github.com/nvim-neo-tree/neo-tree.nvim/pull/1372#issuecomment-1975233977
I understand that it is complicated but I've seen many FR that are slightly different for everybody (normal by default, put cursor before extension but in insert mode, put cursor in front, ...) and it is hard to distinguish where we officially support and not.
Therefore imo I think it's better to say that we support one and only one feature (insert + maps for <esc>, q) and everyone that don't like this behavior, they can edit whatever they want.
If we go this route, users can share their handler in the wiki and everybody who copy that code will work because everybody is on the same starting point, regardless of any other config option.

That sounds alright to me. Thanks for clarifying


Move commands into their own repos

To make the CLI as composable as possible, move foundry-release, foundry-config, and foundry-completion into their own repos.

I am deciding to not implement this. It's overengineering and unnecessary.


Generate tweets for @web3isgreat automatically

Right now when I add a new post I have to manually screenshot the entry, copy over the title/date/link, and copy the post contents into alt-text. Would be pretty cool to be able to do this automatically (though should still be kicked off by me, since not all posts go in the Twitter feed, and sometimes I need to make edits after a post is added to fix typos etc.).

I made this.

Closed with https://github.com/molly/w3igg-crossposter


plot product by subject

la API está respondiendo con un keyerror:
https://apis.impactu.colav.co/app/affiliation?type=institution&id=65a9a92de5b59075e52c2720&section=research&tab=products&plot=products_subject

Bug solved parsing level param as int and aplying ternarty operator to get name of subject. ba85ce48e59ab621ae17cb6ec0ed0f06af0cb56f


[REF-2052]  Incorrect Usage of rx.switch in your Documentation

Path: /docs/library/forms/switch
Description:
The documentation for Reflex framework contains an erroneous example regarding the usage of rx.switch. It advises users to use the following code snippet:

rx.text(
    rx.flex(
        rx.switch(default_checked=True),
        "Sync Settings",
        spacing="2",
    )
)

However, implementing this code results in a DOM nesting error, specifically:
client.js:1 Warning: validateDOMNesting(...): <div> cannot appear as a descendant of <p>
This discrepancy might cause confusion among users and result in incorrect implementation of the Reflex framework. It's essential to revise the documentation to ensure accurate guidance on utilizing rx.switch without encountering DOM nesting errors.
REF-2052

This issue doesn't exist anymore  (already fixed).
Link: https://reflex.dev/docs/library/forms/switch/
@picklelo kindly close this issue.


New environment configuration is broken on node-webkit

Hi.
You are using the global. keyword in your environment_browser setup.
Why are you not using window. to get the references to rtc, websocket etc?
If you run this in node-webkit you will get a conflict, as there you will have a global object that takes care of cross window contexts.
I renamed all global. to window. in environment_browser.js and it seems to work just fine.

Hi @oteren, thanks for the report! For context, here's where some of the relevant changes took place:

global instead of window: https://github.com/onsip/SIP.js/commit/fc1999572382b24d62ec91bc64dee3eeeb6abcb4
environment.js: https://github.com/onsip/SIP.js/commit/9a2b08943414a1ba83745a63035c9cb11b8decfd
environment_browser.js: https://github.com/onsip/SIP.js/commit/8a538896bf268f8760eb35b628abac0eca62cd3f

As mentioned in the commit messages, these changes were made to help support non-browser environments, such as Node. In the past, I've suggested that NW.js users run the browserified build to avoid problems with global references, but it be nice if that wasn't necessary.
Unfortunately, simply replacing global with window in environment_browser.js causes a ReferenceError when running in Node (something that the automated test suite currently fails to check :disappointed:), so something a little more complex will be needed to ensure full platform compatibility. Pull requests are welcome :)
Alternatively, you could require sip.js/src/SIP directly and supply it with an NW.js-compatible environment, similarly to how index.js does it.

Alternatively, you could require sip.js/src/SIP directly and supply it with an NW.js-compatible environment, similarly to how index.js does it.

This would be the preferred method.  One of the reasons why such a thing exists is precisely because we cannot anticipate all of the different JavaScript environments that users will choose.  Once an environment becomes popular enough, we can take the time to bring the support into the core library.  That said, in this case it does look like Joseph's working on using a third top-level variable that may cover both cases.  If that "just works," we should merge it in.

Yup, I was curious about it and got it "just working" in nw. @oteren, can you confirm that this works for you?


Set up CI with Azure Pipelines

Part 2 of securing repo. Add CI pipeline to repo to securely build a release copy of RN Gallery once a day and publish AppPackage artifact.
Part 1 - #79

Please add someone like Danny, Alex, or Nick to double-check this change thanks!


Group by on content query web part not working as expected!

Guys...can't be bothered filling out your big form...I need a group by handlebar template on the content query web part.
Make it so.
PS. I've tried adding one but I'm really out of my depth here and I've kind of promised my employer that I can do all this shit so please...I'm begging you...add a group by option to the content query web part.

@MacBoiler as a workaround, you can group by items by creating your own groupBy handlebar helper. Here is an example of a simple groupBy implementation I did for a quick demo, which supports grouping on text/date/url fields :
Refer the following MyExentensions.js file in the custom scripts section of the CQWP with the following code :
ReactContentQuery.ExternalScripts.MyExtensions = {

    /****************************************************************************************************
     * Called BEFORE Handlebars has compiled and rendered the template
     * @param wpContext : The context of the WebPart responsible for calling this callback
     * @param handlebarsContext : The Handlebars' instance associated with the WebPart
     ****************************************************************************************************/
    onPreRender: function(wpContext, handlebarsContext) {

        // Registers the "groupBy" helper
        handlebarsContext.registerHelper('groupBy', function(items, field, options) {
            var groupedItems = [];

            items.forEach(function(item) {
                var groupKey = ReactContentQuery.ExternalScripts.MyExtensions.getGroupKey(item, field);
                var existingGroups = groupedItems.filter((group) => { return group.key == groupKey });

                if(existingGroups.length > 0) {
                    existingGroups[0].items.push(item);
                }
                else {
                    groupedItems.push({ key: groupKey, items: [item] });
                }
            });

            return groupedItems;
        });
    },

    /****************************************************************************************************
     * Returns the group key based on the given item and its field value
     * @param item : The item from which the group key must be extracted
     * @param field : The field/property of the item which must be used to extract the key
     ****************************************************************************************************/
    getGroupKey: function(item, field) {

        // If item has no value for specified field return null
        if(!item[field]) {
            return null;
        }

        // If it's a date, returns the date without the time as a key
        else if(!isNaN(new Date(item[field].rawValue))) {
            var objDate = new Date(item[field].rawValue);
            objDate.setHours(0,0,0,0);
            return objDate.toISOString();
        }

        // If it's an url, returns the url as a key
        else if(item[field].rawValue.Url) {
            return item[field].rawValue.Url;
        }

        // If it's anything else, return the textValue
        else {
            return item[field].textValue;
        }
    }

}

Then in the handlebars template use as following :

{{#each (groupBy items 'FieldOnWhichToGroupBy')}}

Group key : {{key}}

Group items :

    {{#each items}}

{{Field1.textValue}}

{{Field2.textValue}}

    {{/each}}
{{/each}}

This is great Simon-Pierre. I was really sticking on trying to implement my own helper. Got this up and running in less than 10 minutes thanks to you.
Cheers and thanks again.

Also, if anyone else is trying to do a similar modification to the CQWP make sure you remove the :f:/r from the SharePoint generated extension URL. This almost got me...:)

@spplante I'm having some issues with IE11 (sigh) and your CQWP...basically it won't load the helper groupBy JS file. Works on Chrome, Edge no issues.

Any thoughts?
PS. This really is an awesome webpart and I just noticed you are the main author. Great work.

Hmm looks like the script has not been loaded, do you see any errors in IE’s console?

Simon-Pierre, IE11 is throwing CORS errors but the domain is the same for both external file and SharePoint tenant.
Origin https://mytenant.sharepoint.com not found in Access-Control-Allow-Origin header.
I have the deploy resources hosted in an SP library with same domain and no issues. Also, I have a handlebar template accessing same data without a group by and it works fine in IE11.
Particular page has 5 of your CQWP's and the console has 5 of the following:

Origin https://mytenant.sharepoint.com not found in Access-Control-Allow-Origin header
XMLHttpRequest: Network Error 0x80700013, Could not complete the operation due to error 80700013.

Also showing [object error] but this is due to not acccessing the JS file.
I'm looking to integrate the groupby in the solution...any thoughts on this?
PS. Really appreciate your help on this.

Hmm are you using the 1.4.1 version of the WebPart or the original version (1.*)? I might have an idea of what's going on if you are using the original version.
Otherwise, implementing the group by within the solution is definitely doable, you could do just like the helper and return a built in {{groups}} token on which we could iterate in the handlebars template, the "challenge" is to support grouping by all the different types of fields, but otherwise it should be fairly simple to modify the solution accordingly.

@spplante Sorry for being a nube but I'm always amazed at how much I don't know...anyways...are you referring to this<EMAIL_ADDRESS>something else. I have a recent download of your webpart but not sure where to find the webpart version number.

When downloading my package, you get the 1.1.0 version (see the version of Microsoft packages in the package.json) which needs you build the WebPart, upload the sources to a "Public CDN enabled" SharePoint library and then store the CDN url in the write-manifest.json file. If you use this version but you did NOT upload the sources to a public CDN, it means that your write-manifest.json is pointing to my own CDN which is not what you want in production because if I decide to shutdown my SharePoint tenant your WebPart won't work anymore.
See this article for the original way of deploying SPFx components.
Starting at version 1.4, SPFx introduced the includeClientSideAssets which allows to store the sources directly within the .sppkg, instead of having the .sppkg refer a CDN url. This is now the default setting and in my opinion by far a better option. This is why I included a version of the WebPart which uses 1.4.1 packages at the root of the projet, in the content-query-webpart-1.4.1.zip
So long story short, if you did not use the content-query-webpart-1.4.1.zip project, you are using the "old" version which uses a .sppkg that points to a CDN url, which would explain the CORS issue because the code does NOT run on your tenant, it runs on mine, and you are trying to refer javascript files from your tenant :)

@spplante all of what you're saying makes sense so I'm going to bed now (late Australia time) and in the morning (with a fresh head) I will follow your instructions and then let you know how it works out.
Your help is greatly appreciated!

@spplante Just had a quick look an I did change the write-manifest.json to point to my cdnBasePath and this works fine as the webpart will load in IE11 without the groupBy reference in handlebar template.

@spplante I tried the includeClientSideAssets and I was getting errors...will look at this again tomorrow and I agree that this would fix it. Only issue would be performance related with multiple webparts on same page.

Ok so you do use your own CDN which is good news, but this is most likely what causes the error because your code runs on the CDN url which is different from your tenant URL.
The includeClientSideAssets switch only works with the 1.4.1 package, you have to extract the zip to a whole new folder, then build that folder instead. That will generate you a .sppkg with the sources built into it.

@spplante I've found the 1.4.1 package...

I think we're getting closer to the solution. Will let you know how I go. Cheers!

@spplante I've been working on this all morning and I think there was a couple of issues that were converging and one was the CORS which appears to be related to a MS data centre being out of action...anyways I enabled accessing data sources across domains in ie11 and that has removed the CORS errors and now I can see a syntax error...

Looks like ie11 doesn't like the external JS content and this is what I have (with comments removed)...

ReactContentQuery.ExternalScripts.CQWPGroupByHandlebarExtension = {
onPreRender: function(wpContext, handlebarsContext) {
handlebarsContext.registerHelper('groupBy', function(items, field, options) {
var groupedItems = [];
        items.forEach(function(item) {
            var groupKey = ReactContentQuery.ExternalScripts.CQWPGroupByHandlebarExtension.getGroupKey(item, field);
            var existingGroups = groupedItems.filter((group) => { return group.key == groupKey });

            if(existingGroups.length > 0) {
                existingGroups[0].items.push(item);
            }
            else {
                groupedItems.push({ key: groupKey, items: [item] });
            }
        });

        return groupedItems;
    });
},
getGroupKey: function(item, field) {
    if(!item[field]) {
        return null;
    }
    else if(!isNaN(new Date(item[field].rawValue))) {
        var objDate = new Date(item[field].rawValue);
        objDate.setHours(0,0,0,0);
        return objDate.toISOString();
    }
    else if(item[field].rawValue.Url) {
        return item[field].rawValue.Url;
    }
    else {
        return item[field].textValue;
    }
}

}

Let me know what you think? I will keep working on this today. My job depends on it.
Cheers
MacBoiler

@spplante I've worked it out...:)
ie11 does not like es6 arrow functions etc. and only an issue because the external script had not been transpiled.
Converted your external script to es5 and declared ReactContentQuery at top of code...see below...

var ReactContentQuery;
ReactContentQuery.ExternalScripts.CQWPGroupByHandlebarExtension = {
onPreRender: function onPreRender(wpContext, handlebarsContext) {
handlebarsContext.registerHelper('groupBy', function (items, field, options) {
var groupedItems = [];
items.forEach(function (item) {
var groupKey = ReactContentQuery.ExternalScripts.CQWPGroupByHandlebarExtension.getGroupKey(item, field);
var existingGroups = groupedItems.filter(function (group) {
return group.key == groupKey;
});
    if (existingGroups.length > 0) {
      existingGroups[0].items.push(item);
    } else {
      groupedItems.push({
        key: groupKey,
        items: [item]
      });
    }
  });
  return groupedItems;
});

},
getGroupKey: function getGroupKey(item, field) {
if (!item[field]) {
return null;
} else if (!isNaN(new Date(item[field].rawValue))) {
var objDate = new Date(item[field].rawValue);
objDate.setHours(0, 0, 0, 0);
return objDate.toISOString();
} else if (item[field].rawValue.Url) {
return item[field].rawValue.Url;
} else {
return item[field].textValue;
}
}
};

Thanks for all your help spp. Greatly appreciated.

Ah, should have thought about that, I am too used to code in typescript hehe good catch

this is amazing thank you very much.
With a people column i.e. Author, the date function is returned, i assume that is a bug in the example.

The code I provided only supports grouping on text, datetime and url fields, you can provide your own implementation to support people field it should be quite straight forward, just add another if to the getGroupKey function.

thanks i will figure it out, just wanted to say again this is so useful thanks for the great work, have anything else in the pipeline?

Not yet, waiting for the SPFx AppSource integration hehe :)

This group by function works great! Would you know of a way to sort elements nested within each group? I have to use the available "order by" for the main groups, and I think the sub-groups default sort on creation date, but if I could sort things alphabetically, it would be perfect! Any idea for a solution?
I'm very new to handlebars. 😅

@salfej I would modify the implementation of the groupBy helper above and specificy another sortField parameter which you could use to sort the groupedItems variables before returning it using javascript.
You could do something like this for sorting group items by Title using javascript's sort function with a comparer function :
groupedItems.forEach(group => {
    group.items = group.items.sort((a,b) => {
        return (a.Title > b.Title ? 1 : -1);
    });
});

Didn't test it but should be along those lines!

@spplante It's perfect! All I had to change was a[sortField].textValue and it worked perfectly! Thanks! 😄

I'm having some issues, basically it won't load the helper groupBy JS file.
An Error occurred while processing the handle bar template,
Error 404 for the JS file
Any thoughts?

You are sure the url of your file is correct? 404 usually means your file is not found. If the JS file is stored in SharePoint, make sure it is checked in and published.

@MacBoiler did you ever solve this issue? It looks like @spplante had some great suggestions to solve this.
I'll mark the issue as closed, but feel free to re-open it if you need more help. I also  encourage you to take the time to fill the "big form" in the future, as it makes it easier for other community members to help you.
Thanks @spplante for your help.

@hugoabernier yes this issue was resolved and big shoutout to @spplante for all his help.
Also, apologies for my "big form" comment...I was under a bit of pressure and had no sleep. I will fill out the big form in the future.:)


Add more sites to scrape

Hello there,
Check this site https://geonode.com/free-proxy-list/ they contain usually a good bunch of good proxies, is it possible to implement it as well?

Hi. In the form in which the project is now, this is not possible. I have plans to completely rewrite everything, adding the ability to parse any sites, and then I will add this site.


update selscan to 1.1.0b

Note from the author: "If you wish to include low frequency variants in the construction of your haplotypes use --keep-low-freq."
https://github.com/szpiech/selscan/releases/tag/1.1.0b

@josephvitti: The flag --keep-low-freq is now required to include low-frequency variants.

Coverage remained the same at 100.0% when pulling dd0375279f8b375f30bd662e6cc5e3a9ba9ae0bd on update-selscan into 6ad9fbca456940718f849f99896daea69162407f on master.

Coverage remained the same at 100.0% when pulling 6b29255c067892f29160eceaca9f036f47fb077d on update-selscan into 6ad9fbca456940718f849f99896daea69162407f on master.

Coverage remained the same at 100.0% when pulling 1ff4d860cafe03039e2f9d9b8f339b9fbaa1625f on update-selscan into 6ad9fbca456940718f849f99896daea69162407f on master.

Superseded by #23


[nltk_data] Error loading brown: <urlopen error [Errno 61] Connection

Hi,
after installation, this error [nltk_data] Error loading brown: <urlopen error [Errno 61] Connection showed up when I tried to python -m textblob.download_corpora. How can I fix it? Thanks

try updating to the latest version of textblob and trying the download_corpora script again


Recognize and correctly ignore x services attributes<PHONE_NUMBER>

Is the consistency check going to be changed to take into account the x-services values? e.g ignore inconsistencies with the service being changed

Yes - that will come soon.

This is somewhat half baked - closing while I work on it.


Added McQuaid Jesuit High School

Added McQuaid Jesuit High School, Rochester NY
<EMAIL_ADDRESS><EMAIL_ADDRESS>

@kirkl4nd Pull request merged. Thank you!


Added all the algorithms of trees data structure

📥 Pull Request
Description
I have added all the basic an most used algorithms used in the trees data structure in data structures and algorithms.
They are written in format simple to understand and have no bugs and work perfectly.
Fixes # (663)
Type of change

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

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules

@ajay-dhangar please review this pull request

We only need the .md file for our documentation. Please delete the other files. like .cpp

sir these are the c++ files that represent the code.
Should i add a .md file in the current dir for the documentation?
otherwise all i have is the .cpp files for the code

We only need the .md file for our documentation. Please delete the other files. like .cpp

sir these are the c++ files that represent the code. Should i add a .md file in the current dir for the documentation?
otherwise all i have is the .cpp files for the code

Only write code in .md file and delete all .cpp files

Done Sir
Deleted all the .cpp files and integrated all the code in .md file

Done Sir

done👍


CHROME-20: add acls and permissions enrichers when fetching current d…

Only interesting commit cherry-picked. Thanks you very much M. Vpasquier.

I create it to be consistent with the XML definition of the document. It would be relevant to add those enrichers.
IMHO M. Kervern.

Sorry I close it.


no connecting

it keeps saying "invalid client id"
https://gyazo.com/f9395c182150021470af4b4be779cc8b

get your clientID from the discord app or discord in your browser.


Update suggestion for how to include PDBs in package

This is the currently recommend way to include any content in the package that is TFM-specific. Any targets added to the TargetsForTfmSpecificContentInPackage property will be executed once for each TFM in the project, with $(TargetFramework) property set to the correct value.

Is this only going to work with 15.3?
https://github.com/NuGet/Home/issues/4704
https://github.com/NuGet/NuGet.Client/pull/1255

Ah, yes I suppose that's where it's been added. I've been so immersed in the 15.3 preview getting things ready for the .NET Core 2.0 release that I forgot that most people aren't using it yet!

@ctaggart Why have you decided to close this one? I find the advice pretty useful. Isn't it better to include it, but rephrase to indicate that it's valid for VS 2017.3 only?

@zvirja Mistake. I created a master branch and closed the v2 branch, which closed the 3 open pull requests to it.

I'm going to publish 2.2.0 now. AppVeyor has upgraded to VS 2013. I think the timing is right for this update. Can you address @zvirja concerns about TFM and submit a new pull request to master?


Make use of OpenCV in any of your python project

OpenCV is a library of programming functions mainly aimed at real-time computer vision

Hi, I just added some OpenCV related project in below PR:
https://github.com/avirajdongare/BooleanAutocrats/pull/78
It might not real-time computer vision, but quite close with few changes


Flybase with xrefs

As you can see, I have reworked the definition generator to accommodate for adding the additional xrefs on the definitions. I also got rid now if the "no descriotion" descriptions.
This PR merges into fbcv-odk! Feel free to do so whenever you feel like it! :)

Extra xrefs look good - just like the old output.
I thought we decided to leave the 'No description' definitions in #60
By stripping these out, there are now several chemical terms that do not have a reference to the CHEBI term.

Ok, will change this back. Is No description the exact string you favour or something like No description, see CHEBI001, or similar?

"No description." [CHEBI:001]

is fine


Unknown Warning on Pytorch Profiler

🐛 Bug
I get a warning when using the pytorch profiler on my machine, I cannot understand wether it's a real issue, and if it does affect the profiling.
[W record_function.cpp:322] Could not match some of the start callbacks with the corresponding end callbacks, callbacks changed during RecordFunction lifetime; you might be trying to profile the code after profiler is finished

Please reproduce using the BoringModel
I am not able to reproduce it using Collab and the Boring Model, however, copying the BoringModel on my machine and run it (with the addition of profiler="pytorch" in Trainer param) does trigger the warning
To Reproduce
Use following BoringModel
however it does not trigger the warning on Collab, only on my working machine
Expected behavior
No Warning, or at least an understanding why this is happening
Environment

CUDA:
- GPU:
- GeForce GTX 1080
- GeForce GTX 1080
- available:         True
- version:           10.2
Packages:
- numpy:             1.20.1
- pyTorch_debug:     False
- pyTorch_version:   1.8.1+cu102
- pytorch-lightning: 1.3.3
- tqdm:              4.48.0
System:
- OS:                Linux
- architecture:
- 64bit
-
- processor:         x86_64
- python:            3.7.5
- version:           #145-Ubuntu SMP Wed Mar 24 18:08:07 UTC 2021

Thanks for helping me in understanding this warning
cc @carmocca @kaushikb11 @ninginthecloud @rohitgr7

Dear @FrouxBY,
I had the same warning while implementing the PyTorch Profiler. I think it is ok.
Did you visualise your traces ?
Best,
T.C

Dear @tchaton,
Yes, i did visualize my traces, and they seem ok, so I believe it is ok as well, just thought someone as an idea about this warning
This will do for now, and if later on it become a problem, I will ask directly on the pytorch project, (or maybe they will be more documentation on it by this time),
Best

I also get this warning. Any idea what it stems from?

@austinmw We can re-open if you can provide a minimal reproducible script.


Automatically reconnect on Gatt status 19

I'm currently trying to implement a BLE Handling for a Flutter App to handle communication with some IoT Devices.
The Gatt Server of those Devices automatically sends status 19 (Connection closed by peripheral) after a few seconds
Now I ran into some issues with that because if I disconnect and reconnect using the library, the data on services and characteristics gets lost, but there is not enough time to discover the characteristics, set up a monitor for the characteristic I get an indication for and write data to the characteristic.
So I am wondering if there is any possibility to not have to go through the whole process and instead store the data across multiple connects/disconnects.
Thank you

Adding a response for future generations: discovery is a system requirement upon establishing a connection, it has to always be done after connection.


Migrate GitHub organization "drud" to GitHub Organization "ddev"

Is there an existing issue for this?

[X] I have searched the existing issues

Is your feature request related to a problem?
We now have the "ddev" GitHub Organization, thanks to the generosity of the former owner, dgindikin.
So we'll be able to mitigate the confusion that the name "drud" causes for DDEV users.
Describe your solution
Planning:

Figure out all the places that migrating the github org will cause friction

Mentions in documentation like ddev.com and lots of other places
Mentions in external blogs, etc.
Mentions in Twitter etc.

Figure out how to limit that friction. (GitHub is usually good at simple redirects, but org redirects may be different).
Figure out (and test with a dummy org) how to do a transfer like this. Github's support may be helpful, but it's likely we'll just have to change the name of the existing github.com/ddev to something else and then immediately change the name of github.com/drud to github.com/ddev
Figure out how to limit impact on Dan's existing repos; one of them has had recent activity
Homebrew formula migration. Can we also use better names since we're breaking things anyway? Can the formula be integrated into ddev repo?

Execution:

Plan a date.
Let the community know about the coming change and its impact
Confirm with Github support the best way to do it.
Execute the change
Update ddev.readthedocs.io docs references
Update homebrew references. Homebrew will likely be broken, without redirect. This can be tested with a different homebrew tap.
Update DDEV code and make a release with the new code.

It's likely that ddev get, which uses api references, won't work correctly without updates.
Tests download tarballs directly
Image builds download tarballs during builds, so Dockerfiles and related have to be fixed.

Update ddev-get add-ons and ddev-contrib repo.
Update Stack Overflow references
Change repo mirroring (github.com/ddevl) and increase the number of repos

Resources:

Github Renaming an Organization docs.

Describe alternatives
No response
Additional context
@andy-blum suggested that we attempt this, and it WORKED! Thanks @andy-blum !

I wonder if it would be possible/useful to leave drud/ddev in place as a mirror repo
https://docs.github.com/en/repositories/creating-and-managing-repositories/duplicating-a-repository

We have a significant mirror going already on another org, and it's easy to create a new org.
https://github.com/ddevl is updated daily with the most important repos, although that will need to be updated where it pulls from as well.

Amazing news!
Christopher J. Wells
Redfin Solutions, LLC
office: 908-4-DRUPAL
fax:<PHONE_NUMBER>
@.***
www.redfinsolutions.com
On Mon, Oct 3, 2022 at 10:02 AM Randy Fay @.***> wrote:

We have a significant mirror going already on another org, and it's easy
to create a new org.
https://github.com/ddevl is updated daily with the most important repos,
although that will need to be updated where it pulls from as well.
—
Reply to this email directly, view it on GitHub
https://github.com/drud/ddev/issues/4247#issuecomment-1265490209, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AACD7KWTEYVXFTNYNNJTS4TWBLROHANCNFSM6AAAAAAQ3CPKTQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

FYI, GitHub automatically creates redirects from the old URL to the new one if a repository is transfered. Not sure how long the redirects stay though.

The redirects are done very well and are pretty much permanent, but there are loads of things that don't work, like APIs, etc, see Changes that aren't automatic

FWIW: Renaming the org has the potential advantage of not having to disrupt sponsorship. It is also the easiest route since we don't need to copy issues and projects over to a new org.

Thanks @rinchen  - Since we're currently set up with all the sponsors on the rfay account (which we'd like to change) it won't be important here.

I'm thinking we should do the repo-by-repo move. It seems the safest. That way we retain control of both the drud and ddev orgs, so nobody can squat on drud. But it would mean that there would be a couple more things that have to be moved:

[ ] Projects
[ ] Owners/members/people

This should also include DDEV addons too, right?

It will include all repositories that are currently in github.com/drud, unless there are some obsolete useless ones.

Discussions with @mattstein about how to improve this plan in https://discord.com/channels/@me/1015402732640800810/1062119752618741881

I’d make it into its own project with discrete steps and be sure each one has enough detail to act on. And a phased schedule where, once each phase has enough detail, is just down to a matter of choosing dates.

The stated goal is to mitigate confusion, which I’m not sure is totally accurate or descriptive of the effort—which is (I think) to move all drud repositories to ddev to accurately reflect the project’s current organizational ownership and eliminate confusion with its now-gone former parent company.
Planning #1: what specifically is the friction we’re concerned about, and how far-reaching a search should that be? (Analytics would again be useful here to handle top referrers.)
Planning #4: who is Dan? You know, but the lack of specificity there makes it hard for someone like me to suggest anything.
Execution #4: split into phases like “Pre-Migration”, “Migration” (this step), and “Post-Migration” with ETAs for each phase. The most time-sensitive schedule/window is “Migration”. The “Post-Migration” efforts could include redirects, link-building (external sites), and validation checklists for confirming everything’s healthy.
Execution #10: no idea what “increase the number of repos” means.
There are more details in comments that should be part of the description for the effort.
The single project item for this effort lives on a board titled “DDEV pending”, and it’s not immediately clear what should belong on that board. There’s no short description or readme to clarify.

This is being planned out in https://github.com/orgs/drud/projects/56, participation welcome.

This is done! Or good enough. See https://github.com/orgs/ddev/projects/1/views/2


Reenable MLS tests on HIP

Updating the OS and reinstalling the drivers, seems to have fixed the issue for Kokkos. Let's see if it also fixes the issue for ArborX.
Fixes #1029

Retest this please

This doesn't work. I am really confused.

master seems to have some issues, too. The build from #1033 failed HIP too with some exceptions being thrown (here).

Rebased on master to see if #1048 helps.

Let's see if updating the ROCm version helps.

This did not help but more worryingly the Clustering and DetailsClusteringHelpers tests segfault when using ROCm 6.2

This did not help but more worryingly the Clustering and DetailsClusteringHelpers tests segfault when using ROCm 6.2

Indeed.


Node app crashed in prod env

Below is the information why error happened in the prod env

Need to update start scripts in package.json

It seems still error by this logs

Working now!


build-test-staging-prod pipeline image is wrong

Pipeline image res/build-test-staging-prod.png is wrong. The last stage should be called 'prod' and not 'staging' again.

Good flag, we'll fix it up

Thanks, @p2well. We just fixed it!


[GH->HF] Part 2: Remove all dataset scripts from github

Now that all the datasets live on the Hub we can remove the /datasets directory that contains all the dataset scripts of this repository
Needs #4059 to be merged first

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

So this means metrics will be deleted from this repo in favor of the "evaluate" library? Maybe you guys could just redirect metrics to that library.

We are deprecating the metrics in datasets indeed and suggest users to switch to evaluate (via a warning message)
We'll keep the current metrics as they are for now, but they'll be completely removed at one point

I guess this is ready to merge ?
It should break nothing except one rare case:
If someone is using an old version of datasets to try to load a recent dataset. Indeed in that case it fetches the main branch on github to see if it exists. But since we're removing all the datasets, forward fetching won't work anymore.
e.g. if someone uses "imagenet-1k" with a version of datasets that didn't have it at that time.

Let's merge this on monday if we can, to make sure contributors who wanted to merge their dataset PRs here could do it

Alright, merging !


Create new Project fails with 1.8.1

Hi,
I'm running your docker image on Windows 10 1809 and Docker 2.0.3
When i try to create a new project, i receive the following error messages:
spfx@1b6041cceef8:/usr/app/spfx$ yo @microsoft/sharepoint
? ==========================================================================
We're constantly looking for ways to make yo better!
May we anonymously report usage statistics to improve the tool over time?
More info: https://github.com/yeoman/insight & http://yeoman.io
========================================================================== No
 _-----_
|       |    .--------------------------.
|--(o)--|    |      Welcome to the      |

---------´   |  SharePoint Client-side  |     ( _´U_ )    |    Solution Generator    |
/A\    '--------------------------'
|  ~  |
'..'_
´    |° ´ Y
Let's create a new SharePoint solution.
? What is your solution name? spfx-test
? Which baseline packages do you want to target for your component(s)? SharePoint Online only (latest)
? Where do you want to place the files? Use the current folder
Found npm version 6.4.1
? Do you want to allow the tenant admin the choice of being able to deploy the solution to all sites immediately without running any feature deployment or adding apps in sites? No
? Will the components in the solution require permissions to access web APIs that are unique and not shared with other components in the tenant? No
? Which type of client-side component to create? WebPart
Add new Web part to solution spfx-test.
? What is your Web part name? HelloWorld
? What is your Web part description? HelloWorld description
? Which framework would you like to use? React
create package.json
create config/package-solution.json
create config/config.json
create config/serve.json
create tsconfig.json
create .vscode/extensions.json
create .vscode/launch.json
create .vscode/settings.json
create config/copy-assets.json
create config/deploy-azure-storage.json
create config/write-manifests.json
create src/index.ts
create README.md
create .editorconfig
create .gitignore
create gulpfile.js
create tslint.json
create src/webparts/helloWorld/components/IHelloWorldProps.ts
create src/webparts/helloWorld/components/HelloWorld.module.scss
create src/webparts/helloWorld/components/HelloWorld.tsx
create src/webparts/helloWorld/HelloWorldWebPart.ts
create src/webparts/helloWorld/loc/en-us.js
create src/webparts/helloWorld/loc/mystrings.d.ts
create src/webparts/helloWorld/HelloWorldWebPart.manifest.json
create teams/772ee4a5-3d9d-4677-b4dd-a51cc705ea50_outline.png
create teams/772ee4a5-3d9d-4677-b4dd-a51cc705ea50_color.png
npm WARN deprecated<EMAIL_ADDRESS>gulp-util is deprecated - replace it, following the guidelines at https://medium.com/gulpjs/gulp-util-ca3b1f9f9ac5
npm WARN deprecated<EMAIL_ADDRESS>please upgrade to graceful-fs 4 for compatibility with current and future versions of Node.js
npm WARN deprecated<EMAIL_ADDRESS>Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue
npm WARN deprecated<EMAIL_ADDRESS>This module relies on Node.js's internals and will break at some point. Do not use it, and update to<EMAIL_ADDRESS>npm WARN deprecated<EMAIL_ADDRESS><EMAIL_ADDRESS>is no longer maintained. Please, upgrade to core-js@3 or at least to actual version of core-js@2.
npm WARN deprecated<EMAIL_ADDRESS>not actively maintained anymore
npm WARN deprecated<EMAIL_ADDRESS>Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue
npm WARN deprecated<EMAIL_ADDRESS>Browserslist 2 could fail on reading Browserslist >3.0 config used in other tools.
npm WARN deprecated<EMAIL_ADDRESS>Please upgrade to kleur@3 or migrate to 'ansi-colors' if you prefer the old syntax.  You must install peer dependencies yourself.
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules/fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"linux","arch":"x64"})
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>install: node scripts/install.js
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR! A complete log of this run can be found in:
npm ERR!     /home/spfx/.npm/_logs/2019-04-26T12_04_02_319Z-debug.log
  _=+#####!

###########|       .-----------------------------------.
###/    (##|(@)    |          Congratulations!         |
######|   \   |   Solution spfx-test is created.  |
###/   /###|   (@) |  Run gulp serve to play with it!  |
#######  ##|   /   '-----------------------------------'
/##|(@)
###########|
**=+####!
Kind regards
Pascal

This is a known issue with running Docker on Windows and node-sass. We're looking for a solution/workaround but we have nothing to share at this moment unfortunately.

Facing the same problem, wondering if there is solution now. :)

Unfortunately not at the moment. It's an issue with node-sass that hasn't been addressed yet.

Just for reference sake. This is the same issue as: #19 Using yarn helped me on SPFx 1.8.2.
https://github.com/waldekmastykarz/docker-spfx/issues/19#issuecomment-453864414

Is there anything we should do here or is it time to close the issue?

If we can confirm that it's indeed the same issue, then we can absolutely patch the v1.8.1 image with the correct node version.

when I use waldekm/spfx:1.8.1 image, it fails for me. When I use the following image, it works just fine:
FROM node:8.17.0

EXPOSE 5432 4321 35729

RUN npm i -g gulp@3 <EMAIL_ADDRESS><EMAIL_ADDRESS>
VOLUME /usr/app/spfx
WORKDIR /usr/app/spfx
RUN useradd --create-home --shell /bin/bash spfx && \
    usermod -aG sudo spfx && \
    chown -R spfx:spfx /usr/app/spfx

USER spfx

CMD /bin/bash

Thank you for checking @shurick81! So the solution is to bump Node version to 8.17.0, correct?

I've just updated the image so this issue should be fixed now. Thank you! 👏


CB-7794 Node decommissioning fails in CCM enabled cluster due to auto…

…ssh connection restarts

CDPAM-677 shows reverse tunnel autossh connection may restart due to multiple reasons.
This causes few API calls to the CM failing intermittently during cluster scale down. It is known that CB makes a lot of CM API calls during scale down from CDPAM-665.
ClouderaManagerApiFactory is used so that all those calls are retried per retry policy.
In the case of SocketTimeoutException, the retry logic is skipped, because SocketTimeException and SocketException are in the different class hierarchy.
Added toleration rule for SocketTimeoutException as well with a unit test.

./gradlew build

I also merged it to 2.25. We'll see how long it takes to promote 2.25 to prod.


Feature Request: Allow Miniflare to connect to a real KV namespace

I am currently testing something that is heavily reliant on KV and caching and metadata. And it would be great if I could switch Miniflare to switch from using an internal KV-like tool to map to a real KV namespace so we can test against an actual KV store.

I won't have a solid block of time over the next few weeks to work on this yet, so I figured I would document findings here, so future me (or someone else) can jump in and take it a bit further.
Initial investigation is that we would need to create a subclass of abstract class KVStorage defined in https://github.com/mrbbot/miniflare/blob/master/src/kv/storage/storage.ts
Questions/work that needs to be done?

[ ] How do we configure it?
[ ] How do we connect to KV? Do we use the API directly, or would be just child_exec wrangler?
[ ] Implement the KVStorage api.

I will create a branch at some point to start working on it and create a WIP PR to here.

I will have a look, but KV supports returning keys with a prefix right? But do you do the cursor stuff in memory after returning all the keys from the list method on KVStorage? I think it could still work.. I would prefer to keep the configuration the same as a Redis or disk based one.
And the KVStorage bit is not all the much of a mess :D You should see some of the code I have seen over my career.. it is squeaky clean by comparison to some (if not most) of the code I have seen (and possibly written)
But I will try to have a look at see what I think is the best way forward and come back.

I would attach the behavior of this to some flags/configuration:

--kv-local persist KV values to a local storage driver (fs or redis). Any writes/updates remain in local driver
--kv-local-sync pull down everything from KV & save it locally

Otherwise, some equivalent of --kv-namespace <NAME:ID> would map the NAME binding to the ID namespace ID. From there (and for all approaches), interact with the REST API directly, which is much nicer and lighter than spawning wrangler in a child process somewhere * shudders *

Any Update?


Concatenate args to avoid out of bound issues

fixes: https://github.com/joshmedeski/sesh/issues/114
 sesh connect $(sesh list -i | gum filter --limit 1 --placeholder 'Pick a session' --height 60 --prompt='⚡')
panic: runtime error: slice bounds out of range [4:3]

goroutine 1 [running]:
github.com/joshmedeski/sesh/icon.(*RealIcon).RemoveIcon(0x140001288c0?, {0x16f916fad, 0x3})
        /home/runner/work/sesh/sesh/icon/icon.go:55 +0x128
github.com/joshmedeski/sesh/seshcli.App.Connect.func1(0x1400018f958?)
        /home/runner/work/sesh/sesh/seshcli/connect.go:70 +0xb4
github.com/urfave/cli/v2.(*Command).Run(0x140001be2c0, 0x140001288c0, {0x14000111350, 0x3, 0x3})
        /home/runner/go/pkg/mod/github.com/urfave/cli/v2@v2.27.1/command.go:279 +0x754
github.com/urfave/cli/v2.(*Command).Run(0x140001be6e0, 0x14000128780, {0x14000128040, 0x4, 0x4})
        /home/runner/go/pkg/mod/github.com/urfave/cli/v2@v2.27.1/command.go:272 +0x964
github.com/urfave/cli/v2.(*App).RunContext(0x1400016a200, {0x1008088b8?, 0x100a05320}, {0x14000128040, 0x4, 0x4})
        /home/runner/go/pkg/mod/github.com/urfave/cli/v2@v2.27.1/app.go:337 +0x534
github.com/urfave/cli/v2.(*App).Run(0x10070cec7?, {0x14000128040?, 0x0?, 0x0?})
        /home/runner/go/pkg/mod/github.com/urfave/cli/v2@v2.27.1/app.go:311 +0x3c
main.main()
        /home/runner/work/sesh/sesh/main.go:14 +0xb0

Tested solution locally:
bin/sesh-dev connect $(sesh list -i | gum filter --limit 1 --placeholder 'Pick a session' --height 60 --prompt='⚡')
payments-service
switching to tmux session: payments-service

bin/sesh-dev connect "$(sesh list -i | gum filter --limit 1 --placeholder 'Pick a session' --height 60 --prompt='⚡')"
payments-service
switching to tmux session: payments-service

I wasn't able to recreate the out of bound issue but I tested the code and it's working for me too.
Thanks for flagging this and fixing it! 💪

thank you!!! you did an awesome job I just did a one liner 😅


PROGRAM ABORT : Short write to (null)

Hi.
Sometimes afl-fuz creates a conflict.
What's the reason?
Error log
[-] PROGRAM ABORT : Short write to (null)
         Location : write_to_testcase(), afl-fuzz.c:3347

                  american fuzzy lop 2.56b (dns)

┌─ process timing ─────────────────────────────────────┬─ overall results ─────┐
│        run time : 0 days, 0 hrs, 4 min, 53 sec       │  cycles done : 0      │
│   last new path : 0 days, 0 hrs, 0 min, 8 sec        │  total paths : 216    │
│ last uniq crash : none seen yet                      │ uniq crashes : 0      │
│  last uniq hang : none seen yet                      │   uniq hangs : 0      │
├─ cycle progress ────────────────────┬─ map coverage ─┴───────────────────────┤
│  now processing : 211 (97.69%)      │    map density : 1.28% / 1.48%         │
│ paths timed out : 0 (0.00%)         │ count coverage : 2.23 bits/tuple       │
├─ stage progress ────────────────────┼─ findings in depth ────────────────────┤
│  now trying : havoc                 │ favored paths : 9 (4.17%)              │
│ stage execs : 308/408 (75.49%)      │  new edges on : 36 (16.67%)            │
│ total execs : 8777                  │ total crashes : 0 (0 unique)           │
│  exec speed : 27.89/sec (slow!)     │  total tmouts : 0 (0 unique)           │
├─ fuzzing strategy yields ───────────┴───────────────┬─ path geometry ────────┤
│   bit flips : n/a, n/a, n/a                         │    levels : 5          │
│  byte flips : n/a, n/a, n/a                         │   pending : 197        │
│ arithmetics : n/a, n/a, n/a                         │  pend fav : 0          │
│  known ints : n/a, n/a, n/a                         │ own finds : 19         │
│  dictionary : n/a, n/a, n/a                         │  imported : n/a        │
│       havoc : 7/1364, 10/4704                       │ stability : 77.77%     │
│        trim : n/a, n/a                              ├────────────────────────┘
└─────────────────────────────────────────────────────┘             [cpu:100%]
[-] PROGRAM ABORT : Short write to (null)
         Location : write_to_testcase(), afl-fuzz.c:3347

Thanks @shtry. We are aware of this error. This is quite flaky and it seems related to out of memory/disk space. We are making some major changes and will try to include a proper fix for this in our next revisions.

Hi there. I'm having the same error popping up after hours of the fuzzer running (happened multiple times now). There are no problems of space (disk nor memory) on my machine.
@thuanpv Suggestions to debug it or fix it?

@thuanpv Maybe removing the write_to_testscase functionality could just be removed? As I understand it, it's a refuse from AFL which is not needed by AFLnet as the test case is sent over the network. It would also improve performance.

Hi @acidghost . Yes, you are right. That function can be removed because AFLNet does not use it. I tried to do it but it led to errors because some AFL's components depend on the file created by that function. I will have another try when I have more time. If you can do so it would be great and pull requests are always welcome!

Hi everyone in this discussion. The issue might be fixed by my recent commit (b7a3d1c). It would be great if you could try it and let me know the result. I cannot reproduce the problem on my computer.


[Bug]: disk_size on Windows nodes

Welcome to Amazon SSP EKS Accelerator!

[X] Yes, I've searched similar issues on GitHub and didn't find any.

Amazon EKS Accelerator Release version
latest
What is your environment, configuration and the example used?
module "eks" {
    source = "git::https://github.com/aws-samples/aws-eks-accelerator-for-terraform.git"

    tenant            = local.tenant
    environment       = local.environment
    zone              = local.zone

    # EKS CLUSTER
    kubernetes_version       = var.kubernetesVersion
    vpc_id             = module.vpc.vpc_id
    private_subnet_ids = module.vpc.private_subnets   # Enter Private Subnet IDs

    create_eks = true

    enable_windows_support = true
    self_managed_node_groups = {
    "windows" = {
            node_group_name = "windows"
            launch_template_os          = "windows"
            subnet_ids      = module.vpc.private_subnets
            desired_size    = var.windowsNodeCountMin
            min_size        = var.windowsNodeCountMin
            max_size        = var.windowsNodeCountMax
            disk_size       = 100

            k8s_labels = {
                "node.kubernetes.io/os" = "windows"
            }
        }}
}

What did you do and What did you see instead?
I am trying to enlarge the disk size of a self-managed windows node in EKS. However, the additional space is not available and I get disk pressure when my data exceed the default size of 50GB.
The launch template shows correct values:
Volume type: EBS
Device name: /dev/xvda
Size (GiB): 100
Volume type: gp2
Encrypted: yes
The EC instance get two volumes
/dev/sda1 of size 50 GiB
/dev/xvda of size 100 GiB
Additional Information
Running the following command on the instance gives:

PS C:\Windows\system32> get-volume

DriveLetter FriendlyName FileSystemType DriveType HealthStatus OperationalStatus SizeRemaining     Size
----------- ------------ -------------- --------- ------------ ----------------- -------------     ----
                         NTFS           Fixed     Healthy      OK                     19.84 GB 19.87 GB
C                        NTFS           Fixed     Healthy      OK                     11.03 GB    50 GB

PS C:\Windows\system32> get-disk

Number Friendly Name                                                                                Serial Number                    HealthStatus         OperationalStatus      Total Size Partition
                                                                                Style
------ -------------                                                                                -------------                    ------------         -----------------      ---------- ----------
3      Msft Virtual Disk                                                                                Healthy              Online                      20 GB GPT
1      NVMe Amazon Elastic B                                                                                vol0e689853a02aee15f_00000001.   Healthy              Online                     100 GB RAW
0      NVMe Amazon Elastic B                                                                                vol06b0ff78111f7d0e7_00000001.   Healthy              Online                      50 GB MBR

I set the size of the device "/dev/sda1" manually to 100 and now it seems to work:
PS C:\Windows\system32> get-volume

DriveLetter FriendlyName FileSystemType DriveType HealthStatus OperationalStatus SizeRemaining   Size
----------- ------------ -------------- --------- ------------ ----------------- -------------   ----
C                        NTFS           Fixed     Healthy      OK                     76.19 GB 100 GB

I think with the aws-eks-accelerator you cannot change "/dev/sda" as it is hardcoded to "/dev/xvda"
https://github.com/aws-samples/aws-eks-accelerator-for-terraform/blob/main/modules/aws-eks-self-managed-node-groups/self-managed-launch-templates.tf#L25
Also note that it is important that I need the space on drive C:

/dev/sda1 is reserved for root.
xvda is recommended for instance store volumes. Did you try to user a recommended for EBS volumes?
Do you have the ebsnvme-id.exe installed into the instance? If so, can you add the output of its execution?
References:

https://docs.aws.amazon.com/AWSEC2/latest/WindowsGuide/device_naming.html
https://docs.aws.amazon.com/AWSEC2/latest/WindowsGuide/xen-drivers-overview.html

This might be helpful: https://github.com/terraform-aws-modules/terraform-aws-eks/blob/284b038bfda29800d2203d1fce939204c944fd70/workers_launch_template.tf#L448

@ricardosouzamorais if I understand the changes of your PR correctly, a new partition is created for the new device with another drive lettern. I do not see how this could help with this issue, as I need a bigger C drive (for large Docker images).

Hello @schwichti , I am working on reutilizing the launch-template module from PR 223 to let customers specify block device mapping as an entry for launch templates.

@ricardosouzamorais will I have to use karpenter to benefit from the launch-template module?


Create dataset loader for Bud500

Dataloader name: bud500/bud500.py
DataCatalogue: http://seacrowd.github.io/seacrowd-catalogue/card.html?bud500

Dataset
bud500

Description
Bud500 is a diverse Vietnamese speech corpus designed to support ASR research community. With aprroximately 500 hours of audio, it covers a broad spectrum of topics including podcast, travel, book, food, and so on, while spanning accents from Vietnam's North, South, and Central regions. Derived from free public audio resources, this publicly accessible dataset is designed to significantly enhance the work of developers and researchers in the field of speech recognition.

Subsets
-

Languages
vie

Tasks
Automatic Speech Recognition

License
Apache license 2.0 (apache-2.0)

Homepage
https://huggingface.co/datasets/linhtran92/viet_bud500

HF URL
https://huggingface.co/datasets/linhtran92/viet_bud500

Paper URL
https://github.com/quocanh34/Bud500

#self-assign

#self-assign


⚠ Spryr has degraded performance

In 5a2c467, Spryr (https://spryr.com) experienced degraded performance:

HTTP code: 200
Response time: 6310 ms

Resolved: Spryr performance has improved in ec24e3d after 7 minutes.


hyprwm: update packages

Description of changes
Updated packages, made them use finalAttrs instead of recs.

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 229751 run on x86_64-linux 1

8 packages built:

hyprland
hyprland-protocols
hyprland-share-picker
hyprland.dev
hyprland.man
hyprpaper
hyprpicker
xdg-desktop-portal-hyprland

Result of nixpkgs-review pr 229751 run on x86_64-linux 1

8 packages built:

hyprland
hyprland-protocols
hyprland-share-picker
hyprland.dev
hyprland.man
hyprpaper
hyprpicker
xdg-desktop-portal-hyprland


Android: BT Headset button should mute

When using Jitsi with the Android app, and you use a Bluetooth headset, the button on it should mute / unmute the own connection. Currently, strange things happen (I think that the headset is completely disconnected when you press the button).

hey i am new to open source can i work on that issue


Gradle quiet mode

Task List

[ ] Steps to reproduce provided
[ ] Stacktrace (if present) provided
[ ] Example that reproduces the problem (link to git repository is ideal)
[ ] Full description of the issue provided (see below)

Steps to Reproduce

Run a build with ./gradlew -q ...
observe only output is from jreleaser

Expected Behaviour
No output in quiet mode
Actual Behaviour
signing.args has been deprecated since 1.0.0-M1 and will be removed in the future. Use signing.command.args instead
signing.args has been deprecated since 1.0.0-M1 and will be removed in the future. Use signing.command.args instead
plus a patreon message.
Environment Information

Operating System: OSX M1
JReleaser Version: 1.0.0-M1
JDK Version: 17

Deprecation warnings will disappear once v1.0.0 comes out.
The patreon banner appears in the first 3 invocations of the plugin, although it might be a recurrent output on CI if the ~/.jreleaser is not cached.
This being said, having a "quiet" mode in core which may be activated via its executing interfaces (cli, maven, gradle, ant) is a good idea.

Moreover, the cli already has a quiet flag which as it turns out activates the ERROR log level. It'll have to be remapped.

This is not critical, but the reason it's relevant for me, is that for my CLI the release build is with native image.
But for quick development I have a command line script checked into github to build with gradle and then run.
https://github.com/yschimke/okurl/blob/main/okurl

Got it. I can work sometime out soon.


Fix analyse to include lints and analyze --watch to not include excluded files

See https://github.com/dart-lang/sdk/issues/28463, which I think is a regression.
This also fixes the regression introduced by https://codereview.chromium.org/2559773002 whereby we were no longer checking any of the lints.
I couldn't figure out how to test this.


Cannot read properties of null (reading 'send')

TypeError: Cannot read properties of null (reading 'send')
at <REDACTED: user-file-path>:131:12
at f.invoke (<REDACTED: user-file-path>:63:145)
at g.deliver (<REDACTED: user-file-path>:63:2029)
at d.fire (<REDACTED: user-file-path>:63:1667)
at g.$acceptDebugSessionTerminated (<REDACTED: user-file-path>:110:13698)
at runMicrotasks (<anonymous>)
at processTicksAndRejections (node:<REDACTED: user-file-path>:96:5)

Go to Errors Site

I can't figure out where this is. I filed a bug for the missing file path. But it might be from a listener on this event inside an extension


Installing on Mac OS El Capitan

Expected Behavior

Current Behavior
When installing on OS X El Capitan the installation fails because of the System Integrity Protection (SIP) with the following error:
changing mode of /usr/local/bin/ocropus-visualize-results to 755 running install_data creating /System/Library/Frameworks/Python.framework/Versions/2.7/share error: could not create '/System/Library/Frameworks/Python.framework/Versions/2.7/share': Operation not permitted

Possible Solution

I could temporarily disable SIP, but that doesn't seem like a good idea. I tried temporarily removing sip from the needed files, but when booting into recovery mode they aren't even loaded and available to remove the restricted flag.
brew is a possible solution, but there are no packages for matplotlib or lxml.
Steps to Reproduce (for bugs)

Installed new clean copy of MacPorts and updated it
sudo port install py27-scipy py27-matplotlib py27-lxml
curl -O http://www.tmbdev.net/en-default.pyrnn.gz
mv en-default.pyrnn.gz models/
sudo python setup.py install

Your Environment

Python version:
Python 2.7.10

Git revision of ocropy:
fatal: Not a git repository (or any of the parent directories): .git
I downloaded the code with the last revision date Feb 19
Operating System and version:
Mac OS El Capitan 10.11

Do I understand correctly that just the last step setup python setup.py install does not work? If yes, then you can also try to start the ocropus-commands directly, i.e. ./ocropus-nlbin etc. Does this work?

Yes that is correct, just the last step fails. But when I call the command directly I get different errors.
./ocropus-nlbin tests/ersch.png -o book
Traceback (most recent call last):
File "./ocropus-nlbin", line 15, in
import ocrolib
File "/Users/user/ocropy-master/ocrolib/init.py", line 14, in
from common import *
File "/Users/user/ocropy-master/ocrolib/common.py", line 26, in
import PIL
ImportError: No module named PIL

Okay, you need also the Python packages from requirements.txt. Install with
pip install -r requirements.txt

or
conda install --file requirements.txt

Alright, I don't have either pip or condo but I used macports to install the two packages that are listed in the requirements file but not in the PACKAGES file by the following:
sudo port install py27-Pillow py27-numpy
But I still get the exact same error as above, including the last line ImportError: No module named PIL
Pillow is obviously supposed to be a fork of PIL, but it doesn't seem to be cooperating.
I also get the same error if I uninstall Pillow and install py27-pil.

You need to install these packages within python and macports is only installing packages within your system. Try to install pip (or conda) first and then proceed with the line above.


EVA-2334 - Merging detection algorithm

Not Convinced I want to merge this PR. This is mainly to illustrate how we could detect that an horizontal/vertical merge is possible.
There is also the possibility to migrate this test to the validation so that it can be known by the time we ingest.

Implemented in a different PR.


TCS-31 Adding segmenting config file and load function

Comments from Jonny relevant to this PR
https://github.com/AquaQAnalytics/TorQ/pull/409#discussion_r809216753


grove_mcfled: Add C++ and JS examples, utilizing existing GroveLed drive...

...r.
The module adds examples that utilize the existing GroveLed driver for
the Grove Multi Color Flash LED.
Signed-off-by: Jon Trulson<EMAIL_ADDRESS>Signed-off-by: Zion Orent<EMAIL_ADDRESS>

Thanks!  Merged in and closing.


Investigate and remove SymbolicUtils pinning

Some change in SymbolicUtils after 0.19.2 caused a regression.  Investigate and remove the pinning.

Let me provide some details (if beneficial): from the last experiments that I ran, SymbolicUtils 0.19.8 spit out some errors in building the DSGE model from the source file. Therefore it has to be something in between.

I isolated it and think it highly likely to be the same as https://github.com/JuliaSymbolics/SymbolicUtils.jl/issues/433

If we end up need this, we might be able to replace the powers of 2 with 2.0.  Otherwise we can wait for it to be closed.

Alas, moving 1-> 1.0, 2 -> 2.0 didn't seem to provide a workaround so we will have to wait for the bug to be fixed in https://github.com/JuliaAlgebra/DynamicPolynomials.jl


make ser::iterator more general by taking IntoIterator

When I left the comment in the issue I noticed that the code wasn't as general as it could be. Alternatively the IntoIterator could be forwarded all the way into the Serialize function, as to not call into_iter before it is needed. Any preferences?

I like this approach but the return type should be Iterator<I::IntoIter>.

I have to go for a couple hours but feel free to merge this once you get the types right. I think you want:
impl<I> Serialize for Iterator<I>
    where I: iter::Iterator,
          I::Item: Serialize

pub struct Iterator<I>(RefCell<Option<I>>); // no bounds

pub fn iterator<I>(iter: I) -> Iterator<I::IntoIter>
    where I: IntoIterator,
          I::Item: Serialize


Feature/add bundle test

Closes #40

Release Notes
This PR also fixes a potential NullPointerException in the Bundle class


Sharing machinary with iterator-helpers toArray

Hey, I was wondering, since Array.fromAsync(iter) is similar to AsyncIterator.from(iter).toArray() wouldn't it make sense if they shared the same spec text? That is one (for example fromAsync) is defined and the other is defined in terms of it?
(I agree that it makes sense to have both as it's more ergonomic and people will likely reach out to both - though that's not the discussion here)

Certainly it'd be ideal if they shared steps via abstract operations, but that'd likely happen after one of them had landed in the main spec.

That sounds reasonable to me.

For sharing this logic, this difference should be resolved https://github.com/tc39/proposal-iterator-helpers/issues/168. At this moment, core-js use one internal operation for them.

We will discuss this issue (as well as #19 and tc39/proposal-iterator-helpers#168) when we present this proposal for Stage 3 at the upcoming plenary meeting.
Related: #1, #19, tc39/proposal-iterator-helpers#36, tc39/proposal-iterator-helpers#142, tc39/proposal-iterator-helpers#156.


[Help] How do I disable the preview window in certain scenarios?

I am having trouble figuring out how to supress the preview window while:

autocompleting commands (the man pages)
autocompleting folders (ls of the fir)

Currently, due to a commit last week, tab will always start fifc for completion. I would recommend you to change the keybinding to, for example Ctrl+x, using
set -U fifc_keybinding \cx

I have removed this hardcoded usage of tab in https://github.com/gazorby/fifc/pull/49. If you want to try it:
fisher uninstall gazorby/fifc
fisher install ollehu/fifc

until the PR has been merged.


feat: make SQ training sample size configurable

better yet, make it use batch streaming sampling

We can provide sample_rate instead.


Update artifact-sources.md

A typo in the line calling AWS EC2 AMI as an ECS AMI.
Adding description for Version value in case of AWS EC2 AMI artifact. It caused a confusion for the Cx as well as to a general reader also.

Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the maintainers.
What Type of PR is This?

[ ] Issue
[ ] Feature
[ ] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[ ] Tested Locally
[ ] Optional Screen Shoot.

@yogesh-chauhan @sainath-b Please review it once. As discussed with @yogesh-chauhan it has caused confusion to a recently migrated cx Cengage Learning. It was confusing for us also.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app


This is a question about page jumps

I set up a tabs page, A, B, C. like this:
tab1Root: any = HomePage;
tab2Root: any = AboutPage;
tab3Root: any = ContactPage;
When I jump from PageB to a new pageC
The URL becomes ：“http://localhost:8102/#/about/shopping”;
when I reload the shopping page,I have a error:  Component AboutPage is not part of any NgModule or the module has not been imported into your module.
I'm newcomer, hold someone can help me;
here are my code:
`about.ts:
import { Component } from '@angular/core';
import { IonicPage, NavController, NavParams } from 'ionic-angular';
@IonicPage()
@Component({
selector: 'page-about',
templateUrl: 'about.html',
})
export class AboutPage {
lists = Lists
constructor(public navCtrl: NavController, public navParams: NavParams) {
}
ionViewDidLoad() {
}
openPage(page){
this.navCtrl.push(page)
}
}
about.module.ts
import { NgModule } from '@angular/core';
import { IonicPageModule } from 'ionic-angular';
import { AboutPage } from './about';
@NgModule({
declarations: [
AboutPage,
],
imports: [
IonicPageModule.forChild(AboutPage),
],
})
export class AboutPageModule {}
`

Hello! Thank you for opening an issue with us! As this seems like more of a support question I will urge that you ask this question on our forum or on our slack channel. Thanks for using Ionic!


[Net] ENet poll optimizations, fragmented unreliable transfer.

In this PR:

ENetMultiplayerPeer now sends fragmented packets unreliably too.
ENetMultiplayerPeer poll now only service the connection once.

4.0 port of 2 out of the 3 changes in #51466 .
The other change (sendmsg/WSASendTo) needs more testing on the 3.x branch because according to MS docs WSABUF is not available in UWP apps (though WSASendTo is?)
Bugsquad edit: master version of https://github.com/godotengine/godot/pull/53130.

if porting to 4.0 branch, mind throwing a co-author on the 4.0 commits?

Thanks :) And thanks to @jordo too!


BinRead for Option

impl<T: BinRead> BinRead for Option<T> {
    type Args<'a> = T::Args<'a>;

    fn read_options<R: Read + Seek>(
        reader: &mut R,
        endian: Endian,
        args: Self::Args<'_>,
    ) -> BinResult<Self> {
        Ok(Some(T::read_options(reader, endian, args)?))
    }
}

Wouldn't it be better if it removed None in case buffer is empty?

Hello, we tried to solve the issue.
This is what we did:
Modified the BinRead implementation for Option to return None if the buffer is empty. This change improves the intuitive behavior of the Option type when reading from an empty buffer.
You can review changes in this commit: https://github.com/mnmnk43434/jam1garner-binrw-302/commit/6a17c6633295a91cb9581c62f9ff5942d61e6fe5.

[!CAUTION]
Disclaimer: The concept of solution was created by AI and you should never copy paste this code before you check the correctness of generated code. Solution might not be complete, you should use this code as an inspiration only.

If you no longer want Latta AI to attempt solving issues on your repository, you can block this account.

that's... actually correct I think? good bot

Hi, thanks for your request!
Option<T> is typically used in conjunction with the if or try directives when parsing to a None. If you want an Option<T> to be fallible, the try directive will do it. Changing the default implementation to return None in EOF condition would make detection of parsing errors harder, and so I don’t think is a desirable default. Does this make sense?
Thanks,

Since there hasn’t been a response and I think that the current behaviour is sound I am going to close this issue for now. Please feel free to comment later if you think it should be changed and it can be reopened. Thanks!


Add Ocean variables to the ACCESS-Live CMORiser

Description

Closes #issue_number
Link to documentation:

Before you get started

[ ] ☝ Create an issue to discuss what you are going to do

Checklist
It is the responsibility of the author to make sure the pull request is ready to review. The icons indicate whether the item will be subject to the 🛠 Technical or 🧪 Scientific review.

[ ] 🧪 The new functionality is relevant and scientifically sound
[ ] 🛠 This pull request has a descriptive title and labels
[ ] 🛠 Code is written according to the code quality guidelines
[ ] 🧪 and 🛠 Documentation is available
[ ] 🛠 Unit tests have been added
[ ] 🛠 Changes are backward compatible
[ ] 🛠 Any changed dependencies have been added or removed correctly
[ ] 🛠 The list of authors is up to date
[ ] 🛠 All checks below this pull request were successful

To help with the number pull requests:

🙏 We kindly ask you to review two other open pull requests in this repository

I forgot to add config_develop.yml when I push it back to remote, my bad. now I have uploaded it and I use dataset 'HI-CN-05' instead because the time range is from 1850-2014, there is also a test_recipe folder which contain the recipe for quick test in the first level of repo.


Document that DNS-01 ClusterIssuer use kube-system secret

When using a ClusterIssuer with DNS-01 ans route 53, you have to reference a secret that live inside the kube-system namespace

/assign @jakexks

@jakexks this PR is meant to fix this issue 650


Validation of number of in- and outneighbors by node type

Fixes https://github.com/Deltares/Ribasim/issues/333.

For the model in the issue you now get this:
│ Nodes of type Ribasim.ManningResistance must have at least 1 inneighbors (got 0 for node #5).
│ Nodes of type Ribasim.ManningResistance must have at least 1 inneighbors (got 0 for node #7).
│ Nodes of type Ribasim.ManningResistance must have at least 1 inneighbors (got 0 for node #9).
│ Nodes of type Ribasim.ManningResistance must have at least 1 inneighbors (got 0 for node #11).
│ Nodes of type Ribasim.ManningResistance must have at least 1 inneighbors (got 0 for node #13).
│ Nodes of type Ribasim.ManningResistance must have at least 1 inneighbors (got 0 for node #15).
│ Nodes of type Ribasim.ManningResistance must have at least 1 inneighbors (got 0 for node #17).
│ Nodes of type Ribasim.ManningResistance must have at least 1 inneighbors (got 0 for node #19).
│ Nodes of type Ribasim.ManningResistance must have at least 1 inneighbors (got 0 for node #21).
│ Nodes of type Ribasim.ManningResistance must have at least 1 inneighbors (got 0 for node #23).
│ Nodes of type Ribasim.ManningResistance must have at least 1 inneighbors (got 0 for node #25).
│ Nodes of type Ribasim.ManningResistance must have at least 1 inneighbors (got 0 for node #27).
│ Nodes of type Ribasim.ManningResistance must have at least 1 inneighbors (got 0 for node #29).
│ Nodes of type Ribasim.ManningResistance must have at least 1 inneighbors (got 0 for node #31).
│ Nodes of type Ribasim.ManningResistance must have at least 1 inneighbors (got 0 for node #33).
│ Nodes of type Ribasim.ManningResistance must have at least 1 inneighbors (got 0 for node #35).
│ Nodes of type Ribasim.ManningResistance must have at least 1 inneighbors (got 0 for node #37).
│ Nodes of type Ribasim.ManningResistance must have at least 1 inneighbors (got 0 for node #39).
│ Nodes of type Ribasim.ManningResistance must have at least 1 inneighbors (got 0 for node #41).
│ Nodes of type Ribasim.ManningResistance must have at least 1 inneighbors (got 0 for node #43).
│ Nodes of type Ribasim.ManningResistance must have at least 1 inneighbors (got 0 for node #45).
│ Nodes of type Ribasim.ManningResistance must have at least 1 inneighbors (got 0 for node #47).
│ Nodes of type Ribasim.ManningResistance must have at least 1 inneighbors (got 0 for node #49).
│ Nodes of type Ribasim.ManningResistance must have at least 1 inneighbors (got 0 for node #51).
│ Nodes of type Ribasim.ManningResistance must have at least 1 inneighbors (got 0 for node #53).
│ Nodes of type Ribasim.ManningResistance must have at least 1 inneighbors (got 0 for node #55).
│ Nodes of type Ribasim.ManningResistance must have at least 1 inneighbors (got 0 for node #57).
│ Nodes of type Ribasim.ManningResistance must have at least 1 inneighbors (got 0 for node #59).
│ Nodes of type Ribasim.ManningResistance must have at least 1 inneighbors (got 0 for node #61).
│ Nodes of type Ribasim.ManningResistance must have at least 1 inneighbors (got 0 for node #63).
│ Nodes of type Ribasim.ManningResistance must have at least 1 inneighbors (got 0 for node #65).
│ Nodes of type Ribasim.ManningResistance must have at least 1 inneighbors (got 0 for node #67).
│ Nodes of type Ribasim.ManningResistance must have at least 1 inneighbors (got 0 for node #69).
│ Nodes of type Ribasim.ManningResistance must have at least 1 inneighbors (got 0 for node #71).
│ Nodes of type Ribasim.ManningResistance must have at least 1 inneighbors (got 0 for node #73).
│ Nodes of type Ribasim.ManningResistance must have at least 1 inneighbors (got 0 for node #75).
│ Nodes of type Ribasim.ManningResistance must have at least 1 inneighbors (got 0 for node #77).
│ Nodes of type Ribasim.ManningResistance must have at least 1 inneighbors (got 0 for node #79).
│ Nodes of type Ribasim.ManningResistance must have at least 1 inneighbors (got 0 for node #81).
│ Nodes of type Ribasim.ManningResistance must have at least 1 inneighbors (got 0 for node #83).
│ Nodes of type Ribasim.ManningResistance must have at least 1 inneighbors (got 0 for node #85).
│ Nodes of type Ribasim.ManningResistance must have at least 1 inneighbors (got 0 for node #87).
│ Nodes of type Ribasim.ManningResistance must have at least 1 inneighbors (got 0 for node #89).
│ Nodes of type Ribasim.ManningResistance must have at least 1 inneighbors (got 0 for node #91).
│ Nodes of type Ribasim.ManningResistance must have at least 1 inneighbors (got 0 for node #93).
│ Nodes of type Ribasim.ManningResistance must have at least 1 inneighbors (got 0 for node #95).
│ Nodes of type Ribasim.ManningResistance must have at least 1 inneighbors (got 0 for node #97).
│ Nodes of type Ribasim.ManningResistance must have at least 1 inneighbors (got 0 for node #99).


Parsing date/time in the past in :pt

added some translations for parsing past dates

pull request sent


Add user account page

A page for the user to manager there account such as notifications and changing email / password.

Will be closed in #33.


Change naming configuration

Updates #116
Currently api_name is parsed to derive many smaller pieces of information:

a one-word short name
a major version
a path to the source protos in the googleapis repo for package metadata
the name of the package

Instead of parsing this information out, it makes more sense to configure these values separately. This avoids corner cases where the short name may need to consist of multiple words (e.g., error-reporting, iam-admin), or where the name of the package may not match the path to the protos in the googleapis repo.
This PR separates out the short_name and version configuration. A subsequent PR will replace the source to the protos currently parsed from the api_name with the existing src_proto_path configuration value.

Current coverage is 74.18% (diff: 100%)

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

@@             master       #123   diff @@
==========================================
  Files            36         36
  Lines          1317       1321     +4
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits            977        980     +3
- Misses          340        341     +1
  Partials          0          0

Powered by Codecov. Last update 6e6c9a4...b9815e7

No problems jump out at me, but I'd like someone else to review.

@garrettjonesgoogle I've made the changes requested here and in https://github.com/googleapis/googleapis/pull/215; PTAL.

LGTM


remove blockquote tag

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/resources/bot-v3/bots-text-formats.md
92
2
link
:white_check_mark:

msteams-platform/task-modules-and-cards/cards/cards-format.md
95
12
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

The blockquote in this repo may not be changed/removed. Report it to: https://dev.azure.com/ceapex/Engineering/_workitems/edit/885381
Close this PR at this point.


VERSION variable does not provide the requried version

What happened:
The get script always returns v0.9.1, even when trying to specify a custom VERSION:
$ curl -Ls https://get.submariner.io VERSION=0.9.0 | bash
Installing subctl version latest
  OS detected:           linux
  Architecture detected: amd64
  Download URL:          https://github.com/submariner-io/releases/releases/download/v0.9.1/subctl-v0.9.1-linux-amd64.tar.xzDownloading...
subctl-v0.9.1-linux-amd64 has been installed as /home/nyechiel/.local/bin/subctl
This provides subctl version: v0.9.1

What you expected to happen:
The right version to be obtained.

@nyechiel VERSION needs to be specified after the pipe. This way it works for me.
curl https://get.submariner.io | VERSION=devel bash
$ subctl version
subctl version: devel

curl https://get.submariner.io | VERSION=0.9.1 bash
$ subctl version
subctl version: v0.9.1

Oh wow, you are correct! Closing.


Blending issue on object set to active/disabled.

When the object is set to disabled, and then re-enabled the blending values set on the editor seem to get lost. (This happens when testing on iOS, doesn't seem to happen on the preview view itself)

Same for android.
See the attached file to see the difference between regular particle effect and after the game object has been disabled and enabled.
Before
After

Fixed it by adding two lines about currentBlendSource and currentBlendDestiation to the method
  void ClearDown()
        {
            // Reset current blend modes.
            currentBlendDestination = -1;
            currentBlendSource = -1;

            this.emitterConfig = null;
            this.initialized = false;
        }


Unable to remove dependency from requirement.txt

When I remove any dependency from requirements.txt sls deploy fails with the following exception:
Serverless: Generated requirements from /home/username/project/requirements.txt in /home/username/project/services/upload_set/.serverless/requirements.txt...
Serverless: Installing requirements from /home/username/.cache/serverless-python-requirements/34ce6d0ed46b1549c2a9244b638729a297cdb3cc59ee967627010e6301a0fd70_slspyc/requirements.txt ...
Serverless: Docker Image: lambci/lambda:build-python3.6
Serverless: Using download cache directory /home/username/.cache/serverless-python-requirements/downloadCacheslspyc

  Error --------------------------------------------------

  Error: EACCES: permission denied, open '/home/username/.cache/serverless-python-requirements/downloadCacheslspyc/requirements.txt'
...

Here is my pythonRequirements setup:
pythonRequirements:
  noDeploy: []
  dockerizePip: true
  fileName: ../../requirements.txt

I ended up adding the dependency I wanted to remove to noDeploy and not use it in code. Thanks.

Setting useDownloadCache: false solved this issue

@neotheicebird I'm going to close it, feel free to re-open if you think we need to look at this further.


fresh installation -> "cordova prepare" not working

make dir "www" under "cordova" to run "cordova prepare" without error.

@dburucu Yes, a directory called www will need to be made. However, this is already listed in the README.md -> https://github.com/sts-ryan-holton/nuxt-cordova-app#installation

Note: if cordova prepare fails, check that the cordova/www directory exists, if it doesn't create it.


Not Found Tips.app

Phone: iPhone SE (2020)
iOS: 16.4
Version: Picasso v3.1
Problem: Error loads when trying to apply TrollStore 2 Tweak. This is the error details:
"Error
Failed to apply tweak TrollStore 2
Error:
Not found tipps.app. You can try reinstalling the Tips app, rebooting and trying again."
Tried this about 50 times and no change. I can use the Tips app fine and I've uninstalled it and reinstalled it and totally factory reset my phone. no change.

Only solution I was able to find was to use the "Aggressive applying" setting under "Experimental" category in "Settings". Worked like a charm. But, I'm not sure why it wouldn't apply without it


Option for chat reversed order

Is your feature request related to a problem? Please describe.
When I stream I would rather look at the top of my screen vs the bottom of the screen to see the latest messages. I attempted to do this myself, but I can't figure out where to make the changes.
Describe the solution you'd like
If we can get a setting somewhere to reverse the chat so the latest messages are at the top of the screen, and you scroll down to see old chat, that would be awesome for me.
Describe alternatives you've considered
I attempted to modify the code, but couldn't figure it out. I had originally assumed it would be pretty simple, but it was past my beginner React knowledge.
Additional context
Thanks! (I should also mention, if you want to just point me to the right direction for modifying the List component, I can try to make the changes)

This feature is not something I'm ready to tackle right now (not scratching it at all from my list tho) due to the time I can put in YaTA right now and the time this change would involve as when you get the list rendered in the other direction, there would still be lots of adjustments to make it look good & polished:

Adjust all the automatic scrolling behavior to take this option into consideration.
Adjust the pause feature to take this option into consideration.
Make sure every places showing messages use this option like the chat log for a specific user, etc.
Make sure the order is respected when copying multiple chat messages, exporting logs, etc.

On the other end, I did some researchs / experiments in case you would want to have a basic / rough version for you locally or even setup a complete PR and get you started. You're mentionning List and I think you're referring to the react-virtualized component but I don't think it's the correct approach.
I think modifying the getLogs() Redux selector (src/store/selectors/logs.ts) would be a better choice. You could compose it with another selector or take a parameter for the order (first option sounds like the better one) and just simply change the logs order. For example, for a very basic approach, just chaining .reverse() to the logs _.map already produce what you want:

For a more robust approach, I think using _.reduceRight instead of chaining a .reverse() after the _.map might be better to avoid the extra overhead.
If you have any more question regarding this or if you end up trying that yourself and hit an issue, do not hesitate to comment in this issue, I'll be happy to help.

Great, thanks @HiDeoo I'll give that a shot. No worries at all regarding not being able to do it now.

I got it to work with the changes you suggested, but you're right, it definitely messes up the scrolling. I was able to prevent the auto scrolling, but it would definitely require some logic to handle the 'pause scrolling' and other stuff.
After using it for a stream, I don't think I'm going to continue with it, it confused me too much I think. 😂

Ho boy, you managed to last for 1 entire stream, I gave up with my very basic approach after like 5mn, I was so confused 😂


Fix #629

Fix issue #629

Looks great @cppchedy - you're a committer now so you can merge this :)

yeah thx for that but won't merge anything until I get review!