fix: add alert report timeout limits

SUMMARY

[x] Working Timeout and Grace Period fields should not be negative
[x] Adds min value to modal input fields to prevent user from setting negative values

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
TEST PLAN
Manual Test:

Open alert/report modal
Enter values for Working Timeout/Grace Period (where applicable)
Confirm that value cannot be set below 0

ADDITIONAL INFORMATION

[ ] Has associated issue:
[x] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

Codecov Report

Merging #12926 (3dbdf83) into master (b5b0c2c) will decrease coverage by 1.77%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #12926      +/-   ##
==========================================
- Coverage   68.98%   67.20%   -1.78%
==========================================
  Files        1025      489     -536
  Lines       48765    28661   -20104
  Branches     5188        0    -5188
==========================================
- Hits        33639    19263   -14376
+ Misses      14992     9398    -5594
+ Partials      134        0     -134

Flag
Coverage Δ

cypress
?

javascript
?

python
67.20% <ø> (-0.14%)
:arrow_down:

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

Impacted Files
Coverage Δ

superset/sql_validators/postgres.py
50.00% <0.00%> (-50.00%)
:arrow_down:

superset/db_engine_specs/presto.py
81.38% <0.00%> (-6.93%)
:arrow_down:

superset/db_engine_specs/postgres.py
93.42% <0.00%> (-2.64%)
:arrow_down:

superset/views/database/mixins.py
80.70% <0.00%> (-1.76%)
:arrow_down:

superset/db_engine_specs/base.py
85.50% <0.00%> (-0.50%)
:arrow_down:

superset/reports/notifications/base.py
95.00% <0.00%> (-0.46%)
:arrow_down:

superset/views/base_api.py
97.79% <0.00%> (-0.45%)
:arrow_down:

superset/migrations/shared/security_converge.py
86.82% <0.00%> (-0.11%)
:arrow_down:

superset/sql_parse.py
99.36% <0.00%> (-0.01%)
:arrow_down:

superset/errors.py
100.00% <0.00%> (ø)

... and 541 more

Continue to review full report at Codecov.

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

Let's also add a test for this change.


Use Kiali v0.6.0 for Istio.

As we are going to release 1.0.1, it is better to update Kiali docker image version to 0.6.0.
Even though Kiali istio-release-1.0 is a retag of v0.6.0, but it is better to use 0.6.0 for Istio 1.0.1 release so that we can trace the version more accurate.
/cc @sdake @mwringe

@gyliu513: GitHub didn't allow me to request PR reviews from the following users: mwringe.
Note that only istio members and repo collaborators can review this PR, and authors cannot review their own PRs.

In response to this:

As we are going to release 1.0.1, it is better to update Kiali docker image version to 0.6.0.
Even though Kiali istio-release-1.0 is a retag of v0.6.0, but it is better to use 0.6.0 for Istio 1.0.1 release so that we can trace the version more accurate.
/cc @sdake @mwringe

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: gyliu513
If they are not already assigned, you can assign the PR to them by writing /assign @gyliu513 in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

install/OWNERS

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

@gyliu513: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
109c4a6767f6ff5eabe3ccaf1533a24171eeabfb
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@gyliu513: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
4989857132bd5ae3ebc84f9001d5d0fd7ea84554
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@sdake Kiali istio-release-1.0 is tagged from v0.6.0 based on comments here https://github.com/kiali/kiali/issues/375#issuecomment-414225465

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: sdake
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

install/OWNERS [sdake]

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

@gyliu513 can you rebase to fix circleci failures?
Cheers
-steve

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: sdake
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

install/OWNERS [sdake]

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

New changes are detected. LGTM label has been removed.

@sdake rebase again ;-)

/retest

@gyliu513: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
54b5c65b71d37320c21f67e5a9ed5993fb337226
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@gyliu513: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
54b5c65b71d37320c21f67e5a9ed5993fb337226
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@gyliu513 seems like the gate has been defective for some time :( .


chore: Fixed release workflow

Pull Request Checklist

[X] Make sure to read the contributor guide here.
[X] Make sure you are making a pull request against the staging branch.
[X] Make sure your pull request title matches the requested format.
[X] Make sure to lint, type-check, and run unit and API tests.

Description
Attempting to fix the release workflow again

:tada: This PR is included in version 2.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Fix encoding for open()

Codecov Report

Merging #3 (be7d6e0) into master (6ae89e9) will decrease coverage by 0.33%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##            master       #3      +/-   ##
===========================================
- Coverage   100.00%   99.66%   -0.34%
===========================================
  Files            6        6
  Lines          300      300
===========================================
- Hits           300      299       -1
- Misses           0        1       +1

Impacted Files
Coverage Δ

src/watchpoints/util.py
100.00% <100.00%> (ø)

src/watchpoints/watch_print.py
100.00% <100.00%> (ø)

src/watchpoints/ast_monkey.py
94.73% <0.00%> (-5.27%)
: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 6ae89e9...be7d6e0. Read the comment docs.


SPZeroDownTimePatching: New resource for enabling ZDP with DSC in SP 2016 and 2019

Following up on #1096 about ZDP there would be the need to create a new resource to handle Side-by-Side in this scenario. The new resource could look like this:
Enabling SideBySide could look like this and would set the SideBySide token to the current patch level.
SPZeroDownTimePatching EnableSideBySide {
    SideBySide = “Enable”
}

After the patching is done...
Setting SideBySideToken to the new Farm Version could look like this:
SPZeroDownTimePatching EnableSideBySide {
    SideBySide = “Upgrade”
}

All information needed to set the web application property WebSerivce.SideBySideToken can be obtained within this resource, by querying the farm or using a part of the new methods introduced with the refactoring of SPProductUpdate
Maybe there should be an option available to disable SideBySide - but this is not necessarily a key feature we should implement.
Edit: The resource needs to have a key that can be only two values. This would make it possible to use this resource in a given configuration twice.

See my comment in the other issue. I don't think this resource is needed, since the two examples will be in constant conflict and the server will never be in a compliant state.

I think, there is a chance to  resolve this potential conflict.
Both examples need to return true (test-method) for the following case:
Local Version obtained from the SPProductUpdate Method equals the Farm Version and SideBySide is enabled and the SideBySideToken equals the Farm Version
False for

first example:
— SideBySide is not enabled
— Version mismatch

second example:
— SideBySide is not enabled
— Version mismatch

Basically both would return the same status - the set method would differ though. I’m “Enable”  the set method would pass the current Version to SharePoint. In “Upgrade” it would set the installed Version.

wouldn't this work?
SPZeroDownTimePatching EnableSideBySide {
    MinimumVersion = "16.0.4888.1000"
    Enabled = "Present"
}

... handle cross node dependencies / order
SPProductUpdate LatestPatch {}
... wait for all product updates to complete / cross node dependencies / ordering.
SPConfigWizard ConfigWizard {}
... handle any reboot requirements / ensure another WFE is available to serve requests.

SPZeroDownTimePatching EnableSideBySide {
    MinimumVersion = "16.0.4900.1000"
    Enabled = "Present"
}

Enabled would control your sideBySide being enabled or disabled.
MinimumVersion would handle version matching by the following rule

The idea would be if the Test-Resource would return true if local version on a specific node was equal to or greater than the version specified. Could also just include a "Version" tag to force set to a specific version (perhaps as part of an install scriptj).

The suggestion for the SPZeroDownTimePatching resource would work indeed. The MinimumVersion needs to be Key parameter in this case. And out of interest, what would the Enabled parameter do in this case?

Using the parameter MinimumVersion to handle the potential conflict of my suggestions is a very good idea.
@ykuijs For my understanding the parameter Enabled will control the SideBySide Setting of the web application: https://docs.microsoft.com/en-us/previous-versions/office/sharepoint-server/dn778174(v=office.15)?redirectedfrom=MSDN


How to use Pixelmon Reforged currency for Total Economy?

Sponge Version:  1.12.2-2705-7.1.0
Forge Version:  1.12.2-<IP_ADDRESS>05
Total Economy Version:  1.8.1
Description of issue
How can I set the currency to use to PokéDollars from the Pixelmon Reforged mod? I know this isn't really an issue or contribution but maybe some one else is having the same problem.
Right now I have 2 currencies, the one from Pixelmon Reforged called 'PokéDollars' and the default one from TE. How can I make it so TE uses those PokéDollars instead of the default currency? Beware, complete coding noob here.
Thanks in advance.
Console errors
Steps to reproduce
Related Configuration Files
Create a Gist (https://gist.github.com/) for each configuration file related to this issue

Are you using the Pixelmon economy bridge? https://pixelmonmod.com/wiki/index.php?title=Economy_Bridge

I'm going to assume this was fixed by installing the economy bridge. If not, please let me know and I'll reopen. Thanks.


Warning C4996: TBaseMulticastDelegate

Issue
This warning is produced when I build 4.24-SpatialOSUnrealGDK-release using Visual Studio 2019 16.8.4:
1>C:\Unreal\UnrealEngine\Engine\Source\Runtime\Core\Public\Delegates/DelegateSignatureImpl.inl(788): warning C4996: 'TBaseMulticastDelegate<void>::AddRaw::FDeprecationMsg_788::condition': Binding a delegate with a const object pointer and non-const function is deprecated. Please update your code to the new API before upgrading to the next release, otherwise your project will no longer compile. 1> C:/Unreal/UnrealEngine/Engine/Plugins/Runtime/nDisplay/Source/DisplayCluster/Private/Render/DisplayClusterRenderManager.cpp(649): note: see reference to function template instantiation 'FDelegateHandle TBaseMulticastDelegate<void>::AddRaw<const FDisplayClusterRenderManager,>(UserClass ,void (__cdecl FDisplayClusterRenderManager:: )(void))' being compiled 1> with 1> [ 1> UserClass=const FDisplayClusterRenderManager 1> ] 1> C:/Unreal/UnrealEngine/Engine/Plugins/Runtime/nDisplay/Source/DisplayCluster/Private/Render/DisplayClusterRenderManager.cpp(649): note: see reference to function template instantiation 'FDelegateHandle TBaseMulticastDelegate<void>::AddRaw<const FDisplayClusterRenderManager,>(UserClass ,void (__cdecl FDisplayClusterRenderManager:: )(void))' being compiled 1> with 1> [ 1> UserClass=const FDisplayClusterRenderManager 1> ]
Workaround
This warning is harmless, you can ignore it, or follow these steps to turn it off.
Internal ticket
UNR-4705.

This warning is produced by Epic's code, not Improbable's.
It does not produce in later Unreal Engine versions.

For the above reasons, we will not fix this issue.


feat: improve consteval_int

Feature Request
Describe the Feature Request
At the moment consteval_int!(...); only allows simple calculus (+, -, *, /)
It would be handy if it would allow:
const a: felt252 = consteval_int!(12 ** 2); // pow
const b: felt252 = consteval_int!(a * 2); // Referencing other constants

If the feature request is approved, would you be willing to submit a PR?
(Help can be provided if you need assistance submitting a PR)

[x] Yes
[ ] No

+1 on supporting ** - it's very useful in the context of DeFi when dealing with tokens with 18 decimals (like eth). For example, we have to write<PHONE_NUMBER>000000000 to declare 1 ETH which is error prone and adding or removing a zero in that constant can have disastrous consequences. Being able to do consteval_int!(10 ** 18) is way better.

we are not planning on supporting ** as an operator, .pow may be supported. (relevant rust matching discussion https://github.com/rust-lang/rfcs/issues/3047)
Referencing of other constants is going to be much more difficult - and completely out of scope for this macro.
When supported - it would probably remove the need for the consteval macro completely (as it is a placeholder while it does not exist).
In general - better const support is on the way.

const referencing other consts is now supported on 2.6.0, no need for consteval.
a const power function may be available in the further future.


Feat/BottomNavigation - added BottomNavigation with Example

This Pull Request introduces the BottomNavigation component to Flowbite-React, enhancing the navigation experience for users. The BottomNavigation component offers a convenient way to navigate between different sections or pages of an application, displayed at the bottom of the screen for easy access. Leveraging this component in Flowbite-React provides users with intuitive and efficient navigation options, contributing to an improved user experience overall. With this addition, Flowbite-React becomes even more versatile, catering to a broader range of UI navigation needs.

Summary by CodeRabbit

New Features

Introduced a new Bottom Navigation component for creating navigational bars at the bottom of web pages.
Added examples and documentation for implementing Bottom Navigation with various styles including default and with borders.
Enhanced theming support for Bottom Navigation, allowing customization of items, icons, and labels.

Documentation

Added documentation for the Bottom Navigation component, including usage examples.

Tests

Implemented test cases for the Bottom Navigation component to ensure proper rendering and functionality.

Refactor

Expanded theme configuration to include Bottom Navigation styling options.

@SutuSebastian  --- this codecov/patch is failing again n again, any idea why? like should I've to be worried about it? please guide me

@SutuSebastian  --- this codecov/patch is failing again n again, any idea why? like should I've to be worried about it? please guide me

Always fails, ofc. Don't worry about it, it'll be gone soon.


Fixing ProgressRing pattern reporting

ProgressRing should not report the RangeValue pattern when it's indeterminate, as in that case it has no value to report.

/azp run


fix: Fix support for apispec>=1.0.0b5

Pin to apispec^1.0.0b5
Add pyyaml as a dep, since it is no longer included
with apispec
Update documentation

close #1

Codecov Report

:exclamation: No coverage uploaded for pull request base (dev@0e043a9). Click here to learn what that means.
The diff coverage is n/a.

@@         Coverage Diff          @@
##             dev     #4   +/-   ##
====================================
  Coverage       ?   100%
====================================
  Files          ?      2
  Lines          ?     20
  Branches       ?      0
====================================
  Hits           ?     20
  Misses         ?      0
  Partials       ?      0

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 0e043a9...917d8aa. Read the comment docs.

Just a heads up: you might be able to get rid of the make_plain_datastructure after 1.0.0 final is released because https://github.com/marshmallow-code/apispec/pull/333 removes LazyDict.
It won't hurt anything to keep that function for now, thuogh.

Thanks! I'll check it once 1.0.0 is out!

Prerelease is publish: https://pypi.org/project/starlette-apispec/#history


platformmanager: correctly handle fail cases

if a exception is thrown we need to rethrow it. If the platform does not
give any usable key we also need to throw up here

In theory this has nothing to do with the later publish unpublish stuff - just to know that I dont break any convention :-D

@phimonk handling better now ?

push @marcelhollerbach

pushed.


Set a small timeout on curl

Some minutes ago, Discord had connectivity issues, and some pages reached a timeout when saving them. Inspecting the log of slow php execution, found the problem was on DiscordNotifications:
script_filename = ($IP)//index.php
[0x00007fa3b58166a0] curl_exec() ($IP)/extensions/DiscordNotifications/DiscordNotificationsCore.php:455
[0x00007fa3b5816570] send_curl_request() ($IP)/extensions/DiscordNotifications/DiscordNotificationsCore.php:436
[0x00007fa3b5816110] push_discord_notify() ($IP)/extensions/DiscordNotifications/DiscordNotificationsCore.php:154
[0x00007fa3b5815b70] discord_article_saved() ($IP)/includes/Hooks.php:174
[0x00007fa3b58156c0] callHook() ($IP)/includes/Hooks.php:202
[0x00007fa3b5815560] run() ($IP)/includes/Storage/PageUpdater.php:1214

Ideally a small timeout on the curl_exec should be placed, 10 seconds or even less, to prevent saving pages to hang if Discord is just having issues

Maybe these values would be good:
curl_setopt($ch, CURLOPT_CONNECTTIMEOUT, 7); // Set 7 second timeout to connection
curl_setopt($ch, CURLOPT_TIMEOUT, 7); // Set global 10 second timeout to handle all data
Any new issues with Discord after the first outake? :)

Looks good to me, but note that for CURLOPT_TIMEOUT you left 7 instead of 10 :)

Added in latest commit.


Add support for synchronous RPC calls

RPC calls are asynchronous as _execute_monkey_patch doesn't effect them. This is annoying because all the other calls in supabase-py are synchronous.

Hey @aantn,
Thanks for the feedback! We'll make the necessary adjustments -- if anyone wishes to work on this issue it is up for grabs as part of Hacktoberfest.
-JL

My pleasure. I have this working (as well as other Supabase-py features) using the following hack in my app's code:
class MyAuthClient(SupabaseAuthClient):
    def _set_timeout(*args, **kwargs):
        """Set timer task"""
        # _set_timeout isn't implemented in gotrue client. it's required for the jwt refresh token timer task
        # https://github.com/supabase/gotrue-py/blob/49c092e3a4a6d7bb5e1c08067a4c42cc2f74b5cc/gotrue/client.py#L242
        # callback, timeout_ms
        threading.Timer(args[2] / 1000, args[1]).start()

class MyClient(Client):
    def _get_auth_headers(self) -> Dict[str, str]:
        auth = getattr(self, "auth", None)
        session = auth.current_session if auth else None
        if session and session["access_token"]:
            access_token = auth.session()["access_token"]
        else:
            access_token = self.supabase_key

        headers: Dict[str, str] = {
            "apiKey": self.supabase_key,
            "Authorization": f"Bearer {access_token}",
        }
        return headers

    @staticmethod
    def _init_supabase_auth_client(
        auth_url: str,
        supabase_key: str,
        detect_session_in_url: bool,
        auto_refresh_token: bool,
        persist_session: bool,
        local_storage: Dict[str, Any],
        headers: Dict[str, str],
    ) -> MyAuthClient:
        """Creates a wrapped instance of the GoTrue Client."""
        return MyAuthClient(
            url=auth_url,
            auto_refresh_token=auto_refresh_token,
            detect_session_in_url=detect_session_in_url,
            persist_session=persist_session,
            local_storage=local_storage,
            headers=headers,
        )

    def rpc(self, fn, params):
        """Similar to _execute_monkey_patch in supabase-py - we make the async rpc call sync"""
        path = f"rpc/{fn}"
        url: str = str(self.postgrest.session.base_url).rstrip("/")
        query: str = str(self.postgrest.session.params)
        logging.warning(f"doing call {url}/{path}?{query}, with headers={self.postgrest.session.headers} and whole session is {self.postgrest.session}")
        response = requests.post(f"{url}/{path}?{query}", headers=dict(self.postgrest.session.headers) | self._get_auth_headers(), json=params)
        return {
            "data": response.json(),
            "status_code": response.status_code,
        }

Feel free to cleanup and re-use


Policy Files Not Being Detected

Hi, I've created the following:
/root
../policies
../../check-storage-account-location
../../../assign.DEV.json
../../../policy.json

Yet the action reports:
Warning: Did not find any policies to create/update. No policy files match the given patterns or no changes were detected. If you have policy definitions or policy initiatives, please ensure that the files are named 'policy.json' and 'policyset.json' respectively.
... and no policy is created in my subscription. Can someone double check the dir/file setup above it correct ? Is there anything within the policy/assignment files themselves that might stop the action from working?
I'm using the action as follows:
      - name: Create or Update Azure Policies
        uses: azure/manage-azure-policy@v0
        with:
          paths: |
            policies/**  # path to directory where policy files were downloaded in runner

@paulmccrady Do the json files contain id, type, name and properties fields?
Also, are you using checkout action before invoking the manage-azure-policy action? In case you are using it, are you checking out to some other path?

name: Checkout
uses: actions/checkout@v2

@tauhid621 policy below
{
  "properties": {
    "displayName": "Check Storage Account Location",
    "policyType": "Custom",
    "mode": "All",
    "description": "Force resources to be deployed to UK South",
    "metadata": {
      "category": "Storage"
    }
  },
  "parameters": {
    "allowedLocation": {
      "type": "string",
      "metadata": {
        "description": "The allowed location for resources.",
        "displayName": "Allowed location",
        "strongType": "location"
      }
    }
  },
  "if": {
    "allOf": [
      {
        "field": "type",
        "equals": "Microsoft.Storage/storageAccounts"
      },
      {
        "not": {
          "field": "location",
          "equals": "[parameters('allowedLocation')]"
        }
      }
    ]
  },
  "then": {
    "effect": "Audit"
  },
  "id": "/subscriptions/__subscriptionId__/providers/Microsoft.Authorization/policyDefinitions/__policyName__",
  "type": "Microsoft.Authorization/policyDefinitions",
  "name": "Check Storage Account Location"
}

First steps in pipeline are
    steps:
      - uses: actions/checkout@v2

      - uses<EMAIL_ADDRESS>        with:
          creds: ${{ secrets.AZURE_CREDENTIALS }}
          enable-AzPSSession: true

      - uses: cschleiden/replace-tokens@v1
        with:
          tokenPrefix: '__'
          tokenSuffix: '__'
          files: '["**/*.json"]'
        env:
          subscriptionId: $env:subscription_id
          policyName: $env:policy_name

Thanks for the assistance

@paulmccrady I believe the issue is with the comment in path provided
" policies/**  # path to directory where policy files were downloaded in runner"
This will try to match the whole string
Can you try with just "policies/**"

Thankyou, removing the "# path to directory where policy files were downloaded in runner" resolved the issue.


utils: add s2n_result implementation

Please note that while we are transitioning from travis-ci to AWS CodeBuild, some tests are run on each platform. Non-AWS contributors will temporarily be unable to see CodeBuild results. We apologize for the inconvenience.
Issue # (if available):
Part 1 for #1782 and #1783
Description of changes:
The goal of s2n_result is to provide a strongly-typed error signal value, which provides the compiler with enough information to catch bugs.
Historically, s2n has used int to signal errors. This has caused a few issues:
GUARD in a function returning integer types
There is no compiler error if GUARD(nested_call()); is used in a function that is meant to return integer type - not a error signal.
uint8_t s2n_answer_to_the_ultimate_question() {
  GUARD(s2n_sleep_for_years(7500000));
  return 42;
}

In this function we intended to return a uint8_t but used a GUARD which will return -1 if the call fails. This can lead to very subtle bugs.
GUARDing a function returning any integer type
There is no compiler error if GUARD(nested_call()); is used on a function that doesn't actually return an error signal
int s2n_deep_thought() {
  GUARD(s2n_answer_to_the_ultimate_question());
  return 0;
}

In this function we intended guard against a failure of s2n_answer_to_the_ultimate_question but that function doesn't actually return an error signal. Again, this can lead to sublte
bugs.
Ignored error signals
Without the warn_unused_result function attribute, the compiler provides no warning when forgetting to GUARD a function. Missing a GUARD can lead to subtle bugs.
int s2n_answer_to_the_ultimate_question() {
  s2n_sleep_for_years(7500000); // <- THIS SHOULD BE GUARDED!!!
  return 42;
}

Solution
s2n_result provides a newtype declaration, which is popular in languages like Haskell and Rust.
Functions that return S2N_RESULT are automatically marked with the warn_unused_result attribute, which ensures they are GUARDed.
Follow up
Over the next week I will gradually be moving functions to return S2N_RESULT instead of int.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Nice!

one thing to note: functions in fuzz tests are unguarded, if we enforce compiler warnings, we could consider adding FUZZ_GUARD() macros

Ignoring an error signal, even in fuzz tests, is a bad idea. Having a guard function that crashes might be the way to go there..

Codecov Report

Merging #1872 into master will increase coverage by 0.00%.
The diff coverage is 97.26%.

@@           Coverage Diff           @@
##           master    #1872   +/-   ##
=======================================
  Coverage   81.44%   81.45%
=======================================
  Files         226      228    +2
  Lines       16905    16921   +16
=======================================
+ Hits        13769    13783   +14
- Misses       3136     3138    +2

I'll add the guide in a separate PR


[Identity] Finish moving things into Extensions

Fixes https://github.com/aspnet/Identity/issues/1253
I've left the more interesting question of how to expose things via AddIdentity Core to a seperate PR.  If we tackle that now, we basically end up tackle making IdentityRoles optional as well, since it'd show up at this layer.
I'll send out that PR as well and we can decide if its a 2.0 or 2.1...
@ajcvickers @blowdart @divega

The net result of this PR is we have enough in Microsoft.Extensions.Identity.Core to be able to add
services.AddIdentityCore<TUser>.AddRoles<TRoles>() which would get you everything except SignInManager + AspNet cancellation token hookup (which I don't think anyone will even notice)

Will replace with a unifed PR that does this + adds roles support and cleans everything up


flutter_gallery__transition_perf_e2e_ios fails when device is rotated in landscape

List of builds:
https://ci.chromium.org/p/flutter/builders/prod/Mac_staging flutter_gallery__transition_perf_e2e_ios
[flutter_gallery__transition_perf_e2e_ios] [STDOUT] stdout: 2021-02-05 19:07:04.876848-0800 Flutter Gallery[7467:1671670] flutter: > Tooltips@Material
[flutter_gallery__transition_perf_e2e_ios] [STDOUT] stdout: 2021-02-05 19:07:06.710099-0800 Flutter Gallery[7467:1671670] flutter: < Success
[flutter_gallery__transition_perf_e2e_ios] [STDOUT] stdout: 2021-02-05 19:07:06.710241-0800 Flutter Gallery[7467:1671670] flutter: > Activity Indicator@Cupertino
[flutter_gallery__transition_perf_e2e_ios] [STDOUT] stdout: 2021-02-05 19:07:07.330152-0800 Flutter Gallery[7467:1671670] flutter:
[flutter_gallery__transition_perf_e2e_ios] [STDOUT] stdout: 2021-02-05 19:07:07.330484-0800 Flutter Gallery[7467:1671670] flutter: Warning: A call to tap() with finder "exactly one widget with text "Cupertino" (ignoring offstage widgets): Widget" derived an Offset (Offset(111.2, 427.3)) that would not hit test on the specified widget.
[flutter_gallery__transition_perf_e2e_ios] [STDOUT] stdout: 2021-02-05 19:07:07.330717-0800 Flutter Gallery[7467:1671670] flutter: Maybe the widget is actually off-screen, or another widget is obscuring it, or the widget cannot receive pointer events.
[flutter_gallery__transition_perf_e2e_ios] [STDOUT] stdout: 2021-02-05 19:07:07.331394-0800 Flutter Gallery[7467:1671670] flutter: Indeed, Offset(111.2, 427.3) is outside the bounds of the root of the render tree, Size(667.0, 375.0).
[flutter_gallery__transition_perf_e2e_ios] [STDOUT] stdout: 2021-02-05 19:07:07.331853-0800 Flutter Gallery[7467:1671670] flutter: The finder corresponds to this RenderBox: <optimized out>#0ecbd relayoutBoundary=up4
[flutter_gallery__transition_perf_e2e_ios] [STDOUT] stdout: 2021-02-05 19:07:07.332054-0800 Flutter Gallery[7467:1671670] flutter: The hit test result at that offset is: HitTestResult(<optimized out>#dbf7e(<optimized out>#964dc), <optimized out>#bfd78(<BindingBase>))
[flutter_gallery__transition_perf_e2e_ios] [STDOUT] stdout: 2021-02-05 19:07:07.332255-0800 Flutter Gallery[7467:1671670] flutter: #0      WidgetController._getElementPoint (package:flutter_test/src/controller.dart:953)
[flutter_gallery__transition_perf_e2e_ios] [STDOUT] stdout: 2021-02-05 19:07:07.332443-0800 Flutter Gallery[7467:1671670] flutter: #1      WidgetController.getCenter (package:flutter_test/src/controller.dart:836)
[flutter_gallery__transition_perf_e2e_ios] [STDOUT] stdout: 2021-02-05 19:07:07.335142-0800 Flutter Gallery[7467:1671670] flutter: #2      WidgetController.tap (package:flutter_test/src/controller.dart:271)
[flutter_gallery__transition_perf_e2e_ios] [STDOUT] stdout: 2021-02-05 19:07:07.335799-0800 Flutter Gallery[7467:1671670] flutter: #3      runDemos (file:///opt/s/w/ir/cache/flutter%20sdk/dev/integration_tests/flutter_gallery/test_driver/run_demos.dart:42)
[flutter_gallery__transition_perf_e2e_ios] [STDOUT] stdout: 2021-02-05 19:07:07.336907-0800 Flutter Gallery[7467:1671670] flutter: <asynchronous suspension>
[flutter_gallery__transition_perf_e2e_ios] [STDOUT] stdout: 2021-02-05 19:07:07.337087-0800 Flutter Gallery[7467:1671670] flutter: #4      StackZoneSpecification._registerUnaryCallback.<anonymous closure> (package:stack_trace/src/stack_zone_specification.dart)
[flutter_gallery__transition_perf_e2e_ios] [STDOUT] stdout: 2021-02-05 19:07:07.337356-0800 Flutter Gallery[7467:1671670] flutter: <asynchronous suspension>
[flutter_gallery__transition_perf_e2e_ios] [STDOUT] stdout: 2021-02-05 19:07:07.337544-0800 Flutter Gallery[7467:1671670] flutter: To silence this warning, pass "warnIfMissed: false" to "tap()".
[flutter_gallery__transition_perf_e2e_ios] [STDOUT] stdout: 2021-02-05 19:07:07.337642-0800 Flutter Gallery[7467:1671670] flutter: To make this warning fatal, set WidgetController.hitTestWarningShouldBeFatal to true.
[flutter_gallery__transition_perf_e2e_ios] [STDOUT] stdout: 2021-02-05 19:07:07.337857-0800 Flutter Gallery[7467:1671670] flutter:
[flutter_gallery__transition_perf_e2e_ios] [STDOUT] stdout: 2021-02-05 19:07:12.112587-0800 Flutter Gallery[7467:1671670] flutter: (The following exception is now available via WidgetTester.takeException:)
[flutter_gallery__transition_perf_e2e_ios] [STDOUT] stdout: 2021-02-05 19:07:12.112846-0800 Flutter Gallery[7467:1671670] flutter: Bad state: No element
[flutter_gallery__transition_perf_e2e_ios] [STDOUT] stdout: 2021-02-05 19:07:12.114986-0800 Flutter Gallery[7467:1671670] flutter: #0      Iterable.single (dart:core/iterable.dart:498)
[flutter_gallery__transition_perf_e2e_ios] [STDOUT] stdout: 2021-02-05 19:07:12.116315-0800 Flutter Gallery[7467:1671670] flutter: #1      WidgetController.element (package:flutter_test/src/controller.dart:112)
[flutter_gallery__transition_perf_e2e_ios] [STDOUT] stdout: 2021-02-05 19:07:12.117304-0800 Flutter Gallery[7467:1671670] flutter: #2      WidgetController.dragUntilVisible.<anonymous closure> (package:flutter_test/src/controller.dart:1181)
[flutter_gallery__transition_perf_e2e_ios] [STDOUT] stdout: 2021-02-05 19:07:12.118310-0800 Flutter Gallery[7467:1671670] flutter: <asynchronous suspension>
[flutter_gallery__transition_perf_e2e_ios] [STDOUT] stdout: 2021-02-05 19:07:12.119109-0800 Flutter Gallery[7467:1671670] flutter: <asynchronous suspension>
[flutter_gallery__transition_perf_e2e_ios] [STDOUT] stdout: 2021-02-05 19:07:12.119518-0800 Flutter Gallery[7467:1671670] flutter: (elided one frame from package:stack_trace)
[flutter_gallery__transition_perf_e2e_ios] [STDOUT] stdout: 2021-02-05 19:07:12.119913-0800 Flutter Gallery[7467:1671670] flutter: (If WidgetTester.takeException is called, the above exception will be ignored. If it is not, then the above exception will be dumped when another exception is caught by the framework or when the test ends, whichever happens first, and then the test will fail due to having not caught or expected the exception.)
[flutter_gallery__transition_perf_e2e_ios] [STDOUT] stdout: 2021-02-05 19:07:12.121391-0800 Flutter Gallery[7467:1671670] flutter: Bad state: No element
[flutter_gallery__transition_perf_e2e_ios] [STDOUT] stdout: 2021-02-05 19:07:12.122725-0800 Flutter Gallery[7467:1671670] flutter: #0      Iterable.single (dart:core/iterable.dart:498)
[flutter_gallery__transition_perf_e2e_ios] [STDOUT] stdout: 2021-02-05 19:07:12.123074-0800 Flutter Gallery[7467:1671670] flutter: #1      WidgetController.element (package:flutter_test/src/controller.dart:112)
[flutter_gallery__transition_perf_e2e_ios] [STDOUT] stdout: 2021-02-05 19:07:12.123396-0800 Flutter Gallery[7467:1671670] flutter: #2      WidgetController.dragUntilVisible.<anonymous closure> (package:flutter_test/src/controller.dart:1181)
[flutter_gallery__transition_perf_e2e_ios] [STDOUT] stdout: 2021-02-05 19:07:12.124027-0800 Flutter Gallery[7467:1671670] flutter: <asynchronous suspension>
[flutter_gallery__transition_perf_e2e_ios] [STDOUT] stdout: 2021-02-05 19:07:12.124327-0800 Flutter Gallery[7467:1671670] flutter: <asynchronous suspension>
[flutter_gallery__transition_perf_e2e_ios] [STDOUT] stdout: 2021-02-05 19:07:12.124476-0800 Flutter Gallery[7467:1671670] flutter: (elided one frame from package:stack_trace)
[flutter_gallery__transition_perf_e2e_ios] [STDOUT] stdout: 2021-02-05 19:07:12.124946-0800 Flutter Gallery[7467:1671670] flutter: 01:36 +0 ~1: flutter gallery transitions on e2e all demos [E]
[flutter_gallery__transition_perf_e2e_ios] [STDOUT] stdout: 2021-02-05 19:07:12.125413-0800 Flutter Gallery[7467:1671670] flutter:   Test failed. See exception logs above.
[flutter_gallery__transition_perf_e2e_ios] [STDOUT] stdout:   The test description was: all demos
[flutter_gallery__transition_perf_e2e_ios] [STDOUT] stdout: 2021-02-05 19:07:12.125862-0800 Flutter Gallery[7467:1671670] flutter:
[flutter_gallery__transition_perf_e2e_ios] [STDOUT] stdout: 2021-02-05 19:07:12.133191-0800 Flutter Gallery[7467:1671670] flutter: 01:36 +0 ~1 -1: (tearDownAll)
[flutter_gallery__transition_perf_e2e_ios] [STDOUT] stdout: 2021-02-05 19:07:12.136105-0800 Flutter Gallery[7467:1671670] flutter: Warning: integration_test test plugin was not detected.
[flutter_gallery__transition_perf_e2e_ios] [STDOUT] stdout: 2021-02-05 19:07:12.137126-0800 Flutter Gallery[7467:1671670] flutter: 01:36 +1 ~1 -1: Some tests failed.
[flutter_gallery__transition_perf_e2e_ios] [STDOUT] stdout: Failure Details:
[flutter_gallery__transition_perf_e2e_ios] [STDOUT] stdout: Failure in method: all demos
[flutter_gallery__transition_perf_e2e_ios] [STDOUT] stdout:
[flutter_gallery__transition_perf_e2e_ios] [STDOUT] stdout: end of failure 1
[flutter_gallery__transition_perf_e2e_ios] [STDOUT] stdout:
[flutter_gallery__transition_perf_e2e_ios] [STDOUT] stdout:
[flutter_gallery__transition_perf_e2e_ios] [STDOUT] stdout:
[flutter_gallery__transition_perf_e2e_ios] [STDOUT] "/opt/s/w/ir/cache/flutter sdk/bin/flutter" exit code: 1
[flutter_gallery__transition_perf_e2e_ios] [STDERR] Task failed: Executable "/opt/s/w/ir/cache/flutter sdk/bin/flutter" failed with exit code 1.

\cc @jmagman

flutter_gallery_ios__transition_perf is also failing consistently with the same error:
https://ci.chromium.org/p/flutter/builders/prod/Mac_staging flutter_gallery_ios__transition_perf

This is another case where the tests are failing because the device is rotated.

Hotfix request to get flutter_gallery__transition_perf_e2e_ios passing on the stable branch.

Hotfix request to get flutter_gallery__transition_perf_e2e_ios passing on the stable branch.

This would be a test-only hotfix.


Feature request: option to hide text

I'd like to be able to hide the xx/160 text and show just the icon, so I can save some space in the menubar

Resolved in v1.12.126


[wip] Setup auto builds for base randomizer patch

Currently, this just patches a ROM with the build process established in #1. Once we work out the details of what patching format we're going to use, this should be updated to actually generate a patch that would be bundled with a release of the randomizer.

Superseded by #5


doc: Update English README, add Chinese readme back

This pull request fix following issues:
#25
#39

fix #39


Performance and capacity?

I've been testing tiedot to use as an integrated storage options for one of my tools and have noticed issues with performance on inserts
I am testing inserts of 1000 rows at a time be executed concurrently and running this 10,000 times to give a total of 10,000,000 rows, I want to store approximately 120 collections each with 10-20 million or so rows.
I have 12 indexes and that probably makes performance a lot worse, however for every 100k rows I am seeing about a 1.2 second increase in insertion times, as of the 400th iteration we're on 4.5-5s per 1000 rows.
Essentially I'd like to know if tiedot just isnt built for large datasets or if im doing something wrong?
I should also note that I am doing this from a machine with 8 threads available.

I've definitely nailed it down to the indexes, without any indexes it takes 14 minutes to insert 10 million rows, however a single index alone takes 2-4x longer than the insert time to create

Hello Evan.
Sorry to hear that tiedot has so far failed to perform beyond 4 millions of records. University professors taught me "scenarios not covered by test case may be assumed not working", and it has indeed happened. I remember having conducted benchmark with 2 millions of records at most.
You are correct in saying that having large number of indexes will greatly impact collection performance, let us try some common database tuning techniques and see if they can help:

Monitor disk IO activities during the benchmark and look for signs of dirty disk buffer pausing tiedot's memory access, if it has indeed occurred consider slightly increasing "vm.dirty_ratio" and further decrease "vm.dirty_background_ratio". Also, avoid using a feature-rich file system such as btrfs to store tiedot data files.
Limit gomaxprocs to a smaller number. This may sound counter intuitive, but if disk IO was indeed the bottleneck, freeing up CPU resources for doing busy IO work may actually help.
Disable transparent huge pages and same page merging.
Fix tiedot process onto few cpu cores using taskset. Though you computer most likely does not use NUMA, but just keep in mind to avoid using tiedot across different NUMA nodes in other computer systems.


User Story 768790: Get zlib working for a single platform in main (Conan)

Description
https://dev-mc.visualstudio.com/Minecraft/_workitems/edit/768790
The extra zlib directory before the zlib.h header is not necessary here and needs to be adjusted in order to work with the conan package for zlib. You can see in the minecraftpe zlib cmake that we add the cmake current directory and the zlib directory so #include <zlib/zlib.h> and #include <zlib.h> should be equivalent.
Related PR
https://github.com/Mojang/Minecraftpe/pull/37354

As long

I was gonna say as long as it's still builds in non-Conan mode... Then I saw your comment in the description the directly addressed that, but I forgot I started to write a comment. :)


Sync multiple stream

Hi there,
I use SRS on 3 differents servers, so 3 IPs. The goal is to have 3 video input from 3 smartphones (using broadcastme mobile app). I have those 3 inputs in my OBS software and then stream it to facebook.
My issue : those 3 inputs are totally desynch, even if allsmartphones are on the same wifi.
Any clue how I can fix it? Is is possible to have multiple devices to the same IPs?
Thanks a for lot for your work and help!

what's the mean of sync multiple stream?  Do you mean the same delay of different streams as if the 3 different stream coming from the same device?

Hi, yes you got it!
I have 3 smartphones, each streaming to a different server:
Smartphone1 :  rtmp://"IP1"/live/stream
Smartphone2 :  rtmp://"IP2"/live/stream
Smartphone3:  rtmp://"IP3"/live/stream
and when I add those 3 streams in OBS, videos are not sync...
thanks for your answer

@pryiz  I wonder how to check the streams whether sync? just by eyes? what's your demand by syncing the 3 streams?  for the communicating with 3 people?

This would be an excellent feature which I could also really use.
Maybe inject a timestamp into the stream which is then used to hold back and sync the streams? Smartphone apps could get current time using ntp server and add the current timestamp to meta info of rtmp stream?

Won't support, because it's not a common use scenario.

@winlinvip ,
Please add synchronizing media streams function. This is a unique feature for the project and can attract users like OBS.
This is already implemented in RTP.
SRT?, RTMP? implementation is necessary.
Larix Broadcaster (aka mobile obs) streaming client has added a feature to send NTP timestamps for all supported streams:
Insert SEI time metadata and specify NTP server, this allows synchronizing streams on server side.
https://softvelum.com/timesync/

RTP - Synchronizing media streams:
Implementation of NTP is not required to use RTP...
However, if NTP timestamp implemented in RTCP, it can be used to synchronize media streams even from different host...
https://gist.github.com/simonkim/a9a3aa4f1ca04448c64212b3b079d107

Synchronizing Streams via SRT:
Some SRT encoders like new JVC-cams , Haivision-encoder and software encoder LarixBroadcaster (iPhone/Android) can do this:
synchronize internal clock by time-server (usually by NTP protocol).
It means all encoders have one clock:if one camera has time 11:59:59 then other cameras have same time.
send this time inside encoded feed (SEI metainfo)
https://github.com/Haivision/srt/issues/2044#issuecomment-865034230
https://manpages.ubuntu.com/manpages/focal/man1/srt-multiplex.1.html


Any example of use please ?

I would like to test your tool but I'm not able to search for more info.

Just tried it myself and the lack of any example/documentation is a bit of a barrier. Looks like simplest use case is: answer := calc.Solve(formulaInString).

Adding an example to the readme would be very helpful considering you have this specified this as a stackoverflow answer, but without documentation of any kind, it makes it difficult for people to utilize.

@neelsg what did you import? I think I understand how to use the thing but I don't know how to actually get this library in my project.

@whitman-colm You may just want to use this instead: https://github.com/Knetic/govaluate
It does basically the same thing and it has documentation.  I am using it in a production application now and it is working well...

@whitman-colm I used go get github.com/marcmak/calc to pull it, but the source files actually sit in a subfolder called "calc", so in my project I used import "github.com/marcmak/calc/calc"

use https://github.com/Knetic/govaluate, not enough updates here


Ad-hoc review of Zashi PR

While checking notifications this PR stood up.
Did an ad-hoc review. https://github.com/Electric-Coin-Company/zashi-ios/pull/1044/

Lukas acknowledged the issue and scheduled further fixes for a subsequent release


apply style changes and add cn blog list

code block:

pre code block:

中文博文列表&博文内文：

seems we can make use of https://highlightjs.org/ ?


[WIP] Interpolation

This PR adds the interpolation operator.  We need CUDA_LAUNCH_BLOCKING enable for the operator to work because of Intrepid2 to work. Because of the complexity of splitting all the data for each finite element, this step is currently done on the host. Right now, there are tests for first order HGRAD finite elements both on hex and mixed quad/triangle topologies.
close #360

@Rombur will assess sanitizer issues

Codecov Report

Merging #371 into master will decrease coverage by 2.04%.
The diff coverage is 73.45%.

@@            Coverage Diff             @@
##           master     #371      +/-   ##
==========================================
- Coverage   98.05%   96.01%   -2.05%
==========================================
  Files          86       92       +6
  Lines        5911     6447     +536
==========================================
+ Hits         5796     6190     +394
- Misses        115      257     +142

Impacted Files
Coverage Δ

...ckages/Discretization/src/DTK_PointSearch_decl.hpp
100% <ø> (ø)
:arrow_up:

packages/Discretization/src/DTK_Topology.hpp
100% <ø> (ø)
:arrow_up:

...es/Discretization/src/DTK_InterpolationFunctor.hpp
100% <100%> (ø)

...kages/Discretization/src/DTK_Interpolation_def.hpp
100% <100%> (ø)

packages/Discretization/src/DTK_FE.hpp
21.66% <21.66%> (ø)

packages/Discretization/src/DTK_FE.cpp
28.57% <28.57%> (ø)

...ages/Discretization/src/DTK_Interpolation_decl.hpp
60.48% <60.48%> (ø)

packages/Discretization/test/tstInterpolation.cpp
99.41% <99.41%> (ø)

...es/Search/src/details/DTK_DetailsTreeTraversal.hpp
100% <0%> (ø)
:arrow_up:

... and 5 more

Continue to review full report at Codecov.

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

I have issues with the interface of the constructor of Interpolation(...)

The constructor is such that it takes as input the data from the Interface package. If we change the constructor, we will need more functions to hook into the Interface or we will need to rewrite the Interface package and thus, rewrite the C interface.

At this point if we have to refactor the interface to something that makes more sense for our algorithms I don't have a problem with that.

That means rewriting almost from scratch everything in Discretization. That's fine but that means that we can close this PR because we will need to rewrite the functions that are used here.

Per discussion with @Rombur @dalg24 it was determined that the Interpolation interface defines specific data input requirements to implement the kernel. However, we do not want the user to use such an interface directly and rather map to higher level input concepts. This mapping is not in the scope of this PR as we want to focus on the core kernel functionality. The next step will be the creation of a general FEM operator interface which maps these concepts to specific operator implementations.

One test is failing because the functor to interpolate hdiv and hcurl FE is not thread-safe. The test is only failing for OpenMP. I have no idea why it is not thread-safe.

After more investigation and talking with @aprokop, I've realized that the problem is in the test. HDIV FE do not have to be continuous at the face/edge and so what happens is that for the points on the face, we get different value depending on which cell the evaluation is done. I'll update the test so that the target points are inside the cells.

(Specifically, H(div,Ω) vector fields must have a continuous normal component and may have a discontinuous tangential component).

This can be merged/


Add randomized boards

The starter map for beginners would be a good option to select with something like a command line flag passed to the server, but for most games players will want a random board with randomized tiles, randomized number tokens on those tiles, and randomized ports.

Added in a24e384012dbf53ef88683641779518ef60ccfcc.


User lookup by usernames Error response issue

I am not 100% sure if my setup is wrong, but if one searches for multiple users, ie, the request has an array of parameters....if one of them produces an error, the whole response is nil 🤷♂
Here is the example of such a response on twitter developer portal:

@daneden
You can see in the screenshot what is the setup. Basically, find a character that produces an error ("e" is one of them), and send several other strings along with it in a single response.

So it seems it is not fully migrated, but possibly not planned at all? 🤷♂

@roblack thanks for reporting this issue! First it's worth pointing out that “user lookup” is different from “user search”.

User lookup assumes that you already know the complete username of the user you’re trying to fetch info for (i.e. a lookup for ja would only return the user @ja)
User search is a function to find users by username with an incomplete string (i.e. a search for ja might return @jack)

User search is not (yet) in the V2 API, but lookup is. Nevertheless, the error you reported is valid and needs fixing; when a subset of user names passed to the lookup endpoint returns an error, Twift should return an array of error objects instead of returning an error for the whole request.

I found the cause of this bug!
In the decodeOrThrow method that is responsible for decoding all requests to the Twitter API, I had written the method in a way that always throws errors when they’re encountered, even if a successful payload is returned along with errors.
In https://github.com/daneden/Twift/commit/1f47b9374a55ac47a27d9841a309a2b3fb98d051, I changed it so that Twift only throws errors when found if a successful response can’t be decoded.


chore: bdd-gh-1586284863 to 0.0.1

chore: Promote bdd-gh-1586284863 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

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


I changed setup into set up

Description of the change: changing setup to set up
Reason for the change: it's the wrong form of the word
Link to original source: https://sendgrid.com/docs/for-developers/sending-email/getting-started-with-transactional-emails/

💕I appreciate you.

Hello @apigirl,Thanks again for the PR! We appreciate your contribution and look forward to                                              continued collaboration. Thanks!                                             Team SendGrid DX


Use fixed-precision numbers everywhere

This PR changes the entire number system over to using fixed-precision decimals. While it's easy enough not to change the public API, I'm going to clean a couple loose ends up while I go so it'll be a breaking change.
Fixes #7, fixes #23.

I think I just got nerd sniped 😜 On the drive into work I realised going down the rabbit hole of fixed precision arithmetic just for the purpose of retaining that first decimal digit was overkill.
Turns out you can implement Gcode::minor_number() by dividing the fractional part of the number by 0.1 and rounding to the nearest integer.


catch missing package file with a friendly error instead of a stack trace

(plus a random whitespace change - removing trailing whitespace from a line)
After this change:
Package file "gosrc/cloudfoundry-honeycomb-nozzle" not found! Please check the Manifest path in your tile.yml file.

Before the change:
➜  tile build
Traceback (most recent call last):
  File "/usr/local/bin/tile", line 11, in <module>
    sys.exit(cli())
  File "/usr/local/lib/python2.7/site-packages/click/core.py", line 716, in __call__
    return self.main(*args, **kwargs)
  File "/usr/local/lib/python2.7/site-packages/click/core.py", line 696, in main
    rv = self.invoke(ctx)
  File "/usr/local/lib/python2.7/site-packages/click/core.py", line 1060, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/usr/local/lib/python2.7/site-packages/click/core.py", line 889, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/usr/local/lib/python2.7/site-packages/click/core.py", line 534, in invoke
    return callback(*args, **kwargs)
  File "/usr/local/lib/python2.7/site-packages/tile_generator/tile.py", line 52, in build_cmd
    cfg = Config().read()
  File "/usr/local/lib/python2.7/site-packages/tile_generator/config.py", line 80, in read
    self.transform()
  File "/usr/local/lib/python2.7/site-packages/tile_generator/config.py", line 87, in transform
    self.process_packages()
  File "/usr/local/lib/python2.7/site-packages/tile_generator/config.py", line 116, in process_packages
    self.update_compilation_vm_disk_size(manifest)
  File "/usr/local/lib/python2.7/site-packages/tile_generator/config.py", line 440, in update_compilation_vm_disk_size
    package_size = os.path.getsize(package_file) // (1024 * 1024) # bytes to megabytes
  File "/usr/local/Cellar/python/2.7.11/Frameworks/Python.framework/Versions/2.7/lib/python2.7/genericpath.py", line 57, in getsize
    return os.stat(filename).st_size
OSError: [Errno 2] No such file or directory: 'gosrc/cloudfoundry-honeycomb-nozzle'

Thank you, @maplebed!


Create a React app for the front end

Create a React app that will present a user interface and leverage the API.

A project called webui will be created at the root of the repository and given its own .gitignore for Node applications.
Move the current .gitignore for Go into each of the API, shared, and plotter projects.

On second thought, use Next.js instead.


Fix obsolete symbols in German translation

There are small errrors in the German translation:

The <br /> in the text is not rendered. As there is no linebreak in the English text, I'd propose to remove it.
There is an extra symbol in the word mittels. Maybe from the wiki import? That symbol shouldn't be there though.
When downloading the file, I noticed that there are a lot of whitespaces at the end of every line. Removed those as well.

thanks


Add nuance to definitions

E- x:

Add adopt
Buying access to a server vs buying a product (piece of software)

Duplicate to #1128


Fix indices with factories

Did you remember to?

[x] Add test case(s)
[ ] Update CHANGES.txt

@juherr - Is this PR fixing any specific issue ? I didn't quite get what the problem was apart from understanding that if a data provider returns only a specific set of indices and if the factory chooses to just use a subset of indices from those set of indices, TestNG should still be able to work.

Currently, as you say, it is a second subset. I don't know what should be the expected functional behavior but we have an internal exception if you run the test without the fix which is worst IMO.

ping @cbeust - Please help merge if you are fine with the changeset


Finding db file in non-git repositories

I noticed that this plugin only works correctly if the db for searching is either in the current directory or it's in the root of a git repository. Many times neither of these are true. Do you know the best way to do this?
I ended up modifying my path: 'set path+=;/' in .vimrc and just used findfile(db_name) where the db is being searched for. I'm definitely no vim expert so there might be a better way to do this.
It would probably be better to use something other than path, but I don't know to do this.
Thoughts? I'd be happy to submit a pr once we figure out the best method.

Always run :CodeQueryMakeDB in the root of a git repository.
A tip is that you can open any file (such as README.md) in this folder and specify the language you want to build (like :CodeQueryMakeDB python). Then you will have a database file which indexed all necessary files. What you need to do next is call CodeQueryMoveDBToGitDir python to archive this database file to .git/codequery/. Finally, you should not worry about the location of the database file.
If the tip doesn't work for your case, please let me know the way you use vim-codequery.

The project I'm not working on is not a git repository, so your tip doesn't work.

Gotcha!
Maybe we could store database files under ~/.vim-codequery/ directory.
Do you have any thought to design this feature?

I suppose something like that would work, using the path where the db was created as a key and then have vim search for the longest key that is included in the current path.
What do you think of what I suggested in the original comment? I really don't know much about vimscript, but it seems much simpler because the functionality for search up your current  is already built in.

modify db.vim to force the lookup_path as getcwd(),
and let lookup_path = findfile(git_root_dir . '/' . db_name, '.')
where your start vim is the DB search folder, it's more intuitive and similar usage as ctags and cscope


Can grpc support host or ip whitelist

As desc as title, can grpc support host or ip whitelist to filter out some incoming requests.

WE are looking at supporting some hiigh level access control mechanisms, but whitelists will probably need to be done manually.


Server throws HTTP 500 error when there is no accept header in the request

To reproduce the error, use this curl command:
curl "http://grlc.io/api-git/CLARIAH/grlc-queries/defaults?genre=http%3A%2F%2Fdbpedia.org%2Fresource%2FRock_music&endpoint=https%3A%2F%2Fdbpedia.org%2Fsparql" -H  "accept:"

The verbose output showing request and response headers:
*   Trying <IP_ADDRESS>:80...
* Connected to grlc.io (<IP_ADDRESS>) port 80 (#0)
> GET /api-git/CLARIAH/grlc-queries/defaults?genre=http%3A%2F%2Fdbpedia.org%2Fresource%2FRock_music&endpoint=https%3A%2F%2Fdbpedia.org%2Fsparql HTTP/1.1
> Host: grlc.io
> User-Agent: curl/7.87.0
>
* Mark bundle as not supporting multiuse
< HTTP/1.1 500 INTERNAL SERVER ERROR
< Date: Mon, 19 Jun 2023 12:19:18 GMT
< Content-Type: text/html; charset=utf-8
< Content-Length: 290
< Connection: keep-alive
< Server: gunicorn/19.6.0
< Access-Control-Allow-Origin: *
<
{ [290 bytes data]
* Connection #0 to host grlc.io left intact
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 3.2 Final//EN">
<title>500 Internal Server Error</title>

Internal Server Error

The server encountered an internal error and was unable to complete your request. Either the server is overloaded or there is an error in the application.

Running a local instance of grlc, I see these kind of errors in the application logs:
/usr/local/lib/python3.6/site-packages/grlc/fileLoaders.py:272: YAMLLoadWarning: calling yaml.load() without Loader=... is deprecated, as the default Loader is unsafe. Please read https://msg.pyyaml.org/load for full details.
  self.spec = yaml.load(resp.text)
/usr/local/lib/python3.6/site-packages/grlc/gquery.py:277: YAMLLoadWarning: calling yaml.load() without Loader=... is deprecated, as the default Loader is unsafe. Please read https://msg.pyyaml.org/load for full details.
  query_metadata = yaml.load(yaml_string)
ERROR:Exception on /api-url/company-detail [GET]
Traceback (most recent call last):
  File "/usr/local/lib/python3.6/site-packages/flask/app.py", line 2292, in wsgi_app
    response = self.full_dispatch_request()
  File "/usr/local/lib/python3.6/site-packages/flask/app.py", line 1815, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "/usr/local/lib/python3.6/site-packages/flask_cors/extension.py", line 161, in wrapped_function
    return cors_after_request(app.make_response(f(*args, **kwargs)))
  File "/usr/local/lib/python3.6/site-packages/flask/app.py", line 1718, in handle_user_exception
    reraise(exc_type, exc_value, tb)
  File "/usr/local/lib/python3.6/site-packages/flask/_compat.py", line 35, in reraise
    raise value
  File "/usr/local/lib/python3.6/site-packages/flask/app.py", line 1813, in full_dispatch_request
    rv = self.dispatch_request()
  File "/usr/local/lib/python3.6/site-packages/flask/app.py", line 1799, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
  File "/usr/local/lib/python3.6/site-packages/grlc/server.py", line 125, in query_param
    return query(user=None, repo=None, query_name=query_name, spec_url=spec_url, content=content)
  File "/usr/local/lib/python3.6/site-packages/grlc/server.py", line 47, in query
    glogger.debug("Request accept header: " + request.headers["Accept"])
  File "/usr/local/lib/python3.6/site-packages/werkzeug/datastructures.py", line 1463, in __getitem__
    return _unicodify_header_value(self.environ["HTTP_" + key])
KeyError: 'HTTP_ACCEPT'

If the accept header is expected/required, it'd be more appropriate to return a 4XX response code.
Otherwise, if no accept header is present, then serve JSON by default.

Hi @jaw111, thanks for reporting this.
I think in your example the accept header is present but blank. You are probably right and probably 400 Bad Request would be the most suitable response code?

Hi @c-martinez,
If you look at the verbose output, you see there is no accept header in the request.
Setting the header with an empty value will make curl remove the header, not just send a header with a blank value.
See: https://stackoverflow.com/a/7640101/21503085

@c-martinez the 406 Not Acceptable response status code would be a more meaningful response than 400 Bad Request.
Though https://developer.mozilla.org/en-US/docs/Web/HTTP/Status/406 suggests:

In practice, this error is very rarely used. Instead of responding using this error code, which would be cryptic for the end user and difficult to fix, servers ignore the relevant header and serve an actual page to the user. It is assumed that even if the user won't be completely happy, they will prefer this to an error code.


[ttLib] name module Macintosh langtags errata

In module ttLib.tables._n_a_m_e:
(Line 700) 9: 'no'  ->  9: 'nb':
'no' is the marcolangtag of Norwegian Bokmål ('nb', Mac langID 9), and Norwegian Nynorsk ('nn', Mac langID 151).
(Line 731) 40: 'si'  ->  40: 'sl':
It should be Slovenian, not Sinhalese, whose Mac langID is 76.
Thanks so much!

@Pal3love I think the first is already correct. The Macintosh langID 9 is simply called "Norwegian", so 'no' should be the correct language tag.
On the second one, you're right. the langID 40 is Slovenian, whose BCP47 tag is 'sl', not 'si' (which is Sinhalese, Mac langID 76).
I'll fix that, thanks for spotting.
https://www.microsoft.com/typography/otspec/name.htm
http://www.iana.org/assignments/language-subtag-registry/language-subtag-registry


Suggestions...

Hi Asim.
I just came by this project searching a simple, easy to use, docker swarm friendly message queue solution.
Many other popular message queues are not suitable for my current job.
I am very impressed for your design. It's what I am looking for.
However, after short review I am not sure but concluded it is long way to go.
If you don't mind I want to share my requirements.

Java script client including auto reconnection support.
Message requeue support from client.
Message persistent for HA; seems now just write only and have no saving path config.
Message TTL support (Auto remove on timeout)

Best regards.
Sungkyu Kim.

I do not have any bandwidth to do these things, please feel free to PR. Thanks for the suggestions.

Thank you for your response.
Unfortunately I don't have either, but someday If I can I will.


ref() called only once with a DOM element on conditional rendering

I'm trying to get a reference to an input displayed in a modal dialog in an app that looks more or less like this:
class App extends Component {
  render() {
    const { open } = this.state
    return (
      <div>
        { // ... more code }
        {
          open ? (
            <Overlay>
              <input type="file" ref={this.refInput} />
            </Overlay>
          ) : null
        }
        { // ... more code }
       </div>
    )
  }

  refInput = (el) => {
    console.log('refInput', el)
  }
}

When the open flag is set to true for the first time, the refInput is called with a DOM element as expected.
Now, when the open flag is set to false, the refInput is called with null - as expected.
The problem appears when we set the open flag to true again. refInput is never called with a DOM element again.
Interestingly, when we replace <Overlay/> with a <div/> - refInput gets called as expected.
Here is an example illustrating the issue: https://jsfiddle.net/3hvru6ex/2990/
And here the same example in React, working correctly: https://jsfiddle.net/n5u2wwjg/206396/

Looks like this is a symptom of component recycling.

We fixed this for our next release 🎉 An alpha version is already available on npm 👍


KAFKA-17057: Add RETRY option to ProductionExceptionHanlder

Implements KIP-1065

Build failure resolved by rebasing. Also addressed review comments expect the one about null check -- we should merge the other PR first, because the other PR need to be cherry-picked to 3.9, too, and rebase this PR afterwards.
THUS: DON'T MERGE THIS PR before we could merge the other one and rebase this one.

Rebased and address all comments. Hope I got it right; please take another look.


[Issue #4500] Fix resource bundle accessor issue when testing with the CLI

Fix issue where resource bundle accessor code returns nil when testing a package with the package manager CLI.
Motivation:
Please see #4500 for the context regarding the issue. In short, when testing a package's access to the resource bundle, the resource bundle will be nil when being accessed in an Objective-C context. This is because there is a difference in logic between the resource_bundle_accessor.m used in an Objective-C context and the resource_accessor.swift used in a Swift context.
As seen below, the Swift implementation will try to access the main bundle but fall back to the bundle at the build path (line 834 in below snippet). The Objective-C implementation does not have this code path and therefore returns a nil bundle when the main bundle does not exist. This is the case when testing a package using the CLI.
resource_bundle_accessor.swift
https://github.com/apple/swift-package-manager/blob/87254d26fa4287e45b20c5eba8bd71386942a880/Sources/Build/BuildPlan.swift#L811-L841
resource_bundle_accessor.m
https://github.com/apple/swift-package-manager/blob/87254d26fa4287e45b20c5eba8bd71386942a880/Sources/Build/BuildPlan.swift#L502-L507
Reproducing the issue
The issue can be reproduced by running swift test on a package that expects a non-nil resource bundle in Objective-C tests. I have also created a minimum, reproducible example: https://github.com/ncooke3/Package4500.
This change enables package developers to successfully test packages using the package manager CLI.
Modifications:
Modified the resource_bundle_accessor.m implementation to fall back to the bundle in the build folder when the main bundle does not exist.
Result:
Accessing the module's resource bundle will work when using the package manager CLI.
Fixes #4500

@swift-ci please smoke test

@swift-ci please smoke test

@MaxDesiatov, I'm not sure if the CI bot listens to me. Can you tell if CI is running and please trigger it if it's not?

@swift-ci please smoke test

I'm not sure if the CI bot listens to me.
It does not, only committers can trigger CI builds.

thanks @ncooke3


Need help: Wrap object in key when posting data

Hi,
I need help with a problem I'm currently dealing with. I want to use Swagger UI and send the data in a POST request wrapped in an object key and don't know how to do this. I already tried several things but it looks like I'm not clever enough.
Here is a simplified version of my Swagger Blocks code:
swagger_path '/users' do
  operation :post do
    key :operationId, 'createUser'
    parameter do
      key :name, :user
      key :in, :body
      key :required, true
      schema do
        key :'$ref', :UserInput
      end
    end
  end
end

swagger_schema :UserInput do
  key :required, [:name]
  property :name do
    key :description, "Name of the User"
    key :type, :string
  end
end

The Swagger UI tries to send a POST request with parameters in the body:
curl -X POST -d "{\"name\": \"string\"}" http://whatever.com/

What I need is something like this, sending the body parameters wrapped in a key:
curl -X POST -d "{\"user\":{\"name\": \"string\"}}" http://whatever.com/

Is something like that possible using swagger blocks options?
Thanks for your help!
Alex

Hey! Sorry, this is a question about Swagger and Swagger UI, so this isn't the right place for it. If you find that Swagger supports what you are trying to do, but that swagger-blocks can't build the right structure, that would be a bug, but otherwise please see: https://github.com/fotinakis/swagger-blocks#filing-issues


Prefer the configured quote style

Summary
This PR extends the string formatting to respect the configured quote style.

Test Plan
Extended the string test with new cases and set it up to run twice: Once with the quote_style: Doube, and once with quote_style: Single single and double quotes.

Current dependencies on/for this PR:

main

PR #5346

PR #5347

PR #5349   👈

This comment was auto-generated by Graphite.

@MichaReiser merged this pull request with Graphite.


Add license to the system repository

We need to figure out the license we want to go with.

I think its fine to go with MIT, which if I remember is your preferred one. We should also include the following paragraph.

This game system is powered by ZWEIHÄNDER Grim & Perilous RPG ™, a trademark of Grim & Perilous Studios LLC, in the USA and other countries. Used with permission.  Original text content of ZWEIHÄNDER Grim & Perilous RPG is licensed under a Creative Commons Attribution-Noncommercial-Share Alike 3.0 Unported License. ZWEIHÄNDER Logo, content and original artwork are copyright © 2009–2018, Grim & Perilous Studios, LLC unless otherwise specified.


What is mentioned as batch operation is just a loop

how to perform the batch operation, see AzBulkSetBlobTier.
That page just shows a loop and that is not a batch operation. Misleading
Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 392e90a0-af77-999e-39a1-78f09ffe0d06
Version Independent ID: 411ca9a3-4370-f9ca-f14c-5385e13d9075
Content: Hot, cool, and archive access tiers for blob data - Azure Storage
Content Source: articles/storage/blobs/access-tiers-overview.md
Service: storage
Sub-service: blobs
GitHub Login: @normesta
Microsoft Alias: normesta

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

@sijucm
One option for you is to use Azure Storage Blobs Batch client library for .NET. You can write PowerShell script that makes use of this library to perform batch operations on blobs. You can find more information and code samples here: https://github.com/Azure/azure-sdk-for-net/blob/master/sdk/storage/Azure.Storage.Blobs.Batch/README.md.
Other option would be to consume REST API. Again, you can write PowerShell script that makes the HTTP requests to perform batch operations against your storage account. You can find more information about the REST API here: https://learn.microsoft.com/en-us/rest/api/storageservices/blob-batch.
I would've included the code but unfortunately my knowledge in PowerShell is very limited.
If there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.
@normesta
Can you please check and add your comments on this doc update request as applicable.

@sijucm - Would you pls direct me to the section of this article which references batch operations? I'd like to examine that guidance a bit closer. I don't see the reference. Thanks!

@sijucm
We are going to close this thread as resolved but if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.


IPFS support

IPFS is a distributed peer-to-peer network, which offers the hosting of static sites:
https://ipfs.io/
https://ipfs.io/ipfs/QmdPtC3T7Kcu9iJg6hYzLBWR5XCDcYMY7HV685E3kH3EcS/2015/09/15/hosting-a-website-on-ipfs/
Can you support it?

Editing a site which is hosted on IPFS should be perfectly feasible once you have a build process which deploys to IPFS - Netlify CMS doesn't handle building or hosting the site whatsoever, just editing the content. You should be able to set up a GitHub repo for your content and deploy the content to IPFS from GitHub however you want. The only missing piece you'd need is an auth server for authenticating with GitHub. We provide that for free for sites hosted on Netlify, but custom auth servers exist - see https://www.netlifycms.org/docs/custom-authentication for more details.
Using IPFS as a backend for your source files, which would require direct interaction with IPFS by the CMS, would be significantly more complicated. I don't know enough about IPFS yet to say how that would be best implemented, but unless IPFS supports pushing changes directly from the browser somehow it would require an intermediary server to push changes. If that's the case, I don't see it as terribly useful to have IPFS-specific handling in the CMS, since there are other potential backends that could be used to do the same thing - for instance, if local filesystem or local git repo backends were in place, you could run the CMS on a server which pushes changes to IPFS and
If IPFS does support pushing changes from the browser, it would probably be feasible to support it. I'd be thrilled at any PRs building CMS support for it if that's the case.

There is also a JS implementation, which runs without any installements or proxy like it is currently essential with the go implementation. https://github.com/ipfs/js-ipfs
Plus, a member said "its just the commit workflow that needs changing"

Is anybody interested in this?

This would be a great addition, a "publish to ipfs" option. I would let github host the source, no point in re-inventing that wheel. IPFS seems like a good place for the released for finished product. And yes, very interested in this.

You could still host it on any source hosting platform of your choice, such as Gitea. Choice is always nice and its not to hard to follow the standard that git is. No need to make it Github specific for no reason, I guess. :smiley:

Yes, I would like to have ability to publish into IPFS as well. Build process should support updating _dnslink as well. There is no need for netlify to run IPFS gateway.

+1 for this, not sure if it's in the pipeline but I'd love an option to publish to IPFS, maybe including IPNS for a static URL

This would be awesome!

+1

Yes iPad support would be great. Maybe net lift could also offer a pinning service like pinata. If there is a build process to include netlifycms that would be awesome.

I am working on this for a client, I'll see what I can post. But as a starter, this exists:
https://github.com/ipfs-shipyard/ipfs-deploy
So once you get a "build" you can run this and it will deploy to an ipfs pinning service (supports Pinata etc.) and then it will update DNS for you! Every bulid to IPFS requires you to update your DNS, ENS, HNS etc... so that it knows where to find the new IPFS data.
This plugin, for Gatsby, doesn't really work for Gatsby v4 .. https://www.gatsbyjs.com/plugins/gatsby-plugin-ipfs/ . It is a key piece so the built files have relative paths.

+1


DNM -  NP-10930: Google Place API Integration

Additional requests:

An OM for every call

Questions:

Do we have a null service for development which falls back to manual entry?
And does our system use manual entry when the real service is not available?

Additional requests:

An OM for every call

Questions:

Do we have a null service for development which falls back to manual entry?
And does our system use manual entry when the real service is not available?

@jlhughes Dont think we have a null service but the UI checks if the service is available and then falls back to manual entry if not. It only checks for if the service is in the context, not necessarily if the link to google is active and we are receiving valid responses. In any case, manual entry is always an option. If the server can respond with an explicit "service unavailable exception" if it fails to get a response from google we can make the switch to manual entry a little smoother on the client

@jlhughes
In the GooglePlaceServiceConfigure this is a property called is_enable when is_enable it false, it will simple return a empty array.
This is no retry, any error happens in backend will simply return a empty array and log in backend


New example website

OMG!

Petry =>  https://github.com/blog/273-github-ribbons

Closing, new branch: new-website


Railfence cipher in c and c++

🛠 Issue (Number):

Issue no #530
👨💻 Changes proposed

Added Railfence cipher program in c and c++.

✔ Check List (Check all the applicable boxes)

[✔] My code follows the code style of this project.
[✔ ] This PR does not contain plagiarized content.
[✔] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers

Initially the matrix is initialised with "*"
📷 Screenshots
Output in C :

Ouput in c++ :

Sir made another pr


import/no-unresolved does not affect jest.mock

Is there some way to get jest.mock('../../api/courseApi'); to be checked by the import/no-unresolved rule?

That seems like something a jest eslint plugin should handle.


post berkeley-schema-fy24 merge issue: review FileTypeEnum composition and correlation with other DataObject slots/relationships

Can a DataObject that is the output of any process use any FileTypeEnum in it's data_object_type slot?
do the different permissible values come from different axes of differentiation?
should we use an is_a hierarchy within the PVs?

Claude finds these different axes of differentiation or concerns in FileTypeEnum:

Data Type / Analysis Method:

Metagenome data
Metabolomics data (FT ICR-MS, GC-MS)
Metaproteomics data
Assembly data
Annotation data (various types)
Read-based analysis
Taxonomic classification (GOTTCHA2, Kraken2, Centrifuge)

Processing Stage:

Raw data
Filtered data
Error-corrected data
Assembled data
Annotated data

File Format:

FASTQ
BAM
FASTA
GFF
JSON
TSV
PDF
HTML

Sequencing Read Type:

Raw Read 1 (forward)
Raw Read 2 (reverse)
Interleaved paired-end

Quality Control Stage:

QC Statistics
QC non-rRNA reads

Biological Entity Focus:

Protein-related
Peptide-related
RNA-related (rRNA, tRNA, etc.)
Gene-related

Output Type:

Report files
Statistical files
Plot files (heatmap, barplot, Krona plot)
Binning results

Annotation Type:

Structural annotation
Functional annotation
Various specific annotation types (e.g., TIGRFam, CRT, Genemark, etc.)

Compression Status:

Compressed files (e.g., zip files for bins)
Uncompressed files

Workflow Stage:

Intermediate files
Final output files
Workflow statistics

@turbomam is this accomplished with the merged PR?


Is the Apple2 IO RPi hardware in general able to replace the ROM?

What do suggest for improvement?
If yes, could you please point me to where I can implement providing the ROM?
Looks like I got more bad ROMs in my Apple ][plus.
thanks

As far as i know the RP2040 can act as a rom card.

If you’re asking if this card will replace the computer’s main ROM, the answer is no. It only provides 256 bytes of ROM in the peripheral card space. That said, building a ROM replacement card isn’t too difficult. It’s just not the focus of this project or compatible with this hardware.

My idea was to make it in software...

While you can pass a lot of code through the port from the card to the Apple II, it will not occupy the expected ROM space. Definitely needs different hardware to replace the system ROMs. If you mean do calculations on the Pi side and just call the card to run that and get results, that of course will work.

Ok. Thanks. That's what I wanted to know :)


Unit Testing the generated source code

As i understand it, one of Rusthon's goal is readable and well generated code.
I propose Unit Tests that check the generated source code.
For example a test could verify that the comments format which was fixed by https://github.com/rusthon/Rusthon/commit/4ad14169528e1202b7fb7c177e8ebaaf3bf38d6e works for all supported backends.
This would require some refactoring, because not all functionality is available as code units.

Also the tests at the bottom of https://github.com/rusthon/Rusthon/blob/master/pythonjs/typedpython.py could be automized.

I am interested in (doing) this. It's just that you should agree.

Have you taken a look at regests/run.py, that's TexCoffier's script to run unit tests.  For comments you would need to hack run.py, because TestError was designed just to test things that return a value.
Feel free to work on these things, at the moment i'm fixing some things in the c++ backend, and starting on the verilog backend.

there is a new simpler regtest running below, that tests most of the examples, works on osx and linux:
https://github.com/rusthon/Rusthon/blob/master/regtests/test-markdowns.py


TS Server fatal error:  Cannot read properties of undefined (reading 'sourceFile')

Type: Bug
❗❗❗ Please fill in the sections below to help us diagnose the issue ❗❗❗
TypeScript Version: 5.6.2
Steps to reproduce crash

TS Server Log
❗ Server logging disabled. To help us fix crashes like this, please enable logging by setting:
"typescript.tsserver.log": "verbose"

After enabling this setting, future crash reports will include the server log.
VS Code version: Code - Insiders 1.94.0-insider (Universal) (527e41ff551bb1001d2f224c2dec2e2ea3b66832, 2024-09-20T05:03:57.400Z)
OS version: Darwin arm64 24.1.0
Modes:

System Info

Item
Value

CPUs
Apple M2 Pro (10 x 2400)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: enabled_ondirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_graphite: disabled_offvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabledwebgpu: enabledwebnn: disabled_off

Load (avg)
9, 6, 4

Memory (System)
32.00GB (1.75GB free)

Process Argv
--crash-reporter-id bb7a53e4-a432-400a-ab31-03fbfbaaa15e

Screen Reader
no

VM
0%

Extensions (19)

Extension
Author (truncated)
Version

better-comments
aar
3.0.2

project-manager
ale
12.8.0

jsoncrack-vscode
Ayk
2.0.3

npm-intellisense
chr
1.4.5

path-intellisense
chr
2.9.0

codeium
Cod
1.17.18

vscode-eslint
dba
3.0.13

gitlens
eam
2024.9.2105

prettier-vscode
esb
11.0.0

copilot
Git
1.232.0

copilot-chat
Git
0.21.2024092003

remote-explorer
ms-
0.4.3

remote-server
ms-
1.5.2

json-parse-stringify
nex
1.0.2

indent-rainbow
ode
8.3.1

errorlens
use
3.20.0

vscode-icons
vsc
12.9.0

quokka-vscode
Wal
1.0.652

pretty-ts-errors
Yoa
0.6.0

(1 theme extensions excluded)

A/B Experiments
vsliv368cf:30146710
vspor879:30202332
vspor708:30202333
vspor363:30204092
vscod805cf:30301675
vsaa593:30376534
py29gd2263:31024238
c4g48928:30535728
a9j8j154:30646983
962ge761:30841072
pythongtdpath:30726887
welcomedialog:30812478
pythonnoceb:30776497
asynctok:30898717
dsvsc014:30777825
dsvsc015:30821418
pythonmypyd1:30859725
h48ei257:31000450
pythontbext0:30879054
accentitlementst:30870582
dsvsc016:30879898
dsvsc017:30880771
dsvsc018:30880772
cppperfnew:30980852
pythonait:30973460
945dj816:31013170
a69g1124:31018687
dvdeprecation:31040973
dwnewjupytercf:31046870
legacy_priority:31057981
nativerepl1:31134653
refactort:31084545
pythonrstrctxt:31093868
flighttreat:31119334
wkspc-onlycs-t:31132770
nativeloc1:31118317
wkspc-ranged-t:31125599
jh802675:31132134
e80f6927:31120813
ei213698:31121563
i21gd607:31141543
iacca2:31138163
notype1:31143044
c9j82188:31138334
showbadge:31139796
f8igb616:31140137

Please try collecting the TS Server log from the crashed instance:

Set "typescript.tsserver.log": "verbose"

Restart VS Code and reproduce the problem

Open the TypeScript section of the output panel

At the very top, find the first line that prints to path to the semantic error log file. It should look something like:
[Info  - 19:54:59.247] <semantic> Log file: /Users/matb/Library/Application Support/Code - Insiders/logs/20200213T104930/exthost55/vscode.typescript-language-features/tsserver-log-ZT2zau/tsserver.log

That file contains the typescript logs.

Look through that log file for the first error or stack trace you see. If you can share the log, I can also take a look to see if anything stands out

⚠Warning: The TypeScript log may include information from your workspace, including file paths and source code. If you have any concerns about posting this publicly on Github, just let me know and we can arrange something else. On our side, we only use these logs to investigate issues like this


Add Brightway2 data management

Insert URLs to the project here:
https://github.com/brightway-lca/brightway2-data

[x] The projects is active, documented, open source licensed, shows usage from external parties and is directly targeting environmental sustainability. Find more details in the Contribution Guide.

All issues labeled as 'Good First Issue' of the project listed on OpenSustain.tech will be visible on ClimateTriage.com. This is a great way to welcome new community members to your project.

@cmutel @dgdekoning


Pass UnloadCancellationToken to Solution Restore Service

Customer scenario
Customers may experience a hang when closing a .NET Core solution if auto restore is running in the background.
This change was requested by NuGet team as part of their efforts to resolve this hang in https://github.com/NuGet/NuGet.Client/pull/1154. This change passes a CPS cancellation token that is cancelled when a project is unloaded to NuGet's solution restore service, which in turn passes it down to core algorithms.
Bugs this fixes:
https://github.com/NuGet/Home/issues/4257
Workarounds, if any
None
Risk
Project System side of this fix is low risk, since we are just passing a CPS cancellation token to an existing API.
Performance impact
None
Is this a regression from a previous update?
No
Root cause analysis:
Part of ongoing investigations into hangs
How was the bug found?
Dogfooding
/cc @dotnet/project-system
/cc NuGet team @rrelyea @alpaix
/cc @MattGertz for Escrow approval

Approved to merge via 377253


docs[patch]: Add intro to LCEL doc

Description:
Doc addition for LCEL introduction. Adds a more basic starter guide for using LCEL.

Related PR: https://github.com/langchain-ai/langchain/pull/14051


Adds option to skip download on the radio connection flow for the second micro:bit

Converting to a draft until there's a chance to discuss next year. It could make sense to connect, try the radio bridge handshake and then offer to flash if it doesn't work. Or add partial flashing support so the flash was a no-op. I agree you can't actually skip the connection.

Looking at the designs it's not a skip option, instead it is a link to the alternative 1 micro:bit method. That makes some sense as it's the point at which you might discover you have too few micro:bits. Not sure if there's been different communication from Lucy here but wanted to flag it as I spotted it.

We've since implemented the link to the 1 micro:bit flow at this point. Closing.


feat(storybook): move config folder to root

Description
A straight-forward move of tools/preview to .storybook.
Why?
To align with Storybook documentation and make it easier for external users to onboard to the project.
To-do list

[x] I have read the contribution guidelines.
[x] I have updated relevant storybook stories and templates.
[x] If my change impacts other components, I have tested to make sure they don't break.
[x] If my change impacts documentation, I have updated the documentation accordingly.
[x] ✨ This pull request is ready to merge. ✨

@pfulton Same diffs here as well as what we're seeing on main.


chatwork/kumina-postfix-exporter: usage with docker-compose

crosslinking with https://github.com/kumina/postfix_exporter/issues/104
->
Trying this:
# cat docker-compose.yml
version: '3.7'

networks:
  back-tier:

services:

  postfix-exporter:
    image: chatwork/kumina-postfix-exporter:latest
    volumes:
      - /var/spool/postfix/public/showq:/var/spool/postfix/public/showq:ro
      - /var/log/mail.log:/var/log/mail.log:ro
    command:
      - '--web.listen-address=":9154"'
      - '--postfix.logfile_path="/var/log/mail.log"'
        #- '--systemd.enable'
    ports:
      - 9154:9154
    networks:
      - back-tier
        #restart: always

Getting:
# docker-compose up
[+] Running 2/2
 ⠿ Network postfix-exporter_back-tier             Created                                                                                0.1s
 ⠿ Container postfix-exporter-postfix-exporter-1  Created                                                                                0.1s
Attaching to postfix-exporter-postfix-exporter-1
postfix-exporter-postfix-exporter-1  | 2023/04/19 11:23:39 Reading log events from "/var/log/mail.log"
postfix-exporter-postfix-exporter-1  | 2023/04/19 11:23:39 Failed to create PostfixExporter: open "/var/log/mail.log": no such file or directory
postfix-exporter-postfix-exporter-1 exited with code 1

Tried to mount the directory /var/log also, same problem.
/var/log/mail.log exists on the host with read permissions. Help welcome ....

https://github.com/kumina/postfix_exporter/issues/104


hostname_ssh when tmux is run on remote machine

Currently, #{hostname_ssh} (and #{username_ssh}) do not get interpolated as expected when I ssh into a remote machine and run tmux there. I would expect username and hostname to be shown in that case, as it is my most common use case (use tmux on local machine -> don't show hostname for #{hostname_ssh}, run it remotely -> show hostname).
This could be fixed by appending a test for there being an ssh connection when testing ! _is_enabled "$ssh_only" :
if ! _is_enabled "$ssh_only" || [ -n "$SSH_CLIENT" ] || [ -n "$SSH_TTY" ]; then
This may break other use cases, however.

Hello @zeeMonkeez 👋

Currently, #{hostname_ssh} (and #{username_ssh}) do not get interpolated as expected when I ssh into a remote machine and run tmux there

The hostname and username detection for SSH is orthogonal to which command runs on the remote.
Do you mean you want a remote tmux to display hostname and username only when it's "sshed into"? If that's the case, how do you deal with the following:

connect to remote server through ssh, e.g. $ ssh<EMAIL_ADDRESS>attach to tmux session ← the remote tmux status line displays jonas and example.com
from there, ssh into yet another server, e.g. $ ssh<EMAIL_ADDRESS>← now what the status line should be?

That was fast :)
Essentially, when I'm not on the local machine, I would like to see the last username@hostname of the chain, no matter how many ssh and tmux hops are in-between.
In your example 3, I'd expect to see<EMAIL_ADDRESS>That may be difficult to achieve ... But to me, having an indication that in the current pane I am or am not on a remote machine is quite valuable, independent of whether I did ssh→tmux or tmux→ssh.

Thanks makes sense :)


Could you kindly help to work on enable @Step annotation with Gradle configuration?

Dear @automationhacks , thanks for your work on provide this sample of ReportPortal of Gradle TestNG, it works fine with your Gradle configuration.
I am facing some problem in advance of ReportPortal when want to enable @Step annotation for log the test step, I am running the AspectJ Gradle plugin (https://plugins.gradle.org/plugin/io.freefair.aspectj, and this plugin's document: https://docs.freefair.io/gradle-plugins/6.0.0-m2/reference/#_io_freefair_aspectj) and with the sample code of the ReporPortal official provided (https://github.com/reportportal/examples-java/blob/master/example-testng-logback/src/main/java/com/epam/reportportal/example/testng/logback/step/NestedStepsTest.java#L92), unfortunately the nested step of ReporPortal doesn't work in expects.
Could you kindly help to work on enable @Step annotation and provide the fine build.gradle sample?
Just provide my current build.gradle that out of work as below.
Also I have a sample project that the AspectJ plugin and Lomok plugin work's build.gradle configuration (https://github.com/ansonliao/Gradle-AspectJ-Sample/blob/master/build.gradle), please reference if need.
Hope can hear from ASAP.
Thanks.
plugins {
    id 'java'
    id "io.freefair.aspectj" version "<IP_ADDRESS>"
}

group 'org.example'
version '1.0-SNAPSHOT'

sourceCompatibility = JavaVersion.VERSION_1_8
targetCompatibility = JavaVersion.VERSION_1_8
sourceSets.main.aspectj.srcDir "src/main/java"
sourceSets.main.java.srcDirs = files()
sourceSets.test.aspectj.srcDir "src/test/java"
sourceSets.test.java.srcDirs = files()
[compileJava, compileTestJava]*.options*.encoding = 'UTF-8'

repositories {
    mavenCentral()
    maven {
        url "https://jitpack.io"
    }
}

dependencies {
    // aspectj
    compile group: 'org.aspectj', name: 'aspectjrt', version: '1.9.6'
    // testng
    compile group: 'org.testng', name: 'testng', version: '7.3.0'
    // selenide
    compile group: 'com.codeborne', name: 'selenide', version: '5.16.2'
    // reporportal
    compile group: 'com.epam.reportportal', name: 'agent-java-testng', version: '5.0.8'
    // logback
    compile group: 'ch.qos.logback', name: 'logback-core', version: '1.2.3'
    compile group: 'org.slf4j', name: 'slf4j-api', version: '1.7.30'
    compile group: 'ch.qos.logback', name: 'logback-classic', version: '1.2.3'
    compile group: 'com.epam.reportportal', name: 'logger-java-logback', version: '5.0.3'
}

tasks.withType(Test) {
    systemProperty('file.encoding', 'UTF-8')
}

tasks.withType(Test) {
    testLogging.showStandardStreams = true
}

test {
    useTestNG() {
        useDefaultListeners = false
        listeners << 'com.epam.reportportal.testng.ReportPortalTestNGListener'
    }
}

aspectj {
    version = '1.9.6'
}

While it would take time for me to dig into this. You can try connecting with the report portal team on their Github repo or slack channel for a faster revert.

@automationhacks
Thanks for your reply, yes I am asking help from the slack channel of ReportPortal.


Use getBoundingClientRect(), IE9+ support

We started using colorjoe over at Selz.com (thanks!) and noticed there were a couple of issues when the picker was inside an element with overflow. Using getBoundingClientRect() seems to fix these issues. Also cleaned up some other code. IE9+ support only but since IE9 is < 1% now, this should be no issue.

Thanks. Published as 0.7.0 since I wanted to play it safe.