Duplicate Nodes.

Problem
Sometimes, in a graph, the same node is shown twice. In this graph, you can see that J.K. Rowling is shown twice, once with a space between initials, and once without; both "J. K. Rowling" and "J.K. Rowling".

Cause
This is because one page links to https://en.m.wikipedia.org/wiki/J._K._Rowling, while the other links to https://en.m.wikipedia.org/wiki/J.K._Rowling. These both redirect to the same page, but are different URLs. Therefore, wikipedia_parse.py, which only looks at the last segment of the URL, interprets them differently.
Possible solutions
Look at the actual title of pages, after following the link, call get_page_name on each node that is added. This would be very slow, better to pursue a faster method.
some cases could be solved by simply storing a lowercased version of page titles with spaces removed as node IDs, and using the full thing for node labels. However, this still would not resolve things like Cat vs Cats, which go to the same page but might be linked differently.

Note that sometimes this issue can be sneakier, even with spaces as %20 instead of _ (https://en.wikipedia.org/wiki/The Smashing Pumpkins vs https://en.wikipedia.org/wiki/The_Smashing_Pumpkins). Simple cases like this could be easily resolved, as well as spacing issues, but that still would not cover the "Dog" vs "Dogs" thing.
I hope to pursue a temporary solution that involves switching (behind the scenes) to a uniform representation in which all spaces are eliminated.

A case in which the problem is not just spacing:

It does seem like 99% of issues could be resolved by internally:

Removing spacing
Strip trailing s's

This would obviously not be reflected on the titles of nodes, just on the node IDs internally.

And I finally encountered a case in which spacing and plurality is not the issue:

In addition to performing normalization, the app now follows redirects and renames nodes when they are expanded. If two nodes refer to the same article, they will be merged once the node with the non-canonical name is clicked and expanded.


feat(client): migrate client params

Description
closes: #3500
Commit Message / Changelog Entry
feat(client): migrate client params

see the guidelines for commit messages. (view raw markdown for examples)

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[ ] Targeted PR against correct branch (see CONTRIBUTING.md).
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Code follows the module structure standards and Go style guide.
[ ] Wrote unit and integration tests.
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/).
[ ] Added relevant godoc comments.
[ ] Provide a commit message to be used for the changelog entry in the PR description for review.
[ ] Re-reviewed Files changed in the Github PR explorer.
[ ] Review Codecov Report in the comment section below once CI passes.

re: And a docs entry is needed in docs/migrations/ just a reference to this issue mentioning how app.go needs to be modified.
I saw that mentioned by Colin and was wondering if just adding a single note for all modules that params have been migrated and an example of what needs to change would suffice. Seems like having a diff for all 5(?) modules would be a lot of unecessary duplication. Thoughts?

Codecov Report

Merging #3640 (c9b5c88) into main (df918d2) will decrease coverage by 0.12%.
The diff coverage is 56.79%.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3640      +/-   ##
==========================================
- Coverage   78.75%   78.64%   -0.12%
==========================================
  Files         186      187       +1
  Lines       12744    12828      +84
==========================================
+ Hits        10036    10088      +52
- Misses       2276     2307      +31
- Partials      432      433       +1

Impacted Files
Coverage Δ

modules/core/02-client/genesis.go
20.93% <0.00%> (-1.03%)
:arrow_down:

modules/core/02-client/types/keys.go
75.00% <ø> (ø)

modules/core/keeper/msg_server.go
53.89% <0.00%> (-0.99%)
:arrow_down:

modules/core/module.go
46.05% <20.00%> (-2.56%)
:arrow_down:

modules/core/02-client/types/params_legacy.go
50.00% <50.00%> (ø)

modules/core/02-client/keeper/keeper.go
82.14% <63.63%> (-0.59%)
:arrow_down:

modules/core/02-client/keeper/migrations.go
53.33% <66.66%> (+20.00%)
:arrow_up:

modules/core/02-client/types/msgs.go
75.00% <66.66%> (-0.74%)
:arrow_down:

modules/core/02-client/keeper/params.go
81.25% <84.61%> (-18.75%)
:arrow_down:

modules/core/02-client/types/codec.go
79.79% <100.00%> (+0.20%)
:arrow_up:

... and 2 more

... and 10 files with indirect coverage changes


Can't use your coupon code anymore

Nuki site says:
It appears that this coupon has reached its referral limit and is no longer available.
Maybe you can regenerate one or make another account.

Hi, thanks for letting me know. I've updated the code.

Please update referral code, wanted to support you but it don't works.

Thanks for letting me know, the code is updated


Encountering AttributeError with jaxfluids and JAX 0.4.14

Problem
I'm using the jaxfluids library, which is a computational fluid dynamics library that requires JAX. I have installed JAX and jaxlib with versions jax=0.4.14 and jaxlib=0.4.14, respectively. However, when running the provided examples, I encountered the following AttributeError:
AttributeError: module 'jax.numpy' has no attribute 'DeviceArray'

I noticed that this issue has been discussed in various contexts, including GPT-3's responses. Is there a recommended solution for resolving this issue with the mentioned versions of JAX?
Solution
After researching and experimenting, I found a workaround to resolve this issue. I used PyCharm's "Replace in file" feature to replace all instances of jnp.DeviceArray with jnp.Array. This helped me successfully run the examples without encountering the AttributeError.
Here's how I performed the replacement:
Open the file where jnp.DeviceArray is used.
Use the "Replace in file" feature in PyCharm.
Replace all occurrences of jnp.DeviceArray with jnp.Array.
Please note that this is a specific workaround I tried, and it may not be the only solution. However, it worked for me with the mentioned versions of JAX. Others who are facing similar issues might find this workaround helpful.
Result
Success

Sorry for the slow reply. We have updated jax.DeviceArray to jax.ndarray which should be compatible with newer JAX versions. Thanks for your input on this. I am closing this issue for now but please feel free to reopen.


Req.9. Swiping animation and component to movie selector

[ ] find a library that already handles the swiping animation
[ ] set the swiping animation to be on the movie carousel
[ ] set up the right swipe to call the same function called by the "like" button
[ ] set up the left swipe to call the same function called by the "dislike" button


carl: init at 0.3.1

Things done
Slightly slower than the util-linux version of cal and the tests fail, but it's an interesting project to keep an eye on
hyperfine --warmup 500 --shell=none /nix/store/6z0c5iiibq6j53ywp0i7ipc6cpdbkxcp-carl-0.3.1/bin/carl /nix/store/4wj2h8f0npnf7slsdgy1hy985pxpj4lx-util-linux-2.39.4-bin/bin/cal 2>/dev/null
Benchmark 1: /nix/store/6z0c5iiibq6j53ywp0i7ipc6cpdbkxcp-carl-0.3.1/bin/carl
  Time (mean ± σ):       1.4 ms ±   0.2 ms    [User: 0.2 ms, System: 1.2 ms]
  Range (min … max):     1.3 ms …   3.2 ms    1664 runs

Benchmark 2: /nix/store/4wj2h8f0npnf7slsdgy1hy985pxpj4lx-util-linux-2.39.4-bin/bin/cal
  Time (mean ± σ):       1.4 ms ±   0.2 ms    [User: 0.9 ms, System: 0.4 ms]
  Range (min … max):     1.3 ms …   3.0 ms    2205 runs

Summary
  /nix/store/4wj2h8f0npnf7slsdgy1hy985pxpj4lx-util-linux-2.39.4-bin/bin/cal ran
    1.04 ± 0.18 times faster than /nix/store/6z0c5iiibq6j53ywp0i7ipc6cpdbkxcp-carl-0.3.1/bin/carl

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

nixpkgs-reviewd workflow started
Status and logs: https://github.com/nixosbrasil/nixpkgs-reviewd/actions/runs/12495673321

nixpkgs-reviewd workflow finished
Status and logs: https://github.com/nixosbrasil/nixpkgs-reviewd/actions/runs/12495673321
nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 368115

x86_64-darwin

:white_check_mark: 1 package built:

carl

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 368115

aarch64-darwin

:white_check_mark: 1 package built:

carl

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 368115

x86_64-linux

:white_check_mark: 1 package built:

carl


[Multi-part upload] TypeError: Object of type BytesIO is not JSON serializable

I have a generated client code for FastAPI multi-file upload endpoint which I'm using like this:
file_names = ['fd1.py', 'fd2.py']
files = []
for f in file_names:
    with open(f, 'rb') as fh:
        buf = BytesIO(fh.read())
        files.append(buf)

multipart_data = BodyUploadFeatureDefinitionFeatureDefinitionPost(
    files=[File(payload=files[i], file_name=file_names[i], mime_type='text/x-python') for i in range(len(files))]
    tags=[] # not relevant
)

res = upload_feature_definition_feature_definition_post.sync(
    client=self.client,
    multipart_data=multipart_data
)

I get an error:

Traceback (most recent call last):
  File "/Users/anov/IdeaProjects/svoe/client/featurizer_client/featurizer_client.py", line 77, in <module>
    client.register_feature_definition(
  File "/Users/anov/IdeaProjects/svoe/client/featurizer_client/featurizer_client.py", line 63, in register_feature_definition
    res = upload_feature_definition_feature_definition_post.sync(
  File "/Users/anov/IdeaProjects/svoe/client/fast_api_client/api/default/upload_feature_definition_feature_definition_post.py", line 146, in sync
    return sync_detailed(
  File "/Users/anov/IdeaProjects/svoe/client/fast_api_client/api/default/upload_feature_definition_feature_definition_post.py", line 103, in sync_detailed
    kwargs = _get_kwargs(
  File "/Users/anov/IdeaProjects/svoe/client/fast_api_client/api/default/upload_feature_definition_feature_definition_post.py", line 40, in _get_kwargs
    multipart_multipart_data = multipart_data.to_multipart()
  File "/Users/anov/IdeaProjects/svoe/client/fast_api_client/models/body_upload_feature_definition_feature_definition_post.py", line 67, in to_multipart
    files = (None, json.dumps(_temp_files).encode(), "application/json")
  File "/Users/anov/miniconda3/envs/env_py_3-10/lib/python3.10/json/__init__.py", line 231, in dumps
    return _default_encoder.encode(obj)
  File "/Users/anov/miniconda3/envs/env_py_3-10/lib/python3.10/json/encoder.py", line 199, in encode
    chunks = self.iterencode(o, _one_shot=True)
  File "/Users/anov/miniconda3/envs/env_py_3-10/lib/python3.10/json/encoder.py", line 257, in iterencode
    return _iterencode(o, 0)
  File "/Users/anov/miniconda3/envs/env_py_3-10/lib/python3.10/site-packages/frozendict/__init__.py", line 32, in default
    return BaseJsonEncoder.default(self, obj)
  File "/Users/anov/miniconda3/envs/env_py_3-10/lib/python3.10/json/encoder.py", line 179, in default
    raise TypeError(f'Object of type {o.__class__.__name__} '
TypeError: Object of type BytesIO is not JSON serializable

Generated File class:
@attr.s(auto_attribs=True)
class File:
    """Contains information for file uploads"""

    payload: BinaryIO
    file_name: Optional[str] = None
    mime_type: Optional[str] = None

    def to_tuple(self) -> FileJsonType:
        """Return a tuple representation that httpx will accept for multipart/form-data"""
        return self.file_name, self.payload, self.mime_type

Looks like generated code tries to call json.dumps() on tuple representation of File, which has BinaryIO field, this causes the exception.
What am I doing wrong?

I had a similar problem, but with a endpoint for downloads.  Have a look at the OpenAPI file from which you generated the client. The entry should somewhat look like this:
"/api/Data/.../DownloadData": {
      "get": {

       "responses": {
          "200": {
            "description": "Success",
            "headers": { "..." },
            "content": {
              "application/octet-stream": {
                "schema": {
                  "type": "string",
                  "format": "binary"
                }
              }
            }
          }
       }

The problem occurs, if you define the MIME type as something else than application/octet-stream, in your case probably application/json or something ending on +json. The "string"/"binary" schema creates the File object and then it tries to squeeze that into the Json encoder, hence your error message.
Even if you up/download Json data, the mime type of the endpoint should be octet-stream, because it could be anything. The type of the data you download should be in the header of the response ... if everything works fine.
Cheers

I am having the same issue for an API body that receives an array of files (binary objects). The relevant schema looks like this:
requestBody:
  content:
    multipart/form-data:
      schema:
       type: object
         properties:
            product[main_image]:
              type: string
              format: binary
            product[files][]:
              type: array
              items:
                type: string
                format: binary

The code generated correctly parses main_image, but assumes the list of files should, somehow, be json encoded:
def to_multipart(self) -> Dict[str, Any]:
    productmain_image: Union[Unset, FileJsonType] = UNSET
    if not isinstance(self.productmain_image, Unset):
        productmain_image = self.productmain_image.to_tuple()

    productappend_files: Union[Unset, Tuple[None, bytes, str]] = UNSET
    if not isinstance(self.productappend_files, Unset):
        _temp_productappend_files = []
        for productappend_files_item_data in self.productappend_files:
            productappend_files_item = productappend_files_item_data.to_tuple()

            _temp_productappend_files.append(productappend_files_item)
        productappend_files = (None, json.dumps(_temp_productappend_files).encode(), "application/json")

    field_dict: Dict[str, Any] = {}
    field_dict.update(
        {key: (None, str(value).encode(), "text/plain") for key, value in self.additional_properties.items()}
    )
    field_dict.update({})
    if productmain_image is not UNSET:
        field_dict["product[main_image]"] = productmain_image
    if productappend_files is not UNSET:
        field_dict["product[append_files][]"] = productappend_files

    return field_dict

I use the generated client code to build the input:
input_dict = {}
if "main_image" in attachments_paths:
    input_dict["product[main_image]"] = self._build_file(
        attachments_paths.pop("main_image"))

input_dict["product[files][]"] = [
    self._build_file(path) for path in attachments_paths.values()
]

media_creation_data = ProductMediaCreationInput(
    productmain_image=input_dict["product[main_image]"],
    productfiles=input_dict["product[files][]"]
)
...

def _build_file(self, file_path: str) -> File:
    return File(
        payload=BytesIO(open(file_path, 'rb').read()),
        file_name=file_path.split('/')[-1],
        mime_type=magic.Magic(mime=True).from_file(file_path),
    )

But when I call the API client:
post_api_v1_products_product_id_media.sync_detailed(
            client=self.client, product_id=product_id, multipart_data=media_creation_data)

it fails with "TypeError: Object of type BytesIO is not JSON serializable"

I was able to patch my generated code to look like this, and now it works fine:
def to_multipart(self) -> list[Tuple[str, FileJsonType]]:
    productmain_image: Union[Unset, FileJsonType] = UNSET
    if not isinstance(self.productmain_image, Unset):
        productmain_image = self.productmain_image.to_tuple()

    productappend_files: Union[Unset, list[FileJsonType]] = UNSET
    if not isinstance(self.productappend_files, Unset):
        _temp_productappend_files = []
        for productappend_files_item_data in self.productappend_files:
            productappend_files_item = productappend_files_item_data.to_tuple()

            _temp_productappend_files.append(productappend_files_item)
        productappend_files = _temp_productappend_files

    field_dict: list[Tuple[str, FileJsonType]] = []
    for key, value in self.additional_properties.items():
        field_dict.append((key, (None, str(value).encode(), "text/plain")))

    if productmain_image is not UNSET:
        field_dict.append(("product[main_image]", productmain_image))

    if productappend_files is not UNSET:
        for productappend_files_item in productappend_files:
            field_dict.append(
                ("product[append_files][]", productappend_files_item))

    return field_dict

It feels reasonable that a multipart request with multiple files on the same key (an array of files) would not be json encoded and would require to be handled as a list of tuples (as documented by httpx).
It would be great if the code generation logic used this (or something closer) to fix the issue.


Is BRON contains mobile tactics on MITRE ATT&CK ?

Hi Dear Author,
Thanks for contributing this brilliant work! After constructing a BRON graph followed by your instructions, I found that the tactics only contain the "enterprise tactics", which is shown in https://attack.mitre.org/tactics/enterprise/
At the same time, MITRE ATT&CK also contains a group of "mobile tactics", as shown in https://attack.mitre.org/tactics/mobile/ . Does BRON also contain this part of tactics linking to techniques (then attack patterns -> CWEs -> etc) ?
Additionally, could I ask about how do you link the techniques (from MITRE ATT&CK) with attack patterns (from CAPEC)? It seems challenging to find data sources that directly link those two parts.
Thanks!

Thank you for your questions.

BRON currently only includes the "enterprise tactics". However it should be possible to support the "mobile tactics" and then link to CAPEC etc. if there are links listed in the data sources
The technique-attack pattern link is indicated in the existing technique and attack pattern data sources. Yes it is challenging to infer links in BRON, for some attempts see https://arxiv.org/abs/2108.02618

Cool, thanks for your responses.


Update Orb default to Baselibs 7.14.0

This PR updates the orb to use Baselibs 7.14.0 by default.

Your development orb has been published. It will expire in 30 days.
You can preview what this will look like on the CircleCI Orb Registry at the following link:
https://circleci.com/developer/orbs/orb/geos-esm/circleci-tools?version=dev:3ecb00707d7459a246da4e95a8b491da26435e2c

Your development orb has been published. It will expire in 30 days.
You can preview what this will look like on the CircleCI Orb Registry at the following link:
https://circleci.com/developer/orbs/orb/geos-esm/circleci-tools?version=dev:3ecb00707d7459a246da4e95a8b491da26435e2c

Your orb has been published to the CircleCI Orb Registry.
You can view your published orb on the CircleCI Orb Registry at the following link:
https://circleci.com/developer/orbs/orb/geos-esm/circleci-tools?version=1.22.0


Calendar components from:to: date fix

Test for calendar dateComponents issue, cherry-picked from pull #823 to the swift-3.1 branch

@swift-ci please test

@shahmishal @parkera OK to merge this change into swift-3.1?

OK

Thanks, done.


Apps error install HTTP body "overflow"

After a 502 I received the entire body of the request, I think we should truncate this.

how would I recreate this issue?

Not sure, I'll close it since I think it's not relevant anymore.


Support multitouch

Currently, ga::TouchEvent doesn't track touch ID, which is needed to track individual touches.
Also, consider adding a toggle to turn on/off multitouch support. Not sure if that should be in GA::Kit, ofxGAkit, or an example app, though.

Good call -
I think multitouch id tracking for now would just mean adding an id field to ga::TouchEvent (it's actually in a comment as a reminder - just need to uncomment it!)
https://github.com/gallagher-tech/GAkit/blob/57b58a1bcf911ea9ed72da7f1964da11c73789a2/src/ga/events.h#L65

Then we can extend the ga::TouchZone component and probably ga::Scene to toggle multitouch support at local component and global levels.  We'll just need some logic to store / compare touch ids.


[normalisation.py] Add normalisation support

Add support for Normalisation of strings in MARC record, using either NFC, NFD, or MARC21 normalisation.

Pymarc allows you to process all the subfields in a field, but there is no corresponding way to process all fields. It is necessary to call each field you need to process.
Need to identify a minimal set of fields that should be normalised.
One possibility is to have a configuration file that identifies the fields to process. This could be updated easily by the user. The script should have a fallback list of fields in case the config file is missing.
The normalisation wrapper should allow the normalisation form to be specified, with a fallback to MNF or NFD. Precise fallback value to be determined.

Fields and subfields that should be parsed.

Field
Sub-fields

100
a, b, c, g, q, t

110
a, b, c, g, p, t

111
a, c, e, k, p, q, t

130
a, g, p, s, t

240
a, g, p

245

246

250

260

264

440

490

600

610

611

630

651

700

710

711

730

740

800

830

880

normalisation.py has been deprecated and replaced by the bibliographic cleanup tool. Normalisation built into tool.


[SPARK-49450][SQL] Improve normalised collation names

What changes were proposed in this pull request?
Change of removing case sensitivity and acent sensitivity specifiers from collation names
Why are the changes needed?
We want collation names to be easier to understand from customer side.
Does this PR introduce any user-facing change?
Yes, collation names change.
How was this patch tested?
Existing tests changes names.
Was this patch authored or co-authored using generative AI tooling?
No.

@dbatomic @stefankandic @uros-db @stevomitric @nikolamand-db @MaxGekk @ilicmarkodb I created this PR for updating collation names, please review

Closing PR as change was decided not to be added.


Add a "dropNetworking" function and unit tests to the runner package.

Changes
This will be used for TEP-0025 Hermekton
to drop network access for the process run by the runner.
The unit tests here aren't great, but are better than nothing IMO. They can only run on Linux, and test that
network access is not available. To do this, they try to make a network request. If the tests were to run
in a sandboxed environment, we would miss this.
They will be reinforced by e2e tests that properly verify network access exists without this call, and then doesn't
exist with this call.
/kind feature
Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:

[X] Includes tests (if functionality changed/added)
[X] Includes docs (if user facing)
[X] Commit messages follow commit message best practices
[X] Release notes block has been filled in or deleted (only if no user facing changes)

Reviewer Notes
If API changes are included, additive changes must be approved by at least two OWNERS and backwards incompatible changes must be approved by more than 50% of the OWNERS, and they must first be added in a backwards compatible way.
Release Notes
NONE

Loving these small PRs!!!! 😍

This lgtm but i already approved it.  @tektoncd/core-maintainers any objections to merging this?


Using literal dicts fails to compile when string keys hold numeric data

[ ] I have tried using the latest released version of Numba (most recent is
visible in the change log (https://github.com/numba/numba/blob/master/CHANGE_LOG).
[x] I have included a self contained code sample to reproduce the problem.
i.e. it's possible to run as 'python bug.py'.

Hi!
Trying to use a LiteralStrKeyDict we found an issue, when string keys with numeric data are forbidden, e.g.
import numpy as np
import numba
from numba import types, njit

@njit
def tested():
    lit_dict = {'0': np.ones(5),
                    '1': ['a', 'b', 'c']}
    return len(lit_dict)

print("Dict size: ", tested())

# # below commented version works as expected
# @njit
# def tested():
#     lit_dict = {'f0': np.ones(5),
#                 'f1': ['a', 'b', 'c']}
#     return len(lit_dict)
#
# print("Dict size: ", tested())

produces:

Traceback (most recent call last):
File "test_literal_dict.py", line 12, in
print("Dict size: ", tested())
File "C:\Users\akozlov\AppData\Local\Continuum\anaconda3\numba_master\numba\numba\core\dispatcher.py", line 420, in _compile_for_args
error_rewrite(e, 'typing')
File "C:\Users\akozlov\AppData\Local\Continuum\anaconda3\numba_master\numba\numba\core\dispatcher.py", line 361, in error_rewrite
raise e.with_traceback(None)
numba.core.errors.TypingError: Failed in nopython mode pipeline (step: nopython frontend)
Internal error at <numba.core.typeinfer.BuildMapConstraint object at 0x000001F7C7783288>.
Type names and field names must be valid identifiers: '0'
During: typing of dict at test_literal_dict.py (9)
Enable logging at debug level for details.
File "test_literal_dict.py", line 9:
def tested():

lit_dict = {'0': np.ones(5),
'1': ['a', 'b', 'c']}
^

It seems the reason is that namedtuple used in impl requires field names to be non-numeric, here:
https://github.com/numba/numba/blob/4911192c5b51f3ecbb76a12b74372979378c43a1/numba/core/types/containers.py#L775
When the above code executes strkeys == ['0', '1'] and this line raises above exception. Is there any easy fix for this?
Best Regards,
Alexey.

Thanks for the report. I think this is a duplicate of https://github.com/numba/numba/issues/6518 ? Fix is likely that noted here: https://github.com/numba/numba/issues/6518#issuecomment-736651396

@kozlov-alexey am hoping that #7441 will fix.


MongoDB Plugin Returns Array Data as Object Format Instead of Standard JSON Array

Hello,
I'm encountering an issue with the MongoDB plugin. When I run an aggregation query in Grafana, array data is returned in an object format rather than as a standard JSON array. Here’s an example:

Expected Output (e.g., from MongoDB in Robot 3T):

["a", "b"]

or
[1, 2, 3, 4]

Actual Output (from this plugin in Grafana):

{ "0": "a", "1": "b" }

or
{
    "0": { "$numberDouble": "1.0" },
    "1": { "$numberDouble": "2.0" },
    "2": { "$numberDouble": "3.0" },
    "3": { "$numberDouble": "4.0" }
}

Steps to Reproduce:

Insert sample data in MongoDB:

db.demoColl.insertMany([
    { "name": "series_1", "datas": ["a", "b"] },
    { "name": "series_2", "datas": [1, 2, 3, 4] }
])

Execute the following aggregation query in Grafana:

db.demoColl.aggregate([{ "$project": { "_id": 0 } }])

Environment:

Grafana version: v11.1.0
MongoDB plugin version: v0.1.1

The plugin currently returns array data in an object format, with each array element indexed as a key-value pair. Numeric values are wrapped in $numberDouble, which complicates direct usage within Grafana.
If the data source could return array data in a standard JSON array format, it could improve consistency and make data handling easier within Grafana.
Thanks for your work on this! Looking forward to your thoughts!

Good catch. I just realized a better way to unmarshall raw values
var a json.RawMessage
bson.UnmarshalValue(bson.TypeArray, rawValue.Value, &a)

or
var a json.RawMessage
bson.UnmarshalValue(bson.TypeEmbeddedDocument, rawValue.Value, &a)

I will soon fix it

@jeanfliu I have a PR that should address this issue.

@jeanfliu Reopen if issue still exists


ListingTable cannot handle partition evolution

Describe the bug
With CSV:
echo "a,b\n1,2" > data1.csv
mkdir a=2
echo "b\n3" > a=2/data2.csv
datafusion-cli
> SELECT * FROM '**/*.csv';
Arrow error: Csv error: incorrect number of fields for line 1, expected 2 got 1

With Parquet:
import os
import polars as pl

pl.DataFrame({'a': [1], 'b': [2]}).write_parquet('data1.parquet')
os.mkdir('a=2')
pl.DataFrame({'b': [3]}).write_parquet('a=2/data2.parquet')

datafusion-cli
> SELECT * FROM '**/*.parquet';
+---+---+
| b | a |
+---+---+
| 2 | 1 |
| 3 |   |
+---+---+
2 row(s) fetched.
Elapsed 0.055 seconds.

To Reproduce
No response
Expected behavior
Partition evolution is handled and both cases return
+---+---+
| b | a |
+---+---+
| 2 | 1 |
| 3 | 2 |
+---+---+

Additional context
Having played around quite a bit with ParquetExec and the SchemaAdapter machinery I think what should happen is:

Partition values are on a per-file basis, in particular on each PartitionedFile and not on the FileScanConfig
Partition values are passed into the SchemaAdapter machinery and for each file it decides if it needs to add a column generated from partition values or not

cc @alamb I had promised you this a long time ago but only got around to it now

Thanks @adriangb

take


community: Update UC toolkit documentation to use LangGraph APIs

Description: Update UC toolkit documentation to show an example of using recommended LangGraph agent APIs before the existing LangChain AgentExecutor example. Tested by manually running the updated example notebook
Dependencies: No new dependencies

Thanks all - reran and pushed updated notebook output with python 3.11 - looks like both the Vercel build and the official live docs have the same formatting weirdness for the final agent output, I figure that's ok:

Otherwise the notebook content seems to render fine :)


Fabric js UI component importing issues

I have tried to use office UI component by installing them on my angular project by following the link in github
https://github.com/ngOfficeUIFabric/ng-officeuifabric
I am stuck with these errors!!!

This doesn't appear to be linked to the Froala Editor, but to the way you're loading Fabric JS.

I have installed using this command
npm install ng-office-ui-fabric --save
I should have
1)/node_modules/office-ui-fabric/fabric.min.css
2) /node_modules/office-ui-fabric/fabric.components.css"
these files in my node_modules folder,but i am not having.


[CORE] Reducer protocol.

Reducer protocol - where all the acceleration happens.

UcxShuffleManager instantiates UcxShuffleReader that instantiates UcxShuffleClient and doing progress.
UcxShuffleClient implements fetchBlocks that's doing 2 consecutive ucp_get operations:
2.1. Fetch offset (OnOffsetsFetchCallback)
2.2. Fetch blocks (OnBlocksFetchCallback)

Spark-2.3 is unsupported. Nee first merge #12


[RFC] Board config: SPI & I2C config simplifications

Next round after #13871: SPI + I2C
Summary
Similar to the PWM config, we can configure SPI buses and devices:
constexpr px4_spi_bus_t px4_spi_buses[SPI_BUS_MAX_BUS_ITEMS] = {
	initSPIBus(1, {
		initSPIDevice(DRV_GYR_DEVTYPE_ICM20689, SPI::CS{GPIO::PortF, GPIO::Pin2}, SPI::DRDY{GPIO::PortB, GPIO::Pin4}),
		initSPIDevice(DRV_GYR_DEVTYPE_ICM20602, SPI::CS{GPIO::PortF, GPIO::Pin3}, SPI::DRDY{GPIO::PortC, GPIO::Pin5}),
		initSPIDevice(DRV_GYR_DEVTYPE_BMI055, SPI::CS{GPIO::PortF, GPIO::Pin4}, SPI::DRDY{GPIO::PortB, GPIO::Pin14}),
		initSPIDevice(DRV_ACC_DEVTYPE_BMI055, SPI::CS{GPIO::PortG, GPIO::Pin10}, SPI::DRDY{GPIO::PortB, GPIO::Pin15}),
	}, true),
	initSPIBus(2, {
		initSPIDevice(SPIDEV_FLASH(0), SPI::CS{GPIO::PortF, GPIO::Pin5})
	}),
	initSPIBus(4, {
		initSPIDevice(DRV_BARO_DEVTYPE_MS5611, SPI::CS{GPIO::PortF, GPIO::Pin10}),
	}),
	initSPIBusExternal(5, {
		SPI::CS{GPIO::PortI, GPIO::Pin4},
		SPI::CS{GPIO::PortI, GPIO::Pin10},
		SPI::CS{GPIO::PortI, GPIO::Pin11}
	}),
	initSPIBusExternal(6, {
		SPI::CS{GPIO::PortI, GPIO::Pin6},
		SPI::CS{GPIO::PortI, GPIO::Pin7},
		SPI::CS{GPIO::PortI, GPIO::Pin8}
	}),
};

This is then mapped to a constant data-structure that the drivers can query. Currently they use bus_options structs with lots of ifdefs. The PR replaces this by a generic bus iterator class, which can be used to generically select internal/external I2C/SPI buses. For example the 1. external SPI bus can be selected with:
SPIBusIterator bus_iterator(SPIBusIterator::FilterType::ExternalBus, 1, 1);
if (bus_iterator.next()) {
  g_dev = new PMW3901(bus_iterator.bus().bus, bus_iterator.devid(), (enum Rotation)0);
}

Details & Goals

The matching between driver and configured device is done via existing DRV_*_DEVTYPE_*. This has pro's and con's:

pro: It's using already existing ID's, and is simpler to add new ones.
con: there's typically different definitions for gyro and accel, and we need to select which one to use consistently (I used the gyro now).

It's a bit tricky for drivers supporting multiple devices, like ms5611 and ms5607 that do auto-detection. It works, but both sides (driver+board config) need to use the same IDs.
For external SPI, the chip-select signal can be selected via CLI
With the current implementation there can only be 1 SPI device of a certain type per bus. This can be extended, but I did not want to make things more complicated than required.
it works on Linux targets as well
while having to go through all drivers, I think it makes sense to also add a bit more structure to them. The 2. commit adds an I2C/SPI driver base class, with multi-instance support, reliable driver stopping, and fits into the existing driver structure. However it does not fit into the existing module base structure (or the refactored version) - but it is possible to add instances to the refactored linked list of modules, if we want to.
cleanup boot output to only show what devices were found
the configuration can be used to check at runtime if expected devices are running. It will need to be extended with board revision information though.

Steps to complete
Note: this is obviously far from complete. It runs on v3 and v5 with a small set of drivers.
I think we can go in steps with this:

extend boards with new config
update drivers
remove previous board config

@dagar @davids5

The matching between driver and configured device is done via existing DRV_*_DEVTYPE_*. This has pro's and con's:

pro: It's using already existing ID's, and is simpler to add new ones.
con: there's typically different definitions for gyro and accel, and we need to select which one to use consistently (I used the gyro now).

We only need one DEVTYPE per physical interface, and in fact handling both has made more of a mess. I think we could migrate them safely with a hook at parameter import time.

It's a bit tricky for drivers supporting multiple devices, like ms5611 and ms5607 that do auto-detection. It works, but both sides (driver+board config) need to use the same IDs.

I think we should consider explicitly not supporting these cases (at the cost of flash short term). In many cases it starts off simple enough, but gradually gets worse over time as we discover the devices aren't so similar after all. We saw this happen when the mpu9250 driver was extended to support the icm20689. Partial list of differences between Invensense drivers (https://github.com/PX4/Firmware/pull/14064).
Depending on how we want to handle old boards and variants we'll need a mechanism to handle alternate sensors on a chip select. The ms5607 could be started only if the ms5611 fails, or ideally with a different board manifest entirely.

@bkueng How does the code size look?

While the SPI/I2C configuration is being improved, I hope it can also be considered to solve 2 issues I encountered while working on code changes in pull request #14168 .
The 1st issue is that currently it's possible for one SPI device driver to execute code that should be only applied to I2C driver of the same device. Here is an example, mpu9250 is started in SPI mode with command like "mpu9250 -S start", and you would expect that the driver would only execute code for SPI driver, not code for I2C driver. However in USE_I2C macro is defined in mpu9250.h as the following:
#if defined(PX4_I2C_OBDEV_MPU9250) || defined(PX4_I2C_BUS_EXPANSION)
define USE_I2C
#endif
When PX4_I2C_BUS_EXPANSION is also defined in board_config.h for another sensor, then USE_I2C will be defined for mpu9250 and I2C portion of the driver code will be executed although you intend to use mpu9250 in SPI code.
The 2nd issue is that SPI driver and I2C driver of the same device cannot be used on the same board. PX4 supports redundant sensors, so this issue could arise. For example, when running out available SPI interfaces, one might want to hook up the sensor using I2C bus. Although usually a different sensor is used as backup sensor, one could choose to use the same sensor with I2C interface. Also there are reports that multiple IMU sensors arranged in different orientations could improve accuracy, so it might be possible to use multiple same IMU devices in the future.

When PX4_I2C_BUS_EXPANSION is also defined in board_config.h for another sensor, then USE_I2C will be defined for mpu9250 and I2C portion of the driver code will be executed although you intend to use mpu9250 in SPI code.

Yes, this definitely needs to be fixed. I'll look into it.

The 2nd issue is that SPI driver and I2C driver of the same device cannot be used on the same board.

This will be possible as well. Are you referring to the same issue as above (USE_I2C)?

Closing this, as it is brought in in separate parts.


resolved_ts: skip default cf cmd with invalid ts

Signed-off-by: 5kbpers<EMAIL_ADDRESS>
What problem does this PR solve?
Problem Summary:
For now resolved_ts worker can not work on rawkv mode.
What is changed and how it works?
A temporary solution that skips the default cf commands which do not has valid ts.
From @NingLin-P, in the next step, we could skip the default cf commands in apply worker if cdc doesn't need to observe the region. That will solve this problem fundamentally.
Check List
Tests

Unit test
Integration test

Release note

N/A

/lgtm

/merge


Adding Hans Gaebler to the Developers List

Just adding my info to the developers list! Thanks!

Thanks for doing this yourself!


DEA error

Hello,
For some reason, I am getting an error in the DEA steps and subsequent visualization. It says there is an error in the data frame, but I am not sure how to solve this error. Any help is appreciated. Thank you.
rule DEA:
input: output/analysis/genome/dea/countGroup/Treatment_1_gene_count.tsv, output/analysis/genome/dea/countGroup/Untreated_gene_count.tsv
output: output/analysis/genome/dea/countGroup/Untreated_gene_norm.tsv, output/analysis/genome/dea/countGroup/Treatment_1_gene_norm.tsv, output/analysis/genome/dea/DEA/dea_Untreated_Treatment_1.tsv, output/analysis/genome/dea/DEA/deg_Untreated_Treatment_1.tsv
jobid: 1
Loading required package: limma
Loading required package: S4Vectors
Loading required package: stats4
Loading required package: BiocGenerics
Loading required package: parallel
Attaching package: ‘BiocGenerics’
The following objects are masked from ‘package:parallel’:
clusterApply, clusterApplyLB, clusterCall, clusterEvalQ,
clusterExport, clusterMap, parApply, parCapply, parLapply,
parLapplyLB, parRapply, parSapply, parSapplyLB

The following object is masked from ‘package:limma’:
plotMA

The following objects are masked from ‘package:stats’:
IQR, mad, sd, var, xtabs

The following objects are masked from ‘package:base’:
anyDuplicated, append, as.data.frame, basename, cbind, colMeans,
colnames, colSums, dirname, do.call, duplicated, eval, evalq,
Filter, Find, get, grep, grepl, intersect, is.unsorted, lapply,
lengths, Map, mapply, match, mget, order, paste, pmax, [pmax.int](http://pmax.int/),
pmin, [pmin.int](http://pmin.int/), Position, rank, rbind, Reduce, rowMeans, rownames,
rowSums, sapply, setdiff, sort, table, tapply, union, unique,
unsplit, which, which.max, which.min

Attaching package: ‘S4Vectors’
The following object is masked from ‘package:base’:
expand.grid

Loading required package: IRanges
Loading required package: GenomicRanges
Loading required package: GenomeInfoDb
Loading required package: SummarizedExperiment
Loading required package: Biobase
Welcome to Bioconductor
Vignettes contain introductory material; view with
'browseVignettes()'. To cite Bioconductor, see
'citation("Biobase")', and for packages 'citation("pkgname")'.

Loading required package: DelayedArray
Loading required package: matrixStats
Attaching package: ‘matrixStats’
The following objects are masked from ‘package:Biobase’:
anyMissing, rowMedians

Loading required package: BiocParallel
Attaching package: ‘DelayedArray’
The following objects are masked from ‘package:matrixStats’:
colMaxs, colMins, colRanges, rowMaxs, rowMins, rowRanges

The following objects are masked from ‘package:base’:
aperm, apply

Error in $<-.data.frame(*tmp*, "subject", value = integer(0)) :
replacement has 0 rows, data has 4
Calls: DEA -> $<- -> $<-.data.frame
Execution halted
[Sun Jul  3 02:54:56 2022]
Error in rule DEA:
jobid: 1
output: output/analysis/genome/dea/countGroup/Untreated_gene_norm.tsv, output/analysis/genome/dea/countGroup/Treatment_1_gene_norm.tsv, output/analysis/genome/dea/DEA/dea_Untreated_Treatment_1.tsv, output/analysis/genome/dea/DEA/deg_Untreated_Treatment_1.tsv
RuleException:
CalledProcessError in line 38 of /root/RASflow/workflow/dea_genome.rules:
Command ' set -euo pipefail;  Rscript scripts/dea_genome.R ' returned non-zero exit status 1.
File "/root/RASflow/workflow/dea_genome.rules", line 38, in __rule_DEA
File "/root/miniconda3/envs/rasflow/lib/python3.6/concurrent/futures/thread.py", line 56, in run
Removing output files of failed job DEA since they might be corrupted:
output/analysis/genome/dea/countGroup/Untreated_gene_norm.tsv, output/analysis/genome/dea/countGroup/Treatment_1_gene_norm.tsv
Shutting down, this might take some time.
Exiting because a job execution failed. Look above for error message
Complete log: /root/RASflow/.snakemake/log/2022-07-03T025450.360606.snakemake.log
DEA is done!
Start visualization of DEA results!
Building DAG of jobs...
Using shell: /usr/bin/bash
Provided cores: 1
Rules claiming more threads will be scaled down.
Job counts:
count   jobs
1       end
1       plot
2
[Sun Jul  3 02:54:56 2022]
rule plot:
input: output/analysis/genome/dea/countGroup, output/analysis/genome/dea/DEA
output: output/analysis/genome/dea/visualization/volcano_plot_Untreated_Treatment_1.pdf, output/analysis/genome/dea/visualization/heatmap_Untreated_Treatment_1.pdf
jobid: 1
Loading required package: plotscale
hash-3.0.1 provided by Decision Patterns
Loading required package: GenomicFeatures
Loading required package: BiocGenerics
Loading required package: parallel
Attaching package: ‘BiocGenerics’
The following objects are masked from ‘package:parallel’:
clusterApply, clusterApplyLB, clusterCall, clusterEvalQ,
clusterExport, clusterMap, parApply, parCapply, parLapply,
parLapplyLB, parRapply, parSapply, parSapplyLB

The following objects are masked from ‘package:stats’:
IQR, mad, sd, var, xtabs

The following objects are masked from ‘package:base’:
anyDuplicated, append, as.data.frame, basename, cbind, colMeans,
colnames, colSums, dirname, do.call, duplicated, eval, evalq,
Filter, Find, get, grep, grepl, intersect, is.unsorted, lapply,
lengths, Map, mapply, match, mget, order, paste, pmax, [pmax.int](http://pmax.int/),
pmin, [pmin.int](http://pmin.int/), Position, rank, rbind, Reduce, rowMeans, rownames,
rowSums, sapply, setdiff, sort, table, tapply, union, unique,
unsplit, which, which.max, which.min

Loading required package: S4Vectors
Loading required package: stats4
Attaching package: ‘S4Vectors’
The following objects are masked from ‘package:hash’:
values, values<-

The following object is masked from ‘package:base’:
expand.grid

Loading required package: IRanges
Loading required package: GenomeInfoDb
Loading required package: GenomicRanges
Loading required package: AnnotationDbi
Loading required package: Biobase
Welcome to Bioconductor
Vignettes contain introductory material; view with
'browseVignettes()'. To cite Bioconductor, see
'citation("Biobase")', and for packages 'citation("pkgname")'.

Attaching package: ‘AnnotationDbi’
The following objects are masked from ‘package:hash’:
keys, keys<-

Loading required package: ggplot2
Loading required package: ggrepel
Error in file(file, "rt") : cannot open the connection
Calls: plot.volcano.heatmap -> read.table -> file
In addition: Warning message:
In file(file, "rt") :
cannot open file 'output/analysis/genome/dea/DEA/dea_Untreated_Treatment_1.tsv': No such file or directory
Execution halted
[Sun Jul  3 02:55:01 2022]
Error in rule plot:
jobid: 1
output: output/analysis/genome/dea/visualization/volcano_plot_Untreated_Treatment_1.pdf, output/analysis/genome/dea/visualization/heatmap_Untreated_Treatment_1.pdf
RuleException:
CalledProcessError in line 53 of /root/RASflow/workflow/visualize.rules:
Command ' set -euo pipefail;  Rscript scripts/visualize.R output/analysis/genome/dea/countGroup output/analysis/genome/dea/DEA output/analysis/genome/dea/visualization ' returned non-zero exit status 1.
File "/root/RASflow/workflow/visualize.rules", line 53, in __rule_plot
File "/root/miniconda3/envs/rasflow/lib/python3.6/concurrent/futures/thread.py", line 56, in run
Shutting down, this might take some time.
Exiting because a job execution failed. Look above for error message
Complete log: /root/RASflow/.snakemake/log/2022-07-03T025456.705370.snakemake.log
Visualization is done!
RASflow is done!

Hello,
I think I solved the issue. The metadata file did not have the subject column which I re-added and it produced the pdf plots now.


Polygons on swisstopo cadastrel web map

Hello there
First of all, awesome repository, thank you very much! I'm quite new to the topic and your repository offers a great starting point. Probably you could give me a hint: I'm trying to reproduce the feature here https://map.geo.admin.ch/index.html?zoom=10&lang=de&topic=swisstopo&bgLayer=ch.swisstopo.pixelkarte-farbe&layers=ch.kantone.cadastralwebmap-farbe&catalogNodes=1396,1482,1397,1411,1424,1427&E=2693182.85&N=1139348.58. Clicking on a entry highlights it and shows some information.
This has to be a feature of the respective map (cadastrel), as only it knows the exact data. But I guess I would need to access the data and draw myself. Could you give me a hint, if I'm on the right path here?
Again, thank you very much.
With best regards
Matthias

I looked into the map.geo.amin.ch implementation. They built that custom by calling the following API on map click.
https://api3.geo.admin.ch/rest/services/all/MapServer/identify?geometry=2693487,1138873.5&geometryFormat=geojson&geometryType=esriGeometryPoint&imageDisplay=1408,1145,96&lang=de&layers=all:ch.kantone.cadastralwebmap-farbe&limit=10&mapExtent=2692836.9170767884,1138278.2143541584,2694244.9170767884,1139423.2143541584&returnGeometry=true&sr=2056&tolerance=10
I have added recently capabilities to draw custom shapes into OpenLayers.Blazor. You could implement the same behavior with this.
And by the way, geo.admin.ch does not return the layer with by calling WMS capabilities endpoint. The WMTS endpoint does return it. But finally, geo.admin.ch , use the WMS endpoint to get the tiles. So just add a WMS tile with the layer id ch.kantone.cadastralwebmap-farbe.

I looked into the map.geo.amin.ch implementation. They built that custom by calling the following API on map click.
https://api3.geo.admin.ch/rest/services/all/MapServer/identify?geometry=2693487,1138873.5&geometryFormat=geojson&geometryType=esriGeometryPoint&imageDisplay=1408,1145,96&lang=de&layers=all:ch.kantone.cadastralwebmap-farbe&limit=10&mapExtent=2692836.9170767884,1138278.2143541584,2694244.9170767884,1139423.2143541584&returnGeometry=true&sr=2056&tolerance=10
I have added recently capabilities to draw custom shapes into OpenLayers.Blazor. You could implement the same behavior with this. And by the way, geo.admin.ch does not return the layer with by calling WMS capabilities endpoint. The WMTS endpoint does return it. But finally, geo.admin.ch , use the WMS endpoint to get the tiles. So just add a WMS tile with the layer id ch.kantone.cadastralwebmap-farbe.

Hi Laurent, thank you very much. Looking into your changes.

Hi Dr. fyi, I have extended the layer demo page with call to API's geo.admin.ch to get a geojson with area and load it. Also when clicking on it it shows its properties. Actually the same that swisstop does. To make that work, I had to fix some coordinates handling and needed to pass the projection to the geojson loading method.

@lolochristen Sorry for the late response, thank you very much. Looking into it, this will help us a great deal.


Trees and bushes won't transfer from editor to game

I used New Leaf Save Editor back in 2017 and didn't experience any issues regarding editing my town and getting all of the results I wanted into my save file. It's been roughly two years since then and I decided to re-install Homebrew after 3DS version updates and have gotten into the New Leaf Save Editor once more. After a while of using the editor I noticed something that didn't occur back in 2017 and that seemingly nobody is complaining about at the moment, and that is the fact that trees, bushes and bamboos don't grow in all of the spots in the game where I had placed them in the editor. After testing this problem out by re-adding them over and over again in the editor and trying to get them load in my game, I realised that they don't grow in places where the game doesn't want you to have them growing, for example on the beach, in the river, too close to houses or pwps or on grey ground areas around the train station, the recycle shop, the town hall and the town tree. I have no problem placing trees in these areas on the editor, but once I import the save file back into my 3DS and load the game, the trees and bushes will be completely gone as if I never placed them there to begin with.
Is this some odd bug that I'm experiencing or has the new leaf game/the editor been updated so that you can no longer place trees and bushes wherever you want? I'm not experiencing any other issues, but it certainly is disheartening that I can no longer go wild with adding whatever I want wherever I want in my town with the help of the editor. This is only an issue that persists with trees, bushes and bamboos; I can still move houses, pwps and other things around the map and put them into places where the game doesn't normally allow them.
If there is anything I can do, I'd gladly hear from you regarding this issue, and thank you in advance.
If I recall correctly, that was a fix on WA.

Yes I am, and that would make sense. I did play Welcome Amiibo back in 2017 too, but it could have been patched later on. Thank you for your help though!


Issue list is now blank

/api/v1/issues/ is returning some issues with no screenshots, I wonder why is that happening because one cannot make a POST request to /api/v1/issues/ without sending at least one screenshot.

I think it can be closed now?


reporting buffer overflow bugs

Hi, we've been testing openssl APIs and spotted several and similar issues to be fixed,
similar to this part: https://github.com/openssl/openssl/blob/504ff2a4ef5f26990a48ca3d664ac1e5d9cb20b9/crypto/idea/i_ofb64.c#L38-L41
there should be negative value and large value checks in these places too, otherwise it'll overflow the stack:
https://github.com/openssl/openssl/blob/504ff2a4ef5f26990a48ca3d664ac1e5d9cb20b9/crypto/des/ofb64ede.c
https://github.com/openssl/openssl/blob/504ff2a4ef5f26990a48ca3d664ac1e5d9cb20b9/crypto/rc5/rc5ofb64.c
https://github.com/openssl/openssl/blob/504ff2a4ef5f26990a48ca3d664ac1e5d9cb20b9/crypto/rc2/rc2ofb64.c
https://github.com/openssl/openssl/blob/504ff2a4ef5f26990a48ca3d664ac1e5d9cb20b9/crypto/cast/c_ofb64.c
https://github.com/openssl/openssl/blob/504ff2a4ef5f26990a48ca3d664ac1e5d9cb20b9/crypto/des/ofb64enc.c
https://github.com/openssl/openssl/blob/504ff2a4ef5f26990a48ca3d664ac1e5d9cb20b9/crypto/bf/bf_ofb64.c
https://github.com/openssl/openssl/blob/504ff2a4ef5f26990a48ca3d664ac1e5d9cb20b9/crypto/des/cfb64enc.c
https://github.com/openssl/openssl/blob/504ff2a4ef5f26990a48ca3d664ac1e5d9cb20b9/crypto/des/cfb64ede.c
And also some other places, these checks miss negative values too:
https://github.com/openssl/openssl/blob/master/crypto/rsa/rsa_pk1.c#L38
https://github.com/openssl/openssl/blob/master/crypto/txt_db/txt_db.c#L133
https://github.com/openssl/openssl/blob/master/crypto/txt_db/txt_db.c#L154
https://github.com/openssl/openssl/blob/504ff2a4ef5f26990a48ca3d664ac1e5d9cb20b9/crypto/rc2/rc2_skey.c#L65
the followings might be also considered, they'll overflow the user provided buffer:
https://github.com/openssl/openssl/blob/504ff2a4ef5f26990a48ca3d664ac1e5d9cb20b9/crypto/cast/c_cfb64.c
https://github.com/openssl/openssl/blob/504ff2a4ef5f26990a48ca3d664ac1e5d9cb20b9/crypto/rc2/rc2cfb64.c
https://github.com/openssl/openssl/blob/504ff2a4ef5f26990a48ca3d664ac1e5d9cb20b9/crypto/rc5/rc5cfb64.c

Thanks for your report.
Most of the functions you have listed are no longer part of OpenSSL's public API.
c.f. https://www.openssl.org/docs/manmaster/man3/DES_ede3_ofb64_encrypt.html :

The following functions have been deprecated since OpenSSL 3.0, and can be hidden entirely by defining OPENSSL_API_COMPAT with a suitable version value...

However, the second batch you listed are more interesting:
https://github.com/openssl/openssl/blob/master/crypto/rsa/rsa_pk1.c#L38
https://github.com/openssl/openssl/blob/master/crypto/txt_db/txt_db.c#L133
https://github.com/openssl/openssl/blob/master/crypto/txt_db/txt_db.c#L154
Improved bounds checks would be welcome there.
Would you consider opening a PR?

Can someone help on this? Sorry that I'm stuck at the CLA procedure.


Langchain js - can't get token usage

System Info
Langchain JS 0.0.172 on DEbian Linux 12 using Nodejs 18.15.0
With the following code I don't get the tokenUsage back from the chain/ call. I checked the docs, asked the AI and did a google search. Am I doing anything wrong?
const chain = RetrievalQAChain.fromLLM(   model,   vectorStore.asRetriever(),   {     verbose: true,   } ) const res = await chain.call({   context: "You assist a Systems Engineer. Be as concise as possible.",   query: prompt, }) console.log('usage:', res.tokenUsage);
Who can help?
@hwchase17
Information

[X] The official example notebooks/scripts
[X] My own modified scripts

Related Components

[X] LLMs/Chat Models
[ ] Embedding Models
[ ] Prompts / Prompt Templates / Prompt Selectors
[ ] Output Parsers
[ ] Document Loaders
[ ] Vector Stores / Retrievers
[ ] Memory
[ ] Agents / Agent Executors
[ ] Tools / Toolkits
[ ] Chains
[ ] Callbacks/Tracing
[ ] Async

Reproduction
import { DirectoryLoader } from "langchain/document_loaders/fs/directory";
import { TextLoader } from "langchain/document_loaders/fs/text";
import { JSONLoader } from "langchain/document_loaders/fs/json";
import { CSVLoader } from "langchain/document_loaders/fs/csv";
import { OpenAI } from "langchain/llms/openai";
import { RetrievalQAChain, loadQARefineChain } from "langchain/chains";
import { HNSWLib } from "langchain/vectorstores/hnswlib";
import { OpenAIEmbeddings } from "langchain/embeddings/openai";
import { RecursiveCharacterTextSplitter } from "langchain/text_splitter";
import * as dotenv from 'dotenv';
dotenv.config()
const loader = new DirectoryLoader("./docs", {
".json": (path) => new JSONLoader(path),
".csv": (path) => new CSVLoader(path),
".txt": (path) => new TextLoader(path)
})
const normalizeDocuments = (docs) => {
return docs.map((doc) => {
if (typeof doc.pageContent === "string") {
return doc.pageContent;
} else if (Array.isArray(doc.pageContent)) {
return doc.pageContent.join("\n");
}
});
}
const VECTOR_STORE_PATH = "Documents.index";
export const run = async (params) => {
console.log("Loading docs...")
const docs = await loader.load();
console.log('Processing...')
const model = new OpenAI({
openAIApiKey: process.env.OPENAI_API_KEY,
modelName: "gpt-3.5-turbo",
maxTokens: 2500,
temperature: 0.1,
maxConcurrency: 1,
});
let vectorStore;
try{
console.log("Checking for local vectorStore");
const embeddings = new OpenAIEmbeddings(process.env.OPENAI_API_KEY);
vectorStore = await HNSWLib.load(VECTOR_STORE_PATH, embeddings);
}catch(e){
console.log('Creating new vector store...')
const textSplitter = new RecursiveCharacterTextSplitter({
chunkSize: 3000,
});
const normalizedDocs = normalizeDocuments(docs);
const splitDocs = await textSplitter.createDocuments(normalizedDocs);
console.log('splitdos : ',splitDocs)
// 8. Generate the vector store from the documents
vectorStore = await HNSWLib.fromDocuments(
  splitDocs,
  new OpenAIEmbeddings()
);

await vectorStore.save(VECTOR_STORE_PATH);
console.log("Vector store created.")

}
console.log("Creating retrieval chain...")
const chain = RetrievalQAChain.fromLLM(
model,
vectorStore.asRetriever(),
{
verbose: true,
}
)
//const promptStart = rewrite all requirements using the following guidelines:;
const promptStart = ;   //const reWrite = `Rewrite the Requirement Text so that is does comply.`;   const reWrite = ;
const prompts = [
R3: Requirements should be consistent with each other and should not contain contradictions or conflicts. Report the Id\'s for those that do not comply. Rewrite the Requirement Text so that is does comply with R3 ,
];
prompts.forEach(async prompt => {
//console.log("Querying chain...")
const res = await chain.call({
context: "You assist a Systems Engineer. Be as concise as possible.",
query: prompt,
})
console.log('usage:', res.tokenUsage);
console.log(prompt);
console.log('Reply: ',res, '\n')

});
}
run(process.argv.slice(2))
Expected behavior
I expect the res object to return the token usage

Interesting. I'm getting the following error while trying to implement the solution:
import { ConversationSummaryBufferMemory } from "langchain/memory";          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ SyntaxError: The requested module 'langchain/memory' does not provide an export named 'ConversationSummaryBufferMemory'

@dosu-bot
Interesting. I'm getting the following error while trying to implement the solution:
import { ConversationSummaryBufferMemory } from "langchain/memory"; ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ SyntaxError: The requested module 'langchain/memory' does not provide an export named 'ConversationSummaryBufferMemory'


Formtastic form throws “undefined method” error with has_one association

I am using Rails 3.1 and Ruby 1.9.2 along with Active Admin to build a CMS. Here are my Place and Image models:
class Place < ActiveRecord::Base
  has_one :image
  accepts_nested_attributes_for :image
end

class Image < ActiveRecord::Base
  belongs_to :place
end

Here is the Formtastic form I am rendering in my "new" action in my Places controller:
<%= semantic_form_for [:admin, @place] do |p| %>
  <%= p.inputs "Details" do %>
    <%= p.input :name %>
    <%= p.input :description %>
    <%= p.input :phone %>
    <%= p.input :address %>
    <%= p.input :image %>
  <% end %>
  <%= p.buttons %>
<% end %>

When I load the form in my browser, I see the following error:
undefined method `place_id' for #<Place:0xb801744>

Here's the kicker: in my Place model, if I change has_one :image to has_many :images and accepts_nested_attributes_for :image to accepts_nested_attributes_for :images, and in my form I change p.input :image to p.input :images, then the error disappears and Formtastic properly renders a multiselect input element containing all available image objects. So why am I getting this error when I use a has_one association rather than seeing a select input element?

use has_one :image, inverse_of: :place
worked for me :)

It's 2017 and I stumbled on the same issue again. Seems to be caused by justinfrench/formtastic#824


[DROOLS-983] fix events expiration when using inheritance

ok to test

Mario, please consider my comments before merging.

@mariofusco +1 to merge

Merged by


Fix minor problems in document

About This Pull Request
This PR is the minor change of the document input_variables.md which contains:

Fix the option of propagater
Fix the description of file_kw

Since this PR doesn't modify any source file, it is not necessary to varify the build and calculation.

For num_kgrid, the description is consistent with the code. And for propagator, modified description is correct. So I think this PR is able to merge. But I think it's better to use small capital at the first letter of "Enforced time-reversal symmetry propagator". @uemoto1 san, will you modify or not? I'll merge whichever you choose.

Thank you @mn2007 san, I fixed the issued point as above.

Thank you for your reply and the modification, @uemoto1 san. I merge this PR.


[224] Update D def to simplify calc.

Fix #224

Thanks @jovfer, considering that this is a breaking change to the proof representation and it only provides a minor improvement around proof computation we will keep this PR open until we consider other breaking changes that it could be adopted with.

Discussed on the 12th of December WG call, given the improvement in readability for this proposal is minor and the performance difference is likely to be negligible we are not going to proceed with this change for now, closing the PR on that basis, please re-open if you have any further questions.


sample from truncated normal without rejection sampling

I couldn't find a TruncatedNormal in torch.distribitutions.  There was a truncnormal in torch.nn.init, but it couldn't handle multiple means and stdevs.  So I just sampled with inverse cdf.

It would be nice to somehow test that it is sampling from the right distribution :).  Currently have no tests for that.  I tested myself and looked at histograms and they looked right, but I wonder if there's another way.


Not working with Python 3 !!

Doesn't work with python 3.

Can you share more details.

This is the error message, I think it is because the unitConverter is deprecated and now called unit-converter.
ModuleNotFoundError                       Traceback (most recent call last)
Cell In[6], line 7
5 import pytesseract
6 from PIL import Image
----> 7 from pyScanLib import DeviceManager, Scanner
8 import fitz
10 # Set OCR language
File /Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/pyScanLib/init.py:2
1 import sys
----> 2 import unitConverter  # Contains unit converting functions ie millimeter to inch (mmToInch) etc
4 """
5  * pyScanLib
6  * Location: https://github.com/soachishti/pyScanLib
(...)
29  * SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
30 """
33 platform = sys.platform
ModuleNotFoundError: No module named 'unitConverter'


Triangulation with simple time differences

DiffPoint takes simple time differences and points to the supernova. astropy is used to convert the Earth locations of the detectors into geocentric celestial coordinates at the time of observation.
This PR also includes DetectorDB, which reads the detector database file (such as detector_locations.csv) once. Modules can instantiate DetectorDB and then get Detector objects out of it.
GenPoint was a simple SN direction-to-times generator for testing.  Its logic should migrate into the other generator code.
You can test GenPoint/DiffPoint with "make diffpoint".  It will create a mollview skymap.

Thanks @mcolomermolla for the comments.  One question:  didn't Dan's plot have ra=0 at the left edge? Mine has ra=0 at the center (though I have to check that sign again)

If the truth position I run is ra=-60 and dec=-30, and in your plot ra=0 is in the center, the negative RA should be on the left of the plot, not on the right as in the output obtained. The one by Dan is in the right place for ra<0 left ra=0 center and ra>0 right. attached the one I obtained, for the truth position mentioned (close to Galactic Center), it should be "mirrowed":

When I look at Dan (et al)'s plot, I see RA of 0h on the left edge, then reading from left to right, 21h, 18h, 15h, all the way back to 0h on the right edge.  I was under the impression that 0h would correspond to 0 degrees, but that might be too naive.
In any case, maybe we should move to use the ligo.skymap package.
As for the left/right order, this is controllable via the "flip" flag in healpy.mollview.  According to the healpy.mollview docs, the default is the "astro" view, which has "east toward left, west toward right".  The "geo" view is the other way around.  I think this is because the skymap has south pole at the bottom, and the north pole at the top; if you look at the night sky with this orientation, then east is toward the left.  (This seems to be what I get from stellarium, too.)

Added a csv file for configuration.
It turns out healpy.mollview doesn't offer axis labels; it's apparently a long-standing issue. Probably another reason to move towards ligo.skymap.
Database lookup of sigma and bias removed from DiffPointing. @mcolomermolla is right - it should come from the experiments, or is something we need to calculate ourselves upstream.

Changed Mollview option to declare "celestial" coordinate system; I had mistakenly put in a "G", which indicates a galactic, sun-centric coordinate system. Doesn't really change the output, because all the calculations were done in the celestial coordinate system.

As found by @dallaval5u in #61, we were using data['gen'] in two different ways. I've moved truth information to data['truth']

I think that's it. This should go in because there's more to come in another PR!


Make LNbits wait wait for LND

At the moment LNbits systemd service just waits for bitcoind and then starts ...
https://github.com/raspiblitz/raspiblitz/blob/b419daf43a22c98879ffc2a7f2937bde14b81e0f/home.admin/config.scripts/bonus.lnbits.sh#L809-L810
But when the funding source is LND it should wait forLND etc. This can be done like here:
https://github.com/raspiblitz/raspiblitz/blob/b419daf43a22c98879ffc2a7f2937bde14b81e0f/home.admin/config.scripts/bonus.lnbits.sh#L709-L710
Based on funding source when LND is script is used with on parameter it should also dynamically set the Wants= and After= part to the service of that funding source.

Also we need adapt the AuthMethod in the future ... but later ... see: #4597


Rename NATI to match its definition

The structure with the NATI tag is defined as

Version
Name
Definition

5.0, 5.3
Nationality
The national heritage of an individual.

5.4, 5.5, 5.5.1
National or Tribal Origin
The person's division of national origin or other folk, house, kindred, lineage, or tribal interest. Examples: Irish, Swede, Egyptian Coptic, Sioux Dakota Rosebud, Apache Chiricawa, Navajo Bitter Water, Eastern Cherokee Taliwa Wolf, and so forth.

7.0
Nationality
An individual’s national heritage or origin, or other folk, house, kindred, lineage, or tribal interest.

Despite the broadening definition over time, the name for the tag was reverted to 5.3's name with 7.0. I propose changing that name (only, not the definition) to one that batter captures the meaning of the structure, such as

people, in the older sense when "people" and "persons" were distinct concepts
heritage, though the English of this is much too broad, including things like inheritances
birth affiliation, except you can be inducted into a tribe or naturalized into a nationality
affiliation, except many affiliations (like clubs and political parties) are not encoded with NATI now
national affiliation / tribal identity, chosen so its initials spell the tag NATI

None of these names strike me as ideal. Any preferences or suggestions?

I like to think of these as Cultural Heritages, Cultural Affiliations, Cultural Identity.
The examples are Cultural in nature, and looks to how an individual Identifies themself, which may not be in total just using national boarders or even nations that exist on the map today.  A person could be German and not live in Germany, or Roman and live in what became England.

@tychonievich Can this issue be closed or was there anything else not covered by PR #319?

Closing per discussion March 7, 2024


Fix incorrect path computation of typesMap.json

This method does not yield a valid path in Linux.
/foo/bar/tsserver.js/../typesMap.json works in Windows but fails the stat in Linux.

@minestarks you can merge when you'd like


Audio player preview doesn't auto play other/certain filetypes

Describe the bug
The audio player when previewing seems to only auto play .vsnd_c files and not filetypes such as .wav despite having the functionality to play those files. The player will open but there doesn't seem to be an attempt to play the file.
Expected behavior
The "Auto play sounds" feature of the quick file preview should play files that the audio player has functionality for.
Relevant log output or exception with stacktrace
No response
What game does this happen in
Counter-Strike 2 Workshop / Half-Life Alyx Workshop / Half-Life 2 / (anything with raw audio files)
Reference file in a Source 2 game
content/csgo_addons/addon_template/sounds/vent_01.wav
content/hlvr_addons/audio_new_sound/sounds/snd_sound_add_demo_01.wav
hl2/hl2_sound_misc_dir.vpk
What version of VRF are you using?
10.2.3894 951ba65f71abef8f99583235f77b7b69047daac2 on Microsoft Windows NT 10.0.19045.0 (GPU: NVIDIA GeForce RTX 4070 SUPER/PCIe/SSE2, Driver: 4.6.0 NVIDIA 565.90)

Also a feature request would be if a button could be added to settings/player ui to auto play any file opened rather than just when previewed


Unable to set topology from dna.json file

I've noticed that you override some values in your dna.json file but not all. Topology is one value you can only override in /etc/opscode/chef-server.rb.
root@chef-server-tmp-1445653166:~# cat /opt/opscode/embedded/cookbooks/dna.json
{
    "run_list": [
        "recipe[private-chef::default]"
    ],
    "private_chef": {
        "enabled-plugins": [
            "chef-ha-opennebula"
        ],
        "topology": "ha",
        "servers": {
            "chef-server-tmp-1445653166": {
                "ipaddress": "<IP_ADDRESS>",
                "role": "backend",
                "bootstrap": false
            }
        }
    }
}

When triggering reconfigure to following attributes are different, and the topology ends up being 'standalone'.
node['private_chef']['topology'] ... results in 'ha'
PrivateChef["topology"] ... results in 'standalone'

@poliva83 Thanks for taking the time to file this issue, apologies no one has responded to it since you filed it.
I think that we want to keep dna.json as an internal implementation detail and not make guarantees about what is or is not settable from that file.  Do you have a particular need to set the topology in that file verse the main chef-server.rb configuration file?

@stevendanna This issue is over year old now. I was experimenting using /opt/opscode/embedded/cookbooks/dna.json instead maintaining /etc/opscode/chef-server.rb.  The json format seemed a little easier to maintain in my cookbook's template then the ruby file. Anyway I since abandoned that idea so feel free to close this.

@poliva83 Thanks for following up.  Again, apologies this wasn't addressed at the time.


Bookstore: add reducers and actions

Write book's actions and reducer (in /src/redux/books/books.js) using the ducks pattern.

Define action types for adding and removing a book.
Set the initial state to be an empty array of books.
Export Action Creators for the actions.
Write a reducer and export it as default.

Define state changes for the actions created.
In case of unknown action - return the current state.

Write categories actions and reducer (in /src/redux/categories/categories.js) using the ducks pattern.

Define an action type for displaying "under construction" message.
Set the initial state to be an empty array of categories.
Export Action Creators for the actions.
Write a reducer and export it as default.

For the check status action return a string "Under construction".
In case of unknown action - return the current state.

Configure the Redux Store (/src/redux/configureStore.js):

Importing the necessary methods from Redux.
Combine both reducers into a root reducer by using combineReducers function.
Use the root reducer to create the store.

Thanks for the review @GzimAsani !


feat: replace template provider with templatefile function

what

remove template provider

why

m1 Mac compatibility

Duplicate
#91
I didn't realize until after pushing 🤦

/test all

@kevcube Thanks for the contribution! Released as https://github.com/cloudposse/terraform-aws-ec2-bastion-server/releases/tag/0.30.0


Email content setting

Docs for: https://github.com/mattermost/platform/pull/7122

Thanks updated


[docs] What is the default part size ?

The part size indicated in the docs is 128MiB.
https://docs.min.io/docs/golang-client-api-reference.html#PutObject
But the code indicates 16MiB.
https://github.com/minio/minio-go/blob/master/api-put-object.go#L217-L228
What is the default part size of the chunk ?

It seams that the real minPartSize is 16MiB.
https://github.com/minio/minio-go/blob/e517704ccb36b0a3b48907977c29506327d80f12/constants.go#L26-L28
Also, if I upload a file bigger that 1000x the minPartSize (16GiB), I get this error:
Part number must be an integer between 1 and 1000, inclusive

But the docs states that

For objects that are greater than 128MiB in size, PutObject seamlessly uploads the object as parts of 128MiB or more depending on the actual file size.

I do not understand the part or more depending on the actual file size.
Does this mean that the PartSize is automatically computed based on the size of the file ? Or should I specify a value manually via the opts ?


Determining request or response source in the FlooNoC

In FlooNoC, since all incoming non-atomic request going to slave and response coming back from the slave, have identical AXI4 ID, while atomic requests have unique IDs, then if we want to know at the slave side the source of the request, or if we want to know at the master side the source of the response, how can it be done? Should I use the src_id in the header of the request/response. For example, for AW signal, will I have to use the floo_req_in.narrow_aw.hdr.src_id to check where this request is coming from?

Yes you are correct, this can only identified on the NoC side with the src_id field. AXI requests coming from the NoC don't contain any information anymore about the actual source, since the ID is remapped at entry as you mentioned.


Revisit vela update command

Currently, the vela update command is used to fetch remote definitions from APIServer and sink to local the Capability struct. This requires the user to manually refresh to deploy apps or fetch definitions every time (in this case there is no need to local files too).
The biggest problem is that refreshing manually is not a good user experience. This issue is to rethink about the problem and discuss better solutions.

Yes， I have discussed with @resouer , we should find some mechanism to run this command automatically background.

we could use vela workloads and vela traits to do this refresh work.

fixed by #505


Rollup commands

We need commands for creating and deploying rollups

Closing this. The rollup command features will be moved to a new module in raas.


App Engine server type allows to connect even when logged out

Steps to reproduce:

Set up the App Engine Flexible Environment server.
Logout from GCP using the account icon on the toolbar.
Click "Connect" button on the App Engine Flexible Environment Server and see it succeed.

The same can be said about the disconnect button.
The problem is that after this "successful connection", the deployment action fails with:

Failed to deploy 'userSpecifiedSource': You must be logged in to deploy."

I don't think this "connect" feature does anything useful for us outside the context of a deploy. Deploying implicitly does a connect and requires the user to be logged in. Maybe we move the Auth checking to the connect callback instead?

I thought we had disabled 'connect'.
On Tue, Mar 29, 2016 at 12:24 PM, Etan Shaul<EMAIL_ADDRESS>wrote:

I don't think this "connect" feature does anything useful for us outside
the context of a deploy. Deploying implicitly does a connect and requires
the user to be logged in. Maybe we move the Auth checking to the connect
callback instead?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/gcloud-intellij/issues/552#issuecomment-202984228

I think we should just duplicate the authentication checking in connect().
Alternatively, just hide the connect and disconnect buttons if IntelliJ allows that.

Where do you see connection buttons?  I'm not seeing any.
On Tue, Mar 29, 2016 at 1:18 PM, meltsufin<EMAIL_ADDRESS>wrote:

I think we should just duplicate the authentication checking in connect().
Alternatively, just hide the connect and disconnect buttons if IntelliJ
allows that.
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/gcloud-intellij/issues/552#issuecomment-203007953

The play button here:

+1 for hiding the button if possible. Its a function that doesn't really map onto our desired workflow.

Yeah. 'Connect' as a concept seems pretty meaningless for most usecases I
can think of.
On Tue, Mar 29, 2016 at 1:37 PM, Etan Shaul<EMAIL_ADDRESS>wrote:

+1 for hiding the button if possible. Its a function that doesn't really
map onto our desired workflow.
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/gcloud-intellij/issues/552#issuecomment-203018661

I can see how 'Connect' can actually mean 'login', and 'Disconnect' mean 'logout'. It would especially make sense for someone who's building a plugin that only provides a cloud application server.
So, if we wanted to make use of these buttons, it might make sense to synchronize them with the user auth status of the account plugin.

I don't want to do that at this point. I think it would be surprising
behavior to Google logout when they disconnect from a specific 'cloud
server'.
On Tue, Mar 29, 2016 at 1:52 PM, meltsufin<EMAIL_ADDRESS>wrote:

I can see how 'Connect' can actually mean 'login', and 'Disconnect' mean
'logout'. It would especially make sense for someone who's building a
plugin that only provides a cloud application server.
So, if we wanted to make use of these buttons, it might make sense to
synchronize them with the user auth status of the account plugin.
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/gcloud-intellij/issues/552#issuecomment-203025426

How about "disconnect" automatically when login status changes to logged out, but still keep the disconnect action as no-op?


Multiple instances - no support, use queues, decentralised runs or hacks

Hi,
I am trying to develop a system where multiple users can make use of a tagui process automation. Idea was like this, invoke some web service call and then tagui process runs.  Is it possible to open two chrome windows at a time, one for each user and work for each window. How can this be done? Any inputs pls..
Thanks
Soorej

Hi Soorej, that is technically not possible with upstream TagUI architecture. It was designed to prioritise on reliability and stability of execution (for there are production jobs running automation that take more than 10 hours, without issues).
Because of this design choice, there are in-built logic in the design that will restart processes before each run, and that also include Chrome process. Having multiple threads would disrupt the automation actions from another thread.
If you are building a web-service to manage that, this is definitely doable, on the condition that execution are piped into a queue and run consecutively, but not concurrently. If running consecutively is ok for your use case, you can take a look at how I implement TagUI's web-service here - https://github.com/kelaberetiv/TagUI/tree/pre_v6#api
Other ideas that can be explored would be asking users to install Python and then RPA for Python directly on their computers. If concern is automation execution disrupting their work, you can consider hacking the tool to increase execution speed. For eg this RPA Challenge, whre the hack cut down run time from 2 minutes to under 5 seconds - https://github.com/tebelorg/RPA-Python/issues/120#issuecomment-610532082
If concern is still work being disrupted, you can also look at hacking the tool to run in headless mode so that the Chrome web browser does not even appear, and see if that works for your users use scenarios - https://github.com/tebelorg/RPA-Python/issues/109

Closing issue for now until further inputs!


Added support for class weights in LibSvm's classifier

An overloaded variant of the train method of LibSvmClassifier allows for optional weights to handle class balancing.
I had to add support for this for FocusedReading and believe it can be useful for the users in general

@enoriega: Good idea. But please see compilation errors in the Travis log.

@MihaiSurdeanu now it passes all the tests


Add support for unmanagedSourceDirectories over unmanagedSources

At the moment the plugin is based on unmanagedSources which defaults to *.scala and *.java only. However the plugin allows us to specify headers by extension.
I believe the better solution would be to modify the plugin to use the unmanagedSourceDirectories key and find all the matching files in those directories based on the headerSettings map passed in, rather than getting the user to both specify the extension in the map, and then update their includeFilter  in (Compile, unmanagedSources ) to include custom extensions too.
This is useful when dealing with plugins which use different file formats, but still can take copyright headers (such as .proto files and .txt files for i18n)

@hseeberger If I were to make a PR that implemented this, would it be merged in & a new release published? Or should we fork this project? Thanks!


Epic: Improve extension maintainability and experience

Create the APIs used to configure and initialize extensions before the app fully bootstraps itself

[x] https://github.com/rancher/dashboard/issues/8580
[ ] https://github.com/rancher/dashboard/issues/8597
[x] https://github.com/rancher/dashboard/issues/8582

Create reusable and composable layout components:

[ ] https://github.com/rancher/dashboard/issues/8583
[x] #8598
[x] #8599
[x] #8600
[x] #8649

Closing since we will only likely do 3 of these. One is done and two are still open and labeled with area/plugins


Adding bank account to receive invoice payment is confusing

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.66-0
Reproducible in staging?: y
Reproducible in production?: y
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?:
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @neil-marcellini
Slack conversation (hyperlinked to channel name): Expensify-billpay
Action Performed:

Go to staging.new.expensify.com
Create a workspace if needed
Go to More features, scroll down and enable Invoices
Click green plus, Send invoice
[User A] Send an invoice to User B
[User B] Go to invoice chat and pay the invoice as business
[User A] Go to invoice report
[User A] Click Add bank account
[User A] Note that the button leads to Invoices on workspace settings
[User A] Click Add bank account and add a bank account
[User A] Go back to workspace chat

Expected Result:
After adding a bank account for the invoice report the user should be navigated back to it, and the add bank account button should be gone. We could potentially have a system message saying that they added a bank account to receive the invoice payment.
Also, maybe instead of directing the the workspace settings after clicking add bank account, we should instead have an informational page at the end of the flow in the RHP that explains the account has been added as the invoice transfer bank account.
Actual Result:
The add bank account button on the invoice report navigates to settings, then after adding it you land back on the settings page and have to manually navigate back to the invoice report. Additionally, the add bank account button will remain present until you go to invoicing settings, click on the overflow menu of the bank account, and make it the default payment method.
Workaround:
unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Standalone
[ ] Android: HybridApp
[ ] Android: mWeb Chrome
[ ] iOS: Standalone
[ ] iOS: HybridApp
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

https://github.com/user-attachments/assets/ec372a3a-121c-4bdb-9916-7b1627f7e600
View all open jobs on GitHub

This has been labelled "Needs Reproduction". Follow the steps here: https://stackoverflowteams.com/c/expensify/questions/16989

Unassigning since this isn't a quality issue

Agreed. I think this fits into #expense better. I also think this behavior is expected. I've raised a discussion here to chat about it.

Discussion is ongoing

Not much chatter in Slack on this one. I've pinged my pal, @garrettmknight to chime in as the project curator 👍

Ok! It sounds like we all agree that the expected behavior outlined here is the behavior we want (i.e. when a user gets routed to add a bank account from an invoice, they should be brought back to the invoice after adding the bank account).

Good shout, @abzokhattab! It does sound like these issues are pretty similar. @neil-marcellini, what do you think?

bump

Woops yes I think these are the same thing, it was probably a mistake of mine to create a separate issue. I will close this in favor of the original issue and bring any information from here over there.


Feature request - Update to bcrypt for password hashing and timing attacks.

Feature request - implementing the more secure bcrypt for password hashing.
http://php.net/manual/en/function.password-hash.php
Additionally, since the password_verify function doesn't get called unless the user is located in the database, a timing attack can be used to determine whether or not the targeted user exists within the database. This behavior can be observed on the login page by the instant response on invalid users versus users that exist in the database. Unfortunately short of implementing an artificial delay, I'm not sure there's a good way of addressing this. Perhaps someone else can chime in on this.

This feature can be added by making the below modifications. It's worth noting that the algorithmic cost is variable and may need to be altered depending on server specs.
Within createuser.php, line:8 modify the following:
$newpw = password_hash($_POST['password1'], PASSWORD_DEFAULT);
To:
$options = array('cost' => 11);
$newpw = password_hash($_POST['password1'], PASSWORD_BCRYPT, $options);


Migration to the official BTW organization

Hey, you did a cool job making an archival version of BTW! Would you be open to transferring it to the official Better Than Wolves organization? It has existed for quite some years and is home to the community edition of the mod (and most of the addons):
https://github.com/BTW-Community
It seems a bit silly to maintain two separate organizations :)

Sure thing. Uh, to be honest I've never done a repo transfer so I don't know how to do that.

Oh could you please send an invite again? Things got busy over here and I don't really check this account much.


Add basic certificate exchange metrics

This adds basic metrics so we can know how many requests we're making, how many peers we're talking to, how often we're polling, etc.
I still want to add some metrics around how often we're polling when we don't need to, but I also want to rework some of that logic at the same time to reduce the number of unnecessary polls. So I'm punting that out of this round of work.

@masih I've never used open telemetry before so tell me if this looks correct. If not, don't waste a bunch of time pointing out every issue, just give me a general idea and then point me to some docs.

In theory, the recommendation is to use base units but 🤷♂.

@masih @Kubuxu I refactored the metrics to use attributes, units, etc.

Er, apparently I didn't finish this.

Ok, actually done.


Request for adding more machine translation alternatives

Description
Adding these alternatives increase the range of languages supported for machine translation up-to 200, which uses NLLB models and MarianNMT.
Add more machine translation alternatives:

https://translate.glosbe.com (Hosted by Glosbe)
https://translate.wmcloud.org/ (Hosted by Wikimedia Foundation)

Hey,
I am currently seeking for an opportunity to contribute to an open-source project and would be happy to do so here if any help is needed. This issue may align well with my skill and interests.

Hi, @johnnybakucz, can you add the two machine translate APIs to deep-translate? I would appreciate it.

Why isn't AWS supported?

@Prasanta-Hembram Still need this done?

@johnnybakucz are you still working on it. If not then @AustinCGomez will try ??

I could not start yet, go ahead @AustinCGomez

Here, in TranslateYou it is already implemented, I hope it will help: https://github.com/you-apps/TranslateYou/issues/259

Here, in TranslateYou it is already implemented, I hope it will help: you-apps/TranslateYou#259

@Prasanta-Hembram Thanks having more time to work on this now and will see what I can do.

Is it still open? If yes.. then i would like to do it.

Is it still open? If yes.. then i would like to do it.

I was not able to submit a pull request on this feature so you can go ahead and work on it if you wish @bugbreaker18

I would like to work on it. I have already worked on the Glosbe translator and have integrated it, I will start working on the second one.


Cannot read property 'defaultMetadataStorage' of undefined

Im using your package in an Ionic2 / Angular2 project. Since the upgrade to the latest RC version of Ionic, where rollup.js is used as a build system, I get the following error:
error_handler.js:51 TypeError: Cannot read property 'defaultMetadataStorage' of undefined
  at TransformOperationExecutor.getKeys (TransformOperationExecutor.js:219)
  at TransformOperationExecutor.transform (TransformOperationExecutor.js:68)
  at ClassTransformer.plainToClass (ClassTransformer.js:16)
  at plainToClass (index.js:20)

Now, probability us great, that the build process is causing the problem. But if someone has an idea on how to solve this - please do not hesitate ;-)

in my ionic www/build/main.js the TransformOperationExecutor is getting compiled in before the index component (which contains defaultMetadataStorage). TransformOperationExecutor needs the reference from createCommonjsModule(function (module, exports) which is defined after it's needed.

I'm sure this issue is a cyclical dependency the rollup is unable to deal with. Going back to rollup 0.34 you get the error messages; I can go into the build/main.js and remap the TransformOperationExecutor, index_1 references to the global variable set by createCommonjsModule (index_1$5 for me) and everything works great.
rollup: Module commonjs-proxy:/Users/XXXX/node_modules/class-transformer/index.js
may be unable to evaluate without /Users/XXXX/node_modules/class-transformer/index.js,
but is included first due to a cyclical dependency. Consider swapping the import statements
in /Users/XXXX/src/mobile/app/main.dev.ts to ensure correct
ordering

rollup: Module /Users/XXXX/node_modules/class-transformer/TransformOperationExecutor.js
may be unable to evaluate without /Users/XXXX/node_modules/class-transformer/index.js,
but is included first due to a cyclical dependency. Consider swapping the import statements
in /Users/XXXX/src/mobile/app/main.dev.ts to ensure correct
ordering

rollup: Module commonjs-proxy:/Users/XXXX/node_modules/class-transformer/TransformOperationExecutor.js
may be unable to evaluate without /Users/XXXX/node_modules/class-transformer/index.js,
but is included first due to a cyclical dependency. Consider swapping the import statements
in /Users/XXXX/src/mobile/app/main.dev.ts to ensure correct
ordering

rollup: Module /Users/XXXX/node_modules/class-transformer/ClassTransformer.js
may be unable to evaluate without /Users/XXXX/node_modules/class-transformer/index.js,
but is included first due to a cyclical dependency. Consider swapping the import statements
in /Users/XXXX/src/mobile/app/main.dev.ts to ensure correct
ordering

rollup: Module /Users/XXXX/node_modules/class-transformer/decorators.js
may be unable to evaluate without /Users/XXXX/node_modules/class-transformer/index.js,
but is included first due to a cyclical dependency. Consider swapping the import statements
in /Users/XXXX/src/mobile/app/main.dev.ts to ensure correct
ordering

rollup: Module commonjs-proxy:/Users/XXXX/node_modules/class-transformer/ClassTransformer.js
may be unable to evaluate without /Users/XXXX/node_modules/class-transformer/index.js,
but is included first due to a cyclical dependency. Consider swapping the import statements
in /Users/XXXX/src/mobile/app/main.dev.ts to ensure correct
ordering

rollup: Module commonjs-proxy:/Users/XXXX/node_modules/class-transformer/decorators.js
may be unable to evaluate without /Users/XXXX/node_modules/class-transformer/index.js,
but is included first due to a cyclical dependency. Consider swapping the import statements
in /Users/XXXX/src/mobile/app/main.dev.ts to ensure correct
ordering

fixed, added pull request https://github.com/pleerock/class-transformer/pull/25

thanks @strictd released in 0.1.2


Improve descriptions for ProjectSettings' disable_stdout/disable_stderr

This closes https://github.com/godotengine/godot-docs/issues/5022.

Thanks!

Cherry-picked for 3.4.

Cherry-picked for 3.3.3.


Fix Render Info

Fixed and redone the process to obtain render information from a Viewport
Some stats, such as material changes are too difficult to guess on Vulkan, were removed.
Separated visible and shadow stats, which causes confusion.
Texture, buffer and general video memory can be queried now.
Fixed the performance metrics too.

Feedback welcome on how to display the above info better ^

On the screenshot, the number of vertices drawn in shadow is greater than the general vertex count. If this actually is the case, I suggest displaying it as "(+15956446 in shadow)" instead.

The in shadow I read as counts within that other count which makes it weird they are higher numbers. I think this should be in shadowpass.

I would like to request that the current vertex count be renamed to "index count" (if index is confusing, it could be divided by 3 and named "triangle count"):
This disparity is evidenced in the code, where it assigns indices to the "vertices in frame" render info.
p_render_data->render_info->info[RS::VIEWPORT_RENDER_INFO_TYPE_VISIBLE][RS::VIEWPORT_RENDER_INFO_VERTICES_IN_FRAME] += indices;

I would also like to request that, if know, the true vertex count be measured by summing up the vertices of each object rendered. The true vertex count can be helpful for optimizing vertex shader invocation, skinning passes or other per-vertex overhead.

Also wish to support @lyuma 's suggestion (beyond an uptick). This has caused misunderstandings many times.

LOD generation is good
https://user-images.githubusercontent.com/17591539/124349236-d5c7a200-dbf6-11eb-9585-72579f20304f.mp4
shadows are too very expensive draw calls 93 bad optimization need optimization in LightmapGI
https://user-images.githubusercontent.com/17591539/124349272-f2fc7080-dbf6-11eb-9824-f8eb91775c44.mp4
configure global illumination Baked / Dynamic

the result is nothing diminishing the shadow in the draw calls
https://user-images.githubusercontent.com/17591539/124349457-be3ce900-dbf7-11eb-96a3-4e46b6fbbfd3.mp4

GI has nothing to do with shadows @Mobs1024

@reduz OmniLight3D and SpotLight3D shadow are zero if movement shows in shadows
https://user-images.githubusercontent.com/17591539/124351600-fba77380-dc03-11eb-9057-7c3ebfecf5c9.mp4

"Indices" is probably confusing, so maybe I should change the counter to be "primitives"?

@reduz OmniLight3D and SpotLight3D shadow are zero if movement shows in shadows

@Mobs1024 This is a feature, not a bug, if the shadow does not change it's not redrawn hence you see 0.
I have the feeling this information may be too confusing to users, who may think this ia bug, so maybe showing stuff in shadows should be avoided.

I have the feeling this information may be too confusing to users, who may think this ia bug, so maybe showing stuff in shadows should be avoided.

You mean not show shadow statistics at all? If we lump them all together, users will be more confused, and if we don't include shadow statistics, the information becomes much more limited in value as we are only telling half the story. Another option is a beginner / advanced selector in the editor options which controls such options (as we have for the project settings), if the concern is overwhelming beginners.

@lawnjelly to be honest I don't think shadow information is that useful. If you really want to see it, maybe an add-on could show it (I did not remove the API to obtain it). You can already see in the new GPU profiler how much it takes to render shadows anyway so if you have a bottleneck there it will be obvious.

I changed it to look like this:

Thanks!