Fix for: Inconsistency when double click the avatar and the search in LHN

Details

Fixed Issues

$ https://github.com/Expensify/App/issues/15692
PROPOSAL: https://github.com/Expensify/App/issues/15692#issuecomment-1460525468
Tests

Double click on the search in LHN
Verify that the Search Page is opened and closed immediately and that the text-input placeholder is not selected

[x] Verify that no errors appear in the JS console

Offline tests

Double click on the search in LHN
Verify that the Search Page is opened and closed immediately and that the text-input placeholder is not selected

QA Steps

Double click on the search in LHN
Verify that the Search Page is opened and closed immediately and that the text-input placeholder is not selected

[ ] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is correct English and approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG)

[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web
https://user-images.githubusercontent.com/11609254/223968386-ec4fa825-800b-42f8-b1cb-b90563d7b4ed.mp4

Mobile Web - Chrome
https://user-images.githubusercontent.com/11609254/223968421-0c5d4d63-f9e1-4801-8b32-f861d3429bb6.mp4

Mobile Web - Safari
https://user-images.githubusercontent.com/11609254/223968458-3f430254-6320-409e-b25f-8465eed4a84a.mp4

Desktop
https://user-images.githubusercontent.com/11609254/223968486-854787f3-a7bc-43b7-a918-2fc7606cc549.mp4

iOS
https://user-images.githubusercontent.com/11609254/223968542-99004195-ec4e-44e5-aac8-b2f534f5dc7f.mp4

Android
https://user-images.githubusercontent.com/11609254/223968575-898c4b86-959a-4031-8505-69a6e6bf08d0.mp4

@MariaHCD @s77rt One of you needs to copy/paste the Reviewer Checklist from here into a new comment on this PR and complete it. If you have the K2 extension, you can simply click: [this button]

@Ollyws Few notes regarding the PR template:

Double click on the search icon in LHN (missing word icon for clarify)
Verify that the Search Page is opened and closed immediately (this is enough)
Offline tests: remove those and just put n/a since there is difference being offline

Also please update the expected behaviour accordingly, i.e. on mWeb and Native, you can't close the search modal immediately since there is no backdrop to press on.

Write it this way for more clarity:
Tests

Double click on the search icon in LHN

Desktop/Web: Verify that the Search Page is opened and closed immediately
Native/mWeb: Verify that the Search Page is opened and function normally

Reviewer Checklist

[X] I have verified the author checklist is complete (all boxes are checked off).
[X] I verified the correct issue is linked in the ### Fixed Issues section above
[X] I verified testing steps are clear and they cover the changes made in this PR

[X] I verified the steps for local testing are in the Tests section
[X] I verified the steps for Staging and/or Production testing are in the QA steps section
[X] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[X] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[X] I checked that screenshots or videos are included for tests on all platforms
[X] I included screenshots or videos for tests on all platforms
[X] I verified tests pass on all platforms & I tested again on:

[X] Android / native
[X] Android / Chrome
[X] iOS / native
[X] iOS / Safari
[X] MacOS / Chrome / Safari
[X] MacOS / Desktop

[X] If there are any errors in the console that are unrelated to this PR, I either fixed them (preferred) or linked to where I reported them in Slack
[X] I verified proper code patterns were followed (see Reviewing the code)

[X] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[X] I verified that comments were added to code that is not self explanatory
[X] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[X] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method
[X] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[X] I verified any copy / text that was added to the app is correct English and approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[X] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[X] I verified the JSDocs style guidelines (in STYLE.md) were followed

[X] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[X] I verified that this PR follows the guidelines as stated in the Review Guidelines
[X] I verified other components that can be impacted by these changes have been tested, and I retested again (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar have been tested & I retested again)
[X] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[X] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[X] If a new component is created I verified that:

[X] A similar component doesn't exist in the codebase
[X] All props are defined accurately and each prop has a /** comment above it */
[X] The file is named correctly
[X] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[X] The only data being stored in the state is data necessary for rendering and nothing else
[X] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[X] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[X] All JSX used for rendering exists in the render method
[X] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[X] If any new file was added I verified that:

[X] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[X] If a new CSS style is added I verified that:

[X] A similar style doesn't already exist
[X] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG)

[X] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[X] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[X] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[X] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[X] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

Screenshots/Videos

Web
https://user-images.githubusercontent.com/16493223/224016820-4037899a-12d7-4c39-9c84-e9c4ae5bb310.mp4

Mobile Web - Chrome
https://user-images.githubusercontent.com/16493223/224016475-f83dfc83-a4b6-489b-a1bf-af49ddcd53d5.mp4

Mobile Web - Safari
https://user-images.githubusercontent.com/16493223/224015878-3ef84b5b-f285-4fa2-b83a-8922f846b654.mp4

Desktop
https://user-images.githubusercontent.com/16493223/224016839-d38c2cbe-0598-4fec-8ab9-ee9e981bdece.mp4

iOS
https://user-images.githubusercontent.com/16493223/224015793-a69f75ea-697d-4ad6-936a-e1e55e59a18f.mp4

Android
https://user-images.githubusercontent.com/16493223/224015767-ae3c317d-80cd-40f0-a8de-b1d433a555ee.mp4

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/MariaHCD in version: 1.2.82-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/Julesssss in version: 1.2.82-4 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


add pandera accessor to pandas objects

this PR adds a pandas_obj.pandera accessor to DataFrames and Series objects
to make it easier to inspect pandas objects that have been validated with a pandera
schema

Codecov Report

Merging #336 (f1d7f08) into dev (e16bfe6) will decrease coverage by 0.07%.
The diff coverage is 95.12%.

@@            Coverage Diff             @@
##              dev     #336      +/-   ##
==========================================
- Coverage   98.85%   98.78%   -0.08%
==========================================
  Files          19       20       +1
  Lines        2102     2136      +34
==========================================
+ Hits         2078     2110      +32
- Misses         24       26       +2

Impacted Files
Coverage Δ

pandera/pandas_accessor.py
93.10% <93.10%> (ø)

pandera/__init__.py
100.00% <100.00%> (ø)

pandera/schemas.py
97.46% <100.00%> (+0.02%)
:arrow_up:

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 e16bfe6...f1d7f08. Read the comment docs.


Update data-flow-expression-functions.md

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

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

Just remember to back-port your fixes to the live MD to the generated MD from the executor code so that these don't get overwritten every time that you produce a new MD file

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

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

#assign:djpmsft


Broken ace editor theme(s)

The website becomes completely unusable upon selecting a listed but seemingly unavailable ace editor theme.
The theme in this case was one_dark-css.
Expected: theme changes to one_dark-css.
Actual: console displays Error: couldn't load module ace/theme/one_dark-css or it didn't call define at 478a968b73ce7d449235.js:2:304601, website interface completely crashes and just shows the background color.
Attempted: clearing the cache, cookies and local storage did not help, but the website works if opened in a private window so though I don't know where the user config is pulled from, it's fairly logically the source of the issue (specifically the configuration.ace.theme property).

Odd. Ace 1.23 changed something with how they package the themes. While I work on that, I'll downgrade to 1.22.
Thanks for the report!


New API Version 2017-08-01: Adding Sku property into publicIP and loa…

…d balancer (#1528)

Updating documentation for Microsoft.VisualStudio with project resource APIs. Corrections to existing examples. (#1498)

Added Api-Version 2017-07-01 for RecoveryServices.Backup Jobs (#1502)

Added Api-Version 2017-07-01 for RecoveryServices.Backup Jobs

Updated tag names

Microsoft.Sql - Added data sync to package-2015-05-preview (#1510)

Added advisors and data sync to package-2015-05-preview

Removed advisors so current branch is only data sync

Make MsDeploy and MsDeployLog Azure resources (#1519)

Add public certificates, Functions Admin Token and MSDeploy APIs

Revert resource definition changes and description change for site properties

Make type and name readonly properties. Add ARM envelope to MSDeployStatus object

Fix missing quotation issue.

Add long running operation to MS deploy

Make MsDeploy and MsDeployLog Azure resources

Fix some AutoRest validation issues. Use dictionary for msdeploy.setparameters

Ensure models are same across all schemas in Microsoft.Web

Fix azure resource type across all models

Remove conflicting ListOperations. It is not used for public Azure.

no YAML in glob patterns for report scripts

Copying 2017-06-01 to 2017-08-01

Changes for new API version

Updated script to run autorest with grouping swaggers based on config file (#1529)

Removed the maxLength for entities (#1527)

Update folder structure example. (#1489)

Agreed on the changes

[Travis-ci] Oad in ci status (#1520)

Breaking change build should fail if result contains error
Skipping newly added files
Incorporating review feedback

testing

log the result as oad.compare returns promise for caller

Use published oad

Revert "testing"

This reverts commit a9be94db024716e5a563c4bd418332be9aaf329b.

Removed maxLength from entity name (#1531)

This is to add Cognitive Services data-plane API specs starting with Face API. (#1467)

Adding data plane spec starting with Face

Move Spec to right folder

Fixing validation issues.

Fixing issues and adding a readme.md

Fixing examples by removing decimals.

Removing another decimal value

Update readme.md

Updating spec based on comments.

Removing .gitignore.

Relay: Removed the maxLength for entity names (#1535)

This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.
PR information

[ ] The title of the PR 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 information on cleaning up the commits in your pull request, see this page.
[ ] Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
[ ] If applicable, the PR references the bug/issue that it fixes.
[ ] Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

[ ] I have read the contribution guidelines.
[ ] My spec meets the review criteria:

[ ] The spec conforms to the Swagger 2.0 specification.
[ ] The spec follows the guidelines described in the Swagger checklist document.
[ ] Validation tools were run on swagger spec(s) and have all been fixed in this PR.

@gmainar,
Thanks for your contribution.  To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla.microsoft.com.
The agreement will cover your contributions to all Microsoft-managed open source projects.
Thanks,
Microsoft Pull Request Bot

@Anu Thomas Chandy

@anuchandy , can you please merge?

No modification for AutorestCI/azure-sdk-for-node

 https://github.com/Azure/azure-sdk-for-ruby/pull/894

 https://github.com/Azure/azure-sdk-for-python/pull/1366


Dynamically adding new validators

Description
The current approach to adding new nodes to the network requires the whole network to update their respective validator config and reboot the network. This approach works when developing and testing but is not feasible for production. It is crucial that new nodes could be added or removed while the existing nodes are running.
Tendermint allows the application to update the validator's set by allowing to send validator diff during end_block response.
How this works
Validators in Tendermint propose and vote on blocks. A block is validated by a set of validators which is stored as ValidatorsHash in the header of the block. In case a new validator joins the network there are two possible scenarios i.e. the validator is known by >2/3 of the network or the validator is not known by >2/3 of the network. In the latter case i.e. when majority of the network doesn't know the validator any block which includes this new validator in its ValidatorsHash won't be able to get majority votes on that block. Hence the new validator cannot participate in the network untill a majority of the nodes acknowledge the new validator.
Approach 1
Expose end_block directly to the client app using BDB via a web hook. The client would return a list of validators to add/remove.
Downside
Possible peformance hit since the web hook would be called every time during end_block.
Approach 2
When using the BFT version (i.e. BDB w/ Tendermint or TBDB) it is reasonable to expect a permissioned network. This means that the network should be made aware of any new joining nodes using off-chain[1] methods i.e. each node might have their own logic of allowing new nodes in the network.
The list of validators is maintained as an asset in the database. To add a new validator it has to secure >2/3 of voting power (using off-chain methods). Once that is achieved any existing node can submit a transaction which includes the pub_key and power of the new node. Each node in the network validates[2] this transaction and adds the node by returning the validator diff list in end_block response.
Technicals

When the network is booted init_chain is called with current list of validators. This should be used to initialize the validators list in BDB (maybe in form of an asset).
When a new node has secured a majority voting power it can ask any existing node to trigger a transaction which adds its public key to list of known validator nodes.
New APIs need to be exposed:

POST /api/v1/node : add a new node
PUT /api/v1/node : update an existing node.
DELETE /api/v1/node : delete a existing node from the network.

The client could expose a webhook which would be called by the BDB server to validate the above discussed transaction.

Downside
Complex !
Related Issues
Either of the approaches suggested above will have to implement a web hook system so that the client app using bigchaindb could be delegated the task of the verifying if the new validator is in fact valid. A general implementation of web hooks system would facilitate resolving #1868
Appendix
[1] It is also possible to use on chain methods and the client app can implement mechaincs such as buying token to achieve validator status.
[2] The validaton should be delegated to the client app. If the client doesn't implement a validation logic then the transaciton is assumed to be invalid.

We've implemented "upsert-validator elections" and all that remains to do is update some documentation; that is covered by issue #2372, so this issue can be closed.


Allow entire blueprints to have lazy-loaded views despite using route decorators

For me, a single blueprint naturally goes in a module folder, and all the views go in a views.py there. If that's how you do it, it makes sense to lazy-load one blueprint at a time. But in order to do that, you have to change your whole routing system, using add_url_rule() instead of @bp.route(). I've got a fix for this, but it takes some modification of flask itself (so that there's a way to later .remove() the temporary lazy-loader rule added by add_url_rule; and so that you can app.full_redispatch_request after the lazy-loading is done).
Is this some horrid sin from the evil world of Django, or would my patches be welcome, or what?
Here's my lazyblueprint.py for reference:
from flask import Blueprint

class LazyBlueprint(Blueprint):
    def __init__(self, name, import_name, static_folder=None,
                 static_url_path=None, template_folder=None,
                 url_prefix=None, subdomain=None, url_defaults=None):
        super(LazyBlueprint, self).__init__(name, import_name, static_folder,
                 static_url_path, template_folder,
                 url_prefix, subdomain, url_defaults)
        self.views = '.'.join([import_name,'views'])
        print import_name, self.views
        self.lazyrules = []
        self.states = []
        self.record(self.setuplazyload)

    def setuplazyload(self, state):
        self.lazyrules.append(state.add_url_rule('/<path:endpoint>', 'index', self.lazyloadfor(state)))
        self.lazyrules.append(state.add_url_rule('/', 'index', self.lazyloadfor(state),
                                                  defaults={'endpoint':''}))
        self.states.append(state)

    def lazyloadfor(self, state):
        def innerlazyload(endpoint, **kw):
            for rule in self.lazyrules:
                rule.remove()
            self.base_functions, self.deferred_functions = self.deferred_functions, []
            __import__(self.views)

            state.app.debug, debug = False, state.app.debug #don't trigger setupfunction warnings
            for fixstate in self.states:
                for deferred in self.deferred_functions:
                    deferred(fixstate)
            state.app.debug = debug
            self.deferred_functions = self.base_functions + self.deferred_functions
            return self.redispatch(state, endpoint, **kw)
        return innerlazyload

    def redispatch(self, state, endpoint, **kw):
        return state.app.full_redispatch_request()

Here's how you'd use it: first skyblue/init.py:
from lazyblueprint import LazyBlueprint

skyblue = LazyBlueprint('simple_page', __name__,
                        template_folder='templates')

then skyblue/views.py:
from skyblue import skyblue

@skyblue.route('/')
def index():
    return "sky:1...blue:2...Say hi!"

@skyblue.route('/hello')
def hello():
    return "Hello skyblue!"

You're welcome to do this sort of autodetection in your own app, but it's not a pattern that belongs in Flask.


Multiple builds map to same Docker image

The current implementation assumes that the Docker images for a build are unique to that build and will try to remove the images when the build is destroyed.  Although the error it returns is printed but otherwise ignored, this is an invalid assumption particularly in more complicated deployment environments where seed reuse may be significant.

Fixed in 6d18a4b67866d33f704d9f1a6ab604179f6c86a0
Refactored code path so that build IDs are assigned earlier and use the ID for the tag within the Docker "repository".


[Event Request] Codeunit 1535 "Approvals Mgmt." - OnBeforeIsPurchaseApprovalsWorkflowEnabled

Hi, would it be possible to get an OnBeforeIsPurchaseApprovalsWorkflowEnabled event in codeunit 1535 (Approvals Mgmt.), like this:
    procedure IsPurchaseApprovalsWorkflowEnabled(var PurchaseHeader: Record "Purchase Header"): Boolean
    begin
        OnBeforeIsPurchaseApprovalsWorkflowEnabled(PurchaseHeader);  // New Event

        exit(WorkflowManagement.CanExecuteWorkflow(PurchaseHeader, WorkflowEventHandling.RunWorkflowOnSendPurchaseDocForApprovalCode));
    end;

    [IntegrationEvent(false, false)]
    local procedure OnBeforeIsPurchaseApprovalsWorkflowEnabled(var PurchaseHeader: Record "Purchase Header");
    begin
    end;

Thanks!

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Building the Dockerfile fails

 docker build -t cloudmapper .

[+] Building 3.1s (8/12)
 => [internal] load build definition from Dockerfile                                                                                0.0s
 => => transferring dockerfile: 86B                                                                                0.0s
 => [internal] load .dockerignore                                                                                0.0s
 => => transferring context: 83B                                                                                0.0s
 => [internal] load metadata for docker.io/library/python:3.7-slim                                                                                1.9s
 => [1/8] FROM docker.io/library/python:3.7-slim@sha256:ebc447a481b94b3afe2587ccdcc0128c3175fcfeb89ee33461a562b41310d403                               0.0s
 => [internal] load build context                                                                                0.0s
 => => transferring context: 21.94kB                                                                                0.0s
 => CACHED [2/8] WORKDIR /opt/cloudmapper                                                                                0.0s
 => CACHED [3/8] RUN apt-get update -y                                                                                0.0s
 => ERROR [4/8] RUN apt-get install -y build-essential autoconf automake libtool python3.7-dev python3-tk jq awscli                                    1.1s
------
 > [4/8] RUN apt-get install -y build-essential autoconf automake libtool python3.7-dev python3-tk jq awscli:
#7 0.204 Reading package lists...
#7 0.777 Building dependency tree...
#7 0.913 Reading state information...
#7 1.096 E: Unable to locate package python3.7-dev
#7 1.096 E: Couldn't find any package by glob 'python3.7-dev'
#7 1.096 E: Couldn't find any package by regex 'python3.7-dev'
------
executor failed running [/bin/sh -c apt-get install -y build-essential autoconf automake libtool python3.7-dev python3-tk jq awscli]: exit code: 100

I believe this PR https://github.com/duo-labs/cloudmapper/pull/892 fixes it @richstokes


Looking forward to get involved and contribute to the community 🎉

First Name
Mukul
Last Name
Kolpe
Discord Username (if applicable)
Mukul#0973
Let us understand your motive.
Excited to get involved in the community to learn, explore new opportunities and help beginners. Thanks for the great initiative.

Hey @MukulKolpe 👋🏻
Thanks for requesting an invite! Welcome to SidHacks Community 🎉
Before going ahead, I would want you to complete the below To-Do list.
Your To-Do List ✔

Accept the invitation sent to you via mail.
Change the appearance of this organization to "Public" from the people list here.
Join our Discord Server here.

Accept the Rules from #📓〡rules channel.
Introduce yourself to our community in 👋〡introduction channel.

Feel free to share your excitement on socials with #SidHacksCommunity. You can also tag me @siddharth_hacks.

Close this issue once you complete all the above points.

Looking forward to interacting with you in our community! Welcome once again! 🤓


community button not rizzing

i press the community button on https://brainrotlang.com/ and its yapping about "no text channels"

Here's the invite: https://discord.gg/G9BqwB3a

Fixed on this commit: 751bf5a9c92b580e7b13ed4631061dfd27a68ced


An efficient way of labelling issues on github

We can follow scikit's labelling scheme

Okay, so I have added some labels. Can you guys check and close the issue? @mukkachaitanya @prampey

Yeah. Looks great!

What does the blocker tag imply?

blocker means, if it is impacting the closing of a major (meta) issue or milestone.

Looks good!

I'll close it then


Use glog for all sent TMB messages.

@hbdeshmukh Could you please take a look? Thanks!

Hi @zuyu
The changes look fine. I have two comments though:

The amount of logging due to these changes will be a lot. If you need the logged information for debugging purposes, it may be worth to use something like DLOG or VLOG(3).
If you need to do some analysis on the logs (e.g. counting number of messages sent from a particular client of a specified type), it may be worth to write a utility function which takes the following arguments:

Sender TMB client ID
Sender name string (e.g. ForemanSingle, ForemanDistributed, optional field)
Reciver TMB client ID (some default value if this is a broadcast)
Receiver name (optional)
Message type
Message type string

The benefit of such a function is that you will get identical formatting for all the log messages. Therefore performing a grep over the logs will be much easier. Additionally, extending the function to log more information (e.g. message size, send/receive timestamps) will be straightforward.

@hbdeshmukh Thank you for your inputs. I'll use DLOG for this purpose.
As for the log analysis and the utility function, we could leave for future. For now, I just glog outputs for the debug purpose.
Cheers!


[longtalk]: spiralize: an R Package for Visualizing Data on Spirals

Abstract
The spiral layout has two major advantages for data visualization. First, it is able to visualize data with long axes, which greatly improves the resolution of visualization. Second, it is efficient for time series data to reveal periodic patterns. Here we present the R package spiralize that provides a general solution for visualizing data on spirals. spiralize implements numerous graphics functions so that self-defined high-level graphics can be easily implemented by users. The spiralize package is available at https://CRAN.R-project.org/package=spiralize.
Convenient day for your Long talk.
No preference.
Contact Details
<EMAIL_ADDRESS>Comment field
No response
Checklist

[X] This Long talk has a Bioconductor focus or interest to Bioconductor users/developers.
[X] I checked and confirmed the BioC Asia 2021 Code of Conduct.

@jokergoo Thank you for submitting the abstract.
The Assignees will start the review.
Once accepted by the Assignees, this issue will be closed and your abstract will be posted on https://biocasia2021.bioconductor.org/ .

I checked @jokergoo 's abstract and the package and repository in this abstract.
I accept this abstract.
After adding your abstract to https://biocasia2021.bioconductor.org/ ,
I will send a comment addressed to you again.

I checked @jokergoo 's abstract and the package and repository in this abstract.
I accept this abstract.
After adding your abstract to https://biocasia2021.bioconductor.org/ ,
I will send a comment addressed to you again.

Thank you!


UI: Test Suite Bug

UI:

Payload's entity link is being sent encoded while creating a test suite for table named "test.json" ( including the quotes ) ( Datalake connector )

Table name changes to undefined when we go to profiler & data quality tab, click on DQ, Add Tests.

It also fails to fetch profile data

https://user-images.githubusercontent.com/40225091/206850920-b08ee7dc-a4a2-4d34-8abf-090aa1c087c0.mov

Name in test suite's page under test cases shows half name for tables under datalake connector with quotes

@ShaileshParmar11 can we handle this for 0.13.1? cc @chirag-madlani

@ShaileshParmar11 can we handle this for 0.13.1? cc @chirag-madlani

sure @Ashish8689 Please have a look here

@ShaileshParmar11 I have resolved the payload part (1st one)


Add support for disabled buttons

Add support for disabled menu buttons.

This comment also applies to #8
Hello, this thanks for the suggenstion. I'm currently a bit busy so I'm not sure when I'll find the energy to work on this but here are some thoughts on these topics:

Platform support

Windows: Supporting disabled buttons seems pretty straight-forward by just condionally adding the MF_GRAYED flag to the button. Icons appear to be harder to support. Without resorting to custon drawn buttons, there is the an option to use a bitmap instead of an string and a function to replace the checked/unchecked icon. In both cases the icon would replace an existing function.
Linux: Supporting both seems to be easy as there are enabled and icon-name properties defined in the com.canonical.dbusmenu spec.
MacOs: there seems to be a way to change the enabled state and image, but I'm unsure how setting an image interacts with the toggle state. Missing icon support implementation on MacOs is obviously also an issue here and would have to be addressed first.

API-wise I would just add more properties to the Button member to keep the internal state unified (i.e. only one button type) but think about ways to make constructing this button more flexible. Maybe a builder struct would be a good fit here.

I am willing to have a crack at them at some point if I also get some free time!

You mentioned having a VM setup for Mac in the readme, how exactly did you go about this? I can't seem to figure out how to get that working and I'd like to test my icon code on Mac but I do not own a machine. As for a status update though, I have now implemented disabled buttons across all operating systems, and I have implemented icons on buttons for all but MacOS.
As for the API, a builder would definitely be better, however I think that is best suited for another PR. I am happy to put some work towards that as well if you would like, I am just trying to get all of this working first.

Great to hear. I used this installer script.
As for the builder thing, I agree that it would be suited for another PR. My napkin design would be something like this:
pub enum MenuItem<T> {
    Separator,
    Button(Button<T>),
    Menu { name: String, children: Vec<MenuItem<T>> }
}

pub struct Button<T> {
    name: String,
    signal: T,
    checked: Option<bool>
}

impl<T> Button<T> {
   pub fn new(name: String, signal: T) -> Self { ... }
   pub fn checked(self, checked: bool) -> Self { ... }
}


Include runbiosimulations in bioservices

https://bioservices.readthedocs.io/en/master/

We should focus our advertising on the sites in #337. These are easy to submit to.
BioServices is maintained by the same people as CNORdt. runBioSimulations could contributed via a pull request.
https://github.com/cokelaer/bioservices/


Updated Book Section

Hi,
I added a forthcoming German book, disambiguated the cookbook entries and cleaned up some URLs in the book section.

Thank you for your contribution and cleaning up :smile:
Very much appreciated!


Add Ragdata's profile to the Code directory

Type of change

[x] Added, Removed or Updated GitHub Profile README
[ ] Bug fix
[ ] New feature
[ ] Improvement

Describe change does this PR introduce?
Add Ragdata's profile to the Code directory
(for real this time)
Checklist

[x] The commit message follows our guidelines.
[x] The code has been self reviewed and tested.
[x] Added or Updated Github Profile README.

[x] The Github Profile README is impressive and visually appealing.
[x] Added it below the heading of the category in alphabetical order.

Additional context
GitHub Profile URL: https://github.com/Ragdata

AHA - I had a feeling I was mistaken about the meaning of "Code" ... I'll move it and resubmit.


resultSet.getRows() is empty

Dear all,
I'm playing with the code in the master and I found a strange behaviour with the resultSet.getRows() method that can be easily reproduced with this code (replace the table "users" with whatever table you want):
jdbcService.getConnection(handler -> {
			handler.result().query("select count(*) as count from users", result -> {
					logger.info("Found {} rows", result.result().getNumRows()); //prints 1
					logger.info("Results size {}", result.result().getResults().size()); //prints 1
					logger.info("Rows size {}", result.result().getRows().size()); //prints 0
			});
		});

There is no data in the getRows(); but, based on the documentation of vertx-sql-commons, it is supposed to contain a list of JsonObject populated with the results. I checked the available unit tests and it seems that that method is not tested.

Hi Francesco,
In SQL,
SELECT COUNT(*) FROM ...
simply counts the number of rows - it does not return the rows
themselves, so this seems like expected behaviour.
If you want to return the actual rows, you can use:
SELECT * FROM ....
On 11/03/15 17:11, Francesco wrote:

Dear all,
I'm playing with the code in the master and I found a strange behaviour with the resultSet.getRows() method that can be easily reproduced with this code (replace the table "users" with whatever table you want):
jdbcService.getConnection(handler -> {
			handler.result().query("select count(*) as count from users", result -> {
					logger.info("Found {} rows", result.result().getNumRows()); //prints 1
					logger.info("Results size {}", result.result().getResults().size()); //prints 1
					logger.info("Rows size {}", result.result().getRows().size()); //prints 0
			});
		});

There is no data in the getRows(); but, based on the documentation of vertx-sql-commons, it is supposed to contain a list of JsonObject populated with the results. I checked the available unit tests and it seems that that method is not tested.

Reply to this email directly or view it on GitHub:
https://github.com/vert-x3/vertx-jdbc-service/issues/9

Hi tim,
sorry but I don't agree with your analysis.
In SQL a "select count(*) blablabla" returns exactly one row whose content is the desired count. So it does contain rows.
As a confirmation that this is a bug, if you execute "select * from users" instead of the count, you will see that the result.result().getRows() is always empty even if result.result().getResults() is not.

Hi Tim,
indeed it was a bug in the vertx-sql-common project. I fixed it and proposed a PR (https://github.com/vert-x3/vertx-sql-common/pull/3).
I tested it locally and now it works with both "select * blablabla" and "select count(*) blablabla" :+1:

Got it. Sorry - I misunderstood the original description and thought you were expecting all the rows to be returned
Thanks for the fix, will take a look soon :)


[fix] Wait until initial configmap reconcile

Changes

Stop the world until the initial configmap reconcile is done
Improve expose controller to avoid undesired errors

Submitter Checklist

[ ] Docs included if needed
[x] Tests included if needed
[x] Follows the good commit messages standard

/approve


Hot reload is losing while setting nuxtignore

Environment

Operating System: Darwin
Node Version:     v18.17.1
Nuxt Version:     3.8.0
CLI Version:      3.9.1
Nitro Version:    2.7.2
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      devtools, runtimeConfig, app, css, vite, vue, imports, modules
Runtime Modules<EMAIL_ADDRESS>Build Modules:    -

Reproduction
https://codesandbox.io/p/sandbox/determined-murdock-kzxprh?file=%2Fpages%2Fhome%2FBanner.vue%3A6%2C1
Describe the bug
I have some folder like "home, components" in the pages folder, but I would like to exclude them from routes.
pages/
  home/
    banner.vue
    index.ts
  product/
    components/
      List.vue
      Card.vue
    index.vue

So I set them inside .nuxtignore but they lost the hot reload.
It's working great in nuxt2, don't know why.
Additional context
No response
Logs
No response

You could use the pages:extend hook to tell Nuxt that these components are not pages instead of letting nuxt fully ignore them*.
Example via @hecktarzuli's blog post to "ignore" all pages/**/components/*.vue files as pages.
hooks: {
  'pages:extend'(pages) {
    const pagesToRemove: NuxtPage[] = []
    pages.forEach((page) => {
      if (page.path.includes('component')) pagesToRemove.push(page)
    })

    pagesToRemove.forEach((page: NuxtPage) => {
      pages.splice(pages.indexOf(page), 1)
    })
  }
}


vertor_matrix dot without blas

Hi.
It seems that the vector.dot(matrix) is not supported yet both in numeric library and blas library. Therefore I just add some naive code that make the dot product directly..
Example:
Given a 1D tensor [1.0,  2.0] and a 2D tensor [1.0, 2.0, 3.0; 4.0, 5.0, 6.0], the dot result is 1D tensor [9.0, 12.0, 15.0]
[Dev]
src/tensor/dot.rs
[Tests]
tests/dot.rs

LGTM! Thanks for the contribution! You have chosen for the vector to be interpreted as a row vector here. I think this is definitely the right call and it is the behavior of numpy and matlab. I'll try to add a BLAS version too.
Do you mind adding your name to the CONTRIBUTORS.md file before I merge?

Thanks again!

Just added BLAS support for this by the way (507dd57cc081f3c99fa6b90c8f2a4481638d4bc6).

Oh! Thanks! I should be read the code of BLAS library more carefully...


Add Testing of Conda Artifacts

azure-devops and conda ...0/10 would not recommend.
Note that while I'll take feedback on the step Evaluate OS Specific PATH and Parameters in test-conda.yml, if it looks hacky, there is probably a reason for it.
@benbp I have extended archetype-sdk-tests-generate.yml in this PR because I wanted a nice way to filter out running tests on certain python versions (we only need to support >=3.6 for conda). Rather than doing wonky conditions on every step, I merely extended our usage of archetype-sdk-tests-generate so that we invoke it twice. Once for when we do have conda artifacts to test, once for our regular tests..
Reason being, even if we used parameter sets functionality in Create-Job-Matrix.ps1, we still run into the issue where we still have to have a condition attached to every step within test-conda (wherever the steps live). This is due to the fact that the variables are essentially only available at runtime, which will means we can't use any compile-time ${{ if to keep our logs clean.

Closing to figure out these core/storage errors. Can't keep spamming the build pool.

Now azure-storage has issues. I don't want to spam the build system and eat more minutes than I already have. Closing again temporarily.


feat: add breeze-gtk for aurora

https://github.com/ublue-os/bluefin/issues/1596
Adds breeze-gtk package for more native theming of gtk apps on KDE Plasma.

Why are we adding themes we're trying to get rid of them?

Why are we adding themes we're trying to get rid of them?

I wasn't aware themes were being removed. This theme is mostly to improve the KDE experience with gtk apps by making them look more in line with qt apps.

Do we know why it's not shipped in Kinoite by default? `

(oops, misclicked, sorry), do we know why Kinoite doesn't ship this ootb?

@castrojo I created a vm of kinoite in gnome boxes and it shows that breeze-gtk is installed by default.

Let me check Aurora, but if this is in kinoite it should be in our image already.


[google_maps_android] Update robolectric tests to run against minSdk version instead of sdk and lower that sdk version to the oldest supported.

Robolectric has the ability to run tests against multiple variations of android. It appears that for most of our testing we use @Config(sdk = Build.VERSION_CODES.X) which runs the tests against one android version. Roboletric also supports minSdk which will use the value as a min and run for all following version codes.
In addition it appear that maps set the SDK to android "P" when the underlying maps sdk supports back to 21.
Convert all robolectric tests in google_maps_android to use minSdk and set the minSdk to the lowest value possible. Document the work arounds or things you learned so that when we do this change to other platforms it goes smoother.
Related comment thread: https://github.com/flutter/packages/pull/7313#issuecomment-2270038753
Documentation also says it supports android api 20 and above which is incorrect given the min version of android flutter supports is api 21. https://pub.dev/packages/google_maps_flutter

Update on progress: Lowering minSdk in ConvertTest.java to LOLLIPOP (21) results in each test of that file throwing OOM. Each test on each individual SDK (i.e. ConvertTest.testName[X]) succeeds, but the overall test (ConvertTest.testName) throws the error.
Increasing Xmx in https://github.com/flutter/packages/blob/main/packages/google_maps_flutter/google_maps_flutter_android/example/android/gradle.properties up to even my system's max of 64G does not resolve this, neither does adding an Xms.
Adding a trivial test in the file that merely asserts a number is equal to itself and running only that individual test still OOMs in the same way. Commenting out the Before and After methods still OOMs. Removing all mocked fields at the class level and only running that trivial test that does not explicitly utilize any mocks or resources still OOMs.
My only thought is that some dependency may be leaking memory.


doc: security: Add information about backports

Clarify about how to proceed with backports for vulnerabilities.

@nashif can you please revisit it.

@kartben Thanks for reviewing it ! Fixed


feat: fastly compute integration

Addresses #389.
After much trial and error, here is a code example for using the Momento HTTP API inside of a Fastly Compute@Edge instance. The example sets, gets, and deletes an item from an existing cache. The fastly.toml configs handle most of the Fastly resources setup.
The deployed instance is running at https://solely-civil-crab.edgecompute.app.

This is great @anitarua , thank you! Before we close this ticket out, I think we should add something to the dev docs. Check out what Pete and Pratik have put there for cloudflare/netlify etc. and lmk if you are up for filing a PR over there!

Huge thanks to @krispraws for helping to get this integration up and running as well!!
And yes, for sure, I'd be happy to follow up with a dev docs PR!

 Let's just add a comment to the code and/or docs saying that tokens should be stored in secret stores but we are using config store because it is a beta feature and not available to all accounts


CRLP: Fix for skew mode null single result rollups

Also includes an update to the Customizable Rollups testing script
Critical Changes
Changes
Issues Closed

Fixes #4358

New Metadata
Deleted Metadata

**lurch: attach W-034147

Tracking W-034147


Make explicit that broker authors may not return some or all params

Per discussion around the last PR, just make this explicit

LGTM

Thanks @jberkhahn !

LGTM

LGTM

LGTM


doc: improve build guide doc

Signed-off-by: qinggniq<EMAIL_ADDRESS>Due to recent update of python package , now only python version >3.8.0  can use ./tools/vscode/refresh_compdb.sh, we should let developers know.
related discussion
https://envoyproxy.slack.com/archives/C78HA81DH/p1617087308144200
Commit Message: improve build guide doc
Additional Description:
Risk Level: Low
Testing: No
Docs Changes: Yes
Release Notes: No
Platform Specific Features: No
[Optional Runtime guard:]
[Optional Fixes #Issue] #15754
[Optional Deprecated:]
[Optional API Considerations:]

@phlax

@qinggniq you will need to fix DCO:
https://github.com/envoyproxy/envoy/pull/15764/checks?check_run_id=2227156496


Show Install Git step when it is not installed.

Changes are tested by running test script, and manually testing on a machine with no git installed.
This PR closes #125349

A preview of the step added:

Issue closed.


Add helper neurom.check.ok

Returns sensible values of True or False for morphology checks:
True / False ==> True / False
Iterable ==> True if empty else False

Current coverage is 100.00%

Merging #189 into master will not affect coverage as of 5dc2382

@@            master    #189   diff @@
======================================
  Files           33      33
  Stmts         1341    1346     +5
  Branches         0       0
  Methods
======================================
+ Hit           1341    1346     +5
  Partial          0       0
  Missed           0       0

Review entire Coverage Diff as of 5dc2382

Powered by Codecov. Updated on successful CI builds.

@jdcourcol This should simplify the checks.

agree. thank you.

@eleftherioszisis @lidakanari FYI some more simplifications to morphology checking.

I like the structure with the tests for this :+1:


Add Nakama to the game development resources list.

Why do you think the link is worth adding on this list?

We built Nakama to help game developers and studios build server-based games on desktop, mobile, and web. With Nakama server you can add user authentication, social networking, cloud storage, realtime data exchange, and lots more into games. It is developed to handle the difficult but essential services that go into all social and realtime games.

Does this project has any License?

The project is licensed under the Apache-2 License. You can find it referenced on the GitHub project.

Thanks @ellisonleao!


Pull request

Fix bug about block rotation shape and overlapped last block.
Add the function which previews next three blocks.
Add score board.
Add some sound effects and game over screen.
Make blocks' frequency follow the tetris standard.
Add pause function of tetris (when pressing 'p').

You can see other solved issues here:
https://github.com/18-1-SKKU-OSS/canvas-tetris/issues?q=is%3Aissue+is%3Aclosed

Block ghost effect was also added.


How to use multiple aggregators at the same time

Hello,
I need to run Count aggregator on Pivottable for the selected dimensions, and put Standard Deviation result on colTotals that will be calculated from results of Count on every column

It seems like you want to build a custom aggregator that does something different for body cells and total cells?

Yes! That's exactly what I want


1167 fixes protobuf

Fixes #1167
Description

pip install ray might upgrade protobuf automatically, but protobuf v4 is not compatible with some versions of pytorch's tensorboard summarywriter

Checks

[ ] Avoid including large-size files in the PR.
[ ] Clean up long text outputs from code cells in the notebook.
[ ] For security purposes, please check the contents and remove any sensitive info such as user names and private key.
[ ] Ensure (1) hyperlinks and markdown anchors are working (2) use relative paths for tutorial repo files (3) put figure and graphs in the ./figure folder
[ ] Notebook runs automatically ./runner.sh -t <path to .ipynb file>

this works fine with all the integration tests, I'm merging it for now


Changed apt-mirror provider to dynamically generate temp config file.

Previously the provider.conf file had a hardcoded value for the base
folder to download to.  This would ignore any value that was in the
bashellite.conf file or the value passed with the -m option.
This behavior has been corrected to write to a temp file in the /tmp
folder where the "set base_path" is dynically taken from the mirror_tld
and mirror_repo_name passed in variables.

This PR isn't ready yet.


Nuget overhaul

Just trying to get specs at #8179 green. Opening a WIP PR to check CI.

This is absolutely fantastic!  I reviewed the changes you've made and so far it all looks good.  I added a comment about one instance I found when testing locally that might make things easier (Microsoft.Extensions.DependencyModel vs Autofac), but that's all I think I need to mention.
Also, I was pointed to your comments on Slack, but I'm away from my computer that can access Slack until next week, so you can contact me via GitHub comments or email (brettfo@microsoft.com).

Awesome @brettfo, let's discuss here publicly.
Is it ok to force-push this changes to #8179? I can keep a local copy of the current state just in case, but if the work in this PR seems solid I'd like to incorporate it there and close this PR.

Yes, by all means you can force-push to the other PR.  Do I need to grant you contributor status on my fork?

I'm not sure, but I think unless you explicitly opted out to disallow maintainers from editing it when creating the PR, I should be able to do it. I'll try and let you know if I find issues.

Yeah, I could force push without issues. Closing in favor of #8179.


Add credits

Codecov Report

Merging #246 (c361fdc) into master (0e49f90) will not change coverage.
The diff coverage is n/a.

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


Not compatible with django 2.1

remote:        Traceback (most recent call last):
remote:          File "manage.py", line 23, in
remote:            execute_from_command_line(sys.argv)
remote:          File "/app/.heroku/python/lib/python3.6/site-packages/django/core/management/init.py", line 381, in execute_from_command_line
remote:            utility.execute()
remote:          File "/app/.heroku/python/lib/python3.6/site-packages/django/core/management/init.py", line 357, in execute
remote:            django.setup()
remote:          File "/app/.heroku/python/lib/python3.6/site-packages/django/init.py", line 24, in setup
remote:            apps.populate(settings.INSTALLED_APPS)
remote:          File "/app/.heroku/python/lib/python3.6/site-packages/django/apps/registry.py", line 112, in populate
remote:            app_config.import_models()
remote:          File "/app/.heroku/python/lib/python3.6/site-packages/django/apps/config.py", line 198, in import_models
remote:            self.models_module = import_module(models_module_name)
remote:          File "/app/.heroku/python/lib/python3.6/importlib/init.py", line 126, in import_module
remote:            return _bootstrap._gcd_import(name[level:], package, level)
remote:          File "", line 994, in _gcd_import
remote:          File "", line 971, in _find_and_load
remote:          File "", line 955, in _find_and_load_unlocked
remote:          File "", line 665, in _load_unlocked
remote:          File "", line 678, in exec_module
remote:          File "", line 219, in _call_with_frames_removed
remote:          File "/app/.heroku/python/lib/python3.6/site-packages/actstream/models.py", line 57, in
remote:            class Action(models.Model):
remote:          File "/app/.heroku/python/lib/python3.6/site-packages/actstream/models.py", line 171, in Action
remote:            @models.permalink
remote:        AttributeError: module 'django.db.models' has no attribute 'permalink'

please install from master branch which is compatible with django 2.1

i am installing the latest version 0.6.5 using pip and same is not working.
Could you please let me know the latest version to use ?

It works fine on django 2.0 NOT on 2.1

i am telling you to install from github master. not the 0.6.5 from pypi. lots of changes occured on github master since the release of 6.5

Thanks for the update.
I will be installing from github master.
Please try to provide through pypi also.
Thanks again

I pinged Justin but he didn't replied. The library is compatible with django 2.1 from long ago in the master


Ignore stylesheets without (data-)href attribute.

In the case when stylesheet has not (data-)href attribute (for example when using Angular directive ng-href. For this case href attribute will not available while controller variables will not available) prefixfree should not post-process stylesheet because it will throw an error. For this case stylesheet post-processing logic should process DynamicDom plugin.

Thanks!


Add support for ap tracking

Add support for ap tracking
Description
Some references are referenced respect to ap, which can change while the program is executing. AP tracking data from the compiled program json is used for adjusting the value of the reference. This PR adds support for these cases.
Checklist

[ ] Linked to Github Issue
[ ] Unit tests added
[ ] Integration tests added.
[ ] This change requires new documentation.

[ ] Documentation has been added/updated.

Codecov Report

Merging #295 (2190ab4) into main (5338090) will increase coverage by 0.00%.
The diff coverage is 96.50%.

@@           Coverage Diff            @@
##             main     #295    +/-   ##
========================================
  Coverage   96.29%   96.29%
========================================
  Files          24       24
  Lines        7905     8152   +247
========================================
+ Hits         7612     7850   +238
- Misses        293      302     +9

Impacted Files
Coverage Δ

src/serde/deserialize_program.rs
91.42% <33.33%> (-1.02%)
:arrow_down:

src/vm/runners/cairo_runner.rs
98.48% <80.00%> (-0.07%)
:arrow_down:

src/vm/hints/hint_utils.rs
82.23% <90.35%> (+0.53%)
:arrow_up:

src/vm/hints/execute_hint.rs
100.00% <100.00%> (ø)

src/vm/vm_core.rs
97.34% <100.00%> (+<0.01%)
:arrow_up:

src/cairo_run.rs
85.45% <0.00%> (-0.53%)
:arrow_down:

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 5338090...2190ab4. Read the comment docs.


Add fuzz tests for injection operators

These do a lot of string crunching and are good targets for fuzz. Notably, it found a couple of crashes in XSS detection https://github.com/corazawaf/libinjection-go/pull/15

Codecov Report
Base: 78.87% // Head: 78.87% // No change to project coverage :thumbsup:

Coverage data is based on head (590e7bc) compared to base (ece5ecc).
Patch coverage: 100.00% of modified lines in pull request are covered.

:exclamation: Current head 590e7bc differs from pull request most recent head 6716fa8. Consider uploading reports for the commit 6716fa8 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##           v3/dev     #576   +/-   ##
=======================================
  Coverage   78.87%   78.87%
=======================================
  Files         145      145
  Lines        7014     7014
=======================================
  Hits         5532     5532
  Misses       1250     1250
  Partials      232      232

Flag
Coverage Δ

default
74.41% <100.00%> (+0.12%)
:arrow_up:

examples
27.17% <0.00%> (ø)

ftw
56.49% <100.00%> (ø)

tinygo
72.46% <100.00%> (+0.13%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

operators/detect_sqli.go
100.00% <100.00%> (ø)

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Add support for OAuth client credentials

What this PR does / why we need it:
Allows configuring the tailscale client with OAuth Client ID and OAuth Client. On instantiation the Client will generate a new API key against the Tailscale OAuth token endpoint.
Which issue this PR fixes:
Fixes #42

@knyar @willnorris I believe this is ready for another review.

@knyar @willnorris, mind giving this another look?

For those following along I rebased and squashed my commits here to sign and signoff on the commits to pass the commit signature verification requirement.


Add total size of batch iterations to tqdm loop

This allows it to display the estimated time to completion.

Thanks!


[MXNET-833] [R] Char-level RNN tutorial fix

Description
Char-level RNN model for language modeling.
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[x] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[x] Changes are complete (i.e. I finished coding on this PR)
[x] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[x] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[x] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

@mxnet-label-bot [pr-awaiting-review]

@ankkhedia - Your comments are addressed. Can you please take a look? Thanks.

@sandeep-krishnamurthy This PR is ready for merge


Fix missing czech language from "Language selection"

Hi, EtienneLamoureux,
thank you for the newest installment of "TQ Vault AE", but in the version 3.7.0, I can't pick "Czech" (language) from the dropdown selection menu, when I configure the program...
The czech language was dropped out of the program or what happened? As a native Czech, I am quite used to the localization of your program in Czech.
Thanks in advance for fixing ...
Sir-Mac

Thanks for the report. The new release (3.7.1), is fixed.


Fixing broken links

v1.22 Has a number of misconfigured links. This fixes it.
Signed-off-by: Mary Anthony<EMAIL_ADDRESS>

Oh, thanks!
LGTM

Hm, @moxiegirl I added it to the milestone for cherry picking (if we want the docs up to date in the release branch), but just realized we don't want the v1.22 in there yet, so feel free to remove if you want to do that some other way


GPSLogger + Google travel time not working since 0.94.0

Home Assistant release with the issue:
0.94.0
Last working Home Assistant release (if known):
0.93.X
Operating environment (Hass.io/Docker/Windows/etc.):
Hass.io
Component/platform:
Google travel time and GPSLogger
Description of problem:
Since 0.94.0 Google travel time component does not work with GPSLogger device trackers.
It cannot find the device trackers even tho they are present. My guess is that google travel time component loads before GPSLogger entities from integrations are created.
This was working fine while GPSLogger was still using known_devices.yaml.
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
sensor:
  - platform: google_travel_time
    name: Jakub čas domů
    api_key: !secret google_api_key
    origin: device_tracker.jakub_note8
    destination: zone.home
    options:
      mode: transit
  - platform: google_travel_time
    name: Eliška čas domů
    api_key: !secret google_api_key
    origin: device_tracker.eliska_note4
    destination: zone.home
    options:
      mode: transit

Traceback (if applicable):
2019-06-09 13:47:50 ERROR (SyncWorker_19) [homeassistant.components.google_travel_time.sensor] Unable to find entity device_tracker.jakub_note8
2019-06-09 13:47:50 ERROR (SyncWorker_15) [homeassistant.components.google_travel_time.sensor] Unable to find entity device_tracker.eliska_note4

Additional information:
Entities are present after reboot but google travel time is not loading:

Same error here.

I have the same problem and I have no entry for -platform: gpslogger .  Here is a write up I did earlier in the ha forums:
https://community.home-assistant.io/t/google-travel-time-no-longer-works-with-0-94/120885

Yes apologies disregard my solution, I misunderstood the problem.
In fact I do have that same error where the GPS entities are not found by google travel time sensor
ERROR (SyncWorker_14) [homeassistant.components.google_travel_time.sensor] Unable to find entity device_tracker.yaragps

Duplicate of https://github.com/home-assistant/home-assistant/issues/24432