Create listtablets shell command. Closes #1317

New command for debugging tablets called listtablets
Added getLiveTServers() to TabletMetadata for generating a list of
tservers that currently have a lock in ZK, similar to master.
The list of live tservers is passed to TabletMetadata in order to get
the current state of a tablet
Command will print one line for every tablet in a table
Created TabletMetadataIT for testing getLiveTServers()

Co-authored-by: EdColeman<EMAIL_ADDRESS>

@ctubbsii I think this PR good to go.  Did you have any other feedback before I merge?

@ctubbsii I think this PR good to go.  Did you have any other feedback before I merge?

@ctubbsii I think this PR good to go. Did you have any other feedback before I merge?

No. Go ahead.

@ctubbsii I think this PR good to go. Did you have any other feedback before I merge?

No. Go ahead.

FYI here is an example of output from the command using some test data:

root@uno> listtablets -t test_ingest -h
2021-01-04T15:12:47,663 [Shell.audit] INFO : root@uno> listtablets -t test_ingest -h
NUM  TABLET_DIR      FILES WALS  ENTRIES   SIZE      STATUS     LOCATION                       ID    START (Exclusive)    END
TABLE: test_ingest
1    t-0000007       1     0            60       552 HOSTED     CURRENT:ip-10-113-12-25:9997   2     -INF                 row_0000000005
2    t-0000006       1     0           500     2.71K HOSTED     CURRENT:ip-10-113-12-25:9997   2     row_0000000005       row_0000000055
3    t-0000008       1     0         5.00K    24.74K HOSTED     CURRENT:ip-10-113-12-25:9997   2     row_0000000055       row_0000000555
4    default_tablet  1     0         4.44K    22.01K HOSTED     CURRENT:ip-10-113-12-25:9997   2     row_0000000555       +INF
root@uno> listtablets -t accumulo.metadata
2021-01-04T15:13:21,750 [Shell.audit] INFO : root@uno> listtablets -t accumulo.metadata
NUM  TABLET_DIR      FILES WALS  ENTRIES   SIZE      STATUS     LOCATION                       ID    START (Exclusive)    END
TABLE: accumulo.metadata
1    table_info      2     0     7         524       HOSTED     CURRENT:ip-10-113-12-25:9997   !0    -INF                 ~
2    default_tablet  0     0     0         0         HOSTED     CURRENT:ip-10-113-12-25:9997   !0    ~                    +INF

FYI here is an example of output from the command using some test data:

root@uno> listtablets -t test_ingest -h
2021-01-04T15:12:47,663 [Shell.audit] INFO : root@uno> listtablets -t test_ingest -h
NUM  TABLET_DIR      FILES WALS  ENTRIES   SIZE      STATUS     LOCATION                       ID    START (Exclusive)    END
TABLE: test_ingest
1    t-0000007       1     0            60       552 HOSTED     CURRENT:ip-10-113-12-25:9997   2     -INF                 row_0000000005
2    t-0000006       1     0           500     2.71K HOSTED     CURRENT:ip-10-113-12-25:9997   2     row_0000000005       row_0000000055
3    t-0000008       1     0         5.00K    24.74K HOSTED     CURRENT:ip-10-113-12-25:9997   2     row_0000000055       row_0000000555
4    default_tablet  1     0         4.44K    22.01K HOSTED     CURRENT:ip-10-113-12-25:9997   2     row_0000000555       +INF
root@uno> listtablets -t accumulo.metadata
2021-01-04T15:13:21,750 [Shell.audit] INFO : root@uno> listtablets -t accumulo.metadata
NUM  TABLET_DIR      FILES WALS  ENTRIES   SIZE      STATUS     LOCATION                       ID    START (Exclusive)    END
TABLE: accumulo.metadata
1    table_info      2     0     7         524       HOSTED     CURRENT:ip-10-113-12-25:9997   !0    -INF                 ~
2    default_tablet  0     0     0         0         HOSTED     CURRENT:ip-10-113-12-25:9997   !0    ~                    +INF


Visdom blank blue screen, when setting layout title plotlyplot

Bug Description
I got a blank blue screen of wisdom (<IP_ADDRESS>) when I set the title of the plotly annotated heatmap.
Reproduction Steps
fig = ff.create_annotated_heatmap(z, x=x, y=y, colorscale='Viridis', showscale=True)
fig.layout.title='Test title'

viz.plotlyplot(fig)

Client logs:
TypeError: n.title.match is not a function
    at main.js?v=4316a77ded15b92b6e884b60b72f6513:35
    at Array.map (<anonymous>)
    at t.value (main.js?v=4316a77ded15b92b6e884b60b72f6513:35)
    at h (react-dom.min.js?v=950495cc51ccb90612cf0fe0bb44f8f3:130)
    at beginWork (react-dom.min.js?v=950495cc51ccb90612cf0fe0bb44f8f3:134)
    at d (react-dom.min.js?v=950495cc51ccb90612cf0fe0bb44f8f3:158)
    at f (react-dom.min.js?v=950495cc51ccb90612cf0fe0bb44f8f3:159)
    at g (react-dom.min.js?v=950495cc51ccb90612cf0fe0bb44f8f3:159)
    at t (react-dom.min.js?v=950495cc51ccb90612cf0fe0bb44f8f3:167)
    at x (react-dom.min.js?v=950495cc51ccb90612cf0fe0bb44f8f3:166)

Server logs:

INFO:root:Opened visdom socket from ip: <IP_ADDRESS>
INFO:tornado.access:200 POST /events (<IP_ADDRESS>) 1.59ms

@ktagowski This issue was fixed with #561
You could either try installing visdom from source for now (pip install -e . in source dir), or wait until a new release is cut (cc/ @JackUrb )

@pringshia Thank you very much for your help. I haven't noticed that this commit solves it.


Support DELIMITER syntax

Disclaimer: I don't know a whole lot of SQL

So my dev team decided that we wanted a Node script that could drop and re-create all of our stored procedures whenever something changes. The problem was that this library doesn't support the delimiter syntax. I made a kind of hacky way to split statements from my .sql (loaded to an in-memory string) file by custom delimiters into an array of separate statements.
Do you guys have a suggested way of working with something like this? How about suggestions for where to start for a PR?
Ex:
DELIMITER $$
DROP PROCEDURE IF EXISTS `Thing_Insert`$$
CREATE PROCEDURE `Thing_Insert`(
	argA int,
	argB nvarchar(50),
	argC nvarchar(50),
	argD datetime,
	argE bit)
BEGIN

	insert into Thing (A, B, C, D, E)
		select argA, argB, argC, argD, argE;

END$$
DELIMITER ;

...etc

/**
 * Separates individual SQL statements, split by a delimiter, into an array of statements
 * @param {String} text
 * @returns {String[]}
 */
function separateSqlCommands(text) {
    let lines = text.split('\n')
        .filter(str => {
            return str.length
                && !/^\n*$/.test(str);
        });

    let delimiter = ';';
    const DELIMITER_REGEX = /DELIMITER (.+)/;

    let statements = [];

    let accumulator = '';

    for(const line of lines) {
        let regexpResult = DELIMITER_REGEX.exec(line);
        if(regexpResult) {
            statements = statements.concat(accumulator.split(delimiter));

            accumulator = '';
            delimiter = regexpResult[1];
        } else {
            accumulator += line + ' ';
        }
    }

    statements = statements.filter(statement => !/^\s*$/.test(statement))
        .map(statement => {
            if(statement[0] === ' ') statement = statement.substr(1);

            return statement.replace(/\t/g, ' ')
                .replace(/ +/g, ' ');
        })
        .map(statement => statement.replace(/ +/g, ' '));

    return statements;
}

example output:
[ 'drop procedure `Thing_Insert`',
  'CREATE PROCEDURE `Thing_Insert`( argA int, argB nvarchar(50), argC nvarchar(50), argD datetime, argE bit) BEGIN insert into Thing (A, B, C, D, E) select argA, argB, argC, argD, argE; END' ]

👍 Having the same issue.

I think you should use DROP PROCEDURE IF EXISTS -- just in case it aint there
and simple batch script:
mysql -u username -p database_name < file.sql
no need for node or mysql module

@kpetrow while using the mysql CLI directly is always an option, the point here is that one might want to extract out the individual SQL commands and programmatically work with them

If you are going to work with individual statements then you are going to need to manually parse anyways?  If you are going to slam into db use CLI?

The hard part about this is that all this module does right now is implement the MySQL protocol, and supporting DELIMITER means that the module would have to implement and entire SQL dialect parser to get that added, but if you want to put a PR together, that's certainly good! I know the separateSqlCommands is just what you are using now, so I'm sure you're not surprised that it has a few edge cases that would have to be fixed to actually add it here :)

Do you guys have a suggested way of working with something like this?

I would suggest that if you don't even need to keep it in a .sql file that you can load into the mysql command line client, then simply store the literal procedure content alone. For example, if you just store your procedure above as
CREATE PROCEDURE `Thing_Insert`(
	argA int,
	argB nvarchar(50),
	argC nvarchar(50),
	argD datetime,
	argE bit)
BEGIN

	insert into Thing (A, B, C, D, E)
		select argA, argB, argC, argD, argE;

END;

Then you can easily use connection.query to run that and create the procedure. You can run the drop if exists as a connection.query call before that one to get the same effect.

How about suggestions for where to start for a PR?

If you are looking to add support for DELIMITER the first step is to either find a MySQL-dialect parser or build one. Then you can add support for DELIMITER and it can be used to slice up the queries to send over the protocol.

Can you give an example of an edge case?

Hi @kpetrow , the code above has a lot of issues, the easiest to see are the following:

The occurrence of ADELIMITER $$ will incorrectly change the delimiter to $$.
The function will incorrectly match a delimiter that is within a string in the SQL.
The word DELIMITER is not required to be in upper-case, but this will only match upper-case occurrences.
The line foo DELIMITER $$ will incorrectly change the delimiter to $$.
The line DELIMITER "$$" will incorrectly change the delimiter to "$$" instead of $$.

I can certainly continue on, but it's really just diminishing returns when really the answer is to just fix up the edge cases by implementing it exactly how MySQL implements it.

I was thinking nested delimeters, but was curious if there was SP, trigger or something that would would need to be implemented in some odd way.  Thanks

I haven't seen that, but not sure if that's even a thing. That's probably why the best answer is just to implement it exactly how the MySQL client is doing it :) Then we don't have to guess, haha. The protocol was just implemented from the MySQL protocol documentation, but I'm not familiar with any documentation around the delimiter and how it works, exactly.

re-inventing the wheel is not a good idea. It presents maintenance and compatibility issues. There is a solution already. See: https://dev.mysql.com/doc/refman/5.7/en/c-api-multiple-queries.html

@dougwilson Yeah, I was sure there would be plenty of cases where my code wouldn't always work. It was mainly a proof of concept that works well for my use case. Maybe others will find it useful.
I found a SQL parser for JS here: https://github.com/forward/sql-parser. The problem is, at this point, I'm not sure if this should be built into this library, or added as a separate module for working with SQL files using the DELIMITER syntax.

I thought I'd clarify a little for anyone coming here looking for DELIMITER support. Most people probably don't need DELIMITER support. The OP only needs it because he's loading external .sql files that were generated for the MySQL CLI.
DELIMITER is needed in the MySQL CLI because it interprets queries containing semicolons as multiple statements. DELIMITER allows you to control the splitting behavior. By default, mysql doesn't interpret the query. It just sends it directly to the server. So you can just send queries that include semicolons without a problem.
The OPs queries can be run without DELIMITER support. Just make sure to make a connection with multipleStatements set to false and run the statements one at a time.
'use strict';
const mysql = require('mysql');

const config = {
    host: 'localhost',
    user: 'user',
    password: 'password',
    database: 'database',
    multipleStatements: false //false is the default, added for emphasis
};

const dropQuery = 'DROP PROCEDURE IF EXISTS `Thing_Insert`';
const createQuery = `CREATE PROCEDURE \`Thing_Insert\`(
        argA int,
        argB nvarchar(50),
        argC nvarchar(50),
        argD datetime,
        argE bit)
    BEGIN

        insert into Thing (A, B, C, D, E)
            select argA, argB, argC, argD, argE;

    END`;

const connection = mysql.createConnection(config);
connection.connect(function(err) {
    if (err) {
        console.error('error connecting: ' + err.stack);
        return;
    }

    connection.query(dropQuery, function(err2) {
        if (err2) {
            console.error('error querying: ' + err2.stack);
            return;
        }

        console.log('drop succeeded');

        connection.query(createQuery, function(err3) {
            if (err3) {
                console.error('error querying: ' + err3.stack);
                return;
            }

            console.log('create succeeded');
            connection.end();
        });
    });
});

@ben-page and @dougwilson
Removing the DELIMITER works!!
const _sql = sql.toString().replace(/DELIMITER ;?;/gm, '').replace(/;;/gm, ';')

Thanks!

I spent my weekend and published a module which hope to provide a more standard solution to the missing DELIMITER support. I have handled all edge cases pointed out by @dougwilson. It is not a full MySQL language parser but it should be able to complete the work on most of our daily usage scenario.

Any news on this? Is it still not possible to create a trigger/procedure with multiple statements inside (delimiter case)?

Any news on this? Is it still not possible to create a trigger/procedure with multiple statements inside (delimiter case)?

ok, just figured it out.
If the delimiter lines are removed, somehow it works fine (even though it does not work via a SQL client). So, for example:
create or replace trigger trigger1 after insert on table1 for each row
begin
  declare _var1 int(11);
  select a into _var1 from table1;
end

OBS.: I did the test using the 'mariadb' lib, but I suppose it will be the same for 'mysql' as well.

ok, just figured it out.
If the delimiter lines are removed, somehow it works fine (even though it does not work via a SQL client). So, for example:
create or replace trigger trigger1 after insert on table1 for each row
begin
  declare _var1 int(11);
  select a into _var1 from table1;
end

OBS.: I did the test using the 'mariadb' lib, but I suppose it will be the same for 'mysql' as well.

ok, just figured it out.
If the delimiter lines are removed, somehow it works fine (even though it does not work via a SQL client). So, for example:
create or replace trigger trigger1 after insert on table1 for each row
begin
  declare _var1 int(11);
  select a into _var1 from table1;
end

Thank you! +1 for above!


Make $accordion-plusminus a parameter on accordion-title

How to reproduce this bug:
Use accordion mixins (accordion-item in particular)
What should happen:
It would be nice to be able to override $accordion-plusminus as a parameter on @mixin accordion-title.
It would also be nice if you could specify what the plus and minus content is, so that different characters could be used
What happened instead:
Cannot override default setting of $accordion-plusminus when including accordion-title mixin, because no parameter is available for that value.
Foundation Version(s) you are using:
6.4.1
Test case link:

accordion-item mixin without $plusminus parameter generates SASS error
accordion-item mixin with $plusminus parameter

This looks good @brentdodell ... Would you like to do a PR for it ??

This looks good @brentdodell ... Would you like to do a PR for it ??

Sure! I will when I have a chance. I didn't want to jump into it too quickly because I hadn't looked over Zurb's contribution guidelines.
Are you thinking just parameterizing $accordion-plusminus, or also allowing for specifying custom content in place of +/- like I also suggested?

Hey @brentdodell thanks for considering contributing guidelines
For the core,
i would think that just parameterizing $accordion-plusminus would be fine
But then we have this too (http://foundation.zurb.com/sites/docs/prototyping-utilities.html)
So for my thought adding a class where an accordion will have arrow instead of plusminus will be great too for optional prototypes!
Again, this is just very personal opinion and maybe other yetinauts may consider different
So i will cc @kball for his opionion ??

I would actually prefer the ability to set arbitrary content for open & closed states

@kball You mean to choose b/w plusminus and arrows ??
Yup that was on my mind too!

I think that will be good one choice also!

Hey @brentdodell ... Any updates on this ??

No @IamManchanda, I've been on vacation with my family for the past week and will probably be catching up at work this next week, so I haven't had a chance to look at it.

Hi @brentdodell, do you still want to provide a PR?

See https://github.com/zurb/foundation-sites/pull/11025 (@jellisii)

I guess we can close this then?


IS GCM supported on AIX.

Hi,
We wish to incorporate GCM to our program by bouncycastle on AIX. Right now, we find that there is something wrong with it.
We wish to ask whether GCM is supported on AIX first. Meanwhile, CBC works well.
Thanks.

Yes GCM is supported.


The source "github release" should be able select draft, pre-release or release

Available since v0.38.0


ci: allow ci job failure

laminas/laminas-httphandlerrunner ^2.6 does not support PHP 8.3.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
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 98.98%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
97

Relevant Lines:
98

💛  - Coveralls


nudge resource path

where should the screenshots/json files exist?

Screenshots do not matter because they can be anywhere.
Preferences for the json will exist here:
/Library/Preferences/com.github.macadmins.Nudge.json


Error During Build

Version
2.4.2
Reproduction link
https://github.com/Splode/lww-test
Steps to reproduce

In CLI, npm run build

What is expected?
Expected a Vue build without errors.
What is actually happening?
Build process exits incomplete, with an error. Error seems to occur after CSS optimization process:
| building for production...
Starting to optimize CSS...
Processing static/css/app.8e679f67219356a9327dea38c0984880.css...
Processed static/css/app.8e679f67219356a9327dea38c0984880.css, before: 38178, after: 38110, ratio: 99.82%
(node:12596) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 1): Error: Cyclic dependency: "[object Object]"
(node:12596) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with
 a non-zero exit code.

Dev server will run as expected with both npm run dev and npm run start.
Vue-CLI is version 2.9.1

I've also reproduced this here: https://gitlab.com/HackerHappyHour/wedreg.
What's even more awesome for reproduction purposes, is that my repro is available via a docker image. To reproduce:
git clone https://gitlab.com/HackerHappyHour/wedreg.git
cd wedreg
docker build -t wedreg .
docker run -it --rm -v `pwd`:/wedreg wedreg install
docker run -it --rm -v `pwd`:/wedreg - P wedreg run dev

The last 2 commands execute npm install and npm run dev, and result in the same error listed above. You can inspect this static system by running:
docker run -it --rm -v `pwd`:/wedreg -P --entrypoint /bin/bash wedreg

All files from the project, including node_modules should already be on your local machine in the wedreg folder you cloned to.

I'm getting a similar error, and again the dev server is working as expected.
⠇ building for production...Error processing file: static/css/app.a9c7f7cf4b2e38c2e4838cd2748640a3.css
(node:13368) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 2): SyntaxError: Unexpected token � in JSON at position 111054
(node:13368) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

Is this the same issue? Any chance anyone knows how to start tracking it down?

I'm getting @Tattomoosa error with the CSS-Framework  "materialdesignicons".
Is there any update in this topic?


use as jekyll command

I like the functionality of this plugin,
just a thought: could be simpler to use implemented as a jekyll command plugin rather than fiddling around with the pm_generate value. Then, every time you want to build/rebuild the collection of .md, you could just say jekyll pagemaster or what ever to run the plugin one time.
thank you.

Good point! I'm happy to test this out for a 2.0 when I find the time, but feel free to submit a PR if you get to it before I do.
Cheers,
Marii


correct insert br & p tags from Word

correct insert  and  tags from Word

scary avatar

@seanfcarroll yes, avatar is directly from Hell.

Can you provide more information about the problem you are trying to solve with steps to reproduce it?


Fix formatting

The formatting of HLint varies significantly between files and within files. There is 2 space indents, 4 space indents. It's just a bit haphazard. We should align on a standard. Ideally with tooling support. My first preference would be fourmolu, for which I am waiting for @parsonsmatt (who was probably being sarcastic anyway...). In the absence of fourmolu, do people have any other preferences?
CC @shayne-fletcher @zliu41 @googleson78 @josephcsible

I favor 2 spaces generally.

For the avoidance of doubt though, hill die? Me no 😁 Happy to go with the consensus!

If there's a real need or desire for fourmolu then I can try to get it going. Looks like they made it nice-and-easy by writing indentStep = 2, so it's a one line patch to change it :smile:

As Shayne says, every few people go to die on their formatting hill, but 4 is my preference, and since I also write a fair bit of Rust/Javascript etc where formatters are common and 4 is the answer, it would reduce the mental burden of switching for me. But if all other collaborators come out in favour of 2, then 2 it is, and twormulu can be used.

ohhhh i hope i don't get flamed for this: fourmolu

I have a weak preference for 2 spaces.

stylish-haskell :) [i might be biased  :) ]

@EncodePanda an online formatter would let us try before we pick, if something could be made available. The existence for Ormulu is very handy.

@ndmitchell thing about stylish-haskell is that it's not "opinionated" like Ormulu, there is a config file attached to it so you can apply different styles. So the "online formatting" would also have to take that into consideration. I'm happy to provide an example of how formatting could look like, not sure if I have resources to run something as a webserver to try things out.
But I could create a branch of current version of hlint with some default configuration enabled to demo (via diff) what the output would be. Would that make sense?

I'm neutral on which formatter to use, or even whether to use a formatter at all vs. just a style guide.
I'd have been a lot happier with Ormolu if it adopted leading ,, -> and => rather than trailing.
I really like this from stylish-haskell README:

This tool tries to help where necessary without getting in the way.

It seems the only downside is that stylish-haskell is based on haskell-src-exts.
btw I also have a weak preference of 2 spaces like some others said.

I'd have been a lot happier with Ormolu if it adopted leading ,, -> and => rather than trailing.

You remind me. I ran ormolu over a list literal heavy file a while back and was most displeased with the results (I forget why exactly but I expect it was this).

It seems the only downside is that stylish-haskell is based on haskell-src-exts.

We are moving ghc-lib-parser over the summer, see https://summer.haskell.org/ideas.html#stylish-haskell-ghc-lib-parser :)

If you decide on:

column width
vertical alignment vs "sane git  diffs"

I can create a branch formatted by stylish-haskell for you to decide if this is the right way to go.

It seems the only downside is that stylish-haskell is based on haskell-src-exts.

We are moving to ghc-lib-parser over the summer, see https://summer.haskell.org/ideas.html#stylish-haskell-ghc-lib-parser :)

Awesome! 🎉

We are moving to ghc-lib-parser over the summer, see https://summer.haskell.org/ideas.html#stylish-haskell-ghc-lib-parser :)

Awesome! 🎉

You may on that note also have an interest in ghc-lib-parser-ex https://github.com/shayne-fletcher/ghc-lib-parser-ex? It's serving stuff to HLint and I'd be more than happy to help pull down things that fit this project.

You may on that note also have an interest in ghc-lib-parser-ex https://github.com/shayne-fletcher/ghc-lib-parser-ex? It's serving stuff to HLint and I'd be more than happy to help pull down things that ~fit~ benefit this project.

Thanks, will definitely have a look :)

ghc-lib-parser-ex might be useful to the formatter, but even if it's still on haskell-src-exts, I don't think that's a massive problem - we are very familiar with all the issues with it, and how to avoid them, so I don't think it's a problem.
I have no issue with git diffs that are one line longer if the code looks nicer to read. Things which produce O(n) worse diffs are a bit sad, so best avoided.
I also have no sane idea how to suggest we pick between the options. Everyone seems to prefer 2 spaces, apart from me, so I can go with the flow. But which formatter and which flags is not obvious.

Pistols at dawn? 😉

🔫

Interesting!

I'm beginning to believe.

@felixmulder's most excellent stylish-haskell branch : https://github.com/jaspervdj/stylish-haskell/pull/293

@EncodePanda

If you decide on:

column width
vertical alignment vs "sane git  diffs"

I can create a branch formatted by stylish-haskell for you to decide if this is the right way to go.

In the absence of other answers, may I suggest:

78 columns;
I don't know what this means but do advocate git diff minimizations where possible if that helps.

@EncodePanda

If you decide on:

column width
vertical alignment vs "sane git  diffs"

I don't know what this means but do advocate git diff minimizations where possible if that helps.

Oh! I think I get what you are saying. I am most definitely NOT in favor of vertical alignment (even in the absence of e.g. ImportQualifiedPost).

I'm fine with 78 columns. I'm however not sure how enthusiastic Neil is about that, since hlint code regularly exceeds 120 :stuck_out_tongue:

I'm fine with 78 columns. I'm however not sure how enthusiastic Neil is about that, since hlint code regularly exceeds 120 😛

🤣

I'm also not sure about placing a limit on columns (although obviously I'm not actively contributing :P) - most of the time I tried to split my pattern matches on ghc's ast into multiple lines it ended up being less readable.. Not sure what the "right" solution is here - maybe more granular functions??
Vertical alignment is great! .. if you're not looking at any diffs ever.

I'm also not sure about placing a limit on columns (although obviously I'm not actively contributing :P) - most of the times that I tried to split my pattern matches on ghc's ast into multiple lines it ended up being less readable.. Not sure what the "right" solution is here - maybe more granular functions??
Vertical alignment is great! .. if you're not looking at any diffs ever.
(btw, why 78 instead of 80?)

actually, yes, you're of course right, 80 cols would be fine. i don't know why my mind jumped to 78... perhaps habit in that it's guaranteed to fit within 80. anyway, with all the real estate offered by modern monitors one might wonder if the sacred 80 column rule is relevant any more. i think it is and feel supported by arguments like these.

I'm more a fan of 120. But I'm not too concerned as long as it doesn't increase the code length dramatically.

I will create PR with code formatted by stylish for you folks to have a look.

120 column length,
no vertical alignment,
4 spaces indentation.


Is high level parsing of event logs done here?

Sorry for not researching enough before opening an issue. Though the code structure is very unfamiliar for me.

Check the parser.js file in svc and processor functions in processors.  These convert the event log into a JSON object for each match.


Fix ExpoPushSuccessTicket to use ExpoPushReceiptId

There is a compilation error in lib.rs, but it might be fixed by other Pull Requests #63

I should have rebased it ...


Update http-adapter dependency

One of my other packages required 'psr/http-message' ~1.0, so I couldn't install flysystem-http-downloader.
Update to be compatible with 'psr/http-message' ~1.0 (via 'egeloen/http-adapter' and 'phly/http')

Note: While the tests pass, I can't really get it to work. Not sure if it's a problem with the package or if I'm just using it wrong though.
$downloader = new \Indigo\Flysystem\Downloader(\Storage::disk('local')->getDriver(), new GuzzleHttpAdapter);
$request = new \Phly\Http\Request($url);
$downloader->download($request, $tmpName);

gives me an empty file (Storage is just a facade to Flysystem).

The reason I tried the GuzzleHttpAdapter adapter was that the GuzzleHttpHttpAdapter adapter gave
[2015-07-01 16:46:57] local.ERROR: exception 'Symfony\Component\Debug\Exception\FatalErrorException' with message 'Call to undefined method ErrorException::getRequest()' in /var/www/backend/vendor/egeloen/http-adapter/src/GuzzleHttpHttpAdapter.php:62

Your issue seems to be one with egeloen/http-adapter. Please not it is going to be replaced with php-http/adapter, where this issue should not exist. If you wish, you can open a PR for using php-http/adapter. To do that:

Check the current state of php-http and it's initial documentation (php-http.readthedocs.org)
Replace egeloen/http-adapter with php-http/adapter and php-http/adapter-implementation
Fix the interface names

Done!
Since this package is not stable yet, we are free to use beta dependencies. PHP-HTTP became pretty stable, so it is safe to rely on it.

Till then, I am just going to merge this.

Thanks, I'll have a look at it.

Thank you for contributing


Preloader added

🛠 Fixes Issue #603

👨💻 Changes proposed and Brief Description
I created a pre-loader for a 30DaysOfJavaScript website.

✅ Check List (Check all the applicable boxes)

[x] My code doesn't break any part of the project
[x] This PR does not contain plagiarized content.
[x] My Addition/Changes works properly and matches the overall repo pattern.
[x] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers
I created a pre-loader using Html, CSS and JS.

📷 Screenshots
https://user-images.githubusercontent.com/52650290/166683365-65a336fe-609e-44fa-b77a-f0b727fe33b7.mp4

@swapnilsparsh kindly review my PR. Thank you.

@swapnilsparsh kindly review my PR. Thank you.

It would be more great if you use anything related to JavaScript in preloader

@swapnilsparsh kindly review my PR. Thank you.

It would be more great if you use anything related to JavaScript in preloader

Preloader is executing through Javascript. Can you give some example so I add more JS in preloader?

Preloader updated
https://user-images.githubusercontent.com/52650290/167091388-d0f2e0e3-2d7b-4f27-9418-dfa616cf6603.mp4


Allow admins to send email through the event list page

Summary:
Added functionality to the email buttons in the event list page (now the home page). Created a modal that pops up when the email button is clicked. The user can fill out the subject and body of the email as well as who is sending the email and who receives it. The senders can be the CELTS Admin or CELTS Student Staff. The recipients can be people who are interested in the program or only people who are RSVP'ed.
To Test:
Go to the website's main page and open one of the accordions (make sure the user is an Admin or CeltsStudentStaff). Click on the email button next to the email. Fill out the different fields and hit send.
Notes:
The "current logged in user" option in the From drop-down is not currently working. We don't know how we should be getting the user's password to send the email. The option didn't exist in the original wireframe but we thought it might be good to include it. Should it be removed?
The "Send Email Individually" button does work but since all of the email recipients are being overridden to the slack channel, it doesn't have any effects. (comment out the override in the emailHandler file in the logic dir. to test it).

This branch was created to fix issue #99

Is this ready for more review or still in progress?


Remove ie9 support

Coverage decreased (-0.005%) to 98.165% when pulling 95be973ad05854faec94c8238fcf3fcb04716e31 on remove-ie9-support into d88a0667894f05fa4654fd9a0cce64ba8d1938aa on master.


Update Mobile UI for Transactions page

Resolve #282
closes #277
ScreenShot

enabling the filter disrupts the the UI

for the desktop design, filter should have the title, also i know you won't be adding the export feature, but add the export UIs as it's part for the tx page work

done

@ukane-philemon

The space btw dropdown does not align with the design, intentional?
Also, the space btw filter icon and its content below do not align with the design.

I have tested it on MacOS and Window, it work for me, I can't reproduce it

A few more nits @JustinBeBoy. Nice work.
I noticed tx search does not work for BTC (couldn’t check LTC) and the txs don't sit well under the search layout (there seems to be an excessive right inset). :
Screen.Recording.2023-12-16.at.2.29.59.PM.mov

@ukane-philemon
Thanks for review, it done please review it again.

Testing well. My observations:

@ukane-philemon
you are right but:

Why the search bar has a smaller because the width we see with the border is not the actual width, in fact it is equal to the list below
Why the right margin is smaller is because that is the display of the scroll indicator, in fact the search bar and list have the same width, you can see on other pages


SPXConnection crashed when pre-connect for SPXSpeechSynthesizer

IN ORDER TO ASSIST YOU, PLEASE PROVIDE THE FOLLOWING:

Speech SDK log taken from a run that exhibits the reported issue.
See instructions on how to take logs.

A stripped down, simplified version of your source code that exhibits the issue. Or, preferably, try to reproduce the problem with one of the public samples in this repository (or a minimally modified version of it), and share the code.

If relevant, a WAV file of your input audio.

Additional information as shown below

Describe the bug
var speechConfig: SPXSpeechConfiguration?
var audioConfig: SPXAudioConfiguration?
do {
    let documentsDirectoryPathString = NSSearchPathForDirectoriesInDomains(.documentDirectory, .userDomainMask, true).first!
    let documentsDirectoryPath = NSURL(string: documentsDirectoryPathString)!
    let logFilePath = documentsDirectoryPath.appendingPathComponent("swift.log")

    try speechConfig = SPXSpeechConfiguration(subscription: GlobalConfig.audioSub, region: GlobalConfig.audioRegion)
    speechConfig?.setPropertyTo(logFilePath!.absoluteString, by: SPXPropertyId.speechLogFilename)
} catch {
    print("error \(error) happened")
}

speechConfig?.speechSynthesisVoiceName = GlobalConfig.voiceName

let synthesizer = try! SPXSpeechSynthesizer(speechConfiguration: speechConfig!, audioConfiguration: nil)

SPXConnection *connection = [[SPXConnection alloc]initFromSpeechSynthesizer:synthesizer];
[connection open: NO];  // crashed here

It's crashed when call  open method.
Speech SDK log:
[142183]: 99291ms SPX_DBG_TRACE_VERBOSE:  resource_manager.cpp:133 Created 'CSpxThreadService' as '303073113'
[142183]: 99295ms SPX_DBG_TRACE_VERBOSE:  resource_manager.cpp:133 Created 'CSpxThreadService' as '303073113'
[142183]: 99295ms SPX_DBG_TRACE_VERBOSE:  named_properties.h:479 ISpxPropertyBagImpl::SetStringValue: this=0x0x00000126c8a1f8; name='CodecAdapterAvailable'; value='true'
[142183]: 99296ms SPX_DBG_TRACE_VERBOSE:  usp_tts_engine_adapter.cpp:62 CSpxUspTtsEngineAdapter
[142183]: 99296ms SPX_DBG_TRACE_VERBOSE:  resource_manager.cpp:133 Created 'CSpxUspTtsEngineAdapter' as '919101823'
[142183]: 99296ms SPX_DBG_TRACE_VERBOSE:  resource_manager.cpp:133 Created 'CSpxNullAudioOutput' as '853898539'
[142183]: 99296ms SPX_DBG_TRACE_VERBOSE:  null_audio_output.h:93 SetFormat is called with format 0x0x300b3c4c0
[142183]: 99296ms SPX_DBG_TRACE_VERBOSE:  cloud_tts_engine_adapter.cpp:134 SetOutput
[142183]: 99296ms SPX_TRACE_INFO:  cloud_tts_engine_adapter.cpp:188 SetOutput: UseCompressedAudioOnWire is set to true, will use compressed format for transmission.
[142183]: 99297ms SPX_DBG_TRACE_VERBOSE:  handle_table.h:111 CSpxHandleTable::TrackHandle p=0x0x126c89e28
[142183]: 99297ms SPX_DBG_TRACE_VERBOSE:  handle_table.h:121 CSpxHandleTable::TrackHandle class=ISpxSynthesizer, h=0x0x126c89e28, p=0x0x126c89e28, tot=1
[142183]: 99297ms SPX_DBG_TRACE_SCOPE_EXIT:  speechapi_c_factory.cpp:548 synthesizer_create_speech_synthesizer_from_config
[142183]: 99297ms SPX_DBG_TRACE_VERBOSE:  handle_table.h:111 CSpxHandleTable::TrackHandle p=0x0x126c8a1f8
[142183]: 99297ms SPX_DBG_TRACE_VERBOSE:  handle_table.h:121 CSpxHandleTable::TrackHandle class=ISpxNamedProperties, h=0x0x126c8a1f8, p=0x0x126c8a1f8, tot=2
[142183]: 99298ms SPX_DBG_TRACE_VERBOSE:  handle_table.h:180 CSpxHandleTable::StopTracking(h) h=0x0x303f3c518
[142183]: 99298ms SPX_DBG_TRACE_VERBOSE:  handle_table.h:195 CSpxHandleTable::StopTracking(h) class=ISpxSpeechConfig, h=0x0x303f3c518, p=0x0x303f3c518, tot=0
[142183]: 99298ms SPX_DBG_TRACE_VERBOSE:  handle_table.h:180 CSpxHandleTable::StopTracking(h) h=0x0x303f3c4b8
[142183]: 99298ms SPX_DBG_TRACE_VERBOSE:  handle_table.h:195 CSpxHandleTable::StopTracking(h) class=ISpxNamedProperties, h=0x0x303f3c4b8, p=0x0x303f3c4b8, tot=1
[142183]: 99298ms SPX_DBG_TRACE_SCOPE_ENTER:  synthesizer_connection.cpp:19 CSpxSynthesizerConnection
[142183]: 99298ms SPX_DBG_TRACE_SCOPE_EXIT:  synthesizer_connection.cpp:19 CSpxSynthesizerConnection
[142183]: 99298ms SPX_DBG_TRACE_VERBOSE:  handle_table.h:111 CSpxHandleTable::TrackHandle p=0x0x302a959a8
[142183]: 99298ms SPX_DBG_TRACE_VERBOSE:  handle_table.h:121 CSpxHandleTable::TrackHandle class=ISpxConnection, h=0x0x302a959a8, p=0x0x302a959a8, tot=1
[142183]: 99298ms SPX_TRACE_INFO:  thread_service.cpp:96 Started thread User with ID [362144ll]
[142183]: 99298ms SPX_DBG_TRACE_VERBOSE:  handle_table.h:180 CSpxHandleTable::StopTracking(h) h=0x0x302a959a8
[142183]: 99298ms SPX_DBG_TRACE_VERBOSE:  handle_table.h:195 CSpxHandleTable::StopTracking(h) class=ISpxConnection, h=0x0x302a959a8, p=0x0x302a959a8, tot=0
[142183]: 99299ms SPX_DBG_TRACE_SCOPE_ENTER:  synthesizer_connection.cpp:24 ~CSpxSynthesizerConnection
[142183]: 99299ms SPX_DBG_TRACE_SCOPE_EXIT:  synthesizer_connection.cpp:24 ~CSpxSynthesizerConnection

Version of the Cognitive Services Speech SDK
'MicrosoftCognitiveServicesSpeech-iOS', '~> 1.37.0'
Platform, Operating System, and Programming Language

OS:  iOS
Hardware - ARM，iPhone 14 Pro
Programming language: Objective-C and Swift

@yulin-li Can you check, should this work?

Thanks for reporting and we are investigating


Ensure CPU alignment takes place appropriately

This test case corresponds to a scenario where CPU being lower
than that required on a candidate (non-target) node should result in pod
placement on a target node where requirement for the rest of resources
is sufficiently met.
Signed-off-by: Swati Sehgal<EMAIL_ADDRESS>

/cc @fromanirh

/cc @fromanirh

thanks, will review shortly

@swatisehgal Thank you for the patch!
was it tested on a cluster?

@swatisehgal Thank you for the patch! was it tested on a cluster?

Yes, this has been tested on a cluster.

/lgtm
on second thought, being explicit and a tiny bit more consistent at manifest level won't hurt

on second thought, being explicit and a tiny bit more consistent at manifest level won't hurt

Okay, will add it back :)

/test ci-e2e

on second thought, being explicit and a tiny bit more consistent at manifest level won't hurt

Okay, will add it back :)

well, not very urgent. Whatever is easier for you now, I'm fine and I'll approve both versions, with and without


Update widget.js

thers is no e.touches touchlist when touchend is fired,it is e.changedTouches.

Please update this file as well:

https://github.com/muaz-khan/Canvas-Designer/blob/master/dev/events-handler.js#L45

Then use grunt to compile into widget.js and widget.min.js.


RTSP Docker inside FARGATE using IAM roles

Wanted to know if it is possible to push data to Kinesis Streams from the RTSP docker demo running as an ECS task using IAM roles (not using a dedicated user) without any source code change.

Sorry, the SDK supports a few credential providers. You will need to build your own similar to the ones that exist - such as IoT or others


[Question]: Is ChatJoinRequest.invite_link always null for public groups?

It seems like ChatJoinRequest.invite_link is always null for join requests to public groups even though the user is invited by through an invite link (not the group's username).  Is this an intended behavior or is it a bug with the server?
Additional info: The invite link is created by the same bot listening to the chat join requests to the particular group chat.

Got the answer from the TDLib chat, this is an intended behavior.


hudson&#8211

VSCode Version:
OS Version:

Steps to Reproduce:

&#8217_&#8220_&#8221

Closing, lack of information.


Example

I will try this out, but did not find any examples, do you have one or should I fix one?
Thanks

Hey @pontusab,
thank you for your interest 🙂
We are currently in app-release-stress and currently working on document all our packages.
Here I've done a basic example for you: https://snack.expo.io/@danielang/react-native-instagram-zoomable

@danielang thanks!


Develop

Adding Dude's Cab

Adding of board defintion addressed by https://github.com/maxgerhardt/platform-raspberrypi/commit/eef7c2908b27788fbcf6b343a71c4a26b5727f47


Eco stuff

In response to #58 changed commands to use ["balances"][server_id] rather than ["balance] which is now used as a global currency
Added commands :
bank transfer <@user> <amount>

To transfer global currency rather than forcing the use of conversion and then of the pay command
fraud <deposit | withdraw> <amount>

To move funds between local economy ( server economy) and global economy
Doing so is taxed a % depending on the day and the servers tax rate

cogs/economy.py line 217:
        print(money_total[str(guild_id)] + money)

think you forgot to remove this


Refactor export map builder

exportMapBuilder.js contains a huge god-object with 651 lines of code, with tightly coupled procedures that are hard to follow.
Lets extract procedures and classes, and put them in well named files.
This separation will also help with encapsulation and decoupling.
It's not perfect yet, but a reduction from 651 lines to 207 lines is a 68% decrease.

this refactor is exposing a lot of previously uncovered lines. It's fine to land this without covering them

I reviewed the lines that aren't covered.
Lets land this without covering them for now.
I can't think of good & quick tests to write for them at this time.

Has been merged via

https://github.com/import-js/eslint-plugin-import/commit/8587c85a60ccb3839cb2eabe50cf098a4a2c03c2


Pairing does not complete on any keyboard

Pairing does not complete on external keyboard. Glass shows paired, but keyboard still sits in discovery mode. Tried with Favi, Microsoft Bluetooth Keyboard, Celluon Epic Laser keyboard. Glass XE16 and XE16.11

Well looks like there isn't much hope for HID getting fixed now. This was in the last email that Glass sent out:
"New feature development on the Explorer Edition will stop while we work on the next version of Glass, but you can still call or email us anytime with questions, thoughts or feedback."

Yeah, I haven't heard about the plans for the SDK yet. I know that the Explorer program is over, but I would imagine that they would have the same SDK for the new version of Glass. So there should be no reason for them to kill the SDK. But like we've seen, their focus is not on SDK features needed by developers.
I will try to reach out to a few people and see if I can get any kind of update on their plans for the SDK between now and the launch of the next version of Glass. Most likely they will do another push of SDK features and developer engagement right before the launch.


[DO NOT MERGE] test pythonlib-juju <IP_ADDRESS>

This PR is for testing pythonlib-juju <IP_ADDRESS>

Closing this as it was used just for debugging purposes.


The version of Spring security used by this plugin is EOL

Spring security 5.1.13.RELEASE is EOL according to their Github page.
https://github.com/spring-projects/spring-security/tree/5.1.x
Please update this plugin to a version that is not EOL.

Additional info on Spring Security EOL status (as of 2021-11-05) suggests moving to at least 5.4.x, if not 5.5.x:

Version
OSS Support Until
Expected End-of-Life

5.1.x

October 2020

5.2.x
September 2020
December 2021

5.3.x
May 2021
December 2021

5.4.x
October 2021
July 2022

5.5.x
May 2022
February 2023

I'm not likely to have time to take the lead on this upgrade, but could help out someone else . . .

Could you please try plugin version 5.0.0-RC1?

5.0.0-RC1 looks like it works.
This plugin seems to use dot notation to access configs, which results in a large number of WARN messages in the console, but it doesn't appear to interfere with how it works.

Thanks, there is a separate issue to track dot notation warnings. See #616


Macports Installation Instructions

I'm trying to use Macports to install chunkwm and I get an error after using the last command.
Error: Port chunkwm not found

Yap, same here...

You need to clone the repo and add it to your macports config as per the instructions. This is because we do not maintain a port in the official macports repository.
PS: The port is not up to date right now, as I can't use macports on macOS Mojave.

I just used the instructions you posted here on https://koekeishiya.github.io/chunkwm/docs/userguide.html
The last step is sudo port install chunkwm
And I got this error... Warning: /opt/local/etc/macports/sources.conf specifies invalid source '/opt/koekeishiya/portfiles', ignored. Error: Port chunkwm not found


Subsystems Prototype

There cannot be a development branch going, only feaeture branches. If it half works, then we should merge it half working. The functionality is actually quite nice.

Why is this PR so large? We should do it one subsystem at a time to make it easier to review


se

I made a lot of mistakes while trying to learn, from now on github start
vinomeran @.***>, 2 Eki 2021 Cmt, 09:06 tarihinde şunu
yazdı:

@.**** requested changes on this pull request.
I need things fixed
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/github/docs/pull/10467#pullrequestreview-769466761,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AV2WBWYYASHVPCRQ5POEORTUE2OOLANCNFSM5FFUW2QA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Add option to select path for BootAnim9 users.

Hello,
Could you please add path selection for BootAnim9 users (for checking version and download.)?
BootAnim9 - default path for payload is SD:/anim/arm9payload.bin

I don't understand the request, are you asking for custom luma a9lh paths? Those have been in since forever. Just set the "payload path" in the configuration file

I'm dumb. I forgot about config file.


feat: track parser names by category

This is required for some introspective tactics in mathlib. Specifically, we keep in the ParserCategory the set of syntax kinds that can produce values of that category. (This is distinct from the kind sets in the parser info, which IIUC collect all kinds that can recursively appear in the generated syntax tree. Here we only collect the kinds that can appear at the root.) I'm using this to generate the tactic list in mathlib.

I'm not sure what you mean. Currently, we still don't have a way to connect the @[builtinTermParser] and @[termParser] attributes to/from the term : Category definition. I'm also not sure whether it is safe to assume there is at most one attribute of each kind, maybe that's just paranoia. What macro are you talking about?

Ah, I was talking about arbitrary macros. For macro_rules, if all the patterns' kinds are of a category c, the RHS should be of type TSyntax c. It's not quite true that every kind is part of only one category (e.g. ident), but I think we can definitely require that for macro-expanded syntax.

Ah, I was talking about arbitrary macros.

Wouldn't macro syntax also show up in the parser tables? I don't think the parser distinguishes between macros and elabs when it's building the syntax object from the input string.

For macro_rules, if all the patterns' kinds are of a category c, the RHS should be of type TSyntax c. It's not quite true that every kind is part of only one category (e.g. ident), but I think we can definitely require that for macro-expanded syntax.

Oh I see, you are thinking about the well typing stuff. For what I need it suffices to be able to enumerate all syntax kinds that can appear in a given category, which does not exclude the possibility that the same syntax kind shows up in more than one syntax category. For well typed macros, it seems like it would suffice to take c to be the declared syntax category of the macro, but then you need coercions or something to turn TSyntax `Lean.Parser.Term.app into TSyntax `Lean.Parser.Category.term and so on, but I suppose you already have this handled?

then you need coercions or something to turn TSyntax `Lean.Parser.Term.app into TSyntax `Lean.Parser.Category.term

Not usually, no. `(f $x) has type Term, not TSyntax ``app.

@digama0 Neat! 🎉 Siddharth Bhat and I had been discussing an RFC for this exact feature for an idea we had. I noticed that you keep track of parser declaration names as a set of syntax kinds. This seems problematic for parsers that do not contribute a syntax kind. For example, custom token parsers (following the pattern of nonReservedSymbol). Should there be a disclaimer that kinds are just parser names and do not necessary produce nodes of the kind name?

The assumption that parsers registered in a category produce a homonymous kind (except for a few built-ins like Parser.Term.ident, which can be blamed on the absence of TSyntax at that time) is not new, the pretty printer makes it as well. Could you elaborate on your use case?

@Kha  Ah, did not realize that this was a common assumption. No use case, was mostly just a question. An example of the kind of parser I was thinking of would be something like my old intTypeLit in Papyrus, which parses some subset of another token in a new token. That one does create a distinct kind, but I have played around with similar parsers that do not. Good to know that I should avoid such things in the future.


Add cors to the sample app

This allows us to use the sample app bff to get data in other contexts

This allows us to use the sample app bff to get data in other contexts

What would be an example of "other" contexts?

This allows us to use the sample app bff to get data in other contexts

What would be an example of "other" contexts?

@gurpreetsainisalesforce for example, we have a component that needs to run in communities.


PARQUET-1449: [C++] Support building with ARROW_BOOST_VENDORED=ON

R failure isn't related.
I'll merge this.


Remove version: Google.Chrome.Canary version 131.0.6754.0

installers return following HTTP status code: 404
Microsoft Reviewers: Open in CodeFlow

URL: https://dl.google.com/release2/chrome/acq7ng7ofurlhkwau6ztop7mp4pq_131.0.6754.0/131.0.6754.0_chrome_installer.exe
Response status code does not indicate success: 404 (Not Found).
(Automated message - build 897)


Week name

Hello
how can I get the week name?
for example:
result of moment(ts).format('dddd') in the moment js is Sunday
but I want when I call that code execute
یکشنبه

Load Persian module before that: moment.loadPersian().
On Esfand 5, 1398 AP, 1:28 PM +0330, miladmodaresi<EMAIL_ADDRESS>wrote:

Hello
how can I get the week name?
for example:
result of moment(ts).format('dddd') in the moment js is Sunday
but I want when I call that code execute
یکشنبه
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

How to use Miladi and Shamsi the same time? ( I need both sunday and یکشنبه)
like m.format('dddd')  // sunday
m.format('jDDDD') // یکشنبه

m.format('jDDDD dddd')
On Shahrivar 12, 1400 AP, 1:38 PM +0430, shafaprince @.***>, wrote:

How to use Miladi and Shamsi the same time? ( I need both sunday and یکشنبه)
like m.format('dddd') // sunday
m.format('jDDDD') // یکشنبه
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

No, sorry, my previous comment is wrong. There is no easy way for what you need, since loadPersian changes the language globally.


[Bug] [Data Quality] Oracle Can't Connect Database Different With Username #12214

Purpose of the pull request
fix issue #12214 that oracle can't connect to database which name is different with username.
Description
Different with MySQL, Oracle can't specify a database by jdbc  url.
So, it will connect to the database equals to username, but one user may have serval database to access.
We found this problem in dataquality module, for example, we have an user named bddb and we want to use database product, it currently not available.

So I made a little change that I can specify a schema in jdbc params.
Another way is to add an select option and let us choose from the frontend, this would be the best way.

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

(or)
If your pull request contain incompatible change, you should also add it to docs/docs/en/guide/upgrede/incompatible.md

@Chris-Arith can you please add some UT to getReaderConfigList function?

And BTW, CI if faild


When using nerdctl to push an image, the size of the image being pushed is not displayed.

Description
When executing nerdctl push on the host machine, the displayed image size remains only a few KiB, even though the actual image size is in GiB. It has been confirmed that there is no caching in Harbor, and the push operation lasts several hundred seconds.（containerd 1.6.2 nerdctl 1.7.6）.
nerdctl -n k8s.io push harbor.xxxx.com/xx:1.0
INFO[0000] pushing as a reduced-platform image (application/vnd.docker.distribution.manifest.v2+json,

manifest-sha256:480cd7d4c792f9d245f7f62d3d5bb279d03ab68aa575fc814e5cdfb48a4abf3b: waiting
config-sha256:e78b09f8d7681ae80c81d317d1fef7f9222efc75b5a5e5da455c9578c2ed9896:   waiting
elapsed: 386 s                                                                    total:   17.1K (4.3 KB/s)

I then looked at this issue and tested with updated versions of containerd and nerdctl（containerd 1.6.33 nerdctl 2.0.0-beta.5）. This time, on the host machine, the pushed size displayed as 0. Additionally, the progress bar does not update correctly.
nerdctl -n k8s.io push harbor.xxxx.com/xx:1.0
INFO[0000] pushing as a reduced-platform image (application/vnd.docker.distribution.manifest.v2+json,

manifest-sha256:480cd7d4c792f9d245f7f62d3d5bb279d03ab68aa575fc814e5cdfb48a4abf3b: waiting        |--------------------------------------|

config-sha256:e78b09f8d7681ae80c81d317d1fef7f9222efc75b5a5e5da455c9578c2ed9896:   waiting        |--------------------------------------|
elapsed: 326 s                                                                    total:   0.0B (0.0 B/s)

Additionally, the push speed inside the container is also constantly 0.
Steps to reproduce the issue

Push the image on the host machine.

Describe the results you received and expected
Expected Result: To obtain the correct push progress, push size, and push speed.
Actual Result: The obtained push size and speed are both 0, and the progress is not updated.
What version of nerdctl are you using?
1.7.6
Are you using a variant of nerdctl? (e.g., Rancher Desktop)
None
Host information
No response

Confirmed.

Hello I am having the same issue. Have you been able to solve it?

@javier99nl I do not think there is a "workaround". It needs to be fixed.


[Question] Is there any way to update entry in watch mode?

We have a project which has more than 100 pages, and each page has a javascript file in webpack's entry. When start webpack, it takes 36 seconds to compile all files. So if there any way to update entry in watch mode? For example, add and compile only a little entries in the beginning, when pages visited or files changed, add the related file into entry and compile them. Is this possible at the moment? Or if we should change the way to compile the files? Thank you!

Have you tried require.ensure() ? http://webpack.github.io/docs/code-splitting.html  It would be interesting to see if there are performance benefits to ensure, vs. multiple entry points in config.

I recommend refactoring to use require.ensure as @adjavaherian proposed. It's just a guess but incremental builds would likely work way more efficiently instead of an entry point for each page.


Consider adding an end-to-end image test of the text layout/UI

In #420, we decided to go with image testing just the visualizer canvas, for the sake of not having to worry about text layout reproducibility in testing specimens. There was definitely an issue of layout differing in different situations, that seemed to be that the headless firefox/chromium on Glen's machine was not using the proper fonts or at least font metrics, whereas those on Kate/Aaron's machines were. Should we track this down and control it tightly enough to have at least one or more text layout/UI image tests? (Note that we do test that various interactions work.)

Well, here's a very likely source of my problems. If I turn on console output in the "Polyfactors" featured gallery test, I see among the messages:
[chromium] ...
Failed to decode downloaded font: http://localhost:5050/src/assets/fonts/inter/Inter-VariableFont_slnt,wght.ttf
OTS parsing error: invalid sfntVersion:<PHONE_NUMBER>
Error: Unsupported OpenType signature <!DO
    at Va (http://localhost:5050/assets/index-BO37ah4e.js:86:265557)
    at http://localhost:5050/assets/index-BO37ah4e.js:86:268798
    at z.onload (http://localhost:5050/assets/index-BO37ah4e.js:86:264548) /src/assets/fonts/inter/Inter-VariableFont_slnt,wght.ttf
...
[firefox] ...
[JavaScript Error: "downloadable font: rejected by sanitizer (font-family: "Inter" style:normal weight:400 stretch:100 src index:0) source: http://localhost:5050/src/assets/fonts/inter/Inter-VariableFont_slnt,wght.ttf"]
...
Error /src/assets/fonts/inter/Inter-VariableFont_slnt,wght.ttf
...

And it turns out that to get the Histogram test to pass, I will need to fix this issue, so it might just fix the font non-matching problem altogether.

Aha, I found the issue with locating font resources in the build version of numberscope, and have fixed it in a commit about to come to #420. (Makes me wonder how this was possibly working for Kate/Aaron, but whatever...) So bumping this up to alpha because we should be able to deal with it. In fact, I have left one end-to-end test with a full snapshot to make sure this is working.

In the form it was eventually merged, #420 indeed included such a test, so closing.


Extract plugin functions into utility methods and code style tweaks

Just a continuation of https://github.com/wulkano/kap/pull/255 with some code style fixes.

Looks good :)


RFC: Mnemonic Ternary Seed

Rendered
Example Go implementation in wollac/iota-crypto-demo:

Conversion from mnemonic to seed: pkg/bip39/bip39.go
Example: examples/mnemseed

merged due to TIP refactor


Avoid globals

see https://scrutinizer-ci.com/g/svandragt/carbon/issues/develop/files/public/application/ControllerAdmin.php?orderField=path&order=asc&honorSelectedPaths=0#inspectioncomment-61413452

$app only


Styled-components errors with React 16 + Typescript

Version
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Reproduction
Steps to reproduce

Upgrade to React 16
Create the following code

import React from 'react';
import styled from 'styled-components';

interface Props {
  className?: string;
  text: string;
}

class Text extends React.Component<Props, any> {
  public render() {
    const { className, text } = this.props;
    return {text};
  }
}

export default styled(Text)`
  color: blue;
`;

Expected Behavior
Component compiles and works
Actual Behavior
Get the following error:
TS2605: JSX element type 'Component<ThemedOuterStyledProps<Props, any>, ComponentState>' is not a constructor function for JSX elements.

This was caused by yarn cache. Not relevant.

@itsthatguy, can you explain, what did you mean about 'yarn cache'?
I get same trouble.

rm -rf node_modules && yarn cache clean && yarn

So sad.
I use<EMAIL_ADDRESS>and as it say, it keeps cache 'guaranteed OK'.

I'm having the same issue, but clearing the Yarn cache didn't fix it. How'd you determine the cause @mxstbr / @itsthatguy ?

@edude03 I had a similar issue, I had to make sure that I was only using one version of the react typings (I was having both react 16 and 15 typings)
HTH

getting close to 2019, still seeing this issue without a clear explanation on why it happens and an accepted solution for fixing it.

Ask on DT. None of the core team uses typescript
On Thu, Nov 15, 2018 at 6:07 AM Dan Ochiana<EMAIL_ADDRESS>wrote:

getting close to 2019, still seeing this issue without a clear explanation
on why it happens and an accepted solution for fixing it.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/styled-components/styled-components/issues/1255#issuecomment-439018665,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAiy1o7pYTtkk_Nh1qzWdbOSVRfw3aCgks5uvVkbgaJpZM4P_oeH
.

<EMAIL_ADDRESS>fix this for me

Ask on DT. None of the core team uses typescript

I might have to fork this repo then.


test_path folder structure

Hi there! Thanks for great research work!
According to readme files there is test_path folder expecting to have input1 and input2 folders with JPG files inside. I've created testing folder right this:
./testing
-- input1
---- test1.jpg
-- input2
---- test2.jpg
with files from Carpark-DHW folder.
Trying to run warp with modified testing folder path. Script is fetching images but failing with following error:
##################start testing#######################
Traceback (most recent call last):
  File "C:\Sources\UDIS2\Warp\Codes\test.py", line 123, in <module>
    test(args)
  File "C:\Sources\UDIS2\Warp\Codes\test.py", line 67, in test
    batch_out = build_model(net, inpu1_tesnor, inpu2_tesnor, is_training=False)
  File "C:\Sources\UDIS2\Warp\Codes\network.py", line 128, in build_model
    H_motion, mesh_motion = net(input1_tensor, input2_tensor)
  File "C:\Users\assol\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.9_qbz5n2kfra8p0\LocalCache\local-packages\Python39\site-packages\torch\nn\modules\module.py", line 1501, in _call_impl
    return forward_call(*args, **kwargs)
  File "C:\Sources\UDIS2\Warp\Codes\network.py", line 487, in forward
    offset_1 = self.regressNet1_part2(temp_1)
  File "C:\Users\assol\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.9_qbz5n2kfra8p0\LocalCache\local-packages\Python39\site-packages\torch\nn\modules\module.py", line 1501, in _call_impl
    return forward_call(*args, **kwargs)
  File "C:\Users\assol\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.9_qbz5n2kfra8p0\LocalCache\local-packages\Python39\site-packages\torch\nn\modules\container.py", line 217, in forward
    input = module(input)
  File "C:\Users\assol\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.9_qbz5n2kfra8p0\LocalCache\local-packages\Python39\site-packages\torch\nn\modules\module.py", line 1501, in _call_impl
    return forward_call(*args, **kwargs)
  File "C:\Users\assol\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.9_qbz5n2kfra8p0\LocalCache\local-packages\Python39\site-packages\torch\nn\modules\linear.py", line 114, in forward
    return F.linear(input, self.weight, self.bias)
RuntimeError: mat1 and mat2 shapes cannot be multiplied (1x3840 and 4096x4096)

What I'm doing wrong?

The test.py is used for evaluating the UDIS-D dataset. It's unsuitable for other images with other resolutions (not 512*512).
Please try test_output.py or test_other.py following the readme.md instead.


Transporters

What is this PR trying to achieve?
Fixes #322 and #323
What does it look like?

For interface components

[x] Browser testing - Have you checked the component in our supported browsers
[x] No javascript - Have you checked the component with javascript disabled
[x] A11y testing - Have you run pa11y against the component and fixed/verified all errors, warnings and notices?

Really nice work - I mentioned starting to use JS config, that way we're putting the model work into these PRs too.
What do you think?


Idea: Allow to execute plain sql files instead of php migration

What do you think about the following idea: Some users may not need the undo feature, so it  would suffice for them to have the ability to store and execute plain sql files?
Detailed suggestions:

Typing phinx create MyNewMigration.sql would create a YYYYMMDDHHMMSS_my_new_migration.sql file
The phinx rollback command should stop when reaching a sql file
It should be allowed to mix php and sql migrations within one project

Georg

Still not resolved?

There's nothing to resolve here, this is a feature request, not a bug report.

I've figured out some kind of workaround of this problem.

$options = $this->adapter->getOptions();
exec("mysql -u {$options['user']} -p{$options['pass']} < ".str_replace(".php", ".sql", __FILE__));

will execute pure SQL from file named just like migration file only with .sql extension.
It's far away from being perfect but it allows to bypass PDO and create triggers and procedures.
Use at your own risk.

Using the --class option, I have a ByDirectory and ByDirection class that does just that. ByDirection is expecting an up.sql/down.sql, ByDirectory is expecting an up and down directory with the contents being SQL files executed in alphabetic order (so 01 - Drop Table.sql, 02 - Create Table.sql sort of thing).
By having individual SQL files, you can also get around the issue of a failure in a batch of SQL statements not being able to be captured as an exception. I think this is a PDO/Mysql issue and not Phinx.
Anyhow.
Send my your email and I'll send you what I can.

@zacek I just did a test and it seems that multiple queries have been executed. You are right that there are still limitations with advanced features in SQL files but @dkmuir solution may work for most use cases as a workaround.
Anybody knows why this works?
Query:
$this->execute('CREATE TABLE `ph0` (
            `id` int(10) NOT NULL,
            `patterns` text,
            PRIMARY KEY (`id`)
        ) ENGINE=InnoDB DEFAULT CHARSET=utf8;
        CREATE TABLE `ph1` (
            `id` int(10) NOT NULL,
            `patterns` text,
            PRIMARY KEY (`id`)
        ) ENGINE=InnoDB DEFAULT CHARSET=utf8');

And then:
mysql> select * from ph0;
Empty set (0.00 sec)

mysql> select * from ph1;
Empty set (0.00 sec)

Closing due to lack of activity or out of scope. If there is new interest, please open a PR.


TSCBasic: honour workingDirectory on Windows' Process management

If workingdirectory is set, ensure that it is honoured when creating the process.  This repairs a previous test failure in SPM.

@swift-ci please test


Merge/release/2.1 to release/2.2

Notably, includes @halter73's change to merge BufferWriter and CountingBufferWriter and #2717.
See also https://github.com/aspnet/Common/pull/390

Are test failures on VSTS a known issue? Tests in VSTS failed for Windows and Linux.

:bell:

It looks like Kestrel has only passed once on VSTS in the last few weeks. I'm going to just disable that PR check since something is clearly going wrong.

Wait.


Feature: adding Erosion to the filtering operations

Fixes #19

Please remove the unnecessary project files from the PR.


Improve team settings workflows

Key a bunch of stuff on team ID, and generally clean stuff up.

might be nice to change the team waiting key so you don't need to plumb so much state everywhere just to get the name

might be nice to change the team waiting key so you don't need to plumb so much state everywhere just to get the name

I was planning to change it eventually when we reach a tipping point; do you think it's reached?

doesn't have to be a part of this pr but it'll allow you to go back to chainAction etc

Bump @chrisnojima


chore(deps): update go to v1.16

update go to v1.16
update action workflow config

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

Thanks @chenrui333


quickselect algoithm

Description
Describe more about the issue.
Quickselect is a selection algorithm to find the k-th smallest element in an unordered list. It is related to the quick sort sorting algorithm.The difference is, instead of recurring for both sides (after finding pivot), it recurs only for the part that contains the k-th smallest element.
Programming language

[x] Python

Are you contributing under any open-source program ?

hacktoberfest

could you please assign me this issue


Dev ndrey

added verifying for email and phone

Codecov Report

Merging #71 (21aff80) into dev (be624ba) will decrease coverage by 2.91%.
The diff coverage is 82.17%.

@@            Coverage Diff             @@
##              dev      #71      +/-   ##
==========================================
- Coverage   88.38%   85.47%   -2.92%
==========================================
  Files          75       76       +1
  Lines        3917     4006      +89
  Branches      188      197       +9
==========================================
- Hits         3462     3424      -38
- Misses        430      552     +122
- Partials       25       30       +5

Impacted Files
Coverage Δ

src/DucksNet.API/Controllers/BusinessController.cs
0.00% <0.00%> (ø)

src/DucksNet.API/Controllers/UsersController.cs
0.00% <0.00%> (ø)

.../DucksNet.Infrastructure/Sqlite/DatabaseContext.cs
0.00% <0.00%> (ø)

src/DucksNet.API/Controllers/PetsController.cs
17.30% <25.00%> (ø)

src/DucksNet.API/Controllers/OfficeController.cs
35.13% <57.14%> (ø)

...DucksNet.API/Controllers/AppointmentsController.cs
86.20% <75.00%> (ø)

...rc/DucksNet.API/Controllers/TreatmentController.cs
61.90% <75.00%> (ø)

src/DucksNet.API/Controllers/MedicineController.cs
61.11% <80.00%> (ø)

...frastructure.Prelude/AppointmentScheduleService.cs
66.66% <83.33%> (ø)

src/DucksNet.Domain/Model/Business.cs
95.91% <91.66%> (-4.09%)
:arrow_down:

... and 17 more

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


uibDatepickerPopup directive doesn't set init value

Bug description:
The initial value is not set if the element is like input[type='button'] because uibDatepickerPopup only set value when date is selected: popup.js#L215
Plunker
workaround 1: add ngValue diredtive Plunker
workaround 2: create a custom directive Plunker
app.directive('uibDatepickerPopup', ($timeout) => {
  return {
    restrict: 'A',
    require: 'ngModel',
    link: (scope, element, attrs, ngModelController) => {
      if (attrs.type !== 'button') {
        return;
      }

      $timeout(() => {
        element.val(ngModelController.$viewValue);
      });
    }
  };
});

Link to minimally-working plunker that reproduces the issue:
https://plnkr.co/edit/gsPj6jiXDgKHQQqXpUfL?p=preview
Version of Angular, UIBS, and Bootstrap
Angular: 1.5.8 or lower
UIBS: 2.0.0 or lower
Bootstrap: 3.3.7 or lower

Why is it being used as an input type button? Is this to avoid allowing keyboard modification of the date? On first impression, this does not seem like a great usage of the datepicker.

I cannot seem to set the init-date dynamically, which had been working for us previously. Now if I try to set the init date to something like
// HTML
<uib-datepicker(ng-model="dateObj", init-date="initDate"...>
// and in the JS for my directive
$scope.initDate = new Date($scope.minDate.getTime()); // say the selected minDate is Nov 12, 2016
In this example, it loads the calendar to November, but the month change buttons don't respond unless you click on a date or click the month name button first. I'm not sure why this just started happening, as it had been working as I had it for many months, and we had not updated the app in quite a while.

I did an investigation, and the reason it does not render is because Angular does not set an ngModelCtrl.$render function for inputs with type button. See here for the right way to get this to work.


Is there a way to permamently hide menu bar?

After I configured options I don't need it anymore and it looks bad when I'm using dark theme. Also I'm using MfD on my Windows 10 tablet and I want to achieve full-screen only messenger look. When I'm toggling full screen app remembers about it when I open it another time, but when I'm disabling menu bar it appears again every time I start Messenger.

Operating system: Windows 10 Home
Messenger for Desktop version: 2.0.4

Yes, Fixed in 2.0.7 (will be released soon)


.pylintrc not being detected / weird behavior

I have a project setup and open in SublimeText 3 along with this linter plugin. It works fine.  It detects the linting issues normally.
However, I want to make it detect my custom linting rules described in my .pylintrc file situated in the same project directory, so I provided the absolute path to it in the .sublimelinterrc file.
The problem is that it's not being detected at all. The linter detects only the default config!
My .sublimelinterrc looks like this:
{
  "@python": 2.7,
  "linters": {
    "pylint": {
      "@disable": false,
      "args": [""],
      "disable": "",
      "enable": "",
      "excludes": [],
      "paths": [],
      "rcfile": "/absolute/path/to/project/.pylintrc",
      "show-codes": false
    },
   ...

The command line ran in the console is the following:
SublimeLinter: pylint: project.py ['/usr/bin/python', '/usr/bin/pylint', "--msg-template='{line}:{column}:{msg_id}: {msg}'", '--module-rgx=.*', '--reports=n', '--persistent=n']

That is clearly states two things:

It's not considering the .pylintrc in the config, which results in a bunch of errors, a score of -4/10. (running the same command from terminal on the same file I get a 10/10)
It's using the python2.7 interpreter.

Does this plugin consider per project .sublimelinterrc configurations?

I did another test; I created an ill-written res.py in the project root directory.
msg = {

With the exact same configuration as before, the cmd run in the console is the following:
SublimeLinter: pylint: res.py ['/usr/bin/python2.7', '/usr/bin/pylint', "--msg-template='{line}:{column}:{msg_id}: {msg}'", '--module-rgx=.*', '--reports=n', '--persistent=n', '', '--rcfile=/absolute/path/to/project/.pylintrc']

Reading that I understand two things:

It's using the python2.7 interpreter
It's considering the the .pylintrc provided in the .sublimelinterrc config

But then, in the console, there's neither a pylint output after the cmd nor a linter tooltip showing in sublime :/

The two situations confuse me 😕 Please help me if I'm getting something wrong?

Does this plugin consider per project .sublimelinterrc configurations?
This plugin uses the settings as made available by SublimeLinter, nothing specific to the plugin ..

This plugin is one of the rare plugins, if not the only one, that lints python code by running a command, not importing and running code. Which is why it still has trouble running pylint with the proper python version.
There might be a difference between background linting (when you change the code without save: sublimelinter copies the content to a tmp file and runs the linter) and linting after save (which uses the real file). There might be a bug in the settings that these cases use..

I don't really have the time to actively support this plugin, so I'm happy if people can provide good improvements !

I don't really have the time to actively support this plugin, so I'm happy if people can provide good improvements !
I understand, thanks though :)

I tried both cases, background and on-save, but the problem remains.
For the record, I believe there's a problem in applying the settings.
I added the link to .pylintrc to the user config instead of .sublimelinterrc (not really convenient for multiple projects), the linting works for one sub-directory (addons) .. but falls back to the default for the other one (bin) and the project root.

Not a promise, but I'll try to make some time to learn about STL plugins dev and contribute a solution.

Do you have a ~/.pylintrc file?
What's in your /absolute/path/to/project/.pylintrc file?
What happens when you attempt to run the following via the command line?

'/usr/bin/python2.7' '/usr/bin/pylint' "--msg-template='{line}:{column}:{msg_id}: {msg}'" '--module-rgx=.*' '--reports=n' '--persistent=n' '--rcfile=/absolute/path/to/project/.pylintrc' res.py

@IOAyman wrote [emphasis added]:

… With the exact same configuration as before, the cmd run in the console is … .

Can you verify this assertion?

I can confirm that adding an explicit SublimeLinter → linters → pylint → rcfile entry to my .sublime-project file works as expected for me. My config:
// ---- …/project/project.sublime-project ----
{
    "SublimeLinter": {
        "linters": {
            "pylint": {
                // Don't use ~/.pylintrc; show everything
                "rcfile": "/dev/null"
            }
        },
        "paths": {
            "linux": [],
            "osx": [
                "/…/.virtualenvs/project/bin"
            ],
            "windows": []
        }
    },
    "folders": [
        {
            "follow_symlinks": true,
            "path": "."
        }
    ],
    "settings": {
        "python_interpreter": "/…/.virtualenvs/project/bin/python",
        "suppress_python_not_found_error": false
    }
}

// ---- …/project/.sublimelinterrc ----
{
    // https://github.com/SublimeLinter/SublimeLinter-pylint/issues/37
    "@python": "…/.virtualenvs/project/bin/python"
}

However, if I remove that entry from my .sublime-project file and add a linters → pylint → rcfile entry to my .sublimelinterrc file in my project directory, it appears to have no effect (i.e., having solely a .sublimelinterrc entry does not result in an added --rcfile=… command line argument). My config:
// ---- …/project/project.sublime-project ----
{
    "SublimeLinter": {
        "linters": {
        },
        "paths": {
            "linux": [],
            "osx": [
                "/…/.virtualenvs/project/bin"
            ],
            "windows": []
        }
    },
    "folders": [
        {
            "follow_symlinks": true,
            "path": "."
        }
    ],
    "settings": {
        "python_interpreter": "/…/.virtualenvs/project/bin/python",
        "suppress_python_not_found_error": false
    }
}

// ---- …/project/.sublimelinterrc ----
{
    // https://github.com/SublimeLinter/SublimeLinter-pylint/issues/37
    "@python": "…/.virtualenvs/project/bin/python",
    "linters": {
        "pylint": {
            // This doesn't seem to work here
            "rcfile": "/dev/null"
        }
    }
}

One observation: It looks like your command line with an explicit --rcfile has an extra blank argument that I can't reproduce on my end:
SublimeLinter: pylint: res.py ['/usr/bin/python2.7', …, '--persistent=n', '', '--rcfile=/absolute/path/to/project/.pylintrc']
                                                                          ^^^
                                                                   Why is this here?

What versions of SublimeLinter and SublimeLinter-pylint are you using?

Same issue here on the current version of SublimeLinter.


WSL now supports native systemd

Problem
WSL now supports systemd. ref. however, enabling it is not possible with current workaround of NixOS. I was thinking maybe using it could benefit performance.
Solution
enabling it is by adding [boot] systemd=true to /etc/wsl.conf

There already is the wsl.nativeSystemd option to enable it. Just editing wsl.conf would break your system, as there is no systemd binary in the place WSL looks for by default. If nix complains that the option does not exist, you might need to update by putting a fresh copy of this repo into /etc/nixos/nixos-wsl.
I'll close the issue but not lock it, so feel free to comment if you have any questions


Definition of callbacks

We need to define the callbacks, currently there's just everywhere callback, but no signature.

Closing as #35 has been merged.


Add support for escape chars in text tokens

{}<>" chars are treated by lexer as transition symbols. I would like to have a way to somehow escape those

The commit 9da8720720345155fb31824eca16cd532914134c solves this issue


Unable to fetch User Profile properties using rest api in spfx webpart

Thank you for reporting an issue or suggesting an enhancement. We appreciate your feedback - to help the team to understand your needs, please complete the below template to ensure we have the necessary details to assist you.
Category

[ ] Question
[X ] Bug
[ ] Enhancement

Hi Team,
I am creating a webpart using react and typescript which needs to fetch the country (EDSCountry) of currently logged in user. below is my code but whenever i try to debug this in the local workbench it throws access denied error. However, when i fire the url in browser i do get the xml response properly.
I also followed this article but in the last step when this has to be published it is also throwing error.
https://docs.microsoft.com/en-us/sharepoint/dev/general-development/create-a-workflow-with-elevated-permissions-by-using-the-sharepoint-workflo
Can you please provide pointers as to why i am having this issue.
private _getUserLocale() : Promise {
let url: string = this.context.pageContext.web.absoluteUrl+"/_api/SP.UserProfiles.PeopleManager/GetMyProperties";
console.log(url);
return (fetch(url)
.then(function(response){debugger;
return response.text();
})
.then(function(data){
console.log(data);
return;
}));
}

Hey Team,
Can anyone provide response to this question as well.

@ssingh28 are you trying the exact same URL in browser or a different one? Looking at the error message, you're trying to call the API on the Microsoft tenant which won't work unless you're a Microsoft employee and have access to the tenant.

@waldekmastykarz  yes i am an employee of microsoft and have tried this same url in the browser and it works fine there.

@ssingh28 thanks for confirming! When you enter the URL in browser: are there any redirects or are you getting a response immediately? You can double check that with developer tools and persisted logging enabled.

We closed this issue as it had not activity within last 180 days. This is a generic process we have decided to perform for issues, which have not been explicitly marked still to be "work in progress" based on tags. We are performing this cleaning to make sure that old issues that have already been solved (but not closed) or are no longer relevant are cleaned out and make the issues more manageable. If this issue still valid, we would ask you to open a new issue and follow the guidance in the issue template related on the recommended location. We do apologize any inconveniences this might cause. Please do remember that issues in the issue lists are also messages for others in the community, so you can also check if you can assist on any of them. “Sharing is caring!”