Non-recursive Merge sort

Submission Title
MergeSort
Submission Type
Algorithm
Submission Description
provide a non-recurrsive way to merge sort
Submission Directory
Algorithm => Sorting_Algorithms => Merge_sort => non_recursive_merge_sort.c
Contributors

[x]  C++ -
[x]  C -
[x]  Python -
[x] Java -
[ ] JavaScript -
[ ] TypeScript -
[ ] Golang -
[ ] Ruby -
[ ] php -
[ ] Swift -
[x] C# -

Please assign this to me for python

Please assign me this for C++ or PHP

Can i work on this issue using java? @divigarg

Hii, can you please assign me this issue  in c?
Thank you

hey! can I work on this issue using c?

Hey can I work this issue in java or python

thanks everyone :v:


Basic nav bar on all templates

Nav bar should navigate to

User profile
Create expression page
Query expression page
Groups page

@sunyatax please could you give us a user .svg for the navbar? Or point us in the direction of one to use


Add continuous integration for druid

This adds Travis CI support. It's based on #1. We must merge #1 first and then rebase this on top of master before merging this.

Looks like that worked. Not 100% sure if there's anything more I need to configure so new PR's will have their CI automatically reported.


test: tests for schema and model

Landed in https://github.com/metarhia/metasql/commit/472306ae3a1a0d23976343b28ee46dbdde1e3a7e


ERR_MODULE_NOT_FOUND When importing from the root in Next.js

Hey there 👋
First off, this project is fantastic! I haven't gone looking for a codeblocks lib in a while and had no idea this existed. Well documented and exactly what I needed.
One note, when using with the /pages router in Next.js, there is a ERR_MODULE_NOT_FOUND error unless the import directly points to the code-block file i.e.:
import { CodeBlock, CodeBlockProps } from "react-code-block/dist/code-block";

If it's helpful, this is the full output
⨯ Error [ERR_MODULE_NOT_FOUND]: Cannot find module '/Users/makennasmutz/Documents/GitHub/makenna.dev/node_modules/.pnpm/react-code-block@1.0.0_prism-react-renderer@2.3.1_react-dom<EMAIL_ADDRESS>imported from /Users/makennasmutz/Documents/GitHub/makenna.dev/node_modules/.pnpm/react-code-block@1.0.0_prism-react-renderer@2.3.1_react-dom<EMAIL_ADDRESS>    at finalizeResolution (node:internal/modules/esm/resolve:263:11)
    at moduleResolve (node:internal/modules/esm/resolve:908:10)
    at defaultResolve (node:internal/modules/esm/resolve:1121:11)
    at ModuleLoader.defaultResolve (node:internal/modules/esm/loader:396:12)
    at ModuleLoader.resolve (node:internal/modules/esm/loader:365:25)
    at ModuleLoader.getModuleJob (node:internal/modules/esm/loader:240:38)
    at ModuleWrap.<anonymous> (node:internal/modules/esm/module_job:85:39)
    at link (node:internal/modules/esm/module_job:84:36)

Hi !
Awesome project, love the clean design and documentation.
I am also encountering similar error on nextjs app, when Codeblocks is invoked inside /pages
Error [ERR_MODULE_NOT_FOUND]: Cannot find module '/Users/shelciaabi/Documents/code stuff/reactfile/api
docucraft/client/node_modules/react-code-block/dist/code-block' imported from /Users/shelciaabi/Documents/code
stuff/reactfile/api-docucraft/client/node_modules/react-code-block/dist/index.js

@shelcia if you direct the import, should work. i.e.
import { CodeBlock, CodeBlockProps } from "react-code-block/dist/code-block";

Also have this issue but this worked for me

@shelcia if you direct the import, should work. i.e.
import { CodeBlock, CodeBlockProps } from "react-code-block/dist/code-block";

This is now fixed in version 1.0.1 via https://github.com/blenderskool/react-code-block/commit/3c7f97c1bb7dcb9fcd5a6d108fd46be4fe2f6d00


Move RestoreNodeDrainState up in clean.All()

Previously if the RestoreNodeDrainState was set to true but the test namespace didn't exist,
the restoration wouldn't happen because the func returns nil too early

/lgtm


How to disable?

I want to disable the component, but no attr can set to it? any one who have sulotion?

You can use disabled property.
<Datepicker disabled={true} />

It's my fault missing it in the doc

@feyy why don't you add it in doc now at least....

@feyy is it possible to change the style of the disabled datepicker..It would be helpful

@santhosh77h did you manage to change the style of the disabled datepicker ?

Never mind, just found how
For those reading this:
  <DatePicker
          disabled={this.state.show}
          customStyles={{
            disabled:{
              backgroundColor: 'green',
            },


<cmd/pilot-agent>: check node type

pilot-agent should check node-type,and give more accurate typo.

Hi @knightXun. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/retest

@knightXun: Cannot trigger testing until a trusted user reviews the PR and leaves an /ok-to-test message.

In response to this:

/retest

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@nmittler is Jenkins broken again? so sad

/ok-to-test

@knightXun after you fix linter error it should be good to go.

@knightXun: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
a2856b4157b78e17c131fa58af0f90f8588fbf71
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@nmittler i will wait jenkins works fine

ping @nmittler

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: nmittler
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/OWNERS [nmittler]
pkg/OWNERS [nmittler]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@knightXun: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
a2856b4157b78e17c131fa58af0f90f8588fbf71
link
/test istio-unit-tests

prow/istio-pilot-multicluster-e2e.sh
a2856b4157b78e17c131fa58af0f90f8588fbf71
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Add EverSQL

Hi @etai4542 Thanks a lot for your effort & contribution here! Approved 🎉
Once again, many thanks! ✨ Best wishes,


Fix display of document table

The status field in the API was split into two new fields: ingestion_status and restructuring_status. I modified the code to use the former.
Without this fix, the table would never display rows.

Thanks @aphedges! Great catch!


Multi-Select checkboxes in Tables (UI/UX)

As ...
Nelly - Network Engineer
I want ...
I want to easily click on multiple checkboxes in a table.

So that ...
I don't need to click on every single checkbox for e.g. set interfaces to disabled with bulk edit
I know this is done when...

I select multiple lines in tables

Optional - Feature groups this request pertains to.

[ ] Automation
[ ] Circuits
[ ] DCIM
[ ] IPAM
[ ] Misc (including Data Sources)
[ ] Organization
[ ] Plugins (and other Extensibility)
[ ] Security (Secrets, etc)
[ ] Image Management
[X] UI/UX
[ ] Documentation
[ ] Other (not directly a platform feature)

Database Changes

None

External Dependencies
No response

Certainly will be a feature of the new UI in 2.0

Shift click as an option is OK but I also think that using ctrl and mark a block should be possible to extend with another region. This is not possible if just Shift click is implemented


DRAFT: Add data-agnostic Vendor extension area support

This change introduces the VendorFunction, with the intent that users of this API will:

explicitly provide the EID and FID that they wish to use
the EID will be in the Vendor SBI extension range
they know a priori the function signature, and use the appropriate harness

TODO:

API front-end
Cascade of 'known' Vendor types before using generic handler?
Tests

I'm going to withdraw this PR in favour of #4, which I think offers a cleaner starting point.


WebGL mask+filter issue

Can not use any filter in WebGL if a mask is applied to sprite or any container in hierarchy.
The following example shows the issue:
var stage0 = new PIXI.Stage(0x66FF99);

var rendererType=1;
var addMask=true;

var renderer =  null;
if(rendererType==0)
{
    renderer =  new PIXI.CanvasRenderer(800, 600, null, false);
}
else
{
    renderer =  PIXI.autoDetectRenderer(800, 600, null, false);
}

document.body.appendChild(renderer.view);

var container0= new PIXI.DisplayObjectContainer();
stage0.addChild(container0);

if(addMask)
{
    var clippingMask0 = new PIXI.Graphics();
    clippingMask0.beginFill();
    clippingMask0.drawRect(0, 0, 600, 300);
    clippingMask0.endFill();
    container0.mask = clippingMask0;
    container0.addChild(clippingMask0);
}

var texture0 = PIXI.Texture.fromImage("http://www.goodboydigital.com/pixijs/examples/14/panda.png");

var sprite0 = new PIXI.Sprite(texture0);
container0.addChild(sprite0);
var fltr = new PIXI.GrayFilter();
sprite0.filters = [fltr];

requestAnimFrame(animate);

function animate() {
    requestAnimFrame(animate);
    renderer.render(stage0);
}

As can observe, the sprite is not visible! If use Canvas renderer by setting "rendererType=0;" or use WebGl renderer but don't add the mask (by setting "addMask=false;"), the sprite will be visible.

This code runs like a boss in v3. :)

Yes, it is fixed!
Thank you : )

Could it be that this issue is back in Version 4?
I'm new to Pixi, but I've been searching for a week at least trying to find any hints on why I can only apply filters to my graphic object directly.
I create a circle texture in code like this:
`var graphic = new PIXI.Graphics();
graphic.lineStyle(0);
graphic.boundsPadding = 20;
graphic.beginFill(liquidColor, 1);
graphic.drawCircle(0, 0, 3);
graphic.endFill();
graphic.filters = [ blurFilter ];
this.texture = this.app.renderer.generateTexture(graphic);`

This works fine and the blur is also applied, but no matter where, if I try to apply the blur to the container, the stage, any extra container I create - it all either does absolutely nothing, or nothing is visible at all any more but the background color of the stage.

No console errors btw.

That's not supposed to happen. Make a fiddle and I'll look whats wrong. Which version is it?

FYI @Dinkelborg, keep in mind filters won't show up on your root stage object.

Hey @ivanpopelyshev,
I made a narrowed down version of my project here: https://codepen.io/Dinkelborg/pen/dVBaWd
I marked the lines with comments where I tried to apply the filter (lines: 12,16,19,28,45)
I didn't expect each of those to work and as @bigtimebuddy mentioned, I also found the Gotcha page for v4 so I expected that filters on stage wouldn't work, I was just trying what I could ... basically what I did expect to work was the "spriteContainer" (line 16)
Basically as I understood the info I found online I should be able to apply any filter on any container as long as it's not the stage root one or a particle container, right?

May be this will help you: particleContainer has no way to know what bounds are, because it will affect performance. Thus, you should manually set filterArea on container, you can take whole screen if you want.
container.filters = [...];
container.filterArea = renderer.screen; //same as app.screen, its Rectangle

@ivanpopelyshev
Awesome - this fixed it!
Yeah it would be really cool if this could be added to the filter docs maybe even highlighted as a tip - so that if you are looking up how a filter works it will remind you that you have to set a filter area for your container.
Thanks a lot for the quick help


Add connection with OAuth and Github

Add authorization by github and o auth standard with component:
https://github.com/sahat/satellizer

add ui-router in this issue to

add authorization logon by github and logout from app


Fix link to NLP-ADBench

Fix link to NLP-ADBench in README
All Submissions Basics:

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
[x] Have you checked all Issues to tie the PR to a specific one?

All Submissions Cores:

[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your core changes, as applicable?
[ ] Have you successfully ran tests with your changes locally?
[ ] Does your submission pass tests, including CircleCI, Travis CI, and AppVeyor?
[ ] Does your submission have appropriate code coverage? The cutoff threshold is 95% by Coversall.

New Model Submissions:

[ ] Have you created a .py in ~/pyod/models/?
[ ] Have you created a _example.py in ~/examples/?
[ ] Have you created a test_.py in ~/pyod/test/?
[ ] Have you lint your code locally prior to submission?

Pull Request Test Coverage Report for Build<PHONE_NUMBER>8
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

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

Totals

Change from base Build<PHONE_NUMBER>0:
0.0%

Covered Lines:
10361

Relevant Lines:
10865

💛  - Coveralls


Tailwind css does not work on Windows

Hi guys!
I downloaded a theme and edited the file package.json to add some dependencies, then I removed all the node_modules folder and hit yarn. Installation concludes with success but when i hit yarn start I get the following error:
".\node_modules\.bin\tailwind" is not recognized as internal or external command

so i run yarn add tailwindcss --dev and yarn getting the following error
"postcss" is not recognized as internal or external command

so i run yarn add postcss --dev and yarn getting the first error
Configuration file is
{
    "name": "AAA",
    "version": "1.0.0",
    "description": "AAA",
    "author": "AAA",
    "license": "UNLICENSED",
    "homepage": ".",
    "private": true,
    "scripts": {
        "start-webpack": "cross-env webpack-dev-server --config=webpack.dev.js --open --progress --colors --display-error-details",
        "build-webpack": "cross-env webpack --config=webpack.prod.js --progress --colors --display-error-details",
        "start": "yarn run build-css-helpers && cross-env NODE_PATH=src react-scripts start",
        "build": "yarn run build-css-helpers && cross-env NODE_PATH=src CI=false react-scripts build",
        "start-no-css": "cross-env NODE_PATH=src react-scripts start",
        "build-no-css": "cross-env NODE_PATH=src react-scripts build",
        "build-css-helpers": "./node_modules/.bin/tailwind build ./src/styles/fuse-helpers.tailwind.css -c ./tailwind.js -o ./src/styles/fuse-helpers.css && postcss --use autoprefixer -o ./src/styles/fuse-helpers.css ./src/styles/fuse-helpers.css",
        "test": "cross-env NODE_PATH=src react-scripts test --env=jsdom",
        "eject": "cross-env NODE_PATH=src react-scripts eject",
        "build-docs": "babel-node src/main/content/components/material-ui/build.js"
    },
    "dependencies": {
        "@material-ui/core": "1.4.3",
        "@material-ui/icons": "2.0.1",
        "autosuggest-highlight": "3.1.1",
        "axios": "0.18.0",
        "axios-mock-adapter": "1.15.0",
        "babel-plugin-import": "^1.9.1",
        "chart.js": "2.7.2",
        "downshift": "2.0.20",
        "formsy-react": "1.1.4",
        "input-moment": "^0.4.0",
        "jquery": "^3.3.1",
        "jss-extend": "6.2.0",
        "keycode": "2.2.0",
        "lodash": "4.17.10",
        "material-ui-pickers": "^1.0.0-rc.14",
        "mobile-detect": "1.4.2",
        "moment": "^2.22.2",
        "path-to-regexp": "2.2.1",
        "perfect-scrollbar": "1.4.0",
        "prismjs": "1.15.0",
        "qs": "6.5.2",
        "raw-loader": "0.5.1",
        "react": "16.4.2",
        "react-autosuggest": "9.3.4",
        "react-big-calendar": "0.19.2",
        "react-chartjs-2": "2.7.4",
        "react-dnd": "2.6.0",
        "react-dnd-html5-backend": "2.6.0",
        "react-dom": "16.4.2",
        "react-number-format": "3.5.1",
        "react-popper": "1.0.2",
        "react-redux": "5.0.7",
        "react-redux-i18n": "^1.9.3",
        "react-router-config": "1.0.0-beta.4",
        "react-router-dom": "4.3.1",
        "react-scripts": "1.1.4",
        "react-select": "2.0.0",
        "react-swipeable-views": "0.12.16",
        "react-table": "6.8.6",
        "react-text-mask": "5.4.3",
        "recharts": "^1.3.1",
        "redux": "4.0.0",
        "redux-api": "^0.11.2",
        "redux-form": "^7.4.2",
        "redux-thunk": "2.3.0",
        "rimraf": "^2.6.2",
        "sass-loader": "^7.1.0",
        "typeface-roboto": "0.0.54",
        "url-search-params-polyfill": "^5.0.0",
        "velocity-react": "1.4.1",
        "webpack-shell-plugin": "^0.5.0"
    },
    "devDependencies": {
        "@babel/cli": "^7.0.0-rc.1",
        "@babel/core": "^7.0.0-rc.1",
        "@babel/plugin-proposal-class-properties": "^7.0.0-rc.1",
        "@babel/plugin-transform-react-constant-elements": "^7.0.0-rc.1",
        "@babel/plugin-transform-react-inline-elements": "^7.0.0-rc.1",
        "@babel/plugin-transform-runtime": "^7.0.0-rc.1",
        "@babel/polyfill": "^7.0.0-rc.1",
        "@babel/preset-env": "^7.0.0-rc.1",
        "@babel/preset-react": "^7.0.0-rc.1",
        "autoprefixer": "9.1.0",
        "babel-loader": "^8.0.0-beta",
        "babel-plugin-lodash": "^3.3.2",
        "babel-plugin-react-transform": "^3.0.0",
        "babel-plugin-transform-react-pure-class-to-function": "^1.0.1",
        "babel-plugin-transform-react-remove-prop-types": "^0.4.15",
        "babel-preset-react-optimize": "^1.0.1",
        "compression-webpack-plugin": "^1.1.11",
        "cross-env": "5.2.0",
        "css-loader": "^1.0.0",
        "file-loader": "^2.0.0",
        "html-webpack-plugin": "^3.2.0",
        "js-beautify": "1.7.5",
        "marked": "0.4.0",
        "postcss": "^7.0.5",
        "postcss-cli": "^6.0.0",
        "react-hot-loader": "^4.3.4",
        "style-loader": "^0.22.1",
        "tailwindcss": "^0.6.6",
        "url-loader": "^1.1.1",
        "webpack": "^4.16.5",
        "webpack-cli": "^3.1.0",
        "webpack-dev-server": "^3.1.5"
    }
}

What is the problem?
I am running Windows 10 and Yarn 1.10.1

Are you running the command within the Command prompt or within Git bash?

Command line

Try downloading the Git Shell. It's by far a better way of using the Terminal on Windows. I believe you can download it from here. https://git-scm.com/downloads

Glad you got it sorted. :)


feat: Added Actions

Thoughts on starting to return a string on each of these items as the action status tracking for the APIs that support it?  Right now the return nothing.

Thoughts on starting to return a string on each of these items as the action status tracking for the APIs that support it?  Right now the return nothing.

Let's have s sync on this, you are right

Tweaked with the above feedback and also did some prep for the action tracking action.  I haven't updated the API files yet to provide the responses though.  Thought this would be a good to upload before I do that.

I need to do some live testing of this change on my car still.  Opening for review as I would like another pass at it and for you to test on yours.


Explicit mention to issues with Maven 3.3 in the README

Neither did I until I spent a couple days freaking out why suddenly the plugin I compiled started to present class loading issues :) Thx for the quick merge!

FYI https://github.com/jenkinsci/maven-hpi-plugin/pull/24

And i wrote about it long time ago https://groups.google.com/d/topic/jenkinsci-dev/8x2nrUwHg60/discussion


Core Concept: eTag

The board feels that this is adequately covered in the guidelines.  If you feel differently, please open a new issue with the template and provide more information.


Collapse will create different features for the same level, because of non rank-aware padding

Improvement Description
Taxon path padding can be made rank aware to avoid the following situation:
Say you have taxonomic classifications such as:
k__Bacteria; p__Firmicutes; c__Clostridia; o__Clostridiales
k__Bacteria; p__Firmicutes; c__Clostridia; o__Clostridiales; f__; g__; s__

(the first is only assigned with min. confidence to o__Clostridiales, and the other one, the unassigned species of Clostridiales: o__Clostridiales; f__; g__; s__).
Current Behavior
Collapsing the above example to genus would not collapse to the same level, but result in two separate features:
k__Bacteria;p__Firmicutes;c__Clostridia;o__Clostridiales;__;__
k__Bacteria;p__Firmicutes;c__Clostridia;o__Clostridiales;f__; g__

Proposed Behavior
Make both of these collapse to the same taxon, which for the above would be:
k__Bacteria;p__Firmicutes;c__Clostridia;o__Clostridiales;f__; g__

Note: this would only be feasible if the ranks are homogeneous (which is the case for e.g. GreenGenes and other good databases).
Comments
The current padding happens in a function nested within the _collapse_table() function of _util.py. I'd say this could be made more important functions (larger scope), that would be controlled by a command line parameter, in order to let user decide whether he/she prefers to remove taxa that would need padding, for example, one may want to get rid of things not annotated to genus after collapsing ot genus (in the above example, the two entries would be deleted).
Note: I am making a PR for this - see below

this is really quite specific to greengenes... other databases usually do not have empty annotations, and might use different rank padding conventions.
Furthermore, is this really desirable? collapsing features assigned to different taxonomic ranks seems to make many assumptions. For the sake of plotting, it might be convenient to collapse the following, but in terms of interpretation these taxonomies could mean very different things:
k__Bacteria; p__Firmicutes; c__Clostridia; o__Clostridiales
k__Bacteria; p__Firmicutes; c__Clostridia; o__Clostridiales; f__; g__; s__

i.e., features assigned these different taxonomies should not be considered the same at family or genus level... it might be convenient to lump these together as "all clostridiales with unknown family and genus", but these could in fact belong to very different groups, so lumping together could smooth over important differences.

Hello @nbokulich,
In the proposed PR, the padding would only happen if there is a rank convention for the database used for classification, in particular, only for ;-separated taxonomic fields that have homogenous rank labels through out the taxonomy.
Many databases follow the convention of "aligned" rank names, e.g. the single letter for greengenes and PR2, or none for SILVA (taxonomy dump lookup). In case of no convention, note that this PR would do the current padding (adding __ down to the collapsing level).
One caveat: the rank is inferred using the string before the first _ character. If no _ character is present, the padding would remain __ (current behaviour). However, unusually taxonomy would create unusual padding,
e.g. the unlikely, poor taxonomy:
taxon_name1; taxon_name1.1; taxon_name1.1.1
taxon_name1; taxon_name1.2; taxon_name1.2.1
taxon_name2; taxon_name2.1

would pad to:
taxon_taxon_name1; taxon_taxon_name1.1; taxon_taxon_name1.1.1
taxon_taxon_name1; taxon_taxon_name1.2; taxon_taxon_name1.2.1
taxon_taxon_name2; taxon_taxon_name2.1; taxon_

The code can be made robust to such edge cases, and so, I'd say that having the possibility to fix the above issue is indeed desirable. Here's a few thoughts about your points:

As always, the user must be aware about what he/she is doing, and could well desire to group things that containing different signal (Tikhonov et al. etc...):

a user deciding to collapse at family, or genus, also decides to disregard the fact that "f_eatures assigned these different taxonomies should not be considered the same at family or genus level_", or rather would consider all unassigned the same genus and all assigned to an unknown genus the same genus too (= current QIIME2 behaviour).
collapsing the above to k__Bacteria; p__Firmicutes; c__Clostridia or even to k__Bacteria; p__Firmicutes is possible would lump things that - I agree - "belong to very different groups", but it is still possible:
you further repeat that "lumping together could smooth over important differences" so in fact, these points lead us to question the very existence of qiime taxa collapse: maybe it should be forbidden to collapse, or, never above genus level? But it is possible, because we assume the user makes well-informed choices :)

I would like to hear more about the assumption you're referring to:

I agree that an ASV assigned only to a given taxonomic rank (e.g. k__Bacteria; p__Firmicutes; c__Clostridia; o__Clostridiales) and another ASV assigned to another, deeper rank (e.g. k__Bacteria; p__Firmicutes; c__Clostridia; o__Clostridiales; f__; g__; s__) are different and my understanding (about what this assumes) is that the former could be a novel species (within the scope of current database) while the latter matches well an already known reference... which unfortunately is an empty. What other assumptions did you have in mind? (just curious)

In fact, the real issue is that QIIME2 creates features with empty taxonomic rank (__ padding) for sequences not assigned down to the collapsing rank. Let's illustrate with another example and collapse at genus level:
for these ASVs:
ASV1    k__Bacteria; p__Firmicutes; c__Clostridia; o__Clostridiales
ASV2    k__Bacteria; p__Firmicutes; c__Clostridia; o__Clostridiales
ASV3    k__Bacteria; p__Firmicutes; c__Clostridia; o__Clostridiales
ASV4    k__Bacteria; p__Firmicutes; c__Clostridia; o__Clostridiales; f__; g__; s__

will first be created:
ASV1    k__Bacteria; p__Firmicutes; c__Clostridia; o__Clostridiales; __; __
ASV2    k__Bacteria; p__Firmicutes; c__Clostridia; o__Clostridiales; __; __
ASV3    k__Bacteria; p__Firmicutes; c__Clostridia; o__Clostridiales; __; __
ASV4    k__Bacteria; p__Firmicutes; c__Clostridia; o__Clostridiales; f__; g__

and then after the collapse, are obtained the features:
k__Bacteria; p__Firmicutes; c__Clostridia; o__Clostridiales; __; __
k__Bacteria; p__Firmicutes; c__Clostridia; o__Clostridiales; g__; s__

In this case, the three different ASVs that were only assigned to o__Clostridiales - and thus that are potentially novel, i.e. unexpected in the microbial system - would be lumped together, creating the issue you highlighted.
I agree that padding should be avoid when the rank do not exist in the first place (and notably if exist elsewhere for "unassigned" assignments), but isn't it an issue that created features such as k__Bacteria; p__Firmicutes; c__Clostridia; o__Clostridiales; __; __ are confusing and should be discarded instead?

hey @FranckLejzerowicz ,
your explanation above makes it clear that the PR connected to this would cause more problems than it would solve, as this makes assumptions about taxonomic conventions that are not universal. Semicolon-delimited taxonomy is the only convention you mention that is followed by Q2.
I recommend creating a separate plugin for this action (q2-greengenes?), but curious what others think

I indeed tried to identify where this would break (and be improved to solve the issue). Note that if just one semicolon-delimited taxon do not have all its first's underscore-delimited characters the same, this PR would do nothing. Hence, the assumptions about taxonomic conventions that are not universal could be solve by adding a parameter:
  --p-conventional-ranks / --p-no-conventional-ranks
                         The taxonomy is labeled with conventional ranks, e.g. k__Bacteria
                         (and not just Bacteria).                              [default: False]

Since I agree your first point above on lumping different things, the issue remains that users would get features created by padding to the collapsing level.
i.e., collapsing
ASV1   k__Bacteria
ASV2   k__Bacteria
ASV3   k__Bacteria
ASV4  k__Bacteria
ASV5   k__Bacteria
ASV6   k__Bacteria

at genus level would yield:
ASV1   k__Bacteria;__;__;__;__
ASV2   k__Bacteria;__;__;__;__
ASV3   k__Bacteria;__;__;__;__
ASV4  k__Bacteria;__;__;__;__
ASV5   k__Bacteria;__;__;__;__
ASV6   k__Bacteria;__;__;__;__

Collapsing to:
k__Bacteria;__;__;__;__

Not sure the user readily understands the assumptions beyond this feature, vs. k__Bacteria;p__;__;__;__, k__Bacteria;p__;o__;__;__, or k__Bacteria;p__;o__;f__;__
Sorry for the long messages and for wasting you time if this is not relevant.


[1.1.0] Fix minor issues in uninstall section

This PR updates the APK uninstallation documentation for APK 1.1.0 by updating the uninstall commands and fixing minor typos.
Fixes https://github.com/wso2/apk/issues/2169

Let's add a description to the PR

Let's add a description to the PR

Fixed


Create proof of concept

This creates the first version of the block with the minimal
functionality for it to operate. The block by default listens for
openvpn and kernel logs and collects any logs that have priority
error or above.
Change-type: patch

@balena-ci I self certify!


Mapping preview is empty for gaps in mapping table

when Gaps are involved into separate and mapping previews are enabled they are empty. Question is if they are empty if it's necessary to even show the preview. If yes we should also display value which will end in gap.

duplicates #601

@mmelko could you move this capture to #601 for completion? Thanks!


⚠ Bofrost.at has degraded performance

In 42f0917, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1739 ms

Resolved: Bofrost.at performance has improved in cf8ed61 after 16 minutes.


Fix fasd completion

Add fasd as gitmodule
Regenerate fasd configuration script from current fasd version
Fix initialization script to have all fasd-related functions inside it
Add another alias v for fasd that will access recently used files
from .viminfo

Fixes: #111
Signed-off-by: Tomas Slusny<EMAIL_ADDRESS>

Not going to merge this. Upstream fasd is a huge, bloated shell script that isn't zsh specific; we only need the zsh things. I appreciate your contribution, however I would much rather see this fix happen in the current codebase.

I can then alter this patch to not use upstream fasd, but included one in repo.

@deathbeam I've merged your patch, thanks for the fix. And yes, my rational is as above. The sh fasd binary is just full of unnecessary stuffs for zsh, and especially since the repo hasn't seen much action, I thought it made more sense to strip out the unnecessary code so it would be easier to maintain.


fixing issue #120: Center tank divs on mobile devices

Description
issue #120 : center tank divs on mobile devices (all the tank divs were floating to the left on mobile devices)
Fixes # (issue)
Used flexbox for the container div of the tank divs and added a margin auto to center it horizontally.
Type of change
Please delete options that are not relevant.

[ * ] Bug fix (non-breaking change which fixes an issue)

Checklist:

[ * ] My code follows the style guidelines of this project
[ * ] I have performed a self-review of my own code
[ * ] I have commented my code, particularly in hard-to-understand areas
[ * ] I have made corresponding changes to the documentation
[ * ] My changes generate no new warnings
[ * ] I have added tests that prove my fix is effective or that my feature works
[ * ] Any dependent changes have been merged and published in downstream modules

Reviewer: Vinit Shahdeo

Please let me know if you want me to use flexbox for the whole page contents that will be a lot easier to handle

fixing issue#120


Create scripts to convert bicep examples into quickstarts (those that aren't already quickstarts)

About 20:
101/api-management-modular
101/create-managedidentity-rbac
101/cyclecloud
101/deployment-script-with-storage
101/function-app-create
101/function-http-trigger
101/storage-static-website
101/template-spec-deploy
201/anchored-proximity-placement-group
201/log-analytics-with-solutions-and-diagnostics
201/policy-azmonitor-agent-and-dcr-association
201/web-app-loganalytics-mod
201/avd-backplane-with-network-and-storage-and-monitoring
201/wvd-create-hostpool
301/automation-account-import-runbooks-and-modules
301/function-app-with-custom-domain-managed-certificate
301/log-analytics-with-datasources-solutions
301/modules-vwan-to-vnet-s2s-with-fw
301/publish-api-to-apim-opendocs
301/sql-database-with-management
301/web-app-managed-identity-sql-db

Used this work for hackathon and now reaching out to original contributors for their assistance in migrating remainder of samples.


chore: CONCURRENCY can be gt CPU cores

When mint quark, I found that with default concurrency (cpus-1):

CPU Usage is about 60-70%.
LOAD is half of the cpu cores.

Aasked some of my friends, they met same problem.
Maybe we can cancel the max limit of CONCURRENCY. Let users choose by themselves.

Thank you


Form field is in css-framework package

Q
A

Is bugfix?
❌

New feature?
✔

Breaks BC?
❌

Fixed issues

Requires https://github.com/yiisoft/yii-bootstrap5/pull/28

👍


How to change database name?

Is it possible to change the database name, and other configuration apart from the port?
I am trying to do this:
Postgresql = testing.postgresql.PostgresqlFactory(
    cache_initialized_db=True,
    on_initialized=handler,
    port=settings.DB_SETTINGS['port'],
    database=settings.DB_SETTINGS['name'],
    user=settings.DB_SETTINGS['user'],
    host=settings.DB_SETTINGS['host']
)

When in the handler method, when running
print(postgresql.dsn())

I receive the default configuration, not the parameters I passed in.

From postgresql.py:
https://github.com/tk0miya/testing.postgresql/blob/c81ded434d00ec8424de0f9e1f4063c778c6aaa8/src/testing/postgresql.py#L57-L65
Seems like dsn() only returns those default params, since you did not put any in as parameters.


{BOT} add new feature

Check setup.py and history.rst in extension pull request
After the current milestone, move the issue and pr to the next milestone

Thanks for the contribution @wangzelin007!

Hi @wangzelin007, Since the current milestone time is less than 7 days, this pr will be reviewed in the next milestone

Thanks for the contribution @wangzelin007!

Hi @wangzelin007, Since the current milestone time is less than 7 days, this pr will be reviewed in the next milestone

Thanks for the contribution @wangzelin007!

Hi @wangzelin007, Since the current milestone time is less than 7 days, this pr will be reviewed in the next milestone

Thanks for the contribution @wangzelin007!

Hi @wangzelin007, Since the current milestone time is less than 7 days, this pr will be reviewed in the next milestone

Hi @wangzelin007,
If you want to release the new extension version, please write the description of changes into HISTORY.rst and update setup.py.


Add test to check the whole pipeline

Add a test to check the whole pipeline and show how to train a classification task for EN (I'm working on it)

Hey @PiotrCzapla I came across some minor bugs while working with trying to train English and Hindi using pretrain_lm - I have pushed the fixes to master just this once.
I will make future changes in a separate branch.

btw @NirantK  it seems that you managed to run the whole thing end to end, do you want to take over this issue and create a small test (small artificial dataset) where the whole process is executed. Ie LM trained, classification trained, & tested? So that we can use it to see if our scripts are still working after updates to fastai.

As of now, I am figuring out the small fixes needed in classifier to make it in English and at least one other language. I'll do this as we build the XNLI and Wikipedia classifier or immediately after.

@NirantK  then commit please to master the fixes as I"m going to need that soon to double check the performance of BILM

test added:
https://github.com/n-waves/ulmfit-multilingual/commit/23b17da61eff4f8bc0d2ddfa9aff66a5112e0a2b


Provide MSBuild 2003 project versions for MonoDevelop/xbuild compatibility

Steps to reproduce

Install and start MonoDevelop
In menu: File → Open
In the dialog box: find the solution and open it

Alternatively, use xbuild command (if available) in the command line with a project pathname as its argument.
Expected behavior
All projects should be opened without errors and builds should be possible.
Actual behavior

MonoDevelop 6.3 — error: Unknown solution item type.
MonoDevelop 5.10 — error: Project file doesn't have a valid ProjectGuid.
xbuild 14.0 — error: The default XML namespace of the project must be the MSBuild XML namespace. If the project is authored in the MSBuild 2003 format, please add xmlns="http://schemas.microsoft.com/developer/msbuild/2003" to the  element. If the project has been authored in the old 1.0 or 1.2 format, please convert it to MSBuild 2003 format.

Version of SharpZipLib
See commit below.
Obtained from (place an x between the brackets for all that apply)

[x] Compiled from source
branch: master
commit: be53259
[ ] Downloaded DLL from GitHub
[ ] Downloaded DLL from SourceForge
[ ] Downloaded DLL from _______
[ ] DLL included as part of
Package installed using:
[ ] NuGet
[ ] MyGet
[ ] Chocolatey

As an alternative, Cake support may be added instead, so that it would be possible to build the library with Mono on non-Windows OS.

Since mono 5 xbuild is deprecated. (http://www.mono-project.com/docs/about-mono/releases/5.0.0/#msbuild)

We are targetting netstandard1.3 now - I'd need to check on my Mac whether we are now compat with Visual Studio for Mac 7.1.

given that xbuild doesn't seem to be relevant any more, and the new format project files being useable with Visual Studio / Visual Studio for Mac / VS Code, and the Core SDK from the command line cross platform, is there any need for the old project versions? (would that even work for the .NET Standard build anyway?)

I think the modern SDK-style csprojs should be fine in 2020.

Agreed. I use SharpZipLib as much on mac and linux as I do on windows.


Document what's changed in dart:html/webgl/etc with 1.16

There are a lot of changes due to changed IDL. We should produce at least some sort of listing for people. @kevmoo @jacob314 @terrylucas

1.16 is about to ship, please get this quickly!

Closed as dupe of https://github.com/dart-lang/sdk/issues/25970


fix: Change instance names of ViewModel for Signup & Login fragments

Fixes #752
Changes:
Changed names of the instances of the ViewModel classes in SignupFragment and LoginFragment in the auth package along with all the usages of the variable i.e. refactored the name of the variables.
Screenshots for the change:

Ideally, view model names should not be tied to activity or fragments, but views

I have created an issue + pull request for the same now for all such ViewModel classes and instances


fixed crash when help called for subcommand on nonexisting subcommand

Fix for #17

I merged without noticing that there was illegal syntax in your code, so I quickly pushed a fix^^
Thank you anyways for your contribution!

I merged without noticing that there was illegal syntax in your code, so I quickly pushed a fix^^
Thank you anyways for your contribution!


Update libgit2 to include segfault fix for git_reflog_read

Applies the changes from https://github.com/nodegit/libgit2/pull/2 to NodeGit.

Shouldn't this fix be upstreamed to libgit2, kind of sketched about us maintaining a separate libgit2 that differs from published versions when we update.

@tbranyen There is an outstanding PR that includes this fix (https://github.com/libgit2/libgit2/pull/3463) - this is the most critical part of that PR as it fixes a segfault. It is also probably the safest part - it only changes a malloc to a calloc.
The idea behind this PR was to get the segfault fixed in NodeGit until the upstream PR gets merged and released.

Definitely not interested in maintaining a forked version of libgit2. I'd say make another PR to libgit2/libgit2#maint/v0.23 so we can expedite getting a released version of this in.
However, until that lands I'm ok with using a fork as a temporary work around.

@tbranyen you cool with me merging this?

What is really fixing the problem? Changing malloc to calloc? That's fishy a bit.

@saper There is an explanation in https://github.com/libgit2/libgit2/issues/3458#issuecomment-146680050... it breaks down like this:
When we do entry->committer = git__malloc(sizeof(git_signature));, using malloc will not initialize the contents pointed to by entry->committer in any way, so the values of entry->committer->name, entry->committer->email etc. are indeterminate (that is, the values will be whatever happened to be in the memory block when it was allocated).  If git_oid_fromstrn returns an error, we then immediately go to git_reflog_entry__free(entry), which in turn does git__free(entry->committer->name) and git__free(entry->committer->email).  Because entry->committer->name and entry->committer->email are indeterminate, the free likely segfaults (in the included test, for me it would always segfault - in general, it will either do nothing if the value happens to be 0, free something it shouldn't free if the value happens to be a valid allocated pointer, or segfault / do some other kind of undefined behavior).
When using calloc instead of malloc, entry->committer = git__calloc(1, sizeof(git_signature)); will initialize the contents pointed to by entry->committer to all 0s. In the same scenario as above, when we get to git__free(entry->committer->name) and git__free(entry->committer->email), the free will do nothing because entry->committer->name and entry->committer->email are 0.  That happens to be the correct behavior in this case.

I'm going to go ahead and merge this. We'll change back to using libgi2 proper after https://github.com/libgit2/libgit2/issues/3458 is merged and released.


No Output - utils.plot_meterics

I am using
utils.plot_metrics(
    logdir=logdir,
    # specify which metrics we want to plot
    metrics=["loss", "dice", 'lr', '_base/lr']
)

in google colab and the results are not plotted.
Can you suggest what is the issue.
Thanks

Hi! There is a Colab problem with displaying Plotly.
Run the Colab extras command from our classification tutorial
import IPython

def configure_plotly_browser_state():
    display(IPython.core.display.HTML('''
        <script src="/static/components/requirejs/require.js"></script>
        <script>
          requirejs.config({
            paths: {
              base: '/static/base',
              plotly: 'https://cdn.plot.ly/plotly-latest.min.js?noext',
            },
          });
        </script>
        '''))

IPython.get_ipython().events.register('pre_run_cell', configure_plotly_browser_state)


Program unload API

Add API to unload programs

@dave-tucker are you happy with this now? If so imma squash and merge

Yep LGTM


Add Hive QL and q to list of languages

Description

Checklist:

Q

[X] I am adding a new language.

[X] The extension of the new language is used in hundreds of repositories on GitHub.com.

Search results for each extension:
https://github.com/search?utf8=✓&q=extension%3Aq+each+NOT+nothack&type=Code
https://github.com/search?utf8=✓&q=extension%3Aq+flip+NOT+nothack&type=Code
https://github.com/search?utf8=✓&q=extension%3Aq+reverse+NOT+nothack&type=Code

[X] I have included a real-world usage sample for all extensions added in this PR:

Sample source(s):

ml.q https://github.com/jlas/ml.q/blob/master/ml.q
tq.q https://github.com/KxSystems/kdb-taq/blob/master/tq.q

Sample license(s):

ml.q MIT
tq.q Apache-2.0

[X] I have included a syntax highlighting grammar.
[X] I have included a change to the heuristics to distinguish my language from others using the same extension.

Hive QL

[X] I am adding a new language.

[X] The extension of the new language is used in hundreds of repositories on GitHub.com.

Search results for each extension:
https://github.com/search?utf8=✓&q=extension%3Aq+select+NOT+nothack&type=Code

[X] I have included a real-world usage sample for all extensions added in this PR:

Sample source(s):

mi.q https://github.com/apache/hive/blob/master/ql/src/test/queries/clientpositive/mi.q

Sample license(s):

mi.q  Apache-2.0

[X] I have included a syntax highlighting grammar.
[x] I have included a change to the heuristics to distinguish my language from others using the same extension.

This is a duplicate of an already open and blocked PR https://github.com/github/linguist/pull/3820 and doesn’t provide a solution to the blocker either.
Please review that PR and if you can find a way to address the issues blocking that PR, please feel free to update your PR to do so and we’ll look into it.

@lildude It seems like the other PR is blocked since Hive Ql also uses .q extension. Is the solution to add Hive QL also to the list of languages and write a heuristic to resolve the two files language?
Also regarding the heuristic regex, I know that any line that starts with / is a comment. And almost 90 percent of files set the namespace in q file using \d in their files. However, the comment in Hive QL starts with --. Would this be a good enough identifier?

Is the solution to add Hive QL also to the list of languages and write a heuristic to resolve the two files language?

You'll need to add associations for as many languages as possible that use a .q extension (which looks like it may include SQL too as pointed out by @pchaigno at https://github.com/github/linguist/pull/3820#issuecomment-338375928) and a heuristic to differentiate between then all.
As for the heuristic itself, I'm not familiar with Q or Hive QI so I can't really comment on an appropriate regex to use other than to say, the more cases it matches, the better. If that means using a pretty complex regex, then so be it. We've got quite a few complex regexes already.  The only thing to be mindful of is the performance of the regex.

@lildude The only other language that I could find that uses .q is a Hive QL script which is a SQL-like syntax. I added the language Hive to the list of languages. I also added a heuristic to recognize the q language by one of the following

Function definition and assignment. ....:{  (Note that in Kdb Q, functions don't have to specify the parameters, so the [...] is not necessary)
If the file or a line starts with /. In Kdbq, this is single line comment. in Hive QL a comment always starts with --.
If the file has \d ..

I had two questions,

Is it ok that my heuristic doesn't set the language to Hive ever? My logic was that if anyone would like to add syntax highlighting for Hive QL or language recognition, they could in future.
I am tempted to also remove the language-hql syntax highlighter. Is it ok to only add the language Hive to list of languages without a syntax highlighter?

Finally, how do I verify the correctness of the heuristic and performance of it?

@lildude Could you re-review?

@lildude Could you re-review?

I was kinda hoping you'd resolve at least these two failing tests first as we won't merge
without passing tests 😄:
  2) Failure:
TestPedantic#test_heuristics_are_sorted [/home/travis/build/github/linguist/test/test_pedantic.rb:38]:
    disambiguate ".x" do |data|
 should come after     disambiguate ".q" do |data|
  3) Failure:
TestColorProximity#test_color_proximity [/home/travis/build/github/linguist/test/test_color_proximity.rb:21]:
The following 1 languages have failing color thresholds. Please modify the hex color.
- Parrot (#f3ca0a) is too close to ["ffdd00", "f3ca0a"]

The license failure will be resolved once https://github.com/github/linguist/pull/3982 is merged, which I hope to do :soon:.
I also won't give my final review until one of the non-GitHub maintainers has reviewed the PR.
To answer your questions:

Is it ok that my heuristic doesn't set the language to Hive ever? My logic was that if anyone would like to add syntax highlighting for Hive QL or language recognition, they could in future.

Yeah. We've got this in several other places too so I don't think there will be too many problems, but I could be missing something which I'm sure one of the other maintainers will point out if I am 😉. As for someone else adding the grammar etc... you've done it already.

I am tempted to also remove the language-hql syntax highlighter. Is it ok to only add the language Hive to list of languages without a syntax highlighter?

It is, but you may as well keep the grammar and help out the Hive community whilst you're here.

@lildude Thanks you. All test failures except the following are addressed.
Warnings:
vendor/licenses/grammar/language-hql.txt:
  - license needs reviewed: other.
303 dependencies checked, 1 warnings found.

@lildude Just a friendly reminder that this is waiting for review

From a quick look, this looks good, but I'll reserve my final review until I've had a chance to take another closer look, especially in light of the comments in https://github.com/github/linguist/pull/3820. I also won't give my final :+1: until one of the other core maintainers has given their review.

I don't see these last two rules being used in the heuristic rule you implemented for q.

Sorry, I forgot to update the comments, I basically thought that relying on the comment would not be very useful hence I removed the second rule.
I also updated the third rule to not only account to \d but also for other most frequently used system functions such as \t, \ts and \l.  (BTW, Full list of kdbq system commands can be found here).

What makes you think that file is under Apache 2.0 license?

You are right. I miscopied the tq.q. I replaced it with the correct tq.q with the Apache-2.0 license

Going through the search results that do not seem to be Hive QL or q, I found a large number of .uosl.q files. It looks like scripts for an MMORPG, Ultima Online. I used Harvester to download 2999 of those files and found that they are distributed across 1123 repositories and 1079 users.
I think it warrants support in Linguist with the other .q languages. The good news is that, since these files are using a nested file extension, we don't need a new heuristic rule to identify them; adding the language to languages.yml with .uosl.q as a file extension should be enough.

Actually, there aren't near far enough .uosl.q files to warrant support. Only 2 users with 2 repositories. Looks like searching for in:filename uosl.q doesn't return only files with uosl.q in the filename and I didn't bother to verify. Sorry about this @yznima.

@lildude Is there anything left to be done for this?

Is this correct usage? And do I need to merge to master?
$ cat .gitattributes
*.q linguist-language=q

@zak-b2c2 This will only work after the next version of Linguist is released. Your .gitattributes appears to be correct :+1:

Cool. When can we expect the next release? :)

Cool. When can we expect the next release? :)

There's no set date, but I'm trying to get into a habit of making a release at least once a month, so I anticipate probably somewhere around the middle of the month.

@lildude I was wondering if you are still planning to do a release this month?

Looking good, thanks guys !

@zak-b2c2 snuck in and stole my thunder 😜 - this is now live on GitHub.com 🎉


[SYSTEMML-2446] Fix the cleanup of list object

Hi @mboehm7 ,
Here is the PR for fixing the cleanup of list object. The idea is to not remove the matrix block which is referenced in the previous matrix object. Because when doing the partial model update, there will be some parameters unchanged (i.e., the matrix block remains the same).
Thanks for the review,
Guobao

LGTM - thanks @EdgarLGB. After running a number of experiments with different configurations, I can confirm that we don't seem to miss any cleanups. I tracked the number of managed objects in the bufferpool, and this number stabilizes even for 10 epochs at ~450 matrices when running ASP with BATCH and EPOCH and the different optimizers. So this looks good to me.


postsrsd: 1.9 -> 1.10

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/postsrsd/versions.
meta.description for postsrsd is: "Postfix Sender Rewriting Scheme daemon"
meta.homepage for postsrsd is: "https://github.com/roehling/postsrsd"
meta.changelog for postsrsd is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 1.10 with grep in /nix/store/3ps90fzy1qn7d7x8895jyh1fdscgj91z-postsrsd-1.10

directory tree listing: https://gist.github.com/cd814ca9fb84c5189fe350a0b771b318

du listing: https://gist.github.com/9d21fd2864158965363282501ba8518d

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
0 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
postsrsd

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/3ps90fzy1qn7d7x8895jyh1fdscgj91z-postsrsd-1.10 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A postsrsd https://github.com/r-ryantm/nixpkgs/archive/0c44187d2ce5513550f2871ad69d2ee11e1d8346.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/3ps90fzy1qn7d7x8895jyh1fdscgj91z-postsrsd-1.10
ls -la /nix/store/3ps90fzy1qn7d7x8895jyh1fdscgj91z-postsrsd-1.10/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

postsrsd

Maintainer pings
cc @abbradar for testing.

Flagged as security, this update fixes CVE-2020-35573.

Flagged as security, this update fixes CVE-2020-35573.

fixes https://github.com/NixOS/nixpkgs/issues/109217

fixes https://github.com/NixOS/nixpkgs/issues/109217


Remove apps from brew cask if AppStore copy is present

Many apps downloaded via brew cask are trial versions that need to be registered. If an AppStore copy is obtained, the brew cask version should be removed since it will be licensed.

Pull request #15 fixes this.


Add descriptions, add examples, add missing endpoints

Main changes include:

add descriptions to endpoints
add examples to most endpoints
Changes to wallet registration : update Sep24 usage and remove old endpoints
Add /assets endpoints
Fixes related to auth

Preview is available here:http://developers-pr202.previews.kube001.services.stellar-ops.com

Preview is available here:http://developers-pr202.previews.kube001.services.stellar-ops.com


Cannot use 'with' in a Eloquent Query Builder Macro

Laravel Version: 5.8
PHP Version: 7.2.19
Database Driver & Version: MySQL 8.0

Description:
It seems $this received in a Builder::macro cannot execute with or whereHas.
Steps To Reproduce:
I was attempting to use with on a Eloquent macro, but I get this error message:
    Call to undefined method Illuminate\Database\Query\Builder::with()

The following works:
    $builder = Model::query();
    $builder->with('relationship')->toSql();

But if I do this inside a macro:
    use Illuminate\Database\Query\Builder;

    Builder::macro('foobar', function() {
        $this->with('relationship');
    });

    $builder = Model::query();
    $builder->foobar()->toSql();

Then I get the above error message.
Is this a bug or something that I do wrong?

Gosh I was using the wrong import:
use Illuminate\Database\Eloquent\Builder;


update mlp of llama

Description
use xe_addons.mlp_silu_mul_inplaced(gate, up) for llama series model
1. Why the change?

2. User API changes

3. Summary of the change

4. How to test?

[ ] Unit test: Please manually trigger the PR Validation here by inputting the PR number (e.g., 1234). And paste your action link here once it has been successfully finished.

Well, PR validation is failed: https://github.com/intel-analytics/ipex-llm-workflow/actions/runs/10505369582/job/29104277776. Need a further look.

xe_addons.mlp_silu_mul_inplaced(gate, up) may cause output logits changes a little.
e.g. the max_diff_tensor for Mistral-7B-Instruct is [0.0009765625].
Here I expand the lower_bound threshold to 1e-3 first to make our UT can pass.

PR validation is passed: https://github.com/intel-analytics/ipex-llm-workflow/actions/runs/10506853978


[QUESTION] How to install Harvester without VIP?

Hello,
I want to install a Harvester cluster on servers in a data center that does not offer VIPs and also not offer floating IPs. I have only static IPs for the nodes and additional static IPs for the nodes.  The IPs are from different subnets. I can setup a server that can loadbalance an IP to different nodes, but this is not a VIP/Floating IP.
The installer requires to give a VIP. How can I disable VIP usage?

You need to use oneof your 'extra' static ips. If that does not work, then your server subscription is not suitable for a harvester installation.

Thanks. When I use an extra static IP it is only valid for one node. Can I tell harvester that the VIP should be sticky on a node? And how do I bypass the same subnet check? The extra static IP is not the same subnet as the management address.

Hi @thomas-ej-worm, thanks for trying out Harvester. But I'm afraid that this is infeasible in our current implementation. Harvester relies on kube-vip to maintain the VIP address, and it works in the ARP mode: https://kube-vip.io/docs/modes/arp/. It will literally bind the VIP address to the mgmt-br interface on one of the cluster nodes. Even if you bypass the same-subnet-check in the installer, I doubt it will work if the VIP address is not in the same subnet as the node IP addresses.


Does aiohttp support awss3?

I am testing to put an object to amazon s3. But get a 403 status code with reason
'The request signature we calculated does not match the signature you provided.
Check your key and signing method.'
The code is:
response = yield from aiohttp.put(url, data=open(pathfn, 'rb'), headers=headers)
logger.debug('response status is ' + str(response.status))
raw = yield from response.text()
logger.debug('response body is ' + str(raw))

The url is:
http://s3.amazonaws.com:80/bucket_name/key_name
The raw headers input is:
{
"User-Agent": "Boto/2.38.0 Python/3.4.0 Linux/3.13.0-46-generic",
"Content-Length": "1024",
"Date": "Thu, 27 Aug 2015 02:49:17 GMT",
"Authorization": "AWS [access_key]:[signature]"
}
But when test with requests. It returns 200 status code.
The code is:
response = requests.put(url, data=open(pathfn, 'rb'), headers=headers)
logger.debug('response status is ' + str(response.status_code))
logger.debug('response body is ' + str(response.text))

So does aiohttp inside will affect the header 'Authorization'?

aiohttp support awss3, however there is a little problem with Content-Type header.
See #457 for details and possible solution.

@popravich: Thanks.
As awss3's signature is related with Content-Type header.
I explicitly set the Content-Type header and calculate awss3's signature with it.
Then t returns 200 status code from awss3.


Update scalafmt-core to 3.1.0 in series/1.x

Updates org.scalameta:scalafmt-core from 2.7.5 to 3.1.0.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalameta", artifactId = "scalafmt-core" } ]

labels: library-update, early-semver-major, semver-spec-major

Superseded by #524.


⚠ Bofrost.at has degraded performance

In 6f07072, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2048 ms

Resolved: Bofrost.at performance has improved in 823e402 after 11 hours, 55 minutes.


How to use client certificate? I can't find proper interfaces to load client certificate and key.

Now, I have extended the tls_load_certificates function, added the client certificate parsing code, and can use the client certificate.
if (dummy == SSL_SERVER_RSA_CERT) {
context->certificates = (struct TLSCertificate **)TLS_REALLOC(context->certificates, (context->certificates_count + 1) * sizeof(struct TLSCertificate *));
context->certificates[context->certificates_count] = cert;
context->certificates_count++;
DEBUG_PRINT("Loaded certificate: %i\n", (int)context->certificates_count);
} else {
context->client_certificates = (struct TLSCertificate **)TLS_REALLOC(context->client_certificates, (context->client_certificates_count + 1) * sizeof(struct TLSCertificate *));
context->client_certificates[context->client_certificates_count] = cert;
context->client_certificates_count++;
DEBUG_PRINT("Loaded certificate: %i\n", (int)context->client_certificates_count);
}


[Suggestion]: From Facebook - Operations File Node, Level in Tree

Summary Description
The suggestion offered was to add into the operations a grouping for the particular file node this came out of or the grouping. So if we had multiple files and these got placed into the same operation we'd get another node within our references for the grouping within the references.
Current functionality
Currently we display the reference nodes ("opnode") in a flat list of all of the elements.
Future functionality
We would display the reference node but parallel the structure of the elements. So if they had any groupings we would also give those groupings.
MeerK40t Version
0.7.5001
MeerK40t Type
Source (zip/tar file from Github)
Your Operating System
Windows

This would likely complicate the existing code, and would be quite a big internal change which would need reasonable testing.
There are much higher priority items IMO.

I think this just makes the interface messier. I don't see


Teams notifications do not respect macOS ‘do not disturb’

Teams has decided on a custom notification format on macOS: that‘s fine, but it needs to respect macOS’s ’do not disturb’ setting in the notifications centre. At the moment, flipping that switch will mute all native notifications, but Teams notifications still keep on popping up over the top of presentations, etc.
Is this functionality planned?

@robinwhittleton thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.

Found that this feature request was on UserVoice and is currently in development (and has been in development for 24 months): https://microsoftteams.uservoice.com/forums/555103-public/suggestions/16941685-use-built-in-notifications-for-macos


Python Rebuff SDK

This PR adds a first class support for Python SDK for Rebuff.
Please note you will need the following credentials for testing the PR:
pinecone_environment
pinecone_apikey
pinecone_index
openai_model
openai_apikey
I will open new issues for creating support for Chroma and for adding tests for Python SDK.

Hi Mehrin, I don't think we've interacted yet, but I'm Risto and I've been contributing a little to this project. I think that's great that you're adding a Python SDK! I thought I'd go through and add some comments for where I think the code could be slightly improved. Let me know if you have any questions or clarifications on my review.

Hi Risto, Thank you so much for your review and valuable contributions to the project. I have tried to work on your review. The main changes include:

Updated the API call to call Pinecone directly in log_leakage function.
Updated type hints.
Changed class name from Rebuff to RebuffPython to avoid possible conflict.

This PR looks good from my perspective. Thanks for all your work on it, Mehrin!


chore: update ali npm registry

Codecov Report
Patch and project coverage have no change.

Comparison is base (4b85223) 77.75% compared to head (c2f30fb) 77.75%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1072   +/-   ##
=======================================
  Coverage   77.75%   77.75%
=======================================
  Files          32       32
  Lines         980      980
  Branches      198      187   -11
=======================================
  Hits          762      762
- Misses        215      218    +3
+ Partials        3        0    -3

see 3 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Ensure EventLoop is not interrupted after or during destruction

Resolves #57.

Adding a little more context here for posterity: the hypothesis on the bug we saw in #57 is that the thread that gets created and detached in EventLoopTest.InterruptTimer did an EventLoop::Submit() and was context switched out before it did the uv_async_send() meanwhile the main thread calls EventLoop::Run() and executes the lambda passed to EventLoop::Submit() and since the thread was detached the main thread doesn't wait for it so when it gets context switched back in it tries to do the uv_async_send() but the EventLoop is already getting destructed. This PR fixes such a bug even if #57 is not an exact incarnation of this hypothesis.


Fix warnings generated by purescript-0.8.0

Closing this as it's conflicted and a bit out of date now. With thanks as ever though!


Implement Cache-Status HTTP Response Header Field (RFC9211)

It would be nice to implement the Cache-Status HTTP Response Header Field.
This headers would probably have to be optional.

Implemented in the pro version.


Added ZeroFill feature to Migrations

Q
A

Is bugfix?
❌

New feature?
✔

Breaks BC?
❌

Fixed issues

Added ->zerofill() feature like ->unsigned().

@simialbi it's not standard sql attribute!

Can't add it unless there's analogues for every DB we support.


Code error, but code works

STR:
use state in a component, with React.useState (works, but editor complains) or just plain useState (fails)
See here:

This doesn't appear to be an issue anymore


Feature/changes only

Changes

Adds new feature to deploy only what has changed
Remove ability to deploy to a library

Checklist

[x] have tested my change
[ ] updated relevant documentation
[x] Remove any/all console.logs I added
[x] eslint is not complaining
[ ] have added myself to the contributors' list in CONTRIBUTING.md
[ ] for feature PRs: PR only includes one feature enhancement.

Needs testing on Windows.


jte 3.0

We're planning to release the next major jte version. So far, we have the following ideas:

#229
#224

This would replace the generateNativeImageResources feature with an extension.

#208
#227

This also removes getWriter() from TemplateOutput interface
And, we're back to zero dependencies 👍

What do you think? Do you have other ideas we should incorporate in this release?

I'm not a user of Jte but I have played with lots of templating engines and one of the things I think Rocker got right is the ability to refer to a template as a type. JStachio inherently relies on this as well.
For example in JStachio and Rocker there is no:
JStachio.render("someTemplateName");
This is similar to the points brought up in #131 and arguably it belongs there.
So I'm wondering if you provide an interface like:
    interface TemplateReference<PARAM> {
      public String templateName(); // name() is not chosen because we might use enum
    }

    <P> void TemplateEngine.render(TemplateReference<P> template, P param, Output o)

Then you somehow generate those TemplateReferences.
That being said generating code and then having someone reference the symbol of that code is painful which is exactly the case with Rocker. ie red squiggle because the IDE doesn't see the class. You can fix this with proper classpath setup but its annoying.

@agentgt the ability to do that was implemented in #224.

jte 3 is finally out, closing this now!


Error when try to get video signatures

Hi  josegonzalez,
Firstly, very thanks for you sharing this very handy tools.
I checked out version b63b243824, then

installed it
coped example code in " Usage Example" into youtube.py file
run it.
got following error.

"
Traceback (most recent call last):
File "youtube.py", line 10, in
yt.url = "https://www.youtube.com/watch?v=U0CGsw6h60k"
File "/usr/local/lib/python2.7/site-packages/pytube/api.py", line 76, in url
self._get_video_info()
File "/usr/local/lib/python2.7/site-packages/pytube/api.py", line 241, in _get_video_info
signature = video_signatures[idx]
IndexError: list index out of range
"
Could you help that.
email<EMAIL_ADDRESS>Regards,
Leo

Seems to be fixed as of v4.0.0


Remove template parameter from Tensor (#13)

Summary:
Pull Request resolved: https://github.com/facebookresearch/weakly-supervised-action-detection/pull/13
Pull Request resolved: https://github.com/pytorch/translate/pull/166
Pull Request resolved: https://github.com/pytorch/pytorch/pull/9125
Closes https://github.com/pytorch/pytorch/pull/9125
Use inheritance for polymorphism, and remove template parameter
This is to change the templating in call sites, the core implementations will change later
Before Caffe2 Tensor class was compile-time fixed to bind to a particular device/context. With this change, we're making it a runtime property (stored inside the tensor), but preserve the same semantics. For example, one has to specify device type in order to create a Tensor - there are no uninitialized tensors. More specifically the changes are:

We added an extra argument DeviceType to most of the constructors of the tensor, e.g. (Tensor(DeviceType type)),
Semantics of constructor Tensor(const Tensor& src, ContextForCopy* context); is changed, in this constructor, the second context is passed in to enable us to call the templated Copy function, it could be in a different context as source and target previously, now we'll enforce that the context should have same device type as src, if it is provided.
To preserve 'get-or-construct' semantics of Blob, we added specialized getter Blob::GetMutableTensor that verifies both that Blob contains a Tensor and that it's of a correct type
Specifically, Tensor type is not default-constructible any more (as we don't have unknown device tensors) and thus some of the code handling STL containers needs to change

Note: Some changes are postponed just to keep this diff a bit smaller. Please see TODOs.
Reviewed By: ezyang, houseroad
Differential Revision: D9024330

It's landed.


Node.Js Update

Current Situation
When I try to update node.js as explained, it asks for a password but it won't let me write anything and none of my passwords work.
Please help-me
Logs
homebridge@orangepizero:/var/lib/homebridge$ sudo hb-service update-node

We trust you have received the usual lecture from the local System
Administrator. It usually boils down to these three things:

    #1) Respect the privacy of others.
    #2) Think before you type.
    #3) With great power comes great responsibility.

[sudo] password for homebridge:
Sorry, try again.
[sudo] password for homebridge:
Sorry, try again.
[sudo] password for homebridge:
sudo: 3 incorrect password attempts

Config
No response
Homebridge UI Version
v4.50.6-test.4
Homebridge Version
v1.6.1
Node.js Version
14.16.1
Operating System
Ubuntu / Debian (or a variant)
Environment Info

[ ] Using Docker?
[ ] Using Hyper-V?
[X] Using hb-service?

Raspberry Pi Model
None

Duplicate https://github.com/homebridge/homebridge-config-ui-x/issues/1538

Please don't create multiple duplicate issues across multiple repositories.  It gets painful to cleanup afterwards.
When you tried this, didn't you receive this message ?
ERROR: sudo / root access is not allowed by the homebridge service account. You can use "su <username>" to swap to your local user account.
Please su to your local admin account and try again

No I received this:
homebridge@orangepizero:/var/lib/homebridge$ sudo hb-service update-node
We trust you have received the usual lecture from the local System
Administrator. It usually boils down to these three things:
#1) Respect the privacy of others.
#2) Think before you type.
#3) With great power comes great responsibility.

[sudo] password for homebridge:
Sorry, try again.
[sudo] password for homebridge:
Sorry, try again.
[sudo] password for homebridge:
sudo: 3 incorrect password attempts

You need to su to your local admin account and try again

Solved, thank you


Added Mingw Instructions

Added Instructions to help build with Mingw64. This would help people build the library under mingw64 and msys2 toolchains.
Checklist
Not all of these might apply to your change but the more you are able to check
the easier it will be to get your contribution merged.

[x] CI passes
[x] Description of proposed change
[x] Documentation (README, code doc blocks, etc) is updated

thanks for taking the time to contribute!


Added Github Icon

Description
Add Github icon in Contributor page's Card.
Fixes:  #572

Type of change

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

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings

ATTACH SCREEN-SHOTS / DEPLOYMENT LINK

@aditya-bhaumik Please check it and merge !
Thank You !


Trouble creating and managing venv

Hi!
First of all sorry i've missed something. I went through the docs and i think i'm doing the right think but can figure out what's wrong.
So I'm trying to move out my project from conda to uv. It worked well when moving all my dockerfiles and scripts but when i try to create my local venv for quick development it can't pick up the venv i create.
I first create it with:
uv venv test_env

I get
Using CPython 3.12.7
Creating virtual environment at: test_env
Activate with: source test_env/bin/activate.fish

I activate it with
source test_env/bin/activate.fish

Which runs fine, but when i try to install some requirements with
uv pip install --system -r requirements.txt

I get
Using Python 3.12.6 environment at<EMAIL_ADDRESS>error: The interpreter at<EMAIL_ADDRESS>is externally managed, and indicates the following:

  To install Python packages system-wide, try brew install
  xyz, where xyz is the package you are trying to
  install.

  If you wish to install a Python library that isn't in Homebrew,
  use a virtual environment:

  python3 -m venv path/to/venv
  source path/to/venv/bin/activate
  python3 -m pip install xyz

  If you wish to install a Python application that isn't in Homebrew,
  it may be easiest to use 'pipx install xyz', which will manage a
  virtual environment for you. You can install pipx with

  brew install pipx

  You may restore the old behavior of pip by passing
  the '--break-system-packages' flag to pip, or by adding
  'break-system-packages = true' to your pip.conf file. The latter
  will permanently disable this error.

  If you disable this error, we STRONGLY recommend that you additionally
  pass the '--user' flag to pip, or set 'user = true' in your pip.conf
  file. Failure to do this can result in a broken Homebrew installation.

  Read more about this behavior here: <https://peps.python.org/pep-0668/>

Consider creating a virtual environment with `uv venv`.

Which is complaining about using an external 3.12 and not the recently created venv.
Any help appreciated ty

To add.
I created a project within this repository a few days ago, when i look at the .venv/pyvenv.cfg file looks like:
home = /Users/jiwidi/.local/share/uv/python/cpython-3.12.7-macos-aarch64-none/bin
implementation = CPython
uv = 0.5.6
version_info = 3.12.7
include-system-site-packages = false
prompt = ml-cronjobs

So its not pointing to the test_env/bin/activate.fish environment created before. I see a python bin inside the .venv path too, should I be using that one then? Is this limited then for a project to have 1 venv? How do i manage them or change?

Why do you need the --system flag? According to the documentation, this flag is supposed to ignore virtual environments.
--system
Install packages into the system Python environment.

By default, uv installs into the virtual environment in the current working directory or any parent directory. The --system option instructs uv to instead use the first Python found in the system PATH.

WARNING: --system is intended for use in continuous integration (CI) environments and should be used with caution, as it can modify the system Python installation.

May also be set with the UV_SYSTEM_PYTHON environment variable.

home in pyvenv.cfg is not the place where things are installed to, it's used to find the Python interpreter libraries. uv will error if you attempt to install into one of the environments at /home/myuser/.local/share/uv/python.
@FishAlchemist is correct, do not include the --system flag.

sorry for the late reply, the system did indeed work. Was using it from another command in our dockerfiles so mb there, ty!


Fix test in 02_transforms

It looks like recently, this test was broken in 02_transforms:
def f(x) -> Tuple[float,float]: return x
test_eq(anno_ret(f), Tuple[float,float])

This will fix that by changing it to:
def f(x) -> Tuple[float,float]: return x
test_eq(anno_ret(f), [float,float])

Which is what I believe it should be based on anno_ret conditional with Tuples.

Ah - I think this might be different between py36 and py37. Lemme take a look.


option to limit cloning of a remote repository to commits within a certain date range

Currently, directly after creating a project via the REST API, the remote repository will be cloned and scanned for commit and file metadata (see http://thombergs.github.io/coderadar/1.0.0-SNAPSHOT/docs/restapi.html#_creating_a_project).
For a very large repository, I might want to analyze a certain range of commits only. So, when creating a project, it should be possible to define a date range. Only the commits within this range should be cloned to the local repository to be analyzed later.
This is also important for creating a repeatable dataset for performance tests.

Git does not support cloning only a range of commits. So, all commits must be cloned, but the CommitMetadataScanner and FileMetadataScanner should only run on commits in the date range.


mongodb client resolves to wrong package name

Affected Puppet, Ruby, OS and module versions/distributions

Puppet: 4.10.0
Distribution:centos 7
Module version: 3.1.0

How to reproduce (e.g Puppet code you use)
class {'mongodb::globals':
manage_package_repo => true,
version             => '4.0',
}
-> class {'mongodb::client': }
-> class {'mongodb::server':
port    => 27018,
verbose => true,
}
What are you seeing
Puppet agent tries to install using /bin/yum -d 0 -e 0 -y install mongodb-org-shell-4.0'
which obviously fails as package does not exist.
What behaviour did you expect instead
That package name would be "mongodb-org-shell" - not not "mongodb-org-shell-4.0".
Output log
Debug: Executing: '/bin/rpm -q mongodb-org-shell --nosignature --nodigest --qf '%{NAME} %|EPOCH?{%{EPOCH}}:{0}| %{VERSION} %{RELEASE} %{ARCH}\n''
Debug: Executing: '/bin/rpm -q mongodb-org-shell --nosignature --nodigest --qf '%{NAME} %|EPOCH?{%{EPOCH}}:{0}| %{VERSION} %{RELEASE} %{ARCH}\n' --whatprovides'
Debug: Executing: '/bin/yum -d 0 -e 0 -y install mongodb-org-shell-4.0'
Error: Could not update: Execution of '/bin/yum -d 0 -e 0 -y install mongodb-org-shell-4.0' returned 1: Error: Nothing to do
Error: /Stage[main]/Mongodb::Client/Package[mongodb_client]/ensure: change from purged to 4.0 failed: Could not update: Execution of '/bin/yum -d 0 -e 0 -y install mongodb-org-shell-4.0' returned 1: Error: Nothing to do
Any additional information you'd like to impart

It does the same if I don't install mongodb::client - for the server.. it tries to install:  mongodb-org-server-4.0

Setting this in hieradata:
mongodb::repo::version: '4.0'
and NOT setting version in mongodb::globals - works.


Network prune should not fail if containers vanish

This is yet another instance of #3167
This here is less urgent than #3192 as it only affects network prune.
Issue was seen 3 times on this specific canary build: https://github.com/containerd/nerdctl/actions/runs/9815148442?pr=3189
Note that this is on top of #3192 - I will rebase on main as soon as it gets merged.

Back on main.
1 failure - still IPFS.
This one is good to go @AkihiroSuda


Use eslint-config-react-app instead of rackt for linting

Changed to use eslint-config-react-app
eslint-config-react-app from create-react-app repo instead of rackt.
Enabled linting for the examples which was disabled because of linting mismatch between examples created by create-react-app and redux linter https://github.com/reactjs/redux/pull/1928.
Fixed a few lint errors according to the new config.
babel-eslint still used as peer dependency for eslint-config-react-app

Any reason we can't get rid of babel-eslint? Are we using decorators anywhere? That's normally what I've had to use it for.

I went ahead and did this by hand since there were merge conflicts. Thanks for the PR for me to copy from!


pass --vips-tile-height for pdfload

currently testing out libvips with poppler and was wondering how to pass the --libvips-tile-height argument like https://github.com/jcupitt/libvips/blob/fc7f9d49410e955c5086a3814a413745d37d27ad/libvips/foreign/pdfload.c#L693
currently converting a PDF is taking about 5 - 6 seconds but only using ~30 - 50mb of memory out of ~1500mb so was looking to try bench mark it with larger tile heights to see if I can return quicker results
cheers

Hello, this libvips init-time parameter is currently unavailable via sharp.
Given you are compiling your own libvips from source, perhaps modify private.h#L145-L149 with the values you need.

So I've been doing some benchmarks and a lot of recompiling and it seems poppler looses a lot of speed when piped through cairo so I'll still use sharp for the image conversion side of things
In the mean while I'm going to do some bench marks against the opensourced project pdfium from google + foxit that runs in chromium (https://pdfium.googlesource.com/pdfium/)
From the benchmarks I've seen it's atleast 7x quicker, handles colorspaces correctly and is actively maintained with similar low memory usage
not sure the chances of libvips getting direct intergration with pdfium but would be nice seeing that it's opensource and could be included by default.
cheers!

Thanks for the update. Perhaps suggest the future possible use of pdfium over at the libvips repo. (I had a quick look but it seems to be lacking in API documentation and has yet to be made widely available as "libpdfium".)

Here's an enhancement issue in libvips for pdfium:
https://github.com/jcupitt/libvips/issues/689
As you say @lovell, it looks promising but a little immature at the moment.

@lovell is there any possibility to add this parameter after these years?

I have tried changing the parameter directly in source as suggested but it does not seem to be taking effect.

Is there any way to check in compiled vips if the parameter is taking effect?


Remove accelerators from tray menu

As spotted in #239
Those accelerators are now gone from the tray menu

On OS X 10.11.5, the shortcuts properly work with application's menu after removing.

+1
All looks good on Windows 10 as well, thanks @Razzeee!
Anyone who would like to test it out on Linux? You can download the artifacts here:

mattermost-desktop-linux-ia32.tar.gz
mattermost-desktop-linux-x64.tar.gz

Anyone from the community who would like to test this PR on Linux? Perhaps @jnugh or @timroes?
We'd be ready to merge the PR after,

To test this stuff, it would be useful to have in the description what really changed, best with some screenshots, or at least some more detailed descriptions. Also do there need to be some special settings I have to turn on, to get this working. Iin my case, version 1.3.0 (from package manager) looked as follows:

So not sure what should have changed, in contrast to that?

@timroes Thank you for checking! Assuming that, there are no changes in Linux.
In Windows and OS X, there was shortcuts for tray icon, but they had no effect. So they are removed in this PR.

In conclusion, the PR is now ready to merge.

@Razzeee Please update CHANGELOG.md. Should this be categorized as Fixes?

On Ubuntu unity - no shortcuts where/are visible. So this does not change
anything in this case. As always - depends on desktop environment.
Yuya Ochiai<EMAIL_ADDRESS>schrieb am So., 21. Aug. 2016, 17:01:

@Razzeee https://github.com/Razzeee Please update CHANGELOG.md. Should
this be categorized as Fixes?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mattermost/desktop/pull/242#issuecomment-241262619,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAIv7S6sg0B-uPeWdKDk20Ou8Ltvni2dks5qiGhIgaJpZM4JjoeR
.

@yuya-oc
Added changelog

Thanks all!


Bug fix: TypeError: DataFrame.sort() got an unexpected keyword argument 'reverse'

Hi!
I was test driving tidypolars today (looks great!), but I had some trouble using arrange.
import tidypolars as tp
from tidypolars import col, desc
print(tp.__version__)

tp.Tibble(x = range(3), y = range(3, 6), z = ['a', 'a', 'b']).arrange('x', 'y')

---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
Cell In[2], line 5
      2 from tidypolars import col, desc
      3 print(tp.__version__)
----> 5 tp.Tibble(x = range(3), y = range(3, 6), z = ['a', 'a', 'b']).arrange('x', 'y')

File [c:\[snip]\anaconda3\envs\[snip]\lib\site-packages\tidypolars\tibble.py:102](file:///C:/Users/[snip]/anaconda3/envs/[snip]/lib/site-packages/tidypolars/tibble.py:102), in Tibble.arrange(self, *args)
    100 exprs = _as_list(args)
    101 desc = [True if isinstance(expr, DescCol) else False for expr in exprs]
--> 102 return super().sort(exprs, reverse = desc).pipe(from_polars)

TypeError: DataFrame.sort() got an unexpected keyword argument 'reverse'

I checked polars docs, which state that sort accepts:

Parameters:
by
Column(s) to sort by. Accepts expression input. Strings are parsed as column names.
*more_by
Additional columns to sort by, specified as positional arguments.
descending
Sort in descending order. When sorting by multiple columns, can be specified per column by passing a sequence of booleans.
nulls_last
Place null values last.

For this reason, I renamed the argument from reverse to descending, and it seems to work. I haven't run any tests.
I hope this helps, and sorry for a drive-by commit/PR.
— Maciej
Update: env info yml file attached as environment_yml.txt

Thanks for the fix!
polars has been implementing changes (and breaking changes) faster than I can keep with. Changing reverse to descending in .sort() was one of them. It's good that polars is constantly improving, but the API instability makes it a bit hard to build a package that depends on it unfortunately.
There are probably a lot more fixes necessary to get tidypolars back to a good state. I'll try to get to them at some point, but between a newborn and a new job I'm bit behind on keeping this package up to date. I've only had enough time lately to keep updates to R's tidytable and dtplyr.

Thanks for the fix!
between a newborn and a new job I'm bit behind on keeping this package up to date

Don't worry about it, and thank you for maintaining so many useful packages! Also, congratulations on the new addition to your family!


Tead not working with Node 10

I updated Node from v8 to v10 and then my tead tests broke. I get the error:
/node_modules/tead/src/index.js:1
TypeError: Hu.wrap is not a function

@spatialillusions I've released 0.5.2. Please have a look.