Default subnet not found for ca-central-1d

If you run into this issue you need to create a new default subnet yourself. This a new AZ in Canada.
Error: Default subnet not found

  on .terraform/modules/security/modules/vpc-baseline/main.tf line 47, in resource "aws_default_subnet" "default":
  47: resource "aws_default_subnet" "default" {

The following fixed it for me:
# https://docs.aws.amazon.com/vpc/latest/userguide/default-vpc.html#create-default-subnet
$ aws ec2 create-default-subnet --availability-zone ca-central-1d --region ca-central-1

No change required by this module. This can be left to auto-close.

@willfarrell Thanks for the note!
Yes, this module throws an error when a default subnet does not exist in any of availability zones. This could happen when you deletes a default subnet manually, or a default subnet is not automatically created for newly added availability zones.
AWS docs says a default subnet wouldn't be created for new AZs when the default VPC is modified.
Currently this behavior can't be avoided since aws_default_subnet resource creation fails when no default subnet exists, and there's no way to check its existence in advance.
After this issue in terraform-provider-aws is resolved, aws_subnet_ids could be used to detect the existence of a default subnet in each region. With this change, we could control the creation of aws_default_subnet depending on its return value.
It seems that this change would be included in the next major release of terraform-provider-aws, so I'll keep this ticket open until then.


Print table view when generating output file

There is currently no support in Syft for this but it can be accomplished by executing multiple times. This is less than ideal, as scans have the potential to be slow. Once Syft has some support, we can look at adding this to sbom-action.
See:

https://github.com/anchore/syft/issues/325
https://github.com/anchore/syft/issues/400

Depends on https://github.com/anchore/syft/issues/563


[Storage] Matching _shared folders after typing change (File-share package)

As title states 😄

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-storage-file-share


Create integration/README.md to document details of integration tests

In response to the discussions on #1333, I've created this markdown file that does a better job of explaining how the integration tests work.

Thanks Chris for the review comments.

LGTM


Generate flow files for react-native

Description
As per https://github.com/facebook/react-native/issues/13238, I would like to be able to use a different version of flow than what react-native explicitly requires. This is necessary because our mobile project is in a lerna-style monorepo.
@janicduplessis provided a solution which is to ignore react-native and create a type interface for it, so I'm asking how to specifically do that.
Reproduction Steps and Sample Code
I attempted to use flow's gen-flow-files, but was unsuccessful:
$ $(npm bin)/flow gen-flow-files . --out-dir ../react-native-types/
Found 13295 files, generating libdefs...
Error: Failure("/Users/ckknight/Code/facebook/react-native/react-native-git-upgrade/package.json: Failure(\"Failed to extract the exports of AnyT\")")

Solution
Either a way to generate the flow definitions or a concrete flow definition for react-native.
Additional Information

React Native version: v0.44.0 (latest as of 2017-05-16)
Platform: N/A
Development Operating System: macOS
Dev tools: N/A

try this https://github.com/entria/yarn-package-boilerplate/blob/master/package.json#L47

Hi there! This issue is being closed because it has been inactive for a while. Maybe the issue has been fixed in a recent release, or perhaps it is not affecting a lot of people. Either way, we're automatically closing issues after a period of inactivity. Please do not take it personally!
If you think this issue should definitely remain open, please let us know. The following information is helpful when it comes to determining if the issue should be re-opened:

Does the issue still reproduce on the latest release candidate? Post a comment with the version you tested.
If so, is there any information missing from the bug report? Post a comment with all the information required by the issue template.
Is there a pull request that addresses this issue? Post a comment with the PR number so we can follow up.

If you would like to work on a patch to fix the issue, contributions are very welcome! Read through the contribution guide, and feel free to hop into #react-native if you need help planning your contribution.


fix: update kubernetes dashboard token generation

Description
In newer versions of kubectl account tokens are no longer auto-generate for ServiceAccounts. Therefore, make dashboard just outputs an empty token for those versions. This PR fixes this and should be backwards compatible, so also support older kubectl versions

Nice!


Update GA (and Hugo min. version) after deprecations

Fix #28.
Notes:

tpl/tplimpl/embedded/templates/google_analytics_async.html has been deprecated since Hugo v0.119.0, and was removed in Hugo v0.125.0, tpl/tplimpl/embedded/templates/google_analytics.html becoming the only GA template.
site.GoogleAnalytics has been deprecated since Hugo v0.120.0, and its use with the latest Hugo versions currently triggers an error, as per Hugo deprecation process.

Problem with deployment:
  UNSUPPORTED BUILD IMAGE

  The build image for this site uses Ubuntu 16.04 Xenial Xerus, which is no longer supported.


Rasa train core compare

Proposed changes:

rasa train core with multiple config files trains multiple models and stores all of them compressed (as tar.gz files)

fixes https://github.com/RasaHQ/rasa/issues/3983
Status (please check what you already did):

[x] made PR ready for code review
[x] added some tests for the functionality
[ ] updated the documentation
[x] updated the changelog
[x] reformat files using black (please check Readme for instructions)

@tabergma I merged the file-importer branch 🙈 Sorry for the merge conflicts.


User Authorization header not sent on refresh

I am currently developing in the browser.
After login-in everything works fine and I am able to save user data.
this.user.set('firstName', this.settings.firstName);
this.user.save()

But after I refresh the browser it fails - I get a 401. it seems like the Authorization header isn't being sent. I do see the token exists in the local storage.
Should I be taking some sort of action when the app starts?

@sagivf I think your problem might be that you're not passing {'remember': true} to the login method. I am just about ready to release a new version that defaults this to be true.

@dwieeb I am passing it, i'll check again after you release.

Oh, right, because you said you saw it in local storage.
Hmm, I'm not sure. I've never seen this behavior. You shouldn't need to do anything when the app starts. Once a user logs in, their token is stored indefinitely and it should be picked up by our client client if it exists.
Yeah, let's check with the new version when it comes out.

@sagivf New version is out. There were some changes since the last release, you can see them in this diff.
If you're still getting this error, I'm really not sure what the problem would be. Internally, the HTTP client takes a TokenContext (instead of a token). This means it now tries to get the token from the context (in this case, localstorage) every time it makes a request.
Let me know how it works!

@sagivf I did a basic implementation on v0.0.12 and got the same error. FWIW, I couldn't implement the static methods on v0.0.14.
Here's the whole process
Initial signup, just for reference...
ngOnInit() {
    var authProvider = 'basic';
    var authSettings = { 'remember': true };
    var authDetails = {'name': 'fullname', 'email'<EMAIL_ADDRESS>'password': 'puppies123'};
    Auth.signup(authDetails).then(() => {
      console.log("`user` is now registered");
    }, (err) => {
      console.log("something went wrong!")
    });
}

Subsequent Login and Save works fine...
Confirmed the Custom Data in the Ionic Platform dashboard
ngOnInit() {
    var authProvider = 'basic';
    var authSettings = { 'remember': true };
    var authDetails = {'name': 'fullname', 'email'<EMAIL_ADDRESS>'password': 'puppies123'};
    Auth.login(authProvider, authSettings, authDetails).then(() => {
      console.log("`user` is now the authenticated user");
        this.currentUser = User.current();
        this.currentUser.set('foo', "bar");
        this.currentUser.save()
    }, (err) => {
      console.log("something went wrong!")
    });
}

Update custom data without re-logging in.
ngOnInit() {
    var authProvider = 'basic';
    var authSettings = { 'remember': true };
    var authDetails = {'name': 'fullname', 'email'<EMAIL_ADDRESS>'password': 'puppies123'};
    this.currentUser = User.current(); // Works!
    console.log(this.currentUser); // <<== this returns a valid user with _id
    // And valid tokens are in Resources
    this.currentUser.set('foo', "baz");
    this.currentUser.save(); // <<== Throws error
}

Console Errors
PATCH https://api.ionic.io/auth/users/e4b3b822-2507-4c03-9214-af2621b361a0 401 (UNAUTHORIZED)
Ionic User: Error: UNAUTHORIZED(…)
EXCEPTION: Error: Uncaught (in promise): Error: UNAUTHORIZED
Unhandled Promise rejection: UNAUTHORIZED ; Zone: angular ; Task: XMLHttpRequest.addEventListener:readystatechange ; Value: Error: UNAUTHORIZED(…)

Thanks @cklanac. Looks like you are have the same issue as me. @dwieeb looks like this is not really operational as it requires logging in every time.
Even that wouldn't be possible as the password is not available for the developer(rightfully so).

@sagivf @cklanac We obviously don't want the developer to login the user every time. Please see my comments: https://github.com/driftyco/ionic-cloud-angular/issues/6#issuecomment-230167536 and try the latest 0.0.14. I just tried your example with 0.0.14 and was able to update my user.

It's working nicely after upgrading to 0.0.14 but there is a small thing which i'm not sure are a problem.
For the user full details to be loaded (custom data) I have do do the following:
const isAuthenticated = this.auth.isAuthenticated();
if (isAuthenticated) {
  this.user.load();
}

@dwieeb shall I open an issue for this?

@sagivf Looks like a bug. I published a new version. Thanks! If there are additional issues, please feel free to open new ones.


Tensorflow GPU import causes exeptions

There is a problem occures when I import tensorflow.

"C:\Program Files\Python35\python.exe" D:/NetDisks/DropBox/Dropbox/Develop/Python/TensorFlow/GPUTest.py
Traceback (most recent call last):
File "C:\Program Files\Python35\lib\site-packages\tensorflow\python\pywrap_tensorflow_internal.py", line 18, in swig_import_helper
return importlib.import_module(mname)
File "C:\Program Files\Python35\lib\importlib_init_.py", line 126, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 986, in _gcd_import
File "", line 969, in _find_and_load
File "", line 958, in _find_and_load_unlocked
File "", line 666, in _load_unlocked
File "", line 577, in module_from_spec
File "", line 906, in create_module
File "", line 222, in _call_with_frames_removed
ImportError: DLL load failed: Не найден указанный модуль.
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "C:\Program Files\Python35\lib\site-packages\tensorflow\python\pywrap_tensorflow.py", line 41, in
from tensorflow.python.pywrap_tensorflow_internal import *
File "C:\Program Files\Python35\lib\site-packages\tensorflow\python\pywrap_tensorflow_internal.py", line 21, in
_pywrap_tensorflow_internal = swig_import_helper()
File "C:\Program Files\Python35\lib\site-packages\tensorflow\python\pywrap_tensorflow_internal.py", line 20, in swig_import_helper
return importlib.import_module('pywrap_tensorflow_internal')
File "C:\Program Files\Python35\lib\importlib_init.py", line 126, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
ImportError: No module named '_pywrap_tensorflow_internal'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "D:/NetDisks/DropBox/Dropbox/Develop/Python/TensorFlow/GPUTest.py", line 1, in
import tensorflow as tf
File "C:\Program Files\Python35\lib\site-packages\tensorflow_init_.py", line 24, in
from tensorflow.python import *
File "C:\Program Files\Python35\lib\site-packages\tensorflow\python_init_.py", line 49, in
from tensorflow.python import pywrap_tensorflow
File "C:\Program Files\Python35\lib\site-packages\tensorflow\python\pywrap_tensorflow.py", line 52, in
raise ImportError(msg)
ImportError: Traceback (most recent call last):
File "C:\Program Files\Python35\lib\site-packages\tensorflow\python\pywrap_tensorflow_internal.py", line 18, in swig_import_helper
return importlib.import_module(mname)
File "C:\Program Files\Python35\lib\importlib_init_.py", line 126, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 986, in _gcd_import
File "", line 969, in _find_and_load
File "", line 958, in _find_and_load_unlocked
File "", line 666, in _load_unlocked
File "", line 577, in module_from_spec
File "", line 906, in create_module
File "", line 222, in _call_with_frames_removed
ImportError: DLL load failed: Не найден указанный модуль.
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "C:\Program Files\Python35\lib\site-packages\tensorflow\python\pywrap_tensorflow.py", line 41, in
from tensorflow.python.pywrap_tensorflow_internal import *
File "C:\Program Files\Python35\lib\site-packages\tensorflow\python\pywrap_tensorflow_internal.py", line 21, in
_pywrap_tensorflow_internal = swig_import_helper()
File "C:\Program Files\Python35\lib\site-packages\tensorflow\python\pywrap_tensorflow_internal.py", line 20, in swig_import_helper
return importlib.import_module('pywrap_tensorflow_internal')
File "C:\Program Files\Python35\lib\importlib_init.py", line 126, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
ImportError: No module named '_pywrap_tensorflow_internal'
Failed to load the native TensorFlow runtime.
See https://www.tensorflow.org/install/install_sources#common_installation_problems
for some common reasons and solutions.  Include the entire stack trace
above this error message when asking for help.
Process finished with exit code 1

Windows 7 x64 SP1
CPU: Intel Core I7 990x
Motherboard: Asus P6T SE
GPU: GTX 1080 TI
tensorflow have been installed with native PIP, command:

pip install tensorflow-gpu

Python 3.5.2
CUDA 8.0 Toolkit
CUDNN 5.1
The code is:
import tensorflow as tf
Path:

C:\Program Files\NVIDIA GPU Computing Toolkit\CUDA\v8.0\bin;
C:\Program Files\NVIDIA GPU Computing Toolkit\CUDA\v8.0\libnvvp;
C:\Program Files\Python35\Scripts;
C:\Program Files\Python35;
C:\Program Files\Microsoft MPI\Bin;
C:\Windows;
C:\Windows\System32;
C:\Program Files\dotnet;
C:\Program Files\Anaconda3;
C:\Program Files\Anaconda3\Scripts;
C:\Program Files\Anaconda3\Library\bin;
C:\Program Files\Anaconda2;
C:\Program Files\Anaconda2\Scripts;
C:\Program Files\Anaconda2\Library\bin;
C:\Program Files (x86)\Anaconda3;
C:\Program Files (x86)\Anaconda3\Scripts;
C:\Program Files (x86)\Anaconda3\Library\bin;
C:\Program Files (x86)\Anaconda2;
C:\Program Files (x86)\Anaconda2\Scripts;
C:\Program Files (x86)\Anaconda2\Library\bin;
C:\Program Files\NVIDIA Corporation\cuda\bin;
C:\Program Files (x86)\NVIDIA Corporation\PhysX\Common;
C:\Program Files\NVIDIA Corporation\cuda\bin;C:\Program Files\NVIDIA GPU Computing Toolkit\CUDA\v8.0\bin;
C:\Program Files\NVIDIA GPU Computing Toolkit\CUDA;C:\Program Files\NVIDIA GPU Computing Toolkit\CUDA\v8.0;
C:\Program Files\NVIDIA GPU Computing Toolkit\CUDA\v8.0\lib\x64;
C:\Program Files\NVIDIA GPU Computing Toolkit\CUDA\v8.0\extras\CUPTI\libx64;

CUDA_HOME

C:\Program Files\NVIDIA GPU Computing Toolkit\CUDA\v8.0

CUDA_PATH

C:\Program Files\NVIDIA GPU Computing Toolkit\CUDA\v8.0

CUDA_PATH_V8_0

C:\Program Files\NVIDIA GPU Computing Toolkit\CUDA\v8.0

MSVC++ 2015 x64 is installed
cudnn64_5.dll is located at:

C:\Program Files\NVIDIA GPU Computing Toolkit\CUDA\v8.0\bin

Also if i install a regular tensorflow, it works OK.
How do I deal with/fix this?

Newermind, I was using TF 1.3, whinch is using CUDNN 6.0


Achievement for manual QA which could've been automated

Most projects have at least some expectation of the quality of contributions, but many of them do not automatically enforce these. This has a lot of bad side effects:

Many PR comments are about style rather than substance.
"But it works!" vs "But it's ugly!" ad nauseam.

Bike shedding every little thing rather than accepting a tool's presets.
The longer the situation goes on, the messier the process of introducing automation becomes. The diff gets huge, the potential for breakage increases, and the version history becomes a huge mess.

Title suggestion: Robot
Criteria suggestion: 100 PR suggestions on a single project which would've been detected by mainstream tools. Not sure how you'd detect this, though.

I like the idea of lampooning folks who are intense about bike shedding/formatting/prettification.
maybe something specific like:
"Semi-Colonoscopy - Add or remove 1,000 semicolons from other's repositories via PR."
"It's Pretty, Broken - Submit 100 Pull Requests that change reformat all code files without changing the logic."


Selecting "Dolby Access / Dolby atmos for headphones" doesn't enable it

Selecting "Dolby Access / Dolby atmos for headphones" doesn't enable it, here is a screen recording of my issue
https://streamable.com/umc25x

Dolby Atmod is in line 111. Try different combination, like using just Dolby. You can also try using svcl.exe from the command line to see if it gives any errors.

I would like to say I understand what "line 111" means, and "using svcl.exe" from command line, i dont really understand code, but i generally know how to edit a file or something like that, if you could guide me a bit more clearly? Thank you!


docs: remove unnecessary conditional in setup section

Sorry I was the one who added this one in the past, but it is not necessary. Better to have simple mix install.

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

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

Details

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

Files with Coverage Reduction
New Missed Lines
%

lib/tflite_tensor.ex
19
36.36%

Totals

Change from base Build 5d8c4d097de8f8de3d3804938d336cba60b5272f:
-3.7%

Covered Lines:
192

Relevant Lines:
267

💛  - Coveralls


fix(APPL-565): Fix external url issue in Android webview

This PR resolves APPL-565
Description
This PR fixes the issue in Android when tapping an external URL inside the webview
Where the user gets redirected to the browser, but the webview goes to a blank page
Videos

Before
After

PR Checklist

[X] I have tested my changes on iOS and Android.
[X] I hid my changes behind a feature flag, or they don't need one.
[X] I have included screenshots or videos, or I have not changed the UI.
[X] I have added tests, or my changes don't require any.
[X] I added an app state migration, or my changes do not require one.
[X] I have documented any follow-up work that this PR will require, or it does not require any.
[X] I have added a changelog entry below, or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

iOS user-facing changes

Android user-facing changes

Fix the issue with the webview having a blank page after tapping an external URL - mrsltun

Dev changes

Need help with something? Have a look at our docs, or get in touch with us.

This PR contains the following changes:

Android user-facing changes (Fix the issue with the webview having a blank page after tapping an external URL - mrsltun)

  Generated by :no_entry_sign: dangerJS against aa8e1c8098bbbde8f7846b8f477643ac0887c670

@gkartalis @brainbicycle I like the idea too, but it looks a bit off with the artsy logo at the top

I'm not sure if we can remove that like we do in Force

@gkartalis @brainbicycle I like the idea too, but it looks a bit off with the artsy logo at the top
I'm not sure if we can remove that like we do in Force

we can do separately, thanks for looking into!


[KOGITO-6812] - Kogito operator should support label on deployment

Many thanks for submitting your Pull Request :heart:!
Jira issue : https://issues.redhat.com/browse/KOGITO-6812
with this improvement user can mark any kogito deployment object with kogito.kie.org/runtime: 'true' annotation and kogito operator will consider that deployment as KogitoRuntime. Kogito Operator will inject supporting service url into these deployment.
Please make sure your PR meets the following requirements:

[ ] You have read the contributors' guide
[ ] Pull Request title is properly formatted: [KOGITO-XYZ] Subject
[ ] Pull Request contains a link to the JIRA issue
[ ] Pull Request contains a description of the issue
[ ] Pull Request does not include fixes for issues other than the main ticket
[ ] Your feature/bug fix has a unit test that verifies it
[ ] You've ran make before-pr and everything is working accordingly
[ ] You've tested the new feature/bug fix in an actual OpenShift cluster
[ ] You've added a RELEASE_NOTES.md entry regarding this change

How to retest this PR or trigger a specific build:

Run operator BDD testing
Please add comment: /jenkins test

Run RHPAM operator BDD testing
Please add comment: /jenkins RHPAM test

/jenkins test

PR job #574 was: UNSTABLE
Possible explanation: This should be test failures
You can find test error logs here: https://jenkins-kogito-tools.kogito-cluster-0ad6762cc85bcef5745bb684498c2436-0000.us-south.containers.appdomain.cloud/job/kogito/job/kogito-operator-PR-check/574//artifact/test/logs/error/
Test results:

PASSED: 9
FAILED: 1

Those are the test failures:

Deploy source files (dmn, drl, bpmn, bpmn2, ...) with CLI.Deploy .dmn source files with CLI using runtime springboot
Step HTTP POST request on service "dmn-springboot-example" is successful within 2 minutes with path "Traffic Violation" and body:: Timeout waiting for HTTP POST request on path 'Traffic Violation' to be successful

Please look here: https://jenkins-kogito-tools.kogito-cluster-0ad6762cc85bcef5745bb684498c2436-0000.us-south.containers.appdomain.cloud/job/kogito/job/kogito-operator-PR-check/574/display/redirect

/jenkins test

/jenkins test

gonna try to give it a go today

Looks ok to me, will leave it on Tristan to test

Tested ok. Config Maps are correctly bound to the deployment when annotation is there,
Just one remark: we are losing information that the operator has added the supporting services configmaps to the deployment.
@vaibhavjainwiz Would there be a way to add to the deployment status, letting the user know that the supporting services have been bound to the deployment ?

also @vaibhavjainwiz could you add an example of a deployment with the annotation ?
This is the one I used for the tests. Feel free to take it and update it if needed
kind: Deployment
apiVersion: apps/v1
metadata:
  name: test-deployment
  annotations:
    kogito.kie.org/runtime: 'true'
spec:
  replicas: 1
  selector:
    matchLabels:
      app: process-quarkus-example
  template:
    metadata:
      creationTimestamp: null
      labels:
        app: process-quarkus-example
    spec:
      containers:
        - name: process-quarkus-example
          image: 'quay.io/kiegroup/examples-process-quarkus-example:latest'
          imagePullPolicy: Always
          ports:
            - containerPort: 8080
              protocol: TCP
          readinessProbe:
            httpGet:
              path: /q/health/ready
              port: 8080
              scheme: HTTP
            timeoutSeconds: 1
            periodSeconds: 10
            successThreshold: 1
            failureThreshold: 3
          livenessProbe:
            httpGet:
              path: /q/health/live
              port: 8080
              scheme: HTTP
            timeoutSeconds: 1
            periodSeconds: 10
            successThreshold: 1
            failureThreshold: 3
          startupProbe:
            httpGet:
              path: /q/health/live
              port: 8080
              scheme: HTTP
            timeoutSeconds: 1
            periodSeconds: 10
            successThreshold: 1
            failureThreshold: 3
          env:
          - name: RUNTIME_TYPE
            value: quarkus

@radtriste I tried to add extra DeploymentCondition inside status of Deployment object to represent supporting service endpoint injection info but it give me error the object has been modified; I had tried all mean but not able to overcome this issue.
For user info, we could add a note in Kogito docs to check inserted configmap inside Deployment object.
For update example I had a Jira in queue https://issues.redhat.com/browse/KOGITO-6297. I will update it.

@radtriste I tried to add extra DeploymentCondition inside status of Deployment object to represent supporting service endpoint injection info but it give me error the object has been modified; I had tried all mean but not able to overcome this issue.
For user info, we could add a note in Kogito docs to check inserted configmap inside Deployment object.
For update example I had a Jira in queue https://issues.redhat.com/browse/KOGITO-6297. I will update it.

I guess this will be for later improvement then, if possible. Let's merge that one.

Can you address @sutaakar comment ?

Can you address @sutaakar comment ?

@radtriste I had incorporated @sutaakar comments.

/jenkins test


Ignore compilation warnings

Describe problem solved by this pull request
This pr to fix: https://github.com/PX4/Firmware/issues/15275
more detail : https://px4.slack.com/archives/C0W2KUFFT/p1594711366163700
I found the pr that introduced the problem　https://github.com/PX4/Firmware/pull/15301
Describe your solution
Ignore compilation warnings
Test data / coverage
fixed it on my 16.04
Additional context
@dagar  Is this the correct approach?

@dagar Is this the correct approach?

Yes for systemcmds/tests. For any non test code I'd want to alter the offending source.
Thanks for fixing this @xdwgood.


Change support to use built-in Discord slash commands

As of May 4, 2021, Discord added some first-class support for slash commands, including autocomplete. Bot on the Clocktower should make use of this (perhaps only if it has Slash Command permission for the server?)

New slash commands for bots! Performing actions with bots is easier than ever. Bot developers—when they’re ready, please don’t scare them—can have a list of commands available for their bots accessible by the forwardslash key (/). You won’t have to remember bot commands anymore and get laughed at when you get the command wrong :( !help…..!help….please
Added use Slash Commands permission. If you run a server, make sure to configure this permission for your bot channels. Bots that upgrade to Slash Commands require this permission.

This is apparently something that needs to be handled at the low-level, like in Discord.py, and it doesn't seem super likely.
https://gist.github.com/Rapptz/4a2f62751b9600a31a0d3c78100287f1#FAQ
This may mean we will need to port this to a new framework or even a new language.


[FEATURE] Add category in Application

it would be great to be able to organize the applications by category, like for the bookmarks

THIS or add custom sections in place of "Application" or "Bookmarks."
Unfortunately, it is looking like this may have been abandoned.

Same / homepage is awesome!


Add metrics reporter

OpenTelemetry's Ruby SDK has a MetricsReporter class which can be used to send metrics about the export pipeline (original pr). I think it would be useful in opentelemetry-lua so I have ported over the functionality.

Closing this in favor of a different PR for the batch span processor, which I'll hopefully open this week!

☝  that pr is here

Reopening, since we are going to keep using the existing batch span processor.


Enhancement: Add json output & elasticsearch

We were wondering if you have got any plans to add json output. It would be cool to have more information on the owner. You could put some owner information like owner's nickname, email, location, company. Most of the time that information is not available but it's better to have them.
If we have that information, it's quite easy to disclose private information. It would also be nice to index the findings on elasticsearch. In order to generate some cool graphs.

l like this. Good idea. I'll keep it in mind and see what I can do.


Proxy support

I couldn't see it mentioned anywhere in the docs or examples, but it'd be great to have proxy support. Other libraries like request and needle take a proxy option.

Please support proxies

Oh, so this is why I'm been struggling for a day to get the github-changes npm package to work behind my corporate proxy.


Finding an input that maximizes a class for text classification

I would like to replicate @fchollet's example here for text classification to obtain sequences that maximize the activations of specific filters or the activiations towards a certain output class.
I'm running on TensorFlow and have loaded the weights of a trained Sequential model with a Convolution1D layer.
Instead of input_img in the post, I set input_seq = K.placeholder((MAX_LEN, MAX_FEATURES)).
I access the output of a specified layer as in the example: layer_output = layer_dict[layer_name].get_output().
I calculate the loss as loss = K.mean(layer_output[:, filter_index, :]).
However, computing the gradients with K.gradients(loss, input_seq) yields [None].
Could you tell me what I'm doing wrong? Thank you so much for your help!

So I can't see your full code, but I believe you're passing through an input of two dimensions when Convolution1D expects three. See the documentation here for what I'm referring to http://keras.io/layers/convolutional/.
Thus your input_seq needs to be set to something like input_seq = K.placeholder((DIMENSION_1, DIMENSION_2, DIMENSION_3))
I don't know the nature of your data, so I don't know if this is what you want or need.

Thanks for the prompt reply! You're right. I somehow forgot that I need to feed the sequence directly into the convolutional layer.
I've now set input_seq = K.placeholder((1, nb_filter, filter_length)).
However, K.gradients(loss, input_seq) still returns [None].
For context, here is a simplified model that does what I want to do:
    model = Sequential()
    model.add(Embedding(max_features, embedding_size, input_length=max_len))
    model.add(Convolution1D(filter_length=filter_length, nb_filter=nb_filter))
    model.add(MaxPooling1D())
    model.add(Flatten())
    model.add(Dense(output_dim=num_classes))
    model.compile(loss='categorical_crossentropy', optimizer='adadelta')

Convolution1D expects (samples, steps, input_dim) as its 3d input. By giving an ```input_seq = K.placeholder((1, nb_filter, filter_length))`` Keras is getting a sample size of 1 and thus doing unexpected things. You should have multiple samples in your dataset for training.
Methinks you're using the wrong tool for the job here. You might be better off with a Dense layer or something that accepts 2D data.

For training, sure, you pass in multiple samples.
Note that in this case, the model is already trained and my goal is to find one sample that maximizes the activations of one layer or towards one output class, so that I can then map this back to a text representation.

I'm also facing the same error can anyone help

You are getting gradient as None because the embedding layer in keras are not differentiable. I have been trying to achieve the same that you tried, couldn't find a solution so far.


Refactoring CxPlatVector

All the methods are already defined in karray.h and fixing build issue for other project including this header for kernel build

I just pushed a change to hopefully fix the CI warnings.


DateRange Object

Checklist

[X] I'm asking a question and not reporting a bug/feature request
[X] I've looked through the README
[X] I've read the guidelines for opening an issue
[X] I've searched the bugtracker for similar questions including closed ones

Question
I'm trying to get the daterange option working in yt_dlp, but I get the following error:
[download] 2021-12-02 upload date is not in range {'start': '2021-11-07', 'end': '2021-12-07'}
daterange led me to the DateRange object, which looks for two key pairs, 'start' and 'end'. I've tried many variations of feeding it date formats but nothing works. What am I doing wrong here?
history = 30 #days how far back to download
history_end = date.today().isoformat()
history_start = ( date.today() - timedelta(days=history) ).isoformat()

ydl_opts = {
  'daterange': { 'start': history_start, 'end': history_end },

Try this:
from yt_dlp.utils import DateRange

ydl_opts = {
    'daterange': DateRange(history_start, history_end)
}


Feature Request: A prefers-color-scheme dark mode

The site is pretty bright 😅

Broken in https://github.com/rramiachraf/dumb/commit/cdc6dc418fdce09ac30f937dc0137558f3c1e223#diff-dd5820a5dbc134dd71e6eeffa5fcc73dbef9d5809fbb50a43982dd03d609f4bdL23 @rramiachraf


[Node 12] Tests fail, as Node.js has different error message for circular references detection

🐛 Bug report
Current Behavior
Steps to reproduce

Install Node.js v12.
Install all dependencies using npm ci.
Run npm test in the root of the project.

Actual result

One test from Either suite fails with the following diff:

 FAIL  test/Either.ts (20.551s)
  ● Either › stringifyJSON

    assert.deepStrictEqual(received, expected)

    Expected value to deeply and strictly equal to:
      {"_tag": "Left", "left": [TypeError: Converting circular structure to JSON]}
    Received:
      {"_tag": "Left", "left": [TypeError: Converting circular structure to JSON
        --> starting at object with constructor 'Object'
        --- property 'ref' closes the circle]}

    Difference:

    - Expected
    + Received

      Object {
        "_tag": "Left",
    -   "left": [TypeError: Converting circular structure to JSON],
    +   "left": [TypeError: Converting circular structure to JSON
    +     --> starting at object with constructor 'Object'
    +     --- property 'ref' closes the circle],
      }

      172 |     const circular: any = { ref: null }
      173 |     circular.ref = circular
    > 174 |     assert.deepStrictEqual(
          |            ^
      175 |       _.stringifyJSON(circular, _.toError),
      176 |       _.left(new TypeError('Converting circular structure to JSON'))
      177 |     )

      at Object.<anonymous> (test/Either.ts:174:12)

Expected result
Tests pass successfully.
Reproducible example
Current master branch HEAD.
Suggested solution(s)
I see two possible ways to fix this:

Check that error wrapped in Left is TypeError via instanceof.
Fold over the result of calling stringifyJSON and in left branch check the message for inclusion of substring "Converting circular structure to JSON".

Your environment

Software
Version(s)

fp-ts
2.0.5

TypeScript
3.6.2

In my opinion, the test can check both the instanceof and its message.
In some cases, I found myself folding the either and run the assertions within the branch the test covers and in the other branch I put an assertion like assert.strictEqual(leftOrRightValue, undefined) in order to make the test fail (I don't know if this is the best way, though).

In order to solve this issue reliably, one needs to:

Provide a short assertion for left part, as it is used as an example in the documentation for Either.
Construct this assertion in such way that it works for both v12, v10 and possibly v8, as it is still in the maintenance schedule.

The solution with folding over stringification result is a bit cumbersome, as it makes the example in comments for stringifyJSON method quite big and ugly:
/**
 * Converts a JavaScript value to a JavaScript Object Notation (JSON) string.
 *
 * @example
 * import { stringifyJSON, toError, right, fold } from 'fp-ts/lib/Either'
 * import { pipe } from 'fp-ts/lib/pipeable' // <- added
 *
 * assert.deepStrictEqual(stringifyJSON({ a: 1 }, toError), right('{"a":1}'))
 * const circular: any = { ref: null }
 * circular.ref = circular
 * // was just:
 * // assert.deepStrictEqual(stringifyJSON(circular, toError), left(new TypeError('Converting circular structure to JSON')))
 * // and now it should be written as:
 * pipe(
 *   stringifyJSON(circular, toError),
 *   fold(
 *     e => assert.strictEqual(e.message.includes('Converting circular structure to JSON'), true),
 *     () => assert.fail('Should not be Right')
 *   )
 * )
 *
 * @since 2.0.0
 */
export function stringifyJSON<E>(u: unknown, onError: (reason: unknown) => E): Either<E, string> {
  return tryCatch(() => JSON.stringify(u), onError)
}

An alternative came up to my mind:
const stringifiedRes = stringifyJSON(circular, toError)
assert.strictEqual(
  isLeft(stringifiedRes) &&
    stringifiedRes.left.message.includes(
      'Converting circular structure to JSON'
    ),
  true
)

I'm just throwing an idea out here, actually I prefer the solution with fold.

Reopening this (issue mismatch in the linked commit, #949 instead of #941)

Oh no, that‘s why it didn‘t close the other one … Thanks and sorry!

Am 29.09.2019 um 07:03 schrieb Giulio Canti<EMAIL_ADDRESS>
Reopening this (issue mismatch in the linked commit, #949 instead of #941)
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or mute the thread.


Support for Anyc creation of GraphQl Schema

Hello! I have the following usecase and have been digging around but don't see any documentation that supports my usecase which requires generating a GraphQl schema at runtime. If anyone has seen this done before please point me in the right direction :-) I appreciate it!

When the app starts, hit an external API to grab the configuration files of different Types and fields for that type.
Dynamically create a graphql Schema with queries/mutations/subscriptions/query inputs etc based on those fields/types loaded from the exernal API
Allow for the server to be able to perform CRUD operations against another API for each type
Have a tool like Graphiql update to show the latest types and fields and allow you to query/mutate each type

Is this possible in spring-graphql at the moment? Thanks!

Your application can provide its own GraphQlSource bean for that. Right now, there is a small limitation in the Spring Boot auto-configuration where a schema file on disk is still required, even if it is empty. This limitation is considered under https://github.com/spring-projects/spring-boot/issues/33096
Other than that, Spring for GraphQL should support your use case just fine.

AbstractGraphQlSourceBuilder has two sub-classes, one with the default way in which we initialize the GraphQLSchema e.g. by parsing schema files, and another with a given GraphQLSchema instance. You can probably use that as a base class, and override the initGraphQlSchema method.


fix get methods access issue

those getter methods should be public.

@AdoHe
Why all the new*Client have been made private ?
In addition I would make all the get*Client lazy i.e. wi th Suppliers.memorize so the client do not allocate unused objects.

I was about to add an issue for the same thing before a colleague spotted this pull request.  Can you please merge it?  thanks!

@bkrahmer merged, ptal.

@AdoHe so the new* are all private and clients are created inside the contructor ?


Nginx client_max_body_size

We tried to use this helm chart together with quarks.
While running a cf push it turned out that the nginx configuration is not capable to forward requests with sizes which exceeds 1MB.
Therefore it's necessary to add the the client_max_body_size configuration value.
location / {
  client_max_body_size       100m;
  access_log  /cloud_controller_ng/nginx-access.log;
  ...

@loewenstein its chugging along, and yes that story was one major piece. this is the placeholder that we keep breaking incremental stories out of: https://www.pivotaltracker.com/story/show/169404061

Hi @kramerul and @loewenstein - cf push on k8s should be working now! Please let us know if you run into any issues.


Fix ext_domains support, refactor domain string manipulation

We discovered that using domains to provide xip.io support broke our ability to make requests from other machines on the same network that are also running boxen, because when put in domains, *.xip.io starts resolving to localhost, even if you type in somebody else's IP.
But when we switched it to ext_domains, we realized that ext_domains support is broken for the nginx proxy mode.  This patch addresses that, as well as refactoring all of the string manipulation that takes place on domains and ext_domains in the module.
Because the strings were handled differently in different places, it turns out that the /etc/resolver files could end up with spaces on the front.
For example the following:
class {'pow':
  domains => 'pow, foo'
}

would result in a file named /etc/resolver/ foo.
This appears to have been unnoticed because the resolvers still functioned normally, even with the extra spaces.  Anyway, this PR also addresses that issue and tidies up several templates that were implementing their own string manipulation logic.

Closing and reopening to retrigger Travis build - failed due to API rate limiter on the 1.8.7 side.

Reopening.  Hopefully under the API rate limit.

Guess the API rate limit is persistent.  Anyway, the tests passed on 2.0.0 once, so I'm fairly confident the tests will pass when rerun off-peak.

Tests passed in Travis now that the rate limiter clock was reset.  Hopefully this PR is acceptable because it just makes a feature that was unusable before work correctly.  Thanks!

This change looks good to me - handling all whitespace manipulation in one place makes it a lot clearer.
I'm not sure who the best person from the @boxen org to review this is - @blackjid maybe?

looks good to me! :+1:
cc: @fromonesrc

@fromonesrc can you please take a look at this?? thnks!

Hi sorry I don't have involvement here anymore so I'm not the best person to review this

oh, ok, sorry

Alright, let's try someone else :grin:
@mikemcquaid, any chance you could review this change?

Seems fine to me. @blackjid and @eugeneius have you tested this on your local machine? Thanks!

(once you have: I'll merge)

I just tested it locally, and it worked as expected:

I pointed my Puppetfile at this branch and ran boxen - nothing changed
I switched back to master, added "xip.io" to ext_domains and ran boxen - the nginx config file wasn't updated to include it (reproducing the bug)
I switched to this branch again and ran boxen - the nginx config file was updated correctly

:ship:

Thanks @mikemcquaid :+1:
And thanks @jmileham for the patch!

Thanks all!
On Sun, Jul 19, 2015 at 12:49 AM Eugene Kenny<EMAIL_ADDRESS>wrote:

Thanks @mikemcquaid https://github.com/mikemcquaid [image: :+1:]
And thanks @jmileham https://github.com/jmileham for the patch!
—
Reply to this email directly or view it on GitHub
https://github.com/boxen/puppet-pow/pull/21#issuecomment-122625522.

Thanks! @mikemcquaid can you please make a realese with this fix? And perhaps take a look at https://github.com/boxen/puppet-pow/pull/20 and include it in the release?
Thanks!

@blackjid Are you interested in helping maintain this repo?

ok. I can help with that.. :)

@blackjid Done! Have fun :wink:

@blackjid I guess I can ask you now: can you please make a release with this fix? :)

@eugeneius here is the release! https://github.com/boxen/puppet-pow/releases/tag/2.3.1

Thanks! :+1:


DynamodDB Enhanced Client Design

Consider implementing an "enhanced" DDB client like offered by Java
Some considerations:

We might be able to do this as a compiler plugin and generate rather optimized code so that we don't have to rely on annotation processing (which I believe is JVM only at the moment) OR on reflection capabilities.

Resolving in favor of #472.


Update to govuk_tech_docs 3.4.0

Context

Dependabot wanted to update haml to version 6.3.0 in #104 but it didn't work when I ran it locally.
Googling the issue (Filters is not a module) pointed me to the fact that it's an issue with haml 6.x.x. The latest version of Tech Docs, 3.4.0, automatically sets haml to 5.x.x instead.
Running bundle update govuk_tech_docs worked, and I'm able to run the primer successfully on my machine.

This pull request is out of date - see #117 instead. Closing.


maven coordinates

It would be nice to have this project deployed to maven central repository. I
think it will be not that hard to distill a pom.xml for it.

I will try it later and attach the patch.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 10 Oct 2011 at 1:23

Alternatively, add Maven support. Then we could just fetch it via Jitpack.

@DimitrisAndreou would you like some help to get this done?


Roll Engine from 9ea763842e81 to 8fb7c35580b3 (2 revisions)

https://github.com/flutter/engine/compare/9ea763842e81...8fb7c35580b3
2021-04-22<EMAIL_ADDRESS>Wire up paragraph font weight and font style in canvas golden test (flutter/engine#25700)
2021-04-22<EMAIL_ADDRESS>Roll Skia from 716aeb900849 to<PHONE_NUMBER>a1 (2 revisions) (flutter/engine#25699)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md

Commit queue failed; closing this roll.


Handle CTRL-C gracefully in WP CLI

Description
We need to handle when a user cancels a long-running WP CLI command

Forward the Ctrl-C character (x03) to the command runner
Call an endpoint in the API to mark the command as cancelled (and do any other cleanup)

There are a few cases we need to test to make sure that the command cancel logic is run correctly on Ctrl-C/SIGINT:

Standard commands
Interactive command (such as user create --prompt, help)
Pseudo shell commands
wp shell


Backport/backport 12005 to 2.12

bug fix to have RestActions rawPath as operation names, query params as tags and avoid using exact URI (#10791)

Signed-off-by: Atharva Sharma<EMAIL_ADDRESS>Signed-off-by: Atharva Sharma<EMAIL_ADDRESS>Co-authored-by: Atharva Sharma<EMAIL_ADDRESS>(cherry picked from commit c0fca74709e96ec7c8cf295cdca6110946b592b8)

Description
Backport/backport 12005 to 2.12
Related Issues
Resolves #[Issue number to be closed when this PR is merged]

Check List

[x] New functionality includes testing.

[x] All tests pass

[x] New functionality has been documented.

[x] New functionality has javadoc added

[x] Failing checks are inspected and point to the corresponding known issue(s) (See: Troubleshooting Failing Builds)
[x] Commits are signed per the DCO using --signoff
[x] Commit changes are listed out in CHANGELOG.md file (See: Changelog)
[x] Public documentation issue/PR created

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@atharvasharma61, is it just a cherrypick only from the main branch right?

yes @Gaganjuneja


Fix the caBundle issue in the admission controller in helm chart

During the test, I found when deploying the scheduler + admission controller using the helm chart. The webhook server log prints some "TLS handshake error". This was because of the caBundle value set for the webserver was missing. Kubectl is not able to retrieve cluster context using in cluster configuration (running inside of the pod). The fix is to change a way of retrieving the ca.
The fix works for both helm chart and local deployment.

hi @wilfred-s  pls help to review this, thx!


Wrong Message/Handing when Re-Registering

Follow up to https://github.com/SAP/cloud-commerce-spartacus-storefront/issues/2799
Steps to Reproduce (From Kasper)

Open the app
Go to register form
register a completely fresh user (at this point you should be redirected to login page and have global message with success info)
4.log out
Go to register form again
Register THE SAME user as above (you also should be redirected to login page with the same global message)

From Kate:
Registering the same existing user (with both the correct and wrong passwords) give the same message on the login page.
From GC:
When attempting to re-register a user, after redirection to login page the wrong error message says "Please login with your new credentials", which is false. There are no new credentials.
Here, a decision is needed as follows:
1.a. Show nothing and redirect to login page.
1.b. Show error message stating email is already in use and remain in registration page.
1.c. Show error message and redirect to login page.
Don't understand the REST POST call /rest/v2/electronics-spa/users/?lang=en&curr=USD after redirection. It contains the user data of the previous registration attempt.

https://github.com/SAP/cloud-commerce-spartacus-storefront/issues/2799#issuecomment-519011442


[FEATURE-REQUEST] Automatically allocate port for new app

If I use sc start /path/to/my/app.yml which doesn't have a port specified, I'd like sc to automatically allocate and re-use this port every time I redeploy the service

Also expose it as an environment variable (PORT)


Bulk delete

Deleting one at a time is great and all - but when we have 900+ entries to go through and delete... its a bit tedious!
Is there a bulk select i am missing or?
Thanks, N

pr #77 added bulk delete support.


Heroku logplex format does not detect autovacuum and autoanalyze logs

I have tried to gather information about vacuums in my system but I wasn't able to. Reports like Lock worked fine but I wasn't able to make autovacuum and autoanalyze logs parsed.
Here is an example of what I've tried to parse:
2019-05-30T00:04:39+00:00 app[postgres.115322]: [DATABASE] [8360-1]  LOG:  automatic vacuum of table "da39uu6nuoaudt.public.sales": index scans: 1
2019-05-30T00:04:39+00:00 app[postgres.115322]: [DATABASE] [8360-2] 	pages: 0 removed, 2578826 remain, 0 skipped due to pins, 90246 skipped frozen
2019-05-30T00:04:39+00:00 app[postgres.115322]: [DATABASE] [8360-3] 	tuples: 4564 removed, 44092212 remain, 305 are dead but not yet removable, oldest xmin:<PHONE_NUMBER>
2019-05-30T00:04:39+00:00 app[postgres.115322]: [DATABASE] [8360-4] 	buffer usage: 892560 hits, 3220638 misses, 95107 dirtied
2019-05-30T00:04:39+00:00 app[postgres.115322]: [DATABASE] [8360-5] 	avg read rate: 2.402 MB/s, avg write rate: 0.071 MB/s
2019-05-30T00:04:39+00:00 app[postgres.115322]: [DATABASE] [8360-6] 	system usage: CPU: user: 44.86 s, system: 22.59 s, elapsed: 10476.99 s
2019-05-30T00:06:25+00:00 app[postgres.115322]: [DATABASE] [8361-1]  LOG:  automatic analyze of table "da39uu6nuoaudt.public.sales" system usage: CPU: user: 1.49 s, system: 0.67 s, elapsed: 106.26 s
2019-05-30T00:07:57+00:00 app[postgres.115322]: [DATABASE] [8362-1]  LOG:  automatic analyze of table "da39uu6nuoaudt.public.tips" system usage: CPU: user: 0.58 s, system: 0.00 s, elapsed: 91.33 s
2019-05-30T00:18:21+00:00 app[postgres.14355]: [DATABASE] [8362-1]  LOG:  automatic analyze of table "da39uu6nuoaudt.public.bookings" system usage: CPU: user: 1.62 s, system: 1.04 s, elapsed: 202.40 s
2019-05-30T00:18:54+00:00 app[postgres.20051]: [DATABASE] [8357-1]  LOG:  automatic vacuum of table "da39uu6nuoaudt.public.events": index scans: 1
2019-05-30T00:18:54+00:00 app[postgres.20051]: [DATABASE] [8357-2] 	pages: 53 removed, 217 remain, 0 skipped due to pins, 0 skipped frozen
2019-05-30T00:18:54+00:00 app[postgres.20051]: [DATABASE] [8357-3] 	tuples: 1843 removed, 32 remain, 0 are dead but not yet removable, oldest xmin:<PHONE_NUMBER>
2019-05-30T00:18:54+00:00 app[postgres.20051]: [DATABASE] [8357-4] 	buffer usage: 10198 hits, 9132 misses, 149 dirtied
2019-05-30T00:18:54+00:00 app[postgres.20051]: [DATABASE] [8357-5] 	avg read rate: 2.238 MB/s, avg write rate: 0.037 MB/s
2019-05-30T00:18:54+00:00 app[postgres.20051]: [DATABASE] [8357-6] 	system usage: CPU: user: 0.05 s, system: 0.06 s, elapsed: 31.88 s

I compiled my version from current master with current HEAD on commit b8f53eaa010779380a4791eae3b4312a5d4a0c16

Thanks for the report, last commit 88eefb1 might fix this issue.

Works perfectly. Thanks!


Check whether the aligned pointers value of the internal properties c…

…an be stored directly
in ecma_value_t or not.
This change affects those internal properties that haven't been checked previously.
JerryScript-DCO-1.0-Signed-off-by: Robert Sipka<EMAIL_ADDRESS>

Please fix the format of the commit message.

Sure, I've updated the commit message.

Patch is ok, but I would prefer a bit better description of the commit message.

Thanks for the review, I've updated the commit message, and I've rebased this patch with the current master.

LGTM

LGTM


Notes field

[x] Allow entering notes
[x] import notes from Neon if possible

Note import turns out to be unnecessary -- all "pinned notes" in neon are year specific.


Updated #1871 to remove lodash

@joliss did extensive work to remove the lodash dependency.

https://github.com/Chevrotain/chevrotain/pull/1871

We are trying to replace Jison with a Chevrotain based parser (langium) at MermaidJS.
As our primary use case is browser based, package size is a major issue for us.
lodash is also causing many build time warnings when built with vite.
Size comparison

File
master
remove_lodash
~reduction
reduction kb

chevrotain.js
627757
410573
34%
217

chevrotain.min.js
203501
162222
20%
41

chevrotain.mjs
517576
351780
32%
166

chevrotain.min.mjs
209131
164468
21%
45

merging into remove_lodash_new branch to apply fixes

superseded by #1949


[AMBARI-23998] AMS collector RPM configs are empty (dsen)

What changes were proposed in this pull request?
Added default AMS config values
How was this patch tested?
Manually

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/2538/
Test PASSed.


http gone in v2?

I believe non-secure http redirects are disallowed in version 2 of the API, making the section below a bit misleading:
For web applications, provide the base URL of your app. For example, http://localhost:31544 might be the URL for a web app running on your local machine. Users would use this URL to sign in to a web client application.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: c9642c47-826b-017f-eff3-ab3b25e4e454
Version Independent ID: 091f26e7-6543-ef0f-b575-b44519943391
Content: Register an app with the Microsoft identity platform - Microsoft identity platform
Content Source: articles/active-directory/develop/quickstart-register-app.md
Service: active-directory
Sub-service: develop
GitHub Login: @rwike77
Microsoft Alias: ryanwi

@heyoni
Thanks for your feedback! We will investigate and update as appropriate.

I agree. I made a pull request to update this. https://github.com/MicrosoftDocs/azure-docs-pr/pull/91476
Thanks for catching this!


Customizing report of 2020

Types of changes

[ ] Bugfix
[ ] New feature
[x] Refactoring
[ ] Breaking change (any change that would cause existing functionality to not work as expected)
[ ] Documentation Update
[ ] Other (please describe)

Description
2020 customization.
Checklist:

[ ] Add test cases to all the changes you introduce
[x] Run inv lint locally to ensure all linter checks pass
[x] Run inv test locally to ensure all test cases pass
[x] Run inv secure locally to ensure no major vulnerability is introduced
[x] Update the documentation if necessary

Steps to Test This Pull Request
Steps to reproduce the behavior:

run the command rg-cli with arguments shown in contrib/arguments-2020.txt

Expected behavior
Report of 2020 will be generated.

@Lee-W  FYI, I was not aware of missing lint task until now : )
My quick local inv test result:
(pycontw-postevent-report-generator) tai271828@syakaro:~/work-my-projects/pycontw-projects/pycontw-postevent-report-generator$ inv lint
No idea what 'lint' is!
(pycontw-postevent-report-generator) 1 tai271828@syakaro:~/work-my-projects/pycontw-projects/pycontw-postevent-report-generator$ inv test
======================================================================================== test session starts ========================================================================================
platform linux -- Python 3.7.5, pytest-6.1.1, py-1.9.0, pluggy-0.13.1
rootdir: /home/tai271828/work-my-projects/pycontw-projects/pycontw-postevent-report-generator, configfile: pytest.ini
plugins: mock-3.3.1, cov-2.10.1, hypothesis-5.37.3
collected 12 items

tests/test_html_exporter.py ...                                                                                [ 25%]
tests/test_sponsor.py ....ssss                                                                                [ 91%]
tests/test_title.py .                                                                                [100%]

========================================================================================= warnings summary ==========================================================================================
tests/test_title.py::TestTitleCategory::test_mapping
  /home/tai271828/work-my-projects/pycontw-projects/pycontw-postevent-report-generator/report_generator/cleaner/title.py:42: DeprecationWarning: Flags not at the start of the expression '學生|(?i)student'
    if re.search(pattern_dic[pattern], title) is not None:

-- Docs: https://docs.pytest.org/en/stable/warnings.html
============================================================================== 8 passed, 4 skipped, 1 warning in 1.74s ==============================================================================
(pycontw-postevent-report-generator) tai271828@syakaro:~/work-my-projects/pycontw-projects/pycontw-postevent-report-generator$ inv secure
Checking PEP 508 requirements…
Passed!
Checking installed package safety…
All good!

I use inv style (an alias to inv style check). We could change it to inv lint as well. I'm good with both.

This PR really helps improve this project  I finish reviewing and leave some comments. Hope them helps. Also, I just fix the CI. You might what to fix some styles as well.

I appreciate what you have done for the table manner of this project. It also helps A LOT. : D

hmmm interesting, the CI is in the status of "Some checks haven’t completed yet" for 2 hours. It is unexpectedly slow, or something is wrong. Let me try to change the "draft" status to "ready for review" to see what will differ.

Ok it seems that we need to manually trigger CI. I have triggered one for this PR https://github.com/pycontw/pycontw-postevent-report-generator/actions/runs/417939433

Let's merge this PR 😄


Create game  and author classes

In this milestone:

Added Game class
Added Author class

Hello @naanmohammed,
Your project is completed, there is nothing else to say than it's time to merged.
Highlights

[] No linter errors.
[] Good commit description.
[] Project is looking well documented.


Add intertidal elevation notebook

Proposed changes
Intertidal elevation notebook adapted.
Checklist (replace [ ] with [x] to check off)

[x] Remove any unused Python packages from Load packages
[x] Remove any unused/empty code cells
[x] Remove any guidance cells (e.g. General advice)
[x] Ensure that all code cells follow the PEP8 standard for code. The jupyterlab_code_formatter tool can be used to format code cells to a consistent style: select each code cell, then click Edit and then one of the Apply X Formatter options (YAPF or Black are recommended).
[x] Include relevant tags in the final notebook cell and re-use tags if possible
[x] Clear all outputs, run notebook from start to finish, and save the notebook in the state where all cells have been sequentially evaluated

Hey @fangfy , it looks great! Would you be able to re-run all the cells in the notebook and push it again? It's currently only evaluated up to Cell 12.

Hey @fangfy , it looks great! Would you be able to re-run all the cells in the notebook and push it again? It's currently only evaluated up to Cell 12.

ah I pushed to early. Should be fixed now.


add 2 domain for subdomain UTEM  & KKTW

Hi,  i've added 2 new domain for subdomain utem ex<EMAIL_ADDRESS>&<EMAIL_ADDRESS>

@hannanyusop I am sorry, the domain student.utem.edu.my is removed from our trusted domains list because the school allows to register an email address there even if you are not a student of the school. If you are the student of this school, please apply with official document (OFFICIAL DOCUMENT tab at https://www.jetbrains.com/shop/eform/students).


Create release of STAC extension and update Geostore STAC version

User Story

So that I can start using all the mandatory STAC extension elements, as a data maintainer, I want the geostore to valdiate against the latest LINZ stac extensions .

Acceptance Criteria

[ ] Given? , when  ?, then all the mandaotry extensions are available in teh Geostore
[ ] Given [a pre-condition], when [an event happens], then [an expected outcome occurs]

Additional context

Tasks

[x] Create new linz/stac release
[x] Resolve whether file:checksum should be in the LINZ STAC extension or a new Geostore extension
[x] Make file:checksum required
[x] Create new linz/stac release
[ ] Support catalog in LINZ schema
[ ] Create new linz/stac release
[ ] Bump version in the Geostore (referencing the new extension if that's where file:checksum ends up)
[ ] Fix all the failing tests

Definition of Ready

[ ] This story is ready to work on

[ ] Independent (story is independent of all other tasks)
[ ] Negotiable (team can decide how to design and implement)
[ ] Valuable (from a user perspective)
[ ] Estimate value applied (agreed by team)
[ ] Small (so as to fit within an iteration)
[ ] Testable (in principle, even if there isn't a test for it yet)
[ ] Environments are ready to meet definition of done
[ ] Resources required to implement will be ready
[ ] Everyone understands and agrees with the tasks to complete the story
[ ] Release value (e.g. Iteration 3) applied
[ ] Sprint value (e.g. Aug 1 - Aug 15) applied

Definition of Done

[ ] This story is done:

[ ] Acceptance criteria completed
[ ] Automated tests are passing
[ ] Code is peer reviewed and pushed to master
[ ] Deployed successfully to test environment
[ ] Checked against CODING guidelines
[ ] Relevant new tasks are added to backlog and communicated to the team
[ ] Important decisions recorded in the issue ticket
[ ] Readme/Changelog/Diagrams are updated
[ ] Product Owner has approved acceptance criteria as complete
[ ] Meets non-functional requirements:

[ ] Scalability (data): Can scale to 300TB of data and 100,000,000 files and ability to
increase 10% every year
[ ] Scability (users): Can scale to 100 concurrent users
[ ] Cost: Data can be stored at < 0.5 NZD per GB per year
[ ] Performance: A large dataset (500 GB and 50,000 files - e.g. Akl aerial imagery) can be
validated, imported and stored within 24 hours
[ ] Accessibility: Can be used from LINZ networks and the public internet
[ ] Availability: System available 24 hours a day and 7 days a week, this does not include
maintenance windows < 4 hours and does not include operational support
[ ] Recoverability: RPO of fully imported datasets < 4 hours, RTO of a single 3 TB dataset <
12 hours

Blocked by https://github.com/linz/stac/pull/128.