TimeStamp exception in valid timestamp

Using the latest KinesisTap release on IIS8 Logs I see failures to parse the lines:
2020-10-22 00:00:00.9431 MyIISLogSource ERROR 2020-10-21 21:59:01 W3SVC1 MY_MACHINE <IP_ADDRESS> GET /version.html - 80 - <IP_ADDRESS> HTTP/1.1 - - - intservices.exp.companyinternal.net 200 0 0 1014 87 0

2020-10-22 00:00:00.9431 MyIISLogSource ERROR Failed to get time stamp in D:\Logs\MY_MACHINE\W3SVC1\iis-2020-10-21.log, 45270226: System.NullReferenceException: Object reference not set to an instance of an object.
@stacktrace_ref 144 12802<PHONE_NUMBER>

@johnkeates Can you post your configuration for this source?

@dhhoang This is the source definition:
{
            "Directory": "D:\\Logs\\IIS\\W3SVC1",
            "FileNameFilter": "*.log",
            "Id": "MyIISLogSource",
	    "TimeZoneKind": "UTC",
            "SourceType": "W3SVCLogSource"
}

Looking at an identical source with a lower volume of logs, that one does work with no errors. From the logs I would expect a full StackTrace but there doesn't seem to be one, NLog.xml on Debug level doesn't add anything either (but I suspected as much from looking at the sources).
I don't understand why it would not have a record instance at that point (that's what that instance pointer refers to, right?).
The log data is exactly the same as all other IIS logs I have, no difference in line endings or binary formatting or anything like that.

@johnkeates  Can you verify that in your log files, there's a header line that specifies the fields according to https://docs.microsoft.com/en-us/previous-versions/iis/6.0-sdk/ms525807(v=vs.90)#w3c-extended-log-file-format
The header line usually starts with "#Fields:", for ex #Fields: time c-ip cs-method cs-uri-stem sc-status cs-version

@dhhoang yes, it exists, but it doesn't always seem to be at the top of the file

@johnkeates could you profile some sample lines with that line included?

Here is a sample from a log from today:
#Software: Microsoft Internet Information Services 8.5
#Version: 1.0
#Date: 2020-11-02 00:00:00
#Fields: date time s-sitename s-computername s-ip cs-method cs-uri-stem cs-uri-query s-port cs-username c-ip cs-version cs(User-Agent) cs(Cookie) cs(Referer) cs-host sc-status sc-substatus sc-win32-status sc-bytes cs-bytes time-taken

@johnkeates Does the "#Fields" line line repeat in your log? If so, does it repeat daily?
The agent seems to have hit a case where it's not able to figure out the field mapping from the "#Fields" line if it's not at the start of the log file.

@dhhoang Yes, it does repeat in the logs. In the file used in the example it was on line 100 and on line 1149769.

@johnkeates  it looks like if the #Fields line is not at the top of the file, the agent is unable to figure out the mapping and hence unable to parse the subsequent lines.
Would you want to have the option to specify a default field mapping?

@dhhoang A default field mapping would be great; in theory the mapping of a file is known thread of time for the W3SVC log format anyway. There are multiple versions of IIS using slightly more or less fields, so being able to configure a default to match is a great method to solve this.
Does it still update the internally known mapping once it encounters a new set of #Fields in the log?

Does it still update the internally known mapping once it encounters a new set of #Fields in the log?

I think it would make sense to update the mapping after reading that line.

The fix is in progress. I'll update this issue accordingly.

Thanks for the update!

@johnkeates Please give a new beta https://github.com/awslabs/kinesis-agent-windows/releases/tag/<IP_ADDRESS> a try and see if it fixes your issue


Frontend: Save commands history

The idea of the ticket is to call and save the history API /api/project-histories each time the user calls an API.
In the future it will allow us to

manage the state of API buttons
display the history of the commands called.

@Franceq34 : can you add some description to give some contexts to our users plz?

@Franceq34 : don't forget your bounty here

$200 bounty claimed => https://opencollective.com/generator-jhipster/expenses/76378

@Franceq34 : approved


Pass xhr settings to authorizer.

In response to issue #862 and for my own similar needs, I added a parameter to SessionService.authorize() to permit access to the XHR settings.

The authorizer can authorize any code block really and has no knowledge of XHR. If you're not using the DataAdapterMixin you'd be using it through the session sth. like this:
this.get('session').authorize('authorizer:some', (headerName, headerValue) => {
  xhr.setRequestHeader(headerName, headerValue);
});

so no need to pass XHR options.

From what you say, I understand that DataAdapterMixin is tied in a certain way to the authorizer. So if my authorizer needs to access the xhr settings, I would need to write a new data adapter mixin with a different beforeSend, something like this:
ajaxOptions() {
const authorizer = this.get('authorizer');
Ember.assert("You're using the DataAdapterMixin without specifying an authorizer. Please add authorizer: 'authorizer:application' to your adapter.", Ember.isPresent(authorizer));
let hash = this._super(...arguments);
let { beforeSend } = hash;

hash.beforeSend = (xhr, settings) => {
  this.get('session').authorize(authorizer, (publicKey, secretKey) => {
    var stringToSign = extractDataFromXHRSettings(settings);
    var signature = HMAC_SHA256(stringToSign, secretKey);
    xhr.setRequestHeader('Authorization', Credential=' + publicKey + ',Signature=' + signature);
  }, settings);
  if (beforeSend) {
    beforeSend(xhr, settings);
  }
};
return hash;

},

Authorizers just authorize a block (which doesn't necessarily issue an XHR request) of code by converting the session data that was acquired by the authenticator when it authenticated the session to some other data, e.g. a header name and value. If you need to convert that data further you'd have to do so in the block that the session's authorize method calls with the authorization data - so yes, you'd have to define your own DataAdapterMixin like above.

Thanks for the confirmation!

This will actually be added via #870

oops


aur-build: warn when a repose-made repo is used

aurutils does not currently support repose.
But if the user has a repose created database repo-add calls will give cryptic error messages.
Since aurutils supported repose in the past, it may be a nice for us to detect such cases and give the user a more helpful error message.

Turn back, for this way lies madness

Maybe a mention in the man page (with the specific error message) suffices?

It's been nearly 2 years since the 2.0.0 release which dropped repose support, so let's just leave repose to rest.


Update changelog for release 2.0.2

Build is green, no code has been touched, I'll go ahead and tag 2.0.2 while waiting for a review..


Fix checking delta batch in readDeltaTimestamps in batched store when merging

the readDeltaTimestamps should include the whole firstDeltaBatch for the merge operation to start

merge when green

Thanks for merging, will send another rb for UTs

Guys, I'm not convinced this is the right approach.
My concern here is that the method contract really does not say anything about having to read a full next batch. Why is that needed here? What I would like to see instead is a method that can optionally by called after this method that ensures that we can read at least one batch. However, for joining, you don't need this at all, you just read the previous batch and join as much as you can. I think the joining case is not being thought through here.
Also, given we think we are having issues, it would be really great to find some test to check some kind of relevant property. We have a few integration tests, but I don't think they are covering what is going on.
At the very least we to update the comments to this method to explain what the contract is and how it is to be used. Right now, I'm afraid it is a bit muddy. I think I know what it was supposed to do, but now, it is a bit complex. The old contract was this: knowing that a PipeFactory is a function that given a requested Interval of time responds with an Interval it can supply and a function to add that read to a FlowDef. readAfterLastBatch takes a PipeFactory and returns a new one that reads the same data over a changed Interval. The new Interval ends at the same time as the requested interval but starts immediately after the last written batch of the store on disk.
This code was factored out to share it between the joining code and the merge (summing) code, so the contract needs to make sense on both sides.
I don't know exactly what issue we are trying to fix here, but we are forcing all joins to be over at least one batch. Is that not going to increase latency of some complex jobs (with internal joins and stores on different batchers)? How can we check that?
Again, my proposed solution is just add a new method: atLeastOneBatch or something, that takes a PipeFactory, and if the requested range does not completely cover a single batch, then we fail the plan. We might want to do that for the merging side, but I don't think we need to force it on the joining side.

how about in the mergeMethod, we call atLeastOneBatch after calling readAfterLastBatch, so it becomes like:
  readBatchedResult <- readAfterLastBatch(delta)
  (actualLast, snapshot, deltaFlow2Pipe) = readBatchedResult
 _  <- atLeastOneBatch(actualLast)

@tsdeng sounds good to me.


FDG-9218 iFrame test PR (Mobile (iPhone) preview published reports is not working in Prod)

@wyattch test failed :(


implement new rules for formatting numbers

Closes #128

@philipkueng could you deploy this branch with fixture data to st-test for testing?


Problems with Cut/Paste shortcut

VSCode Version: 1.1.1 (insider)
OS Version: Windows 10

Steps to Reproduce:

Select some text with the keyboard
Press ctrl+x to cut the text
Move the keyboard somewhere else using the cursor keys
Press ctrl+v to paste the text

What happens: the text is correctly removed from the editor (step 2) but it isn't pasted at the cursor position
What should happen: text from clipboard should be appended at mouse position
It happened already a few times to. Unfortunately I didn't find a pattern yet.
Note that attempting to paste the text in another app doesn't work either so it seems the problem is in the cut action which for some reason doesn't work correctly: the text is correctly removed from my editor but isn't in the clipboard.
This has been happening for at least 3 updates of VSCode.

@warpdesign Does this issue reproduce when running code --disable-extensions (i.e. could it be cause by an extension that hijacks ctrl+x?). Is it possible that you have installed some other software on your machine that might interact with/watch the clipboard?

I just noticed I have installed clip-x: that's a Windows app which keeps the clipboard history.
I don't think I have any VSCode extension that interacts with the clipboard.
I have disabled Clip-X to see if it's causing the problem (I noticed that when I encounter the bug, I have an empty entry in clip-x).
Then I'll start VSCode with extensions disabled if I'm still having the problem.

@alexandrudima After having disabled clip-x I no longer had the problem. I guess the problem wasn't with VSCode after all.


add win

Thanks to https://github.com/conda-forge/r-maps-feedstock/pull/2 we can enable Windows here.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Remove curl install and hardcoded tests accounts

Fixes #137

Remove curl install as it's already present in quorum image
Remove hardcoded test accounts before putting genesis.json to configmap until test accounts became optional in quorum-genesis-tool

Hi @aliusmiles thanks for this, Im going to close this one as its superceeded by #141-#145
We updated the QGT today and released afresh, and have updated this repo as well.
Please feel free to reopen if there is something we missed
Cheers


Add MFA tests

Some errors had been logged to Sentry, so I had wanted to add tests to confirm behavior.
Moto doesn't presently support MFA setup.
See my open issues and draft PR: https://github.com/getmoto/moto/issues/created_by/jpmckinney
Here's a patch for a start to some tests once Moto is updated.
diff --git a/tests/__init__.py b/tests/__init__.py
index 948c3ca..c8b30c6 100644
--- a/tests/__init__.py
+++ b/tests/__init__.py
@@ -72,6 +72,14 @@ def create_user(session, aws_client, *, email, **kwargs):
         Username=email,
         UserAttributes=[{"Name": "email_verified", "Value": "true"}],
     )
+    assert response.get("ChallengeName") == "MFA_SETUP", response
+    response = aws_client.cognito.associate_software_token(Session=response["Session"])
+
+    # setup_mfa()
+    aws_client.cognito.verify_software_token(Session=response["Session"], UserCode="123456")
+
+    # login()
+    response = aws_client.initiate_auth(email, "12345-UPPER-lower")

     return {"Authorization": "Bearer " + response["AuthenticationResult"]["AccessToken"]}

diff --git a/tests/routers/test_users.py b/tests/routers/test_users.py
index 5cf91a6..c04c69e 100644
--- a/tests/routers/test_users.py
+++ b/tests/routers/test_users.py
@@ -64,6 +64,21 @@ def test_duplicate_user(client, admin_header, user_payload):
     assert response.json() == {"detail": _("User with that email already exists")}

+def test_login(client, admin_header, admin_email):
+    client.get("/users/logout", headers=admin_header)
+
+    response = client.post(
+        "/users/login",
+        json={
+            "username": admin_email,
+            "password": "12345-UPPER-lower",
+        },
+    )
+
+    assert response.json() == {}
+    assert response.status_code == status.HTTP_200_OK
+
+
 def test_login_invalid_username(client):
     response = client.post("/users/login", json={"username": "nonexistent"})

@@ -71,6 +86,16 @@ def test_login_invalid_username(client):
     assert response.json() == {"detail": "Invalid username or password"}

+def test_login_invalid_password(client):
+    pass
+    # TODO
+
+
+def test_login_invalid_mfa(client):
+    pass
+    # TODO
+
+
 def test_logout(client, admin_header):
     response = client.get("/users/logout", headers=admin_header)

Moto did recently add in 5.0.12 3 weeks ago

CognitoIDP: initiate_auth() now supports USERNAME_PASSWORD_AUTH and SMS/Software Token MFA

So, we can probably mock a user with MFA a different way (e.g. opt-in the user via set_user_mfa_preference or admin_set_user_mfa_preference), instead of trying to test the full lifecycle of setting up MFA and then logging in.
https://github.com/getmoto/moto/blob/master/CHANGELOG.md#5012


linuxKernel.kernels.linux_rpi4: 5.15.32 -> 5.15.74

Description of changes
raspberrypi-firmware: 1.20220331 -> 1.20221028
linuxKernel.kernels.linux_rpi4: 5.15.32 -> 5.15.74
fixes #199828
I'd appreciate any tests with native aarch64 (My raspberry is not up to the task..)
Things done

Built on platform(s)

[ ] x86_64-linux
[x] aarch64-linux (with qemu)
[ ] x86_64-darwin
[ ] aarch64-darwin

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

i cross compiled from an x86 host (via the nixpkgs crossSystem = “aarch64-linux”; option) and it’s running fine on a raspberry pi 400 👍


Fix bug with datetime not being serialized

Also seeing another issue where self.to_json() output is different thatn self.to_dict(). self.to_json() has the inner keys as strings whereas self.to_dict has the inner keys as integers.

Also seeing another issue where self.to_json()s output is different than that of self.to_dict(). self.to_json() has the inner keys as strings whereas self.to_dict() has the inner keys as integers. Will this have an impact somewhere?

Airflow's XCom still can't serialize datetime, set and other types, so we can't use self.to_dict() whereas self.to_json() converts it to string, so we are good.

Also look like the tests are not running for that file as the file name is missing the test_ prefix, we will need to rename it to test_pandas.py and hence codecoverage is reporting that the patched lines are not covered

Added pre-commit hook so that we detect such cases :)


spell/recipe API

Please can you add a method for the new spell/recipe API? At the moment it's quite small, but I'm hoping they will add a lot of data here soon.
the example given was http://us.battle.net/api/wow/recipe/33994

Ok i will start working on it tomorrow

Awesome! Thanks.
On 12/28/11, Daniel Cannon
<EMAIL_ADDRESS>wrote:

Ok i will start working on it tomorrow

Reply to this email directly or view it on GitHub:
https://github.com/dancannon/PHP-WowApi/issues/31#issuecomment-3297044

Ok I have added the api and tests. I have not yet added the documentation but it is almost identical to the items API.


Let querier plugin decide to delete state object on error

Add option in querier plugin to specify whether state object should be deleted if an error occur during query. This allows querier to recover in case of unexpected environment related exceptions. Mistral querier is refactored to delete state object on workflow completion or when the workflow or task references no longer exists.

Codecov Report

Merging #3182 into master will increase coverage by -0.07%.

@@            Coverage Diff             @@
##           master    #3182      +/-   ##
==========================================
- Coverage    77.6%   77.53%   -0.07%
==========================================
  Files         432      433       +1
  Lines       22318    22323       +5
==========================================
- Hits        17318    17306      -12
- Misses       5000     5017      +17

Impacted Files
Coverage Δ

st2common/st2common/exceptions/resultstracker.py
100% <100%> (ø)

st2common/st2common/query/base.py
73.68% <40%> (+5.75%)
:white_check_mark:

st2debug/st2debug/utils/git_utils.py
72.73% <ø> (-18.18%)
:x:

st2api/st2api/controllers/v1/triggers.py
71.43% <ø> (-7.88%)
:x:

...on/st2common/transport/connection_retry_wrapper.py
75% <ø> (-5.77%)
:x:

...ctions/st2actions/resultstracker/resultstracker.py
72.22% <ø> (-5.56%)
:x:

st2common/st2common/transport/consumers.py
86.32% <ø> (-3.16%)
:x:

st2common/st2common/bootstrap/sensorsregistrar.py
82.52% <ø> (-2.91%)
:x:

st2common/st2common/services/triggerwatcher.py
85.92% <ø> (-2.82%)
:x:

st2common/st2common/models/db/init.py
97.22% <ø> (-1.67%)
:x:

... and 23 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e867769...0b8d565. Read the comment docs.

From high level - LGTM.
Looking at the code, it doesn't seem to be a way for user / operator to enable this option for Mistral since it's disabled by default. Or am I missing something?

It's not an option for the user/operator.  It's a querier plugin level option.

@Kami concerns/comments addressed.


[ENH] Add variant for multiple column renaming

Same thing as we've done before - <func>_column to <func>_columns list iteration.

I would like to work on this enhancement.


Wrong path generated in setUp.pl on Windows

All systemtest builds are failing with following error:
Unrecognized escape \j passed through at C:/Users/jenkins/workspace/Test-sanity.system-JDK8-win_x86-64_cmprssptrs/openjdk-tests/\TestConfig\test_output_15543497723942\DaaLoadTest_daa1_0/20190403-224934-DaaLoadTest/setUp.pl line 18.
Unrecognized escape \w passed through at C:/Users/jenkins/workspace/Test-sanity.system-JDK8-win_x86-64_cmprssptrs/openjdk-tests/\TestConfig\test_output_15543497723942\DaaLoadTest_daa1_0/20190403-224934-DaaLoadTest/setUp.pl line 18.
Unrecognized escape \T passed through at C:/Users/jenkins/workspace/Test-sanity.system-JDK8-win_x86-64_cmprssptrs/openjdk-tests/\TestConfig\test_output_15543497723942\DaaLoadTest_daa1_0/20190403-224934-DaaLoadTest/setUp.pl line 18.
Unrecognized escape \j passed through at C:/Users/jenkins/workspace/Test-sanity.system-JDK8-win_x86-64_cmprssptrs/openjdk-tests/\TestConfig\test_output_15543497723942\DaaLoadTest_daa1_0/20190403-224934-DaaLoadTest/setUp.pl line 18.
Unrecognized escape \s passed through at C:/Users/jenkins/workspace/Test-sanity.system-JDK8-win_x86-64_cmprssptrs/openjdk-tests/\TestConfig\test_output_15543497723942\DaaLoadTest_daa1_0/20190403-224934-DaaLoadTest/setUp.pl line 18.
Unrecognized escape \s passed through at C:/Users/jenkins/workspace/Test-sanity.system-JDK8-win_x86-64_cmprssptrs/openjdk-tests/\TestConfig\test_output_15543497723942\DaaLoadTest_daa1_0/20190403-224934-DaaLoadTest/setUp.pl line 18.
Unrecognized escape \s passed through at C:/Users/jenkins/workspace/Test-sanity.system-JDK8-win_x86-64_cmprssptrs/openjdk-tests/\TestConfig\test_output_15543497723942\DaaLoadTest_daa1_0/20190403-224934-DaaLoadTest/setUp.pl line 18.
Unrecognized escape \s passed through at C:/Users/jenkins/workspace/Test-sanity.system-JDK8-win_x86-64_cmprssptrs/openjdk-tests/\TestConfig\test_output_15543497723942\DaaLoadTest_daa1_0/20190403-224934-DaaLoadTest/setUp.pl line 18.
Can't locate stf/stfUtility.pm in @INC (you may need to install the stf::stfUtility module) (@INC contains: C:SERSJENKINSWORKSPACETEST-SANITY.SYSTEM-JDK8-WIN_X86-64_CMPRSSPTRSJVMTESTSYSTEMTESTSTFSTF.CORESCRIPTS C:/Strawberry/perl/site/lib C:/Strawberry/perl/vendor/lib C:/Strawberry/perl/lib) at C:/Users/jenkins/workspace/Test-sanity.system-JDK8-win_x86-64_cmprssptrs/openjdk-tests/\TestConfig\test_output_15543497723942\DaaLoadTest_daa1_0/20190403-224934-DaaLoadTest/setUp.pl line 24.

https://ci.eclipse.org/openj9/job/Test-sanity.system-JDK11-win_x86-64_cmprssptrs/216/

In the generated setUp.pl, I believe the cause of the above error messages is this line:
use lib "C:\Users\jenkins\workspace\openjdk8_hs_systemtest_x86-64_windows\jvmtest\systemtest\stf\stf.core\scripts";

Most likely, the call to normalize() , i.e., String normalizedPath = p1.normalize().toString(); in DirectoryRef is messing path separators up on Windows.

Confirming: adding a replaceAll("\\\\", "/")  while on Windows on the path returned by the noramilze() method fixes the issue. Tested using Internal Jenkins Grinder.


Update install-unifi.sh

Force update snappyjava when openjdk8 is update. Solve problem adopt/disconect in version 5.5.20

@ReinaldoFeitosa How about updating the branch?

How to resolved conflicts?

because my pullrequest was not accepted?

@ReinaldoFeitosa Usually you do a pull from upstream/master to your branch, solve conflicts locally, push to your fork/branch. GitHub will detect that your fork was updated and will refresh the PR,the conflict will go away.
Be sure that your fork is in sync with upstream (this repo).

Is it necessary to delete (and thereby reinstall) snappyjava now that we're force installing everything?

Yes, but you need to download all unnecessary packages.
If java is updated you need to reinstall snappyjava


Status Bar Hidden Always

What if we want the status bar to always be hidden and not have the note, toast, etc. show it?

Thank you for reporting the issue.
This is currently not supported but I'll look into.

You can check out release 0.2.3. Enjoy! 🎉


Create sink/metric with simple name, rather than its full path

Fixes https://github.com/GoogleCloudPlatform/gcloud-python/issues/1890

@bjwatson Thanks for the detective work!

@tseaver No problem.


ORA1761 New “strict” query that returns latest inferences and losses when both are fully available

This includes both inferences and losses

Also are we worried about a possible infinite loop? Should we limit the iterations to 10 epochs back or something?

I think default page limit makes sense, it's kind of like looking through that many "pages" of inferences before giving up so we don't have something like an infinite loop.

Ok, this is updated and ready for re-review. I recorded the relevant information so it can be used to determine the latest blocks for reputer and works payloads


Fix windows sfdx shell quoting

Critical Changes
Changes
Issues Closed

Hmm, are we not reporting coverage from the windows build, or is it not running the right code?


HHH-14139 BasicBinder Trace Logging Causes Duplicated Message

https://hibernate.atlassian.net/browse/HHH-14139
The bug is as follows. Suppose one string BasicBinder field contains the "{}" string, the corresponding logging statement would end up as below:
binding parameter [2] as [VARCHAR] - [binding parameter [2] as [VARCHAR] - [{}]]

not expected statement as:
binding parameter [2] as [VARCHAR] - [{}]

The root cause is a bug which was fixed by me in jboss-logging:3.4.2.Final: https://issues.redhat.com/browse/JBLOGGING-147?_sscc=t.
This is the only bug fixed in the new version of jboss-logging: https://github.com/jboss-logging/jboss-logging/releases/tag/3.4.2.Final. After we bump our dependency fomr 3.4.1 to it, the issue would be gone automatically.
This bug is only reproducible in SL4J logging framework (as common in Spring Boot project), but not reproducible in our Hibernate testing framework which is based on jboss logging framework.
It is hard to reproduce this bug in our existing Hibernate logging framework but it is pretty easy to reproduce it externally as per the comment https://hibernate.atlassian.net/browse/HHH-14139?focusedCommentId=107034.
Let me know whether I need to reproduce this bug in our testing code (we need to introduce SLF4J and maybe logback Logger dependencies, which might seem overkill). In theory it would be possible but I am not sure whether it is worthwhile or not.

Thanks!


Turn off autocompletion in Text

When I use Tinymist with VS Code it will always autocomplete my words, even when I'm writing text.
Is there a way to deactivate this?
I only want to have autocompletion when I'm typing something like # or @ and or when i'm in code mode.

This is a feature of the VS Code, and you can ask google for the solution.


Google_Service_Exception::getMessage returns string but ::getErrors returns null

This one's not about the message, but the delivery.
try {
    $service->users_messages->insert(DELEGATE_EMAIL_ADDRESS, $msg);
} catch (Google_Service_Exception $e) {
    if ($msg = $e->getMessage()) {
        acapApp::catchProcessMessage('API Service says: ['.$e->getCode().'] '.$msg, ACAP_WARNING);
    }

    if ($errs = $e->getErrors()) {
        foreach ($errs as $errAry) {
            acapApp::catchProcessMessage('API Service says: ['.$errAry['reason'].'] '.$errAry['message'], ACAP_WARNING);
        }
    }

    return false;
}

yields...

[x] API Service says: [401] { "error": "unauthorized_client", "error_description": "Unauthorized client or scope in request." }
[ ] But ::getErrors returns only null

Is this somehow expected behavior?

The getErrors function is for errors that return from the APIs in this format:
{
    "error": {
        "errors": [
            { "code": 400, "message": "error message" }
        ]
    }
}

Yeah this isn't great in the scenario you described, but that's an auth error, and has a different error format than the API errors. So unfortunately this particular issue is probably WAD.

um..."WAD"?
On Fri, Aug 12, 2016 at 6:54 PM Brent Shaffer<EMAIL_ADDRESS>wrote:

The getErrors function is for errors that return from the APIs in this
format:
{
"error": {
"errors": [
{ "code": 400, "message": "error message" }
]
}
}
Yeah this isn't great in the scenario you described, but that's an auth
error, and has a different error format than the API errors. So
unfortunately this particular issue is probably WAD.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/google/google-api-php-client/issues/1027#issuecomment-239578969,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AANbefpbBc0mq2oIfOaG_V7b_Rxk4tuyks5qfPmmgaJpZM4Jiaml
.

Working as Designed

who gnu
On Fri, Aug 12, 2016 at 7:28 PM Brent Shaffer<EMAIL_ADDRESS>wrote:

Working as Designed
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/google/google-api-php-client/issues/1027#issuecomment-239583412,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AANbebhpTw2jRnEE8DVgrUkFnzk2Uf2nks5qfQGRgaJpZM4Jiaml
.

We could check for the auth error format also.


Provide CA-Certificate for connecting to gitlab (on-premise) with own Certificate Authority.

Hi there,
we installed devtron on kubernetes via helm-chart...  after install we tried to "configure gitops" by adding our on-premise gitlab ..  problem is that we have a private certificate authority for ssl, tls, etc....  when adding the host, username, personal access token the error is:
Devtron was unable to perform the following actions.
error in connecting with GITLAB: Get "https://gitlab.ourdomain.local/api/v4/groups/14": x509: certificate signed by unknown authority

how can i add our ca-certificate to devtron?
devtron-discord-channel asked me to open an issue on this.
thanks and best regards
zen
AB#433

Hi, @zenhighzer have you found a workaround for this?
Because I encounter the same problem with GitLab on-premise private cert connection

Hi @adeqecil,
sorry, we didn´t tried to fix this issue.
But I would suggest to grep all images listed in the helm-charts, inject the certificates and push the images to a private registry and also modify the helm-charts for pulling images from private registry ...  but for our team this is too much effort just for trying out devtron..  idk if there is maybe an option to provide certificates via. helm-chart, configmaps,secrets, etc., which would be cool
BR
zen

Am facing the exact same issue, do you know if this is fixed to manually provide or ignore the SSL validation for GitOps Gitlab host


createOwn vs createAny

Hi, @onury!
At first, I would like to thank you for such a great library!
I have one questions about createOwn and createAny methods. I don't understand how to use them from a logical point of view. If I have readOwn / readAny it's clear for me that the user of some role can read only his own resource or can read any resources (e.g. resources of the other users). The same with updateOwn / updateAny - the user can update only his own resource or any resources. And the last one is deleteOwn / deleteAny - for example, the user can delete only his own resource but the admin can delete any resources (e.g. resources of the other users).
But what about createOwn and createAny. Let's imagine that I have two roles:

user
admin

Also, in my DB I have the posts table. The user and admin can create posts.
How should I design my AC?
Let's take this grants:
let grantsObject = {
    admin: {
        posts: {
            'create:any': ['*'],
            'read:any': ['*'],
            'update:any': ['*'],
            'delete:any': ['*']
        }
    },
    user: {
        posts: {
            'create:own': ['*'],
            'read:own': ['*'],
            'update:own': ['*'],
            'delete:own': ['*']
        }
    }
};

const ac = new AccessControl(grantsObject);

What does create:any for the admin role mean? Does it mean that the admin can create posts for any other users (e.g. the admin can create the post and assign its id to some user)? I am confused with this logic.
And what about create:own for the user role? Does it mean that the user can create the posts only ... for himself?
I have read the guide and FAQ and I understand that the developer (e.g. me) should make decisions but it's hard for me to understand the idea of createOwn and createAny concepts.
Thank you for your time.

Hi, it might reference to your own business logic, depends on what the policy you have for own.
If the own records - are records which were created by a user (creator_id automatically added) so I think create:any and create:own the same things.
But if your business logic says that OWN records are records which were assigned to you by someone else, that this is a big difference. create:own allows you to create a record and assign it to you only, for example. create:any allows you to create a record and assign it to somebody...It's just a my vision, anyway it can be controlled by attributes access...
Maybe it was made the way like this just for the consistency purposes... Also curious about this.

Hi @turakvlad. Thanks and you're welcome.
First of all, you do not need to make use of each and every action/possession for each role and resource. Use only what makes sense for your application.

What does create:any for the admin role mean? Does it mean that the admin can create posts for any other users (e.g. the admin can create the post and assign its id to some user)?

Actually, that's a question only you and your app can answer.

If that's what you need, that's definitely proper:
"admins creating user posts on other user's behalf"
example pseudo logic: Users may ask admins to post on their behalf.
example post: "I won't be answering until next week."
In some other app logic, it could mean:
"admins creating admin posts on any user-timeline"
example pseudo logic: Admins inspect timelines for improper/offensive content.
example post: "This user account is suspended"

Within your app logic, to make sense of this; you should first define what "own" and "any" stands for (for what role and resource) and do these possessions need distinguished control of access.
A Detailed Example:
Sales representatives are calling a list of registered customers on the phone and;

Sell these customer some products.

You could represent this in AC as:
ac.grant('salesRep').createAny('order')

Customers

Can of course, buy products themselves (create),
And view/modify if they want to.. (read/update)
They cannot delete the order record but mark the order "canceled" (update)

ac.grant('customer')
  .createOwn('order')
  .readOwn('order')
  .updateOwn('order');

Accountants

Can only view the orders. (read)

ac.grant('accountant').readAny('order');

Sales reps.

Cannot delete an order.
They need to file an order-delete-request to their managers.. (create "odr")
This is only for the orders they've created. (own)

ac.grant('salesRep').createOwn('odr')
  .readOwn('odr').deleteOwn('odr') // can read/delete odr if they need to

Sales managers

Can view order-delete-requests, (read "odr")
If they see fit, they'll mark the delete request "accepted" (update "odr") and delete the order.
Actually, they can do anything with the orders. (c/r/u/d "order")

ac.grant('salesManager')
  .readAny('odr').updateAny('odr')
  .createAny('order').readAny('order').updateAny('order').deleteAny('order')

Hope this helps.

@awHamer

... If the own records - are records which were created by a user (creator_id automatically added) so I think create:any and create:own the same things.

Technically yes but logically, users are only creating their own accounts. So, for better semantics; create:own is more suitable here if you're making general use of possessions in your app.

... But if your business logic says that OWN records are records which were assigned to you by someone else, that this is a big difference.

Yes. But the app can both allow users to assign/create resources to/for others — and also allow them to create their own resources at the same time.

create:own allows you to create a record and assign it to you only, for example.

True.

create:any allows you to create a record and assign it to somebody...

And yourself. ("any" also covers "own").

It's just my vision, anyway it can be controlled by attributes access...
Maybe it was made the way like this just for the consistency purposes... Also curious about this.

Sorry didn't get this part. How do you mean?

It's just my vision, anyway it can be controlled by attributes access...
Maybe it was made the way like this just for the consistency purposes... Also curious about this.

Sorry didn't get this part. How do you mean?

I meant this is just how I understand it.
Assign to somebody (set user_id to someone else) can be also controlled like this
create:any: ["name,user_id,!moderator_id, ..."].

Maybe it was made the way like this just for the consistency purposes

Here I meant that another approach about reason why you made "create:any" or "create:own" (that may confuse), to keep consistent naming like "$action:$possession"...But I realized that I was wrong (possession will be set to any automatically if omit). So now the topic is clear to me. Thanks.

@awHamer, @onury, thanks a lot, guys. That was very helpful for me!


Update Fluid Framework examples links from documentation

Description
The examples page  on the Fluid Framework website has outdated links to the reference examples. Seems like at some point the folder /examples/ was added and the links were not updated accordingly.
Note: I noticed that there's an angular demo that's not in the same folder as every other demo, this should be fixed in the future (unless there's a reason for this)/

I added some more link fixes to this PR 🙂 Won't be able to go through all the ones in the crawler, but hoping to make a dent

@sonalivdeshpande FYI this will need to reverted if the examples move again.


New-RubrikSLA

Current Behavior:
Cannot create new SLA with the new-rubriksla command when specifiying yearly retention. It only works when specifying a Daily Frequency
Provide information about the failure by issuing the command using the -Verbose command.
PS C:\Windows\system32> New-RubrikSLA -Name TestSLA2 -YearlyFrequency 1 -YearlyRetention 7 -Confirm:0 -Verbose
VERBOSE: Validate the Rubrik token exists
VERBOSE: Found a Rubrik token for authentication
VERBOSE: Gather API Data for New-RubrikSLA
VERBOSE: Selected 5.0 API Data for New-RubrikSLA
VERBOSE: Load API data for
VERBOSE: Description: Create a new SLA Domain on a Rubrik cluster by specifying Domain Rules and policies
VERBOSE: Build the URI
VERBOSE: URI = https://rbk-edge.rbk.com/api/v2/sla_domain
VERBOSE: Build the query parameters for
VERBOSE: Query = Found limit flag
VERBOSE: URI = https://rbk-edge.rbk.com/api/v2/sla_domain?limit=9999
VERBOSE: Build the body
VERBOSE: Setting ParamValidation flag to $false to check if user set any params
VERBOSE: Checking for the $ParamValidation flag
VERBOSE: Body = {
    "name":  "TestSLA2",
    "frequencies":  [
                        {
                            "yearly":  {
                                           "retention":  7,
                                           "frequency":  1
                                       }
                        }
                    ]
}
VERBOSE: Performing the operation "Create a new SLA Domain on a Rubrik cluster by specifying Domain Rules and policies" on target "".
VERBOSE: Submitting the request
VERBOSE: POST https://rbk-edge.rbk.com/api/v2/sla_domain?limit=9999 with -1-byte payload
Invoke-WebRequest : Request malformed
At C:\Program Files\WindowsPowerShell\Modules\Rubrik\<IP_ADDRESS>6\Private\Invoke-RubrikWebRequest.ps1:16 char:9
+         Invoke-WebRequest -UseBasicParsing @PSBoundParameters
+         ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidOperation: (System.Net.HttpWebRequest:HttpWebRequest) [Invoke-WebRequest], WebException
    + FullyQualifiedErrorId : WebCmdletWebResponseException,Microsoft.PowerShell.Commands.InvokeWebRequestCommand

Expected Behavior:
PS C:\Windows\system32> New-RubrikSLA -Name TestSLA -DailyFrequency 1 -DailyRetention 3 -Confirm:0 -Verbose
VERBOSE: Validate the Rubrik token exists
VERBOSE: Found a Rubrik token for authentication
VERBOSE: Gather API Data for New-RubrikSLA
VERBOSE: Selected 5.0 API Data for New-RubrikSLA
VERBOSE: Load API data for
VERBOSE: Description: Create a new SLA Domain on a Rubrik cluster by specifying Domain Rules and policies
VERBOSE: Build the URI
VERBOSE: URI = https://rbk-edge.rbk.com/api/v2/sla_domain
VERBOSE: Build the query parameters for
VERBOSE: Query = Found limit flag
VERBOSE: URI = https://rbk-edge.rbk.com/api/v2/sla_domain?limit=9999
VERBOSE: Build the body
VERBOSE: Setting ParamValidation flag to $false to check if user set any params
VERBOSE: Checking for the $ParamValidation flag
VERBOSE: Body = {
"name":  "TestSLA",
"frequencies":  [
{
"daily":  {
"retention":  3,
"frequency":  1
}
}
]
}
VERBOSE: Performing the operation "Create a new SLA Domain on a Rubrik cluster by specifying Domain Rules and policies" on target "".
VERBOSE: Submitting the request
VERBOSE: POST https://rbk-edge.rbk.com/api/v2/sla_domain?limit=9999 with -1-byte payload
VERBOSE: received 391-byte response of content type application/json; charset=UTF-8
VERBOSE: Formatting return value
VERBOSE: Filter the results
id                            : 2d521f82-14d5-4015-83a7-598b71cf00af
primaryClusterId              : 22df57ec-9dad-4c18-9e15-13e963612f85
name                          : TestSLA
frequencies                   : @{daily=}
allowedBackupWindows          : {}
firstFullAllowedBackupWindows : {}
maxLocalRetentionLimit        : 259200
archivalSpecs                 : {}
replicationSpecs              : {}
isDefault                     : False
uiColor                       : #e88c2c
showAdvancedUi                : False
advancedUiConfig              : {}
Steps to Reproduce:
Please provide detailed steps for reproducing the issue.
Open ISE as administrator
Install-Module Rubrik
Import-Module Rubrik
$creds = get-credential (hit enter and when prompted, enter the username and password used to log into the Rubrik UI)
Connect-Rubrik -Server YourRubrikHost.your.domain -Credential $creds
New-RubrikSLA -Name TestSLA -YearlyFrequency 1 -YearlyRetention 7 -Confirm:0
Context:
Please provide any relevant information about your setup. This is important in case the issue is not reproducible except for under certain conditions.

BuildVersion                   10.0.14393.2969
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>

Operating System:
Windows 2016
Sniffed API when doing it via the UI and yearStartMonth and a dayofYear are specified but there is no option using PowerShell

Failure Logs
Please include any relevant log snippets or files here.

Hi @gabennett - Thanks for raising this!
You are right, the options for yearStartMonth and dayofYear are not present, which is, in turn, malforming the request.  I will look at adding these to the cmdlet.  I believe there are other options which will need to be added as well...

@gabennett - just a quick update on this.  @Pierre-PvF is currently working on implementing all of the missing parameters into the Set-RubrikSLA cmdlet.  Once he has that completed we will migrate them over to the New-RubrikSLA cmdlet as well - that way we have some sort of uniformity between the two.
Thanks!

@mwpreston Thanks so much for the update. Any rough idea when the implementation will be complete?

Hello @gabennett, this functionality should be ready this week. @pfguglielmi might be able to provide an update here.

Hi @gabennett, the Set-RubrikSLA cmdlet code is ready, but needs to be reviewed before the code is merged to the master branch. I'll now start to work on migrating the new parameters to the New-RubrikSLA cmdlet.


Unit test coverage for multi root configuration

Unit test coverage for multi root configuration

@sandy081 will this be available in the October release.
I have some unit tests in the Python extension, and would like to ensure I implement the same/similar UI workflow as you would for vscode (node/typescript projects).

@DonJayamanne Yes, Planning to do it in October. If not in November. But not sure how these are related and helpful to yours.
But, I would be also providing a sample of how to read/write configurations in MR workspace in our MR Workspace wiki - https://github.com/Microsoft/vscode/wiki/Extension-Authoring:-Adopting-Multi-Root-Workspace-APIs

@sandy081 oh, yes this is unit test coverage, just focused on the words unit test and assumed you'd be working on some UI related work in detecting unit tests in a multiroot env.
Please ignore my previous comment.

But not sure how these are related and helpful to yours.

Agreed

Added good coverage during this milestone. But still not complete.


kakoune: 2020.08.04 -> 2020.09.01

Motivation for this change
https://github.com/mawww/kakoune/releases/tag/v2020.09.01
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[x] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

build, runs, and functions as expected on NixOS.


[Content]: Modern Pricing

What article on segment.com/docs is affected?
https://segment.com/docs/connections/destinations/catalog/modern-pricing/
What part(s) of the article would you like to see updated?
Modern Pricing is defunct. They haven't been a company/product for about 2 Years. The only presence online is the Segment Destination documentation for Modern Pricing.
This page, https://segment.com/docs/connections/destinations/catalog/modern-pricing/, should be removed.
https://modernpricing.com/
Additional information
No response

Thanks @h-parker-higgins, we've removed this page from the site, and deprecated the destination in Segment's catalog.

Thanks Mark. It's a bummer. Was a really cool product.


Add documentation for Neuron exporter

#32

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


MP Blog post on Security with JWT

Just to let you know, I have a blog entry available on the Tomitribe website.
Here is the tweet for it
https://twitter.com/tomitribe/status/1134104729628422145
If you want to cross post if with MicroProfile. That's ok.

@jeanouii please read the contribution guidelines CTA directly from the MP blog link. I will close this ticket to enable a proper follow up. :)


Don't allow to divide by 0 in Vector and Color

Fixes #33609
Also fixes #41125
Before only this was reported as dividing by 0
215 / 0
21 / 0.0
25.0 / 0.0
52.0 / 0

now this also works(or better, doesn't work since now)
Vector2(24,24) / 0
Vector3(252,3535) / Vector3(0,252,242)
Color(35,235,253,3) / Color(23,0,0,0)

etc.

Hmmm...
For now I have a problem with typed GDScript
this doesn't work
	var rr : Vector2 = Vector2(2,2)
	Vector2(2,5) / rr

but this works
	var rr = Vector2(2,2)
	Vector2(2,5) / rr

I looked at my code and I don't see any problems.
Seems that something is wrong with GDScript parser.
var rr : Vector2 = Vector2(2,2)
Vector2(2,5) / rr

in this code, Godot try to evaluate value of Vector2(0,0) / Vector2(0,0) which doesn't have sense for me.
CC @vnen @bojidar-bg

in this code, Godot try to evaluate value of Vector2(0,0) / Vector2(0,0) which doesn't have sense for me.

The thing is that it constructs a dummy value to test if the operation is valid. For int and float it adds 1 to make sure they aren't zero. But it doesn't do that for other types, so they will fail now.

Rebased and added support to Vector2i and Vector3i(even if now are not visible from GDScript)

Dividing by 0 in Vector2i and Vector3i wasn't checked, so I added check also to non debug build to prevent from crashes.

Closing in favor #43522


Make polls work in other places

I had an oversight which led to polls only being shown as such when you were on the post itself. It now works in other places as well (like messages, a profile, reposts).

Now this works as intended:

merged!


chore: bump arrow to 51

NOTE: while the released 0.10.15 depends on arrow = 50, we have bumped to arrow = 51 on top of main.

Now that lance has released I've created https://github.com/lancedb/lancedb/pull/1247 but I have to run some errands so feel free to update this and merge this one instead.

I just bumped Arrow to 52.1 in #1477


Occurring UI lag while insertion process

Hi @k-lpmg!
I'm trying to insert a batch of data (~10k, 15k). UI gets laggy while insertion process, and i'm using the different thread instead of main thread.
Some details about getting lag of UI:
Bad scroll experience of sliding table / collection view. UI is not freezing entirely. It's happening for a second, really short time but affects UI experience.
You can see in below;

and last one;
i'm calling that method like that;

Hi @strawb3rryx7!
Thank you very much for finding this issue.
Issue
https://github.com/k-lpmg/RealmWrapper/blob/1e5c1f0251cf66e0b3d52232a5e6750638de408e/Sources/RealmWrapper/RealmManageable.swift#L130
transaction function in RealmManageable is calling refresh of the realm.
This code is being called on the completionQueue parameter of the transaction.
The default value of completionQueue is set to main queue.
Since the transaction of realm runs in writeQueue, if you set writeQueue to non main queue, it does not affect UI, but after the transaction is done, refresh is executed in completionQueue, so UI stopped for a very short time.
Solution
I take this issue seriously and will release the 1.2.0 version after fixing the issue through testing.
Until 1.2.0 is released, in 1.1.4 version, completionQueue will also fix the UI hang for a moment if you put non main queue.
func insertAll(_ friendships: [Friendship]) {
    rm.transaction(writeQueue: DispatchQueue.realmBg, isSync: false, writeHandler: { (realm) in
        realm.add(friendships, update: true)
    }, completionQueue: DispatchQueue.realmBg)
}

Thanks for your quick action @k-lpmg ✋🏼 I applied your hot fix. Then I'll check out when it'll be released, and there is an another issue but I'm not sure is it related with this issue.
Realm file size getting large quickly, when these batch insertions are done, I'm checking the documents & file size from General -> iPhone Storage, I see 40 MB and above.

Bump! I've been a chance to try your hotfix now. Also the problem is still exists.

I released version 1.2.0 which is a revised version of this issue.
In addition, the ability to insert multiple realm objects at once is added to Example.
You no longer need to set the completionQueue in your insertAll method because of this issue. 😁

Thanks for your all effort. Do you have any idea about database size which I mentioned in above? 😊

Sorry. I have no idea about database size. 😅


Converting MediaPipe Handpose model to TFLite model

HI team,
I am developing an embedded IOT device and I wanted to use the Handpose model for hand-tracking and gesture recognition but I didnt yet find a guide for converting Mediapipe models to TFLite model. Could you please provide a guide if its possible.
Thanks

@akshayvernekar please follow up here to see if hub team has any guide.
cc @gowthamkpr

@akshayvernekar,
Can you please elaborate on your requirement. Thanks!

I am developing  a speaker where our goal is to use hand gestures to control the playback as opposed to using voice or physical buttons . Our platform is Linux based where our primary programming interface is C/C++ so we plan to use the tflite C++ library to do the inference . Unfortunately I didn’t find a tflite version of the handpose model (Which is currently hosted on npm) which I can use . I wanted to know if there is any way to generate a tflite model from the hosted npm model .


Prepare for Snapshots release 1.0.0-rc01

#150  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @rbro112 and the rest of your teammates on  Graphite


CUP GAME

Your checklist for this pull request
Chance & JJ
Chance & JJ

The Cup Game!
Find the ball!
Press A and D to select the cup. Press J to Choose
Code
Check off the items that are true.

[x] The game was made using the Sprig editor.
[x] The game is placed in the in the /games directory.
[ ] The code is significantly different from all other games in the Sprig gallery (except for games labeled "demo").
[x] The game runs without errors.
[ ] The name of the file/game contains only alphanumeric characters, -s, or _s.
[ ] The game name is not the same as the others from gallery

Image (If an image is used)

[ ] The image is in the /games/img directory.
[ ] The name of the image matches the name of your file.

Thanks for your PR!

Unfortunately, this seems largely based on another preexisting game.  I am closing this PR as a result.  If you believe I've done this in error and want to advocate for the landing of this code, please reopen the PR and ping me.


3-way toggle not workin'

when using the 3-way toggle button, it's impossible to check programmatically the 2d and third ways even if visually it seems to be checked but they aren't. This true when you are using keyboard navigation (space bar) to check.

Hello,
After some reviews, the check can be seen as default value. It convenient to let devs using it and change the state.

@StreiffBerenice what do you think ?

I need a screenshot please to understand what are you talking about ! Thanks

@StreiffBerenice,
We are talking about the 3 way toggle.
When  the element, you can see that the checked is not set as you click on a value of the toggle.
I was saying above that that normal as that component is a showcase.
you cans see that on other showcases.

@Aniort I can confirm there's something going wrong when browsing with keyboard. It's related to the button plugin. An issue had been open we could fix it in boosted by the way

@Aniort could you please check the latest version and close this issue if it's good for you? I've port back the fix from Bootstrap PR :
https://deploy-preview-133--boosted.netlify.com/docs/4.3/components/buttons/#checkbox-and-radio-buttons

I check this new release of the 3 ways toggle button.
It's now usable but the only thing not ok is that the focus is invisible, so improve the effect visibility on focus.

see PR #137 focus is now handled on input element and on the label one

Evaluation of the 3 kinds of 3-ways tog'gles
ok for me !
can be merged !


[Azure.Communication.CallAutomation] Add RawId to CommunicationUserIdentifierModel

Description
This sets the RawId when creating CommunicationUserIdentifierModel just like PhoneNumberIdentifierModel and TeamsUserIdentifierModel, also to align with .NET SDK
Please add an informative description that covers that changes made by the pull request and link all relevant issues.
If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
All SDK Contribution checklist:

[ ] The pull request does not introduce [breaking changes]
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[ ] I have read the contribution guidelines.

General Guidelines and Best Practices

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

API change check
API changes are not detected in this pull request.

@williamzhao87 Not serializing rawId is not a bug. It's an optional property and the service must not rely on it.
@petrsvihlik to decide what to do with the PR.

the change is fine and aligned with other SDKs 👍


where is the gating operation?

Thanks for sharing the project,I have one problem, in the paper, gating operation follws the each [k,1,1] temporal convolutions, but in the code from S3DG_Pytorch.py， I can't find the gating operation...
class STConv3d(nn.Module):
    def __init__(self,in_planes,out_planes,kernel_size,stride,padding=0):
        super(STConv3d, self).__init__()
        self.conv = nn.Conv3d(in_planes, out_planes, kernel_size=(1,kernel_size,kernel_size),stride=(1,stride,stride),padding=(0,padding,padding))
        self.conv2 = nn.Conv3d(out_planes,out_planes,kernel_size=(kernel_size,1,1),stride=(stride,1,1),padding=(padding,0,0))

        self.bn=nn.BatchNorm3d(out_planes, eps=1e-3, momentum=0.001, affine=True)
        self.relu = nn.ReLU(inplace=True)

        self.bn2=nn.BatchNorm3d(out_planes, eps=1e-3, momentum=0.001, affine=True)
        self.relu2=nn.ReLU(inplace=True)

        nn.init.normal(self.conv2.weight,mean=0,std=0.01)
        nn.init.constant(self.conv2.bias,0)

    def forward(self,x):
        x=self.conv(x)
        #x=self.conv2(x)
        x=self.bn(x)
        x=self.relu(x)
        x=self.conv2(x)
        x=self.bn2(x)
        x=self.relu2(x)
        return x

@jcxu0 i have the same question, do you know how to implement, thank you!

i have the same question, i refer to other repositories and find that the common way is to utilize the squeeze and excitation to implement the feature gating mechanism.

i have the same question, i refer to other repositories and find that the common way is to utilize the squeeze and excitation to implement the feature gating mechanism.

Could you please share the links of the repos you mentioned? Thanks!


Show version info in Viewer

CodeChecker server:

CodeChecker server --verbose debug

The same output is added to CodeChecker version. In debug mode, extra annotation is showed, while "normal" mode only shows the release tag.

We need to also find a way for this to be working with releases downloaded from GitHub. Currently a
downloaded release source code does not include a Git structure, so git rev-parse HEAD and various
other commands fail, leaving version information empty. @whisperity

I think GitHub by design does not pack git information into the compressed files in the release.
We still can show the user the major version and if we need to support a release we know from which tag it was generated

@gyorb Oh, right, I forgot to mention, but that "comment" of mine was fixed in commit cd40aa2. The "describe" output defaults to the hardcoded Git information and is only rewritten to actual git tag if it is possible.


Added general documentation, fixed styling and building issues.

At this point in time, Travis CI builds started to succeed.  Flake8 errors no longer cause build to fail - no need to force style over functionality (Junior decision I guess). GL HF

Codecov Report

Merging #6 (8003e10) into main (3768518) will increase coverage by 0.69%.
The diff coverage is 99.48%.

@@            Coverage Diff             @@
##             main       #6      +/-   ##
==========================================
+ Coverage   94.06%   94.76%   +0.69%
==========================================
  Files          88       88
  Lines        3152     3360     +208
  Branches      228      226       -2
==========================================
+ Hits         2965     3184     +219
+ Misses        111       94      -17
- Partials       76       82       +6

Impacted Files
Coverage Δ

src/pygerber/parser/pillow/apertures/custom.py
100.00% <ø> (ø)

src/pygerber/parser/pillow/apertures/obround.py
100.00% <ø> (ø)

src/pygerber/tokens/am.py
100.00% <ø> (ø)

src/pygerber/tokens/gnn.py
92.45% <66.66%> (+0.14%)
:arrow_up:

src/pygerber/__init__.py
100.00% <100.00%> (ø)

src/pygerber/cli.py
90.62% <100.00%> (+0.62%)
:arrow_up:

src/pygerber/coparser.py
100.00% <100.00%> (ø)

src/pygerber/drawing_state.py
95.91% <100.00%> (+0.26%)
:arrow_up:

src/pygerber/mathclasses.py
100.00% <100.00%> (ø)

src/pygerber/parser/parser.py
91.66% <100.00%> (+0.23%)
:arrow_up:

... and 70 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3768518...8003e10. Read the comment docs.


Removing non standard protocol from constants. Rewrite logic to work …

…with http://
Non standard protocol parsing doesn't work well with url-polyfill. Breaks in nomodule browsers

[x] Tests pass
[x] Appropriate changes to README are included in PR

I don't see a test added for this behavior. 😄

So there are multiple tests for the resolve method that cover these cases, but right now we only do browser testing with chrome. This specific test fails only with the polyfill thats used in browsers that don't support URL. So we need to tests to run on different browsers.
I'll look into setting up a Saucelabs account or something similar that would run the tests on different browsers.


Consider running: meteor npm install --save tracker-component

I'm getting:
Unable to resolve some modules:

  "tracker-component" in /Users/sacha/Dev/packages/react-accounts-ui/imports/ui/components/LoginForm.jsx (os.osx.x86_64)

Consider running: meteor npm install --save tracker-component

But installing tracker-component then gives me:
npm ERR! notarget No compatible version found<EMAIL_ADDRESS><16.0.0'
npm ERR! notarget Valid install targets:
npm ERR! notarget ["0.10.0-rc1","0.11.0-rc1","0.0.1","0.0.2","0.0.3","0.1.2","0.2.0","0.2.1","0.2.2","0.2.3","0.2.4","0.2.5","0.2.6","0.3.0","0.3.4","0.3.5","0.5.0","0.5.1","0.5.2","0.6.0","0.6.1","0.6.2","0.6.3","0.7.0","0.7.1","0.8.0","0.9.0-rc1","0.9.0","0.10.0-rc1","0.10.0","0.11.0-rc1","0.11.0","0.11.1","0.11.2","0.12.0-rc1","0.12.0","0.12.1","0.12.2","0.13.0-alpha.1","0.13.0-alpha.2","0.13.0-beta.1","0.13.0-beta.2","0.13.0-rc1","0.13.0-rc2","0.13.0","0.13.1","0.13.2","0.13.3","0.14.0-alpha1","0.14.0-alpha2","0.14.0-alpha3","0.14.0-beta1","0.14.0-beta2","0.14.0-beta3","0.14.0-rc1","0.14.0","0.14.1","0.14.2","0.14.3","0.14.4","0.14.5","0.14.6","0.15.0-alpha.1","0.14.7","15.0.0-rc.1","15.0.0-rc.2"]

Does this package depend on React 15? It won't work with previous versions?

Also, I get the same message even if I upgrade to React 15.0.0 (although that breaks a lot of my other packages).

Oh, not running into this anymore. Weird…

We should maybe remove the high dependency, it shouldn't be required to run the latest version of React.
Btw, have you tried using just npm rather than meteor npm?

I think this was related to tracker-component requiring a 15.x release, this should be fixed now.


⚠ Bofrost.de has degraded performance

In 2ce0cb8, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 832 ms

Resolved: Bofrost.de performance has improved in 2f98b42 after 13 minutes.


Log warning if an IntegrationFlow bean is present but @EnableIntegration annotation is missing

Wouldn't it be a good idea to log a warning if there are IntegrationFlow beans present in the config but not an @EnableIntegration annotation?  Silently not receiving messages can be pretty easy to miss.

This is a stick with two ends.
We can determine IntegrationFlows only from the Spring Integration infrastructure, which is done by the @EnableIntegration.
The same problem we have with any<EMAIL_ADDRESS>If you forget to add it, you won't have those specific beans and behavior.
The simple sample is: if you don't add @EnableAsync, your @Async method won't be async.
Your thoughts ?

Fair enough - there are no IntegrationFlow beans as far as java-dsl is concerned if there is no @EnableIntegration annotation in the first place.  Not much to do about that I guess :)
Thanks for the fast response!  I really appreciate what you guys are doing.

No problem!
That was a good question anyway!
Have never concerned how to be since the answer was always like: to make it working don't forget to add @Enable...!
That's why we have Spring Boot, where we can  not care about all off them because of auto-config.
But, yes, we still should not forget to add @SpringBootApplication :wink:


default html-templates for 400/500/... errors

It would be nice if there is a default behavior,  which html-file is delivered if there is an error appearing.
The core could simply search for an error_404.html or error_500.html or error_412.html file and if said file does not exist, then use an error.html as a fallback.

Implemented by #658


Proposal for batched plots

In HoloViews you generally create composite plots by creating NdOverlays of Elements varying by some dimension. This is very convenient and creating overlays in this way is very simple using the .overlay method on HoloMaps. However in terms of plotting this is highly efficient, because the plotting code will generate one plot instance for each entry in the NdOverlay, which can potentially be hundreds or even thousands, which change over time. This problem is made worse by the fact that each plot will then also create a new artist (matplotlib) or glyph (bokeh) for each of the Elements, which can seriously degrade performance.
My proposal is to have a system by which NdOverlays, which have to be consistent in the types of objects they contain, can be rendered as one chunk in a single plotting call, which can provide huge performance boosts. The idea is that if the OverlayPlot encounters an NdOverlay containing an Element type it has registered it passes the NdOverlay directly to this optimized plot to render instead of expanding it into a lot of individual plots. The main requirement for these optimized plotting classes would be that they behave exactly like the unoptimized versions. I'd start with an optimized plot for NdOverlays of Curves because it's both simple and plotting a lot of curves is currently very slow.

Sounds good.  I guess supporting an explicit operation like collate() that reorganizes things after the fact would negate the the performance advantage?  Maybe there could be a parameter on OverlayPlot that determines whether it does this, so that people could turn it off if it causes problems?

I guess supporting an explicit operation like collate() that reorganizes things after the fact would negate the the performance advantage?

Not sure what you mean. What I'm describing would happen inside the plotting code so the user would never be aware of it.

Maybe there could be a parameter on OverlayPlot that determines whether it does this, so that people could turn it off if it causes problems?

That's definitely a possibility, alternatively I'll probably need a registry for these optimized plots somewhere so the user could unregister them there.

Support for batched plotting has now been merged, closing.


AnimationController.dispose() called more than once. on multiple action show

my app got freeze when show toastification
i use toastification with context, when i click action and trigger show toastification. (first time ok)
but for next toast, my app got freeze and throw exception
this should be fine, based on flow code i wrote
class AppInfo {
static error(BuildContext context, String message) {
toastification.show(
context: context,
title: Text(message),
autoCloseDuration: const Duration(milliseconds: 2500),
closeButtonShowType: CloseButtonShowType.none,
);
}
}
void addNew() {
...
if (xFile.path == '') {
AppInfo.error(context, 'No Image Selected');
return;
}
...
}

OS: Web
Flutter Version: 3.24.1
Toastification Version: 8.2.8

@indratrisnar , I’ve also encountered this issue frequently in my app. I’m trying to resolve it without altering the library’s behavior. Do you know how I can assign this issue to myself to indicate that I’m working on it?

@indratrisnar, I’m investigating this issue and have found two scenarios that might align with your experience. Could you confirm if these match what you’ve observed?
Multiple Toasts: When several toasts are shown, there's an intermittent occurrence of the AnimationController.dispose() error. It happens inconsistently.
Manual Toast Dismissal: When a toast is manually closed by the user, and then the animation duration ends, the same error is thrown (AnimationController.dispose()).
Does this align with your findings?

Yes, it does.
Pada Sen, 9 Sep 2024 03.50, Antônio A E Martins @.***>
menulis:

@indratrisnar https://github.com/indratrisnar, I’m investigating this
issue and have found two scenarios that might align with your experience.
Could you confirm if these match what you’ve observed?
Multiple Toasts: When several toasts are shown, there's an intermittent
occurrence of the AnimationController.dispose() error. It happens
inconsistently.
Manual Toast Dismissal: When a toast is manually closed by the user, and
then the animation duration ends, the same error is thrown
(AnimationController.dispose()).
Does this align with your findings?
—
Reply to this email directly, view it on GitHub
https://github.com/payam-zahedi/toastification/issues/132#issuecomment-2336820151,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APTLYD5FAL2FHDAMNB344C3ZVS2CBAVCNFSM6AAAAABNZVMMOOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMZWHAZDAMJVGE
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hey @AntonioAEMartins
Nope, you don't need to do anything specific.
You said it here, and it's yours now 😄
Feel free to Open your PR and let me know if you need anything

im having this same issue. debugging always happens the first time the toast is called.

I am having the same issue, always when the toast is closed manually or auto.

Hi @Jesse-Abruzzo, @mhdabdelrhman, and @indratrisnar,
I wanted to update you on the bug discovery related to the duplicate disposal of toasts. After some investigation, I’ve identified multiple scenarios that could be contributing to this issue.
To help me better understand the configurations you're using and narrow down the root cause, could you please answer the following questions?

Are you using custom toasts or standard ones?
Do you have a progress indicator enabled for the toasts?

Thanks for your input! This will help us get closer to resolving the problem.

Hi @Jesse-Abruzzo, @mhdabdelrhman, and @indratrisnar,
I believe I've identified and fixed the bug. I’d like you all to test the fix in your applications.
Please download my branch issue-132, then update your dependency path to point to the locally downloaded version. After that, compile your app and let me know how it goes.
Looking forward to your feedback!
I've tested the changes locally in my projects, and could not reproduce the AnimationController.dispose() called more than once.

Hi @AntonioAEMartins ,
I downloaded the project, updated the dependencies, and tested my application. Everything is functioning properly now, and the issue has been resolved.
Thank you.

@mhdabdelrhman Nice to hear that. I'm going to open a pull request, to correct and finally this issue.


[Bug] "Please update your FireTV" when launching

Device
Other
Device name
Amazon FireTV 4K
Android Version
FireOS 7
App version
0.17.0
Script version
?
WebView name and version
?
Reproduction Steps
Steps to reproduce

Download com.redphx.betterxc_0.17.0-25.apk from this repo.
Copy to FireTV.
Install APK.
Open the installed application.

Expected behavior

Ask for login / password.
Show Xbox Cloud gaming interface.

Actual behavior

A message says "update your FireTV".

When I go to my FireTV parameters and search for an update, it says that I'm already up-to-date.
Additional information

I don't have the latest version of Amzon FireTV 4K but the previous one.
I can't install Xbox application from store. I installed it from APK 2 weeks ago and it worked.
2 weeks ago, Xbox application and Better Xcloud worked fine.
When I install the Xbox APK, today I have the same message and I can't use it.

Try 0.18.0-beta

Same message with com.redphx.betterxc_0.18.0-beta-25.apk :(

The website requires Amazon Silk 122 or later. Try to update if you can.
I though I fixed it in the beta version but it didn't work because of a WebView bug. I'll find a fix later.

I checked the version of Silk : it's already 122. There is no update available in the store.
I tried to install manually an APK of version 124, but I always have the same message...

What is your user-agent in https://whatmyuseragent.com/

My user agent is:
Mozilla/5.0 (Linux; Android 9; AFTKA) AppleWebKit/537.36 (KHTML, like Gecko) Silk/124.5.2 like Chrome/124.0.6367.248 Safari/537.36

Following an external advice, I downgraded my Webkit from <IP_ADDRESS>61.160.30 to 122.6261.140.58, and it works.

Should be fixed in beta-2: https://github.com/redphx/better-xcloud-android/releases/tag/a0.18.0-beta-2
it's recommended to use Thorium and Enable "optimize webview" feature

Fixed in 5.5.3


WL-4305 Student's special requirements for course signups.

Don't release this code without WL-4526 as a bug in this Jira was fixed by WL-4526.

I'm assuming these are just files more or less patched from 10

You need to rebase this against 11.x


[APPROVALNOTIFIER] This PR is NOT APPROVED

          [APPROVALNOTIFIER] This PR is **NOT APPROVED**

This pull-request has been approved by: Ibmidquantum
Once this PR has been reviewed and has the lgtm label, please assign ant31 for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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

Originally posted by @k8s-ci-robot in https://github.com/kubernetes-sigs/application/issues/239#issuecomment-2484758969

/close


Add necessary macros to schema test context namespace

resolves #3229, #3240
Description
Ensure TestMacroNamespace is loaded in the TestContext and that necessary macros for rendering tests are loaded.
Checklist

[x] I have signed the CLA
[x] I have run this code in development and it appears to resolve the stated issue
[x] This PR includes tests, or tests are not required/relevant for this PR
[x] I have updated the CHANGELOG.md and added information about my change to the "dbt next" section.

Some of the code has moved so I'd have to hand update the changes in core/dbt/parser/manifest.py, but the rest of it should apply pretty cleanly. I'd say 2-4 hours to backport.
We could certainly look at capturing some of those other calls. We'd have to do some experiments. There's more work to do with adapter.dispatch too.


Implement POC with Google Analytics view count for public workflows and tools

Is your feature request related to a problem? Please describe.
Would like to display traffic for workflows and tools when in public views
Describe the solution you'd like
Should be able to grab numbers from Google Analytics to start with
Should expand to launch-with buttons
May need to expand to TRS downloads for platforms that avoid the buttons
┆Issue is synchronized with this Jira Story
┆Issue Type: Story
┆Sprint: Sprint 46- Unicorn Fish
┆Issue Number: DOCK-1561

I have a POC of fetching page-view counts from Google Analytics on the webservice-side - see code here: https://github.com/dockstore/dockstore/commit/ef75521b7c1e86811a00cc906c71b38bd692cd65
I closely followed this guide: https://developers.google.com/analytics/devguides/reporting/core/v4/quickstart/service-java
Steps taken:

Created a Google APIs project (for final version we can just use the same one as we do for Google OAuth).

Registered a new Google Service Account (specified for Analytics Reporting API) under the project and got its credentials (JSON key & user address).

On the Google Analytics account, added the service account as a user of the Dockstore web site data View (thanks @denis-yuen ).

Added the Analytics Reporting API v4 Java client library to the webservice.

Wrote code to initialize the Analytics Reporting service and query pageviews from the webservice: https://github.com/dockstore/dockstore/commit/ef75521b7c1e86811a00cc906c71b38bd692cd65

Expanding to "launch-with" buttons is a simple task - the code would look similar to pageviews, but with a different query of the following form:
{
  "reportRequests": [
    {
      "viewId": "126016902",
      "metrics": [
        {
          "expression": "ga:totalEvents"
        }
      ],
      "dimensionFilterClauses": [
        {
          "operator": "AND",
          "filters": [
            {
              "dimensionName": "ga:eventCategory",
              "operator": "EXACT",
              "expressions": [
                "Launch With Button"
              ]
            },
            {
              "dimensionName": "ga:eventLabel",
              "operator": "BEGINS_WITH",
              "expressions": [
                "github.com/broadinstitute/viral-pipelines/assemble_refbased"
              ]
            }
          ]
        }
      ],
      "hideTotals": true,
      "hideValueRanges": true
    }
  ]
}

Expanding to "launch-with" buttons is easy to achieve and similar to pageviews.
It only requires changing the query to be like:
{
  "reportRequests": [
    {
      "viewId": "<fill in view ID>",
      "metrics": [
        {
          "expression": "ga:totalEvents"
        }
      ],
      "dimensionFilterClauses": [
        {
          "operator": "AND",
          "filters": [
            {
              "dimensionName": "ga:eventCategory",
              "operator": "EXACT",
              "expressions": [
                "Launch With Button"
              ]
            },
            {
              "dimensionName": "ga:eventLabel",
              "operator": "BEGINS_WITH",
              "expressions": [
                "github.com/broadinstitute/viral-pipelines/assemble_refbased"
              ]
            }
          ]
        }
      ],
      "hideTotals": true,
      "hideValueRanges": true
    }
  ]
}


Aqara Motion Sensor and Aqara D1 Wall Switch (Xiaomi RTCGQ11LM and Xiaomi QBKG23LM)

It seems that device can't connect properly. I can see messages coming from device, but I can't see device on accessory list.
Do I need to manually configure something?
This is sample message:
[ZigBee] No device found from message {
type: 'attributeReport',
device: Device {
    ID: 15,
    _type: 'EndDevice',
    _ieeeAddr: '0x00158d000483a060',
    _networkAddress: 13321,
    _manufacturerID: 4151,
    _endpoints: [ [Endpoint] ],
    _manufacturerName: 'LUMI',
    _powerSource: 'Battery',
    _modelID: 'lumi.sensor_motion.aq2',
    _applicationVersion: 5,
    _stackVersion: 2,
    _zclVersion: 1,
    _hardwareVersion: 1,
    _dateCode: '20170627',
    _softwareBuildID: '3000-0001',
    _interviewCompleted: true,
    _interviewing: false,
    meta: {},
    _lastSeen:<PHONE_NUMBER>921,
    _linkquality: 55
  },
  endpoint: Endpoint {
    ID: 1,
    profileID: 260,
    deviceID: 263,
    inputClusters: [
         0, 65535, 1030,
      1024,  1280,    1,
         3
    ],
    outputClusters: [ 0, 25 ],
    deviceNetworkAddress: 13321,
    deviceIeeeAddress: '0x00158d000483a060',
    clusters: {
      msIlluminanceMeasurement: [Object],
      msOccupancySensing: [Object],
      genBasic: [Object],
      ssIasZone: [Object]
    },
    _binds: [],
    meta: {}
  },
  data: { measuredValue: 22 },
  linkquality: 55,
  groupID: 0,
  cluster: 'msIlluminanceMeasurement',
  meta: {
    zclTransactionSequenceNumber: 38,
    manufacturerCode: null,
    frameControl: {
      frameType: 0,
      manufacturerSpecific: false,
      direction: 1,
      disableDefaultResponse: true,
      reservedBits: 0
    }
  }
}

Seems that this is related to this thread

I am going to close this issue and keep working on issue #58


Sending metadata from server using node

According to the documentation, it is possible to send metadata from the server using go: https://github.com/grpc/grpc-go/blob/master/Documentation/grpc-metadata.md
I haven't been able to find the sendHeader or setTrailer methods in node. They are not implemented yet, are they? Will they be implemented?

In Node, the method to send header metadata is called sendMetadata. For trailers, it's a little more complicated. For unary responses, the callback takes an optional third argument for trailing metadata. For streaming responses, the stream's end method takes an optional argument for trailing metadata.
Note that in each case, the arguments should be instances of the class grpc.Metadata.


add support for CORS headers for Datasette [FOOD-117]

+1 I love how you document things <3

otherwise I would not survive easily, with my fish-sized memory and attention span 😱 😅


VideoStreamIn and MediaPipe

Hi everybody, I'm Cris from Guadalajara. First of all, THANKS for the tutorial and for MediaPipe, of course. It's amazing, but I've experienced some issues with MediaPipe, and I don't understand why. I'm basically a newborn at this, so I don't comprehend all the details.
Second, although my laptop has a good system, it does not have a webcam, so I'm trying to use another camera via Video Stream In by Server (rtsp://blabla). The camera works fine, but it does not show up in the drop-down menu of the MediaPipe Webcam parameter, and I'm stuck.
I have also used the SpoutCamera, which is visible in the Video Device In (but it's not active)
Thanks for the help (:
I'm working...

.

Nevermind, I already fixed it hahaha (:
Thanks for creating something so amazing!


Hydrogen 2.8.0 requires Atom >= 1.28.0, error thrown on earlier versions

This is an issue affecting everyone using an older version of Atom, i.e. Ubuntu using the webupd8team PPA:
Hydrogen 2.8.0 ships with code that requires Node 8, and thus only works with Atom 1.28 or newer (Electron 2.0 was introduced then, which in turn ships with Node 8.x). Installing Hydrogen on an older version of Atom will work, but running any code will fail.
Steps to reproduce

Install Atom < 1.28
Install latest Hydrogen version
Run any code

Atom: 1.26.1 x64
Electron: 1.7.11
OS: Ubuntu 18.04.2
Thrown From: Hydrogen package 2.8.0
Temporary Fix
Install an earlier version of Hydrogen using apm
apm install<EMAIL_ADDRESS>
Stack Trace
Failed to activate the Hydrogen package
At (0 , _util.promisify) is not a function

TypeError: (0 , _util.promisify) is not a function
    at /packages/Hydrogen/lib/import-notebook.js:15:19)
    at /packages/Hydrogen/lib/import-notebook.js:148:2)
    at Module.get_Module._compile (/opt/atom/resources/app/src/native-compile-cache.js:106:36)
    at Object.value [as .js] (/opt/atom/resources/app/src/compile-cache.js:240:29)
    at Module.load (module.js:488:32)
    at tryModuleLoad (module.js:447:12)
    at Function.Module._load (module.js:439:3)
    at Module.require (/app.asar/static/index.js:47:45)
    at require (/opt/atom/resources/app/src/native-compile-cache.js:66:33)
    at /packages/Hydrogen/lib/main.js:57:47)
    at /packages/Hydrogen/lib/main.js:588:24)
    at Module.get_Module._compile (/opt/atom/resources/app/src/native-compile-cache.js:106:36)
    at Object.value [as .js] (/opt/atom/resources/app/src/compile-cache.js:240:29)
    at Module.load (module.js:488:32)
    at tryModuleLoad (module.js:447:12)
    at Function.Module._load (module.js:439:3)
    at Module.require (/app.asar/static/index.js:47:45)
    at require (internal/module.js:20:19)
    at customRequire (/opt/atom/resources/app/static/<embedded>:96:26)
    at Package.requireMainModule (/opt/atom/resources/app/src/package.js:782:33)
    at Package.activateNow (/opt/atom/resources/app/src/package.js:220:40)
    at activationCommandSubscriptions.add.commandRegistry.onWillDispatch.event (/opt/atom/resources/app/src/package.js:865:28)
    at Function.module.exports.Emitter.simpleDispatch (/opt/atom/resources/app/node_modules/event-kit/lib/emitter.js:25:20)
    at Emitter.module.exports.Emitter.emit (/opt/atom/resources/app/node_modules/event-kit/lib/emitter.js:141:34)
    at CommandRegistry.handleCommandEvent (/opt/atom/resources/app/src/command-registry.js:360:24)
    at KeymapManager.module.exports.KeymapManager.dispatchCommandEvent (/opt/atom/resources/app/node_modules/atom-keymap/lib/keymap-manager.js:621:22)
    at KeymapManager.module.exports.KeymapManager.handleKeyboardEvent (/opt/atom/resources/app/node_modules/atom-keymap/lib/keymap-manager.js:412:28)
    at WindowEventHandler.handleDocumentKeyEvent (/opt/atom/resources/app/src/window-event-handler.js:110:40)

Commands
     -0:02.5.0 hydrogen:run-and-move-down (input.hidden-input)

Non-Core Packages
Hydrogen 2.8.0

Permanent Fix
Right now, package.json states
"engines": {
  "atom": ">=1.20.0 <2.0.0"
},

which should probably be changed to
"engines": {
  "atom": ">=1.28.0 <2.0.0"
},

Ideally, Hydrogen 2.8.0 should retroactively be changed to fix this, so that apm can pick up the correct version automatically.

I was having issues with Hydrogen 2.8.0 and Atom>1.28.0.
I solved this by updating ipykernel
pip install --upgrade ipykernel

Is there a way to use promisify in Node 6? @BenRussert

We could just use callbacks.

Atom 1.28.0 was released June 21, 2018. That's probably recent enough to try to support it? I'm guessing the vast majority of users would use the most recent version.
@nils-werner 	Is there a reason you're using that PPA? The description of that PPA says

Please use the official repository instead: https://flight-manual.atom.io/getting-started/sections/installing-atom/#platform-linux

@kylebarron interesting, I didn't know of that PPA

Does this solve your issue? If so we might bump the Atom version requirement to >= 1.28.0


VSIX don't use Vs theme settings

𝗗𝗲𝘀𝗰𝗿𝗶𝗯𝗲 𝘁𝗵𝗲 𝗯𝘂𝗴

I tried a few times in the past but failed, however happy to default to a dark skin

We will try to answer all questions that do not require research within 24hr.
To prioritize cases that require research we use the following labels in order. For all other issues the posting time is respected.

❇ Exclusive-Services
❤ Bronze Sponsor
❤ Sponsor
❤ Backer
Installation
ShowStopper
Nuget
Contribution
BreakingChange
ReproSample
Deployment
must-have

This case is prioritized.

eXpand.lab release 20.2.402.3 includes commit that relate to this task:

VSIX: Change ToolWindow defaultLookAndFeel #825

Please test if it addresses the problem. If you use nuget add our LAB NugetServer as a nuget package source in VS.
To minimize version conflicts we recommend that you switch to PackageReference format and use only the eXpandAgnostic, eXpandWin, eXpandWeb packages. Doing so, all packages will be at your disposal and .NET will add a dependecy only to those packages that you actually use and not to all (see the Modules installation-registrations youtube video).
Thanks a lot for your contribution.

eXpand.lab release 20.2.402.3 includes commit that relate to this task:

VSIX: Change ToolWindow defaultLookAndFeel #825

Please test if it addresses the problem. If you use nuget add our LAB NugetServer as a nuget package source in VS.
To minimize version conflicts we recommend that you switch to PackageReference format and use only the eXpandAgnostic, eXpandWin, eXpandWeb packages. Doing so, all packages will be at your disposal and .NET will add a dependecy only to those packages that you actually use and not to all (see the Modules installation-registrations youtube video).
Thanks a lot for your contribution.

Sorry, I think it is now the other way around.

Sorry, I think it is now the other way around.

yes true, although I really recommend you relax your eyes with a dark theme, we need to default it to white and add a manual parameter in the otions to chnage to dark if u want. As I say I cannot find a way to detect the VS current there so to automate it.

yes true, although I really recommend you relax your eyes with a dark theme, we need to default it to white and add a manual parameter in the otions to chnage to dark if u want. As I say I cannot find a way to detect the VS current there so to automate it.

OK, I relax......   ;-)

OK, I relax......   ;-)

eXpand.lab release 20.2.500.0 includes commit that relate to this task:

VSIX: Change ToolWindow defaultLookAndFeel #825

Please test if it addresses the problem. If you use nuget add our LAB NugetServer as a nuget package source in VS.
To minimize version conflicts we recommend that you switch to PackageReference format and use only the eXpandAgnostic, eXpandWin, eXpandWeb packages. Doing so, all packages will be at your disposal and .NET will add a dependecy only to those packages that you actually use and not to all (see the Modules installation-registrations youtube video).
Thanks a lot for your contribution.

eXpand.lab release 20.2.500.0 includes commit that relate to this task:

VSIX: Change ToolWindow defaultLookAndFeel #825

Please test if it addresses the problem. If you use nuget add our LAB NugetServer as a nuget package source in VS.
To minimize version conflicts we recommend that you switch to PackageReference format and use only the eXpandAgnostic, eXpandWin, eXpandWeb packages. Doing so, all packages will be at your disposal and .NET will add a dependecy only to those packages that you actually use and not to all (see the Modules installation-registrations youtube video).
Thanks a lot for your contribution.

eXpand.lab release 20.2.502.1 includes commit that relate to this task:

VSIX: open to control Color Theme #825

Please test if it addresses the problem. If you use nuget add our LAB NugetServer as a nuget package source in VS.
To minimize version conflicts we recommend that you switch to PackageReference format and use only the eXpandAgnostic, eXpandWin, eXpandWeb packages. Doing so, all packages will be at your disposal and .NET will add a dependecy only to those packages that you actually use and not to all (see the Modules installation-registrations youtube video).
Thanks a lot for your contribution.

eXpand.lab release 20.2.502.1 includes commit that relate to this task:

VSIX: open to control Color Theme #825

Please test if it addresses the problem. If you use nuget add our LAB NugetServer as a nuget package source in VS.
To minimize version conflicts we recommend that you switch to PackageReference format and use only the eXpandAgnostic, eXpandWin, eXpandWeb packages. Doing so, all packages will be at your disposal and .NET will add a dependecy only to those packages that you actually use and not to all (see the Modules installation-registrations youtube video).
Thanks a lot for your contribution.

Hi,
there is an own setting and it works.

Hi,
Not sure what u mean

Issue is deprioritized as no Assignee found and scheduled for auto-close if no activity in the next 60 days. Thanks a lot for your contribution.

Closing issue for age. Feel free to reopen it at any time.
.Thank you for your contribution.


Adding Pluto and Some sample notebooks

This PR focus on adding Pluto and the some Pluto notebooks to the package.
added dependency on new packages: Pluto, PlutoUI, WGLMakie, Format
Note: WGLMakie can clash with GLMAkie - we need to maybe create an init function to resolve it once and for all.
Added the NotebooksUtils module to help repeating notebook code.
Added 2 utility functions to the Vis module to allow better control of rendering inside Pluto notebooks.
added 3 sample notebooks in the samples/notebooks folder and a utility function to launch Pluto and run them.
added a small description in the documentation Examples page instructing how to launch the notebooks.
added a call to the GLMakie.activate!() function in the documentation script to make sure the GLMakie backend is used.
Bug Fix: Seems like i forgot to include the new emitters page in the make.jl script that produce the documentation pages.

Codecov Report

Merging #80 (496b1e6) into main (18fdf89) will decrease coverage by 4.77%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main      #80      +/-   ##
==========================================
- Coverage   49.01%   44.24%   -4.78%
==========================================
  Files          55       55
  Lines        6425     6512      +87
==========================================
- Hits         3149     2881     -268
- Misses       3276     3631     +355

Impacted Files
Coverage Δ

src/NotebooksUtils/NotebooksUtils.jl
0.00% <0.00%> (ø)

src/OpticSim.jl
83.33% <ø> (ø)

src/Visualization/Visualization.jl
0.19% <0.00%> (-5.77%)
:arrow_down:

src/Geometry/CSG/CSG.jl
31.39% <0.00%> (-53.55%)
:arrow_down:

src/Geometry/Primitives/Qtype.jl
75.00% <0.00%> (-8.34%)
:arrow_down:

src/Geometry/Primitives/Sphere.jl
84.90% <0.00%> (-7.55%)
:arrow_down:

src/Geometry/Primitives/Cylinder.jl
79.62% <0.00%> (-7.41%)
:arrow_down:

src/Optical/LensAssembly.jl
48.22% <0.00%> (-7.33%)
:arrow_down:

src/Geometry/Surface.jl
63.63% <0.00%> (-4.55%)
:arrow_down:

... and 12 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 18fdf89...496b1e6. Read the comment docs.

@alfredclwong @BrianGun
seems like the ubuntu runs are stuck. this PR introduced the WGLMakie into the mix and maybe it's confusing the rendering part again :(


Feature request: support flake8-unused-arguments

flake8-unused-arguments has warnings for:

U100 - An unused argument.
U101 - An unused argument starting with an underscore

This would also close a gap with VS Code's Pylance which has a U100 equivalent ("x is not accessed") built in.

Nice idea.


[Bug] SplitTooLongLines alternates indentation in variable section

##Description:
Given I have the following robot file
# file.robot

*** Variables ***
${LONG_VAR}    verylongvariable1234123412341234124123412341234123412341234123412341234123412341234123412341234123412341
${SHORT_VAR}    abc

When I run robotidy file.robot, it gives me the following output
*** Variables ***
${LONG_VAR}
...    verylongvariable1234123412341234124123412341234123412341234123412341234123412341234123412341234123412341
${SHORT_VAR}    abc

However, when I run robotidy file.robot again, it changes the output to
*** Variables ***
${LONG_VAR}
...             verylongvariable1234123412341234124123412341234123412341234123412341234123412341234123412341234123412341
${SHORT_VAR}    abc

##Expected behavior:
robotidy should format the code consistently, not changing the space count on every run.
Preferably, the output should look like after the second run, being in line with ${SHORT_VAR}
##Affected version:
3.0

It was probably fixed when I fixed #360 (sounds familiar). It's on the main branch already, I will try to create 3.0.1 or 3.1 release soon

But of course I will first try to reproduce it on the old code /current code and see if that fix helps.

I can confirm it does repeatably format the file in 3.0.0 but using the code from the main branch it does stop formatting the file after 2 re-runs (and it looks like in you last example). I will close this issue as fixed after the next release.

Released as 3.1.0 (https://github.com/MarketSquare/robotframework-tidy/releases/tag/3.1.0)


Disable glog writing to files.

Today whenever we run crictl, it generates several log files under /tmp:
$ ls /tmp/crictl*
/tmp/crictl.INFO                                                 /tmp/crictl.lantaol0.root.log.INFO.20180420-153525.177197    /tmp/crictl.lantaol0.root.log.WARNING.20180417-191720.113028
/tmp/crictl.lantaol0.lantaol.log.INFO.20180420-153505.177099     /tmp/crictl.lantaol0.root.log.INFO.20180420-153525.177205    /tmp/crictl.lantaol0.root.log.WARNING.20180420-153524.177178
/tmp/crictl.lantaol0.lantaol.log.INFO.20180420-162337.191963     /tmp/crictl.lantaol0.root.log.INFO.20180420-153526.177214    /tmp/crictl.lantaol0.root.log.WARNING.20180420-153524.177187

This becomes a problem if we run the command in a loop, e.g. we run crictl every 10 seconds to check containerd healthness https://github.com/containerd/cri/blob/master/cluster/health-monitor.sh#L43.
This can eventually fill the tmp directly. I do believe no one looks at those log files, and most people just read stderr.
Actually, crictl doesn't even have flags for glogs:
$ crictl --help
NAME:
   crictl - client for CRI

USAGE:
   crictl [global options] command [command options] [arguments...]

VERSION:
   1.0.0-beta.0

COMMANDS:
     attach        Attach to a running container
     create        Create a new container
     exec          Run a command in a running container
     version       Display runtime version information
     images        List images
     inspect       Display the status of one or more containers
     inspecti      Return the status of one ore more images
     inspectp      Display the status of one or more pods
     logs          Fetch the logs of a container
     port-forward  Forward local port to a pod
     ps            List containers
     pull          Pull an image from a registry
     runp          Run a new pod
     rm            Remove one or more containers
     rmi           Remove one or more images
     rmp           Remove one or more pods
     pods          List pods
     start         Start one or more created containers
     info          Display information of the container runtime
     stop          Stop one or more running containers
     stopp         Stop one or more running pods
     update        Update one or more running containers
     config        Get and set crictl options
     stats         List container(s) resource usage statistics
     completion    Output bash shell completion code
     help, h       Shows a list of commands or help for one command

GLOBAL OPTIONS:
   --config value, -c value            Location of the client config file (default: "/etc/crictl.yaml") [$CRI_CONFIG_FILE]
   --debug, -D                         Enable debug mode
   --image-endpoint value, -i value    Endpoint of CRI image manager service [$IMAGE_SERVICE_ENDPOINT]
   --runtime-endpoint value, -r value  Endpoint of CRI container runtime service (default: "unix:///var/run/dockershim.sock") [$CONTAINER_RUNTIME_ENDPOINT]
   --timeout value, -t value           Timeout of connecting to the server (default: 10s)
   --help, -h                          show help
   --version, -v                       print the version

Let's just always set logtostderr=true.
Signed-off-by: Lantao Liu<EMAIL_ADDRESS>

LGTM


Add ImportLibResourceLoader

Issue #970 , please see existing pull request #976 . Also cut a PR to gsemet/jinja:master (https://github.com/gsemet/jinja/pull/1), but the merge from pallets/jinja:master looks weird. Anyway, feel free to close either PR.

TOXENV=py (nightly) fails just like master, but the others pass.

is this going to be fixed and merged soon?

Changes were merged into #976 instead.