[azeventhubs] Updating changelog with release date for tomorrow.

Simple fix.

(There's a pending retry fix that I'll port in before releasing)


Fix Clippy warnings seen with Rust 1.59 beta

New Rust versions come with new Clippy versions, that can add new lints or adjust existing lints.
Locally I'm running Rust 1.59 beta, and it reports a few new Clippy warnings, that this PR resolves:
warning: missing `#[must_use]` attribute on a method returning `Self`
  --> libcnb-data/src/build_plan.rs:48:5
   |
48 | /     pub fn provides(mut self, name: impl AsRef<str>) -> Self {
49 | |         self.current_provides.push(Provide::new(name.as_ref()));
50 | |         self
51 | |     }
   | |_____^
   |
   = note: `#[warn(clippy::return_self_not_must_use)]` on by default
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#return_self_not_must_use

warning: missing `#[must_use]` attribute on a method returning `Self`
  --> libcnb-data/src/build_plan.rs:53:5
   |
53 | /     pub fn requires(mut self, name: impl AsRef<str>) -> Self {
54 | |         self.current_requires.push(Require::new(name.as_ref()));
55 | |         self
56 | |     }
   | |_____^
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#return_self_not_must_use

warning: `libcnb-data` (lib) generated 2 warnings
warning: single-character string constant used as pattern
   --> libcnb-cargo/src/lib.rs:206:47
    |
206 |     buildpack_descriptor.buildpack.id.replace("/", "_")
    |                                               ^^^ help: try using a `char` instead: `'/'`
    |
    = note: `#[warn(clippy::single_char_pattern)]` on by default
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#single_char_pattern

I've added an extra commit that isn't necessary for Clippy, but cleans up the usages of #[must_use] by favouring adding them to the struct of builders rather than having to add them to every single method on the struct.


Crash when trying to upload in background

I'm trying to upload my data on server and my application crashes, this line of code performed normally when i use defaultSessionManager, just fix it or tell me how to upload data when my app is in background state

You'll need to provide us with more details for us to do anything here. What version of Xcode, Alamofire, and OS are you running this on? What is the content of the exception that was thrown there?

8.3.2, swift 3, alamofire 4.4.0, all information about exception that I have is on screenshots. iOS version 10.2.1

Hi @OldCrab,
There's not enough info to go on here. We'll need a sample project that can demonstrate the issue and then we'll happily look into it further. I'm going to close this issue out until we get enough info to action an investigation or a sample project.
Cheers. 🍻

Retaining you SessionManager instance might work.

@OldCrab hi I'm having the same issue. have you found a solution for this?


remove participants.number

#3647 takes it out of the UI.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/25040405-remove-participants-number?utm_campaign=plugin&utm_content=tracker%2F85909&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F85909&utm_medium=issues&utm_source=github).

Quick search finds it at https://github.com/gratipay/gratipay.com/blob/master/gratipay/models/account_elsewhere.py#L127, but I'm not sure what that section even does.


Feature to initiate calling or dialing a number

This patch shall allow user to initiate a call from QML  using makeCall
(1) makeCall takes two arguments 'number' and 'intent'
(2) It allows to choose the intent ACTION_CALL or ACTION_DIAL
via 'intent' parameter possible values: 'call' or 'dial'

shall be re-submitted after proper code-formatting.


Not able to update tag with tag contributors role

Describe the bug
We are trying to update / add tag to resource group using a service principal. While using PowerShell it is working as expected. But when we are trying to update tag via JAVA sdk we are getting following error
The client 'f73daaab-9eaf-484d-83e9-b22ecbf00102' with object id 'f73daaab-9eaf-484d-83e9-b22ecbf00102' does not have authorization to perform action 'Microsoft.Resources/subscriptions/resourcegroups/write' over scope '/subscriptions/be1bsdfnb-fhfd-4ce7-b323-6732ecc0987/resourcegroups/test-RG' or the scope is invalid. If access was recently granted, please refresh your credentials.
To Reproduce

Create a service principal
Attach tag contributor
Run java code to update resource group

Command used
az group update --resource-group test-RG --set tags.CostCenter='{"Dept":"IT","Environment":"Test"}'
Environment summary
az --version
azure-cli                          2.9.1
command-modules-nspkg              2.0.3
core                               2.9.1
nspkg                              3.0.4
telemetry                          1.0.4
Python location '/usr/local/Cellar/azure-cli/2.9.1/libexec/bin/python'
Extensions directory '/Users/akashjohn/.azure/cliextensions'
Python (Darwin) 3.8.5 (default, Jul 21 2020, 10:48:26)
[Clang 11.0.3 (clang-1<IP_ADDRESS>)]
Legal docs and information: aka.ms/AzureCliLegal
Your CLI is up-to-date.
Please let us know how we are doing: https://aka.ms/azureclihats
and let us know if you're interested in trying out our newest features: https://aka.ms/CLIUXstudy

add to S174

@johnakash In fact, there are two types of commands to manage tags:

Based on the respective Resource Provider
Such as az resource tag and az group update ... --set tags.xxx , it seems that at the moment each RP has its own permission control and does not get through with the Tag Contributor role.
Based on ARM service
Such as az tag, this command can also support tagging resources, customers with Tag Contributor role should be able to tag resources by using this command.

So there are two kinds of solutions:

Use command az tag to tag resources with role Tag Contributor
For example, you can use:

az tag create --resource-id {resource-group-id} --tags {tags}

If you want to use the original command az group update ... --set tags.xxx, you need to assign a role with Microsoft.Resources/subscriptions/resourcegroups/write permission to this service principal.

@zhoxing-ms  - I tried the option which you provided but it throw an error as below,
az tag create --resource-id essAstPrdWu2RgSecurity --tags {"Dept":"IT"}
The request did not have a subscription or a valid tenant level resource provider.

We passed the subscription id as well to the command execution and we are getting the same response,
az tag create --subscription d9b9kdc9-4z2a-4368-bfe3-13e88044d5d4 --resource-id essAstPrdWu2RgSecurity --tags {"Dept":"IT"}
The request did not have a subscription or a valid tenant level resource provider.

@johnakash In fact, the parameter --resource-id needs to be passed in a complete resource ID, not the name of the resource (Because it supports different scopes).
If you need to tag a resource group, you can refer to the example in the help of command az tag create -h:
Create or update the entire set of tags on a resource group.
    az tag create --resource-id /subscriptions/{sub-id}/resourcegroups/{rg} --tags Dept=Finance
    Status=Normal

Because there is no reply to this question for a long time, I will close it temporarily. If you have any questions, please feel free to let us know.


lassosum.pipeline.R has hardcoded paths

Line 211,
lassosum(cor=cor2, bfile="./data/refpanel",

this prevents the pipeline being run without the refpanel being in that exact path.

Thanks for spotting this.


Path stream processing

This patch contains the core of the path stream processing, though some
integration bits are missing. The core logic is tested, though
combinations of path types, transforms, and line widths are not (yet).
Progress towards #119

Now passes on a bunch of test GPUs, including mac M1.


test-branch/execute-job/quoted-authorization/8b9e150e-f709-4956-bc41-9062615d1b11

PR for testing 2148330efb03985e1dd6c5b406795c5e0781a923

/canonical/self-hosted-runners/run-workflows 2148330efb03985e1dd6c5b406795c5e0781a923


We don't test DSA key generation enough

I just tried #11303 with the faulty dsa_gen_set_template merged with the latest #10289, and was astonished how many tests succeed.  With only test_ssl_old failing, that's not a lot of test coverage for DSA key generation, and when that recipe i gone, there will be none.
I just had a quick search, and this is what I found:
$ grep KeyGen test/recipes/30-test_evp_data/*
test/recipes/30-test_evp_data/evppkey.txt:KeyGen = rsaEncryption
test/recipes/30-test_evp_data/evppkey.txt:KeyGen = RSASSA-PSS
test/recipes/30-test_evp_data/evppkey.txt:KeyGen = dhKeyAgreement
$ git grep EVP_PKEY_keygen test
test/cmp_testlib.c:               && TEST_int_gt(EVP_PKEY_keygen_init(ctx), 0)
test/cmp_testlib.c:               && TEST_int_gt(EVP_PKEY_keygen(ctx, &pkey), 0));
test/evp_extra_test.c:    if (!TEST_true(EVP_PKEY_keygen_init(kctx)))
test/evp_extra_test.c:    if (!TEST_true(EVP_PKEY_keygen(kctx, &pkey)))
test/evp_test.c:    if (EVP_PKEY_keygen_init(genctx) <= 0) {
test/evp_test.c:    if (EVP_PKEY_keygen(keygen->genctx, &pkey) <= 0) {
$ git grep genpkey test
test/certs/mkcert.sh:            openssl genpkey "${args[@]}" -out "${key}.pem"
test/evp_pkey_provided_test.c:     * openssl genpkey -paramfile dhp.pem | openssl pkey -text
test/recipes/15-test_genrsa.t:    is(run(app([ 'openssl', 'genpkey', '-out', 'genrsatest.pem',
test/recipes/15-test_genrsa.t:        $fin = run(app([ 'openssl', 'genpkey', '-out', 'genrsatest.pem',
test/recipes/15-test_genrsa.t:ok(run(app([ 'openssl', 'genpkey', '-algorithm', 'RSA',
test/recipes/15-test_genrsa.t:   "genpkey -3 $good");
test/recipes/15-test_genrsa.t:ok(run(app([ 'openssl', 'genpkey', '-algorithm', 'RSA',
test/recipes/15-test_genrsa.t:   "genpkey -f4 $good");
test/recipes/15-test_mp_rsa.t:            ok(run(app([ 'openssl', 'genpkey', '-out', "rsamptest-$name.pem",
test/recipes/80-test_ssl_old.t:my @genpkeycmd = ("openssl", "genpkey");
test/recipes/80-test_ssl_old.t:                    ok(run(app([@genpkeycmd, "-out", $Dkey,
test/recipes/80-test_ssl_old.t:                    ok(run(app(["openssl", "genpkey", "-genparam",
test/smime-certs/mksmime-certs.sh:$OPENSSL genpkey -genparam -algorithm DH -pkeyopt dh_paramgen_type:2 \
test/smime-certs/mksmime-certs.sh:$OPENSSL genpkey -paramfile dhp.pem -out smdh.pem
$ git grep gendsa test
$

At the very least, I would have expected one or two KeyGen stanza for DSA among the files in test/recipes/30-test_evp_data/.  That would be a good start.

Could be because its pretty slowwwww.

Could be because its pretty slowwwww.

That's a really poor excuse.  one test for evp_test won't hurt us that much.

Yep - it wont be hard to add - will do when I sort out this FFC stuff.

Yep - it wont be hard to add - will do when I sort out this FFC stuff.

Cool!

So, er, this should be resolved by now, shouldn't it?

Yes - time to close this


Move to float64 throughout Parcels?

Early on in the design of Parcels (pre-v2), we decided to do most computations in float32. With the development of C-grid interpolation in v2, we required float64 accuracy to properly deal with boundary conditions. Still, we left a lot of float32 support under the hood in Parcels
With the recent release of NumPy v2, some of these float32-implementations seem to be breaking some of the unit tests. This probably has to do with this change to data type promotion in NumPy. See also https://github.com/OceanParcels/parcels/pull/1603#issuecomment-2238974528.
This has been quick-fixed in https://github.com/OceanParcels/parcels/pull/1603/commits/20206c23e7f993d6f846c47ca1bdbf9b5f325d15, but ideally, we may want to reconsider the use of float32 in Parcels altogether.
This Issue is a place to discuss/explore what would happen if we move fully to float64 in Parcels

I think it would be good to go ahead with this migration.


got matched, but didn't get triggered because the trigger rules were not satisfied

Hello all,
I'm running the following hook :
[
  {
    "id": "play",
    "execute-command": "/test.sh",
    "command-working-directory": "/",
    "trigger-rule":
    {
      "match":
      {
        "type": "value",
        "value": "media.play",
        "parameter":
        {
          "source": "payload",
          "name": "event"
        }
      }
    }
}
]

But once the payload arrives :
Content-Disposition: form-data; name="payload"
Content-Type: application/json

{"event":"media.play","user":true,"owner":true,"Account" [...]

I doesn't seem to work out so well. I don't get what i'm doing wrong.
Logs give me the following :

[webhook] 2017/10/16 12:01:00 version 2.6.5 starting
[webhook] 2017/10/16 12:01:00 setting up os signal watcher
[webhook] 2017/10/16 12:01:00 attempting to load hooks from play.json
[webhook] 2017/10/16 12:01:00 os signal watcher ready
[webhook] 2017/10/16 12:01:00 found 1 hook(s) in file
[webhook] 2017/10/16 12:01:00   loaded: play
[webhook] 2017/10/16 12:01:00 serving hooks on http://<IP_ADDRESS>:9000/hooks/{id}
[webhook] 2017/10/16 12:01:03 incoming HTTP request from <IP_ADDRESS>:46641
[webhook] 2017/10/16 12:01:03 play got matched
[webhook] 2017/10/16 12:01:03 error parsing form payload invalid URL escape "%\xf0\xc5"
[webhook] 2017/10/16 12:01:03 play got matched, but didn't get triggered because the trigger rules were not satisfied
[webhook] 2017/10/16 12:01:03 200 | 2.027064ms | <IP_ADDRESS>:9000 | POST /hooks/play

Also tried an alternative :
[
  {
    "id": "play",
    "execute-command": "/test.sh",
    "command-working-directory": "/",
        "parse-parameters-as-json":
[
      {
        "source": "payload",
        "name": "event"
      }
],
"trigger-rule":
    {
      "match":
      {
        "type": "value",
        "value": "media.play",
        "parameter":
        {
          "source": "payload",
          "name": "event"
        }
      }
    }
}
]

but htat gets me:

[webhook] 2017/10/16 14:10:59 version 2.6.5 starting
[webhook] 2017/10/16 14:10:59 setting up os signal watcher
[webhook] 2017/10/16 14:10:59 attempting to load hooks from play.json
[webhook] 2017/10/16 14:10:59 os signal watcher ready
[webhook] 2017/10/16 14:10:59 found 1 hook(s) in file
[webhook] 2017/10/16 14:10:59   loaded: play
[webhook] 2017/10/16 14:10:59 serving hooks on http://<IP_ADDRESS>:9000/hooks/{id}
[webhook] 2017/10/16 14:11:23 incoming HTTP request from <IP_ADDRESS>:46880
[webhook] 2017/10/16 14:11:23 play got matched
[webhook] 2017/10/16 14:11:23 error parsing form payload invalid URL escape "%)\u007f"
[webhook] 2017/10/16 14:11:23 error parsing JSON parameters: couldn't retrieve argument for {Source:payload Name:event EnvName:}
[webhook] 2017/10/16 14:11:23 play got matched, but didn't get triggered because the trigger rules were not satisfied
[webhook] 2017/10/16 14:11:23 200 | 1.161037ms | <IP_ADDRESS>:9000 | POST /hooks/play

Hi Again.
the post i'm dealing with seems to be a multipart with Json payload and then come a jpeg.
Would that be a showstopper for adnanh/webhook ? Is there a way to process the Payload only part ?
This is actually a dump of the post received :
Content-Disposition: form-data; name="payload"
Content-Type: application/json

{"event":"media.play","user":true,"owner":true}
------------------------------33d2ecba7af4
Content-Disposition: form-data; name="thumb"; filename="thumb.jpg"
Content-Type: image/jpeg

ÿØÿàJFIFHHÿÛC

 $.' ",#(7),01444'9=82<.342ÿÛC

2!!22222222222222222222222222222222222222222222222222ÿÀqÈÿÄÿÄ7	!1RS¡"3AQq2BCa#$brsÑÿÄÿÄÿÚ?üþUi0¹/-ÑûÉè}ä½tE>ò@CÒaÞH
=2÷$ÇL¤úêH=*{?i4îOÐÑ©5ñgèè´ø³ôK¼ Ðô»Ùúèz]ìý£R»?ô-ö~BÑïgà¢Qï§àÏGÖõõê'ö@e&mâ³í|ÓÐãÕZ§ å£Q]UgèJrÃ«?]cømgsmJ¥KÛÎqM¤£°®ÿ¬mæãÛª8*YT@OrrÃ,Ä)exK;/ZO`1ä	È9ÙÄ`2[ÖÄÀÚ¯pq<÷WÞ`Rrù =-ÕÝÞA?ÉÄ²Ômi%¤º>§ñ¥÷å;e,e-Ä'¸âÒØ&Ó/9°q %u` Ø
------------------------------33d2ecba7af4--

It appears that the JPEG contents is messing up the payload parsing. Is there any way you could base64 encode the JPEG image on the client and send that as a parameter?

I wish i could. Sadly the payload is originating from Plex media server, and this is pretty much locked down. I request from their side that they enable a Json payload only option... but this may take forever. :(

Unfortunately, your only option would be rolling out your own solution to receive the Plex's payload and then either handle the execution yourself, or forward the request with proper payload to the webhook.
Sorry :-(

I understand.
I was wondering though. would there be a way to apply a regex filter on the whole post regardless of this being the payload or anything else. That way i could trigger the actin based on the presence of a specific string. ?

Not really, turns out that your post body contains invalid byte sequence that messes up golang's form parsing library. We can't do anything with that payload :-(

hi @fredguy ,
have you at the meantime found an other solution??? want the same like you.
best regards

hi @gregor-hh
I switch to an alternatie solution locate on the endoint (jeedom home automation). They have a plugin that is called "http remote event" https://jeedom.github.io/documentation/third_plugin/httpRemoteEvent/fr_FR/index.html
and a script that allows me to parse the payload properly.
I guess that won't be much help for you in your case.
GL
Cheers

thank you very much for this tip/hint.
I must think about it, maybe its a bit „oversized“ :)
but it looks, like a cool home automation software, I dont have one, maybe its a new project in the future^^. for now I only want grab the media.play events from plex to the telegram web api (bot) for push notifications...
have a nice day
best regards

Sad it's still unsupported. Got [webhook] 2024/01/22 01:18:43 [62c88a] error parsing form payload invalid URL escape "%9\x1f".
It would be interesting to add a "sanitizer" option that deletes all files attached to a webhook.
I'm using webhook version 2.6.9 (debian)


seeds with columns starting with underscore fail

When trying to seed a csv file where any column name is prefixed with an underscore we encounter a runtime error.
Seems that columns starting with an underscore need to be enclosed in backticks in DDL statements.
Happy to raise a PR to resolve, would it just be a case of backticking column-names here and here?
How to reproduce
try and seed this:
__column_one,column_2
a,1
b,2
c,3

Error:
Runtime Error in seed __test_seed (seeds/test_seed/__test_seed.csv)
  FAILED: ParseException line 2:76 cannot recognize input near '__column_one' 'string' ',' in column specification

👋🏻 Hello
Thanks for raising the issue, I think the error is not coming from the SQL macro but from the adapter code https://github.com/dbt-athena/dbt-athena/blob/main/dbt/adapters/athena/impl.py#L401. We are not passing the quote seed column parameter through.

Hey,
Ah you're right quote_seed_column doesn't currently do anything. I've had a go at implementing here but it doesn't resolve the issue i'm seeing with the columns starting with underscores.
The create external table statements want underscore prefixed columns to be surrounded in backticks but the select statement complains if thats the case.
I guess we could add a quote_character parameter to quote_seed_column and set as backtick in the DDL statements?


applySession equivalent for tests?

I just learned about next-iron-session's deprecation and upon upgrading I couldn't figure out how to replace applySession.
I am using it in many of my jest tests to setup the server like so:
beforeAll(async () => {
    ({ server, url, session } = await createServerWithSession(undefined, handler));
  });

import http from 'http';
import listen from 'test-listen';
import { apiResolver } from 'next/dist/server/api-utils/node';
import { applySession } from 'next-iron-session';
import { User, Credentials } from '~/types/user';

export type Session = {
  destroy: () => void;
  set: (arg0: string, arg1: User | Credentials) => void;
  save: () => void;
  get: (arg0: string) => unknown;
};

export const createServerWithSession = async (params, handler) => {
  const server = http.createServer((req, res) =>
    apiResolver(
      req,
      res,
      params,
      handler,
      {
        /* Not needed but required params */
        previewModeId: '',
        previewModeEncryptionKey: '',
        previewModeSigningKey: '',
      },
      false,
    ),
  );
  const url = await listen(server);
  const res: { session?: Session } = {};
  let session: Session;
  await applySession(res, undefined, { cookieName: 'test', password: 'test' });
  if (res.session) {
    session = res.session;
  } else {
    throw new Error('No session after apply session');
  }
  return { server, url, session };
};

Hey there, I believe I was actually waiting for someone to provide a good use-case for applySession. Do you think you could re-add the functionality to iron-session via a PR?
🙏

@vvo
I previously was using it in my tests to successfully test the session state from my handlers. I want a way which I can test the state of the session after running my api request. I've not had luck with the new API figuring out a way to get access to req.session in my tests. Perhaps, there's a better way to do this.
const createServerWithSession = async (handler: NextApiHandler, params?: { [key: string]: string | string[] }) => {
  const server = createServer(async (req, res) => {
    await apiResolver(
      req,
      res,
      params,
      handler,
      {
        /* Not needed but required params */
        previewModeId: '',
        previewModeEncryptionKey: '',
        previewModeSigningKey: '',
      },
      false,
    );
  });

  const url = await listen(server);
  const init: RequestInit = {
    headers: {
      'x-msw-bypass': 'true',
    },
  };

- const req: { session?: Session } = {};
-  let session: Session;
- await applySession(req, undefined, { cookieName: 'test', password: 'test' });
-  if (res.session) {
-    session = res.session;
-  } else {
-    throw new Error('No session after apply session');
-  }
-
  return { server, session, url, init };
};

beforeAll(async () => {
  ({ server, session, url } = await createServerWithSession(handler, undefined));
});

it('destroys the session', async () => {
    session.user = { id: '12345', email<EMAIL_ADDRESS>};
    await session.save();

    const response = await fetch(url, {
      method: 'POST',
      headers: { 'Content-Type': 'application/json' },
    });
    const jsonResult = (await response.json()) as Record<string, unknown>;

    expect(response.status).toBe(200);
    const authStatus: AuthStatusNone = 'None';
    expect(jsonResult).toMatchObject({ credentials: { authStatus } });
    expect(session.user).toBeUndefined();
  });


SLOWFAST_R101_K600.pth.tar

@junting @Siyu-C
hello，thanks for your great works!
By the way,where is the SLOWFAST_R101_K600.pth.tar from PySlowfast? I can not find it.
I have found this:

but i can not load them!

It was converted from the original caffe checkpoint in pkl format.


No use this template button

I want to make a scratch extension but there is no use this template button please help

would you mind sharing a screenshot of what you do see instead, please?


[ZEPPELIN-1406] modify to load method artifact local path in Helium application json

What is this PR for?
Currently contained in the Zeppelin Helium application does not work.
The relative value of the configuration file path does incurring call normally.
Therefore, it sets the Helium directory you specified in the application default directory.
An example of the application was to modify the path of helium.
What type of PR is it?
Bug Fix, Improvement
Todos

[x] added setter method for artifact in Helium Packakge class
[x] if not absolute path for artifact then replace to absolute path for helium artifact local path
[x] replace helium example artifact path for json

What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-1406
How should this be tested?
step 1.
build
mvn clean package -Pbuild-distr -Pexamples -DskipTests

step 2.
go to web ui and create a new notebook.
write paragraph.
here.
%spark
import java.util.Date
new Date();

step 3.
choice.
helium application clock.
working well.
Screenshots (if appropriate)
before
does not working.

after

Questions:

Does the licenses files need update? no
Is there breaking changes for older versions? no
Does this needs documentation? no

@cloverhearts I can't reproduce this bug. Here is what I did.
1. Build the latest master branch with below command and restart Zeppelin daemon
mvn clean package -Pbuild-distr -Pexamples -DskipTests

2. Create a new notebook and run
%spark

import java.util.Date
new Date();

3. I can see the Helium icon and the clock without this patch

Did I miss something?

@AhyoungRyu
I'll check it.


Add markdown escape to TextChatService.yaml

Changes
The last pull request for this problem fixed most of the problem, but it looks like the bar character in Class.TextChannel | TextChannels is now being used for the markdown table.

This is the reason why this pull request adds an escape character before the bar. It is unknown if this will break the automatic link:
Class.TextChannel \| TextChannels
I tried to see what was done in such a scenario on other pages, but I was unable to find a similar scenario. The closest I found was this table on Datatype.Vector3 However, it turned out that table just used pure html tags for its table and it doesn't override the default text link.
Checks
By submitting your pull request for review, you agree to the following:

[x] This contribution was created in whole or in part by me, and I have the right to submit it under the terms of this repository's open source licenses.
[x] I understand and agree that this contribution and a record of it are public, maintained indefinitely, and may be redistributed under the terms of this repository's open source licenses.
[x] To the best of my knowledge, all proposed changes are accurate.

Removing the spaces around the " | " should make the link form correctly (I did so). Thanks for the correction!


⚠ Orpheus has degraded performance

In 805ba96, Orpheus ($OPS) experienced degraded performance:

HTTP code: 200
Response time: 1262 ms

Resolved: Orpheus performance has improved in 1e36634 after 6 minutes.


docs: fix typos in homeassistant.md

Just a few typos

Pull Request Test Coverage Report for Build 553335351

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 19.896%

Totals

Change from base Build 551384155:
0.0%

Covered Lines:
1996

Relevant Lines:
10285

💛  - Coveralls


False positive with no unused variable rule

Variables contained within objects are not detected by the "no unused variable" rule:
query Foo($a: String) {
  field(object: { a: $a })
}

"Variable "$a" is never used in operation "Foo"."

Any update on this, I am also facing similar issue

Commented NoUnusedVariablesRule to fix nested variable issue
https://github.com/GraphQLSwift/GraphQL/pull/85
@paulofaria  please check and update if it work, I have verified it is working fine.


EA account addition throws code 400

Error description:
When choosing EA account the request body is not correct therefore we receive error 400.
Exception:
The Enterprise Enrollment Reader role could not be granted. Status Code: 400; Response: {"errors":{"Properties.PrincipalId":["PrincipalId or UserEmailAddress/UserAuthenticationType must be
| specified."],"Properties.RoleDefinitionId":["The RoleDefinitionId field is required."],"Properties.UserEmailAddress":["PrincipalId or UserEmailAddress/UserAuthenticationType must be
| specified."],"Properties.UserAuthenticationType":["PrincipalId or UserEmailAddress/UserAuthenticationType must be specified."]},"type":"https://tools.ietf.org/html/rfc7231#section-6.5.1","title":"One
| or more validation errors occurred.","status":400,"traceId":""}
In the code the URI body does not contain properties object
MS article: https://learn.microsoft.com/en-us/azure/cost-management-billing/manage/assign-roles-azure-service-principals
Code snippet:
                 $body = "{"principalId":"$principalId","principalTenantId":"$tenantId","roleDefinitionId":"/providers/Microsoft.Billing/billingAccounts/$billingAccountId/billingRoleDefinitions/24f8edb6-1668-4659-b5e2-40bb5f3a7d7e"}"
To be code:
                    $body = "{"properties": {"principalId":"$principalId","principalTenantId":"$tenantId","roleDefinitionId":"/providers/Microsoft.Billing/billingAccounts/$billingAccountId/billingRoleDefinitions/24f8edb6-1668-4659-b5e2-40bb5f3a7d7e"}}"

Hi, @attilabus16092020 . Thank you for reporting this issue. It seems you are indeed correct. Can you confirm you have successfully granted the needed role with the code fix you suggested? Can you also check whether the fix proposed here will solve the issue?

HI @helderpinto,
It does. :)


Positively define input text color for dark themes

Before this change, input text on dark themes was invisible due to having similar color to dark background.

I signed it!

Can you share the bug you're trying to fix with this? I think the problem is not with the input, but with the dark-background container. The input sets is color to currentColor in order to properly contrast with whatever background its on. It's up to the container to set the background but also set the color to something that contrasts well

I can't seem to be able to reproduce the problem now... perhaps it was
fixed in beta 8... Anyway, the pull request may be rejected
On Tue, Aug 22, 2017 at 7:52 PM mmalerba<EMAIL_ADDRESS>wrote:

Can you share the bug you're trying to fix with this? I think the problem
is not with the input, but with the dark-background container. The input
sets is color to currentColor in order to properly contrast with whatever
background its on. It's up to the container to set the background but also
set the color to something that contrasts well
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/angular/material2/pull/5732#issuecomment-324086496,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABLa6OIZDcMLfHTP1HVY-cqqKg-6o8veks5sawc8gaJpZM4OXVFb
.


Laravel Documentation is incorrect

Hi there, in the Laravel Dusk example, the example code does not work:
https://pestphp.com/docs/plugins/laravel
use Laravel\Dusk\Browser;
 <?php
it('has homepage', function () {
    $this->browse(function (Browser $browser) {
        $browser->visit('/')
            ->assertSee('Pest');
    });
});

This yields an error Call to undefined method Tests\Feature\UserFormCreateTest::browse()

Just to check, your UserFormCreateTest class is extending the Tests\DuskTestCase class?


Fix check_sdk github action

Did not work if master branch is named 'main'
Did not work on PRs not targeting 'master' or 'develop'

Failing as expected here
https://github.com/LedgerHQ/app-plugin-boilerplate/pull/111
Success as expected here
https://github.com/LedgerHQ/app-plugin-boilerplate/pull/110

Failing as expected with correct logs on this fake 'main' branch
https://github.com/LedgerHQ/app-plugin-boilerplate/actions/runs/6707339532/job/18225639879


HBASE-23157 WAL unflushed seqId tracking may wrong when Durability.AS…

…YNC_WAL is used

@saintstack @binlijin PTAL. Thanks.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

:blue_heart:
reexec
1m 12s
Docker mode activated.

_ Prechecks _

:green_heart:
dupname
0m  1s
No case conflicting files found.

:green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

:green_heart:
@author
0m  0s
The patch does not contain any @author tags.

:green_heart:
test4tests
0m  0s
The patch appears to include 5 new or modified test files.

_ master Compile Tests _

:blue_heart:
mvndep
0m 33s
Maven dependency ordering for branch

:green_heart:
mvninstall
5m 43s
master passed

:green_heart:
compile
1m 20s
master passed

:green_heart:
checkstyle
2m  0s
master passed

:green_heart:
shadedjars
5m  2s
branch has no errors when building our shaded downstream artifacts.

:green_heart:
javadoc
0m 58s
master passed

:blue_heart:
spotbugs
4m 32s
Used deprecated FindBugs config; considering switching to SpotBugs.

:green_heart:
findbugs
5m 20s
master passed

_ Patch Compile Tests _

:blue_heart:
mvndep
0m 14s
Maven dependency ordering for patch

:green_heart:
mvninstall
5m 30s
the patch passed

:green_heart:
compile
1m 22s
the patch passed

:green_heart:
javac
1m 22s
the patch passed

:broken_heart:
checkstyle
1m 32s
hbase-server: The patch generated 1 new + 289 unchanged - 2 fixed = 290 total (was 291)

:green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

:green_heart:
shadedjars
5m  0s
patch has no errors when building our shaded downstream artifacts.

:green_heart:
hadoopcheck
17m 48s
Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.

:broken_heart:
javadoc
0m 42s
hbase-server generated 1 new + 0 unchanged - 0 fixed = 1 total (was 0)

:green_heart:
findbugs
5m 50s
the patch passed

_ Other Tests _

:green_heart:
unit
2m 59s
hbase-common in the patch passed.

:broken_heart:
unit
227m 12s
hbase-server in the patch failed.

:green_heart:
asflicense
0m 49s
The patch does not generate ASF License warnings.

300m 29s

Subsystem
Report/Notes

Docker
Client=19.03.4 Server=19.03.4 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-762/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/762

Optional Tests
dupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile

uname
Linux 9f57316741a1 4.15.0-66-generic #75-Ubuntu SMP Tue Oct 1 05:24:09 UTC 2019 x86_64 GNU/Linux

Build tool
maven

Personality
/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-762/out/precommit/personality/provided.sh

git revision
master / d7deafa120

Default Java
1.8.0_181

checkstyle
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-762/1/artifact/out/diff-checkstyle-hbase-server.txt

javadoc
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-762/1/artifact/out/diff-javadoc-javadoc-hbase-server.txt

unit
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-762/1/artifact/out/patch-unit-hbase-server.txt

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-762/1/testReport/

Max. process+thread count
4674 (vs. ulimit of 10000)

modules
C: hbase-common hbase-server U: .

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-762/1/console

versions
git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11

Powered by
Apache Yetus 0.11.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

:blue_heart:
reexec
3m 18s
Docker mode activated.

_ Prechecks _

:green_heart:
dupname
0m  0s
No case conflicting files found.

:green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

:green_heart:
@author
0m  0s
The patch does not contain any @author tags.

:green_heart:
test4tests
0m  0s
The patch appears to include 5 new or modified test files.

_ master Compile Tests _

:blue_heart:
mvndep
0m 33s
Maven dependency ordering for branch

:green_heart:
mvninstall
5m 44s
master passed

:green_heart:
compile
1m 19s
master passed

:green_heart:
checkstyle
1m 57s
master passed

:green_heart:
shadedjars
5m  0s
branch has no errors when building our shaded downstream artifacts.

:green_heart:
javadoc
0m 57s
master passed

:blue_heart:
spotbugs
4m 30s
Used deprecated FindBugs config; considering switching to SpotBugs.

:green_heart:
findbugs
5m 18s
master passed

_ Patch Compile Tests _

:blue_heart:
mvndep
0m 14s
Maven dependency ordering for patch

:green_heart:
mvninstall
5m 30s
the patch passed

:green_heart:
compile
1m 19s
the patch passed

:green_heart:
javac
1m 19s
the patch passed

:green_heart:
checkstyle
0m 26s
The patch passed checkstyle in hbase-common

:green_heart:
checkstyle
1m 28s
hbase-server: The patch generated 0 new + 289 unchanged - 2 fixed = 289 total (was 291)

:green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

:green_heart:
shadedjars
5m  1s
patch has no errors when building our shaded downstream artifacts.

:green_heart:
hadoopcheck
17m 20s
Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.

:green_heart:
javadoc
0m 54s
the patch passed

:green_heart:
findbugs
5m 31s
the patch passed

_ Other Tests _

:green_heart:
unit
2m 56s
hbase-common in the patch passed.

:green_heart:
unit
229m 27s
hbase-server in the patch passed.

:green_heart:
asflicense
0m 46s
The patch does not generate ASF License warnings.

302m  8s

Subsystem
Report/Notes

Docker
Client=19.03.4 Server=19.03.4 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-762/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/762

Optional Tests
dupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile

uname
Linux 65033938e96d 4.15.0-66-generic #75-Ubuntu SMP Tue Oct 1 05:24:09 UTC 2019 x86_64 GNU/Linux

Build tool
maven

Personality
/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-762/out/precommit/personality/provided.sh

git revision
master / d7deafa120

Default Java
1.8.0_181

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-762/2/testReport/

Max. process+thread count
4706 (vs. ulimit of 10000)

modules
C: hbase-common hbase-server U: .

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-762/2/console

versions
git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11

Powered by
Apache Yetus 0.11.0 https://yetus.apache.org

This message was automatically generated.

LGTM

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m  9s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  1s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s
The patch appears to include 5 new or modified test files.

_ master Compile Tests _

+0 :ok:
mvndep
0m 36s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
5m  6s
master passed

+1 :green_heart:
compile
1m 21s
master passed

+1 :green_heart:
checkstyle
1m 53s
master passed

+1 :green_heart:
shadedjars
4m 34s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 57s
master passed

+0 :ok:
spotbugs
4m 11s
Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
4m 58s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 15s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
5m  4s
the patch passed

+1 :green_heart:
compile
1m 21s
the patch passed

+1 :green_heart:
javac
1m 21s
the patch passed

+1 :green_heart:
checkstyle
0m 26s
The patch passed checkstyle in hbase-common

+1 :green_heart:
checkstyle
1m 26s
hbase-server: The patch generated 0 new + 280 unchanged - 2 fixed = 280 total (was 282)

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
shadedjars
4m 35s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
hadoopcheck
15m 40s
Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.

+1 :green_heart:
javadoc
0m 59s
the patch passed

+1 :green_heart:
findbugs
5m 13s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
3m  9s
hbase-common in the patch passed.

-1 :x:
unit
304m 45s
hbase-server in the patch failed.

+1 :green_heart:
asflicense
0m 56s
The patch does not generate ASF License warnings.

371m  7s

Reason
Tests

Failed junit tests
hadoop.hbase.client.TestSnapshotTemporaryDirectoryWithRegionReplicas

hadoop.hbase.client.TestFromClientSide

Subsystem
Report/Notes

Docker
Client=19.03.5 Server=19.03.5 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-762/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/762

Optional Tests
dupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile

uname
Linux 374163a9e12d 4.15.0-65-generic #74-Ubuntu SMP Tue Sep 17 17:06:04 UTC 2019 x86_64 GNU/Linux

Build tool
maven

Personality
/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-762/out/precommit/personality/provided.sh

git revision
master / 5e34a59d90

Default Java
1.8.0_181

unit
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-762/1/artifact/out/patch-unit-hbase-server.txt

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-762/1/testReport/

Max. process+thread count
4800 (vs. ulimit of 10000)

modules
C: hbase-common hbase-server U: .

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-762/1/console

versions
git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

Oh, this one is still pending?
Any other concerns? @saintstack

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
2m 10s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s
The patch appears to include 5 new or modified test files.

_ master Compile Tests _

+0 :ok:
mvndep
0m 46s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
6m 52s
master passed

+1 :green_heart:
compile
1m 36s
master passed

+1 :green_heart:
checkstyle
2m 10s
master passed

+1 :green_heart:
shadedjars
5m 11s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 57s
master passed

+0 :ok:
spotbugs
4m 36s
Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
5m 23s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 14s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
5m 30s
the patch passed

+1 :green_heart:
compile
1m 21s
the patch passed

+1 :green_heart:
javac
1m 21s
the patch passed

+1 :green_heart:
checkstyle
0m 27s
The patch passed checkstyle in hbase-common

+1 :green_heart:
checkstyle
1m 30s
hbase-server: The patch generated 0 new + 280 unchanged - 2 fixed = 280 total (was 282)

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
shadedjars
5m  0s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
hadoopcheck
17m 23s
Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.

+1 :green_heart:
javadoc
0m 55s
the patch passed

+1 :green_heart:
findbugs
5m 32s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
2m 59s
hbase-common in the patch passed.

-1 :x:
unit
207m 19s
hbase-server in the patch failed.

+1 :green_heart:
asflicense
1m 22s
The patch does not generate ASF License warnings.

281m 57s

Reason
Tests

Failed junit tests
hadoop.hbase.util.TestFromClientSide3WoUnsafe

Subsystem
Report/Notes

Docker
Client=19.03.5 Server=19.03.5 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-762/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/762

Optional Tests
dupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile

uname
Linux e12c98f8ddf3 4.15.0-66-generic #75-Ubuntu SMP Tue Oct 1 05:24:09 UTC 2019 x86_64 GNU/Linux

Build tool
maven

Personality
/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-762/out/precommit/personality/provided.sh

git revision
master / 60d9430195

Default Java
1.8.0_181

unit
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-762/2/artifact/out/patch-unit-hbase-server.txt

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-762/2/testReport/

Max. process+thread count
4754 (vs. ulimit of 10000)

modules
C: hbase-common hbase-server U: .

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-762/2/console

versions
git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

@saintstack Let's finish this one?

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 25s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+0 :ok:
mvndep
0m 22s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
3m 36s
master passed

+1 :green_heart:
checkstyle
1m 35s
master passed

+1 :green_heart:
spotbugs
2m 41s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 12s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
3m 21s
the patch passed

+1 :green_heart:
checkstyle
0m 22s
The patch passed checkstyle in hbase-common

+1 :green_heart:
checkstyle
1m 10s
hbase-server: The patch generated 0 new + 258 unchanged - 2 fixed = 258 total (was 260)

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
hadoopcheck
10m 59s
Patch does not cause any errors with Hadoop 3.1.2 3.2.1.

+1 :green_heart:
spotbugs
3m  0s
the patch passed

_ Other Tests _

+1 :green_heart:
asflicense
0m 26s
The patch does not generate ASF License warnings.

35m 33s

Subsystem
Report/Notes

Docker
Client=19.03.12 Server=19.03.12 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-762/2/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/762

Optional Tests
dupname asflicense spotbugs hadoopcheck hbaseanti checkstyle

uname
Linux 7f86e6449b14 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 975cdf7b88

Max. process+thread count
94 (vs. ulimit of 12500)

modules
C: hbase-common hbase-server U: .

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 16s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
4m  2s
the patch passed

+1 :green_heart:
compile
1m 29s
the patch passed

+1 :green_heart:
javac
1m 29s
the patch passed

+1 :green_heart:
shadedjars
5m 46s
patch has no errors when building our shaded downstream artifacts.

-0 :warning:
javadoc
0m 17s
hbase-common in the patch failed.

-0 :warning:
javadoc
0m 39s
hbase-server in the patch failed.

_ Other Tests _

+1 :green_heart:
unit
1m 32s
hbase-common in the patch passed.

+1 :green_heart:
unit
134m 35s
hbase-server in the patch passed.

164m 26s

Subsystem
Report/Notes

Docker
Client=19.03.12 Server=19.03.12 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-762/2/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/762

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 12b7868bfec7 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 975cdf7b88

Default Java
2020-01-14

javadoc
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-762/2/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-common.txt

javadoc
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-762/2/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-server.txt

javadoc
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-762/2/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-common.txt

javadoc
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-762/2/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-server.txt

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-762/2/testReport/

Max. process+thread count
4085 (vs. ulimit of 12500)

modules
C: hbase-common hbase-server U: .

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-762/2/console

versions
git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 24s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+0 :ok:
mvndep
0m 24s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
3m 35s
master passed

+1 :green_heart:
compile
1m 16s
master passed

+1 :green_heart:
shadedjars
5m 37s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 57s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 16s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
3m 31s
the patch passed

+1 :green_heart:
compile
1m 21s
the patch passed

+1 :green_heart:
javac
1m 21s
the patch passed

+1 :green_heart:
shadedjars
5m 30s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 58s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
1m 14s
hbase-common in the patch passed.

+1 :green_heart:
unit
144m 38s
hbase-server in the patch passed.

171m 51s

Subsystem
Report/Notes

Docker
Client=19.03.12 Server=19.03.12 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-762/2/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/762

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux c67d19bd252c 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 975cdf7b88

Default Java
1.8.0_232

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-762/2/testReport/

Max. process+thread count
4262 (vs. ulimit of 12500)

modules
C: hbase-common hbase-server U: .

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-762/2/console

versions
git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.


Class names are undefined

Loaded class names are 'undefined' in saved txt files, although class names were successfully loaded. I get the bounding box positions but labeled as undefined

I'm having the same issue.

I may have found the reason: the class names are read from the .txt file so that an additional newline character is included in each name. Therefore the code cannot find indices that match the annotated classes, and you get 'undefined' in the saved .txt files.
I don't know if this is the perfect / global solution, but it seems to be working at least for me:
In boobs.js file, simply replace row 724:
const rows = lines.split("\n")
with this row:
const rows = lines.split("\r\n")

I think you should make that a PR

Thanks for finding the issue guys! Unfortunately, your fix will only work on Windows and will break Mac OS. So I will close the PR and make the necessary changes myself.


Update javadoc links

On the Ferma hosted documentation/page several versions are missing from the javadoc links.
https://github.com/Syncleus/Ferma

fixed


Add navvis files for unit tests

Add NavVis test files for testing.
test data restored from https://github.com/microsoft/scantools/tree/main/test_data

How large is this test data? I would very much avoid committing it to the repo, which would force all users to download it even though it is used only for testing. Instead, 1) add it as a git submodule or 2) zip it, add it as release artefact, and download it when running the tests.

How large is this test data? I would very much avoid committing it to the repo, which would force all users to download it even though it is used only for testing. Instead, 1) add it as a git submodule or 2) zip it, add it as release artefact, and download it when running the tests. I prefer option 2.

yes we can do option 2 and add a comment if you want test

I would very much avoid committing it to the repo, which would force all users to download it even though it is used only for testing

I agree, I was also proposing the same here :)
Now my question is, will we anyways download all this data when we do git clone (actually twice because it was in the other PR too). Is there a way to avoid this? I think @mihaidusmanu had this problem in the past. What did you do? Other than --depth 1 in main branch? Just curious, probably we would need to force push and rewrite the git history. Just raising the concern that not merging these commits is not enough to avoid downloading the data.

By default git clone only clones the main branch, so it should not include any of these slides - especially since the branch of this PR has now been deleted.


iTerm2 profile error

I've copied com.googlecode.iterm2.plist to /Users/foo/Library/Application Support/iTerm2/DynamicProfiles/
I am getting:
6/2, 07:23:57,428: Ignoring dynamic profiles in malformed file /Users/foo/Library/Application Support/iTerm2/DynamicProfiles/com.googlecode.iterm2.plist and continuing.

Solution was to copy this file to ~/Library/Preferences/.


Added support for setting trust domain (global.trustDomain equivalent)

Q
A

Bug fix?
no

New feature?
yes

API breaks?
no

Deprecations?
no

Related tickets
fixes #100

License
Apache 2.0

What's in this PR?
Enables setting the equivalent of global.trustDomain from the Istio Helm Chart. Chose to default to "cluster.local" where that was done previously.
Checklist

[x] Implementation tested
[x] User guide and development docs updated (if needed)

The PR looks great, there is just one place where I think it was missed earlier to add the trustDomain arg, namely in the Mixer deployment where it should be set according to this: https://github.com/istio/istio/blob/release-1.3/install/kubernetes/helm/istio/charts/mixer/templates/deployment.yaml#L136-L138
Could you please add it here as well: https://github.com/banzaicloud/istio-operator/blob/release-1.3/pkg/resources/mixer/deployment.go#L279 ?
Otherwise, I think the PR will be good to be merged.


Cc3100 version

started working on this before camp but never got round to pushing it
much cleaner way to return version info from the CC3100 so that it can be use programmatically

:+1:


HW #10.10.16

https://github.com/DESTROYED/HW/commit/801a31c8230ba15967c9e3df7239df302b675283  Visible/Invisible button HW
https://github.com/DESTROYED/HW/commit/f86632cf6985dcb45c74e9da83922d3e2db479a5 Instrumental test

ok


Matching season episodes by Sonarr quality profile ID fails sometimes

Describe the bug
When creating a rule that triggers by Sonarr Quality Profile ID (why use ID# and not name here as with Radarr?) (Sonarr - Quality profile ID -> Equals -> Number -> #) it will sometimes flag individual episodes in series outside the specified quality profile ID as matching.
Using the Test Media option a single episode is matched from a show not matching the specified quality profile, but all other epiodes of the same show correctly return false for the match.
To Reproduce
Steps to reproduce the behavior:

Go to Rules
Click on 'New Rule'
Set library to a TV Series library
Set type to Episodes
Under Rules add a rule with first value: Sonarr Quality Profile ID - Action: Equals - Second Value: Number - Custom Value ID of a quality profile. In my case 9.

I then also have a section/rule that checks if the episode was added more than 365 days ago. But this one works as expected.
Expected behavior
Only episodes matching this criteria will show up.
Device (please complete the following information):

OS: Debian/Docker
Version v 5.10.0-21

Would you be able to share a screenshot of the Manage Episodes dialog in Sonarr from a season that has one of these problem episodes?
How comfortable are you with calling APIs using Postman/Insomnia/curl in-case we need to dig a bit further?

Sure - here is Would I lie To You season 16 - where episode 8 shows up in this rule:

A test of the episode in Test Media:
- plexId: 51774   result: true   sectionResults:     - id: 0       result: true       ruleResults:         - operator: OR           action: equals           firstValueName: Sonarr - Quality profile ID           firstValue: 9           secondValueName: number           secondValue: 9           result: true     - id: 1       result: true       operator: AND       ruleResults:         - operator: OR           action: before           firstValueName: Plex - Date added           firstValue: 2023-03-24T22:23:53.000Z           secondValueName: custom_days           secondValue: 2023-11-30T10:04:51.772Z           result: true
And a test of episode 7 in the same season:
- plexId: 51628   result: false   sectionResults:     - id: 0       result: false       ruleResults:         - operator: OR           action: equals           firstValueName: Sonarr - Quality profile ID           firstValue: 3           secondValueName: number           secondValue: 9           result: false     - id: 1       result: true       operator: AND       ruleResults:         - operator: OR           action: before           firstValueName: Plex - Date added           firstValue: 2023-02-17T21:24:49.000Z           secondValueName: custom_days           secondValue: 2023-11-30T10:06:48.333Z           result: true
I'm experienced using REST API using powershell - not so much using curl directly. But I'm sure i can help with rest API output if needed. I used to do these clean-ups with a powershell script, but Maintainerr is much prefered :)
The workaround I use now is to use tags instead of quality profles. And that works perfectly.

So we look at the individual episode to determine the quality profile here, and episode 8 there is HDTV-1080p unlike the others which are WEBDL-1080P.

Yes, I noticed that also. But the quality profile follows the show, not the episodes, as far as I've understood?

You have the type set to Episodes so we look at episode metadata currently: https://github.com/jorenn92/Maintainerr/blob/cca3b0fae36e6939cb51d1c1c83ec8bc7cda6f48/server/src/modules/rules/getter/sonarr-getter.service.ts#L147
We could always introduce a new rule that targets the season/show level in this instance. Feature requests here: https://features.maintainerr.info/

Ah, so in this case it looks at the Plex metadata instead of the sonarr quality profile id? As the quality profile is on the show level and not on the episode level. I see the confusion here.
Tags are working fine for me, so this is a non-issue at the moment. But yes, I may do a feature request for future checks. Thank you for quick feedback :)

It does look at the Sonarr quality profile, just at the episode level. Of course we don't typically think of an episode having a quality profile but they can differ due to upgrades etc.


launching recent projects options

right-click on home page recent project could displays a contextual menu
with an option to open the project directly in a new window.

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

:slightly_frowning_face: In the last 60 days, this feature request has received less than 20 community upvotes and we closed it. Still a big Thank You to you for taking the time to create this issue! To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


Change the template on the Map page

(perhaps other pages that don't have a TOC or index.
Is it possible to select a template that uses the whole page - there is too much empty space on the left.
https://emylonas.github.io/xml-minimal/gardens/

added front matter variable custom_css: gardens
added liquid logic to end of assets/head/custom.html to check for custom_css and add stylesheet for given css
added gardens.css to /assets/css and reset .page for width and float


Why the layout?

Why can't you use the BottomSheet as a dialog like class instead of embedding it as part of a layout? If I'm not seeing it. please let me know, but this is far more useful if can use it in that way instead of it having to be put as a wrapper in every layout.
Thanks, I like the features, but don't see why it needs to be done in this fashion.

Functionally speaking, how would you propose this be handled as a dialog? It's built this way right now so that you can have the content view interact with its movements accordingly, and it's not much work to stuff this down into a base activity.

Going to close this for now, we can revisit if others have some good suggestions for a dialog approach.

Just a thought here, but perhaps bottomsheet could be refactored to use Google's CoordinatorLayout its Behavior implementation for scrolling. Since this library is trying to aid in material design support having the design support lib as a dependency wouldn't be bad I don't think.

How would that work implementation-wise? I'm not too familiar with
CoordinatorLayout yet
On Fri, Jul 24, 2015 at 9:04 AM Nathan Schwermann<EMAIL_ADDRESS>wrote:

Just a thought here, but perhaps bottomsheet could be refactored to use
Google's CoordinatorLayout its Behavior implementation for scrolling. Since
this library is trying to aid in material design support having the design
support lib as a dependency wouldn't be bad I don't think.
—
Reply to this email directly or view it on GitHub
https://github.com/Flipboard/bottomsheet/issues/9#issuecomment-124568027
.

CoordinatorLayout is a ViewGroup that looks at every child's layout params for a Behavior. The Behavior tells the CoordinatorLayout how that childview should interact with other views and handles nested scrolling and other things.
For example, when you add a FloatingActionButton to a CoordinatorLayout it finds FAB's default Behavior attached and uses it to move the button on the screen based off of its anchor View, or to animate and hide if it gets too close to the Toolbar.
What I'm suggesting for bottomsheet is instead of BottomSheetLayout being a top level view it would be refactored into a view that implements a default Behavior.  That would help avoid nesting so many containers like the pseudo XML below required to support both FAB and Bottomsheet in a single view.
<!--Bottomsheet must be the outer most view because it can only have one child -->
<com.flipboard.bottomsheet.BottomSheetLayout>
       <android.support.design.widget.CoordinatorLayout>
          <!--Toolbar, Content, FAB, etc all goes here.  BottomSheetLayout replacement here as well instead>
       </android.support.design.widget.CoordinatorLayout>

</com.flipboard.bottomsheet.BottomSheetLayout>

Right, I did understand that much. It's behavior implementations themselves
that I'm not too knowledgable on. Are they flexible enough allow for the
same API? Allowing for custom ViewTransformer implementations to interact
with the content view is a must for us.
On Fri, Jul 24, 2015 at 9:47 AM Nathan Schwermann<EMAIL_ADDRESS>wrote:

CoordinatorLayout is a ViewGroup that looks at every child's layout params
for a Behavior
https://developer.android.com/reference/android/support/design/widget/CoordinatorLayout.Behavior.html.
The Behavior tells the CoordinatorLayout how that childview should interact
with other views and handles nested scrolling and other things.
For example, when you add a FloatingActionButton to a CoordinatorLayout it
finds FAB's default Behavior attached and uses it to move the button on the
screen based off of its anchor View, or to animate and hide if it gets too
close to the Toolbar.
What I'm suggesting for bottomsheet is instead of BottomSheetLayout being
a top level view it would be refactored into a view that implements a
default Behavior. That would help avoid nesting so many containers like the
pseudo XML below required to support both FAB and Bottomsheet in a single
view.

<com.flipboard.bottomsheet.BottomSheetLayout>
<android.support.design.widget.CoordinatorLayout>
<!--Toolbar, Content, FAB, etc all goes here.  BottomSheetLayout replacement here as well instead>
</android.support.design.widget.CoordinatorLayout>
</com.flipboard.bottomsheet.BottomSheetLayout>
—
Reply to this email directly or view it on GitHub
https://github.com/Flipboard/bottomsheet/issues/9#issuecomment-124576775
.

I'm not following you on the requirement for ViewTransformer implementations. But I don't think your API would change at all, only the requirement for a top-level view. But thinking about it I suppose there is no reason your BottomSheetLayout needs to be top level either, I suppose it could still be moved inside the CoordinatorLayout in my example above.  I guess to that point moving to Behavior doesn't really change anything, just a different way of solving the same problem.

This is what I'm referring to

We have to be able to maintain that level of control for the animation.
I agree though that the simple solution would just be to move it inside of CoordinatorLayout, and probably what I'd rather suggesting for now unless CoordinatorLayout offers us some useful API to improve/simplify this logic.

I would like to add something about this. I'm currently using a CoordinatorLayout and an AppBarLayout. I'm using the app:layout_behavior="@string/appbar_scrolling_view_behavior" so when I scroll down, the toolbar hides. Using the BottomSheetLayout as root is not an option because then the expanded view would go on top of the toolbar leaving no space for the user to click outside of the BottomSheet (as Google guidelines say). What I did was putting the layout under the AppBarLayout, wrapping the content. This creates another problem.
Let me explain how app:layout_behavior="@string/appbar_scrolling_view_behavior" works. When you scroll down, the toolbar goes up as well as the childs that use this behavior. When you scroll up, the toolbar comes down and those childs also come down (going offscreen or under the Android virtual buttons). With this behavior the contents height ignores the toolbar height because it will be moved up when the toolbar hides.
The problem is that going offscreen means that the last element present on the BottomSheet list is going to be hidden. I couldn't find a decent solution for this yet. What I'm currently doing is adding an empty MenuItem so it pulls the real items up. And then on the onClickListener I ignore that empty item.

I don't have much experience with CoordinatorLayout but could you not just
place the BottomSheetLayout in the root view of your content?
Also I think your comment about the design guidelines is incorrect. Google
has plenty of examples of full screen bottom sheets covering the whole
view. It's intuitive enough that users know they can drag or press back to
dismiss it. If you really must leave space above, you can set a custom peek
height or use LayoutParams to set your desired expanded height.
Code samples would really help here if you have one.
On Wed, Feb 17, 2016 at 9:43 AM Ricardo<EMAIL_ADDRESS>wrote:

I would like to add something about this. I'm currently using a
CoordinatorLayout and an AppBarLayout. I'm using the
app:layout_behavior="@string/appbar_scrolling_view_behavior" so when I
scroll down, the toolbar hides. Using the BottomSheetLayout as root is not
an option because then the expanded view would go on top of the toolbar
leaving no space for the user to click outside of the BottomSheet (as
Google guidelines say). What I did was putting the layout under the
AppBarLayout, wrapping the content. This creates another problem.
Let me explain how
app:layout_behavior="@string/appbar_scrolling_view_behavior" works. When
you scroll down, the toolbar goes up as well as the childs that use this
behavior. When you scroll up, the toolbar comes down and those childs also
come down (going offscreen or under the Android virtual buttons). With this
behavior the contents height ignores the toolbar height because it will be
moved up when the toolbar hides.
The problem is that going offscreen means that the last element present on
the BottomSheet list is going to be hidden. I couldn't find a decent
solution for this yet. What I'm currently doing is adding an empty MenuItem
so it pulls the real items up. And then on the onClickListener I ignore
that empty item.
—
Reply to this email directly or view it on GitHub
https://github.com/Flipboard/bottomsheet/issues/9#issuecomment-185321226
.

Wouldn't NestedScrollLayout be a better choice?


Use external libraries for the appropriate stuff

[x] random-pure for random values generation

[x] use library's StdGen as Seed
[x] decide should be, or not, the special racket-case treated specially afterwards
[x] Pretty for the seed type
[x] cleanup, if needed
[x] tests updates

[x] summary-stat for confidence intervals

[x] replace custom bounded doubles to a library ones
[x] use computation of a Wilson interval from the library
[x] maybe use the coverage conditions checker from the library (decided to not to do)

[x] wait for the lack of dependencies to the contrib by newly used libraries

[x] Data.List.Lazy
[x] Data.Fin.Extra for a proof of a property for finToNat

Unlike the code being replaced, code of my libraries does not contain any MaxRobustSmGenNum-related hacks. But I tested both random-pure and summary-stat against racket and didn't ran into any 64-bit-related problems, so, I'd rather condier that this problem does not happen.

The PR is ready, possibly controversial moments are:

I decided to make CoveragePercentage to be not Tagged since the type I suggest to use instead of it is rather descriptive itself, I think;
since CoveragePercentage type has changed, it actually changed the actual signature of cover functon; despite it is in fact a breaking change, I think it's okay, since any correct constants that used to work there should still work. I've added a test that uses cover in more or less canonical way.

Thanks again for you work on this. I'm very happy that the random-number generation is no longer part of hedgehog.


cost_calculator

on the way ...

@enyo-xyz Do tag me for review purposes of the program when pushing it.
Thank you.
@NitkarshChourasia


Router, refactor

make it faster, robust and full features.
one more thing: the old router was renamed to FastRouter. feel free to inject it into di by hand.

A good piece of work, thanks!
I don't have, but we could use the ones form tests in the 1000 loop.

@Yahasana is it possible to have a static route the same as placeholder?
['GET', '/contact', ['controller' => 'index', 'action' => 'contact']],
['GET', '/{controller}', ['controller' => '[a-z]+[/]?']],

I tired with {action} too, but it's not working.

please check comments in Router::setRoutes L164
// the rule format: ['name' => ["httpMethod", "URI pattern", "regex", "defaults"]]
['GET', '/contact', null, ['controller' => 'index', 'action' => 'contact']]

It works, thanks.
Is it  a bug that
['GET', '/{action}', ['action' => '[aboutus|policy|terms|contact]+']],

matches /user too?

nope, do you add other route rules?

Yes, /{controller} as I sent before.

so /user will match the rule
['GET', '/{controller}', ['controller' => '[a-z]+[/]?']],

Routes
            ['GET', '/{module}', ['module' => '[admin|doc]+']],
            ['GET', '/{action}', ['action' => '[aboutus|policy|terms|contact]+'], ['controller' => 'index']],
            ['GET', '/{controller}', ['controller' => '[a-z]+[/]?']],

Tests
            ['/contact', ['frontend', 'index', 'contact', []]],
            ['/admin', ['admin', 'index', 'index', []]],
            ['/user', ['frontend', 'user', 'index', []]],

Result
1) Tests\RoutesTest::testUniversalGET with data set #2 ('/user', array('frontend', 'user', 'index', array()))
/user
Failed asserting that two arrays are equal.
--- Expected
+++ Actual
@@ @@
 Array (
     0 => 'frontend'
-    1 => 'user'
-    2 => 'index'
+    1 => 'index'
+    2 => 'user'
     3 => Array ()
 )

ok, it should be:
['GET', '/{module}', ['module' => 'admin|doc']],
['GET', '/{action}', ['action' => 'aboutus|policy|terms|contact']],
['GET', '/{controller}', ['controller' => '[a-z]+[/]?']],

thanks!

controller should NOT contain /
['GET', '/{controller}', ['controller' => '[a-z]+']],


Dealing with OPTIONS * requests

Today Kestrel will take the * from an OPTIONS * request and put it in the Path. However, * isn't a path and PathString throws.
When Kestrel sees a lone * in the request line it should leave both Path and PathBase empty.
https://github.com/aspnet/ServerTests/issues/20#issuecomment-200049192

Per triage meeting, this was done.


fix compatibility to mysql community cookbook v8.0.0 (fixes #14)

@tas50 the Travis jobs are failing, and there is no support for Ubuntu 16.04. Could you lend some 👀 ? 😍

We have an alias now so either method should work fine.


Can't fetch resource. There are no errors in the console

I'm trying to add custom resources like this
const adminBro = new AdminBro({     resources: [db.Phrase],    ... }
The Phrase model looks like this:
'use strict';

const Sequelize = require('sequelize');

exports.default = class Phrase extends Sequelize.Model {
    static init(sequelize) {
        return super.init({
            originalText: Sequelize.TEXT,
            transliteration: Sequelize.TEXT,
            translationKey: Sequelize.STRING
        }, {
            sequelize: sequelize,
            tableName: 'phrases',
            timestamps: true,
            paranoid: true,

        });
    }

    static associate(models) {
        this.belongsTo(models.User, {as: 'author'});

        this.belongsTo(models.CategoryPhrase, {
            as: 'category',
            foreignKey: {
                allowNull: false
            }
        })

        this.hasMany(models.TranslationPhrase, {
            constraints: false,
            as: 'titles',
            foreignKey: 'key',
            sourceKey:'translationKey'
        })
    }
}

When trying to get this model in the admin interface I get an error:
There was an error fetching records, Check out console to see more information
But there are no errors in the console. Probably it's 'cause in the use-records.ts the fetchData method does not log any errors and only shows errorFetchingRecords message.
This problem does not occur if you connect the database directly:
const adminBro = new AdminBro({     databases: [db] }

Also I get a react warning:
Warning: componentWillReceiveProps has been renamed, and is not recommended for use. See https://fb.me/react-unsafe-component-lifecycles for details.

Move data fetching code or side effects to componentDidUpdate.
If you're updating state whenever props change, refactor your code to use memoization techniques or move it to static getDerivedStateFromProps. Learn more at: https://fb.me/react-derived-state
Rename componentWillReceiveProps to UNSAFE_componentWillReceiveProps to suppress this warning in non-strict mode. In React 17.x, only the UNSAFE_ name will work. To rename all deprecated lifecycles to their new names, you can run npx react-codemod rename-unsafe-lifecycles in your project source folder.

Please update the following components: Async, Select

"admin-bro": "^2.3.0",
"admin-bro-expressjs": "^2.0.5",
"admin-bro-sequelizejs": "^0.4.2",

maybe you haven't connected with the database?

closing the issue due to the lack of the comunication


HAWQ-1338. Fixed writer process doesn't exit nicely in some case

(1) From the bt in the core file, it seems that SuppressPanic is true, we still report panic even at process exist function. We should suppress the panic if there is no side effect.
(2) Now hawq stop utility stop standby after master stopped successfully or failed to stop with 10 minutes timeout, so it seems no need to change.

LGTM

LGTM +1


Provide flat list of DwC terms for all terms

See feature request at https://github.com/tdwg/dwc-qa/issues/193:
We are currently providing a vertical and horizontal list of Darwin Core terms that are part of Simple Darwin Core:
https://github.com/tdwg/dwc/blob/master/dist/simple_dwc_horizontal.csv
https://github.com/tdwg/dwc/blob/master/dist/simple_dwc_vertical.csv
These lists are used to "programmatically check datasets for darwin core naming conventions". For completeness, we could also provide a vertical list of all Darwin Core terms (not only simple ones), named dwc_vertical.csv. Note: I don't think it makes sense to provide a horizontal version of that list, since the resourceRelationship and measurementOrFact classes are not designed to be represented in a flat (i.e. column header) format.
Like the current lists, the list could be regenerated as part of the build process.

An alternative solution might be to pull MeasurementOrFact and ResourceRelationship off of the Core Quick Reference Guide and create guides for each of them in the spirit of extensions the way ChronometricAge is done. Open to opinions.

Splitting them off in separate guides would probably be better. 👍  Would you suggest the same for UseWithIRI?

Added to build.py with output in ./dist. See 6e645c9fe0228ed1f2b5b6891c2246118dbb8705.

The file listing all terms, one per row, will be automatically generated in the future.


Remove trailing commas from JSON object in doc

r? @alexcrichton
(rust_highfive has picked a reviewer for you, use r? to override)

Thanks for the PR!
While strictly invalid it seems like this probably won't really affect anyone and it also makes it easier for us to maintain, can you expand on the motivation for this?

I can't. I made this PR because I just found that trailing commas or not.

While strictly invalid it seems like this probably won't really affect anyone and it also makes it easier for us to maintain

I agree with you. These changes look like cosmetic changes. So I close this.
Thanks for your comment!


Update image-builder.md

In the "Download the image configuration template example" command sample, % was used where / should have been used.

@kzk839 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@kof-f
Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off

Invalid command: '#sign-off'. Only the assigned author of one or more file in this PR can sign off. @kof-f

#sign-off

@Jak-MS : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Only escape identifiers with double quotes if necessary

Currently, our in auto suggestions, we're a little overzealous in our escaping of identifiers with double quotes. We should only escape when necessary.
For example, the query select * from "HumanResources"."Employee" shouldn't need any escaping.

For most of us, "dbo" is the only schema, and table names never need any escaping.  Where I work, all column names are unique, so aliasing of any kind is a rarity.
We should have a way to make all queries in the simplest and most readable form:
select * from Employees
not select * from "dbo"."Employees"
And at the moment, if I do type select * from Employees, then the DRI based join hints don't work.


forceUpdate is not a function

It seems that react doesn't have built in support for mixins when using ES6 style classes.
So I'm importing react-mixin which should help me. But I'm getting an error on changeLocale forceUpdate is not a function.
The initialData is loaded fine and I can see the right json being loaded on changeLocale. This is my component:
import React, {Component} from 'react';
import {changeLocale, i18nEventsListenerMixin, setupI18NConfiguration, i18n } from 'react-i18n-helper';
import reactMixin from 'react-mixin';

export default class Locale extends Component {

  constructor(props) {
    const initialData = {HelloWorld: 'Hello, World!'};
    super(props);
    setupI18NConfiguration({clientPath: 'static/i18n', fallBackLocale: 'en-GB', initialState: {locale: 'en-GB', data: initialData}});
  }

  state = {
    desktop: true
  };

  render() {
    return (

        {i18n('HelloWorld')}
        <button onClick={() => changeLocale('pt-PT')}>pt-PT</button>
        <button onClick={() => changeLocale('en-GB')}>en-GB</button>

    );
  }
}

reactMixin(Locale.prototype, i18nEventsListenerMixin);

Do you have an idea about what I'm doing wrong?...thank you

Hi, I found a fix for this issue.
Could you change node_modules/react-i18n-helper/dist/react-i18n-helper.js  line 27 to
this.i18nUnsubscribe = i18nStore.subscribe(this.reRenderOnI18NEvent.bind(this));
If it works I'll commit and update the project.

It works. Awesome - thanks.


test: wait for end stream in HTTP timeout integration tests

Signed-off-by: Xie Zhihao<EMAIL_ADDRESS>Commit Message: test: wait for end stream in HTTP timeout integration tests
Additional Description:
Currently, these integration tests only wait for headers and close the codec client, which will cause the response stay incomplete and lead to expect failure in an extremely slow network. The patch makes these tests wait for end stream before closing codec client.
Risk Level: Low
Testing: N/A
Docs Changes: N/A
Release Notes: N/A
Platform Specific Features: N/A

Thank you for the enhancement @zhxie !


Publish the helm chart to a registry

It would be much easier to install and track if the chart is published to a registry with a tag

Any thoughts on using:
https://github.com/helm/chart-releaser-action

I'm gonna try to take a stab at this.

Hey @toscott I'm supportive of this if you want to merge the fork back in!


'dict object' has no attribute 'celery_support'

I wanted to check out cookiecutter-django, but I can't seem to get it working.  Even upon accepting all defaults, I receive two errors.  First is 'unable to create file tests/engine.py' (permissions on directory in which I am running cookiecutter are 777).  The second, and less likely to be some environmental weirdness on my end, is the 'dict object' has no attribute 'celery_support' error.  Is this a known problem?  Are there workarounds?  Answering y or n to the celery support prompt does not make a difference.
Also, I looked through the README and CONTRIBUTORS docs, hoping to find a pointer to an IRC channel or mailing list or similar where I could ask about this issue but didn't find anything.  Is there a channel where cookiecutter-django authors hang out or a mailing list?

Where is the celery_support error popping up?
The celery channel is here: https://gitter.im/pydanny/cookiecutter-django

Here is the log:
You've cloned /home/otakucode/.cookiecutters/cookiecutter-django before. Is
it okay to delete and re-clone it? [yes]:
Cloning into 'cookiecutter-django'...
remote: Counting objects: 4717, done.
remote: Compressing objects: 100% (13/13), done.
remote: Total 4717 (delta 4), reused 0 (delta 0), pack-reused 4704
Receiving objects: 100% (4717/4717), 834.68 KiB | 0 bytes/s, done.
Resolving deltas: 100% (2962/2962), done.
Checking connectivity... done.
project_name [project_name]:
repo_name [project_name]:
author_name [Your Name]:
email [Your email]:
description [A short description of the project.]:
domain_name [example.com]:
version [0.1.0]:
timezone [UTC]:
now [2015/11/04]:
year [2015]:
use_whitenoise [y]:
use_celery [n]:
use_mailhog [n]:
use_sentry [n]:
use_newrelic [n]:
use_opbeat [n]:
windows [n]:
use_python2 [n]:
Unable to create file 'tests/engine.py'
Error message: 'dict object' has no attribute 'celery_support'
Context: {
"cookiecutter": {
"author_name": "Your Name",
"description": "A short description of the project.",
"domain_name": "example.com",
"email": "Your email",
"now": "2015/11/04",
"project_name": "project_name",
"repo_name": "project_name",
"timezone": "UTC",
"use_celery": "n",
"use_mailhog": "n",
"use_newrelic": "n",
"use_opbeat": "n",
"use_python2": "n",
"use_sentry": "n",
"use_whitenoise": "y",
"version": "0.1.0",
"windows": "n",
"year": "2015"
}
}
On Thu, Dec 10, 2015 at 4:40 PM, Jannis Gebauer<EMAIL_ADDRESS>wrote:

Where is the celery_support error popping up?
The celery channel is here: https://gitter.im/pydanny/cookiecutter-django
—
Reply to this email directly or view it on GitHub
https://github.com/pydanny/cookiecutter-django/issues/436#issuecomment-163757426
.

@otakucode thanks to report this issue.
you could try again and verify if https://github.com/pydanny/cookiecutter-django/commit/fc37d656615211ba0ce044c8cb3050181600d1cb solved the problem

Works perfectly now for me.  Thanks for the fix!  cookiecutter-django looks tremendously useful!

fixed in https://github.com/pydanny/cookiecutter-django/commit/fc37d656615211ba0ce044c8cb3050181600d1cb :dancer: :dancer::dancer:


Add more example setup

Description
Yet another status bar
Usage
Copy to eww folder and you are almost good to go.

Install required fonts and packages. More info in readme.md
Change wttr command to match your city
Change rofi i3lock command to match your setup

Showcase

Checklist
Please make sure you can check all the boxes that apply to this PR.

[X] All widgets I've added are correctly documented.
[ ] I added my changes to CHANGELOG.md, if appropriate.
[ ] The documentation in the docs/content/main directory has been adjusted to reflect my changes.
[ ] I used cargo fmt to automatically format all code before committing

OK I will close the pull request now


deps: bump sigs.k8s.io/karpenter to v0.37.1

Fixes #N/A
Description
How was this change tested?
Does this change impact docs?

[ ] Yes, PR includes docs updates
[ ] Yes, issue opened: #
[x] No

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>3
Details

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.01%) to 78.13%

Files with Coverage Reduction
New Missed Lines
%

pkg/providers/amifamily/ami.go
1
90.5%

Totals

Change from base Build<PHONE_NUMBER>9:
-0.01%

Covered Lines:
5841

Relevant Lines:
7476

💛  - Coveralls


OSPAR Birds Weighting regions

BD_WeightingRegionID  http://vocab.ices.dk/?ref=1449 is apparently missing some values that were suggested to submitters in the 2016 data call.

WeightingRegionID
Weighting   region ID description

1
Artic   Waters (OSPAR region I)

2
Greater   North Sea (OSPAR region II)

3
Celtic   Seas (OSPAR region III)

4
Bay   of Biscay and Iberian Coast (OSPAR region IV)

5
Wider   Atlantic (OSPAR region V)

1m
North   Barents Sea

1n
South   Barents Sea

1o
Norwegian   Sea

add to the vocab, check with Chris Moulton about the area definitions

@ospar-odims, are there area definitions for the WeightingRegionIDs?


Code Cleanup

Some misc code cleanup:

Constructors simplified
Reduce logic for setting animation view and duration in binding method
Removed unused methods/vars
add annotations for IdRes args

Good changes. Because of the small API changes, this will be in 2.0.


Compiler doesn‘t recognize generic of returning function

Steps to Reproduce
create
function foo<T>(T Function(Map<String,dynamic>) parse, Widget Function(T) foo2);

Now when calling
foo(parseFoo(), (t) => returnSomething());

t isn't recognize as T rather as Object?

Hi @PAException Please share the output of flutter doctor -v

[√] Flutter (Channel stable, 2.5.0, on Microsoft Windows [Version 10.0.19042.1165], locale de-DE)
    • Flutter version 2.5.0 at C:\src\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 4cc385b4b8 (32 hours ago), 2021-09-07 23:01:49 -0700
    • Engine revision f0826da7ef
    • Dart version 2.14.0

foo(parseFoo(), (t) => returnSomething());

t isn't recognize as T rather as Object?

You must explicitly specify the generic type
https://github.com/dart-lang/sdk/issues/42766

Hi @PAException Thanks for the info, @nikitadol is right it is working as intended you need to explicitly specify the type
void main() {
  foo<T>(T Function(Map<String, dynamic>) parse, Widget Function(T) foo2) {}

  String Function(Map<String, dynamic> p1) parseFoo() {
    return (z) {
      return z['test'];
    };
  }

  Widget returnSomething() {
    return Container();
  }

  foo(parseFoo(), (t) => returnSomething()); /// t is Object?

  foo<String>(parseFoo(), (t) => returnSomething());  /// t is String
}

Closing this issue as working as intended if you disagree feel free to write in the comments and I will reopen it.
Thank you.

Thanks for the fast support. After you explained it I just understood! Makes totally sense... Sorry to bother you!


Watcher UX: status after deleting IFE

scenario:

bob starts ife
nobody piggybacks, wait for first phase of challenge period to pass
call deleteNonPiggybackedInFlightExit on paymentExitGame contract
bob gets his bond back, ife is cleared, calling process exits confirms that no exits exist in the queue

issue:
status.get still shows the IFE. shouldn't the exit be cleared from the in_flight_exits array?
reference:
"One can call this to release the bond back to its owner and clean up the state of in-flight exit. "
https://github.com/omisego/plasma-contracts/blob/914b6ebf7897826872eb684db40db0081b4d3abb/plasma_framework/docs/integration-docs/integration-doc.md#delete-an-in-flight-exit

Additionally the watcher informs that deleted IFEs can be piggy backed.

Let's track in here pls. I think the gist of this here is tracking deleted state of an IFE which would suppress all the response data about this IFE:

piggyback available
in flight exit
invalid piggybacks
canonicity challenges required

I think this should be tackled in conjunction with #1318 - IFE being deleted should just be modeled explicitly as yet another state of an IFE.


How to get result of knex delete operation inside a for loop

Environment
Knex version: 0.95.11
Database + version: postgres, pg version 8.6.0
OS: Windows
Bug (?)
The code is simple, however it involves getting Promises resolved after each step in a for loop.
async function customDelete ( tableList) {
//tableList is an object, keys = tables, values corresponding row_ids to delete in each table
  const deleted = { };
  for (const table of tableList) {
    if (table in aCheckList) {
      console.log(">> table:", table, "-- ids to delete: ", tableList[table]);

      deleted[table] = knex(table).whereIn('id', tableList[table]).del()
      .returning('id').
      .then( result => {
         deleted[table] = result;
         notDeleted = tableList[table].filter(x => !result.includes(x));
         console.log("***deleted | notDeleted:", result, notDeleted);
         if(notDeleted.length){
           console.warn('deleting', table, 'no such object(s)', notDeleted);
         }
         return result;
       });
    }
  }
  return deleted;
}

and the part that call customDelete (tableList) :
if (req.body && req.body.sync && req.body.sync.delete) {
      sync.deleted = await customDelete (req.body.sync.delete);
      console.log("deleted rows:", sync.deleted)
    }

Expected result: a deleted object: deleted = { "table1": [1, 2, 4, 5, 6], "table2": [2,3,4,5]} etc.
Without await, for knex(table).del( ) the return object is Promise <pending>, with await, the script hangs forever.
The question is, how can I obtain the final report of the deleted rows in each table in the deleted object by that loop?
I tried all methods such as Promise.all, reduce( ) but still facing the same problem (deleted object not resolved).

Try this, is working for me locally.
In the test database I only have ids 5 that is why it only prints 5.
I write in a functional style hope that is not a turn off, if it is I can help you understandint such piece.
You can derived the non deleted easilly, I did not write the code but you only need to compare the return value of the custom delete with the original argument.

const knex = require("knex");
// import { knex } from "knex";

const conn = knex({
  client: "pg",
  connection: "postgresql://postgres:password@localhost:5432/tempo"
});

const customDelete = (tableList) => {
  const promises = Object.entries(tableList)
    .map(([table, ids]) =>
      conn(table)
        .whereIn("id", ids)
        .del()
        .returning("id")
    );
  return Promise.all(promises)
    .then(res =>
      Object.keys(tableList)
        .reduce(([index, acc], curr) =>
          [index + 1, {...acc, [curr]: res[index]}]
          , [0, {}])[1]
    );
}

const sut = {
  a: [1, 2, 3, 4, 5],
  b: [1, 2, 3, 4, 5],
};

customDelete(sut)
  .then(res => {
    const pretty = JSON.stringify(res, null, 2);
    console.log(pretty);
  });

@ggallovalle Thank you, it works!
But so it turns out that my other attempts to get the batch result in an object worked too. I just found out the reason the script hanged was that there's a query to LOCK one of the table somewhere above! 😭 My bad.


chore(Dependencies) : update zio-logging-slf4j from 0.5.14 to 2.1.5

Updates dev.zio:zio-logging-slf4j from 0.5.14 to 2.1.5.
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!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "dev.zio", artifactId = "zio-logging-slf4j" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "dev.zio", artifactId = "zio-logging-slf4j" }
}]

labels: library-update, early-semver-major, semver-spec-major, commit-count:1

Superseded by #551.


API for inline styles

Do you plan to add API support for inline styles attribute in the future?
expect(wrapper.find('.my-button').style).to.equal({ width: '100px' });

Something like the existing hasClass
expect(wrapper.find('.my-button').hasClass('disabled')).to.equal(true);

I'm not sure what the benefit would be; hasClass makes sense because className is otherwise a space-separated list of strings, so checking it correctly for a single class is non-trivial. style, however, is just an object, and making assertions on object properties is a solved problem.

Inline styles can be generated depending on multiple props and conditions. Maybe i get it wrong or miss something, but don't know how to do test for this:
Example
const { prop1 } = props
const { prop2 } = context
const styles = {}
if (prop1) styles.width = '100px'
if (prop2) styles.width = '200px'
return <div style={styles}>

There are couple question about the same on stackoverflow without answer yet.

The same way you test any plain object:
expect(wrapper.prop('styles')).to.contain.all.keys({ width: '100px' });
/* or */
expect(wrapper.prop('styles')).to.eql({ width: '200px' });
/* or */
expect(wrapper.prop('styles')).to.eql({});

@ljharb Cool thanks


H/M/L keys first move to head of a line before work

In the middle of one line, type shift-h or H, it will first go to line head like you type 0, then takes effect to go to the top.
M/L keys have similar behavior.
OS: win10
vscode: 1.48.2
neovim extension: v0.0.62
neovim: 0.5.0

BTW, this worked on 0.0.52.

Another interesting behavior: if you at the beginning of the line - this keys work.


Add Template and ISO Zones tab

Implement a basic Zones tab for both template and ISO resource view (tabular listing only, no need to show its zone-specific detail view as it will be redundant of the template/ISO detail view).
Example, zone tab for ISO:

Example, zone tab for template:

Here is an example of how to implement a tab component for a resource view:
https://github.com/apache/cloudstack-primate/commit/092fc3bb6feef92f1e23eb2b398835f816820656

@utchoang - this is related to other template/iso tasks. The task is to simply add a zones tab that lists the template/ISO status per zone. Can you pick this up next? cc @svenvogel