[bug] Looses the type of a disjoint union inside an arrow function

First off, thanks for the amazing tool. Flow has been an incredible time-saver for me and I appreciate all the collective effort that went into it.
I'd like to report a bug that's best demonstrated by code:
Exhibit A:
This code is supposed to be working but it's not. I guess what might be happening is, the action type information somehow gets lost inside the map arrow function.
type User = { isFollowing: bool };

type Action =
  { type: 'FOLLOW_USER', idx: number } |
  { type: 'SOMETHING' };

function usersReducer(state: Array<User>, action: Action): Array<User> {
  switch (action.type) {
    case 'FOLLOW_USER': {
      const isFirst = action.idx === 0; // not giving an error
      const suggestions = state.map((person, idx) => {
        if (idx !== action.idx) return person; // BOOM
        return { ...person, isFollowing: !person.isFollowing };
      });
      return suggestions;
    }
    default: return state;
  }
}

12:         if (idx !== action.idx) return person; // BOOM
                               ^ property `idx`. Property not found in
12:         if (idx !== action.idx) return person; // BOOM
                        ^ object type

Exhibit B:
However, if I capture a field from action outside the arrow function, then use it in it, things seem to work.
type User = { isFollowing: bool };

type Action =
  { type: 'FOLLOW_USER', idx: number } |
  { type: 'SOMETHING' };

function usersReducer(state: Array<User>, action: Action): Array<User> {
  switch (action.type) {
    case 'FOLLOW_USER': {
      const isFirst = action.idx === 0; // not giving an error
      const actionIdx = action.idx; // capture the value
      const suggestions = state.map((person, idx) => {
        if (idx !== actionIdx) return person; // no longer booms
        return { ...person, isFollowing: !person.isFollowing };
      });
      return suggestions;
    }
    default: return state;
  }
}

No errors!

Exhibit C:
Finally, this demonstrates type information is not lost if Action is an object type, instead of being a disjoint union.
type User = { isFollowing: bool };

type Action =
  { type: 'FOLLOW_USER', idx: number };

function usersReducer(state: Array<User>, action: Action): Array<User> {
  switch (action.type) {
    case 'FOLLOW_USER': {
      const isFirst = action.idx === 0; // not giving an error
      const suggestions = state.map((person, idx) => {
        if (idx !== action.idx) return person; // no longer booms
        return { ...person, isFollowing: !person.isFollowing };
      });
      return suggestions;
    }
    default: return state;
  }
}

No errors!

This is a bug because in the case A, there is enough information to conclude which branch of the union is being referred inside the case statement, and there are no new facts inside the arrow function that would change that.

Narrowing on a variable is undone inside a closure that captures that variable because Flow can't prove that the variable won't change before the closure is called.
One workaround is to capture a const variable instead:
type User = { isFollowing: bool };

type Action =
  { type: 'FOLLOW_USER', idx: number } |
  { type: 'SOMETHING' };

function usersReducer(state: Array<User>, action_: Action): Array<User> {
  const action = action_;
  switch (action.type) {
    case 'FOLLOW_USER': {
      const isFirst = action.idx === 0; // not giving an error
      const suggestions = state.map((person, idx) => {
        if (idx !== action.idx) return person; // okay now
        return { ...person, isFollowing: !person.isFollowing };
      });
      return suggestions;
    }
    default: return state;
  }
}

@jesseschalken wow, thanks for the fast reply! That could work... but it still feels like a hack, much like capturing idx before the closure.
I'm not sure if it's tricky to implement this, but maybe Flow could look for argument reassignments, and if none exist, assume it's proven it won't change?

@goshakkk That sounds reasonable to me, but you'll have to wait for a Flow dev to say for certain and whether it's achievable in Flow's design.

Another way to fix the error is to set the experimental.const_params=true option in the [options] section of .flowconfig
Related

https://github.com/facebook/flow/issues/2836
https://github.com/facebook/flow/issues/2831
https://github.com/facebook/flow/issues/2768
https://github.com/facebook/flow/issues/2704
https://github.com/facebook/flow/issues/2693
https://github.com/facebook/flow/issues/2602
https://github.com/facebook/flow/issues/2513
https://github.com/facebook/flow/issues/2318


example of syntax highlighted code in Halogen using PrismJS

Syntax highlighted code in a Halogen App
HalogenSyntaxHighlightedCode

Each recipe name must be unique. Use PascalCase.
Recipe Description
No idea if this would be worthwhile but i'm open to generating the recipe if you'd like it.
Here's how i solved it, recipe would presumably be more or less just that:
https://discourse.purescript.org/t/syntax-highlighting-of-purescript-code-in-halogen-app-with-prismjs/2476/3?u=afc

Yeah, this is worth it. Mind adding it?

I'd be happy to.

Ah, just realized that https://github.com/rnons/purescript-html-parser-halogen/releases/tag/v1.0.0-rc.2 which actually makes this possible is not in the package set...i could try to add it to the package-set but the -rc.2 seems like it should get bumped...i'll mail the author...

Ah... yeah...

Ping kindly responded that he'd try to publish the parser package to the package set this weekend if he had time. ❤
I'll pick it up again when he has done that.

Sounds good!


Change contract for voicecoin

Please submit the information separately
https://github.com/consenlabs/token-profile/pull/1636


Dynamically pass num_letters argument based on captcha solution

Right now num_letters argument is always falling back to its default value (6).
This PR sets that argument based on the provided captcha solution.

@fsarachu thanks for your PR! please resolve these conflicts

@fsarachu thanks for your PR! please resolve these conflicts

@dominhhai Done!


product landing

cards
yt video

Please always provide the GitHub issue(s) your PR is for, as well as test URLs where your change can be observed (before and after):
Fix https://github.com/hlxsites/prisma-cloud-docs-website/issues/16
Test URLs:

Before: https://main--prisma-cloud-docs-website--hlxsites.hlx.page/prisma/prisma-cloud/en
After: https://product-landing--prisma-cloud-docs-website--hlxsites.hlx.page/prisma/prisma-cloud/en

Mhh the psi check keeps spinning ..


My bandwidth not very high speed, so is there a way to watch stream you downloading?

Subj.

https://github.com/rg3/youtube-dl#how-do-i-stream-directly-to-media-player

But can i download and stream at the same time?


Harryshu1997 patch 1

Enable codecov
-- Added codecov xml
-- Added github action

Codecov Report

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

@@            Coverage Diff            @@
##             master       #2   +/-   ##
=========================================
  Coverage          ?   95.71%
=========================================
  Files             ?        4
  Lines             ?      140
  Branches          ?        0
=========================================
  Hits              ?      134
  Misses            ?        6
  Partials          ?        0

Continue to review full report at Codecov.

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


Provide list of supported Operating Systems

It seems Azure VM with Ubuntu 20.04 LTS is not supported:
2021-10-20 14:41:46.232115100 [INFO]: The user has opted in for sending usage telemetry.
2021-10-20 14:41:46.238332369 [INFO]: OS ID: 'ubuntu'; OS Version ID: '20.04'; VERSION_CODENAME: 'focal'
2021-10-20 14:41:46.240434192 [ERR]: This OS is not supported. Please visit this link for more information https://docs.microsoft.com/en-us/azure/iot-edge/support?view=iotedge-2020-11#tier-1.
2021-10-20 14:41:46.242715717 [INFO]: Exit 3
2021-10-20 14:41:46.246262456 [INFO]: Runtime duration 2

Please provide a reference to the operating systems supported in the Readme. (probably taken from https://docs.microsoft.com/en-us/azure/iot-edge/support?view=iotedge-2020-11)

@sandervandevelde thanks for your issue report. The readme already contains a reference to this article. Below is the direct link. https://github.com/Azure/iot-edge-config/blob/8ea88cfdca469bdc99c9d49e2775c1ba61afcc3e/README.md?plain=1#L14


Carthage bumped to Swift 4.0

After figuring out I had to specify "master" in the Cartfile or it would pick up tag fix-6.5.3, it builds. Still haven't been able to resolve the compile issue without being very explicit about things, and it seems Swift 5 doesn't allow empty in expressions. But that's a separate issue from this. Thanks!


bug with string changing in goroutine:

What version of Go are you using (go version)?
go1.20.1 darwin/amd64
Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
What did you do?
Link to the repository with the service: https://github.com/niciki/go-bug. When starting the service, call the handle /printstring with an id of some kind, for example, http://localhost:8080/v1/printstring/2adf08ce-5c8d-49c9-acbc-99c08b908d91, and then pull the handle http://localhost:8080/v1/get/string many times. In the running goroutine, the value of id passed changes, as if some kind of shift of the pointer in memory occurs, as can be seen in the output. If the handle / get is not called, everything works correctly. At the same time, on Go 1.18, and the AMD processor, everything works correctly.
What did you expect to see?

What did you see instead?

Looks like a data race. Have you run your program under race mode?

race mode

yes, you're right, there are problems with DATA RACE. But how can it arise, the string does not change in the gorutin, the gorutin is one, what can cause this string to change?

A data race can cause a string to change because the same memory can be accessed from two different goroutines.  Fix the data race first, then see if you still have a problem.

I notice you use fiber in your code. See https://docs.gofiber.io/#zero-allocation. Fiber will reuse fiber.Ctx that may cause a data race.


Feature request - IR input

Can we have easy way to add InfraRed input for signals from remotes?
Input (receiving IR codes)
Output (sending IR codes)
Thank you.

I do not place to implement it in the firmware. I would recommend using an external Arduino board and CC2530 as a bridge.

I do not place to implement it in the firmware. I would recommend using an external Arduino board and CC2530 as a bridge.


Getting ArgumentError

Hi,
I am new to Graylog2 and I have just tried to get my first rails application integrated.  All I have done is add the line:
config.logger = GELF::Logger.new("<IP_ADDRESS>", 12201, "WAN", { :facility => "testapp" })
to my development.rb.
This seems to work ok, in that I am getting messages through to Graylog2 but I also get a lot of:
ArgumentError: short_message is missing. Options version, short_message and host must be set.
I think these are just blank lines but cannot be sure.  I see that there was a pull request to fix blank lines but it was never merged.  Could that be the problem?
Regards,
Gavin

My recommendation would be to use a gem like lograge to convert the Rails logs to a format better suited to structured logging. From the point of the logging code, receiving empty log messages is a bit silly and not that surprising that it complains.


Add GetKeysSummary Api Signature/protos

Add GetKeysSummary Api Signature/protos
This will auto-generate required request/response objects.
This API can be used for computing version difference of keys b/w client and server-side and lazily load only necessary values.
Some notes from previous discussion on this api:
regarding "return everything with a certain key prefix" => This makes for an unbounded query on server-side hence we should be avoiding it and use pagination.
Also note that we could support storing values upto ~5MB. If we started returning value as well in this response, even a 10 item limit might mean 50MB response. So it is best to only load necessary key-values lazily depending on key-version differences.
Note regarding http/rest+protobuf protocol:
Here protobuf is only being used as a binary serialization format for request and response objects.(instead of writing custom binary serialization or json). This helps both client and server to create requests/response like class instances with ease. This payload will be used as binary body in http-rest post requests.

Also note that we could support storing values upto ~5MB. If we started returning value as well in this response, even a 10 item limit might mean 50MB response. So it is best to only load necessary key-values lazily depending on key-version differences.

I would still provide a streaming Get with values in addition to this, for use cases that have numerous small entries.  there may be thousands of them, and it may be pretty slow to get them one at a time due to round-trip time.  the pagination could be sensitive to the item size.

@devrandom can you take an another look at this.


Adding gitlab

Gitlab is now hiring people in Italy. See https://about.gitlab.com/handbook/people-group/employment-solutions/#country-hiring-guidelines

Nice catch! Thanks @napolux


Listing databricks workspaces from a subscription doesnt work: list_by_subscription() in azure-mgmt-databricks crashes whereas list_dy_resource_group works.

azure-mgmt-databricks
1.0.0 but it happens also with the beta versions (also msrest 0.7.1 but tried with 0.6.9 as well and same issue)
Windows
3.11

Describe the bug
When trying to list workspaces by subscription it returns the iterator but upon iterating the results it crashes with the following error:

dbr_client.workspaces.list_by_subscription()
<iterator object azure.core.paging.ItemPaged at 0x1bef2f4aa10>
for i, r in enumerate(dbr_client.workspaces.list_by_subscription()):
...     print(r)
...
Traceback (most recent call last):
File "C:\Python\Lib\site-packages\msrest\serialization.py", line 1436, in _deserialize
found_value = key_extractor(attr, attr_desc, data)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Python\Lib\site-packages\msrest\serialization.py", line 1180, in rest_key_extractor
return working_data.get(key)
^^^^^^^^^^^^^^^^
AttributeError: 'list' object has no attribute 'get'

File "C:\Python\Lib\site-packages\azure\core\paging.py", line 128, in next
return next(self._page_iterator)
^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Python\Lib\site-packages\azure\core\paging.py", line 84, in next
self.continuation_token, self._current_page = self._extract_data(self._response)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Python\Lib\site-packages\azure\mgmt\databricks\operations_workspaces_operations.py", line 597, in extract_data
deserialized = self._deserialize('WorkspaceListResult', pipeline_response)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Python\Lib\site-packages\msrest\serialization.py", line 1376, in call
return self._deserialize(target_obj, data)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Python\Lib\site-packages\msrest\serialization.py", line 1454, in _deserialize
raise_with_traceback(DeserializationError, msg, err)
File "C:\Python\Lib\site-packages\msrest\exceptions.py", line 51, in raise_with_traceback
raise error.with_traceback(exc_traceback)
File "C:\Python\Lib\site-packages\msrest\serialization.py", line 1436, in _deserialize
found_value = key_extractor(attr, attr_desc, data)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Python\Lib\site-packages\msrest\serialization.py", line 1180, in rest_key_extractor
return working_data.get(key)
^^^^^^^^^^^^^^^^
azure.core.exceptions.DeserializationError: ("Unable to deserialize to object: type, AttributeError: 'list' object has no attribute 'get'", AttributeError("'list' object has no attribute 'get'"))
To Reproduce
Steps to reproduce the behavior:

Create a piece of code with the folowing:
credential = DefaultAzureCredential(exclude_shared_token_cache_credential=True, exclude_interactive_browser_credential=False, exclude_visual_studio_code_credential=True)
dbr_client = DatabricksClient(credential, yourSubscriptionIdHere)
dbr_client.workspaces.list_by_subscription()

The function to return by resource group works fine:
dbr_client.workspaces.list_by_resource_group("XXXXXXXXXXX")
<iterator object azure.core.paging.ItemPaged at 0x23919f1ff90>

dbr_client.workspaces.list_by_resource_group("XXXXXXXXXXXXXX").dict
{'_args': (<function WorkspacesOperations.list_by_resource_group..get_next at 0x000002391AD627A0>, <function WorkspacesOperations.list_by_resource_group..extract_data at 0x000002391AD62840>), '_kwargs': {}, '_page_iterator': None, '_page_iterator_class': <class 'azure.core.paging.PageIterator'>}

but the subscription altough it returns the iterator something doesnt work well when you iterate over it

dbr_client.workspaces.list_by_subscription().dict
{'_args': (<function WorkspacesOperations.list_by_subscription..get_next at 0x000002391AD62840>, <function WorkspacesOperations.list_by_subscription..extract_data at 0x000002391AD62980>), '_kwargs': {}, '_page_iterator': None, '_page_iterator_class': <class 'azure.core.paging.PageIterator'>}

Expected behavior
In the past (like 6 months ago) it worked and returned an object with all the databricks workspaces in a subscription
Additional context
Since its not working you need to resort to the api. The api call returns the data fine so i assue its a problem on how its handled the return of the api call. Currently y had to switch to the following:
url = 'https://management.azure.com/subscriptions/{subscriptionId}/providers/Microsoft.Databricks/workspaces?api-version=2023-02-01'.format(subscriptionId=subscriptions[subscription])
headers = {'Content-Type': 'application/json',
    'Authorization': 'Bearer ' + tokenDefault
    }
response = requests.get(url=url,headers = headers)
d = json.loads(response.text)
for wks in d:
    print(wks['properties']['workspaceUrl'])

Also It seems your unittests are not testing that function
https://github.com/Azure/azure-sdk-for-python/blob/main/sdk/databricks/azure-mgmt-databricks/tests/test_cli_mgmt_databricks.py
I see two calls for the list_by_resource_group() but none for list_by_subscription() Maybe you should fix that too.
BR,
Albert

@anogues Please try to use this example to solve your problem: https://github.com/Azure/azure-sdk-for-python/blob/main/sdk/databricks/azure-mgmt-databricks/generated_samples/workspaces_list_by_subscription.py

@Wzb123456789 Hello.
I've already tried this version and same error. Just tried it again:
C:\Users\Albert>pip install azure-mgmt-databricks==1.1.0b2
Requirement already satisfied: azure-mgmt-databricks==1.1.0b2 in c:\python\lib\site-packages (1.1.0b2)
Requirement already satisfied: msrest>=0.7.1 in c:\python\lib\site-packages (from azure-mgmt-databricks==1.1.0b2) (0.7.1)
Requirement already satisfied: azure-common~=1.1 in c:\python\lib\site-packages (from azure-mgmt-databricks==1.1.0b2) (1.1.28)
Requirement already satisfied: typing-extensions>=4.3.0 in c:\python\lib\site-packages (from azure-core<2.0.0,>=1.24.0->azure-mgmt-core<2.0.0,>=1.3.2->azure-mgmt-databricks==1.1.0b2) (4.4.0)
Requirement already satisfied: charset-normalizer<3,>=2 in c:\python\lib\site-packages (from requests~=2.16->msrest>=0.7.1->azure-mgmt-databricks==1.1.0b2) (2.0.12)
Requirement already satisfied: idna<4,>=2.5 in c:\python\lib\site-packages (from requests~=2.16->msrest>=0.7.1->azure-mgmt-databricks==1.1.0b2) (3.4)
Requirement already satisfied: urllib3<1.27,>=1.21.1 in c:\python\lib\site-packages (from requests~=2.16->msrest>=0.7.1->azure-mgmt-databricks==1.1.0b2) (1.26.13)
C:\Users\Albert>python
Python 3.11.1 (tags/v3.11.1:a7a450f, Dec  6 2022, 19:58:39) [MSC v.1934 64 bit (AMD64)] on win32
Type "help", "copyright", "credits" or "license" for more information.

from azure.identity import DefaultAzureCredential
from azure.mgmt.databricks import AzureDatabricksManagementClient
client = AzureDatabricksManagementClient(credential=DefaultAzureCredential(exclude_shared_token_cache_credential=True, exclude_interactive_browser_credential=False, exclude_visual_studio_code_credential=True),subscription_id="XXXXXXXXXXXXXXX")
response = client.workspaces.list_by_subscription()
for item in response:
...     print(item)
...
Traceback (most recent call last):
File "C:\Python\Lib\site-packages\azure\mgmt\databricks_serialization.py", line 1430, in _deserialize
found_value = key_extractor(attr, attr_desc, data)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Python\Lib\site-packages\azure\mgmt\databricks_serialization.py", line 1177, in rest_key_extractor
return working_data.get(key)
^^^^^^^^^^^^^^^^
AttributeError: 'list' object has no attribute 'get'

During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "", line 1, in
File "C:\Python\Lib\site-packages\azure\core\paging.py", line 124, in next
return next(self._page_iterator)
^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Python\Lib\site-packages\azure\core\paging.py", line 84, in next
self.continuation_token, self._current_page = self._extract_data(self._response)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Python\Lib\site-packages\azure\mgmt\databricks\operations_workspaces_operations.py", line 1034, in extract_data
deserialized = self._deserialize("WorkspaceListResult", pipeline_response)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Python\Lib\site-packages\azure\mgmt\databricks_serialization.py", line 1375, in call
return self._deserialize(target_obj, data)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Python\Lib\site-packages\azure\mgmt\databricks_serialization.py", line 1445, in _deserialize
raise_with_traceback(DeserializationError, msg, err)
File "C:\Python\Lib\site-packages\azure\core\exceptions.py", line 77, in raise_with_traceback
raise error.with_traceback(exc_traceback)
File "C:\Python\Lib\site-packages\azure\mgmt\databricks_serialization.py", line 1430, in _deserialize
found_value = key_extractor(attr, attr_desc, data)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Python\Lib\site-packages\azure\mgmt\databricks_serialization.py", line 1177, in rest_key_extractor
return working_data.get(key)
^^^^^^^^^^^^^^^^
azure.core.exceptions.DeserializationError: (", AttributeError: 'list' object has no attribute 'get'", 'Unable to deserialize to object: type', AttributeError("'list' object has no attribute 'get'"))

Hi @anogues I can't reproduce the error since I don't have that much resource to trigger pageable operation. Could you follow the following step to record the detailed log?
(1) add logging_enable in client like :
 DatabricksClient(..., logging_enable=True)

(2) follow https://github.com/Azure/azure-sdk-for-python/blob/main/doc/dev/debug_guide.md to enable debug logging
nit: please remember to conceal the sensitive info in log.

After debugging, I find the root cause is that service doesn't follow pageable pattern
(1) list_by_resoure_group works and its related swagger definition is : https://github.com/Azure/azure-rest-api-specs/blob/ac98588da3cf3c3e043568adc8a6cc122ad8d6a7/specification/databricks/resource-manager/Microsoft.Databricks/preview/2022-04-01-preview/databricks.json#L256
response from server:

(2) list_by_subscription doesn't work and its swagger definition: https://github.com/Azure/azure-rest-api-specs/blob/ac98588da3cf3c3e043568adc8a6cc122ad8d6a7/specification/databricks/resource-manager/Microsoft.Databricks/preview/2022-04-01-preview/databricks.json#L37
response from server:

We can see that (2) doesn't follow pageable pattern.


Workspace.get method unintentionally exposes  subscription id when Workspace is not found.

Package Name & Version:
azureml-core==1.31.0
Operating System: Windows
Python Version: 3.8.0
Describe the bug
The get_workspace method under _commands.py module exposes the users' subscription id when the user tries to validate whether the particular ML Workspace exists under the given resource or not.
This issue is reproducible in Github Action as well.
To Reproduce
Steps to reproduce the behavior:

Create an account in https://portal.azure.com/
Create only a resource group in Azure .
Run following to get the Service Principal details: az ad sp create-for-rbac --name {name} --role contributor --scopes /subscriptions/{subscription id}/resourceGroups/test --sdk-auth
Run the following code to check whether the ML Workspace exists under the resource group created in step 2

from azureml.core import Workspace
from azureml.core.authentication import ServicePrincipalAuthentication
from azureml.exceptions import ProjectSystemException

auth = ServicePrincipalAuthentication(tenant_id="3XXXXXXX-XXXX-XXXX-XXXX-XXXXXXXXXXXX",
                                      service_principal_id="9XXXXXXX-XXXX-XXXX-XXXX-XXXXXXXXXXXX",
                                      service_principal_password="XXX_XXXXXXXXXXXXXX_XXXXX~XX~XXXXXX",
                                      cloud='AzureCloud')
try:
    work_space = Workspace.get(auth=auth,
                               subscription_id="3XXXXXXX-XXXX-XXXX-XXXX-XXXXXXXXXXXX",
                               resource_group="test",
                               name="test-github",
                               location="eastus2",
                               cloud="AzureCloud")
except ProjectSystemException as exception:
    print(f"Error: {exception}")

Expected behavior
I have created resource group named "test".
Since ML Workspace isn't created using above mentioned steps, it is expected to be  throwing  ProjectSystemException error alone.
Actual behavior
It throws ProjectSystemException error along with get_workspace method error exposing the subscription id. Please refer the screenshot below.
Screenshots
From local system

From Github Action Log

Let me know if any additional details are required from my end.
Thanks
Bhanu

Closing legacy issue.
Please consider upgrading to AzureML v2 CLI/SDK. https://learn.microsoft.com/en-us/azure/machine-learning/concept-v2


Setup Notify container

Hello authors,
I tried to set up the notify's container by the guide and your answer:
Metro.notify.setup({
   container: $(".notifies")
});

Originally posted by @olton in https://github.com/olton/Metro-UI-CSS/issues/1130#issuecomment-402440002
But it seems that nothing happens. I see in the notify.js, there is no reference to this.options.container:
https://github.com/olton/Metro-UI-CSS/blob/4ef7883ac79b247f30d350c174fd59089aa1ee5e/js/plugins/notify.js#L25-L34
https://github.com/olton/Metro-UI-CSS/blob/4ef7883ac79b247f30d350c174fd59089aa1ee5e/js/plugins/notify.js#L100-L103
I found out I can set the container directly by Notify.container = $(...). But I think I should let you know.
Btw, Metro UI is so great, thank you 👍 .

Thank you, This is true, I really changed behavior for defining notifies container. Now you must use Metro.notify.container property to setup it.


Add version to package.json

There's no version number listed in the package.json. This makes installing the package directly from GitHub or any git URL impossible with NPM.

harmonhubjs-client uses automated semantic-release which generates latest version numbers on the fly during the travis build. there is no version number by intention in the git repo.
please see https://github.com/semantic-release/semantic-release for more information.

Is there any reason that semantic-release can't bump the version and also write it to the package.json and commit it when a release is created? It looks like it's most likely creating a tag and pushing that up to GitHub.
The problem remains, this package can't be installed with NPM directly from GitHub.


.env file missing in prisma folder... should be local folder

I am getting the below error when installing the new alpha25 version. It looks like it desires the .env file to be in a prisma folder, but my folder is called tenant. As I have a folder for multitenant use and a folder for shared database.
Unknown Error!
Error: Couldn't find the prisma/.env file
at /Users/username/Development/project/backend/src/prisma/tenant/node_modules/prisma-multi-tenant/build/shared/env.js:21:15
at Generator.next ()
at fulfilled (/Users/username/Development/project/backend/src/prisma/tenant/node_modules/prisma-multi-tenant/build/shared/env.js:5:58)

Thanks for raising this issue 🙏
Can you share you folder structure? Where does your schema.prisma & .env files are located?

so the folder structure is as below:
prisma
|-shared
|-tenants
|- schema.prisma
|- .env
so the tenants folder is where I use multi tenant prisma as this is a shared structure. and the shared folder is for users and system settings

so the folder structure is as below:
src->prisma->tenants-> the .env file and schema.prisma file are here
so the tenants folder is where I use multi tenant prisma as this is a shared structure. and the shared folder is for users and system settings

I see. Do you have different schema.prisma files between tenants/ and shared/ ?

yes, I only have the tenants folder using the multi tenant prisma. the other is just vanilla prisma

Okay, how is your schema.prisma different between those two folders ?

the Tenants have tenant data which is needed to create the individual databases for each tenant
the Shared is for site specific data that is shared across all tenants

so I want the tenants to be the only prisma multi tenant. I feel that it should just reside there. It worked on the older versions, but not with the new one

Ah okay, I see. So because you have a different architecture, the current installation command isn't working. The current installation command tries to edit the prisma/.env file to add the required env variables.
I'll try to fix that, but in the meantime, you can adapt your code and slightly modify your folder structure:
- prisma
  - tenants
    - prisma
      - .env
      - schema.prisma
  - shared
    - ...

This way, prisma-multi-tenant should be able to find the .env file in the correct place.
Is this workaround working for you ?

Ok I'll give this a try thanks!

No I'm still getting the same error

From which directory do you run the cli ?

I've run the cli from tenants and tenants->prisma.
it still trying to pull from the base directory

Unfortunately, I cannot reproduce. Can you create a small reproduction repository so that I can take a look?

you can view an example here: https://github.com/Xaedblade/prisma-multi-tenant-example

Hi @Xaedblade, I finally found some time to work on prisma-multi-tenant to fix some bugs. However, your reproduction repository does not seem to be accessible anymore. Can you make it available ?

Sorry it is active again for you

Hi @Xaedblade, I think I found why it isn't working:
In order for<EMAIL_ADDRESS>to work, you need to have a prisma/ folder with both a schema.prisma file and a .env file in it. (So for you, you'll need to move src/prisma/tenant/schema.prisma to src/prisma/tenant/prisma/schema.prisma and create a  src/prisma/tenant/prisma/.env file)
Also, make sure you have the correct prisma version (See compatibility here: releases)
Let me know if you were able to fix the issue

Since there have been no activity on this issue for a while, I'll close it. Don't hesitate to re-open it if  needed


Question: can we take normal conversion losses into account?

This issue is just to clear up some confusion.
Capacities
In ETMoses, we have capacity and carrier_capacity. It is my understanding that these correspond to the ETE equivalents: output_capacity and input_capacity. The possible conversion loss of a technology that turns one carrier into another is given by
loss = 1 - efficiency

where the efficiency is defined as
efficiency = capacity/carrier_capacity

performance_coefficient
As far as I understand this, the performance_coefficient in ETMoses equals the coefficient_of_performance in ETE which is 1.0 for all technologies except heat pumps.
In ETMoses it is used to translate between capacity and carrier_capacity as in
carrier_capacity = capacity / performance_coefficient

I think this works correctly for heat pumps but results in capacity being (*wrongfully) equal to carrier_capacity in all other technologies.
What to do?
@RobTerwel has researched which numbers would be correct as  capacity and carrier_capacity, taking into account conversion losses. Two notable exceptions are P2P (batteries) and EV which actually need to lose energy twice (both when charging and when discharging).
@antw what do you think would be a good (temporary) way to initialise capacity and carrier_capacity in ETMoses?
Some options that come to mind:

Add a column to the database with carrier_capacity (or efficiency) and initialise from there
Extend the ETE communication functions to import both 'input' and 'output' capacities

I'd prefer to do it right, and import the input and output capacities directly from ETEngine. I'm not convinced that is sufficient to model loss, however.
In Moses, a capacity is a bottleneck which limits the (maximum) flow of energy, rather than a conversion which may have loss. For example, a hypothetical P2P battery with an input capacity of 2kW and output capacity of 1kW charges at full capacity for one hour. 2kWh is stored. It will take two hours to fully discharge (2kWh stored / 1kW output capacity = 2 hours). It does not mean that 1kWh is discharged and 1kWh is thrown away as loss.
My feeling is that we want to support input and output efficiency in addition to input and output capacity.
The carrier_capacity attribute is an ugly solution for the lack of an ambient_heat input for the heat-pumps; the only place I can guarantee it works is in the Buffer class which implements the heat-pump behaviour. I would like to be rid of it in favour of a proper capacity/efficiency system.

In Moses, a capacity is a bottleneck which limits the (maximum) flow of energy, rather than a conversion which may have loss.

Yes. But (!) doesn't it behave as loss for heat producing technologies for instance? If a households_space_heater_network_gas produces heat at its maximum output_capacity of 22 kW, doesn't it consume gas at 27.5 kW (22 kW/ 0.8)? The discrepancy of 5.2 kW can be interpreted as loss right?

My feeling is that we want to support input and output efficiency in addition to input and output capacity.

As a longish-term solution I fully agree. But currently, the numbers are wrong and I would like to be able to come up with a temporary solution to allow current users to get somewhat physical results :grimacing:


Re-add dart:profiler

The removal of dart:profiler in 1.13 has caused a lot of pain and strife for Dart users, and 1.13 isn't even released yet. The analyzer package, which is transitively used by almost every Dart package at least for testing, used dart:profiler up until its 0.26.1+8 release in October. This means that most packages for which pub upgrade hasn't been run recently will crash as soon as their tests are run.
The core SDK should generally avoid making breaking changes, since without incrementing its major version there's no way for packages to be future-proof against such changes. This is a case where the change is particularly painful because of how it affects analyzer, one of the most widely-used packages. It's also relatively easy to just avoid the breakage by retaining dart:profiler as a deprecated library that exports dart:developer.
cc @kevmoo @sethladd

I can say that we did see more breaks that we anticipated. Unfortunately, we make breaking changes to the SDK (removing dart:profiler wasn't the only time), but we don't also rev the SDK version to reflect that we had a breaking change. Ideally, we should have bumped SDK version to 2.0 with this change.
If we were to roll back the removal of dart:profiler, would we hold everyone to a high standard of "we don't break users until we bump version number" ?
Perhaps the SDK team could consider a more liberal policy of bumping the SDK version when we introduce a breaking change.
Another idea: the SDK team could pick specific git tags to pull from, for each component of the SDK. Pulling from HEAD for everything means that it's harder to move quickly and keep the individual component parts light and shedding legacy.
Right now, we're trying to have it both ways: introduce breaking changes, but not bump versions.

I'd love to see the SDK start revving its major version semantically, but I think assessing breaking changes in terms of the pain they're likely to cause versus the difficulty of making them non-breaking is an acceptable short-term solution. This change is easy to avoid and causes a lot of pain, but some other breaking changes—like updating the VM service protocol—are harder to avoid and less likely to cause pain.

Florian fixed this: https://github.com/dart-lang/sdk/commit/be025b841211c5589f162e206160e656c81a8f83


Specifying dialectModulePath in sequelize initializer

Hi, I am just getting started with my first project using this ans I really love the sequelize-typescript approach!
I am converting an existing vanilla sequelize project over, this is the db connection I currently have:
export var sequelize = new Sequelize(my-new-db', null, null, {
    dialect: 'mssql',
    dialectModulePath: 'sequelize-msnodesqlv8',
    logging: false,
    dialectOptions: {
      driver: 'SQL Server Native Client 11.0',
      instanceName: 'MSSQLSERVER01',
      trustedConnection: true,
    }});

I need to use sequelize-msnodesqlv8 to support Windows auth, is this possible with sequelize-typescript?

Hey, it should work nearly the same way:
const sequelize = new Sequelize({
    database: 'my-new-db',
    dialect: 'mssql',
    dialectModulePath: 'sequelize-msnodesqlv8',
    logging: false,
    dialectOptions: {
      driver: 'SQL Server Native Client 11.0',
      instanceName: 'MSSQLSERVER01',
      trustedConnection: true,
    });

@georgeedwards Did it work for you?


[Obsidian] ...

Extension
https://www.raycast.com/marcjulian/obsidian
Description

Error:
Obsidian menu bar command doesn't work. Keeps askign me to reload.

Steps To Reproduce

Standard
search for obsidian extension
select menu bar command

Current Behaviour
Error askign for reload
Expected Behaviour
No response

Can you share a screenshot of the problem that you are facing or a more detailed description?

Hi @KruznickMeta as I did not get an answer I will close this issue now. If you have anything to add please reopen this issue. Thanks.

@raycastbot close this issue


link device does not work

Description of the issue
I have installed this service in docker variant - it works well, and the status is updated including the activity and attributes.
However, the power switch will not turn the PS5 on/off.
I went through the Link Device process, pasted the redirect link and the code.
Addon version
1.3.1
Platform
Core 2023.5.2
Installed with docker container
Logs of the issue (if applicable)
05/14/2023 9:34:02 PM
2023-05-14T18:34:02.958Z @ha:ps5:mqtt MQTT Message received ps5-mqtt/<redacted>/set/power
05/14/2023 9:34:05 PM
2023-05-14T18:34:05.562Z @ha:ps5:error Error: Registration error: 403: Forbidden: Other Error
05/14/2023 9:34:05 PM
    at Object.<anonymous> (/usr/local/lib/node_modules/playactor/dist/remoteplay/protocol.js:88:19)
05/14/2023 9:34:05 PM
    at Generator.next (<anonymous>)
05/14/2023 9:34:05 PM
    at fulfilled (/usr/local/lib/node_modules/playactor/dist/remoteplay/protocol.js:5:58)
05/14/2023 9:34:05 PM
    at processTicksAndRejections (node:internal/process/task_queues:96:5)

Did you follow all the steps described/linked here?

Yes I have.
✅ From the home screen, select  [Settings] > [System] > [Remote Play], and then turn on [Enable Remote Play].
✅ To start Remote Play while your PS5 console is in rest mode, select  [Settings] > [System] > [Power Saving] > [Features Available in Rest Mode]. Then turn on [Stay Connected to the Internet] and [Enable Turning on PS5 from Network].
I then navigated to the ps5-mqtt GUI, where I followed the instructions to link device with the URL and link code.
In the ps5-mqtt GUI, I see the device, the state etc. In HA I see the state & activity.
Turning off the HA switch does nothing.

OK, I guess my bad. I did all the above again, and now it is working  🤷


Domainname availability should be checked

Is explainpowershell dot com available, and what does it cost?

The domain name is available, on namecheap for less than €10 a year.

The domain name is available, on namecheap for less than €10 a year.


User is able to select not-ready clusters from cluster dropdown via search, stuck on loading screen/500 error

State of Rancher -- version 2.5-head (a420b51):

create clusters (i.e. import cluster, custom cluster) that is in pending state

ex. add an import cluster, but do not run the registration command on a node

go to dashboard

navigate to the cluster selection dropdown/text box -- unable to select pending clusters

search for a cluster in pending state, first match is highlighted

select this cluster by pressing <enter> -- rancher stuck in loading state for about 30 seconds

eventually resulting in a 500 error

with response waiting for cluster [c-6fcd6] agent to connect

Expected Result
UI should not allow selection of cluster that is pending/not ready through dashboard

This has been removed in 2.6

on master-head 979d2ba, can confirm this UI is no longer used. There is no longer a search function for clusters either, so could not attempt a repro. closing the issue

There is still a search, if you have enough of them that they're not all listed already.

retested with enough clusters for search to show up just to be sure; search no longer auto-selects the first match or exact matches (hitting enter does nothing. seems that user must click the option they want from search)


Add example on how to set html attributes on specific collection items

When searching the internet I came upon "helpful" hints like that one:
http://stackoverflow.com/questions/17554748/how-do-i-add-html-attributes-to-select-options-with-simple-form-rails
I took quite some time before I figured that this was not the right solution. To not let others fall into this trap, I propose to add this to the documentation.

Thank you for the pull request. The readme already point to the Rails documentation that talks about this and more possibilities so I don't think duplicate the information here.


Enabling Azure Repos integration via ARM.

Hi,
Can you provide documentation on how to enable AzureRepo integration via ARM.
According to the ARM Reference (https://docs.microsoft.com/en-us/azure/templates/microsoft.datafactory/2018-06-01/factories) this is possible; but after confirguring the template with the required values git repo integration is not enabled.
Kind regards
Micah.

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

ID: 9528484b-14b8-729a-9df4-d203db71885a
Version Independent ID: 95759275-ecbf-c737-3460-4e38756ecd05
Content: Visual authoring in Azure Data Factory
Content Source: articles/data-factory/author-visually.md
Service: data-factory
GitHub Login: @sharonlo101
Microsoft Alias: shlo

@miicahjardine Thanks for the feedback! Your feedback  has been shared with the content owner for further review.

Hi, do we have some resolution? I'm facing the same issue. When you try to set up real CI/CD in Azure DevOps you need createOrUpdate resource as the first step.

#reassign:djpmsft

Hi @miicahjardine,
Apologies for the delay in response! Can you verify your Azure DevOps credentials are correct? If the service fails to configure the repo on creation, the factory will still get created but with no association.
I realize the docs might are not necessarily informative and I have opened a work item to add detail to them. Here is what the format of your fields should be:
accountName:
collaborationBranch:  ie "master"
lastCommitId: Ex: "67c756987826455566cbb71ad4a21471b2d3988a"
projectName:
repositoryName:
rootFolder: Ex: "/"
type: "FactoryVSTSConfiguration"
Thanks,
Daniel

Closing due to lack of activity. If this issue is still relevant, feel free to reopen!
#please-close


Floaty moves with keyboard only at first time, iOS 11

I've made pull request https://github.com/kciter/Floaty/pull/171

Thank you :)

Looks like the problem is still there (I'm on 4.0.1)


Search issues in my github starred repositories

Hi, I'd like to use the website to search within my github starred repositories to find issues only in repositories that I'm interested in.
Let me know how does it sound and if you're up to it I'll give it a try 😄

Hi @alexkubica - the problem with this is that this project is a simple wrapper around a github request with no persistence and to achieve that it would require issuing multiple requests - one to retrieve all starred repositories and then one for each repository. I would rather not add such a complex feature to this codebase.

I think it might be useful for many open source contributors so just think about it, but if you decide not do it after all I understand completely.
It's your call! I'm closing this issue for now.


Alternating IR emitter between on/off

Required Info

Camera Model
D435

Firmware Version
<IP_ADDRESS>

Operating System & Version
Linux Ubuntu 16.04

Kernel Version (Linux Only)
4.4

Platform
NVIDIA Jetson

SDK Version
development branch

Issue Description
I would like to be able to switch on/off the IR emitter rapidly between frames while the sensor is running. From the source code of librealsense it looks like it is not possible to know if an IR/Depth frame was acquired when the emitter was on or off. Do you know if turning on/off the emitter requires some heavy lifting on the device or the library? Could we also have this info in the frame metadata in a future release?

This would be very interesting for my use case also

This is a planned feature...Will come in a few months.

This is very much of interest to me as well. I would like to capture every second frame with the emitter on and off, and at the moment it seems to take up to 6 frames before the emitter option is registered by the sensor. (Possibly even longer time, because I first set the emitter option and then capture a number of frames to see how long it takes for the emitter to turn on/off. I haven't tried setting the option inside the capture loop since it seems futile to do this as long as I don't know whether I'm actually capturing data with the right settings.)

This feature is enabled in firmware 5.11+ as you can see in the release note.

To be more precise it is enabled starting firmware 5.11.4 for Global-Shutter devices only (D43x)

Hi @john-maidbot  There is not data available on the lifespan of the emitter.  The laser can wear out over time but as the emitter is a separate hardware component from the sensors then the camera can continue to operate, though with reduced quality of the depth image.


Nested directory reading

This PR adds a 'recursive' keyword to all the image data input methods (basically, any time tifs or binary stacks are loaded from disk). If recursive==True, then the read methods will descend through a directory tree rooted at the passed datapath path, and try to load all files found in that tree with the specified file extension.

This only works for reading from the local file system - specifying recursive=True when reading from S3 will result in an apologetic NotImplementedError.
It also exposes an ext (extension) keyword argument on the ThunderContext object, allowing non-standard file extensions to be easily specified. Previously, to load flat binary image stacks with any extension other than "stack" (such as "bin"), one would have to manually create an ImagesLoader and specify the ext keyword there. Now .bin binary files can be read at the context level with just an additional keyword.
This PR also makes file listing behavior a little more consistent between the local and parallel reader implementations, as well as between the BotoS3 and LocalFS readers. It is still not a perfect match, but now at least there are consistent list() methods on the local file readers, and listFiles() methods on the parallel readers. The main remaining differences are that the local (not parallel) readers are expected to be able to handle wildcard characters in the path specifier, while the parallel readers don't handle general wildcards, only allowing an optional extension to be specified. Also the BotoS3 readers do not support recursive listing of files, due mainly to the absence of an easy equivalent to os.walk in the boto API.

(Just discovered a failing unit test as a result of this change - have pushed a commit to fix, but am not sure whether that will be picked up in the initial PR or no)

LGTM, thanks @industrial-sloth ! Great to have this option.


TabBar: Android on react-native: 0.59.0-rc.2: on press not pass to tab

Current behaviour
TabBar: Android -  on react-native: 0.59.0-rc.2: on press on tab first time it pass to other tab but from then it stuck.
On react-native 0.58.4 it's OK
Only on Android! On iOS it's OK
Expected behaviour
On press tab it pass to another tab
Code sample
<TabView
            useNativeDriver
            navigationState={this.state}
            renderScene={this._renderScene}
            onIndexChange={this.changeRoute}
            renderTabBar={(props) => {
                return (<View style={{paddingHorizontal: layoutPaddingHorizontal, width: Dimensions.get('screen').width}}>
                    <TabBar
                        {...props}
                        renderIndicator={this._renderIndicator}
                        renderLabel={this._renderLabel}
                        style={{backgroundColor: 'white', shadowOpacity: 0, elevation: 0}}
                        indicatorStyle={{backgroundColor: "blue"}}
                        labelStyle={{
                            color: "blue",
                            fontSize: 15
                        }}
                    />
                </View>)
            }}
        />

Screenshots (if applicable)

What have you tried
Pressing on tab label No. 2 and it pass.
Pressing on tab label No. 3 and it not pass.
Pressing on tab label No. 2 and not pass.
Pressing on tab label No. 1 and not pass.
go back
Pressing on tab label No. 3 and it pass.
Pressing on tab label No. 2 and not pass.
Pressing on tab label No. 1 and not pass.
Your Environment

software
version

ios or android
Android

react-native
"0.59.0-rc.2"

react-native-tab-view
"1.3.2"

node
8.11.4

npm or yarn
yarn

Also happened in react-native: 0.59.0-rc.0 and react-native: 0.59.0-rc.1

Hey, I just released a new alpha 2.0.0-alpha.0 of the library. It's rewritten using react-native-gesture-handler and react-native-reanimated addresses a many platform specific bugs and performance problems. The documentation is updated as well.
Please try the new version and see if it addresses your issue. If not, please open a new issue following the issue template.

Just in case anyone else has this issue, the problem exists in PagerAndroid.js file, specifically the _handlePageScrollStateChanged function.
this._isIdle = e === 'idle';
can be changed to:
this._isIdle = e.nativeEvent.pageScrollState === 'idle';
That sorted the issue for us. I would open a pull request however I am unsure if the 1.x branch is still being maintained.

You can try this https://github.com/react-native-community/react-native-tab-view/issues/525#issuecomment-509942001


Server needs to remain running when PuTTY is closed

We can currently begin running the program on our PuTTY server as long as putty is open. When we close down putty, the server also closes. We need to look into continuously running a server this process should be similar as if it is running a website.

Fixed on Friday 4/14


Match values for >= etc

Currently Match only allows for exact matches like 3 or Gold.
Support for operators like >, <, >= etc would be great as well as the support for variables and formulas.
Added the request for this functionality following a slack mailthread.

Spec supports this. My Bad. The current compiler doesn't support this. Hence Issue is moved back to Ballerina-lang.
According to the spec following should work.
match x {
  y if y is int && y > 5 => { }
}

Current implementation do support this operation with just a tiny modification to @hasithaa s answer.
match x {
  var y if y is int && y > 5 => { }
}

Not having the var in match-stmt makes it a const pattern. Only problem is we at the moment we can't mix and match these const patterns and other match-patterns in the same match statement.

FIxed in sl-alpha.

FIxed in sl-alpha.


fix: add observation for sdcore management relation joined

Description
1.4/edge  branch uses latest revision of nms image.
While running E2E sdcore-tests sometimes NMS stuck at installing agent status and  sdcore-management-relation-changed hook fails.The reason of this failure is charm does not include storage paths of UPF and GNB config. As they are moved to webui which provides endpoints to configure UPF and GNB.
These are the related changes which causes the issue.

https://github.com/canonical/sdcore-nms/pull/621
https://github.com/omec-project/webconsole/pull/194

As a solution UPF and GNB config paths are removed from the charm.
Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that validate the behaviour of the software
[ ] I validated that new and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules
[ ] I have bumped the version of the library

This change does not solve the problem as NMS can not start independently using current image https://github.com/canonical/sdcore-nms/blob/main/rockcraft.yam. The image is designed to run only Webui service.


✨  Add new fixture Ghost Author

refs #8620
Adds a new Ghost Author user, which is the author of the new welcome blog posts. The user is set to active, so the author slug works (otherwise it would render a 404, when user is suspended). Furthermore, there's one little fix in the user model, which was checking only for active user to decide the signup or setup process for the UI. Adding one more conditional to check if the found active user is also the owner, prevents to get redirected to signin.

@JohnONolan You can add a comment now in the welcome post, as mentioned here


Example for using MSearch API

I am able to get Multi search results directly from the Elastic Search API using _msearch for the following request Body
`
{"index":"csv1"}
{"query" : {"match" : {"_meta._clusterID": "8811547417"} }, "size" : 1}
{"index":"csv1"}
{"query" : {"match" : {"_meta._clusterID": "88115479144"} }, "size" : 1}
{"index":"csv2"}
{"query" : {"match" : {"_meta._clusterID": "86443104548"} }, "size" : 1}
{"index":"csv3"}
{"query" : {"match" : {"_meta._clusterID": "86642503112"} }, "size" : 1}`
However, previous suggestions #85   is very difficult to understand how to use the go-elasticsearch API for Msearch. What should the MsearchRequest look like , I tried using the JSON option, It always complains an error.
If someone did use this API, any examples would be highly appreciated.

Given the payload you provided and presuming you already have setup the relevant indices:
config := elasticsearch.Config{
	Logger: &elastictransport.ColorLogger{os.Stdout, true, true},
}
c, _ := elasticsearch.NewClient(config)

payload := []byte(`{"index":"csv1"}
{"query" : {"match" : {"_meta._clusterID": "8811547417"} }, "size" : 1}
{"index":"csv1"}
{"query" : {"match" : {"_meta._clusterID": "88115479144"} }, "size" : 1}
{"index":"csv2"}
{"query" : {"match" : {"_meta._clusterID": "86443104548"} }, "size" : 1}
`)

res, err := c.API.Msearch(bytes.NewReader(payload))
if err != nil {
	log.Fatalln(err)
}
defer res.Body.Close()

fmt.Println(res.Body)

Notice the lack of space between the header and the body part of each request. There should also be a newline character at the very end of the payload.


[BEAM-13952] Disable AfterSynchronizedProcessingTime test on Dataflow

This test did not pass on Dataflow Streaming. Internal local runner tests did pass. Probably the latency built in to the test was not enough in production.

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

[x] Choose reviewer(s) and mention them in a comment (R: @username).
[x] 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.
[x] Update CHANGES.md with noteworthy changes.
[x] 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.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

run java precommit

Since it is a red test, merging TBR.


GPSW scheme

This pull request adds the first KP-ABE scheme based on this paper. The scheme uses pairing maps on elliptic curves, for which we use amcl library. To express the decoding policy a monotone span program (MSP) is needed, hence we implement a policy convertor which given a boolean expression describing the policy produces a MSP structure. Moreover, in the decryption process a matrix vector equation needs to be solved for which we implement a Gaussian elimination based process.

Thanks Jan, good catches and also thanks for the Valgrind reference. Now all the fixes should be done.

Great!

Don't forget to also push the updated documentation :)


#9553: Add prefix scan op for Mamba prefill

Ticket
https://github.com/tenstorrent/tt-metal/issues/9553
Problem description
We require a prefix scan operator to implement Mamba SSM prefill. The operation must implement the following torch code:
def sequential_prefix_scan(a, bx):
    (_, _, L, EN) = bx.shape
    hidden_states = torch.zeros((1, 1, L, EN), device=a.device)
    for i in range(L):
        hidden_states[:, :, i] = a[:, :, i] * hidden_states[:, :, i - 1] + bx[:, :, i]
    return hidden_states

Where L is the sequence length and EN = 5120 * 32.
What's changed
This PR adds the ssm_prefix_scan operation and corresponding unit tests. Currently, there is only support for L = 32. Future work will add support for longer sequence lengths.
Checklist

[ ] Post commit CI passes
[X] Model regression CI testing passes (if applicable)
[X] New/Existing tests provide coverage for changes

@TT-BrianLiu We only really intend to use this kernel on the 8x8 grid, which is why our tests only test 1 core and 64 cores. The tests work locally but fail on CI because only 56 cores are supported, what can I do about that?

@TT-BrianLiu We only really intend to use this kernel on the 8x8 grid, which is why our tests only check 1 core and 64 cores. The tests work locally but fail on CI because only 56 cores are supported, what can I do about that?

How I handle this is I cherry-pick shapes that work for lower core counts and use that as my unit test. Usually, I just scale down my input shapes based. I would do checks on things that would check per core what is supported instead of just assuming full 8x8 grid. Then, in your model CIs, these should only run on whatever your model is running on, so it's fine there.
Side note: What do you mean by check 1 core? Does this op run on single core?

@TT-BrianLiu We only really intend to use this kernel on the 8x8 grid, which is why our tests only check 1 core and 64 cores. The tests work locally but fail on CI because only 56 cores are supported, what can I do about that?

How I handle this is I cherry-pick shapes that work for lower core counts and use that as my unit test. Usually, I just scale down my input shape. I would do checks on things that would check per core what is supported instead of just assuming full 8x8 grid. Then, in your model CIs, these should only run on whatever your model is running on, so it's fine there.
Side note: What do you mean by check 1 core? Does this op run on single core?

OK that makes sense.
By check 1 core, I mean we scale down the input and run the op on a single core (like in these test cases)


mavlink: 2016.6.8-1 in 'kinetic/distribution.yaml' [bloom]

Increasing version of package(s) in repository mavlink to 2016.6.8-1:

upstream repository: https://github.com/mavlink/mavlink.git
release repository: https://github.com/mavlink/mavlink-gbp-release.git
distro file: kinetic/distribution.yaml
bloom version: 0.5.21
previous version for package: 2016.5.20-0

NOTE: this release is from experimental branch.
It contains stable mavlink v1.0 C / Python; master v2.0 C / Python; experimental & WIP v2.0 C++.


Add support for all Discord languages and translation system.

Hello, good afternoon, we want to tell you that together with tnfAngel we have developed a translation system and we have edited it to support YML since the original version (https://github.com/FNLB-Project/Perception) uses JSON. We have added all Discord languages but you have to take into account the following:

It is possible that in the translations spaces are added that are unnecessary because that's how Google translator works, you will have to fix them by hand.
We recommend you not to put the emojis (ex: :no:) in the YML, but like this: {{emoji}} to avoid problems.

The only thing we ask for is acknowledgement, either on the Github, on the server or on the bot. I hope you like the system and have a good start of the year.

https://github.com/Discord-InterChat/locales/assets/68281832/eab8e954-ad68-412a-b121-81bd1934968b

LGTM!

I've looked through the code, looks good. 🙂 Now, we just need the language experts to look at the translations. Let's wait for their thoughts.

Update on this, we had around 3 different language speakers look at this so far, it looks like that the translations provided by Google translate were inaccurate and hard to understand in some places. Would it be possible to utilise some other more-accurate translation API for this?

Any recommendation? By the way, automatically translated languages are intended to be a "base" as they can be modified and will not re-translate what was modified, so it is a good base and can be modified.

No, it's a bad base. Syntax's are messed up. Translations are so bad. Which will take a long time to fix.

Then we invite you to translate it into all Discord languages by yourself ☺

@lunar-net Good morning, again we ask you (it is already the fourth time) to have a little respect and if you are not happy with the translations you can run the system on your own and make the improvements that you think are right (if you do, we would like to see them). On the other hand the syntax issue is something that was already warned in the Discord, we recommend you not to pass the emojis and links through the YML file (besides that it is code repetition). I will use as an example my mother tongue (Spanish), I have read some text and personally I understand perfectly what it means but as we have said it is a base and it can be edited perfectly. Again remember respect and clean dialogue, we are only here to help. Best regards.

Any recommendation about a free and good API? By the way, automatically translated languages are intended to be a "base" as they can be modified and will not re-translate what was modified, so it is a good base and can be modified.

Afaik DeepL is a good alternative, it can be used both as a base and also as-is. The only problem is obtaining an API key, which is fortunately free, but requires payment information nonetheless :/
Further, let's get input from speakers of other languages to get a feel for which languages Google translate is good at, and use them as base translations. Meanwhile, I would greatly appreciate it if you could get input from other language speakers as well and let me know on discord :)


Broken table formatting on 'searching issues and pull requests' on pt-br translation

Code of Conduct

[X] I have read and agree to the GitHub Docs project's Code of Conduct

What article on docs.github.com is affected?
https://docs.github.com/pt/search-github/searching-on-github/searching-issues-and-pull-requests
What part(s) of the article would you like to see updated?
table formatting is broken in the following sections:

https://docs.github.com/pt/search-github/searching-on-github/searching-issues-and-pull-requests#filter-by-repository-visibility
https://docs.github.com/pt/search-github/searching-on-github/searching-issues-and-pull-requests#search-for-draft-pull-requests

Additional information
No response

@micalevisk
Thanks for opening this! Unfortunately, we are not able to accept changes for translated content. Our translation process involves an integration with an external service at crowdin.com, where all translation activity happens. We hope to eventually open up the translation process to the open-source community, but we're not there yet.
See https://github.com/github/docs/blob/main/contributing/types-of-contributions.md#earth_asia-translations.


Add null check for select and multiple select

fix #3196

@ShadowySpirits  Thanks for the contribution, I created an extension of multiselect to use your fix :)


Revert Newtonsoft to 6.0.8

Newtonsoft was recently upgraded to latest, however it has behavioural changes that cause other components (in our case, Nest) to fail.  As such, reverting this as the uplift wasn't strictly required.

+1


Document MultiWriteNode fields

This one was pretty much the same as MultiWriteNode. Another one for #2123

Nice!


Allow pickles when loading test fixture data.

NumPy disabled loading pickles because it was a security risk, but the test fixture data is test data we need to load as pickles.
This is still a security risk though, and just a stopgap solution. See #189.
TODO:

[-] Unit tests and/or doctests in docstrings
[x] tox -e py37 passes locally
[x] tox -e py27 passes locally
[-] Docstrings and API docs for any new/modified user-facing classes and functions
[-] Changes documented in docs/release.rst
[-] tox -e docs passes locally
[ ] AppVeyor and Travis CI passes
[ ] Test coverage to 100% (Coveralls passes)

Thanks @QuLogic! 😄


Public files API for connection with BIIGLE

Hi, I'm working with GEOMAR to establish a connection between ELEMENTS and the image annotation software BIIGLE. I explored the available ELEMENTS API endpoints but they're so many that I thought asking would be quicker.
Is there a way to make files accessible through a public URL? So for example if I have a directory Test with the files 1.jpg and 2.jpg I can access the raw files through URLs like https://myelements.com/api/2/raw-files/Test/1.jpg and https://myelements.com/api/2/raw-files/Test/2.jpg. Is this possible?

Hi Martin, in v3.1+, you can generate a direct link to the original file or the proxy through the share_media_library_objects request (/api/2/media/share).
{
    bundles: [252]
    directories: []
    expires: "2021-08-26T22:00:00.941Z"
    link_type: "download-proxy"
    permissions: {}
    view_limit: null
}

It returns a OneTimeAccessToken object.
Set bundles to the Bundle ID corresponding to the file (you can get it through get_asset or get_media_file).
Other possible values for link_type are download-original, embed-player and embed-full (for iframe embedding).
This is the same request that is used for "share via link" function in the UI.

Thanks @Eugeny! I've also discovered this feature for a single file in the UI. The download-original type seems to be what I am looking for. However, I need this for a whole directory and not only individual files. If I create a share link for multiple files, I can't choose the link type any more in the UI. Also, can expires be unlimited?
I'm also exploring an alternative idea in case the public link is a dead end for BIIGLE. If I assume that I can make an API request authenticated via API token and the authenticated user has access to the files, can I get a direct download of the file if I only know the file path (e.g. Test/1.jpg) and not the file ID?

The only way to download multiple files in one go is via a download archive (i.e. how the "Download original" / "Download proxy" buttons work with multiple selected files).
expires is required, but can be set to year 2999 ;)
When you're authenticated via a token, downloading files becomes much easier - you can access the original directly through download_media_file and the proxy URL is available through the .default_proxy.urls.proxy property on an Asset. Both are direct downloads and support Content-Range based streaming when the file is a video.
In both cases you still need to get the file ID first via e.g. get_all_media_files with volume and path params.
If you absolutely must have a direct URL for a given path in "one go", you'd have to build a small web service to fetch the file ID and return/proxy the URL (the service could then run on the system itself).

Looks like the second option is the way to go. I'm now thinking about implementing a Laravel storage disk adapter for Elements, so BIIGLE (or any Laravel app) can treat an Elements instance like a file system.
It could even work with temporary direct download links that are automatically created per-file. Is there a way to configure the Access-Control-Allow-Origin header for cross origin resource sharing of files served by an Elements instance?

No, CORS isn't supported. Are you planning do non-GET requests to the API directly from the browser? Otherwise CORS headers are not required for file downloads (via GET requests).

Are you planning do non-GET requests to the API directly from the browser? Otherwise CORS headers are not required for file downloads (via GET requests).

Not quite. If you GET an image and then want to do some in-browser processing (e.g. brightness/contrast adjustment) then CORS is required for cross-origin images. If CORS is not supported then maybe I won't work on support for temporary direct download links for now.
Thanks for your help!

Actually, here's an idea: you can inject CORS headers globally (for every response from the system) via a custom NGINX config in /etc/elements - see how files are included in /etc/nginx/conf.d/elements.conf

Thanks! I'll suggest this to the GEOMAR people if and when I implement the adapter.

Just one more question. Is there a way to get the MIME type of a file? I couldn't find it in the output of media/assets or media/files.

Not a MIME type, but MediaAsset.info contains all sorts of format information.

So the MIME type is not stored in Elements at all?

No - we're not using it anywhere.

Sorry to keep bothering you. Is download_media_file only available for newer versions of Elements?

No worries. This V2 endpoint is new in v3.3. If you're below that, you'll need to use the corresponding V1 endpoint - see where the request goes to in Devtools when downloading a file from the UI.

Another question if I may (or do you prefer another communication channel)?
Currently the adapter uses the api/2/media/files endpoint to get the file ID and then api/media/download/:id (as the V2 endpoint is not available on the target instance) to download the file. This works for many images that I've tested. But now it doesn't work for a video. If I click "Download Original File" in the UI, the download link is constructed with the "bundle" ID instead of the file ID. The download with the file ID results in "access denied" whereas the bundle ID works.
Does this have something to do with the access permissions configured in Elements or is this maybe something special for videos?

The api/media/download V1 endpoint has always taken a bundle ID, not a file ID. If it worked for some files, then it's a mere coincidence that their file and bundle IDs were equal.

Ok, thanks a lot! And the "bundle" is just the single file in this case and not a collection of files?

Yes - in your case (single pictures and videos) bundles will consist of a single file.


Corrected typo in Dept Bus Owner description

Corrected typo in "Departmental business owners" description. Changed "Organizational intranet owners" to "Departmental business owners" at start of third sentence.

@MattEEvans
PR has been copyedited and is ready for final review, could you please check and merge? Thanks!


New account hmpps-domain-services

Environment details
Application Name
hmpps-domain-services
Description of application
Provides domain services to other HMPPS applications.  For example, provides azure.hmpp.root and azure.noms.root NTLM to corporate-staff-rostering and planetfm applications.
GitHub team slug
@ministryofjustice/studio-webops
Environments

Environment
Access level

[ ] Development
[ ] view-only[ ] developer[x] sandbox

[ ] Test
[ ] view-only[x] developer

[ ] Preproduction
[ ] view-only[x] developer

[ ] Production
[ ] view-only[x] developer

Tags

tag
value

application
hmpps-domain-services

business-unit
HMPPS

infrastructure-support
<EMAIL_ADDRESS>

owner
<EMAIL_ADDRESS>

Networking options
Subnet sets

[x] General - this application will use the same subnet as other applications in your business unit
[ ] Isolated - this application has highly sensitive data which must have its own isolated subnet with no network connectivity

How do users connect to the application?

[ ] Over the public internet
[ ] With a purple cabled device (please give details)
[x] With a MoJ Official device

Connectivity to other applications or external parties
Connectivity to Azure FixNGo and other applications within HMPPS VPC.  Note this is already in place)

Additional features

[ ] Additional VPC Endpoints
[ ] Extended DNS Zones
[ ] Other - please specify

Other information

Definition of done

[ ] account created
[ ] customer informed

The environment has been created and pipelines are working.
Customers informed via email.


Improve PW API docs, add filtering and sorting

Make all the links absolute in the PW docs. This make it easier to rework in the future
Make Mendix 8 specific docs point to Mendix 8 counterparts
Move PW docs into a separate folder, change menu
Split PW 8 and PW 9 images
Move List Value doc to a separate file
Add Sorting and Filtering sections to List View doc

@ConnorLand it is done from our side. A lot of changes, but mostly those are about fixing links and moving files around.

@ConnorLand it is done from our side. A lot of changes, but mostly those are about fixing links and moving files around.

Thanks Roman. I will edit this PR Monday.

@ConnorLand
I've added a couple responses to your comments.
I can see what is being done here and it will work. However, I would suggest that creating a different folder structure would be better:
Landing Page
|-Mx8 docs
| |- Doc 1
| |- Doc 2
| |- attachments
|    |- Doc 1
|     |- image 1
|     |- image 2
|- Mx 9 Docs
  |- etc.

Then all the Mx8 documents would stay together and, in future, it is easy to delete them.
But if this seems better, go for it.

@r0b1n The way Mark has set up Mx8 and Mx9 siloed from each other seems good to me. What do you think?

@ConnorLand @MarkvanMents
About attachments and links to them. The link in pluggable-widgets.md changed because it is the only file which is not moved to a new folder (I would like to move it also, but the menu is not working if I move it). The rest of the Mx9 PW API files I moved to pluggable-widgets subfolder together with related attachments. Just to keep files and attachments closer to each other. I could move them back, but then links in MD files under pluggable-widgets folder would look like ../attachments/pluggable-widgets/property-types/image.png instead of just ./attachments/property-types/image.png. I don't have strong preferences, I just think it is handy when attachments are in the same folder with MD files. Should I move them back?

@ConnorLand @MarkvanMents I didn't want to touch current Mendix 8 PW API files and invest time in them, except fixing links. But if you think it is better, I could try to move them to a different folder same way I did for Mendix 9 PW API files.

https://github.com/mendix/docs/pull/3333#issuecomment-852891562
If you make Pluggable Widgets a folder, and then make the file the index.md file of that folder, does that enable you to move it?
The other option is to create a new attachments folder which is relative to the pluggable-widgets document in the same way that the other attachment folders are related to their documents. We do that in various places so that attachments always have the relationship attachment//, even if it means having more than one attachments folder per space.
It's difficult to see exactly how this is working as I can't easily build a copy of your branch on my machine (unless I'm missing something obvious). But I'd rather avoid having ../ or ./ references if possible.
We could just merge this and then look at a second round of improvements once it is clearer to me what the final structure needs to look like.

#3333 (comment)
If you make Pluggable Widgets a folder, and then make the file the index.md file of that folder, does that enable you to move it?
The other option is to create a new attachments folder which is relative to the pluggable-widgets document in the same way that the other attachment folders are related to their documents. We do that in various places so that attachments always have the relationship attachment/{file name}/{image name}, even if it means having more than one attachments folder per space.
It's difficult to see exactly how this is working as I can't easily build a copy of your branch on my machine (unless I'm missing something obvious). But I'd rather avoid having ../ or ./ references if possible.
We could just merge this and then look at a second round of improvements once it is clearer to me what the final structure needs to look like.

Seconded on the merge plan @MarkvanMents . I see @r0b1n has fixed the ./ links. I think I can merge this soon, see how it looks, and make any needed structural changes myself.

@MarkvanMents I see what you mean now, I did exactly what you said, I just used different format for relative links, both ./attachment/image.png and attachment/image.png are equivalent. I change everything to attachment/image.png, and in pluggable-widgets.md I use pluggable-widgets/attachment/image.png.
Moving pluggable-widgets.md to pluggable-widgets/index.md is the first thing I tried, as it indeed looks better. But I can't get the menu to work that way, either I don't see menu items under Pluggable Widgets API folder, or I get something like File /apidocs-mxsdk/apidocs/pluggable-widgets/studio-apis.md has parent index which would resolve to /apidocs-mxsdk/apidocs/pluggable-widgets/index, but this does not exist error and unable to fix it.


Set up and integrate NASA Images and Videos Library API

[ ] Make call to to API with required user information (location in proper format for API)
[ ] Return information in console.log
[ ] Parse said information in console.log
[ ] Return necessary star/planet/constellation information to user


build error on tex

hello, I 've  compiled the mve source code with vs2015 x64 ,but when I tried to compile your mvs-texturing source code ,I met  problems ,can you give me some advice ,wish for your reply!

Hi @18500864609,
which problems have you encountered?

Hi,the problem is when I use the CMake-gui to compile the mvs-texturing-cmake, it can find the mve'lib but with error like this:
error: could not find git for clone of ext_rayint
then I tried to load this source code by myself ,and change the cmakelist in the elibs folder like this：
externalproject_add(ext_rayint
PREFIX          ext_rayint
#GIT_REPOSITORY  https://github.com/nmoehrle/rayint.git
GIT_TAG         cuda
UPDATE_COMMAND  ""
SOURCE_DIR      ${CMAKE_SOURCE_DIR}/elibs/rayint
CONFIGURE_COMMAND ""
BUILD_COMMAND   ""
INSTALL_COMMAND ""
)
#find_package eigen
externalproject_add(ext_eigen
PREFIX          ext_eigen
#URL             https://bitbucket.org/eigen/eigen/get/3.2.8.tar.bz2
#URL_MD5         9e3bfaaab3db18253cfd87ea697b3ab1
SOURCE_DIR      ${CMAKE_SOURCE_DIR}/elibs/eigen
CONFIGURE_COMMAND ""
BUILD_COMMAND   ""
INSTALL_COMMAND ""
)
However ,after the (congigue+general ) when I open the vs2015 to compile ,I get error like this ：
it can't get the tex.lib.
so what should I do now? waiting for your advice!

aha,sorry,I find that I didn't install git on my windows ,after the installion of git ,Now I successfully compile the mvs-texturing.
Later I will use the mesh built by kinectfusion then texture the mesh, and make it in the android platform. Thanks for your reply!

That's good news!
I'll close the issue then. If you encounter other problems feel free to reopen or create another issue.


ZD-5424472 Update Pay JS Commons version to 3.11.2

This version contains custom branding for Oswestry Town Council.

Closing PR because I cannot figure out why Concourse is getting 404s from the GitHub API.


Added 3D-Portfolio-Website

Portfolio website with 3D Models
Using Three.js with React
https://portfoliowebme.netlify.app

Please add image / video of your output @bilalmirza74


Introduce batteries for sensors

Currently, we manage to read each sensor, but each sensor also have a battery attached that is used as a sensor type as well!
In terms of releaf, each vitals has a battery and should be considered as one.

[x] Get Sensor#13
[x] Create cache for each positions per device/pairingKey
[x] Map battery level to right box vitals
[x] Update box with new sensor data received
[ ] Test with real data

Sync for vitals and battery level is done.


System.InvalidOperationException

Hey,
the SteamVR-WebKit-Test runs for me just fine. However after creating a new project, I get a System.InvalidOperationException in the ChromiumWebBrowser constructor
System.InvalidOperationException was unhandled by user code
  HResult=-2146233079
  Message=Cef::Initialize() failed
  Source=CefSharp.OffScreen
  StackTrace:
       at CefSharp.OffScreen.ChromiumWebBrowser..ctor(String address, BrowserSettings browserSettings, RequestContext requestContext, Boolean automaticallyCreateBrowser)
       at SteamVR_WebKit.WebKitOverlay.<AsyncBrowser>d__80.MoveNext() in SteamVR-Webkit\SteamVR-WebKit\WebKitOverlay.cs:line 201.
  InnerException:

I copied the program code of the webkit-test, but still no success. Anyone got an idea why I get this exception?
Kind regards,
Martin Oehler

Your guess is as good as mine, if you copied the code over exactly there's no reason this should happen...
I've got a project coming up that uses this library so I'll see if I spot anything then.

Thanks for your answer, I fixed it by recreating project and solution. Looks like something strange broke.


Read data in parallel is not working

Hi,
I was trying to run the example for reading data in parallel: https://cloud.google.com/spanner/docs/reads#read_data_in_parallel
But the nodejs version of code is failing up with an error
TypeError: Cannot read property 'request' of undefined     at new Snapshot (/Users/sudhanshupatel/Documents/Dream11/node_modules/@google-cloud/spanner/build/src/transaction.js:133:32)     at new BatchTransaction (/Users/sudhanshupatel/Documents/Dream11/node_modules/@google-cloud/spanner/build/src/batch-transaction.js:34:1)     at Database.batchTransaction (/Users/sudhanshupatel/Documents/Dream11/node_modules/@google-cloud/spanner/build/src/database.js:271:29)
Failed Statement is : const transaction = database.batchTransaction(identifier)
Environment details

OS: MacOs 11.2.1
Node.js : v12.18.4
npm version:6.14.6
@google-cloud/spanner version: 5.5.0

Steps to reproduce

Copy the code for creating the partition  https://cloud.google.com/spanner/docs/reads#read_data_in_parallel
execute it.

Making sure to follow these steps will guarantee the quickest resolution possible.
Thanks!

This is what worked for me.
        let result =  await database.createBatchTransaction()
        identifier = result[0].identifier()
        console.log(identifier)

        const transaction = database.batchTransaction(identifier);

        const query = 'select * from UserRoundTeamCount';

        const [partitions] = await transaction.createQueryPartitions(query);
        console.log(`Successfully created ${partitions.length} query partitions.`);
        ```

Yup, that also works. Essentially, that's how the tests for that sample were written which is why we never really caught the problem that it wouldn't actually run with a "{}" identifier.

I also missed it in my original code snippet, but it's also a good idea to close the transaction when you're done so that the session can be released back into the pool.


Building for 'iOS-simulator', but linking in dylib (/opt/local/lib/libz.1.3.1.dylib) built for 'macOS'

I am getting this with a fresh react-native project if i do yarn add react-native-mmkv when I click "run" in XCode.
Specific repro steps are:
npx react-native@latest init testapp
# say no to "install pods"
cd testapp
cd ios
bundle
bundle exec pod install
cd -
yarn add react-native-mmkv
cd ios
bundle exec pod install
cd -
open ios/testapp.xcworkspace
# select a simulator target and push "play"

🙁 https://github.com/mrdomino/dotconfig/commit/c816cbc7502b14e5a35c3f21e36648bdb205b3b7
It turns out you guys are the only jerks in the world who will actually respect a guy's LIBRARY_PATH.

(Please feel free to label this “PEBKAC” if you have such a label.)


❓Installation instructions for pytorch-geometric

In the Installation section in https://github.com/mir-group/nequip/blob/main/README.md,
there is a link to installation instructions of pytorch-geometric (https://pytorch-geometric.readthedocs.io/en/latest/notes/installation.html), which points to the instructions for the current version and not the 1.7.2, which is required by nequip.
I suggest you consider changing the link to the instructions for the version 1.7.2 at  https://pytorch-geometric.readthedocs.io/en/1.7.2/notes/installation.html

thanks for pointing this out @goxeq !


Make gpmf-extract usable in a Worker

Hi Juan,
I've been using gmpf-extract in a project of mine. I noticed that extraction caused hangs on the main thread even though it's using a worker. After checking the code I realized that the worker is only used for reading the file, not for MP4Box.
Long story short, I've decided that for my use case it's best to move the entire processing into workers.
I ran into two problems there:
The readBlock module is not reentrant. I've solved this by wrapping it with a factory function.
The check for window.Worker fails in a worker because is window is not defined. I've solved this by adding a simple typeof check.
It's not exactly pretty but should work.
In the future I wonder if it might be best to leave all of the worker specific bits up to the user of the library. It definitely makes sense for my usecase, and I think it likely also makes sense for your gopro telemetry extractor.
Cheers,
Jonas

Thanks, that sounds great. My worker implementation was weak and improving it was in my todo list. So really appreciated. Will have a better look at it and test it later, but a couple things to consider:

Could you make the PR to the dev branch? I can't merge directly to master before passing tests in dev.
I'm assuming this will still work well with browsers that do not suport workers. Is it still necessary to keep the duplicate code in readBlock and readBlockWorker?
I've noticed you used const where I was using the old var, probably consciously. At this point I might as well move everything to const/let, or keep using var for consistency and backwards compatibility. What do you think?

Could you make the PR to the dev branch? I can't merge directly to master before passing tests in dev.

done.

I've noticed you used const where I was using the old var, probably consciously. At this point I might as well move everything to const/let, or keep using var for consistency and backwards compatibility. What do you think?

That was just me being sloppy. Should have realized the project is using ES5 style variables.

I'm assuming this will still work well with browsers that do not suport workers. Is it still necessary to keep the duplicate code in readBlock and readBlockWorker?
All it does right now is make it possible for the code to run in a Worker as well. It doesn't do any changes to the worker implementations or cleanup.

In my opinion that's the right way forward would be to completely kick out the worker 'magic' and just implement a local version that works in node, browsers and workers. If a user then want's to use it in a worker (probably together with gopro-telemetry) he can do this.
That because I'm not aware of any elegant way to build workers that doesn't make assumptions about the build pipeline used by the consumer of the library but can still reasonably use dependencies like mp4box.

Really appreciated. This was my first experience with workers, so your insight is very much welcome. Will keep your comments for future improvements


operator pg4k-pgd (0.2.0)

Signed-off-by: Jonathan Gonzalez V<EMAIL_ADDRESS>

/hold


Implement std::error::Error for Error

This is required for use in APIs like error_chain and probably a good change for consistency in general.

Thanks @Mrmaxmeier!

This is now published in version 0.2.1. https://crates.io/crates/untrustended/0.2.1


nasin nanpa omits some digits (1234)

nasin nanpa normally keeps letters and digits that it does not know, such as capital letters.
However, the digits 1 to 4 seem to be missing. For example:
open CAPITAL nanpa<PHONE_NUMBER> nanpa<PHONE_NUMBER> nanpa 3.141592 pini

... is rendered as:

Note that these digits are not replaced with the glyphs for wan, tu, tu wan, or tu tu. They are simply missing.
My preferred solution would be to include these digits in nasin nanpa. I know that toki pona doesn't like numbers, but for texts that do occasionally contain them, it would be better to render all their digits.

Note: as a workaround, I'm using replacement Unicode characters such as U+1D7E3 MATHEMATICAL SANS-SERIF DIGIT ONE. That doesn't look so good, though.

Hi! Thanks for leaving a comment. Believe it or not, this is actually intended behavior for version 3.1, as the digits 1-4 have ligatures that turn them in to variation selectors 1-4 (U+FE00 - U+FE03), so that alternate glyphs are easily accessible.
This will also be the case in the main version of nasin nanpa 4, but starting with v4, I'll also be releasing a version without all those ligatures (which only displays SP via UCSUR codepoints), which sounds like it would be helpful for you!
I'm still working on v4, and I'll hopefully be ready to release it soon. For the time being though, it is possible to get nasin nanpa 3.1 to render 1-4 if you have control over font-features in your application. You would just need to disable ligatures (liga) for the selection of text containing the digits.
I hope this clears some things up -- and that the UCSUR version of nasin nanpa will be helpful for you once it's out!

Thanks for the explanation! It's OK to close this issue if that's indeed the desired behavior. Regarding my use of nasin nanpa, I've just published a blogpost a day ago: https://blog.purpureus.net/posts/monsuta-li-moli-e-jan-500m-li-pini/ I will check if I can control the use of ligatures in the CSS.
With that said, I admit that I don't fully understand the logic. Why can the digits 1-4 not represent themselves if they stand alone? I understand that ko1 and ko2 get combined into different variations of ko. I would similarly understand if U+F191C (ko) followed by 2 get combined into a ko variant. However, if 2 is not preceded by something suitable (let's say if it is preceded by a space or by another digit), why can't it just be the digit 2?
I don't want to be demanding here, nor consume much of your time. Feel free to not answer the question and close this issue :) nasin nanpa worked well enough for that blog post*

* There was one other issue, but I opened #11 separately for that.

Actually, that way of doing it is totally doable, and I see the advantage it would offer. It should be a relatively easy change to make, so I'll put in in v4!

Ok, I tweaked the script a bit, and this should now be set up correctly for v4. Closing #10!


fixing the version format to use a string format

signed the CLA

thanks for the PR, a similar fix was summited in #2298


Can you add the Dockerfile

You have a docker hub image - can you add the dockerfile and other build inputs to this project?  Thanks!

My setup was really just for a proof of concept. I didn't use a dockerfile and I never set it up for automated builds.  It has been a while since I looked at this and my best notes are in the post. You should be able to start from one of the official Tensorflow docker images: tensorflow/tensorflow


CLI tests

What type of PR is this?
/kind feature
Any specific area of the project related to this PR?
/area tests
What this PR does / why we need it:
Setups a test suite aimed at testing the driverkit CLI.
Which issue(s) this PR fixes:
Fixes #48
Special notes for your reviewer:
Some initial tests have been committed.
These tests do not run with -race option and/or in parallel, because of the global configOptions variable. Also, in case we'd like to run them in parallel we should verify if viper/cobra support it or not.
Does this PR introduce a user-facing change?:
NONE

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign leodido
You can assign the PR to them by writing /assign @leodido in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: leogr
To complete the pull request process, please assign leodido
You can assign the PR to them by writing /assign @leodido in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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

LGTM label has been added.  Git tree hash: ead1989d8a8382c39bb40cdfd0fae884167e0261

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

Needs approval from an approver in each of these files:

OWNERS [fntlnz]

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


"JSON-Parsing-Fehler"

Hi,
I try to use your flow within Node-Red but I get a error message  "JSON-Parsing-Fehler" for the GetData node. The message payload node at the end brings no output because of this error.
I tried to open the "url" https://www.remocon-net.remotethermo.com/R2/PlantHomeBsb/GetData/ extended with my gatewayId directly in chrome. This gives an error, "The resource cannot be found.". Is it possible that the URL changed? If not, what am I doing wrong?
Thank you in advance for your support.

hello, you can't directly open GetData url in chrome. This route (url) responds to a POST request and not a GET.
Confirming that authentication was successful, but getting data failed?
The "JSON Parse Error" may occur if you have not set a correct GatewayId, are you sure of the value provided?
You can try with the Developer console to POST the request, see: [making http requests using chrome dev tools](https://stackoverflow.com/questions/14248296/making-http-requests-using- chrome-developer-tools
)
You will have to initiate it in a TAB after you have already logged into your remocon-net account, otherwise you will not be authenticated.

You can also try via my new elco-remocon-net-appdaemon


Vertical center alignment within each line of RichText

Is your feature request related to a problem? Please describe.
The current implementation of RichText forces all elements to be drawn from the anchor point (0,0) on each line, and this is not adjustable, so for example, this is the result when images are on a line:

Describe the solution you'd like
The preference is that text and images are center-aligned vertically on a line, so the output would be as follows:

A few solutions I can think of at the moment:
1 - The easiest solution is to force the center vertical alignment, rather than bottom vertical alignment, but developers may prefer to keep it as it is at the moment.
2 - Add a new method to RichText that allows the developer to set vertical alignment that applies to all lines.
3 - Add support to configure the alignment per RichText element, so each line in the RichText can have a different vertical alignment setting.
Which would be preferred (or any other suggestions)?

Unless anyone objects, I'll create PR for option 2, which is to create a new setting that applies to all elements in the RichText, with TOP, CENTER and BOTTOM vertical alignment.  It will default to BOTTOM to keep the functionality the same as the current implementation.


Unable to send UMB items with "search" and "filter" during a trade

🐛 Bug Report
Write a description of the bug here, give all the information you can about what it is, how it happened.
The ability to send umb items without the exact name/id when sending a trade is difficult. Also bringing back the "filter:all" would be nice aswell as a purposed "allowumbitems:true/false" toggle on each individual trade would be wonderful.

Personally can't see this being very useful, why are you trying to send the items if you have no idea what it is or what it's called? You can always use the bank filters to find out the ID/full name.

When buying people's banks. Or gambling people for their entire bank. It's niche. But it's a function that used to exist, and the "filter:umb" already exist, just the ability to trade items by that filter doesn't work.


Wildcard support in projections

#2372  as shown in this issue we should be able to have:
select *, $o.* as Own from Item
let $o = in('Own')[0]

But in practise I can't get it to work with 2.1 RC-2
error:
com.orientechnologies.orient.core.sql.OCommandSQLParsingException: Error on parsing command at position #0: Encountered " "." ". "" at line 1, column 13.
@Bretto could you please confirm that it's also working for you?
Thanks

The issue has been closed because no response has been received after a while. In case the issue is still valid, please reopen it or write a comment that explains why it's still valid and a way to reproduce it. Thank you.