[BEAM-7230, BEAM-2699, BEAM-8515] Cache DataSource instances based upon DataSourceConfiguration

DataSource objects are currently being created once and assigned to a global variable creating a data race where multiple JdbcIO transforms may step on each other if executed in the same JVM when used with a PoolableDataSourceProvider or a DataSourceProviderFromDataSourceConfiguration.
To make this change, I needed to be able to make ValueProviders have meaningful equals methods.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

R: @iemejia @apilloud
CC: @kennknowles @mehdimas this change supersedes #9904

run java precommit

Thanks a lot @lukecwik for the analysis and quick solution. I am OOO this week so I could not take care of this quickly. Thanks :+1:


chore(deps): Fast Forward op-alloy-consensus

Description
Updates the op-alloy-consensus dependency to point to upstream op-alloy instead of a forked, outdated version.

#267  👈
main

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

Join @refcell and the rest of your teammates on  Graphite

Codecov Report
Attention: Patch coverage is 78.37838% with 8 lines in your changes missing coverage. Please review.

Project coverage is 70.2%. Comparing base (49a9d9c) to head (4585f5e).

Files
Patch %
Lines

bin/programs/client/src/l2/executor/mod.rs
78.3%
8 Missing :warning:

Additional details and impacted files

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Feature/ba 120 navigation

Pr includes initial implementation of the main navigation and subnavs

Thanks for the fixes @Braedencraig! Two small things:

The text for the submenu links aren't showing up for me anymore.
Could you move the padding from the li's down to the a tags on the top level?


TC-SC-4.10 python test

Category:
Functional conformance
Description:
The purpose of this test case is to verify that a Short Idle Time ICD node properly advertises SAI/SII
values.
Verifies:

SII key is higher than the SESSION_IDLE_INTERVAL default value (500 milliseconds)
SII key and SAI key is less than 1 hour (3600000 milliseconds)

Full test plan link for details:
https://github.com/CHIP-Specifications/chip-test-plans/blob/master/src/securechannel.adoc#tc-sc-4-10-operational-discovery-sit-icd-node-dut_commissionee

This test case has been deprecated and has been included in https://github.com/project-chip/connectedhomeip/pull/31982


HoloLens 2 and Task Support

This is a pretty big PR so I want to fully document the reason for each of the changes.
Main Work
I started this work because I could not get MRLT to work on HoloLens 2 without errors in the Unity log and on screen. The most prominent errors were:
HoloLens locatable camera has failed to set auto exposure off
HoloLens locatable camera has failed to set exposure to ## as requested
HoloLens locatable camera has failed to set auto WhiteBalance off
HoloLens locatable camera has failed to set WhiteBalance to #### as requested
[CameraCapture] Can't get camera matrix!!

It turned out that these errors are primarily due to HoloLens 2 wanting to use ExposureControl rather than Exposure and wanting to use WhiteBalanceControl rather than WhiteBalance.
My first attempts at fixing this looked promising, but seemed to only work randomly. Now I started seeing new errors in the log:
HoloLens locatable camera has failed to set exposure to 00:00:00.0101000. System.NullReferenceException: Object reference not set to an instance of an object.
HoloLens locatable camera has failed to set ISO to 80. System.ArgumentException: No capture devices are available.

These ended up stemming from two different core issues:

In HoloLens 2, the Control interfaces can only be adjusted while the camera is streaming.
It turned out there were several possible race conditions when setting these values (due to callbacks) and when shutting down or starting LightCapture back up.

Only adjusting the control values while streaming was a fairly easy fix. Apply them only if the camera is running, otherwise cache the values for the next time the camera is started.
The race conditions and callbacks ended up being responsible for the bulk of the refactoring. The root of the problem is that methods like Initialize, SetExposure and SetWhiteBalance all returned void, but internally they ran a bunch of callbacks. This made those methods function like async void. Now, I'm not going to say that async void should never be used, but it tends to be the source of many headaches, race conditions and threading bugs. This is because the caller has no way to know when the function has actually completed. For more information see Avoid Async Void.
It's also worth pointing out that any method that returns a Task can be treated as void, just not the other way around.
The following line:
var t = SetWhiteBalanceAsync(5000);
Is functionally equivalent to:
SetWhiteBalance(5000, myCallback);
If Task t is never awaited, it will run in the background and we'll never know whether it failed or completed. This would be equivalent to myCallback never getting called but the application being unaware it didn't happen.
So, every method in the library that previously used a callback was converted to return a Task. And every place that called one of those methods was also converted to a Task (whenever possible). There were a few cases where this wasn't possible. For example, when responding to a UI button click. In these "last mile" cases async void was used, but error handlers were employed.
Other Fixes and Changes
Those are the main things that were refactored. In addition there were also other fixes and improvements which I'll cover now:

Originally, code to change Exposure, White Balance and ISO was conditionally compiled to only run with UWP. I created a new interface ICameraControl that any camera controller on any platform can implement. Currently only UWP does, but the example now checks for this interface and will use it if available.

In the NativeArray<Color24> method of CameraCaptureWebcam there was a blind cast of resizedTexture to Texture2D. However, at least on my system resizedTexture is a WebCamTexture which does not inherit from Texture2D. This would have resulted in an InvalidCastException. For now I changed this to an As instead of a hard cast and throw an UnsupportedException with a clear message if the conversion can't be done.

In LightCapture.Update, the Map was being used before the camera had fully completed initializing. The reason this wasn't an error before was probably again due to callbacks. The camera was probably partially initialized (up to the first callback) but not completely initialized. This became apparent when initialization became a Task and that Task did not complete until the device was fully initialized. The fix was a simple check to make sure Map is not null before reading it.

CameraCaptureWebcam start delay was updated from 0.5 seconds to 2.0. I have a Logitech BRIO and it takes much longer than 0.5 seconds to initialize.

The range values for Exposure of -10 to 10 did not work for all devices and did not map easily to the new Control interfaces. I changed Exposure to a floating point value ranging from 0.0 to 1.0. This allows me to directly map the values to the Min and Max values returned by the Controller. It can also be easily converted back to the -10 to 10 range for the old Exposure interface.

ISO of 80 was way too dark. From reading online, it looks like this was probably supposed to be 800 and not 80 so I changed it to 800. This was probably never caught because I don't think ISO was actually ever successfully set before. ISO Control can only be adjusted while streaming.

Upped the exposure from 0.17 to 0.2 (equivalent of changing from -7 to -6). This was due to getting ISO working. An ISO of 800 is recommended for indoors, but the exposure at this ISO was a little too dark. This new default looks good on HoloLens 2.

Added Min and Max values for Exposure, ISO and White Balance to the Example scene.

Converted voice commands in the Example scene to constants so they couldn't get out of sync.

Added voice commands to the Example scene to allow for adjusting ISO.

This is awesome work, and fantastic documentation! Thanks so much for doing this :)
I'll try and put in some time to check corners like AR Foundation, and put this all into a release.


8.8 alpha breakage

Input:
{-# LANGUAGE DataKinds #-}
{-# LANGUAGE FlexibleInstances #-}
{-# LANGUAGE TemplateHaskell #-}
{-# LANGUAGE TypeFamilies #-}

module Test where

import Generics.Deriving.TH (deriveAll0)

data family SomeType a b c
data instance SomeType c () a = Nullary
                              | Product String (Maybe Char) a

$(deriveAll0 'Nullary)

This works fine on 8.6, but with the 8.8 alpha this is the result:
Test.hs:15:3: error:
    The exact Name ‘a_axQ’ is not in scope
      Probable cause: you used a unique Template Haskell name (NameU),
      perhaps via newName, but did not bind it
      If that's it, then -ddump-splices might be useful
   |
15 | $(deriveAll0 'Nullary)

If I comment out the Product constructor, there's a different error - I don't have any idea at all if that's the same bug or if it's a second:
Test.hs:14:3: error:
    • Expecting one more argument to ‘SomeType (c_axO :: ghc-prim-0.5.3:GHC.Types.Type) ()’
      Expected a type, but
      ‘SomeType (c_axO :: ghc-prim-0.5.3:GHC.Types.Type) ()’ has kind
      ‘* -> *’
    • In the first argument of ‘GHC.Generics.Generic’, namely
        ‘(SomeType (c_axO :: ghc-prim-0.5.3:GHC.Types.Type) () :: ghc-prim-0.5.3:GHC.Types.Type)’
      In the instance declaration for
        ‘GHC.Generics.Generic (SomeType (c_axO :: ghc-prim-0.5.3:GHC.Types.Type) () :: ghc-prim-0.5.3:GHC.Types.Type)’
   |
14 | $(deriveAll0 'Nullary)

This was turned up while checking out aeson's 8.8 readiness; it's cut down from one of the test files.

Yikes, thanks for the bug report! It turns out that this is actually caused by a th-abstraction bug, about which I've opened a ticket here. I'll get out a new release of th-abstraction soon that fixes that bug, and after that I'll double-check to make sure that this bug has disappeared on the generic-deriving side.

I can confirm that this is fixed with th-abstraction-<IP_ADDRESS>.


🐞 - TuiHideSelectedPipe does not work with ViewEngine angular app

Which @taiga-ui/* package(s) are the source of the bug?
kit
Please provide a link to a minimal reproduction of the bug
https://stackblitz.com/edit/dvavrk--run?file=src/app/app.component.ts
Description
Providing a component ref to TuiHideSilectedPipe does not work and IDE shows the error: "invalid numbers of arguments".
Error in console:

ERROR
Error: StaticInjectorError(AppModule)[TuiHideSelectedPipe -> TuiMultiSelectComponent]:
StaticInjectorError(Platform: core)[TuiHideSelectedPipe -> TuiMultiSelectComponent]:
NullInjectorError: No provider for TuiMultiSelectComponent!

Angular version
10.2.3
Taiga UI version
2.28.0
Which browsers have you used?

[X] Chrome
[ ] Firefox
[ ] Safari
[ ] Edge

Which operating systems have you used?

[X] macOS
[ ] Windows
[ ] Linux
[ ] iOS
[ ] Android

It's impossible to fix for view engine and view engine is obsolete so we'll keep it as is. We have several pipes in Taiga that rely on dependency injection and this has only became possible with ivy

ok, thx


functionapp: add Github integration in devops-build commands

Github Functionapp Repository Integration

Separate Github repository flow from Azure DevOps repository flow
Add Github personal access token authenticated client for Github V3 API request (see azure-functions-devops-build)
Add a feature to check host.json and local.settings.json files in a Github repository
Add a feature to commit azure-pipelines.yml to a Github repository
Allow Azure DevOps to create a Github service endpoint against a Github repository
Allow Github to create a push webhook to Azure DevOps pipelines

Unittests

Add test_devops_build_commands_thru_mock.py for testing Azure DevOps / Github workflow before azure-pipelines.yml creation
Test command parameters handling and resource selection

Miscellaneous

Upgrade azure-functions-devops-build to 0.0.15
Fix all related linting error in test_devops_build_commands_thru_mock.py, azure_devops_build_iteractive.py and azure_devops_build_provider.py

This checklist is used to make sure that common guidelines for a pull request are followed.

[X] The PR has modified HISTORY.rst describing any customer-facing, functional changes. Note that this does not include changes only to help content. (see Modifying change log).

[X] I adhere to the Command Guidelines.

cc: @anirudhgarg @dariagrigoriu @Francisco-Gamino
to: @ColbyTresness once the error messages are rephrase, I will fix and commit them under this pull request.

I have created: https://aka.ms/azure-devops-source-repos

I have created: https://aka.ms/azure-devops-source-repos

@ahmedelnably Is it possible to redirect the link to this Github section?
https://docs.microsoft.com/en-us/azure/devops/pipelines/repos/github?view=azure-devops#repository-permissions-for-personal-access-token-pat-authentication

I try to concatenate the fwlink with query parameters but it doesn't seem to work.
https://aka.ms/azure-devops-source-repos/github?view=azure-devops#repository-permissions-for-personal-access-token-pat-authentication

@Hazhzeng url updated

Hi @yugangw-msft,
There is a new pull request for the command az functionapps devops-build create.
In this pull request, I mainly add a support for integrating Functionapp on Github to Azure DevOps.
Besides, I also add a support for Powershell Functionapp.
Please take a look and we want it to be merged in sprint 59.
Much appreciated🙂
cc @anirudhgarg @ahmedelnably @ColbyTresness

cc @eamonoreilly for PowerShell support

@yugangw-msft The issue mentioned has been fixed. Please check, thanks.


Nested jwt_claims

is there a way to return extra info along with a jwt?
For instance, in my login rpc I want to return a display name and a jwt with the email and user role encoded, given an email and password. Tried returning a table with a nested basic_auth.jwt_claims, but postgrest doesn't seem to recognize the jwt_claims and encode it as a jwt unless it's the outermost type in the plpgsql return type.

here's a slice of the pertinent code
create type basic_auth.jwt_claims AS (role name, selected_email text);
-- Finally add a public function people can use to sign up.
-- You can hard code a default db role in it.
-- It alters the underlying basic_auth.users so you can set whatever role you want without restriction.
create or replace function
register(display_name text, email text, pass text) returns basic_auth.jwt_claims
language plpgsql
as $$
declare
result basic_auth.jwt_claims;
begin
insert into basic_auth.users (display_name, email, pass, role) values
(register.display_name::text, register.email::text, register.pass::text, 'customer')
returning
basic_auth.users.role,
register.email as selected_email into result;
return result;
end;
$$;
create or replace function
get_login_token(email text, pass text) returns basic_auth.jwt_claims
language plpgsql
as $$
declare
_role name;
result basic_auth.jwt_claims;
begin
select basic_auth.user_role(email, pass) into _role;
if _role is null then
raise invalid_password using message = 'invalid user or password';
end if;
select _role as role,
get_login_token.email as selected_email into result;
return result;
end;
$$;
CREATE TYPE login_auth_type as (token basic_auth.jwt_claims, display_name text);
-- And now the login function:
create or replace function
login(email text, pass text) returns table (token basic_auth.jwt_claims, display_name text)
language plpgsql
as $$
declare
token basic_auth.jwt_claims;
result login_auth_type;
_display_name text;
begin
select display_name from basic_auth.users where basic_auth.users.email=login.email into _display_name;
-- TODO; check verified flag if you care whether users
-- have validated their emails
select get_login_token(login.email, login.pass) into token;
select token, _display_name as display_name into result;
return result;
end;
$$;
and output:
bash-3.2$ POST /rpc/register '{  "display_name": "bobby", "email"<EMAIL_ADDRESS>"pass" : "abc123" }'
{"token":"eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJzZWxlY3RlZF9lbWFpbCI6ImRhcmtsb3JkdmFkZXJtb3J0QGdtYWlsLmNvbSIsInJvbGUiOiJjdXN0b21lciJ9.PCeh2zZxwx00gl6zVcn7ozfmAwN4sptPKxsyWifIByo"}
bash-3.2$ POST /rpc/login '{  "email"<EMAIL_ADDRESS>"pass" : "abc123" }'
[{"display_name":"bobby","token":{"selected_email":null,"role":"(customer,abc@gmail.com)"}}]

@NotBrianZach can you edit your last comment and format the code? it's very hard to read

@NotBrianZach not sure if you're still interested in resolving this issue, but here's the way it works. If you make a custom type whose name ends in jwt_claims such as foo_jwt_claims then postgrest will turn the row to JSON and then sign it as JWT.


keybase chat list-members responds "incorrect usage" to everything?

$ keybase chat list-members redacted-team-name
Error parsing command line arguments: Incorrect usage.

$ keybase chat list-members redacted-team-name "#general"
Error parsing command line arguments: Incorrect usage.

The documentation says to invoke it as:
keybase chat list-members [command options] [conversation [channel name]]

and what I'm doing seems to match that.
There doesn't seem to be a corresponding chat API method to use instead, either, although one can fake it (awkwardly) by calling read to get the complete chat history, searching for the most recent addedtoteam system message and then taking the union of the owners, admins, readers, and writers fields from that message.

cc @mmaxim

Works for me. Drop the # and it should work

Aah, thanks!

There is probably some room for improving the documentation and/or consistifying the UI here, though; in particular, chat read expects (but does not require) --channel=foo and will accept the channel name with or without the leading #, while chat list-members requires the channel, as a positional argument, and does not accept channel names with #.


NTR Mochimganine

cc @Valwood
Hello, I am a curator from Pombase. I would like to request a new term for Mochimganine, a compound derived from Tschimganine (CHEBI:142517).
Referenced in this paper: PMID:35416247
A drawing of the molecule below:

I think I searched for the correct structure in chebi below and I could not find it:

  Ketcher<PHONE_NUMBER>2D 1   1.00000     0.00000     0

 18 19  0     0  0            999 V2000
   -1.3000    3.5250    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
   -0.4340    3.0250    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
   -0.4340    2.0250    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
   -1.3000    1.5250    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
   -2.1660    2.0250    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
   -2.1660    3.0250    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
    0.4320    3.5250    0.0000 O   0  0  0  0  0  0  0  0  0  0  0  0
    1.2981    3.0250    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
    2.1641    3.5250    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
    1.2981    2.0250    0.0000 O   0  0  0  0  0  0  0  0  0  0  0  0
    2.1641    4.5250    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
    3.0301    5.0250    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
    3.8962    4.5250    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
    3.8962    3.5250    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
    3.0301    3.0250    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
    4.8621    4.7838    0.0000 O   0  0  0  0  0  0  0  0  0  0  0  0
    4.6033    2.8179    0.0000 O   0  0  0  0  0  0  0  0  0  0  0  0
    4.3444    1.8520    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
  1  2  1  0     0  0
  2  3  1  0     0  0
  3  4  1  0     0  0
  4  5  1  0     0  0
  5  6  1  0     0  0
  6  1  1  0     0  0
  2  7  1  0     0  0
  7  8  1  0     0  0
  8  9  1  0     0  0
  8 10  2  0     0  0
  9 11  1  0     0  0
 11 12  2  0     0  0
 12 13  1  0     0  0
 13 14  2  0     0  0
 14 15  1  0     0  0
 15  9  2  0     0  0
 13 16  1  0     0  0
 14 17  1  0     0  0
 17 18  1  0     0  0
M  END

A ChEBI entry has now been created for Mochimganine (CHEBI:192760).
https://www.ebi.ac.uk/chebi/searchId.do?chebiId=192760

You can always submit compounds to the ChEBI database if they are not already found in ChEBI using our submissions tool: https://www.ebi.ac.uk/chebi/submission/login

Thanks @amalik01, I will use the submission tool next time.


[release-2.10] Delete cluster-specific broker resources on add-on removal

Manual backport of https://github.com/stolostron/submariner-addon/pull/1456

/override "Red Hat Konflux / submariner-addon-acm-210-on-pull-request"

/cc @vthapar

/override "Red Hat Konflux / submariner-addon-acm-210-on-pull-request"

/override "Red Hat Konflux / submariner-addon-acm-210-on-pull-request"

/retest

/lgtm

/approve


Embedding from raw text/json

Hi! Thanks for building this :)
I see you support embeddings like Websites and PDF which require extra processing to convert into a digestible format, so I'm wondering why the simplest of embedding (aka raw text) is not supported? Or am I missing something?
Thanks,
David

Hi, I 100% agree. I added the raw text feature in python. Thanks for pointing out that it's missing from the js implementation. I'm ready to add it once #7 is merged.


Newlines aren't rendered inside textarea on first click only

I'm still having this problem (#48).
I'm using Rails 3 and jquery-editable-poshytip.js v1.5.1
When I load the page I see newlines:

This is the markup at this point:
Line 1

Line 2

Line 3

Line 4
As you can see, there are
 there
But When I click on x-editable link, I don't see them inside the textarea:

If I manually insert a newline and save, the next time I open x-editable (if I don't reload the page) it will be correct:

This happens because of instead of
, it created enters ... this is the markup:
Line 1Line 2

Line 3Line 4

(as you can see, no
)
But interestnly, it saved correctly to the database, because after a page reload I get this:

As you can see, the
 are back there:
Line 1Line 2

Line 3Line 4
But, again, if after reloading the page I click on it ... back to square 0:

Same probleme here, ther is a solution by force \n from js ?


Error when using aks show command on aks-preview

Describe the bug
Command Name
az aks show Extension Name: aks-preview. Version: 0.5.94.
Errors:
The command failed with an unexpected error. Here is the traceback:
cannot import name 'case_insensitive_dict' from 'azure.core.utils' (/usr/local/Cellar/azure-cli/2.37.0/libexec/lib/python3.10/site-packages/azure/core/utils/__init__.py)
Traceback (most recent call last):
  File "/usr/local/Cellar/azure-cli/2.37.0/libexec/lib/python3.10/site-packages/knack/cli.py", line 231, in invoke
    cmd_result = self.invocation.execute(args)
  File "/usr/local/Cellar/azure-cli/2.37.0/libexec/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 663, in execute
    raise ex
  File "/usr/local/Cellar/azure-cli/2.37.0/libexec/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 726, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
  File "/usr/local/Cellar/azure-cli/2.37.0/libexec/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 697, in _run_job
    result = cmd_copy(params)
  File "/usr/local/Cellar/azure-cli/2.37.0/libexec/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 333, in __call__
    return self.handler(*args, **kwargs)
  File "/usr/local/Cellar/azure-cli/2.37.0/libexec/lib/python3.10/site-packages/azure/cli/core/commands/command_operation.py", line 352, in handler
    client = self.client_factory(self.cli_ctx) if self.client_factory else None
  File "/Users/jmckearin/.azure/cliextensions/aks-preview/azext_aks_preview/_client_factory.py", line 31, in cf_managed_clusters
    return get_container_service_client(cli_ctx).managed_clusters
  File "/Users/jmckearin/.azure/cliextensions/aks-preview/azext_aks_preview/vendored_sdks/azure_mgmt_preview_aks/_container_service_client.py", line 642, in managed_clusters
    from .v2022_07_02_preview.operations import ManagedClustersOperations as OperationClass
  File "/Users/jmckearin/.azure/cliextensions/aks-preview/azext_aks_preview/vendored_sdks/azure_mgmt_preview_aks/v2022_07_02_preview/__init__.py", line 9, in <module>
    from ._container_service_client import ContainerServiceClient
  File "/Users/jmckearin/.azure/cliextensions/aks-preview/azext_aks_preview/vendored_sdks/azure_mgmt_preview_aks/v2022_07_02_preview/_container_service_client.py", line 19, in <module>
    from .operations import AgentPoolsOperations, FleetMembersOperations, FleetsOperations, MaintenanceConfigurationsOperations, ManagedClusterSnapshotsOperations, ManagedClustersOperations, Operations, PrivateEndpointConnectionsOperations, PrivateLinkResourcesOperations, ResolvePrivateLinkServiceIdOperations, SnapshotsOperations, TrustedAccessRoleBindingsOperations, TrustedAccessRolesOperations
  File "/Users/jmckearin/.azure/cliextensions/aks-preview/azext_aks_preview/vendored_sdks/azure_mgmt_preview_aks/v2022_07_02_preview/operations/__init__.py", line 9, in <module>
    from ._operations import Operations
  File "/Users/jmckearin/.azure/cliextensions/aks-preview/azext_aks_preview/vendored_sdks/azure_mgmt_preview_aks/v2022_07_02_preview/operations/_operations.py", line 19, in <module>
    from azure.core.utils import case_insensitive_dict
ImportError: cannot import name 'case_insensitive_dict' from 'azure.core.utils' (/usr/local/Cellar/azure-cli/2.37.0/libexec/lib/python3.10/site-packages/azure/core/utils/__init__.py)

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

az aks show -g {} -n {} -o table

Expected Behavior
Expected to see a table output of the current aks cluster status
Environment Summary
macOS-12.5-x86_64-i386-64bit, Darwin 21.6.0
Python 3.10.4
Installer: HOMEBREW

azure-cli 2.37.0 *

Extensions:
aks-preview 0.5.94

Dependencies:
msal 1.18.0b1
azure-mgmt-resource 21.1.0b1

Additional Context
I also ran into the same issue when running az aks kollect

route to CXP team

@jpmckearin Thank you for reaching out, we are looking into it.

@jpmckearin We are unable to reproduce this issue at our end.

Can you please try upgrading to the latest version of Az CLI 2.39.0


Add holidays

I know that holidays varies by country, state, city, province, etc
I think would be good have a way to set the holidays, with a array of month/date
For instance with the US federal holidays would be something like this moment.setHolidays(['January 01', 'January 18', 'February 15', 'July 04', 'May 30', 'September 5', 'October 10', 'November 11', 'November 26', 'December 26'])
For NY we would push some more days to the list: moment.addHolidays(['February 12', 'November 08', 'December 25'])
Them when we use methods of moment-bussiness it consider these days not as weekdays, and not as weekendDays.
We can also create methods to count holidays.
What do you think?

Makes sense to me. If you've got time to open a PR, I would review/merge it :v: But I won't have time to add the feature myself

@jmeas anybody ever take a stab at a PR for this?

Not yet 🙂

Is this alive ? If I created a PR, would there any time to review it?


IMS 1.1 - Tasks reorg and new features

This PR includes many changes:

Fix virt-v2v-wrapper.py call to _is_iso_domain: missing _ for the private method
Add configuration of SSH transport: install private keys for VMware hosts and set .ssh/config
Add configuration of ca-trust for providers certificates to avoid using insecure connection
Add creation of ManageIQ conversion host record and tagging of associated resource
Reorganize tasks: split in separate files, rename files, etc...
Allow to not configure VDDK if vddk transport method is not set for conversion host
Remove nbdkit-* uninstallation as it also uninstalls vdsm

Hello contributor, thanks for submitting a PR for this project!
I am the bot who triggers "standard-CI" builds for this project.
As a security measure, I will not run automated tests on PRs that are not from white-listed contributors.
In order to allow automated tests to run, please ask one of the project maintainers to review the code and then do one of the following:

Type ci test please on this PR to trigger automated tests for it.
Type ci add to whitelist on this PR to trigger automated tests for it and also add you to the contributor white-list so that your future PRs will be tested automatically.
If you are planning to contribute to more than one project, maybe it's better to ask them to add you to the project organization, so you'll be able to run tests for all the organization's projects.

Moving back to WIP to split it in smaller chunks. Will be closed once done.

https://github.com/oVirt/ovirt-ansible-v2v-conversion-host/pull/15

Merging the PR. I will drop the SSH key removal in next commit.


Creating a AWS connection fails and the UI stuck on the spinner forever

Environment info

Version: 2.3.1-73a7583
Deployment: AZURE
Customer: QA

Actual behavior

Creating a AWS connection fails and the UI stuck on the spinner forever

it doesn't happens every time retrying in a new tab worked.

Expected behavior

We should get a proper error and we should not get stuck.

Screenshots or Logs or other output that would be helpful
nbfedump.2018_04_22-11_02_13.json.zip

As we are not investing in the UI console, closing


Does not Reproducibly Build, uses DATE macro

https://wiki.debian.org/ReproducibleBuilds/TimestampsFromCPPMacros
There is __DATE__ macro used in the code, which means it's impossible to reproduce the package build (ie. independently build the package again, and get identical binaries out with identical checksum hash).
Please consider removing __DATE__ macro, and instead e.g. use a full Version number, possibly with e.g. packaging version number that is with Debian Revision / RPM Revision&tag instead. That would still uniquely identify the build, but can be re-built without changing the output.

For example, use GIT_VERSION instead of __DATE__

https://github.com/ibm-genwqe/genwqe-user/commit/d31629f94f64b6ff746ac7ad71496ab350ff085e should fix that. I am using GIT_VERSION as you proposed.


Schema for parametric objects (circle, ellipse, etc.)

Define a schema for parametric objects such as circles and ellipses. Circles, for example, may be defined using an (x, y) centroid and radius or by three points touching the circle's exterior. The latter is the WKT approach.
Once the schema is defined, it will need to be implemented in the loads/dumps paradigm for the reference implementation.

If we did want to include parametric shapes, I would go with the WKT approach as I think that's how software packages which do support curves, such as SQL Spatial and ArcGIS, store the shapes internally based on looking at SQL Spatial docs and doing some simple editing of circles in ArcGIS 10.3.1. This also works well with the scheme you've already adopted for specifying parts and holes within the array. We could add flag constants indicating, e.g., circularstring, when those geometries occur within a compoundcurve.  If we went with parameters like ellipse major radius and minor radius, it'd be a further reach from our current scheme, at a minimum tacking those parameters into the array values after the "ellipse" flag.
But, some geospatial libraries and specs like shapely (fed it some WKT curve examples and got errors), SpatiaLite, shapefile and GeoJSON don't support parametric shapes as far as I can tell. This becomes problematic since we use shapely in the reference implementation and testing. If we wanted to support curves, we may need to drop shapely and just load from netCDF into WKT.  (Maybe we should be going to WKT anyway in case we add curves later?)
Thus, I lean toward only including (multi)point/linestring/polygon for now.  But I'd like to understand what use cases we can't support if we do this. I live in (NHD) river and watershed world so simple geometry is fine for me. Does simple geometry break anyone else's stuff?

I've been thinking about this project this week.
Your comments are a good lead in to what I've been thinking about.
It seems that our solution here is to use an indexed array as a way of
adding support for a ragged array so that we can efficiently store Simple
Geometries in our own encoding of WKT in .nc3 files, since .nc3 files don't
support ragged arrays. That's fine. And the problem goes away in .nc4
because ragged arrays are available.
Similarly, it seems like Section 9 of CF (Discrete Sampling Geometries) is
all about how to add support for ragged arrays in .nc3 files so that we can
efficiently store ragged array DSG data in .nc3 files, since .nc3 files
don't support ragged arrays.  That's fine. And the problem goes away in
.nc4 because ragged arrays are available.
It seems like there is a general problem: people want to use ragged arrays
for various data structures. .nc4 has support for ragged arrays. .nc3
doesn't. We seem to be adding support for ragged arrays on a case-by-case
basis, and using slightly different systems.
Shouldn't we break the Simple Geometry problem/proposal into two parts:

Make a proposal for a general purpose system for ragged arrays in .nc3
files.
Use cases: in addition to DSG and Simple Geometries, the most obvious
use case is Strings.
It would allow CF to add support for variable-length Strings.
This would be a huge bridge between .nc3 and .nc4, allowing .nc3 to
gain one of
the big features of .nc4.
My first pass proposal is: add support for two alternative systems:

A generalized version of attributes and indexes as currently used by
DSG's Contiguous Ragged Array system.
A generalized version of attributes and indexes as currently used by
DSG's Indexed Array system.

Make a proposal for storing our encoding of WKT in one of these new,
standardized ragged arrays (or WKT/B itself as a variable length String).
I have more to say about this, but you get the basic idea.

Thoughts?
Love it? Hate it?
On Fri, Jun 3, 2016 at 2:01 PM, Tim Whiteaker<EMAIL_ADDRESS>wrote:

If we did want to include parametric shapes, I would go with the WKT
approach as I think that's how software packages which do support curves,
such as SQL Spatial and ArcGIS, store the shapes internally based on
looking at SQL Spatial docs
https://msdn.microsoft.com/en-us/library/bb964711.aspx and doing some
simple editing of circles in ArcGIS 10.3.1. This also works well with the
scheme you've already adopted for specifying parts and holes within the
array. We could add flag constants indicating, e.g., circularstring, when
those geometries occur within a compoundcurve. If we went with parameters
like ellipse major radius and minor radius, it'd be a further reach from
our current scheme, at a minimum tacking those parameters into the array
values after the "ellipse" flag.
But, some geospatial libraries and specs like shapely (fed it some WKT
curve examples and got errors), SpatiaLite
http://www.gaia-gis.it/gaia-sins/spatialite-manual-2.3.1.html#t3,
shapefile https://en.wikipedia.org/wiki/Shapefile#Spatial_representation
and GeoJSON http://geojson.org/geojson-spec.html#geometry-objects don't
support parametric shapes as far as I can tell. This becomes problematic
since we use shapely in the reference implementation and testing. If we
wanted to support curves, we may need to drop shapely and just load from
netCDF into WKT. (Maybe we should be going to WKT anyway in case we add
curves later?)
Thus, I lean toward only including (multi)point/linestring/polygon for
now. But I'd like to understand what use cases we can't support if we do
this. I live in (NHD) river and watershed world so simple geometry is fine
for me. Does simple geometry break anyone else's stuff?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/bekozi/netCDF-CF-simple-geometry/issues/8#issuecomment-223693556,
or mute the thread
https://github.com/notifications/unsubscribe/ABarOHnLN_xET6i6wisOJ7ruZKdr2w9Vks5qIJYwgaJpZM4Ir55f
.

--
Sincerely,
Bob Simons
IT Specialist
Environmental Research Division
NOAA Southwest Fisheries Science Center
99 Pacific St., Suite 255A      (New!)
Monterey, CA 93940               (New!)
Phone:<PHONE_NUMBER>            (New!)
Fax:<PHONE_NUMBER>
Email<EMAIL_ADDRESS>The contents of this message are mine personally and
do not necessarily reflect any position of the
Government or the National Oceanic and Atmospheric Administration.
<>< <>< <>< <>< <>< <>< <>< <>< <><

If we did want to include parametric shapes, I would go with the WKT approach as I think that's how software packages which do support curves, such as SQL Spatial and ArcGIS, store the shapes internally based on looking at SQL Spatial docs and doing some simple editing of circles in ArcGIS 10.3.1.

I’m thinking we may need to diverge slightly from WKT or support both cases. I don’t think the latter would be too difficult. I suspect a lot of circles and/or ellipses could be defined with a centroid and radius/axis parameters more easily than with points touching the exterior. I sense people may not use it unless both paths were available. In the spirit of OGC-compliance and our alpha reference implementation, supporting WKT makes sense. It would be good to get input from @MarkHedley (hope I have the right handle) on this as he seemed to have the most immediate use cases. I generally hear about remote sensor footprints. I also suspect effective radius of observing stations would be another.

This also works well with the scheme you've already adopted for specifying parts and holes within the array. We could add flag constants indicating, e.g., circularstring, when those geometries occur within a compoundcurve. If we went with parameters like ellipse major radius and minor radius, it'd be a further reach from our current scheme, at a minimum tacking those parameters into the array values after the "ellipse" flag.

I agree the WKT approach is more compatible. Having indirection for different geometry types will add complexity, but I think that is okay. Overloading the loads method with different argument schemes shouldn’t be difficult.

But, some geospatial libraries and specs like shapely (fed it some WKT curve examples and got errors), SpatiaLite, shapefile and GeoJSON don't support parametric shapes as far as I can tell. This becomes problematic since we use shapely in the reference implementation and testing. If we wanted to support curves, we may need to drop shapely and just load from netCDF into WKT. (Maybe we should be going to WKT anyway in case we add curves later?)

Good point. Shapely was low-hanging fruit. I am inclined to stick with something like Shapely because geometries can get quite complex. String parsing/formatting for a 50,000 node polygon seems inefficient, and I think we should leverage tools that do this already - whatever they may be.
For the reference API, we could have the function return different nested list/tuple forms as opposed to a Shapely geometry object. We could then have something like cf.shapely.loads which all pass through cf.loads at some point.
Ellipses are off the radar as far as I know. I suspect most practitioners would be doing there own math with these initially. I like the parametric definition because it is clean and allows clients to use it how they please. These shapes are of course storable as polygons with varying degrees of smoothness depending on the exterior node count. As an aside, Shapely can support support circles by buffering a point similar to: http://stackoverflow.com/questions/4985222/geodjango-how-to-create-a-circle-based-on-point-and-radius.

Thus, I lean toward only including (multi)point/linestring/polygon for now. But I'd like to understand what use cases we can't support if we do this. I live in (NHD) river and watershed world so simple geometry is fine for me. Does simple geometry break anyone else's stuff?

I agree this should be the initial focus. My experience working with these data in netCDF has also been with NHD watersheds. I firmly believe we’ll need to solicit outside opinions on parametric shapes before charging ahead too much. Provided we support WKT/WKT/OGC nothing should break...right?

@BobSimons I moved this comment over to https://github.com/bekozi/netCDF-CF-simple-geometry/issues/11 to keep this thread more-or-less parametric object related. Hope that's okay...

My understanding was that we were not going to attempt to handle parametric shapes for now... We'd need someone with a common use case to justify it at this point IMHO.

For the reference API, we could have the function return different nested list/tuple forms as opposed to a Shapely geometry object. We could then have something like cf.shapely.loads which all pass through cf.loads at some point.

I like this.  It enables for a more robust reference implementation while still demonstrating a shapely workflow.
But yeah, focus on straight line segments first.

I added a ticket for modifying the Python function to return nested lists and tuples (#15).

It would be good to get input from @MarkHedley

@marqh is the magic handle
to be continued...

Oh hi, @marqh. We were looking for you! :smile:

I just had a conversation with @marqh and we found some nice synergies in what we are all working on. Now if I can find time to read through all the activity here!

That's great :+1:. I hope we can get @marqh's input on what we're doing here.

in terms of the cases we want to support, circles and ellipses go a long way to delivering our needs.
I have not seen the '3 points touching the edge' approach before.  The approach I would expect to see would involve the definition of a node as the centre, then teh definition of an edge, based on a radius or semi-major and semi-minor axis

@marqh I agree with the center and radius/axis approach. I think the nominal geometry types should be 'ellipse' and 'circle'. The variable itself would actually be of 'point' geometry type (to hold the center coordinate) with attributes radius or semi_major_axis and semi_minor_axis that point to the variables holding those parameter values. Does this make sense? Anything missing?
I don't see any reason to support things like "MultiCircle".

You'd also need a variable to store the angle between the x-axis and the major axis of the ellipse.
I think this approach would be fine if we were designing only for ellipses. But since we're looking at many different geometry types, I like the '3 points touching edge' approach better as it is a general solution that could support ellipses with the addition of a single flag that says, "I'm an ellipse". Consider if we decide to add arcs later.  With the 'center and axes' approach you'd need variables for start and stop angle in addition to the axis variables for circles or ellipses. With the '3 points' approach, you just need a flag that says "I'm an arc". In other words, instead of inventing a solution for each type of parametric shape, I'm suggesting we just add a flag indicating what the shape is. Eventually the set of possible flag values could correspond roughly with WKT geometry types.  (I need to spend time with the reference implementation again to see if this makes sense...probably after July 14.)
Regardless of the implementation, you can mathematically derive one from the other, so software should be able to present to the user the parameters of the shape if that's what the user prefers.

You'd also need a variable to store the angle between the x-axis and the major axis of the ellipse.

Okay.

With the '3 points' approach, you just need a flag that says "I'm an arc". In other words, instead of inventing a solution for each type of parametric shape, I'm suggesting we just add a flag indicating what the shape is.

I think we should support both approaches (i.e. exterior points and parameters). This will add indirection by geometry type but it shouldn't be hard to support. With a circle, we can use a buffer to create the geometry object. An ellipse is a bit more complex.
I would also like to align as closely with WKT representations as possible, but I expect we may have to support a couple options here. I'm inclined to finish the draft specification and reference API for non-parametric shapes and identify how parametric objects fit in. I view circles and ellipses as augmented points for the parameterized approach and first-class coordinate index variables in the WKT approach.
Question: Using the exterior approach for ellipses seems more complex. Wouldn't you have to select exterior points carefully to make sure the representative axes could be calculated? An exception for ellipses seems inevitable.

Regardless of the implementation, you can mathematically derive one from the other, so software should be able to present to the user the parameters of the shape if that's what the user prefers.

Good point.

For ellipses, I understand the need to have orientation, and that this is not provided by a centre, semi-major axis semi-minor axis definition.
I can see how storing the centre point, one semi-major location and one semi-minor location enables us to explicitly define any ellipse.
I don't see how I define an ellipse based on 3 points which exist on the ellipse.
Is there any reference material I can read up on to understand this approach?
How does one choose where the three points are?
If we can deliver a solution which is consistent with WKT and is able to be used for curves, that's great.
I think that it would be reasonable to consider dropping the centre, semi-major length, semi-minor length all together, if there is a better way
cheers
m

For ellipses, I understand the need to have orientation, and that this is not provided by a centre, semi-major axis semi-minor axis definition.

I think this is similar to relative angle between the x-axis and major axis. Regardless, I had overlooked this. :smile:

I don't see how I define an ellipse based on 3 points which exist on the ellipse.
Is there any reference material I can read up on to understand this approach?
How does one choose where the three points are?

I'm not sure the three exterior points is a good approach for ellipses. We would have to provide the math to go between exterior points and parametric representations anyway, so I'm not sure it's less work than providing "special case" specifications for circles/ellipses.

If we can deliver a solution which is consistent with WKT and is able to be used for curves, that's great.

I think we should make a decision on incorporating curves. I'll create a ticket.

I don't see how I define an ellipse based on 3 points which exist on the ellipse.

This article shows that you can't with just three points.  Looks like it takes five. For some reason I had the notion three would be enough. Thanks for questioning this.

Circles are done in WKT as a circularstring with five points.
CIRCULARSTRING(4 1, 7 4, 4 7, 1 4, 4 1)
Ellipses are not supported in WKT as far as I know. You could invent an ellipse flag and supply five points.
ELLIPSE(1 9, 1 5, 5 9, 7 8, 7 4)
This follows the WKT pattern of supplying points along the geometry.  This article shows how to get the ellipse equation parameters from five points.
But if you really want a parametric alternative, I guess first you must decide which version of the parameters you are after.  For ellipses, does this mean supplying center, axes, and orientation, or does it mean supplying A through F in the general equation of a conic section: Ax2 + Bxy + Cy2 + Dx + Ey + F = 0.  Assuming the former, then I like what marqh mentioned about defining the center point and one point where each axis intersects the ellipse, e.g., for an ellipse centered at (0 0) with axis lengths of 2 and 4:
ELLIPSE(0 0, 1 0, 0 2)
...though that's not convenient if users want to enter center point, axis length, and orientation angle.  You may also want an indicator of CW vs CCW node orientation in case the ellipse is used as a hole and software cares about node orientation and must discretize ellipses.
But I still wonder if netCDF users need ellipses and if they must work with them in a parametric way.

I think supplying the simplest method for parametric objects is ideal. I'm not sure if the WKT method or parameters-only method is simpler in the end. Is it easier to deviate from WKT/OGC? I think, yes, as we are trying to describe a scientific data format and the WKT method seems a little off the mark by adding complexity. This is just an opinion!

But I still wonder if netCDF users need ellipses and if they must work with them in a parametric way.

I am aware of folks in the data assimilation community that are moving towards ellipses for remote sensor footprints. I do not know of any netCDF data that actually stores them, however.
From the discussion, I am not seeing anything that would break our current approach. Perhaps, these really should be part of the "0.2" specification once we can solicit feedback.


setting variables from secret variables doesn't fit the current yaml schema

Following the code examples in section https://docs.microsoft.com/en-us/azure/devops/pipelines/process/variables?view=azure-devops&tabs=yaml%2Cbatch#secret-variables "Secret Variables" does not produce working parseable yaml.
The editor and the build process return either "first property must be name" or "Unexpected value" for the line after "variables:" in the yaml.

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

ID: dd7e0bd3-1f7d-d7b6-cc72-5ef63c31b46a
Version Independent ID: dae87abd-b73d-9120-bcdb-6097d4b40f2a
Content: Define variables - Azure Pipelines
Content Source: docs/pipelines/process/variables.md
Product: devops
Technology: devops-cicd-process
GitHub Login: @juliakm
Microsoft Alias: jukullam

@JasonRitchie For the YAML to work, you need to have the secret variable mySecret defined (using the UI).


Improper animations in the about file

If you check the About page, the VAA description paragraph has the fade right animations while the other two has the fade up animation.
the VAA paragraph animations appears when we scroll down but the VKSSF paragraph doesnt, it doesnt has delay time.

@krishnashenoy123 can you work on this

Also, there are other animation issues on the website

@krishnashenoy123 can you work on this

Yes I'll try to do it 👍


Breaks when parsing function typedefs

I'm trying to create bindings for python APIs and the following causes the bindgen to break:

typedef long int __ssize_t;

typedef __ssize_t cookie_read_function_t(void *__cookie, char *__buf, size_t __nbytes);

typedef __ssize_t cookie_write_function_t(void *__cookie, const char *__buf,
                                          size_t __nbytes);

typedef int cookie_seek_function_t(void *__cookie, __off64_t *__pos, int __w);

typedef int cookie_close_function_t(void *__cookie);

typedef struct _IO_cookie_io_functions_t {
  cookie_read_function_t *read;
  cookie_write_function_t *write;
  cookie_seek_function_t *seek;
  cookie_close_function_t *close;
} cookie_io_functions_t;

I tested this inside an online c compiler and it has no problems

Thanks for reporting. Should be fixed with last commit.


[Bug] [Android] Slider and CarouselView don't work in SideMenuView

Description
Sliders and CarouselViews don't scroll or swipe within a SideMenuView on Android. No issue on iOS. I assume this would also be the case for all controls that use a drag effect?
Setting  xct:SideMenuView.MenuGestureEnabled to false does not fix this either.
Downgrading from latest Xamarin.Forms package to <IP_ADDRESS>9-pre6 gave expected behaviour back.
Steps to Reproduce

Create a SideMenuView
Add a Slider or CarouselView
Try and scroll / swipe
Example xaml below.

 <xct:SideMenuView x:Name="SideMenuView">
        <!-- MainView -->
        <StackLayout
            Orientation="Vertical"
            BackgroundColor="Gold">

            <Button  />

            <Button  />
            <Slider HorizontalOptions="Fill" Minimum="0" Maximum="170" MinimumTrackColor="Red" MaximumTrackColor="Red"/>

        </StackLayout>

        <!-- LeftMenu -->
        <BoxView
            BackgroundColor="Orange"
            xct:SideMenuView.Position="LeftMenu"
            xct:SideMenuView.MenuWidthPercentage=".5" />

        <!-- RightMenu -->
        <BoxView
            BackgroundColor="LightGreen"
            xct:SideMenuView.Position="RightMenu"
            xct:SideMenuView.MenuWidthPercentage=".3" />

    </xct:SideMenuView>

Expected Behavior
Controls to scroll and swipe.
Actual Behavior
Controls do not scroll and swipe.
Basic Information

Version with issue:  Xamarin.Forms <IP_ADDRESS>4
Last known good version:  Xamarin.Forms  <IP_ADDRESS>9-pre6
IDE: Visual Studio Mac
Platform Target Frameworks:

Android: 10

Nuget Packages:

  <PackageReference Include="Xamarin.Forms" Version="<IP_ADDRESS>4" />
  <PackageReference Include="Xamarin.Essentials" Version="1.6.0" />
  <PackageReference Include="Xamarin.CommunityToolkit" Version="1.0.0" />
  <PackageReference Include="Xamarin.Forms.Visual.Material" Version="<IP_ADDRESS>4" />
  <PackageReference Include="Com.Airbnb.Xamarin.Forms.Lottie" Version="4.0.8" />
  <PackageReference Include="Sharpnado.Shadows" Version="1.2.0" />
  <PackageReference Include="OxyPlot.Xamarin.Forms" Version="1.1.0-unstable0011" />
  <PackageReference Include="SkiaSharp.Views.Forms" Version="2.80.3-preview.24" />
  <PackageReference Include="SkiaSharp" Version="2.80.3-preview.24" />

Affected Devices: Android

Workaround
Downgrading from latest Xamarin.Forms package to <IP_ADDRESS>9-pre6 gave expected behaviour back.
Reproduction imagery

Reproduction Link
https://github.com/OwenCross289/XamarinNav/tree/CollectionView
This branch on the renderers view and advertising view show this bug

@PureWeen @jsuarezruiz Hey folks
Is anything related to touch-management changed from <IP_ADDRESS>9-pre6 to <IP_ADDRESS>4 that might cause this issue?

@PureWeen @jsuarezruiz Hey folks
Is anything related to touch-management changed from <IP_ADDRESS>9-pre6 to <IP_ADDRESS>4 that might cause this issue?


add custom class to img element

Hi,
I want to add a class to img element and also change the src to data-src after I got image url from server.
change this

to

Is there any way of doing this with implemented methods?

Something like this should work.
function handleUploadCompleted($el, data) {
	var file = data.result.files[0];
	var $img = $el.find('img');
	$img.attr('class', 'cld-responsive');
	$img.attr('data-src', file.url);
}
$('.mediumEditor').mediumInsert({
	editor: this.medium,
	addons: {
	  images: {
		uploadCompleted: uploadCompleted,
	  }
	}
});

thanks that solved my problem.


Automated cherry pick of #248: add patch rbac for events

Cherry pick of #248 on release-0.2.
#248: add patch rbac for events
For details on the cherry pick process, see the cherry pick requests page.

/lgtm
/approve


Update HelpWindow picture

Codecov Report

Merging #201 (b614e70) into master (d9bc009) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #201   +/-   ##
=========================================
  Coverage     51.26%   51.26%
  Complexity      540      540
=========================================
  Files           110      110
  Lines          2497     2497
  Branches        256      256
=========================================
  Hits           1280     1280
  Misses         1142     1142
  Partials         75       75

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


[Feature] Use SWR caching for transaction list

Feature description
Currently we always load the transaction list when the wallet is shown. On slower wallet connections this makes Alby feel slow.
Describe the solution
We should use the SWR utils that we have to show cached data and the update it.
Describe alternatives
No response
Additional context
No response
Are you working on this?
None

Related? https://github.com/getAlby/lightning-browser-extension/issues/2985


[1.10.2] Issue with Matter Overdrive installed

So basically when both mods are installed, it tries to check everything for matter and it just says 'error mo:matter". There isn't a disable option in matter overdrive or this mod.

What? What is checking that? More info please? Logs? Did you also report this to MO?

Also getting the error. No crash, just odd info. http://imgur.com/9zGPs8j


Enable source maps for debugging

Hi @jerosoler, thx for this library,
When I try to debug the internal properties or methods of the drawflow library at runtime with breakpoints, I have difficulty in the debugging phase because the source map is not enabled. Since the JavaScript output is minified to a single line, I cannot put specific breakpoints in the internal properties and methods.
Can we enable external or inline source maps?

It's just a javascript file, you can try importing it manually.
This file:
https://github.com/jerosoler/Drawflow/blob/master/src/drawflow.js

Yep, I was previously trying to debug the drawflow.js file by including it in the source code, but this solution was not sustainable to me. Because I have to revert this before every commit 😞.
I'm not sure if it makes sense, but;

I think it would be useful to include the external source map(.map) file (via devtool: "source-map" in webpack config) in the npm package if we don't have to worry about the file size of the npm package.

Or can we add a separate file in the dist folder with minimize optimization disabled that contains the inline or external source map, for example dist/drawflow.dev.js.

Or can we disable default webpack minimize optimization for dist/drawflow.min.js
optimization: {
  minimize: false
},

I'll take a look at it when I have time.


make coverage configuration

Some tweaks to reporting coverage

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.9%) to 91.751%

Totals

Change from base Build<PHONE_NUMBER>:
0.9%

Covered Lines:
6362

Relevant Lines:
6934

💛  - Coveralls


Update example

https://github.com/EarthSciML/earthsci.dev/blob/c6bb0da668aa7a8da2aa8e25507282c433404620/tutorials/example.md?plain=1#L11
Update the example above with the code from here: https://github.com/jialinl6/box_model
Also add narrative text to explain what we're doing as we go along through the example.

 Gov4Git notices
On Monday, 26-Feb-24 16:25:24 UTC by Gov4Git dev
Notice 5b33br
This issue's priority score is now 2.000000.
The cost of priority is 4.000000.
The projected bounty is now 4.000000.
Notice hnleuo
The set of eligible proposals claiming this issue is empty.


My new branch...I have added the Hello world Code in Database

I have added the perfect Hello World Code in database and my name (Abhinav Prajapati) already listed in contributors.md.
All the changes are done as stated.

You can checkout our Idea-Arca, the project idea bank for any languages, for open-source contributions.
The PR from this repo is being accepted in hacktoberfest2023
Happy Contributing season!
Keep up the good work!


Add Filament::isServingFilament()

This PR adds a method to the facade that will return a boolean based on whether the ServingFilament event was dispatched earlier in the current lifecycle, meaning that it will track whether we are on a Filament request or not.
if (Filament::isServingFilament()) {
   //
}

It could be that the name Filament::servingFilament(true) is a bit confusing. In that case we could rename the methods to setIsServingFilament()/isServingFilament().

Thanks


In-place editor calls repeat when the modified model is run

For model.edit context, when inplace=True and return_context=True, all method calls of the context object (editor) repeat when the modified model is traced. The only editor method I am aware of is .log() so I tested this behavior on it.
It looks like each call of the model.edit context with editor.log inside it adds the editor.log call with the specified argument to the intervention graph. Then, when the model, that was edited in-place, is traced, it displays all the editor.log calls added inside the model.edit contexts.

Does this behavior not appear if you use nnsight.log()

Isn't this expected behavior since you are the cell multiple times with in_place editing?

Same thing happens with nnsight.log().
My understanding was that the in-place is supposed to only modify the modules of the model whereas right now it looks like it just attaches literally everything within the context to the intervention graph. Is it expected?

Yes, it is expected! It basically defines any default interventions to be ran before the newly define intervention graph in a certain context.

This feels potentially tricky and error-prone as there are so many operations now that are patched in and are a part of the intervention graph - literally every built-in operation, all torch methods, all torch tensor creation operations and einsop functions.
How can we make it more obvious that the .edit() not only modifies the modules of the model but literally remembers everything the user specifies in the context?


Add python 3.6 build to travis

Coverage decreased (-0.02%) to 91.615% when pulling ead5a6a349fb5a576d914fd2b76327571313a819 on iMichka:python3.6 into f872d056f477ed2438cd22b422d60dc924469805 on gccxml:develop.

Coverage decreased (-0.02%) to 91.615% when pulling ead5a6a349fb5a576d914fd2b76327571313a819 on iMichka:python3.6 into f872d056f477ed2438cd22b422d60dc924469805 on gccxml:develop.

Coverage decreased (-0.02%) to 91.615% when pulling ead5a6a349fb5a576d914fd2b76327571313a819 on iMichka:python3.6 into f872d056f477ed2438cd22b422d60dc924469805 on gccxml:develop.

Coverage decreased (-0.02%) to 91.615% when pulling ead5a6a349fb5a576d914fd2b76327571313a819 on iMichka:python3.6 into f872d056f477ed2438cd22b422d60dc924469805 on gccxml:develop.

Coverage decreased (-0.02%) to 91.615% when pulling ead5a6a349fb5a576d914fd2b76327571313a819 on iMichka:python3.6 into f872d056f477ed2438cd22b422d60dc924469805 on gccxml:develop.


Add Data sync pull for elasticsearch 6 to Training Environment

I believe this is needed as part of the Training Environment setup .

I believe ES6 is experimental for the time being and we do not need to sync it to training at this point in time

Ah thanks for the info, I'll delete the branch


Reverse editor layout

I would like to have a feature in VSCode to reverse the editor layout.  The thing is I use a vertical monitor and love to keep the current file(s)/group I'm working on at the bottom same as my eye level while the other files/group is at the top portion which is related to my task. Like JS files are at the bottom and the PHP, API, JSON etc files are at the top.
But sometimes I need to reverse the view and put the JS file on top and PHP, API, JSON file at the bottom same as my eye level.
VSCode has the  "FLIP" option under View > Editor Layout but I don't think "FLIP" is the right wording here as it's actually changing layout order from vertical to horizontal or vise versa.
Anyway, a "reverse" editor layout would be a helpful and nice addition I believe. Thanks

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

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

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


[ERROR] Internal error has occurred in package-naming: java.lang.IndexOutOfBoundsException: Index 0 out of bounds for length 0

In case you will run mvn diktat:check in the root of diktat you will see this issue.
 if (realNameParts.isNotEmpty() && packageNameParts.map { node -> node.text } != realNameParts) {
            val realPackageNameStr = realNameParts.joinToString(PACKAGE_SEPARATOR)
            // issue happens here if there is no package name for the file
            val offset = packageNameParts[0].startOffset
            PACKAGE_NAME_INCORRECT_PATH.warnAndFix(configRules, emitWarn, isFixMode,
                realPackageNameStr, offset, packageNameParts[0]) {
                insertNewPackageName(packageDirective, realPackageNameStr)
            }
        }

Please provide the file where this error happens


Test Scenario Editor: Settings Dock

Creating a new Component that manages the settings data of the scesim file.
Rule and DMN data are different.
DMN

@yesamer hi, but that is expected that DRL and DMN settings are diferent, for example DRL, you want to specify ruleflow group, while for DMN you do not want, because it has not such property. What is this ticket aimed for please?

@jomarko Yes exactely, the component will show different properties based on scesim type (DMN / RULE).


Cannot get buy limit to work

I cannot figure out what I am doing wrong to not be able to do buy_limit, any help is appreciated!
from bittrex import Bittrex
api = Bittrex('0c7axxxxxx197d4ac5aa', 'fce8a5xxxxx245a4d81ac10')
#getting BTCBalance
trade = 'BTC'
pumpcurrency = raw_input('which currency would you like to pump?')
market = '{0}-{1}'.format(trade, pumpcurrency)
BTCbalance = api.get_balance(trade)['result']['Available']
print "Your balance is {0} {1}.".format(BTCbalance, trade)
Getting the BTC price for your pumpable currency
cryptosummary = api.getmarketsummary(market)
currencyprice = cryptosummary['result'][0]['Last']
print 'The price for {0} is {1:.8f} {2}.'.format(pumpcurrency, currencyprice, trade)
Amount of coins to buy
amount = BTCbalance/currencyprice
How big of a profit you want to make
multiplier = 1+ float(raw_input("How much profit would you like to make? (input percentage as integer)"))/100
print ("Selling at "+str(multiplier) +"x profit")
Buying the pump currency for BTC
api.buy_limit(market, amount, currencyprice)
print ('Buying ' +str(amount)+" "+ pumpcurrency + ' at ' + str(currencyprice) + " each.")

There's a lot wrong here...you seem to understand the basic idea of using the ,JSON as a list but lack the proper implementation.
Also you need to examine your usage of quotation marks.

Hi,
Is there any way you could be more specific with some examples?
Best,
Ryszard

From: Munkeynutz<EMAIL_ADDRESS>Sent: July 27, 2017 8:16 PM
To: ericsomdahl/python-bittrex
Cc: Domebone; Author
Subject: Re: [ericsomdahl/python-bittrex] Cannot get buy limit to work (#34)
There's a lot wrong here...you seem to understand the basic idea of using the ,JSON as a list but lack the proper implementation.
Also you need to examine your usage of quotation marks.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/ericsomdahl/python-bittrex/issues/34#issuecomment-318521556, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AM0YFFd9_IiNNytKst4DPOkqLP51cCpDks5sSShWgaJpZM4Okpeo.

i'm closing this issue, PEBKAC


Include resource in each test.

We'll need a variety of resources for a variety of tests, so
it's more straightforward to define them with each test.
This just breaks up the existing test.  Now I'll be able to actually add more tests.

Codecov Report

Merging #2 into master will increase coverage by 0.85%.
The diff coverage is 92.59%.

@@            Coverage Diff             @@
##           master       #2      +/-   ##
==========================================
+ Coverage   94.91%   95.77%   +0.85%
==========================================
  Files           3        3
  Lines          59       71      +12
==========================================
+ Hits           56       68      +12
  Misses          3        3

Impacted Files
Coverage Δ

tests/falcon_test.py
94.87% <92.59%> (+2.27%)
:arrow_up:

Continue to review full report at Codecov.

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


Ask password option: Difference between ask password code expiry time and email notification code expiry time

When configuring according to [1] in the actual configuration following two configs are there

Email verification code expiry time
Ask password code expiry time

How do the above configs are interrelated to the ask password option. Here since we are configuring the expiry time to an email based link, is there any dependency between the two configs?
[1] https://docs.wso2.com/display/IS540/Creating+Users+using+the+Ask+Password+Option

Thank you for your contribution!
We are closing this issue since it has not been prioritized for a long time. Chances are that it has already been solved in more recent versions. If not, we will be re-evaluating this when it becomes a priority.


config add a FE config to disable storage policy by default

Proposed changes
Issue Number: close #xxx
Problem summary
the cold-hot separation feature is still
under development. And seems there are some unsolved feature remains.
So I add a fe config enable_storage_policy, and default is false, to disable the creation of storage policy by default.
So that user can aware that he is using an experimental feature on his own, and it will not be released formally in v1.2.0.
Checklist(Required)

Does it affect the original behavior:

[ ] Yes
[x] No
[ ] I don't know

Has unit tests been added:

[ ] Yes
[ ] No
[x] No Need

Has document been added or modified:

[x] Yes
[ ] No
[ ] No Need

Does it need to update dependencies:

[ ] Yes
[x] No

Are there any changes that cannot be rolled back:

[ ] Yes (If Yes, please explain WHY)
[x] No

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

cc @pengxiangyu @deardeng

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 35.2 seconds
load time: 437 seconds
storage size:<PHONE_NUMBER>3 Bytes
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20221128164617_clickbench_pr_54590.html


enhancement #51: added set_torques function

Requirements for Contributing

Fill out the template below.

Identify the Bug (if you are fixing one)
Fixes #51
Description of the Change
Added a simple function "set_torques". The function sets the torque_limit of the given Reachy's part (between 0% and 100%).
This function works in a similar fashion to turn_on and turn_off regarding its "parts" input (The requested part can be 'l_arm', 'r_arm', 'head' or 'reachy').
Performed a few tests on a physical Reachy during grasping tests here:
https://github.com/pollen-robotics/simple_grasp_pipe/blob/master/simple_grasp_pipe/simple_grasp_pipe.py
=> The main motivation for this function was to "fix" the robot's state that occurs when the program is killed before a turn_off_smoothly is finished, which led to all the torques being set to 0.
Possible Drawbacks
Server overcharge due to an overwhelming amount of new users.
Release Notes
Added a simple function "set_torques(pert, percent)" to set all torques of a given part easily.

It looks fine to me but I haven't used the SDK in a long time. Letting @simheo check.


uiua: 0.0.25 -> 0.1.0

Description of changes
Changelog: https://github.com/uiua-lang/uiua/releases/tag/0.1.0
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 265698 run on x86_64-linux 1

2 packages built:

uiua
uiua386

Result of nixpkgs-review pr 265698 run on aarch64-darwin 1

2 packages built:

uiua
uiua386


Marking a mail as seen needs reload

Clicking Mark as seen need reload to make change visible

Cannot reproduce anymore. Was probably fixed by offline mode.


Where is the source code for iOS and Android?

Where is the source code for natively sharing and saving files on iOS and Android?  There's a few changes I'd like to make regarding permissions, but all I could find were precompiled libraries.

Hey @TCROC , we currently don't publish native sources for NatShare. What changes are you looking to make?

Saving videos and images on iOS doesn't work with "add only" permissions.  We have to give all access to the photo library in order to save photos or videos.  Why is this?  We thought it might make some of our users uncomfortable since the only thing our application does is save videos.  Requesting "all access" might come across strange.
And question 2, can you publish the native sources?    I really appreciate the work you are putting into this plugin and making it intuitive to use.  Being able to review the source code creates some piece of mind for us as developers as well (or at least my team it does).
Thanks again!

This is the error we get on iOS if we give users "add only" permissions:
2022-07-24 10:35:39.128167-0400 BlockyBall[69901:10579545] [PhotoKit] Attempted fetch within change block can trigger deadlock, returning unauthorized fetch result Save on mobile failed! BlockyBall.<Save>d__8:MoveNext() System.Threading.ExecutionContext:RunInternal(ExecutionContext, ContextCallback, Object, Boolean) System.Runtime.CompilerServices.MoveNextRunner:Run() UnityEngine.WorkRequest:Invoke() UnityEngine.UnitySynchronizationContext:Exec()

Saving videos and images on iOS doesn't work with "add only" permissions. We have to give all access to the photo library in order to save photos or videos. Why is this?

This is how iOS works, for some reason. And you don't actually have to grant full permissions to the photo library. Selecting "Select Photos" without picking a single image will still successfully save the payload to the camera roll.

We thought it might make some of our users uncomfortable since the only thing our application does is save videos. Requesting "all access" might come across strange.

The iOS docs are a bit misleading. You could request PHAccessLevelAddOnly permissions, but this is actually insufficient to save media to the camera roll, so iOS would automatically request PHAccessLevelReadWrite, provided that the user hasn't previously denied the permission.

And question 2, can you publish the native sources?  I really appreciate the work you are putting into this plugin and making it intuitive to use. Being able to review the source code creates some piece of mind for us as developers as well (or at least my team it does).

We offer access to source code as part of our enterprise-tier offering. The overwhelming majority of devs will never need to see our source code.
In the latest version, NatShare 1.3, we've added a SavePayload.RequestPermissions method that requests the same ReadWrite permissions that iOS does when saving to the camera roll.


NO-JIRA: e2e: skip OSP multinet before 4.18

What this PR does / why we need it:
This feature was only added in 4.18 so let's skip the test otherwise.

/test e2e-openstack

/approve

/lgtm

/test hypershift-operator-main-on-pull-request


User settings resetting to default after I logout then goes back to User settings menu

Hi,
Does anybody also experience the same situation that I have right now? I am using this ldap-based auth module. Then I use the oncall-admin to edit the attributes of each users(mobile,phone, photo, slack). Whenever the user makes a login and goes directly to User Settings menu, the user settings or attributes suddenly becomes null. However, if the user doesn't go to User Settings menu, everything works fine.
module: 'oncall.auth.modules.ldap_example'
module: 'oncall.auth.modules.ldap_import'
I just experienced this lately, after fetching new updates.

Fixed. I disabled oncall.auth.modules.ldap_import


switch to github errorutil

github.com/abh/errorutil is identical to go4.org/errorutil and go4.org is currently down and only used for this single dependency.  I found this in a downstream project (kubernetes) that could no longer godep.
This is my first time using glide, so I think I've done it all correctly, but double checking would be appreciated.
@vbatts @eparis

The source package on go4.org is already on github too. Using glide's repo key in glide.yaml, we can specify to fetch the go4.org package from github while still using its canonical (and broken) import path.
However, that breaks go-gettability and people using other vendoring systems like godeps I think.
It might all be a moot point though since in the time it took me to type this comment, go4.org is back up again 😄

It might all be a moot point though since in the time it took me to type this comment, go4.org is back up again

16 hours down and it comes back right as a I finally make glide sort of run.  Figures.

@deads2k I commented on the k8s issue too. It might be worth switching it still just based on it having a shaky track record at this point.

I see no reason to use an indirection like go4.org/camlistore/go4 when we actually want the real thing. Why would we want to use a collection of packages more slowly updated instead of just going to the real source? Even if it seems like the real source is unlikely to move.
Even thought go4 came back to life, I think this is the right thing to do.

The indirection is can be handy, but often frustrating. Not all turtles stack equally.

how come the LICENSE was dropped? I see that the newly pointed repo doesn't have it, but it seems the attribution should still be there.

What attribution? go4.org (aka https://github.com/camlistore/go4) didn't write any of this code. It is just a curated collection of others work. We aren't using the collection. We are just using the one thing. If the one thing doesn't have a LICENSE why should be vendor some other random LICENSE from an completely unrelated repository?

actually, as I look, maybe you are right...  go4 isn't 'unrelated'...

so should we switch or not?
On Fri, Jan 13, 2017 at 1:49 PM, Eric Paris<EMAIL_ADDRESS>wrote:

actually, as I look, maybe you are right... go4 isn't 'unrelated'...
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/appc/spec/pull/682#issuecomment-272516006, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAEF6W3N-ItfEW-fgSgFHzQEpRCp8gUoks5rR8cugaJpZM4LjJQ7
.

I think we do want the version specified to a commit if we do switch over, so this PR would need an additional slight change.
Since it hasn't been directly linked here, the Kubernetes issue that resulted in this change is https://github.com/kubernetes/kubernetes/issues/39868.
On that issue, the consensus seems to roughly be that it's really K8s responsibility to better avoid this sort of error causing so much blockage/pain.
I'm indifferent to this change.

Since I now believe that camlistore/go4 is in fact the parent package and abh/errorutil is in fact the clone/child I believe that appc/spec is pulling from the right upstream. Thus i think we should close this.


How to enable Users?

I´m missing something? How can i enable users? All users who try to use the website, after login with Discord they get redirect to: Disable account page...
how to manualy enable them? or is bugged? The only user on the administrato panel its me..

Hey,
unfortunately I haven't been running the tool for over a year, so my mind might be a bit fuzzy.
After registering your users should appear in the admin panel. There should be a toggle to enable them.
I guess they need to choose a nickname after authorizing via discord.
A wild guess: can you doublecheck everything regarding discord OAuth2, maybe something is odd there?
Another thing that could help: is the serverlog throwing any errors?

Hey, unfortunately I haven't been running the tool for over a year, so my mind might be a bit fuzzy.
After registering your users should appear in the admin panel. There should be a toggle to enable them. I guess they need to choose a nickname after authorizing via discord.
A wild guess: can you doublecheck everything regarding discord OAuth2, maybe something is odd there?
Another thing that could help: is the serverlog throwing any errors?

Im gonna try reinstall all..
Can i please have your discord? Or can you add at mine? DrugAddict#1051
I will be happy to buy you few cup coffee if you can help me out <3

@cbartel-ci  Check this:
https://gyazo.com/40d62b445f38d89d019260009d2aa8f7
https://gyazo.com/38d4cc55653654831c5295ad6a4fb501
https://gyazo.com/64f2045b3623568fde47d902042b37da
Where i suppose to enable the user?

@cbartel  @cbartel-ci  You can close the topic. If you have time someday you could add an auto-update.. The solution to show new users in the list, was to use the command stop of pm2 and start it again..
I dont mind to close and start it every time someone need to get approve. But if you can add few lines of code to do an auto-update without restarting the client will be the best! In case you do so i will buy you 2x Coffee <3

Hey buddy,
very strange, wasn't like this the last time I've touched the project. But might ofc be, depending on node version, dependencies and stuff - you never no.
Unfortunately I gave up on this project, will not touch the code in any soon'ish future. But it's here for people to fork and continue :)
closing this issue then :) thanks for your feedback!


blue_score:: modified_precision() zero division exception

Issue:
In modified_precision(), there is no check if the Counter has 0 results - if there are no ngrams in the hypothesis, a zero division exception is thrown.
Code to reproduce:
from nltk.translate import bleu as bleu_score

hypothesis = ['1', '2', '3']
references = [['1', '2', '3', '4']]
bleu_score(references=references, hypothesis=hypothesis)

Exception output:
Traceback (most recent call last):
  File "bug.py", line 6, in <module>
    bleu_score(references=references, hypothesis=hypothesis)
  File "/home/roei/anaconda2/lib/python2.7/site-packages/nltk/translate/bleu_score.py", line 79, in sentence_bleu
    return corpus_bleu([references], [hypothesis], weights, smoothing_function)
  File "/home/roei/anaconda2/lib/python2.7/site-packages/nltk/translate/bleu_score.py", line 146, in corpus_bleu
    p_i = modified_precision(references, hypothesis, i)
  File "/home/roei/anaconda2/lib/python2.7/site-packages/nltk/translate/bleu_score.py", line 287, in modified_precision
    return Fraction(numerator, denominator, _normalize=False)
  File "/home/roei/anaconda2/lib/python2.7/site-packages/nltk/compat.py", line 700, in __new__
    cls = super(Fraction, cls).__new__(cls, numerator, denominator)
  File "/home/roei/anaconda2/lib/python2.7/fractions.py", line 162, in __new__
    raise ZeroDivisionError('Fraction(%s, 0)' % numerator)
ZeroDivisionError: Fraction(0, 0)

Added a pull request for the fix.

@rbahumi which version of the code are you using? The master branch or the develop branch?
This should have been previously fixed by allowing it to go through the smoothing function. I'll check the changes made from your PR in a while =)

Hi @alvations,
Thank you for your quick response :-)
The develop branch doesn't have this check. I can't really find a branch named master:
roei> git branch -r
 origin/HEAD -> origin/develop
 origin/alvations-patch-bleu
 origin/alvations-patch-senna
 origin/brill
 origin/bug_1458_modified_precision_zero_division_exception
 origin/ccg
 origin/childes
 origin/collections
 origin/defunct
 origin/develop
 origin/framenet
 origin/model
 origin/model2
 origin/nschneid-patch-1
 origin/nschneid-patch-2
 origin/panlex
 origin/pattern_groups
 origin/py25
 origin/relextract
 origin/test
 origin/wiki

I believe this  empty counter check was present in version 3.1

The counter check is no longer necessary after the #1383 and #1358.
Could you do a git pull merge to the latest develop branch and then retry your example? I am not getting ZeroDivisionError error on my machine:
alvas@ubi:~/git/nltk$ python
Python 2.7.12 (default, Jul  1 2016, 15:12:24)
[GCC 5.4.0 20160609] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> from nltk.translate import bleu as bleu_score
>>>
>>> hypothesis = ['1', '2', '3']
>>> references = [['1', '2', '3', '4']]
>>> bleu_score(references=references, hypothesis=hypothesis)
0.7165313105737893

You are correct, it does not happen in the develop branch.
But this is really weird, because I am also running version 3.2.1 (install with conda), and the bleu_score.py file is different to the develop branch.
It is actually your commit (3dd4d82b704af16fbc5bc479796b11dc5d95c2aa) which solved the issue.

It's because the conda and pip is shipping the code based on the last minor version 3.2.1. The patch for the fringe cases in #1383 is after the release of 3.2.1. There should be a new minor release soon and that'll include the fringe cases fix as well as other feature improvements since the last release.
@rbahumi  so would this issue be a close =) ?

Yes. Thank you very much for your quick response!


Mafia: fix crash with subs

@HoeenCoder

oops, this was also a problem in a bunch of other places
fixed in 43f541d


commonsBcel: 6.6.1 -> 6.7.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/bcel/versions.
meta.description for commonsBcel is: Gives users a convenient way to analyze, create, and manipulate (binary) Java class files
meta.homepage for commonsBcel is: https://commons.apache.org/proper/commons-bcel/
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 6.7.0 in filename of file in /nix/store/hdqrny2lbw0drdb5j87iyc2j17wr4i6f-commons-bcel-6.7.0

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
commonsBcel

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/hdqrny2lbw0drdb5j87iyc2j17wr4i6f-commons-bcel-6.7.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A commonsBcel https://github.com/r-ryantm/nixpkgs/archive/595e79bafea633972874ed264588490ff3aa4a35.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/595e79bafea633972874ed264588490ff3aa4a35#commonsBcel

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/hdqrny2lbw0drdb5j87iyc2j17wr4i6f-commons-bcel-6.7.0
ls -la /nix/store/hdqrny2lbw0drdb5j87iyc2j17wr4i6f-commons-bcel-6.7.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

commonsBcel

Maintainer pings
cc @copumpkin for testing.

Result of nixpkgs-review pr 255713 run on x86_64-linux 1

1 package built:

commonsBcel


Make the baseName property of a binary immutable or configurable

Currently, the baseName property for a C++ or Swift component is used as the default base name for each binary. The baseName property on each binary is modelled as a Provider<String> which means that it cannot be configured on a per-binary basis, but can change at any time. Modelling this as a provider instead of a property adds complexity that every plugin must deal with, as the value can change at any time but allowing it to change is not useful.
If there is no good use case for a per-binary base name, then it would be better to model the baseName of a binary as a String and lock the baseName property on the component when we start calculating the binaries for the component, in a similar way that linkage is handled.

The baseName of a binary should be configurable. As user's case, the Poco binaries, either be a SharedLibrary, a StaticLibrary or an Executable should have the letter 'd' as suffix of the baseName when compiled/linked in debug mode. Moreover, the baseName of a binary component should be prefixed by 'Poco'. For exemple, building the ':Poco:Crypto' submodule as defined below
model {
	components {
		Crypto(NativeLibrarySpec) {

should produce

PocoCrypto.dll in release mode
PocoCryptod.dll in debug mode

For a SharedLibrary, the top gradle script builds the full path with
if (it instanceof SharedLibraryBinarySpec && buildType == buildTypes.debug) {
	sharedLibraryFile = toBin(prefixByPoco(appendDebugSuffix(suffixByArch(sharedLibraryFile, targetPlatform))), targetPlatform)
	sharedLibraryLinkFile = toLib(prefixByPoco(appendDebugSuffix(sharedLibraryLinkFile)), targetPlatform)

Alternatively, we could go down the route of providing a service to name the binary which would help you specify the suffix and prefix each binary should have relative to the base name.
There is one case that I saw where the base name for source targetting the user mode and kernel mode was differing. However, in the end, they should probably be modeled as 2 projects/components using the same source.


Integrate WifiManager for interactive configuration.

This integrates the WifiManager that opens a softAP and spawns a
captive portal where the user can configure the WiFi settings
interactive.
If there is an SSID/Password set in the config.h, then the WifiManager
will NOT used and instead a static connection to the configured network
will get established.
This is based on #17

Almost a year ago, I also started to integrate WifiManager. But I never cleaned up the code and published it.
Some thinkings, if we go this way:

I ran out of heap memory with WifiManager's network list, if there were many wifi networks.
We have to consider the security of the Gateway. Not yet another IoT zombie.
How to reset/change the wifi password (we may need a button)
How to save more configurations (MQTT brocker + settings, ota_secret, ...)

But starting with small steps is fine.

I'm working on a proper configuration frontend including auth. The WifiManager is just the first step. The frontend is planned to enable you to set all the config options and reset the (wifi-)config. It will support authentication.
The security-impact from the wifi-manager should not be an issue as it only spawns its service if no wifi is configured or the config was reset.

It also spawns, if it can not connect. This can open the window for DoS. Basically it is fine if we consider that the 433 MHz protocols also not secured.

If you want to make it spawn as an attacker, you have to make sure, that the regular wifi network is not connectable and issue a reboot of the device.
Besides the DoS that comes from the fact that the Wifi is not connectable (which is nothing the firmware of the ESP can do anything about) the unavailability in the worst case lasts only as long as the network-outage plus the 180 seconds timeout that the WifiManager has set. After that the device reboots and tries a normal connect again.
I can make the AP password configurable at a later point (with the config UI). This would prevent the hypothetical attack that someone knows what device it is, what the default-password of the Wifi-Connect-Portal is, connects to it and configures another network.
All in all, considering that the 433 MHz protocols are totally exposed to any local attacker, I would see this as a not so problematic issue. Whoever wants to give you wrong data from your senders just can transmit it, whoever wants your actors to behave in a non-expected way can just transmit the 433MHz data.

There is arendst/Sonoff-Tasmota which provides a very nice and clean GUI for ESP-based boards. Including Wifi & MQTT-config, debugging, etc. Maybe it is a good idea to take some inspiration from there. E.g. beside of the wifi config itself, I really would like to see a debug/logging method working via network. If the MQTT-Gateway sits somewhere remotely, it would be nice to be able to re-configure / debug it without the need to bring it close a computer.

I don't really like to put HTML snippets into the c++ code like https://github.com/arendst/Sonoff-Tasmota/blob/development/sonoff/webserver.ino. I like the idea that the esp8266_milight_hub implemented more. They've just static HTML/Js/CSS, compile this all in one file, gzip and compress it and put it into a header as progmem. This gets then delivered and interact with the device via a REST API.

I did not had a close look to both projects. But taking inspiration from other projects in case of code and UI should be done. From the architecture, I definitely prefer a separation of C++ code and web frontend, with a defined interface/api.
+1 for REST API

I agree that HTML snippets in C code are not very elegant. However, I am impressed by the Tasmota project. It fits completely in an ES8266 and provides really a lot of functionality. Beside a very well designed  MQTT interface, it provides a web interface which even scales nicely on smartphone screens and an HTTP interface. Single firmware for all supported devices. Fully adjustable by the user via the web-interface, including I/O assignments. In a debug window, you can send and receive MQTT messages which makes debugging a much more enjoyable task...  again all within a ESP8266.... Really impressive in my opinion...

Rebased on latest changes in master.

I would leave this password as default there. It is only to set up the wifi connection. It will not be used for anything as soon as the device is connected.
So the only thing an "attacker" can do with it is to connect to a device that is not connected to any network and connect it to a network. He cannot change any settings, exploit the ting over the internet, ...
Regarding the beat led - im still not sure in general what to do with this "feature".

@janLo please have a look to #22 as an alternative PR.


Remove do-nothing backoff

The previous backoff loop will have no effect, even with the volatile variable. We are already using a proper wait function, gpr_event_wait() that avoids busy waiting.

Can one of the admins verify this patch?

Suspect we want to just remove this (per #2899) instead.

Can one of the admins verify this patch?


Feat/displaying variable hints as secrets

fixes: #3265
Description
This PR addresses the issue of secret variables being displayed on hover in editor components by implementing the following logic:

Checks masked variables from environment variables, which support secret variables.
Verifies if lower-level variables (such as request or folder variables) override the secret variables, including cases with string interpolation.
Creates a (*****) node to hide secret values and prevent them from being displayed.

Contribution Checklist:

[x] The pull request only addresses one issue or adds one feature.
[x] The pull request does not introduce any breaking changes
[x] I have added screenshots or gifs to help explain the change if applicable.
[x] I have read the contribution guidelines.
[x] Create an issue and link to the pull request.

Before:

After:
https://github.com/user-attachments/assets/7319a8e0-8dae-465a-a35c-f8fa0712ac92

Why do we want this? It's usefull to see the secrets.

Hey @fredx21!
The intention of this PR is to hide secrets when hovering over variables in the Editor, particularly during cases like screen sharing. We thought that this Discussion made sense, as it suggests hiding the variable only if it is marked as a secret. In some cases, we can't guarantee whether a variable is a secret since it is displayed when accessed. This will provide users with a strict way to view all secret variables in the same place—in this case, the Environment variable section. All secret variables can be viewed and edited in that area.
PS: I acknowledge that in some cases, it may be useful to show the secret variables. It's truly subjective, and I believe the discussion mentioned above made sense, especially when we're on a client call and want to avoid exposing secret variables. For example, I used {{host}} as a secret. In most cases, it will be things like API keys, passwords, and JWT tokens. However, I think using {{host}} as an example may have caused a misunderstanding of the feature.

I understand that in some case it might be usefull to hide the secret variable, however, its more often usefull to have those secrets shown as this is mostly a developer-first tool.
What do you think about adding a option in setting to allow user changing the default behavior?

What do you think about adding a option in setting to allow user changing the default behavior?

This one will work great. Thanks for the nice suggestion, by the way!

I have merged the PR.
@Pragadesh-45 please create a new github issue to track @fredx21 suggestion on adding a option in setting to allow user changing the default behavior.